Skip to content

re adding logic for apply on merge #778

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.

Sign up for GitHub

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

Merged
JakeM-Kainos merged 2 commits into from
Mar 16, 2026
Merged

re adding logic for apply on merge #778

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d262f64
re adding logic for apply on merge
JakeM-Kainos Mar 13, 2026
84cce51
Updating read me with learned lesson from upgrades
JakeM-Kainos Mar 16, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,14 @@

Terraform code to deploy Shared Services AKS Cluster and underlying infrastructure.

## Warning

When merging to this repo please check your plans for AKS version changes.

If you see a change similar to 1.34.X -> 1.34 this will bring the cluster into a failed state when applied as it rebuilds the nodes and the PDBs in the cluster will block this.

When applying with a change like this in the plan you will need to delete the PDBs with allowed disruptions set to 0 from the cluster as it applies.

## Following resources are being deployed in each stage of the pipeline

1) Genesis
Expand Down
5 changes: 4 additions & 1 deletion azure-pipelines.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@


trigger: none
trigger:
branches:
include:
- master

pool:
vmImage: 'ubuntu-latest'
Expand Down
Loading