Skip to content

Comments

build: updating migration check action requirement installation to match ci.yml requirements installation#4660

Merged
UsamaSadiq merged 2 commits intoopenedx:masterfrom
DawoudSheraz:dsheraz/fixing-migration-error
Aug 21, 2025
Merged

build: updating migration check action requirement installation to match ci.yml requirements installation#4660
UsamaSadiq merged 2 commits intoopenedx:masterfrom
DawoudSheraz:dsheraz/fixing-migration-error

Conversation

@DawoudSheraz
Copy link
Contributor

  • Updates migration check action requirement installation to match ci.yml requirements installation to fix the migration check failure
    • Unsure why it is failing in first place. Most likely, it is OS dependency flow in that particular action as the flow does not fail on my local Mac or even in ci.yml
  • Run make upgrade to update requirement files

Migration checks were failing with the following error (https://github.com/openedx/course-discovery/actions/runs/17066352522/job/48384347446?pr=4659) :

Collecting cairocffi==1.4.0 (from -r requirements/production.txt (line 76))
  Downloading cairocffi-1.4.0.tar.gz (69 kB)
  Preparing metadata (setup.py): started
  Preparing metadata (setup.py): finished with status 'error'
  error: subprocess-exited-with-error
  
  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [92 lines of output]
      /opt/hostedtoolcache/Python/3.12.11/x64/lib/python3.12/site-packages/setuptools/dist.py:759: SetuptoolsDeprecationWarning: License classifiers are deprecated.
      !!
      
              ********************************************************************************
              Please consider removing the following classifiers in favor of a SPDX license expression:
      
              License :: OSI Approved :: BSD License
      
              See https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#license for details.
              ********************************************************************************
      
      !!
        self._finalize_license_expression()
      /opt/hostedtoolcache/Python/3.12.11/x64/lib/python3.12/site-packages/setuptools/dist.py:483: SetuptoolsDeprecationWarning: Please provide a valid glob pattern.
      !!
      
              ********************************************************************************
              Pattern '[ LICENSE ]' contains invalid characters.
      
              By 2026-Mar-20, you need to update your project and remove deprecated calls
              or your builds will no longer be supported.
      
              See https://packaging.python.org/en/latest/specifications/glob-patterns/ for details.
              ********************************************************************************
      
      !!
        for path in sorted(cls._find_pattern(pattern, enforce_match))
      /opt/hostedtoolcache/Python/3.12.11/x64/lib/python3.12/site-packages/setuptools/dist.py:483: SetuptoolsDeprecationWarning: Cannot find any files for the given pattern.
      !!
      
              ********************************************************************************
              Pattern '[ LICENSE ]' did not match any files.
      
              By 2026-Mar-20, you need to update your project and remove deprecated calls
              or your builds will no longer be supported.
              ********************************************************************************
      
      !!
        for path in sorted(cls._find_pattern(pattern, enforce_match))
      /opt/hostedtoolcache/Python/3.12.11/x64/lib/python3.12/site-packages/setuptools/__init__.py:92: _DeprecatedInstaller: setuptools.installer and fetch_build_eggs are deprecated.
      !!
      
              ********************************************************************************
              Requirements should be satisfied by a PEP 517 installer.
              If you are using pip, you can try `pip install --use-pep517`.
      
              By 2025-Oct-31, you need to update your project and remove deprecated calls
              or your builds will no longer be supported.
              ********************************************************************************
      
      !!
        dist.fetch_build_eggs(dist.setup_requires)
      Traceback (most recent call last):
        File "/tmp/pip-install-zotjirau/cairocffi_0f5d3037b6aa4c768aa1222f2a0ad209/.eggs/cffi-1.17.1-py3.12-linux-x86_64.egg/cffi/cparser.py", line 5, in <module>
          from . import _pycparser as pycparser
      ImportError: cannot import name '_pycparser' from 'cffi' (/tmp/pip-install-zotjirau/cairocffi_0f5d3037b6aa4c768aa1222f2a0ad209/.eggs/cffi-1.17.1-py3.12-linux-x86_64.egg/cffi/__init__.py)
      
      During handling of the above exception, another exception occurred:
      
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 35, in <module>
        File "/tmp/pip-install-zotjirau/cairocffi_0f5d3037b6aa4c768aa1222f2a0ad209/setup.py", line 10, in <module>
          setup(
        File "/opt/hostedtoolcache/Python/3.12.11/x64/lib/python3.12/site-packages/setuptools/__init__.py", line 115, in setup
          return distutils.core.setup(**attrs)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/opt/hostedtoolcache/Python/3.12.11/x64/lib/python3.12/site-packages/setuptools/_distutils/core.py", line 148, in setup
          _setup_distribution = dist = klass(attrs)
                                       ^^^^^^^^^^^^
        File "/opt/hostedtoolcache/Python/3.12.11/x64/lib/python3.12/site-packages/setuptools/dist.py", line 321, in __init__
          _Distribution.__init__(self, dist_attrs)
        File "/opt/hostedtoolcache/Python/3.12.11/x64/lib/python3.12/site-packages/setuptools/_distutils/dist.py", line 309, in __init__
          self.finalize_options()
        File "/opt/hostedtoolcache/Python/3.12.11/x64/lib/python3.12/site-packages/setuptools/dist.py", line 784, in finalize_options
          ep(self)
        File "/opt/hostedtoolcache/Python/3.12.11/x64/lib/python3.12/site-packages/setuptools/dist.py", line 804, in _finalize_setup_keywords
          ep.load()(self, ep.name, value)
        File "/tmp/pip-install-zotjirau/cairocffi_0f5d3037b6aa4c768aa1222f2a0ad209/.eggs/cffi-1.17.1-py3.12-linux-x86_64.egg/cffi/setuptools_ext.py", line 216, in cffi_modules
          add_cffi_module(dist, cffi_module)
        File "/tmp/pip-install-zotjirau/cairocffi_0f5d3037b6aa4c768aa1222f2a0ad209/.eggs/cffi-1.17.1-py3.12-linux-x86_64.egg/cffi/setuptools_ext.py", line 49, in add_cffi_module
          execfile(build_file_name, mod_vars)
        File "/tmp/pip-install-zotjirau/cairocffi_0f5d3037b6aa4c768aa1222f2a0ad209/.eggs/cffi-1.17.1-py3.12-linux-x86_64.egg/cffi/setuptools_ext.py", line 25, in execfile
          exec(code, glob, glob)
        File "cairocffi/ffi_build.py", line 28, in <module>
          ffi = FFI()
                ^^^^^
        File "/tmp/pip-install-zotjirau/cairocffi_0f5d3037b6aa4c768aa1222f2a0ad209/.eggs/cffi-1.17.1-py3.12-linux-x86_64.egg/cffi/api.py", line 65, in __init__
          from . import cparser
        File "/tmp/pip-install-zotjirau/cairocffi_0f5d3037b6aa4c768aa1222f2a0ad209/.eggs/cffi-1.17.1-py3.12-linux-x86_64.egg/cffi/cparser.py", line 7, in <module>
          import pycparser
      ModuleNotFoundError: No module named 'pycparser'
      [end of output]
  
  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

@DawoudSheraz DawoudSheraz requested a review from a team August 21, 2025 09:53
@DawoudSheraz
Copy link
Contributor Author

@hamza-56 Hi, can you please merge it (if 2nd review is not needed)? I dont have permission to merge it.

cc: @UsamaSadiq

@UsamaSadiq UsamaSadiq merged commit 3648af9 into openedx:master Aug 21, 2025
14 checks passed
@DawoudSheraz DawoudSheraz deleted the dsheraz/fixing-migration-error branch August 25, 2025 11:00
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