kubernetes/api/types.go (20 lines): - line 24: // TODOs are ignored from the parser. (e.g. TODO(andronat):... || TODO:...) iff - line 200: // TODO: replace map[string]string with labels.LabelSet type - line 388: // TODO: implement w/ DeletableVolumePlugin - line 556: // TODO: how do we prevent errors in the filesystem from compromising the machine - line 647: // TODO: how do we prevent errors in the filesystem from compromising the machine - line 674: // TODO: how do we prevent errors in the filesystem from compromising the machine - line 733: // TODO: how do we prevent errors in the filesystem from compromising the machine - line 988: // TODO: pass structured data to these actions, and document that data here. - line 997: // TODO: implement a realistic TCP lifecycle hook - line 1134: // TODO: add LastTransitionTime, Reason, Message to match NodeCondition api. - line 1655: // TODO: This should allow hostname or IP, See #4447. - line 1962: // TODO: merge w/ PodExecOptions below for stdin, stdout, etc - line 1992: // TODO: This is largely identical to PodAttachOptions above, make sure they stay in sync and see about merging - line 2086: // TODO: move to apiserver - line 2214: // TODO: this design is not final and this field is subject to change in the future. - line 2223: // TODO: Add other useful fields. apiVersion, kind, uid? - line 2243: // TODO: Decide whether to store these separately or with the object they apply to. - line 2255: // TODO: provide exact specification for format. - line 2259: // TODO: decide on maximum length. - line 2294: // TODO: Undelete if needed: Items []runtime.RawExtension `json:"items"` maintner/github.go (18 lines): - line 47: // TODO: more validation. whatever GitHub requires. - line 323: // TODO: keep these sorted in the corpus - line 350: // TODO: keep these sorted in the corpus - line 413: // TODO: color? - line 555: // TODO: parse it and see if we've since learned how - line 583: // TODO: this struct is a little wide. change it to an interface - line 709: // TODO: parse it and see if we've since learned how - line 725: // TODO: this field is dumb. break it down. - line 964: // TODO: add other fields here when we add them for user. - line 1094: // TODO: emit a StringChange if we ever have a problem that we - line 1110: // TODO: detect milestone renames and emit mutation for that? - line 1480: // TODO: verify that the httpcache package guarantees that the - line 1725: // TODO: use res.Rate (sleep until Reset if Limit == 0) - line 1768: // TODO: return an error type up that the sync loop can use - line 1980: // TODO: does a reaction update a comment's UpdatedAt time? - line 2170: // TODO: store these two more compactly: - line 2416: // TODO: store these two more compactly: - line 2559: // TODO: use FindAllStringSubmatchIndex instead, so we can maintner/gerrit.go (12 lines): - line 8: // TODO: write about Gerrit's Git API. - line 130: // TODO: rename this method. - line 227: // TODO: support prefix lookups. build a trie. But - line 345: // TODO: Merge the *GitPerson object here and for a person's Git commits - line 452: // TODO: git footers are treated as multimaps. Account for this. - line 523: // TODO: make this as small as - line 532: // TODO: remove ref from gerrit.clsReferencingGithubIssue - line 586: // TODO: option to ignore mutation if user isn't interested. - line 591: // TODO: option to ignore mutation if user isn't interested. - line 765: // TODO: this doesn't delete change refs (from - line 1509: // TODO: implement Gerrit's vote copying. For example, - line 1518: // TODO: this should really use cmd/coordinator/buildstatus.go (11 lines): - line 222: // TODO: base this on historical measurements, instead of statically configured. - line 223: // TODO: move this to dashboard/builders.go? But once we based on on historical - line 238: // TODO: move this to dashboard/builders.go? But once we based on on historical - line 260: return 0 // TODO: remove this once we're using VMware - line 603: // TODO: buildlet instance name - line 685: // Don't wrap this error. TODO: use xerrors. - line 698: // TODO: currently we ditch this buildlet when we're done with - line 1243: // TODO: migrate to a GKE internal load balancer with an internal static IP - line 1689: Output: &buf, // see "maybe stream lines" TODO below - line 1725: // TODO: maybe stream lines (set Output to a line-reading - line 1931: // TODO: give it two tries at least later (store state dashboard/builders.go (9 lines): - line 25: // TODO: we'll likely expand this, or move it, or change the matching - line 537: HermeticReverse: false, // TODO: run in chroots with overlayfs? https://github.com/golang/go/issues/34830#issuecomment-543386764 - line 870: // TODO: unexport this and make buildsRepoAtAll return false on too-old - line 1033: // TODO: consider using distTests? We never did before, but - line 1680: // TODO: Issue 25963, get the misc-compile trybots for Android/iOS. - line 1722: tryBot: nil, // TODO: add a func to conditionally run this trybot if compiler dirs are touched - line 1771: tryBot: nil, // TODO: add a func to conditionally run this trybot if compiler dirs are touched - line 2885: // TODO: dev.boringcrypto.go1.N branches may be the - line 2952: // TODO: more. Look at bigquery results once we have more data. cmd/gopherbot/gopherbot.go (7 lines): - line 556: // TODO: check for gopherbot as author? check for exact match? - line 605: // TODO: check for gopherbot as author? check for exact match? - line 1015: if strings.Contains(gi.Title, "gccgo") { // TODO: better gccgo bug report heuristic? - line 1186: // TODO: write a task that reopens issues if the OP speaks up. - line 1315: // TODO: Improve this message. Some ideas: - line 1426: // TODO: once gopherbot starts posting these messages and we - line 2365: // TODO: if maintner isn't fast enough (or is too fast) and it re-runs this cmd/coordinator/remote.go (7 lines): - line 264: // TODO: delete !wantStream support 3-6 months after 2019-11-19. - line 409: // TODO: deal with HTTP/2 requests if https://farmer.golang.org enables it later. - line 430: // TODO: implement support for non-VM types if/when needed. - line 521: const listenAddr = ":2222" // TODO: flag if ever necessary? - line 576: // TODO: make ListenAndServe errors Fatal, once it has a proven track record. starting paranoid. - line 725: fmt.Fprintf(s, "# - env: %s\n", strings.Join(bconf.Env(), " ")) // TODO: shell quote? - line 806: // TODO: caching, rate limiting. internal/coordinator/schedule/schedule.go (7 lines): - line 116: // TODO: recognize certain pools like the reverse pool - line 151: ctx := context.Background() // TODO: make these cancelable and cancel unneeded ones earlier? - line 291: // TODO: flesh out this policy more. For now this is much - line 294: // TODO: consider SchedItem.Branch. - line 295: // TODO: pass in a context to schedLess that includes current time and current - line 309: // TODO: if IsTry, consider how many TryBot requests - line 344: tryFor string // TODO: which user. (user with 1 trybot >> user with 50 trybots) cmd/coordinator/status.go (6 lines): - line 243: // TODO: add a JSON mode to gitmirror so we don't need to parse HTML. - line 674: // TODO: make this prettier. - line 782: GCEPoolStatus template.HTML // TODO: embed template - line 783: EC2PoolStatus template.HTML // TODO: embed template - line 784: KubePoolStatus template.HTML // TODO: embed template - line 785: ReversePoolStatus template.HTML // TODO: embed template cmd/makemac/makemac.go (5 lines): - line 235: // TODO: Add a new GOS definition for darwin19_64 (macOS 10.15) in HWV >= 17 - line 237: // TODO: investigate updating the guestid when we upgrade vSphere past version 6.7u3. - line 673: // TODO: do these all concurrently. - line 785: // TODO: improve this logic now that the coordinator has a - line 828: // TODO: probably more status, as needed. cmd/buildlet/buildlet.go (4 lines): - line 832: // TODO: ignore these for now. They were breaking x/build tests. - line 1480: // TODO: use a more explicit env variable or on-disk signal - line 1515: // TODO: using sshd -D isn't great as it only - line 1657: // TODO: setup RAM disk for tmp and set *workDir buildenv/envs.go (4 lines): - line 113: // TODO: rename. this is not just for buildlets; also for bootstrap. - line 178: // TODO: this method used to do much more. maybe remove it - line 179: // when TODO below is addressed, pushing scopes to caller? Or - line 187: // TODO: remove compute and DevstorageFullControlScope once verified to work cmd/updatecontrib/updatecontrib.go (3 lines): - line 33: // TODO: automatically use Gerrit names like we do with GitHub - line 171: // TODO: print for debugging, shouldn't happen - line 177: // TODO: print for debugging, shouldn't happen devapp/owners/owners.go (3 lines): - line 90: // TODO: increment expvar for monitoring. - line 103: // TODO: increment expvar for monitoring. - line 120: // TODO: increment expvar for monitoring. cmd/coordinator/coordinator.go (3 lines): - line 535: // TODO: panic if status[work] already exists. audit all callers. - line 904: // TODO: remove this hard-coded 15 second ticker and instead - line 2145: // TODO: this a transitional step on our way towards merging build.golang.org into maintner/maintnerd/maintapi/api.go (3 lines): - line 55: // TODO: wait for it? kick off a fetch of it and then answer? - line 403: // TODO: This special case can be removed when Go 1.17 is out and 1.15 is no longer supported. - line 844: // TODO: care about returning the history from both maintner/maintpb/maintner.proto (3 lines): - line 60: // TODO: allow num or title to be used if Github only returns those? So far unneeded. - line 215: string body = 3; // may not be present in edits later (if only reactions changed? TODO: investigate) - line 243: // TODO: code.googlesource.com and github repos later. kubernetes/client.go (3 lines): - line 179: // TODO: path escape serviceName? - line 380: getURL := c.nsEndpoint() + "pods/" + podName // TODO: escape podName? - line 412: url := c.nsEndpoint() + "pods/" + podName + "/log" // TODO: escape podName? maintner/netsource.go (3 lines): - line 316: // TODO: optimization: if already on GCE, skip sync to disk part and just - line 476: // TODO: add a constructor for this? or simplify it. make it Size + - line 564: // TODO: this is a quadratic amount of write I/O as the 16 MB maintner/git.go (3 lines): - line 217: // TODO: temporary vs permanent failure? reschedule? fail hard? - line 338: // TODO: care about m.Repo? - line 509: // TODO: support prefix lookups. build a trie. But gerrit/gerrit.go (2 lines): - line 299: // TODO: "avatars" is also returned, but not added here yet (add if required) - line 363: // TODO: more internal/coordinator/schedule/span.go (2 lines): - line 33: el pool.EventTimeLogger // where we log to at the end; TODO: this will change - line 94: // TODO: This is a copy of the function in cmd/coordinator/status.go. This should be removed once status cmd/gopherstats/gopherstats.go (2 lines): - line 235: // TODO: do this allocation-free? preculate them in init? - line 1181: // TODO: get from GCE metadata, etc. cmd/cl/cl.go (2 lines): - line 59: TODO: Support do-not-review, output as JSON. - line 345: // TODO: Support do not review, close, and postpone to next go release maintner/maintnerd/apipb/api.proto (2 lines): - line 32: // TODO: github, if/when needed. - line 107: // TODO: deprecate, replace with time or commit continuation token. cmd/releasebot/main.go (2 lines): - line 524: // TODO: go tool api -allow_new=false - line 715: // TODO: print sha256 cmd/updatestd/updatestd.go (2 lines): - line 68: // TODO: Consider a future enhancement of fetching build status for all - line 126: // TODO: Maybe do GOBIN=$(mktemp -d) go install golang.org/x/tools/cmd/bundle@version or so, internal/coordinator/pool/reverse.go (2 lines): - line 87: // TODO: switch to a map[hostType][]buildlets or map of set. - line 470: return "TODO: some reverse buildlet summary" internal/buildgo/basepin.go (2 lines): - line 133: // TODO: clean this up with respect to status updates & logging. - line 136: // TODO: move logging to Client c.logger. and add Client.WithLogger shallow copier. buildlet/buildletclient.go (2 lines): - line 621: // TODO: update docs & callers to just use the context value. - line 756: // TODO: break into parsed fields? kubernetes/api/quantity.go (2 lines): - line 72: // TODO: Remove above disclaimer after all bikeshedding about format is over, - line 265: // TODO: If BinarySI formatting is requested but would cause rounding, upgrade to maintner/maintnerd/maintnerd.go (2 lines): - line 193: // TODO: if Initialize only partially syncs the data, we need to delete - line 287: // TODO: add these to the repos package at some point? Or buildlet/remote.go (2 lines): - line 108: // TODO: accept a context for deadline/cancelation - line 119: // TODO: delete this once the server's been deployed with it. cmd/perfrun/perfrun.go (2 lines): - line 81: // TODO: Overhead of multiple Exec calls might be significant; should we ship over a shell script to do this in one go? - line 108: // TODO: defer this so we don't leak clients? internal/coordinator/pool/kube.go (2 lines): - line 212: // TODO: clean these in cleanupOldPods once they're - line 217: // TODO: clean these in cleanupOldPods once they're maintner/maintner.go (2 lines): - line 179: // TODO: figure out if this is accurate. - line 343: // TODO: handle errors better? failing is only safe option. maintner/maintnerd/apipb/api.pb.go (1 line): - line 422: // TODO: deprecate, replace with time or commit continuation token. version/version.go (1 line): - line 235: // TODO: return the same exit status maybe. cmd/relnote/relnote.go (1 line): - line 153: fmt.Printf("\n

\n TODO: %s: %s\n

\n", cmd/xb/xb.go (1 line): - line 190: // TODO: return with exact exit status? when needed. devapp/server.go (1 line): - line 74: s.mux.Handle("/owners/", http.RedirectHandler("/owners", http.StatusPermanentRedirect)) // TODO: remove after clients updated to use URL without trailing slash maintner/maintpb/maintner.pb.go (1 line): - line 158: // TODO: allow num or title to be used if Github only returns those? So far unneeded. cmd/coordinator/internal/legacydash/handler.go (1 line): - line 230: // TODO: correctly validate a hash: check that it's exactly 40 devapp/title.go (1 line): - line 48: // TODO: Parse "image/{png,jpeg}" as ["image/png", "image/jpeg"], maybe? buildlet/gce.go (1 line): - line 219: // TODO: catch Code=="QUOTA_EXCEEDED" and "Message" and return cmd/gerritbot/gerritbot.go (1 line): - line 858: // TODO: check for exact match? cmd/gomote/list.go (1 line): - line 121: log.Fatalf("Builder type %q not known to this gomote binary. Update your gomote binary. TODO: teach gomote to fetch build configs from the server (Issue 30929)", builderType) cmd/rundockerbuildlet/rundockerbuildlet.go (1 line): - line 93: time.Sleep(time.Second) // TODO: docker wait on the running containers? maintner/cmd/maintserve/maintserve.go (1 line): - line 93: // TODO: Implement background updates for corpus while the application is running. internal/gitauth/gitauth.go (1 line): - line 42: if proj != "symbolic-datum-552" { // TODO: don't hard-code this; use buildenv package maintner/logger.go (1 line): - line 93: // TODO: support subsequent Update? for now we only cmd/racebuild/racebuild.go (1 line): - line 42: // TODO: use buildlet package instead of calling out to gomote. maintner/maintnerd/gcslog/gcslog.go (1 line): - line 470: // TODO: that^ kubernetes/dialer.go (1 line): - line 28: // TODO: cache the result of GetServiceEndpoints, at least for cmd/coordinator/internal/legacydash/ui.go (1 line): - line 460: // TODO: this doesn't include the TagState commit. It would be internal/coordinator/log/log.go (1 line): - line 59: // TODO: version, who deployed, CoreOS version, Docker version, types/types.go (1 line): - line 179: // TODO: add number of active builds, and number of builds