-
Notifications
You must be signed in to change notification settings - Fork 50
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add test-migrate-ceph target #520
Conversation
d949fec
to
6fca179
Compare
recheck |
31d1537
to
cf3e268
Compare
ansible.builtin.shell: | ||
"os-net-config -c {{ os_net_conf_path }}" | ||
vars: | ||
os_net_conf_path: "/etc/os-net-config/tripleo_config.yaml" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
controller nodes have tripleo_config
instead of just config.yaml
that we can find in the EDPM nodes, hence we override the default value.
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/7278f4d2d0624fb0a5759148559bcf3d ✔️ data-plane-adoption-osp-17-to-extracted-crc SUCCESS in 2h 40m 02s |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as discussed yesterday this is safe to merge/will not affect any ci or any of the existing adoption tests/roles.
adding the new target will make it a bit easier for Francesco to continue working on this and eventually we can wire up into the 'with ceph' job
Last CI run is transient issue and anyway none of this is exercised in CI we don't need to run again before merge
This patch introduces the test-ceph-migration target, required to conditionally trigger the ceph-migration from rdo-jobs. In addition, it provides some changes related to the upstream CI integration. Signed-off-by: Francesco Pantano <fpantano@redhat.com>
This patch introduces the
test-ceph-migration
target, required to conditionally trigger the ceph-migration fromrdo-jobs
.