-
Notifications
You must be signed in to change notification settings - Fork 298
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
NO-JIRA: Remove CLI inspection of release image #3548
NO-JIRA: Remove CLI inspection of release image #3548
Conversation
@davidvossel: This pull request explicitly references no jira issue. 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 openshift-eng/jira-lifecycle-plugin repository. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: davidvossel 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 |
Removing
|
See #3844 |
b3c7155
to
f2d5dcd
Compare
f2d5dcd
to
0950d63
Compare
0950d63
to
d39526f
Compare
/retest-required |
/lgtm |
d39526f
to
2ab3ebf
Compare
/lgtm |
/retest "Red Hat Konflux / hypershift-operator-main-on-pull-request" |
@davidvossel: The
The following commands are available to trigger optional jobs:
Use
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. |
/test "Red Hat Konflux / hypershift-operator-main-on-pull-request" |
@davidvossel: The specified target(s) for
The following commands are available to trigger optional jobs:
Use
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. |
/retest |
Signed-off-by: David Vossel <davidvossel@gmail.com>
2ab3ebf
to
cf27872
Compare
/lgtm |
1 similar comment
/lgtm |
/test e2e-aws e2e-azure |
/retest |
1 similar comment
/retest |
@davidvossel: 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-sigs/prow repository. I understand the commands that are listed here. |
/cherry-pick 4.15 |
@LiangquanLi930: cannot checkout 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-sigs/prow repository. |
/cherry-pick release-4.15 |
/cherry-pick release-4.14 |
@LiangquanLi930: #3548 failed to apply on top of branch "release-4.15":
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-sigs/prow repository. |
@LiangquanLi930: #3548 failed to apply on top of branch "release-4.14":
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-sigs/prow repository. |
Previously, the hcp client tool would dynamically inspect the release image to determine what the version was in use, and use that information to default the network type.
That inspection of the release image causes issues in the disconnected use case. It's possible that the machine someone is executing the client on can't directly access the image registry. In this situation, the client fails and prevents a user from being able to create a cluster.
Moving forward, we need the client to only require access to the api server of the mgmt cluster.