Alan-Jowett / sonde / 26073465191
82%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 129
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 82.971%. Remained the same
26073465191

push

github

web-flow 
fix(azure): move Entra app+SP creation from Bicep to CLI (#958)

* fix(azure): retry Bicep deployment on Entra replication delay

On clean deploys the Microsoft Graph Bicep extension can fail because
Entra ID has not yet replicated the newly created app registration to
the read replica that handles the service principal creation request.
This is a known upstream issue (microsoftgraph/msgraph-bicep-types#193).

The first attempt creates the app (which persists even on overall
deployment failure); after a 60-second wait for replication, the retry
finds the app fully propagated and succeeds.

Applied to both Azure Live CI workflow and bootstrap.sh.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix(azure): move Entra app+SP creation from Bicep to CLI

The Microsoft Graph Bicep extension never returns server-generated
read-only properties (like \ppId\) in the resource response body,
even on re-deploys. This is a known upstream issue
(microsoftgraph/msgraph-bicep-types#193) with no fix available.

Move the Entra app registration and service principal creation to CLI
commands (\z ad app create\, \z ad sp create\, \z rest PATCH\)
that run before the Bicep deployment. The resolved \companionClientId\
and \companionServicePrincipalObjectId\ are passed as parameters to
the Bicep template.

Changes:
- main.bicep: replace \companionCertificateBase64\ with
  \companionClientId\ and \companionServicePrincipalObjectId\ params;
  remove \companionIdentity\ module call
- Delete modules/companion-identity.bicep
- azure-live-ci.yml: create app+SP via CLI before Bicep deployment;
  add Entra app cleanup in preflight and teardown steps
- bootstrap.sh: create app+SP via CLI before Bicep deployment;
  configure cert and SPA redirect URIs via Graph API PATCH
- Update bootstrap test stubs for new CLI call patterns
- Remove microsoftGraphV1 extension from bicepconfig.json

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix... (continued)

37696 of 45433 relevant lines covered (82.97%)

249.14 hits per line

Jobs
ID Job ID Ran Files Coverage
1 26073465191.1 129
82.97
 GitHub Action Run
Source Files on build 26073465191