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

libvirt_domain_attach_device() in v0.8.1 unexpectedly calls virDomainAttachDevice instead of virDomainAttachDeviceFlags #1

Closed
mrc0mmand opened this issue Feb 8, 2024 · 1 comment

Comments

@mrc0mmand
Copy link

As described in [0], libvirt_domain_attach_device() in v0.8.1 calls virDomainAttachDevice when flags == 0, which doesn't work when the target domain isn't active, whereas calling virDomainAttachDeviceFlags works even on inactive domains . This was changed in [1] and seems to be an unfortunate design flaw [2], where virDomainAttachDeviceFlags(0) != virDomainAttachDevice. Similarly for virDomainDetachDeviceFlags and virDomainDettachDevice.

[0] vagrant-libvirt/vagrant-libvirt#1800
[1] 798c0ca
[2] vagrant-libvirt/vagrant-libvirt#1800 (comment)

Copy link

github-actions bot commented Feb 8, 2024

Thank you for your interest in the libvirt project.

Since this repository is a read-only mirror of the project's master
repostory hosted on GitLab, issues opened here are not processed.

We kindly request that new issues are reported to

https://gitlab.com/libvirt/libvirt-ruby/-/issues/new

Thank you for your time and understanding.

@github-actions github-actions bot closed this as completed Feb 8, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Feb 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant