Re: [VOTE] Release Apache Mina SSHD 2.9.3

2023-10-20 Thread Guillaume Nodet
+1

Le jeu. 12 oct. 2023 à 16:33, Guillaume Nodet  a écrit :

> I've staged a candidate release for 2.9.3 at:
> Official staging repo:
>   https://dist.apache.org/repos/dist/dev/mina/sshd/2.9.3/
> Maven staging repo:
>   https://repository.apache.org/content/repositories/orgapachemina-1088
> Git tag:
>   https://github.com/apache/mina-sshd/commits/sshd-2.9.3
> Changelog:
>
> https://github.com/apache/mina-sshd/blob/sshd-2.9.3/docs/changes/2.9.3.md
>
> Please review and vote !
>
> --
> 
> Guillaume Nodet
>
>

-- 

Guillaume Nodet


Re: [VOTE] Release Apache Mina SSHD 2.9.3

2023-10-17 Thread Emmanuel Lécharny

Guillaume, Thomas, thanks for the feedback.

Here, I think we are at a crossroad:
- some of the tests are pure unit tests (and generally don't require TC)
- some other tests are integration tests (and may require some more 
environment, which may mean TC)


I understand the need of having to check that SSHd is working in 
different envs, I have the exact same issue at ApacheDS, where I need to 
generate packages for Windows, MacOS, Linux (debian and RH). I can't do 
that anymore on my mac, due to the deprecation fo some of the tools by 
Apple.

Bottom line, I had two possibilities:
- using a Docker image locally to generate those packages
- or leaving the burden to the CI (my current choice)

What bothers me is that the default tests are now taking 25+ minute son 
my CoreI9 pretty fast machine, when it was just eating 5 mins more or 
less a few releases back.


Not that I'm running those tests frequently - I just do my duty as a PMC 
member when a release is called for (and as a side note, I wish all the 
PMC members do the same for the other MINA projects ;-) - but I'm afraid 
that with such long tests, committers will start committing before 
having ran tests.


In any case, it's not my decision to make, and I'm just offering my 
perception. Maybe it would be a valid option, as suggested by Guillaume, 
to split the tests into 2 categories, as explained at the top of my 
response, and having the integration ran by the CI only, unless 
specifically asked for through a profile or a parameter (like a 
-DintegrationTests for instance)


Your call, really. I can perfectly live with the current tests requiring 
TC, I have Docker installed on my machine. Maybe document this 
requirement somwehere could be a good option too (a BUILD_README.txt for 
instance?)


Anyway, thanks, and congrats for the release :-) !

On 17/10/2023 08:03, Guillaume Nodet wrote:

Le lun. 16 oct. 2023 à 22:02, Thomas Wolf  a écrit :


On 16.10.23 01:04 , Emmanuel Lécharny wrote:


My main concern is that we now need to install Docker to get the test
passing, due to the usage of TestContainer. I don't really like it, the
build should be self-supported.


The use of TestContainers isn't new.


Add to that it took 25mins to ran the tests. TC is clearly slowing down
the whole tests (it takes around 10 s to start a new container):

[INFO] Apache Mina SSHD :: Common support utilits . SUCCESS [01:30 min]
...
[INFO] Apache Mina SSHD :: Core ... SUCCESS [08:37 min]
[INFO] Apache Mina SSHD :: Mina ... SUCCESS [03:41 min]
[INFO] Apache Mina SSHD :: Netty .. SUCCESS [02:56 min]
...
[INFO] Apache Mina SSHD :: SFTP ... SUCCESS [04:27 min]
[INFO]



Frankly, I'm not a fan...


Do you know of a better and faster way of testing against different
versions of real OpenSSH (running on different OSes)?

Some of the tests even capture the OpenSSH debug logging and verify that
it indicates certain conditions.



I'm not sure there are other ways.
However, we could split tests between unit tests and integration tests (if
needed) and across different profiles.  Some profiles could only be
activated by default in CI.




Cheers,

Thomas



-
To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org
For additional commands, e-mail: dev-h...@mina.apache.org






--
*Emmanuel Lécharny* P. +33 (0)6 08 33 32 61
elecha...@apache.org

-
To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org
For additional commands, e-mail: dev-h...@mina.apache.org



Re: [VOTE] Release Apache Mina SSHD 2.9.3

2023-10-17 Thread Gary Gregory
I think the CI should run ALL tests. It's for the developer in a hurry to
disable some tests from the command line if they want to take the chance.

Gary

On Tue, Oct 17, 2023, 2:04 AM Guillaume Nodet  wrote:

> Le lun. 16 oct. 2023 à 22:02, Thomas Wolf  a écrit :
>
> > On 16.10.23 01:04 , Emmanuel Lécharny wrote:
> >
> > > My main concern is that we now need to install Docker to get the test
> > > passing, due to the usage of TestContainer. I don't really like it, the
> > > build should be self-supported.
> >
> > The use of TestContainers isn't new.
> >
> > > Add to that it took 25mins to ran the tests. TC is clearly slowing down
> > > the whole tests (it takes around 10 s to start a new container):
> > >
> > > [INFO] Apache Mina SSHD :: Common support utilits . SUCCESS [01:30 min]
> > > ...
> > > [INFO] Apache Mina SSHD :: Core ... SUCCESS [08:37 min]
> > > [INFO] Apache Mina SSHD :: Mina ... SUCCESS [03:41 min]
> > > [INFO] Apache Mina SSHD :: Netty .. SUCCESS [02:56 min]
> > > ...
> > > [INFO] Apache Mina SSHD :: SFTP ... SUCCESS [04:27 min]
> > > [INFO]
> > >
> 
> > >
> > >
> > > Frankly, I'm not a fan...
> >
> > Do you know of a better and faster way of testing against different
> > versions of real OpenSSH (running on different OSes)?
> >
> > Some of the tests even capture the OpenSSH debug logging and verify that
> > it indicates certain conditions.
> >
>
> I'm not sure there are other ways.
> However, we could split tests between unit tests and integration tests (if
> needed) and across different profiles.  Some profiles could only be
> activated by default in CI.
>
>
> >
> > Cheers,
> >
> >Thomas
> >
> >
> >
> > -
> > To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org
> > For additional commands, e-mail: dev-h...@mina.apache.org
> >
> >
>
> --
> 
> Guillaume Nodet
>


Re: [VOTE] Release Apache Mina SSHD 2.9.3

2023-10-17 Thread Guillaume Nodet
Le lun. 16 oct. 2023 à 22:02, Thomas Wolf  a écrit :

> On 16.10.23 01:04 , Emmanuel Lécharny wrote:
>
> > My main concern is that we now need to install Docker to get the test
> > passing, due to the usage of TestContainer. I don't really like it, the
> > build should be self-supported.
>
> The use of TestContainers isn't new.
>
> > Add to that it took 25mins to ran the tests. TC is clearly slowing down
> > the whole tests (it takes around 10 s to start a new container):
> >
> > [INFO] Apache Mina SSHD :: Common support utilits . SUCCESS [01:30 min]
> > ...
> > [INFO] Apache Mina SSHD :: Core ... SUCCESS [08:37 min]
> > [INFO] Apache Mina SSHD :: Mina ... SUCCESS [03:41 min]
> > [INFO] Apache Mina SSHD :: Netty .. SUCCESS [02:56 min]
> > ...
> > [INFO] Apache Mina SSHD :: SFTP ... SUCCESS [04:27 min]
> > [INFO]
> > 
> >
> >
> > Frankly, I'm not a fan...
>
> Do you know of a better and faster way of testing against different
> versions of real OpenSSH (running on different OSes)?
>
> Some of the tests even capture the OpenSSH debug logging and verify that
> it indicates certain conditions.
>

I'm not sure there are other ways.
However, we could split tests between unit tests and integration tests (if
needed) and across different profiles.  Some profiles could only be
activated by default in CI.


>
> Cheers,
>
>Thomas
>
>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org
> For additional commands, e-mail: dev-h...@mina.apache.org
>
>

-- 

Guillaume Nodet


Re: [VOTE] Release Apache Mina SSHD 2.9.3

2023-10-16 Thread Thomas Wolf

On 16.10.23 01:04 , Emmanuel Lécharny wrote:

My main concern is that we now need to install Docker to get the test 
passing, due to the usage of TestContainer. I don't really like it, the 
build should be self-supported.


The use of TestContainers isn't new.

Add to that it took 25mins to ran the tests. TC is clearly slowing down 
the whole tests (it takes around 10 s to start a new container):


[INFO] Apache Mina SSHD :: Common support utilits . SUCCESS [01:30 min]
...
[INFO] Apache Mina SSHD :: Core ... SUCCESS [08:37 min]
[INFO] Apache Mina SSHD :: Mina ... SUCCESS [03:41 min]
[INFO] Apache Mina SSHD :: Netty .. SUCCESS [02:56 min]
...
[INFO] Apache Mina SSHD :: SFTP ... SUCCESS [04:27 min]
[INFO] 




Frankly, I'm not a fan...


Do you know of a better and faster way of testing against different
versions of real OpenSSH (running on different OSes)?

Some of the tests even capture the OpenSSH debug logging and verify that
it indicates certain conditions.

Cheers,

  Thomas



-
To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org
For additional commands, e-mail: dev-h...@mina.apache.org



Re: [VOTE] Release Apache Mina SSHD 2.9.3

2023-10-15 Thread Emmanuel Lécharny

Hi,

My +1.

I've tje sameremarks thah for the other SSHD vote.

Th  nks ! !

On 14/10/2023 16:10, Gary D. Gregory wrote:

Hi again:

I should add that there is no way that I can tell if we've broken binary 
compatibility (or source compatibility) since there is no use of japicmp (or 
revapi or ...)

Gary

On 2023/10/14 14:04:07 "Gary D. Gregory" wrote:

Hi All:

+1

Lots of comments on this one; I am reviewing the src zip file:

- ASC OK

- Can't validate SHA file with:
 shasum --check apache-sshd-2.9.3-src.zip.sha512
   This is due to non-standard format for the file.
   Eyeballing the file looks OK when compared to shasum -a 512 output.

- IMO, the VOTE email should include a link to the KEYS file.
   The KEYS file is not even listed on 
https://mina.apache.org/sshd-project/downloads.html
   I had to go hunt for it: https://dist.apache.org/repos/dist/release/mina/KEYS

- Non-Blocker: Can't we just have SHA-512, instead of both SHA-512 and SHA-256?

- mvn apache-rat:check OK

- mvn clean install OK (but...)

- Tests fail unless you have admin rights, this should be documented in the 
README IMO:
[ERROR] Errors:
[ERROR] org.apache.sshd.common.util.io.IoUtilsTest.testCheckExists
[ERROR]   Run 1: IoUtilsTest.testCheckExists:74->testCheckExists:91 » 
FileSystem 
C:\Users\ggregory\Downloads\apache-sshd-2.9.3\sshd-common\target\IoUtilsTest\folder1\folder2\link:
 A required privilege is not held by the client
[ERROR]   Run 2: IoUtilsTest.testCheckExists:74->testCheckExists:91 » 
FileSystem 
C:\Users\ggregory\Downloads\apache-sshd-2.9.3\sshd-common\target\IoUtilsTest\folder1\folder2\link:
 A required privilege is not held by the client
[ERROR]   Run 3: IoUtilsTest.testCheckExists:74->testCheckExists:91 » 
FileSystem 
C:\Users\ggregory\Downloads\apache-sshd-2.9.3\sshd-common\target\IoUtilsTest\folder1\folder2\link:
 A required privilege is not held by the client

- Tests fail randomly because they do not use ephemeral ports.
   IOW, don't hardwire server socket ports in tests.
   The tests should report which port is in error, whether or not ephemeral 
ports are in use.
   In this case, I had to hunt down what was using port 8080 on my machine and 
temporarily stop that service.
[INFO]
[INFO] Results:
[INFO]
[ERROR] Errors:
[ERROR] 
org.apache.sshd.common.forward.PortForwardingTest.testLocalBindingOnDifferentInterfaces
[ERROR]   Run 1: PortForwardingTest.testLocalBindingOnDifferentInterfaces:814 » 
Bind Address already in use: bind
[ERROR]   Run 2: PortForwardingTest.testLocalBindingOnDifferentInterfaces:814 » 
Bind Address already in use: bind
[ERROR]   Run 3: PortForwardingTest.testLocalBindingOnDifferentInterfaces:814 » 
Bind Address already in use: bind
[INFO]
[INFO]
[ERROR] Tests run: 556, Failures: 0, Errors: 1, Skipped: 8

- Using:
Apache Maven 3.9.5 (57804ffe001d7215b5e7bcb531cf83df38f93546)
Maven home: C:\java\apache-maven-3.9.5
Java version: 17.0.8, vendor: Eclipse Adoptium, runtime: C:\Program 
Files\Eclipse Adoptium\jdk-17.0.8.7-hotspot
Default locale: en_US, platform encoding: Cp1252
OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"

Gary

On 2023/10/12 14:33:21 Guillaume Nodet wrote:

I've staged a candidate release for 2.9.3 at:
Official staging repo:
   https://dist.apache.org/repos/dist/dev/mina/sshd/2.9.3/
Maven staging repo:
   https://repository.apache.org/content/repositories/orgapachemina-1088
Git tag:
   https://github.com/apache/mina-sshd/commits/sshd-2.9.3
Changelog:
   https://github.com/apache/mina-sshd/blob/sshd-2.9.3/docs/changes/2.9.3.md

Please review and vote !

--

Guillaume Nodet



-
To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org
For additional commands, e-mail: dev-h...@mina.apache.org




-
To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org
For additional commands, e-mail: dev-h...@mina.apache.org



--
*Emmanuel Lécharny* P. +33 (0)6 08 33 32 61
elecha...@apache.org

-
To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org
For additional commands, e-mail: dev-h...@mina.apache.org



Re: [VOTE] Release Apache Mina SSHD 2.9.3

2023-10-15 Thread Emmanuel Lécharny

Hi!

My +1.

- Checked the signature
- Built with Java 8
  - test passed green when I installed Docker

My main concern is that we now need to install Docker to get the test 
passing, due to the usage of TestContainer. I don't really like it, the 
build should be self-supported.


Add to that it took 25mins to ran the tests. TC is clearly slowing down 
the whole tests (it takes around 10 s to start a new container):


[INFO] Apache Mina SSHD :: Common support utilits . SUCCESS [01:30 min]
...
[INFO] Apache Mina SSHD :: Core ... SUCCESS [08:37 min]
[INFO] Apache Mina SSHD :: Mina ... SUCCESS [03:41 min]
[INFO] Apache Mina SSHD :: Netty .. SUCCESS [02:56 min]
...
[INFO] Apache Mina SSHD :: SFTP ... SUCCESS [04:27 min]
[INFO] 




Frankly, I'm not a fan...

On 14/10/2023 16:04, Gary D. Gregory wrote:

Hi All:

+1

Lots of comments on this one; I am reviewing the src zip file:

- ASC OK

- Can't validate SHA file with:
 shasum --check apache-sshd-2.9.3-src.zip.sha512
   This is due to non-standard format for the file.
   Eyeballing the file looks OK when compared to shasum -a 512 output.

- IMO, the VOTE email should include a link to the KEYS file.
   The KEYS file is not even listed on 
https://mina.apache.org/sshd-project/downloads.html
   I had to go hunt for it: https://dist.apache.org/repos/dist/release/mina/KEYS

- Non-Blocker: Can't we just have SHA-512, instead of both SHA-512 and SHA-256?

- mvn apache-rat:check OK

- mvn clean install OK (but...)

- Tests fail unless you have admin rights, this should be documented in the 
README IMO:
[ERROR] Errors:
[ERROR] org.apache.sshd.common.util.io.IoUtilsTest.testCheckExists
[ERROR]   Run 1: IoUtilsTest.testCheckExists:74->testCheckExists:91 » 
FileSystem 
C:\Users\ggregory\Downloads\apache-sshd-2.9.3\sshd-common\target\IoUtilsTest\folder1\folder2\link:
 A required privilege is not held by the client
[ERROR]   Run 2: IoUtilsTest.testCheckExists:74->testCheckExists:91 » 
FileSystem 
C:\Users\ggregory\Downloads\apache-sshd-2.9.3\sshd-common\target\IoUtilsTest\folder1\folder2\link:
 A required privilege is not held by the client
[ERROR]   Run 3: IoUtilsTest.testCheckExists:74->testCheckExists:91 » 
FileSystem 
C:\Users\ggregory\Downloads\apache-sshd-2.9.3\sshd-common\target\IoUtilsTest\folder1\folder2\link:
 A required privilege is not held by the client

- Tests fail randomly because they do not use ephemeral ports.
   IOW, don't hardwire server socket ports in tests.
   The tests should report which port is in error, whether or not ephemeral 
ports are in use.
   In this case, I had to hunt down what was using port 8080 on my machine and 
temporarily stop that service.
[INFO]
[INFO] Results:
[INFO]
[ERROR] Errors:
[ERROR] 
org.apache.sshd.common.forward.PortForwardingTest.testLocalBindingOnDifferentInterfaces
[ERROR]   Run 1: PortForwardingTest.testLocalBindingOnDifferentInterfaces:814 » 
Bind Address already in use: bind
[ERROR]   Run 2: PortForwardingTest.testLocalBindingOnDifferentInterfaces:814 » 
Bind Address already in use: bind
[ERROR]   Run 3: PortForwardingTest.testLocalBindingOnDifferentInterfaces:814 » 
Bind Address already in use: bind
[INFO]
[INFO]
[ERROR] Tests run: 556, Failures: 0, Errors: 1, Skipped: 8

- Using:
Apache Maven 3.9.5 (57804ffe001d7215b5e7bcb531cf83df38f93546)
Maven home: C:\java\apache-maven-3.9.5
Java version: 17.0.8, vendor: Eclipse Adoptium, runtime: C:\Program 
Files\Eclipse Adoptium\jdk-17.0.8.7-hotspot
Default locale: en_US, platform encoding: Cp1252
OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"

Gary

On 2023/10/12 14:33:21 Guillaume Nodet wrote:

I've staged a candidate release for 2.9.3 at:
Official staging repo:
   https://dist.apache.org/repos/dist/dev/mina/sshd/2.9.3/
Maven staging repo:
   https://repository.apache.org/content/repositories/orgapachemina-1088
Git tag:
   https://github.com/apache/mina-sshd/commits/sshd-2.9.3
Changelog:
   https://github.com/apache/mina-sshd/blob/sshd-2.9.3/docs/changes/2.9.3.md

Please review and vote !

--

Guillaume Nodet



-
To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org
For additional commands, e-mail: dev-h...@mina.apache.org



--
*Emmanuel Lécharny* P. +33 (0)6 08 33 32 61
elecha...@apache.org

-
To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org
For additional commands, e-mail: dev-h...@mina.apache.org



Re: [VOTE] Release Apache Mina SSHD 2.9.3

2023-10-14 Thread Gary D. Gregory
Hi again:

I should add that there is no way that I can tell if we've broken binary 
compatibility (or source compatibility) since there is no use of japicmp (or 
revapi or ...)

Gary

On 2023/10/14 14:04:07 "Gary D. Gregory" wrote:
> Hi All:
> 
> +1
> 
> Lots of comments on this one; I am reviewing the src zip file:
> 
> - ASC OK
> 
> - Can't validate SHA file with:
> shasum --check apache-sshd-2.9.3-src.zip.sha512
>   This is due to non-standard format for the file.
>   Eyeballing the file looks OK when compared to shasum -a 512 output.
> 
> - IMO, the VOTE email should include a link to the KEYS file. 
>   The KEYS file is not even listed on 
> https://mina.apache.org/sshd-project/downloads.html
>   I had to go hunt for it: 
> https://dist.apache.org/repos/dist/release/mina/KEYS
> 
> - Non-Blocker: Can't we just have SHA-512, instead of both SHA-512 and 
> SHA-256?
> 
> - mvn apache-rat:check OK
> 
> - mvn clean install OK (but...)
> 
> - Tests fail unless you have admin rights, this should be documented in the 
> README IMO:
> [ERROR] Errors:
> [ERROR] org.apache.sshd.common.util.io.IoUtilsTest.testCheckExists
> [ERROR]   Run 1: IoUtilsTest.testCheckExists:74->testCheckExists:91 » 
> FileSystem 
> C:\Users\ggregory\Downloads\apache-sshd-2.9.3\sshd-common\target\IoUtilsTest\folder1\folder2\link:
>  A required privilege is not held by the client
> [ERROR]   Run 2: IoUtilsTest.testCheckExists:74->testCheckExists:91 » 
> FileSystem 
> C:\Users\ggregory\Downloads\apache-sshd-2.9.3\sshd-common\target\IoUtilsTest\folder1\folder2\link:
>  A required privilege is not held by the client
> [ERROR]   Run 3: IoUtilsTest.testCheckExists:74->testCheckExists:91 » 
> FileSystem 
> C:\Users\ggregory\Downloads\apache-sshd-2.9.3\sshd-common\target\IoUtilsTest\folder1\folder2\link:
>  A required privilege is not held by the client
> 
> - Tests fail randomly because they do not use ephemeral ports. 
>   IOW, don't hardwire server socket ports in tests. 
>   The tests should report which port is in error, whether or not ephemeral 
> ports are in use.
>   In this case, I had to hunt down what was using port 8080 on my machine and 
> temporarily stop that service.
> [INFO]
> [INFO] Results:
> [INFO]
> [ERROR] Errors:
> [ERROR] 
> org.apache.sshd.common.forward.PortForwardingTest.testLocalBindingOnDifferentInterfaces
> [ERROR]   Run 1: PortForwardingTest.testLocalBindingOnDifferentInterfaces:814 
> » Bind Address already in use: bind
> [ERROR]   Run 2: PortForwardingTest.testLocalBindingOnDifferentInterfaces:814 
> » Bind Address already in use: bind
> [ERROR]   Run 3: PortForwardingTest.testLocalBindingOnDifferentInterfaces:814 
> » Bind Address already in use: bind
> [INFO]
> [INFO]
> [ERROR] Tests run: 556, Failures: 0, Errors: 1, Skipped: 8
> 
> - Using:
> Apache Maven 3.9.5 (57804ffe001d7215b5e7bcb531cf83df38f93546)
> Maven home: C:\java\apache-maven-3.9.5
> Java version: 17.0.8, vendor: Eclipse Adoptium, runtime: C:\Program 
> Files\Eclipse Adoptium\jdk-17.0.8.7-hotspot
> Default locale: en_US, platform encoding: Cp1252
> OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"
> 
> Gary
> 
> On 2023/10/12 14:33:21 Guillaume Nodet wrote:
> > I've staged a candidate release for 2.9.3 at:
> > Official staging repo:
> >   https://dist.apache.org/repos/dist/dev/mina/sshd/2.9.3/
> > Maven staging repo:
> >   https://repository.apache.org/content/repositories/orgapachemina-1088
> > Git tag:
> >   https://github.com/apache/mina-sshd/commits/sshd-2.9.3
> > Changelog:
> >   https://github.com/apache/mina-sshd/blob/sshd-2.9.3/docs/changes/2.9.3.md
> > 
> > Please review and vote !
> > 
> > -- 
> > 
> > Guillaume Nodet
> > 
> 
> -
> To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org
> For additional commands, e-mail: dev-h...@mina.apache.org
> 
> 

-
To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org
For additional commands, e-mail: dev-h...@mina.apache.org



Re: [VOTE] Release Apache Mina SSHD 2.9.3

2023-10-14 Thread Gary D. Gregory
Hi All:

+1

Lots of comments on this one; I am reviewing the src zip file:

- ASC OK

- Can't validate SHA file with:
shasum --check apache-sshd-2.9.3-src.zip.sha512
  This is due to non-standard format for the file.
  Eyeballing the file looks OK when compared to shasum -a 512 output.

- IMO, the VOTE email should include a link to the KEYS file. 
  The KEYS file is not even listed on 
