Skip to content

Commit 1fa2876

Browse files
authored
Merge pull request #576 from dprince/to_core_volume_fix
Simplify and fix name issue with ToCoreVolume
2 parents a8404d0 + fd3b69f commit 1fa2876

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)