Currently, revisions in an "Unknown" state don't have a pruning limit, so they can pile up forever. The revision pruner controller should be able to set a limit for these, similar to how it does for Succeeded and Failed revisions.
This bug hasn't had any activity in the last 30 days. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.
As such, we're marking this bug as "LifecycleStale" and decreasing severity from "medium" to "low".
If you have further information on the current state of the bug, please update it, otherwise this bug will be automatically closed in 7 days. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.
I've opened 2 PRs to add this feature request:
1. https://github.com/openshift/api/pull/643 - API change to add the field
2. https://github.com/openshift/library-go/pull/794 - to implement the limit in the pruning controller
This will also require bumps to our static pod operators to actually take effect, once the above 2 PRs merge (in order)
This bug has moved to Jira (https://issues.redhat.com/browse/WRKLDS-172) as we consider this an RFE.
Re-opening this as upon further discussion we found that this is a symptom of a bigger issue in the revision controllers. Linked PR has been updated
It looks this won't make 4.5 this week, I'm pushing this to 4.6, we'll consider backport when this lands.
The above PRs have closed and the main bug has moved to this PR: https://github.com/openshift/library-go/pull/806
Followed the process below to verify the bug here.
1) clone the repo https://github.com/openshift/library-go
2) export KUBECONFIG file
3) cd to the pr file directory which is /home/ramakasturinarra/automation/OpenShift/library-go/pkg/operator/revisioncontroller
4) Now run the command go test -v -run TestRevisionController
[ramakasturinarra@dhcp35-60 revisioncontroller]$ go test -v -run TestRevisionController
go: downloading k8s.io/apimachinery v0.18.3
go: downloading k8s.io/client-go v0.18.3
go: downloading k8s.io/api v0.18.3
go: downloading github.com/openshift/api v0.0.0-20200521101457-60c476765272
go: downloading k8s.io/kube-aggregator v0.18.3
go: downloading golang.org/x/net v0.0.0-20200421231249-e086a090c8fd
go: downloading k8s.io/apiextensions-apiserver v0.18.3
go: downloading k8s.io/kube-openapi v0.0.0-20200410145947-61e04a5be9a6
go: downloading k8s.io/component-base v0.18.3
go: downloading github.com/robfig/cron/v3 v3.0.1
go: downloading github.com/prometheus/client_golang v1.1.0
go: downloading github.com/prometheus/client_model v0.2.0
go: downloading golang.org/x/sys v0.0.0-20200323222414-85ca7c5b95cd
go: downloading github.com/prometheus/procfs v0.0.3
go: downloading github.com/json-iterator/go v1.1.9
go: downloading github.com/blang/semver v3.5.0+incompatible
go: downloading github.com/prometheus/common v0.6.0
go: downloading github.com/beorn7/perks v1.0.1
go: downloading github.com/matttproud/golang_protobuf_extensions v1.0.1
=== RUN TestRevisionController
=== RUN TestRevisionController/update_InProgress_to_Abandoned_revisions_when_interrupted
=== RUN TestRevisionController/set-latest-revision-by-configmap
=== RUN TestRevisionController/operator-unmanaged
=== RUN TestRevisionController/missing-source-resources
=== RUN TestRevisionController/copy-resources
=== RUN TestRevisionController/copy-resources-opt
=== RUN TestRevisionController/copy-resources-opt-missing
=== RUN TestRevisionController/latest-revision-current
=== RUN TestRevisionController/latest-revision-current-optionals-missing
--- PASS: TestRevisionController (0.00s)
--- PASS: TestRevisionController/update_InProgress_to_Abandoned_revisions_when_interrupted (0.00s)
--- PASS: TestRevisionController/set-latest-revision-by-configmap (0.00s)
--- PASS: TestRevisionController/operator-unmanaged (0.00s)
--- PASS: TestRevisionController/missing-source-resources (0.00s)
--- PASS: TestRevisionController/copy-resources (0.00s)
--- PASS: TestRevisionController/copy-resources-opt (0.00s)
--- PASS: TestRevisionController/copy-resources-opt-missing (0.00s)
--- PASS: TestRevisionController/latest-revision-current (0.00s)
--- PASS: TestRevisionController/latest-revision-current-optionals-missing (0.00s)
ok github.com/openshift/library-go/pkg/operator/revisioncontroller 0.024s
Based on comment 11 moving the bug to verified state. Below is the payload i have verified the bug.
[ramakasturinarra@dhcp35-60 revisioncontroller]$ oc get clusterversion
NAME VERSION AVAILABLE PROGRESSING SINCE STATUS
version 4.5.0-0.nightly-2020-06-08-010711 True False 10h Cluster version is 4.5.0-0.nightly-2020-06-08-010711
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 (OpenShift Container Platform 4.6 GA Images), and where to find the updated
files, follow the link below.
If the solution does not work for you, open a new bug report.