Skip to content

Remove trailing dash from tunnelstat cache directory name#4545

Open
LGH-12 wants to merge 4 commits into
sonic-net:masterfrom
LGH-12:fix_tunnelstat_tag
Open

Remove trailing dash from tunnelstat cache directory name#4545
LGH-12 wants to merge 4 commits into
sonic-net:masterfrom
LGH-12:fix_tunnelstat_tag

Conversation

@LGH-12

@LGH-12 LGH-12 commented May 15, 2026

Copy link
Copy Markdown

What I did

Remove trailing dash from tunnelstat cache directory name

admin@sonic:~$ tunnelstat -c
Cleared counters
admin@sonic:~$ ls /tmp/cache/tunnelstat
1000-

# Other cache directory name
admin@sonic:~$ ls /tmp/cache/intfstat/
1000
admin@sonic:~$ ls /tmp/cache/portstat/
1000

How I did it

Tag should be None,if no tag parameter is provided,

How to verify it

on a DUT:

admin@sonic:~$ tunnelstat -c
Cleared counters
admin@sonic:~$ ls /tmp/cache/tunnelstat/
1000

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: LTeng <liuteng@asterfusion.com>
@LGH-12 LGH-12 force-pushed the fix_tunnelstat_tag branch from 68f4501 to 9dfed0f Compare May 15, 2026 09:09
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

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.

2 participants