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

Update lab-example clos02 #1512

Merged
merged 3 commits into from
Aug 10, 2023
Merged

Update lab-example clos02 #1512

merged 3 commits into from
Aug 10, 2023

Conversation

ubaumann
Copy link
Contributor

To be able to configure the lab, I needed to add an address family in BGP and to bring the IP layer up, I had to add the "admin-state".

I also needed to install gnmic and change the password in the script.

This is my first time using Nokia SR Linux, so I hope I did everything correctly. I am happy to discuss the changes and update it.

lab-examples/clos02/setup.sh Outdated Show resolved Hide resolved
@hellt
Copy link
Member

hellt commented Aug 10, 2023

Thanks @ubaumann, this is a great contribution.
The reason you had to make those changes is because we didn't pin the SR Linux container to a particular version these initial config snippets were tested against.

Would you be able to ping the version in the clab file to 23.3.3 (current latest) so that others won't have the same issues should they appear with later releases?
Thanks!

@ubaumann
Copy link
Contributor Author

Yes, I thought it must have been a change in the YANG model in one of the versions, but I was not in the mood to try to find the correct version, so it was easier (and more fun) to update the data.

@ubaumann
Copy link
Contributor Author

@hellt @steiler Hope it looks good now. If I can change anything else, please let me know.

@hellt hellt merged commit 46f8c52 into srl-labs:main Aug 10, 2023
15 checks passed
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

3 participants