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

bad file type for MLNXOS #3251

Open
jloms opened this issue Aug 31, 2024 · 3 comments
Open

bad file type for MLNXOS #3251

jloms opened this issue Aug 31, 2024 · 3 comments
Labels
model_maintainer_needed This issue needs a maintainter for the model. See CONTRIBUTING.md#model-maintainers

Comments

@jloms
Copy link

jloms commented Aug 31, 2024

For MLNXOS class (mellanox whith Onyx OS), the config file contains ASCII but the linux type is binary.
For git output it is not pretty good.
I missed something in the config file ?

@robertcheramy robertcheramy added the model_maintainer_needed This issue needs a maintainter for the model. See CONTRIBUTING.md#model-maintainers label Sep 5, 2024
@robertcheramy
Copy link
Collaborator

mlnxos.rb is somewhat ugly, as it codes \e as binary char in the source code.
This must be addressed by someone having access to the hardware.
When I have finished my work on device simulation, I may help without access to the hardware.

@jlo-lirmm
Copy link

Thanks for the idea .... on the config file in oxidized stange char correspond to pager prompt
Adding cmd 'terminal length 999' in model/mlnxos.rb solved the issue.

@dtsname
Copy link

dtsname commented Sep 25, 2024

have a similar issue, who can give an advice what should I fix in mlnxos.rb? some unprinted codes in config output needs to be filtered.
interface ethernet 1/8 switchport trunk allowed-vlan add 100-101 interface ethernet 1/8 switchport trunk allowed-vlan add 150 interface ethernet 1/8 switchport trunk allowed-vlan remove 1 interface ethernet 1/9 switchport trunk allowed-vlan add 100-101 interface ethernet 1/9 switchport trunk allowed-vlan add 150 interface ethernet 1/9 switchport trunk allowed-vlan remove 1 interface ethernet 1/10 switchport trunk allowed-vlan add 100-101 ������������ interface ethernet 1/10 switchport trunk allowed-vlan add 150 interface ethernet 1/10 switchport trunk allowed-vlan remove 1 interface ethernet 1/11 switchport trunk allowed-vlan add 100-101 interface ethernet 1/11 switchport trunk allowed-vlan add 150 interface ethernet 1/11 switchport trunk allowed-vlan remove 1 interface ethernet 1/12 switchport trunk allowed-vlan add 100-101 interface ethernet 1/12 switchport trunk allowed-vlan add 150 interface ethernet 1/12 switchport trunk allowed-vlan remove 1 interface ethernet 1/13 switchport trunk allowed-vlan add 100-101

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
model_maintainer_needed This issue needs a maintainter for the model. See CONTRIBUTING.md#model-maintainers
Projects
None yet
Development

No branches or pull requests

4 participants