-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Workload Protection's new Coverage feature #31134
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
Workload Protection's new Coverage feature #31134
Conversation
- revised for clarity - added more prescriptive content
✅ Documentation Team ReviewThe documentation team has approved this pull request. Thank you for your contribution! |
Preview links (active after the
|
deleting file from PR
…of github.com:DataDog/documentation into mcretzman/DOCS-11460-workload-protection-coverage-map
* Track coverage for serverless functions | ||
* Filter by environment, enabled features, agent version, etc. | ||
- [Coverage Map][3]: | ||
* View a real-time map of workload protection status |
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.
I would say near real-time (usually 1-2 mins, worst - 5)
content/en/security/workload_protection/inventory/coverage_map.md
Outdated
Show resolved
Hide resolved
content/en/security/workload_protection/inventory/coverage_map.md
Outdated
Show resolved
Hide resolved
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.
hi @michaelcretzman this is a very cool feature but I feel like the docs currently dive straight into the functionality, and especially without any screenshots or more description about the UI, I found it hard to follow along to use as a reference. Let me know if you have any questions about my feedback as you go through it.
content/en/security/workload_protection/inventory/coverage_map.md
Outdated
Show resolved
Hide resolved
content/en/security/workload_protection/inventory/coverage_map.md
Outdated
Show resolved
Hide resolved
content/en/security/workload_protection/inventory/coverage_map.md
Outdated
Show resolved
Hide resolved
content/en/security/workload_protection/inventory/coverage_map.md
Outdated
Show resolved
Hide resolved
content/en/security/workload_protection/inventory/coverage_map.md
Outdated
Show resolved
Hide resolved
incorporating peer review Co-authored-by: Rosa Trieu <107086888+rtrieu@users.noreply.github.com>
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.
making structural changes from peer review
content/en/security/workload_protection/inventory/coverage_map.md
Outdated
Show resolved
Hide resolved
content/en/security/workload_protection/inventory/coverage_map.md
Outdated
Show resolved
Hide resolved
icorp peer review structural changes
…of github.com:DataDog/documentation into mcretzman/DOCS-11460-workload-protection-coverage-map
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.
Remaining feedback for consistency throughout the doc!
Inventory is organized into the following tools: | ||
|
||
- [Hosts & Containers][1]: | ||
* List all physical and virtual hosts and container workloads. |
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.
* List all physical and virtual hosts and container workloads. | |
* List all physical and virtual hosts and container workloads |
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.
Not sure if you want to leave the periods in or leave them out, but should be consistent.
content/en/security/workload_protection/inventory/coverage_map.md
Outdated
Show resolved
Hide resolved
content/en/security/workload_protection/inventory/coverage_map.md
Outdated
Show resolved
Hide resolved
content/en/security/workload_protection/inventory/coverage_map.md
Outdated
Show resolved
Hide resolved
content/en/security/workload_protection/inventory/coverage_map.md
Outdated
Show resolved
Hide resolved
incorp peer edit Co-authored-by: Rosa Trieu <107086888+rtrieu@users.noreply.github.com>
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.
A few capitalization suggestions for you to consideer but lgtm
content/en/security/workload_protection/inventory/coverage_map.md
Outdated
Show resolved
Hide resolved
content/en/security/workload_protection/inventory/coverage_map.md
Outdated
Show resolved
Hide resolved
content/en/security/workload_protection/inventory/coverage_map.md
Outdated
Show resolved
Hide resolved
coverage -> Coverage Co-authored-by: Rosa Trieu <107086888+rtrieu@users.noreply.github.com>
DOCS-11460
What does this PR do? What is the motivation?
Documents Workload Protection's new Coverage feature.
Merge instructions
Merge readiness: