Allow create to handle multiple resources, remove createall #3196
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Only the last commit two commits are new, the others are represented in #3004 |
smarterclayton
force-pushed
the
allow_create_to_span_resources
branch
3 times, most recently
from
d8243ff to
63d65ee
Compare
|
Nice. Waiting on #3004 to merge. Have you started doing this to update, also? |
|
Not yet, that was the last. Update had some weirdness with defaulting values I wanted to read over your proposals on various management style things. |
|
Update I also need to look at PUT behavior on the client (offering an alternative to our current behavior, which is total overwrite) |
smarterclayton
changed the title
|
Fair enough. Yes, I made a number of comments about different flavors of updates on #3233, which should be in line with my previous comments on that topic. |
smarterclayton
force-pushed
the
allow_create_to_span_resources
branch
4 times, most recently
from
1e7f3b5 to
8a55a83
Compare
Like Delete, which can now run over multiple types:
kubectl delete pods,services -l name=foo
Get should be able to span items for label selection
kubectl get pods,services -l name=foo
smarterclayton
force-pushed
the
allow_create_to_span_resources
branch
from
8a55a83 to
bc86b31
Compare
|
I merged #3004. Please rebase, and I'll merge this one. |
|
Doesn't seem to need rebase |
|
Since the #3004 was a straight merge of two tips without a rebase |
bgrant0607
added a commit
that referenced
this pull request
Jan 13, 2015
…ources Allow create to handle multiple resources, remove createall
This was referenced Dec 4, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Create now can leverage resource.Builder to take as input multiple objects, and will report any failures that occur. The '-f' command is extended to support multiple inputs:
And will enumerate in the order provided. Directories are non-recursive and will process any JSON or YAML files found.
In addition, "List" is supported as a type, so given:
you can export a subset of your app to another namespace.