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

Add CSM Vulns limitations for Windows #26641

Merged

Conversation

janine-c
Copy link
Contributor

@janine-c janine-c commented Dec 5, 2024

What does this PR do? What is the motivation?

See https://datadoghq.atlassian.net/browse/DOCS-9640. There was a fairly lengthy discussion on Slack about why Datadog was returning odd results on Windows, resulting in this PR to clarify some of those limitations.

Merge instructions

Merge readiness:

  • Ready for merge

Merge queue is enabled in this repo. To have it automatically merged after it receives the required reviews, create the PR (from a branch that follows the <yourname>/description naming convention) and then add the following PR comment:

/merge

Additional notes

@janine-c janine-c requested a review from cyrbouchiat December 5, 2024 22:43
@janine-c janine-c requested a review from a team as a code owner December 5, 2024 22:43
@github-actions github-actions bot added the Architecture Everything related to the Doc backend label Dec 5, 2024
Copy link
Contributor

github-actions bot commented Dec 5, 2024

@janine-c janine-c requested a review from tiphainePLQ December 5, 2024 22:53
janine-c and others added 2 commits December 5, 2024 19:06
Co-authored-by: Sandra (neko) <[email protected]>
Co-authored-by: Sandra (neko) <[email protected]>
@@ -25,6 +25,12 @@ Cloud Security Management Vulnerabilities supports vulnerability scanning for ho
| Ubuntu | All versions supported by Canonical | apt/dpkg | {{< X >}} | {{< X >}} |
| Windows | Windows Server 2016/2019/2022, Windows 10 and later | Windows OS | | {{< X >}} |

{{% collapse-content title="Windows limitations" level="h4" %}}
- Datadog detects vulnerabilities in Windows by identifying the Windows version and the security KB updates that are installed. With this information, it can determine which vulnerabilities a Windows host is subject to and which updates have been released to address them. However, some KB updates are cumulative and contain other KB updates, which might cause Datadog to misidentify which updates have been installed.
Copy link
Contributor

Choose a reason for hiding this comment

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

  1. Could we make it slightly shorter. I'm afraid it could feel heavy
  2. KB might not be obvious to everyone -> could we have 1 occurence that it mean Knowledge Base?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@cyrbouchiat Sure! It now reads:

  • Datadog detects vulnerabilities in Windows by identifying the Windows version and installed security knowledge base (KB) updates to address vulnerabilities associated with that version. However, some KB updates are cumulative and contain other KB updates, which might cause Datadog to misidentify which updates have been installed.

Let me know if that works better :)

Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like there are two files with KB in them and you only spelled it out in one instance... is that shortcode only getting used in this page before the mentions of KB, or does this page need that change too?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Wow, a weekend really does make you forget what you were doing before. Thank you for flagging! Just applied the change to the other file.

@cyrbouchiat
Copy link
Contributor

cyrbouchiat commented Dec 9, 2024 via email

@janine-c janine-c removed the request for review from tiphainePLQ December 9, 2024 19:25
@janine-c
Copy link
Contributor Author

janine-c commented Dec 9, 2024

Tiphaine approved in Slack, so removing the requirement for her to duplicate that here as well

@janine-c janine-c requested a review from neko-dd December 9, 2024 19:27
@@ -26,7 +26,7 @@ Cloud Security Management Vulnerabilities supports vulnerability scanning for ho
| Windows | Windows Server 2016/2019/2022, Windows 10 and later | Windows OS | | {{< X >}} |

{{% collapse-content title="Windows limitations" level="h4" %}}
- Datadog detects vulnerabilities in Windows by identifying the Windows version and the security KB updates that are installed. With this information, it can determine which vulnerabilities a Windows host is subject to and which updates have been released to address them. However, some KB updates are cumulative and contain other KB updates, which might cause Datadog to misidentify which updates have been installed.
- Datadog detects vulnerabilities in Windows by identifying the Windows version and installed security knowledge base (KB) updates to address vulnerabilities associated with that version. However, some KB updates are cumulative and contain other KB updates, which might cause Datadog to misidentify which updates have been installed.
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
- Datadog detects vulnerabilities in Windows by identifying the Windows version and installed security knowledge base (KB) updates to address vulnerabilities associated with that version. However, some KB updates are cumulative and contain other KB updates, which might cause Datadog to misidentify which updates have been installed.
- Datadog detects vulnerabilities in Windows by identifying the Windows version and the installed security knowledge base (KB) updates to address vulnerabilities associated with that version. However, some KB updates are cumulative and contain other KB updates, which might cause Datadog to misidentify which updates have been installed.

@janine-c
Copy link
Contributor Author

janine-c commented Dec 9, 2024

/merge

@dd-devflow
Copy link

dd-devflow bot commented Dec 9, 2024

Devflow running: /merge

View all feedbacks in Devflow UI.


2024-12-09 22:15:11 UTC ℹ️ MergeQueue: pull request added to the queue

The median merge time in master is 7m.

@dd-mergequeue dd-mergequeue bot merged commit 2d83fc8 into master Dec 9, 2024
19 of 22 checks passed
@dd-mergequeue dd-mergequeue bot deleted the janine.chan/docs-9640-windows-csm-vulns-limitations branch December 9, 2024 22:19
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 mergequeue-status: done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants