Skip to content

Commit 1a1a707

Browse files
Zuulopenstack-gerrit
authored andcommitted
Merge "Deprecate PReP partition support?"
2 parents fac7bee + 1739fa3 commit 1a1a707

File tree

5 files changed

+35
-1
lines changed

5 files changed

+35
-1
lines changed

ironic_python_agent/disk_utils.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -296,6 +296,9 @@ def make_partitions(dev, root_mb, swap_mb, ephemeral_mb,
296296
# here.
297297
if (cpu_arch.startswith("ppc64") and boot_mode == "bios"
298298
and boot_option == "local"):
299+
LOG.warning("PReP partition support is deprecated and will be "
300+
"removed in a future release. PowerPC (ppc64*) hardware "
301+
"support is being phased out.")
299302
LOG.debug("Add PReP boot partition (8 MB) to device: "
300303
"%(dev)s for node %(node)s",
301304
{'dev': dev, 'node': node_uuid})

ironic_python_agent/extensions/image.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -203,6 +203,9 @@ def _install_grub2(device, root_uuid, efi_system_part_uuid=None,
203203

204204
# For power we want to install grub directly onto the PreP partition
205205
if prep_boot_part_uuid:
206+
LOG.warning("PReP partition support is deprecated and will be "
207+
"removed in a future release. PowerPC (ppc64*) "
208+
"hardware support is being phased out.")
206209
device = partition_utils.get_partition(
207210
device, uuid=prep_boot_part_uuid)
208211

@@ -700,6 +703,11 @@ def install_bootloader(self, root_uuid, efi_system_part_uuid=None,
700703
else:
701704
ignore_failure = ignore_bootloader_failure
702705

706+
if prep_boot_part_uuid:
707+
LOG.warning("PReP partition support is deprecated and will be "
708+
"removed in a future release. PowerPC (ppc64*) "
709+
"hardware support is being phased out.")
710+
703711
try:
704712
if _efi_boot_setup(device, efi_system_part_uuid, target_boot_mode):
705713
return

ironic_python_agent/partition_utils.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -332,6 +332,9 @@ def work_on_disk(dev, root_mb, swap_mb, ephemeral_mb, ephemeral_format,
332332
}
333333

334334
if cpu_arch.startswith('ppc'):
335+
LOG.warning("PReP partition support is deprecated and will be "
336+
"removed in a future release. PowerPC (ppc64*) "
337+
"hardware support is being phased out.")
335338
uuids_to_return[
336339
'PReP Boot partition uuid'
337340
] = part_dict.get('PReP Boot partition')

ironic_python_agent/raid_utils.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -320,7 +320,8 @@ def get_volume_name_of_raid_device(raid_device, examine=False):
320320
return None
321321

322322

323-
# TODO(rg): handle PreP boot parts relocation as well
323+
# NOTE(cid): PReP partition support is deprecated and will be removed in a
324+
# future release. RAID support for PReP partitions will not be implemented.
324325
def prepare_boot_partitions_for_softraid(device, holders, efi_part,
325326
target_boot_mode):
326327
"""Prepare boot partitions when relevant.
Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
---
2+
deprecations:
3+
- |
4+
PReP (PowerPC Reference Platform) partition support is now deprecated and
5+
will be removed in a future release. This affects ppc64* (PowerPC 64-bit)
6+
hardware support. While POWER hardware remains an active platform, it
7+
represents a small, specialized share of typical ironic-python-agent
8+
deployment patterns.
9+
10+
The feature was originally added to support booting partition images
11+
locally on PowerPC hardware, but usage in the ironic-python-agent
12+
ecosystem may be limited.
13+
14+
Note that RAID support for PReP partitions was never implemented and
15+
will not be added before removal.
16+
17+
Users with ppc64* hardware deployments should plan to migrate to
18+
alternative solutions or contact the Ironic community if they have
19+
specific requirements.

0 commit comments

Comments
 (0)