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

Fixes the integration test #7

Merged
merged 1 commit into from
Jul 25, 2024

Conversation

claudiubelu
Copy link
Contributor

During the integration test, we're cloning the SRIOV Network Device Plugin repository in order to deploy it.

However, it currently fails in the CI because we're cloning it in the test instance, rather than cloning it locally. This results in FileNotFoundErrors because the files we're trying to read to not exist locally.

This updates the integration test to clone the git repository locally, rather than in the test instance.

During the integration test, we're cloning the SRIOV Network Device
Plugin repository in order to deploy it.

However, it currently fails in the CI because we're cloning it in the
test instance, rather than cloning it locally. This results in
FileNotFoundErrors because the files we're trying to read to not exist
locally.

This updates the integration test to clone the git repository locally,
rather than in the test instance.
Copy link
Collaborator

@bschimke95 bschimke95 left a comment

Choose a reason for hiding this comment

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

LGTM

@bschimke95 bschimke95 merged commit 1bfa968 into canonical:main Jul 25, 2024
8 checks passed
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.

2 participants