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

chore(updatecli) correct Azure VM agent controller credentials manifests #839

Conversation

dduportal
Copy link
Contributor

@dduportal dduportal commented Sep 19, 2024

Fixes the following problems:

@dduportal dduportal force-pushed the chore/updatecli/fix-infra.ci-azure-vm-manifest branch from b1c1a09 to 848076b Compare September 19, 2024 15:53
@dduportal dduportal changed the title Chore/updatecli/fix infra.ci azure vm manifest chore(updatecli) correct Azure VM agent controller credentials manifests Sep 19, 2024
@dduportal dduportal marked this pull request as ready for review September 19, 2024 15:53
@dduportal dduportal merged commit 7b81693 into jenkins-infra:main Sep 19, 2024
3 checks passed
@dduportal dduportal deleted the chore/updatecli/fix-infra.ci-azure-vm-manifest branch September 19, 2024 16:09
dduportal added a commit that referenced this pull request Sep 20, 2024
…ined (#842)

Fixup of #838 and #839

We need to keep some outputs. This PR either move them or add them back
but in the `outputs.tf` file which is the Terraform convention.

Note: adding a comment explaining which script is using them.

Signed-off-by: Damien Duportal <[email protected]>
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