-
Notifications
You must be signed in to change notification settings - Fork 151
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(cache): since we add new secret from modelreg with cache, we need to lose the cache for istio-system ns #1223
fix(cache): since we add new secret from modelreg with cache, we need to lose the cache for istio-system ns #1223
Conversation
… to lose the cache for istio-system ns Signed-off-by: Wen Zhou <wenzhou@redhat.com>
1 node(s) had untolerated taint {node-role.kubernetes.io/ci-builds-worker: ci-builds-worker}, 13 node(s) were unschedulable, 3 node(s) had untolerated taint {node-role.kubernetes.io/infra: }, 3 node(s) had untolerated taint {node-role.kubernetes.io/master: }, 6 node(s) didn't match Pod's node affinity/selector. no new claims to deallocate, preemption: 0/26 nodes are available: 26 Preemption is not helpful for scheduling.................. |
/retest |
1 similar comment
/retest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: VaishnaviHire The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
c49b501
into
opendatahub-io:incubation
… to lose the cache for istio-system ns (opendatahub-io#1223) Signed-off-by: Wen Zhou <wenzhou@redhat.com> (cherry picked from commit c49b501)
… to lose the cache for istio-system ns (opendatahub-io#1223) Signed-off-by: Wen Zhou <wenzhou@redhat.com> (cherry picked from commit c49b501)
Description
it is causing problem in modelreg with reconcile in Operator pod
related to #1190
How Has This Been Tested?
Screenshot or short clip
Merge criteria