Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rebase and update S3 config #1

Merged
merged 8 commits into from
Aug 26, 2020
Merged

Rebase and update S3 config #1

merged 8 commits into from
Aug 26, 2020

Conversation

cuppett
Copy link

@cuppett cuppett commented Aug 26, 2020

Here's a rebase + all updates suggested in the PR.

tilosp-bot and others added 7 commits August 20, 2020 14:00
Signed-off-by: tilosp-bot <[email protected]>
Signed-off-by: tilosp-bot <[email protected]>
Signed-off-by: tilosp-bot <[email protected]>
- usepath defaults to false (and is deprecated by AWS)
- autocreate is not used/necessary by the objectstorage code (always tries to create)

Signed-off-by: Stephen Cuppett <[email protected]>
Copy link
Owner

@adamjenkins1 adamjenkins1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything looks good, just had some minor comments. Thanks for taking the initiative on this!

.config/objectstore.config.php Outdated Show resolved Hide resolved
.config/objectstore.config.php Outdated Show resolved Hide resolved
@adamjenkins1 adamjenkins1 merged commit 2b370b5 into adamjenkins1:dev-add-s3-env-vars Aug 26, 2020
adamjenkins1 added a commit that referenced this pull request Aug 26, 2020
* Run update.sh

Signed-off-by: tilosp-bot <[email protected]>

* Run update.sh

Signed-off-by: tilosp-bot <[email protected]>

* Run update.sh

Signed-off-by: tilosp-bot <[email protected]>

* Adds ability to configure S3 object stores via environment variables

Signed-off-by: Adam Jenkins <[email protected]>

* Updates objectstore autoconfiguration to match server defaults

- usepath defaults to false (and is deprecated by AWS)
- autocreate is not used/necessary by the objectstorage code (always tries to create)

Signed-off-by: Stephen Cuppett <[email protected]>

* Dropping NULL check

Co-authored-by: tilosp-bot <[email protected]>
Co-authored-by: Tilo Spannagel <[email protected]>
Co-authored-by: Adam Jenkins <[email protected]>
Signed-off-by: Adam Jenkins <[email protected]>
@cuppett cuppett deleted the dev-add-s3-env-vars branch August 26, 2020 17:44
adamjenkins1 added a commit that referenced this pull request Aug 26, 2020
Signed-off-by: Adam Jenkins <[email protected]>

Rebase and update S3 config (#1)

* Run update.sh

Signed-off-by: tilosp-bot <[email protected]>

* Run update.sh

Signed-off-by: tilosp-bot <[email protected]>

* Run update.sh

Signed-off-by: tilosp-bot <[email protected]>

* Adds ability to configure S3 object stores via environment variables

Signed-off-by: Adam Jenkins <[email protected]>

* Updates objectstore autoconfiguration to match server defaults

- usepath defaults to false (and is deprecated by AWS)
- autocreate is not used/necessary by the objectstorage code (always tries to create)

Signed-off-by: Stephen Cuppett <[email protected]>

* Dropping NULL check

Co-authored-by: tilosp-bot <[email protected]>
Co-authored-by: Tilo Spannagel <[email protected]>
Co-authored-by: Adam Jenkins <[email protected]>
Signed-off-by: Adam Jenkins <[email protected]>
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.

4 participants