containers
/
podman
mirror of https://github.com/containers/podman.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #21857 from vrothberg/fix-21849 

machine config: make write atomic
This commit is contained in:
openshift-merge-bot[bot] 2024-02-28 10:40:23 +00:00 committed by GitHub
parent 9d1704396a f8abd7ff75
commit 2d4ef6f161
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
pkg/machine/vmconfigs/machine.go
View File

@ -16,6 +16,7 @@ import (
"github.com/containers/podman/v5/pkg/machine/define" "github.com/containers/podman/v5/pkg/machine/define"
"github.com/containers/podman/v5/pkg/machine/lock" "github.com/containers/podman/v5/pkg/machine/lock"
"github.com/containers/podman/v5/utils" "github.com/containers/podman/v5/utils"
"github.com/containers/storage/pkg/ioutils"
"github.com/sirupsen/logrus" "github.com/sirupsen/logrus"
) )
@ -133,7 +134,7 @@ func (mc *MachineConfig) write() error {
return err return err
} }
logrus.Debugf("writing configuration file %q", mc.configPath.Path) logrus.Debugf("writing configuration file %q", mc.configPath.Path)
return os.WriteFile(mc.configPath.GetPath(), b, define.DefaultFilePerm) return ioutils.AtomicWriteFile(mc.configPath.GetPath(), b, define.DefaultFilePerm)
} }
func (mc *MachineConfig) SetRootful(rootful bool) error { func (mc *MachineConfig) SetRootful(rootful bool) error {