ci/lava: Let LAVA job submitter run without JWT file
Make jwt-file argument optional, this means that this submitter could deal with more generic use cases, such as running it locally, since the MINIO JWT is a sensitive information, which is not really required to test if the submitter is working well. Signed-off-by: Guilherme Gallo <guilherme.gallo@collabora.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/15938>
This commit is contained in:

committed by
Marge Bot

parent
18d80f25ee
commit
9666ab7172
@@ -146,17 +146,23 @@ def generate_lava_yaml(args):
|
|||||||
# - inline .gitlab-ci/common/init-stage1.sh
|
# - inline .gitlab-ci/common/init-stage1.sh
|
||||||
# - fetch and unpack per-pipeline build artifacts from build job
|
# - fetch and unpack per-pipeline build artifacts from build job
|
||||||
# - fetch and unpack per-job environment from lava-submit.sh
|
# - fetch and unpack per-job environment from lava-submit.sh
|
||||||
# - exec .gitlab-ci/common/init-stage2.sh
|
# - exec .gitlab-ci/common/init-stage2.sh
|
||||||
run_steps = []
|
|
||||||
|
|
||||||
with open(args.first_stage_init, 'r') as init_sh:
|
with open(args.first_stage_init, 'r') as init_sh:
|
||||||
run_steps += [ x.rstrip() for x in init_sh if not x.startswith('#') and x.rstrip() ]
|
run_steps += [ x.rstrip() for x in init_sh if not x.startswith('#') and x.rstrip() ]
|
||||||
|
|
||||||
with open(args.jwt_file) as jwt_file:
|
if args.jwt_file:
|
||||||
|
with open(args.jwt_file) as jwt_file:
|
||||||
|
run_steps += [
|
||||||
|
"set +x",
|
||||||
|
f'echo -n "{jwt_file.read()}" > "{args.jwt_file}" # HIDEME',
|
||||||
|
"set -x",
|
||||||
|
f'echo "export CI_JOB_JWT_FILE={args.jwt_file}" >> /set-job-env-vars.sh',
|
||||||
|
]
|
||||||
|
else:
|
||||||
run_steps += [
|
run_steps += [
|
||||||
"set +x",
|
"echo Could not find jwt file, disabling MINIO requests...",
|
||||||
f'echo -n "{jwt_file.read()}" > "{args.jwt_file}" # HIDEME',
|
"unset MINIO_RESULTS_UPLOAD",
|
||||||
"set -x",
|
|
||||||
]
|
]
|
||||||
|
|
||||||
run_steps += [
|
run_steps += [
|
||||||
@@ -164,7 +170,9 @@ def generate_lava_yaml(args):
|
|||||||
'wget -S --progress=dot:giga -O- {} | tar -xz -C {}'.format(args.build_url, args.ci_project_dir),
|
'wget -S --progress=dot:giga -O- {} | tar -xz -C {}'.format(args.build_url, args.ci_project_dir),
|
||||||
'wget -S --progress=dot:giga -O- {} | tar -xz -C /'.format(args.job_rootfs_overlay_url),
|
'wget -S --progress=dot:giga -O- {} | tar -xz -C /'.format(args.job_rootfs_overlay_url),
|
||||||
f'echo "export CI_JOB_JWT_FILE={args.jwt_file}" >> /set-job-env-vars.sh',
|
f'echo "export CI_JOB_JWT_FILE={args.jwt_file}" >> /set-job-env-vars.sh',
|
||||||
'exec /init-stage2.sh',
|
# Putting CI_JOB name as the testcase name, it may help LAVA farm
|
||||||
|
# maintainers with monitoring
|
||||||
|
f"lava-test-case 'mesa-ci_{args.mesa_job_name}' --shell /init-stage2.sh",
|
||||||
]
|
]
|
||||||
|
|
||||||
values['actions'] = [
|
values['actions'] = [
|
||||||
|
Reference in New Issue
Block a user