From 194f03c86a4b7b07a413576ed45c1c9aa69477ea Mon Sep 17 00:00:00 2001 From: Alejandro Mery Date: Sun, 29 Oct 2023 01:04:24 +0000 Subject: [PATCH 1/3] cluster: refactor Machine.ReadFile() and Machine.OpenFile() Signed-off-by: Alejandro Mery --- pkg/cluster/machine_file.go | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/pkg/cluster/machine_file.go b/pkg/cluster/machine_file.go index 40dde3a..69d4c23 100644 --- a/pkg/cluster/machine_file.go +++ b/pkg/cluster/machine_file.go @@ -12,10 +12,9 @@ import ( // OpenFile opens a file on the machine's config directory with the specified flags func (m *Machine) OpenFile(name string, flags int, args ...any) (fs.File, error) { - base := m.zone.zones.dir fullName := m.getFilename(name, args...) - return fs.OpenFile(base, fullName, flags, 0644) + return m.zone.zones.OpenFile(fullName, flags) } // CreateTruncFile creates or truncates a file on the machine's config directory @@ -57,10 +56,9 @@ func (m *Machine) RemoveFile(name string, args ...any) error { // ReadFile reads a file from the machine's config directory func (m *Machine) ReadFile(name string, args ...any) ([]byte, error) { - base := m.zone.zones.dir fullName := m.getFilename(name, args...) - return fs.ReadFile(base, fullName) + return m.zone.zones.ReadFile(fullName) } // WriteStringFile writes the given content to a file on the machine's config directory From bdf0192ac5d507327c5bd810d67fa43cbdc35a68 Mon Sep 17 00:00:00 2001 From: Alejandro Mery Date: Sun, 29 Oct 2023 00:49:56 +0000 Subject: [PATCH 2/3] cluster: add MkdirAll() support Signed-off-by: Alejandro Mery --- pkg/cluster/cluster_file.go | 9 +++++++++ pkg/cluster/machine_file.go | 7 +++++++ 2 files changed, 16 insertions(+) diff --git a/pkg/cluster/cluster_file.go b/pkg/cluster/cluster_file.go index e1deb11..1843274 100644 --- a/pkg/cluster/cluster_file.go +++ b/pkg/cluster/cluster_file.go @@ -48,3 +48,12 @@ func (m *Cluster) ReadFile(name string, args ...any) ([]byte, error) { return fs.ReadFile(m.dir, name) } + +// MkdirAll creates directories relative to the cluster's config directory +func (m *Cluster) MkdirAll(name string, args ...any) error { + if len(args) > 0 { + name = fmt.Sprintf(name, args...) + } + + return fs.MkdirAll(m.dir, name, 0755) +} diff --git a/pkg/cluster/machine_file.go b/pkg/cluster/machine_file.go index 69d4c23..9e037f7 100644 --- a/pkg/cluster/machine_file.go +++ b/pkg/cluster/machine_file.go @@ -74,6 +74,13 @@ func (m *Machine) WriteStringFile(value string, name string, args ...any) error return err } +// MkdirAll creates directories relative to the machine's config directory +func (m *Machine) MkdirAll(name string, args ...any) error { + fullName := m.getFilename(name, args...) + + return m.zone.zones.MkdirAll(fullName) +} + func (m *Machine) getFilename(name string, args ...any) string { if len(args) > 0 { name = fmt.Sprintf(name, args...) From da38250465bedd48bd60287265cb0d6f6747db5b Mon Sep 17 00:00:00 2001 From: Alejandro Mery Date: Sat, 28 Oct 2023 20:42:16 +0000 Subject: [PATCH 3/3] jpictl: create machine directories on `jpictl write` Signed-off-by: Alejandro Mery --- pkg/cluster/sync.go | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/pkg/cluster/sync.go b/pkg/cluster/sync.go index ded0abb..7f0dd8b 100644 --- a/pkg/cluster/sync.go +++ b/pkg/cluster/sync.go @@ -3,6 +3,7 @@ package cluster // SyncAll updates all config files func (m *Cluster) SyncAll() error { for _, fn := range []func() error{ + m.SyncMkdirAll, m.SyncAllWireguard, m.SyncAllCeph, m.WriteHosts, @@ -15,6 +16,20 @@ func (m *Cluster) SyncAll() error { return nil } +// SyncMkdirAll creates the directories needed to store files +// required to represent the cluster. +func (m *Cluster) SyncMkdirAll() error { + err := m.MkdirAll(".") + if err == nil { + m.ForEachMachine(func(p *Machine) bool { + err = p.MkdirAll(".") + return err != nil + }) + } + + return err +} + // SyncAllWireguard updates all wireguard config files func (m *Cluster) SyncAllWireguard() error { var err error