Skip to content

Conversation

@cognifloyd
Copy link
Member

Backport of #5186

  • make add-itest-user scripts more verbose
  • use sudo -E to preserve env vars

Github Actions has this in their Default: env_reset in their sudoers config file. This means that HOME and other vars are set to match the sudo target user. Using sudo -E preserves the environment allowing ~ to resolve correctly.

I'm not sure why we didn't hit this earlier. Maybe Github changed their sudoers config? In any case, this should fix it.

@pull-request-size pull-request-size bot added the size/S PR that changes 10-29 lines. Very easy to review. label Mar 11, 2021
@cognifloyd cognifloyd added this to the 3.4.1 milestone Mar 11, 2021
Copy link
Contributor

@m4dcoder m4dcoder left a comment

Choose a reason for hiding this comment

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

LGTM

@m4dcoder m4dcoder merged commit cc21f8a into StackStorm:v3.4 Mar 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infrastructure: ci/cd size/S PR that changes 10-29 lines. Very easy to review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants