)]}'
{"doc/source/dev/code-contribution-guide.rst":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5feca8f5deda5aa127471b04f2ef20b1277fd4bb","unresolved":false,"context_lines":[{"line_number":11,"context_line":"Adding new features"},{"line_number":12,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"In Mitaka release, Ironic started to track new features using Requests for"},{"line_number":15,"context_line":"feature enhancements instead of blueprints. These are bugs with \u0027rfe\u0027 tag, and"},{"line_number":16,"context_line":"they should be submitted before a spec or code is proposed. When the drivers"},{"line_number":17,"context_line":"team will decide that the proposal is worth implementing, a spec (if needed)"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_f9b9637a","line":14,"updated":"2015-12-17 15:09:55.000000000","message":"how about \u0027Starting with the Mitaka development cycle, Ironic tracks new features using RFEs (Requests for Feature Enhancements) ...\u0027","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"8a94895c63495f1652901adf7a4a6ca0b9dd4fa9","unresolved":false,"context_lines":[{"line_number":11,"context_line":"Adding new features"},{"line_number":12,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"In Mitaka release, Ironic started to track new features using Requests for"},{"line_number":15,"context_line":"feature enhancements instead of blueprints. These are bugs with \u0027rfe\u0027 tag, and"},{"line_number":16,"context_line":"they should be submitted before a spec or code is proposed. When the drivers"},{"line_number":17,"context_line":"team will decide that the proposal is worth implementing, a spec (if needed)"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_df080430","line":14,"in_reply_to":"3a7e1126_f9b9637a","updated":"2015-12-17 16:29:53.000000000","message":"Done","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":10697,"name":"Thiago Paiva Brito","email":"thiagop@lsd.ufcg.edu.br","username":"thiagop","inactive":true},"change_message_id":"eba75401dea07d30bfd7c1b9970065166c7e5ab3","unresolved":false,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"In Mitaka release, Ironic started to track new features using Requests for"},{"line_number":15,"context_line":"feature enhancements instead of blueprints. These are bugs with \u0027rfe\u0027 tag, and"},{"line_number":16,"context_line":"they should be submitted before a spec or code is proposed. When the drivers"},{"line_number":17,"context_line":"team will decide that the proposal is worth implementing, a spec (if needed)"},{"line_number":18,"context_line":"and code should be submitted, referencing the RFE bug."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Here is a list of steps to do during the new process of adding functional to"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_f9d403be","line":17,"range":{"start_line":16,"start_character":69,"end_line":17,"end_character":4},"updated":"2015-12-17 15:07:26.000000000","message":"drivers team? This is intended just for driver features? If you meant ironic-drivers team on launchpad, it worth being more specific.","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"8a94895c63495f1652901adf7a4a6ca0b9dd4fa9","unresolved":false,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"In Mitaka release, Ironic started to track new features using Requests for"},{"line_number":15,"context_line":"feature enhancements instead of blueprints. These are bugs with \u0027rfe\u0027 tag, and"},{"line_number":16,"context_line":"they should be submitted before a spec or code is proposed. When the drivers"},{"line_number":17,"context_line":"team will decide that the proposal is worth implementing, a spec (if needed)"},{"line_number":18,"context_line":"and code should be submitted, referencing the RFE bug."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Here is a list of steps to do during the new process of adding functional to"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_ba73c67f","line":17,"range":{"start_line":16,"start_character":69,"end_line":17,"end_character":4},"in_reply_to":"3a7e1126_f9d403be","updated":"2015-12-17 16:29:53.000000000","message":"Done","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5feca8f5deda5aa127471b04f2ef20b1277fd4bb","unresolved":false,"context_lines":[{"line_number":15,"context_line":"feature enhancements instead of blueprints. These are bugs with \u0027rfe\u0027 tag, and"},{"line_number":16,"context_line":"they should be submitted before a spec or code is proposed. When the drivers"},{"line_number":17,"context_line":"team will decide that the proposal is worth implementing, a spec (if needed)"},{"line_number":18,"context_line":"and code should be submitted, referencing the RFE bug."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Here is a list of steps to do during the new process of adding functional to"},{"line_number":21,"context_line":"Ironic:"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_8be32633","line":18,"updated":"2015-12-17 15:09:55.000000000","message":"Instead of this last sentence...\n\nMaybe we should copy/paste some of the stuff at https://wiki.openstack.org/wiki/Ironic/Specs_Process.","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"8a94895c63495f1652901adf7a4a6ca0b9dd4fa9","unresolved":false,"context_lines":[{"line_number":15,"context_line":"feature enhancements instead of blueprints. These are bugs with \u0027rfe\u0027 tag, and"},{"line_number":16,"context_line":"they should be submitted before a spec or code is proposed. When the drivers"},{"line_number":17,"context_line":"team will decide that the proposal is worth implementing, a spec (if needed)"},{"line_number":18,"context_line":"and code should be submitted, referencing the RFE bug."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Here is a list of steps to do during the new process of adding functional to"},{"line_number":21,"context_line":"Ironic:"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_ba78a6c5","line":18,"in_reply_to":"3a7e1126_8be32633","updated":"2015-12-17 16:29:53.000000000","message":"Or maybe just add a reference to it?","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":10697,"name":"Thiago Paiva Brito","email":"thiagop@lsd.ufcg.edu.br","username":"thiagop","inactive":true},"change_message_id":"eba75401dea07d30bfd7c1b9970065166c7e5ab3","unresolved":false,"context_lines":[{"line_number":17,"context_line":"team will decide that the proposal is worth implementing, a spec (if needed)"},{"line_number":18,"context_line":"and code should be submitted, referencing the RFE bug."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Here is a list of steps to do during the new process of adding functional to"},{"line_number":21,"context_line":"Ironic:"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"#. Submit a bug report at https://bugs.launchpad.net/ironic/+filebug."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_d99fc701","line":20,"range":{"start_line":20,"start_character":63,"end_line":20,"end_character":73},"updated":"2015-12-17 15:07:26.000000000","message":"s/functional/feature/","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5feca8f5deda5aa127471b04f2ef20b1277fd4bb","unresolved":false,"context_lines":[{"line_number":17,"context_line":"team will decide that the proposal is worth implementing, a spec (if needed)"},{"line_number":18,"context_line":"and code should be submitted, referencing the RFE bug."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Here is a list of steps to do during the new process of adding functional to"},{"line_number":21,"context_line":"Ironic:"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"#. Submit a bug report at https://bugs.launchpad.net/ironic/+filebug."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_4b621eaf","line":20,"updated":"2015-12-17 15:09:55.000000000","message":"s/new //\ns/functional/a new feature/ ?","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"8a94895c63495f1652901adf7a4a6ca0b9dd4fa9","unresolved":false,"context_lines":[{"line_number":17,"context_line":"team will decide that the proposal is worth implementing, a spec (if needed)"},{"line_number":18,"context_line":"and code should be submitted, referencing the RFE bug."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Here is a list of steps to do during the new process of adding functional to"},{"line_number":21,"context_line":"Ironic:"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"#. Submit a bug report at https://bugs.launchpad.net/ironic/+filebug."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_7f30f001","line":20,"in_reply_to":"3a7e1126_4b621eaf","updated":"2015-12-17 16:29:53.000000000","message":"Done","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5feca8f5deda5aa127471b04f2ef20b1277fd4bb","unresolved":false,"context_lines":[{"line_number":34,"context_line":"   to be addressed, or what is the enhancement that would make the platform"},{"line_number":35,"context_line":"   more capable, both for a functional and a non-functional standpoint."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"#. Submit the bug and add an \u0027rfe\u0027 tag to it."},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"#. As soon as a member of the ironic-drivers team acknowledges the bug, it"},{"line_number":40,"context_line":"   will be moved into the \u0027Confirmed\u0027 state. The importance will be set to"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_cb504e61","line":37,"updated":"2015-12-17 15:09:55.000000000","message":"Also, \u0027Importance\u0027 \u003d \u0027Wishlist\u0027, \u0027Assign to\u0027 \u003d themselves.","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"8a94895c63495f1652901adf7a4a6ca0b9dd4fa9","unresolved":false,"context_lines":[{"line_number":34,"context_line":"   to be addressed, or what is the enhancement that would make the platform"},{"line_number":35,"context_line":"   more capable, both for a functional and a non-functional standpoint."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"#. Submit the bug and add an \u0027rfe\u0027 tag to it."},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"#. As soon as a member of the ironic-drivers team acknowledges the bug, it"},{"line_number":40,"context_line":"   will be moved into the \u0027Confirmed\u0027 state. The importance will be set to"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_1f3f6cd7","line":37,"in_reply_to":"3a7e1126_cb504e61","updated":"2015-12-17 16:29:53.000000000","message":"Done","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"4da31da1057692ba80fee96d386a5a6c8ba3d9e3","unresolved":false,"context_lines":[{"line_number":36,"context_line":""},{"line_number":37,"context_line":"#. Submit the bug and add an \u0027rfe\u0027 tag to it."},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"#. As soon as a member of the ironic-drivers team acknowledges the bug, it"},{"line_number":40,"context_line":"   will be moved into the \u0027Confirmed\u0027 state. The importance will be set to"},{"line_number":41,"context_line":"   \u0027Wishlist\u0027 to signal the fact that the report is indeed a feature and there"},{"line_number":42,"context_line":"   is no severity associated to it."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_6b1b5a1c","line":39,"updated":"2015-12-17 15:05:51.000000000","message":"Not about this change, but ironic-drivers needs to be updated :)\n\nAlso, currently any member of the ironic-bugs team can edit bugs, and membership to that is open. Do we want to change that? /cc dmitry","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"4da31da1057692ba80fee96d386a5a6c8ba3d9e3","unresolved":false,"context_lines":[{"line_number":36,"context_line":""},{"line_number":37,"context_line":"#. Submit the bug and add an \u0027rfe\u0027 tag to it."},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"#. As soon as a member of the ironic-drivers team acknowledges the bug, it"},{"line_number":40,"context_line":"   will be moved into the \u0027Confirmed\u0027 state. The importance will be set to"},{"line_number":41,"context_line":"   \u0027Wishlist\u0027 to signal the fact that the report is indeed a feature and there"},{"line_number":42,"context_line":"   is no severity associated to it."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_6bf27ad4","line":39,"updated":"2015-12-17 15:05:51.000000000","message":"Not about this change, but ironic-drivers needs to be updated :) https://launchpad.net/~ironic-drivers/+members#active\n\nAlso, currently any member of the ironic-bugs team can edit bugs, and membership to that is open. Do we want to change that? /cc dmitry","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5feca8f5deda5aa127471b04f2ef20b1277fd4bb","unresolved":false,"context_lines":[{"line_number":36,"context_line":""},{"line_number":37,"context_line":"#. Submit the bug and add an \u0027rfe\u0027 tag to it."},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"#. As soon as a member of the ironic-drivers team acknowledges the bug, it"},{"line_number":40,"context_line":"   will be moved into the \u0027Confirmed\u0027 state. The importance will be set to"},{"line_number":41,"context_line":"   \u0027Wishlist\u0027 to signal the fact that the report is indeed a feature and there"},{"line_number":42,"context_line":"   is no severity associated to it."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_4b99be1e","line":39,"updated":"2015-12-17 15:09:55.000000000","message":"We should discuss this. I was wondering when/how the status will change. Can anyone change the status?\n\nIs it only ironic-cores, or ironic-spec-cores or ???","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"8a94895c63495f1652901adf7a4a6ca0b9dd4fa9","unresolved":false,"context_lines":[{"line_number":36,"context_line":""},{"line_number":37,"context_line":"#. Submit the bug and add an \u0027rfe\u0027 tag to it."},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"#. As soon as a member of the ironic-drivers team acknowledges the bug, it"},{"line_number":40,"context_line":"   will be moved into the \u0027Confirmed\u0027 state. The importance will be set to"},{"line_number":41,"context_line":"   \u0027Wishlist\u0027 to signal the fact that the report is indeed a feature and there"},{"line_number":42,"context_line":"   is no severity associated to it."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_1a80da45","line":39,"in_reply_to":"3a7e1126_4b99be1e","updated":"2015-12-17 16:29:53.000000000","message":"Yes, right now it is everyone in ironic-bugs team who can change any bugs, rfe or not, this needs to be changed somehow. Is it possible to give rights to change status of bugs with rfe tag only to drivers?","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5feca8f5deda5aa127471b04f2ef20b1277fd4bb","unresolved":false,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"#. As soon as a member of the ironic-drivers team acknowledges the bug, it"},{"line_number":40,"context_line":"   will be moved into the \u0027Confirmed\u0027 state. The importance will be set to"},{"line_number":41,"context_line":"   \u0027Wishlist\u0027 to signal the fact that the report is indeed a feature and there"},{"line_number":42,"context_line":"   is no severity associated to it."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"#. The bug goes into the \u0027Triaged\u0027 state while the discussion is ongoing."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_0b7176e0","line":41,"updated":"2015-12-17 15:09:55.000000000","message":"Oh. Guess that\u0027s OK.","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5feca8f5deda5aa127471b04f2ef20b1277fd4bb","unresolved":false,"context_lines":[{"line_number":41,"context_line":"   \u0027Wishlist\u0027 to signal the fact that the report is indeed a feature and there"},{"line_number":42,"context_line":"   is no severity associated to it."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"#. The bug goes into the \u0027Triaged\u0027 state while the discussion is ongoing."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"#. The ironic-drivers team will evaluate the RFE and may advise the submitter"},{"line_number":47,"context_line":"   to file a spec in ironic-specs to elaborate on the feature request, in case"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_cb5a8e50","line":44,"updated":"2015-12-17 15:09:55.000000000","message":"What discussion? I guess we need to figure out who/how it is known if a spec is needed.","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"8a94895c63495f1652901adf7a4a6ca0b9dd4fa9","unresolved":false,"context_lines":[{"line_number":41,"context_line":"   \u0027Wishlist\u0027 to signal the fact that the report is indeed a feature and there"},{"line_number":42,"context_line":"   is no severity associated to it."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"#. The bug goes into the \u0027Triaged\u0027 state while the discussion is ongoing."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"#. The ironic-drivers team will evaluate the RFE and may advise the submitter"},{"line_number":47,"context_line":"   to file a spec in ironic-specs to elaborate on the feature request, in case"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_f5b63f89","line":44,"in_reply_to":"3a7e1126_cb5a8e50","updated":"2015-12-17 16:29:53.000000000","message":"Just in bug comments I guess.","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5feca8f5deda5aa127471b04f2ef20b1277fd4bb","unresolved":false,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"#. The bug goes into the \u0027Triaged\u0027 state while the discussion is ongoing."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"#. The ironic-drivers team will evaluate the RFE and may advise the submitter"},{"line_number":47,"context_line":"   to file a spec in ironic-specs to elaborate on the feature request, in case"},{"line_number":48,"context_line":"   the RFE requires extra scrutiny, more design discussion, etc."},{"line_number":49,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_4b1c5e77","line":46,"updated":"2015-12-17 15:09:55.000000000","message":"ironic-specs core team?","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"8a94895c63495f1652901adf7a4a6ca0b9dd4fa9","unresolved":false,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"#. The bug goes into the \u0027Triaged\u0027 state while the discussion is ongoing."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"#. The ironic-drivers team will evaluate the RFE and may advise the submitter"},{"line_number":47,"context_line":"   to file a spec in ironic-specs to elaborate on the feature request, in case"},{"line_number":48,"context_line":"   the RFE requires extra scrutiny, more design discussion, etc."},{"line_number":49,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_bf8ad8bd","line":46,"in_reply_to":"3a7e1126_4b1c5e77","updated":"2015-12-17 16:29:53.000000000","message":"This should be set in launchpad, and there is ironic-drivers team already, so it seems that we should give the ability to change rfes only to ironic-drivers?","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5feca8f5deda5aa127471b04f2ef20b1277fd4bb","unresolved":false,"context_lines":[{"line_number":45,"context_line":""},{"line_number":46,"context_line":"#. The ironic-drivers team will evaluate the RFE and may advise the submitter"},{"line_number":47,"context_line":"   to file a spec in ironic-specs to elaborate on the feature request, in case"},{"line_number":48,"context_line":"   the RFE requires extra scrutiny, more design discussion, etc."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"#. Once the discussion has happened and there is positive consensus on the RFE,"},{"line_number":51,"context_line":"   the report is \u0027approved\u0027, and the bug goes through the usual states -- first"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_2e4ec06f","line":48,"updated":"2015-12-17 15:09:55.000000000","message":"The problem with this process is unless we can guarantee that some decision will be made w/i some time frame about whether a spec is needed or not, folks will be stuck waiting 1. for this decision to be made and 2. for spec to be approved, before actually coding.","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"8a94895c63495f1652901adf7a4a6ca0b9dd4fa9","unresolved":false,"context_lines":[{"line_number":45,"context_line":""},{"line_number":46,"context_line":"#. The ironic-drivers team will evaluate the RFE and may advise the submitter"},{"line_number":47,"context_line":"   to file a spec in ironic-specs to elaborate on the feature request, in case"},{"line_number":48,"context_line":"   the RFE requires extra scrutiny, more design discussion, etc."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"#. Once the discussion has happened and there is positive consensus on the RFE,"},{"line_number":51,"context_line":"   the report is \u0027approved\u0027, and the bug goes through the usual states -- first"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_df500461","line":48,"in_reply_to":"3a7e1126_2e4ec06f","updated":"2015-12-17 16:29:53.000000000","message":"Yep, we should state that rfe bugs should be watched on. As for coding before the spec, I think it can be allowed, you can always put -2 :)","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"4da31da1057692ba80fee96d386a5a6c8ba3d9e3","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"#. Once the discussion has happened and there is positive consensus on the RFE,"},{"line_number":51,"context_line":"   the report is \u0027approved\u0027, and the bug goes through the usual states -- first"},{"line_number":52,"context_line":"   to \u0027In progress\u0027 when the spec/code is being worked on, then \u0027Fix Committed\u0027"},{"line_number":53,"context_line":"   when it is implemented. Drivers team may also move the bug to \"Won\u0027t Fix\""},{"line_number":54,"context_line":"   status, meaning that RFE is rejected."},{"line_number":55,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_cb3b6e37","line":52,"updated":"2015-12-17 15:05:51.000000000","message":"Nit: bugs are now set to \"fix released\" when the commit is merged.","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"8a94895c63495f1652901adf7a4a6ca0b9dd4fa9","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"#. Once the discussion has happened and there is positive consensus on the RFE,"},{"line_number":51,"context_line":"   the report is \u0027approved\u0027, and the bug goes through the usual states -- first"},{"line_number":52,"context_line":"   to \u0027In progress\u0027 when the spec/code is being worked on, then \u0027Fix Committed\u0027"},{"line_number":53,"context_line":"   when it is implemented. Drivers team may also move the bug to \"Won\u0027t Fix\""},{"line_number":54,"context_line":"   status, meaning that RFE is rejected."},{"line_number":55,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_1f10cca7","line":52,"in_reply_to":"3a7e1126_cb3b6e37","updated":"2015-12-17 16:29:53.000000000","message":"Done","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":10697,"name":"Thiago Paiva Brito","email":"thiagop@lsd.ufcg.edu.br","username":"thiagop","inactive":true},"change_message_id":"eba75401dea07d30bfd7c1b9970065166c7e5ab3","unresolved":false,"context_lines":[{"line_number":47,"context_line":"   to file a spec in ironic-specs to elaborate on the feature request, in case"},{"line_number":48,"context_line":"   the RFE requires extra scrutiny, more design discussion, etc."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"#. Once the discussion has happened and there is positive consensus on the RFE,"},{"line_number":51,"context_line":"   the report is \u0027approved\u0027, and the bug goes through the usual states -- first"},{"line_number":52,"context_line":"   to \u0027In progress\u0027 when the spec/code is being worked on, then \u0027Fix Committed\u0027"},{"line_number":53,"context_line":"   when it is implemented. Drivers team may also move the bug to \"Won\u0027t Fix\""},{"line_number":54,"context_line":"   status, meaning that RFE is rejected."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Live Upgrade Related Concerns"},{"line_number":57,"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"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_4b169ea8","line":54,"range":{"start_line":50,"start_character":0,"end_line":54,"end_character":40},"updated":"2015-12-17 15:07:26.000000000","message":"I know that probably you discussed that on the meeting or in the ML, but I think adding another step to contribute code is getting the process very bureaucratic. RFE-\u003eblueprint-\u003espec-\u003ecode, wow...\n\nMakes me wonder if we do really need that...","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5feca8f5deda5aa127471b04f2ef20b1277fd4bb","unresolved":false,"context_lines":[{"line_number":51,"context_line":"   the report is \u0027approved\u0027, and the bug goes through the usual states -- first"},{"line_number":52,"context_line":"   to \u0027In progress\u0027 when the spec/code is being worked on, then \u0027Fix Committed\u0027"},{"line_number":53,"context_line":"   when it is implemented. Drivers team may also move the bug to \"Won\u0027t Fix\""},{"line_number":54,"context_line":"   status, meaning that RFE is rejected."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Live Upgrade Related Concerns"},{"line_number":57,"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"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_6ee84811","line":54,"updated":"2015-12-17 15:09:55.000000000","message":"Need to get rid of all \u0027drivers team\u0027 :)","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"8a94895c63495f1652901adf7a4a6ca0b9dd4fa9","unresolved":false,"context_lines":[{"line_number":47,"context_line":"   to file a spec in ironic-specs to elaborate on the feature request, in case"},{"line_number":48,"context_line":"   the RFE requires extra scrutiny, more design discussion, etc."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"#. Once the discussion has happened and there is positive consensus on the RFE,"},{"line_number":51,"context_line":"   the report is \u0027approved\u0027, and the bug goes through the usual states -- first"},{"line_number":52,"context_line":"   to \u0027In progress\u0027 when the spec/code is being worked on, then \u0027Fix Committed\u0027"},{"line_number":53,"context_line":"   when it is implemented. Drivers team may also move the bug to \"Won\u0027t Fix\""},{"line_number":54,"context_line":"   status, meaning that RFE is rejected."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Live Upgrade Related Concerns"},{"line_number":57,"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"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_ff35c045","line":54,"range":{"start_line":50,"start_character":0,"end_line":54,"end_character":40},"in_reply_to":"3a7e1126_4b169ea8","updated":"2015-12-17 16:29:53.000000000","message":"Thiago, actually there is no blueprint step now, we move from bp to bugs :)","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"4da31da1057692ba80fee96d386a5a6c8ba3d9e3","unresolved":false,"context_lines":[{"line_number":52,"context_line":"   to \u0027In progress\u0027 when the spec/code is being worked on, then \u0027Fix Committed\u0027"},{"line_number":53,"context_line":"   when it is implemented. Drivers team may also move the bug to \"Won\u0027t Fix\""},{"line_number":54,"context_line":"   status, meaning that RFE is rejected."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Live Upgrade Related Concerns"},{"line_number":57,"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"},{"line_number":58,"context_line":"Ironic implements upgrade with the same methodology of Nova:"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_4bf21e71","line":55,"updated":"2015-12-17 15:05:51.000000000","message":"Can we add something about commit messages and review topics here? Something like:\n\n#. Please be sure to tag your commits properly; \"Related-Bug: #xxxx\" for intermediate commits for the feature, and \"Closes-Bug: #xxxx\" for the final commit. It is also helpful to set a consistent review topic, such as \"bug/xxxx\" for all patches related to the RFE.","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"8a94895c63495f1652901adf7a4a6ca0b9dd4fa9","unresolved":false,"context_lines":[{"line_number":52,"context_line":"   to \u0027In progress\u0027 when the spec/code is being worked on, then \u0027Fix Committed\u0027"},{"line_number":53,"context_line":"   when it is implemented. Drivers team may also move the bug to \"Won\u0027t Fix\""},{"line_number":54,"context_line":"   status, meaning that RFE is rejected."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Live Upgrade Related Concerns"},{"line_number":57,"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"},{"line_number":58,"context_line":"Ironic implements upgrade with the same methodology of Nova:"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a7e1126_bf4098a6","line":55,"in_reply_to":"3a7e1126_4bf21e71","updated":"2015-12-17 16:29:53.000000000","message":"Done","commit_id":"c9b56e2cf74a4651fa4bf4180de2ce92ea35af3e"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"4f690f1b6dd4ad035cffa1b7a7979d0df30d6fb6","unresolved":false,"context_lines":[{"line_number":50,"context_line":"   to file a spec in ironic-specs to elaborate on the feature request, in case"},{"line_number":51,"context_line":"   the RFE requires extra scrutiny, more design discussion, etc. For the spec"},{"line_number":52,"context_line":"   submission process, please see the"},{"line_number":53,"context_line":"   `specs process \u003chttps://wiki.openstack.org/wiki/Ironic/Specs_Process\u003e`_"},{"line_number":54,"context_line":"   wiki page."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"#. Once the discussion has happened and there is positive consensus on the RFE,"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a7e1126_5035e1f4","line":53,"updated":"2015-12-17 16:32:28.000000000","message":"I\u0027ll update the wiki when there is some consensus on the process.","commit_id":"b0975830c50df5fd44bb5e709b57350a426bc6c0"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"90783bd0fa6c6c1bc9a7eed8ac09517fb2fc6b26","unresolved":false,"context_lines":[{"line_number":62,"context_line":"When working on RFE, please be sure to tag your commits properly:"},{"line_number":63,"context_line":"\"Partial-Bug: #xxxx\" for intermediate commits for the feature, and"},{"line_number":64,"context_line":"\"Closes-Bug: #xxxx\" for the final commit. It is also helpful to set a"},{"line_number":65,"context_line":"consistent review topic, such as \"bug/xxxx\" for all patches related to the RFE."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"Live Upgrade Related Concerns"},{"line_number":68,"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"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a7e1126_d9049d7f","line":65,"updated":"2015-12-17 16:58:54.000000000","message":"I have a patch to python-ironicclient, for a RFE bug, so I used the RFE bug even though it is in ironic. Maybe we should mention that (unless someone thinks we should open bugs in each project, please do not think that!)","commit_id":"b0975830c50df5fd44bb5e709b57350a426bc6c0"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"3c25fe9c5830b441765be3e7923129b05901bf5e","unresolved":false,"context_lines":[{"line_number":62,"context_line":"When working on RFE, please be sure to tag your commits properly:"},{"line_number":63,"context_line":"\"Partial-Bug: #xxxx\" for intermediate commits for the feature, and"},{"line_number":64,"context_line":"\"Closes-Bug: #xxxx\" for the final commit. It is also helpful to set a"},{"line_number":65,"context_line":"consistent review topic, such as \"bug/xxxx\" for all patches related to the RFE."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"Live Upgrade Related Concerns"},{"line_number":68,"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"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a7e1126_5fa40da5","line":65,"in_reply_to":"3a7e1126_d9049d7f","updated":"2015-12-17 17:26:49.000000000","message":"Done","commit_id":"b0975830c50df5fd44bb5e709b57350a426bc6c0"},{"author":{"_account_id":7711,"name":"Yuriy Zveryanskyy","email":"yzveryanskyy@mirantis.com","username":"yuriyz"},"change_message_id":"5514944333ff2c1688df0566d21ad7645c4baba8","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Starting with the Mitaka development cycle, Ironic tracks new features using"},{"line_number":15,"context_line":"RFEs (Requests for Feature Enhancements) instead of blueprints. These are bugs"},{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_e6717f45","line":17,"updated":"2015-12-18 15:03:06.000000000","message":"Agree with Ruby, \"Need to get rid of all \u0027drivers team\u0027 :)\" or at least mention that it\u0027s true info :) because this can confusing people.","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"22048f1b2908ca275e7d27d3ea8fb9f7149108e4","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Starting with the Mitaka development cycle, Ironic tracks new features using"},{"line_number":15,"context_line":"RFEs (Requests for Feature Enhancements) instead of blueprints. These are bugs"},{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_a0cc77bb","line":17,"in_reply_to":"3a7e1126_00622bd8","updated":"2015-12-21 14:04:07.000000000","message":"there\u0027s no spec at the moment of RFE submission (and it might not appear for some time until someone picks the work). and we still need to ack/nack the feature request.","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"85c01c918f3070091c6a0a2da8d62fd192a5ce47","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Starting with the Mitaka development cycle, Ironic tracks new features using"},{"line_number":15,"context_line":"RFEs (Requests for Feature Enhancements) instead of blueprints. These are bugs"},{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_91b8b376","line":17,"in_reply_to":"3a7e1126_31ce9f63","updated":"2015-12-21 12:40:18.000000000","message":"Yeah I agree ironic-drivers is wrong.","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"7d59002110f78b97d01be7f8a7b357fcd76d1325","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Starting with the Mitaka development cycle, Ironic tracks new features using"},{"line_number":15,"context_line":"RFEs (Requests for Feature Enhancements) instead of blueprints. These are bugs"},{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_9ad69526","line":17,"in_reply_to":"3a7e1126_49c64d69","updated":"2015-12-21 09:57:25.000000000","message":"Yep, e.g. here https://launchpad.net/ironic you can see that ironic-drivers is both driver and maintainer. I\u0027ll add this as a separate paragraph if something else needs to be fixed or in a followup.","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ecee8f87f35318bea774b13db240bca80c4e6567","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Starting with the Mitaka development cycle, Ironic tracks new features using"},{"line_number":15,"context_line":"RFEs (Requests for Feature Enhancements) instead of blueprints. These are bugs"},{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_feda2616","line":17,"in_reply_to":"3a7e1126_7b3ac8e8","updated":"2015-12-21 12:19:07.000000000","message":"The text implies ironic-bugs right now :) if we want to limit people who can do it, we should start moderating this group, as this particular people can triage bugs. Our core team is only about gerrit right now.","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":7711,"name":"Yuriy Zveryanskyy","email":"yzveryanskyy@mirantis.com","username":"yuriyz"},"change_message_id":"72bfe31071cf5448a96758283468505d8d74f421","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Starting with the Mitaka development cycle, Ironic tracks new features using"},{"line_number":15,"context_line":"RFEs (Requests for Feature Enhancements) instead of blueprints. These are bugs"},{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_49c64d69","line":17,"in_reply_to":"3a7e1126_8bae747f","updated":"2015-12-19 07:48:27.000000000","message":"Looks strange that team with name \"ironic-drivers\" works with all RFEs, but OK if it\u0027s widely used practice.","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":7711,"name":"Yuriy Zveryanskyy","email":"yzveryanskyy@mirantis.com","username":"yuriyz"},"change_message_id":"c12aefbcea793b76436b1e0e952bc67098040b0b","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Starting with the Mitaka development cycle, Ironic tracks new features using"},{"line_number":15,"context_line":"RFEs (Requests for Feature Enhancements) instead of blueprints. These are bugs"},{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_00622bd8","line":17,"in_reply_to":"3a7e1126_910e738a","updated":"2015-12-21 14:02:02.000000000","message":"\"Maybe we don\u0027t need to decide anything in the RFE bug ticket, leave the approval/disapproval to be done in gerrit as part of the spec (or if no spec is needed just decide in the code review).\" - Strongly agree with Lucas. Reasons: 1) the spec (or short spec as an alternative) can contains detailed description which help people to make agree/disagree decision. 2) RFE bug approval from limited team creates a new step that causes delay for developing process as result.","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"ce89aec470de3e594bdb60357b0e46c968c137a7","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Starting with the Mitaka development cycle, Ironic tracks new features using"},{"line_number":15,"context_line":"RFEs (Requests for Feature Enhancements) instead of blueprints. These are bugs"},{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_910e738a","line":17,"in_reply_to":"3a7e1126_91ad139d","updated":"2015-12-21 12:38:55.000000000","message":"I mean, if anyone can do it why there\u0027s a need for approval ?\n\nMaybe we don\u0027t need to decide anything in the RFE bug ticket, leave the approval/disapproval to be done in gerrit as part of the spec (or if no spec is needed just decide in the code review).","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"000fb97fd80c7a145641f7a1169584291a58b69f","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Starting with the Mitaka development cycle, Ironic tracks new features using"},{"line_number":15,"context_line":"RFEs (Requests for Feature Enhancements) instead of blueprints. These are bugs"},{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_31ce9f63","line":17,"in_reply_to":"3a7e1126_91ad139d","updated":"2015-12-21 12:34:31.000000000","message":"We should not recommend it, but with the current ACL we can\u0027t enforce it. Anyway the text here is not correct: the majority of us is not in ironic-drivers","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"66be01e6ddf56fe4f12e91ea66eb5d765d07891d","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Starting with the Mitaka development cycle, Ironic tracks new features using"},{"line_number":15,"context_line":"RFEs (Requests for Feature Enhancements) instead of blueprints. These are bugs"},{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_bd64abb8","line":17,"in_reply_to":"3a7e1126_9ad69526","updated":"2015-12-21 10:14:52.000000000","message":"s/ironic-drivers/ironic-bugs/\n\nironic-drivers team is a tiny team managing our launchpad page settings, it\u0027s not related to the bugs triaging (I personally am not part of ironic-drivers, for example)","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"6663b684a58f5216b042c81876c6dc41f1f4b9ad","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Starting with the Mitaka development cycle, Ironic tracks new features using"},{"line_number":15,"context_line":"RFEs (Requests for Feature Enhancements) instead of blueprints. These are bugs"},{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_00030bb9","line":17,"in_reply_to":"3a7e1126_a0cc77bb","updated":"2015-12-21 14:13:55.000000000","message":"Dmitry, +1.\n\nWe should use the ironic-drivers group. I need to get admin access to that group, and then I\u0027ll make it the same as spec cores.","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"03ac93b464af4c0cecd4ea43a96a53d190fc99a5","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Starting with the Mitaka development cycle, Ironic tracks new features using"},{"line_number":15,"context_line":"RFEs (Requests for Feature Enhancements) instead of blueprints. These are bugs"},{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_7b3ac8e8","line":17,"in_reply_to":"3a7e1126_bd64abb8","updated":"2015-12-21 12:13:56.000000000","message":"Anyone can join ironic-bugs. I think that the text means \"Ironic core team\"","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"edfc8cea78fbc8e8c6a26b23dd83d9fe4aab8542","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Starting with the Mitaka development cycle, Ironic tracks new features using"},{"line_number":15,"context_line":"RFEs (Requests for Feature Enhancements) instead of blueprints. These are bugs"},{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_8bae747f","line":17,"in_reply_to":"3a7e1126_e6717f45","updated":"2015-12-18 21:12:31.000000000","message":"Hi Yuriy, not sure what you mean - to add a new paragraph describing who are the members of ironic-drivers and what they should do wrt RFEs?","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"46501482ff9db053e990262a0da8d68d77991bc6","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Starting with the Mitaka development cycle, Ironic tracks new features using"},{"line_number":15,"context_line":"RFEs (Requests for Feature Enhancements) instead of blueprints. These are bugs"},{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_91ad139d","line":17,"in_reply_to":"3a7e1126_feda2616","updated":"2015-12-21 12:33:24.000000000","message":"I think it\u0027s great that anyone can join ironic-bugs and triage bugs, I wouldn\u0027t change that.\n\nBut I don\u0027t think the text means that anyone can decide that the proposal is worth implementing and is should to tell the author of the RFE to go ahead and submit a spec+code for it.","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},"change_message_id":"41030939fa810ada8f2050a806d5d37d11b328a5","unresolved":false,"context_lines":[{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Here is a list of steps to do during the new process of adding a new feature to"},{"line_number":22,"context_line":"Ironic:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_c5b995cd","line":19,"updated":"2015-12-21 13:54:15.000000000","message":"From the sentence, it makes me feel that before submit a spec, the RFEs has to be triaged or accepted. But in BP era, people usually only submit a BP and reference it on the spec. Do we change the process? I prefer to discuss in spec rather than in bug pages. And I think RFEs should only track the patches related to the topic.","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"6663b684a58f5216b042c81876c6dc41f1f4b9ad","unresolved":false,"context_lines":[{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Here is a list of steps to do during the new process of adding a new feature to"},{"line_number":22,"context_line":"Ironic:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_60cd2f84","line":19,"in_reply_to":"3a7e1126_c5b995cd","updated":"2015-12-21 14:13:55.000000000","message":"If someone knows it will need a spec, they may submit the spec right away. If they don\u0027t think it needs a spec, the team will review the RFE and decide if it needs one.","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"66be01e6ddf56fe4f12e91ea66eb5d765d07891d","unresolved":false,"context_lines":[{"line_number":35,"context_line":"   to be addressed, or what is the enhancement that would make the platform"},{"line_number":36,"context_line":"   more capable, both for a functional and a non-functional standpoint."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"#. Submit the bug, add an \u0027rfe\u0027 tag to it and assign yourself or whoever is"},{"line_number":39,"context_line":"   going to work on this feature."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"#. As soon as a member of the ironic-drivers team acknowledges the bug, it"},{"line_number":42,"context_line":"   will be moved into the \u0027Confirmed\u0027 state. The importance will be set to"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_bd0b4bd5","line":39,"range":{"start_line":38,"start_character":3,"end_line":39,"end_character":33},"updated":"2015-12-21 10:14:52.000000000","message":"this can be done before bug is created under \"more options\" link","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"66be01e6ddf56fe4f12e91ea66eb5d765d07891d","unresolved":false,"context_lines":[{"line_number":38,"context_line":"#. Submit the bug, add an \u0027rfe\u0027 tag to it and assign yourself or whoever is"},{"line_number":39,"context_line":"   going to work on this feature."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"#. As soon as a member of the ironic-drivers team acknowledges the bug, it"},{"line_number":42,"context_line":"   will be moved into the \u0027Confirmed\u0027 state. The importance will be set to"},{"line_number":43,"context_line":"   \u0027Wishlist\u0027 to signal the fact that the report is indeed a feature and there"},{"line_number":44,"context_line":"   is no severity associated to it."}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_3df8bbc2","line":41,"range":{"start_line":41,"start_character":30,"end_line":41,"end_character":44},"updated":"2015-12-21 10:14:52.000000000","message":"ditto about team","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"66be01e6ddf56fe4f12e91ea66eb5d765d07891d","unresolved":false,"context_lines":[{"line_number":43,"context_line":"   \u0027Wishlist\u0027 to signal the fact that the report is indeed a feature and there"},{"line_number":44,"context_line":"   is no severity associated to it."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"#. The bug goes into the \u0027Triaged\u0027 state while the discussion is ongoing, this"},{"line_number":47,"context_line":"   discussion should happen in bug comments."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"#. The ironic-drivers team will evaluate the RFE and may advise the submitter"},{"line_number":50,"context_line":"   to file a spec in ironic-specs to elaborate on the feature request, in case"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_5d258759","line":47,"range":{"start_line":46,"start_character":3,"end_line":47,"end_character":44},"updated":"2015-12-21 10:14:52.000000000","message":"this is not clear to me. I though the bug goes to Triaged as soon as we agree on doing it in the near future and/or approve the spec","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"03ac93b464af4c0cecd4ea43a96a53d190fc99a5","unresolved":false,"context_lines":[{"line_number":43,"context_line":"   \u0027Wishlist\u0027 to signal the fact that the report is indeed a feature and there"},{"line_number":44,"context_line":"   is no severity associated to it."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"#. The bug goes into the \u0027Triaged\u0027 state while the discussion is ongoing, this"},{"line_number":47,"context_line":"   discussion should happen in bug comments."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"#. The ironic-drivers team will evaluate the RFE and may advise the submitter"},{"line_number":50,"context_line":"   to file a spec in ironic-specs to elaborate on the feature request, in case"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_fe6b0683","line":47,"range":{"start_line":46,"start_character":3,"end_line":47,"end_character":44},"in_reply_to":"3a7e1126_5d258759","updated":"2015-12-21 12:13:56.000000000","message":"As stated on the comment at L58. The but state will move to \"In Progress\" as soon as people start submitting code to it (or the spec the match the RFE if the bug tag is used). Maybe we should not rely on the bug states at all for RFE? (Apart from fix merged, fix released)\n\nI think it\u0027s a good thing to be able to submit the code for the spec before it\u0027s approved, I do that all the time because I prefer to write the code (or parts of it) as I write the spec, I usually find things I wouldn\u0027t find otherwise.","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"6663b684a58f5216b042c81876c6dc41f1f4b9ad","unresolved":false,"context_lines":[{"line_number":43,"context_line":"   \u0027Wishlist\u0027 to signal the fact that the report is indeed a feature and there"},{"line_number":44,"context_line":"   is no severity associated to it."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"#. The bug goes into the \u0027Triaged\u0027 state while the discussion is ongoing, this"},{"line_number":47,"context_line":"   discussion should happen in bug comments."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"#. The ironic-drivers team will evaluate the RFE and may advise the submitter"},{"line_number":50,"context_line":"   to file a spec in ironic-specs to elaborate on the feature request, in case"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_201ba7cb","line":47,"range":{"start_line":46,"start_character":3,"end_line":47,"end_character":44},"in_reply_to":"3a7e1126_fe6b0683","updated":"2015-12-21 14:13:55.000000000","message":"That\u0027s a good point - Neutron uses \u0027rfe\u0027 and \u0027rfe-approved\u0027 tags, maybe we should do the same?","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"66be01e6ddf56fe4f12e91ea66eb5d765d07891d","unresolved":false,"context_lines":[{"line_number":46,"context_line":"#. The bug goes into the \u0027Triaged\u0027 state while the discussion is ongoing, this"},{"line_number":47,"context_line":"   discussion should happen in bug comments."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"#. The ironic-drivers team will evaluate the RFE and may advise the submitter"},{"line_number":50,"context_line":"   to file a spec in ironic-specs to elaborate on the feature request, in case"},{"line_number":51,"context_line":"   the RFE requires extra scrutiny, more design discussion, etc. For the spec"},{"line_number":52,"context_line":"   submission process, please see the"},{"line_number":53,"context_line":"   `specs process \u003chttps://wiki.openstack.org/wiki/Ironic/Specs_Process\u003e`_"},{"line_number":54,"context_line":"   wiki page."}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_fd1473e9","line":51,"range":{"start_line":49,"start_character":3,"end_line":51,"end_character":62},"updated":"2015-12-21 10:14:52.000000000","message":"this happens before the bug goes to triaged","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"66be01e6ddf56fe4f12e91ea66eb5d765d07891d","unresolved":false,"context_lines":[{"line_number":55,"context_line":""},{"line_number":56,"context_line":"#. Once the discussion has happened and there is positive consensus on the RFE,"},{"line_number":57,"context_line":"   the report is \u0027approved\u0027, and the bug goes through the usual states -- first"},{"line_number":58,"context_line":"   to \u0027In progress\u0027 when the spec/code is being worked on, then \u0027Fix Released\u0027"},{"line_number":59,"context_line":"   when it is implemented. ironic-drivers team may also move the bug to"},{"line_number":60,"context_line":"   \"Won\u0027t Fix\" status, meaning that RFE is rejected."},{"line_number":61,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_3dcd1b85","line":58,"range":{"start_line":58,"start_character":6,"end_line":58,"end_character":57},"updated":"2015-12-21 10:14:52.000000000","message":"here we have a problem: if the spec is submitted at the same time, the bug will go to \"in progress\" right away, avoiding both confirmed and triaged states. maybe we need to ask folks NOT to link a spec patch to a bug.","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"6663b684a58f5216b042c81876c6dc41f1f4b9ad","unresolved":false,"context_lines":[{"line_number":55,"context_line":""},{"line_number":56,"context_line":"#. Once the discussion has happened and there is positive consensus on the RFE,"},{"line_number":57,"context_line":"   the report is \u0027approved\u0027, and the bug goes through the usual states -- first"},{"line_number":58,"context_line":"   to \u0027In progress\u0027 when the spec/code is being worked on, then \u0027Fix Released\u0027"},{"line_number":59,"context_line":"   when it is implemented. ironic-drivers team may also move the bug to"},{"line_number":60,"context_line":"   \"Won\u0027t Fix\" status, meaning that RFE is rejected."},{"line_number":61,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_a000f766","line":58,"range":{"start_line":58,"start_character":6,"end_line":58,"end_character":57},"in_reply_to":"3a7e1126_3dcd1b85","updated":"2015-12-21 14:13:55.000000000","message":"We can always change the status back, so I\u0027m not sure this is a huge deal.","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":10697,"name":"Thiago Paiva Brito","email":"thiagop@lsd.ufcg.edu.br","username":"thiagop","inactive":true},"change_message_id":"de425d07636312ef9569b57df5341b7e0afd17a2","unresolved":false,"context_lines":[{"line_number":67,"context_line":""},{"line_number":68,"context_line":"If the RFE spans across several projects (e.g. ironic and python-ironicclient),"},{"line_number":69,"context_line":"but the main work is going to happen within ironic, please use the same bug for"},{"line_number":70,"context_line":"all the code you\u0027re submitting, there is no need to create a separate RFE in"},{"line_number":71,"context_line":"every project."},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"Live Upgrade Related Concerns"},{"line_number":74,"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"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa69d971_4e451c6e","line":71,"range":{"start_line":70,"start_character":32,"end_line":71,"end_character":14},"updated":"2016-01-04 14:17:41.000000000","message":"Since a RFE is a bug, you can set more than one project that the bug affects. This could be useful to indicate RFE\u0027s that spans to more than one project.","commit_id":"8acd9b1dd07fb698c082d3dbc9878f9e89f2532a"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5dfc40737c1552385f15cae240e99966a717129f","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Starting with the Mitaka development cycle, Ironic tracks new features using"},{"line_number":15,"context_line":"RFEs (Requests for Feature Enhancements) instead of blueprints. These are bugs"},{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_c602e1b3","line":17,"updated":"2016-01-06 19:33:39.000000000","message":"is it worth putting a link to ironic-drivers: https://launchpad.net/~ironic-drivers/+members\n\nand/or mentioning that this includes the ironic-specs-core folks https://review.openstack.org/#/admin/groups/352,members? Not sure why we don\u0027t just use ironic-specs-core instead of ironic-drivers. (Can you add ironic-specs-core to ironic-drivers, instead of individuals, otherwise will need to update both groups when the cores change.)","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"49a8b662b182d8f2a70c2732ff99e7b26637aef0","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Starting with the Mitaka development cycle, Ironic tracks new features using"},{"line_number":15,"context_line":"RFEs (Requests for Feature Enhancements) instead of blueprints. These are bugs"},{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_c8ae642d","line":17,"in_reply_to":"fa69d971_c602e1b3","updated":"2016-01-06 22:40:26.000000000","message":"gerrit and launchpad groups are totally separate and can\u0027t be linked.\n\nI can put a link to the group on launchpad but I don\u0027t think it\u0027s terribly valuable, personally. If you\u0027re reviewing RFEs you should know how to do that.\n\nI intentionally did not link specs-core and ironic-drivers in this document, to leave us room to have those be different in the future if we want.","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"ff5021fdcf7d3661ab05bb8f2cb4c695f56c7cec","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Starting with the Mitaka development cycle, Ironic tracks new features using"},{"line_number":15,"context_line":"RFEs (Requests for Feature Enhancements) instead of blueprints. These are bugs"},{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_7b14e4e7","line":17,"in_reply_to":"fa69d971_c8ae642d","updated":"2016-01-06 22:56:37.000000000","message":"So \u0027ironic-drivers\u0027 doesn\u0027t mean much to me. Or it didn\u0027t, now I know. So I think it is worth mentioning/providing more info about it. (Initially i thought it had something to do with the drivers in ironic).\n\nIt isn\u0027t the core reviewers that I am concerned about (like you said, we know), it is newbies that read this to try to understand the process and what they should do and who they think have powers over their future :)\n\nSo maybe just put a link to ironic drivers: https://launchpad.net/~ironic-drivers/+members here.","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5dfc40737c1552385f15cae240e99966a717129f","unresolved":false,"context_lines":[{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Here is a list of steps to do during the new process of adding a new feature to"},{"line_number":22,"context_line":"Ironic:"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_86a4e9b9","line":19,"updated":"2016-01-06 19:33:39.000000000","message":"Tan mentioned it -- I think it is worth mentioning here that the submitter should feel free to submit a spec and/or code before anyone indicates whether it is worth implementing. Otherwise, people might be waiting for awhile...","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"49a8b662b182d8f2a70c2732ff99e7b26637aef0","unresolved":false,"context_lines":[{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of ironic-drivers launchpad team will decide that the proposal is"},{"line_number":18,"context_line":"worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":19,"context_line":"referencing the RFE bug."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Here is a list of steps to do during the new process of adding a new feature to"},{"line_number":22,"context_line":"Ironic:"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_68a97823","line":19,"in_reply_to":"fa69d971_86a4e9b9","updated":"2016-01-06 22:40:26.000000000","message":"Yeah +1","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5dfc40737c1552385f15cae240e99966a717129f","unresolved":false,"context_lines":[{"line_number":32,"context_line":"   description should provide enough details for a knowledgeable developer to"},{"line_number":33,"context_line":"   understand what is the existing problem in the current platform that needs"},{"line_number":34,"context_line":"   to be addressed, or what is the enhancement that would make the platform"},{"line_number":35,"context_line":"   more capable, both for a functional and a non-functional standpoint."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"#. Submit the bug, add an \u0027rfe\u0027 tag to it and assign yourself or whoever is"},{"line_number":38,"context_line":"   going to work on this feature."}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_a1a794a5","line":35,"updated":"2016-01-06 19:33:39.000000000","message":"s/for/from/","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"49a8b662b182d8f2a70c2732ff99e7b26637aef0","unresolved":false,"context_lines":[{"line_number":32,"context_line":"   description should provide enough details for a knowledgeable developer to"},{"line_number":33,"context_line":"   understand what is the existing problem in the current platform that needs"},{"line_number":34,"context_line":"   to be addressed, or what is the enhancement that would make the platform"},{"line_number":35,"context_line":"   more capable, both for a functional and a non-functional standpoint."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"#. Submit the bug, add an \u0027rfe\u0027 tag to it and assign yourself or whoever is"},{"line_number":38,"context_line":"   going to work on this feature."}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_88a4ec4a","line":35,"in_reply_to":"fa69d971_a1a794a5","updated":"2016-01-06 22:40:26.000000000","message":"Will do","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5dfc40737c1552385f15cae240e99966a717129f","unresolved":false,"context_lines":[{"line_number":38,"context_line":"   going to work on this feature."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"#. As soon as a member of the ironic-drivers team acknowledges the bug, it"},{"line_number":41,"context_line":"   will be moved into the \u0027Confirmed\u0027 state. The importance will be set to"},{"line_number":42,"context_line":"   \u0027Wishlist\u0027 to signal the fact that the report is indeed a feature and there"},{"line_number":43,"context_line":"   is no severity associated to it."},{"line_number":44,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_e6d5a5e4","line":41,"updated":"2016-01-06 19:33:39.000000000","message":"Can we set this up so that only ironic-drivers can move this into Confirmed? If I recall correctly, if someone submits a patch, the status will change to \u0027In progress\u0027. Do we need to look to check that someone Confirmed it?\n\nLet\u0027s be explicit here. \"Acknowledging\" means changing the status to \u0027Confirmed\u0027 or \u0027Invalid\u0027? Would we use \u0027Opinion\u0027 or \u0027Incomplete\u0027?","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"49a8b662b182d8f2a70c2732ff99e7b26637aef0","unresolved":false,"context_lines":[{"line_number":38,"context_line":"   going to work on this feature."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"#. As soon as a member of the ironic-drivers team acknowledges the bug, it"},{"line_number":41,"context_line":"   will be moved into the \u0027Confirmed\u0027 state. The importance will be set to"},{"line_number":42,"context_line":"   \u0027Wishlist\u0027 to signal the fact that the report is indeed a feature and there"},{"line_number":43,"context_line":"   is no severity associated to it."},{"line_number":44,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_c87504cb","line":41,"in_reply_to":"fa69d971_e6d5a5e4","updated":"2016-01-06 22:40:26.000000000","message":"Nope, bug management is all or nothing. Reviewers will need to check the bug history.","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5dfc40737c1552385f15cae240e99966a717129f","unresolved":false,"context_lines":[{"line_number":43,"context_line":"   is no severity associated to it."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"#. The bug goes into the \u0027Triaged\u0027 state while the discussion is ongoing. This"},{"line_number":46,"context_line":"   discussion should happen in bug comments."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"#. The ironic-drivers team will evaluate the RFE and may advise the submitter"},{"line_number":49,"context_line":"   to file a spec in ironic-specs to elaborate on the feature request, in case"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_e1d90cf4","line":46,"updated":"2016-01-06 19:33:39.000000000","message":"How/who sets this to Triaged? What discussion? Is this a discussion to decide if a spec is needed, or is this the discussion wrt the spec itself?","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"49a8b662b182d8f2a70c2732ff99e7b26637aef0","unresolved":false,"context_lines":[{"line_number":43,"context_line":"   is no severity associated to it."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"#. The bug goes into the \u0027Triaged\u0027 state while the discussion is ongoing. This"},{"line_number":46,"context_line":"   discussion should happen in bug comments."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"#. The ironic-drivers team will evaluate the RFE and may advise the submitter"},{"line_number":49,"context_line":"   to file a spec in ironic-specs to elaborate on the feature request, in case"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_c85c244d","line":46,"in_reply_to":"fa69d971_e1d90cf4","updated":"2016-01-06 22:40:26.000000000","message":"The triager? Maybe this step isn\u0027t needed. This would be the discussion about whether to approve the RFE.","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5dfc40737c1552385f15cae240e99966a717129f","unresolved":false,"context_lines":[{"line_number":45,"context_line":"#. The bug goes into the \u0027Triaged\u0027 state while the discussion is ongoing. This"},{"line_number":46,"context_line":"   discussion should happen in bug comments."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"#. The ironic-drivers team will evaluate the RFE and may advise the submitter"},{"line_number":49,"context_line":"   to file a spec in ironic-specs to elaborate on the feature request, in case"},{"line_number":50,"context_line":"   the RFE requires extra scrutiny, more design discussion, etc. For the spec"},{"line_number":51,"context_line":"   submission process, please see the"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_e1aeec2c","line":48,"updated":"2016-01-06 19:33:39.000000000","message":"This point is tied in with the previous point? I\u0027m a bit lost here.\n\nWe had \u0027acknowledge\u0027 and now we have \u0027evaluate\u0027. So maybe \u0027acknowledge\u0027 just means that someone saw the bug but has made no decision/opinion on it?","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"ff5021fdcf7d3661ab05bb8f2cb4c695f56c7cec","unresolved":false,"context_lines":[{"line_number":45,"context_line":"#. The bug goes into the \u0027Triaged\u0027 state while the discussion is ongoing. This"},{"line_number":46,"context_line":"   discussion should happen in bug comments."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"#. The ironic-drivers team will evaluate the RFE and may advise the submitter"},{"line_number":49,"context_line":"   to file a spec in ironic-specs to elaborate on the feature request, in case"},{"line_number":50,"context_line":"   the RFE requires extra scrutiny, more design discussion, etc. For the spec"},{"line_number":51,"context_line":"   submission process, please see the"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_3b950c32","line":48,"in_reply_to":"fa69d971_4837d4fe","updated":"2016-01-06 22:56:37.000000000","message":"In that case, we should just skim the ack (\u0027Confirmed\u0027) part and go directly to \u0027Triaged\u0027. I see no purpose to confirm. What we want is to discuss/decide if it is a go or no go, right?","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"49a8b662b182d8f2a70c2732ff99e7b26637aef0","unresolved":false,"context_lines":[{"line_number":45,"context_line":"#. The bug goes into the \u0027Triaged\u0027 state while the discussion is ongoing. This"},{"line_number":46,"context_line":"   discussion should happen in bug comments."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"#. The ironic-drivers team will evaluate the RFE and may advise the submitter"},{"line_number":49,"context_line":"   to file a spec in ironic-specs to elaborate on the feature request, in case"},{"line_number":50,"context_line":"   the RFE requires extra scrutiny, more design discussion, etc. For the spec"},{"line_number":51,"context_line":"   submission process, please see the"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_4837d4fe","line":48,"in_reply_to":"fa69d971_e1aeec2c","updated":"2016-01-06 22:40:26.000000000","message":"Correct, we first acknowledge it exists, then evaluate it.","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5dfc40737c1552385f15cae240e99966a717129f","unresolved":false,"context_lines":[{"line_number":53,"context_line":"   wiki page."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"#. If a spec is not required, once the discussion has happened and"},{"line_number":56,"context_line":"   there is positive consensus on the RFE, the report is \u0027approved\u0027, and its"},{"line_number":57,"context_line":"   tag will move from \u0027rfe\u0027 to \u0027rfe-approved\u0027. This means that the feature is"},{"line_number":58,"context_line":"   approved and the related code may be merged."},{"line_number":59,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_a417a243","line":56,"updated":"2016-01-06 19:33:39.000000000","message":"what does \u0027positive consensus\u0027 entail? One person saying they like it and no one voicing any negative opinion w/i X hours or days?","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"ff5021fdcf7d3661ab05bb8f2cb4c695f56c7cec","unresolved":false,"context_lines":[{"line_number":53,"context_line":"   wiki page."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"#. If a spec is not required, once the discussion has happened and"},{"line_number":56,"context_line":"   there is positive consensus on the RFE, the report is \u0027approved\u0027, and its"},{"line_number":57,"context_line":"   tag will move from \u0027rfe\u0027 to \u0027rfe-approved\u0027. This means that the feature is"},{"line_number":58,"context_line":"   approved and the related code may be merged."},{"line_number":59,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_5bda4018","line":56,"in_reply_to":"fa69d971_88006c03","updated":"2016-01-06 22:56:37.000000000","message":"So there are a couple? of decisions that may need to be made:\n\n1. go/no go\n2. if go, with or without a spec\n\nAnd/or it might not be clear w/o a spec, whether it is a go/no go.\n\nAt this point, I suppose it is the same as the old BP, where they would put a description and either submit a spec or not. (And to be honest, I never looked at the BPs w/o a spec and left it for others to nix things.","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"49a8b662b182d8f2a70c2732ff99e7b26637aef0","unresolved":false,"context_lines":[{"line_number":53,"context_line":"   wiki page."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"#. If a spec is not required, once the discussion has happened and"},{"line_number":56,"context_line":"   there is positive consensus on the RFE, the report is \u0027approved\u0027, and its"},{"line_number":57,"context_line":"   tag will move from \u0027rfe\u0027 to \u0027rfe-approved\u0027. This means that the feature is"},{"line_number":58,"context_line":"   approved and the related code may be merged."},{"line_number":59,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_88006c03","line":56,"in_reply_to":"fa69d971_a417a243","updated":"2016-01-06 22:40:26.000000000","message":"Reasonable consensus among the ironic-drivers team. We should use common sense here.\n\nSomething like \"Allow multiple nodes in cli commands\" isn\u0027t super contentious and I\u0027d be fine with one or two people rubber stamping that.\n\nSomething like \"Make ironic into a full fledged CMDB\" would obviously need everyone in on the discussion. :)","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5dfc40737c1552385f15cae240e99966a717129f","unresolved":false,"context_lines":[{"line_number":54,"context_line":""},{"line_number":55,"context_line":"#. If a spec is not required, once the discussion has happened and"},{"line_number":56,"context_line":"   there is positive consensus on the RFE, the report is \u0027approved\u0027, and its"},{"line_number":57,"context_line":"   tag will move from \u0027rfe\u0027 to \u0027rfe-approved\u0027. This means that the feature is"},{"line_number":58,"context_line":"   approved and the related code may be merged."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"#. If a spec is required, the spec must be submitted (with the bug properly"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_c444164b","line":57,"updated":"2016-01-06 19:33:39.000000000","message":"Who can move this tag? only ironic-drivers team?","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"49a8b662b182d8f2a70c2732ff99e7b26637aef0","unresolved":false,"context_lines":[{"line_number":54,"context_line":""},{"line_number":55,"context_line":"#. If a spec is not required, once the discussion has happened and"},{"line_number":56,"context_line":"   there is positive consensus on the RFE, the report is \u0027approved\u0027, and its"},{"line_number":57,"context_line":"   tag will move from \u0027rfe\u0027 to \u0027rfe-approved\u0027. This means that the feature is"},{"line_number":58,"context_line":"   approved and the related code may be merged."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"#. If a spec is required, the spec must be submitted (with the bug properly"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_db071016","line":57,"in_reply_to":"fa69d971_c444164b","updated":"2016-01-06 22:40:26.000000000","message":"Technically, anyone, but only ironic-drivers is supposed to move it.","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5dfc40737c1552385f15cae240e99966a717129f","unresolved":false,"context_lines":[{"line_number":58,"context_line":"   approved and the related code may be merged."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"#. If a spec is required, the spec must be submitted (with the bug properly"},{"line_number":61,"context_line":"   tagged in the commit message), reviewed, and merged before the RFE will be"},{"line_number":62,"context_line":"   \u0027approved\u0027 (and the tag changed to \u0027rfe-approved\u0027)."},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"#. The bug then goes through the usual process -- first to \u0027In progress\u0027 when"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_64d68a50","line":61,"updated":"2016-01-06 19:33:39.000000000","message":"partial-bug, right?","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"ff5021fdcf7d3661ab05bb8f2cb4c695f56c7cec","unresolved":false,"context_lines":[{"line_number":58,"context_line":"   approved and the related code may be merged."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"#. If a spec is required, the spec must be submitted (with the bug properly"},{"line_number":61,"context_line":"   tagged in the commit message), reviewed, and merged before the RFE will be"},{"line_number":62,"context_line":"   \u0027approved\u0027 (and the tag changed to \u0027rfe-approved\u0027)."},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"#. The bug then goes through the usual process -- first to \u0027In progress\u0027 when"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_5bf1e093","line":61,"in_reply_to":"fa69d971_5b1c8069","updated":"2016-01-06 22:56:37.000000000","message":"Since this is the spec patch and we mention tagging it with the bug, might as well be explicit and say \u0027Partial-Bug: #bug\u0027 blah blah. (the use of \u0027tag\u0027 now is overloaded, since the bug itself can be tagged. In a commit msg, maybe \u0027tag\u0027 isn\u0027t the right term, i don\u0027t recall what those things are called.)\n\nOh. Looks like they are referred to as \u0027external references\u0027 : https://wiki.openstack.org/wiki/GitCommitMessages#Including_external_references","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"49a8b662b182d8f2a70c2732ff99e7b26637aef0","unresolved":false,"context_lines":[{"line_number":58,"context_line":"   approved and the related code may be merged."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"#. If a spec is required, the spec must be submitted (with the bug properly"},{"line_number":61,"context_line":"   tagged in the commit message), reviewed, and merged before the RFE will be"},{"line_number":62,"context_line":"   \u0027approved\u0027 (and the tag changed to \u0027rfe-approved\u0027)."},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"#. The bug then goes through the usual process -- first to \u0027In progress\u0027 when"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_5b1c8069","line":61,"in_reply_to":"fa69d971_64d68a50","updated":"2016-01-06 22:40:26.000000000","message":"Sure, this is discussed below.","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5dfc40737c1552385f15cae240e99966a717129f","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"#. If a spec is required, the spec must be submitted (with the bug properly"},{"line_number":61,"context_line":"   tagged in the commit message), reviewed, and merged before the RFE will be"},{"line_number":62,"context_line":"   \u0027approved\u0027 (and the tag changed to \u0027rfe-approved\u0027)."},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"#. The bug then goes through the usual process -- first to \u0027In progress\u0027 when"},{"line_number":65,"context_line":"   the spec/code is being worked on, then \u0027Fix Released\u0027 when it is"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_e4ad9ae9","line":62,"updated":"2016-01-06 19:33:39.000000000","message":"Once a spec patch exists, the status will be \u0027In Progress\u0027 right?\n\nironic-drivers team will move the tag to rfe-approved?","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"49a8b662b182d8f2a70c2732ff99e7b26637aef0","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"#. If a spec is required, the spec must be submitted (with the bug properly"},{"line_number":61,"context_line":"   tagged in the commit message), reviewed, and merged before the RFE will be"},{"line_number":62,"context_line":"   \u0027approved\u0027 (and the tag changed to \u0027rfe-approved\u0027)."},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"#. The bug then goes through the usual process -- first to \u0027In progress\u0027 when"},{"line_number":65,"context_line":"   the spec/code is being worked on, then \u0027Fix Released\u0027 when it is"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_3b53cc18","line":62,"in_reply_to":"fa69d971_e4ad9ae9","updated":"2016-01-06 22:40:26.000000000","message":"Yes, and yes.","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5dfc40737c1552385f15cae240e99966a717129f","unresolved":false,"context_lines":[{"line_number":62,"context_line":"   \u0027approved\u0027 (and the tag changed to \u0027rfe-approved\u0027)."},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"#. The bug then goes through the usual process -- first to \u0027In progress\u0027 when"},{"line_number":65,"context_line":"   the spec/code is being worked on, then \u0027Fix Released\u0027 when it is"},{"line_number":66,"context_line":"   implemented. ironic-drivers team may also move the bug to \"Won\u0027t Fix\""},{"line_number":67,"context_line":"   status, meaning that RFE is rejected."},{"line_number":68,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_24f5f2eb","line":65,"updated":"2016-01-06 19:33:39.000000000","message":"too late, above spec will put it \u0027in progress\u0027 already :)","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"49a8b662b182d8f2a70c2732ff99e7b26637aef0","unresolved":false,"context_lines":[{"line_number":62,"context_line":"   \u0027approved\u0027 (and the tag changed to \u0027rfe-approved\u0027)."},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"#. The bug then goes through the usual process -- first to \u0027In progress\u0027 when"},{"line_number":65,"context_line":"   the spec/code is being worked on, then \u0027Fix Released\u0027 when it is"},{"line_number":66,"context_line":"   implemented. ironic-drivers team may also move the bug to \"Won\u0027t Fix\""},{"line_number":67,"context_line":"   status, meaning that RFE is rejected."},{"line_number":68,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_bb3fbcb2","line":65,"in_reply_to":"fa69d971_24f5f2eb","updated":"2016-01-06 22:40:26.000000000","message":"If there\u0027s a spec...","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5dfc40737c1552385f15cae240e99966a717129f","unresolved":false,"context_lines":[{"line_number":64,"context_line":"#. The bug then goes through the usual process -- first to \u0027In progress\u0027 when"},{"line_number":65,"context_line":"   the spec/code is being worked on, then \u0027Fix Released\u0027 when it is"},{"line_number":66,"context_line":"   implemented. ironic-drivers team may also move the bug to \"Won\u0027t Fix\""},{"line_number":67,"context_line":"   status, meaning that RFE is rejected."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"#. If the RFE is not approved, the ironic-drivers team may also move the bug to"},{"line_number":70,"context_line":"   \"Won\u0027t Fix\" status."}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_c475f663","line":67,"updated":"2016-01-06 19:33:39.000000000","message":"This is too late. Seems like we should reject the RFE before this step. Which doesn\u0027t mean that we can\u0027t change our mind and reject it later but I think we need to mention rejection sooner.","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"49a8b662b182d8f2a70c2732ff99e7b26637aef0","unresolved":false,"context_lines":[{"line_number":64,"context_line":"#. The bug then goes through the usual process -- first to \u0027In progress\u0027 when"},{"line_number":65,"context_line":"   the spec/code is being worked on, then \u0027Fix Released\u0027 when it is"},{"line_number":66,"context_line":"   implemented. ironic-drivers team may also move the bug to \"Won\u0027t Fix\""},{"line_number":67,"context_line":"   status, meaning that RFE is rejected."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"#. If the RFE is not approved, the ironic-drivers team may also move the bug to"},{"line_number":70,"context_line":"   \"Won\u0027t Fix\" status."}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_3b7aec88","line":67,"in_reply_to":"fa69d971_c475f663","updated":"2016-01-06 22:40:26.000000000","message":"Yeah, I meant to delete this part and just have the part below. Also maybe these should be bullets, not numbers.","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5dfc40737c1552385f15cae240e99966a717129f","unresolved":false,"context_lines":[{"line_number":67,"context_line":"   status, meaning that RFE is rejected."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"#. If the RFE is not approved, the ironic-drivers team may also move the bug to"},{"line_number":70,"context_line":"   \"Won\u0027t Fix\" status."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"When working on RFE, please be sure to tag your commits properly:"},{"line_number":73,"context_line":"\"Partial-Bug: #xxxx\" or \"Related-Bug: #xxxx\" for intermediate commits for the"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_04254e5a","line":70,"updated":"2016-01-06 19:33:39.000000000","message":"Isn\u0027t this point the same one above in line 66?","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"49a8b662b182d8f2a70c2732ff99e7b26637aef0","unresolved":false,"context_lines":[{"line_number":67,"context_line":"   status, meaning that RFE is rejected."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"#. If the RFE is not approved, the ironic-drivers team may also move the bug to"},{"line_number":70,"context_line":"   \"Won\u0027t Fix\" status."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"When working on RFE, please be sure to tag your commits properly:"},{"line_number":73,"context_line":"\"Partial-Bug: #xxxx\" or \"Related-Bug: #xxxx\" for intermediate commits for the"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_db6cf0d1","line":70,"in_reply_to":"fa69d971_04254e5a","updated":"2016-01-06 22:40:26.000000000","message":"Yes","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"5dfc40737c1552385f15cae240e99966a717129f","unresolved":false,"context_lines":[{"line_number":84,"context_line":"team, not the ironic-drivers team. This means that anyone may edit bug details,"},{"line_number":85,"context_line":"and there is room to game the system here. **RFEs may only be approved by"},{"line_number":86,"context_line":"members of the ironic-drivers team**. Attempts to sneak around this rule will"},{"line_number":87,"context_line":"not be tolerated, and will be called out in public on the mailing list."},{"line_number":88,"context_line":""},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"Live Upgrade Related Concerns"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_c4d17624","line":87,"updated":"2016-01-06 19:33:39.000000000","message":"OH. Ha ha...\n\nIs there *anything* that someone on ironic-bugs team CANNOT do?","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"49a8b662b182d8f2a70c2732ff99e7b26637aef0","unresolved":false,"context_lines":[{"line_number":84,"context_line":"team, not the ironic-drivers team. This means that anyone may edit bug details,"},{"line_number":85,"context_line":"and there is room to game the system here. **RFEs may only be approved by"},{"line_number":86,"context_line":"members of the ironic-drivers team**. Attempts to sneak around this rule will"},{"line_number":87,"context_line":"not be tolerated, and will be called out in public on the mailing list."},{"line_number":88,"context_line":""},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"Live Upgrade Related Concerns"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa69d971_5b61e0d1","line":87,"in_reply_to":"fa69d971_c4d17624","updated":"2016-01-06 22:40:26.000000000","message":"In the bug tracker, only admin things (deleting the bug tracker, etc)","commit_id":"688b59ee42b63a2a2f87c29d06e0fa694e39b072"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"fd5421c89a37ff0c6495dc39ce72a526177e72ac","unresolved":false,"context_lines":[{"line_number":15,"context_line":"RFEs (Requests for Feature Enhancements) instead of blueprints. These are bugs"},{"line_number":16,"context_line":"with \u0027rfe\u0027 tag, and they should be submitted before a spec or code is proposed."},{"line_number":17,"context_line":"When a member of `ironic-drivers launchpad team"},{"line_number":18,"context_line":"https://launchpad.net/~ironic-drivers/+members\u003e`_ will decide that the proposal"},{"line_number":19,"context_line":"is worth implementing, a spec (if needed) and code should be submitted,"},{"line_number":20,"context_line":"referencing the RFE bug. Contributors are welcome to submit a spec and/or code"},{"line_number":21,"context_line":"before the RFE is approved, however those patches will not land until the RFE"}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa69d971_2a9ef463","line":18,"updated":"2016-01-07 14:37:02.000000000","message":"That didn\u0027t render. Do you need s/https/\u003chttps/ ?\n\ns/will decide/decides/","commit_id":"36fdcc22b3adf065fd85a31b9e69ea844c9cfa30"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"fd5421c89a37ff0c6495dc39ce72a526177e72ac","unresolved":false,"context_lines":[{"line_number":41,"context_line":"   going to work on this feature."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"#. As soon as a member of the ironic-drivers team acknowledges the bug, it"},{"line_number":44,"context_line":"   will be moved into the \u0027Triaged\u0027 state. The importance will be set to"},{"line_number":45,"context_line":"   \u0027Wishlist\u0027 to signal the fact that the report is indeed a feature and there"},{"line_number":46,"context_line":"   is no severity associated to it. Discussion about the RFE, and whether to"},{"line_number":47,"context_line":"   approve it, happens in bug comments while in the \u0027Triaged\u0027 state."}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa69d971_6adaaceb","line":44,"updated":"2016-01-07 14:37:02.000000000","message":"I guess it is either \u0027Triaged\u0027 or \u0027Confirmed\u0027. I looked at the BugTriage page [1], and they tend to use \u0027Confirmed\u0027. For \u0027Triaged\u0027 it says \"If the bug contains the solution, or a patch, set the bug status to Triaged\". (If it has a patch, doesn\u0027t it usually say \u0027in progress\u0027 but anyway I digress.)\n\nSo maybe we want Confirmed and should skip Triaged. What does Dmitry think?\n\n[1] https://wiki.openstack.org/wiki/BugTriage","commit_id":"36fdcc22b3adf065fd85a31b9e69ea844c9cfa30"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"fd5421c89a37ff0c6495dc39ce72a526177e72ac","unresolved":false,"context_lines":[{"line_number":54,"context_line":"   wiki page."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"#. If a spec is not required, once the discussion has happened and there is"},{"line_number":57,"context_line":"   positive consensus among the ironic-drivers team on the RFE, the report is"},{"line_number":58,"context_line":"   \u0027approved\u0027, and its tag will move from \u0027rfe\u0027 to \u0027rfe-approved\u0027. This means"},{"line_number":59,"context_line":"   that the feature is approved and the related code may be merged."},{"line_number":60,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa69d971_6a5dcc47","line":57,"updated":"2016-01-07 14:37:02.000000000","message":"s/report/bug/ Or s/report/RFE/","commit_id":"36fdcc22b3adf065fd85a31b9e69ea844c9cfa30"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"fd5421c89a37ff0c6495dc39ce72a526177e72ac","unresolved":false,"context_lines":[{"line_number":66,"context_line":"   the spec/code is being worked on, then \u0027Fix Released\u0027 when it is"},{"line_number":67,"context_line":"   implemented."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"#. If the RFE is not approved, the ironic-drivers team may also move the bug to"},{"line_number":70,"context_line":"   \"Won\u0027t Fix\" status."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"When working on RFE, please be sure to tag your commits properly:"}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa69d971_458f9975","line":69,"updated":"2016-01-07 14:37:02.000000000","message":"Why \u0027may also move\u0027. Why not \u0027will move\u0027 ?\n\nMaybe \u0027At any point in the process, if the decision is not to approve the RFE, the ironic-drivers team will do \u003cwhatever\u003e to the bug.\u0027\n\nOR maybe put this bullet after the 4th bullet (line 55).","commit_id":"36fdcc22b3adf065fd85a31b9e69ea844c9cfa30"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"fd5421c89a37ff0c6495dc39ce72a526177e72ac","unresolved":false,"context_lines":[{"line_number":69,"context_line":"#. If the RFE is not approved, the ironic-drivers team may also move the bug to"},{"line_number":70,"context_line":"   \"Won\u0027t Fix\" status."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"When working on RFE, please be sure to tag your commits properly:"},{"line_number":73,"context_line":"\"Partial-Bug: #xxxx\" or \"Related-Bug: #xxxx\" for intermediate commits for the"},{"line_number":74,"context_line":"feature, and \"Closes-Bug: #xxxx\" for the final commit. It is also helpful to"},{"line_number":75,"context_line":"set a consistent review topic, such as \"bug/xxxx\" for all patches related to"}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa69d971_e5f6cdf8","line":72,"updated":"2016-01-07 14:37:02.000000000","message":"s/RFE/an RFE/","commit_id":"36fdcc22b3adf065fd85a31b9e69ea844c9cfa30"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"a21c20dba4b390c504a8a00d040ee53cec0625b9","unresolved":false,"context_lines":[{"line_number":27,"context_line":"#. Submit a bug report at https://bugs.launchpad.net/ironic/+filebug."},{"line_number":28,"context_line":"   There are two fields that must be filled: \u0027summary\u0027 and"},{"line_number":29,"context_line":"   \u0027further information\u0027. The \u0027summary\u0027 must be brief enough to fit in one"},{"line_number":30,"context_line":"   line: if you can’t describe it in a few words it may mean that you are"},{"line_number":31,"context_line":"   either trying to capture more than one RFE at once, or that you are having"},{"line_number":32,"context_line":"   a hard time defining what you are trying to solve at all."},{"line_number":33,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"fa69d971_7157ae02","line":30,"updated":"2016-01-08 13:35:33.000000000","message":"Also should be mention that the summary should contain a \"[RFE]\" string prefixing it?\n\ne.g\n\n[RFE] Implement foo bar","commit_id":"ffb654ba29f1bfe56385ecc995e177b593c78e96"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"e15291225c5d5cb0251218f5ef3e26ca51dc31b5","unresolved":false,"context_lines":[{"line_number":27,"context_line":"#. Submit a bug report at https://bugs.launchpad.net/ironic/+filebug."},{"line_number":28,"context_line":"   There are two fields that must be filled: \u0027summary\u0027 and"},{"line_number":29,"context_line":"   \u0027further information\u0027. The \u0027summary\u0027 must be brief enough to fit in one"},{"line_number":30,"context_line":"   line: if you can’t describe it in a few words it may mean that you are"},{"line_number":31,"context_line":"   either trying to capture more than one RFE at once, or that you are having"},{"line_number":32,"context_line":"   a hard time defining what you are trying to solve at all."},{"line_number":33,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"fa69d971_31d29601","line":30,"in_reply_to":"fa69d971_7157ae02","updated":"2016-01-08 13:43:39.000000000","message":"That was there before; I honestly don\u0027t think this is useful. The tag is good enough.","commit_id":"ffb654ba29f1bfe56385ecc995e177b593c78e96"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"5d0b33eadb83f46029138f6d7a7975e24f14cbe5","unresolved":false,"context_lines":[{"line_number":37,"context_line":"   to be addressed, or what is the enhancement that would make the platform"},{"line_number":38,"context_line":"   more capable, both from a functional and a non-functional standpoint."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"#. Submit the bug, add an \u0027rfe\u0027 tag to it and assign yourself or whoever is"},{"line_number":41,"context_line":"   going to work on this feature."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"#. As soon as a member of the ironic-drivers team acknowledges the bug, it"},{"line_number":44,"context_line":"   will be moved into the \u0027Triaged\u0027 state. The importance will be set to"}],"source_content_type":"text/x-rst","patch_set":6,"id":"fa69d971_ce2fd73c","line":41,"range":{"start_line":40,"start_character":3,"end_line":41,"end_character":33},"updated":"2016-01-08 13:28:09.000000000","message":"nit: this can be done before submitting a bug","commit_id":"ffb654ba29f1bfe56385ecc995e177b593c78e96"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"e15291225c5d5cb0251218f5ef3e26ca51dc31b5","unresolved":false,"context_lines":[{"line_number":37,"context_line":"   to be addressed, or what is the enhancement that would make the platform"},{"line_number":38,"context_line":"   more capable, both from a functional and a non-functional standpoint."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"#. Submit the bug, add an \u0027rfe\u0027 tag to it and assign yourself or whoever is"},{"line_number":41,"context_line":"   going to work on this feature."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"#. As soon as a member of the ironic-drivers team acknowledges the bug, it"},{"line_number":44,"context_line":"   will be moved into the \u0027Triaged\u0027 state. The importance will be set to"}],"source_content_type":"text/x-rst","patch_set":6,"id":"fa69d971_9101026d","line":41,"range":{"start_line":40,"start_character":3,"end_line":41,"end_character":33},"in_reply_to":"fa69d971_ce2fd73c","updated":"2016-01-08 13:43:39.000000000","message":"Meh :)","commit_id":"ffb654ba29f1bfe56385ecc995e177b593c78e96"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"5d0b33eadb83f46029138f6d7a7975e24f14cbe5","unresolved":false,"context_lines":[{"line_number":55,"context_line":""},{"line_number":56,"context_line":"#. If a spec is not required, once the discussion has happened and there is"},{"line_number":57,"context_line":"   positive consensus among the ironic-drivers team on the RFE, the RFE is"},{"line_number":58,"context_line":"   \u0027approved\u0027, and its tag will move from \u0027rfe\u0027 to \u0027rfe-approved\u0027. This means"},{"line_number":59,"context_line":"   that the feature is approved and the related code may be merged."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"#. If a spec is required, the spec must be submitted (with the bug properly"}],"source_content_type":"text/x-rst","patch_set":6,"id":"fa69d971_ce8b571d","line":58,"range":{"start_line":58,"start_character":19,"end_line":58,"end_character":65},"updated":"2016-01-08 13:28:09.000000000","message":"note that everyone can set tags","commit_id":"ffb654ba29f1bfe56385ecc995e177b593c78e96"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"e15291225c5d5cb0251218f5ef3e26ca51dc31b5","unresolved":false,"context_lines":[{"line_number":55,"context_line":""},{"line_number":56,"context_line":"#. If a spec is not required, once the discussion has happened and there is"},{"line_number":57,"context_line":"   positive consensus among the ironic-drivers team on the RFE, the RFE is"},{"line_number":58,"context_line":"   \u0027approved\u0027, and its tag will move from \u0027rfe\u0027 to \u0027rfe-approved\u0027. This means"},{"line_number":59,"context_line":"   that the feature is approved and the related code may be merged."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"#. If a spec is required, the spec must be submitted (with the bug properly"}],"source_content_type":"text/x-rst","patch_set":6,"id":"fa69d971_f1f2be99","line":58,"range":{"start_line":58,"start_character":19,"end_line":58,"end_character":65},"in_reply_to":"fa69d971_ce8b571d","updated":"2016-01-08 13:43:39.000000000","message":"Yeah, noted below.","commit_id":"ffb654ba29f1bfe56385ecc995e177b593c78e96"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"90aadb164b0cbc85885b752102319f78601850f0","unresolved":false,"context_lines":[{"line_number":67,"context_line":"   implemented."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"#. If the RFE is not approved, the ironic-drivers team will move the bug to"},{"line_number":70,"context_line":"   \"Won\u0027t Fix\" status."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"When working on an RFE, please be sure to tag your commits properly:"},{"line_number":73,"context_line":"\"Partial-Bug: #xxxx\" or \"Related-Bug: #xxxx\" for intermediate commits for the"}],"source_content_type":"text/x-rst","patch_set":6,"id":"fa69d971_ee341bdf","line":70,"updated":"2016-01-08 13:21:31.000000000","message":"nit:\n\ns/not approved/rejected\n\nCause \"not approved\" IMO sounds like no decision was taken yet","commit_id":"ffb654ba29f1bfe56385ecc995e177b593c78e96"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"e15291225c5d5cb0251218f5ef3e26ca51dc31b5","unresolved":false,"context_lines":[{"line_number":67,"context_line":"   implemented."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"#. If the RFE is not approved, the ironic-drivers team will move the bug to"},{"line_number":70,"context_line":"   \"Won\u0027t Fix\" status."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"When working on an RFE, please be sure to tag your commits properly:"},{"line_number":73,"context_line":"\"Partial-Bug: #xxxx\" or \"Related-Bug: #xxxx\" for intermediate commits for the"}],"source_content_type":"text/x-rst","patch_set":6,"id":"fa69d971_d12e9af9","line":70,"in_reply_to":"fa69d971_ce92774b","updated":"2016-01-08 13:43:39.000000000","message":"Done in a follow-up patch. https://review.openstack.org/265255","commit_id":"ffb654ba29f1bfe56385ecc995e177b593c78e96"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"5d0b33eadb83f46029138f6d7a7975e24f14cbe5","unresolved":false,"context_lines":[{"line_number":67,"context_line":"   implemented."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"#. If the RFE is not approved, the ironic-drivers team will move the bug to"},{"line_number":70,"context_line":"   \"Won\u0027t Fix\" status."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"When working on an RFE, please be sure to tag your commits properly:"},{"line_number":73,"context_line":"\"Partial-Bug: #xxxx\" or \"Related-Bug: #xxxx\" for intermediate commits for the"}],"source_content_type":"text/x-rst","patch_set":6,"id":"fa69d971_ce92774b","line":70,"in_reply_to":"fa69d971_ee341bdf","updated":"2016-01-08 13:28:09.000000000","message":"+1","commit_id":"ffb654ba29f1bfe56385ecc995e177b593c78e96"}]}