Skip to content

Commit

Permalink
Properly set version of the produced binaries
Browse files Browse the repository at this point in the history
  • Loading branch information
Enmk committed Feb 16, 2025
1 parent d6fb6f0 commit d130ef3
Showing 1 changed file with 7 additions and 17 deletions.
24 changes: 7 additions & 17 deletions tests/ci/build_check.py
Original file line number Diff line number Diff line change
Expand Up @@ -262,6 +262,13 @@ def main():
version._flavour = version_type
logging.info("Using version from tag: %s => %s", tag_name, version)

# NOTE(vnemkov): make sure that version is set correctly for the generated binaries
update_version_local(version, version_type)

logging.info(f"Updated local files with version : {version.string} / {version.describe}")

logging.info("Build short name %s", build_name)

release_or_pr, performance_pr = get_release_or_pr(pr_info, version)

s3_path_prefix = "/".join((release_or_pr, pr_info.sha, build_name))
Expand All @@ -279,23 +286,6 @@ def main():
docker_image = get_image_with_version(IMAGES_PATH, IMAGE_NAME, version=pr_info.docker_image_tag)
image_version = docker_image.version

logging.info("Got version from repo %s", version.string)

official_flag = True
# version._flavour = version_type = CLICKHOUSE_STABLE_VERSION_SUFFIX
# TODO (vnemkov): right now we'll use simplified version management:
# only update git hash and explicitly set stable version suffix.
# official_flag = pr_info.number == 0
# version_type = "testing"
# if "release" in pr_info.labels or "release-lts" in pr_info.labels:
# version_type = CLICKHOUSE_STABLE_VERSION_SUFFIX
# official_flag = True
# update_version_local(version, version_type)

logging.info(f"Updated local files with version : {version.string} / {version.describe}")

logging.info("Build short name %s", build_name)

build_output_path = temp_path / build_name
build_output_path.mkdir(parents=True, exist_ok=True)
cargo_cache = CargoCache(
Expand Down

0 comments on commit d130ef3

Please sign in to comment.