Skip to content

[Docs] Fix the platform option for building docker-ptf.gz#2905

Merged
lguohan merged 1 commit intosonic-net:masterfrom
MuLinForest:pr_docker-ptf_build
Feb 2, 2021
Merged

[Docs] Fix the platform option for building docker-ptf.gz#2905
lguohan merged 1 commit intosonic-net:masterfrom
MuLinForest:pr_docker-ptf_build

Conversation

@MuLinForest
Copy link
Copy Markdown
Contributor

Description of PR

Summary:
Fix the platform option for building docker-ptf.gz

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Approach

What is the motivation for this PR?

When I try to build target/docker-ptf.gz according to the document, it will show "No rule to make target 'target/docker-ptf.gz".

How did you do it?

According to sonic-net/sonic-buildimage#6540, the platform option to build docker-ptf.gz should change to vs.

How did you verify/test it?

After changed the platform option to vs, target/docker-ptf.gz can be successfully built.

What is the motivation for this PR?
When I try to build target/docker-ptf.gz according to the document, it will show "No rule to make target 'target/docker-ptf.gz".

How did you do it?
According to sonic-net/sonic-buildimage#6540, the platform option to build docker-ptf.gz should change to vs.

How did you verify/test it?
After changed the platform option to vs, target/docker-ptf.gz can be successfully built.
@lguohan lguohan merged commit 5215198 into sonic-net:master Feb 2, 2021
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…lly (sonic-net#16532)

src/sonic-swss

* de7186c6 - (HEAD -> 202205, origin/202205) [202205][CodeQL]: Use dependencies with relevant versions in azp template. (sonic-net#2905) (13 days ago) [Nazarii Hnydyn]
* 106dd9ed - [CodeQL]: Use dependencies with relevant versions in azp template. (sonic-net#2845) (3 weeks ago) [Nazarii Hnydyn]
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.

2 participants