adr for external references #1200
Draft
+924
−0
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
JimFuller-RedHat
force-pushed
the
adr-external-references
branch
2 times, most recently
from
a056048 to
c41ecff
Compare
JimFuller-RedHat
force-pushed
the
adr-external-references
branch
6 times, most recently
from
a6867e7 to
0c490d6
Compare
chirino
reviewed
Jan 27, 2025
JimFuller-RedHat
force-pushed
the
adr-external-references
branch
from
0c490d6 to
6082c7b
Compare
jcrossley3
reviewed
Jan 29, 2025
JimFuller-RedHat
force-pushed
the
adr-external-references
branch
2 times, most recently
from
3c2d0ce to
2f5f624
Compare
JimFuller-RedHat
force-pushed
the
adr-external-references
branch
13 times, most recently
from
5a4a688 to
691197b
Compare
JimFuller-RedHat
force-pushed
the
adr-external-references
branch
3 times, most recently
from
22638f2 to
1714c73
Compare
JimFuller-RedHat
force-pushed
the
adr-external-references
branch
from
1714c73 to
d429993
Compare
|
Now that we have some example record entries written down, we should think about the types of queries that the analysis layer will do to find the related nodes for these 3 external ref types. That may reveal the need for some additional indexed fields. |
|
I can flesh out the analysis graph queries though they will be similar to what we do today aka search sbom_node table then do petgraph query ... for external node we would be external_sbom_package_node then petgraph query - observation - the ADR is not a full blown design spec eg. we are going to find out some things out when we go implement. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Preview: https://github.com/trustification/trustify/blob/adr-external-references/docs/adrs/00003-external-references.md