Skip to content

use client.New for tests to get consistent reads [K8SSAND-144]#43

Open
jsanda wants to merge 1 commit intok8ssandra:masterfrom
jsanda:use-client-new
Open

use client.New for tests to get consistent reads [K8SSAND-144]#43
jsanda wants to merge 1 commit intok8ssandra:masterfrom
jsanda:use-client-new

Conversation

@jsanda
Copy link
Copy Markdown
Contributor

@jsanda jsanda commented Apr 8, 2021

Fixes #42

┆Issue is synchronized with this Jira Bug by Unito

@jsanda jsanda requested a review from burmanm April 8, 2021 03:07
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 8, 2021

SonarCloud Quality Gate failed.

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
8.0% 8.0% Duplication

By("check that the reaper is ready")
waitForReaperCondition(key, func(reaper *api.Reaper) bool {
return reaper.Status.Ready
})
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the commented out code is not in use.. remove.

@jdonenine jdonenine changed the title use client.New for tests to get consistent reads use client.New for tests to get consistent reads [K8SSAND-144] Sep 13, 2021
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.

Do not use the manager client in tests

4 participants