Skip to content

[TestbedV2][202012]Improve the display of pipeline.#13124

Merged
wangxin merged 1 commit intosonic-net:202012from
yutongzhang-microsoft:improve_pipeline_202012
Jan 4, 2023
Merged

[TestbedV2][202012]Improve the display of pipeline.#13124
wangxin merged 1 commit intosonic-net:202012from
yutongzhang-microsoft:improve_pipeline_202012

Conversation

@yutongzhang-microsoft
Copy link
Copy Markdown
Contributor

The display of azure pipeline is not specific now, such as when the step Run test fails, the display of itself shows successful, but the display of step Kvmdump shows fails, but actually, the step Kvmdump doesn't fail. I improve the display of azure pipeline in this pr, each step has its own success or failure, and is shown in azure pipeline.

Why I did it

The display of azure pipeline is not specific now, such as when the step Run test fails, the display of itself shows successful, but the display of step Kvmdump shows fails, but actually, the step Kvmdump doesn't fail. I improve the display of azure pipeline in this pr, each step has its own success or failure, and is shown in azure pipeline.

How I did it

  1. Each step has its own signature of success or failure.
  2. Using the chain of responsibility pattern to manage all status.
  3. Modify the expected-state in each step.

How to verify it

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

Copy link
Copy Markdown
Contributor

@yejianquan yejianquan left a comment

Choose a reason for hiding this comment

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

LGTM

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants