Skip to content

[ci] Add auto-updating last-successful branch#10056

Merged
Mousius merged 3 commits into
apache:mainfrom
driazati:auto_green
Feb 21, 2022
Merged

[ci] Add auto-updating last-successful branch#10056
Mousius merged 3 commits into
apache:mainfrom
driazati:auto_green

Conversation

@driazati

@driazati driazati commented Jan 25, 2022

Copy link
Copy Markdown
Member

This adds a script that runs on a cron to discover the last commit where CI all passed (every job was successful and tvm-ci/branch is included) and updates a git branch last-successful that tracks this commit from main. This can be used for checking out the latest unbroken TVM, which can be useful for developers wanting a good changeset to base their changes on or for infra needing a clean, up-to-date TVM.

Also soliciting better names than green

@Mousius

Mousius commented Jan 25, 2022

Copy link
Copy Markdown
Member

I suggest last-successful as a tag so we're not coupled to the colour 😸

@driazati driazati changed the title [ci] Add auto-updating 'green' tag [ci] Add auto-updating last-successful tag Jan 25, 2022
Comment thread tests/scripts/update_tag.py Outdated
Comment thread .github/workflows/update_tag.yml Outdated
Comment thread tests/scripts/update_tag.py Outdated
Comment thread tests/scripts/update_tag.py
@areusch

areusch commented Feb 1, 2022

Copy link
Copy Markdown
Contributor

one problem here is that a tag doesn't automatically update with git fetch. A branch would. it's a little confusing to have a branch instead of a tag, but a branch is confined to origin/ namespace and will auto-update with git fetch. i think we should switch to updating a branch to avoid confusing behavior where last-successful is not updated after a fetch.

cc @leandron @mbrookhart if there are other ideas

@github-actions

github-actions Bot commented Feb 2, 2022

Copy link
Copy Markdown
Contributor

It has been a while since this PR was updated, @leandron @mbrookhart @AndrewZhaoLuo @driazati @areusch please leave a review or address the outstanding comments

@driazati driazati changed the title [ci] Add auto-updating last-successful tag [ci] Add auto-updating last-successful branch Feb 2, 2022

@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.

This looks ready to me, @areusch can you take another look?

@github-actions

github-actions Bot commented Feb 9, 2022

Copy link
Copy Markdown
Contributor

It has been a while since this PR was updated, @leandron @mbrookhart @Mousius @AndrewZhaoLuo @driazati @areusch please leave a review or address the outstanding comments

@driazati

driazati commented Feb 9, 2022

Copy link
Copy Markdown
Member Author

Sorry about the overactive bot everyone, it's still missing some checks to look at before sending out these pings

This adds a script that runs on a cron to discover the last commit where CI all passed (every job was successful and `tvm-ci/branch` is included) and updates a git tag `green` to point to this commit on `main`. This can be used for checking out the latest unbroken TVM, which can be useful for developers wanting a good changeset to base their changes on or for infra needing a clean, up-to-date TVM.
@Mousius Mousius self-assigned this Feb 15, 2022
@Mousius Mousius merged commit 7d831e1 into apache:main Feb 21, 2022
@Mousius

Mousius commented Feb 21, 2022

Copy link
Copy Markdown
Member

Let's get this in so we can start iterating if we need to 😸

pfk-beta pushed a commit to pfk-beta/tvm that referenced this pull request Apr 11, 2022
This adds a script that runs on a cron to discover the last commit where CI all passed (every job was successful and `tvm-ci/branch` is included) and updates a git tag `green` to point to this commit on `main`. This can be used for checking out the latest unbroken TVM, which can be useful for developers wanting a good changeset to base their changes on or for infra needing a clean, up-to-date TVM.
@fpedd

fpedd commented Apr 13, 2023

Copy link
Copy Markdown
Contributor

What happened to this branch? The CI job got manually disabled about 5 months ago, it seems.

@driazati

Copy link
Copy Markdown
Member Author

I filed an issue with the reason for disabling and potential fix: #14618

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.

5 participants