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

Update VMClarity init so stack upgrades apply correctly #253

Merged
1 commit merged into from
Apr 14, 2023

Conversation

ghost
Copy link

@ghost ghost commented Apr 14, 2023

Description

This change tweaks the init configs for VMClarity server so that cfn-hup and cfn-init hooks restart the services correctly when the cloud formation stack updates the VMClarity server configuration.

Without this change the cfn-hup and cfn-init notice the change and update the systemd unit files, but the services don't get restarted so they continue to run the old version.

This change also lowers the cfn-hup polling time from 15 minutes to 5 minutes, so that the VMClarityServer notices that the cloudformation has been updated sooner.

Type of Change

[X] Bug Fix
[ ] New Feature
[ ] Breaking Change
[ ] Refactor
[ ] Documentation
[ ] Other (please describe)

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

This change tweaks the init configs for VMClarity server so that cfn-hup
and cfn-init hooks restart the services correctly when the cloud
formation stack updates the VMClarity server configuration.

Without this change the cfn-hup and cfn-init notice the change and
update the systemd unit files, but the services don't get restarted so
they continue to run the old version.

This change also lowers the cfn-hup polling time from 15 minutes to 5
minutes, so that the VMClarityServer notices that the cloudformation has
been updated sooner.
@ghost ghost self-requested a review as a code owner April 14, 2023 13:57
@ghost ghost merged commit c247bee into main Apr 14, 2023
@ghost ghost deleted the cloudformation_upgrade branch April 14, 2023 16:21
This pull request was closed.
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.

None yet

2 participants