Skip to content

Event and Alarm Managment Framework HLD.#1064

Merged
renukamanavalan merged 2 commits into
sonic-net:masterfrom
bhaveshdell:event-alarm-framewk
Aug 23, 2022
Merged

Event and Alarm Managment Framework HLD.#1064
renukamanavalan merged 2 commits into
sonic-net:masterfrom
bhaveshdell:event-alarm-framewk

Conversation

@bhaveshdell

@bhaveshdell bhaveshdell commented Aug 18, 2022

Copy link
Copy Markdown
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

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

Copy link
Copy Markdown

CLA Signed

The committers listed above are authorized under a signed CLA.

@zhangyanzhao zhangyanzhao mentioned this pull request Aug 22, 2022
Closed
@zhangyanzhao

zhangyanzhao commented Aug 22, 2022

Copy link
Copy Markdown
Collaborator

The code PR is tracked in #761

@bhavini-gada

bhavini-gada commented Aug 22, 2022

Copy link
Copy Markdown

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

@qiluo-msft

qiluo-msft commented Aug 22, 2022

Copy link
Copy Markdown
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

@mint570 mint570 left a comment

Copy link
Copy Markdown
Collaborator

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

bhavini-gada commented Aug 23, 2022

Copy link
Copy Markdown

@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

vboykox commented Feb 16, 2023 •
edited
Loading

Copy link
Copy Markdown
Member

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

liat-grozovik commented Feb 16, 2023

Copy link
Copy Markdown
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.

@jeffyin-dell jeffyin-dell mentioned this pull request Sep 7, 2023
Merged
@bmridul

bmridul commented Nov 2, 2023

Copy link
Copy Markdown
Contributor

@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