Skip to content

Run all containers as an unprivileged user (resolves #528)#546

Merged
jessesuen merged 1 commit intoargoproj:masterfrom
jessesuen:non-root
Sep 4, 2018
Merged

Run all containers as an unprivileged user (resolves #528)#546
jessesuen merged 1 commit intoargoproj:masterfrom
jessesuen:non-root

Conversation

@jessesuen
Copy link
Member

No description provided.

@jessesuen jessesuen merged commit e3adb30 into argoproj:master Sep 4, 2018
leoluz pushed a commit to leoluz/argo-cd that referenced this pull request Mar 13, 2025
…ly (argoproj#546)

* fix: use server side dry run in case if it is server side apply

Signed-off-by: pashakostohrys <pavel@codefresh.io>

* fix: use server side dry run in case if it is server side apply

Signed-off-by: pashakostohrys <pavel@codefresh.io>

---------

Signed-off-by: pashakostohrys <pavel@codefresh.io>
leoluz added a commit to leoluz/argo-cd that referenced this pull request Mar 13, 2025
…idation even with server side apply (argoproj#564)

* Revert "feat: retry with client side dry run if server one was failed (argoproj#548)"

This reverts commit c0c2dd1.

Signed-off-by: Anand Francis Joseph <anjoseph@redhat.com>

* Revert "fix(server): use server side dry run in case if it is server side apply (argoproj#546)"

This reverts commit 4a5648e.

Signed-off-by: Anand Francis Joseph <anjoseph@redhat.com>

* Fixed the logic to disable server side apply if it is a dry run

Signed-off-by: Anand Francis Joseph <anjoseph@redhat.com>

* Added more values in the log message for better debugging

Signed-off-by: Anand Francis Joseph <anjoseph@redhat.com>

* Fixed compilation error

Signed-off-by: Anand Francis Joseph <anjoseph@redhat.com>

* Written an inline fn to get string value of dry-run strategy

Signed-off-by: Anand Francis Joseph <anjoseph@redhat.com>

* Added comment as requested with reference to the issue number

Signed-off-by: Anand Francis Joseph <anjoseph@redhat.com>

---------

Signed-off-by: Anand Francis Joseph <anjoseph@redhat.com>
Co-authored-by: Leonardo Luz Almeida <leoluz@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants