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

[build] docker-sonic-mgmt replace USER by whoami #9702

Merged
merged 2 commits into from
Jan 11, 2022

Conversation

liushilongbuaa
Copy link
Contributor

Why I did it

When build docker-sonic-mgmt image, variable USER is not set.

How I did it

Replace USER with whoami

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

A picture of a cute animal (not mandatory but encouraged)

@liushilongbuaa liushilongbuaa requested review from xumia and removed request for lguohan January 7, 2022 10:05
xumia
xumia previously approved these changes Jan 7, 2022
@xumia xumia self-requested a review January 7, 2022 13:55
@liushilongbuaa liushilongbuaa merged commit 94b778c into sonic-net:master Jan 11, 2022
liushilongbuaa added a commit to liushilongbuaa/sonic-buildimage that referenced this pull request Apr 28, 2022
@xumia xumia added the Request for 202111 Branch For PRs being requested for 202111 branch label Apr 29, 2022
xumia pushed a commit to xumia/sonic-buildimage-1 that referenced this pull request Apr 29, 2022
liushilongbuaa added a commit to liushilongbuaa/sonic-buildimage that referenced this pull request May 10, 2022
liushilongbuaa added a commit to liushilongbuaa/sonic-buildimage that referenced this pull request May 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Included in 202111 Branch Request for 202111 Branch For PRs being requested for 202111 branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants