Skip to content

Fix error in testcase doc#1039

Merged
lguohan merged 2 commits intosonic-net:masterfrom
neethajohn:fix_doc
Jul 31, 2019
Merged

Fix error in testcase doc#1039
lguohan merged 2 commits intosonic-net:masterfrom
neethajohn:fix_doc

Conversation

@neethajohn
Copy link
Copy Markdown
Contributor

Description of PR

Fix error in command

Type of change

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

@neethajohn neethajohn requested a review from yxieca July 29, 2019 16:58
@yxieca
Copy link
Copy Markdown
Collaborator

yxieca commented Jul 29, 2019

Can you also state that the new_sonic_image, stay_in_target_image (default true) and cleanup_old_sonic_images (default false) are optional?

@neethajohn
Copy link
Copy Markdown
Contributor Author

Can you also state that the new_sonic_image, stay_in_target_image (default true) and cleanup_old_sonic_images (default false) are optional?

'stay_in_target_image' defaults to false
I will mention the optional part. I have already specified that 'stay_in_target_image' and 'cleanup_old_sonic_images' default to false

@lguohan lguohan merged commit 188aba4 into sonic-net:master Jul 31, 2019
sdszhang pushed a commit to sdszhang/sonic-mgmt that referenced this pull request Mar 23, 2026
…net#1039)

<!--
Please make sure you've read and understood our contributing guidelines;
https://github.com/sonic-net/SONiC/blob/gh-pages/CONTRIBUTING.md

Please provide following information to help code review process a bit
easier:
-->
### Description of PR
<!--
- Please include a summary of the change and which issue is fixed.
- Please also include relevant motivation and context. Where should
reviewer start? background context?
- List any dependencies that are required for this change.
-->

Summary:

This fixes the headroom related tests for TH5 devices

### Type of change

<!--
- Fill x for your type of change.
- e.g.
- [x] Bug fix
-->

- [x] Bug fix
- [ ] Testbed and Framework(new/improvement)
- [ ] New Test case
    - [ ] Skipped for non-supported platforms
- [ ] Test case improvement

### Back port request
- [ ] 202205
- [ ] 202305
- [ ] 202311
- [ ] 202405
- [ ] 202411
- [x] 202503
- [x] 202505
- [x] 202511

### Approach
#### How did you verify/test it?
Manually tested
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants