Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update CI/CD to run for CERTEU #126

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

barrjam
Copy link
Contributor

@barrjam barrjam commented Nov 23, 2022

I added an extra step to the deployment.

To be discussed, is this the best way?

@augustincolle-digit augustincolle-digit marked this pull request as ready for review January 16, 2023 09:32
@augustincolle-digit
Copy link
Contributor

This won't work because the CERT-EU subscription is not in the same tenant as the DIGIT subscription.

  • create a service principal with 'az ad sp create-for-rbac'
  • create a github secret with the output of previous
  • create a copy of "deploy" script and replace AZURE_CREDENTIALS with the new secret

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.

2 participants