Skip to content

Add DX cli support to multi-tenant environments - #2002

Merged
christian-calabrese merged 9 commits into
mainfrom
christian-calabrese-ces-1753-multi-tenant
Aug 4, 2026
Merged

Add DX cli support to multi-tenant environments#2002
christian-calabrese merged 9 commits into
mainfrom
christian-calabrese-ces-1753-multi-tenant

Conversation

@christian-calabrese

@christian-calabrese christian-calabrese commented Jul 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

  • Allow dx add environment to accept tenant-qualified names such as ced-prod and cgn-dev.
  • Derive Azure short environment codes from the lifecycle suffix while keeping generated folders and GitHub environments tenant-qualified.
  • Ensure repository-specific bootstrapper OIDC federated credentials and GitHub environment secrets are configured even when Azure bootstrap resources already exist.
  • Document the multi-tenant workflow in the website and update the DX CLI companion skill for future agents.

Resolves CES-1753

@christian-calabrese
Support multi-tenant environments
b3ebba5 
Allow add environment to scaffold tenant-qualified environments while keeping Azure naming derived from the lifecycle suffix. Document the new workflow for users and agents.

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

christian-calabrese commented Jul 20, 2026

Copy link
Copy Markdown
Contributor Author
image

@christian-calabrese
Ensure bootstrapper GitHub access
c1fb2cf 
Configure repository-specific OIDC credentials and GitHub environment secrets even when the Azure bootstrap resources already exist.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@christian-calabrese
Update environment action order test
f03403d 
Include repository-specific GitHub environment configuration in the expected deployment environment generator action sequence.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@christian-calabrese
christian-calabrese requested a review from a team as a code owner July 22, 2026 09:44
krusty93
krusty93 reviewed Jul 22, 2026
View reviewed changes
Comment thread apps/cli/src/adapters/commander/commands/add.ts
@dx-release-internal

dx-release-internal Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

⚠️ Missing Nx Version Plan

Nx Release reports that this PR has touched projects without version plan coverage.

Run pnpm nx release plan and add the generated version plans.

Nx Plan Check result: 
NX   Touched projects based on changed files

  - docs
  - @pagopa/dx-cli

NOTE: You can adjust your "versionPlans.ignorePatternsForPlanCheck" config to stop certain files from resulting in projects being classed as touched for the purposes of this command.


 NX   Project "@pagopa/dx-cli" has pending bumps in version plan(s)

  - "minor" in version-plan-1784551273866.md


 NX   Project "renew-tls-certificate" has pending bumps in version plan(s)

  - "patch" in version-plan-1785250247375.md


 NX   Project "@pagopa/docker-build-push" has pending bumps in version plan(s)

  - "patch" in version-plan-1785250247375.md



 NX   Touched projects missing version plans

The following touched projects do not feature in any version plan files:
  - docs

Please use `nx release plan` to generate missing version plans, or adjust your "versionPlans.ignorePatternsForPlanCheck" config stop certain files from affecting the projects for the purposes of this command.

Run with --verbose to see the full list of changed files used for the touched projects logic.

This comment is managed automatically by Nx Release. Version plan guide

@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://proud-mud-0405c6703-2002.westeurope.1.azurestaticapps.net

@christian-calabrese
christian-calabrese enabled auto-merge (squash) July 30, 2026 09:08
@christian-calabrese
christian-calabrese merged commit be5c852 into main Aug 4, 2026
17 checks passed
@christian-calabrese
christian-calabrese deleted the christian-calabrese-ces-1753-multi-tenant branch August 4, 2026 07:26
@dx-release-internal dx-release-internal Bot mentioned this pull request Aug 4, 2026
Merged
christian-calabrese pushed a commit that referenced this pull request Aug 4, 2026
@dx-release-internal
Version Packages (#2027)
08e6de3 
This PR was opened by the [Nx
Release](https://github.com/pagopa/dx/tree/main/actions/nx-release)
GitHub Action. When you're ready to do a release, you can merge this and
the packages will be published automatically. If you're not ready to do
a release yet, that's fine, whenever you add more Nx version plans to
main, this PR will be updated.

# Releases
## @pagopa/docker-build-push@0.0.4

### 🩹 Fixes
- Upgrade actions ([#2018](#2018))
### ❤️ Thank You
- Copilot @Copilot
- Marco Comi @kin0992

## @pagopa/dx-cli@0.27.0

### 🚀 Features
- Support tenant-qualified environment names in add environment for
multi-tenant scaffolding.
([#2002](#2002))
### ❤️ Thank You
- Christian Calabrese
- Copilot App @Copilot

## renew-tls-certificate@0.0.5

### 🩹 Fixes
- Upgrade actions ([#2018](#2018))
### ❤️ Thank You
- Copilot @Copilot
- Marco Comi @kin0992

## self-hosted-runner@0.0.4

### 🩹 Fixes
- Add mise to the DX self-hosted runner image and move cloud CLI
provisioning to repository mise configurations
([#2032](#2032))
### ❤️ Thank You
- Christian Calabrese
- Copilot App @Copilot

<!-- nx-release-tags:
[{"path":"actions/docker-build-push","tag":"@pagopa/docker-build-push@0.0.4","version":"0.0.4"},{"path":"apps/cli","tag":"@pagopa/dx-cli@0.27.0","version":"0.27.0"},{"path":"actions/renew-tls-certificate","tag":"renew-tls-certificate@0.0.5","version":"0.0.5"},{"path":"containers/self-hosted-runner","tag":"self-hosted-runner@0.0.4","version":"0.0.4"}]
-->

Co-authored-by: dx-release-internal[bot] <282845343+dx-release-internal[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mamu0 mamu0 mamu0 left review comments
@kin0992 kin0992 kin0992 approved these changes
@krusty93 krusty93 Awaiting requested review from krusty93

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@christian-calabrese @krusty93 @kin0992 @mamu0