Repository navigation
docs: update onboarding docs - #27
Conversation
There was a problem hiding this comment.
license-eye has checked 116 files.
| Valid | Invalid | Ignored | Fixed |
|---|---|---|---|
| 32 | 1 | 83 | 0 |
Click to see the invalid file list
- lychee.toml
Use this command to fix any missing license headers
```bash
docker run -it --rm -v $(pwd):/github/workspace apache/skywalking-eyes header fix
</details>
There was a problem hiding this comment.
license-eye has checked 116 files.
| Valid | Invalid | Ignored | Fixed |
|---|---|---|---|
| 32 | 1 | 83 | 0 |
Click to see the invalid file list
- lychee.toml
Use this command to fix any missing license headers
```bash
docker run -it --rm -v $(pwd):/github/workspace apache/skywalking-eyes header fix
</details>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
There was a problem hiding this comment.
🟡 Changes recommended
CONTRIBUTING.md contains incorrect/non-existent build instructions (and an incorrect clone URL), and a couple of DNS “fallback” claims in docs don’t reliably match the current charm implementation.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
This PR refreshes the Cloudflared Operators documentation to replace charm-template placeholders with source-aligned onboarding material, including tutorials, how-to guides, and reference/explanation pages, plus documentation-lint configuration.
Changes:
- Rewrites tutorial, how-to, reference, and explanation content to document the
cloudflared(subordinate machine) andcloudflare-configuratorcharms and their relations/configuration. - Updates docs navigation (indexes/toctrees) and adds new how-to pages (expose frontend, use configurator).
- Adds documentation-check configuration (Lychee config + local Vale vocabulary).
File summaries
| File | Description |
|---|---|
| README.md | Adjusts repository README heading. |
| lychee.toml | Adds Lychee exclusions for internal doc URLs. |
| docs/index.rst | Replaces template landing page text and updates navigation/table links. |
| docs/tutorial/index.rst | Updates tutorial index copy and toctree entries. |
| docs/tutorial/basic-deployment.rst | Replaces placeholder tutorial with machine-model deployment/integration steps. |
| docs/tutorial/advanced-deployment.rst | Adds advanced operational guidance (DNS, multiple tunnels, snap channels, COS). |
| docs/how-to/index.rst | Adds/updates how-to index entries. |
| docs/how-to/configure-dns.rst | Documents DNS resolution behavior and nameserver option usage. |
| docs/how-to/expose-frontend.rst | New guide for exposing a frontend via ingress. |
| docs/how-to/use-configurator.rst | New guide for configurator usage and secret handling. |
| docs/how-to/integrate-with-cos.rst | Documents COS integration via cos-agent. |
| docs/how-to/redeploy.rst | Documents redeploy workflow in a new model. |
| docs/how-to/upgrade.rst | Documents refresh/upgrade workflow for both charms. |
| docs/how-to/troubleshoot.rst | Adds troubleshooting checks for relations/config/secrets. |
| docs/how-to/terraform.rst | States Terraform is not currently supported and gives alternative path. |
| docs/how-to/back-up-restore.rst | Describes what to preserve (topology/config/secrets) given no DB. |
| docs/how-to/contribute.rst | Updates contribution links to repository sources and issue tracker. |
| docs/reference/index.rst | Replaces template reference index copy with Cloudflared-specific structure. |
| docs/reference/actions.rst | Links to Charmhub actions page for configurator. |
| docs/reference/configurations.rst | Documents charm configuration keys and intent. |
| docs/reference/relation-endpoints.rst | Documents relation endpoints, directions, and example integrate commands. |
| docs/reference/metrics.rst | Documents COS metrics exposure and port allocation scheme. |
| docs/reference/juju-events.rst | Documents observed Juju events for both charms. |
| docs/reference/high-level-deployment.rst | Documents deployment model and relation topology (subordinate + configurator). |
| docs/reference/charm-architecture.rst | Replaces template architecture text with snap/subordinate architecture overview. |
| docs/reference/cryptographic-overview.rst | Documents security boundary (Juju secrets + Cloudflare tunnel). |
| docs/reference/terraform.rst | Notes absence of Terraform modules in repo and links to Juju deployment path. |
| docs/release-notes/index.rst | Replaces template release-notes landing page with project-specific policy/links. |
| docs/explanation/index.rst | Updates explanation index to Cloudflared-specific context. |
| docs/explanation/charm-design.rst | Explains the split between workload execution and configuration provider. |
| docs/explanation/security.rst | Documents secret flow and points to upstream Cloudflare guidance. |
| docs/changelog.rst | Adds/links a changelog page in docs navigation. |
| CONTRIBUTING.md | Updates contribution guide links and commands. |
| .vale/styles/config/vocabularies/local/accept.txt | Adds project terms to Vale local vocabulary. |
Review details
- Files reviewed: 34/34 changed files
- Comments generated: 5
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
erinecon
left a comment
There was a problem hiding this comment.
Thanks so much for all your work on expanding this documentation set 🚀 I've gone through the explanation material (no comments) and how-to guides today. Thanks for your patience on the remaining pages :)
Co-authored-by: Erin Conley <erin.conley@canonical.com>
…nical/cloudflared-operators into docs/update-onboarding-docs
erinecon
left a comment
There was a problem hiding this comment.
Made it through the rest of the docs today -- thank you so much for all your work here!!
Co-authored-by: Erin Conley <erin.conley@canonical.com>
…nical/cloudflared-operators into docs/update-onboarding-docs
erinecon
left a comment
There was a problem hiding this comment.
Some remaining nits, but nothing that's blocking the merge. Thank you so much for all your work and effort on the documentation 🎉
Test results for commit a07823eTest coverage for a07823e Static code analysis report |
Test results for commit a07823eTest coverage for a07823e Static code analysis report |
What this PR does
Why we need it
The documentation still contained generic charm-template text and incomplete references. It also did not clearly explain that
cloudflaredis a subordinate machine charm, how it relates to a principal application, or how the configurator passes tunnel settings. This PR gives operators source-backed instructions for deploying, configuring, integrating, troubleshooting, and upgrading the charms.Test plan
make -f Makefile.docs docs-checkgit diff --check: passed.docs/.Review focus
cloudflared-route,juju-info,ingress, andcos-agentrelation directions match the charm metadata.Potential breaking changes
None. This PR changes documentation and documentation lint configuration only. It does not change charm runtime behavior, interfaces, configuration keys, or workflows.
Dependencies, APIs, and workflow changes
No runtime dependencies or APIs are added.
lychee.tomlconfigures the existing documentation link checker, and the local Vale vocabulary adds project-specific terms.Checklist