edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Core/routing/ModuleEndpoint.cs (5 lines):
	- line 46: // TODO - No-op
	- line 85: // TODO - Add more info to this log message
	- line 91: // TODO - Add more info to this log message
	- line 138: // TODO - No-op
	- line 154: // TODO - figure out if we can use cancellation token to cancel send


edge-modules/metrics-collector/src/FixedSetTableUpload/FixedSetTableUpload.cs (4 lines):
	- line 30: // TODO: is this a good fallback?
	- line 31: // TODO: test
	- line 47: // TODO: split up metricList so that no individual post is greater than 1mb
	- line 100: //TODO: what to do with origin?


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Core/storage/MessageStore.cs (3 lines):
	- line 99: // TODO - Need to support transactions for these operations. The underlying storage layers support it.
	- line 248: /// // TODO - Update cleanup logic to cleanup expired messages from entity store as well.
	- line 551: // TODO - Currently, this does not iterate over a snapshot. This should work as the cleanup and reference counting is managed at


edgelet/docker-rs/docker-swagger.yaml (3 lines):
	- line 1127: # TODO is SecondaryIPAddresses actually used?
	- line 1135: # TODO is SecondaryIPv6Addresses actually used?
	- line 1143: # TODO properties below are part of DefaultNetworkSettings, which is


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Amqp/AmqpRuntimeProvider.cs (2 lines):
	- line 95: // TODO: implement the rules below
	- line 178: // TODO: Handle token expiry exception case here by closing AMQP connection.


edge-modules/metrics-collector/src/FixedSetTableUpload/AzureFixedSetTable.cs (2 lines):
	- line 84: // TODO: replace with actual version number
	- line 162: // TODO: test


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Mqtt/MqttProtocolHead.cs (2 lines):
	- line 118: // TODO: gracefully shutdown the MultithreadEventLoopGroup in MqttWebSocketListener?
	- line 119: // TODO: this.webSocketListenerRegistry.TryUnregister("mqtts")?


edge-modules/metrics-collector/src/Constants.cs (2 lines):
	- line 5: public static readonly string VersionNumber = "0.1.2.0";  // TODO: grab this from somewhere else
	- line 8: public static readonly string MetricComputer = "";  // TODO: maybe put a short value here? Customers pay for the storage though...


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.CloudProxy/CloudProxy.cs (2 lines):
	- line 493: // TODO: update device SDK to unregister callback for desired properties by passing null
	- line 533: // TODO: check if message delivery count exceeds the limit?


edgelet/edgelet-utils/src/yaml_file_source.rs (2 lines):
	- line 90: // TODO: Figure out in what cases this can fail?
	- line 106: // TODO: should we do anything for non-string keys?


edge-agent/src/Microsoft.Azure.Devices.Edge.Agent.IoTHub/EdgeAgentConnection.cs (2 lines):
	- line 440: // TODO: Localize this error?
	- line 446: // TODO: Localize this error?


edge-hub/core/src/Microsoft.Azure.Devices.Routing.Core/Dispatcher.cs (2 lines):
	- line 85: // TODO handle case where endpoint is not in dispatcher's list of endpoints
	- line 217: // TODO add logging?


mqtt/mqtt3/src/client/mod.rs (1 line):
	- line 129: // TODO: username / password / will can be too large and prevent a CONNECT packet from being encoded.


edge-modules/SimulatedTemperatureSensor/src/MessageBody.cs (1 line):
	- line 8: // TODO Put message body in a common lib


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Amqp/ClientConnectionHandler.cs (1 line):
	- line 54: // TODO: Implement plug and play for AMQP


edge-agent/src/Microsoft.Azure.Devices.Edge.Agent.Core/IRuntimeInfoProvider.cs (1 line):
	- line 12: /// TODO: Consider replacing this with IEnvironment and the decorator pattern.


edge-agent/src/Microsoft.Azure.Devices.Edge.Agent.Core/ModuleIdentityHelper.cs (1 line):
	- line 8: // TODO - This will fail if the user adds modules with the same module name as a system module - for example a module called


edge-util/src/Microsoft.Azure.Devices.Edge.Storage.RocksDb/DbStoreProvider.cs (1 line):
	- line 21: readonly Timer compactionTimer; // TODO: Bug logged to be fixed to proper dispose and test.


edge-util/src/Microsoft.Azure.Devices.Edge.Util/metrics/prometheus.net/MetricsProvider.cs (1 line):
	- line 28: // TODO:


edge-agent/src/Microsoft.Azure.Devices.Edge.Agent.Core/Agent.cs (1 line):
	- line 140: // TODO - Update this logic to create identities only when needed, in the Command factory, instead of creating all the identities


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Amqp/settings/AmqpSettingsProvider.cs (1 line):
	- line 49: // TODO: Verify if this handler still needs to be added. It looks like at some point in the past


smoke/IotEdgeQuickstart/details/Details.cs (1 line):
	- line 399: // TODO: [Improvement] should verify test results without using event hub, which introduce latency.


smoke/IotEdgeQuickstart/details/IotedgedLinux.cs (1 line):
	- line 44: // TODO: should use an internal IP address--e.g. docker0's address--instead


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.CloudProxy/ConnectivityAwareClient.cs (1 line):
	- line 126: // @TODO: Ignore callback from Device SDK since it seems to be generating a lot of spurious Connected/NotConnected callbacks


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Core/device/DeviceMessageHandler.cs (1 line):
	- line 452: // TODO - Should we throw here instead?


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Amqp/SaslIdentityType.cs (1 line):
	- line 4: // TODO: What is the difference between these 2 enum values?


edge-agent/src/Microsoft.Azure.Devices.Edge.Agent.Diagnostics/storage/IMetricsStorage.cs (1 line):
	- line 20: // TODO: change to IAsyncEnumerable when c# 8 is valid


edge-agent/src/Microsoft.Azure.Devices.Edge.Agent.Core/ModuleSet.cs (1 line):
	- line 41: // TODO use equality comparer instead of equals?


edge-agent/src/Microsoft.Azure.Devices.Edge.Agent.Core/commands/RemoveFromStoreCommand.cs (1 line):
	- line 27: // TODO: Consider caching previous value, so that undo can add it back.


edge-agent/src/Microsoft.Azure.Devices.Edge.Agent.Core/requests/ModuleLogsRequestHandler.cs (1 line):
	- line 117: // TODO: make/get aka link for documentation


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Amqp/EdgeSaslPlainAuthenticator.cs (1 line):
	- line 50: // TODO: Figure out where the device client type parameter value should come from.


edge-agent/src/Microsoft.Azure.Devices.Edge.Agent.Core/configsources/BackupConfigSource.cs (1 line):
	- line 42: // TODO - Backing up the config every time for now, probably should optimize this.


edge-agent/src/Microsoft.Azure.Devices.Edge.Agent.Service/Program.cs (1 line):
	- line 413: // TODO: Move this function to a common location that can be shared between EdgeHub and EdgeAgent


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Service/modules/RoutingModule.cs (1 line):
	- line 382: // TODO - Should the number of retries be tied to the Store and Forward ttl? Not


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Mqtt/ProtocolGatewayMessageConverter.cs (1 line):
	- line 33: // TODO - What about the other properties (like sequence number, etc)? Ignoring for now, as they are not used anyways.


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.CloudProxy/ClientTokenCloudConnection.cs (1 line):
	- line 113: // TODO - instead of this, make convert Option<ICloudProxy> CloudProxy to Task<Option<ICloudProxy>> GetCloudProxy


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Mqtt/SessionState.cs (1 line):
	- line 60: // TODO: Check if this needs to be a deep copy.


edge-agent/src/Microsoft.Azure.Devices.Edge.Agent.Service/modules/EdgeletModule.cs (1 line):
	- line 31: /// TODO: Right now, it assumes Edgelet supports docker. Need to make it completely implementation agnostic


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Amqp/linkhandlers/DeviceBoundLinkHandler.cs (1 line):
	- line 35: // TODO: Check if we need to worry about credit available on the link


edge-util/src/Microsoft.Azure.Devices.Edge.Storage/EntityStore.cs (1 line):
	- line 14: /// TODO - Since Key/Value types are generic, need to look into the right behavior to handle null values here.


edge-agent/src/Microsoft.Azure.Devices.Edge.Agent.Core/IModule.cs (1 line):
	- line 20: Unknown, // TODO: Consider removing this status entirely since it doesn't seem to be used.


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Service/modules/MqttModule.cs (1 line):
	- line 26: // TODO: This causes reSharperWarning. Remove this TODO once code below are uncommented.


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Service/DependencyManager.cs (1 line):
	- line 356: // TODO: Move this function to a common location that can be shared between EdgeHub and EdgeAgent


edge-agent/src/Microsoft.Azure.Devices.Edge.Agent.Docker/DockerEnvironment.cs (1 line):
	- line 19: /// TODO: This could be made generic (not docker specific) and moved to Core.


edge-agent/src/Microsoft.Azure.Devices.Edge.Agent.Core/metrics/MetricsCommandFactory.cs (1 line):
	- line 148: // TODO: determine if module name is PII


edge-agent/src/Microsoft.Azure.Devices.Edge.Agent.IoTHub/EdgeClientWebSocket.cs (1 line):
	- line 14: // TODO: Check if locking is necessary here and if separate locks can be used for reading and writing.


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Amqp/AmqpExceptionsHelper.cs (1 line):
	- line 27: // TODO: Make sure EdgeAmqpException is thrown from the right places.


edge-agent/src/Microsoft.Azure.Devices.Edge.Agent.Core/planner/HealthRestartPlanner.cs (1 line):
	- line 262: // TODO: Windows native containers have an outstanding bug where "docker restart"


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Http/MethodRequest.cs (1 line):
	- line 48: // TODO - Should we allow only UTF8 or other encodings as well? Need to check what IoTHub does.


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Core/routing/CloudEndpoint.cs (1 line):
	- line 62: // TODO - No-op for now


edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Core/TwinManager.cs (1 line):
	- line 124: // TODO: Move to a Twin helper class (along with Twin manager update).


builds/misc/addons-release.yaml (1 line):
	- line 24: # TODO: Consider removing this consolidate stage, which is here mostly for


edge-agent/src/Microsoft.Azure.Devices.Edge.Agent.Core/RestartPolicyManager.cs (1 line):
	- line 24: // TODO: If the module state is "running", when we have health-probes implemented,