Skip to content

Prefix partition_id to Ci::Build#token

Max Orefice requested to merge morefice/prefix-partition-id-to-ci-build-token into master

Ref: #382033 (closed)

What does this MR do and why?

This MR adds partition_id as prefix to Ci::Build#token.

It implements the proposal discussed in #360148 (comment 1067650469) and documented in our blueprint.

# Before
[6] pry(main)> build.token
=> "y44AchDSZB-6QUbiK5Eb"

# After encoded in hex
[9] pry(main)> build.token
=> "64_-SCXvoU_4ZUs6hu3x1ce"

This is not the same strategy we used when rotating tokens for project & group runner token, just leveraging format_token for our use case.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Max Orefice

Merge request reports