Skip to content

Remove case studies - #57043

Open
katcosgrove wants to merge 3 commits into
kubernetes:mainfrom
katcosgrove:case-studies
Open

Remove case studies#57043
katcosgrove wants to merge 3 commits into
kubernetes:mainfrom
katcosgrove:case-studies

Conversation

@katcosgrove

@katcosgrove katcosgrove commented Aug 17, 2026 •
edited by lmktfy
Loading

Copy link
Copy Markdown
Contributor

Description

Removes the call to the case studies block and the case studies themselves, since we no longer use or update these.

cc: @kubernetes/sig-docs-leads @kubernetes/steering-committee

More context: #48432

/hold
/area web-development
/kind design

@kubernetes-prow kubernetes-prow Bot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. area/web-development Issues or PRs related to the kubernetes.io's infrastructure, design, or build processes kind/design Categorizes issue or PR as related to design. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Aug 17, 2026
@kubernetes-prow kubernetes-prow Bot added the language/en Issues or PRs related to English language label Aug 17, 2026
@netlify

netlify Bot commented Aug 17, 2026 •
edited
Loading

Copy link
Copy Markdown

✅ Pull request preview available for checking

Name Link
🔨 Latest commit f58092c
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-main-staging/deploys/6a837ac91af43f0008be5dcf
😎 Deploy Preview https://deploy-preview-57043--kubernetes-io-main-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@lmktfy lmktfy mentioned this pull request Aug 17, 2026
Open
8 tasks
@lmktfy

lmktfy commented Aug 17, 2026

Copy link
Copy Markdown
Member

This PR does more than remove the case studies from https://k8s.io/

I am happy to see them go, but I should point out that https://kubernetes.io/community/ and https://kubernetes.io/case-studies/ would also be affected.

@katcosgrove

katcosgrove commented Aug 17, 2026

Copy link
Copy Markdown
Contributor Author

Thanks, I'll go address those.

@lmktfy

lmktfy commented Aug 17, 2026

Copy link
Copy Markdown
Member

See https://deploy-preview-57043--kubernetes-io-main-staging.netlify.app/community/ for a page that - at the time of writing - has a broken link. We should (or maybe just could) update that link to go to https://www.cncf.io/case-studies?_sft_lf-project=kubernetes

However, soft LGTM. Making this change improves our site.

@lmktfy

lmktfy commented Aug 17, 2026

Copy link
Copy Markdown
Member

/retitle Remove case studies
(seems clearer)

/lgtm

I'll let others handle approval.

@kubernetes-prow kubernetes-prow Bot changed the title Remove case studies from homepage Remove case studies Aug 17, 2026
@kubernetes-prow kubernetes-prow Bot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 17, 2026
@kubernetes-prow

kubernetes-prow Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

LGTM label has been added.

DetailsGit tree hash: 52ebbf2c96e6e77d158b5444d7df864742e89e13

@lmktfy

lmktfy commented Aug 17, 2026

Copy link
Copy Markdown
Member

I edited the PR description to mention #48432 as extra context.

Arhell
Arhell reviewed Aug 18, 2026
View reviewed changes

@Arhell Arhell left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@natalisucks

natalisucks commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

I would love this to be mentioned in today's SIG Docs community meeting, since we'll also want to give our localizations a heads up that this change has taken place (not all of them have the case studies translated, but some do feature them on site in English anyhow), but to help, I'll add the approval needed so that it can be unheld once the community has met – thanks @katcosgrove 🚀
/approve

@kubernetes-prow

kubernetes-prow Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: natalisucks

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubernetes-prow kubernetes-prow Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 18, 2026
@TineoC

TineoC commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

https://deploy-preview-57043--kubernetes-io-main-staging.netlify.app/docs/contribute/new-content/case-studies/ can this page be removed?

@TineoC

TineoC commented Aug 18, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Also check if these could be removed

static/css/new-case-studies.css
static/css/case-studies-gradient.css

TineoC
TineoC previously requested changes Aug 18, 2026
View reviewed changes
Comment thread content/en/community/_index.html
@lmktfy

lmktfy commented Aug 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

@katcosgrove when is this change OK to unhold?

Maybe defer unholding 'til after the 1.37 minor release, but tip off localization subproject(s) about the time line. How does that sound?

@Caesarsage Caesarsage mentioned this pull request Aug 23, 2026
Closed
@lmktfy

lmktfy commented Aug 24, 2026

Copy link
Copy Markdown
Member

Relevant to:

This was referenced Sep 2, 2026
Closed
Open
@Kyle-Law

Kyle-Law commented Sep 2, 2026 •
edited
Loading

Copy link
Copy Markdown

I opened a PR against this fork to cover TineoC comments - katcosgrove#2

@Kyle-Law Kyle-Law mentioned this pull request Sep 2, 2026
Closed
@Kyle-Law

This comment was marked as off-topic.

Sign in to view
Kyle-Law added a commit to Kyle-Law/website that referenced this pull request Sep 2, 2026
@Kyle-Law
Clean up remaining case studies cruft flagged by review
23c8369 
Addresses the two review comments from @TineoC on kubernetes#57043 that were
left unresolved:

- Remove content/en/docs/contribute/new-content/case-studies.md
  (kubernetes#57043 (comment)),
  which documented how to submit a case study for a section that no
  longer exists, and drop the now-dangling links to it from
  content/en/docs/contribute/docs.md and
  content/en/docs/contribute/new-content/_index.md.
- Remove static/css/new-case-studies.css and
  static/css/case-studies-gradient.css
  (kubernetes#57043 (comment)),
  which were already unused anywhere in the site before this change.
Kyle-Law added a commit to Kyle-Law/website that referenced this pull request Sep 2, 2026
@Kyle-Law
Remove orphaned contribute doc flagged by review
01d352c 
Addresses one of two review comments from @TineoC on kubernetes#57043:
kubernetes#57043 (comment)

Removes content/en/docs/contribute/new-content/case-studies.md, which
documented how to submit a case study for a section that no longer
exists, and drops the now-dangling links to it from
content/en/docs/contribute/docs.md and
content/en/docs/contribute/new-content/_index.md.

Does not touch static/css/new-case-studies.css or
static/css/case-studies-gradient.css, which TineoC also flagged
(kubernetes#57043 (comment)):
per katcosgrove's own review reply
(kubernetes#57043 (comment)),
"We can't remove the CSS. The case studies still exist in other
localizations." Leaving those alone per that call.
@Kyle-Law

Kyle-Law commented Sep 2, 2026 •
edited
Loading

Copy link
Copy Markdown

Update: closed katcosgrove#3 (the localized-content removal). It ran ahead of the actual process, katcosgrove's reply above says the case studies still exist in other localizations, and @lmktfy's note about tipping off localization subprojects on timing (#57043 (comment)) hasn't happened yet even for the English-only removal. That's a call for those subprojects to make, not something to pre-empt in a follow-up PR.

katcosgrove#2 (removing the orphaned docs/contribute/new-content/case-studies.md page TineoC flagged) is still open and unaffected by this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@divya-mohan0209 divya-mohan0209 Awaiting requested review from divya-mohan0209
@salaxander salaxander Awaiting requested review from salaxander
2 more reviewers
@Arhell Arhell Arhell left review comments
@TineoC TineoC TineoC left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

@lmktfy lmktfy

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. area/web-development Issues or PRs related to the kubernetes.io's infrastructure, design, or build processes cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. kind/design Categorizes issue or PR as related to design. language/en Issues or PRs related to English language lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@katcosgrove @lmktfy @natalisucks @TineoC @Kyle-Law @Arhell