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.

Sign up for GitHub

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

Jump to bottom

feat(api, sql): worker hooks #5980

Merged
merged 11 commits into from
Oct 26, 2021
Merged

feat(api, sql): worker hooks #5980

merged 11 commits into from
Oct 26, 2021

Conversation

fsamin
Copy link
Member

@fsamin fsamin commented Oct 18, 2021

No description provided.

@ovh ovh deleted a comment from ovh-cds Oct 19, 2021
@ovh ovh deleted a comment from ovh-cds Oct 19, 2021
@ovh ovh deleted a comment from ovh-cds Oct 19, 2021
@ovh ovh deleted a comment from ovh-cds Oct 19, 2021
@ovh ovh deleted a comment from ovh-cds Oct 19, 2021
@ovh ovh deleted a comment from ovh-cds Oct 19, 2021
@fsamin fsamin force-pushed the feat_api_worker_hooks branch from 73a9ed6 to e570210 Compare October 20, 2021 07:20
@ovh ovh deleted a comment from ovh-cds Oct 20, 2021
sguiheux
sguiheux requested changes Oct 20, 2021
View reviewed changes
sdk/worker_hooks.go Outdated Show resolved Hide resolved
yesnault
yesnault requested changes Oct 20, 2021
View reviewed changes
cli/cdsctl/project_integration.go Outdated Show resolved Hide resolved
sdk/worker_hooks.go Outdated Show resolved Hide resolved
sdk/worker_hooks.go Outdated Show resolved Hide resolved
sdk/worker_hooks.go Outdated Show resolved Hide resolved
@ovh ovh deleted a comment from ovh-cds Oct 21, 2021
@ovh ovh deleted a comment from ovh-cds Oct 21, 2021
@ovh ovh deleted a comment from ovh-cds Oct 21, 2021
@ovh-cds
Copy link
Collaborator

ovh-cds commented Oct 21, 2021

CDS Report build-all-cds#16684.2 ✘

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

@ovh-cds
Copy link
Collaborator

ovh-cds commented Oct 21, 2021

CDS Report build-all-cds#16684.3 ✘

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

@ovh-cds
Copy link
Collaborator

ovh-cds commented Oct 21, 2021

CDS Report build-all-cds#16685.0 ✘

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

sguiheux
sguiheux requested changes Oct 22, 2021
View reviewed changes
sdk/cdsclient/client_project_integration.go Outdated
@@ -118,3 +117,20 @@ func (c *client) ProjectIntegrationImport(projectKey string, content io.Reader,
}
return pf, nil
}

func (c *client) ProjectIntegrationWorkerHooksList(projectKey string, integrationName string) ([]sdk.WorkerHookProjectIntegrationModel, error) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

List -> GET

@fsamin
fix: cr
b43c698 
Signed-off-by: francois  samin <[email protected]>
@fsamin fsamin force-pushed the feat_api_worker_hooks branch from 1bf95fb to b43c698 Compare October 25, 2021 07:37
@ovh-cds
Copy link
Collaborator

ovh-cds commented Oct 25, 2021

CDS Report build-all-cds#16692.0 ✘

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

@ovh-cds
Copy link
Collaborator

ovh-cds commented Oct 25, 2021

CDS Report build-all-cds#16692.1 ✘

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

@fsamin
fix: cr
cd61388 
Signed-off-by: francois  samin <[email protected]>
@ovh-cds
Copy link
Collaborator

ovh-cds commented Oct 25, 2021

CDS Report build-all-cds#16693.0 ✘

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

@ovh-cds
Copy link
Collaborator

ovh-cds commented Oct 25, 2021

CDS Report build-all-cds#16693.1 ✘

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

@fsamin
fix: cr
a2e1cd9 
Signed-off-by: francois  samin <[email protected]>
@ovh-cds
Copy link
Collaborator

ovh-cds commented Oct 25, 2021

CDS Report build-all-cds#16694.0 ✘

  • Build
    • Build and Package All ✔
    • Build Debpacker ✔
    • Build OS-Ansible-Inventory ✔
    • Build SMTPMock ✔
    • Lint UI Translate ✔
    • Test CLI ✔
    • Test Engine ✘
    • Test SDK ✔
    • Test SDK Rust ✔
    • Test UI ✔

@sonarqubecloud
Copy link

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 0 Code Smells

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

@sguiheux sguiheux merged commit 4e320d2 into master Oct 26, 2021
@fsamin fsamin deleted the feat_api_worker_hooks branch November 17, 2021 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@richardlt richardlt richardlt approved these changes

@sguiheux sguiheux sguiheux approved these changes

@yesnault yesnault yesnault 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
@fsamin @ovh-cds @yesnault @sguiheux @richardlt