Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-07-13
14:56:58 edleafe jaypipes: I've bookmarked that quote
14:58:24 jaypipes edleafe: ha! it was paraphrasing White Goodman from "Dodgeball: A True Underdog Story" (http://www.imdb.com/title/tt0364725/quotes)
14:58:38 jaypipes "I know you. You know you. And I know you know that I know you."
14:58:51 edleafe oh, not that quote. http://p.anticdent.org/38I
14:59:19 jaypipes edleafe: ooohhhhh, that one.
14:59:31 edleafe jaypipes: :)
15:02:52 mriedem stvnoyes: i've gone through the compute manager changes on https://review.openstack.org/#/c/463987/ - there is a definite bug in the way that _detach_volume is called
15:03:32 mriedem stvnoyes: beyond that, i'm concerned that since we don't always call rollback_live_migration_at_destination, we might not cleanup properly
15:04:52 mordred efried_bbiab, mriedem: ping me when you're ready
15:25:01 edleafe jaypipes: let me check my understanding of compute-land with you
15:25:13 edleafe jaypipes: There is a compute service. It has a single resource tracker. That RT can have 1 or more compute nodes
15:25:30 edleafe jaypipes: Normally it's one (VMs), but for Ironic it can be (and usually is) >1
15:25:36 edleafe jaypipes: is that correct?
15:25:40 jaypipes yup
15:25:56 edleafe jaypipes: ok thx
15:25:58 dansmith correct in the technical sense
15:26:04 jaypipes edleafe: always a single resource tracker. that resource tracker tracks >1 compute node objects in case of ironic.
15:26:05 dansmith not correct in the moral should-this-be-done sense
15:26:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
15:26:28 edleafe dansmith: my soul is already unclean
15:26:52 bauzas edleafe: because jaypipes fixed that :)
15:27:10 bauzas edleafe: we previously had 1 RT per compute "node"
15:27:21 bauzas <3 jaypipes
15:28:19 edleafe bauzas: get a room, you two
15:28:45 jaypipes lol
15:29:05 isantosp Hi, I have an issue on nova newton, im trying to set ipv6 taking it from the port creation in neutron newton, apparently neutron is doing his job perfectly, the problem is when nova try to call port_crete_minimal, I have the following: ```
15:29:09 bauzas edleafe: you mean #openstack-loves-jaypipes IRC room ? :p
15:29:33 isantosp subnet_id_v4 = host_subnets['host']['available_random_subnet_v4']
15:29:35 isantosp port_req_body['port']['fixed_ips'] = [
15:29:37 isantosp {'subnet_id': str(subnet_id_v4)}]
15:29:39 isantosp yes
15:29:55 edleafe bauzas: there are several such rooms, I imagine
15:30:40 isantosp In the creation of the instance, i see this error: {"message": "'subnets'", "code": 500}
15:31:07 dansmith melwitt: I think you should roll on with the update to that patch. I'm to the tests, and my will to live is waning
15:31:35 isantosp and this one in the logs
15:31:38 isantosp File "/usr/lib/python2.7/site-packages/nova/network/neutronv2/api.py", line 392, in _create_port_minimal
15:31:40 isantosp host_subnets = port_client.show_host(instance.host)
15:31:42 isantosp KeyError: 'subnets'
15:32:46 isantosp any ideas? the line 392 that is pointed there, it has nothing to solve
15:36:15 openstackgerrit Alex Szarka proposed openstack/nova master: Transform the transformed notifications functional tests https://review.openstack.org/483448
15:40:34 sean-k-mooney jangutter: well what specifically is the gap. you can use https://github.com/Juniper/contrail-nova-vif-driver/blob/master/vif_plug_vrouter/vrouter.py as an example of an external plugin
15:40:47 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
15:43:10 sean-k-mooney jangutter: for the sriov type you should call your plugins plug method with a VIFHostDevice and for the virtio forworder case you should use VIFVHostUser with a costom port profile if you need to pass vendor specific info.
15:43:54 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
15:44:01 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
15:45:22 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926
15:45:31 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926
15:48:17 openstackgerrit Alex Szarka proposed openstack/nova master: Transform the transformed notifications functional tests https://review.openstack.org/483448
15:48:27 sean-k-mooney jangutter: was your question more how do i tell nova to allocate the vf and call the plugin?
15:49:16 sean-k-mooney jangutter: without a review too look at its a little hard to advise what do next
15:52:31 openstackgerrit Alex Szarka proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660
15:52:50 openstackgerrit Alex Szarka proposed openstack/nova master: Transform instance.resize.error notifications https://review.openstack.org/417882
15:53:09 mriedem jaypipes: dansmith: can one of you put the priority reviews docs patch through https://review.openstack.org/#/c/470420/
15:53:45 dansmith sure
15:53:58 jangutter sean-k-mooney: your wish is my command, just as we speak the clearance came through.
15:54:52 jangutter So my next newbie question to the room is: should I throw away the old reviews and create new ones, or replace them? what's the status quo?
15:55:52 mriedem dansmith: https://review.openstack.org/#/c/445142/ also needs a rebase
15:56:29 dansmith mriedem: ac
15:56:30 dansmith k
15:56:31 sean-k-mooney jangutter: cool. am it really up to you. you can keep the change ids if the new code is a nautal evolution of the previous patches or abandon the old patchs and leave a link to the new patchset if its substially different
15:57:38 jaypipes jangutter: just Abandon the old patches.
15:57:45 jaypipes jangutter: patches are cheap :)
15:58:27 jangutter *sniff* they're going to data heaven now.
15:59:06 sean-k-mooney jangutter: openstack never delete the review so they will live forever in bit rot land
15:59:32 jangutter *sniff* they're going to data purgatory now.
16:01:21 openstackgerrit Merged openstack/nova-specs master: Document Pike release review priorities https://review.openstack.org/470420
16:01:36 sean-k-mooney jangutter: jaypipes apparently this is the oldest review still visable https://review.openstack.org/#/c/131 jay you were a core on keystone once?
16:03:15 breton lol. Even Dolph was not a core.
16:03:31 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Add datapath type information to OVS vif objects https://review.openstack.org/474892
16:03:54 jaypipes sean-k-mooney: lol
16:07:46 sean-k-mooney jangutter: am when you push the new patches add me to them and ill take a look at what you have end to end
16:12:47 jaypipes jangutter: yes, purgatory is more appropriate for the patches that I personally abandon (there have been many, many of them)
16:13:04 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
16:15:29 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.52 - List/Show all server migration types https://review.openstack.org/430839
16:16:00 openstackgerrit Jan Gutter proposed openstack/nova master: Netronome SmartNIC Enablement https://review.openstack.org/483459
16:17:16 jangutter Warning: these patches have been ripped from the Netronomicon.
16:17:47 openstackgerrit Alex Szarka proposed openstack/nova master: Refactor create_delete_server_with_instance_update https://review.openstack.org/466296
16:20:52 jangutter jaypipes, sean-k-mooney: thanks very much for the infinite patience. The ML2/OS-VIF external plugin's on github, and I'll upload the neutron bits in an hour or so.
16:23:54 jaypipes jangutter: no problem at all.
16:33:26 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.52 - Enable cold migration with target host https://review.openstack.org/406707
16:35:31 mriedem bauzas: ok email to the ML sent about bps close to done
16:36:00 openstackgerrit zhang zhenzhong proposed openstack/nova master: fix resource-provider regist failed in db while placement endpoint created after nova-compute start https://review.openstack.org/483460
16:38:22 openstackgerrit zhang zhenzhong proposed openstack/nova master: fix resource-provider regist failed in db while placement endpoint created after nova-compute start https://review.openstack.org/483460
16:45:47 mriedem jaypipes: https://review.openstack.org/#/c/482383/ is the current focus right now right?
16:45:55 mriedem it's not https://review.openstack.org/#/c/482381/
16:46:02 mriedem the dep tree is confusing me
16:47:10 jaypipes mriedem: I needed to go in a different direction on those. I will abandon both.
16:47:20 mriedem ok
16:47:31 mriedem well claim_resources() is +W so yay
16:47:39 jaypipes yeah, that's fine.
16:47:41 jaypipes self contained.
16:47:51 jaypipes should have new series pushed today.
16:47:54 mriedem ack
16:59:25 cdent use as you see fit
17:00:34 jaypipes cdent: you can keep the spare pug. just double the jack.
17:00:42 cdent noted
17:00:52 cdent I’ve kinda sorta always wanted a pug of my very own
17:01:14 openstackgerrit zhang zhenzhong proposed openstack/nova master: fix rp regist failed while placement endpoint created after nova-compute start https://review.openstack.org/483460
17:03:40 jaypipes cdent: they are great, in moderation.
17:03:47 jaypipes cdent: three is my limit.

Earlier   Later