Adding Log Analytics query capabilities#3008
Conversation
ghost
added
the
|
Can one of the admins verify this patch? |
RikkiGibson
left a comment
RikkiGibson
left a comment
There was a problem hiding this comment.
Make sure you bump the minor version on the root package and update the changelog.
|
Ping me at rigibson@microsoft and I can onboard you to our npm publishing flow. |
|
BTW, did #3009 basically do some of this work automatically? |
|
Oh -- #3009 is because I added azure-sdk-for-node to the azure-rest-spec repo README's sdk generation, as well as here. I can remove it from the other repo, since i'm adding the mapping as part of this PR? The client code is the same, this PR adds 9 files:
|
|
I think it's good to have it the config in azure-rest-api-specs. Feel free to close #3009 (or I can if you don't have permission). |
|
Our generator config is slightly strange, but the gist is that package_service_mapping doesn't really get used by the AutorestCI bot, but swagger_to_sdk_config does get used, and all the configs over in azure-rest-api-specs get used. |
|
Got it. Made those fixes, and closed the auto PR. |
3 participants
Hi! This PR is to add client generation for the data plane portion of Log Analytics The corresponding treatment to generation the SDK can be found here.
I also wasn't able to find info about some parts of the process like npm release -- looks like there are some dead links around and the adx repo didn't have better info? Looking for some guidance/resources around what my next steps would be to publicly release after review :)