fix(rules): rule archiving and re-activation bugfixes (backport #518)

[mergify[bot]]

Welcome to Cryostat3! 👋

Before contributing, make sure you have:

• [x] Read the contributing guidelines
• [x] Linked a relevant issue which this PR resolves
• [x] Linked any other relevant issues, PR's, or documentation, if any
• [x] Resolved all conflicts, if any
• [x] Rebased your branch PR on top of the latest upstream main branch
• [x] Attached at least one of the following labels to the PR: [chore, ci, docs, feat, fix, test]
• [x] Signed all commits using a GPG signature

To recreate commits with GPG signature git fetch upstream && git rebase --force --gpg-sign upstream/main

---

Fixes: #517

Description of the change:

Fixes up some issues that arose due to previous refactorings, where collections were improperly modified during traversal, or methods went from blocking operations to returning Unis which went ignored, or active recordings were improperly left running, or duplicate target definitions were not de-duplicated by JVM ID and thus were acted on multiple times.

How to manually test:

1. Check out and build PR
2. ./smoketest.bash -Ot
3. Create an automated rule using the JSON below
4. Verify that targets have the active recording created and started
5. Verify that rule archives copies, then prunes extras later
6. Disable rule with clean option checked and ensure recordings are stopped
7. Re-enable rule and ensure recordings are restarted

{
  "id": 1,
  "name": "test",
  "description": "",
  "matchExpression": "target.connectUrl.startsWith('service') && target.jvmId != null",
  "eventSpecifier": "template=Continuous,type=TARGET",
  "archivalPeriodSeconds": 20,
  "initialDelaySeconds": 0,
  "preservedArchives": 2,
  "maxAgeSeconds": 0,
  "maxSizeBytes": 26214400,
  "enabled": true
}
```<hr>This is an automatic backport of pull request #518 done by [Mergify](https://mergify.com).

[andrewazores]

/build_test

[github-actions[bot]]

Workflow started at 6/13/2024, 5:00:27 PM. View Actions Run.

[github-actions[bot]]

CI build and push: All tests pass ✅ (JDK17) https://github.com/cryostatio/cryostat/actions/runs/9506683972