-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Support RegistryUnavailable
type
#6842
Support RegistryUnavailable
type
#6842
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: saschagrunert 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 |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #6842 +/- ##
==========================================
- Coverage 49.75% 49.60% -0.16%
==========================================
Files 127 127
Lines 14994 14971 -23
==========================================
- Hits 7461 7426 -35
- Misses 6645 6655 +10
- Partials 888 890 +2 |
/test e2e-gcp-ovn |
/unhold @cri-o/cri-o-maintainers PTAL |
66c7547
to
eda824d
Compare
If we agree on the fix in kubernetes/kubernetes#117612, then we can support the `RegistryUnavailable` error message as well. This has the benefit that we display the information directly to the end user as better error indicator. Signed-off-by: Sascha Grunert <sgrunert@redhat.com>
eda824d
to
bf23f5c
Compare
@haircommander PTAL again |
thank you! /lgtm |
What type of PR is this?
/kind feature
What this PR does / why we need it:
If we agree on the fix in kubernetes/kubernetes#117612, then we can support the
RegistryUnavailable
error message as well. This has the benefit that we display the information directly to the end user as better error indicator./hold
Which issue(s) this PR fixes:
None
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?