Skip to content
This repository has been archived by the owner on Aug 16, 2024. It is now read-only.

Deprecate che-golang-1.10 in favor of che-golang-1.12 #126

Merged
merged 1 commit into from
Oct 23, 2019

Conversation

amisevsk
Copy link
Contributor

What does this PR do?

Remove che-golang-1.10 from list of arbitrary UID patched images in favor of che-golang-1.12. This PR will stop nightly/release builds of that image and make new workspaces created using to go devfile use 1.12. We still need to decide what to do with the existing che-golang-1.10 images on quay.io, since old workspaces may depend on it.

What issues does this PR fix or reference?

eclipse-che/che#14265

Signed-off-by: Angel Misevski <amisevsk@redhat.com>
@nickboldt
Copy link
Contributor

Two thoughts here

  • For CRW 2 we'll be using golang 1.11 as there's no 1.12 in RHCC
  • You said "deprecate" but then you deleted stuff. That sounds more like "remove" than "deprecate". :) Where's the grace period? (FWIW, I like the idea of just removing the old stuff, but then you should rename this issue to "remove" rather than "deprecate". :) )

Copy link
Contributor

@nickboldt nickboldt left a comment

Choose a reason for hiding this comment

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

+1, see comments

@amisevsk
Copy link
Contributor Author

It doesn't remove it per se, but does remove it from the nightly build; the image will still be available to be referenced by old workspaces. With the changes, new workspaces would automatically use 1.12. I wouldn't call this a "remove", since remove implies that che-golang-1.10 will no longer be available.

In terms of deprecation procedure, we don't currently have one for these images. We'd need a way to notify users of old golang workspaces that we're no longer building the image, and I don't know a good way of doing that. Part of the issue is that the devfile registry only serves devfiles for workspace creation; already-existing workspaces store their own devfiles and so the registry is completely disconnected from the process.

@nickboldt WDYT about adding a #deprecated to the image in base_images? My main goal here is to signal to people that we should avoid using 1.10 unless necessary, since maintaining both 1.12 and 1.10 is weird.

@nickboldt
Copy link
Contributor

I'm good with you deleting it completely from here.

Maybe you could add a note on the registry's image listing, like we did here:

https://quay.io/repository/eclipse-che/che7-java11-maven?tab=info ??

Then it's labelled at the source, and the build script/process doesn't have to put #deprecated on stuff that has died and won't be coming back.

@ibuziuk ibuziuk merged commit 22a9f31 into eclipse-che:master Oct 23, 2019
@ibuziuk
Copy link
Member

ibuziuk commented Oct 23, 2019

@amisevsk could you please add deprecation note to the https://quay.io/repository/eclipse/che-golang-1.10

@amisevsk
Copy link
Contributor Author

sparkoo pushed a commit to sparkoo/che-devfile-registry that referenced this pull request Dec 4, 2020
Add CircleCI config
Add CircleCI badge to readme
Signed-off-by: Oleksandr Garagatyi <ogaragat@redhat.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants