Age | Commit message (Collapse) | Author |
|
It doesn't make any sense to build it when skip compile.
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
|
|
This makes debug easier.
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
|
|
-f, --apply-failed Apply failed patch in the repo after upgrade is done
And add "FAILED:" in subject when failed. So that the user can go on
working based on the commit.
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
|
|
E.g.,:
$ upgradehelper.py less
commit d9b551438037e105eebabbbcb2da6d8b8cae7504
Author: Robert Yang <liezhi.yang@windriver.com>
Date: Wed Dec 6 16:38:55 2017 +0800
less: upgrade to 529
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
|
|
E.g.:
$ upgradehelper.py less strace bash git
The commit is removed when failed, and kept when succeed, but it would be
removed when next recipe runs, so only run clean_repo once can keep the commit,
which is helpful for the user.
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
|
|
For example, first run:
$ upgradehelper.py less -d 5
It did the upgrade
Second run:
$ upgradehelper.py less -d 5
DEBUG: Skipping upgrade of less: is in history and not 30 days passed
Let it always do the upgrade makes it easier to use when do upgrade locally.
It will still do the check when the recipe is all.
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
|
|
Fixed:
$ upgradehelper.py less
ERROR: smtp host not set! Sending emails disabled!
ERROR: 'From' address not set! Sending emails disabled!
Only check email settings when "-e" is specified can fix the problem.
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
|
|
* The previous code lacks a "% var" in the end:
W(" %s: Compilation was skipped by user choice!")
* Move the skipping steps to upgradehelper.py rather than
modules/steps.py, do not run compile is more straight-forward than
return early from it when skipping.
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
|
|
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
|
|
* Use UniverseUpdater() for the following 3 cases:
+ Upgrade all recipes
+ Upgrade 1 recipe
- '--maintainer' is not a must when any more when use --send-emails, the
maintainer be got from distrodata.
+ Upgrade multiple recipes
* Use "args" as the parameter of UniverseUpdater() and Updater(), this can make
the parameters simple, and easy for extending.
* Make upgrade "all" recipes respect the args, it didn't care --send-emails or
--maintainer which would suprise the user.
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
|
|
Now we can use:
$ upgradehelper.py recipe1 recipe2
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
|
|
The error message in the log is:
Initialising tasks...ERROR: Task do_checkpkg does not exist for target strace [snip]
So line.find("ERROR: Task do_checkpkg does not exist") == 0 doesn't
work, use != -1 to fix the problem.
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
|
|
When is executing in normal mode the poky respository may be
leaves on uncleaned state, so clean it.
Signed: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
When the layer mode is enabled and the upgradehelper is run
with all, there is a need to discover what recipes are provides
the layer to only try to upgrade those.
[YOCTO #8962]
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
[YOCTO #8962]
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
[YOCTO #8962]
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
[YOCTO #8962]
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
[YOCTO #8962]
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
Those settings are to support layer recipe upgrades runs
into the AUH.
[YOCTO #8962]
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
This handler looks for files not shipped but currently isn't
working causing a infinite recursion loop.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
Upgrade status email should indicate the URL of the log folder instead of the tar URL
[YOCTO #10670]
Signed-off-by: Edwin Plauchu <edwin.plauchu.camacho@linux.intel.com>
|
|
If do_package fails and isn't files not shipped related don't try to
execute again the compile step because will cause infinite recursion.
Now raise a new exception for PackageError.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
In certain cases a patch could be fail to apply at level of integration
for do a testimage, so if a patch fails to apply add the ability to
abort git am session and mark the package as integration error.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
The _order_pkgs_to_upgrade function order a set of packages to
be upgraded based on bitbake dependency graph, currently _order_pkgs_to_upgrade
is broken so disable it while fix.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
If no packages are found to upgrade, it didn't make sense to build
gcc runtime before is only a waste of time.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
[YOCTO #9747]
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
The original workdir directory is needed for make diff's for example
when license change [1], in order to avoid deletion of original workdir
when recipe is upgraded.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
[1] https://lists.yoctoproject.org/pipermail/yocto/2016-January/028025.html
|
|
Now buildhistroy generate two diff files,
- buildhistory-diff.txt: Simple diff generated without -a
option.
- buildhistory-diff-full.txt: Full diff generated contains
all the packages including -dbg and -dev.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
Add support for generate a tarball with the work directory.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
Add case for take into account QA error and if the recipe that
fails is image one don't try to handle IntegrationError because
the error isn't directly caused by recipe upgrade.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
If recipe upgrade fails in testimage integration send the patch
to the maintainer pointing it instead of treat as error.
This helps the maintainer to find the cause and the patch will
be useful.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
If an IntegrationError happen the recipe upgrade is removed from the
build for try again to avoid QA error caused for the recipe version
that is minor now.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
Instead of remove patches when fails to apply now comment it
this helps the maintainer to review why the patch failed to
apply.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
When AUH is trying to do upgrades every time that fails the error
log is overwrite so now append the errors this provides better
complete error info to the Maintainer.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
Modularize code in order to keep upgradehelper small so move
the code from main module to testimage.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
If patch_file isn't generated treat it as an error this avoid
future exceptions in package_handler when try to send/save email.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
To enable more generic functions and to pass state over multiple calls
add args and kwargs to modify_recipe_files.
Some functions require to maintain states and pass information to
functions using this decorator.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
default one.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
Since recipe upgrades are done in isolate manner cleaning the repo
for every recipe [1] the drop_previous_commits becomes deprecated.
Also now we have Testimage integration phase that handles/reports
this kind of errors.
[1] http://git.yoctoproject.org/cgit/cgit.cgi/auto-upgrade-helper/commit/?h=boot_images_v2&id=8769030a02753f09ed6b5e7c98ed1c442a51a466
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
A recipe can compile alone when the upgrade proccess is made
but when integrate it can fail due to dependencies of another
recipes.
If build image fail then analyze logs to find what recipe failed
if recipe is found directly on the set of recipe upgrades remove it,
if not recipe found directly then search within dependencies of
failed recipe.
Since IntegrationError can happen now, move the statistics update and
directory symlink creation after testimage.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
Instead of add manually the ptest packages use IMAGE_FEATURE to
install ptest packages this save time validating if ptest pkg
exist.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
Before this commit core-image-sato was hardcoded for image
to test with this commit image name can be specified in
the upgrade-helper.conf.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|
|
Add FileHandler log to store the log into workdir, this needs
to move up the work directory creation.
Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
|