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

chore: don't generate theia resources; remove sidecars; add devfile.yaml to work with the registry #726

Merged
merged 10 commits into from
Mar 31, 2023

Conversation

svor
Copy link
Contributor

@svor svor commented Mar 30, 2023

What does this PR do?

  • Don't generate devworkspaces with che-theia editor
  • Remove dockerfiles that used to use in dev components
  • Add devfile.yaml to develop che-devfile-registry on the dogfooding cluster

Screenshot/screencast of this PR

I've used a devfile.yaml provided by the current changes to test if the registry works. For this I prepared OS instance with Che, added build capabilities to the cluster and:

  1. Build registry by executing 1. Build and Publish a Custom Devfile Registry command:
    screenshot-nimbusweb me-2023 03 30-15_52_46

  2. Executed a command to use resulted image: 2. Configure Che to use the Custom Devfile Registry
    screenshot-nimbusweb me-2023 03 30-15_56_28

  3. Checked che-devfile-registry pod:
    screenshot-nimbusweb me-2023 03 30-15_58_04

What issues does this PR fix or reference?

eclipse-che/che#22091

How to test this PR?

  • Build che-devfile registry
  • Check generated htdocs/devfiles/stack/ folders, they should contain generated devworkspaces only for che-code and che-idea editors

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

Signed-off-by: Valerii Svydenko <vsvydenk@redhat.com>
@github-actions
Copy link

Click here to review and test in web IDE: Contribute

@svor svor changed the title chore: remove sidecars; remove theia resources [WIP]chore: remove sidecars; remove theia resources Mar 30, 2023
Signed-off-by: Valerii Svydenko <vsvydenk@redhat.com>
Signed-off-by: Valerii Svydenko <vsvydenk@redhat.com>
Signed-off-by: Valerii Svydenko <vsvydenk@redhat.com>
Signed-off-by: Valerii Svydenko <vsvydenk@redhat.com>
Signed-off-by: Valerii Svydenko <vsvydenk@redhat.com>
Signed-off-by: Valerii Svydenko <vsvydenk@redhat.com>
Signed-off-by: Valerii Svydenko <vsvydenk@redhat.com>
@svor svor self-assigned this Mar 30, 2023
@svor svor changed the title [WIP]chore: remove sidecars; remove theia resources chore: don't generate theia resources; remove sidecars; add devfile.yaml to work with the registry Mar 30, 2023
@svor svor requested a review from nickboldt March 30, 2023 14:52
patch-checluster.sh Outdated Show resolved Hide resolved
devfile.yaml Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
svor and others added 2 commits March 30, 2023 22:17
Signed-off-by: Valerii Svydenko <vsvydenk@redhat.com>
Co-authored-by: Nick Boldt <nboldt@redhat.com>
@nickboldt nickboldt self-requested a review March 30, 2023 19:59
@svor svor merged commit 95a24f0 into main Mar 31, 2023
@svor svor deleted the sv-remove-theia branch March 31, 2023 09:33
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.

2 participants