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

[sonic-restapi] ARM architecture not supported #9896

Closed
dflynn-Nokia opened this issue Jan 31, 2022 · 5 comments
Closed

[sonic-restapi] ARM architecture not supported #9896

dflynn-Nokia opened this issue Jan 31, 2022 · 5 comments
Assignees
Labels
Platform: Nokia Triaged this issue has been triaged

Comments

@dflynn-Nokia
Copy link
Contributor

Description

The sonic-restapi module currently seems to be only applicable to the 'amd64' architecture.
As part of the ongoing effort to support the ARM platform architecture, sonic-restapi should also be provided for ('armhf' and 'arm64').

Here's the results when attempting to build sonic-restapi for 'armhf'
Note the result of building sonic-rest-api_1.0.1_amd64.deb instead of sonic-rest-api_1.0.1_armhf.deb
I am building within the sonic-buildimage 202012 branch.

[ FAIL LOG START ] [ target/debs/stretch/sonic-rest-api_1.0.1_amd64.deb ]
[ REASON ] :      target/debs/stretch/sonic-rest-api_1.0.1_amd64.deb does not exist   NON-EXISTENT PREREQUISITES: target/debs/stretch/libhiredis-dev_0.14.0-3~bpo9+1_armhf.deb-install target/debs/stretch/libnl-3-dev_3.5.0-1_armhf.deb-install target/debs/stretch/libnl-genl-3-dev_3.5.0-1_armhf.deb-install target/debs/stretch/libnl-route-3-dev_3.5.0-1_armhf.deb-install target/debs/stretch/libswsscommon-dev_1.0.0_armhf.deb-install target/debs/stretch/libswsscommon_1.0.0_armhf.deb-install 
[ FLAGS  FILE    ] : [] 
[ FLAGS  DEPENDS ] : [] 
[ FLAGS  DIFF    ] : [] 
/sonic/src/sonic-restapi /sonic
dpkg-buildpackage: info: source package sonic-rest-api-server
dpkg-buildpackage: info: source version 1.0.1
dpkg-buildpackage: info: source distribution stable
dpkg-buildpackage: info: source changed by Prince Sunny <prsunny@microsoft.com>
 dpkg-source --before-build sonic-restapi
dpkg-buildpackage: info: host architecture armhf
 fakeroot debian/rules clean
make[1]: Entering directory '/sonic/src/sonic-restapi'
dh clean  --with autotools-dev
   dh_auto_clean
        make -j1 clean
make[2]: Entering directory '/sonic/src/sonic-restapi'
rm -rf /var/dflynn/go
make[2]: Leaving directory '/sonic/src/sonic-restapi'
   dh_autotools-dev_restoreconfig
   dh_clean
make[1]: Leaving directory '/sonic/src/sonic-restapi'
 debian/rules build
make[1]: Entering directory '/sonic/src/sonic-restapi'
dh build  --with autotools-dev
   dh_update_autotools_config
   dh_autotools-dev_updateconfig
   debian/rules override_dh_auto_configure
make[2]: Entering directory '/sonic/src/sonic-restapi'
dh_auto_configure --
make[2]: Leaving directory '/sonic/src/sonic-restapi'
   dh_auto_build
        make -j1
make[2]: Entering directory '/sonic/src/sonic-restapi'
cd libcswsscommon && sudo make install
ar rvs libcswsscommon.a src/dbconnector.o src/producertable.o src/producerstatetable.o src/table.o
r - src/dbconnector.o
r - src/producertable.o
r - src/producerstatetable.o
r - src/table.o
cp libcswsscommon.a /usr/lib # FIXME: better use install
cp -r include/* /usr/include
mkdir -p               /var/dflynn/go/src
cp -r go-server-server /var/dflynn/go/src/go-server-server
cp -r swsscommon       /var/dflynn/go/src/swsscommon
cd /var/dflynn/go/src/go-server-server && /usr/local/go/bin/go get -v && /usr/local/go/bin/go build -v
go: downloading github.com/vharitonsky/iniflags v0.0.0-20180513140207-a33cd0b5f3de
go: downloading github.com/satori/go.uuid v1.2.1-0.20180404165556-75cca531ea76
go: downloading github.com/go-redis/redis/v7 v7.3.0
go: downloading github.com/comail/colog v0.0.0-20160416085026-fba8e7b1f46c
go: downloading github.com/gorilla/mux v1.7.4
runtime/cgo
github.com/go-redis/redis/v7/internal/consistenthash
github.com/go-redis/redis/v7/internal/util
github.com/go-redis/redis/v7/internal/hashtag
github.com/go-redis/redis/v7/internal/proto
github.com/go-redis/redis/v7/internal
github.com/comail/colog
swsscommon
net
# swsscommon
../swsscommon/producerstatetable.go:55:17: type [1073741824]_Ctype_struct_field_value_tuple larger than address space
../swsscommon/producerstatetable.go:55:17: type [1073741824]_Ctype_struct_field_value_tuple too large
../swsscommon/table.go:55:17: type [1073741824]_Ctype_struct_field_value_tuple larger than address space
../swsscommon/table.go:55:17: type [1073741824]_Ctype_struct_field_value_tuple too large
vendor/golang.org/x/net/http/httpproxy
github.com/go-redis/redis/v7/internal/pool
github.com/satori/go.uuid
net/textproto
crypto/x509
vendor/golang.org/x/net/http/httpguts
crypto/tls
net/http/httptrace
github.com/go-redis/redis/v7
net/http
github.com/vharitonsky/iniflags
github.com/gorilla/mux
Makefile:19: recipe for target '/var/dflynn/go/bin/go-server-server' failed
make[2]: *** [/var/dflynn/go/bin/go-server-server] Error 2
make[2]: Leaving directory '/sonic/src/sonic-restapi'
dh_auto_build: make -j1 returned exit code 2
debian/rules:22: recipe for target 'build' failed
make[1]: *** [build] Error 2
make[1]: Leaving directory '/sonic/src/sonic-restapi'
dpkg-buildpackage: error: debian/rules build gave error exit status 2
[  FAIL LOG END  ] [ target/debs/stretch/sonic-rest-api_1.0.1_amd64.deb ]

Steps to reproduce the issue:

  1. make configure PLATFORM=marvell-armhf PLATFORM_ARCH=armhf
  2. make target/debs/stretch/sonic-rest-api_1.0.1_armhf.deb

Describe the results you received:

Failed sonic-rest-api debian package build for amd64

Describe the results you expected:

Successful sonic-rest-api debian package build for armhf

Output of show version:

N/A

Output of show techsupport:

N/A

Additional information you deem important (e.g. issue happens only occasionally):

@prsunny
Copy link
Contributor

prsunny commented Feb 1, 2022

@sumukhatv , @xumia for viz

@zhangyanzhao zhangyanzhao added the Triaged this issue has been triaged label Feb 2, 2022
@zhangyanzhao
Copy link
Collaborator

Feature gap, need be handled in future release if effort is too big.

@dflynn-Nokia
Copy link
Contributor Author

Sorry for being late to join this week's triage meeting.
Please note that this is not urgent but it would be good to finish off our SONiC support for ARM

@Blueve
Copy link
Contributor

Blueve commented Apr 22, 2022

@xumia Do we have any plan to enable this module in arm platform?

@xumia xumia self-assigned this Apr 24, 2022
xumia added a commit that referenced this issue Apr 28, 2022
Why I did it
Fix target target/debs/bullseye/sonic-rest-api_1.0.1_arm64.deb not existing issue, the correct target is target/debs/bullseye/sonic-rest-api_1.0.1_armhf.deb.
Fix issue: #9896

[ FAIL LOG START ] [ target/debs/stretch/sonic-rest-api_1.0.1_amd64.deb ]
[ REASON ] :      target/debs/stretch/sonic-rest-api_1.0.1_amd64.deb does not exist   NON-EXISTENT PREREQUISITES: 
[ FLAGS  FILE    ] : []
@xumia
Copy link
Collaborator

xumia commented Apr 28, 2022

Fixed on master branch, it will take some time for release manages to cherry-pick the PR to 202012/202106/202111.

@xumia xumia closed this as completed Apr 28, 2022
judyjoseph pushed a commit that referenced this issue May 2, 2022
Why I did it
Fix target target/debs/bullseye/sonic-rest-api_1.0.1_arm64.deb not existing issue, the correct target is target/debs/bullseye/sonic-rest-api_1.0.1_armhf.deb.
Fix issue: #9896

[ FAIL LOG START ] [ target/debs/stretch/sonic-rest-api_1.0.1_amd64.deb ]
[ REASON ] :      target/debs/stretch/sonic-rest-api_1.0.1_amd64.deb does not exist   NON-EXISTENT PREREQUISITES: 
[ FLAGS  FILE    ] : []
qiluo-msft pushed a commit that referenced this issue May 3, 2022
Why I did it
Fix target target/debs/bullseye/sonic-rest-api_1.0.1_arm64.deb not existing issue, the correct target is target/debs/bullseye/sonic-rest-api_1.0.1_armhf.deb.
Fix issue: #9896

[ FAIL LOG START ] [ target/debs/stretch/sonic-rest-api_1.0.1_amd64.deb ]
[ REASON ] :      target/debs/stretch/sonic-rest-api_1.0.1_amd64.deb does not exist   NON-EXISTENT PREREQUISITES: 
[ FLAGS  FILE    ] : []
liushilongbuaa pushed a commit to liushilongbuaa/sonic-buildimage that referenced this issue Jun 20, 2022
)

Why I did it
Fix target target/debs/bullseye/sonic-rest-api_1.0.1_arm64.deb not existing issue, the correct target is target/debs/bullseye/sonic-rest-api_1.0.1_armhf.deb.
Fix issue: sonic-net#9896

[ FAIL LOG START ] [ target/debs/stretch/sonic-rest-api_1.0.1_amd64.deb ]
[ REASON ] :      target/debs/stretch/sonic-rest-api_1.0.1_amd64.deb does not exist   NON-EXISTENT PREREQUISITES:
[ FLAGS  FILE    ] : []
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Platform: Nokia Triaged this issue has been triaged
Projects
None yet
Development

No branches or pull requests

5 participants