-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updated OSGi data #202
Updated OSGi data #202
Conversation
44064b8
to
18c2de9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are the changes that I just tested out, so looks good
The CI run failed, it looks like Jenkins or something. I will re-run it |
retest this please |
1 similar comment
retest this please |
Something is up with the glassfish build: https://ci.eclipse.org/glassfish/view/GlassFish/job/glassfish_build-and-publish-to-eclipse-download/627/ - the previous build of the same SHA passed: https://ci.eclipse.org/glassfish/view/GlassFish/job/glassfish_build-and-publish-to-eclipse-download/626/ |
The TCK run is failing because of a new dependency on a snapshot release of org.glassfish.web:tagsdoc:
You will need to add -Psnapshots to the build to allow the snapshots staging repo to be included. |
I think I have that:
(from https://ci.eclipse.org/jta/job/eclipse-ee4j_jta-api-pulls-openjdk-jdk11-latest/configure) |
When I looked at the glassfish CI, one of their builds for SHA |
That mvn command is just overriding the version of transactions API jar used in the build. You need to enable the snapshots profile in addition to the staging profile: mvn -Pstaging -Psnapshots clean install -D$OVERRIDE_TRANSACTION_API_PROPERTY=$SNAPSHOT_VERSION -DskipTests |
Sorry, I missed it (I presume it was that I was seeing |
retest this please |
It's still not working for CI :( The job config currently says:
I will try to run it locally too to see what I get ( |
I am getting what seems to be the same outcome locally. |
Curious because it is working for me, and a pull showed no new changes since the job ran:
The -DskipTests property is not preventing the some of the TCK tests from running. Maybe the build command in the CI job should be updated to use the 'mvn clean install -Pfastest ...' from the glassfish readme. I'm going to look at the snapshots/tags module in more detail to see how it is building on my machine. |
I removed my local ~/.m2/repository/org/glassfish/web and can build the snapshots/tags module even without the -Psnapshots profile, but I also don't see any attempt to resolve the org.glassfish.web:tagsdoc:1.0-SNAPSHOT/maven-metadata.xml even with debug mode enabled:
|
What does running |
I created a simple freestyle job in the CDI CI project to just build the glassfish master and it is also seeing the same failure: TOOLS_PREFIX='/opt/tools' git clone https://github.com/eclipse-ee4j/glassfish.git Output of job is here: https://ci.eclipse.org/cdi/job/GlassFish7Build/2/console I'll create a bug on the EF CI infrastructure. |
After looking into the error more, this seems to be a problem with the glassfish build using a non-https URL to access a maven repository. I have created the following glassfish issue: |
retest this please |
api/pom.xml
Outdated
javax.interceptor.*;version="1.2", | ||
jakarta.enterprise.context;version=!, | ||
jakarta.enterprise.util;version=!, | ||
jakarta.interceptor;version="2.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
version="2.1"
seems like it might not be working:
[caused by: Unable to resolve jakarta.transaction-api [153](R 153.0): missing requirement [jakarta.transaction-api [153](R 153.0)] osgi.wiring.package; (&(osgi.wiring.package=jakarta.interceptor)(version>=2.1.0))]] Unresolved requirements: [[org.glassfish.main.resources.mail-connector [207](R 207.0)] osgi.wiring.package; (&(osgi.wiring.package=jakarta.resource)(version>=2.0.0)(!(version>=3.0.0)))]
18c2de9
to
0ed8255
Compare
javax.interceptor.*;version="1.2", | ||
jakarta.enterprise.context;version=!, | ||
jakarta.enterprise.util;version=!, | ||
jakarta.interceptor;version="2.0.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The current version of intercept builds with 2.1.0 (IIRC) but the one in glassfish 7.0.0 nightly (that I just downloaded from https://download.eclipse.org/ee4j/glassfish/) has 2.0.1 so going with that in here
Signed-off-by: Tom Jenkinson <[email protected]>
0ed8255
to
b5848ca
Compare
javax.transaction.xa, | ||
* | ||
</Import-Package> | ||
<Export-Package> | ||
javax.transaction;version="1.2";uses:="javax.interceptor,javax.transaction.xa,javax.enterprise.util,javax.enterprise.context", | ||
javax.transaction;version="1.3";uses:="javax.interceptor,javax.transaction.xa,javax.enterprise.util,javax.enterprise.context" | ||
jakarta.transaction;version="2.0.1";uses:="javax.transaction.xa,jakarta.interceptor,jakarta.enterprise.context,jakarta.enterprise.util" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Jakarta Transactions project no longer exports the javax.transaction package (https://github.com/eclipse-ee4j/jta-api/tree/master/api/src/main/java) - OSGI Release 6 3.6.5
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
javax.enterprise.context;version=!, | ||
javax.enterprise.util;version=!, | ||
javax.interceptor.*;version="1.2", | ||
jakarta.enterprise.context;version=!, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
!
rationale is at https://github.com/eclipse-ee4j/jta-api/pull/187/files#r595102746
Thank you @LanceAndersen ! |
BTW, is there analogous change in https://github.com/eclipse-ee4j/jca-api? See some comments under jakartaee/connectors#120. |
Jakarta Connectors on They possibly missed this intermediary stage where the Jakarta Transactions jar would be declaring to export packages from javax.transaction (and we exported javax.transaction previously in EE8 https://jakarta.ee/specifications/transactions/1.3/). In summary, I hope there should be no reason for Jakarta Connectors to change |
Resolved #201 |
Signed-off-by: Tom Jenkinson [email protected]