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
Kube - support List documents #19762
Kube - support List documents #19762
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I lack understanding on how we want this to behave on the command level. Running the example YAML for the unit test gives me Error: pod does not have a name
. Can we add an e2e test?
The unit test does not check the content of the items just that the items were separated into different documents. The content mimics the other cases in the same unit test (which does not include a name either).
Will do |
4e52337
to
c7d8e96
Compare
Done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: giuseppe, vrothberg, ygalblum The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Flatten List into documents Add List type to meta/v1 Add unittest Add e2e test Signed-off-by: Ygal Blum <ygal.blum@gmail.com>
c7d8e96
to
7ef97fa
Compare
The test failures look like network flakes. Will rerun. |
/hold cancel |
Flatten List into documents
Add List type to meta/v1
Add unittest
Does this PR introduce a user-facing change?
Yes
Resolves: #19052