[CASSANDRASC-131] Fix SidecarLoadBalancingPolicy unexpectedly removing local node and improve CI stability

• Type: Task
• Status: Resolved
• Priority: Normal
• Resolution: Fixed
• Fix Version/s: 1.0
• Component/s: Rest API
• Labels: pull-request-available
• Change Category: Quality Assurance
• Complexity: Normal

Source Control Link

https://github.com/apache/cassandra-sidecar/commit/7ecef7c3c2e2137d4cb045ccf429703cacf96370

Test and Documentation Plan

ci; integration test

• Assignee: Yifan Cai
• Reporter: Yifan Cai

Reviewers

Francisco Guerrero

• Created: 13/May/24 18:50
• Updated: 16/May/24 04:18
• Resolved: 16/May/24 04:18

Description

Some integration test alters the System properties in order to start operations like replacement. Due to the test specific setup, there is a race condition on rolling back the system properties and causing pollution for the next test run and failure.

The patch is to have a common solution for integration test that records the initial system properties and restores on completion.

Issue Links

• links to: GitHub Pull Request #122