-
Notifications
You must be signed in to change notification settings - Fork 20
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
chore(recordings): check target's jvmId for recording events #1126
chore(recordings): check target's jvmId for recording events #1126
Conversation
This PR/issue depends on:
|
ade8102
to
d36c54d
Compare
4679516
to
bef3a97
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Topology view needs updates too.
if (!isOwned || (extractedUrl && extractedUrl === targetNode.target.connectUrl)) { |
I would recommend doing a global search for these target-owned resources' notification category and ensure all are also or just using jvmId for checking.
422bc0d
to
e9075d9
Compare
d3c2b83
to
c6c55b5
Compare
had to re-add the connectUrl check in notification messages because the integration tests were failing without them |
/build_test |
Test image available:
|
/build_test |
Test image available:
|
68e9eb4
to
d36b002
Compare
d36b002
to
6d7bdc8
Compare
(cherry picked from commit c75c36a)
…1151) (cherry picked from commit c75c36a) Co-authored-by: Ming Yu Wang <[email protected]>
Welcome to Cryostat! 👋
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
Depends on https://github.com/cryostatio/cryostat/pull/1689
Related to https://github.com/cryostatio/cryostat/issues/1452