11306 2017-04-06 20:53:21 +0000 eSDK fails to build for openbmc project 2017-12-06 17:27:53 +0000 1 1 6 Yocto Project Subprojects eSDK eSDK 2.2 x86 Multiple RESOLVED FIXED Medium+ normal 2.4 M4 1 henry.bruce bluelightning california.l.sullivan david.j.cobbley dl9pf --- --- Don't know oldest_to_newest 72163 0 3691 henry.bruce 2017-04-06 20:53:21 +0000 Created attachment 3691 do_populate_sdk_ext The openBMC project builds the OS for a server's baseboard management controller (BMC). Trying to build an eSDK for this image fails. Log file "do_populate_sdk_ext" has been uploaded. Steps to re-create ------------------ $ git clone https://github.com/openbmc/openbmc.git $ cd openbmc $ export TEMPLATECONF=meta-openbmc-machines/meta-evb/meta-evb-aspeed/meta-evb-ast2500/conf/ $ source openbmc-env $ bitbake obmc-phosphor-image -c populate_sdk_ext 72168 1 bluelightning 2017-04-07 02:23:32 +0000 So I think the issue - which I've realised we've seen elsewhere but not tracked down until now - is that we are generating an image and then that gets combined into a kernel "fitimage". Unfortunately the signature for the image changes with the date/time (possibly by inclusion of the os-release package), and thus the do_assemble_fitimage task gets re-triggered - but that requires the kernel work tree to be present, which means we have to extract and build it again, and the internal bitbake operation that we run as part of probing dependencies blocks that since otherwise the same failure would just happen later upon installation. At the moment I'm not quite sure what the correct solution to this is, but I at least have an idea of what's causing it. 72433 2 bluelightning 2017-04-13 05:41:37 +0000 I have a workaround for the immediate issue, which is to set the following in the configuration: SDK_TARGETS_append = " linux-obmc" This isn't ideal as the kernel will be rebuilding as part of the eSDK installation process, but until we untangle the dependencies so we can avoid it thinking it needs to rebuild, this is the best available workaround I can think of. Once that's applied there is another issue. In meta-phosphor/conf/distro/openbmc-phosphor.conf there is an INHERIT_remove = " uninative", which is very bad for the eSDK since it relies upon uninative and the installation will fail without it. The quick workaround of course is just to comment out this line. If you do need to do your normal builds without uninative and have it on only for the eSDK, you'd have to change openbmc-phosphor to *not* require conf/distro/poky.conf and just duplicate the bits it needs without inheriting uninative or requiring yocto-uninative.inc so that you don't have to use the INHERIT_remove. (Typically setting up your distro config as independent from poky.conf is what I would recommend anyway, regardless of whether you use uninative or not - then your distro config will only have the things you want in it and not what we deem appropriate to have in the poky reference distro at some point in future; ultimately it comes down to what you're trying to achieve I suppose.) There's one other possible issue that I haven't verified, but I think if you have any recipes included that set SRCREV = "${AUTOREV}" then you'd need to set an explicit SRCREV for those. It so happens there is one in the layers, common/recipes-devtools/python/python-tftpy.bb, but I don't see any explicit references to it so maybe it's not being built by default. (This is probably something we should be handling automatically, I've filed bug 11350 to track that.) 77155 3 bluelightning 2017-09-26 22:51:45 +0000 We have recently done some work in this area under bug 12102. I just backported the patches that resolved that to openbmc and re-tested and they appear to have fixed the issue there also. Here are the backported patches: https://github.com/bluelightning/openbmc/tree/paule/esdk-initramfs-fix These are in poky/OE-Core master now for the upcoming 2.4 release and I have also sent backports for pyro, so I'm now marking this bug resolved. 77156 4 bluelightning 2017-09-26 22:52:57 +0000 (Just one thing to keep in mind - you still need to drop the INHERIT_remove = " uninative" as noted above). 3691 2017-04-06 20:53:21 +0000 2017-04-06 20:53:21 +0000 do_populate_sdk_ext log.do_populate_sdk_ext application/octet-stream 3732 henry.bruce REVCVUc6IEV4ZWN1dGluZyBweXRob24gZnVuY3Rpb24gc3N0YXRlX3Rhc2tfcHJlZnVuYwpERUJV RzogUHl0aG9uIGZ1bmN0aW9uIHNzdGF0ZV90YXNrX3ByZWZ1bmMgZmluaXNoZWQKREVCVUc6IEV4 ZWN1dGluZyBweXRob24gZnVuY3Rpb24gZG9fcG9wdWxhdGVfc2RrX2V4dApOT1RFOiBJbnN0YWxs aW5nIFRBUkdFVCBwYWNrYWdlcwpOT1RFOiBjb25maWd1cmluZyBSUE0gcGxhdGZvcm0gc2V0dGlu Z3MKTk9URTogY29uZmlndXJpbmcgUlBNIHN5c3RlbSBwcm92aWRlcwpOT1RFOiBjb25maWd1cmlu ZyBSUE0gREIgc2V0dGluZ3MKTk9URTogY29uZmlndXJpbmcgU21hcnQgc2V0dGluZ3MKTk9URTog QWRkaW5nIFNtYXJ0IGNoYW5uZWwgZXZiX2FzdDI1MDAgKDUwKQpOT1RFOiBBZGRpbmcgU21hcnQg Y2hhbm5lbCBhcm12NiAoNDUpCk5PVEU6IEFkZGluZyBTbWFydCBjaGFubmVsIGFsbCAoNDApCk5P VEU6IGFkZGluZyBTbWFydCBSUE0gREIgY2hhbm5lbApOT1RFOiBjb25maWd1cmluZyBSUE0gY3Jv c3MtaW5zdGFsbCBzY3JpcHRsZXRfd3JhcHBlcgpOT1RFOiBFeGVjdXRpbmcgJy9tZWRpYS9tZDEy NnAxL3lvY3RvL2hicnVjZS9tb3J0eS9vcGVuYm1jL2J1aWxkL3RtcC9zeXNyb290cy94ODZfNjQt bGludXgvdXNyL2Jpbi9jcmVhdGVyZXBvIC0tZGJwYXRoIC9tZWRpYS9tZDEyNnAxL3lvY3RvL2hi cnVjZS9tb3J0eS9vcGVuYm1jL2J1aWxkL3RtcC93b3JrL2V2Yl9hc3QyNTAwLW9wZW5ibWMtbGlu dXgtZ251ZWFiaS9vYm1jLXBob3NwaG9yLWltYWdlLzEuMC1yMC9ycG1kYi9ldmJfYXN0MjUwMCAt LXVwZGF0ZSAtcSAvbWVkaWEvbWQxMjZwMS95b2N0by9oYnJ1Y2UvbW9ydHkvb3BlbmJtYy9idWls ZC90bXAvd29yay9ldmJfYXN0MjUwMC1vcGVuYm1jLWxpbnV4LWdudWVhYmkvb2JtYy1waG9zcGhv ci1pbWFnZS8xLjAtcjAvcnBtcy9ldmJfYXN0MjUwMCcgLi4uCk5PVEU6IEV4ZWN1dGluZyAnL21l ZGlhL21kMTI2cDEveW9jdG8vaGJydWNlL21vcnR5L29wZW5ibWMvYnVpbGQvdG1wL3N5c3Jvb3Rz L3g4Nl82NC1saW51eC91c3IvYmluL2NyZWF0ZXJlcG8gLS1kYnBhdGggL21lZGlhL21kMTI2cDEv eW9jdG8vaGJydWNlL21vcnR5L29wZW5ibWMvYnVpbGQvdG1wL3dvcmsvZXZiX2FzdDI1MDAtb3Bl bmJtYy1saW51eC1nbnVlYWJpL29ibWMtcGhvc3Bob3ItaW1hZ2UvMS4wLXIwL3JwbWRiL3g4Nl82 NF9uYXRpdmVzZGsgLS11cGRhdGUgLXEgL21lZGlhL21kMTI2cDEveW9jdG8vaGJydWNlL21vcnR5 L29wZW5ibWMvYnVpbGQvdG1wL3dvcmsvZXZiX2FzdDI1MDAtb3BlbmJtYy1saW51eC1nbnVlYWJp L29ibWMtcGhvc3Bob3ItaW1hZ2UvMS4wLXIwL3JwbXMveDg2XzY0X25hdGl2ZXNkaycgLi4uCk5P VEU6IEV4ZWN1dGluZyAnL21lZGlhL21kMTI2cDEveW9jdG8vaGJydWNlL21vcnR5L29wZW5ibWMv YnVpbGQvdG1wL3N5c3Jvb3RzL3g4Nl82NC1saW51eC91c3IvYmluL2NyZWF0ZXJlcG8gLS1kYnBh dGggL21lZGlhL21kMTI2cDEveW9jdG8vaGJydWNlL21vcnR5L29wZW5ibWMvYnVpbGQvdG1wL3dv cmsvZXZiX2FzdDI1MDAtb3BlbmJtYy1saW51eC1nbnVlYWJpL29ibWMtcGhvc3Bob3ItaW1hZ2Uv MS4wLXIwL3JwbWRiL2FsbCAtLXVwZGF0ZSAtcSAvbWVkaWEvbWQxMjZwMS95b2N0by9oYnJ1Y2Uv bW9ydHkvb3BlbmJtYy9idWlsZC90bXAvd29yay9ldmJfYXN0MjUwMC1vcGVuYm1jLWxpbnV4LWdu dWVhYmkvb2JtYy1waG9zcGhvci1pbWFnZS8xLjAtcjAvcnBtcy9hbGwnIC4uLgpOT1RFOiBFeGVj dXRpbmcgJy9tZWRpYS9tZDEyNnAxL3lvY3RvL2hicnVjZS9tb3J0eS9vcGVuYm1jL2J1aWxkL3Rt cC9zeXNyb290cy94ODZfNjQtbGludXgvdXNyL2Jpbi9jcmVhdGVyZXBvIC0tZGJwYXRoIC9tZWRp YS9tZDEyNnAxL3lvY3RvL2hicnVjZS9tb3J0eS9vcGVuYm1jL2J1aWxkL3RtcC93b3JrL2V2Yl9h c3QyNTAwLW9wZW5ibWMtbGludXgtZ251ZWFiaS9vYm1jLXBob3NwaG9yLWltYWdlLzEuMC1yMC9y cG1kYi9hcm12NiAtLXVwZGF0ZSAtcSAvbWVkaWEvbWQxMjZwMS95b2N0by9oYnJ1Y2UvbW9ydHkv b3BlbmJtYy9idWlsZC90bXAvd29yay9ldmJfYXN0MjUwMC1vcGVuYm1jLWxpbnV4LWdudWVhYmkv b2JtYy1waG9zcGhvci1pbWFnZS8xLjAtcjAvcnBtcy9hcm12NicgLi4uCk5PVEU6IFByZXZpb3Vz IHJlcG8gZmlsZSBtaXNzaW5nOiAvbWVkaWEvbWQxMjZwMS95b2N0by9oYnJ1Y2UvbW9ydHkvb3Bl bmJtYy9idWlsZC90bXAvd29yay9ldmJfYXN0MjUwMC1vcGVuYm1jLWxpbnV4LWdudWVhYmkvb2Jt Yy1waG9zcGhvci1pbWFnZS8xLjAtcjAvcnBtcy9ldmJfYXN0MjUwMC9yZXBvZGF0YS9wcmltYXJ5 LnhtbC5negoKTk9URTogUHJldmlvdXMgcmVwbyBmaWxlIG1pc3Npbmc6IC9tZWRpYS9tZDEyNnAx L3lvY3RvL2hicnVjZS9tb3J0eS9vcGVuYm1jL2J1aWxkL3RtcC93b3JrL2V2Yl9hc3QyNTAwLW9w ZW5ibWMtbGludXgtZ251ZWFiaS9vYm1jLXBob3NwaG9yLWltYWdlLzEuMC1yMC9ycG1zL2FsbC9y ZXBvZGF0YS9wcmltYXJ5LnhtbC5negoKTk9URTogUHJldmlvdXMgcmVwbyBmaWxlIG1pc3Npbmc6 IC9tZWRpYS9tZDEyNnAxL3lvY3RvL2hicnVjZS9tb3J0eS9vcGVuYm1jL2J1aWxkL3RtcC93b3Jr L2V2Yl9hc3QyNTAwLW9wZW5ibWMtbGludXgtZ251ZWFiaS9vYm1jLXBob3NwaG9yLWltYWdlLzEu MC1yMC9ycG1zL2FybXY2L3JlcG9kYXRhL3ByaW1hcnkueG1sLmd6CgpOT1RFOiBUaGVyZSBhcmUg bm8gcGFja2FnZXMgdG8gaW5zdGFsbApOT1RFOiBUaGVyZSBhcmUgbm8gcGFja2FnZXMgdG8gaW5z dGFsbApOT1RFOiBJbnN0YWxsaW5nIGNvbXBsZW1lbnRhcnkgcGFja2FnZXMgLi4uCk5PVEU6IFJ1 bm5pbmcgWycvbWVkaWEvbWQxMjZwMS95b2N0by9oYnJ1Y2UvbW9ydHkvb3BlbmJtYy9zY3JpcHRz L29lLXBrZ2RhdGEtdXRpbCcsICctcCcsICcvbWVkaWEvbWQxMjZwMS95b2N0by9oYnJ1Y2UvbW9y dHkvb3BlbmJtYy9idWlsZC90bXAvc3lzcm9vdHMvZXZiLWFzdDI1MDAvcGtnZGF0YScsICdnbG9i JywgJy9tZWRpYS9tZDEyNnAxL3lvY3RvL2hicnVjZS9tb3J0eS9vcGVuYm1jL2J1aWxkL3RtcC93 b3JrL2V2Yl9hc3QyNTAwLW9wZW5ibWMtbGludXgtZ251ZWFiaS9vYm1jLXBob3NwaG9yLWltYWdl LzEuMC1yMC9pbnN0YWxsZWRfcGtncy50eHQnLCAnKi1kZXYgKi1kYmcnXQpOT1RFOiBUaGVyZSBh cmUgbm8gcGFja2FnZXMgdG8gaW5zdGFsbApOT1RFOiBFeGVjdXRpbmcgcm9vdGZzX3N5c3Jvb3Rf cmVsYXRpdmVsaW5rcyAuLi4KREVCVUc6IEV4ZWN1dGluZyBzaGVsbCBmdW5jdGlvbiByb290ZnNf c3lzcm9vdF9yZWxhdGl2ZWxpbmtzCkRFQlVHOiBTaGVsbCBmdW5jdGlvbiByb290ZnNfc3lzcm9v dF9yZWxhdGl2ZWxpbmtzIGZpbmlzaGVkCk5PVEU6IEV4ZWN1dGluZyB3cml0ZV90YXJnZXRfc2Rr X21hbmlmZXN0IC4uLgpERUJVRzogRXhlY3V0aW5nIHB5dGhvbiBmdW5jdGlvbiB3cml0ZV90YXJn ZXRfc2RrX21hbmlmZXN0CkRFQlVHOiBQeXRob24gZnVuY3Rpb24gd3JpdGVfdGFyZ2V0X3Nka19t YW5pZmVzdCBmaW5pc2hlZApOT1RFOiBJbnN0YWxsaW5nIE5BVElWRVNESyBwYWNrYWdlcwpOT1RF OiBjb25maWd1cmluZyBSUE0gcGxhdGZvcm0gc2V0dGluZ3MKTk9URTogY29uZmlndXJpbmcgUlBN IHN5c3RlbSBwcm92aWRlcwpOT1RFOiBjb25maWd1cmluZyBSUE0gREIgc2V0dGluZ3MKTk9URTog Y29uZmlndXJpbmcgU21hcnQgc2V0dGluZ3MK