Delete project should delete all project related resources

[derekwaynecarr]

@smarterclayton @liggitt @jwforres - please take a quick look.

this builds off the cherry pick from:
#1570

[derekwaynecarr]

It would have helped if my pushed branch included the unit testing changes.

Summary of changes:

1. When running embedded Kubernetes, namespace controller is started to handle life-cycle.
2. Update origin controller to now do finalize correctly when it purges OpenShift content.
3. Update unit tests for origin controller.
4. Align Project REST patterns with latest upstream.
5. Added support for editing a project (since we are now on latest patterns)
6. Added tests to ensure our strategy does the right thing on create vs update to align with upstream.

@smarterclayton @jwforres - this should be good to review now. this fixes the inability to delete a project in OpenShift right now ;-)

[derekwaynecarr]

@deads2k - if you have time, I just need someone to review ;-)

[deads2k]

This seems slow. Does this mean that my namespace may not be cleaned up for a minute? During that minute, can I keep creating things?

[derekwaynecarr]

This means you will see a namespace appear in "Terminating" status for ~1 min. The value is the same default as upstream. You are unable to create content in a terminating namespace. There are two controllers that need to run in order to delete content (the k8s one, and the openshift one) so it will take ~1-2 minutes for something to truly be gone.

[deads2k]

Minor comments. Otherwise, lgtm

[derekwaynecarr]

Assuming no other comments will look to merge in morning. Thanks for review David

Sent from my iPhone

On Apr 6, 2015, at 4:13 PM, David Eads notifications@github.com wrote:

Minor comments. Otherwise, lgtm

—
Reply to this email directly or view it on GitHub.

[deads2k]

after strategies move to pkg/project/registry/project/rest.go, lgtm.

[derekwaynecarr]

changes made per request.

will merge this now.

[derekwaynecarr]

[merge]

[openshift-bot]

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_openshift3/1497/) (Image: devenv-fedora_1224)

[derekwaynecarr]

re [merge]

[openshift-bot]

Evaluated for origin up to cc1b809

[deads2k]

Which transient did you fail on?