Skip to content

ansible: update RHEL 8 machines to Python 3.11#4079

Merged
richardlau merged 1 commit intonodejs:mainfrom
richardlau:python3
May 9, 2025
Merged

ansible: update RHEL 8 machines to Python 3.11#4079
richardlau merged 1 commit intonodejs:mainfrom
richardlau:python3

Conversation

@richardlau
Copy link
Member

@richardlau richardlau commented May 8, 2025

@richardlau
Copy link
Member Author

@richardlau
Copy link
Member Author

richardlau commented May 8, 2025

Test CI (rhel8-s390x): https://ci.nodejs.org/job/node-test-commit-v8-linux/6538/nodes=rhel8-s390x,v8test=v8test/

Well that failed.

Exception: 1: Traceback (most recent call last):
  File "/home/iojs/build/workspace/node-test-commit-v8-linux/deps/v8/depot_tools/external_bin/gsutil/gsutil_4.68/gsutil/gsutil", line 21, in <module>
    gsutil.RunMain()
  File "/home/iojs/build/workspace/node-test-commit-v8-linux/deps/v8/depot_tools/external_bin/gsutil/gsutil_4.68/gsutil/gsutil.py", line 121, in RunMain
    import gslib.__main__
  File "/home/iojs/build/workspace/node-test-commit-v8-linux/deps/v8/depot_tools/external_bin/gsutil/gsutil_4.68/gsutil/gslib/__main__.py", line 36, in <module>
    from six.moves import configparser
ModuleNotFoundError: No module named 'six.moves'

Investigating.

@richardlau

This comment was marked as outdated.

@richardlau

This comment was marked as outdated.

@richardlau

This comment was marked as outdated.

@richardlau

This comment was marked as outdated.

The V8 CI now needs Python >= 3.10. Update to a newer version of
Python on the RHEL 8 machines used to run the V8 CI.

Also add `filecheck` dependency for running V8 tests.
@richardlau
Copy link
Member Author

Exception: 1: Traceback (most recent call last):
  File "/home/iojs/build/workspace/node-test-commit-v8-linux/deps/v8/depot_tools/external_bin/gsutil/gsutil_4.68/gsutil/gsutil", line 21, in <module>
    gsutil.RunMain()
  File "/home/iojs/build/workspace/node-test-commit-v8-linux/deps/v8/depot_tools/external_bin/gsutil/gsutil_4.68/gsutil/gsutil.py", line 121, in RunMain
    import gslib.__main__
  File "/home/iojs/build/workspace/node-test-commit-v8-linux/deps/v8/depot_tools/external_bin/gsutil/gsutil_4.68/gsutil/gslib/__main__.py", line 36, in <module>
    from six.moves import configparser
ModuleNotFoundError: No module named 'six.moves'

My current theory is that this is an issue with gsutil_4.68.

There is a bundled version of six in deps/v8/depot_tools/external_bin/gsutil/gsutil_4.68/gsutil/third_party/six/.

[iojs@test-osuosl-rhel8-ppc64--le-2 v8]$ head depot_tools/external_bin/gsutil/gsutil_4.68/gsutil/third_party/six/CHANGES
Changelog for six
=================

This file lists the changes in each six version.

1.12.0
------

- Issue #259, pull request #260: `six.add_metaclass` now preserves
  `__qualname__` from the original class.
[iojs@test-osuosl-rhel8-ppc64--le-2 v8]$

which is old (current version is 1.17.0). The current version of gsutil itself is 5.34 and Python 3.12 support was added to gsutil in 5.27 by GoogleCloudPlatform/gsutil#1751.

I'll try rolling back to Python 3.11.

@richardlau
Copy link
Member Author

richardlau commented May 8, 2025

@richardlau richardlau changed the title ansible: update RHEL 8 machines to Python 3.12 ansible: update RHEL 8 machines to Python 3.11 May 8, 2025
@richardlau richardlau merged commit ab82927 into nodejs:main May 9, 2025
1 check passed
@richardlau richardlau deleted the python3 branch May 9, 2025 16:28
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.

Update Python on machines used to test V8

2 participants