Skip to content

Conversation

hilookas
Copy link
Contributor

@hilookas hilookas commented Sep 9, 2022

After commit dabb285, kw_path was deprecated.

See update_deploy_variables in deploy.sh.

This commit also adds missing dependency libguestfs-tools for guestmount and guestfish.

Closes #246
Closes #677

Signed-off-by: Haiqin Cui i@18kas.com

@codecov-commenter
Copy link

Codecov Report

Base: 74.28% // Head: 74.28% // No change to project coverage 👍

Coverage data is based on head (f2fecab) compared to base (d6e3da1).
Patch has no changes to coverable lines.

❗ Current head f2fecab differs from pull request most recent head 7d15ade. Consider uploading reports for the commit 7d15ade to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##           unstable     #678   +/-   ##
=========================================
  Coverage     74.28%   74.28%           
=========================================
  Files            39       39           
  Lines          6273     6273           
=========================================
  Hits           4660     4660           
  Misses         1613     1613           
Impacted Files Coverage Δ
src/deploy.sh 65.40% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@hilookas hilookas changed the title Add dependency for vm deploy and update remote kw path src: vm: Add dependency for vm deploy and update remote kw path Sep 11, 2022
After commit dabb285, `kw_path` was
deprecated.

See `update_deploy_variables` in `deploy.sh`.

An test is added to cover this case.

This commit also adds missing dependency `libguestfs-tools` for
`guestmount` and `guestfish` in debian and arch linux.

Closes kworkflow#246
Closes kworkflow#677

Signed-off-by: Haiqin Cui <i@18kas.com>
@rodrigosiqueira
Copy link
Collaborator

Nice addition! Thanks a lot!

Merged into the unstable branch.

@hilookas hilookas deleted the vm-fix branch September 18, 2022 05:47
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.

3 participants