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

Make aah app reload configuration Signal configurable and also propagate this event #218

Closed
jeevatkm opened this issue Oct 20, 2018 · 1 comment
Assignees
Labels
Projects

Comments

@jeevatkm
Copy link
Member

@jeevatkm jeevatkm commented Oct 20, 2018

The goal is to make aah app reload/reinitialize signal configurable. Currently aah app listens to SIGHUP for reinitializing app without restart on non-dev env profile (it was implemented in #123).

However in real world usage aah user need this signal value configurable and defaults to SIGHUP. So that aah user can use the signal they want e.g. SIGUSR1 and SIGUSR2.

Supported signals for reload configuration would be SIGHUP (default), SIGUSR1 and SIGUSR2.

  • Configuration signal option
  • Configuration for enable/disable hot-reload
  • Propagate this event
  • Documentation
@jeevatkm jeevatkm added this to the v0.12.0 Milestone milestone Oct 20, 2018
@jeevatkm jeevatkm added this to v0.12.0 - Iteration in aah Roadmap via automation Oct 20, 2018
@jeevatkm jeevatkm self-assigned this Nov 2, 2018
jeevatkm added a commit that referenced this issue Nov 2, 2018
…ignal and event callback
@jeevatkm jeevatkm moved this from v0.12.0 - Iteration to v0.12.0 - In Progress in aah Roadmap Nov 2, 2018
jeevatkm added a commit to go-aah/docs that referenced this issue Nov 3, 2018
@jeevatkm
Copy link
Member Author

@jeevatkm jeevatkm commented Nov 3, 2018

Done 😄

@jeevatkm jeevatkm closed this Nov 3, 2018
aah Roadmap automation moved this from v0.12.0 - In Progress to v0.12.0 - Completed Nov 3, 2018
@jeevatkm jeevatkm moved this from v0.12.0 - Completed to Released to Audience in aah Roadmap Dec 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
aah Roadmap
  
Released to Audience
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.