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

Allow user to set build.image: testing in the config file #6676

Merged
merged 1 commit into from
Mar 3, 2020

Conversation

humitos
Copy link
Member

@humitos humitos commented Feb 19, 2020

We have a new testing image now that we can recommend to users when they need something extra that it's now in our current image, but will be available in the following image.

This allow us to not have to do work manually (like modifying container_image in the db), clearly communicate to the user that they are using something that it's not released yet and also allow them to opt-in/out when they want.

@humitos humitos requested review from ericholscher and a team February 19, 2020 11:28
@ericholscher
Copy link
Member

Believe @agjohnson had some reservations about letting users set this "forever". I think I'm +0 on it, so that users can opt into a new image without having to pester us to setup a feature flag -- I think it will lead to less support over time than folks who have broken builds because of testing, but not sure.

@agjohnson
Copy link
Contributor

Yeah, a bit. I worry users are opting in to an image that solves a problem they are having, where the testing image is much more of a beta test area for us. I'd much rather we increase release cycles and tell the user "use X image when it's released, which won't be long", not "become a beta tester for our docker images".

Copy link
Contributor

@agjohnson agjohnson left a comment

Choose a reason for hiding this comment

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

Changes look good, and I'm solidly on the fence on needing a testing image in general. It feels like heading in the opposite direction from shortening release cycles on the image. However, our issues here are coming from the python3.8 upgrade, and this has introduced significant friction to docker image releases. It seems we have to keep more images around as python3.8 was a significant upgrade. I think switching our docker image release pattern from semver is probably the ultimate answer here, so I'm okay with a change like this in the interim

Copy link
Member

@ericholscher ericholscher left a comment

Choose a reason for hiding this comment

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

Agreed, docker image versioning, and custom docker images is definitely the future, so I'm 👍 this for now to work around our current issue.

@ericholscher ericholscher merged commit 31c9f50 into master Mar 3, 2020
@ericholscher ericholscher deleted the humitos/allow-use-testing-image branch March 3, 2020 14:55
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