Adding support for openshift installation#439
Adding support for openshift installation#439djkormo wants to merge 20 commits intolitmuschaos:masterfrom
Conversation
Signed-off-by: Krzysztof Pudłowski <djkormo@gmail.com>
* Added support for proxy envs Signed-off-by: Jonsy13 <vedant.shrotria@harness.io> * Added support for proxy envs Signed-off-by: Jonsy13 <vedant.shrotria@harness.io> * Updated docs version Signed-off-by: Jonsy13 <vedant.shrotria@harness.io> --------- Signed-off-by: Jonsy13 <vedant.shrotria@harness.io> Signed-off-by: Krzysztof Pudłowski <djkormo@gmail.com>
Signed-off-by: Jonsy13 <vedant.shrotria@harness.io> Signed-off-by: Krzysztof Pudłowski <djkormo@gmail.com>
Signed-off-by: Krzysztof Pudłowski <djkormo@gmail.com>
Signed-off-by: Krzysztof Pudłowski <djkormo@gmail.com>
Signed-off-by: Krzysztof Pudłowski <djkormo@gmail.com>
Signed-off-by: Krzysztof Pudłowski <djkormo@gmail.com>
Signed-off-by: Krzysztof Pudłowski <djkormo@gmail.com>
Signed-off-by: Krzysztof Pudłowski <djkormo@gmail.com>
Signed-off-by: Krzysztof Pudłowski <djkormo@gmail.com>
Signed-off-by: Krzysztof Pudłowski <djkormo@gmail.com>
…#437) Signed-off-by: Shubham Chaudhary <shubham.chaudhary@harness.io> Signed-off-by: Krzysztof Pudłowski <djkormo@gmail.com>
Signed-off-by: Jonsy13 <vedant.shrotria@harness.io> Signed-off-by: Krzysztof Pudłowski <djkormo@gmail.com>
Signed-off-by: Krzysztof Pudłowski <djkormo@gmail.com>
1f354fb to
aedf6d0
Compare
Signed-off-by: Krzysztof Pudłowski <djkormo@gmail.com>
|
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
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.]