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 data server build step to run with ubuntu 20.04 and change platform tag #6636

Merged
merged 1 commit into from
Oct 18, 2023

Conversation

groszewn
Copy link
Contributor

@groszewn groszewn commented Oct 16, 2023

This should ensure that the generated wheels are compatible with the manylinux_2_31 platform tag, since that's the version of glibc shipped with ubuntu 20.04. This aligns us with https://peps.python.org/pep-0600/ instead of reconfiguring our CI workflow to build on a system with glibc<=2.17 to keep supporting the manylinux2014 tag (https://peps.python.org/pep-0599/).

@groszewn groszewn force-pushed the rustboard branch 8 times, most recently from 149199b to ad57d76 Compare October 17, 2023 00:44
@groszewn groszewn changed the title Update data server build step to run with manylinux2014 container Update data server build step to run with ubuntu 20.04 and change platform tag Oct 17, 2023
@groszewn groszewn marked this pull request as ready for review October 17, 2023 01:12
@groszewn
Copy link
Contributor Author

Downloading the created wheels from CI and auditing the wheel show that the manylinux_2_31_x86_64 platform tag is correct for the generated wheel.

❯ unzip tensorboard-data-server.zip
Archive:  tensorboard-data-server.zip
  inflating: tensorboard_data_server-0.8.0a0-py3-none-any.whl
  inflating: tensorboard_data_server-0.8.0a0-py3-none-macosx_10_9_x86_64.whl
  inflating: tensorboard_data_server-0.8.0a0-py3-none-manylinux_2_31_x86_64.whl
❯ ls
tensorboard_data_server-0.8.0a0-py3-none-any.whl
tensorboard_data_server-0.8.0a0-py3-none-macosx_10_9_x86_64.whl
tensorboard_data_server-0.8.0a0-py3-none-manylinux_2_31_x86_64.whl
❯ pip install auditwheel
...
❯ auditwheel show tensorboard_data_server-0.8.0a0-py3-none-manylinux_2_31_x86_64.whl

tensorboard_data_server-0.8.0a0-py3-none-manylinux_2_31_x86_64.whl is
consistent with the following platform tag: "manylinux_2_31_x86_64".

The wheel references external versioned symbols in these
system-provided shared libraries: libdl.so.2 with versions
{'GLIBC_2.2.5'}, libm.so.6 with versions {'GLIBC_2.2.5'},
libpthread.so.0 with versions {'GLIBC_2.12', 'GLIBC_2.2.5'},
libgcc_s.so.1 with versions {'GCC_3.0', 'GCC_3.3', 'GCC_4.2.0'},
libc.so.6 with versions {'GLIBC_2.28', 'GLIBC_2.25', 'GLIBC_2.3.2',
'GLIBC_2.29', 'GLIBC_2.3.4', 'GLIBC_2.18', 'GLIBC_2.15', 'GLIBC_2.3',
'GLIBC_2.10', 'GLIBC_2.17', 'GLIBC_2.9', 'GLIBC_2.14', 'GLIBC_2.4',
'GLIBC_2.7', 'GLIBC_2.2.5'}

This constrains the platform tag to "manylinux_2_31_x86_64". In order
to achieve a more compatible tag, you would need to recompile a new
wheel from source on a system with earlier versions of these
libraries, such as a recent manylinux image.

@groszewn groszewn requested a review from arcra October 17, 2023 18:34
tensorboard/data/server/pip_package/build.py Outdated Show resolved Hide resolved
…tform tag

This should ensure that the generated wheels are compatible with the
`manylinux_2_31` platform tag, since that's the version of glibc shipped
with ubuntu 20.04. This aligns us with https://peps.python.org/pep-0600/
instead of reconfiguring our CI workflow to build on a system with
glibc<=2.17 to keep supporting the `manylinux2014` tag
(https://peps.python.org/pep-0599/)
@groszewn groszewn merged commit 717dec8 into tensorflow:master Oct 18, 2023
13 checks passed
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.

None yet

2 participants