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

Add GitHub Action Workflow: create-deploy-tag #165213

Merged
merged 1 commit into from
Aug 30, 2023

Conversation

watson
Copy link
Contributor

@watson watson commented Aug 30, 2023

This is intended to be used by the Kibana Release Manager when kicking off the promotion of a new Kibana Serverless release into our Serverless QA environment (see #165009 for details).

This GitHub Action workflow is inspired by this. The main difference is that it has an input field to allow tagging something other than HEAD on main.

Note to reviewers

I've tried to test this locally using act, but ran into some permission issues in the "Create tag" step. However, the other steps completed fine. I'm not sure if there's a good way to verify this before merging. Alternatively we can merge, test and make follow-up PR's accordingly.

@watson watson added release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting labels Aug 30, 2023
@watson watson self-assigned this Aug 30, 2023
@watson watson requested a review from a team as a code owner August 30, 2023 11:07
@watson watson added the skip-ci label Aug 30, 2023
Copy link
Contributor

@delanni delanni left a comment

Choose a reason for hiding this comment

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

Looks good 👍 Let's play around with this and refine the access rights while the rest of the pipeline is not present.

git tag ${TAG_NAME} ${COMMIT}
git push origin "refs/tags/${TAG_NAME}"
- if: always()
uses: elastic/apm-pipeline-library/.github/actions/notify-build-status@current
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Btw, I just copied this from the APM team and is using their notify script. I'm not sure if we have a more Elastic-wide generic script to do the same? It seems odd to me that APM has a custom one?

} || true
- name: Verify branch
run: |
if [[ "${GITHUB_REF}" != "refs/heads/main" ]]; then
Copy link
Member

Choose a reason for hiding this comment

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

We probably want to skip this on forks?

@watson watson merged commit 3128c46 into elastic:main Aug 30, 2023
@watson watson deleted the add-create-deploy-tag branch August 30, 2023 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes skip-ci v8.11.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants