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

source-mysql and source-mssql failing due to missing arm64 image #29463

Closed
bnchrch opened this issue Aug 16, 2023 · 1 comment
Closed

source-mysql and source-mssql failing due to missing arm64 image #29463

bnchrch opened this issue Aug 16, 2023 · 1 comment

Comments

@bnchrch
Copy link
Contributor

bnchrch commented Aug 16, 2023

Problem

When running either of

  • airbyte-ci connectors --name=source-mysql test
  • airbyte-ci connectors --name=source-mssql test

Locally or in CI we get the following failure:

Could not pull image: no matching manifest for linux/arm64/v8 in the manifest list entries

Possible Causes

test containers do not have proper variant

Quote from @alafanechere

we've put a bit more reflection on the source-mysql testing issue with Ben.
So this connector uses testcontainers to spin up test db instance in docker containers.
I think the local root cause of your error is because this tools uses a mysql image which does not have an AMD variant (cf this stack overflow link).
In the CI (running on ARM) we do not have this problem. We have another one. The tests can't connect to the test containers db.
I think this is because there's some networking tweaks required to happen when running testcontainers inside a container like we do with dagger... I overcame this kind of issue in the past for source-postgres with this PR. I think it's worthwile trying to implement the same changes I did for source-postgres for source-mysql. I did not do it for this connector in the past because I wanted airbyte-ci to be on par with /legacy-test for GA connectors mainly.

However there is a variant for the mysql test container

Appenix

@bnchrch bnchrch changed the title source-mysql failing due to missing arm64 image source-mysql and source-mssql failing due to missing arm64 image Aug 16, 2023
@alafanechere alafanechere self-assigned this Aug 31, 2023
@postamar
Copy link
Contributor

This can be closed now: source-mysql is fixed and source-mssql fails for some other, unrelated reason involving secrets. Feel free to reopen if/when applicable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants