Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-07-24
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
17:53:12 s-dean ok cheers
17:53:22 melwitt these are stored in the database now because the API needs to know how to contact each of the cells you have
17:53:58 melwitt mriedem: agreed, I just didn't mean to set the Completed state because I think usually you do that
17:54:12 melwitt but it does it automatically if you put Implemented
17:55:05 mriedem melwitt: i like to yeah because i have a spreadsheet keeping track of when things are closed out
17:55:08 mriedem b/c i'm a nerd
17:55:35 melwitt mriedem: heh. I would probably do the same thing. don't want to mess up your system
17:56:01 mriedem just please be sure to put scissors, pens and dusters back where you got them if you're ever at my house
17:56:13 mriedem and windex
17:56:17 melwitt lol
18:00:05 s-dean do ineed to sync the db's after running that update command
18:00:32 melwitt s-dean: no, just need to restart your nova-api services I think
18:00:52 dansmith unless he never properly sync'd it as a result
18:01:15 s-dean yeah, still timing out
18:01:36 melwitt hm
18:02:14 openstackgerrit Matt Riedemann proposed openstack/nova master: Provide hints when nova-manage db sync fails to sync cell0 https://review.openstack.org/486660
18:03:18 dansmith s-dean: so if you grep transport_url out of your configs on all nodes, they match exactly what is in the db for the cell1 transport_url?
18:06:35 s-dean ok so i can confirm, that on my controller node the URL's are correct, however on the compute node it is not because of: https://docs.openstack.org/security-guide/messaging/security.html
18:07:32 mriedem sdague: did you ever report a bug for the request log thing with uwsgi?
18:07:52 sdague mriedem: I don't think I did actually
18:10:07 mriedem sdague: ok, did the same thing need to be done in other projects like cinder?
18:10:18 mriedem if so, could be good to track with a bug for those other projects too
18:13:50 dansmith s-dean: so...the compute node is not correct?
18:14:00 s-dean does the username need to be consistent across all nodes, or is having different users for each compute node ok ?
18:14:13 s-dean yes i gave the compute node its on users
18:14:15 s-dean user
18:14:23 s-dean and assigned it all permissions
18:14:31 s-dean own*
18:14:59 dansmith s-dean: they can be different, they just all have to point at the same vhost of course
18:15:12 s-dean in my case my vhost is /

Earlier   Later