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

Issue with Arista version in quickstart topology #2033

Closed
james-m-henderson opened this issue May 4, 2024 · 5 comments · Fixed by #2034
Closed

Issue with Arista version in quickstart topology #2033

james-m-henderson opened this issue May 4, 2024 · 5 comments · Fixed by #2034

Comments

@james-m-henderson
Copy link
Contributor

I went through the quickstart guide and have noted a couple of issues.

The most significant is that the suggested version of Arista (4.25.0F) is not present on the Arista website for download. The closest I could find was 4.25.11M.

After downloading this version and running, containerlab seemed to hang indefinitely after doing containerlab deploy.

I noticed that while the docker VMs were up, if I try to use the Cli command on Arista, it doesn't work:

jmh@the-pot:~/clab-quickstart$ docker exec -it clab-srlceos01-ceos bash
bash-4.2# exit
jmh@the-pot:~/clab-quickstart$ docker exec -it clab-srlceos01-ceos Cli
OCI runtime exec failed: exec failed: unable to start container process: exec: "Cli": executable file not found in $PATH: unknown

In order to fix, I tried downloading the latest available Arista image, 4.32.0F. This works without flaw, topology starts up and the above Cli command works.

There was one other small flaw I noticed. The command given to connect to the srl container is wrong.

It is listed as:

docker exec -it clab-srlceos01-srl1 bash

It should be:

docker exec -it clab-srlceos01-srl bash
@james-m-henderson
Copy link
Contributor Author

Here is the topology file with the "good" version activated and the "bad" version commented out:

# topology documentation: http://containerlab.dev/lab-examples/srl-ceos/
name: srlceos01

topology:
  nodes:
    srl:
      kind: nokia_srlinux
      image: ghcr.io/nokia/srlinux
    ceos:
      kind: arista_ceos
      image: ceos:4.32.0F
      #image: ceos:4.25.11M

  links:
    - endpoints: ["srl:e1-1", "ceos:eth1"]

james-m-henderson added a commit to james-m-henderson/containerlab that referenced this issue May 4, 2024
The existing version couldn't be found and 4.25.11M didn't work.

srl-labs#2033
@hellt
Copy link
Member

hellt commented May 4, 2024

thanks @james-m-henderson
do you want to raise a PR with those fixes and get into the prestigious contributors ranks? =)

thank you!

@james-m-henderson
Copy link
Contributor Author

james-m-henderson commented May 4, 2024 via email

@hellt
Copy link
Member

hellt commented May 4, 2024

Feel free to update all of them to the version you tested. We don't have sophisticated labs that exercise advanced platform features, so you can safely bump them all to the 4.32.0F

james-m-henderson added a commit to james-m-henderson/containerlab that referenced this issue May 4, 2024
srl-labs#2033

Also corrects some trailing whitespace issues.
@james-m-henderson
Copy link
Contributor Author

#2034

I have to checkout for the day, and I haven't yet installed all the test dependencies into my lab, so I haven't done full regression yet. However I did run the updated .yml files and it seems everything is OK.

hellt pushed a commit that referenced this issue May 5, 2024
* Bump version of Arista in quickstart lab to 4.32.0F

The existing version couldn't be found and 4.25.11M didn't work.

#2033

* Correct command to get into srl bash

* Fix typo

* Standardize on ceos:4.32.0F

#2033

Also corrects some trailing whitespace issues.
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 a pull request may close this issue.

2 participants