mrproliu commented on a change in pull request #62:
URL: 
https://github.com/apache/skywalking-infra-e2e/pull/62#discussion_r739628583



##########
File path: docs/en/setup/Run-E2E-Tests.md
##########
@@ -31,9 +31,23 @@ e2e cleanup
 
 To use skywalking-infra-e2e in GitHub Actions, add a step in your GitHub 
workflow.
 
+The working directory is uploaded to GitHub Action Artifact after the task is 
completed, which contains environment variables and container logs in the 
environment.
+
 ```yaml
 - name: Run E2E Test
   uses: apache/skywalking-infra-e2e@main      # always prefer to use a 
revision instead of `main`.
   with:
-    e2e-file: e2e.yaml                        # need to run E2E file path
+    e2e-file: e2e.yaml                        # (required)need to run E2E file 
path
+    log-dir: /path/to/log/dir                 # (not required)the container 
logs path, if not provide it would be auto generation
+```
+
+If you want to upload the log directory to the GitHub Action Artifact when 
this E2E test failure, you could define the below content in your GitHub Action 
Job.
+
+```yaml
+- name: Upload E2E Log
+  uses: actions/upload-artifact@v2
+  if: ${{ failure() }}                      # Only upload the artifact when 
E2E testing failure
+  with:
+    name: e2e-log
+    path: "${{ env.SW_INFRA_E2E_LOG_DIR }}" # The SkyWalking Infra E2E action 
would provide this environment

Review comment:
       Yes, is support. The Infra E2E GHA could convert the path(absolute path 
or not) and setup to the environment.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to