[pve-devel] [PATCH manager 1/3] api: ceph: add endpoint to fetch config keys
a.lauterer at proxmox.com
Mon Mar 13 13:58:01 CET 2023
On 3/11/23 18:07, Thomas Lamprecht wrote:
> Am 08/03/2023 um 13:14 schrieb Dominik Csapak:
>> high level:
>> as you mentioned the path 'configkey' is not really optimal
>> i recently mentioned off-list that we could clean this up on
>> the next breaking major release with a breaking api change:
>> have a 'config' dir and a
>> and 'key'( or 'value') api endpoint inside
>> that represents the different things
>> for now a possible change could be to do it in 'config'
>> but with a new parameter, though that's also not ideal
>> any further ideas/suggestions @Thomas?
> We could add the full
> now already, re-mount the 'cfg/raw' one on the current 'config' (or just keep
> the code duplicated, not much gain if we remove it anyway) one and then drop that
> old 'config' one in PVE 8.0; slightly hacky but IMO not that much.
> Might want to check what other uses of config, cfg, conf, configuration there are
> in API path's though, as ideally we keep the total unique count of them the same ;-)
AFAICT we basically only have "config" in the API paths according to the API
Viewer. So 'cfg' would be something new, not yet used.
I do like 'cfg' more than 'conf'. Once we dropped support for 'config', we could
wait a full major release and then move it back? Not sure but 'cfg' is also only
somewhat nice ;)
More information about the pve-devel