)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"dc586e79682bf7d57d91bfaaabd998b5de4d65c9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"2946cc83_7178f506","updated":"2024-05-16 11:26:54.000000000","message":"Failing with: \n\n    subprocess.TimeoutExpired: Command \u0027[\u0027ipmitool\u0027, \u0027-I\u0027, \u0027lanplus\u0027, \u0027-H\u0027, \u00271.2.3.4\u0027, \u0027-L\u0027, \u0027ADMINISTRATOR\u0027, \u0027-U\u0027, \u0027test-user\u0027, \u0027-R\u0027, \u00277\u0027, \u0027-N\u0027, \u00275\u0027, \u0027-f\u0027, \u0027/tmp/tmpgxiwnx1x\u0027, \u0027power\u0027, \u0027status\u0027]\u0027 timed out after 60 seconds\n\nDon\u0027t know if we want to use fake-hardware to begin with? Or we could rig up a emulated BMC. What do you reckon?","commit_id":"634c6cf076192c95e20eff3f39e8480969820f57"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"e7d10a67b79f7fad276e9ec0c289a957a3d191a3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e574cd58_13c7aaa7","updated":"2024-05-16 09:52:21.000000000","message":"Thanks for fixing my sloppy code guys :)","commit_id":"634c6cf076192c95e20eff3f39e8480969820f57"},{"author":{"_account_id":36238,"name":"Hollie Hutchinson","display_name":"Hollie Hutchinson","email":"hollie@stackhpc.com","username":"assumptions"},"change_message_id":"81f9d9cd773f7e7105afb57d89f8e7f046b8e993","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a79c583a_e3e7735f","in_reply_to":"2946cc83_7178f506","updated":"2024-05-16 11:44:19.000000000","message":"Perhaps we can use the fake-hardware Ironic driver for the time being.\n\nMaybe this lends itself to virtual baremetal CI testing with Tenks in the future though?","commit_id":"634c6cf076192c95e20eff3f39e8480969820f57"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"a15821c7467dcfd9fd31184d3eaf68220a1387df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a7f80506_7b76b95c","in_reply_to":"439874e4_7b7dc523","updated":"2024-05-20 15:32:35.000000000","message":"Done","commit_id":"634c6cf076192c95e20eff3f39e8480969820f57"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"2be7b7fefcb31b0d8d9ae918ee70f71af07488fe","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"439874e4_7b7dc523","in_reply_to":"a79c583a_e3e7735f","updated":"2024-05-16 12:37:55.000000000","message":"fake-hardware sounds good to me. We already to virtual baremetal testing using Tenks. We could use this to register the Tenks nodes, but that seemed like a bigger job.","commit_id":"634c6cf076192c95e20eff3f39e8480969820f57"},{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"57d0da1a299e27f82be97d0ec995891f9cb4f0fb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c55e83f0_3bf7e07e","in_reply_to":"e574cd58_13c7aaa7","updated":"2024-05-16 09:53:58.000000000","message":"I was hoping to see the CI pass before +2 Jake\u0027s change - nice addition, thanks :)","commit_id":"634c6cf076192c95e20eff3f39e8480969820f57"},{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"f710022ca9f7ccb74f539ae58e8f72d38032108b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"501b0756_bc3c72a7","updated":"2024-05-30 10:34:15.000000000","message":"Shipping as this is just a CI job and it would be good to have this in 2024.1.","commit_id":"6a83e1d75b9b3b1b83d6573ab8cd73c5e660606c"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"0fbb6819435f23dc527678e32b7400a6e3226341","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"e8f09dfd_1a734abd","updated":"2024-05-20 15:32:47.000000000","message":"Should be good to go","commit_id":"6a83e1d75b9b3b1b83d6573ab8cd73c5e660606c"}],"playbooks/kayobe-overcloud-base/baremetal-compute.j2":[{"author":{"_account_id":36238,"name":"Hollie Hutchinson","display_name":"Hollie Hutchinson","email":"hollie@stackhpc.com","username":"assumptions"},"change_message_id":"42d76615aa80c8afe9eebd1b1b9dcb08369f5461","unresolved":true,"context_lines":[{"line_number":6,"context_line":"  ipmi_password: test-pass"},{"line_number":7,"context_line":"ironic_properties:"},{"line_number":8,"context_line":"  capabilities: \"{{ ironic_capabilities }}\""},{"line_number":9,"context_line":"ironic_resource_class: \"example_resouce_class\""},{"line_number":10,"context_line":"ironic_capabilities: \"boot_option:local,boot_mode:uefi\""}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"a16893de_14632bd1","line":9,"updated":"2024-05-15 16:30:43.000000000","message":"nit: \u0027resource\u0027 not \u0027resouce\u0027","commit_id":"20eeb15551bb17f80b540c41fc15dccccd541f03"},{"author":{"_account_id":36238,"name":"Hollie Hutchinson","display_name":"Hollie Hutchinson","email":"hollie@stackhpc.com","username":"assumptions"},"change_message_id":"81f9d9cd773f7e7105afb57d89f8e7f046b8e993","unresolved":false,"context_lines":[{"line_number":6,"context_line":"  ipmi_password: test-pass"},{"line_number":7,"context_line":"ironic_properties:"},{"line_number":8,"context_line":"  capabilities: \"{{ ironic_capabilities }}\""},{"line_number":9,"context_line":"ironic_resource_class: \"example_resouce_class\""},{"line_number":10,"context_line":"ironic_capabilities: \"boot_option:local,boot_mode:uefi\""}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"62043e54_72839ee9","line":9,"in_reply_to":"a16893de_14632bd1","updated":"2024-05-16 11:44:19.000000000","message":"Done","commit_id":"20eeb15551bb17f80b540c41fc15dccccd541f03"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"0019d1e77e3480ae54d3c3ff1f27d964b5f710fe","unresolved":true,"context_lines":[{"line_number":9,"context_line":"ironic_properties:"},{"line_number":10,"context_line":"  capabilities: \"{{ ironic_capabilities }}\""},{"line_number":11,"context_line":"ironic_resource_class: \"example_resource_class\""},{"line_number":12,"context_line":"ironic_capabilities: \"boot_option:local,boot_mode:uefi\""},{"line_number":13,"context_line":"{% endraw %}"}],"source_content_type":"text/x-jinja2","patch_set":7,"id":"2cef827b_9fbfd1af","line":12,"updated":"2024-05-17 10:22:14.000000000","message":"I think we will need to set lots of the interfaces here, because the Kayobe config is really annoying and always wants to set a default interface, which I think is ipmi in this case. So to use the second driver type you need to specify all the interfaces to be the correct choices here, likely different from the default in the config:\n\n$ openstack baremetal node create | grep _interface\nusage: openstack baremetal node create [-h] [-f {json,shell,table,value,yaml}]\n                                       [-c COLUMN] [--noindent]\n                                       [--prefix PREFIX]\n                                       [--max-width \u003cinteger\u003e] [--fit-width]\n                                       [--print-empty]\n                                       [--chassis-uuid \u003cchassis\u003e] --driver\n                                       \u003cdriver\u003e [--driver-info \u003ckey\u003dvalue\u003e]\n                                       [--property \u003ckey\u003dvalue\u003e]\n                                       [--extra \u003ckey\u003dvalue\u003e] [--uuid \u003cuuid\u003e]\n                                       [--name \u003cname\u003e]\n                                       [--bios-interface \u003cbios_interface\u003e]\n                                       [--boot-interface \u003cboot_interface\u003e]\n                                       [--console-interface \u003cconsole_interface\u003e]\n                                       [--deploy-interface \u003cdeploy_interface\u003e]\n                                       [--inspect-interface \u003cinspect_interface\u003e]\n                                       [--management-interface \u003cmanagement_interface\u003e]\n                                       [--network-data \u003cnetwork data\u003e]\n                                       [--network-interface \u003cnetwork_interface\u003e]\n                                       [--power-interface \u003cpower_interface\u003e]\n                                       [--raid-interface \u003craid_interface\u003e]\n                                       [--rescue-interface \u003crescue_interface\u003e]\n                                       [--storage-interface \u003cstorage_interface\u003e]\n                                       [--vendor-interface \u003cvendor_interface\u003e]\n                                       [--resource-class \u003cresource_class\u003e]\n                                       [--conductor-group \u003cconductor_group\u003e]\n                                       [--automated-clean | --no-automated-clean]\n                                       [--owner \u003cowner\u003e] [--lessee \u003clessee\u003e]\n                                       [--description \u003cdescription\u003e]\n                                       [--shard \u003cshard\u003e]\n                                       [--parent-node \u003cparent_node\u003e]\n                                       [--firmware-interface \u003cfirmware_interface\u003e]\nopenstack baremetal node create: error: the following arguments are required: --driver\n\nZuul says this is the current config, there lots of the defaults will not work with the fake driver, so they all need overriding I suspect:\n\nenabled_hardware_types \u003d ipmi,fake-hardware\nenabled_boot_interfaces \u003d ipxe,pxe,fake\ndefault_boot_interface \u003d ipxe\n\nenabled_console_interfaces \u003d ipmitool-socat,no-console\ndefault_console_interface \u003d ipmitool-socat\n\nenabled_deploy_interfaces \u003d direct,fake\ndefault_deploy_interface \u003d direct\n\nenabled_inspect_interfaces \u003d inspector,no-inspect\ndefault_inspect_interface \u003d inspector\n\nenabled_management_interfaces \u003d ipmitool,fake\ndefault_management_interface \u003d ipmitool\n\nenabled_network_interfaces \u003d noop,flat,neutron\ndefault_network_interface \u003d neutron\ndefault_power_interface \u003d ipmitool","commit_id":"334611607ce1337dbe7ae963ebdf015814963bc7"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"8125d0f17173ed7e2bd2f2d0f684cc8573116371","unresolved":false,"context_lines":[{"line_number":9,"context_line":"ironic_properties:"},{"line_number":10,"context_line":"  capabilities: \"{{ ironic_capabilities }}\""},{"line_number":11,"context_line":"ironic_resource_class: \"example_resource_class\""},{"line_number":12,"context_line":"ironic_capabilities: \"boot_option:local,boot_mode:uefi\""},{"line_number":13,"context_line":"{% endraw %}"}],"source_content_type":"text/x-jinja2","patch_set":7,"id":"62523a5c_0a393245","line":12,"in_reply_to":"2cef827b_9fbfd1af","updated":"2024-05-20 15:32:23.000000000","message":"Acknowledged","commit_id":"334611607ce1337dbe7ae963ebdf015814963bc7"}]}
