Merge ~dannf/charms/+source/scalebot-jenkins:rename-mkjobs into ~ce-hyperscale/charms/+source/scalebot-jenkins:master

Proposed by dann frazier
Status: Merged
Approved by: Andrew Cloke
Approved revision: 1c513851afb964620ff27f843f139c62022cf64f
Merged at revision: 1c513851afb964620ff27f843f139c62022cf64f
Proposed branch: ~dannf/charms/+source/scalebot-jenkins:rename-mkjobs
Merge into: ~ce-hyperscale/charms/+source/scalebot-jenkins:master
Diff against target: 38 lines (+6/-6)
2 files modified
README.md (+3/-3)
bin/pull-and-reload-jobs (+3/-3)
Reviewer Review Type Date Requested Status
Andrew Cloke Approve
Review via email: mp+374042@code.launchpad.net

Description of the change

Thought to do this when Alexandre wanted to update the python-maas library after his release job was added.

To post a comment you must log in.
Revision history for this message
Andrew Cloke (andrew-cloke) wrote :

Agreed. The renaming makes sense.

Revision history for this message
Andrew Cloke (andrew-cloke) wrote :

Agreed. The renaming makes sense.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/README.md b/README.md
index 4e8cfd6..43e5e2f 100644
--- a/README.md
+++ b/README.md
@@ -67,7 +67,7 @@ labs/<labname>/clouds.yaml
67labs/<labname>/credentials.yaml67labs/<labname>/credentials.yaml
68labs/<labname>/model-defaults.yaml68labs/<labname>/model-defaults.yaml
69scalebot.d/init69scalebot.d/init
70scalebot.d/mkjobs70scalebot.d/refresh
71```71```
7272
73Jobs must be in the YAML format used by [Jenkins Job Builder](https://docs.openstack.org/infra/jenkins-job-builder), as that is the tool used to load them into Jenkins. The clouds.yaml, credentials.yaml, and model-defaults.yaml files are passed directly to Juju when bootsrapping the Lab controller, so see [Juju documentation](http://jujucharms.com/docs) for details on their contents.73Jobs must be in the YAML format used by [Jenkins Job Builder](https://docs.openstack.org/infra/jenkins-job-builder), as that is the tool used to load them into Jenkins. The clouds.yaml, credentials.yaml, and model-defaults.yaml files are passed directly to Juju when bootsrapping the Lab controller, so see [Juju documentation](http://jujucharms.com/docs) for details on their contents.
@@ -88,5 +88,5 @@ In addition to the environment variables set by Jenkins, the following variable
88## scalebot.d/init hook88## scalebot.d/init hook
89If it exists, ```$SCALEBOT_REPO/scalebot.d/init``` will be executed each time the configuration repository is refreshed. You can use this to install any additional dependencies needed for your jobs.89If it exists, ```$SCALEBOT_REPO/scalebot.d/init``` will be executed each time the configuration repository is refreshed. You can use this to install any additional dependencies needed for your jobs.
9090
91## scalebot.d/mkjobs hook91## scalebot.d/refresh hook
92If it exists, ```$SCALEBOT_REPO/scalebot.d/mkjobs``` will be executed after the repository is updated, and before loading jobs. This is useful in case something job yaml files need to be dynamically generated.92If it exists, ```$SCALEBOT_REPO/scalebot.d/refresh``` will be executed after the repository is updated, and before loading jobs. This is useful in case something job yaml files need to be dynamically generated.
diff --git a/bin/pull-and-reload-jobs b/bin/pull-and-reload-jobs
index 8a1775a..330f824 100755
--- a/bin/pull-and-reload-jobs
+++ b/bin/pull-and-reload-jobs
@@ -10,9 +10,9 @@ set -x
10[ -n "$SCALEBOT_REPO" ]10[ -n "$SCALEBOT_REPO" ]
1111
12git -C ${SCALEBOT_REPO} pull --ff-only12git -C ${SCALEBOT_REPO} pull --ff-only
13mkjobs="${SCALEBOT_REPO}/scalebot.d/mkjobs"13refresh="${SCALEBOT_REPO}/scalebot.d/refresh"
14if [ -x "$mkjobs" ]; then14if [ -x "$refresh" ]; then
15 $mkjobs15 $refresh
16fi16fi
17jobsync="${SCALEBOT_HOME}/bin/jobsync.py"17jobsync="${SCALEBOT_HOME}/bin/jobsync.py"
18${jobsync} -j ${SCALEBOT_JOBSET} -s18${jobsync} -j ${SCALEBOT_JOBSET} -s

Subscribers

People subscribed via source and target branches

to all changes: