(exit.Error).Wrap => (exit.Error).WrapError

This commit moves from the deprecated (exit.Error).Wrap function to the
(exit.Error).WrapError instead.
This commit is contained in:
Tom Wiesing 2024-04-08 21:32:06 +02:00
parent 7763644ebe
commit d268728f7d
No known key found for this signature in database
20 changed files with 49 additions and 49 deletions

View file

@ -83,10 +83,10 @@ func (bs cBootstrap) Run(context wisski_distillery.Context) error {
{ {
logging.LogMessage(context.Stderr, "Creating root deployment directory") logging.LogMessage(context.Stderr, "Creating root deployment directory")
if err := umaskfree.MkdirAll(root, umaskfree.DefaultDirPerm); err != nil { if err := umaskfree.MkdirAll(root, umaskfree.DefaultDirPerm); err != nil {
return errBootstrapFailedToCreateDirectory.WithMessageF(root).Wrap(err) return errBootstrapFailedToCreateDirectory.WithMessageF(root).WrapError(err)
} }
if err := cli.WriteBaseDirectory(root); err != nil { if err := cli.WriteBaseDirectory(root); err != nil {
return errBootstrapFailedToSaveDirectory.WithMessageF(root).Wrap(err) return errBootstrapFailedToSaveDirectory.WithMessageF(root).WrapError(err)
} }
context.Println(root) context.Println(root)
} }
@ -102,7 +102,7 @@ func (bs cBootstrap) Run(context wisski_distillery.Context) error {
// and use thge defaults // and use thge defaults
if err := tpl.SetDefaults(); err != nil { if err := tpl.SetDefaults(); err != nil {
return errBootstrapWriteConfig.Wrap(err) return errBootstrapWriteConfig.WrapError(err)
} }
{ {
@ -122,7 +122,7 @@ func (bs cBootstrap) Run(context wisski_distillery.Context) error {
{ {
isFile, err := fsx.IsRegular(cfgPath, false) isFile, err := fsx.IsRegular(cfgPath, false)
if err != nil { if err != nil {
return errBootstrapWriteConfig.Wrap(err) return errBootstrapWriteConfig.WrapError(err)
} }
if !isFile { if !isFile {
// generate the configuration from the template // generate the configuration from the template
@ -150,7 +150,7 @@ func (bs cBootstrap) Run(context wisski_distillery.Context) error {
return nil return nil
}, context.Stderr, "Creating custom config files"); err != nil { }, context.Stderr, "Creating custom config files"); err != nil {
return errBootstrapCreateFile.Wrap(err) return errBootstrapCreateFile.WrapError(err)
} }
// Validate configuration file! // Validate configuration file!
@ -176,7 +176,7 @@ func (bs cBootstrap) Run(context wisski_distillery.Context) error {
return err return err
} }
}, context.Stderr, "Installing primary configuration file"); err != nil { }, context.Stderr, "Installing primary configuration file"); err != nil {
return errBootstrapWriteConfig.Wrap(err) return errBootstrapWriteConfig.WrapError(err)
} }
} }
@ -186,13 +186,13 @@ func (bs cBootstrap) Run(context wisski_distillery.Context) error {
logging.LogMessage(context.Stderr, "Configuration is now complete") logging.LogMessage(context.Stderr, "Configuration is now complete")
f, err := os.Open(cfgPath) f, err := os.Open(cfgPath)
if err != nil { if err != nil {
return errBootstrapOpenConfig.Wrap(err) return errBootstrapOpenConfig.WrapError(err)
} }
defer f.Close() defer f.Close()
var cfg config.Config var cfg config.Config
if err := cfg.Unmarshal(f); err != nil { if err := cfg.Unmarshal(f); err != nil {
return errBootstrapOpenConfig.Wrap(err) return errBootstrapOpenConfig.WrapError(err)
} }
context.Println(cfg) context.Println(cfg)

View file

@ -35,7 +35,7 @@ func (cfg cfg) Run(context wisski_distillery.Context) error {
return nil return nil
} }
if err := context.Environment.Config.Marshal(context.Stdout); err != nil { if err := context.Environment.Config.Marshal(context.Stdout); err != nil {
return errMarshalConfig.Wrap(err) return errMarshalConfig.WrapError(err)
} }
return nil return nil
} }

View file

@ -76,12 +76,12 @@ var errNoKey = exit.Error{
func (ds disSSH) parseOpts(context wisski_distillery.Context) (user *auth.AuthUser, key gossh.PublicKey, err error) { func (ds disSSH) parseOpts(context wisski_distillery.Context) (user *auth.AuthUser, key gossh.PublicKey, err error) {
user, err = context.Environment.Auth().User(context.Context, ds.Positionals.User) user, err = context.Environment.Auth().User(context.Context, ds.Positionals.User)
if err != nil { if err != nil {
return nil, nil, errSSHManageFailed.Wrap(err) return nil, nil, errSSHManageFailed.WrapError(err)
} }
content, err := os.ReadFile(ds.Positionals.Path) content, err := os.ReadFile(ds.Positionals.Path)
if err != nil { if err != nil {
return nil, nil, errSSHManageFailed.Wrap(err) return nil, nil, errSSHManageFailed.WrapError(err)
} }
pk, _, _, _, err := gossh.ParseAuthorizedKey(content) pk, _, _, _, err := gossh.ParseAuthorizedKey(content)
@ -99,7 +99,7 @@ func (ds disSSH) runAdd(context wisski_distillery.Context) error {
} }
if err := context.Environment.Keys().Add(context.Context, user.User.User, ds.Comment, key); err != nil { if err := context.Environment.Keys().Add(context.Context, user.User.User, ds.Comment, key); err != nil {
return errSSHManageFailed.Wrap(err) return errSSHManageFailed.WrapError(err)
} }
return nil return nil
} }
@ -111,7 +111,7 @@ func (ds disSSH) runRemove(context wisski_distillery.Context) error {
} }
if err := context.Environment.Keys().Remove(context.Context, user.User.User, key); err != nil { if err := context.Environment.Keys().Remove(context.Context, user.User.User, key); err != nil {
return errSSHManageFailed.Wrap(err) return errSSHManageFailed.WrapError(err)
} }
return nil return nil
} }

View file

