eventing icon indicating copy to clipboard operation
eventing copied to clipboard

When the svc associated with inmemorychannel does not have an OwnerReference, it cannot work.

Open kycheng opened this issue 1 year ago • 3 comments

Describe the bug

When the ownerReference of the service associated with InMemoryChannel is deleted, the following error will appear.

    - lastTransitionTime: 2023-11-02T09:06:28Z
      message: 'Channel Service failed: inmemorychannel: dev-1/default-kne-trigger
        does not own Service: "default-kne-trigger-kn-channel"'
      reason: ChannelServiceFailed
      status: "False"
      type: ChannelServiceReady

At this time, the controller does not add ownerReference back

Expected behavior

At this time, the controller automatically adds ownerReference

To Reproduce

Manually delete the ownerReference of the service

Knative release version

1.2.0

kycheng avatar Nov 02 '23 09:11 kycheng

Hey @kycheng are you able to reproduce this bug on a newer version of Knative? I tried on 1.12 and was unable to reproduce the error

Cali0707 avatar Dec 05 '23 20:12 Cali0707

Hey @kycheng are you able to reproduce this bug on a newer version of Knative? I tried on 1.12 and was unable to reproduce the error

Later we found the root cause of the problem. The associated svc has been manually modified and the ownerReference has been removed. This may be an unexpected behavior.

I think this is a rather extreme case, and it might be better if we can identify this situation and fix it.

kycheng avatar Dec 06 '23 08:12 kycheng

This issue is stale because it has been open for 90 days with no activity. It will automatically close after 30 more days of inactivity. Reopen the issue with /reopen. Mark the issue as fresh by adding the comment /remove-lifecycle stale.

github-actions[bot] avatar Mar 06 '24 01:03 github-actions[bot]

The associated svc has been manually modified and the ownerReference has been removed

By doing this, the controller won't recognize that it's its own resource and it will fail, I don't think there is much we can do

pierDipi avatar Mar 26 '24 10:03 pierDipi

Closing for now, please reopen in case it happens also when the OwnerReference is not manually updated

pierDipi avatar Mar 26 '24 10:03 pierDipi