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

Do not ignore failures in edpm_deploy_instance #462

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

karelyatin
Copy link
Contributor

If CI or other bits need to ignore failures, that can be done outside of the Makefile.

Also update edpm-deploy-instance.sh to run ping
command only if it exists.

@openshift-ci openshift-ci bot requested review from abays and slagle August 7, 2023 05:39
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Aug 7, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: karelyatin

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 Aug 7, 2023
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Aug 7, 2023

@karelyatin: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/install-yamls-deploy 0bd5369 link false /test install-yamls-deploy

Full PR test history. Your PR dashboard.

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/test-infra repository. I understand the commands that are listed here.

@softwarefactory-project-zuul
Copy link

@softwarefactory-project-zuul
Copy link

@karelyatin
Copy link
Contributor Author

check-rdo

@softwarefactory-project-zuul
Copy link

@karelyatin
Copy link
Contributor Author

check-rdo tls timeout

@softwarefactory-project-zuul
Copy link

@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/996fccb58ebb4d29bf9bb2222d28b6ec

✔️ install-yamls-content-provider SUCCESS in 1h 59m 18s
install-yamls-crc-podified-edpm-baremetal FAILURE in 27m 21s
install-yamls-crc-podified-galera-deployment FAILURE in 1h 43m 55s
install-yamls-crc-podified-edpm-deployment FAILURE in 47m 30s

@karelyatin
Copy link
Contributor Author

check-rdo

@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/e974212c22e54da38e66f954f8c772cb

install-yamls-content-provider FAILURE in 10m 33s
⚠️ install-yamls-crc-podified-edpm-baremetal SKIPPED Skipped due to failed job install-yamls-content-provider
⚠️ install-yamls-crc-podified-galera-deployment SKIPPED Skipped due to failed job install-yamls-content-provider
⚠️ install-yamls-crc-podified-edpm-deployment SKIPPED Skipped due to failed job install-yamls-content-provider

@karelyatin
Copy link
Contributor Author

check-rdo error: RPC failed; curl 16 Error in the HTTP2 framing layer git ls-remote

@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/912aff912bfc4b70beb06f3af539b042

✔️ install-yamls-content-provider SUCCESS in 1h 15m 30s
install-yamls-crc-podified-edpm-baremetal FAILURE in 44m 55s
✔️ install-yamls-crc-podified-galera-deployment SUCCESS in 58m 51s
install-yamls-crc-podified-edpm-deployment FAILURE in 44m 22s

If CI or other bits need to ignore failures, that can
be done outside of the Makefile.

Also update edpm-deploy-instance.sh to run ping
command only if it exists.
@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/f1821da65f1a4ad1ae5ba47886fe5bd6

✔️ install-yamls-content-provider SUCCESS in 22m 28s
install-yamls-crc-podified-edpm-deployment RETRY_LIMIT in 9m 00s

@karelyatin
Copy link
Contributor Author

check-rdo retry

@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/b0cbe3cdd866434ead2a0261e6ba192a

✔️ install-yamls-content-provider SUCCESS in 1h 10m 21s
install-yamls-crc-podified-edpm-deployment FAILURE in 45m 12s

@karelyatin
Copy link
Contributor Author

check-rdo unauthorized

@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/72c7f1a7b50549d7adb6365b2bfdac33

✔️ install-yamls-content-provider SUCCESS in 1h 04m 01s
install-yamls-crc-podified-edpm-deployment FAILURE in 51m 30s

@openshift-merge-robot
Copy link
Contributor

PR needs rebase.

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/test-infra 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.

3 participants