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

libvirt: wrong key name for fw_cfg #244

Closed
tormath1 opened this issue Aug 12, 2019 · 5 comments
Closed

libvirt: wrong key name for fw_cfg #244

tormath1 opened this issue Aug 12, 2019 · 5 comments

Comments

@tormath1
Copy link
Contributor

Bug

Hi ! ignition file is not mounted with qemu if we use opt/com.flatcar/config as key name for the firewall configuration. (as suggested in the doc: https://docs.flatcar-linux.org/os/booting-with-libvirt/#running-flatcar-linux-on-libvirt). We need to use opt/com.coreos/config. We need to fix the documentation or the const value. (I don't know what is the best)

Operating System Version

$ uname -r 
5.2.8-arch1-1-ARCH
$ virsh version
Compiled against library: libvirt 5.5.0
Using library: libvirt 5.5.0
Using API: QEMU 5.5.0
Running hypervisor: QEMU 4.0.0

Ignition Version

?

Environment

I am using QEMU hypervisor.

Expected Behavior

System should boot and read ignition provided file.

Actual Behavior

Ignition file is ignored

Reproduction Steps

  1. Follow the doc

Other Information

https://github.com/flatcar-linux/ignition/blob/8e2ef5adcc07692ab0c76c9d45e5fad82cedb7a4/internal/providers/qemu/qemu.go#L31-L33
@dongsupark
Copy link
Member

Thanks for the bug report!

That definitely looks like a bug.
As @iaguis suggested, we should probably change the string to opt/org.flatcar-linux/config.
Will soon prepare the fix.

@tormath1
Copy link
Contributor Author

I can open a PR if you want :)

@dongsupark
Copy link
Member

Thanks.

But I suppose, fixing only the ignition part is not enough.
We should fix the same thing also in mantle, scripts, etc.
Also there seems to be a corner case in coreos-overlay, about go mod and mantle, etc.
I'll try to create those PRs as soon as possible, and get feedback from other folks.

dongsupark referenced this issue in flatcar/mantle Aug 13, 2019
platform: fix qemu key name to org/flatcar-linux
3df76c3 
To fix a wrong key name when running a qemu VM with ignition, we need to
replace `opt/com.coreos` with `opt/org.flatcar-linux`.

See also https://github.com/flatcar-linux/ignition/issues/2
dongsupark referenced this issue in flatcar/scripts Aug 13, 2019
build_library: fix qemu key name to org/flatcar-linux
3e6c459 
To fix a wrong key name when running a qemu VM with ignition, we need to
replace `opt/com.coreos` with `opt/org.flatcar-linux`.

See also https://github.com/flatcar-linux/ignition/issues/2
dongsupark referenced this issue in flatcar-archive/coreos-overlay Aug 13, 2019
update coreos-overlay to fix the qemu key of mantle & ignition
b1e089d 
To fix a wrong key name when running a qemu VM with ignition, we need to
replace `opt/com.coreos` with `opt/org.flatcar-linux`.

See also https://github.com/flatcar-linux/ignition/issues/2
@dongsupark
Copy link
Member

Created the following PRs:

dongsupark referenced this issue in flatcar-archive/coreos-overlay Aug 14, 2019
update coreos-overlay to fix the qemu key of mantle & ignition
d1db62d 
To fix a wrong key name when running a qemu VM with ignition, we need to
replace `opt/com.coreos` with `opt/org.flatcar-linux`.

See also https://github.com/flatcar-linux/ignition/issues/2
@dongsupark
Copy link
Member

Oh, it was automatically closed. I didn't mean it. :-)
Anyway, we will release new Flatcar images in the next days. With the releases, this issue should be fixed.
If not, feel free to reopen this issue.
Thanks!

@vbatts vbatts transferred this issue from flatcar/ignition Nov 12, 2020
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

No branches or pull requests
2 participants
@dongsupark @tormath1