Skip to content
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

add serviceAccount and runAsUser to kaniko build (resolves #3267) #3965

Merged

Conversation

DanielSel
Copy link
Contributor

Fixes: #3267

Description
Add configuration options for serviceAccount and runAsUser.
This allows building in clusters with PodSecurityPolicies (bound to serviceAccount) and OKD/OpenShift 3.11 (requires runAsUser to be set to avoid auto-apply of 'restricted' scc)

@tejal29
Copy link
Member

tejal29 commented Apr 23, 2020

Thanks @DanielSel for your PR.
Looks like you are updating a released config. Can you please follow the steps mentioned here, https://github.com/GoogleContainerTools/skaffold/blob/master/DEVELOPMENT.md#making-a-config-change

Thank you so much! Looking forward to get this in!

@codecov
Copy link

codecov bot commented Apr 27, 2020

Codecov Report

Merging #3965 into master will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted Files Coverage Δ
pkg/skaffold/schema/latest/config.go 100.00% <ø> (ø)
pkg/skaffold/build/cluster/pod.go 87.35% <100.00%> (+0.45%) ⬆️
...affold/kubernetes/portforward/kubectl_forwarder.go 65.85% <0.00%> (-2.44%) ⬇️

@DanielSel DanielSel mentioned this pull request Apr 27, 2020
@DanielSel
Copy link
Contributor Author

@tejal29 I submitted a pr for the new version: #4029

After this is merged, I will update and rebase this PR (and #3974)

@tejal29 tejal29 added the kokoro:run runs the kokoro jobs on a PR label Apr 29, 2020
@kokoro-team kokoro-team removed the kokoro:run runs the kokoro jobs on a PR label Apr 29, 2020
@tejal29
Copy link
Member

tejal29 commented Apr 29, 2020

LGTM! will get this in today!

@dgageot dgageot merged commit a4852a3 into GoogleContainerTools:master May 5, 2020
@DanielSel DanielSel deleted the feature/security-context branch September 18, 2020 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants