Skip to content

Revert "Move fuse-overlayfs to suggests" for all Fedoras#2206

Merged
openshift-merge-bot[bot] merged 1 commit intocontainers:mainfrom
debarshiray:wip/rishi/revert-447945e59a01cb67
Oct 18, 2024
Merged

Revert "Move fuse-overlayfs to suggests" for all Fedoras#2206
openshift-merge-bot[bot] merged 1 commit intocontainers:mainfrom
debarshiray:wip/rishi/revert-447945e59a01cb67

Conversation

@debarshiray
Copy link
Member

Commit 5ad221d ("Revert "Move fuse-overlayfs to suggests" for Fedora 40 and older") restored the dependency on fuse-overlayfs for Fedora 40 and older to not disrupt stable Fedora releases by breaking backwards compatibility with existing containers.

Fedora completely ignores Suggests by default [1]. So, listing anything as Suggests is as good as not mentioning it at all, unless it's intended for tools and users who specifically respect it.

It turns out that there are important use-cases where the Linux kernel's overlay file system doesn't work, and one could really benefit from having fuse-overlayfs(1). A container cannot use an overlayfs when the underlying file system is also an overlayfs, such as on the Fedora Workstation live media, or a network file system.

Therefore, it's worth restoring the dependency on all Fedora releases to cover these use-cases.

As suggested by Giuseppe Scrivano.

This reverts Fedora commit 447945e59a01cb6715ed2a21877d45bf0b91ef67 for all Fedora releases.

[1] https://docs.fedoraproject.org/en-US/packaging-guidelines/WeakDependencies/

Fixes: coreos/fedora-coreos-tracker#1749
Fixes: containers/toolbox#1512
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=2299284

Commit 5ad221d ("Revert \"Move fuse-overlayfs to suggests\" for
Fedora 40 and older") restored the dependency on fuse-overlayfs for
Fedora 40 and older to not disrupt stable Fedora releases by breaking
backwards compatibility with existing containers.

Fedora completely ignores Suggests by default [1].  So, listing anything
as Suggests is as good as not mentioning it at all, unless it's intended
for tools and users who specifically respect it.

It turns out that there are important use-cases where the Linux kernel's
overlay file system doesn't work, and one could really benefit from
having fuse-overlayfs(1).  A container cannot use an overlayfs when the
underlying file system is also an overlayfs, such as on the Fedora
Workstation live media, or a network file system.

Therefore, it's worth restoring the dependency on all Fedora releases to
cover these use-cases.

As suggested by Giuseppe Scrivano.

This reverts Fedora commit 447945e59a01cb6715ed2a21877d45bf0b91ef67 for
all Fedora releases.

[1] https://docs.fedoraproject.org/en-US/packaging-guidelines/WeakDependencies/

Fixes: coreos/fedora-coreos-tracker#1749
Fixes: containers/toolbox#1512
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=2299284

Signed-off-by: Debarshi Ray <rishi@fedoraproject.org>
@debarshiray
Copy link
Member Author

debarshiray commented Oct 17, 2024

/cc @giuseppe

I didn't touch any Fedora derivatives, because I assume that it needs further discussion.

@openshift-ci openshift-ci bot requested a review from giuseppe October 17, 2024 21:37
@debarshiray
Copy link
Member Author

@debarshiray
Copy link
Member Author

@travier
Copy link
Member

travier commented Oct 18, 2024

Thanks for doing this. We'll need a Bugzilla and a freeze exception for F41.

@travier
Copy link
Member

travier commented Oct 18, 2024

Ah we already have both: https://bugzilla.redhat.com/show_bug.cgi?id=2299284

@rhatdan
Copy link
Member

rhatdan commented Oct 18, 2024

/approve
/lgtm

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 18, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: debarshiray, rhatdan, travier

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:

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

@openshift-merge-bot openshift-merge-bot bot merged commit 7e6f2b4 into containers:main Oct 18, 2024
@debarshiray
Copy link
Member Author

Thanks for the quick review, @rhatdan !

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.

Native overlay storage has significant impact on rootless containers creation time Old containers don't start when fuse-overlayfs(1) is missing

3 participants