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]: add show docker tag #1833

Merged
merged 1 commit into from
Jul 1, 2018
Merged

[build]: add show docker tag #1833

merged 1 commit into from
Jul 1, 2018

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 <vkarri@nvidia.com>

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 <samaity@linkedin.com>

* 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 <ying.xie@microsoft.com>
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