)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"5405cd9f1b82e9551a9bdc4cb669239d9a6e1aa9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e461708f_4f05f251","updated":"2022-01-25 09:56:58.000000000","message":"Hmmmm nope, we\u0027re \"regressing\" from the other patch apparently, at least on my env:\n\n2022-01-25 09:45:52.717 | + echo \u0027--- Clearing BLS entries ---\u0027                                                        \n2022-01-25 09:45:52.717 | --- Clearing BLS entries ---                                                                 \n2022-01-25 09:45:52.717 | + ls /boot/loader/entries/255b194483dd4d37be5e11cabda27344-5.14.0-44.el9.x86_64.conf         \n2022-01-25 09:45:52.719 | /boot/loader/entries/255b194483dd4d37be5e11cabda27344-5.14.0-44.el9.x86_64.conf              \n2022-01-25 09:45:52.719 | + rm /boot/loader/entries/255b194483dd4d37be5e11cabda27344-5.14.0-44.el9.x86_64.conf         \n2022-01-25 09:45:52.720 | + grub2-switch-to-blscfg                              \n2022-01-25 09:45:52.742 | Couldn\u0027t find config file                                                                    \n","commit_id":"06866191b7f0ad6b22b3b4d7956c9fe01240b6da"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"b6001fc86546d4b6e9b09010be926e590f1e79b6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c284cff2_5111be38","updated":"2022-01-26 01:06:40.000000000","message":"I\u0027m going to Depends-On: https://review.opendev.org/c/openstack/diskimage-builder/+/825695 to make it clear that both are required","commit_id":"06866191b7f0ad6b22b3b4d7956c9fe01240b6da"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"d7a9b9ce0b82f3f08def297379fa149eb228be69","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d81ed715_d3ced699","in_reply_to":"00e26d7d_fc3069e6","updated":"2022-01-26 06:25:41.000000000","message":"with the new depends-on, it\u0027s far better - thanks!","commit_id":"06866191b7f0ad6b22b3b4d7956c9fe01240b6da"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"2b18b20e060dbd6840cd886472fd5b11a4d9cbbf","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"00e26d7d_fc3069e6","in_reply_to":"5ed9365b_43b0a601","updated":"2022-01-25 12:41:40.000000000","message":"Tried with removing the 03-reset-bls-entries, and I\u0027m ending up with the wrong \"options\" in the /loader/entries/*.conf file (so basically, back to day-1 issue for me).\n\nAfter inspecting the logfile, I don\u0027t think it actually exec the \"bootloader/finalise.d/50-bootloader\" script. At least, I don\u0027t match anything related to the potential \"echo\" nor \"tests\" from that file in IPA and oc-full log files... I\u0027m a bit lost right now.","commit_id":"06866191b7f0ad6b22b3b4d7956c9fe01240b6da"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"7a2b09a3d832802886544e5af9600a29e6b4de58","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5ed9365b_43b0a601","in_reply_to":"aafc2af8_242b4bdc","updated":"2022-01-25 10:20:09.000000000","message":"maybe we should just remove the centos/pre-install.d/03-reset-bls-entries since the different contents should now be managed by the bootloader element?","commit_id":"06866191b7f0ad6b22b3b4d7956c9fe01240b6da"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"d7a9b9ce0b82f3f08def297379fa149eb228be69","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fe1836f2_db74b84c","in_reply_to":"c284cff2_5111be38","updated":"2022-01-26 06:25:41.000000000","message":"+1 !","commit_id":"06866191b7f0ad6b22b3b4d7956c9fe01240b6da"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"4614d6d048264c0d88b3ec5df5fd97313e9bf751","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"aafc2af8_242b4bdc","in_reply_to":"e461708f_4f05f251","updated":"2022-01-25 10:05:11.000000000","message":"(this happens during the IPA build - so I guess it\u0027s the overcloud-image-python3.yaml ?)","commit_id":"06866191b7f0ad6b22b3b4d7956c9fe01240b6da"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"d7a9b9ce0b82f3f08def297379fa149eb228be69","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d2009892_a679fb72","updated":"2022-01-26 06:25:41.000000000","message":"Pushing this on my lab, we\u0027ll see shortly - thanks for the added clarity :)","commit_id":"30f8f0dc05bc5acc329a4b16f2c7af215ce786da"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"97fbd9aefcb44a93ee1a782e8d66e451cd6a16c1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e23a7da1_29c72ae2","updated":"2022-01-26 07:52:54.000000000","message":"So:\n- with the depends-on, I get my IPA + oc-full images\n- but the /boot/loader/entries/*.conf content is wrong, and points to the host LVM info instead of the LABEL\u003d... - this prevents the second boot to happen properly.\n\nI think we\u0027re still missing a piece that properly overrides the \"options\" line in the loader.","commit_id":"30f8f0dc05bc5acc329a4b16f2c7af215ce786da"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"2fd2915f8d7b2e124f84e792559c8842713ffe0f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6392eff0_d442072a","updated":"2022-01-26 06:26:45.000000000","message":"recheck\n\nIssue while connecting to buildlogs.centos.org","commit_id":"30f8f0dc05bc5acc329a4b16f2c7af215ce786da"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"1065f512242f1e81614713650c7e824bdf2b8ded","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3c1b44a5_6423332c","updated":"2022-01-30 21:34:18.000000000","message":"WIP, it looks like block-device-mbr results in an undeployable image","commit_id":"0a81ae6969b0fd5f7723ba7a0224c6a5cedb63ce"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"aeca047f153f5f42b2f0625752321e40076027e6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b618c9fa_4dcd0a4b","updated":"2022-01-28 00:46:39.000000000","message":"depends-on has landed","commit_id":"0a81ae6969b0fd5f7723ba7a0224c6a5cedb63ce"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"daeea8c8ca979d3fdd48abcdabc814494ee187dc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2f133cb8_9293ab05","updated":"2022-02-10 02:00:09.000000000","message":"Actually we have a way forward, so I\u0027m removing the -2 but leaving the WIP until the changes are up","commit_id":"29324699829354575e58ea2bac37165943acd6f1"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"31909a0ae89a0697757ed98c8ae22ad1a8c4baf0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2154fa75_99cff285","updated":"2022-04-20 21:09:24.000000000","message":"It turns out that legacy bios boot for overcloud-full is also broken because the BLS entry has a root disk which is a uuid that doesn\u0027t exist on the host","commit_id":"29324699829354575e58ea2bac37165943acd6f1"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"ab6439629c3ccfcc1367bd07cb5d7dd625b888c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8e28dafa_5d0d2c80","updated":"2022-02-09 22:51:39.000000000","message":"The diskimage-builder change this depends on is unlikely to land so a new approach will be needed. One option is for the grub2 and bootloader elements to share a script which sets the kernel arguments","commit_id":"29324699829354575e58ea2bac37165943acd6f1"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"060fce9e48ed173af8c0d82ff7dc303db90aac44","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"5ed63ee0_2e3d2ad9","updated":"2022-01-31 10:03:11.000000000","message":"There\u0027s a new issue now:\n\u0027message\u0027: \u0027Command execution failed\u0027, \u0027details\u0027: \u0027Installing GRUB2 boot loader to device /dev/sda failed with Unexpected error while running command.\\nCommand: chroot /tmp/tmp3h6q7fnv /bin/sh -c \"grub2-install /dev/sda\"\\nExit code: 1\\nStdout: \\\u0027\\\u0027\\nStderr: \"grub2-install: error: /usr/lib/grub/x86_64-efi/modinfo.sh doesn\\\u0027t exist. Please specify --target or --directory.\\\\n\".\u0027\n\nReported during the node provisioning, in ironic logs.","commit_id":"29324699829354575e58ea2bac37165943acd6f1"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"f094c690e6c6a61719da8ab5771a14fcc2413e08","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"0a45f244_014a67c9","updated":"2022-02-04 07:38:42.000000000","message":"let\u0027s just go if it breaks something we can revert.\n","commit_id":"29324699829354575e58ea2bac37165943acd6f1"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"3da669c68628e9e91572e25da3d056700c7c4fd3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3befae72_44054d76","updated":"2022-02-02 08:23:01.000000000","message":"looks OK but the images built here are not tested in this upstream ci (they are pulling images from current-tripleo)\n\ntrying a sanity check there https://review.rdoproject.org/r/c/testproject/+/36267/49#message-dd2884d4c1fb70fd5c07e883e8eefdf77d194f7a ","commit_id":"29324699829354575e58ea2bac37165943acd6f1"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"8441fc9f621e508d328831fa81c794813542a28c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"7f15ecb4_66ca1e9c","updated":"2022-01-31 07:10:34.000000000","message":"recheck","commit_id":"29324699829354575e58ea2bac37165943acd6f1"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"7296b074671fffdbc7b51a1d8541001f033554d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"bb0ed1b0_58867a5f","updated":"2022-04-20 21:07:23.000000000","message":"recheck","commit_id":"29324699829354575e58ea2bac37165943acd6f1"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"90963e7be7366a63bf1ee60a442f39a4700630e5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"c41682ee_fc5e6c27","updated":"2022-03-20 20:03:14.000000000","message":"recheck","commit_id":"29324699829354575e58ea2bac37165943acd6f1"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"de0fd5c15ea20923507658e41d8cc1037d6294a6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b75aef60_41dc37e0","in_reply_to":"3befae72_44054d76","updated":"2022-02-02 13:06:05.000000000","message":"my bad didn\u0027t have the jobs dependencies it was commented out fixed and running again with https://review.rdoproject.org/r/c/testproject/+/36267/50#message-8e484bcfef450b0f6569356393d42c0ab572cf81","commit_id":"29324699829354575e58ea2bac37165943acd6f1"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"2214a3330ce908557cb3ac8e724958b5621ac0de","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b6c27520_f3aaf107","in_reply_to":"b75aef60_41dc37e0","updated":"2022-02-04 07:37:17.000000000","message":"waiting for a rerun - node provision failed\n\nhttps://review.rdoproject.org/r/c/testproject/+/36267/50#message-cb78c67c2a12a2966796c5ac2cf17c0acd5f2551","commit_id":"29324699829354575e58ea2bac37165943acd6f1"}]}