Skip to content
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

Merged
merged 1 commit into from
Jul 8, 2024

Conversation

fmount
Copy link
Contributor

@fmount fmount commented Jul 3, 2024

This patch introduces the test-ceph-migration target, required to conditionally trigger the ceph-migration from rdo-jobs.

@fmount fmount requested review from jistr, cescgina and marios July 3, 2024 17:32
@fmount fmount force-pushed the ceph_migrate branch 3 times, most recently from d949fec to 6fca179 Compare July 3, 2024 21:20
@fmount fmount changed the title Add test-migrate-ceph target Draft: Add test-migrate-ceph target Jul 3, 2024
@fmount
Copy link
Contributor Author

fmount commented Jul 4, 2024

recheck

@fmount fmount changed the title Draft: Add test-migrate-ceph target Add test-migrate-ceph target Jul 4, 2024
@fmount fmount force-pushed the ceph_migrate branch 4 times, most recently from 31d1537 to cf3e268 Compare July 4, 2024 16:16
ansible.builtin.shell:
"os-net-config -c {{ os_net_conf_path }}"
vars:
os_net_conf_path: "/etc/os-net-config/tripleo_config.yaml"
Copy link
Contributor Author

@fmount fmount Jul 4, 2024

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.

Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://review.rdoproject.org/zuul/buildset/7278f4d2d0624fb0a5759148559bcf3d

✔️ data-plane-adoption-osp-17-to-extracted-crc SUCCESS in 2h 40m 02s
data-plane-adoption-osp-17-to-extracted-crc-minimal-no-ceph FAILURE in 1h 47m 17s

Copy link
Contributor

@marios marios left a 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>
@jistr jistr merged commit 0c46289 into openstack-k8s-operators:main Jul 8, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants