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

ci: add a workflow to check for entrypoint tags #9912

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nshy
Copy link
Contributor

@nshy nshy commented Apr 5, 2024

Check check-entrypoint.sh comment for explanation of what entrypoint tag is. The workflow fails if current branch does not have a most recent entrypoint tag that it should have.

Part of #8319

Check check-entrypoint.sh comment for explanation of what entrypoint tag
is. The workflow fails if current branch does not have a most recent
entrypoint tag that it should have.

NO_TEST=ci
NO_CHANGELOG=ci
NO_DOC=ci
@nshy nshy requested a review from a team as a code owner April 5, 2024 11:24
@nshy nshy requested a review from locker April 5, 2024 11:24
@coveralls
Copy link

Coverage Status

coverage: 87.042% (-0.003%) from 87.045%
when pulling 3208fcf on nshy:workflow-for-entrypoint
into 30524dd
on tarantool:master
.

@@ -0,0 +1,89 @@
#!/bin/bash
# vim: set ts=4 sts=4 sw=4 et:
Copy link
Member

Choose a reason for hiding this comment

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

Please remove this line.


if ! git --version >/dev/null 2>&1; then
error "git must be installed."
fi
Copy link
Member

Choose a reason for hiding this comment

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

I don't think we need to explicitly check presence of git. The script will fail with a proper error message if it isn't available.

startsWith(github.ref, 'refs/tags/')) &&
format('{0}-{1}', github.run_id, github.run_attempt) ||
format('{0}-{1}', github.workflow, github.ref) }}
cancel-in-progress: true
Copy link
Member

Choose a reason for hiding this comment

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

Since this job works only for the release branches and it's pretty short, I don't think we need to bother with cancelling in-progress jobs, but that's up to devx.

# We need to find the commit that starts this branch (i.e. that the first
# commit on this branch after the commit that is common for two branches.)
#
# In order to archive this we find all the commits of this branch that
Copy link
Member

Choose a reason for hiding this comment

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

achieve

Copy link
Member

Choose a reason for hiding this comment

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

Please rename to check-entrypoint-tag.sh.

@locker locker assigned nshy and unassigned locker Apr 5, 2024
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.

None yet

3 participants