public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Bruce Wainer <brwainer@gmail.com>
To: Wolfgang Link <w.link@proxmox.com>
Cc: PVE development discussion <pve-devel@pve.proxmox.com>
Subject: Re: [pve-devel] [PATCH pve-zsync 0/1] Allow pve-zsync jobs to share dest
Date: Wed, 25 Nov 2020 17:12:24 -0500	[thread overview]
Message-ID: <CAPEXuiwFvLa7uct_h_t9UPJpcPrC0jzvZVgNLbNfNRVvYxdDsw@mail.gmail.com> (raw)
In-Reply-To: <1101863268.50.1592902077180@webmail.proxmox.com>

 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
> > >
>
>


       reply	other threads:[~2020-11-25 22:13 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <817953729.92.1592826225983@webmail.proxmox.com>
     [not found] ` <D1795812-455D-4EAB-A0DD-0C323B907863@brucewainer.com>
     [not found]   ` <1101863268.50.1592902077180@webmail.proxmox.com>
2020-11-25 22:12     ` Bruce Wainer [this message]
2020-11-26  5:06       ` Dietmar Maurer
2020-11-26  8:05         ` Bruce Wainer

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=CAPEXuiwFvLa7uct_h_t9UPJpcPrC0jzvZVgNLbNfNRVvYxdDsw@mail.gmail.com \
    --to=brwainer@gmail.com \
    --cc=pve-devel@pve.proxmox.com \
    --cc=w.link@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