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

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

Closed
wants to merge 1 commit into from

Conversation

stephenxs
Copy link
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 <[email protected]>
@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)
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