Skip to content

[ci] Fix the 202012 branch PR validation issue caused by enabling ACR.#10543

Closed
liushilongbuaa wants to merge 1 commit intosonic-net:202012from
liushilongbuaa:shilongliu/fix-repd
Closed

[ci] Fix the 202012 branch PR validation issue caused by enabling ACR.#10543
liushilongbuaa wants to merge 1 commit intosonic-net:202012from
liushilongbuaa:shilongliu/fix-repd

Conversation

@liushilongbuaa
Copy link
Copy Markdown
Contributor

@liushilongbuaa liushilongbuaa commented Apr 12, 2022

Why I did it

PR9426(#9426) is not included in 202012 branch.
Cherry-pick involved un-expected codes.

How I did it

How to verify it

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

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 fix [ci] Fix the 202012 branch PR validation issue caused by enabling ACR. Apr 12, 2022
@liushilongbuaa liushilongbuaa marked this pull request as ready for review April 12, 2022 06:49
@qiluo-msft
Copy link
Copy Markdown
Collaborator

Please resolve conflict.

@qiluo-msft qiluo-msft requested a review from xumia April 12, 2022 07:08
Copy link
Copy Markdown
Collaborator

@xumia xumia Apr 12, 2022

Choose a reason for hiding this comment

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

Why we remove it? How about keep the same as the master one? Keep template files are the same as possible as we can with it in different branches, the only different is the variable file.

…et#10341)

Why I did it
docker hub will limit the pull rate.
Use ACR instead to pull debian related docker image.

How I did it
Set DEFAULT_CONTAINER_REGISTRY in pipeline.
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