Comment 7 for bug 1526382

Revision history for this message
Nisha Agarwal (agarwalnisha1980) wrote :

rloo, please see the discussion here. I wasn't aware that I shud have added the discussion here. I removed the tag so it can be analyzed again as per discussion.
the link is : http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2016-11-09.log.html
---
Nisha_away jroll, other ironic members, i have a query. I wanted to understand when does a RFE require a spec.
Nisha_away I have heard that RFE can be approved without the spec also 15:29
vdrok Nisha_away: I think it's decided on case by case basis 15:30
vdrok Nisha_away: just file an rfe, if it needs a spec, you eill be told about that in the bug 15:30
Nisha_away vdrok, ok. i wanted to understand it for the existing RFE 15:30
jroll Nisha_away: yes, what vdrok said. anything that is a bit complex, changes API/RPC, affects upgrades, etc usually needs a spec 15:31
jroll Nisha_away: then ask about that RFE :) 15:31
Nisha_away vdrok, for instance https://bugs.launchpad.net/ironic/+bug/1526382 15:31
openstack Launchpad bug 1526382 in Ironic "[RFE] Enhance ilo drivers to do inband inspection along with OOB inspection" [Wishlist,In progress] - Assigned to Aparna (aparnavtce) 15:31
Nisha_away this actually requires https://review.openstack.org/305864 to be merged 15:31
patchbot patch 305864 - ironic-specs - Boot management for in-band inspection 15:32

Nisha_away and then when inspector start supporting virtual media as boot device, then ilo drivers just use that 15:32
Nisha_away to start supporting inband inspection 15:33
Nisha_away vdrok ^^^ 15:33
Nisha_away jroll ^^^ 15:35
vdrok Nisha_away: soo, the rfe bug you linked has needs-spec tag, which is pretty self-describing :) 15:35
jroll Nisha_away: to be approved without a spec, the RFE would need more details on the changes... since you already have the spec written I think we should just review that 15:35
vdrok yep, it needs the spec 15:35
jroll vdrok: I added that because it had a spec, to be honest 15:35
vdrok ah