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

Add new properties to Log Analytics SDK #20200

Closed
dhaiman opened this issue Aug 10, 2021 · 7 comments
Closed

Add new properties to Log Analytics SDK #20200

dhaiman opened this issue Aug 10, 2021 · 7 comments
Labels
Client This issue points to a problem in the data-plane of the library. feature-request This issue requires a new behavior in the product in order be resolved. Monitor Monitor, Monitor Ingestion, Monitor Query

Comments

@dhaiman
Copy link

dhaiman commented Aug 10, 2021

Please add Log Analytics Workspace missing properties to SDK.
Attaching last PR: https://github.com/Azure/azure-rest-api-specs/blob/56cd4784756e65c303ed7aba007ffa8c7540a587/specification/operationalinsights/resource-manager/Microsoft.OperationalInsights/stable/2020-10-01/Workspaces.json#L593

Thanks!

@swathipil swathipil added Monitor - Log Analytics Client This issue points to a problem in the data-plane of the library. labels Aug 10, 2021
@swathipil swathipil added the feature-request This issue requires a new behavior in the product in order be resolved. label Aug 10, 2021
@lzchen
Copy link
Member

lzchen commented Aug 10, 2021

@swathipil
I believe this is mis-assigned. I do not own the Log analytics SDK.

@swathipil
Copy link
Member

@lzchen, thanks for the heads up! will look into who should own this.

@swathipil swathipil assigned srnagar and unassigned lzchen Aug 10, 2021
@swathipil
Copy link
Member

@dhaiman - @srnagar has a better idea of who should be able to help you with this.

@yoramsinger
Copy link

@dabenhamMic can you please take a look. Thanks.

@dabenhamMic
Copy link

dabenhamMic commented Aug 11, 2021

hey @yoramsinger , @dhaiman
please follow these steps:

Open a ticket here
a. Check all Languages that you need (Python)
b.Paste the PR url from swagger update

  • Important note: make sure that the swagger itself is up to date and also the package configures in the readme.md file - under " ### Basic Information"

@srnagar srnagar assigned dabenhamMic and unassigned srnagar Aug 13, 2021
@dabenhamMic
Copy link

Hi @dhaiman , please follow the above instructions I added.

@dabenhamMic dabenhamMic removed their assignment Aug 15, 2021
@scottaddie scottaddie added Monitor Monitor, Monitor Ingestion, Monitor Query and removed Monitor - Log Analytics labels Sep 28, 2023
Copy link

Hi @dhaiman, we deeply appreciate your input into this project. Regrettably, this issue has remained inactive for over 2 years, leading us to the decision to close it. We've implemented this policy to maintain the relevance of our issue queue and facilitate easier navigation for new contributors. If you still believe this topic requires attention, please feel free to create a new issue, referencing this one. Thank you for your understanding and ongoing support.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Mar 15, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Mar 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Client This issue points to a problem in the data-plane of the library. feature-request This issue requires a new behavior in the product in order be resolved. Monitor Monitor, Monitor Ingestion, Monitor Query
Projects
None yet
Development

No branches or pull requests

7 participants