AC-15461:Migration New Relic from REST v2 to NerdGraph (GraphQL)

Author: sandesh-as

app/code/Magento/NewRelicReporting/Console/Command/DeployMarker.php

@@ -166,8 +166,8 @@ private function displayDeploymentDetails(OutputInterface $output, array $deploy
                 ) : 'N/A']
         ];
 
-        if (!empty($deployment['change_log'])) {
-            $rows[] = ['Change log', $deployment['change_log']];
+        if (!empty($deployment['changelog'])) {
+            $rows[] = ['Change log', $deployment['changelog']];
         }
         if (!empty($deployment['commit'])) {
             $rows[] = ['Commit', $deployment['commit']];

app/code/Magento/NewRelicReporting/Model/Apm/Deployments.php

@@ -151,7 +151,7 @@ private function setV2RestDeployment(string $description, bool|string $changelog
         $params = [
             'deployment' => [
                 'description' => $description,
-                'change_log' => $changelog,
+                'changelog' => $changelog,
                 'user' => $user,
                 'revision' => $revision
             ]
@@ -166,7 +166,15 @@ private function setV2RestDeployment(string $description, bool|string $changelog
         }
 
         if ($response->getStatusCode() < 200 || $response->getStatusCode() > 210) {
-            $this->logger->warning('Deployment marker request did not send a 200 status code.');
+            $this->logger->warning(
+                'Deployment marker request did not send a 200 status code.',
+                [
+                    'status_code' => $response->getStatusCode(),
+                    'response_body' => $response->getBody(),
+                    'request_url' => $apiUrl,
+                    'request_params' => $params
+                ]
+            );
             return false;
         }
 

app/code/Magento/NewRelicReporting/Model/NerdGraph/DeploymentTracker.php

@@ -216,7 +216,7 @@ private function addOptionalFields(
         ?string $groupId
     ): void {
         if ($changelog) {
-            $variables['deployment']['change_log'] = $changelog;
+            $variables['deployment']['changelog'] = $changelog;
         }
 
         if ($user) {
@@ -278,7 +278,7 @@ private function processDeploymentResponse(
                 'entityGuid' => $deploymentData['entityGuid'],
                 'version' => $deployedVersion,
                 'description' => $description,
-                'change_log' => $changelog,
+                'changelog' => $changelog,
                 'user' => $user,
                 'commit' => $commit,
                 'deepLink' => $deepLink,

app/code/Magento/NewRelicReporting/Test/Unit/Console/Command/DeployMarkerTest.php

@@ -181,7 +181,7 @@ public function testExecuteSuccessWithArrayResultNerdGraphStyle()
             'description' => $message,
             'user' => $user,
             'timestamp' => 1234567890000, // Feb 13, 2009 23:31:30 UTC
-            'change_log' => 'New features',
+            'changelog' => 'New features',
             'commit' => $commit,
             'deepLink' => $deepLink,
             'groupId' => $groupId
@@ -452,7 +452,7 @@ public function testExecuteSuccessWithCompleteDeploymentDataNerdGraphStyle()
             'description' => $message,
             'user' => 'complete-user',
             'timestamp' => 1640995200000, // 2022-01-01 00:00:00 UTC
-            'change_log' => 'Added new features',
+            'changelog' => 'Added new features',
             'commit' => 'abc123def456',
             'deepLink' => 'https://github.com/test/releases/v2.0.0',
             'groupId' => 'production'

app/code/Magento/NewRelicReporting/Test/Unit/Model/Apm/DeploymentsTest.php

@@ -872,7 +872,7 @@ private function getDataVariables(): array
         $params = [
             'deployment' => [
                 'description' => $description,
-                'change_log' => $changelog,
+                'changelog' => $changelog,
                 'user' => $user,
                 'revision' => $revision
             ]

app/code/Magento/NewRelicReporting/Test/Unit/Model/NerdGraph/DeploymentTrackerTest.php

@@ -112,7 +112,7 @@ public function testCreateDeploymentSuccess()
                            $variables['deployment']['entityGuid'] === $entityGuid &&
                            $variables['deployment']['description'] === $description &&
                            $variables['deployment']['version'] === $version &&
-                           $variables['deployment']['change_log'] === $changelog &&
+                           $variables['deployment']['changelog'] === $changelog &&
                            $variables['deployment']['user'] === $user &&
                            $variables['deployment']['commit'] === $commit &&
                            $variables['deployment']['deepLink'] === $deepLink &&
@@ -140,7 +140,7 @@ public function testCreateDeploymentSuccess()
         $this->assertEquals($entityGuid, $result['entityGuid']);
         $this->assertEquals($version, $result['version']);
         $this->assertEquals($description, $result['description']);
-        $this->assertEquals($changelog, $result['change_log']);
+        $this->assertEquals($changelog, $result['changelog']);
         $this->assertEquals($user, $result['user']);
         $this->assertEquals($commit, $result['commit']);
         $this->assertEquals($deepLink, $result['deepLink']);
@@ -180,7 +180,7 @@ public function testCreateDeploymentWithMinimalParameters()
                     return isset($variables['deployment']) &&
                            $variables['deployment']['entityGuid'] === $entityGuid &&
                            $variables['deployment']['description'] === $description &&
-                           !isset($variables['deployment']['change_log']) &&
+                           !isset($variables['deployment']['changelog']) &&
                            !isset($variables['deployment']['user']);
                 })
             )
@@ -542,7 +542,7 @@ public function testCreateDeploymentWithLongParameters()
                     'entityGuid' => $entityGuid,
                     'timestamp' => 1234567890000,
                     'description' => $description,
-                    'change_log' => $changelog,
+                    'changelog' => $changelog,
                     'user' => $user
                 ]
             ]
@@ -606,7 +606,7 @@ public function testCreateDeploymentMutationStructure()
                     return isset($variables['deployment']) &&
                            $variables['deployment']['entityGuid'] === $entityGuid &&
                            $variables['deployment']['description'] === $description &&
-                           $variables['deployment']['change_log'] === $changelog &&
+                           $variables['deployment']['changelog'] === $changelog &&
                            $variables['deployment']['user'] === $user &&
                            $variables['deployment']['version'] === $version &&
                            $variables['deployment']['commit'] === $commit &&