Skip to content

[Eventhub] az eventhubs namespace: Add nsp-configuration show and nsp-configuration list#31846

Merged
kairu-ms merged 4 commits intoAzure:devfrom
schaudhari6254888:2025
Jul 22, 2025
Merged

[Eventhub] az eventhubs namespace: Add nsp-configuration show and nsp-configuration list#31846
kairu-ms merged 4 commits intoAzure:devfrom
schaudhari6254888:2025

Conversation

@schaudhari6254888
Copy link
Copy Markdown
Member

@schaudhari6254888 schaudhari6254888 commented Jul 20, 2025

Related command

Description

New cmdlets added

  1. az eventhubs namespace nsp-configuration show
  2. az eventhubs namespace nsp-configuration list
    Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature


This checklist is used to make sure that common guidelines for a pull request are followed.

Copilot AI review requested due to automatic review settings July 20, 2025 10:07
@azure-client-tools-bot-prd
Copy link
Copy Markdown

azure-client-tools-bot-prd bot commented Jul 20, 2025

️✔️AzureCLI-FullTest
️✔️acr
️✔️latest
️✔️3.12
️✔️3.9
️✔️acs
️✔️latest
️✔️3.12
️✔️3.9
️✔️advisor
️✔️latest
️✔️3.12
️✔️3.9
️✔️ams
️✔️latest
️✔️3.12
️✔️3.9
️✔️apim
️✔️latest
️✔️3.12
️✔️3.9
️✔️appconfig
️✔️latest
️✔️3.12
️✔️3.9
️✔️appservice
️✔️latest
️✔️3.12
️✔️3.9
️✔️aro
️✔️latest
️✔️3.12
️✔️3.9
️✔️backup
️✔️latest
️✔️3.12
️✔️3.9
️✔️batch
️✔️latest
️✔️3.12
️✔️3.9
️✔️batchai
️✔️latest
️✔️3.12
️✔️3.9
️✔️billing
️✔️latest
️✔️3.12
️✔️3.9
️✔️botservice
️✔️latest
️✔️3.12
️✔️3.9
️✔️cdn
️✔️latest
️✔️3.12
️✔️3.9
️✔️cloud
️✔️latest
️✔️3.12
️✔️3.9
️✔️cognitiveservices
️✔️latest
️✔️3.12
️✔️3.9
️✔️compute_recommender
️✔️latest
️✔️3.12
️✔️3.9
️✔️computefleet
️✔️latest
️✔️3.12
️✔️3.9
️✔️config
️✔️latest
️✔️3.12
️✔️3.9
️✔️configure
️✔️latest
️✔️3.12
️✔️3.9
️✔️consumption
️✔️latest
️✔️3.12
️✔️3.9
️✔️container
️✔️latest
️✔️3.12
️✔️3.9
️✔️containerapp
️✔️latest
️✔️3.12
️✔️3.9
️✔️core
️✔️latest
️✔️3.12
️✔️3.9
️✔️cosmosdb
️✔️latest
️✔️3.12
️✔️3.9
️✔️databoxedge
️✔️latest
️✔️3.12
️✔️3.9
️✔️dls
️✔️latest
️✔️3.12
️✔️3.9
️✔️dms
️✔️latest
️✔️3.12
️✔️3.9
️✔️eventgrid
️✔️latest
️✔️3.12
️✔️3.9
️✔️eventhubs
️✔️latest
️✔️3.12
️✔️3.9
️✔️feedback
️✔️latest
️✔️3.12
️✔️3.9
️✔️find
️✔️latest
️✔️3.12
️✔️3.9
️✔️hdinsight
️✔️latest
️✔️3.12
️✔️3.9
️✔️identity
️✔️latest
️✔️3.12
️✔️3.9
️✔️iot
️✔️latest
️✔️3.12
️✔️3.9
️✔️keyvault
️✔️latest
️✔️3.12
️✔️3.9
️✔️lab
️✔️latest
️✔️3.12
️✔️3.9
️✔️managedservices
️✔️latest
️✔️3.12
️✔️3.9
️✔️maps
️✔️latest
️✔️3.12
️✔️3.9
️✔️marketplaceordering
️✔️latest
️✔️3.12
️✔️3.9
️✔️monitor
️✔️latest
️✔️3.12
️✔️3.9
️✔️mysql
️✔️latest
️✔️3.12
️✔️3.9
️✔️netappfiles
️✔️latest
️✔️3.12
️✔️3.9
️✔️network
️✔️latest
️✔️3.12
️✔️3.9
️✔️policyinsights
️✔️latest
️✔️3.12
️✔️3.9
️✔️privatedns
️✔️latest
️✔️3.12
️✔️3.9
️✔️profile
️✔️latest
️✔️3.12
️✔️3.9
️✔️rdbms
️✔️latest
️✔️3.12
️✔️3.9
️✔️redis
️✔️latest
️✔️3.12
️✔️3.9
️✔️relay
️✔️latest
️✔️3.12
️✔️3.9
️✔️resource
️✔️latest
️✔️3.12
️✔️3.9
️✔️role
️✔️latest
️✔️3.12
️✔️3.9
️✔️search
️✔️latest
️✔️3.12
️✔️3.9
️✔️security
️✔️latest
️✔️3.12
️✔️3.9
️✔️servicebus
️✔️latest
️✔️3.12
️✔️3.9
️✔️serviceconnector
️✔️latest
️✔️3.12
️✔️3.9
️✔️servicefabric
️✔️latest
️✔️3.12
️✔️3.9
️✔️signalr
️✔️latest
️✔️3.12
️✔️3.9
️✔️sql
️✔️latest
️✔️3.12
️✔️3.9
️✔️sqlvm
️✔️latest
️✔️3.12
️✔️3.9
️✔️storage
️✔️latest
️✔️3.12
️✔️3.9
️✔️synapse
️✔️latest
️✔️3.12
️✔️3.9
️✔️telemetry
️✔️latest
️✔️3.12
️✔️3.9
️✔️util
️✔️latest
️✔️3.12
️✔️3.9
️✔️vm
️✔️latest
️✔️3.12
️✔️3.9

@azure-client-tools-bot-prd
Copy link
Copy Markdown

azure-client-tools-bot-prd bot commented Jul 20, 2025

⚠️AzureCLI-BreakingChangeTest
⚠️eventhubs
rule cmd_name rule_message suggest_message
⚠️ 1010 - ParaPropUpdate eventhubs eventhub update cmd eventhubs eventhub update update parameter min_compaction_lag_in_mins: updated property name from min_compaction_lag_in_mins to min_compaction_lag_time_in_minutes
⚠️ 1011 - SubgroupAdd eventhubs namespace nsp-configuration sub group eventhubs namespace nsp-configuration added

@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented Jul 20, 2025

Thank you for your contribution! We will review the pull request and get back to you soon.

@github-actions
Copy link
Copy Markdown

The git hooks are available for azure-cli and azure-cli-extensions repos. They could help you run required checks before creating the PR.

Please sync the latest code with latest dev branch (for azure-cli) or main branch (for azure-cli-extensions).
After that please run the following commands to enable git hooks:

pip install azdev --upgrade
azdev setup -c <your azure-cli repo path> -r <your azure-cli-extensions repo path>

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

This pull request removes multiapi references from the azure-mgmt-Eventhub and azure-mgmt-servicebus packages. The changes primarily involve standardizing the naming convention for compaction lag parameters across EventHub commands and ensuring consistent use of the updated AAZ (Azure CLI Auto-generated) command structure.

Key changes:

  • Standardized parameter naming from min_compaction_lag_in_mins to min_compaction_lag_time_in_minutes
  • Added a new cli_eventhub_update function to handle EventHub updates using the Create command class
  • Updated schema definitions to use consistent property names for retention description

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
event_hub_entity.py Added new update function using AAZ Create command with proper parameter mapping
_update.py Updated parameter names and schema mappings for compaction lag settings
_create.py Standardized parameter naming convention for min compaction lag time

Comment thread src/azure-cli/azure/cli/command_modules/eventhubs/operations/event_hub_entity.py Outdated
Comment thread src/azure-cli/azure/cli/command_modules/eventhubs/operations/event_hub_entity.py Outdated
@schaudhari6254888 schaudhari6254888 changed the title {Eventhub/Servicebus} Remove azure-mgmt-Eventhub azure-mgmt-servicebus multiapi references {Eventhub} az eventhubs eventhub Jul 20, 2025
@schaudhari6254888 schaudhari6254888 changed the title {Eventhub} az eventhubs eventhub {Eventhub} az eventhubs namespace Jul 21, 2025
@kairu-ms kairu-ms changed the title {Eventhub} az eventhubs namespace [Eventhub] az eventhubs namespace: Add nsp-configuration show and nsp-configuration list Jul 22, 2025
@kairu-ms kairu-ms self-assigned this Jul 22, 2025
@kairu-ms kairu-ms merged commit 12e11b4 into Azure:dev Jul 22, 2025
48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Auto-Assign Auto assign by bot Service Bus az servicebus

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants