From 572bb5a1ba7bef7df4bb73daca650b066e8c9ef0 Mon Sep 17 00:00:00 2001 From: Alexandru Avadanii Date: Thu, 23 Mar 2017 15:58:23 +0100 Subject: f-l: Temporary: Revert upstream patches In order for our patches to still apply, revert latest changes from upstream Fuel@Openstack. Change-Id: Icca323a71669b413d9777df889b2e2b8b884e261 Signed-off-by: Alexandru Avadanii --- ...0000-Revert-Fix-building-ironic-bootstrap.patch | 82 ++++++++++++++++++++++ ...task-dependencies-for-provision-and-delet.patch | 60 ++++++++++++++++ 2 files changed, 142 insertions(+) create mode 100644 patches/fuel-library/0000-Revert-Fix-building-ironic-bootstrap.patch create mode 100644 patches/fuel-library/0000-Revert-Fix-task-dependencies-for-provision-and-delet.patch 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 index 00000000..652bb276 --- /dev/null +++ b/patches/fuel-library/0000-Revert-Fix-building-ironic-bootstrap.patch @@ -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 +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 index 00000000..d692a73a --- /dev/null +++ b/patches/fuel-library/0000-Revert-Fix-task-dependencies-for-provision-and-delet.patch @@ -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 +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 -- cgit 1.2.3-korg