-
Notifications
You must be signed in to change notification settings - Fork 93
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
scancode-toolkit update breaks Docker pipelines #438
Comments
* Adapt code from previous version of scancode-toolkit for use in the debian pipeline Signed-off-by: Jono Yang <jyang@nexb.com>
I brought in the code for
|
Skipping the license detection part allows the docker pipeline to complete, but we get a lot of errors: This error shows up for the codebase resource created for the binaries in
This error shows up on copyright files from
|
Running |
* Update DiscoveredPackage.create_from_data to create packages without a version Signed-off-by: Jono Yang <jyang@nexb.com>
* Adapt code from previous version of scancode-toolkit for use in the debian pipeline Signed-off-by: Jono Yang <jyang@nexb.com>
* Update DiscoveredPackage.create_from_data to create packages without a version Signed-off-by: Jono Yang <jyang@nexb.com>
* Remove step for scanning application packages Signed-off-by: Jono Yang <jyang@nexb.com>
* Check for existence of installed_file attribute before using it Signed-off-by: Jono Yang <jyang@nexb.com>
* Ensure both installed_file and codebase_resource have the same checksum field before comparing them Signed-off-by: Jono Yang <jyang@nexb.com>
* Update mappings_keys_by_fieldname * Look for package data in package_data field instead of packages in save_scan_package_results Signed-off-by: Jono Yang <jyang@nexb.com>
* Move get_installed_packages to rootfs.py * Use get_package_data instead of get_package_info * Rename all instances of packages to package_data when scanning for application packages * Update test docker images and test results * Add test for basic rootfs Signed-off-by: Jono Yang <jyang@nexb.com>
Signed-off-by: Jono Yang <jyang@nexb.com>
Signed-off-by: Jono Yang <jyang@nexb.com>
* Update expected test results * Remove old ubuntu.tar Signed-off-by: Jono Yang <jyang@nexb.com>
Signed-off-by: Jono Yang <jyang@nexb.com>
Signed-off-by: Jono Yang <jyang@nexb.com>
Signed-off-by: Jono Yang <jyang@nexb.com>
* Add package_uid to test package data * Update expected test result Signed-off-by: Jono Yang <jyang@nexb.com>
* Add package_uid to test package data * Update expected test result Signed-off-by: Jono Yang <jyang@nexb.com>
Signed-off-by: Jono Yang <jyang@nexb.com>
Signed-off-by: Jono Yang <jyang@nexb.com>31.0.0rc1
Signed-off-by: Jono Yang <jyang@nexb.com>
* In the LoadInventory pipeline, create the DiscoveredPackages from a scan before creating the CodebaseResources Signed-off-by: Jono Yang <jyang@nexb.com>
* Upgrade scancode-toolkit to latest beta release #411 Signed-off-by: Thomas Druez <tdruez@nexb.com> * Add a test class to regen test data #411 Signed-off-by: Thomas Druez <tdruez@nexb.com> * Upgrade container_inspector to latest 31.0.0 version #411 Signed-off-by: Thomas Druez <tdruez@nexb.com> * Handle new scan format in scancode pipes #411 Signed-off-by: Jono Yang <jyang@nexb.com> * Handle package_uids for DiscoveredPackages #411 * Remove create_discovered_packages2 and create_codebase_resources2 Signed-off-by: Jono Yang <jyang@nexb.com> * Update deprecated code #411 * Normalize package_uids before comparing results in tests * Update expected test results Signed-off-by: Jono Yang <jyang@nexb.com> * Regenerate asgiref 3.3.0 test data #411 * Mark ProjectCodebase tests with expectedFailure * We will revisit ProjectCodebase and update it to fit our current models Signed-off-by: Jono Yang <jyang@nexb.com> * Add asgiref-3.3.0_scancode_scan.json #411 * We are using a scancode scan results for tests since asgiref-3.3.0_scan.json is not exactly the same format as scancode's json output Signed-off-by: Jono Yang <jyang@nexb.com> * Add asgiref-3.3.0_walk_test_fixtures.json #411 * Update regen_test_data.py to generate asgiref-3.3.0_walk_test_fixtures.json Signed-off-by: Jono Yang <jyang@nexb.com> * Signed-off-by: Jono Yang <jyang@nexb.com> * Update make_results_summary() #411 * No need to explicity get license_clarity_score in make_results_summary() * Update expected test results Signed-off-by: Jono Yang <jyang@nexb.com> * Exclude system_environment from diff #411 * Add .vscode to .gitignore Signed-off-by: Jono Yang <jyang@nexb.com> * Upgrade scancode-toolkit and extractcode to latest version #411 Signed-off-by: Thomas Druez <tdruez@nexb.com> * Update package_getter #434 #438 * Adapt code from previous version of scancode-toolkit for use in the debian pipeline Signed-off-by: Jono Yang <jyang@nexb.com> * Allow packages to be created without versions #438 * Update DiscoveredPackage.create_from_data to create packages without a version Signed-off-by: Jono Yang <jyang@nexb.com> * Update expected test results Signed-off-by: Jono Yang <jyang@nexb.com> * Report DiscoveredPackage correctly in summary #411 * Ensure that DiscoveredPackages are reported one time in the scan_package pipeline summary * Add test to check key_file_packages field in the summary output Signed-off-by: Jono Yang <jyang@nexb.com> * Add test for docker pipeline for alpine #411 Signed-off-by: Jono Yang <jyang@nexb.com> * Add docker pipeline test for rpm images #411 Signed-off-by: Jono Yang <jyang@nexb.com> * Track package_uids in make_results_summary #435 * Avoid checking if package_data dictionary is already in the key_files_packages list * Keep track of package_uids instead Signed-off-by: Jono Yang <jyang@nexb.com> * Add truncated ubuntu docker image for testing #435 Signed-off-by: Jono Yang <jyang@nexb.com> * Bump scancode and commoncode versions #435 Signed-off-by: Jono Yang <jyang@nexb.com> * Update docker pipeline #435 * We now run scancode-toolkit on the docker image resources using the new --system-package option * This gives us the installed system packages in the returned scan * We use the scan to create the DiscoveredPackages and CodebaseResources * The rest of the pipeline is unchanged Signed-off-by: Jono Yang <jyang@nexb.com> * Fix code validity #411 Signed-off-by: Thomas Druez <tdruez@nexb.com> * Simplify the filtering of key_files_packages using a QuerySet #411 Signed-off-by: Thomas Druez <tdruez@nexb.com> * Remove copied code from docker.py #411 #435 * Create Docker pipeline from combining the rootfs pipeline and scan_package pipeline Signed-off-by: Jono Yang <jyang@nexb.com> * Update alpine test image and results #411 #435 * TODO: create smaller test images for ubuntu and redhat docker image tests Signed-off-by: Jono Yang <jyang@nexb.com> * Properly create multiple package instances #411 * Do not attempt to combine multiple instances of the same package * Store package_uid in extra data by itself * Add test for multiple package instances Signed-off-by: Jono Yang <jyang@nexb.com> * Sort packages in JSON output by type and name #411 * Normalize package_uid in extra_data fields Signed-off-by: Jono Yang <jyang@nexb.com> * Get file info and packages in initial scan #438 * Remove step for scanning application packages Signed-off-by: Jono Yang <jyang@nexb.com> * Revert changes to docker pipes and pipeline #438 * Check for existence of installed_file attribute before using it Signed-off-by: Jono Yang <jyang@nexb.com> * Use generic package_getter for all distros #438 * Ensure both installed_file and codebase_resource have the same checksum field before comparing them Signed-off-by: Jono Yang <jyang@nexb.com> * Use get_path() with strip_root to get paths #438 * Update mappings_keys_by_fieldname * Look for package data in package_data field instead of packages in save_scan_package_results Signed-off-by: Jono Yang <jyang@nexb.com> * Remove distro specific pipes #438 * Move get_installed_packages to rootfs.py * Use get_package_data instead of get_package_info * Rename all instances of packages to package_data when scanning for application packages * Update test docker images and test results * Add test for basic rootfs Signed-off-by: Jono Yang <jyang@nexb.com> * Use list comprehension for key_file_packages #438 Signed-off-by: Jono Yang <jyang@nexb.com> * Add package_uid field to DiscoveredPackage #411 * Update expected test results Signed-off-by: Jono Yang <jyang@nexb.com> * Add test docker image for Ubuntu #438 * Update expected test results * Remove old ubuntu.tar Signed-off-by: Jono Yang <jyang@nexb.com> * Update formatting #411 #438 Signed-off-by: Jono Yang <jyang@nexb.com> * Use smaller rpm docker image for testing #438 Signed-off-by: Jono Yang <jyang@nexb.com> * Replace ubuntu docker test image #438 Signed-off-by: Jono Yang <jyang@nexb.com> * Use purl data in update_or_create_packages #438 * Add package_uid to test package data * Update expected test result Signed-off-by: Jono Yang <jyang@nexb.com> * Bump scancode version to v31.0.0rc1 #438 #411 Signed-off-by: Jono Yang <jyang@nexb.com> * Consider all PURL fields when ordering Packages #411 #438 Signed-off-by: Jono Yang <jyang@nexb.com> * Create Packages before Resources #411 #438 * In the LoadInventory pipeline, create the DiscoveredPackages from a scan before creating the CodebaseResources Signed-off-by: Jono Yang <jyang@nexb.com> * Add test for load_inventory pipeline #411 Signed-off-by: Jono Yang <jyang@nexb.com> * Code cleanups and formatting #411 Signed-off-by: Thomas Druez <tdruez@nexb.com> * Upgrade dependencies #411 Signed-off-by: Thomas Druez <tdruez@nexb.com> * Refactor create_inventory_from_scan to remove duplicated code #411 Signed-off-by: Thomas Druez <tdruez@nexb.com> * Add changelog entry #411 Signed-off-by: Thomas Druez <tdruez@nexb.com> Co-authored-by: Thomas Druez <tdruez@nexb.com>
Several package related functions used by the docker pipeline have been removed in the latest version of scancode-toolkit. Debian-related package handling, especially code that deals with handling the status files and such, work slightly differently than the way it used to. Functions that the pipelines relied on, like
get_installed_packages
, have been removed.The text was updated successfully, but these errors were encountered: