Request: eval context to help DRY up jobs

[iautom8things]

Could you please evaluate the context parameter like here: https://github.com/cloudfoundry-community/slack-notification-resource/blob/c1686fb4b6ed41a4a28d6de497889e5fae7e2cfd/out#L99

This will help D.R.Y. up job definitions where there are several build contexts and statuses used.

[jtarchie]

Are you able to try out the feature? Use the test docker image jtarchie/pr#test.

[iautom8things]

I'll try this out today. Currently trying to figure out why my mbp is stuck in a reboot loop with security update 10.12.6 o.O

[iautom8things]

I'm not sure what I'm doing wrong:

I changed:

resource_types:
  - name: pull-request
     type: docker-image
     source:
       repository: jtarchie/pr

to

resource_types:
  - name: pull-request
     type: docker-image
     source:
       repository: jtarchie/pr:test

This results in resource script '/opt/resource/in [/tmp/build/get]' failed: exit status 1

And if I force a re-trigger of the pipeline it fails after pulling the image:

Successfully pulled jtarchie/pr:test@sha256:9951a43f8972f69253677f4ab2cc563a79ddb9a34206c3c1cac36f1a18e1c4fa.

"docker inspect" requires at least 1 argument(s).
See 'docker inspect --help'.

I tried switching the repository to jtarchie/pr#test as you suggested with a similar check error message, and triggering the job manually gives this error:

Pulling jtarchie/pr#test@sha256:9951a43f8972f69253677f4ab2cc563a79ddb9a34206c3c1cac36f1a18e1c4fa...
invalid reference format

Pulling jtarchie/pr#test@sha256:9951a43f8972f69253677f4ab2cc563a79ddb9a34206c3c1cac36f1a18e1c4fa (attempt 2 of 3)...
invalid reference format

Pulling jtarchie/pr#test@sha256:9951a43f8972f69253677f4ab2cc563a79ddb9a34206c3c1cac36f1a18e1c4fa (attempt 3 of 3)...
invalid reference format


Failed to pull image jtarchie/pr#test@sha256:9951a43f8972f69253677f4ab2cc563a79ddb9a34206c3c1cac36f1a18e1c4fa.

[iautom8things]

Is there a reason that the docker image has nearly tripled?

[jtarchie]

resource_types:
  - name: pull-request
     type: docker-image
     source:
       repository: jtarchie/pr
       tag: test

And the extra-size is because it has test dependencies, don't worry about that for the final image.

[iautom8things]

Thanks, that worked! Sorry about that >_<

Yeah, this is the desired result!

[jtarchie]

Great! I'll release an official release tonight.

[jtarchie]

Done, pushed it to latest.

[iautom8things]

Awesome, thank you!

…

On Mon, Aug 21, 2017 at 10:56 PM JT Archie ***@***.***> wrote: Done, pushed it to latest. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#90 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAf3OFi99XX1dD6fA1D4GhoefAqQ8fOnks5salFxgaJpZM4Owzgc> .