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

feat: update infrastructure to use macOS 14 #210

Merged
merged 1 commit into from
Dec 7, 2023

Conversation

weikequ
Copy link
Contributor

@weikequ weikequ commented Nov 28, 2023

Please review, will hold off on merging until infrastructure is available.

Issue #, if available:

Description of changes:

Testing 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.

@weikequ weikequ self-assigned this Nov 28, 2023
Copy link
Member

@pendo324 pendo324 left a 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 👍

Copy link
Contributor

@vsiravar vsiravar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice job formatting!

@weikequ
Copy link
Contributor Author

weikequ commented Nov 30, 2023

Getting lots of registry failures on the e2e tests - though it seems the last few commits have been failing the e2e tests.

@pendo324
Copy link
Member

pendo324 commented Dec 5, 2023

Can you merge in main to see if the latest commit fixes the e2e tests?

@KevinLiAWS KevinLiAWS closed this Dec 5, 2023
@KevinLiAWS KevinLiAWS reopened this Dec 5, 2023
@KevinLiAWS
Copy link
Contributor

Can you merge in main to see if the latest commit fixes the e2e tests?

Requeued the e2e tests and now it is running

@weikequ
Copy link
Contributor Author

weikequ commented Dec 6, 2023

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/common-tests@v0.7.9/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/common-tests@v0.7.9/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 <weikequ@amazon.com>
@weikequ weikequ merged commit eb14219 into runfinch:main Dec 7, 2023
6 checks passed
@weikequ weikequ deleted the update-to-sonoma branch December 7, 2023 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants