Remove Knative history extension

[slinkydeveloper]

Signed-off-by: Francesco Guardiani francescoguard@gmail.com

Fixes #4295

Proposed Changes

• Remove KnativeHistory extension

Release Note

KnativeHistory extension is not added anymore to events going through channels

Docs

[slinkydeveloper]

/assign @grantr

[vaikas]

Looks like this failed:

ok  	knative.dev/eventing/pkg/adapter/apiserver/events	0.052s	coverage: 96.2% of statements
# knative.dev/eventing/pkg/channel [knative.dev/eventing/pkg/channel.test]
pkg/channel/message_receiver_test.go:123:32: undefined: EventHistory

[codecov]

Codecov Report

Merging #4366 into master will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #4366      +/-   ##
==========================================
- Coverage   80.81%   80.79%   -0.02%     
==========================================
  Files         288      287       -1     
  Lines        8001     7984      -17     
==========================================
- Hits         6466     6451      -15     
+ Misses       1147     1146       -1     
+ Partials      388      387       -1     

Impacted Files	Coverage Δ
pkg/channel/message_receiver.go	68.05% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3d3be5f...3999e31. Read the comment docs.

[slinkydeveloper]

@vaikas Fixed

[vaikas]

/lgtm
/approve
/hold
Adding a hold until the discussion is finished. Looks like that's slated for today.

[knative-prow-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: slinkydeveloper, vaikas

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [slinkydeveloper,vaikas]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[matzew]

I am +1 on removing it. The case in issue 4295 IMO describes very well, why it's perhaps not a good idea. Also, there is end-to-end tracing.

[vaikas]

/unhold
Discussed in the WG meeting and in triage.