Skip to content

[RELEASE 1.10] Incorporate reconciler-test backports#300

Closed
ReToCode wants to merge 1 commit intoopenshift-knative:release-v1.10from
ReToCode:reconciler-test-fix
Closed

[RELEASE 1.10] Incorporate reconciler-test backports#300
ReToCode wants to merge 1 commit intoopenshift-knative:release-v1.10from
ReToCode:reconciler-test-fix

Conversation

@ReToCode
Copy link
Copy Markdown

@ReToCode ReToCode commented Aug 3, 2023

@openshift-ci openshift-ci bot requested review from aliok and matzew August 3, 2023 09:06
@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Aug 3, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: ReToCode
Once this PR has been reviewed and has the lgtm label, please assign mgencur for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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

{{ end }}
spec:
serviceAccountName: "{{ .name }}"
serviceAccountName: "{{ .namespace }}"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Your changes are already included in this branch, in fact, this commit is reverting them

They are missing on the EKB side

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

🙈 So much for comparing left and right. Sorry about that.

@ReToCode ReToCode closed this Aug 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants