[PR #332] fix(oss): use if_not_exists for put-if-absent on S3 backends #354

New issue

Open

opened 2026-10-29 14:22:33 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-29 14:22:33 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/kvcache-ai/AgentENV/pull/332
Author: @Bnjoroge1
Created: 10/1/2026
Status: 🔄 Open

Base: main ← Head: fix/oss-if-none-match

📝 Commits (1)

f2331b7 fix(oss): use if_not_exists for put-if-absent on S3 backends

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 src/snapshot/repository/backends/oss/client.rs (+1 -1)

📄 Description

opendal 0.55's S3 service declares write_with_if_not_exists but not write_with_if_none_match, so put_bytes_conditionally(etag=None) is rejected client-side and every conditional create against an S3-compatible store (RustFS, MinIO, R2) fails with HTTP 500 claim volume alias / backend error. Verified end-to-end against a local RustFS backend: volume create previously 500s, now succeeds. if_not_exists(true) is the same put-if-absent semantics the S3 service already supports.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/kvcache-ai/AgentENV/pull/332 **Author:** [@Bnjoroge1](https://github.com/Bnjoroge1) **Created:** 10/1/2026 **Status:** 🔄 Open **Base:** `main` ← **Head:** `fix/oss-if-none-match` --- ### 📝 Commits (1) - [`f2331b7`](https://github.com/kvcache-ai/AgentENV/commit/f2331b721717286c5273354bf151735d3fac39df) fix(oss): use if_not_exists for put-if-absent on S3 backends ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `src/snapshot/repository/backends/oss/client.rs` (+1 -1) </details> ### 📄 Description opendal 0.55's S3 service declares `write_with_if_not_exists` but not `write_with_if_none_match`, so `put_bytes_conditionally(etag=None)` is rejected client-side and every conditional create against an S3-compatible store (RustFS, MinIO, R2) fails with HTTP 500 `claim volume alias / backend error`. Verified end-to-end against a local RustFS backend: volume create previously 500s, now succeeds. `if_not_exists(true)` is the same put-if-absent semantics the S3 service already supports. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi added the

pull-request

label

2026-10-29 14:22:33 +00:00

bsakaguchi referenced this issue

2026-10-29 14:22:45 +00:00

[PR #354] [MERGED] ci: request medium reasoning effort in OpenCodeReview #373