Ability to control juju backup tmp directory

[Thomas Cuthbert]

GH PR https://github.com/juju/juju/pull/8339

In instances where the disk /tmp is stored on is too full to store the tmp files required for juju create-backup it would be beneficial to control the /tmp location so we can set it to a volume with greater storage capacity.

[Paul Gear (paulgear)]

Given that this can prevent safe upgrades due to lack of backups, the papercut tag seems inappropriate.

[Tim Penhey (thumper)]

/tmp comes from:

state/backups/create.go:112

func newBuilder(filesToBackUp []string, db DBDumper) (b *builder, err error) {
 // Create the backups workspace root directory.
 rootDir, err := ioutil.TempDir("", tempPrefix)
 if err != nil {
  return nil, errors.Annotate(err, "while making backups workspace")
 }
...

ioutil.TempDir uses os.TempDir if empty string passed in.

To resolve this issue, a configurable temp dir needs to be passed in to this method from the outside.

@Paul, there are two options:
 * model config value
 * command line argument

Do you have a strong preference one way or the other?

[John A Meinel (jameinel)]

(FWIW, they could edit the /etc/.../jujud-machine-?.conf file and add the
Environment line to set temp to what they want.

On Fri, Nov 24, 2017 at 6:36 AM, Tim Penhey <email address hidden>
wrote:

> /tmp comes from:
>
> state/backups/create.go:112
>
>
> func newBuilder(filesToBackUp []string, db DBDumper) (b *builder, err
> error) {
> // Create the backups workspace root directory.
> rootDir, err := ioutil.TempDir("", tempPrefix)
> if err != nil {
> return nil, errors.Annotate(err, "while making backups
> workspace")
> }
> ...
>
>
> ioutil.TempDir uses os.TempDir if empty string passed in.
>
> To resolve this issue, a configurable temp dir needs to be passed in to
> this method from the outside.
>
> @Paul, there are two options:
> * model config value
> * command line argument
>
> Do you have a strong preference one way or the other?
>
> --
> You received this bug notification because you are subscribed to juju.
> Matching subscriptions: juju bugs
> https://bugs.launchpad.net/bugs/1693068
>
> Title:
> Ability to control juju backup tmp directory
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/juju/+bug/1693068/+subscriptions
>

[Paul Gear (paulgear)]

There are pros & cons to both model config and CLI. How hard would it be to implement both?

[Adrian Garcia Prado (agprado)]

GitHub PR https://github.com/juju/juju/pull/8339