Wednesday, 2017-05-03

melwittthe instance string repr is a bytes string in python2, that's what the safe_encode does in the __repr__ function in ovo. the instance.display_name is of type unicode. I don't know if that's relevant to what's happening here00:00
*** thorst_afk has joined #openstack-nova00:00
*** catintheroof has joined #openstack-nova00:00
*** ssurana has joined #openstack-nova00:01
*** catintheroof has quit IRC00:01
*** harlowja has quit IRC00:04
*** catintheroof has joined #openstack-nova00:04
mriedemooo fun i pulled the journal file down from a failed job and i'm poking through the nova-compute logs using journalctl00:05
melwittsuper fun00:06
melwittthe OSJournalHandler is new as of about a month ago, fwiw00:07
mriedemyeah00:07
melwittif the syslog one was the old default, it seems like it should have been making the same traces. I don't know what the old handler was00:08
*** iceyao has joined #openstack-nova00:11
*** zz_dimtruck is now known as dimtruck00:13
*** dave-mccowan has quit IRC00:13
*** iceyao has quit IRC00:16
*** sbezverk_ has joined #openstack-nova00:19
*** sbezverk has quit IRC00:22
*** gjayavelu has quit IRC00:22
*** sbezverk has joined #openstack-nova00:23
*** sbezverk_ has quit IRC00:26
*** felipemonteiro has joined #openstack-nova00:26
*** nic1 is now known as nic00:32
*** nic has quit IRC00:32
*** browne has joined #openstack-nova00:33
*** catintheroof has quit IRC00:33
*** ijw has quit IRC00:35
*** sbezverk_ has joined #openstack-nova00:36
*** ijw has joined #openstack-nova00:36
*** ijw has quit IRC00:37
*** ijw has joined #openstack-nova00:37
*** sbezverk has quit IRC00:37
*** thorst_afk has quit IRC00:37
*** tovin07_ has joined #openstack-nova00:39
*** catintheroof has joined #openstack-nova00:39
*** iceyao has joined #openstack-nova00:39
*** mriedem has quit IRC00:40
*** markvoelker has quit IRC00:41
*** ijw has quit IRC00:41
*** markvoelker has joined #openstack-nova00:42
*** dtp has quit IRC00:42
*** artom has quit IRC00:43
*** thorst_afk has joined #openstack-nova00:44
*** artom has joined #openstack-nova00:46
*** thorst_afk has quit IRC00:46
*** markvoelker has quit IRC00:46
*** dave-mccowan has joined #openstack-nova00:47
*** dillaman has quit IRC00:52
*** browne has quit IRC00:56
*** yamamoto has joined #openstack-nova00:57
*** iceyao_ has joined #openstack-nova00:58
*** iceyao has quit IRC00:58
*** kevinz has joined #openstack-nova01:01
*** Apoorva_ has joined #openstack-nova01:02
*** phuongnh has joined #openstack-nova01:02
*** ssurana has quit IRC01:03
*** yamamoto has quit IRC01:05
*** Apoorva has quit IRC01:05
*** Apoorva_ has quit IRC01:07
*** mingyu has joined #openstack-nova01:07
*** catintheroof has quit IRC01:09
*** dillaman has joined #openstack-nova01:10
*** mingyu has quit IRC01:13
*** xinliang has quit IRC01:13
*** xinliang has joined #openstack-nova01:16
*** cNilesh has joined #openstack-nova01:23
*** Shunli has joined #openstack-nova01:24
*** litao has joined #openstack-nova01:25
*** kaisers has joined #openstack-nova01:26
openstackgerritmelanie witt proposed openstack/nova master: WIP Use six.text_type() when logging Instance object  https://review.openstack.org/46191201:27
*** cNilesh has quit IRC01:29
*** yolkfull__ has joined #openstack-nova01:29
*** kaisers_ has quit IRC01:30
*** cNilesh has joined #openstack-nova01:31
*** felipemonteiro has quit IRC01:31
*** mriedem has joined #openstack-nova01:36
openstackgerritZhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4)  https://review.openstack.org/45959301:37
*** gouthamr has quit IRC01:37
*** Jack_Iv has joined #openstack-nova01:41
*** nicolasbock has quit IRC01:41
*** Jack_Iv has quit IRC01:45
*** tbachman has quit IRC01:45
*** felipemonteiro has joined #openstack-nova01:45
*** thorst_afk has joined #openstack-nova01:47
*** zhurong has joined #openstack-nova01:51
*** aleph1 has quit IRC01:51
*** thorst_afk has quit IRC01:52
*** nicolasbock has joined #openstack-nova01:55
*** yamahata has joined #openstack-nova01:58
*** bkopilov_ has quit IRC01:58
*** jamielennox is now known as jamielennox|away02:03
*** zhurong has quit IRC02:07
*** Shunli has quit IRC02:10
*** tbachman has joined #openstack-nova02:14
*** nicolasbock has quit IRC02:17
*** jamielennox|away is now known as jamielennox02:17
*** mingyu has joined #openstack-nova02:18
openstackgerritjichenjc proposed openstack/nova master: Use plain routes list for limits endpoint instead of stevedore  https://review.openstack.org/45880502:19
openstackgerritjichenjc proposed openstack/nova master: Use plain routes list for hosts endpoint instead of stevedore  https://review.openstack.org/45923102:19
openstackgerritjichenjc proposed openstack/nova master: use plain routes list for hypervisor endpoint instead of stevedore  https://review.openstack.org/45923502:19
openstackgerritjichenjc proposed openstack/nova master: Use plain routes list for os-instance-usage-audit-log  endpoint instead of stevedore  https://review.openstack.org/46164702:21
*** ihrachys has quit IRC02:29
*** nicolasbock has joined #openstack-nova02:33
*** ijw has joined #openstack-nova02:33
openstackgerritjichenjc proposed openstack/nova master: Use plain routes list for os-quota-classes endpoint instead of stevedore  https://review.openstack.org/46164102:34
openstackgerritjichenjc proposed openstack/nova master: Use plain routes list for os-migrations endpoint instead of stevedore  https://review.openstack.org/46162802:36
*** dave-mccowan has quit IRC02:36
*** iceyao_ has quit IRC02:37
*** arunman has joined #openstack-nova02:37
openstackgerritjichenjc proposed openstack/nova master: Use plain routes list for os-quota-sets endpoint instead of stevedore  https://review.openstack.org/46163002:38
*** ijw has quit IRC02:38
*** cfriesen has joined #openstack-nova02:39
*** yamahata has quit IRC02:40
*** iceyao has joined #openstack-nova02:41
openstackgerritZhenyu Zheng proposed openstack/nova master: Use plain routes list for server-metadata endpoint instead of stevedore  https://review.openstack.org/46170302:45
*** armax has quit IRC02:48
*** thorst_afk has joined #openstack-nova02:48
*** psachin has joined #openstack-nova02:51
*** hongbin has quit IRC02:59
*** gcb has quit IRC02:59
*** fragatin_ has joined #openstack-nova03:03
*** armax has joined #openstack-nova03:05
*** fragatina has quit IRC03:06
*** mingyu has quit IRC03:06
*** fragatin_ has quit IRC03:08
*** thorst_afk has quit IRC03:08
*** nicolasbock has quit IRC03:10
*** gcb has joined #openstack-nova03:12
*** mingyu has joined #openstack-nova03:18
*** armax has quit IRC03:19
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: Encode ovo String field type values in __repr__  https://review.openstack.org/46197503:19
mriedemmelwitt: ^ trying something since we can't test this easily in ovo itself03:19
*** bkopilov_ has joined #openstack-nova03:20
*** felipemonteiro has quit IRC03:24
*** mriedem has quit IRC03:25
*** vks1 has joined #openstack-nova03:34
*** armax has joined #openstack-nova03:35
*** zhurong has joined #openstack-nova03:37
*** ijw has joined #openstack-nova03:38
*** zhurong has quit IRC03:40
*** ijw has quit IRC03:42
*** mingyu has quit IRC03:46
*** masber has quit IRC03:46
*** iceyao has quit IRC03:48
*** mingyu has joined #openstack-nova03:48
*** tbachman has quit IRC03:48
openstackgerritZhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4)  https://review.openstack.org/45959303:48
*** links has joined #openstack-nova03:49
*** mingyu has quit IRC03:52
openstackgerritChris Friesen proposed openstack/python-novaclient master: 2.46: Show flavor info in server details  https://review.openstack.org/43514103:57
*** mingyu has joined #openstack-nova03:59
*** markvoelker has joined #openstack-nova04:00
*** mingyu has quit IRC04:04
*** thorst_afk has joined #openstack-nova04:04
*** ssurana has joined #openstack-nova04:05
*** thorst_afk has quit IRC04:09
*** iceyao has joined #openstack-nova04:10
*** iceyao has quit IRC04:15
*** salv-orlando has joined #openstack-nova04:18
openstackgerritfalseuser proposed openstack/nova master: [BugFix] Change the parameter of the exception error message  https://review.openstack.org/46198504:21
*** salv-orlando has quit IRC04:23
*** gyee has quit IRC04:23
*** mingyu has joined #openstack-nova04:23
openstackgerritmelanie witt proposed openstack/nova master: WIP Use six.text_type() when logging Instance object  https://review.openstack.org/46191204:24
*** fragatina has joined #openstack-nova04:24
*** fragatina has quit IRC04:25
*** mingyu has quit IRC04:25
*** fragatina has joined #openstack-nova04:25
openstackgerritguang-yee proposed openstack/nova master: libvirt: Reconnect volumes and encryptors during a hard reboot  https://review.openstack.org/40038404:26
*** mingyu has joined #openstack-nova04:27
*** armax has quit IRC04:27
*** trinaths has joined #openstack-nova04:28
*** armax has joined #openstack-nova04:28
*** iceyao has joined #openstack-nova04:29
*** nmathew has joined #openstack-nova04:30
*** krtaylor has joined #openstack-nova04:33
*** iceyao has quit IRC04:33
*** Shunli has joined #openstack-nova04:34
*** iceyao has joined #openstack-nova04:35
*** zhurong has joined #openstack-nova04:37
*** armax has quit IRC04:38
*** yamahata has joined #openstack-nova04:41
*** mingyu has quit IRC04:47
*** salv-orlando has joined #openstack-nova04:49
*** mdnadeem has joined #openstack-nova04:52
*** adisky_ has joined #openstack-nova04:52
*** mingyu has joined #openstack-nova04:53
*** yamahata has quit IRC04:55
*** udesale has joined #openstack-nova04:58
*** ayogi has joined #openstack-nova04:58
*** udesale has quit IRC05:00
*** Shunli has quit IRC05:01
*** udesale has joined #openstack-nova05:01
*** Shunli has joined #openstack-nova05:02
*** huanxie has quit IRC05:02
*** huanxie has joined #openstack-nova05:03
*** huanxie has quit IRC05:04
*** mingyu has quit IRC05:05
*** huanxie has joined #openstack-nova05:05
*** zsli_ has joined #openstack-nova05:08
*** Shunli has quit IRC05:08
*** coreywright has quit IRC05:08
*** gcb has quit IRC05:10
*** cfriesen has quit IRC05:20
*** iceyao has quit IRC05:23
*** sree has joined #openstack-nova05:25
*** sree has quit IRC05:26
*** sree has joined #openstack-nova05:27
*** gcb has joined #openstack-nova05:27
*** coreywright has joined #openstack-nova05:27
*** Sukhdev has joined #openstack-nova05:33
*** zsli_ is now known as Shunli05:37
*** iceyao has joined #openstack-nova05:38
*** Jack_Iv has joined #openstack-nova05:41
*** iceyao has quit IRC05:42
*** salv-orl_ has joined #openstack-nova05:46
*** ltomasbo|away is now known as ltomasbo05:47
*** salv-orlando has quit IRC05:49
*** salv-orl_ has quit IRC05:50
openstackgerritAlex Xu proposed openstack/nova master: Add policy description for extensions  https://review.openstack.org/46199505:51
openstackgerritAlex Xu proposed openstack/nova master: Add policy description for os-quota-classes  https://review.openstack.org/46199605:51
*** trinaths has left #openstack-nova05:52
*** Jack_Iv has quit IRC05:54
*** Jack_Iv has joined #openstack-nova05:55
*** masber has joined #openstack-nova05:58
*** Jack_Iv has quit IRC05:59
*** Jack_Iv has joined #openstack-nova06:00
*** mingyu has joined #openstack-nova06:05
*** thorst_afk has joined #openstack-nova06:06
*** prateek has joined #openstack-nova06:06
*** thorst_afk has quit IRC06:11
*** mingyu has quit IRC06:11
*** tovin07 has joined #openstack-nova06:12
*** andreas_s has joined #openstack-nova06:13
openstackgerritAlex Xu proposed openstack/nova master: Move all the deprecated server actions together into the bottom of doc  https://review.openstack.org/45873806:14
openstackgerritAlex Xu proposed openstack/nova master: Adjust the order of server actions in api-ref  https://review.openstack.org/46199806:14
*** mingyu has joined #openstack-nova06:17
*** ratailor has joined #openstack-nova06:18
*** markus_z has joined #openstack-nova06:19
*** jaosorior_away is now known as jaosorior06:19
*** larainema has quit IRC06:20
*** gongysh has joined #openstack-nova06:21
*** voelzmo has joined #openstack-nova06:26
*** pcaruana has joined #openstack-nova06:27
*** ekuris has joined #openstack-nova06:29
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventory  https://review.openstack.org/45521506:32
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for PUT RP inventory  https://review.openstack.org/45521406:32
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for GET RP inventory  https://review.openstack.org/45521306:32
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for GET RP inventories  https://review.openstack.org/44976506:32
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventories  https://review.openstack.org/45270806:32
openstackgerritAndrey Volkov proposed openstack/nova master: [WIP] [placement] Add api-ref resource provider  https://review.openstack.org/45617606:32
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for PUT RP inventories  https://review.openstack.org/45270706:32
*** ratailor_ has joined #openstack-nova06:33
*** ratailor has quit IRC06:34
*** bkopilov has joined #openstack-nova06:41
*** ratailor_ has quit IRC06:42
*** ratailor has joined #openstack-nova06:42
*** mingyu has quit IRC06:44
*** mingyu has joined #openstack-nova06:46
*** mingyu has quit IRC06:47
*** mingyu has joined #openstack-nova06:47
openstackgerritZhenyu Zheng proposed openstack/nova master: Send out notifications when instance tags changed  https://review.openstack.org/45992306:50
*** Jack_Iv has quit IRC06:50
*** gouthamr has joined #openstack-nova06:56
*** Jack_Iv has joined #openstack-nova07:00
*** Jack_Iv has quit IRC07:00
*** gouthamr has quit IRC07:00
*** slaweq has joined #openstack-nova07:00
*** Jack_Iv has joined #openstack-nova07:00
*** mingyu has quit IRC07:01
openstackgerritZhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4)  https://review.openstack.org/45959307:04
*** Jack_Iv has quit IRC07:05
*** slaweq has quit IRC07:06
*** aarefiev has joined #openstack-nova07:06
*** thorst_afk has joined #openstack-nova07:06
*** ociuhandu has quit IRC07:08
*** Jack_Iv has joined #openstack-nova07:08
*** thorst_afk has quit IRC07:11
*** mingyu has joined #openstack-nova07:11
*** bauwser is now known as bauzas07:12
bauzasp!spy07:12
bauzasgood morning Nova07:12
* bauzas catching up after a few days off07:12
*** sridharg has joined #openstack-nova07:14
openstackgerritHuan Xie proposed openstack/nova master: WIP: Fix live migration on XS7.1  https://review.openstack.org/46201707:23
*** zhurong has quit IRC07:24
openstackgerritAlex Xu proposed openstack/nova master: Add policy description for os-quota-classes  https://review.openstack.org/46199607:25
openstackgerritAlex Xu proposed openstack/nova master: Add policy description for extensions  https://review.openstack.org/46199507:25
*** ijw has joined #openstack-nova07:27
*** psachin has quit IRC07:29
*** bjolo has joined #openstack-nova07:31
*** ijw has quit IRC07:32
*** jpena|off is now known as jpena07:33
*** Qiming has quit IRC07:36
*** gaurangt has joined #openstack-nova07:38
*** salv-orlando has joined #openstack-nova07:38
*** damien_r has joined #openstack-nova07:40
*** iceyao has joined #openstack-nova07:42
*** zhurong has joined #openstack-nova07:42
*** Qiming has joined #openstack-nova07:42
*** Sukhdev has quit IRC07:43
*** slaweq has joined #openstack-nova07:46
*** salv-orlando has quit IRC07:46
*** iceyao has quit IRC07:46
*** slaweq has quit IRC07:49
*** kiwi_rot has joined #openstack-nova07:52
*** oanson has quit IRC07:52
*** irenab has quit IRC07:53
*** oanson has joined #openstack-nova07:53
*** irenab has joined #openstack-nova07:53
*** gongysh has quit IRC07:55
*** ssurana has quit IRC07:56
*** ralonsoh has joined #openstack-nova07:56
*** masber has quit IRC07:56
*** masber has joined #openstack-nova07:59
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-nova08:00
*** ratailor has quit IRC08:01
*** ijw has joined #openstack-nova08:05
*** salv-orlando has joined #openstack-nova08:05
*** thorst_afk has joined #openstack-nova08:07
*** slaweq has joined #openstack-nova08:09
*** ijw has quit IRC08:09
*** Jack_Iv has quit IRC08:10
*** ratailor has joined #openstack-nova08:11
*** Jack_Iv has joined #openstack-nova08:13
*** gjayavelu has joined #openstack-nova08:16
*** mingyu has quit IRC08:17
*** fragatina has quit IRC08:20
*** mingyu has joined #openstack-nova08:23
*** Oku_OS-away is now known as Oku_OS08:25
*** mingyu has quit IRC08:25
*** lucas-afk is now known as lucasagomes08:26
*** iceyao has joined #openstack-nova08:26
*** thorst_afk has quit IRC08:26
*** mingyu has joined #openstack-nova08:26
*** iceyao has quit IRC08:30
*** markvoelker has quit IRC08:32
*** mingyu has quit IRC08:34
*** mingyu has joined #openstack-nova08:34
*** mikal has quit IRC08:39
*** derekh has joined #openstack-nova08:39
openstackgerritChris Dent proposed openstack/nova master: Optional separate database for placement API  https://review.openstack.org/36276608:39
*** mikal has joined #openstack-nova08:41
*** mingyu has quit IRC08:45
*** slaweq has quit IRC08:45
*** slaweq has joined #openstack-nova08:45
*** Oku_OS is now known as Oku_OS-away08:45
*** zhurong has quit IRC08:47
*** iceyao has joined #openstack-nova08:48
*** slaweq has quit IRC08:50
*** slaweq__ has joined #openstack-nova08:50
*** bhagyashris has quit IRC08:52
*** iceyao has quit IRC08:53
*** mingyu has joined #openstack-nova08:54
*** salv-orlando has quit IRC08:56
*** mingyu has quit IRC08:57
*** mingyu has joined #openstack-nova08:59
*** slaweq__ is now known as slaweq09:01
*** zhurong has joined #openstack-nova09:01
*** bhagyashris has joined #openstack-nova09:04
*** gszasz has joined #openstack-nova09:04
*** abhishek has joined #openstack-nova09:04
openstackgerritjichenjc proposed openstack/nova master: Use plain routes list for os-instance-usage-audit-log  endpoint instead of stevedore  https://review.openstack.org/46164709:04
*** yamamoto has joined #openstack-nova09:07
*** hieulq_ has joined #openstack-nova09:08
*** yamamoto has quit IRC09:11
*** mingyu has quit IRC09:12
*** hieulq_ has quit IRC09:12
*** hieulq_ has joined #openstack-nova09:12
*** iceyao_ has joined #openstack-nova09:13
*** abalutoiu_ has joined #openstack-nova09:16
*** abalutoiu has quit IRC09:17
*** iceyao_ has quit IRC09:18
*** gjayavelu has quit IRC09:18
*** mingyu has joined #openstack-nova09:19
openstackgerritZhenyu Zheng proposed openstack/nova master: Use plain routes list for server-password endpoint instead of stevedore  https://review.openstack.org/46204909:19
*** hieulq_ has quit IRC09:20
*** sapcc-bot has quit IRC09:23
*** david_6 has quit IRC09:23
*** carthaca_3 has quit IRC09:23
*** artom has quit IRC09:23
*** artom has joined #openstack-nova09:23
*** john51 has quit IRC09:23
*** david_ has joined #openstack-nova09:23
*** sapcc-bot has joined #openstack-nova09:23
*** artom has quit IRC09:23
*** john51_ has joined #openstack-nova09:23
*** thorst_afk has joined #openstack-nova09:24
*** carthaca_ has joined #openstack-nova09:24
*** artom has joined #openstack-nova09:24
*** vipul- has joined #openstack-nova09:25
*** vipuls has quit IRC09:25
*** Jeffrey4l has quit IRC09:25
*** Jeffrey4l has joined #openstack-nova09:27
openstackgerritAlex Xu proposed openstack/nova master: Add policy description for os-quota-classes  https://review.openstack.org/46199609:27
openstackgerritAlex Xu proposed openstack/nova master: Add policy description for extensions  https://review.openstack.org/46199509:27
*** thorst_afk has quit IRC09:28
*** jose-phi_ has joined #openstack-nova09:29
*** adreznec has quit IRC09:30
*** jose-phillips has quit IRC09:31
*** adreznec has joined #openstack-nova09:31
*** ijw has joined #openstack-nova09:32
*** markvoelker has joined #openstack-nova09:33
*** Shunli has quit IRC09:35
johnthetubaguygmann: alex_xu: just looking at this ips API, seems like we should have deprecated that? https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/ips.py09:37
*** ijw has quit IRC09:37
*** markvoelker has quit IRC09:38
*** hieulq_ has joined #openstack-nova09:40
*** zhurong has quit IRC09:41
openstackgerritAlex Szarka proposed openstack/nova master: Transform instance-live_migration_abort notification  https://review.openstack.org/42045309:41
asettleGood morning \o/ it's been a whole week since I've been here09:42
asettleI've got a quick nova doc patch ifs omeone could take a wee lookie09:42
asettlehttps://review.openstack.org/#/c/462009/209:42
*** rmart04 has joined #openstack-nova09:44
*** udesale has quit IRC09:45
johnthetubaguyasettle: seems like a good add to me09:47
asettleThanks johnthetubaguy :) I just did my little edit, and +209:47
openstackgerritjichenjc proposed openstack/nova master: [placement] Add doc for resource provider usages  https://review.openstack.org/45010509:48
*** Oku_OS-away is now known as Oku_OS09:49
alex_xujohnthetubaguy: I remember we talk about in the before09:52
johnthetubaguyalex_xu: ah, I forgot about that one completely09:53
alex_xujohnthetubaguy: I can't remeber the detail also, try to search the log09:54
*** sdague has joined #openstack-nova09:55
*** tesseract has joined #openstack-nova09:56
*** abalutoiu_ has quit IRC09:57
*** iceyao has joined #openstack-nova09:58
openstackgerritGábor Antal proposed openstack/nova master: [WIP] Transform aggregate.update_metadata notification  https://review.openstack.org/46062509:59
alex_xujohnthetubaguy: there is a roadmap for the interface API in the bottom of https://etherpad.openstack.org/p/nova-os-interfaces, it didn't have record about ips :( but I guess that isn't something we want10:01
johnthetubaguyalex_xu: ah, oops10:01
alex_xujohnthetubaguy: here is the talk, it is very long http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2017-04-11.log.html#t2017-04-11T14:02:10 if you are interesting10:02
*** cNilesh has quit IRC10:02
*** hieulq_ has quit IRC10:03
*** tbachman has joined #openstack-nova10:03
*** iceyao has quit IRC10:03
*** tovin07_ has quit IRC10:04
*** gcb has quit IRC10:04
*** Jack_Iv has quit IRC10:05
*** salv-orlando has joined #openstack-nova10:06
*** Jack_Iv has joined #openstack-nova10:06
*** slaweq has quit IRC10:06
*** slaweq has joined #openstack-nova10:07
*** slaweq has quit IRC10:10
*** slaweq has joined #openstack-nova10:10
openstackgerritAbhishek Sharma M proposed openstack/nova master: Catching OverQuota Exception  https://review.openstack.org/46030510:10
*** slaweq has quit IRC10:15
*** mingyu has quit IRC10:17
*** mingyu has joined #openstack-nova10:19
*** zhurong has joined #openstack-nova10:19
*** kevinz has quit IRC10:19
*** oanson has quit IRC10:19
*** oanson has joined #openstack-nova10:21
*** mvk has quit IRC10:22
*** nicolasbock has joined #openstack-nova10:28
*** Jack_Iv has quit IRC10:29
*** Jack_Iv has joined #openstack-nova10:29
*** salv-orlando has quit IRC10:30
*** zhurong has quit IRC10:30
*** ratailor_ has joined #openstack-nova10:32
*** Jack_Iv has quit IRC10:34
*** markvoelker has joined #openstack-nova10:34
*** mingyu has quit IRC10:35
*** ratailor has quit IRC10:35
*** hieulq_ has joined #openstack-nova10:39
*** markvoelker has quit IRC10:40
*** iceyao has joined #openstack-nova10:40
*** iceyao has quit IRC10:45
*** mingyu has joined #openstack-nova10:46
*** gszasz has quit IRC10:47
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Remove dead code  https://review.openstack.org/46066210:47
openstackgerritStephen Finucane proposed openstack/os-vif master: Require OVS 2.8 and kernel 4.8 for offload  https://review.openstack.org/46009310:50
*** lpetrut has joined #openstack-nova10:51
*** hieulq_ has quit IRC10:54
*** hferenc has quit IRC10:54
openstackgerritStephen Finucane proposed openstack/nova master: Ensure sample policy help text correctly wrapped  https://review.openstack.org/46009710:55
*** mingyu has quit IRC10:55
*** ssurana has joined #openstack-nova10:57
*** slaweq has joined #openstack-nova10:57
*** ijw has joined #openstack-nova10:59
*** yolkfull__ has quit IRC11:00
*** eric_young has joined #openstack-nova11:00
*** slaweq has quit IRC11:00
*** slaweq has joined #openstack-nova11:01
*** ssurana has quit IRC11:01
*** mingyu has joined #openstack-nova11:02
*** sree has quit IRC11:02
*** ratailor_ has quit IRC11:02
*** sree has joined #openstack-nova11:02
*** ericyoung has quit IRC11:02
*** bkopilov has quit IRC11:03
*** abhishek has quit IRC11:03
*** bkopilov_ has quit IRC11:04
*** iceyao has joined #openstack-nova11:04
*** ijw has quit IRC11:04
*** slaweq has quit IRC11:05
*** sree has quit IRC11:07
*** mingyu has quit IRC11:07
*** sree has joined #openstack-nova11:08
*** Jack_Iv has joined #openstack-nova11:09
*** mingyu has joined #openstack-nova11:09
*** ijw has joined #openstack-nova11:10
*** sree has quit IRC11:13
*** iceyao has quit IRC11:14
*** ijw has quit IRC11:15
*** mvk has joined #openstack-nova11:20
*** dimtruck is now known as zz_dimtruck11:20
*** smatzek has joined #openstack-nova11:21
*** smatzek_ has joined #openstack-nova11:21
*** ijw has joined #openstack-nova11:21
*** Jack_Iv has quit IRC11:21
*** phuongnh has quit IRC11:22
*** Jack_Iv has joined #openstack-nova11:23
*** smatzek has quit IRC11:24
*** smatzek_ is now known as smatzek11:24
*** thorst_afk has joined #openstack-nova11:25
asettlejohnthetubaguy: could I Get your email with something? How much do you know about Compute schedulers?11:25
*** ijw has quit IRC11:25
*** Jack_Iv has quit IRC11:26
sdaguemelwitt: yeh, there was a reason this broke down on me before, and I really don't remember what it is11:26
*** Jack_Iv has joined #openstack-nova11:27
*** lucasagomes is now known as lucas-hungry11:28
sfinucansdague: Could you take a punt at https://review.openstack.org/#/c/431037/ and successors today?11:28
*** mingyu has quit IRC11:28
*** mingyu has joined #openstack-nova11:29
*** mingyu has quit IRC11:29
*** slaweq has joined #openstack-nova11:30
*** thorst_afk has quit IRC11:31
*** ijw has joined #openstack-nova11:31
*** winston-d_ has joined #openstack-nova11:31
*** ijw has quit IRC11:36
*** markvoelker has joined #openstack-nova11:36
*** slaweq has quit IRC11:36
*** nmathew has quit IRC11:39
sdaguesfinucan: question on https://review.openstack.org/#/c/431037/10 (though +2 for now anyway)11:40
*** markvoelker has quit IRC11:40
*** Drankis has joined #openstack-nova11:41
*** iceyao has joined #openstack-nova11:42
*** bhagyashris has quit IRC11:43
*** rfolco has joined #openstack-nova11:44
*** tbachman has quit IRC11:45
*** iceyao has quit IRC11:47
*** mingyu has joined #openstack-nova11:48
*** mingyu has quit IRC11:48
*** litao has quit IRC11:49
*** pcaruana has quit IRC11:53
*** yingjun has joined #openstack-nova11:54
*** thorst_afk has joined #openstack-nova11:55
*** pcaruana has joined #openstack-nova11:57
*** hferenc has joined #openstack-nova11:59
*** iceyao has joined #openstack-nova12:00
*** links has quit IRC12:00
mordredefried: the keystoneauth change has landed - so I think next step is getting you a ksa patch cut, yeah?12:01
*** zhurong has joined #openstack-nova12:02
*** coreywright has quit IRC12:02
*** ijw has joined #openstack-nova12:02
*** iceyao has quit IRC12:05
*** Qiming has quit IRC12:06
*** huanxie has quit IRC12:07
*** ijw has quit IRC12:08
*** coreywright has joined #openstack-nova12:15
*** liverpooler has joined #openstack-nova12:16
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/46144612:19
*** Qiming has joined #openstack-nova12:20
*** iceyao has joined #openstack-nova12:22
openstackgerritOpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements  https://review.openstack.org/46212412:23
*** ijw has joined #openstack-nova12:23
*** vks1 has quit IRC12:26
*** tbachman has joined #openstack-nova12:26
*** iceyao has quit IRC12:26
*** ijw has quit IRC12:28
*** abhishek has joined #openstack-nova12:28
*** trinaths has joined #openstack-nova12:29
*** iceyao has joined #openstack-nova12:29
*** trinaths1 has joined #openstack-nova12:30
*** catintheroof has joined #openstack-nova12:30
*** trinaths has quit IRC12:33
*** ijw has joined #openstack-nova12:33
*** nicolasbock has quit IRC12:34
*** iceyao has quit IRC12:36
*** markvoelker has joined #openstack-nova12:37
*** sree has joined #openstack-nova12:37
*** jpena is now known as jpena|lunch12:40
*** links has joined #openstack-nova12:41
*** baoli has joined #openstack-nova12:41
*** cfriesen has joined #openstack-nova12:41
*** markvoelker has quit IRC12:41
*** iceyao has joined #openstack-nova12:41
*** lucas-hungry is now known as lucasagomes12:42
*** slaweq has joined #openstack-nova12:42
*** peter-hamilton has joined #openstack-nova12:42
*** ijw has quit IRC12:42
*** gszasz has joined #openstack-nova12:42
*** yingjun has quit IRC12:43
*** Jack_Iv has quit IRC12:43
*** djohnsto has joined #openstack-nova12:44
*** eharney has joined #openstack-nova12:44
*** Jack_Iv has joined #openstack-nova12:44
*** slaweq has quit IRC12:46
*** slaweq has joined #openstack-nova12:46
alex_xunova api meeting is in 14 mins at #openstack-meeting-412:46
*** nicolasbock has joined #openstack-nova12:47
*** mingyu has joined #openstack-nova12:49
*** felipemonteiro has joined #openstack-nova12:49
*** lyan has joined #openstack-nova12:49
*** slaweq has quit IRC12:51
*** mriedem has joined #openstack-nova12:51
*** Drankis has quit IRC12:51
johnthetubaguyasettle: I know bits, but there might be better folks to ask, whats up?12:53
*** mingyu has quit IRC12:54
asettlejohnthetubaguy: I'm staring at this bug: https://bugs.launchpad.net/openstack-manuals/+bug/168426112:54
openstackLaunchpad bug 1684261 in openstack-manuals "AggregateImagePropertiesIsolation example doesn't actually indicate how it works" [Low,Confirmed] - Assigned to Alexandra Settle (alexandra-settle)12:54
efriedmordred - lbragstad did that yesterday; 2.20.0 is out.12:54
asettleAnd I admit, I'm also very confused.12:54
efriedIt's working its way through requirements now.  The bot proposed u-c bump, which merged; and I proposed g-r bump, which merged.  Waiting for the bot to propose nova requirements.txt bump, but I think the nova change set will now verify, cause it uses u-c, right?12:55
*** karthiks has joined #openstack-nova12:55
lbragstadefried mordred https://pypi.python.org/pypi/keystoneauth1/2.20.012:55
*** gcb has joined #openstack-nova12:56
*** nmathew has joined #openstack-nova12:56
johnthetubaguyasettle: good question, sounds like we need a better worked example. We have more docs in here: https://docs.openstack.org/developer/nova/filter_scheduler.html12:58
*** mdrabe has joined #openstack-nova12:59
*** bkopilov_ has joined #openstack-nova13:00
efriedmriedem Do you want me to include links to merged and/or proposed change sets in the use-service-catalog-for-endpoints edit?  https://review.openstack.org/#/c/461481/13:00
*** bkopilov has joined #openstack-nova13:00
alex_xunova api meeting is running at #openstack-meeting-413:00
*** masber has quit IRC13:01
mriedemefried: nope13:01
asettlejohnthetubaguy: yeah I was reading that this morning. I couldn't, admittedly, find too many good examples to transfer across. The only thing I could find for AggregateImagePropertiesIsolation is the literal explanation about halfway down the page.13:01
*** claudiub has joined #openstack-nova13:02
*** mpaolino has joined #openstack-nova13:03
*** slaweq has joined #openstack-nova13:03
mriedemdansmith: can you hit this simple grenade change for using wsgi on master https://review.openstack.org/#/c/460543/ ?13:04
mriedemi.e. the devstack change to use uwsgi on master depends on that https://review.openstack.org/#/c/457715/13:05
*** gouthamr has joined #openstack-nova13:05
*** nicolasbock has quit IRC13:05
*** trinaths1 has quit IRC13:06
mriedemEmilienM: we've said in the past that when we make changes to grenade that we should give the deployment teams a heads up, do you think this change warrants that https://review.openstack.org/#/c/460543/ ?13:06
*** nmathew has quit IRC13:06
mriedemthat's needed for using uwsgi for nova in devstack on master https://review.openstack.org/#/c/457715/13:06
*** felipemonteiro_ has joined #openstack-nova13:08
*** jamesdenton has joined #openstack-nova13:08
openstackgerritEric Fried proposed openstack/nova master: nova.utils.get_service_url()  https://review.openstack.org/45825713:08
*** felipemonteiro has quit IRC13:11
*** eharney has quit IRC13:11
*** felipemonteiro__ has joined #openstack-nova13:13
*** felipemonteiro_ has quit IRC13:13
* bauzas got a headache when trying to use the Summit agenda to find sessions to attend 13:13
bauzassched.org, you <313:13
*** mingyu has joined #openstack-nova13:14
*** sree has quit IRC13:14
*** sree has joined #openstack-nova13:15
*** trinaths has joined #openstack-nova13:17
*** jaypipes has joined #openstack-nova13:18
*** sree has quit IRC13:19
*** nicolasbock has joined #openstack-nova13:22
dansmithmriedem: yep got it13:24
*** salv-orlando has joined #openstack-nova13:25
sfinucansdague: One more review here for you, if you would be so kind https://review.openstack.org/46009713:26
*** eharney has joined #openstack-nova13:26
*** tongli has joined #openstack-nova13:26
*** zhurong has quit IRC13:28
*** cleong has joined #openstack-nova13:29
*** cfriesen has quit IRC13:32
*** liusheng has quit IRC13:32
*** cfriesen has joined #openstack-nova13:32
*** smatzek has quit IRC13:32
*** slaweq has quit IRC13:33
*** liusheng has joined #openstack-nova13:33
*** ltomasbo is now known as ltomasbo|away13:33
*** sbezverk_ is now known as sbezverk13:35
*** rmart04 has quit IRC13:35
*** ltomasbo|away is now known as ltomasbo13:36
openstackgerritJohn Garbutt proposed openstack/nova master: Add policy description for fping  https://review.openstack.org/46176313:37
*** vks1 has joined #openstack-nova13:38
*** rmart04 has joined #openstack-nova13:39
*** gcb has quit IRC13:40
*** gcb has joined #openstack-nova13:40
*** jaosorior is now known as jaosorior_away13:41
*** jpena|lunch is now known as jpena13:41
*** xyang1 has joined #openstack-nova13:41
*** vks1 has quit IRC13:42
dansmithsdague: so your native call logging hack suggestion...13:43
*** gcb has quit IRC13:43
dansmithsdague: I dunno if you saw, but we're actually missing logs, and the two requests don't line up. we're not just seeing a big delay13:43
dansmithdo you still think it's worth trying that?13:43
*** gcb has joined #openstack-nova13:44
*** lucasxu has joined #openstack-nova13:44
*** psachin has joined #openstack-nova13:45
*** irenab has quit IRC13:45
*** pchavva has joined #openstack-nova13:45
*** mlavalle has joined #openstack-nova13:45
openstackgerritmelanie witt proposed openstack/nova master: WIP Use six.text_type() when logging Instance object  https://review.openstack.org/46191213:46
*** irenab has joined #openstack-nova13:46
*** tbachman has quit IRC13:47
*** mriedem has quit IRC13:48
*** karimb has joined #openstack-nova13:48
*** mriedem has joined #openstack-nova13:51
*** armax has joined #openstack-nova13:51
*** crushil has joined #openstack-nova13:52
*** felipemonteiro_ has joined #openstack-nova13:53
openstackgerritDan Smith proposed openstack/nova master: WIP: Debug race in delete-while-booting test  https://review.openstack.org/46151313:53
*** Swami has joined #openstack-nova13:54
sdaguedansmith: sure13:54
dansmithokay13:54
sdaguedansmith: you want me to push it out there?13:54
*** smatzek has joined #openstack-nova13:54
sdaguehonestly, the only thing it loses us is the search by request-id, which would be nice13:55
dansmithsdague: I'll just slam it into my patch13:55
sdaguebut not hugely a thing13:55
dansmithI'm going to put something else to log in the delete path13:55
sdagueany idea if there was an exception in this path?13:55
dansmithwait, what?13:55
*** irenab has quit IRC13:55
sdaguewait what what?13:55
*** tbachman has joined #openstack-nova13:56
*** felipemonteiro__ has quit IRC13:57
dansmithcommenting the use_journal thing means we don't log request_id?13:57
*** markvoelker has joined #openstack-nova13:57
dansmithor you mean it doesn't do the rich field thing?13:57
*** vks1 has joined #openstack-nova13:57
*** lpetrut has quit IRC13:57
openstackgerritDan Smith proposed openstack/nova master: WIP: Debug race in delete-while-booting test  https://review.openstack.org/46151313:57
*** irenab has joined #openstack-nova13:57
openstackgerritmelanie witt proposed openstack/nova master: WIP Use six.text_type() when logging Instance object  https://review.openstack.org/46191213:58
*** links has quit IRC13:58
sdaguedansmith: it doesn't do the rich field thing13:59
dansmithokay gotcha13:59
sdagueso with that on, the log path is through journal.send, which gives us all the extra metadata13:59
sdaguewith that off, it's just going to spew to stdout14:00
dansmithcool, just wanted to make sure it wasn't going to change more stuff14:00
sdaguesystemd will send it all to the journal but without any of the fun bits14:00
sdaguedansmith: https://review.openstack.org/46216314:00
sdagueI also tweaked the debug line, because processid is going to be in the systemd logs anyway14:01
mriedemmelwitt: heh, display_name='tempest-Íñstáñcé-2027474072' is better than UnicodeDecodeError at least14:01
dansmithsdague: oh i just commented it out of my patch14:01
sdaguemriedem: mriedem how did we get to that?14:02
*** lucasxu has quit IRC14:02
mriedemsdague: https://review.openstack.org/#/c/461912/14:02
*** ayogi has quit IRC14:02
*** zhurong has joined #openstack-nova14:02
melwittmriedem: true. it's correct when I download the .txt.gz file though and open it with the less command. not sure what's up with that considering it let me copy-paste the name into the review comment from the log file I had downloaded14:02
sdaguedansmith: sure, honestly, for right now I think it's fine to turn it off if we have concerns, so I'll put the general patch out there as well14:02
*** lucasxu has joined #openstack-nova14:03
sdaguemelwitt: that really feels like something that should be happening at the logging levels below14:03
mriedemsdague: you mean in the oslo.log handlers?14:04
jaypipessdague, dansmith, mriedem, bauzas: https://hangouts.google.com/call/b6rgkm2kzbh57pdgv6vey2kgxau14:04
*** lpetrut has joined #openstack-nova14:04
sdaguemriedem: yeh14:04
melwittsdague: maybe, if we wanna convert everything to unicode before logging. which we probably do I guess14:04
mriedemsdague: i think melwitt is just trying to figure out how to fix it14:04
sdaguemriedem: sure14:04
*** acabot___ has joined #openstack-nova14:04
sdaguemriedem: you going to jump on hangout about onboarding room?14:05
mriedemyes14:05
openstackgerritGábor Antal proposed openstack/nova master: Add json style checking for sample notifications  https://review.openstack.org/44367714:05
dansmithsdague: okay14:06
*** awaugama has joined #openstack-nova14:06
openstackgerritJohn Garbutt proposed openstack/nova master: Add policy description for instance actions  https://review.openstack.org/46175714:06
*** lpetrut has quit IRC14:07
*** djohnsto has quit IRC14:07
*** djohnsto has joined #openstack-nova14:07
dansmithjaypipes: I came to mock, staying to mock14:09
abhishekmelwitt: Hi Melanie. I have added the changes which you had suggested. Please review https://review.openstack.org/#/c/460305/14:09
openstackgerritJohn Garbutt proposed openstack/nova master: Add policy description for instance actions  https://review.openstack.org/46175714:10
melwittabhishek: ack, will take a look14:10
jaypipesdansmith: :)14:10
abhishekmelwitt: thanks14:10
openstackgerritZhenyu Zheng proposed openstack/nova master: Use plain routes list for os-attach-interfaces endpoint instead of stevedore  https://review.openstack.org/46216614:10
*** lucasxu has quit IRC14:15
*** zz_dimtruck is now known as dimtruck14:15
*** lucasxu has joined #openstack-nova14:17
*** openstackgerrit has quit IRC14:18
*** openstackgerrit has joined #openstack-nova14:19
openstackgerritZhenyu Zheng proposed openstack/nova master: Use plain routes list for os-server-groups endpoint instead of stevedore  https://review.openstack.org/46216914:19
*** hshiina has joined #openstack-nova14:20
mriedemdansmith: https://etherpad.openstack.org/p/BOS-forum-nova-project-onboarding14:21
peter-hamiltonhi all, the bulk of the cert validation support in cursive is done: https://review.openstack.org/#/c/357202/14:22
peter-hamiltoni'd like to get additional nova folks eyes on it before we go further (thanks again for the review jaypipes)14:23
*** mdrabe_ has joined #openstack-nova14:23
*** mdrabe has quit IRC14:24
*** lucasxu has quit IRC14:24
*** lucasxu has joined #openstack-nova14:24
*** dave-mccowan has joined #openstack-nova14:25
*** amotoki has joined #openstack-nova14:26
*** lucasxu has quit IRC14:26
*** yangyape_ has joined #openstack-nova14:27
openstackgerritZhenyu Zheng proposed openstack/nova master: use plain routes list for os-virtual-interfaces  https://review.openstack.org/46217314:29
*** yangyapepeng has quit IRC14:30
*** lpetrut has joined #openstack-nova14:31
*** mdnadeem has quit IRC14:33
openstackgerritJohn Garbutt proposed openstack/nova master: Add policy description for os-quota-classes  https://review.openstack.org/46199614:34
*** iceyao has quit IRC14:35
*** lpetrut has quit IRC14:37
*** lpetrut has joined #openstack-nova14:37
*** marst has quit IRC14:38
*** tbachman_ has joined #openstack-nova14:38
*** VAhl has joined #openstack-nova14:38
*** prateek has quit IRC14:39
efriedjohnthetubaguy jaypipes mordred mriedem edmondsw FYI, the necessary ksa and requirements pieces for use-service-catalog-for-endpoints are in place, so the nova change https://review.openstack.org/#/c/458257/ is passing Jenkins and (I believe) ready for a solid review hammering.14:39
openstackgerritZhenyu Zheng proposed openstack/nova master: Use plain routes list for os-consoles and os-console-auth-tokens endpoint instead of stevedore  https://review.openstack.org/46217514:39
jaypipesefried: will try to review this afternoon.14:40
efriedjaypipes Thanks!14:40
edmondswefried roger that14:40
*** tbachman has quit IRC14:40
*** tbachman_ is now known as tbachman14:40
*** udesale has joined #openstack-nova14:43
openstackgerritJohn Garbutt proposed openstack/nova master: Add description to policies in extended_az and extend_ser_attrs  https://review.openstack.org/46160014:43
edmondswmriedem edleafe is there spec for the capabilities work that was discussed at the PTG?14:46
edmondswAll I'm finding is https://blueprints.launchpad.net/nova/+spec/expose-host-capabilities which is languishing and doesn't talk about what was discussed at PTG14:46
openstackgerritZhenyu Zheng proposed openstack/nova master: Use plain routes list for block_device_mapping instead of stevedore  https://review.openstack.org/46218014:47
*** iceyao has joined #openstack-nova14:47
jaypipesedmondsw: are you referring to the API capabilities or are you referring to the resource provider traits stuff?14:48
edmondswjaypipes API14:48
edleafeedmondsw: The api-wg spec for capabilities was abandoned: https://review.openstack.org/#/c/386555/14:49
*** fragatina has joined #openstack-nova14:49
*** marst has joined #openstack-nova14:50
openstackgerritZhenyu Zheng proposed openstack/nova master: Use plain routes list for user_data instead of stevedore  https://review.openstack.org/46218314:50
edmondswedleafe the abandon comment talks about "first fixing the immediate problems14:51
edmondswwhere is that happening?14:51
openstackgerritGábor Antal proposed openstack/nova master: Transform aggregate.add_host notification  https://review.openstack.org/39621014:51
edleafeedmondsw: as far as I know no one has picked that up14:52
*** zhurong has quit IRC14:52
edmondswk14:52
edleafeedmondsw: so if you've got some spare cycles... :)14:53
edmondswedleafe wish I did...14:53
*** salv-orl_ has joined #openstack-nova14:53
edleafeheh, welcome to the club!14:53
*** slaweq has joined #openstack-nova14:55
*** slaweq has quit IRC14:56
*** salv-orlando has quit IRC14:56
*** bmace has quit IRC14:56
melwittsdague: when I was debugging locally, I found this is where it blows up in the log formatting, it's trying to convert the message to unicode, but with the Instance object I found it only works if the encoding 'utf-8' is specified to six.text_type(), and that requires the instance arg be a string, not an Instance object type14:56
*** bmace has joined #openstack-nova14:56
melwitthttps://github.com/openstack/oslo.log/blob/3ee14a1/oslo_log/formatters.py#L83-L9314:56
mriedemsdague: jaypipes: dansmith: bauzas: i assume none of us have a touchscreen laptop to project if we don't have a flip chart or whiteboard right?14:58
mriedemthat would be easier to see than a flip chart if we're expecting 40 people14:58
melwittso in my patch I'm doing repr(instance) followed by six.text_type(instance_repr, 'utf-8') if six.PY214:58
*** erlon has joined #openstack-nova14:58
dansmithmriedem: drawing on that kind of thing is pretty hard to do well, IMHO, but no I don't have one14:59
*** annegentle has joined #openstack-nova14:59
mriedemdansmith: i thought they were supposed to be super awesome nowadays?14:59
mriedembut yeah i don't have one either14:59
melwittI have a msft surface and drawing pens, but I dunno how to project with it15:01
mriedemefried: ah ok so this is what you needed https://github.com/openstack/requirements/commit/f7ebce161764236086957451d21d0fcae9fa86e915:01
mriedemmelwitt: i assume projecting is same as always15:01
mriedemusb adapter probably15:01
*** gcb has quit IRC15:02
*** tbachman has quit IRC15:02
efriedmriedem Yes, that was one piece of the puzzle.15:02
melwittyeah, it has limited ports. it has usb though15:02
*** Oku_OS is now known as Oku_OS-away15:03
*** gcb has joined #openstack-nova15:03
*** voelzmo has quit IRC15:04
mriedemdansmith: jaypipes: for the nova project update slides, we could talk over those tomorrow if you want. i think it would be pretty simple. i figured i could start, then jay covers any placement slides, dan covers any cells slides, and i can do the rest of the misc stuff15:04
sdaguemriedem: honestly, I also find drawing on those live quite hard. Flipboard would definitely be more useful15:04
mriedemsdague: well someone should tell all of the marketers that for every time i watch an NFL game15:05
dansmithmriedem: sounds good to me15:05
mriedembecause they all said it was super easy and awesome15:05
*** ayogi has joined #openstack-nova15:09
*** bmace has quit IRC15:10
*** gcb has quit IRC15:10
sdaguedansmith: did changing the use_journal option with your runs help?15:11
dansmithsdague: they're not done yet15:12
sdaguedansmith: ok15:12
*** tjones has joined #openstack-nova15:12
*** hongbin has joined #openstack-nova15:13
*** gcb has joined #openstack-nova15:17
*** salv-orl_ has quit IRC15:18
*** eharney has quit IRC15:18
*** salv-orlando has joined #openstack-nova15:18
dansmithmriedem: so, looking at the few other onboarding threads on the ML, I'm kinda curious... what are the other projects doing for this slot?15:20
*** kiwi_rot has quit IRC15:22
*** udesale has quit IRC15:22
*** mlakat has quit IRC15:23
mdboothdansmith: Perhaps a hazing?15:25
bauzasdansmith: mriedem: FWIW,  most of the Upstream Training attendees previously were trying to look at devstack and provide a change against a project15:26
bauzasdansmith: mriedem: now that the Institute doesn't really provide devstack, I think they just want to have some technical discussions :)15:27
*** aarefiev is now known as aarefiev_afk15:27
*** gszasz has quit IRC15:28
openstackgerritGábor Antal proposed openstack/nova master: Transform aggregate.add_host notification  https://review.openstack.org/39621015:29
*** djohnsto has quit IRC15:31
mriedemdansmith: i've wondered the same, and i saw the cinder etherpad at one point15:31
mriedembut can't find it now15:31
*** gszasz has joined #openstack-nova15:32
mordredefried: nice! looking now15:32
*** eharney has joined #openstack-nova15:34
*** giuseppe1992 has joined #openstack-nova15:34
*** tbachman has joined #openstack-nova15:34
mriedemsmcginnis: where is your guys' cinder onboarding etherpad?15:35
mriedemkevinbenton: ^ do you guys have one?15:35
giuseppe1992Hello all!!!15:35
*** chyka has joined #openstack-nova15:37
openstackgerritGábor Antal proposed openstack/nova master: Transform aggregate.add_host notification  https://review.openstack.org/39621015:38
mordredefried: oh - also - I'm making the patch to keystoneauth right now to deal with teh service type list - so hopefully we can make a follow up soon to your patch to consume that ..15:38
openstackgerritMathieu Gagné proposed openstack/nova master: Add ability to signal and perform online volume size change  https://review.openstack.org/45432215:39
mordredefried: patch looks mostly great on first read - excellent work -I've got a few nitpicky things I'll leave in just a bit15:41
*** felipemonteiro_ has quit IRC15:43
*** ltomasbo is now known as ltomasbo|away15:43
openstackgerritGábor Antal proposed openstack/nova master: Transform aggregate.remove_host notification  https://review.openstack.org/39621115:44
*** tbachman_ has joined #openstack-nova15:45
smcginnismriedem: I dont believe we have an etherpad for that.15:45
efriedmordred Roger that, and thanks.15:47
*** tbachman has quit IRC15:47
*** tbachman_ is now known as tbachman15:47
efriedmordred Please add me to that service type list patch once it's up.15:48
mordredwill do!15:48
* mordred is now expecting efried to review all of his crazy patches related to this topic15:49
efriedmordred I will certainly do that, for whatever it's worth.  I still have your ML threads awaiting deeper dives.15:49
* mordred mails efried a case of advil15:50
*** felipemonteiro has joined #openstack-nova15:50
*** mpaolino has quit IRC15:53
*** felipemonteiro_ has joined #openstack-nova15:53
*** NostawRm has quit IRC15:54
jaypipesmriedem: works for me. I'll be on IRC all day of course.15:54
jaypipesmriedem: well, after around 9:30am EDT... it's Thursday morning golf before that. :)15:55
mriedemooo this looks like a new fun gate bug http://logs.openstack.org/73/461473/3/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/a277636/logs/screen-n-cpu.txt.gz#_May_02_17_49_07_77124815:56
*** Apoorva has joined #openstack-nova15:56
jaypipesmriedem, sdague: are we still seeing issues with the live-migration jobs?15:57
*** sridharg has quit IRC15:57
mriedemjaypipes: no15:57
mriedemcells v1 job is the main failure right now15:57
*** felipemonteiro has quit IRC15:57
*** slaweq has joined #openstack-nova15:58
dansmithFWIW, it looks like my job is going to pass without journal usage15:59
dansmithI'm not sure that makes sense15:59
dansmithI was hoping to just get the rest of my debug with that hack,15:59
*** tongli has quit IRC15:59
dansmithbut I suppose it's possible that it's occasionally causing us to deadlock something15:59
dansmithI'll recheck a few times to see if I just got uber lucky this time15:59
*** gjayavelu has joined #openstack-nova16:00
*** slaweq has quit IRC16:01
*** felipemonteiro_ has quit IRC16:02
*** felipemonteiro_ has joined #openstack-nova16:02
*** giuseppe1992 has quit IRC16:03
*** slaweq has joined #openstack-nova16:05
*** slaweq has quit IRC16:07
sdaguedansmith: ok, cool16:07
sdagueI did one more tweak on my general patch, once that's passes, I'll get that in16:08
dansmithsdague: that "okay cool" was for the rechecking right?16:08
dansmithI was not expecting it to actually make it pass16:08
dansmithI was expecting it to let me see my own debug for the real failure16:08
sdagueyes, for rechecking16:08
sdagueok, so someone that understands more about eventlet than might be needed to understand why this fails16:09
sdaguehttps://github.com/systemd/python-systemd/blob/c06c5d401d60ae9175367be0797a6c2b562ac5ba/systemd/journal.py#L412-L457 is the function we're calling into16:11
sdaguehttps://github.com/systemd/python-systemd/blob/c06c5d401d60ae9175367be0797a6c2b562ac5ba/systemd/_journal.c which I goes into native code16:11
sdaguejaypipes so presumably related to the same reason gRPC doesn't work16:12
*** hshiina has quit IRC16:12
*** gszasz has quit IRC16:12
mriedemmelwitt: ^ see in that first link to journal.py "MESSAGE                           must be a string and will be sent as UTF-8 to the journal."16:15
jaypipessdague: maybe, not sure though. the grpc vs. eventlet thing was around how grpc's generated python code doesn't play well with monkey-patched envs16:16
sdaguejaypipes: potentially similar issue?16:17
mriedemsdague: was it intentional to enable syslog handler to put all of the service logs into syslog? because syslog is huge now, and basically unusable in a browser16:17
mriedemhttp://logs.openstack.org/73/461473/3/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/a277636/logs/syslog.txt.gz16:17
jaypipessdague: maybe. I'm no eventlet expert though :( perhaps worth asking vstinner or dims to take a look into it.16:17
jaypipessdague: or perhaps one of the swift folks?16:17
sdaguemriedem: we didn't enable the syslog handler16:17
sdaguethis is the journal16:17
*** felipemonteiro__ has joined #openstack-nova16:18
sdaguemriedem: this is what things under systemd look like, we can trim what ends up in the extracted syslog.txt16:18
openstackgerritmelanie witt proposed openstack/nova master: WIP Use six.text_type() when logging Instance object  https://review.openstack.org/46191216:18
mriedemsdague: i thought this did something16:18
mriedemhttps://github.com/openstack/oslo.log/commit/391635475a5877c2078856b5c3a7d2929682764a16:18
sdaguemriedem: https://github.com/openstack-infra/devstack-gate/blob/87a9ef290fa371862f30702497238fffdfdfe8fe/functions.sh#L647-L64916:18
sdaguemriedem: nope, unrelated16:19
dansmithsdague: don't we have a global log lock or something?16:19
dansmiththat was the reason why logging in the db api was so  problematic I think16:20
dansmithwe wouldn't have deadlocks in the same way here I think, but it could still be a problem16:20
melwittmriedem: okay. I wrote a better commit message to explain what I was seeing in local debugging. the ContextFormatter tries to convert every message to unicode, it was failing on the Instance object, and I found to convert it properly I had to specify the encoding to six.text_type()16:20
dansmithwe can't really hold any locks while making a native call or we're going to cause problems16:20
*** felipemonteiro_ has quit IRC16:21
*** lucasagomes is now known as lucas-afk16:21
*** nic has joined #openstack-nova16:22
sdaguedansmith: I didn't think we did16:23
sdagueat least not in our code16:23
dansmithnot in nova, but I thought oslo.logging did or something16:23
sdagueoslo.logging is really thin16:23
dansmithI didn't follow that whole thing way back then16:24
sdagueI don't think there is any locking there16:24
*** pcaruana has quit IRC16:24
dansmithokay, well, the problem was that we'd do a database thing, which would grab a lock in mysql, then do a logging statement, which would grab a lock,16:24
dansmithand if the opposite was happening elsewhere, we'd deadlock16:24
*** gszasz has joined #openstack-nova16:24
dansmithnot sure how that would have happened if it wasn't a lock in oslo.logging or python.16:25
sdaguethere is nothing that looks like a lock in oslo.logging16:25
dansmithmaybe logging to a file in python does a flock on the log file?16:25
*** andreas_s has quit IRC16:25
sdaguemaybe, but we're not doing that here16:25
dansmithno, I know,16:25
dansmithI'm just trying to point to something similar that may be happening16:26
melwittmriedem: this is where I saw it blowing up https://github.com/openstack/oslo.log/blob/3ee14a1/oslo_log/formatters.py#L83-L9316:26
dansmithwith making a native call that could block16:26
sdaguedansmith: yeh16:27
sdagueso this eventually turns into a C sendmsg to push things over the unix domain socket for systemd16:27
sdagueanyway, I'm going to drop the debug thread here, as I'm over my depth on what bits of eventlet would make it weird16:28
mriedemmelwitt: here https://github.com/openstack/oslo.log/blob/3ee14a1/oslo_log/formatters.py#L89 ?16:29
*** karimb has quit IRC16:30
melwittmriedem: eh, I'm not sure how I saw that locally now that I think about it. it's not the same as what we see in the gate16:30
*** gjayavelu has quit IRC16:30
*** kaisers_ has joined #openstack-nova16:30
sdaguemelwitt: in the gate isn't it that we're sending it an Instance Object16:31
sdagueit's implicitly turning it into a string with __repr__16:31
sdagueand that is the thing that might be encoded weird?16:31
sdaguemelwitt: at least, your work around seemed to imply that16:31
clarkbthe journald write is blocking I think16:31
melwittsdague: yeah. and then at the log message format() step it blows up but I don't know exactly how16:31
clarkbbut I don't know that it will hold a lock16:31
sdagueclarkb: I don't know, the man pages say it's thread safe16:32
sdaguewhich could mean locks in there16:32
sdagueit also keeps monotonic time (or can) which makes me wonder16:32
dansmithor locks in journald that cause that call to block16:32
dansmitheven if not locks in the sending code16:32
sdaguebut, it's a whole research bit on it's own16:32
*** arunman has quit IRC16:33
clarkbdansmith: well that call can block regardless of locks16:34
clarkbbecause its a synchronous write16:34
dansmithyeah16:34
melwittmriedem, sdague: in the gate it blows up here https://github.com/python/cpython/blob/master/Lib/logging/__init__.py#L33816:34
*** arunman has joined #openstack-nova16:34
*** arunman has quit IRC16:34
*** kaisers_68 has joined #openstack-nova16:35
sdaguemelwitt: right16:35
sdaguebut one of those args is an Instance16:36
sdagueobject16:36
*** dtp has joined #openstack-nova16:37
*** Jack_Iv has quit IRC16:37
*** trinaths has quit IRC16:37
*** tesseract has quit IRC16:38
sdaguedansmith: are there utf8 tests in oslo.versionedobjects for object stringification?16:38
*** felipemonteiro__ has quit IRC16:38
melwittsdague: yeah. and like you said it gets turned into a str by __repr__ but there are unicode chars in it16:38
sdagueclarkb / dansmith - anyway on the journal issue, I think the near term is to shut that bit off16:39
*** felipemonteiro__ has joined #openstack-nova16:39
dansmithsdague: you mean for the repr?16:39
sdaguedansmith: yeh16:39
sdagueor just in general, I'm not seeing any grepping around16:39
*** kaisers_ has quit IRC16:39
dansmithsdague: is the problem really repr'ing the instance in the log call? I thought we just grab the uuid out of the instance when we do LOG.foo(..., instance=instance) ?16:39
mriedemdansmith: no,16:39
mriedemthis is a full dump of the instance16:40
mriedemand all of it's fields16:40
dansmithah16:40
sdagueright16:40
mriedemand the display_name in one test is unicode16:40
sdaguewhich is when it all blows up16:40
mriedemnot using the instance=instance kwarg16:40
dansmithI dunno, I thought someone did a few utf tweaks when they py3-ified it, but I dunno16:40
melwittthe __repr__ returns a bytes string in PY2 to be native with the rest16:40
*** gszasz has quit IRC16:41
melwittand converting it to unicode before logging it makes it work. it's just I had to repr(instance) and then six.text_type(instance_repr, 'utf-8')16:41
*** ayogi has quit IRC16:42
*** armax has quit IRC16:43
*** rmart04 has quit IRC16:43
*** gyee has joined #openstack-nova16:45
clarkbthis would be a general issue for things like https://git.openstack.org/cgit/openstack/oslo.log/tree/oslo_log/handlers.py#n56 too right?16:47
*** hieulq_ has joined #openstack-nova16:47
clarkbeg its not specific to the journald handler we juts happen to be using it in testing so catch it16:47
*** mvk has quit IRC16:47
*** harlowja has joined #openstack-nova16:48
*** markus_z has quit IRC16:48
sdagueclarkb: it blows up on the format()16:49
sdaguenot on the send16:49
sdaguethat solution would be too late16:49
sdaguethere would be an exception at L7316:49
*** imacdonn has quit IRC16:49
clarkbright I'm saying the syslog handler does the format in emit too16:49
*** hieulq_ has quit IRC16:49
clarkbso the bug is also in that handler16:49
*** imacdonn has joined #openstack-nova16:50
sdagueclarkb: ok, so the solution is?16:51
*** armax has joined #openstack-nova16:51
clarkbI think what melwitt described, do your formatting outside of the logger (which has the downside of possibly being unnecessary depending on log level)?16:52
clarkbor possibly you can manipulate locks within logging?16:53
clarkbthat seems fragile16:53
sdagueclarkb: ?? the locking issue is different16:53
sdagueso, if I dive down the FileHandler -> StreamHandler path16:54
clarkbsdague: oh I understood it as oyu need db lock to format the instance repr and you need logging lock to use the logger16:54
sdagueclarkb: no you have mixed up 2 different issues16:54
clarkbah sorry16:54
sdagueStreamHandler does the same - msg = self.format(record)16:54
*** vks1 has quit IRC16:54
abhishekmelwitt: just a reminder, https://review.openstack.org/#/c/460305/16:55
*** kaisers_68 has quit IRC16:55
melwittsdague: yeah, I'm trying to figure out what is different. does StreamHandler use the same formatter as OSJournalHandler? I don't get what is different here16:55
*** Jack_Iv has joined #openstack-nova16:55
sdaguemelwitt: yep16:55
sdaguemelwitt: so these things have definitely been around before16:55
sdagueI went diving after one of these before16:56
sdagueand there are as many successes as fails on mriedem's issue16:56
clarkbthe format issue is purely a unicode problem?16:56
sdagueclarkb: that is my believe16:56
sdaguebelief16:56
melwittsdague: oh, so something else about the change to the journal logger has just made this happen or timed in a way that we see it in gate failures now?16:56
sdagueI'm actually starting to think it's a red herring16:57
sdaguemelwitt: I expect the deadlocking that was causing dansmith's bugs is actually the issue16:57
melwittsdague: yeah, I think you're right16:58
sdaguehttps://review.openstack.org/#/c/462163/2/functions will fix that16:58
sdaguemelwitt: that being said, I still think there is a general utf8 bug in ovo repr that would be good to fix :)16:58
sdagueat least adding utf8 tests to ovo16:59
cfriesenwhere does the service version get determined?  I don't see it in the Service object.16:59
*** hieulq_ has joined #openstack-nova16:59
*** ralonsoh has quit IRC17:00
*** Jack_Iv has quit IRC17:00
clarkbsdague: so with that change in place journald will still get the service logs out of stdout? we just won't use the native writer right?17:00
*** slaweq has joined #openstack-nova17:00
sdagueclarkb: correct17:00
sdaguesystemd will still injest them17:01
sdaguethey just miss out on the native annotations, which include things like code line, correct process_id, and extras17:01
melwittsdague: yeah ... I'll look at that. it's not straightforward to me bc the __repr__ intentionally converts to bytes (opposite of what we want for the logging) and it seems to work fine outputting to stdout for example, so I don't yet understand what should be done there17:01
*** clenimar has quit IRC17:02
sdaguemelwitt: yeh, I'm not sure I do either. But it also doesn't look like any of the ovo tests have utf8 strings in them17:02
sdaguethat would at least be a start17:02
*** derekh has quit IRC17:02
melwittsdague: ack. will do17:04
*** psachin has quit IRC17:05
*** jpena is now known as jpena|off17:06
mriedemcfriesen: nova.service._update_service_ref17:07
mriedemcfriesen: and __init__ of the Service object17:08
mriedemwhen creating a new one17:08
mriedemself.version = SERVICE_VERSION17:09
sdaguemriedem: what things do you want in syslog.txt ?17:10
*** Swami has quit IRC17:10
*** peter-hamilton has quit IRC17:11
*** hieulq__ has joined #openstack-nova17:11
*** lucasxu has joined #openstack-nova17:12
*** hieulq_ has quit IRC17:12
*** tbachman has quit IRC17:19
mriedemsdague: not the openstack service logs17:24
*** annegentle has quit IRC17:24
sdaguemriedem: sure, but we have to build back up which things you want in there17:25
*** zul has quit IRC17:26
*** hieulq__ has quit IRC17:26
sdaguehttps://review.openstack.org/#/c/462231/17:28
mriedemsdague: i don't know honestly, syslog was my place to go look for native process things on the system, like libvirt, mysql, tgtd, kernel stuff, etc17:29
*** zul has joined #openstack-nova17:29
mriedemi'd use the libvirt logs for libvirt debug info,17:29
mriedemand the openstack service logs for openstack service things17:29
mriedemthe journal gz files are less useful to me since i can't use them in the browser, and have to pull them down17:29
sdaguemriedem: right, well, we're in systemd world, everything is in the journal17:30
sdagueand the export has to be based on inclusion, not exclusion17:30
sdagueso going to start with kernel / sudo17:31
sdagueand if you want other topics please propose17:31
openstackgerritGábor Antal proposed openstack/nova master: Transform aggregate.remove_host notification  https://review.openstack.org/39621117:32
*** iceyao has quit IRC17:33
mriedemsdague: that's likely a fine start, and if i really need to debug anything that i'd normally find in syslog, or look in syslog for, i'll have to pull the journal down17:33
mriedemsdague: that means we might lose some things that we were indexing from syslog into logstash before,17:33
mriedembut that's probably a small set, idk17:33
mriedemlibvirt things for sure though...17:33
sdaguemriedem: it does, so if there are important things, I'd suggest just adding topics17:34
sdaguemriedem: there is a reason I'm asking you what topics you want, because I expect you are one of the top users :)17:34
*** thorst_afk is now known as thorst17:34
sdagueso I'm totally good with mriedems private label topics list17:34
clarkbdnsmasq17:35
clarkbthats been useful to debug dhcp things int he past17:35
*** Jack_Iv has joined #openstack-nova17:35
sdagueclarkb: it has, though usually not with a spot check17:35
mriedemi dumped the current open e-r queries that rely on syslog in the patch17:35
sdagueit will still be in the journal dump17:35
mriedemi haven't dug into what they are looking for though, because i'm lunching17:36
sdaguemriedem: ok17:36
sdaguespeaking of, I need to do that17:36
sdaguethen will be intermittent for the rest of the day17:36
sdagueI fast approved the disable on use_journal - https://review.openstack.org/#/c/462163/ - so hopefully in a couple of hours that will be in17:37
*** Jack_Iv has quit IRC17:38
mriedemcool17:38
*** Jack_Iv has joined #openstack-nova17:38
mriedemcfriesen: are you going to be at the summit?17:39
mriedembecause there is a session where all of your questions can be answered!17:39
mriedemunless they have something to do with numa and pci, then we'll be asking you for the answeres17:40
mriedem*answers17:40
*** tbachman has joined #openstack-nova17:43
*** gjayavelu has joined #openstack-nova17:43
*** mtreinish has quit IRC17:46
*** slaweq has quit IRC17:46
*** slaweq has joined #openstack-nova17:46
*** fragatina has quit IRC17:49
mriedemjaypipes: i had no idea wookiee had 2 es17:51
*** slaweq has quit IRC17:51
jaypipesmriedem: you're welcome :)17:51
jaypipesTIL...17:51
mriedemi actually like this format17:51
mriedemputs mine to shame17:51
mriedemdamn you17:51
mriedemrxtx factors, ha17:53
*** felipemonteiro_ has joined #openstack-nova17:53
mriedemdidn't i see sfinucan trying to remove all of that?17:53
mriedemor maybe that's a joke b/c rxtx factors would normally be an extra spec, but aren't17:55
*** hongbin has quit IRC17:55
mriedemfor whatever reason17:55
*** annegentle has joined #openstack-nova17:55
*** felipemonteiro__ has quit IRC17:56
melwittdansmith: do you know what is the purpose of this stringify that puts extra quotes around a string? https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/fields.py#L26917:57
*** baoli has quit IRC17:57
sean-k-mooneymriedem: aren't rx/tx factors more of a qos thing. e.g. sould they still exist now that neutron has a qos api17:57
mriedemsean-k-mooney: they are only used for port qos yeah17:57
dansmithmelwitt: it's because that ends up in the repr, right?17:58
mriedemsean-k-mooney: https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L108817:58
mriedemsean-k-mooney: that's the only place we ues them for neutron in nova17:58
mriedembut they probably shouldn't be in nova17:58
melwittdansmith: yeah. I'm trying to wrap my head around this stuff for when a StringField contains a unicode thing. and I was wondering what the adding of the quotes around the StringField value does if it's already a string?17:58
mriedemif you want qos on your port, create the port in neutron and provide it when creating the server17:59
melwittdansmith: I'm probably dumbly missing something here17:59
dansmithmelwitt: it's just for display17:59
dansmithmelwitt: so you end up with Instance<name="foo bar"> and not Instance<name=foo bar>17:59
sean-k-mooneymriedem: so they were part of the old port qos support. i think this inclunces the libvirt xml generation somehow if i remember correctly17:59
*** baoli has joined #openstack-nova17:59
melwittdansmith: right, okay. thank you17:59
*** annegent_ has joined #openstack-nova17:59
*** annegentle has quit IRC18:02
*** tbachman_ has joined #openstack-nova18:02
mriedemsean-k-mooney: i don't see that happening18:02
mriedemsean-k-mooney: in fact the only place i see rxtx* in nova/virt is in the xenapi driver18:03
sean-k-mooneyya im looking at the git repo currently and i cant find it either but i taguht it used too..18:03
mriedemand it's based on a flag that's only set if you're using nova-network18:03
sean-k-mooneywell its set here https://github.com/openstack/nova/blob/501a0fe65afabb7e6a8b1e2d1ca0fbcea941d2d1/nova/tests/unit/virt/libvirt/test_vif.py#L50718:03
sean-k-mooneybut i dont know if its used18:03
mriedemsean-k-mooney: that's just a flavor18:03
mriedemthe rxtx_factor is defined on the nova flavor18:04
sean-k-mooneyyes18:04
*** tbachman has quit IRC18:04
mriedemwe don't use that when plugging the vifs in libvirt though, or creating the domain xml, from what i can tell18:04
*** tbachman_ is now known as tbachman18:04
johnthetubaguymriedem: thats a rackspace special for quark18:04
johnthetubaguymriedem: AFAIK18:04
mriedemit's probably time to deprecate rxtx_factor in our flavor api18:04
*** mvk has joined #openstack-nova18:04
mriedemand say if you need qos on ports, get it from neutron18:04
johnthetubaguymriedem: +1 that18:05
*** mtreinish has joined #openstack-nova18:05
mriedemsdague: jaypipes: dansmith: let's remember to put something in the onboarding session about "if you don't know why it's in nova, or what uses/used it, it's probably something to enable out of tree things for rackspace"18:05
mriedemi need constant reminding of that18:05
sean-k-mooneymriedem: ah look at https://docs.openstack.org/admin-guide/compute-flavors.html "this parameter is only available for Xen or NSX based systems."18:06
mriedemsean-k-mooney: ha18:06
mriedemnsx18:06
mriedemthe vmware driver doesn't use it18:06
mriedemwe should probably update the admin guide to basically say don't use that18:07
sean-k-mooneymriedem: if you try to use that with nutons bandwith limit qos policies you will get consusing behavior18:07
mriedemsean-k-mooney: this is the description from our compute api-ref: "The receive / transmit factor (as a float) that will be set on ports if the network backend supports the QOS extension. Otherwise it will be ignored. It defaults to 1.0."18:07
*** kaisers has quit IRC18:07
johnthetubaguymriedem: so here it is: https://github.com/openstack/nova/blob/cbbe486cd961e7f002c6e9ebdfdfa10527aa6b03/nova/network/manager.py#L55118:07
*** kaisers has joined #openstack-nova18:08
mriedemjohnthetubaguy: yeah that's what got me to the xen driver18:08
mriedemthe rxtx_cap18:08
johnthetubaguyyeah18:08
sean-k-mooneyis this code used only when nova networks is used?18:09
johnthetubaguymriedem: looks very nova-network only I think, given where that code lives.18:09
openstackgerritMatthew Treinish proposed openstack/nova master: Remove cloudpipe APIs  https://review.openstack.org/45947018:10
mtreinishmriedem: ^^^ I added the missing release note18:10
sean-k-mooneyjohnthetubaguy: mriedem so it can die with nova-networks in queens when cells v1 is removed18:10
openstackgerritJay Pipes proposed openstack/nova master: placement: tests for non-shared with shared  https://review.openstack.org/46189918:10
openstackgerritJay Pipes proposed openstack/nova master: placement: test for agg association not sharing  https://review.openstack.org/46190118:10
openstackgerritJay Pipes proposed openstack/nova master: placement: test non-shared out of inventory  https://review.openstack.org/46190018:10
openstackgerritJay Pipes proposed openstack/nova master: placement: shared resources when finding providers  https://review.openstack.org/46157418:10
*** lucasxu has quit IRC18:11
*** kaisers has quit IRC18:11
*** kaisers has joined #openstack-nova18:11
mriedemsean-k-mooney: no it's also used here for neutron https://github.com/openstack/nova/blob/501a0fe65afabb7e6a8b1e2d1ca0fbcea941d2d1/nova/network/neutronv2/api.py#L108718:11
*** lucasxu has joined #openstack-nova18:11
mriedemsean-k-mooney: but as you said, having a nova flavor override your qos settings from neutron is confusing at best18:12
mriedembroken at worst18:12
mriedemso in queens we should deprecate that field18:12
jaypipesdansmith: separated _get_all_with_shared() into a module-level function per your request. ^^^18:12
*** salv-orlando has quit IRC18:12
*** krtaylor has quit IRC18:12
sean-k-mooneymriedem: yes its prably best to have neutron be the only source of truth when it come to network qos18:12
sean-k-mooneymriedem: have the quots/rate limits been removed form nova yet?18:13
mriedemsean-k-mooney: the extra specs?18:14
mriedemprobably not18:14
sean-k-mooneyyes. i think they go hand in had with the rxtx_factor removal18:14
mriedemnote there is 0 contract on extra specs really18:14
jaypipesand edleafe, I added a fun little ASCII diagram to (probably poorly) describe the "butterfly join" ;)18:15
sean-k-mooneyyes but if is reqst a max bandwith or min in neutron nova should proably not override it18:15
mriedemsean-k-mooney: i don't disagree18:16
cfriesenmriedem: thanks....I am going to be at the Summit, as will Steven Webster (our actual PCI guy)18:16
cfriesenmriedem: reason I asked about the version was that we've got a weird case where one compute node is coming up after an upgrade with version 0 instead of version 918:17
jaypipesrfolco: answered your question on that error patch :)18:18
jaypipesrfolco: thanks much for hopping on that!18:18
mriedemcfriesen: i feel like that's something dansmith has seen before18:18
mriedemwell, rax has seen18:18
mriedemand with that i'm out for an hour for haircutting time18:19
*** slaweq has joined #openstack-nova18:19
*** mriedem is now known as mriedem_away18:19
sean-k-mooneymriedem: the bandwidth io extra spec discribed here https://docs.openstack.org/admin-guide/compute-flavors.html are the ones i was thinking of18:20
*** mnestratov has quit IRC18:21
*** fragatina has joined #openstack-nova18:21
sean-k-mooneymriedem_away: im pretty sure the dont always work correctly currently e.g. quota:vif_outbound_average=10 will have no effect on vhost-user and perhaps sriov also18:21
dansmithsdague: passed again..18:22
*** voelzmo has joined #openstack-nova18:23
rfolcojaypipes, nice. Thanks for clarifying. I first got the impression the change was impossible. Another thing that came to my attention is that previously we had in those errors what was the inventory conflicting. Now, without the DB API error, the error message is more generic. Do you see any problem there? Not sure how to get the info for inventory from DB API error.18:23
*** david-lyle has quit IRC18:24
*** dklyle has joined #openstack-nova18:24
jaypipesrfolco: not sure it's all that necessary to include the actual inventory in use. all that really matters is that there are *some* allocations for something... at that point, you can't delete the inventory at all...18:25
rfolcojaypipes, sounds reasonable. Thank you Jay.18:26
jaypipesany time.18:26
sean-k-mooneyjaypipes: quick question while you answering palcement api questions :) how would you feel about calls to the placement api from neutron as aprt of port binding?18:27
sean-k-mooneyi know if 2 backend right now that use vhost-user in a sligly less flexably way where the last part of the socket is used as an id to identify the port e.g. vhost-user-1 is port 1 on the backend18:29
edleafejaypipes: ooh, I like butterflies!18:29
sean-k-mooneyjaypipes: do you think it would be appropriate to create a resouce provider per backend to track the allocated ports via the placement api or would it be better to track the seperatly in the neutron db or something like etcd in the neutron side18:31
jaypipessean-k-mooney: thinking..18:32
jaypipessean-k-mooney: is the "backend" something that provides a particular resource class? and if so, what is that resource class?18:33
sean-k-mooneyjaypipes: the backend is a vswitch and it proveds a limited number of acclerated port and effectivly infinity software non acclerated ports18:34
jaypipessean-k-mooney: would there be multiple "backends" per compute node?18:35
sean-k-mooneyjaypipes: that would depend on how the rest of the intergration was done but gernarally no18:35
sean-k-mooneyyou could support multiple but i dont see a reason too18:35
jaypipessean-k-mooney: then why not have the "accelerated port" resources belong to the compute node resource provider?18:36
sean-k-mooneyjaypipes: you could i just was wondering if the idea of neutron caliming a resouce via the placement api so it new what numerb to use when creating the vhost user socket path would be ok18:37
jaypipessean-k-mooney: no, I think it's best that the flow be this:18:37
jaypipesa) MANO system calls neutron port create to create the port object18:38
jaypipesb) MANO system calls nova boot --nic <port_uuid> ...18:38
jaypipesc) Nova conductor constructs a request to scheduler/placement that asks for a provider which has 1 accelerated port available (possibly also with some NIC-related traits on the compute node resource provider record)(18:39
jaypipesd) Nova conductor makes the "claim" for all the resources in the allocation18:40
*** annegent_ has quit IRC18:40
*** annegentle has joined #openstack-nova18:41
jaypipese) nova-compute sees that the claim includes an allocation of an accelerated port and passes binding information to Neutron in the plug/bind operation.18:41
jaypipessean-k-mooney: ^^18:41
sean-k-mooneyjaypipes:  i get where your going with that. ya that makes sense more or less though it may mean i wont be able to reuse vnic types entirely the way i was hoping but it should still work18:42
jaypipessean-k-mooney: elaborate for me... how will you not be able to reuse vnic types?18:42
sean-k-mooneyjaypipes: well effectivly what resouce would be claimed may depend on the neutron backend so i was hoping to reuse the existing vnic types but that may or may not work18:43
jaypipessean-k-mooney: I guess I don't understand that... when is it decided which "backend" should be used? at neutron port create time, no?18:44
sean-k-mooneyjaypipes: the tenant is not directly aware of the ovs backend in use.18:45
sean-k-mooneybasically i would like to reuse vnic type direct for something that is like sriov but is slightly different18:45
sean-k-mooneylibvirt thinks its sriov18:46
jaypipessean-k-mooney: the request should be "get me an accelerated port with these characterstics", not "get my an accelerated port on this backend on this compute node", right?18:47
abhishekmelwitt: just a reminder, https://review.openstack.org/#/c/460305/18:47
sean-k-mooneyjaypipes: yes the request would be boot me vm with this port of vnic type (normal,dirct or virtio-fowrader)18:48
sean-k-mooneyvnic type direct could be sriov passthrough of a vf or a hardware offloaded port such as the melonox hardware datapath for ovs.18:49
jaypipessean-k-mooney: k. for the direct vnic_type, I pretty much understand the various resource classes and traits involved. for normal, there's not really any traits/resource classes involved (since it's as you say "infinite resource capacity for software vnics"). It's the virtio-forwarder vnic types I'm just not familiar with...18:49
*** dtp has quit IRC18:50
sean-k-mooneyvirtio-forwarder is a new vnic introduced by netronome for there smartnics it is vhost user with hardware accleration18:50
*** felipemonteiro_ has quit IRC18:50
*** felipemonteiro_ has joined #openstack-nova18:51
*** annegentle has quit IRC18:51
*** avolkov has quit IRC18:53
-openstackstatus- NOTICE: Gerrit on review.openstack.org is being restarted to accomodate a memory leak in Gerrit. Service should return shortly.18:53
*** slagle has quit IRC18:53
melwitthah, I was just about to say "gee, gerrit has been really slow today"18:53
*** krtaylor has joined #openstack-nova18:54
*** annegentle has joined #openstack-nova18:54
sean-k-mooneyjaypipes: anyway cool ill think about what you said because i didn want to also think about how to schdule base on the request also. thanks jay18:54
jaypipessean-k-mooney: not sure I was very helpful here :(18:55
*** kaisers has quit IRC18:55
*** felipemonteiro_ has quit IRC18:55
sean-k-mooneyyou were i was thinking of useing the workflow you were suggesting also but for the second backend that has this contratint but it should work for both.18:55
*** kaisers has joined #openstack-nova18:55
melwittabhishek: sorry, been busy working on a gate bug. about to comment now18:56
abhishekmelwitt: np18:56
*** mingyu has quit IRC18:56
armaxstevemar: ping18:57
*** baoli has quit IRC18:57
stevemarameade: pong18:57
*** baoli has joined #openstack-nova18:57
sean-k-mooneythe first backend was http://dpdk.org/browse/apps/spp/tree/ which NTT would like to enable next cycle. you can speciy the vhost-user socket path as it is hard coded to be /tmp/socket# where # is the id of the nfv forwarding instance.18:57
*** kaisers has quit IRC18:58
*** mingyu has joined #openstack-nova18:58
*** hongbin has joined #openstack-nova18:59
openstackgerritIldiko Vancsa proposed openstack/nova master: Remove check_detach  https://review.openstack.org/44667119:00
*** Jack_Iv has quit IRC19:00
sean-k-mooneysince neutron has to calulate the vhost-user path as part of the port bind it needs to calulate/reserve an nfvincstace id for that interface as part of the port bind so that it can tell nova what path to use and os-vif what instance id to use to map to that socket19:01
*** Jack_Iv has joined #openstack-nova19:01
openstackgerritAbhishek Sharma M proposed openstack/nova master: Catching OverQuota Exception  https://review.openstack.org/46030519:02
sean-k-mooneyjaypipes: anyway its time for dinner so ill catch you later.19:02
*** kaisers has joined #openstack-nova19:03
*** hongbin has quit IRC19:06
*** hongbin has joined #openstack-nova19:07
openstackgerritRafael Folco proposed openstack/nova master: Delete all inventory has its own method DELETE  https://review.openstack.org/46014719:09
*** baoli has quit IRC19:10
*** baoli has joined #openstack-nova19:11
*** baoli has quit IRC19:27
*** baoli has joined #openstack-nova19:27
*** crushil has quit IRC19:39
*** iceyao has joined #openstack-nova19:40
*** lpetrut has quit IRC19:41
*** salv-orlando has joined #openstack-nova19:41
*** Guest73837 has quit IRC19:42
*** iceyao has quit IRC19:44
*** tbachman has quit IRC19:44
*** awaugama has quit IRC19:44
*** tbachman has joined #openstack-nova19:46
*** mdrabe_ is now known as mdrabe19:49
*** mriedem_away is now known as mriedem19:49
*** markvoelker has quit IRC19:51
*** crushil has joined #openstack-nova19:53
*** dtp has joined #openstack-nova19:54
*** Guest73837 has joined #openstack-nova19:55
*** tbachman has quit IRC19:55
*** gyee has quit IRC19:55
*** artom has quit IRC19:57
*** artom has joined #openstack-nova19:57
*** markvoelker has joined #openstack-nova19:58
*** voelzmo has quit IRC20:00
*** voelzmo has joined #openstack-nova20:00
EmilienMmriedem: is deploying nova-api in wsgi is now "stable" ?20:02
*** abhishek has quit IRC20:03
*** liverpooler has quit IRC20:04
mriedemEmilienM: the nova patches are all merged i think20:05
mriedemyeah,20:06
mriedemthe devstack patch is passing but not merged yet https://review.openstack.org/#/c/457715/20:06
mriedemEmilienM: this was the docs patch from cdent https://review.openstack.org/#/c/461289/20:06
dansmithnot running on devstack in the gate for a while means it's not "stable" yet IMHO20:07
mriedemhttps://docs.openstack.org/developer/nova/wsgi.html20:07
dansmithespecially given the journal excitement from this morning20:07
mriedemdansmith: i'd agree with that20:07
mriedemand that20:07
mriedemEmilienM: so kick the tires, but don't come screaming if there are issues :)20:07
dansmiththe service creation patch looks good, I hadn't seen that until today20:07
dansmithbut that's an important piece20:07
*** iceyao has joined #openstack-nova20:08
*** pchavva has quit IRC20:09
*** ijw has joined #openstack-nova20:10
EmilienMmriedem: ok good to know20:11
EmilienMmriedem: i'll give a try where it's easily revertable (Puppet CI) and wait a bit in TripleO20:11
*** iceyao has quit IRC20:12
mriedemjaypipes: i'm on slide 27 btw20:12
mriedemnearly done20:12
*** voelzmo has quit IRC20:13
*** winston-d_ has quit IRC20:13
jaypipesmriedem: thx for reviewing. really helpful.20:15
*** abhishek has joined #openstack-nova20:15
jaypipesmriedem: btw, added a graphic for the nested resource providers...20:15
jaypipesmriedem: slide 30...20:15
mriedemjaypipes: i'm hoping it's a picture of the sarlacc http://media.liveauctiongroup.net/i/20282/19782537_3.jpg?v=8D147F2BC4F6C5020:16
dansmithmriedem: it'd have to be a lego sarlacc20:17
jaypipesright :)20:17
mriedemoh right20:17
mriedemoops20:17
*** tbachman has joined #openstack-nova20:17
dansmithjay has like one slide per minute.. I hope none of the audience members have epilepsy20:17
*** toure is now known as toure|gone20:18
jaypipesdansmith: :)20:18
*** abhishek has quit IRC20:19
jaypipesdansmith: especially since I added that strobe-light effect...20:19
mriedemjay is a professional slidesmith20:19
dansmithnot sure the refresh rate on the projector will tolerate jay's slide-per-second rate20:19
mriedemi'm sure it will be perfectly timed for effect20:19
jaypipeslmao20:19
mriedemjulie will be working the fog machine20:20
dansmithmaybe he's trying to make his deck like a flipbook animation sort of thing?20:20
mriedem"ladies and gentlemen, introducing, the beast from the southeast....JAY!"20:20
dansmitha mic will descend from the ceiling and the lights will dim to a spotlight20:20
mriedemright20:20
mriedemi just like the idea of julie handling the effects20:21
dansmithare you ready .... toooooo..... PLACE SOME RESOURCES SMARTLY????20:21
melwittat first I was dreading these presentations. but now ...20:21
*** catinthe_ has joined #openstack-nova20:21
jaypipesrofl20:21
jaypipesDJ Jules on the effects board.20:21
dansmith"Introducing Jay Pipes.. He hates inefficient schedulers and loves pug cuddles..."20:22
jaypipeshehe20:23
jaypipesdansmith: don't MAKE me add pug pics to my slides. you KNOW I will do that on a drop of a hat.20:23
dansmithlego pugs?20:23
jaypipesheh20:23
*** catintheroof has quit IRC20:24
melwittthe pug tucked into the blanket. that can be you sleeping soundly, using the new scheduler20:25
jaypipesmelwitt: this one? :) https://twitter.com/jaypipes/media20:26
sdaguemriedem / clarkb: I went through the list that mriedem posted in https://review.openstack.org/#/c/462231 - the only thing with an active hit touching syslog is coming in under the kernel tag (which will still be in that file)20:26
jaypipesor rather, this... https://twitter.com/jaypipes/status/801233017993719808/photo/120:26
melwittjaypipes: that one is awesome too. I was thinking of the one where it's tucked into bed. I don't remember which pug it was20:28
*** lucasxu has quit IRC20:28
openstackgerritMatthew Treinish proposed openstack/nova master: Fix typo in wsgi applications release note  https://review.openstack.org/46228320:28
mriedemsdague: ok20:28
mriedemdropped the -120:28
*** karthiks has quit IRC20:28
jaypipesmelwitt: ah. this one :) https://twitter.com/jaypipes/status/631102588319080448/photo/120:28
sdaguemriedem: it was a good thing to check20:28
melwittjaypipes: hahaha yeah that's it20:28
melwittI forgot the eyes were open, so that wouldn't be good for sleeping soundly20:29
*** mnestratov has joined #openstack-nova20:29
mriedemsean-k-mooney: btw https://bugs.launchpad.net/openstack-manuals/+bug/168805420:29
openstackLaunchpad bug 1688054 in openstack-manuals "Flavors in Administrator Guide - confusing description for rxtx factor" [Undecided,New]20:29
mriedemso we don't forget20:29
mriedemjaypipes: ok so a tree20:33
jaypipesya20:33
mriedem:)20:33
mriedemgdi20:33
mriedemyou and your dag20:33
jaypipes:)20:33
mriedemmutable media,20:33
mriedemdag20:33
mriedemnerd alert!20:33
*** cleong has quit IRC20:33
*** smatzek has quit IRC20:34
jaypipesmriedem: all trees are DAGs but not all DAGs are trees..20:34
mriedemi know20:34
mriedemoh, well,20:34
mriedemyeah nvm20:34
mriedemanyway, when i think of bidirectional i just think of graphs,20:34
mriedemand single parent directional i think of trees20:35
mriedemwhich are also graphs20:35
jaypipesit's cool, I removed the term DAG.20:35
mriedemi thought about not saying anything lest i look the fool20:35
dansmithand yet.. still happened20:37
melwittlol20:37
mriedemhey, you didn't know what sight words ere20:38
mriedem*were20:38
mriedemdummy20:38
mriedemthat reminds me to send you a picture of the thing i'm now a bad parent for20:39
dansmiththey definitely didn't call them that when I was a kid20:39
dansmithat least, not in my school20:39
*** krtaylor has quit IRC20:39
*** dave-mccowan has quit IRC20:41
*** Jack_Iv_ has joined #openstack-nova20:41
*** Jack_Iv has quit IRC20:44
*** catinthe_ has quit IRC20:45
*** kaisers has quit IRC20:45
*** dimtruck is now known as zz_dimtruck20:49
jaypipesmriedem, dansmith: OK, removed a few slides from the New Hope deck. Down to 26 "real" slides, which should be perfect for the slot.20:51
jaypipesnow... to whittle down the 1582 slides in my second deck...20:51
*** kaisers has joined #openstack-nova20:51
mriedemi can't possibly go through them all again, but i think it looks really nice20:53
jaypipesmriedem: I appreciate both your and dansmith's comments. thanks again.20:54
mriedemdansmith: cells v2 meeting in 6 minutes?20:54
dansmithI didn't offer any useful feedback20:54
dansmithmriedem: yeah20:54
jaypipesdansmith: you just being you is useful feedback. :)20:54
dansmithheh20:55
*** salv-orl_ has joined #openstack-nova20:56
*** kaisers has quit IRC20:57
*** kaisers has joined #openstack-nova20:58
melwittsdague: here's the unit test I'm trying for oslo.versionedobjects https://review.openstack.org/46228820:59
*** salv-orlando has quit IRC20:59
*** mdrabe has quit IRC21:00
*** Apoorva_ has joined #openstack-nova21:00
melwittto me it seems like __repr__ is doing the right thing to encode as ascii (the default) before returning if not python3, else things you could expect to work like str() would blow up in python2 if applied to the returned repr21:00
*** kaisers has quit IRC21:02
*** jose-phillips has joined #openstack-nova21:03
*** Apoorva has quit IRC21:04
*** zz_dimtruck is now known as dimtruck21:04
*** jose-phi_ has quit IRC21:05
*** jose-phillips has quit IRC21:05
*** dtp has quit IRC21:05
*** eharney has quit IRC21:05
*** jose-phillips has joined #openstack-nova21:07
*** rfolco has quit IRC21:09
*** bmace has joined #openstack-nova21:09
*** hongbin has quit IRC21:12
*** hongbin has joined #openstack-nova21:12
*** thorst has quit IRC21:13
*** krtaylor has joined #openstack-nova21:19
*** mingyu has quit IRC21:20
*** mingyu has joined #openstack-nova21:21
*** mingyu has quit IRC21:22
*** mingyu has joined #openstack-nova21:22
*** kiwi_rot has joined #openstack-nova21:22
*** gyee has joined #openstack-nova21:25
*** Sukhdev has joined #openstack-nova21:25
*** kiwi_rot has quit IRC21:27
mriedemdansmith: i'm not sure how you don't love 'modularity' as a theme...21:29
mriedemit makes total sense if you're trying to synergize21:30
dansmithmriedem: my mistake21:30
*** mnestratov has quit IRC21:31
*** esberglu has quit IRC21:34
*** kaisers has joined #openstack-nova21:36
*** tbachman has quit IRC21:36
clarkbmelwitt: if thats the case maybe you need to do a lossy repr?21:37
melwittmriedem, dansmith: what about hyperconvergence21:38
melwittclarkb: what do you mean?21:38
*** thorst has joined #openstack-nova21:38
mriedemmelwitt: only thought leaders who drive innovative results deal with hyperconvergance21:38
clarkbmelwitt: convert to ascii and ignore errors21:38
clarkbmelwitt: https://docs.python.org/2/library/codecs.html#codec-base-classes has a list of the encoding optisn you can use21:39
clarkbstrict is default which is why you get the error. But you could ignore or replace. or use backslashreplace21:40
*** kaisers has quit IRC21:41
*** thorst has quit IRC21:43
mriedemjaypipes: does 11am ET work tomorrow for you?21:51
mriedem30 min hangout?21:51
jaypipesmriedem: yup.21:51
*** brad[] has quit IRC21:52
*** jose-phillips has quit IRC21:54
*** kaisers has joined #openstack-nova21:55
*** Jack_Iv_ has quit IRC21:58
*** Jack_Iv has joined #openstack-nova21:58
EmilienMmriedem: sounds like re-enabling WSGI didn't work well: http://logs.openstack.org/81/462281/1/check/gate-puppet-openstack-integration-4-scenario002-tempest-centos-7/7a37824/logs/nova/nova-compute.txt.gz#_2017-05-03_20_58_25_08322:00
*** jose-phillips has joined #openstack-nova22:01
mriedemEmilienM: that doesn't look like anything that would be related to running with wsgi22:02
EmilienMI agree22:02
*** cfriesen has quit IRC22:03
*** brad[] has joined #openstack-nova22:05
*** ijw has quit IRC22:06
*** ijw has joined #openstack-nova22:06
*** gouthamr has quit IRC22:07
*** jamesdenton has quit IRC22:08
*** baoli has quit IRC22:09
*** Apoorva_ has quit IRC22:10
*** Apoorva has joined #openstack-nova22:10
*** ijw has quit IRC22:11
*** ijw has joined #openstack-nova22:11
*** marst has quit IRC22:12
*** tbachman has joined #openstack-nova22:15
*** fragatin_ has joined #openstack-nova22:21
*** Sukhdev has quit IRC22:23
*** haplo37 has joined #openstack-nova22:23
*** Guest73837 has quit IRC22:23
*** ediardo_ has joined #openstack-nova22:23
*** mlavalle has quit IRC22:25
*** raorn_ has joined #openstack-nova22:25
*** jgrassle1 has joined #openstack-nova22:25
*** raorn_ has quit IRC22:26
*** raorn_ has joined #openstack-nova22:26
*** mikal_ has joined #openstack-nova22:26
*** igordcar1 has joined #openstack-nova22:26
*** tobasco_ has joined #openstack-nova22:26
openstackgerritmelanie witt proposed openstack/nova master: Use six.text_type() when logging Instance object  https://review.openstack.org/46191222:26
*** Guest73837 has joined #openstack-nova22:26
*** alex_xu_ has joined #openstack-nova22:27
*** htruta` has joined #openstack-nova22:27
*** mariojv_ has joined #openstack-nova22:27
*** irenab_ has joined #openstack-nova22:27
*** rodrigod` has joined #openstack-nova22:27
*** slagle has joined #openstack-nova22:27
*** erlon has quit IRC22:28
*** alex_xu has quit IRC22:28
*** haplo37_ has quit IRC22:28
*** ediardo has quit IRC22:28
*** fragatina has quit IRC22:28
*** raorn has quit IRC22:28
*** rodrigods has quit IRC22:28
*** htruta has quit IRC22:28
*** jgrassler has quit IRC22:28
*** irenab has quit IRC22:28
*** mikal has quit IRC22:28
*** igordcard has quit IRC22:28
*** mariojv has quit IRC22:28
*** test222 has quit IRC22:28
*** tobasco has quit IRC22:28
*** harlowja has quit IRC22:28
*** xgerman has quit IRC22:28
*** rushiagr has quit IRC22:28
*** irenab_ is now known as irenab22:28
*** ediardo_ is now known as ediardo22:29
melwittmriedem, dansmith: there's the hopefully final version. ^ I'm not sure if the unit test is good enough, if I should instead do an exact comparison of the repr22:29
*** artom has quit IRC22:30
*** artom has joined #openstack-nova22:30
*** gouthamr has joined #openstack-nova22:31
*** artom has quit IRC22:32
*** salv-orl_ has quit IRC22:32
*** artom has joined #openstack-nova22:32
*** artom has quit IRC22:33
*** artom has joined #openstack-nova22:33
melwittclarkb: the repr is already converting to ascii with no errors. the UnicodeDecodeError is raised here https://github.com/python/cpython/blob/2e576f5/Lib/logging/__init__.py#L338 which implies it's trying to do an implicit conversion to unicode. I don't understand that22:36
*** jose-phillips has quit IRC22:36
*** lyan has quit IRC22:40
mriedemmelwitt: nit inline22:40
mriedemotherwise lgtm22:40
melwittmriedem: k, looking22:40
melwittah, thanks22:41
*** baoli has joined #openstack-nova22:43
*** baoli has quit IRC22:43
*** baoli has joined #openstack-nova22:44
openstackgerritmelanie witt proposed openstack/nova master: Use six.text_type() when logging Instance object  https://review.openstack.org/46191222:45
*** xyang1 has quit IRC22:47
*** rushiagr has joined #openstack-nova22:48
mriedemmelwitt: should you be using assertIsInstance instead of assertIs?22:50
mriedemi'm not entirely sure what the difference is, but i thought assertIs was like, == but without checking __eq__22:50
mriedemlike same object in memory22:50
*** aloga has quit IRC22:50
mriedemhttps://docs.python.org/2/library/unittest.html#unittest.TestCase.assertIs22:51
*** aloga has joined #openstack-nova22:51
melwittmriedem: I tried that first and it failed and said to check exact type, use assertIs https://docs.python.org/2/library/unittest.html#unittest.TestCase.assertIsInstance22:51
*** annegentle has quit IRC22:51
mriedemoh huh, weird22:51
mriedemok22:52
mriedem+222:52
melwittthanks22:52
*** mingyu has quit IRC22:53
mriedemmtreinish: a few things in https://review.openstack.org/#/c/459470/ yet22:59
dansmithmriedem: is and == are different23:01
*** sdague has quit IRC23:02
mtreinishmriedem: ok, I'll respin that now23:03
dansmith>>> 1 == True23:03
dansmithTrue23:03
dansmith>>> 1 is True23:03
dansmithFalse23:03
dansmithmriedem: ah you said it above and I missed.. "same object in memory" is right23:04
dansmithmriedem: but assertIsInstance is a subclass check23:05
*** hongbin has quit IRC23:08
*** larainema has joined #openstack-nova23:10
*** annegentle has joined #openstack-nova23:12
openstackgerritMatthew Treinish proposed openstack/nova master: Remove cloudpipe APIs  https://review.openstack.org/45947023:16
mtreinishmriedem: ^^^ that should address your comments23:16
*** jamesdenton has joined #openstack-nova23:16
*** annegentle has quit IRC23:17
*** Jack_Iv has quit IRC23:17
*** Jack_Iv has joined #openstack-nova23:18
*** adisky_ has quit IRC23:19
*** dklyle has quit IRC23:23
*** rfolco has joined #openstack-nova23:24
*** rfolco has quit IRC23:24
*** jamesdenton has quit IRC23:25
*** jamesdenton has joined #openstack-nova23:26
*** harlowja has joined #openstack-nova23:31
*** chyka has quit IRC23:33
*** chyka has joined #openstack-nova23:34
*** Apoorva_ has joined #openstack-nova23:37
*** chyka has quit IRC23:39
*** Apoorva__ has joined #openstack-nova23:40
*** Apoorva_ has quit IRC23:40
*** Apoorva has quit IRC23:41
*** gcb has quit IRC23:42
*** ijw has quit IRC23:52
*** ijw has joined #openstack-nova23:52
*** mingyu has joined #openstack-nova23:54
*** ijw has quit IRC23:55
*** ijw has joined #openstack-nova23:57
*** ijw has quit IRC23:57
*** ijw has joined #openstack-nova23:58
*** thorst has joined #openstack-nova23:58
*** iceyao has joined #openstack-nova23:58
*** mingyu has quit IRC23:58
*** coreywright has quit IRC23:58
*** thorst has quit IRC23:58
*** masber has joined #openstack-nova23:58

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