Skip to content

Comments

Introduce a new hook-powered chaos test configuration#666

Merged
jviotti merged 1 commit intomainfrom
chaos-config
Feb 25, 2026
Merged

Introduce a new hook-powered chaos test configuration#666
jviotti merged 1 commit intomainfrom
chaos-config

Conversation

@jviotti
Copy link
Member

@jviotti jviotti commented Feb 24, 2026

Signed-off-by: Juan Cruz Viotti jv@jviotti.com

@jviotti jviotti force-pushed the chaos-config branch 8 times, most recently from bb8329a to 6bdc053 Compare February 25, 2026 13:06
@jviotti jviotti marked this pull request as ready for review February 25, 2026 13:09
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 8 files

@augmentcode
Copy link

augmentcode bot commented Feb 25, 2026

🤖 Augment PR Summary

Summary: Adds a new “chaos” sandbox configuration to validate that the server stays up even when indexed output artifacts are corrupted.
Changes:

  • Extends the sandbox GitHub Action + CI workflow to run an additional chaos sandbox pass.
  • Adds one-chaos-{community,enterprise}.json configs (extending the existing HTML configs).
  • Introduces a post-index hook (test/sandbox/postindex.sh) that intentionally corrupts selected output files when configuration=chaos.
  • Adds a new Hurl E2E suite under test/e2e/chaos/ that exercises endpoints and checks the server doesn’t crash.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. 3 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@jviotti jviotti merged commit 6d8f54e into main Feb 25, 2026
8 of 9 checks passed
@jviotti jviotti deleted the chaos-config branch February 25, 2026 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant