[riot-notifications] [RIOT-OS/RIOT] murdock: enable esp32-wroom-32 for CI testing (#11449)

Kaspar Schleiser notifications at github.com
Mon Apr 29 15:40:11 CEST 2019

kaspar030 commented on this pull request.

> @@ -22,9 +22,9 @@
-    '\tpid | state    Q | pri',
-    '\t  1 | pending  Q |  15',
-    '\t  2 | running  Q |   7'
+    r'\s+pid | state    Q | pri',
+    r'\s+1 | pending  Q |\s+\d+',

> What couple of tests fail? I would like to take a look why.

Take a look at the CI output: https://ci.riot-os.org/RIOT-OS/RIOT/11449/3c5e7c9b76d0f2da5629c9298ae3234b8f75c2c1/output.html

I only skimmed the outputs. Some are expecting default priorities, some fail for other reasons.

> IMHO, it is a more general problem. If there is a variable that allows to define the number of priorities, I would like to be able to use it. Not using it because some tests fail doesn't seem to be the right solution.

Sure, but rearranging one architecture's configuration a bit (to make it more like all the others) is IMO preferable than making the tests less exact (by accepting *any* priority instead of what it should be in the default configuration.

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190429/bb9ea49f/attachment.html>

More information about the notifications mailing list