Re: [pve-devel] [PATCH manager 1/1] Fix #7175: use timedatectl for timezone handling

[Stefan Mayr <stefan@mayr-stefan.de>]

Am 16.01.2026 um 22:30 schrieb Stefan Mayr:
> Am 15.01.2026 um 11:25 schrieb Fabian Grünbichler:
>> On December 25, 2025 9:05 am, Stefan Mayr wrote:
>>> Suggested-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
>>> Signed-off-by: Stefan Mayr <stefan@mayr-stefan.de>
>>> ---
>>>   PVE/API2/Nodes.pm | 9 +++++++--
>>>   1 file changed, 7 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/PVE/API2/Nodes.pm b/PVE/API2/Nodes.pm
>>> index 6a6465b6..54ed712e 100644
>>> --- a/PVE/API2/Nodes.pm
>>> +++ b/PVE/API2/Nodes.pm
>>> @@ -1582,10 +1582,15 @@ __PACKAGE__->register_method({
>>>       code => sub {
>>>           my ($param) = @_;
>>> +        my $timezone;
>>>           my $ctime = time();
>>>           my $ltime = timegm_nocheck(localtime($ctime));
>>> +        PVE::Tools::run_command(
>>> +            ['timedatectl', 'show', '--property=Timezone', '--value'],
>>> +            outfunc => sub { $timezone //= shift },
>>> +        );
>>>           my $res = {
>>> -            timezone => PVE::INotify::read_file('timezone'),
>>> +            timezone => $timezone,
>>>               time => $ctime,
>>>               localtime => $ltime,
>>>           };
>>> @@ -1619,7 +1624,7 @@ __PACKAGE__->register_method({
>>>       code => sub {
>>>           my ($param) = @_;
>>> -        PVE::INotify::write_file('timezone', $param->{timezone});
>>> +        PVE::Tools::run_command(['timedatectl', 'set-timezone', 
>>> $param->{timezone}]);
>>
>> should we restrict this parameter? AFAICT, only [a-zA-Z][a-zA-Z/_\-]*
>> are currently valid in timezone values?
> 
> Correct me if I'm wrong: this is already done in pve-commen:src/PVE/ 
> JSONSchema.pm: pve_verify_timezone.
> 
>> alternatively we could query the valid names first (list-timezones) and
>> then check that the passed value is contained in that set..
> 
> The current pve_verify_timezone function still uses /usr/share/zoneinfo/ 
> zone.tab which seems to be a subset of timedatectl list-timezones (418 
> vs 598 entries). That could be subject of another improvement.

vs 
https://git.proxmox.com/?p=proxmox-widget-toolkit.git;a=blob_plain;f=src/data/TimezoneStore.js;hb=HEAD 
which uses a static list instead of querying the API for valid values.

>>>           return;
>>>       },
>>> -- 
>>> 2.34.1
>>>

_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel