Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

feat(devworkspace): Use new name for workspaceRouting: DevWorkspaceRouting #1034

Merged
merged 1 commit into from
Mar 17, 2021

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented Mar 17, 2021

What does this PR do?

Use new DevWorkspaceRouting objects

Screenshot/screencast of this PR

image

What issues does this PR fix or reference?

devfile/devworkspace-operator#161

How to test this PR?

Use this image quay.io/crw_pr/che-theia:1034 with current DevWorkspaceOperator

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.

Happy Path Channel

HAPPY_PATH_CHANNEL=next

Change-Id: Id032a090c93ada7244e1738725e3482a239c29b4
Signed-off-by: Florent Benoit fbenoit@redhat.com

…uting

Change-Id: Id032a090c93ada7244e1738725e3482a239c29b4
Signed-off-by: Florent Benoit <fbenoit@redhat.com>
@che-bot
Copy link
Contributor

che-bot commented Mar 17, 2021

✅ E2E Happy path tests succeed 🎉

Try Che-Theia editor only Try Che-Theia with Java/maven example Try Che-Theia with NodeJs example

See Details

name link
che-theia quay.io/crw_pr/che-theia:1034
che-theia-endpoint-runtime-binary quay.io/crw_pr/che-theia-endpoint-runtime-binary:1034

Tested with Eclipse Che Single User on K8S (minikube v1.1.1)

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

@benoitf benoitf marked this pull request as ready for review March 17, 2021 14:54
Copy link

@amisevsk amisevsk left a comment

Choose a reason for hiding this comment

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

I haven't reviewed the code, but this PR fixes an issue where Theia won't load in DWO so 👍 from me 😄

@codecov
Copy link

codecov bot commented Mar 17, 2021

Codecov Report

Merging #1034 (93c6fa0) into master (47ea4c2) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1034   +/-   ##
=======================================
  Coverage   20.80%   20.80%           
=======================================
  Files         302      302           
  Lines       11234    11234           
  Branches     1703     1703           
=======================================
  Hits         2337     2337           
  Misses       8747     8747           
  Partials      150      150           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 47ea4c2...93c6fa0. Read the comment docs.

@benoitf benoitf merged commit 2073699 into eclipse-che:master Mar 17, 2021
@benoitf benoitf deleted the DWO-161 branch March 17, 2021 20:15
@che-bot che-bot added this to the 7.28 milestone Mar 17, 2021
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.

4 participants