Skip to content

Conversation

michaelcretzman
Copy link
Contributor

DOCS-11460

What does this PR do? What is the motivation?

Documents Workload Protection's new Coverage feature.

Merge instructions

Merge readiness:

  • Ready for merge

- revised for clarity
- added more prescriptive content
@michaelcretzman michaelcretzman requested review from a team as code owners August 15, 2025 23:10
@github-actions github-actions bot added Architecture Everything related to the Doc backend Images Images are added/removed with this PR labels Aug 15, 2025
Copy link
Contributor

github-actions bot commented Aug 15, 2025

✅ Documentation Team Review

The documentation team has approved this pull request. Thank you for your contribution!

deleting file from PR
@michaelcretzman michaelcretzman self-assigned this Aug 15, 2025
* 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

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)

@michaelcretzman michaelcretzman added the editorial review Waiting on a more in-depth review label Aug 18, 2025
@michaelcretzman michaelcretzman changed the title draft Workload Protection's new Coverage feature Aug 20, 2025
Copy link
Contributor

@rtrieu rtrieu left a 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.

incorporating peer review

Co-authored-by: Rosa Trieu <107086888+rtrieu@users.noreply.github.com>
Copy link
Contributor Author

@michaelcretzman michaelcretzman left a 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

icorp peer review structural changes
@michaelcretzman michaelcretzman requested a review from rtrieu August 22, 2025 21:20
Copy link
Contributor

@rtrieu rtrieu left a 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.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* List all physical and virtual hosts and container workloads.
* List all physical and virtual hosts and container workloads

Copy link
Contributor

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.

incorp peer edit

Co-authored-by: Rosa Trieu <107086888+rtrieu@users.noreply.github.com>
Copy link
Contributor

@rtrieu rtrieu left a 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

coverage -> Coverage

Co-authored-by: Rosa Trieu <107086888+rtrieu@users.noreply.github.com>
@michaelcretzman michaelcretzman merged commit b4a87b2 into master Aug 25, 2025
19 checks passed
@michaelcretzman michaelcretzman deleted the mcretzman/DOCS-11460-workload-protection-coverage-map branch August 25, 2025 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Architecture Everything related to the Doc backend editorial review Waiting on a more in-depth review Images Images are added/removed with this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants