lucasdiedrich / ojs

Open Journal Systems (OJS) is a journal management and publishing system.
GNU General Public License v3.0
19 stars 64 forks source link

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

Open marcbria opened 5 years ago

marcbria commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

OK, going to make this after the upgrade.