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

OVA: Change the scanning for ova and ovf files inside the NFS directory #586

Merged
merged 1 commit into from
Sep 11, 2023

Conversation

bkhizgiy
Copy link
Member

@bkhizgiy bkhizgiy commented Sep 6, 2023

With the current change the files will be scanned as followed:

  • For OVA files we will search the zero level (the NFS itself) and first level (the share sub folders).
    For example giving the nfs share is /nfs these files will be selected:

    • /nfs/vm.ova
    • /nfs/dc/vm.ova
  • For extracted OVA files (ovf format) we will search the first level (the NFS sub folders) and second level (the sub folders sub folders).
    For example giving the nfs share is /nfs these files will be selected:

    • /nfs/vm/vm.ovf
    • /nfs/dc/vm1/vm1.ovf

cmd/ova-provider-server/ova-provider-server.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/ova-provider-server.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/ova-provider-server.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/ova-provider-server.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/ova-provider-server.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/ova-provider-server.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/ova-provider-server.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/ova-provider-server.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/nfs_test.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/nfs_test.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/nfs_test.go Show resolved Hide resolved
cmd/ova-provider-server/nfs_test.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/nfs_test.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/nfs_test.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/nfs_test.go Outdated Show resolved Hide resolved
Signed-off-by: Bella Khizgiyaev <bkhizgiy@redhat.com>
@sonarcloud
Copy link

sonarcloud bot commented Sep 11, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 4 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@ahadas ahadas added this to the v2.5.1 milestone Sep 11, 2023
@ahadas ahadas merged commit ea39abb into kubev2v:main Sep 11, 2023
7 checks passed
bkhizgiy added a commit to bkhizgiy/forklift that referenced this pull request Sep 14, 2023
backport kubev2v#586

Signed-off-by: Bella Khizgiyaev <bkhizgiy@redhat.com>
ahadas pushed a commit that referenced this pull request Sep 18, 2023
backport #586

Signed-off-by: Bella Khizgiyaev <bkhizgiy@redhat.com>
@ahadas ahadas removed this from the v2.5.1 milestone Sep 18, 2023
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.

4 participants