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: variables for scale: id, hostname #1006

Closed
wants to merge 1 commit into from

Conversation

ioggstream
Copy link

This patch enables parametric hostnames and commands when using scale
avoiding all bash variables expansion issues.

# docker-compose scale test=4
test:
  image: busybox:latest
  hostname: test-%%id%%
  command: runme.sh %%hostname%%

Signed-off-by: Roberto Polli robipolli@gmail.com

Signed-off-by: Roberto Polli <robipolli@gmail.com>
@StefanScherer
Copy link
Member

@ioggstream +1 for that! I'm struggling giving another container an URL to a scaled container with the correct host name. I'll try your PR.

But the Jenkins Compose-PR build seems to have problems:

CONFLICT (content): Merge conflict in compose/service.py

Perhaps you have to rebase your PR.

@ioggstream
Copy link
Author

@StefanScherer rebased and nosetesting... hopefully you'll have the patch in a while

@ioggstream ioggstream closed this Mar 18, 2015
@ioggstream ioggstream deleted the scale_variables branch March 18, 2015 17:09
@ioggstream
Copy link
Author

@StefanScherer: Moved here #1131

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

Successfully merging this pull request may close these issues.

2 participants