Wednesday, 2017-12-06

ildikovmriedem_away: ok, I'll look into those, thanks!00:00
*** cdent has quit IRC00:02
*** Guest35678 has quit IRC00:07
*** _pewp_ has quit IRC00:20
*** yamahata has quit IRC00:20
*** sdague has quit IRC00:20
*** _pewp_ has joined #openstack-nova00:21
*** jmlowe has quit IRC00:22
*** takashin has joined #openstack-nova00:24
*** threestrands has quit IRC00:25
*** sdague has joined #openstack-nova00:26
*** chyka_ has quit IRC00:31
*** gmann_afk is now known as gmann00:36
*** mlavalle has quit IRC00:37
*** TuanLA has joined #openstack-nova00:44
*** hongbin has quit IRC00:49
*** hieulq has joined #openstack-nova00:50
*** yassine has joined #openstack-nova01:00
*** yassine is now known as Guest1893001:00
*** itlinux has quit IRC01:00
openstackgerritWangpan proposed openstack/nova master: Handle glance exception during rotating instance backup  https://review.openstack.org/52339501:02
*** esberglu has joined #openstack-nova01:02
*** tbachman has joined #openstack-nova01:06
*** esberglu has quit IRC01:07
*** jichen has joined #openstack-nova01:07
*** tristanC has quit IRC01:08
*** phuongnh has joined #openstack-nova01:09
*** tristanC has joined #openstack-nova01:09
*** frickler has quit IRC01:09
*** frickler has joined #openstack-nova01:10
*** gyee has quit IRC01:10
*** yikun has joined #openstack-nova01:11
*** yangyapeng has joined #openstack-nova01:14
*** yamahata has joined #openstack-nova01:25
*** auggy has joined #openstack-nova01:25
*** psachin has joined #openstack-nova01:30
*** Dinesh_Bhor has joined #openstack-nova01:31
*** liusheng has quit IRC01:32
*** Eran_Kuris_ has quit IRC01:32
*** _pewp_ has quit IRC01:33
*** Dinesh_Bhor has quit IRC01:35
*** Dinesh_Bhor has joined #openstack-nova01:35
*** _pewp_ has joined #openstack-nova01:36
*** markvoelker has quit IRC01:38
*** liusheng has joined #openstack-nova01:46
*** zhurong has joined #openstack-nova01:50
*** mriedem_away has quit IRC01:50
*** namnh has joined #openstack-nova01:53
*** hshiina has joined #openstack-nova01:56
*** chyka has joined #openstack-nova02:00
*** chyka has quit IRC02:04
*** Tom-Tom has joined #openstack-nova02:05
*** Tom-Tom has quit IRC02:06
*** Tom-Tom has joined #openstack-nova02:06
*** mriedem has joined #openstack-nova02:10
mriedemedleafe: i led you astray on the compute rpcapi conversion stuff from https://review.openstack.org/#/c/516707/02:10
mriedemyou need that isinstance check in the rpcapi client to convert the SchedulerLimits object to a dict before sending to compute, that's why the functional tests are failing,02:11
mriedemwe can't send the object form until the computes are upgraded to handle that, which would be an rpc api version change02:11
*** oanson has quit IRC02:12
*** hongbin has joined #openstack-nova02:12
*** oanson has joined #openstack-nova02:12
*** mriedem has quit IRC02:23
*** fragatin_ has quit IRC02:27
*** hieulq has quit IRC02:32
*** hongbin_ has joined #openstack-nova02:33
*** Apoorva_ has joined #openstack-nova02:33
*** hieulq has joined #openstack-nova02:34
*** Apoorva has quit IRC02:36
*** hongbin has quit IRC02:37
*** Apoorva_ has quit IRC02:37
*** markvoelker has joined #openstack-nova02:39
*** itlinux has joined #openstack-nova02:45
*** dave-mccowan has joined #openstack-nova02:45
*** esberglu has joined #openstack-nova02:51
*** esberglu has quit IRC02:52
*** esberglu has joined #openstack-nova02:52
*** itlinux has quit IRC02:53
*** esberglu has quit IRC02:57
*** Tom-Tom has quit IRC02:59
*** Tom-Tom has joined #openstack-nova02:59
*** Tom-Tom has quit IRC03:00
*** Tom-Tom has joined #openstack-nova03:00
*** moshele has joined #openstack-nova03:01
*** zhurong has quit IRC03:02
*** gongysh has joined #openstack-nova03:04
*** zhurong has joined #openstack-nova03:10
*** takashin has quit IRC03:12
*** takashin has joined #openstack-nova03:12
*** sree_ has joined #openstack-nova03:16
*** takashin has quit IRC03:16
*** sree_ is now known as Guest5093503:16
*** takashin has joined #openstack-nova03:16
*** takashin has quit IRC03:18
*** takashin has joined #openstack-nova03:19
*** sree has joined #openstack-nova03:20
*** takashin has quit IRC03:20
*** takashin has joined #openstack-nova03:20
*** Guest50935 has quit IRC03:22
*** takashin has quit IRC03:22
*** takashin has joined #openstack-nova03:23
*** Tom-Tom has quit IRC03:28
*** Tom-Tom has joined #openstack-nova03:29
*** abhishekk has joined #openstack-nova03:29
*** Tom-Tom has quit IRC03:29
*** Tom-Tom has joined #openstack-nova03:29
*** sridharg has joined #openstack-nova03:31
*** Tom-Tom has quit IRC03:34
*** takashin has quit IRC03:34
*** takashin has joined #openstack-nova03:34
*** annp has joined #openstack-nova03:37
*** gongysh has quit IRC03:45
*** hshiina has quit IRC03:48
*** Tom-Tom has joined #openstack-nova03:53
*** hshiina has joined #openstack-nova03:53
*** Tom-Tom has quit IRC03:53
*** links has joined #openstack-nova03:58
*** cfriesen_ has joined #openstack-nova03:59
*** takashin has quit IRC03:59
*** cfriesen has quit IRC03:59
*** takashin has joined #openstack-nova04:00
*** links has quit IRC04:01
*** links has joined #openstack-nova04:01
*** sdague has quit IRC04:05
*** coreywright has quit IRC04:05
*** takashin has quit IRC04:09
*** takashin has joined #openstack-nova04:09
openstackgerritZhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware  https://review.openstack.org/50748604:11
*** yamamoto has joined #openstack-nova04:12
*** Apoorva has joined #openstack-nova04:12
*** takashin has quit IRC04:16
*** takashin has joined #openstack-nova04:16
*** takashin has quit IRC04:18
*** takashin has joined #openstack-nova04:19
*** Tom-Tom has joined #openstack-nova04:20
*** takashin has quit IRC04:21
*** zhurong has quit IRC04:21
*** takashin has joined #openstack-nova04:21
*** coreywright has joined #openstack-nova04:23
*** takashin has quit IRC04:23
*** dave-mccowan has quit IRC04:23
*** takashin has joined #openstack-nova04:23
*** gongysh has joined #openstack-nova04:24
*** Tom-Tom has quit IRC04:24
*** sree_ has joined #openstack-nova04:26
*** sree_ is now known as Guest1400204:27
*** moshele has quit IRC04:28
*** sree has quit IRC04:28
*** diga has joined #openstack-nova04:29
*** gbarros_ has joined #openstack-nova04:29
*** gbarros has quit IRC04:30
*** sree has joined #openstack-nova04:31
*** Guest14002 has quit IRC04:32
*** takashin has quit IRC04:32
*** david-lyle has joined #openstack-nova04:32
*** takashin has joined #openstack-nova04:32
*** kalyan has joined #openstack-nova04:33
*** janki has joined #openstack-nova04:33
*** takashin has quit IRC04:34
*** takashin has joined #openstack-nova04:35
*** gouthamr has quit IRC04:36
*** hamzy has joined #openstack-nova04:40
*** trungnv has quit IRC04:41
*** namnh has quit IRC04:41
*** TuanLA has quit IRC04:41
*** hoangcx has quit IRC04:41
*** annp has quit IRC04:41
*** takashin has quit IRC04:41
*** phuongnh has quit IRC04:41
*** trungnv has joined #openstack-nova04:42
*** annp has joined #openstack-nova04:42
*** takashin has joined #openstack-nova04:42
*** hoangcx has joined #openstack-nova04:42
*** TuanLA has joined #openstack-nova04:42
*** phuongnh has joined #openstack-nova04:42
*** namnh has joined #openstack-nova04:42
*** david-lyle has quit IRC04:42
*** sree has quit IRC04:45
*** udesale has joined #openstack-nova04:46
*** sapd_ has quit IRC04:48
*** sree has joined #openstack-nova04:49
*** sree_ has joined #openstack-nova04:50
*** sree_ is now known as Guest7657304:50
*** sapd has joined #openstack-nova04:50
*** sree has quit IRC04:53
*** janki has quit IRC04:54
*** Apoorva_ has joined #openstack-nova04:54
*** gbarros_ has quit IRC04:55
*** Apoorva has quit IRC04:57
*** ratailor has joined #openstack-nova04:58
*** Dinesh_Bhor has quit IRC05:01
*** janki has joined #openstack-nova05:01
*** ratailor has quit IRC05:05
*** zhurong has joined #openstack-nova05:08
*** ratailor has joined #openstack-nova05:08
*** threestrands has joined #openstack-nova05:09
*** threestrands has quit IRC05:09
*** diga has quit IRC05:10
*** Alex_Staf has joined #openstack-nova05:17
*** chyka has joined #openstack-nova05:19
*** chyka has quit IRC05:24
*** liverpooler has quit IRC05:27
*** gongysh has quit IRC05:30
*** esberglu has joined #openstack-nova05:33
*** hongbin_ has quit IRC05:34
openstackgerritjichenjc proposed openstack/nova master: remove glance usage inside compute  https://review.openstack.org/51139205:35
openstackgerritjichenjc proposed openstack/nova master: Remove direct usage of glance.generate_image_url  https://review.openstack.org/51139705:35
*** esberglu has quit IRC05:38
openstackgerritjichenjc proposed openstack/nova master: Remove 'nova-manage host' and 'nova-manage agent'  https://review.openstack.org/52183705:41
openstackgerritjichenjc proposed openstack/nova master: Remove 'nova-manage shell' command  https://review.openstack.org/52183505:41
openstackgerritjichenjc proposed openstack/nova master: Remove 'nova-manage account' and 'nova-manage project'  https://review.openstack.org/52183305:41
openstackgerritjichenjc proposed openstack/nova master: Remove 'nova-manage logs' command  https://review.openstack.org/52213305:41
*** mdnadeem has joined #openstack-nova05:43
*** janki has quit IRC05:44
openstackgerritHuan Xie proposed openstack/nova master: XenAPI: Don't use nicira-iface-id for XenServer VIF  https://review.openstack.org/45164105:46
*** karthiks has quit IRC05:48
*** yamamoto_ has joined #openstack-nova05:50
*** fragatina has joined #openstack-nova05:53
openstackgerritwanghongxu proposed openstack/nova master: Update cinder info when attach volume that attached  https://review.openstack.org/52554005:53
*** yamamoto has quit IRC05:53
*** sree has joined #openstack-nova05:54
*** janki has joined #openstack-nova05:56
*** Guest76573 has quit IRC05:57
*** rocky-shiyan has joined #openstack-nova05:59
*** Apoorva_ has quit IRC05:59
*** Dinesh_Bhor has joined #openstack-nova06:00
*** yamamoto has joined #openstack-nova06:04
fricklercould someone take a look at https://bugs.launchpad.net/bugs/1733933 and possibly give it some increased importance? seems it is masking any error happening on the compute side towards the user06:05
openstackLaunchpad bug 1733933 in OpenStack Compute (nova) "nova-conductor is masking error when rescheduling" [Undecided,Confirmed]06:05
*** Tom-Tom has joined #openstack-nova06:05
*** AlexeyAbashkin has joined #openstack-nova06:07
*** yamamoto_ has quit IRC06:07
*** udesale__ has joined #openstack-nova06:10
*** AlexeyAbashkin has quit IRC06:11
*** AlexeyAbashkin has joined #openstack-nova06:12
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/52479506:12
*** udesale has quit IRC06:12
*** karthiks has joined #openstack-nova06:14
openstackgerritRajesh Tailor proposed openstack/nova master: Fix case-sensitivity for metadata keys  https://review.openstack.org/50488506:19
*** Rambo has joined #openstack-nova06:19
Rambohello!everyone,I am new to this,please correct me!06:21
*** nmathew has joined #openstack-nova06:21
*** AlexeyAbashkin has quit IRC06:26
*** xinliang has quit IRC06:27
*** sree_ has joined #openstack-nova06:31
*** moshele has joined #openstack-nova06:32
*** sree_ is now known as Guest7805806:32
*** bkopilov has quit IRC06:35
*** sree has quit IRC06:35
*** janki has quit IRC06:36
*** janki has joined #openstack-nova06:38
*** xinliang has joined #openstack-nova06:40
*** diga has joined #openstack-nova06:51
openstackgerritjichenjc proposed openstack/nova master: Refactor placement version check  https://review.openstack.org/51249706:55
*** udesale has joined #openstack-nova06:58
openstackgerritRajesh Tailor proposed openstack/nova master: Host addition host-aggregate should be case-sensitive  https://review.openstack.org/49833407:00
*** udesale__ has quit IRC07:00
*** Eran_Kuris has joined #openstack-nova07:00
*** rcernin has quit IRC07:02
*** lajoskatona has joined #openstack-nova07:04
*** andreas__ has joined #openstack-nova07:12
*** diga has quit IRC07:14
*** sshwarts has joined #openstack-nova07:20
*** esberglu has joined #openstack-nova07:23
*** sahid has joined #openstack-nova07:24
*** karthiks has quit IRC07:26
*** esberglu has quit IRC07:28
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for snapshot instances  https://review.openstack.org/52592107:28
*** hshiina has quit IRC07:29
*** gongysh has joined #openstack-nova07:31
openstackgerritWangpan proposed openstack/nova master: Handle glance exception during rotating instance backup  https://review.openstack.org/52339507:31
openstackgerritWangpan proposed openstack/nova master: Handle glance exception during rotating instance backup  https://review.openstack.org/52339507:34
*** karthiks has joined #openstack-nova07:39
*** rcernin has joined #openstack-nova07:39
*** yamamoto_ has joined #openstack-nova07:42
*** yamamoto has quit IRC07:46
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Fix a description for 'guest_format'  https://review.openstack.org/52592807:47
*** slaweq_ has joined #openstack-nova07:47
*** kalyan has quit IRC07:49
*** ildikov has quit IRC08:03
*** samueldmq has quit IRC08:04
*** ildikov has joined #openstack-nova08:04
*** samueldmq has joined #openstack-nova08:05
*** knikolla has quit IRC08:06
*** knikolla has joined #openstack-nova08:07
*** yamahata has quit IRC08:07
*** mnaser has quit IRC08:08
*** csatari has quit IRC08:09
*** gmann has quit IRC08:09
*** gongysh has quit IRC08:10
*** mnaser has joined #openstack-nova08:11
*** Yingxin has quit IRC08:12
*** csatari has joined #openstack-nova08:12
*** alexchadin has joined #openstack-nova08:14
*** Yingxin has joined #openstack-nova08:16
*** tesseract has joined #openstack-nova08:19
*** Rambo__ has joined #openstack-nova08:19
Rambo__Now the boot from volume 's instance of rebuild operation has a problem.For example,after the rebuild operation,the instance 's root disk is not replace.08:20
Rambo__I found the reason is that when we use the "_build_resources" function to prepare source,it obtains the block devices according to the previous instance 's uuid and attaches them to instance.So boot from volume 's instance of rebuild operation doesn't update data.08:20
Rambo__       To solve it,I plan to use CLI 's "metadata" option,to increase a key name "source_type".The "source_type" includes "snapshot" and "image".We can judge from "source_type".If the "source_type" is "snapshot",we can transform the given snapshot to a volume and attach this volume to instance.If the "source_type" is "image",we don't handle it.08:20
Rambo__       Can you give me some advice?Help in troubleshooting this issue will be appreciated.08:20
*** takashin has left #openstack-nova08:20
*** ralonsoh has joined #openstack-nova08:24
openstackgerritwanghongxu proposed openstack/nova master: Update cinder info when attach volume that attached  https://review.openstack.org/52554008:25
*** ragiman has joined #openstack-nova08:30
*** AlexeyAbashkin has joined #openstack-nova08:32
*** zhurong has quit IRC08:33
*** Rambo has quit IRC08:34
*** diga has joined #openstack-nova08:34
*** diga has quit IRC08:38
*** AlexeyAbashkin has quit IRC08:38
*** imacdonn has quit IRC08:39
*** imacdonn has joined #openstack-nova08:40
*** udesale has quit IRC08:42
openstackgerritzhangyanxian proposed openstack/nova-specs master: Sriov-bond  https://review.openstack.org/46352608:43
*** damien_r has joined #openstack-nova08:44
*** AlexeyAbashkin has joined #openstack-nova08:44
openstackgerritzhangyanxian proposed openstack/nova-specs master: Sriov-bond  https://review.openstack.org/46352608:49
*** AlexeyAbashkin has quit IRC08:49
openstackgerritZhenyu Zheng proposed openstack/nova master: [WIP] POC to use neutron port_list when filtering instance by ip  https://review.openstack.org/52550508:51
*** zhurong has joined #openstack-nova08:53
*** slaweq has joined #openstack-nova08:55
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API  https://review.openstack.org/47661208:55
*** qsyqian has joined #openstack-nova08:55
*** yamamoto_ has quit IRC08:56
*** slaweq has quit IRC09:00
openstackgerritWangpan proposed openstack/nova master: Handle glance exception during rotating instance backup  https://review.openstack.org/52339509:00
*** hieulq_ has joined #openstack-nova09:01
*** yamamoto has joined #openstack-nova09:04
*** hieulq_ has quit IRC09:04
*** hieulq has quit IRC09:04
openstackgerritZhenyu Zheng proposed openstack/nova master: [WIP] POC to use neutron port_list when filtering instance by ip  https://review.openstack.org/52550509:05
openstackgerritGyorgy Szombathelyi proposed openstack/nova stable/ocata: Use proper user and tenant in the owner section of libvirt.xml.  https://review.openstack.org/52599709:05
*** hieulq has joined #openstack-nova09:06
*** hieulq_ has joined #openstack-nova09:07
*** hieulq has quit IRC09:08
*** hieulq_ has quit IRC09:08
*** Dinesh_Bhor has quit IRC09:09
*** hieulq has joined #openstack-nova09:10
*** esberglu has joined #openstack-nova09:11
*** esberglu has quit IRC09:12
*** esberglu has joined #openstack-nova09:12
*** esberglu has quit IRC09:12
*** esberglu has joined #openstack-nova09:13
*** vishwanathj has quit IRC09:13
*** vishwanathj has joined #openstack-nova09:14
*** AlexeyAbashkin has joined #openstack-nova09:14
*** esberglu has quit IRC09:17
openstackgerritzhurong proposed openstack/nova master: Remove the unused request_id filter from api-paste.ini  https://review.openstack.org/52600109:17
openstackgerritBalazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild  https://review.openstack.org/52524209:30
*** lucas-afk is now known as lucasagomes09:31
openstackgerritBalazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild  https://review.openstack.org/52524209:32
*** derekh has joined #openstack-nova09:39
openstackgerritStephen Finucane proposed openstack/nova master: test: Store the OutputStreamCapture fixture  https://review.openstack.org/51514609:51
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Rename 'policy_check' -> 'policy'  https://review.openstack.org/51514809:51
openstackgerritStephen Finucane proposed openstack/nova master: nova-status: Migrate to cliff  https://review.openstack.org/51514709:51
openstackgerritStephen Finucane proposed openstack/nova master: nova-policy: Migrate to cliff  https://review.openstack.org/51514909:51
*** andreas_s has joined #openstack-nova09:53
*** jichen has quit IRC09:54
*** andreas__ has quit IRC09:56
*** linkmark has joined #openstack-nova09:57
*** rmart04 has joined #openstack-nova09:57
*** sdague has joined #openstack-nova09:59
*** yamamoto has quit IRC10:05
*** gmann has joined #openstack-nova10:05
*** yamamoto has joined #openstack-nova10:08
openstackgerritMerged openstack/python-novaclient master: Avoid tox_install.sh for constraints support  https://review.openstack.org/52483710:12
*** yamamoto has quit IRC10:13
*** annp has quit IRC10:14
*** andreas_s has quit IRC10:14
*** andreas_s has joined #openstack-nova10:15
*** andreas_s has quit IRC10:15
*** andreas_s has joined #openstack-nova10:15
*** Tom-Tom has quit IRC10:24
*** Tom-Tom has joined #openstack-nova10:25
*** namnh has quit IRC10:25
*** TuanLA has quit IRC10:27
openstackgerritMerged openstack/python-novaclient master: Microversion 2.56 - Enable cold migration with target host  https://review.openstack.org/40670710:30
*** sambetts|afk is now known as sambetts10:32
*** Tom-Tom has quit IRC10:33
*** zhurong has quit IRC10:37
*** gszasz has joined #openstack-nova10:38
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Migrate from 'ip' commands to 'pyroute2'  https://review.openstack.org/48438610:43
*** chyka has joined #openstack-nova10:43
*** alexchadin has quit IRC10:48
*** chyka has quit IRC10:48
*** Tom-Tom has joined #openstack-nova10:49
openstackgerritMerged openstack/nova master: Fail fast if changing image on a volume-backed server rebuild  https://review.openstack.org/52066010:49
openstackgerritMerged openstack/nova master: Updated from global requirements  https://review.openstack.org/52538610:49
*** andreas_s has quit IRC10:52
*** mvk has quit IRC10:52
*** andreas_s has joined #openstack-nova10:53
*** Tom-Tom has quit IRC10:53
*** udesale has joined #openstack-nova10:55
*** abhishekk has quit IRC11:00
*** coreywright has quit IRC11:00
*** esberglu has joined #openstack-nova11:00
*** yamamoto has joined #openstack-nova11:01
*** andreas_s has quit IRC11:01
*** andreas_s has joined #openstack-nova11:03
*** esberglu has quit IRC11:04
*** andreas_s has quit IRC11:11
*** coreywright has joined #openstack-nova11:17
*** Tom-Tom has joined #openstack-nova11:17
*** andreas_s has joined #openstack-nova11:18
*** yamamoto has quit IRC11:19
*** cdent has joined #openstack-nova11:19
*** udesale has quit IRC11:20
*** Tom-Tom has quit IRC11:21
*** lpetrut has joined #openstack-nova11:22
*** qsyqian has quit IRC11:27
*** andreas_s has quit IRC11:32
*** andreas_s has joined #openstack-nova11:32
*** links has quit IRC11:34
*** andreas_s has quit IRC11:35
*** andreas_s has joined #openstack-nova11:36
*** yangyapeng has quit IRC11:36
*** yangyapeng has joined #openstack-nova11:36
*** rcernin has quit IRC11:39
*** qsyqian has joined #openstack-nova11:39
*** Rambo__ has quit IRC11:42
*** BryanS68 has joined #openstack-nova11:44
*** BryanS68 has quit IRC11:46
*** Rambo__ has joined #openstack-nova11:46
*** diga has joined #openstack-nova11:47
*** ratailor has quit IRC11:47
*** links has joined #openstack-nova11:47
*** andreas_s has quit IRC11:48
*** Rambo__ has quit IRC11:48
*** andreas_s has joined #openstack-nova11:48
*** alexchadin has joined #openstack-nova11:49
*** BryanS68 has joined #openstack-nova11:49
*** qsyqian has quit IRC11:51
*** janki has quit IRC11:52
*** smatzek has joined #openstack-nova11:53
*** dtantsur|afk is now known as dtantsur11:53
*** nmathew has quit IRC11:54
*** ankit_ has quit IRC11:54
*** Rambo has joined #openstack-nova11:55
*** Rambo has quit IRC11:57
*** AlexeyAbashkin has quit IRC11:58
*** Rambo has joined #openstack-nova12:02
*** qsyqian has joined #openstack-nova12:02
*** openstackgerrit has quit IRC12:03
*** phuongnh has quit IRC12:05
*** yamamoto has joined #openstack-nova12:05
*** links has quit IRC12:08
*** qsyqian has quit IRC12:09
*** dave-mccowan has joined #openstack-nova12:13
*** links has joined #openstack-nova12:13
*** alexchadin has quit IRC12:16
*** alexchadin has joined #openstack-nova12:16
*** yangyapeng has quit IRC12:18
*** yangyapeng has joined #openstack-nova12:18
*** yangyapeng has quit IRC12:25
*** openstackgerrit has joined #openstack-nova12:31
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Migrate from 'ip' commands to 'pyroute2'  https://review.openstack.org/48438612:31
*** pchavva has joined #openstack-nova12:32
*** rambo__ has joined #openstack-nova12:33
*** mvk has joined #openstack-nova12:41
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Microversion 2.57 - List/Show all server migration types  https://review.openstack.org/43083912:42
openstackgerritMerged openstack/nova master: Genericify the instance_list stuff  https://review.openstack.org/52425312:43
openstackgerritMerged openstack/nova master: Split instance_list into instance and multi_cell  https://review.openstack.org/52430212:43
*** lucasagomes is now known as lucas-hungry12:44
*** slaweq_ has quit IRC12:45
*** efried_cya_wed is now known as efried12:46
*** janki has joined #openstack-nova12:46
*** AlexeyAbashkin has joined #openstack-nova12:46
*** yangyapeng has joined #openstack-nova12:50
*** tesseract has quit IRC12:53
*** nmathew has joined #openstack-nova12:55
*** tesseract has joined #openstack-nova12:57
*** rambo__ has quit IRC13:01
efriedGreetings cdent.  How goes the grenade sleuthing?13:04
cdentsome progress at: https://review.openstack.org/#/c/525605/13:04
cdentgist is that grenade and devstack’s handing of placemen a bit out of wack: no stop from grenade, important config being erased by devstack13:05
*** gmann is now known as gmann_afk13:05
efriedgrisly.13:05
sean-k-mooney2bauzas: johnthetubaguy  ye are both on the nova satable matainance team correct? can ye take a look at this trival change in os-vif https://review.openstack.org/#/c/488670/ we dont have a seperate release team so we use nova's13:05
cdentefried: I think that because placement went wsgi before everything else, some later learnings didn’t get incorporated into its devstack handkling13:06
cdentand because placement is managed in grenade as part of nova, despite being its own thing in devstack, caused some packet drops13:06
*** takedakn has joined #openstack-nova13:09
*** Rambo has quit IRC13:10
*** diga has quit IRC13:10
*** maciejjo1 has quit IRC13:13
*** Tom-Tom has joined #openstack-nova13:16
bauzassean-k-mooney2: lemme look13:17
*** dtantsur is now known as dtantsur|lunch13:18
mdboothFolks, has anybody else encountered MySQL-python FTB on Fedora 27?13:19
sean-k-mooney2bauzas: its just an update to the .gitreview but its been open for a few months now so would be good to merge13:19
mdboothThis django bug shows the issue I hit: https://github.com/OWASP/django-DefectDojo/issues/40713:20
bauzassean-k-mooney2: +213:20
sean-k-mooney2bauzas: thanks13:20
*** links has quit IRC13:23
*** markvoelker has quit IRC13:24
*** markvoelker has joined #openstack-nova13:25
jaypipesefried: welcome back13:28
jaypipescdent: any prog on that bug?13:29
openstackgerritSurya Seetharaman proposed openstack/nova master: archive_deleted_instances is not atomic for insert/delete  https://review.openstack.org/52606513:30
cdentjaypipes: yeah, it has fixes that need be merged, the tests underneath this are passing: https://review.openstack.org/#/c/525605/ there’s a dependent devstack change and a backport of that devstack change13:30
cdentsdague, dansmith ^13:30
cdentjaypipes: it was a bit of an onion13:30
jaypipescdent: sounds like it indeed :(13:31
*** READ10 has joined #openstack-nova13:32
sdaguecdent: I approved the 2 devstack changes, I guess we are waiting on grenade results to come in?13:34
efriedjaypipes Howdy, thanks.  I'm poring over emails to see where you got to with those patches.13:34
cdentsdague: Yeah, I’ve been watrching zuul and the two main grenade changes have passed13:34
efriedjaypipes I didn't intend for you to have to do that stuff - I realized it had gotten pretty ugly at this point with all the stuff that changed underneath it.13:34
cdentheat (non-voting) has vailed and ironic is being slow13:34
jaypipesefried: well, I had to give up after f**king up the series in a failed rebase.13:35
cdentvailed is a word I will use to mean something like “veiled in failure”13:35
jaypipesefried: heh13:36
efriedcdent Or a verb related to an overpriced ski resort.13:36
jaypipesefried: https://review.openstack.org/#/c/385693/ is where I stopped after a revert of the rebase attempt.13:36
cdentthat fails to satisfy13:36
jaypipesI'm also at my wit's end w.r.t the REST API patch for nested resource providers.13:37
jaypipesso much for trying to merge things quickly and tweak afterwards.13:37
* efried notices the singular possessive - jaypipes only has one wit13:37
sdaguecdent: is the heat one normally passing?13:37
jaypipeslet's bikeshed on shit some more, just for fun.13:37
efriedjaypipes What's the rub there?  Are we just bikeshedding tree vs in_tree?13:37
jaypipesefried: that, yes, and placement-api-ref weirdness.13:38
jaypipesefried: I'm tired of the placement api ref docs crap, frankly.13:39
cdentsdague: it’s hard to tell, there haven’t been many changes to grenade since zuul13:39
jaypipesefried: in particular, these comments: https://review.openstack.org/#/c/384807/85/placement-api-ref/source/resource_provider.inc13:39
*** takedakn has quit IRC13:39
jaypipesefried: the parameters are not required. they are optional.13:39
cdentsdague: but this recent change passes: https://review.openstack.org/#/c/525720/13:39
jaypipesefried: but takashi wants to indicate that the *response* attributes will always be there, so insists I put required: true in the parameters file.13:39
efriedSo I agree that in_tree would be clearer; else we're sorta giving the impression that a tree is an entity in its own right with a UUID.  But super-meh.  This is an API, and people are going to have to read the docs to figure out what it means regardless of whether it's called `tree` or `in_tree`.13:40
*** dtantsur|lunch is now known as dtantsur13:40
efriedWRT the docs, my *strong* preference at this point would be to merge what we have so we can get *code* going, and then follow up with docs tweaks.13:40
jaypipesefried: that was my preference as well, but I've now attempted to comply with the review requests on the docs stuff and broken the api-ref build.13:41
*** mriedem has joined #openstack-nova13:41
*** takedakn has joined #openstack-nova13:41
efriedjaypipes Let me catch up on the review comments.  I can take over fixing up the api-ref build if you're getting burned out.13:42
*** nmathew- has joined #openstack-nova13:42
jaypipesefried: I'm way burned out on this, yes.13:42
jaypipesefried: and other than not wanting to go with "tree_member", I don't care about tree vs. in_tree.13:42
efriedjaypipes What's it at right now in the patch?13:43
jaypipestree13:43
*** esberglu has joined #openstack-nova13:44
jaypipesefried: I'll fix it. gimme 10 mins13:44
efriedjaypipes cdent edleafe bauzas can we take a quick vote?  Sounds like jaypipes is +1 on tree/in_tree and -1 on tree_member; I'm the same.13:44
jaypipesefried: but you can do the rebase for your WIP patches above it.13:44
efriedjaypipes ight.13:45
*** nmathew has quit IRC13:45
*** alexchadin has quit IRC13:45
jaypipesefried: I'm +1 on just get this friggin done.13:45
efried++13:45
*** psachin has quit IRC13:45
efriedYou have to respin the patch anyway; but renaming that query param is slightly nontrivial compared to fixing the doc build.13:45
cdent-1 on tree_member, slight preference for in_tree over tree, but not enough to be upset if tree is the result13:45
*** gbarros has joined #openstack-nova13:46
cdentbrb13:47
*** esberglu has quit IRC13:48
*** lucas-hungry is now known as lucasagomes13:49
*** maciejjozefczyk has joined #openstack-nova13:49
bauzasjaypipes: I agree with you on the fact it's just a verbiage problem, so a nit, but honestly, given it would be an API parameter, I prefer to make sure we discuss that13:50
jaypipesbauzas: this patch is >1 year old. let's make a decision and do this.13:52
bauzasexactly13:52
bauzasso13:52
bauzastree, -1 for me13:52
bauzasin_tree : +013:52
maciejjozefczykHello Guys, im trying to debug why I've faced a problem with scheduling. My instances stucks in 'scheduling' state. After enabling debug on nova-scheduler I've found that a host was selected. On nova-compute side there were 0 info about building an instance. It could be possible that the failure point is nova-conductor?13:53
bauzastree_member : +113:53
bauzasor tree_node13:53
jaypipesalex_xu: around? your vote on tree, in_tree or tree_member please.13:53
efriedin_tree has it by a whisker.13:57
efriedYou know, assuming it's a democracy in the first place13:57
efriedand the need to delta what's there isn't being factored in.13:57
sean-k-mooney2:) in_tree would be my choice out of those too13:58
*** lyan has joined #openstack-nova13:59
sean-k-mooney2tree_member almost seams like a boolean to me. e.g. is that resouce a tree member or standalone14:00
*** nmathew- has quit IRC14:01
*** liverpooler has joined #openstack-nova14:01
sean-k-mooney2but as cdent said between tree and in_tree i dont have a stong enough preference to mind which was used14:02
*** hamzy has quit IRC14:02
efriedjaypipes How far up the stack are you rebasing currently?14:07
*** kuzko has quit IRC14:07
jaypipesefried: only to the update client to set parent provider patch (third from the bottom)14:08
jaypipesefried: since when I tried to rebase all the way up to your WIP patches, I failed miserably14:08
*** sree has joined #openstack-nova14:08
efriedjaypipes k.  I just started looking, but in particular I think this delta is wrong, please confirm: https://review.openstack.org/#/c/520643/8..9/nova/scheduler/client/report.py14:09
efriedThat reverts the exception handling bugfix.14:09
*** kuzko has joined #openstack-nova14:09
efriedIt's like an un-rebase14:09
jaypipesefried: yeah, I messed it up. totally. :(14:09
*** mdnadeem has quit IRC14:09
efriedbut I need to wait until you've published the bottom three to start rebasing on top.14:10
jaypipesefried: I ended up having to "revert" the rebase for the three patches at the bottom using clarkb's trick of git review -d XXXXX,N-1 && git commit --amend14:10
jaypipesefried: yes14:10
jaypipesefried: I'm just running tests now.14:10
efriedk, I shall stop distracting you :)14:10
jaypipesefried: gimme another ten minutes to complete.14:10
efriedrgr14:10
*** udesale has joined #openstack-nova14:12
*** Guest78058 has quit IRC14:12
*** slaweq_ has joined #openstack-nova14:13
*** david-lyle has joined #openstack-nova14:13
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Check if interface belongs to a Linux Bridge before removing  https://review.openstack.org/52607914:14
kashyapmdbooth: Have you resolved your MySQL-Python on F27 thing?  I'm on F2714:15
* kashyap is surprised that MySQL is still being used14:15
mdboothkashyap: I think so... by not building it :)14:15
kashyapAh, then something somewhere still has stale deps file14:16
*** armax has quit IRC14:17
*** dave-mccowan has quit IRC14:17
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API  https://review.openstack.org/47661214:18
*** takedakn has quit IRC14:18
efriedcdent Would it be feasible/appropriate/relevant for jaypipes to put Depends-On to  https://review.openstack.org/#/c/525605/ (grenade fix) from https://review.openstack.org/#/c/385693/ in his current spin?14:20
cdentefried: Maybe, I’m never quite sure well depends-on _to_ a grenade change works. It might be just fine. mriedem ?14:21
efriedI know certain projects don't pick up Depends-Ons to certain other projects.14:21
efriedI'm also pretty sure some of those rules changed (for the better IIRC) when we went to zuulv314:21
efriedSo - no idea at this point.14:22
efriedGuess it couldn't hurt.  If it winds up being a no-op, we haven't lost anything; but if it winds up going green, we know it works.14:22
mriedemit's valid for nova to depend on a grenade fix yes14:23
mriedemcdent: questions in https://review.openstack.org/#/c/521639/14:24
*** sree_ has joined #openstack-nova14:24
cdentmriedem: roger, working on the contributor doc stuff now14:25
*** sree_ is now known as Guest3163314:25
mriedemmelwitt: lyarwood: i went over https://review.openstack.org/#/c/400384/ - only thing of note is that power_on (start API) is now going to do this full hard reboot, and i'm not sure if that's an intended side effect or not14:25
mriedemdefinitely feels like one of those changes that will break something else we aren't aware of, so i'll definitely want to avoid backporting that too far too fast until it has some time to sink in14:26
sdaguecdent: can you let me know if you feel the heat fail on https://review.openstack.org/#/c/525605/ is valid or not14:26
sdagueif not, I'm happy to get it merged14:26
*** sree has quit IRC14:26
*** smatzek has quit IRC14:26
*** esberglu has joined #openstack-nova14:27
cdentsdague: I was unable to trace the failure to anything related to placement, it looked like a timeout, but it’s hard to tell: the (single) test that failed doesn’t make what it’s doing all that obvious14:27
cdentso I’d say go for it14:27
openstackgerritJay Pipes proposed openstack/nova master: placement: allow filter providers in tree  https://review.openstack.org/37721514:29
openstackgerritJay Pipes proposed openstack/nova master: placement: adds REST API for nested providers  https://review.openstack.org/38480714:29
openstackgerritJay Pipes proposed openstack/nova master: placement: update client to set parent provider  https://review.openstack.org/38569314:29
jaypipesefried: et voila. ^14:29
efriedjaypipes Merci bien.14:29
*** Guest31633 has quit IRC14:29
jaypipesefried: changed to in_tree and also added that additional func test you wanted for the in_tree=CHILD_RP_UUID and returned grandchild RP with resources query14:29
efriedjaypipes beaut.14:30
jaypipesefried: and fixed (hopefully?) the placement-api-ref shit.14:30
mriedemsdague: failopotamus might be busted on something with the zuulv3 move14:30
mriedemcan't get any data on the grenade-heat job14:30
*** andreas_s has quit IRC14:31
*** david-lyle has quit IRC14:31
*** andreas_s has joined #openstack-nova14:31
openstackgerritSurya Seetharaman proposed openstack/nova master: archive_deleted_instances is not atomic for insert/delete  https://review.openstack.org/52606514:32
*** _ix has joined #openstack-nova14:32
mdboothmriedem: That's not a bug, that's a feature! https://review.openstack.org/#/c/400384/2014:35
*** salv-orlando has joined #openstack-nova14:35
*** andreas_s has quit IRC14:36
_ixGood morning folks. I was curious if it were possible to take an existing hypervisor from one Openstack implementation and join it to a different one.14:36
mriedemdefine "join"14:36
_ixBasically, repoint the hypervisor to start using a different controller and attendant services.14:37
_ixBut have that controller aware of the VMs running on it.14:37
*** tesseract has quit IRC14:37
mriedemno14:38
mriedemyou can probably make a different controller aware of and manage the compute node,14:38
mriedembut not the existing VMs14:38
mriedemthat would be like creating VMs in virsh directly and expecting nova to start managing them14:38
mriedemwhich won't happen14:38
_ixA man can dream, can't he?14:38
mdbooth_ix: Only approved dreams.14:39
efried_ix There are orchestrators (that live on top of nova) that can do "discovery" like that.14:39
mdboothefried: orly?14:39
* mdbooth should probably know about this already14:39
_ixYes, do tell, efried.14:39
efriedmdbooth _ix I'm not necessarily talking about OpenStack projects.  I just happen to know PowerVC does that.14:40
mdboothefried: How would you create db entries without creating the guest?14:40
mdboothAh, ok14:40
efriedmdbooth Yeah, we don't create 'em in nova.  Just let you manage 'em whether they were created in nova or not.14:41
_ixAre there any other concepts of having an existing kvm vm and giving OS control over it?14:41
_ixMaybe by creating an image?14:41
mdbooth_ix: In the general case, Nova is very opinionated not only about having various metadata (mostly in the db) about an instance, but also exactly how that instance exists on the hypervisor.14:42
*** hamzy has joined #openstack-nova14:42
efried_ix It sounds like something that's possible.  I don't know of an example of any turnkey tools to do it, but that doesn't mean they don't exist.14:42
*** andreas_s has joined #openstack-nova14:42
mdboothImporting a guest which was created by something else is an open-ended problem with countless edge cases14:42
_ixI can appreciate that, and I don't mind doing the work.14:43
efriedYou could certainly create a disk image from a non-Nova VM and stuff it in Glance and use it to create Nova VMs.  That's how you bootstrap Nova VMs in the first place, nah?14:43
mdboothThe guest would have to already be something which nova can describe exactly14:43
mdboothAnd then you'd have to create its metadata14:44
*** tesseract has joined #openstack-nova14:44
efriedyeah, you'd have to specify the metadata...14:44
*** slaweq_ has quit IRC14:44
efriedjaypipes FYI, built the placement-api-ref locally for that patch and it passes.14:44
mdboothIt would be a pretty hard problem, anyway.14:44
*** esberglu has quit IRC14:44
jaypipesefried: cool.14:44
mdbooth_ix: Incredibly useful, though.14:44
efriedjaypipes Oh, maybe I built it for the wrong patch.  Pretend I didn't say that, but stay tuned.14:45
_ixMaybe this isn't as edgey of an edge case as I had thought. The requirement was to create a new openstack deployment while leaving the previous one largely intact.14:45
openstackgerritChris Dent proposed openstack/nova master: [placement] Add info about last-modified to contrib docs  https://review.openstack.org/52608414:45
*** gouthamr has joined #openstack-nova14:46
_ixNow that the new deployment is starting to outpace the old deployment in maturity, I'd like to cut over to the new one. But cutting over isn't that simple with existing VMs.14:46
sdaguemriedem: it might be, I haven't tried to use it lately14:46
mriedem_ix: does "largely intact" include keep the ports and volumes connected to the old VMs?14:47
mriedemif the new thing should look like the old thing, then you have a problem with ports and volumes too14:47
mriedemi.e. the IP will change14:47
_ixFortunately, there are no volumes to be concerned with, and the ports aren't as important with dns.14:48
*** andreas_s has quit IRC14:48
mriedemif that's not a problem, just snapshot the vms from the old deployment and create new vms from those snapshot images in the new deployment,14:48
_ixLargely intact would really just mean running and reachable.14:48
mriedemif you don't care about downtime14:48
efriedjaypipes If https://review.openstack.org/#/c/384807/ was the docs-broken one -- that one passes now :)14:48
bauzasdansmith: jaypipes: question about libvirt VGPU, given we recreate the XML when migrating or hard rebooting an instance, we don't have passed allocations, so the new XML wouldn't know about the existing VGPUs14:48
mriedemif you care about downtime, use live snapshot14:48
_ixDowntime isn't an issue.14:48
bauzasdansmith: jaypipes: so I wonder if we should persist that14:48
bauzasfor spawning it's not a problem14:49
bauzasgiven we pass the allocations14:49
dansmithbauzas: we can look them up easily right?14:49
bauzasdansmith: you mean about the change ?14:49
mriedem_ix: so your old cloud is running juno and you want to move to pike, but aren't doing upgrades, right?14:49
_ixThanks for the discussion mriedem mdbooth efried. I've got some work to do.14:49
efriedenjoy,14:49
mriedem_ix: in that case, easiest thing is just the snapshot move to the new cloud14:49
_ixIt's newton to pike.14:49
bauzasdansmith: I can upload it soon yeah14:49
*** armax has joined #openstack-nova14:49
bauzasdansmith: oh, you mean about the allocations ?14:50
_ixI think that's what I'm planning on now.14:50
mriedemhell you could share glance between deployments to make the image move easier14:50
_ixThat's a really good idea.14:50
dansmithbauzas: I'm not sure what you mean, but we can now easily look up the _new_ allocations intended for the live migration from placement14:51
*** andreas_s has joined #openstack-nova14:52
bauzasdansmith: mmm, yeah14:52
mriedemedleafe: are you going to handle https://review.openstack.org/#/c/516707/ soonish or would you like me to? i'd like to see a passing run through the stack on the functional tests and then skim the top patch and if nothing major, we can start merging early stuff in the series14:52
*** hamzy has quit IRC14:53
bauzasdansmith: anyway, just uploading my change and we'll be discussing14:53
*** andreas_s has quit IRC14:53
*** andreas_s has joined #openstack-nova14:54
*** mlavalle has joined #openstack-nova14:54
*** huanxie has joined #openstack-nova14:54
edleafemriedem: I'm currently mired in fixing all the broken unit tests for the resize changes14:54
edleafeBut I can get to that in a little while14:55
mriedemok, i'll start skimming the top patch in the series then14:55
*** hamzy has joined #openstack-nova14:55
ildikovmriedem: morning :)14:57
ildikovmriedem: checked the comments and answered to a couple14:57
*** tidwellr has joined #openstack-nova14:57
*** hamzy has quit IRC14:58
ildikovmriedem: do you plan to touch that chain or should I start updating the virt/block_device changes?14:58
*** esberglu has joined #openstack-nova14:59
huanxieHi jaypipes, I have a patch https://review.openstack.org/#/c/451641/ to remove using of nicira-iface-id for XenServer VIF, could you please review it when you are available? This is a follow up patch as my previous patch in Pike has deprecated the using of nicira-iface-id.14:59
*** hamzy has joined #openstack-nova15:00
mriedemildikov: i'll handle the block_device.py changes in a bit - i think we definitely want to rollback if attachment_complete fails, just like in the old flow15:00
mriedemi just needed some time away from the series to think about that15:00
ildikovmriedem: yeah, we did when the call was first added15:00
ildikovit got uncovered when I moved, forgot to add another try-except block... :/15:01
ildikovmy bad15:01
mriedemi'm less sure about when we call save() before doing the driver attach, but i think we want to do similar to what we did before - only save() the connection_info after we've attached on the driver15:01
*** awaugama has joined #openstack-nova15:01
mriedemotherwise we're not actually connected15:01
openstackgerritEric Berglund proposed openstack/nova master: WIP: PowerVM Driver: vSCSI  https://review.openstack.org/52609415:02
ildikovmriedem: the save() calls were put there a while ago and I removed the second one when johnthetubaguy pointed it out15:02
ildikovI'm a bit confused all the time when I get those parts that what gets saved to where and when...15:02
*** salv-orlando has quit IRC15:03
ildikovI think we ended up with two calls to be extra sure we have everything as at least on the Cinder side we got into trouble due to missing save() calls15:03
*** owalsh_ has joined #openstack-nova15:03
*** huanxie has quit IRC15:04
ildikovand my brain is definitely too small for keeping both code bases in it :)15:04
*** huanxie has joined #openstack-nova15:04
dansmithcdent: that patch to grenade makes sense to me, but I haven't gone digging to see if it actually makes sense15:05
dansmithcdent: but sdague will know, so let him +2 it first and then I'll hit it15:05
*** owalsh has quit IRC15:05
cdentmany are the layers to sense15:05
*** hamzy has quit IRC15:06
sdaguedansmith / cdent the change looks good to me conceptually, I just want to make sure that the heat fail seems unrelated. cdent: you able to take a look and give an assessment?15:07
dansmithI thought he said he did, and couldn't15:07
dansmithmaybe a recheck to see?15:07
*** tidwellr has quit IRC15:07
cdentsdague: I’ve responded to you twice about that already, last one was [t 3ahR]15:07
purplerbot<cdent> sdague: I was unable to trace the failure to anything related to placement, it looked like a timeout, but it’s hard to tell: the (single) test that failed doesn’t make what it’s doing all that obvious [2017-12-06 14:27:39.279804] [n 3ahR]15:07
openstackgerritEric Fried proposed openstack/nova master: placement: update client to set parent provider  https://review.openstack.org/38569315:08
openstackgerritBalazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild  https://review.openstack.org/52524215:09
openstackgerritBalazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407  https://review.openstack.org/52609515:09
openstackbug 1735407 in OpenStack Compute (nova) "[Nova] Evacuation doesn't respect anti-affinity rules" [Medium,In progress] https://launchpad.net/bugs/1735407 - Assigned to Balazs Gibizer (balazs-gibizer)15:09
efriedjaypipes cdent ^ Removed some test stuff that got accidentally re-introduced at PS86; and added that Depends-On.15:09
jaypipesefried: tyvm15:09
sdagueoh, sorry, apparently I missed that last message15:09
*** amodi has joined #openstack-nova15:09
*** salv-orlando has joined #openstack-nova15:10
*** tidwellr has joined #openstack-nova15:10
cdentsdague: no worries, that’s pretty much exactly the reason I made that [t stuff15:10
*** lajoskatona has quit IRC15:11
*** belmoreira has joined #openstack-nova15:12
*** emccormick has quit IRC15:12
*** salv-orlando has quit IRC15:13
*** hamzy has joined #openstack-nova15:13
*** hamzy has quit IRC15:14
*** huanxie has quit IRC15:16
*** jmlowe has joined #openstack-nova15:16
openstackgerritChris Dent proposed openstack/nova master: [placement] Object changes to support last-modified headers  https://review.openstack.org/52163915:16
openstackgerritChris Dent proposed openstack/nova master: [placement] Add cache headers to placement api requests  https://review.openstack.org/52164015:16
openstackgerritChris Dent proposed openstack/nova master: [placement] Add info about last-modified to contrib docs  https://review.openstack.org/52608415:16
*** huanxie has joined #openstack-nova15:16
*** janki has quit IRC15:17
cdentmriedem: that ^ tries to address your comments15:17
*** hamzy has joined #openstack-nova15:17
*** huanxie has quit IRC15:18
*** hamzy has quit IRC15:18
mriedemsdague: cdent: fwiw i looked at the grenade heat failure too, there is a test timing out as chris mentioned, no failures in the heat logs that i could see, but without a history of runs on that job it's hard to know if it's just always unstable15:18
*** huanxie has joined #openstack-nova15:19
mriedemsince it's non-voting and heat, i'm assuming it's just unstable15:19
mriedemas i recall, we removed heat from tempest full runs because of random timeouts15:19
dansmithI've sent it to its destiny15:20
* cdent will be back later to look at the nrp stack15:23
*** cdent has quit IRC15:23
belmoreiramriedem dansmith Quick question. Ocata requires cells mappings to be defined (it needs at least one cell defined). With cellsV1 are these mappings used in anyway?15:25
mriedemedleafe: comments in https://review.openstack.org/#/c/511358/15:26
dansmithbelmoreira: yeah, but they're not the same cells15:26
*** david-lyle has joined #openstack-nova15:26
*** rmart04 has quit IRC15:26
dansmithbelmoreira: meaning, the mappings should just be straight-through to whatever the regular nova db for each cell is15:27
*** rmart04 has joined #openstack-nova15:27
*** huanxie has quit IRC15:27
*** huanxie has joined #openstack-nova15:28
belmoreirathe cells DBs endpoints are actually used?15:28
belmoreiraI mean, with cellsV1 and no conductor on top cell15:28
dansmithbelmoreira: they are, yeah, but for you, each api db will have one cell mapping pointing at the corresponding cell db15:29
dansmithbelmoreira: remember, cellsv2 is fully woven into the main code.. there is no way to separate it.. no mode where the cell mappings aren't used15:29
*** smatzek has joined #openstack-nova15:30
*** smatzek has quit IRC15:30
*** smatzek has joined #openstack-nova15:30
belmoreiradansmith: but I continue to have a global api_db15:31
dansmithbelmoreira: I thought you were doing separate api dbs?15:31
*** owalsh_ is now known as owalsh15:32
*** huanxie has quit IRC15:32
dansmithI thought the question was over separate/global placement15:32
belmoreiradansmith yes, but I still need to have a global (top cell in cellsV1). local nova_apis (in each child cell) are basically for placement15:33
*** yikun has quit IRC15:33
dansmithI think we've always assumed that api_db would be replicated in each cellsv1 cell15:33
*** liusheng has quit IRC15:33
*** huanxie has joined #openstack-nova15:33
dansmithbelmoreira: well, the api db at the top isn't global though, right? it just has all instances in it because of the fact that the top-level cell does too...15:34
openstackgerritBalazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407  https://review.openstack.org/52609515:34
openstackbug 1735407 in OpenStack Compute (nova) "[Nova] Evacuation doesn't respect anti-affinity rules" [Medium,In progress] https://launchpad.net/bugs/1735407 - Assigned to Balazs Gibizer (balazs-gibizer)15:34
openstackgerritBalazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild  https://review.openstack.org/52524215:34
*** david-lyle has quit IRC15:34
*** gbarros has quit IRC15:35
*** moshele has quit IRC15:35
belmoreiraI didn't test everything yet, but was expecting to have instances, build_request, request_spec, aggregates, flavors... everything defined there. Without need to define it again in the locals nova_api DBs15:36
*** tidwellr has quit IRC15:36
*** udesale has quit IRC15:36
dansmithwell, the bit of compute/api that gets re-run in the child cells will need to talk to an api database,15:36
dansmithbut perhaps that will be okay for it to be global15:37
* dansmith is thinking15:37
*** huanxie has quit IRC15:37
*** Apoorva has joined #openstack-nova15:37
efriedjaypipes If I ask for ?tree=<uuid that doesn't exist>, should I get an empty list or a 404?15:37
dansmithbelmoreira: regardless, in that top-level nova_api db, I think you just want the one cell mapping to the api cell's nova db15:37
jaypipesefried: empty list15:37
*** salv-orlando has joined #openstack-nova15:37
efriedjaypipes is that The Right Thing?15:37
jaypipesefried: ask cdent.15:37
edleafemriedem: ok; still cleaning up the functional tests in https://review.openstack.org/#/c/51670715:37
*** hemna_ has joined #openstack-nova15:37
* efried waits patiently for cdent...15:38
*** huanxie has joined #openstack-nova15:38
*** tidwellr has joined #openstack-nova15:39
belmoreiradansmith: hummm... I was not thinking about that cell mapping (top level nova DB).15:40
mriedemefried: jaypipes: i think empty list is the right thing15:41
mriedem404 is for a specific resource not being found15:41
mriedeme.g. GET /resource_providers/{id} where the RP for {id} isn't found15:41
mriedemsomething not found in a body is a 400, but this is a filter parameter15:42
mriedemfiltering on something which doesn't exist gets you nothing back15:42
*** huanxie has quit IRC15:42
mriedemi seem to remember this same quandary in another spec, maybe for the traits stuff15:42
belmoreiradansmith: I need to test it. But still not sure why it's required.15:42
dansmithbelmoreira: not sure why the cell mapping is required?15:43
*** lpetrut has quit IRC15:43
efriedmriedem jaypipes okay, that makes sense, thanks.15:43
belmoreiradansmith: I understood that you said that in nova_api global I need to have a mapping to the nova DB (father cell)15:44
dansmithright15:44
*** lpetrut has joined #openstack-nova15:44
edleafeefried: what mriedem said. Empty list is correct.15:45
*** salv-orlando has quit IRC15:45
efriedthanks15:46
belmoreiradansmith: I completely forgot to do that mapping. And in my early tests I haven't seen any problem15:46
dansmithbelmoreira: cellsv1 will probably side-step some things that would make things continue to work without that mapping, but not all15:46
dansmithbelmoreira: just because it overrides certain code paths that we're not updating for cellsv215:47
*** egonzalez has joined #openstack-nova15:47
belmoreiradansmith: fair enough. Must be that15:47
dansmithbelmoreira: I wouldn't think it would override enough to make it seem like things are really working, but I'd have to test, as you say15:47
*** dave-mccowan has joined #openstack-nova15:47
mriedemin ocata,15:48
mriedemit looks like the only things that use cell mappings are the api when listing instances, and conductor when burying an instance that fails to shedule in cell015:48
mriedemfor cellsv1, are those things that only happen in the child api?15:48
*** mvk has quit IRC15:48
dansmithno, that's the thing,15:48
openstackgerritHuan Xie proposed openstack/nova master: XenAPI: Don't use nicira-iface-id for XenServer VIF  https://review.openstack.org/45164115:48
dansmithlisting instances should be mostly the same,15:48
dansmithalthouuuuugh...15:48
dansmithwe do have some "if cellsv1: then do_old_thing()" escapes in that listing logic15:49
dansmithso maybe that's it15:49
mriedemoh wait,15:49
mriedemyup15:49
dansmithyeah, okay15:49
mriedemwhen listing instances, if cellsv1 we do the old thing15:49
mriedemwe do look in cell0 though15:49
dansmithyeah15:49
mriedemso you'll need the cell0 mapping at the top yes?15:49
mriedemok15:49
dansmithyes15:49
mriedembelmoreira: ^15:49
dansmithbelmoreira: you will want the record there anyway, at some point15:49
mriedemscheduler and api don't use cell mappings until pike15:49
dansmithbelmoreira: also, while you're here.. this fix is tanking the cellsv1 job: https://review.openstack.org/#/c/523187/15:50
dansmithwhich is, like, not a good sign,15:50
belmoreirayeah, cell0 I defined15:50
dansmithbut I'm not sure why, especially since you said it works15:50
dansmithlike, 100% fail on anything that hits nova15:50
dansmithwhich seems hard to believe15:50
*** edand has joined #openstack-nova15:51
*** huanxie has joined #openstack-nova15:52
belmoreiradansmith: I'm running this patch already. Haven't detected any issue yet. However, the functionality that we expose is very limited. Maybe that's why.15:55
huanxieHi jaypipes, I have updated the reno to make it precise and please help review it again https://review.openstack.org/#/c/451641/ Thanks a lot :)15:55
mriedembelmoreira: can't even build an instance it looks like15:55
jaypipeshuanxie: already +2d.15:55
dansmithbelmoreira: I dunno, this seems to fail everything all the time, but I can't really see any errors in the logs15:55
mriedemtimes out waiting to go to ACTIVE15:55
huanxieMany thanks jaypipes :)15:56
belmoreiradansmith: but, as you said, not a good sign :) Will keep eyes open on this15:56
*** awaugama has quit IRC15:56
*** huanxie has quit IRC15:56
dansmithwell, anyway, I don't want to spend too much time on it, but unless we figure that out, we can't merge it15:56
*** emccormick has joined #openstack-nova15:57
belmoreiradansmith: definitely create/delete instances is working for us with this patch15:57
dansmithalthough that's ocata I guess15:57
belmoreirabut I'm running newton15:57
dansmithoh, newton15:57
dansmithwell, still, I'd expect to see an error somewhere if this was actually blowing something up15:58
belmoreirain pike I will not need this patch15:58
openstackgerritMerged openstack/nova master: Add a new check to volume attach  https://review.openstack.org/52562215:59
mriedemdansmith: looking at the instance create flow, if cellsv1 the api will create the instance before casting to build_instances in conductor,15:59
mriedembuild_instances in conductor creates the instance mapping15:59
mriedemwell, api creates the instance mapping16:00
mriedemconductor gets the host mapping for the chosen host and pulls the cell mapping from that host mapping to set on the instance mapping16:00
mriedemthen deletes the build request16:00
mriedemah,16:00
mriedemand if the build request is already destroyed when conductor tries to do it,16:01
mriedemconductor deletes the instance mapping16:01
mriedemwhich the api relies on i think16:01
*** awaugama has joined #openstack-nova16:01
mriedemhttps://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L599-L60516:01
dansmithoh you think we're deleting the build request early enough that it just nukes the instance when it tries because it assumes the user did it?16:01
*** lpetrut has quit IRC16:02
mriedemmaybe16:02
mriedemand when getting the instance from the api, we'll pull it from the top cell https://github.com/openstack/nova/blob/master/nova/compute/api.py#L222116:02
dansmiththe update at top thing should really happen pretty late, after the cell sync, so I didn't think that was a problem16:02
mriedemupdate at top happens for any state change doesn't it?16:03
*** emccormick has quit IRC16:03
mriedemhttps://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L57316:03
mriedemso ^ should trigger an update at the top16:03
mriedemwhich will delete the build request16:03
mriedemand then later we'll get build request not found which will make conductor delete the instance mapping https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L60416:04
mriedemthinking the instance was deleted by the user during build16:04
mriedemyup, and https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L605 means we don't cast to the compute to build the instance16:04
dansmithyeah, but I expected it to happen late enough16:04
mriedemi think i get a cookie now16:05
mriedemfrosted sugar with sprinkles please16:06
dansmithweyl.. I'm not sure what to do, other than to weaken conductor's interpretation of the BR being missing if cellsv116:06
mriedemcan we conditionally delete the build request in cells/messaging based on the instance state?16:06
dansmithwell, which state?16:07
ildikovmriedem: I'm at a place selling cupcakes one of which is called 'Uniporn & Rainho', sucks I can't send over one through IRC :/16:07
mriedemdansmith: is ACTIVE too late?16:07
dansmithI would think,16:07
dansmithsince we might not even make it to ACTIVE16:07
mriedemi'm not even sure why we need this16:08
dansmithwhat if we do it only if save() succeeded16:08
mriedemgiven https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L59716:08
dansmithmriedem: I think it's because that might've happened in a separate api db16:08
*** andreas_s has quit IRC16:08
mriedemwe could alternatively check if we've set the cell mapping on the instance mapping, meaning we've picked a host, https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L509 but that's racy16:08
mriedembecause then conductor and cells/messaging are racing to delete the build request16:09
*** andreas_s has joined #openstack-nova16:09
mriedembut cells/messaging will only do it on the next update at top16:09
mriedemwhich presumably would come after conductor has set the mapping and already deleted the build request16:09
dansmithif they're sharing an api db, but not the cell db,16:09
mriedemhmm, so the thing conductor deletes isn't necessarily the same thing that cells/messaging deletes?16:10
dansmiththen an update will delete the BR in the shared api db, which will then make conductor delete from the cell db, which will sync back up as a delete to the top level db16:10
dansmithbut if they're separate api dbs,16:10
dansmiththen you have to sync the BR delete up or you shadow the actual instance16:10
*** andreaf_ has joined #openstack-nova16:10
dansmithsince we were in the cell db when we did the BR delete16:10
dansmithmriedem: you wanna borrow my rusty spoon when I'm done with it?16:11
mriedemfor your eyeballs?16:11
dansmithyes16:11
mriedemsure16:11
mriedemsend'er over16:11
dansmithheh16:11
mriedempre-gooped please16:11
*** gbarros has joined #openstack-nova16:12
*** andreas_s has quit IRC16:13
*** erlon has joined #openstack-nova16:13
*** andreas_s has joined #openstack-nova16:13
*** rmart04 has quit IRC16:14
dansmithmriedem: anyway, I want to fix this, but I can't really justify spending time on it ahead of other stuff on my plate16:14
dansmiththe patch is there for people to apply if they need it for that scenario, and maybe I can circle back after FF or something16:14
belmoreiradansmith: +1, for me this patch makes sense in older versions (newton, ocata). Not sure how useful it will be in Pike, Queens.16:17
belmoreiradansmith mriedem thanks. I will keep you posted16:17
*** andreaf has quit IRC16:17
*** andreaf_ is now known as andreaf16:17
mriedemdansmith: edleafe: so after careful contemplation in the last 5 minutes,16:19
mriedemi think we can unpin the Selection object patch16:19
*** faridda has joined #openstack-nova16:19
dansmithokay16:19
mriedemmy concerns in the top patch in the series about filtering out the hosts that have already been tried can be done in other ways using the filter_properties retry list of hosts16:19
mriedemi.e. knowing what's already been claims16:19
mriedem*claimed16:19
bauzasI need to look at those Selection changes me too16:20
dansmithgdi gerrit16:20
mriedemjaypipes: https://review.openstack.org/#/c/495854/ if you will +W16:21
*** belmoreira has quit IRC16:22
mriedemor i can just re-approve, it was a rebase plus some minor changes16:23
mriedemi'll just re-approve :)16:23
*** damien_r has quit IRC16:23
*** gouthamr has quit IRC16:23
*** andreas_s has quit IRC16:24
*** cdent has joined #openstack-nova16:24
*** andreas_s has joined #openstack-nova16:24
gibimriedem: I proposed a fix for bug 1735407 but I feel that it only solves part of the race.16:25
openstackbug 1735407 in OpenStack Compute (nova) "[Nova] Evacuation doesn't respect anti-affinity rules" [Medium,In progress] https://launchpad.net/bugs/1735407 - Assigned to Balazs Gibizer (balazs-gibizer)16:25
cdentefried: jaypipes, mriedem and edleafe are all correct16:25
mriedemefried: per your earlier question about empty vs 404 ,this was the spec discussion i was thinking of https://review.openstack.org/#/c/497713/9/specs/queens/approved/add-trait-support-in-allocation-candidates.rst@7816:25
gibimriedem: I would appreciate your view about the possible solution I drafted in the bug report (see my last 3 comments there)16:25
efriedcdent mriedem ack16:26
cdentthe only time a 404 should happen on a collection resource is if the URL doesn’t exist (in which case it’s not a collection resource)16:26
mriedemgibi: in general i think reducing the window is a positive step forward, despite other known limitations as pointed out16:27
mdboothjaypipes: Pretty sure that anywhere that uses retry without creating a transaction context would be a bug, no?16:27
*** andreas__ has joined #openstack-nova16:27
mriedemgibi: #3 in comment 11 would be a further improvement in case we still have issues16:28
*** andreas_s has quit IRC16:28
gibimriedem: I agree. We have to see if the current fix solves the problem in the environment it was found earlier16:29
gibimriedem: if yes then we are OK, if no then I can try to implement option #316:29
sean-k-mooney2mriedem: dansmith when ye have a second could ye take alook at this trival update to the .gitreview file for stable/pike in os-vif https://review.openstack.org/#/c/488670/16:30
*** ragiman has quit IRC16:30
dansmithsnagged16:31
mriedemgibi: there is an issue in your patch16:31
mriedemgibi: the compute can't up-call to get the request spec16:31
mriedemb/c the reqspec is in the api db16:31
mriedemand the compute should be isolated fromthat16:31
*** andreas__ has quit IRC16:32
gibimriedem: that is bad16:32
mriedem:)16:32
* gibi going back to the drawing board16:32
mriedemit just means that you would have to plumb the request_spec through to the compute rebuild_instance method16:32
mriedemusing rpc api parms16:32
gibimriedem: would that mean an rpc version bump as well?16:32
mriedemyes16:32
mriedemgranted, _validate_instance_group_policy is already doing an up-call to the api db16:32
mriedemby getting the server groups16:32
mriedemand that's why we have CONF.workarounds.disable_group_policy_check_upcall16:33
gibiwould it make sense doing the request spec upcall in the _validate_instance_group_policy ?16:34
gibiin _do_validation16:34
mriedemno,16:34
mriedemwe need fewer up-calls, not more16:35
gibitrue16:35
mriedemhttps://docs.openstack.org/nova/latest/user/cellsv2-layout.html#caveats-of-a-multi-cell-deployment16:35
mriedem^ that list needs to shrink16:35
gibithis would be inside the 'The late anti-affinity check' item in that list, but I agree to look at the other option instead16:36
gibiso, if the rpc version is bumped, can I still backport the fix to stable branches?16:36
mriedemno16:36
mriedemgibi: given the super latent nature of this bug,16:37
gibiit seems this bug exists from at least Mitaka16:38
mriedemi don't think we need to add more up-calls within the validate method just to backport16:38
mriedemthe bug has existed since evacuate i assume16:38
gibicould be16:38
gibiso we say we only fix it in master and not on any stable16:39
mriedemi think so16:39
*** nmathew- has joined #openstack-nova16:39
gibiOK, I will do the rpc change16:39
openstackgerritMike Perez proposed openstack/nova master: Replace support matrix ext with common library  https://review.openstack.org/48130416:40
gibimriedem: thanks for feedback on the fix16:40
mriedemnp16:40
*** gbarros has quit IRC16:40
*** nmathew has joined #openstack-nova16:41
jaypipesmdbooth: the transaction context is automatically managed by the engine facade. I'm saying there's no need to do the secondary independent transaction context thing16:43
openstackgerritEric Fried proposed openstack/nova master: placement: allow filter providers in tree  https://review.openstack.org/37721516:43
openstackgerritEric Fried proposed openstack/nova master: placement: adds REST API for nested providers  https://review.openstack.org/38480716:43
openstackgerritEric Fried proposed openstack/nova master: placement: update client to set parent provider  https://review.openstack.org/38569316:43
openstackgerritEric Fried proposed openstack/nova master: Scheduler set_inventory_for_provider does nested  https://review.openstack.org/52064316:43
openstackgerritEric Fried proposed openstack/nova master: SchedulerReportClient._get_providers_in_tree  https://review.openstack.org/52066316:43
openstackgerritEric Fried proposed openstack/nova master: SchedulerReportClient._get_providers_in_aggregates  https://review.openstack.org/52109716:43
openstackgerritEric Fried proposed openstack/nova master: ProviderTree.populate_from_iterable  https://review.openstack.org/52075616:43
openstackgerritEric Fried proposed openstack/nova master: WIP: Scheduler[Report]Client.get_provider_tree  https://review.openstack.org/52109816:43
openstackgerritEric Fried proposed openstack/nova master: WIP: ComputeDriver.update_provider_tree()  https://review.openstack.org/52118716:43
openstackgerritEric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker  https://review.openstack.org/52024616:43
efriedjaypipes cdent edleafe ^ The bottom three are just rebases onto latest master.  The rest that aren't WIP ought to be clear and ready for reviews.16:44
jaypipesefried: cool. hopefully you didn't leave the same carnage as I did trying to rebase those...16:44
cdentefried: roger. just header into that stack now16:44
*** nmathew- has quit IRC16:44
mdboothjaypipes: But it's not managed by the retry, right? So if you retry without an enginefacade call at the same scope, you'll still have an aborted transaction.16:44
efriedjaypipes I am also hopeful :)16:44
*** nmathew- has joined #openstack-nova16:44
efriedjaypipes I think I mainly undid carnage16:45
mdboothjaypipes: That is, all your retries will fail.16:45
*** nmathew- has quit IRC16:45
jaypipesmdbooth: no.16:45
jaypipesmdbooth: the retry decorator is designed to work with the enginefacade's transaction context...16:45
jaypipeszzzeek: you around? need your expert opinion on something.16:46
mdboothjaypipes: Ah... lemme check that. I may have missed that.16:46
efriedjaypipes Argh, found a pep8 error in your REST API change.16:46
efriedjaypipes in_tree pushed a docstring line over 80c :(16:46
jaypipesefried: that I did or that you did? :)16:47
zzzeekjaypipes: gotta leave in about 3 minutes but whats up16:47
jaypipesefried: ah16:47
jaypipeszzzeek: sec, grabbing link16:47
efriedjaypipes You did.  But let me fix it, since I've got the stack right here.16:47
*** egonzalez has quit IRC16:47
mriedemcdent: done https://review.openstack.org/#/c/521639/ thanks16:47
jaypipeszzzeek: line 123, my comment here: https://review.openstack.org/#/c/242603/25/nova/objects/block_device.py16:47
cdentmriedem: cool, yeah on the IRON_NFV cleanup, I didn’t want to disrupt the entire functional test16:48
jaypipeszzzeek: AFAIK, there's no need to do that16:48
openstackgerritEric Fried proposed openstack/nova master: placement: allow filter providers in tree  https://review.openstack.org/37721516:48
openstackgerritEric Fried proposed openstack/nova master: placement: adds REST API for nested providers  https://review.openstack.org/38480716:48
*** nmathew has quit IRC16:48
openstackgerritEric Fried proposed openstack/nova master: placement: update client to set parent provider  https://review.openstack.org/38569316:48
jaypipeszzzeek: i.e. no need for the txtct.using(context) thing on line 12516:48
openstackgerritEric Fried proposed openstack/nova master: Scheduler set_inventory_for_provider does nested  https://review.openstack.org/52064316:48
openstackgerritEric Fried proposed openstack/nova master: SchedulerReportClient._get_providers_in_tree  https://review.openstack.org/52066316:48
openstackgerritEric Fried proposed openstack/nova master: SchedulerReportClient._get_providers_in_aggregates  https://review.openstack.org/52109716:48
openstackgerritEric Fried proposed openstack/nova master: ProviderTree.populate_from_iterable  https://review.openstack.org/52075616:48
openstackgerritEric Fried proposed openstack/nova master: WIP: Scheduler[Report]Client.get_provider_tree  https://review.openstack.org/52109816:48
openstackgerritEric Fried proposed openstack/nova master: WIP: ComputeDriver.update_provider_tree()  https://review.openstack.org/52118716:48
openstackgerritEric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker  https://review.openstack.org/52024616:48
*** jangutter has quit IRC16:48
efriedjaypipes Fixed.  Very bottom of the stack, of course.16:48
*** gyee has joined #openstack-nova16:48
jaypipesefried: ack16:48
jaypipesty16:48
mdboothjaypipes: I'm looking at the code for wrap_db_retry, and I don't see anything which would create a new transaction.16:48
mdboothWe normally use this decorator on a function which also has a decorator to create a transaction context, though.16:49
jaypipesmdbooth: the whole *point* of the retry decorator is to catch the ROLLBACK and restart a transaction :)16:49
mdboothI don't think so, no.16:49
mdboothI think it just retries.16:49
*** lpetrut has joined #openstack-nova16:50
mdboothAnd the way we always use it is that we also create a transaction.16:50
zzzeekjaypipes: i belive your comment is accurate but the method itself has to be called before there's some larger transaction going on , unless the retry decorators know to propagate all the way up to the top one16:50
efriedbauzas gibi Would you please re-+A https://review.openstack.org/377215 (rebase & trivial pep8 change)16:50
mdboothThere's no nested context stuff going on with the retry decorator16:51
*** Alex_Staf has quit IRC16:51
bauzasefried: did16:51
efriedbauzas Merci16:51
mdboothjaypipes: I'm looking at the code: it just runs it again16:51
dansmiththe transaction is in the decorator, so if the retry is above that, it'll do another transaction16:52
*** syjulian has quit IRC16:52
mdboothdansmith: Right.16:52
mdboothThey're 2 separate things, but we normally use them together.16:52
zzzeekjaypipes mdbooth i dont see the point of the retry decorator if you have a context manager inside to handle the thing anyway16:52
mdboothzzzeek: Well the context manager always has to be inside the retry decorator16:53
jaypipeszzzeek: me neither, which is why I wrote that comment.16:53
*** jangutter has joined #openstack-nova16:53
mdboothHowever, we normally put it inside the retry decorator using another decorator16:53
mdboothSo:16:53
mdbooth@retry16:53
mdbooth@transaction16:53
mdboothdef my_foo():16:53
*** hamzy has joined #openstack-nova16:53
zzzeekmdbooth jaypipes haven't looked at this stuff in awhile, the context manager looks unfamiliar to me, would want to see other places the rerty decorator is used and see if a. they also have a nested trans or b. they dont, and then how do these work?16:54
* zzzeek has to run16:54
*** hongbin has joined #openstack-nova16:54
*** AlexeyAbashkin has quit IRC16:54
gibiefried: done16:54
efriedgibi Thanks!16:54
*** chyka has joined #openstack-nova16:54
mdboothjaypipes: Let me think again if there's a good enough reason not to use the decorator there.16:54
*** fragatina has quit IRC16:55
dansmithsales meeting in 516:55
jaypipesmdbooth: I'm not saying don't use the decorator. I'm saying no need to use the inner tctxt.using() thing.16:55
mdboothjaypipes: So I'm also entirely sure we must have an enginefacade transaction scope there16:56
mdboothWhich we normally do at function scope using a decorator16:56
mdboothI'm actually being super-defensive there, I think16:56
openstackgerritBalazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407  https://review.openstack.org/52609516:57
openstackbug 1735407 in OpenStack Compute (nova) "[Nova] Evacuation doesn't respect anti-affinity rules" [Medium,In progress] https://launchpad.net/bugs/1735407 - Assigned to Balazs Gibizer (balazs-gibizer)16:57
mdboothIf the transaction fails on commit, it'll trigger a retry16:57
mdboothWhen we retry, we want a new transaction16:57
mdboothSo we need to create one somehow16:57
mdboothI'm explicitly using an independent transaction16:58
mdboothSo I'm explicitly *not* using any enclosing transaction scope16:58
jaypipesmdbooth: ahhhh... sorry, yeah, I didn't realize there was no enginefacade decorator being used on any of these object entrypoints (BlockDeviceMapping.create()/save(), etc)16:58
mdboothSo it definitely looks weird because I'm being super-defensive.16:58
*** salv-orlando has joined #openstack-nova16:58
mriedemildikov: replied in https://review.openstack.org/#/c/525787/16:59
jaypipesmdbooth: my presumption was that there was an enginefacade trx context decorator on one of the BDM object methods. :(16:59
mriedemildikov: if you agree, i'll start on those changes after my next meeting16:59
jaypipesmdbooth: and I see now there isn't...16:59
mdboothActually, when I originally wrote that I was less defensive and jsut wrote a comment about how this might fail on multi-master galera16:59
mdboothBut we don't care, because it almost definitely won't16:59
mdboothAnd that version looked more normal16:59
jaypipesmdbooth: instead of using that independent context using() thing, I believe you can instead just decorate the _create_uuid() method with the @writer enginefacade decorator, in addition to the @wrap_db_retry decorator.17:00
melwittmriedem: replied on the review about power_on17:00
mdbooth(but it failed safely)17:00
mdboothjaypipes: Note the 'independent' in there17:00
*** tssurya_ has joined #openstack-nova17:00
mdboothThat's so a race there wouldn't cause a failure of the encompassing transaction17:01
mdboothjaypipes: Again, I could be convinced to roll back to my big comment explaining why this might fail sometimes.17:01
jaypipesmdbooth: *what* encompassing trx though? I had originally thought there was an encompassing transaction managed by an enginefacade transaction context manager, but I see now there isn't any enginefacade decorator being used on any of the BDM object methods like create(), save(), etc17:02
mdboothjaypipes: Not today, no.17:02
jaypipesmdbooth: you're implying there will be? :)17:03
mdboothjaypipes: The design of the enginefacade encourages it, and we have no way of knowing.17:03
mdboothe.g. if the thing *calling* save had its own transaction17:03
mdboothwe'd be in an encompassing scope17:03
mdboothDo we ever call save() within a transaction scope?17:04
mdboothWe never used to, because they were confined to db.api()17:04
mdboothHowever, they've been moving out of there for a while.17:05
mdboothjaypipes: Anyway, this is very defensive. I wrote it such that it shouldn't fail however you call it. That makes it look different, so it's a tossup whether it's worth it.17:06
openstackgerritTheodoros Tsioutsias proposed openstack/python-novaclient master: CommandError is raised for invalid server fields  https://review.openstack.org/52511017:06
jaypipesmdbooth: commented on the patch again...17:06
jaypipesmdbooth: lemme know if that code makes more sense.17:06
jaypipesmdbooth: sorry, I originally had missed the fact that no enginefacade trx context decorators were in use for BDM object17:07
bauzasquick question for privsep specialists17:07
openstackgerritTheodoros Tsioutsias proposed openstack/python-novaclient master: CommandError is raised for invalid server fields  https://review.openstack.org/52511017:07
mdboothjaypipes: Yeah, I think the only reason I didn't do that was to have an independent transaction17:07
*** Guest18930 has quit IRC17:07
bauzasgiven I need to open a file for write attributes, do I need to use a privsep method doing os.open() or can I just call it directly?17:07
mdboothjaypipes: i.e. a transaction which is explicitly divorced from any encompassing scope17:08
*** itlinux has joined #openstack-nova17:08
mdboothBecause if that was called from within an encompassing scope, it would still fail on retry, as well as failing the encompassing scope17:08
mdboothBut as I said, this is super-unlikely.17:08
bauzasAFAICT, I can see in the libvirt driver some os.open calls that write directly17:08
*** hamzy has quit IRC17:08
mdboothSo a comment might suffice.17:09
sean-k-mooney2bauzas: if you dont need elevated privlages to open the file in the mode you need then you dont need privsep17:09
*** hamzy has joined #openstack-nova17:09
*** salv-orlando has quit IRC17:09
bauzassean-k-mooney2: well, it's a sysfs call17:09
bauzasI need to open("/sys/class/mdev_bus/<device>/mdev_supported_types/<type>/create", O_WRITE)17:10
*** salv-orlando has joined #openstack-nova17:10
mdboothjaypipes: It's an interesting point, btw. I think everywhere that we use the retry decorator would be broken unless it's guaranteed to be a top-level transaction scope.17:10
sean-k-mooney2bauzas:  oh are you working a lib for that beacues we are planning to write one soon17:10
*** david-lyle has joined #openstack-nova17:10
bauzassean-k-mooney2: no, I'm implementing the libvirt side for creating a VGPU17:11
bauzasbut for that, nova needs to instanciate a mediated device17:11
bauzasthen, once that done, it needs to update the guest XML using that UUID it just created17:11
sean-k-mooney2bauzas: or libvirt could17:11
bauzassean-k-mooney2: libvirt doesn't do that atm17:11
bauzascall it experimental :)17:12
*** hamzy has quit IRC17:12
bauzassean-k-mooney2 https://libvirt.org/drvnodedev.html#MDEV (bottom page)17:12
bauzasyou'll see the lovely sysfs call I need to make17:12
sean-k-mooney2bauzas: i know that is why we where planning on writing a singel file pytyon wrapper around sysfs to do it17:12
jaypipesmdbooth: certainly something to chat with zzzeek about at some later time.17:12
*** hamzy has joined #openstack-nova17:12
sean-k-mooney2and in our case call it for os-vif17:12
bauzashum17:12
bauzassean-k-mooney2: okay, so that's definitely a privsep decoy then17:13
sean-k-mooney2so i would assume that /sys/class/mdev_bus/<device>/mdev_supported_types/<type>/create is owned by root17:13
*** sdague has quit IRC17:13
openstackgerritMerged openstack/os-vif stable/pike: Update .gitreview for stable/pike  https://review.openstack.org/48867017:13
bauzassean-k-mooney2: zactly, so I'll just make a public method in the nova.privsep helper module17:13
bauzascall it tech debt if you wish17:13
*** salv-orlando has quit IRC17:14
bauzasif the libvirt API was supporting to do such things like creating a mediated device without asking to use sysfs, then I wouldn't need to do such crap things in nova17:14
sean-k-mooney2bauzas: yep but the real question is what capablitis do you need?17:14
*** damien_r has joined #openstack-nova17:14
*** fragatina has joined #openstack-nova17:14
bauzassean-k-mooney2: AFAIK, nova privsep uses the global ones17:15
ildikovmriedem: agree, replied, thanks17:15
*** lucasagomes is now known as lucas-afk17:15
sean-k-mooney2bauzas: well my point is there are two parts to this. the mode of the file e.g. is it globally writeabe or do you have to be a specific user/group and second do you need cap_sys_admin for exampel to write to it ignoring the filesystem mode17:16
bauzassean-k-mooney2: for the first question, I'm supposing to use the uid 0 for opening and writing the file17:17
bauzasso the basic nova.privsep context17:17
sean-k-mooney2bauzas: so you will reuse this one https://github.com/openstack/nova/blob/2af92486b2dff6eb50c2e1f52e23eeae50d4bf98/nova/privsep/__init__.py#L2117:18
bauzassean-k-mooney2 : what I need is actually pretty identical to https://github.com/openstack/nova/blob/master/nova/privsep/libvirt.py#L11717:19
bauzassean-k-mooney2 yup that's what I meant17:19
sean-k-mooney2bauzas: CAP_DAC_OVERRIDE allow you to ignore filesystem permissions17:19
bauzassean-k-mooney2: nevermind, I'll basically copy/shame the hairpin cut sysfs call17:20
bauzasI need to bail out17:20
sean-k-mooney2bauzas: yes just reuse @nova.privsep.sys_admin_pctxt.entrypoint17:20
bauzasChristmas tree implementation time17:20
sean-k-mooney2cool enjoy17:21
bauzassean-k-mooney2: I knew how to use privsep, I was just wondering whether it was needed17:21
bauzasbut yeah, since it's a sysfs call, I need it, so...17:21
bauzas++17:21
*** damien_r has quit IRC17:23
*** yamamoto has quit IRC17:23
cdenthere’s a question I’ve been wondering about for a long time: How do people decide when a comment is a NOTE and when it is just a comment?17:24
sean-k-mooney2bauzas: i do feel i might end up duplicating your work so can you add me to the review if you push it up. im working on using mdev's for netwoking currently though it wont be pushed upstream until rocky at the earliset as the qemu changes arent even submitted upstream yet17:24
mdboothjaypipes: As for creating null uuids in a test, I still think that my 1-liner is better than 22 lines.17:24
jaypipesmdbooth: and adding a testing-only kwarg to a public API? nah...17:25
mdboothjaypipes: It's not a public api17:25
jaypipesmdbooth: close enough to a public API in my book...17:25
mdboothAnd it's clearly called out17:25
mdboothIt's got test_ in the name ;)17:26
*** salv-orlando has joined #openstack-nova17:26
*** salv-orlando has quit IRC17:26
jaypipesmdbooth: sorry, I just disagree on that. from a style and a code structure perspective, I think it's better to explicitly create test fixtures and expectations within the test itself.17:26
mdboothjaypipes: We've also similar in a bunch of places.17:27
mdboothjaypipes: Meh, ok.17:27
jaypipesmdbooth: can you point me to something similar please?17:27
*** salv-orlando has joined #openstack-nova17:27
*** Apoorva has quit IRC17:27
mdboothjaypipes: I'm thinking class reset methods17:27
mdboothfunctions which only exist so they can be unit tested17:28
openstackgerritChhavi Agarwal proposed openstack/nova master: Volume detach should have instance uuid  https://review.openstack.org/51180417:28
mdboothfunctions which only exist so that can be mocked17:28
jaypipesmdbooth: that's a separate function. it's not changing a real function just for the purpose of testing.17:28
mdboothI'm pretty sure I've also seen test only args17:28
jaypipesmdbooth: and we also discourage creating functions that exist only to be mocked.17:28
dansmithI too am interested,17:28
mdboothIt's a wart, but it allows code reuse.17:29
dansmithbecause I don't think that we do that anywhere I'm familiar with17:29
sean-k-mooney2mdbooth: test only args kindo of defets the reason for testing17:29
dansmithyeah17:29
mdboothOtherwise we just have to duplicate code17:29
sean-k-mooney2mdbooth: you are not testing what runs in production17:29
mdboothsean-k-mooney2: The use is to create something only for use in testing17:29
mdboothSo no, it doesn't run in production :)17:29
mdboothsean-k-mooney2: In this case, it's to tweak a function to not default something which was formally not defaulted.17:30
* mdbooth thinks it's a pragmatic wart17:30
mdboothAnyway, if it's not going to pass muster I'll change it17:30
*** sambetts is now known as sambetts|afk17:31
*** ralonsoh has quit IRC17:31
*** salv-orlando has quit IRC17:32
sean-k-mooney2mdbooth: can you send me a link?17:32
*** david-lyle has quit IRC17:32
mdboothhttps://review.openstack.org/#/c/242603/25/nova/db/sqlalchemy/api.py17:32
*** hamzy has quit IRC17:32
*** david-lyle has joined #openstack-nova17:32
*** dtantsur is now known as dtantsur|afk17:33
mdboothThat function is small enough the cut/pasting it into a test isn't entirely evil17:33
sean-k-mooney2and why do you want to not see the uuid automatically?17:35
mdboothsean-k-mooney2: So as to test upgrade of legacy objects which don't have it17:35
sean-k-mooney2and you cant mock the call to uuidutils.generate_uuid() in that case to retrun None?17:36
mdboothsean-k-mooney2: Cut/paste is less evil than that.17:36
*** tssurya_ has quit IRC17:37
*** sshwarts has quit IRC17:41
*** gouthamr has joined #openstack-nova17:44
*** david-lyle has quit IRC17:45
*** rmart04 has joined #openstack-nova17:46
*** Kevin_Zheng has quit IRC17:46
*** tssurya_ has joined #openstack-nova17:47
mdboothdansmith: jaypipes So, I'm going to recut that second patch. No happy faces for https://review.openstack.org/#/c/242602/24, though ?17:47
mdboothWhile I'm respinning anyway, that is.17:47
*** rmart04 has left #openstack-nova17:47
mdboothIt's the patch immediately before.17:48
*** tssurya_ has quit IRC17:52
jaypipesmdbooth: gimme a few17:52
mdboothjaypipes: Thanks.17:53
mdboothjaypipes: That patch version is mostly dansmith with some stuff deleted that we don't need any more.17:53
jaypipesmdbooth: in that case, automatic -2 from me17:53
mdboothjaypipes: Understood.17:53
jaypipeshehe17:54
mdboothProbably for the best.17:54
jaypipes:)17:54
*** abhishekk has joined #openstack-nova17:54
*** abhishekk has quit IRC17:54
*** abhishekk has joined #openstack-nova17:58
*** derekh has quit IRC18:01
*** moshele has joined #openstack-nova18:04
mdboothSpeaking of anti-patterns, btw: functions which modify their input parameters. I ran across a unit test last week which only passed because the method it called modified the dict it was given as well as its output, so the later assertion that the 2 were equal passed coincidentally.18:06
mdboothIf it had actually mattered, that would be pretty obtuse.18:06
*** tesseract has quit IRC18:07
mdboothCommon culprit is popping values from input dicts.18:07
*** jroll has joined #openstack-nova18:10
*** salv-orlando has joined #openstack-nova18:10
sean-k-mooney2mdbooth: that depends on the fucntion. if the fucntion returned nothing the modifying the inpu arguments may be correct. like a sort fuction there are other exampels though18:10
openstackgerritEd Leafe proposed openstack/nova master: Change RPC for select_destinations()  https://review.openstack.org/51670718:10
openstackgerritEd Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils  https://review.openstack.org/51135718:10
edleafemriedem: ^^ Got these working. Now starting on your comments on https://review.openstack.org/#/c/511358/18:11
mdboothsean-k-mooney2: Right, if the purpose of the function is to modify its input parameters, that's different.18:11
sean-k-mooney2mdbooth: though i do agreee that in general a fucntion should retrun someting or have sideefect but not both18:11
*** david-lyle has joined #openstack-nova18:13
*** Apoorva has joined #openstack-nova18:15
*** Apoorva has quit IRC18:15
*** BryanS68 has quit IRC18:15
*** Apoorva has joined #openstack-nova18:15
*** BryanS68 has joined #openstack-nova18:16
mdboothjaypipes: So, I just realised that another reason to use an independent transaction in _create_uuid is that if it were ever called in the future in the context of a read transaction (because the caller is just reading from the db), it's going to be a failure because we can't promote a read transaction to a write transaction.18:17
mdboothSo basically, this only works robustly as long as we continue not using enginefacade18:17
*** david-lyle has quit IRC18:18
*** david-lyle has joined #openstack-nova18:18
jaypipesmdbooth: all the more reason to use enginefacade, no? :)18:18
mdboothjaypipes: No, the exact opposite18:18
mdboothIf we were using enginefacade the way it was intended, i.e. to create larger-scoped transactionss18:19
mdboothWe'd be hitting bugs here18:19
mdboothWe're only relatively safe because we don't do that18:19
mdboothIdeally something which might read a few bdms would have its own read transaction18:20
mdboothWhich covered all of them18:20
mdboothThen suddenly one of them doesn't have a uuid and we need to do a write18:20
mdboothThis would fail, because we're in a read transaction.18:20
*** dave-mccowan has quit IRC18:20
*** damien_r has joined #openstack-nova18:20
mdboothIf we always do the write in an independent transaction, that doesn't matter18:21
*** salv-orlando has quit IRC18:21
mdboothActually, I wonder if we'd hit that in practise...18:21
* mdbooth audits18:21
*** salv-orlando has joined #openstack-nova18:21
*** burt has joined #openstack-nova18:21
*** abhishekk has quit IRC18:22
*** yamamoto has joined #openstack-nova18:24
mdboothNo, we currently still have separate micro-transactions everywhere18:24
*** salv-orlando has quit IRC18:26
_ixHello again. I was curious about how to best segment my deployment. I'd like some hypervisors running off of rados block devices, and others running off of local ssd. Are availabiltiy zones what I'm looking for?18:26
mdboothBut if, for eg, we annotated BlockDeviceMapping.get_by_volume_id with @reader so that it fetched the joined instance in the same transaction as the bdm, that would be a bug. It would be pretty obtuse, though, because it's not at all obvious that _from_db_object can write18:27
*** yamamoto has quit IRC18:30
_ixI must be looking for host aggregates.18:30
*** damien_r has quit IRC18:30
mdboothjaypipes: Nah, the comments are longer than the code.18:31
mdbooths/code/code to fix the problems requiring comments/18:31
*** david-lyle has quit IRC18:33
sean-k-mooney2every time i read bdms i keep wanting to change it to dbms as i assume i reversed the letter because of my dyslexia18:33
*** sahid has quit IRC18:36
*** gbarros has joined #openstack-nova18:38
jeblairhi, we're looking at having nodepool ask nova for its quota periodically to make it more responsive to changes.  does nova cache limit information for users (cores, ram, instances)?  if so, what's a typical cache period?18:38
melwittjeblair: I don't think limit info is cached18:40
sean-k-mooney2melwitt: is limit info still stored in nova or is that contolled by keystone now for cores,ram and insances?18:42
jaypipesmdbooth: actually, decorating the _from_db_object() with the @writer context is actually a very *explicit* way of saying the expectations of that method. https://github.com/openstack/nova/blob/master/nova/objects/resource_provider.py#L90218:42
jaypipesmdbooth: and functional tests of the data online migration will blow up nicely if it's set to @reader instead of @writer.18:43
jaypipesmdbooth: and blowing up is a good thing in this case...18:43
mdboothjaypipes: Absolutely, but it's unnecessary except in this really weird edge case.18:43
jaypipesmdbooth: agreed18:43
mdboothWhich will go away...18:44
jaypipesmdbooth: thus the comments saying so :)18:44
mdboothExcept that now everything above it was unnecessarily promoted to writer18:44
jaypipesmdbooth: https://github.com/openstack/nova/blob/master/nova/objects/resource_provider.py#L904-L90518:44
mdboothAnd now you have to go audit all the things to find out after the fact which ones were really supposed to be writers.18:44
jaypipesyeah, that's the tradeoff I suppose18:45
*** hamzy has joined #openstack-nova18:45
*** moshele has quit IRC18:45
dansmithsean-k-mooney2: still in our db18:45
*** hamzy has quit IRC18:46
mdboothjaypipes: That races, btw18:46
sean-k-mooney2dansmith: oh ok was i imagining that we wanted to move it to keystone at some point?18:46
*** hamzy has joined #openstack-nova18:46
mdboothjaypipes: Concurrent reads can create different uuids18:47
jeblairmelwitt: thanks, that's easy then :)18:47
sean-k-mooney2mdbooth in which case it needs to be a writer lock no?18:49
mdboothsean-k-mooney2: Doesn't matter, unless both prior reads are done with 'for update' (or equivalent)18:49
mdboothWhich I don't think they are18:50
*** damien_r has joined #openstack-nova18:50
mdboothYou'll get 2 reads of 2 null values. They'll race to update. The first will write a value and block the second one. When the first commits, the second will write its own different value.18:50
mdboothBoth functions will return objects with different uuids.18:51
sean-k-mooney2mdbooth: what you effectivly want to do is have a cas so only one updates the record18:51
sean-k-mooney2mdbooth: im guessing thats what 'for update' will give you18:52
mdboothcas?18:52
jaypipescompare and swap18:52
mdboothAh, yeah18:52
jaypipesbut we're overthinking this I think.18:52
mdboothThat's what I wrote18:52
jaypipesyeah, the update_with_match thing is the compare and swap18:52
jaypipesregardless...18:52
mdboothhttps://review.openstack.org/#/c/242603/25/nova/objects/block_device.py18:52
mdboothjaypipes: Thing is, if you overthink the building blocks, it means you don't need to overthink everything else. Not so much, anyway.18:54
mdboothThe caller doesn't need to be concerned with all these weird edge cases. It just works.18:54
*** rmart04 has joined #openstack-nova18:56
*** esberglu has quit IRC18:56
*** sridharg has quit IRC18:59
openstackgerritMerged openstack/nova master: Fix doubling allocations on rebuild  https://review.openstack.org/52166219:00
zzzeekmdbooth / jaypipes the retry decorartor is wanting, IMO19:01
zzzeekthere was originally some idea to integrate one into enginefacade itself so that it knows how to nest19:02
mdboothzzzeek: Could you use a savepoint for this, btw?19:02
* mdbooth isn't entirely sure how they work19:02
zzzeekmdbooth: not if you're looking to respond to other transactions also making changes b.c. isolation may prevent you from seeing those changes19:03
zzzeekfor an UPDATE i think repeatable read would mean you can't see another transactions updates19:03
mdboothzzzeek: Anyway, my approach was to use an independent transaction context explicitly. I figured this would always be safe.19:05
*** gbarros has quit IRC19:05
mdboothThen I was convinced it wasn't worth writing slight weird code for an extremely obtuse edge case where failure was safe.19:06
zzzeekmdbooth: i think you are correct assuming that record is not part of the external tranasction.  pre-enginefacade nova was all independent transactions.19:06
mdboothBut then I flipped again because I figured this is a read transaction which only needs to write in an edge case (migrating a legacy record).19:06
mdboothSo having an independent transaction is useful to avoid forcing the enclosing transaction and all its potential parents to become writers.19:07
*** READ10 has quit IRC19:07
mdboothAnd that's where I am now.19:08
*** sree has joined #openstack-nova19:09
zzzeekmdbooth: i think leave it and goad me into working up a better retry situation for enginefacade19:10
mdboothzzzeek: So all of the above is in the context of code we'd want to delete next cycle anyway, as it's only for a migration :)19:11
mdboothWell cycle +1, because the online migration runs it too.19:11
zzzeekmdbooth: nova's magical migrations...sure19:11
zzzeekmdbooth: sure19:11
zzzeekmdbooth: this is not the most concerning thing to me :)19:12
mdboothI think it's fine, tbh. I also doubt that many of nova's uses of the retry wrapper are a problem today, because we don't nest many transactions.19:13
mdboothIt would be a bigger problem if we made wider use of the enginefacade19:13
*** sree has quit IRC19:13
* mdbooth has to run19:14
mdboothjaypipes: Thanks again19:14
*** erlon has quit IRC19:17
openstackgerritChris Dent proposed openstack/nova master: Optional separate database for placement API  https://review.openstack.org/36276619:22
*** esberglu has joined #openstack-nova19:24
*** dave-mccowan has joined #openstack-nova19:25
*** yamamoto has joined #openstack-nova19:26
*** gbarros has joined #openstack-nova19:30
*** faridda has quit IRC19:30
*** yamamoto has quit IRC19:31
*** harlowja has quit IRC19:32
cdentcan someone besides mriedem put some eyes on https://review.openstack.org/#/c/523403/ is an auth config error that would be useful to backport. dansmith, jaypipes ?19:33
efriedcdent Patches are passing with your grenade fix - thanks for that.19:33
cdenthuzzah19:34
mriedemcdent: i think melwitt enjoys auth config bug fixes19:34
melwittI totally do19:34
cdentah yes melwitt would be a fine choice19:35
cdentapologies had not hear you around recently19:35
*** moshele has joined #openstack-nova19:35
cdentwhereas dansmith and jaypipes …19:35
efriedcdent Is the kwarg really called oslo_config_config?19:35
cdentyeah, isn’t that awesome, there are a few different olso_config_*19:36
*** harlowja has joined #openstack-nova19:36
efriedwhee19:36
cdentI think it is to try to be explicit about the fact that it can also accept keyword arguments which are configuration keys and values19:37
cdenta sort of namespacing19:38
* cdent seems to recall writing at least some of that code, from way back in the ceilometer days19:38
*** mvk has joined #openstack-nova19:39
sean-k-mooney2cdent: we were going to do something similar with networking-ovs-dpdk devstack setting then reallised that would haver ment setting OVS_DPDK_OVS_REPO AND OVS_DPDK_DPDK_REPO so decided not to prepend everyting with OVS_DPDK19:41
cdentoh my19:42
sean-k-mooney2we still have pretty verbose setting without that https://github.com/openstack/networking-ovs-dpdk/blob/master/devstack/settings19:42
efriedcdent Left a comment on that patch.  Is there a reason we would need to keep that arg?19:42
cdentefried: because we don’t know who might have use the deploy() method in the past19:43
cdentthat’s how you build a custom wsgi application, if you’re so inclined19:44
efriedcdent Is it somehow public?19:44
cdentit is public by virtue of being python19:44
efriedWhat I mean is, is it used by some known external entity that we care about?19:44
cdentand in the early days of placement was the entry point to the wsgi application stack until sean added the stuff in wsgi.py19:44
cdentthe point is that we don’t know19:44
efriedBut we care?19:45
cdentyes, of course19:45
efriedOh, not of course.19:45
cdentthe whole point of a wsgi application is to be able to integrate it in whatever way you want19:45
efriedNova changes "public APIs" like ComputeDriver all the time without worrying about external consumption.19:45
sean-k-mooney2was it ever the entrypoint in a version of the placement api that shipped in a release19:45
efriedAnd when I've asked about that before, it was basically explained away to "we don't care".19:46
efriedSo in this case, why do we care?19:46
cdentsean-k-mooney2: a) I _think_ so but am not certain, b) it has always been my intent that deploy be available as a public interface19:46
cdentbecause placement is not nova?19:46
cdentit has a _much_ different attitude towards building the wsgi application than the nova-api ever did19:46
sean-k-mooney2cdent: well if that was the intent then we should treat it as public and document that fact.19:47
cdentI think I probably made the mistake of assuming that was somehow obvious, by virtue of its position in the wsgi stack, but I think I may be one of the few people around here who think of the wsgi stack in that fashion19:48
cdentwe can certainly document it as such19:48
cdentefried: would your preference be that I add that documentation to that patch now, or is it cool to worry about it later?19:49
efriedLater is fine.19:49
efriedcdent Changed sides.19:50
cdentthanks, I’ll do the documenting likely tomorrow, I think I’m out of code mode for today19:51
cdentalso, all my promises to myself to not work late tonight seem to be for naught19:51
sean-k-mooney2cdent: your based in the uk right?19:52
cdentyeah19:52
cdentand I often work this late, but last night I worked past midnight19:52
cdentso was going to give myself a break19:52
cdentbut everybody here was just too interesting today19:52
sean-k-mooney2i have a daily reminder to head home at 8 so im normally here till around now too.19:53
cdentas it is 5 to 8 I think I’ll go, have a good night every one19:56
*** cdent has quit IRC19:56
*** Sandy619 has joined #openstack-nova19:56
*** smatzek has quit IRC20:08
melwittmriedem: in case you didn't see, I had posted a follow up for the old quotas code removal that we talked about some days ago https://review.openstack.org/#/c/52423420:09
*** smatzek has joined #openstack-nova20:10
*** smatzek_ has joined #openstack-nova20:12
melwittalthough I think I just realized an issue with it and commented20:12
mriedemlgtm20:12
mriedemoh right20:13
mriedemgawd20:13
melwittyeah. guh20:14
*** smatzek has quit IRC20:14
mriedemhttps://review.openstack.org/#/c/400384/20:15
melwittI need to add some kind of equivalent test to the API tests somewhere. the api samples test cover the presence of the 'reserved' key but not the expected value if no-op quotas20:15
mriedemwell,20:16
mriedemeasiest is probably an api samples test with the noop quota driver and it's own samples20:16
melwittlemme see if I can do that20:23
openstackgerritHongbin Lu proposed openstack/nova master: Add support for listing hosts in cellv2  https://review.openstack.org/52475520:25
*** smatzek_ has quit IRC20:27
*** yamamoto has joined #openstack-nova20:27
*** smatzek has joined #openstack-nova20:29
*** smatzek has quit IRC20:29
*** smatzek has joined #openstack-nova20:30
*** pchavva has quit IRC20:31
*** dtantsur|afk has quit IRC20:31
*** ttsiouts has quit IRC20:31
*** brault has quit IRC20:31
*** mhenkel has quit IRC20:31
*** purplerbot has quit IRC20:31
*** Anticime1 has quit IRC20:31
*** jdurgin has quit IRC20:31
*** fmccrthy has quit IRC20:31
*** fyxim has quit IRC20:31
*** zhenguo has quit IRC20:31
*** dgonzalez has quit IRC20:31
*** khappone has quit IRC20:31
*** raorn has quit IRC20:31
*** sgordon has quit IRC20:31
*** moshele has quit IRC20:31
*** dave-mccowan has quit IRC20:31
*** chyka has quit IRC20:31
*** hongbin has quit IRC20:31
*** gyee has quit IRC20:31
*** awaugama has quit IRC20:31
*** jmlowe has quit IRC20:31
*** Eran_Kuris has quit IRC20:31
*** xinliang has quit IRC20:31
*** rocky-shiyan has quit IRC20:31
*** ekhugen_alt has quit IRC20:31
*** pprokop has quit IRC20:31
*** MasterofJOKers has quit IRC20:31
*** larainema has quit IRC20:31
*** FL1SK has quit IRC20:31
*** jroll has quit IRC20:31
*** yamamoto has quit IRC20:32
*** syjulian has joined #openstack-nova20:38
*** mikal has quit IRC20:38
*** itlinux has quit IRC20:40
openstackgerritMatt Riedemann proposed openstack/nova master: Add new style volume attachment support to block_device.py  https://review.openstack.org/52578720:41
openstackgerritMatt Riedemann proposed openstack/nova master: Implement new attach Cinder flow  https://review.openstack.org/33028520:41
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: libvirt: Allow multiple volume attachments  https://review.openstack.org/26758720:41
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Allow multi-attach in compute api  https://review.openstack.org/27104720:41
openstackgerritMatt Riedemann proposed openstack/nova master: Avoid stashed connector lookup for new style detach  https://review.openstack.org/52618220:41
*** mikal has joined #openstack-nova20:43
*** lpetrut has quit IRC20:45
efriedjaypipes I seem to have flubbed https://review.openstack.org/#/c/521605/ which was supposed to be two separate commits (one to add agg stuff to ProviderTree, one to add traits stuff).20:46
efriedjaypipes I was about to split it up, but thought I should ask you whether it's worth bothering20:47
*** pchavva has joined #openstack-nova20:47
*** dtantsur|afk has joined #openstack-nova20:47
*** ttsiouts has joined #openstack-nova20:47
*** brault has joined #openstack-nova20:47
*** mhenkel has joined #openstack-nova20:47
*** purplerbot has joined #openstack-nova20:47
*** Anticime1 has joined #openstack-nova20:47
*** jdurgin has joined #openstack-nova20:47
*** fyxim has joined #openstack-nova20:47
*** fmccrthy has joined #openstack-nova20:47
*** zhenguo has joined #openstack-nova20:47
*** dgonzalez has joined #openstack-nova20:47
*** khappone has joined #openstack-nova20:47
*** raorn has joined #openstack-nova20:47
*** sgordon has joined #openstack-nova20:47
efriedjaypipes i.e. should I just do both in one change set?20:47
*** moshele has joined #openstack-nova20:47
*** dave-mccowan has joined #openstack-nova20:47
*** jroll has joined #openstack-nova20:47
*** chyka has joined #openstack-nova20:47
*** hongbin has joined #openstack-nova20:47
*** gyee has joined #openstack-nova20:47
*** jmlowe has joined #openstack-nova20:47
*** Eran_Kuris has joined #openstack-nova20:47
*** xinliang has joined #openstack-nova20:47
*** rocky-shiyan has joined #openstack-nova20:47
*** ekhugen_alt has joined #openstack-nova20:47
*** pprokop has joined #openstack-nova20:47
*** MasterofJOKers has joined #openstack-nova20:47
*** larainema has joined #openstack-nova20:47
*** FL1SK has joined #openstack-nova20:47
*** faridda has joined #openstack-nova20:50
*** moshele has quit IRC20:51
*** itlinux has joined #openstack-nova20:53
*** harlowja has quit IRC21:00
*** liverpooler has quit IRC21:02
*** harlowja has joined #openstack-nova21:03
mriedemildikov: ok, final comments in https://review.openstack.org/#/c/330285/17321:03
mriedemildikov: easy things to address and then i'm ready to go on that21:03
ildikovmriedem: ok21:04
mriedemwe'll have to decide if we hold it for the shared_targets microversion in cinder21:04
*** smatzek has quit IRC21:04
ildikovmriedem: let me ask Jay about the client21:04
*** smatzek has joined #openstack-nova21:04
*** smatzek has quit IRC21:04
ildikovmriedem: as for the new attach patch, did you mean I should take over on those?21:05
mriedemildikov: i was thinking, in the api change to support multiattach, if/when we check the volume to see if it's multiattach=True, couldn't we just check the cinder microversion at that point if the volume already has >0 attachments?21:05
ildikovmriedem: I haven't touched the API patch in almost years now21:05
mriedembasically, nova api can't attach a multiattach volume to >1 instance until we are sure we can get shared_targets info21:05
ildikovmriedem: so I will take a look shortly and figure out21:05
mriedemok21:05
ildikovI didn't want to keep those checks, those are from the ancient times21:06
mriedemi can update https://review.openstack.org/#/c/330285/173 quick21:06
mriedemand then i'm done with it i think21:06
ildikovanother question is whether it's ok to do the lock for detach in a separate patch?21:06
ildikovso we could land the libvirt change and then the lock and then the API21:06
mriedemif it's a separate patch, it has to come before we start multiattaching things21:06
mriedemoh,21:06
ildikovalso, we will bump a microversion for this, right?21:06
mriedemas long as we don't turn on multiattach until the parts in the compute are ready21:07
mriedemi think we can do that21:07
mriedemi think we will bump a microversion as an indication of multiattach yes21:07
*** dtruong has joined #openstack-nova21:07
mriedemi thought we updated the spec about that21:07
ildikovok, so we will need an API level check whether the volume is attached already or not independently from Cinder21:07
mriedemoh it's a todo https://specs.openstack.org/openstack/nova-specs/specs/queens/approved/cinder-volume-multi-attach.html#rest-api-impact21:07
mriedem"Whether or not a new compute API microversion is needed will be determined during implementation and code review. API users will need some way to discover if they can perform volume multiattach and a microversion might be the signal, but it is unclear if Nova would block those requests on a lower microversion, e.g. 2.1. It probably makes sense to do a microversion like 2.49 for tagged attach capabilities."21:08
*** salv-orlando has joined #openstack-nova21:08
ildikovyeah, we were a bit hesitant21:08
ildikovand it'll kinda require another check_attach too as basically that's what turns multi-attach on on the Nova side21:09
ildikovevery other check is in CInder21:09
ildikovthe libvirt change is supposed to be harmless as it plays with things only for multi-attach volumes on intention, but it doesn't do any attaching related stuff21:10
*** yamahata has joined #openstack-nova21:10
*** inara has quit IRC21:10
*** sree has joined #openstack-nova21:10
ildikovjust turns on the shareable flag and adds the capabilities info21:10
*** inara has joined #openstack-nova21:10
*** itlinux has quit IRC21:13
*** sree has quit IRC21:14
*** salv-orlando has quit IRC21:16
*** salv-orlando has joined #openstack-nova21:17
*** salv-orlando has quit IRC21:21
*** rmart04 has quit IRC21:22
*** edand has quit IRC21:22
ildikovmriedem: so will you upload any more versions, or it's on me now? I got lost on that one.21:25
*** itlinux has joined #openstack-nova21:26
*** yamahata has quit IRC21:27
mriedemi'm doing it21:28
*** yamamoto has joined #openstack-nova21:28
ildikovmriedem: cool, tnx21:29
ildikovmriedem: replied to your comments21:29
*** liusheng has joined #openstack-nova21:30
*** pchavva has quit IRC21:30
ildikovmriedem: as we're adding a new API microversion I think we can land this and deal with the new Cinder microversion in the multi-attach change21:30
*** yikun has joined #openstack-nova21:31
ildikovmriedem: as we will need to do some extra checks for that anyway therefore we might as well add the new Cinder microversion to that one21:31
mriedemi tend to agree,21:31
ildikovmriedem: we can do a version check now independently from the attachment_create call so that's not an issue either21:31
mriedemespecially since the point of microversions is that the client can opt into whatever they want when it's available (or not)21:31
mriedemand in this case, nova is the client21:31
*** harlowja has quit IRC21:32
mriedemso we just won't support a multiattach volume to >1 instance if the new cinder microversion for shared_targets isn't htere21:32
mriedembecause we won't be able to safely detach21:32
ildikovyeah, true, I just wanted to avoid the check_attach-like checks as much as possible, but the API microversion in Nova will require that anyway21:32
*** tidwellr has quit IRC21:32
ildikovyep21:32
mriedemwell, the compute microversoin is mainly just a signal21:33
*** tidwellr has joined #openstack-nova21:33
ildikovI mean the Nova API microversion21:33
mriedem'i want to attach this multiattach volume to >1 instance and i'm opting into that, but i know it might not work too'21:33
*** yamamoto has quit IRC21:33
mriedemmight not work b/c (1) old cinder (2) old computes (3) policy (4) virt driver doesn't support it (5) volume type doens't support it, etc21:34
mriedem(6) uranus is not aligned with pluto21:34
ildikovyeah, if Cinder is mid-old then they will not be able to create a multi-attach volume on the first place21:34
ildikovlol21:34
ildikovI hope I21:34
ildikov'll never need to debug (6) :)21:35
ildikovanyway, so if we don't want to hold the new attach patch now we can just check whether the volume is already attached and continue if the Cinder microversion is high enough21:36
*** gouthamr has quit IRC21:36
*** abalutoiu has joined #openstack-nova21:36
ildikovhowever the new flow currently doesn't allow multi-attach in general so we might not need that check21:38
*** sdague has joined #openstack-nova21:38
*** AlexeyAbashkin has joined #openstack-nova21:38
*** openstackgerrit has quit IRC21:38
*** _pewp_ has quit IRC21:38
*** tlbr has quit IRC21:38
*** SamYaple has quit IRC21:38
*** lyarwood has quit IRC21:38
*** gryf has quit IRC21:38
*** ericyoung has quit IRC21:38
*** lifeless has quit IRC21:38
*** mfisch has quit IRC21:38
*** afazekas has quit IRC21:38
*** diegows has quit IRC21:38
*** antonym has quit IRC21:38
*** david-lyle has joined #openstack-nova21:39
ildikovI need to think about it, but the point is still that I think we all had enough of this patch by now...21:39
mriedemincoming21:41
mriedemor not21:41
mriedemanyway, new patches are up21:41
mriedemhttps://review.openstack.org/#/c/330285/174/releasenotes/notes/bp-cinder-new-attach-apis-eca854e27a255e3e.yaml is the reno i added21:41
ildikovmriedem: looks good to me21:43
ildikovnot that I would've worded anything so far good enough to judge, but still :)21:43
*** liverpooler has joined #openstack-nova21:44
mriedem+2 on the new attach flow patch21:45
mriedemi think johnthetubaguy is on paternity leave21:45
mriedemso we'll likely need gibi and/or melwitt to go through this21:45
ildikovmriedem: wow, we haven't even reach 200... :)21:46
edleafeso it looks like I need to rebase my series. Will that kill the patches that are in the gate?21:46
mriedemedleafe: i can re-approve21:46
ildikovmriedem: gibi is familiar with the patch, but he might not familiar with the whole Cinder-side new flow to +A this21:47
*** david-lyle has quit IRC21:47
mriedemildikov: i don't think anyone from nova besides john will be21:47
ildikovmriedem: can check with him tomorrow or look at melwitt with cat eyes from Shrek :)21:47
ildikovmriedem: I know, I'm here if anyone wants to ask questions21:48
mriedemat this point i think i can answer any questions on the nova patches21:48
edleafemriedem: sure, but won't that mean restarting the whole CI process? They are mostly through the gate now.21:48
mriedemedleafe: yes if they are rebased21:48
*** SamYaple has joined #openstack-nova21:49
edleafeok, then I'll wait21:49
ildikovmriedem: if I could have just 1/10 of your brain :)21:50
*** gszasz has quit IRC21:54
*** smatzek has joined #openstack-nova21:55
melwittildikov: if gibi is already familiar with the patch, check with him first. if he won't be able to look, let me know and I'll look at it21:55
*** damien_r has quit IRC21:55
mriedemmelwitt: the bottom 2 patches in the series are non-destructive.21:55
mriedemi.e. they lay some groundwork but aren't run through until the 3rd in the series21:56
mriedemturns it all on21:56
mriedemthe first patch is a straight up simple refactor, no functional change21:56
melwittah, okay21:56
ildikovmelwitt: I will check with him tomorrow on the Implement new attach Cinder flow patch21:56
ildikovmelwitt: if you can look into the two under it that mriedem mentioned that would be pretty great21:57
melwittk, can do21:57
*** itlinux has quit IRC21:59
melwittgrrr something weird is going on in the api sample tests I'm trying to do for noop quotas. when I set the noop driver in a derived test class, it seems to leak into the parent class tests where I didn't set it21:59
*** smatzek has quit IRC21:59
ildikovmelwitt: I had that with a mocked value earlier...22:01
*** openstackgerrit has joined #openstack-nova22:01
*** _pewp_ has joined #openstack-nova22:01
*** tlbr has joined #openstack-nova22:01
*** lyarwood has joined #openstack-nova22:01
*** gryf has joined #openstack-nova22:01
*** ericyoung has joined #openstack-nova22:01
*** lifeless has joined #openstack-nova22:01
*** mfisch has joined #openstack-nova22:01
*** afazekas has joined #openstack-nova22:01
*** diegows has joined #openstack-nova22:01
*** antonym has joined #openstack-nova22:01
*** rcernin has joined #openstack-nova22:01
ildikovnot 100% sure on the reason still, setting up the mock in a different way helped there22:02
*** rcernin has quit IRC22:03
*** rcernin has joined #openstack-nova22:03
*** smatzek has joined #openstack-nova22:08
*** moshele has joined #openstack-nova22:10
*** takashin has joined #openstack-nova22:11
*** smatzek has quit IRC22:13
*** BryanS68 has quit IRC22:24
*** yassine has joined #openstack-nova22:26
*** salv-orlando has joined #openstack-nova22:27
*** yassine is now known as Guest9919622:27
*** gouthamr has joined #openstack-nova22:28
*** yamamoto has joined #openstack-nova22:30
*** salv-orlando has quit IRC22:32
_ixHello again. I'm having some trouble creating a larger node on my new pike deployment.22:35
*** yamamoto has quit IRC22:35
_ixAny tips on where I ought to look to get around this dreadful 'No valid host was found. There are not enough hosts available' error?22:35
*** harlowja has joined #openstack-nova22:36
_ixIt's a 10vcpu 112gb 20gb root and 3000gb machine. I've definitely got the resources.22:36
efried_ix The compute log is where I would start.22:36
*** tidwellr has quit IRC22:36
_ixThe request doesn't seem to be making it to the compute log.22:37
efriedokay, have you checked the sched or conductor logs?22:37
melwittin that case, I suggest turning on log level debug on the scheduler node, restart scheduler, try it again and check the scheduler log to see which filter is blocking the request out22:37
*** lyan has quit IRC22:38
_ixThanks. I'll give that a shot.22:38
*** AlexeyAbashkin has quit IRC22:43
*** Tom-Tom has quit IRC22:43
_ixI'm trying to force the build on a single host... but it says it can't find it.22:48
openstackgerritEric Fried proposed openstack/nova master: Traits ops on ProviderTree  https://review.openstack.org/52160522:49
openstackgerritEric Fried proposed openstack/nova master: Move aggregates from report client to ProviderTree  https://review.openstack.org/52168522:49
openstackgerritEric Fried proposed openstack/nova master: Track provider traits in report client  https://review.openstack.org/52168622:49
openstackgerritEric Fried proposed openstack/nova master: Aggregate ops on ProviderTree  https://review.openstack.org/52621622:49
_ixI'll keep plugging away at this. Thanks again for the suggestions.22:52
*** gouthamr has quit IRC22:53
*** faridda has quit IRC22:55
*** gouthamr has joined #openstack-nova22:56
*** salv-orlando has joined #openstack-nova22:57
melwitt_ix: how are you forcing to a host? like this? https://ask.openstack.org/en/question/47877/how-to-launch-an-instance-on-specific-host/23:00
_ixJust like that, only with the openstack server create instead.23:01
_ixI found my issue. It's me. It's always me.23:01
mriedemyikun: you might consider splitting the DB API and object changes out of this https://review.openstack.org/#/c/326326/ into a separate patch so the patch with the API changes is not so big23:02
melwitt_ix: okay. I was just gonna say that doing that bypasses the scheduler so the scheduler log might not show much if anything23:02
*** salv-orlando has quit IRC23:05
*** burt has quit IRC23:05
_ixHaven't looked into this just yet... A CPU model name should not be set when a host CPU model is requested23:05
_ixOh well, I've got to detach for a few. Have a pleasant day!23:05
*** salv-orlando has joined #openstack-nova23:05
*** mriedem has quit IRC23:10
*** salv-orlando has quit IRC23:10
*** tbachman has quit IRC23:15
openstackgerritMike Perez proposed openstack/nova master: Replace support matrix ext with common library  https://review.openstack.org/48130423:17
*** mlavalle has quit IRC23:21
*** amodi has quit IRC23:21
*** david-lyle has joined #openstack-nova23:22
*** _ix has quit IRC23:22
*** Sandy619 has quit IRC23:25
*** _ix has joined #openstack-nova23:27
*** openstack has joined #openstack-nova23:29
*** ChanServ sets mode: +o openstack23:29
*** takedakn has joined #openstack-nova23:30
*** yamamoto has joined #openstack-nova23:32
*** gmann_afk is now known as gmann23:33
*** takedakn has quit IRC23:34
*** moshele has quit IRC23:36
*** yamamoto has quit IRC23:37
*** edmondsw has joined #openstack-nova23:42
*** sdague has quit IRC23:42
*** tbachman has joined #openstack-nova23:44
*** david-lyle has quit IRC23:44
*** armax has quit IRC23:45
*** salv-orlando has joined #openstack-nova23:46
*** Tom-Tom has joined #openstack-nova23:47
*** tbachman_ has joined #openstack-nova23:48
*** tbachman has quit IRC23:49
*** tbachman_ is now known as tbachman23:49
*** salv-orlando has quit IRC23:50
*** salv-orlando has joined #openstack-nova23:51
*** salv-orlando has quit IRC23:55
*** david-lyle has joined #openstack-nova23:57
*** esberglu has quit IRC23:57
*** salv-orlando has joined #openstack-nova23:57

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