Expand source error (#293)

This commit is contained in:
Itxaka 2024-04-15 13:24:10 +00:00 committed by GitHub
parent 28e5622d3c
commit d24489e074
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 7 additions and 4 deletions

View File

@ -120,6 +120,9 @@ const (
PassiveBootSuffix = " (fallback)"
RecoveryBootSuffix = " recovery"
StateResetBootSuffix = " state reset (auto)"
// Error
UpgradeNoSourceError = "Could not find a proper source for the upgrade.\nThis can be configured in the cloud config files under the 'upgrade.system.uri' key or via cmdline using the '--source' flag."
)
func UkiDefaultMenuEntries() []string {

View File

@ -187,14 +187,14 @@ type UpgradeSpec struct {
func (u *UpgradeSpec) Sanitize() error {
if u.RecoveryUpgrade {
if u.Recovery.Source.IsEmpty() {
return fmt.Errorf("undefined upgrade source")
return fmt.Errorf(constants.UpgradeNoSourceError)
}
if u.Partitions.Recovery == nil || u.Partitions.Recovery.MountPoint == "" {
return fmt.Errorf("undefined recovery partition")
}
} else {
if u.Active.Source.IsEmpty() {
return fmt.Errorf("undefined upgrade source")
return fmt.Errorf(constants.UpgradeNoSourceError)
}
if u.Partitions.State == nil || u.Partitions.State.MountPoint == "" {
return fmt.Errorf("undefined state partition")

View File

@ -425,7 +425,7 @@ var _ = Describe("Types", Label("types", "config"), func() {
It("fails with empty source", func() {
err := spec.Sanitize()
Expect(err).To(HaveOccurred())
Expect(err.Error()).To(ContainSubstring("undefined upgrade source"))
Expect(err.Error()).To(ContainSubstring(constants.UpgradeNoSourceError))
})
It("fails with missing state partition", func() {
spec.Active.Source = v1.NewFileSrc("/tmp")
@ -449,7 +449,7 @@ var _ = Describe("Types", Label("types", "config"), func() {
It("fails with empty source", func() {
err := spec.Sanitize()
Expect(err).To(HaveOccurred())
Expect(err.Error()).To(ContainSubstring("undefined upgrade source"))
Expect(err.Error()).To(ContainSubstring(constants.UpgradeNoSourceError))
})
It("fails with missing recovery partition", func() {
spec.Recovery.Source = v1.NewFileSrc("/tmp")