Skip to content

Comments

Mtbroker to just broker it is cleaner#5021

Merged
knative-prow-robot merged 1 commit intoknative:masterfrom
vaikas:mtbroker-to-just-broker-it-is-cleaner
Mar 6, 2021
Merged

Mtbroker to just broker it is cleaner#5021
knative-prow-robot merged 1 commit intoknative:masterfrom
vaikas:mtbroker-to-just-broker-it-is-cleaner

Conversation

@vaikas
Copy link
Contributor

@vaikas vaikas commented Mar 6, 2021

Proposed Changes

  • Back when we had the original broker I created mtbroker to distinguish. It's now been around for so long that it's safe to rename it to more clearer broker. mtbroker was just a different implementation, there's no need to carry that around in the name / package anymore (especially folks that don't know the history), it's just confusing. So change that.
  • NOTE Not changing the brokerclass type, hence there are no user visible changes.

Release Note


Docs

@knative-prow-robot knative-prow-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Mar 6, 2021
@google-cla google-cla bot added the cla: yes Indicates the PR's author has signed the CLA. label Mar 6, 2021
Copy link
Contributor

@n3wscott n3wscott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

Good job!

@knative-prow-robot knative-prow-robot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Mar 6, 2021
@codecov
Copy link

codecov bot commented Mar 6, 2021

Codecov Report

Merging #5021 (a30e3a5) into master (e8bc8ed) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5021   +/-   ##
=======================================
  Coverage   83.09%   83.09%           
=======================================
  Files         283      283           
  Lines        7933     7933           
=======================================
  Hits         6592     6592           
  Misses        968      968           
  Partials      373      373           
Impacted Files Coverage Δ
pkg/broker/filter/filter_handler.go 80.40% <ø> (ø)
pkg/broker/filter/stats_reporter.go 87.30% <ø> (ø)
pkg/broker/ingress/ingress_handler.go 62.76% <ø> (ø)
pkg/broker/ingress/stats_reporter.go 87.75% <ø> (ø)
pkg/broker/ttl.go 72.72% <ø> (ø)
pkg/reconciler/broker/broker.go 80.46% <ø> (ø)
pkg/reconciler/broker/config.go 86.66% <ø> (ø)
pkg/reconciler/broker/controller.go 93.02% <ø> (ø)
pkg/reconciler/broker/resources/channel.go 100.00% <ø> (ø)
pkg/reconciler/broker/resources/subscription.go 100.00% <ø> (ø)
... and 2 more

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 e8bc8ed...a30e3a5. Read the comment docs.

@vaikas vaikas force-pushed the mtbroker-to-just-broker-it-is-cleaner branch from 8c44703 to a30e3a5 Compare March 6, 2021 17:55
@knative-prow-robot knative-prow-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Mar 6, 2021
@knative-metrics-robot
Copy link

The following is the coverage report on the affected files.
Say /test pull-knative-eventing-go-coverage to re-run this coverage report

File Old Coverage New Coverage Delta
pkg/broker/filter/filter_handler.go Do not exist 88.5%
pkg/broker/filter/stats_reporter.go Do not exist 84.6%
pkg/broker/ingress/ingress_handler.go Do not exist 78.0%
pkg/broker/ingress/stats_reporter.go Do not exist 83.3%
pkg/broker/ttl.go Do not exist 85.0%
pkg/reconciler/broker/broker.go Do not exist 84.3%
pkg/reconciler/broker/config.go Do not exist 93.3%
pkg/reconciler/broker/controller.go Do not exist 88.9%
pkg/reconciler/broker/resources/channel.go Do not exist 100.0%
pkg/reconciler/broker/resources/subscription.go Do not exist 100.0%
pkg/reconciler/broker/trigger/controller.go Do not exist 88.9%
pkg/reconciler/broker/trigger/trigger.go Do not exist 86.1%

@vaikas
Copy link
Contributor Author

vaikas commented Mar 6, 2021

/retest
another flake for rekt with 0.3

Copy link
Contributor

@n3wscott n3wscott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

@knative-prow-robot knative-prow-robot added the lgtm Indicates that a PR is ready to be merged. label Mar 6, 2021
@knative-prow-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: n3wscott

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

The pull request process is described here

Details 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

@knative-prow-robot knative-prow-robot merged commit 6136d09 into knative:master Mar 6, 2021
@vaikas vaikas deleted the mtbroker-to-just-broker-it-is-cleaner branch March 8, 2021 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cla: yes Indicates the PR's author has signed the CLA. lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants