Re: [pve-devel] [PATCH pve-zsync 0/1] Allow pve-zsync jobs to share dest

Re: [pve-devel] [PATCH pve-zsync 0/1] Allow pve-zsync jobs to share dest

[Bruce Wainer]

 Hello,
It has been 5 months since my patch has been applied, however the version
for pve-zsync has not been incremented and this patch is not in the version
presented by the repo. What needs to be done?
Thank you,
Bruce Wainer

On Tue, Jun 23, 2020 at 4:48 AM Wolfgang Link <w.link@proxmox.com> wrote:

> Hi Bruce,
>
> Once you've submitted the CLA, a member of the release team will take care
> of this patch.
> You will be notified when it is committed or when something is missing.
>
> Regards
>
> Wolfgang
>
> > On 06/22/2020 2:38 PM Bruce Wainer <bruce@brucewainer.com> wrote:
> >
> >
> > Wolfgang,
> > Thanks for the confirmation. What is the next step?
> > Thank you,
> > Bruce
> >
> > > On Jun 22, 2020, at 7:43 AM, Wolfgang Link <w.link@proxmox.com> wrote:
> > >
> > > Look good to me
> > > I tested it and it works. There are no upgrade problems.
> > > Even if jobs already exist.
> > >
> > > Regards
> > >
> > > Wolfgang
> > >
> > >> On 06/17/2020 6:44 AM Wolfgang Link <w.link@proxmox.com> wrote:
> > >>
> > >>
> > >> Hi,
> > >>
> > >> thank you for this patch and the work.
> > >> I will look at this patch and give you feedback.
> > >>
> > >> Regards
> > >> Wolfgang
> > >>
> > >>>> On 06/16/2020 8:53 PM Bruce Wainer <brwainer@gmail.com> wrote:
> > >>>
> > >>>
> > >>> By flipping Source and Dest in snapshot_get and snapshot_exist, we
> can allow
> > >>> multiple sync jobs to share the same source.
> > >>> snapshot_get now checks the destination instead of source, and sets
> last_sync to
> > >>> the last snapshot regardless of name. old_sync and whether to delete
> it is still
> > >>> based on the job/name.
> > >>> snapshot_exist now checks the source instead of the destination.
> > >>> Other functions and/or their calls are changed to match the new
> situation.
> > >>>
> > >>> Bruce Wainer (1):
> > >>>  pve-zsync: Flip Source and Dest in functions to so jobs can share
> Dest
> > >>>
> > >>> pve-zsync | 42 +++++++++++++++++++++++++-----------------
> > >>> 1 file changed, 25 insertions(+), 17 deletions(-)
> > >>>
> > >>> --
> > >>> 2.20.1
> > >>>
> > >>> _______________________________________________
> > >>> pve-devel mailing list
> > >>> pve-devel@pve.proxmox.com
> > >>> https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
> > >>
> > >> _______________________________________________
> > >> pve-devel mailing list
> > >> pve-devel@pve.proxmox.com
> > >> https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
> > >
> > > Best Regards,
> > > Wolfgang Link
> > > w.link@proxmox.com
> > > http://www.proxmox.com
> > > ____________________________________________________________________
> > > Proxmox Server Solutions GmbH
> > > Bräuhausgasse 37, 1050 Vienna, Austria
> > > Commercial register no.: FN 258879 f
> > > Registration office: Handelsgericht Wien
> > >
>
>

Re: [pve-devel] [PATCH pve-zsync 0/1] Allow pve-zsync jobs to share dest

[Dietmar Maurer]

> It has been 5 months since my patch has been applied, however the version
> for pve-zsync has not been incremented and this patch is not in the version
> presented by the repo. What needs to be done?

Ok, just bumped the version and created a new package. So this will be part
of the next release (soon) ...

Re: [pve-devel] [PATCH pve-zsync 0/1] Allow pve-zsync jobs to share dest

[Bruce Wainer]

Awesome, thank you very much!

On Thu, Nov 26, 2020 at 12:06 AM Dietmar Maurer <dietmar@proxmox.com> wrote:

> > It has been 5 months since my patch has been applied, however the version
> > for pve-zsync has not been incremented and this patch is not in the
> version
> > presented by the repo. What needs to be done?
>
> Ok, just bumped the version and created a new package. So this will be part
> of the next release (soon) ...
>
>