From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by lists.proxmox.com (Postfix) with ESMTPS id 02A12F0BF for ; Wed, 14 Dec 2022 09:27:32 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id D7DF527431 for ; Wed, 14 Dec 2022 09:27:31 +0100 (CET) Received: from proxmox-new.maurer-it.com (proxmox-new.maurer-it.com [94.136.29.106]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by firstgate.proxmox.com (Proxmox) with ESMTPS for ; Wed, 14 Dec 2022 09:27:30 +0100 (CET) Received: from proxmox-new.maurer-it.com (localhost.localdomain [127.0.0.1]) by proxmox-new.maurer-it.com (Proxmox) with ESMTP id 51BEF43CD2 for ; Wed, 14 Dec 2022 09:27:30 +0100 (CET) Message-ID: <84f29d93-4941-6c16-1e99-bea166bb31e9@proxmox.com> Date: Wed, 14 Dec 2022 09:27:28 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.5.0 To: Proxmox VE development discussion , Noel Ullreich References: <20221213145624.110018-1-n.ullreich@proxmox.com> Content-Language: en-US From: Stefan Sterz In-Reply-To: <20221213145624.110018-1-n.ullreich@proxmox.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-SPAM-LEVEL: Spam detection results: 0 AWL -0.233 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment NICE_REPLY_A -0.001 Looks like a legit reply (A) POISEN_SPAM_PILL 0.1 Meta: its spam POISEN_SPAM_PILL_1 0.1 random spam to be learned in bayes POISEN_SPAM_PILL_3 0.1 random spam to be learned in bayes SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record URIBL_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to URIBL was blocked. See http://wiki.apache.org/spamassassin/DnsBlocklists#dnsbl-block for more information. [wikipedia.org] Subject: Re: [pve-devel] [PATCH pve-docs v2] updated tags docs X-BeenThere: pve-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox VE development discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 14 Dec 2022 08:27:32 -0000 On 12/13/22 15:56, Noel Ullreich wrote: > updated the tags doc with slight rephrasing, corrected grammar, > and fixed typos. > > Signed-off-by: Noel Ullreich > --- > changes since v1: > * removed changes in datacenter.cfg.5-opts.adoc since it is an > autogenerated file (thanks @Dominik;) ) > pve-gui.adoc | 70 +++++++++++++++++++++++++--------------------------- > 1 file changed, 34 insertions(+), 36 deletions(-) > > diff --git a/pve-gui.adoc b/pve-gui.adoc > index ff95cfb..d6d9426 100644 > --- a/pve-gui.adoc > +++ b/pve-gui.adoc > @@ -386,68 +386,66 @@ Tags > > [thumbnail="screenshot/gui-qemu-summary-tags-edit.png", float="left"] > > -For organizational purposes, it is possible to set `tags` on guests. These have > -currently only informational value and they are display by default in the gui > -in two places, the `Resource Tree` and in the status line when a guest is > -selected. > +For organizational purposes, it is possible to set `tags` for guests. > +Currently, these only provide informational value to users. > +Tags are displayed in two places in the web interface: in the `Resource Tree` and > +in the status line when a guest ist selected. > typo: ist/is > -They can be added, edited and removed in that status line of the guest by > -clicking on the `pencil` icon. After the tags have the desired values, you > -can click on the `check mark` button to apply the changes, or click the > -`X` button to cancel you editing. > +Tags can be added, edited, and removed in the status line of the guest by > +clicking on the `pencil` icon. You can add multiple tags by pressing the `+` > +button and remove them by pressing the `-` button and save or cancel the changes > +with the `checkmark` and `x` button respectively. > imo: it is a bit clumsy to use three "and"s in a row here. it might be nice to replace the first one with a comma (…`+` button, remove them…) or split this up into two sentences. consider something like this: You can add multiple tags by pressing the `+` button and remove them with the `-` button. To save or cancel the changes, you can use the `check mark` and `x` button, respectively. on a pedantic sidenote: "check mark" is correct in us english. "checkmark" exists in philippine english [1]. [1]: https://en.wikipedia.org/wiki/Check_mark > -The tags can also be set via the CLI, where multiple tags are semi-colon > -separated, for example: > +Tags can also be set via the CLI, where multiple tags are separated by semicolons. > +For example: > > ---- > # qm set ID --tags myfirsttag;mysecondtag > ---- > > -Styling > -~~~~~~~ > +Style Configuration > +~~~~~~~~~~~~~~~~~~~ > > [thumbnail="screenshot/gui-datacenter-tag-style.png"] > > -To adapt to various use cases and preferences, there are some options in the > -datacenter configuration that helps an admin control the behaviour and style > -of tags, such as the colors, ordering, and basic permission settings. > - > - > -By default the tag colors are derived from their text in a deterministic way, > -but can be overridden in the datacenter configuration (The background color as > -well as the text color) as hexadecimal RGB values. You can also set these > -overrides via the CLI, for example: > +By default, the tag colors are derived from their text in a deterministic way. > +The color, shape in the resource tree, and case-sensitivity, as well as how tags > +are sorted, can be customized. This can be done via the web interface under > +__Datacenter -> Options -> Tag Style Override__. Alternatively, this can be done > +via the CLI. For example: > > ---- > # pvesh set /cluster/options --tag-style color-map=example:000000:FFFFFF > ---- > > -Sets the background color of the tag `example` to black (#000000) and the text > +sets the background color of the tag `example` to black (#000000) and the text > color to white (#FFFFFF). > > -You can also change the style of the tags in the tree and if the tags should > -be alphabetically sorted (the default) or should retain their order from the > -guest configuration. > - > Permissions > ~~~~~~~~~~~ > > [thumbnail="screenshot/gui-datacenter-options.png"] > > By default, users with the privilege `VM.Config.Options` on a guest (`/vms/ID`) > -can set any tags they want. If you want to restrict this behaviour, it is > -possible to set various modes via the datacenter configuration: > +can set any tags they want (see > +xref:pveum_permission_management[Permission Management]). Under __Datacenter -> > +Options -> User Tag Access__ permissions for tag access can be set: imo: this flowed better with the "If" clause there. consider: By default, users with the privilege `VM.Config.Options` on a guest (`/vms/ID`) can set any tags they want (see xref:pveum_permission_management[Permission Management]). If you want to restrict this behavior, appropriate permissions can be set under __Datacenter -> Options -> User Tag Access__: > + > +* `free`: users are not restricted in setting tags (Default) > +* `list`: users can set tags based on a predefined list of tags > +* `existing`: like list but users can also use already existing tags > +* `none`: users are restricted from using tags > + > +The same can also be done via the CLI. > > -* a list of allowed tags > -* existing tags > -* no tags at all > +Note that a user with the `Sys.Modify` privileges on `/` is always able to set > +or delete any tags, regardless of the settings here. Additionally, there is a > +configurable list of `registered tags` which can only be added and removed by > +users with the privilege `Sys.Modify` on `/`. The list of registered tags be mising can? "The list of registered tags *can* be edited…" > +edited under __Datacenter -> Options -> Registered Tags__ or via the CLI. > > -Note that a user with `Sys.Modify` on `/` is always able to set or delete > -any tags, regardless of the settings here. Additionally, there is a configurable > -list of `registered tags` which will also require the privilege `Sys.Modify` > -on `/` to be added or removed independent of the mode. > > -For more details on the exact options, see the > +For more details on the exact options and how to invoke them in the CLI, see > xref:datacenter_configuration_file[Datacenter Configuration]. > > ifdef::wiki[]