Added s3 dependency
Needs ReviewPublic
Actions

Authored by mollekopf on Sat, Mar 11, 12:50 PM.

Details

Reviewers

None

Group Reviewers

Restricted Project

Commits

rKa4719d17b2b2: Added s3 dependency
rK94be59e08adf: Added s3 dependency
rKe11b25330d82: Added s3 dependency
rK36cac71480bc: Added s3 dependency
rK6deb54f6dc77: Added s3 dependency
rKbaef77033f05: Added s3 dependency

Diff Detail

Repository

rK kolab

Lint

Lint OK

Unit

No Unit Test Coverage

Build Status

Buildable 41987
Build 16865: arc lint + arc unit

Event Timeline

mollekopf requested review of this revision.Sat, Mar 11, 12:50 PM

mollekopf created this revision.

Harbormaster completed remote builds in B41987: Diff 11891.Sat, Mar 11, 12:50 PM

The tests require some adjustment, that's currently in a separate commit (I can fix that before pushing).

Makes sense, but please note that it will require some configuration when we deploy this change (as the default is 'local').

In D4151#49970, @machniak wrote:

Makes sense, but please note that it will require some configuration when we deploy this change (as the default is 'local').

That's what FILESYSTEM_DISK=minio is for in the .env

My point was that we're using this in production, so we need to migrate from local to minio or stay at local until we're ready to switch.

Revision Contents
Changeset List

			Path	Packages
M			bin/quickstart.sh (3 lines)
M			config.demo/src/.env (4 lines)
M			config.prod/src/.env (5 lines)
M			docker-compose.yml (24 lines)
M			src/app/Backends/Storage.php (12 lines)
M			src/composer.json (1 line)
M			src/config/filesystems.php (10 lines)