Skip to content

Conversation

@dabenhamMic
Copy link
Contributor

@dabenhamMic dabenhamMic commented Jan 13, 2022

Description

OperationalInsights cmdlets update for the following resources:

Workspace
Tables
Operations
once the auto code gen will be approved, we intend to delete the old cmdlets (for the resources specified above) and make a gradual transition to the auto-gen code

Checklist

  • I have read the Submitting Changes section of CONTRIBUTING.md
  • The title of the PR is clear and informative
  • The appropriate ChangeLog.md file(s) has been updated:
    • For any service, the ChangeLog.md file can be found at src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
    • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header -- no new version header should be added
  • The PR does not introduce breaking changes
  • If applicable, the changes made in the PR have proper test coverage
  • For public API changes to cmdlets:

@dabenhamMic
Copy link
Contributor Author

dabenhamMic commented Jan 13, 2022

I require some help here.
when using autorest to generate the cmdlets, "Operations" swagger: https://github.com/Azure/azure-rest-api-specs/blob/main/specification/operationalinsights/resource-manager/Microsoft.OperationalInsights/preview/2021-12-01-preview/Operations.json
created some files but no cmdlet and no documentation file (.md) were created as far as i saw.

@dingmeng-xue dingmeng-xue changed the base branch from generation to feature/OperationalInsights January 13, 2022 13:35
@dabenhamMic
Copy link
Contributor Author

@dingmeng-xue , I tried looking at the build failure reason but couldn't understand the root cause for it to fail.

@wyunchi-ms
Copy link
Contributor

  1. OperationalInsights is already migrated to autorest. Please retarget to generation branch.
  2. Please follow the same folder structure with generation branch.

@wyunchi-ms wyunchi-ms changed the base branch from feature/OperationalInsights to generation January 14, 2022 05:59
@wyunchi-ms
Copy link
Contributor

Closing this PR. New PR is #16903

@wyunchi-ms wyunchi-ms closed this Jan 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants