Skip to content

Commit

Permalink
Restore Endpoints before Services (#6317)
Browse files Browse the repository at this point in the history
Restore Endpoints before Services

Fixes #6280

Signed-off-by: Wenkai Yin(尹文开) <[email protected]>
  • Loading branch information
ywk253100 authored Jun 20, 2023
1 parent 05dcbcd commit 788013b
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 1 deletion.
1 change: 1 addition & 0 deletions changelogs/unreleased/6317-ywk253100
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Restore Endpoints before Services
2 changes: 2 additions & 0 deletions pkg/cmd/server/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -508,6 +508,7 @@ func (s *server) veleroResourcesExist() error {
// - Replica sets go before deployments/other controllers so they can be explicitly
// restored and be adopted by controllers.
// - CAPI ClusterClasses go before Clusters.
// - Endpoints go before Services so no new Endpoints will be created
// - Services go before Clusters so they can be adopted by AKO-operator and no new Services will be created
// for the same clusters
//
Expand Down Expand Up @@ -538,6 +539,7 @@ var defaultRestorePriorities = restore.Priorities{
// in the backup.
"replicasets.apps",
"clusterclasses.cluster.x-k8s.io",
"endpoints",
"services",
},
LowPriorities: []string{
Expand Down
2 changes: 1 addition & 1 deletion pkg/uploader/provider/provider.go
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ func NewUploaderProvider(
log logrus.FieldLogger,
) (Provider, error) {
if credGetter.FromFile == nil {
return nil, errors.New("uninitialized FileStore credentail is not supported")
return nil, errors.New("uninitialized FileStore credential is not supported")
}
if uploaderType == uploader.KopiaType {
// We use the hardcode repositoryType velerov1api.BackupRepositoryTypeKopia for now, because we have only one implementation of unified repo.
Expand Down

0 comments on commit 788013b

Please sign in to comment.