start controller for subscription cache and prove it works #3716
Conversation
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: deads2k The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
deads2k
force-pushed
the
cs-80-subscription-informer
branch
from
27bff9e to
cfe150b
Compare
|
simulation timeout. I wonder if that is widespread /retest |
|
Decently significance: https://search.dptools.openshift.org/?search=408+Request+timed+out&maxAge=336h&context=1&type=junit&name=ARO-HCP&excludeName=&maxMatches=5&maxBytes=20971520&groupBy=job perhaps we need to reduce parallelism |
We're serializing this value so that we can use the standard .Get function in two weeks once all the older records have TTL'd out.
This needs to merge first because it allows HA frontends with old and new functioning in parallel to read the content each writes.
This will let a refactor depend on functions there instead of being a massive rewrite.
deads2k
force-pushed
the
cs-80-subscription-informer
branch
from
ec6e963 to
05e3c50
Compare
|
/retest |
|
role assignments /retest |
|
@deads2k: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
prove involves using the subscription cache to power a replacement for the cluster creation operation.
/hold
holding I haven't tried this yet.