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
Fixes in control plane services CLI flags #6925
Fixes in control plane services CLI flags #6925
Conversation
- Removed flags that are deprecated and have no effect - Remove `tokenAudience` flag from Helm chart since it's now unused - Fixed description of some flags - Use constants for default trust anchors file path Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6925 +/- ##
==========================================
- Coverage 65.10% 65.09% -0.02%
==========================================
Files 221 231 +10
Lines 20799 21049 +250
==========================================
+ Hits 13541 13701 +160
- Misses 6120 6207 +87
- Partials 1138 1141 +3
☔ View full report in Codecov by Sentry. |
…l-plane-cli Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Close this in favor of #7029 |
Per @mukundansundar this should be merged first |
@mukundansundar @daixiang0 is this ready to be merged? |
…l-plane-cli Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
tokenAudience
flag from Helm chart since it's now unused