Closed andrewazores closed 3 weeks ago
/build_test
Workflow started at 6/10/2024, 4:59:41 PM. View Actions Run.
No GraphQL schema changes detected.
No OpenAPI schema changes detected.
CI build and push: All tests pass ✅ (JDK17) https://github.com/cryostatio/cryostat3/actions/runs/9455445258
Welcome to Cryostat3! 👋
Before contributing, make sure you have:
main
branch[chore, ci, docs, feat, fix, test]
To recreate commits with GPG signature
git fetch upstream && git rebase --force --gpg-sign upstream/main
Fixes #511
The most important change is in
MatchExpressions.java
. The backend should not trust the client to provide the list of wholeTarget
objects to evalute match expressions against. This endpoint should really just take a list of target database IDs that the client wants to test with the provided match expression, but it is late to make such a change now. With this patch the server picks only theconnectUrl
from each of the client's provided targets, then looks up each actual Target instance from the database that has thatconnectUrl
, and tests the provided match expression against this reconstructed list. This ensures that the client is not lying about any properties of the targets, and also ensures that the server doesn't skip evaluating any properties of the target that the client did not include with the request (such as the labels and annotations, which are the root of the associated bug).How to manually test:
./smoketest.bash -Ot
'PORT' in target.annotations.cryostat