Skip to content
This repository was archived by the owner on Mar 31, 2023. It is now read-only.

cni plugin may need to retry query on port status #7

Open
h-w-chen opened this issue Jan 17, 2020 · 0 comments
Open

cni plugin may need to retry query on port status #7

h-w-chen opened this issue Jan 17, 2020 · 0 comments
Labels
P2

Comments

@h-w-chen
Copy link
Contributor

current implementation of cni plugin stops the Add op if it fails to query port status, or the response code is not 200. It may make sense to categorize some errors as retry-on-site, and immediately give it retry, in a hope the retry overcome the transient problem and get response body with 200 status code.

This is not high priority issue, as CNI plugin would be invoked by CRI periodically in case of Add operation failure.
@h-w-chen h-w-chen added bug Something isn't working P2 labels Jan 17, 2020
@h-w-chen h-w-chen mentioned this issue Jan 17, 2020
Merged
@h-w-chen h-w-chen removed the bug Something isn't working label Jan 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
P2
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@h-w-chen