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

fix: functional tests #284

Merged
merged 6 commits into from
Apr 16, 2024
Merged

fix: functional tests #284

merged 6 commits into from
Apr 16, 2024

Conversation

filipelautert
Copy link
Contributor

No description provided.

@filipelautert
Copy link
Contributor Author

Hey @maximevw - I'm trying to bring back integration tests that stopped working with the new driver. First thing I found was that url was incorrect - that's fixed now. But now I'm facing the "NoNodeAvailableException: No node was available to execute the query" as in https://github.com/liquibase/liquibase-cassandra/actions/runs/8696263897/job/23849274875?pr=284#step:11:2268 . Already trying set the datacenter url parameter but nothing changed... any ideas?

@maximevw
Copy link
Contributor

Hello @filipelautert
If I'm right, you use cassandra:3 or 4 as Docker image to run tests. So, I think the localdatacenter is effectively missing. I don't know the value(s) you tested and I don't remember the default value configured in the image... but you can try to do the same thing I did for the driver's tests: specifying the environment variable CASSANDRA_DC (in your Docker compose file) with a value you can re-use in the jdbc url. More information about the image configuration is available here: https://hub.docker.com/_/cassandra (section "Configuring Cassandra").

@filipelautert filipelautert marked this pull request as ready for review April 16, 2024 12:08
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@filipelautert filipelautert changed the title fix: functional tests that were using previous driver connection url fix: functional tests Apr 16, 2024
@filipelautert filipelautert merged commit fb75873 into main Apr 16, 2024
15 checks passed
@filipelautert filipelautert deleted the DAT-17445 branch April 16, 2024 13:13
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