Comments (3)
The AddressBlock resource should have the index last allocated by Coil (lastAllocatedIndex)
@terassyi
Do you mean AddressBlock resource will be updated every time a new address is
assigned to / freed from a Pod? If so, I don't think it's a good idea.
Coil v1 had a performance problem because of this very behavior, so
Coil v2 was redesigned not to update resources in the kube-apiserver when
it allocates IP addresses from an AddressBlock.
from coil.
Thank you for your comment.
Do you mean AddressBlock resource will be updated every time a new address is
assigned to / freed from a Pod? If so, I don't think it's a good idea.
I misunderstand what struct should have information.
The AddressBlock
resource is the target of the reconciliation loop.
So if AddressBlock has the lastAllocatedIndex
parameter, every time lastAllocatedIndex
is updated, the reconciliation loop is triggered.
So we should avoid that AddressBlock
has the lastAllocatedIndex
.
Is it correct?
Now, I think the struct that should have information on the index last allocated is allocator
.
How about this idea?
Line 11 in cdab798
from coil.
So we should avoid that AddressBlock has the lastAllocatedIndex.
Is it correct?
Correct.
Now, I think the struct that should have information on the index last allocated is allocator.
SGTM.
from coil.
Related Issues (20)
- [BUG] When using the following DualStack Pool I cannot create an interface with IPs for the pod. HOT 29
- Support Kubernetes 1.23 and update dependencies
- Coil-egress accidentally deletes a peer
- Enhance CNI delete delay implementation HOT 1
- Create PDB for Egress NAT pods
- Enhance the graceful termination for Egress NAT HOT 2
- Support Kubernetes 1.25 and update dependencies
- Support Kubernetes 1.26 and update dependencies
- useless replace usage left in go.mod HOT 1
- Coil egress has downtime due to the timing of updating coild and coil controller HOT 7
- Use encap-sport auto in FOU tunnel setting for coil-egress HOT 1
- Support Kubernetes 1.27 and update dependencies
- Remove the unnecessary code block for v1 migration
- CNI issue in kind-created cluster HOT 5
- Unable to delete AddressPool
- Rename the CRD's finalizer name
- Two AddressBlocks are created when coil-controller is temporarily down HOT 5
- Standalone Egress NAT HOT 7
- Fix not to allocate addresses from pools while its pools are deleting
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from coil.