Merge ~pwlars/snappy-hwe-snaps/+git/jenkins-jobs:add-deploy-creds into ~snappy-hwe-team/snappy-hwe-snaps/+git/jenkins-jobs:master

Proposed by Paul Larson
Status: Merged
Approved by: Simon Fels
Approved revision: c8c80ac67ed99779356e0a5d87d1a4baba01d60e
Merged at revision: b1351ef59751a90b4067223539eba67044a0dea9
Proposed branch: ~pwlars/snappy-hwe-snaps/+git/jenkins-jobs:add-deploy-creds
Merge into: ~snappy-hwe-team/snappy-hwe-snaps/+git/jenkins-jobs:master
Diff against target: 41 lines (+8/-0)
3 files modified
jobs/infrastructure/credentials-2-launchpad-plugin.sh (+4/-0)
jobs/infrastructure/credentials-2-launchpad-plugin.yaml (+3/-0)
local.yaml (+1/-0)
Reviewer Review Type Date Requested Status
Simon Fels Approve
System Enablement Bot continuous-integration Approve
Review via email: mp+330687@code.launchpad.net

Description of the change

This adds some extra lines to the config to allow us to pass in credentials for the -jb bot user to deploy jobs

To post a comment you must log in.
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Simon Fels (morphis) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/jobs/infrastructure/credentials-2-launchpad-plugin.sh b/jobs/infrastructure/credentials-2-launchpad-plugin.sh
index d47164f..b963c69 100644
--- a/jobs/infrastructure/credentials-2-launchpad-plugin.sh
+++ b/jobs/infrastructure/credentials-2-launchpad-plugin.sh
@@ -47,6 +47,10 @@ DEBFULLNAME:
47jenkins_user: "{bot_username}"47jenkins_user: "{bot_username}"
48jenkins_password: "${{jenkins_api_token}}"48jenkins_password: "${{jenkins_api_token}}"
4949
50#These credentials are only used for actually deploying jenkins jobs
51jenkins_deploy_user: "{deploy_bot_username}"
52jenkins_deploy_password: "${{jenkins_deploy_api_token}}"
53
50#Actual URL of your jenkins (e.g. the jenkins backend URL)54#Actual URL of your jenkins (e.g. the jenkins backend URL)
51jenkins_url: "{backend_url}"55jenkins_url: "{backend_url}"
5256
diff --git a/jobs/infrastructure/credentials-2-launchpad-plugin.yaml b/jobs/infrastructure/credentials-2-launchpad-plugin.yaml
index 0d8ca61..95eb8e3 100644
--- a/jobs/infrastructure/credentials-2-launchpad-plugin.yaml
+++ b/jobs/infrastructure/credentials-2-launchpad-plugin.yaml
@@ -12,6 +12,9 @@
12 - password:12 - password:
13 name: jenkins_api_token13 name: jenkins_api_token
14 description: Jenkins API key of the "{bot_username}" account14 description: Jenkins API key of the "{bot_username}" account
15 - password:
16 name: jenkins_deploy_api_token
17 description: Jenkins API key of the "{deploy_bot_username}" account
15 axes:18 axes:
16 - axis:19 - axis:
17 type: slave20 type: slave
diff --git a/local.yaml b/local.yaml
index eb6ea71..c5b7820 100644
--- a/local.yaml
+++ b/local.yaml
@@ -9,6 +9,7 @@
9 jenkins-instance: localhost9 jenkins-instance: localhost
1010
11 bot_username: system-enablement-ci-bot11 bot_username: system-enablement-ci-bot
12 deploy_bot_username: system-enablement-jb-bot
12 credentials_path: /var/lib/jenkins/.launchpad.credentials13 credentials_path: /var/lib/jenkins/.launchpad.credentials
13 allowed_users: "canonical-system-enablement"14 allowed_users: "canonical-system-enablement"
14 backend_url: http://localhost:8080/15 backend_url: http://localhost:8080/

Subscribers

People subscribed via source and target branches

to all changes: