-
-
Notifications
You must be signed in to change notification settings - Fork 26
Update readme to have a more complete compose file #5
Conversation
command: storm supervisor | ||
command: > | ||
storm supervisor | ||
-c storm.zookeeper.servers="[\"zookeeper\"]" |
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.
This is done by default if no config provided. Please see https://github.com/31z4/storm-docker/blob/master/1.0.2/docker-entrypoint.sh#L15-L16
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.
Nice didn't spot that.
-c nimbus.seeds="[\"nimbus\"]" | ||
depends_on: | ||
- nimbus | ||
- zookeeper |
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 don't think the UI depends on Zookeeper.
- zookeeper | ||
links: | ||
- nimbus | ||
- zookeeper |
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.
Linking with Zookeeper is unnecessary.
@edbo could you please clarify on this
What exactly do you mean? |
I wasn't able to upload a topology to the storm cluster without setting a |
@edbo could you please create an issue for this describing all the details? Or do it right here. |
I get the error when using a command like tihs:
Setting the hostname to nimbus gets it working. I'm about to tweak it a bit to see if I can drop the other bits. |
I realised after your message about the config settings you were setting that I was running your new docker compose file against your old docker image that didn't have those values set. Was confusing the hell out of me. Anyway, I'm closing this now in favour of two simpler ones that separate the changes (#6 & #7). |
I changed the docker-compose a bit so you can also see the ui and it includes a fix that allows for uploading to the cluster more easily.