-
Epic
-
Resolution: Won't Do
-
Major
-
None
-
Support the NTO/PAO merge
-
False
-
False
OCP/Telco Definition of Done
Epic Template descriptions and documentation.
<--- Cut-n-Paste the entire contents of this description into your new Epic --->
Epic Goal
- As outlined in the OpenShift enhancement https://github.com/openshift/enhancements/pull/867, The Node Tuning Operator and Performance Addon Operator are getting merged into a single operator running as a core OpenShift component. The CNF team is working on refactoring NTO to use controller-runtime as described by https://issues.redhat.com/browse/CNF-2448 and https://issues.redhat.com/browse/CNF-3431. The PSAP team needs to support this effort in the following ways:
-
- Code reviews and support (possibly even pair-programming) with the PAO team.
- Expand the e2e test coverage (e.g. Tuned spec.ManagementState) to catch issues with the controller-runtime NTO rewrite.
- Prepare tests to assess the impact of the NTO/PAO merge on CPU/Memory/API server/…
Why is this important?
- The NTO rewrite to use controller-runtime is a huge undertaking that needs to be well supported from the PSAP team side. Otherwise we risk regressions not only in terms of bugs, but also operator performance impact.
Scenarios
- ...
Acceptance Criteria
- CI - MUST be running successfully with tests automated
- Release Technical Enablement - Provide necessary release enablement details and documents.
- ...
Dependencies (internal and external)
- ...
Previous Work (Optional):
- …
Open questions::
- …
Done Checklist
- CI - CI is running, tests are automated and merged.
- Release Enablement <link to Feature Enablement Presentation>
- DEV - Upstream code and tests merged: <link to meaningful PR or GitHub Issue>
- DEV - Upstream documentation merged: <link to meaningful PR or GitHub Issue>
- DEV - Downstream build attached to advisory: <link to errata>
- QE - Test plans in Polarion: <link or reference to Polarion>
- QE - Automated tests merged: <link or reference to automated tests>
- DOC - Downstream documentation merged: <link to meaningful PR>