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

WIP: Make fake broker act like an OSB broker #533

Closed
wants to merge 1 commit into from

Conversation

pmorie
Copy link
Contributor

@pmorie pmorie commented Mar 13, 2017

🚧 WIP 🚧

The OSB fake server should:

  1. Correctly implement the OSB API
  2. Allow you to record a set of reactions for different instances/bindings
  3. Allow you to make assertions on the expected actions that should have happened to the fake

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Mar 13, 2017
@pmorie pmorie changed the title Make fake broker act like as OSB broker WIP: Make fake broker act like as OSB broker Mar 13, 2017
@pmorie
Copy link
Contributor Author

pmorie commented Mar 14, 2017

Still has a ton of work left, keep in mind if doing review.

@pmorie pmorie added this to the MVP 2 milestone Mar 17, 2017
@pmorie
Copy link
Contributor Author

pmorie commented Mar 22, 2017

I doubt that I'm going to get back to this before kubecon.

@pmorie pmorie changed the title WIP: Make fake broker act like as OSB broker WIP: Make fake broker act like an OSB broker Apr 2, 2017
@pmorie pmorie force-pushed the fake-broker-server branch from b7978d5 to c7f187d Compare April 2, 2017 15:36
@arschles arschles modified the milestones: 0.0.2, MVP 2 Apr 3, 2017
@pmorie pmorie modified the milestones: 0.0.3, 0.0.2 Apr 6, 2017
@pmorie
Copy link
Contributor Author

pmorie commented Apr 6, 2017

This will not be done for 0.0.2

@pmorie
Copy link
Contributor Author

pmorie commented Apr 14, 2017

Moving to next alpha release. Someone besides me probably needs to own this.

@arschles arschles modified the milestones: 0.1.0, 0.0.4 Apr 21, 2017
@arschles arschles added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 21, 2017
@arschles
Copy link
Contributor

arschles commented May 10, 2017

@pmorie what is the status of this PR? And/or, who is owning this?

@pmorie
Copy link
Contributor Author

pmorie commented May 12, 2017

@arschles Someone else needs to adopt it. It might be a good place for someone new to get started. I'm going to make an issue for this and probably close this PR.

@pmorie
Copy link
Contributor Author

pmorie commented May 15, 2017

I'm closing this now. I will probably spin this into a separate repo.

@pmorie pmorie closed this May 15, 2017
arschles added a commit to arschles/kubernetes-service-catalog that referenced this pull request Jun 9, 2017
This is a continuation of
kubernetes-retired#533, and
is a pre-requisite for
kubernetes-retired#923
pmorie pushed a commit that referenced this pull request Jun 13, 2017
* Adding a fake broker server

This is a continuation of
#533, and
is a pre-requisite for
#923

* converting a single test to use the broker server

* adding boilerplate and docs

* naming the pivotal brokerapi package

to eliminate possible ambiguity

* adding clarifying comment to the catalog requests var

* adding better godocs
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. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants