Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bugzilla 1261860] fix adding labels to build-pod #4983

Closed

Conversation

php-coder
Copy link
Contributor

Fix for https://bugzilla.redhat.com/show_bug.cgi?id=1261860

I've fixed adding labels to build-pod. @rhcarvalho PTAL.

One doubt that I have is we should add all labels? Because after this fix I see the following

[vagrant@openshiftdev sample-app]$ oc new-app application-template-stibuild.json -l bug=1261860 
...
[vagrant@openshiftdev sample-app]$ oc describe pod ruby-sample-build-1-build | grep Labels
Labels:             bug=1261860,buildconfig=ruby-sample-build,name=ruby-sample-build,openshift.io/build.name=ruby-sample-build-1,template=application-template-stibuild

As we can see there is one mine label, one standard and 3 additional. I'm not sure about these extra labels. Do we need them?

}

// append adds elements from one map to another
func append(source *map[string]string, dest *map[string]string) {
Copy link
Contributor

Choose a reason for hiding this comment

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

i'd be surprised if we don't already have this function somewhere else.

@bparees
Copy link
Contributor

bparees commented Oct 7, 2015

yes i think copying all the labels from the build into the build pod is ok, it ties them together in a useful way.

@liggitt
Copy link
Contributor

liggitt commented Oct 7, 2015

Wait, didn't we decide to stop adding all labels to the build pod a while ago? Why are we readding them now

@liggitt
Copy link
Contributor

liggitt commented Oct 7, 2015

See #3502, #3508

@bparees
Copy link
Contributor

bparees commented Oct 7, 2015

hm. @liggitt is correct. we should just close the associated bug as won't fix.

@php-coder php-coder closed this Oct 7, 2015
@php-coder
Copy link
Contributor Author

we should just close the associated bug as won't fix.

done

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.

3 participants