Skip to content

Use /dev/disk/by-id/coreos-boot-disk when creating /var partition#99875

Open
jlebon wants to merge 1 commit into
openshift:mainfrom
jlebon:pr/var-partition
Open

Use /dev/disk/by-id/coreos-boot-disk when creating /var partition#99875
jlebon wants to merge 1 commit into
openshift:mainfrom
jlebon:pr/var-partition

Conversation

@jlebon
Copy link
Copy Markdown
Member

@jlebon jlebon commented Sep 30, 2025

This has been supported for a while and simplifies things because then your Ignition config is no longer specific to a device name.

Version(s):

It's quite old, but we can just do 4.19+.

Issue:

No direct issue. Came up while I was looking at https://issues.redhat.com/browse/OCPBUGS-62461.

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:

@openshift-ci openshift-ci Bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Sep 30, 2025
@ocpdocs-previewbot
Copy link
Copy Markdown

ocpdocs-previewbot commented Sep 30, 2025

🤖 Tue Apr 21 19:26:29 - Prow CI generated the docs preview:
https://99875--ocpdocs-pr.netlify.app
Complete list of updated preview URLs: artifacts/updated_preview_urls.txt

@openshift-bot
Copy link
Copy Markdown

Issues go stale after 90d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle stale

@openshift-ci openshift-ci Bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Dec 30, 2025
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 30, 2025
@openshift-bot
Copy link
Copy Markdown

Stale issues rot after 30d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle rotten
/remove-lifecycle stale

@openshift-ci openshift-ci Bot added lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Jan 29, 2026
@jlebon
Copy link
Copy Markdown
Member Author

jlebon commented Feb 11, 2026

This is still relevant.

/remove-lifecycle rotten

@openshift-ci openshift-ci Bot removed the lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. label Feb 11, 2026
@jlebon
Copy link
Copy Markdown
Member Author

jlebon commented Feb 11, 2026

cc @jldohmann

@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 11, 2026
Comment thread modules/installation-disk-partitioning-upi-templates.adoc Outdated
Comment thread modules/installation-disk-partitioning.adoc Outdated
Copy link
Copy Markdown
Contributor

@jldohmann jldohmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jlebon we have updated guidance for callouts as we prepare for migrating to DITA, so i've added some things to show what that looks like. overall LGTM 😄

Comment thread modules/installation-disk-partitioning-upi-templates.adoc Outdated
Comment thread modules/installation-disk-partitioning-upi-templates.adoc Outdated
Comment thread modules/installation-disk-partitioning-upi-templates.adoc Outdated
Comment thread modules/installation-disk-partitioning-upi-templates.adoc Outdated
Comment thread modules/installation-disk-partitioning-upi-templates.adoc Outdated
Comment thread modules/installation-disk-partitioning.adoc Outdated
Comment thread modules/installation-disk-partitioning.adoc Outdated
Comment thread modules/installation-disk-partitioning.adoc Outdated
Comment thread modules/installation-disk-partitioning.adoc Outdated
Comment thread modules/installation-disk-partitioning.adoc Outdated
@jlebon
Copy link
Copy Markdown
Member Author

jlebon commented Mar 17, 2026

Thanks for the review! Updated!

Copy link
Copy Markdown
Contributor

@jldohmann jldohmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cc: @mburke5678 😄 it LGTM, but now that i'm on the ROSA team i think it would benefit from another pair of eyes

@mburke5678
Copy link
Copy Markdown
Contributor

@jlebon Can you please squash the commits?

This has been supported for a while and simplifies things because then
your Ignition config is no longer specific to a device name.
@jlebon
Copy link
Copy Markdown
Member Author

jlebon commented Apr 21, 2026

Done!

@jlebon jlebon force-pushed the pr/var-partition branch from c36e886 to f6aae71 Compare April 21, 2026 19:16
@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Apr 21, 2026

@jlebon: all tests passed!

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants