-
Notifications
You must be signed in to change notification settings - Fork 26
Add dev docker compose #50
Add dev docker compose #50
Conversation
908fd1d to
55b7caf
Compare
156da6f to
cc283c3
Compare
|
Once we have #48 and this PR anyone will be able to start Apache Kibble by simply running: This should be enough for development purposes 👍 |
|
Great. I like it. |
00e7e91 to
1902ce3
Compare
|
|
||
| # Apache Kibble web ui server | ||
| ui: | ||
| image: nginx:latest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I decided to use nginx as I'm more familiar with it than with httpd
|
@sharanf @Humbedooh happy to get a review from you 🎉 |
|
Hi all, is there anything that should be done to get this PR merged? @Humbedooh @sharanf |
|
@turbaszek I see #48 is already merged and it is showing a conflct here. Please can you take a look. |
1902ce3 to
1523b11
Compare
|
Will need #58 to make CI green |
1523b11 to
77333f4
Compare
|
@Humbedooh @sharanf this one is rebased and ready for your review |
|
@turbaszek - still looking at this as I haven't been able to get it to run locally on my machine yet. |
No problem. You may find this tool useful: Once you have github cli tool it's enough to do in your local Apache Kibble repository: to checkout this PR and test everything locally 👍 |
Thanks @turbaszek, I installed the github cli tool and am still getting an error. This time it is "ERROR: pull access denied for kibble, repository does not exist or may require 'docker login': denied: requested access to the resource is denied" so it could be something I have missed as part of the setup. I will have another try later in the week but please don't let me hold things up for you if you are happy things are working OK. |
@sharanf good catch! I will fix it as this is a real problem |
Hi @turbaszek I have had another go and it got a lot further this time. |
Sounds like ElasticSearch isn't running... |
docker-compose-dev.yaml
Outdated
| build: | ||
| context: . | ||
| dockerfile: Dockerfile.dev | ||
| command: bash -c "sleep 25 && python setup/setup.py -e elasticsearch -a -k" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds like ElasticSearch isn't running...
That's why there is this sleep... it's a hack but usually worked. I can figure a better health check (this is tricky in docker compose) or we can increase this value for time being
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Or I can add some loop in python code to retry it few times
|
Should be good now: @sharanf you may need to do |
|
@turbaszek I did the the build setup and got a whole load of text followed by: Successfully built 46142dc0a961 Then ran the docker-compose -f docker-compose-dev.yaml up setup |
|
@sharanf can I ask you to do two things:
Apart from that can you do: This should show us if elasticsearch is working 👍 |
|
As discussed on #kibble slack we are good with merging this one and fixing potential problems in follow up PRs 👌 |
fixup! Add development docker-compose fixup! fixup! Add development docker-compose fixup! fixup! fixup! Add development docker-compose
3b37a97 to
a57a680
Compare
I would love to make this to work out of the box but this will require #48. Otherwise, the setup will be so confusing...
Depends on #48
Related to #40