Adding support for openshift installation #439
Conversation
djkormo
requested review from
Jasstkn,
Jonsy13,
Saranya-jena,
ajeshbaby,
gdsoumya,
imrajdas,
ispeakc0de and
ksatchit
as code owners
Signed-off-by: Krzysztof Pudłowski <[email protected]>
* Added support for proxy envs Signed-off-by: Jonsy13 <[email protected]> * Added support for proxy envs Signed-off-by: Jonsy13 <[email protected]> * Updated docs version Signed-off-by: Jonsy13 <[email protected]> --------- Signed-off-by: Jonsy13 <[email protected]> Signed-off-by: Krzysztof Pudłowski <[email protected]>
Signed-off-by: Jonsy13 <[email protected]> Signed-off-by: Krzysztof Pudłowski <[email protected]>
Signed-off-by: Krzysztof Pudłowski <[email protected]>
Signed-off-by: Krzysztof Pudłowski <[email protected]>
Signed-off-by: Krzysztof Pudłowski <[email protected]>
Signed-off-by: Krzysztof Pudłowski <[email protected]>
Signed-off-by: Krzysztof Pudłowski <[email protected]>
Signed-off-by: Krzysztof Pudłowski <[email protected]>
Signed-off-by: Krzysztof Pudłowski <[email protected]>
Signed-off-by: Krzysztof Pudłowski <[email protected]>
…#437) Signed-off-by: Shubham Chaudhary <[email protected]> Signed-off-by: Krzysztof Pudłowski <[email protected]>
Signed-off-by: Jonsy13 <[email protected]> Signed-off-by: Krzysztof Pudłowski <[email protected]>
Signed-off-by: Krzysztof Pudłowski <[email protected]>
djkormo
force-pushed
the
litmus-helm-for-openshift
branch
from
1f354fb to
aedf6d0
Compare
Signed-off-by: Krzysztof Pudłowski <[email protected]>
|
Knock, Knock..... |
|
Hi @djkormo , Thanks for raising this PR. Once that is done, can you also paste some screenshots supporting successful installation with used scc template. |
charts/litmus/values.yaml
Outdated
| <<<<<<< HEAD | ||
| VERSION: "3.19.0" | ||
| ======= | ||
| VERSION: "3.20.0" | ||
| >>>>>>> upstream/master |
There was a problem hiding this comment.
Hey @djkormo Can you please check, looks like the conflicts are not resolved correctly.
charts/litmus-agent/Chart.yaml
Outdated
| <<<<<<< HEAD | ||
| version: 3.19.0 | ||
| condition: chaos-operator.enabled | ||
| - name: chaos-exporter | ||
| version: 3.19.0 | ||
| condition: chaos-exporter.enabled | ||
| - name: event-tracker | ||
| version: 3.19.0 | ||
| condition: event-tracker.enabled | ||
| - name: subscriber | ||
| version: 3.19.0 | ||
| ======= |
There was a problem hiding this comment.
Hi @djkormo Still seeing some conflicts here, can you check, I think the changes themselves are not required in this PR for agent.
|
Corrected |
charts/litmus/values.yaml
Outdated
| <<<<<<< HEAD | ||
| tag: 3.19.0 | ||
| ======= |
There was a problem hiding this comment.
Hi @djkormo
We are still seeing conflicts here in this file too.
Can you please go through complete PR & fix the conflicts.
Also please update the version again for chart as we have done a new release yesterday.
There was a problem hiding this comment.
You are absolutely right. Added changes for version 3.21.0
3 participants
What this PR does / why we need it:
Installations for openshift cluster out of the box.
Helm chart should cover installation of litmus for the openshift platform.
To move litmus to enterprise customers using this kind of software.
Dedicated scc (security context constraint) should be added to template rbac objects.
I think I can help with this
Proposed values excerpt
Which issue this PR fixes
litmuschaos/litmus#5113
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]