Skip to content

Staged configuration addition and removal audit for Vrf_lite - #494

Draft
sivakasi-cisco wants to merge 3 commits into
nd_vrf_litefrom
issue_468
Draft

Staged configuration addition and removal audit for Vrf_lite#494
sivakasi-cisco wants to merge 3 commits into
nd_vrf_litefrom
issue_468

Conversation

@sivakasi-cisco

Copy link
Copy Markdown
Collaborator

VRF Lite attachments staged with deploy=false were skipped as "no change"
on a subsequent run that requested deploy=true with the same configuration,
leaving controller-pending intent undeployed. Discover that pending intent
during query and redeploy it instead of treating it as idempotent.

  • query: derive attach_state and flag PENDING/OUT-OF-SYNC/FAILED attachments
    as pending; publish _pending_deploy_targets (write/delete op per attachment)
    and _not_in_sync_vrfs for downstream consumers.
  • deploy: scope pending targets to the requested state (merged -> configured
    pairs, replaced -> configured VRFs, overridden -> all, deleted -> delete ops),
    honor disabled_vrfs/disabled_pairs, and gate deployment on scoped pending
    targets in _needs_deployment; swallow the non-fatal HTTP 500 vPC config-save
    sanity error.
  • config_transform: suppress the false "No matching VRF Lite attachment found
    to delete" warning for staged pending-deletes.
  • orchestrators/manage_vrf_lite: seed pending-target params, fold pending
    targets into changed_vrfs, and gate on _needs_deployment.
  • tests: add unit coverage for pending discovery, state scoping, and gating;
    update merge integration tasks to re-deploy staged intent until idempotent.

Fixes #468

@sivakasi-cisco
sivakasi-cisco changed the base branch from develop to nd_vrf_lite August 11, 2026 09:24
@sivakasi-cisco sivakasi-cisco self-assigned this Aug 12, 2026
@sivakasi-cisco sivakasi-cisco added the ready for review Submitter is requesting a PR review label Aug 12, 2026
@sivakasi-cisco
sivakasi-cisco marked this pull request as ready for review August 13, 2026 05:34
@sivakasi-cisco
sivakasi-cisco marked this pull request as draft August 13, 2026 05:42
@allenrobel allenrobel removed the ready for review Submitter is requesting a PR review label Aug 13, 2026
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.

[Staged configuration audit] VRF-Lite module

2 participants