This repository was archived by the owner on Aug 5, 2022. It is now read-only.
meta-iotqa: Ignore one alsa boot error #42
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
|
I had not seen this. It smells like a race which are hard to replicate. Is this with both 570x and minnowboard? |
Contributor
Author
|
@mythi I can only see this error one time in recent history with Joule but I think it happens with both. Here is one example: https://ostroproject.org/jenkins/view/All/job/intel-iot-refkit_pull-requests/228/testReport/junit/570x.refkit-image-common.oeqa.runtime.sanity.baseos/BaseOsTest/test_baseos_systemd_boot_error/ |
Contributor
|
I think it would be better to try to find the bug. Let's wait for @juimonen to comment. |
Contributor
Author
Contributor
|
I'd propose from now on (we have bugzilla) we don't ignore errors until a bug is filed. With that the problems don't get forgotten + we get to triage the priority of fixing the problem. The bug ID could be added in the log |
skuusela
force-pushed
the
ignore_alsa_error
branch
from
a63acc3 to
89e795d
Compare
The error: "/usr/sbin/alsactl: state_lock:125: file /var/lib/alsa/asound.state lock error: File exists" happens randomly under 2% of the time so it's really hard to replicate and test if it affects anything. Also there seems to be no fix for it so ignore the error. Signed-off-by: Simo Kuusela <simo.kuusela@intel.com>
Contributor
Author
|
@mythi Submitted a bug to Refkit Bugzilla and added it's id as comment to the ignored error. |
nagineni
pushed a commit
to nagineni/intel-iot-refkit
that referenced
this pull request
Jun 13, 2017
* meta-iot-web e4310f8...1e515ff > Merge pull request intel#42 from nagineni/update_restserver > iot-rest-api-server.bb: Update iot-rest-api-server to latest > Merge pull request intel#41 from TizenTeam:sandbox/pcoval/on/master/latest > iotivity-node: Update to 1.2.1-1 > Merge pull request intel#40 from nagineni/update_node > recipes-web: Update iotivity-node and iot-rest-api-server to latest version > nodejs: Upgrade to latest LTS v6.11.0 > Merge pull request intel#35 from gunnarx/add_arm64_squashed > Support aarch64 targets by passing arm64 to npm > Merge pull request intel#36 from nagineni/update_license > Update license to proper. Signed-off-by: Sudarsana Nagineni <sudarsana.nagineni@intel.com>
nagineni
pushed a commit
to nagineni/intel-iot-refkit
that referenced
this pull request
Jun 13, 2017
* meta-iot-web e4310f8...1e515ff (11): > Merge pull request intel#42 from nagineni/update_restserver > iot-rest-api-server.bb: Update iot-rest-api-server to latest > Merge pull request intel#41 from TizenTeam:sandbox/pcoval/on/master/latest > iotivity-node: Update to 1.2.1-1 > Merge pull request intel#40 from nagineni/update_node > recipes-web: Update iotivity-node and iot-rest-api-server to latest version > nodejs: Upgrade to latest LTS v6.11.0 > Merge pull request intel#35 from gunnarx/add_arm64_squashed > Support aarch64 targets by passing arm64 to npm > Merge pull request intel#36 from nagineni/update_license > Update license to proper. Signed-off-by: Sudarsana Nagineni <sudarsana.nagineni@intel.com>
mythi
pushed a commit
to mythi/intel-iot-refkit
that referenced
this pull request
Jun 15, 2017
* meta-iot-web e4310f8...1e515ff (11): > Merge pull request intel#42 from nagineni/update_restserver > iot-rest-api-server.bb: Update iot-rest-api-server to latest > Merge pull request intel#41 from TizenTeam:sandbox/pcoval/on/master/latest > iotivity-node: Update to 1.2.1-1 > Merge pull request intel#40 from nagineni/update_node > recipes-web: Update iotivity-node and iot-rest-api-server to latest version > nodejs: Upgrade to latest LTS v6.11.0 > Merge pull request intel#35 from gunnarx/add_arm64_squashed > Support aarch64 targets by passing arm64 to npm > Merge pull request intel#36 from nagineni/update_license > Update license to proper. Signed-off-by: Sudarsana Nagineni <sudarsana.nagineni@intel.com>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The error: "/usr/sbin/alsactl: state_lock:125: file
/var/lib/alsa/asound.state lock error: File exists" happens randomly under 2% of
the time so it's really hard to replicate and test if it affects
anything. Also there seems to be no fix for it so ignore the error.
Signed-off-by: Simo Kuusela simo.kuusela@intel.com