8000 chore: temporarily deactivate test for 920390 by theseion · Pull Request #4027 · coreruleset/coreruleset · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

chore: temporarily deactivate test for 920390 #4027

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 2, 2025

Conversation

theseion
Copy link
Contributor
@theseion theseion commented Mar 2, 2025

We currently override the test result for 920390 to expect an error because the httpd image segfaults on that test. The new images we are building no longer segfault, making the test fail. To resolve the situation, we'll disable the test until the new images have been released (the images are verified using the CRS test setup, which currently fails because the test does not fail).

@theseion theseion requested a review from fzipi March 2, 2025 16:02
We currently override the test result for 920390 to expect an error
because the httpd image segfaults on that test. The new images we are
building no longer segfault, making the test fail. To resolve the
situation, we'll disable the test until the new images have been
released (the images are verified using the CRS test setup, which
currently fails because the test _does not fail_).
@theseion theseion force-pushed the temporarily-deactivate-920390 branch from 2385270 to 37d2577 Compare March 2, 2025 16:02
@fzipi fzipi added the release:ignore Ignore for changelog release label Mar 2, 2025
@theseion theseion added this pull request to the merge queue Mar 2, 2025
Merged via the queue into coreruleset:main with commit 02a2df1 Mar 2, 2025
5 checks passed
@theseion theseion deleted the temporarily-deactivate-920390 branch March 2, 2025 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:ignore Ignore for changelog release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0