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

Fix Docker integration test to use status.waitFor #237

Open
brynblack opened this issue Jul 9, 2024 · 1 comment
Open

Fix Docker integration test to use status.waitFor #237

brynblack opened this issue Jul 9, 2024 · 1 comment
Labels
development Standard development

Comments

@brynblack
Copy link
Member

Specification

As a stopgap for fixing the CI jobs we had to remove the usage of status.waitFor in the Docker integration tests. The reason for this is despite fixing how the filesystems were mounted for testing, any file changes inside of the container were not propagated back to the host filesystem. This means we could not check the status for any state change inside the test.

We need to figure out how to properly mount the filesystem between the host and the container filesystem to make sure that changes are propagated between both, and we need to update the test to use status.waitFor.

Additional context

Tasks

  1. Work out how to properly bind mount
  2. Fix up the tests
@brynblack brynblack added the development Standard development label Jul 9, 2024
Copy link

linear bot commented Jul 9, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
development Standard development
Development

No branches or pull requests

1 participant