fix: Changed runAsUser uid #359
Conversation
|
With the latest changes to runatlantis/atlantis#4304 this change is no longer needed |
| @@ -260,7 +260,7 @@ statefulSet: | |||
| labels: {} | |||
| securityContext: | |||
| fsGroup: 1000 | |||
There was a problem hiding this comment.
We need to discuss if this is the right thing to do.
We made changes to Alpine and Debian to match the UID recently and we need to keep backwards compatibility
this could affect users that are not enforcing the default.
There was a problem hiding this comment.
This is going to be closed. There’s another PR fixes the uid to be 100 on both -> runatlantis/atlantis#4304. Please let us know what you think.
There was a problem hiding this comment.
Prior to merging #4304, is this fix #359 even needed when the values can be overridden for runAsUser?
There was a problem hiding this comment.
If runatlantis/atlantis#4304 gets merged in its current state this won't be necessary anymore no.
what
This changes the default runAsUser uid to 1000 instead of 100
why
Together with runatlantis/atlantis#4304 this will fix #305. This should only be merged when runatlantis/atlantis#4304 has been released.
tests
references
Closes #305