Skip to content

Update GlassFish EJB runner to compensate for TCK 11.0.2#2666

Merged
arjantijms merged 1 commit intojakartaee:11.0.xfrom
arjantijms:110x_update_ejb_runner
Mar 30, 2026
Merged

Update GlassFish EJB runner to compensate for TCK 11.0.2#2666
arjantijms merged 1 commit intojakartaee:11.0.xfrom
arjantijms:110x_update_ejb_runner

Conversation

@arjantijms
Copy link
Copy Markdown
Contributor

The 11.0.2 TCK renamed a beanName, but did not update that name in the corresponding sun-application-client.xml file. Since it concerns a vendor specific file there's no need to challenge and we can compensate for it via the runner.

CC @alwin-joseph @anajosep @arjantijms @cesarhernandezgt @dblevins @m0mus @edbratt @gurunrao @jansupol @jgallimore @kazumura @kwsutter @LanceAndersen @bhatpmk @RohitKumarJain @shighbar @gthoman @brideck @OndroMih @dmatej
@starksm64 @scottmarlow

@arjantijms arjantijms self-assigned this Mar 30, 2026
@scottmarlow
Copy link
Copy Markdown
Member

Good catch!

@arjantijms
Copy link
Copy Markdown
Contributor Author

For reference it concerns this rename:

378fe74#diff-ef8b099f91d1e82d70426dfea37913f3ced3df6bb9f7c706f6d6d34000beccd4L30

The change is from the non-portable "@ejb(name = "ejb/EnvEntrySetterBean")" to the portable " @ejb(beanName = "EnvEntrySetterBean")". But as part of that change the name is also changed from "ejb/EnvEntrySetterBean" to "EnvEntrySetterBean".

@arjantijms arjantijms merged commit 9264aa6 into jakartaee:11.0.x Mar 30, 2026
2 checks passed
@scottmarlow
Copy link
Copy Markdown
Member

For reference it concerns this rename:

378fe74#diff-ef8b099f91d1e82d70426dfea37913f3ced3df6bb9f7c706f6d6d34000beccd4L30

The change is from the non-portable "@ejb(name = "ejb/EnvEntrySetterBean")" to the portable " @ejb(beanName = "EnvEntrySetterBean")". But as part of that change the name is also changed from "ejb/EnvEntrySetterBean" to "EnvEntrySetterBean".

^ sounds important to include in https://github.com/jakartaee/platform-tck/blob/11.0.x/release/README.adoc#1102 for lack of any other way to describe the change you had to make, that others that passed with 11.0.1 might have to make as well with 11.0.2.

We already packaged up the 11.0.2 release but still would be good to update the readme

arjantijms added a commit that referenced this pull request Mar 31, 2026
Clarify ejb30 bb.session porting errors and update bean names in documentation.

See #2666 (comment)
@arjantijms
Copy link
Copy Markdown
Contributor Author

We already packaged up the 11.0.2 release but still would be good to update the readme

Sure, I added the information there.

@arjantijms arjantijms deleted the 110x_update_ejb_runner branch March 31, 2026 09:37
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.

2 participants