| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-15 | |||
| 19:17:27 | prometheanfire | sean-k-mooney: cool, if that works we can just depend on that | |
| 19:18:28 | sean-k-mooney | prometheanfire: ya i think so i was going to update https://review.openstack.org/#/c/532987/1 also with a depends on. just need to copy some ssh keys as the vm im using has not been used for dev before. | |
| 19:18:47 | prometheanfire | sean-k-mooney: wgm | |
| 19:18:48 | prometheanfire | wfm | |
| 19:19:03 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: SEA https://review.openstack.org/523216 | |
| 19:19:16 | melwitt | dansmith: do you think multiple cells with a global MQ could be a thing? I can't recall if you ever mentioned that in your cells talks | |
| 19:19:26 | melwitt | asking in reference to this review https://review.openstack.org/#/c/518556 | |
| 19:19:39 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094 | |
| 19:20:02 | dansmith | melwitt: you can't really because a compute will ask for conductor to do something, and you might get a conductor from your cell and you might not | |
| 19:20:34 | melwitt | dansmith: oh, yeah ... I didn't think of that. okay, cool | |
| 19:22:43 | openstackgerrit | sean mooney proposed openstack/nova master: make unit tests compatible with os-vif 0.8.0 https://review.openstack.org/533766 | |
| 19:23:12 | sean-k-mooney | ... that should be 1.8.0 | |
| 19:23:41 | openstackgerrit | sean mooney proposed openstack/nova master: make unit tests compatible with os-vif 1.8.0 https://review.openstack.org/533766 | |
| 19:26:29 | sean-k-mooney | prometheanfire: is it Depend-On or Depends-On: | |
| 19:28:23 | sean-k-mooney | prometheanfire: its Depends-On: .. one sec | |
| 19:30:32 | sean-k-mooney | stephenfin: prometheanfire https://review.openstack.org/#/c/533766/ + https://review.openstack.org/#/c/532987/ should now hopefully work | |
| 19:30:51 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: SEA https://review.openstack.org/523216 | |
| 19:31:07 | prometheanfire | cool | |
| 19:32:38 | sean-k-mooney | prometheanfire: the issue was only in the unit tests. really execution paths would have worked so this is just a minor change. | |
| 19:39:01 | edleafe | golang flag parse | |
| 19:39:17 | edleafe | wrong channel? | |
| 19:49:28 | cdent | no edleafe, we've started, nova is going go | |
| 19:53:33 | sean-k-mooney | cdent: is that toung and cheek or have we decided nova is not hipster enought lol | |
| 19:54:03 | cdent | if we wanted to be hipster, go is surely too mainstream | |
| 19:54:29 | edleafe | nova is hipster only ironically | |
| 19:55:21 | sean-k-mooney | cdent: but the alternitive would be rust and that too painful | |
| 19:55:48 | edleafe | Ironic should be written in rust | |
| 19:56:02 | cdent | of course | |
| 19:56:22 | sean-k-mooney | edleafe: are you a parent because that right there is a dad joke | |
| 19:56:48 | edleafe | oh, I could dad-joke you to death | |
| 20:16:07 | jaypipes | edleafe: is it wrong that I dad-joke but have no children (that I know of) | |
| 20:17:45 | edleafe | jaypipes: without kids it | |
| 20:17:52 | edleafe | it's just bad-joke | |
| 20:17:54 | cdent | jaypipes: your brother is your twin, yeah? maybe that's it? | |
| 20:25:37 | jaypipes | cdent: yes. yes, that's it. I'll blame him. | |
| 20:26:11 | cdent | always happy so sow brotherly discord | |
| 20:27:08 | openstackgerrit | Matthew Edmonds proposed openstack/nova master: Add taskflow to requirements https://review.openstack.org/533779 | |
| 20:58:27 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408 | |
| 21:10:50 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Refactor WSGI apps and utils to limit imports https://review.openstack.org/533797 | |
| 21:11:11 | cdent | efried, edleafe, jaypipes: you might find that ^ interesting. Not done yet, but a potentially useful tidy up | |
| 21:11:15 | cdent | will get back to it tomorrow | |
| 21:13:09 | jaypipes | bauzas: https://review.openstack.org/#/c/528832/9/nova/virt/libvirt/config.py :) | |
| 21:13:22 | jaypipes | cdent: ack | |
| 21:13:42 | jaypipes | bauzas: maybe 10th time is the charm? ;P | |
| 21:15:35 | jaypipes | ok, now on to efried' | |
| 21:15:38 | jaypipes | s series :) | |
| 21:20:36 | bauzas | jaypipes: oh fuuuuuuu | |
| 21:20:53 | bauzas | childs is because I stupidly copied some other patch :p | |
| 21:21:03 | bauzas | I have two children | |
| 21:21:06 | bauzas | so I know that :p | |
| 21:22:02 | jaypipes | bauzas: :) | |
| 21:24:50 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Recreate mediated devices on reboot https://review.openstack.org/533642 | |
| 21:24:50 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: create vGPU for instance https://review.openstack.org/528832 | |
| 21:24:57 | bauzas | hop là | |
| 21:25:54 | bauzas | dansmith: just in case you're still around, I'm thinking about hard reboot | |
| 21:26:28 | bauzas | dansmith: I could just ask for the existing mdev before destroying the guest, and then pass it back to _get_guest_xml() like I said | |
| 21:27:45 | bauzas | dansmith: but then, that means that for example, if we have a lot of months in between stopping the instance and restarting it, then we won't verify if that's still okay | |
| 21:28:04 | bauzas | the reason actually why we try to retry the xml | |
| 21:28:14 | bauzas | and not just reusing it | |
| 21:28:31 | bauzas | if we agree with that, fine | |
| 21:28:41 | bauzas | if we don't want that, then we would need to pass again the allocation | |
| 21:28:46 | bauzas | thoughts ? | |
| 21:29:10 | sean-k-mooney | jaypipes: bauzas melwitt dansmith when ye get a chance can ye take a look at https://review.openstack.org/#/c/533766/ to unblock https://review.openstack.org/#/c/532987/ | |
| 21:29:54 | bauzas | sean-k-mooney: easy +2 | |
| 21:30:18 | bauzas | oh wait | |
| 21:30:36 | bauzas | sean-k-mooney: what is the min os-vif version for Nova ? | |
| 21:30:59 | sean-k-mooney | currently 1.6 i think let me check | |
| 21:31:45 | sean-k-mooney | 1.7.0 | |
| 21:34:05 | bauzas | sean-k-mooney: -1 because https://github.com/openstack/os-vif/blob/1.7.0/os_vif/objects/vif.py | |
| 21:34:26 | jaypipes | why do we have open-ended dependencies like >=1.7.0 again? :( | |
| 21:35:08 | sean-k-mooney | well we have a catch 22 then | |
| 21:35:22 | sean-k-mooney | the change that increases the upper constriants need this change | |
| 21:36:08 | sean-k-mooney | jaypipes: they are closed by upper-constratints | |
| 21:36:29 | bauzas | sean-k-mooney: before merging the nova change, we should IMHO first bump the min version | |
| 21:36:50 | bauzas | sean-k-mooney: is g-r accepting >= 1.8.0 ? | |
| 21:36:59 | bauzas | if so, just change directly nova requirements.txt | |
| 21:37:04 | bauzas | in the same change | |
| 21:37:14 | bauzas | if not, you need first to modify g-r | |
| 21:37:51 | sean-k-mooney | bauzas: no g-r is os-vif>=1.7.0 u-c is os-vif===1.7.0 | |
| 21:38:23 | sean-k-mooney | we cant increase g-r without first increaseing u-c and u-c need that patch | |
| 21:42:52 | bauzas | sean-k-mooney: mmm, looks like a Depends-On change then | |
| 21:43:26 | sean-k-mooney | the depends-on is currently from https://review.openstack.org/#/c/532987/ to https://review.openstack.org/#/c/533766/2 | |
| 21:43:27 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Consider hostdev devices when building metadata https://review.openstack.org/533805 | |
| 21:43:27 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Refactor _build_device_metadata https://review.openstack.org/533804 | |
| 21:43:35 | bauzas | sean-k-mooney: I guess the gate is not accepting to upgrade upper-constraints because it tests nova ? | |
| 21:43:46 | sean-k-mooney | yep | |
| 21:43:48 | bauzas | shit | |
| 21:44:11 | sean-k-mooney | we could pin os-vif in nova then increase the upperconstraints then fix the test and bump minium | |
| 21:44:26 | sean-k-mooney | and unpin | |
| 21:45:15 | bauzas | sean-k-mooney: I think we can possibly ask nova to bump os-vif to 1.8.0 and in the meantime have the g-r change that bumps both global-reqs.txt and upper-constraints.txt | |
| 21:45:37 | bauzas | since there will be a depends-on, that should gate correctly | |
| 21:46:03 | bauzas | but I'm not a requirements expert | |
| 21:46:05 | bauzas | tonyb: around ? | |
| 21:46:30 | bauzas | tonyb: questions for a requirements upgrade loop | |
| 21:46:49 | sean-k-mooney | the change that fixes the tests for 1.8 also works on 1.7 because of weird ness with ovo hence why the patch is currently passing zuul without using os-vif 1.8.0 | |
| 21:48:42 | bauzas | sean-k-mooney: no I think the nova change uses 1.8.0 | |
| 21:48:57 | sean-k-mooney | basically https://github.com/openstack/nova/blob/6a5673a16f8329007880b67a6dbecbe62ac368d7/nova/tests/unit/network/test_os_vif_util.py#L34-L38 means that the extra datapath_type field is discarded | |
| 21:48:57 | bauzas | sean-k-mooney: because pypi will give you the last version | |
| 21:49:11 | sean-k-mooney | bauzas: no i checked locally it was using 1.7.0 | |
| 21:49:25 | sean-k-mooney | let me check the test logs | |