Skip to content

Commit 56e233f

Browse files
authored
Merge pull request #50 from platform9/pushkar/host-cluster-role-import
fix(resmgr): an imported pcd_host_cluster_role plans clean
2 parents fe547a3 + 4f27525 commit 56e233f

6 files changed

Lines changed: 997 additions & 3 deletions

File tree

CHANGELOG.md

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,18 @@ All notable changes to this project are documented here. The format is based on
66

77
## [Unreleased]
88

9+
### Fixed
10+
11+
- `pcd_host_cluster_role`: importing a role no longer leaves a permanent `wait_until_converged`
12+
diff. Import set only `id`, `host_id` and `role`, so the flag stayed null and the schema
13+
default planned `null -> false` on every imported role; applying that re-PUT the role
14+
assignment to resmgr for a change resmgr never sees. Import now writes the default, and an
15+
update whose only change is `wait_until_converged` stores the value without calling resmgr and
16+
does not wait for convergence — including a `false -> true` flip on an already-assigned role,
17+
which previously PUT the role again and then waited. To block on an already-assigned role,
18+
taint it or run `terraform apply -replace`. `host_cluster` and `backends` are still not read
19+
back, so a configuration that sets them on an imported role plans the update it always did.
20+
921
### Documentation
1022

1123
- **Every import now says where its ID comes from** (PCD-9784). Each resource's Import section

0 commit comments

Comments
 (0)