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
remove bundle.Dockerfile #227
remove bundle.Dockerfile #227
Conversation
@cpmeadors shouldn't |
/test 4.9-ci-index-sandboxed-containers-operator-bundle |
I restarted the 4.9 bundle build, but I think it is obsolete and we should just waive it if it doesn't pass, which it will since I would need to change it to point to the custom bundle dockerfile as well. I plan on removing it when I update the master file to 4.12 configuration. |
Yes, that sounds reasonable to me. |
This seems to be part of a bigger problem which is that the CI as a whole appears effectively unmaintained. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks @cpmeadors !
/override 4.9-ci-index-sandboxed-containers-operator-bundle |
@cpmeadors: cpmeadors unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file, and the following github teams:openshift: openshift-release-oversight. In response to this:
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. |
/override 4.9-ci-index-sandboxed-containers-operator-bundle |
@gkurz: /override requires failed status contexts, check run or a prowjob name to operate on.
Only the following failed contexts/checkruns were expected:
If you are trying to override a checkrun that has a space in it, you must put a double quote on the context. In response to this:
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. |
/override ci/prow/4.9-ci-index-sandboxed-containers-operator-bundle |
@gkurz: Overrode contexts on behalf of gkurz: ci/prow/4.9-ci-index-sandboxed-containers-operator-bundle In response to this:
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. |
@cpmeadors: The following tests 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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Thanks @cpmeadors
Nothing expects this auto-generated file to exist so we don't need to track it
Closes: KATA-1835