Skip to content

Commit fd3b69f

Browse files
committed
Simplify and fix name issue with ToCoreVolume
1 parent a8404d0 commit fd3b69f

File tree

1 file changed

+5
-9
lines changed

1 file changed

+5
-9
lines changed

modules/storage/storage.go

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -167,17 +167,13 @@ func (s *VolumeSource) ToCoreVolumeSource() (*corev1.VolumeSource, error) {
167167

168168
// ToCoreVolume - convert Volume to corev1.Volume
169169
func (s *Volume) ToCoreVolume() (*corev1.Volume, error) {
170-
coreVolume := &corev1.Volume{}
171-
172-
coreVolumeBytes, err := json.Marshal(s)
170+
volSource, err := s.VolumeSource.ToCoreVolumeSource()
173171
if err != nil {
174-
return nil, fmt.Errorf("error marshalling Volume: %w", err)
172+
return nil, err
175173
}
176-
177-
err = json.Unmarshal(coreVolumeBytes, coreVolume)
178-
if err != nil {
179-
return nil, fmt.Errorf("error unmarshalling Volume: %w", err)
174+
coreVolume := &corev1.Volume{
175+
Name: s.Name,
176+
VolumeSource: *volSource,
180177
}
181-
182178
return coreVolume, nil
183179
}

0 commit comments

Comments
 (0)