Skip to content

fix(commons): prevent global attachment collisions across processes - #49

Merged
baev merged 1 commit into
mainfrom
fix-id-collisions
Sep 24, 2026
Merged

baev merged 1 commit into
mainfrom
fix-id-collisions

Conversation

@baev

@baev baev commented Sep 24, 2026

Copy link
Copy Markdown
Member

Context

Global attachments created without a bound lifecycle now use UUIDv4 identifiers, preventing concurrent test processes from overwriting each other's attachments in a shared results directory. This addresses the remaining attachment ID path missed in #48.

Checklist

@baev baev added the pr:bug Pull requests that fix broken or incorrect behavior. label Sep 24, 2026
@github-actions

Copy link
Copy Markdown

Allure Report Summary

Environments: ubuntu-latest-rust-stable, windows-latest-rust-stable

     Name Duration Stats      New Flaky Retry Report
     Allure Rust – Pull request #49 6m 4s Passed tests⁠ 242 2 0 0 View
     Allure Rust GitHub actions run (2026-09-24T13:57:02.699Z) 6m 4s Passed tests⁠ 242 2 0 0 View
Artifacts used (6)
Name Path
allure-results-smoke-nextest-ubuntu-latest-rust-stable.zip allure-results-smoke-nextest-ubuntu-latest-rust-stable.zip
allure-results-smoke-nextest-windows-latest-rust-stable.zip allure-results-smoke-nextest-windows-latest-rust-stable.zip
allure-results-smoke-ubuntu-latest-rust-stable.zip allure-results-smoke-ubuntu-latest-rust-stable.zip
allure-results-smoke-windows-latest-rust-stable.zip allure-results-smoke-windows-latest-rust-stable.zip
allure-results-ubuntu-latest-rust-stable.zip allure-results-ubuntu-latest-rust-stable.zip
allure-results-windows-latest-rust-stable.zip allure-results-windows-latest-rust-stable.zip

@baev
baev merged commit fcd506d into main Sep 24, 2026
15 checks passed
@baev
baev deleted the fix-id-collisions branch September 24, 2026 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:bug Pull requests that fix broken or incorrect behavior.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants