Skip to content

Instantly share code, notes, and snippets.

@eriknelson
Created January 19, 2018 04:16
Show Gist options
  • Save eriknelson/90b23355ec5c8bc2aeb2f913682b25db to your computer and use it in GitHub Desktop.
Save eriknelson/90b23355ec5c8bc2aeb2f913682b25db to your computer and use it in GitHub Desktop.
handler/io.go:54: // TODO: ke.StatusError import is not working, compiler thinks ke.StatusError is undefined...
clients/etcd.go:110: // TODO: Config validation
clients/etcd.go:149: // TODO: Determine if transport needs optimization
clients/zz_generated.deepcopy.go:3:// TODO: We need to remove this file by using the openshift api client.
handler/handler.go:62:// TODO: implement asynchronous operations
handler/handler.go:75: // TODO: determine what to do with the Principal. We don't really have a
handler/handler.go:177: // TODO: Reintroduce router restriction based on API version when settled upstream
handler/handler.go:234: // TODO: typically the methods on the broker return a response this
broker/types.go:240:// TODO: What belongs on this thing?
broker/broker.go:85: // TODO: consider returning a struct + error
broker/broker.go:176:// TODO: Response here? Async?
broker/broker.go:177:// TODO: How do we handle a large amount of data on this side as well? Pagination?
broker/broker.go:406: // TODO: ExtractCredentials is doing more than it should
broker/broker.go:417: // TODO: do we restart the job or mark it as failed?
broker/broker.go:468: // TODO: Should we blow up or warn and continue?
broker/broker.go:522: // -> TODO: Check to see if the spec supports or requires async, and reconcile
broker/broker.go:666: // TODO: do we want to do synchronous provisioning?
broker/broker.go:680: // TODO: What data needs to be sent back on a response?
broker/broker.go:745: // TODO: do we want to do synchronous deprovisioning?
broker/broker.go:951: // TODO: decide if we need to keep this at all
broker/broker.go:973: // TODO: if we're sync we need to return the Credentials, otherwise the
broker/broker.go:1148: // -> TODO: Check to see if the spec supports or requires async, and reconcile
broker/broker.go:1189: // TODO -- HACK!: Update will report a 202 if it finds any jobs
broker/broker.go:1311: // TODO: do we want to do synchronous updating?
broker/broker.go:1374: // TODO:validate the format to avoid some sort of injection hack
auth/auth.go:51: // TODO: add roles?
auth/auth.go:113: // TODO: add some error checking
auth/auth.go:188: // TODO: really need to figure out a better way to define what
registries/adapters/adapter.go:45:// TODO: needs to remain ansibleapp UNTIL we redo the apps in dockerhub
origin/copy/authorization/types.go:79:// TODO make this the authoritative rule
app/app.go:160: // TODO: Let's take all these validations and delegate them to the client
app/app.go:271:// TODO: Make this a go routine once we have a strong and well tested
app/app.go:285: // TODO: probably return an error or some sort of message such that we can
app/app.go:381: //TODO: Add Flag so we can still use the old way of doing this.
apb/unbind.go:26:// TODO: Figure out the right way to allow apb to log
apb/deprovision.go:40: // TODO: Move this validation to a Spec creation function (yet to be created)
apb/executor.go:152: // TODO: Probably some more permutations of these that should be validated?
apb/executor.go:198:// TODO: Instead of putting namespace directly as a parameter, we should create a dictionary
apb/provision_or_update.go:45: // TODO: Move this validation to a Spec creation function (yet to be created)
apb/types.go:283:// TODO: Remove in favor of calling the same method.
runtime/runtime_test.go:46: //TODO: Write tests for ValidateRuntime using the fake kubernetes client
runtime/runtime_test.go:51: //TODO: Write tests for CreateSandbox using the fake kubernetes client
runtime/runtime_test.go:56: //TODO: Write tests for DestroySandbox using the fake kubernetes client
dao/dao.go:70:// TODO: Streaming interface? Going to need to optimize all this for
dao/dao.go:124: // TODO: Is there no batch insert in the etcd api?
dao/dao.go:158: // TODO: Is there no batch insert in the etcd api?
config/config.go:264: //TODO: we should eventually add this back to the original map,
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment