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

{ServiceBus} Change expiration version to next breaking change window #25344

Merged
merged 3 commits into from
Feb 3, 2023

Conversation

evelyn-ys
Copy link
Member

@evelyn-ys evelyn-ys commented Feb 3, 2023

Description

CLI 2.45.0 release failed with

-  FAIL - HIGH severity: expired_parameter
    Parameter: servicebus queue update, `enable_partitioning` - Deprecated parameter is expired and should be removed.
    Parameter: servicebus queue update, `requires_duplicate_detection` - Deprecated parameter is expired and should be removed.
    Parameter: servicebus queue update, `requires_session` - Deprecated parameter is expired and should be removed.

Since 2.45.0 doesn't allow breaking changes, so change the deprecation info to next breaking change window which would be 2.49.0 then

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.

@ghost ghost added Auto-Assign Auto assign by bot Service Bus az servicebus labels Feb 3, 2023
@ghost ghost requested review from calvinhzy and yonzhan February 3, 2023 05:08
@ghost ghost assigned evelyn-ys Feb 3, 2023
@ghost ghost added this to the Feb 2023 (2023-03-07) milestone Feb 3, 2023
help='A boolean value that indicates whether the queue is to be partitioned across multiple message brokers.', deprecate_info=c.deprecate(hide=True, expiration='2.45.0'))
c.argument('requires_session', options_list=['--enable-session'], arg_type=get_three_state_flag(), help='A boolean value indicating whether the queue supports the concept of sessions.', deprecate_info=c.deprecate(hide=True, expiration='2.45.0'))
help='A boolean value that indicates whether the queue is to be partitioned across multiple message brokers.', deprecate_info=c.deprecate(hide=True, expiration='2.49.0'))
c.argument('requires_session', options_list=['--enable-session'], arg_type=get_three_state_flag(), help='A boolean value indicating whether the queue supports the concept of sessions.', deprecate_info=c.deprecate(hide=True, expiration='2.49.0'))
Copy link
Member

@jiasli jiasli Feb 3, 2023

Choose a reason for hiding this comment

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

This expiration was introduced by #24937.

Same issue happened multiple times before, such as #21892

Setting expiration is like setting a timebomb for ourselves. Maybe we can remove totally expiration, otherwise it can trigger failure again in 2.49.0 release.

@yonzhan
Copy link
Collaborator

yonzhan commented Feb 3, 2023

ServiceBus

jiasli
jiasli previously approved these changes Feb 3, 2023
zhoxing-ms
zhoxing-ms previously approved these changes Feb 3, 2023
@evelyn-ys evelyn-ys dismissed stale reviews from zhoxing-ms and jiasli via 2df439d February 3, 2023 06:14
@evelyn-ys evelyn-ys merged commit 32ab2fa into Azure:dev Feb 3, 2023
avgale pushed a commit to avgale/azure-cli that referenced this pull request Aug 24, 2023
…Azure#25344)

* change expiration version to next breaking change window

* warning message

* code owner
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.

4 participants