github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-08-11 21:12:07 +00:00
Code Issues Projects Releases Wiki Activity

remove TODO: validate namespace

Browse Source
This commit is contained in:
mqliang 2015-11-26 15:54:42 +08:00
parent 7644d34759
commit 8547a5c1ae
2 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
pkg/api/validation/events.go
View File

@ -24,7 +24,11 @@ import (
// ValidateEvent makes sure that the event makes sense.
func ValidateEvent(event *api.Event) validation.ErrorList {
allErrs := validation.ErrorList{}
// TODO: There is no namespace required for node.
// There is no namespace required for node.
if event.InvolvedObject.Kind == "Node" &&
event.Namespace != "" {
allErrs = append(allErrs, validation.NewInvalidError("involvedObject.namespace", event.InvolvedObject.Namespace, "namespace is not required for node"))
}
if event.InvolvedObject.Kind != "Node" &&
event.Namespace != event.InvolvedObject.Namespace {
allErrs = append(allErrs, validation.NewInvalidError("involvedObject.namespace", event.InvolvedObject.Namespace, "namespace does not match involvedObject"))

2
test/integration/auth_test.go
View File

@ -173,7 +173,7 @@ var aEvent string = `
"name": "a"%s
},
"involvedObject": {
"kind": "Node",
"kind": "Pod",
"namespace": "default",
"name": "a",
"apiVersion": "v1"