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

Tests fail on NixOS #132

Closed
MagicRB opened this issue Jan 11, 2021 · 11 comments
Closed

Tests fail on NixOS #132

MagicRB opened this issue Jan 11, 2021 · 11 comments
Labels
kind/bug Categorizes issue or PR as related to a bug. lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed.

Comments

@MagicRB
Copy link

MagicRB commented Jan 11, 2021

What happened:
Test case test/e2e/e2e_suite_test.go:157 fails when buiding with Nix

What you expected to happen:
I expected tests to succeed as checking if the project directory ends in csi-driver-nfs doesn't make sense

How to reproduce it:
Try to build csi-driver-nfs on NixOS with pkgs.buildGoModule and you'll encounter this fun bug

Anything else we need to know?:

Environment:

  • CSI Driver version: N/A
  • Kubernetes version (use kubectl version): N/A
  • OS (e.g. from /etc/os-release): NixOS
  • Kernel (e.g. uname -a): N/A
  • Install tools: N/A
  • Others: N/A

Recommended Fix:
Delete that line

@andyzhangx
Copy link
Member

hi @MagicRB would you submit a PR directly? thanks.

@andyzhangx
Copy link
Member

and you could also contribute how to build this project in NixOS

@MagicRB
Copy link
Author

MagicRB commented Jan 11, 2021

I really don't want to go through the license agreement stuff, so unfortunately not at this time

@andyzhangx
Copy link
Member

while I don't have NixOS, not sure how to fix this.

@MagicRB
Copy link
Author

MagicRB commented Jan 11, 2021

and another issue has cropped up, test/e2e/e2e_suite_test.go:26:2: imported and not used: "strings" which is really an easy fix as the first one. And as to NixOS building, that should really go into nixpkgs, which I'll do at some point.

while I don't have NixOS, not sure how to fix this.

there really only two things, which we can do, either delete that project name check or disable tests as I can't tell Nix to use a different folder, it would break the pureness of the build process.

@andyzhangx
Copy link
Member

cc @dr0pdb

@andyzhangx andyzhangx added the kind/bug Categorizes issue or PR as related to a bug. label Jan 11, 2021
@MagicRB
Copy link
Author

MagicRB commented Jan 11, 2021

BTW, I pushed the Docker container builder to my Gitea instance, the builder builds this driver from Nix. It's easy to extract the Nix package from that. The advantage over regular Dockerfiles is truly reproducible builds and (generally) smaller images, seems to not be that pronounced with go apps

pohly added a commit to pohly/csi-driver-nfs that referenced this issue Feb 3, 2021
7bc70e5 Merge pull request kubernetes-csi#129 from pohly/squash-documentation
e0b02e7 README.md: document usage of --squash
316cb95 Merge pull request kubernetes-csi#132 from yiyang5055/bugfix/boilerplate
26e2ab1 fix: default boilerplate path
1add8c1 Merge pull request kubernetes-csi#133 from pohly/kubernetes-1.20-tag
3e811d6 prow.sh: fix "on-master" prow jobs

git-subtree-dir: release-tools
git-subtree-split: 7bc70e5
@fejta-bot
Copy link

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-contributor-experience at kubernetes/community.
/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Apr 11, 2021
@fejta-bot
Copy link

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

Send feedback to sig-contributor-experience at kubernetes/community.
/lifecycle rotten

@k8s-ci-robot k8s-ci-robot added lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels May 11, 2021
@fejta-bot
Copy link

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.

Send feedback to sig-contributor-experience at kubernetes/community.
/close

@k8s-ci-robot
Copy link
Contributor

@fejta-bot: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.

Send feedback to sig-contributor-experience at kubernetes/community.
/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

TerryHowe pushed a commit to TerryHowe/csi-driver-nfs that referenced this issue Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed.
Projects
None yet
Development

No branches or pull requests

4 participants