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

Cassandra_Scaler improvement - Fixed port error #4151

Closed
wants to merge 4 commits into from

Conversation

ithesadson
Copy link
Contributor

Signed-off-by: ithesadson thesadson@gmail.com

Since the change I made before was a breaking change, I just fix the mistake and send a pr again.
Issue : #4110
Previous PR: #4079

If the clusterIPAddress contains port information, checking if the port information is entered in the clusterIPAddress will cause some errors.

Error example in current code: User enters the following values.
clusterIPAddress: "https://cassandra.test/"
port: "9042"
->Expected clusterIPAddress : https://cassandra.test:9042/
The actual clusterIPAddress: https://cassandra.test/

Checklist

  • Commits are signed with Developer Certificate of Origin (DCO - learn more)

Signed-off-by: ithesadson <thesadson@gmail.com>
@ithesadson ithesadson requested a review from a team as a code owner January 21, 2023 13:01
@ithesadson ithesadson changed the title Fixed Port Error Cassandra_Scaler improvement - Fixed port error Jan 21, 2023
Signed-off-by: ithesadson <thesadson@gmail.com>
Signed-off-by: Emircan Agac <thesadson@gmail.com>
@ithesadson ithesadson closed this Jan 28, 2023
@ithesadson ithesadson deleted the cassandra_scaler_fixport branch January 28, 2023 11:02
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.

1 participant