Skip to content

Commit d98a281

Browse files
committed
add comment
1 parent b60c139 commit d98a281

File tree

1 file changed

+13
-0
lines changed

1 file changed

+13
-0
lines changed

.github/workflows/application-signals-e2e-test.yml

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -248,5 +248,18 @@ jobs:
248248
caller-workflow-name: 'main-build'
249249
staging-wheel-name: ${{ inputs.staging-wheel-name }}
250250

251+
# This validation is to ensure that all test workflows relevant to this repo are actually
252+
# being used in this repo, which is referring to all the other jobs in this file.
253+
#
254+
# If this starts failing, then it most likely means that new e2e test workflow was
255+
# added to `aws-observability/aws-application-signals-test-framework`, but was not
256+
# added to this file. It could also mean that a test in this file has been removed.
257+
#
258+
# If a particular test file is intended to not be tested in this repo and should not
259+
# be failing this particular validation, then choose one of the following options:
260+
# - Add the test file to the exclusions input (CSV format) to the workflow
261+
# (see: https://github.com/aws-observability/aws-application-signals-test-framework/blob/main/.github/workflows/validate-e2e-tests-are-accounted-for.yml#L1)
262+
# - Update the `validate-e2e-tests-are-accounted-for` job to change which "workflow files are expected to be used by this repo"
263+
# (see: https://github.com/aws-observability/aws-application-signals-test-framework/blob/main/.github/workflows/validate-e2e-tests-are-accounted-for.yml)
251264
validate-all-tests-are-accounted-for:
252265
uses: aws-observability/aws-application-signals-test-framework/.github/workflows/validate-e2e-tests-are-accounted-for.yml@main

0 commit comments

Comments
 (0)