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

Error while specifying Ingestion as the data source type for linked storage accounts #18322

Closed
sapphire-sk opened this issue Jun 6, 2022 · 7 comments · Fixed by Azure/azure-rest-api-specs#19365
Assignees
Labels
bug This issue requires a change to an existing behavior in the product in order to be resolved. customer-reported Issues that are reported by GitHub users external to the Azure organization. Mgmt This issue is related to a management-plane library. Monitor Monitor, Monitor Ingestion, Monitor Query

Comments

@sapphire-sk
Copy link

Bug Report

I am trying to link storage accounts with data source type as Ingestion
The API version 2020-03-01-preview supported DataSourceType enum with 'Ingestion' as one of the valid values for log analytics linked storage account.
However, this value was removed in 2020-08-01 leading to 'Invalid value' error when storage accounts are linked to a workspace where the data source type is specified as Ingestion

  • import path: ../services/operationalinsights/mgmt/2020-08-01/operationalinsights/
  • SDK version: github.com/Azure/azure-sdk-for-go v64.0.0
  • output of go version 1.18

Why is 'Ingestion' data source type no longer as part of the enum?

@ghost ghost added needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. customer-reported Issues that are reported by GitHub users external to the Azure organization. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Jun 6, 2022
@jhendrixMSFT jhendrixMSFT added Monitor - Operational Insights Mgmt This issue is related to a management-plane library. labels Jun 7, 2022
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Jun 7, 2022
@ghost ghost added the needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team label Jun 7, 2022
@ghost
Copy link

ghost commented Jun 7, 2022

Thank you for your feedback. This has been routed to the support team for assistance.

@navba-MSFT navba-MSFT self-assigned this Jun 8, 2022
@navba-MSFT
Copy link

@sapphire-sk Thanks for reaching out to us and sharing this feedback. We are looking into this issue. I will update this thread once I have more details on this.

@navba-MSFT
Copy link

@sapphire-sk Could you please share your use case here ? Also what what scenario you are trying to fulfill with linked storage account for ingestion? Awaiting your reply.

@sapphire-sk
Copy link
Author

Linking storage accounts to Log Analytics workspace allows having your own managed storage account to store the logs instead of a service owned storage account.

  • In previous API, it was possible to link a storage account where the data source type is 'Ingestion' meaning this allowed the logging agents to send 'Ingestion' data captured from the resources to the linked storage account. These logs can then be analyzed and then categorize them as Failed or Succeeded Ingestion

  • However, since the API no longer supports 'Ingestion', the logging agents cannot be configured to exclusively collect 'Ingestion' logs and send them to a customer managed storage account

@navba-MSFT
Copy link

@sapphire-sk I had filed this PR to address this Azure/azure-rest-api-specs#19365

Once the swagger update will be approved and merged by ARM, our product Owners will trigger the GO SDK generation.
Then it is a matter of a few days until it will be publicly available.

@navba-MSFT navba-MSFT added needs-author-feedback Workflow: More information is needed from author to address the issue. bug This issue requires a change to an existing behavior in the product in order to be resolved. and removed needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team labels Jun 14, 2022
@sapphire-sk
Copy link
Author

@navba-MSFT Thanks for the PR!

@ghost ghost added needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team and removed needs-author-feedback Workflow: More information is needed from author to address the issue. labels Jun 14, 2022
@navba-MSFT navba-MSFT removed the needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team label Jun 15, 2022
@navba-MSFT
Copy link

@sapphire-sk The PR has been merged now. So our Product Owners will check this further and then trigger the GO SDK generation. This will take few days to be available publicly. No fixed ETA can be shared for this now. With that we will proceed with archival of this thread. If you need any further assistance on this in future feel free to reopen this thread. We would be happy to help.

@navba-MSFT navba-MSFT removed the question The issue doesn't require a change to the product in order to be resolved. Most issues start as that label Jun 27, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Apr 11, 2023
@scottaddie scottaddie added Monitor Monitor, Monitor Ingestion, Monitor Query and removed Monitor - Operational Insights labels Sep 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug This issue requires a change to an existing behavior in the product in order to be resolved. customer-reported Issues that are reported by GitHub users external to the Azure organization. Mgmt This issue is related to a management-plane library. Monitor Monitor, Monitor Ingestion, Monitor Query
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants