Skip to content

Commit

Permalink
Merge pull request #245 from garethahealy/sonarqube-webhook
Browse files Browse the repository at this point in the history
Added jenkins url param for webhook
  • Loading branch information
pabrahamsson authored Aug 16, 2019
2 parents dcaac2c + 4e0c410 commit 50b0f43
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 4 deletions.
2 changes: 2 additions & 0 deletions sonarqube/.applier/group_vars/seed-hosts.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
namespace: sonarqube
app_name: sonarqube
jenkins_url: http://jenkins:80

build:
NAME: "{{ app_name }}"
Expand All @@ -14,6 +15,7 @@ postgresql:
POSTGRESQL_USER: sonar
deploy:
POSTGRES_DATABASE_NAME: "{{ postgresql.POSTGRESQL_DATABASE }}"
JENKINS_URL: "{{ jenkins_url }}"

openshift_cluster_content:
- object: Environment Setup
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ objects:
- /bin/sh
- -c
- sleep 30 && curl http://admin:admin@sonarqube:9000/api/webhooks/create
-X POST -d "name=jenkins&url=http://jenkins/sonarqube-webhook/"
-X POST -d "name=jenkins&url=${JENKINS_URL}/sonarqube-webhook/"
containerName: sonarqube
failurePolicy: Abort
type: Recreate
Expand Down Expand Up @@ -197,9 +197,6 @@ objects:
deploymentconfig: sonarqube
sessionAffinity: None
type: ClusterIP
status:
loadBalancer: {}
status: {}
parameters:
- description: Database name for the Posgres Database to be used by Sonarqube
displayName: Postgres database name
Expand Down Expand Up @@ -272,3 +269,7 @@ parameters:
description: The ContextFactory implementation to be used when communicating with the LDAP server
displayName: LDAP Context Factory
value: com.sun.jndi.ldap.LdapCtxFactory
- name: JENKINS_URL
description: The Jenkins URL used for the webhook
displayName: Jenkins URL
value: http://jenkins

0 comments on commit 50b0f43

Please sign in to comment.