Bicep/WebRP Monthly Sync

webrp-review-august.md

Bicep/WebRP issues sync

To discuss customer reported problems encountered with Web RP APIs consumed through Bicep or ARM Templates.

Outstanding issues

• idempotency issue with SSL certs:
  • Azure/bicep#4564
  • waiting on support ticket # to send to WebRP control plane team
  • Support Ticket: 2109270010002207
  • Navy following up on support ticket as of 11/18
• resource status completing prematurely
  • Azure/bicep#4436
  • Navy to create task to follow up as of 11/18
• basicPublishingCredentialsPolicies requires location property, but is not declared in swagger, and does not care what you provide
  • Azure/bicep#4815
  • Navy to create bug as of 11/18: https://msazure.visualstudio.com/Antares/_workitems/edit/12633212
  • Issue was re-using validation for tracked resource on proxy resource
• swagger:
  • As of 11/18, Navy to create work item to "fix swagger" so we can fix as many swagger issues as possible in one stream of work
  • missing properties on web/connections
    • Azure/bicep#5056
    • https://github.com/Azure/bicep/issues/5516
  • missing properties on web/sites
    • Azure/bicep#4393
  • swagger should have enums (extensible?) for SKU
    • Azure/bicep#4345
  • missing swagger for Web/serverfarms/firstPartyApps/settings@2019-08-01
    • https://github.com/Azure/bicep/issues/4346
  • siteExtensions.properties.version is marked as readOnly
    • Azure/bicep#4271
  • site.properties.siteConfig.metadata[] missing from schema. Is it important?
    • https://github.com/Azure/bicep/issues/3314
  • swagger case insensitivity issue
    • Azure/bicep#3368
  • sku, kind should be marked required (and have enums)
    • Azure/bicep#3462
    • sku, kind should be marked required and have enums
    • kind is an open string, portal uses it for a variety of things
    • sku is a known set, but would add tax to adding new SKUs
    • Navy will follow up if we can make these properties required
    • Marcin to share info on extensible enums
  • kind: 'v2' for web/connections is missing from swagger
    • https://github.com/Azure/bicep/issues/3494
    • Azure/bicep#3512
    • Navy following up with LogicApps team
• preflight too agressive with ACR auth config for web/site deploy
  • https://msazure.visualstudio.com/Antares/_workitems/edit/10449580
  • known issue

Needs justification and investigation

• confusing that you need both kind: 'linux' and properties.reserved: true
  • https://twitter.com/MrThePlague/status/1436909856980561920?s=19
  • kind is a free form value
  • might be worth discussing with Jenny

Need more info

• adding a specific app setting changed downtime behavior
  • Azure/bicep#3991
• listCredentials() for ACR auth inside appSettings did not work when called directly
  • had to be done outside of a module and passed as an @secure() param
  • Azure/bicep#3725
  • NEXT STEP: Bicep team to investigate further

🎉 Resolved issues 🎉

• authSettingsv2 swagger issue
  • Azure/azure-rest-api-specs#14824
  • found here: Azure/bicep#2905 (comment)
  • did not make it into most recent types, will be in next release
• double-check that joseph cawthorne's work is done
  • https://msazure.visualstudio.com/Antares/_workitems/edit/9230996
• Missing "networkConfig" discriminator in swagger
  • https://msazure.visualstudio.com/One/_git/48b88806-5a4d-45da-b342-3fa25e682def/pullrequest/4382832
  • need confirmation from Navy that the next API will have this change
  • route changed from site/config/networkConfig --> site/networkConfig/virtualNetwork
• logic apps sku WorkflowStandard not recognized
  • Azure/bicep#784 (comment)
  • need more clarity on scenario, awaiting response from customer
• return type of *.properties in web/sites/config changes depending on discriminator
  • Azure/bicep#4454
  • recommendation is to use list(), GET will return anything useful as it is marked as secret
  • closed as not actionable