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

Make RADV service script executable #5186

Merged
merged 1 commit into from
Aug 15, 2020

Conversation

vaibhavhd
Copy link
Contributor

- Why I did it
Make RADV script executable.

- How I did it
Make RADV script executable.

- How to verify it

Starting service:

admin@str-s6100-acs-4:~$ sudo systemctl stop radv
admin@str-s6100-acs-4:~$ 
admin@str-s6100-acs-4:~$ sudo systemctl status radv
â—� radv.service - Router advertiser container
   Loaded: loaded (/lib/systemd/system/radv.service; enabled; vendor preset: enabled)
   Active: inactive (dead) since Fri 2020-08-14 07:30:39 UTC; 2s ago
  Process: 3204 ExecStart=/usr/local/bin/radv.sh wait (code=exited, status=0/SUCCESS)
  Process: 9243 ExecStop=/usr/local/bin/radv.sh stop (code=exited, status=0/SUCCESS)
 Main PID: 3204 (code=exited, status=0/SUCCESS)

Aug 14 07:27:19 str-s6100-acs-4 systemd[1]: Starting Router advertiser container...
Aug 14 07:27:23 str-s6100-acs-4 radv.sh[2561]: Creating new radv container with HWSKU Force10-S6100
Aug 14 07:27:25 str-s6100-acs-4 radv.sh[2561]: 6f9c6d553b7e0d3b0b0e75f216e14bcf4b3901f1cd33631be57934b6bfa8d6cf
Aug 14 07:27:29 str-s6100-acs-4 radv.sh[2561]: radv
Aug 14 07:27:29 str-s6100-acs-4 admin[3196]: Started radv service...
Aug 14 07:27:29 str-s6100-acs-4 systemd[1]: Started Router advertiser container.
Aug 14 07:30:38 str-s6100-acs-4 systemd[1]: Stopping Router advertiser container...
Aug 14 07:30:39 str-s6100-acs-4 radv.sh[3204]: 137
Aug 14 07:30:39 str-s6100-acs-4 systemd[1]: radv.service: Succeeded.
Aug 14 07:30:39 str-s6100-acs-4 systemd[1]: Stopped Router advertiser container.
admin@str-s6100-acs-4:~$ 

Stopping service:

admin@str-s6100-acs-4:~$ sudo systemctl start radv
admin@str-s6100-acs-4:~$ sudo systemctl status radv
â—� radv.service - Router advertiser container
   Loaded: loaded (/lib/systemd/system/radv.service; enabled; vendor preset: enabled)
   Active: active (running) since Fri 2020-08-14 07:30:51 UTC; 1s ago
  Process: 9314 ExecStartPre=/usr/local/bin/radv.sh start (code=exited, status=0/SUCCESS)
 Main PID: 9398 (radv.sh)
    Tasks: 12 (limit: 4915)
   Memory: 24.0M
   CGroup: /system.slice/radv.service
           ├─9398 /bin/bash /usr/local/bin/radv.sh wait
           ├─9399 /bin/bash /usr/bin/radv.sh wait
           └─9401 docker wait radv

Aug 14 07:30:49 str-s6100-acs-4 systemd[1]: Starting Router advertiser container...
Aug 14 07:30:51 str-s6100-acs-4 radv.sh[9314]: Starting existing radv container with HWSKU Force10-S6100
Aug 14 07:30:51 str-s6100-acs-4 radv.sh[9314]: radv
Aug 14 07:30:51 str-s6100-acs-4 systemd[1]: Started Router advertiser container.
admin@str-s6100-acs-4:~$ 

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

  • 201811
  • 201911
  • 202006

- Description for the changelog

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

@vaibhavhd vaibhavhd requested a review from yxieca August 14, 2020 07:47
@vaibhavhd vaibhavhd self-assigned this Aug 14, 2020
@vaibhavhd
Copy link
Contributor Author

retest this please

@vaibhavhd
Copy link
Contributor Author

retest this please

2 similar comments
@vaibhavhd
Copy link
Contributor Author

retest this please

@vaibhavhd
Copy link
Contributor Author

retest this please

@vaibhavhd
Copy link
Contributor Author

Retest this please

@vaibhavhd vaibhavhd merged commit b193af2 into sonic-net:master Aug 15, 2020
@vaibhavhd vaibhavhd deleted the radv-service-update branch February 22, 2021 17:01
santhosh-kt pushed a commit to santhosh-kt/sonic-buildimage that referenced this pull request Feb 25, 2021
Co-authored-by: Vaibhav Hemant Dixit <vadixit@microsoft.com>
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