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

Docker Demo fails to startup: bad URI error #2610

Closed
jefferya opened this issue Nov 1, 2021 · 3 comments
Closed

Docker Demo fails to startup: bad URI error #2610

jefferya opened this issue Nov 1, 2021 · 3 comments

Comments

@jefferya
Copy link
Contributor

jefferya commented Nov 1, 2021

Describe the bug
Following the Demo instructions with Docker Docker version 20.10.10, build b485636 and docker-compose version 1.29.2 startup of the web container fails with bad URI(is not URI?): "'redis://redis/1'" (URI::InvalidURIError)

To Reproduce
Steps to reproduce the behavior:

  1. Install latest Docker: version 20.10.10
  2. Install latest docker-compose: version 1.29.2
  3. clone repo 5c8bfce (head of the main branch)
  4. docker-compose -f docker-compose.yml up -d fails to start web container with error bad URI(is not URI?): "'redis://redis/1'" (URI::InvalidURIError)

Expected behavior
web container starts normally with connection to redis

Desktop (please complete the following information):
Ubuntu 20.04

Additional context

- REDIS_URL='redis://redis/1'

@murny
Copy link
Contributor

murny commented Nov 1, 2021

My guess is maybe its the quotes in the yaml file?

https://github.com/ualbertalib/jupiter/blob/master/docker-compose.demo.yml#L37

Should be just - REDIS_URL=redis://redis/1

(Production docker compose which UAT uses does the same thing and its working 🤔 . Can check it's version from the .env_deployment dot file)

Edit: Ah saw your commit with this fix, once you PR is up I'll approve it 😄

@jefferya
Copy link
Contributor Author

jefferya commented Nov 1, 2021

New versions of docker-compose, if I understand some of the reports, became subtly more strict when interpreting yaml.

jefferya added a commit that referenced this issue Nov 2, 2021
…e_redis

Fixes: Docker demo Redis bad URI error #2610
@murny
Copy link
Contributor

murny commented Nov 7, 2021

Believe this can be closed now, should be fixed by #2611

@murny murny closed this as completed Nov 7, 2021
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

2 participants