Skip to content

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.

@github-actions
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
@softwarefactory-project-zuul
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

@softwarefactory-project-zuul
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
Contributor

/approve

@openshift-ci
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

Details 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
@softwarefactory-project-zuul
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

@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
@hgoelredhat
Copy link

/cherry-pick 18.0-fr1

@openshift-cherrypick-robot

@hgoelredhat: only openstack-k8s-operators org members may request cherry picks. If you are already part of the org, make sure to change your membership to public. Otherwise you can still do the cherry-pick manually.

Details

In response to this:

/cherry-pick 18.0-fr1

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@sdatko
Copy link
Contributor

sdatko commented Mar 5, 2025

/cherry-pick 18.0-fr1

@openshift-cherrypick-robot

@sdatko: new pull request created: #2774

Details

In response to this:

/cherry-pick 18.0-fr1

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

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.

6 participants