-
Notifications
You must be signed in to change notification settings - Fork 17
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
feat: update infrastructure to use macOS 14 #210
Conversation
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.
LGTM. Just need the infra PR merged / deployed first 👍
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.
Nice job formatting!
Getting lots of registry failures on the e2e tests - though it seems the last few commits have been failing the e2e tests. |
a39186f
to
05c86f3
Compare
Can you merge in main to see if the latest commit fixes the e2e tests? |
Requeued the e2e tests and now it is running |
05c86f3
to
5564bdb
Compare
Still e2e test failures, but a different test this time. The same failure is experience in previous commits to main as well. Finch Core E2E Tests Build container image Build container image using default image Build an image with --ssh option
/Users/ec2-user/go/pkg/mod/github.com/runfinch/[email protected]/tests/build.go:129
No containers to be removed
No images to be removed
No volumes to be removed
host
none
No networks to be removed
error: invalid empty ssh agent socket: make sure SSH_AUTH_SOCK is set
[FAILED] in [It] - /Users/ec2-user/go/pkg/mod/github.com/runfinch/[email protected]/command/command.go:122 @ 12/06/23 17:55:39.229
No containers to be removed
No images to be removed
No volumes to be removed
host
none
No networks to be removed
• [FAILED] [0.698 seconds]
Finch Core E2E Tests Build container image Build container image using default image [It] Build an image with --ssh option |
Signed-off-by: Weike Qu <[email protected]>
5564bdb
to
42ee7c2
Compare
Please review, will hold off on merging until infrastructure is available.
Issue #, if available:
Description of changes:
Updated references toVentura
in our workflow to refer to thelatest
macOS version insteadTesting done:
none
I've reviewed the guidance in CONTRIBUTING.md
License Acceptance
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.