Skip to content

fix(ci-lease): serialise callers + servicecatalog perm#400

Merged
chrisns merged 1 commit into
mainfrom
chore/serialize-and-svc-catalog
May 22, 2026
Merged

fix(ci-lease): serialise callers + servicecatalog perm#400
chrisns merged 1 commit into
mainfrom
chore/serialize-and-svc-catalog

Conversation

@chrisns
Copy link
Copy Markdown
Member

@chrisns chrisns commented May 22, 2026

ISB caps 1 active lease per user; serialise scenario CI globally so fan-out doesn't 409. Plus add servicecatalog:* (council-chatbot uses AppRegistryApplication).

1. Add servicecatalog:* to CIDeployRole. Council-chatbot uses
   AWS::ServiceCatalogAppRegistry::Application; without this it fails
   with:

     AccessDenied: ... is not authorized to perform:
     servicecatalog:CreateApplication

   Confirmed via the now-captured CFN events on run 26279585440 (which
   reached Deploy but rolled back). The events also show the
   KnowledgeBaseVectorBucket creation succeeded — the s3vectors fix
   from the previous PR worked.

2. Serialise scenario callers via a single repo-wide concurrency group
   (scenario-ci-pool, cancel-in-progress=false). ISB enforces
   maxLeasesPerUser=1 on ci-bot@ndx-try.local; concurrent dispatches
   would all hit 409 except the first. PR fan-out on shared-infra
   edits now queues runs one-at-a-time, taking longer wallclock but
   actually completing each scenario instead of 15 hitting 409.
@chrisns chrisns had a problem deploying to smoke-test-deploy May 22, 2026 09:28 — with GitHub Actions Failure
@chrisns chrisns added this pull request to the merge queue May 22, 2026
Merged via the queue into main with commit 0072fd5 May 22, 2026
5 of 6 checks passed
@chrisns chrisns deleted the chore/serialize-and-svc-catalog branch May 22, 2026 09:29
@chrisns chrisns had a problem deploying to smoke-test-deploy May 22, 2026 09:29 — with GitHub Actions Failure
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant