-
Notifications
You must be signed in to change notification settings - Fork 77
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
[Bugfix] remove sudo from kubectl/install make command and small refactor around os/arch #2037
Conversation
[CHATOPS:HELP] ChatOps commands.
|
c465e22
to
eda6ddc
Compare
Deploying with Cloudflare Pages
|
eda6ddc
to
489a010
Compare
489a010
to
a5b09de
Compare
a5b09de
to
924bb17
Compare
Signed-off-by: kpango <[email protected]>
924bb17
to
abab3f1
Compare
[FOSSA] The scan result will be available at https://app.fossa.com/projects/custom%2B21465%2Fvald/refs/branch/bugfix%2Ftools%2Fremove-sudo-from-kubectl-install/80693e6fd0b6f30a0ea7954938680a2d6802a993 |
Profile Report
|
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.
Thanks for the catch. Didn't realize it was failing.
I think we should stop DeepSource: Test coverage
for now so that we realize some tests are actually failing.
To disabling the DeepSource-TestCoverage, we have to |
👍 |
Description:
Fixed CI Container build failure due to use of sudo in kubectl/install in the Makefile.
And refactored redundant OS/Arch switches.
Related Issue:
Versions:
Checklist:
Special notes for your reviewer: