Skip to content
Snippets Groups Projects

Disable crond if LetsEncrypt disabled

Merged Stan Hu requested to merge sh-fix-disable-crond into master

Previously the Omnibus-provided crond would still run if LetsEncrypt were enabled and then disabled later. This also goes for the auto-renew configuration. In both cases, crond is not needed, so we can disable the service if LetsEncrypt is not used. This also prevents stale configuration files from lingering.

While this isn't technically a regression, the upgrade of go-crond exposed this problem because the configuration flags changed, and Omnibus was no longer managing the runit files.

Closes #5541 (closed)

Edited by Stan Hu

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Stan Hu added 1 commit

    added 1 commit

    • f6d5dc45 - Disable crond if LetsEncrypt disabled

    Compare with previous version

  • Stan Hu added 1 commit

    added 1 commit

    • c607a861 - Disable crond if LetsEncrypt disabled

    Compare with previous version

  • Stan Hu resolved all threads

    resolved all threads

  • Stan Hu changed title from Disable crond if LetsEncrypt auto-renew not enabled to Disable crond if LetsEncrypt disabled

    changed title from Disable crond if LetsEncrypt auto-renew not enabled to Disable crond if LetsEncrypt disabled

  • Stan Hu added 1 commit

    added 1 commit

    • 99c068a4 - Disable crond if LetsEncrypt disabled

    Compare with previous version

  • Stan Hu changed the description

    changed the description

  • Stan Hu assigned to @balasankarc and unassigned @stanhu

    assigned to @balasankarc and unassigned @stanhu

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading