Skip to content
This repository has been archived by the owner on Jul 25, 2024. It is now read-only.

ci/backend/tuxsuite.py: Support for fetching test metadata file #1116

Merged
merged 1 commit into from
Dec 12, 2023

Conversation

katieworton
Copy link
Member

Add support for fetching the TuxSuite test metadata file and adding its data to the SQUAD metadata file for the given test.

@katieworton
Copy link
Member Author

katieworton commented Dec 11, 2023

Note: As far as I could see, download_url does not appear to be available in the results for tests (only builds). For this reason, I perform a get request on the job_url to get the download_url from the job API data.

@chaws
Copy link
Collaborator

chaws commented Dec 12, 2023

Note: As far as I could see, download_url does not appear to be available in the results for tests (only builds). For this reason, I perform a get request on the job_url to get the download_url from the job API data.

download_url isn't in current sample tests in SQUAD because that was recently added by Tuxsuite. You can do it like that:

        job_data = requests.get(job_url).json() # Remove this line

        # Add extra metadata from metadata file if it exists
        self.update_metadata_from_file(job_data=results, metadata=metadata)

Add support for fetching the TuxSuite test metadata file and adding its
data to the SQUAD metadata file for the given test.

Signed-off-by: Katie Worton <katie.worton@linaro.org>
@katieworton
Copy link
Member Author

Note: As far as I could see, download_url does not appear to be available in the results for tests (only builds). For this reason, I perform a get request on the job_url to get the download_url from the job API data.

download_url isn't in current sample tests in SQUAD because that was recently added by Tuxsuite. You can do it like that:

        job_data = requests.get(job_url).json() # Remove this line

        # Add extra metadata from metadata file if it exists
        self.update_metadata_from_file(job_data=results, metadata=metadata)

Perfect, thank you! I've now updated the patch to read download_url from the test results 😄

Copy link
Collaborator

@chaws chaws left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@chaws chaws merged commit ded8a23 into Linaro:master Dec 12, 2023
7 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants