Skip to content

docs(infrahub): restore lineage source and owner documentation#110

Merged
minitriga merged 2 commits intomainfrom
fix/restore-lineage-docs
Feb 3, 2026
Merged

docs(infrahub): restore lineage source and owner documentation#110
minitriga merged 2 commits intomainfrom
fix/restore-lineage-docs

Conversation

@minitriga
Copy link
Copy Markdown
Contributor

@minitriga minitriga commented Feb 3, 2026

Summary by CodeRabbit

  • New Features

    • Added source and owner configuration settings for Infrahub to support explicit lineage source and owner selection.
  • Documentation

    • Expanded Infrahub documentation: renamed section to "Lineage tracking", clarified default behavior, explained account-group usage, added configuration examples and a usage matrix detailing source/owner combinations.

Test plan

  • Verify documentation renders correctly in Docusaurus
  • Confirm settings table is properly formatted

🤖 Generated with Claude Code

Re-adds documentation for the configurable `source` and `owner` settings
that was inadvertently removed in PR #109. This includes:
- YAML example with source/owner settings
- Settings reference table entries
- Full lineage tracking section with account group configuration

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 3, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

Documentation for the Infrahub adapter was updated to add lineage tracking. Two new destination settings keys, source and owner, were introduced to specify CoreAccountGroup names for lineage source and owner. The section title was changed from "Source account tracking" to "Lineage tracking" and expanded with default behavior, override mechanisms, a usage matrix, and updated configuration examples showing source and owner under destination.settings. The change set is 38 lines added and 2 lines removed.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title accurately describes the main change: restoring documentation for lineage source and owner settings in the Infrahub adapter documentation, which is confirmed by the raw summary and PR objectives.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Replace netbox-specific references with generic names in the
lineage tracking documentation examples.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying infrahub-sync with  Cloudflare Pages  Cloudflare Pages

Latest commit: a5af217
Status:⚡️  Build in progress...

View logs

@minitriga minitriga merged commit dfce765 into main Feb 3, 2026
9 of 10 checks passed
@minitriga minitriga deleted the fix/restore-lineage-docs branch February 3, 2026 02:25
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@docs/docs/adapters/infrahub.mdx`:
- Around line 97-98: Update the table entries for the `source` and `owner`
settings to reflect the effective default described later in the lineage
section: replace the "-" placeholder with a concise description like "defaults
to CoreAccountGroup matching the source name" for `source`, and for `owner`
state its default derivation (e.g., "defaults to CoreAccountGroup derived from
source when omitted"); ensure the wording matches the phrasing in the "lineage"
section so readers see the same default behavior.

Comment thread docs/docs/adapters/infrahub.mdx
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.

1 participant