Enhance Internal CI reliability #42
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Gabor Abonyi <[email protected]>
For non-ML applications there is no need to build and run for AUDIO and INFERENCE variants as these configurations should be relevant for ML applications only. Non-ML applications would use the default values only. Thus, separating non-ML applications pipeline configurations from the ML applications pipeline configurations. Signed-off-by: Ahmed Ismail <[email protected]>
Ensure the URL of the files reported in the TPIP comment added to the Merge Request contains the correct branch name. Signed-off-by: Hugues Kamba-Mpiana <[email protected]>
FVP VSI python workaround shall be done as part of the docker image generation rather than being done for each VSI test which should significantly reduce CI execution time. Signed-off-by: Ahmed Ismail <[email protected]>
This commit is meant to do the following: * Modify test-application-output job to test blinky application only as it can be used as a smoke test. * Test ML inference output for both keyword-detection and speech-recognition applications as part of the OTA test job which is now renamed to test_applications as it is testing OTA update and ML inference outputs as well. Note: * The mentioned changes would result in a better reliability and better CI performance. Signed-off-by: Ahmed Ismail <[email protected]>
urutva
approved these changes
Feb 6, 2024
amazonKamath
approved these changes
Feb 6, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR introduce two major improvements:
Enhance internal pipeline execution time and reliability.
Fix internal CI TPIP checker reported file URL.
Enhancing the internal pipeline execution time is done by the following:
Output metrics:
Totally 40 jobs have been eliminated (8 build jobs and 32 test jobs).
Totally around 50 minutes have been reduced.
Test Steps
Checklist:
Related Issue
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.