Skip to content

Conversation

@mike-bailey
Copy link

Added it so latest docker version is pulled, not two older seemingly random versions. Pulling the older version resulted in me having to pull latest before getting help on an issue, so I figured I'd fix that in the future.

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@mike-bailey
Copy link
Author

I signed it!

@googlebot
Copy link

CLAs look good, thanks!

@mike-bailey mike-bailey changed the title Fix GRR version docs Fix GRR docker version docs May 19, 2017
@mbushkov
Copy link
Collaborator

Thanks for the PR! The reason I asked you to use grr:latest in google/grr#499 is to check that the issue still persists in HEAD (since GetMRU was removed and replaced by an artifact). Normally we recommend to try the release version first, and switch to the HEAD version if needed (i.e. if latest features/changes/bugfixes are needed).

@mike-bailey
Copy link
Author

mike-bailey commented May 22, 2017

Thanks for the reply! Shouldn't the release version be the latest docker version....?

@mbushkov
Copy link
Collaborator

Latest Docker version is GitHub's HEAD. It should work (it passes the automated tests) but doesn't go through a release process.

The release version (currently - 3.1.0.2) is released periodically and goes through a release process: it is tested with a more comprehensive integration testing suite and should be more reliable.

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.

3 participants