Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-10-10
20:22:15 jgriffith_ mriedem and yes, you're correct
20:22:17 mriedem i don't know where it's coming from
20:24:10 ildikov mriedem: regardless of how refresh is behaving, there still doesn't seem to be a logical path to that attachment_update call :/
20:25:58 mriedem L269 https://etherpad.openstack.org/p/cinder-ptg-queens
20:26:02 mriedem apparently my notes were removed...
20:26:21 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/511021
20:28:00 ildikov mriedem: https://etherpad.openstack.org/p/cinder-ptg-queens-thursday-notes
20:28:09 ildikov L121
20:28:41 mriedem whew
20:28:43 mriedem thanks
20:29:04 mriedem "With the new style Cinder API, does nova.virt.block_device.DriverVolumeBlockDevice.refresh_connection_info change? Do we do an attachment_show or do we delete the old attachment and create a new one (or does that only happen on a HARD reboot - if we make that change)?"
20:29:36 mriedem "if bdm.attachment_id then we should fetch connection info from Cinder (on hard reboot) What I heard is this actually needs to be an attachment_delete/create swap thing because that's what gives us the connection_info from the storage backend. Although I wonder if attachment_update would refresh the connection_info on the cinder side...since attachment_update == os-initialize_connection."
20:33:53 ildikov not sure that's fully equal
20:34:23 ildikov I'm always open to be corrected by jgriffith_ :)
20:36:17 ildikov but with reboot we call the _get_instance_block_device_info with default on refresh, which is False
20:37:53 mriedem ildikov: what i'm going to do is just push a debug patch on top of john's patch which will dump a stacktrace if we call attachment_update while rebooting an instance
20:38:12 ildikov mriedem: sounds good, thank you
20:38:13 mriedem i've spent way too much time looking at logs today
20:38:23 ildikov mriedem: sorry :(
20:38:32 mriedem nah it's fine,
20:38:38 mriedem i just need to let the computer do some work
20:38:51 ildikov mriedem: I swear I just rebased the thing and bumped the service version :S
20:39:14 ildikov mriedem: +1 on the last comment :)
20:42:27 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements https://review.openstack.org/493146
20:48:35 openstackgerrit Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
20:50:01 openstackgerrit Matt Riedemann proposed openstack/nova master: DNM: Fail if we try to update volume connections during reboot https://review.openstack.org/511031
20:54:51 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/511021
20:56:56 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
21:08:42 cdent edleafe: in the commit message on https://review.openstack.org/#/c/510625/4 there are some questions about possible todos, did you have an opinion on those?
21:09:34 edleafe cdent: you mean about including the project_id/user_id in the GET?
21:09:44 cdent yeah
21:09:58 edleafe I don't see why we shouldn't
21:10:38 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/511021
21:11:10 coreycb hi all, cells v2 question
21:11:38 cdent edleafe: does that mean I should put a -1 or -W or a something on there?
21:12:40 coreycb i'm hitting a window where after 'nova-manage cells_v2 list_cells' starts returning the cell i'm interested in, mapping hosts or instances for that cell fails during the window until the database is finished. is that expected?
21:12:44 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
21:13:58 mriedem coreycb: until the database is finished with what?
21:16:29 openstackgerrit Chris Dent proposed openstack/nova master: Move project_id and user_id to Allocation object https://review.openstack.org/500410
21:16:30 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Allow _set_allocations to delete allocations https://review.openstack.org/501051
21:16:31 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Limit number of attempts to delete allocations https://review.openstack.org/507224
21:16:32 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Enable PUT dict to /allocations/{consumer_uuid} https://review.openstack.org/510625
21:16:33 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Document the changes to PUT /allocations/{consumer_uuid} https://review.openstack.org/510626
21:16:34 openstackgerrit Chris Dent proposed openstack/nova master: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073
21:17:37 edleafe cdent: You mean so they can be in the same microversion? Meh, they're cheap. You can do it in a follow-up patch if you like
21:18:24 coreycb mriedem: it seems to still be initializing the DB from prior nova-manage commands. here's some output of 'list_cells' looping and then running 'discover_hosts' once the cell starts showing up in list_cells output: https://launchpadlibrarian.net/339984742/printscreen
21:18:44 cdent edleafe: no, not worried about the microversion issue, just that somewhere it needs to be marked as we’re not done here. If it is to be done, I’d be inclined to do it in a prior patch.
21:18:53 cdent i’ll leave a comment-to-self
21:19:09 coreycb mriedem: search for 'cell1'
21:20:27 mriedem coreycb: Table 'nova.compute_nodes' doesn't exist
21:20:40 mriedem you can't run discover_hosts or map_instances unti you've synced the dbs on the cells
21:20:55 mriedem meaning, you need to run nova-manage db sync on each cell database
21:21:27 mriedem https://docs.openstack.org/nova/pike/user/cells.html#setup-of-cells-v2
21:22:09 mriedem "At this point, the API database can now find the cell database, and further commands will attempt to look inside. If this is a completely fresh database (such as if you’re adding a cell, or if this is a new deployment), then you will need to run nova-manage db sync on it to initialize the schema."
21:22:16 coreycb mriedem: i think we do that, we basically follow the order of what's in doc/source/cells.rst
21:22:35 coreycb mriedem: it's just a small window and then you'll see at the end of that output discover_hosts works.
21:23:37 mriedem coreycb: so there is a separate process that's calling create_cell and doing the db sync on each cell?
21:28:31 coreycb mriedem: no it's same process, but we don't target the cell on the db sync if that's possible
21:29:15 mriedem nova-manage db sync will sync the current cell db (so cell1 if you're running it on a nova.conf with [database] configured for cell1) and it will query the nova_api database looking for cell0 and sync that as well
21:29:30 mriedem that has to happen before you start discovering hosts in cell1
21:30:53 coreycb mriedem: ok. yes we're doing that. basically we do steps 1-6 of the document you linked, and then if i run list_cells and discover_hosts in a loop, there will be a window (i've not timed it, maybe 30 seconds?) where list_cells returns 'cell1' but discover_hosts fails. then after eventually discover_hosts works.
21:31:27 coreycb mriedem: just curious really if that's normal behavior
21:33:26 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/511021
21:33:33 mriedem well i can tell from that dump that you're trying to discover_hosts before cell1 db schema is fully sync'ed
21:33:45 mriedem because
21:33:45 mriedem ProgrammingError: (pymysql.err.ProgrammingError) (1146, u"Table 'nova.compute_nodes' doesn't exist")
21:34:21 coreycb mriedem: right. my impression from that is that nova-manage commands aren't necessarily finished when they return. but i'm guessing.
21:34:59 mriedem db sync is synchronoous
21:35:36 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
21:36:38 coreycb mriedem: ok that's good to know. if that's the case it must be something on my end then.
21:36:54 coreycb mriedem: thanks
21:36:59 mriedem np
21:38:41 rm_work hey, when an instance is live-migrated, does nova update the AZ in the database / api response?
21:39:11 rm_work I *assume* yes, but it's one of those things I'd just like to be sure of
21:44:02 mriedem dansmith: remind me, new style migration allocation things, the dest node allocation is held by the instance, and the source node allocation is held by the migration, correct?
21:44:12 dansmith yes
21:44:25 openstackgerrit Jackie Truong proposed openstack/python-novaclient master: Microversion 2.54 - Add trusted_image_certificates https://review.openstack.org/500396
22:08:38 mriedem dansmith: ok comments in https://review.openstack.org/#/c/498949/
22:34:41 openstackgerrit Merged openstack/nova master: Transform aggregate.remove_host notification https://review.openstack.org/396211
23:07:04 openstackgerrit Hongbin Lu proposed openstack/nova master: [WIP] Handle not found error on taking snapshot https://review.openstack.org/511074
23:09:12 openstackgerrit Hongbin Lu proposed openstack/nova master: [WIP] Handle not found error on taking snapshot https://review.openstack.org/511074
23:09:13 openstackgerrit Merged openstack/nova master: Ensure instance can migrate when launched concurrently https://review.openstack.org/506093
#openstack-nova - 2017-10-11
00:27:08 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements https://review.openstack.org/493146
00:39:53 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements https://review.openstack.org/493146
00:52:42 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements https://review.openstack.org/493146
01:12:40 Kevin_Zheng mriedem, hi, for https://review.openstack.org/#/c/509326/
01:16:26 Kevin_Zheng I responded some comment, as for the neutron part, I submitted a RFE and seems Amando is fine with it and they will discuss it at the driver meeting this friday: https://bugs.launchpad.net/neutron/+bug/1718605, how should I put it in the spec?
01:16:27 openstack Launchpad bug 1718605 in neutron "[RFE] Support regex matching when filtering port with IP address" [Wishlist,Triaged] - Assigned to Zhenyu Zheng (zhengzhenyu)
01:17:38 openstackgerrit Jay Pipes proposed openstack/nova master: placement: integrate ProviderTree to report client https://review.openstack.org/415921
01:17:38 openstackgerrit Jay Pipes proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138
01:17:39 openstackgerrit Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215
01:17:39 openstackgerrit Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807
01:17:40 openstackgerrit Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
02:32:41 openstackgerrit Merged openstack/nova master: Fix minor input items from previous patches https://review.openstack.org/506416
03:39:07 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/511021
03:41:11 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
05:50:02 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/511021
05:52:13 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
06:53:48 openstackgerrit Elod Illes proposed openstack/nova master: WIP: Transform scheduler.select_destinations notification https://review.openstack.org/508506
07:22:24 openstackgerrit Elod Illes proposed openstack/nova master: WIP: Transform scheduler.select_destinations notification https://review.openstack.org/508506

Earlier   Later