pkg/controller/direct/resourcemanager/tagkey_controller.go (12 lines): - line 79: // TODO: Just fetch this object? - line 87: // TODO: How do we do resource acquisition - maybe by shortname? - line 131: // TODO: Delete via status selfLink? - line 147: // TODO: Do we need to check that it was deleted? - line 159: // TODO: Should be ref - line 206: // TODO: Should be metav1.Time (?) - line 222: // TODO: Skip updates at the higher level if no changes? - line 233: // TODO: Where/how do we want to enforce immutability? - line 249: // TODO: Do we need to check that the operation succeeeded? - line 252: // TODO: Return updated object status - line 265: // TODO: Just fetch this object? - line 270: // TODO: Merge to avoid overwriting conditions? pkg/controller/direct/apikeys/apikeyskey_controller.go (12 lines): - line 62: Ignore("name"), // TODO: Should be ResourceID? - line 63: Ignore("annotations"), // TODO: Should not ignore - line 109: // TODO: Just fetch this object? - line 120: // TODO: Use name or request resourceID to be set on create? - line 172: // TODO: Delete via status selfLink? - line 222: // TODO: Is the resourceID returned if it is dynamically created? Maybe we should create the UUID? - line 228: // TODO: Return created object - line 236: // TODO: Skip updates if no changes - line 237: // TODO: Where/how do we want to enforce immutability? - line 240: // TODO: I think we can do this with a helper - line 248: // TODO: Annotations - line 274: // TODO: update status in u pkg/controller/direct/alloydb/cluster_controller.go (9 lines): - line 129: // TODO: Support URLs - line 164: // TODO: Scenario test cases: both networkConfig.networkRef and networkRef set; none set. - line 254: // TODO: Scenario test case: ContinuousBackupConfig.Enabled unset. - line 287: // TODO: Scenario test case: Update initialUser.password from `value` to `valueFrom` and vise versa. - line 301: // TODO: Test once backup is supported or using scenario: set restoreBackupSource and restoreContinuousBackupSource (either and both). - line 516: // TODO: Check immutability for optional and immutable fields. - line 556: // TODO: Figure out how to keep the network immutable. - line 587: // TODO: Decide if we want to clean up default fields set in desired state. - line 654: // TODO: Scenario test case: Delete after the cluster is gone; not forcing delete a secondary cluster. pkg/controller/direct/alloydb/instance_controller.go (7 lines): - line 84: // TODO: Read instance type from observed state because it's necessary during deletion. - line 133: // TODO: Support URLs - line 252: // TODO: Change to CompareProtoMessage once we support all the files in the instance pb. - line 323: // TODO: Test case with availability type unset. - line 328: // TODO: Test "copied" behavior for read pool - line 329: // TODO: Test "overridden" behavior for read pool - line 348: // TODO: Test machineConfig unset and empty struct dev/tools/controllerbuilder/pkg/io/replace.go (7 lines): - line 43: // TODO: replace other go type like function, var, import.. - line 87: // TODO: Parse the entire package (rather than a single `file`) to avoid redeclaration errors for structs - line 120: // TODO: add comments for VAR, CONST, FUNC and IMPORT. - line 234: // TODO: Add comments to funcDecl - line 251: // TODO Add non struct type from newFile - line 346: // TODO Support ImportSpec and ValueSpec - line 460: // TODO Support other types experiments/composite/pkg/watchset/manager.go (7 lines): - line 79: // TODO: Clean up closed InterestSets from the slice - line 122: // TODO: rw-mutex - line 123: // TODO: register interests directly with watchers? - line 136: // TODO: Should we replace this with a direct pointer to the target(s)? - line 176: // TODO: ResourceVersion (maybe) - line 178: // TODO: Can we use metadata only (probably not if we have to compute the health) - line 190: // TODO: Update resource version? pkg/controller/direct/dataflow/dataflowflextemplatejob_controller.go (6 lines): - line 94: // TODO: Why don't we support resourceID? - line 119: // TODO: Move from monitoring package into shared package (and make refs implement an interface) - line 253: // TODO: Delete via status selfLink? - line 273: // TODO: not right! - line 284: // TODO: Poll for status - line 445: // TODO: If job fails to update, we probably need to "put back" the old job id pkg/controller/direct/monitoring/dashboard_mappings.go (6 lines): - line 78: // TODO: The format is not documented, we need to validate - line 83: // TODO: The format is not documented, we need to validate - line 88: // TODO: The format is not documented, we need to validate - line 93: // TODO: The format is not documented, we need to validate - line 98: // TODO: The format is not documented, we need to validate - line 103: // TODO: The format is not documented, we need to validate dev/tools/controllerbuilder/pkg/codegen/mappergenerator.go (4 lines): - line 135: // TODO: Move to flag? How many of these are there? - line 191: // TODO: Can we figure out a better way here? - line 304: fmt.Fprintf(out, "\t// TODO: map type %v %v for field %v\n", keyKind, valueKind, krmFieldName) - line 482: fmt.Fprintf(out, "\t// TODO: map type %v %v for field %v\n", keyKind, valueKind, krmFieldName) pkg/controller/direct/redis/cluster/rediscluster_controller.go (4 lines): - line 81: // TODO: Just fetch this object? - line 180: // TODO: Delete via status selfLink? - line 236: /* TODO: Any reason to write resourceID back? It's required anyway - line 260: // TODO: What if a different field (immutability again) experiments/composite/pkg/watchset/interestset.go (4 lines): - line 28: // TODO: Should we have a status on the InterestSet, to indicate that all our watches are healthy? - line 29: // TODO: Can we / should we fold this into applyset? Similar data structure... - line 88: // TODO: clean up interest? - line 203: // TODO: Cache this apis/alloydb/v1alpha1/cluster_types.go (4 lines): - line 182: // TODO: Remove "// +required" marker for StartTime after using direct controller. - line 215: // TODO: Verify if "// +required" marker is needed for Day after using direct controller. - line 222: // TODO: Verify if "// +required" marker is needed for StartTime after using direct controller. - line 296: // TODO: Verify if "// +required" marker is needed for PrimaryClusterNameRef after using direct controller. pkg/controller/direct/vertexai/featurestore_controller.go (4 lines): - line 94: // TODO: Support URLs - line 128: // TODO: resolve reference fields in CREATE and UPDATE. - line 175: // TODO: we need customized diff logic for primitive types such as "OnlineStorageTtlDays int32" and "Etag string", use something like a.desired.Spec.OnlineStorageTtlDays != nil - line 244: // TODO: support force delete pkg/controller/direct/networkconnectivity/serviceconnectionpolicy_controller.go (4 lines): - line 127: // TODO: What is format? - line 208: // TODO: Delete via status selfLink? - line 281: // TODO: Where/how do we want to enforce immutability? - line 300: // TODO: Other calls pkg/cli/preview/streaming_informer.go (4 lines): - line 116: // TODO: Backoff - line 216: // TODO: Propagate correctly to ListWatch - line 220: // TODO: Snapshot - line 295: // TODO: How do we want to copy objects? dev/tools/controllerbuilder/pkg/codegen/typegenerator.go (4 lines): - line 89: msgs, err := FindDependenciesForMessage(message, nil) // TODO: explicitly set ignored fields when generating Go types - line 302: fmt.Fprintf(out, "\n\t// TODO: %v\n\n", err) - line 551: // TODO: explicitly set ignored fields when generating Go types - line 561: // TODO: explicitly set ignored fields when generating Go types apis/networkmanagement/v1alpha1/connectivitytest_types.go (4 lines): - line 139: // TODO: Should be reference. - line 166: // TODO: Should be reference. - line 173: // TODO: Should be reference. - line 180: // TODO: Should be reference. apis/alloydb/v1beta1/cluster_types.go (4 lines): - line 182: // TODO: Remove "// +required" marker for StartTime after using direct controller. - line 215: // TODO: Verify if "// +required" marker is needed for Day after using direct controller. - line 222: // TODO: Verify if "// +required" marker is needed for StartTime after using direct controller. - line 296: // TODO: Verify if "// +required" marker is needed for PrimaryClusterNameRef after using direct controller. experiments/compositions/composition/internal/controller/expander_reconciler.go (3 lines): - line 217: // TODO: Accumulates LastPruned. - line 581: // TODO check if json.Marshall is escaping quotes - line 719: // TODO: If there are lots of objects, this will result in very many reconciles. Have not tested to see how the dev/tools/proto-to-mapper/main.go (3 lines): - line 168: // TODO: Use ParentFile instead? - line 190: // TODO: Precompute this! - line 270: fmt.Fprintf(out, "\t// TODO: map type %v %v for field %v\n", keyKind, valueKind, jsonName) experiments/composite/pkg/controllers/instancereconciler.go (3 lines): - line 105: // TODO: Why do we have to pass String() ? - line 184: // TODO: Don't prune until objects are healthy - line 198: // TODO: Applier should return the objects in their post-apply state, so we don't have to requery pkg/controller/direct/datastream/connectionprofile_controller.go (3 lines): - line 88: // TODO: Support URLs - line 186: // TODO: handle other fields - line 297: // TODO: PostgresqlProfile is not implemented yet pkg/controller/direct/vmwareengine/privatecloud_controller.go (3 lines): - line 87: // TODO: Support URLs - line 176: // TODO: spec.managementCluster is not readable from GCP - line 177: // TODO: spec.networkConfig is has generated fields from GCP pkg/controller/direct/datacatalog/tagtemplate_controller.go (3 lines): - line 108: // TODO: Support URLs - line 202: // TODO: Implement reconciliation logic for TagTemplateFields if they are managed inline. - line 215: // TODO: After successful update, reconcile TagTemplateFields if managed inline. experiments/conductor/cmd/runner/mock_commands.go (3 lines): - line 653: // TODO: why should we check this? - line 660: // TODO: Populate the ProtoPath in branches-all.yaml. - line 668: // TODO: review later. PROTO_PACKAGE = branch.ProtoPath or branch.ProtoPackage? experiments/composite/pkg/controllers/compositedefinition_controller.go (3 lines): - line 48: // TODO: Locking - line 121: // TODO: Share watchset manager across controllers - line 157: // TODO: Handle/log errors pkg/controller/direct/logging/link_controller.go (3 lines): - line 98: // TODO: Support URLs - line 221: /* TODO Bucket Ref? - line 237: // TODO uObj is set in the commented out code pkg/controller/direct/compute/forwardingrule/client.go (2 lines): - line 44: // TODO: Set UserAgent in this scenario (error is: WithHTTPClient is incompatible with gRPC dial options) - line 58: // TODO: support endpoints? pkg/cli/preview/intercepting_kube_client.go (2 lines): - line 59: // TODO: Replace with rest.DefaultServerUrlFor - line 80: // TODO: Verify restConfig or httpClient? scripts/generate-google3-docs/resource-reference/main.go (2 lines): - line 316: // TODO: Add annotation and IAM settings handling logic for direct GKs. - line 328: // TODO: Samples should also be required for direct CRDs. pkg/controller/direct/vmwareengine/externalaccessrule_controller.go (2 lines): - line 86: // TODO: Support URLs - line 190: // TODO: handle source_ip_ranges and destination_ip_ranges which contain oneof fields which requires special handling pkg/controller/direct/securesourcemanager/repository_controller.go (2 lines): - line 113: // TODO: Support URLs - line 236: // TODO - remove after the Go protobuf fix is in. https://github.com/golang/protobuf/issues/1620#issuecomment-2402608919 pkg/controller/direct/backupdr/backupvault_controller.go (2 lines): - line 87: // TODO: Support URLs - line 180: // TODO: etag pkg/controller/direct/directbase/directbase_controller.go (2 lines): - line 494: // TODO: even in the event of an actual update to GCP, this function will - line 500: // TODO: Implement these checks pkg/cli/asset/asset.go (2 lines): - line 86: // TODO: this is hard-coded until we rename IAMCustomRole to IAMRole at which point this can be removed - line 93: // TODO: do something for compute instance here dev/tools/controllerbuilder/pkg/typeupdater/fieldupdateplan.go (2 lines): - line 89: goType, err := codegen.GoTypeForField(protoField, false) // TODO: add support for transitive output fields - line 100: // TODO: add support for transitive output fields when writing the field pkg/controller/direct/registry/registry.go (2 lines): - line 123: // TODO: Move to registration somehow? - line 134: // TODO: Move to registration somehow? dev/tools/controllerbuilder/cmd/resourcerunner/main.go (2 lines): - line 106: // TODO: LLM decides what to do with this resource - line 396: // TODO: AI generate message pkg/controller/direct/mappings/build.go (2 lines): - line 64: // TODO: We should have something here, because right now everything is a FieldMapping - line 95: // They are different from TODO fields, in that we have actively determined they should not be mapped. pkg/controller/direct/bigquerydatatransfer/bigquerydatatransferconfig_controller.go (2 lines): - line 193: // TODO: Support URLs - line 325: // TODO: sensitive fields maybe masked by the service, leading to constant diff. dev/tools/controllerbuilder/pkg/commands/updatetypes/updatetypescommand.go (2 lines): - line 29: ignoredFields string // TODO: could be part of GenerateOptions - line 45: // TODO: Update this flag to accept a file path pointing to the ignored fields YAML file. experiments/kubectl-plan/pkg/plan/planner.go (2 lines): - line 40: // TODO: prefetch & invalidate REST Mappings here, so we know they are valid later? - line 64: // TODO: We should invalidate mappings above, in case our cache is out of date dev/tools/controllerbuilder/pkg/codebot/tool_edit_file.go (2 lines): - line 115: // TODO: Response? - line 203: // TODO: Hold on ... this is an exact match comparison! We could do this with strings.Index! experiments/compositions/composition/api/v1alpha1/composition_types.go (2 lines): - line 69: // TODO (barney-s): Make ConfigReference the only way to specify and dont have any inline expander configs - line 120: // TODO (barney -s) rename to FacadeAPIGroup,facadeAPIGroup pkg/webhook/immutable_fields_validator.go (2 lines): - line 525: // TODO: get rid of list.List by changing the function to return a []string recursively - line 557: // TODO: terraform schema doc says that TypeMap only support Elem to be a *Schema with a Type that is one of the primitives experiments/compositions/composition/internal/controller/composition_controller.go (2 lines): - line 61: // TODO: To simplify preview for customers, grant superuser to the composition controller. This should be revisited going forward. - line 242: // TODO check if json.Marshall is escaping quotes pkg/controller/direct/kms/autokeyconfig/autokeyconfig_controller.go (2 lines): - line 105: // TODO: Support URLs - line 247: // TODO: uncomment once we found a valid solution pkg/config/controllerconfig.go (2 lines): - line 80: // TODO: support endpoints? - line 103: // TODO: support endpoints? pkg/controller/direct/secretmanager/secretversion_controller.go (2 lines): - line 90: // TODO: Support URLs - line 154: // TODO: safer option to pass in the checksum. pkg/controller/direct/documentai/processor_controller.go (2 lines): - line 89: // TODO: Support URLs - line 147: // TODO: is the default version also an output field? pkg/controller/direct/cloudbuild/workerpool_controller.go (2 lines): - line 130: // TODO: need to support more cases - line 327: // TODO: Delete can rely on status.externalRef and do not need spec.projectRef. pkg/controller/direct/monitoring/monitoringdashboard_controller.go (2 lines): - line 181: // TODO: Delete via status selfLink? - line 238: // TODO: Where/how do we want to enforce immutability? pkg/controller/direct/vmwareengine/vmwareenginenetwork_controller.go (2 lines): - line 86: // TODO: Support URLs - line 169: // TODO: etag pkg/controller/direct/bigquerydatatransfer/bigquerydatatransfer_mappings.go (2 lines): - line 130: // TODO: if we need to support more types, we need to change KRM type of Params to map[string]interface{} - line 147: // TODO: if we need to support more types, we need to change KRM type of Params to map[string]interface{} pkg/webhook/iam_validator.go (2 lines): - line 248: // TODO: (b/182505291) DCL-based resources do not currently support IAMAuditConfigs - line 313: // TODO (b/228226694): IAMPolicyMember does not currently support conditions. pkg/controller/direct/compute/forwardingrule/forwardingrule_controller.go (2 lines): - line 136: // TODO: need to support more cases - line 171: // TODO: Support URLs pkg/webhook/cert/writer/fs.go (2 lines): - line 119: // TODO: figure out if we can reduce the permission. (Now it's 0777) - line 194: // TODO: figure out if we can reduce the permission. (Now it's 0666) apis/datacatalog/v1alpha1/entry_types.go (2 lines): - line 98: // TODO: Known issue: recursive types are tripping CRD generation - line 302: // TODO: Looks like externalRef is the same as name. Remove name? dev/tools/controllerbuilder/pkg/commands/generatefuzzer/generatefuzzercommand.go (1 line): - line 60: // TODO: extract messages from generate.sh or api directory pkg/controller/direct/documentai/processorversion_controller.go (1 line): - line 88: // TODO: Support URLs apis/refs/v1beta1/alloydbref.go (1 line): - line 46: // TODO: Remove after AlloyDBCluster is migrated to direct controller. dev/tools/controllerbuilder/pkg/io/comments.go (1 line): - line 63: // TODO: support IMPORT, CONST, VAR pkg/controller/direct/firestore/firestoredatabase_controller.go (1 line): - line 177: // TODO: Support URLs pkg/controller/direct/backupdr/backupplanassociation_controller.go (1 line): - line 86: // TODO: Support URLs pkg/controller/direct/bigqueryreservation/assignment_controller.go (1 line): - line 116: // TODO: Support URLs apis/metastore/v1alpha1/backup_identity.go (1 line): - line 24: // TODO: Add import for the parent service reference if needed, e.g., experiments/compositions/expanders/cel-expander/pkg/cel/engine.go (1 line): - line 31: // TODO: what withtagname ? can we pass yaml ? dev/tools/controllerbuilder/template/controller/controller.go (1 line): - line 127: // TODO: Support URLs pkg/resourceskeleton/uri/uri.go (1 line): - line 46: // TODO: remove 'Direct' field from ResourceConfig and remove the if statement. pkg/controller/direct/eventarc/channel_controller.go (1 line): - line 88: // TODO: Support URLs pkg/controller/direct/apigee/envgroup_controller.go (1 line): - line 88: // TODO: Support URLs pkg/cli/preview/utils.go (1 line): - line 28: // TODO: move the default to secure when the apiserver supports TLS by default pkg/controller/direct/workstations/cluster_controller.go (1 line): - line 172: // TODO: Support URLs pkg/resourceoverrides/utils.go (1 line): - line 73: // TODO: Handle this edge case once there is a concrete example. apis/cloudbuild/v1beta1/workerpool_types.go (1 line): - line 44: // TODO: support annotations pkg/controller/direct/netapp/backuppolicy_controller.go (1 line): - line 92: // TODO: Support URLs experiments/compositions/expanders/cel-expander/main.go (1 line): - line 174: // TODO: consume the error and mark result failed ? pkg/text/text.go (1 line): - line 23: // TODO: there must be some package out there that does this well apis/backupdr/v1alpha1/backupplanassociation_types.go (1 line): - line 32: // TODO: add other resource types pkg/controller/direct/datastream/route_controller.go (1 line): - line 85: // TODO: Support URLs pkg/controller/direct/vmwareengine/externaladdress_controller.go (1 line): - line 86: // TODO: Support URLs pkg/controller/direct/kms/keyhandle/keyhandle_controller.go (1 line): - line 93: // TODO: Support URLs dev/tools/controllerbuilder/pkg/commands/detectnewfields/detectnewfieldscommand.go (1 line): - line 47: // TODO: create this file pkg/controller/direct/pubsub/pubsubsnapshot_controller.go (1 line): - line 120: // TODO: Support URLs pkg/controller/direct/managedkafka/topic_controller.go (1 line): - line 92: // TODO: Support URLs pkg/controller/direct/workstations/workstation_controller.go (1 line): - line 118: // TODO: Support URLs pkg/controller/direct/notebooks/environment_controller.go (1 line): - line 90: // TODO: Support URLs pkg/cli/preview/streaming_client.go (1 line): - line 151: // TODO: Implement true streaming parsing (likely with https://github.com/golang/go/issues/71497) dev/tools/controllerbuilder/pkg/codebot/tool_find_in_workspace.go (1 line): - line 85: // TODO: Response? pkg/controller/direct/workflows/workflow_controller.go (1 line): - line 110: // TODO: Support URLs pkg/controller/direct/workflowexecutions/execution_controller.go (1 line): - line 89: // TODO: Support URLs scripts/generate-crds/main.go (1 line): - line 127: // TODO: remove 'Direct' field from ResourceConfig and remove the if statement. pkg/cli/preview/intercepting_gcp_client.go (1 line): - line 163: // TODO: Add this back in to support GRPC resources (e.g. bigtable) pkg/controller/direct/datacatalog/tag_controller.go (1 line): - line 117: // TODO: Support URLs pkg/controller/direct/apigee/endpointattachment_controller.go (1 line): - line 76: // TODO: Support URLs pkg/controller/direct/bigqueryreservation/reservation_controller.go (1 line): - line 91: // TODO: Support URLs config/servicemappings/compute.yaml (1 line): - line 1640: # TODO: (b/233123518) Config Connector CLI cannot currently support ComputeRegionNetworkEndpointGroup. pkg/controller/direct/apigee/envgroupattachment_controller.go (1 line): - line 77: // TODO: Support URLs pkg/controller/direct/managedkafka/cluster_controller.go (1 line): - line 93: // TODO: Support URLs pkg/k8s/constants.go (1 line): - line 26: // TODO: clean up old conditions used in handcrafted controllers pkg/controller/direct/clouddeploy/deliverypipeline_controller.go (1 line): - line 93: // TODO: Support URLs dev/tools/controllerbuilder/pkg/codebot/tool_verify_code.go (1 line): - line 107: // TODO: Response? apis/dataproc/v1alpha1/job_types.go (1 line): - line 613: // TODO: unsupported map type with key string and value enum pkg/controller/direct/networkmanagement/connectivitytest_controller.go (1 line): - line 85: // TODO: Support URLs pkg/controller/direct/notebooks/instance_controller.go (1 line): - line 91: // TODO: Support URLs pkg/controller/direct/workstations/config_controller.go (1 line): - line 127: // TODO: Support URLs pkg/controller/direct/speech/customclass_controller.go (1 line): - line 87: // TODO: Support URLs cmd/recorder/kube/dynamic_watch.go (1 line): - line 156: // TODO: Once the WatchList feature gate is on by default, we can use SendInitialEvents to get a precise notification of the bookmark. pkg/controller/direct/asset/feed_controller.go (1 line): - line 109: // TODO: Support URLs dev/tools/controllerbuilder/pkg/newfieldsdetector/targetfields.go (1 line): - line 46: // TODO: Create a structured file to store this information for more reliable parsing. The file can be shared between the type generator and the new field detector. pkg/controller/direct/dataproc/batch_controller.go (1 line): - line 108: // TODO: Support URLs pkg/controller/direct/datacatalog/entry_controller.go (1 line): - line 109: // TODO: Support URLs pkg/controller/iam/iamclient/helpers.go (1 line): - line 116: // TODO: handle more direct resource reference pkg/controller/iam/auditconfig/iamauditconfig_controller.go (1 line): - line 392: // TODO: even in the event of an actual update to GCP, this function will pkg/crd/crdgeneration/crdboilerplate/crdboilerplate.go (1 line): - line 74: //TODO See how CRD generation in the k8s "crd" tool works, so you don't have to create this scaffolding by hand pkg/controller/direct/dataform/repository_controller.go (1 line): - line 170: // TODO: Support URLs experiments/compositions/composition/pkg/containerexecutor/jobcontainerexecutor/jobcontainerrunner.go (1 line): - line 266: // TODO (barney-s@) : Move this to composition namespace pkg/controller/direct/vertexai/metadatastore_controller.go (1 line): - line 121: // TODO: Support URLs pkg/controller/kccmanager/nocache/clientbuilder.go (1 line): - line 38: // * CRDs: Build our schema (TODO: should we just list and then restart if these change?) dev/tools/controllerbuilder/pkg/codebot/tool_create_file.go (1 line): - line 107: // TODO: Response? apis/apigee/v1beta1/environment_reference.go (1 line): - line 73: // TODO: Use general-purpose refsv1beta1.Normalize function once direct controller is implemented. cmd/recorder/main.go (1 line): - line 244: // TODO: consolidate the logic with krmtotf.GetReadyCondition pkg/controller/direct/recaptchaenterprise/firewallpolicy_controller.go (1 line): - line 83: // TODO: Support URLs pkg/controller/direct/compute/targettcpproxy/targettcpproxy_controller.go (1 line): - line 161: // TODO: Support URLs pkg/controller/direct/dataplex/dataplexlake_controller.go (1 line): - line 89: // TODO: Support URLs pkg/controller/direct/compute/networkedgesecurityservice_controller.go (1 line): - line 89: // TODO: Support URLs pkg/controller/direct/compute/networkattachment_controller.go (1 line): - line 88: // TODO: Support URLs dev/tools/controllerbuilder/pkg/commands/iteratetypes/command.go (1 line): - line 76: // TODO: Update this flag to accept a file path pointing to the ignored fields YAML file. pkg/controller/direct/vmwareengine/networkpeering_controller.go (1 line): - line 88: // TODO: Support URLs pkg/controller/direct/networkconnectivity/internalrange_controller.go (1 line): - line 94: // TODO: Support URLs pkg/webhook/cert/writer/secret.go (1 line): - line 172: // TODO: figure out how to get the UID pkg/controller/direct/bigquerydataset/dataset_controller.go (1 line): - line 102: // TODO: Support URLs apis/dataflow/v1beta1/flextemplatejob_types.go (1 line): - line 198: // TODO: Description for JobID? apis/batch/v1alpha1/job_types.go (1 line): - line 374: // TODO: comment out the password for now and add it back when secreteManager pkg/controller/direct/privilegedaccessmanager/entitlement_controller.go (1 line): - line 209: // TODO: Support URLs pkg/webhook/cert/certclient/certclient.go (1 line): - line 167: // TODO: retry mutiple times with backoff if necessary. pkg/controller/direct/backupdr/backupplan_controller.go (1 line): - line 92: // TODO: Support URLs experiments/compositions/composition/pkg/applier/applier.go (1 line): - line 200: // TODO (barney-s): This would result in some objects not being cleaned up. pkg/controller/direct/metastore/backup_controller.go (1 line): - line 84: // TODO: Support URLs apis/secretmanager/v1beta1/secretversion_types.go (1 line): - line 59: // TODO: support getting secret data from other places. apis/spanner/v1alpha1/backupschedule_types.go (1 line): - line 45: // TODO: Confirm whether FullBackupSpec and IncrementalBackupSpec should be references before promoting to beta. pkg/controller/direct/composer/environment_controller.go (1 line): - line 90: // TODO: Support URLs apis/bigqueryconnection/v1beta1/connection_reference.go (1 line): - line 65: // TODO: what if multiple BigQueryConnectionConnection points to the same GCP Connection? pkg/controller/direct/mappings/field.go (1 line): - line 48: // TODO: Do we want to normalize this here? pkg/cli/krmtohcl/krmtohcl.go (1 line): - line 51: // TODO: it has not been 100% verified what the canonical purpose of krmResource.TFInfo.ID is. pkg/controller/direct/gkebackup/restore_controller.go (1 line): - line 93: // TODO: Support URLs pkg/controller/direct/apphub/application_controller.go (1 line): - line 92: // TODO: Support URLs pkg/controller/direct/metastore/federation_controller.go (1 line): - line 88: // TODO: Support URLs pkg/controller/direct/essentialcontacts/contact_controller.go (1 line): - line 93: // TODO: Support URLs pkg/controller/iam/partialpolicy/iampartialpolicy_controller.go (1 line): - line 438: // TODO: even in the event of an actual update to GCP, this function will pkg/controller/direct/dataplex/task_controller.go (1 line): - line 91: // TODO: Support URLs pkg/cli/powertools/forcesetfield/cmd.go (1 line): - line 165: // TODO: What about things that aren't strings? pkg/controller/direct/apigee/instanceattachment_controller.go (1 line): - line 77: // TODO: Support URLs apis/refs/v1beta1/appengineref.go (1 line): - line 24: // TODO: move to apis/appengine when we implement the AppEngine resource dev/tools/controllerbuilder/pkg/typeupdater/insertfield-gemini.go (1 line): - line 69: // TODO: provide content of the proto message for reference on the field ordering. pkg/controller/direct/apigateway/api_controller.go (1 line): - line 85: // TODO: Support URLs pkg/controller/iam/policy/iampolicy_controller.go (1 line): - line 391: // TODO: even in the event of an actual update to GCP, this function will pkg/cli/outputsink/sink.go (1 line): - line 123: // TODO: need to pass in smloader to the constructor pkg/webhook/cert/writer/atomic/atomic_writer.go (1 line): - line 250: // TODO: somehow unify this with the similar api validation, dev/tools/controllerbuilder/pkg/typeupdater/fieldinserter.go (1 line): - line 183: codegen.WriteField(&buf, u.newField.proto, u.newField.parent, 0, false) // TODO: add support for transitive output fields pkg/controller/direct/dataplex/entrygroup_controller.go (1 line): - line 87: // TODO: Support URLs dev/tools/controllerbuilder/pkg/codebot/toolbox.go (1 line): - line 73: // TODO: Fatal or return an error? pkg/controller/direct/bigquerybiglake/table_controller.go (1 line): - line 92: // TODO: Support URLs pkg/controller/direct/batch/batchjob_controller.go (1 line): - line 116: // TODO: Support URLs pkg/controller/iam/policymember/iampolicymember_controller.go (1 line): - line 403: // TODO: even in the event of an actual update to GCP, this function will dev/tools/controllerbuilder/pkg/codebot/tool_run_command.go (1 line): - line 99: // TODO: Response? apis/datastream/v1alpha1/connectionprofile_types.go (1 line): - line 360: // TODO: ssl_config proto field is not generated pkg/controller/direct/cloudtasks/queue_controller.go (1 line): - line 92: // TODO: Support URLs dev/tools/controllerbuilder/pkg/typeupdater/insertfield-ast.go (1 line): - line 127: // TODO: use the same field ordering as in proto message? pkg/webhook/state_into_spec_annotation_validator.go (1 line): - line 61: // TODO: Verify if the state-into-spec annotation will be defaulted to 'merge'. scripts/dclsampleconverter/sample.py (1 line): - line 96: #TODO: Handle update dependencies which are not already in the create dependencies array. pkg/controller/direct/networkservices/servicebinding_controller.go (1 line): - line 89: // TODO: Support URLs pkg/controller/direct/vmwareengine/networkpolicy_controller.go (1 line): - line 87: // TODO: Support URLs apis/servicedirectory/v1alpha1/service_reference.go (1 line): - line 89: // TODO: refactor after the ServiceDirectoryService is added. pkg/controller/direct/metastore/service_controller.go (1 line): - line 87: // TODO: Support URLs pkg/controller/direct/gkebackup/backupplan_controller.go (1 line): - line 89: // TODO: Support URLs pkg/controller/direct/dataplex/entrytype_controller.go (1 line): - line 89: // TODO: Support URLs dev/tools/controllerbuilder/pkg/codebot/tool_list_files.go (1 line): - line 85: // TODO: Response? pkg/controller/direct/certificatemanager/dnsauthorization_controller.go (1 line): - line 142: // TODO: Support URLs pkg/controller/direct/eventarc/googlechannelconfig_controller.go (1 line): - line 86: // TODO: Support URLs experiments/compositions/composition/cmd/inline/main.go (1 line): - line 83: // TODO (barni@): assuming name is fixed pkg/controller/direct/secretmanager/secret_controller.go (1 line): - line 119: // TODO: Support URLs config/servicemappings/certificatemanager.yaml (1 line): - line 60: # TODO (b/298972959): Remove it once the CA enabling issue is resolved. pkg/controller/direct/directbase/operations.go (1 line): - line 207: // TODO: Do we want ObservedGeneration in our conditions? apis/storagecontrol/v1alpha1/folder_identity.go (1 line): - line 67: //TODO: Update to use storagebucket parseExternal once the resource is migrated to SciFi. pkg/controller/direct/bigqueryanalyticshub/dataexchange_controller.go (1 line): - line 137: // TODO: Support URLs pkg/controller/direct/logging/logbucketref.go (1 line): - line 92: // TODO: This is a recursive resolve ... we really need status.selfLink or similar pkg/controller/direct/sql/sqlinstance_controller.go (1 line): - line 106: // TODO: Support URLs dev/tools/controllerbuilder/pkg/codebot/tool_run_cmd.go (1 line): - line 125: // TODO: Response? pkg/controller/direct/compute/firewallpolicyrule/firewallpolicyrule_controller.go (1 line): - line 108: // TODO: Support URLs pkg/controller/direct/cloudquota/quotapreference_controller.go (1 line): - line 96: // TODO: Support URLs pkg/controller/direct/spanner/instance_controller.go (1 line): - line 113: // TODO: Support URLs apis/apigee/v1beta1/organization_reference.go (1 line): - line 73: // TODO: Use general-purpose refsv1beta1.Normalize function once direct controller is implemented. pkg/controller/direct/bigqueryanalyticshub/listing_controller.go (1 line): - line 121: // TODO: Support URLs pkg/cli/cmd/printresources/resourcedescription/resourcedescription.go (1 line): - line 107: // TODO: remove 'Direct' field from ResourceConfig and remove the if statement. dev/tools/controllerbuilder/pkg/codebot/tool_ast_edit.go (1 line): - line 125: // TODO: Response? pkg/controller/direct/colab/runtime_controller.go (1 line): - line 102: // TODO: Support URLs pkg/controller/direct/iap/iapsettings_controller.go (1 line): - line 89: // TODO: Support URLs pkg/controller/direct/datastream/privateconnection_controller.go (1 line): - line 86: // TODO: Support URLs dev/tools/controllerbuilder/pkg/benchmark/const.go (1 line): - line 24: // TODO: We should exclude these ground truth from the gemnimi input datapoints. pkg/controller/direct/metastore/service_mapper.go (1 line): - line 307: // TODO: map type int32 message for field BackendMetastores pkg/controller/direct/speech/phraseset_controller.go (1 line): - line 87: // TODO: Support URLs experiments/compositions/composition/pkg/cel/engine.go (1 line): - line 31: // TODO: what withtagname ? can we pass yaml ? apis/networkconnectivity/v1alpha1/serviceconnectionpolicy_types.go (1 line): - line 66: /* TODO:AFTER-MAPPINGS apis/bigqueryconnection/v1alpha1/connection_reference.go (1 line): - line 76: // TODO: what if multiple BigQueryConnectionConnection points to the same GCP Connection? apis/apigateway/v1alpha1/api_types.go (1 line): - line 46: ManagedService *string `json:"managedService,omitempty"` // TODO: Before promote to v1beta. Check if this field should be a ref to https://cloud.google.com/service-infrastructure/docs/service-management/reference/rest/v1/services#ManagedService. pkg/controller/direct/spanner/backupschedule_controller.go (1 line): - line 111: // TODO: Support URLs pkg/controller/direct/asset/savedquery_controller.go (1 line): - line 111: // TODO: Support URLs pkg/controller/direct/datacatalog/entrygroup_controller.go (1 line): - line 109: // TODO: Support URLs pkg/controller/direct/dataplex/zone_controller.go (1 line): - line 92: // TODO: Support URLs pkg/controller/direct/backupdr/managementserver_controller.go (1 line): - line 87: // TODO: Support URLs pkg/krmtotf/resource.go (1 line): - line 84: // TODO: remove 'Direct' field from ResourceConfig and remove the if statement. pkg/controller/direct/cloudquota/quotaadjustersettings_controller.go (1 line): - line 91: // TODO: Support URLs pkg/apis/common/v1alpha1/types.go (1 line): - line 26: // TODO: Do we still need a CommonSpec for resources that do not need project dependency? Those seems to be exceptional any ways. pkg/controller/direct/mappings/convert.go (1 line): - line 45: // TODO: What if destElem is not valid pkg/controller/direct/storage/control/folder_controller.go (1 line): - line 86: // TODO: Support URLs dev/tools/controllerbuilder/pkg/codebot/tool_read_file.go (1 line): - line 81: // TODO: Response? pkg/controller/direct/apigee/instance_controller.go (1 line): - line 79: // TODO: Support URLs pkg/controller/direct/gkebackup/backup_controller.go (1 line): - line 86: // TODO: Support URLs pkg/cli/preview/typeinfo.go (1 line): - line 95: // TODO: Do partial fetch pkg/controller/direct/gkebackup/restoreplan_controller.go (1 line): - line 100: // TODO: Support URLs pkg/controller/direct/spanner/instanceconfig_controller.go (1 line): - line 88: // TODO: Support URLs experiments/conductor/cmd/runner/utilities.go (1 line): - line 293: // TODO: Return a log object so we can run in parrellel. pkg/cli/powertools/kubecli/client.go (1 line): - line 190: // TODO: Print fully-qualified names scripts/generate-third-party-licenses/main.go (1 line): - line 136: // TODO: instead of assuming "blob/master", link to the specific SHA we use