Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Github Source doesn't surface failures registering webhook #554

Closed
mattmoor opened this issue Nov 17, 2018 · 1 comment
Closed

Github Source doesn't surface failures registering webhook #554

mattmoor opened this issue Nov 17, 2018 · 1 comment

Comments

@mattmoor
Copy link
Member

I was trying to use @mattmoor-sockpuppet as the account for the Github webhook sample, and (despite being a collaborator) it didn't have permission to configure webhooks on mattmoor/kontext.

I see errors in the logs for knative-sources/controller-manager-0 (not the best name):

{"level":"warn","ts":1542471029.8092682,"logger":"fallback","caller":"sdk/reconciler.go:75","msg":"Failed to reconcile &TypeMeta{Kind:,APIVersion:,}: failed to create webhook: failed to create the webhook: POST https://api.github.com/repos/mattmoor/kontext/hooks: 404 Not Found []"}

The subscription resource I'd expect to fail:

apiVersion: eventing.knative.dev/v1alpha1
kind: Subscription
metadata:
  creationTimestamp: 2018-11-17T16:28:31Z
  finalizers:
  - subscription-controller
  generation: 1
  name: github-source-sample
  namespace: default
  resourceVersion: "183212"
  selfLink: /apis/eventing.knative.dev/v1alpha1/namespaces/default/subscriptions/github-source-sample
  uid: d29797c1-ea85-11e8-bce9-42010a960037
spec:
  channel:
    apiVersion: eventing.knative.dev/v1alpha1
    kind: Channel
    name: githubchannel
  generation: 1
  subscriber:
    ref:
      apiVersion: serving.knative.dev/v1alpha1
      kind: Service
      name: github-message-dumper
status:
  conditions:
  - lastTransitionTime: 2018-11-17T16:28:32Z
    severity: Error
    status: "True"
    type: ChannelReady
  - lastTransitionTime: 2018-11-17T16:28:32Z
    severity: Error
    status: "True"                   <-- Why is this True?  It is not Ready...
    type: Ready
  - lastTransitionTime: 2018-11-17T16:28:32Z
    severity: Error
    status: "True"
    type: Resolved
  physicalSubscription:
    replyURI: ""
    subscriberURI: http://github-message-dumper.default.svc.cluster.local/

This led to me running into: #553

cc @vaikas-google @evankanderson

@mattmoor
Copy link
Member Author

This probably belongs on eventing-sources. It looks like this TODO.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant