AzureResourceCollector returning resources with the wrong type about apimanagementarmtemplatecreator HOT 23 CLOSED

Hmm I need to check this up

from apimanagementarmtemplatecreator.

You are correct, the latest version is corrupt.

Name is also empty

from apimanagementarmtemplatecreator.

Thanks for confirming! Yes, forgot about name.

from apimanagementarmtemplatecreator.

AzureResourceCollector.cs hasn't changed since August last year, so it must be a recent change on the Azure side. FWIW, I tried using 2018-06-01-preview for the api version in AzureResourceCollector, but didn't help.

from apimanagementarmtemplatecreator.

For the REST API it looks like you have to retrieve an operation policy with the path /operation/{opid}/policies/policy instead of /operation/{opid}/policies (and looping through it).

from apimanagementarmtemplatecreator.

This is a regression in recent APIM update. Fix is pending as soon as possible.

from apimanagementarmtemplatecreator.

Thanks for the update.

from apimanagementarmtemplatecreator.

The problem is at the moment that the type returned is "one level to high" from the APIM REST Api when returned in lists.
So if we request for operations ie: apis/{apiid}/operations, the type property is set to apis even if the json object is for apis/operations.
If we request policies for an operation i.e apis/{apiid}/operations/{opid}/policies type property is set to apis/operations even if the json object is for apis/operations/policies.
I've asked the product team for info so let's see I'll come back with more information as soon as I have updates.

from apimanagementarmtemplatecreator.

The fix for the issue is being rolled out now.

from apimanagementarmtemplatecreator.

@VitaliyKurokhtin How long will it take to get rolled out? Still seeing issues.

from apimanagementarmtemplatecreator.

May take till end of the day and early tomorrow. If you don't mind you could share your service name.

from apimanagementarmtemplatecreator.

Tested and at least one of the instances I have access to is working now as normal now.

from apimanagementarmtemplatecreator.

Still broken for me. @VitaliyKurokhtin where can I email you my service name? Wait are you actually from Microsoft? ;)

from apimanagementarmtemplatecreator.

@joeyeng send me a mail at [email protected]. All services will be updated soon enough though.

from apimanagementarmtemplatecreator.

Thanks, I'll check again on Monday. If it's still not updated I'll let you know. Have a good weekend!

from apimanagementarmtemplatecreator.

It's working for me now. Thanks!

from apimanagementarmtemplatecreator.

@VitaliyKurokhtin I'm seeing "contentFormat" property missing for operation policies and "generatedSample" missing for operation response representation.

from apimanagementarmtemplatecreator.

Only when you're getting list of policies, correct? If you fetch single policy it does return contentFormat. How sure are you that it was returned before? I see that it wasn't. Definitely a bug though and will be fixed.

For generated sample I'll need to look at your data, so send me details over email.

from apimanagementarmtemplatecreator.

Yes, the tool is getting list of policies instead of single. I'm just basing off the ARM templates we generated with the tool prior to the bug. The tool doesn't seem to have any code that is adding those properties, so I assume the REST api was providing them before. I'll send you an email of the generated sample we used to get.

from apimanagementarmtemplatecreator.

@VitaliyKurokhtin Sorry, false alarm! I was looking at an ARM template I generated where I was retrieving single policy (because of the bug) instead of the list.

from apimanagementarmtemplatecreator.

@joeyeng ok. There is still an issue where querying for a list does not return contentFormat for each policy, and it will be fixed.

from apimanagementarmtemplatecreator.

@VitaliyKurokhtin Any ETA on the contentFormat fix? Is there an open issue for this on github or somewhere else?

from apimanagementarmtemplatecreator.

There is no public bugtracker for that. ETA is a couple of weeks since it's not a regression.

from apimanagementarmtemplatecreator.