Skip to content

add copycomp Makefile targets for component docs - #452

Merged
k8s-ci-robot merged 1 commit into
kubernetes-sigs:masterfrom
Caesarsage:feat/copycomp-targets
May 20, 2026
Merged

k8s-ci-robot merged 1 commit into
kubernetes-sigs:masterfrom
Caesarsage:feat/copycomp-targets

Conversation

@Caesarsage

Copy link
Copy Markdown
Contributor

What

Three new make targets that copy gen-compdocs/build/ output into the corresponding hand-curated destinations under k/website:

Target Source Destination
copycomp-core kube-apiserver.md, kube-controller-manager.md, kube-scheduler.md, kube-proxy.md content/en/docs/reference/command-line-tools-reference/
copycomp-kubeadm kubeadm.md + every kubeadm_*/ subdir content/en/docs/reference/setup-tools/kubeadm/generated/
copycomp-kubectl kubectl.md + every kubectl_*/ subdir content/en/docs/reference/kubectl/generated/

Plus a copycomp umbrella that runs all three.

Why

Mirrors the per-component handoff pattern @tengqm described — each component's regenerated docs go into its specific hand-curated tree (which contains curated _index.md, README.md, and unrelated subdirs like feature-gates/), so we overlay rather than rm -rf.

Verification

  • make -n copycomp-core clean syntax
  • End-to-end run against a real k/website tree: 4 kube-* files copied to command-line-tools-reference/, kubeadm tree copied with 11 subdirs, kubectl tree copied with 45 subdirs. No errors.

@k8s-ci-robot k8s-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels May 15, 2026
@Caesarsage
Caesarsage force-pushed the feat/copycomp-targets branch from 486ba5d to a49dca5 Compare May 15, 2026 12:08
@k8s-ci-robot k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels May 15, 2026
Comment thread Makefile
@Caesarsage
Caesarsage force-pushed the feat/copycomp-targets branch from a49dca5 to 3a21b77 Compare May 16, 2026 13:24

@tengqm tengqm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label May 20, 2026
@k8s-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Caesarsage, tengqm

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 May 20, 2026
@k8s-ci-robot
k8s-ci-robot merged commit 95b65bd into kubernetes-sigs:master May 20, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants