Skip to content

Fix backup disk driver configuration for S3 support#18710

Merged
snipe merged 1 commit intogrokability:masterfrom
ArturoSirvent:fix/backup-disk-s3-driver
Apr 17, 2026
Merged

Fix backup disk driver configuration for S3 support#18710
snipe merged 1 commit intogrokability:masterfrom
ArturoSirvent:fix/backup-disk-s3-driver

Conversation

@ArturoSirvent
Copy link
Copy Markdown
Contributor

@ArturoSirvent ArturoSirvent commented Mar 14, 2026

Fixes #14057

Summary

  • Fix the backup disk in config/filesystems.php to use a dedicated BACKUP_FILESYSTEM_DRIVER env var instead of PRIVATE_FILESYSTEM_DISK
  • Add AWS credential fields to the backup disk config so S3 backups work
  • Use BACKUP_FILESYSTEM_ROOT with safe default (storage_path('app')) for backward compatibility
  • Document BACKUP_FILESYSTEM_DRIVER and BACKUP_FILESYSTEM_ROOT in .env.example

Problem

The current backup disk config uses PRIVATE_FILESYSTEM_DISK as its driver value:

'backup' => [
'driver' => env('PRIVATE_FILESYSTEM_DISK', 'local'),
'root' => storage_path('app'),
],

This can lead to "s3_private" as the driver which is not valid (should be "s3")

- Fix the backup disk in config/filesystems.php to use a dedicated BACKUP_FILESYSTEM_DRIVER env var instead of PRIVATE_FILESYSTEM_DISK
- Add AWS credential fields to the backup disk config so S3 backups work
- Use BACKUP_FILESYSTEM_ROOT with safe default (storage_path('app')) for local driver
- Document BACKUP_FILESYSTEM_DRIVER and BACKUP_FILESYSTEM_ROOT in .env.example

Fixes grokability#14057
@ArturoSirvent ArturoSirvent requested a review from snipe as a code owner March 14, 2026 22:27
@snipe
Copy link
Copy Markdown
Member

snipe commented Mar 17, 2026

Hi there - thanks for this! Can you please re-target this PR to point to the develop branch, per our Contributing documentation?

You don't usually need to close and re-open. After you create a pull request, you can modify the base branch so that the changes in the pull request are compared against a different branch. By changing the base branch of your original pull request rather than opening a new one with the correct base branch, you’ll be able to keep valuable work and discussion.

If you change that branch and you see bunch of someone else's commits in the "files changed" tab of this PR, you might need to rebase. (Leaving those changes in can make it kinda confusing to see this PR actually changes.)

To Rebase

Assuming you started with this repo as the origin, and added yourself as a named remote:

git fetch origin; git rebase origin/develop

otherwise git fetch whatever your remote is.

mhuxq

Thanks!

1 similar comment
@snipe
Copy link
Copy Markdown
Member

snipe commented Mar 23, 2026

Hi there - thanks for this! Can you please re-target this PR to point to the develop branch, per our Contributing documentation?

You don't usually need to close and re-open. After you create a pull request, you can modify the base branch so that the changes in the pull request are compared against a different branch. By changing the base branch of your original pull request rather than opening a new one with the correct base branch, you’ll be able to keep valuable work and discussion.

If you change that branch and you see bunch of someone else's commits in the "files changed" tab of this PR, you might need to rebase. (Leaving those changes in can make it kinda confusing to see this PR actually changes.)

To Rebase

Assuming you started with this repo as the origin, and added yourself as a named remote:

git fetch origin; git rebase origin/develop

otherwise git fetch whatever your remote is.

mhuxq

Thanks!

@snipe
Copy link
Copy Markdown
Member

snipe commented Mar 31, 2026

Gentle ping :)

@codacy-production
Copy link
Copy Markdown

codacy-production Bot commented Apr 2, 2026

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity

Metric Results
Complexity 0

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

@snipe snipe merged commit cf44119 into grokability:master Apr 17, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Connection error when trying to connect to disk named backup

2 participants