Repository: redhat-developer/service-binding-operator
Pull request #524 (2020-06-26 11:45:50)

Add owner reference to secret
### Github Issue

#512 

### Jira Task

https://issues.redhat.com/browse/APPSVC-610

### Motivation

[1] Sets `ownerReference` to secret, so that after SBR deletion, secret can be deleted by garbage collector.

### Changes

[1] OwnerReference information is added to intermediate secret.

### Testing

[1] Follow the steps [here](https://github.com/redhat-developer/service-binding-operator/tree/master/examples/nodejs_postgresql) to create application, DB and SBR successfully.

[2] Delete SBR:

```
➜ oc delete sbr binding-request
```

[3] Check the presence of secret `binding-request`

```
➜ oc get secret binding-request               
Error from server (NotFound): secrets "binding-request" not found
```

The secret `binding-request` should be deleted successfully.

## Tests

[1] We already have a e2e test to test successful deletion of secret after SBR is deleted -> https://github.com/redhat-developer/service-binding-operator/blob/master/test/e2e/servicebindingrequest_test.go#L607

openshift-ci-robot: [APPROVALNOTIFIER] This PR is **NOT APPROVED**

This pull-request has been approved by:
To complete the [pull request process](https://git.k8s.io/community/contributors/guide/owners.md#the-code-review-process), please assign **dhritishikhar**
You can assign the PR to them by writing `/assign @dhritishikhar` in a comment when ready.

The full list of commands accepted by this bot can be found [here](?repo=redhat-developer%2Fservice-binding-operator).

<details open>
Needs approval from an approver in each of these files:

- **[OWNERS](https://github.com/redhat-developer/service-binding-operator/blob/master/OWNERS)**

Approvers can indicate their approval by writing `/approve` in a comment
Approvers can cancel approval by writing `/approve cancel` in a comment
</details>
<!-- META={"approvers":["dhritishikhar"]} -->
openshift-ci-robot: @DhritiShikhar: The following test **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
ci/prow/4.5-lint | 6f19959f79b2d0995a4cffbc17f884c43f48cbac | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-lint/1276481802583674880) | `/test 4.5-lint`

[Full PR test history](https://prow.svc.ci.openshift.org/pr-history?org=redhat-developer&repo=service-binding-operator&pr=524). [Your PR dashboard](https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:DhritiShikhar). Please help us cut down on flakes by [linking to](https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/redhat-developer/service-binding-operator/issues?q=is:issue+is:open) when you hit one in your PR.

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
openshift-ci-robot: @DhritiShikhar: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
ci/prow/4.5-lint | 6f19959f79b2d0995a4cffbc17f884c43f48cbac | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-lint/1276481802583674880) | `/test 4.5-lint`
ci/prow/4.5-unit | 6f19959f79b2d0995a4cffbc17f884c43f48cbac | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-unit/1276481802680143872) | `/test 4.5-unit`

[Full PR test history](https://prow.svc.ci.openshift.org/pr-history?org=redhat-developer&repo=service-binding-operator&pr=524). [Your PR dashboard](https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:DhritiShikhar). Please help us cut down on flakes by [linking to](https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/redhat-developer/service-binding-operator/issues?q=is:issue+is:open) when you hit one in your PR.

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
openshift-ci-robot: @DhritiShikhar: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
ci/prow/4.5-lint | 6f19959f79b2d0995a4cffbc17f884c43f48cbac | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-lint/1276481802583674880) | `/test 4.5-lint`
ci/prow/4.5-unit | 6f19959f79b2d0995a4cffbc17f884c43f48cbac | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-unit/1276481802680143872) | `/test 4.5-unit`
ci/prow/unit | 6f19959f79b2d0995a4cffbc17f884c43f48cbac | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-unit/1276481802868887552) | `/test unit`

[Full PR test history](https://prow.svc.ci.openshift.org/pr-history?org=redhat-developer&repo=service-binding-operator&pr=524). [Your PR dashboard](https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:DhritiShikhar). Please help us cut down on flakes by [linking to](https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/redhat-developer/service-binding-operator/issues?q=is:issue+is:open) when you hit one in your PR.

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
openshift-ci-robot: @DhritiShikhar: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
ci/prow/4.5-lint | 6f19959f79b2d0995a4cffbc17f884c43f48cbac | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-lint/1276481802583674880) | `/test 4.5-lint`
ci/prow/4.5-unit | 6f19959f79b2d0995a4cffbc17f884c43f48cbac | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-unit/1276481802680143872) | `/test 4.5-unit`
ci/prow/unit | 6f19959f79b2d0995a4cffbc17f884c43f48cbac | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-unit/1276481802868887552) | `/test unit`
ci/prow/lint | 6f19959f79b2d0995a4cffbc17f884c43f48cbac | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-lint/1276481802776612864) | `/test lint`

[Full PR test history](https://prow.svc.ci.openshift.org/pr-history?org=redhat-developer&repo=service-binding-operator&pr=524). [Your PR dashboard](https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:DhritiShikhar). Please help us cut down on flakes by [linking to](https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/redhat-developer/service-binding-operator/issues?q=is:issue+is:open) when you hit one in your PR.

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
openshift-ci-robot: @DhritiShikhar: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
ci/prow/4.5-lint | 6f19959f79b2d0995a4cffbc17f884c43f48cbac | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-lint/1276481802583674880) | `/test 4.5-lint`
ci/prow/4.5-unit | 6f19959f79b2d0995a4cffbc17f884c43f48cbac | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-unit/1276481802680143872) | `/test 4.5-unit`
ci/prow/lint | 6f19959f79b2d0995a4cffbc17f884c43f48cbac | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-lint/1276481802776612864) | `/test lint`
ci/prow/unit | dcab017b8cf6f84e892c96563f5cf76d7f3ccfe6 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-unit/1276548108372676608) | `/test unit`

[Full PR test history](https://prow.svc.ci.openshift.org/pr-history?org=redhat-developer&repo=service-binding-operator&pr=524). [Your PR dashboard](https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:DhritiShikhar). Please help us cut down on flakes by [linking to](https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/redhat-developer/service-binding-operator/issues?q=is:issue+is:open) when you hit one in your PR.

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
openshift-ci-robot: @DhritiShikhar: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
ci/prow/unit | dcab017b8cf6f84e892c96563f5cf76d7f3ccfe6 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-unit/1276548108372676608) | `/test unit`
ci/prow/4.5-unit | dcab017b8cf6f84e892c96563f5cf76d7f3ccfe6 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-unit/1276548108288790528) | `/test 4.5-unit`

[Full PR test history](https://prow.svc.ci.openshift.org/pr-history?org=redhat-developer&repo=service-binding-operator&pr=524). [Your PR dashboard](https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:DhritiShikhar). Please help us cut down on flakes by [linking to](https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/redhat-developer/service-binding-operator/issues?q=is:issue+is:open) when you hit one in your PR.

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
DhritiShikhar: 
/test unit



openshift-ci-robot: @DhritiShikhar: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
ci/prow/4.5-unit | dcab017b8cf6f84e892c96563f5cf76d7f3ccfe6 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-unit/1276548108288790528) | `/test 4.5-unit`
ci/prow/unit | dcab017b8cf6f84e892c96563f5cf76d7f3ccfe6 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-unit/1276553219610251264) | `/test unit`

[Full PR test history](https://prow.svc.ci.openshift.org/pr-history?org=redhat-developer&repo=service-binding-operator&pr=524). [Your PR dashboard](https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:DhritiShikhar). Please help us cut down on flakes by [linking to](https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/redhat-developer/service-binding-operator/issues?q=is:issue+is:open) when you hit one in your PR.

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
openshift-ci-robot: @DhritiShikhar: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
ci/prow/4.5-unit | dcab017b8cf6f84e892c96563f5cf76d7f3ccfe6 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-unit/1276548108288790528) | `/test 4.5-unit`
ci/prow/unit | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-unit/1276556747007332352) | `/test unit`

[Full PR test history](https://prow.svc.ci.openshift.org/pr-history?org=redhat-developer&repo=service-binding-operator&pr=524). [Your PR dashboard](https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:DhritiShikhar). Please help us cut down on flakes by [linking to](https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/redhat-developer/service-binding-operator/issues?q=is:issue+is:open) when you hit one in your PR.

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
openshift-ci-robot: @DhritiShikhar: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
ci/prow/unit | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-unit/1276556747007332352) | `/test unit`
ci/prow/4.5-unit | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-unit/1276556745757429760) | `/test 4.5-unit`

[Full PR test history](https://prow.svc.ci.openshift.org/pr-history?org=redhat-developer&repo=service-binding-operator&pr=524). [Your PR dashboard](https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:DhritiShikhar). Please help us cut down on flakes by [linking to](https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/redhat-developer/service-binding-operator/issues?q=is:issue+is:open) when you hit one in your PR.

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
sbose78: Does the right thing!

/lgtm
openshift-ci-robot: @DhritiShikhar: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
ci/prow/unit | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-unit/1276556747007332352) | `/test unit`
ci/prow/4.5-unit | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-unit/1276556745757429760) | `/test 4.5-unit`
ci/prow/e2e | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-e2e/1276556746554347520) | `/test e2e`

[Full PR test history](https://prow.svc.ci.openshift.org/pr-history?org=redhat-developer&repo=service-binding-operator&pr=524). [Your PR dashboard](https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:DhritiShikhar). Please help us cut down on flakes by [linking to](https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/redhat-developer/service-binding-operator/issues?q=is:issue+is:open) when you hit one in your PR.

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
openshift-ci-robot: @DhritiShikhar: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
ci/prow/unit | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-unit/1276556747007332352) | `/test unit`
ci/prow/4.5-unit | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-unit/1276556745757429760) | `/test 4.5-unit`
ci/prow/e2e | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-e2e/1276556746554347520) | `/test e2e`
ci/prow/4.5-e2e | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-e2e/1276556745459634176) | `/test 4.5-e2e`

[Full PR test history](https://prow.svc.ci.openshift.org/pr-history?org=redhat-developer&repo=service-binding-operator&pr=524). [Your PR dashboard](https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:DhritiShikhar). Please help us cut down on flakes by [linking to](https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/redhat-developer/service-binding-operator/issues?q=is:issue+is:open) when you hit one in your PR.

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
DhritiShikhar: /test unit
openshift-ci-robot: @DhritiShikhar: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
ci/prow/4.5-unit | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-unit/1276556745757429760) | `/test 4.5-unit`
ci/prow/e2e | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-e2e/1276556746554347520) | `/test e2e`
ci/prow/4.5-e2e | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-e2e/1276556745459634176) | `/test 4.5-e2e`
ci/prow/unit | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-unit/1277487371994009600) | `/test unit`

[Full PR test history](https://prow.svc.ci.openshift.org/pr-history?org=redhat-developer&repo=service-binding-operator&pr=524). [Your PR dashboard](https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:DhritiShikhar). Please help us cut down on flakes by [linking to](https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/redhat-developer/service-binding-operator/issues?q=is:issue+is:open) when you hit one in your PR.

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
DhritiShikhar: The tests are passing locally. 
pratikjagrut: /retest
openshift-ci-robot: @DhritiShikhar: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
ci/prow/e2e | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-e2e/1276556746554347520) | `/test e2e`
ci/prow/4.5-e2e | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-e2e/1276556745459634176) | `/test 4.5-e2e`
ci/prow/unit | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-unit/1277487371994009600) | `/test unit`
ci/prow/4.5-unit | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-unit/1277525178460409856) | `/test 4.5-unit`

[Full PR test history](https://prow.svc.ci.openshift.org/pr-history?org=redhat-developer&repo=service-binding-operator&pr=524). [Your PR dashboard](https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:DhritiShikhar). Please help us cut down on flakes by [linking to](https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/redhat-developer/service-binding-operator/issues?q=is:issue+is:open) when you hit one in your PR.

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
openshift-ci-robot: @DhritiShikhar: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
ci/prow/e2e | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-e2e/1276556746554347520) | `/test e2e`
ci/prow/4.5-e2e | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-e2e/1276556745459634176) | `/test 4.5-e2e`
ci/prow/4.5-unit | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-unit/1277525178460409856) | `/test 4.5-unit`
ci/prow/unit | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-unit/1277525178527518720) | `/test unit`

[Full PR test history](https://prow.svc.ci.openshift.org/pr-history?org=redhat-developer&repo=service-binding-operator&pr=524). [Your PR dashboard](https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:DhritiShikhar). Please help us cut down on flakes by [linking to](https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/redhat-developer/service-binding-operator/issues?q=is:issue+is:open) when you hit one in your PR.

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
openshift-ci-robot: @DhritiShikhar: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
ci/prow/4.5-e2e | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-e2e/1276556745459634176) | `/test 4.5-e2e`
ci/prow/4.5-unit | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-unit/1277525178460409856) | `/test 4.5-unit`
ci/prow/unit | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-unit/1277525178527518720) | `/test unit`
ci/prow/e2e | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-e2e/1277525178485575680) | `/test e2e`

[Full PR test history](https://prow.svc.ci.openshift.org/pr-history?org=redhat-developer&repo=service-binding-operator&pr=524). [Your PR dashboard](https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:DhritiShikhar). Please help us cut down on flakes by [linking to](https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/redhat-developer/service-binding-operator/issues?q=is:issue+is:open) when you hit one in your PR.

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
openshift-ci-robot: @DhritiShikhar: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
ci/prow/4.5-unit | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-unit/1277525178460409856) | `/test 4.5-unit`
ci/prow/unit | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-unit/1277525178527518720) | `/test unit`
ci/prow/e2e | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-e2e/1277525178485575680) | `/test e2e`
ci/prow/4.5-e2e | 480a24045c6547fc972712caa094a8d9f5a2b199 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-4.5-e2e/1277525178435244032) | `/test 4.5-e2e`

[Full PR test history](https://prow.svc.ci.openshift.org/pr-history?org=redhat-developer&repo=service-binding-operator&pr=524). [Your PR dashboard](https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:DhritiShikhar). Please help us cut down on flakes by [linking to](https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/redhat-developer/service-binding-operator/issues?q=is:issue+is:open) when you hit one in your PR.

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
openshift-ci-robot: [APPROVALNOTIFIER] This PR is **NOT APPROVED**

This pull-request has been approved by:
To complete the [pull request process](https://git.k8s.io/community/contributors/guide/owners.md#the-code-review-process), please assign **dhritishikhar**
You can assign the PR to them by writing `/assign @dhritishikhar` in a comment when ready.

The full list of commands accepted by this bot can be found [here](https://go.k8s.io/bot-commands?repo=redhat-developer%2Fservice-binding-operator).

<details open>
Needs approval from an approver in each of these files:

- **[OWNERS](https://github.com/redhat-developer/service-binding-operator/blob/master/OWNERS)**

Approvers can indicate their approval by writing `/approve` in a comment
Approvers can cancel approval by writing `/approve cancel` in a comment
</details>
<!-- META={"approvers":["dhritishikhar"]} -->
openshift-ci-robot: @DhritiShikhar: The following test **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
ci/prow/e2e | ab99fc5a0b49421493036404ddb755c1c6d2ab5f | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-e2e/1278965748722569216) | `/test e2e`

[Full PR test history](https://prow.svc.ci.openshift.org/pr-history?org=redhat-developer&repo=service-binding-operator&pr=524). [Your PR dashboard](https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:DhritiShikhar). Please help us cut down on flakes by [linking to](https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/redhat-developer/service-binding-operator/issues?q=is:issue+is:open) when you hit one in your PR.

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
openshift-ci-robot: @DhritiShikhar: The following test **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
ci/prow/e2e | 3a5341c37bd4cecf1c551364f847f7fbaa242a53 | [link](https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/redhat-developer_service-binding-operator/524/pull-ci-redhat-developer-service-binding-operator-master-e2e/1279012453086138368) | `/test e2e`

[Full PR test history](https://prow.svc.ci.openshift.org/pr-history?org=redhat-developer&repo=service-binding-operator&pr=524). [Your PR dashboard](https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:DhritiShikhar). Please help us cut down on flakes by [linking to](https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/redhat-developer/service-binding-operator/issues?q=is:issue+is:open) when you hit one in your PR.

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
DhritiShikhar: > @DhritiShikhar please clarify if some additional behaviour should be added?

What sort of additional behaviour?

That commit has been removed by the way @pedjak 

Patch:
diff --git a/README.md b/README.md
index 479f3b0d..00649cb6 100644
--- a/README.md
+++ b/README.md
@@ -95,3 +95,13 @@ The following example scenarios are available:
 [Binding an Imported app to an Off-cluster Operator Managed IBM Cloud Service](examples/nodejs_ibmcloud_operator/README.md)
 
 [Binding an Imported app in one namespace with an In-cluster Managed PostgreSQL Database in another namespace](examples/nodejs_postgresql_namespaces/README.md)
+
+## Community, discussion, contribution, and support
+
+The Service Binding community meets weekly on Thursdays at 11:15 AM UTC via [Google Meet](https://meet.google.com/jti-pujt-zqs).
+
+Meeting Agenda is maintained [here](https://github.com/redhat-developer/service-binding-operator/issues?q=is%3Aissue+is%3Aopen+label%3Ameeting)
+
+Please file bug reports on [Github](https://github.com/redhat-developer/service-binding-operator/issues/new). For any other questions, reach out on `service-binding-support@redhat.com`.
+
+
diff --git a/pkg/apis/apps/v1alpha1/servicebindingrequest_types.go b/pkg/apis/apps/v1alpha1/servicebindingrequest_types.go
index 321ae486..472c4c7b 100644
--- a/pkg/apis/apps/v1alpha1/servicebindingrequest_types.go
+++ b/pkg/apis/apps/v1alpha1/servicebindingrequest_types.go
@@ -112,3 +112,14 @@ type ServiceBindingRequestList struct {
 func init() {
 	SchemeBuilder.Register(&ServiceBindingRequest{}, &ServiceBindingRequestList{})
 }
+
+func (sbr ServiceBindingRequest) AsOwnerReference() metav1.OwnerReference {
+	var ownerRefController bool = true
+	return metav1.OwnerReference{
+		Name:       sbr.Name,
+		UID:        sbr.UID,
+		Kind:       sbr.Kind,
+		APIVersion: sbr.APIVersion,
+		Controller: &ownerRefController,
+	}
+}
diff --git a/pkg/controller/servicebindingrequest/secret.go b/pkg/controller/servicebindingrequest/secret.go
index 22eba8ac..f05af5c7 100644
--- a/pkg/controller/servicebindingrequest/secret.go
+++ b/pkg/controller/servicebindingrequest/secret.go
@@ -27,12 +27,13 @@ func (s *secret) buildResourceClient() dynamic.ResourceInterface {
 
 // createOrUpdate will take informed payload and either create a new secret or update an existing
 // one. It can return error when Kubernetes client does.
-func (s *secret) createOrUpdate(payload map[string][]byte) (*unstructured.Unstructured, error) {
+func (s *secret) createOrUpdate(payload map[string][]byte, ownerReference metav1.OwnerReference) (*unstructured.Unstructured, error) {
 	logger := s.logger.WithValues("Namespace", s.ns, "Name", s.name)
 	secretObj := &corev1.Secret{
 		ObjectMeta: metav1.ObjectMeta{
-			Namespace: s.ns,
-			Name:      s.name,
+			Namespace:       s.ns,
+			Name:            s.name,
+			OwnerReferences: []metav1.OwnerReference{ownerReference},
 		},
 		Data: payload,
 	}
@@ -59,12 +60,6 @@ func (s *secret) createOrUpdate(payload map[string][]byte) (*unstructured.Unstru
 	return u, nil
 }
 
-// commit will store informed data as a secret, commit it against the API server. It can forward
-// errors from custom environment parser component, or from the API server itself.
-func (s *secret) commit(payload map[string][]byte) (*unstructured.Unstructured, error) {
-	return s.createOrUpdate(payload)
-}
-
 // get an unstructured object from the secret handled by this component. It can return errors in case
 // the API server does.
 func (s *secret) get() (*unstructured.Unstructured, bool, error) {
@@ -76,16 +71,6 @@ func (s *secret) get() (*unstructured.Unstructured, bool, error) {
 	return u, u != nil, nil
 }
 
-// delete the secret represented by this component. It can return error when the API server does.
-func (s *secret) delete() error {
-	resourceClient := s.buildResourceClient()
-	err := resourceClient.Delete(s.name, &metav1.DeleteOptions{})
-	if err != nil && !errors.IsNotFound(err) {
-		return err
-	}
-	return nil
-}
-
 // newSecret instantiate a new Secret.
 func newSecret(
 	client dynamic.Interface,
diff --git a/pkg/controller/servicebindingrequest/servicebinder.go b/pkg/controller/servicebindingrequest/servicebinder.go
index f3e767f1..03e3aa32 100644
--- a/pkg/controller/servicebindingrequest/servicebinder.go
+++ b/pkg/controller/servicebindingrequest/servicebinder.go
@@ -152,12 +152,6 @@ func (b *serviceBinder) unbind() (reconcile.Result, error) {
 		return requeueError(err)
 	}
 
-	logger.Info("Deleting intermediary secret")
-	if err := b.secret.delete(); err != nil {
-		logger.Error(err, "On deleting intermediary secret.")
-		return requeueError(err)
-	}
-
 	logger.Debug("Removing resource finalizers...")
 	b.sbr.SetFinalizers(removeStringSlice(b.sbr.GetFinalizers(), finalizer))
 	if _, err := b.updateServiceBindingRequest(b.sbr); err != nil {
@@ -247,7 +241,8 @@ func (b *serviceBinder) bind() (reconcile.Result, error) {
 	sbrStatus := b.sbr.Status.DeepCopy()
 
 	b.logger.Info("Saving data on intermediary secret...")
-	secretObj, err := b.secret.commit(b.envVars)
+
+	secretObj, err := b.secret.createOrUpdate(b.envVars, b.sbr.AsOwnerReference())
 	if err != nil {
 		b.logger.Error(err, "On saving secret data..")
 		return b.onError(err, b.sbr, sbrStatus, nil)


Test patch:
diff --git a/pkg/controller/servicebindingrequest/secret_test.go b/pkg/controller/servicebindingrequest/secret_test.go
index 8f579eb0..ba7c4934 100644
--- a/pkg/controller/servicebindingrequest/secret_test.go
+++ b/pkg/controller/servicebindingrequest/secret_test.go
@@ -4,14 +4,26 @@ import (
 	"testing"
 
 	"github.com/stretchr/testify/assert"
+	v1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
 
 	"github.com/redhat-developer/service-binding-operator/test/mocks"
 )
 
+var ownerRefController bool = true
+var secretOwnerReference = v1.OwnerReference{
+	Name:       "binding-request",
+	UID:        "c77ca1ae-72d0-4fdd-809f-58fdd37facf3",
+	Kind:       "ServiceBindingRequest",
+	APIVersion: "apps.openshift.io/v1alpha1",
+	Controller: &ownerRefController,
+}
+
 func assertSecretNamespacedName(t *testing.T, u *unstructured.Unstructured, ns, name string) {
 	assert.Equal(t, ns, u.GetNamespace())
 	assert.Equal(t, name, u.GetName())
+	ownerReference := u.GetOwnerReferences()
+	assert.Equal(t, secretOwnerReference, ownerReference[0])
 }
 
 func TestSecretNew(t *testing.T) {
@@ -29,18 +41,7 @@ func TestSecretNew(t *testing.T) {
 	)
 
 	t.Run("createOrUpdate", func(t *testing.T) {
-		u, err := s.createOrUpdate(data)
-		assert.NoError(t, err)
-		assertSecretNamespacedName(t, u, ns, name)
-	})
-
-	t.Run("Delete", func(t *testing.T) {
-		err := s.delete()
-		assert.NoError(t, err)
-	})
-
-	t.Run("Commit", func(t *testing.T) {
-		u, err := s.commit(data)
+		u, err := s.createOrUpdate(data, secretOwnerReference)
 		assert.NoError(t, err)
 		assertSecretNamespacedName(t, u, ns, name)
 	})