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

fixed pull notification for tagged images #203

Merged
merged 1 commit into from Oct 24, 2014

Conversation

hobofan
Copy link
Contributor

@hobofan hobofan commented Aug 19, 2014

The image pull action now uses image_and_tag_arg instead of only the image_name.

Only comparing the id of the image with the image_name will result in no update of the resource when you are actually pulling a tagged image and therefore no notify actions will be raised.

@joshpasqualetto
Copy link

+1 to this fix, I am now using this fork/branch as the current one is non functional in any meaningful deployment scenario for me.

If you use a tag with the docker_image resource and have any notifies/subscribes -- nothing will work without this fix. For my use case having a tag and not downloading the world from my registry is important.

Without this fix the entire cookbook seems broken when using tags and will result in much frustration. Please merge.

@djdefi
Copy link

djdefi commented Sep 17, 2014

+1 need some of these fixes merged. We are about to scrap using this cookbook and switch to Ansible's docker module or something else that is being maintained.

@tduffield
Copy link
Contributor

👍

tduffield pushed a commit that referenced this pull request Oct 24, 2014
fixed pull notification for tagged images
@tduffield tduffield merged commit d5543f4 into sous-chefs:master Oct 24, 2014
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

4 participants