PR Comments

.github/composite_actions/log_cw_metric/action.yaml

@@ -1,5 +1,5 @@
 name: Log CW Metric
-description: Test to log a CW metric
+description: Logs a metric to Cloudwatch.
 inputs:
   # For getting failing step
   job-status:

.github/composite_actions/log_cw_metric_wrapper/action.yaml

@@ -1,5 +1,5 @@
 name: Log CW Metric Wrapper
-description: Test to log a CW metric
+description: Wrap additional inputs around the log_cw_metric action.
 
 # Wraps log metric by configuring AWS credentials and adding github repo + run id inputs.
 inputs:
@@ -58,17 +58,17 @@ runs:
       run: |
         if [ "${{ github.event_name }}" != "schedule" ]; then
           echo "This was not triggered by a schedule, skipping."
-          echo "SKIP=true" >> $GITHUB_ENV
+          echo "SKIP_CW=true" >> $GITHUB_ENV
         fi
 
     - uses: actions/checkout@f43a0e5ff2bd294095638e18286ca9a3d1956744 # 3.6.0
-      if: env.SKIP != 'true'
+      if: env.SKIP_CW != 'true'
       with:
         persist-credentials: false
         submodules: true
 
     - name: Configure AWS credentials
-      if: env.SKIP != 'true'
+      if: env.SKIP_CW != 'true'
       uses: aws-actions/configure-aws-credentials@04b98b3f9e85f563fb061be8751a0352327246b0 # 3.0.1
       with:
         unset-current-credentials: true
@@ -77,7 +77,7 @@ runs:
         role-duration-seconds: 900
 
     - name: Log the CW Metrics
-      if: env.SKIP != 'true'
+      if: env.SKIP_CW != 'true'
       uses: ./.github/composite_actions/log_cw_metric
       with:
         job-status: ${{ inputs.job-status }}

actions/bin/log_cw_metric.dart

@@ -40,8 +40,6 @@ Future<void> logMetric() async {
       ? await getFailingStep(jobIdentifier, githubToken, repo, runId)
       : '';
 
-  core.info('Failing step was: $failingStep');
-
   // Inputs for Metric
   final metricName = core.getRequiredInput('metric-name');
   final testType = core.getRequiredInput('test-type');
@@ -137,12 +135,11 @@ Future<String> getFailingStep(
     final matchingJob =
         jobsList.jobs.firstWhere((job) => job.name == jobIdentifier);
     final steps = matchingJob.steps;
-    core.info('steps $steps');
 
     final failingStep = steps.firstWhere(
       (element) => element.conclusion == 'failure',
     );
-    core.info('failingStep was: ${failingStep.name}');
+    core.info('FailingStep was: ${failingStep.name}');
 
     return failingStep.name;
   } on Exception catch (e) {