-
Notifications
You must be signed in to change notification settings - Fork 501
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
update roadmap for 2022 #445
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.
I have some improvements in my backup list, @localvar could decide whether to add (part of) them into the roadmap:
- Eliminate global components(such as logger) in order to get a better experience of unit testing, even we need a global/simple unit testing framework for Easegress itself.
- API layer uses standard json(generated by protobuf), and CLI could produce/consume both yaml and json like kubectl.
- Make json schema failure messages more readable(current ones are not indexed).
- Support updating config for system controller.
- Clarify the location of config and status of static config (fix httpserver httppipeline status not show error #441 (comment)).
I strongly agree with the second/third point. |
We have |
we also have |
IMHO, yes. My deep opinion is actually we need to make logger modularized and automatical environment perception, for example: func (rl *RateLimiter) Init() {
// ...
if x {
// It prints `[filter/RateLimiter/ratelimiter-001]: BUG: duration is invalid` to stderr and file stdout.log.
// But in unit testing, it prints the same message to the stderr only.
rl.Log().Bugf("duration is invalid")
return
}
// It prints `[filter/RateLimiter/ratelimiter-001]: initialization completed`.
// The printing endpoint chosen are the same above.
rl.Log().Infof("initialization completed")
} And object/supervisor/cluster and so on got different prefixes for their own modules/scopes, I know it seems a little complex for Golang stuff. But as Easegress codebase inflates, the modularized logger should be the road ahead. |
updated according to comments. For "API layer uses standard json(generated by protobuf)", I think it has been covered by "Protobuf models generating". |
How about adding |
added.
|
Do we need to add some enterprise features to the roadmap? For example:
A general mTLS support? The #467 is dedicated to the EaseMesh, we could figure out a more general mTLS support? Continue refining cookbooks and manuals should be a long term goal |
No description provided.