Skip to content
This repository has been archived by the owner on May 6, 2022. It is now read-only.

Update instance loop support for namespaced classes and plans #2133

Merged
merged 24 commits into from
Jun 26, 2018

Conversation

jmrodri
Copy link
Contributor

@jmrodri jmrodri commented Jun 19, 2018

No description provided.

@k8s-ci-robot k8s-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Jun 19, 2018
@k8s-ci-robot
Copy link
Contributor

Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please follow instructions at https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to sign the CLA.

It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please sign in with your organization's credentials at https://identity.linuxfoundation.org/projects/cncf to be authorized.
  • If you have done the above and are still having issues with the CLA being reported as unsigned, please email the CNCF helpdesk: [email protected]

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. label Jun 19, 2018
@jmrodri
Copy link
Contributor Author

jmrodri commented Jun 19, 2018

I've signed it

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. and removed cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels Jun 19, 2018
@eriknelson
Copy link
Contributor

Blocked by #2137, which explicitly allows namespaced ref updates to pass through the instance's references subresource.

@@ -25,3 +25,4 @@ contrib/examples/consumer/Gopkg.lock
contrib/examples/consumer
contrib/examples/vendor/*
integration.test*
debug.test*
Copy link
Contributor

Choose a reason for hiding this comment

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

where is this coming from?

Copy link
Contributor

Choose a reason for hiding this comment

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

Was some kind of binary that showed up after running a failed make command, I can't remember which. I'm assuming we would prefer not to have that checked in.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah I've seen those pop up too after running the tests.

@eriknelson
Copy link
Contributor

/retest

Copy link
Contributor

@jeremyrickard jeremyrickard left a comment

Choose a reason for hiding this comment

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

LGTM

@n3wscott n3wscott added the LGTM2 label Jun 25, 2018
@carolynvs carolynvs merged commit 36ae67e into kubernetes-retired:master Jun 26, 2018
kikisdeliveryservice pushed a commit to kikisdeliveryservice/service-catalog that referenced this pull request Jun 29, 2018
…etes-retired#2133)

* Initial ns instance loop updates

* Disambiguate getServiceInstanceWithRefs

* Update .gitignore to ignore debug.test

* Disambiguate getTestServiceInstanceWithClusterRefs

* Also adds some namespaced helpers

* Add namespaces to the test type producers

* Add TestReconcileServiceInstanceWithNamespacedRefs

* Buildable prepareUpdateInstanceRequest

* Passing cp

* Add sync provision test for namespaced class/plan

* Add reconcile async with namespaced refs

* Give instance tests their own file

* Add ResolveNamespacedReferences test

* Add instance DeleteWithNamespacedRefs sync test

* Add instance DeleteAsyncWithNamespaced refs test

* Fix in progress polling for instances

* Add inprogress provision polling test

* Cover the nil plan case for last op reqs

* Add success async provision ns type test

* check properties for nil early

* Add inprogress provision async failure ns test

* Add inprogress depro poll ns test

* Add polling success deprovision ns test

* Add failed polling deprovision ns test

* Update new files with 2018 copyright header
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. LGTM1 LGTM2 size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants