Skip to content
This repository has been archived by the owner on Nov 28, 2022. It is now read-only.

[Backport] [Release 0.23] Use transformers to extract metrics for Kafka #1318

Merged

Conversation

skonto
Copy link

@skonto skonto commented Jun 16, 2021

Same as in #1316, cherry picked the related commits from main upstream. I will the create a PR against eventing-kafka 0.23, this is for aligning the code bases.

)

* expose dispatch result to be used by other implementations

* expose fields

* add getters etc

* less verbose
… Channel (knative#5505)

* use transformers for extracting event type for metrics

* fix

* fix
@skonto skonto changed the title [Release 0.23] Use transformers to extract metrics for Kafka [Backport] [Release 0.23] Use transformers to extract metrics for Kafka Jun 16, 2021
@openshift-ci openshift-ci bot requested review from alanfx and matzew June 16, 2021 13:46
@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 16, 2021
@skonto
Copy link
Author

skonto commented Jun 16, 2021

/assign @slinkydeveloper

@slinkydeveloper
Copy link

/lgtm
/approve

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jun 16, 2021
@openshift-ci
Copy link

openshift-ci bot commented Jun 16, 2021

[APPROVALNOTIFIER] This PR is APPROVED

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

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:

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

@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

11 similar comments
@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-merge-robot openshift-merge-robot merged commit f7036f2 into openshift:release-v0.23 Jun 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants