Earlier  
Posted Nick Remark
#openstack-nova - 2018-09-20
17:43:51 dansmith bauzas: not sure the details
17:43:59 lucidguy cdent: Sadly I don't know how to do that.
17:44:19 dansmith lucidguy: luckily there is a script that fixes this for you so you don't haveto
17:44:41 cdent yeah, lucidguy, if you're not comfortable running curl, use the scripts
17:44:53 bauzas dansmith: no worries, I'll test it on my devbox then
17:45:04 bauzas I should dump the DB to fake a reshape then
18:29:25 openstackgerrit Eric Fried proposed openstack/nova master: Nix update_instance_allocation, _allocate_for_instance https://review.openstack.org/604186
18:45:39 efried jaypipes-ooo, gibi, mriedem: --^
18:46:56 mriedem fake news
18:49:21 mriedem lemme -1 quick
18:50:44 mriedem just need to update some docstrings
18:50:56 mriedem otherwise i'm in love with your patch
18:51:30 efried mm, good eye
18:51:46 mriedem i only have one
18:51:55 efried I used ^g, not grep
18:53:06 openstackgerrit Eric Fried proposed openstack/nova master: Nix update_instance_allocation, _allocate_for_instance https://review.openstack.org/604186
18:53:14 efried mriedem: Not sure if I did that right
18:53:27 efried I didn't go and look at the resource tracker
18:54:00 mriedem i did, with jay's change underneath it's good
18:54:29 efried coo
18:54:41 openstackgerrit Elancheran S proposed openstack/nova master: Add exact match aggregate image properties matcher/filter https://review.openstack.org/593167
18:56:18 mriedem baby talk will get you everywhere
19:01:48 openstackgerrit Merged openstack/nova master: doc trivial: additional info to admin-password-injection https://review.openstack.org/603414
19:14:02 openstackgerrit Merged openstack/nova stable/pike: Fix DB archiver AttributeError due to wrong table name attribute used https://review.openstack.org/599883
19:23:12 openstackgerrit Jim Rollenhagen proposed openstack/nova master: ironic: stop hammering ironic API in power sync loop https://review.openstack.org/602127
19:23:13 jroll mriedem: that has a bug report and such now per your request, should also be passing tests now ^
19:24:01 mriedem ok
19:25:59 openstackgerrit Merged openstack/nova master: XenAPI/Stops the migration of volume backed VHDS https://review.openstack.org/533168
19:41:01 openstackgerrit Merged openstack/nova master: Refactor NeutronFixture https://review.openstack.org/588338
19:57:39 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: XenAPI/Stops the migration of volume backed VHDS https://review.openstack.org/604203
20:05:14 openstackgerrit Surya Seetharaman proposed openstack/nova master: Add get_by_cell_and_project() method to InstanceMappingList https://review.openstack.org/591656
20:05:14 openstackgerrit Surya Seetharaman proposed openstack/nova master: Making instance/migration listing skipping down cells configurable https://review.openstack.org/592428
20:05:15 openstackgerrit Surya Seetharaman proposed openstack/nova master: Return a minimal construct for nova list when a cell is down https://review.openstack.org/567785
20:05:16 openstackgerrit Surya Seetharaman proposed openstack/nova master: Return a minimal construct for nova show when a cell is down https://review.openstack.org/591658
20:05:16 openstackgerrit Surya Seetharaman proposed openstack/nova master: Add scatter-gather-single-cell utility https://review.openstack.org/594947
20:05:17 openstackgerrit Surya Seetharaman proposed openstack/nova master: API microversion bump for handling-down-cell https://review.openstack.org/591657
20:05:17 openstackgerrit Surya Seetharaman proposed openstack/nova master: Return a minimal construct for nova service-list when a cell is down https://review.openstack.org/584829
20:06:27 openstackgerrit Matthew Edmonds proposed openstack/nova master: [WIP] PowerVM: implement instance_on_disk https://review.openstack.org/604204
20:28:34 mriedem jroll: i'll wait until the ironic ci vote shows up, probably in a couple of days
20:29:18 mriedem i've got a change that's been in the gate for over 20 hours
20:59:09 mriedem dansmith: so random question, if i want to poll an instance for it's vm_state to change, i'm guessing Instance.get_by_uuid() is going to be faster since i can control the joins i don't care about in the db, rather than just calling instance.refresh()
20:59:58 mriedem nvm, just talking out loud
21:04:42 dansmith mriedem: yeah I guess probably
21:05:02 mriedem i already know a faster thing
21:05:09 mriedem muwahahaha
21:10:42 openstackgerrit Elancheran S proposed openstack/nova stable/pike: Add exact match aggregate image properties matcher/filter https://review.openstack.org/599870
21:27:06 openstackgerrit Merged openstack/nova stable/queens: Fix the request context in ServiceFixture https://review.openstack.org/599762
21:33:36 openstackgerrit Merged openstack/nova stable/queens: Add functional test for affinity with multiple cells https://review.openstack.org/599765
21:33:42 openstackgerrit Merged openstack/nova stable/queens: Make scheduler.utils.setup_instance_group query all cells https://review.openstack.org/599766
21:51:02 openstackgerrit Merged openstack/nova master: Add contributor guide for upgrade status checks https://review.openstack.org/596902
21:58:42 openstackgerrit Merged openstack/nova master: libvirt: mdevs returning parent and vendor PCI info https://review.openstack.org/562304
22:12:45 mriedem dansmith: do instances get saved as we pass them over rpc? kind of confusing how the instance.az updates in conductor are getting persisted in a few cases b/c we're not calling instance.save() right after we update the az
22:13:08 mriedem e.g. https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L815
22:13:22 dansmith mriedem: the act of sending an instance over rpc does not save it, no
22:13:25 dansmith is that what you're asking?
22:13:26 mriedem hmm,
22:13:28 mriedem yeah,
22:13:35 mriedem so it must be getting passed dirty over the wire,
22:13:39 mriedem and then saved on the compute side
22:13:44 dansmith yep
22:13:55 mriedem either when we're done (vm_state update) or fail (record a fault and revert task _state)
22:14:01 mriedem ok
22:14:02 dansmith yar
22:32:16 mriedem welp,
22:32:20 mriedem i think i've got a thing here
22:32:27 mriedem TODOs aplenty
22:32:57 mriedem can we pretend this is grizzly and i just don't need tests or to worry about rolling back and cleaning up on failure?
22:34:46 mriedem efried: i think i might actually consider thinking about maybe using taskflow
22:35:05 efried mriedem: Or you could use go
22:35:16 mriedem should i just shell out to some go script?
22:35:27 efried what could possibly go wrong?
22:35:30 mriedem i've never written any go
22:35:42 efried I started learning yesterday
22:35:47 efried 'defer' ftw
22:35:51 mriedem for fun?
22:36:03 efried ahem, got "volunteered" for a side project.
22:36:07 mriedem or for PowerK*S?
22:36:13 mriedem *PowerK8S
22:36:16 efried something like that.
22:36:19 mriedem haha
22:36:27 mriedem i'm sorry
22:36:47 mriedem so just uh, you know, get all the powervm stuff done, some placement stuff, some cyborg stuff, etc etc
22:36:50 mriedem kthx
22:58:10 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Cross-cell resize https://review.openstack.org/603930
22:58:12 mriedem dansmith: pretty sure this is ready for production ^
23:35:18 openstackgerrit Brin Zhang proposed openstack/python-novaclient master: Add support changes-before for microversion 2.66 https://review.openstack.org/603549
#openstack-nova - 2018-09-21
00:21:04 openstackgerrit Merged openstack/nova stable/pike: Filter out instances without a host when populating AZ https://review.openstack.org/594184
00:21:12 openstackgerrit Merged openstack/nova stable/ocata: Make ResourceTracker.stats node-specific https://review.openstack.org/588077
00:41:07 openstackgerrit Mohammed Naser proposed openstack/nova master: Default zero disk flavor to RULE_ADMIN_API in Stein https://review.openstack.org/603910
01:40:37 openstackgerrit Tao Li proposed openstack/python-novaclient master: Remove the unused instance-name https://review.openstack.org/602520
02:07:46 gmann mnaser: yeah, there can be more instance like that. as matt mentioned i can add this in my API improvement spec. exact number of fields always in response is better API even with empty value
02:41:28 openstackgerrit huanhongda proposed openstack/nova master: Fix instance delete stuck in deleting task_state https://review.openstack.org/598084
04:35:24 openstackgerrit OpenStack Proposal Bot proposed openstack/nova stable/rocky: Imported Translations from Zanata https://review.openstack.org/604260
04:42:16 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/601047
06:03:05 openstackgerrit Naichuan Sun proposed openstack/nova master: os-xenapi(n-rp)(WIP): add traits for vgpu n-rp https://review.openstack.org/604269
06:17:41 openstackgerrit Tao Li proposed openstack/nova master: Rollback instance vm_state to original where instance claims failed https://review.openstack.org/592252
06:42:59 openstackgerrit YI-JIE,SYU proposed openstack/nova master: Because nova-consoleauth had deprecated since version 18.0.0, fix verify operation documentation file context. https://review.openstack.org/604277
07:05:55 gibi efried: thanks for the fup on the cleanup I've +A-d it
07:13:57 openstackgerrit OpenStack Proposal Bot proposed openstack/nova stable/rocky: Imported Translations from Zanata https://review.openstack.org/604260
07:23:26 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/601047

Earlier   Later