litmus icon indicating copy to clipboard operation
litmus copied to clipboard

Wizard to create experiment is broken

Open pierluigilenoci opened this issue 1 year ago • 3 comments

What happened:

When using the interface to create an experiment with the "Pod Killer" template, the generated YAML consistently references kube-proxy and contains an unmodified template annotation, regardless of the selected namespace or label. The YAML produced shows the following line:

subject: "{{workflow.parameters.appNamespace}}_kube-proxy"

Screenshot 2024-12-04 at 15 47 02

What you expected to happen:

The generated YAML should reflect the correct namespace or label based on the user's input.

Where can this issue be corrected? (optional)

I don't know.

How to reproduce it (as minimally and precisely as possible):

Create a basic experiment using the "Pod Killer" template and try to specify a namespace and a selector label..

Anything else we need to know?:

Nope

pierluigilenoci avatar Dec 04 '24 14:12 pierluigilenoci

@Saranya-jena @namkyu1999 @S-ayanide, could you please take a look?

pierluigilenoci avatar Jan 10 '25 14:01 pierluigilenoci

@uditgaurav, @ispeakc0de, @ksatchit, @Vr00mm, could you please take a look?

pierluigilenoci avatar Jan 21 '25 14:01 pierluigilenoci

Hello, I can get a look this evening ( PARIS timezone).

I dont think there is any problems here. Some parameters are dynamics. Did you tried to finish the build and schedule your experiment ? it should ask for target options later ?

Best Regards

Vr00mm avatar Jan 21 '25 14:01 Vr00mm

Hi @pierluigilenoci , as @Vr00mm mentioned, this doesn’t appear to be a bug. Some values are added at runtime as defined in the Argo template. However, this can be updated here: https://github.com/litmuschaos/chaos-charts/blob/f4dfe58d78d0efa8ce9f35b60f4b4a48c560ca52/experiments/pod-delete/experiment.yaml#L7C14-L7C63

We can update the subject and replace it with pod-delete. Please feel free to raise a PR with this change — contributions are always welcome! 🙌

amityt avatar Sep 17 '25 10:09 amityt