Skip to content

fix: add U flag to sqlite volume mount for podman 5.x compatibility (PROJQUAY-9329)#259

Merged
harishsurf merged 1 commit intoquay:mirror-registry-2.0-rhel-8from
harishsurf:PROJQUAY-9329-volume-mount
Apr 15, 2026
Merged

fix: add U flag to sqlite volume mount for podman 5.x compatibility (PROJQUAY-9329)#259
harishsurf merged 1 commit intoquay:mirror-registry-2.0-rhel-8from
harishsurf:PROJQUAY-9329-volume-mount

Conversation

@harishsurf
Copy link
Copy Markdown
Contributor

Summary

  • Adds the :U volume mount flag to the sqlite volume in quay.service.j2, which tells podman to recursively chown the volume to match the container's runtime user
  • Fixes upgrade failures (1.3.x → 2.x) on podman 5.x where the sqlite-storage volume ownership resets to root:root after the quay-copy container is removed, causing
    peewee.OperationalError: attempt to write a readonly database
  • Complements the existing fix in PR fix: set sqlite storage ownership for Quay user after pg-to-sqlite migration (PROJQUAY-9799) #243 (which addressed the migration path only via hardcoded podman unshare chown to UID 1001) by handling the upgrade path and supporting containers
    running as any UID

Notes

  • The :U flag is only applied to the sqlite volume, not quay-storage, as recursive chown on a volume with many blobs could delay startup or hit the 3-minute health check timeout
  • Issue does not reproduce on podman 4.9.x, only on podman 5.x

@harishsurf
Copy link
Copy Markdown
Contributor Author

/cherry-pick main

@openshift-cherrypick-robot
Copy link
Copy Markdown
Contributor

@harishsurf: once the present PR merges, I will cherry-pick it on top of main in a new PR and assign it to you.

Details

In response to this:

/cherry-pick main

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.

@harishsurf harishsurf added the ok-to-test Indicates a non-member PR verified by an org member that is safe to test. label Apr 14, 2026
@harishsurf harishsurf merged commit 9508593 into quay:mirror-registry-2.0-rhel-8 Apr 15, 2026
8 checks passed
@openshift-cherrypick-robot
Copy link
Copy Markdown
Contributor

@harishsurf: new pull request created: #261

Details

In response to this:

/cherry-pick main

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.

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

Labels

ok-to-test Indicates a non-member PR verified by an org member that is safe to test.

Development

Successfully merging this pull request may close these issues.

3 participants