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
cAdvisor-less, CRI-full Container and Pod Stats #2371
Comments
/sig node |
/milestone v1.21 |
Hey @haircommander and @bobbypage, enhancements 1.21 shadow here, Enhancements Freeze is 2 days away, Feb 9th EOD PST The enhancements team is aware that KEP update is currently in progress (PR #2364). Please make sure to work on PRR questionnaires and requirements and get it merged before the freeze. For PRR related questions or to boost the PR for PRR review, please reach out in Slack on the #prod-readiness channel. Any enhancements that do not complete the following requirements by the freeze will require an exception.
|
Hi @haircommander and @bobbypage, 1.21 Enhancements Lead here. Enhancements Freeze is now in effect. Unfortunately, your KEP needed to be updated and the PR has not yet merged. If you wish to be included in the 1.21 Release, please submit an Exception Request as soon as possible. /milestone clear |
/milestone v1.22 |
Hey @haircommander and @bobbypage - 1.22 enhancements team here! Just a reminder that enhancements freeze is coming up on 5/13. We'll need the KEP merged before then to get this included in the 1.22 release. Let us know if there's anything we can do to help before then! |
@haircommander @bobbypage #2364 merged so we've got you tracked for 1.22 :) |
Hello @haircommander @bobbypage 👋, 1.22 Docs release lead here. Please follow the steps detailed in the documentation to open a PR against dev-1.22 branch in the k/website repo. This PR can be just a placeholder at this time and must be created before Fri July 9, 11:59 PM PDT.
Also, take a look at Documenting for a release to familiarize yourself with the docs requirement for the release. |
thanks for the heads up @PI-Victor ! |
xref initial kubelet implementation kubernetes/kubernetes#103095 |
Hey @bobbypage and @haircommander - Just checking in as we're about 2 weeks away from 1.22 code freeze. I've got kubernetes/kubernetes#103095 and kubernetes/kubernetes#103095 tracked as the open k/k PRs. Are there any other open or merged PRs we should be tracking? Thanks! |
none opened yet! I will post them here if we do. Thanks for your work @salaxander |
Hey @haircommander - One more check-in as we're a week out from 1.22 code freeze. Any updates on if you expect kubernetes/kubernetes#103095 and kubernetes/kubernetes#103095 to merge before the deadline? Thanks! |
Thanks for checking! I do expect them to merge. We're waiting on a e2e POC which I'm working on :) |
Hi @haircommander - One last ping (sorry!). Code freeze is tomorrow evening (PST), so those two open PRs will need to merge before then for this to be included in 1.22. Let me know if there's anything we can do to help :) |
/milestone v1.29 @haircommander can you please send a PR to bump kep.yaml metadata? |
Hi @haircommander 👋, 1.29 Enhancements team here! Just checking in as we approach enhancements freeze on 01:00 UTC, Friday, 6th October, 2023. This enhancement is targeting for stage Here's where this enhancement currently stands:
It looks like https://github.com/kubernetes/enhancements/pull/4249/files will address most of these issues. The status of this enhancement is marked as |
/stage alpha |
Hi @haircommander, just checking in once more as we approach the 1.29 enhancement freeze deadline this week on 01:00 UTC, Friday, 6th October, 2023. The status of this enhancement is marked as It looks like when #4249 is merged in it will address most of the requirements. Let me know if I missed anything. Thanks! |
With #4249 merged this enhancement is |
Hello @bobbypage @haircommander 👋, v1.29 Docs Shadow here. |
I don't believe so. thanks! |
Hi @bobbypage @haircommander, 👋 from the v1.29 Release Team-Communications! We would like to check if you have any plans to publish a blog for this KEP regarding new features, removals, and deprecations for this release. If so, you need to open a PR placeholder in the website repository. |
Hello @haircommander @bobbypage 👋, v1.29 Enhancements team here. Just checking in as we approach code freeze at 01:00 UTC Wednesday 1st November 2023 Here's where this enhancement currently stands:
Also, please let me know if there are other PRs in k/k we should be tracking for this KEP. With all this, the status of this KEP is As always, we are here to help if any questions come up. Thanks! |
Hello @haircommander @bobbypage 👋, 1.29 Enhancements team here! With all the implementation(code related) PRs merged as per the issue description, this enhancement is now marked as The test freeze is 01:00 UTC Wednesday 15th November 2023 / 18:00 PDT Tuesday 14th November 2023. Please make sure all test PRs are merged in by then. The tracked test PRs for this KEP are: Please let me know if there are additional test PRs we should track. Thanks! |
/remove-label lead-opted-in |
Hello 👋 1.30 Enhancements Lead here, I'm closing milestone 1.29 now, /milestone clear |
For containerd:
|
yeah containerd needs to be extended to pass the required container and pod metrics through to the kubelet. That said, for metrics that weren't formerly exposed by cadvisor (like the pull metrics that this comment was done on), it may be more efficient to continue to read them directly from containerd |
As a note: I think we are deferring this (again 🙃 ) from 1.30 until containerd and cri-o have support. The cri-o support is actively being worked on, and containerd is being taken by @adisky . I am hoping to make forward progress in 1.31 |
Hi @haircommander, Just to make sure I understand correct: This KEP won't help with exposing containerd metrics (like the image pulls ones as above) via kubelet, right? Hence, if we need the containerd metrics for the image pulls, we need somehow to collect/scrape these metrics from containerd directly, right? |
yes correct! |
Enhancement Description
KEP (
k/enhancements
) update PR(s):Code (
k/k
) update PR(s):Docs (
k/website
) update PR(s): Document PodAndContainerStatsFromCRI feature website#30535k/enhancements
) update PR(s): KEP 2371: target to beta for 1.28 #3770k/k
) update PR(s):k/website
) update PR(s):The text was updated successfully, but these errors were encountered: