Skip to content

Add delay on executing#1458

Closed
danpawlik wants to merge 2 commits into
openstack-k8s-operators:mainfrom
danpawlik:add-delay-on-executing
Closed

Add delay on executing#1458
danpawlik wants to merge 2 commits into
openstack-k8s-operators:mainfrom
danpawlik:add-delay-on-executing

Conversation

@danpawlik

@danpawlik danpawlik commented May 27, 2025

Copy link
Copy Markdown
Contributor

The test still fails on some infra, probably because it does not have enough time to replace all required things.

danpawlik added 2 commits May 27, 2025 13:53
…erts

The script might make some troubles for OpenShift API to reply
when there is no delay on repeating command.
Also add few seconds delay on checking service certs to avoid errors on
some infrastructures like:

    case.go:398: command "echo \"Checking rotation of non API service certificates...\"
    NAMES..." exceeded 3 sec timeout, context deadline exceeded

Signed-off-by: Daniel Pawlik <dpawlik@redhat.com>
The test still fails on some infra, probably because it does not have
enough time to replace all required things.

Signed-off-by: Daniel Pawlik <dpawlik@redhat.com>
@openshift-ci

openshift-ci Bot commented May 27, 2025

Copy link
Copy Markdown
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci

openshift-ci Bot commented May 27, 2025

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: danpawlik
Once this PR has been reviewed and has the lgtm label, please assign jpodivin for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

@danpawlik danpawlik closed this May 28, 2025
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.

1 participant