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 AS7312-54X,AS7312-54XS,AS7315-27XB config.bcm file #4702

Merged
merged 1 commit into from
Jun 9, 2020
Merged

Update AS7312-54X,AS7312-54XS,AS7315-27XB config.bcm file #4702

merged 1 commit into from
Jun 9, 2020

Conversation

chihhan123
Copy link
Contributor

@chihhan123 chihhan123 commented Jun 4, 2020

Update AS7312-54X,AS7312-54XS,AS7315-27XB config.bcm file to make sure there is no the following error message.

configuration: format error in /usr/share/sonic/hwsku/th-as7312-48x25G+6x100G.config.bcm on line 110 (ignored)#15

@chihhan123
Copy link
Contributor Author

retest vs please

@chihhan123
Copy link
Contributor Author

retest vsimage please

@chihhan123
Copy link
Contributor Author

retest mellanox please

2 similar comments
@chihhan123
Copy link
Contributor Author

retest mellanox please

@chihhan123
Copy link
Contributor Author

retest mellanox please

@tanpinsiang
Copy link

Hi chihhan123,

I tested this commit but I still get the error:
sonic INFO syncd#supervisord: syncd Platform configuration: format error in /usr/share/sonic/hwsku/th-as7312-48x25G+6x100G.config.bcm on line 110 (ignored)#015

I found that this file is not exist in the location:

ls -l /usr/share/sonic/hwsku/th-as7312-48x25G+6x100G.config.bcm
ls: cannot access '/usr/share/sonic/hwsku/th-as7312-48x25G+6x100G.config.bcm': No such file or directory

The file do exist in the location with the changes:

/usr/share/sonic/device/x86_64-accton_as7312_54xs-r0/Accton-AS7312-54XS/th-as7312-48x25G+6x100G.config.bcm

/usr/share/sonic/device/x86_64-kvm_x86_64-r0/Accton-AS7312-54XS/th-as7312-48x25G+6x100G.config.bcm

@chihhan123
Copy link
Contributor Author

HI Tan Pin Siang,
You should find the file /usr/share/sonic/hwsku/th-as7312-48x25G+6x100G.config.bcm
under docker syncd.
The following command could go to docker syncd environment.
docker exec –it syncd bash

Sincerely,
Chih Han

@chihhan123
Copy link
Contributor Author

Hi Tan Pin Siang,
please make sure the line 110 in /usr/share/sonic/device/x86_64-accton_as7312_54xs-r0/Accton-AS7312-54XS/th-as7312-48x25G+6x100G.config.bcm is right.
#/* Port Map */

Thanks!
Sincerely,
Chih Han

@tanpinsiang
Copy link

@chihhan123 The previous error I observed are from the last version. Now the error do not appearing after a few reboot. So this PR does fix the format error.
I checked the config.bcm according to your instruction. The results are normal with the changes:

admin@sonic:~$ docker exec -it syncd bash
root@sonic:/# sed -n 110p /usr/share/sonic/hwsku/th-as7312-48x25G+6x100G.config.bcm
#/* Port Map */
root@sonic:/# exit
admin@sonic:~$ sed -n 110p /usr/share/sonic/device/x86_64-accton_as7312_54xs-r0/Accton-AS7312-54XS/th-as7312-48x25G+6x100G.config.bcm
#/* Port Map */

I also included the dump for your further inspection.

Thanks for your attention on this issue.

@lguohan lguohan merged commit 23ddfb8 into sonic-net:master Jun 9, 2020
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.

3 participants