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

Feature request: Make it run over play-with-docker #12

Open
marcbria opened this issue Mar 6, 2019 · 5 comments
Open

Feature request: Make it run over play-with-docker #12

marcbria opened this issue Mar 6, 2019 · 5 comments

Comments

@marcbria
Copy link
Contributor

marcbria commented Mar 6, 2019

As you know, play-with-docker (PWD) is a great platform to make software demos.

Some projects include a button to build a full stack and test before create your own infrastructure:
https://hub.docker.com/r/hackmdio/hackmd/

I think it could be a great addition to you README.md and I wanted to send you a PR to your documentation but I tested and it's not working.

It's supposed to be as easy as a link to your docker-compose.yml file in raw format, as follows:
https://labs.play-with-docker.com/?stack=https://raw.githubusercontent.com/lucasdiedrich/ojs/3.1.1-4/docker-compose.yml

Any clue about this is not working?

@lucasdiedrich
Copy link
Owner

Marc, adding PWD to the docs will be an awesome upgrade, i created an test branch i made some changeds at the docker-compose.yml.

You can test at: https://labs.play-with-docker.com/?stack=https://raw.githubusercontent.com/lucasdiedrich/ojs/test/docker-compose.yml

It seems that`s is working now, i just have to add the env variables again. Can you test it??

Going to look the other issue this afternoon. Thanks.

@marcbria
Copy link
Contributor Author

marcbria commented Mar 7, 2019

Thanks a lot Lucas.

Looks like PWD don't like variable expansion and you need to hardcode the variables. :-(

What about keeping a docker-compose.yml with variables as before (for development and production environments) and create a copy (stack.yml or if it works, demo.yml) with hardcoded vars, just for PWD?

I tested and now works better than before, because the full stack is build, but when you visit the url of the site shows a WSOD till timeout.

No clue about why it happens. I will review it next week or so. We are so close.

Thanks a lot for your work and time,
m.

@lucasdiedrich
Copy link
Owner

Marc, there is a reasonable for not working with .env variable, as we are just pointing to the docker-compose.yml when PWD loads this file he doesn't know the existence of the .env file over the repository.

I created this file just to test that out, i'm going to rollback it and just remove the .env lines, i think its going to work then. I'm thinking about creating an examples folder and put this docker-compose files inside it, what do you think?

Thanks.

@marcbria
Copy link
Contributor Author

marcbria commented Mar 7, 2019

I was expecting some magic from PWD but yes, it makes a lot of sense.
An examples folder with "hardcoded variables" docker-compose is a perfect solution.

But please, think in this as a "feature request"... I mean, if I were you, I will focus on the new release upgrade instead of this. ;-)

Please, let me know if you need help or you want a PR.

Thanks!

@lucasdiedrich
Copy link
Owner

OK, going to make this after the upgrade.

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

No branches or pull requests

2 participants