Skip to content

[drop counters] Re-add missing json import#2093

Merged
daall merged 1 commit intosonic-net:masterfrom
daall:missing_import
Aug 12, 2020
Merged

[drop counters] Re-add missing json import#2093
daall merged 1 commit intosonic-net:masterfrom
daall:missing_import

Conversation

@daall
Copy link
Copy Markdown
Contributor

@daall daall commented Aug 12, 2020

Signed-off-by: Danny Allen [email protected]

Description of PR

Summary: [drop counters] Re-add missing json import
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Approach

What is the motivation for this PR?

In a recent clean-up I accidentally deleted the json import from the drop counter tests, causing test failures.

How did you do it?

Added the import.

How did you verify/test it?

Re-run the tests locally.

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

@daall daall added the Bug 🐛 label Aug 12, 2020
@daall daall requested a review from yxieca August 12, 2020 16:52
@daall daall merged commit 8f65fb4 into sonic-net:master Aug 12, 2020
@daall daall deleted the missing_import branch August 12, 2020 19:19
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
c5c105f (HEAD -> 202111, origin/202111) [PBH] Implement Edit Flows (sonic-net#2093)
7050581 [techsupport] Handle minor fixes of TS Lock and update auto-TS (sonic-net#2114)
3602f99 Fix issues in clear_qos (sonic-net#2122)
4f96d3b Fixing get port speed when oper status is down (sonic-net#2123)
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