-
Notifications
You must be signed in to change notification settings - Fork 80
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
Bug 2089918: adds fix for --skip-missing flag processing #484
Bug 2089918: adds fix for --skip-missing flag processing #484
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jpower432 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 |
632a9e0
to
0e710d7
Compare
@jpower432: This pull request references Bugzilla bug 2089918, which is invalid:
Comment 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. |
/bugzilla refresh |
@dinhxuanvu: This pull request references Bugzilla bug 2089918, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker. 3 validation(s) were run on this bug
Requesting review from QA contact: 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. |
/bugzilla refresh |
@dinhxuanvu: This pull request references Bugzilla bug 2089918, which is valid. 3 validation(s) were run on this bug
Requesting review from QA contact: 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. |
@jpower432: This pull request references Bugzilla bug 2089918, which is valid. 3 validation(s) were run on this bug
Requesting review from QA contact: 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. |
1 similar comment
@jpower432: This pull request references Bugzilla bug 2089918, which is valid. 3 validation(s) were run on this bug
Requesting review from QA contact: 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. |
8f4f0b6
to
1f9a835
Compare
1f9a835
to
ee4bd15
Compare
ee4bd15
to
7318359
Compare
bumps oc import version to commit d91e0087a8a6120cbbea65397d25267a8de1cf42 bumps operator-registry import to v1.23.0 bumos helm import version to v3.9.0 Signed-off-by: Jennifer Power <barnabei.jennifer@gmail.com>
The version of containerd used in helm 3.9.0 is not compatibile with version 0.24.0 of k8s/component-base. To allow the upgrade of other components, the render method has been refactored to removed the uneccessary import of kube packages from helm. Signed-off-by: Jennifer Power <barnabei.jennifer@gmail.com>
Signed-off-by: Jennifer Power <barnabei.jennifer@gmail.com>
Signed-off-by: Jennifer Power <barnabei.jennifer@gmail.com>
Fixes openshift#476 Signed-off-by: Jennifer Power <barnabei.jennifer@gmail.com>
…helm.go Signed-off-by: Jennifer Power <barnabei.jennifer@gmail.com>
7318359
to
9c053f1
Compare
/test integration |
/test integration |
@jpower432: all tests passed! 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
@jpower432: All pull requests linked via external trackers have merged: Bugzilla bug 2089918 has been moved to the MODIFIED state. 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. |
Description
This PR updates the
oc
import to get the fix forskip missing
inoc image mirror
to address images with digest.To maintain compatibility between imports the following two imports were updated:
Operator Registry 1.22.1 => 1.23.0
Helm 3.7.2 => 3.8.1
Due to a compatibility issue with the version of
containerd
in Helm versions greater than 3.8.1 andk8s.io/component-base/traces
v0.24.0, the render method inhelm.go
needed to be refactored to remove the Helmaction
package.Fixes #476
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
How to Test
Run
oc-mirror --config imageset-config.yaml file://out
Checklist: