f-l: Temporary: Revert upstream patches 89/31589/1
authorAlexandru Avadanii <Alexandru.Avadanii@enea.com>
Thu, 23 Mar 2017 14:58:23 +0000 (15:58 +0100)
committerAlexandru Avadanii <Alexandru.Avadanii@enea.com>
Thu, 23 Mar 2017 14:59:35 +0000 (15:59 +0100)
In order for our patches to still apply, revert latest changes from
upstream Fuel@Openstack.

Change-Id: Icca323a71669b413d9777df889b2e2b8b884e261
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
patches/fuel-library/0000-Revert-Fix-building-ironic-bootstrap.patch [new file with mode: 0644]
patches/fuel-library/0000-Revert-Fix-task-dependencies-for-provision-and-delet.patch [new file with mode: 0644]

diff --git a/patches/fuel-library/0000-Revert-Fix-building-ironic-bootstrap.patch b/patches/fuel-library/0000-Revert-Fix-building-ironic-bootstrap.patch
new file mode 100644 (file)
index 0000000..652bb27
--- /dev/null
@@ -0,0 +1,82 @@
+::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::
+: Copyright (c) 2017 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: Thu, 23 Mar 2017 15:57:02 +0100
+Subject: [PATCH] Revert "Fix building ironic bootstrap"
+
+This reverts commit c90b8a2fab0b8deab06d75b93394f32c60963be7.
+---
+ .../manifests/provision/build_image.pp             | 34 +++++++++++-----------
+ 1 file changed, 17 insertions(+), 17 deletions(-)
+
+diff --git a/deployment/puppet/osnailyfacter/manifests/provision/build_image.pp b/deployment/puppet/osnailyfacter/manifests/provision/build_image.pp
+index b40f3e2..02fa0f0 100644
+--- a/deployment/puppet/osnailyfacter/manifests/provision/build_image.pp
++++ b/deployment/puppet/osnailyfacter/manifests/provision/build_image.pp
+@@ -3,9 +3,7 @@ class osnailyfacter::provision::build_image {
+   $data_file = "/var/lib/fuel/configs/${::cluster_id}/provision.yaml"
+   $data = loadyaml($data_file)
+
+-  if $data['ironic']['enabled'] == true {
+-    # TODO(vsaienko): Use the same system packages for fuel image and ironic bootstrap, but exclude
+-    # not needed for ironic like nailgun-agent, etc...
++  if $data['ironic']['enabled'] == 'true' {
+     $ironic_packages = [
+       'openssh-server',
+       'ntp',
+@@ -17,7 +15,9 @@ class osnailyfacter::provision::build_image {
+       'squashfs-tools',
+       'linux-firmware',
+       'msmtp-mta',
++      'hpsa-dkms',
+       'i40e-dkms',
++      'linux-firmware-nonfree',
+       'xz-utils',
+       'linux-headers-generic',
+     ]
+@@ -29,25 +29,25 @@ class osnailyfacter::provision::build_image {
+     $out_dir = "/var/www/nailgun/bootstrap/ironic/${::cluster_id}/"
+     $out_params = "--output-dir ${out_dir}"
+
+-    $ironic_extra_params = "--extra-dir /usr/share/ironic-fa-bootstrap-configs/ --no-compress --no-default-extra-dirs --no-default-packages"
+-    $ironic_log_params = '--log-file /var/log/fuel-ironic-bootstrap-image-build.log'
++    $extra_params = "--extra-dir /usr/share/ironic-fa-bootstrap-configs/ --no-compress --no-default-extra-dirs --no-default-packages"
++    $log_params = '--log-file /var/log/fuel-ironic-bootstrap-image-build.log'
+
+     exec { 'generate_image_with_ironic':
+-      command => "fuel-bootstrap build ${package_list} ${ssh_params} ${out_params} ${ironic_extra_params} ${ironic_log_params}",
++      command => "fuel-bootstrap build ${package_list} ${ssh_params} ${out_params} ${extra_params} ${log_params}",
+       path    => ['/bin', '/usr/bin'],
+-      timeout => 1800,
+-      unless  => "test -e /var/www/nailgun/bootstrap/ironic/${cluster_id}/vmlinuz",
++      unless  => "test -e /var/www/nailgun/bootstrap/ironic/${::cluster_id}/vmlinuz",
+     }
+
+-  }
+-
+-  $build_dir = '--image_build_dir /var/lib/fuel/ibp'
+-  $log_params = "--log-file /var/log/fuel-agent-env-${cluster_id}.log"
+-  $extra_params = '--data_driver nailgun_build_image'
++  } else {
++    $build_dir = '--image_build_dir /var/lib/fuel/ibp'
++    $log_params = "--log-file /var/log/fuel-agent-env-${::cluster_id}.log"
++    $extra_params = '--data_driver nailgun_build_image'
+
+-  exec { 'generate_image_with_fuel':
+-    command => "fa_build_image ${build_dir} ${log_params} ${extra_params} --input_data_file ${data_file}",
+-    path    => ['/bin', '/usr/bin'],
+-    timeout => 1800,
++    exec { 'generate_image_with_fuel':
++      command => "fa_build_image ${build_dir} ${log_params} ${extra_params} --input_data_file ${data_file}",
++      path    => ['/bin', '/usr/bin'],
++      timeout => 1800,
++      unless  => "test -e /var/www/nailgun/bootstrap/ironic/${::cluster_id}/vmlinuz",
++    }
+   }
+ }
diff --git a/patches/fuel-library/0000-Revert-Fix-task-dependencies-for-provision-and-delet.patch b/patches/fuel-library/0000-Revert-Fix-task-dependencies-for-provision-and-delet.patch
new file mode 100644 (file)
index 0000000..d692a73
--- /dev/null
@@ -0,0 +1,60 @@
+::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::
+: Copyright (c) 2017 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: Thu, 23 Mar 2017 15:54:11 +0100
+Subject: [PATCH] Revert "Fix task dependencies for provision and deletion
+ graphs"
+
+This reverts commit 399611521195d75225a4cbf5613739bec5798fd0.
+---
+ graphs/deletion/tasks.yaml  | 4 +---
+ graphs/provision/tasks.yaml | 6 ++----
+ 2 files changed, 3 insertions(+), 7 deletions(-)
+
+diff --git a/graphs/deletion/tasks.yaml b/graphs/deletion/tasks.yaml
+index a725dda..2fd0188 100644
+--- a/graphs/deletion/tasks.yaml
++++ b/graphs/deletion/tasks.yaml
+@@ -35,10 +35,8 @@
+   type: reboot
+   version: 2.1.0
+   role: ['/.*/']
++  requires: [reconfigure_dhcpd]
+   required_for: [node_erase]
+-  cross-depends:
+-    - name: reconfigure_dhcpd
+-      role: master
+   parameters:
+     timeout: 300
+
+diff --git a/graphs/provision/tasks.yaml b/graphs/provision/tasks.yaml
+index 8a5c45e..1494176 100644
+--- a/graphs/provision/tasks.yaml
++++ b/graphs/provision/tasks.yaml
+@@ -60,10 +60,8 @@
+   type: upload_file
+   version: 2.1.0
+   role: ['/.*/']
++  requires: [provision_master]
+   required_for: [system_provision]
+-  cross-depends:
+-    - name: provision_master
+-      role: master
+   parameters:
+     path: /tmp/provision.json
+     timeout: 180
+@@ -74,7 +72,7 @@
+   type: shell
+   version: 2.1.0
+   role: ['/.*/']
+-  requires: [upload_provision_info]
++  requires: [provision_master, upload_provision_info]
+   required_for: [node_reboot]
+   cross-depends:
+     - name: build_deploy_image