Don't include port in config when not specified.

When no port is specified the flag default of "-1" is taken.

Fixes #4305
pull/6/head
Victor Marmol 2015-02-11 13:20:51 -08:00
parent d10ae1ee89
commit 6a80553d7b
2 changed files with 9 additions and 4 deletions

View File

@ -79,10 +79,14 @@ func (BasicReplicationController) Generate(params map[string]string) (runtime.Ob
if err != nil { if err != nil {
return nil, err return nil, err
} }
controller.Spec.Template.Spec.Containers[0].Ports = []api.Port{
{ // Don't include the port if it was not specified.
ContainerPort: port, if port > 0 {
}, controller.Spec.Template.Spec.Containers[0].Ports = []api.Port{
{
ContainerPort: port,
},
}
} }
} }
return &controller, nil return &controller, nil

View File

@ -34,6 +34,7 @@ func TestGenerate(t *testing.T) {
"name": "foo", "name": "foo",
"image": "someimage", "image": "someimage",
"replicas": "1", "replicas": "1",
"port": "-1",
}, },
expected: &api.ReplicationController{ expected: &api.ReplicationController{
ObjectMeta: api.ObjectMeta{ ObjectMeta: api.ObjectMeta{