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

[config] Restart telemetry service upon config (re)load #992

Merged
merged 1 commit into from
Jul 21, 2020
Merged

[config] Restart telemetry service upon config (re)load #992

merged 1 commit into from
Jul 21, 2020

Conversation

jleveque
Copy link
Contributor

Add telemetry service to the list of services to stop/reset-failed/restart during config load/reload operations.

@jleveque
Copy link
Contributor Author

Retest this please

4 similar comments
@jleveque
Copy link
Contributor Author

Retest this please

@jleveque
Copy link
Contributor Author

Retest this please

@jleveque
Copy link
Contributor Author

Retest this please

@jleveque
Copy link
Contributor Author

Retest this please

@jleveque jleveque merged commit 995cf39 into sonic-net:master Jul 21, 2020
@jleveque jleveque deleted the restart_telemetry_config_reload branch July 21, 2020 22:53
lguohan pushed a commit that referenced this pull request Aug 6, 2020
Add telemetry service to the list of services to stop/reset-failed/restart during config load/reload operations.
stepanblyschak pushed a commit to stepanblyschak/sonic-utilities that referenced this pull request Apr 18, 2022
Commits:

02dea43 [202012] Add support for BFD notification (sonic-net#990)
337fd97 Change sonic-buildimage.vs artifact source from CI build to official build. (sonic-net#992)
490f442 Support for cisco-8000 platform for sonic-sairedis/syncd (sonic-net#823)
stepanblyschak pushed a commit to stepanblyschak/sonic-utilities that referenced this pull request Apr 28, 2022
ef0b1fa 2020-07-21 | [config] Restart telemetry service upon config (re)load (sonic-net#992)

Signed-off-by: Guohan Lu <lguohan@gmail.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