[7.17] Remove github-checks-reporter (#228507)

We've mostly moved over to fewer checks in favor of PR comments to
reduce the number of API calls. This is already removed on other
branches and the application will be shutdown after.

Author: jbudz

.buildkite/scripts/common/env.sh

@@ -61,12 +61,6 @@ if is_pr; then
     export ELASTIC_APM_ACTIVE=false
   fi
 
-  if [[ "${GITHUB_STEP_COMMIT_STATUS_ENABLED:-}" != "true" ]]; then
-    export CHECKS_REPORTER_ACTIVE=true
-  else
-    export CHECKS_REPORTER_ACTIVE=false
-  fi
-
   # These can be removed once we're not supporting Jenkins and Buildkite at the same time
   # These are primarily used by github checks reporter and can be configured via /github_checks_api.json
   export ghprbGhRepository="elastic/kibana"

.buildkite/scripts/common/util.sh

@@ -2,15 +2,6 @@
 
 source "$(dirname "${BASH_SOURCE[0]}")/vault_fns.sh"
 
-checks-reporter-with-killswitch() {
-  if [ "$CHECKS_REPORTER_ACTIVE" == "true" ] ; then
-    yarn run github-checks-reporter "$@"
-  else
-    arguments=("$@");
-    "${arguments[@]:1}";
-  fi
-}
-
 is_pr() {
   [[ "${GITHUB_PR_NUMBER-}" ]] && return
   false

.buildkite/scripts/saved_object_field_metrics.sh

@@ -6,8 +6,7 @@ source .buildkite/scripts/common/util.sh
 
 echo '--- Default Saved Object Field Metrics'
 cd "$XPACK_DIR"
-checks-reporter-with-killswitch "Capture Kibana Saved Objects field count metrics" \
-  node scripts/functional_tests \
-      --debug --bail \
-      --kibana-install-dir "$KIBANA_BUILD_LOCATION" \
-      --config test/saved_objects_field_count/config.ts
+node scripts/functional_tests \
+    --debug --bail \
+    --kibana-install-dir "$KIBANA_BUILD_LOCATION" \
+    --config test/saved_objects_field_count/config.ts

.buildkite/scripts/steps/check_types.sh

@@ -7,5 +7,4 @@ source .buildkite/scripts/common/util.sh
 .buildkite/scripts/bootstrap.sh
 
 echo --- Check Types
-checks-reporter-with-killswitch "Check Types" \
-  node scripts/type_check
+node scripts/type_check

.buildkite/scripts/steps/checks/bundle_limits.sh

@@ -5,6 +5,4 @@ set -euo pipefail
 source .buildkite/scripts/common/util.sh
 
 echo --- Check Bundle Limits
-
-checks-reporter-with-killswitch "Check Bundle Limits" \
-  node scripts/build_kibana_platform_plugins --validate-limits
+node scripts/build_kibana_platform_plugins --validate-limits

.buildkite/scripts/steps/checks/commit/commit.sh

@@ -10,5 +10,4 @@ source .buildkite/scripts/common/util.sh
 # If files are more than 200 we will skip it and just use
 # the further ci steps that already check linting and file casing for the entire repo.
 echo --- Quick commit checks
-checks-reporter-with-killswitch "Quick commit checks" \
-  "$(dirname "${0}")/commit_check_runner.sh"
+"$(dirname "${0}")/commit_check_runner.sh"

.buildkite/scripts/steps/checks/doc_api_changes.sh

@@ -5,5 +5,4 @@ set -euo pipefail
 source .buildkite/scripts/common/util.sh
 
 echo --- Check Doc API Changes
-checks-reporter-with-killswitch "Check Doc API Changes" \
-  node scripts/check_published_api_changes
+node scripts/check_published_api_changes

.buildkite/scripts/steps/checks/file_casing.sh

@@ -5,5 +5,4 @@ set -euo pipefail
 source .buildkite/scripts/common/util.sh
 
 echo --- Check File Casing
-checks-reporter-with-killswitch "Check File Casing" \
-  node scripts/check_file_casing --quiet
+node scripts/check_file_casing --quiet

.buildkite/scripts/steps/checks/i18n.sh

@@ -5,5 +5,4 @@ set -euo pipefail
 source .buildkite/scripts/common/util.sh
 
 echo --- Check i18n
-checks-reporter-with-killswitch "Check i18n" \
-  node scripts/i18n_check --ignore-missing
+node scripts/i18n_check --ignore-missing

.buildkite/scripts/steps/checks/jest_configs.sh

@@ -5,5 +5,4 @@ set -euo pipefail
 source .buildkite/scripts/common/util.sh
 
 echo --- Check Jest Configs
-checks-reporter-with-killswitch "Check Jest Configs" \
-  node scripts/check_jest_configs
+node scripts/check_jest_configs