Skip to content

Refactor: run Azure Pipeline on all repo changes#3769

Merged
lalver1 merged 1 commit into
mainfrom
refactor/azure-pipelines
May 13, 2026
Merged

Refactor: run Azure Pipeline on all repo changes#3769
lalver1 merged 1 commit into
mainfrom
refactor/azure-pipelines

Conversation

@lalver1
Copy link
Copy Markdown
Member

@lalver1 lalver1 commented May 13, 2026

Closes #3767

This PR removes the file path filter on the Azure Pipeline so that it runs on all repository changes. This is to ensure that the side-by-side deployments of App Services and Container Apps have the same application code.

@lalver1 lalver1 self-assigned this May 13, 2026
@github-actions github-actions Bot added the infrastructure Terraform, Azure, etc. label May 13, 2026
@github-actions
Copy link
Copy Markdown

Coverage report

This PR does not seem to contain any modification to coverable code.

@lalver1 lalver1 force-pushed the refactor/azure-pipelines branch from e57fa26 to 9a6f008 Compare May 13, 2026 18:58
@lalver1 lalver1 marked this pull request as ready for review May 13, 2026 19:08
@lalver1 lalver1 requested a review from a team as a code owner May 13, 2026 19:08
Copy link
Copy Markdown
Member

@jgravois jgravois left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

makes sense to me!

since this PR touches terraform/ we can make a point of checking the deployed SHA for the dev container app after the next trivial PR is merged.

bon voyage! ⛵

@lalver1
Copy link
Copy Markdown
Member Author

lalver1 commented May 13, 2026

Just one thing I noticed, terraform plan showed one change for the App Service but it's fine, it's just a setting that when it is changed, it refreshes secrets. I assume we used it some time last week but anyway it's not any data that we track in Terraform.

@jgravois
Copy link
Copy Markdown
Member

it's just a setting that when it is changed, it refreshes secrets

ah, this could have been me adding the CHANGE_ME secret before i was informed that the instructions here are stale.

@lalver1 lalver1 merged commit 0b8bd1e into main May 13, 2026
13 checks passed
@lalver1 lalver1 deleted the refactor/azure-pipelines branch May 13, 2026 19:34
@Scotchester
Copy link
Copy Markdown
Member

Confirming that the dev Container App's SHA.txt is showing the latest commit on main now 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infrastructure Terraform, Azure, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Turn off file path filter for existing Terraform pipeline

4 participants