-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat: support private helm repos #132
Conversation
TylerGillson
commented
Nov 22, 2023
•
edited
Loading
edited
- Add support for installing plugin charts within Helm repos using basic auth
- Fix previously unhandled race conditions when updating the ValidatorConfig object and its status subresource
Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>
Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>
183fae4
to
2527569
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #132 +/- ##
==========================================
+ Coverage 52.60% 55.17% +2.57%
==========================================
Files 9 9
Lines 806 850 +44
==========================================
+ Hits 424 469 +45
+ Misses 356 354 -2
- Partials 26 27 +1
Continue to review full report in Codecov by Sentry.
|
Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>
f550342
to
dfe6058
Compare
Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>
Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>
Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>
a644f90
to
f9476f1
Compare
Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>
Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>
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.
Left a couple questions but LGTM
Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>
…cloud-labs/validator into feat-private-helm
Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>
Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>