Skip to content

Add labels to each Jenkins step#9556

Merged
Mousius merged 4 commits into
apache:mainfrom
driazati:driazati/jenkins_names
Nov 25, 2021
Merged

Add labels to each Jenkins step#9556
Mousius merged 4 commits into
apache:mainfrom
driazati:driazati/jenkins_names

Conversation

@driazati

@driazati driazati commented Nov 23, 2021

Copy link
Copy Markdown
Member

This uses the optional label argument on shell scripts in Jenkins to assign each one in CI a human-friendly name. This should help make it more clear what is going on in CI without having to dig into individual commands.

Tested in the test Jenkins here.

@areusch

@driazati driazati marked this pull request as ready for review November 23, 2021 17:37
@driazati driazati requested a review from a team as a code owner November 23, 2021 17:37
Comment thread Jenkinsfile Outdated
Comment thread Jenkinsfile Outdated
Comment thread Jenkinsfile Outdated
@Mousius

Mousius commented Nov 23, 2021

Copy link
Copy Markdown
Member

Thanks for the simple quality of life improvement here @driazati, just have a few questions but in general I think this can only improve things 😸

@driazati driazati requested a review from Mousius November 24, 2021 19:11

@Mousius Mousius left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@Mousius Mousius merged commit f833b4b into apache:main Nov 25, 2021
@Mousius

Mousius commented Nov 25, 2021

Copy link
Copy Markdown
Member

Thanks @driazati, this improvement has landed 😸

dchauhan-arm pushed a commit to dchauhan-arm/tvm that referenced this pull request Nov 29, 2021

Co-authored-by: driazati <driazati@users.noreply.github.com>
mehrdadh pushed a commit to mehrdadh/tvm that referenced this pull request Dec 1, 2021

Co-authored-by: driazati <driazati@users.noreply.github.com>
mehrdadh pushed a commit to mehrdadh/tvm that referenced this pull request Dec 1, 2021

Co-authored-by: driazati <driazati@users.noreply.github.com>
@driazati driazati deleted the driazati/jenkins_names branch December 2, 2021 09:44
ylc pushed a commit to ylc/tvm that referenced this pull request Jan 7, 2022
Co-authored-by: driazati <driazati@users.noreply.github.com>
yangulei pushed a commit to yangulei/tvm that referenced this pull request Jan 11, 2022

Co-authored-by: driazati <driazati@users.noreply.github.com>
yangulei pushed a commit to yangulei/tvm that referenced this pull request Jan 12, 2022

Co-authored-by: driazati <driazati@users.noreply.github.com>
ylc pushed a commit to ylc/tvm that referenced this pull request Jan 13, 2022
Co-authored-by: driazati <driazati@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants