Age | Commit message (Collapse) | Author |
|
get-target-branch was not stripping trailing square brackets. Modify the
strip call when a branch is detected so that it does.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
get-target-branch would only report master as the target branch for any
patch in its previous form. Refactor the script so that it correctly
parses branch names from the subject line, and more simply falls back to
master if no branch matches are detected.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Upgrade to match meta-python version. See:
https://git.openembedded.org/meta-openembedded/commit/?id=a38f430cfda457d32fc7b04eab22ebec3f30305b
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Update to be in sync with meta-openembedded. See:
https://git.openembedded.org/meta-openembedded/commit/?id=49b9f848a93083bd2b1b5ecfcf4646564d610c21
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
This was left behind during a previous upgrade.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Now that patchtest-oe's contents are merged to oe-core, we don't need to
reference the patchtest-oe repo or cloned instances of it, so remove
them.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Remove five scripts that are no longer needed.
- test-mboxes is obsolete now that the patchtest script can target a
directory of patches
- post-summary is not relevant with the current Patchwork instance
- guest and host scripts are obsoleted by new patchtest workflow
- any-fail relies on stdin workflow that is no longer used
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
- Add installation requirements and setup steps
- Provide basic instructions for using get-latest-series
- Describe patchtest usage in 'host' mode
- Mark 'guest' mode section as 'Under Construction'
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
git-pw is required for scripts such as get-latest-series to work, so add
it to the requirements list.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Update to use markdown syntax and replace old URLs with current
versions.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
The patchtest script requires a patch argument, but cannot handle a
directory or list of files. Add a mutual exclusion group in the argument
parser that allows a directory or a single patch, then make patchtest
iterate over the list of resulting files (which will consist of a single
element if only one patch is provided).
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Both of these scripts rely on functionality that is no longer available
in patchwork. The get-latest-series script added to patchtest in 2b397c3
is the replacement.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
The legacy PatchTestStdIn class and associated functions were intended
to allow patches to be sent from stdin, but the logic causes issues with
automation of selftests and other non-interactive workflows. Remove the
code in the patchtest script, and strip out the PatchTestStdIn class
since it is then no longer needed. Also rename the PatchTestArgs class
to PatchTestInput, since the original PatchTestInput class isn't needed
to wrap PatchTestArgs alone and provides the same functionality anyway.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
While testing on Ubuntu systems it was noticed that there was a
difference in selftest results, notably the SignedOffBy and
PatchSignedOffBy tests. After some debugging, it was narrowed down to
this error:
|Test error: test_patch_upstream_status (unittest.loader._FailedTest)
|(<class 'ImportError'>, ImportError('Failed to import test module:
|test_patch_upstream_status\nTraceback (most recent call last):\n File
|"/usr/lib/python3.10/unittest/loader.py", line 436, in _find_test_path\n
|module = self._get_module_from_name(name)\n File
|"/usr/lib/python3.10/unittest/loader.py", line 377, in
|_get_module_from_name\n __import__(name)\n File
|"/workspace/yocto/patchtest-oe/tests/test_patch_upstream_status.py",
|line 12, in <module>\n class PatchUpstreamStatus(base.Base):\n File
|"/workspace/yocto/patchtest-oe/tests/test_patch_upstream_status.py",
|line 14, in PatchUpstreamStatus\n upstream_status_re gex =
|pyparsing.AtLineStart("+" + "Upstream-Status")\nAttributeError: module
|\'pyparsing\' has no attribute \'AtLineStart\'\n'), <traceback object at
|0x7f934eb159c0>)
Comparing the version of pyparsing on the Ubuntu system (2.4.7) with a
Fedora 38 system (3.0.9) explained the error. To avoid this problem in
the future, make sure that pyparsing is pinned to 3.0.9 or newer when
installing the requirements.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
The current layout of patchtest is inconsistent in module and class
naming, as well as inter-dependencies. Start making the code more
maintainable by:
- Simplifying module names to remove namespacing
- Add namespacing to the classes themselves
- Make imports of classes explicit, instead of using aliases like "pti"
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
As a first step in replacing the non-functional poll-events script, add
a new one that uses git-pw and the date command to download patch series
by ID, with configurable time intervals, maximum number of series,
download locations, log files, and Patchwork projects to query. Use a
log file (defined as $SERIES_TEST_LOG in the script) to track
previously-consumed series so that they don't get tested multiple times.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
There's no real reason to keep the patchtestdata module in the root
directory given its similarity to the modules already present in lib.
Move it there to keep the repo more organized.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Changelog: https://github.com/sdispater/tomlkit/releases
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Changelog: https://github.com/platformdirs/platformdirs/releases
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Changelog: https://github.com/pylint-dev/astroid/releases
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Add some dependencies that patchtest now relies on to requirements.txt.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
patchtest's method of mounting a shared directory into the guest image
means that the associated content may not be owned by the image's user.
Add /home/patchtest/openembedded-core and
/home/patchtest/openembedded-core/bitbake to the patchtest user's
.gitconfig so that it can still run tests if the provided repo is owned
by someone else.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
These were mistakenly omitted during the python3-pylint upgrade and are
required for the new version to build, and to run ptests successfully.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
python3-pylint requires python3-tomlkit, so copy the recipe from
meta-python to avoid requiring that layer for patchtest.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
python3-pylint requires python3-platformdirs, so copy it from
meta-python to avoid requiring that layer for patchtest.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
python3-pylint requires python3-mccabe, so copy it from meta-python to
avoid requiring that layer for patchtest.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
python3-pylint requires python3-dill, so copy it from meta-python
to avoid requiring that layer for patchtest.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Copy the recipe from meta-python to handle the upgrade, so that
patchtest can build it without requiring the layer.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|
|
Copy the recipe from meta-python to handle the upgrade, so that
patchtest can build it without requiring the layer.
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
|