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.

Sign up for GitHub

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

Jump to bottom

feat: OLM operator installer #611

Merged
merged 46 commits into from
Apr 24, 2020
Merged

feat: OLM operator installer #611

merged 46 commits into from
Apr 24, 2020

Conversation

AndrienkoAleksandr
Copy link
Contributor

@AndrienkoAleksandr AndrienkoAleksandr commented Mar 31, 2020 •
edited
Loading

What does this PR do?

Add ability to install Che using OLM installer. In the current state we install only stable channel. Also provided flags to use custom catalog source in the same namespace with Che installation.

What issues does this PR fix or reference?

eclipse-che/che#15279

@AndrienkoAleksandr
Implement olm installer and tasks
93454d9 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Add update tasks.
cb2eab1 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Improve delete tasks and reuse tls tasks.
f54c691 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Add api OLM api group check.
0edc6fe 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Fix up.
117b372 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Clean up.
7d04cd6 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr AndrienkoAleksandr changed the title Operator installer OLM operator installer Mar 31, 2020
@AndrienkoAleksandr AndrienkoAleksandr changed the title OLM operator installer feat: OLM operator installer Mar 31, 2020
@tolusha tolusha mentioned this pull request Mar 31, 2020
Closed
43 tasks
@AndrienkoAleksandr
Improve OLM checks.
0772409 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Fix installation using OLM on the minikube.
12c91fa 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Merge branch 'master' of github.com:che-incubator/chectl into operato…
c2ff468 
…rInstaller
@AndrienkoAleksandr
Fix catalogsource namespaces for minikube, so crc and minikube should…
47da78b 
… work both.

Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Install stable Che using community catalog.
43a94c3 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Don't remove operatorsource, for optimization we can share it between…
25da32d 
… Che instalations...

Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Code clean up.
ba19ec0 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
tolusha
tolusha reviewed Apr 8, 2020
View reviewed changes
src/api/kube.ts Outdated Show resolved Hide resolved
src/commands/server/start.ts Outdated Show resolved Hide resolved
src/tasks/installers/olm.ts Outdated Show resolved Hide resolved
src/tasks/installers/olm.ts Outdated Show resolved Hide resolved
src/tasks/installers/olm.ts Outdated Show resolved Hide resolved
@AndrienkoAleksandr
Address request changes.
dc1ad64 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Notify user that minishift platform is not supported.
c0e6701 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Merge branch 'master' of github.com:che-incubator/chectl into operato…
0971a1f 
…rInstaller
@AndrienkoAleksandr
Improve olm update method.
b63a21d 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Fix up.
e204fb9 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@tolusha
Copy link
Collaborator

tolusha commented Apr 14, 2020 

 Check if OLM is pre-installed on the platform
      → OLM isn't installed on your platfrom. If your platform hasn't got embedded OML, you need install it manually.

Why don't install it automatically? Are there any pitfalls?

@AndrienkoAleksandr
Add ability to use OLM automatic installation mode.
b13ec1c 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Add ability to set up starting csv for olm installer.
2ff5de0 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Simplify code .
2cbaf98 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Fix format.
13d39f1 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Fix up.
3d3eeaa 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@che-osio-probot-app
Copy link

Open Developer Workspace:
Contribute

@AndrienkoAleksandr
Add more checks
c11fee4 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@che-osio-probot-app
Copy link

Open Developer Workspace:
Contribute

@AndrienkoAleksandr
Clean up unused const
2f9c873 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@che-osio-probot-app
Copy link

Open Developer Workspace:
Contribute

@AndrienkoAleksandr
Fix installation stable Che image for nightly chectl, add useful warn…
fa6a7a2 
…ings for user.

Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@che-osio-probot-app
Copy link

Open Developer Workspace:
Contribute

@che-osio-probot-app
Copy link

Open Developer Workspace:
Contribute

@AndrienkoAleksandr
Fix compilation after resolve merge conflict.
320161c 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@che-osio-probot-app
Copy link

Open Developer Workspace:
Contribute

@AndrienkoAleksandr
Fix lints.
2c06ec2 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@che-osio-probot-app
Copy link

Open Developer Workspace:
Contribute

@AndrienkoAleksandr
Copy link
Contributor Author

[test]

1 similar comment
@AndrienkoAleksandr
Copy link
Contributor Author

[test]

@che-osio-probot-app
Copy link

Open Developer Workspace:
Contribute

@AndrienkoAleksandr
Fix up.
87a4e8e 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@che-osio-probot-app
Copy link

Open Developer Workspace:
Contribute

@AndrienkoAleksandr
Copy link
Contributor Author

[test]

1 similar comment
@AndrienkoAleksandr
Copy link
Contributor Author

[test]

@AndrienkoAleksandr
Fix custom catalog source with nightly channel.
bbdf48c 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@che-osio-probot-app
Copy link

Open Developer Workspace:
Contribute

@AndrienkoAleksandr
Fix lint again.
5609ff9 
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@che-osio-probot-app
Copy link

Open Developer Workspace:
Contribute

@AndrienkoAleksandr
Copy link
Contributor Author

[test]

@AndrienkoAleksandr AndrienkoAleksandr merged commit 63b9ad0 into master Apr 24, 2020
@AndrienkoAleksandr AndrienkoAleksandr deleted the operatorInstaller branch April 24, 2020 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mmorhun mmorhun mmorhun left review comments

@tolusha tolusha tolusha approved these changes

@davidfestal davidfestal Awaiting requested review from davidfestal

@l0rd l0rd Awaiting requested review from l0rd

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@AndrienkoAleksandr @tolusha @flacatus @mmorhun