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

Multiple Sidekick functionality added. #33

Merged
merged 1 commit into from
Feb 27, 2018
Merged

Multiple Sidekick functionality added. #33

merged 1 commit into from
Feb 27, 2018

Conversation

Rakhmanov
Copy link
Contributor

Adding ability to update sidekick images.
In following format for multiple sidekicks: --sidekick nginx nginx:latest --sidekick node node:alpine
Or just one flag for single.

@Rakhmanov
Copy link
Contributor Author

To test it you can use my dockerhub image.

This deployment step from my drone.yml:

deploy:
image: rakhmanov/drone-rancher
url: yourRancherAPIURL
access_key: ***
secret_key: ***
service: some/service
docker_image: php php
sidekick: nginx nginx:latest
sidekick: storage tianon/true

@josmo
Copy link
Owner

josmo commented Feb 26, 2018

@Rakhmanov thanks for the PR, I'll take a look at it his week and merge it in :)

@Rakhmanov
Copy link
Contributor Author

@josmo You are very welcome, let me know.

@josmo josmo merged commit ce418ca into josmo:master Feb 27, 2018
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.

None yet

2 participants