[pve-devel] [PATCH manager v2] Allow to set the firewall log rate limit and burst from the UI
Christian Ebner
c.ebner at proxmox.com
Mon Apr 1 14:45:11 CEST 2019
Signed-off-by: Christian Ebner <c.ebner at proxmox.com>
---
Version 2:
* Values are retained in config while disable/enable
* Removed disabling of the textfield and combobox when unchecking enable
in order to get the values instead of undefined.
* initComponent -> autoLoad: true
www/manager6/Makefile | 1 +
www/manager6/grid/FirewallOptions.js | 8 +++
www/manager6/window/FirewallLograteEdit.js | 94 ++++++++++++++++++++++++++++++
3 files changed, 103 insertions(+)
create mode 100644 www/manager6/window/FirewallLograteEdit.js
diff --git a/www/manager6/Makefile b/www/manager6/Makefile
index 962a3f4d..5ad70933 100644
--- a/www/manager6/Makefile
+++ b/www/manager6/Makefile
@@ -85,6 +85,7 @@ JSSRC= \
window/StartupEdit.js \
window/CephInstall.js \
window/FirewallEnableEdit.js \
+ window/FirewallLograteEdit.js \
panel/NotesView.js \
grid/ResourceGrid.js \
grid/PoolMembers.js \
diff --git a/www/manager6/grid/FirewallOptions.js b/www/manager6/grid/FirewallOptions.js
index 72fad34f..2952e854 100644
--- a/www/manager6/grid/FirewallOptions.js
+++ b/www/manager6/grid/FirewallOptions.js
@@ -103,6 +103,14 @@ Ext.define('PVE.FirewallOptions', {
} else if (me.fwtype === 'dc') {
add_boolean_row('enable', gettext('Firewall'), 0);
add_boolean_row('ebtables', 'ebtables', 1);
+ me.rows.log_ratelimit = {
+ header: gettext('Log rate limit'),
+ required: true,
+ defaultValue: 'enable=0',
+ editor: {
+ xtype: 'pveFirewallLograteEdit'
+ }
+ };
}
if (me.fwtype === 'dc' || me.fwtype === 'vm') {
diff --git a/www/manager6/window/FirewallLograteEdit.js b/www/manager6/window/FirewallLograteEdit.js
new file mode 100644
index 00000000..c48be192
--- /dev/null
+++ b/www/manager6/window/FirewallLograteEdit.js
@@ -0,0 +1,94 @@
+/*jslint confusion: true*/
+Ext.define('PVE.FirewallLograteInputPanel', {
+ extend: 'Proxmox.panel.InputPanel',
+ xtype: 'pveFirewallLograteInputPanel',
+
+ viewModel: {},
+
+ items: [
+ {
+ xtype: 'proxmoxcheckbox',
+ name: 'enable',
+ reference: 'enable',
+ fieldLabel: gettext('Enable'),
+ value: false
+ },
+ {
+ layout: 'hbox',
+ border: false,
+ items: [
+ {
+ xtype: 'numberfield',
+ name: 'rate',
+ fieldLabel: gettext('Log rate limit'),
+ minValue: 1,
+ maxValue: 99,
+ allowBlank: false,
+ flex: 2,
+ value: 1
+ },
+ {
+ html: '<div style="margin: auto; padding: 2.5px;"><b>/</b></div>'
+ },
+ {
+ xtype: 'proxmoxKVComboBox',
+ name: 'unit',
+ comboItems: [['second', 'second'], ['minute', 'minute'],
+ ['hour', 'hour'], ['day', 'day']],
+ allowBlank: false,
+ flex: 1,
+ value: 'second'
+ }
+ ]
+ },
+ {
+ xtype: 'numberfield',
+ name: 'burst',
+ fieldLabel: gettext('Log burst limit'),
+ minValue: 1,
+ maxValue: 99,
+ value: 5
+ }
+ ],
+
+ onGetValues: function(values) {
+ var me = this;
+
+ var vals = {};
+ vals.enable = values.enable !== undefined ? 1 : 0;
+ vals.rate = values.rate + '/' + values.unit;
+ vals.burst = values.burst;
+ var properties = PVE.Parser.printPropertyString(vals, undefined);
+ if (properties == '') {
+ return { 'delete': 'log_ratelimit' };
+ }
+ return { log_ratelimit: properties };
+ },
+
+ setValues: function(values) {
+ var me = this;
+
+ var properties = {};
+ if (values.log_ratelimit !== undefined) {
+ properties = PVE.Parser.parsePropertyString(values.log_ratelimit);
+ var matches = properties.rate.match(/^(\d+)\/(second|minute|hour|day)$/);
+ if (matches) {
+ properties.rate = matches[1];
+ properties.unit = matches[2];
+ }
+ }
+ me.callParent([properties]);
+ }
+});
+
+Ext.define('PVE.FirewallLograteEdit', {
+ extend: 'Proxmox.window.Edit',
+ xtype: 'pveFirewallLograteEdit',
+
+ subject: gettext('Log rate limit'),
+
+ items: [{
+ xtype: 'pveFirewallLograteInputPanel'
+ }],
+ autoLoad: true
+});
--
2.11.0
More information about the pve-devel
mailing list