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

Improve and regroup agentless costs mentions #27539

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

aviat
Copy link
Contributor

@aviat aviat commented Feb 10, 2025

What does this PR do? What is the motivation?

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

@aviat aviat added the under_review Sent to someone outside of the docs team for review. label Feb 10, 2025
@aviat aviat requested a review from a team as a code owner February 10, 2025 19:37
@aviat aviat removed the under_review Sent to someone outside of the docs team for review. label Feb 11, 2025
@aviat
Copy link
Contributor Author

aviat commented Feb 11, 2025

/merge

@dd-devflow
Copy link

dd-devflow bot commented Feb 11, 2025

Devflow running: /merge

View all feedbacks in Devflow UI.


2025-02-11 16:04:37 UTC ℹ️ MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.

Use /merge -c to cancel this operation!


⏳ Processing ...

Copy link
Contributor

@maycmlee maycmlee left a comment

Choose a reason for hiding this comment

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

Left some feedback!

When using Agentless Scanning, there are additional costs for running scanners in your cloud environments. To optimize on costs while being able to reliably scan every 12 hours, Datadog recommends setting up [Agentless Scanning with Terraform][6] as the default template, as this also avoids cross-region networking.
When using Agentless Scanning, there are additional cloud provider costs for running scanners and analyzing your cloud environments.

Cloud provider costs are under $1 per scanned host per year for accounts following the [recommended configuration][13]. Reach out to your [Datadog Customer Success Manager][7] if you have more questions.
Copy link
Contributor

Choose a reason for hiding this comment

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

We usually don't talk about costs in the docs, and usually link to the pricing page. In this situation, can we just have customers reach out to CSM about costs? So something like:

Suggested change
Cloud provider costs are under $1 per scanned host per year for accounts following the [recommended configuration][13]. Reach out to your [Datadog Customer Success Manager][7] if you have more questions.
Reach out to your [Datadog Customer Success Manager][7] if you have questions about Cloud provider costs for the [recommended configuration][13].

[1]: mailto:[email protected]
[2]: /security/cloud_security_management/agentless_scanning#cloud-storage-scanning
[1]: /security/cloud_security_management/agentless_scanning#cloud-storage-scanning
[1]: /security/cloud_security_management/agentless_scanning#cloud-service-provicer-cost
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
[1]: /security/cloud_security_management/agentless_scanning#cloud-service-provicer-cost
[2]: /security/cloud_security_management/agentless_scanning#cloud-service-provicer-cost

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants