Wednesday, 2016-09-14

*** lucasagomes has quit IRC00:00
*** hoangcx2 has quit IRC00:01
*** lucasagomes has joined #openstack-nova00:02
*** ssurana has quit IRC00:04
*** tblakes has quit IRC00:04
*** thorst_ has joined #openstack-nova00:04
melwittmriedem: a-ha you're here! I thought you should look at this fix for nova-manage db archive_deleted_rows. I tagged it as newton-rc-potential, let me know if it shouldn't be https://review.openstack.org/#/c/323684/00:05
*** Swami has quit IRC00:06
*** ducttape_ has joined #openstack-nova00:07
*** lucasagomes has quit IRC00:07
*** tblakes has joined #openstack-nova00:08
*** penick has quit IRC00:08
*** zul has quit IRC00:08
mriedemi don't think that's a regression00:09
*** lucasagomes has joined #openstack-nova00:10
*** thorst_ has quit IRC00:11
melwittmriedem: oh. I thought it had been operational until this00:13
mriedemmelwitt: similar https://review.openstack.org/#/c/345191/400:13
melwittoh, okay. my bad00:13
*** lucasagomes has quit IRC00:14
*** ociuhandu has quit IRC00:15
*** ducttape_ has quit IRC00:16
*** lucasagomes has joined #openstack-nova00:17
*** MikeG451 has quit IRC00:17
*** stewie925 has quit IRC00:17
*** cmartin has quit IRC00:17
*** xenogear has quit IRC00:17
*** sdake_ has quit IRC00:17
*** sdake has joined #openstack-nova00:17
*** jinli has quit IRC00:18
openstackgerritTakashi NATSUME proposed openstack/nova: Add a hacking rule for string interpolation at logging  https://review.openstack.org/33484400:18
*** sai has quit IRC00:19
*** sai has joined #openstack-nova00:19
*** syjulian has joined #openstack-nova00:22
*** gongysh has quit IRC00:26
*** sai has quit IRC00:27
*** lucasagomes has quit IRC00:27
*** sai has joined #openstack-nova00:30
*** krtaylor has joined #openstack-nova00:30
*** lucasagomes has joined #openstack-nova00:30
mriedemmelwitt: do we delete migrations when we delete instances?00:30
mriedemor is the problem that we don't have a foreign key between instances and migrations?00:30
melwittmriedem: I doubt it00:30
melwittmriedem: I think there is a FK which causes the problem. it's that migrations are never deleted but instances are. similar to instance_actions was my assumption00:31
mriedemyeah, so https://github.com/openstack/nova/blob/8211d3fade1b44173a8d6d54d1918b9f0bc26d4e/nova/db/sqlalchemy/api.py#L180100:32
mriedemwe should probably also delete migrations when the instance is deleted00:32
*** MikeG451 has joined #openstack-nova00:32
mriedemat some point we should just make https://github.com/openstack/nova/blob/8211d3fade1b44173a8d6d54d1918b9f0bc26d4e/nova/db/sqlalchemy/api.py#L1801 smarter and look for referenced entries in other tables and delete those rather than hard-coding it00:32
*** kaisers1 has joined #openstack-nova00:32
*** kaisers2 has joined #openstack-nova00:33
*** kaisers has quit IRC00:34
*** lucasagomes has quit IRC00:34
*** 32NABYQA1 has quit IRC00:35
melwittmriedem: makes sense00:35
*** lixiaoy1 has joined #openstack-nova00:37
*** lucasagomes has joined #openstack-nova00:38
*** itlinux has quit IRC00:40
*** itlinux has joined #openstack-nova00:41
*** gongysh has joined #openstack-nova00:41
pumaranikarI am checking os-vif logging in unit tests. https://bugs.launchpad.net/nova/+bug/1615676. I don't find logging config in os-vif repo. any pointers?00:42
openstackLaunchpad bug 1615676 in OpenStack Compute (nova) "os-vif log messages corrupt and pollute test output" [Wishlist,New] - Assigned to Pushkar Umaranikar (pushkar-umaranikar)00:42
*** vladikr has quit IRC00:42
*** browne has quit IRC00:43
*** dave-mccowan has joined #openstack-nova00:43
*** smatzek has joined #openstack-nova00:44
*** rrecio_ has quit IRC00:44
*** raunak has quit IRC00:45
*** jinli has joined #openstack-nova00:45
*** lucasagomes has quit IRC00:46
*** sai has quit IRC00:47
*** lucasagomes has joined #openstack-nova00:48
*** tuanla_ has joined #openstack-nova00:49
openstackgerritHieu LE proposed openstack/nova: Config options: improve libvirt help text (2)  https://review.openstack.org/31655000:52
*** ijw has joined #openstack-nova00:54
*** tjones has left #openstack-nova00:55
*** stewie925 has joined #openstack-nova00:56
*** itlinux has quit IRC00:56
*** lucasagomes has quit IRC00:57
*** jfding has quit IRC00:58
*** cheneydc has joined #openstack-nova00:59
*** devananda is now known as devananda|dinner00:59
*** ijw has quit IRC00:59
*** lucasagomes has joined #openstack-nova01:00
*** itlinux has joined #openstack-nova01:00
*** gyee has quit IRC01:03
*** tbachman has quit IRC01:03
*** lucasagomes has quit IRC01:04
*** smatzek has quit IRC01:07
*** lucasagomes has joined #openstack-nova01:07
*** sai has joined #openstack-nova01:08
*** thorst_ has joined #openstack-nova01:09
*** ducttape_ has joined #openstack-nova01:09
*** cmartin has joined #openstack-nova01:09
*** xenogear has joined #openstack-nova01:10
*** Sukhdev has quit IRC01:10
*** thorst_ has quit IRC01:12
*** lucasagomes has quit IRC01:12
*** thorst_ has joined #openstack-nova01:14
*** lucasagomes has joined #openstack-nova01:14
*** yuntongjin has joined #openstack-nova01:15
*** gouthamr has quit IRC01:17
*** lucasagomes has quit IRC01:19
*** baoli has joined #openstack-nova01:20
*** lucasagomes has joined #openstack-nova01:21
*** yassou has quit IRC01:22
*** hoangcx has joined #openstack-nova01:26
*** jamielennox is now known as jamielennox|away01:26
*** jdurgin has joined #openstack-nova01:27
*** cheneydc has quit IRC01:27
*** moshele has joined #openstack-nova01:29
*** lucasagomes has quit IRC01:30
*** sai has quit IRC01:32
*** mingyu has joined #openstack-nova01:32
openstackgerrithuan proposed openstack/nova: xenapi: support the hotplug of a neutron port  https://review.openstack.org/36550501:33
*** lucasagomes has joined #openstack-nova01:33
*** hoangcx has quit IRC01:33
*** tri2sing has quit IRC01:34
*** sai has joined #openstack-nova01:34
*** mingyu has quit IRC01:36
*** artom has quit IRC01:38
*** itlinux has quit IRC01:39
*** artom has joined #openstack-nova01:39
*** salv-orlando has joined #openstack-nova01:41
*** tblakes has quit IRC01:41
*** sai has quit IRC01:42
*** nic has quit IRC01:42
*** salv-orl_ has quit IRC01:43
*** sai has joined #openstack-nova01:44
*** penick has joined #openstack-nova01:44
*** sdake_ has joined #openstack-nova01:45
*** sdake_ has quit IRC01:45
*** sdake_ has joined #openstack-nova01:45
*** itlinux has joined #openstack-nova01:46
*** sdake has quit IRC01:47
*** mriedem has quit IRC01:49
*** hoangcx has joined #openstack-nova01:50
*** Apoorva_ has joined #openstack-nova01:51
*** ducttape_ has quit IRC01:52
*** thorst_ has quit IRC01:52
*** yuntongjin has quit IRC01:53
*** thorst_ has joined #openstack-nova01:53
*** jaypipes has joined #openstack-nova01:54
*** annegentle has joined #openstack-nova01:54
*** yamahata has quit IRC01:55
*** Apoorva has quit IRC01:55
*** ijw has joined #openstack-nova01:56
*** Apoorva_ has quit IRC01:56
*** itlinux has quit IRC01:56
*** hoangcx has quit IRC01:57
*** sai has quit IRC01:59
*** sai has joined #openstack-nova02:00
*** thorst_ has quit IRC02:02
*** unicell has quit IRC02:02
*** ijw has quit IRC02:02
*** ducttape_ has joined #openstack-nova02:04
*** hoangcx has joined #openstack-nova02:04
*** ducttape_ has quit IRC02:04
*** lucasagomes has quit IRC02:06
*** lucasagomes has joined #openstack-nova02:07
*** yuntongjin has joined #openstack-nova02:10
*** sai has quit IRC02:13
*** gjayavelu has quit IRC02:14
*** gongysh has quit IRC02:14
*** lucasagomes has quit IRC02:15
*** sai has joined #openstack-nova02:15
*** lucasagomes has joined #openstack-nova02:17
*** yuntongjin has quit IRC02:17
*** hoangcx has quit IRC02:18
*** ijw has joined #openstack-nova02:19
*** woodster_ has quit IRC02:19
*** thorst_ has joined #openstack-nova02:20
*** thorst_ has quit IRC02:21
*** lucasagomes has quit IRC02:22
*** jdurgin has quit IRC02:22
*** ayoung has joined #openstack-nova02:23
*** lucasagomes has joined #openstack-nova02:23
*** ijw has quit IRC02:23
*** zul has joined #openstack-nova02:27
*** lucasagomes has quit IRC02:28
*** zul has quit IRC02:28
*** zul has joined #openstack-nova02:28
*** lucasagomes has joined #openstack-nova02:29
*** lucasagomes has quit IRC02:34
*** thorst_ has joined #openstack-nova02:36
*** lucasagomes has joined #openstack-nova02:37
*** dave-mccowan has quit IRC02:38
*** lucasagomes has quit IRC02:41
alex_xujaypipes: thanks for review https://review.openstack.org/#/c/369147/02:41
*** moshele has quit IRC02:42
*** ociuhandu has joined #openstack-nova02:42
*** lucasagomes has joined #openstack-nova02:43
*** sai has quit IRC02:45
*** sai has joined #openstack-nova02:47
*** akshai has joined #openstack-nova02:48
*** lucasagomes has quit IRC02:48
*** akshai_ has joined #openstack-nova02:49
*** stevemar_ is now known as stevemar02:49
*** lucasagomes has joined #openstack-nova02:49
*** akshai has quit IRC02:53
*** mdrabe has quit IRC02:53
openstackgerritZhenyu Zheng proposed openstack/nova: Remove mox in tests/unit/objects/test_security_group  https://review.openstack.org/30713502:53
*** sai has quit IRC02:54
*** ssurana has joined #openstack-nova02:55
*** sai has joined #openstack-nova02:55
*** lucasagomes has quit IRC02:57
*** itlinux has joined #openstack-nova02:57
*** lucasagomes has joined #openstack-nova02:58
openstackgerritZhenyu Zheng proposed openstack/nova: Remove mox in tests/unit/objects/test_instance_info_cache  https://review.openstack.org/30715503:00
*** jfding has joined #openstack-nova03:01
*** annegentle has quit IRC03:02
*** lucasagomes has quit IRC03:03
openstackgerritZhenyu Zheng proposed openstack/nova: Remove mox in tests/unit/objects/test_pci_device  https://review.openstack.org/30761503:03
*** zul has quit IRC03:05
*** ducttape_ has joined #openstack-nova03:05
*** lucasagomes has joined #openstack-nova03:05
openstackgerritZhenyu Zheng proposed openstack/nova: Remove mox in tests/unit/objects/test_compute_node  https://review.openstack.org/30762503:06
*** gjayavelu has joined #openstack-nova03:07
*** itlinux has quit IRC03:07
*** NightKhaos has quit IRC03:08
*** mingyu has joined #openstack-nova03:08
*** sai has quit IRC03:08
*** sai has joined #openstack-nova03:10
*** lucasagomes has quit IRC03:10
*** raunak has joined #openstack-nova03:10
*** lucasagomes has joined #openstack-nova03:13
*** akshai_ has quit IRC03:19
*** lucasagomes has quit IRC03:21
*** lucasagomes has joined #openstack-nova03:23
*** sai has quit IRC03:26
*** sai has joined #openstack-nova03:27
*** lucasagomes has quit IRC03:28
*** lucasagomes has joined #openstack-nova03:29
*** Sukhdev has joined #openstack-nova03:36
*** ducttape_ has quit IRC03:38
*** baoli has quit IRC03:39
*** hoangcx has joined #openstack-nova03:40
*** lucasagomes has quit IRC03:41
*** lucasagomes has joined #openstack-nova03:43
*** itlinux has joined #openstack-nova03:46
*** takedakn has joined #openstack-nova03:47
*** sai has quit IRC03:47
*** lucasagomes has quit IRC03:48
*** sai has joined #openstack-nova03:49
*** lucasagomes has joined #openstack-nova03:50
*** raunak has quit IRC03:50
*** thorst_ has quit IRC03:50
*** jdurgin has joined #openstack-nova03:51
*** lucasagomes has quit IRC03:54
*** shangxdy has joined #openstack-nova03:54
*** itlinux has quit IRC03:56
*** lucasagomes has joined #openstack-nova03:56
*** sai has quit IRC03:58
*** sai has joined #openstack-nova03:59
*** lucasagomes has quit IRC04:01
*** thorst_ has joined #openstack-nova04:02
*** takedakn has quit IRC04:03
*** shangxdy has quit IRC04:04
*** lucasagomes has joined #openstack-nova04:04
*** Sukhdev has quit IRC04:05
*** sai has quit IRC04:05
*** sai_ has joined #openstack-nova04:08
*** vishwanathj is now known as vishwanathj_zzz04:09
*** lucasagomes has quit IRC04:12
*** lucasagomes has joined #openstack-nova04:13
*** phuongnh has joined #openstack-nova04:15
*** lucasagomes has quit IRC04:17
*** lucasagomes has joined #openstack-nova04:19
*** thorst_ has quit IRC04:20
*** tbachman has joined #openstack-nova04:23
*** Sukhdev has joined #openstack-nova04:24
*** lucasagomes has quit IRC04:24
*** takashin has joined #openstack-nova04:24
*** lucasagomes has joined #openstack-nova04:26
*** diga has joined #openstack-nova04:28
*** thorst_ has joined #openstack-nova04:28
*** armax has quit IRC04:29
*** thorst_ has quit IRC04:33
*** lucasagomes has quit IRC04:33
*** lucasagomes has joined #openstack-nova04:35
*** itlinux has joined #openstack-nova04:37
*** ducttape_ has joined #openstack-nova04:39
*** pbandark has joined #openstack-nova04:40
*** raunak has joined #openstack-nova04:41
*** kaisers has joined #openstack-nova04:43
*** ducttape_ has quit IRC04:43
*** jaosorior has joined #openstack-nova04:44
*** yamahata has joined #openstack-nova04:44
*** sai_ has quit IRC04:44
*** lucasagomes has quit IRC04:44
*** sai_ has joined #openstack-nova04:45
*** itlinux has quit IRC04:47
*** lucasagomes has joined #openstack-nova04:47
*** Sukhdev has quit IRC04:47
*** mingyu has quit IRC04:50
*** lucasagomes has quit IRC04:51
*** thorst_ has joined #openstack-nova04:52
*** lucasagomes has joined #openstack-nova04:53
*** lixiaoy1 has quit IRC04:53
*** moshele has joined #openstack-nova04:54
*** thorst_ has quit IRC04:56
*** lucasagomes has quit IRC04:58
*** lucasagomes has joined #openstack-nova04:58
*** sai_ has quit IRC04:59
*** Jeffrey4l_ has joined #openstack-nova05:00
*** jaosorior has quit IRC05:00
*** claudiub has joined #openstack-nova05:01
*** jaosorior has joined #openstack-nova05:01
*** penick_ has joined #openstack-nova05:02
*** sai has joined #openstack-nova05:03
*** lucasagomes has quit IRC05:03
*** thorst_ has joined #openstack-nova05:03
*** Jeffrey4l has quit IRC05:04
*** lucasagomes has joined #openstack-nova05:04
*** penick has quit IRC05:04
*** penick_ is now known as penick05:04
*** sai has quit IRC05:07
*** jaypipes has quit IRC05:08
*** thorst_ has quit IRC05:08
*** lucasagomes has quit IRC05:09
*** sai_ has joined #openstack-nova05:09
*** edand has joined #openstack-nova05:11
*** lucasagomes has joined #openstack-nova05:11
*** scsnow has quit IRC05:14
*** armax has joined #openstack-nova05:15
*** ratailor has joined #openstack-nova05:15
*** thorst_ has joined #openstack-nova05:15
*** phschwartz has quit IRC05:15
*** mordred has quit IRC05:16
*** gongysh has joined #openstack-nova05:16
*** sai_ has quit IRC05:17
*** lpetrut has joined #openstack-nova05:18
*** raunak has quit IRC05:18
*** sai_ has joined #openstack-nova05:19
*** phschwartz has joined #openstack-nova05:19
*** takedakn has joined #openstack-nova05:19
*** lucasagomes has quit IRC05:19
*** ssurana has quit IRC05:20
*** mordred has joined #openstack-nova05:20
*** thorst_ has quit IRC05:20
*** lucasagomes has joined #openstack-nova05:23
*** sai_ has quit IRC05:25
*** sai_ has joined #openstack-nova05:27
*** lucasagomes has quit IRC05:27
*** takedakn has quit IRC05:27
*** lucasagomes has joined #openstack-nova05:28
*** rcernin has joined #openstack-nova05:33
*** lucasagomes has quit IRC05:33
*** lucasagomes has joined #openstack-nova05:35
*** sai_ has quit IRC05:35
*** thorst_ has joined #openstack-nova05:37
*** sai has joined #openstack-nova05:37
*** giri has joined #openstack-nova05:40
*** karthiks has joined #openstack-nova05:40
*** jamielennox|away is now known as jamielennox05:40
*** ducttape_ has joined #openstack-nova05:40
*** gjayavelu has quit IRC05:44
*** sai has quit IRC05:44
*** ducttape_ has quit IRC05:45
*** lucasagomes has quit IRC05:45
*** armax has quit IRC05:45
*** sai has joined #openstack-nova05:45
*** lucasagomes has joined #openstack-nova05:47
*** unicell has joined #openstack-nova05:47
*** itlinux has joined #openstack-nova05:47
*** lucasagomes has quit IRC05:51
*** nkrinner_afk is now known as nkrinner05:52
*** sai has quit IRC05:52
*** sai_ has joined #openstack-nova05:55
*** lucasagomes has joined #openstack-nova05:55
*** JoseMello has joined #openstack-nova05:55
*** raunak has joined #openstack-nova05:55
*** lucasagomes has quit IRC05:59
*** mjura has joined #openstack-nova06:00
*** itlinux has quit IRC06:01
*** adisky has quit IRC06:01
*** lucasagomes has joined #openstack-nova06:02
*** sai_ has quit IRC06:02
*** artom has quit IRC06:02
*** sai_ has joined #openstack-nova06:03
*** artom has joined #openstack-nova06:03
*** gcb has joined #openstack-nova06:04
*** mingyu has joined #openstack-nova06:07
*** paul-carlton2 has joined #openstack-nova06:08
*** lucasagomes has quit IRC06:09
*** lucasagomes has joined #openstack-nova06:10
*** mingyu has quit IRC06:12
*** sai_ has quit IRC06:16
*** abalutoiu has quit IRC06:18
*** sai_ has joined #openstack-nova06:18
*** salv-orlando has quit IRC06:18
*** lucasagomes has quit IRC06:18
*** sridharg has joined #openstack-nova06:19
*** lucasagomes has joined #openstack-nova06:20
*** nherciu has joined #openstack-nova06:22
*** unicell1 has joined #openstack-nova06:25
*** unicell has quit IRC06:26
*** lpetrut has quit IRC06:28
*** raunak has quit IRC06:29
*** jaosorior has quit IRC06:29
*** jaosorior has joined #openstack-nova06:30
*** pcaruana has joined #openstack-nova06:32
*** hoangcx has left #openstack-nova06:34
*** aswadr_ has joined #openstack-nova06:34
*** liverpooler has joined #openstack-nova06:35
*** itlinux has joined #openstack-nova06:35
*** dimtruck is now known as zz_dimtruck06:36
*** prateek has joined #openstack-nova06:38
*** code-R has joined #openstack-nova06:38
*** sahid has joined #openstack-nova06:39
*** sai_ has quit IRC06:40
*** code-R_ has joined #openstack-nova06:41
*** sai has joined #openstack-nova06:42
*** andreas_s has joined #openstack-nova06:42
*** code-R has quit IRC06:44
*** lucasagomes has quit IRC06:44
*** itlinux has quit IRC06:46
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Use assertEqual() instead of assertDictEqual()  https://review.openstack.org/34660906:46
*** lucasagomes has joined #openstack-nova06:46
*** namnh has joined #openstack-nova06:48
*** salv-orlando has joined #openstack-nova06:50
*** lucasagomes has quit IRC06:51
*** unicell has joined #openstack-nova06:52
*** sai has quit IRC06:52
*** lucasagomes has joined #openstack-nova06:53
*** sai has joined #openstack-nova06:53
*** unicell1 has quit IRC06:54
*** EinstCrazy has joined #openstack-nova06:55
*** tesseract- has joined #openstack-nova06:57
*** zz_dimtruck is now known as dimtruck06:57
*** EinstCra_ has joined #openstack-nova06:58
*** EinstCrazy has quit IRC06:58
*** lucasagomes has quit IRC07:00
*** lucasagomes has joined #openstack-nova07:00
*** sai has quit IRC07:01
*** abalutoiu has joined #openstack-nova07:01
*** ManishD has joined #openstack-nova07:02
*** FL1SK has quit IRC07:02
*** ManishD has quit IRC07:03
*** sai has joined #openstack-nova07:03
*** ManishD has joined #openstack-nova07:03
*** hoangcx has joined #openstack-nova07:04
*** EinstCra_ has quit IRC07:04
*** trams_ has quit IRC07:04
*** lucasagomes has quit IRC07:05
*** trams has joined #openstack-nova07:05
*** jsheeren has joined #openstack-nova07:05
openstackgerritGhanshyam Mann proposed openstack/nova: Complete verification for os-cloudpipe.inc  https://review.openstack.org/36918207:06
*** lucasagomes has joined #openstack-nova07:06
*** jdurgin has quit IRC07:06
*** garyk has joined #openstack-nova07:06
*** jistr is now known as jistr|mtgs07:09
*** jistr|mtgs is now known as jistr|mtg07:09
*** garyk has quit IRC07:10
*** dimtruck is now known as zz_dimtruck07:10
*** sai has quit IRC07:10
openstackgerritGhanshyam Mann proposed openstack/nova: Correct image.inc for heading  https://review.openstack.org/36914107:11
gmannalex_xu: hope this is fine now - https://review.openstack.org/#/c/369182/07:11
gmannalex_xu: also very small one  - https://review.openstack.org/#/c/369141/107:11
gmannhttps://review.openstack.org/#/c/369141/07:11
openstackgerritNguyen Phuong An proposed openstack/nova: Prevent use filter() to meet python2,3  https://review.openstack.org/36986207:12
alex_xugmann: let me check it07:12
openstackgerritAlvaro Lopez Garcia proposed openstack/nova: Ensure that periodic reclaim cleans DB deleted instances  https://review.openstack.org/32325007:12
*** thorst_ has quit IRC07:12
gmannalex_xu: Thanks07:12
*** sai has joined #openstack-nova07:12
gmannalex_xu: next week i will be in qa code sprint so thinking to fix api-ref one as many i can07:13
*** garyk has joined #openstack-nova07:13
alex_xugmann: cool07:13
*** lucasagomes has quit IRC07:13
alex_xugmann: will try to review those asap07:13
alex_xugmann: should we say list, show, create, update image and image metadata?07:14
gmannalex_xu: ah, yea for metadata07:14
gmannll fix07:14
*** lucasagomes has joined #openstack-nova07:15
*** hoangcx has quit IRC07:15
alex_xugmann: thanks07:15
*** wendar has quit IRC07:15
gmannalex_xu: oh you mean to merge both?07:15
*** wendar has joined #openstack-nova07:15
gmannalex_xu: actually there is  no create, update image thing so it will be explicitly for that07:16
alex_xugmann: yea, just think of maybe we should metion metadata?07:16
alex_xugmann: yea, agree07:16
alex_xugmann: oops, there is second line, it mentioned metadata07:17
gmannyea07:17
alex_xugmann: sorry, my fault, let me +207:17
gmannalex_xu: np. Thanks07:17
alex_xugmann: np07:17
*** paul-carlton2 has quit IRC07:19
*** unicell1 has joined #openstack-nova07:19
*** unicell has quit IRC07:19
*** mingyu has joined #openstack-nova07:20
*** lucasagomes has quit IRC07:20
*** paul-carlton2 has joined #openstack-nova07:20
*** lucasagomes has joined #openstack-nova07:21
*** rmart04 has joined #openstack-nova07:21
*** gongysh has quit IRC07:24
*** markus_z has joined #openstack-nova07:24
*** jaosorior has quit IRC07:27
*** EinstCrazy has joined #openstack-nova07:27
*** jaosorior has joined #openstack-nova07:27
*** NightKhaos has joined #openstack-nova07:28
*** jaosorior has quit IRC07:29
bauzasmarkus_z: around ?07:29
bauzasmarkus_z: I need some of your console skills for https://bugs.launchpad.net/nova/+bug/162125707:29
openstackLaunchpad bug 1621257 in OpenStack Compute (nova) "VNC console keeps reporting "setkeycodes 00" exception" [Undecided,Invalid]07:29
markus_zbauzas: yep, I'm here, let me have a look07:29
openstackgerritMaciej Szankin proposed openstack/nova: Remove redundant check from code  https://review.openstack.org/35012207:29
*** sai has quit IRC07:30
*** jlanoux has joined #openstack-nova07:30
*** rosmaita has quit IRC07:30
*** jaosorior has joined #openstack-nova07:30
*** rosmaita has joined #openstack-nova07:30
bauzasmarkus_z: basically, using the serial console works fine, but when using novnc, there is a keymap issue07:31
bauzasmarkus_z: I try to find whether it's due to novnc or nova07:31
*** lucasagomes has quit IRC07:31
bauzasyesterday, I was thinking it was due to cirros, but the old image is also impacted07:31
*** EinstCrazy has quit IRC07:32
markus_zbauzas: IIRC there was a pending libvirt driver change regarding keymaps.07:32
*** paul-carlton2 has quit IRC07:33
markus_zI have a meeting in 30 min, let's see what I can do until then.07:33
alex_xuKevin_Zheng: re: https://review.openstack.org/#/c/359134/11 looks like we are still missing a lot of fields compare to db schema07:33
*** inara` has quit IRC07:34
*** lucasagomes has joined #openstack-nova07:34
bauzasmarkus_z: but the serial console uses the same libvirt driver for keymaps, right?07:34
*** phuongnh has quit IRC07:34
markus_zyes07:34
*** inara has joined #openstack-nova07:34
*** oanson has joined #openstack-nova07:34
*** phuongnh has joined #openstack-nova07:35
Kevin_Zhengalex_xu: yes, I thought about it07:35
bauzasand the serial console uses the same proxy code for reaching the websockets agent ?07:35
bauzasmarkus_z: ^07:35
bauzasif so, that sounds a novnc issue07:35
Kevin_Zhengalex_xu: looks like some of them are not good to use as sort key07:36
markus_zbauzas: there is a "serialproxy" but it uses the same "baseproxy" as the "novncproxy"07:36
markus_zso many proxies...07:36
Kevin_Zhengsuch as vcpu07:36
bauzasmarkus_z: yeah that's what I meant07:37
*** fragatina has quit IRC07:37
openstackgerritMaciej Szankin proposed openstack/nova: Improve consistency in serial_console  https://review.openstack.org/34890207:37
markus_zbauzas: The latest novnc release was in July, it would fit into the timeline07:37
bauzasyeah07:37
markus_zhttps://github.com/kanaka/noVNC/commit/da82b3426c27bf1a79f671c5825d68ab8c0c5d9f07:37
*** jlanoux has quit IRC07:38
*** jlanoux has joined #openstack-nova07:38
*** lucasagomes has quit IRC07:38
*** sai has joined #openstack-nova07:39
*** lucasagomes has joined #openstack-nova07:39
alex_xuKevin_Zheng: but I think it is part of API contract, I prefer keep it as before. we may remove some of them, but that should be with Microversions07:39
bauzasmarkus_z: either way, it's out of nova scope07:39
bauzasmarkus_z: I can ask the bug reporter to test a former release07:39
*** EinstCrazy has joined #openstack-nova07:40
*** zz_dimtruck has quit IRC07:40
Kevin_Zhengalex_xu: Ok then I will add all fields to the list07:40
*** salv-orl_ has joined #openstack-nova07:40
*** zenoway has joined #openstack-nova07:40
openstackgerritMaciej Szankin proposed openstack/nova: Move notification_format and delete rpc.py  https://review.openstack.org/34960007:41
*** EinstCrazy has quit IRC07:41
markus_zbauzas: I'll check a little more in Nova itself to be sure.07:41
*** leifz has quit IRC07:42
*** zz_dimtruck has joined #openstack-nova07:42
*** salv-orlando has quit IRC07:42
openstackgerritMaciej Szankin proposed openstack/nova: Improve help text for service options.  https://review.openstack.org/33286107:43
*** leifz has joined #openstack-nova07:43
*** zz_dimtruck is now known as dimtruck07:43
openstackgerritMaciej Szankin proposed openstack/nova: Improve help text for ldap_dns_opts  https://review.openstack.org/34198407:44
*** lucasagomes has quit IRC07:44
bauzasmarkus_z: I just did07:44
bauzasmarkus_z: nova just runs a separate service that ships a baseproxy which runs a websockify handler for listening on websockets and connecting to TCP services07:45
*** paul-carlton2 has joined #openstack-nova07:45
bauzasmarkus_z: all that code is shared between the serial console and novnc as far as I can see07:46
openstackgerritMaciej Szankin proposed openstack/nova: conf: Make list->dict conversion more specific  https://review.openstack.org/35933807:46
openstackgerritMaciej Szankin proposed openstack/nova: Reorder scheduler options  https://review.openstack.org/35008507:46
*** lucasagomes has joined #openstack-nova07:47
openstackgerritMaciej Szankin proposed openstack/nova: Reorder scheduler options  https://review.openstack.org/35008507:47
markus_zbauzas: Yes. Trying an earlier version of novnc is the best choice I think.07:47
*** sai has quit IRC07:47
openstackgerritMaciej Szankin proposed openstack/nova: conf: Make list->dict conversion more specific  https://review.openstack.org/35933807:48
*** sai has joined #openstack-nova07:48
*** phuongnh has quit IRC07:48
*** phuongnh has joined #openstack-nova07:49
openstackgerritMaciej Szankin proposed openstack/nova: conf: Add 'deprecated_reason' to osapi opts  https://review.openstack.org/35706107:49
openstackgerritMaciej Szankin proposed openstack/nova: Consider deprecating isolated_images  https://review.openstack.org/27135407:50
*** lyarwood is now known as lyarwood_07:50
*** lyarwood_ is now known as lyarwood07:50
*** hoangcx has joined #openstack-nova07:51
*** RuiChen has quit IRC07:51
openstackgerritMaciej Szankin proposed openstack/nova: conf: Move consoleauth options to a group  https://review.openstack.org/35661107:52
*** lucasagomes has quit IRC07:52
*** lucasagomes has joined #openstack-nova07:52
*** RuiChen has joined #openstack-nova07:52
jsheerenhi all, is it possible to auto assign a floating ip to a newly booted instance? i see in the config reference for nova an option auto_assign_floating_ip, but this does not seem to work with Neutron?07:52
openstackgerritMaciej Szankin proposed openstack/nova: Add related options to floating ip config options  https://review.openstack.org/35556107:53
openstackgerritMaciej Szankin proposed openstack/nova: Improve help text for xenapi_session_opts  https://review.openstack.org/32232207:53
openstackgerritMaciej Szankin proposed openstack/nova: Group scheduler options  https://review.openstack.org/34995907:54
openstackgerritNguyen Phuong An proposed openstack/nova: Prevent use filter() to meet python2,3  https://review.openstack.org/36986207:54
openstackgerritMaciej Szankin proposed openstack/nova: config options: Improve help for SPICE  https://review.openstack.org/31704607:55
*** itlinux has joined #openstack-nova07:55
*** aunnam has quit IRC07:55
*** sai has quit IRC07:55
openstackgerritMikhail Feoktistov proposed openstack/nova: Check hypervisor version while getting bus type  https://review.openstack.org/28239807:55
*** thorst_ has joined #openstack-nova07:56
openstackgerritMaciej Szankin proposed openstack/nova: Improve help text for glance options  https://review.openstack.org/32400707:56
*** lucasagomes has quit IRC07:57
openstackgerritMaciej Szankin proposed openstack/nova: Improve consistency in libvirt  https://review.openstack.org/35367907:57
*** sai has joined #openstack-nova07:58
*** gjayavelu has joined #openstack-nova07:58
openstackgerritMaciej Szankin proposed openstack/nova: Improve help text for upgrade_levels options  https://review.openstack.org/32915407:58
*** thorst_ has quit IRC07:59
*** lucasagomes has joined #openstack-nova07:59
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-nova08:00
*** artom has quit IRC08:00
*** takashin has left #openstack-nova08:00
*** thorst_ has joined #openstack-nova08:01
*** artom has joined #openstack-nova08:01
*** artom has quit IRC08:02
*** artom has joined #openstack-nova08:02
*** ManishD has quit IRC08:02
openstackgerritZhenyu Zheng proposed openstack/nova: Should not be able to sort instances based on joined tables  https://review.openstack.org/35913408:03
*** permalac has joined #openstack-nova08:03
*** CristinaPauna has quit IRC08:03
*** lucasagomes has quit IRC08:03
*** paul-carlton2 has quit IRC08:03
*** danpb has joined #openstack-nova08:05
*** sai has quit IRC08:05
*** itlinux has quit IRC08:05
*** sai has joined #openstack-nova08:07
*** lucasagomes has joined #openstack-nova08:08
alex_xuKevin_Zheng: thanks08:10
*** sai has quit IRC08:11
jsheerennevermind, just read this http://lists.openstack.org/pipermail/openstack-dev/2016-September/103056.html08:11
*** sai has joined #openstack-nova08:12
*** derekh has joined #openstack-nova08:12
*** kevinz has quit IRC08:12
*** code-R_ has quit IRC08:13
*** kevinz has joined #openstack-nova08:13
openstackgerritMaciej Szankin proposed openstack/nova: Remove deprecated manager classes in service.py  https://review.openstack.org/33855308:15
*** thorst_ has quit IRC08:16
openstackgerritMaciej Szankin proposed openstack/nova: Improve consistency in scheduler opts  https://review.openstack.org/34966608:17
openstackgerritMaciej Szankin proposed openstack/nova: Fix opt description for network.py  https://review.openstack.org/34168808:18
*** gjayavelu has quit IRC08:19
*** giri has quit IRC08:19
*** jdurgin has joined #openstack-nova08:20
*** thorst_ has joined #openstack-nova08:21
openstackgerritMaciej Szankin proposed openstack/nova: Fix opt indentation for scheduler.py  https://review.openstack.org/33973608:21
*** yassine has joined #openstack-nova08:22
openstackgerritMaciej Szankin proposed openstack/nova: Config options consistency of ephemeral_storage.py  https://review.openstack.org/34848708:22
*** yassine is now known as Guest7954308:22
openstackgerritMaciej Szankin proposed openstack/nova: conf: Move PCI options to a PCI group  https://review.openstack.org/35660408:23
openstackgerritMaciej Szankin proposed openstack/nova: Fix check_config_option_in_central_place  https://review.openstack.org/35559708:24
*** liverpoo1er has joined #openstack-nova08:25
*** gjayavelu has joined #openstack-nova08:25
*** gjayavelu1 has joined #openstack-nova08:25
openstackgerritMaciej Szankin proposed openstack/nova: Improve help text for interval_opts  https://review.openstack.org/33176108:25
*** liverpooler has quit IRC08:26
openstackgerritMaciej Szankin proposed openstack/nova: Deprecate all topic conf options  https://review.openstack.org/35160808:27
*** lucasagomes has quit IRC08:27
*** sai has quit IRC08:27
openstackgerritMaciej Szankin proposed openstack/nova: conf: Remove invalid floating_ips TODO  https://review.openstack.org/35657208:28
*** matrohon has joined #openstack-nova08:28
openstackgerritMaciej Szankin proposed openstack/nova: Check Config Options Consistency  for xenserver.py  https://review.openstack.org/35028408:28
*** gjayavelu1 has quit IRC08:29
*** gjayavelu has quit IRC08:29
*** paul-carlton2 has joined #openstack-nova08:29
openstackgerritMaciej Szankin proposed openstack/nova: Add deprecated_since parameter  https://review.openstack.org/35363108:30
*** sai has joined #openstack-nova08:30
*** priteau has joined #openstack-nova08:30
openstackgerritMaciej Szankin proposed openstack/nova: Improving help text for xenapi_vmops_opts  https://review.openstack.org/34708508:31
*** gjayavelu has joined #openstack-nova08:31
*** yamahata has quit IRC08:31
*** gjayavelu1 has joined #openstack-nova08:31
openstackgerritPrateek Arora proposed openstack/nova: Delete traces of in-progress snapshot on VM being deleted  https://review.openstack.org/29451308:31
*** oomichi has quit IRC08:31
*** lucasagomes has joined #openstack-nova08:32
*** oomichi has joined #openstack-nova08:32
openstackgerritMaciej Szankin proposed openstack/nova: Fixed deprecation status for xvp options  https://review.openstack.org/35365808:32
*** gjayavelu1 has quit IRC08:33
*** gjayavelu has quit IRC08:33
*** andreykurilin has left #openstack-nova08:35
*** andreykurilin has joined #openstack-nova08:36
*** garyk has quit IRC08:38
*** thorst_ has quit IRC08:38
*** thorst_ has joined #openstack-nova08:39
*** sai has quit IRC08:41
namnhjohnthetubaguy, Hello Mr.Garbutt> I am who discussed rolling-upgrade with you last time. I already thought your command again. And you are quite right.08:42
johnthetubaguynamnh: hello, hello, I totally have better docs on the upgrade process on my team's TODO list08:42
*** RuiChen has quit IRC08:43
*** RuiChen has joined #openstack-nova08:43
*** thorst_ has quit IRC08:44
*** thorst_ has joined #openstack-nova08:46
*** lucasagomes has quit IRC08:46
*** lucasagomes has joined #openstack-nova08:46
namnhjohnthetubaguy, :)) would you mind sharing something about the docs with me. I am really appreciate that. :)))08:48
*** thorst_ has quit IRC08:48
johnthetubaguynamnh: they are currently in my head, the plan is to get those ones you were reading updated with something a bit more real08:48
johnthetubaguynamnh: just checking did you see this blog post: http://superuser.openstack.org/articles/upgrading-nova-to-kilo-with-minimal-downtime/08:49
johnthetubaguynamnh: its not totally up to date, we have made things easier since then, but gives you some good background08:49
*** NightKhaos has quit IRC08:52
*** gjayavelu has joined #openstack-nova08:53
*** NightKhaos has joined #openstack-nova08:53
*** thorst_ has joined #openstack-nova08:54
namnhjohnthetubaguy, Hoping that I can your archive/blog about that soon. Yes, I did. One more thing, did you see this archive: http://www.danplanet.com/blog/2015/10/07/upgrades-in-nova-database-migrations/08:56
namnhjohnthetubaguy, that's why I asked you about --expand, --contract options. :)08:57
johnthetubaguynamnh: yeah, I helped review some of that before dan posted it08:57
johnthetubaguybasically we do everything that happens before upgrade in "db sync" (and "api_db syc"), and everything after you are upgraded in "db online_data_migrations"08:58
*** thorst_ has quit IRC08:59
*** hoangcx has quit IRC08:59
*** hoangcx has joined #openstack-nova09:02
*** FL1SK has joined #openstack-nova09:02
*** prateek has quit IRC09:03
namnhjohnthetubaguy, Great :). Oh. I have encountered a bug: https://bugs.launchpad.net/devstack/+bug/1619554 after I finished nova. My log is similar the log of the bug.09:04
openstackLaunchpad bug 1621257 in OpenStack Compute (nova) "duplicate for #1619554 VNC console keeps reporting "setkeycodes 00" exception" [Undecided,Invalid]09:04
*** lucasagomes has quit IRC09:05
openstackgerritjaveme proposed openstack/nova: Missing to update task state for confirm_resize  https://review.openstack.org/24443309:05
*** itlinux has joined #openstack-nova09:06
*** lucasagomes has joined #openstack-nova09:08
*** claudiub has quit IRC09:08
*** sai has joined #openstack-nova09:09
*** thorst_ has joined #openstack-nova09:09
*** coolsvap_ is now known as coolsvap09:09
*** dimtruck is now known as zz_dimtruck09:11
*** thorst__ has joined #openstack-nova09:12
*** thorst_ has quit IRC09:13
*** sai has quit IRC09:13
*** lucasagomes has quit IRC09:14
*** itlinux has quit IRC09:16
*** lucasagomes has joined #openstack-nova09:17
openstackgerritAlberto Planas proposed openstack/nova: Add migrate_uri for invoking the migration  https://review.openstack.org/35655809:19
*** mvk has quit IRC09:19
*** thorst__ has quit IRC09:20
*** efoley has joined #openstack-nova09:21
*** lucasagomes has quit IRC09:21
*** prateek has joined #openstack-nova09:21
*** efoley_ has joined #openstack-nova09:22
*** lucasagomes_ has joined #openstack-nova09:23
*** lucasagomes_ has quit IRC09:23
*** jsheeren has quit IRC09:24
*** gjayavelu has quit IRC09:25
csatariCan anyone help in novaclient functional testing?09:25
*** efoley has quit IRC09:25
*** achinm has joined #openstack-nova09:25
*** cdent has joined #openstack-nova09:26
*** jsheeren has joined #openstack-nova09:26
*** hoangcx has quit IRC09:27
*** code-R has joined #openstack-nova09:27
*** sdake_ has quit IRC09:28
*** thorst_ has joined #openstack-nova09:28
*** code-R_ has joined #openstack-nova09:29
*** hoangcx has joined #openstack-nova09:30
*** hoangcx has quit IRC09:31
*** lucasagomes has joined #openstack-nova09:32
*** code-R has quit IRC09:32
*** efoley_ is now known as efoley09:34
*** thorst_ has quit IRC09:35
*** sai has joined #openstack-nova09:37
*** permalac has quit IRC09:38
*** hoangcx has joined #openstack-nova09:38
*** mingyu_ has joined #openstack-nova09:41
*** hoangcx has quit IRC09:45
*** mingyu has quit IRC09:45
*** sai has quit IRC09:46
*** daemontool has joined #openstack-nova09:46
*** sai has joined #openstack-nova09:47
sfinucanmelwitt: damn it, yeah that's ignored since 1.6.0. So much for random internet articles. I'll revert it.09:50
openstackgerritStephen Finucane proposed openstack/nova: Revert "tox: Don't create '.pyc' files"  https://review.openstack.org/36998609:50
*** JoseMello has quit IRC09:51
sfinucanmelwitt: and done ^09:51
*** mvk has joined #openstack-nova09:52
*** hoangcx has joined #openstack-nova09:53
*** sai has quit IRC09:54
*** jaosorior is now known as jaosorior_lunch09:54
*** takedakn has joined #openstack-nova09:55
*** takedakn has quit IRC09:55
*** rfolco has joined #openstack-nova09:56
*** lpetrut has joined #openstack-nova09:57
*** sai has joined #openstack-nova09:57
*** alexchadin has joined #openstack-nova09:59
alexchadinhi09:59
alexchadincan someone help me with booting instance?09:59
sfinucanalexchadin: #openstack might be a better place for that question10:01
alexchadinsfinucan: ok, thank you!10:02
sfinucanalexchadin: np :)10:02
*** thorst_ has joined #openstack-nova10:02
*** efoley has quit IRC10:04
*** hoangcx has quit IRC10:04
*** takedakn has joined #openstack-nova10:04
*** jdurgin has quit IRC10:05
*** acabot has quit IRC10:06
*** acabot has joined #openstack-nova10:07
*** thorst_ has quit IRC10:11
*** efoley has joined #openstack-nova10:12
*** zenoway has quit IRC10:12
*** thorst_ has joined #openstack-nova10:12
*** itlinux has joined #openstack-nova10:17
*** liverpooler has joined #openstack-nova10:17
*** liverpoo1er has quit IRC10:19
openstackgerritNguyen Phuong An proposed openstack/nova: Prevent use filter(lambda ...) to meet python2,3  https://review.openstack.org/36986210:19
*** thorst_ has quit IRC10:20
*** jdurgin has joined #openstack-nova10:20
*** mingyu_ has quit IRC10:20
*** prateek has quit IRC10:22
*** priteau has quit IRC10:23
*** gcb has quit IRC10:25
*** jaosorior_lunch is now known as jaosorior10:27
*** itlinux has quit IRC10:28
*** takedakn has quit IRC10:28
*** takedakn has joined #openstack-nova10:29
*** EinstCrazy has joined #openstack-nova10:30
*** jdurgin has quit IRC10:32
*** alexchadin has quit IRC10:33
*** prateek has joined #openstack-nova10:33
*** alexchadin has joined #openstack-nova10:34
*** gszasz has joined #openstack-nova10:36
*** alexchadin has quit IRC10:36
*** alexchadin has joined #openstack-nova10:36
*** gszasz has quit IRC10:38
*** thorst_ has joined #openstack-nova10:39
openstackgerritRikimaru Honjo proposed openstack/nova: Add checks for preventing unexpected detaching volume  https://review.openstack.org/35461710:41
*** garyk has joined #openstack-nova10:42
*** tbachman has quit IRC10:45
*** EinstCrazy has quit IRC10:48
openstackgerritjaveme proposed openstack/nova: Missing to update task state for confirm_resize  https://review.openstack.org/24443310:48
*** takedakn has quit IRC10:48
*** thorst_ has quit IRC10:51
*** gszasz has joined #openstack-nova10:52
*** phuongnh has quit IRC10:54
*** CristinaPauna has joined #openstack-nova10:54
*** itlinux has joined #openstack-nova10:58
*** thorst_ has joined #openstack-nova10:59
*** cdelatte has joined #openstack-nova11:00
*** zz_dimtruck is now known as dimtruck11:01
*** mfedosin has joined #openstack-nova11:02
*** ekuris has joined #openstack-nova11:03
*** sai has quit IRC11:08
*** sdague has joined #openstack-nova11:08
*** itlinux has quit IRC11:09
johnthetubaguysahid: is it possible to quickly add a release note to this patch? https://review.openstack.org/#/c/367909/311:09
mdboothCan anybody tell me why we don't setup_network_on_host(dest) *before* starting a live migration?11:09
*** sai has joined #openstack-nova11:10
johnthetubaguymdbooth: you mean for neutron or nova-network?11:11
*** thorst_ has quit IRC11:11
mdboothjohnthetubaguy: If there's a difference between the semantics of the 2, that may be the answer :)11:11
*** dimtruck is now known as zz_dimtruck11:11
mdboothI'm just talking about the api call11:11
mdboothIn fact, we already do exactly this in pre_live_migration()11:13
mdboothI wonder why we do it again11:13
*** sahid has quit IRC11:13
* mdbooth asks git history11:13
*** ihrachys has joined #openstack-nova11:13
*** adisky has joined #openstack-nova11:14
johnthetubaguymdbooth: most of it is about setting up the iptables or ovs rules on the destination, so the network traffic flows there as soon as the instance goes active11:14
johnthetubaguymdbooth: the neutron case has only just had this added for DVR, where we tell neutron where we are heading towards11:14
mdboothjohnthetubaguy: That's what I'd have assumed, so it sounds like something you'd want to do early as it's time consuming11:15
*** rtheis has joined #openstack-nova11:15
*** alexchadin has quit IRC11:15
johnthetubaguymdbooth: honestly, that method really needs splitting into three as I think it does prep, setup and teardown, depending on when its called11:15
mdboothThere's also migrate_instance_start() and migration_instance_finish()11:15
mdboothWe currently do:11:15
mdboothmigration_instance_start(), setup_networks_on_host(), migrate_instance_finish()11:15
mdboothmigrate_instance_start() is a no-op in neutron11:16
mdboothsetup_networks_on_host seems to do what you just described11:16
mdboothand migrate_instance_finish() seems to swap the ports over11:16
*** thorst_ has joined #openstack-nova11:16
johnthetubaguyyeah, the finish is more about "we are done"11:16
mdboothSo intuitively I'd say we should do the first 2 *before* migration, and only _finish() afterwards.11:17
johnthetubaguyI think we call setup_networks_on_host on the source after we are finsihed too, but I could be miss-remembering11:17
mdboothHowever, intuition tends to get me in trouble in Nova.11:17
johnthetubaguymdbooth: +111:17
johnthetubaguyall I remember is, its a bit freeky11:17
mdboothWe do call it both before and after, just looking to see if git can tell me why.11:17
*** diga has quit IRC11:18
*** bvanhav has joined #openstack-nova11:19
johnthetubaguyso the after is the teardown bit, removing the ip tables rules11:20
johnthetubaguyI believe11:20
johnthetubaguyor its adding in floating ips if its on the destination11:21
*** thorst_ has quit IRC11:21
lyarwooddansmith: Morning, I've updated https://review.openstack.org/#/c/290793/ (duplicate bdm bug) with a few notes if you have a chance to take a look once you're online. Still work to do but hopefully straight forward now.11:22
mdboothjohnthetubaguy: The answer is in a comment :) Who thought to put useful information in a comment?!?!?11:25
mdbooth        # NOTE(tr3buchet): setup networks on destination host11:25
mdbooth        #                  this is called a second time because11:25
mdbooth        #                  multi_host does not create the bridge in11:25
mdbooth        #                  plug_vifs11:25
mdboothI wonder if that is still tru11:25
mdboothe11:25
* mdbooth wonders what multi_host is11:26
sdaguemdbooth: it's a specific nova-network mode, but the one that most run in11:27
mdboothOther than 'OpenStack', that is11:27
mdboothsdague: I guessed it meant something specific. I'll dig there for anything relating to the above.11:28
sdaguewhere ever node dumps straight onto the physical interface for network traffice, instead of going through a network node11:28
*** namnh has quit IRC11:29
mdboothNova networks not quite dead yet, is it?11:29
sdagueyeh, not quiet yet11:31
*** tuanla_ has quit IRC11:32
*** sai has quit IRC11:32
*** vladikr has joined #openstack-nova11:32
*** sai has joined #openstack-nova11:33
*** openstackgerrit has quit IRC11:34
mdboothtr3buchet: Do you happen to have an insight into whether the above comment is still true?11:34
*** openstackgerrit has joined #openstack-nova11:35
mdboothdanpb: Actually, looking at the libvirt code, does ^^^ intersect os-vif?11:35
danpbmdbooth: what in particular ?11:36
mdboothI posted a comment just above from tr3buchet11:36
mdboothI'm wondering if the thing it describes is covered by os-vif11:36
mdboothAnd even better, if you happen to know anything about it personally11:37
*** gjayavelu has joined #openstack-nova11:37
*** efoley_ has joined #openstack-nova11:39
*** efoley has quit IRC11:39
danpbmdbooth: whre does that comment even come from ?11:41
mdboothdanpb: post_live_migration_at_destination11:41
*** yassine has joined #openstack-nova11:41
*** gjayavelu has quit IRC11:41
mdboothIt's explaining why we call setup_networks_on_host(dest) twice11:41
*** yassine is now known as Guest855011:42
*** sai has quit IRC11:43
danpbmdbooth: the explanation doesn't make alot of sense to me11:43
danpbmdbooth: it is saying that this method we call doesn't create the bridge11:43
danpbmdbooth: but somehow this method does create the bridge if you call it a second time, which makes no sense at all11:43
mdboothsdague: If I delete it, do we have test coverage of live migration using multi_host?11:43
*** sai has joined #openstack-nova11:44
mdboothdanpb: That's what I thought, but I don't know the networking code well enough to second guess.11:44
*** claudiub has joined #openstack-nova11:44
*** Guest79543 has quit IRC11:45
mdboothdanpb: I think that's the slowest operation during network switch-over, btw11:45
mdboothIf it's not required, that would be good11:45
danpbdamned if i know - as the comment makes zero sense11:46
mdboothdanpb: Ok, thanks11:46
*** claudiub has quit IRC11:46
mdboothhttps://bugs.launchpad.net/nova/+bug/93906011:47
openstackLaunchpad bug 939060 in OpenStack Compute (nova) "live migrations do not update dnsmasq entries or setup networking on destination node when using multi_host" [Critical,Fix released] - Assigned to Vish Ishaya (vishvananda)11:47
*** dave-mccowan has joined #openstack-nova11:47
*** woodster_ has joined #openstack-nova11:49
*** claudiub has joined #openstack-nova11:50
*** lucasagomes is now known as lucas-hungry11:50
*** ekuris has quit IRC11:52
*** edmondsw has joined #openstack-nova11:55
*** markus_zz has joined #openstack-nova11:55
*** markus_z has quit IRC11:55
*** takedakn has joined #openstack-nova11:56
*** artom has quit IRC11:56
*** artom has joined #openstack-nova11:57
*** lbeliveau has quit IRC11:58
*** achinm has quit IRC12:00
*** lbeliveau has joined #openstack-nova12:00
*** ratailor has quit IRC12:01
*** maticue_ has joined #openstack-nova12:03
*** efoley__ has joined #openstack-nova12:03
openstackgerritStephen Finucane proposed openstack/nova: Update references for split up hardware module  https://review.openstack.org/36896812:04
*** efoley_ has quit IRC12:05
*** Kevin_Zheng has quit IRC12:06
*** itlinux has joined #openstack-nova12:09
*** gouthamr has joined #openstack-nova12:09
*** thorst_ has joined #openstack-nova12:10
*** sahid has joined #openstack-nova12:10
*** sai has quit IRC12:10
openstackgerritStephen Finucane proposed openstack/nova: conf: Make list->dict conversion more specific  https://review.openstack.org/35933812:10
openstackgerritStephen Finucane proposed openstack/nova: conf: Add 'deprecated_reason' to osapi opts  https://review.openstack.org/35706112:10
*** sai has joined #openstack-nova12:11
*** oanson has quit IRC12:11
*** ducttape_ has joined #openstack-nova12:12
*** sudipto has joined #openstack-nova12:13
*** sudipto_ has joined #openstack-nova12:13
*** alexchadin has joined #openstack-nova12:13
*** ducttape_ has quit IRC12:13
*** sudipto has quit IRC12:13
*** sudipto_ has quit IRC12:13
*** baoli has joined #openstack-nova12:14
*** sudipto has joined #openstack-nova12:14
*** sudipto has quit IRC12:14
*** ducttape_ has joined #openstack-nova12:14
*** sudipto has joined #openstack-nova12:14
*** baoli has quit IRC12:14
*** sudipto_ has joined #openstack-nova12:15
*** baoli has joined #openstack-nova12:15
openstackgerritStephen Finucane proposed openstack/nova: Doc fix for Nova API Guide, added missing word.  https://review.openstack.org/36906112:15
openstackgerritStephen Finucane proposed openstack/nova: Doc fix for Nova API Guide, added missing word.  https://review.openstack.org/36906112:15
*** lamt has quit IRC12:17
*** baoli_ has joined #openstack-nova12:18
*** maticue has joined #openstack-nova12:18
openstackgerritMatthew Booth proposed openstack/nova: Don't call setup_networks_on_host(dest) multiple times  https://review.openstack.org/37008912:19
*** itlinux has quit IRC12:20
*** tbachman has joined #openstack-nova12:21
*** baoli has quit IRC12:21
*** annegentle has joined #openstack-nova12:21
*** jsheeren has quit IRC12:22
openstackgerritsahid proposed openstack/nova: libvirt: add ps2mouse in choice for pointer_model  https://review.openstack.org/36790912:23
*** alexchadin has quit IRC12:23
*** alexchadin has joined #openstack-nova12:23
openstackgerritMatthew Booth proposed openstack/nova: Initiate all network changes from source during live migration  https://review.openstack.org/36942312:26
*** roeyc has joined #openstack-nova12:27
*** mriedem has joined #openstack-nova12:27
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: add ps2mouse in choice for pointer_model  https://review.openstack.org/36790912:31
openstackgerritRadoslav Gerganov proposed openstack/nova: Fix API doc for os-console-auth-tokens  https://review.openstack.org/34096912:31
*** efoley__ has quit IRC12:32
*** annegentle has quit IRC12:34
mriedemfor any stable cores out there, we have quite a few stable/mitaka backports that could use some review https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/mitaka12:36
*** xyang1 has joined #openstack-nova12:37
*** sudipto has quit IRC12:39
*** tbachman has quit IRC12:39
sdaguemriedem: I think I have a new theory on your unicode bug12:39
*** sudipto has joined #openstack-nova12:39
sdaguehttps://bugs.launchpad.net/nova/+bug/158072812:39
openstackLaunchpad bug 1580728 in OpenStack Compute (nova) "UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 386: ordinal not in range(128) in nova.virt.libvirt.vif:unplug with unicode instance.display_name" [Medium,Triaged]12:39
*** sudipto_ has quit IRC12:39
*** sudipto_ has joined #openstack-nova12:40
mriedemsdague: ah weird, that would make sense because i could not recreate the failure12:40
sdagueyeh12:41
mriedemand kept getting that formatted Instance str12:41
sdagueplus the base images run a weird C.utf8 locale, because they aren't built by distros12:41
*** ociuhandu has quit IRC12:42
*** ducttape_ has quit IRC12:43
*** lpetrut has quit IRC12:43
*** gabor_antal has quit IRC12:43
*** lpetrut has joined #openstack-nova12:43
*** ftersin has joined #openstack-nova12:43
ftersinmdbooth: hi, are you around?12:45
mdboothftersin: Yup. Wassup?12:45
ftersinabout fix https://review.openstack.org/#/c/35541512:45
*** sai has quit IRC12:46
mdboothftersin: Yes.12:46
mdboothDoes it cause a regression?12:46
ftersinwhy not self.get_disk_size(self.path)?12:47
ftersinnot yet12:47
ftersinbut i'm wondering how this is work with resizing12:47
mdboothIt's to do with resize of an image12:48
ftersinparticularly with the second and the next resizings12:48
*** lamt has joined #openstack-nova12:48
mdboothAlthough, yeah, given that we haven't resized by that point we could have just used path instead.12:48
ftersinwhen the base image is 1Gb, and flavor gives 2Gb, this code resizes initial disk to 2Gb12:48
johnthetubaguymriedem: I was thinking this should have a reno note, or should we just push that in and put the note in a separate patch? https://review.openstack.org/#/c/367909/312:49
mdboothYes12:49
mdboothftersin: That's intentional though, right?12:49
ftersinbut when i resize the instance to 10 Gb flavor, how the disk will be resized?12:49
mriedemjohnthetubaguy: it has a reno now12:49
mriedemjohnthetubaguy: i'm +212:49
* johnthetubaguy thinking about getting an eye test12:49
*** zz_dimtruck is now known as dimtruck12:50
alex_xunova api subteam meeting is in 10 mins at #openstack-meeting-412:50
*** tbachman has joined #openstack-nova12:50
johnthetubaguymriedem: thanks, my spelling is terrible otherwise I would have pushed a reno12:50
mriedemheh np12:50
sdaguemriedem: yeh, it's totally locale environment12:51
mdboothftersin: Let me just check finish_migration again12:51
mdboothftersin: Although... I think this *might* have caused a regression with Rbd :(12:51
paul-carlton2pkoniszewski, re post copy implementation, is the switch to post copy mode only manually initiated via force-complete or will it occur automatically when qemu thinks it is time?12:51
sdagueso... you get some really odd garbage in the display in dumping instances if it's not a utf8 locale, though it doesn't always break12:51
mdboothBecause Rbd overrides get_disk_size() and ignores its argument :/12:51
mdboothAnd therefore would have previously worked correctly12:52
sdaguethough, interestingly enough, placement api didn't seem to start when I make this a utf8 domain12:52
ftersinmdbooth: the idea of this code is to resize a disk, when its size is smaller than requested one12:53
*** jheroux has joined #openstack-nova12:53
*** sai has joined #openstack-nova12:53
mdboothftersin: Resize still works here, because size is passed in12:53
ftersinbut cheking base image, this code can work with some crutches only12:54
mdboothftersin: This code is a disaster, nobody's going to argue anything else12:54
mdboothThis was just a bandaid12:54
pkoniszewskipaul-carlton2: so there isnt any mechanism in QEMU to trigger post-copy automatically12:54
*** MVenesio has joined #openstack-nova12:54
mdboothftersin: I don't think it broke resize in the general case, because size is passed in correctly from the flavor12:54
*** MVenesio has quit IRC12:55
ftersinresize_image is never called for resize operation. is it correct?12:55
*** kylek3h has joined #openstack-nova12:55
alex_xumriedem: sdague, maybe this patch is worth for rc1 https://review.openstack.org/35913412:55
mdboothftersin: see _create_and_inject_local_root(), which is called from _create_image(), which is called from finish_migration()12:55
mdboothftersin: size comes from the flavor12:55
paul-carlton2pkoniszewski, but nova automatically trigger it when we decide it is time?12:55
pkoniszewskipaul-carlton2: yes12:55
sdaguealex_xu: that patch feels kind of high risk right now12:56
mdboothftersin: So we'll be saying: if (size from flavor)  > (size of backing file): resize_image()12:56
mdboothwhich is correct12:56
sdaguealex_xu: I think it should probably wait until ocata opens up12:56
alex_xusdague: ah, yeah, have risk break the api contract12:56
paul-carlton2pkoniszewski, ta12:56
pkoniszewskipaul-carlton2: so nova might change pre-copy to post-copy, currently switch occurs when percentage increase is less than 10%12:56
mdbooth*except* for Rbd, which has its own disgusting hack, which I forgot about :(12:56
sdaguealex_xu: or just other fallout12:56
sdaguegiven that it's doing things with db query filtering12:57
alex_xusdague: yea, agree12:57
sdaguedon't get me wrong, I think it's a good fix12:57
ftersinmdbooth: hmm... need to think again...12:57
sdagueI just think it might have fallout, and don't want to hurt the release12:57
*** lucas-hungry is now known as lucasagomes12:57
mdboothmriedem: I think https://review.openstack.org/#/c/355415/ has a critical bug12:58
alex_xusdague: yea, I see you12:58
mdboothBlocker12:58
* mdbooth will propose a revert and write a new patch12:58
mdboothActually, wait...12:58
mdboothMaybe not12:58
* mdbooth reads the Rbd backend again12:58
mdboothmriedem: Nope, it's a bug, because the prepare function passed in from driver has Rbd-specific gubbins which will ignore the path it was given if it's cloning from a ceph glance store13:00
*** Kevin_Zheng has joined #openstack-nova13:00
alex_xunova api meeting is running13:01
*** jaosorior has quit IRC13:01
mdboothThe bug is that when creating an Rbd instance using Rbd glance, the instance disk won't be resized to the flavor size.13:01
mdboothSorry about that.13:01
*** jaosorior has joined #openstack-nova13:01
mdboothftersin's solution was better: just check path13:02
*** pradk has quit IRC13:03
*** efoley__ has joined #openstack-nova13:04
*** rmart04_ has joined #openstack-nova13:04
*** rmart04 has quit IRC13:04
*** rmart04_ is now known as rmart0413:04
*** sai has quit IRC13:05
ftersinmdbooth: yeah, you're right - this code will resize disks (except rbd), but also will try to do it when resizing is not required13:05
mdboothftersin: That last part isn't a regression, though.13:05
mdboothftersin: I'm just writing up a bug for the Rbd thing, though, and I'll propose a revert and a new patch13:06
mdboothRC1 is Friday :/13:06
bauzasmoshele: lbeliveau: hola, is that a regression ? https://bugs.launchpad.net/nova/+bug/162308913:06
openstackLaunchpad bug 1623089 in OpenStack Compute (nova) "Fail to suspend guest with direct-physical ports" [Undecided,New]13:06
bauzasmdbooth: s/Friday/Thur EOB13:06
lbeliveaubauzas: no it's been there forever, or at least not covered when introducing direct-physical13:07
*** sai has joined #openstack-nova13:07
bauzaslbeliveau: okay, thanks for that13:09
mriedemmdbooth: hmm, dansmith asked about rbd paths in that one13:09
mriedemmdbooth: rc1 is really tomorrow eod13:09
mriedemPST13:09
mriedemmdbooth: that's pretty annoying given all of the people saying it was fixing their issues and how it was just right and didn't need tests or an explanation13:10
*** rmart04 has quit IRC13:10
mdboothmriedem: I forgot about this: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L3064-L307013:11
mdboothmriedem: Point taken.13:11
* alex_xu won't be online too much next two days, it is holiday in china13:11
mriedemjust surprising since i thought thouse ops that were pushing it were all rbd users13:11
mriedemoh well, shit happens13:12
mdboothPerhaps they didn't notice?13:12
*** efoley__ has quit IRC13:12
mdboothI'll get a new patch out today.13:12
mdboothAnd a revert.13:12
mriedemjust tag the bug with newton-rc-potential and we can throw it into the etherpad13:12
mriedemmdbooth: probably want to get those guys to test the new change if possible13:12
*** takedakn has quit IRC13:12
alex_xudansmith: morning, when you wakup, hope you can take a look at https://review.openstack.org/369147, if you happy with rename the method, then I will update the patch.13:12
*** mdrabe has joined #openstack-nova13:14
*** tbachman_ has joined #openstack-nova13:14
*** efoley__ has joined #openstack-nova13:14
*** tbachman has quit IRC13:15
*** tbachman_ is now known as tbachman13:15
mdboothmriedem: That code really is total spaghetti, though :/ The way it's structured you can't touch it unless you retain all these gross hacks in your head simultaneously.13:15
* mdbooth remembered 2, but forgot 2 more.13:16
mriedemyeah that's why i was asking for an explanation in the commit message13:16
mriedemand tests etc13:16
mdboothThat was only going to be picked up by functional or CI. The call path length is enormous.13:16
bauzasmriedem: FWIW, we still have 31 open bugs that haven't been triaged yet, trying to reduce that number today13:17
mriedemwe do have a ceph ci job,13:17
mriedembauzas: i've gone through some of those, no red flags as of last night13:17
mdboothYup. I don't think tempest checks disk sizes though, does it? diana_clarke hit that one trying to write a test for a resize bug.13:18
mriedemi've got to get my kid on the bus for school so will be back on later13:18
mriedemmdbooth: nope13:18
mriedemwe need to talk about in-tree functional/integration testing for ocata at the summit i think13:18
mriedemfor stuff like this13:18
*** lamt has quit IRC13:18
bauzasmriedem: as well, I did a few of those this (EU) morning13:19
*** itlinux has joined #openstack-nova13:20
*** rmart04 has joined #openstack-nova13:21
*** rmart04 has quit IRC13:21
moshelebauzas: no, I would say missing pieces of PF  pass-through implementation (it never worked before)13:21
ftersinmdbooth: i've spent a day about to implement ephemeral resizing... there is a bit more work than 'update bdms and write tests'13:21
mdboothftersin: Cool. Got patches?13:22
ftersinmdbooth: heh, not yet13:22
ftersinmdbooth: at the first - we need to support revert resize operation13:22
mdboothftersin: We're just using copies for that, right?13:23
ftersinmdbooth: i.e. we have to save old ephemeral sizes somewhere to restore bdms13:23
ftersinmdbooth: copies of what? of bdms?13:23
mdboothCopies of disks13:23
*** hamzy has quit IRC13:24
ftersinmdbooth: disks stored on files are copied, root disk on rbd is copied as well13:24
*** jistr|mtg is now known as jistr13:24
mdboothRight. Don't they all do that?13:24
ftersinmdbooth: this is another task - to snapshot (and restore) ephemerals on ceph13:25
*** mriedem has quit IRC13:25
*** sudipto_ has quit IRC13:25
*** sudipto has quit IRC13:25
ftersinmdbooth: but now i'm stopped on _create_image method. i updated bdms, but resize does not work13:25
*** sudipto has joined #openstack-nova13:26
ftersinmdbooth: because _create_image calls imagebackend.cache with new ephemeral base image13:26
*** sudipto_ has joined #openstack-nova13:26
mdboothftersin: So, getting rid of ephemeral base images is also on my todo list.13:27
mdboothThey're a bug.13:27
*** ducttape_ has joined #openstack-nova13:27
ftersinmdbooth: _create_image gets new bdms and knows nothing about resizing and original ephemeral sizes13:27
mdboothAh.. hence you're wondering why not (path)?13:28
mdboothYes, that check doesn't make sense for swap or ephemerals, which is partly why I was ok with the change I'm about to revert13:28
ftersinmdbooth: it was my first think. moreover i thought to fix ephemeral bug on flat backend with this (path)13:29
ftersinmdbooth: but currently i understand that it is not correct - imagebackend must handle real base image, i.e. _create_image must pass original ephemeral base image13:31
*** adisky has quit IRC13:31
ftersinmdbooth: but you're right - it was the reason why i went to look on that fix13:32
mdboothftersin: Of course... cache_name is passed in13:32
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: libvirt: Acquire TCP ports for console during live migration  https://review.openstack.org/27580113:32
*** markus_zz is now known as markus_z13:32
mdboothAnd it's dependent on size13:32
*** burgerk has joined #openstack-nova13:32
mdboothEurgh, that's nasty13:32
openstackgerritMatthew Treinish proposed openstack/nova: Use tempest tox with regex first  https://review.openstack.org/36856613:33
mdboothftersin: In all seriousness, why don't you work on removing ephemeral and swap base images first?13:33
*** tblakes has joined #openstack-nova13:33
*** itlinux has quit IRC13:34
mdboothThe goal of that refactor series, btw, is to persist storage metadata.13:34
mdboothI haven't got that far yet, but that's where it's going.13:35
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: Add functional regression test for bug 1595962  https://review.openstack.org/33841613:35
openstackbug 1615613 in OpenStack Compute (nova) "duplicate for #1595962 Live migration always fails when VNC/SPICE is listening at non-local, non-catch-all address" [High,Fix released] https://launchpad.net/bugs/1615613 - Assigned to John Garbutt (johngarbutt)13:35
mdboothWith persistent storage metadata, you'd have somewhere to stash the info you need to stash.13:35
dansmithalex_xu: +113:35
mdboothWithout it, I'm not sure where it would go. It's currently inferred from host config, image metadata and bdms.13:35
mdboothI think it would be easier, and better to get rid of it.13:35
lyarwood.13:37
*** hemanthm|away is now known as hemanthm13:40
*** artom has quit IRC13:40
*** salv-orlando has joined #openstack-nova13:40
*** cleong has joined #openstack-nova13:41
*** artom has joined #openstack-nova13:41
*** ducttape_ has quit IRC13:41
*** artom has quit IRC13:42
*** artom has joined #openstack-nova13:42
*** markvoelker has joined #openstack-nova13:43
ftersinmdbooth: what do you mean with 'removing base images'?13:43
*** salv-orl_ has quit IRC13:43
mdboothftersin: Ephemeral and swap disks shouldn't use the image cache.13:43
*** tongli has joined #openstack-nova13:43
mdboothThey should create the target disk, and run the template function to initialize it13:44
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: WIP: live-migration test hook for serial console  https://review.openstack.org/34747113:44
mdboothi.e. They should be flat in all cases.13:44
ftersinmdbooth: got it. but... recently you mention huge time requred to format an empty ephemeral...13:44
*** mgoddard_ has joined #openstack-nova13:45
mdboothftersin: That's not for all formats. e.g. ext4 is very quick. ext3 isn't.13:45
mdboothftersin: However, we also don't create them for Lvm.13:45
*** marst has quit IRC13:45
mdboothAnd also, flat copies the whole thing13:46
mdboothAnd Rbd copies it over the wire, which is very inefficient13:46
ftersinmdbooth: yes. probably it's a good idea. for the future13:46
*** burgerk has quit IRC13:46
ftersinmdbooth: though it will require support of existing instances, i.e. we cannot just drop base ephemerals13:47
mdboothftersin: And it's a data corruptor bug during both live and cold migration.13:47
mdboothftersin: Yup, that would be the biggest issue to address. We either continue to support them having base images, or flatten them when we find them.13:48
mdboothLatter would probably be simplest, tbh, as it only affects Qcow213:48
*** mgoddard has quit IRC13:49
*** jistr is now known as jistr|mtg13:49
*** mtanino has joined #openstack-nova13:49
openstackgerritMatthew Booth proposed openstack/nova: Revert "Fix resizing in imagebackend.cache()"  https://review.openstack.org/37013513:49
*** lamt has joined #openstack-nova13:52
*** mgoddard_ has quit IRC13:52
*** alexchadin has left #openstack-nova13:52
*** mgoddard has joined #openstack-nova13:52
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: WIP: live-migration test hook for serial console  https://review.openstack.org/34747113:52
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: libvirt: Acquire TCP ports for console during live migration  https://review.openstack.org/27580113:52
*** mmedvede has joined #openstack-nova13:53
ftersinmdbooth: as the result we have: a) horrible code; b) some ideas to refactor it, wich require a lot of time (one's already ate 1.5 version cycles); c) disability to improve current state of code (including new features) due to its horribleness; d) customers still wait for the features and bug fixes. All this is sad13:55
*** penick has quit IRC13:56
mdboothftersin: This is all true13:57
openstackgerritAlex Xu proposed openstack/nova: Clean up allocation when update available resources  https://review.openstack.org/36914713:57
alex_xudansmith: ^ done13:57
*** sdake has joined #openstack-nova13:58
*** marst has joined #openstack-nova13:59
*** anush__ has joined #openstack-nova14:00
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: Add functional regression test for bug 1595962  https://review.openstack.org/33841614:00
openstackbug 1615613 in OpenStack Compute (nova) "duplicate for #1595962 Live migration always fails when VNC/SPICE is listening at non-local, non-catch-all address" [High,Fix released] https://launchpad.net/bugs/1615613 - Assigned to John Garbutt (johngarbutt)14:00
*** akshai has joined #openstack-nova14:01
*** takedakn has joined #openstack-nova14:01
*** hamzy has joined #openstack-nova14:01
*** Swami has joined #openstack-nova14:03
*** Swami has quit IRC14:03
*** Swami has joined #openstack-nova14:03
*** pradk has joined #openstack-nova14:05
*** pradk- has joined #openstack-nova14:05
*** pradk- has quit IRC14:05
*** rodrigods has quit IRC14:06
*** rodrigods has joined #openstack-nova14:06
*** sajeesh has joined #openstack-nova14:06
*** lamt has quit IRC14:06
*** eharney has joined #openstack-nova14:07
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: WIP: live-migration test hook for serial console  https://review.openstack.org/34747114:07
*** ayoung has quit IRC14:08
openstackgerritAlex Xu proposed openstack/nova: Clean up allocation when update available resources  https://review.openstack.org/36914714:08
*** hshiina has joined #openstack-nova14:09
*** claudiub has quit IRC14:09
*** READ10 has joined #openstack-nova14:11
*** ducttape_ has joined #openstack-nova14:11
*** efoley__ has quit IRC14:11
*** mriedem has joined #openstack-nova14:11
*** mriedem has quit IRC14:11
*** rmart04 has joined #openstack-nova14:12
*** Daviey_ is now known as Daviey14:12
*** matrohon has quit IRC14:14
*** artom has quit IRC14:14
*** artom has joined #openstack-nova14:16
*** artom has quit IRC14:16
*** artom has joined #openstack-nova14:17
*** code-R_ has quit IRC14:17
*** code-R has joined #openstack-nova14:17
*** rlrossit has joined #openstack-nova14:18
*** rmart04_ has joined #openstack-nova14:20
dansmithsdague: a bunch of things just failed a bunch of tests.. should I recheck or is some known thing going on?14:20
dansmithheh14:20
*** moshele has quit IRC14:20
dansmithnevermind then14:20
openstackgerritChris Dent proposed openstack/nova: [placement] prevent a KeyError in webob.dec.wsgify  https://review.openstack.org/37015414:20
openstackgerritSergey Nikitin proposed openstack/nova-specs: Enable to share PCI devices between numa nodes  https://review.openstack.org/36114014:21
*** rmart04 has quit IRC14:21
*** rmart04_ is now known as rmart0414:21
sdagueyeh, something is odd with the cloud8 osic region14:21
dansmithokay14:21
sdaguethe fails are all over the place there, but may be related to things being slow due to a non optimal ceph setup14:21
sdagueinfra is doing a temporary disable on that region until it can be debugged14:21
dansmithsdague: okay I got two fails, only one on ceph14:22
*** tri2sing has joined #openstack-nova14:23
dansmithgrenade fail on the old side it looks like14:23
*** tonytan4ever has joined #openstack-nova14:24
dansmithhttp://logs.openstack.org/33/365633/9/gate/gate-grenade-dsvm-neutron-ubuntu-trusty/92cb41c/logs/grenade.sh.txt.gz#_2016-09-14_12_44_45_68114:24
*** mriedem has joined #openstack-nova14:25
mriedemi'm back!14:25
*** anush__ has quit IRC14:26
mriedemdanpb: i thought you might enjoy this bug https://bugs.launchpad.net/nova/+bug/162107614:26
openstackLaunchpad bug 1621076 in OpenStack Compute (nova) "Can't detach interface from VM (if VM has two interface with same mac addresses)" [Undecided,New] - Assigned to leehom (feli5)14:26
melwitto/14:26
mriedemdanpb: you can attach multiple interfaces with the same mac to a libvirt guest but can't detach them because libvirt can't find the unique device to detach i guess14:26
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: WIP: live-migration test hook for serial console  https://review.openstack.org/34747114:28
danpbmriedem: this is still nova's fault14:29
*** yamahata has joined #openstack-nova14:29
danpbmriedem: the API contract for virDomainDetachDevice says you must pass the full XML for the device shown in the current guest XML14:29
danpbmriedem: nova doesn't do this - it re-creates the minimal <interface> XML from scratch14:30
*** raunak has joined #openstack-nova14:30
danpbmriedem: as a result it is missing the PCI device ID libvirt would normally use to uniquely identify the device14:30
*** mingyu has joined #openstack-nova14:30
danpbmriedem: so libvirt has no option but to try and find it using mac address14:30
mriedemok, do we have any existing code to build that interface xml from the vif?14:30
mriedemseems like a simple fix14:30
mriedemit's a latent bug, so not an rc issue, just wanted your ack on it14:31
danpbyeah, its been broken forever14:31
*** pbandark has quit IRC14:31
*** jaosorior has quit IRC14:31
*** psachin has joined #openstack-nova14:33
*** pwnall has joined #openstack-nova14:33
*** pwnall is now known as Guest2761714:33
*** fragatina has joined #openstack-nova14:34
*** itlinux has joined #openstack-nova14:35
*** Swami_ has joined #openstack-nova14:35
*** nherciu has quit IRC14:36
sdaguedansmith: not our ceph14:36
*** matrohon has joined #openstack-nova14:36
sdaguethat cloud is on ceph14:36
dansmithsdague: ah, heh14:36
dansmithgotcha14:36
*** rfolco has quit IRC14:37
*** brian_price has joined #openstack-nova14:37
*** nkrinner is now known as nkrinner_afk14:37
*** fragatina has quit IRC14:38
*** Swami has quit IRC14:38
*** Swami_ has quit IRC14:39
*** jamesden_ has joined #openstack-nova14:41
*** pbandark has joined #openstack-nova14:41
*** mlavalle has joined #openstack-nova14:42
mriedemjwcroppe: did you (PRS) have a scheduler filter that took network bandwidth into account and if a host was flooded would not schedule to it?14:42
mriedemi remember you trying to get some network info into the scheduler filter info awhile back14:43
*** raunak has quit IRC14:43
*** jamesde__ has joined #openstack-nova14:43
*** claudiub has joined #openstack-nova14:43
*** claudiub has quit IRC14:43
*** claudiub has joined #openstack-nova14:44
bauzasmriedem: I remember some change about proposing to integrate a new monitor for giving network bandwidth information, maybe from IBM but I need to remember that one14:44
*** amotoki has joined #openstack-nova14:45
bauzasmriedem: nevermind, it was for memory bandwidth14:45
mriedemyeah that was sudipto's14:45
bauzasyeah the pcp stuff14:45
*** jamesden_ has quit IRC14:45
sudiptoyeah14:45
*** GordonTX has joined #openstack-nova14:46
*** anush__ has joined #openstack-nova14:46
sudiptoNot for network bandwidth though.14:46
sudiptoIt was for memory bandwidth.14:46
alaskidansmith: bauzas melwitt mriedem thoughts on skipping the cells meeting today? Since nothing there affects RC I don't want to take up peoples time.14:47
mriedemalaski: sounds fine to me14:47
dansmithalaski: +114:47
bauzasalaski: nope, we merged the cellsv2 related bugfix, so I'm all clear14:47
melwitt+114:47
mriedemalaski: my only thing was to start thinking about a summit session14:47
bauzasunless some bug pops up14:47
* alaski slams gavel14:47
alaskiit's done14:47
alaskimriedem: the wheels are already turning14:48
openstackgerritMatthew Booth proposed openstack/nova: Fix resizing in imagebackend.cache()  https://review.openstack.org/37018014:48
*** rmart04_ has joined #openstack-nova14:48
*** Swami has joined #openstack-nova14:48
*** rmart04 has quit IRC14:49
*** rmart04_ is now known as rmart0414:49
*** tblakes has quit IRC14:49
*** burgerk has joined #openstack-nova14:49
*** erlon has joined #openstack-nova14:50
*** mingyu has quit IRC14:53
*** mingyu has joined #openstack-nova14:54
*** roeyc has quit IRC14:54
*** rrecio has joined #openstack-nova14:55
*** rrecio_ has joined #openstack-nova14:56
*** rfolco has joined #openstack-nova14:58
*** tbachman has quit IRC14:58
*** jamesde__ has quit IRC14:58
mdboothftersin: Could you please review https://review.openstack.org/#/c/370180/ ?14:58
mdboothftersin: This is an rc-potential with imminent deadline :) I think you're best placed to spot any new bugs in there.14:59
*** akshai has quit IRC14:59
*** rrecio has quit IRC14:59
mdboothftersin: Also, I've argued in the commit message against adding a test for the Rbd case because it's too hard. This is going to make mriedem understandably sad, so if you can think of a good way to test that I'm sure he'd be very happy.15:00
*** mjura has quit IRC15:01
*** akshai has joined #openstack-nova15:01
ftersinmdbooth: i've started. btw, why do you use resolves-bug tag? closes-bug is obsolete?15:01
*** jistr|mtg is now known as jistr15:01
mdboothftersin: Ah, it's because our internal gerrit uses Resolves.15:02
mdboothI'll edit it.15:02
*** BjoernT has joined #openstack-nova15:03
ftersindo not worry, i just wondered15:03
openstackgerritMatthew Booth proposed openstack/nova: Fix resizing in imagebackend.cache()  https://review.openstack.org/37018015:03
*** lykinsbd has joined #openstack-nova15:04
*** pbandark has quit IRC15:05
*** david-lyle_ has joined #openstack-nova15:06
*** gjayavelu has joined #openstack-nova15:06
*** david-lyle_ has quit IRC15:06
bauzasmdbooth: re: https://bugs.launchpad.net/nova/+bug/1623497 are you able to reproduce some regression ?15:06
openstackLaunchpad bug 1623497 in OpenStack Compute (nova) "Booting Ceph instance using Ceph glance doesn't resize root disk to flavor size" [Undecided,New]15:06
*** tjones has joined #openstack-nova15:06
kashyapmdbooth: The commit summary message is same as the one you cite  (I46b5658e) - is that intentional?15:07
ftersinmdbooth: as for rbd test - as i remember diana_clakre was going to write a test, which verifies that disk is resized via ssh, iianm15:07
ftersinmdbooth: does this test exist?15:07
mdboothkashyap: Yep. It's still a fix for the same bug.15:08
*** jlanoux has quit IRC15:08
kashyapMaybe mark it part-2, please, if you re-spin.  Sorry for the OCD15:08
mdboothftersin: I don't think this has to involve ssh, as it's not exclusive to resize.15:08
bauzasmdbooth: mriedem: tagging some bugreport as RC blocker needs us to make sure there *is* a regression, so I wonder if we should possibly try to find some way to reproduce that like in https://review.openstack.org/#/c/369645/ ?15:08
bauzasd/?/15:08
mdboothbauzas: I'm moderately confident, but I take your point. Does anybody have a ceph devstack to hand? This is trivial to reproduce.15:09
bauzasmdbooth: we have a ceph job15:09
mriedemthe ceph job only runs tempest15:09
bauzasmdbooth: so we could write some DNM patch for canary testing it15:10
mdboothYeah, but it doesn't test disks sizes.15:10
bauzasoh f*15:10
mdboothDoesn't, and won't.15:10
mriedemwhatever happened to jbernard?15:10
bauzasgraaaah15:10
*** hemnafk is now known as hemna15:10
*** rmart04 has quit IRC15:11
melwittI think he was recently replicated15:11
bauzasmriedem: sec, what does the ceph job ? running a couple of tempest scenarios against a tuned devstack that uses ceph, right?15:12
mdboothbauzas: Yes.15:12
mriedembauzas: it runs full tempest but with ceph for storage15:12
mdboothThe bug in question won't cause anything to fail, though.15:12
mriedemcinder and glance are backed by ceph15:12
mdboothIt'll just cause the root disk to be too small.15:12
*** armax has joined #openstack-nova15:12
mriedemand the nova imagebackend is rbd15:13
bauzassec, sec, finding15:13
diana_clarkeftersin: I abandoned my attempt to write that tempest test b/c it wasn't going to merge. We'll need a different framework for tests like those, I suspect.15:13
bauzasthe real problem is that we aren't testing that path, right?15:13
mdboothbauzas: What diana_clarke said ^^^. That's the 'won't' part.15:13
dansmithmriedem: is this the thing you pointed danpb at recently? http://logs.openstack.org/33/365633/9/check/gate-nova-python27-db-ubuntu-xenial/5a6acb4/console.html#_2016-09-14_14_41_07_86914515:13
mriedemdansmith: i don't remember that15:14
dansmithokay15:14
*** lamt has joined #openstack-nova15:15
*** moshele has joined #openstack-nova15:16
openstackgerritKashyap Chamarthy proposed openstack/nova: libvirt: fix detach volume race condition  https://review.openstack.org/29051015:17
*** gjayavelu has quit IRC15:17
*** dnalezyty has joined #openstack-nova15:19
diana_clarkeHere's that first attempt at a tempest test, perhaps it'll be a useful jumping off point: https://review.openstack.org/#/c/338411/15:19
diana_clarke(I'm only half paying attention, so perhaps not)15:19
openstackgerritChris Dent proposed openstack/nova: [placement] Full request and response representation gabbit  https://review.openstack.org/37020415:19
*** dnalezyty has quit IRC15:19
cdentdansmith, bauzas, mriedem that's ^ a start at something at least some of your were asking about. I'm not sure it is a total success, but gets it going.15:20
bauzascdent: cool, if that doesn't mind you, will defer that to post-RC115:20
cdentbauzas: oh yeah, definitely15:20
bauzasbut I totally appreciate the change15:21
cdentjust wanted to be sure people were aware it was there as a referece15:21
cdentreference15:21
bauzaskewl15:21
bauzasdiana_clarke: yeah, mtreinish's answer totally makes sense, I just wonder if we could test that in the nova tree15:23
diana_clarkebauzas: I don't think so. IIUC that important stuff is stubbed out currently.15:24
bauzasin the functional tests tree ?15:24
diana_clarkes/that/the15:24
diana_clarkebauzas: yes, but I could be wrong. I asked but I don't think I ever got an answer15:24
diana_clarke(and then I moved on)15:25
bauzasokay15:25
*** lykinsbd has quit IRC15:25
*** rmcall has joined #openstack-nova15:25
*** tblakes has joined #openstack-nova15:25
*** Kevin_Zheng has quit IRC15:26
mtreinishbauzas: I made sense? That can't be right :)15:26
*** lykinsbd has joined #openstack-nova15:26
bauzasmtreinish: heh, was talking of https://review.openstack.org/#/c/33841115:26
mriedembauzas: diana_clarke: the functional tests in nova use the fake virt driver15:26
mriedemand no real services external to nova, like cinder or glance or neutron15:26
diana_clarkeyup, that's what I figured15:27
mriedemfunctional tests only have the api and db15:27
bauzasmriedem: that, I know15:27
mriedemthere has been talk of making an in-tree integration test bucket that's backed by devstack15:27
mriedemso we can hit actual things15:27
bauzasmriedem: I just need to figure the best approach, and if we could mock out some specific stuff15:27
bauzasmriedem: yeah I remember the functional devstack talk15:27
*** zenoway has joined #openstack-nova15:27
mdboothEurgh15:28
mdboothdevstack's ceph stuff has moved15:28
*** edand has quit IRC15:28
openstackgerritChris Dent proposed openstack/nova: [placement] Full request and response representation gabbit  https://review.openstack.org/37020415:28
*** diana_clarke is now known as diana-meeting15:28
mriedemmdbooth: it's in devstack-plugin-ceph15:29
mriedemhttps://github.com/openstack/devstack-plugin-ceph15:29
*** claudiub has quit IRC15:29
mdboothYeah, I've still got old config here15:29
* mdbooth hates new things15:29
*** sridharg has quit IRC15:29
*** psachin has quit IRC15:30
mriedemmdbooth: http://logs.openstack.org/15/355415/7/check/gate-tempest-dsvm-full-devstack-plugin-ceph/5a2985a/logs/reproduce.sh15:33
mriedemi haven't tried the reproduce.sh script yet15:33
mriedembut, if those work it'd be sweet15:33
mdboothI think I've got it. It's a one-liner in local.conf15:33
mdboothenable_plugin devstack-plugin-ceph git://git.openstack.org/openstack/devstack-plugin-ceph15:33
mdboothIf that works, I can't complain :)15:34
mriedemi don't think that's everything15:34
mriedemwell maybe15:34
*** rcernin has quit IRC15:34
*** ducttape_ has quit IRC15:35
melwittif you're only running one replica, that should work15:35
*** ducttape_ has joined #openstack-nova15:35
*** jlanoux has joined #openstack-nova15:36
*** jlanoux has quit IRC15:36
*** moshele has quit IRC15:37
*** jlanoux has joined #openstack-nova15:38
cfriesenbauzas: live migration resource tracking question for you...in _post_live_migration() we do a cast to post_live_migration_at_destination(), then do some stuff and then set the migration status to 'completed'.   How do we know that the destination has run the post_live_migration_at_destination() code yet?15:39
cfriesenIf it hasn't and the dest runs the resource audit code, don't we run the risk of "losing" the instance resources?15:40
*** pcaruana has quit IRC15:40
bauzascfriesen: I'm sorry, I'm not expert of that area but I can read code :)15:41
sdaguecdent / dansmith there is a real unit test fail in here now - http://logs.openstack.org/33/365633/9/check/gate-nova-python27-db-ubuntu-xenial/5a6acb4/console.html15:41
bauzascfriesen: from a pure resource standpoint, we only account on resource usage every periodic call to update_avail_res() since we haven't merged the live-mig claim change15:41
mdboothmriedem: Seems it's just that one line15:42
mdrabejaypipes: I'm taking a look at https://github.com/openstack/nova/blob/master/nova/pci/stats.py#L209-L211, which is _filter_pools_for_spec in the pci stats. This doesn't appear to be correct to me because it's calling utils.pci_device_prop_match with the pool and request specs, but that method expects a pci device and request specs15:42
dansmithsdague: it's not real, AFAICT.. I don't see it locally15:42
dansmithsdague: and doesn't look related15:42
sdaguedansmith: well, maybe a merge issue?15:43
*** rlrossit has quit IRC15:43
*** code-R has quit IRC15:43
cfriesenbauzas: ah, I forgot that the claim-over-migration wasn't merged yet.15:43
dansmithsdague: I just pulled and ran it and I don't see it locally (a few minutes ago)15:43
mriedemmdbooth: good to know, i'm testing out reproduce.sh since i've been meaning to try that anyway15:43
dansmithsdague: I guess maybe I need to rebase on master and try, just a sec15:44
sdaguedansmith: yeh, I don't know. It odd enough though that we really shouldn't have races there. If some other bad code snuck in we should try to get that sorted15:44
*** slaweq_ has joined #openstack-nova15:44
dansmithsdague: yeah, which is why I went to run it locally and rechecked to see if it happens again, but it's far far away from the content of any of those patches15:45
sdagueyeh15:45
* cdent will try too15:45
cfriesensahid: are you still planning on pushing https://review.openstack.org/#/c/244489 and friends in N?15:45
dansmithsdague: cdent: re-running rebased on master now15:45
*** daemontool_ has joined #openstack-nova15:45
dansmithlocally, I mean15:45
*** BjoernT has quit IRC15:45
dansmithsdague: this is what I asked mriedem about above, because I thought he had said something suck in related to libvirt and os-vif, which is this near15:46
sdaguedansmith: ok, I was otp, so I guess I missed the backscroll15:46
sdagueI thought the os-vif thing he was looking at was the unicode issue15:47
sdaguewhich I think is an artifact of our test env15:47
mdrabeWould someone be able to help me out with a concern I have related to pci stats?15:48
*** devananda|dinner is now known as devananda15:48
dansmithsdague: maybe I dunno, I just recalled a similar "vif-y" relation15:48
*** daemontool has quit IRC15:48
sdagueyeh, it was showing up during vif-unplug15:49
sdaguebut it was message processing and printing out the instance15:49
sdaguein a debug message15:49
dansmithokay, well, whatever, not related apparently ;)15:50
*** mriedem has quit IRC15:51
*** mriedem1 has joined #openstack-nova15:51
*** code-R has joined #openstack-nova15:52
*** BjoernT has joined #openstack-nova15:52
sdaguecdent: on the back burner plate for you - http://logs.openstack.org/90/369490/1/check/gate-nova-tox-db-functional-ubuntu-xenial/085ac3e/console.html#_2016-09-13_14_37_49_882469 - gabbi fixture setups don't seem to be doing the right thing with log / output capture so they are printing inline from 8 workers simultaneously and making the output stream a bit messy15:53
cdentsdague: yeah, I've seen that but don't know what the stories is on how it is supposed to work?15:54
sahidcfriesen: yes it something needed, just feel demotivated to keep it up to date so far15:54
cdentIs stdout and stderr getting eaten somewhere in the other tests?15:54
openstackgerritDave Johnston proposed openstack/nova-specs: Implements: blueprint multi-tenant-exclusion-filter  https://review.openstack.org/37024315:54
cdentand then replayed in some fashion?15:54
*** ihrachys has quit IRC15:54
*** prateek has quit IRC15:55
cdentsdague: those fixtures never did anything with any logging because there never was any until recently :)15:55
dansmithsdague: cdent passes locally still15:55
*** burt has quit IRC15:55
* cdent is still waiting15:55
cdentI did a limited run and it was fine, now doing everything15:56
sdaguedansmith: ok, I guess just recheck and hope for the best15:56
cdentsdague: anyway, I'll create a bug so it doesn't get lost15:56
*** Sukhdev has joined #openstack-nova15:56
dansmithsdague: dun did that already15:56
ftersinmdbooth: some comments looks obsolete, iianm. see inside15:57
*** rama_y has joined #openstack-nova15:57
*** ayoung has joined #openstack-nova15:57
*** andreas_s has quit IRC15:58
mdboothftersin: Ah, yes.15:58
mdboothThe code is still good, but the comment are indeed out of date.15:58
sdaguecdent: https://github.com/openstack/nova/blob/04e94df9112a03727f040fd55e2cb2cdc8840d60/nova/test.py#L181-L191 is where all that is sorted15:59
*** kfarr has joined #openstack-nova16:00
cdentsdague: yeah, just found it16:00
*** mgoddard_ has joined #openstack-nova16:01
*** zul has joined #openstack-nova16:01
ftersinmdbooth: should we ask diana to restore her review with the test to let us run it against master and patched code? i tried to run ci, but it's not started probably because the review is abandoned16:02
*** marst has quit IRC16:02
*** tbachman has joined #openstack-nova16:03
mdboothdiana-meeting: ^^^ ?16:03
*** marst has joined #openstack-nova16:03
*** browne has joined #openstack-nova16:03
*** mfedosin has quit IRC16:04
*** mgoddard has quit IRC16:04
*** ayoung has quit IRC16:04
*** efoley has joined #openstack-nova16:04
*** Dinesh_Bhor has quit IRC16:07
mdboothmriedem1 ftersin: Hah, it works anyway because Rbd implements this resize itself :)16:07
mdboothSo it was previously doing the resize twice16:07
mdboothNow it's only doing it once16:07
mdboothWhich is nice16:07
*** mriedem1 is now known as mriedem16:08
mriedemmdbooth:16:08
mdboothSo, meh16:08
mriedem?16:08
mriedemso, the bug isn't an rc blocker16:08
mdboothIt's up to you whether to take the new patch or not16:08
mriedemit's an optimization16:08
markus_zsdague: macsz: I created https://github.com/sdague/rst-burndown/pull/1 for the config opts burndown. Maybe I can work tomorrow on that to make it more generic. I'm on vacation for 2 weeks starting on Friday.16:08
mdboothmriedem: I guess you could spin it that way :)16:09
mriedemwell when you say now it's only doing it once, you mean with your patch?16:09
mdboothNo, I mean the bug I reported is real16:09
mdboothAnd it wasn't doing the resize there16:09
mdboothHowever, it was doing an additional resize anyway16:10
mdboothSo the 'bug' makes it more efficient16:10
ftersinmdbooth: rbd backend implements resizing in create_image, but it may be not called16:10
*** GordonTX has quit IRC16:10
*** krtaylor has quit IRC16:10
mdboothftersin: When would it not be called?16:11
mriedemmdbooth: and your change would introduce a redundant explicit resize?16:11
mdboothmriedem: My change would re-introduce the redundant explicit resize16:11
diana-meetingftersin: sorry about that (was in a meeting)16:11
*** diana-meeting is now known as dinaa16:11
mriedemok16:11
dinaaftersin: I've restored the abandoned patch16:11
*** dinaa has quit IRC16:12
mdboothmriedem: TBH, I'm slightly happier with the new one than the old, but give that I view both versions as hacks I'm not bothered either way.16:12
*** diana_clarke has joined #openstack-nova16:12
*** moshele has joined #openstack-nova16:12
mdboothIf it's not actually on fire, lets leave it alone16:12
mriedemyeah i agree, i'd like to have nic take a look at it16:13
*** lykinsbd has quit IRC16:13
mriedemi'm assuming you'll want to update the commit message?16:13
mdboothWhich one?16:13
mriedemyour new patch16:13
mriedemmdbooth: can you update the bug and remove the rc-potential tag?16:13
openstackgerritSTEW TY proposed openstack/nova: Complete verification for os-floating-ips  https://review.openstack.org/32970416:13
mdboothmriedem: Sure16:13
mriedemthanks16:13
melwittdo we need to merge the revert before rc1?16:14
*** slaweq_ has quit IRC16:14
mriedemno16:14
mdboothmelwitt: no16:14
melwittokay, thanks. was a bit confused16:14
ftersinmdbooth: ahh, it's called always since base image never exists16:15
ftersindinaa: thx16:15
dansmithsdague: recheck passed those tests16:17
*** mfedosin has joined #openstack-nova16:17
openstackgerritSTEW TY proposed openstack/nova: Body Verification of api-ref os-volume-attachments.inc  https://review.openstack.org/33633416:17
*** markus_z has quit IRC16:20
mdboothmriedem: Compare and contrast: https://review.openstack.org/#/c/317694/20/nova/virt/libvirt/imagebackend.py create_from_image()16:21
*** Apoorva has joined #openstack-nova16:21
*** code-R has quit IRC16:21
mdboothThere are no other calls: that's everything that it does.16:22
*** Guest27617 has quit IRC16:22
mdboothAll in 1 place16:22
*** nic has joined #openstack-nova16:22
*** lucasagomes has quit IRC16:22
mdboothThe cloning behaviour has complete test coverage: https://review.openstack.org/#/c/317694/20/nova/tests/unit/virt/libvirt/test_imagebackend.py16:23
*** lucasagomes has joined #openstack-nova16:23
openstackgerritDave Johnston proposed openstack/nova-specs: Implements: blueprint multi-tenant-exclusion-filter  https://review.openstack.org/37024316:24
*** raunak has joined #openstack-nova16:25
*** amotoki has quit IRC16:26
mriedemwhy is that abandoned?16:27
mriedemthat series is going to continue into ocata16:27
*** abalutoiu has quit IRC16:29
*** cdent has quit IRC16:32
*** jlanoux has quit IRC16:32
*** matrohon has quit IRC16:32
*** marst has quit IRC16:33
*** ssurana has joined #openstack-nova16:34
*** tbachman has quit IRC16:34
*** rcernin has joined #openstack-nova16:34
dansmithsdague: but it failed on that osic timeout again :(16:34
diana_clarkemriedem: I abandoned my initial contributions because I'm no longer working on this.16:35
diana_clarkemriedem: I figured any parts that are still relevant when there's review bandwidth for them can be pulled in fresh (because code rots).16:35
diana_clarkemriedem: There were things I had hoped to discuss during reviews, but in the months since I wrote that code, I've swapped most of that context out.16:35
*** rlrossit has joined #openstack-nova16:35
*** tesseract- has quit IRC16:35
diana_clarkemriedem: Old code like this makes me nervous. Anyhoo, that's why I abandoned it, but I can restore them if you like.16:35
mriedemdiana_clarke: ok...i thought this was all part of a series so it doesn't rot if it's actively rebased and worked on16:35
*** tbachman has joined #openstack-nova16:36
diana_clarkemriedem: Except I no longer recall the bits I thought might be suspect. That context has since been lost.16:36
mriedemalright16:37
mriedemi guess i'd leave TODOs or something in code i wasn't sure about while working on it16:37
mriedemidk16:37
mriedemthen someone else can pick it up16:37
mriedemcdent does that quite often16:37
mriedemanyway, i suspect mdbooth will pick it up in ocata16:38
diana_clarkeI have pretty extensive notes, but I had hoped for more of a back and forth with a subject matter expert.16:38
*** garyk has quit IRC16:38
*** kylek3h is now known as kylek3h_away16:38
mriedemok i'm honestly not sure who that is, mikal maybe, or danpb, or now just mdbooth16:39
diana_clarkeTo see this refactor through to completion, I suspect I nova core will need to be assigned to work closely with mdbooth.16:40
mriedemi go to nic for rbd imagebackend questions16:40
diana_clarkeThat is, it's pretty crazy code. The reviewers need to be knee deep in it as well.16:40
*** tbachman_ has joined #openstack-nova16:41
*** Matias has quit IRC16:41
mriedemyeah i don't disagree there16:41
diana_clarkeAnyhoo, everyone is really busy. I don't mean that in a bad way. Just that it's a really hard area of the code to change and review.16:41
diana_clarketypo: "I suspect I nova core" -> "I suspect *a* nova core"16:42
*** tbachman has quit IRC16:42
*** tbachman_ is now known as tbachman16:42
dansmithmriedem: do you know what this osic ceph issue is that's causing fails?16:43
mriedemlink?16:43
dansmithmriedem: the one sdague was talking about16:44
dansmithmriedem: he said an infra-related osic backend ceph misconfiguration or something that was causing spurious fails16:44
*** yamahata has quit IRC16:44
*** Swami has quit IRC16:44
mriedemuh16:45
mriedemlink?16:45
*** ircuser-1 has quit IRC16:45
*** takedakn1 has joined #openstack-nova16:46
*** takedakn has quit IRC16:47
*** takedakn1 is now known as takedakn16:47
*** lpetrut has quit IRC16:47
dansmithmriedem: I'd have to link to eavesdrop, but jsut a sec16:47
*** mvk has quit IRC16:47
dansmithmriedem: trying to figure out if this is that failure: http://logs.openstack.org/33/365633/9/check/gate-tempest-dsvm-cells-ubuntu-xenial/71ae727/console.html#_2016-09-14_16_31_32_34312816:47
mriedemthat's cells v116:48
mriedemyou said ceph above16:48
dansmithmriedem: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2016-09-14.log.html#t2016-09-14T14:21:1116:48
dansmithmriedem: he said ceph16:48
diana_clarkemriedem: I've restore my abandoned patches. cc: mdbooth16:49
diana_clarkemdbooth: please change the author if you end up using those patches but change them significantly (which I suspect you'll need to). Thanks!16:49
mriedemdansmith: oh b/c osic is ceph backed,16:49
mriedemwhich might make the image snapshot time out?16:49
*** diogogmt has joined #openstack-nova16:49
dansmithdiana_clarke: you can't change the author in gerrit16:49
dansmithdiana_clarke: so if you want that, abandon and let him re-propose them16:49
dansmithmriedem: yes because it uses ceph on the backend, but not sure if that's the related bit here or not16:49
*** jaypipes has joined #openstack-nova16:50
mriedemi think that's this http://status.openstack.org//elastic-recheck/index.html#162076116:50
dansmithmriedem: I don't see a ton of issues in the glance logs16:50
dansmithokay16:51
diana_clarkedansmith: Oh, I thought we had this discussion and it was just the gerrit owner (vs author) that couldn't be changed.16:51
*** ircuser-1 has joined #openstack-nova16:51
melwittdansmith, diana_clarke: I think that's the case. owner can't be changed but author and committer can16:51
* diana_clarke shrugs, I'll leave to mdbooth to play with when the time comes16:51
*** gszasz has quit IRC16:51
dansmithmelwitt: I thought only committer can change? I know that the gerrit owner can't change, but thought that informs the author, but maybe not16:52
*** burt has joined #openstack-nova16:52
dansmitheither way, not sure why it matters :)16:52
melwittdansmith: author can be changed but it's a manual thing you have to do in git before you upload again16:52
mriedemthat logstash hit is only 4 in 10 days16:52
mriedemall on rax-ord which i'm not sure if that's osic or not16:52
dansmithmelwitt: okay I thought gerrit refuses upload if you --reset-author?16:52
dansmithmaybe I'm thinking of something else16:53
diana_clarkedansmith: I dislike having my name on code I didn't write. And I can usually tell I didn't write it my looking at it :)16:53
sdaguemriedem: osic-cloud8 was the ceph backed cloud with issues16:53
diana_clarkea/my/by16:53
melwittdansmith: I might have old info but I've seen someone change the author line before and it worked. it's just owner can never change16:53
dansmithmelwitt: okay16:53
*** derekh has quit IRC16:53
*** mdrabe has quit IRC16:53
dansmithsdague: what was the failure symptom of that?16:53
sdaguebut there wasn't any particular signature, mostly things being quite slow16:53
dansmithokay16:54
sdagueand some jobs fully timing out16:54
dansmithsdague: yeah, i first read this as a job timeout because I wasn't paying enough attention16:54
sdagueand it has to do with image format on top of ceph IIRC that was making performance tank16:54
dansmithsdague: the bottom placement test seems to have poor karma and just hits everything16:54
dansmiths/test/patch/16:54
*** lamt has quit IRC16:54
sdagueanyway, they are working on it, and pulled the cloud of out rotation until post RC1 to reduce variability for patch landing16:55
dansmithoh okay16:55
*** Sukhdev has quit IRC16:56
*** kfarr has quit IRC16:56
*** kfarr has joined #openstack-nova16:57
*** krtaylor has joined #openstack-nova16:57
*** mdrabe has joined #openstack-nova16:59
*** jed56 has joined #openstack-nova16:59
*** hshiina has quit IRC16:59
*** karthiks has quit IRC16:59
dansmithmelwitt: my last experience with that might've been wrapped up with any of these: https://gerrit-review.googlesource.com/Documentation/error-invalid-author.html17:01
*** pwnall has joined #openstack-nova17:01
*** tblakes has quit IRC17:01
*** pwnall is now known as Guest3254617:01
melwittdansmith: I was just about to link that. I was searching around and found whether or not it will work is based on the gerrit access config17:02
pkholkinmriedem: hi Matt, are you here?17:02
dansmithyeah17:02
mriedempkholkin: i'm about to run out for lunch17:03
pkholkinmriedem: ok,  no problem, had a question17:03
melwittI just thought I've seen someone do it before. I might have dreamed it17:03
pkholkindansmith: hi Dan)17:04
mriedempkholkin: feel free to ask17:04
*** fragatina has joined #openstack-nova17:04
pkholkinmriedem: dansmith: today I played with metadata on aggregates and images/flavors17:04
pkholkinI understand that if on aggregate we do 'os_distro=xxx,yyy,zzz'17:04
pkholkinwe will think that we have a set defaultdict(<type 'set'>, {u'os_distro': set([u'xxx', u'yyy', u'zzz'])})17:05
mriedemthere is a certain frenchman that would probably love talking about this17:05
pkholkinand I was confused about it, and have some other questions17:05
*** _gryf has quit IRC17:06
dansmithpkholkin: I'm not the right person to ask about that.. maybe jaypipes ?17:06
pkholkinI am not sure)17:07
dansmithI'd have to go read code to act like I know :)17:07
pkholkinI can ask him17:07
dansmithpkholkin: my understanding is that jaypipes knows everything, but he's only on the ground for short layovers between flights to important meetings, so...17:07
pkholkindansmith: :)17:07
dansmith:P17:08
pkholkinjaypipes: hello Jay, are you here)?17:08
*** brian_price has quit IRC17:08
*** hemanthm is now known as hemanthm|lunch17:08
*** Guest32546 has quit IRC17:08
jaypipesdansmith: hi. just reviewed alex_xu's patch that you had pushed a revision for yesterday. please see comments.17:08
jaypipespkholkin: hey Pavel, what's up?17:08
pkholkindansmith: mriedem: I was asking because I want to implement this spec https://review.openstack.org/#/c/85399/1717:09
dansmithjaypipes: okay, still trying to ramrod the last few of the other set through the gate :/17:09
pkholkinbut also for flavors17:09
pkholkinjaypipes: I have a question17:09
jaypipespkholkin: yes, that is correct. you will get set(...) of those values.17:09
jaypipesdansmith: need reviews from me on things?17:10
pkholkinjaypipes: what is the use case for this17:10
*** fragatina has quit IRC17:10
*** _gryf has joined #openstack-nova17:10
pkholkinI want to implement the strict host isolation, but I always think that metadata is key=value17:10
*** Swami has joined #openstack-nova17:10
*** fragatina has joined #openstack-nova17:11
pkholkinwhy not just Strings17:11
jaypipespkholkin: there is no "use case" for this :) it's just a giant pile of hacky poo to indicate that the aggregate has computes that can support operating system distros of XXX, YYY, or ZZZ17:11
*** danpb has quit IRC17:12
dansmithjaypipes: no, they're all locked and loaded, I just have to convince jenkins to let us through17:12
*** mgoddard_ has quit IRC17:13
pkholkinjaypipes: for flavors and images we can only set one key=value17:13
*** mgoddard has joined #openstack-nova17:13
pkholkinI thought that for aggregates it is the same17:13
pkholkinjaypipes: do we have some documentation about this 'feature') ?17:14
pkholkinalso e.g. if we set 'os_distro=xxx,yyy,zzz' for image/flavor your filter will not pass17:15
*** takedakn has quit IRC17:16
*** lamt has joined #openstack-nova17:17
*** unicell has joined #openstack-nova17:18
*** unicell1 has quit IRC17:19
jaypipespkholkin: for flavors and images, you can only have one because the flavor and image represents the *requested* thing and the aggregate represents the *available* thing. It's like saying "I request an orange and the things that can fulfill my request are providers that include 'orange' in their list of fruit types."17:20
pkholkinjaypipes: agree with you, but sometimes I want the provider that does only oranges :)17:21
*** daemontool_ has quit IRC17:21
jaypipespkholkin: in other words, a flavor or image says "I am designed for Windows". It doesn't say "I am designed for Windows OR Linux"...17:21
pkholkinjaypipes: I want to implement this spec for images and for flavors, too https://review.openstack.org/#/c/85399/1717:21
pkholkinjaypipes: os_distro was just an example, this could be any keys of course)17:22
openstackgerritSTEW TY proposed openstack/nova: Complete verification of servers-action-fixed-ip.inc  https://review.openstack.org/33860617:22
jaypipespkholkin: the provider never "does only oranges". A provider has a list of accepted fruit types (the aggregate metadata value). However what *you* are looking for is not what the provider *allows* but rather what the provider *has already provided*. So, you need to look in an entirely different table (the instances table instead of hte aggregate_metadata table) for your answers.17:22
jaypipespkholkin: this is, incidentally, why server groups are a total pile of poo.17:23
pkholkin)17:24
pkholkinjaypipes: do you look the spec above? maybe before?17:24
*** slaweq_ has joined #openstack-nova17:25
*** lucasagomes is now known as lucas-dinner17:25
*** annegentle has joined #openstack-nova17:25
pkholkintoday I worked for PoC for this but these sets broke everything)17:26
*** unicell has quit IRC17:26
pkholkinseems that if some set has at least two values we will not get strict isolation17:26
jaypipespkholkin: correct.17:28
*** mdrabe has quit IRC17:29
*** mdrabe_ has joined #openstack-nova17:29
*** hemanthm|lunch is now known as hemanthm17:29
*** kfarr has quit IRC17:31
*** sudipto_ has quit IRC17:31
*** sudipto has quit IRC17:31
jaypipesbiab17:32
*** jaypipes has quit IRC17:32
clarkbhello nova. managed to get libvirt info on https://bugs.launchpad.net/nova/+bug/1558807 so I went ahead and changed the status from expired to new17:35
openstackLaunchpad bug 1558807 in OpenStack Compute (nova) "Volume attached at different path than reported by nova/cinder when using config drive" [Undecided,New]17:35
*** jaypipes has joined #openstack-nova17:36
*** claudiub has joined #openstack-nova17:36
*** kfarr has joined #openstack-nova17:38
*** yamahata has joined #openstack-nova17:38
*** itlinux has quit IRC17:39
*** rhagarty has joined #openstack-nova17:39
*** mingyu has quit IRC17:41
*** thorst_ has quit IRC17:45
*** sahid has quit IRC17:45
*** brian_price has joined #openstack-nova17:45
*** thorst_ has joined #openstack-nova17:46
ftersinmdbooth: diana's test's passed on master against ceph: http://logs.openstack.org/11/338411/2/check/gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial/499f2b0/console.html#_2016-09-14_17_11_14_58304417:50
*** READ10 has quit IRC17:50
auggyclarkb: did you update the bug with the libvirt info? i saw a comment from another person providing some info, is it the same as that?17:50
clarkbauggy: its not my libvirt info its theirs17:50
auggyclarkb: right yeah, are they going to update the bug with that info? or is the comment from Jim the same?17:51
*** baoli_ has quit IRC17:51
clarkbI am saying the comment from jim is why I unexpired the review17:51
clarkber bug17:51
*** tblakes has joined #openstack-nova17:51
clarkbsince the lacking piece of info has been provided17:51
*** unicell has joined #openstack-nova17:52
auggyclarkb: cool, wfm, just wanted to make sure that was the missing info you were referring to :) thanks17:52
*** baoli has joined #openstack-nova17:52
*** READ10 has joined #openstack-nova17:52
diana_clarkeftersin: And by "pass" we actually mean "fail" b/c that test demonstrates a bug. PS. I suspect mdbooth is gone for the day.17:52
*** thorst_ has quit IRC17:54
ftersindiana_clarke: why? it resizes to 2Gb and verifies that the instance get 2Gb17:54
*** zenoway has quit IRC17:54
diana_clarkeftersin: I mean the ephemeral resize part.17:56
ftersindiana_clarke: there is a bug of ephemeral resizing, but today matthew spoke about resizing of root17:56
* diana_clarke nods17:56
ftersinahh, yes. it does not work at all17:57
*** pwnall has joined #openstack-nova17:59
*** edand has joined #openstack-nova17:59
*** pwnall is now known as Guest298417:59
*** vishwanathj_zzz is now known as vishwanathj18:01
*** BjoernT has quit IRC18:03
*** mvk has joined #openstack-nova18:03
*** moshele has quit IRC18:08
*** brian_price has quit IRC18:11
*** brian_price has joined #openstack-nova18:13
openstackgerritSean Dague proposed openstack/nova: DNM: test hacking os-api-ref-src target  https://review.openstack.org/37035418:14
*** Sukhdev has joined #openstack-nova18:16
dansmitheffing finally18:27
*** Guest8550 has quit IRC18:27
dansmithclarkb: so, libvirt's notion of where devices are is a total guess18:28
*** READ10 has quit IRC18:28
dansmithclarkb: and we even recently stopped trying so hard18:29
dansmithbecause we can't enforce anything18:29
dansmithclarkb: the notion of where something is attached is a xenism18:29
clarkbthe problem is there isn't any other reasonable way to know where your volume is18:29
clarkbbecause kvm and xen and different and nova is wrong18:29
clarkbits actually a hugely important use case for people trying to use volumes18:29
dansmithclarkb: there is now, with device metadata18:29
clarkbdansmith: so I am supposed to ask cinder for the location now?18:30
* clarkb doesn't know what device metadata is or where to find it18:30
dansmithclarkb: no, cinder has no idea18:30
dansmithclarkb: device metadata is in configdrive or metadata server and should give you information you can use, like volume serial number or something18:30
clarkbdansmith: but that won't update because config drive is static18:31
clarkbwhich means its only useful if I have all my volumes attached at boot and never chagne them18:31
dansmithclarkb: yeah if you use configdrive18:31
dansmithclarkb: my point is, a bug that says the api shows the wrong device path is somewhat pointless at this point I think18:32
clarkbexcept users still need it18:32
clarkbis my point18:32
clarkbnone of the other options come close18:32
dansmithclarkb: but we really can't ever make that correct for anything but xen18:32
dansmithclarkb: we can't control where your udev decides to put a thing18:32
clarkbdansmith: you don't have to control it though you just have to report it back accurately18:33
dansmithclarkb: we can give you *other* information perhaps, but the pathing is not under our control18:33
clarkbI don't care where it goes just that I am tiold where it went18:33
clarkband that I am told accurately18:33
*** mgoddard_ has joined #openstack-nova18:33
dansmithclarkb: the only thing that knows that is the guest OS18:33
*** artom has quit IRC18:33
clarkbok then we need something else (and static data on config drive doesn't cut it) that tells users what volumes map to which devices18:34
clarkbit turns out that we add and remove volumes all the time and its a fairly normal thing to do18:34
*** baoli has quit IRC18:34
dansmithclarkb: so despite it not helping you for configdrive, this is the device metadata: https://github.com/openstack/nova/blob/master/nova/api/metadata/base.py#L378-L43718:34
*** artom has joined #openstack-nova18:35
*** mgoddard has quit IRC18:36
*** ducttape_ has quit IRC18:36
clarkblooks like serial is optional? and either it or path are the things that would help you find the volume?18:36
*** ducttape_ has joined #openstack-nova18:37
dansmithclarkb: they're optional in that we get some things from some virt drivers and other things from others, AFAIK18:37
dansmithclarkb: meaning, on libvirt we might be able to give you serial, and on xen we might only be able to give you device node18:37
*** mgoddard_ has quit IRC18:38
clarkbbut we should get at least one of them?18:38
dansmithclarkb: yeah, and I think bus/address is the most reliable18:38
clarkbdansmith: the thing we get on kvm with /dev/disk/by-id/$uuidprefixhere is actually pretty nice too but xen can't do that?18:39
*** lpetrut has joined #openstack-nova18:40
dansmithclarkb: I think we can't shove the full uuid in or something though right?18:40
dansmithclarkb: dunno about xen18:40
clarkbya there is a length limit and I want to say full uuid goes past that18:40
clarkbsomething like that at least18:40
dansmithclarkb: vish did a bunch of work trying to get the volume uuid into or something and couldn't because of length18:41
dansmithyeah18:41
dansmithclarkb: correlating a tag you provide with a bus, controller, device is pretty accurate,18:41
dansmithand cloud-init could do all the nice stuff for you, modulo the synchronization problem of configdrive of course18:42
dansmithbut configdrive is kinda well-understood to be static, so..18:42
clarkbya our problem is our volumes have never really been static18:42
clarkbwe semi frequently add new ones to increase disk availability or swap out slow volumes for fast ones and so on18:42
dansmithclarkb: so I dunno what to tell you.. I'm not sure I'd want to expose this out the public API, but I guess we could.. only helps if you're instrumenting this from outside, and this was really added for things inside the instance to consume18:42
dansmithclarkb: sure, I'm not discounting your use case of course18:43
clarkbya the reason we use the external public api is because it appeared to work the same across clouds (but doesnt and took a while to figure that out)18:43
clarkbwhereas we knew pretty early on the on instance info was spotty and not consistent across clouds18:43
dansmithclarkb: yeah it totally did when all public clouds were RAX on Xen :)18:43
*** sajeesh has quit IRC18:44
*** mgoddard has joined #openstack-nova18:44
dansmithwe talked about deprecating and removing that field from the api in a microversion, but not sure we actually did it18:45
clarkbwhat would be really neat is if the cinder uuid could match the uuid reported by blkid18:46
clarkbI wonder if thats something that could be forced over iscis18:46
dansmithclarkb: right but that blkid uuid is on the filesystem right?18:46
dansmith -U uuid18:47
dansmith              Look up the device that uses this filesystem uuid.18:47
*** lpetrut has quit IRC18:47
dansmithlooks like if you use a fancy partition table maybe there's some correlation there, but either way, not a block thing AFAIK18:47
clarkbI have partuuid and uuid18:47
clarkbpartuuid looks to belong to partition table18:48
dansmithright, but a completely blank block device should have neither I think18:48
clarkbah18:48
dansmitha DOS partition table won't have partuuid and uuid is on the filesystem I think18:48
dansmithserial is the only thing that belongs to the actual block device as far as I know, and that's the short one18:48
dansmithshort based on 1980s ATA specs or something I think :P18:49
*** lpetrut has joined #openstack-nova18:49
dansmithclarkb: but yes, I agree with your assertion about what /would/ be nice :P18:49
openstackgerritLudovic Beliveau proposed openstack/nova: PCI: Release PCI devices on drop_move_claim()  https://review.openstack.org/37037418:50
*** tonytan4ever has quit IRC18:51
*** thorst_ has joined #openstack-nova18:53
*** edand has quit IRC18:53
*** zul_ has quit IRC18:55
*** liverpooler has quit IRC18:58
dansmithsdague: I think the convo with clarkb just now has me pretty convinced we need to go ahead and deprecate that piece of the api to avoid confusion going forward18:58
dansmithsdague: thoughts?18:58
*** zul has quit IRC18:59
*** thorst_ has quit IRC19:00
*** moshele has joined #openstack-nova19:00
*** ayoung has joined #openstack-nova19:01
openstackgerritFeodor Tersin proposed openstack/nova: libvirt: Fix wrong call of _is_booted_from_volume  https://review.openstack.org/33460919:01
openstackgerritFeodor Tersin proposed openstack/nova: libvirt: Delete duplicate check when live-migrating  https://review.openstack.org/33705619:01
openstackgerritFeodor Tersin proposed openstack/nova: libvirt: Simplify internal usage of get_instance_disk_info  https://review.openstack.org/33490119:01
*** fragatina has quit IRC19:02
*** efoley has quit IRC19:02
mriedemthis reminded me that https://review.openstack.org/#/c/305120/ isn't merged yet19:02
mriedemheh, i read 'piece of the api' as something else19:02
dansmithheh19:02
*** moshele has quit IRC19:03
*** zul_ has joined #openstack-nova19:06
*** pcaruana has joined #openstack-nova19:07
mriedemalex_xu: are you working through the night to update this? https://review.openstack.org/#/c/369147/19:10
openstackgerritTom Patzig proposed openstack/nova: refresh instances_path when shared storage used  https://review.openstack.org/36685719:10
dansmithmriedem: I can update it, I was mostly waiting for the other few things that were in the queue19:11
mriedemworks for me19:11
mriedemwe have 2 things left in the rc bugs etherpad19:11
mriedemhttps://etherpad.openstack.org/p/nova-newton-rc-potential19:11
*** kfarr has quit IRC19:17
dansmithjaypipes: can you see my comments just now on alex_xu's patch? https://review.openstack.org/#/c/369147/19:17
jaypipesdansmith: reading now.19:18
*** kylek3h_away is now known as kylek3h19:19
jaypipesdansmith: done19:21
dansmithjaypipes: thanks19:21
dansmithmriedem: I have edleafe's nit corrected and will rebase when the stuff in the gate makes it through, as this conflicts with those19:22
mriedemok19:22
*** kaisers has quit IRC19:22
dansmithI would do it now, but it is at the bottom of a stack from alex and I wouldn't want him pushing that stack again and messing up the ones in the gate19:22
*** prateek has joined #openstack-nova19:24
*** prateek has quit IRC19:24
*** cdent has joined #openstack-nova19:25
*** jed56 has quit IRC19:25
mriedemoh i see it's ready to go, just needs the rebase when the time comes19:27
dansmithyup19:27
cdentdiana_clarke: [t 9jB] <- I had similar hopes and expectations. I'm wondering what we can do (down the road, next cycle, not during rc1, etc) to make more of that happen.19:28
purplerbot<diana_clarke> I have pretty extensive notes, but I had hoped for more of a back and forth with a subject matter expert. [2016-09-14 16:38:36] [n 9jB]19:28
*** akshai has quit IRC19:28
*** sdake_ has joined #openstack-nova19:29
*** ducttape_ has quit IRC19:30
dansmithcdent: really? jaypipes was holding out on you? :)19:31
*** matrohon has joined #openstack-nova19:31
*** aswadr_ has quit IRC19:32
*** bvanhav has quit IRC19:32
*** lpetrut has quit IRC19:32
cdentdansmith: heh. not just him. I think much of the problem was on my side: I didn't manage my expectations. I was hoping for a lot more iteration, earlier. What's happened in the past week and a half has been great (in the way it has proved things out) but it would have been nice a lot earlier.19:33
*** sdake has quit IRC19:33
*** Apoorva_ has joined #openstack-nova19:33
dansmithcdent: wow, I think you got an incredible amount of back and forth early on, but.. maybe that's just me19:33
dansmithdefinitely more hands-on than most people get with their features :)19:33
cdentdansmith: that's what I'm saying, my expectations were out of wack with reality19:34
*** Sukhdev has quit IRC19:34
dansmithheh, okay19:34
mriedemcdent: just interested, how was velocity like this kind of thing in ceilometer? i think working in different projects gives different expectations19:34
cdenton the other hand if the attention I got was way above average then it sucks to be other people and my comment about making it better, for everyone, down the road is even more important19:35
mriedemwell, you got more attention b/c you're working on a big ass priority item19:35
mriedembig ass as in, big change19:36
mriedema new api isn't a trivial vendor bp thingy19:36
*** lpetrut has joined #openstack-nova19:36
*** Sukhdev has joined #openstack-nova19:36
dansmithyeah, we've all been banking up attention, hesitant to spend it...19:36
*** Apoorva has quit IRC19:36
cdentreview latency in ceilo, for important stuff was on the order of hours, not days or weeks19:37
cdentbut the volume of stuff was obviously _way_ less19:37
mriedemyeah i was going to say19:37
mriedemwe had 100 approved bps19:37
cdentand the ratio of cores to contributions was more balanced19:37
mriedemwith half the core team driving several of them19:37
diana_clarkemriedem: Do you think a nova core can be assigned to the image backend refactor for the next cycle?19:38
cdentIn case I'm not being clear here, I'm not trying to bitch, I'm merely trying to support what diana_clarke was saying and wondering how we can make it better for everyone19:38
mriedemdiana_clarke: i can't assign anyone to anything19:38
diana_clarkemriedem: :)19:38
mriedemi've been reminded a few times this release that people are going to work on and review what they want to work on and review19:38
mriedemi think we'll have fewer priorities for starters in ocata19:39
mriedemit's a shorter cycle19:39
mriedemso i think it's going to be a continuation of newton for some of the things that didn't get done19:39
dansmithyeah19:40
*** salv-orl_ has joined #openstack-nova19:40
mriedemexcept ocata we have the py3 thing...19:40
*** lpetrut has quit IRC19:40
mriedema core might need to be literally assigned to reviewing those19:41
dansmithlol19:41
diana_clarkeThere are only like 1000 mox tests, at an hours each... plus reviews. lol19:42
mriedemyes super fun19:42
*** salv-orlando has quit IRC19:43
*** baoli has joined #openstack-nova19:43
* cdent shrugs19:44
*** akshai has joined #openstack-nova19:45
cdentI'm getting the impression that there's not much enthusiasm explore ways to change things? Is that because now is the wrong time to be talking about it, or because there's no need to change things, or because of something else?19:45
*** akshai_ has joined #openstack-nova19:46
*** Apoorva_ has quit IRC19:47
diana_clarkecdent: I'm nervous about discussing it b/c I don't want to burn any goodwill. If I have any left.., lol.19:47
*** Apoorva has joined #openstack-nova19:47
* mriedem gets out his folder of permanent records19:48
mriedemcdent: there is a chunk in the ocata summit ideas etherpad about a newton retrospective in barcelona19:49
mriedemi was going to bring it up in the nova meeting tomorrow too19:49
diana_clarkecdent: I think delegating some of the things like reviewing mox patches to non-cores is one thing that might help.19:49
diana_clarkecdent: but I gather that's been discussed and declined may times before19:49
mriedemsubteams haven't been declined19:49
*** akshai has quit IRC19:49
* dansmith literally smashes his face into the table19:49
mriedemthat's what the review priorities etherpad was mainly for, like 60% of it was for subteams19:50
mriedemgranted, the etherpad grows stale19:50
* cdent gives dansmith a cold cloth19:50
mriedembut it was meant as a place for subteams to post things that need to be reviewed by the subteam and then put in a queue for cores to look at19:50
mriedemmarkus_z (i think) floated the idea of a gerrit dashboard for subteams but i think that was dependent on gerrit tag support or something19:51
diana_clarkeI mean more like, "you get to +2 only mox patches. 3 bad calls and your lose +2. proceed with caution."19:51
mriedembasically star/tag reviews that are deemed ready by a subteam for review by cores19:51
diana_clarkeman... there's a typo in everything I type today. baby is keeping me up most nights recently, sorry.19:52
mriedemi'm not sure if gerrit can even grant +2 on something like that w/o just having +2 on the entire repo19:52
*** tonytan4ever has joined #openstack-nova19:52
mriedemsubtrees maybe19:52
mriedemi'm not crazy about +2s like that anyway because it's easier to give someone +2 than it is to take it away19:53
mriedemit gets messy19:53
diana_clarkemriedem: I don't think gerrit necessarily needs to enforce it.19:53
*** hoonetorg has quit IRC19:53
openstackgerritedan david proposed openstack/nova: Remove mox in network/test_linux_net.py  https://review.openstack.org/31642719:53
diana_clarkeBut anyhoo, I know this has been discussed to death, so I won't make you re-hash it.19:53
*** tonytan4ever has quit IRC19:53
*** tonytan4ever has joined #openstack-nova19:54
*** moshele has joined #openstack-nova19:54
*** moshele has quit IRC19:55
*** can8dnSix has joined #openstack-nova19:55
sdaguedansmith: the device setting part? yeh, agreed.19:57
dansmithsdague: yeah, cool, lets NFO that early in ocata19:57
mriedemcdent: question in https://review.openstack.org/#/c/370154/119:57
*** garyk has joined #openstack-nova19:58
*** ijw has joined #openstack-nova19:58
*** thorst_ has joined #openstack-nova19:58
cdentmriedem: https://www.python.org/dev/peps/pep-0333/#environ-variables19:58
*** garyk has quit IRC19:58
cdentthe _values_ are unpredictable, but the key names for the incoming environ have a spec19:59
mriedemok, your upper threw me off19:59
mriedemoh19:59
mriedemyeah19:59
mriedemderp, mixed that up19:59
*** pcaruana has quit IRC20:00
*** garyk has joined #openstack-nova20:01
*** lamt has quit IRC20:01
*** rhagarty has quit IRC20:02
*** zul has joined #openstack-nova20:02
*** jvgrant has quit IRC20:02
cdentmriedem: I'm away for the next 5 days so if you or others would like that to move sooner, please feel free to have at20:02
mriedemis it just me or does this not work? tox -e functional -- nova.tests.functional.api.openstack.placement20:02
mriedemi'm guessing that doesn't fly with gabbi?20:03
cdentmriedem: that does not work because of the magical way gabbi names tests, so it busts discover20:03
*** rama_y has quit IRC20:03
cdenttox -efunctional placement will work20:03
cdentso will tox -efunctional gabbi20:03
*** ijw has quit IRC20:03
*** thorst_ has quit IRC20:04
*** rhagarty has joined #openstack-nova20:04
sdaguemriedem: it's using the test loader standard, which testr uses, so yeh, it all works20:06
openstackgerritMatt Riedemann proposed openstack/nova: [placement] prevent a KeyError in webob.dec.wsgify  https://review.openstack.org/37015420:06
*** gouthamr has quit IRC20:07
*** garyk has quit IRC20:07
*** lamt has joined #openstack-nova20:07
*** hoonetorg has joined #openstack-nova20:08
*** can8dnSix has quit IRC20:08
*** ftersin has quit IRC20:08
cdentmriedem, sdague: the gabbi tests end up with names like this: gabbi.suitemaker.test_placement_api_allocations_check_allocations_by_consumer_id20:08
*** baoli has quit IRC20:09
cdentthat's because the loader is letting a default prefix happen. that can be changed to use the package path to the loader if it is helpful20:09
*** baoli has joined #openstack-nova20:09
*** zul has quit IRC20:11
*** zul has joined #openstack-nova20:12
*** jvgrant has joined #openstack-nova20:12
*** ducttape_ has joined #openstack-nova20:12
*** can8dnSix has joined #openstack-nova20:13
*** Apoorva has quit IRC20:15
*** Apoorva has joined #openstack-nova20:16
cdentgood night all20:16
*** cdent has quit IRC20:16
*** fragatina has joined #openstack-nova20:17
*** ijw has joined #openstack-nova20:19
*** tonytan4ever has quit IRC20:22
*** ijw has quit IRC20:24
openstackgerritMatt Riedemann proposed openstack/nova: [placement] Mark HTTP error responses for translation  https://review.openstack.org/36903520:24
*** mfedosin has quit IRC20:25
*** ducttape_ has quit IRC20:27
*** kfox1111 has joined #openstack-nova20:28
kfox1111anyone ever see this out of libvirt: qemu-system-x86_64: -chardev pty,id=charserial1: Fai20:28
kfox1111led to create chardev20:28
*** ducttape_ has joined #openstack-nova20:33
*** rcernin has quit IRC20:36
kfox1111a strace shows: chown("/dev/pts/4", 107, 5)       = -1 EPERM (Operation not permitted)20:37
kfox1111weird...20:37
*** tongli has quit IRC20:38
*** rcernin has joined #openstack-nova20:40
*** rcernin has quit IRC20:41
*** rcernin has joined #openstack-nova20:41
*** rcernin has quit IRC20:42
*** rcernin has joined #openstack-nova20:42
*** gouthamr has joined #openstack-nova20:43
*** hemna is now known as hemnafk20:44
openstackgerritSujitha proposed openstack/nova: Improve help text for glance options  https://review.openstack.org/32400720:45
*** kfarr has joined #openstack-nova20:48
*** mgoddard has quit IRC20:48
*** akshai_ has quit IRC20:51
*** coolsvap has quit IRC20:52
*** ijw has joined #openstack-nova20:52
*** can8dnSix has quit IRC20:59
*** thorst_ has joined #openstack-nova21:03
*** jheroux has quit IRC21:03
*** matrohon has quit IRC21:04
*** ijw has quit IRC21:07
*** slaweq_ has quit IRC21:08
*** thorst_ has quit IRC21:09
*** xinliang has quit IRC21:10
*** kfarr has quit IRC21:14
*** ducttape_ has quit IRC21:20
*** edmondsw has quit IRC21:21
*** xinliang has joined #openstack-nova21:23
*** ijw has joined #openstack-nova21:28
openstackgerritSujitha proposed openstack/nova: Improve help text for network  options.  https://review.openstack.org/34120321:30
*** rtheis has quit IRC21:31
*** ijw has quit IRC21:32
*** cleong has quit IRC21:34
*** rhagarty has quit IRC21:39
*** fragatin_ has joined #openstack-nova21:39
*** mriedem has quit IRC21:40
*** fragatina has quit IRC21:42
*** ijw has joined #openstack-nova21:44
*** rlrossit has quit IRC21:47
*** dave-mccowan has quit IRC21:48
*** ijw has quit IRC21:49
*** Swami has quit IRC21:49
*** mikal has quit IRC21:51
*** mikal has joined #openstack-nova21:51
*** tonytan4ever has joined #openstack-nova21:53
*** mikal has quit IRC21:53
*** mikal has joined #openstack-nova21:56
*** claudiub has quit IRC21:57
*** tonytan4ever has quit IRC21:58
*** fragatin_ has quit IRC21:58
*** fragatina has joined #openstack-nova21:59
*** ijw has joined #openstack-nova22:01
*** krtaylor has quit IRC22:02
*** burgerk has quit IRC22:04
*** mikal has quit IRC22:05
*** lpetrut has joined #openstack-nova22:06
*** mikal has joined #openstack-nova22:07
*** thorst has joined #openstack-nova22:08
jaypipesdansmith: seems we're in recheck hel...22:09
dansmithjaypipes: well, I merged one thing22:09
dansmithwas about to recheck the others22:09
dansmithbeen watching like a large bird22:09
*** eharney has quit IRC22:11
*** browne has quit IRC22:13
*** rama_y has joined #openstack-nova22:14
*** thorst has quit IRC22:15
*** mdrabe_ has quit IRC22:15
*** TravT has joined #openstack-nova22:16
openstackgerritDan Smith proposed openstack/nova: Clean up allocation when update available resources  https://review.openstack.org/36914722:17
*** ducttape_ has joined #openstack-nova22:17
*** lpetrut has quit IRC22:19
*** yamahata has quit IRC22:20
*** yamahata has joined #openstack-nova22:21
openstackgerritPavel Kholkin proposed openstack/nova: [WIP] scheduler: strictly isolate group of hosts for image/flavor  https://review.openstack.org/37045822:22
*** mtanino has quit IRC22:23
*** lamt has quit IRC22:29
*** baoli has quit IRC22:30
*** baoli has joined #openstack-nova22:31
*** ducttape_ has quit IRC22:32
*** ducttape_ has joined #openstack-nova22:32
*** hemanthm is now known as hemanthm|away22:32
*** ociuhandu has joined #openstack-nova22:33
*** Apoorva_ has joined #openstack-nova22:36
*** ducttape_ has quit IRC22:37
*** Apoorva has quit IRC22:39
*** sdake_ has quit IRC22:40
*** rcernin has quit IRC22:43
*** mlavalle has quit IRC22:43
*** zul has quit IRC22:44
*** Swami has joined #openstack-nova22:45
*** Swami has quit IRC22:46
*** Swami has joined #openstack-nova22:46
*** pradk has quit IRC22:47
*** diogogmt has quit IRC22:47
*** jaypipes has quit IRC22:50
*** tblakes has quit IRC22:56
*** takashin has joined #openstack-nova22:56
openstackgerritAnusha Unnam proposed openstack/nova: Improve help text for interval_opts  https://review.openstack.org/33176122:59
*** slagle has quit IRC23:01
openstackgerritAnusha Unnam proposed openstack/nova: Improve help text for interval_opts  https://review.openstack.org/33176123:01
*** sdague has quit IRC23:02
alex_xudansmith: thanks for take care the patch23:06
dansmithalex_xu: np23:06
*** lamt has joined #openstack-nova23:06
*** baoli has quit IRC23:06
* alex_xu can prepare to go to the zoo with baby now \o/23:07
*** tblakes has joined #openstack-nova23:08
*** kuzko has quit IRC23:09
*** ducttape_ has joined #openstack-nova23:10
*** xyang1 has quit IRC23:10
*** kuzko has joined #openstack-nova23:11
*** thorst has joined #openstack-nova23:12
*** browne has joined #openstack-nova23:13
*** nic has quit IRC23:19
*** thorst has quit IRC23:20
*** Swami has quit IRC23:21
*** Swami has joined #openstack-nova23:21
*** jamielennox is now known as jamielennox|away23:22
openstackgerritAnusha Unnam proposed openstack/nova: Remove deprecated flag in neutron.py  https://review.openstack.org/37047523:22
*** tri2sing has quit IRC23:22
*** takedakn has joined #openstack-nova23:24
*** ababich has quit IRC23:27
*** ababich has joined #openstack-nova23:27
*** kylek3h has quit IRC23:27
*** ijw has quit IRC23:27
*** ijw has joined #openstack-nova23:28
*** ijw has quit IRC23:29
*** Swami_ has joined #openstack-nova23:30
*** Swami has quit IRC23:32
openstackgerritVu Tran proposed openstack/nova: libvirt: incorrectly create nova/instances/<uuid>/<uuid>_disk with lvm backend  https://review.openstack.org/37048023:33
*** rfolco has quit IRC23:36
*** ducttape_ has quit IRC23:36
*** Apoorva_ has quit IRC23:36
*** Apoorva has joined #openstack-nova23:37
clarkbdansmith: http://logs.openstack.org/42/370442/1/check/gate-tempest-dsvm-multinode-full-ubuntu-xenial/13109f3/logs/screen-n-cpu.txt.gz?level=WARNING I think I fixed one issue with nova net multinode testing then ran into that. Is that a known thing?23:38
*** annegentle has quit IRC23:38
clarkbdansmith: almost looks like nova computes are just not registering themselves properly?23:38
*** gjayavelu has joined #openstack-nova23:38
dansmithclarkb: mmm, nope23:39
clarkbI have rechecked to see if thats a consistent thing. also oddly doesn't seem to affect the live migration job itself23:39
*** maticue_ has quit IRC23:40
*** maticue has quit IRC23:40
clarkbdansmith: both computes have the same complaints too "this is another computes instance"23:40
*** ducttape_ has joined #openstack-nova23:41
*** ducttape_ has quit IRC23:44
*** ducttape_ has joined #openstack-nova23:44
*** browne has quit IRC23:45
*** mriedem has joined #openstack-nova23:48
*** ociuhandu has quit IRC23:49
mriedemo/23:50
*** ijw has joined #openstack-nova23:50
*** ijw has quit IRC23:50
*** ijw has joined #openstack-nova23:51
pumaranikarhieulq,23:51
pumaranikarI have a question regarding libvirt config options patches that you submitted in gerrit, I can see few options were left in that file, do you have patches for them?23:51
*** ducttape_ has quit IRC23:51
dansmithclarkb: both complain about the same instance/23:56
*** hieulq has quit IRC23:56
*** mmedvede has quit IRC23:57

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