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

fix: Use Olm CR sample from CSV #929

Merged
merged 2 commits into from
Oct 22, 2020
Merged

fix: Use Olm CR sample from CSV #929

merged 2 commits into from
Oct 22, 2020

Conversation

AndrienkoAleksandr
Copy link
Contributor

@AndrienkoAleksandr AndrienkoAleksandr commented Oct 15, 2020

What does this PR do?

Use Olm CR sample from CSV

What issues does this PR fix or reference?

None. Code enhancement.

How to test this PR?

Install Che on the Openshift platform:

$ chectl server:start -n che -p openshift -m

Take a look on the CR:

$ oc get checluster eclipse-che  -n che -o  yaml

Check Che using url.

And also make sure that operator installer still works too:

$ chectl server:start -n che2 -p openshift -m --installer operator

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: Oleksandr Andriienko oandriie@redhat.com

@AndrienkoAleksandr AndrienkoAleksandr changed the title Use Olm CR sample from CSV fix: Use Olm CR sample from CSV Oct 15, 2020
src/api/kube.ts Outdated Show resolved Hide resolved
@flacatus
Copy link
Collaborator

/retest

src/api/kube.ts Outdated Show resolved Hide resolved
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Copy link
Contributor Author

rebased

@openshift-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: AndrienkoAleksandr, mmorhun, tolusha
To complete the pull request process, please assign after the PR has been reviewed.
You can assign the PR to them by writing /assign in a comment when ready.

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

Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@openshift-ci-robot
Copy link
Collaborator

New changes are detected. LGTM label has been removed.

@AndrienkoAleksandr
Copy link
Contributor Author

/retest

@AndrienkoAleksandr
Copy link
Contributor Author

CI is finally green...

@AndrienkoAleksandr AndrienkoAleksandr merged commit 2bbd800 into master Oct 22, 2020
@AndrienkoAleksandr AndrienkoAleksandr deleted the useOlmCRSample branch October 22, 2020 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants