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

support for vJunosEvolved #152

Merged
merged 5 commits into from
Dec 9, 2023
Merged

support for vJunosEvolved #152

merged 5 commits into from
Dec 9, 2023

Conversation

akielaries
Copy link

Adds support for vJunosEvolved. 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: srlvjunos02

topology:
  nodes:
    srl:
      kind: nokia_srlinux
      image: ghcr.io/nokia/srlinux:23.7.1
      startup-config: srl.cli

    vevo:
      kind: vr-juniper_vjunosevolved
      image: vrnetlab/vr-vjunosevolved:23.2R1-S1.8-EVO
      startup-config: vjunos.cfg

  links:
    - endpoints: ["srl:e1-1", "vevo:eth1"]
    - endpoints: ["srl:e1-2", "vevo:eth2"]
    - endpoints: ["srl:e1-3", "vevo:eth3"]

@hellt
Copy link
Owner

hellt commented Dec 4, 2023

Thanks @akielaries
Do you mind posting here the clab binary you used for testing so that more people could give it a try?

@akielaries
Copy link
Author

@hellt I had built my containerlab binary from this branch: https://github.com/akielaries/containerlab/tree/vJunosEvolved

vjunosevolved/README.md Outdated Show resolved Hide resolved
@@ -0,0 +1,28 @@
FROM ubuntu:20.04
Copy link
Owner

Choose a reason for hiding this comment

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

let's test it with debian:bookworm-slim as this is our current base image.

Copy link
Author

Choose a reason for hiding this comment

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

Sure thing. I was able to change the image to debian:bookworm-slim and tested it to make sure it works as well. Also went ahead and adjusted the launch.py script to remove some repeated QEMU arguments

@hellt
Copy link
Owner

hellt commented Dec 9, 2023

thnx @akielaries

@hellt hellt merged commit 6486509 into hellt:master Dec 9, 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.

2 participants