Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[conf] append nos-config-part for s6100 #5234

Merged
merged 3 commits into from
Sep 8, 2020

Conversation

Blueve
Copy link
Contributor

@Blueve Blueve commented Aug 21, 2020

- Why I did it

We want to support pick configuration from /dev/sda12 automatically after cold reboot for s6100

- How I did it

  1. Append nos-config-part to ONIE_PLATFORM_EXTRA_CMDLINE_LINUX
  2. Always mkdir before migrate nos configuration
  3. Always migrate nos configuration

- How to verify it

  1. Build image and install to a 6100 SONiC DUT
  2. Copy configurations to /etc/sda12
  3. Clear /host/old_config
  4. Reboot and verify if the configuration been picked

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

  • 201811
  • 201911
  • 202006

- Description for the changelog

[conf] support pick configuration from /dev/sda12 by configure installer.conf

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

Blueve and others added 2 commits August 23, 2020 01:50
Signed-off-by: Guohan Lu <lguohan@gmail.com>
@lguohan
Copy link
Collaborator

lguohan commented Sep 8, 2020

retest vs please

@yxieca yxieca merged commit 01fb32f into sonic-net:master Sep 8, 2020
yxieca added a commit that referenced this pull request Sep 8, 2020
* [conf] append nos-config-part for s6100

* modify rc.local

Signed-off-by: Guohan Lu <lguohan@gmail.com>

* Update rc.local

Co-authored-by: Blueve <jika@microsoft.com>
Co-authored-by: Guohan Lu <lguohan@gmail.com>
Co-authored-by: Ying Xie <yxieca@users.noreply.github.com>
abdosi pushed a commit that referenced this pull request Sep 19, 2020
* [conf] append nos-config-part for s6100

* modify rc.local

Signed-off-by: Guohan Lu <lguohan@gmail.com>

* Update rc.local

Co-authored-by: Blueve <jika@microsoft.com>
Co-authored-by: Guohan Lu <lguohan@gmail.com>
Co-authored-by: Ying Xie <yxieca@users.noreply.github.com>
santhosh-kt pushed a commit to santhosh-kt/sonic-buildimage that referenced this pull request Feb 25, 2021
* [conf] append nos-config-part for s6100

* modify rc.local

Signed-off-by: Guohan Lu <lguohan@gmail.com>

* Update rc.local

Co-authored-by: Blueve <jika@microsoft.com>
Co-authored-by: Guohan Lu <lguohan@gmail.com>
Co-authored-by: Ying Xie <yxieca@users.noreply.github.com>
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