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

MP70: Implement MicroProfile 7.0 #27106

Open
1 of 49 tasks
Emily-Jiang opened this issue Dec 7, 2023 · 10 comments
Open
1 of 49 tasks

MP70: Implement MicroProfile 7.0 #27106

Emily-Jiang opened this issue Dec 7, 2023 · 10 comments
Assignees
Labels
Design Approved Epic Used to track Feature Epics that are following the UFO process focalApproved:demo Approval that a Demo has been scheduled focalApproved:externals Focal Approval granted for APIs/Externals for the feature focalApproved:fat Focal Approval granted for FAT for the feature focalApproved:id Focal Approval granted for ID for the feature focalApproved:instantOn Focal Approval granted for InstantOn for the feature focalApproved:performance Focal Approval granted for Performance for the feature focalApproved:serviceability Focal Approval granted for Serviceability for the feature focalApproved:ste Focal Approval granted for STE for the feature focalApproved:svt Focal Approval granted for SVT for the feature ID Required in:MicroProfile In Progress Items that are in active development. target:beta The Epic or Issue is targetted for the next beta target:ga The Epic is ready for focal approvals, after which it can GA. target:24008-beta target:240012 team:MicroProfileUK Translation - Complete All translation has been delivered to release branch

Comments

@Emily-Jiang
Copy link
Member

Emily-Jiang commented Dec 7, 2023

Description

Implement MicroProfile 7.0. MicroProfile 7.0 has the following update (the content might change).

  • Adopt OpenTelemetry Metrics and/or Logging in MicroProfile Telemetry 2.0
  • Move MicroProfile Metrics to be a standalone spec
  • Make Jakarta EE Core Profile dependency loosely coupled
  • Update other specifications accordingly: Rest Client, Fault Tolerance, Open API

MP Open API 4.0
MP Telemetry 2.0
MP Fault Tolerance 4.1
MP Rest Client 4.0

This MP Compatibility issue also needs resolving before GA: #28980


Documents

When available, add links to required feature documents. Use "N/A" to mark particular documents which are not required by the feature.


Process Overview

General Instructions

The process steps occur roughly in the order as presented. Process steps occasionally overlap.

Each process step has a number of tasks which must be completed or must be marked as not applicable ("N/A").

Unless otherwise indicated, the tasks are the responsibility of the Feature Owner or a Delegate of the Feature Owner.

If you need assistance, reach out to the OpenLiberty/release-architect.

Important: Labels are used to trigger particular steps and must be added as indicated.


Prioritization (Complete Before Development Starts)

The (OpenLiberty/chief-architect) and area leads are responsible for prioritizing the features and determining which features are being actively worked on.

Prioritization

  • Feature added to the "New" column of the Open Liberty project board
    • Epics can be added to the board in one of two ways:
      • From this issue, use the "Projects" section to select the appropriate project board.
      • From the appropriate project board click "Add card" and select your Feature Epic issue
  • Priority assigned
    • Attend the Liberty Backlog Prioritization meeting

Design (Complete Before Development Starts)

Design preliminaries determine whether a formal design, which will be provided by an Upcoming Feature Overview (UFO) document, must be created and reviewed. A formal design is required if the feature requires any of the following: UI, Serviceability, SVT, Performance testing, or non-trivial documentation/ID. Furthermore, each identified item places a blocking requirement on another team so it must be identified early in the process. The feature owner may check-off the item if they know it doesn't apply, but otherwise they should work with the focal point to determine what work, if any, will be necessary and make them aware of it.

Design Preliminaries

  • UI requirements identified, or N/A. (Feature owner and UI focal point)
  • Accessibility requirements identified, or N/A. (Feature owner and Accessibility focal point)
  • ID requirements identified, or N/A. (Feature owner and ID focal point)
    • Refer to Documenting Open Liberty.
    • Feature Owner adds label ID Required, if non-trivial documentation needs to be created by the ID team.
    • ID adds label ID Required - Trivial, if no design will be performed and only trivial ID updates are needed.
  • Serviceability requirements identified, or N/A. (Feature owner and Serviceability focal point)
  • SVT requirements identified, or N/A. (Feature owner and SVT focal point)
  • Performance testing requirements identified, or N/A. (Feature owner and Performance focal point)

Design

  • POC Design / UFO review requested.
    • Feature owner adds label Design Review Request
  • POC Design / UFO review scheduled.
    • Follow the instructions in POC-Forum repo
  • POC Design / UFO review completed.
  • POC / UFO Review follow-ons completed.
  • POC Design / UFO approval requested.
    • Feature owner adds label Design Approval Request
  • Design / UFO approved. (OpenLiberty/chief-architect) or N/A
    • (OpenLiberty/chief-architect) adds label Design Approved
    • Add the public link to the UFO in Box to the Documents section.
    • The UFO must always accurately reflect the final implementation of the feature. Any changes must be first approved. Afterwards, update the UFO by creating a copy of the original approved slide(s) at the end of the deck and prepend "OLD" to the title(s). A single updated copy of the slide(s) should take the original's place, and have its title(s) prepended with "UPDATED".

No Design

  • No Design requested.
    • Feature owner adds label No Design Approval Request
  • No Design / No UFO approved. (OpenLiberty/chief-architect) or N/A
    • Approver adds label No Design Approved
  • Feature / Capability stabilization or discontinuation or N/A
    • Feature owner adds label Product Management Approval Request and notifies OpenLiberty/product-management
    • Approver adds label Product Management Approved (OpenLiberty/product-management)
    • Note: For stabilized, superseded, and discontinued feature/capability, skip the Beta section of the template (you may delete it). Otherwise, proceed as normal.

FAT Documentation


Implementation

A feature must be prioritized before any implementation work may begin to be delivered (inaccessible/no-ship). However, a design focused approach should still be applied to features, and developers should think about the feature design prior to writing and delivering any code.
Besides being prioritized, a feature must also be socialized (or No Design Approved) before any beta code may be delivered. All new Liberty content must be inaccessible in our GA releases until it is Feature Complete by either marking it kind=noship or beta fencing it.
Code may not GA until this feature has obtained the Design Approved or No Design Approved label, along with all other tasks outlined in the GA section.

Feature Development Begins

  • Add the In Progress label

Legal and Translation

In order to avoid last minute blockers and significant disruptions to the feature, the legal items need to be done as early in the feature process as possible, either in design or as early into the development as possible. Similarly, translation is to be done concurrently with development. Both MUST be completed before Beta or GA is requested.

Legal (Complete before Feature Complete Date)

  • Changed or new open source libraries are cleared and approved, or N/A. (Legal Release Services/Cass Tucker/Release PM).

Innovation (Complete 1 week before Feature Complete Date)

  • Consider whether any aspects of the feature may be patentable. If any identified, disclosures have been submitted.

Translation (Complete by Feature Complete Date)

  • PII (Program Integrated Information) updates are merged (i.e. all English strings due for translation have been delivered), or N/A.

Beta

In order to facilitate early feedback from users, all new features and functionality should first be released as part of a beta release.

Beta Code

  • Beta fence the functionality
    • E.g. kind=beta, ibm:beta, ProductInfo.getBetaEdition()
  • Beta development complete and feature ready for inclusion in a beta release
    • Add label target:beta and the appropriate target:YY00X-beta (where YY00X is the targeted beta version).
  • Feature delivered into beta

Beta Blog (Complete by beta eGA)

  • Beta blog issue created and populated using the Open Liberty BETA blog post template.
    • Add a link to the beta blog issue in the Documents section.
    • Note: This is for inclusion into the overall beta release blog post. If, in addition, you'd also like to create a dedicated blog post about your feature, then follow the "Standalone Feature Blog Post" instructions under the Other Deliverables section.

GA

A feature is ready to GA after it is Feature Complete and has obtained all necessary Focal Point Approvals.

Feature Complete

  • Feature implementation and tests completed.
    • All PRs are merged.
    • All related/child issues are closed.
    • All stop ship issues are completed.
  • Legal: all necessary approvals granted.
  • Translation: Feature may only proceed to GA if it has either Translation - Complete or Translation - Missing label
    • If all translation has been delivered to release branch, feature owner adds label Translation - Complete.
    • If missing translation does not cause a break in functionality, nor a security or production outage risk, feature owner adds label Translation - Missing.
      • Once all missing translations are delivered, the Translation - Missing label is replaced with Translation - Complete.
    • If missing translation could cause a break in functionality or a security or production outage risk, feature owner adds the Translation - Blocked label.
      • Featues with Translation - Blocked may NOT proceed to GA until the label has been replaced with either Translation - Missing or Translation - Complete.
    • For further guidance, contact Globalization focal point or the Release Architect.
  • GA development complete and feature ready for inclusion in a GA release
    • Add label target:ga and the appropriate target:YY00X (where YY00X is the targeted GA version).
    • Inclusion in a release requires the completion of all Focal Point Approvals.

Focal Point Approvals (Complete by Feature Complete Date)

These occur only after GA of this feature is requested (by adding a target:ga label). GA of this feature may not occur until all approvals are obtained.

All Features

  • APIs/Externals - Externals have been reviewed or N/A. (OpenLiberty/externals-approvers)
    • Approver adds label focalApproved:externals
  • Demo - Demo is scheduled for an upcoming EOI or N/A. (OpenLiberty/demo-approvers)
    • Add comment @OpenLiberty/demo-approvers Demo scheduled for EOI [Iteration Number] to this issue.
    • Approver adds label focalApproved:demo.
  • FAT - All Tests complete and running successfully in SOE or N/A. (OpenLiberty/fat-approvers)
    • Approver adds label focalApproved:fat.

Design Approved Features

  • ID - Documentation is complete or N/A. (OpenLiberty/id-approvers)
    • Approver adds label focalApproved:id.
    • NOTE: If only trivial documentation changes are required, you may reach out to the ID Feature Focal to request a ID Required - Trivial label. Unlike features with regular ID requirement, those with ID Required - Trivial label do not have a hard requirement for a Design/UFO.

  • InstantOn - InstantOn capable or N/A. (OpenLiberty/instantOn-approvers)
    • Approver adds label focalApproved:instantOn.
  • Performance - Performance testing is complete or N/A. (OpenLiberty/performance-approvers)
    • Approver adds label focalApproved:performance.
  • Serviceability - Serviceability has been addressed or N/A. (OpenLiberty/serviceability-approvers)
    • Approver adds label focalApproved:sve.
  • STE - Skills Transfer Education chart deck is complete or N/A. (OpenLiberty/ste-approvers)
    • Approver adds label focalApproved:ste.
  • SVT - System Verification Test is complete or N/A. (OpenLiberty/svt-approvers)
    • Approver adds label focalApproved:svt.

Remove Beta Fencing (Complete by Feature Complete Date)

  • Beta guards are removed, or N/A
    • Only after all necessary Focal Point Approvals have been granted.

GA Blog (Complete by Friday after GM)

  • GA Blog issue created and populated using the Open Liberty GA release blog post template.
    • Add a link to the GA Blog issue in the Documents section.
    • Note: This is for inclusion into the overall release blog post. If, in addition, you'd also like to create a dedicated blog post about your feature, then follow the "Standalone Feature Blog Post" instructions under the Other Deliverables section.

Post GM (Complete before GA)

  • After confirming this feature has been included in the GM driver, feature owner closes this issue.

Post GA


Other Deliverables


@Emily-Jiang Emily-Jiang added the Epic Used to track Feature Epics that are following the UFO process label Dec 7, 2023
@tevans78 tevans78 changed the title Implement MicroProfile 7.0 MP70: Implement MicroProfile 7.0 Jan 10, 2024
@tevans78 tevans78 added the In Progress Items that are in active development. label Feb 12, 2024
@tevans78 tevans78 added target:beta The Epic or Issue is targetted for the next beta target:24008-beta labels Jun 5, 2024
@idlewis
Copy link
Member

idlewis commented Jun 26, 2024

Comments from the UFO socialization below. Some of these were addressed during the socialization.

  • Q. Slide 7: Should the mp-7 convenience feature rely on the version-less Jakarta Core Profile feature?
    A. No, this was discussed on the UFO socialization for the version-less features
  • Q. Slide 8: When EE12 is released, will the MP7 features be updated to work with it?
    A. Maybe, it will depend on whether they are compatible or not. A new MP release may be needed
  • Slide 9: The colours are a little unclear
  • Slide 11: 'Trace system such as prometheus' should be 'Metrics system such as prometheus'
  • Slide 13: 'Support multiparts' is a little unclear, it isn't a change to the API
  • Slide 14: Would be good to add tempo/Grafana for traces on the RHS.
  • Slide 26: SVT needs to include MP Metrics 5.1
  • Slide 28: It isn't clear which Java versions are needed to ratify the specification
  • Slide 28: Are the existing Liberty MP zip packages going to be deprecated in favour of the 7.0 zip package?
  • Slide 32: Migration impact - customers will need to add metrics 5.1 feature when they didn't previously

@Emily-Jiang
Copy link
Member Author

Comments from the UFO socialization below. Some of these were addressed during the socialization.

* Q. Slide 7: Should the mp-7 convenience feature rely on the version-less Jakarta Core Profile feature?
  A. No, this was discussed on the UFO socialization for the version-less features

* Q. Slide 8: When EE12 is released, will the MP7 features be updated to work with it?
  A. Maybe, it will depend on whether they are compatible or not. A new MP release may be needed

* Slide 9: The colours are a little unclear

updated

* Slide 11: 'Trace system such as prometheus' should be 'Metrics system such as prometheus'

reworked

* Slide 13: 'Support multiparts' is a little unclear, it isn't a change to the API

updated

* Slide 14: Would be good to add tempo/Grafana for traces on the RHS.

done

* Slide 26: SVT needs to include MP Metrics 5.1

done

* Slide 28: It isn't clear which Java versions are needed to ratify the specification

This is not related to this UFO. For releasing the spec, Java 11, 17 and 21 are needed to ratify the specification.

* Slide 28: Are the existing Liberty MP zip packages going to be deprecated in favour of the 7.0 zip package?

done

* Slide 32: Migration impact - customers will need to add metrics 5.1 feature when they didn't previously

done

@Emily-Jiang
Copy link
Member Author

This is a umbrella feature for MicroProfile. There is no new functionalities delivered under this feature. STE is not needed. Please add n/a for this one. @tngiang73

@Emily-Jiang Emily-Jiang added the target:ga The Epic is ready for focal approvals, after which it can GA. label Oct 2, 2024
@Emily-Jiang
Copy link
Member Author

Id issue is here

@abutch3r abutch3r added the focalApproved:demo Approval that a Demo has been scheduled label Oct 2, 2024
@tngiang73 tngiang73 added the focalApproved:ste Focal Approval granted for STE for the feature label Oct 2, 2024
@donbourne
Copy link
Member

donbourne commented Oct 10, 2024

automatic serviceability approval for MP umbrella features

@donbourne donbourne added the focalApproved:serviceability Focal Approval granted for Serviceability for the feature label Oct 10, 2024
@Emily-Jiang Emily-Jiang added the focalApproved:globalization Focal Approval granted for Globalization for the feature label Oct 16, 2024
@NottyCode
Copy link
Member

@tevans78 @Emily-Jiang the UFO link is not resolving.

@Emily-Jiang
Copy link
Member Author

@tevans78 @Emily-Jiang the UFO link is not resolving.

The link has been updated My apologies for the unresolved links. I was not sure what happened as I thought I set the expiry date.

@Emily-Jiang
Copy link
Member Author

@tjwatson please do the approval for InstantOn

@Emily-Jiang
Copy link
Member Author

@cbridgha please add External approval as this feature has no API/SPIs exposed.

@tjwatson tjwatson added the focalApproved:instantOn Focal Approval granted for InstantOn for the feature label Oct 30, 2024
@jdmcclur jdmcclur added the focalApproved:performance Focal Approval granted for Performance for the feature label Nov 6, 2024
@malincoln malincoln added Translation - Complete All translation has been delivered to release branch and removed focalApproved:globalization Focal Approval granted for Globalization for the feature labels Nov 8, 2024
@chirp1
Copy link
Contributor

chirp1 commented Nov 8, 2024

Ram has the info that he needs to document MP 7.0 in doc issue at OpenLiberty/docs#7610. Approving the epic.

@chirp1 chirp1 added the focalApproved:id Focal Approval granted for ID for the feature label Nov 8, 2024
@cbridgha cbridgha added the focalApproved:externals Focal Approval granted for APIs/Externals for the feature label Nov 8, 2024
@hanczaryk hanczaryk added the focalApproved:svt Focal Approval granted for SVT for the feature label Nov 12, 2024
@dave-waddling dave-waddling added the focalApproved:fat Focal Approval granted for FAT for the feature label Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Design Approved Epic Used to track Feature Epics that are following the UFO process focalApproved:demo Approval that a Demo has been scheduled focalApproved:externals Focal Approval granted for APIs/Externals for the feature focalApproved:fat Focal Approval granted for FAT for the feature focalApproved:id Focal Approval granted for ID for the feature focalApproved:instantOn Focal Approval granted for InstantOn for the feature focalApproved:performance Focal Approval granted for Performance for the feature focalApproved:serviceability Focal Approval granted for Serviceability for the feature focalApproved:ste Focal Approval granted for STE for the feature focalApproved:svt Focal Approval granted for SVT for the feature ID Required in:MicroProfile In Progress Items that are in active development. target:beta The Epic or Issue is targetted for the next beta target:ga The Epic is ready for focal approvals, after which it can GA. target:24008-beta target:240012 team:MicroProfileUK Translation - Complete All translation has been delivered to release branch
Projects
Status: Epics in progress
Development

No branches or pull requests