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

Implement creating sysinfo fwcfg entries #1809

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Implement creating sysinfo fwcfg entries #1809

wants to merge 2 commits into from

Conversation

dev-zero
Copy link

With the following configuration this can be used to provide an Ignition configuration to boxes using ignition.machine.id=qemu:

  config.vm.provider :libvirt do |libvirt|
    libvirt.sysinfo_fwcfg :name => "opt/com.coreos/config", :file => '/tmp/provision.ign'

@dev-zero
implement passing sysinfo fwcfg entries
9aa002e 
This allows to pass an Ignition file via Qemu FwCfg which can then get
picked up by a box running Ignition with the `ignition.platform.id=qemu`
kernel option.
@dev-zero
driver: fix qemu_use_session
9caae1d 
Detection of network connections via system is broken, but directly querying libvirt is working properly. Using the system connection with qemu_use_session is enabled will lead to polkit trying to acquire root privileges, which is unavailable in some setups. The client is nowadays capable of enumerating the network devices as unprivileged user.
Fixes vagrant-libvirt#1808.
@dev-zero
Copy link
Author

Alternatively we could do what the Terraform libvirt provider is doing and have only a single attribute fw_cfg_name to tune a single name, only support :file and then an abstract logic to automatically create an ISO on systems/hypervisors where fw_cfg is not supported.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@dev-zero