Wednesday, 2016-03-02

*** yamahata has joined #openstack-nova00:00
*** unicell has quit IRC00:00
*** haomaiwang has quit IRC00:01
*** haomaiwang has joined #openstack-nova00:01
*** ijw has quit IRC00:04
*** unicell has joined #openstack-nova00:04
*** MVenesio has joined #openstack-nova00:08
*** zenoway has joined #openstack-nova00:08
*** yamahata has quit IRC00:11
*** unicell has quit IRC00:12
jaypipesdansmith, bauzas: meh, sorry, the statistics compute node thing took a little longer than I expected. running a final check of unit tests now...00:13
*** zenoway has quit IRC00:13
mriedemtonyb: this should get us around the kilo fixtures issue for now https://review.openstack.org/#/c/286863/00:14
*** MVenesio has quit IRC00:14
*** josecastroleon has joined #openstack-nova00:15
*** mylu has quit IRC00:17
*** zz_dimtruck is now known as dimtruck00:18
openstackgerritClaudiu Belu proposed openstack/nova: db: Adds host_capabilities column in compute_nodes  https://review.openstack.org/22827900:19
*** mylu has joined #openstack-nova00:19
*** unicell has joined #openstack-nova00:21
*** ctrath has quit IRC00:23
tonybmriedem: okay.00:24
*** mylu has quit IRC00:25
*** Sukhdev has quit IRC00:25
*** thorst_ has joined #openstack-nova00:30
*** mylu has joined #openstack-nova00:32
*** ducttape_ has joined #openstack-nova00:32
*** erlon has quit IRC00:36
*** mylu has quit IRC00:37
*** thorst_ has quit IRC00:38
*** thorst_ has joined #openstack-nova00:38
*** derekh has quit IRC00:38
mriedemcarl_baldwin: you might want to hold up on https://review.openstack.org/#/c/260738/ until the tempest change that's testing it is done00:40
*** thorst_ has quit IRC00:40
mriedemhttps://review.openstack.org/#/c/286855/00:41
*** thorst_ has joined #openstack-nova00:41
*** help has joined #openstack-nova00:41
mriedemand failed http://logs.openstack.org/55/286855/1/experimental/gate-tempest-dsvm-neutron-dvr-multinode-full/8c4b90c/00:41
*** mylu has joined #openstack-nova00:41
carl_baldwinmriedem: ack00:41
*** help is now known as Guest5279200:41
mriedemhmm, well, failure in the test runner http://logs.openstack.org/55/286855/1/experimental/gate-tempest-dsvm-neutron-dvr-multinode-full/8c4b90c/console.html#_2016-03-02_00_17_36_78000:42
Guest52792Hi men. I'm look00:42
Guest52792I'm looking for the good way to enable keystone middleware authentication.00:43
Guest52792can you please help me?00:43
mriedemGuest52792: see the channel topic, you should try the #openstack channel00:44
*** zenoway has joined #openstack-nova00:44
*** josecastroleon has quit IRC00:45
*** dimtruck is now known as zz_dimtruck00:46
*** mylu has quit IRC00:47
*** mylu has joined #openstack-nova00:47
*** bjornar has quit IRC00:48
*** Swami has quit IRC00:48
*** zenoway has quit IRC00:49
*** ducttape_ has quit IRC00:55
*** ijw has joined #openstack-nova00:56
Guest52792thanks you mriedem01:00
Guest52792regards01:00
*** haomaiwang has quit IRC01:01
*** haomaiwang has joined #openstack-nova01:01
*** zz_dimtruck is now known as dimtruck01:02
*** irenab has quit IRC01:03
*** irenab_ has joined #openstack-nova01:03
*** achanda has quit IRC01:03
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: don't attempt to get baseline cpu features if host cpu model is None  https://review.openstack.org/28686801:06
*** zenoway has joined #openstack-nova01:08
*** EinstCrazy has joined #openstack-nova01:09
*** josecastroleon has joined #openstack-nova01:10
*** zenoway has quit IRC01:12
*** ssurana has quit IRC01:18
*** mylu has quit IRC01:19
*** ducttape_ has joined #openstack-nova01:20
*** ducttape_ has joined #openstack-nova01:21
*** Kevin_Zheng has quit IRC01:22
*** Kevin_Zheng has joined #openstack-nova01:22
*** Sukhdev has joined #openstack-nova01:23
*** irenab_ has quit IRC01:28
mriedembauzas: can this be abandoned? https://review.openstack.org/#/c/221384/01:30
*** irenab has joined #openstack-nova01:31
*** rk4n has quit IRC01:34
*** Daisy has joined #openstack-nova01:38
*** zengine has joined #openstack-nova01:39
*** rwmjones has quit IRC01:40
*** irenab has quit IRC01:41
*** irenab has joined #openstack-nova01:42
*** josecastroleon has quit IRC01:42
*** haomaiwang has quit IRC01:42
*** fawadkhaliq has quit IRC01:42
*** rk4n has joined #openstack-nova01:43
*** penick has quit IRC01:43
*** rwmjones has joined #openstack-nova01:44
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Add changes-since support when list servers  https://review.openstack.org/28642301:45
*** apoorvad has quit IRC01:48
*** ducttape_ has quit IRC01:49
*** zenoway has joined #openstack-nova01:50
*** gongysh has joined #openstack-nova01:51
*** achanda has joined #openstack-nova01:51
*** mtanino has quit IRC01:51
mriedemmgagne: are you going to update this? https://review.openstack.org/#/c/227440/01:53
mgagnemriedem: probably not due to feature freeze in cells v101:53
mriedemis it a feature or a bug fix?01:53
mgagneunless you tell me there is a chance this will get merged01:53
openstackgerritJay Pipes proposed openstack/nova: Make compute_node_statistics() use new schema  https://review.openstack.org/28693101:53
openstackgerrityuntongjin proposed openstack/nova: Add members in InstanceGroup object members field  https://review.openstack.org/28644301:54
*** jaypipes has quit IRC01:54
mriedemmgagne: well i guess it depends on if it's a bug or not01:54
mgagnemriedem: we already had that discussion and I feel like people see most changes as features and not bug fixes01:54
mriedemyou never provided that info01:54
*** Daisy has quit IRC01:54
mgagnemriedem: for me: lack of feature = bug01:54
mgagnefeature parity*01:54
*** zenoway has quit IRC01:54
*** ducttape_ has joined #openstack-nova01:54
*** mingyu has joined #openstack-nova01:55
*** Daisy has joined #openstack-nova01:55
mriedemso this 5 line code change makes evacuate work with cells v1?01:55
mgagneI just don't have time and energy to deal with those sort of discussions unless you tell me there is a 100% for this to get merged without fuzz and debate01:55
mgagnemriedem: yes01:55
mgagnemriedem: we used it01:55
*** Daisy has quit IRC01:55
mgagnemriedem: I'm not sure with latest code01:55
mgagnewe run kilo01:55
*** mingyu has quit IRC01:55
*** Daisy has joined #openstack-nova01:55
mriedemi'd say file a bug with details and comment that in the patch, i can have alaski take a look01:56
*** sheel has joined #openstack-nova01:56
mgagnealright, will try to replicate issue on my side to get more details01:56
mriedemit seems like a regression fix more than anything01:56
*** thorst_ has quit IRC01:56
*** thorst_ has joined #openstack-nova01:57
*** Sukhdev has quit IRC01:58
*** ducttape_ has quit IRC01:58
*** ducttape_ has joined #openstack-nova02:00
*** gokrokve has quit IRC02:01
lifelessmriedem: what version do you want tagged again?02:01
mriedemlifeless: 1.2.102:02
*** irenab has quit IRC02:02
mriedemlifeless: b/c https://github.com/openstack/requirements/blob/stable/kilo/global-requirements.txt#L21702:02
lifelessmriedem: and a dep on testtools <2.0.002:03
mriedemyeah02:03
mriedemsince testtools 2.0.0 requires fixtures>=1.3.002:03
*** irenab has joined #openstack-nova02:03
mriedemhttps://github.com/testing-cabal/testtools/blob/2.0.0/requirements.txt#L302:04
*** thorst_ has quit IRC02:05
lifelessmriedem: tonyb: done02:07
mriedemlifeless: thanks02:07
*** rk4n has quit IRC02:07
lifelesssorry it took so long02:08
*** rfolco_ has joined #openstack-nova02:08
*** rfolco_ has quit IRC02:08
*** MVenesio has joined #openstack-nova02:09
mriedemnp. did you create a local branch for that or something?02:09
mriedemi don't see the tag in github02:09
lifelessits up there now, you raced :)02:09
*** nic has quit IRC02:10
mriedemhttps://github.com/testing-cabal/fixtures/commit/255f9cd00c9bc4dfd90cf9073e6c935d3009e16302:10
mriedemnice, thanks again02:10
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Add changes-since support when list servers  https://review.openstack.org/28642302:10
*** ducttape_ has quit IRC02:11
*** Daisy has quit IRC02:12
*** Guest52792 has quit IRC02:12
*** Daisy has joined #openstack-nova02:12
*** MVenesio has quit IRC02:15
*** Daisy has quit IRC02:17
*** irenab has quit IRC02:19
*** irenab has joined #openstack-nova02:19
*** haomaiwang has joined #openstack-nova02:21
*** ducttape_ has joined #openstack-nova02:22
mgagnemriedem: hard to explain in a bug, the symptom are: instance stays in "rebuilding" task state forever02:22
mgagneopened bug #155204602:25
openstackbug 1552046 in OpenStack Compute (nova) "Cannot evacuate instances with cells v1" [Undecided,New] https://launchpad.net/bugs/155204602:25
mgagneI hope it will be enough, as for the explanation in the commit message, I'm not sure how to word it :-/02:25
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Use novaclient/utils directly and remove openstack/common  https://review.openstack.org/28644202:29
*** gokrokve has joined #openstack-nova02:29
*** ducttape_ has quit IRC02:32
*** josecastroleon has joined #openstack-nova02:33
*** gokrokve has quit IRC02:34
*** ducttape_ has joined #openstack-nova02:35
*** vilobhmm11 has quit IRC02:36
*** diana_clarke has quit IRC02:37
*** diana_clarke has joined #openstack-nova02:38
*** Daisy has joined #openstack-nova02:40
*** shinobu_ has joined #openstack-nova02:40
*** shinobu_ is now known as Guest6486402:41
*** suresh12 has quit IRC02:41
Guest64864can anyone tell me what bdm stands for? sorry for pretty much easy question -;02:41
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Use novaclient/utils directly and remove openstack/common  https://review.openstack.org/28644202:42
*** Daisy has quit IRC02:42
*** Daisy has joined #openstack-nova02:42
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Use novaclient/utils directly and remove openstack/common  https://review.openstack.org/28644202:43
*** unicell has quit IRC02:44
openstackgerritBrian Elliott proposed openstack/nova: Manually set changed fields for cell healing  https://review.openstack.org/28694002:45
*** gokrokve has joined #openstack-nova02:45
*** ducttape_ has quit IRC02:45
*** mriedem has quit IRC02:46
*** Daisy has quit IRC02:47
*** penick has joined #openstack-nova02:47
*** dimtruck is now known as zz_dimtruck02:48
openstackgerritBrian Elliott proposed openstack/nova: Manually set changed fields for cell healing  https://review.openstack.org/28694002:48
*** houming has joined #openstack-nova02:53
*** mingyu has joined #openstack-nova02:56
*** haomaiwang has quit IRC03:01
*** mingyu has quit IRC03:01
*** haomaiwa_ has joined #openstack-nova03:01
*** josecastroleon has quit IRC03:02
openstackgerritMathieu Gagné proposed openstack/nova: Fix evacuate support with Nova cells v1  https://review.openstack.org/22744003:04
*** thorst_ has joined #openstack-nova03:04
mgagneGuest64864: block device mapping03:04
Guest64864mgagne: thank you -;03:06
*** sudipto has joined #openstack-nova03:08
*** zz_dimtruck is now known as dimtruck03:08
*** hemna_ has joined #openstack-nova03:09
*** thorst_ has quit IRC03:10
*** suresh12 has joined #openstack-nova03:12
*** gokrokve has quit IRC03:12
*** diana_clarke has quit IRC03:12
*** dims has quit IRC03:15
*** gyee has quit IRC03:16
*** dimtruck is now known as zz_dimtruck03:18
*** gokrokve has joined #openstack-nova03:23
*** ijw_ has joined #openstack-nova03:24
*** ijw has quit IRC03:24
*** david-lyle has joined #openstack-nova03:24
*** hemna_ has quit IRC03:24
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Use novaclient/utils directly and remove openstack/common  https://review.openstack.org/28695503:25
*** penick has quit IRC03:27
*** gokrokve has quit IRC03:27
*** penick has joined #openstack-nova03:28
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Use novaclient/utils directly and remove openstack/common  https://review.openstack.org/28695803:29
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Use novaclient/utils directly and remove openstack/common  https://review.openstack.org/28695903:32
*** zz_dimtruck is now known as dimtruck03:33
*** sudipto has quit IRC03:39
*** kaisers has quit IRC03:42
*** fawadkhaliq has joined #openstack-nova03:43
*** kaisers has joined #openstack-nova03:44
*** haomaiw__ has joined #openstack-nova03:44
*** haukebruno has quit IRC03:44
*** rosmaita is now known as brianr-gone03:44
*** haomaiwa_ has quit IRC03:45
*** haukebruno has joined #openstack-nova03:45
*** dimtruck is now known as zz_dimtruck03:47
*** gokrokve has joined #openstack-nova03:51
*** gokrokve has quit IRC03:55
*** haomaiw__ has quit IRC04:01
*** haomaiwang has joined #openstack-nova04:01
*** zz_dimtruck is now known as dimtruck04:01
*** thorst_ has joined #openstack-nova04:08
openstackgerritShiina, Hironori proposed openstack/nova: Abstract a driver API for triggering crash dump  https://review.openstack.org/28221404:09
*** MVenesio has joined #openstack-nova04:10
*** gokrokve has joined #openstack-nova04:15
*** MVenesio has quit IRC04:15
*** thorst_ has quit IRC04:16
*** zengine has quit IRC04:18
*** zengine has joined #openstack-nova04:19
tonyblifeless: thanks!04:23
*** zengine has quit IRC04:24
*** ducttape_ has joined #openstack-nova04:25
*** Jeffrey4l has joined #openstack-nova04:28
*** ianw has joined #openstack-nova04:29
*** haomaiwang has quit IRC04:31
*** ijw_ has quit IRC04:31
*** ducttape_ has quit IRC04:31
*** sridharg has joined #openstack-nova04:31
*** ijw has joined #openstack-nova04:32
*** phuongnh has joined #openstack-nova04:33
*** haomaiwang has joined #openstack-nova04:33
*** haomaiw__ has joined #openstack-nova04:38
*** haomaiwang has quit IRC04:39
*** oomichi_ has joined #openstack-nova04:40
*** yamahata has joined #openstack-nova04:41
*** ishant has joined #openstack-nova04:41
*** efried has quit IRC04:44
*** haomaiw__ has quit IRC04:44
*** gokrokve_ has joined #openstack-nova04:44
*** haomaiwa_ has joined #openstack-nova04:46
*** gokrokve has quit IRC04:47
*** ijw has quit IRC04:49
*** links has joined #openstack-nova04:51
*** Jeffrey4l has quit IRC04:55
*** mingyu has joined #openstack-nova04:56
*** Sukhdev has joined #openstack-nova04:57
*** fawadkhaliq has quit IRC04:58
*** haomaiwa_ has quit IRC05:01
*** mingyu has quit IRC05:01
*** haomaiwa_ has joined #openstack-nova05:01
*** dimtruck is now known as zz_dimtruck05:02
*** efried has joined #openstack-nova05:03
*** sudipto has joined #openstack-nova05:07
*** vilobhmm11 has joined #openstack-nova05:08
*** RuiChen has joined #openstack-nova05:11
*** zenoway has joined #openstack-nova05:12
*** thorst_ has joined #openstack-nova05:14
*** _RuiChen has quit IRC05:15
*** zenoway has quit IRC05:16
*** amotoki has joined #openstack-nova05:16
*** salv-orl_ has joined #openstack-nova05:19
*** thorst_ has quit IRC05:21
*** salv-orlando has quit IRC05:22
*** y_sawai has joined #openstack-nova05:25
*** ducttape_ has joined #openstack-nova05:27
*** harlowja_at_home has joined #openstack-nova05:29
*** ducttape_ has quit IRC05:32
*** sayalilunkad has quit IRC05:33
*** achanda has quit IRC05:35
*** pcaruana has joined #openstack-nova05:36
*** achanda has joined #openstack-nova05:39
*** achanda has quit IRC05:39
*** sayalilunkad has joined #openstack-nova05:45
*** pcaruana has quit IRC05:49
*** gokrokve has joined #openstack-nova05:50
*** rcernin has joined #openstack-nova05:51
*** dave-mccowan has quit IRC05:51
*** gokrokve_ has quit IRC05:53
*** gokrokve_ has joined #openstack-nova05:53
*** zz_dimtruck is now known as dimtruck05:53
*** gokrokve has quit IRC05:55
*** gokrokve has joined #openstack-nova05:56
*** gokrokve_ has quit IRC05:57
*** Sukhdev has quit IRC05:57
*** Sukhdev has joined #openstack-nova05:58
*** achanda has joined #openstack-nova06:00
*** haomaiwa_ has quit IRC06:01
*** hoangcx has joined #openstack-nova06:01
*** esracelik has joined #openstack-nova06:01
*** nkrinner has joined #openstack-nova06:01
*** haomaiwang has joined #openstack-nova06:01
*** jdurgin1 has joined #openstack-nova06:01
*** sudipto has quit IRC06:03
*** jaosorior has joined #openstack-nova06:06
*** lbeliveau_ has quit IRC06:06
*** suresh12 has quit IRC06:07
openstackgerritzwei proposed openstack/nova: Nova generate nova.conf.sample configuration file is incomplete     eg: Not Found option #iscsi_use_multipath in [ libvirt ]     eg: iser_volume_opts in [ libvirt ]  https://review.openstack.org/27664906:08
*** lbeliveau has joined #openstack-nova06:08
*** shaohe_robot has joined #openstack-nova06:09
*** MVenesio has joined #openstack-nova06:10
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Use novaclient/utils directly and remove openstack/common (1/4)  https://review.openstack.org/28644206:13
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Use novaclient/utils directly and remove openstack/common (2/4)  https://review.openstack.org/28695506:14
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Use novaclient/utils directly and remove openstack/common (3/4)  https://review.openstack.org/28695806:15
openstackgerritgaryk proposed openstack/nova: VMware: make the opaque network attachment more robust  https://review.openstack.org/28412106:16
*** MVenesio has quit IRC06:16
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Use novaclient/utils directly and remove openstack/common (4/4)  https://review.openstack.org/28695906:16
*** dimtruck is now known as zz_dimtruck06:17
*** sudipto has joined #openstack-nova06:18
*** thorst_ has joined #openstack-nova06:19
*** shaohe_feng_ has joined #openstack-nova06:21
openstackgerritEli Qiao proposed openstack/python-novaclient: [microversion] Bump to 2.25  https://review.openstack.org/26038306:21
*** shaohe_feng_ has quit IRC06:21
*** shaohe_feng_ has joined #openstack-nova06:22
*** gongysh has quit IRC06:24
*** gongysh has joined #openstack-nova06:26
*** thorst_ has quit IRC06:26
*** ducttape_ has joined #openstack-nova06:28
*** sudipto has quit IRC06:32
*** ducttape_ has quit IRC06:32
*** ekuris_ has joined #openstack-nova06:32
*** amotoki has quit IRC06:33
*** amotoki has joined #openstack-nova06:34
*** sridharg has quit IRC06:34
*** armax has quit IRC06:35
*** amotoki has quit IRC06:36
*** krtaylor has quit IRC06:37
*** jamielennox is now known as jamielennox|away06:38
*** zenoway has joined #openstack-nova06:39
*** suresh12 has joined #openstack-nova06:42
*** hoangcx has quit IRC06:44
*** shaohe_robot has left #openstack-nova06:44
*** paul-carlton2 has joined #openstack-nova06:45
*** jamielennox|away is now known as jamielennox06:49
*** krtaylor has joined #openstack-nova06:49
*** sudipto has joined #openstack-nova06:49
sheelHello All06:52
sheelI have one small query06:52
sheelcould anyone provide some information about metadata agent used during instance creation06:53
*** suresh12 has quit IRC06:53
sheelbasicall what is the use of metadata06:53
openstackgerritJianghua Wang proposed openstack/nova: XenAPI: introduce unit test for XenAPI plugins  https://review.openstack.org/28396206:54
*** gokrokve has quit IRC06:56
haukebrunosheel, I guess the main use-case for metadata service is to set up the hostname of an instance properly + inject the ssh key information06:57
sheelhaukebruno: hey thanks06:58
sheelhaukebruno: what does inject ssh keys mean here?06:58
sheelhaukebruno: is it for just accessiong vm by inserting ssh keys06:59
sheel?06:59
sheellike we do for password less usage in our day  to day activities06:59
sheel?06:59
haukebrunoyes, via dashboard/horizon the user creates (or imports) its personal keypair and the metadata service injects it while booting the instance07:00
sheelhaukebruno: oh great07:00
*** zenoway has quit IRC07:00
haukebrunoin the normal cloud images password login is not permitted07:00
sheelhaukebruno: this will surely help me07:00
*** haomaiwang has quit IRC07:01
*** haomaiwang has joined #openstack-nova07:01
sheel could find below as well as Instance metadata contents07:01
sheelinstance-id07:01
sheelinstance-type07:01
sheelkernel-id07:01
sheellocal-hostname07:01
sheelpublic-hostname07:01
sheelramdisk-id07:01
*** hoangcx has joined #openstack-nova07:02
*** scheuran has joined #openstack-nova07:06
*** zenoway has joined #openstack-nova07:06
*** e0ne has joined #openstack-nova07:06
*** sudipto has quit IRC07:07
*** jamielennox is now known as jamielennox|away07:08
sheelhaukebruno: thank you, I could connect with the concept now07:14
*** amotoki has joined #openstack-nova07:14
*** oomichi_ has quit IRC07:16
*** lpetrut has joined #openstack-nova07:18
*** sudipto has joined #openstack-nova07:22
*** thorst_ has joined #openstack-nova07:23
*** sridharg has joined #openstack-nova07:23
*** ducttape_ has joined #openstack-nova07:28
*** cfcp1 has joined #openstack-nova07:30
*** moshele has joined #openstack-nova07:30
*** thorst_ has quit IRC07:31
*** cfcp1 has quit IRC07:32
*** ducttape_ has quit IRC07:33
*** amotoki has quit IRC07:35
*** vishwanathj is now known as vishwanathj_zzz07:38
*** belmoreira has joined #openstack-nova07:38
openstackgerritmeretiko proposed openstack/python-novaclient: apiclient.base.Resource.get method requires manager.client to have last_request_id member [1]. Otherwise get operation fails with AttributeError exception.  https://review.openstack.org/28702407:39
*** mjura has joined #openstack-nova07:39
*** amotoki has joined #openstack-nova07:43
*** sridhargaddam has joined #openstack-nova07:44
*** liverpooler has joined #openstack-nova07:44
*** haomaiwang has quit IRC07:44
*** sridharg has quit IRC07:44
*** ekuris_ has quit IRC07:46
*** hoangcx has quit IRC07:46
*** amotoki has quit IRC07:48
*** harlowja_at_home has quit IRC07:55
*** markvoelker has quit IRC07:58
*** rmart04 has joined #openstack-nova07:58
*** jichen has joined #openstack-nova07:58
*** e0ne has quit IRC07:59
*** mpavone has joined #openstack-nova08:00
bauzasgood morning Nova08:00
*** PaulMurray has joined #openstack-nova08:00
*** haomaiwang has joined #openstack-nova08:01
*** pcaruana has joined #openstack-nova08:02
*** achanda has quit IRC08:03
*** achanda has joined #openstack-nova08:07
*** Sukhdev has quit IRC08:09
*** Sukhdev has joined #openstack-nova08:10
*** zz_dimtruck is now known as dimtruck08:11
*** MVenesio has joined #openstack-nova08:11
*** ekuris_ has joined #openstack-nova08:12
*** mingyu has joined #openstack-nova08:12
*** Sukhdev has quit IRC08:13
*** matrohon has joined #openstack-nova08:14
*** rdopiera has joined #openstack-nova08:16
*** mingyu has quit IRC08:16
openstackgerritWenzhi Yu proposed openstack/nova: Fix missing of unit in HostState.__repr__()  https://review.openstack.org/28703208:17
*** MVenesio has quit IRC08:17
*** achanda has quit IRC08:18
*** erno has joined #openstack-nova08:21
*** lpetrut has quit IRC08:22
*** andrearosa_web has joined #openstack-nova08:24
*** vilobhmm11 has quit IRC08:24
*** jamielennox|away is now known as jamielennox08:25
*** jlanoux has joined #openstack-nova08:26
*** thorst_ has joined #openstack-nova08:29
*** ducttape_ has joined #openstack-nova08:29
*** lxsli0 has joined #openstack-nova08:31
*** tcammann has joined #openstack-nova08:32
*** ducttape_ has quit IRC08:34
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Use novaclient/utils directly and remove openstack/common  https://review.openstack.org/28695808:35
*** thorst_ has quit IRC08:35
*** Daisy has joined #openstack-nova08:36
*** sudipto has quit IRC08:36
*** lxsli0 has quit IRC08:38
*** tcammann has quit IRC08:39
*** permalac has joined #openstack-nova08:42
*** permalac has quit IRC08:42
*** zhangjn has quit IRC08:43
*** Guest64864 has quit IRC08:43
*** haomai___ has joined #openstack-nova08:44
*** haomai___ has quit IRC08:45
*** haomai___ has joined #openstack-nova08:45
*** haomaiwang has quit IRC08:45
*** haomai___ has quit IRC08:46
*** haomaiw__ has joined #openstack-nova08:46
*** haomaiw__ has quit IRC08:47
*** haomaiwa_ has joined #openstack-nova08:47
openstackgerritAlex Xu proposed openstack/nova: Generate better validation error message when using name regexes  https://review.openstack.org/28114308:47
*** zhangjn has joined #openstack-nova08:48
*** haomaiwa_ has quit IRC08:48
*** haomaiwang has joined #openstack-nova08:48
*** haomaiwang has quit IRC08:49
*** haomaiwa_ has joined #openstack-nova08:49
*** haomaiwa_ has quit IRC08:50
*** haomaiwang has joined #openstack-nova08:50
*** haomaiwang has quit IRC08:51
*** danpb has joined #openstack-nova08:51
*** ihrachys has joined #openstack-nova08:51
tonybbauzas: Morning, how are you today?08:51
*** haomaiwa_ has joined #openstack-nova08:51
*** haomaiwa_ has quit IRC08:52
*** haomaiwa_ has joined #openstack-nova08:52
*** haomaiwa_ has quit IRC08:53
*** haomaiwa_ has joined #openstack-nova08:53
*** hoangcx has joined #openstack-nova08:53
*** haomaiwa_ has quit IRC08:54
*** dimtruck is now known as zz_dimtruck08:54
*** haomaiwa_ has joined #openstack-nova08:54
*** haomaiwa_ has quit IRC08:55
*** haomaiwang has joined #openstack-nova08:55
*** haomaiwang has quit IRC08:56
*** haomaiwa_ has joined #openstack-nova08:56
bauzastonyb: 'evening :)08:57
*** haomaiwa_ has quit IRC08:57
*** speller has joined #openstack-nova08:57
*** haomaiwang has joined #openstack-nova08:57
*** haomaiwang has quit IRC08:58
*** 7GHAAFCRL has joined #openstack-nova08:58
*** markvoelker has joined #openstack-nova08:58
*** 7GHAAFCRL has quit IRC08:59
*** haomaiwang has joined #openstack-nova08:59
*** haomaiwang has quit IRC09:00
*** haomaiwang has joined #openstack-nova09:00
*** permalac has joined #openstack-nova09:00
*** haomaiwang has quit IRC09:01
openstackgerritTakashi NATSUME proposed openstack/nova: Fix string interpolations at logging calls  https://review.openstack.org/28162709:01
*** haomaiwa_ has joined #openstack-nova09:01
*** haomaiwa_ has quit IRC09:02
*** haomaiwa_ has joined #openstack-nova09:02
*** haomaiwa_ has quit IRC09:03
PaulMurraybauzas, tonyb are you two on 24/7 these days?09:03
tonybPaulMurray: far from it09:03
bauzasPaulMurray: heh09:03
*** haomaiwang has joined #openstack-nova09:03
*** vishwanathj_zzz has quit IRC09:03
bauzasPaulMurray: life is too short09:03
PaulMurrayI think bauzas was on when I gave up last night and his on again now09:03
*** y_sawai has quit IRC09:03
*** haomaiwang has quit IRC09:04
tonybPaulMurray: I'm s'posed to be chatting with markus_z but I may have gotten the time wrong09:04
markus_ztonyb: no no, I'm here, I was distracted, sorry09:04
PaulMurraytonyb, its 10am in markus_z land at the mo09:04
*** haomaiwang has joined #openstack-nova09:04
tonybmarkus_z: huzzah09:04
*** gokrokve has joined #openstack-nova09:05
*** haomaiwang has quit IRC09:05
tonybmarkus_z: so did you get a chance to look at the github repo?09:05
*** haomaiwang has joined #openstack-nova09:05
markus_ztonyb: yep, and I have two questions09:05
tonybmarkus_z: is one of them "what on earth are you thinking?"09:06
*** haomaiwang has quit IRC09:06
markus_zthe simple one first, the ":" in the functions is just a bash thing for "true", right?09:06
tonybPaulMurray: nice markus_z has a whole land now :)09:06
tonybmarkus_z: yeah.09:06
*** haomaiwang has joined #openstack-nova09:06
*** lucas-dinner is now known as lucasagomes09:06
* markus_z hereby declares the free nation of markus_z-land. beer is free09:06
*** hoangcx has quit IRC09:06
tonybmarkus_z: it's ':' is built into bash so it's the "quickest" noop there is :)09:07
*** haomaiwang has quit IRC09:07
markus_ztonyb: the second one is about the "cloud-archive:liberty"09:07
tonybmarkus_z: \o/ free beer09:07
bauzasPaulMurray: shht, don't mention my work hours when some French people could argue on that ;-)09:07
tonybmarkus_z: shoot09:07
*** haomaiwang has joined #openstack-nova09:07
*** haomaiwang has quit IRC09:08
markus_ztonyb: A few lines later the code installs one very specific libvirt version09:08
PaulMurraybauzas, don't tell me - if you're working long hours they should all be paid more09:08
PaulMurray:)09:08
markus_ztonyb: and that confused me a little and left it open for me how the updates will work09:08
PaulMurrayjust kidding09:08
*** daemontool_ has joined #openstack-nova09:08
*** haomaiwa_ has joined #openstack-nova09:08
bauzasPaulMurray: you know the current French context ? :)09:08
*** daemontool_ is now known as daemontool09:08
bauzasanyway09:08
*** haomaiwa_ has quit IRC09:09
bauzastonyb: 'sup ?09:09
tonybmarkus_z: That's a very good question09:09
*** haomaiwang has joined #openstack-nova09:09
* tonyb ponders it ....09:09
markus_ztonyb: I somehow had the impression that it would be possible to declare a "install libvirt_$latest"09:09
*** gokrokve has quit IRC09:09
*** haomaiwang has quit IRC09:10
tonybmarkus_z: yes but not in this first itteration09:10
markus_ztonyb: ah, ok, just get it running for the experimental gate job?09:10
PaulMurraybauzas, http://despair.com/products/effort09:10
tonybmarkus_z: Yeah.09:10
*** haomaiwa_ has joined #openstack-nova09:10
*** derekh has joined #openstack-nova09:10
tonybmarkus_z: I wanted to make it explicit what we where using and avoid random changes.09:10
*** haomaiwa_ has quit IRC09:11
*** Daisy has quit IRC09:11
*** daemontool__ has quit IRC09:11
tonybmy idea also is that the current 'libvirt' codebase/plugin I have would just be one and we'd be able to use alternate repos once they exist09:11
markus_ztonyb: yeah, it has its advantages and disadvantages09:11
*** haomaiwang has joined #openstack-nova09:11
*** Daisy has joined #openstack-nova09:11
tonybmarkus_z: I agree09:11
PaulMurraybauzas, I should stop joking about that - it never turns out well09:11
*** haomaiwang has quit IRC09:12
tonybmarkus_z: it was largley to address something I heard durig the discussion at the mid-cycle that uca was too volitile09:12
tonybmarkus_z: I don't recall who said it09:12
bauzasPaulMurray: I'm a grown-up and I'm a remote guy, so I don't except any problem at all09:12
*** haomaiwang has joined #openstack-nova09:12
bauzasexpect*09:12
*** haomaiwang has quit IRC09:13
*** sudipto has joined #openstack-nova09:13
* markus_z looks at the midcycle etherpad09:13
*** haomaiwang has joined #openstack-nova09:13
bauzasPaulMurray: just make sure you don't visit France by March or April, it could become difficult09:13
PaulMurraybauzas, I was really thinking about all the other french people who could read this public forum09:13
*** haomaiwang has quit IRC09:14
bauzaslike I said, I'm not asked to09:14
markus_zno notes on that09:14
bauzasPaulMurray: but I understand your concern and appreciate it09:14
*** haomaiwang has joined #openstack-nova09:14
tonybmarkus_z: :( doesn't surprise me.09:14
*** haomaiwang has quit IRC09:15
tonybmarkus_z: so I guess the next step is to add a job pointing at the github repo09:15
markus_ztonyb: yes, that's my part I guess. I got all the pointers I need.09:15
tonybmarkus_z: If that's cool with infra great! otherwise I can move it into the big tent09:15
*** haomaiwang has joined #openstack-nova09:15
tonybmarkus_z: I don't mind doign it.09:15
*** haomaiwang has quit IRC09:16
tonybmarkus_z: I was lookign at the big tent stuff *first* but I'll try without that and see what happens09:16
*** Daisy has quit IRC09:16
markus_ztonyb: What do you mean with that?09:16
*** haomaiwa_ has joined #openstack-nova09:16
markus_ztonyb: I'd like to do the gate job, I want to learn how this works.09:16
*** haomaiwa_ has quit IRC09:17
markus_ztonyb: But thanks for your offer09:17
tonybmarkus_z: moving from github -> git.o.o/openstack/devstack.....git09:17
*** haomaiwa_ has joined #openstack-nova09:17
tonybmarkus_z: okay, let me know if it all goes to custard becise the plugin isn't in the big tent09:17
markus_ztonyb: We can do that in parallel I guess? First point to github and discuss the move to git.o.o.openstack09:18
*** haomaiwa_ has quit IRC09:18
tonybmarkus_z: Yeah. for sure.09:18
tonybmarkus_z: I lost momentum and focus09:18
*** 16WAAA910 has joined #openstack-nova09:18
johnthetubaguytonyb: did you hear from angus if we got any further with privsep?09:18
*** 16WAAA910 has quit IRC09:19
*** haomaiwa_ has joined #openstack-nova09:19
markus_ztonyb: yeah, momentum and focus get lost extremely easily09:19
*** jistr has joined #openstack-nova09:19
tonybjohnthetubaguy: I think he's wiating on you and smcginnis to decide if it's a priority enough to get an FFE09:19
tonybjohnthetubaguy: is the upgrade impact worth the risk etc09:19
markus_ztonyb: I work on the gate job on Friday and discuss with infra folks if the github repo is a no-go-09:20
*** haomaiwa_ has quit IRC09:20
tonybmarkus_z: okay cool.09:20
tonybmarkus_z: I can rough out the yaml for you to use as as base if you like09:20
*** 16WAAA93P has joined #openstack-nova09:20
markus_ztonyb: only if it doesn't cost you a big amount of time09:20
*** 16WAAA93P has quit IRC09:21
tonybmarkus_z: Nah that's pretty easy09:21
*** gongysh has quit IRC09:21
markus_ztonyb: That would be nice, thanks a lot.09:21
*** haomaiwa_ has joined #openstack-nova09:21
*** haomaiwa_ has quit IRC09:22
*** BobBall has joined #openstack-nova09:22
markus_ztonyb: I would come back to you on Monday on this?09:22
*** haomaiwa_ has joined #openstack-nova09:22
*** haomaiwa_ has quit IRC09:23
tonybmarkus_z: Hmmm I'm traveling Mon->Wed my time so IRC will be hard but email woudl be fine.09:23
*** haomaiwang has joined #openstack-nova09:23
*** haomaiwang has quit IRC09:24
markus_ztonyb: Sure, email is good. Maybe I make an etherpad with todos and done and stuff.09:24
tonybmarkus_z: +109:24
*** haomaiwang has joined #openstack-nova09:24
markus_ztonyb: Do you have any open questions for me about this?09:24
*** haomaiwang has quit IRC09:25
openstackgerritEli Qiao proposed openstack/python-novaclient: [microversion] Bump to 2.25  https://review.openstack.org/26038309:25
tonybmarkus_z: DO you recall what version of libvirt you needed for virtlogd?09:25
markus_ztonyb: 1.30 I guess, let me double-check09:25
*** haomaiwa_ has joined #openstack-nova09:25
*** haomaiwa_ has quit IRC09:26
*** 18VAACCG7 has joined #openstack-nova09:26
markus_ztonyb: yep, 1.3.0 and 1.3.1 seems to have a few fixes for that09:26
*** 18VAACCG7 has quit IRC09:27
tonybmarkus_z: okay.  So clearly the UCA stuff wont be helpful initially :(09:27
*** haomaiwa_ has joined #openstack-nova09:27
tonybjohnthetubaguy: did that make sense?09:27
*** haomaiwa_ has quit IRC09:28
*** e0ne has joined #openstack-nova09:28
markus_ztonyb: no, not for that. We need another way there. But step by step.09:28
*** haomaiwa_ has joined #openstack-nova09:28
*** haomaiwa_ has quit IRC09:29
tonybmarkus_z: Yeah.  We'll make a plan in Austin if not befoer09:29
markus_ztonyb: sounds good09:29
*** rk4n has joined #openstack-nova09:29
*** haomaiwang has joined #openstack-nova09:29
tonybmarkus_z: so I can't find an example of a devstack plugin on github in project-config09:29
openstackgerritFeng Xi Yan proposed openstack/nova: Enable VMWare driver to attach one more SCSI LUN to VM.  https://review.openstack.org/28705509:29
*** haomaiwang has quit IRC09:30
markus_ztonyb: let me check my notes where this should be done09:30
*** haomaiwa_ has joined #openstack-nova09:30
johnthetubaguytonyb: OK, I haven't seen that conversation about os-brick, we decided at the midcycle we should do it, the problem is getting a library released after the deadline, which is a hard no, as I understand it. I guess thats on the ML somewhere?09:30
*** haomaiwa_ has quit IRC09:31
openstackgerritFeng Xi Yan proposed openstack/nova: Enable VMWare driver to attach more than one SCSI LUN to VM.  https://review.openstack.org/28705509:31
*** haomaiwang has joined #openstack-nova09:31
*** Daisy has joined #openstack-nova09:31
tonybjohnthetubaguy: I'll look for dates09:31
*** haomaiwang has quit IRC09:32
*** mpavone has quit IRC09:32
*** haomaiwa_ has joined #openstack-nova09:32
*** haomaiwa_ has quit IRC09:33
*** thorst_ has joined #openstack-nova09:33
BobBalljohnthetubaguy: XenAPI meeting in #openstack-meeting09:33
*** haomaiwa_ has joined #openstack-nova09:33
tonybjohnthetubaguy: I don't know about the hard no part.09:33
*** haomaiwa_ has quit IRC09:34
*** dedery has joined #openstack-nova09:34
*** haomaiwa_ has joined #openstack-nova09:34
*** haomaiwa_ has quit IRC09:35
johnthetubaguyBobBall: yeah, this is release critical right now, I am afraid09:35
*** 7GHAAFDDU has joined #openstack-nova09:35
*** 7GHAAFDDU has quit IRC09:36
*** danpb has left #openstack-nova09:36
*** 7GHAAFDD9 has joined #openstack-nova09:36
markus_ztonyb: I don't find it right now09:36
tonybmarkus_z: the plugin needs to be on git.o.o :(09:37
*** danpb has joined #openstack-nova09:37
*** 7GHAAFDD9 has quit IRC09:37
*** gcb has quit IRC09:37
*** haomaiwang has joined #openstack-nova09:37
markus_ztonyb: I've read it in the devstack docs too but I wasn't sure if that information is still valid09:37
*** e0ne has quit IRC09:37
*** haomaiwang has quit IRC09:38
tonybmarkus_z: Well outside of CI it works on githib09:38
*** haomaiwang has joined #openstack-nova09:38
*** haomaiwang has quit IRC09:39
tonybjohnthetubaguy: Yeah dhellmann's last email on the subject says we're out of luck09:39
*** 7JTAACEZI has joined #openstack-nova09:39
tonybjohnthetubaguy: privsep for os-brick/nova/cinder is a Newton thing now :(09:39
*** 7JTAACEZI has quit IRC09:40
*** roeyc has joined #openstack-nova09:40
tonybjohnthetubaguy: http://lists.openstack.org/pipermail/openstack-dev/2016-February/087590.html09:40
*** 7GHAAFDF6 has joined #openstack-nova09:40
*** thorst_ has quit IRC09:40
*** 7GHAAFDF6 has quit IRC09:41
*** 7GHAAFDGO has joined #openstack-nova09:41
openstackgerritMoshe Levi proposed openstack/nova: ironic driver: Adding InfiniBand support  https://review.openstack.org/26654009:41
*** 7GHAAFDGO has quit IRC09:42
tonybmarkus_z: So I'll switch to including the new repo in the big tent09:42
*** haomaiwa_ has joined #openstack-nova09:42
tonybmarkus_z: I'm about 50% through http://docs.openstack.org/infra/manual/creators.html09:42
*** haomaiwa_ has quit IRC09:43
*** haomaiwang has joined #openstack-nova09:43
johnthetubaguytonyb: yeah, thats what I thought09:43
markus_ztonyb: mhm, a lot of extra work...09:43
*** haomaiwang has quit IRC09:44
johnthetubaguytonyb: deadline was defo last week09:44
*** haomaiwang has joined #openstack-nova09:44
tonybjohnthetubaguy: Yeah09:44
tonybmarkus_z: not too much but a lot for an experiment09:44
danpbtonyb: johnthetubaguytime to rip out all use of os-brick then :-)09:44
*** haomaiwang has quit IRC09:45
tonybdanpb: If only that'd work09:45
*** haomaiwang has joined #openstack-nova09:45
*** haomaiwang has quit IRC09:46
danpbor we could just admit that existing rootwrap protection is worthless and go back to using an allow-all sudo rule ,avoiding the problem of dealing with os-brick rootwrap configs09:46
*** 18VAACCPL has joined #openstack-nova09:46
*** ndipanov has joined #openstack-nova09:46
*** 18VAACCPL has quit IRC09:47
*** haomaiwa_ has joined #openstack-nova09:47
*** haomaiwa_ has quit IRC09:48
tonybdanpb: Yeah that'd work ... but I don't know if it's really a path forward09:48
*** haomaiwa_ has joined #openstack-nova09:48
*** haomaiwa_ has quit IRC09:49
markus_ztonyb: What about this sentence in the devstack docs "Ideally a plugin will be included within the devstack directory of the project they are being tested. "09:49
*** huats_ has quit IRC09:49
markus_ztonyb: http://docs.openstack.org/developer/devstack/plugins.html#using-plugins-in-the-openstack-gate09:49
*** haomaiwang has joined #openstack-nova09:49
*** gcb_ has joined #openstack-nova09:49
markus_zIs that a possibility for us?09:49
*** haomaiwang has quit IRC09:50
*** haomaiwa_ has joined #openstack-nova09:50
*** huats_ has joined #openstack-nova09:50
*** haomaiwa_ has quit IRC09:51
tonybmarkus_z: Yeah that's what I'm doign09:51
*** haomaiwa_ has joined #openstack-nova09:51
*** haomaiwa_ has quit IRC09:52
ttxbauzas / johnthetubaguy: release management question... You have the os-vif library as a deliverable in the nova team, which is marked cycle-with-intermediary, but hasn't had a release yet. What's the plan there ?09:52
*** haomaiwang has joined #openstack-nova09:52
bauzasttx: good point, passing to danpb09:52
tonybttx: that's a question for danpb09:52
*** haomaiwang has quit IRC09:53
bauzasttx: during last midcycle, we discussed on that new library but I wasn' expecting any release by Mitaka AFAIR09:53
markus_ztonyb: That means "nova/devstack/<job-name>" is a pointer to the repo.o.o you are creating?09:53
*** haomaiwang has joined #openstack-nova09:53
*** haomaiwang has quit IRC09:54
johnthetubaguyttx: its not useful yet, I believe the plan is no release09:54
tonybmarkus_z: Oh hmmm I didn't think of that.09:54
*** haomaiwa_ has joined #openstack-nova09:54
tonybmarkus_z: I see what you're saying now.09:54
*** haomaiwa_ has quit IRC09:55
markus_ztonyb: Other jobs enable the plugin in the job description itself: http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/devstack-gate.yaml#n20709:55
*** mingyu has joined #openstack-nova09:55
johnthetubaguyttx: as we have not go to the point of integrating the Nova yet, its not in a point to be used, as I understand it, certainly that was due last week, so is now not possible, as part of mitaka09:55
*** deepakcs has joined #openstack-nova09:55
ttxok09:56
johnthetubaguyttx: bauzas: we need to loop back with jaypipes and danpb again, to double check09:56
tonybmarkus_z: Yeah, that'd be the template we'd probably use once it was on git.o.o09:56
*** haomaiwa_ has joined #openstack-nova09:56
johnthetubaguyttx: but yeah, lets drop that from the mitaka randar, let me know if we need to give the appropriate bat signal for that09:56
bauzasjohnthetubaguy: agreed to drop from mitaka scope09:56
*** haomaiwa_ has quit IRC09:57
johnthetubaguyttx: we are waiting on patches for another python-novaclient release, I need to double check on those today, and make the call if we are still waiting on them09:57
ttxI'll sync with dhellmann but I don't think we need to adjust the release model as long as the long-term plan is to release it09:57
*** haomaiwang has joined #openstack-nova09:57
*** rk4n has quit IRC09:57
johnthetubaguyttx: cool, yeah, that is the long term plan09:57
*** haomaiwang has quit IRC09:58
johnthetubaguyttx: its very similar to os-brick, at least in release terms09:58
*** haomaiwang has joined #openstack-nova09:58
tonybmarkus_z: So I think that incliding it in the nova code is a bad idea.09:58
johnthetubaguybauzas: I need to ask you about your check-destinations stuff, I think we have to kick out the remaining patches right?09:58
*** haomaiwang has quit IRC09:59
bauzasjohnthetubaguy: so I'm struggling with cold migrate, but I feel you should fence the axe on my WIP09:59
markus_ztonyb: I think so too. I feels like another piece one has to know to maintain it without offering benefits.09:59
tonybmarkus_z: It's desigend to be reuseable for neutron or horizon etc so making it a full big tent project is the right thing long term I think09:59
*** haomaiwa_ has joined #openstack-nova09:59
bauzasjohnthetubaguy: because I would only be able to send a nice patch by today EOB, and that's too close to the FF09:59
* tonyb will just knuckle down and make it so09:59
markus_ztonyb: true, I totally forgot about the other projects09:59
johnthetubaguybauzas: yeah, it way past that point now09:59
*** haomaiwa_ has quit IRC10:00
bauzasjohnthetubaguy: no worries, I'll submit a newton spec once FF is done10:00
johnthetubaguybauzas: did we merge the bug fixes comming out of that series now?10:00
*** e0ne has joined #openstack-nova10:00
bauzasjohnthetubaguy: indeed10:00
*** haomaiwang has joined #openstack-nova10:00
bauzasjohnthetubaguy: so, I think we can mark the BP partially-implemented and I'll create another newton on10:01
bauzasone10:01
*** haomaiwang has quit IRC10:01
johnthetubaguybauzas: ack, done half of that10:01
*** haomaiwang has joined #openstack-nova10:01
bauzasjohnthetubaguy: ack, thanks10:01
bauzasjohnthetubaguy: I saw you were noting some changes missing reno notes ? nice10:01
*** haomaiwang has quit IRC10:02
bauzasjohnthetubaguy: that's my next duty10:02
* bauzas just saw that testr was accepting regexes for sublists of tests, woah :D10:02
*** haomaiwa_ has joined #openstack-nova10:02
*** rk4n has joined #openstack-nova10:02
*** mingyu has quit IRC10:02
*** haomaiwa_ has quit IRC10:03
johnthetubaguybauzas: not sure that was me recently, but yeah, we need to do that check10:03
johnthetubaguybauzas: but thats more a pre-RC check at this point10:03
bauzasagreed10:03
bauzasbut I like adding notes for every milestone too10:03
*** haomaiwang has joined #openstack-nova10:03
bauzasthe prelude and other big stuff is for pre-RC I agree10:03
johnthetubaguybauzas: I am going through every blueprint trying to work out where its add, so we have a small set of inprogress ones to track10:03
bauzasnice10:03
*** haomaiwang has quit IRC10:04
ernoHello, i would like to call the 'delete_tokens_for_instance' method as in nova.consoleauth.rpcapi. I tried using RPCClient and using as Target(topic=str('consoleauth'), exchange=NOVA_EXCHANGE). The message is sent to rabbitmq, but the method isn't doing anything.10:04
ndipanovjohnthetubaguy, I am assuming we're not looking at bugs this week right?10:04
*** haomaiwang has joined #openstack-nova10:04
*** haomaiwang has quit IRC10:05
*** mingyu has joined #openstack-nova10:05
*** 7GHAAFDT9 has joined #openstack-nova10:05
*** 7GHAAFDT9 has quit IRC10:06
*** haomaiwang has joined #openstack-nova10:06
johnthetubaguyndipanov: ish, I mean we are still free to merge any bug fixes, although its the last day to get +2s and +Ws on the features10:06
johnthetubaguysome of the bugs might be more important, and thats cool10:07
*** haomaiwang has quit IRC10:07
*** RuiChen has quit IRC10:07
*** haomaiwang has joined #openstack-nova10:07
*** RuiChen has joined #openstack-nova10:07
*** haomaiwang has quit IRC10:08
*** haomaiwa_ has joined #openstack-nova10:08
*** haomaiwa_ has quit IRC10:09
markus_zndipanov: I added a gerrit query to the priorities etherpad for high prio bug fixes if that's for interest for you.10:09
*** haomaiwa_ has joined #openstack-nova10:09
markus_zsection "General High|Critical Bug Fixes in Gerrit" in https://etherpad.openstack.org/p/mitaka-nova-priorities-tracking10:09
*** haomaiwa_ has quit IRC10:10
*** haomaiwang has joined #openstack-nova10:10
*** haomaiwang has quit IRC10:11
*** haomaiwang has joined #openstack-nova10:11
*** mpavone has joined #openstack-nova10:11
ndipanovmarkus_z, nice10:11
*** haomaiwang has quit IRC10:12
*** haomaiwang has joined #openstack-nova10:12
*** MVenesio has joined #openstack-nova10:12
ndipanovmarkus_z, can you include my stuff in there?10:12
markus_zndipanov: And you were right with the "swap_volume" thing I asked you yesterday.10:12
ndipanovmarkus_z, we need this one before RC https://review.openstack.org/#/c/278079/10:12
*** haomaiwang has quit IRC10:13
ndipanovmarkus_z, ;)10:13
*** Jeffrey4l has joined #openstack-nova10:13
markus_zndipanov: Ah, that's new, I make an update which includes it.10:13
*** haomaiwang has joined #openstack-nova10:13
ndipanovand I would reaaaaaaly like for this to get merged this cycle: https://review.openstack.org/#/c/244489/10:13
*** haomaiwang has quit IRC10:14
ndipanovjohnthetubaguy, bauzas PaulMurray ^ and others interested in live migration10:14
*** 7JTAACFFQ has joined #openstack-nova10:14
johnthetubaguyah, the live-migrate team haven't been tracking that one, AFAIK10:14
*** 7JTAACFFQ has quit IRC10:15
johnthetubaguybeen focusing on the other things they were point too10:15
PaulMurrayndipanov, will look today - I've been trying to get the feature freeze stuff through10:15
johnthetubaguythat looks like a today or newton patch10:15
johnthetubaguyPaulMurray: thats almost certainly affected by FF though... due to its size10:15
ndipanovjohnthetubaguy, why?10:15
*** haomaiwang has joined #openstack-nova10:15
ndipanovit's a bugfix10:15
PaulMurrayjohnthetubaguy, what's the deadline for bugs - is it the same?10:15
ndipanovwhat does size have to do with it?10:16
*** haomaiwang has quit IRC10:16
*** 77CAACDGI has joined #openstack-nova10:16
ndipanovthe reason I held off on it for so long is because I believed it can go in as one10:16
*** 77CAACDGI has quit IRC10:17
ndipanovotherwise I would have done it instead of the pci crap I was working on that also got told to f-off due to another arbitrary deadline10:17
*** haomaiwang has joined #openstack-nova10:17
johnthetubaguysorry, I lost IRC there10:17
johnthetubaguyRC1 is the real deadline for that10:17
*** MVenesio has quit IRC10:17
ndipanovthat makes more sense10:17
*** haomaiwang has quit IRC10:18
*** amotoki has joined #openstack-nova10:18
ndipanovthanks10:18
johnthetubaguybut between m-3 and RC1 we have to look at regression risk vs gain, but the gain is high on these ones so its probably all cool10:18
*** haomaiwang has joined #openstack-nova10:18
*** haomaiwang has quit IRC10:19
*** haomaiwang has joined #openstack-nova10:19
ernoHello, can i call 'delete_tokens_for_instance' method from nova.consoleauth.rpcapi from my application somehow ?10:19
*** haomaiwang has quit IRC10:20
*** haomaiwa_ has joined #openstack-nova10:20
ndipanovjohnthetubaguy, well the only problem with risk mitigation I see is that we can't just revert it10:20
*** mvk has quit IRC10:20
*** Daisy has quit IRC10:20
*** haomaiwa_ has quit IRC10:21
ndipanovdue to RPC changes - which makes it by default riskier than a patch without it10:21
PaulMurrayndipanov, in the live migration meeting I told everyone to turn attention to these bugs now. So I'll push it along10:21
ndipanovPaulMurray, thanks10:21
*** haomaiwa_ has joined #openstack-nova10:21
*** Daisy has joined #openstack-nova10:21
johnthetubaguyndipanov: ack, yeah, lets try get the live-migrate subteam on that ASAP, and try get it merged ASAP10:21
*** haomaiwa_ has quit IRC10:22
johnthetubaguyagreed its a long standing big issue with live-migrate10:22
*** 7GHAAFD4Z has joined #openstack-nova10:22
johnthetubaguywe certainly hit those issues in production10:22
*** 7GHAAFD4Z has quit IRC10:23
danpbttx: johnthetubaguy  we're still planning to convert nova to use os-vif before the priority feature freeze10:23
*** haomaiwa_ has joined #openstack-nova10:23
danpbwe've not done any release of os-vif, because we don't want to commit to declaring its API stable until the last minute10:23
*** haomaiwa_ has quit IRC10:24
johnthetubaguydanpb: really, that is basically today, and we are not allowed to release a lib any more10:24
danpbwe'll do a release immediately before we merge the feature in nova10:24
PaulMurrayjohnthetubaguy, BTW - this is the last feature patch: https://review.openstack.org/#/c/260383/ - needs a couple of core reviews10:24
PaulMurrayalex_xu, ^^10:24
johnthetubaguyI did reach out to jaypipes about the deadlines for the lib release, and he did seem OK with that10:24
*** 7GHAAFD50 has joined #openstack-nova10:24
johnthetubaguydanpb: we can't release now, that deadline was last week10:24
johnthetubaguywhich I was trying to commincate the week before that, but clearly I totally failed on that front10:24
*** 7GHAAFD50 has quit IRC10:25
johnthetubaguydanpb: do we have any patches up for this in Nova, the bp was empty, so I just deffered it10:25
*** haomaiwa_ has joined #openstack-nova10:25
*** Daisy has quit IRC10:25
*** haomaiwa_ has quit IRC10:26
danpbjohnthetubaguy: final freeze was March 15 wasn't it ??10:26
*** 7GHAAFD6W has joined #openstack-nova10:26
johnthetubaguydanpb: no its tomorrow10:26
*** 7GHAAFD6W has quit IRC10:27
johnthetubaguydanpb: well, technically any time after yesterday morning10:27
danpbhmm, damn, i wasn't expecting that10:27
openstackgerritRadomir Dopieralski proposed openstack/nova: Don't use locals(), use a dict instead  https://review.openstack.org/28708510:27
johnthetubaguydanpb: the lib release freeze was last week10:27
*** haomaiwang has joined #openstack-nova10:27
*** haomaiwang has quit IRC10:28
johnthetubaguyits been the first line of ever nova meeting for the last month, I have tried to shout really loud about this one, appologies if the message got muddled somewhere10:28
*** yassine__ has joined #openstack-nova10:28
*** haomaiwa_ has joined #openstack-nova10:28
johnthetubaguythis is the official list: http://releases.openstack.org/mitaka/schedule.html10:28
danpbright but i can rarely make it to the nova meeting so that's basically invisible10:28
johnthetubaguyit was on the ML several times to be sure as well, at least I thought it was10:28
*** haomaiwa_ has quit IRC10:29
*** haomaiwa_ has joined #openstack-nova10:29
danpbanyway, i guess its too late to worry about it now10:29
johnthetubaguysadly, yes, I think this ship has sailed now10:29
*** haomaiwa_ has quit IRC10:30
johnthetubaguyideas on better ways to reach people very welcome10:30
*** haomaiwang has joined #openstack-nova10:30
*** haomaiwang has quit IRC10:31
*** ducttape_ has joined #openstack-nova10:31
*** haomaiwang has joined #openstack-nova10:31
*** gszasz has joined #openstack-nova10:31
*** haomaiwang has quit IRC10:31
openstackgerritDaniel Berrange proposed openstack/nova-specs: VIF port config versioned objects and driver plugin library  https://review.openstack.org/28709010:32
* danpb re-proposed same spec for newton10:32
*** rk4n has quit IRC10:34
*** mvk has joined #openstack-nova10:35
*** ducttape_ has quit IRC10:35
openstackgerritRadomir Dopieralski proposed openstack/nova: Don't use locals() and globals(), use a dict instead  https://review.openstack.org/28708510:37
*** thorst_ has joined #openstack-nova10:38
johnthetubaguyPaulMurray: andreykurilin: what state are we with python-novaclient? I am thinking about cutting the final mitaka release today, if so, what do we miss?10:39
andreykurilinjohnthetubaguy: hi! 2.25 microversion was not merged yet10:40
PaulMurrayjohnthetubaguy, there is one patch going through CI now - will finish check queue soon10:40
PaulMurrayjohnthetubaguy, its looking good but needs +2/+W10:40
johnthetubaguycools, this one I guess: https://review.openstack.org/#/c/260383/10:40
andreykurilinyes10:40
johnthetubaguycool, I added that on the blocking list on here: https://etherpad.openstack.org/p/mitaka-nova-priorities-tracking10:41
PaulMurrayjohnthetubaguy, I was just reviewing it - andreykurilin could you have a look too - you have better knowledge than me10:41
johnthetubaguyalthough I have a feeling its not going to make it through the gate10:41
andreykurilinPaulMurray: sure10:41
PaulMurrayeliqiao, added some tests in the last patchset10:42
johnthetubaguythe good news, is we could release as soon as mitaka is out the door to include that feature, which wouldn't be the end of the world10:42
johnthetubaguybut lets try for including it, if we can10:42
*** mingyu has quit IRC10:42
johnthetubaguyandreykurilin: are you happy with us doing a release otherwise, do we have any nasty outstanding bug fixes you have seen at all?10:43
andreykurilinjohnthetubaguy: I have no seen critical bugs, so, imo, we can do release today.10:44
johnthetubaguycool, we have till tomorrow, if we have to, I believe10:44
andreykurilinjohnthetubaguy: it looks like only this fix(high bug) https://review.openstack.org/#/c/287024/1 is ready to be merged, but it requires changes in commit msg10:45
*** thorst_ has quit IRC10:45
johnthetubaguyandreykurilin: maybe just edit the commit message yourself, and add a +1 if you are good with that?10:46
johnthetubaguyandreykurilin: it does look worth fixing, if we can10:46
openstackgerritJianghua Wang proposed openstack/nova: XenAPI: add unit test for plugin bandwidth  https://review.openstack.org/28644510:46
andreykurilinjohnthetubaguy: sure, I have already opened edit mode:)10:47
johnthetubaguy:)10:47
*** ihrachys has quit IRC10:47
PaulMurrayjohnthetubaguy, https://review.openstack.org/#/c/260383/ has cleared checks so we need core eyes on it now10:48
*** rk4n has joined #openstack-nova10:48
PaulMurraynone of the ones that have been looking at this are around right now10:49
openstackgerritGábor Antal proposed openstack/nova: Replaced unittest and unittest2 to testtools  https://review.openstack.org/28520910:49
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Add last_request_id parameter to SessionClient  https://review.openstack.org/28702410:49
*** dedery has quit IRC10:50
*** yamahata has quit IRC10:51
*** dims_ has joined #openstack-nova10:51
andreykurilinPaulMurray: patch 2.25 looks good to me:)10:52
PaulMurrayandreykurilin, thanks10:52
andreykurilinjohnthetubaguy: commit msg for https://review.openstack.org/287024 is fixed now10:52
*** phuongnh has quit IRC10:53
johnthetubaguycool10:53
*** mingyu has joined #openstack-nova10:54
johnthetubaguymarkus_z: we should work on an email to the ML on how we co-ordinate the bug fixing over the next few weeks, till we hit RC110:55
*** roeyc has quit IRC10:56
johnthetubaguymarkus_z: I am really thinking about trying to get reviewers and subgroups to keep the lists in here up to date, and simple: https://etherpad.openstack.org/p/mitaka-nova-priorities-tracking, I think we might have to go through there and do a tidy up so its useful again10:56
johnthetubaguymarkus_z: folks can use your tracker system to find good candidates for a sub team to focus on, I think?10:57
openstackgerritSergey Nikitin proposed openstack/nova: Added db API layer to add instance tag-list filtering support  https://review.openstack.org/24880510:57
openstackgerritSergey Nikitin proposed openstack/nova: Added server tags controller  https://review.openstack.org/12894010:57
openstackgerritSergey Nikitin proposed openstack/nova: Added server tags support in nova-api  https://review.openstack.org/26893210:57
markus_zjohnthetubaguy: Let me think during lunch about it, I'll come back to you in 1h10:57
johnthetubaguymarkus_z: sounds good, thank you10:58
johnthetubaguymarkus_z: there is that bug bash thing happening next week, giving them a list of higher priority bugs that need fixing would be good stuff, probably just an LP query really10:58
*** cdent has joined #openstack-nova10:59
openstackgerritJianghua Wang proposed openstack/nova: XenAPI: add unit test for plugin bandwidth  https://review.openstack.org/28644511:01
*** lpetrut has joined #openstack-nova11:02
*** sdague has joined #openstack-nova11:02
*** Daisy has joined #openstack-nova11:08
*** Mackie2k has quit IRC11:09
*** pbrooko has joined #openstack-nova11:09
PaulMurraysdague, good morning11:09
sdaguePaulMurray: morning11:12
PaulMurraysdague, when you have woken up... this is the last novaclient patch waiting for core review: https://review.openstack.org/#/c/26038311:12
sdagueok, looking11:13
PaulMurray(or at least the last for an API change)11:13
*** daemontool_ has joined #openstack-nova11:15
sdagueyeh, that all seems fine11:16
PaulMurraysdague, great, thanks11:16
johnthetubaguyPaulMurray: sdague: andreykurilin: great work, let me jump on that one now11:17
*** rk4n has quit IRC11:17
*** daemontool has quit IRC11:17
*** salv-orlando has joined #openstack-nova11:19
*** Daisy has quit IRC11:21
*** Daisy has joined #openstack-nova11:21
*** salv-orl_ has quit IRC11:22
johnthetubaguybauzas: cdent: you know the allocation ratios, it looks like the new model requires us to do the same with reserved_host_disk_mb, if you look at: https://review.openstack.org/#/c/279313/5/nova/objects/compute_node.py11:23
johnthetubaguybauzas: cdent: what do you think are the chance of getting this fixed and +Wed today? I am wondering where/when we have to draw the line11:23
openstackgerritRadomir Dopieralski proposed openstack/nova: Don't use locals() and globals(), use a dict instead  https://review.openstack.org/28708511:24
cdentjohnthetubaguy: I think the odds are good for today, assuming edleafe and jaypipes are around. I'm insufficiently confident about the guts of the compute_node object to feel comfy flailing around in there. :(11:24
*** EinstCrazy has quit IRC11:24
*** Daisy has quit IRC11:26
cdentsdague: where do you fall on the product v kit debate? I've seen you use a phrase like "back when openstack _was_ a kit X made sense" (referring to customisable hooks)11:28
*** pbrooko has quit IRC11:28
danpbdoes anyone know where there are docs describing how to use oslo.privsep ?11:29
danpbanyone suggesting oslo.privsep/doc/source/usage.rst gets a slap11:29
sdaguedanpb: I think that's the best you get11:30
sdaguethat code is all quite new, and I don't think it's doced very well yet11:30
*** tbachman has quit IRC11:30
danpbthe entirity of that doc says "import oslo_privsep"  which I'd figured out myself already :-)11:31
sdagueheh11:31
danpbhttp://docs.openstack.org/developer/oslo.privsep/usage.html11:31
sdaguedanpb: reminds me of - https://xkcd.com/353/11:31
danpblol11:32
*** ducttape_ has joined #openstack-nova11:32
* johnthetubaguy giggles11:32
dims_danpb : sdague : the library is not ready for general use. there's only one person working on it. we could use help11:32
johnthetubaguydanpb: angus had some os-brick patches up, but thats not really docs either11:32
dims_danpb : sdague : johnthetubaguy : the best resource we have at the moment is the spec itself - https://specs.openstack.org/openstack/oslo-specs/specs/liberty/privsep.html11:33
johnthetubaguydanpb: although one was bad (emulate sudo), an the other where you have a set of clear functions as entry points, was the better option, from what I remember him saying, unsure if thats helps11:34
danpbah yes, the spec does look to have some examples in it11:34
dims_danpb : yes, took a while for the spec to solidify11:34
danpbmight be enough for me to get going at least11:35
danpbjohnthetubaguy: yeah i looked briefly as the os-brick patch, but they're kind of scary huge11:35
johnthetubaguycdent: I am wondering if we need to add things to the compute node first, so we can have the correct information for the migration of data to the new stuff11:35
johnthetubaguydanpb: ah, that sucks11:36
johnthetubaguyI guess thats why they never merged11:36
danpb(probably why cinder doesn't want to rush them in for M)11:36
johnthetubaguyyeah, totally11:36
dims_danpb : gus is in the main person working on it. he is on the other side of the world. fyi11:36
*** ducttape_ has quit IRC11:36
danpbdims_: you mean australia like timezone ?11:36
johnthetubaguyyeah, near tonyb and mikal I believe11:37
danpbok11:37
*** rmart04 has quit IRC11:38
*** daemontool_ has quit IRC11:39
*** daemontool_ has joined #openstack-nova11:39
dims_danpb : yep11:39
*** ociuhandu has joined #openstack-nova11:39
*** sudipto has quit IRC11:41
*** thorst_ has joined #openstack-nova11:43
*** zul has quit IRC11:44
*** zul has joined #openstack-nova11:45
sdagueso, I guess there is a question here. We assumed that privsep was the thing because it was going to solve this problem we had.11:46
sdaguebut, it looks massively understaffed11:46
sdagueand the only place we need it is on the compute worker11:46
sdaguewhere there is no security anyway, we rootwrap dd11:47
sdaguewith no arg filtering11:47
sdagueshould we fully give up on privsep, and just admit the compute worker is not security scopable11:48
sdagueinstead of pretending that our rootwrap rules provide real security11:48
*** thorst_ has quit IRC11:51
*** rk4n has joined #openstack-nova11:51
*** ihrachys has joined #openstack-nova11:52
*** pbrooko has joined #openstack-nova11:53
*** dave-mccowan has joined #openstack-nova11:53
johnthetubaguysdague: so I quite like that we could get a decent separation with privsep, it doesn't feel like a dead effort yet, but equally its not really proved its worth yet11:53
openstackgerritPawel Koniszewski proposed openstack/nova: live-migration: Update instance availability_zone  https://review.openstack.org/25404111:54
sdaguejohnthetubaguy: I get concerned about us having another piece of critical infrastructure that doesn't have enough contributors.11:54
*** sneti has joined #openstack-nova11:54
johnthetubaguysdague: if we got that separation right, I feel like any mistakes we make further up would have much more limited impact11:54
*** sahid has joined #openstack-nova11:55
sdaguesure, it feels pretty weird though that people care about this bit, when things like the rabbit password are on disk11:55
sdagueand there is no secure membership11:55
johnthetubaguysdague: thats fair, but it feels a bit chicken and egg, if we relied on it more, it would likely get a bigger set of resources11:55
sdagueit's all very window dressing11:55
sdaguejohnthetubaguy: really... I don't think so11:55
sdaguesee: oslo.messaging11:56
johnthetubaguyyeah, thats fair11:56
johnthetubaguyI am trying to think about cases where I thought it did help us11:56
sdaguethe oslo.messaging folks are doing good work, but their aren't enough of them11:56
*** doug-fish has joined #openstack-nova11:56
johnthetubaguyI think its escaping input further up that goes bad somehow, making checking that at the bottom is useful to do in a structured way11:57
danpblooking at privsep, I think actually oslo.privsep doesn't really need significant number contributors11:57
dims_sdague : i have 3 people full time on oslo.messaging + rabbitmq upstream11:57
danpbits lacking docs, but from what i can tell the code is basically all there11:57
danpbwhere resources are needed is in converting projects to use it11:57
*** MVenesio has joined #openstack-nova11:57
sdaguejohnthetubaguy / danpb - ok, if you think it's solid, that's fine11:57
danpbwhich isn't ultimately the job of the oslo.privsep maintainers todo11:58
sdaguedims_: we still have a 30% chance of having to block a release in GR11:58
johnthetubaguyI think my point is, when its important, we can make stronger arguments about getting resources, now it may not happen, but its an easier conversation11:58
dims_danpb : i still want more contributors to both oslo.messaging and oslo.privsep (more eyes, more testing)11:58
johnthetubaguydanpb: honestly, I think os-vif is a good way to see if this is useful, do let us know if it seems helpful or not, and that can inform how the other stuff might migrate, I guess?11:59
danpbyeah i will do11:59
danpbfrom the look of it i think it'll actually be trivial in os-vif - as we really just have 2 top level methods which should always run privileged12:00
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: [extensions] cache discovered extensions  https://review.openstack.org/28713112:00
*** moshele has quit IRC12:00
dims_sdague : last block for oslo.messaging was on Dec 8th12:00
dims_we have had a lot of releases since then12:00
sdagueand this is still open - https://bugs.launchpad.net/oslo.messaging/+bug/154500212:01
openstackLaunchpad bug 1545002 in oslo.messaging "Grenade Failure - MessagingTimeout on floating IP remove" [High,Incomplete]12:01
johnthetubaguydims_: I wonder if we can take this to the product working group (again), they are starting to think about how they give their devs time to work on reviews, this feels like its in that bucket, keep the supporting infrastructure working, I will try raise that with the folks I know in there.12:01
dims_sdague : we don't see it in logstash anymore12:01
andreykurilinjohnthetubaguy:  Can you look at https://review.openstack.org/287131 ? After merging https://review.openstack.org/#/c/280726/, cache of discovered extension can be required.12:01
dims_sdague : that way you can see the list of bugs nova has too!12:01
sdaguedims_:  :)12:02
dims_johnthetubaguy : not arguing that we don't need more folks. we should appreciate progress when we see it12:02
dims_johnthetubaguy : +112:02
sdaguedims_: like I said, I think the work being done by the folks doing it is good. But I'd still consider it under staffed12:03
johnthetubaguydims_: oh, gotcha12:03
sdaguewhen we go a full cycle without a release that regresses us, I'll consider it under control12:03
dims_sdague : right. the main issues we are seeing right now is in upstream rabbitmq and i have one person dedicated there12:03
dims_sdague : good metric12:03
sdaguedims_: that was sorted by 4.2.0 ?12:03
johnthetubaguyandreykurilin: maybe we should delay that other patch, while we sort the caching, that sounds a little complicted?12:04
*** gokrokve has joined #openstack-nova12:04
sdaguewe didn't block 4.1.0, but it would have counted as one we might12:04
*** qeelee has joined #openstack-nova12:05
bauzasjohnthetubaguy: not sure I understand your point, because I already asked jaypipes to provide it https://review.openstack.org/#/c/284484/3/nova/db/sqlalchemy/api.py@75612:05
*** smatzek has joined #openstack-nova12:05
*** aysyd has joined #openstack-nova12:05
andreykurilinjohnthetubaguy: sounds reasonable. I'll rebase 280726 on 28713112:06
johnthetubaguybauzas: other way around, we need that value to flow up from the compute nodes into the DB, I think12:06
dims_sdague : we need more people actively reviewing, more testing, more performance testing, regression testing. yes12:06
*** cdent has quit IRC12:06
sdaguedims_: right, I think we're agreed on that point right?12:06
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Turn on extensions from contrib dir for v2.1  https://review.openstack.org/28072612:06
bauzasjohnthetubaguy: oh, you mean https://review.openstack.org/#/c/279313/5/nova/objects/compute_node.py@374 ?12:06
johnthetubaguybauzas: right now, in the conductor, when we try to create the inventory, we don't know the value on the compute node12:06
bauzasjohnthetubaguy: if so, yes I agree12:06
johnthetubaguybauzas: yeah, thats it12:07
johnthetubaguybauzas: it feel like we need to do a copy of the allocation ratio first, which isn't a small job12:07
bauzasjohnthetubaguy: yeah, I had no time to review yet this patch, but I agree with Yingxin12:07
bauzasjohnthetubaguy: since the allocation ratios are now per computes12:07
johnthetubaguybauzas: we certainly have many and varying values for the RAM reserved, its not a theoretical issue12:07
dims_sdague : yep.12:07
bauzasjohnthetubaguy: well, the reserved values are different opts12:08
bauzasjohnthetubaguy: that's not for allocation ratios12:08
johnthetubaguydims_: I am wondering how we feed these requirements into the Product working group, so they get us more resources for that stuff that is not so visible to them otherwise12:09
*** gokrokve has quit IRC12:09
johnthetubaguybauzas: agreed they are totally different, but we need both to follow the same pattern now, since the scheduler needs both values now12:09
bauzasjohnthetubaguy: https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L45-L4612:09
johnthetubaguybauzas: yes, that needs to be in the scheduler DB12:10
bauzasjohnthetubaguy: right, so providing those as fields for the ComputeNode object?12:10
dims_johnthetubaguy : everyone complains about rabbitmq and oslo.messaging, but no one wants to help :( if you remember the kilo/liberty boundry with heartbeats problem etc12:10
johnthetubaguydims_: agreed, the product working group hold the strings on what many of our developers are allowed to work on, so its a possible route to change that12:10
bauzasjohnthetubaguy: I mean, we need to set the inventories tuple by using the "reserved" conf opts, if that's what you want to see12:11
dims_johnthetubaguy : interesting12:11
johnthetubaguydims_: its totally not working right now, but I have asked they do that for Reviews, and they have taken that as an action from their meetup12:11
johnthetubaguybauzas: right, but thats happening on the conductor, I think12:12
*** rodrigods has quit IRC12:12
*** rodrigods has joined #openstack-nova12:12
bauzasjohnthetubaguy: if that's RT stuff, it's in the compute manager12:12
dims_johnthetubaguy : cool. i'll try to raise the issue as well. thanks12:12
johnthetubaguybauzas: OK, we are talking cross purposes12:13
bauzasjohnthetubaguy: the conductor is only used by the scheduler for getting the compute values, right?12:13
johnthetubaguybauzas: the conductor runs the object calls made on the compute node12:13
bauzasjohnthetubaguy: yup, I agree12:13
bauzasoh, I see your point12:14
bauzassec, sec12:14
bauzasjohnthetubaguy: okay, I understand the problem, since create() and save() are remotable, it means that the calls are made by the conductor, agreed12:15
johnthetubaguynow if we could make that bit run on the compute node, that would be great, even if that means we make the compute manager call the data migration stuff12:15
bauzasjohnthetubaguy: so, IMHO, reserved values should maybe be fields, no ?12:15
bauzaslike we have for ratios12:15
*** mingyu has quit IRC12:16
bauzasso, you can set those in the compute side by setting like compute.reserved_mb = <> but then the save will persist it by the conductor12:16
johnthetubaguybauzas: well that was my earlier argument, yes12:16
johnthetubaguybauzas: I made it poorly though12:16
alex_xuPaulMurray: sorry +2 2.25 client patch late...just back from a meeting12:17
bauzasjohnthetubaguy: for some reason, since yesterday, my english comprehension skills dropped the bar12:17
johnthetubaguybauzas: its possible called being sleepy/warn out, at least thats why my skills are poor right now12:17
PaulMurrayalex_xu, no worries - its in gate12:18
johnthetubaguybauzas: cool, anyways, appreciate your thoughts on that, I though I was missing something magic that fixed that12:18
alex_xuPaulMurray: cool~12:18
bauzasjohnthetubaguy: lemme consider the other option, ie. if we don't want those fields in12:18
bauzasand in the context of us supporting liberty nodes12:19
bauzasjohnthetubaguy: that would need that the _create_inventory call would import the compute opts, right?12:20
johnthetubaguybauzas: yeah, I think thats the alternative12:20
bauzaswhich would be a conductor service importing some nova.conf value that couldn't be identical to the ones in the computes' nova.conf12:20
johnthetubaguyso here is the killer... nova-manage would be unable to do the data migrations, if we don't add this some other way12:21
johnthetubaguyactually, thats not really important for dead nodes12:21
bauzaswait, wait12:21
johnthetubaguybauzas: so I need to go get some food, before I get super grouchy, lets catch up in a bit12:22
bauzassure12:22
johnthetubaguyprobably when the others are around as well12:22
* johnthetubaguy gets food12:22
bauzasI already refueled my stomach, so it's your turn12:22
openstackgerritjichenjc proposed openstack/python-novaclient: move migrations from extension  https://review.openstack.org/28521312:22
ndipanovbauzas, johnthetubaguy I -2ed https://review.openstack.org/#/c/242087/ which is related to resource providers... I assume we were not thinking it was going to land this cycle anyway - but I think it needs a rethink12:24
pkoniszewskialex_xu: PaulMurray: 2.25 client patch passed gate, so we have everything merged12:25
alex_xupkoniszewski: so cool12:25
pkoniszewskieliqiao: ^^12:25
ndipanovanyone know Dimitry's IRC handle?12:25
PaulMurraypkoniszewski, eliqiao alex_xu well done on those12:25
bauzasndipanov: yes, that's not part of what we agreed, it was a best-effort review for the above12:25
*** erlon has joined #openstack-nova12:25
*** houming has quit IRC12:26
*** roeyc has joined #openstack-nova12:26
PaulMurrayjohnthetubaguy, ^^ we're done with python-novaclient for live migration features12:26
alex_xuPaulMurray: np, thanks to eliqiao's patient12:26
openstackgerritmeretiko proposed openstack/python-novaclient: The novaclient Python API doc keystoneauth example fixed  https://review.openstack.org/28716312:26
ndipanovbauzas, yeah that one needs to go back to square one sadly12:26
bauzasndipanov: https://www.openstack.org/community/members/profile/10595 isn't that verbose12:27
ndipanovone more example of how specs are net very useful for certain class of changes...12:27
ndipanovnot*12:27
bauzasndipanov: FWIW, you could also add a few here https://www.openstack.org/community/members/profile/8555 :)12:28
bauzasI usually go to https://www.openstack.org/community/members/ for that kind of purpose12:28
ndipanovbauzas, I have no idea where that text comes from an I am not happy it's autogenerated12:28
bauzashttps://www.openstack.org/community/members/profile/8407 as my example :)12:28
bauzasndipanov: it's your Foundation account12:29
*** deepakcs has quit IRC12:29
*** kiran-r has joined #openstack-nova12:29
markus_zjohnthetubaguy: The bug smash days sound like it possibility to get the bug list in a sane state. I haven't participated in one before, so I don't know how well they are suited for *writing* bug fixes.12:29
*** ishant has quit IRC12:29
ndipanovbauzas, oooookay12:29
*** rtheis has joined #openstack-nova12:29
bauzashttps://www.openstack.org/profile/12:29
markus_zjohnthetubaguy: I can prepare a list for such a big cleanup.12:29
*** subscope has joined #openstack-nova12:30
markus_zjohnthetubaguy: The coordination of the bug fixing, do you mean "merging reviews" or actually wring bug fixes?12:30
*** raildo-afk is now known as raildo12:31
*** gcb_ has quit IRC12:32
*** ducttape_ has joined #openstack-nova12:32
markus_zjohnthetubaguy: I'm in meetings in 30 minutes. I write something up and post it on the ML.12:35
*** zengine has joined #openstack-nova12:36
*** subscope has quit IRC12:36
*** ducttape_ has quit IRC12:37
*** jaosorior has quit IRC12:37
*** jaosorior has joined #openstack-nova12:38
*** jaosorior has quit IRC12:39
*** jaosorior has joined #openstack-nova12:40
*** claudiub has joined #openstack-nova12:40
*** dedery has joined #openstack-nova12:42
*** Daisy has joined #openstack-nova12:43
*** takedakn has joined #openstack-nova12:43
*** subscope has joined #openstack-nova12:43
*** rfolco has joined #openstack-nova12:44
*** subscope has quit IRC12:44
*** subscope has joined #openstack-nova12:46
*** edmondsw has joined #openstack-nova12:46
*** atuvenie has joined #openstack-nova12:46
*** dedery has quit IRC12:47
*** thorst_ has joined #openstack-nova12:48
*** markvoelker has quit IRC12:48
*** markvoelker has joined #openstack-nova12:49
*** lucasagomes is now known as lucas-hungry12:51
openstackgerritMichael Krotscheck proposed openstack/nova: Added Keystone and RequestID headers to CORS middleware  https://review.openstack.org/26541512:53
*** fawadkhaliq has joined #openstack-nova12:54
*** baoli has joined #openstack-nova12:56
ansiwenmarkus_z: re: https://bugs.launchpad.net/nova/+bug/154981412:57
openstackLaunchpad bug 1549814 in OpenStack Compute (nova) "Metadata caching is fundamentally broken" [High,In progress] - Assigned to Sven Anderson (ansiwen)12:57
markus_zansiwen: yes, what's with that?12:57
ansiwenmarkus_z: I'm just limiting the defect at the moment, later someone else from my team will probably pick it up and fix it properly12:58
ansiwenmarkus_z: dansmith raised the possibility of security issues12:59
*** links has quit IRC12:59
ansiwenmarkus_z: in any case it's scary code12:59
ansiwenmarkus_z: did you bring that up here already like you mentioned in the comment?13:00
*** kylek3h has quit IRC13:00
markus_zansiwen: Only that it has no the "mitaka-rc-potential" tag which gets extra attention. I'll also mention that in the next nova meeting.13:01
ansiwenmarkus_z: ok, cool13:02
*** dedery has joined #openstack-nova13:02
markus_zansiwen: I have to go to a meeting. Thanks for your notice. Its maybe also worth to note that down in the bug report to inform other interested parties.13:03
*** mpavone has quit IRC13:03
danpbansiwen: nb, in future if theres a chance of security issue, then do not file a public bug / discuss on public irc - report it to the security  team in private13:03
ansiwenmarkus_z: I was talking to dansmith and dims here the other night about it, and both agreed that this is considered broken and should be fixed13:04
ansiwendanpb: I see, since it is just a "gut feeling", I thought it might be ok to do it public13:05
*** Daisy has quit IRC13:08
*** Daisy has joined #openstack-nova13:08
*** ducttape_ has joined #openstack-nova13:09
ansiwendanpb: actually, for me it just looked very dysfunctional but not as an security issue.13:09
*** takedakn has quit IRC13:10
danpbok, just if there's a reasonable liklihood of it being a sec issue13:10
ansiwendanpb: sure, I understand...13:11
*** pbrooko has quit IRC13:11
*** Jeffrey4l has quit IRC13:12
*** baoli has quit IRC13:12
*** Daisy has quit IRC13:13
johnthetubaguyndipanov: I was just about to -2 for "there is no time to review this now", but "on dear this is all wrong" reasons is a way better reason to hold tight on that, thanks for the heads up13:14
*** Jeffrey4l has joined #openstack-nova13:15
*** pbrooko has joined #openstack-nova13:16
*** rk4n_ has joined #openstack-nova13:19
*** baoli has joined #openstack-nova13:19
*** fawadkhaliq has quit IRC13:21
*** lixiaoy1 has quit IRC13:22
*** rk4n has quit IRC13:22
ndipanovjohnthetubaguy, well I think so at least let's see once Jay and Dimitry respond13:22
ndipanovjohnthetubaguy, I think that it's ultimately important for the whole resource pools effort so we'll have to figure it out soon13:23
johnthetubaguyndipanov: totally agreed, its critical for that effort13:23
*** baoli has quit IRC13:25
bauzassdague: just out of my curiousity, do you have any documentation, specs or something about the project URL removal you're working on ?13:27
*** baoli has joined #openstack-nova13:27
lbeliveaundipanov: strange that this patch didn't break any unit tests regarding matching on 'physical_network', I'll have a look13:27
sdaguebauzas: the cross project spec, I can dig it up13:27
*** Bjoern_ has joined #openstack-nova13:28
bauzassdague: kewl, will look into13:28
bauzassdague: I was just explaining some rules about the API endpoint versioning internally, and I figured out that the project removal thing could be somehow in Mitaka too, right?13:28
*** baoli has quit IRC13:29
ndipanovlbeliveau, I don't think it was hooked in yet13:29
*** pbrooko has joined #openstack-nova13:29
sdaguebauzas: it is in mitaka13:29
*** kylek3h has joined #openstack-nova13:30
bauzasnice13:30
sdaguehttps://github.com/openstack/nova/blob/master/nova/api/openstack/rest_api_version_history.rst#21813:30
sdaguewe used a microversion to signal it13:30
bauzassdague: nice, was reading https://github.com/openstack/openstack-specs/blob/master/specs/service-catalog.rst13:31
sdagueright, that's the one13:31
bauzassdague: okay, haven't looked thru the microversions list13:31
bauzassdague: but since I'm working on the relnotes today, I guess I'll find out :)13:31
sdagueheh13:32
sdagueso we should get this in - https://review.openstack.org/#/c/265415/ as it completes our cors work13:32
sdaguemy only question is do we want that function somewhere else in the tree13:33
*** rewriteurl has joined #openstack-nova13:33
*** josecastroleon has joined #openstack-nova13:33
bauzassdague: oh good, that's already in the relnotes http://docs.openstack.org/releasenotes/nova/unreleased.html#b2-82613:33
sdaguebauzas: yeh, I think we've tried to be pretty strict that all microversions have reno entries13:34
bauzasawesome13:34
*** Bjoern_ has quit IRC13:34
sdaguehopefully we didn't miss any13:34
bauzasthat's something I'll verify pre-RC13:34
johnthetubaguybauzas: could you take a look at this one for me please, its the python-novaclient release: https://review.openstack.org/#/c/287191/13:34
bauzasjohnthetubaguy: sure13:35
*** mpavone has joined #openstack-nova13:35
*** ducttape_ has quit IRC13:35
*** mingyu has joined #openstack-nova13:37
*** baoli has joined #openstack-nova13:37
*** vivekd has joined #openstack-nova13:37
*** ekuris_ has quit IRC13:39
*** ptm_away has joined #openstack-nova13:39
*** pbrooko has quit IRC13:40
bauzasjohnthetubaguy: I'm preparing the milestone patch, any other blockers than the ones mentioned in https://etherpad.openstack.org/p/mitaka-nova-priorities-tracking ?13:40
johnthetubaguybauzas: probably, yes, I am still going through all the BPs13:40
bauzasjohnthetubaguy: about the resource-providers thingy, maybe considering it as non-blocker for M-3 but possibly be part of RC1 if we agree to cut FF after M-313:42
*** PaulMurray has quit IRC13:42
*** haomaiwa_ has joined #openstack-nova13:43
johnthetubaguybauzas: maybe, thats a call for tomorrow morning13:43
bauzasjohnthetubaguy: I can prepare the patch and respin it with a later SHA1 if needed13:43
johnthetubaguybauzas: as a WIP patch, makes sense, so we are ready to go quickly, when we are ready13:43
bauzasyeah of course, a DNM13:44
johnthetubaguybauzas: it depends if we get that fix up we were talking about this morning by the end of today, I feel.13:44
johnthetubaguyI think I said that yesterday, though...13:44
sdaguejohnthetubaguy: we should really get the cors patch in as well - https://review.openstack.org/#/c/26541513:45
bauzasokay, we can hold that13:45
*** qeelee has quit IRC13:46
*** jed56 has joined #openstack-nova13:46
openstackgerritScott DAngelo proposed openstack/nova: Api_version_request.matches does not accept a string or None  https://review.openstack.org/28719413:47
johnthetubaguysdague: oh, that would be good to have, will jump on that now13:48
*** rlrossit has joined #openstack-nova13:48
*** rk4n has joined #openstack-nova13:49
johnthetubaguysdague: oh, so we set the defaults until we can call into the lib for that?13:49
sdaguejohnthetubaguy: afaik we always have to set the defaults13:50
sdaguethe cors middleware defaults to nothing13:50
sdaguebut, at least this is not in the paste config any moe13:51
johnthetubaguysdague: I was thinking eventually we call into the middleware, a bit like with log.register_options(CONF) that is done by oslo_log: https://review.openstack.org/#/c/265415/4/nova/config.py@5413:52
sdaguemaybe, that was not my understanding13:53
*** rk4n_ has quit IRC13:53
*** doug-fish has quit IRC13:55
johnthetubaguysdague: anyways, its gets it out of paste.ini, which is the massive win13:56
*** haukebruno has quit IRC13:57
sdagueagreed13:58
openstackgerritScott DAngelo proposed openstack/nova: Api_version_request.matches does not accept a string or None  https://review.openstack.org/28720213:58
sdagueso everything is tweakable without upgrade implications later13:58
johnthetubaguysdague: agreed13:58
*** daemontool_ is now known as daemontool13:59
*** haomaiwa_ has quit IRC14:01
*** brianr-gone is now known as rosmaita14:01
*** haomaiwang has joined #openstack-nova14:01
*** haomaiwang has quit IRC14:01
*** haomaiwang has joined #openstack-nova14:02
sdaguescottda: is there a reason you didn't reuse the changeid?14:02
*** haomaiwang has quit IRC14:03
*** lucas-hungry is now known as lucasagomes14:03
*** haomaiwa_ has joined #openstack-nova14:03
scottdasdague: no, by bad14:04
scottdaI can fix that if that makes things better. Shall I?14:04
sdaguethere's a typo in https://review.openstack.org/287202 for python syntax error, if you fix that should be good14:04
*** bapalm has quit IRC14:05
sdaguewell, it was just confusing, because it means the tab I had up with the other change didn't refresh after14:05
*** jwcroppe_ is now known as jwcroppe14:05
*** jbernard_ has joined #openstack-nova14:06
*** bapalm has joined #openstack-nova14:06
*** EinstCrazy has joined #openstack-nova14:06
scottdasdague: Yes, sorry. It was just the way I got back to the original doc lost my commit, and then I had a cut-and-paste failure.14:06
openstackgerritSylvain Bauza proposed openstack/nova: Reno for VMWare opaque networks  https://review.openstack.org/28721014:07
openstackgerritScott DAngelo proposed openstack/nova: Api_version_request.matches does not accept a string or None  https://review.openstack.org/28720214:08
*** busterswt has joined #openstack-nova14:09
sdaguescottda: thanks14:09
sdague+214:09
*** mingyu has quit IRC14:09
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Use novaclient/utils directly and remove openstack/common (4/4)  https://review.openstack.org/28695914:10
*** pbrooko has joined #openstack-nova14:10
*** ccarmack has joined #openstack-nova14:10
*** mingyu has joined #openstack-nova14:10
*** pbrooko has quit IRC14:11
scottdasdague: cool, thanks. BTW, here is the direction I'm going with the Cinder changes to microversion header: https://review.openstack.org/#/c/286901/14:11
*** delattec has joined #openstack-nova14:11
*** pbrooko has joined #openstack-nova14:11
*** sahid has quit IRC14:12
*** ctrath has joined #openstack-nova14:12
kaisersHi Novas! Can someone pls give me a hint on if 'is_shared_instance_path' is explicitely configured somewhere (could not find it so far) or where it is derived from? I'm looking into a migration issue and so far cannot make out how to control this 'is_shared_instance_path' flag.14:13
*** doug-fish has joined #openstack-nova14:13
johnthetubaguykaisers: you mean when doing live-migration, you hit that?14:13
*** rodrigods has quit IRC14:14
*** rodrigods has joined #openstack-nova14:14
kaisersjohnthetubaguy: yep, in that context. I'm hitting a permission issue and i'm trying to find out what exactly nova is doing in that context...14:14
johnthetubaguykaisers: if its live-migrate, we write a file at one side, and see if it turns up at the other14:14
kaisers<---- looks baffled14:14
johnthetubaguykaisers: maybe we don't have permission for that directory?14:14
*** pbrooko has quit IRC14:15
kaisersjohnthetubaguy: the issue if with ownership of console.log. Regarding is_shared_instance_path i'm just trying to understand how it is controlled14:15
johnthetubaguykaisers: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L574214:15
kaiserss/if/is/14:15
*** mdrabe has joined #openstack-nova14:16
*** thumpba has joined #openstack-nova14:16
kaisersjohnthetubaguy: thanks!14:16
johnthetubaguykaisers: that eventually gets to here: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L554014:16
johnthetubaguykaisers: there are plans to change that, once we get the scheduler resource pools stuff in place14:17
*** ducttape_ has joined #openstack-nova14:17
kaisersjohnthetubaguy: ok, thanks for the hint! Will look deeper into the testing approach for now...14:17
*** ducttape_ has quit IRC14:17
*** pbrooko has joined #openstack-nova14:18
pkoniszewskikaisers: you mean that LM fails because nova can't claim ownership on console.log ?14:18
*** irenab_ has joined #openstack-nova14:19
*** irenab has quit IRC14:19
*** irenab_ is now known as irenab14:19
kaiserspkoniszewski: yep, more details here: https://bugs.launchpad.net/nova/+bug/154982814:19
openstackLaunchpad bug 1549828 in OpenStack Compute (nova) "permission denied on console.log during some migrations" [Undecided,New]14:19
*** subscope has quit IRC14:20
*** jbernard has left #openstack-nova14:20
kaisersconsole.log is created as root:root 600 ownership/permissions and nova fails on access because it runs as user nova.14:21
kaisersRunning a simple nova console-log corrects this (it does a chown on the log)14:21
pkoniszewskikaisers: last time i heard about it it was actually an issue with permissions to instances dir rather than console.log, but the error message was incorrectly pointing to console.log14:21
pkoniszewskikaisers: you might want to check this if it helps14:21
*** pbrooko has quit IRC14:22
kaiserspkoniszewski: Thanks for the hint. I tested this  behaviour multiple times today and can see the files beeing created with the root:root ownership during that process. So I'm pretty sure about that currently.14:23
*** ildikov has quit IRC14:23
pkoniszewskikaisers: here is my source https://www.redhat.com/archives/rdo-list/2014-September/msg00099.html14:23
pkoniszewskikaisers: but if you are sure about it, then i can't find anything else in my head now14:24
kaiserspkoniszewski: Thanks anyway, this is interesting for me in a another context... :-D14:24
*** baoli has quit IRC14:24
*** bklei has joined #openstack-nova14:30
*** smatzek has quit IRC14:31
*** mriedem has joined #openstack-nova14:31
*** baoli has joined #openstack-nova14:31
*** mingyu has quit IRC14:32
*** erno has quit IRC14:32
mriedemhuzzah14:32
Sam-I-Ammriedem: moo.14:32
*** moshele has joined #openstack-nova14:33
*** baoli has quit IRC14:34
*** jichen has quit IRC14:35
*** mylu has joined #openstack-nova14:35
*** josecastroleon has quit IRC14:36
mriedemclaudiub: i'm seeing this in cinder py34 on stable/liberty, but i think we had something similar in nova at one point right?14:36
mriedemhttp://logs.openstack.org/77/286677/1/check/gate-cinder-python34/de93680/console.html#_2016-03-01_16_41_35_22114:36
*** baoli has joined #openstack-nova14:37
*** Swami has joined #openstack-nova14:38
*** burt has joined #openstack-nova14:39
*** whenry_ has joined #openstack-nova14:40
openstackgerritRadomir Dopieralski proposed openstack/nova: Don't use locals() and globals(), use a dict instead  https://review.openstack.org/28708514:41
claudiubmriedem: no, it was quite different. In the nova hyper-v driver, the original trace was being lost at one point and I've fixed that. In this case, the issue is that LOG.exception is called when there is no exception.14:43
*** baoli has quit IRC14:44
*** sahid has joined #openstack-nova14:44
mriedemclaudiub: yeah, i remember pushing a patch for that14:44
mriedemit was using exc_info=True when there wasn't an exception14:44
*** mgoddard_ has joined #openstack-nova14:44
mriedemhttps://github.com/openstack/cinder/blame/stable/liberty/cinder/volume/drivers/windows/vhdutils.py#L21314:44
claudiubmriedem: anyways LOG.exception should only be called in an except: block14:44
*** RuiChen has quit IRC14:45
claudiubmriedem: yeah, looking at the same line. chaning from LOG.exception to LOG.error should fix this.14:45
mriedemyup14:45
claudiubmriedem: is there a bug open for this?14:45
mriedemhttps://bugs.launchpad.net/cinder/+bug/155226114:45
openstackLaunchpad bug 1552261 in Cinder "gate-cinder-python34 fails hyper-v tests with "AttributeError: 'NoneType' object has no attribute '__context__'"" [Undecided,Confirmed]14:45
claudiubwe should send a fix.14:46
*** mpavone has quit IRC14:46
*** mylu has quit IRC14:46
*** Swami_ has joined #openstack-nova14:46
*** RuiChen has joined #openstack-nova14:46
claudiubmriedem: cool, I'll send a commit asap.14:46
*** gokrokve has joined #openstack-nova14:47
*** mgoddard has quit IRC14:47
*** baoli has joined #openstack-nova14:48
*** baoli has quit IRC14:48
*** baoli has joined #openstack-nova14:48
*** knikolla has joined #openstack-nova14:49
*** alejandrito has joined #openstack-nova14:49
*** Swami has quit IRC14:50
*** baoli has quit IRC14:51
*** gokrokve has quit IRC14:52
*** baoli has joined #openstack-nova14:52
*** doug-fish has quit IRC14:52
*** ducttape_ has joined #openstack-nova14:53
*** doug-fish has joined #openstack-nova14:54
*** baoli has quit IRC14:54
*** ivase has joined #openstack-nova14:54
*** pbrooko has joined #openstack-nova14:54
*** pradk has joined #openstack-nova14:55
*** josecastroleon has joined #openstack-nova14:55
openstackgerritSylvain Bauza proposed openstack/nova: Reno for VMWare opaque networks  https://review.openstack.org/28721014:55
openstackgerritSylvain Bauza proposed openstack/nova: Reno for Ironic api_version opt deprecation  https://review.openstack.org/28724414:55
*** liverpooler has quit IRC14:56
*** xyang1 has joined #openstack-nova14:56
*** doug-fish has quit IRC14:56
*** ildikov has joined #openstack-nova14:56
*** doug-fish has joined #openstack-nova14:57
*** ctrath has quit IRC14:57
*** mpavone has joined #openstack-nova14:58
*** ctrath has joined #openstack-nova14:58
openstackgerritDan Smith proposed openstack/nova: DNM: Canary test  https://review.openstack.org/28724814:58
*** gongysh has joined #openstack-nova14:58
*** smatzek has joined #openstack-nova14:59
claudiubmriedem: done: https://review.openstack.org/287245  This issue doesn't affect master because the *utils have been replaced by os-win.14:59
*** mgoddard_ has quit IRC14:59
mriedemclaudiub: yup, thanks14:59
*** mgoddard has joined #openstack-nova14:59
mriedemsdague: dansmith: tonyb: claudiub: mikal: can we get stable eyes on this kilo patch? https://review.openstack.org/#/c/273430/ it is a failure seen in the linuxbridge job in kilo15:00
*** jschlueter|food is now known as jschlueter15:00
*** haomaiwa_ has quit IRC15:01
*** haomaiwang has joined #openstack-nova15:01
*** ctrath has quit IRC15:01
*** sridhargaddam has quit IRC15:03
*** edtubill has joined #openstack-nova15:04
*** ctrath has joined #openstack-nova15:04
*** sfinucan has joined #openstack-nova15:07
*** gcb_ has joined #openstack-nova15:07
*** mrkz has joined #openstack-nova15:07
*** zz_dimtruck is now known as dimtruck15:08
*** mtanino has joined #openstack-nova15:09
*** irenab has quit IRC15:09
*** q3k is now known as Q3K15:10
openstackgerritRongze Zhu proposed openstack/nova: Check volume status before call detach-volume  https://review.openstack.org/28725315:10
*** irenab has joined #openstack-nova15:11
*** roeyc has quit IRC15:11
*** roeyc has joined #openstack-nova15:11
*** Q3K is now known as KEK15:11
*** KEK is now known as Guest3279415:12
*** Guest32794 is now known as TOPKEK15:12
bauzasjohnthetubaguy: FYI, I'm going to draft all the missing reno files using the https://etherpad.openstack.org/p/nova-milestone-reno etherpad and put them in the same branch https://etherpad.openstack.org/p/nova-milestone-reno15:12
bauzasoh snap15:12
bauzasin the same branch https://review.openstack.org/#/q/topic:reno_m3 I meant15:13
bauzasthat branch being mentioned in https://etherpad.openstack.org/p/mitaka-nova-priorities-tracking15:13
bauzasas a potential blocker, but we can cut M-3 whenever we want and merge those pre-RC115:14
bauzasthey're just in the pipe15:14
*** whenry_ has quit IRC15:14
bauzasdansmith: so, you saw johnthetubaguy's and mine's points on https://review.openstack.org/279313 ?15:15
johnthetubaguybauzas: cool, sounds like the right way to do it15:15
bauzasjohnthetubaguy: I haven't seen yet zuul state15:16
*** sudipto has joined #openstack-nova15:16
bauzasand how the gate is happy15:16
*** jaosorior has quit IRC15:16
sdaguewe've got an 8 hour delay, and I have no idea why15:16
*** jaypipes has joined #openstack-nova15:17
bauzasthat's what I'm seeing15:17
*** TOPKEK is now known as q3k15:17
*** mylu has joined #openstack-nova15:18
*** delatte has joined #openstack-nova15:18
*** delattec has quit IRC15:18
bauzassdague: mtreinish: btw. I guess you know that http://status.openstack.org//openstack-health/#/ is one-week delayed ? I guess due to some known problem ?15:18
bauzasjust for my knowledge15:19
sdague2 weeks15:19
mtreinishbauzas: yes, I'm trying to debug a very weird bug in: https://github.com/openstack-infra/puppet-subunit2sql/blob/master/files/subunit-gearman-worker.py15:19
bauzasokay, good to know15:20
mtreinishthe worker is stack tracing while trying to read subunit streams saying the file is closed15:20
mtreinishthe 1st one works but all the others fail15:20
*** pushkaru has joined #openstack-nova15:21
*** delattec has joined #openstack-nova15:22
*** baoli has joined #openstack-nova15:23
*** sridharg has joined #openstack-nova15:23
sdaguemtreinish: log?15:23
*** delatte has quit IRC15:24
efriedmtreinish, is this a performance-critical module?  Noticed (because working on similar locally) some opportunities to use comma rather than % in logging calls.15:24
mtreinishsdague: http://paste.openstack.org/show/488878/ is the traceback15:24
mtreinishI have a full log locally15:25
*** gcb_ has quit IRC15:25
*** josecastroleon has quit IRC15:25
jaypipesndipanov: excellent catch on the PCI devices series. really appreciate you reviewing that and preventing a disaster.15:26
*** vishwanathj_zzz has joined #openstack-nova15:26
ndipanovwell I wouldn't call it a disaster really in the grand scheme of things...15:26
ndipanovbut it's definitely an angle you guys seem to have missed :)15:27
*** vishwanathj_zzz is now known as vishwanathj15:27
jaypipesndipanov: totally.15:27
ndipanovbut I am not at all happy that we can't do it, and hate the whitelist crap deeply just for the record15:27
jaypipesdansmith: I have meetings this morning for next 1.5 hours unfortunately. are you looking into that compute_node_statistics() thing?15:27
*** sridharg has quit IRC15:28
edleafedansmith: maybe try a copy.deepcopy() there?15:28
johnthetubaguyalaski: are you good for a quick sync around cells things and the freeze?15:28
*** baoli has quit IRC15:28
*** ptm__ has joined #openstack-nova15:29
mtreinishsdague: the full log: http://blog.kortar.org/wp-content/uploads/2016/02/subunit-worker-A-debug.log215:29
*** vishwana_ has joined #openstack-nova15:29
alaskijohnthetubaguy: I'm intermittent atm, but can try15:29
sdaguemtreinish: out of curiosity, why is the stringio buffer closed manually?15:30
johnthetubaguyalaski: I am thinking this one has missed the boat now: https://blueprints.launchpad.net/nova/+spec/cells-cell015:30
*** sneti_ has joined #openstack-nova15:30
mtreinishthere was a bug a while ago where we exhausted the max number of open files on the system and everything crashed15:31
johnthetubaguyalaski: I am thinking we could merge this one today, if you are happy with it( https://review.openstack.org/#/c/272329/8) but in a way it depends on this one that we don't totally agree on yet: https://review.openstack.org/#/c/277543/3, but I am thinking we should probably just merge the current plan, and tweak as we go?15:31
*** vishwana_ is now known as vishwanathj_15:31
alaskijohnthetubaguy: yes, not all of the necessary code will make it in for that one15:31
*** vishwanathj has quit IRC15:31
*** baoli has joined #openstack-nova15:31
mtreinishsdague: so we added a bunch of closes explicitly to make sure we didn't leak anything15:32
alaskijohnthetubaguy: agreed on https://review.openstack.org/#/c/272329/8 , that can all be tweaked later if needed15:33
*** ptm_away has quit IRC15:33
johnthetubaguyalaski: OK, lets +2 that now, and get that in, and we can call that BP completed I think?15:33
*** rewriteurl has quit IRC15:34
*** baoli has quit IRC15:35
sdaguemtreinish: but a cStringIO doesn't actually have an fd, does it?15:35
alaskijohnthetubaguy: yeah, looks like it15:35
*** sneti_ has quit IRC15:35
*** atuvenie has quit IRC15:36
*** ctrath has quit IRC15:37
bauzasalaski: johnthetubaguy: well, I'm okay to merge https://review.openstack.org/#/c/272329/8 and will just leave a note in gerrit that the list of tables is subject to change15:37
mtreinishsdague: I don't think so, it was just us being overly cautious IIRC. We used to subprocess out to pipe things through subunit-1to2 and that was the real cause of the bug15:37
mriedembauzas: wow, that's a lot of release notes15:37
bauzasalaski: johnthetubaguyfair per se ?15:37
johnthetubaguybauzas: sounds fine to me15:37
alaskibauzas: works for me. all it means is the decorated db/api methods might change later15:37
*** baoli has joined #openstack-nova15:38
johnthetubaguybauzas: and chance you could look at this one? https://blueprints.launchpad.net/nova/+spec/host-state-level-locking15:38
mtreinishsdague: https://review.openstack.org/234465 was the real fix back then and https://review.openstack.org/234469 was the follow-on15:38
bauzasmriedem: indeed, that's a long story15:38
bauzasmriedem: I'm only at the beginning of my journey15:38
sdaguemtreinish: well the suspicious thing is - https://docs.python.org/2/library/stringio.html#StringIO.StringIO.close15:38
johnthetubaguyalaski: I marked this one as partially complete: https://blueprints.launchpad.net/nova/+spec/flavor-cell-api15:38
mtreinishsdague: we can remove the close() but it shouldn't matter because we should be done with that object since the stream was already inserted into the db15:38
*** ctrath has joined #openstack-nova15:39
*** mdrabe has quit IRC15:39
*** esracelik has quit IRC15:39
bauzasalaski: johnthetubaguy: yeah, I just think changing the connection context doesn't seem a problem, right ?15:39
sdagueyou are getting the error that it says will be caused by trying to use a stringio buffer that has been explicitly closed15:39
johnthetubaguybauzas: seems fine15:39
bauzasalaski: it's just a matter of changing the decorator, right ? no need to care of rolling upgrades ?15:39
*** yamahata has joined #openstack-nova15:39
*** harlowja_at_home has joined #openstack-nova15:40
alaskijohnthetubaguy: okay. we hit a hidden dependency on that one, so it will need to get pushed to N for full completion15:40
johnthetubaguybauzas: its new code only that hits that stuff, its a no op if the context is not changed anyways15:40
alaskibauzas: right15:40
mtreinishsdague: right, but the clsoe is only done after we're done with a stream it's the last line in that method15:40
johnthetubaguyalaski: the hidden dependency is on the flavor-db stuff, and its that API change?15:40
bauzasjohnthetubaguy: right, say that an old compute goes to the DB API per a remotable method, it will get the new connection string anyway15:40
alaskijohnthetubaguy: yeah.15:40
mtreinishsdague: it should be using the next stream from the queue which the log confirms is open (right before the stack trace)15:41
bauzasjohnthetubaguy: alaski: okay, lemme call Jenkins on that one15:41
*** ivase_ has joined #openstack-nova15:41
*** ivase has quit IRC15:41
johnthetubaguyalaski: I am just about to hit your big chain here, to see if there is anything that can make the freeze: https://blueprints.launchpad.net/nova/+spec/cells-scheduling-interaction15:42
alaskijohnthetubaguy: awesome. there's a lot up for that, but it doesn't complete the spec. I would love to see the code merged, but I'll need to repropose at least part of the spec for N15:42
*** gokrokve has joined #openstack-nova15:42
*** baoli has quit IRC15:43
johnthetubaguyalaski: gotcha, yeah, would be good to see some of that groundwork land, been on my TODO for way too long15:43
johnthetubaguythe review on my TODO that is15:43
*** baoli has joined #openstack-nova15:43
alaskiyeah, there's been a lot to untangle and get in place in order to move forward15:43
dansmithedleafe: I don't think that'll do it, but maybe15:44
dansmithjaypipes: I put a patch on top to try to figure out what I should do, yeah15:45
dansmithjaypipes: I also have meetings15:45
*** gokrokve has quit IRC15:47
sdaguemtreinish: ok, well that was exploding on the flush, which you've since removed. Is there a more recent trace?15:47
mtreinishsdague: that's the most recent I have15:47
sdagueok, curious what it looks like with the flush removed.15:48
mtreinishbut the worker is running with subunit2sql 1.3.0 (which still has the flush)15:48
sdagueah15:48
mtreinishI'll have to push a release for it to get pulled in15:48
bauzasjohnthetubaguy: re: https://blueprints.launchpad.net/nova/+spec/host-state-level-locking15:48
pkoniszewskiare we already in freeze?15:50
bauzasjohnthetubaguy:  so https://review.openstack.org/#/q/topic:bp/host-state-level-locking,n,z are most in a good shape, but I literally had zero time to review those open15:50
johnthetubaguypkoniszewski: not quite, but almost15:50
johnthetubaguypkoniszewski: what makes you ask?15:50
bauzaspkoniszewski: and the gate is 8 hour delayed, which makes us very close to the FF15:50
*** belmoreira has quit IRC15:50
*** zenoway has quit IRC15:51
*** Sukhdev has joined #openstack-nova15:51
pkoniszewskiwell, im testing all live migration patches, and just found out that all block live migrations are broken when we provide 'block_migration': 'auto' in the body15:51
*** jistr has quit IRC15:51
*** zenoway has joined #openstack-nova15:51
*** josecastroleon has joined #openstack-nova15:51
sdaguepkoniszewski: ok, file that as a critical bug15:52
sdaguethat can get resolved post freeze15:52
bauzasyup15:52
pkoniszewskiokay15:52
sdagueand keep digging into it to see if you can expose why15:52
pkoniszewskiwithout 'auto' thing everything still works fine15:52
pkoniszewskii can, found a root cause already15:53
*** gokrokve has joined #openstack-nova15:53
*** gokrokve has quit IRC15:53
sdaguepkoniszewski: ok, cool, yeh that's a bug that will clearly be addressed post feature freeze15:53
*** gokrokve has joined #openstack-nova15:53
pkoniszewskisdague: okay, thanks15:53
*** mdrabe has joined #openstack-nova15:53
*** zenoway has quit IRC15:53
*** ptm__ is now known as PaulMurray15:54
johnthetubaguypkoniszewski: yeah, thats totally a bug fix, lets get that up ASAP15:54
*** zenoway has joined #openstack-nova15:54
mriedemalaski: i'm still confused a bit by the db_req.request_spec stuff happening in https://review.openstack.org/#/c/278124/12/nova/objects/build_request.py15:54
andrearosa_webpkoniszewski: do you have already something to review?15:54
pkoniszewskiyes, thanks guys, wanted to be sure what you think about it15:54
PaulMurraypkoniszewski, let me know when you have a patch up and I'll look. Also currently building a system to test15:55
pkoniszewskinot yet, just discovered that's not my environment, need still to test if my fix won't break, e.g., Xen15:55
*** gongysh has quit IRC15:56
andrearosa_webpkoniszewski: ok15:56
pkoniszewskiandrearosa_web: PaulMurray: will let you know when I submit fix for that15:56
andrearosa_webthanks15:57
mtreinishsdague: I can push a release with the flush removed, I'm trying to get a local env spun up too (I lost my old snapshot on hp cloud)15:57
*** zenoway has quit IRC15:58
johnthetubaguymriedem: does this bit help? https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api_models.py#L15615:58
mriedemnope15:58
*** haplo37 has joined #openstack-nova15:59
*** baoli has quit IRC15:59
*** delattec has quit IRC15:59
johnthetubaguypkoniszewski: I can read through with my XenAPI hat on if thats uploaded15:59
pkoniszewskijohnthetubaguy: thanks, will let you know too15:59
*** baoli has joined #openstack-nova16:00
*** haomaiwang has quit IRC16:01
*** bklei has quit IRC16:01
mriedemjohnthetubaguy: alaski: so calling db_req.request_spec on the api model object actually performs the query to set the attribute from here? https://github.com/openstack/nova/blob/d411d60587e54dd364b7a329035726f2e281a9f7/nova/db/sqlalchemy/api_models.py#L15616:01
*** delattec has joined #openstack-nova16:01
*** haomaiwang has joined #openstack-nova16:01
johnthetubaguymriedem: i think its loaded here, when you read it in from the DB, its not a lazy load: https://review.openstack.org/#/c/278124/12/nova/objects/build_request.py@6216:02
johnthetubaguymriedem: on save it looks like just the id gets updated, just getting my head around that bit atm16:03
*** matrohon has quit IRC16:03
johnthetubaguymriedem: so I see your questions now16:03
*** pcaruana has quit IRC16:05
*** fawadkhaliq has joined #openstack-nova16:06
*** jaypipes has quit IRC16:08
*** moshele has quit IRC16:08
alaskimriedem: yeah, it does the additional db lookup to pull that relationship. I can modify that method to pull the request_spec now that I learned the trick, it's the save that I can't fix that way16:09
*** READ10 has joined #openstack-nova16:10
mriedemalaski: the trick being some onload flag or something in the db model?16:10
mriedemi know i've seen something like that before16:10
*** armax has joined #openstack-nova16:11
*** daemontool_ has joined #openstack-nova16:12
*** daemontool has quit IRC16:13
*** fawadkhaliq has quit IRC16:14
*** bauzas_ has joined #openstack-nova16:14
*** mrkz1 has joined #openstack-nova16:14
*** mrkz has quit IRC16:15
*** rushiagr has quit IRC16:15
*** bauzas has quit IRC16:15
*** edtubill has quit IRC16:15
*** burt has quit IRC16:15
*** thorst_ has quit IRC16:15
*** briancurtin has quit IRC16:15
*** johnthetubaguy has quit IRC16:15
*** markmcclain has quit IRC16:15
*** sulo has quit IRC16:15
*** alaski has quit IRC16:15
*** bauzas_ is now known as bauzas16:15
*** alaski_ has joined #openstack-nova16:15
*** johnthetubaguy has joined #openstack-nova16:15
*** markmcclain has joined #openstack-nova16:15
pkoniszewskiandrearosa_web: PaulMurray: johnthetubaguy: in case you want to know what the problem with block live migration is - https://bugs.launchpad.net/nova/+bug/155230316:15
openstackLaunchpad bug 1552303 in OpenStack Compute (nova) "Block live migrations are broken when nova calculates live migration type by itself" [Critical,In progress] - Assigned to Pawel Koniszewski (pawel-koniszewski)16:15
*** thorst_ has joined #openstack-nova16:16
*** burt has joined #openstack-nova16:16
*** briancurtin has joined #openstack-nova16:16
*** sulo has joined #openstack-nova16:17
*** alaski_ is now known as alaski16:17
*** jplopezgu_ has joined #openstack-nova16:17
*** rushiagr has joined #openstack-nova16:17
andrearosa_webpkoniszewski: I'll have a look, thanks16:17
*** jplopezgu_ is now known as jlopezgu16:18
johnthetubaguypkoniszewski: there are cases where the object was not set, on purpose, to mean None, as the object was not nullable16:18
johnthetubaguypkoniszewski: not sure if thats related16:18
alaskimriedem: it may clarify things to know that the build_request object has one purpose, this method in https://review.openstack.org/#/c/263927/9/nova/objects/build_request.py16:18
alaskimriedem: so request_spec is needed every time and it'll be better to just ensure it's there when querying the db16:19
*** EinstCrazy has quit IRC16:19
*** yassine__ has quit IRC16:20
johnthetubaguyalaski: and we don't just use the request_spec because there might be API changes before the instance hits the cell database?16:21
*** josecastroleon has quit IRC16:21
pkoniszewskijohnthetubaguy: I think that's fine, because if object is not set everything will read from block_migration (at least this is not libvirt case), but if object is there we have a mismatch between block_migration and migrate_data.block_migration16:21
alaskijohnthetubaguy: because the request_spec doesn't containg things like display_name, task_state, security_groups, etc...16:22
*** Kevin_Zheng has quit IRC16:22
*** jaypipes has joined #openstack-nova16:22
johnthetubaguyalaski: ah, good point16:22
*** Swami__ has joined #openstack-nova16:22
johnthetubaguyalaski: I am just thinking about when metata data gets updated in the API before the instance makes it to a child cell DB, etc16:23
alaskijohnthetubaguy: task and vm state checking prevent that16:24
johnthetubaguyalaski: oh, I thought we allowed that during the build process right now16:24
*** delatte has joined #openstack-nova16:24
alaskiI thing the only thing that can be updated before it's active is the locked status16:25
johnthetubaguyoh, OK, thats good news16:25
alaskihttp://git.openstack.org/cgit/openstack/nova/tree/nova/compute/api.py#n323316:25
johnthetubaguyalaski: phew16:26
*** delattec has quit IRC16:26
*** suresh12 has joined #openstack-nova16:26
*** Swami_ has quit IRC16:26
johnthetubaguyalaski: I am thinking about this bit of code really: https://review.openstack.org/#/c/278125/11/nova/compute/api.py@96916:27
*** mylu has quit IRC16:28
*** josecastroleon has joined #openstack-nova16:28
*** gongysh has joined #openstack-nova16:28
alaskiyeah, there will be a small window where locked_by could be updated on the build_request and not make it to the instance, so we should consider a sync16:29
*** zenoway has joined #openstack-nova16:30
*** mylu has joined #openstack-nova16:32
openstackgerritThomas Maddox proposed openstack/nova: [WIP - DO NOT MERGE] Testing LXC Gate  https://review.openstack.org/27479216:33
*** mylu has quit IRC16:33
*** zenoway has quit IRC16:34
johnthetubaguyalaski: I just wanted to check about skipping the object version update in this one: https://review.openstack.org/#/c/278598/10 I guess thats because it has not been used yet?16:34
*** eglynn has joined #openstack-nova16:34
alaskijohnthetubaguy: right. That object is not used or sent across any wires yet16:35
openstackgerritDan Smith proposed openstack/nova: DNM: Canary test  https://review.openstack.org/28724816:35
openstackgerritDan Smith proposed openstack/nova: Make compute_node_statistics() use new schema  https://review.openstack.org/28693116:35
*** david-lyle has quit IRC16:36
*** Marga__ has joined #openstack-nova16:36
*** atuvenie has joined #openstack-nova16:37
mriedemalaski: johnthetubaguy: ok, i'm done commenting on https://review.openstack.org/#/c/278124/ for now :)16:37
*** jwcroppe has quit IRC16:37
openstackgerritThomas Maddox proposed openstack/nova: [WIP - DO NOT MERGE] Testing LXC Gate  https://review.openstack.org/27479216:37
*** david-lyle has joined #openstack-nova16:38
alaskimriedem: cool. thanks for the review.16:38
*** yamahata has quit IRC16:39
*** apoorvad has joined #openstack-nova16:40
*** krtaylor has quit IRC16:41
*** mylu has joined #openstack-nova16:44
*** mylu has quit IRC16:45
*** ildikov has quit IRC16:46
*** sc68cal has joined #openstack-nova16:46
mriedemi'm quite proud of16:47
mriedem'spanish spanish spanish create spanish already created'16:47
mriedemwhen presidente trump builds the wall we won't have to care about i18n16:47
danpblol16:48
thomasemD=16:49
bauzasmriedem: speak for you, I've been told that opensource is an US thing16:49
bauzasso we prefer buying Microsoft licences for our schools16:50
mriedemb/c ms pays people to translate their products?16:50
bauzasbecause it's Microsoft France (c)16:50
bauzas(and b/c MS does have an incubator in France...)16:51
*** belmoreira has joined #openstack-nova16:53
mriedemdansmith: is there anything up with dash? i get 401s after new gerrit was rolled out awhile back16:53
dansmithmriedem: it's working for me.. are you up to date?16:53
mriedemjust recloned it16:53
mriedemshould be using my public ssh key that i use for git review right?16:53
alaskimriedem: you might need the -P option16:54
dansmithit uses http now16:54
mriedemoooo16:54
alaskimriedem: if you check your gerrit settings there's a generated password in there that you use16:54
dansmithah yeah16:54
* alaski went through this exercise a bit ago16:54
dansmiththat's the http change16:54
*** ttx has quit IRC16:54
mriedemhttps://review.openstack.org/#/settings/http-password16:54
mriedemso i have to generate a password and supply that to -P?16:54
*** ijw has joined #openstack-nova16:55
*** jwcroppe has joined #openstack-nova16:55
alaskiyeah, if there's not already one there16:55
*** mjura has quit IRC16:55
*** ttx has joined #openstack-nova16:56
mriedemok, i seem to be in16:56
mriedemmayhap i'll update the readme16:57
dansmithcool16:57
*** efried has quit IRC16:57
*** josecastroleon has quit IRC16:58
mriedemooo cells meeting16:59
johnthetubaguyalaski: it feels like if we get to this one: https://review.openstack.org/#/c/263925 life will be easier next release as we can rely on instance mappings being present for all new instances, I guess?17:00
*** efried has joined #openstack-nova17:00
*** haomaiwang has quit IRC17:01
*** penick has quit IRC17:01
*** haomaiwang has joined #openstack-nova17:01
*** speller has quit IRC17:01
mriedemmgagne: see, that wasn't so hard17:04
*** kfarr has joined #openstack-nova17:04
*** zenoway has joined #openstack-nova17:05
*** lixiaoy1 has joined #openstack-nova17:06
*** scheuran has quit IRC17:07
*** anush has joined #openstack-nova17:07
*** baoli has quit IRC17:09
*** yamahata has joined #openstack-nova17:09
*** zenoway has quit IRC17:09
*** rdopiera has quit IRC17:10
*** baoli has joined #openstack-nova17:11
*** e0ne has quit IRC17:11
*** Swami__ has quit IRC17:12
*** ildikov has joined #openstack-nova17:12
dansmithjaypipes: your result set was a dict, but where the keys were Decimal() objects.. so I slapped a conversion in there to int-ify those things on return, which I think will solve it.. waiting for the test results to see17:12
*** achanda has joined #openstack-nova17:12
*** rk4n_ has joined #openstack-nova17:13
*** delattec has joined #openstack-nova17:14
*** mylu has joined #openstack-nova17:14
*** roeyc has quit IRC17:15
*** suresh12 has quit IRC17:15
*** jaugustine has joined #openstack-nova17:15
*** rk4n has quit IRC17:16
dansmithmriedem: thanks17:16
mriedem:)17:17
*** rfolco has quit IRC17:17
edleafedansmith: some of those values can be None, according to my tests17:17
edleafedansmith: I left a comment on the patch17:17
dansmithedleafe: ah, crap17:17
*** MVenesio has quit IRC17:17
*** delatte has quit IRC17:17
openstackgerritSylvain Bauza proposed openstack/nova: Reno for libvirt libosinfo with OS  https://review.openstack.org/28732817:17
dansmithedleafe: in my sample they were all ints.. let's let it finish to see if this works for that case and then I'll push up a tweak17:17
*** jlanoux has quit IRC17:18
*** mylu has quit IRC17:19
*** salv-orl_ has joined #openstack-nova17:19
edleafedansmith: I added a 'print(results)' to the method, and got this output on the first two calls when running the tests: http://paste.openstack.org/show/489013/17:20
*** fawadkhaliq has joined #openstack-nova17:21
dansmithedleafe: Returning compute node stats: {'count': 1, 'vcpus_used': Decimal('3'), 'local_gb_used': Decimal('0'), 'memory_mb': Decimal('8164'), 'current_workload': Decimal('3'), 'vcpus': Decimal('4'), 'running_vms': Decimal('3'), 'free_disk_gb': Decimal('77'), 'disk_available_least': Decimal('69'), 'local_gb': Decimal('77'), 'free_ram_mb': Decimal('6116'), 'memory_mb_used': Decimal('2048')}17:21
dansmithis what I saw in the actual run17:21
dansmithwhich is why tests that involve the database are very useful!17:22
*** salv-orlando has quit IRC17:22
edleafedansmith: heh :)17:22
*** jaypipes has quit IRC17:22
*** Bjoern_ has joined #openstack-nova17:23
edleafedansmith: I looked at the original code for that method, which did:17:23
edleafe    return {field: int(result[idx] or 0)17:23
edleafe            for idx, field in enumerate(fields)}17:23
edleafeso I thought None was a possibility.17:23
dansmithint(None) will give you a typeerror,17:24
dansmithso I think not17:24
*** mriedem has left #openstack-nova17:24
dansmithoh17:24
dansmithright, or 0 is inside17:24
dansmithso yeah17:24
edleafeyeah17:24
*** mriedem has joined #openstack-nova17:24
*** krtaylor has joined #openstack-nova17:24
dansmithcool, well, I have another short meeting now and then this should be done and I'll push up the change17:25
*** efried has quit IRC17:25
edleafeyay meetings!17:25
*** delatte has joined #openstack-nova17:25
* edleafe has one in 5 minutes too17:25
*** ssurana has joined #openstack-nova17:26
*** achanda has quit IRC17:27
*** kiran-r has quit IRC17:28
*** efried has joined #openstack-nova17:28
*** delattec has quit IRC17:28
*** sfinucan has quit IRC17:30
*** ssurana has quit IRC17:30
*** cdent has joined #openstack-nova17:30
*** rk4n_ has quit IRC17:30
*** MVenesio has joined #openstack-nova17:32
*** ssurana has joined #openstack-nova17:32
*** PaulMurray has quit IRC17:32
*** andrearosa_web has left #openstack-nova17:33
*** Swami has joined #openstack-nova17:33
*** boris-42 has joined #openstack-nova17:36
*** jaypipes has joined #openstack-nova17:36
*** vivekd has quit IRC17:37
*** efried has quit IRC17:37
*** danpb has quit IRC17:39
*** zenoway has joined #openstack-nova17:40
*** sudipto has quit IRC17:41
*** claudiub has quit IRC17:41
*** browne has joined #openstack-nova17:43
*** vilobhmm11 has joined #openstack-nova17:43
*** bjornar has joined #openstack-nova17:43
*** mpavone has quit IRC17:43
*** zenoway has quit IRC17:45
*** suresh12 has joined #openstack-nova17:45
*** vilobhmm111 has joined #openstack-nova17:45
*** achanda has joined #openstack-nova17:45
*** jaypipes has quit IRC17:47
*** vilobhmm11 has quit IRC17:47
openstackgerritMatt Riedemann proposed openstack/nova: Add release note for nova using neutron mtu value for vif plugging  https://review.openstack.org/28734417:48
mriedembauzas: ^17:48
mriedemkevinbenton: ^17:48
mriedemi figure that's useful as a release note for operators to know that the mtu value is now coming from neutron rather than nova config17:48
openstackgerritSylvain Bauza proposed openstack/nova: Add a cell and host mapping utility to nova-manage  https://review.openstack.org/27056517:50
*** efried has joined #openstack-nova17:51
*** rfolco has joined #openstack-nova17:52
*** browne has quit IRC17:53
*** gongysh has quit IRC17:55
*** haplo37 has quit IRC17:55
bauzasmriedem: had a few points on https://review.openstack.org/#/c/287344/1 but definitely thanks for helping me17:55
*** Jeffrey4l has quit IRC17:56
bauzasmriedem: or maybe upgrades, but I guess fixes isn't the good section17:57
* bauzas bbiab17:57
*** gokrokve has quit IRC17:58
*** ssurana has quit IRC17:58
*** browne has joined #openstack-nova17:59
*** josecastroleon has joined #openstack-nova18:00
*** derekh has quit IRC18:00
*** whenry_ has joined #openstack-nova18:00
*** josecastroleon has quit IRC18:00
*** ssurana has joined #openstack-nova18:00
*** Sukhdev has quit IRC18:00
*** haomaiwang has quit IRC18:01
*** atuvenie has quit IRC18:01
*** haomaiwang has joined #openstack-nova18:01
*** lucasagomes is now known as lucas-dinner18:01
openstackgerritMatt Riedemann proposed openstack/nova: Add release note for nova using neutron mtu value for vif plugging  https://review.openstack.org/28734418:01
*** belmoreira has quit IRC18:03
*** e0ne has joined #openstack-nova18:04
*** cdent has left #openstack-nova18:06
*** ctrath has quit IRC18:06
johnthetubaguy#success Nova is down to four pending blueprints for feature freeze now (https://blueprints.launchpad.net/nova/mitaka), sort of one day left. Better than it was this morning at least.18:06
openstackstatusjohnthetubaguy: Added success to Success page18:07
*** suresh12 has quit IRC18:08
*** ctrath has joined #openstack-nova18:10
*** nic has joined #openstack-nova18:10
openstackgerritRongze Zhu proposed openstack/nova: Check volume status before call detach-volume  https://review.openstack.org/28725318:10
*** efried has quit IRC18:10
*** efried has joined #openstack-nova18:11
*** mgoddard has quit IRC18:11
*** david-lyle has quit IRC18:12
*** suresh12 has joined #openstack-nova18:12
*** david-lyle has joined #openstack-nova18:13
openstackgerritPushkar Umaranikar proposed openstack/nova: "nova list-extensions" not showing summary for all  https://review.openstack.org/28613018:13
*** suresh12 has quit IRC18:14
*** ihrachys has quit IRC18:14
*** suresh12 has joined #openstack-nova18:14
*** ccarmack has quit IRC18:14
openstackgerritPushkar Umaranikar proposed openstack/nova: "nova list-extensions" not showing summary for all  https://review.openstack.org/28613018:16
*** lpetrut has quit IRC18:16
_diana_dims_: My apologies for the delay (I'm on maternity leave). I'll try testing the oslo service changes today, and let you know if they fix the nova heartbeat issue. Thanks for keeping me in the loop.18:17
*** mriedem has quit IRC18:18
*** jwcroppe has quit IRC18:19
*** penick has joined #openstack-nova18:21
*** angdraug has joined #openstack-nova18:21
*** zenoway has joined #openstack-nova18:21
*** jichen has joined #openstack-nova18:22
openstackgerritPawel Koniszewski proposed openstack/nova: Make block_migration and migrate_data.block_migration consistent  https://review.openstack.org/28736318:22
*** mylu has joined #openstack-nova18:25
*** sahid has quit IRC18:28
*** mriedem has joined #openstack-nova18:29
*** e0ne has quit IRC18:29
*** e0ne has joined #openstack-nova18:30
*** doug-fish has quit IRC18:30
*** ctrath has quit IRC18:30
*** doug-fish has joined #openstack-nova18:31
*** e0ne has quit IRC18:31
*** e0ne has joined #openstack-nova18:32
*** ctrath has joined #openstack-nova18:32
*** doug-fis_ has joined #openstack-nova18:34
*** mylu has quit IRC18:35
*** doug-fish has quit IRC18:35
*** mylu has joined #openstack-nova18:35
*** zhangjn has quit IRC18:37
*** zhangjn has joined #openstack-nova18:38
*** doug-fis_ has quit IRC18:39
*** ccarmack has joined #openstack-nova18:40
*** alejandrito has quit IRC18:43
*** lpetrut has joined #openstack-nova18:45
*** MVenesio has quit IRC18:46
*** MVenesio has joined #openstack-nova18:48
*** anush has quit IRC18:49
*** haplo37 has joined #openstack-nova18:56
*** READ10 has quit IRC18:57
*** Bjoern_ is now known as Bjoern_zZzZzZzZ18:59
*** thorst_ has quit IRC19:00
*** Bulacco has joined #openstack-nova19:00
*** thorst has joined #openstack-nova19:01
*** haomaiwang has quit IRC19:01
*** thorst has quit IRC19:01
*** thorst has joined #openstack-nova19:01
*** haomaiwang has joined #openstack-nova19:01
*** Bjoern_zZzZzZzZ is now known as Bjoern_19:03
BulaccoHi all! I have a little problem. I'm doing a toy accademic project, and I need a little help. I have modified the parser to have a new custom parameter at that level https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/shell.py , next i add the parameter to the dictionary in that class https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/servers.py but I need to retrieve this parameter19:06
*** suresh12 has quit IRC19:06
BulaccoI modified the class https://github.com/openstack/nova/blob/master/nova/scheduler/chance.py and https://github.com/openstack/nova/blob/master/nova/objects/request_spec.py and https://github.com/openstack/nova/blob/master/nova/scheduler/utils.py but i need some info about how to link this class with the novaclient that i modified19:09
*** anush has joined #openstack-nova19:09
openstackgerritDan Smith proposed openstack/nova: DNM: Canary test  https://review.openstack.org/28724819:09
openstackgerritDan Smith proposed openstack/nova: Make compute_node_statistics() use new schema  https://review.openstack.org/28693119:09
*** daemontool_ has quit IRC19:10
Bulaccoknikolla: yesterday help me.. but I need some more info...19:11
*** doug-fish has joined #openstack-nova19:12
*** rk4n has joined #openstack-nova19:15
openstackgerritGábor Antal proposed openstack/nova: Replaced unittest and unittest2 to testtools  https://review.openstack.org/28520919:17
*** suresh12 has joined #openstack-nova19:18
openstackgerritMark Doffman proposed openstack/nova: PoC: Scheduler: Placement using random weights distribution.  https://review.openstack.org/28739719:19
*** abitha has joined #openstack-nova19:20
*** ijw_ has joined #openstack-nova19:20
dansmithdammit people.. stop increasing SERVICE_VERSION!19:21
*** Sukhdev has joined #openstack-nova19:21
*** rk4n has quit IRC19:21
*** Sukhdev has quit IRC19:22
* alaski decreases service version19:23
*** jed56 has quit IRC19:23
*** abitha has left #openstack-nova19:24
bjornarIs it possible to have availability zones composed of other az's?19:24
*** ijw has quit IRC19:24
alaskibjornar: I've never tested it, but my understanding is no. you can have nested aggregates but not nested azs19:25
bjornarI for example want to only give certain projects access to "leaf" "availablity zones"19:25
bjornarI think my case is shared by others, I have dc1, dc2 and dc3 that I want available as az's for certain projects, but hide from most. Visible for everyone should be a all-dcs az composed of the 3 others..19:26
*** ociuhandu has quit IRC19:27
alaskibjornar: you could propose that capability to Nova through a spec, https://specs.openstack.org/openstack/nova-specs/readme.html but I don't believe it exists today19:29
*** anush has quit IRC19:31
*** mvk has quit IRC19:31
openstackgerritIvan Udovichenko proposed openstack/nova: Fix path to nova-config-generator.conf file  https://review.openstack.org/28740319:32
*** ssurana has quit IRC19:33
*** doug-fis_ has joined #openstack-nova19:35
*** Sukhdev has joined #openstack-nova19:35
bjornaralaski, I sketched up something: https://blueprints.launchpad.net/nova/+spec/nested-availability-zones19:35
*** doug-fish has quit IRC19:36
openstackgerritAndrew Laski proposed openstack/nova: Add BuildRequest object  https://review.openstack.org/27812419:36
*** mylu has quit IRC19:36
openstackgerritIvan Udovichenko proposed openstack/nova: Fix path to nova-config-generator.conf file  https://review.openstack.org/28740319:36
*** mylu has joined #openstack-nova19:36
alaskibjornar: a spec will get more visibility. if you aren't planning to do the work yourself you could open a backlog spec https://specs.openstack.org/openstack/nova-specs/readme.html#backlog-specifications19:38
alaskimriedem: I pieced https://review.openstack.org/278124 back together after you shredded it19:40
*** davideagnello is now known as davideagnello__19:40
*** anush has joined #openstack-nova19:41
bjornaralaski, I just hope someone will pick up the idea and implement :)19:43
bjornaralaski, afaik, this is a missing feature19:43
openstackgerritBaodong (Robert) Li proposed openstack/nova: [WIP] use neutron ironic port mapping  https://review.openstack.org/28740819:44
*** davideagnello__ is now known as davide85_19:45
mriedemalaski: alright19:45
*** doug-fish has joined #openstack-nova19:46
mriedemSwami: here is the next dvr multinode job run with that tempest change to use floating IPs before and after live migration https://jenkins03.openstack.org/job/gate-tempest-dsvm-neutron-dvr-multinode-full/1581/console19:46
*** Sukhdev has quit IRC19:46
mriedemhopefully i didn't totally botch the tempest change this time19:46
mriedemalaski: so can you dumb down with_entities for me?19:47
*** doug-fis_ has quit IRC19:47
mriedemthat's just the joined result set?19:47
Swamimriedem: will take a look at it.19:47
mriedemso rather than just BuildRequest results we also get RequestSpec results19:48
mriedemala the tuple o doom19:48
*** ducttape_ has quit IRC19:48
alaskimriedem: right. unless I declare a backref on the model I don't get both results with just join, with_entities gets us the tuple19:48
*** ducttape_ has joined #openstack-nova19:48
mriedemok, i'm used to the backref19:48
mriedemwhich is why this is all new and wild to me19:49
mriedemwas there a reason not to use a backref?19:49
alaskiI didn't fully realize the implications of leaving it off19:49
*** jwcroppe has joined #openstack-nova19:49
*** ijw_ has quit IRC19:50
*** jwcroppe has quit IRC19:50
alaskiit's something that should be easy to cleanup later, as it's just a sqlalchemy thing and doesn't affect the db19:50
mriedemok, and i guess we can leave it off19:50
mriedemright, no migratoin19:50
*** belmoreira has joined #openstack-nova19:50
mriedemStableObjectJsonFixture worked for you huh19:50
*** jwcroppe has joined #openstack-nova19:50
alaskiyep. it just wasn't there when I started19:50
*** belmoreira has quit IRC19:51
mriedemok, +219:53
alaski\o/19:53
*** tongli has joined #openstack-nova19:53
*** penick has quit IRC19:57
*** gszasz has quit IRC19:58
*** pushkaru has quit IRC19:58
mriedemlyarwood: you might want to put the bat signal out to mark for this https://review.openstack.org/#/c/266095/19:58
mriedembecause i'm ready to +219:58
*** rk4n has joined #openstack-nova19:59
mriedemjohnthetubaguy: bauzas: melwitt: if you're around, i'm +2 on the BuildRequest object patch now https://review.openstack.org/#/c/278124/19:59
bauzasmriedem: that's my duty tonight20:00
*** BobBall is now known as BobBall_AWOL20:00
*** BobBall_AWOL has quit IRC20:00
mriedemaye aye20:00
bauzasmriedem: in the meantime, I also updated the nova-manage change from melwitt20:00
*** kfarr has quit IRC20:00
*** rlrossit has quit IRC20:00
*** thorst has quit IRC20:01
*** haomaiwang has quit IRC20:01
*** penick has joined #openstack-nova20:01
*** haomaiwang has joined #openstack-nova20:01
*** thorst has joined #openstack-nova20:01
*** ducttape_ has quit IRC20:02
mriedemi'm trudging on through the rest of this series20:04
*** kfarr has joined #openstack-nova20:04
* melwitt looks20:04
melwittbauzas: I saw that, thanks for doing that!20:04
bauzasmelwitt: np20:05
bauzasit was a nit20:05
mriedemsdague: https://review.openstack.org/#/c/286841/20:05
mriedembackport city20:05
*** thorst has quit IRC20:05
openstackgerritMark Sturdevant proposed openstack/nova: Provide correct connector for evacuate terminate  https://review.openstack.org/26609520:06
*** thorst has joined #openstack-nova20:08
*** ctrath has quit IRC20:10
openstackgerritAndrew Laski proposed openstack/nova: Create BuildRequest object during boot process  https://review.openstack.org/27812520:13
openstackgerritAndrew Laski proposed openstack/nova: Make InstanceMappings.cell_id nullable  https://review.openstack.org/27859820:13
openstackgerritAndrew Laski proposed openstack/nova: Populate instance_mappings during boot  https://review.openstack.org/26392520:13
openstackgerritAndrew Laski proposed openstack/nova: Include CellMapping in InstanceMapping object  https://review.openstack.org/28630920:13
*** ducttape_ has joined #openstack-nova20:13
*** ducttape_ has quit IRC20:14
*** ducttape_ has joined #openstack-nova20:15
*** claudiub has joined #openstack-nova20:15
*** doug-fis_ has joined #openstack-nova20:17
*** claudiub has quit IRC20:17
*** claudiub has joined #openstack-nova20:17
*** doug-fish has quit IRC20:18
*** paul-carlton2 has quit IRC20:19
*** penick has quit IRC20:21
*** mylu has quit IRC20:22
*** apoorvad has quit IRC20:23
*** mylu has joined #openstack-nova20:23
mriedemodd...20:24
mriedemin the case of neutron,20:24
*** rlrossit_ has joined #openstack-nova20:25
mriedemif security groups are passed in on server create we wipe those out from the request and set to an empty list on the instance20:25
bauzasalaski: just a question on https://review.openstack.org/#/c/278124/13/nova/objects/build_request.py@4320:25
mriedemi guess that is fine since the security groups are on the port in neutron if port security is enabled20:25
bauzasalaski: since we accept a max number of instances per requests, then the display_name is different for all of these20:25
*** mylu has quit IRC20:25
bauzasalaski: all of them will have the same other fields, so that's fine, but display_name could be different20:26
*** mylu has joined #openstack-nova20:27
alaskibauzas: one build_request is created for each instance, so each display_name could be different.20:27
bauzasactually, I wonder the same for access_ip_v4...20:27
alaskithat being said, I didn't set display_name properly for that case20:27
alaskiin the next patch20:27
bauzasalaski: oh, really? I missed that20:27
alaskibauzas: it would be set in this one https://review.openstack.org/#/c/278125/12/nova/compute/api.py20:28
openstackgerritjichenjc proposed openstack/nova: Change wording of microversion bump about 503  https://review.openstack.org/28681920:28
bauzasalaski: because you know we create only one RequestSpec per request, right?20:28
*** andreykurilin__ has joined #openstack-nova20:28
bauzasoh sec, no20:28
bauzaswe are creating one RequestSpec per instance20:28
bauzassince it's in the api.create20:29
alaskiyeah20:29
alaskione per request would be much harder on the cleanup side20:29
bauzastbc, in the loop20:29
*** matrohon has joined #openstack-nova20:29
mriedemalaski: i'm reviewing https://review.openstack.org/#/c/278125/12/nova/compute/api.py atm so don't rev it if you're going to20:29
*** permalac has quit IRC20:30
*** doug-fish has joined #openstack-nova20:31
*** doug-fis_ has quit IRC20:33
mriedemso what's wrong with display_name?20:33
mriedemposted comments btw20:33
alaskimriedem: I've already rebased so no need to rev anything until comments happen20:33
alaskimriedem: despite display_name being set in base_options it can be updated in _create_db_entry_for_new_instance in the multiple boot scenariou20:35
*** haukebruno has joined #openstack-nova20:36
*** rcernin has quit IRC20:36
mriedemhmm20:37
mriedemhttps://github.com/openstack/nova/blob/master/nova/compute/api.py#L132320:37
mriedemsemi related20:37
mriedemwhy don't we store the launch_index in the build request20:37
mriedemor any bdm information?20:37
mriedemb/c they aren't exposed in the api?20:38
alaskiright20:38
*** claudiub has quit IRC20:39
mriedemoh i rue the day that someone adds a microversion to change what we return in the rest api20:39
*** ctrath has joined #openstack-nova20:39
mriedemb/c that will mean database migrations20:39
alaskiheh. that's likely to be me for tasks work :)20:40
alaskimriedem: the beauty of this table is the data is short lived so migrations won't take long20:40
mriedemi guess that's true20:41
alaskiafaict launch_index is not actually used anywhere20:41
alaskioh, it's exposed in metadata service. that seems to be it20:42
mriedemdoffm: https://review.openstack.org/#/c/221280/20:42
mriedemand like shutdown_terminate20:42
mriedemwhich is an ec2 thing20:42
*** Marga__ has quit IRC20:43
mriedem...did you keep in mind the ec2 API that's out of tree when defining the build_requests table? :)20:43
*** Marga_ has joined #openstack-nova20:43
openstackgerritTim Pownall proposed openstack/nova: Updating while loop for tar extraction in utils.py.  https://review.openstack.org/28742620:44
*** thumpba has quit IRC20:44
*** david-lyle has quit IRC20:45
*** david-lyle has joined #openstack-nova20:47
*** Marga_ has quit IRC20:50
*** rlrossit_ has quit IRC20:51
dims__diana_ : ping about https://review.openstack.org/#/c/286838/20:53
*** haplo37 has quit IRC20:53
dims__diana_ : thanks for testing!20:53
_diana_dims_: My pleasure! I'm back to work in two weeks & will be around more then. Cheers20:57
alaskimriedem: I did not :) I only looked at what is exposed in the OS API. Though we may end up needing things like shutdown_terminate and launch_index for creating the instance20:57
dims__diana_ : awesome! :)20:57
*** apoorvad has joined #openstack-nova20:59
*** kylek3h has quit IRC20:59
*** anush has quit IRC21:00
*** haomaiwang has quit IRC21:01
*** haplo37 has joined #openstack-nova21:01
*** mylu has quit IRC21:01
*** mylu has joined #openstack-nova21:01
*** haomaiwa_ has joined #openstack-nova21:01
*** browne has quit IRC21:02
*** jichen has quit IRC21:02
*** raildo is now known as raildo-afk21:02
*** mingyu has joined #openstack-nova21:03
*** kylek3h has joined #openstack-nova21:04
mriedemalaski: ok, i think i have my comments in https://review.openstack.org/#/c/278125/ then21:05
mriedemlooks like the display_name thing is the hairiest part21:05
mriedemi guess i haven't reviewed the tests yet21:05
*** haplo37 has quit IRC21:06
*** anush has joined #openstack-nova21:07
alaskiawesome. thanks for reviewing21:08
*** mingyu has quit IRC21:09
*** aysyd has quit IRC21:09
*** doug-fish has quit IRC21:09
*** smatzek_ has joined #openstack-nova21:10
*** smatzek has quit IRC21:11
mriedemadded comments to the test,21:11
mriedemwould be nice to merge those tests together21:11
mriedemand test for multpile instances21:12
alaskimriedem: bool_from_string won't do anything in this case, it doesn't consider '' to be true or false so it will just pass through to the default of False. it works out the same in the end but it's roundabout21:12
mriedemalaski: yeah, i only pointed it out since it's a common utility for converting strings to bools in nova21:12
alaskiI usually use it, but this is an odd case because it's True or '', it's not really converting a string to a bool21:13
*** doug-fish has joined #openstack-nova21:15
*** smatzek_ has quit IRC21:15
*** doug-fis_ has joined #openstack-nova21:16
openstackgerritLee Yarwood proposed openstack/nova: Retry partition path checks after calling kpartx  https://review.openstack.org/28744221:17
mriedemhttps://review.openstack.org/#/c/278598/11/nova/db/sqlalchemy/api_migrations/migrate_repo/versions/007_instance_mapping_nullable_cellid.py21:18
mriedemoops21:18
*** haplo37 has joined #openstack-nova21:20
*** doug-fish has quit IRC21:20
*** _ducttape_ has joined #openstack-nova21:21
*** doug-fish has joined #openstack-nova21:23
*** doug-fis_ has quit IRC21:25
*** ducttape_ has quit IRC21:25
*** kylek3h has quit IRC21:25
*** kylek3h has joined #openstack-nova21:26
mriedemalaski: dansmith: this doesn't require a version bump on the object? https://review.openstack.org/#/c/278598/11/nova/objects/instance_mapping.py21:27
dansmithit's never been over the wire I think he said21:27
alaskimriedem: it's never gone over the wire. there's no place for it to fail compatibility21:27
mriedema virgin21:27
*** kylek3h has quit IRC21:30
*** aunnam has joined #openstack-nova21:31
openstackgerritNikolai Korablin proposed openstack/nova: Typo fix in documentation  https://review.openstack.org/28745321:31
mriedemdoffm: ^21:34
mriedemyou can rub some olde english on that21:34
*** doug-fis_ has joined #openstack-nova21:35
*** kylek3h has joined #openstack-nova21:37
*** baoli has quit IRC21:38
*** ZZelle_ has joined #openstack-nova21:38
*** doug-fish has quit IRC21:38
bauzasalaski: mriedem: saw some discussion on https://review.openstack.org/#/c/278125/21:38
bauzasalaski: planning to respin soon ?21:38
bauzasalaski: I'd ideally like at least this one for Mitaka, because it could mean that we would have to wait for another cycle for knowing if existing instances are already having this21:39
bauzas(or we should think about some nova-manage method for having the existing running instances to be set using a RequestSpec + a BuildRequest)21:40
alaskimriedem: prepare to weep http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/api.py#n57121:41
*** MVenesio has quit IRC21:41
alaskibauzas: I'm working on it now, there are a few big changes to make21:41
bauzasokay21:42
*** Bulacco has quit IRC21:42
bauzasTBH, I'm thinking on providing some nova-manage method for the running instances like I said21:42
bauzasalaski: ^21:42
*** ndipanov has quit IRC21:42
mriedemalaski: ah, so given that and http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/api.py#n87021:43
mriedembase_options['config_drive'] is already a boolean21:43
bauzasbecause you know that I'm checking every instance if they have an existing ReqSpec and that could be something we could try to set21:43
mriedemand you don't need the check in your build request create code right?21:43
alaskibauzas: well, remember that build_request only exists until the instance is in the db. so it's not something we'll need for running instances21:43
*** doug-fish has joined #openstack-nova21:43
bauzasalaski: oh right21:43
bauzasI should be going to bed21:44
alaskimriedem: base_options['config_drive'] would be a valid boolean except for http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/api.py#n834 mangling it21:44
*** angdraug has quit IRC21:45
mriedemoh http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/api.py#n58421:45
mriedemreturn True if bool_val else ''21:45
mriedemthat does make me weep21:45
mriedemcomstud: !!!!21:45
*** browne has joined #openstack-nova21:46
*** doug-fis_ has quit IRC21:46
*** erlon has quit IRC21:46
*** krtaylor has quit IRC21:48
*** sc68cal has quit IRC21:48
*** matrohon has quit IRC21:49
*** eglynn has quit IRC21:49
*** suresh12 has quit IRC21:50
*** sc68cal has joined #openstack-nova21:50
*** neophy has joined #openstack-nova21:52
mriedem"who makes the sex with the queen bee?"21:53
*** armax has quit IRC21:55
*** Marga_ has joined #openstack-nova21:56
*** rlrossit has joined #openstack-nova21:58
*** armax has joined #openstack-nova21:59
*** haomaiwa_ has quit IRC22:01
*** haomaiwang has joined #openstack-nova22:01
*** thorst has quit IRC22:01
*** thorst has joined #openstack-nova22:02
*** BobBall_AWOL has joined #openstack-nova22:05
*** mingyu has joined #openstack-nova22:05
*** achanda has quit IRC22:06
*** thorst has quit IRC22:06
*** rk4n has quit IRC22:07
*** BobBall_1WOL has quit IRC22:08
*** mingyu has quit IRC22:08
*** mingyu has joined #openstack-nova22:08
mriedemholy docstring https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L62622:10
*** doug-fis_ has joined #openstack-nova22:10
*** doug-fish has quit IRC22:11
*** ssurana has joined #openstack-nova22:12
*** mingyu has quit IRC22:13
*** achanda has joined #openstack-nova22:13
*** e0ne has quit IRC22:14
*** rk4n has joined #openstack-nova22:15
*** mylu has quit IRC22:15
*** mylu has joined #openstack-nova22:15
*** e0ne has joined #openstack-nova22:17
*** rahulunair has joined #openstack-nova22:19
*** kfarr has quit IRC22:19
*** achanda has quit IRC22:20
mriedemalaski: melwitt: bauzas: doffm: some questions in https://review.openstack.org/#/c/270565/22:20
*** rk4n_ has joined #openstack-nova22:22
*** claudiub has joined #openstack-nova22:22
dansmithhmm, postgres does not like jay http://logs.openstack.org/31/286931/3/check/gate-tempest-dsvm-postgres-full/34eba38/logs/screen-n-api.txt.gz?level=TRACE#_2016-03-02_20_14_03_68522:23
dansmithguess he's not around to address it thouggh22:23
*** thorst has joined #openstack-nova22:23
*** claudiub has quit IRC22:24
*** rk4n has quit IRC22:24
*** claudiub has joined #openstack-nova22:24
*** rtheis has quit IRC22:25
mriedemoh wow22:26
*** ijw has joined #openstack-nova22:26
*** e0ne has quit IRC22:27
*** thorst has quit IRC22:27
*** suresh12 has joined #openstack-nova22:28
alaskimriedem: answered22:28
mriedemalaski: not sure i understand the thing on the name parameter, "Yes. But for now no user will have that set, unless a cellsv1 user is crazy enough to make one cell a v2 cell."22:32
*** edmondsw has quit IRC22:32
mriedemour cells v1 job in the gate has cells.name set in nova-cells.conf22:32
mriedemor are these things totally unrelated?22:32
alaskiI mean that no cellsv1 user should be running this command22:32
alaskiit's currently aimed at non cells users upgrading to cellsv222:33
*** rlrossit has quit IRC22:33
alaskiwe'll probably need to shore it up before v1 users migrate, in addition to needing multiple cells support22:33
mriedemthat's not how i read the reno22:33
mriedemand if no cellsv1 users are supposed to run this command, why do we mention using --config-file /etc/nova/nova-cells.conf?22:34
alaskicellsv1 users could use it, it would just be silly until we have multiple cells support22:34
alaskiit shows the basics of how this will be expanded to support v1 migrations22:35
edleafedansmith: ugh, postgres uses COALESCE. So obvious!22:35
alaskimriedem: that could all be ripped out for now22:35
mriedemalaski: but does doing said silly thing mess something up in your deployment? or just creates garbage data in your db that you have to manually clean up later?22:36
dansmithedleafe: mmkay22:36
melwittmriedem brought the hammer down on my patch22:36
*** anush has quit IRC22:36
mriedemand you don't know about said garbage until later when said garbage causes a bug in how we expect this to all be working22:36
edleafedansmith: I thought sqla was supposed to hide all those details22:37
alaskimriedem: it creates garbage data. it could potentially split instances between two cells claiming to have the same hosts.22:37
mriedemok....22:38
mriedemcan we prevent that?22:38
*** jaugustine has quit IRC22:38
mriedemlike if 'cells' in CONF: exit(1)?22:38
*** zenoway has quit IRC22:39
mriedemi just think we should be explicit about what this is and who should and shouldn't use it to avoid later pain22:39
*** Sukhdev has joined #openstack-nova22:39
mriedemdespite the big fat EXPERIMENTAL22:39
*** zenoway has joined #openstack-nova22:39
*** doug-fis_ has quit IRC22:39
alaskiwe could turn it of for current cells users. that doesn't prevent non cells users from running it twice and having the same effect22:40
alaskiand we really don't have anything to guarantee uniqueness22:40
mriedemwell, HostMapping.host is unique22:40
mriedemso if you run it twice, the 2nd time will fail on creating the duplicate HostMapping22:40
mriedembut you'll have created a garbage CellMapping22:40
doffmWhich isn't a disaster.22:41
doffmThere won't be any hosts in it.22:41
doffmIts just garbage data.22:41
mriedemso the running it twice thing vs running it once with nova-cells.conf as a cells v1 user are different concerns i have22:41
*** doug-fish has joined #openstack-nova22:42
*** rahulunair has quit IRC22:42
mriedemrunning it twice isn't the end of the world it sounds like, unless we later have code somewhere in nova that expects that if you have a CellMapping, it has hosts22:42
melwittI wonder if we could check for the hosts first and bail if we find some22:43
melwittHostMappings I mean22:43
doffmmelwitt: That fixes the 'run it twice' issue.22:43
mriedemright now HostMapping.cell_id is a non-nullable foreign key22:43
*** zenoway has quit IRC22:44
*** doug-fis_ has joined #openstack-nova22:44
mriedemunless you mean query the hostmappings table by host22:44
mriedembefore create22:44
doffmBut I think mriedem is more worried about cells v1 users running this incorrectly.22:44
mriedemyeah, i thin kso22:44
openstackgerritVasyl Saienko proposed openstack/nova: WIP Allow to update resource per single node.  https://review.openstack.org/28748522:45
mriedemanyway, i have to run soon, sorry for the last minute turd in your sandwich22:45
dansmithlol22:45
*** xyang1 has quit IRC22:45
melwitthaha, these are good points. I'll think on them22:46
*** vilobhmm111 has quit IRC22:46
*** doug-fish has quit IRC22:47
openstackgerritMatt Riedemann proposed openstack/nova: Add release note for nova using neutron mtu value for vif plugging  https://review.openstack.org/28734422:47
*** kylek3h has quit IRC22:48
*** knikolla has quit IRC22:48
mriedemok, i'll probably be back on later tonight after 8pm cst22:48
*** mriedem has quit IRC22:48
*** doug-fis_ has quit IRC22:48
*** haplo37 has quit IRC22:49
*** mylu has quit IRC22:49
*** ccarmack has quit IRC22:50
*** ctrath has quit IRC22:52
*** krtaylor has joined #openstack-nova22:52
*** yarkot_ has joined #openstack-nova22:53
*** sc68cal_ has joined #openstack-nova22:56
*** dims_ has quit IRC22:56
*** yarkot_ has quit IRC22:57
*** yarkot_ has joined #openstack-nova22:57
*** sc68cal has quit IRC22:58
*** mylu has joined #openstack-nova22:58
*** macsz has joined #openstack-nova22:59
*** yarkot_ has quit IRC22:59
*** Jeffrey4l has joined #openstack-nova23:00
*** busterswt has quit IRC23:00
*** macsz has quit IRC23:00
*** haomaiwang has quit IRC23:01
*** haomaiwang has joined #openstack-nova23:01
*** browne has quit IRC23:04
*** neophy has quit IRC23:06
*** browne has joined #openstack-nova23:06
*** browne has quit IRC23:06
*** ssurana has quit IRC23:06
*** ccarmack has joined #openstack-nova23:07
*** rk4n_ has quit IRC23:08
*** browne has joined #openstack-nova23:08
*** Sukhdev has quit IRC23:08
*** burt has quit IRC23:11
*** haukebruno has quit IRC23:11
*** ccarmack has quit IRC23:11
*** lpetrut has quit IRC23:11
*** y_sawai has joined #openstack-nova23:12
*** tongli has quit IRC23:14
*** zenoway has joined #openstack-nova23:15
*** mylu has quit IRC23:15
openstackgerritAndrew Laski proposed openstack/nova: Create BuildRequest object during boot process  https://review.openstack.org/27812523:16
openstackgerritAndrew Laski proposed openstack/nova: Make InstanceMappings.cell_id nullable  https://review.openstack.org/27859823:16
openstackgerritAndrew Laski proposed openstack/nova: Populate instance_mappings during boot  https://review.openstack.org/26392523:16
openstackgerritAndrew Laski proposed openstack/nova: Include CellMapping in InstanceMapping object  https://review.openstack.org/28630923:16
*** sheel has quit IRC23:17
*** mdrabe has quit IRC23:17
*** zenoway has quit IRC23:19
*** salv-orlando has joined #openstack-nova23:19
*** achanda has joined #openstack-nova23:20
*** zengine has quit IRC23:20
*** salv-orl_ has quit IRC23:22
*** rhagarty has quit IRC23:25
*** rhagarty_ has joined #openstack-nova23:25
*** dimtruck is now known as zz_dimtruck23:25
*** mylu has joined #openstack-nova23:27
*** mylu has quit IRC23:27
*** mylu has joined #openstack-nova23:27
*** achanda has quit IRC23:28
*** Sukhdev has joined #openstack-nova23:31
*** rhagarty_ has quit IRC23:32
*** mylu has quit IRC23:33
*** rhagarty has joined #openstack-nova23:34
*** browne has quit IRC23:36
*** ssurana has joined #openstack-nova23:39
*** browne has joined #openstack-nova23:39
*** rhagarty_ has joined #openstack-nova23:40
openstackgerritEd Leafe proposed openstack/nova: Make compute_node_statistics() use new schema  https://review.openstack.org/28693123:41
edleafedansmith: ^^ I think this should work for ifnull issues23:41
*** rhagarty has quit IRC23:42
*** MVenesio has joined #openstack-nova23:42
*** ssurana has quit IRC23:44
openstackgerritDavid Edery proposed openstack/nova: Fix failed snapshot on VMs with SRIOV ports  https://review.openstack.org/28751523:47
*** MVenesio has quit IRC23:48
*** salv-orlando has quit IRC23:50
*** salv-orlando has joined #openstack-nova23:51
openstackgerritDavid Edery proposed openstack/nova: Fix failed snapshot on VMs with SRIOV ports  https://review.openstack.org/28751523:52
*** pradk has quit IRC23:53
*** y_sawai has quit IRC23:54
*** Jeffrey4l has quit IRC23:55
*** delatte has quit IRC23:55
*** ssurana has joined #openstack-nova23:55
*** Jeffrey4l has joined #openstack-nova23:56
*** mrkz1 has quit IRC23:57

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