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

Fix sonobuoy image tag and pull policy #732

Merged
merged 2 commits into from
Aug 1, 2018

Conversation

ivan4th
Copy link
Contributor

@ivan4th ivan4th commented Jul 31, 2018

Use build tag if available.
Allow specifying the tag.
Don't use imagePullPolicy: Never


This change is Reviewable

Copy link
Contributor

@jellonek jellonek left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 4 of 4 files at r1.
Reviewable status: 0 of 2 LGTMs obtained, and 1 stale

Copy link
Contributor

@pigmej pigmej left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 4 of 4 files at r1.
Reviewable status: 1 of 2 LGTMs obtained, and 1 stale

Use build tag if available.
Allow specifying the tag.
Don't use `imagePullPolicy: Never`
@jellonek jellonek force-pushed the ivan4th/fix-sonobuoy-image-tag branch from 5e55cb6 to 057275e Compare August 1, 2018 12:16
Copy link
Contributor

@jellonek jellonek left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 1 files at r2.
Reviewable status: 0 of 2 LGTMs obtained, and 2 stale

Copy link
Contributor

@pigmej pigmej left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 1 files at r2, 1 of 1 files at r3.
Reviewable status: 0 of 2 LGTMs obtained, and 2 stale

@pigmej pigmej merged commit b1d5cda into master Aug 1, 2018
@pigmej pigmej deleted the ivan4th/fix-sonobuoy-image-tag branch August 1, 2018 14:00
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