Skip to content

Conversation

@robertmaynard
Copy link
Contributor

No description provided.

@copy-pr-bot
Copy link

copy-pr-bot bot commented Jan 7, 2026

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@robertmaynard
Copy link
Contributor Author

/okay to test

@robertmaynard robertmaynard force-pushed the fea/add_spdx_license_builder_to_ci_jobs branch from 6e69111 to c64c622 Compare January 7, 2026 19:03
@robertmaynard
Copy link
Contributor Author

/okay to test

@robertmaynard robertmaynard force-pushed the fea/add_spdx_license_builder_to_ci_jobs branch from c64c622 to 77b4bd8 Compare January 7, 2026 20:10
@robertmaynard
Copy link
Contributor Author

/okay to test

@robertmaynard robertmaynard force-pushed the fea/add_spdx_license_builder_to_ci_jobs branch from 77b4bd8 to 9484455 Compare January 7, 2026 20:11
@robertmaynard
Copy link
Contributor Author

/okay to test

@robertmaynard
Copy link
Contributor Author

Waiting on rapidsai/shared-workflows#488 before we can merge

@robertmaynard robertmaynard force-pushed the fea/add_spdx_license_builder_to_ci_jobs branch 2 times, most recently from a143845 to 4a0897b Compare January 8, 2026 17:28
@robertmaynard robertmaynard changed the base branch from main to release/26.02 January 16, 2026 18:48
@msarahan
Copy link
Contributor

CI should be re-run when rapidsai/spdx-license-builder#3 is merged

@msarahan msarahan force-pushed the fea/add_spdx_license_builder_to_ci_jobs branch from 7606fc7 to 1642778 Compare January 21, 2026 22:10
@copy-pr-bot
Copy link

copy-pr-bot bot commented Jan 21, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@msarahan
Copy link
Contributor

/okay to test

@copy-pr-bot
Copy link

copy-pr-bot bot commented Jan 21, 2026

/okay to test

@msarahan, there was an error processing your request: E1

See the following link for more information: https://docs.gha-runners.nvidia.com/cpr/e/1/

@msarahan
Copy link
Contributor

/ok to test 1642778

@msarahan
Copy link
Contributor

@robertmaynard how do you want to treat situations where licenses don't get bundled?

Fetching license NCSA from SPDX API...
Warning: Error fetching license NCSA from SPDX API: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1017)>
Fetching license BSL-1.0 from SPDX API...
Warning: Error fetching license BSL-1.0 from SPDX API: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1017)>
Fetching license BSD-2-Clause from SPDX API...
Warning: Error fetching license BSD-2-Clause from SPDX API: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1017)>
============================================================
Machine-friendly JSON output written to: c/build/install/licenses.json
Fetching license Apache-2.0 from SPDX API...
Warning: Error fetching license Apache-2.0 from SPDX API: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1017)>
User-friendly text output written to: c/build/install/LICENSE

My inclination is to fail this build, because leaving the license out could/would mean that we are not in compliance with the license?

To be clear, I am working on fixing the network issues here, but just wanted to check with you for future error handling.

@robertmaynard
Copy link
Contributor Author

@robertmaynard how do you want to treat situations where licenses don't get bundled?

Fetching license NCSA from SPDX API...
Warning: Error fetching license NCSA from SPDX API: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1017)>
Fetching license BSL-1.0 from SPDX API...
Warning: Error fetching license BSL-1.0 from SPDX API: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1017)>
Fetching license BSD-2-Clause from SPDX API...
Warning: Error fetching license BSD-2-Clause from SPDX API: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1017)>
============================================================
Machine-friendly JSON output written to: c/build/install/licenses.json
Fetching license Apache-2.0 from SPDX API...
Warning: Error fetching license Apache-2.0 from SPDX API: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1017)>
User-friendly text output written to: c/build/install/LICENSE

