Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-01-23
15:50:29 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Address comments from original inventory patch https://review.openstack.org/521578
15:50:30 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Add missing runtime requirements https://review.openstack.org/536870
15:50:30 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Usage docs and initial release note for osc-placement https://review.openstack.org/536858
15:50:31 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Address review comments from allocations patch https://review.openstack.org/536871
15:50:31 alex_xu dansmith: ok, thanks
16:00:18 openstackgerrit Merged openstack/nova master: Add reference to policy sample https://review.openstack.org/531713
16:00:31 openstackgerrit Merged openstack/nova master: Raise on API errors getting aggregates/traits https://review.openstack.org/526540
16:01:02 openstackgerrit Merged openstack/python-novaclient master: Microversion 2.59 - Migrations list pagination https://review.openstack.org/516168
16:02:36 openstackgerrit melanie witt proposed openstack/nova master: Set server status to ERROR if rebuild failed https://review.openstack.org/536268
16:04:49 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: CLI for resource classes (v1.2) https://review.openstack.org/511182
16:04:49 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: CLI for aggregates (v1.1) https://review.openstack.org/505643
16:04:50 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: RP delete inventories (v1.5) https://review.openstack.org/514642
16:04:50 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: RP list: member_of and resources parameters (v1.3, v1.4) https://review.openstack.org/511183
16:04:51 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644
16:04:51 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: CLI for traits (v1.6) https://review.openstack.org/514643
16:04:52 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647
16:04:52 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646
16:04:53 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: [WIP] Get resource provider by uuid or name https://review.openstack.org/527791
16:07:58 bauzas sorry folks, I just migrated my IRC bouncer
16:08:06 bauzas in case some pinged me, lemme know
16:09:16 openstackgerrit Jackie Truong proposed openstack/python-novaclient master: Microversion 2.61 - Add trusted_image_certificates https://review.openstack.org/500396
16:10:40 openstackgerrit Matthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice https://review.openstack.org/526347
16:10:40 openstackgerrit Matthew Booth proposed openstack/nova master: Give volume DriverBlockDevice classes a common prefix https://review.openstack.org/526346
16:10:41 openstackgerrit Matthew Booth proposed openstack/nova master: Add local_root to block_device_info https://review.openstack.org/529029
16:10:41 openstackgerrit Matthew Booth proposed openstack/nova master: Expose driver_block_device fields consistently https://review.openstack.org/528362
16:10:42 openstackgerrit Matthew Booth proposed openstack/nova master: Fix libvirt volume tests passing invalid disk_info https://review.openstack.org/529328
16:10:42 openstackgerrit Matthew Booth proposed openstack/nova master: Pass DriverBlockDevice to driver.attach_volume https://review.openstack.org/528363
16:10:43 openstackgerrit Matthew Booth proposed openstack/nova master: Expose volume host type and path independent of libvirt config https://review.openstack.org/530786
16:10:43 openstackgerrit Matthew Booth proposed openstack/nova master: Pass disk_info dict to libvirt_info https://review.openstack.org/529329
16:10:44 openstackgerrit Matthew Booth proposed openstack/nova master: Don't generate fake disk_info in swap_volume https://review.openstack.org/530787
16:10:45 openstackgerrit Matthew Booth proposed openstack/nova master: Local disk serial numbers for the libvirt driver https://review.openstack.org/529380
16:16:40 openstackgerrit melanie witt proposed openstack/nova master: Set server status to ERROR if rebuild failed https://review.openstack.org/536268
16:16:49 mriedem melwitt: suggestion in https://review.openstack.org/#/c/536268/
16:16:56 mriedem right before you pushed
16:17:06 mriedem we can assert the fault is set in the functional test that is also being updated
16:17:14 melwitt I think maybe I did what you asked
16:17:31 melwitt oh, nope, different func test
16:17:48 mriedem ah that works too
16:18:00 melwitt k, lemme add that
16:18:03 mriedem that's fine,
16:18:20 mriedem this is better actually because a rebuild can be done for a non-admin and we want to make sure the non-admin can see the fault
16:18:44 mriedem i don't know if we want to say something specific in the fault message about attempting to rebuild with a different image, that could be a slippery slope of assumptions about why we failed
16:18:45 melwitt okay, cool
16:19:13 mriedem like, image might be fine, but some other filter failed because a resource is unavailable or something
16:19:33 melwitt yeah, I was thinking about that too
16:19:46 mriedem +2
16:20:23 mriedem heh stable/newton is still around
16:21:29 melwitt :)
16:22:37 mriedem this is probably not worth yet another newton release before eol though
16:22:46 mriedem tonyb: when you gonna pull that newton-eol trigger?
16:23:27 melwitt wah wah
16:30:05 bauzas wait, Newton is not EOL'd yet ?
16:30:08 bauzas woah
16:30:14 mriedem melwitt: want to start on the backports?
16:30:47 bauzas melwitt: mriedem: dansmith: I think we should document a relnote with https://review.openstack.org/#/c/536268/6
16:30:47 melwitt mriedem: yes
16:31:01 bauzas I was about to say that we will provide a new behaviour
16:31:16 dansmith bauzas: it's really the behavior that every other action has though
16:31:18 bauzas I'm not asking for a microversion, but we need to explain for operators I think
16:31:31 bauzas dansmith: I know
16:31:41 bauzas dansmith: I'm just thinking that it's a game changer
16:31:56 bauzas and documenting it would be IMHO a small thing for a better point
16:32:00 mriedem *shrug* i thought about a "fixes" release note just to point out that if scheduling fails during rebuild the instance will go to ERROR state and a fault will be recorded
16:32:02 bauzas I just don't want to see rants here :p
16:32:20 bauzas mriedem: exactly my point
16:32:26 bauzas not more than that
16:32:35 mriedem i'd be fine with that
16:32:37 bauzas so we can point out the relnote if someone is aking that
16:32:47 dansmith meh.
16:33:03 bauzas backporting that means also that we will change the logic
16:33:27 bauzas hopefully the stable version will be not a .z one
16:33:57 mriedem umm
16:34:00 mriedem it's not a new feature
16:34:03 mriedem it's a bug fix
16:34:16 bauzas sure, but we're changing the API right?
16:34:26 mriedem we didn't bump the .y for the change that made the API fail fast if you tried to rebuild a volume-backed instance and change the image
16:34:28 bauzas again, I'm not asking for a microversion, please
16:34:36 bauzas okay
16:34:51 bauzas whatever the semver will be, I just think we need to explain that to ops
16:35:32 bauzas os-instance-actions API is admin by default ?
16:35:42 mriedem no
16:35:44 bauzas I don't remember, I need to look at the default policy
16:35:50 bauzas ok, cool then
16:35:54 mriedem os-instance-actions events with the traceback is admin by default
16:36:03 bauzas hah, right
16:36:09 mriedem i think you can't see the events before some microversion if you're not an admin either
16:36:22 bauzas I can test on my box
16:36:53 mriedem don't bother: https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id46
16:37:18 bauzas cool cool
16:38:59 openstackgerrit Merged openstack/python-novaclient master: Add support for microversion 2.60 - volume multiattach https://review.openstack.org/536621
16:39:52 bauzas mriedem: dansmith: melwitt: looks like my proposal to add a relnote didn't get a huge consensus, still okay if I'm providing it ?
16:39:53 melwitt mriedem, bauzas: so, should I add a fixes relnote to the patch then?
16:40:01 bauzas hah, jinxed
16:40:02 mriedem incoming
16:40:15 openstackgerrit Matt Riedemann proposed openstack/nova master: Set server status to ERROR if rebuild failed https://review.openstack.org/536268
16:40:16 bauzas who's doing that ? I can take my pen if nobody does
16:40:32 bauzas oh, a new rev
16:40:36 bauzas mriedem: on it
16:41:07 bauzas mriedem: +2
16:41:22 bauzas I can fast-approve or dansmith can +W

Earlier   Later