Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Group configurations by a single Yaml file #29

Closed
zhao-kun opened this issue Jun 16, 2021 · 1 comment · Fixed by #230
Closed

Group configurations by a single Yaml file #29

zhao-kun opened this issue Jun 16, 2021 · 1 comment · Fixed by #230
Assignees
@benja-wu
Labels
enhancement New feature or request
Projects
Easegress Project

Comments

@zhao-kun
Copy link
Collaborator

zhao-kun commented Jun 16, 2021 •
edited

Many applications require multiple configurations to be created, such as an HttpServer and a PipelineFilter. Management of multiple configurations that grouped together in a single file (separated by --- in YAML) will bring huge simplicity to users.
like Kubernetes [1]

[1] https://kubernetes.io/docs/concepts/cluster-administration/manage-deployment/
@zhao-kun zhao-kun added the enhancement New feature or request label Jun 16, 2021
@zhao-kun zhao-kun changed the title Group configuration by a single Yaml file Group configurations by a single Yaml file Jun 16, 2021
@benja-wu
Copy link
Contributor

I agree this feature can bring huge simplicity for Easegress CLI users. Since we are dealing with manage API framework replacement, I will implement this feature as soon as the replacement completed.

@zhao-kun zhao-kun added this to To do in Easegress Project Jun 17, 2021
nevill referenced this issue in nevill/easegress Sep 2, 2021
@nevill
improve egctl to support sending multi configs at once (fix #29)
bf3240f
@nevill nevill mentioned this issue Sep 2, 2021
Merged
@benja-wu benja-wu linked a pull request Sep 2, 2021 that will close this issue
improve egctl to support sending multi configs at once (fix #29) #230
Merged
nevill referenced this issue in nevill/easegress Sep 3, 2021
@nevill
improve egctl to support sending multi configs at once (fix #29)
ab208eb
Easegress Project automation moved this from To do to Done Sep 7, 2021
benja-wu pushed a commit that referenced this issue Sep 7, 2021
@nevill @localvar
improve egctl to support sending multi configs at once (fix #29) (#230)
4749392 
* improve egctl to support sending multi configs at once (fix #29)

* add a visitor to do the YAML parsing

* rollback function readFromFileOrStdin

* Update comments.

Co-authored-by: Bomin Zhang <[email protected]>

* some improvements
- change the signature of VisitorFunc
- remove useless function readFromFileOrStdin

Co-authored-by: Bomin Zhang <[email protected]>
xmh19936688 referenced this issue in xmh19936688/easegress Sep 13, 2021
@nevill @localvar @xmh19936688
improve egctl to support sending multi configs at once (fix #29) (eas…
71cbf1e 
…egress-io#230)

* improve egctl to support sending multi configs at once (fix #29)

* add a visitor to do the YAML parsing

* rollback function readFromFileOrStdin

* Update comments.

Co-authored-by: Bomin Zhang <[email protected]>

* some improvements
- change the signature of VisitorFunc
- remove useless function readFromFileOrStdin

Co-authored-by: Bomin Zhang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@benja-wu benja-wu

Labels
enhancement New feature or request
Projects
Easegress Project
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

improve egctl to support sending multi configs at once (fix #29) nevill/easegress
2 participants
@zhao-kun @benja-wu