{Core} Refactor client factory for Track 2 SDK

[jiasli]

Description

• Rename configure_common_settings_track2 to _prepare_client_kwargs_track2

  • Add a leading underscore (_), because it is a private function which is only used by _get_mgmt_service_client

  • In Track 1 SDK, config is a public member of the client and can be configured later:
    https://github.com/Azure/azure-sdk-for-python/blob/azure-mgmt-resource_10.2.0/sdk/resources/azure-mgmt-resource/azure/mgmt/resource/subscriptions/v2019_11_01/_subscription_client.py#L45

    self.config = SubscriptionClientConfiguration(credentials, base_url)

    In Track 2 SDK, _config is a private member, so configuration should be done via kwargs during __init__ of the client:

    https://github.com/Azure/azure-sdk-for-python/blob/azure-mgmt-resource_15.0.0/sdk/resources/azure-mgmt-resource/azure/mgmt/resource/subscriptions/v2019_11_01/_subscription_client.py#L51

    self._config = SubscriptionClientConfiguration(credential, **kwargs)

    Replace the name to better reflect the functionality of the function.

• Disable INFO log from Track 2 SDK for now until it is included in the --verbose log of Azure CLI ([Core] Print INFO log from SDK #15676).

[yonzhan]

Core

[jiasli]

@qwordy, This will remove the duplicated network trace.

azure.core.pipeline.policies._universal : Request URL: 'https://management.azure.com/subscriptions/0b1f6471-1bf0-4dda-aec3-cb9272f09590?api-version=2016-06-01'
azure.core.pipeline.policies._universal : Request method: 'GET'
azure.core.pipeline.policies._universal : Request headers:
azure.core.pipeline.policies._universal :     'Accept': 'application/json'
...
azure.core.pipeline.policies.http_logging_policy : Request URL: 'https://management.azure.com/subscriptions/0b1f6471-1bf0-4dda-aec3-cb9272f09590?api-version=REDACTED'
azure.core.pipeline.policies.http_logging_policy : Request method: 'GET'
azure.core.pipeline.policies.http_logging_policy : Request headers:
azure.core.pipeline.policies.http_logging_policy :     'Accept': 'application/json'
...
azure.core.pipeline.policies.http_logging_policy : Response status: 200
azure.core.pipeline.policies.http_logging_policy : Response headers:
azure.core.pipeline.policies.http_logging_policy :     'Cache-Control': 'no-cache'
...
azure.core.pipeline.policies._universal : Response status: 200
azure.core.pipeline.policies._universal : Response headers:
azure.core.pipeline.policies._universal :     'Cache-Control': 'no-cache'
...

Logs from azure.core.pipeline.policies.http_logging_policy will now be disabled.

[qwordy]

Can we add a timestamp for each log?

[jiasli]

Can we add a timestamp for each log?

This is very tricky because python's logging doesn't support UTC formatting with microseconds. We need to write our own customized logger for this. This will be addressed in the future in a separate PR.