Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: add docs for RTD #631

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

docs: add docs for RTD #631

wants to merge 10 commits into from

Conversation

tmihoc
Copy link
Member

@tmihoc tmihoc commented Nov 15, 2024

Description

This PR creates a docs/docs-rtd directory with a Sphinx project and docs content from Discourse for a newly created RTD project.

The content for now mainly consists of a homepage, tutorial, and how-to guides.

The links to juju (i.e., juju/juju repo) concepts are for now pointing to juju,is. That should work, because any pages moved out of there will have redirects, so the links should be fine. The links to terraform-provider-juju reference are for now to the Terraform Registry home of the provider.

TODO: In a follow-up PR it would be good to use the docs/docs-rtd/reference directory to hold all tool reference materials. These are currently under docs/data-sources and docs/resources. When we move them to reference we should consider how that will affect the content on https://registry.terraform.io/providers/juju/juju/latest/docs . In principle, the Terraform Registry page should contain nothing more than autogenerated reference plus a link to the new terraform-provider-juju RTD project, which will contain the complete set of terraform-provider-juju docs, user docs as well as developer docs.

Fixes:

Type of change

  • Other (please describe)

Documentation.

Environment

  • Juju controller version:

  • Terraform version:

QA steps

make run

Additional notes

JIRA card: JUJU-7097

Copy link
Member

@alesstimec alesstimec left a comment

Choose a reason for hiding this comment

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

LGTM in general, but it would be easier to review if the first PR just pulled in the spinx starter pack and the second PR added the actual docs. Also there seem to be a bunch of linting warnings.

terraform {
required_providers {
juju = {
version = "~> 0.10.0"
Copy link
Member

Choose a reason for hiding this comment

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

should we mention at least 0.15.0?

@tmihoc
Copy link
Member Author

tmihoc commented Nov 18, 2024

/merge

add docs from Discourse

remove juju and python-libjuju content

fix links and formatting errors (notes, detail blocks)

customise homepage for terraform-provider-juju
@hmlanigan
Copy link
Member

@tmihoc some of the new GitHub Actions are failing due to:

make: *** No rule to make target 'linkcheck'. Stop. from https://github.com/juju/terraform-provider-juju/actions/runs/11911988831/job/33194655759?pr=631

the markdown-lint action doesn't like the current changelog.md file.
https://github.com/juju/terraform-provider-juju/actions/runs/11911988797/job/33194652445?pr=631

+ make -f docs-rtd/Makefile.sp sp-install 'PIPOPTS=--no-binary :all:'
make: docs-rtd/Makefile.sp: No such file or directory
make: *** No rule to make target 'docs-rtd/Makefile.sp'.  Stop.

from https://github.com/juju/terraform-provider-juju/actions/runs/11911988799/job/33194654028?pr=631

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants