Closed sarah-witt closed 1 year ago
The merge of https://github.com/jenkinsci/datadog-plugin/pull/346 did not take into account the changes from https://github.com/jenkinsci/datadog-plugin/pull/345 causing build failures. This PR refactors the getAWSInstanceId to use the new HTTP client
Additionally changes the order of the hostname collection so that the AWS instance param takes precedence over the HOSTNAME variable
changelog/
backward-incompatible
do-not-merge/
kind/
severity/
What does this PR do?
The merge of https://github.com/jenkinsci/datadog-plugin/pull/346 did not take into account the changes from https://github.com/jenkinsci/datadog-plugin/pull/345 causing build failures. This PR refactors the getAWSInstanceId to use the new HTTP client
Additionally changes the order of the hostname collection so that the AWS instance param takes precedence over the HOSTNAME variable
Description of the Change
Alternate Designs
Possible Drawbacks
Verification Process
Additional Notes
Release Notes
Review checklist (to be filled by reviewers)
changelog/
label attached. If applicable it should have thebackward-incompatible
label attached.do-not-merge/
label attached.kind/
andseverity/
labels attached at least.