Skip to content

fix(dashmate): dashmate helper is running under root user#895

Merged
shumkov merged 1 commit into
v0.24-devfrom
fix/wrong-helper-user
Apr 7, 2023
Merged

fix(dashmate): dashmate helper is running under root user#895
shumkov merged 1 commit into
v0.24-devfrom
fix/wrong-helper-user

Conversation

@shumkov
Copy link
Copy Markdown
Collaborator

@shumkov shumkov commented Apr 7, 2023

Issue being fixed or feature implemented

Dashmate helper is running with user root which breaks access rights to dashmate home dir

What was done?

  • Fix entrypoint to run helper using dashmate user

How Has This Been Tested?

None

Breaking Changes

None

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

@shumkov shumkov added this to the v0.24.0 milestone Apr 7, 2023
@shumkov shumkov requested a review from antouhou as a code owner April 7, 2023 18:31
Copy link
Copy Markdown
Collaborator

@pshenmic pshenmic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@shumkov shumkov merged commit c668c54 into v0.24-dev Apr 7, 2023
@shumkov shumkov deleted the fix/wrong-helper-user branch April 7, 2023 18:32
@markin-io markin-io mentioned this pull request Apr 26, 2023
5 tasks
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