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

Ent - PkgEqualsID #1473

Merged
merged 1 commit into from
Nov 9, 2023
Merged

Conversation

desmax74
Copy link
Contributor

@desmax74 desmax74 commented Nov 7, 2023

Description of the PR

Ent backend

  • IngestPkgEqualID (with bulk) implementations
  • IngestPkgEqual, removed
  • changes consistently all of the tests involved

Refers to #1198

PR Checklist

  • All commits have a Developer Certificate of Origin (DCO) -- they are generated using -s flag to git commit.
  • All new changes are covered by tests
  • If GraphQL schema is changed, make generate has been run
  • If collectsub protobuf has been changed, make proto has been run
  • All CI checks are passing (tests and formatting)
  • All dependent PRs have already been merged

@desmax74 desmax74 force-pushed the 1116-pkgequals-ID branch 2 times, most recently from 94b1cc3 to 3369c9e Compare November 7, 2023 21:18
Signed-off-by: desmax74 <mdessi@redhat.com>
@desmax74 desmax74 marked this pull request as ready for review November 7, 2023 21:29
@jeffmendoza jeffmendoza merged commit 3966ba9 into guacsec:1116-return-ID Nov 9, 2023
8 checks passed
desmax74 added a commit to desmax74/guacsec-guac that referenced this pull request Nov 14, 2023
Signed-off-by: desmax74 <mdessi@redhat.com>
pxp928 pushed a commit that referenced this pull request Nov 16, 2023
Signed-off-by: desmax74 <mdessi@redhat.com>
pxp928 pushed a commit that referenced this pull request Nov 21, 2023
Signed-off-by: desmax74 <mdessi@redhat.com>
pxp928 pushed a commit that referenced this pull request Nov 30, 2023
Signed-off-by: desmax74 <mdessi@redhat.com>
pxp928 pushed a commit that referenced this pull request Nov 30, 2023
Signed-off-by: desmax74 <mdessi@redhat.com>
Signed-off-by: pxp928 <parth.psu@gmail.com>
kodiakhq bot pushed a commit that referenced this pull request Nov 30, 2023
… main) (#1542)

* Backend interface - Methods for returning 'string' (#1285)

Signed-off-by: mrizzi <mrizzi@redhat.com>
Signed-off-by: pxp928 <parth.psu@gmail.com>

* Enable GitHub CI tests (#1296)

* Enable GitHub CI tests

Signed-off-by: mrizzi <mrizzi@redhat.com>

* Mocks backend updated

Signed-off-by: mrizzi <mrizzi@redhat.com>

* Ent - Bump v0.12.4-0.20230918073025-797534a0d1ca (#1283)

* Ent - Bump v0.12.4-0.20230918073025-797534a0d1ca

Signed-off-by: mrizzi <mrizzi@redhat.com>

* Ent - Pin postgres image to hash

Signed-off-by: mrizzi <mrizzi@redhat.com>

---------

Signed-off-by: mrizzi <mrizzi@redhat.com>

---------

Signed-off-by: mrizzi <mrizzi@redhat.com>
Signed-off-by: pxp928 <parth.psu@gmail.com>

* Ent - IngestArtifactID,IngestArtifactIDs implementations (#1300)

Signed-off-by: mrizzi <mrizzi@redhat.com>
Signed-off-by: pxp928 <parth.psu@gmail.com>

* Ent - IngestBuilderID, IngestPackageID, IngestSourceID, IngestVulnerabilityID (#1329)

* 1116-return-ID

Signed-off-by: desmax74 <mdessi@redhat.com>

* Ent - upsertPackageIDDoNothing vs upsertPackageIDIgnore (#1270)

* Ent - upsertPackageIDDoNothing vs upsertPackageIDIgnore

Signed-off-by: mrizzi <mrizzi@redhat.com>

* Ent - final upsertPackage implementation

Signed-off-by: mrizzi <mrizzi@redhat.com>

---------

Signed-off-by: mrizzi <mrizzi@redhat.com>
Signed-off-by: desmax74 <mdessi@redhat.com>

* Ent - upsertPackage: enhanced ID management

Signed-off-by: mrizzi <mrizzi@redhat.com>
Signed-off-by: desmax74 <mdessi@redhat.com>

* 1116-return-ID

Signed-off-by: desmax74 <mdessi@redhat.com>

* Ent - Packages: subpath query issue  (#1330)

* Ent - Packages: subpath query issue reproducer

Signed-off-by: mrizzi <mrizzi@redhat.com>

* Ent - Packages: subpath query issue fix

Signed-off-by: mrizzi <mrizzi@redhat.com>

---------

Signed-off-by: mrizzi <mrizzi@redhat.com>
Signed-off-by: desmax74 <mdessi@redhat.com>

* 1116-return-ID

Signed-off-by: desmax74 <mdessi@redhat.com>

* 1116-return-ID

Signed-off-by: desmax74 <mdessi@redhat.com>

* 1116-return-ID

Signed-off-by: desmax74 <mdessi@redhat.com>

* Ent - TestIngestVulnerabilities changes to support ID

Signed-off-by: mrizzi <mrizzi@redhat.com>
Signed-off-by: desmax74 <mdessi@redhat.com>

* 1116-return-ID

Signed-off-by: desmax74 <mdessi@redhat.com>

* Ent - Node: added SourceName management

Signed-off-by: mrizzi <mrizzi@redhat.com>
Signed-off-by: desmax74 <mdessi@redhat.com>

* 1116-return-ID

Signed-off-by: desmax74 <mdessi@redhat.com>

* 1116-return-ID

Signed-off-by: desmax74 <mdessi@redhat.com>

* 1116-return-ID

Signed-off-by: desmax74 <mdessi@redhat.com>

* 1116-return-ID

Signed-off-by: desmax74 <mdessi@redhat.com>

* 1116-return-ID

Signed-off-by: desmax74 <mdessi@redhat.com>

---------

Signed-off-by: desmax74 <mdessi@redhat.com>
Signed-off-by: mrizzi <mrizzi@redhat.com>
Co-authored-by: Marco Rizzi <mrizzi@users.noreply.github.com>
Co-authored-by: mrizzi <mrizzi@redhat.com>
Signed-off-by: pxp928 <parth.psu@gmail.com>

* Ent - IngestHasSBOMID (#1470)

Signed-off-by: desmax74 <mdessi@redhat.com>
Signed-off-by: pxp928 <parth.psu@gmail.com>

* Ent - ScorecardID (#1469)

Signed-off-by: desmax74 <mdessi@redhat.com>
Signed-off-by: pxp928 <parth.psu@gmail.com>

* Ent - PkgEqualsID (#1473)

Signed-off-by: desmax74 <mdessi@redhat.com>
Signed-off-by: pxp928 <parth.psu@gmail.com>

* update ent IngestHasSbomID and IngestHasSBOMIDs with the new includes field

Signed-off-by: pxp928 <parth.psu@gmail.com>

* Feature/1116 arango noun mutation update to return only IDs (#1511)

* change artifact mutation to return only ID

Signed-off-by: pxp928 <parth.psu@gmail.com>

* change builder and license to return only ID during ingestion

Signed-off-by: pxp928 <parth.psu@gmail.com>

* update package to return flat struct of IDs

Signed-off-by: pxp928 <parth.psu@gmail.com>

* update source and vulnerability to return mutation ID

Signed-off-by: pxp928 <parth.psu@gmail.com>

* update ENT for vulnerability and package mutation IDs

Signed-off-by: pxp928 <parth.psu@gmail.com>

* update ENT for source mutation IDs

Signed-off-by: pxp928 <parth.psu@gmail.com>

* update tests for arango and ent

Signed-off-by: pxp928 <parth.psu@gmail.com>

---------

Signed-off-by: pxp928 <parth.psu@gmail.com>

* Update all resolvers for Return ID (#1514)

* Update resolvers to use ID backend methods

Signed-off-by: Jeff Mendoza <jlm@jlm.name>

* Remove old methods from backend interface, update pkg/src/vuln resolvers.

Signed-off-by: Jeff Mendoza <jlm@jlm.name>

---------

Signed-off-by: Jeff Mendoza <jlm@jlm.name>
Signed-off-by: pxp928 <parth.psu@gmail.com>

* Update keyvalue backend to return ID (#1524)

* Update keyvalue backend to return ID

Signed-off-by: Jeff Mendoza <jlm@jlm.name>

* Lint gosimple

Signed-off-by: Jeff Mendoza <jlm@jlm.name>

* Regen backend mock

Signed-off-by: Jeff Mendoza <jlm@jlm.name>

---------

Signed-off-by: Jeff Mendoza <jlm@jlm.name>
Signed-off-by: pxp928 <parth.psu@gmail.com>

* Feature/1116 arango verb mutation (#1529)

Signed-off-by: pxp928 <parth.psu@gmail.com>

* Complete Ent Return -ID (#1536)

* Updated ent with ID

Signed-off-by: Jeff Mendoza <jlm@jlm.name>

* Update ent tests for ID.

Signed-off-by: Jeff Mendoza <jlm@jlm.name>

---------

Signed-off-by: Jeff Mendoza <jlm@jlm.name>
Signed-off-by: pxp928 <parth.psu@gmail.com>

* change mutations back to original name (#1541)

Signed-off-by: pxp928 <parth.psu@gmail.com>

* remove branch from ci workflow

Signed-off-by: pxp928 <parth.psu@gmail.com>

---------

Signed-off-by: mrizzi <mrizzi@redhat.com>
Signed-off-by: pxp928 <parth.psu@gmail.com>
Signed-off-by: desmax74 <mdessi@redhat.com>
Signed-off-by: Jeff Mendoza <jlm@jlm.name>
Co-authored-by: Marco Rizzi <mrizzi@users.noreply.github.com>
Co-authored-by: Massimiliano Dessì - (Fast Chauffeur) <desmax74@users.noreply.github.com>
Co-authored-by: mrizzi <mrizzi@redhat.com>
Co-authored-by: Jeff Mendoza <jlm@jlm.name>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants