-
Notifications
You must be signed in to change notification settings - Fork 65
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
Fix retrieval of minio keys in e2e #98
Conversation
The bitnami/minio chart changed the name of the fields in the Secret that hold the keys. This updates the field names so we can extract them correctly. Signed-off-by: John Strunk <jstrunk@redhat.com>
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JohnStrunk 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 |
/hold This breaks the openshift e2e, but fixes the GH action 😿 |
Signed-off-by: John Strunk <jstrunk@redhat.com>
/unhold I've run manually against openshift w/ this chart version and e2e passes. |
/hold to my surprise, openshift/release#23074 just merged... Let's see what happens. |
/retest |
1 similar comment
/retest |
Codecov Report
@@ Coverage Diff @@
## main #98 +/- ##
=======================================
- Coverage 75.4% 75.0% -0.5%
=======================================
Files 16 16
Lines 2044 2044
=======================================
- Hits 1543 1534 -9
- Misses 397 402 +5
- Partials 104 108 +4
|
/unhold |
/lgtm |
Describe what this PR does
The bitnami/minio chart changed the name of the fields in the Secret
that hold the keys. This updates the field names so we can extract them
correctly.
Is there anything that requires special attention?
Related issues: