Monday, 2017-07-24

*** yangyapeng has joined #openstack-nova00:00
*** yangyapeng has quit IRC00:05
*** yamamoto has joined #openstack-nova00:08
*** yangyapeng has joined #openstack-nova00:09
*** gmann has quit IRC00:14
*** gmann has joined #openstack-nova00:14
*** jmlowe has quit IRC00:18
*** jmlowe has joined #openstack-nova00:19
*** yangyapeng has quit IRC00:20
*** jmlowe_ has joined #openstack-nova00:22
*** jmlowe has quit IRC00:24
*** lbragsta_ has quit IRC00:28
*** thorst has joined #openstack-nova00:31
*** catintheroof has joined #openstack-nova00:33
*** kiennt has joined #openstack-nova00:35
*** thorst has quit IRC00:36
*** Shunli has joined #openstack-nova00:46
*** gouthamr has joined #openstack-nova01:01
*** mriedem has joined #openstack-nova01:03
*** thorst has joined #openstack-nova01:03
*** thorst has quit IRC01:03
*** trungnv has joined #openstack-nova01:05
*** acormier_ has joined #openstack-nova01:06
*** mdnadeem has joined #openstack-nova01:08
*** acormier has quit IRC01:09
*** r-daneel has joined #openstack-nova01:18
*** acormier_ has quit IRC01:20
*** acormier has joined #openstack-nova01:21
*** phuongnh has joined #openstack-nova01:21
*** Daisy_ has joined #openstack-nova01:23
*** yufei has joined #openstack-nova01:23
*** acormier has quit IRC01:25
*** Daisy_ has quit IRC01:30
*** Daisy_ has joined #openstack-nova01:33
openstackgerritTristan Cacqueray proposed openstack/nova master: Mask the token used to allow access to consoles  https://review.openstack.org/22062201:38
openstackgerritZhenyu Zheng proposed openstack/nova master: Add tags to instance.create Notification  https://review.openstack.org/45949301:39
*** Daisy_ has quit IRC01:40
*** Daisy_ has joined #openstack-nova01:45
*** mtanino has joined #openstack-nova01:55
*** mtanino has quit IRC01:55
*** r-daneel has quit IRC02:06
*** armax has joined #openstack-nova02:07
*** catintheroof has quit IRC02:22
*** acormier has joined #openstack-nova02:22
*** r-daneel has joined #openstack-nova02:23
*** acormier has quit IRC02:27
*** mriedem has quit IRC02:43
*** sbezverk has quit IRC02:44
*** lbragsta_ has joined #openstack-nova02:48
*** gouthamr has quit IRC02:49
*** Daisy_ has quit IRC03:03
*** thorst has joined #openstack-nova03:04
*** Shunli has quit IRC03:05
*** Shunli has joined #openstack-nova03:05
*** vishwanathj has quit IRC03:08
*** vishwanathj has joined #openstack-nova03:08
*** thorst has quit IRC03:10
*** mtreinish has quit IRC03:14
*** mtreinish has joined #openstack-nova03:20
openstackgerritJianghua Wang proposed openstack/nova master: WIP: support os_xenapi's VDI streaming  https://review.openstack.org/48647503:26
*** fragatina has quit IRC03:38
*** fragatina has joined #openstack-nova03:39
*** mamandle has joined #openstack-nova03:40
openstackgerritTristan Cacqueray proposed openstack/nova master: Mask the token used to allow access to consoles  https://review.openstack.org/22062203:40
*** udesale has joined #openstack-nova03:42
*** links has joined #openstack-nova03:46
*** Nel1x has quit IRC04:01
*** lbragsta_ has quit IRC04:02
*** lbragsta_ has joined #openstack-nova04:03
*** anish has quit IRC04:09
*** mamandle has quit IRC04:11
*** jjung has joined #openstack-nova04:17
*** bhagyashris has quit IRC04:23
*** adisky__ has joined #openstack-nova04:28
*** fragatina has quit IRC04:31
*** fragatina has joined #openstack-nova04:32
*** lbragsta_ has quit IRC04:35
*** prateek has joined #openstack-nova04:48
*** bhagyashris has joined #openstack-nova04:50
*** sbezverk has joined #openstack-nova04:51
*** Dinesh_Bhor has joined #openstack-nova04:56
openstackgerritsudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid()  https://review.openstack.org/48100704:56
*** ratailor has joined #openstack-nova04:57
*** wasmum has quit IRC05:01
*** wasmum has joined #openstack-nova05:02
*** thorst has joined #openstack-nova05:05
*** thorst has quit IRC05:10
*** bkopilov_ has joined #openstack-nova05:10
openstackgerritJianghua Wang proposed openstack/nova-specs master: Support virtual GPU resources  https://review.openstack.org/45012205:11
*** sridharg has joined #openstack-nova05:13
openstackgerritJianghua Wang proposed openstack/nova-specs master: Support virtual GPU resources  https://review.openstack.org/45012205:18
*** psachin has joined #openstack-nova05:30
*** boris-42____ has joined #openstack-nova05:31
*** kiennt has quit IRC05:32
*** moshele has joined #openstack-nova05:35
*** sree_ has joined #openstack-nova05:36
*** sree_ is now known as Guest5591505:36
openstackgerritsudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid()  https://review.openstack.org/48100705:44
*** mamandle has joined #openstack-nova06:03
*** kiennt has joined #openstack-nova06:04
*** Oku_OS-away is now known as Oku_OS06:07
*** udesale__ has joined #openstack-nova06:07
*** udesale has quit IRC06:10
*** kiennt has quit IRC06:18
*** kiennt has joined #openstack-nova06:18
*** rcernin has joined #openstack-nova06:23
*** cfriesen has quit IRC06:25
*** thorst has joined #openstack-nova06:27
*** andreas_s has joined #openstack-nova06:27
*** mamandle has quit IRC06:28
*** thorst has quit IRC06:31
*** mamandle has joined #openstack-nova06:32
*** tuanluong has joined #openstack-nova06:32
*** yufei has quit IRC06:33
*** yamamoto has quit IRC06:34
*** mlakat has joined #openstack-nova06:34
*** zsli_ has joined #openstack-nova06:41
*** Shunli has quit IRC06:44
*** moshele has quit IRC06:48
*** sshwarts has joined #openstack-nova07:01
*** udesale has joined #openstack-nova07:02
*** nicolasbock has joined #openstack-nova07:03
*** udesale__ has quit IRC07:04
*** dixiaoli has joined #openstack-nova07:04
*** mamandle has quit IRC07:06
*** dixiaoli has quit IRC07:06
*** migi_ is now known as migi07:12
*** ralonsoh has joined #openstack-nova07:13
*** damien_r has joined #openstack-nova07:18
*** lennyb has quit IRC07:19
*** ralonsoh_ has joined #openstack-nova07:22
*** ralonsoh has quit IRC07:22
*** wasmum has quit IRC07:28
*** wasmum has joined #openstack-nova07:28
*** rmart04 has joined #openstack-nova07:29
*** lennyb has joined #openstack-nova07:32
*** rmart04 has quit IRC07:34
*** belmoreira has joined #openstack-nova07:39
*** rmart04 has joined #openstack-nova07:43
*** yamamoto has joined #openstack-nova07:45
*** moshele has joined #openstack-nova07:47
*** david-lyle has quit IRC07:50
*** dklyle has joined #openstack-nova07:50
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2)  https://review.openstack.org/40895507:51
*** alexchadin has joined #openstack-nova07:54
*** markus_z has joined #openstack-nova08:06
*** trinaths has joined #openstack-nova08:09
*** wasmum has quit IRC08:13
*** wasmum has joined #openstack-nova08:14
openstackgerritMoshe Levi proposed openstack/nova master: hardware offload support for openvswitch  https://review.openstack.org/39826508:15
*** yamahata has quit IRC08:15
*** markus_z has quit IRC08:16
openstackgerritClaudiu Belu proposed openstack/nova master: hyperv: Cleans up live migration Planned VM  https://review.openstack.org/47894308:16
*** danpawlik has quit IRC08:17
*** danpawlik has joined #openstack-nova08:17
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove an unnecessary argument in _prep_resize  https://review.openstack.org/48652108:22
*** derekh has joined #openstack-nova08:23
*** lucas-afk is now known as lucasagomes08:26
openstackgerritsudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid()  https://review.openstack.org/48100708:26
*** thorst has joined #openstack-nova08:27
*** jjung has quit IRC08:29
*** thorst has quit IRC08:32
*** dklyle has quit IRC08:32
*** dklyle has joined #openstack-nova08:33
*** dtantsur|afk is now known as dtantsur08:34
*** dklyle has quit IRC08:36
*** mamandle has joined #openstack-nova08:39
*** chohoor has quit IRC08:40
*** jangutter has joined #openstack-nova08:42
*** markus_z has joined #openstack-nova08:42
*** david-lyle has joined #openstack-nova08:47
kashyaplyarwood: mdbooth: When you get a moment, mind taking a gander at this.  It's in your wheelhouse: https://review.openstack.org/#/c/485752 -- "libvirt/driver.py: Set cache value for Cinder volume post-migration"08:47
*** david-lyle has quit IRC08:49
*** david-lyle has joined #openstack-nova08:50
*** dklyle has joined #openstack-nova08:53
*** david-lyle has quit IRC08:53
openstackgerritMoshe Levi proposed openstack/nova master: hardware offload support for openvswitch  https://review.openstack.org/39826508:55
*** litao__ has joined #openstack-nova08:58
*** yamamoto has quit IRC09:02
*** jaosorior has joined #openstack-nova09:02
openstackgerritIldiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow  https://review.openstack.org/33028509:09
*** yamamoto has joined #openstack-nova09:15
*** sambetts|afk is now known as sambetts09:16
*** cdent has joined #openstack-nova09:17
*** boris-42____ has quit IRC09:19
openstackgerritHangdong Zhang proposed openstack/os-vif master: Update the documentation link for doc migration  https://review.openstack.org/48653609:21
openstackgerritGábor Antal proposed openstack/nova master: Add helper method for waiting migrations in functional tests  https://review.openstack.org/48653709:27
*** zsli_ has quit IRC09:29
*** claudiub has joined #openstack-nova09:38
*** yamamoto has quit IRC09:47
openstackgerritsudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid()  https://review.openstack.org/48100709:48
*** jjung has joined #openstack-nova09:50
openstackgerritBéla Vancsics proposed openstack/nova master: Transform rescue/unrescue instance notifications  https://review.openstack.org/38564410:00
*** tuanluong has quit IRC10:02
*** sdague has joined #openstack-nova10:02
lyarwoodkashyap: done btw10:02
*** yamamoto has joined #openstack-nova10:02
*** yamamoto has quit IRC10:02
kashyaplyarwood: Cool, thank you.  Will look.  Yes, tests are missing, but wanted to get the core change out of the way first10:03
kashyaplyarwood: On your point about: "We can remove this now given the _get_volume_config call for each item in devices above."10:04
*** udesale__ has joined #openstack-nova10:04
kashyapI think we should make such changes a separate one, to make backporting easier?10:04
kashyapOr does that fall under "related change, and therefore it's OK"10:05
lyarwoodkashyap: erm, it's the same change, you shouldn't introduce duplicate calls to the same method in these flows10:05
kashyaplyarwood: Oeps, yes agreed.  Too much switching between tasks, I'll address it10:06
lyarwoodkashyap: np10:06
kashyapThanks for looking!10:06
lyarwoodanytime, I really need to get on top of the stable branch review queue10:06
*** udesale has quit IRC10:07
openstackgerritAlex Szarka proposed openstack/nova master: Transform instance.exists notification  https://review.openstack.org/40366010:07
*** cdent has quit IRC10:08
*** mdnadeem has quit IRC10:16
*** Guest55915 has quit IRC10:21
*** sree_ has joined #openstack-nova10:22
*** sree_ is now known as Guest423410:22
*** cdent has joined #openstack-nova10:23
*** ratailor has quit IRC10:23
*** Guest4234 has quit IRC10:26
*** thorst has joined #openstack-nova10:29
openstackgerritAlex Szarka proposed openstack/nova master: Transform the transformed notifications functional tests  https://review.openstack.org/48344810:32
*** yamamoto has joined #openstack-nova10:33
*** thorst has quit IRC10:33
*** kiennt has quit IRC10:33
openstackgerritMoshe Levi proposed openstack/nova master: hardware offload support for openvswitch  https://review.openstack.org/39826510:41
*** udesale has joined #openstack-nova10:42
*** udesale__ has quit IRC10:43
*** chohoor has joined #openstack-nova10:44
sdaguealex_xu: nice catch on - https://review.openstack.org/#/c/485602 I didn't realize we had a conf var for that10:47
sdaguealex_xu: would you want them merged?10:47
openstackgerritBalazs Gibizer proposed openstack/nova master: query instance after save to avoid partial update  https://review.openstack.org/48656110:48
*** phuongnh has quit IRC10:50
*** lucasagomes is now known as lucas-hungry10:52
*** smatzek has joined #openstack-nova10:55
*** udesale has quit IRC10:55
*** r-daneel has quit IRC10:55
openstackgerritChris Dent proposed openstack/nova master: [placement] Use wsgi_intercept in PlacementFixture  https://review.openstack.org/48623710:56
cdentbauzas: you might be interested in that ^10:57
*** jjung has quit IRC11:02
*** kristian__ has joined #openstack-nova11:06
*** Dinesh_Bhor has quit IRC11:07
*** cdent has quit IRC11:08
*** kristia__ has joined #openstack-nova11:09
*** kristian__ has quit IRC11:09
*** cdent has joined #openstack-nova11:16
*** mamandle has quit IRC11:21
*** acormier_ has joined #openstack-nova11:24
*** jwcroppe has quit IRC11:29
*** jjung has joined #openstack-nova11:35
*** acormier_ has quit IRC11:39
*** acormier has joined #openstack-nova11:39
claudiubmoshele: hello. around? :)11:40
*** acormier has quit IRC11:41
*** acormier has joined #openstack-nova11:42
*** owalsh_ is now known as owalsh11:42
*** jwcroppe has joined #openstack-nova11:43
*** tbachman has joined #openstack-nova11:44
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.live_migration_force_complete notification  https://review.openstack.org/48056111:46
*** acormier has quit IRC11:46
openstackgerritClaudiu Belu proposed openstack/nova master: hyperv: Cleans up live migration Planned VM  https://review.openstack.org/47894311:48
*** sree has joined #openstack-nova11:52
*** thorst has joined #openstack-nova11:54
*** dikonoor has joined #openstack-nova11:55
sdaguecdent: if you want to redo OSAPIFixture to use wsgi intercept, I'd be all pro that11:55
*** thorst_ has joined #openstack-nova11:56
*** thorst_ has quit IRC11:56
cdentsdague: cool, I’ll look into that when I have another window of opportunity (which is likely relatively soon)11:56
*** thorst_ has joined #openstack-nova11:57
*** belmoreira has quit IRC11:58
*** litao__ has quit IRC11:58
*** thorst has quit IRC11:58
alex_xusdague: i'm ok to merge that. we can add note for that conf later if people want11:59
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/47709112:00
openstackgerritsahid proposed openstack/nova master: libvirt: fix memory locking limit  https://review.openstack.org/47263312:01
alex_xusdague: the patch https://review.openstack.org/#/c/486414/ removed stevedore loading from the nova api. after that patch merged, we finish the goal of pike. The later cleanup can be done later, since we still have some garbage in the unittest.12:02
*** lucas-hungry is now known as lucasagomes12:03
alex_xusdague: oh, I missed understand your words. I think you mean the merge of log_format in the new middelware and config var. I think that sounds good.12:07
alex_xusdague: the only strange thing is 'microversion' won't work with eventlet. If the config val includes $(microversion)s, it will generate a KeyError I guess?12:09
sdaguealex_xu: yeh, that's going to be the issue12:12
sdaguemaybe wait for mriedem to get up and figure out what approach he wants12:13
*** lyan has joined #openstack-nova12:13
mosheleclaudiub: hi12:14
*** gszasz has joined #openstack-nova12:15
*** cdent has quit IRC12:18
*** khappone has quit IRC12:19
*** yamamoto has quit IRC12:19
*** khappone has joined #openstack-nova12:20
openstackgerritHironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector  https://review.openstack.org/46835312:22
*** mamandle has joined #openstack-nova12:23
*** baoli has joined #openstack-nova12:24
*** jjung has quit IRC12:24
*** ekuris has quit IRC12:26
*** cdent has joined #openstack-nova12:27
*** jjung has joined #openstack-nova12:29
*** pchavva has joined #openstack-nova12:31
*** baoli has quit IRC12:34
*** baoli has joined #openstack-nova12:34
*** cleong has joined #openstack-nova12:35
*** yamamoto has joined #openstack-nova12:36
*** yamamoto has quit IRC12:36
claudiubmoshele: so, i have a few other issues with sr-iov, if you have some time. :)12:36
mosheleclaudiub: I am in a meeting, but I can try to help12:37
*** jwcroppe has quit IRC12:37
*** ratailor has joined #openstack-nova12:37
claudiubmoshele: so, I see that the vendor_id, and product_id are mandatory, but the issue i'm having is that I cannot extract those from *all* the sr-iov capable nics.12:37
claudiubmoshele: for example, for Intel NICs, it's straight-forward and easy to do, as they are included in the DeviceID, but not for the Mellanox or Chelsio NICs12:38
claudiubmoshele: so, I was wondering - wouldn't reporting the device_id sufficient?12:38
claudiub*be sufficient12:39
mosheleclaudiub: I am not sure is mandatory, can you point me to the place in the code. as I remember it can be use in the pci whitelist12:39
claudiubsure, just a second.12:40
*** ekuris has joined #openstack-nova12:41
claudiubmoshele: currently, if i'm not reporting any vendor_id or product_id, it fails here: https://github.com/openstack/nova/blob/master/nova/pci/devspec.py#L27712:41
mosheleclaudiub: this for the pci whitelist, you can use also pci address/ pf name instead12:42
*** catintheroof has joined #openstack-nova12:42
mosheleclaudiub:  did you try to whitelist with pci address?12:43
jangutterclaudiub: where do you need to extract the SR-IOV capable PCI nics, is this for capabilities?12:43
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Post-migration, set cache value for Cinder volume(s)  https://review.openstack.org/48575212:43
jangutterclaudiub: I was confused with device_id for a bit, until I realised it's a synonym for product_id.12:44
claudiubjangutter: nope, product_id, vendor_id, and device_id are distinct12:44
claudiubyou can have multiple devices with the same product_id, vendor_id, but not with the same device_id12:45
jangutterclaudiub: whoops, you're right! I forgot about that!12:45
claudiubfor example, 2 identical devices will have the same product_id and vendor_id, but different device_ids12:45
jangutterclaudiub: Is there something in lspci that indicates SR-IOV capability?12:46
claudiubjangutter: i'm on windows / hyper-v. :)12:46
claudiubjangutter: but yeah.12:46
jangutterclaudiub: you poor soul.12:47
jangutterclaudiub: "Capabilities: [200 v1] Single Root I/O Virtualization (SR-IOV)"12:48
claudiubmoshele: hm, I should be able to whitelist it using "devname" too, right?12:52
mosheleclaudiub: yes12:52
claudiubhm, let's see.12:52
claudiubjangutter: hah. :))12:53
claudiubjangutter: what is that "Capabilities: [200 v1] Single Root I/O Virtualization (SR-IOV)" about?12:54
*** ralonsoh_ is now known as ralonsoh12:54
*** trinaths has quit IRC12:57
ralonsohclaudiub: can I ask you what capabilities do you need? rso, tso, txvlan, etc?12:57
*** mriedem has joined #openstack-nova12:59
*** edmondsw has joined #openstack-nova12:59
jangutterclaudiub: sorry, that's what I see with lspci.13:00
*** mgiles has joined #openstack-nova13:02
*** vladikr has joined #openstack-nova13:02
claudiubjangutter: ah, i see.13:03
*** kylek3h has joined #openstack-nova13:03
jangutterclaudiub: is there perhaps something like that in the Windows Device Manager registry?13:03
*** jaypipes has joined #openstack-nova13:04
claudiubjangutter: you mean for sr-iov? it can easily be checked through this: Get-VMHost (check if the host supports SR-IOV), and Get-NetAdapterSriov (check if the NICs support SR-IOV)13:05
claudiubi'm good in both cases. :)13:05
claudiubralonsoh: any / all. :) currently looking into hyper-v sr-iov. will have to see what capabilities are available.13:06
*** esberglu has joined #openstack-nova13:07
ralonsohclaudiub: hmmm, sorry, I didn't read that part (hyper-v). I'll take a look13:07
*** r-daneel has joined #openstack-nova13:09
*** lucasxu has joined #openstack-nova13:11
*** ratailor has quit IRC13:11
*** eharney has joined #openstack-nova13:11
*** r-daneel has quit IRC13:13
sdaguemriedem: when you get a chance this morning, I'd like to figure out what's needed to merge this - https://review.openstack.org/#/c/485602/ - I think the outstanding question is around using the existing config definition13:15
sdaguethough that might be a reasonable follow on patch to pull that in for both nova-api and placement13:15
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2)  https://review.openstack.org/40895513:17
*** takashin has left #openstack-nova13:17
openstackgerritBéla Vancsics proposed openstack/nova master: Transform rescue/unrescue instance notifications  https://review.openstack.org/38564413:17
*** jamesdenton has joined #openstack-nova13:18
*** links has quit IRC13:20
*** mgiles has quit IRC13:22
mriedemsdague: so the question is if you're using eventlet, wsgi_log_format config is how you tailor the log format however you want, but with this it's hard-coded so you lose the ability13:23
*** mgiles has joined #openstack-nova13:23
sdaguemriedem: yes13:23
sdagueI guess we should determine if that is valuable as config or not13:24
sdagueplacement goes the hard coded route here as well13:24
*** prateek has quit IRC13:24
sdagueso this is consistent with placement13:24
mriedem%(request_line)s must come from eventlet?13:24
mriedemand we're hard-coding: %(REQUEST_METHOD)s %(REQUEST_URI)s13:24
*** jamesdenton has quit IRC13:25
mriedemthis is the eventlet wsgi default13:25
mriedemlog_format='%(client_ip)s - - [%(date_time)s] "%(request_line)s" %(status_code)s %(body_length)s %(wall_seconds).6f'13:25
*** jamesdenton has joined #openstack-nova13:25
mriedemseems odd that we define a default for our config option that's different from the eventlet wsgi default13:26
sdaguemriedem: https://github.com/openstack/oslo.service/blob/5a8a712c26cb053cbabf49044e6831ed98a2c2d5/oslo_service/wsgi.py#L18413:26
sdaguemriedem: yeh, well, it's all leaky abstraction13:26
sdaguethose are eventlet native stanzas13:27
mriedemnova uses it here https://github.com/openstack/nova/blob/master/nova/wsgi.py#L18413:27
*** zz9pzza has quit IRC13:29
*** efried_zzz is now known as efried13:29
sdaguemriedem: ah, I guess we're duping more of oslo.service than I realized, I thought that had been fully factored back out13:30
sdaguebut, regardless, the current keying are eventlet server keywords13:30
mriedemright, what i'm thinking is if we used the config,13:31
mriedemit would be nice to have it default to None so we could tell if the operator actually cares about overriding it,13:31
mriedemif it's None, and you're using eventlet wsgi, then you get eventlet wsgi defaults for the log format13:31
bauzasmdbooth: do you remember some problems with volume migrations for their type ? https://bugs.launchpad.net/nova/+bug/170570013:31
sdaguemriedem: yeh, very much good point13:31
openstackLaunchpad bug 1705700 in OpenStack Compute (nova) "live migration does not work after volume migration" [Undecided,New]13:31
mriedemif it's None and you're using apache, then you get our hard-coded line13:31
sdaguemriedem: we'd also have to normalize the keywords, because things like microversion aren't going to exist where eventlet does this13:32
mriedemright, i'd munge that in with request_line13:32
bauzasmdbooth: tl;dr cinder block volumes seem to be transformed as files13:32
mdboothbauzas: Looking13:32
sdaguehttps://github.com/eventlet/eventlet/blob/ce72c4378b176cdb0d8b6039023e3ad0026becde/eventlet/wsgi.py#L603-L611 - is where eventlet does it's thing13:33
mdboothbauzas: That does ring a bell, actually13:33
mriedemsdague: fyi it's also a default value in oslo.service https://github.com/openstack/oslo.service/blob/master/oslo_service/_options.py#L5913:33
sdaguemriedem: yep13:33
mdboothkashyap: You've fixed ^^^ recently, haven't you?13:33
bauzasmdbooth: yup, I remember some discussion we had for that13:33
* kashyap reads the scroll13:33
bauzaskashyap: https://bugs.launchpad.net/nova/+bug/170570013:33
openstackLaunchpad bug 1705700 in OpenStack Compute (nova) "live migration does not work after volume migration" [Undecided,New]13:33
sdaguemriedem: so, honestly, my inclination is to just say this isn't customizable any more, and only bring that back if people really complain about it13:34
kashyapmdbooth: It's the cache value being not set on the Cinder volume post-migration?13:34
mriedemseems we could deprecate the nova specific wsgi_log_format, although if we want to default it to None we'd then end up relying on the config from oslo.service, which gets messy13:34
sdaguebecause the value in making that site specific seems really weird13:34
mdboothkashyap: No, different bug13:34
kashyapIf so -- https://review.openstack.org/#/c/485752/ -- libvirt: Post-migration, set cache value for Cinder volume(s)13:34
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.resize_revert notification  https://review.openstack.org/39681113:34
kashyapmdbooth: Oh, is it the one bauzas pointed out just now?13:34
mdboothkashyap: Yes13:34
* kashyap reads13:34
sdaguemriedem: because it feels like you could do a lot of effort here for 0 people ever doing this thing13:35
kashyapmdbooth: Oh, this one -- I recall it13:35
mdboothhttps://bugs.launchpad.net/nova/+bug/170570013:35
openstackLaunchpad bug 1705700 in OpenStack Compute (nova) "live migration does not work after volume migration" [Undecided,New]13:35
mriedemsdague: yeah i'm asking in the operators channel, but i don't care to bend over backwards for the config option13:35
bauzasmdbooth: looked related but not the same problem13:35
bauzasoops kashyap rather13:35
kashyapmdbooth: Not me (Artom), but I did the triage of it with Eric Blake.  The change was simple13:36
mdboothbauzas: Yeah, I've definitely seen a patch for the file='block device' thing13:36
bauzasclaudiub: I'm unclear whether https://bugs.launchpad.net/nova/+bug/1705683 is only for hyper-v driver13:36
openstackLaunchpad bug 1705683 in compute-hyperv "vif ports are not unplugged after cold migration" [Undecided,In progress] - Assigned to Alexandru Muresan (amuresan)13:36
mriedemsdague: https://github.com/openstack/nova/commit/c01c852c808970ff649c26a0e52d34b3c2d80b0b13:36
*** yamamoto has joined #openstack-nova13:36
kashyapWe need both the flags: VIR_DOMAIN_BLOCK_REBASE_COPY _and_ VIR_DOMAIN_BLOCK_REBASE_COPY_DEV13:36
mriedemsdague: sounds like that was a bug fix at one point or a workaround for a limitation in eventlet13:37
mriedemsdague: and then oslo.service copied it from nova13:37
mriedemhttps://github.com/openstack/oslo.service/commit/6dd3ad50b6fee79b4cc08fe0830327803d4b2f5b13:37
bauzasmdbooth: if you can find it, you'll get eternal gratitudes13:37
mdboothbauzas: I'll have a hunt in a bit. Just doing some fun, fun, fun bugzilla diving.13:38
bauzasmdbooth: but I can lookup gerrit for libvirt/volume.py recent changes13:38
mriedemsdague: so i'd say we keep them separate and actually deprecate the nova wsgi_log_format option since it's in oslo.service, and doesn't even really seem useful anymore13:38
bauzasmdbooth: nah, I can try to find so13:38
kashyapmdbooth: Whoops, I think I see the cause: "The issue is caused by the fact that dist type is changed from 'block' to 'file' after volume migration."13:39
bauzasmdbooth: you'll just miss unlimited beer credits13:39
kashyapIt should _retain_ 'block' as the disk type post-migration, too.13:39
sdaguemriedem: ah, right, because we're going through the python logger the default logger in eventlet is also putting in timestamping13:39
bauzaskashyap: that is the root cause as per the reporter indeed13:39
* mdbooth modifies the xslt he wrote to process bugzilla query output, and weeps :(13:39
sdaguemriedem: ok, same patch for deprecation, or follow on?13:40
bauzaskashyap: I lean to mark the bug as confirmed, but finding a duplicate or even a bugfix would be awesomeness13:40
mriedemfollow on13:40
*** kristia__ has quit IRC13:40
*** Dinesh_Bhor has joined #openstack-nova13:40
sdaguemriedem: then you want to hit the +A on the current one?13:40
kashyapbauzas: Yeah, have to look for the dupliate.  First, trying to wrap my head around another unit test fix.  Want to get this out before I'm out for 2-ish weeks starting 01 Aug13:40
sdagueit has to +2s, but was waiting for you to come back around13:40
*** gouthamr has joined #openstack-nova13:40
mriedemsdague: i'm goign to push a devstack change that makes nova-api run under eventlet again and depends-on this change to make sure the eventlet stuff is as expected13:41
*** kristian__ has joined #openstack-nova13:41
bauzaskashyap: np, will try to wake up my gerrit search skills13:41
kashyapbauzas: I don't think there's a duplicate.   I'm looking at this change for the first time13:41
kashyaps/change/bug/13:41
sdaguemriedem: sure, though the unit tests mostly cover that as well13:41
kashyapbauzas: And the above bug you point to seems caused by this (merged) change: https://review.openstack.org/#/c/465205/13:41
gibimriedem: hi! I made some progress on the missing update on the updated_at field. However fix I'm currently proposing might not what we want at the end https://review.openstack.org/#/c/486561/13:41
bauzaskashyap: was just digging in https://review.openstack.org/#/q/project:openstack/nova+file:%255Enova/virt/libvirt/volume.py13:42
gibimriedem: I will try to dig deeper in oslo.db as time allows13:42
bauzaskashyap: ok, I'll set the bug as confirmed13:43
mriedemgibi: ok13:43
*** yamamoto has quit IRC13:44
*** xyang1 has joined #openstack-nova13:44
bauzasmriedem: would it be reasonable to target bug reports as pike-rc-potential if anyone didn't provided a fix yet (ie. not in Progress) ?13:44
bauzasmriedem: having in-progress bugs in the bucket is cool for reviews, but I'd also like to make sure we're like telling to the world that the release is worth getting those unassigned bugs fixed so someone could step up ?13:45
mriedembauzas: yes13:46
bauzasmriedem: tbc, I'm worried of any bad press of https://bugs.launchpad.net/nova/+bug/1705700 if not fixed by Ocata timeframe13:46
openstackLaunchpad bug 1705700 in OpenStack Compute (nova) "live migration does not work after volume migration" [High,Confirmed]13:46
claudiubbauzas: we've only observed that bug on hyper-v. libvirt seems to unplug the vifs on confirm_migration.13:47
mriedembauzas: is it latent or a regression in pike?13:48
bauzasmriedem: it's Ocata13:48
mriedemso a regression introduced in ocata13:48
bauzasmriedem: but looks like https://review.openstack.org/#/c/465205/ was backported to Ocata13:48
bauzasmriedem: I can try to investigate further13:49
bauzasat least the bugfix above was backported in a point release13:49
bauzaswe can ask to test some compute with some older release13:49
openstackgerritsahid proposed openstack/nova-specs master: virt: provide a mask to selection pCPUs to run emu. threads  https://review.openstack.org/48661713:50
edleafeScheduler subteam meeting in 10 minutes in #openstack-meeting-alt13:50
*** eharney has quit IRC13:50
*** randomhack has joined #openstack-nova13:51
*** chyka has joined #openstack-nova13:51
openstackgerritJianghua Wang proposed openstack/nova-specs master: Support virtual GPU resources  https://review.openstack.org/45012213:51
*** mamandle has quit IRC13:52
*** MVenesio has joined #openstack-nova13:53
*** jwcroppe has joined #openstack-nova13:54
*** eharney has joined #openstack-nova13:55
*** randomha1k has joined #openstack-nova13:56
*** crushil has joined #openstack-nova13:56
*** chyka has quit IRC13:57
*** randomhack has quit IRC13:57
*** awaugama has joined #openstack-nova13:58
*** tylerderosagrund has joined #openstack-nova13:58
*** mamandle has joined #openstack-nova13:58
edleafeScheduler subteam meeting running now in #openstack-meeting-alt14:01
openstackgerritSean Dague proposed openstack/nova master: deprecate ``wsgi_log_format`` config variable  https://review.openstack.org/48662314:01
*** zz9pzza has joined #openstack-nova14:03
*** efoley has joined #openstack-nova14:04
*** alexchadin has quit IRC14:06
*** crushil has quit IRC14:06
artomkashyap, mdbooth, you mean14:08
artomErr14:09
artomkashyap, mdbooth, you mean https://review.openstack.org/#/c/471356/ ?14:09
openstackgerritJay Pipes proposed openstack/nova master: claim resources in placement API during schedule()  https://review.openstack.org/48356614:09
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Post-migration, set cache value for Cinder volume(s)  https://review.openstack.org/48575214:09
artomThis is the yet unmerged newton backport, the other ones have merged14:09
mdboothbauzas: artom gets all your beer :)14:09
* kashyap clicks at artom's link14:10
*** tylerderosagrund has quit IRC14:10
kashyapartom: Ah, nice one.  Yep, that's it14:10
*** ekuris has quit IRC14:10
bauzasmdbooth: FWIW I just marked the bug we discussed as incomplete since I'd like the reporter to test the last ocata point release14:11
bauzasas it includes artom's patch14:11
mdbooth+114:11
*** crushil has joined #openstack-nova14:12
*** crushil has quit IRC14:13
*** crushil has joined #openstack-nova14:13
bauzasdansmith: mmm, we have lots of docs mentioning it's worth doing SIGHUPs for upgrades or mutable config but I don't see how nova-compute service is hooking up this signal :)14:14
bauzasdansmith: since it's not inheriting from oslo.service AFAICT14:14
bauzasand we don't have any signal handling in that code14:15
dansmithbauzas: oh, this reminds me, someone recently asked me about a doc they found that says they could change the log level at runtime14:15
dansmithby SIGHUP14:15
dansmithit clearly was not working and I told them I expected that was oslo documentation, but never circled back14:16
bauzasdansmith: https://bugs.launchpad.net/nova/+bug/1705680 led me investigating and honesly I don't see how the magic can happen14:17
openstackLaunchpad bug 1705680 in OpenStack Compute (nova) "nova compute does nothing on receiving sighup signal" [Undecided,New]14:17
bauzasif we were inheriting from oslo.service manager, then OK14:17
bauzasbut we're not14:17
dansmithwe do process sighup for rpc version pins14:17
dansmithbauzas: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L53914:18
bauzasdansmith: I saw the hook14:18
bauzasdansmith: but I don't see how we define that hook to be called on a sighup signal14:18
dansmithI think we do get it from oslo.service14:19
dansmithbut it's buried pretty deep14:19
bauzasdansmith: that was my assumption14:19
bauzasbut14:19
bauzaswe don't inherit from it14:19
dansmithwe do14:19
dansmithbauzas: https://github.com/openstack/nova/blob/master/nova/service.py#L98-L9814:19
dansmithbauzas: service is oslo.service there14:19
dansmithL2714:20
bauzashttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L48214:20
bauzasmanager is nova.manager, right?14:20
dansmithyes, but the service hooks Service.reset() to Manager.reset()14:20
dansmithhttps://github.com/openstack/nova/blob/master/nova/service.py#L289-L28914:20
*** annegentle has joined #openstack-nova14:20
bauzasbut https://github.com/openstack/nova/blob/master/nova/manager.py#L9114:21
mriedemdansmith: the mutable config stuff is in oslo.config docs14:21
bauzasdansmith: ooooh, it's fucking cryptic14:21
mriedemhttps://docs.openstack.org/oslo.config/latest/reference/mutable.html14:21
dansmithbauzas: not really, managers and services have always had this relationship14:21
bauzasI see14:21
dansmithmriedem: oh right and I think we're writing into our default nova config that log level is mutable or something14:21
dansmithbecause of that14:22
mriedemi thought it was only 'debug'14:22
*** acormier has joined #openstack-nova14:22
mriedemhttps://review.openstack.org/#/c/28085114:23
mriedemhttps://review.openstack.org/#/c/254821/14:23
bauzasmriedem: I was just looking at the mutable-config series14:23
dansmithmriedem: okay I never saw this, but it completely wasn't working14:23
mriedemso default_log_levels config wouldn't use that14:23
dansmithI think it was debug= they were toggling, but maybe not14:24
mriedemwe only have 414:24
mriedemhttps://docs.openstack.org/nova/latest/sample_config.html14:24
mriedemif you search for "Note: This option can be changed without restarting."14:24
dansmithwonder if we're supposed to be hooking our sighup handler to oslo.log somehow?14:25
bauzasmriedem: correct, the mutable-config series was mostly still in progress when lxsli left14:25
dansmithunless it registers its own signal handler quietly14:25
*** sree has quit IRC14:26
bauzasthe point with https://bugs.launchpad.net/nova/+bug/1705680 is that I suspect the sighup to be caught but just the fact that we only reload a very few flags made the reporter thinking it wasn't working14:26
openstackLaunchpad bug 1705680 in OpenStack Compute (nova) "nova compute does nothing on receiving sighup signal" [Undecided,New]14:26
bauzaseither way, I can call out for details14:26
*** yamahata has joined #openstack-nova14:28
*** felipemonteiro has joined #openstack-nova14:30
*** baoli has quit IRC14:31
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request  https://review.openstack.org/45177714:31
*** felipemonteiro_ has joined #openstack-nova14:31
*** crushil has quit IRC14:31
*** crushil has joined #openstack-nova14:32
*** mgiles has quit IRC14:34
*** mgiles has joined #openstack-nova14:34
*** felipemonteiro has quit IRC14:34
*** yufei has joined #openstack-nova14:36
*** yufei has quit IRC14:36
*** yufei has joined #openstack-nova14:36
*** mtanino has joined #openstack-nova14:38
*** kristian__ has quit IRC14:39
*** kristian__ has joined #openstack-nova14:39
*** marst has quit IRC14:39
*** kristian__ has quit IRC14:43
*** coreywright has quit IRC14:44
*** marst has joined #openstack-nova14:47
*** chyka has joined #openstack-nova14:50
*** jamesdenton has quit IRC14:51
*** jamesdenton has joined #openstack-nova14:52
*** baoli has joined #openstack-nova14:52
*** sree has joined #openstack-nova14:53
openstackgerritEric Fried proposed openstack/nova master: Adopt new pypowervm power_off APIs  https://review.openstack.org/47694514:55
openstackgerritSean Dague proposed openstack/nova master: Increase cpu time for image conversion  https://review.openstack.org/48664214:56
*** s-dean has joined #openstack-nova14:56
*** coreywright has joined #openstack-nova14:57
*** cdent has quit IRC14:59
*** jamesdenton has quit IRC14:59
s-deancan somebody please confirm that the cells table in nova db is meant to be empty, I have been trying to setup the cells database and every time i run su -s /bin/sh -c "nova-manage db sync" nova , I get the following output ERROR: could not access cell mapping database - has api db been created?, I have been at this for 5 days and same error everytime i reinstall14:59
janguttersean-k-mooney: are you inline?14:59
jangutters/inline/online/?14:59
*** baoli has quit IRC14:59
*** zz9pzza has quit IRC15:00
*** zz9pzza has joined #openstack-nova15:00
*** annegentle has quit IRC15:00
*** kristian__ has joined #openstack-nova15:01
*** sree has quit IRC15:01
*** kristian__ has quit IRC15:02
*** kristian__ has joined #openstack-nova15:02
*** kristian__ has quit IRC15:02
*** sree has joined #openstack-nova15:02
sean-k-mooneyyes though i have to drop for meeting in an hour15:02
*** kristian__ has joined #openstack-nova15:02
*** jamesdenton has joined #openstack-nova15:02
sean-k-mooneyjangutter: ^15:03
*** lucasxu has quit IRC15:03
janguttersean-k-mooney: I split off https://review.openstack.org/#/c/486426/ but I'm not sure I wrote the test right.15:03
*** MVenesio has quit IRC15:04
*** MVenesio has joined #openstack-nova15:04
sean-k-mooneyjangutter: well that is partly a technically question and partly a political one. you added _set_config_VIFHostDevice15:05
*** sree has quit IRC15:06
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Migration from 'ip' commands to pyroute2  https://review.openstack.org/48438615:06
*** markus_z has quit IRC15:07
*** kristian__ has quit IRC15:07
*** kristian__ has joined #openstack-nova15:07
*** crushil has quit IRC15:07
mriedems-dean: yes the cells table is for cells v1 only15:08
*** crushil has joined #openstack-nova15:08
janguttersean-k-mooney: yep, and used "unplugin" rather than "plugin" -> other VIF tests that don't go out via an os-vif plugin seem to use that, rather than "plugin"15:08
sean-k-mooneyjangutter:  technically  in test_vif.py you should only assert that the designer was called. and then in https://github.com/openstack/nova/blob/master/nova/tests/unit/virt/libvirt/test_designer.py you should assert that correct xml is generated when you pass in the os_vif_hostdevice15:08
mriedems-dean: you have a nova_api db yes?15:08
*** yufei has quit IRC15:08
mriedems-dean: did you run nova-manage api_db sync ?15:08
mriedems-dean: also https://docs.openstack.org/nova/latest/cells.html and https://docs.openstack.org/ocata/install-guide-ubuntu/nova.html for docs15:09
sean-k-mooneyjangutter: but politcally nova has not done it this way else where in the file so you should follow the files convention of have test_vif.py also test desighern fuctionality though really this should be change across the board in that file in a seperate patch15:09
janguttersean-k-mooney: ah, that makes it a lot finer-grained.15:09
*** mamandle has quit IRC15:10
*** baoli has joined #openstack-nova15:10
sean-k-mooneyjangutter: yes unfutrnetlly that is not how the other unitest have been written so it better to follow the convention in the file though we should proably file a bug to make it more granular15:10
sean-k-mooneyjangutter: i think the current patch is fine given the convention that is inplace15:11
mriedems-dean: it could also mean that the nova.conf that you're running nova-manag db sync against doesn't have the [api_database]connection option set?15:11
s-deanyes I have nova_api database, I have run the commands in order as specified in the documentation, However ERROR: could not access cell mapping database - has api db been created?15:12
s-deankeeps appearing15:12
mriedems-dean: the command is looking for entries in the cell_mappings table in the nova_api db15:12
*** cfriesen has joined #openstack-nova15:12
s-deanthey exist15:12
janguttersean-k-mooney: thanks, I and that clears up the other questions I also had. I had wondered why there seemed to be two sets of tests in test_vif.py15:13
mriedemor whatever you called it, but it would be defined by the [api_database]connection url in nova.conf15:13
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2)  https://review.openstack.org/40896415:13
mriedems-dean: how do the cell_mappings entries exist if you don't have a nova_api db?15:13
mriedemoh you said you have it15:13
stvnoyesmriedem good morning Matt, when you get some time, please take a look at the updated cinder v3 migrate review. thanks. - https://review.openstack.org/#/c/463987/15:13
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004215:13
*** rmart04 has quit IRC15:13
mriedems-dean: is [api_database]connection set in nova.conf when running nova-manage db sync?15:13
s-deanI have created it, Is this a potential bug15:13
*** dklyle is now known as david-lyle15:14
*** dikonoor has quit IRC15:14
s-deanyes my connection string is correct15:14
mriedems-dean: just to be clear, so you have both [database]/connection and [api_database]/connection set in nova.conf?15:14
mriedemand they are different values, yes?15:14
s-deanyes15:14
s-deanyes15:14
s-deanone for nova_api15:15
mriedemok, and you ran nova-manage api_db sync ?15:15
s-deanand on for nova db15:15
s-deanyes15:15
sean-k-mooneyjangutter: i think its a legacy thing. but generally when writhing unit test you should mock any  fuction call to other modules in the current project and test only how the output of the fuction is used and test the internals of that fuction in a seperate test in that modules test file15:15
mriedemdid you run nova-manage cell_v2 map_cell015:15
openstackgerritGábor Antal proposed openstack/nova master: Transform HostAPI-set_enabled notifications  https://review.openstack.org/48262215:16
sean-k-mooneyjangutter: but in all cases the convention of the file you are extending take presedence over teh convetions of the project as a whole though one would hope they more or less agree15:16
mriedemstephenfin: fyi, docs theme is jacked here https://docs.openstack.org/nova/latest/cells.html15:16
s-deanI have run Step 3,4,5,6 at step 6 following error15:17
s-deanERROR: could not access cell mapping database - has api db been created?15:17
janguttersean-k-mooney: yep. No joke "writhing unit test" is pretty accurate.15:17
mriedems-dean: steps in which doc? the install guide?15:17
s-deanubuntu 16.0415:17
s-deanocata install guide15:18
mriedems-dean: so you have a nova_cell0 database as well?15:18
mriedemthere are 3 dbs involved15:18
s-deanyeah15:18
mriedemok15:18
s-deannova_api15:18
s-deannova15:18
*** Oku_OS is now known as Oku_OS-away15:18
s-dean_cell015:18
s-dean nova_cell015:19
mriedemok and the cell_mappings.database_connection url for the cell_0 entry in the nova_api database is correct?15:19
s-deanall exist15:19
knikollao/ is the rbd information for the ephemeral disks only in the nova,ceph.conf files? We've changed monitor IPs but old VMs still have the old information with old IPs.15:19
*** annegentle has joined #openstack-nova15:20
s-deani shall check15:20
*** Apoorva has joined #openstack-nova15:21
s-deantransport url for cell1 exists, cell0 has the following none:///15:21
mriedemfor the database_connection/15:21
mriedem?15:21
s-deanno15:21
s-deandatabase connection on both cell0 and cell1 are the same15:21
mriedemexcept one should point at cell0 and one should point at cell115:22
mriedemwell, the cell0 mapping should point at the nova_cell0 db,15:22
*** kashyap has quit IRC15:22
mriedemand the cell1 mapping should point at the nova db15:22
s-deanmysql+pymysql://nova:mypassword@my_ip/nova?charset=utf8&ssl_ca=/etc/nova/tls/mysql/ca-cert.pem&ssl_cert=/etc/nova/tls/mysql/server-cert.pem&ssl_key=/etc/nova/tls/mysql/server-key.pemnova_cell015:23
s-deanis this the issue > nova_cell015:23
s-deanis this the issue > server-key.pemnova_cell15:23
sdaguemriedem / dansmith - is there a new timing issue where GET right after POST is going to 404 - https://bugs.launchpad.net/nova/+bug/1682423 ?15:23
openstackLaunchpad bug 1682423 in OpenStack Compute (nova) "create instance with dashbaord / 404 returned" [Undecided,New]15:23
mriedems-dean: i think the issue is that's pointing at the nova db15:24
mriedemnova?15:24
mriedemi think that should be nova_cell0?15:25
s-deanwhy is it on the end of the connection string ?15:25
dansmithsdague: in ocata?15:25
mriedems-dean: my guess is a string substitution bug in map_cell015:25
mriedems-dean: do you have the latest stable/ocata code? or 15.0.0?15:25
s-deanfrom the cloud archive15:25
sdaguedansmith: yeh, I was wondering if the cells v2 transition opens up a window there15:25
dansmithsdague: it's not supposed to, no15:26
dansmithif it was master just now I'd suspect some quota thing15:26
openstackgerritTakashi NATSUME proposed openstack/nova master: Stop using mox stubs in test_console_auth_tokens.py  https://review.openstack.org/47737415:26
*** yufei has joined #openstack-nova15:26
mriedems-dean: what's the version of nova?15:27
s-deanto seconds15:27
s-deantwo15:27
*** sridharg has quit IRC15:27
*** kristia__ has joined #openstack-nova15:27
s-deannova-api | 2:13.0.0-0ubuntu215:27
mriedems-dean: whoa that's mitaka15:27
mriedemocata is 15.0.015:27
s-deanwait15:28
s-deansorry15:28
s-deannova-api | 2:15.0.5-0ubuntu1~cloud015:28
*** kristian__ has quit IRC15:28
mriedemok that's better15:28
*** _pewp_ has joined #openstack-nova15:28
mriedems-dean: did you run map_cell0 with 15.0.5 or earlier?15:29
mriedembecause https://docs.openstack.org/releasenotes/nova/ocata.html15:29
s-deanfresh install15:29
mriedemhttps://launchpad.net/bugs/167361315:29
openstackLaunchpad bug 1673613 in OpenStack Compute (nova) ocata "Special characters are not allowed in connection string" [High,Fix committed] - Assigned to Matt Riedemann (mriedem)15:29
mriedemif your an map_cell0 at 15.0.5 then my guess is the ssl stuff in the url got messed up and put nova_cell0 at the end of the URL15:30
mriedem*if you ran15:30
s-deanok15:30
s-deanthat makes more sense15:30
s-deanI've been scratching my head over this for over a week15:30
s-deanim required to implement TLS throughout openstack15:31
mriedems-dean: ok can you file a bug?15:31
s-deanof course15:31
s-deanhappy to help15:31
openstackgerritMatt Riedemann proposed openstack/nova master: Provide hints when nova-manage db sync fails to sync cell0  https://review.openstack.org/48666015:31
mriedemhttps://bugs.launchpad.net/nova/15:31
mriedemthanks15:31
mriedems-dean: also ^15:31
mriedemshould help with that crappy error message15:31
s-deanok in the mean time to get this fixed, can i just modify the database_connection string,15:32
s-deando both cell0 and cell1 need to point at the nova_cell0 ?15:33
mriedems-dean: you can modify the db yes, but cell0 points at nova_cell015:33
mriedemand cell1 points at the 'nova' db15:33
s-deanok perfect15:34
mriedemcell1 is where your compute services and instances live15:34
openstackgerritMerged openstack/nova master: Update policy description for 'instance_actions'  https://review.openstack.org/48460915:34
mriedembdm table, etc15:34
s-deani will file a big report thank you so much for your help15:34
s-deanmuch appreciated15:34
mriedemyw, just ping me here with the bug number later, i'll be afk for awhile - sorry for the headache here15:34
*** rcernin has quit IRC15:34
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: Microversion 2.53 - services and hypervisors using UUIDs  https://review.openstack.org/48543515:34
s-deanwill do, take care15:34
*** wasmum has quit IRC15:36
*** wasmum has joined #openstack-nova15:39
*** _pewp_ has quit IRC15:40
*** _pewp_ has joined #openstack-nova15:40
ildikovmriedem: hi15:42
ildikovmriedem: so I did some polishing with the tests and I have one single functional test failing15:42
ildikovmriedem: I cannot figure out whether what I do is a completely bad idea or I'm just missing some small thing15:42
ildikovmriedem: ping me if you have a few seconds or just give it a review when you can and then I'll fix things up as you say15:43
ildikovmriedem: either works15:43
ildikovmriedem: thanks in advance15:43
claudiubmoshele: eh, it didn't work to whitelist by devname, since the implementation for devname is linux-specific. :)15:44
mosheleclaudiub: so just add windows support15:45
mosheleclaudiub:  also the pci address is linux spacifc15:45
openstackgerritMerged openstack/nova master: Updated from global requirements  https://review.openstack.org/48643415:45
openstackgerritMerged openstack/nova master: Dump versioned notifications when test_create_delete_server  https://review.openstack.org/48630115:46
*** Apoorva has quit IRC15:46
*** imacdonn has quit IRC15:46
openstackgerritMerged openstack/python-novaclient master: Updated from global requirements  https://review.openstack.org/48643515:46
*** armax has quit IRC15:47
*** crushil has quit IRC15:47
*** kristia__ has quit IRC15:48
*** ratailor has joined #openstack-nova15:49
*** kristian__ has joined #openstack-nova15:49
jaypipesedleafe, 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:49
*** prateek has joined #openstack-nova15:50
*** kashyap has joined #openstack-nova15:50
*** armax has joined #openstack-nova15:52
claudiubmoshele: 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:53
*** kristian__ has quit IRC15:54
*** damien_r has quit IRC15:55
edleafejaypipes: I still don't understand how instance_uuids could be None at that point, since you got instance_uuid from it in line 18515:55
*** pchavva has quit IRC15:55
mosheleclaudiub: the pci.alias is used for pci passthrough. you don't need it for SR-IVO with networking15:55
*** jamesden_ has joined #openstack-nova15:55
mosheleclaudiub:  isn't product_id and device_id the same15:55
mosheleclaudiub: ?15:56
*** kristian__ has joined #openstack-nova15:56
claudiubmoshele: no. if you have 2 identical devices, they will have the same vendor_id and product_id, but different device_ids15:56
jaypipesedleafe: if I de-indent block 205-215 one level, line 185 may not have run.15:56
*** randomha1k has quit IRC15:56
*** nicolasbock has quit IRC15:57
*** jamesdenton has quit IRC15:57
jaypipesedleafe: oh, nm, I think I know what you're after.15:58
jaypipesedleafe: duh, sorry... one sec.15:58
edleafejaypipes: Then don't de-indent. I'm wondering why there is a condition on instance_uuids is not None15:58
*** rcernin has joined #openstack-nova15:58
jaypipesedleafe: yep, sorry, I was being dense15:58
*** imacdonn has joined #openstack-nova15:58
mosheleclaudiub:  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:58
*** jjung_ has joined #openstack-nova15:59
*** jmlowe_ has quit IRC15:59
mosheleclaudiub: I have to go home but you can send me a mail moshele@mellanox.com with all you questions ..15:59
openstackgerritJay Pipes proposed openstack/nova master: claim resources in placement API during schedule()  https://review.openstack.org/48356616:00
jaypipesedleafe: ok, sorry about that. done now :) ^^16:00
jaypipesmriedem: ^16:00
claudiubopenstackgerrit: sure, thanks. :)16:00
jaypipesclaudiub: I don't think openstackgerrit is very responsive to thanks. :P16:01
*** moshele has quit IRC16:01
*** randomhack has joined #openstack-nova16:01
claudiubha, openstackgerrit wanted some attention, and jumped in front of my click. :D16:02
*** baoli has quit IRC16:02
*** baoli has joined #openstack-nova16:02
*** lucasagomes is now known as lucas-afk16:04
*** armax has quit IRC16:04
*** ijw has joined #openstack-nova16:05
*** baoli has quit IRC16:07
*** andreas_s has quit IRC16:08
openstackgerritEd Leafe proposed openstack/nova master: Migrate Ironic Flavors  https://review.openstack.org/48494916:08
edleafedansmith: ^^ hope this is closer to what you want16:08
*** trinaths has joined #openstack-nova16:08
dansmithedleafe: I was about to ask ... I'll go have a look16:08
dansmithedleafe: the move from eventlet is purely a nova-api thing for wsgi stuff, fwiw16:09
openstackgerritChris Friesen proposed openstack/nova master: Add validation for cpu_realtime_mask  https://review.openstack.org/46820316:10
openstackgerritChris Friesen proposed openstack/nova master: Tweak the cpu_realtime_mask handling slightly  https://review.openstack.org/46145616:10
*** ijw has quit IRC16:10
*** jjung_ has quit IRC16:10
mdboothkashyap: https://review.openstack.org/#/c/485752/ Can you remind me where we rewrite the guest xml post migration?16:14
mdboothkashyap: NM, I didn't read your commit message carefully enough16:15
*** ratailor has quit IRC16:15
*** claudiub has quit IRC16:15
*** randomhack has quit IRC16:20
*** jamesden_ has quit IRC16:21
*** jamesdenton has joined #openstack-nova16:23
openstackgerritmelanie witt proposed openstack/nova master: Add online migration to move quotas to API database  https://review.openstack.org/41094616:27
melwittmriedem: your comments addressed ^16:29
*** yufei has quit IRC16:29
*** lyan has quit IRC16:29
s-deanmriedem: https://bugs.launchpad.net/nova/+bug/170611816:31
openstackLaunchpad bug 1706118 in OpenStack Compute (nova) "Adding TLS Connection String To Database Connection Causes malformed URL in cell_mappings DB" [Undecided,New]16:31
edleafedansmith: 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:32
*** baoli has joined #openstack-nova16:33
*** dtp has joined #openstack-nova16:35
*** crushil has joined #openstack-nova16:40
*** rajathagasthya has joined #openstack-nova16:41
*** Apoorva has joined #openstack-nova16:42
*** sshwarts has quit IRC16:42
*** ratailor has joined #openstack-nova16:43
*** jjung has quit IRC16:47
melwittmriedem: 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-api16:47
melwittsorry about that16:48
*** rajathagasthya has quit IRC16:48
*** rajathag_ has joined #openstack-nova16:48
melwittoh, it let me change it back16:50
melwittphew.16:50
*** armax has joined #openstack-nova16:52
dansmithedleafe: by batching I mean do 25 at a time, in that thread16:52
dansmithedleafe: like for i in range(0, len(uuids), 25): do_instances(uuids[i:i+25])16:53
*** mgiles has quit IRC16:56
edleafebatching which? I'm going through the nodes 1 by 1, and creating the instances one at a time16:56
*** mgiles has joined #openstack-nova16:56
*** derekh has quit IRC16:56
edleafeI thought that doing it that way would satisfy your memory explosion concern16:56
dansmithedleafe: it does, but now we're hammering the database and conductor with one-by-one queries on every single boot16:58
*** felipemonteiro_ has quit IRC16:59
*** ralonsoh has quit IRC17:02
*** randomhack has joined #openstack-nova17:05
*** lyan has joined #openstack-nova17:05
*** moshele has joined #openstack-nova17:05
*** fragatina has quit IRC17:06
*** ircuser-1 has joined #openstack-nova17:08
*** harlowja has joined #openstack-nova17:09
*** moshele has quit IRC17:10
*** randomhack has quit IRC17:13
*** efoley has quit IRC17:15
*** gszasz has quit IRC17:15
melwittdansmith, 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:16
dansmithmelwitt: I talked to mdbooth last week and it sounded like it wasn't going to happen17:17
dansmithmelwitt: especially since to do it right I think we'll need some refactoring of our db layer17:17
openstackgerritSean Dague proposed openstack/nova master: Suppress some test warnings  https://review.openstack.org/48670017:17
melwittdansmith: 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 up17:18
melwittone of us could be me, I mean17:18
dansmithwell, we could, but I don't think it's a reasonable thing to throw in at the last minute17:18
melwittk17:19
*** baoli has quit IRC17:20
openstackgerritChris Friesen proposed openstack/nova master: Ensure we unshelve in the cell the instance is mapped  https://review.openstack.org/48620817:21
cfriesenI 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:22
*** jmlowe has joined #openstack-nova17:22
*** jmlowe has quit IRC17:22
*** baoli has joined #openstack-nova17:23
*** jmlowe has joined #openstack-nova17:25
*** ratailor has quit IRC17:25
*** jmlowe has quit IRC17:26
*** randomhack has joined #openstack-nova17:28
*** jmlowe has joined #openstack-nova17:29
*** jmlowe has quit IRC17:31
*** jmlowe has joined #openstack-nova17:31
*** sambetts is now known as sambetts|afk17:33
*** fragatin_ has joined #openstack-nova17:34
*** fragatin_ has quit IRC17:34
*** fragatin_ has joined #openstack-nova17:34
s-deanhi, any pointers on how i may solve this issue ? MessagingTimeout: Timed out waiting for a reply to message ID17:35
s-deanits telling me to submit a bug report17:35
*** eharney has quit IRC17:35
s-deansorry not it> Nova*17:35
*** trinaths has quit IRC17:35
s-deannova.api.openstack.extensions [req-3ed5c135-64bd-4c39-934d-3d3d7a32a6b5 14d1055c2e2d4f0eab6739671b2688a8 cf7b827f55de4406b9325ab3505b973d - default default] Unexpected exception in API method17:36
*** fragatina has joined #openstack-nova17:37
*** fragatina has quit IRC17:37
*** baoli has quit IRC17:37
*** fragatin_ has quit IRC17:37
*** fragatina has joined #openstack-nova17:38
*** thorst_ has quit IRC17:38
melwitts-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 correct17:39
s-deanok cheers will do that now17:40
*** felipemonteiro has joined #openstack-nova17:43
*** felipemonteiro_ has joined #openstack-nova17:44
*** bkopilov has quit IRC17:44
*** ed___ has joined #openstack-nova17:44
*** thorst has joined #openstack-nova17:44
*** ed___ has quit IRC17:45
*** baoli has joined #openstack-nova17:45
*** edleafe_ has joined #openstack-nova17:45
*** bkopilov_ has quit IRC17:46
*** felipemonteiro has quit IRC17:47
s-deanyeah, cell1's RPC mapping was wrong, i have changed it and restarted all nova services, no dice :/17:49
s-deanstill same error17:49
*** thorst has quit IRC17:49
*** bkopilov_ has joined #openstack-nova17:49
s-deancan i ask would it be better to implement SSL/TLS after successfully setting up openstack unencrypted ?17:50
*** bkopilov has joined #openstack-nova17:50
*** baoli has quit IRC17:50
dansmiths-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:50
*** eharney has joined #openstack-nova17:50
*** yamahata has quit IRC17:50
melwitts-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-v217:51
mriedemmelwitt: i think https://blueprints.launchpad.net/nova/+spec/cells-count-resources-to-check-quota-in-api is done17:51
mriedemhttps://review.openstack.org/#/c/446243/ was the last change to merge17:52
*** mvk has quit IRC17:52
s-deanno i did not17:52
*** edleafe- has joined #openstack-nova17:52
melwitts-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 have17:52
s-deanok cheers17:53
melwittthese are stored in the database now because the API needs to know how to contact each of the cells you have17:53
melwittmriedem: agreed, I just didn't mean to set the Completed state because I think usually you do that17:53
melwittbut it does it automatically if you put Implemented17:54
*** kristian__ has quit IRC17:54
*** felipemonteiro_ has quit IRC17:54
*** felipemonteiro has joined #openstack-nova17:54
*** kristian__ has joined #openstack-nova17:55
mriedemmelwitt: i like to yeah because i have a spreadsheet keeping track of when things are closed out17:55
mriedemb/c i'm a nerd17:55
*** edleafe has quit IRC17:55
melwittmriedem: heh. I would probably do the same thing. don't want to mess up your system17:55
mriedemjust please be sure to put scissors, pens and dusters back where you got them if you're ever at my house17:56
mriedemand windex17:56
melwittlol17:56
*** edleafe_ has quit IRC17:56
*** rajathag_ is now known as rajathagasthya17:56
*** edleafe- is now known as edleafe17:56
*** claudiub has joined #openstack-nova17:58
*** kristian__ has quit IRC17:59
s-deando ineed to sync the db's after running that update command18:00
melwitts-dean: no, just need to restart your nova-api services I think18:00
dansmithunless he never properly sync'd it as a result18:00
*** FL1SK has quit IRC18:00
s-deanyeah, still timing out18:01
melwitthm18:01
openstackgerritMatt Riedemann proposed openstack/nova master: Provide hints when nova-manage db sync fails to sync cell0  https://review.openstack.org/48666018:02
dansmiths-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:03
*** mgiles has quit IRC18:05
*** dtantsur is now known as dtantsur|afk18:05
*** mgiles has joined #openstack-nova18:05
s-deanok 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.html18:06
*** jbernard has quit IRC18:06
mriedemsdague: did you ever report a bug for the request log thing with uwsgi?18:07
sdaguemriedem: I don't think I did actually18:07
*** wasmum has quit IRC18:09
mriedemsdague: ok, did the same thing need to be done in other projects like cinder?18:10
*** dtp has quit IRC18:10
mriedemif so, could be good to track with a bug for those other projects too18:10
*** wasmum has joined #openstack-nova18:10
*** jbernard has joined #openstack-nova18:11
*** thorst has joined #openstack-nova18:12
dansmiths-dean: so...the compute node is not correct?18:13
s-deandoes the username need to be consistent across all nodes, or is having different users for each compute node ok ?18:14
s-deanyes i gave the compute node its on users18:14
s-deanuser18:14
s-deanand assigned it all permissions18:14
s-deanown*18:14
dansmiths-dean: they can be different, they just all have to point at the same vhost of course18:14
s-deanin my case my vhost is /18:15
dansmithas long as that is the same for all the nodes then that's fine18:15
dansmiths-dean: aside from that, we need more context on which thing is failing, so more detailed logs, and a sanitized dump of your config and cell_mappings table are needed18:16
*** psachin has quit IRC18:16
*** rcernin has quit IRC18:16
s-deanpaste bin urls ok ?18:17
dansmithyeah18:17
*** prateek has quit IRC18:17
s-deanhttps://pastebin.com/WEu7JCWW18:18
dansmiths-dean: are you running n-net?18:20
dansmiths-dean: and looks like you're configured for cellsv1, is that intentional or did you think that was related?18:20
dansmithbecause this timeout is trying to call the cellsv1 service, which if you're not running it, will always time out :)18:20
s-deanoh ok18:21
s-deani didnt know i was running cells v118:22
s-deanis this because of the following config line compute_api_class=nova.compute.cells_api.ComputeCellsAPI18:22
s-deanmy config is here18:22
s-deanhttps://bugs.launchpad.net/nova/+bug/170611818:22
dansmiths-dean: you definitely know if you're running cellsv1 because it's a huge friggin nightmare,18:22
openstackLaunchpad bug 1706118 in OpenStack Compute (nova) ocata "Adding TLS Connection String To Database Connection Causes malformed URL in cell_mappings DB" [High,Confirmed]18:22
dansmithso I'm guessing maybe you configured that while trying to resolve this problem?18:22
s-deanyeah i have been poking about quite a bit18:23
dansmiths-dean: this is an upgrade from a previously working deployment, right?18:23
s-deanno fresh install, but because of that DB error, i have been trying to debug and find the right settings18:23
dansmithoh, okay18:24
dansmiths-dean: well, you went down a bad path18:24
*** felipemonteiro_ has joined #openstack-nova18:24
openstackgerritMatt Riedemann proposed openstack/nova master: deprecate ``wsgi_log_format`` config variable  https://review.openstack.org/48662318:24
openstackgerritMatt Riedemann proposed openstack/nova master: request_log addition for running under uwsgi  https://review.openstack.org/48560218:24
dansmiths-dean: might want to start fresh if it's not too much trouble and avoid any of that cellsv1 stuff, depending18:24
s-deanhaha, is my install screwed ?18:24
s-deanok i will od18:24
s-deando18:24
mriedemfear leads to anger, anger leads to hate, hate leads to cells v118:24
dansmithit's definitely rescue-able, but if you're trying to nail down a procedure it'll be confusing from here18:24
dansmithcellsv1 leads to?18:25
mriedemshowing up in nova freenode irc?18:25
openstackgerritKen'ichi Ohmichi proposed openstack/nova master: Remove the useless FakeExt  https://review.openstack.org/48641518:25
s-deanxD18:25
mriedemwhich is equivalent to the dark side18:25
openstackgerritKen'ichi Ohmichi proposed openstack/nova master: Remove the useless extension block_device_mapping_v1 object  https://review.openstack.org/48606918:25
openstackgerritKen'ichi Ohmichi proposed openstack/nova master: Remove the useless fake ExtensionManager from API unittests  https://review.openstack.org/48641618:25
s-deanawww man wish i came here sooner, i dont like asking for help i prefer to bash it out, and use google18:25
openstackgerritKen'ichi Ohmichi proposed openstack/nova master: Move the note about '/os-volume_boot' to the correct place  https://review.openstack.org/48607118:25
mriedemsdague: requestlog thing is approved18:27
mriedemsdague: you want to skim over this hot little number? https://review.openstack.org/#/c/485435/18:27
openstackgerritEd Leafe proposed openstack/nova master: Migrate Ironic Flavors  https://review.openstack.org/48494918:27
*** felipemonteiro has quit IRC18:27
*** yamahata has joined #openstack-nova18:30
*** baoli has joined #openstack-nova18:32
*** mgiles has quit IRC18:34
*** mgiles has joined #openstack-nova18:34
dansmithjaypipes: edleafe: I was thinking that the ironic node class was required now.. do we really need to concern ourselves with that not being set?18:35
s-deanshouldnt that config option be removed from the docs ?18:35
s-deanhttps://docs.openstack.org/ocata/config-reference/compute/cells.html18:36
dansmiths-dean: we can't yet18:36
dansmiths-dean: people still run that stuff and have to for a bit longer18:36
s-deana fair enough18:36
*** baoli has quit IRC18:37
*** wasmum has quit IRC18:41
mriedems-dean: we're in the process of moving those docs into nova, at which point we can put a big fat warning at the top saying that's for cells v118:42
edleafejaypipes: regarding your comments about splitting the patch for https://review.openstack.org/#/c/486215/, do you want me to build the list of alternates, but continue to only return the selected hosts? And then change all the method signatures in https://review.openstack.org/#/c/486253/ ?18:43
*** wasmum has joined #openstack-nova18:43
edleafedansmith: I asked dtantsur|afk about that. He said that while adding the resource_class attribute was required, populating it completely was not18:43
dansmithedleafe: okay so, we should check it and if it's not present, log a warning and bail?18:44
edleafedansmith: if it's not present, just skip18:45
dansmithedleafe: we have to log a warning, since not having it set will lead to brokenness later, but.. your code currently is not skipping right?18:46
dansmithin PS10 I mean18:46
edleafedansmith: correct. Want me to add some safety stuff there?18:47
dansmithedleafe: yeah, warn and skip if unset, test for those cases and then I think we can land this and argue about the batching after18:47
s-deanmriedem: that would be brilliant. before i head home for the night, can i specify in keystone, glance and nova their database connections to be localhost skipping the need for TLS, or do they need to be specified with a network address like in the docs? do the other service such as neutron and and nova-compute need to access the DB over the network?18:48
edleafedansmith: ok cool. I asked in -ironic about how often the compute service would be restarted. Got one answer from jroll, and that was he only did it during upgrades18:49
edleafeIOW, not a common thing for him18:49
*** jamesden_ has joined #openstack-nova18:49
mriedems-dean: nova-compute definitely does not use the db locally18:49
mriedemi don't know about the various neutron agents18:49
*** jamesdenton has quit IRC18:49
dansmithedleafe: heh, sure, but jroll knows what he's doing18:49
mriedemor cinder-volume18:49
dansmithedleafe: like I said there are other config management approaches that are pretty darn restart happy18:49
dansmithI know of one very specifically18:50
dansmithone that has lots of Os in it18:50
*** itlinux has joined #openstack-nova18:53
mriedems-dean: got it https://review.openstack.org/#/c/486724/18:53
jaypipesedleafe: re: separate patch, yeah, I'm thinking it's best to generate that list of alternates in the first patch (but not change the returned value from select_destinations() and then in the followup patch, change the RPC API signature and the corresponding conductor stuff18:53
s-deanmriedem: I'm installing a general compute cloud, nothing to fancy,18:53
openstackgerritMerged openstack/nova master: placement: add retry tight loop claim_resources()  https://review.openstack.org/48617018:54
s-deanmriedem: cheers :)18:54
dansmithjaypipes: edleafe agreed18:54
mriedems-dean: but the nova-compute services are running on separate hosts right/18:54
mriedem?18:54
s-deanyes18:54
mriedemnova-compute will interact with the db via nova-conductor18:54
s-deancompute0118:54
s-deanok sweet18:54
edleafejaypipes: ok, so the last change will go from the current return of a list of hosts to the monsterous list of hosts+alternates+allocation_candidates, right?18:54
mriedemso if nova-conductor is on the control nodes local to the db, then you could use localhost for nova.conf for conductor18:54
mriedembut not nova-compute18:54
mriedems-dean: nova-compute shouldn't even need the [database] section filled in for nova.conf18:55
s-deanok awsome, yeah all the scheduler conductor novncproxy etc are on 1 controller node18:55
mriedemthen localhost or 127.0.0.1 is fine for those18:55
s-deanlooks like neutron needs access to the DB from across the network :/18:55
s-deanill cross that brige when i come to it i guess18:55
mriedemi'm not familar with everything they do18:56
jaypipesedleafe: not sure that we need to return a list of alternates instead of just a list of allocation candidates, but wondering what dansmith thinks.18:56
mriedems/everything/most anything/18:56
*** claudiub has quit IRC18:56
jaypipesedleafe: the only reason to do so would be to keep the allocation candidates entirely opaque18:56
*** jwcroppe has quit IRC18:58
dansmithjaypipes: we have to keep allocation candidates18:58
dansmithjaypipes: the cell conductor can't know what else to claim otherwise.. a hostname/uuid isn't enough18:58
*** jwcroppe has joined #openstack-nova18:59
jaypipesdansmith: no, I'm referring to whether we return a list of alternate hosts PLUS the allocation candidates OR just a list of allocation candidates.18:59
dansmithjaypipes: ah okay18:59
*** rajathag_ has joined #openstack-nova19:00
mriedemwould the alternate hosts be used for anything?19:00
mriedemdoes conductor use them to monkey with the request spec or scheduler hints?19:00
*** annegentle has quit IRC19:00
jaypipesmriedem: nothing other than preventing the conductor from needing to know anything about the allocation_candidates.19:00
jaypipesmriedem: the alternate_hosts would replace the retry block in the request spec essentially.19:01
*** rajathagasthya has quit IRC19:01
dansmithjaypipes: if that allows conductor to be ignorant of theallocation then yes, good plan19:01
jaypipesdansmith: k.\19:01
*** jwcroppe_ has joined #openstack-nova19:01
jaypipesthe decision comes down to how much opacity we want those allocation_request blocks to be. :)19:01
mriedemwhere is the code being proposed that needs them together?19:02
jaypipesmriedem: edleafe's currently working on the series.19:02
*** jwcroppe has quit IRC19:03
edleafejaypipes: the way I understood it is that we would return a series of alternate hosts in case the cell had to retry a build. Each of those hosts would need its corresponding allocation_candidate so that the cell conductor could do the proper claiming.19:03
dansmithright19:04
dansmithhowever, I was thinking that:19:04
jaypipesedleafe: yes, that's exactly correct. I was only pointing out that technically the allocation_requests contain all the information in the alternate_hosts list.19:04
dansmithconductor would throw the first allocation candidate at placement, and then parse the result to determine which compute host it should send the rpc message to19:04
jaypipesedleafe: but like I said, that would require the cell conductor to understand what an allocation request was (i.e. the allocation_request would no longer be opaque)19:05
dansmiththat would mean no extra list, but also opaque allocation request19:05
dansmithjaypipes: not if ^19:05
jaypipesright. :)19:05
mriedem"conductor would throw the first allocation candidate at placement" - that's the cell conductor yes?19:05
edleafedansmith: "parse the result"?19:05
mriedemduring a retry19:05
dansmithedleafe: parse the result of the POST of the allocation19:06
jaypipesI think the most appropriate return value from select_destinations() would be a list of (host, allocation_request) tuples.19:06
jaypipesfor the first item in that list, the allocation_request would be the one that had already been successfully claimed for the selected host.19:07
jaypipesdansmith: agree?19:07
mriedemselect_destinations today returns as the first entry the one that was chosen, right?19:07
dansmithsure that's fine, if that's how you want it to look19:07
jaypipesmriedem: for each instance in num_instances, yes19:07
edleafedansmith: reportclient.claim_resources returns a boolean19:08
jaypipesso actually, the return value needs to be list of list of that tuple.19:08
dansmithedleafe: what's your point?19:08
jaypipeswith the outer list being for the num_instances19:08
edleafedansmith: what's there to parse?19:08
mriedema list of lists of tuples19:08
mriedemwhat could go wrong19:08
edleafedansmith: the cell conductor would still need to "know" about the allocation_candidate structure19:09
dansmithedleafe: the actual POST call for /allocations returns the allocation you made right?19:09
mriedemit's not a POST19:09
edleafeIt's a PUT19:09
dansmithedleafe: the cell conductor can, but I think it should look at the result of the http call not the thing it was passed in rpc, otherwise we've got version mess19:09
edleafeAnd it returns a 204 on success19:10
dansmithchrist, whatever19:10
mriedemyeah no content on success19:10
jaypipesmriedem: well, tell me if you want to stop supporting num_instances > 1 and I'll gladly submit that patch ;)19:10
dansmithokay then that clearly won't work19:10
edleafeFor each host, you get a list of (host, alloc) tuples.19:11
jaypipesright19:11
jaypipes++19:11
edleafeOn a retry in the cell, you try claiming the alloc. If that succeeds, you build on that host19:12
jaypipes+119:12
dansmithI don't love it, but I'm also not sure why we're even discussing it19:12
edleafeIf it fails, move to the next one in the list19:12
jaypipesright, zactly.\19:12
edleafejaypipes: so I don't understand why you would want to only return alloc19:12
mriedemand just to confirm my understand, we only ever care about the list of lists for the server create case, b/c for everything else, like migrations and unshelve, it gets back the list of host states today but just takes the first one for the rpc cast to compute19:14
mriedem*understanding19:14
jaypipesedleafe: never mind my thought about only returning the allocations. I've been convinced that's a bad idea.19:14
edleafejaypipes: roger that19:15
*** thorst is now known as thorst_afk19:15
mriedemseems you have to have both the HostState and allocation requests because of all the filter properties and az and limits and node crap that's embedded in the HostState object19:15
mriedemwhich conductor is using before casting to compute19:16
mriedemyeah?19:16
dansmithhope not since hoststate is very scheduler-specific19:16
dansmithall conductor needs is the hostname of the target compute19:16
mriedemsec19:16
mriedemhttps://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L689-L70019:16
mriedem^ just for unshelve19:16
dansmithoh, that's not HostState,19:17
dansmiththat's the dict of randomness19:17
mriedembut there is all sorts of redonkulous in there for limits and such19:17
dansmithwhich was based on HostState19:17
mriedemthis? https://github.com/openstack/nova/blob/master/nova/scheduler/manager.py#L4819:17
dansmithyeah, that's going to fsck us19:17
mriedemstill has limits in it19:17
mriedemb/c N-mfing-UMA19:17
dansmithoh well, nice knowing you gents19:17
* dansmith pulls the trigger19:18
mriedemwell now that dan is taken care of19:18
mriedemand i guess it's getting the az from the chosen host via the aggregates on that host19:19
mriedemlimits go into filter properties for the claim in the compute19:19
mriedemand i guess the node is also passed down explicitly for the RT claim19:19
mriedemi wonder if anyone has ever tried shelve offloading and unshelving an instance that had pci/numa stuff on it :)19:20
dansmiththat is also an unversioned dict result from that scheduler rpc call19:22
dansmithwe should *not* commit that same sin in the boot calls19:22
*** baoli has joined #openstack-nova19:22
jaypipesok, let's focus on what we need to do here to get retries working in cellsv219:23
jaypipesare we all in agreement about the proposed return value from select_destinations() for edleafe's patch?19:23
dansmithno I think that's mriedem's point19:23
jaypipesa list of lists of (host, alloc_request) tuples, yes?19:23
mriedemwe agree we need allocation requests and the host_state dict thingies i think19:23
mriedemor maybe we don't agree19:24
dansmithI think he's saying we need to pass the HostState mess, right mriedem ?19:24
mriedemyes19:24
* jaypipes goes to investigate whether limits['numa'19:24
jaypipesis even used....19:24
mriedemit is19:24
mriedemin the claim code19:24
dansmithoh it is19:24
dansmithyeah19:24
* dansmith hands jaypipes the 9mm19:24
mriedemnote that all of the docstrings say the limits are vcpus/ram/disk19:24
mriedemnot numa19:24
mriedemso it's totally f'ing confusing19:24
mriedemno desert eagle?19:25
mriedemyou want open casket?19:25
jaypipesyeah, it is. :(19:25
dansmithmriedem: it's what I have within reach19:25
jaypipesdansmith, mriedem: k, so the returned value needs to be list of lists of (host_dict_with_limits_thing, alloc_request)19:25
jaypipesedleafe: ^19:25
dansmithno19:26
dansmithbecause19:26
dansmithhost_dict_with_limits is an unversioned structure and we're not adding a parameter with a LIST OF TUPLES OF THAT THING to one of our clean rpc calls19:26
mriedemclean rpc calls?19:26
jaypipesdansmith: don't we already pass the limits stuff down to build_instance()?19:27
* jaypipes wonders where we get the limits thing...19:27
mriedemselect_destinations today returns the unversioned host_dict_with_limits_thing19:27
dansmithjaypipes: no we get it fromthe scheduler19:27
mriedemjaypipes: the NUMATopologyFilter19:27
jaypipesmriedem: no, I know that. I meant where does the limits get passed to instance_claim() on the compute host.19:27
mriedemjaypipes: let me present exhibit Z98 https://review.openstack.org/#/c/484439/19:27
mriedemvia the filter props19:28
mriedemactually no,19:28
mriedembuild_and_run_instance has a limits kwarg19:28
mriedemwhich is the limits from the scheduler19:28
* dansmith pours gas on this whole thing and lights a match19:28
mriedemwhich as far as i know just contains the numa topology limits19:28
mriedemEXCEPT19:28
mriedemfor out of tree scheduler drivers that stash crap in there too :)19:28
mriedem*and filters19:28
dansmithhmm19:29
dansmithso maybe we talk to scheduler late enough in this process that we're only calling ugly rpc calls with limits as a param from here on out?19:29
mriedemdo we need a hangout?19:29
dansmithI need a therapist.19:29
mriedemlaura's mom went to grad school for that i think, she'll talk your ear off19:30
mriedemabout vacuums that also polish wood floors19:30
mriedemcontrolled via your iphone app19:30
jaypipesI'm wondering if we can't just recreate the limits['numa_topology'] on the compute host....19:30
mriedemi guess i'm failing to see the issue with sending both back from select_destinations19:31
mriedemwe're already sending host_dict_with_limits_thing19:31
jaypipesdansmith: ^19:31
mriedemwe're just tacking allocation_request(s)? onto that19:31
mriedemas a tuple19:31
mriedembtw, are these allocation requests plural or singular?19:31
mriedemone per host right? so singular19:32
edleafemriedem: we're changing 2 things19:32
jaypipesmriedem: singular per host, yeah.19:32
edleafeinstead of a single host19:32
dansmithyeah, like I said above, I think I was missing that we're already passing that grossness in the rpc calls downstream from where we get them19:32
*** rajathag_ has quit IRC19:32
edleafewe're sending a list of hosts19:32
edleafeand each of those has an associated allocation_candidate19:32
mriedemedleafe: please provide context19:32
mriedem"sending" from where to where?19:32
edleafeto the cell conductor19:32
edleafefrom the super conductor19:32
dansmithsuper conductor doesn't call cell conductor19:33
dansmithsuper conductor calls the first compute node, which will call cell conductor on retry19:33
mriedemedleafe: ok so here https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L104619:33
edleafeI thought that was changing so that we supported alternates19:33
*** randomhack has quit IRC19:33
dansmithno19:34
edleafeSo we're sending the compute node the big honking list of list of stuff?19:34
mriedemapi -> super conductor > scheduler > super conductor > compute > cell conductor (retry) > compute19:34
dansmithright19:34
edleafeAnd then it sends it to cell conductor on retry?19:34
mriedemyes19:34
mriedemthe allocation requests are getting passed through19:35
mriedemlike barnacles19:35
mriedemor a tube sock in mortier19:35
mriedem*mortimer19:35
edleafemore like kidney stones19:35
mriedem2nd question,19:35
melwittlol omg, mortimer19:35
mriedemare we changing the build_and_run_instances compute rpc api to pass allocation requests, or shoving those into something else already being sent, like request spec or filter properties?19:36
*** vladikr has quit IRC19:37
dansmithone of those19:37
mriedemwhich would also impact the build_and_run_instance method in conductor rpc api19:37
*** vladikr has joined #openstack-nova19:37
mriedem*build_instances19:38
dansmithif we put it into something like reqspec, we won't be able to send them to older computes19:38
dansmithwhich will break our upgrade process19:38
dansmithbecause they'll kick the newer version back19:38
mriedempersonally i think it's cleaner as a new parameter on the rpc api19:39
dansmithwhich I guess is the same for the new param approach19:39
dansmithwe just need to handle the case in the retry logic,19:39
dansmithif we didn't get these new things, assume we can talk to the scheduler and do a reschedule19:39
mriedembut if your cell conductor is blocked from up calls to the scheduler, how would that work?19:40
dansmithif you have old computes,19:40
dansmiththen you don't have a multi-tier cellsv2 environment,19:40
dansmithbecause we didn't support it before,19:40
dansmiththus it must be fine19:40
openstackgerritEd Leafe proposed openstack/nova master: Migrate Ironic Flavors  https://review.openstack.org/48494919:40
mriedemidk, we don't really have anything doc'ed for this19:41
dansmitheh?19:41
dansmithit can't work right now19:41
dansmithno need to doc it :)19:41
mriedemwhat does old computes have to do with multi-tier cells v2?19:41
dansmithcan we discuss on a hangout? I'm about out of steam and have to move onto something else soon19:42
dansmithbe quicker if we can hash it out like that I think19:42
mriedemi'm fine with a hangout19:42
jaypipesme too.19:42
*** avolkov has joined #openstack-nova19:42
* dansmith urls19:42
dansmithhttps://hangouts.google.com/call/pjno3bssgba33a343nf47d357iu19:43
*** jwcroppe_ has quit IRC19:45
*** jwcroppe has joined #openstack-nova19:45
*** randomhack has joined #openstack-nova19:46
*** mgiles has quit IRC19:48
*** danpawlik has quit IRC19:48
*** mgiles has joined #openstack-nova19:48
*** jwcroppe has quit IRC19:50
*** danpawlik has joined #openstack-nova19:50
openstackgerritJackie Truong proposed openstack/nova master: Add trusted certificates to InstanceExtras  https://review.openstack.org/45771119:52
*** jwcroppe has joined #openstack-nova19:53
openstackgerritMerged openstack/nova master: Remove an unnecessary argument in _prep_resize  https://review.openstack.org/48652119:53
openstackgerritMerged openstack/python-novaclient master: Expect id and disabled_reason in GET /os-services response  https://review.openstack.org/48540919:54
*** rajathagasthya has joined #openstack-nova19:54
*** ekhugen has quit IRC20:03
*** Guest30741 has quit IRC20:04
*** jwcroppe has quit IRC20:05
*** ekhugen_alt has quit IRC20:05
*** patriciadomin has quit IRC20:05
*** mdrabe has quit IRC20:05
*** Sukhdev has joined #openstack-nova20:05
*** Apoorva_ has joined #openstack-nova20:08
*** crushil has quit IRC20:08
*** Apoorva_ has quit IRC20:09
openstackgerritEric Fried proposed openstack/nova master: nova.utils.get_service_url()  https://review.openstack.org/45825720:09
*** Apoorva_ has joined #openstack-nova20:09
*** mdrabe has joined #openstack-nova20:09
*** patriciadomin has joined #openstack-nova20:09
*** ekhugen has joined #openstack-nova20:09
*** ekhugen_alt has joined #openstack-nova20:10
*** egarbade has joined #openstack-nova20:10
*** Apoorva_ has quit IRC20:10
*** egarbade is now known as Guest5173820:10
*** Apoorva_ has joined #openstack-nova20:10
efriedmriedem mordred os-service-types didn't get released last week.  If it gets released soon, is it too late to get it through and use it in nova for pike?20:10
*** Apoorva has quit IRC20:11
mriedemefried: i haven't reviewed the nova code so can't really say what the risk is, or if this is all disabled by default and then people have to opt-in, or what20:11
efriedmriedem I was about to crank back up on the nova side, now that all the ksa stuff has landed.  But one of the main pieces I need is os-service-types.  If I can't have that, I'm going to have to basically inline a bunch of it.20:12
efriedmriedem os-service-types is brand new, so there's no risk of breaking backward compatibility or whatever.20:13
efriedmriedem It'll be a brand new dep, for whatever that means.20:13
*** mgiles has quit IRC20:16
*** mgiles has joined #openstack-nova20:17
*** mgiles has quit IRC20:19
*** mgiles has joined #openstack-nova20:19
*** randomhack has quit IRC20:22
*** jwcroppe has joined #openstack-nova20:24
mriedemedleafe: so let's hold off on the alternatives stuff for pike, it's too high risk at this point in the schedule, we'll work on that for queens. it means you can't do multi-tier multi-cell with retries, but if you don't care about retries, like CERN, then you're still golden for pike.20:27
mriedemedleafe: so focus on the ironic flavor migration stuff for pike20:28
edleafemriedem: ok20:30
*** baoli has quit IRC20:31
*** mgiles has quit IRC20:35
*** mgiles has joined #openstack-nova20:35
*** mgiles has quit IRC20:37
*** mgiles has joined #openstack-nova20:38
*** kristian__ has joined #openstack-nova20:40
*** jdurgin has quit IRC20:41
*** baoli has joined #openstack-nova20:44
*** kristian__ has quit IRC20:44
*** baoli has quit IRC20:45
*** baoli has joined #openstack-nova20:45
*** eharney has quit IRC20:47
*** crushil has joined #openstack-nova20:48
*** kristian__ has joined #openstack-nova20:50
*** burt has quit IRC20:52
*** annegentle has joined #openstack-nova20:54
*** mgiles has quit IRC21:00
*** mgiles has joined #openstack-nova21:01
*** cleong has quit IRC21:01
*** edmondsw has quit IRC21:04
openstackgerritJeroen van Bemmel proposed openstack/nova master: Closes-Bug: 1702475  https://review.openstack.org/48675321:04
openstackbug 1702475 in OpenStack Compute (nova) "IPv6 data missing from latest/metadata info" [Medium,Confirmed] https://launchpad.net/bugs/170247521:04
*** edmondsw has joined #openstack-nova21:04
*** crushil has quit IRC21:05
openstackgerritJeroen van Bemmel proposed openstack/nova master: Closes-Bug: 1702475  https://review.openstack.org/48675321:05
*** armax has quit IRC21:05
*** edmondsw_ has joined #openstack-nova21:05
*** crushil has joined #openstack-nova21:08
*** edmondsw has quit IRC21:09
*** yamamoto_ has joined #openstack-nova21:09
*** crushil has quit IRC21:10
*** edmondsw_ has quit IRC21:10
*** crushil has joined #openstack-nova21:10
*** jmlowe has quit IRC21:11
*** cdent has joined #openstack-nova21:12
*** baoli has quit IRC21:15
*** crushil has quit IRC21:16
*** baoli has joined #openstack-nova21:16
*** crushil has joined #openstack-nova21:18
*** MVenesio has quit IRC21:19
*** smatzek has quit IRC21:20
*** yamamoto_ has quit IRC21:20
*** baoli has quit IRC21:20
*** crushil has quit IRC21:21
*** jwcroppe has quit IRC21:26
*** crushil has joined #openstack-nova21:26
*** mvk has joined #openstack-nova21:28
*** crushil has quit IRC21:28
*** baoli has joined #openstack-nova21:28
*** jdurgin has joined #openstack-nova21:29
openstackgerritMerged openstack/nova master: Make Quotas object favor the API database  https://review.openstack.org/41094521:31
mriedemcdent: the thing that your wsgi-intercept patch failed on failed in another unrelated change http://logs.openstack.org/02/485602/6/check/gate-nova-tox-functional-ubuntu-xenial/edf4c41/testr_results.html.gz21:32
mriedemso probably just some new fun21:32
cdentle sigh21:32
*** jwcroppe has joined #openstack-nova21:32
cdenti have the OSAPIFixture using wsgi-intercept in progress21:32
cdentI might stack them and see if that gets us anywhere21:33
*** Apoorva_ has quit IRC21:34
*** Apoorva has joined #openstack-nova21:34
*** crushil has joined #openstack-nova21:38
*** cdent has quit IRC21:38
*** edmondsw has joined #openstack-nova21:39
*** awaugama has quit IRC21:39
*** crushil has quit IRC21:40
*** baoli has quit IRC21:41
*** crushil has joined #openstack-nova21:41
*** jwcroppe has quit IRC21:42
openstackgerritEd Leafe proposed openstack/nova master: Migrate Ironic Flavors  https://review.openstack.org/48494921:42
*** crushil has quit IRC21:43
*** edmondsw has quit IRC21:44
*** crushil has joined #openstack-nova21:44
*** jwcroppe has joined #openstack-nova21:45
*** esberglu has quit IRC21:46
*** Apoorva has quit IRC21:47
*** crushil has quit IRC21:48
*** Apoorva has joined #openstack-nova21:48
*** crushil has joined #openstack-nova21:49
*** jmlowe has joined #openstack-nova21:50
*** crushil has quit IRC21:50
mriedemthink i know what's causing the spike in the functional tests failing21:51
mriedemhttps://review.openstack.org/#/c/484154/2/nova/tests/functional/api/openstack/placement/gabbits/resource-class-in-use.yaml21:51
mriedemi think we have gabbits racing that work against the same custom resource class21:52
*** chyka has quit IRC21:52
*** chyka has joined #openstack-nova21:52
*** marst has quit IRC21:52
dansmiththey use the same db instance in parallel?21:52
mriedemcreating the same custom resource class,21:52
mriedemin one case it already exists i think so it returns 20421:53
mriedemwhich fails the assertion for a 20121:53
mriedemhttps://bugs.launchpad.net/nova/+bug/170620721:53
openstackLaunchpad bug 1706207 in OpenStack Compute (nova) "resource-class-in-use_delete_resource_class fails with "AssertionError: '404' not found in ['204']" since 7/22" [High,Confirmed]21:53
dansmithoh, the same placement fixture I guess?21:54
mriedemyeah21:54
dansmithI see21:54
*** xyang1 has quit IRC21:54
*** armax has joined #openstack-nova21:56
mriedemalthough these tests should be using isolated sqlite dbs21:57
mriedemso i'm not sure why this should happen21:57
*** mgiles has quit IRC21:59
*** felipemonteiro_ has quit IRC22:00
mriedem"On a successful create return 201. Return 204 if the class already22:00
mriedem    exists."22:00
mriedemnot sure why the test is doing a PUT instead of a POST22:00
mriedemi guess because the PUT handles both cases22:01
openstackgerritDan Smith proposed openstack/nova master: Migrate Ironic Flavors  https://review.openstack.org/48494922:01
*** yamamoto has joined #openstack-nova22:01
dansmithjaypipes: can haz: https://review.openstack.org/#/c/48494922:01
dansmithmriedem: well, I thought you meant we're hitting the wrong fixture22:02
dansmithlike starting two but always hitting the first one from the tests or something22:02
*** itlinux has quit IRC22:04
openstackgerritMatt Riedemann proposed openstack/nova master: Add tags to instance.create Notification  https://review.openstack.org/45949322:05
*** esberglu has joined #openstack-nova22:05
dansmithmriedem: I'll fix that since I have it open22:06
openstackgerritDan Smith proposed openstack/nova master: Migrate Ironic Flavors  https://review.openstack.org/48494922:06
jaypipesdansmith: done22:07
dansmithjaypipes: thanks22:07
jaypipespas de probleme22:08
dansmith"done" about sums it up for me too22:08
mriedemhttps://review.openstack.org/#/c/459493/ closes out the searchlight notifications bp22:08
jaypipesheh22:08
*** esberglu has quit IRC22:10
*** jgriffith has quit IRC22:10
*** jgriffith has joined #openstack-nova22:10
openstackgerritMerged openstack/nova master: doc: Populate the 'configuration' section  https://review.openstack.org/47847322:12
*** jgriffith has quit IRC22:12
openstackgerritMerged openstack/nova master: doc: Populate the 'contributor' section  https://review.openstack.org/47847422:12
openstackgerritMerged openstack/nova master: doc: Populate the 'reference' section  https://review.openstack.org/47848422:13
*** kylek3h has quit IRC22:13
*** fragatin_ has joined #openstack-nova22:14
mriedemso there is one other set of gabbits that create a CUSTOM_GOLD resource class22:14
mriedembut they both use the APIFixture which creates separate db fixtures22:15
mriedemso i'm not sure how those would overlap22:15
mriedemunless it's eventlet again22:15
*** jgriffith has joined #openstack-nova22:15
*** jgriffith has quit IRC22:16
*** fragatina has quit IRC22:18
mriedemwhen this failed, both tests were running on the same process22:18
mriedemso maybe22:18
*** jgriffith has joined #openstack-nova22:19
*** jgriffith has quit IRC22:19
openstackgerritMatt Riedemann proposed openstack/nova master: Use different resource class for resource-class-in-use tests  https://review.openstack.org/48678222:22
mriedemsee if you can stomach this ^22:22
openstackgerritMatt Riedemann proposed openstack/nova master: Use different resource class for resource-class-in-use tests  https://review.openstack.org/48678222:23
*** jgriffith has joined #openstack-nova22:23
openstackgerritMerged openstack/nova master: doc: Populate the 'user' section  https://review.openstack.org/47847522:25
openstackgerritMerged openstack/nova master: Update ironic feature matrix  https://review.openstack.org/48614822:25
*** gouthamr has quit IRC22:27
mriedemmelwitt: simple issue in https://review.openstack.org/#/c/410946/22:31
*** acormier has quit IRC22:31
mriedemand then i'll +2 before dansmith is completely out22:31
*** jaypipes has quit IRC22:32
*** acormier has joined #openstack-nova22:32
dansmithmriedem: worth trying that rc change right?22:32
dansmithmriedem: jay is out so unless melwitt responds soon and is willing, we might as well slam that in22:33
mriedemi'll update it22:34
dansmithmriedem: I mean your resource class gabbit thing22:34
mriedemoh22:34
mriedemyes it's worth trying22:34
mriedemwon't hurt22:34
mriedemslamaro22:34
mriedem*slamaroo22:34
dansmithslamaroo'd22:35
*** acormier has quit IRC22:36
mriedemrebuild reno for the quotas change and then will push that up22:36
mriedem*rebuilding22:37
cfriesenwhat will be the ordering of steps when adding a new cellsv2 cell?  create the new cell in the central DB and then start up the services or vice versa?22:37
*** jwcroppe has quit IRC22:37
mriedemcreate the cell first22:38
mriedemand then start services within it22:39
mriedemjust like with the single cell case22:39
cfriesenokay, that makes sense22:40
*** gouthamr has joined #openstack-nova22:42
*** fragatin_ has quit IRC22:42
cfriesenmriedem: I guess we need the DB server for the new cell up and running before creating the new cell in nova22:43
*** fragatina has joined #openstack-nova22:43
cfriesenor at least we need to know what it will be22:43
*** jwcroppe has joined #openstack-nova22:44
mriedemyes22:46
openstackgerritMatt Riedemann proposed openstack/nova master: Add online migration to move quotas to API database  https://review.openstack.org/41094622:46
mriedembecause when you run nova-manage cell_v2 create_cell you need to specify the MQ and DB URLs for the new cell22:46
mriedemwhich get inserted into the cell_mappings table in the nova_api db22:46
mriedemdansmith: ^ addressed the comments in the quotas changetest_migrate_quota_limits22:47
mriedemoops22:47
mriedemhttps://review.openstack.org/41094622:47
openstackgerritGage Hugo proposed openstack/nova master: Admin API Policy contingent on is_admin_project  https://review.openstack.org/38414822:55
*** crushil has joined #openstack-nova22:57
*** crushil has quit IRC23:01
*** itlinux has joined #openstack-nova23:05
*** Sukhdev has quit IRC23:05
*** danpawlik has quit IRC23:06
*** jwcroppe has quit IRC23:06
*** itlinux has quit IRC23:10
melwittmriedem, dansmith: thanks yall23:16
melwittI was at the dentist, being angry23:17
*** randomhack has joined #openstack-nova23:19
*** catintheroof has quit IRC23:21
*** randomhack has quit IRC23:23
*** chyka has quit IRC23:26
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2)  https://review.openstack.org/40895523:28
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2)  https://review.openstack.org/40896423:29
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004223:30
*** acormier has joined #openstack-nova23:33
*** takashin has joined #openstack-nova23:34
*** acormier has quit IRC23:37
openstackgerritKen'ichi Ohmichi proposed openstack/nova master: Remove the useless fake ExtensionManager from API unittests  https://review.openstack.org/48641623:38
*** rajathagasthya has quit IRC23:42
*** coreywright has quit IRC23:42

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!