* Tue Apr 23 2024 Miroslav Rezanina <mrezanin@redhat.com> - 23.4-7.el8_10.2
- ci-fix-Fall-back-to-cached-local-ds-if-no-valid-ds-foun.patch [RHEL-32841] - Resolves: RHEL-32841 ([cloud-init][ESXi]VMware datasource resets on every boot causing it to lose network configuration [rhel-8.10.z])
This commit is contained in:
Miroslav Rezanina
parent
c2db87484b
commit
27b34ef3f9
156
ci-fix-Fall-back-to-cached-local-ds-if-no-valid-ds-foun.patch
Normal file
156
ci-fix-Fall-back-to-cached-local-ds-if-no-valid-ds-foun.patch
Normal file
@ -0,0 +1,156 @@
|
|||||||
|
From d17e05b1709e3b7148e889512282603f7399c857 Mon Sep 17 00:00:00 2001
|
||||||
|
From: PengpengSun <40026211+PengpengSun@users.noreply.github.com>
|
||||||
|
Date: Fri, 29 Mar 2024 22:39:13 +0800
|
||||||
|
Subject: [PATCH] fix: Fall back to cached local ds if no valid ds found
|
||||||
|
(#4997)
|
||||||
|
|
||||||
|
RH-Author: Ani Sinha <anisinha@redhat.com>
|
||||||
|
RH-MergeRequest: 133: fix: Fall back to cached local ds if no valid ds found (#4997)
|
||||||
|
RH-Jira: RHEL-32841
|
||||||
|
RH-Acked-by: Cathy Avery <cavery@redhat.com>
|
||||||
|
RH-Acked-by: Miroslav Rezanina <mrezanin@redhat.com>
|
||||||
|
RH-Commit: [1/1] df9c6fda66dee9622725ff2d52e64999796324b8
|
||||||
|
|
||||||
|
Rebooting an instance which has finished VMware guest
|
||||||
|
customization with DataSourceVMware will load
|
||||||
|
DataSourceNone due to metadata is NOT available.
|
||||||
|
|
||||||
|
This is mostly a re-post of PR#229, few differences are:
|
||||||
|
1. Let ds decide if fallback is allowed, not always fall back
|
||||||
|
to previous cached LOCAL ds.
|
||||||
|
2. No comparing instance-id of cached ds with previous instance-id
|
||||||
|
due to I think they are always identical.
|
||||||
|
|
||||||
|
Fixes GH-3402
|
||||||
|
|
||||||
|
(cherry picked from commit 9929a00580d50afc60bf4e0fb9f2f39d4f797b4b)
|
||||||
|
Signed-off-by: Ani Sinha <anisinha@redhat.com>
|
||||||
|
|
||||||
|
Conflicts:
|
||||||
|
cloudinit/sources/__init__.py
|
||||||
|
Conflicts because of changes in upstream source coming from
|
||||||
|
30d5e9a3358f4cbaced ("refactor: Use _unpickle rather than hasattr() in sources")
|
||||||
|
---
|
||||||
|
cloudinit/sources/DataSourceVMware.py | 14 +++++++++-
|
||||||
|
cloudinit/sources/__init__.py | 14 ++++++++++
|
||||||
|
cloudinit/stages.py | 40 +++++++++++++++++----------
|
||||||
|
3 files changed, 53 insertions(+), 15 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/cloudinit/sources/DataSourceVMware.py b/cloudinit/sources/DataSourceVMware.py
|
||||||
|
index 1591121d..2d5d42eb 100644
|
||||||
|
--- a/cloudinit/sources/DataSourceVMware.py
|
||||||
|
+++ b/cloudinit/sources/DataSourceVMware.py
|
||||||
|
@@ -197,7 +197,7 @@ class DataSourceVMware(sources.DataSource):
|
||||||
|
break
|
||||||
|
|
||||||
|
if not self.data_access_method:
|
||||||
|
- LOG.error("failed to find a valid data access method")
|
||||||
|
+ LOG.debug("failed to find a valid data access method")
|
||||||
|
return False
|
||||||
|
|
||||||
|
LOG.info("using data access method %s", self._get_subplatform())
|
||||||
|
@@ -291,6 +291,18 @@ class DataSourceVMware(sources.DataSource):
|
||||||
|
self.metadata["instance-id"] = str(id_file.read()).rstrip().lower()
|
||||||
|
return self.metadata["instance-id"]
|
||||||
|
|
||||||
|
+ def check_if_fallback_is_allowed(self):
|
||||||
|
+ if (
|
||||||
|
+ self.data_access_method
|
||||||
|
+ and self.data_access_method == DATA_ACCESS_METHOD_IMC
|
||||||
|
+ and is_vmware_platform()
|
||||||
|
+ ):
|
||||||
|
+ LOG.debug(
|
||||||
|
+ "Cache fallback is allowed for : %s", self._get_subplatform()
|
||||||
|
+ )
|
||||||
|
+ return True
|
||||||
|
+ return False
|
||||||
|
+
|
||||||
|
def get_public_ssh_keys(self):
|
||||||
|
for key_name in (
|
||||||
|
"public-keys-data",
|
||||||
|
diff --git a/cloudinit/sources/__init__.py b/cloudinit/sources/__init__.py
|
||||||
|
index c207b5ed..453801be 100644
|
||||||
|
--- a/cloudinit/sources/__init__.py
|
||||||
|
+++ b/cloudinit/sources/__init__.py
|
||||||
|
@@ -312,6 +312,10 @@ class DataSource(CloudInitPickleMixin, metaclass=abc.ABCMeta):
|
||||||
|
self.vendordata2_raw = None
|
||||||
|
if not hasattr(self, "skip_hotplug_detect"):
|
||||||
|
self.skip_hotplug_detect = False
|
||||||
|
+
|
||||||
|
+ if not hasattr(self, "check_if_fallback_is_allowed"):
|
||||||
|
+ setattr(self, "check_if_fallback_is_allowed", lambda: False)
|
||||||
|
+
|
||||||
|
if hasattr(self, "userdata") and self.userdata is not None:
|
||||||
|
# If userdata stores MIME data, on < python3.6 it will be
|
||||||
|
# missing the 'policy' attribute that exists on >=python3.6.
|
||||||
|
@@ -914,6 +918,16 @@ class DataSource(CloudInitPickleMixin, metaclass=abc.ABCMeta):
|
||||||
|
# quickly (local check only) if self.instance_id is still
|
||||||
|
return False
|
||||||
|
|
||||||
|
+ def check_if_fallback_is_allowed(self):
|
||||||
|
+ """check_if_fallback_is_allowed()
|
||||||
|
+ Checks if a cached ds is allowed to be restored when no valid ds is
|
||||||
|
+ found in local mode by checking instance-id and searching valid data
|
||||||
|
+ through ds list.
|
||||||
|
+
|
||||||
|
+ @return True if a ds allows fallback, False otherwise.
|
||||||
|
+ """
|
||||||
|
+ return False
|
||||||
|
+
|
||||||
|
@staticmethod
|
||||||
|
def _determine_dsmode(candidates, default=None, valid=None):
|
||||||
|
# return the first candidate that is non None, warn if not valid
|
||||||
|
diff --git a/cloudinit/stages.py b/cloudinit/stages.py
|
||||||
|
index 3b6405f5..0b795624 100644
|
||||||
|
--- a/cloudinit/stages.py
|
||||||
|
+++ b/cloudinit/stages.py
|
||||||
|
@@ -353,20 +353,32 @@ class Init:
|
||||||
|
LOG.debug(myrep.description)
|
||||||
|
|
||||||
|
if not ds:
|
||||||
|
- util.del_file(self.paths.instance_link)
|
||||||
|
- (cfg_list, pkg_list) = self._get_datasources()
|
||||||
|
- # Deep copy so that user-data handlers can not modify
|
||||||
|
- # (which will affect user-data handlers down the line...)
|
||||||
|
- (ds, dsname) = sources.find_source(
|
||||||
|
- self.cfg,
|
||||||
|
- self.distro,
|
||||||
|
- self.paths,
|
||||||
|
- copy.deepcopy(self.ds_deps),
|
||||||
|
- cfg_list,
|
||||||
|
- pkg_list,
|
||||||
|
- self.reporter,
|
||||||
|
- )
|
||||||
|
- LOG.info("Loaded datasource %s - %s", dsname, ds)
|
||||||
|
+ try:
|
||||||
|
+ cfg_list, pkg_list = self._get_datasources()
|
||||||
|
+ # Deep copy so that user-data handlers can not modify
|
||||||
|
+ # (which will affect user-data handlers down the line...)
|
||||||
|
+ ds, dsname = sources.find_source(
|
||||||
|
+ self.cfg,
|
||||||
|
+ self.distro,
|
||||||
|
+ self.paths,
|
||||||
|
+ copy.deepcopy(self.ds_deps),
|
||||||
|
+ cfg_list,
|
||||||
|
+ pkg_list,
|
||||||
|
+ self.reporter,
|
||||||
|
+ )
|
||||||
|
+ util.del_file(self.paths.instance_link)
|
||||||
|
+ LOG.info("Loaded datasource %s - %s", dsname, ds)
|
||||||
|
+ except sources.DataSourceNotFoundException as e:
|
||||||
|
+ if existing != "check":
|
||||||
|
+ raise e
|
||||||
|
+ ds = self._restore_from_cache()
|
||||||
|
+ if ds and ds.check_if_fallback_is_allowed():
|
||||||
|
+ LOG.info(
|
||||||
|
+ "Restored fallback datasource from checked cache: %s",
|
||||||
|
+ ds,
|
||||||
|
+ )
|
||||||
|
+ else:
|
||||||
|
+ raise e
|
||||||
|
self.datasource = ds
|
||||||
|
# Ensure we adjust our path members datasource
|
||||||
|
# now that we have one (thus allowing ipath to be used)
|
||||||
|
--
|
||||||
|
2.39.3
|
||||||
|
|
||||||
@ -6,7 +6,7 @@
|
|||||||
|
|
||||||
Name: cloud-init
|
Name: cloud-init
|
||||||
Version: 23.4
|
Version: 23.4
|
||||||
Release: 7%{?dist}.1
|
Release: 7%{?dist}.2
|
||||||
Summary: Cloud instance init scripts
|
Summary: Cloud instance init scripts
|
||||||
|
|
||||||
Group: System Environment/Base
|
Group: System Environment/Base
|
||||||
@ -45,6 +45,8 @@ Patch24: ci-fix-Correct-v2-NetworkManager-route-rendering-4637.patch
|
|||||||
Patch25: ci-feat-apply-global-DNS-to-interfaces-in-network-manag.patch
|
Patch25: ci-feat-apply-global-DNS-to-interfaces-in-network-manag.patch
|
||||||
# For RHEL-29710 - Suggest to backport patch ff40d1a to undeprecate 'network' in schema route definition [rhel-8.10.0.z]
|
# For RHEL-29710 - Suggest to backport patch ff40d1a to undeprecate 'network' in schema route definition [rhel-8.10.0.z]
|
||||||
Patch26: ci-fix-Undeprecate-network-in-schema-route-definition-5.patch
|
Patch26: ci-fix-Undeprecate-network-in-schema-route-definition-5.patch
|
||||||
|
# For RHEL-32841 - [cloud-init][ESXi]VMware datasource resets on every boot causing it to lose network configuration [rhel-8.10.z]
|
||||||
|
Patch27: ci-fix-Fall-back-to-cached-local-ds-if-no-valid-ds-foun.patch
|
||||||
|
|
||||||
BuildArch: noarch
|
BuildArch: noarch
|
||||||
|
|
||||||
@ -260,6 +262,11 @@ fi
|
|||||||
%config(noreplace) %{_sysconfdir}/rsyslog.d/21-cloudinit.conf
|
%config(noreplace) %{_sysconfdir}/rsyslog.d/21-cloudinit.conf
|
||||||
|
|
||||||
%changelog
|
%changelog
|
||||||
|
* Tue Apr 23 2024 Miroslav Rezanina <mrezanin@redhat.com> - 23.4-7.el8_10.2
|
||||||
|
- ci-fix-Fall-back-to-cached-local-ds-if-no-valid-ds-foun.patch [RHEL-32841]
|
||||||
|
- Resolves: RHEL-32841
|
||||||
|
([cloud-init][ESXi]VMware datasource resets on every boot causing it to lose network configuration [rhel-8.10.z])
|
||||||
|
|
||||||
* Fri Apr 05 2024 Jon Maloy <jmaloy@redhat.com> - 23.4-7.el8.1
|
* Fri Apr 05 2024 Jon Maloy <jmaloy@redhat.com> - 23.4-7.el8.1
|
||||||
- ci-fix-Correct-v2-NetworkManager-route-rendering-4637.patch [RHEL-27134]
|
- ci-fix-Correct-v2-NetworkManager-route-rendering-4637.patch [RHEL-27134]
|
||||||
- ci-feat-apply-global-DNS-to-interfaces-in-network-manag.patch [RHEL-27134]
|
- ci-feat-apply-global-DNS-to-interfaces-in-network-manag.patch [RHEL-27134]
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user