Skip to content

[build]: add show docker tag#1833

Merged
lguohan merged 1 commit intosonic-net:masterfrom
lguohan:showtag
Jul 1, 2018
Merged

[build]: add show docker tag#1833
lguohan merged 1 commit intosonic-net:masterfrom
lguohan:showtag

Conversation

@lguohan
Copy link
Collaborator

@lguohan lguohan commented Jul 1, 2018

- What I did

- How I did it

- How to verify it

lgh@gulv-vm1:/data2/sonic/sonic-buildimage$ make showtag
sonic-slave-lgh:1c266fd5adc
sonic-slave-base:71e6bba7b53

- Description for the changelog

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

@lguohan lguohan merged commit 7edbf5d into sonic-net:master Jul 1, 2018
vivekrnv added a commit to vivekrnv/sonic-buildimage that referenced this pull request Sep 21, 2021
Signed-off-by: Vivek Reddy Karri <[email protected]>

7a58e84 [techsupport] [202012] Removed -i option for docker commands and Improved Error Reporting (sonic-net#1833)
qiluo-msft pushed a commit that referenced this pull request Sep 22, 2021
7a58e84 [techsupport] [202012] Removed -i option for docker commands and Improved Error Reporting (#1833)
praveen-li pushed a commit to praveen-li/sonic-buildimage that referenced this pull request Feb 15, 2022
Signed-off-by: Sangita Maity <[email protected]>

* src/sonic-utilities 9fde08a...595e322 (3):
  > [202012][show techsupport] address show techsupport return none zero code issue (sonic-net#1847)
  > [show techsupport] fix bash errors in generate_dump script (sonic-net#1844)
  > [techsupport] [202012] Removed -i option for docker commands and Improved Error Reporting (sonic-net#1833)
taras-keryk pushed a commit to taras-keryk/sonic-buildimage that referenced this pull request Apr 28, 2022
…#1844)

What I did
Fix: sonic-net#8850

Issue was introduced by sonic-net#1723, sonic-net#1833, and sonic-net#1843 (pending merge)

The error_handler is a great idea but the bash script needs to be error free first.

How I did it
Fix bash script errors.

How to verify it
run show techsupport test..

Signed-off-by: Ying Xie <[email protected]>
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.

1 participant