Merge lp:~chad.smith/landscape-charm/package-search-threshold-800 into lp:~landscape/landscape-charm/trunk

Proposed by Chad Smith
Status: Merged
Approved by: Chad Smith
Approved revision: 386
Merged at revision: 386
Proposed branch: lp:~chad.smith/landscape-charm/package-search-threshold-800
Merge into: lp:~landscape/landscape-charm/trunk
Diff against target: 25 lines (+2/-2)
2 files modified
lib/tests/test_templates.py (+1/-1)
templates/service.conf (+1/-1)
To merge this branch: bzr merge lp:~chad.smith/landscape-charm/package-search-threshold-800
Reviewer Review Type Date Requested Status
Alberto Donato (community) Approve
Adam Collard (community) Approve
🤖 Landscape Builder test results Approve
Review via email: mp+320988@code.launchpad.net

Commit message

Set package-search account-threshold back to 800 for landscape-charm until lp:1675569 is resolved.

Description of the change

Set package-search account-threshold back to 800 for landscape-charm until lp:1675569 is resolved.
=
This is a complete revert of lp:landscape-charm rev 372. Just backing out the change that activated package-search on all OnPrem installs (by setting account-threshold = 0).

To post a comment you must log in.
Revision history for this message
🤖 Landscape Builder (landscape-builder) :
review: Abstain (executing tests)
Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :
review: Approve (test results)
Revision history for this message
Adam Collard (adam-collard) wrote :

+1, looks good

review: Approve
Revision history for this message
Alberto Donato (ack) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lib/tests/test_templates.py'
2--- lib/tests/test_templates.py 2016-11-03 11:28:38 +0000
3+++ lib/tests/test_templates.py 2017-03-24 21:02:39 +0000
4@@ -96,7 +96,7 @@
5 "/var/run/landscape/landscape-package-search.pid",
6 config.get("package-search", "pid-path"))
7 self.assertEqual(
8- "0", config.get("package-search", "account-threshold"))
9+ "800", config.get("package-search", "account-threshold"))
10
11 def test_render_with_package_search_on_non_leader(self):
12 """
13
14=== modified file 'templates/service.conf'
15--- templates/service.conf 2016-11-03 11:28:38 +0000
16+++ templates/service.conf 2017-03-24 21:02:39 +0000
17@@ -109,7 +109,7 @@
18 port = 9099
19 stores = main package resource-1
20 pid-path = /var/run/landscape/landscape-package-search.pid
21-account-threshold = 0
22+account-threshold = 800
23
24 [package-upload]
25 stores = main account-1

Subscribers

People subscribed via source and target branches