Closed AObuchow closed 8 months ago
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: amisevsk, AObuchow
The full list of commands accepted by this bot can be found here.
The pull request process is described here
New changes are detected. LGTM label has been removed.
/retest
All modified lines are covered by tests :white_check_mark:
Comparison is base (
b12f682
) 52.90% compared to head (bdac5e8
) 53.10%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
What does this PR do?
Updates the devfile/api dependency to the v2.2.1 (commit https://github.com/devfile/api/commit/1ae41b52c5f2bea1031b16277666e1b6baa0358c). This is required for https://github.com/devfile/devworkspace-operator/issues/1179, as the devfile validation library was previously preventing a deworkspace from having a container component and openshift/kubernetes component from using the same target port on an endpoint.
I also added a controller env test to test the case from https://github.com/devfile/devworkspace-operator/issues/1179, but this might be redundant as this functionality should already be tested in the devfile API repo.
What issues does this PR fix or reference?
https://github.com/devfile/devworkspace-operator/issues/1179
Is it tested? How?
Install DWO with the changes from this PR, and create a devworkspace with a container component and kubernetes (or openshift) component that uses the same endpoint target port, such as the following:
Ensure the workspace starts up as expected. You should see a pod get deployed to the
devworkspace-controller
namespace calledwebserver-dwo-deployed
. The main thing to ensure is the workspace does not fail to startup with the following reason:* devfile contains multiple containers with same endpoint targetPort: 8080
PR Checklist
/test v8-devworkspace-operator-e2e, v8-che-happy-path
to trigger)v8-devworkspace-operator-e2e
: DevWorkspace e2e testv8-che-happy-path
: Happy path for verification integration with Che Signed-off-by: Andrew Obuchowicz aobuchow@redhat.com