-
Notifications
You must be signed in to change notification settings - Fork 39.4k
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 message of failed sync #75260
Fix message of failed sync #75260
Conversation
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
lgtm |
/assign @tallclair @derekwaynecarr |
@jsafrane |
Are they the same? |
@@ -166,7 +166,7 @@ func (c *objectCache) Get(namespace, name string) (runtime.Object, error) { | |||
return nil, fmt.Errorf("object %q/%q not registered", namespace, name) | |||
} | |||
if err := wait.PollImmediate(10*time.Millisecond, time.Second, item.hasSynced); err != nil { | |||
return nil, fmt.Errorf("couldn't propagate object cache: %v", err) | |||
return nil, fmt.Errorf("failed to sync %s cache: %v", c.groupResource.Resource, err) |
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.
return nil, fmt.Errorf("failed to sync %s cache: %v", c.groupResource.Resource, err) | |
return nil, fmt.Errorf("failed to sync %s cache: %v", c.groupResource.String(), err) |
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.
fixed
From: MountVolume.SetUp failed for volume "secret-prometheus-k8s-proxy" : couldn't propagate object cache: timed out waiting for the condition To: MountVolume.SetUp failed for volume "secret-prometheus-k8s-proxy" : failed to sync secret cache: timed out waiting for the condition
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
i think this issue can be closed. |
ec7dd92
to
96c41f8
Compare
I forgot about this one .I still think this PR is useful, @tallclair, would you please check again? |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jsafrane, tallclair 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 |
Message
couldn't propagate object cache
ends up in Event about failed mount and it looks confusing to users. Tryingfailed to sync X cache
(where X is eithersecret
orconfigmap
).From:
MountVolume.SetUp failed for volume "secret-prometheus-k8s-proxy" : couldn't propagate object cache: timed out waiting for the condition
To:
MountVolume.SetUp failed for volume "secret-prometheus-k8s-proxy" : failed to sync secret cache: timed out waiting for the condition
/kind cleanup
/sig node