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

Release New-Relic CLI for stable version 2024-01-01 #7402

Conversation

dipeshbhakat-microsoft
Copy link
Member

@dipeshbhakat-microsoft dipeshbhakat-microsoft commented Mar 21, 2024


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

Related command

General Guidelines

  • Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
  • My extension version conforms to the Extension version schema

For new extensions:

About Extension Publish

There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Copy link

azure-client-tools-bot-prd bot commented Mar 21, 2024

⚠️Azure CLI Extensions Breaking Change Test
⚠️new-relic
rule cmd_name rule_message suggest_message
⚠️ 1006 - ParaAdd new-relic account list cmd new-relic account list added parameter pagination_limit
⚠️ 1006 - ParaAdd new-relic account list cmd new-relic account list added parameter pagination_token
⚠️ 1001 - CmdAdd new-relic monitor get-billing-info cmd new-relic monitor get-billing-info added
⚠️ 1006 - ParaAdd new-relic monitor list cmd new-relic monitor list added parameter pagination_limit
⚠️ 1006 - ParaAdd new-relic monitor list cmd new-relic monitor list added parameter pagination_token
⚠️ 1006 - ParaAdd new-relic monitor list-app-service cmd new-relic monitor list-app-service added parameter pagination_limit
⚠️ 1006 - ParaAdd new-relic monitor list-app-service cmd new-relic monitor list-app-service added parameter pagination_token
⚠️ 1001 - CmdAdd new-relic monitor list-connected-partner-resource cmd new-relic monitor list-connected-partner-resource added
⚠️ 1006 - ParaAdd new-relic monitor list-host cmd new-relic monitor list-host added parameter pagination_limit
⚠️ 1006 - ParaAdd new-relic monitor list-host cmd new-relic monitor list-host added parameter pagination_token
⚠️ 1001 - CmdAdd new-relic monitor list-linked-resource cmd new-relic monitor list-linked-resource added
⚠️ 1006 - ParaAdd new-relic monitor monitored-resource cmd new-relic monitor monitored-resource added parameter pagination_limit
⚠️ 1006 - ParaAdd new-relic monitor monitored-resource cmd new-relic monitor monitored-resource added parameter pagination_token
⚠️ 1011 - SubgroupAdd new-relic monitor monitored-subscription sub group new-relic monitor monitored-subscription added
⚠️ 1006 - ParaAdd new-relic monitor tag-rule list cmd new-relic monitor tag-rule list added parameter pagination_limit
⚠️ 1006 - ParaAdd new-relic monitor tag-rule list cmd new-relic monitor tag-rule list added parameter pagination_token
⚠️ 1006 - ParaAdd new-relic organization list cmd new-relic organization list added parameter pagination_limit
⚠️ 1006 - ParaAdd new-relic organization list cmd new-relic organization list added parameter pagination_token
⚠️ 1006 - ParaAdd new-relic plan list cmd new-relic plan list added parameter pagination_limit
⚠️ 1006 - ParaAdd new-relic plan list cmd new-relic plan list added parameter pagination_token

Copy link

Hi @dipeshbhakat-microsoft,
Please write the description of changes which can be perceived by customers into HISTORY.rst.
If you want to release a new extension version, please update the version in setup.py as well.

@yonzhan
Copy link
Collaborator

yonzhan commented Mar 21, 2024

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

@dipeshbhakat-microsoft dipeshbhakat-microsoft marked this pull request as draft March 21, 2024 07:17
@yonzhan yonzhan requested a review from jsntcy March 21, 2024 09:08
@dipeshbhakat-microsoft dipeshbhakat-microsoft changed the title NewRelic CLI refresh Release for New-Relic CLI for stable version 2024-01-01 Mar 21, 2024
@dipeshbhakat-microsoft dipeshbhakat-microsoft changed the title Release for New-Relic CLI for stable version 2024-01-01 Release New-Relic CLI for stable version 2024-01-01. Mar 21, 2024
@dipeshbhakat-microsoft dipeshbhakat-microsoft marked this pull request as ready for review March 21, 2024 12:54
@yonzhan
Copy link
Collaborator

yonzhan commented Mar 21, 2024

Please fix CI issues

@@ -51,6 +51,8 @@ def _build_arguments_schema(cls, *args, **kwargs):
required=True,
)
_args_schema.resource_group = AAZResourceGroupNameArg(
options=["--resource-group"],
Copy link
Member

Choose a reason for hiding this comment

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

It seems this change introduces a breaking change (Previously it supported both "--resource-group" and "--g", but now it only supports "--resource-group"). Please check other breaking changes as well.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks @jsntcy for pointing out. I think this is not expected. I will revert this change. We should be able to support both "--resource-group" and "--g"

Copy link
Member Author

Choose a reason for hiding this comment

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

@jsntcy I have reverted the changes which were breaking changes. Please have a look if anything else is breaking.

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

I have now added the remaining test cases and fixed the breaking changes.
We can merge if everything gets approved

Copy link
Member Author

Choose a reason for hiding this comment

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

@necusjz all work from my side is done now. Let me know if anything else is required from my side. Also please remove do not merge tag. We can merge now if the PR gets approved.

@jsntcy
Copy link
Member

jsntcy commented Mar 25, 2024

❌Azure CLI Extensions Breaking Change Test

@dipeshbhakat-microsoft,
It seems there are lots of breaking changes introduced by this PR and I think many are unexpected, for example previously supported both "--resource-group" and "--g", but now only support "--resource-group".
If you do want those breaking changes, you need release them in breaking change window.

@necusjz, @AllyW, please help confirm.

@jsntcy jsntcy requested a review from AllyW March 25, 2024 05:49
@dipeshbhakat-microsoft dipeshbhakat-microsoft changed the title Release New-Relic CLI for stable version 2024-01-01. Release New-Relic CLI for stable version 2024-01-01(Please don't merge it, addition of 2 test cases pending) Mar 26, 2024
@dipeshbhakat-microsoft dipeshbhakat-microsoft changed the title Release New-Relic CLI for stable version 2024-01-01(Please don't merge it, addition of 2 test cases pending) Release New-Relic CLI for stable version 2024-01-01 Mar 26, 2024
@dipeshbhakat-microsoft dipeshbhakat-microsoft force-pushed the features/users/dipeshbhakat/nrclirefresh branch from dc2ecb7 to bcc1242 Compare March 26, 2024 10:52
Copy link

⚠️ Suggestions

Module: new-relic

  • Update version to 1.0.0b2 in setup.py
  • Set azext.isPreview to true in azext_new-relic/azext_metadata.json if not exists

Notes

  • Stable/preview tag is inherited from last release. If needed, please add stable/preview label to modify it.
  • Major/minor/patch/pre increment of version number is calculated by pull request code changes automatically. If needed, please add major/minor/patch/pre label to adjust it.
  • For more info about extension versioning, please refer to Extension version schema

@dipeshbhakat-microsoft
Copy link
Member Author

⚠️ Suggestions

Module: new-relic

  • Update version to 1.0.0b2 in setup.py
  • Set azext.isPreview to true in azext_new-relic/azext_metadata.json if not exists

Notes

  • Stable/preview tag is inherited from last release. If needed, please add stable/preview label to modify it.
  • Major/minor/patch/pre increment of version number is calculated by pull request code changes automatically. If needed, please add major/minor/patch/pre label to adjust it.
  • For more info about extension versioning, please refer to Extension version schema

This is a stable version release so updated the version to 1.0.0 and removed azext.isPreview = true from azext_new-relic/azext_metadata.json.

Please let me know if anything else is required

@dipeshbhakat-microsoft
Copy link
Member Author

@jsntcy @necusjz can we please approve and merge it if anything is not pending.

@jsntcy jsntcy requested a review from kairu-ms April 2, 2024 05:13
@dipeshbhakat-microsoft
Copy link
Member Author

@kairu-ms / @jsntcy can we please remove do-not-merge lable and complete the review. Its pending from long.

Copy link
Contributor

@kairu-ms kairu-ms left a comment

Choose a reason for hiding this comment

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

LGTM

@kairu-ms kairu-ms merged commit 7429e7a into Azure:main Apr 8, 2024
17 checks passed
@azclibot
Copy link
Collaborator

azclibot commented Apr 8, 2024

[Release] Update index.json for extension [ new-relic ] : https://dev.azure.com/azclitools/release/_build/results?buildId=146406&view=results

blackchoey pushed a commit to blackchoey/azure-cli-extensions that referenced this pull request Apr 29, 2024
* NewRelic CLI refresh

* fix issues found in manual testing

* Fix tag rules commands

* Update command for monitored-subscription

* Update history and version number

* use short abbreviation for monitored_subscription_list

* Delete recordings and fix flake8 issues

* Remove default value from configuration-name

* fix tests

* Fix tests and remove unused reference

* Fix issues

* Fix all integration tests

* Add monitored subscription test cases

* remove commented test cases

* Remove trailling whitespace

* Undo breaking changes

* Revert get-metric-status to get-metric-statu

* Add more test cases and fix breaking changes

* Fix GetMetric Status breaking change

* Fix ingestionkey secret in recordings

* Update changelog to trigger build again

* Use Production region to run tests as partner staging is not working

* Fix build

* Attempt to fix build

* Fix versioning
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.

7 participants