Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-01-23
21:35:37 jackie-truong If any core reviewer has a chance to take a look at this patch, it would be very much appreciated: https://review.openstack.org/#/c/486204/
21:37:32 jackie-truong jaypipes was looking at the patch series, but he's out at a work conference at the moment. He had taken away +2 from its dependent patch (https://review.openstack.org/#/c/489408/) due to the patch series not being complete. But patch series is complete now :-)
21:38:15 jackie-truong Usage and testing instructions can be found here: https://etherpad.openstack.org/p/queens-nova-certificate-validation
21:38:35 jackie-truong And we also wrote a barbican-tempest-plugin test for it: https://review.openstack.org/#/c/515210/
21:46:48 openstackgerrit melanie witt proposed openstack/nova stable/pike: Set server status to ERROR if rebuild failed https://review.openstack.org/536897
21:46:48 openstackgerrit melanie witt proposed openstack/nova stable/pike: Fix false positive server group functional tests https://review.openstack.org/536981
22:03:19 openstackgerrit Peter Penchev proposed openstack/nova master: Add the Nova libvirt StorPool attachment driver. https://review.openstack.org/140733
22:05:37 mriedem dansmith: on alex_xu's 1.12 allocation candidate change, i assume you ran into the same sticky part here https://review.openstack.org/#/c/536083/4/nova/scheduler/client/report.py@1153
22:05:47 mriedem with assuming that the client is getting a 1.12 format request,
22:06:14 mriedem we could be in the middle of a server create / cold migrate where the format is actually 1.10, you upgrade and restart conductor, and now the client code assumes it's going to get a 1.12 but it's 1.10 and kablammo
22:06:27 mriedem via reschedule from the compute
22:06:40 mriedem because we don't have a versioned object facade for this payload
22:10:17 Roamer` mriedem, new patchset up; thanks a lot for the very useful comments!
22:10:58 Roamer` (I guess now we wait for 12 hours or so for the checks to pass...)
22:17:41 mriedem Roamer`: do you have a 3rd party CI that will eventually comment on this change to say if it's working?
22:19:26 Roamer` mriedem, actually, yes, we do; the logs are at http://logs.ci-openstack.storpool.com/ but the run it's doing right now is not with this change yet, the next one will be
22:19:41 Roamer` so I guess in about two hours or so there'll be a run with this one
22:19:58 Roamer` (yeah, I should really look into speeding the CI runs up a bit)
22:20:04 Roamer` it's just Cinder though, not Nova
22:20:22 Roamer` I mean, it's the Tempest volume tests, not all the Nova-related tests
22:20:53 mriedem but at some point it's going to at least run a tempest compute api test to attach and detach a volume to a server instance yeah?
22:20:58 Roamer` yes, it will
22:21:00 mriedem ok
22:21:28 Roamer` I may not be awake when it does (just after midnight here), but I'll post a link in about seven hours
22:21:34 Roamer` then you won't be awake :)
22:21:50 mriedem oh i'll be awake
22:21:55 mriedem sitting by the phone
22:24:15 mriedem anyone know when jaypipes is going to be back?
22:24:47 jackie-truong mriedem: All I heard is "at a work conference this week"
22:25:11 mriedem alright
22:25:13 Roamer` hopefully you won't have to rescue the mailman again... here in Sofia we have a suburb up in the mountain where I used to live for like 20 years, and the snow is a bit of a problem sometimes, like public transport going "nuh-uh, not today"
22:25:37 mriedem i was unable to rescue the mailman - he had to get a tow truck
22:26:06 mriedem rear-wheel drive with go-kart tires on an inline in a foot of snow isn't going to cut it
22:26:11 mriedem *incline
22:27:05 Roamer` oh, I hear you... there was this New Year's Eve when a neighbor had a party, he specifically warned people not to drive if they don't have the tires, and as a result there were three taxi cabs stuck in the snow on our street
22:27:24 mriedem i did enjoy watching the neighbor across the street, who is new to snow, try to drive his bmw sedan over the 3 foot wall of ice that the plow left in front of his driveway, and of course get it stuck halfway
22:28:28 melwitt I have read that the most important thing is to have winter tires. and that 4x4 isn't necessarily needed?
22:28:37 mriedem i've never had winter tires
22:28:39 mriedem or 4x3
22:28:41 mriedem *4x4
22:28:45 mriedem just don't be a jackhole
22:28:54 mriedem and put a shovel in the trunk
22:29:00 melwitt heh, good to know
22:29:19 mriedem laura's got the 4x4 in our family
22:29:36 jroll melwitt: 4wd helps you go, winter tires help you stop :P
22:29:51 mriedem pump those brakes
22:29:59 openstackgerrit melanie witt proposed openstack/nova stable/ocata: Fix false positive server group functional tests https://review.openstack.org/536995
22:29:59 openstackgerrit melanie witt proposed openstack/nova stable/ocata: Set server status to ERROR if rebuild failed https://review.openstack.org/536904
22:30:15 jroll don't be a jackhole is definitely the best advice though
22:30:24 mriedem ha, that reminds me - saw a minivan racing down the hill towing like 6 kids on a sled yesterday,
22:30:31 mriedem if they would have had to stop short, all of those kids could have been destoryed
22:30:33 mriedem *destroyed
22:31:20 Roamer` well, they *might* have been able to roll to the side, fall off the sled, but still... no, not a good image at all
22:31:47 mriedem the ones that are destined to pass on their genes, yes they might have had the instinct
22:32:03 mriedem or, be in the back
22:32:48 melwitt *facepalm*
22:32:52 dansmith mriedem: hadn't even gotten that far actually, but yeah, newer code will probably need to handle the case and transform older ones to newer ones
22:33:22 dansmith mriedem: and yes, that's related enough to what I was saying about passing this dict of allocations/resources around to need some wrapage
22:34:13 mriedem i guess you could determine format based on whether or not the 'allocations' entry within is a list or dict, and swaparoo if it's not a dict
22:35:11 dansmith for this case I think it can be done, yeah, but it's not a good precedent
22:36:24 mriedem agree
22:36:39 openstackgerrit melanie witt proposed openstack/nova stable/ocata: Fix false positive server group functional tests https://review.openstack.org/536995
22:36:39 openstackgerrit melanie witt proposed openstack/nova stable/ocata: Set server status to ERROR if rebuild failed https://review.openstack.org/536904
23:10:11 openstackgerrit Hongbin Lu proposed openstack/nova master: [WIP] Use revision_number on adding/removing security group https://review.openstack.org/535510
23:41:00 Roamer` mriedem, FTR, the StorPool CI is now doing a run with the latest patchset, and it just passed some tempest.api.compute.* tests; I'm going to sleep now, but I'll send you a link to the logs when I wake up, or you could check http://logs.ci-openstack.storpool.com/ for 534854/4 in an hour - hour and a half
#openstack-nova - 2018-01-24
00:48:57 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.61 - List/Show all server migration types https://review.openstack.org/430839
00:55:15 takashin
00:55:25 Spazmotic
00:56:22 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix missing marker functions https://review.openstack.org/514579
00:56:50 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for traits API https://review.openstack.org/524094
00:57:26 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (resource_provider) https://review.openstack.org/528629
00:58:00 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix the order of target host checks https://review.openstack.org/526225
00:58:56 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514
01:01:18 openstackgerrit Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289
01:14:11 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/537093
01:23:43 openstackgerrit OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/533994
01:23:52 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/533918
01:31:09 openstackgerrit OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/537171
01:56:20 jackie-truong If anyone has a free moment to review this patch, it would be much appreciated: https://review.openstack.org/#/c/486204/
02:00:57 Spazmotic Thanks jianghuaw, I will spin up that bug report tomorrow night when I am at my dev machine and can pull some XenAPI logs for it :)
02:01:40 jianghuaw Spazmotic, good:-)
02:01:58 Spazmotic Also should go ahead and rebase and fix that inline while i'm at it tomorrow.
02:14:59 openstackgerrit OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/533994
02:15:08 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/533918
02:24:28 alex_xu mriedem: dansmith good news is we have allocation_req_version in Selection obj, then I can convert the alloc_req dict to consistent format in the begining of claim method probably
02:25:31 alex_xu mriedem: dansmith here is https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L629
02:31:34 openstackgerrit sean mooney proposed openstack/nova master: Change 'InstancePCIRequest' spec field https://review.openstack.org/449257
02:32:59 sean-k-mooney stephenfin: https://review.openstack.org/#/c/449257/ now uses objects. ill be in around lunch tomorow if you have any questions.
03:10:40 openstackgerrit Dao Cong Tien proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487
04:06:19 openstackgerrit Deepak Mourya proposed openstack/nova master: Handle TZ change in iso8601 >=0.1.12 https://review.openstack.org/535700
04:25:30 openstackgerrit sean mooney proposed openstack/nova master: Change 'InstancePCIRequest' spec field https://review.openstack.org/449257
04:25:31 openstackgerrit sean mooney proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051
04:25:31 openstackgerrit sean mooney proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777
04:47:57 openstackgerrit Alex Xu proposed openstack/nova master: placement: using the dict format for the allocation in claim_resources https://review.openstack.org/536083
04:48:28 openstackgerrit Alex Xu proposed openstack/nova master: placement: enable required traits from the flavor extra specs https://review.openstack.org/536085
05:19:29 openstackgerrit Merged openstack/nova master: Set server status to ERROR if rebuild failed https://review.openstack.org/536268
05:37:17 openstackgerrit Alex Xu proposed openstack/nova master: placement: using the dict format for the allocation in claim_resources https://review.openstack.org/536083
05:37:18 openstackgerrit Alex Xu proposed openstack/nova master: placement: enable required traits from the flavor extra specs https://review.openstack.org/536085
06:18:03 ameeda Morning :)
06:18:32 ameeda can you please check this code to detach volumes "http://paste.openstack.org/show/651488/" and let me know if that need to fix or anything else

Earlier   Later