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

Event and Alarm Managment Framework HLD. #1064

Merged
merged 2 commits into from
Aug 23, 2022
Merged

Event and Alarm Managment Framework HLD. #1064

merged 2 commits into from
Aug 23, 2022

Conversation

bhaveshdell
Copy link
Contributor

This HLD has been reviewed and approved on the following PR.
#761

This PR is created to meet the CLA requirements.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Aug 18, 2022 •
edited
Loading

CLA Signed

The committers listed above are authorized under a signed CLA.

@zhangyanzhao zhangyanzhao mentioned this pull request Aug 22, 2022
Closed
@zhangyanzhao
Copy link
Collaborator

The code PR is tracked in #761

@bhavini-gada
Copy link

@qiluo-msft - This new PR has EasyCLA check passed. Please help with the merge so we can start updating code PRs

@qiluo-msft
Copy link
Contributor

@renukamanavalan could you check and merge?

@mint570 mint570 self-requested a review August 23, 2022 19:58
mint570
mint570 approved these changes Aug 23, 2022
View reviewed changes
Copy link
Collaborator

@mint570 mint570 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm from me

@bhavini-gada
Copy link

@renukamanavalan - Can you please help merge this PR?

@renukamanavalan renukamanavalan merged commit 6411027 into sonic-net:master Aug 23, 2022
itamar-talmon pushed a commit to itamar-talmon/SONiC that referenced this pull request Oct 2, 2022
@bhaveshdell
Event and Alarm Managment Framework HLD. (sonic-net#1064)
b342d86
@vboykox
Copy link
Member

vboykox commented Feb 16, 2023 •
edited
Loading

Hi! Are there plans to merge this feature?

sonic-net/sonic-buildimage#7813
sonic-net/sonic-swss-common#490
sonic-net/sonic-mgmt-framework#85
sonic-net/sonic-mgmt-common#48

@liat-grozovik
Copy link
Collaborator

@vboykox can you please have the list of the PRs in the PR description like recently sent by Yanzhao?
I also suggest to add it to 202305 release planning so reviewers can assign themselves.

@jeff-yin jeff-yin mentioned this pull request Sep 7, 2023
Merged
@bmridul
Copy link
Contributor

bmridul commented Nov 2, 2023

@bhaveshdell , Can u pls reply to this question. What is the plan for the following PRs.

Hi! Are there plans to merge this feature?

sonic-net/sonic-buildimage#7813 sonic-net/sonic-swss-common#490 sonic-net/sonic-mgmt-framework#85 sonic-net/sonic-mgmt-common#48

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@renukamanavalan renukamanavalan renukamanavalan approved these changes

@liat-grozovik liat-grozovik Awaiting requested review from liat-grozovik

@qiluo-msft qiluo-msft Awaiting requested review from qiluo-msft

@mint570 mint570 Awaiting requested review from mint570

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@bhaveshdell @zhangyanzhao @bhavini-gada @qiluo-msft @vboykox @liat-grozovik @bmridul @renukamanavalan @mint570