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

dokku_config: allow config:set --global #52

Open
decentral1se opened this issue Mar 8, 2020 · 4 comments
Open

dokku_config: allow config:set --global #52

decentral1se opened this issue Mar 8, 2020 · 4 comments

Comments

@decentral1se
Copy link
Member

dokku config:set --global [email protected]

AFAICT, it isn't possible now and only scoped to an app?

Up for accepting a PR to allow for global config setting?

Very useful for initial provisioning of a host when letsencrypt is installed also.

@decentral1se
Copy link
Member Author

API could be:

- name: set LE global email
  dokku_config:
    global: True
    config:
      DOKKU_LETSENCRYPT_EMAIL: [email protected]

@josegonzalez
Copy link
Member

Partial PR here: #51

But an alternate implementation is always welcome.

@nerg4l
Copy link
Contributor

nerg4l commented Oct 3, 2020

By looking at the linked PR and the code it seems it is doable at the moment by setting --global for the app name. It is a dirty hack but should work until a proper implementation.

- name: set LE global email
  dokku_config:
    app: '--global'
    config:
      DOKKU_LETSENCRYPT_EMAIL: [email protected]

@iloveitaly
Copy link
Contributor

Worth noting that DOKKU_LETSENCRYPT_EMAIL doesn't work anymore, you'll need to do something like

    - name: set letsencrypt email and request certificate
      block:
        - name: Your pre-task (dependency for dokku_letsencrypt)
          ansible.builtin.shell: dokku letsencrypt:set app email email@domain

        - name: letsencrypt
          dokku_letsencrypt:
            app: *appname

Would be great if we could add an email param to dokku_letsencrypt!

iloveitaly added a commit to iloveitaly/ansible-dokku that referenced this issue Sep 23, 2023
this isn't a valid option anymore. There's not a replacement for this using the standard ansible dokku stuff
you need to run a shell script.

dokku#52 (comment)
iloveitaly added a commit to iloveitaly/ansible-dokku that referenced this issue Nov 6, 2023
this isn't a valid option anymore. There's not a replacement for this using the standard ansible dokku stuff
you need to run a shell script.

dokku#52 (comment)
ltalirz added a commit that referenced this issue Nov 7, 2023
* fix: remove DOKKU_LETSENCRYPT_EMAIL

this isn't a valid option anymore. There's not a replacement for this using the standard ansible dokku stuff
you need to run a shell script.

#52 (comment)
---------

Co-authored-by: Leopold Talirz <[email protected]>
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

No branches or pull requests

4 participants