Skip to content

feat(hooks,api,cli): add analyze endpoint#6236

Merged
sguiheux merged 41 commits into
masterfrom
userGpgKeys
Aug 23, 2022
Merged

feat(hooks,api,cli): add analyze endpoint#6236
sguiheux merged 41 commits into
masterfrom
userGpgKeys

Conversation

@sguiheux

@sguiheux sguiheux commented Jul 13, 2022 •
edited
Loading

Copy link
Copy Markdown
Collaborator
  1. Allow user to add their gpg keys
  2. Hooks call new endpoints analyze
  3. CDS API save analyze history (50 analyzes max per repository)
  4. The analyze endpoint is only implemented for bitbucketserver and gitea (IT)
  5. The analyze endpoint start an Operation that will verify commit for bitbucket, GetCommit is enough for Gitea
  6. New API Go Routine that check all Operations InProgress : Waiting the commit check, check user right, get all files (name and path) in .cds folder
  7. Analyze handler are tested through IT

@ovh ovh deleted a comment from ovh-cds Jul 19, 2022
@ovh ovh deleted a comment from sonarqubecloud Bot Jul 19, 2022
@ovh ovh deleted a comment from ovh-cds Jul 19, 2022
@ovh ovh deleted a comment from ovh-cds Jul 19, 2022
@ovh ovh deleted a comment from ovh-cds Jul 19, 2022
@ovh ovh deleted a comment from ovh-cds Jul 19, 2022
@ovh ovh deleted a comment from ovh-cds Jul 22, 2022
@sguiheux sguiheux marked this pull request as ready for review July 22, 2022 12:08
@ovh ovh deleted a comment from ovh-cds Jul 22, 2022
@sonarqubecloud

sonarqubecloud Bot commented Jul 22, 2022

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 4 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

fsamin
fsamin requested changes Aug 22, 2022
View reviewed changes
Comment thread engine/api/repository/dao_vcs_project_repository_analyze.go Outdated
Comment thread engine/api/repository/dao_vcs_project_repository_test.go
Comment thread engine/api/services/http.go Outdated
Comment thread engine/api/user/dao_gpgkey.go Outdated
Comment thread engine/api/user/dao_gpgkey.go
Comment thread engine/repositories/processor_checkout.go Outdated
Comment thread engine/vcs/bitbucketserver/http.go
yesnault
yesnault requested changes Aug 22, 2022
View reviewed changes
Comment thread cli/cdsctl/experimental_project_repository_analyze.go Outdated
Comment thread cli/cdsctl/experimental_project_repository_analyze.go Outdated
Comment thread cli/cdsctl/user_gpg.go Outdated
richardlt
richardlt requested changes Aug 22, 2022
View reviewed changes
Comment thread engine/hooks/tasks.go Outdated
Comment thread engine/vcs/bitbucketserver/client_file.go Outdated
Comment thread sdk/repositories_operation.go Outdated
Comment thread cli/cdsctl/experimental_project_repository.go Outdated
Comment thread cli/cdsctl/user_gpg.go Outdated
Comment thread engine/api/repository/dao_vcs_project_repository_analyze.go Outdated
Comment thread engine/api/repository/dao_vcs_project_repository_analyze.go Outdated
Comment thread engine/api/user/dao_gpgkey.go Outdated
Comment thread engine/api/user/dao_gpgkey.go Outdated
Comment thread engine/api/v2_repository_analyze.go
@ovh ovh deleted a comment from ovh-cds Aug 23, 2022
@ovh ovh deleted a comment from ovh-cds Aug 23, 2022
@ovh ovh deleted a comment from ovh-cds Aug 23, 2022
@ovh ovh deleted a comment from ovh-cds Aug 23, 2022
@ovh-cds

ovh-cds commented Aug 23, 2022

Copy link
Copy Markdown
Collaborator

CDS Report build-all-cds#17771.0 ✘

  • Build
    • Build and Package All ✔
    • Build Debpacker ✔
    • Build OS-Ansible-Inventory ✔
    • Build SMTPMock ✔
    • Lint UI Translate ✔
    • Test CLI ✔
    • Test Engine ✔
    • Test SDK ✔
    • Test UI ✔
  • Package
    • Docker ✔
  • Integration
    • Docker Compose Tests ✔
    • Standalone Tests ✘

@sonarqubecloud

sonarqubecloud Bot commented Aug 23, 2022

Copy link
Copy Markdown

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

43.8% 43.8% Coverage
8.1% 8.1% Duplication

@sguiheux sguiheux merged commit f41c6c5 into master Aug 23, 2022
@sguiheux sguiheux deleted the userGpgKeys branch August 24, 2022 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yesnault yesnault yesnault approved these changes

@fsamin fsamin fsamin approved these changes

@richardlt richardlt richardlt approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@sguiheux @ovh-cds @yesnault @fsamin @richardlt