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

Jump to bottom

Wait longer for csv to be installed #1554

Closed
wants to merge 1 commit into from
Closed

Wait longer for csv to be installed #1554

wants to merge 1 commit into from

Conversation

rabi
Copy link
Contributor

@rabi rabi commented Apr 24, 2024

This is less than 360secs cofigured earlier.

As a pull request owner and reviewers, I checked that:

  • Appropriate testing is done and actually running

@openshift-ci openshift-ci bot requested review from dsariel and psathyan April 24, 2024 04:47
@openshift-ci openshift-ci bot added the lgtm label Apr 24, 2024
@raukadah
Copy link
Contributor

/approve

@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Apr 24, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: raukadah

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

@raukadah
Copy link
Contributor

Earlier wait_timeout was 360 d65a2af#diff-cbbd4258e91858c67d85e82bd0df1e962cd4a237720fba9d34360c9acd677174L80 it corrects the same.

raukadah pushed a commit to raukadah/openstack-must-gather that referenced this pull request Apr 24, 2024
@chkumar246
Collect all the logs related to openshift-marketplace
f15b43d 
Many times certmanager operator installation fails in CI.
certmanager-operator comes from openshift-marketplace
namespace.

os-must-gather does not collects logs related to
openshift-marketplace namespace.

Capturing the logs from openshift-marketplace will help
us to diagnose certmanager operator related issues.

Depends-On: openstack-k8s-operators/ci-framework#1554

Signed-off-by: Chandan Kumar <[email protected]>
@raukadah raukadah mentioned this pull request Apr 24, 2024
Closed
raukadah added a commit to raukadah/openstack-must-gather that referenced this pull request Apr 24, 2024
@raukadah @chkumar246
Collect all the logs related to openshift-marketplace
5e45f15 
Many times certmanager operator installation fails in CI.
certmanager-operator comes from openshift-marketplace
namespace.

os-must-gather does not collects logs related to
openshift-marketplace namespace.

Capturing the logs from openshift-marketplace will help
us to diagnose certmanager operator related issues.

Depends-On: openstack-k8s-operators/ci-framework#1554

Signed-off-by: Chandan Kumar <[email protected]>
@rabi
Wait longer for csv to be installed
d7e8bbb 
This is less than 360secs cofigured earlier.
@openshift-ci openshift-ci bot removed the lgtm label Apr 24, 2024
@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Apr 24, 2024

New changes are detected. LGTM label has been removed.

@softwarefactory-project-zuul 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://review.rdoproject.org/zuul/buildset/2dd7daa91e1145c09097a3065c2455b7

✔️ openstack-k8s-operators-content-provider SUCCESS in 48m 19s
❌ podified-multinode-edpm-deployment-crc FAILURE in 29m 54s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 49s (non-voting)
❌ cifmw-baremetal-nested-crc FAILURE in 27m 49s
❌ cifmw-molecule-cert_manager FAILURE in 23m 07s

@rabi
Copy link
Contributor Author

rabi commented Apr 24, 2024

This is not helping.

@rabi rabi closed this Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@psathyan psathyan psathyan approved these changes

@bshephar bshephar bshephar approved these changes

@lewisdenny lewisdenny lewisdenny approved these changes

@dsariel dsariel Awaiting requested review from dsariel

Assignees

@lewisdenny lewisdenny

@bshephar bshephar

@psathyan psathyan

Labels
approved
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@rabi @raukadah @lewisdenny @bshephar @psathyan