Skip to content
This repository has been archived by the owner on Nov 7, 2024. It is now read-only.

Modify Original AVD Guide - Add L3LS CI AVD Guide #157

Merged
merged 16 commits into from
Aug 22, 2024
Merged

Modify Original AVD Guide - Add L3LS CI AVD Guide #157

merged 16 commits into from
Aug 22, 2024

Conversation

RestlessWanderer
Copy link
Collaborator

This PR continues multiple changes.

  • Created two new directories

    • l2ls
    • l3ls
  • Rename original AVD lab guide files and fix dead links

    • Rename from AVD to AVD L2LS
    • avd.md renamed to overview.md ( moved under l2ls dir )
    • avd_lab_guide.md renamed to l2ls_lab_guide.md ( moved under l2ls dir )
  • Added images for l3ls guide to assets/images

  • Added new files for l3ls lab guide

    • New section called AVD L3LS EVPN/VXLAN
    • overview.md ( under l3ls dir )
    • l3ls_lab_guide.md ( under l3ls dir )
  • Fixed all dead links as a result of guide move.

@mthiel117
Copy link
Collaborator

Need to add this section to the L2LS Lab Guide.

Change To Lab Working Directory

Now that AVD is updated, lets move into the appropriate directory so we can access the files necessary for this L3LS Lab!

cd labs/L2LS

@mthiel117
Copy link
Collaborator

This section on L3LS_EVPN Lab guide is not rendering correct. Tabs need removed.

???+ abstract "Where did those VLANs come from?" You should notice some VLANs that we didn't define anywhere in the _NETWORK_SERVICES.yml data model which aren't related to MLAG. Specifically, these will be VLAN SVIs Vlan1199 and Vlan3009.

@mthiel117
Copy link
Collaborator

newline need after line 145, before the next heading

@mthiel117
Copy link
Collaborator

L3LS_EVPN, under section "Build and Deploy Changes for EVPN Gateway Functionality". The code blocks for s1-bdr1 and s1-bdr2 are not rendering correctly.

@mthiel117
Copy link
Collaborator

same thing under this section:

Verify the EVPN overlay peerings to Site 2.

@mthiel117
Copy link
Collaborator

L3LS_EVPN Lab

Under Day 2 Operations.... change:

Our multi-site L2LS network

to

Our multi-site L3LS_EVPN network

@RestlessWanderer
Copy link
Collaborator Author

L3LS_EVPN Lab

Under Day 2 Operations.... change:

Our multi-site L2LS network

to

Our multi-site L3LS_EVPN network

This has been fixed.

@RestlessWanderer
Copy link
Collaborator Author

newline need after line 145, before the next heading

I already have a blank line between the ``` and ### Verification

@RestlessWanderer
Copy link
Collaborator Author

L3LS_EVPN, under section "Build and Deploy Changes for EVPN Gateway Functionality". The code blocks for s1-bdr1 and s1-bdr2 are not rendering correctly.

This is caused by the markdown extension - mdx_truly_sane_lists. Proposing removing this extension.

@RestlessWanderer
Copy link
Collaborator Author

same thing under this section:

Verify the EVPN overlay peerings to Site 2.

This is caused by the markdown extension - mdx_truly_sane_lists. Proposing removing this extension.

@RestlessWanderer
Copy link
Collaborator Author

This section on L3LS_EVPN Lab guide is not rendering correct. Tabs need removed.

???+ abstract "Where did those VLANs come from?" You should notice some VLANs that we didn't define anywhere in the _NETWORK_SERVICES.yml data model which aren't related to MLAG. Specifically, these will be VLAN SVIs Vlan1199 and Vlan3009.

This is caused by the markdown extension - mdx_truly_sane_lists. Proposing removing this extension.

Copy link
Collaborator

@mthiel117 mthiel117 left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Collaborator

@JulioPDX JulioPDX left a comment

Choose a reason for hiding this comment

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

LGTM

@JulioPDX JulioPDX merged commit e67ceaa into main Aug 22, 2024
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants