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

[BUG] post_renew not active when use_package:True #81

Open
arthurzenika opened this issue May 19, 2021 · 3 comments
Open

[BUG] post_renew not active when use_package:True #81

arthurzenika opened this issue May 19, 2021 · 3 comments
Labels

Comments

@arthurzenika
Copy link

Your setup

Formula commit hash / release tag

git hash : df1b31e

Pillar / config used

local:
    ----------
    post_renew:
        ----------
        cmds:
            - systemctl reload apache2
    config:
        ----------
        rsa-key-size:
            4096
    domainsets:
        ----------
        www:
            - www.test.localhost
            - test.localhost


Bug details

Describe the bug

post_renew is documented in pillar.example, but seems only used in letsencrypt/files/renew_letsencrypt_cert.sh.jinja which is not deployed when use_package: True (default) is used.

Expected behaviour

Deploy hooks

Attempts to fix the bug

Not yet

Additional context

@arthurzenika
Copy link
Author

This might just be a documentation bug, the renew hook might be automatic with certbot plugins.

@javierbertoli
Copy link
Member

@arthurlogilab I agree it's a documentation error. I'm using

letsencrypt:
  config:
    deploy-hook = systemctl reload nginx

successfully

@heimdull
Copy link

I was just hit with this and should someone not update the piller.example? or maybe fix the state so that it puts what you put in post_renew in renewal-hooks ? following the example currently creates a broken setup IF you are using use_package: True

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants