Skip to content
This repository has been archived by the owner on Jan 3, 2024. It is now read-only.

CI: Switch to S3 Test Excuses File #221

Merged
merged 3 commits into from
Oct 12, 2023
Merged

Conversation

irq0
Copy link
Member

@irq0 irq0 commented Oct 4, 2023

Switches S3 Test CI step to the new s3tr excuses.csv feature.

Requires s3tr update: https://github.com/aquarist-labs/s3gw/pull/700

Fixes: https://github.com/aquarist-labs/s3gw/issues/665

Checklist

  • Tracker (select at least one)
    • References tracker ticket
    • Very recent bug; references commit where it was introduced
    • New feature (ticket optional)
    • Doc update (no ticket needed)
    • Code cleanup (no ticket needed)
  • Documentation (select at least one)
    • Updates relevant documentation
    • No doc update is appropriate
  • Tests (select at least one)

Marcel Lauhoff added 2 commits October 4, 2023 17:13
Add CSV file with S3 Test name, Issue URL and short description why
this test fails.

Signed-off-by: Marcel Lauhoff <[email protected]>
Use new s3tr excuse file feature instead of "new-successes" "new-failures"

Signed-off-by: Marcel Lauhoff <[email protected]>
@irq0 irq0 self-assigned this Oct 5, 2023
@irq0 irq0 added the area/CI Continuous Integration label Oct 5, 2023
@irq0
Copy link
Member Author

irq0 commented Oct 10, 2023

Test s3gw can't go green until https://github.com/aquarist-labs/s3gw/pull/700 is merged. Merging that will fail every PR without this PR. Suggest we get both in together once this gets a +1

@irq0 irq0 requested a review from jecluis October 10, 2023 13:31
@irq0 irq0 mentioned this pull request Oct 12, 2023
3 tasks
jecluis
jecluis previously approved these changes Oct 12, 2023
Signed-off-by: Marcel Lauhoff <[email protected]>
@irq0 irq0 merged commit 3a2771d into aquarist-labs:s3gw Oct 12, 2023
5 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/CI Continuous Integration
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

ci: s3 tests - switch build fail decision from "shall pass" list to "ok to fail" list
2 participants