Skip to content

Conversation

@aamir-tiwari-sumo
Copy link

This PR introduces support for executing the getExperimentRun and podLog GraphQL queries directly from the codebase. These additions will enable fetching logs after running an experiment, providing enhanced visibility and facilitating troubleshooting. These queries also offer flexibility for potential future use cases where log retrieval or experiment run details are needed.

Signed-off-by: Aamir Tiwari <aamir.tiwari@sumologic.com>
@SarthakJain26
Copy link
Collaborator

@aamir-tiwari-sumo thanks for the PR.
Need some more info on how you plan to integrate the getPodLogs with litmusctl, since it is a subscription and not query.

@aamir-tiwari-sumo
Copy link
Author

aamir-tiwari-sumo commented Apr 17, 2025

@aamir-tiwari-sumo thanks for the PR. Need some more info on how you plan to integrate the getPodLogs with litmusctl, since it is a subscription and not query.

@SarthakJain26 Thanks for the response, there is another set of change that we plan on top of this to support fetching logs and displaying the same when needed -> master...SumoLogic:litmusctl-sumo:display-logs

@SarthakJain26
Copy link
Collaborator

@SarthakJain26 Thanks for the response, there is another set of change that we plan on top of this to support fetching logs and displaying the same when needed -> master...SumoLogic:litmusctl-sumo:display-logs

@aamir-tiwari-sumo looks good 🙌 , can you please raise a draft PR with your next set of changes and add a description as well as some working screenshots so that the team can review this as well. cc: @Jonsy13 @Saranya-jena

@aamir-tiwari-sumo
Copy link
Author

@SarthakJain26 Thanks for the response, there is another set of change that we plan on top of this to support fetching logs and displaying the same when needed -> master...SumoLogic:litmusctl-sumo:display-logs

@aamir-tiwari-sumo looks good 🙌 , can you please raise a draft PR with your next set of changes and add a description as well as some working screenshots so that the team can review this as well. cc: @Jonsy13 @Saranya-jena

@SarthakJain26 Apologies for the delayed response. As requested, here’s the link to the draft PR: #277

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.

2 participants