[pve-devel] applied-series: [PATCH v2 manager 1/3] ui: rbd: cephfs: add keyring/secret field for external clusters
Aaron Lauterer
a.lauterer at proxmox.com
Fri Jan 28 12:24:39 CET 2022
On 1/28/22 12:23, Fabian Ebner wrote:
> Am 26.01.22 um 11:18 schrieb Aaron Lauterer:
>> Manual switching of xtype because binding 'hidden' does not work with
>> pmxDisplayEditField.
>>
>> Signed-off-by: Aaron Lauterer <a.lauterer at proxmox.com>
>> ---
>> changes: added spaces in front of `?` operator that were missed.
>>
>>
>> From previous iteration for this patch:
>>
>> Reviewed-by: Fabian Ebner <f.ebner at proxmox.com>
>> Tested-by: Fabian Ebner <f.ebner at proxmox.com>
>>
>> www/manager6/storage/CephFSEdit.js | 39 ++++++++++++++++++++----------
>> www/manager6/storage/RBDEdit.js | 11 +++++++++
>> 2 files changed, 37 insertions(+), 13 deletions(-)
>>
>
> thanks! Added two small followups after a brief discussion with Thomas,
> namely using gettext('Secret Key') for the field label for CephFS and
> making the "is done automatically for local cluster" more obvious in the
> docs. Also added the missing f in the second commit title again ;)
thanks for that... :D git commit --amend and not seeing it...
>
>
More information about the pve-devel
mailing list