Skip to content
This repository was archived by the owner on Aug 5, 2022. It is now read-only.

Conversation

@jairglez
Copy link

@jairglez jairglez commented May 5, 2017

This PR cleans up and includes single-node Bluetooth test cases in the refkit-image-common manifest (this is tracked on Yocto bug #11255). It also adds bluetoothctl back to development images as some of the enabled test cases make use of this tool.

This PR replaces #116


REFKIT_IMAGE_COMPUTERVISION_EXTRA_FEATURES_append = " computervision-test"
REFKIT_IMAGE_FEATURES_COMMON_append = " common-test"
BAD_RECOMMENDATIONS_remove = "bluez5-client"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

both are not needed. either this or the packagegroup change

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When I only add the packagegroup entry, the build fails with a message indicating there is no provider for bluez5-client. I had to add the BAD_RECOMMENDATIONS_remove entry for the build to work. I had left the packagegroup entry to indicate the dependency for bluetoothctl in common-test, but I´ll test again without it.

Copy link
Contributor

@mythi mythi May 5, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Something is not right because according to BAD_RECOMMENDATIONS doc it must work:

It is important to realize that if you choose to not install packages using this variable and some other packages are dependent on them (i.e. listed in a recipe's RDEPENDS variable), the OpenEmbedded build system ignores your request and will install the packages to avoid dependency errors.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I rebased the commit and amended it to only include the BAD_RECOMMENDATIONS_remove entry.

@mythi
Copy link
Contributor

mythi commented May 5, 2017

@jairglez thanks. did the tests work locally for you? They are failing now.

@jairglez jairglez changed the title Jairglez/bluetooth Clean up and enablement of single-node Bluetooth test cases May 5, 2017
@jairglez
Copy link
Author

jairglez commented May 5, 2017

@mythi The tests worked for a 570x, which it's the device I have available, I'm waiting for a Bluetooth card to be able to test over Turbot. For qemu, the tests definitely need to be masked out thought.

@jairglez jairglez force-pushed the jairglez/bluetooth branch from c9a5615 to 8272c46 Compare May 25, 2017 22:32
@jairglez jairglez changed the title Clean up and enablement of single-node Bluetooth test cases meta-iotqa: Clean up and enablement of single-node Bluetooth test cases May 25, 2017
@refkit-ci
Copy link

Can one of the admins verify this patch?

@mythi
Copy link
Contributor

mythi commented Jun 1, 2017

@jairglez are you looking to finish this?

@jairglez
Copy link
Author

@mythi I'm still looking to finish it, the problem is in my tests with an USB Bluetooth adapter, the Minnowboard tests work correctly, but on the CI, they fail on 'hciconfig hci0 reset', however the logs don't provide any info besides that.
Long ago, I requested Bluetooth modules for the Silverjaw Lures to be able to test it, however it seems the PO had a problem because I have not received them yet. I will contact the provider again to try to confirm the order is still open and remove any blocker. Meanwhile, could you please provide me with some insight about what could be happening? I could probably remove the reset command from the tests, but I would like to know the exact cause of the failure first.

@mythi
Copy link
Contributor

mythi commented Jun 13, 2017

@jairglez I've not seen much debugging happening in this PR. If you don't have the hardware, you can use this PR for debugging. Here's what you need to do: rebase to the latest master, change the test case code to print out 'output' from hciconfig. You can also run other commands on the target to understand what goes wrong.

@jairglez
Copy link
Author

Thanks @mythi, I wasn't aware I could use that option. I'll execute some debugging tests then.

@mythi
Copy link
Contributor

mythi commented Jun 14, 2017

@jairglez that's very common option when something works locally but not in CI setup

@jairglez
Copy link
Author

@mythi, with the latest changes to the refkit-layers what is now the best place to set up BAD_RECOMMENDATIONS_remove = "bluez5-client" for development images? Is it meta-refkit-core/classes/refkit-image.bbclass a good selection?

@mythi
Copy link
Contributor

mythi commented Jun 15, 2017

@jairglez refkit-image.bbclass is OK but BAD_RECOMMENDATIONS_remove now needs to be made conditional to check IMAGE_MODE = 'development' (see examples in that bbclass)

@jairglez jairglez force-pushed the jairglez/bluetooth branch from 8272c46 to 985d9c5 Compare June 20, 2017 02:18
@ipuustin
Copy link
Contributor

You need to rebase this PR due to file conflicts.

@jairglez jairglez force-pushed the jairglez/bluetooth branch 3 times, most recently from 188d8f2 to 7ec900d Compare June 21, 2017 01:15
@jairglez
Copy link
Author

jairglez commented Jun 22, 2017

@mythi, @ipuustin: According to the test results, the Bluetooth module doesn't seem to be working at all on the Minnowboard Turbot. I consider we should file a bug for this case, and in order to merge this patchset and start testing the Joule at least, we should mask these tests out for Minnowboard Turbot while the issue is fixed. @epyeoh, do you agree?

@jairglez
Copy link
Author

jairglez commented Jun 23, 2017

@mythi, @ipuustin, @epyeoh: I finally received the same Wireless pcie adapter that it's used on the CI (Intel Dual Band Wireless-AC 7260 - 7260HMW), and I was able to debug the issue further. Basically, the wireless adapter stops working after the command 'connmanctl disable bluetooth' is issued, while on dmesg, the following message is shown: "[ 421.158820] Bluetooth: hci0: turning off Intel device LED failed (-61)". After that, the hci0 interface stops responding and any attempt to bring the interface up times out until the DUT is rebooted.
Additionally, I found a similar occurrence on Red Hat: https://bugzilla.redhat.com/show_bug.cgi?id=1290011

@jairglez jairglez force-pushed the jairglez/bluetooth branch from 7ec900d to e7f9e30 Compare June 23, 2017 22:07
@jairglez
Copy link
Author

I've created bug https://bugzilla.yoctoproject.org/show_bug.cgi?id=11714 detailing the previous issue. I also rebased the code again and added the new Bluetooth test cases to the mask file for MinnowBoard Turbot.

@mythi
Copy link
Contributor

mythi commented Jun 27, 2017

@jairglez one thing you can easily test/try still if this is a kernel problem. please try setting PREFERRED_PROVIDER_virtual/kernel = "linux-yocto"

@jairglez
Copy link
Author

@mythi I used the configuration you mentioned to create a custom refkit-common-image build, but the same issue occurs on it.

Jair Gonzalez added 4 commits July 5, 2017 17:51
Remove not used tags and update comments and variable names for
added clarity.

Signed-off-by: Jair Gonzalez <jair.de.jesus.gonzalez.plascencia@linux.intel.com>
Signed-off-by: Jair Gonzalez <jair.de.jesus.gonzalez.plascencia@linux.intel.com>
Disable Bluetooth after completing Bluetooth testing in order
to not interfere with other tests.

Signed-off-by: Jair Gonzalez <jair.de.jesus.gonzalez.plascencia@linux.intel.com>
bluetoothctl has been added to the list of bad recommendations to reduce
GPLv3 dependencies. However, there are Bluetooth test cases that make use
of this tool.

This change removes it from the list, only for development images.

Signed-off-by: Jair Gonzalez <jair.de.jesus.gonzalez.plascencia@linux.intel.com>
Jair Gonzalez added 2 commits July 5, 2017 17:51
Bluetooth tests can not be executed on QEMU, so add single node
tests to its mask file.

Signed-off-by: Jair Gonzalez <jair.de.jesus.gonzalez.plascencia@linux.intel.com>
Mask Bluetooth tests on MinnowBoard Turbot as the hci0 interface stops
responding when Bluetooth is disabled on the current CI configuration:

Expansion board: Silverjaw Lure (SKU 1000)
Wireless adapter: Intel Dual Band Wireless-AC 7260 - 7260HMW

YOCTO #11714

Signed-off-by: Jair Gonzalez <jair.de.jesus.gonzalez.plascencia@linux.intel.com>
@jairglez jairglez force-pushed the jairglez/bluetooth branch from e7f9e30 to 345cfa3 Compare July 6, 2017 00:52
@pohly
Copy link
Contributor

pohly commented Jul 10, 2017

@jairglez is this now considered ready for merging? At least the test results are clean.

@jairglez
Copy link
Author

@pohly I would consider it ready for merging. As I previously mentioned, I had to skip testing the MinnowBoard because of bug https://bugzilla.yoctoproject.org/show_bug.cgi?id=11714 affecting the current Bluetooth device used on RefKit's CI setup. The tests can be enabled back once the issue is solved in some way, maybe by using a different device on the CI setup (I tested it successfully with Bluetooth USB adapter TRENDnet TBW-107UB), or by correcting the issue, which I suspect is driver or firmware related.

@intel intel deleted a comment from jairglez Jul 12, 2017
@intel intel deleted a comment from jairglez Jul 12, 2017
@intel intel deleted a comment from jairglez Jul 12, 2017
@intel intel deleted a comment from jairglez Jul 12, 2017
@intel intel deleted a comment from jairglez Jul 12, 2017
@intel intel deleted a comment from jairglez Jul 12, 2017
pohly added a commit to pohly/intel-iot-refkit that referenced this pull request Jul 12, 2017
This merges PR intel#143, PR intel#222, PR intel#235, PR intel#242.

Signed-off-by: Patrick Ohly <patrick.ohly@intel.com>
@pohly pohly mentioned this pull request Jul 12, 2017
@pohly pohly merged commit 345cfa3 into intel:master Jul 12, 2017
pohly added a commit that referenced this pull request Jul 12, 2017
@jairglez jairglez deleted the jairglez/bluetooth branch July 18, 2017 22:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants