-
Notifications
You must be signed in to change notification settings - Fork 263
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 suggestions for offline mode (kn service --target) #1195
Comments
For reference: Here's the previous discussion that resulted in the |
In the PR discussion I stated the following list |
This issue is stale because it has been open for 90 days with no |
/remove-lifecycle stale |
I think we should use this issue as well for nailing down the final option name for the offline mode. |
@rhuss I've been looking at this I was wondering if we can improve the target mechanism to just print all the content of all the generated files using a separator ( What do you think? |
Per today' WG call with @lance and @maximilien we agreed that should be feasible solution to redirect the output. Add a special case e.g. Update: thinking twice something like nicer constructor for |
@dsimansk yes.. 100% for a nicer constructor for STDOUT. I don't want to open an old discussion, but maybe we can have an extra flag |
@salaboy, I'm not convinced about For the helm case that you mention, I consider this a bit of a misusage of For the use case, we want to support here, I think |
@rhuss yes I tend to agree, and using The primary use case that I had in mind is to be able to do something like: How can we make this happen? Is there someone working on this already? |
I'm open for renaming it to Nobody is currently working on this, any help is appreciated. |
Feature request
Some minor tuning for the
kn service create --target
offline mode:--target
naming. Not everyone liked it and has a different association with it. Maybe we should revisit it. Let's discuss again on this ticket whether we want to change and maybe find a better alternative.The text was updated successfully, but these errors were encountered: