[pve-devel] applied-series (rebased): [PATCH pve-storage 1/6] PVE/Storage/Plugin.pm: add ctime for backup files

Thomas Lamprecht t.lamprecht at proxmox.com
Fri Mar 6 19:36:24 CET 2020


On 3/4/20 11:46 AM, Dietmar Maurer wrote:
> Signed-off-by: Dietmar Maurer <dietmar at proxmox.com>
> ---
>  PVE/Storage/Plugin.pm | 6 ++++++
>  1 file changed, 6 insertions(+)
> 


I applied this series, rebased it a bit though:

* replacing the file paths in the commit subjects with a bit human friendlier
  and shorter tags - one can always use `git show --stat` to quickly get an
  overview about what files got changed

* I moved patch "[PATCH pve-storage 3/6] PVE/API2/Storage/Content.pm - index:
  add ctime to return schema" in front, as else there would be a temporary breakage
  due to the result schema verification (ctime already present but not yet defined)

* dropped the "Currently only set for backup volumes." from the ctime property
  description as it's not true after patch 4/6 got applied.

* very minor "code cleanup" followup, effectively just moving two statements into
  all separate lines, just mentioning here for completeness sake.

Anyway, thanks a lot for this!




More information about the pve-devel mailing list