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

Analysis logs should be available for debugging #61

Closed
pranavgaikwad opened this issue Nov 13, 2023 · 2 comments · Fixed by #68
Closed

Analysis logs should be available for debugging #61

pranavgaikwad opened this issue Nov 13, 2023 · 2 comments · Fixed by #68
Assignees

Comments

@pranavgaikwad
Copy link

It is hard to debug issues without having access to full analysis logs. It'd be great if analysis logs could be stored somewhere, I don't know the right answer where, but there seems to be a need of being able to look at full analysis logs.

@jortel jortel self-assigned this Dec 12, 2023
@jortel jortel added this to Planning Dec 12, 2023
@github-project-automation github-project-automation bot moved this to 🆕 New in Planning Dec 12, 2023
@jortel
Copy link
Contributor

jortel commented Dec 13, 2023

Solution

For all addons: command output will be uploaded to the hub and attached to the task (TaskReport).

Add new checkbox to the analysis advanced options page:

[ ] Enable enhanced details.

When enabled (true) analyzer command output will be reported (Live) to the hub as it is emitted.

A new checkbox added to the analysis detail modal. When checked, the attached (output) files are merged into the activity after the command is reported.

[ ] Merged 

When enabled (true) output is appended to the file (uploaded as it's output by the command) to the hub. This is rate limited <= 1 second using a auto-adjusted back-off. The files will be injected to the Task.Activity by the GET /tasks/:id?merged=1 endpoint.

Long term: The files can be listed, viewed and downloaded on a new Tab on the "Analysis Detail" modal.

@jortel jortel moved this from 🆕 New to 🏗 In progress in Planning Dec 18, 2023
@jortel jortel moved this from 🏗 In progress to ✅ Done in Planning Dec 20, 2023
@jortel
Copy link
Contributor

jortel commented Dec 21, 2023

Closed: #68

@jortel jortel closed this as completed Dec 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

2 participants