Skip to content

Commit

Permalink
Add the Carvel package related resources to the restore priority list
Browse files Browse the repository at this point in the history
Add the Carvel package related resources to the restore priority list

Signed-off-by: Wenkai Yin(尹文开) <[email protected]>
  • Loading branch information
ywk253100 committed Sep 19, 2024
1 parent 0b74a73 commit 390ac49
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 0 deletions.
1 change: 1 addition & 0 deletions changelogs/unreleased/8228-ywk253100
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Add the Carvel package related resources to the restore priority list
4 changes: 4 additions & 0 deletions pkg/cmd/server/config/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,8 @@ var (
- CAPI Clusters come before ClusterResourceSets because failing to do so means the CAPI controller-manager will panic.
Both Clusters and ClusterResourceSets need to come before ClusterResourceSetBinding in order to properly restore workload clusters.
See https://github.com/kubernetes-sigs/cluster-api/issues/4105
- apps.kappctrl.k14s.io and packageinstalls.packaging.carvel.dev go after workloads(pod/replicaset/etc.), otherwise the controller may
creates new workloads before restoring them
*/
defaultRestorePriorities = types.Priorities{
HighPriorities: []string{
Expand Down Expand Up @@ -135,6 +137,8 @@ var (
"clusterbootstraps.run.tanzu.vmware.com",
"clusters.cluster.x-k8s.io",
"clusterresourcesets.addons.cluster.x-k8s.io",
"apps.kappctrl.k14s.io",
"packageinstalls.packaging.carvel.dev",
},
}
)
Expand Down

0 comments on commit 390ac49

Please sign in to comment.