Skip to content

Fix auditd container monit startup issue#21979

Merged
qiluo-msft merged 3 commits intosonic-net:masterfrom
FengPan-Frank:auditd_monit
Mar 11, 2025
Merged

Fix auditd container monit startup issue#21979
qiluo-msft merged 3 commits intosonic-net:masterfrom
FengPan-Frank:auditd_monit

Conversation

@FengPan-Frank
Copy link
Copy Markdown
Contributor

Why I did it

new added auditd container failed to be launched due to monit blocker.

Work item tracking
  • Microsoft ADO (number only):31121982

How I did it

Skip the unexpected container check logic

How to verify it

Verify via auditd image

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@maipbui
Copy link
Copy Markdown
Contributor

maipbui commented Mar 10, 2025

LGTM

@qiluo-msft qiluo-msft enabled auto-merge (squash) March 10, 2025 17:06
@qiluo-msft qiluo-msft merged commit 7a69613 into sonic-net:master Mar 11, 2025
21 checks passed
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202411: #22012

qiluo-msft pushed a commit to qiluo-msft/sonic-buildimage that referenced this pull request Jan 26, 2026
Why I did it
new added auditd container failed to be launched due to monit blocker.

How I did it
Skip the unexpected container check logic

How to verify it
Verify via auditd image
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants