-
Notifications
You must be signed in to change notification settings - Fork 2.1k
[CI] Pipeline optimisations #10267
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
Open
sean-breen
wants to merge
40
commits into
main
Choose a base branch
from
chore/sb-ci-updates
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[CI] Pipeline optimisations #10267
Changes from 6 commits
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
ede9fd9
fix bug in run_tests logic
sean-breen a207440
wider net for docs_only
sean-breen 5742781
Merge branch 'main' into chore/sb-ci-updates
sean-breen 35c8d2c
fix docs_only detection, and enable for merge queue
sean-breen c7a72a9
Merge branch 'main' into chore/sb-ci-updates
sean-breen caea766
address feedback
sean-breen 9ffea18
Potential fix for pull request finding
sean-breen 531af52
gate all logic via variables.sh
sean-breen 1fa0c69
merge main into chore/sb-ci-updates
sean-breen b102857
add back missing file
sean-breen a94baeb
cant use empty needs
sean-breen 1681951
fix tag_stable logic, doesnt run on docs_only
sean-breen ec1dd2a
allow inspection of which jobs failed
sean-breen 6fba7c7
Merge branch 'main' into chore/sb-ci-updates
sean-breen 53cc407
update logic around CI flags
sean-breen 02c7b1f
prevent unnecessary rebuilds when modifying files that do not affect …
sean-breen 63a86d2
dont build for helm tests when a stable tag exists
sean-breen 1c99f97
Merge branch 'main' into chore/sb-ci-updates
sean-breen abca8c6
add prompts dir to exclusion list
sean-breen 2abbd40
remove redundant run_build variable
sean-breen cd52acc
full-build only when force is true
sean-breen db9ef51
Merge branch 'main' into chore/sb-ci-updates
sean-breen ed135cf
remove test make targets
sean-breen 04a454a
add ci-preview scripts and make targets
sean-breen d06c778
Merge branch 'main' into chore/sb-ci-updates
sean-breen 58f6b5f
short-circuit the e2e and tag-stable checks when build-artifacts did …
sean-breen bb85131
set-matrix
sean-breen a25440a
fix gating for e2e jobs + final ci results
sean-breen 12ec6a0
Merge branch 'main' into chore/sb-ci-updates
vepatel 65b0338
check for target_exists, and force a rebuild if not
sean-breen 631d567
Merge branch 'main' into chore/sb-ci-updates
sean-breen 9517d8a
Merge branch 'main' into chore/sb-ci-updates
sean-breen b77abba
Merge branch 'main' into chore/sb-ci-updates
sean-breen a93dccb
pr feedback
sean-breen e952897
merge main
sean-breen 05034cb
add developer doc, and update skill
sean-breen 8660787
fix repo paths
sean-breen 63ab118
fix failing test
sean-breen 169ffc9
fix failing test
sean-breen 33c1dd8
fix arch mismatch?
sean-breen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
any specific reason for 50 commit depth?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could we rely on the the checkout action using a fetch depth of 0 here?