Skip to content

[monit] Avoid monit error log by removing "-l" from monit_swss#58

Closed
stephenxs wants to merge 1 commit intomasterfrom
fix-monit-buffermgrd
Closed

[monit] Avoid monit error log by removing "-l" from monit_swss#58
stephenxs wants to merge 1 commit intomasterfrom
fix-monit-buffermgrd

Conversation

@stephenxs
Copy link
Copy Markdown
Owner

Why I did it

Avoid the following error messages while dynamic buffer calculation is enabled

ERR monit[491]: 'swss|buffermgrd' status failed (1) -- '/usr/bin/buffermgrd -l' is not running in host

How I did it

Change /usr/bin/buffermgrd -l to /usr/bin/buffermgrd. The buffermgrd is started by -l for traditional model or -a for dynamic model.
So we need to use the common section of both.

How to verify it

Manually test.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012

Description for the changelog

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

@stephenxs stephenxs requested a review from keboliu April 6, 2021 02:44
Avoid the following error messages while dynamic buffer calculation is enabled
```
ERR monit[491]: 'swss|buffermgrd' status failed (1) -- '/usr/bin/buffermgrd -l' is not running in host
```

Signed-off-by: Stephen Sun <stephens@nvidia.com>
@stephenxs stephenxs force-pushed the fix-monit-buffermgrd branch from 676b593 to 2929794 Compare April 6, 2021 02:52
@stephenxs stephenxs closed this Apr 6, 2021
@stephenxs stephenxs deleted the fix-monit-buffermgrd branch April 6, 2021 22:11
stephenxs pushed a commit that referenced this pull request May 6, 2022
submodule update, includes:

ec32690 CVE-2020-25614: Update xmlquery, jsonquery and xpath packages. (#58)
5156527 Showtech sonic mgmt framework: Add Management Framework functionality for "show tech-support" (#49)
stephenxs pushed a commit that referenced this pull request Apr 7, 2025
…ly (sonic-net#22245)

#### Why I did it
src/sonic-stp
```
* 8320909 - (HEAD -> master, origin/master, origin/HEAD) MSTP configuration handling - Initialization APIS (#58) (14 hours ago) [Divya Kumaran Chandralekha]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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