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

airbyte-ci: use a new cache volume for docker cache #42905

Conversation

alafanechere
Copy link
Contributor

@alafanechere alafanechere commented Jul 31, 2024

What

It appears that the cache volume we use to cache /var/lib/docker on the side car docker engine was corrupted.
It led to failure of integration test run for java connectors (cf thread here)
I can't identify what let to the cache volume corruption but using a fresh volume (by renaming the volume name) fixes the testing problem.

Copy link

vercel bot commented Jul 31, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
airbyte-docs ⬜️ Ignored (Inspect) Visit Preview Jul 31, 2024 3:13pm

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @alafanechere and the rest of your teammates on Graphite Graphite

@alafanechere alafanechere force-pushed the augustin/07-31-DNM_debug_java_connector_integration_test_failures branch from b883a07 to c171e0e Compare July 31, 2024 14:25
@octavia-squidington-iii octavia-squidington-iii removed the area/connectors Connector related issues label Jul 31, 2024
@alafanechere alafanechere force-pushed the augustin/07-31-DNM_debug_java_connector_integration_test_failures branch from 2e5b876 to 69121ed Compare July 31, 2024 15:13
@alafanechere alafanechere changed the title DNM: debug java connector integration test failures airbyte-ci: use a new cache volume for docker cache Jul 31, 2024
@alafanechere alafanechere marked this pull request as ready for review July 31, 2024 15:14
@alafanechere alafanechere requested a review from a team as a code owner July 31, 2024 15:14
@alafanechere alafanechere merged commit b7cbad0 into master Jul 31, 2024
36 checks passed
@alafanechere alafanechere deleted the augustin/07-31-DNM_debug_java_connector_integration_test_failures branch July 31, 2024 16:33
@@ -41,7 +41,7 @@
DOCKER_CLI_IMAGE = f"docker:{DOCKER_VERSION}-cli"
DOCKER_REGISTRY_MIRROR_URL = os.getenv("DOCKER_REGISTRY_MIRROR_URL")
DOCKER_REGISTRY_ADDRESS = "docker.io"
DOCKER_VAR_LIB_VOLUME_NAME = "docker-cache"
DOCKER_VAR_LIB_VOLUME_NAME = "docker-cache-2"
Copy link
Contributor

Choose a reason for hiding this comment

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

Are we in the position where we should make a directory per day, so that this self heals?

BenGalewsky pushed a commit to BenGalewsky/airbyte that referenced this pull request Aug 4, 2024
jtruty pushed a commit to jtruty/airbyte that referenced this pull request Aug 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants