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

Skip test_secure_upgrade.py if current image is not secured. #9747

Merged
merged 26 commits into from
Aug 31, 2023

Conversation

abdosi
Copy link
Contributor

@abdosi abdosi commented Aug 29, 2023

What I did:
Skip the test tests/platform_tests/test_secure_upgrade.py if the current install image is not secured image.

ADO: 24319390

How I verify:
Manually Run the test case on image having this change : sonic-net/sonic-buildimage#16191

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
chassis

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
@abdosi
Copy link
Contributor Author

abdosi commented Aug 29, 2023

@StormLiangMS / @wangxin

@abdosi abdosi requested a review from wangxin August 30, 2023 23:13
Copy link
Collaborator

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@StormLiangMS StormLiangMS merged commit 47dbed1 into sonic-net:master Aug 31, 2023
11 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Sep 15, 2023
…et#9747)

What I did:
Skip the test tests/platform_tests/test_secure_upgrade.py if the current install image is not secured image.

ADO: 24319390

How I verify:
Manually Run the test case on image having this change : sonic-net/sonic-buildimage#16191What I did:
Skip the test tests/platform_tests/test_secure_upgrade.py if the current install image is not secured image.

ADO: 24319390

How I verify:
Manually Run the test case on image having this change : sonic-net/sonic-buildimage#16191
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202305: #10003

mssonicbld pushed a commit that referenced this pull request Sep 15, 2023
What I did:
Skip the test tests/platform_tests/test_secure_upgrade.py if the current install image is not secured image.

ADO: 24319390

How I verify:
Manually Run the test case on image having this change : sonic-net/sonic-buildimage#16191What I did:
Skip the test tests/platform_tests/test_secure_upgrade.py if the current install image is not secured image.

ADO: 24319390

How I verify:
Manually Run the test case on image having this change : sonic-net/sonic-buildimage#16191
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Jan 25, 2024
…et#9747)

What I did:
Skip the test tests/platform_tests/test_secure_upgrade.py if the current install image is not secured image.

ADO: 24319390

How I verify:
Manually Run the test case on image having this change : sonic-net/sonic-buildimage#16191What I did:
Skip the test tests/platform_tests/test_secure_upgrade.py if the current install image is not secured image.

ADO: 24319390

How I verify:
Manually Run the test case on image having this change : sonic-net/sonic-buildimage#16191
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.

4 participants