Merge ~barryprice/charm-k8s-wordpress/+git/charm-k8s-wordpress:master into charm-k8s-wordpress:master

Proposed by Barry Price
Status: Merged
Approved by: Thomas Cuthbert
Approved revision: b3ab4e10b78beccce251ebf3c6258f2888f669f6
Merged at revision: 347aed5ac0b9814182877e732abdda0b4edafdd2
Proposed branch: ~barryprice/charm-k8s-wordpress/+git/charm-k8s-wordpress:master
Merge into: charm-k8s-wordpress:master
Diff against target: 45 lines (+21/-1)
3 files modified
config.yaml (+6/-0)
mod/operator (+1/-1)
src/charm.py (+14/-0)
Reviewer Review Type Date Requested Status
Thomas Cuthbert (community) Approve
Canonical IS Reviewers Pending
Review via email: mp+383120@code.launchpad.net

Commit message

To post a comment you must log in.
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
Thomas Cuthbert (tcuthbert) wrote :

LGTM +1

review: Approve
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision 347aed5ac0b9814182877e732abdda0b4edafdd2

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/config.yaml b/config.yaml
2index 8e2081c..7bc9cb2 100644
3--- a/config.yaml
4+++ b/config.yaml
5@@ -90,3 +90,9 @@ options:
6
7 "site-sysadmins=administrator,site-editors=editor,site-executives=editor"
8 default: ""
9+ wp_plugin_openstack-objectstorage_config:
10+ type: string
11+ description: |
12+ YAML dictionary with keys named after wordpress settings and the desired values.
13+ Please note that the settings will be reset to values provided every time hooks run
14+ default: ""
15diff --git a/mod/operator b/mod/operator
16index 11a1849..60c43f8 160000
17--- a/mod/operator
18+++ b/mod/operator
19@@ -1 +1 @@
20-Subproject commit 11a1849205d750e28aaa4a13938b5864659f928b
21+Subproject commit 60c43f81e36139ab4044c185247eb27fe389bce6
22diff --git a/src/charm.py b/src/charm.py
23index 3cfc259..48396f1 100755
24--- a/src/charm.py
25+++ b/src/charm.py
26@@ -42,6 +42,20 @@ def generate_pod_config(config, secured=True):
27 pod_config["WORDPRESS_DB_PASSWORD"] = config["db_password"]
28 if config.get("wp_plugin_akismet_key"):
29 pod_config["WP_PLUGIN_AKISMET_KEY"] = config["wp_plugin_akismet_key"]
30+ if config.get("wp_plugin_openstack-objectstorage_config"):
31+ # actual plugin name is 'openstack-objectstorage', but 'swift' will do us!
32+ wp_plugin_swift_config = config.get("wp_plugin_openstack-objectstorage_config")
33+ pod_config["SWIFT_AUTH_URL"] = wp_plugin_swift_config.get('auth-url')
34+ pod_config["SWIFT_BUCKET"] = wp_plugin_swift_config.get('bucket')
35+ pod_config["SWIFT_PASSWORD"] = wp_plugin_swift_config.get('password')
36+ pod_config["SWIFT_PREFIX"] = wp_plugin_swift_config.get('prefix')
37+ pod_config["SWIFT_REGION"] = wp_plugin_swift_config.get('region')
38+ pod_config["SWIFT_TENANT"] = wp_plugin_swift_config.get('tenant')
39+ pod_config["SWIFT_URL"] = wp_plugin_swift_config.get('url')
40+ pod_config["SWIFT_USERNAME"] = wp_plugin_swift_config.get('username')
41+ pod_config["SWIFT_COPY_TO_SWIFT"] = wp_plugin_swift_config.get('copy-to-swift')
42+ pod_config["SWIFT_SERVE_FROM_SWIFT"] = wp_plugin_swift_config.get('serve-from-swift')
43+ pod_config["SWIFT_REMOVE_LOCAL_FILE"] = wp_plugin_swift_config.get('remove-local-file')
44
45 return pod_config
46

Subscribers

People subscribed via source and target branches