-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
lmp/build: check availability of the image manifest #313
base: master
Are you sure you want to change the base?
Conversation
Validated with |
433915d
to
d57120e
Compare
df8154b
to
4df8310
Compare
In this case the file not exist and so the glob will not expand
|
4df8310
to
4c9335b
Compare
What causes the image manifest to not be generated? |
Good question, it could be that it is generated with a different name but I don't know. |
instead of having a message we can make this an error and so it becomes clearer what happened. |
We had working lmp-base ci runs before as well, which is why the question. |
I will debug it more to get the reason. |
6f1f69f
to
cfacbf9
Compare
Signed-off-by: Jose Quaresma <[email protected]>
Skip the loop if image manifest not found, instead of failing blindly. Also show some status about not found license manifests. Fix the following: | + for img in ${DEPLOY_DIR_IMAGE}/*${MACHINE}.manifest | ++ basename '/srv/oe/build/deploy/images/intel-corei7-64/*intel-corei7-64.manifest' | ++ sed -e s/.manifest// | + image_name='*intel-corei7-64' | ++ readlink '/srv/oe/build/deploy/images/intel-corei7-64/*intel-corei7-64.manifest' | ++ sed -e 's/\..*manifest//' | + image_name_id= | Script completed with error(s) Signed-off-by: Jose Quaresma <[email protected]>
Skip the loop if image manifest not found, instead of failing blindly.
Also show some status about not found license manifests.
Fix the following: