Age | Commit message (Collapse) | Author |
|
Some tests fail if relative paths are provided to patchtest.
Converting --repo-dir and --start-dir arguments to abosolute
paths should make paths absolute, which is what patchset
code currently assumes.
Signed-off-by: Ed Bartosh <ed.bartosh@linux.intel.com>
|
|
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
This allows complete separation between GUEST and SHARE folder, each having
its own repositories.
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
(pattern)
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Starting at a certain commit in the past, run patchtest until HEAD.
This is useful for scripts requiring pretest execution because
testing patch can be merge in every iteration.
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
The oe-core setup is the way is indicated in the subject, so oe-init-build-env
script finds bitbake in the standard place.
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Target commit and branch are set depending the context, so base names
may not match target ones. Use target instead of base at least for the
logging part.
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Specify command line arguments have greater preference that the rest. In this
case, if commit is not given through the command line, look for the branch
(also from the cmd line) before going into the rest (patch data, then current repo)
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
If user either specify the branch through the command line or through
the patch, use that data instead of the branch hash, allowing a more
useful target branch on reports.
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
patchtest creates a temporal branch to do its work and at the end it moves
back to the branch where it started. Default to master in case initial
repo state is detached.
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
If a particular command fails, the system needs to show at least
what the command was.
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Patchtest and its test suite is intended for oe-core so set it as the reference
repo.
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
This commit partially reverts [1] and the reason is that patchtest and its
test suite is intended for oe-core, not Poky.
[1] 2563d4624dbf546d100d401027c2903338edcd39
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
The guest machine should be constructed outside the BASE folder, which
is the folder to be shared between host and guest, so changing the
instructions to indicate the latter.
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Poky, not oe-core, is the repo that patchtest uses as final repository to
do all its work, so correct the host documentation.
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
This filter is already done inside the create-summary script so removing it
from post script.
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
The series/revision is included by default by patchwork [1], so there is no
need to include it again at the summary.
[1] c1cea7c880fb2babd36c6eaa2204ed92ef5da843 at patchwork repo.
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Now the results contain a commit "object" (really a dict) we need to
select what to show. Select the subject since that will contain the
[PATCH x/y] sequence number that may be useful in identifying the patch
in question.
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
|
|
At the moment we're referring to a module-level variable ("opt") in the
function, and we were passing certain parameters into the rendering that
aren't necessary (template name, output file name). To fix both of
these, pass some of these explicitly to the function and then pass the
parameters to go into the rendering separately.
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
|
|
In order to place the emphasis on information that is most important to
the submitter, we need to alter the presentation in the default jinja2
template to present the commit first (if applicable), the issue (suffixed
with the test name), the suggested fix (if any), and then any optional
additional fields.
In order to do this and at the same time allow the template a bit more
control over formatting, require tests to send a dict with "id" and
"issue" elements, plus optionally "fix", "commit" and "data". These are
passed verbatim into the jinja2 template when formatting the summary.
(As a general rule though for human readability I believe we should be
expressing additional information as part of the issue / fix text rather
than supplying it in field form via the data parameter.)
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
|
|
The script has a --fail option, it seems logical that it should filter
the results passed through it to include only failures rather than
relying on an intermediate grep to do the same.
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
|
|
Touch up the grammar slightly.
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
|
|
Importing the functions directly from Python modules such as re and
os is unusual, not to mention that Python has a built-in compile()
function already. Improve readability by importing the modules as
normal and calling functions within them.
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
|
|
Important for future Python 3 compatibility. Drop one useless print
statement in the process.
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
|
|
When I ran patchtest I ended up with IDs such as
"tests.test_metadata_src_uri.SrcUri.test_src_uri_checksums_not_changed",
which has one extra item beyond what the code here was expecting. Use
the last three items so that it works in either situation.
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
|
|
These trigger dependencies on GitPython and requests and aren't actually
being used right now, so remove them.
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
|
|
If we point to /usr/bin/python we break the ability to use the scripts
within virtualenv, so use /usr/bin/env python instead.
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
|
|
If the user has invoked/set up patchtest incorrectly such that there
aren't any tests, we should tell them rather than silently exiting.
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
|
|
* Rewrap lines
* Remove erroneous - after project name
* Fix first sentence to be a bit more grammatically correct
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
|
|
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
With this change, merge works correctly on non-master branches.
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
kernel and rootfs command line arguments effectively replace the image argument.
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
In the past, there was the intention to use patchtest-oe as git submodule,
but at the end it was decided to move it apart of the project, thus patchtest
is test suite agnostic.
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
In the past, the vagrant project was used to manage a guest machine. Currently,
we use Poky to generate and a its qemu runner to launch it. There is no need
to track this folder anymore.
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|
|
Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
|