| kata-irc-bot | <zhuxiaoyang1996> https://github.com/kata-containers/kata-containers/issues/4632 is there anyone can help to take a look this issue? | 01:57 |
|---|---|---|
| kata-irc-bot | <fidencio> cc @gmaglion DAX support has never been stable to make its way upstream. I don't think we have it implemented as part of the rust virtiofsd. | 06:20 |
| kata-irc-bot | <zhuxiaoyang1996> @fidencio what should I do if I want to use DAX? although it still a experimental feature. | 06:42 |
| kata-irc-bot | <zhuxiaoyang1996> thanks @fidencio, BTW, Is C version of virtiofsd compatible with version 2.5.0 of kata? | 06:48 |
| kata-irc-bot | <fidencio> I don't remember any QEMU bump, so should be. But I'd recommend building QEMU yourself then, with virtiofsd support enabled there | 06:49 |
| kata-irc-bot | <fidencio> The C daemon of virtiofsd comes together with QEMU | 06:53 |
| kata-irc-bot | <zhuxiaoyang1996> got it. | 07:00 |
| kata-irc-bot | <fidencio> Folks, if I may bother y'all for a review, https://github.com/kata-containers/kata-containers/pull/4628 is something I'd like to have merged as the CC work has some dependencies on it | 07:15 |
| kata-irc-bot | <zhuxiaoyang1996> @fidencio Hi, How do I modify the build-static-qemu-experimental script to support building virtiofsd and qemu together? is there any example? | 07:24 |
| kata-irc-bot | <bbolroc> @fidencio could you review this PR? I got instructed by Gaby and had all the tests passed. | 07:54 |
| kata-irc-bot | <gmaglion> @zhuxiaoyang1996 As @fidencio said, DAX is still experimental and incomplete, you need a patched qemu + virtiofsd (C version), see: https://gitlab.com/virtio-fs/qemu/-/commits/virtio-fs-dev | 08:01 |
| kata-irc-bot | <gmaglion> those are the latest patches for DAX support | 08:02 |
| kata-irc-bot | <zhuxiaoyang1996> @gmaglion so can I clone this repo with branch `virtio-fs-dev` and building it followed: ```mkdir build cd build ../configure make``` | 08:07 |
| kata-irc-bot | <fidencio> @bbolroc, you're missing the ¨Fixes" in the commit message | 08:08 |
| kata-irc-bot | <fidencio> It's not. We need to create an issue and associate that PR with the newly created issue | 08:10 |
| kata-irc-bot | <fidencio> Sorry about the bureaucracy | 08:10 |
| kata-irc-bot | <zkaiser> If there is no issue why even fix things? ;) | 08:11 |
| kata-irc-bot | <bbolroc> oh, then I will create one and get back to you. Thanks! | 08:11 |
| kata-irc-bot | <fidencio> Yeah, then you'll need to force push the PR, we'll re-run the tests, and that will get merged as soon as the tests pass. | 08:12 |
| kata-irc-bot | <fidencio> You already have my Ack! :slightly_smiling_face: | 08:12 |
| kata-irc-bot | <fidencio> Thanks for working on that! | 08:12 |
| kata-irc-bot | <bbolroc> Done, thanks. ^^ | 08:21 |
| kata-irc-bot | <fidencio> s/test/action | 10:23 |
| kata-irc-bot | <fidencio> And I merged your PR | 10:23 |
| kata-irc-bot | <bbolroc> oh, is it a bug? (The bot is supposed to add a tag, right?) | 10:24 |
| kata-irc-bot | <bbolroc> Yeah, I will open up an issue for this. (under the `tests` repo) | 10:25 |
| kata-irc-bot | <bergwolf> Hi folks, no AC meeting agenda for today. Let's cancel this week. Talk to you next week! | 14:03 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!