[GitHub] [skywalking-infra-e2e] mrproliu commented on pull request #62: Support save pod/container std log on the Environment

[GitBox <gi...@apache.org>]

mrproliu commented on pull request #62:
URL: https://github.com/apache/skywalking-infra-e2e/pull/62#issuecomment-954439141


   > > I hope to solve the problem by creating multiple directories by obtaining `job.name`, but I only found a solution [through API](https://docs.github.com/en/rest/reference/actions#workflow-jobs). Is anyone have another good idea?
   > 
   > 1. `job.name` doesn't distinguish the jobs spawned from a matrix, unless the `jobs.name` itself contains the matrix name (e.g. `- name: job-name-${{ matrix.storage }}`), this is not the case always.
   > 2. [the API](https://docs.github.com/en/rest/reference/actions#workflow-jobs) doesn't allow you to get the job id/name spawned from a matrix, also, it requires you to pass a `job_id`, if you already have the job id, you don't need this API to get id.
   
   I notice the `job_id` could not be found in GitHub Action Environment or Context. If we could get the `job_id`, then we could query the API to get the job name. 
   Should we generated my job name with matrix, such as: `${{github.job}}_$(echo '${{ toJSON(matrix) }}'|jq -r 'to_entries|map(.value)|join("_")')`, It's similar to the real job name. WDYT?


-- 
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: notifications-unsubscribe@skywalking.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org