Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds a new integration test job to the CI workflow to improve test coverage by running integration tests in the build pipeline.
Key changes:
- Added a new
integrationjob to the GitHub Actions workflow - Configured the job to build the project and run integration tests using Go
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
austinvazquez
force-pushed
the
add-integration-test-to-ci
branch
2 times, most recently
from
48a516c to
0e0ef9f
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 8 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
austinvazquez
force-pushed
the
add-integration-test-to-ci
branch
from
0e0ef9f to
ea4838d
Compare
austinvazquez
force-pushed
the
add-integration-test-to-ci
branch
from
ea4838d to
84890ed
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
austinvazquez
force-pushed
the
add-integration-test-to-ci
branch
from
84890ed to
a7e64a6
Compare
austinvazquez
force-pushed
the
add-integration-test-to-ci
branch
from
a7e64a6 to
31f586d
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 7 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
austinvazquez
force-pushed
the
add-integration-test-to-ci
branch
from
31f586d to
8a9ee83
Compare
austinvazquez
force-pushed
the
add-integration-test-to-ci
branch
from
8a9ee83 to
1af3e23
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 10 out of 11 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
austinvazquez
force-pushed
the
add-integration-test-to-ci
branch
from
1af3e23 to
f6a5b11
Compare
austinvazquez
force-pushed
the
add-integration-test-to-ci
branch
from
f6a5b11 to
8a835c3
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 10 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
austinvazquez
force-pushed
the
add-integration-test-to-ci
branch
from
8a835c3 to
991c714
Compare
austinvazquez
force-pushed
the
add-integration-test-to-ci
branch
from
991c714 to
a041e1f
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 10 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
austinvazquez
force-pushed
the
add-integration-test-to-ci
branch
from
a041e1f to
9e58257
Compare
|
Downgrade to 6.12.44 (from #74) |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 9 changed files in this pull request and generated 7 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
austinvazquez
force-pushed
the
add-integration-test-to-ci
branch
2 times, most recently
from
5d56f95 to
9efaabd
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
austinvazquez
force-pushed
the
add-integration-test-to-ci
branch
from
a582968 to
50ebc4f
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 9 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
austinvazquez
force-pushed
the
add-integration-test-to-ci
branch
2 times, most recently
from
2be7860 to
54921d9
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Signed-off-by: Austin Vazquez <austin.vazquez@docker.com>
Signed-off-by: Austin Vazquez <austin.vazquez@docker.com>
Signed-off-by: Austin Vazquez <austin.vazquez@docker.com>
austinvazquez
force-pushed
the
add-integration-test-to-ci
branch
from
54921d9 to
a4eb9d7
Compare
This change adds the existing integration tests to CI to validate changes. Additionally this change introduces a kernel build cache to reduce time spent waiting on kernel builds which change infrequently.