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

[build][arm64] disable p4rt compile on arm64 for bazel not work #12798

Merged
merged 1 commit into from
Dec 4, 2022

Conversation

LuiSzee
Copy link
Contributor

@LuiSzee LuiSzee commented Nov 22, 2022

Why I did it

pre-compiled bazel is not work in arm64 docker container

shil@2f910d8d37b2:/sonic/src/sonic-p4rt/sonic-pins$ uname -a
Linux 2f910d8d37b2 5.4.0-132-generic #148-Ubuntu SMP Mon Oct 17 16:02:06 UTC 2022 aarch64 GNU/Linux
shil@2f910d8d37b2:/sonic/src/sonic-p4rt/sonic-pins$ bazel
Opening zip "/proc/self/exe": lseek(): Bad file descriptor
FATAL: Failed to open '/proc/self/exe' as a zip file: (error: 9): Bad file descriptor
shil@2f910d8d37b2:/sonic/src/sonic-p4rt/sonic-pins$

How I did it

disable p4rt compile on arm64

How to verify it

compile centec arm64 sonic image

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@LuiSzee
Copy link
Contributor Author

LuiSzee commented Nov 24, 2022

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@xumia
Copy link
Collaborator

xumia commented Nov 24, 2022

@LuiSzee , please specify the [pipeline] as below:

/azpw run Azure.sonic-buildimage 

@LuiSzee
Copy link
Contributor Author

LuiSzee commented Nov 25, 2022

/azpw run Azure.sonic-buildimage (BuildVS vs), Azure.sonic-buildimage (Build marvell_armhf)

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage (BuildVS vs), Azure.sonic-buildimage (Build marvell_armhf)

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@LuiSzee
Copy link
Contributor Author

LuiSzee commented Dec 2, 2022

@liushilongbuaa Please help to review this PR, for p4rt compile error.

@lguohan lguohan merged commit 5281f6c into sonic-net:master Dec 4, 2022
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Dec 6, 2022
…c-net#12798)

pre-compiled bazel is not work in arm64 docker container

shil@2f910d8d37b2:/sonic/src/sonic-p4rt/sonic-pins$ uname -a
Linux 2f910d8d37b2 5.4.0-132-generic sonic-net#148-Ubuntu SMP Mon Oct 17 16:02:06 UTC 2022 aarch64 GNU/Linux
shil@2f910d8d37b2:/sonic/src/sonic-p4rt/sonic-pins$ bazel
Opening zip "/proc/self/exe": lseek(): Bad file descriptor
FATAL: Failed to open '/proc/self/exe' as a zip file: (error: 9): Bad file descriptor
shil@2f910d8d37b2:/sonic/src/sonic-p4rt/sonic-pins$
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202211: #12956

mssonicbld pushed a commit that referenced this pull request Dec 6, 2022
pre-compiled bazel is not work in arm64 docker container

shil@2f910d8d37b2:/sonic/src/sonic-p4rt/sonic-pins$ uname -a
Linux 2f910d8d37b2 5.4.0-132-generic #148-Ubuntu SMP Mon Oct 17 16:02:06 UTC 2022 aarch64 GNU/Linux
shil@2f910d8d37b2:/sonic/src/sonic-p4rt/sonic-pins$ bazel
Opening zip "/proc/self/exe": lseek(): Bad file descriptor
FATAL: Failed to open '/proc/self/exe' as a zip file: (error: 9): Bad file descriptor
shil@2f910d8d37b2:/sonic/src/sonic-p4rt/sonic-pins$
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants