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

- add support for pulling an image once per (reactor) build #512

Merged
merged 1 commit into from
Jul 19, 2016

Conversation

jgangemi
Copy link
Collaborator

resolves #504 and allows images to be auto-pulled once for an entire build.

@rhuss
Copy link
Collaborator

rhuss commented Jul 13, 2016

Thanks a lot, will have look soon.

@jgangemi jgangemi force-pushed the jae/auto-pull-once branch 2 times, most recently from 6c50392 to b41154e Compare July 13, 2016 15:52
Signed-off-by: Jae Gangemi <jgangemi@gmail.com>
@jgangemi jgangemi force-pushed the jae/auto-pull-once branch from b41154e to 136bc01 Compare July 13, 2016 15:56
@jgangemi
Copy link
Collaborator Author

any updates on this?

@rhuss
Copy link
Collaborator

rhuss commented Jul 19, 2016

Uff, everyone is on PTO except for me. Tons of stuff todo, but yeah, you are right, this should be a bit upper in the queue ;-) will try to do it this evening. If not, please keep nagging me ;-)

@jgangemi
Copy link
Collaborator Author

cool - i'm going to try and get the push retry finished off today (i just need to do the tests) as well.


- Allow images to only be pulled once per build (useful for reactor projects) ([#504](https://github.com/fabric8io/docker-maven-plugin/issues/504))

>>>>>>> Stashed changes
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops ;-)

@rhuss
Copy link
Collaborator

rhuss commented Jul 19, 2016

Thanks a lot, LGTM. If you don't mind (and agree to my nit-picking ;-) I'd fix the minor gotchas myself, in order to speed things up.

@rhuss
Copy link
Collaborator

rhuss commented Jul 19, 2016

And yes, thanks to get into jmockit. Not easy in the beginning, I know, but quite powerful (especially when it comes to partial mocks).

Also kudos for providing unit tests and docs from the very beginning. Not the usual case.

@rhuss rhuss merged commit 136bc01 into fabric8io:integration Jul 19, 2016
rhuss added a commit that referenced this pull request Jul 19, 2016
rhuss added a commit that referenced this pull request Jul 20, 2016
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