@ -47,19 +47,19 @@ var errSettingWissKI = exit.Error{
func (ds setting) Run(context wisski_distillery.Context) error { func (ds setting) Run(context wisski_distillery.Context) error {
instance, err := context.Environment.Instances().WissKI(context.Context, ds.Positionals.Slug) instance, err := context.Environment.Instances().WissKI(context.Context, ds.Positionals.Slug)
if err != nil { if err != nil {
return errSettingWissKI.Wrap(err) return errSettingWissKI.WrapError(err)
} }
if ds.Positionals.Value == "" { if ds.Positionals.Value == "" {
// get the setting // get the setting
value, err := instance.Settings().Get(context.Context, nil, ds.Positionals.Setting) value, err := instance.Settings().Get(context.Context, nil, ds.Positionals.Setting)
if err != nil { if err != nil {
return errSettingGet.Wrap(err) return errSettingGet.WrapError(err)
} }
// and print it // and print it
if err := json.NewEncoder(context.Stdout).Encode(value); err != nil { if err := json.NewEncoder(context.Stdout).Encode(value); err != nil {
return errSettingGet.Wrap(err) return errSettingGet.WrapError(err)
} }
// finish with a newline // finish with a newline
@ -70,12 +70,12 @@ func (ds setting) Run(context wisski_distillery.Context) error {
// serialize the setting into json // serialize the setting into json
var data any var data any
if err := json.Unmarshal([]byte(ds.Positionals.Value), &data); err != nil { if err := json.Unmarshal([]byte(ds.Positionals.Value), &data); err != nil {
return errSettingSet.Wrap(err) return errSettingSet.WrapError(err)
} }
// set the serialized value! // set the serialized value!
if err := instance.Settings().Set(context.Context, nil, ds.Positionals.Setting, data); err != nil { if err := instance.Settings().Set(context.Context, nil, ds.Positionals.Setting, data); err != nil {
return errSettingSet.Wrap(err) return errSettingSet.WrapError(err)
} }
// and we're done // and we're done

View file

@ -53,7 +53,7 @@ var errInstanceLockWissKI = exit.Error{
func (l instanceLock) Run(context wisski_distillery.Context) error { func (l instanceLock) Run(context wisski_distillery.Context) error {
instance, err := context.Environment.Instances().WissKI(context.Context, l.Positionals.Slug) instance, err := context.Environment.Instances().WissKI(context.Context, l.Positionals.Slug)
if err != nil { if err != nil {
return errInstanceLockWissKI.Wrap(err) return errInstanceLockWissKI.WrapError(err)
} }
if l.Unlock { if l.Unlock {

View file

@ -42,7 +42,7 @@ var errInstancePauseWissKI = exit.Error{
func (i instancepause) Run(context wisski_distillery.Context) error { func (i instancepause) Run(context wisski_distillery.Context) error {
instance, err := context.Environment.Instances().WissKI(context.Context, i.Positionals.Slug) instance, err := context.Environment.Instances().WissKI(context.Context, i.Positionals.Slug)
if err != nil { if err != nil {
return errInstancePauseWissKI.Wrap(err) return errInstancePauseWissKI.WrapError(err)
} }
if i.Stop { if i.Stop {

View file

@ -33,7 +33,7 @@ var errLsWissKI = exit.Error{
func (l ls) Run(context wisski_distillery.Context) error { func (l ls) Run(context wisski_distillery.Context) error {
instances, err := context.Environment.Instances().Load(context.Context, l.Positionals.Slug...) instances, err := context.Environment.Instances().Load(context.Context, l.Positionals.Slug...)
if err != nil { if err != nil {
return errLsWissKI.Wrap(err) return errLsWissKI.WrapError(err)
} }
for _, instance := range instances { for _, instance := range instances {

View file

@ -73,7 +73,7 @@ func (mb makeBlock) Run(context wisski_distillery.Context) error {
// get the wisski // get the wisski
instance, err := context.Environment.Instances().WissKI(context.Context, mb.Positionals.Slug) instance, err := context.Environment.Instances().WissKI(context.Context, mb.Positionals.Slug)
if err != nil { if err != nil {
return errPathbuilderWissKI.Wrap(err) return errPathbuilderWissKI.WrapError(err)
} }
// get the footer (if any) // get the footer (if any)
@ -81,7 +81,7 @@ func (mb makeBlock) Run(context wisski_distillery.Context) error {
zerolog.Ctx(context.Context).Info().Msg("checking for footer") zerolog.Ctx(context.Context).Info().Msg("checking for footer")
region, err := instance.Blocks().GetFooterRegion(context.Context, nil) region, err := instance.Blocks().GetFooterRegion(context.Context, nil)
if err != nil { if err != nil {
return errBlocksFooterFailed.Wrap(err) return errBlocksFooterFailed.WrapError(err)
} }
if region == "" { if region == "" {
return errBlocksNoFooter return errBlocksNoFooter
@ -96,7 +96,7 @@ func (mb makeBlock) Run(context wisski_distillery.Context) error {
// read the content // read the content
content, err := io.ReadAll(context.Stdin) content, err := io.ReadAll(context.Stdin)
if err != nil { if err != nil {
return errBlocksNoContent.Wrap(err) return errBlocksNoContent.WrapError(err)
} }
{ {

View file

@ -46,17 +46,17 @@ func (mma makeMysqlAccount) Run(context wisski_distillery.Context) error {
context.Printf("Username>") context.Printf("Username>")
username, err := context.ReadLine() username, err := context.ReadLine()
if err != nil { if err != nil {
return errUnableToReadUsername.Wrap(err) return errUnableToReadUsername.WrapError(err)
} }
context.Printf("Password>") context.Printf("Password>")
password, err := context.ReadPassword() password, err := context.ReadPassword()
if err != nil { if err != nil {
return errUnableToReadPassword.Wrap(err) return errUnableToReadPassword.WrapError(err)
} }
if err := dis.SQL().CreateSuperuser(context.Context, username, password, false); err != nil { if err := dis.SQL().CreateSuperuser(context.Context, username, password, false); err != nil {
return errUnableToMakeAccount.Wrap(err) return errUnableToMakeAccount.WrapError(err)
} }
return nil return nil

View file

@ -46,14 +46,14 @@ func (pb pathbuilders) Run(context wisski_distillery.Context) error {
// get the wisski // get the wisski
instance, err := context.Environment.Instances().WissKI(context.Context, pb.Positionals.Slug) instance, err := context.Environment.Instances().WissKI(context.Context, pb.Positionals.Slug)
if err != nil { if err != nil {
return errPathbuilderWissKI.Wrap(err) return errPathbuilderWissKI.WrapError(err)
} }
// get all of the pathbuilders // get all of the pathbuilders
if pb.Positionals.Name == "" { if pb.Positionals.Name == "" {
names, err := instance.Pathbuilder().All(context.Context, nil) names, err := instance.Pathbuilder().All(context.Context, nil)
if err != nil { if err != nil {
return errPathbuilders.Wrap(err) return errPathbuilders.WrapError(err)
} }
for _, name := range names { for _, name := range names {
context.Println(name) context.Println(name)

View file

@ -38,12 +38,12 @@ var errPrefixesWissKI = exit.Error{
func (p prefixes) Run(context wisski_distillery.Context) error { func (p prefixes) Run(context wisski_distillery.Context) error {
instance, err := context.Environment.Instances().WissKI(context.Context, p.Positionals.Slug) instance, err := context.Environment.Instances().WissKI(context.Context, p.Positionals.Slug)
if err != nil { if err != nil {
return errPrefixesWissKI.Wrap(err) return errPrefixesWissKI.WrapError(err)
} }
prefixes, err := instance.Prefixes().All(context.Context, nil) prefixes, err := instance.Prefixes().All(context.Context, nil)
if err != nil { if err != nil {
return errPrefixesGeneric.Wrap(err) return errPrefixesGeneric.WrapError(err)
} }
for _, p := range prefixes { for _, p := range prefixes {

View file

@ -52,7 +52,7 @@ func (p purge) Run(context wisski_distillery.Context) error {
// do the purge! // do the purge!
if err := dis.Purger().Purge(context.Context, context.Stdout, slug); err != nil { if err := dis.Purger().Purge(context.Context, context.Stdout, slug); err != nil {
return errPurgeFailed.Wrap(err) return errPurgeFailed.WrapError(err)
} }
return nil return nil
} }

View file

@ -64,7 +64,7 @@ func (r reserve) Run(context wisski_distillery.Context) (err error) {
logging.LogMessage(context.Stderr, "Checking that base directory %s does not exist", instance.FilesystemBase) logging.LogMessage(context.Stderr, "Checking that base directory %s does not exist", instance.FilesystemBase)
exists, err := fsx.Exists(instance.FilesystemBase) exists, err := fsx.Exists(instance.FilesystemBase)
if err != nil { if err != nil {
return errProvisionGeneric.Wrap(err) return errProvisionGeneric.WrapError(err)
} }
if exists { if exists {
return errReserveAlreadyExists.WithMessageF(slug) return errReserveAlreadyExists.WithMessageF(slug)

View file

@ -51,7 +51,7 @@ func (s server) Run(context wisski_distillery.Context) error {
// if the caller requested a trigger, just trigger the cron tasks // if the caller requested a trigger, just trigger the cron tasks
if s.Trigger { if s.Trigger {
if err := dis.Control().Trigger(context.Context); err != nil { if err := dis.Control().Trigger(context.Context); err != nil {
return errServerTrigger.Wrap(err) return errServerTrigger.WrapError(err)
} }
} }
@ -70,7 +70,7 @@ func (s server) Run(context wisski_distillery.Context) error {
// and start the server // and start the server
public, internal, err := dis.Control().Server(context.Context, context.Stderr) public, internal, err := dis.Control().Server(context.Context, context.Stderr)
if err != nil { if err != nil {
return errServerGeneric.Wrap(err) return errServerGeneric.WrapError(err)
} }
// start the public listener // start the public listener
@ -95,7 +95,7 @@ func (s server) Run(context wisski_distillery.Context) error {
zerolog.Ctx(context.Context).Info().Str("bind", s.InternalBind).Msg("listening internal server") zerolog.Ctx(context.Context).Info().Str("bind", s.InternalBind).Msg("listening internal server")
internalL, err := net.Listen("tcp", s.InternalBind) internalL, err := net.Listen("tcp", s.InternalBind)
if err != nil { if err != nil {
return errServerListen.Wrap(err) return errServerListen.WrapError(err)
} }
defer internalS.Shutdown(context.Context) defer internalS.Shutdown(context.Context)
go func() { go func() {
@ -111,5 +111,5 @@ func (s server) Run(context wisski_distillery.Context) error {
internalS.Shutdown(context.Context) internalS.Shutdown(context.Context)
}() }()
return errServerListen.Wrap(errors.Join(<-internalC, <-publicC, err)) return errServerListen.WrapError(errors.Join(<-internalC, <-publicC, err))
} }

View file

@ -41,7 +41,7 @@ var errShellWissKI = exit.Error{
func (sh shell) Run(context wisski_distillery.Context) error { func (sh shell) Run(context wisski_distillery.Context) error {
instance, err := context.Environment.Instances().WissKI(context.Context, sh.Positionals.Slug) instance, err := context.Environment.Instances().WissKI(context.Context, sh.Positionals.Slug)
if err != nil { if err != nil {
return errShellWissKI.Wrap(err) return errShellWissKI.WrapError(err)
} }
{ {

View file

@ -57,7 +57,7 @@ func (sn snapshot) Run(context wisski_distillery.Context) error {
// find the instance! // find the instance!
instance, err := dis.Instances().WissKI(context.Context, sn.Positionals.Slug) instance, err := dis.Instances().WissKI(context.Context, sn.Positionals.Slug)
if err != nil { if err != nil {
return errSnapshotWissKI.Wrap(err) return errSnapshotWissKI.WrapError(err)
} }
// do a snapshot of it! // do a snapshot of it!
@ -72,7 +72,7 @@ func (sn snapshot) Run(context wisski_distillery.Context) error {
}) })
if err != nil { if err != nil {
return errSnapshotFailed.Wrap(err) return errSnapshotFailed.WrapError(err)
} }
return nil return nil
} }

View file

@ -40,7 +40,7 @@ func (s ssh) Run(context wisski_distillery.Context) error {
dis := context.Environment dis := context.Environment
server, err := dis.SSH().Server(context.Context, s.PrivateKeyPath, context.Stderr) server, err := dis.SSH().Server(context.Context, s.PrivateKeyPath, context.Stderr)
if err != nil { if err != nil {
return errSSHServer.Wrap(err) return errSSHServer.WrapError(err)
} }
context.Printf("Listening on %s\n", s.Bind) context.Printf("Listening on %s\n", s.Bind)
@ -48,7 +48,7 @@ func (s ssh) Run(context wisski_distillery.Context) error {
// make a new listener // make a new listener
listener, err := net.Listen("tcp", s.Bind) listener, err := net.Listen("tcp", s.Bind)
if err != nil { if err != nil {
return errSSHListen.Wrap(err) return errSSHListen.WrapError(err)
} }
go func() { go func() {
@ -58,5 +58,5 @@ func (s ssh) Run(context wisski_distillery.Context) error {
// and serve that listener // and serve that listener
err = server.Serve(listener) err = server.Serve(listener)
return errServerListen.Wrap(err) return errServerListen.WrapError(err)
} }

View file

@ -33,7 +33,7 @@ var errStatusGeneric = exit.Error{
func (s cStatus) Run(context wisski_distillery.Context) error { func (s cStatus) Run(context wisski_distillery.Context) error {
status, _, err := context.Environment.Info().Status(context.Context, true) status, _, err := context.Environment.Info().Status(context.Context, true)
if err != nil { if err != nil {
return errStatusGeneric.Wrap(err) return errStatusGeneric.WrapError(err)
} }
if s.JSON { if s.JSON {

View file

@ -99,7 +99,7 @@ func (si systemupdate) Run(context wisski_distillery.Context) (err error) {
} { } {
context.Println(d) context.Println(d)
if err := umaskfree.MkdirAll(d, umaskfree.DefaultDirPerm); err != nil { if err := umaskfree.MkdirAll(d, umaskfree.DefaultDirPerm); err != nil {
return errBoostrapFailedToCreateDirectory.WithMessageF(d).Wrap(err) return errBoostrapFailedToCreateDirectory.WithMessageF(d).WrapError(err)
} }
} }
@ -129,7 +129,7 @@ func (si systemupdate) Run(context wisski_distillery.Context) (err error) {
logging.LogMessage(context.Stderr, "Checking that the 'docker' api is reachable") logging.LogMessage(context.Stderr, "Checking that the 'docker' api is reachable")
ping, err := dis.Docker().Ping(context.Context) ping, err := dis.Docker().Ping(context.Context)
if err != nil { if err != nil {
return errDockerUnreachable.Wrap(err) return errDockerUnreachable.WrapError(err)
} }
context.Printf("API Version: %s (experimental: %t)\nBuilder Version: %s\n", ping.APIVersion, ping.Experimental, ping.BuilderVersion) context.Printf("API Version: %s (experimental: %t)\nBuilder Version: %s\n", ping.APIVersion, ping.Experimental, ping.BuilderVersion)
} }
@ -148,7 +148,7 @@ func (si systemupdate) Run(context wisski_distillery.Context) (err error) {
for _, name := range dis.Config.Docker.Networks() { for _, name := range dis.Config.Docker.Networks() {
id, existed, err := dis.Docker().CreateNetwork(context.Context, name) id, existed, err := dis.Docker().CreateNetwork(context.Context, name)
if err != nil { if err != nil {
return errNetworkCreateFailed.Wrap(err) return errNetworkCreateFailed.WrapError(err)
} }
if existed { if existed {
context.Printf("Network %s (id %s) already existed\n", name, id) context.Printf("Network %s (id %s) already existed\n", name, id)
@ -213,7 +213,7 @@ func (si systemupdate) Run(context wisski_distillery.Context) (err error) {
} }
return item.Update(context.Context, context.Stderr) return item.Update(context.Context, context.Stderr)
}, context.Stderr, "Updating Component: %s", name); err != nil { }, context.Stderr, "Updating Component: %s", name); err != nil {
return errBootstrapComponent.WithMessageF(name).Wrap(err) return errBootstrapComponent.WithMessageF(name).WrapError(err)
} }
} }
return nil return nil

View file

@ -26,7 +26,7 @@ func (instances *Instances) Update(ctx context.Context, progress io.Writer) erro
fmt.Fprintf(progress, "[copy] %s\n", dst) fmt.Fprintf(progress, "[copy] %s\n", dst)
}) })
if err != nil { if err != nil {
return errBootstrapFailedRuntime.Wrap(err) return errBootstrapFailedRuntime.WrapError(err)
} }
return nil return nil
} }