https://mina.apache.org/sshd-project/downloads.html
  I had to go hunt for it: https://dist.apache.org/repos/dist/release/mina/KEYS

- Non-Blocker: Can't we just have SHA-512, instead of both SHA-512 and SHA-256?

- mvn apache-rat:check OK

- mvn clean install OK (but...)

- Tests fail unless you have admin rights, this should be documented in the 
README IMO:
[ERROR] Errors:
[ERROR] org.apache.sshd.common.util.io.IoUtilsTest.testCheckExists
[ERROR]   Run 1: IoUtilsTest.testCheckExists:74->testCheckExists:91 » 
FileSystem 
C:\Users\ggregory\Downloads\apache-sshd-2.9.3\sshd-common\target\IoUtilsTest\folder1\folder2\link:
 A required privilege is not held by the client
[ERROR]   Run 2: IoUtilsTest.testCheckExists:74->testCheckExists:91 » 
FileSystem 
C:\Users\ggregory\Downloads\apache-sshd-2.9.3\sshd-common\target\IoUtilsTest\folder1\folder2\link:
 A required privilege is not held by the client
[ERROR]   Run 3: IoUtilsTest.testCheckExists:74->testCheckExists:91 » 
FileSystem 
C:\Users\ggregory\Downloads\apache-sshd-2.9.3\sshd-common\target\IoUtilsTest\folder1\folder2\link:
 A required privilege is not held by the client

- Tests fail randomly because they do not use ephemeral ports. 
  IOW, don't hardwire server socket ports in tests. 
  The tests should report which port is in error, whether or not ephemeral 
ports are in use.
  In this case, I had to hunt down what was using port 8080 on my machine and 
temporarily stop that service.
[INFO]
[INFO] Results:
[INFO]
[ERROR] Errors:
[ERROR] 
org.apache.sshd.common.forward.PortForwardingTest.testLocalBindingOnDifferentInterfaces
[ERROR]   Run 1: PortForwardingTest.testLocalBindingOnDifferentInterfaces:814 » 
Bind Address already in use: bind
[ERROR]   Run 2: PortForwardingTest.testLocalBindingOnDifferentInterfaces:814 » 
Bind Address already in use: bind
[ERROR]   Run 3: PortForwardingTest.testLocalBindingOnDifferentInterfaces:814 » 
Bind Address already in use: bind
[INFO]
[INFO]
[ERROR] Tests run: 556, Failures: 0, Errors: 1, Skipped: 8

- Using:
Apache Maven 3.9.5 (57804ffe001d7215b5e7bcb531cf83df38f93546)
Maven home: C:\java\apache-maven-3.9.5
Java version: 17.0.8, vendor: Eclipse Adoptium, runtime: C:\Program 
Files\Eclipse Adoptium\jdk-17.0.8.7-hotspot
Default locale: en_US, platform encoding: Cp1252
OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"

Gary

On 2023/10/12 14:33:21 Guillaume Nodet wrote:
> I've staged a candidate release for 2.9.3 at:
> Official staging repo:
>   https://dist.apache.org/repos/dist/dev/mina/sshd/2.9.3/
> Maven staging repo:
>   https://repository.apache.org/content/repositories/orgapachemina-1088
> Git tag:
>   https://github.com/apache/mina-sshd/commits/sshd-2.9.3
> Changelog:
>   https://github.com/apache/mina-sshd/blob/sshd-2.9.3/docs/changes/2.9.3.md
> 
> Please review and vote !
> 
> -- 
> 
> Guillaume Nodet
> 

-
To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org
For additional commands, e-mail: dev-h...@mina.apache.org



Re: [VOTE] Release Apache Mina SSHD 2.9.3

2023-10-12 Thread Thomas Wolf

On 12.10.23 16:33 , Guillaume Nodet wrote:

I've staged a candidate release for 2.9.3 at:


+1

* Src tar unpacks and builds fine; all tests pass locally (OS X)
* Signatures verify
* Class version 52 (Java 8)

Cheers,

  Thomas


-
To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org
For additional commands, e-mail: dev-h...@mina.apache.org