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

return devfile v2 as a default devfile #702

Merged
merged 2 commits into from
Jul 23, 2024
Merged

return devfile v2 as a default devfile #702

merged 2 commits into from
Jul 23, 2024

Conversation

vinokurig
Copy link
Contributor

@vinokurig vinokurig marked this pull request as ready for review July 17, 2024 09:39
@ibuziuk ibuziuk requested a review from artaleks9 July 17, 2024 10:27
() ->
newDto(FactoryDevfileV2Dto.class)
.withDevfile(Map.of("schemaVersion", "2.3.0"))
.withV(CURRENT_VERSION)
Copy link
Member

Choose a reason for hiding this comment

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

what is the CURRENT_VERSION stand for ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is Che specific factory API version constant. CURRENT_VERSION = "4.0"

.orElseGet(
() ->
newDto(FactoryDevfileV2Dto.class)
.withDevfile(Map.of("schemaVersion", "2.3.0"))
Copy link
Member

Choose a reason for hiding this comment

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

should we extract it to const?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Copy link
Member

@ibuziuk ibuziuk left a comment

Choose a reason for hiding this comment

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

@vinokurig could you please clarify if we removed everything related to devfile v1? or this is smth. to be done separately?

@vinokurig
Copy link
Contributor Author

@ibuziuk

could you please clarify if we removed everything related to devfile v1? or this is smth. to be done separately?

No, Here is the issue to clean up the code base: eclipse-che/che#23043

@vinokurig
Copy link
Contributor Author

/retest

1 similar comment
@vinokurig
Copy link
Contributor Author

/retest

@artaleks9
Copy link
Contributor

Verified on Eclipse Che with quay.io/eclipse/che-server:pr-702 - the functionality works properly

Copy link

openshift-ci bot commented Jul 23, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: artaleks9, ibuziuk, vinokurig

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

@vinokurig vinokurig merged commit f3635b9 into main Jul 23, 2024
28 checks passed
@vinokurig vinokurig deleted the che-23007 branch July 23, 2024 09:35
@devstudio-release
Copy link

Build 3.16 :: server_3.x/348: Console, Changes, Git Data

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

Build 3.16 :: server_3.x/348: SUCCESS

Upstream sync done; /DS_CI/sync-to-downstream_3.x/7296 triggered

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

Build 3.16 :: copyIIBsToQuay/2709: Console, Changes, Git Data

@devstudio-release
Copy link

Build 3.16 :: sync-to-downstream_3.x/7297: SUCCESS

Build container: devspaces-operator-bundle synced; /DS_CI/get-sources-rhpkg-container-build_3.x/7276 triggered; /job/DS_CI/job/dsc_3.x triggered;

@devstudio-release
Copy link

Build 3.16 :: operator-bundle_3.x/3351: SUCCESS

Upstream sync done; /DS_CI/sync-to-downstream_3.x/7297 triggered

@devstudio-release
Copy link

Build 3.16 :: dsc_3.x/1948: Console, Changes, Git Data

@devstudio-release
Copy link

Build 3.16 :: update-digests_3.x/7118: SUCCESS

Detected new images: rebuild operator-bundle
* server; /DS_CI/operator-bundle_3.x/3351 triggered

@devstudio-release
Copy link

Build 3.16 :: dsc_3.x/1948: SUCCESS

3.16.0-CI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
4 participants