Juju Lint

Merge ~lihuiguo/juju-lint:bug/1930892 into juju-lint:master

  1. Git
  2. lp:~lihuiguo/juju-lint
  3. bug/1930892
  4. Merge into master
Proposed by Linda Guo
Status: Merged
Approved by: Alvaro Uria
Approved revision: 52c519394d35f72c91a96168db2512a524516f14
Merged at revision: e4c541425d8cb266dcc65f837b94379889b1b72b
Proposed branch: ~lihuiguo/juju-lint:bug/1930892
Merge into: juju-lint:master
Diff against target: 13 lines (+2/-0)
1 file modified
contrib/includes/database/percona-cluster.yaml (+2/-0)
Reviewer Review Type Date Requested Status
Eric Chen Approve
Alvaro Uria (community) Approve
Review via email: mp+419742@code.launchpad.net

Commit message

Percona cluster tuning-level needs to be set to "fast" while the default is safest.

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
Alvaro Uria (aluria) wrote :

+1, ty.

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

Change successfully merged at revision e4c541425d8cb266dcc65f837b94379889b1b72b

Revision history for this message
Eric Chen (eric-chen) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/contrib/includes/database/percona-cluster.yaml b/contrib/includes/database/percona-cluster.yaml
2index 7102f52..604d0ab 100644
3--- a/contrib/includes/database/percona-cluster.yaml
4+++ b/contrib/includes/database/percona-cluster.yaml
5@@ -4,6 +4,8 @@ openstack database config: &openstack-config-database
6 gte: 6G
7 max-connections:
8 gte: 2000
9+ tuning-level:
10+ eq: "fast"
11
12 openstack mandatory deps database: &openstack-mandatory-deps-database
13 - percona-cluster

Subscribers

People subscribed via source and target branches