Skip to content

Conversation

@Jing-song
Copy link
Contributor

@Jing-song Jing-song commented Jan 13, 2022

Description

Microsot.OperationalInsights: Required CLI changes #20903
close #20903
wait SDK release(done)

Testing Guide

History Notes

[Monitor] az monitor log-analytics workspace table: Add new command create, delete and search-job create to support Microsoft/Custom log/Search Results table operations,
[Monitor] az monitor log-analytics workspace update: Add a new parameter --data-collection-rule to support update defaultDataCollectionRuleResourceId


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

@Jing-song Jing-song self-assigned this Jan 13, 2022
@Jing-song Jing-song changed the title [Monitor] az monitor log-analytics workspace table: Add new command create and delete to support table operations [Monitor] az monitor log-analytics workspace table: Add new command create and delete to support table operations, az monitor log-analytics workspace update: Add a new parameter --data-collection-rule to support update defaultDataCollectionRuleResourceId Jan 13, 2022
@yonzhan yonzhan requested review from jsntcy and kairu-ms January 13, 2022 09:24
@yonzhan yonzhan added this to the Jan 2022 (2022-02-08) milestone Jan 13, 2022
@yonzhan
Copy link
Collaborator

yonzhan commented Jan 13, 2022

Monitor

@Jing-song Jing-song marked this pull request as ready for review January 20, 2022 09:52
@Jing-song
Copy link
Contributor Author

Edge build

@Jing-song
Copy link
Contributor Author

Jing-song commented Feb 15, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@Jing-song Jing-song changed the title [Monitor] az monitor log-analytics workspace table: Add new command create and delete to support table operations, az monitor log-analytics workspace update: Add a new parameter --data-collection-rule to support update defaultDataCollectionRuleResourceId [Monitor] az monitor log-analytics workspace table: Add new command create, delete and search-job create to support Microsoft/Custom log/Search Results table operations, az monitor log-analytics workspace update: Add a new parameter --data-collection-rule to support update defaultDataCollectionRuleResourceId Feb 16, 2022
@jsntcy
Copy link
Member

jsntcy commented Feb 17, 2022

@PARADISSEEKR Overall LGTM, just a minor suggestion.

@Jing-song
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@evelyn-ys
Copy link
Member

PR title is so loooong. Better to move to History Notes part in PR description

'azure-mgmt-keyvault==9.3.0',
'azure-mgmt-kusto~=0.3.0',
'azure-mgmt-loganalytics~=12.0.0',
'azure-mgmt-loganalytics~=13.0.0b1',
Copy link
Member

Choose a reason for hiding this comment

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

If the SDK is beta version, why not support in cli extension?

@jsntcy jsntcy merged commit 0998922 into Azure:dev Feb 17, 2022
@Jing-song Jing-song deleted the jins-logana branch July 5, 2022 02:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Microsot.OperationalInsights: Required CLI changes.

7 participants