patches: Squash maas region changes 67/49667/1
authorAlexandru Avadanii <Alexandru.Avadanii@enea.com>
Tue, 26 Dec 2017 23:35:15 +0000 (00:35 +0100)
committerAlexandru Avadanii <Alexandru.Avadanii@enea.com>
Tue, 26 Dec 2017 23:35:15 +0000 (00:35 +0100)
When re-deploying with `-f` flag, `patch -R` cannot cleanly revert
maas region changes with overlapping context lines, so squash them
into a single file.

Change-Id: I87dae72a12fea833e9e6729de21d4ce5f262695e
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
mcp/patches/0010-maas-region-allow-timeout-override.patch
mcp/patches/0011-maas-region-interface-default-mode-DHCP.patch [deleted file]
mcp/patches/patches.list

index 6763b8e..37bdf58 100644 (file)
@@ -10,7 +10,10 @@ From: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
 Date: Sat, 23 Sep 2017 02:03:01 +0200
 Subject: [PATCH] maas: region: allow timeout override
 
-Poor implementation that allows overiding comissioning/deploying
+Two changes are squashed into a single patch so we can test for
+reverse-applying the cummulative diff.
+
+1. Poor implementation that allows overiding comissioning/deploying
 timeouts, which are hardcoded in MaaS source code.
 
 Add the following reclass _param:
@@ -19,13 +22,24 @@ Add the following reclass _param:
 
 The defaults are now 10/15 minutes (MaaS defaults to 20/40 min).
 
+2. maas: region: interface: default mode DHCP
+
+Switch MaaS interface mode default from AUTO to DHCP.
+This way, MaaS will not statically assign IPs *outside* the dynamic
+reserved range, but instead DHCP will assign them *inside* the range.
+
+One alternative would be to handle this via API calls, for each
+node's PXE physical interface, but that overcomplicates things.
+
+JIRA: FUEL-316
+
 Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
 ---
 
 diff --git a/maas/region.sls b/maas/region.sls
 --- a/maas/region.sls
 +++ b/maas/region.sls
-@@ -19,6 +19,26 @@
+@@ -19,6 +19,36 @@
    - require:
      - pkg: maas_region_packages
 
@@ -48,6 +62,16 @@ diff --git a/maas/region.sls b/maas/region.sls
 +      - pkg: maas_region_packages
 +    - require_in:
 +      - service: maas_region_services
++
++maas_interface_default_mode_dhcp:
++  file.replace:
++    - name: "/usr/lib/python3/dist-packages/maasserver/models/node.py"
++    - pattern: '^(\s+INTERFACE_LINK_TYPE)\.AUTO'
++    - repl: \1.DHCP
++    - require:
++      - pkg: maas_region_packages
++    - require_in:
++      - service: maas_region_services
 +
  {%- if region.get('enable_iframe', False)  %}
 
diff --git a/mcp/patches/0011-maas-region-interface-default-mode-DHCP.patch b/mcp/patches/0011-maas-region-interface-default-mode-DHCP.patch
deleted file mode 100644 (file)
index d161a95..0000000
+++ /dev/null
@@ -1,44 +0,0 @@
-::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::
-: Copyright (c) 2017 Mirantis Inc., Enea AB and others.
-:
-: All rights reserved. This program and the accompanying materials
-: are made available under the terms of the Apache License, Version 2.0
-: which accompanies this distribution, and is available at
-: http://www.apache.org/licenses/LICENSE-2.0
-::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::
-From: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
-Date: Mon, 18 Dec 2017 02:03:01 +0200
-Subject: [PATCH] maas: region: interface: default mode DHCP
-
-Switch MaaS interface mode default from AUTO to DHCP.
-This way, MaaS will not statically assign IPs *outside* the dynamic
-reserved range, but instead DHCP will assign them *inside* the range.
-
-One alternative would be to handle this via API calls, for each
-node's PXE physical interface, but that overcomplicates things.
-
-JIRA: FUEL-316
-
-Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
----
-
-diff --git a/maas/region.sls b/maas/region.sls
---- a/maas/region.sls
-+++ b/maas/region.sls
-@@ -39,6 +39,16 @@
-     - require_in:
-       - service: maas_region_services
-
-+maas_interface_default_mode_dhcp:
-+  file.replace:
-+    - name: "/usr/lib/python3/dist-packages/maasserver/models/node.py"
-+    - pattern: '^(\s+INTERFACE_LINK_TYPE)\.AUTO'
-+    - repl: \1.DHCP
-+    - require:
-+      - pkg: maas_region_packages
-+    - require_in:
-+      - service: maas_region_services
-+
- {%- if region.get('enable_iframe', False)  %}
-
- /etc/apache2/conf-enabled/maas-http.conf:
index 56c3f1c..cd32378 100644 (file)
@@ -13,5 +13,4 @@
 /usr/share/salt-formulas/env: 0006-maas-module-Add-VLAN-DHCP-enable-support.patch
 /usr/share/salt-formulas/env: 0008-Handle-file_recv-option.patch
 /usr/share/salt-formulas/env: 0010-maas-region-allow-timeout-override.patch
-/usr/share/salt-formulas/env: 0011-maas-region-interface-default-mode-DHCP.patch
 /usr/share/salt-formulas/env: 0012-linux.storage.lvm-Disable-filter.patch