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

[Ops] create-deploy-tag workflow: Improve speed #168907

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

watson
Copy link
Contributor

@watson watson commented Oct 14, 2023

This PR improves the speed of the workflow by reducing the fetch depth from everything to just the last 500 commits. As long as the commit you're trying to deploy is within the last 500 commits this will work. This speeds up the checkout from 5 minutes to less than one minute!

However, the workflow will fail if the chosen commit is not within the last 500 commits. In the rare case that we need to deploy an older commit, it's possible to override the default 500 when starting the workflow.

This PR also tidies up the workflow yaml and removes the link to the workflow run in case of a successful run (as it's only really needed in case you want to debug a failed run).

@watson watson added release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting labels Oct 14, 2023
@watson watson self-assigned this Oct 14, 2023
@watson watson requested a review from a team as a code owner October 14, 2023 05:41
@watson
Copy link
Contributor Author

watson commented Oct 14, 2023

@elastic/kibana-tech-leads @delanni FYI 🚤

@watson watson merged commit 6fdd1f1 into elastic:main Oct 17, 2023
5 checks passed
@watson watson deleted the improve-deploy-workflow branch October 17, 2023 03:42
dej611 pushed a commit to dej611/kibana that referenced this pull request Oct 17, 2023
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 v8.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants