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

[DRAFT] Fix some incomplete metadata issue partially #46

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

KAWAHARA-souta
Copy link
Contributor

When data cannot be retrieved from immudb, or when data for package is
missing, if the --rpm-package option was used to specify an rpm package,
I made it so that the rpm package is used to supplement the information.

The following issues have been partially fixed.

And the following issues have been fixed.

Note1:
This MR nees to merge #39 .
Please merge that MR first.

Note2:
This MR may conflict with other MRs.
(At least, it will definitely conflict with #45.)
I will resolve the conflicts after #45 is merged, so please let me know when that happens.
Also, alma_sbom.py has become bloated, and there have been many conflicts in the past. We may need to reconsider its structure.

This patch allows users to create a package SBOM by specifying the
RPM package itself without having to calculate a hash value.

This also allows the SBOM field to be extended since the package
information can be referenced within alma_sbom.
@KAWAHARA-souta
Copy link
Contributor Author

From the issue #48 , I discovered a difference in NEVRA processing when api_ver == '0.1'.
This MR was implemented without considering the differences in NEVRA processing based on api_ver, so when api_ver == '0.1', there's a possibility that it will always incorrectly determine that required_data is insufficient.
Therefore, please do not merge this MR.
Also, I will mark this MR as draft for now.

@KAWAHARA-souta KAWAHARA-souta changed the title Fix some incomplete metadata issue partially [DRAFT] Fix some incomplete metadata issue partially Sep 12, 2024
@KAWAHARA-souta KAWAHARA-souta marked this pull request as draft September 12, 2024 08:28
When data cannot be retrieved from immudb, or when data for package is
missing, if the --rpm-package option was used to specify an rpm package,
I made it so that the rpm package is used to supplement the information.

The following issues have been partially fixed.
 - AlmaLinux#42
 - AlmaLinux#44

And the following issues have been fixed.
 - AlmaLinux#26
Apply common.check_required_data for _add_package_source_info()
and _add_package_build_info().

This patch fixes the following issues.
 - AlmaLinux#47
@KAWAHARA-souta
Copy link
Contributor Author

KAWAHARA-souta commented Sep 18, 2024

9a863f7 fixes issue #47 .
b0dc04e fixes issue #48 .

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.

1 participant