Skip to content

Main 1907 workload health endpoint - #106

Merged
RoiGlinik merged 15 commits into
masterfrom
main-1907-workload-health
Aug 22, 2024
Merged

RoiGlinik merged 15 commits into
masterfrom
main-1907-workload-health

Conversation

@RoiGlinik

Copy link
Copy Markdown
Collaborator

No description provided.

@CLAassistant

CLAassistant commented Aug 6, 2024 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@arikalon1 arikalon1 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, left a minor comment

Comment thread holmes/core/supabase_dal.py Outdated
)
issue_data["evidence"] = evidence.data
return issue_data No newline at end of file
enrichment_blacklist = {"text_file", "alert_labels", "ai_analysis", "holmes"}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are you also removing the alert labels? and text files?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

text files are zipped and require some extra work. also the logs are taken in the investigation so it seems redundant.
alert labels just not sure how useful they are

Comment thread holmes/core/supabase_dal.py Outdated
.execute()
)

enrichment_blacklist = {"text_file", "alert_labels", "ai_analysis", "holmes"}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same

@RoiGlinik
RoiGlinik merged commit a4efa4f into master Aug 22, 2024
@RoiGlinik
RoiGlinik deleted the main-1907-workload-health branch August 22, 2024 17:56
Avi-Robusta added a commit that referenced this pull request Jul 29, 2025
Here we see holmes prints log data with only the logs from finding
<img width="343" height="450" alt="Screenshot 2025-07-28 at 10 01 01"
src="https://github.com/user-attachments/assets/049489a4-5c27-41aa-80ad-fcc062b8fa5f"
/>
Here you can see the unzipped logs
<img width="663" height="348" alt="Screenshot 2025-07-28 at 10 01 20"
src="https://github.com/user-attachments/assets/9b084cb7-e2b0-420f-867b-b3cc84c43de7"
/>

here is the PR where files were removed due to them being zipped
#106
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.

3 participants