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

Check in initial e2e tests for cloud operator. #447

Open
wants to merge 1 commit into
base: poc-deploy-operator
Choose a base branch
from

Conversation

jmcarp
Copy link

@jmcarp jmcarp commented Jan 17, 2025

Add a test to deploy the operator, then a single-node crdb cluster, and test basic functionality.

@jmcarp jmcarp requested a review from pritesh-lahoti January 17, 2025 18:05
@jmcarp jmcarp force-pushed the jmcarp/poc-deploy-operator-e2e branch 3 times, most recently from 9990a05 to 304c916 Compare January 17, 2025 20:42
Add an e2e test that installs the operator, then a crdbcluster, and
makes assertions about basic crdb functionality.
@jmcarp jmcarp force-pushed the jmcarp/poc-deploy-operator-e2e branch from 304c916 to fcb5c37 Compare January 17, 2025 21:56
Copy link
Contributor

@pritesh-lahoti pritesh-lahoti left a comment

Choose a reason for hiding this comment

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

Pre-approving


if rotate {
t.Log("Verifying the existing data in the database after certificate rotation")
}

// Create database only if we are testing crdb install
if !rotate {
if _, err := db.Exec("CREATE DATABASE test_db"); err != nil {
if _, err := db.Exec("CREATE DATABASE IF NOT EXISTS testdb"); err != nil {
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
if _, err := db.Exec("CREATE DATABASE IF NOT EXISTS testdb"); err != nil {
if _, err := db.Exec("CREATE DATABASE IF NOT EXISTS test_db"); err != nil {

Looks like this is causing the test to fail on CI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants