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.

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

[chassis][Arista] add supervisor to the platform_env.conf #12615

Merged
merged 1 commit into from
Nov 7, 2022

Conversation

arlakshm
Copy link
Contributor

@arlakshm arlakshm commented Nov 6, 2022

Signed-off-by: Arvindsrinivasan Lakshmi Narasimhan arlakshm@microsoft.com

Why I did it

Fixes #12614

How I did it

In the container_checker the database_chassis is added to expected container if device is supervisor
To detect the device is superviso, add supervisor=1 to the platform_env.conf of 7808 sup platform

How to verify it

run container_checker monit check

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • [x ] 202205

Description for the changelog

)

Signed-off-by: Arvindsrinivasan Lakshmi Narasimhan <arlakshm@microsoft.com>
@arlakshm arlakshm requested a review from Staphylo November 6, 2022 23:58
@arlakshm arlakshm added Platform: Arista Chassis 🤖 Modular chassis support labels Nov 6, 2022
@arlakshm arlakshm requested a review from wenyiz2021 November 6, 2022 23:58
@wenyiz2021
Copy link
Contributor

/azp run Azure.sonic-buildimage (Test kvmtest-t0-sonic)

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@wenyiz2021
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@arlakshm arlakshm merged commit c4be3a5 into sonic-net:master Nov 7, 2022
yxieca pushed a commit that referenced this pull request Nov 10, 2022
Why I did it
Fixes #12614

How I did it
In the container_checker the database_chassis is added to expected container if device is supervisor
To detect the device is superviso, add supervisor=1 to the platform_env.conf of 7808 sup platform

How to verify it
run container_checker monit check
Signed-off-by: Arvindsrinivasan Lakshmi Narasimhan <arlakshm@microsoft.com>
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.

[chassis] monit checker failing on supervisor
4 participants