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

[swss.sh] fix an issue that dependent services are not read from a file #8943

Merged
merged 3 commits into from
Oct 27, 2021

Conversation

stepanblyschak
Copy link
Collaborator

This is due to the SERVICE variable declared after reading a file

Signed-off-by: Stepan Blyschak stepanb@nvidia.com

Why I did it

To fix an issue that dhcp_relay does not restart with swss.

How I did it

Fixed in the swss.sh script

How to verify it

sudo systemctl restart swss
verify dhcp_relay restarts as well.

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

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

This is due to the SERVICE variable declared after reading a file

Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
liat-grozovik
liat-grozovik previously approved these changes Oct 20, 2021
@liat-grozovik
Copy link
Collaborator

@qiluo-msft could you please help to review and merge?

@qiluo-msft
Copy link
Collaborator

. /usr/local/bin/asic_status.sh

The file is previously organized as functions first, then main logic. Could you move this part to the file end?


Refers to: files/scripts/swss.sh:23 in 2b4d9ee. [](commit_id = 2b4d9ee, deletion_comment = False)

Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
…services

Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
Copy link
Collaborator Author

@qiluo-msft Please check latest changes.

@qiluo-msft qiluo-msft merged commit 4ad5f2a into sonic-net:master Oct 27, 2021
judyjoseph pushed a commit that referenced this pull request Oct 27, 2021
…le (#8943)

This is due to the SERVICE variable declared after reading a file

#### Why I did it

To fix an issue that dhcp_relay does not restart with swss.

#### How I did it

Fixed in the swss.sh script

#### How to verify it

sudo systemctl restart swss
verify dhcp_relay restarts as well.
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