[PATCH manager 1/3] api: backup: drop starttime and dow from return schema

[Lukas Wagner <l.wagner@proxmox.com>]

For backup jobs returned by the API, the `dow` and `starttime`
parameters are only ever set for legacy-jobs that are defined in the old
vzdump.cron configuration file. The newer backup jobs based on the
jobs.cfg file were introduced in [0]. Ever since then, any new or
updated backup job was stored in jobs.cfg, meaning that it is rather
unlikely to have any backup jobs in the old configuration file anyways,
from which subsequently follows that the 'dow' and 'starttime' params
will hardly ever be included in the response anyways. And even if a
legacy job is read from the old file, an appropriate 'schedule' key is
automatically synthesized from 'dow' and 'starttime'.

In summary:
  - 'dow' and 'starttime' will hardly ever be set
  - 'schedule' will ALWAYS be set
  - any client needed to understand the 'schedule' parameter since 7.1
    anyways

It should therefore be fine to just drop both old keys from the return
schema; this is a good chance to clean up some legacy cruft.

[0]: https://git.proxmox.com/?p=pve-manager.git;a=commitdiff;h=305921b1a

Suggested-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>
---
 PVE/API2/Backup.pm | 15 ---------------
 1 file changed, 15 deletions(-)

diff --git a/PVE/API2/Backup.pm b/PVE/API2/Backup.pm
index 1d6d4257..2ab78834 100644
--- a/PVE/API2/Backup.pm
+++ b/PVE/API2/Backup.pm
@@ -129,21 +129,6 @@ my $backup_job_return_schema = PVE::VZDump::Common::json_config_properties({
         maxLength => 128,
         optional => 1,
     },
-    starttime => {
-        type => 'string',
-        description => "Job start-time (server timezone); deprecated and replaced by schedule.",
-        pattern => '\d{1,2}:\d{1,2}',
-        typetext => 'HH:MM',
-        optional => 1,
-    },
-    dow => {
-        type => 'string',
-        format => 'pve-day-of-week-list',
-        optional => 1,
-        description => "Day of week selection; deprecated and replaced by schedule.",
-        requires => 'starttime',
-        default => ALL_DAYS,
-    },
     enabled => {
         type => 'boolean',
         optional => 1,
-- 
2.47.3