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.

Sign up for GitHub

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 data-plane-adoption job to the edpm-ansible zuul github-check layout #662

Merged
merged 1 commit into from
Jun 13, 2024
Merged

Add data-plane-adoption job to the edpm-ansible zuul github-check layout #662

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 34 additions & 0 deletions zuul.d/projects.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,3 +16,37 @@
- edpm-ansible-molecule-edpm_ovn_bgp_agent
- edpm-ansible-molecule-edpm_ovs
- edpm-ansible-molecule-edpm_tripleo_cleanup
- openstack-k8s-operators-content-provider
- cifmw-data-plane-adoption-osp-17-to-extracted-crc:
dependencies:
- openstack-k8s-operators-content-provider
# files based on https://github.com/marios/data-plane-adoption/blob/ec0f395a8321dc4c3ced71b777102ad7dc9de1b2/tests/roles/dataplane_adoption/tasks/main.yaml#L256
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i updated the files: list based on yesterday's conversation.

As far as I can see these are the specific plays and roles we are consuming in this job cc @slagle @jpodivin @jistr @rebtoor @cescgina

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Most of those I agree with. Not the plugins however. Those will be sufficiently tested indirectly through molecule tests for all roles. The molecule tests on running through github actions are already doing their part, triggering by plugin changes, I don't know about those running on Zuul.

On the other hand, you should include pre-adoption-validation role in the list.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ack fair enough I will update and remove plugins

and I'll add the pre-adoption-validation

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome.

files:
- ^playbooks/bootstrap.yml
- ^playbooks/download_cache.yml
- ^playbooks/configure_network.yml
- ^playbooks/validate_network.yml
- ^playbooks/install_os.yml
- ^playbooks/configure_os.yml
- ^playbooks/ssh_known_hosts.yml
- ^playbooks/run_os.yml
- ^playbooks/install_certs.yml
- ^playbooks/libvirt.yml
- ^playbooks/ovn.yml
- ^playbooks/neutron_metadata.yml
- ^roles/edpm_bootstrap/*
- ^roles/edpm_download_cache/*
- ^roles/edpm_network_config/*
- ^roles/edpm_nodes_validation/*
- ^roles/edpm_podman/*
- ^roles/edpm_sshd/*
- ^roles/edpm_timezone/*
- ^roles/edpm_container_manage/*
- ^roles/edpm_nftables/*
- ^roles/edpm_logrotate_crond/*
- ^roles/edpm_ssh_known_hosts/*
- ^roles/edpm_install_certs/*
- ^roles/edpm_libvirt/*
- ^roles/edpm_ovn/*
- ^roles/edpm_neutron_metadata/*
- ^roles/edpm_pre_adoption_validation/*
Loading