github/containerd
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 2 Wiki Activity

config: migrate version before merging

Browse Source 
Prior to this commit, `/etc/containerd/config.toml` with no version
was parsed as version 3.

Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
This commit is contained in:
Akihiro Suda 2024-10-29 06:09:50 +09:00
parent b70cce2085
commit 4b2bca00b8
No known key found for this signature in database
GPG Key ID: 49524C6F9F638F1A
1 changed files with 15 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
cmd/containerd/server/config/config.go
View File

@ -133,7 +133,12 @@ func (c *Config) ValidateVersion() error {
// MigrateConfig will convert the config to the latest version before using
func (c *Config) MigrateConfig(ctx context.Context) error {
for c.Version < version.ConfigVersion {
return c.MigrateConfigTo(ctx, version.ConfigVersion)
}
// MigrateConfigTo will convert the config to the target version before using
func (c *Config) MigrateConfigTo(ctx context.Context, targetVersion int) error {
for c.Version < targetVersion {
if m := migrations[c.Version]; m != nil {
if err := m(ctx, c); err != nil {
return err
@ -297,6 +302,15 @@ func LoadConfig(ctx context.Context, path string, out *Config) error {
return err
}
switch config.Version {
case 0, 1:
if err := config.MigrateConfigTo(ctx, out.Version); err != nil {
return err
}
default:
// NOP
}
if err := mergeConfig(out, config); err != nil {
return err
}