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

awsdms: don't use secure roachprod cluster #119394

Merged

Conversation

Jeremyyang920
Copy link
Collaborator

With a change to MakeClusterSettings, roachprod clusers were set to be in secure mode by default. We don't need this for our test so we will be going back to the default behavior so we can use sslmode None for the dms endpoint settings.

Fixes: #119051
Release note: None

With a change to MakeClusterSettings, roachprod clusers were
set to be in secure mode by default. We don't need this for
our test so we will be going back to the default behavior
so we can use sslmode None for the dms endpoint settings.

Fixes: cockroachdb#119051
Release note: None
@Jeremyyang920 Jeremyyang920 requested a review from a team as a code owner February 20, 2024 17:05
@Jeremyyang920 Jeremyyang920 requested review from DarrylWong and renatolabs and removed request for a team February 20, 2024 17:05
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@Jeremyyang920
Copy link
Collaborator Author

No need to backport as it seems the change has only affected the 24.1 test run on Feb 10 and 17.

@Jeremyyang920
Copy link
Collaborator Author

bors r=ZhouXing19

@DarrylWong
Copy link
Contributor

DarrylWong commented Feb 20, 2024

Ah sorry for not catching this earlier. The weekly tests always slip my mind.

I think you'll want to revert these diffs for awsdms as well?

https://github.com/cockroachdb/cockroach/pull/118504/files#diff-7697c62e401f1b860cd317c4408bd011d4584b856660ca026163371bd72199ffL769-R771

Actually on a deeper look, it looks like the password is ignored anyway. Probably fine as is, up to you if you want to revert the comment for clarity.

@Jeremyyang920
Copy link
Collaborator Author

Ah sorry for not catching this earlier. The weekly tests always slip my mind.

I think you'll want to revert these diffs for awsdms as well?

https://github.com/cockroachdb/cockroach/pull/118504/files#diff-7697c62e401f1b860cd317c4408bd011d4584b856660ca026163371bd72199ffL769-R771

Actually on a deeper look, it looks like the password is ignored anyway. Probably fine as is, up to you if you want to revert the comment for clarity.

@DarrylWong Thanks for the followup. Yea the password is ignored either way and the test passed on my local run.

@craig
Copy link
Contributor

craig bot commented Feb 20, 2024

Build succeeded:

@craig craig bot merged commit 84ca65f into cockroachdb:master Feb 20, 2024
12 of 13 checks passed
@Jeremyyang920 Jeremyyang920 deleted the jyang/fix-secure-roachtest-cluster branch February 20, 2024 20:29
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.

roachtest: awsdms failed
4 participants