sonic-net / sonic-swss

SONiC Switch State Service (SwSS)
https://azure.github.io/SONiC
Other
170 stars 503 forks source link

[action] [PR:3209] Fix in switchorch: unsupported attribute causes skipping of processing the rest of configurations (#3209) #3225

Closed mssonicbld closed 1 month ago

mssonicbld commented 1 month ago

What I did Changed it so that if the configuration has an unsupported attribute, it would continue processing the rest of the configuration rather than break out of the loop and end the processing immediately. Also added syslogs to make it more clear.

mssonicbld commented 1 month ago

Original PR: https://github.com/sonic-net/sonic-swss/pull/3209

mssonicbld commented 1 month ago

/azp run Azure.sonic-swss

azure-pipelines[bot] commented 1 month ago
Azure Pipelines successfully started running 1 pipeline(s).
mssonicbld commented 1 month ago

/azp run Azure.sonic-swss

azure-pipelines[bot] commented 1 month ago
Azure Pipelines successfully started running 1 pipeline(s).
mssonicbld commented 1 month ago

/azp run Azure.sonic-swss

azure-pipelines[bot] commented 1 month ago
Azure Pipelines successfully started running 1 pipeline(s).
mssonicbld commented 1 month ago

/azp run Azure.sonic-swss

azure-pipelines[bot] commented 1 month ago
Azure Pipelines successfully started running 1 pipeline(s).
amazor commented 1 month ago

/azp run Azure.sonic-swss

azure-pipelines[bot] commented 1 month ago
Commenter does not have sufficient privileges for PR 3225 in repo sonic-net/sonic-swss
mssonicbld commented 1 month ago

/azp run Azure.sonic-swss

azure-pipelines[bot] commented 1 month ago
Azure Pipelines successfully started running 1 pipeline(s).
amazor commented 1 month ago

@prsunny, I am not sure what the issue here is. Can you give more details?

mssonicbld commented 1 month ago

/azp run Azure.sonic-swss

azure-pipelines[bot] commented 1 month ago
Azure Pipelines successfully started running 1 pipeline(s).
amazor commented 1 month ago

@prsunny Can you take a look to explain why build is failing?

mssonicbld commented 1 month ago

/azp run Azure.sonic-swss

azure-pipelines[bot] commented 1 month ago
Azure Pipelines successfully started running 1 pipeline(s).