Also, from here https://github.com/kubernetes/kubernetes/blob/master/pkg/master/master.go#L522

From what I understand, if I try to register a new Third Party resource whose group already exists, it considers that thirdpartyreource to already exist and would not take any action to register the new resource.

cc @brendandburns @wojtek-t

@lavalamp @kubernetes/sig-api-machinery

Also, once I create a thirdpartyresource and delete it, for some reason I have not been able to recreate any other thirdparty resources. I am running master on head using vagrant.

Looks like bugs. Thanks for reporting.

The gist of the problem is that, there is no way we can modify a webservice with more routes and storages once it is installed. So we have to delete and reinstall the webservice with all the right things whenever a new thirdpartyresource is to be installed. But the removal of the webservice from the container is leaving some trails in the ServeMux.m map, which is a map of routes to handler functions.

go-restful does 2 things whenever you want to add a new webservice to the container.

1. add the webservice to the registeredwebservices list and
2. register a handle function. .

But, during a remove, it undos the 1 but not 2.

From the go-restful perspective, if we add a webservice and remove it and if I try to add it back, it results in a panic.

I will try to reproduce it with a simple http application using go-restful and will file an issue if I am able to reproduce the issue.

Thanks for all your effort, Uday!

On Wed, Apr 6, 2016 at 4:51 PM, Uday Ruddarraju notifications@github.com
wrote:

I will try to reproduce it with a simple http application using go-restful
and will file an issue if I am able to reproduce the issue.

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#23831 (comment)

Was able to reproduce the issue with just go-restful today, where I was able to successfully add a webservice to a container, remove it and when I add it back, I see the following issue:

panic: http: multiple registrations for /users

I tried the example here: https://github.com/emicklei/go-restful/blob/master/examples/restful-user-resource.go. And changed the lines https://github.com/emicklei/go-restful/blob/master/examples/restful-user-resource.go#L74 to do,

    container.Add(ws)
    container.Remove(ws)
    container.Add(ws)

This created the panic above.

This explains why a thirdparty registration followed by unregistration, cannot be registered back (Which I think is critical for us.)

@lavalamp Created an issue with go-restful: emicklei/go-restful#274. Please add any details if I may have missed something.

There was an other issue with the RemoveRoute call in go-restful for webservices having more than one route. Fixed it in emicklei/go-restful#275

@lavalamp I still have to submit a fix to fix this. The PR only updates the godep :)

/sub

FYI - I'm getting similar errors even when I use a different API group (domain name on the resources).

Here's a gist with a simple test showing the issue: https://gist.github.com/duglin/ffe983ba6f15ad8f1bad968ccbcdcba3

I think this one should be reopened, the problem is still there on c7ec11b

I believe that #29724 should have the fix, can you test it out?

Thanks @brendandburns. The added test-cmd.sh test in #29724 looks proper.

Closing this since #29724 is merged.