Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-07-24
15:31:39 mriedem s-dean: ok can you file a bug?
15:31:43 s-dean of course
15:31:47 s-dean happy to help
15:31:49 openstackgerrit Matt Riedemann proposed openstack/nova master: Provide hints when nova-manage db sync fails to sync cell0 https://review.openstack.org/486660
15:31:49 mriedem https://bugs.launchpad.net/nova/
15:31:50 mriedem thanks
15:31:54 mriedem s-dean: also ^
15:31:58 mriedem should help with that crappy error message
15:32:39 s-dean ok in the mean time to get this fixed, can i just modify the database_connection string,
15:33:21 s-dean do both cell0 and cell1 need to point at the nova_cell0 ?
15:33:53 mriedem s-dean: you can modify the db yes, but cell0 points at nova_cell0
15:33:59 mriedem and cell1 points at the 'nova' db
15:34:03 s-dean ok perfect
15:34:07 mriedem cell1 is where your compute services and instances live
15:34:08 openstackgerrit Merged openstack/nova master: Update policy description for 'instance_actions' https://review.openstack.org/484609
15:34:10 mriedem bdm table, etc
15:34:16 s-dean i will file a big report thank you so much for your help
15:34:26 s-dean much appreciated
15:34:35 mriedem yw, just ping me here with the bug number later, i'll be afk for awhile - sorry for the headache here
15:34:51 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Microversion 2.53 - services and hypervisors using UUIDs https://review.openstack.org/485435
15:34:52 s-dean will do, take care
15:42:17 ildikov mriedem: hi
15:42:35 ildikov mriedem: so I did some polishing with the tests and I have one single functional test failing
15:42:54 ildikov mriedem: I cannot figure out whether what I do is a completely bad idea or I'm just missing some small thing
15:43:19 ildikov mriedem: ping me if you have a few seconds or just give it a review when you can and then I'll fix things up as you say
15:43:24 ildikov mriedem: either works
15:43:30 ildikov mriedem: thanks in advance
15:44:47 claudiub moshele: eh, it didn't work to whitelist by devname, since the implementation for devname is linux-specific. :)
15:45:13 moshele claudiub: so just add windows support
15:45:47 moshele claudiub: also the pci address is linux spacifc
15:45:55 openstackgerrit Merged openstack/nova master: Updated from global requirements https://review.openstack.org/486434
15:46:31 openstackgerrit Merged openstack/nova master: Dump versioned notifications when test_create_delete_server https://review.openstack.org/486301
15:46:59 openstackgerrit Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/486435
15:49:49 jaypipes edleafe, mriedem: so, I was wrong about that "good spot" comment. That block of code is actually where it needs to be. instance_uuid is defined within that block of code and if I de-indent, I get a local variable instance_uuid referenced before assignment error.
15:53:43 claudiub moshele: cool. will do. anyways. i still have one more question though. The CONF.pci.alias is a list of jsons, but the only valid keys are: name, product_id, vendor_id, and device_type, which doesn't really help me. can we add some other fields, like device_id, or devname?
15:55:05 edleafe jaypipes: I still don't understand how instance_uuids could be None at that point, since you got instance_uuid from it in line 185
15:55:30 moshele claudiub: the pci.alias is used for pci passthrough. you don't need it for SR-IVO with networking
15:55:58 moshele claudiub: isn't product_id and device_id the same
15:56:09 moshele claudiub: ?
15:56:45 claudiub moshele: no. if you have 2 identical devices, they will have the same vendor_id and product_id, but different device_ids
15:56:51 jaypipes edleafe: if I de-indent block 205-215 one level, line 185 may not have run.
15:58:02 jaypipes edleafe: oh, nm, I think I know what you're after.
15:58:09 jaypipes edleafe: duh, sorry... one sec.
15:58:27 edleafe jaypipes: Then don't de-indent. I'm wondering why there is a condition on instance_uuids is not None
15:58:39 jaypipes edleafe: yep, sorry, I was being dense
15:58:56 moshele claudiub: so what is the device_id stand for in windows? in linux we have vendor_id and prodct_id (which is the device_id there is not such thing as product_id in linux)
15:59:26 moshele claudiub: I have to go home but you can send me a mail moshele@mellanox.com with all you questions ..
16:00:23 openstackgerrit Jay Pipes proposed openstack/nova master: claim resources in placement API during schedule() https://review.openstack.org/483566
16:00:24 jaypipes edleafe: ok, sorry about that. done now :) ^^
16:00:31 jaypipes mriedem: ^
16:00:37 claudiub openstackgerrit: sure, thanks. :)
16:01:15 jaypipes claudiub: I don't think openstackgerrit is very responsive to thanks. :P
16:02:14 claudiub ha, openstackgerrit wanted some attention, and jumped in front of my click. :D
16:08:03 openstackgerrit Ed Leafe proposed openstack/nova master: Migrate Ironic Flavors https://review.openstack.org/484949
16:08:17 edleafe dansmith: ^^ hope this is closer to what you want
16:08:48 dansmith edleafe: I was about to ask ... I'll go have a look
16:09:01 dansmith edleafe: the move from eventlet is purely a nova-api thing for wsgi stuff, fwiw
16:10:03 openstackgerrit Chris Friesen proposed openstack/nova master: Add validation for cpu_realtime_mask https://review.openstack.org/468203
16:10:04 openstackgerrit Chris Friesen proposed openstack/nova master: Tweak the cpu_realtime_mask handling slightly https://review.openstack.org/461456
16:14:44 mdbooth kashyap: https://review.openstack.org/#/c/485752/ Can you remind me where we rewrite the guest xml post migration?
16:15:44 mdbooth kashyap: NM, I didn't read your commit message carefully enough
16:27:48 openstackgerrit melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946
16:29:13 melwitt mriedem: your comments addressed ^
16:31:05 s-dean mriedem: https://bugs.launchpad.net/nova/+bug/1706118
16:31:06 openstack Launchpad bug 1706118 in OpenStack Compute (nova) "Adding TLS Connection String To Database Connection Causes malformed URL in cell_mappings DB" [Undecided,New]
16:32:38 edleafe dansmith: still not clear how nodes could be batched in the ironic driver. Wouldn't that require some way of persisting what has been done already?
16:47:37 melwitt mriedem: I went to change the counting quotas blueprint to Implemented and it automatically changed it to Complete (I didn't know it would do that) https://blueprints.launchpad.net/nova/+spec/cells-count-resources-to-check-quota-in-api
16:48:12 melwitt sorry about that
16:50:13 melwitt oh, it let me change it back
16:50:46 melwitt phew.
16:52:53 dansmith edleafe: by batching I mean do 25 at a time, in that thread
16:53:20 dansmith edleafe: like for i in range(0, len(uuids), 25): do_instances(uuids[i:i+25])
16:56:24 edleafe batching which? I'm going through the nodes 1 by 1, and creating the instances one at a time
16:56:53 edleafe I thought that doing it that way would satisfy your memory explosion concern
16:58:42 dansmith edleafe: it does, but now we're hammering the database and conductor with one-by-one queries on every single boot
17:16:37 melwitt dansmith, mriedem: I was wondering whether we're gonna try to get this list instances across cells merge done this week? https://review.openstack.org/#/c/463618/
17:17:03 dansmith melwitt: I talked to mdbooth last week and it sounded like it wasn't going to happen
17:17:15 dansmith melwitt: especially since to do it right I think we'll need some refactoring of our db layer
17:17:49 openstackgerrit Sean Dague proposed openstack/nova master: Suppress some test warnings https://review.openstack.org/486700
17:18:10 melwitt dansmith: oh. I took a quick look at the patch but wasn't sure the detail of what's left to do with it and whether one of us can finish it up
17:18:32 melwitt one of us could be me, I mean
17:18:38 dansmith well, we could, but I don't think it's a reasonable thing to throw in at the last minute
17:19:14 melwitt k
17:21:41 openstackgerrit Chris Friesen proposed openstack/nova master: Ensure we unshelve in the cell the instance is mapped https://review.openstack.org/486208
17:22:29 cfriesen I added some assertions to existing tests here ^....if you want entirely new tests I can do that, but I wasn't sure it was worth the overhead.
17:35:01 s-dean hi, any pointers on how i may solve this issue ? MessagingTimeout: Timed out waiting for a reply to message ID
17:35:17 s-dean its telling me to submit a bug report
17:35:33 s-dean sorry not it> Nova*
17:36:27 s-dean nova.api.openstack.extensions [req-3ed5c135-64bd-4c39-934d-3d3d7a32a6b5 14d1055c2e2d4f0eab6739671b2688a8 cf7b827f55de4406b9325ab3505b973d - default default] Unexpected exception in API method
17:39:38 melwitt s-dean: I think that's an RPC timeout, so you will want to check your transport_url for your cell1 and make sure it's correct
17:40:07 s-dean ok cheers will do that now
17:49:09 s-dean yeah, cell1's RPC mapping was wrong, i have changed it and restarted all nova services, no dice :/
17:49:19 s-dean still same error
17:50:17 s-dean can i ask would it be better to implement SSL/TLS after successfully setting up openstack unencrypted ?
17:50:27 dansmith s-dean: I'm not following along, but you're sure that the cell1 mapping points to the same rpc endpoint that your other services do?
17:51:51 melwitt s-dean: did you change it using "nova-manage cell_v2 update_cell"? as shown https://docs.openstack.org/nova/latest/cli/nova-manage.html#man-page-cells-v2
17:51:54 mriedem melwitt: i think https://blueprints.launchpad.net/nova/+spec/cells-count-resources-to-check-quota-in-api is done
17:52:05 mriedem https://review.openstack.org/#/c/446243/ was the last change to merge
17:52:24 s-dean no i did not
17:52:59 melwitt s-dean: yeah you need to use the nova-manage commands to get it set right in the database. you can use "nova-manage cell_v2 list_cells --verbose" to see what you currently have

Earlier   Later