Skip to content

Add CCM quickstart#1027

Open
LogicalShark wants to merge 1 commit intokubernetes:masterfrom
LogicalShark:docs
Open

Add CCM quickstart#1027
LogicalShark wants to merge 1 commit intokubernetes:masterfrom
LogicalShark:docs

Conversation

@LogicalShark
Copy link
Copy Markdown
Contributor

@LogicalShark LogicalShark commented Mar 31, 2026 •
edited
Loading

Fixes #1019

@k8s-ci-robot k8s-ci-robot added the needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. label Mar 31, 2026
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

k8s-ci-robot commented Mar 31, 2026

This issue is currently awaiting triage.

If the repository mantainers determine this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

k8s-ci-robot commented Mar 31, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: LogicalShark

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

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 31, 2026
@k8s-ci-robot k8s-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Mar 31, 2026
@LogicalShark
Copy link
Copy Markdown
Contributor Author

LogicalShark commented Mar 31, 2026

/assign hdp617

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Mar 31, 2026
@LogicalShark LogicalShark force-pushed the docs branch 2 times, most recently from eb87994 to 11f7455 Compare March 31, 2026 21:01
@hdp617
Copy link
Copy Markdown
Contributor

hdp617 commented Mar 31, 2026

  • Could you link this PR to issue Add quickstart doc for running CCM #1019?

  • I'll move the kops-down cleanup changes to a new, dedicated issue to streamline the review process for the CCM quickstart and easier cherrypick if needed.

@LogicalShark LogicalShark changed the title Add CCM quickstart and expand kops-down cleanup Add CCM quickstart Apr 1, 2026
@hdp617
Copy link
Copy Markdown
Contributor

hdp617 commented Apr 2, 2026

For the quickstart, I was approaching it from a customer's perspective: 'What's the fastest way to get a self-managed k8s cluster?' The current approach is great for local dev changes to the component, but I'm more interested in targeting the customer experience here. How about separating this into 2?

  • Quickstart - essentially just run make kops-up.
  • CCM local testing - which is what we have here.

@LogicalShark LogicalShark force-pushed the docs branch 4 times, most recently from 03427c3 to 1cc9372 Compare April 3, 2026 19:08
@k8s-ci-robot k8s-ci-robot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Apr 3, 2026
@LogicalShark LogicalShark force-pushed the docs branch 3 times, most recently from 2ed2381 to 4102b67 Compare April 3, 2026 20:09
@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Apr 3, 2026
@LogicalShark LogicalShark force-pushed the docs branch 3 times, most recently from b9ed23d to 644854b Compare April 3, 2026 21:02
zhang-xuebin
zhang-xuebin reviewed Apr 9, 2026
View reviewed changes
Copy link
Copy Markdown

@zhang-xuebin zhang-xuebin left a comment

Choose a reason for hiding this comment

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

Nice work, thanks @LogicalShark !

Comment thread docs/ccm-manual.md Outdated
Comment thread docs/ccm-manual.md Outdated
Comment thread docs/ccm-manual.md Outdated
Comment thread docs/ccm-manual.md Outdated
Comment thread docs/manual-cluster-gce.md Outdated
Comment thread docs/manual-cluster-gce.md Outdated
Comment thread docs/manual-cluster-gce.md Outdated
@LogicalShark LogicalShark force-pushed the docs branch 2 times, most recently from 1c9e926 to 67dc025 Compare April 16, 2026 17:36
@aojea
Copy link
Copy Markdown
Member

aojea commented Apr 17, 2026

fantastic 🥇

hdp617
hdp617 reviewed Apr 20, 2026
View reviewed changes
Comment thread docs/manual-cluster-gce.md Outdated
--tags=k8s-worker
```

## Step 2: Access and Configure Nodes (Master and Worker)
Copy link
Copy Markdown
Contributor

@hdp617 hdp617 Apr 20, 2026

Choose a reason for hiding this comment

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

IIUC, steps 2-6 are essentially https://github.com/kelseyhightower/kubernetes-the-hard-way. How about adding a reference to that doc instead?

hdp617
hdp617 reviewed Apr 20, 2026
View reviewed changes
Comment thread docs/kops-quickstart.md
@LogicalShark LogicalShark force-pushed the docs branch 2 times, most recently from cb97a06 to 69ebc43 Compare April 20, 2026 22:04
@k8s-ci-robot k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Apr 20, 2026
@hdp617
Copy link
Copy Markdown
Contributor

hdp617 commented Apr 20, 2026

Hold in case you want to make minor changes. Feel free to cancel it once you're ready to merge.

/lgtm
/hold

@k8s-ci-robot k8s-ci-robot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. lgtm "Looks good to me", indicates that a PR is ready to be merged. labels Apr 20, 2026
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

k8s-ci-robot commented Apr 20, 2026

@LogicalShark: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
cloud-provider-gcp-e2e-full 201a196 link true /test cloud-provider-gcp-e2e-full

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

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

Reviewers

@hdp617 hdp617 hdp617 left review comments

@JoelSpeed JoelSpeed Awaiting requested review from JoelSpeed

@seans3 seans3 Awaiting requested review from seans3

+1 more reviewer

@zhang-xuebin zhang-xuebin zhang-xuebin left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@hdp617 hdp617

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. 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. lgtm "Looks good to me", indicates that a PR is ready to be merged. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add quickstart doc for running CCM

5 participants

@LogicalShark @k8s-ci-robot @hdp617 @aojea @zhang-xuebin