Skip to content

docs: update add-model docs#1808

Merged
alesstimec merged 6 commits intocanonical:v3from
kian99:update-model-create-doc
Jan 19, 2026
Merged

docs: update add-model docs#1808
alesstimec merged 6 commits intocanonical:v3from
kian99:update-model-create-doc

Conversation

@kian99
Copy link
Contributor

@kian99 kian99 commented Jan 15, 2026

Description

This PR recreates canonical/jaas-documentation#124 from the jaas-documentation repo. It updates our "how-to manage models" document to explain how to use the new --target-controller flag and in general how model creation works.

Fixes JUJU-8867

Engineering checklist

  • Documentation updated
  • Covered by unit tests
  • Covered by integration tests

@kian99 kian99 requested a review from a team as a code owner January 15, 2026 12:56
@kian99 kian99 requested a review from tmihoc January 15, 2026 12:57
@kian99
Copy link
Contributor Author

kian99 commented Jan 15, 2026

@tmihoc I see we are not getting the read-the-docs rendered link on these PRs.

@tmihoc
Copy link
Member

tmihoc commented Jan 15, 2026

@tmihoc I see we are not getting the read-the-docs rendered link on these PRs.

I noticed the same. It works if I trigger the build manually from the RTD interface, but the question is why does the build not start automatically. I'll investigate.

Copy link
Member

@tmihoc tmihoc left a comment

Choose a reason for hiding this comment

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

Thank you very much for your careful and comprehensive work on this, and for porting this from the previous repo to the new one!

I've tried to build it locally and it failed until I updated docs/requirements to have myst-parser~=4.0 -- apparently there has been a new major release there and the starter pack isn't ready for that, so pinning to the previous version helps. (The current directive from our starter pack maintainers is to do this for all of our repos.) Please add that fix.

I've left a couple of other small comments -- please see those as well.

@alesstimec alesstimec merged commit 28e8021 into canonical:v3 Jan 19, 2026
7 of 8 checks passed
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