[Chaos] - Add ansible to also provision chaos when installing

[see-quick]

No description provided.

[github-actions]

Thanks for pull request!!!
hey @kornys @Frawless @obabec

[Frawless]

You should create new task file for chaos I think. Adding it to infra doesn't sounds right. Maybe put it to strimzi instead?

[see-quick]

You should create new task file for chaos I think. Adding it to infra doesn't sounds right. Maybe put it to strimzi instead?

I was also thinking about it. So I think it would be best to choose own chaos file then

[kornys]

do not use definition if you using templates

[see-quick]

So should I change it to use resource_definition i.e., also change fileglob to loop and query...?

- name: Deploy chaos engineering resources for Tekton from Jinja2 templates
  kubernetes.core.k8s:
    kubeconfig: "{{ kubeconfig_path }}/{{ infra_context_name }}"
    namespace: "{{ infra_ci_namespace }}"
    state: present
    resource_definition: "{{ lookup('template', item) }}"
    verify_ssl: no
    apply: true
  loop: "{{ query('fileglob', 'templates/tekton/pipelines/chaos/*.j2') }}"

[kornys]

see other examples, instead of resource_definition use template

[kornys]

https://github.com/skodjob/automation-hub/blob/main/install/roles/automation-hub/tasks/scenario-deployment/tekton/infra-tkn-pipelines.yaml#L12
for example :)

[kornys]

some ansible course would be useful :)

[Frawless]

I think it is fine to use definition in that case, however, we use template even if we don't process it with Jinja because in case we add some variable it is not straightforward to find the issue. An example with fileglob is here https://github.com/skodjob/automation-hub/blob/main/install/roles/automation-hub/tasks/scenario-deployment/tekton/infra-tkn-pipelines.yaml#L28

[see-quick]

I think it is fine to use definition in that case, however, we use template even if we don't process it with Jinja because in case we add some variable

Thanks for the explanation I was wondering why you use a template even if you don't process it with Jinja. I will update it accordingly :)

[see-quick]

updated @Frawless @kornys

[Frawless]

Guess in the future it would make sense to use some variables for config the cluster names, etc. Have some chaos config part. But definitely not in this PR.

[see-quick]

Guess in the future it would make sense to use some variables for config the cluster names, etc. Have some chaos config part. But definitely not in this PR.

For sure that's a good idea.