Skip to content

CI: Build script should not overwrite job timer set by praktika#89390

Merged
maxknv merged 2 commits intomasterfrom
ci_fix_build_time
Nov 5, 2025
Merged

CI: Build script should not overwrite job timer set by praktika#89390
maxknv merged 2 commits intomasterfrom
ci_fix_build_time

Conversation

@maxknv
Copy link
Copy Markdown
Member

@maxknv maxknv commented Nov 3, 2025

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Details

The build script sets its own job timer, overriding the timer set by Praktika and thus dropping the time used by the ci infrastructure (pulling the build image). This change removes the job's internal timer, so the entire job execution time is tracked in CI report and in CIDB

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh Bot commented Nov 3, 2025

Workflow [PR], commit [03ed516]

Summary:

@clickhouse-gh clickhouse-gh Bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Nov 3, 2025
@maxknv maxknv changed the title CI: Build script should not overwrite timers set by praktika CI: Build script should not overwrite job timer set by praktika Nov 3, 2025
@antaljanosbenjamin antaljanosbenjamin self-assigned this Nov 3, 2025
@maxknv maxknv requested a review from leshikus November 3, 2025 23:14
@maxknv maxknv added this pull request to the merge queue Nov 5, 2025
Merged via the queue into master with commit 2f60a79 Nov 5, 2025
124 checks passed
@maxknv maxknv deleted the ci_fix_build_time branch November 5, 2025 13:52
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-synced-to-cloud The PR is synced to the cloud repo label Nov 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants