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.

Sign up for GitHub

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

Jump to bottom

varfish-cli >0.6.x uses new import syntax #220

Closed
Nicolai-vKuegelgen opened this issue Jan 26, 2024 · 2 comments · Fixed by #221
Closed

varfish-cli >0.6.x uses new import syntax #220

Nicolai-vKuegelgen opened this issue Jan 26, 2024 · 2 comments · Fixed by #221

Comments

@Nicolai-vKuegelgen
Copy link
Contributor

Describe the bug
The current varfish-cli syntax for case upload used by cubi-tk does not exist anymore in varfish-cli 0.6.2 (varfish-cli --no-verify-ssl --verbose case create-import-info <UUID> <paths> now throws an error).

To Reproduce
Run current cubi-tk snappy varfish-upload with varfsih-cli 0.6.2 (auto installed by pip).

Expected behavior
Use proper syntax for current varfish-cli
@Nicolai-vKuegelgen Nicolai-vKuegelgen linked a pull request Jan 26, 2024 that will close this issue
fix: adapt to varfish-cli >=0.6.2 syntax #221
Merged
@Nicolai-vKuegelgen
Copy link
Contributor Author

calling the new varfish-cli from python is not atcually as easy as before. Need to use a subprocess call.

Nicolai-vKuegelgen added a commit that referenced this issue Jan 26, 2024
@Nicolai-vKuegelgen
fix: varfish cli 06x uses new import syntax (#220) (#222)
cb10dbb
@Nicolai-vKuegelgen
Copy link
Contributor Author

Fixed with #222

This was referenced Jan 13, 2025
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: adapt to varfish-cli >=0.6.2 syntax bihealth/cubi-tk
1 participant
@Nicolai-vKuegelgen