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

Atlas support #754

Merged
merged 3 commits into from
Sep 20, 2024
Merged

Atlas support #754

merged 3 commits into from
Sep 20, 2024

Conversation

goldmann
Copy link
Contributor

This adds support for uploading manifests to Atlas. I suggest to read commit messages, these contain all the explanations.

https://issues.redhat.com/browse/SBOMER-141

This adds support for pushing manifests to Atlas. This feature is
guarded using the `atlas-publish` feature flag.

This implementation adds a `AtlasHandler` which uses `AtlasClient` to
connect to the Atlas REST API. It uses a PUT request to publish the
message.

Fixes: SBOMER-141
This triggers the upload after the manifest generation is completed. It
will only publish manifests that are related to a product. Everything
else will be skipped.

Some refactoring to make the code more useful.
@goldmann goldmann requested a review from vibe13 September 19, 2024 20:24
In order to make the PNC REST API integration in the service with
updated Resteasy Reactive we had to drop use of the (provided by the PNC
team) REST client library.

Fortunately writing PNC client was easy.
@goldmann goldmann force-pushed the SBOMER-141-atlas-handler branch from d872c23 to 0023a2f Compare September 19, 2024 20:26
@goldmann goldmann requested a review from dwalluck September 19, 2024 20:31
@goldmann goldmann enabled auto-merge (rebase) September 20, 2024 07:53
@goldmann goldmann disabled auto-merge September 20, 2024 07:54
@goldmann goldmann merged commit ea086e3 into project-ncl:main Sep 20, 2024
7 checks passed
@goldmann goldmann deleted the SBOMER-141-atlas-handler branch September 20, 2024 07:54
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