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 integration tests #26

Merged
merged 2 commits into from
Sep 16, 2024

Conversation

claudiubelu
Copy link
Contributor

There are a few issues running the Longhorn integration tests on an LXD runner, as we currently are, mainly having to do with ISCSI volumes being exposed and mounted in LXD.

Running on host has its own issues, mainly the Pods have cluster-wide connectivity issues.

Instead, we'll be testing Longhorn in a multipass VM. We also remove some of the GitHub runner's bloat in order to free up sufficient space for the VM and test. Note that according to the Longhorn deployment guide, Longhorn requires 25% of the root disk to be available (if no other disks are available).

According to the deployment guide, the node requires open-iscsi and nfs-common. These daemonsets will install them on the host.

@claudiubelu claudiubelu requested a review from a team as a code owner September 16, 2024 08:41
@claudiubelu claudiubelu marked this pull request as draft September 16, 2024 08:53
@claudiubelu claudiubelu marked this pull request as ready for review September 16, 2024 09:19
.github/workflows/run_tests.yaml Outdated Show resolved Hide resolved
.github/workflows/run_tests.yaml Outdated Show resolved Hide resolved
.github/workflows/run_tests.yaml Outdated Show resolved Hide resolved
There are a few issues running the Longhorn integration tests on an LXD
runner, as we currently are, mainly having to do with ISCSI volumes
being exposed and mounted in LXD.

Running on host has its own issues, mainly the Pods have cluster-wide
connectivity issues.

Instead, we'll be testing Longhorn in a multipass VM. We also remove
some of the GitHub runner's bloat in order to free up sufficient space
for the VM and test. Note that according to the Longhorn deployment
guide, Longhorn requires 25% of the root disk to be available (if no
other disks are available).
According to the deployment guide, the node requires open-iscsi and
nfs-common. These daemonsets will install them on the host.
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 a0a40c5 into canonical:main Sep 16, 2024
16 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