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
Use containerized kustomize #1395
Conversation
Per our conversation, we should Dockerize all invocations so the build file is using a consistent version of Kustomize across the board |
809dd5b
to
566964d
Compare
Codecov Report
@@ Coverage Diff @@
## master #1395 +/- ##
==========================================
- Coverage 50.06% 50.04% -0.03%
==========================================
Files 73 75 +2
Lines 5019 5061 +42
==========================================
+ Hits 2513 2533 +20
- Misses 2155 2178 +23
+ Partials 351 350 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
566964d
to
bd3c80b
Compare
590349d
to
4829dcc
Compare
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.
LGTM
4829dcc
to
37e6b43
Compare
37e6b43
to
a31cb92
Compare
Rather than relying on the developer's installed kustomize, use dockerized kustomize so behavior isn't dependnet on setup. Also update kustomize version to v4.1.3 This commit does not change other tools to be dockerized, for example kubectl. Signed-off-by: Will Beason <willbeason@google.com>
a31cb92
to
9bba778
Compare
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.
LGTM
Co-authored-by: Sertaç Özercan <852750+sozercan@users.noreply.github.com> Signed-off-by: juliankatz <juliankatz@google.com>
The flag is
--load-restrictor
, not--load_restrictor
Without change,
make deploy-mutation
shows:With the change,
make deploy-mutation
succeeds.I'm using kustomize v4.1.3:
Signed-off-by: Will Beason willbeason@google.com