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

[OSPRH-12761] Adjust post-deploy image collection for OpenStack init controller pod #2631

Merged

Conversation

abays
Copy link
Contributor

@abays abays commented Jan 6, 2025

When gathering the environment's operator images in playersbooks/99_logs.yml, we need to be mindful of the upcoming new OpenStack install paradigm. The OpenStack operator's "init" controller manager does not have two containers like all other controller managers, so we need to exclude it from the task that gathers controller manager images and add a special task to handle it instead.

Copy link

github-actions bot commented Jan 6, 2025

Thanks for the PR! ❤️
I'm marking it as a draft, once your happy with it merging and the PR is passing CI, click the "Ready for review" button below.

@github-actions github-actions bot marked this pull request as draft January 6, 2025 21:05
@abays
Copy link
Contributor Author

abays commented Jan 6, 2025

/retest

@abays abays marked this pull request as ready for review January 6, 2025 23:11
@abays abays changed the title Collect image for OpenStack init controller pod separately [OSPRH-12761] Collect image for OpenStack init controller pod separately Jan 7, 2025
@abays abays force-pushed the osp_init_image branch 2 times, most recently from f54e952 to b01d110 Compare January 7, 2025 13:39
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://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/2ecfea166e53454584ff36ed5b187019

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 52m 51s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 20m 16s
cifmw-crc-podified-edpm-baremetal FAILURE in 1h 40m 11s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 7m 55s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 23s
✔️ build-push-container-cifmw-client SUCCESS in 36m 59s
✔️ cifmw-molecule-env_op_images SUCCESS in 24m 46s

@abays
Copy link
Contributor Author

abays commented Jan 7, 2025

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://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/2ecfea166e53454584ff36ed5b187019

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 52m 51s ✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 20m 16s ❌ cifmw-crc-podified-edpm-baremetal FAILURE in 1h 40m 11s ✔️ noop SUCCESS in 0s ✔️ cifmw-pod-ansible-test SUCCESS in 7m 55s ✔️ cifmw-pod-pre-commit SUCCESS in 7m 23s ✔️ build-push-container-cifmw-client SUCCESS in 36m 59s ✔️ cifmw-molecule-env_op_images SUCCESS in 24m 46s

Error unrelated to PR:

error: timed out waiting for the condition on openstackdataplanenodesets/openstack-edpm-ipam

@abays
Copy link
Contributor Author

abays commented Jan 7, 2025

recheck

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://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/a9e89bd64bc74921a49610106b39e187

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 17m 06s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 19m 38s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 34m 37s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 07s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 05s
✔️ build-push-container-cifmw-client SUCCESS in 37m 13s
cifmw-molecule-env_op_images FAILURE in 28m 05s

@abays
Copy link
Contributor Author

abays commented Jan 7, 2025

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://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/a9e89bd64bc74921a49610106b39e187

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 17m 06s ✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 19m 38s ✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 34m 37s ✔️ noop SUCCESS in 0s ✔️ cifmw-pod-ansible-test SUCCESS in 8m 07s ✔️ cifmw-pod-pre-commit SUCCESS in 7m 05s ✔️ build-push-container-cifmw-client SUCCESS in 37m 13s ❌ cifmw-molecule-env_op_images FAILURE in 28m 05s

Error unrelated to PR:

timeout 300s bash -c 'until $(oc get csv -l operators.coreos.com/openstack-operator.openstack-operators -n openstack-operators | grep -q Succeeded); do sleep 1; done'
    No resources found in openstack-operators namespace.
    ... (ad infinitum)

@abays
Copy link
Contributor Author

abays commented Jan 7, 2025

recheck

@pablintino
Copy link
Collaborator

/approve

Copy link
Contributor

openshift-ci bot commented Jan 9, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: pablintino

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved label Jan 9, 2025
@abays abays changed the title [OSPRH-12761] Collect image for OpenStack init controller pod separately [OSPRH-12761] Adjust post-deploy image collection for OpenStack init controller pod Jan 9, 2025
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://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/ca6315f8d83749c38f6567c32d7115b7

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 46m 37s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 20m 21s
cifmw-crc-podified-edpm-baremetal RETRY_LIMIT in 12m 53s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 7m 52s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 01s
✔️ build-push-container-cifmw-client SUCCESS in 38m 56s
✔️ cifmw-molecule-env_op_images SUCCESS in 24m 46s

@abays
Copy link
Contributor Author

abays commented Jan 9, 2025

recheck

Copy link
Collaborator

@lewisdenny lewisdenny left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Jan 13, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit 4afdeec into openstack-k8s-operators:main Jan 13, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants