Skip to content

Adding support for openshift installation#439

Open
djkormo wants to merge 20 commits intolitmuschaos:masterfrom
djkormo:litmus-helm-for-openshift
Open

Adding support for openshift installation#439
djkormo wants to merge 20 commits intolitmuschaos:masterfrom
djkormo:litmus-helm-for-openshift

Conversation

@djkormo
Copy link
Copy Markdown

@djkormo djkormo commented May 21, 2025

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

openshift:
  enabled: true # default false for vanilla kubernetes
  anyuid: false # # if true service account will obtain permission for using anyuid scc 

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.]

  • DCO signed
  • Chart Version bumped
  • Variables are documented in the README.md

djkormo and others added 14 commits May 21, 2025 22:07
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>
@djkormo djkormo force-pushed the litmus-helm-for-openshift branch from 1f354fb to aedf6d0 Compare May 21, 2025 20:08
@djkormo djkormo requested a review from uditgaurav as a code owner May 21, 2025 20:08
Signed-off-by: Krzysztof Pudłowski <djkormo@gmail.com>
@djkormo
Copy link
Copy Markdown
Author

djkormo commented Jun 2, 2025

Knock, Knock.....

@Jonsy13
Copy link
Copy Markdown
Contributor

Jonsy13 commented Sep 1, 2025

Hi @djkormo , Thanks for raising this PR.
Just looked at it & we would like to take it. For that, lets start with reverting the version bumps in charts other than litmus & also fixing the conflicts.

Once that is done, can you also paste some screenshots supporting successful installation with used scc template.
Thanks!

Comment on lines +19 to +23
<<<<<<< HEAD
VERSION: "3.19.0"
=======
VERSION: "3.20.0"
>>>>>>> upstream/master
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @djkormo Can you please check, looks like the conflicts are not resolved correctly.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corrected.

Comment on lines +24 to +35
<<<<<<< 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
=======
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @djkormo Still seeing some conflicts here, can you check, I think the changes themselves are not required in this PR for agent.

@djkormo
Copy link
Copy Markdown
Author

djkormo commented Sep 16, 2025

Corrected

Comment on lines +131 to +133
<<<<<<< HEAD
tag: 3.19.0
=======
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are absolutely right. Added changes for version 3.21.0

@Jonsy13
Copy link
Copy Markdown
Contributor

Jonsy13 commented Nov 17, 2025

Hi @djkormo Please bump the chart version.
Also remove the versioned values.yamls
Lets only keep the CI & Latest ones.

I have a PR which will conclude the release (3.23.x) - #462
If you could make this change, We can take this in current release itself.

@imrajdas imrajdas removed their request for review January 26, 2026 07:00
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.

3 participants