-
Notifications
You must be signed in to change notification settings - Fork 517
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
create CI dhall components for building and caching the test-executive and executing integration tests #7743
Conversation
0cddd9e
to
ae0deba
Compare
ae0deba
to
af96b67
Compare
Command.Config::{ | ||
commands = [ | ||
Cmd.run ( | ||
"if [ ! -f ${deployEnv} ]; then " ++ |
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.
created #7745 to address DRY'ing this code up a bit
bca4b40
to
f788a4c
Compare
f788a4c
to
6914b32
Compare
JobSpec::{ | ||
dirtyWhen = [ | ||
S.strictlyStart (S.contains "src"), | ||
S.strictlyStart (S.contains "src/app/test-executive"), |
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.
remove
75c95cf
to
5255f46
Compare
…to ci_integration_test
af7784b
to
57c8044
Compare
automation/terraform/modules/kubernetes/buildkite-agent/helm.tf
Outdated
Show resolved
Hide resolved
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.
Looks good aside from the existing comments, a makefile directive definitely seems worthwhile
b33ecb1
to
c5870b8
Compare
Also increase Buildkite agent cluster capacity to handle increased CI load and add artifact caching helper script to temporarily address #7745.
Test: Buildkite CI
Checklist: