Skip to content

Attach CI_JOB_URL to json report as well

Lin Jen-Shin requested to merge attach-ci_job_url-to-qa-json-report into master

What does this MR do?

Attach CI_JOB_URL to json report as well

In !44602 (merged) we added CI_JOB_ID but what we really need is actually CI_JOB_URL 😞

I might be confused with this because we also defeind ci_job_url in a separate way, and they'll be conflicting with each others so we need some more changes in gitlab-qa to separate the concern...

Now I remembered why I missed this. At first I was trying to use CI_JOB_URL but it's not available, then I figured I can parse the report filename for CI_JOB_ID then I went for it, however then we figured we can just pass it along from ENV and then I forgot what actually needed was CI_JOB_URL 😞

Edited by Lin Jen-Shin

Merge request reports