)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4328,"name":"Steven Hardy","email":"steven.hardy@suse.com","username":"shardy"},"change_message_id":"d791ef06a271a1b9ee4d5097321faf4ceced95b7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f5f06e37_23449f48","updated":"2024-06-07 15:43:08.000000000","message":"Just to cross reference, discussed with Julia on IRC and the following patches were mentioned which resolve the config-2 issue (thanks!):\n\nhttps://review.opendev.org/c/openstack/ironic-python-agent-builder/+/920184 \n\nhttps://review.opendev.org/c/openstack/ironic-python-agent-builder/+/915170\n              https://review.opendev.org/q/I9b74ec977fabc0a7f8ed6f113595a3f1624f6ee6","commit_id":"939623294293c8fe8f53a25fe6c91b2667040dad"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ac7fd6dd819ee2600f2d92832c7133eaab33e0bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"177a4a54_ddf7b189","updated":"2024-01-26 15:56:16.000000000","message":"Side note: we need a new spec type \"discussion\" or \"resolution\" (similar to the TC).","commit_id":"939623294293c8fe8f53a25fe6c91b2667040dad"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"bc5a744e0303cdf72bd9f27f5e4694c29e88416b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"38edce87_5d0cc187","in_reply_to":"177a4a54_ddf7b189","updated":"2024-06-12 18:14:54.000000000","message":"++","commit_id":"939623294293c8fe8f53a25fe6c91b2667040dad"}],"specs/approved/fix-vmedia-boot-config.rst":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"bc5a744e0303cdf72bd9f27f5e4694c29e88416b","unresolved":true,"context_lines":[{"line_number":5,"context_line":" http://creativecommons.org/licenses/by/3.0/legalcode"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":8,"context_line":"Fix Virtual Meida Embedded Network Configuration"},{"line_number":9,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://bugs.launchpad.net/ironic/+bug/2032377"}],"source_content_type":"text/x-rst","patch_set":1,"id":"38804f61_b08a751b","line":8,"updated":"2024-06-12 18:14:54.000000000","message":"sp: Media","commit_id":"939623294293c8fe8f53a25fe6c91b2667040dad"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"bc5a744e0303cdf72bd9f27f5e4694c29e88416b","unresolved":true,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"This is not intended as a feature design document, but more as a document to"},{"line_number":17,"context_line":"attempt to figure out what we are going to do. Discussion has led to use a spec"},{"line_number":18,"context_line":"in lue of the ironic whiteboard."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"The overall issue is we have a feature where the API consumer can supply"},{"line_number":21,"context_line":"standardized nework configuration data to the node ``network_data`` field,"}],"source_content_type":"text/x-rst","patch_set":1,"id":"deece1cd_10b483ab","line":18,"updated":"2024-06-12 18:14:54.000000000","message":"sp: lue \u003d\u003e lieu","commit_id":"939623294293c8fe8f53a25fe6c91b2667040dad"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ac7fd6dd819ee2600f2d92832c7133eaab33e0bd","unresolved":true,"context_lines":[{"line_number":30,"context_line":"and `glean`_ would read the old instances data and then \"do things\"."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"On a positive note, out of the box, `cloud-init`_ and `glean`_ just sort of"},{"line_number":33,"context_line":"\"fall over\" in these cases. Worse so they conflict, becuase historically we\u0027ve"},{"line_number":34,"context_line":"not removed cloud-init when glean\u0027s `simple-init`_ element was loaded."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Specifically, `cloud-init`_, which we previously"}],"source_content_type":"text/x-rst","patch_set":1,"id":"8cc280f8_d3f76af2","line":33,"updated":"2024-01-26 15:56:16.000000000","message":"nit: because","commit_id":"939623294293c8fe8f53a25fe6c91b2667040dad"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ac7fd6dd819ee2600f2d92832c7133eaab33e0bd","unresolved":true,"context_lines":[{"line_number":70,"context_line":""},{"line_number":71,"context_line":"  The down side, is this is a generally breaking change pattern and behavior"},{"line_number":72,"context_line":"  wise since it forces the adoption of launch agents to achieve the needful."},{"line_number":73,"context_line":"  This approach also wouldn\u0027t be viable with static, pre-built ISO images"},{"line_number":74,"context_line":"  which leverage the ``deploy_iso`` setting, unless we relied upon some"},{"line_number":75,"context_line":"  sort of static configuration, and quickly we are sort of back at the same"},{"line_number":76,"context_line":"  logic issues of \"config-2\"."}],"source_content_type":"text/x-rst","patch_set":1,"id":"9badfdc1_55e890d3","line":73,"updated":"2024-01-26 15:56:16.000000000","message":"This is not quite relevant: the whole feature does not really work with pre-built ISOs.","commit_id":"939623294293c8fe8f53a25fe6c91b2667040dad"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ac7fd6dd819ee2600f2d92832c7133eaab33e0bd","unresolved":true,"context_lines":[{"line_number":76,"context_line":"  logic issues of \"config-2\"."},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"  Another aspect here is we would need to create our own launcher to handle"},{"line_number":79,"context_line":"  the case of cloud init."},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"* Teach IPA, since it fingerprints the device anyway to load base"},{"line_number":82,"context_line":"  agent configuration from any iso image, to just identify the virtual media"}],"source_content_type":"text/x-rst","patch_set":1,"id":"c4b9a28f_091a2f8a","line":79,"updated":"2024-01-26 15:56:16.000000000","message":"FWIW I suspect the launcher is the simplest part of everything that is discussed here.","commit_id":"939623294293c8fe8f53a25fe6c91b2667040dad"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ac7fd6dd819ee2600f2d92832c7133eaab33e0bd","unresolved":true,"context_lines":[{"line_number":84,"context_line":"  configuration directly. The idea here was that the agent could do the"},{"line_number":85,"context_line":"  needful, and then cause the agent to exit so it restarted with new"},{"line_number":86,"context_line":"  networking configuration in a fresh state. This was dismissed because"},{"line_number":87,"context_line":"  of not wantingto introduce \"the complexity of glean\" into IPA, even"},{"line_number":88,"context_line":"  though the data struture and format was fairly minimal if we were using"},{"line_number":89,"context_line":"  standardized network configuration tools."},{"line_number":90,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"23b60924_a823998d","line":87,"updated":"2024-01-26 15:56:16.000000000","message":"nit: \"wanting to\"","commit_id":"939623294293c8fe8f53a25fe6c91b2667040dad"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ac7fd6dd819ee2600f2d92832c7133eaab33e0bd","unresolved":true,"context_lines":[{"line_number":86,"context_line":"  networking configuration in a fresh state. This was dismissed because"},{"line_number":87,"context_line":"  of not wantingto introduce \"the complexity of glean\" into IPA, even"},{"line_number":88,"context_line":"  though the data struture and format was fairly minimal if we were using"},{"line_number":89,"context_line":"  standardized network configuration tools."},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"  Further discussion has also pointed out that some operators have earlier"},{"line_number":92,"context_line":"  boot elements which assume network connectivity from an early networking"}],"source_content_type":"text/x-rst","patch_set":1,"id":"d5b7d562_b0291fe1","line":89,"updated":"2024-01-26 15:56:16.000000000","message":"I wish they were so standardized... At the base level, we have NM and systemd-network, both popular and different. Then, there is a desire to use nmstate as the configuration language. We discussed that on the last Metal3 meeting (and OCP already uses nmstate, although through its completely different mechanism).\n\nMaybe we can have Glean as a library? :) Maybe it\u0027s already usable as a library?","commit_id":"939623294293c8fe8f53a25fe6c91b2667040dad"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ac7fd6dd819ee2600f2d92832c7133eaab33e0bd","unresolved":true,"context_lines":[{"line_number":92,"context_line":"  boot elements which assume network connectivity from an early networking"},{"line_number":93,"context_line":"  state, and then download IPA or updated hardware managers, or both to"},{"line_number":94,"context_line":"  the running state of the ramdisk which means we would be breaking that"},{"line_number":95,"context_line":"  case should the utilize virtual media."},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"* Another \"Teaching IPA\" option was to utilize the knowledge and understanding"},{"line_number":98,"context_line":"  of the state of the machine to then identify the correct"}],"source_content_type":"text/x-rst","patch_set":1,"id":"04ba2d1a_44b1d7b9","line":95,"updated":"2024-01-26 15:56:16.000000000","message":"nit: \"should they\"?","commit_id":"939623294293c8fe8f53a25fe6c91b2667040dad"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ac7fd6dd819ee2600f2d92832c7133eaab33e0bd","unresolved":true,"context_lines":[{"line_number":107,"context_line":"  which is after IPA is launched, you cannot necessarilly patch IPA"},{"line_number":108,"context_line":"  because now only network is online. We could also have an IPA"},{"line_number":109,"context_line":"  triggered \"two step launch\" which could allow operators to"},{"line_number":110,"context_line":"  compose and scripts to patch their deployments between the two steps."},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"But at the end of the day, there is a compounding issue of CI system"},{"line_number":113,"context_line":"resources. TinyIPA has allowed the Ironic community to continue"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9fcd9c29_14c5135e","line":110,"updated":"2024-01-26 15:56:16.000000000","message":"Missing here is the \"teaching $something\" approach, where $something is not IPA, but is installed and run much earlier. I guess that more or less falls under the \"customer launcher\" bullet point above, but without creating a custom ID (and thus easier to backport or, in case of an IPA-builder element, just use for older branches).","commit_id":"939623294293c8fe8f53a25fe6c91b2667040dad"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"bc5a744e0303cdf72bd9f27f5e4694c29e88416b","unresolved":true,"context_lines":[{"line_number":117,"context_line":"is not one of them, because of TinyCore Linux and lacking support for"},{"line_number":118,"context_line":"`glean`_ and `cloud_init`_. Two possible option paths exist here,"},{"line_number":119,"context_line":"in that we could add support to Glean, or we could move towards"},{"line_number":120,"context_line":"abandoning TinyIPA for a lightweight image. This is only"},{"line_number":121,"context_line":"feasible if we are willing to perform invasive stripping of non-necessary"},{"line_number":122,"context_line":"artifacts like firmware blobs, optional kernel modules, etc."},{"line_number":123,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"975a6679_b48b7f4e","line":120,"updated":"2024-06-12 18:14:54.000000000","message":"Abandon tinyipa++, I think we could get a working gentoo image slimmed down for CI in a couple of days of spiking on it. We should try to find time for that in June.","commit_id":"939623294293c8fe8f53a25fe6c91b2667040dad"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ac7fd6dd819ee2600f2d92832c7133eaab33e0bd","unresolved":true,"context_lines":[{"line_number":119,"context_line":"in that we could add support to Glean, or we could move towards"},{"line_number":120,"context_line":"abandoning TinyIPA for a lightweight image. This is only"},{"line_number":121,"context_line":"feasible if we are willing to perform invasive stripping of non-necessary"},{"line_number":122,"context_line":"artifacts like firmware blobs, optional kernel modules, etc."},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"Ignoring CI testing entirely, assuming it is a solvable problem, it should"},{"line_number":125,"context_line":"be feasible for us to find a path forward, yet we also need to determine"}],"source_content_type":"text/x-rst","patch_set":1,"id":"289f5e32_d40e560d","line":122,"updated":"2024-01-26 15:56:16.000000000","message":"We do that already. Unfortunately, a real OS image is still huge. Even Debian-minimal will surely exceed 200M.","commit_id":"939623294293c8fe8f53a25fe6c91b2667040dad"}]}