Skip to content
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

Jump to bottom

Add clusterclass to restore order list #4767

Closed
reasonerjt opened this issue Mar 22, 2022 · 0 comments · Fixed by #4866
Closed

Add clusterclass to restore order list #4767

reasonerjt opened this issue Mar 22, 2022 · 0 comments · Fixed by #4866
Assignees
@reasonerjt
Milestone
1.9.0

Comments

@reasonerjt
Copy link
Contributor

This should be updated:

velero/pkg/cmd/server/server.go

Lines 501 to 502 in 56d625e

"clusters.cluster.x-k8s.io",
"clusterresourcesets.addons.cluster.x-k8s.io",

so that it is restored before clusters.

Need to check with CAPI developers for more details
@reasonerjt reasonerjt added this to the 1.9.0 milestone Mar 22, 2022
@reasonerjt reasonerjt self-assigned this Mar 22, 2022
reasonerjt added a commit to reasonerjt/velero that referenced this issue Apr 27, 2022
@reasonerjt
Add ClusterClasses to the restore priority list
9e930e4 
Make sure ClusterClasses are stored before Clusters.
Fixes vmware-tanzu#4767

Signed-off-by: Daniel Jiang <[email protected]>
@reasonerjt reasonerjt mentioned this issue Apr 27, 2022
Merged
3 tasks
reasonerjt added a commit to reasonerjt/velero that referenced this issue Apr 27, 2022
@reasonerjt
Add ClusterClasses to the restore priority list
f89691c 
Make sure ClusterClasses are stored before Clusters.
Fixes vmware-tanzu#4767

Signed-off-by: Daniel Jiang <[email protected]>
gyaozhou pushed a commit to gyaozhou/velero-read that referenced this issue May 14, 2022
@reasonerjt @gyaozhou
Add ClusterClasses to the restore priority list
6502a8f 
Make sure ClusterClasses are stored before Clusters.
Fixes vmware-tanzu#4767

Signed-off-by: Daniel Jiang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@reasonerjt reasonerjt

Labels
None yet
Projects
None yet
Milestone
1.9.0
Development

Successfully merging a pull request may close this issue.

Add ClusterClasses to the restore priority list reasonerjt/velero
1 participant
@reasonerjt