mirror of
https://github.com/edgelesssys/constellation.git
synced 2025-01-23 05:41:19 -05:00
aws: fix get version error (#2127)
* init * only add awsLB to versions if installed
This commit is contained in:
parent
92abb890ef
commit
39cea48741
@ -206,17 +206,20 @@ func (c *Client) Versions() (ServiceVersions, error) {
|
|||||||
return ServiceVersions{}, fmt.Errorf("getting %s version: %w", constellationServicesInfo.releaseName, err)
|
return ServiceVersions{}, fmt.Errorf("getting %s version: %w", constellationServicesInfo.releaseName, err)
|
||||||
}
|
}
|
||||||
awsLBVersion, err := c.currentVersion(awsLBControllerInfo.releaseName)
|
awsLBVersion, err := c.currentVersion(awsLBControllerInfo.releaseName)
|
||||||
if !errors.Is(err, errReleaseNotFound) {
|
if err != nil && !errors.Is(err, errReleaseNotFound) {
|
||||||
return ServiceVersions{}, fmt.Errorf("getting %s version: %w", awsLBControllerInfo.releaseName, err)
|
return ServiceVersions{}, fmt.Errorf("getting %s version: %w", awsLBControllerInfo.releaseName, err)
|
||||||
}
|
}
|
||||||
|
|
||||||
return ServiceVersions{
|
res := ServiceVersions{
|
||||||
cilium: compatibility.EnsurePrefixV(ciliumVersion),
|
cilium: compatibility.EnsurePrefixV(ciliumVersion),
|
||||||
certManager: compatibility.EnsurePrefixV(certManagerVersion),
|
certManager: compatibility.EnsurePrefixV(certManagerVersion),
|
||||||
constellationOperators: compatibility.EnsurePrefixV(operatorsVersion),
|
constellationOperators: compatibility.EnsurePrefixV(operatorsVersion),
|
||||||
constellationServices: compatibility.EnsurePrefixV(servicesVersion),
|
constellationServices: compatibility.EnsurePrefixV(servicesVersion),
|
||||||
awsLBController: compatibility.EnsurePrefixV(awsLBVersion),
|
}
|
||||||
}, nil
|
if awsLBVersion != "" {
|
||||||
|
res.awsLBController = compatibility.EnsurePrefixV(awsLBVersion)
|
||||||
|
}
|
||||||
|
return res, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
// currentVersion returns the version of the currently installed helm release.
|
// currentVersion returns the version of the currently installed helm release.
|
||||||
|
Loading…
Reference in New Issue
Block a user