Skip to content
This repository was archived by the owner on Oct 7, 2020. It is now read-only.

add serviceAnnotation to KubernetesResourcesSpec.#740

Merged
istio-testing merged 1 commit intoistio:masterfrom
morvencao:br_add_service_annotation_support
Jan 12, 2020
Merged

add serviceAnnotation to KubernetesResourcesSpec.#740
istio-testing merged 1 commit intoistio:masterfrom
morvencao:br_add_service_annotation_support

Conversation

@morvencao
Copy link
Copy Markdown
Member

@morvencao morvencao commented Jan 9, 2020

resubmit: #736 due to #713
depends on: istio/api#1230

@morvencao morvencao requested a review from a team as a code owner January 9, 2020 05:01
@googlebot googlebot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Jan 9, 2020
@istio-testing istio-testing added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jan 9, 2020
@elfinhe
Copy link
Copy Markdown
Member

elfinhe commented Jan 9, 2020

lgtm

Copy link
Copy Markdown
Member

@elfinhe elfinhe left a comment

Choose a reason for hiding this comment

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

maybe we need to wait until the tests on master gets fixed?

@morvencao
Copy link
Copy Markdown
Member Author

morvencao commented Jan 10, 2020

@elfinhe These failed tests depend on istio/api#1230

@istio-testing istio-testing added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jan 11, 2020
@morvencao morvencao force-pushed the br_add_service_annotation_support branch 2 times, most recently from 3853168 to 85daebc Compare January 11, 2020 01:07
@morvencao
Copy link
Copy Markdown
Member Author

/test test_operator

@morvencao morvencao force-pushed the br_add_service_annotation_support branch from 85daebc to 66ceef1 Compare January 12, 2020 13:13
@istio-testing istio-testing added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jan 12, 2020
@istio-testing istio-testing merged commit dc2f1b3 into istio:master Jan 12, 2020
@morvencao morvencao deleted the br_add_service_annotation_support branch January 12, 2020 13:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants