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 e2e tests, change from wget to curl #564

Merged
merged 10 commits into from
Aug 31, 2023

Conversation

burmanm
Copy link
Contributor

@burmanm burmanm commented Aug 25, 2023

What this PR does:
This PR changes the default Kubernetes -> management-api communication from using wget to using curl. This allows better compatibility between different images.

  • Fixes scale_down_unbalanced_test by setting all correct parameters to it (M_SERVER_TYPE & M_SERVER_VERSION).
  • Fixes DSE tests to use -Ddse prefix if -Dcassandra is used instead for replication factors.

Which issue(s) this PR fixes:
Fixes #565

Checklist

  • Changes manually tested
  • Automated Tests added/updated
  • Documentation added/updated
  • CHANGELOG.md updated (not required for documentation PRs)
  • CLA Signed: DataStax CLA

@burmanm burmanm requested a review from a team as a code owner August 25, 2023 15:51
@burmanm burmanm changed the title Fix e2e tests (DSE test all and Cassandra downscale unbalanced) Fix e2e tests, change from wget to curl Aug 29, 2023
@burmanm
Copy link
Contributor Author

burmanm commented Aug 30, 2023

decommission dc is fine (well, as fine as it's always been, it's a bit flaky), it's just that our e2e tests can't be rerun on the next day at this point:

Run kind load docker-image --name=kind k8ssandra/cass-operator:v1.17.0-dev.2e60096-20230830
ERROR: image: "k8ssandra/cass-operator:v1.17.0-dev.2e60096-20230830" not present locally

I probably need to ditch the -date at the end of the version number for GHA runs.

@burmanm burmanm merged commit ffb6a8a into k8ssandra:master Aug 31, 2023
37 of 38 checks passed
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.

Replace wget with curl
2 participants