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

OperationalInsights - Unsupported API version #24369

Closed
1 of 6 tasks
aarononeal opened this issue Jan 5, 2023 · 7 comments · Fixed by #24512
Closed
1 of 6 tasks

OperationalInsights - Unsupported API version #24369

aarononeal opened this issue Jan 5, 2023 · 7 comments · Fixed by #24512
Assignees
Labels
Client This issue points to a problem in the data-plane of the library. Mgmt This issue is related to a management-plane library. Monitor Monitor, Monitor Ingestion, Monitor Query needs-author-feedback Workflow: More information is needed from author to address the issue.

Comments

@aarononeal
Copy link

  • Package Name: @azure/arm-operationalinsights
  • Package Version: "^8.0.1"
  • Operating system: All
  • nodejs
    • version: v18.12.1
  • browser
    • name/version:
  • typescript
    • version:
  • Is the bug related to documentation in

Describe the bug
Calls with OperationalInsightsManagementClient fail due to unsupported API version.

To Reproduce

managementClient = new OperationalInsightsManagementClient(
    this.credential,
    this.subscriptionId
  );
managementClient.tables.listByWorkspace(...);

Exception: No registered resource provider found for location '{location}' and API version '2021-06-01' for type 'workspaces/tables'. The supported api-versions are '2017-04-26-preview, 2020-03-01-preview, 2020-08-01, 2021-12-01-preview, 2022-10-01'.

Expected behavior
Call should succeed without API version error.

Screenshots
N/A

Additional context
Not sure why the supported service versions were changed without updating the SDK but this appears to be out of sync. Perhaps there should be a way to override the API version used by the client.

@ghost ghost added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Jan 5, 2023
@aarononeal
Copy link
Author

Until this is fixed, you can work around as follows.

    this.managementClient.pipeline.addPolicy({ name: "API Version",
      sendRequest: (request: PipelineRequest, next: SendRequest) => {
        let url = new URL(request.url);
        url.searchParams.set("api-version", "2022-10-01");
        request.url = url.toString();
        return next(request);
    }}, { phase: "Serialize" });

@xirzec xirzec added Client This issue points to a problem in the data-plane of the library. Mgmt This issue is related to a management-plane library. Monitor - Operational Insights labels Jan 5, 2023
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Jan 5, 2023
@MaryGao
Copy link
Member

MaryGao commented Jan 11, 2023

@aarononeal Thanks for the reporting! I will take a look at this issue.

@MaryGao MaryGao assigned MaryGao and unassigned qiaozha Jan 11, 2023
@MaryGao
Copy link
Member

MaryGao commented Jan 17, 2023

@aarononeal I could re-produce in my local. For short-term solution I would suggest you to have a try with our latest preview version(9.0.0-beta.1) which uses the 2021-12-01-preview as the default version.

For long-term We would suggest the service team to refresh our package with the latest one 2022-10-01, I will keep you updated!

@kazrael2119 Could you try to release the package with latest tag?

@MaryGao MaryGao added the needs-author-feedback Workflow: More information is needed from author to address the issue. label Jan 17, 2023
@ghost ghost added the no-recent-activity There has been no recent activity on this issue. label Jan 25, 2023
@ghost
Copy link

ghost commented Jan 25, 2023

Hi, we're sending this friendly reminder because we haven't heard back from you in a while. We need more information about this issue to help address it. Please be sure to give us your input within the next 7 days. If we don't hear back from you within 14 days of this comment the issue will be automatically closed. Thank you!

@MaryGao
Copy link
Member

MaryGao commented Feb 8, 2023

The pr is ready #24512, but considering there is breaking change so we are confirming with service team.

@ghost ghost removed the no-recent-activity There has been no recent activity on this issue. label Feb 8, 2023
@MaryGao
Copy link
Member

MaryGao commented Feb 8, 2023

The pr is ready #24512, but considering there is breaking change so we are confirming with service team.

Offline confirmed with service team we could release with latest change

MaryGao pushed a commit that referenced this issue Feb 9, 2023
@kazrael2119
Copy link
Contributor

@github-actions github-actions bot locked and limited conversation to collaborators May 10, 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
Client This issue points to a problem in the data-plane of the library. Mgmt This issue is related to a management-plane library. Monitor Monitor, Monitor Ingestion, Monitor Query needs-author-feedback Workflow: More information is needed from author to address the issue.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants