Skip to content

Commit d95cc5f

Browse files
committed
jobs/build: factor out image signing and compose repo import
No functional change. Prep for next patch.
1 parent b94e601 commit d95cc5f

File tree

3 files changed

+29
-30
lines changed

3 files changed

+29
-30
lines changed

jobs/build-arch.Jenkinsfile

Lines changed: 2 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -370,23 +370,10 @@ lock(resource: "build-${params.STREAM}-${basearch}") {
370370
pipeutils.tryWithMessagingCredentials() {
371371
def parallelruns = [:]
372372
parallelruns['Sign Images'] = {
373-
pipeutils.shwrapWithAWSBuildUploadCredentials("""
374-
cosa sign --build=${newBuildID} --arch=${basearch} \
375-
robosignatory --s3 ${s3_stream_dir}/builds \
376-
--aws-config-file \${AWS_BUILD_UPLOAD_CONFIG} \
377-
--extra-fedmsg-keys stream=${params.STREAM} \
378-
--images --gpgkeypath /etc/pki/rpm-gpg \
379-
--fedmsg-conf \${FEDORA_MESSAGING_CONF}
380-
""")
373+
pipeutils.signImages(params.STREAM, newBuildID, basearch, s3_stream_dir)
381374
}
382375
parallelruns['OSTree Import: Compose Repo'] = {
383-
shwrap("""
384-
cosa shell -- \
385-
/usr/lib/coreos-assembler/fedmsg-send-ostree-import-request \
386-
--build=${newBuildID} --arch=${basearch} \
387-
--s3=${s3_stream_dir} --repo=compose \
388-
--fedmsg-conf \${FEDORA_MESSAGING_CONF}
389-
""")
376+
pipeutils.composeRepoImport(newBuildID, basearch, s3_stream_dir)
390377
}
391378
// process this batch
392379
parallel parallelruns

jobs/build.Jenkinsfile

Lines changed: 2 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -459,23 +459,10 @@ lock(resource: "build-${params.STREAM}") {
459459
pipeutils.tryWithMessagingCredentials() {
460460
def parallelruns = [:]
461461
parallelruns['Sign Images'] = {
462-
pipeutils.shwrapWithAWSBuildUploadCredentials("""
463-
cosa sign --build=${newBuildID} --arch=${basearch} \
464-
robosignatory --s3 ${s3_stream_dir}/builds \
465-
--aws-config-file \${AWS_BUILD_UPLOAD_CONFIG} \
466-
--extra-fedmsg-keys stream=${params.STREAM} \
467-
--images --gpgkeypath /etc/pki/rpm-gpg \
468-
--fedmsg-conf \${FEDORA_MESSAGING_CONF}
469-
""")
462+
pipeutils.signImages(params.STREAM, newBuildID, basearch, s3_stream_dir)
470463
}
471464
parallelruns['OSTree Import: Compose Repo'] = {
472-
shwrap("""
473-
cosa shell -- \
474-
/usr/lib/coreos-assembler/fedmsg-send-ostree-import-request \
475-
--build=${newBuildID} --arch=${basearch} \
476-
--s3=${s3_stream_dir} --repo=compose \
477-
--fedmsg-conf \${FEDORA_MESSAGING_CONF}
478-
""")
465+
pipeutils.composeRepoImport(newBuildID, basearch, s3_stream_dir)
479466
}
480467
// process this batch
481468
parallel parallelruns

utils.groovy

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -657,6 +657,31 @@ def AWSBuildUploadCredentialExists() {
657657
return utils.credentialsExist(creds)
658658
}
659659

660+
// Calls `cosa sign robosignatory --images ...`. Assumes to have access to the
661+
// messaging credentials.
662+
def signImages(stream, version, basearch, s3_stream_dir) {
663+
shwrapWithAWSBuildUploadCredentials("""
664+
cosa sign --build=${version} --arch=${basearch} \
665+
robosignatory --s3 ${s3_stream_dir}/builds \
666+
--aws-config-file \${AWS_BUILD_UPLOAD_CONFIG} \
667+
--extra-fedmsg-keys stream=${stream} \
668+
--images --gpgkeypath /etc/pki/rpm-gpg \
669+
--fedmsg-conf \${FEDORA_MESSAGING_CONF}
670+
""")
671+
}
672+
673+
// Requests OSTree commit to be imported into the compose repo. Assumes to have
674+
// access to the messaging credentials.
675+
def composeRepoImport(version, basearch, s3_stream_dir) {
676+
shwrap("""
677+
cosa shell -- \
678+
/usr/lib/coreos-assembler/fedmsg-send-ostree-import-request \
679+
--build=${version} --arch=${basearch} \
680+
--s3=${s3_stream_dir} --repo=compose \
681+
--fedmsg-conf \${FEDORA_MESSAGING_CONF}
682+
""")
683+
}
684+
660685
// Grabs the jenkins.io/emoji-prefix annotation from the slack-api-token
661686
def getSlackEmojiPrefix() {
662687
def emoji = shwrapCapture("""

0 commit comments

Comments
 (0)