telia-oss / github-pr-resource

Github pull request resource for Concourse
MIT License
182 stars 5 forks source link

Name clash while using the same name for resource and resource type #175

Closed bbodensieck closed 4 years ago

bbodensieck commented 4 years ago

When trying the example https://github.com/telia-oss/github-pr-resource#example we discovered a problem by using the same name for resource and resource type. We are using concourse v5.6.0.

We got a Error 500: base resource type not found: pull-request

Seems that there is a name clash. After renaming the resource_type to e.g. pull-request-resource it worked.

Before

resource_types:
- name: pull-request
  type: docker-image
  source:
    repository: teliaoss/github-pr-resource

resources:
- name: pull-request
  type: pull-request
  check_every: 24h
  webhook_token: ((webhook-token))
  source:
    repository: itsdalmo/test-repository
    access_token: ((github-access-token))

After

resource_types:
- name: pull-request-resource
  type: docker-image
  source:
    repository: teliaoss/github-pr-resource

resources:
- name: pull-request
  type: pull-request-resource
  check_every: 24h
  webhook_token: ((webhook-token))
  source:
    repository: itsdalmo/test-repository
    access_token: ((github-access-token))

Could you update the example please?

itsdalmo commented 4 years ago

This is working fine from my testing - is it something that changed in v5.6.0, and is it documented and intended?

bbodensieck commented 4 years ago

We upgraded to v5.7.2 already. With which concourse version did you test it? I mean if the name clash is not happening anymore you can close this issue.