Skip to content
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

Revert to /etc/cassandra-reaper #1139

Merged

Conversation

jeffbanks
Copy link
Contributor

Adjustment to revert changes made in cassandra-reaper PR#1113 to support this k8ssandra PR in k8ssandra for mount path changes.

Moving change of /etc/reaper to /etc/cassandra-reaper to reduce complexity of those deploying Reaper outside of a container.

@codecov
Copy link

codecov bot commented Nov 9, 2021

Codecov Report

Merging #1139 (3967c73) into master (923f02e) will increase coverage by 0.07%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1139      +/-   ##
==========================================
+ Coverage   80.26%   80.34%   +0.07%     
==========================================
  Files         109      109              
  Lines        8910     8910              
  Branches      924      924              
==========================================
+ Hits         7152     7159       +7     
+ Misses       1238     1230       -8     
- Partials      520      521       +1     
Impacted Files Coverage Δ
.../java/io/cassandrareaper/service/RepairRunner.java 83.72% <0.00%> (-0.52%) ⬇️
...java/io/cassandrareaper/service/SegmentRunner.java 83.14% <0.00%> (-0.47%) ⬇️
...java/io/cassandrareaper/service/RepairManager.java 78.57% <0.00%> (-0.45%) ⬇️
...assandrareaper/ReaperApplicationConfiguration.java 71.27% <0.00%> (ø)
...a/io/cassandrareaper/storage/CassandraStorage.java 86.11% <0.00%> (+0.17%) ⬆️
...o/cassandrareaper/resources/RepairRunResource.java 73.27% <0.00%> (+0.60%) ⬆️
...rc/main/java/io/cassandrareaper/service/Heart.java 85.00% <0.00%> (+0.71%) ⬆️
...ain/java/io/cassandrareaper/jmx/ClusterFacade.java 83.87% <0.00%> (+1.43%) ⬆️
...io/cassandrareaper/jmx/HostConnectionCounters.java 82.35% <0.00%> (+2.94%) ⬆️
...eaper/resources/DiagEventSubscriptionResource.java 84.61% <0.00%> (+3.84%) ⬆️

@jeffbanks jeffbanks merged commit 5c91ad9 into thelastpickle:master Nov 10, 2021
@jeffbanks jeffbanks deleted the jeffb/revert-etc-path-name branch November 10, 2021 15:05
@jeffbanks jeffbanks restored the jeffb/revert-etc-path-name branch November 10, 2021 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants