Created
January 19, 2018 04:16
-
-
Save eriknelson/90b23355ec5c8bc2aeb2f913682b25db to your computer and use it in GitHub Desktop.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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