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.

Sign up for GitHub

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

Jump to bottom

[code sync] Merge code from sonic-net/sonic-utilities:202405 to 202405 #100

Merged
merged 2 commits into from
Dec 28, 2024
Merged

[code sync] Merge code from sonic-net/sonic-utilities:202405 to 202405 #100

merged 2 commits into from
Dec 28, 2024

Conversation

mssonicbld
Copy link
Collaborator 

* e0a268ae - (head/202405) Fix monit connection failure when doing routeCheck/contianerCheck  monitor enable after config reload (#3698) (2024-12-27) [abdosi]<br>```

abdosi and others added 2 commits December 27, 2024 12:21
@abdosi @mssonicbld
Fix monit connection failure when doing routeCheck/contianerCheck mon…
e0a268a 
…itor enable after config reload (#3698)

Fix:
sonic-net/sonic-buildimage#21268

How I did:
Reorder the sequence of doing enabling container_cheek and routeCheck before doing monit reload to avoid transient issue of monit sock error.

Also I add a sleep of 1 sec to make sure monitor enable configuration takes effect before we do reload.
@mssonicbld
Merge branch '202405' of https://github.com/sonic-net/sonic-utilities …
48683d8 
…into 202405
@mssonicbld mssonicbld merged commit 74374d9 into Azure:202405 Dec 28, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mssonicbld @abdosi