🐛 More needed for podman workaround in kind-load#788
Closed
bentito wants to merge 13 commits intooperator-framework:mainfrom
Closed
🐛 More needed for podman workaround in kind-load#788bentito wants to merge 13 commits intooperator-framework:mainfrom
bentito wants to merge 13 commits intooperator-framework:mainfrom
Conversation
* Fix suite_test.go Signed-off-by: Todd Short <tshort@redhat.com> * Copy over some rukpak code and replace BundleDeployment Replace BundleDeployment in the Unpack APIs with a combination of BundleSource and ClusterExtension. It builds... Signed-off-by: Todd Short <tshort@redhat.com> * fixup! Copy over some rukpak code and replace BundleDeployment --------- Signed-off-by: Todd Short <tshort@redhat.com>
Signed-off-by: Varsha Prasad Narsing <varshaprasad96@gmail.com> Co-authored-by: varshaprasad96@gmail.com <vnarsing@vnarsing-mac.hsd1.ca.comcast.net>
Signed-off-by: Todd Short <tshort@redhat.com>
* Install the helm chart Signed-off-by: Todd Short <tshort@redhat.com> * fixup! Install the helm chart Signed-off-by: Todd Short <tshort@redhat.com> --------- Signed-off-by: Todd Short <tshort@redhat.com>
…rk#763) Co-authored-by: varshaprasad96@gmail.com <vnarsing@vnarsing-mac.hsd1.ca.comcast.net>
* Update owner keys Signed-off-by: Todd Short <tshort@redhat.com> * fixup! Update owner keys Signed-off-by: Todd Short <tshort@redhat.com> --------- Signed-off-by: Todd Short <tshort@redhat.com>
Signed-off-by: Todd Short <tshort@redhat.com>
…ramework#776) Co-authored-by: varshaprasad96@gmail.com <vnarsing@vnarsing-mac.hsd1.ca.comcast.net>
✅ Deploy Preview for olmv1 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Contributor
|
Oh nice, I was just about to put up something for this. |
Contributor
|
note: the changes here and in #790 were required for me to successfully run |
…r-framework#789) * Add relevant RBAC to enable controller to watch resources * Debugging iteration - one * ClusterExtension installing the operator - working --------- Co-authored-by: varshaprasad96@gmail.com <vnarsing@vnarsing-mac.hsd1.ca.comcast.net>
Signed-off-by: Todd Short <tshort@redhat.com>
grokspawn
reviewed
Apr 26, 2024
| e2e-coverage: | ||
| COVERAGE_OUTPUT=./e2e-cover.out ./hack/e2e-coverage.sh | ||
|
|
||
| .PHONY: kind-load |
Contributor
There was a problem hiding this comment.
Why remove the .PHONY target annotation here? We don't generate a versioned artifact as a result of this make target.
Contributor
Author
There was a problem hiding this comment.
ugh, I just blew up this PR with an ill-advised force push. For a small change like this I'm going to start over.
bentito
force-pushed
the
kind-podman-fix
branch
from
9033d64 to
41751c7
Compare
openshift-merge-robot
added
the
needs-rebase
|
PR needs rebase. DetailsInstructions 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. |
Contributor
Author
|
I'm going to try this again from clean upstream:main new PR is: #794 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
The commit adding the podman workaround for the kind-load target seems to need a bit more work:
It needs to have the tag name set before image-archive is saved
Reviewer Checklist