Merge lp:~jml/launchpad/codebrowse-port-config into lp:launchpad
Proposed by
Jonathan Lange
| Status: | Merged |
|---|---|
| Approved by: | Robert Collins |
| Approved revision: | no longer in the source branch. |
| Merged at revision: | 11748 |
| Proposed branch: | lp:~jml/launchpad/codebrowse-port-config |
| Merge into: | lp:launchpad |
| Diff against target: |
34 lines (+13/-2) 2 files modified
lib/canonical/config/schema-lazr.conf (+10/-0) scripts/start-loggerhead.py (+3/-2) |
| To merge this branch: | bzr merge lp:~jml/launchpad/codebrowse-port-config |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Robert Collins (community) | Approve | ||
|
Review via email:
|
|||
Commit message
Codebrowse host and port are now configured with codebrowse.
Description of the change
We now need to deploy codebrowse to machines that already have another codebrowse deployed. That means we need to use a different port, and to use a different port, we need to be able to configure the port.
This branch moves away from hard-coded literals in the start-loggerhead.py script to actual config variables. I've made the default in the schema match what the literals were before, so none of the current config files need to be changed.
To post a comment you must log in.
Revision history for this message
| Robert Collins (lifeless) | # |
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
