From: "Fabian Grünbichler" <f.gruenbichler@proxmox.com>
To: pbs-devel@lists.proxmox.com, Christian Ebner <c.ebner@proxmox.com>
Subject: [pbs-devel] partially-applied: [PATCH proxmox-backup v2 00/12] s3 store: fix issues with chunk s3 backend upload and cache eviction
Date: Wed, 15 Oct 2025 10:35:58 +0200 [thread overview]
Message-ID: <176051731270.564502.7721346566375625758.b4-ty@proxmox.com> (raw)
In-Reply-To: <20251008152125.849216-1-c.ebner@proxmox.com>
On Wed, 08 Oct 2025 17:21:13 +0200, Christian Ebner wrote:
> These patches fix 2 issues with the current s3 backend implementation
> and reduce code duplication.
>
> Patch 1 to 3 rework the garbage collection, deduplicating the common atime
> check and garbage collection status update logic and marks the chunk removal
> function signature as unsafe, as this requires specific pre-conditions.
>
> [...]
Applied the first 8 patches, thanks:
[01/12] datastore: gc: inline single callsite method
commit: a9e25866aa8a91e21e9d7cff8ebb3a1d933484e4
[02/12] gc: chunk store: rework atime check and gc status into common helper
commit: 4dc6f5dcf48fb89b401abc6c11f7e4a64c955fff
[03/12] chunk store: add unsafe signature to cache remove method
commit: 6951280989d009ac3fefd5023ebf835a1deb1f49
[04/12] local store cache: replace evicted cache chunks instead of truncate
commit: 4ed3710556defb71245055c35e0e577ec895b6bb
[05/12] local store cache: serve response fetched from s3 backend
commit: 9f9792612a56e8663c57a44071a4ebb1dc1e501f
[06/12] local store cache: refactor fetch and insert of chunks for s3 backend
commit: 0ba88fa2b314f5b6280a5cda01e1c8395581be88
[07/12] local store cache: rework access cache fetching and insert logic
commit: 94657440a800864550ee924f7bf5db51d3186041
[08/12] local store cache: drop obsolete cacher implementation
commit: d37dbabed6f127f5a14f203f2214fbf22f32a229
Left these for for further rework:
[09/12] chunk store: refactor method for chunk insertion
[10/12] api: chunk upload: fix race between chunk backend upload and insert
[11/12] api: chunk upload: fix race with garbage collection for no-cache on s3
[12/12] pull: guard chunk upload and only insert into cache after upload
Best regards,
--
Fabian Grünbichler <f.gruenbichler@proxmox.com>
_______________________________________________
pbs-devel mailing list
pbs-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel
next prev parent reply other threads:[~2025-10-15 8:36 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-10-08 15:21 [pbs-devel] " Christian Ebner
2025-10-08 15:21 ` [pbs-devel] [PATCH proxmox-backup v2 01/12] datastore: gc: inline single callsite method Christian Ebner
2025-10-08 15:21 ` [pbs-devel] [PATCH proxmox-backup v2 02/12] gc: chunk store: rework atime check and gc status into common helper Christian Ebner
2025-10-08 15:21 ` [pbs-devel] [PATCH proxmox-backup v2 03/12] chunk store: add unsafe signature to cache remove method Christian Ebner
2025-10-08 15:21 ` [pbs-devel] [PATCH proxmox-backup v2 04/12] local store cache: replace evicted cache chunks instead of truncate Christian Ebner
2025-10-08 15:21 ` [pbs-devel] [PATCH proxmox-backup v2 05/12] local store cache: serve response fetched from s3 backend Christian Ebner
2025-10-08 15:21 ` [pbs-devel] [PATCH proxmox-backup v2 06/12] local store cache: refactor fetch and insert of chunks for " Christian Ebner
2025-10-08 15:21 ` [pbs-devel] [PATCH proxmox-backup v2 07/12] local store cache: rework access cache fetching and insert logic Christian Ebner
2025-10-08 15:21 ` [pbs-devel] [PATCH proxmox-backup v2 08/12] local store cache: drop obsolete cacher implementation Christian Ebner
2025-10-08 15:21 ` [pbs-devel] [PATCH proxmox-backup v2 09/12] chunk store: refactor method for chunk insertion Christian Ebner
2025-10-08 15:21 ` [pbs-devel] [PATCH proxmox-backup v2 10/12] api: chunk upload: fix race between chunk backend upload and insert Christian Ebner
2025-10-14 13:46 ` Fabian Grünbichler
2025-10-14 14:57 ` Christian Ebner
2025-10-08 15:21 ` [pbs-devel] [PATCH proxmox-backup v2 11/12] api: chunk upload: fix race with garbage collection for no-cache on s3 Christian Ebner
2025-10-08 15:21 ` [pbs-devel] [PATCH proxmox-backup v2 12/12] pull: guard chunk upload and only insert into cache after upload Christian Ebner
2025-10-15 8:35 ` Fabian Grünbichler [this message]
2025-10-15 16:47 ` [pbs-devel] superseded: [PATCH proxmox-backup v2 00/12] s3 store: fix issues with chunk s3 backend upload and cache eviction Christian Ebner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=176051731270.564502.7721346566375625758.b4-ty@proxmox.com \
--to=f.gruenbichler@proxmox.com \
--cc=c.ebner@proxmox.com \
--cc=pbs-devel@lists.proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox