Skip to content
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

Jakarta Interceptors 2.0 #233

Merged
merged 4 commits into from
Nov 19, 2020
Merged

Conversation

arjantijms
Copy link
Contributor

@arjantijms arjantijms commented Jul 20, 2020

Signed-off-by: arjantijms arjan.tijms@gmail.com

Specification PR template

When creating a specification project release review, create PRs with the content defined as follows.

Include the following in the PR:

Signed-off-by: arjantijms <arjan.tijms@gmail.com>
@arjantijms
Copy link
Contributor Author

Note that Interceptors doesn't have its own TCK, but piggybacks on the platform TCK.

@netlify
Copy link

netlify bot commented Jul 20, 2020

Deploy preview for jakartaee-specifications ready!

Built with commit 5e5a4de

https://deploy-preview-233--jakartaee-specifications.netlify.app

@kwsutter kwsutter requested a review from kazumura July 20, 2020 19:52
@kwsutter
Copy link
Contributor

Assigning @kazumura as the Mentor for this Spec PR.

@kwsutter kwsutter added draft Work in Progress wave:1 Used for release tracking purposes labels Jul 20, 2020
@kwsutter kwsutter added this to the Jakarta EE 9 milestone Jul 20, 2020
@kazumura
Copy link
Contributor

kazumura commented Jul 20, 2020

Spec Review Checklist

  1. Spec PR
  1. _index.md
  1. javadocs
  • Footer contains Eclipse copyright and link to license
  • ESFL license is included, usually as doc-files/speclicense.html
  • no META-INF directory in PR
  • javadocs-jar artifact matches apidocs (optional for this release)
  1. Spec PDF
  • Correct spec title
  • Version number of the form x.y, not x.y.z
  • Correct Eclipse copyright line
  • No DRAFT or SNAPSHOT
  • Correct Logo
  1. Spec HTML
  • Same as PDF
  1. TCK zip file
  • README file (optional for this release)
    (N/A) depend on Platform TCK
  • EFTL license file, preferably named LICENSE.md
    (N/A) depend on Platform TCK
  • User's Guide (or equivalent documentation)
    (N/A) depend on Platform TCK
  • How to test the Compatible Implementation(s) listed in _index.md above with the TCK (may be in UG)
    (N/A) depend on Platform TCK
  1. TCK User's Guide (or equivalent documentation)
  • Software requirements listed
    (N/A) depend on Platform TCK
  • Installation and configuration described
    (N/A) depend on Platform TCK
  • How to run tests
    (N/A) depend on Platform TCK
  • Where to file challenges
    (N/A) depend on Platform TCK
  1. Compatibility certification request
  • Request follows template
  • SHA-256 fingerprint matches staged TCK zip file
    (N/A) depend on Platform TCK
  • Request issue has certification label.
  1. TCK results summary
  • Page is hosted by Compatible Implementation project
    (N/A) depend on Platform TCK
  • Includes all information from certification request
    (N/A) depend on Platform TCK
  • Summary includes number of tests passed, failed, errors
    (N/A) depend on Platform TCK
  • SHA-256 fingerprint matches staged TCK zip file on cert request
    (N/A) depend on Platform TCK
  1. Update Jakarta EE API jar
  • Update the Jakarta EE API jar by submitting a PR to the jakartaee-api project that updates the version number of your API jar file.

@kwsutter
Copy link
Contributor

Tagging with "Platform" to indicate it's dependency on the Platform TCK.

@kwsutter kwsutter added the platform Run ballot with the Jakarta Platform label Jul 22, 2020
Copy link
Contributor

@kazumura kazumura left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks, @arjantijms

@arjantijms
Copy link
Contributor Author

Thanks @kazumura for the extra pair of eyes!

@kazumura kazumura added ballot Delivered to the Specification Committee for ballot final Ready for Vote and removed draft Work in Progress labels Aug 10, 2020
Copy link
Contributor

@kwsutter kwsutter left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks, @arjantijms

jeanouii
jeanouii previously approved these changes Aug 21, 2020
@jeanouii
Copy link
Contributor

There is one mention to javax.xml.rpc.handler.MessageContext in the api for InvocationContext.

It should be jakarta.xml.rpc.handler.MessageContext

@jeanouii jeanouii dismissed their stale review August 26, 2020 16:45

One javax missing in the apidocs

@arjantijms
Copy link
Contributor Author

There is one mention to javax.xml.rpc.handler.MessageContext in the api for InvocationContext.
It should be jakarta.xml.rpc.handler.MessageConte

Thanks! I've just updated it, and will now re-stage the API and update the javadocs in the PR.

@arjantijms
Copy link
Contributor Author

Signed-off-by: arjantijms <arjan.tijms@gmail.com>
@arjantijms
Copy link
Contributor Author

@hendrikebbers
Copy link

@arjantijms must this be updated somehow? Just trying to understand the workflows :)

@arjantijms
Copy link
Contributor Author

@hendrikebbers yes indeed. This is easiest for myself todo.

It's essentially running mvn clean install in the spec folder of interceptors, overriding the release to Final and the version to 2.0.0. Then checking out this repo, and copying it to the appropriate folder (see existing PR).

I'll do it now, though if you wanted to do it, the easiest way would be to checkout my PR branch and then do a PR against that.

By @hendrikebbers



Signed-off-by: arjantijms <arjan.tijms@gmail.com>
@arjantijms
Copy link
Contributor Author

Updated PR: d01e6bf

Signed-off-by: arjantijms <arjan.tijms@gmail.com>
@dblevins dblevins self-requested a review August 29, 2020 17:46
Copy link
Contributor

@dblevins dblevins left a comment

Choose a reason for hiding this comment

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

Minor javadoc issue pointing to non-existent jakarta.xml.rpc API

PR available jakartaee/interceptors#79

@scottmarlow
Copy link
Contributor

FYI, the RC1 Platform TCK link is https://download.eclipse.org/ee4j/jakartaee-tck/jakartaee9-eftl/promoted/jakarta-jakartaeetck-9.0.0-RC1.zip which has the same SHA 75b2493a117e7f8fe775cc7e2e8a605a203611895091f4bb9aaabed57f813392 as was used to pass 100% of the Full Platform TCK yesterday (we are running again due to a system networking failure that occurred after the test run while collecting test results). The Jakarta EE 9 TCK results wiki page will be updated after the run completes, not that you need to wait since I said, we already passed 100%.

We also passed the Web Profile tests as well!.

This testing was done with https://download.eclipse.org/ee4j/glassfish/glassfish-6.0.0-RC2.zip.

@arjantijms
Copy link
Contributor Author

@kwsutter
Copy link
Contributor

Per the discussion on our Spec Committee call on 11/11, I updated the reference to the EFTL TCK in the top level comment to be this: https://download.eclipse.org/ee4j/jakartaee-tck/jakartaee9-eftl/promoted/jakarta-jakartaeetck-9.0.0.zip.

I based this update on the CCR and TCK Results which both referenced the "-RC1" version of the TCK. This type of update was acceptable and expected by the Spec Committee.

@kazumura kazumura added the approved The ballot was approved by the Specification Committee label Nov 18, 2020
@kazumura
Copy link
Contributor

kazumura commented Nov 18, 2020

  • On ballot completion, the specification committee mentor:
  • adds this final checklist to the main PR.
  • adds the approved label to the PRs, and sends out the Ballot Summary per this template to the public Jakarta EE Specification Committee email list
  • calculates the staged EFTL TCK signature and promotes it to the committee download area
    using the https://ci.eclipse.org/jakartaee-spec-committee/job/promote-release/ job. Manually editing the jenkins Build Information will help identify the build (ie. Mail 2.0 or CDI 3.0).
  • merges the specification (and apidocs) PRs, ensuring the "date:" field in the _index.md file has an appropriate value to allow publishing.
  • updates the specification page with the ballot results. This is normally done via a separate PR that should be reviewed, approved, and merged.
  • notifies the EMO of the ballot results by email to emo@eclipse-foundation.org. Just forward the ballot summary note sent earlier to the public Spec Committee email list.
  • creates an issue in the specification project that includes the following checklist for the specification project team:
    <https://github.com/eclipse-ee4j/interceptor-api/issues/82>

  • promotes api staging release promotes the specification api jars to maven central. An example release job script can be found here https://wiki.eclipse.org/MavenReleaseScript.
  • go through the merged jakarta.ee specification website page to verify all the links are valid.
  • approve the compatibility request.
  • The compatible implementation project/vendor MUST send an email to tck@eclipse.org for approval of the compatible implementation for trademark usage.
  • merge any final release branch as appropriate for the branch management for the project.

@kazumura kazumura merged commit 11f7f35 into jakartaee:master Nov 19, 2020
starksm64 pushed a commit to starksm64/specifications that referenced this pull request Dec 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved The ballot was approved by the Specification Committee ballot Delivered to the Specification Committee for ballot final Ready for Vote platform Run ballot with the Jakarta Platform wave:1 Used for release tracking purposes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants