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 MVC 2.0 #315

Merged
merged 5 commits into from Dec 18, 2020
Merged

Jakarta MVC 2.0 #315

merged 5 commits into from Dec 18, 2020

Conversation

ivargrimstad
Copy link
Member

@ivargrimstad ivargrimstad commented Nov 21, 2020

Specification PR template

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

Include the following in the PR:

Note: If any item does not apply, check it and mark N/A below it.

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>
Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>
Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>
Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>
@ivargrimstad ivargrimstad added the draft Work in Progress label Nov 21, 2020
@ivargrimstad ivargrimstad self-assigned this Nov 21, 2020
@netlify
Copy link

netlify bot commented Nov 21, 2020

✔️ Deploy preview for jakartaee-specifications ready!

🔨 Explore the source changes: 39de768

🔍 Inspect the deploy logs: https://app.netlify.com/sites/jakartaee-specifications/deploys/5fdb6bbf2c735807fee68e41

😎 Browse the preview: https://deploy-preview-315--jakartaee-specifications.netlify.app

@ivargrimstad ivargrimstad added final Ready for Vote and removed draft Work in Progress labels Nov 22, 2020
@ivargrimstad ivargrimstad marked this pull request as ready for review November 22, 2020 14:53
@ivargrimstad ivargrimstad added draft Work in Progress final Ready for Vote and removed final Ready for Vote draft Work in Progress labels Nov 22, 2020
@ivargrimstad
Copy link
Member Author

ivargrimstad commented Dec 3, 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
  • [x ] 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)
  • EFTL license file, preferably named LICENSE.md
  • User's Guide (or equivalent documentation)
  • How to test the Compatible Implementation(s) listed in _index.md above with the TCK (may be in UG)
  1. TCK User's Guide (or equivalent documentation)
  • Software requirements listed
  • Installation and configuration described
  • How to run tests
  • Where to file challenges
  1. Compatibility certification request
  • Request follows template
  • SHA-256 fingerprint matches staged TCK zip file
  • Request issue has certification label.
  1. TCK results summary
  • Page is hosted by Compatible Implementation project
  • Includes all information from certification request
  • Summary includes number of tests passed, failed, errors
  • SHA-256 fingerprint matches staged TCK zip file on cert request
  1. If a Release Review is required, the specification project team contacts the EMO to initiate the review by sending an email to emo@eclipse.org.
    (A Release Review is not required if the current release is a Service Release based on a previously successful Major or Minor
    release as indicated by a release record on the project's Releases page, e.g., the Jakarta Servlet releases page.)

  2. 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. N/A

@starksm64 starksm64 added the ballot Delivered to the Specification Committee for ballot label Dec 4, 2020
@starksm64
Copy link
Contributor

@ivargrimstad
Copy link
Member Author

Thanks @starksm64 !

@kwsutter
Copy link
Contributor

@ivargrimstad I found a couple of items while doing the release review...

  • In the Javadoc, the release specified at the top of the page is "Jakarta MVC 2.0.0-SNAPSHOT API".
  • In the Javadoc, the link at the bottom of the page for the license gets a 404...
  • Question... On the Krazo 2.0 page, there is this statement: "Eclipse Krazo requires at least the Jakarta EE 8 Web-Profile or the Eclipse MicroProfile." I thought MVC 2.0 was updated for Jakarta EE 9 and the updated namespace. Does Krazo 2.0 work with either Jakarta EE 8 (javax) or Jakarta EE 9 (jakarta)? And, since you spelled a specific version of Jakarta EE, do you also need a specific version of MicroProfile?

Not sure if the first two items warrant a re-review? You'd have to push a new version of the javadoc. I downloaded the staged version of the javadoc jar file. The release is correct at the top of this javadoc: "Jakarta MVC 2.0.0 API". But, the license link still gives me a 404 error...

I'll hold off on my vote until we can have this discussion. Thanks.

@ivargrimstad
Copy link
Member Author

  • In the Javadoc, the release specified at the top of the page is "Jakarta MVC 2.0.0-SNAPSHOT API".

Yes, that is weird. AS you point out, the stated javadoc jar has the correct header.

  • In the Javadoc, the link at the bottom of the page for the license gets a 404...

I see that now. Seems like the doc-files directory weren't copied to the target folder because of the default value of the configuration of the javadoc plugin.

  • Question... On the Krazo 2.0 page, there is this statement: "Eclipse Krazo requires at least the Jakarta EE 8 Web-Profile or the Eclipse MicroProfile." I thought MVC 2.0 was updated for Jakarta EE 9 and the updated namespace. Does Krazo 2.0 work with either Jakarta EE 8 (javax) or Jakarta EE 9 (jakarta)? And, since you spelled a specific version of Jakarta EE, do you also need a specific version of MicroProfile?

That statement is a simple copy/paste error. I have updated the gh-pages now, so it states that Jakarta EE 9 is the minimum requirement.

Not sure if the first two items warrant a re-review? You'd have to push a new version of the javadoc. I downloaded the staged version of the javadoc jar file. The release is correct at the top of this javadoc: "Jakarta MVC 2.0.0 API". But, the license link still gives me a 404 error...

I can either copy the correctly generated javadoc files into this PR and update the staged javadoc artifact OR do a 2.0.1 release of the API artifacts directly after the ballot. I find the first option better.

@kwsutter
Copy link
Contributor

Not sure if the first two items warrant a re-review? You'd have to push a new version of the javadoc. I downloaded the staged version of the javadoc jar file. The release is correct at the top of this javadoc: "Jakarta MVC 2.0.0 API". But, the license link still gives me a 404 error...

I can either copy the correctly generated javadoc files into this PR and update the staged javadoc artifact OR do a 2.0.1 release of the API artifacts directly after the ballot. I find the first option better.

I'm okay with not restarting the ballot. I'll put my comments in the ballot thread and see if there's any pushback. Otherwise, I think we're okay to move forward. Thanks!

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>
@starksm64 starksm64 added approved The ballot was approved by the Specification Committee and removed ballot Delivered to the Specification Committee for ballot labels Dec 17, 2020
@starksm64
Copy link
Contributor

Representative Representative for: Vote
Kenji Kazumura Fujitsu +1
Dan Bandera, Kevin Sutter IBM +1
Ed Bratt, Dmitry Kornilov Oracle +1
Andrew Pielage, Matt Gill Payara +1
Scott Stark, Mark Little Red Hat +1
David Blevins, Jean-Louis Monteiro Tomitribe +1
Ivar Grimstad EE4J PMC +1
Marcelo Ancelmo, Martijn Verburg Participant Members +1
Werner Keil Committer Members +1
Scott (Congquan) Wang Enterprise Members +1
  Total 10

@starksm64
Copy link
Contributor

starksm64 commented Dec 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:
    Finalize MVC 2.0 release mvc#58

@starksm64 starksm64 merged commit 9f29396 into jakartaee:master Dec 18, 2020
@ivargrimstad ivargrimstad deleted the mvc-2.0-release branch March 2, 2021 11:03
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 final Ready for Vote
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants