)]}'
{"specs/kilo/implement-zapping-states.rst":[{"author":{"_account_id":5805,"name":"Chris Krelle","email":"nobodycam@gmail.com","username":"nobodycam"},"change_message_id":"d363b2f00d916f10da38f95f4090346a64940795","unresolved":false,"context_lines":[{"line_number":21,"context_line":"* Operators need some long running work done on nodes before they can be"},{"line_number":22,"context_line":"  successfully provisioned."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"* Things like firmware updates, setting up new RAID levels, or burning in"},{"line_number":25,"context_line":"  nodes take too long to do before a deploy."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"* Many of these tasks will provide useful scheduling hints to Nova. Operators"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a961159_766b19f1","line":24,"updated":"2014-12-11 23:38:39.000000000","message":"Nit: I would never want to do a burn in before a deploy, thou I would defiantly want to do it before giving the system over to the end user.","commit_id":"266ce19754f25bc62feffd77451597b5fa35586c"},{"author":{"_account_id":10380,"name":"Josh Gachnang","email":"josh@pcsforeducation.com","username":"JoshNang"},"change_message_id":"fe21a871bf95c91210c3fb639cb7e457061f6b93","unresolved":false,"context_lines":[{"line_number":21,"context_line":"* Operators need some long running work done on nodes before they can be"},{"line_number":22,"context_line":"  successfully provisioned."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"* Things like firmware updates, setting up new RAID levels, or burning in"},{"line_number":25,"context_line":"  nodes take too long to do before a deploy."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"* Many of these tasks will provide useful scheduling hints to Nova. Operators"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a961159_1e8e522e","line":24,"in_reply_to":"3a961159_766b19f1","updated":"2014-12-22 21:21:58.000000000","message":"Done","commit_id":"266ce19754f25bc62feffd77451597b5fa35586c"},{"author":{"_account_id":5805,"name":"Chris Krelle","email":"nobodycam@gmail.com","username":"nobodycam"},"change_message_id":"d363b2f00d916f10da38f95f4090346a64940795","unresolved":false,"context_lines":[{"line_number":24,"context_line":"* Things like firmware updates, setting up new RAID levels, or burning in"},{"line_number":25,"context_line":"  nodes take too long to do before a deploy."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"* Many of these tasks will provide useful scheduling hints to Nova. Operators"},{"line_number":28,"context_line":"  could use these scheduling hints to create flavors, such as a nova compute"},{"line_number":29,"context_line":"  flavor that requires a node with RAID 1 for extra durability."},{"line_number":30,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a961159_76423969","line":27,"updated":"2014-12-11 23:38:39.000000000","message":"the zapping tasks will provide scheduling hints? can you provide more detail here?","commit_id":"266ce19754f25bc62feffd77451597b5fa35586c"},{"author":{"_account_id":10202,"name":"Wan-yen Hsu","email":"wanyenhsu@gmail.com"},"change_message_id":"1fe12425d7709055c46497d072e2f96fb8fac8e8","unresolved":false,"context_lines":[{"line_number":24,"context_line":"* Things like firmware updates, setting up new RAID levels, or burning in"},{"line_number":25,"context_line":"  nodes take too long to do before a deploy."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"* Many of these tasks will provide useful scheduling hints to Nova. Operators"},{"line_number":28,"context_line":"  could use these scheduling hints to create flavors, such as a nova compute"},{"line_number":29,"context_line":"  flavor that requires a node with RAID 1 for extra durability."},{"line_number":30,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a961159_a724bfc6","line":27,"in_reply_to":"3a961159_76423969","updated":"2014-12-18 01:52:54.000000000","message":"I can think of a couple examples, for instance, select a node with a speciifc bios or BMC firmware version, or select a node with certain level of RAID","commit_id":"266ce19754f25bc62feffd77451597b5fa35586c"},{"author":{"_account_id":10380,"name":"Josh Gachnang","email":"josh@pcsforeducation.com","username":"JoshNang"},"change_message_id":"fe21a871bf95c91210c3fb639cb7e457061f6b93","unresolved":false,"context_lines":[{"line_number":24,"context_line":"* Things like firmware updates, setting up new RAID levels, or burning in"},{"line_number":25,"context_line":"  nodes take too long to do before a deploy."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"* Many of these tasks will provide useful scheduling hints to Nova. Operators"},{"line_number":28,"context_line":"  could use these scheduling hints to create flavors, such as a nova compute"},{"line_number":29,"context_line":"  flavor that requires a node with RAID 1 for extra durability."},{"line_number":30,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a961159_f197f7e9","line":27,"in_reply_to":"3a961159_a724bfc6","updated":"2014-12-22 21:21:58.000000000","message":"^ Those sum it up, those I\u0027ll drop this section from the spec in favor of letting hardware capabilities do it.","commit_id":"266ce19754f25bc62feffd77451597b5fa35586c"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"c957a1f29dbdc6cd5fa13ef453a0112a72f2b9d7","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"* Many of these tasks will provide useful scheduling hints to Nova. Operators"},{"line_number":28,"context_line":"  could use these scheduling hints to create flavors, such as a nova compute"},{"line_number":29,"context_line":"  flavor that requires a node with RAID 1 for extra durability."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"* Operators may want to execute individual clean steps"},{"line_number":32,"context_line":"  on nodes in MANAGED state. This unfortunately means a lot of overlap"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a961159_89705173","line":29,"updated":"2014-12-12 16:02:16.000000000","message":"it\u0027s dangerously close to hardware capabilities, which we decided are out of scope for K...","commit_id":"266ce19754f25bc62feffd77451597b5fa35586c"},{"author":{"_account_id":10380,"name":"Josh Gachnang","email":"josh@pcsforeducation.com","username":"JoshNang"},"change_message_id":"fe21a871bf95c91210c3fb639cb7e457061f6b93","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"* Many of these tasks will provide useful scheduling hints to Nova. Operators"},{"line_number":28,"context_line":"  could use these scheduling hints to create flavors, such as a nova compute"},{"line_number":29,"context_line":"  flavor that requires a node with RAID 1 for extra durability."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"* Operators may want to execute individual clean steps"},{"line_number":32,"context_line":"  on nodes in MANAGED state. This unfortunately means a lot of overlap"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a961159_7104e749","line":29,"in_reply_to":"3a961159_89705173","updated":"2014-12-22 21:21:58.000000000","message":"Sure, I\u0027ll drop that part for now. It\u0027s still fairly useful without capabilities.","commit_id":"266ce19754f25bc62feffd77451597b5fa35586c"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"c957a1f29dbdc6cd5fa13ef453a0112a72f2b9d7","unresolved":false,"context_lines":[{"line_number":41,"context_line":"* Add an API call which will allow a node in MANAGED state to go to a ZAPPING"},{"line_number":42,"context_line":"  state and perform a ZAPPING step."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"* Add zapping steps to drivers, using the @clean_step decorator with a default"},{"line_number":45,"context_line":"  priority of 0. This will ensure the step isn\u0027t run as part of the automated"},{"line_number":46,"context_line":"  cleaning between DELETED and AVAILABLE that happens in CLEANING."},{"line_number":47,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a961159_c9fdf9b2","line":44,"updated":"2014-12-12 16:02:16.000000000","message":"why not @zapping_step? even if just implemented via @clean_step internally.","commit_id":"266ce19754f25bc62feffd77451597b5fa35586c"},{"author":{"_account_id":10380,"name":"Josh Gachnang","email":"josh@pcsforeducation.com","username":"JoshNang"},"change_message_id":"fe21a871bf95c91210c3fb639cb7e457061f6b93","unresolved":false,"context_lines":[{"line_number":41,"context_line":"* Add an API call which will allow a node in MANAGED state to go to a ZAPPING"},{"line_number":42,"context_line":"  state and perform a ZAPPING step."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"* Add zapping steps to drivers, using the @clean_step decorator with a default"},{"line_number":45,"context_line":"  priority of 0. This will ensure the step isn\u0027t run as part of the automated"},{"line_number":46,"context_line":"  cleaning between DELETED and AVAILABLE that happens in CLEANING."},{"line_number":47,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a961159_51460b7c","line":44,"in_reply_to":"3a961159_c9fdf9b2","updated":"2014-12-22 21:21:58.000000000","message":"If we use zapping_step, we\u0027re drawing a line on what can be used for clean_step and what cannot. I cannot think of many things that someone wouldn\u0027t want to run during cleaning. For example, most people will probably use zapping for setting RAID levels. However, some will want it for cleaning, because tenants may have changed the RAID level or otherwise changed RAID settings. Same with firmware, etc. The only one I can think of is burn-in, but frankly, someone could make the argument. \n\nIf we leave them both as clean_step (or some better name?), with default priority of 0 (I\u0027m going to change this to cleaning_priority for clarity), we give the operators the option of running what they want as part of the automated, between every tenant cleaning, and what is run on demand as part of zapping.","commit_id":"266ce19754f25bc62feffd77451597b5fa35586c"},{"author":{"_account_id":5805,"name":"Chris Krelle","email":"nobodycam@gmail.com","username":"nobodycam"},"change_message_id":"d363b2f00d916f10da38f95f4090346a64940795","unresolved":false,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"* Add zapping steps to drivers, using the @clean_step decorator with a default"},{"line_number":45,"context_line":"  priority of 0. This will ensure the step isn\u0027t run as part of the automated"},{"line_number":46,"context_line":"  cleaning between DELETED and AVAILABLE that happens in CLEANING."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"* The list of possible ZAPPING steps will be pulled from the list of functions"},{"line_number":49,"context_line":"  decorated with @clean_step, which is documented in [1]."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a961159_1601f505","line":46,"updated":"2014-12-11 23:38:39.000000000","message":"using the clean_step decorator will an operator be able to have a step in zapping that is not in cleaning, or run them at different priorities in the two states?","commit_id":"266ce19754f25bc62feffd77451597b5fa35586c"},{"author":{"_account_id":10202,"name":"Wan-yen Hsu","email":"wanyenhsu@gmail.com"},"change_message_id":"1fe12425d7709055c46497d072e2f96fb8fac8e8","unresolved":false,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"* Add zapping steps to drivers, using the @clean_step decorator with a default"},{"line_number":45,"context_line":"  priority of 0. This will ensure the step isn\u0027t run as part of the automated"},{"line_number":46,"context_line":"  cleaning between DELETED and AVAILABLE that happens in CLEANING."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"* The list of possible ZAPPING steps will be pulled from the list of functions"},{"line_number":49,"context_line":"  decorated with @clean_step, which is documented in [1]."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a961159_e26155d4","line":46,"in_reply_to":"3a961159_1601f505","updated":"2014-12-18 01:52:54.000000000","message":"Similar question- will it allows some steps to be performed during zapping and some to performed at cleaning if @clean_step is used for both zapping and cleaning steps?","commit_id":"266ce19754f25bc62feffd77451597b5fa35586c"},{"author":{"_account_id":10380,"name":"Josh Gachnang","email":"josh@pcsforeducation.com","username":"JoshNang"},"change_message_id":"fe21a871bf95c91210c3fb639cb7e457061f6b93","unresolved":false,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"* Add zapping steps to drivers, using the @clean_step decorator with a default"},{"line_number":45,"context_line":"  priority of 0. This will ensure the step isn\u0027t run as part of the automated"},{"line_number":46,"context_line":"  cleaning between DELETED and AVAILABLE that happens in CLEANING."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"* The list of possible ZAPPING steps will be pulled from the list of functions"},{"line_number":49,"context_line":"  decorated with @clean_step, which is documented in [1]."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a961159_d4166948","line":46,"in_reply_to":"3a961159_e26155d4","updated":"2014-12-22 21:21:58.000000000","message":"There won\u0027t be any \u0027priority\u0027 sorting in zapping. The operator passes in a list of zap steps, and they\u0027re executed in whatever order they\u0027re passed in. The priority here is just to ensure it isn\u0027t run during cleaning (any step with priority 0 isn\u0027t run). Any step you can run in cleaning you can also run via zapping. Any clean_step with a priority will be run automatically between every tenant. See my above reply to Dmitry for justification.","commit_id":"266ce19754f25bc62feffd77451597b5fa35586c"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"c957a1f29dbdc6cd5fa13ef453a0112a72f2b9d7","unresolved":false,"context_lines":[{"line_number":49,"context_line":"  decorated with @clean_step, which is documented in [1]."},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"* Operators will be able to get a list of possible steps by querying"},{"line_number":52,"context_line":"  /nodes/\u003cuuid\u003e/zapping/zap_steps. This will provide a superset of the"},{"line_number":53,"context_line":"  states listed in /nodes/\u003cuuid\u003e/cleaning/clean_steps, which doesn\u0027t list"},{"line_number":54,"context_line":"  clean_steps with a priority of 0."},{"line_number":55,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a961159_69232556","line":52,"updated":"2014-12-12 16:02:16.000000000","message":"/nodes/\u003cuuid\u003e/zapping/steps to avoid redudancy","commit_id":"266ce19754f25bc62feffd77451597b5fa35586c"},{"author":{"_account_id":10380,"name":"Josh Gachnang","email":"josh@pcsforeducation.com","username":"JoshNang"},"change_message_id":"fe21a871bf95c91210c3fb639cb7e457061f6b93","unresolved":false,"context_lines":[{"line_number":49,"context_line":"  decorated with @clean_step, which is documented in [1]."},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"* Operators will be able to get a list of possible steps by querying"},{"line_number":52,"context_line":"  /nodes/\u003cuuid\u003e/zapping/zap_steps. This will provide a superset of the"},{"line_number":53,"context_line":"  states listed in /nodes/\u003cuuid\u003e/cleaning/clean_steps, which doesn\u0027t list"},{"line_number":54,"context_line":"  clean_steps with a priority of 0."},{"line_number":55,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a961159_142c7198","line":52,"in_reply_to":"3a961159_69232556","updated":"2014-12-22 21:21:58.000000000","message":"Done","commit_id":"266ce19754f25bc62feffd77451597b5fa35586c"},{"author":{"_account_id":5805,"name":"Chris Krelle","email":"nobodycam@gmail.com","username":"nobodycam"},"change_message_id":"d363b2f00d916f10da38f95f4090346a64940795","unresolved":false,"context_lines":[{"line_number":55,"context_line":""},{"line_number":56,"context_line":"* When the conductor attempts to execute a zap step, it will call"},{"line_number":57,"context_line":"  execute_clean_step() on the driver responsible for that zap step."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Alternatives"},{"line_number":60,"context_line":"------------"},{"line_number":61,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a961159_d193f758","line":58,"updated":"2014-12-11 23:38:39.000000000","message":"are you planing on adding a node.zapping_step field like the node.cleaning_step field purposed in the cleaning spec?","commit_id":"266ce19754f25bc62feffd77451597b5fa35586c"},{"author":{"_account_id":10380,"name":"Josh Gachnang","email":"josh@pcsforeducation.com","username":"JoshNang"},"change_message_id":"fe21a871bf95c91210c3fb639cb7e457061f6b93","unresolved":false,"context_lines":[{"line_number":55,"context_line":""},{"line_number":56,"context_line":"* When the conductor attempts to execute a zap step, it will call"},{"line_number":57,"context_line":"  execute_clean_step() on the driver responsible for that zap step."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Alternatives"},{"line_number":60,"context_line":"------------"},{"line_number":61,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a961159_341fad5c","line":58,"in_reply_to":"3a961159_d193f758","updated":"2014-12-22 21:21:58.000000000","message":"They can use the same field. A node is either cleaning or zapping, so we don\u0027t need to differentiate internally. Via the API, they should be represented as current zapping_step and cleaning_step (based on state) when doing a GET on the node.","commit_id":"266ce19754f25bc62feffd77451597b5fa35586c"},{"author":{"_account_id":10202,"name":"Wan-yen Hsu","email":"wanyenhsu@gmail.com"},"change_message_id":"1fe12425d7709055c46497d072e2f96fb8fac8e8","unresolved":false,"context_lines":[{"line_number":62,"context_line":"* We could make zap steps and clean steps mutually exclusive, simplifying"},{"line_number":63,"context_line":"  some of the API and possible confusion, but limiting zapping and requiring"},{"line_number":64,"context_line":"  a second, nearly identical API for executing individual CLEANING states or"},{"line_number":65,"context_line":"  duplicating cleaning steps as zap and clean steps."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"Data model impact"},{"line_number":68,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a961159_c21a9917","line":65,"updated":"2014-12-18 01:52:54.000000000","message":"Is it possible to add a flag to indicate whether the step is for zapping, cleaning or both? Or define the priority value is such a way that with a certain bit(s) on means the step is applicable to cleaning, zapping or both?","commit_id":"266ce19754f25bc62feffd77451597b5fa35586c"},{"author":{"_account_id":10380,"name":"Josh Gachnang","email":"josh@pcsforeducation.com","username":"JoshNang"},"change_message_id":"fe21a871bf95c91210c3fb639cb7e457061f6b93","unresolved":false,"context_lines":[{"line_number":62,"context_line":"* We could make zap steps and clean steps mutually exclusive, simplifying"},{"line_number":63,"context_line":"  some of the API and possible confusion, but limiting zapping and requiring"},{"line_number":64,"context_line":"  a second, nearly identical API for executing individual CLEANING states or"},{"line_number":65,"context_line":"  duplicating cleaning steps as zap and clean steps."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"Data model impact"},{"line_number":68,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a961159_9465c1e2","line":65,"in_reply_to":"3a961159_c21a9917","updated":"2014-12-22 21:21:58.000000000","message":"I don\u0027t think they should, see above.","commit_id":"266ce19754f25bc62feffd77451597b5fa35586c"},{"author":{"_account_id":11278,"name":"Anusha Ramineni","email":"anusha.ramineni@india.nec.com","username":"anusha08"},"change_message_id":"dd92e77c124b9273db49b0907daede04cc2b54d1","unresolved":false,"context_lines":[{"line_number":8,"context_line":"Implement Zapping States"},{"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"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/ironic/+spec/decom-nodes"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Zapping encompasses all long running, destructive tasks an operator may"},{"line_number":14,"context_line":"want to take either between workloads, or before the first workload has been"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a961159_dba791ba","line":11,"updated":"2015-01-07 03:21:17.000000000","message":"blueprint url doesn\u0027t exist. needs to be updated.","commit_id":"a05dc9f11bb9e93e4871b37bd9d5382da3306020"},{"author":{"_account_id":10380,"name":"Josh Gachnang","email":"josh@pcsforeducation.com","username":"JoshNang"},"change_message_id":"c7183209e8329fd3a16aa44fe5fd07cbd2f87d84","unresolved":false,"context_lines":[{"line_number":8,"context_line":"Implement Zapping States"},{"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"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/ironic/+spec/decom-nodes"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Zapping encompasses all long running, destructive tasks an operator may"},{"line_number":14,"context_line":"want to take either between workloads, or before the first workload has been"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a930d6b_5d71d042","line":11,"in_reply_to":"3a961159_dba791ba","updated":"2015-01-26 18:49:51.000000000","message":"Done","commit_id":"a05dc9f11bb9e93e4871b37bd9d5382da3306020"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"c4f3d97bbc735a0404993b2a92830d00bd1c6bea","unresolved":false,"context_lines":[{"line_number":26,"context_line":"  too long to reasonably do at deploy time."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"* Operators may want to execute individual clean steps"},{"line_number":29,"context_line":"  on nodes in MANAGED state. This unfortunately means a lot of overlap"},{"line_number":30,"context_line":"  with CLEANING states. One example is running verification of node.properties"},{"line_number":31,"context_line":"  over a large group of nodes, but not wanting to wait hours for a full"},{"line_number":32,"context_line":"  CLEANING."}],"source_content_type":"text/x-rst","patch_set":6,"id":"1a930d6b_f0de236c","line":29,"updated":"2015-01-26 18:59:51.000000000","message":"s/MANAGED/MANAGEABLE/ here and below","commit_id":"e97a92c89dc8cf2f691f7ac3aca7b32e16a18c0b"},{"author":{"_account_id":10380,"name":"Josh Gachnang","email":"josh@pcsforeducation.com","username":"JoshNang"},"change_message_id":"681c6503c12c9a364854602a68676945bf9cb55b","unresolved":false,"context_lines":[{"line_number":26,"context_line":"  too long to reasonably do at deploy time."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"* Operators may want to execute individual clean steps"},{"line_number":29,"context_line":"  on nodes in MANAGED state. This unfortunately means a lot of overlap"},{"line_number":30,"context_line":"  with CLEANING states. One example is running verification of node.properties"},{"line_number":31,"context_line":"  over a large group of nodes, but not wanting to wait hours for a full"},{"line_number":32,"context_line":"  CLEANING."}],"source_content_type":"text/x-rst","patch_set":6,"id":"1a930d6b_2e74cc34","line":29,"in_reply_to":"1a930d6b_f0de236c","updated":"2015-01-26 19:50:47.000000000","message":"Thanks!","commit_id":"e97a92c89dc8cf2f691f7ac3aca7b32e16a18c0b"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"c4f3d97bbc735a0404993b2a92830d00bd1c6bea","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"* Add zapping steps to drivers, using the @clean_step decorator with a default"},{"line_number":46,"context_line":"  cleaning_priority of 0. This will ensure the step isn\u0027t run as part of the"},{"line_number":47,"context_line":"  automated cleaning between DELETED and AVAILABLE that happens in CLEANING."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"* The list of possible ZAPPING steps will be pulled from the list of functions"},{"line_number":50,"context_line":"  decorated with @clean_step, which is documented in [1]."}],"source_content_type":"text/x-rst","patch_set":6,"id":"1a930d6b_706073a5","line":47,"updated":"2015-01-26 18:59:51.000000000","message":"doesn\u0027t it make sense to also have some prioritization between steps? maybe a separate decorator would be more helpful?","commit_id":"e97a92c89dc8cf2f691f7ac3aca7b32e16a18c0b"},{"author":{"_account_id":10380,"name":"Josh Gachnang","email":"josh@pcsforeducation.com","username":"JoshNang"},"change_message_id":"681c6503c12c9a364854602a68676945bf9cb55b","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"* Add zapping steps to drivers, using the @clean_step decorator with a default"},{"line_number":46,"context_line":"  cleaning_priority of 0. This will ensure the step isn\u0027t run as part of the"},{"line_number":47,"context_line":"  automated cleaning between DELETED and AVAILABLE that happens in CLEANING."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"* The list of possible ZAPPING steps will be pulled from the list of functions"},{"line_number":50,"context_line":"  decorated with @clean_step, which is documented in [1]."}],"source_content_type":"text/x-rst","patch_set":6,"id":"1a930d6b_6e9df4bb","line":47,"in_reply_to":"1a930d6b_706073a5","updated":"2015-01-26 19:50:47.000000000","message":"No, not for zapping, because it\u0027s not automated, it\u0027s operator controlled. The operator passes in a list of steps in the PUT request to move the node to ZAPPING (detailed below). That list will be executed in that order. I\u0027ll clear that up in the spec.","commit_id":"e97a92c89dc8cf2f691f7ac3aca7b32e16a18c0b"},{"author":{"_account_id":10202,"name":"Wan-yen Hsu","email":"wanyenhsu@gmail.com"},"change_message_id":"8f86cf4869871700ee7fe3b96734131901649ff7","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"* When the conductor attempts to execute a zap step, it will call"},{"line_number":58,"context_line":"  execute_clean_step() on the driver responsible for that zap step."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Alternatives"},{"line_number":61,"context_line":"------------"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"1a930d6b_69a32eb2","line":59,"updated":"2015-01-26 20:18:13.000000000","message":"if zapping is to be done in-band by a ramdsk, how and when will the ramdisk boot up?","commit_id":"e97a92c89dc8cf2f691f7ac3aca7b32e16a18c0b"},{"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":"4e86aa1acac47948cd7c9443d47c657e682a9029","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"* When the conductor attempts to execute a zap step, it will call"},{"line_number":58,"context_line":"  execute_clean_step() on the driver responsible for that zap step."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Alternatives"},{"line_number":61,"context_line":"------------"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"fa81d914_7278f08f","line":59,"in_reply_to":"1a930d6b_69a32eb2","updated":"2015-01-28 21:02:26.000000000","message":"I don\u0027t think that needs to be elaborated on in this spec; as the same mechanism that\u0027s used to boot ramdisks for deploy and cleaning will be utilized.","commit_id":"e97a92c89dc8cf2f691f7ac3aca7b32e16a18c0b"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"c4f3d97bbc735a0404993b2a92830d00bd1c6bea","unresolved":false,"context_lines":[{"line_number":119,"context_line":"      \u0027management.configure_hardware_raid\u0027 : {"},{"line_number":120,"context_line":"        \u0027raid_level\u0027: 10"},{"line_number":121,"context_line":"      }"},{"line_number":122,"context_line":"    }]"},{"line_number":123,"context_line":""},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"RPC API impact"}],"source_content_type":"text/x-rst","patch_set":6,"id":"1a930d6b_702eb344","line":122,"updated":"2015-01-26 18:59:51.000000000","message":"should it be:\n\n \u0027zap_steps\u0027: [\n   {\u0027name\u0027:  \u0027management.configure_hardware_raid\u0027,\n     \u0027raid_level\u0027: 10},\n    ...\n\nI\u0027m not sure I understand why you have dict inside a list","commit_id":"e97a92c89dc8cf2f691f7ac3aca7b32e16a18c0b"},{"author":{"_account_id":10380,"name":"Josh Gachnang","email":"josh@pcsforeducation.com","username":"JoshNang"},"change_message_id":"681c6503c12c9a364854602a68676945bf9cb55b","unresolved":false,"context_lines":[{"line_number":119,"context_line":"      \u0027management.configure_hardware_raid\u0027 : {"},{"line_number":120,"context_line":"        \u0027raid_level\u0027: 10"},{"line_number":121,"context_line":"      }"},{"line_number":122,"context_line":"    }]"},{"line_number":123,"context_line":""},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"RPC API impact"}],"source_content_type":"text/x-rst","patch_set":6,"id":"1a930d6b_6ed5f41a","line":122,"in_reply_to":"1a930d6b_702eb344","updated":"2015-01-26 19:50:47.000000000","message":"The list is for ordering, and the second, internal dict was for the list of kwargs. I\u0027ll use your style.","commit_id":"e97a92c89dc8cf2f691f7ac3aca7b32e16a18c0b"}]}