From: Filip Schauer <f.schauer@proxmox.com>
To: pve-devel@lists.proxmox.com
Subject: Re: [pve-devel] [PATCH manager v16 12/12] ui: add options to add virtio-fs to qemu config
Date: Mon, 7 Apr 2025 14:03:24 +0200 [thread overview]
Message-ID: <544b2160-853c-4cf8-bdbe-ce0c63215a09@proxmox.com> (raw)
In-Reply-To: <20250404160847.489922-13-m.frank@proxmox.com>
This does not apply on the most recent master.
On 04/04/2025 18:08, Markus Frank wrote:
> Signed-off-by: Markus Frank <m.frank@proxmox.com>
> ---
> v16:
> * added hint for directory mapping in VirtiofsEdit
>
> www/manager6/Makefile | 1 +
> www/manager6/Utils.js | 1 +
> www/manager6/qemu/HardwareView.js | 19 ++++
> www/manager6/qemu/VirtiofsEdit.js | 143 ++++++++++++++++++++++++++++++
> 4 files changed, 164 insertions(+)
> create mode 100644 www/manager6/qemu/VirtiofsEdit.js
>
> diff --git a/www/manager6/Makefile b/www/manager6/Makefile
> index fabbdd24..fdf0e816 100644
> --- a/www/manager6/Makefile
> +++ b/www/manager6/Makefile
> @@ -271,6 +271,7 @@ JSSRC= \
> qemu/Smbios1Edit.js \
> qemu/SystemEdit.js \
> qemu/USBEdit.js \
> + qemu/VirtiofsEdit.js \
> sdn/Browser.js \
> sdn/ControllerView.js \
> sdn/Status.js \
> diff --git a/www/manager6/Utils.js b/www/manager6/Utils.js
> index aa415759..5a642076 100644
> --- a/www/manager6/Utils.js
> +++ b/www/manager6/Utils.js
> @@ -1645,6 +1645,7 @@ Ext.define('PVE.Utils', {
> serial: 4,
> rng: 1,
> tpmstate: 1,
> + virtiofs: 10,
> },
>
> // we can have usb6 and up only for specific machine/ostypes
> diff --git a/www/manager6/qemu/HardwareView.js b/www/manager6/qemu/HardwareView.js
> index c6d193fc..34aeb51e 100644
> --- a/www/manager6/qemu/HardwareView.js
> +++ b/www/manager6/qemu/HardwareView.js
> @@ -319,6 +319,16 @@ Ext.define('PVE.qemu.HardwareView', {
> never_delete: !caps.nodes['Sys.Console'],
> header: gettext("VirtIO RNG"),
> };
> + for (let i = 0; i < PVE.Utils.hardware_counts.virtiofs; i++) {
> + let confid = "virtiofs" + i.toString();
> + rows[confid] = {
> + group: 50,
> + order: i,
> + iconCls: 'folder',
> + editor: 'PVE.qemu.VirtiofsEdit',
> + header: gettext('Virtiofs') + ' (' + confid +')',
> + };
> + }
>
> var sorterFn = function(rec1, rec2) {
> var v1 = rec1.data.key;
> @@ -595,6 +605,7 @@ Ext.define('PVE.qemu.HardwareView', {
> const noVMConfigDiskPerm = !caps.vms['VM.Config.Disk'];
> const noVMConfigCDROMPerm = !caps.vms['VM.Config.CDROM'];
> const noVMConfigCloudinitPerm = !caps.vms['VM.Config.Cloudinit'];
> + const noVMConfigOptionsPerm = !caps.vms['VM.Config.Options'];
>
> me.down('#addUsb').setDisabled(noHWPerm || isAtUsbLimit());
> me.down('#addPci').setDisabled(noHWPerm || isAtLimit('hostpci'));
> @@ -604,6 +615,7 @@ Ext.define('PVE.qemu.HardwareView', {
> me.down('#addRng').setDisabled(noSysConsolePerm || isAtLimit('rng'));
> efidisk_menuitem.setDisabled(noVMConfigDiskPerm || isAtLimit('efidisk'));
> me.down('#addTpmState').setDisabled(noVMConfigDiskPerm || isAtLimit('tpmstate'));
> + me.down('#addVirtiofs').setDisabled(noVMConfigOptionsPerm || isAtLimit('virtiofs'));
> me.down('#addCloudinitDrive').setDisabled(noVMConfigCDROMPerm || noVMConfigCloudinitPerm || hasCloudInit);
>
> if (!rec) {
> @@ -748,6 +760,13 @@ Ext.define('PVE.qemu.HardwareView', {
> disabled: !caps.nodes['Sys.Console'],
> handler: editorFactory('RNGEdit'),
> },
> + {
> + text: gettext("Virtiofs"),
> + itemId: 'addVirtiofs',
> + iconCls: 'fa fa-folder',
> + disabled: !caps.nodes['Sys.Console'],
> + handler: editorFactory('VirtiofsEdit'),
> + },
> ],
> }),
> },
> diff --git a/www/manager6/qemu/VirtiofsEdit.js b/www/manager6/qemu/VirtiofsEdit.js
> new file mode 100644
> index 00000000..77199928
> --- /dev/null
> +++ b/www/manager6/qemu/VirtiofsEdit.js
> @@ -0,0 +1,143 @@
> +Ext.define('PVE.qemu.VirtiofsInputPanel', {
> + extend: 'Proxmox.panel.InputPanel',
> + xtype: 'pveVirtiofsInputPanel',
> + onlineHelp: 'qm_virtiofs',
> +
> + insideWizard: false,
> +
> + onGetValues: function(values) {
> + var me = this;
> + var confid = me.confid;
> + var params = {};
> + delete values.delete;
> + params[confid] = PVE.Parser.printPropertyString(values, 'dirid');
> + return params;
> + },
> +
> + setSharedfiles: function(confid, data) {
> + var me = this;
> + me.confid = confid;
> + me.virtiofs = data;
> + me.setValues(me.virtiofs);
> + },
> + initComponent: function() {
> + let me = this;
> +
> + me.nodename = me.pveSelNode.data.node;
> + if (!me.nodename) {
> + throw "no node name specified";
> + }
> + me.items = [
> + {
> + xtype: 'pveDirMapSelector',
> + emptyText: 'dirid',
> + nodename: me.nodename,
> + fieldLabel: gettext('Directory ID'),
> + name: 'dirid',
> + allowBlank: false,
> + },
> + {
> + xtype: 'displayfield',
> + userCls: 'pmx-hint',
> + value: gettext('Directory Mappings can be managed under Datacenter -> Resource Mapping'),
> + },
> + ];
> + me.advancedItems = [
> + {
> + xtype: 'proxmoxKVComboBox',
> + fieldLabel: gettext('Cache'),
> + name: 'cache',
> + value: '__default__',
> + deleteDefaultValue: false,
> + comboItems: [
> + ['__default__', Proxmox.Utils.defaultText + ' (auto)'],
> + ['auto', 'auto'],
> + ['always', 'always'],
> + ['metadata', 'metadata'],
> + ['never', 'never'],
> + ],
> + },
> + {
> + xtype: 'proxmoxcheckbox',
> + fieldLabel: gettext('Writeback cache'),
> + name: 'writeback',
> + },
> + {
> + xtype: 'proxmoxcheckbox',
> + fieldLabel: gettext('Enable xattr support'),
> + name: 'expose-xattr',
> + },
> + {
> + xtype: 'proxmoxcheckbox',
> + fieldLabel: gettext('Enable POSIX ACLs support (implies xattr support)'),
> + name: 'expose-acl',
> + listeners: {
> + change: function(f, value) {
> + let xattr = me.down('field[name=expose-xattr]');
> + xattr.setDisabled(value);
> + xattr.setValue(value);
> + },
> + },
> + },
> + {
> + xtype: 'proxmoxcheckbox',
> + fieldLabel: gettext('Allow Direct IO'),
> + name: 'direct-io',
> + },
> + ];
> +
> + me.virtiofs = {};
> + me.confid = 'virtiofs0';
> + me.callParent();
> + },
> +});
> +
> +Ext.define('PVE.qemu.VirtiofsEdit', {
> + extend: 'Proxmox.window.Edit',
> +
> + subject: gettext('Filesystem Passthrough'),
> +
> + initComponent: function() {
> + var me = this;
> +
> + me.isCreate = !me.confid;
> +
> + var ipanel = Ext.create('PVE.qemu.VirtiofsInputPanel', {
> + confid: me.confid,
> + pveSelNode: me.pveSelNode,
> + isCreate: me.isCreate,
> + });
> +
> + Ext.applyIf(me, {
> + items: ipanel,
> + });
> +
> + me.callParent();
> +
> + me.load({
> + success: function(response) {
> + me.conf = response.result.data;
> + var i, confid;
> + if (!me.isCreate) {
> + var value = me.conf[me.confid];
> + var virtiofs = PVE.Parser.parsePropertyString(value, "dirid");
> + if (!virtiofs) {
> + Ext.Msg.alert(gettext('Error'), 'Unable to parse virtiofs options');
> + me.close();
> + return;
> + }
> + ipanel.setSharedfiles(me.confid, virtiofs);
> + } else {
> + for (i = 0; i < PVE.Utils.hardware_counts.virtiofs; i++) {
> + confid = 'virtiofs' + i.toString();
> + if (!Ext.isDefined(me.conf[confid])) {
> + me.confid = confid;
> + break;
> + }
> + }
> + ipanel.setSharedfiles(me.confid, {});
> + }
> + },
> + });
> + },
> +});
_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
prev parent reply other threads:[~2025-04-07 12:03 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-04-04 16:08 [pve-devel] [PATCH cluster/guest-common/docs/qemu-server/manager v16 0/12] virtiofs Markus Frank
2025-04-04 16:08 ` [pve-devel] [PATCH cluster v16 1/12] add mapping/dir.cfg for resource mapping Markus Frank
2025-04-04 17:03 ` [pve-devel] applied: " Thomas Lamprecht
2025-04-04 16:08 ` [pve-devel] [PATCH guest-common v16 2/12] add dir mapping section config Markus Frank
2025-04-07 9:58 ` [pve-devel] applied: " Thomas Lamprecht
2025-04-04 16:08 ` [pve-devel] [PATCH docs v16 3/12] add doc section for the shared filesystem virtio-fs Markus Frank
2025-04-04 16:08 ` [pve-devel] [PATCH qemu-server v16 4/12] control: add virtiofsd as runtime dependency for qemu-server Markus Frank
2025-04-07 10:43 ` Thomas Lamprecht
2025-04-07 10:59 ` Markus Frank
2025-04-07 12:00 ` Thomas Lamprecht
2025-04-07 12:03 ` Markus Frank
2025-04-04 16:08 ` [pve-devel] [PATCH qemu-server v16 5/7] fix #1027: virtio-fs support Markus Frank
2025-04-04 16:08 ` [pve-devel] [PATCH qemu-server v16 6/12] migration: check_local_resources for virtiofs Markus Frank
2025-04-04 16:08 ` [pve-devel] [PATCH qemu-server v16 7/12] disable snapshot (with RAM) and hibernate with virtio-fs devices Markus Frank
2025-04-04 16:08 ` [pve-devel] [PATCH manager v16 08/12] api: add resource map api endpoints for directories Markus Frank
2025-04-04 16:08 ` [pve-devel] [PATCH manager v16 09/12] ui: add edit window for dir mappings Markus Frank
2025-04-04 16:08 ` [pve-devel] [PATCH manager v16 10/12] ui: add resource mapping view for directories Markus Frank
2025-04-07 9:51 ` Filip Schauer
2025-04-04 16:08 ` [pve-devel] [PATCH manager v16 11/12] ui: form: add selector for directory mappings Markus Frank
2025-04-04 16:08 ` [pve-devel] [PATCH manager v16 12/12] ui: add options to add virtio-fs to qemu config Markus Frank
2025-04-07 12:03 ` Filip Schauer [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=544b2160-853c-4cf8-bdbe-ce0c63215a09@proxmox.com \
--to=f.schauer@proxmox.com \
--cc=pve-devel@lists.proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal