~kotodama/charm-k8s-discourse:s3_storage

Branch merges

Propose for merging

Merged into charm-k8s-discourse:master at revision c4e0953f8b850856079dde66612718b4611b81b3

Tom Haddon: Approve on 2022-02-10

🤖 prod-jenkaas-is (community): Approve (continuous-integration) on 2022-02-10

Canonical IS Reviewers: Pending requested 2022-02-10

Diff: 13 lines (+1/-1)

1 file modified

config.yaml (+1/-1)

api

Merged into charm-k8s-discourse:master at revision 0df9bf214e523b084497453bba033bdc71214c4c

Tom Haddon: Approve on 2022-02-10

🤖 prod-jenkaas-is (community): Approve (continuous-integration) on 2022-02-10

Canonical IS Reviewers: Pending requested 2022-02-02

Diff: 512 lines (+413/-0)

7 files modified

config.yaml (+62/-0)
image/scripts/pod_setup (+10/-0)
src/charm.py (+30/-0)
tests/unit/fixtures/config_invalid_missing_required_s3_options.yaml (+28/-0)
tests/unit/fixtures/config_valid_complete.yaml (+26/-0)
tests/unit/fixtures/config_valid_no_s3.yaml (+120/-0)
tests/unit/fixtures/config_valid_s3_only_required_options.yaml (+137/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

14f249a... by Loïc Gomez on 2022-02-10

Set default image to `discoursecharmers/discourse:v2.7.10.1-20.04_edge`

70d5de3... by Loïc Gomez on 2022-02-02

Add S3 support for uploaded objects storage

418f095... by Loïc Gomez on 2022-02-03

Fix unit tests failing due to changes in the Operator Framework.

- cleanup removed charm options
- use new `unset` harness.update_config parameter for config keys which aren't actual charm options (listed in self.not_actual_charm_config)
- populate config with dummy values for settings we'd have obtained through relations when we're not actually testing relations

Reviewed-on: https://code.launchpad.net/~kotodama/charm-k8s-discourse/+git/charm-k8s-discourse/+merge/414969
Reviewed-by: Tom Haddon <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-is <email address hidden>

fcbab74... by Loïc Gomez on 2022-02-02

Fix unit tests since we removed some options / use juju relations

d1369de... by Tom Haddon on 2021-11-30

Update image location to the published location on dockerhub

Reviewed-on: https://code.launchpad.net/~mthaddon/charm-k8s-discourse/+git/charm-k8s-discourse/+merge/412584
Reviewed-by: Loïc Gomez <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-is <email address hidden>

b98ef74... by Tom Haddon on 2021-11-30

Update image location to the published location on dockerhub

71f7b9d... by Barry Price on 2021-11-30

Remove extra precompile line

Reviewed-on: https://code.launchpad.net/~barryprice/charm-k8s-discourse/+git/charm-k8s-discourse/+merge/412564
Reviewed-by: Tom Haddon <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-is <email address hidden>

6c68be8... by Barry Price on 2021-11-30

Remove extra precompile line

54f81a9... by Barry Price on 2021-11-30

We can't tell from bundler's return code whether it's successfully completed our assets:precompile or not, let's just take the dumb approach for now and run it three times

Reviewed-on: https://code.launchpad.net/~barryprice/charm-k8s-discourse/+git/charm-k8s-discourse/+merge/412557
Reviewed-by: Tom Haddon <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-is <email address hidden>

8ae33ea... by Barry Price on 2021-11-30

Clarify exactly which file our first run fails on