Skip to content

NO-JIRA: docs/FAQ: link KCS for mixed composefs state#1939

Open
jbtrystram wants to merge 1 commit into
openshift:masterfrom
jbtrystram:faq-composefs-add-kcs
Open

NO-JIRA: docs/FAQ: link KCS for mixed composefs state#1939
jbtrystram wants to merge 1 commit into
openshift:masterfrom
jbtrystram:faq-composefs-add-kcs

Conversation

@jbtrystram
Copy link
Copy Markdown
Contributor

In some cases, after a cluster upgrades, some nodes may have composefs enabled but not others. This is due to a MCO bug [1] that triggers an extra deployment then removes it, which end up with the composefs EROFS image being created on those.
This result in the node using composefs sooner than the other nodes. This inconsistency is not causing any problem and will solve itself at the next upgrade, but we have had the questions asked enough times to link this KCS here.

[1] https://redhat.atlassian.net/browse/OCPBUGS-59958

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label May 27, 2026
@openshift-ci-robot
Copy link
Copy Markdown

@jbtrystram: This pull request explicitly references no jira issue.

Details

In response to this:

In some cases, after a cluster upgrades, some nodes may have composefs enabled but not others. This is due to a MCO bug [1] that triggers an extra deployment then removes it, which end up with the composefs EROFS image being created on those.
This result in the node using composefs sooner than the other nodes. This inconsistency is not causing any problem and will solve itself at the next upgrade, but we have had the questions asked enough times to link this KCS here.

[1] https://redhat.atlassian.net/browse/OCPBUGS-59958

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci Bot requested review from c4rt0 and cverna May 27, 2026 10:13
@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 27, 2026
@cverna
Copy link
Copy Markdown
Contributor

cverna commented May 27, 2026

/lgtm

Comment thread docs/faq.md Outdated
@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label May 27, 2026
Comment thread docs/faq.md Outdated
Copy link
Copy Markdown
Contributor

@angelcerveraroldan angelcerveraroldan left a comment

Choose a reason for hiding this comment

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

Other than the small nits above, LGTM

In some cases, after a cluster upgrades, some nodes may have composefs
enabled but not others. This is due to a MCO bug [1] that triggers
an extra deployment then removes it, which end up with the composefs
EROFS image being created on those.
This result in the node using composefs sooner than the other nodes.
This inconsistency is not causing any problem and will solve itself
at the next upgrade, but we have had the questions asked enough times
to link this KCS here.

[1] https://redhat.atlassian.net/browse/OCPBUGS-59958
@jbtrystram jbtrystram force-pushed the faq-composefs-add-kcs branch from 47fa7ef to d548b24 Compare May 27, 2026 10:26
@openshift-ci openshift-ci Bot removed the lgtm Indicates that a PR is ready to be merged. label May 27, 2026
Copy link
Copy Markdown
Contributor

@angelcerveraroldan angelcerveraroldan left a comment

Choose a reason for hiding this comment

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

/lgtm, thanks!

Copy link
Copy Markdown
Contributor

@c4rt0 c4rt0 left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label May 27, 2026
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 27, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: angelcerveraroldan, c4rt0, cverna, jbtrystram

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [angelcerveraroldan,c4rt0,cverna,jbtrystram]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Copy link
Copy Markdown
Member

@Rolv-Apneseth Rolv-Apneseth left a comment

Choose a reason for hiding this comment

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

Just minor wording suggestion, but LGTM

Comment thread docs/faq.md
With composefs, `/` is backed by `/sysroot` and any disk monitoring tools should be updated to watch the `/sysroot` mountpoint, which is the real physical root.

In some cases, after an upgrade, some nodes may have composefs enabled but not all of them. This is a known state caused by a bug in the MCO that
cause some nodes to apply the update twice, causing them to get composefs enabled sooner than other. This has no impact and will resolve itself
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
cause some nodes to apply the update twice, causing them to get composefs enabled sooner than other. This has no impact and will resolve itself
causes some nodes to apply the update twice, causing them to get composefs enabled sooner than others. This has no impact and will resolve itself

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 27, 2026

@jbtrystram: 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.

@travier
Copy link
Copy Markdown
Member

travier commented May 27, 2026

Hum, https://access.redhat.com/solutions/7139041 is marked as internal only. Maybe we should open it up.
To get the nits fixed:
/hold
For the CI
/verified by @travier

@openshift-ci openshift-ci Bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 27, 2026
@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label May 27, 2026
@openshift-ci-robot
Copy link
Copy Markdown

@travier: This PR has been marked as verified by @travier.

Details

In response to this:

Hum, https://access.redhat.com/solutions/7139041 is marked as internal only. Maybe we should open it up.
To get the nits fixed:
/hold
For the CI
/verified by @travier

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 openshift-eng/jira-lifecycle-plugin repository.

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

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. verified Signifies that the PR passed pre-merge verification criteria

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants