Skip to content

Event and Alarm Managment Framework HLD. #1064

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
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

@bhaveshdell bhaveshdell commented Aug 18, 2022

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

zhangyanzhao commented Aug 22, 2022

The code PR is tracked in #761

@bhavini-gada
Copy link

bhavini-gada commented Aug 22, 2022

@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

qiluo-msft commented Aug 22, 2022

@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

bhavini-gada commented Aug 23, 2022

@renukamanavalan - Can you please help merge this PR?

@renukamanavalan renukamanavalan merged commit 6411027 into sonic-net:master Aug 23, 2022
@zhangyanzhao zhangyanzhao requested a review from mint570 August 24, 2022 00:42
@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

liat-grozovik commented Feb 16, 2023

@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

@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

1 more reviewer

@renukamanavalan renukamanavalan renukamanavalan approved these changes

Reviewers whose approvals may not affect merge requirements

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