Comments (6)
To address the issue where Consul watch does not trigger notifications when the server goes offline, unlike with etcd, you might need to implement additional logic in your Kratos project. The current implementation in Kratos for Consul watches, as seen in contrib/registry/consul/watcher.go
, waits for an event or the context to be done without specifically handling server offline events.
Here's a conceptual approach to enhance the watch functionality to detect when the Consul server goes offline:
-
Implement Health Checks: Create health checks for the Consul server. When the server is detected as offline, trigger an event on the
w.event
channel. -
Extend the Watch Mechanism: Modify the existing watch mechanism to periodically verify the connectivity to the Consul server. On detecting a disconnection, trigger an event on the
w.event
channel. -
Utilize Consul Events: Leverage Consul's event system to detect server offline events and integrate them with the Kratos service discovery watch mechanism.
For example, you could implement a health check that triggers an event when the server goes offline as follows:
func (w *watcher) monitorConsulServer() {
ticker := time.NewTicker(10 * time.Second)
defer ticker.Stop()
for {
select {
case <-w.ctx.Done():
return
case <-ticker.C:
if !isConsulServerAlive() {
w.event <- struct{}{}
}
}
}
}
func isConsulServerAlive() bool {
// Logic to check if the Consul server is alive, e.g., a ping or health check
return true // Placeholder return value
}
This function should be called as a goroutine, possibly during the watcher's initialization. Adjust this example based on your specific requirements and environment. For more details on the watcher implementation, refer to the Kratos repository, specifically the contrib/registry/consul/watcher.go
file.
Details
Sources
- contrib/registry/consul/watcher.go
- contrib/config/etcd/watcher.go
- contrib/registry/etcd/watcher.go
- contrib/registry/consul/service.go
- contrib/registry/consul/registry.go
- contrib/config/consul/watcher.go
About Dosu
This response is meant to be useful and save you time. It is not meant to be a precise solution, but rather a starting point for your own research.
Help me be more useful! Please leave a 👍 if this is helpful and 👎 if it is irrelevant.
To continue the conversation, mention @dosu.
from kratos.
你好,@noovertime7!我是 Dosu,一个可以帮助你解决问题、回答问题以及成为贡献者的机器人。我在这里帮助你,同时等待人类维护者的回复。我会立即查看你的问题,并尽快给你一个完整的答案。
from kratos.
Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿
Hello, @noovertime7! I'm Dosu, a bot that helps you solve problems, answer questions, and become a contributor. I'm here to help you while I wait to hear back from the human maintainers. I will review your question immediately and give you a complete answer as soon as possible.
from kratos.
from kratos.
Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿
The message will only be received when the server is online, not when offline.
from kratos.
I'm not sure what you're referring to. I've been using this feature all along. Please provide a complete explanation or reproduce an example
from kratos.
Related Issues (20)
- [Feature] consule register need suport tags[] HOT 2
- When the App stops, the ctx in the afterStop callback is not available HOT 3
- [Question] How to return other Status Code in a successful HTTP request? HOT 7
- [Feature] Add Unwrap to custom response writer. HOT 3
- [Proposal] config feature roadmap HOT 22
- [Feature] replace kratos metrics with otel metrics HOT 3
- [The modification of header file information of http response does not take effect] ctx.Header().Set("Content-Type", "application/octet-stream") HOT 5
- [Question] How to return a custom response json? HOT 1
- Package version dependency issue HOT 4
- go generate ./...failed HOT 5
- [Feature]Add svrType related fields in proto to protoc-gen-go-http and http transport HOT 3
- proto file used to build the api document reported an error HOT 1
- [Feature] Add NewClient method to gRPC Client HOT 3
- [Question] DDD的最佳实践方式?包括CQRS、Clean Architecture、六边形结构 HOT 5
- [Question]请教下kratos如何定义支持上传文件接口? HOT 2
- metrics statistics and usage issues [Question] HOT 5
- `stream` response in proto failed to generate http client HOT 2
- [Feature] Add kratos errors to buf remote plugin HOT 1
- [Question] I modified it based on examples/helloworld and found that client/main.go reported an error when executing. I am not sure if it is a BUG.
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 kratos.