Skip to content

Conversation

@stang
Copy link
Contributor

@stang stang commented Jan 18, 2019

Tentative of implementation of inherit_labels

see: #456

stang added 2 commits January 18, 2019 16:24
  Signed-off-by: Stephane Tang <hi@stang.sh>
Signed-off-by: Stephane Tang <hi@stang.sh>
@sdudoladov
Copy link
Member

@stang is there a chance that existing cluster_labels will work for you ?

@coveralls
Copy link

coveralls commented Jan 18, 2019

Coverage Status

Coverage remained the same at 23.705% when pulling 33dd488 on stang:add-inherited-labels into f2dddb0 on zalando-incubator:master.

@stang
Copy link
Contributor Author

stang commented Jan 18, 2019

@sdudoladov - sorry, I was replying on the Issue (see: #456 (comment))

TL;DR: I don't think the cluster_labels would suit our use-case.

@sdudoladov
Copy link
Member

@stang I left a couple of doc-related comments, otherwise I am happy with the change.

@stang stang force-pushed the add-inherited-labels branch from c382453 to 0a6f8d2 Compare February 3, 2019 05:40
@stang stang requested a review from FxKu as a code owner February 3, 2019 05:40
@stang stang force-pushed the add-inherited-labels branch 2 times, most recently from 340795c to e141ae0 Compare February 3, 2019 05:59
Copy link
Member

@FxKu FxKu left a comment

Choose a reason for hiding this comment

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

I like your contribution as well :)

Signed-off-by: Stephane Tang <hi@stang.sh>
@stang stang force-pushed the add-inherited-labels branch from e141ae0 to 33dd488 Compare February 7, 2019 17:47
Copy link
Member

@sdudoladov sdudoladov left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution, @stang

@sdudoladov sdudoladov merged commit d11b23b into zalando:master Feb 14, 2019
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.

5 participants