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

What other kn config options makes sense to load and save? #259

Closed
maximilien opened this issue Jul 11, 2019 · 4 comments
Closed

What other kn config options makes sense to load and save? #259

maximilien opened this issue Jul 11, 2019 · 4 comments
Labels
kind/question Issue type question kind/usage Issue type referring to usage of client lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale.

Comments

@maximilien
Copy link
Contributor

maximilien commented Jul 11, 2019

In what area(s)?

Configuration / startup.

Classifications:

/kind usage

Ask your question here:

With #249 we will have one option in the kn config pluginDir that allows the end user to load and save their chosen directory for plugins.

Since we added config to support this, the question is, are there other config options we should be saving and loading. For instance:

  • quiet
  • namespace
  • ?

Or maybe nothing. Just want to start a discussion on the topic since that might dictate how #249 loads and saves the config (in terms of when and how eager).

@maximilien maximilien added the kind/question Issue type question label Jul 11, 2019
@knative-prow-robot knative-prow-robot added the kind/usage Issue type referring to usage of client label Jul 11, 2019
@sixolet
Copy link
Contributor

sixolet commented Jul 19, 2019

  • Do you want to client-side generate revision names?
  • Do you want to protect yourself from unexpected code changes by locking images to digest in non-image updates?

@maximilien
Copy link
Contributor Author

maximilien commented Jul 19, 2019

We should probably start suggesting names and defaults too. I am changing how I save config and only do so when command succeeds. So the following in pkg/cmd/kn/main.go instead of current.

[...]

func init() {
	core.InitializeConfig()
}

func main() {
	err := core.NewDefaultKnCommand().Execute()
	if err != nil {
		fmt.Fprintln(os.Stderr, err)
		os.Exit(1)
	}
	cleanup()
}

func cleanup() {
	viper.WriteConfig()
}

Will be in #249. So intermediary config won't be saved until success. Let me know if that's OK with your thinking of future usages.

coryrc pushed a commit to coryrc/client that referenced this issue May 14, 2020
* use a separate dashboard to avoid poluting the master dashboard
* releases are not published anywhere

Fixes knative/serving#2483.
@github-actions
Copy link

This issue is stale because it has been open for 90 days with no
activity. It will automatically close after 30 more days of
inactivity. Reopen the issue with /reopen. Mark the issue as
fresh by adding the comment /remove-lifecycle stale.

@github-actions github-actions bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Oct 15, 2020
@maximilien
Copy link
Contributor Author

Closing this as I think config is all done and setup now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/question Issue type question kind/usage Issue type referring to usage of client lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale.
Projects
None yet
Development

No branches or pull requests

3 participants