Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

control-service/vdk-cli/vdk-heartbeat: set Pipeline ID as op_id for vdk-heartbeat run #654

Merged
merged 4 commits into from
Dec 21, 2021

Conversation

antoniivanov
Copy link
Collaborator

This PR contains 3 commits 2 of them set op id to be same as $CI_PIPELINE_ID

By setting op_id to be the Pipeline ID we can easier track a given run of vdk-heartebaet as op_id is printed in the logs, it is passed to the job logs, the job name also contains the Op ID . So one can easier see which pipeline generated which job and vice-versa

And small change in vdk-heartbeat that make sure do not accidentally create job name with too long name (as op id is included in the job name

Truncate job name if longer than 45 symbols (allowed by VDK Control
Service)

Signed-off-by: Antoni Ivanov <[email protected]>
By setting op_id to be the Pipeline ID we can easier track a given run
of vdk-heartebaet as op_id is printed in the logs, it is passed to the
job logs, the job name also contains the Op ID . So one can easier see
which pipeline generated which job and vice-versa

Signed-off-by: Antoni Ivanov <[email protected]>
By setting op_id to be the Pipeline ID we can easier track a given run
of vdk-heartebaet as op_id is printed in the logs, it is passed to the
job logs, the job name also contains the Op ID . So one can easier see
which pipeline generated which job and vice-versa

Signed-off-by: Antoni Ivanov <[email protected]>
@antoniivanov antoniivanov enabled auto-merge (rebase) December 21, 2021 13:47
Copy link
Contributor

@doks5 doks5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@antoniivanov antoniivanov merged commit ef51d3c into main Dec 21, 2021
@antoniivanov antoniivanov deleted the person/aivanov/vdk-heartbeat branch December 21, 2021 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants