Skip to content

[201911][Monit] Install and enable the unit generate_monit_config.service#7558

Merged
yozhao101 merged 1 commit intosonic-net:201911from
yozhao101:install_generate_monit_config_unit
May 7, 2021
Merged

[201911][Monit] Install and enable the unit generate_monit_config.service#7558
yozhao101 merged 1 commit intosonic-net:201911from
yozhao101:install_generate_monit_config_unit

Conversation

@yozhao101
Copy link
Copy Markdown
Contributor

@yozhao101 yozhao101 commented May 7, 2021

Signed-off-by: Yong Zhao yozhao@microsoft.com

Why I did it

The service file generate_monit_config.service is used to generate the Monit configuration file from template. I also should install this service file and enable it.

How I did it

I appended this service file name at the end of /etc/sonic/generated_services.conf.

How to verify it

I verified this on the device str2-7260cx3-acs-1.

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

  • 201811
  • [x ] 201911
  • 202006
  • 202012

Description for the changelog

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

Signed-off-by: Yong Zhao <yozhao@microsoft.com>
@yozhao101 yozhao101 requested a review from jleveque May 7, 2021 15:06
@yozhao101 yozhao101 changed the title [201911][Monit] Install the unit of generate_monit_config.service. [201911][Monit] Install and enable the unit file of generate_monit_config.service. May 7, 2021
@yozhao101 yozhao101 changed the title [201911][Monit] Install and enable the unit file of generate_monit_config.service. [201911][Monit] Install and enable the unit generate_monit_config.service. May 7, 2021
@yozhao101 yozhao101 changed the title [201911][Monit] Install and enable the unit generate_monit_config.service. [201911][Monit] Install and enable the unit generate_monit_config.service May 7, 2021
@yozhao101 yozhao101 merged commit f0bbd9d into sonic-net:201911 May 7, 2021
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.

2 participants