Skip to content

CI: Respect submodules updates in ci cache digest#76124

Merged
maxknv merged 1 commit intomasterfrom
ci_account_submodules_in_build_job_digest
Feb 14, 2025
Merged

CI: Respect submodules updates in ci cache digest#76124
maxknv merged 1 commit intomasterfrom
ci_account_submodules_in_build_job_digest

Conversation

@maxknv
Copy link
Copy Markdown
Member

@maxknv maxknv commented Feb 14, 2025

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Respect submodule updates in the build job digest so that jobs are not unfairly skipped if only a submodule is updated.

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@maxknv maxknv changed the title CI: Account submodules state in ci cache digest CI: Respect submodules state in ci cache digest Feb 14, 2025
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Feb 14, 2025

Workflow [PR], commit [7aaf919]

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Feb 14, 2025
@maxknv maxknv force-pushed the ci_account_submodules_in_build_job_digest branch 3 times, most recently from 779cf6f to b0603f6 Compare February 14, 2025 10:32
@maxknv maxknv force-pushed the ci_account_submodules_in_build_job_digest branch from b0603f6 to 7aaf919 Compare February 14, 2025 12:26
@maxknv maxknv added this pull request to the merge queue Feb 14, 2025
@maxknv maxknv changed the title CI: Respect submodules state in ci cache digest CI: Respect submodules updates in ci cache digest Feb 14, 2025
Merged via the queue into master with commit f9b70ce Feb 14, 2025
38 of 91 checks passed
@maxknv maxknv deleted the ci_account_submodules_in_build_job_digest branch February 14, 2025 13:19
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-synced-to-cloud The PR is synced to the cloud repo label Feb 14, 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.

2 participants