Skip to content

Migrate current compose examples to PWD capable stacks #941

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

Merged
merged 3 commits into from
Jun 21, 2017

Conversation

marcosnils
Copy link
Contributor

Added stack files for the current official images:

  • joomla
  • drupal
  • geonetwork
  • owncloud

@tianon
Copy link
Member

tianon commented Jun 19, 2017

cc image maintainers @TimWolla @doublebyte1 @mbabker @brianteeman -- extra eyes on your relevant parts (or the whole thing!) of this would be A+ appreciated 👍 ❤️

@tianon
Copy link
Member

tianon commented Jun 19, 2017

Just to thread the needle, this relates to the discussion in #912 and the implementations in #913 and #933.

@mbabker
Copy link
Contributor

mbabker commented Jun 19, 2017

Looks fine to me.

@@ -30,6 +30,10 @@ Then point your web server to port 9000 of the container.

Note: This exposes the FastCGI socket to the Internet. Make sure to add proper firewall rules or use a private Docker network instead to prevent a direct access.

## %%STACK%%

Run `docker stack deploy -c stack.yml %%REPO%%` (or `docker-compose -f stack.yml up`), wait for it to initialize completely, and visit `http://swarm-ip:8080`, `http://localhost:8080`, or `http://host-ip:8080` (as appropriate).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not really sure whether this really provides a value-add for Adminer: Usually you would use an existing database, instead of creating a new one to use.

The online demo using this stack.yml works fine, from what I've seen though.

Copy link
Contributor Author

@marcosnils marcosnils Jun 19, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not really sure whether this really provides a value-add for Adminer: Usually you would use an existing database, instead of creating a new one to use.

@TimWolla thx for the feedback. What about people that just want to try it the first time to see if it'll suite their needs?. Another use-case is whenever the image gets updated you can just launch the new version with a single click to try new features.

IMHO the fact that you can just launch it with a single click and have it running without having to do anything at all is quite powerful.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@marcosnils Is this paragraph required to the online demo?

In any case, I don't object this change, but this paragraph refers to a downloaded swarm.yml which IMO is unnecessary clutter for adminer.

@brianteeman
Copy link
Contributor

Thanks. Looks good

@@ -30,6 +30,10 @@ Then point your web server to port 9000 of the container.

Note: This exposes the FastCGI socket to the Internet. Make sure to add proper firewall rules or use a private Docker network instead to prevent a direct access.

## %%STACK%%
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This headline is either at the wrong position or of the wrong size.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed!.

@doublebyte1
Copy link
Contributor

@tianon @marcosnils This looks fine to me, Thanks!

@yosifkit yosifkit merged commit 78dce8e into docker-library:master Jun 21, 2017
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

Successfully merging this pull request may close these issues.

7 participants