Skip to content

Commit b77e0ca

Browse files
committed
v1alpha8: Make all restorers no-ops
v1alpha8 does not yet have any lossy conversions, so doesn't require any restorer functions.
1 parent 7edf6a1 commit b77e0ca

File tree

1 file changed

+7
-67
lines changed

1 file changed

+7
-67
lines changed

api/v1alpha7/conversion.go

Lines changed: 7 additions & 67 deletions
Original file line numberDiff line numberDiff line change
@@ -25,26 +25,9 @@ import (
2525

2626
var _ ctrlconversion.Convertible = &OpenStackCluster{}
2727

28-
var v1alpha7OpenStackClusterRestorer = conversion.RestorerFor[*OpenStackCluster]{
29-
"spec": conversion.HashedFieldRestorer[*OpenStackCluster, OpenStackClusterSpec]{
30-
GetField: func(c *OpenStackCluster) *OpenStackClusterSpec {
31-
return &c.Spec
32-
},
33-
},
34-
"status": conversion.HashedFieldRestorer[*OpenStackCluster, OpenStackClusterStatus]{
35-
GetField: func(c *OpenStackCluster) *OpenStackClusterStatus {
36-
return &c.Status
37-
},
38-
},
39-
}
28+
var v1alpha7OpenStackClusterRestorer = conversion.RestorerFor[*OpenStackCluster]{}
4029

41-
var v1alpha8OpenStackClusterRestorer = conversion.RestorerFor[*infrav1.OpenStackCluster]{
42-
"status": conversion.HashedFieldRestorer[*infrav1.OpenStackCluster, infrav1.OpenStackClusterStatus]{
43-
GetField: func(c *infrav1.OpenStackCluster) *infrav1.OpenStackClusterStatus {
44-
return &c.Status
45-
},
46-
},
47-
}
30+
var v1alpha8OpenStackClusterRestorer = conversion.RestorerFor[*infrav1.OpenStackCluster]{}
4831

4932
func (r *OpenStackCluster) ConvertTo(dstRaw ctrlconversion.Hub) error {
5033
dst := dstRaw.(*infrav1.OpenStackCluster)
@@ -84,13 +67,7 @@ func (r *OpenStackClusterList) ConvertFrom(srcRaw ctrlconversion.Hub) error {
8467

8568
var _ ctrlconversion.Convertible = &OpenStackClusterTemplate{}
8669

87-
var v1alpha7OpenStackClusterTemplateRestorer = conversion.RestorerFor[*OpenStackClusterTemplate]{
88-
"spec": conversion.HashedFieldRestorer[*OpenStackClusterTemplate, OpenStackClusterSpec]{
89-
GetField: func(c *OpenStackClusterTemplate) *OpenStackClusterSpec {
90-
return &c.Spec.Template.Spec
91-
},
92-
},
93-
}
70+
var v1alpha7OpenStackClusterTemplateRestorer = conversion.RestorerFor[*OpenStackClusterTemplate]{}
9471

9572
var v1alpha8OpenStackClusterTemplateRestorer = conversion.RestorerFor[*infrav1.OpenStackClusterTemplate]{}
9673

@@ -118,34 +95,9 @@ func (r *OpenStackClusterTemplate) ConvertFrom(srcRaw ctrlconversion.Hub) error
11895

11996
var _ ctrlconversion.Convertible = &OpenStackMachine{}
12097

121-
var v1alpha7OpenStackMachineRestorer = conversion.RestorerFor[*OpenStackMachine]{
122-
"spec": conversion.HashedFieldRestorer[*OpenStackMachine, OpenStackMachineSpec]{
123-
GetField: func(c *OpenStackMachine) *OpenStackMachineSpec {
124-
return &c.Spec
125-
},
126-
FilterField: func(s *OpenStackMachineSpec) *OpenStackMachineSpec {
127-
// Despite being spec fields, ProviderID and InstanceID
128-
// are both set by the machine controller. If these are
129-
// the only changes to the spec, we still want to
130-
// restore the rest of the spec to its original state.
131-
if s.ProviderID != nil || s.InstanceID != nil {
132-
f := *s
133-
f.ProviderID = nil
134-
f.InstanceID = nil
135-
return &f
136-
}
137-
return s
138-
},
139-
},
140-
}
98+
var v1alpha7OpenStackMachineRestorer = conversion.RestorerFor[*OpenStackMachine]{}
14199

142-
var v1alpha8OpenStackMachineRestorer = conversion.RestorerFor[*infrav1.OpenStackMachine]{
143-
"spec": conversion.HashedFieldRestorer[*infrav1.OpenStackMachine, infrav1.OpenStackMachineSpec]{
144-
GetField: func(c *infrav1.OpenStackMachine) *infrav1.OpenStackMachineSpec {
145-
return &c.Spec
146-
},
147-
},
148-
}
100+
var v1alpha8OpenStackMachineRestorer = conversion.RestorerFor[*infrav1.OpenStackMachine]{}
149101

150102
func (r *OpenStackMachine) ConvertTo(dstRaw ctrlconversion.Hub) error {
151103
dst := dstRaw.(*infrav1.OpenStackMachine)
@@ -183,21 +135,9 @@ func (r *OpenStackMachineList) ConvertFrom(srcRaw ctrlconversion.Hub) error {
183135

184136
var _ ctrlconversion.Convertible = &OpenStackMachineTemplate{}
185137

186-
var v1alpha7OpenStackMachineTemplateRestorer = conversion.RestorerFor[*OpenStackMachineTemplate]{
187-
"spec": conversion.HashedFieldRestorer[*OpenStackMachineTemplate, OpenStackMachineSpec]{
188-
GetField: func(c *OpenStackMachineTemplate) *OpenStackMachineSpec {
189-
return &c.Spec.Template.Spec
190-
},
191-
},
192-
}
138+
var v1alpha7OpenStackMachineTemplateRestorer = conversion.RestorerFor[*OpenStackMachineTemplate]{}
193139

194-
var v1alpha8OpenStackMachineTemplateRestorer = conversion.RestorerFor[*infrav1.OpenStackMachineTemplate]{
195-
"spec": conversion.HashedFieldRestorer[*infrav1.OpenStackMachineTemplate, infrav1.OpenStackMachineSpec]{
196-
GetField: func(c *infrav1.OpenStackMachineTemplate) *infrav1.OpenStackMachineSpec {
197-
return &c.Spec.Template.Spec
198-
},
199-
},
200-
}
140+
var v1alpha8OpenStackMachineTemplateRestorer = conversion.RestorerFor[*infrav1.OpenStackMachineTemplate]{}
201141

202142
func (r *OpenStackMachineTemplate) ConvertTo(dstRaw ctrlconversion.Hub) error {
203143
dst := dstRaw.(*infrav1.OpenStackMachineTemplate)

0 commit comments

Comments
 (0)