Merge lp:~dohse/graphite/option-noreload into lp:~graphite-dev/graphite/main
Proposed by
Jonas Dohse
Status: | Merged |
---|---|
Merged at revision: | 689 |
Proposed branch: | lp:~dohse/graphite/option-noreload |
Merge into: | lp:~graphite-dev/graphite/main |
Diff against target: |
32 lines (+14/-4) 1 file modified
bin/run-graphite-devel-server.py (+14/-4) |
To merge this branch: | bzr merge lp:~dohse/graphite/option-noreload |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Michael Leinartas | Approve | ||
Review via email: mp+93377@code.launchpad.net |
Description of the change
Allow option `--noreload` for run-graphite-
Use `os.execvp` instead of `os.system` to allow better control of django.
To be able to cleanly shutdown django in a development or integration environment pass --noreload to run-graphite-
To post a comment you must log in.
The changes make sense and test out ok. Looks good, thanks