rcreate action broken with Borg 1.4 #820

New Issue

Closed

opened 2024-01-24 23:19:54 +00:00 by witten · 2 comments

witten commented 2024-01-24 23:19:54 +00:00

Owner

Copy Link

What I'm trying to do and why

#798 started calling Borg with the BORG_EXIT_CODES environment variable set to modern, which causes Borg 1.4 (only) to return different exit codes. It appears that the borgmatic rcreate action, which calls borg info to determine if the repository already exists, doesn't handle the new exit code (13) properly when the repository doesn't exit. The result is that the borg info call causes borgmatic to error instead of continuing on to run borg rcreate as expected.

This issue does not occur with other versions of Borg besides 1.4.

Steps to reproduce

Using Borg 1.4.0b1, run:

# borgmatic -v 2 -c test.yaml rcreate -e repokey

Actual behavior

/root/tmp/test.yaml: Skipping check/prune/compact actions due to configured skip_actions
BORG_EXIT_CODES=*** /root/borg1.4.0b1 --version --debug --show-rc
/root/tmp/test.yaml: Borg 1.4.0b1
backup: Running actions for repository
/root/tmp/test.yaml: No commands to run for pre-actions hook
backup: Creating repository
BORG_EXIT_CODES=*** /root/borg1.4.0b1 info --json 1.2.borg
Repository /root/tmp/1.2.borg does not exist.
backup: Error running actions for repository
Command '('/root/borg1.4.0b1', 'info', '--json', '1.2.borg')' returned non-zero exit status 13.
/root/tmp/test.yaml: An error occurred
...

Expected behavior

The Repository /root/tmp/1.2.borg does not exist. message is expected, but then instead of an error, I'd expect borgmatic to continue on to running borg rcreate.

Other notes / implementation ideas

No response

borgmatic version

1.8.7 or main

borgmatic installation method

pipx install --editable

Borg version

1.4.0b1

Python version

3.11.6

Database version (if applicable)

No response

Operating system and version

Manjaro stable

### What I'm trying to do and why #798 started calling Borg with the `BORG_EXIT_CODES` environment variable set to `modern`, which causes Borg 1.4 (only) to return different exit codes. It appears that the borgmatic `rcreate` action, which calls `borg info` to determine if the repository already exists, doesn't handle the new exit code (13) properly when the repository doesn't exit. The result is that the `borg info` call causes borgmatic to error instead of continuing on to run `borg rcreate` as expected. This issue does not occur with other versions of Borg besides 1.4. ### Steps to reproduce Using Borg 1.4.0b1, run: ``` # borgmatic -v 2 -c test.yaml rcreate -e repokey ``` ### Actual behavior ``` /root/tmp/test.yaml: Skipping check/prune/compact actions due to configured skip_actions BORG_EXIT_CODES=*** /root/borg1.4.0b1 --version --debug --show-rc /root/tmp/test.yaml: Borg 1.4.0b1 backup: Running actions for repository /root/tmp/test.yaml: No commands to run for pre-actions hook backup: Creating repository BORG_EXIT_CODES=*** /root/borg1.4.0b1 info --json 1.2.borg Repository /root/tmp/1.2.borg does not exist. backup: Error running actions for repository Command '('/root/borg1.4.0b1', 'info', '--json', '1.2.borg')' returned non-zero exit status 13. /root/tmp/test.yaml: An error occurred ... ``` ### Expected behavior The `Repository /root/tmp/1.2.borg does not exist.` message is expected, but then instead of an error, I'd expect borgmatic to continue on to running `borg rcreate`. ### Other notes / implementation ideas _No response_ ### borgmatic version 1.8.7 or main ### borgmatic installation method pipx install --editable ### Borg version 1.4.0b1 ### Python version 3.11.6 ### Database version (if applicable) _No response_ ### Operating system and version Manjaro stable

witten added the

bug

label 2024-01-24 23:44:34 +00:00

witten referenced this issue from a commit 2024-01-24 23:46:17 +00:00

Fix broken repository detection in the "rcreate" action with Borg 1.4 (#820).

witten commented 2024-01-24 23:46:32 +00:00

Author

Owner

Copy Link

Fixed in main. Will be part of the next release.

Fixed in main. Will be part of the next release.

witten closed this issue 2024-01-24 23:46:32 +00:00

witten commented 2024-01-31 18:56:26 +00:00

Author

Owner

Copy Link

Released in borgmatic 1.8.8!

Released in borgmatic 1.8.8!

Sign in to join this conversation.

No Branch/Tag Specified

Branches Tags

main

spot-check

1.8.10

1.8.9

1.8.8

1.8.7

1.8.6

1.8.5

1.8.4

1.8.3

1.8.2

1.8.1

1.8.0

1.7.15

1.7.14

1.7.13

1.7.12

1.7.11

1.7.10

1.7.9

1.7.8

1.7.7

1.7.6

1.7.5

1.7.4

1.7.3

1.7.2

1.7.1

1.7.0

1.6.6

1.6.5

1.6.4

1.6.3

1.6.2

1.6.1

1.6.0

1.5.24

1.5.23

1.5.22

1.5.21

1.5.20

1.5.19

1.5.18

1.5.17

1.5.16

1.5.15

1.5.14

1.5.13

1.5.12

1.5.11

1.5.10

1.5.9

1.5.8

1.5.7

1.5.7.dev0

1.5.6

1.5.5

1.5.4

1.5.3

1.5.2

1.5.1

1.5.0

1.4.22

1.4.21

1.4.20

1.4.19

1.4.18

1.4.17

1.4.16

1.4.15

1.4.14

1.4.13

1.4.12

1.4.11

1.4.10

1.4.9

1.4.8

1.4.7

1.4.6

1.4.5

1.4.4

1.4.3

1.4.2

1.4.1

1.4.0

1.3.26

1.3.25

1.3.24

1.3.23

1.3.22

1.3.21

1.3.20

1.3.19

1.3.18

1.3.17

1.3.16

1.3.15

1.3.14

1.3.13

1.3.12

1.3.11

1.3.10

1.3.9

1.3.8

1.3.7

1.3.6

1.3.5

1.3.4

1.3.3

1.3.2

1.3.1

1.3.0

1.2.18

1.2.17

1.2.16

1.2.15

1.2.14

1.2.13

1.2.12

1.2.11

1.2.10

1.2.9

1.2.8

1.2.7

1.2.6

1.2.5

1.2.4

1.2.3

1.2.2

1.2.1

1.2.0

1.1.15

1.1.14

1.1.13

1.1.12

1.1.11

1.1.10

1.1.9

1.1.7

1.1.6

1.1.5

1.1.4

1.1.3

1.1.2

1.1.1

1.1.0

1.0.3

1.0.2

1.0.1

1.0.0

0.1.8

0.1.7

0.1.6

0.1.5

0.1.4

0.1.3

0.1.2

0.1.1

0.1.0

0.0.7

0.0.6

0.0.5

0.0.4

0.0.3

0.0.2

Labels

Clear labels   

bug

  

data loss

  

design finalized

  

good first issue

  

new feature area

  

question / support

  

security

  

waiting for response

No Label

bug

data loss

design finalized

good first issue

new feature area

question / support

security

waiting for response

Milestone

Clear milestone

No items

No Milestone

Assignees

Clear assignees

No Assignees

1 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: borgmatic-collective/borgmatic#820

No description provided.