Skip to content

chore: upgrade udi/devfile to use UBI9 based image #604

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

Merged
merged 1 commit into from
Apr 29, 2025
Merged

chore: upgrade udi/devfile to use UBI9 based image #604

merged 1 commit into from
Apr 29, 2025

Conversation

svor
Copy link
Collaborator

@svor svor commented Apr 25, 2025

Description of Changes

This PR updates image that is used in UDI stack.

quay.io/devfile/universal-developer-image:ubi8-latest is not supported anymore, it was replaced by UBI9 based image.
For more details check eclipse-che/che#23034.

Related Issue(s)

eclipse-che/che#23034

Tests Performed

Tested on Eclipse Che

screenshot-github_com-2025_04_25-12_03_19

How To Test

Workspace should start from the udi/devfile.yaml

@svor svor requested review from ibuziuk, l0rd and a team as code owners April 25, 2025 09:06
@thepetk
Copy link
Contributor

thepetk commented Apr 28, 2025

Looks good to me! Needs approval from @devfile/che-team and from a stack owner (@l0rd or @ibuziuk) :)

@thepetk
Copy link
Contributor

thepetk commented Apr 28, 2025

/hold

@openshift-ci openshift-ci bot added the lgtm Looks good to me label Apr 28, 2025
Copy link

openshift-ci bot commented Apr 28, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: ibuziuk, svor
Once this PR has been reviewed and has the lgtm label, please assign johnmcollier for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@thepetk
Copy link
Contributor

thepetk commented Apr 28, 2025

@svor could you rebase?

@openshift-ci openshift-ci bot removed the lgtm Looks good to me label Apr 28, 2025
Copy link

openshift-ci bot commented Apr 28, 2025

New changes are detected. LGTM label has been removed.

@svor
Copy link
Collaborator Author

svor commented Apr 28, 2025

@svor could you rebase?

@thepetk done

@thepetk
Copy link
Contributor

thepetk commented Apr 29, 2025

@svor could you rebase?

@thepetk done

Nice! I think just an approval from another person of @devfile/che-team to remain consistent to the process and we can merge. If is urgent let me know and I can go ahead and merge anyway. Let me know

@svor
Copy link
Collaborator Author

svor commented Apr 29, 2025

@svor could you rebase?

@thepetk done

Nice! I think just an approval from another person of @devfile/che-team to remain consistent to the process and we can merge. If is urgent let me know and I can go ahead and merge anyway. Let me know

@thepetk @ibuziuk is a member of devfile/che-team and he has approved the PR. I think it's good enough to merge it/

@thepetk thepetk merged commit cf40b5b into main Apr 29, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants