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

github/workflows/integration-tests: extract RELOC_VERSION as an env variable #567

Merged
merged 2 commits into from
Mar 27, 2024

Conversation

tchaikov
Copy link
Contributor

encode RELOC_VERSION in the key of binary, so we can use different binary for testing. there is chance that newer ccm would like to use newer scylla executables for testing.

this series is more a refactory, and it does not change the version of scylla we are using, but in a follow up change, we will use a different RELOC_VERSION for testing newer scylla.

@tchaikov
Copy link
Contributor Author

see also #566 for the similar changes applied to nix.yml .

@tchaikov tchaikov requested review from fruch and denesb March 25, 2024 02:09
Copy link
Contributor

@fruch fruch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fruch
Copy link
Contributor

fruch commented Mar 25, 2024

@tchaikov please check why CI didn't run at all

…variable

so we can customize it if necessary.
also, wire it up with "SCYLLA_VERSION" as another env variable, so it
can be consumed by `test_config.py`.

the same change has been already applied to nix.yml in 51e9d91 and 8666049.

Signed-off-by: Kefu Chai <[email protected]>
…ey of cache

encode RELOC_VERSION in the key of binary, so we can use different
binary for testing. there is chance that newer ccm would like to use
newer scylla executables for testing.

the same change has been applied to nix.yml in 96ab738.

Signed-off-by: Kefu Chai <[email protected]>
@tchaikov tchaikov force-pushed the github-integration-tests-binary-cache branch from 7dfed77 to 3a45d1d Compare March 25, 2024 06:44
@tchaikov
Copy link
Contributor Author

changelog:

@tchaikov
Copy link
Contributor Author

@tchaikov please check why CI didn't run at all

@fruch sorry, it was a regression. should be fixed by #568

@tchaikov
Copy link
Contributor Author

@fruch ping?

@tchaikov
Copy link
Contributor Author

next step, i will extract the setting of scylla version into another yaml file as a dedicated action shared by these two existing action, so we don't need to repeat it in two workflows.

@fruch fruch merged commit e02cfe1 into scylladb:next Mar 27, 2024
3 checks passed
@tchaikov tchaikov deleted the github-integration-tests-binary-cache branch March 27, 2024 06:42
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.

3 participants