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

missing response body #1360

Closed
Tracked by #1356
Alancere opened this issue May 21, 2024 · 4 comments
Closed
Tracked by #1356

missing response body #1360

Alancere opened this issue May 21, 2024 · 4 comments
Assignees
Labels

Comments

@Alancere
Copy link
Member

tsp: https://github.com/Azure/azure-rest-api-specs/blob/main/specification/databasewatcher/DatabaseWatcher.Management/watcher.tsp#L126

image

@jhendrixMSFT
Copy link
Member

This is coming from tcgc, the SdkMethod doesn't have a response type.

@tadelesh
Copy link
Member

tadelesh commented May 22, 2024

the root cause is the tsp has a wrong definition of custom lro.

@tadelesh tadelesh closed this as not planned Won't fix, can't repro, duplicate, stale May 22, 2024
@jhendrixMSFT
Copy link
Member

@tadelesh so I can learn more about this, what should the definition look like?

@tadelesh
Copy link
Member

tadelesh commented May 22, 2024

start(
    ...ResourceInstanceParameters<Watcher>,
  ): ArmAcceptedLroResponse<ArmLroLocationHeader<LroHeaders=
  ArmLroLocationHeader<
    FinalResult=Watcher
  >>> | ArmResponse<Watcher> | ErrorResponse;

then we could get right finalResult from getLroMetadata.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants