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.

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

Remove Per-Route Options in the Apps and the App commands [v8] #3409

Merged
merged 4 commits into from
Feb 19, 2025

Conversation

Dariquest
Copy link
Contributor

Per-Route Options in the Apps and the App commands

CLI Support for Generic Per-Route Options were introduced with the extension of the output of the commands app and apps. The per-route options are being shown directly behind the routes as a comma separated list of key-value pairs in the curly braces. For example:

cf app
Getting apps in org org1 / space space1 as user1...
 
name         requested state   processes   routes
app1           started                 web:1/1        test1.test/bu2 {loadbalancing=least-connections}
 
 
cf apps
Showing health and status for app app1 in org org1 / space space1 as user1...
 
name:                     app1
requested state:   started
routes:                   test1.test, test1.test/bu2 {loadbalancing=least-connections}
 
 
cf route test --hostname test1 --path /bu16
 Showing route test1.test/bu16 in org org1 / space space1 as user1...
 
domain:     test
host:       test1
port:       
path:       /bu16
protocol:   http
options:    {loadbalancing=round-robin}

There were some concerns, that this output might become too verbose and is considered being a breaking change.
This PR is removing the per-route options from the app and apps commands.

Applicable Issues

GitHub Issue

How Urgent Is The Change?

No urgency

@Dariquest Dariquest changed the title Per-Route Options in the Apps and the App commands [v8] Remove Per-Route Options in the Apps and the App commands [v8] Feb 14, 2025
Copy link
Contributor

@joaopapereira joaopapereira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@joaopapereira joaopapereira merged commit a648dd2 into cloudfoundry:v8 Feb 19, 2025
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants