Skip to content

[setup-container.sh] setup-container minor fixes#3003

Merged
lguohan merged 1 commit intosonic-net:masterfrom
myronsosyak:myron/fix_setup_container
Feb 23, 2021
Merged

[setup-container.sh] setup-container minor fixes#3003
lguohan merged 1 commit intosonic-net:masterfrom
myronsosyak:myron/fix_setup_container

Conversation

@myronsosyak
Copy link
Copy Markdown
Contributor

Description of PR

  • Fix -i option in the script.
  • Update USER env variable

Summary: MInor fixes
Fixes # (issue)

Type of change

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

Approach

What is the motivation for this PR?

Fix script to be up to date with help output

How did you do it?

See changes

How did you verify/test it?

Use -i option

Any platform specific information?

N/A

Supported testbed topology if it's a new test case?

Documentation

@myronsosyak myronsosyak requested a review from a team as a code owner February 19, 2021 17:19
@myronsosyak myronsosyak changed the title [testbed] setup-container minor fixes [setup-container.sh] setup-container minor fixes Feb 19, 2021
@myronsosyak
Copy link
Copy Markdown
Contributor Author

retest this please

@theasianpianist
Copy link
Copy Markdown
Contributor

/Azurepipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@lguohan lguohan merged commit 63c3781 into sonic-net:master Feb 23, 2021
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…atically (sonic-net#16717)

src/sonic-utilities

* ff8a0643 - (HEAD -> 202205, origin/202205) [202205][acl-loader] Identity ICMP v4/v6 based on IP_PROTOCOL for custom ACL table types (sonic-net#3003) (6 days ago) [Zhijian Li]
* d9bc820e - Handle NotImplementedError exception while changing optoe write max (sonic-net#2985) (8 days ago) [mihirpat1]
* 4bf29fe2 - [sonic-package-manager] Increate timeout for sonic-package-manager migrate (sonic-net#2973) (8 days ago) [Yaqiang Zhu]
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