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

chore: update registry deployment configuration #790

Merged
merged 2 commits into from
Oct 12, 2023
Merged

Conversation

svor
Copy link
Contributor

@svor svor commented Oct 9, 2023

What does this PR do?

Fix deployment configuration for OpenShift

What issues does this PR fix or reference?

eclipse-che/che#20133

How to test this PR?

  1. Login into OS cluster
  2. Execute command from README to deploy che plugin registry:
  oc new-app -f deploy/openshift/che-devfile-registry.yaml \
             -p IMAGE="quay.io/eclipse/che-devfile-registry" \
             -p IMAGE_TAG="next" \
             -p PULL_POLICY="Always"
  1. Devfile registry should be deployed and available. In the output you should see a message:
--> Success
    Access your application via route 'che-devfile-registry-pr.apps.rosa.dzhix-cyddx-yrv.1697.p3.openshiftapps.com' 
    Run 'oc status' to view your app.
  1. Try to open route URL from the message and add /devfiles at the end
  2. You should see:
    screenshot-che-devfile-registry-pr apps rosa dzhix-cyddx-yrv 1697 p3 openshiftapps com-2023 10 09-18_04_53

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.

@svor svor requested review from tolusha and artaleks9 October 9, 2023 10:05
@svor svor self-assigned this Oct 9, 2023
@svor svor requested review from vinokurig and ibuziuk as code owners October 9, 2023 10:05
@github-actions
Copy link

github-actions bot commented Oct 9, 2023

Click here to review and test in web IDE: Contribute

Signed-off-by: Valeriy Svydenko <[email protected]>
@artaleks9
Copy link

  • Verified on OpenShift v4.13.1 with
$ oc version
Client Version: 4.11.13
Kustomize Version: v4.5.4
Server Version: 4.13.1

the functionality works properly

@svor svor merged commit a51685f into main Oct 12, 2023
4 checks passed
@svor svor deleted the sv-fix-deployment branch October 12, 2023 12:28
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