)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"ffa76a5b31b6bb234d62d9feb8f6418ca1998203","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-07-20 22:34:50 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add docs for node.resource_class and flavor creation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I4f7259c8c13c9080d649731311f9bb8f4bdf1d99"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bacf61ea_f108fa2a","line":8,"updated":"2016-07-27 14:38:14.000000000","message":"need to add the bug info.","commit_id":"69a57946a604d0ec29bcdae34c559f9e47465b43"}],"doc/source/deploy/install-guide.rst":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"fd2fa9f1d313781c2d7f6c3426832b5505bd0ae4","unresolved":false,"context_lines":[{"line_number":772,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":773,"context_line":""},{"line_number":774,"context_line":"TODO(jroll) docs about node.resource_class and how to make the flavor use"},{"line_number":775,"context_line":"it go here."},{"line_number":776,"context_line":""},{"line_number":777,"context_line":"You\u0027ll need to create a special bare metal flavor in the Compute service."},{"line_number":778,"context_line":"The flavor is mapped to the bare metal node through the hardware specifications."}],"source_content_type":"text/x-rst","patch_set":1,"id":"bacf61ea_6e81cb8f","line":775,"updated":"2016-07-27 14:37:40.000000000","message":"Is this  TODO before this patch lands, or did you want it to land with this TODO? If the latter, I would disagree :)","commit_id":"69a57946a604d0ec29bcdae34c559f9e47465b43"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"4ea5ce71cbda139255c9dd4b840eee1819bd717b","unresolved":false,"context_lines":[{"line_number":811,"context_line":"    driver_info/deploy_kernel\u003d$DEPLOY_VMLINUZ_UUID \\"},{"line_number":812,"context_line":"    driver_info/deploy_ramdisk\u003d$DEPLOY_INITRD_UUID"},{"line_number":813,"context_line":""},{"line_number":814,"context_line":"The 6.1.0 release of ironic includes a field on the node resource called"},{"line_number":815,"context_line":"``resource_class``.  It\u0027s expected that the Newton release of the Compute"},{"line_number":816,"context_line":"service will read and store information about this field, and the Ocata release"},{"line_number":817,"context_line":"of the Compute service will begin using it for scheduling, if available.  This"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9ad45d7e_5ad938f2","line":814,"updated":"2016-08-11 22:28:23.000000000","message":"OH. I mean, it is good that we are refering to actual release versions, but this is the first time we\u0027re doing that in this install guide. And this install guide is meant to go with openstack guides, which i think, are based on the naming releases like Newton, which is how we\u0027ve described things to-date.\n\nDid you really want to be specific? Or use the handwaving, eg \u0027Starting with the Newton release, the Bare Metal service provides a ``resource_class`` field for node resources.\u0027","commit_id":"4c42e88df5c6b3316363d57a4f5f95b47ca0bc19"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"d437295ac2325b8987cf878f800ac8c8b6603f08","unresolved":false,"context_lines":[{"line_number":811,"context_line":"    driver_info/deploy_kernel\u003d$DEPLOY_VMLINUZ_UUID \\"},{"line_number":812,"context_line":"    driver_info/deploy_ramdisk\u003d$DEPLOY_INITRD_UUID"},{"line_number":813,"context_line":""},{"line_number":814,"context_line":"The 6.1.0 release of ironic includes a field on the node resource called"},{"line_number":815,"context_line":"``resource_class``.  It\u0027s expected that the Newton release of the Compute"},{"line_number":816,"context_line":"service will read and store information about this field, and the Ocata release"},{"line_number":817,"context_line":"of the Compute service will begin using it for scheduling, if available.  This"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3ac371cc_0886f212","line":814,"in_reply_to":"9ad45d7e_5ad938f2","updated":"2016-08-16 10:59:45.000000000","message":"That\u0027s a good question - I\u0027m happy to change it for consistency, but we should think more about it overall. :)","commit_id":"4c42e88df5c6b3316363d57a4f5f95b47ca0bc19"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"4ea5ce71cbda139255c9dd4b840eee1819bd717b","unresolved":false,"context_lines":[{"line_number":814,"context_line":"The 6.1.0 release of ironic includes a field on the node resource called"},{"line_number":815,"context_line":"``resource_class``.  It\u0027s expected that the Newton release of the Compute"},{"line_number":816,"context_line":"service will read and store information about this field, and the Ocata release"},{"line_number":817,"context_line":"of the Compute service will begin using it for scheduling, if available.  This"},{"line_number":818,"context_line":"field is available in version 1.21 of the Bare Metal service API."},{"line_number":819,"context_line":""},{"line_number":820,"context_line":"In the future, a Compute flavor will use this field for scheduling, instead of"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9ad45d7e_5d3812d3","line":817,"updated":"2016-08-11 22:28:23.000000000","message":"if what is available, scheduling in compute service or if node.resource_class is set?","commit_id":"4c42e88df5c6b3316363d57a4f5f95b47ca0bc19"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"d437295ac2325b8987cf878f800ac8c8b6603f08","unresolved":false,"context_lines":[{"line_number":814,"context_line":"The 6.1.0 release of ironic includes a field on the node resource called"},{"line_number":815,"context_line":"``resource_class``.  It\u0027s expected that the Newton release of the Compute"},{"line_number":816,"context_line":"service will read and store information about this field, and the Ocata release"},{"line_number":817,"context_line":"of the Compute service will begin using it for scheduling, if available.  This"},{"line_number":818,"context_line":"field is available in version 1.21 of the Bare Metal service API."},{"line_number":819,"context_line":""},{"line_number":820,"context_line":"In the future, a Compute flavor will use this field for scheduling, instead of"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3ac371cc_c8b97acd","line":817,"in_reply_to":"9ad45d7e_5d3812d3","updated":"2016-08-16 10:59:45.000000000","message":"The latter, will update.","commit_id":"4c42e88df5c6b3316363d57a4f5f95b47ca0bc19"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"4ea5ce71cbda139255c9dd4b840eee1819bd717b","unresolved":false,"context_lines":[{"line_number":817,"context_line":"of the Compute service will begin using it for scheduling, if available.  This"},{"line_number":818,"context_line":"field is available in version 1.21 of the Bare Metal service API."},{"line_number":819,"context_line":""},{"line_number":820,"context_line":"In the future, a Compute flavor will use this field for scheduling, instead of"},{"line_number":821,"context_line":"the CPU/RAM/disk properties defined in the flavor above. A flavor will require"},{"line_number":822,"context_line":"*exactly one* of some bare metal resource class. For example, an ironic"},{"line_number":823,"context_line":"deployment might have the following nodes::"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9ad45d7e_dda082f6","line":820,"updated":"2016-08-11 22:28:23.000000000","message":"what future is this? Starting with the Ocata release of the Compute service?","commit_id":"4c42e88df5c6b3316363d57a4f5f95b47ca0bc19"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"d437295ac2325b8987cf878f800ac8c8b6603f08","unresolved":false,"context_lines":[{"line_number":817,"context_line":"of the Compute service will begin using it for scheduling, if available.  This"},{"line_number":818,"context_line":"field is available in version 1.21 of the Bare Metal service API."},{"line_number":819,"context_line":""},{"line_number":820,"context_line":"In the future, a Compute flavor will use this field for scheduling, instead of"},{"line_number":821,"context_line":"the CPU/RAM/disk properties defined in the flavor above. A flavor will require"},{"line_number":822,"context_line":"*exactly one* of some bare metal resource class. For example, an ironic"},{"line_number":823,"context_line":"deployment might have the following nodes::"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3ac371cc_08eb92b1","line":820,"in_reply_to":"9ad45d7e_dda082f6","updated":"2016-08-16 10:59:45.000000000","message":"Well, that\u0027s the goal, but I\u0027d rather not try to predict the future here. :/","commit_id":"4c42e88df5c6b3316363d57a4f5f95b47ca0bc19"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"4ea5ce71cbda139255c9dd4b840eee1819bd717b","unresolved":false,"context_lines":[{"line_number":819,"context_line":""},{"line_number":820,"context_line":"In the future, a Compute flavor will use this field for scheduling, instead of"},{"line_number":821,"context_line":"the CPU/RAM/disk properties defined in the flavor above. A flavor will require"},{"line_number":822,"context_line":"*exactly one* of some bare metal resource class. For example, an ironic"},{"line_number":823,"context_line":"deployment might have the following nodes::"},{"line_number":824,"context_line":""},{"line_number":825,"context_line":"    - node-1:"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9ad45d7e_ddd52243","line":822,"updated":"2016-08-11 22:28:23.000000000","message":"s/an ironic/a Bare Metal service/ ?\n\nActually how about \u0027For example, the following nodes may be enrolled in the Bare Metal service::\u0027","commit_id":"4c42e88df5c6b3316363d57a4f5f95b47ca0bc19"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"d437295ac2325b8987cf878f800ac8c8b6603f08","unresolved":false,"context_lines":[{"line_number":819,"context_line":""},{"line_number":820,"context_line":"In the future, a Compute flavor will use this field for scheduling, instead of"},{"line_number":821,"context_line":"the CPU/RAM/disk properties defined in the flavor above. A flavor will require"},{"line_number":822,"context_line":"*exactly one* of some bare metal resource class. For example, an ironic"},{"line_number":823,"context_line":"deployment might have the following nodes::"},{"line_number":824,"context_line":""},{"line_number":825,"context_line":"    - node-1:"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3ac371cc_88ee62b2","line":822,"in_reply_to":"9ad45d7e_ddd52243","updated":"2016-08-16 10:59:45.000000000","message":"Sure, done","commit_id":"4c42e88df5c6b3316363d57a4f5f95b47ca0bc19"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"4ea5ce71cbda139255c9dd4b840eee1819bd717b","unresolved":false,"context_lines":[{"line_number":851,"context_line":""},{"line_number":852,"context_line":"    ironic --ironic-api-version 1.21 node-update $NODE_UUID \\"},{"line_number":853,"context_line":"    add resource_class\u003d$RESOURCE_CLASS"},{"line_number":854,"context_line":""},{"line_number":855,"context_line":""},{"line_number":856,"context_line":"Setup the drivers for the Bare Metal service"},{"line_number":857,"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"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9ad45d7e_fd062684","line":854,"updated":"2016-08-11 22:28:23.000000000","message":"Oh gosh. I was wondering how we were going to advertise this. I feel like it is premature to be writing this. Or if we do, that it doesn\u0027t belong in the install guide. Maybe in our upgrades section. Or a different page, linked to here. Or a subsection?\n\nWe\u0027re asking people to plan ahead for a non-existing feature in nova. It is probably bad enough having to sit down and decide on flavors and flavor names, but to do so now when it won\u0027t be used for 6+ months seems ... I don\u0027t see what\u0027s wrong with asking folks to do the node-update call in a few month\u0027s down the road, eg in the Ocata release notes. We upgrade ironic before nova, right?","commit_id":"4c42e88df5c6b3316363d57a4f5f95b47ca0bc19"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"d437295ac2325b8987cf878f800ac8c8b6603f08","unresolved":false,"context_lines":[{"line_number":851,"context_line":""},{"line_number":852,"context_line":"    ironic --ironic-api-version 1.21 node-update $NODE_UUID \\"},{"line_number":853,"context_line":"    add resource_class\u003d$RESOURCE_CLASS"},{"line_number":854,"context_line":""},{"line_number":855,"context_line":""},{"line_number":856,"context_line":"Setup the drivers for the Bare Metal service"},{"line_number":857,"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"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3ac371cc_e8c71610","line":854,"in_reply_to":"9ad45d7e_fd062684","updated":"2016-08-16 10:59:45.000000000","message":"Well... a couple things here.\n\n1) This needs to happen for new deployments, so it needs to be here (or linked from here, I guess).\n\n2) We should link to this from release notes, yes.\n\n3) This needs to happen *before* upgrading Nova to Ocata, so I\u0027d like to get it out there as soon as possible. Keep in mind that \"Ocata\" is nebulous for CD folks - what if this code lands in nova on day one of the Ocata cycle?\n\n4) Folks need to define flavors no matter what. This just puts a key/value on the node that ties the node to that flavor. I don\u0027t think the upfront work is a burden. Personally, I\u0027d rather get it out of the way before I forget about it. :)","commit_id":"4c42e88df5c6b3316363d57a4f5f95b47ca0bc19"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"57a4ae61666f89ea01d02719d06ddbea1f87941f","unresolved":false,"context_lines":[{"line_number":810,"context_line":"    ironic node-update $NODE_UUID add \\"},{"line_number":811,"context_line":"    driver_info/deploy_kernel\u003d$DEPLOY_VMLINUZ_UUID \\"},{"line_number":812,"context_line":"    driver_info/deploy_ramdisk\u003d$DEPLOY_INITRD_UUID"},{"line_number":813,"context_line":""},{"line_number":814,"context_line":"The Newton release release of the Bare Metal service includes a field on the"},{"line_number":815,"context_line":"node resource called ``resource_class``. It\u0027s expected that the Newton release"},{"line_number":816,"context_line":"of the Compute service will read and store information about this field, and"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa7ab95a_5a6d64cb","line":813,"updated":"2016-08-31 10:02:20.000000000","message":"could you please add a subheading here? this way it will be easier to refer people to this new section.","commit_id":"33c1ef8d3c7083115fa48a4565ee043ff47f121d"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"b24879f3b6c6885d751f6aa147fdb59fcaf49396","unresolved":false,"context_lines":[{"line_number":810,"context_line":"    ironic node-update $NODE_UUID add \\"},{"line_number":811,"context_line":"    driver_info/deploy_kernel\u003d$DEPLOY_VMLINUZ_UUID \\"},{"line_number":812,"context_line":"    driver_info/deploy_ramdisk\u003d$DEPLOY_INITRD_UUID"},{"line_number":813,"context_line":""},{"line_number":814,"context_line":"The Newton release release of the Bare Metal service includes a field on the"},{"line_number":815,"context_line":"node resource called ``resource_class``. It\u0027s expected that the Newton release"},{"line_number":816,"context_line":"of the Compute service will read and store information about this field, and"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a8ec9b2_05a434b7","line":813,"in_reply_to":"fa7ab95a_5a6d64cb","updated":"2016-09-20 12:45:34.000000000","message":"Done","commit_id":"33c1ef8d3c7083115fa48a4565ee043ff47f121d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"57a4ae61666f89ea01d02719d06ddbea1f87941f","unresolved":false,"context_lines":[{"line_number":846,"context_line":"that the Compute service\u0027s resource tracker has time to read and store this"},{"line_number":847,"context_line":"metadata before the flavors are deployed. It\u0027s recommended to set this field"},{"line_number":848,"context_line":"after deploying ironic 6.1.0 and before deploying the Ocata version of the"},{"line_number":849,"context_line":"Compute service."},{"line_number":850,"context_line":""},{"line_number":851,"context_line":"As usual, this is a simple ``node-update`` call::"},{"line_number":852,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa7ab95a_7a7228af","line":849,"updated":"2016-08-31 10:02:20.000000000","message":"Set it to what? E.g. if I planned to make this change to TripleO, what should I put as a resource class? Flavor name?","commit_id":"33c1ef8d3c7083115fa48a4565ee043ff47f121d"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"b24879f3b6c6885d751f6aa147fdb59fcaf49396","unresolved":false,"context_lines":[{"line_number":846,"context_line":"that the Compute service\u0027s resource tracker has time to read and store this"},{"line_number":847,"context_line":"metadata before the flavors are deployed. It\u0027s recommended to set this field"},{"line_number":848,"context_line":"after deploying ironic 6.1.0 and before deploying the Ocata version of the"},{"line_number":849,"context_line":"Compute service."},{"line_number":850,"context_line":""},{"line_number":851,"context_line":"As usual, this is a simple ``node-update`` call::"},{"line_number":852,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a8ec9b2_0a1b6381","line":849,"in_reply_to":"fa7ab95a_7a7228af","updated":"2016-09-20 12:45:34.000000000","message":"Well, it can be anything, as long as it matches the flavor properties. I think flavor name is a good method for installers like tripleo, but it doesn\u0027t need to be flavor name.\n\nI thought the paragraph above this one laid it out well enough, but maybe not?","commit_id":"33c1ef8d3c7083115fa48a4565ee043ff47f121d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"57a4ae61666f89ea01d02719d06ddbea1f87941f","unresolved":false,"context_lines":[{"line_number":851,"context_line":"As usual, this is a simple ``node-update`` call::"},{"line_number":852,"context_line":""},{"line_number":853,"context_line":"    ironic --ironic-api-version 1.21 node-update $NODE_UUID \\"},{"line_number":854,"context_line":"    add resource_class\u003d$RESOURCE_CLASS"},{"line_number":855,"context_line":""},{"line_number":856,"context_line":""},{"line_number":857,"context_line":"Setup the drivers for the Bare Metal service"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa7ab95a_9a67ece9","line":854,"updated":"2016-08-31 10:02:20.000000000","message":"nit: I\u0027d prefer an indentation before \"add\" for ease of understanding","commit_id":"33c1ef8d3c7083115fa48a4565ee043ff47f121d"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"b24879f3b6c6885d751f6aa147fdb59fcaf49396","unresolved":false,"context_lines":[{"line_number":851,"context_line":"As usual, this is a simple ``node-update`` call::"},{"line_number":852,"context_line":""},{"line_number":853,"context_line":"    ironic --ironic-api-version 1.21 node-update $NODE_UUID \\"},{"line_number":854,"context_line":"    add resource_class\u003d$RESOURCE_CLASS"},{"line_number":855,"context_line":""},{"line_number":856,"context_line":""},{"line_number":857,"context_line":"Setup the drivers for the Bare Metal service"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a8ec9b2_e5b588d0","line":854,"in_reply_to":"fa7ab95a_9a67ece9","updated":"2016-09-20 12:45:34.000000000","message":"Done","commit_id":"33c1ef8d3c7083115fa48a4565ee043ff47f121d"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"acf794cdd2ddb76b327a71590cc325d9e84a3270","unresolved":false,"context_lines":[{"line_number":814,"context_line":"Resource classes"},{"line_number":815,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":816,"context_line":""},{"line_number":817,"context_line":"The Newton release release of the Bare Metal service includes a field on the"},{"line_number":818,"context_line":"node resource called ``resource_class``. It\u0027s expected that the Newton release"},{"line_number":819,"context_line":"of the Compute service will read and store information about this field, and"},{"line_number":820,"context_line":"the Ocata release of the Compute service will begin using this field for"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a8ec9b2_91ca434e","line":817,"updated":"2016-09-20 14:22:34.000000000","message":"s/release release/release/","commit_id":"c52eb6d57b1fb2c3ea29d4526a734e079365294e"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"acf794cdd2ddb76b327a71590cc325d9e84a3270","unresolved":false,"context_lines":[{"line_number":815,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":816,"context_line":""},{"line_number":817,"context_line":"The Newton release release of the Bare Metal service includes a field on the"},{"line_number":818,"context_line":"node resource called ``resource_class``. It\u0027s expected that the Newton release"},{"line_number":819,"context_line":"of the Compute service will read and store information about this field, and"},{"line_number":820,"context_line":"the Ocata release of the Compute service will begin using this field for"},{"line_number":821,"context_line":"scheduling, if it is set. This field is available in version 1.21 of the Bare"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a8ec9b2_d1fccb40","line":818,"updated":"2016-09-20 14:22:34.000000000","message":"You\u0027re going to hate me but s/It\u0027s expected that the/The/\n\ncuz JayF said that it is in newton (so 99.99% sure): https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L329\n\nJroll just said that it isn\u0027t being saved in nova\u0027s db.","commit_id":"c52eb6d57b1fb2c3ea29d4526a734e079365294e"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"acf794cdd2ddb76b327a71590cc325d9e84a3270","unresolved":false,"context_lines":[{"line_number":819,"context_line":"of the Compute service will read and store information about this field, and"},{"line_number":820,"context_line":"the Ocata release of the Compute service will begin using this field for"},{"line_number":821,"context_line":"scheduling, if it is set. This field is available in version 1.21 of the Bare"},{"line_number":822,"context_line":"Metal service API."},{"line_number":823,"context_line":""},{"line_number":824,"context_line":"In the future, a Compute flavor will use this field for scheduling, instead of"},{"line_number":825,"context_line":"the CPU/RAM/disk properties defined in the flavor above. A flavor will require"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a8ec9b2_31340ffa","line":822,"updated":"2016-09-20 14:22:34.000000000","message":"I am wondering whether it might be better to put this last sentence after the first sentence (to group the ironic part, instead of splitting it with the nova stuff).","commit_id":"c52eb6d57b1fb2c3ea29d4526a734e079365294e"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"60ec7d369e68779bee91e5afe1ba4812f6b6a9b6","unresolved":false,"context_lines":[{"line_number":818,"context_line":"resource called ``resource_class``. This field is available in version 1.21 of"},{"line_number":819,"context_line":"the Bare Metal service API."},{"line_number":820,"context_line":""},{"line_number":821,"context_line":"The Newton release of the Compute service will reads information about this"},{"line_number":822,"context_line":"field, and the Ocata release of the Compute service will begin storing this and"},{"line_number":823,"context_line":"using this field for scheduling, if it is set."},{"line_number":824,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"7a8ec9b2_10e346a5","line":821,"updated":"2016-09-21 14:40:17.000000000","message":"s/reads/read/\n\nDo we even need to mention that Newton will read it. It does nothing with it so it doesn\u0027t matter.","commit_id":"aaeea63ed971cd27c0b5abd1b770b9adfbfcda95"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"b73ea05c2102808a65bdb488f185c01dbd3a3031","unresolved":false,"context_lines":[{"line_number":822,"context_line":"field, and the Ocata release of the Compute service will begin storing this and"},{"line_number":823,"context_line":"using this field for scheduling, if it is set."},{"line_number":824,"context_line":""},{"line_number":825,"context_line":"In the future, a Compute flavor will use this field for scheduling, instead of"},{"line_number":826,"context_line":"the CPU/RAM/disk properties defined in the flavor above. A flavor will require"},{"line_number":827,"context_line":"*exactly one* of some bare metal resource class. For example, the following"},{"line_number":828,"context_line":"nodes may be enrolled in the Bare Metal service::"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7a8ec9b2_b6e9aab9","line":825,"updated":"2016-09-21 14:52:46.000000000","message":"Compute service flavor?","commit_id":"aaeea63ed971cd27c0b5abd1b770b9adfbfcda95"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"b73ea05c2102808a65bdb488f185c01dbd3a3031","unresolved":false,"context_lines":[{"line_number":827,"context_line":"*exactly one* of some bare metal resource class. For example, the following"},{"line_number":828,"context_line":"nodes may be enrolled in the Bare Metal service::"},{"line_number":829,"context_line":""},{"line_number":830,"context_line":"    - node-1:"},{"line_number":831,"context_line":"      resource_class: ironic-gold"},{"line_number":832,"context_line":"      properties:"},{"line_number":833,"context_line":"        cpus: 4"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7a8ec9b2_d6cb9651","line":830,"updated":"2016-09-21 14:52:46.000000000","message":"We might mention the possibility of doing \u0027ironic create\u0027 and providing a valid yaml for it here (there won\u0027t be many changes, just add an upper-level \"nodes\" list, and adding name and driver attributes to the nodes themselves)","commit_id":"aaeea63ed971cd27c0b5abd1b770b9adfbfcda95"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"60ec7d369e68779bee91e5afe1ba4812f6b6a9b6","unresolved":false,"context_lines":[{"line_number":846,"context_line":"The syntax for this ``flavor-create`` call is yet to be defined."},{"line_number":847,"context_line":""},{"line_number":848,"context_line":"It\u0027s important to set the ``resource_class`` field for nodes in the deployment"},{"line_number":849,"context_line":"before deploying Compute flavors that use the new model (expected in Ocata), so"},{"line_number":850,"context_line":"that the Compute service\u0027s resource tracker has time to read and store this"},{"line_number":851,"context_line":"metadata before the flavors are deployed. It\u0027s recommended to set this field"},{"line_number":852,"context_line":"after deploying ironic 6.1.0 and before deploying the Ocata version of the"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7a8ec9b2_30012207","line":849,"updated":"2016-09-21 14:40:17.000000000","message":"here you mention expected in Ocata, but above at line 825, you have \u0027In the future\u0027.","commit_id":"aaeea63ed971cd27c0b5abd1b770b9adfbfcda95"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"60ec7d369e68779bee91e5afe1ba4812f6b6a9b6","unresolved":false,"context_lines":[{"line_number":849,"context_line":"before deploying Compute flavors that use the new model (expected in Ocata), so"},{"line_number":850,"context_line":"that the Compute service\u0027s resource tracker has time to read and store this"},{"line_number":851,"context_line":"metadata before the flavors are deployed. It\u0027s recommended to set this field"},{"line_number":852,"context_line":"after deploying ironic 6.1.0 and before deploying the Ocata version of the"},{"line_number":853,"context_line":"Compute service."},{"line_number":854,"context_line":""},{"line_number":855,"context_line":"As usual, this is a simple ``node-update`` call::"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7a8ec9b2_6bef3b1e","line":852,"updated":"2016-09-21 14:40:17.000000000","message":"i wonder if we should also do s/6.1.0/6.1.0 or higher (for example Newton)/","commit_id":"aaeea63ed971cd27c0b5abd1b770b9adfbfcda95"}],"install-guide/source/configure-nova-flavors.rst":[{"author":{"_account_id":11297,"name":"Nisha Agarwal","email":"agarwalnisha1980@gmail.com","username":"Nisha"},"change_message_id":"dd44e3aab2128d78f7d67c96a5ec48e018b58baf","unresolved":false,"context_lines":[{"line_number":44,"context_line":"instead of the CPU, RAM, and disk properties defined in the flavor"},{"line_number":45,"context_line":"above. A flavor will require *exactly one* of some bare metal resource class."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"This work is still in progress, and the syntax for the ``flavor-create`` call"},{"line_number":48,"context_line":"to associate flavors with resource classes is yet to be defined."}],"source_content_type":"text/x-rst","patch_set":9,"id":"7f231b9d_5540ed35","line":48,"range":{"start_line":47,"start_character":0,"end_line":48,"end_character":64},"updated":"2017-06-12 04:20:55.000000000","message":"can we have a link added here so that it can be tracked? or alternatively we can add depends on.","commit_id":"ccbc796cdfcec6f51e6805e33a8825984d6d7791"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"0c1fa4f6949d2849b5d7a766842a6aeb8b95af8b","unresolved":false,"context_lines":[{"line_number":44,"context_line":"instead of the CPU, RAM, and disk properties defined in the flavor"},{"line_number":45,"context_line":"above. A flavor will require *exactly one* of some bare metal resource class."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"This work is still in progress, and the syntax for the ``flavor-create`` call"},{"line_number":48,"context_line":"to associate flavors with resource classes is yet to be defined."}],"source_content_type":"text/x-rst","patch_set":9,"id":"7f231b9d_c4ffa365","line":48,"range":{"start_line":47,"start_character":0,"end_line":48,"end_character":64},"in_reply_to":"7f231b9d_5540ed35","updated":"2017-06-12 11:15:22.000000000","message":"The spec for this stuff is here: https://review.openstack.org/#/c/446570/","commit_id":"ccbc796cdfcec6f51e6805e33a8825984d6d7791"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"24fa52c14d1346fa0617c49fda73afb42d7756e1","unresolved":false,"context_lines":[{"line_number":44,"context_line":"instead of the CPU, RAM, and disk properties defined in the flavor"},{"line_number":45,"context_line":"above. A flavor will require *exactly one* of some bare metal resource class."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"This work is still in progress, and the syntax for the ``flavor-create`` call"},{"line_number":48,"context_line":"to associate flavors with resource classes is yet to be defined."}],"source_content_type":"text/x-rst","patch_set":9,"id":"7f231b9d_a0f8e848","line":48,"range":{"start_line":47,"start_character":0,"end_line":48,"end_character":64},"in_reply_to":"7f231b9d_c4ffa365","updated":"2017-06-13 12:02:43.000000000","message":"Done","commit_id":"ccbc796cdfcec6f51e6805e33a8825984d6d7791"},{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"824d2906f27f8d705e8de0ec473c66487e3a2e26","unresolved":false,"context_lines":[{"line_number":40,"context_line":"resource called ``resource_class``. This field is available in version 1.21 of"},{"line_number":41,"context_line":"the Bare Metal service API."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"In the future (Pike or Queens release), a Compute service flavor will use this"},{"line_number":44,"context_line":"field for scheduling, instead of the CPU, RAM, and disk properties defined in"},{"line_number":45,"context_line":"the flavor above. A flavor will require *exactly one* of some bare metal"},{"line_number":46,"context_line":"resource class."}],"source_content_type":"text/x-rst","patch_set":10,"id":"5f201791_208fda75","line":43,"range":{"start_line":43,"start_character":7,"end_line":43,"end_character":19},"updated":"2017-06-27 04:58:28.000000000","message":"nit: Since we are in \u0027Pike\u0027, this cannot be termed as a \u0027future\u0027 release","commit_id":"622b3f30769ea0a8451ab07fabdffdf295cf1df9"}],"install-guide/source/enrollment.rst":[{"author":{"_account_id":10206,"name":"Madhuri Kumari","email":"madhuri.kumari@intel.com","username":"Madhuri"},"change_message_id":"dde79fabdf32ea23a240273ecf707aae2836f5d3","unresolved":false,"context_lines":[{"line_number":317,"context_line":""},{"line_number":318,"context_line":"   .. code-block:: console"},{"line_number":319,"context_line":""},{"line_number":320,"context_line":"    $ export IRONIC_API_VERSION\u003d1.21"},{"line_number":321,"context_line":"    $ ironic --ironic-api-version\u003d1.21 node-update $NODE_UUID \\"},{"line_number":322,"context_line":"        replace resource_class\u003d$CLASS_NAME"},{"line_number":323,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"7f231b9d_d97c41b7","line":320,"range":{"start_line":320,"start_character":13,"end_line":320,"end_character":31},"updated":"2017-06-11 16:07:12.000000000","message":"This is not used anywhere. Is it needed?","commit_id":"33973a8725d0003f03099079decbf98d3f4ae743"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ef76d3d148115643b57c16f8e49946b78273d5d9","unresolved":false,"context_lines":[{"line_number":317,"context_line":""},{"line_number":318,"context_line":"   .. code-block:: console"},{"line_number":319,"context_line":""},{"line_number":320,"context_line":"    $ export IRONIC_API_VERSION\u003d1.21"},{"line_number":321,"context_line":"    $ ironic --ironic-api-version\u003d1.21 node-update $NODE_UUID \\"},{"line_number":322,"context_line":"        replace resource_class\u003d$CLASS_NAME"},{"line_number":323,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"7f231b9d_2e5406ca","line":320,"range":{"start_line":320,"start_character":13,"end_line":320,"end_character":31},"in_reply_to":"7f231b9d_d97c41b7","updated":"2017-06-11 19:02:53.000000000","message":"redundant indeed","commit_id":"33973a8725d0003f03099079decbf98d3f4ae743"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"0c1fa4f6949d2849b5d7a766842a6aeb8b95af8b","unresolved":false,"context_lines":[{"line_number":247,"context_line":""},{"line_number":248,"context_line":"   See :doc:`configure-glance-images` for details."},{"line_number":249,"context_line":""},{"line_number":250,"context_line":"#. You must also inform the Bare Metal service of the network interface cards"},{"line_number":251,"context_line":"   which are part of the node by creating a port with each NIC\u0027s MAC address."},{"line_number":252,"context_line":"   These MAC addresses are passed to the Networking service during instance"},{"line_number":253,"context_line":"   provisioning and used to configure the network appropriately:"},{"line_number":254,"context_line":""}],"source_content_type":"text/x-rst","patch_set":9,"id":"7f231b9d_2418971b","line":251,"range":{"start_line":250,"start_character":3,"end_line":251,"end_character":77},"updated":"2017-06-12 11:15:22.000000000","message":"You must also provide the Bare Metal service with information about the network interface cards you want expose on the node by creating a port with each NIC\u0027s MAC address.\n\n(Also maybe worth mentioning the local link connection information too, e.g. switch_id, port_id)","commit_id":"ccbc796cdfcec6f51e6805e33a8825984d6d7791"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"24fa52c14d1346fa0617c49fda73afb42d7756e1","unresolved":false,"context_lines":[{"line_number":247,"context_line":""},{"line_number":248,"context_line":"   See :doc:`configure-glance-images` for details."},{"line_number":249,"context_line":""},{"line_number":250,"context_line":"#. You must also inform the Bare Metal service of the network interface cards"},{"line_number":251,"context_line":"   which are part of the node by creating a port with each NIC\u0027s MAC address."},{"line_number":252,"context_line":"   These MAC addresses are passed to the Networking service during instance"},{"line_number":253,"context_line":"   provisioning and used to configure the network appropriately:"},{"line_number":254,"context_line":""}],"source_content_type":"text/x-rst","patch_set":9,"id":"7f231b9d_4046b47e","line":251,"range":{"start_line":250,"start_character":3,"end_line":251,"end_character":77},"in_reply_to":"7f231b9d_2418971b","updated":"2017-06-13 12:02:43.000000000","message":"This part was merely copied from below, I\u0027d prefer to not change it in the same patch.","commit_id":"ccbc796cdfcec6f51e6805e33a8825984d6d7791"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"d38035b40dcecb34711ae3f920e5bca967279d26","unresolved":false,"context_lines":[{"line_number":325,"context_line":""},{"line_number":326,"context_line":"   .. note::"},{"line_number":327,"context_line":"      Scheduling based on resource classes will replace scheduling based on"},{"line_number":328,"context_line":"      properties in the future."},{"line_number":329,"context_line":""},{"line_number":330,"context_line":"#. If you wish to perform more advanced scheduling of the instances based on"},{"line_number":331,"context_line":"   hardware capabilities, you may add metadata to each node that will be"}],"source_content_type":"text/x-rst","patch_set":10,"id":"5f201791_6be14a2a","line":328,"updated":"2017-06-26 13:30:20.000000000","message":"Nit, the repetition of the note below, when there is a note above feels redundant.","commit_id":"622b3f30769ea0a8451ab07fabdffdf295cf1df9"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"754cfe6e81f92b597baa2caa0855278077318622","unresolved":false,"context_lines":[{"line_number":325,"context_line":""},{"line_number":326,"context_line":"   .. note::"},{"line_number":327,"context_line":"      Scheduling based on resource classes will replace scheduling based on"},{"line_number":328,"context_line":"      properties in the future."},{"line_number":329,"context_line":""},{"line_number":330,"context_line":"#. If you wish to perform more advanced scheduling of the instances based on"},{"line_number":331,"context_line":"   hardware capabilities, you may add metadata to each node that will be"}],"source_content_type":"text/x-rst","patch_set":10,"id":"5f201791_f682beff","line":328,"in_reply_to":"5f201791_4072d67c","updated":"2017-06-27 12:50:41.000000000","message":"+1","commit_id":"622b3f30769ea0a8451ab07fabdffdf295cf1df9"},{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"824d2906f27f8d705e8de0ec473c66487e3a2e26","unresolved":false,"context_lines":[{"line_number":325,"context_line":""},{"line_number":326,"context_line":"   .. note::"},{"line_number":327,"context_line":"      Scheduling based on resource classes will replace scheduling based on"},{"line_number":328,"context_line":"      properties in the future."},{"line_number":329,"context_line":""},{"line_number":330,"context_line":"#. If you wish to perform more advanced scheduling of the instances based on"},{"line_number":331,"context_line":"   hardware capabilities, you may add metadata to each node that will be"}],"source_content_type":"text/x-rst","patch_set":10,"id":"5f201791_4072d67c","line":328,"in_reply_to":"5f201791_6be14a2a","updated":"2017-06-27 04:58:28.000000000","message":"+1","commit_id":"622b3f30769ea0a8451ab07fabdffdf295cf1df9"}],"install-guide/source/include/configure-nova-flavors.rst":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"db7315899c83893133515426fc63547fb0d04164","unresolved":false,"context_lines":[{"line_number":45,"context_line":"resource called ``resource_class``. This field is available in version 1.21 of"},{"line_number":46,"context_line":"the Bare Metal service API."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"In the future (as early as the Pike release), a Compute service flavor will use"},{"line_number":49,"context_line":"this field for scheduling, instead of the CPU/RAM/disk properties defined in"},{"line_number":50,"context_line":"the flavor above. A flavor will require *exactly one* of some bare metal"},{"line_number":51,"context_line":"resource class. For example, the following nodes may be enrolled in the Bare"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3a461143_104e3996","line":48,"updated":"2017-01-26 20:54:02.000000000","message":"do we know yet, whether it will be IN PIKE? I know I like to be pessimistic, but at this point, maybe we can put \u0027In Pike, the next release,\u0027 and edit it later if it doesn\u0027t happen?","commit_id":"24945c379981021976a1791d839716c38a75d8ef"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"be0832f8298d6f8e06d40533f7d1173343cbed07","unresolved":false,"context_lines":[{"line_number":46,"context_line":"the Bare Metal service API."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"In the future (as early as the Pike release), a Compute service flavor will use"},{"line_number":49,"context_line":"this field for scheduling, instead of the CPU/RAM/disk properties defined in"},{"line_number":50,"context_line":"the flavor above. A flavor will require *exactly one* of some bare metal"},{"line_number":51,"context_line":"resource class. For example, the following nodes may be enrolled in the Bare"},{"line_number":52,"context_line":"Metal service::"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3a461143_331458f4","line":49,"range":{"start_line":49,"start_character":50,"end_line":49,"end_character":54},"updated":"2017-01-25 20:32:43.000000000","message":"might be capitalize disk? looks odd on rendered webpage.","commit_id":"24945c379981021976a1791d839716c38a75d8ef"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"29436f7b6976e2d93261d5c9a1fb4f008556fe29","unresolved":false,"context_lines":[{"line_number":46,"context_line":"the Bare Metal service API."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"In the future (as early as the Pike release), a Compute service flavor will use"},{"line_number":49,"context_line":"this field for scheduling, instead of the CPU/RAM/disk properties defined in"},{"line_number":50,"context_line":"the flavor above. A flavor will require *exactly one* of some bare metal"},{"line_number":51,"context_line":"resource class. For example, the following nodes may be enrolled in the Bare"},{"line_number":52,"context_line":"Metal service::"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3a461143_36d96e84","line":49,"range":{"start_line":49,"start_character":50,"end_line":49,"end_character":54},"in_reply_to":"3a461143_331458f4","updated":"2017-01-26 20:35:34.000000000","message":"Will do","commit_id":"24945c379981021976a1791d839716c38a75d8ef"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"db7315899c83893133515426fc63547fb0d04164","unresolved":false,"context_lines":[{"line_number":46,"context_line":"the Bare Metal service API."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"In the future (as early as the Pike release), a Compute service flavor will use"},{"line_number":49,"context_line":"this field for scheduling, instead of the CPU/RAM/disk properties defined in"},{"line_number":50,"context_line":"the flavor above. A flavor will require *exactly one* of some bare metal"},{"line_number":51,"context_line":"resource class. For example, the following nodes may be enrolled in the Bare"},{"line_number":52,"context_line":"Metal service::"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3a461143_d6574a00","line":49,"range":{"start_line":49,"start_character":50,"end_line":49,"end_character":54},"in_reply_to":"3a461143_331458f4","updated":"2017-01-26 20:54:02.000000000","message":"it might look odd but \u0027disk\u0027 is correct. DISK would be an acronym for something and that isn\u0027t what is meant here.\n\nWould \u0027CPU, RAM, and disk properties\u0027 be better?","commit_id":"24945c379981021976a1791d839716c38a75d8ef"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"be0832f8298d6f8e06d40533f7d1173343cbed07","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        memory_mb: 8192"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"Rather than creating an ``ironic-gold`` flavor that requires 16384 MB of RAM"},{"line_number":68,"context_line":"and 4 CPUs, one would create an ``ironic-gold`` flavor that requires one"},{"line_number":69,"context_line":"``ironic-gold`` resource. That flavor would then be scheduled to ``node-1``."},{"line_number":70,"context_line":"The syntax for this ``flavor-create`` call is yet to be defined."},{"line_number":71,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"3a461143_13411c0c","line":68,"updated":"2017-01-25 20:32:43.000000000","message":"Can we add here how to map resource class with flavor? or add link to nova doc where it is described, not sure it such exist.","commit_id":"24945c379981021976a1791d839716c38a75d8ef"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"29436f7b6976e2d93261d5c9a1fb4f008556fe29","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        memory_mb: 8192"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"Rather than creating an ``ironic-gold`` flavor that requires 16384 MB of RAM"},{"line_number":68,"context_line":"and 4 CPUs, one would create an ``ironic-gold`` flavor that requires one"},{"line_number":69,"context_line":"``ironic-gold`` resource. That flavor would then be scheduled to ``node-1``."},{"line_number":70,"context_line":"The syntax for this ``flavor-create`` call is yet to be defined."},{"line_number":71,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"3a461143_d6f3ca08","line":68,"in_reply_to":"3a461143_13411c0c","updated":"2017-01-26 20:35:34.000000000","message":"We don\u0027t know how it will be mapped to a flavor yet.","commit_id":"24945c379981021976a1791d839716c38a75d8ef"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"db7315899c83893133515426fc63547fb0d04164","unresolved":false,"context_lines":[{"line_number":67,"context_line":"Rather than creating an ``ironic-gold`` flavor that requires 16384 MB of RAM"},{"line_number":68,"context_line":"and 4 CPUs, one would create an ``ironic-gold`` flavor that requires one"},{"line_number":69,"context_line":"``ironic-gold`` resource. That flavor would then be scheduled to ``node-1``."},{"line_number":70,"context_line":"The syntax for this ``flavor-create`` call is yet to be defined."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"It\u0027s important to set the ``resource_class`` field for nodes in the deployment"},{"line_number":73,"context_line":"before deploying Compute flavors that use the new scheduling model, so"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3a461143_366ece96","line":70,"updated":"2017-01-26 20:54:02.000000000","message":"so we still don\u0027t know what this call is?","commit_id":"24945c379981021976a1791d839716c38a75d8ef"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"db7315899c83893133515426fc63547fb0d04164","unresolved":false,"context_lines":[{"line_number":72,"context_line":"It\u0027s important to set the ``resource_class`` field for nodes in the deployment"},{"line_number":73,"context_line":"before deploying Compute flavors that use the new scheduling model, so"},{"line_number":74,"context_line":"that the Compute service\u0027s resource tracker has time to read and store this"},{"line_number":75,"context_line":"metadata before the flavors are deployed. It\u0027s recommended to set this field"},{"line_number":76,"context_line":"before deploying the Pike version of the Compute service."},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"As usual, this is a simple ``node-update`` call::"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3a461143_d6b66a89","line":75,"updated":"2017-01-26 20:54:02.000000000","message":"not sure why, but I don\u0027t really like contractions that much. nit: s/It\u0027s/It is/\n\nIn fact, maybe make it even more clear:\n\nIt is recommended that all the nodes\u0027 ``resource_class`` fields are set \u003citalicize\u003ebefore\u003cend\u003e using the Pike version...\n\n(you use \u0027deploy\u0027 four times in this para, they don\u0027t quite mean the same deploy.)\n\nShould we even be more explicit and itemize the order:\n1. set resource_class fields\n2. upgrade to Pike\n3. add new-fangled flavors in nova\n4. profit? (or is there more to do)","commit_id":"24945c379981021976a1791d839716c38a75d8ef"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"88b3d72774c7c936e55af639cd03cc6c53a32fe7","unresolved":false,"context_lines":[{"line_number":77,"context_line":""},{"line_number":78,"context_line":"As usual, this is a simple ``node-update`` call::"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    ironic --ironic-api-version 1.21 node-update $NODE_UUID \\"},{"line_number":81,"context_line":"        add resource_class\u003d$RESOURCE_CLASS"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"Or can be included in a ``node-create`` call::"},{"line_number":84,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"5a3905b3_e63680e6","line":81,"range":{"start_line":80,"start_character":0,"end_line":81,"end_character":42},"updated":"2017-01-25 10:41:09.000000000","message":"Mostly a nit but to be consistent with the rest of the command samples in this file we should use the \"code-block\" tag for syntax highlight (see L11, L20, etc...)\n\n .. code-block:: console\n\n   $ ironic --ironic-api-version 1.21 ...","commit_id":"24945c379981021976a1791d839716c38a75d8ef"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"db7315899c83893133515426fc63547fb0d04164","unresolved":false,"context_lines":[{"line_number":77,"context_line":""},{"line_number":78,"context_line":"As usual, this is a simple ``node-update`` call::"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    ironic --ironic-api-version 1.21 node-update $NODE_UUID \\"},{"line_number":81,"context_line":"        add resource_class\u003d$RESOURCE_CLASS"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"Or can be included in a ``node-create`` call::"},{"line_number":84,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"3a461143_d644aa75","line":81,"range":{"start_line":80,"start_character":0,"end_line":81,"end_character":42},"in_reply_to":"3a461143_5341e438","updated":"2017-01-26 20:54:02.000000000","message":"++ to both","commit_id":"24945c379981021976a1791d839716c38a75d8ef"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"29436f7b6976e2d93261d5c9a1fb4f008556fe29","unresolved":false,"context_lines":[{"line_number":77,"context_line":""},{"line_number":78,"context_line":"As usual, this is a simple ``node-update`` call::"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    ironic --ironic-api-version 1.21 node-update $NODE_UUID \\"},{"line_number":81,"context_line":"        add resource_class\u003d$RESOURCE_CLASS"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"Or can be included in a ``node-create`` call::"},{"line_number":84,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"3a461143_b6d39e5e","line":81,"range":{"start_line":80,"start_character":0,"end_line":81,"end_character":42},"in_reply_to":"3a461143_5341e438","updated":"2017-01-26 20:35:34.000000000","message":"Will do (for both)","commit_id":"24945c379981021976a1791d839716c38a75d8ef"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"be0832f8298d6f8e06d40533f7d1173343cbed07","unresolved":false,"context_lines":[{"line_number":77,"context_line":""},{"line_number":78,"context_line":"As usual, this is a simple ``node-update`` call::"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    ironic --ironic-api-version 1.21 node-update $NODE_UUID \\"},{"line_number":81,"context_line":"        add resource_class\u003d$RESOURCE_CLASS"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"Or can be included in a ``node-create`` call::"},{"line_number":84,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"3a461143_5341e438","line":81,"range":{"start_line":80,"start_character":0,"end_line":81,"end_character":42},"in_reply_to":"5a3905b3_e63680e6","updated":"2017-01-25 20:32:43.000000000","message":"lets use openstack client here instead.","commit_id":"24945c379981021976a1791d839716c38a75d8ef"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"88b3d72774c7c936e55af639cd03cc6c53a32fe7","unresolved":false,"context_lines":[{"line_number":82,"context_line":""},{"line_number":83,"context_line":"Or can be included in a ``node-create`` call::"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"    ironic --ironic-api-version 1.21 node-create ... \\"},{"line_number":86,"context_line":"        --resource-class $RESOURCE_CLASS"}],"source_content_type":"text/x-rst","patch_set":6,"id":"5a3905b3_2648885b","line":86,"range":{"start_line":85,"start_character":0,"end_line":86,"end_character":40},"updated":"2017-01-25 10:41:09.000000000","message":"ditto","commit_id":"24945c379981021976a1791d839716c38a75d8ef"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"29436f7b6976e2d93261d5c9a1fb4f008556fe29","unresolved":false,"context_lines":[{"line_number":82,"context_line":""},{"line_number":83,"context_line":"Or can be included in a ``node-create`` call::"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"    ironic --ironic-api-version 1.21 node-create ... \\"},{"line_number":86,"context_line":"        --resource-class $RESOURCE_CLASS"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3a461143_d6caeaa0","line":86,"range":{"start_line":85,"start_character":0,"end_line":86,"end_character":40},"in_reply_to":"5a3905b3_2648885b","updated":"2017-01-26 20:35:34.000000000","message":"ditto","commit_id":"24945c379981021976a1791d839716c38a75d8ef"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"bc9c55a7c50333827b87573ab379c000f65e5fcb","unresolved":false,"context_lines":[{"line_number":42,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"The Newton release of the Bare Metal service includes a field on the node"},{"line_number":45,"context_line":"resource called ``resource_class``. This field is available in version 1.21 of"},{"line_number":46,"context_line":"the Bare Metal service API."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"In the Pike release, a Compute service flavor will use this field for"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3a461143_1b02550a","line":45,"updated":"2017-01-31 14:26:20.000000000","message":"nit: Perhaps \"starting with version 1.21\"? or \"1.21 and higher\"?","commit_id":"a61a75f445761072558448cb651a2af2fafebe8d"},{"author":{"_account_id":13295,"name":"Mario Villaplana","email":"mario.villaplana@gmail.com","username":"mariojv"},"change_message_id":"1bdaa361da279d60e7c731675bd56aa1fa74802f","unresolved":false,"context_lines":[{"line_number":45,"context_line":"resource called ``resource_class``. This field is available in version 1.21 of"},{"line_number":46,"context_line":"the Bare Metal service API."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"In the Pike release, a Compute service flavor will use this field for"},{"line_number":49,"context_line":"scheduling, instead of the CPU, RAM, and disk properties defined in the flavor"},{"line_number":50,"context_line":"above. A flavor will require *exactly one* of some bare metal resource class."},{"line_number":51,"context_line":"For example, the following nodes may be enrolled in the Bare Metal service::"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3a461143_42593f34","line":48,"range":{"start_line":48,"start_character":0,"end_line":48,"end_character":2},"updated":"2017-01-31 21:39:13.000000000","message":"nit: \"As of the Pike release\" ? This sounds as though this feature will be done in the middle of the Pike release.","commit_id":"a61a75f445761072558448cb651a2af2fafebe8d"},{"author":{"_account_id":11297,"name":"Nisha Agarwal","email":"agarwalnisha1980@gmail.com","username":"Nisha"},"change_message_id":"cfaa3d19bac2a09d554bfc0175701656c864b31d","unresolved":false,"context_lines":[{"line_number":46,"context_line":"the Bare Metal service API."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"In the Pike release, a Compute service flavor will use this field for"},{"line_number":49,"context_line":"scheduling, instead of the CPU, RAM, and disk properties defined in the flavor"},{"line_number":50,"context_line":"above. A flavor will require *exactly one* of some bare metal resource class."},{"line_number":51,"context_line":"For example, the following nodes may be enrolled in the Bare Metal service::"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    - node-1:"}],"source_content_type":"text/x-rst","patch_set":7,"id":"1a1ced50_6c7f4166","line":50,"range":{"start_line":49,"start_character":1,"end_line":50,"end_character":5},"updated":"2017-03-22 03:34:22.000000000","message":"so what will be the impact on inspection? these were the mandatory attributes to be inspected as Nova used to schedule nodes based on them. Do we plan to have them as mandatory attributes still. It may not be the proper place to leave my comment, but just a concern.","commit_id":"a61a75f445761072558448cb651a2af2fafebe8d"},{"author":{"_account_id":10202,"name":"Wan-yen Hsu","email":"wanyenhsu@gmail.com"},"change_message_id":"5cb0236ca0c08e7d151ad3b7679f69f91c8180f0","unresolved":false,"context_lines":[{"line_number":47,"context_line":""},{"line_number":48,"context_line":"In the Pike release, a Compute service flavor will use this field for"},{"line_number":49,"context_line":"scheduling, instead of the CPU, RAM, and disk properties defined in the flavor"},{"line_number":50,"context_line":"above. A flavor will require *exactly one* of some bare metal resource class."},{"line_number":51,"context_line":"For example, the following nodes may be enrolled in the Bare Metal service::"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    - node-1:"}],"source_content_type":"text/x-rst","patch_set":7,"id":"7ffa3b31_12cbbc53","line":50,"in_reply_to":"1a1ced50_6c7f4166","updated":"2017-04-13 22:52:23.000000000","message":"I have questions and concerns regarding resource class and inspection as well.   Inspection discovers cpus, memory_mb, capabilities and populates them into node properties and capabilities so that a user does not need to manually configure these info for every node.  Resource class requires these info to be configured for every node.\nI am afraid  that resource class is taking away this very useful inspection-and-self-configuration feature from users.  I also felt that the behavior of the resource_class filtering is different from the current filtering behavior.  Currently, if a user requests a flavor with 2cpus, a node with 2 or more cpus will satisfy the requirement.  With resource_class, it seems to me that it has be the exact match of the resource_class.  So, if a user requests a ironic_silver and there are no more ironic_silver available but some ironic_gold are still available, the scheduling will fail.  This may or may not be the desirable behavior for the users.   Let me know if I mis-understood how Nova scheduling and resource-class works.\n\n  My fundamental question is that is there a mechanism to auto populate the inspection-discovered info into resource class?   Can Ironic allow more than one resource classes?  Some can be auto-created by the inspection?  Thanks!","commit_id":"a61a75f445761072558448cb651a2af2fafebe8d"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"3b8ea851a759aaae2fbfedc3f49ef0cbea7cd042","unresolved":false,"context_lines":[{"line_number":47,"context_line":""},{"line_number":48,"context_line":"In the Pike release, a Compute service flavor will use this field for"},{"line_number":49,"context_line":"scheduling, instead of the CPU, RAM, and disk properties defined in the flavor"},{"line_number":50,"context_line":"above. A flavor will require *exactly one* of some bare metal resource class."},{"line_number":51,"context_line":"For example, the following nodes may be enrolled in the Bare Metal service::"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    - node-1:"}],"source_content_type":"text/x-rst","patch_set":7,"id":"ff0f0b1f_f03b5ac5","line":50,"in_reply_to":"7ffa3b31_12cbbc53","updated":"2017-05-19 12:14:52.000000000","message":"The impact of resource class based scheduling shouldn\u0027t affect how inspection is done, however I expect that we *might* create a new optional inspector hook for generating a default resource class, and we can document how an operator can write an inspector rule to process inspection results into a resource class for more fine grain operator control.\n\nIronic Nova flavors even with CPU/RAM/DISK have always been intended to be for doing exact matches, resource classes help us move to a world where this works correctly instead of having to hack this into nova using custom filters and host managers, and ending up with things like what Wan-yen has described with flavors being a minimum spec for what your getting.","commit_id":"a61a75f445761072558448cb651a2af2fafebe8d"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"bc9c55a7c50333827b87573ab379c000f65e5fcb","unresolved":false,"context_lines":[{"line_number":70,"context_line":""},{"line_number":71,"context_line":"It is recommended to set the ``resource_class`` field for nodes registered"},{"line_number":72,"context_line":"with the Bare Metal service *before* using the Pike version of the Compute"},{"line_number":73,"context_line":"service. The upgrade process from Ocata to Pike is still being worked on. It"},{"line_number":74,"context_line":"will be published before the Pike release, and will likely look like:"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    1. Set ``resource_class`` for each bare metal node."},{"line_number":77,"context_line":"    2. Modify nova flavors to use the resource classes."}],"source_content_type":"text/x-rst","patch_set":7,"id":"3a461143_3ba1d1f8","line":74,"range":{"start_line":73,"start_character":9,"end_line":74,"end_character":69},"updated":"2017-01-31 14:26:20.000000000","message":"I\u0027m kind of uncomfortable with this in the text due to the tentative nature and tone.  I really think we could go with a Warning block as a reminder, or explicitly lay out that as part of resource classes, this is what ironic expects to occur before and after an operator upgrades to pike.","commit_id":"a61a75f445761072558448cb651a2af2fafebe8d"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"7b6b3a082d8fd129628d2ac4184de5fbf9074859","unresolved":false,"context_lines":[{"line_number":70,"context_line":""},{"line_number":71,"context_line":"It is recommended to set the ``resource_class`` field for nodes registered"},{"line_number":72,"context_line":"with the Bare Metal service *before* using the Pike version of the Compute"},{"line_number":73,"context_line":"service. The upgrade process from Ocata to Pike is still being worked on. It"},{"line_number":74,"context_line":"will be published before the Pike release, and will likely look like:"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    1. Set ``resource_class`` for each bare metal node."},{"line_number":77,"context_line":"    2. Modify nova flavors to use the resource classes."}],"source_content_type":"text/x-rst","patch_set":7,"id":"3a461143_e61ecd0a","line":74,"range":{"start_line":73,"start_character":9,"end_line":74,"end_character":69},"in_reply_to":"3a461143_3ba1d1f8","updated":"2017-01-31 15:50:12.000000000","message":"I\u0027m fine with this in a separate paragraph. I\u0027m not sure a warning is a good idea, but see what it looks like. It makes me think that we are warning folks that this is a danger zone. A Note might be sufficient. Or is there some other way to highlight it so it isn\u0027t in blinking warning signs? :) Italicize it?","commit_id":"a61a75f445761072558448cb651a2af2fafebe8d"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"7b6b3a082d8fd129628d2ac4184de5fbf9074859","unresolved":false,"context_lines":[{"line_number":74,"context_line":"will be published before the Pike release, and will likely look like:"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    1. Set ``resource_class`` for each bare metal node."},{"line_number":77,"context_line":"    2. Modify nova flavors to use the resource classes."},{"line_number":78,"context_line":"    3. Upgrade to Pike"},{"line_number":79,"context_line":"    4. Modify nova flavors to remove the (now unnecessary) CPU, RAM, and disk"},{"line_number":80,"context_line":"       properties."}],"source_content_type":"text/x-rst","patch_set":7,"id":"3a461143_a6f5a5ed","line":77,"updated":"2017-01-31 15:50:12.000000000","message":"s/nova/Compute service/","commit_id":"a61a75f445761072558448cb651a2af2fafebe8d"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"7b6b3a082d8fd129628d2ac4184de5fbf9074859","unresolved":false,"context_lines":[{"line_number":76,"context_line":"    1. Set ``resource_class`` for each bare metal node."},{"line_number":77,"context_line":"    2. Modify nova flavors to use the resource classes."},{"line_number":78,"context_line":"    3. Upgrade to Pike"},{"line_number":79,"context_line":"    4. Modify nova flavors to remove the (now unnecessary) CPU, RAM, and disk"},{"line_number":80,"context_line":"       properties."},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"As usual, this is a simple ``node-update`` call:"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3a461143_c6e0f12b","line":79,"updated":"2017-01-31 15:50:12.000000000","message":"s/nova/Compute service/","commit_id":"a61a75f445761072558448cb651a2af2fafebe8d"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"7b6b3a082d8fd129628d2ac4184de5fbf9074859","unresolved":false,"context_lines":[{"line_number":77,"context_line":"    2. Modify nova flavors to use the resource classes."},{"line_number":78,"context_line":"    3. Upgrade to Pike"},{"line_number":79,"context_line":"    4. Modify nova flavors to remove the (now unnecessary) CPU, RAM, and disk"},{"line_number":80,"context_line":"       properties."},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"As usual, this is a simple ``node-update`` call:"},{"line_number":83,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"3a461143_065599c4","line":80,"updated":"2017-01-31 15:50:12.000000000","message":"Did I ask this before? Will the old (non-resource class) flavors still work?\n\nI think some clusters will have the flavor offerings available to users via some UI (horizon or other dashboard). So this step may not be \u0027just\u0027 modifications to nova, but I guess that is outside the scope of this (and we don\u0027t even want to mention it).","commit_id":"a61a75f445761072558448cb651a2af2fafebe8d"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"3b8ea851a759aaae2fbfedc3f49ef0cbea7cd042","unresolved":false,"context_lines":[{"line_number":77,"context_line":"    2. Modify nova flavors to use the resource classes."},{"line_number":78,"context_line":"    3. Upgrade to Pike"},{"line_number":79,"context_line":"    4. Modify nova flavors to remove the (now unnecessary) CPU, RAM, and disk"},{"line_number":80,"context_line":"       properties."},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"As usual, this is a simple ``node-update`` call:"},{"line_number":83,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"ff0f0b1f_f0923a9f","line":80,"in_reply_to":"3a461143_065599c4","updated":"2017-05-19 12:14:52.000000000","message":"I think the current plan is that at some point Ironic Virt driver will stop populating the resource classes for CPUs RAM and disk on the resource providers representing Ironic nodes. At this point the old flavors would stop working because the ironic nodes wouldn\u0027t provide the resources requested by those flavors any more.","commit_id":"a61a75f445761072558448cb651a2af2fafebe8d"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"bc9c55a7c50333827b87573ab379c000f65e5fcb","unresolved":false,"context_lines":[{"line_number":79,"context_line":"    4. Modify nova flavors to remove the (now unnecessary) CPU, RAM, and disk"},{"line_number":80,"context_line":"       properties."},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"As usual, this is a simple ``node-update`` call:"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"   .. code-block:: console"},{"line_number":85,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"3a461143_1bd33561","line":82,"range":{"start_line":82,"start_character":0,"end_line":82,"end_character":26},"updated":"2017-01-31 14:26:20.000000000","message":"nit: Maybe put the commands prior to the upgrade process?  Perhaps the expectation that this must be done prior to upgrading to pike should be stressed in a warning block.\n\nAlso, I think this like might be better as \"Adding the resource class to a node requires a simple ``node-update`` call:\" to avoid any confusion for the reader.","commit_id":"a61a75f445761072558448cb651a2af2fafebe8d"},{"author":{"_account_id":13295,"name":"Mario Villaplana","email":"mario.villaplana@gmail.com","username":"mariojv"},"change_message_id":"1bdaa361da279d60e7c731675bd56aa1fa74802f","unresolved":false,"context_lines":[{"line_number":79,"context_line":"    4. Modify nova flavors to remove the (now unnecessary) CPU, RAM, and disk"},{"line_number":80,"context_line":"       properties."},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"As usual, this is a simple ``node-update`` call:"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"   .. code-block:: console"},{"line_number":85,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"3a461143_42113f03","line":82,"range":{"start_line":82,"start_character":10,"end_line":82,"end_character":14},"updated":"2017-01-31 21:39:13.000000000","message":"step 1 is still a simple node-update call","commit_id":"a61a75f445761072558448cb651a2af2fafebe8d"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"7b6b3a082d8fd129628d2ac4184de5fbf9074859","unresolved":false,"context_lines":[{"line_number":79,"context_line":"    4. Modify nova flavors to remove the (now unnecessary) CPU, RAM, and disk"},{"line_number":80,"context_line":"       properties."},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"As usual, this is a simple ``node-update`` call:"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"   .. code-block:: console"},{"line_number":85,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"3a461143_867b497a","line":82,"range":{"start_line":82,"start_character":0,"end_line":82,"end_character":26},"in_reply_to":"3a461143_1bd33561","updated":"2017-01-31 15:50:12.000000000","message":"Yes, that\u0027s a good idea. As it is now, it seems out of context. This (the rest of this) could go after the first sentence that starts at L71.\n\nThen the big warning/notes/whatever paragraph that Julia would like to see about possible upgrade steps.","commit_id":"a61a75f445761072558448cb651a2af2fafebe8d"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"bc9c55a7c50333827b87573ab379c000f65e5fcb","unresolved":false,"context_lines":[{"line_number":86,"context_line":"      $ openstack --os-baremetal-api-version 1.21 baremetal node set \\"},{"line_number":87,"context_line":"          --resource-class $RESOURCE_CLASS $NODE_UUID"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"Or can be included in a ``node-create`` call:"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"   .. code-block:: console"},{"line_number":92,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"3a461143_7b119903","line":89,"range":{"start_line":89,"start_character":0,"end_line":89,"end_character":6},"updated":"2017-01-31 14:26:20.000000000","message":"nit: Maybe \"This can\", or \"The resource class setting can also be added upon the creation of a node entry.\" ?","commit_id":"a61a75f445761072558448cb651a2af2fafebe8d"},{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"bdda863b7b23b184f9685c5d7bd23353af293419","unresolved":false,"context_lines":[{"line_number":91,"context_line":"   .. code-block:: console"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"      $ openstack --os-baremetal-api-version 1.21 baremetal node create \\"},{"line_number":94,"context_line":"          --resource-class $RESOURCE_CLASS ..."}],"source_content_type":"text/x-rst","patch_set":7,"id":"bff0334d_07306ae7","line":94,"range":{"start_line":94,"start_character":10,"end_line":94,"end_character":46},"updated":"2017-04-11 05:30:16.000000000","message":"A Openstack CLI example on how to set the \u0027resource-class\u0027 in nova flavor would be useful.","commit_id":"a61a75f445761072558448cb651a2af2fafebe8d"}]}