2a13f189be
Resolves: RHEL-58465 - package rhel-system-roles.noarch does not provide docs for ansible-doc [rhel-8.10.z] Resolves: RHEL-58494 ad_integration - fix: Sets domain name lower case in realmd.conf section header [rhel-8.10.z] Resolves: RHEL-58917 bootloader - bootloader role tests do not work on ostree [rhel-8.10.z] Resolves: RHEL-45711 bootloader - fix: Set user.cfg path to /boot/grub2/ on EL 9 UEFI [rhel-8] Resolves: RHEL-58515 cockpit - cockpit install all wildcard match does not work in newer el9 [rhel-8.10.z] Resolves: RHEL-58485 logging - RFE - system-roles - logging: Add truncate options for local file inputs [rhel-8.10.z] Resolves: RHEL-58481 logging - redhat.rhel_system_roles.logging role fails to process logging_outputs: of type: "custom" [rhel-8.10.z] Resolves: RHEL-58477 logging - [RFE] Add the umask settings or enable a variable in linux-system-roles.logging [rhel-8.10.z] Resolves: RHEL-37550 logging - Setup imuxsock using rhel-system-roles.logging causing an error EL8 Resolves: RHEL-58519 nbde_client - feat: Allow initrd configuration to be skipped [rhel-8.10.z] Resolves: RHEL-58525 podman - fix: proper cleanup for networks; ensure cleanup of resources [rhel-8.10.z] Resolves: RHEL-58511 podman - fix: grab name of network to remove from quadlet file [rhel-8.10.z] Resolves: RHEL-58507 podman - Create podman secret when skip_existing=True and it does not exist [rhel-8.10.z] Resolves: RHEL-58503 podman - fix: do not use become for changing hostdir ownership, and expose subuid/subgid info [rhel-8.10.z] Resolves: RHEL-58498 podman - fix: use correct user for cancel linger file name [rhel-8.10.z] Resolves: RHEL-58460 podman - redhat.rhel_system_roles.podman fails to configure and run containers with podman rootless using different username and groupname. [rhel-8.10.z] Resolves: RHEL-58473 sshd - second SSHD service broken [rhel-8.10.z] Resolves: RHEL-58469 storage - rhel-system-role.storage is not idempotent [rhel-8.10.z] Resolves: RHEL-58489 timesync - System Roles: No module documentation [rhel-8.10.z] (cherry picked from commit 350d523452546e35bb0805af9ad9cc74712899d7)
65 lines
2.3 KiB
Diff
65 lines
2.3 KiB
Diff
From 9f561445271a14fee598e9a793f72297f66eae56 Mon Sep 17 00:00:00 2001
|
|
From: Vojtech Trefny <vtrefny@redhat.com>
|
|
Date: Wed, 10 Apr 2024 17:05:46 +0200
|
|
Subject: [PATCH 02/10] fix: Fix recreate check for formats without labelling
|
|
support
|
|
|
|
Formats like LUKS or LVMPV don't support labels so we need to skip
|
|
the label check in BlivetVolume._reformat.
|
|
|
|
Resolves: RHEL-29874
|
|
(cherry picked from commit a70e8108110e30ebc5e7c404d39339c511f9bd09)
|
|
---
|
|
library/blivet.py | 3 +++
|
|
tests/tests_volume_relabel.yml | 20 ++++++++++++++++++++
|
|
2 files changed, 23 insertions(+)
|
|
|
|
diff --git a/library/blivet.py b/library/blivet.py
|
|
index 20389ea..18807de 100644
|
|
--- a/library/blivet.py
|
|
+++ b/library/blivet.py
|
|
@@ -826,6 +826,9 @@ class BlivetVolume(BlivetBase):
|
|
if ((fmt is None and self._device.format.type is None)
|
|
or (fmt is not None and self._device.format.type == fmt.type)):
|
|
# format is the same, no need to run reformatting
|
|
+ if not hasattr(self._device.format, "label"):
|
|
+ # not all formats support labels
|
|
+ return
|
|
dev_label = '' if self._device.format.label is None else self._device.format.label
|
|
if dev_label != fmt.label:
|
|
# ...but the label has changed - schedule modification action
|
|
diff --git a/tests/tests_volume_relabel.yml b/tests/tests_volume_relabel.yml
|
|
index 8916b73..6624fbd 100644
|
|
--- a/tests/tests_volume_relabel.yml
|
|
+++ b/tests/tests_volume_relabel.yml
|
|
@@ -111,6 +111,26 @@
|
|
- name: Verify role results
|
|
include_tasks: verify-role-results.yml
|
|
|
|
+ - name: Format the device to LVMPV which doesn't support labels
|
|
+ include_role:
|
|
+ name: linux-system-roles.storage
|
|
+ vars:
|
|
+ storage_volumes:
|
|
+ - name: test1
|
|
+ type: disk
|
|
+ fs_type: lvmpv
|
|
+ disks: "{{ unused_disks }}"
|
|
+
|
|
+ - name: Rerun to check we don't try to relabel preexisitng LVMPV (regression test for RHEL-29874)
|
|
+ include_role:
|
|
+ name: linux-system-roles.storage
|
|
+ vars:
|
|
+ storage_volumes:
|
|
+ - name: test1
|
|
+ type: disk
|
|
+ fs_type: lvmpv
|
|
+ disks: "{{ unused_disks }}"
|
|
+
|
|
- name: Clean up
|
|
include_role:
|
|
name: linux-system-roles.storage
|
|
--
|
|
2.46.0
|
|
|