Skip to content

[build]: change user name to lower case when used in sonic-slave tag#6319

Merged
lguohan merged 1 commit intosonic-net:masterfrom
lguohan:user
Dec 30, 2020
Merged

[build]: change user name to lower case when used in sonic-slave tag#6319
lguohan merged 1 commit intosonic-net:masterfrom
lguohan:user

Conversation

@lguohan
Copy link
Collaborator

@lguohan lguohan commented Dec 29, 2020

- Why I did it
sonic-slave tag only allows all lower case. In case the user
name is mixed case, we need to change user name to all lower case.

https://stackoverflow.com/questions/36858837/lowercase-images-names-in-docker

- How I did it
change user name to all lower case

- How to verify it
test username with mixed case.

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

  • 201811
  • 201911
  • 202006

- Description for the changelog

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

sonic-slave tag only allows all lower case. In case the user
name is mixed case, we need to change user name to all lower case.

Signed-off-by: Guohan Lu <lguohan@gmail.com>
@lguohan lguohan requested a review from xumia December 29, 2020 23:15
@lguohan lguohan requested a review from qiluo-msft December 30, 2020 01:35
@lguohan lguohan merged commit de4a3c8 into sonic-net:master Dec 30, 2020
@lguohan lguohan deleted the user branch December 30, 2020 12:58
lguohan added a commit that referenced this pull request Jan 19, 2021
…6319)

sonic-slave tag only allows all lower case. In case the user
name is mixed case, we need to change user name to all lower case.

Signed-off-by: Guohan Lu <lguohan@gmail.com>
lguohan added a commit to lguohan/sonic-buildimage that referenced this pull request Jan 20, 2021
…onic-net#6319)

sonic-slave tag only allows all lower case. In case the user
name is mixed case, we need to change user name to all lower case.

Signed-off-by: Guohan Lu <lguohan@gmail.com>
lguohan added a commit that referenced this pull request Jan 25, 2021
…6319)

sonic-slave tag only allows all lower case. In case the user
name is mixed case, we need to change user name to all lower case.

Signed-off-by: Guohan Lu <lguohan@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants