Skip to content

Commit 6d5bcaa

Browse files
committed
Add PRR
Signed-off-by: Jian Zeng <[email protected]>
1 parent 78cda8a commit 6d5bcaa

File tree

3 files changed

+26
-19
lines changed

3 files changed

+26
-19
lines changed
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
kep-number: 3288
2+
alpha:
3+
approver: "@deads2k"

keps/sig-node/3288-separate-stdout-from-stderr/README.md

Lines changed: 19 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -131,20 +131,20 @@ checklist items _must_ be updated for the enhancement to be released.
131131

132132
Items marked with (R) are required *prior to targeting to a milestone / release*.
133133

134-
- [ ] (R) Enhancement issue in release milestone, which links to KEP dir in [kubernetes/enhancements] (not the initial KEP PR)
135-
- [ ] (R) KEP approvers have approved the KEP status as `implementable`
136-
- [ ] (R) Design details are appropriately documented
134+
- [x] (R) Enhancement issue in release milestone, which links to KEP dir in [kubernetes/enhancements] (not the initial KEP PR)
135+
- [x] (R) KEP approvers have approved the KEP status as `implementable`
136+
- [x] (R) Design details are appropriately documented
137137
- [ ] (R) Test plan is in place, giving consideration to SIG Architecture and SIG Testing input (including test refactors)
138138
- [ ] e2e Tests for all Beta API Operations (endpoints)
139-
- [ ] (R) Ensure GA e2e tests for meet requirements for [Conformance Tests](https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/conformance-tests.md)
139+
- [ ] (R) Ensure GA e2e tests for meet requirements for [Conformance Tests](https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/conformance-tests.md)
140140
- [ ] (R) Minimum Two Week Window for GA e2e tests to prove flake free
141141
- [ ] (R) Graduation criteria is in place
142-
- [ ] (R) [all GA Endpoints](https://github.com/kubernetes/community/pull/1806) must be hit by [Conformance Tests](https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/conformance-tests.md)
142+
- [ ] (R) [all GA Endpoints](https://github.com/kubernetes/community/pull/1806) must be hit by [Conformance Tests](https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/conformance-tests.md)
143143
- [ ] (R) Production readiness review completed
144144
- [ ] (R) Production readiness review approved
145-
- [ ] "Implementation History" section is up-to-date for milestone
145+
- [x] "Implementation History" section is up-to-date for milestone
146146
- [ ] User-facing documentation has been created in [kubernetes/website], for publication to [kubernetes.io]
147-
- [ ] Supporting documentation—e.g., additional design documents, links to mailing list discussions/SIG meetings, relevant PRs/issues, release notes
147+
- [x] Supporting documentation—e.g., additional design documents, links to mailing list discussions/SIG meetings, relevant PRs/issues, release notes
148148

149149
<!--
150150
**Note:** This checklist is iterative and should be reviewed and updated every time this enhancement is being considered for a milestone.
@@ -157,7 +157,7 @@ Items marked with (R) are required *prior to targeting to a milestone / release*
157157

158158
## Summary
159159

160-
Currently, kubelet actually [has the potential](https://github.com/kubernetes/kubernetes/blob/69cabe778bccfc5f4fa9b6788c9def005e7d959d/pkg/kubelet/kuberuntime/logs/logs.go#L281)
160+
Currently, kubelet actually [has the potential](https://github.com/kubernetes/kubernetes/blob/69cabe778bccfc5f4fa9b6788c9def005e7d959d/pkg/kubelet/kuberuntime/logs/logs.go#L281)
161161
to return certain log stream of a container, but this ability is not exposed to the user.
162162
This proposal enables users to view certain log stream of a container,
163163
and aims to extend kubelet and api-server with the ability to return certain
@@ -167,8 +167,8 @@ container log stream.
167167

168168
Users could fetch logs of the container, but now kubelet always returns
169169
combined stdout and stderr logs, which is not convenient for users who only want to
170-
view certain log stream.
171-
Meanwhile, [many users](https://github.com/kubernetes/kubernetes/issues/28167) have been interested in the ability to
170+
view certain log stream.
171+
Meanwhile, [many users](https://github.com/kubernetes/kubernetes/issues/28167) have been interested in the ability to
172172
retrieve certain log stream of a container, so it is great to implement this long-wanted feature.
173173

174174
### Goals
@@ -235,7 +235,7 @@ type PodLogOptions struct {
235235
...
236236
// If set to "stdout" or "stderr", return the given log stream of the container.
237237
// If set to "all" or not set, the combined stdout and stderr from the container is returned.
238-
// Available values are: "stdout", "stderr", "all", "" (empty string).
238+
// Available values are: "stdout", "stderr", "all", "" (empty string).
239239
// +optional
240240
Stream LogStreamType
241241
}
@@ -333,7 +333,7 @@ function to make it be able to filter out the unwanted stream.
333333

334334
### Changes of kubectl
335335

336-
Add a new flag `--stream`, whose value defaults to "all", to `kubectl logs`, hence users are able to specify the
336+
Add a new flag `--stream`, whose value defaults to "all", to `kubectl logs`, hence users are able to specify the
337337
log stream to return.
338338

339339
### Test Plan
@@ -669,10 +669,10 @@ Recall that end users cannot usually observe component logs or access metrics.
669669
-->
670670

671671
- [ ] Events
672-
- Event Reason:
672+
- Event Reason:
673673
- [ ] API .status
674-
- Condition name:
675-
- Other field:
674+
- Condition name:
675+
- Other field:
676676
- [ ] Other (treat as last resort)
677677
- Details:
678678

@@ -792,6 +792,9 @@ Major milestones might include:
792792
- the version of Kubernetes where the KEP graduated to general availability
793793
- when the KEP was retired or superseded
794794
-->
795+
2022-05-01: KEP opened
796+
797+
2022-06-08: KEP marked implementable
795798

796799
## Drawbacks
797800

@@ -806,7 +809,7 @@ What other approaches did you consider, and why did you rule them out? These do
806809
not need to be as detailed as the proposal, but should include enough
807810
information to express the idea and why it was not acceptable.
808811
-->
809-
Instead of filtering log stream on the server side, we could return a stream of CRI-format logs,
812+
Instead of filtering log stream on the server side, we could return a stream of CRI-format logs,
810813
something like the following:
811814
```
812815
2016-10-06T00:17:09.669794202Z stdout P log content 1

keps/sig-node/3288-separate-stdout-from-stderr/kep.yaml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,13 @@ participating-sigs:
77
- sig-node
88
- sig-apimachinery
99
- sig-cli
10-
status: provisional
10+
status: implementable
1111
creation-date: 2022-05-01
1212
reviewers:
13-
- "@sig-node-leads"
13+
- "@SergeyKanzhelev"
14+
- "@mrunalp"
1415
approvers:
15-
- "@sig-node-leads"
16+
- "@dchen1107"
1617

1718
# The target maturity stage in the current dev cycle for this KEP.
1819
stage: alpha

0 commit comments

Comments
 (0)