Skip to content
Success

Changes

Summary

  1. chore(pipelines): update cache keys for TiFlash and TiDB pipelines (commit: 34d54b8) (details)
Commit 34d54b8061f87c943fcc71921f1b344f219442df by noreply
chore(pipelines): update cache keys for TiFlash and TiDB pipelines (#3740)

This pull request standardizes and updates the cache key naming
conventions across multiple pipeline Groovy files for TiFlash and TiDB
projects. The changes primarily aim to unify cache key prefixes (e.g.,
replacing `tiflash` and `tiflash-next-gen` with `binary` or `ng-binary`)
to improve cache management and consistency across different pipelines
and environments.

**Cache key naming standardization:**

* Updated TiFlash integration and unit test pipelines to use the
`binary` prefix for cache keys instead of `tiflash`, across release
branches `release-8.5`, `release-9.0-beta`, and `latest`. This affects
both build and test cache steps.
[[1]](diffhunk://#diff-68854d1a3b7109cd070fd473690ed2404d1d96c64cb969c04801076f0ec81c4bL61-R64)
[[2]](diffhunk://#diff-68854d1a3b7109cd070fd473690ed2404d1d96c64cb969c04801076f0ec81c4bL378-R378)
[[3]](diffhunk://#diff-68854d1a3b7109cd070fd473690ed2404d1d96c64cb969c04801076f0ec81c4bL419-R420)
[[4]](diffhunk://#diff-b60236e2947bea8efdb59e569ee92c1465ff0bdff4a7ff7a59b6250dee78adaeL61-R64)
[[5]](diffhunk://#diff-b60236e2947bea8efdb59e569ee92c1465ff0bdff4a7ff7a59b6250dee78adaeL379-R379)
[[6]](diffhunk://#diff-b60236e2947bea8efdb59e569ee92c1465ff0bdff4a7ff7a59b6250dee78adaeL420-R421)
[[7]](diffhunk://#diff-3467d5681b1b3accbe5123903088fe06e99fb7c98ee874828dd8117b6c2250dfL60-R63)
[[8]](diffhunk://#diff-9cc399c6082102a34a0a5168de8b6ffb82e28c26b0629fc7994c39178549194fL63-R66)
[[9]](diffhunk://#diff-9cc399c6082102a34a0a5168de8b6ffb82e28c26b0629fc7994c39178549194fL416-R416)
[[10]](diffhunk://#diff-9cc399c6082102a34a0a5168de8b6ffb82e28c26b0629fc7994c39178549194fL457-R458)
[[11]](diffhunk://#diff-6cb9a78c69f40a1202a4b39e6302b0de75973f961a8bd3d4846fb3e1e1bc6bccL62-R65)
[[12]](diffhunk://#diff-6cb9a78c69f40a1202a4b39e6302b0de75973f961a8bd3d4846fb3e1e1bc6bccL390-R392)
[[13]](diffhunk://#diff-df67aa4ba09f7a017b59273e4f1a2aa90be19ea1a9bc568dea51394c6580c15dL60-R63)
[[14]](diffhunk://#diff-df67aa4ba09f7a017b59273e4f1a2aa90be19ea1a9bc568dea51394c6580c15dL351-R353)

* Updated TiFlash "next gen" integration and unit test pipelines to use
the `ng-binary` prefix for cache keys, replacing the previous `tiflash`
or `tiflash-next-gen` prefixes.
[[1]](diffhunk://#diff-b7313096b352b01b9c404b9faa008c3da9d1baaeb221771f74497228291d8dbdL63-R66)
[[2]](diffhunk://#diff-b7313096b352b01b9c404b9faa008c3da9d1baaeb221771f74497228291d8dbdL417-R417)
[[3]](diffhunk://#diff-b7313096b352b01b9c404b9faa008c3da9d1baaeb221771f74497228291d8dbdL458-R459)
[[4]](diffhunk://#diff-7dab3bf65dc1845c06e85b3c38a6ef76e3b051087489afd37acaa662bd2e2572L62-R65)
[[5]](diffhunk://#diff-7dab3bf65dc1845c06e85b3c38a6ef76e3b051087489afd37acaa662bd2e2572L391-R393)

* Updated TiDB integration pipeline to use the `ng-binary` prefix for
cache keys instead of `binary-nextgen`.

**Logging consistency:**

* Updated log statements to reflect the new cache key names, ensuring
that logs accurately describe which cache key is being used or restored.
[[1]](diffhunk://#diff-b7313096b352b01b9c404b9faa008c3da9d1baaeb221771f74497228291d8dbdL63-R66)
[[2]](diffhunk://#diff-b7313096b352b01b9c404b9faa008c3da9d1baaeb221771f74497228291d8dbdL458-R459)
[[3]](diffhunk://#diff-7dab3bf65dc1845c06e85b3c38a6ef76e3b051087489afd37acaa662bd2e2572L62-R65)
[[4]](diffhunk://#diff-7dab3bf65dc1845c06e85b3c38a6ef76e3b051087489afd37acaa662bd2e2572L391-R393)
[[5]](diffhunk://#diff-9cc399c6082102a34a0a5168de8b6ffb82e28c26b0629fc7994c39178549194fL63-R66)
[[6]](diffhunk://#diff-9cc399c6082102a34a0a5168de8b6ffb82e28c26b0629fc7994c39178549194fL457-R458)
[[7]](diffhunk://#diff-6cb9a78c69f40a1202a4b39e6302b0de75973f961a8bd3d4846fb3e1e1bc6bccL62-R65)
[[8]](diffhunk://#diff-6cb9a78c69f40a1202a4b39e6302b0de75973f961a8bd3d4846fb3e1e1bc6bccL390-R392)
[[9]](diffhunk://#diff-68854d1a3b7109cd070fd473690ed2404d1d96c64cb969c04801076f0ec81c4bL61-R64)
[[10]](diffhunk://#diff-68854d1a3b7109cd070fd473690ed2404d1d96c64cb969c04801076f0ec81c4bL419-R420)
[[11]](diffhunk://#diff-b60236e2947bea8efdb59e569ee92c1465ff0bdff4a7ff7a59b6250dee78adaeL61-R64)
[[12]](diffhunk://#diff-b60236e2947bea8efdb59e569ee92c1465ff0bdff4a7ff7a59b6250dee78adaeL420-R421)
[[13]](diffhunk://#diff-df67aa4ba09f7a017b59273e4f1a2aa90be19ea1a9bc568dea51394c6580c15dL60-R63)
[[14]](diffhunk://#diff-df67aa4ba09f7a017b59273e4f1a2aa90be19ea1a9bc568dea51394c6580c15dL351-R353)

These changes will help ensure more predictable and maintainable caching
behavior across all relevant CI pipelines.
(commit: 34d54b8)
The file was modifiedpipelines/pingcap/tiflash/latest/pull_integration_next_gen.groovy (diff)
The file was modifiedpipelines/pingcap/tidb/latest/pull_integration_realcluster_test_next_gen/pipeline.groovy (diff)
The file was modifiedpipelines/pingcap/tiflash/latest/pull_integration_test.groovy (diff)
The file was modifiedpipelines/pingcap/tiflash/latest/pull_unit_next_gen.groovy (diff)
The file was modifiedpipelines/pingcap/tiflash/release-9.0-beta/pull_unit_test.groovy (diff)
The file was modifiedpipelines/pingcap/tiflash/release-8.5/pull_unit_test.groovy (diff)
The file was modifiedpipelines/pingcap/tiflash/latest/pull_unit_test.groovy (diff)
The file was modifiedpipelines/pingcap/tiflash/release-9.0-beta/pull_integration_test.groovy (diff)
The file was modifiedpipelines/pingcap/tiflash/release-8.5/pull_integration_test.groovy (diff)