-
Notifications
You must be signed in to change notification settings - Fork 932
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
RFC0027 CLI command "update-route" to manage generic per-route options #3338
base: v8
Are you sure you want to change the base?
Conversation
a67ba18
to
95cb4be
Compare
FYI @gururajsh, I also pinged you via gChat |
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.
Thank you for your contribution. However, we do not make changes in the cf
folder because it predates version 6 and is marked as deprecated. We hope to remove it soon, as plugins are the only reason it remains in the code.
Also, we noticed that you've made your original request against the |
Hi @a-b, I was advised in the cli channel, that I should develop against the version branches. The new command is intended for the v8 and not for the v7 (as far as I understood from the contribution guide, main will become v9 later, and v8 is the target branch if I intend to add a v8 command). Have I got it wrong? |
CLI Support for Generic Per-Route Options
Introducing a new cli command 'update-route', which allows updates to route resources.
Extending the API of the Route to contain generic options map.
Detailed specification including that of for all the involved components is provided in the RFC
It should be possible to provide options in the create-route and map-route commands.
Why Is This PR Valuable?
Allow users to manage per-route options via the CLI instead of having to talk to the API directly.
Applicable Issues
GitHub Issue
How Urgent Is The Change?
No urgency