-
Notifications
You must be signed in to change notification settings - Fork 438
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
[squid] Add dashboard, improve documentation, GA integration #11145
[squid] Add dashboard, improve documentation, GA integration #11145
Conversation
- Improve the documentation of the integration - Add dashboard - Release integration as GA
🚀 Benchmarks reportTo see the full report comment with |
💚 Build Succeeded
|
Quality Gate passedIssues Measures |
Pinging @elastic/sec-deployment-and-devices (Team:Security-Deployment and Devices) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
I have a few questions, but mostly for the sake of me understanding this better.
@@ -1,11 +1,64 @@ | |||
# Squid integration | |||
# Squid Proxy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This _dev/build/docs/README.md
looks like it's exactly the same as docs/README.md
minus additional data in the latter.
Why is this the case? And is there something in place to ensure that if something in this one gets updated, that the docs/README.md
one get updated, too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
_dev/build/docs/README.md
is the "template" version and docs/README.md
is the "generated" version. We edit the template version. When you run elastic-package build
, it will regenerate the docs/README.md
file, bringing in field definitions and sample events as part of the process.
There is a CI check (done by elastic-package check
if I'm not mistaken) that verifies you've done the build process and that you've checked-in the result.
@@ -0,0 +1,2604 @@ | |||
{ | |||
"attributes": { | |||
"description": "Overview of Squid Proxy access log events", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Curious how this file is generated.
Do you create the dashboard in Kibana, and then do some sort of export?
I assume these files are rarely modified by hand, and instead overwritten by a newer version of the dashboard json?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, we create the dashboard in kibana and use elastic-package export dashboards
to get the json files. Same applies for modifications.
screenshots: | ||
- src: /img/dashboard-1.png | ||
title: Dashboard | ||
size: 2432x2368 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you have to use a photo program to get the size here? Seems a bit odd that we have to specify the size. What happens if the size is wrong?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To be honest, I don't think the size does anything. Neither does the type for that matter, as you can give the wrong mime type and it'll fine (in fact in some cases, giving the correct mime type will actually crash Kibana, but that's a separate issue). Regarding the size, most GUI shells (Finder, Windows Explorer) will show the image size in the properties of the file.
Package squid - 1.0.0 containing this change is available at https://epr.elastic.co/search?package=squid |
Proposed commit message
Checklist
changelog.yml
file.How to test this PR locally
Related issues
Screenshots