Re: [ovs-dev] [PATCH 1/2] appveyor: Don't download OpenSSL.

2023-07-03 Thread Alin Serdean
Thanks for taking care of it Ilya! Alin. Sent from phone > On 3 Jul 2023, at 14:59, Ilya Maximets wrote: > > On 7/1/23 01:55, Ilya Maximets wrote: >>> On 7/1/23 01:43, Ilya Maximets wrote: >>> On 6/8/23 12:07, Alin Serdean wrote: Ack, I will try to see if I can get GH actions. It

Re: [ovs-dev] [PATCH 1/2] appveyor: Don't download OpenSSL.

2023-07-03 Thread Ilya Maximets
On 7/1/23 01:55, Ilya Maximets wrote: > On 7/1/23 01:43, Ilya Maximets wrote: >> On 6/8/23 12:07, Alin Serdean wrote: >>> Ack, I will try to see if I can get GH actions. >>> It would also be better to have everything integrated in one place. >>> >>> I remember the discussion and the PR… I had some

Re: [ovs-dev] [PATCH 1/2] appveyor: Don't download OpenSSL.

2023-06-30 Thread Ilya Maximets
On 7/1/23 01:43, Ilya Maximets wrote: > On 6/8/23 12:07, Alin Serdean wrote: >> Ack, I will try to see if I can get GH actions. >> It would also be better to have everything integrated in one place. >> >> I remember the discussion and the PR… I had some comments which never got >> addressed. >>

Re: [ovs-dev] [PATCH 1/2] appveyor: Don't download OpenSSL.

2023-06-30 Thread Ilya Maximets
On 6/8/23 12:07, Alin Serdean wrote: > Ack, I will try to see if I can get GH actions. > It would also be better to have everything integrated in one place. > > I remember the discussion and the PR… I had some comments which never got > addressed. > > Windows is pretty slow when starting

Re: [ovs-dev] [PATCH 1/2] appveyor: Don't download OpenSSL.

2023-06-08 Thread Alin Serdean
Ack, I will try to see if I can get GH actions. It would also be better to have everything integrated in one place. I remember the discussion and the PR… I had some comments which never got addressed. Windows is pretty slow when starting processes ,especially if the builtin AV is not disabled

Re: [ovs-dev] [PATCH 1/2] appveyor: Don't download OpenSSL.

2023-06-08 Thread Ilya Maximets
On 6/8/23 11:01, Alin Serdean wrote: > > Ack. I will try to see if I can address it by the end of the week. > > Thanks for clarifying the questions. > > Would GH actions be better than appveyor? Functionally, I don't think they are that much different. But GHA is better integrated into our CI,

Re: [ovs-dev] [PATCH 1/2] appveyor: Don't download OpenSSL.

2023-06-08 Thread Alin Serdean
Ack. I will try to see if I can address it by the end of the week. Thanks for clarifying the questions. Would GH actions be better than appveyor? — Alin. On 2 Jun 2023, at 12:25, Ilya Maximets wrote: > > On 6/1/23 13:50, Ilya Maximets wrote: >>> On 5/31/23 23:05, Alin Serdean wrote: >>>

Re: [ovs-dev] [PATCH 1/2] appveyor: Don't download OpenSSL.

2023-06-02 Thread Ilya Maximets
On 6/1/23 13:50, Ilya Maximets wrote: > On 5/31/23 23:05, Alin Serdean wrote: >> >> That makes sense. >> >> We can leverage the  following commit: >> >> https://patchwork.ozlabs.org/project/openvswitch/patch/20201013124655.1408-1-aserd...@cloudbasesolutions.com/ >> >>

Re: [ovs-dev] [PATCH 1/2] appveyor: Don't download OpenSSL.

2023-06-01 Thread Ilya Maximets
On 5/31/23 23:05, Alin Serdean wrote: > > That makes sense. > > We can leverage the  following commit: > > https://patchwork.ozlabs.org/project/openvswitch/patch/20201013124655.1408-1-aserd...@cloudbasesolutions.com/ > >

Re: [ovs-dev] [PATCH 1/2] appveyor: Don't download OpenSSL.

2023-05-31 Thread Alin Serdean
That makes sense. We can leverage the following commit: https://patchwork.ozlabs.org/project/openvswitch/patch/20201013124655.1408-1-aserd...@cloudbasesolutions.com/ But I still need to fix the permissions. I’ll try to find some time and address it. I remember there were some discussions to

Re: [ovs-dev] [PATCH 1/2] appveyor: Don't download OpenSSL.

2023-05-31 Thread Ilya Maximets
On 5/31/23 22:36, Alin Serdean wrote: > > It would be best to change the link with the latest version of OpenSSL. That > will ensure there are no mishaps . I think the problem here is that slproweb.com only provides OpenSSL 1.1.1+ right now and our build system doesn't work with that version.

Re: [ovs-dev] [PATCH 1/2] appveyor: Don't download OpenSSL.

2023-05-31 Thread Alin Serdean
It would be best to change the link with the latest version of OpenSSL. That will ensure there are no mishaps . Alin. > > On 31 May 2023, at 21:23, Ilya Maximets wrote: > > OpenSSL is already available in the exact location we need it [1]. > Also, the download itself fails for a long time

[ovs-dev] [PATCH 1/2] appveyor: Don't download OpenSSL.

2023-05-31 Thread Ilya Maximets
OpenSSL is already available in the exact location we need it [1]. Also, the download itself fails for a long time already, because the version we're trying to download is not available. [1] https://www.appveyor.com/docs/windows-images-software/#tools Signed-off-by: Ilya Maximets ---