Bug 1835004
Summary: | keepYoungerThan is expecting one integer instead of one String | ||
---|---|---|---|
Product: | OpenShift Container Platform | Reporter: | Odilon Sousa <osousa> |
Component: | Image Registry | Assignee: | Oleg Bulatov <obulatov> |
Status: | CLOSED ERRATA | QA Contact: | Wenjing Zheng <wzheng> |
Severity: | urgent | Docs Contact: | |
Priority: | unspecified | ||
Version: | 4.4 | CC: | aos-bugs, apjagtap, cblecker, dahernan, palonsor, rh-container, skrenger |
Target Milestone: | --- | ||
Target Release: | 4.5.0 | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: |
Cause: keepYoungerThan is expected to be an integer. It is an integer that defines a minimal image age in nanoseconds.
Consequence: It's inconvenient to use nanoseconds to tune the pruner.
Fix: Deprecate keepYoungerThan and introduce a new field named keepYoungerThanDuration.
Result: keepYoungerThanDuration can be set to "60m" and similar values.
|
Story Points: | --- |
Clone Of: | Environment: | ||
Last Closed: | 2020-07-13 17:37:59 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Odilon Sousa
2020-05-12 21:26:08 UTC
Below error still return when define like "keepYoungerThan: 60m" with 4.5.0-0.nightly-2020-05-27-174108: # imagepruners.imageregistry.operator.openshift.io "cluster" was not valid: # * spec.keepYoungerThan: Invalid value: "string": spec.keepYoungerThan in body must be of type integer: "string" Below setting can be saved: spec: failedJobsHistoryLimit: 3 keepTagRevisions: 3 keepYoungerThan: 60 schedule: "" successfulJobsHistoryLimit: 3 suspend: false This config change works: spec: failedJobsHistoryLimit: 3 keepTagRevisions: 3 keepYoungerThan: 60 keepYoungerThanDuration: 60m I will report a doc bug to deprecate "keepYoungerThan". If keepYoungerThan is deprecated, can we pop up a warning let user know to use keepYoungerThanDuration when user wants to config keepYoungerThan? Unfortunately we cannot change the type of keepYoungerThan from an integer to string. We should replace keepYoungerThan with keepYoungerThanDuration in docs. It's better to almost completely remove keepYoungerThan from documentation, we can just mention that this field exists and shouldn't be used. (In reply to Oleg Bulatov from comment #8) > Unfortunately we cannot change the type of keepYoungerThan from an integer > to string. > > We should replace keepYoungerThan with keepYoungerThanDuration in docs. It's > better to almost completely remove keepYoungerThan from documentation, we > can just mention that this field exists and shouldn't be used. OK, get it, thanks! I will add your suggestion to the doc bug: https://bugzilla.redhat.com/show_bug.cgi?id=1840973 . Feel free to change this bug back to ON_QA. Verified on 4.5.0-0.nightly-2020-05-29-005153. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHBA-2020:2409 |