Proposal for sync operator - #2231
Open
geetikabatra wants to merge 1 commit into
Open
Conversation
cardoe
reviewed
Aug 25, 2026
cardoe
left a comment
cardoe
left a comment
Contributor
There was a problem hiding this comment.
So I think this would be good to bring into the ADR repo since it'll be rendered there appropriately but this is great. Thank you for working on it.
| Nautobot merely records. The stricter reading — every VLAN declared in git | ||
| before a tenant can use it — would mean giving up tenant self-service network | ||
| creation, since a tenant API call cannot wait on a merge. This needs agreement | ||
| before VLANs are implemented, because it decides whether the event path stays. |
Contributor
There was a problem hiding this comment.
No. Tenant allocations are going to be dynamic which is why we're just providing the VLAN range, which is provided by the VLAN group.
| 2. **What should happen to a hand edit made directly in Nautobot** to a static | ||
| type? As specified it is reverted on the next reconcile, silently. Reverting is | ||
| right, but it may be worth reporting rather than doing quietly, so the person | ||
| who made the edit finds out. |
Contributor
There was a problem hiding this comment.
Things should be reverted. Ultimately hand edits is what got us here in the first place but folks keep getting around the removal of permissions.
| relationship — and how do we backfill objects nautobotop already created? | ||
| 2. **Can the global cluster reach site Neutron/Ironic APIs?** Sites reaching | ||
| global is confirmed; this direction is not, and it decides whether we need a | ||
| per-site agent. |
Contributor
There was a problem hiding this comment.
It can but we'll have to define that mechanism for it to do so.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this change do?
Upgrade impact
upgrade-impactlabel and a release note: runscriv createfrom therepository root and describe the required action in the generated
changelog.d/file. See RELEASING.md.Operator action means anything a deployment has to do beyond a normal resync:
deploy repo or values changes, new or removed secrets, enabling or disabling a
component, or a manual one-time step.