Skip to content

Add Confluent PowerShell commands for API version 2025-08-18-preview#29246

Merged
VeryEarly merged 6 commits intoAzure:mainfrom
Shekar2004:pgnanashekar/cfconnPSH
Mar 31, 2026
Merged

Add Confluent PowerShell commands for API version 2025-08-18-preview#29246
VeryEarly merged 6 commits intoAzure:mainfrom
Shekar2004:pgnanashekar/cfconnPSH

Conversation

@Shekar2004
Copy link
Copy Markdown
Member

Description

Mandatory Checklist

  • SHOULD update ChangeLog.md file(s) appropriately
    • Update src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.
      • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
    • Should not change ChangeLog.md if no new release is required, such as fixing test case only.
  • SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
  • SHOULD have proper test coverage for changes in pull request.
  • SHOULD NOT adjust version of module manually in pull request

Copilot AI review requested due to automatic review settings March 8, 2026 07:19
@azure-client-tools-bot-prd
Copy link
Copy Markdown

Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Az.Confluent module to onboard Confluent management-plane cmdlets generated from the 2025-08-18-preview API version, including refreshed docs, UX metadata, and test scaffolding.

Changes:

  • Added new cmdlet help/docs and UX descriptors for preview API resources (environments, clusters, topics, connectors, API keys, access endpoints).
  • Updated module manifest/solution wiring and bumped assembly version to 0.3.0.
  • Added/updated example markdowns and Pester test stubs + recordings for the new cmdlets.

Reviewed changes

Copilot reviewed 151 out of 175 changed files in this pull request and generated 13 comments.

Show a summary per file
File Description
src/Confluent/Confluent/help/Get-AzConfluentOrganizationEnvironment.md New cmdlet help for listing/getting organization environments
src/Confluent/Confluent/help/Get-AzConfluentOrganizationClusterApiKey.md New cmdlet help for fetching cluster API key details
src/Confluent/Confluent/help/Get-AzConfluentOrganization.md Updated help metadata/types/descriptions for consistency with newer generator output
src/Confluent/Confluent/help/Get-AzConfluentMarketplaceAgreement.md Updated help metadata/type casing + parameter descriptions
src/Confluent/Confluent/help/Get-AzConfluentAccessUser.md New cmdlet help for listing organization users
src/Confluent/Confluent/help/Get-AzConfluentAccessCluster.md New cmdlet help for cluster access listing
src/Confluent/Confluent/help/Az.Confluent.md Updated module cmdlet index to include new cmdlets
src/Confluent/Confluent/Az.Confluent.psd1 Manifest updates (required Az.Accounts version, exported functions list, generated date)
src/Confluent/Confluent.sln Solution wiring updated to new generated project path/name
src/Confluent/Confluent.Autorest/test/env.json Updated test environment values used by Pester tests
src/Confluent/Confluent.Autorest/test/Update-AzConfluentTopic.Tests.ps1 New Pester test stub for Update-AzConfluentTopic
src/Confluent/Confluent.Autorest/test/Update-AzConfluentOrganization.Tests.ps1 Existing Pester tests marked skipped
src/Confluent/Confluent.Autorest/test/Update-AzConfluentConnector.Tests.ps1 New Pester test stub for Update-AzConfluentConnector
src/Confluent/Confluent.Autorest/test/Test-AzConfluentValidationOrganizationV2.Tests.ps1 New Pester test stub for org validation V2
src/Confluent/Confluent.Autorest/test/Test-AzConfluentValidationOrganization.Tests.ps1 New Pester test stub for org validation
src/Confluent/Confluent.Autorest/test/Set-AzConfluentEnvironment.Tests.ps1 New Pester test stub for Set-AzConfluentEnvironment
src/Confluent/Confluent.Autorest/test/Set-AzConfluentConnector.Tests.ps1 New Pester test stub for Set-AzConfluentConnector
src/Confluent/Confluent.Autorest/test/Set-AzConfluentCluster.Tests.ps1 New Pester test stub for Set-AzConfluentCluster
src/Confluent/Confluent.Autorest/test/Remove-AzConfluentTopic.Tests.ps1 New Pester test stub for Remove-AzConfluentTopic
src/Confluent/Confluent.Autorest/test/Remove-AzConfluentOrganizationClusterApiKey.Tests.ps1 New Pester test stub for removing API keys
src/Confluent/Confluent.Autorest/test/Remove-AzConfluentEnvironment.Tests.ps1 New Pester test stub for removing environments
src/Confluent/Confluent.Autorest/test/Remove-AzConfluentConnector.Tests.ps1 New Pester test stub for removing connectors
src/Confluent/Confluent.Autorest/test/Remove-AzConfluentCluster.Tests.ps1 New Pester test stub for removing clusters
src/Confluent/Confluent.Autorest/test/Remove-AzConfluentAccessRoleBinding.Tests.ps1 New Pester test stub for removing role bindings
src/Confluent/Confluent.Autorest/test/New-AzConfluentTopic.Tests.ps1 New Pester test stub for creating topics
src/Confluent/Confluent.Autorest/test/New-AzConfluentOrganizationApiKey.Tests.ps1 New Pester test stub for creating org API keys
src/Confluent/Confluent.Autorest/test/New-AzConfluentOrganization.Tests.ps1 Existing Pester tests marked skipped
src/Confluent/Confluent.Autorest/test/New-AzConfluentEnvironment.Tests.ps1 New Pester test stub for creating environments
src/Confluent/Confluent.Autorest/test/New-AzConfluentConnector.Tests.ps1 New Pester test stub for creating connectors
src/Confluent/Confluent.Autorest/test/New-AzConfluentCluster.Tests.ps1 New Pester test stub for creating clusters
src/Confluent/Confluent.Autorest/test/New-AzConfluentAccessRoleBinding.Tests.ps1 New Pester test stub for creating role bindings
src/Confluent/Confluent.Autorest/test/Invoke-AzConfluentInviteAccessUser.Tests.ps1 New Pester test stub for inviting users
src/Confluent/Confluent.Autorest/test/Get-AzConfluentTopic.Tests.ps1 New Pester test stub for listing/getting topics
src/Confluent/Confluent.Autorest/test/Get-AzConfluentOrganizationSchemaRegistryCluster.Tests.ps1 New Pester test stub for schema registry clusters
src/Confluent/Confluent.Autorest/test/Get-AzConfluentOrganizationRegion.Tests.ps1 New Pester test stub for regions
src/Confluent/Confluent.Autorest/test/Get-AzConfluentOrganizationEnvironment.Tests.ps1 New Pester test stub for environments
src/Confluent/Confluent.Autorest/test/Get-AzConfluentOrganizationClusterApiKey.Tests.ps1 New Pester test stub for API keys
src/Confluent/Confluent.Autorest/test/Get-AzConfluentOrganizationCluster.Tests.ps1 New Pester test stub for clusters
src/Confluent/Confluent.Autorest/test/Get-AzConfluentOrganization.Tests.ps1 Existing Pester tests marked skipped
src/Confluent/Confluent.Autorest/test/Get-AzConfluentMarketplaceAgreement.Recording.json Updated recording to new subscription + preview API version
src/Confluent/Confluent.Autorest/how-to.md Updated module naming references in developer how-to
src/Confluent/Confluent.Autorest/generate-info.json Updated autorest generation id
src/Confluent/Confluent.Autorest/examples/Update-AzConfluentTopic.md New example content for updating topics
src/Confluent/Confluent.Autorest/examples/Update-AzConfluentConnector.md New example content for updating connectors
src/Confluent/Confluent.Autorest/examples/Test-AzConfluentValidationOrganizationV2.md New example content for org validation V2
src/Confluent/Confluent.Autorest/examples/Test-AzConfluentValidationOrganization.md New example content for org validation
src/Confluent/Confluent.Autorest/examples/Set-AzConfluentEnvironment.md New example content for setting environments
src/Confluent/Confluent.Autorest/examples/Set-AzConfluentConnector.md New example content for setting connectors
src/Confluent/Confluent.Autorest/examples/Set-AzConfluentCluster.md New example content for setting clusters
src/Confluent/Confluent.Autorest/examples/Remove-AzConfluentTopic.md New example content for removing topics
src/Confluent/Confluent.Autorest/examples/Remove-AzConfluentOrganizationClusterApiKey.md New example content for removing API keys
src/Confluent/Confluent.Autorest/examples/Remove-AzConfluentEnvironment.md New example content for removing environments
src/Confluent/Confluent.Autorest/examples/Remove-AzConfluentConnector.md New example content for removing connectors
src/Confluent/Confluent.Autorest/examples/Remove-AzConfluentCluster.md New example content for removing clusters
src/Confluent/Confluent.Autorest/examples/Remove-AzConfluentAccessRoleBinding.md New example content for removing role bindings
src/Confluent/Confluent.Autorest/examples/New-AzConfluentTopic.md New example content for creating topics
src/Confluent/Confluent.Autorest/examples/New-AzConfluentOrganizationApiKey.md New example content for creating org API keys
src/Confluent/Confluent.Autorest/examples/New-AzConfluentEnvironment.md New example content for creating environments
src/Confluent/Confluent.Autorest/examples/New-AzConfluentConnector.md New example content for creating connectors
src/Confluent/Confluent.Autorest/examples/New-AzConfluentCluster.md New example content for creating clusters
src/Confluent/Confluent.Autorest/examples/New-AzConfluentAccessRoleBinding.md New example content for creating role bindings
src/Confluent/Confluent.Autorest/examples/Invoke-AzConfluentInviteAccessUser.md New example content for inviting users
src/Confluent/Confluent.Autorest/examples/Get-AzConfluentTopic.md New example content for listing/getting topics
src/Confluent/Confluent.Autorest/examples/Get-AzConfluentOrganizationSchemaRegistryCluster.md New example content for schema registry clusters
src/Confluent/Confluent.Autorest/examples/Get-AzConfluentOrganizationRegion.md New example content for regions
src/Confluent/Confluent.Autorest/examples/Get-AzConfluentOrganizationEnvironment.md New example content for environments
src/Confluent/Confluent.Autorest/examples/Get-AzConfluentOrganizationClusterApiKey.md New example content for API keys
src/Confluent/Confluent.Autorest/examples/Get-AzConfluentOrganizationCluster.md New example content for clusters
src/Confluent/Confluent.Autorest/examples/Get-AzConfluentConnector.md New example content for connectors
src/Confluent/Confluent.Autorest/examples/Get-AzConfluentAccessUser.md New example content for listing users
src/Confluent/Confluent.Autorest/examples/Get-AzConfluentAccessServiceAccount.md New example content for listing service accounts
src/Confluent/Confluent.Autorest/examples/Get-AzConfluentAccessRoleBindingNameList.md New example content for role binding name list
src/Confluent/Confluent.Autorest/examples/Get-AzConfluentAccessRoleBinding.md New example content for listing role bindings
src/Confluent/Confluent.Autorest/examples/Get-AzConfluentAccessInvitation.md New example content for listing invitations
src/Confluent/Confluent.Autorest/examples/Get-AzConfluentAccessEnvironment.md New example content for listing environments (access)
src/Confluent/Confluent.Autorest/examples/Get-AzConfluentAccessCluster.md New example content for listing clusters (access)
src/Confluent/Confluent.Autorest/docs/Update-AzConfluentOrganization.md Updated reference doc metadata/type casing + description capitalization
src/Confluent/Confluent.Autorest/docs/Remove-AzConfluentOrganizationClusterApiKey.md New reference doc for removing org cluster API keys
src/Confluent/Confluent.Autorest/docs/Remove-AzConfluentOrganization.md Updated reference doc metadata/type casing
src/Confluent/Confluent.Autorest/docs/Remove-AzConfluentAccessRoleBinding.md New reference doc for removing role bindings
src/Confluent/Confluent.Autorest/docs/README.md Updated docs README module name casing
src/Confluent/Confluent.Autorest/docs/New-AzConfluentOrganization.md Updated reference doc (new parameters + metadata/type casing)
src/Confluent/Confluent.Autorest/docs/Get-AzConfluentOrganizationRegion.md New reference doc for regions
src/Confluent/Confluent.Autorest/docs/Get-AzConfluentOrganizationClusterApiKey.md New reference doc for API keys
src/Confluent/Confluent.Autorest/docs/Get-AzConfluentOrganization.md Updated reference doc metadata/type casing
src/Confluent/Confluent.Autorest/docs/Get-AzConfluentMarketplaceAgreement.md Updated reference doc metadata/type casing
src/Confluent/Confluent.Autorest/docs/Get-AzConfluentAccessRoleBindingNameList.md New reference doc for role binding name list
src/Confluent/Confluent.Autorest/docs/Get-AzConfluentAccessRoleBinding.md New reference doc for role bindings
src/Confluent/Confluent.Autorest/docs/Get-AzConfluentAccessCluster.md New reference doc for access cluster listing
src/Confluent/Confluent.Autorest/docs/Az.Confluent.md Updated module doc index and module metadata
src/Confluent/Confluent.Autorest/custom/README.md Updated custom cmdlets README module name/type casing
src/Confluent/Confluent.Autorest/custom/Az.Confluent.custom.psm1 Updated internal/private module import paths to new casing
src/Confluent/Confluent.Autorest/UX/Microsoft.Confluent/organizations.json Updated API version to 2025-08-18-preview
src/Confluent/Confluent.Autorest/UX/Microsoft.Confluent/organizations-environments.json New UX descriptor for environments endpoints/cmdlets
src/Confluent/Confluent.Autorest/UX/Microsoft.Confluent/organizations-environments-schemaRegistryClusters.json New UX descriptor for schema registry cluster get
src/Confluent/Confluent.Autorest/UX/Microsoft.Confluent/organizations-environments-clusters.json New UX descriptor for clusters get/remove
src/Confluent/Confluent.Autorest/UX/Microsoft.Confluent/organizations-environments-clusters-topics.json New UX descriptor for topics get/remove
src/Confluent/Confluent.Autorest/UX/Microsoft.Confluent/organizations-environments-clusters-connectors.json New UX descriptor for connectors get/remove
src/Confluent/Confluent.Autorest/UX/Microsoft.Confluent/organizations-deleteRoleBinding.json New UX descriptor for role binding deletion
src/Confluent/Confluent.Autorest/UX/Microsoft.Confluent/organizations-apiKeys.json New UX descriptor for API keys get/remove
src/Confluent/Confluent.Autorest/README.md Updated autorest config to preview spec + commit, and module naming
src/Confluent/Confluent.Autorest/Properties/AssemblyInfo.cs Version bump to 0.3.0

You can also share your feedback on Copilot code review. Take the survey.

@NoriZC
Copy link
Copy Markdown
Contributor

NoriZC commented Mar 8, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@NoriZC
Copy link
Copy Markdown
Contributor

NoriZC commented Mar 15, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

Copilot AI review requested due to automatic review settings March 15, 2026 06:12
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 150 out of 175 changed files in this pull request and generated 13 comments.


You can also share your feedback on Copilot code review. Take the survey.

@NoriZC
Copy link
Copy Markdown
Contributor

NoriZC commented Mar 15, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@NoriZC
Copy link
Copy Markdown
Contributor

NoriZC commented Mar 15, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

Copilot AI review requested due to automatic review settings March 15, 2026 15:19
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


You can also share your feedback on Copilot code review. Take the survey.

@NoriZC
Copy link
Copy Markdown
Contributor

NoriZC commented Mar 15, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

Copilot AI review requested due to automatic review settings March 15, 2026 16:50
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


You can also share your feedback on Copilot code review. Take the survey.

@NoriZC
Copy link
Copy Markdown
Contributor

NoriZC commented Mar 15, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

Copilot AI review requested due to automatic review settings March 15, 2026 19:28
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


You can also share your feedback on Copilot code review. Take the survey.

@NoriZC
Copy link
Copy Markdown
Contributor

NoriZC commented Mar 15, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@Shekar2004 Shekar2004 force-pushed the pgnanashekar/cfconnPSH branch from e64d27e to aaa5482 Compare March 15, 2026 20:22
Copilot AI review requested due to automatic review settings March 15, 2026 20:37
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


You can also share your feedback on Copilot code review. Take the survey.

@NoriZC
Copy link
Copy Markdown
Contributor

NoriZC commented Mar 15, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@Shekar2004 Shekar2004 force-pushed the pgnanashekar/cfconnPSH branch from 982812f to 6a28a8b Compare March 22, 2026 10:23
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

@NoriZC
Copy link
Copy Markdown
Contributor

NoriZC commented Mar 22, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@Shekar2004 Shekar2004 force-pushed the pgnanashekar/cfconnPSH branch from 6a28a8b to c687735 Compare March 22, 2026 12:30
@NoriZC
Copy link
Copy Markdown
Contributor

NoriZC commented Mar 22, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@Shekar2004
Copy link
Copy Markdown
Member Author

@NoriZC can you please approve the PR

@Shekar2004 Shekar2004 requested review from NoriZC and Copilot March 28, 2026 19:06
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 948 changed files in this pull request and generated 4 comments.

@NoriZC
Copy link
Copy Markdown
Contributor

NoriZC commented Mar 31, 2026

@Shekar2004 Please remove the changes under generated folder from the PR

Copilot AI review requested due to automatic review settings March 31, 2026 00:28
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 717 changed files in this pull request and generated 2 comments.

Write-Output "Written to $outPath"

# Verify
$parsed = Get-Content $outPath | ConvertFrom-Json
Copy link

Copilot AI Mar 31, 2026

Choose a reason for hiding this comment

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

Get-Content without -Raw returns an array of lines, and piping that into ConvertFrom-Json will attempt to parse each line as standalone JSON (likely failing when ConvertTo-Json outputs multi-line JSON). Use Get-Content -Raw (optionally with -Encoding utf8) or read the file via [System.IO.File]::ReadAllText(...) before ConvertFrom-Json.

Suggested change
$parsed = Get-Content $outPath | ConvertFrom-Json
$parsed = Get-Content -Path $outPath -Raw -Encoding UTF8 | ConvertFrom-Json

Copilot uses AI. Check for mistakes.
Comment on lines +1 to +4
$sub = "00000000-0000-0000-0000-000000000001"
$rg = "confluentorg-rg-fwh8oe"
$org = "confluentorg-63y1vz"
$apiVer = "2025-08-18-preview"
Copy link

Copilot AI Mar 31, 2026

Choose a reason for hiding this comment

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

The script hardcodes subscription/resource group/org names and writes a recording to a fixed relative location. That makes it easy to accidentally generate recordings with inconsistent identifiers or overwrite the expected file. Consider adding a param(...) block for these values (with sensible defaults) and validating/echoing the resolved output path before writing so usage is safer and more repeatable.

Copilot uses AI. Check for mistakes.
Copilot AI review requested due to automatic review settings March 31, 2026 00:35
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 150 out of 176 changed files in this pull request and generated 11 comments.


# ID used to uniquely identify this module
GUID = '957f617c-6cbe-4ea3-b46c-ab3ae8eddb3c'
GUID = '00000000-0000-0000-0000-000000000000'
Copy link

Copilot AI Mar 31, 2026

Choose a reason for hiding this comment

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

The module manifest GUID must be a stable, unique identifier. Setting it to all zeros can break module identity (e.g., side-by-side installs, caching, update behavior). Restore the previous GUID or generate a new non-zero GUID and keep it constant going forward.

Suggested change
GUID = '00000000-0000-0000-0000-000000000000'
GUID = '8f4b8b5e-2f60-4c5a-9f46-9b1a2d3c7e21'

Copilot uses AI. Check for mistakes.
@@ -1,9 +1,9 @@
#
# Module manifest for module 'Az.Confluent'
# Module manifest for module 'Az.confluent'
Copy link

Copilot AI Mar 31, 2026

Choose a reason for hiding this comment

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

This changes the module identity and asset filenames from 'Az.Confluent' to 'Az.confluent'. Module renames/casing changes are effectively breaking changes for users (Import-Module, documentation, build output paths, packaging) and can also break on case-sensitive file systems. If a rename is not intentional, keep the canonical module name/casing (Az.Confluent) and revert the associated DLL/psm1/format filenames accordingly.

Copilot uses AI. Check for mistakes.
Comment on lines +66 to +72
FormatsToProcess = 'Confluent.Autorest/Az.confluent.format.ps1xml'

# Modules to import as nested modules of the module specified in RootModule/ModuleToProcess
NestedModules = @('Confluent.Autorest/Az.Confluent.psm1')
NestedModules = @('Confluent.Autorest/Az.confluent.psm1')

# Functions to export from this module, for best performance, do not use wildcards and do not delete the entry, use an empty array if there are no functions to export.
FunctionsToExport = 'Get-AzConfluentMarketplaceAgreement',
'Get-AzConfluentOrganization', 'New-AzConfluentOrganization',
'Remove-AzConfluentOrganization', 'Update-AzConfluentOrganization'
FunctionsToExport = 'Get-AzConfluentAccessCluster', 'Get-AzConfluentAccessEnvironment',
Copy link

Copilot AI Mar 31, 2026

Choose a reason for hiding this comment

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

This changes the module identity and asset filenames from 'Az.Confluent' to 'Az.confluent'. Module renames/casing changes are effectively breaking changes for users (Import-Module, documentation, build output paths, packaging) and can also break on case-sensitive file systems. If a rename is not intentional, keep the canonical module name/casing (Az.Confluent) and revert the associated DLL/psm1/format filenames accordingly.

Copilot uses AI. Check for mistakes.
Comment on lines +23 to +24
[assembly: System.Reflection.AssemblyFileVersionAttribute("0.1.0.0")]
[assembly: System.Reflection.AssemblyVersionAttribute("0.1.0.0")]
Copy link

Copilot AI Mar 31, 2026

Choose a reason for hiding this comment

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

The assembly version is being reset from 0.2.2 to 0.1.0.0, which is a regression and can confuse versioning, packaging, and troubleshooting. Align these values with the intended module/release versioning strategy (at minimum, avoid decreasing versions).

Suggested change
[assembly: System.Reflection.AssemblyFileVersionAttribute("0.1.0.0")]
[assembly: System.Reflection.AssemblyVersionAttribute("0.1.0.0")]
[assembly: System.Reflection.AssemblyFileVersionAttribute("0.2.2.0")]
[assembly: System.Reflection.AssemblyVersionAttribute("0.2.2.0")]

Copilot uses AI. Check for mistakes.
- Additional information about change #1
-->
## Upcoming Release
* Fixed failing playback tests for `Update-AzConfluentOrganization` by adding the missing HTTP recording file
Copy link

Copilot AI Mar 31, 2026

Choose a reason for hiding this comment

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

The PR title/contents indicate new/updated cmdlets and an API version bump, but the Upcoming Release note only mentions a playback test fix. If this PR is intended for a user-facing release, add changelog entries describing the new cmdlets / API version update from the user's perspective (and include issue references if applicable).

Copilot uses AI. Check for mistakes.
Get-AzConfluentAccessCluster_ListExpanded: Environment ID not specified.
```

This This command lists all confluent clusters under a organization and resource group
Copy link

Copilot AI Mar 31, 2026

Choose a reason for hiding this comment

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

Remove the duplicated word 'This' to improve readability.

Suggested change
This This command lists all confluent clusters under a organization and resource group
This command lists all confluent clusters under a organization and resource group

Copilot uses AI. Check for mistakes.
Get-AzConfluentOrganizationRegion_ListExpanded: Expected '{' or '['. Was Null: null.
```

This commands lists all available regions for create schema registery clusters No newline at end of file
Copy link

Copilot AI Mar 31, 2026

Choose a reason for hiding this comment

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

Correct spelling/grammar in the example description ('registery' -> 'registry', 'This commands' -> 'This command', 'for create' -> 'to create').

Suggested change
This commands lists all available regions for create schema registery clusters
This command lists all available regions to create schema registry clusters

Copilot uses AI. Check for mistakes.
Comment on lines +17 to +20
Describe 'Update-AzConfluentTopic' {
It 'UpdateExpanded' -skip {
{ throw [System.NotImplementedException] } | Should -Not -Throw
}
Copy link

Copilot AI Mar 31, 2026

Choose a reason for hiding this comment

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

These newly added Pester tests are all skipped and contain NotImplementedException stubs, which provides no effective test coverage for the new cmdlets and can hide regressions. Convert at least one parameter set into a real playback test using the recording framework (and remove -skip), or avoid adding placeholder skipped tests if coverage isn't ready yet.

Copilot generated this review using guidance from repository custom instructions.
{
$loadEnvPath = Join-Path $PSScriptRoot 'loadEnv.ps1'
if (-Not (Test-Path -Path $loadEnvPath)) {
$loadEnvPath = Join-Path $PSScriptRoot '..\loadEnv.ps1'
Copy link

Copilot AI Mar 31, 2026

Choose a reason for hiding this comment

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

Using a Windows-style relative path ('..\loadEnv.ps1') can break on non-Windows agents/filesystems. Prefer Join-Path with '..' and 'loadEnv.ps1' as separate segments (or Resolve-Path) to keep tests cross-platform.

Suggested change
$loadEnvPath = Join-Path $PSScriptRoot '..\loadEnv.ps1'
$loadEnvPath = Join-Path (Join-Path $PSScriptRoot '..') 'loadEnv.ps1'

Copilot uses AI. Check for mistakes.
Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "Confluent.Autorest", "Confluent.Autorest", "{4CFEDFE4-24E2-0196-92A3-5C421A4353C3}"
EndProject
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Az.Confluent", "..\..\generated\Confluent\Confluent.Autorest\Az.Confluent.csproj", "{93473FB8-4CF6-459B-9D22-655A08F26979}"
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Az.confluent", "..\..\generated\confluent\Confluent.Autorest\Az.confluent.csproj", "{C35330FD-5A77-4151-8251-24C5AA707308}"
Copy link

Copilot AI Mar 31, 2026

Choose a reason for hiding this comment

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

The generated project path changed from '..\..\generated\Confluent\...' to '..\..\generated\confluent\...'. On case-sensitive filesystems (Linux/macOS), a casing mismatch between the actual folder name and this path will break builds. Ensure the generated output directory casing matches what the build produces, or keep the previous casing convention consistently across generated paths.

Suggested change
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Az.confluent", "..\..\generated\confluent\Confluent.Autorest\Az.confluent.csproj", "{C35330FD-5A77-4151-8251-24C5AA707308}"
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Az.confluent", "..\..\generated\Confluent\Confluent.Autorest\Az.confluent.csproj", "{C35330FD-5A77-4151-8251-24C5AA707308}"

Copilot uses AI. Check for mistakes.
@NoriZC
Copy link
Copy Markdown
Contributor

NoriZC commented Mar 31, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@VeryEarly VeryEarly enabled auto-merge (squash) March 31, 2026 00:59
@NoriZC
Copy link
Copy Markdown
Contributor

NoriZC commented Mar 31, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@VeryEarly VeryEarly merged commit e25cdb5 into Azure:main Mar 31, 2026
12 checks passed
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.

4 participants