Skip to content

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

Merged
lguohan merged 1 commit intosonic-net:masterfrom
chihhan123:master
Jun 9, 2020
Merged

Update AS7312-54X,AS7312-54XS,AS7315-27XB config.bcm file#4702
lguohan merged 1 commit intosonic-net:masterfrom
chihhan123:master

Conversation

@chihhan123
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

retest vs please

@chihhan123
Copy link
Copy Markdown
Contributor Author

retest vsimage please

@chihhan123
Copy link
Copy Markdown
Contributor Author

retest mellanox please

2 similar comments
@chihhan123
Copy link
Copy Markdown
Contributor Author

retest mellanox please

@chihhan123
Copy link
Copy Markdown
Contributor Author

retest mellanox please

@tanpinsiang
Copy link
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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
Copy Markdown

@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