Skip to content

Commit

Permalink
prs: update pending step in separate context
Browse files Browse the repository at this point in the history
from the docs:

---
When specifying skip_download the pull request volume mounted to
subsequent tasks will be empty, which is a problem when you set e.g. the
pending status before running the actual tests. The workaround for this
is to use an alias for the put (see
telia-oss/github-pr-resource#32 for more
details). Example here:

put: update-status <-- Use an alias for the pull-request resource
resource: pull-request
params:
    path: pull-request
    status: pending
get_params: {skip_download: true}
---

Signed-off-by: Toby Lorne <toby.lornewelch-richards@digital.cabinet-office.gov.uk>
  • Loading branch information
Toby Lorne committed Jun 23, 2020
1 parent 824fdf5 commit d0d1169
Show file tree
Hide file tree
Showing 2 changed files with 36 additions and 18 deletions.
3 changes: 2 additions & 1 deletion pipelines/build-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,8 @@ jobs:
trigger: true
params:
integration_tool: checkout
- put: bosh-release-pr
- put: update-bosh-release-pr
resource: bosh-release-pr
params:
path: bosh-release-pr
context: ((github_status_context))
Expand Down
51 changes: 34 additions & 17 deletions pipelines/plain_pipelines/build-docker-pull-requests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,8 @@ jobs:
trigger: true
params:
integration_tool: checkout
- put: paas-docker-cloudfoundry-tools-pr
- put: update-paas-docker-cloudfoundry-tools-pr
resource: paas-docker-cloudfoundry-tools-pr
params:
path: paas-docker-cloudfoundry-tools-pr
context: ((github_status_context))
Expand Down Expand Up @@ -219,7 +220,8 @@ jobs:
trigger: true
params:
integration_tool: checkout
- put: paas-docker-cloudfoundry-tools-pr
- put: update-paas-docker-cloudfoundry-tools-pr
resource: paas-docker-cloudfoundry-tools-pr
params:
path: paas-docker-cloudfoundry-tools-pr
context: ((github_status_context))
Expand Down Expand Up @@ -264,7 +266,8 @@ jobs:
trigger: true
params:
integration_tool: checkout
- put: paas-docker-cloudfoundry-tools-pr
- put: update-paas-docker-cloudfoundry-tools-pr
resource: paas-docker-cloudfoundry-tools-pr
params:
path: paas-docker-cloudfoundry-tools-pr
context: ((github_status_context))
Expand Down Expand Up @@ -309,7 +312,8 @@ jobs:
trigger: true
params:
integration_tool: checkout
- put: paas-docker-cloudfoundry-tools-pr
- put: update-paas-docker-cloudfoundry-tools-pr
resource: paas-docker-cloudfoundry-tools-pr
params:
path: paas-docker-cloudfoundry-tools-pr
context: ((github_status_context))
Expand Down Expand Up @@ -354,7 +358,8 @@ jobs:
trigger: true
params:
integration_tool: checkout
- put: paas-docker-cloudfoundry-tools-pr
- put: update-paas-docker-cloudfoundry-tools-pr
resource: paas-docker-cloudfoundry-tools-pr
params:
path: paas-docker-cloudfoundry-tools-pr
context: ((github_status_context))
Expand Down Expand Up @@ -399,7 +404,8 @@ jobs:
trigger: true
params:
integration_tool: checkout
- put: paas-docker-cloudfoundry-tools-pr
- put: update-paas-docker-cloudfoundry-tools-pr
resource: paas-docker-cloudfoundry-tools-pr
params:
path: paas-docker-cloudfoundry-tools-pr
context: ((github_status_context))
Expand Down Expand Up @@ -444,7 +450,8 @@ jobs:
trigger: true
params:
integration_tool: checkout
- put: paas-docker-cloudfoundry-tools-pr
- put: update-paas-docker-cloudfoundry-tools-pr
resource: paas-docker-cloudfoundry-tools-pr
params:
path: paas-docker-cloudfoundry-tools-pr
context: ((github_status_context))
Expand Down Expand Up @@ -489,7 +496,8 @@ jobs:
trigger: true
params:
integration_tool: checkout
- put: paas-docker-cloudfoundry-tools-pr
- put: update-paas-docker-cloudfoundry-tools-pr
resource: paas-docker-cloudfoundry-tools-pr
params:
path: paas-docker-cloudfoundry-tools-pr
context: ((github_status_context))
Expand Down Expand Up @@ -534,7 +542,8 @@ jobs:
trigger: true
params:
integration_tool: checkout
- put: paas-docker-cloudfoundry-tools-pr
- put: update-paas-docker-cloudfoundry-tools-pr
resource: paas-docker-cloudfoundry-tools-pr
params:
path: paas-docker-cloudfoundry-tools-pr
context: ((github_status_context))
Expand Down Expand Up @@ -579,7 +588,8 @@ jobs:
trigger: true
params:
integration_tool: checkout
- put: paas-docker-cloudfoundry-tools-pr
- put: update-paas-docker-cloudfoundry-tools-pr
resource: paas-docker-cloudfoundry-tools-pr
params:
path: paas-docker-cloudfoundry-tools-pr
context: ((github_status_context))
Expand Down Expand Up @@ -624,7 +634,8 @@ jobs:
trigger: true
params:
integration_tool: checkout
- put: paas-docker-cloudfoundry-tools-pr
- put: update-paas-docker-cloudfoundry-tools-pr
resource: paas-docker-cloudfoundry-tools-pr
params:
path: paas-docker-cloudfoundry-tools-pr
context: ((github_status_context))
Expand Down Expand Up @@ -669,7 +680,8 @@ jobs:
trigger: true
params:
integration_tool: checkout
- put: paas-docker-cloudfoundry-tools-pr
- put: update-paas-docker-cloudfoundry-tools-pr
resource: paas-docker-cloudfoundry-tools-pr
params:
path: paas-docker-cloudfoundry-tools-pr
context: ((github_status_context))
Expand Down Expand Up @@ -714,7 +726,8 @@ jobs:
trigger: true
params:
integration_tool: checkout
- put: paas-docker-cloudfoundry-tools-pr
- put: update-paas-docker-cloudfoundry-tools-pr
resource: paas-docker-cloudfoundry-tools-pr
params:
path: paas-docker-cloudfoundry-tools-pr
context: ((github_status_context))
Expand Down Expand Up @@ -759,7 +772,8 @@ jobs:
trigger: true
params:
integration_tool: checkout
- put: paas-docker-cloudfoundry-tools-pr
- put: update-paas-docker-cloudfoundry-tools-pr
resource: paas-docker-cloudfoundry-tools-pr
params:
path: paas-docker-cloudfoundry-tools-pr
context: ((github_status_context))
Expand Down Expand Up @@ -804,7 +818,8 @@ jobs:
trigger: true
params:
integration_tool: checkout
- put: paas-docker-cloudfoundry-tools-pr
- put: update-paas-docker-cloudfoundry-tools-pr
resource: paas-docker-cloudfoundry-tools-pr
params:
path: paas-docker-cloudfoundry-tools-pr
context: ((github_status_context))
Expand Down Expand Up @@ -849,7 +864,8 @@ jobs:
trigger: true
params:
integration_tool: checkout
- put: paas-tech-docs-pr
- put: update-paas-tech-docs-pr
resource: paas-tech-docs-pr
params:
path: paas-tech-docs-pr
context: ((github_status_context))
Expand Down Expand Up @@ -921,7 +937,8 @@ jobs:
trigger: true
params:
integration_tool: checkout
- put: paas-semver-resource-pr
- put: update-paas-semver-resource-pr
resource: paas-semver-resource-pr
params:
path: paas-semver-resource-pr
context: ((github_status_context))
Expand Down

0 comments on commit d0d1169

Please sign in to comment.