-
Notifications
You must be signed in to change notification settings - Fork 4.1k
Datadog update powershell commands to 2025-06-11 spec #28832
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
base: main
Are you sure you want to change the base?
Conversation
| Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated 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.
Pull Request Overview
This PR updates the Az.Datadog PowerShell module to add new cmdlets and enhance existing functionality for managing Datadog monitors and subscriptions in Azure.
Key Changes
- Added new cmdlets for managing monitored subscriptions (Get, Update, Remove, Set-Creator)
- Added cmdlets for billing information and creation support checks
- Added Invoke-AzDatadogResubscribeOrganization cmdlet
- Updated existing cmdlets with new parameters (agent rules, custom metrics, CSPM, resource collection)
- Updated help documentation with parameter descriptions
- Added comprehensive test coverage and test recordings
Reviewed Changes
Copilot reviewed 80 out of 81 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| src/Datadog/Datadog/help/*.md | Updated help documentation for new and existing cmdlets with new parameters and descriptions |
| src/Datadog/Datadog/Az.Datadog.psd1 | Updated module manifest with new function exports and Az.Accounts version requirement |
| src/Datadog/Datadog.Autorest/test/*.Tests.ps1 | Added test files for new cmdlets with test cases |
| src/Datadog/Datadog.Autorest/test/*.Recording.json | Added test recording files for playback testing |
| src/Datadog/Datadog.Autorest/test/utils.ps1 | Updated test setup with new organization name variables |
| src/Datadog/Datadog.Autorest/test/env.json | Updated test environment configuration |
| src/Datadog/Datadog.Autorest/test/README.md | Minor path formatting update |
src/Datadog/Datadog/help/Update-AzDatadogMonitoredSubscription.md
Outdated
Show resolved
Hide resolved
src/Datadog/Datadog/help/Set-AzDatadogMonitoredSubscriptionCreateor.md
Outdated
Show resolved
Hide resolved
src/Datadog/Datadog/help/Remove-AzDatadogMonitoredSubscription.md
Outdated
Show resolved
Hide resolved
src/Datadog/Datadog/help/Invoke-AzDatadogResubscribeOrganization.md
Outdated
Show resolved
Hide resolved
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
| @@ -0,0 +1,22 @@ | |||
| ### Example 1: {{ Add title here }} | |||
| ```powershell | |||
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.
Please add examples here. When you run build-module.ps1, these examples will be merged into the documentation in the docs folder.
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.
Done! Thanks for the info! had added them manually before.
dolauli
left a comment
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.
Please add a log for your change in the changelog file.
|
@dolauli accomodated the changes, can you please relook? |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
Description
Mandatory Checklist
Please choose the target release of Azure PowerShell. (⚠️ Target release is a different concept from API readiness. Please click below links for details.)
Check this box to confirm: I have read the Submitting Changes section of
CONTRIBUTING.mdand reviewed the following information:ChangeLog.mdfile(s) appropriatelysrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.## Upcoming Releaseheader in the past tense.ChangeLog.mdif no new release is required, such as fixing test case only.