Comments (7)
ncdc
commented on August 20, 2024
It really depends on your use case. Do you need to reprocess the entire cached set of endpoints/services/etc every 30 minutes? In the event that the shared informer loses its connection to the apiserver for the watch, it should do a full re-list, generating events for you to process.
from contour.
davecheney
commented on August 20, 2024
tbh, the reading I've done about this generally boils down to "you
shouldn't need this, but it's probably a good idea", which is non committal
and not useful. The resync feels like the watcher doesn't trust itself, or
the api server, fully.
…On Fri, Jan 5, 2018 at 8:56 AM, Andy Goldstein ***@***.***> wrote:
It really depends on your use case. Do you need to reprocess the entire
cached set of endpoints/services/etc every 30 minutes? In the event that
the shared informer loses its connection to the apiserver for the watch, it
should do a full re-list, generating events for you to process.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#130 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAAcA-w6N4okn3VhUOabTVrcwa7qhOcwks5tHUkqgaJpZM4RTvP4>
.
from contour.
davecheney
commented on August 20, 2024
Note for someone taking this ticket. The change is going to be pretty straight forward, a one line patch to change or disable the resync timer. But, what I want to see in this issue beforehand is some research about the cost and impact of this change.
Thank you.
from contour.
davecheney
commented on August 20, 2024
Assigning this to 0.5 for someone to take a swing at. The Jan 18 comment is pretty accurate if you want to try.
from contour.
davecheney
commented on August 20, 2024
Bumping to 0.6, this is a good first issue.
from contour.
ncdc
commented on August 20, 2024
For a bit more clarification - if you are combining state of k8s objects w/some external system, you would probably want to have a resync period > 0 so you can periodically reconcile the k8s objects' expectations with the reality of the external system. If you're only working with k8s objects, you shouldn't need a resync period > 0 .
from contour.
davecheney
commented on August 20, 2024
Nice. Thanks for the clarification. As you said, I'm watching k8s as the
source of truth, so YAGNI resync timers.
…On 18 May 2018 at 22:11, Andy Goldstein ***@***.***> wrote:
For a bit more clarification - if you are combining state of k8s objects
w/some external system, you would probably want to have a resync period > 0
so you can periodically reconcile the k8s objects' expectations with the
reality of the external system. If you're only working with k8s objects,
you shouldn't need a resync period > 0 .
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#130 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAAcA9e2zUxCG8RuatlnuuPkxSchSYa8ks5tzrqBgaJpZM4RTvP4>
.
from contour.
Related Issues (20)
- Consider making GRPCRoute a required informer now that it is v1 HOT 1
- Annotate envoy fleet HOT 2
- Contour gateway api provisionner revert annotations HOT 3
- Periodically having `connection failure` HOT 4
- Bug: Remove unnecessary permissions in Helm Charts HOT 2
- Add a separate label for envoy or contour HOT 1
- Add support for HTTP based external auth server HOT 5
- do we have cookie based routing for httpproxy? HOT 4
- Cost saving: Feature request to enable switch to toggle Envoy compression filter HOT 3
- Allow External Auth when Fallback Certificate is used HOT 1
- JWKS Async Fetch HOT 1
- `responseTimeout` not applied for external auth extension service HOT 1
- Support Circuit breakers for extension services
- Add missing headers without overwriting in the `requestHeaderPolicy`
- HTTPRoute conflicts when specifying a fallback https listener HOT 1
- Status.LoadBalancer is empty when using service port other than 80 HOT 2
- Service AppProtocol should allow IANA standard service names like "http", "https" HOT 3
- Gateway-API: InvalidCertificateRef error if certificateRefs.group is set to "core" HOT 3
- Support for adaptive concurrency HOT 1
- PathRewitePolicy behaving weird with condition prefix and header match HOT 1
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 contour.