Skip to content

Conversation

vttranlina
Copy link
Contributor

The pr #1647 has a lot of files changed.
The ci can't green after several re-runs, So this pr is a split part of 1674 for easier

<junit.vintage.version>5.9.3</junit.vintage.version>
<concurrent.version>1.3.4</concurrent.version>
<xbean-spring.version>4.23</xbean-spring.version>
<xbean-spring.version>4.21</xbean-spring.version>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Anything wrong with that?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI fail at Spring module
https://ci-builds.apache.org/blue/organizations/jenkins/james%2FApacheJames/detail/PR-1655/2/tests/

Error creating bean with name 'serverConnector' defined in class path resource [META-INF/org/apache/james/spring-server.xml]: Invocation of init method failed; nested exception is java.io.IOException: Cannot bind to URL [rmi://127.0.0.1:9999/jmxrmi]: javax.naming.CommunicationException [Root exception is java.rmi.ServerException: RemoteException occurred in server thread; nested exception is: 
 java.rmi.UnmarshalException: error unmarshalling arguments; nested exception is: 
 java.io.InvalidClassException: filter status: REJECTED]
Stacktrace
org.springframework.beans.factory.BeanCreationException: 
Error creating bean with name 'serverConnector' defined in class path resource [META-INF/org/apache/james/spring-server.xml]: Invocation of init method failed; nested exception is java.io.IOException: Cannot bind to URL [rmi://127.0.0.1:9999/jmxrmi]: javax.naming.CommunicationException [Root exception is java.rmi.ServerException: RemoteException occurred in server thread; nested exception is: 
	java.rmi.UnmarshalException: error unmarshalling arguments; nested exception is: 
	java.io.InvalidClassException: filter status: REJECTED]
	at org.apache.james.app.spring.JamesAppSpringMainTest.testServer(JamesAppSpringMainTest.java:26)
Caused by: java.io.IOException: 
Cannot bind to URL [rmi://127.0.0.1:9999/jmxrmi]: javax.naming.CommunicationException [Root exception is java.rmi.ServerException: RemoteException occurred in server thread; nested exception is: 
	java.rmi.UnmarshalException: error unmarshalling arguments; nested exception is: 
	java.io.InvalidClassException: filter status: REJECTED]
	at org.apache.james.app.spring.JamesAppSpringMainTest.testServer(JamesAppSpringMainTest.java:26)
Caused by: javax.naming.CommunicationException
	at org.apache.james.app.spring.JamesAppSpringMainTest.testServer(JamesAppSpringMainTest.java:26)
Caused by: java.rmi.ServerException: 
RemoteException occurred in server thread; nested exception is: 
	java.rmi.UnmarshalException: error unmarshalling arguments; nested exception is: 
	java.io.InvalidClassException: filter status: REJECTED
	at org.apache.james.app.spring.JamesAppSpringMainTest.testServer(JamesAppSpringMainTest.java:26)
Caused by: java.rmi.UnmarshalException: 
error unmarshalling arguments; nested exception is: 
	java.io.InvalidClassException: filter status: REJECTED
Caused by: java.io.InvalidClassException: filter status: REJECTED

@vttranlina
Copy link
Contributor Author

CI error

setFlagsShouldWorkWithConcurrencyWithRemove – org.apache.james.mailbox.cassandra.mail.CassandraMessageMapperTest

Error
Expecting empty but was: ["custom-3-9"]
Stacktrace
java.lang.AssertionError: 
Expecting empty but was: ["custom-3-9"]
Standard Output
08:27:05.879 [ERROR] o.a.j.m.c.m.CassandraMessageMapper - Can not update following UIDs [ComposedMessageId{mailboxId=CassandraId{id=33f35700-2b8e-11ee-8d16-b744774e0118}, messageId=CassandraMessageId{uuid=33f552d0-2b8e-11ee-8d16-b744774e0118}, uid=MessageUid{uid=1}}] for mailbox 33f35700-2b8e-11ee-8d16-b744774e0118

It is not related to other commits in this pr.
test cases: setFlagsShouldWorkWithConcurrencyWithRemove and userFlagsUpdateShouldWorkInConcurrentEnvironment (concurrency test) is unstable (exactly is it not work, but the threadCount and updateCount is quite small, so it is not enough to easier make the test fail)
=> I marked @Disabled it

@chibenwa
Copy link
Contributor

Tag it as unstable instead please

@Arsnael
Copy link
Contributor

Arsnael commented Jul 27, 2023

Green!

Pity there is a fixup I can't merge... Please squash and rebase it :)

@chibenwa
Copy link
Contributor

Quock perf test maybe?

- 6.0.0 (supports javax.{inject,servlet,persistence}, mostly supports jakarta.inject)
- 7.0.0 (supports jakarta.{inject,servlet,persistence})
@vttranlina vttranlina force-pushed the upgradeDependencies_Lib branch from 551e530 to bb61fd7 Compare July 28, 2023 02:01
@Arsnael Arsnael merged commit bb49990 into apache:master Jul 31, 2023
@vttranlina vttranlina deleted the upgradeDependencies_Lib branch December 12, 2023 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants