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

Deprecate plugin #15

Closed
angusfretwell opened this issue Jan 22, 2015 · 6 comments
Closed

Deprecate plugin #15

angusfretwell opened this issue Jan 22, 2015 · 6 comments

Comments

@angusfretwell
Copy link

v0.3.10 of Dokku added a native domain management plugin. I've found that this plugin actually breaks deployment of apps on newer versions of Dokku. Would be good if you could add a warning about this in the README.

Thanks for making this awesome plugin, though. It served me well until domains became a standard feature of Dokku 😸

@DenisIzmaylov
Copy link

Could you please explain? I didn't found any related information about 'native' plugin.

@blakeembrey
Copy link

@DenisIzmaylov I've been using the latest version of Dokku and it exists for me.

    domains:add <app> DOMAIN                        Add a custom domain to app
    domains <app>                                   List custom domains for app
    domains:clear <app>                             Clear all custom domains for app
    domains:remove <app> DOMAIN                     Remove a custom domain from app

@marblegravy
Copy link

I can confirm that this plugin was causing my deploy issues.

Removing it (rm -rf /var/lib/dokku/pluginsdomains-plugin/) fixed me up.

@wmluke
Copy link
Owner

wmluke commented Mar 2, 2015

Thanks for the heads up guys. I will add a deprecation notice to the readme.

To Uninstall

Remove the following files:

  • /etc/nginx/conf.d/dokku-domains.conf
  • /home/dokku/*/nginx-domains.conf

Something like this should work, but I have NOT tested...

$ rm /etc/nginx/conf.d/dokku-domains.conf
$ find /home/dokku -wholename '*/nginx-domains.conf' -delete

Additionally, it looks like that upon upgrading the new built-in domains plugin will replace this deprecated domains plugin, but you should verify that everything looks right in /var/lib/dokku/plugins/domains ie PLUGINS_PATH.

Please comment in this thread how well this works for you or how your mileage varies.

Thanks,
Luke

@josegonzalez
Copy link

@wmluke Note, this and other plugins were the inspiration for the official version. There are some things we feel make dokku a great tool out of the box, and domain handling is one of them. Thanks for all your hard work!

@wmluke
Copy link
Owner

wmluke commented Mar 17, 2015

Hey @josegonzalez,

Thanks for the shoutout. In all honesty, I suspected that my simple plugin wasn't keeping up with the needs of users, so I was very pleased to see a domains like plugin built into 0.3.10. The new built-in support looks great!

Cheers,
Luke

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

6 participants