-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Feature/initial view #2866
base: master
Are you sure you want to change the base?
Feature/initial view #2866
Conversation
Created a new pull request, because I messed up my git. |
Hello there! Thank you for your PR and your work on k9s! There is already a way of setting active views per cluster - in the cluster specific configuration file ( k9s:
[...]
view:
active: po The problem here is: the value for the cluster specific config is automatically generated and sets the view to a default value (I think So to implement the @derailed I think your input is required here as to make a decision how to proceed with this feature. |
@KevinGimbel I believe this behavior corresponds to the last active view for the cluster/context. By default, K9s always starts in the last active view unless the The changes proposed in this PR don't alter that behavior:
|
@derailed have you had a chance to look at this? |
This pull request introduces a new configuration setting called
initialView
that allows users to specify the initial view in the application. The changes span across multiple files to integrate this new setting into the application logic, configuration, and documentation.Configuration and Documentation Updates:
README.md
: Added a description of theinitialView
setting to the configuration section.internal/config/json/schemas/k9s.json
: AddedinitialView
as a new string type configuration option.Application Logic Updates:
cmd/root.go
: Modified therun
function to set the active view based on theinitialView
configuration if it is specified.internal/config/k9s.go
:InitialView
as a new field in theK9s
struct.Merge
method to include merging theInitialView
field.