Skip to content
This repository has been archived by the owner on Feb 5, 2024. It is now read-only.

Gatebreaker #11

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Gatebreaker #11

wants to merge 4 commits into from

Conversation

saitho
Copy link
Contributor

@saitho saitho commented Nov 24, 2019

Resolves #9.

Adds a new option to allow aborting the pipeline when quality gate is not met.
If gatebreaker option is enabled, the step will wait until the analysis report was processed by the server and check the state of the quality gate.

The state of the Quality Gate is fetched via the Web API. I'm using a forked and slightly adjusted version of https://github.com/magicsong/sonargo which is needed to work with the current Sonar version.

Container with binary available at https://hub.docker.com/repository/docker/saitho/drone-sonar-plugin

@roganw
Copy link
Member

roganw commented Feb 6, 2020

Thanks for your pr.
Please fix the conflict before I could merge it.

@saitho
Copy link
Contributor Author

saitho commented Mar 14, 2020

Hey there,

sorry for the late reply. I rebased the branch, but I wasn't able to test it yet...

I'm now using the official repository of sonargo, as my pull request there was merged.
Building the binary locally works fine, but using the Dockerfile it says that it couldn't find the package. :(

@fauzanelka
Copy link

Could you merge this ? @roganw

@ojenikoh
Copy link

Hello,

@roganw Is there any chance of this being merged anytime soon

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature request] Quality gate support
4 participants