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

Properly set system namespace when reconciling default brokercell #1716

Merged
merged 1 commit into from
Sep 17, 2020

Conversation

ian-mi
Copy link
Contributor

@ian-mi ian-mi commented Sep 16, 2020

Backport #1705 for release-0.16

@google-cla google-cla bot added the cla: yes (override cla status due to multiple authors bug) label Sep 16, 2020
@knative-prow-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ian-mi

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

@knative-metrics-robot
Copy link

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

File Old Coverage New Coverage Delta
pkg/reconciler/brokercell/controller.go 54.3% 77.3% 23.0

Copy link
Contributor

@Harwayne Harwayne left a comment

Choose a reason for hiding this comment

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

/lgtm

@ian-mi
Copy link
Contributor Author

ian-mi commented Sep 17, 2020

/retest

1 similar comment
@grac3gao-zz
Copy link
Contributor

/retest

@knative-test-reporter-robot

The following jobs failed:

Test name Triggers Retries
pull-google-knative-gcp-wi-tests 0/3

Failed non-flaky tests preventing automatic retry of pull-google-knative-gcp-wi-tests:

github.com/google/knative-gcp/test/e2e.TestSmokeChannel
github.com/google/knative-gcp/test/e2e.TestCloudStorageSourceWithTarget

@grac3gao-zz
Copy link
Contributor

/retest

@knative-prow-robot knative-prow-robot merged commit 02d6c70 into google:release-0.16 Sep 17, 2020
@ian-mi ian-mi deleted the release-0.16 branch September 17, 2020 05:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved cla: yes (override cla status due to multiple authors bug) lgtm size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants