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

docker image cron update #314

Merged
merged 1 commit into from
Oct 8, 2019
Merged

docker image cron update #314

merged 1 commit into from
Oct 8, 2019

Conversation

lwsanty
Copy link
Member

@lwsanty lwsanty commented Oct 4, 2019

This PR contains changes to travis and Makefile configs for periodical docker image update using travis ci cron jobs

Behavior

  1. if TRAVIS_EVENT_TYPE is cron then
    a) try to get tag using $( git describe --exact-match "$(git rev-parse HEAD)
    b) if tag is not empty run make push-drivers
    c) in push-drivers: if TRAVIS_EVENT_TYPE is cron pull base bblfshd image instead of rebuilding it from the scratch
  2. else build as always

Changes to docker tags:

  • timestamp is added to the version tag, so the final image will look like bblfsh/bblfshd:v2.14.0-drivers-2019-10-04

Problems:

closes #309

Signed-off-by: lwsanty [email protected]


This change is Reviewable

@lwsanty lwsanty requested a review from dennwc October 4, 2019 11:30
@lwsanty lwsanty requested a review from a team as a code owner October 4, 2019 11:30
@lwsanty lwsanty self-assigned this Oct 4, 2019
This PR contains changes to travis and Makefile configs for periodical docker image update using travis ci cron jobs

Behavior
1) if TRAVIS_EVENT_TYPE is cron then
  a) try to get tag using `$( git describe --exact-match "$(git rev-parse HEAD)`
  b) if tag is not empty run make push-drivers
  c) in push-drivers: if TRAVIS_EVENT_TYPE is cron pull base bblfshd image instead of rebuilding it from the scratch
2) else build as always

Changes to docker tags:
- timestamp is added to the version tag, so the final image will look like bblfsh/bblfshd:v2.14.0-drivers-2019-10-04

Problems:
- afaik deploy will not be triggered on cron job because tags are not accessible with this event type travis-ci/travis-ci#8146 ... cannot check it on practice thus 1.a) workaround has been implemented as suggested travis-ci/travis-ci#8146 (comment)

closes #309

Signed-off-by: lwsanty <[email protected]>

change tag datetime to date only

Signed-off-by: lwsanty <[email protected]>
@lwsanty lwsanty force-pushed the image-cron-update branch from 8a4fcb3 to e2988a9 Compare October 4, 2019 12:44
Copy link
Member

@dennwc dennwc left a comment

Choose a reason for hiding this comment

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

Reviewed 2 of 2 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@dennwc dennwc merged commit 49ac0e4 into master Oct 8, 2019
@dennwc dennwc deleted the image-cron-update branch October 8, 2019 08:16
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.

Periodically update image with pre-installed drivers
3 participants