From: Dominik Csapak <d.csapak@proxmox.com>
To: Lukas Wagner <l.wagner@proxmox.com>,
Proxmox Datacenter Manager development discussion
<pdm-devel@lists.proxmox.com>
Subject: Re: [pdm-devel] [PATCH datacenter-manager] server: fake remotes: adapt to changed ClientFactory trait
Date: Wed, 14 May 2025 10:02:39 +0200 [thread overview]
Message-ID: <b849698b-9a95-43a8-bb08-d1795dbcdaf6@proxmox.com> (raw)
In-Reply-To: <4bad071f-f49e-4545-b19a-bec24f30ee3b@proxmox.com>
On 5/14/25 09:49, Lukas Wagner wrote:
> On 2025-05-14 09:30, Dominik Csapak wrote:
>> On 5/13/25 15:25, Lukas Wagner wrote:
>>> For the record, I already sent a patch for this in a previous patch series.
>>>
>>> https://lore.proxmox.com/pdm-devel/20250418083210.74982-6-l.wagner@proxmox.com/T/#u
>>>
>>
>> yes, i noticed just before you sent this mail (see my other response, that weirdly
>> only contains a html part....)
>>
>> but good to see that we basically have sent the same diff ^^
>>
>> also, would it make sense to extend the fake remote part?
>>
>> 3 improvements i noticed that would be nice imho
>> * simulate more api calls
>> * fix some values (e.g. maxdisk is smaller than disk for hosts)
>> * add some pseudo random dynamic values? so that it changes over time a bit (e.g. via a configurable jitter percentage)
>> this would be nice to fully test the ui without having to setup multiple test clusters
>>
>
> Sure, makes sense. Feel free to send some patches, otherwise I can have a go at it
> when I have the time. The stuff that I already implemented mostly served my own development
> needs, for sure there are things missing for other areas of the system.
>
> Another thing that I would like to add is to have a simple way to switch between
> multiple fake "profiles" (e.g. different PDM setups sizes, tiny, medium, huge) so
> that it is easier to do performance testing in a standardized way. Already
> talked to Thomas about this, he provided me some of the variables for these profiles (e.g.
> number of remotes, guests per remote, etc.) - but haven't had the time yet to integrate that.
>
sure, I'll send patches for that. I noticed already that in some areas the UI is
not optimal to use when getting to certain large numbers (mostly layout issues, not performance)
so it'll be good to have for ui testing too
as for different profiles, for the short term it could be enough to check in sample
json files into the git repository together with the documentation.
then it'd be a simply copy & restart
(e.g. i have setup my systemd services to provide the ENV variable)
_______________________________________________
pdm-devel mailing list
pdm-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel
prev parent reply other threads:[~2025-05-14 8:02 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-05-13 13:04 Dominik Csapak
2025-05-13 13:25 ` Lukas Wagner
2025-05-14 7:30 ` Dominik Csapak
2025-05-14 7:49 ` Lukas Wagner
2025-05-14 8:02 ` Dominik Csapak [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=b849698b-9a95-43a8-bb08-d1795dbcdaf6@proxmox.com \
--to=d.csapak@proxmox.com \
--cc=l.wagner@proxmox.com \
--cc=pdm-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