Add blog post about upcoming API removals#15203
Add blog post about upcoming API removals#15203k8s-ci-robot merged 1 commit intokubernetes:masterfrom
Conversation
|
I wanted to clarify - are the "actual removal" vs "not enabled" timelines different? It would be nice to specify. |
|
/assign @zacharysarah |
|
This is also at the "minimum level of approachability" I feel comfortable publishing. I know this will be a mysterious/scary announcement for many users. I'm not sure how much more approachable I should aim to make it - @zacharysarah I would appreciate your guidance there. |
17f17d3 to
216549e
Compare
|
Deploy preview for kubernetes-io-master-staging ready! Built with commit 17f17d3 https://deploy-preview-15203--kubernetes-io-master-staging.netlify.com |
|
Deploy preview for kubernetes-io-master-staging ready! Built with commit 537e76e https://deploy-preview-15203--kubernetes-io-master-staging.netlify.com |
|
/cc @liggitt |
216549e to
39bce56
Compare
geekygirldawn
left a comment
There was a problem hiding this comment.
This is great! I only made a few tiny, nit-picky suggestions.
content/en/blog/_posts/2019-06-29-some-apis-are-being-deprecated.md
Outdated
Show resolved
Hide resolved
| re-organized or upgraded. When this happens, the old API is deprecated, and then | ||
| eventually removed. | ||
|
|
||
| There are **4** sets of resources with APIs that are deprecated, will be stop |
There was a problem hiding this comment.
maybe this would be better as
... are deprecated, and will stop being ...
content/en/blog/_posts/2019-06-29-some-apis-are-being-deprecated.md
Outdated
Show resolved
Hide resolved
| your configuration and integrations now! | ||
|
|
||
| * Change YAML files to use the newer APIs. | ||
| * Update the `apiVersion` field. |
There was a problem hiding this comment.
For Ingress, this is sufficient.
For the workload APIs, there is more involved to get objects you can create. At minimum:
- change apiVersion to
apps/v1 - populate
spec.selector.matchLabelswith the contents ofspec.template.metadata.labelsifspec.selectordoes not already exist - populate
metadata.labelswith the contents ofspec.template.metadata.labelsifmetadata.labelsdoes not already exist
kubectl convert -f ... --output-version apps/v1 will also perform the conversion for you, if everything in the manifest can be converted to apps/v1.
That said, an automated conversion will also carry over some sub-optimal defaults that were improved in apps/v1. A few examples:
- Deployments changed spec.revisionHistoryLimit to default to 10 to avoid infinite replicaset growth on repeated rollouts. Old versions defaulted to MAX_INT.
- DaemonSets changed default update strategy from OnDelete to RollingUpdate
@kubernetes/sig-apps-api-reviews is there a comprehensive doc of the differences in defaults between apps/v1 and earlier versions, and a migration guide?
|
/hold wanted to make sure #15203 (comment) was addressed before merge |
zacharysarah
left a comment
There was a problem hiding this comment.
@vllry You're off to a great start! ✨ I left some edits for clarity and mechanics.
content/en/blog/_posts/2019-06-29-some-apis-are-being-deprecated.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2019-06-29-some-apis-are-being-deprecated.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2019-06-29-some-apis-are-being-deprecated.md
Outdated
Show resolved
Hide resolved
| * DaemonSet, Deployment, StatefulSet, and ReplicaSet | ||
| * Ingress | ||
|
|
||
| None of these resources are being removed from Kubernetes. NetworkPolicies, |
There was a problem hiding this comment.
None of these resources are being removed from Kubernetes.
This is a great thing to specify. 👍
content/en/blog/_posts/2019-06-29-some-apis-are-being-deprecated.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2019-06-29-some-apis-are-being-deprecated.md
Outdated
Show resolved
Hide resolved
|
|
||
| * Change YAML files to use the newer APIs. | ||
| * Update the `apiVersion` field. | ||
| * Update custom integrations and controllers to use the newer APIs. |
There was a problem hiding this comment.
Remove periods from the ends of bullet points.
If "call" is more specifically accurate, s/use/call.
content/en/blog/_posts/2019-06-29-some-apis-are-being-deprecated.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2019-06-29-some-apis-are-being-deprecated.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2019-06-29-some-apis-are-being-deprecated.md
Outdated
Show resolved
Hide resolved
|
I've addressed all current comments, but should have another pass. It's a separate commit for ease of review, but I can squash if folks prefer. |
dd24535 to
ca88f06
Compare
content/en/blog/_posts/2019-07-09-some-apis-are-being-deprecated.md
Outdated
Show resolved
Hide resolved
|
one suggestion on the convert command, technical content lgtm otherwise |
Done, thanks! |
onlydole
left a comment
There was a problem hiding this comment.
Great topic, well written and looks good to me! 👍
|
I'm not sure how to address the failing tests - I'm guessing it's a quotation issue in the metadata. |
| @@ -0,0 +1,89 @@ | |||
| --- | |||
| layout: blog | |||
| title: 'APIs Deprecating Soon: Here\'s What You Need to Know' | |||
There was a problem hiding this comment.
Might need to be ”APIs deprecating Soon: Here’s What You Need To Know”
There was a problem hiding this comment.
nit, these were deprecated ~years ago... they're being removed in 1.16
|
@vllry 👋
Based on this error, that seems accurate: Maybe try this, inclusive of @liggitt's feedback about timeframe: |
df4f179 to
ee847a9
Compare
|
@zacharysarah I fixed the build, will squash commits with your approval. |
|
@vllry Thanks for your patience! LGTM, squash away. /lgtm |
ee847a9 to
ce7552d
Compare
37be202 to
8efdf19
Compare
|
There we are, also bumped the date again. Sorry that took so many back and forths! @liggitt can we remove the hold? |
|
Ugh the rebase dropped some edits, I'll fix it when I get to the office. |
8efdf19 to
04aa97c
Compare
04aa97c to
537e76e
Compare
|
Re-bumped the date. @kbarnard10 / @castrojo ? |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: kbarnard10 The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/hold cancel 🚀 |

Adds a blog post about the deprecated APIs that are due for removal soon in Kubernetes.