Merge lp:~jml/launchpad/codebrowse-port-config into lp:launchpad
Proposed by
Jonathan Lange
on 2010-10-19
| Status: | Merged |
|---|---|
| Approved by: | Robert Collins on 2010-10-19 |
| 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) | 2010-10-19 | Approve on 2010-10-19 | |
|
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.
review:
Approve
