Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-12-18
14:52:24 stephenfin bauzas: Aye. Figure out if it's something we could do with libvirt internals and propose the change upstream if possible
14:52:38 bauzas I dunno
14:52:47 bauzas lemme write something and we'll discuss
14:52:53 stephenfin Neither do I. Just as a suggestion
14:53:05 stephenfin Given that I imagine other people would benefit if libvirt had it too
15:07:31 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Make request_spec.spec MediumText https://review.openstack.org/528330
15:07:34 mriedem artom: added the release note to the pike backport
15:07:36 mriedem lyarwood: ^
15:11:14 leakypipes bauzas: sorry, went for breakfast... reading back now
15:11:40 bauzas leakypipes: no worries, just trying a way to not have a huuuuuuge tech debt because $libvirt
15:11:57 bauzas leakypipes: maybe a hangout would be cool for discussing
15:11:59 leakypipes bauzas: ack.
15:12:09 leakypipes bauzas: gimme about 10 minutes and I can do that, sure
15:12:30 bauzas leakypipes: I'll need to get my girls in 15 but we can discuss that later
15:12:35 leakypipes np
15:13:11 bauzas or prozac
15:13:22 leakypipes lol :)
15:13:36 openstackgerrit Matt Riedemann proposed openstack/nova stable/ocata: Make request_spec.spec MediumText https://review.openstack.org/528332
15:18:22 openstackgerrit Matthew Booth proposed openstack/nova master: Pass DriverBlockDevice to driver.attach_volume https://review.openstack.org/528363
15:18:22 openstackgerrit Matthew Booth proposed openstack/nova master: Expose driver_block_device fields as attributes https://review.openstack.org/528362
15:18:39 mdbooth lyarwood: That second one ^^^^
15:21:10 mdbooth lyarwood: It's a bit disruptive in tests, but amongst its advantages is that it will expose both attachment_id and connection_info without further changes.
15:21:25 mdbooth (Also bdm.uuid, which is what I want it for)
15:23:52 openstackgerrit Matt Riedemann proposed openstack/nova stable/newton: Make request_spec.spec MediumText https://review.openstack.org/528338
15:24:48 artom mriedem, ah, thanks
15:25:34 artom mriedem, and https://review.openstack.org/#/c/528712/ passed Zuul, so I think we're good, yeah
15:26:57 mriedem ok i've got the ocata and newton ones updated too
15:27:49 openstackgerrit Mr Rambo proposed openstack/nova master: Fix the problems that volume-backed server rebuild https://review.openstack.org/528740
15:29:41 artom mriedem, I'll push a more complete reno if you don't mind, mentionning that the migration is optional unless you've been affected by the bug, and how to tell if you've been affected
15:30:03 mriedem go nuts
15:32:04 artom http://www.risd.edu/news/stories/go-nads/
15:37:32 openstackgerrit Artom Lifshitz proposed openstack/nova stable/pike: Make request_spec.spec MediumText https://review.openstack.org/528330
15:40:24 openstackgerrit Hironori Shiina proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487
15:40:30 openstackgerrit Artom Lifshitz proposed openstack/nova stable/ocata: Make request_spec.spec MediumText https://review.openstack.org/528332
15:41:58 openstackgerrit Artom Lifshitz proposed openstack/nova stable/newton: Make request_spec.spec MediumText https://review.openstack.org/528338
15:42:46 openstackgerrit Claudiu Belu proposed openstack/nova master: tests: autospecs all the mock.patch usages https://review.openstack.org/470775
15:55:10 openstackgerrit Jay Pipes proposed openstack/nova master: allow compute nodes to be associated with host agg https://review.openstack.org/526753
16:09:44 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: create vGPU for instance https://review.openstack.org/516899
16:11:25 jianghuaw leakypipes, bauzas: ^ Please help to check if it has addressed your comments? Thanks.
16:12:56 mdbooth mriedem: I vaguely recall there's some reason we don't use docstrings in tests. Am I imagining things?
16:13:42 mriedem mdbooth: that's no longer an issue
16:13:46 mriedem it was because of limitations with nose
16:13:49 mriedem but we don't use nose
16:14:23 mdbooth mriedem: Ah, ok. Glad I didn't stick a -1 on lyarwood 's patch :) Figured that stuff should be in HACKING anyway if it's to be enforceable.
16:14:38 mdbooth lyarwood: Ignore that one ^^^
16:14:55 leakypipes jianghuaw: +2 from me. :)
16:15:02 bauzas jianghuaw: sorry, today I'm working on implementing the libvirt change, so tomorrow will look
16:15:21 jianghuaw leakypipes, thanks:-)
16:15:40 jianghuaw bauzas, no worries. tomorrow will be very good also. Thanks.
16:15:42 mdbooth leakypipes: Is this now super Monday?
16:15:45 leakypipes jianghuaw: no problem! hope I picked the correct Chinese characters from Google translate in my review comment ;)
16:15:52 leakypipes oh, crap...
16:16:01 mdbooth Hehe
16:16:02 jaypipes thanks mdbooth :)
16:18:09 jianghuaw jaypipes, haha. I was thinking why your Chinese was so good:-)
16:18:39 kashyap Folks using Ubuntu, there's some repo that provides newer packages than the system packages
16:18:42 kashyap What's it called again?
16:18:59 mdbooth jaypipes: https://boingboing.net/2008/07/15/chinese-restaurant-c.html
16:19:24 jaypipes mdbooth: ha! that's awesome
16:20:03 mriedem kashyap: UCA
16:20:05 mriedem ubuntu cloud archive
16:20:34 jaypipes lol
16:37:26 kashyap mriedem: Ah, thank you.
17:03:14 melwitt stephenfin: hey, are you around?
17:03:19 stephenfin I am
17:04:16 melwitt I was just thinking about the xen ci failure on the patch and I realize, TLS is supposed to be disabled by default and you have to opt-in, and I don't think xen ci is doing anything to opt in. so I'm not sure yet why it's going through that path
17:05:30 melwitt so something is probably wrong there if it's going through the TLS path without being enabled. I'm still looking at it but just wanted to bring it up
17:05:40 stephenfin Yeah, I saw that comment on Thursday. I don't know what it is either
17:05:54 melwitt like, if they didn't enable it, their CI should be passing :\
17:06:06 stephenfin I know sross has some tests merged to DevStack way back related to this, but I don't know how that could possibly trigger it
17:06:27 stephenfin Agreed
17:06:56 stephenfin melwitt: I've been working on the PCI-NUMA policy code all day, but soon as that's done I'll take a look at that and see if I can suss it out
17:08:11 melwitt yeah I'm not yet sure. in devstack you're supposed to have to use NOVA_CONSOLE_PROXY_COMPUTE_TLS=True to turn on TLS, which I have done in a DNM patch dependent on the devstack change https://review.openstack.org/#/c/513160
17:08:41 melwitt okay, cool. I'll look at it some more too
17:20:56 melwitt stephenfin: I think I see what it is, left a comment on the patch. I don't know what to do about it though, I'd want to ask danpb
17:22:31 melwitt or sross/someone who knew how they wanted this to work
17:30:55 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add PCI NUMA policy fields https://review.openstack.org/527470
17:30:56 openstackgerrit Stephen Finucane proposed openstack/nova master: Add PCI NUMA policies https://review.openstack.org/527472
17:30:57 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Modify signature of _filter_non_requested_pfs https://review.openstack.org/527473
17:31:03 stephenfin jaypipes: Ta-da! ^
17:31:14 jaypipes stephenfin: cool, will look in a bit.
17:31:48 stephenfin jaypipes: Cheers :) I think that should satisfy most things. Only odd condition I had was what happened when two similar aliases requested different NUMA policies. I ruled that out as daft
17:32:01 stephenfin and now I've to run home. Ciao o/
17:32:06 jaypipes ciao
17:37:21 openstackgerrit Balazs Gibizer proposed openstack/nova master: Send soft_delete from context manager https://review.openstack.org/476459
17:37:21 openstackgerrit Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297
17:55:26 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Add support for the 2.57 microversion https://review.openstack.org/528128
18:00:43 openstackgerrit Jay Pipes proposed openstack/nova master: allow compute nodes to be associated with host agg https://review.openstack.org/526753
18:06:46 cdent that's not what makes you evil mriedem
18:07:11 mriedem |..|_
18:13:34 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Add support for the 2.57 microversion https://review.openstack.org/528128
18:26:31 jaypipes mriedem: how exactly can an instance group (server group) have >1 policy? :(
18:27:05 mriedem can't
18:27:29 mriedem https://review.openstack.org/#/c/415482/
18:27:35 jaypipes mriedem: then why is objects.InstanceGroup.policies a ListOfStringsField?
18:27:40 mriedem ^
18:27:43 mriedem "legacy reasons"
18:27:47 jaypipes uhg
18:27:49 jaypipes k
18:29:06 mriedem nice, we also have a separate dedicated table for the policies

Earlier   Later