Skip to content

docs(readme): update ecosystem overview for planner integration#795

Open
cbartz wants to merge 6 commits into
mainfrom
docs/update-ecosystem-diagram-ISD-5817
Open

docs(readme): update ecosystem overview for planner integration#795
cbartz wants to merge 6 commits into
mainfrom
docs/update-ecosystem-diagram-ISD-5817

Conversation

@cbartz
Copy link
Copy Markdown
Collaborator

@cbartz cbartz commented May 20, 2026

What this PR does

Update the ecosystem diagram in the README

Why we need it

The diagram references components that are no longer in use

Checklist

  • I followed the contributing guide
  • I added or updated the documentation (if applicable)
  • I updated docs/changelog.md with user-relevant changes
  • I used AI to assist with preparing this PR
  • I added or updated tests as needed (unit and integration)
  • If this is a Grafana dashboard: I added a screenshot of the dashboard
  • If this is Terraform: terraform fmt passes and tflint reports no errors
  • If the github-runner-manager application has been changed: The application version number is updated in github-runner-manager/pyproject.toml.

Copy link
Copy Markdown
Contributor

@erinecon erinecon left a comment

Choose a reason for hiding this comment

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

Thanks! Nits only

Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
@erinecon erinecon added the documentation Improvements or additions to documentation label May 21, 2026
- Clarify Planner description: replace ambiguous "without it the charm"
  with explicit "without the Planner, the github-runner charm"
- Wrap RabbitMQ and PostgreSQL in subgraph nodes for consistency with
  other charm components in the flowchart
- Add verb to OpenStack credentials edge label: "provides OpenStack credentials"
- Fix verb tense on webhook gateway edge: "validates webhooks"
@cbartz cbartz marked this pull request as ready for review May 22, 2026 05:59
cbartz added 2 commits May 22, 2026 11:09
The org-level default GITHUB_TOKEN permissions changed from read+write
to read. The check-libraries action needs pull-requests: write to post
labels on PRs.
The org-level default GITHUB_TOKEN permissions changed from read+write
to read. The bot_pr_approval action needs pull-requests: write to
approve PRs via the GitHub API.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation Libraries: OK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants