-
Notifications
You must be signed in to change notification settings - Fork 101
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
base: main
Are you sure you want to change the base?
Do not ignore failures in edpm_deploy_instance #462
Conversation
[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 |
@karelyatin: The following test failed, say
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. |
Build failed (check pipeline). For information on how to proceed, see https://review.rdoproject.org/zuul/buildset/bac3c11ac78841d6a5b5e262906fae70 ✔️ install-yamls-content-provider SUCCESS in 1h 32m 48s |
0bd5369
to
1ff90f6
Compare
Build failed (check pipeline). For information on how to proceed, see https://review.rdoproject.org/zuul/buildset/5a2d591fc4b04941a97aee6a9afe9cd1 ✔️ install-yamls-content-provider SUCCESS in 1h 02m 14s |
check-rdo |
Build failed (check pipeline). For information on how to proceed, see https://review.rdoproject.org/zuul/buildset/c75c90360d1d478d87af38b6205afc91 ✔️ install-yamls-content-provider SUCCESS in 1h 14m 56s |
check-rdo tls timeout |
Build failed (check pipeline). For information on how to proceed, see https://review.rdoproject.org/zuul/buildset/88976c255a6a4df69c4e97ad44807e9a ✔️ install-yamls-content-provider SUCCESS in 1h 47m 56s |
1ff90f6
to
e73018f
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/996fccb58ebb4d29bf9bb2222d28b6ec ✔️ install-yamls-content-provider SUCCESS in 1h 59m 18s |
check-rdo |
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/e974212c22e54da38e66f954f8c772cb ❌ install-yamls-content-provider FAILURE in 10m 33s |
check-rdo error: RPC failed; curl 16 Error in the HTTP2 framing layer git ls-remote |
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/912aff912bfc4b70beb06f3af539b042 ✔️ install-yamls-content-provider SUCCESS in 1h 15m 30s |
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.
e73018f
to
2870f4d
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/f1821da65f1a4ad1ae5ba47886fe5bd6 ✔️ install-yamls-content-provider SUCCESS in 22m 28s |
check-rdo retry |
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/b0cbe3cdd866434ead2a0261e6ba192a ✔️ install-yamls-content-provider SUCCESS in 1h 10m 21s |
check-rdo unauthorized |
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/72c7f1a7b50549d7adb6365b2bfdac33 ✔️ install-yamls-content-provider SUCCESS in 1h 04m 01s |
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. |
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.