Merge ~mpontillo/maas:move-network-monitoring-to-maas-common--part2 into maas:master

Proposed by Mike Pontillo
Status: Rejected
Rejected by: Mike Pontillo
Proposed branch: ~mpontillo/maas:move-network-monitoring-to-maas-common--part2
Merge into: maas:master
Prerequisite: ~mpontillo/maas:move-network-monitoring-to-maas-common--part1
Diff against target: 189 lines (+40/-13)
8 files modified
Makefile (+6/-5)
buildout.cfg (+2/-0)
debian/changelog (+2/-0)
debian/compat (+1/-1)
debian/control (+3/-1)
debian/extras/maas-rack (+0/-0)
debian/maas-common.install (+7/-0)
src/provisioningserver/__main__.py (+19/-6)
Reviewer Review Type Date Requested Status
MAAS Maintainers Pending
Review via email: mp+329413@code.launchpad.net

Commit message

Add /usr/lib/maas/maas-common command.

To post a comment you must log in.
Revision history for this message
Mike Pontillo (mpontillo) wrote :

Rejecting in favor of proposing a single branch.

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/Makefile b/Makefile
index faea62e..61c5518 100644
--- a/Makefile
+++ b/Makefile
@@ -52,6 +52,7 @@ build: \
52 bin/buildout \52 bin/buildout \
53 bin/database \53 bin/database \
54 bin/maas \54 bin/maas \
55 bin/maas-common \
55 bin/maas-rack \56 bin/maas-rack \
56 bin/maas-region \57 bin/maas-region \
57 bin/twistd.rack \58 bin/twistd.rack \
@@ -118,7 +119,7 @@ bin/maas-region bin/twistd.region: \
118119
119bin/test.region: \120bin/test.region: \
120 bin/buildout buildout.cfg versions.cfg setup.py $(js_enums) \121 bin/buildout buildout.cfg versions.cfg setup.py $(js_enums) \
121 bin/maas-region bin/maas-rack122 bin/maas-region bin/maas-rack bin/maas-common
122 $(buildout) install region-test123 $(buildout) install region-test
123 @touch --no-create $@124 @touch --no-create $@
124125
@@ -152,7 +153,7 @@ bin/test.testing: \
152 $(buildout) install testing-test153 $(buildout) install testing-test
153 @touch --no-create $@154 @touch --no-create $@
154155
155bin/maas-rack bin/twistd.rack: \156bin/maas-rack bin/twistd.rack bin/maas-common: \
156 bin/buildout buildout.cfg versions.cfg setup.py157 bin/buildout buildout.cfg versions.cfg setup.py
157 $(buildout) install rack158 $(buildout) install rack
158 @touch --no-create $@159 @touch --no-create $@
@@ -568,13 +569,13 @@ services/dns/@deps: bin/py
568569
569services/database/@deps: bin/database570services/database/@deps: bin/database
570571
571services/rackd/@deps: bin/twistd.rack bin/maas-rack572services/rackd/@deps: bin/twistd.rack bin/maas-rack bin/maas-common
572573
573services/reloader/@deps:574services/reloader/@deps:
574575
575services/regiond/@deps: bin/maas-region bin/maas-rack576services/regiond/@deps: bin/maas-region bin/maas-rack bin/maas-common
576577
577services/regiond2/@deps: bin/maas-region bin/maas-rack578services/regiond2/@deps: bin/maas-region bin/maas-rack bin/maas-common
578579
579#580#
580# Package building581# Package building
diff --git a/buildout.cfg b/buildout.cfg
index 59f18a0..43b1b77 100644
--- a/buildout.cfg
+++ b/buildout.cfg
@@ -279,11 +279,13 @@ eggs =
279 ${common:test-eggs}279 ${common:test-eggs}
280entry-points =280entry-points =
281 maas-rack=provisioningserver.__main__:main281 maas-rack=provisioningserver.__main__:main
282 maas-common=provisioningserver.__main__:main
282 twistd.rack=twisted.scripts.twistd:run283 twistd.rack=twisted.scripts.twistd:run
283extra-paths =284extra-paths =
284 ${common:extra-paths}285 ${common:extra-paths}
285scripts =286scripts =
286 maas-rack287 maas-rack
288 maas-common
287 twistd.rack289 twistd.rack
288initialization =290initialization =
289 ${common:initialization}291 ${common:initialization}
diff --git a/debian/changelog b/debian/changelog
index ca64683..f3ba3d5 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -2,6 +2,8 @@ maas (2.3.0~alpha3-0ubuntu1) UNRELEASED; urgency=medium
22
3 * Move scripts to maas-common that are used by the MAAS3 * Move scripts to maas-common that are used by the MAAS
4 region controller in addition to the rack.4 region controller in addition to the rack.
5 * Add /usr/lib/maas/maas-common script.
6 * Move to debhelper compat level 9.
57
6 -- Mike Pontillo <mike.pontillo@canonical.com> Tue, 22 Aug 2017 15:28:42 -07008 -- Mike Pontillo <mike.pontillo@canonical.com> Tue, 22 Aug 2017 15:28:42 -0700
79
diff --git a/debian/compat b/debian/compat
index 7f8f011..ec63514 100644
--- a/debian/compat
+++ b/debian/compat
@@ -1 +1 @@
1719
diff --git a/debian/control b/debian/control
index d3fb250..e0beb12 100644
--- a/debian/control
+++ b/debian/control
@@ -4,6 +4,7 @@ Priority: optional
4Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>4Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
5Build-Depends: debhelper (>= 8.1.0~),5Build-Depends: debhelper (>= 8.1.0~),
6 dh-apport,6 dh-apport,
7 dh-exec,
7 dh-python,8 dh-python,
8 dh-systemd,9 dh-systemd,
9 po-debconf,10 po-debconf,
@@ -57,7 +58,8 @@ Description: "Metal as a Service" is a physical cloud and IPAM
57Package: maas-common58Package: maas-common
58Architecture: all59Architecture: all
59Depends: rsyslog,60Depends: rsyslog,
60 ${misc:Depends}61 ${misc:Depends},
62 python3-maas-provisioningserver
61Breaks: maas ( <= 0.1+bzr1048+dfsg-0ubuntu1 )63Breaks: maas ( <= 0.1+bzr1048+dfsg-0ubuntu1 )
62Replaces: maas ( <= 0.1+bzr1048+dfsg-0ubuntu1 )64Replaces: maas ( <= 0.1+bzr1048+dfsg-0ubuntu1 )
63Description: MAAS server common files65Description: MAAS server common files
diff --git a/debian/extras/maas-rack b/debian/extras/maas-rack
64old mode 10064466old mode 100644
65new mode 10075567new mode 100755
index 4b69afa..4b69afa
--- a/debian/extras/maas-rack
+++ b/debian/extras/maas-rack
diff --git a/debian/maas-common.install b/debian/maas-common.install
66old mode 10064468old mode 100644
67new mode 10075569new mode 100755
index aea3a3e..fa268cc
--- a/debian/maas-common.install
+++ b/debian/maas-common.install
@@ -1,3 +1,5 @@
1#!/usr/bin/dh-exec --with=install
2
1debian/tmp/usr/share/maas/maas-rsyslog.conf3debian/tmp/usr/share/maas/maas-rsyslog.conf
2debian/extras/99-maas-common-sudoers etc/sudoers.d4debian/extras/99-maas-common-sudoers etc/sudoers.d
3debian/tmp/usr/lib/maas/maas-delete-file5debian/tmp/usr/lib/maas/maas-delete-file
@@ -7,3 +9,8 @@ debian/tmp/usr/lib/maas/maas-write-file
7debian/tmp/usr/lib/maas/network-monitor9debian/tmp/usr/lib/maas/network-monitor
8debian/tmp/usr/lib/maas/beacon-monitor10debian/tmp/usr/lib/maas/beacon-monitor
9debian/tmp/usr/lib/maas/dhcp-monitor11debian/tmp/usr/lib/maas/dhcp-monitor
12
13# Install an alias to maas-rack as maas-common.
14# For information on the usage of '=>' here, see:
15# https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=245554
16debian/extras/maas-rack => /usr/lib/maas/maas-common
diff --git a/src/provisioningserver/__main__.py b/src/provisioningserver/__main__.py
index 4016e35..042f6b3 100644
--- a/src/provisioningserver/__main__.py
+++ b/src/provisioningserver/__main__.py
@@ -4,6 +4,8 @@
44
5"""Command-line interface for the MAAS provisioning component."""5"""Command-line interface for the MAAS provisioning component."""
66
7import sys
8
7from provisioningserver import security9from provisioningserver import security
8import provisioningserver.boot.install_grub10import provisioningserver.boot.install_grub
9import provisioningserver.cluster_config_command11import provisioningserver.cluster_config_command
@@ -19,17 +21,20 @@ from provisioningserver.utils.script import MainScript
19import provisioningserver.utils.send_beacons21import provisioningserver.utils.send_beacons
2022
2123
22script_commands = {24COMMON_COMMANDS = {
23 'check-for-shared-secret': security.CheckForSharedSecretScript,
24 'config': provisioningserver.cluster_config_command,
25 'install-shared-secret': security.InstallSharedSecretScript,
26 'install-uefi-config': provisioningserver.boot.install_grub,
27 'observe-arp': provisioningserver.utils.arp,25 'observe-arp': provisioningserver.utils.arp,
28 'observe-beacons': provisioningserver.utils.beaconing,26 'observe-beacons': provisioningserver.utils.beaconing,
29 'observe-mdns': provisioningserver.utils.avahi,27 'observe-mdns': provisioningserver.utils.avahi,
30 'observe-dhcp': provisioningserver.utils.dhcp,28 'observe-dhcp': provisioningserver.utils.dhcp,
31 'send-beacons': provisioningserver.utils.send_beacons,29 'send-beacons': provisioningserver.utils.send_beacons,
32 'scan-network': provisioningserver.utils.scan_network,30 'scan-network': provisioningserver.utils.scan_network,
31}
32
33RACK_ONLY_COMMANDS = {
34 'check-for-shared-secret': security.CheckForSharedSecretScript,
35 'config': provisioningserver.cluster_config_command,
36 'install-shared-secret': security.InstallSharedSecretScript,
37 'install-uefi-config': provisioningserver.boot.install_grub,
33 'register': provisioningserver.register_command,38 'register': provisioningserver.register_command,
34 'support-dump': provisioningserver.support_dump,39 'support-dump': provisioningserver.support_dump,
35 'upgrade-cluster': provisioningserver.upgrade_cluster,40 'upgrade-cluster': provisioningserver.upgrade_cluster,
@@ -37,6 +42,14 @@ script_commands = {
3742
3843
39main = MainScript(__doc__)44main = MainScript(__doc__)
40for name, command in sorted(script_commands.items()):45
46commands = COMMON_COMMANDS.copy()
47
48# If 'maas-common' isn't being executed, add rack-specific commands in addition
49# to the generic set of commands.
50if not sys.argv[0].endswith('/maas-common'):
51 commands.update(RACK_ONLY_COMMANDS)
52
53for name, command in sorted(commands.items()):
41 main.register(name, command)54 main.register(name, command)
42main()55main()

Subscribers

People subscribed via source and target branches