-
Notifications
You must be signed in to change notification settings - Fork 20
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
Removed unnecessary EL7 commands and optimized CW agent download script #17
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@eemperor Quick query: did you validate that the CWAgent works as expected on EL6? That it comes with a systemd unit-file but not a legacy-init file is somewhat concerning?
One of my tests was EL6 on CentOS but not RedHat. Let me double check. |
Cool. CentOS should be adequately representative Did the CWA processes come back online after a reboot of the CentOS 6.9 instance? If so, did the installer create |
The CWA process does come back running after reboot following completion of the Cfn stack. I will have to do additional testing to speak on the resilincy across multiple reboots. It appears the CWA installer populates both a |
Good to know: I hadn't noticed the But, yeah, the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚢
The default CloudWatch agent config script,
amazon-cloudwatch-agent-ctl
, already handles setting up the CloudWatch agent service in EL6 and EL7. The two systemd commands were not required.In additon, the CloudWatch agent download steps have been optimized into fewer commands.
This PR addresses the following issues:
#15
#16