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

Deprecate Ubuntu 18.04 #1468

Merged
merged 5 commits into from
May 5, 2023
Merged

Conversation

CyanDevs
Copy link
Collaborator

@CyanDevs CyanDevs commented Apr 18, 2023

This deprecates Ubuntu 18.04 support and migrates anything still dependent on Ubuntu 18.04 to Ubuntu 20.04.

@CyanDevs CyanDevs force-pushed the cyan/deprecate-ubuntu-18.04 branch 3 times, most recently from def9aad to c54c8c1 Compare April 20, 2023 21:46
tests/azure-sdk-for-cpp/Dockerfile-ubuntu Outdated Show resolved Hide resolved
self.assertRaises(OverflowError, posix.sched_setparam, 0, param)

@unittest.skipUnless(hasattr(posix, "sched_rr_get_interval"), "no function")
+ @unittest.skip("Temporarily disabled in Mystikos")
def test_sched_rr_get_interval(self):
try:
interval = posix.sched_rr_get_interval(0)
@@ -1509,6 +1515,7 @@ def test_path_with_null_byte(self):
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test is disabled due to the following error
*** kernel panic: syscall.c(7828): _syscall(): unknown syscall: SYS_pidfd_open(): 434

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its unlikely we'll support this syscall.

solutions/cpython-tests/test_config_v3.10.2/patch Outdated Show resolved Hide resolved
libreadline-dev libsqlite3-dev libgdbm-dev libdb5.3-dev libbz2-dev \
libexpat1-dev liblzma-dev libffi-dev git uuid-dev lsb-core wget

# Workaround broken crc32_z symbol on zlib 1.2.11 for Ubuntu 20.04
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

crc32_z symbol is different between Ubuntu 20.04 and 18.04 even though both are v1.2.11. This is the error
ImportError: Error relocating /usr/lib/x86_64-linux-gnu/libz.so.1: crc32_z: symbol not found
image

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm thats seems puzzling. The symbol type is different in the above output - FUNC(18.04) vs IFUNC(20.4). It might be our musl loader doesn't support IFUNC symbols.
cc: @anakrish

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the traceback:

test test_exceptions crashed -- Traceback (most recent call last):
  File "/cpython/Lib/test/libregrtest/runtest.py", line 335, in _runtest_inner
    refleak = _runtest_inner2(ns, test_name)
  File "/cpython/Lib/test/libregrtest/runtest.py", line 280, in _runtest_inner2
    the_module = importlib.import_module(abstest)
  File "/cpython/Lib/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/cpython/Lib/test/test_exceptions.py", line 13, in <module>
    from test.support import (captured_stderr, check_impl_detail,
  File "/cpython/Lib/test/support/script_helper.py", line 11, in <module>
    import zipfile
  File "/cpython/Lib/zipfile.py", line 6, in <module>
    import binascii
ImportError: Error relocating /usr/lib/x86_64-linux-gnu/libz.so.1: crc32_z: symbol not found

test_exceptions failed (uncaught exception)

== Tests result: FAILURE ==

1 test failed:
    test_exceptions

@CyanDevs CyanDevs force-pushed the cyan/deprecate-ubuntu-18.04 branch from c54c8c1 to c985e32 Compare April 27, 2023 19:07
Signed-off-by: Chris Yan <chrisyan@microsoft.com>
Signed-off-by: Chris Yan <chrisyan@microsoft.com>
Signed-off-by: Chris Yan <chrisyan@microsoft.com>
Signed-off-by: Chris Yan <chrisyan@microsoft.com>
Signed-off-by: Chris Yan <chrisyan@microsoft.com>
@CyanDevs CyanDevs force-pushed the cyan/deprecate-ubuntu-18.04 branch from c985e32 to 7f1b5b8 Compare April 27, 2023 19:08
@CyanDevs CyanDevs changed the title [WIP] Deprecate Ubuntu 18.04 Deprecate Ubuntu 18.04 Apr 27, 2023
@CyanDevs CyanDevs marked this pull request as ready for review April 27, 2023 21:06
@CyanDevs CyanDevs merged commit 7953bd8 into deislabs:main May 5, 2023
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