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

Revert "[debugcounterorch] check if counter type is supported #1884

Merged
merged 1 commit into from
Aug 26, 2021

Conversation

prsunny
Copy link
Collaborator

@prsunny prsunny commented Aug 25, 2021

This reverts commit 04105a4.

What I did
Revert PR #1789 from 202012 branch.

Why I did it
Fix issue - sonic-net/sonic-buildimage#8538 in 202012 branch.

How I verified it

Details if related

@prsunny
Copy link
Collaborator Author

prsunny commented Aug 25, 2021

@stepanblyschak for viz

@stepanblyschak
Copy link
Contributor

@prsunny Is there a plan to take this change back to 202012 soon? As I understand this is platform's SAI issue.

@prsunny
Copy link
Collaborator Author

prsunny commented Aug 25, 2021

@prsunny Is there a plan to take this change back to 202012 soon? As I understand this is platform's SAI issue.

No, we don't have plans to take to 202012 as of now.

@qiluo-msft qiluo-msft merged commit 8471f42 into sonic-net:202012 Aug 26, 2021
@prsunny prsunny deleted the revert_dbg branch August 27, 2021 01:15
EdenGri pushed a commit to EdenGri/sonic-swss that referenced this pull request Feb 28, 2022
- What I did
This PR include some fixes which were missed for sonic-net#1723
i.e. removing -t option from the docker exec commands. to understand why the -it option was removed, refer sonic-net#1723.
Also, the show techsupport exits with $RETURN_CODE only when --redirect-stderr option is used.

Signed-off-by: Vivek Reddy Karri <vkarri@nvidia.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.

4 participants