Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chaos result should support json blob #1254

Closed
sumitnagal opened this issue Mar 4, 2020 · 3 comments
Closed

Chaos result should support json blob #1254

sumitnagal opened this issue Mar 4, 2020 · 3 comments
Assignees
Labels
area/litmus kind/feature project/community Issues raised by community members
Milestone

Comments

@sumitnagal
Copy link
Contributor

Due to data insight as well as maintain data, it is important that we maintain certain metadata. this could be achieved when we provide a way of json object. Adding this will help meaningful outcomes across the team over the organisation.

@ksatchit ksatchit added kind/feature area/litmus project/community Issues raised by community members labels Mar 4, 2020
@ksatchit ksatchit added this to the 1.3 milestone Mar 4, 2020
@ksatchit
Copy link
Member

This ability to push such a data structure (json) in to chaosresult CR will help users create their own context wrt the result of a chaos experiment. The initial idea is to allow : data: | inside the chaosresult status to allow user-defined patches.

@ksatchit
Copy link
Member

ksatchit commented Mar 21, 2020

(a) Enhance the chaosresult schema to accept json blob in a data:| field to push in custom info, here: https://github.com/litmuschaos/chaos-operator/blob/master/pkg/apis/litmuschaos/v1alpha1/chaosresult_types.go

(b) The python function could patch that jsob report here: https://github.com/litmuschaos/test-tools/blob/master/chaostoolkit/data/k8_wrapper.py#L29

@ksatchit
Copy link
Member

ksatchit commented Oct 3, 2022

The chaos workflows are based on argo which supports the ability to push output artifacts generated from experiment runs to a dedicated sink. This way, we don't have to store data blobs on CRDs like chaosresults.

@ksatchit ksatchit closed this as completed Oct 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/litmus kind/feature project/community Issues raised by community members
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants