Skip to content

[Nokia][Device] Fix monState in farbic monitor config#19196

Merged
kperumalbfn merged 1 commit intosonic-net:masterfrom
saksarav-nokia:saksarav-fabricmon-config
Jun 28, 2024
Merged

[Nokia][Device] Fix monState in farbic monitor config#19196
kperumalbfn merged 1 commit intosonic-net:masterfrom
saksarav-nokia:saksarav-fabricmon-config

Conversation

@saksarav-nokia
Copy link
Contributor

Why I did it

PR #19144 was missing quotes in fabric monitor config which causes load_minigraph failure

Work item tracking
  • Microsoft ADO (number only):

How I did it

Added fabric monState config in quotes

How to verify it

Did load_minigraph and verified that there are no errors.

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)

@saksarav-nokia
Copy link
Contributor Author

@arlakshm pls review

@arlakshm
Copy link
Contributor

/Azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202405: #19463

@mssonicbld
Copy link
Collaborator

@saksarav-nokia cherry pick PR didn't pass PR checker. Please check!!!
#19463

2 similar comments
@mssonicbld
Copy link
Collaborator

@saksarav-nokia cherry pick PR didn't pass PR checker. Please check!!!
#19463

@mssonicbld
Copy link
Collaborator

@saksarav-nokia cherry pick PR didn't pass PR checker. Please check!!!
#19463

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.

5 participants