Skip to content

[ci] Remove marvell-armhf and vsonic test from PR checker#16635

Closed
liushilongbuaa wants to merge 5 commits intosonic-net:masterfrom
liushilongbuaa:disable-vtest
Closed

[ci] Remove marvell-armhf and vsonic test from PR checker#16635
liushilongbuaa wants to merge 5 commits intosonic-net:masterfrom
liushilongbuaa:disable-vtest

Conversation

@liushilongbuaa
Copy link
Copy Markdown
Contributor

@liushilongbuaa liushilongbuaa commented Sep 21, 2023 •
edited
Loading

Why I did it

make marvell-armhf and vsonic test as config to unblock new features.

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

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)

@liushilongbuaa liushilongbuaa changed the title [ci] Remove marvell-armhf and vsonic test in PR checker [ci] Remove marvell-armhf and vsonic test from PR checker Sep 21, 2023
@liushilongbuaa liushilongbuaa marked this pull request as ready for review September 21, 2023 06:38
@liushilongbuaa
Copy link
Copy Markdown
Contributor Author

liushilongbuaa commented Sep 21, 2023

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Copy Markdown
Collaborator

mssonicbld commented Sep 21, 2023

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

azure-pipelines bot commented Sep 21, 2023

Azure Pipelines successfully started running 1 pipeline(s).

lguohan
lguohan previously approved these changes Sep 21, 2023
View reviewed changes
@lguohan lguohan enabled auto-merge (squash) September 21, 2023 08:02
auto-merge was automatically disabled September 21, 2023 10:07

Head branch was pushed to by a user without write access

@saiarcot895
Copy link
Copy Markdown
Contributor

saiarcot895 commented Sep 21, 2023

@liushilongbuaa Can I ask what those new features are that break the t0-sonic checker?

@Pterosaur
Copy link
Copy Markdown
Contributor

Pterosaur commented Sep 22, 2023 •
edited
Loading

@liushilongbuaa Can I ask what those new features are that break the t0-sonic checker?

@saiarcot895
Type 7 for MACsec. The t0-sonic might be re-enabled once the PR: sonic-net/sonic-mgmt#9812 merged.

@Pterosaur Pterosaur requested a review from lguohan September 22, 2023 01:41
@saiarcot895
Copy link
Copy Markdown
Contributor

saiarcot895 commented Sep 22, 2023

Is it possible to instead disable just the macsec test case, instead of the whole test suite? I would like the warm reboot test to still run in t0-sonic.

@Pterosaur
Copy link
Copy Markdown
Contributor

Pterosaur commented Sep 22, 2023

Is it possible to instead disable just the macsec test case, instead of the whole test suite? I would like the warm reboot test to still run in t0-sonic.

I thought there was only macsec test in t0-sonic. That's my mistake.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Pterosaur Pterosaur Pterosaur approved these changes

@lguohan lguohan Awaiting requested review from lguohan lguohan is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@liushilongbuaa @mssonicbld @saiarcot895 @Pterosaur @lguohan