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
- Remove dependency on github.com/pkg/errors
Why is this important?
- github.com/pkg/errors is an archived repo see https://github.com/pkg/errors#roadmap
- how is was done on prometheus: https://github.com/prometheus/prometheus/issues/12960
- how it was done on prometheus-operator: https://github.com/prometheus-operator/prometheus-operator/pull/5930
- One could enable depguard to avoid re-adding the package in the future.
- One could also consider https://github.com/polyfloyd/go-errorlint
NOTE:
- github.com/pkg/errors.Wrap knows how to handle err=nil, you'd need to handle that case separately.
- https://github.com/xdg-go/go-rewrap-errors may help with the migration but it isn't complete (it doesn't handle err=nil cases e.g.)
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>
- is related to
-
MON-3646 Direct deps still using github.com/pkg/errors
- Closed