My inclination is to fail this build, because leaving the license out could/would mean that we are not in compliance with the license?

To be clear, I am working on fixing the network issues here, but just wanted to check with you for future error handling.

I agree we should have license packaging issues fail the build.

If this becomes a frequent occurance on PRs we can make just the 'nighly' builds ( build.yaml ) require success for license while the PR.yaml could allow failures since those binaries shouldn't be distributed.

@msarahan
Copy link
Contributor

/ok to test 1642778

@msarahan
Copy link
Contributor

/ok to test 1642778

@msarahan
Copy link
Contributor

looks happy:

Looking in indexes: https://pypi.org/simple, https://pypi.anaconda.org/rapidsai-wheels-nightly/simple, https://pypi.nvidia.com/
Collecting git+https://github.com/rapidsai/spdx-license-builder.git
  Cloning https://github.com/rapidsai/spdx-license-builder.git to /tmp/pip-req-build-kqbfyyvi
  Running command git clone --filter=blob:none --quiet https://github.com/rapidsai/spdx-license-builder.git /tmp/pip-req-build-kqbfyyvi
  Resolved https://github.com/rapidsai/spdx-license-builder.git to commit 5cf9fa29a2a75c1ce842329716a78157cbf1b85a
  Installing build dependencies: started
  Installing build dependencies: finished with status 'done'
  Getting requirements to build wheel: started
  Getting requirements to build wheel: finished with status 'done'
  Preparing metadata (pyproject.toml): started
  Preparing metadata (pyproject.toml): finished with status 'done'
Requirement already satisfied: certifi>=2023.0.0 in /pyenv/versions/3.10.19/lib/python3.10/site-packages (from spdx-license-builder==26.4.0) (2026.1.4)
Building wheels for collected packages: spdx-license-builder
  Building wheel for spdx-license-builder (pyproject.toml): started
  Building wheel for spdx-license-builder (pyproject.toml): finished with status 'done'
  Created wheel for spdx-license-builder: filename=spdx_license_builder-26.4.0-py3-none-any.whl size=70915 sha256=54aa7bd1e89d672f2f6940a3961a8b603cb118d76fbaf503e36d39f4c6818246
  Stored in directory: /tmp/pip-ephem-wheel-cache-skr43vsy/wheels/0f/26/d7/e51c8676ba92887a9d9b3ba3045560c75b6c7253cb0c8aac81
Successfully built spdx-license-builder
Installing collected packages: spdx-license-builder
Successfully installed spdx-license-builder-26.4.0
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager, possibly rendering your system unusable. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv. Use the --root-user-action option if you know what you are doing and want to suppress this warning.
Project path(s): .
============================================================
============================================================
Project path(s): .
============================================================

Checking for project LICENSE file in: .
  Found 1 license(s) in LICENSE: Apache-2.0
Extracting SPDX copyright entries from source files...
  Scanning directory: /__w/cuvs/cuvs
  Scanned 8535 files
  Found 3042 copyright entries
  In 1914 unique files with third-party licenses

Cache Statistics:
  Hits: 33
  Misses: 8502
  Invalidated: 0
  Hit Rate: 0.4%
  Cached Files: 8500
Extracting LICENSE files from dependencies...
  Scanning project: /__w/cuvs/cuvs
  Found 34 license file(s)
  Found 34 total LICENSE files
  Found 23 unique LICENSE contents
============================================================
Machine-friendly JSON output written to: c/build/install/licenses.json
Fetching license Apache-2.0 from SPDX API...
Cached license Apache-2.0 to /__w/cuvs/cuvs/infrequent_licenses/Apache-2.0.txt
User-friendly text output written to: c/build/install/LICENSE

@msarahan
Copy link
Contributor

/ok to test 91e8c19

@robertmaynard
Copy link
Contributor Author

/merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature request New feature or request non-breaking Introduces a non-breaking change

Development

Successfully merging this pull request may close these issues.

2 participants