-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
revert back pulling CLI if local image older than 24h #3695
Conversation
We need to do testing on this again - changed the logic. |
Fixes #3684 |
@@ -381,12 +381,14 @@ function verify_nightly_accuracy() { | |||
# than the one stored on DockerHub. | |||
if is_nightly; then | |||
|
|||
REMOTE_NIGHTLY_JSON=$(curl -s https://hub.docker.com/v2/repositories/${CHE_IMAGE_NAME}/tags/nightly/) | |||
# REMOTE_NIGHTLY_JSON=$(curl -s https://hub.docker.com/v2/repositories/${CHE_IMAGE_NAME}/tags/nightly/) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can drop the commented line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
* revert back pulling CLI if local image older than 24h
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/1609/ |
* revert back pulling CLI if local image older than 24h
after I've reverted f7ca10c this I've also removed logic which do pull CLI and exit if CLI is older than 24h
so I'm adding it back with this PR.