In debian/watch no matching files for watch source https://gitlab.com/lava/lava/tags?sort=updated_desc
commit 13c42aa067d3880d54aef3e15a73395fed45b743
Merge: 36ec173f 826e87b0
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Fri Nov 28 10:18:06 2025 +0000
Merge branch 'pylint-enable-R1733' into 'master'
pylint: Enable R1733 Unnecessary dictionary index lookup
See merge request lava/lava!3071
commit 36ec173fe3f821ff3868c815bb7ac43f4baa3fdc
Merge: 98e0f9d6 534f37f9
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Fri Nov 28 10:17:30 2025 +0000
Merge branch 'pylint-enable-C0201' into 'master'
pylint: Enable C0201 Consider iterating the dictionary directly
See merge request lava/lava!3070
commit 98e0f9d6d0dd13943e5dd1a1f946b50b396a1a2a
Merge: 819cf883 84eb1ab3
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Fri Nov 28 10:16:38 2025 +0000
Merge branch 'expected-tc' into 'master'
lava_dispatcher: allow users to define a list of expected test cases
See merge request lava/lava!3072
commit 84eb1ab3c5ab5346b505ab255e8431bd5d8ac23b
Author: Chase Qi <chase.qi@linaro.org>
Date: Tue Nov 25 16:20:26 2025 +0800
lava_dispatcher: allow users to define a list of expected test cases
The expected test case list can be defined in either the test definition
or the job definition. If both are provided, the job definition takes
precedence.
At the end of each test run, missing expected test cases from the test
run results are marked as fail. Conversely, test cases present in the
results but not in the expected list are logged as warnings.
Signed-off-by: Chase Qi <chase.qi@linaro.org>
commit 826e87b044ab5c7f7cf0793bb8f4680434b3ad14
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Thu Nov 27 10:55:53 2025 +0000
pylint: Enable R1733 Unnecessary dictionary index lookup
This warning gets emitted when `items()` second value can be used
instead of performing a lookup.
commit 534f37f9f0bf8311c8e1ba0b79d431a17e10b35b
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Tue Nov 25 16:14:29 2025 +0000
pylint: Enable C0201 Consider iterating the dictionary directly
Remove `.keys()` calls where they are not necessary.
commit 819cf88364aa7d68fb9a6ace3f678e7df5accc2d
Merge: 62ab1807 96346f38
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 24 16:38:49 2025 +0000
Merge branch 'skip-bl-connection-reset' into 'master'
lava_dispatcher: allow bootloader boot action to skip connection reset
See merge request lava/lava!3028
commit 62ab18079ec255f942b680054170a713eb2cbf98
Merge: 036a611d 30780af1
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 24 16:38:09 2025 +0000
Merge branch 'usbg-ms-of-cmd' into 'master'
lava_dispatcher: allow job to run usbg_ms_commands.disable command
See merge request lava/lava!3027
commit 036a611dc2835c8427d47e9b878b1665fdada08f
Merge: a962de1d e2158646
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 24 16:33:37 2025 +0000
Merge branch 'requests-codes-use-subscription' into 'master'
Use subscription instead of attributes for request.codes
See merge request lava/lava!3048
commit a962de1d56b93dadd9dfd560003e29ef799512bc
Merge: a66e1112 de9828f0
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 24 16:33:03 2025 +0000
Merge branch 'pylint-enable-C0206' into 'master'
pylint: Enable C0206 Consider iterating with .items()
See merge request lava/lava!3020
commit a66e111211c9441d69803057d81973454b14954a
Merge: 1d0a00bc 58b9e0fb
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 24 16:31:10 2025 +0000
Merge branch 'pylint-enable-W0106' into 'master'
pylint: Enable W0106 Expression is assigned to nothing
See merge request lava/lava!3055
commit 1d0a00bca4e95350348b9d0d3703aa3aeca4e803
Merge: 22489a3a c6a273d1
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 24 16:27:09 2025 +0000
Merge branch 'mt8188-geralt-ciri-sku4-device-type' into 'master'
device-types: add mt8188-geralt-ciri-sku4
See merge request lava/lava!3057
commit 22489a3aee6db28c65d65784cc0be195dfc29da6
Merge: 9fe93083 bfb039bd
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 24 16:26:50 2025 +0000
Merge branch 'pylint-enable-w0130-duplicate-value-in-set' into 'master'
pylint: Enable W0130 duplicate value in set check
See merge request lava/lava!3007
commit 9fe930839e2bd0693fc3e28033cafed5e52509bc
Merge: 7941cef0 7e88de04
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 24 16:26:36 2025 +0000
Merge branch 'logs-firestore-fix-submit-time-attr' into 'master'
Fix LogsFirestore.line_count using incorrect submit_time attr
See merge request lava/lava!3058
commit 7941cef0a76d7621281af67cf406265d1e31fca3
Merge: 122ff375 3dced4a2
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 24 16:25:43 2025 +0000
Merge branch 'lava-results-fix-pylint-R1728' into 'master'
lava_results_app: Fix pylint R1728 Consider using a generator
See merge request lava/lava!3062
commit 122ff3758c8107e09b28094e492d4c658575ecb5
Merge: 81908a17 d9c059ae
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 24 16:25:13 2025 +0000
Merge branch 'lava-results-fix-pylint-W0102' into 'master'
lava_results_app: Fix pylint W0102 Dangerous default value
See merge request lava/lava!3061
commit 81908a17688dc5a8f9dc956a17fa98a3ed8c31b7
Merge: e65aaf20 add1f8e1
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 24 16:24:33 2025 +0000
Merge branch 'lava-results-fix-pylint-C0207' into 'master'
lava_results_app: Fix pylint C0207 Use max split
See merge request lava/lava!3063
commit e65aaf20d7c3d4971fdce90268f56360145b8212
Merge: ea2f6a92 6ee79edd
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 24 16:24:08 2025 +0000
Merge branch 'lava-results-fix-pylint-R5102' into 'master'
lava_results_app: Fix pylint R5102 Use JsonResponse
See merge request lava/lava!3064
commit ea2f6a923188aa1058f70455764873f687124a8c
Merge: 85d93576 4782ebfd
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 24 16:22:30 2025 +0000
Merge branch 'xmlrpc-change-worker-fix-fault-attr' into 'master'
Fix xmlrpc.client.Fault member in lava_scheduler_app/api/workers.py
See merge request lava/lava!3059
commit 6ee79edd037ff031448c43236a743570b2d872a1
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Mon Nov 24 16:00:45 2025 +0000
lava_results_app: Fix pylint R5102 Use JsonResponse
Instead of using `HttpResponse(content_type='application/json')`
use `JsonResponse()`. It will add all required headers automatically.
Use `safe=False` because in very old web browsers only dictionaries
were safe to initialize from JSON. All modern browsers are safe.
commit add1f8e1f88acaff64f1169ac8d2f4ddd9a2629a
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Mon Nov 24 14:57:08 2025 +0000
lava_results_app: Fix pylint C0207 Use max split
commit 3dced4a267913c7d8d64522354fe021103e71b0b
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Mon Nov 24 14:02:06 2025 +0000
lava_results_app: Fix pylint R1728 Consider using a generator
The `sum()` can use iterables directly. Give it the `.values()`
generator to sum the values of all dictionary entries.
commit d9c059aedec1e4e524b59f17ec62049b0bcbac4e
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Mon Nov 24 13:56:21 2025 +0000
lava_results_app: Fix pylint W0102 Dangerous default value
Using list as default argument is dangerous because any changes
to it will persist between calls.
Use immutable tuple. In these functions this argument is only
used to be unpacked to `QuetySet.order_by()` functions for which
unpacking tuple is the same as unpacking list.
commit 4782ebfd803d7e680ed7cc615f93d1ff9f14ae92
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Mon Nov 24 12:56:59 2025 +0000
Fix xmlrpc.client.Fault member in lava_scheduler_app/api/workers.py
The correct member is `xmlrpc.client.Fault` not `xmlrpc.client.fault`.
https://docs.python.org/3/library/xmlrpc.client.html#xmlrpc.client.Fault
commit 7e88de0440239851b4a7e1b5ad06bac41a6dd7a5
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Fri Nov 21 16:20:12 2025 +0000
Fix LogsFirestore.line_count using incorrect submit_time attr
The `job` objects have `submit_time` not `LogsFirestore` itself.
Found using pylint.
commit bfb039bdef7944113ea0eb4a55bdff38a375aa86
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Tue Oct 21 11:12:59 2025 +0100
pylint: Enable W0130 duplicate value in set check
commit c6a273d1f2987343aa6c769f4e4ecfc24635b882
Author: Laura Nao <laura.nao@collabora.com>
Date: Tue Oct 21 14:57:14 2025 +0100
device-types: add mt8188-geralt-ciri-sku4
Add device types for the Lenovo Chromebook Duet SKU4.
Signed-off-by: Laura Nao <laura.nao@collabora.com>
commit 58b9e0fb742338d2962c1f84dd63fb1c488272ce
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Thu Nov 20 14:59:33 2025 +0000
pylint: Enable W0106 Expression is assigned to nothing
All of the issues were in tests where `assert` statements were
missing.
commit de9828f0893fa7a5a622e401f1203d62f66499db
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Fri Oct 24 15:55:02 2025 +0100
pylint: Enable C0206 Consider iterating with .items()
Instead of iterating through keys and making value lookups you
can iterate over key and value at the same time with `.items()`.
`split_multinode_yaml` has a complicated dictionary iterations.
Add type hints and better variable names to better understand
what was actually iterated.
commit 85d9357612e8209195689e3613f3513558f791dd
Merge: a956ac1f 7c19daac
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Thu Nov 20 13:39:12 2025 +0000
Merge branch 'fix-cpio-overlay' into 'master'
utils: compression: use fakeroot for cpio operations
See merge request lava/lava!3054
commit a956ac1fe9738fbc8093e03dce6b42231ef643be
Merge: dd6e7952 7882f441
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Thu Nov 20 13:37:56 2025 +0000
Merge branch 'pylint-enable-W0108' into 'master'
pylint: Enable W0108 Lambda may not be necessary
See merge request lava/lava!3032
commit dd6e79520fc7ce5393c7962241ebf74e890827da
Merge: cd9a1a00 481b0f66
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Thu Nov 20 13:36:58 2025 +0000
Merge branch 'pylint-enable-R1718' into 'master'
pylint: Enable R1718 Consider using a set comprehension
See merge request lava/lava!3031
commit cd9a1a00dddda869557da5067f7a7b77be11a41d
Merge: 55ee8d97 bf124a7b
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Thu Nov 20 13:36:43 2025 +0000
Merge branch 'pylint-enable-R1729' into 'master'
pylint: Enable R1729 Use a generator instead
See merge request lava/lava!3030
commit 55ee8d978004861b95a9daa89aaf0a86bb01c361
Merge: bf8c7ab9 c1b7741a
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Thu Nov 20 13:36:05 2025 +0000
Merge branch 'pylint-enable-C0123' into 'master'
pylint: Enable C0123 Use isinstance() rather than type()
See merge request lava/lava!3021
commit 7c19daac343e027d8b322cf81a93c7a84d529710
Author: Anders Roxell <anders.roxell@linaro.org>
Date: Wed Nov 19 07:41:13 2025 +0100
utils: compression: use fakeroot for cpio operations
Wrap cpio extraction and creation commands with fakeroot to allow
handling of device nodes without requiring root privileges.
This fixes cpio overlay application where device nodes (like
/dev/console, /dev/null, etc.) need to be created or extracted. Without
fakeroot, these operations fail with permission errors.
Tested with cpio rootfs with modules overlay: extraction and repackaging
succeeds.
Signed-off-by: Anders Roxell <anders.roxell@linaro.org>
commit c1b7741a3d27b57e90e53497f96e9500aeb1e4f5
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Fri Oct 24 16:05:26 2025 +0100
pylint: Enable C0123 Use isinstance() rather than type()
`isinstance()` works on subclasses but `type() is` does not.
commit bf124a7bc004657ff762faaabc83a838513d39d9
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Tue Oct 28 11:27:15 2025 +0000
pylint: Enable R1729 Use a generator instead
The functions `any` and `all` can accept generators instead of
a list. This has several advantages like generator not having
to allocate memory for entire list. Also using generator the
`any` and `all` can perform early iterator exit. For example,
`any` can stop as soon as it encounters the first object that
evaluates to True and `all` can exit when it encounters first
object to evaluate to False.
commit 481b0f6672330fb5506483a1d2d21cb76074c7cf
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Tue Oct 28 11:42:22 2025 +0000
pylint: Enable R1718 Consider using a set comprehension
Instead of creating a transient list it is faster to use set
comprehension instead.
commit 7882f4417eacd54de41a3daea8259b2b31de2deb
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Tue Oct 28 13:12:43 2025 +0000
pylint: Enable W0108 Lambda may not be necessary
If a lambda takes one argument and calls one function with this
argument it can be replaced with the function it calls.
For example, `lambda p: otg_path_fsm.fullmatch(p)` ->
`otg_path_fsm.fullmatch`.
commit bf8c7ab9ffa0883edaec3234f91edcec90765282
Merge: afb14545 9d87427c
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 17 12:26:39 2025 +0000
Merge branch 'pylint-enable-C2801' into 'master'
pylint: Enable C2801 Unnecessarily calls dunder method
See merge request lava/lava!3019
commit afb14545e7fdd7b53936b18bb0fb9cbfd390fd44
Merge: 30207fee 5a883b45
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 17 12:25:33 2025 +0000
Merge branch 'pylint-enable-R1734' into 'master'
pylint: Enable R1734 Consider using [] instead of list()
See merge request lava/lava!3023
commit 30207fee75a0bff66f5ee050a5aca3ab8d71300f
Merge: 2391a0c9 9ee4f3e0
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 17 12:25:22 2025 +0000
Merge branch 'pylint-enable-R1731' into 'master'
pylint: Enable R1731 Consider using max instead
See merge request lava/lava!3029
commit 2391a0c9ef119353f476a2fa9f854951d8df63a2
Merge: a9aa82d7 74c56dfb
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 17 12:24:16 2025 +0000
Merge branch 'pylint-enable-W0127' into 'master'
pylint: Enable W0127 Assigning the same variable to itself
See merge request lava/lava!3033
commit a9aa82d7ca129e801f9edbfac8b2ee76c2cff3b6
Merge: bf48099d 6b5e685f
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 17 12:23:56 2025 +0000
Merge branch 'pylint-enable-R1728' into 'master'
pylint: Enable R1728 Consider using a generator
See merge request lava/lava!3034
commit bf48099de33c9b567dd5223639abc0dfcf6b593c
Merge: 23409f23 c18cf678
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 17 12:23:28 2025 +0000
Merge branch 'fix-oserror-in-coordinator-status' into 'master'
Fix OSError attribute in lava/coordinator/status.py
See merge request lava/lava!3047
commit 23409f23323a2878179437b82dd0525b84242835
Merge: 5ca60303 4143681d
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 17 12:23:19 2025 +0000
Merge branch 'pylint-enable-C0121' into 'master'
pylint: Enable C0121 Singleton comparison should use `is`
See merge request lava/lava!3035
commit 5ca60303f9e0b4503e894d259e19767819fa6d1a
Merge: 7c5fd9b7 f23c9da9
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 17 12:22:58 2025 +0000
Merge branch 'pylint-enable-R1719' into 'master'
pylint: Enable R1719 The if expression can be replaced with test
See merge request lava/lava!3036
commit 7c5fd9b7b920c1e67a75963c247e2dc4b410d65e
Merge: 0c6b75be 147d86d9
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 17 12:22:20 2025 +0000
Merge branch 'repo-action-stage-add-default-value' into 'master'
Add default value for RepoAction.stage
See merge request lava/lava!3051
commit 0c6b75bea616e55f4580caa90e169510f03cd2fe
Merge: 9bc99bec c8ffdf0f
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 17 12:22:03 2025 +0000
Merge branch 'testdef-merge-isinstance' into 'master'
Merge isinstance calls in lava_dispatcher/actions/deploy/testdef.py
See merge request lava/lava!3052
commit 9bc99bece665e986367a2e15d286c2ae896231c5
Merge: 5212ce6d 7ecfdc6d
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 17 12:21:28 2025 +0000
Merge branch 'labgrid2lava' into 'master'
share: add labgrid to lava configuration converter
See merge request lava/lava!3022
commit c8ffdf0f8bdec57fce410e8ca87620fd73d8e418
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Thu Nov 13 13:49:02 2025 +0000
Merge isinstance calls in lava_dispatcher/actions/deploy/testdef.py
Using De Morgan's laws `(not A) and (not B)` is equivalent to
`not (A or B)`. This means the `not instance(x, A) and not isinstance(x, B)`
is equivalent to `not instance(x (A,B))`. This makes code easier
to understand and take less lines.
https://en.wikipedia.org/wiki/De_Morgan%27s_laws
commit 5212ce6d5a4128c82fd55615b31d110a3d8c5818
Merge: b30f84ec 2c3fa6a7
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 17 11:29:43 2025 +0000
Merge branch 'disconnect-conn' into 'master'
lava_dispatcher: allow docker test shell to disconnect connection
See merge request lava/lava!3045
commit 147d86d94e212430485314e31475575d241de26e
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Thu Nov 13 12:17:33 2025 +0000
Add default value for RepoAction.stage
This attribute is set in the `TestDefinitionAction.populate`
but there was no default value on newly initialized `RepoAction`.
This raised warnings in static analysis tools like pylint.
Simply add 0 as a default value.
commit b30f84ecf7c2b5a3a6277afa2cb37355d857067f
Merge: 37cfb9b1 40d6df3b
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Thu Nov 13 11:53:35 2025 +0000
Merge branch 'lava-schema-fix-template-error-attributes' into 'master'
Fix non-existent TemplateError attribute in share/lava-schema.py
See merge request lava/lava!3050
commit 37cfb9b16428f54b65d085bdada25a97cf29e563
Merge: 07c19256 f3d0cfec
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Thu Nov 13 11:52:19 2025 +0000
Merge branch 'conn-delay' into 'master'
lava_dispatcher: expose character delay to docker test shell
See merge request lava/lava!3046
commit 07c192565138e5ea16353f27d17aca9c55c21f88
Merge: 62003b5c 27779ae1
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Thu Nov 13 11:46:36 2025 +0000
Merge branch 'rzv2l' into 'master'
Add RZV2L device-type
See merge request lava/lava!3049
commit 40d6df3b316b7dc59ad270e64aa560e31eb71e48
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Thu Nov 13 10:58:22 2025 +0000
Fix non-existent TemplateError attribute in share/lava-schema.py
The `TemplateError` is a base exception for all jinja exceptions
and does not have `lineno` attribute. Only `TemplateSyntaxError`
has it.
https://github.com/pallets/jinja/blob/5ef70112a1ff19c05324ff889dd30405b1002044/src/jinja2/exceptions.py#L7
commit e215864695477d1739da064da04677556d549317
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Wed Nov 12 14:00:34 2025 +0000
Use subscription instead of attributes for request.codes
The issue is that the attributes like `requests.codes.OK` are
dynamically generated which results in static analysis tools like
pylint to raise warnings that attribute does not exists.
Instead use subscription `["OK"]` which will make the code pass
the pylint's check.
commit f3d0cfec56d5cafae54d659487cd92c837507f93
Author: Chase Qi <chase.qi@linaro.org>
Date: Tue Nov 11 06:53:57 2025 +0000
lava_dispatcher: expose character delay to docker test shell
Signed-off-by: Chase Qi <chase.qi@linaro.org>
commit 27779ae15980561386c96e0592bbb1cd4fdebf61
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Thu Nov 13 09:11:45 2025 +0100
Add RZV2L device-type
commit c18cf678dc72a6a41e739afdc0df4444bdcdf292
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Wed Nov 12 13:17:36 2025 +0000
Fix OSError attribute in lava/coordinator/status.py
There is no `message` attribute for OSError. Instead use
`strerror` attribute which exists and is documented.
https://docs.python.org/3/library/exceptions.html#OSError.strerror
commit f23c9da923e65569cf65640e6291871edb180b11
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Thu Oct 30 15:00:58 2025 +0000
pylint: Enable R1719 The if expression can be replaced with test
The `==` produces a bool that can be assigned to a variable
instead of `if else` statement that assigns a boolean.
commit 4143681dece03ed2cd079f2bb8e8c098527b8224
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Tue Oct 28 16:43:18 2025 +0000
pylint: Enable C0121 Singleton comparison should use `is`
PEP 8 states that:
> Comparisons to singletons like None should always be done with
> is or is not, never the equality operators.
https://peps.python.org/pep-0008/#programming-recommendations
commit 6b5e685f634515b01ff72741b96d5d4c57ed9c93
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Tue Oct 28 13:44:18 2025 +0000
pylint: Enable R1728 Consider using a generator
Using generator instead of a list comprehension can result in
better performance.
commit 74c56dfbe9eb8901c074ff517099214f19faf89e
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Tue Oct 28 13:32:41 2025 +0000
pylint: Enable W0127 Assigning the same variable to itself
No point in assigning variable to itself.
commit 9ee4f3e0756c33b44cd02e0227f1f599b7eebe83
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Tue Oct 28 10:56:10 2025 +0000
pylint: Enable R1731 Consider using max instead
`max()` returns the largest of the passed arguments which can
be used instead of an if statement.
commit 5a883b455c147f426c8bc1c5f5ffcc15c750b266
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Mon Oct 27 08:55:30 2025 +0000
pylint: Enable R1734 Consider using [] instead of list()
List literal is faster than a function call.
commit 9d87427c2c8a3667f4a9affb4d0c1c41e7e2daf3
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Fri Oct 24 15:24:54 2025 +0100
pylint: Enable C2801 Unnecessarily calls dunder method
This is raised when a `__magick__` method is called when a regular
call is available. Some tests need to call the actual `__magick__`
method to verify mocks.
commit 2c3fa6a7b5432e417277503ca78fd1c9b18c54f5
Author: Chase Qi <chase.qi@linaro.org>
Date: Tue Nov 11 06:46:55 2025 +0000
lava_dispatcher: allow docker test shell to disconnect connection
When only one connection is acceptable, this will allow the action to
disconnect the previous connection established by `ConnectDevice`
action and allow the test definition to use the connection.
Signed-off-by: Chase Qi <chase.qi@linaro.org>
commit 62003b5cf446d29a3149d97cb9551b0d32b715b0
Merge: 3a4dd275 75abe779
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 10 12:57:40 2025 +0000
Merge branch 'pylint-enable-W0101' into 'master'
pylint: Enable W0101 Unreachable code
See merge request lava/lava!3038
commit 3a4dd275ffa3a3b8ef9246e6326ded72573c9f7f
Merge: ea3d3ced 0cacd659
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 10 12:56:46 2025 +0000
Merge branch 'pylint-enable-R1721' into 'master'
pylint: Enable R1721 Unnecessary use of a comprehension
See merge request lava/lava!3039
commit ea3d3ced534e33784a221729745647521dceeada
Merge: d706fbbd 931afc58
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 10 12:55:30 2025 +0000
Merge branch 'pylint-enable-W3101' into 'master'
pylint: Enable W3101 Without timeout argument requests can hang
See merge request lava/lava!3040
commit d706fbbd4e8dd15a9a9458763b257a41bd872647
Merge: f161bdb0 8ff350ae
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 10 12:54:02 2025 +0000
Merge branch 'lava-docker-worker-use-if-main' into 'master'
Use `if __name__ == "__main__"` guard in lava-docker-worker
See merge request lava/lava!3041
commit f161bdb0cc8a9eaeec458790767eb94e7404dfe7
Merge: 55b85502 f54ebe6d
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Nov 10 12:53:46 2025 +0000
Merge branch 'lava-management-devices-no-int-passing' into 'master'
Avoid exit code integer in lava_server/management/commands/devices.py
See merge request lava/lava!3043
commit f54ebe6d2425f282a7c9d591f2d200bd4c3e696f
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Thu Nov 6 13:49:52 2025 +0000
Avoid exit code integer in lava_server/management/commands/devices.py
Passing around exit code in C-ism and not Pythonic.
commit 8ff350aecf74f9ff6857c0967d76b88220e87095
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Tue Nov 4 16:08:16 2025 +0000
Use `if __name__ == "__main__"` guard in lava-docker-worker
This prevents the lava-docker-worker main function from running
when the module is imported rather then executed. Some analytics
tools import modules.
commit 931afc584fdddff6cf880a50fe84ed98cbafa2b7
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Tue Nov 4 13:43:41 2025 +0000
pylint: Enable W3101 Without timeout argument requests can hang
Add `timeout` argument to all requests calls where it was missing.
In `lava_dispatcher/actions/boot/nodebooter.py` use `max_end_time`
to calculate the remaining timeout value.
Everywhere else use a large timeout value that should only happen
if there was an actual connection issue.
commit 0cacd659f1485b9132f8002324ef283487fa3e6f
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Mon Nov 3 16:43:36 2025 +0000
pylint: Enable R1721 Unnecessary use of a comprehension
If no modification or filtering is done in the comprehension
there is no point in doing it instead of `list()`, `set()`...
commit 75abe779fd5deff35b890428cf47f4f626e9ca93
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Mon Nov 3 16:02:01 2025 +0000
pylint: Enable W0101 Unreachable code
Fix `share/release.py` trying to `return` after `raise`.
commit 55b855024afba7192b1521772047a383196f3ba2
Merge: 4a3d9f63 6ae36d75
Author: Chase Qi <chase.qi@linaro.org>
Date: Fri Oct 31 13:03:28 2025 +0100
Merge branch 'testjob_exporter.py' into 'master'
lava_server: commands: testjob_exporter and transition exporter
See merge request lava/lava!2992
commit 6ae36d753c1c12579b3a17663be39d4ad82bf468
Author: Ben Copeland <ben.copeland@linaro.org>
Date: Fri Oct 10 11:47:51 2025 +0100
lava_server: commands: transition_exporter
A simple script that exports the transition states out in json file. The
file then can be used for future processing. This is a useful approach
to get the transition states of DUT's showing if they go from Good ->
Bad, Good -> Good and how often those state changes happen.
Signed-off-by: Ben Copeland <ben.copeland@linaro.org>
Tested-by: Chris Paterson <chris.paterson2@renesas.com>
commit 47ab87943aea30b6d3a24cd779741d0b70028806
Author: Ben Copeland <ben.copeland@linaro.org>
Date: Fri Oct 10 11:39:03 2025 +0100
lava_server: commands: testjob_exporter
This script simply gives a json formatted file of the testjob data. This
comes out in json which then allows for ability to create, graphs,
charts and get other insights.
Signed-off-by: Ben Copeland <ben.copeland@linaro.org>
Tested-by: Chris Paterson <chris.paterson2@renesas.com>
commit 96346f38e49336e64aaad4ea55376404043780a4
Author: Chase Qi <chase.qi@linaro.org>
Date: Tue Oct 28 09:54:43 2025 +0000
lava_dispatcher: allow bootloader boot action to skip connection reset
Signed-off-by: Chase Qi <chase.qi@linaro.org>
commit 30780af193225a25fe5be24a6e726f573cbd599c
Author: Chase Qi <chase.qi@linaro.org>
Date: Tue Oct 28 15:51:19 2025 +0800
lava_dispatcher: allow job to run usbg_ms_commands.disable command
`usbg_ms_commands` is needed for enabling usbg-ms deploy method. This
change adds the disable command to the list of available user commands so
users can turn it off when needed.
Job definition example:
```
- command:
name: usbg_ms_commands_disable
```
Signed-off-by: Chase Qi <chase.qi@linaro.org>
commit 7ecfdc6d1a057995087584102db2f389cf2b7052
Author: Chase Qi <chase.qi@linaro.org>
Date: Mon Oct 27 06:48:05 2025 +0000
share: add labgrid to lava configuration converter
Signed-off-by: Chase Qi <chase.qi@linaro.org>
commit 4a3d9f63eb1970b17e9b1edf91376e282c53bf36
Merge: 86344c9b 4e8752d5
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Oct 27 15:51:51 2025 +0000
Merge branch 'pylint-enable-W4701' into 'master'
pylint: Enable W4701 iterated list is being modified inside for loop body
See merge request lava/lava!3018
commit 86344c9b6ea34eaa96a3718a1fa71f02ebfe8f21
Merge: 5c04eb5c da9955b8
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Oct 27 15:49:02 2025 +0000
Merge branch 'pylint-enable-r1735' into 'master'
pylint: Enable R1735 consider using {} instead of dict()
See merge request lava/lava!3017
commit 5c04eb5c6df67152ddebac510e0e690bbf342f45
Merge: f41ec2b6 506b5ab3
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Oct 27 15:48:22 2025 +0000
Merge branch 'pylint-enable-r1701' into 'master'
pylint: Enable R1701 Consider merging these isinstance calls
See merge request lava/lava!3016
commit f41ec2b6c1b6f4444b86bc1a173d1eb062b93303
Merge: e3f4256b 8dd07200
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Oct 27 15:48:03 2025 +0000
Merge branch 'pylint-enable-c0207' into 'master'
pylint: Enable C0207 use `.split(maxsplit=1)[0]` instead
See merge request lava/lava!3015
commit e3f4256b34f3390c4642e663733868e53a850705
Merge: 06099977 217fa90f
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Oct 27 15:47:00 2025 +0000
Merge branch 'pylint-enable-w0706' into 'master'
pylint: Enable W0706 the except handler raises immediately
See merge request lava/lava!3014
commit 06099977521f46c525f4ec29b02ec1b0fe0eb449
Merge: 2ea11e81 66ecd043
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Oct 27 15:46:26 2025 +0000
Merge branch 'pylint-enable-W0102' into 'master'
pylint: Enable W0102 Dangerous default value as argument
See merge request lava/lava!3024
commit 2ea11e815f6fe4696cc0f5d1d1212dea18c20f7a
Merge: bc4b724d 978c5ad9
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Oct 27 15:44:07 2025 +0000
Merge branch 'pylint-enable-W0715' into 'master'
pylint: Enable W0715 Exception arguments suggest string formatting
See merge request lava/lava!3026
commit bc4b724d8813ce365c7dcb3cd132e5880d3d2242
Merge: 7775f64f 786e75ca
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Oct 27 15:43:46 2025 +0000
Merge branch 'pylint-enable-W0640' into 'master'
pylint: Enable W0640 Cell variable defined in loop
See merge request lava/lava!3025
commit 7775f64f4ba0969976e9fe8d5d12aec41a5f74f8
Merge: b438c81d 50cb9292
Author: Rémi Duraffort <remi.duraffort@linaro.org>
Date: Mon Oct 27 15:43:06 2025 +0000
Merge branch 'shell-cmd-return-check' into 'master'
lava_dispatcher: allow to check returncode of the line sent to shell session
See merge request lava/lava!2957
commit 978c5ad92b878577d32d204845abb275726d72d5
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Mon Oct 27 10:21:53 2025 +0000
pylint: Enable W0715 Exception arguments suggest string formatting
This warning was fixed in 8e7136ae7c0b796ffe198907a3ae7cb332e9b2f0
but pylint configuration was not updated.
commit 786e75ca0fc51bf3f04b758f659e594083bf071f
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Mon Oct 27 09:46:15 2025 +0000
pylint: Enable W0640 Cell variable defined in loop
This warning is raised when a closure captures a loop variable.
The issue is that this loop variable will change while loop
is progressing leading to unexpected behavior.
commit 66ecd0433a40016dca301727d0566cf162c839f2
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Mon Oct 27 09:14:53 2025 +0000
pylint: Enable W0102 Dangerous default value as argument
Using a mutable collection like `[]` or `{}` as a default argument
is dangerous because it will stay modified between function calls
which can have an unexpected behavior.
commit 4e8752d5b7b8eb0539fa741c9662ddf2fd790f65
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Fri Oct 24 15:11:25 2025 +0100
pylint: Enable W4701 iterated list is being modified inside for loop body
This can result in unpredictable behavior:
https://andrewwegner.com/python-gotcha-modify-list-while-iterating.html
commit da9955b857a73a4160d04f2b77ac430046ba9b53
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Fri Oct 24 14:57:35 2025 +0100
pylint: Enable R1735 consider using {} instead of dict()
Dictionaries literals are faster than function calls.
commit 506b5ab3b6b6d9967716b028e8c3e0c4ac891efe
Author: Igor Ponomarev <igor.ponomarev@collabora.com>
Date: Fri Oct 24 14:49:02 2025 +0100
pylint: Enable R1701 Consider merging these isinstance calls
`isinstance` accepts a sequence of classes as an argument which
can be used instead of chaining `isinstance` with `or`.
Among the 1 debian patch available in version 2024.09-1.1 of the package, we noticed the following issues:
![[bug history graph]](https://qa.debian.org/data/bts/graphs/l/lava.png){kind=link}