Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adding support for vJunos-switch #138

Merged
merged 7 commits into from
Aug 25, 2023
Merged

adding support for vJunos-switch #138

merged 7 commits into from
Aug 25, 2023

Conversation

akielaries
Copy link

@akielaries akielaries commented Aug 22, 2023

This is a PR to add support for Juniper's vJunos-switch image that emulates an EX2914. vJunos Labs is a recent initiative that features lab images vJunos-switch and vJunosEvolved, the latter I am working out some kinks and will add this in a later PR. I used the below topology file with containerlab for testing. Once this is verified and upstream for vrnetlab I will make the PR for containerlab as well.

name: demo

topology:
  nodes:
    n1:
      kind: srl
      image: ghcr.io/nokia/srlinux
      startup-config: srl.cfg

    vswitch0:
      kind: vr-vjunosswitch
      image: vrnetlab/vr-vjunosswitch:23.2R1.14
      startup-config: vjunos.cfg

  links:
    - endpoints: ["n1:e1-1", "vswitch0:eth1"]

The existing Juniper nodes (vSRX, vMX, vQFX) feature some send/wait type logic on the Junos CLI to configure the nodes with their bare minimum configs enforced by vrnetlab (such as user, password, various set commands, etc). Instead of using this logic, I used the shipped make-config.sh shell script featured in the support.juniper.net site that creates a disk image based on a juniper.conf file (this is where the various bare minimum configs lie) and then mounts this disk to the main vJunos-switch VM image.

@hellt
Copy link
Owner

hellt commented Aug 23, 2023

Hi @akielaries
thank you. I believe a matching PR should be opened against containerlab with the added kind.

Ah, I see that you have noted this...

vjunosswitch/README.md Outdated Show resolved Hide resolved
vjunosswitch/README.md Outdated Show resolved Hide resolved
Co-authored-by: Roman Dodin <dodin.roman@gmail.com>
@hellt
Copy link
Owner

hellt commented Aug 25, 2023

thanks @akielaries

@hellt hellt merged commit e28c846 into hellt:master Aug 25, 2023
1 check failed
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.

None yet

2 participants