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

CI: Use CrateDB nightly also for regular testing #624

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

amotl
Copy link
Member

@amotl amotl commented Jun 17, 2024

About

CrateDB version was fixed to 5.4.5. This patch intends to use latest nightly.

Thoughts

latest will probably not work, because the test layer used here only understands specific version numbers, and nightly?

@matriv
Copy link
Contributor

matriv commented Jun 17, 2024

Do you have any concern to test on CI with nightly? It could potentially help us catch issues of incompatibilities between CrateDB and python, before a release.

@amotl
Copy link
Member Author

amotl commented Jun 17, 2024

Thanks. Let's go for nightly right away, also for regular testing. CrateDB Nightly is used for crate-python nightly CI jobs anyway, so let's synchronize this.

@amotl amotl changed the title CI: Use CrateDB latest for regular testing CI: Use CrateDB nightly also for regular testing Jun 17, 2024
@amotl amotl requested a review from matriv June 17, 2024 19:06
@amotl amotl marked this pull request as ready for review June 17, 2024 19:06
@amotl
Copy link
Member Author

amotl commented Jun 17, 2024

Dialogue with self

Q: 404 Not Found @ https://cdn.crate.io/downloads/releases/cratedb/x64_mac/crate-latest.tar.gz -- what's that?
A: Ah right, that's why we pinned specific versions. 💡

Explanation

There are no nightly builds for x64_mac.

Conclusion

Putting this back into draft mode for now, and updating manually instead.

@amotl amotl marked this pull request as draft June 17, 2024 19:12
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