Merge lp:~allenap/maas/packaging.remove-maas into lp:~maas-maintainers/maas/packaging

Proposed by Gavin Panella
Status: Merged
Approved by: Gavin Panella
Approved revision: 541
Merged at revision: 537
Proposed branch: lp:~allenap/maas/packaging.remove-maas
Merge into: lp:~maas-maintainers/maas/packaging
Diff against target: 55 lines (+6/-9)
4 files modified
debian/changelog (+3/-0)
debian/extras/maas-region (+2/-4)
debian/maas-region-api.install (+0/-3)
debian/maas-region-api.maas-regiond-worker@.service (+1/-2)
To merge this branch: bzr merge lp:~allenap/maas/packaging.remove-maas
Reviewer Review Type Date Requested Status
Blake Rouse (community) Approve
Review via email: mp+312330@code.launchpad.net

Commit message

Don't install Django settings into /usr/share/maas/maas.

For regiond these were squatting the `maas` namespace, preventing its use elsewhere.

To post a comment you must log in.
Revision history for this message
Gavin Panella (allenap) wrote :
Revision history for this message
Blake Rouse (blake-rouse) wrote :

What about the upgrade path? Will the /usr/share/maas/maas be removed?

review: Needs Information
Revision history for this message
Gavin Panella (allenap) wrote :

> What about the upgrade path? Will the /usr/share/maas/maas be removed?

I tried this out on my NUCs and /usr/share/maas/maas was removed, I assume by the default packaging machinery.

Revision history for this message
Blake Rouse (blake-rouse) wrote :

Perfect!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2016-11-22 01:03:30 +0000
+++ debian/changelog 2016-12-02 09:33:29 +0000
@@ -1,6 +1,9 @@
1maas (2.2.0~alpha1+bzr5562-0ubuntu1) UNRELEASED; urgency=medium1maas (2.2.0~alpha1+bzr5562-0ubuntu1) UNRELEASED; urgency=medium
22
3 * UNRELEASED3 * UNRELEASED
4 - Don't install Django settings into /usr/share/maas/maas. For
5 regiond these were squatting the `maas` namespace, preventing its
6 use elsewhere (LP: 1646748)
47
5 -- Andres Rodriguez <andreserl@ubuntu.com> Mon, 21 Nov 2016 20:01:31 -05008 -- Andres Rodriguez <andreserl@ubuntu.com> Mon, 21 Nov 2016 20:01:31 -0500
69
710
=== modified file 'debian/extras/maas-region'
--- debian/extras/maas-region 2016-03-07 22:01:56 +0000
+++ debian/extras/maas-region 2016-12-02 09:33:29 +0000
@@ -41,10 +41,8 @@
4141
42def run():42def run():
43 # Force the production MAAS Django configuration.43 # Force the production MAAS Django configuration.
44 os.environ.setdefault("DJANGO_SETTINGS_MODULE", "maas.settings")44 os.environ.setdefault(
4545 "DJANGO_SETTINGS_MODULE", "maasserver.djangosettings.settings")
46 # The Django configuration lives outside of sys.path.
47 sys.path.append('/usr/share/maas')
4846
49 # Let Django do the rest.47 # Let Django do the rest.
50 from django.core import management48 from django.core import management
5149
=== modified file 'debian/maas-region-api.install'
--- debian/maas-region-api.install 2016-09-01 14:56:04 +0000
+++ debian/maas-region-api.install 2016-12-02 09:33:29 +0000
@@ -1,6 +1,3 @@
1# Install project
2debian/tmp/usr/lib/python*/dist-packages/maas usr/share/maas
3
4# Install static files1# Install static files
5debian/tmp/usr/share/maas/web/static2debian/tmp/usr/share/maas/web/static
63
74
=== modified file 'debian/maas-region-api.maas-regiond-worker@.service'
--- debian/maas-region-api.maas-regiond-worker@.service 2016-11-08 16:16:57 +0000
+++ debian/maas-region-api.maas-regiond-worker@.service 2016-12-02 09:33:29 +0000
@@ -13,8 +13,7 @@
13Restart=always13Restart=always
14RestartSec=10s14RestartSec=10s
15Environment="MAAS_REGIOND_WORKER_ID=%i"15Environment="MAAS_REGIOND_WORKER_ID=%i"
16Environment="DJANGO_SETTINGS_MODULE=maas.settings"16Environment="DJANGO_SETTINGS_MODULE=maasserver.djangosettings.settings"
17Environment="PYTHONPATH=/usr/share/maas"
18Environment="LOGFILE=/var/log/maas/regiond.log"17Environment="LOGFILE=/var/log/maas/regiond.log"
19# Logs go to the journal; read them with:18# Logs go to the journal; read them with:
20# journalctl -u maas-regiond-worker@*19# journalctl -u maas-regiond-worker@*

Subscribers

People subscribed via source and target branches