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 (#1… #1717

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 into release-0.17

@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

@google-cla google-cla bot added the cla: yes (override cla status due to multiple authors bug) label Sep 16, 2020
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

@knative-test-reporter-robot

The following jobs failed:

Test name Triggers Retries
pull-google-knative-gcp-wi-tests 2020-09-17 00:13:35.243 +0000 UTC
2020-09-17 00:54:34.982 +0000 UTC
2020-09-17 01:32:04.989 +0000 UTC
3/3

Automatically retrying due to test flakiness...
/test pull-google-knative-gcp-wi-tests

@knative-prow-robot knative-prow-robot merged commit 8ed7643 into google:release-0.17 Sep 17, 2020
@ian-mi ian-mi deleted the release-0.17 branch September 17, 2020 02:41
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.

4 participants