diff --git a/libcontainer/configs/validate/validator.go b/libcontainer/configs/validate/validator.go index 81eebba797f..11b80ddaae6 100644 --- a/libcontainer/configs/validate/validator.go +++ b/libcontainer/configs/validate/validator.go @@ -38,11 +38,11 @@ func Validate(config *configs.Config) error { } // Relaxed validation rules for backward compatibility warns := []check{ - mounts, // TODO (runc v1.x.x): make this an error instead of a warning + mountsWarn, } for _, c := range warns { if err := c(config); err != nil { - logrus.WithError(err).Warn("invalid configuration") + logrus.WithError(err).Warn("configuration") } } return nil @@ -300,10 +300,10 @@ func checkIDMapMounts(config *configs.Config, m *configs.Mount) error { return nil } -func mounts(config *configs.Config) error { +func mountsWarn(config *configs.Config) error { for _, m := range config.Mounts { if !filepath.IsAbs(m.Destination) { - return fmt.Errorf("invalid mount %+v: mount destination not absolute", m) + return fmt.Errorf("mount %+v: relative destination path is **deprecated**, using it as relative to /", m) } } return nil