DataDog / datadogpy

The Datadog Python library
https://datadoghq.com/
Other
609 stars 302 forks source link

Add a dogshell option to change Datadog site to call API #691

Closed aquiladayc closed 2 years ago

aquiladayc commented 2 years ago

What does this PR do?

Issue to fix https://github.com/DataDog/datadogpy/issues/652

Description of the Change

Adding --site option to change the site to call the API

Verification Process

Tested with my sandbox account in EU and US3 site.

aquiladayc commented 2 years ago

@DataDog/integrations-tools-and-libraries can anyone review please?

aquiladayc commented 2 years ago

@therve Updated, can you review please?

therve commented 2 years ago

/azp run

azure-pipelines[bot] commented 2 years ago
Azure Pipelines successfully started running 2 pipeline(s).
therve commented 2 years ago

/azp run

azure-pipelines[bot] commented 2 years ago
Azure Pipelines successfully started running 2 pipeline(s).
therve commented 2 years ago

/azp run

azure-pipelines[bot] commented 2 years ago
Azure Pipelines successfully started running 2 pipeline(s).
janorn commented 2 years ago

I know this has already been merged but just a tiny comment. The new option --api_host uses an under score where both other api options just uses a hyphen. --api-key --application-key