Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-09-20
17:43:08 cdent lucidguy: so the is issue is that you've got the allocations showing up on two different resource providers and it should just be one. If that's the case, edit the json to remove the block identified by the wrong one, and PUT it back to the same URL
17:43:14 bauzas dansmith: and then we call again this method by passing an allocations param
17:43:31 bauzas so we only reshape when allocations is non-None
17:43:50 bauzas the other cases are then periodic
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

Earlier   Later