Monday, 2017-07-17

*** artom_ has joined #openstack-nova00:07
*** sapcc-bot has joined #openstack-nova00:09
*** sapcc-bot has quit IRC00:09
*** sapcc-bot has joined #openstack-nova00:10
*** sapcc-bot has quit IRC00:10
*** gmann has quit IRC00:12
*** gmann has joined #openstack-nova00:12
*** sapcc-bot has joined #openstack-nova00:15
*** sapcc-bot has quit IRC00:15
*** sapcc-bot has joined #openstack-nova00:16
*** sapcc-bot has quit IRC00:16
*** sapcc-bot has joined #openstack-nova00:21
*** sapcc-bot has joined #openstack-nova00:27
*** sapcc-bot has quit IRC00:27
*** kiennt has joined #openstack-nova00:35
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/48421800:40
*** sapcc-bot has joined #openstack-nova00:44
*** sapcc-bot has quit IRC00:44
*** sapcc-bot has joined #openstack-nova00:45
*** sapcc-bot has quit IRC00:45
*** mdnadeem has joined #openstack-nova00:45
*** acormier has joined #openstack-nova00:51
*** sapcc-bot has joined #openstack-nova00:56
*** sapcc-bot has quit IRC00:56
*** acormier has quit IRC00:56
*** acormier has joined #openstack-nova00:57
*** thorst has joined #openstack-nova00:59
*** thorst has quit IRC01:04
*** huanxie has joined #openstack-nova01:05
*** sapcc-bot has joined #openstack-nova01:09
*** sapcc-bot has quit IRC01:09
*** yufei has joined #openstack-nova01:16
*** zhurong has joined #openstack-nova01:20
*** sapcc-bot has joined #openstack-nova01:20
*** sapcc-bot has quit IRC01:20
*** zhurong has quit IRC01:23
*** liujiong has joined #openstack-nova01:24
*** markvoelker has joined #openstack-nova01:29
*** phuongnh has joined #openstack-nova01:30
*** sapcc-bot has joined #openstack-nova01:34
*** sapcc-bot has quit IRC01:34
*** liujiong_lj has joined #openstack-nova01:36
*** liujiong has quit IRC01:37
*** sapcc-bot has joined #openstack-nova01:39
*** sapcc-bot has quit IRC01:39
*** liujiong_lj is now known as liujiong01:39
*** sbezverk has quit IRC01:42
*** sapcc-bot has joined #openstack-nova01:50
*** sapcc-bot has quit IRC01:50
*** sapcc-bot has joined #openstack-nova01:58
*** sapcc-bot has quit IRC01:58
*** yufei has quit IRC01:59
*** itlinux_ has quit IRC02:00
*** mdnadeem has quit IRC02:01
*** markvoelker has quit IRC02:03
*** yufei has joined #openstack-nova02:05
*** kiennt has quit IRC02:07
*** sapcc-bot has joined #openstack-nova02:08
*** sapcc-bot has quit IRC02:08
*** itlinux has joined #openstack-nova02:10
*** gcb has joined #openstack-nova02:11
*** mdnadeem has joined #openstack-nova02:14
*** sapcc-bot has joined #openstack-nova02:15
*** sapcc-bot has quit IRC02:15
*** litao__ has joined #openstack-nova02:16
*** phuongnh has quit IRC02:17
*** phuongnh has joined #openstack-nova02:21
*** sapcc-bot has joined #openstack-nova02:26
*** sapcc-bot has quit IRC02:26
*** zhurong has joined #openstack-nova02:31
*** sapcc-bot has joined #openstack-nova02:31
*** sapcc-bot has quit IRC02:31
*** sapcc-bot has joined #openstack-nova02:37
*** sapcc-bot has quit IRC02:37
*** sapcc-bot has joined #openstack-nova02:43
*** sapcc-bot has quit IRC02:43
*** sapcc-bot has joined #openstack-nova02:43
*** sapcc-bot has quit IRC02:43
*** yamamoto has joined #openstack-nova02:47
*** sapcc-bot has joined #openstack-nova02:50
*** sapcc-bot has quit IRC02:50
*** acormier has quit IRC02:52
*** acormier has joined #openstack-nova02:52
*** yufei has quit IRC02:54
*** yufei has joined #openstack-nova02:55
*** acormier has quit IRC02:57
*** markvoelker has joined #openstack-nova03:00
*** thorst has joined #openstack-nova03:01
*** sapcc-bot has joined #openstack-nova03:01
*** sapcc-bot has quit IRC03:01
*** dave-mccowan has quit IRC03:04
*** thorst has quit IRC03:05
*** baoli has quit IRC03:05
*** yamamoto has quit IRC03:09
*** sapcc-bot has joined #openstack-nova03:12
*** sapcc-bot has quit IRC03:12
*** s1061123 has joined #openstack-nova03:13
*** kristian__ has joined #openstack-nova03:16
*** baoli has joined #openstack-nova03:16
*** kiennt has joined #openstack-nova03:18
*** sapcc-bot has joined #openstack-nova03:18
*** sapcc-bot has quit IRC03:18
*** Nel1x has quit IRC03:20
*** Sukhdev has quit IRC03:20
*** itlinux has quit IRC03:23
*** sapcc-bot has joined #openstack-nova03:25
*** sapcc-bot has quit IRC03:25
*** itlinux has joined #openstack-nova03:30
*** markvoelker has quit IRC03:34
*** sapcc-bot has joined #openstack-nova03:36
*** sapcc-bot has quit IRC03:36
*** hongbin has joined #openstack-nova03:39
*** udesale has joined #openstack-nova03:40
*** baoli has quit IRC03:41
*** mikal has quit IRC03:42
*** sapcc-bot has joined #openstack-nova03:42
*** sapcc-bot has quit IRC03:42
*** gouthamr has quit IRC03:42
*** links has joined #openstack-nova03:44
*** rajathagasthya has joined #openstack-nova03:47
openstackgerritTuan Luong-Anh proposed openstack/python-novaclient master: Replace uuid.uuid4().hex with uuidutils.generate_uuid()  https://review.openstack.org/48424403:48
*** sapcc-bot has joined #openstack-nova03:48
*** sapcc-bot has quit IRC03:48
*** kiennt has quit IRC03:52
*** kiennt has joined #openstack-nova03:59
*** sapcc-bot has joined #openstack-nova04:00
*** sapcc-bot has quit IRC04:00
*** sapcc-bot has joined #openstack-nova04:01
*** sapcc-bot has quit IRC04:01
*** hoonetorg has quit IRC04:05
*** sapcc-bot has joined #openstack-nova04:05
*** sapcc-bot has quit IRC04:05
*** yamamoto has joined #openstack-nova04:09
*** sapcc-bot has joined #openstack-nova04:11
*** sapcc-bot has quit IRC04:11
*** hongbin has quit IRC04:13
*** adisky__ has joined #openstack-nova04:13
*** sapcc-bot has joined #openstack-nova04:13
*** yamamoto has quit IRC04:14
*** trungnv has quit IRC04:15
*** trungnv has joined #openstack-nova04:15
*** psachin has joined #openstack-nova04:18
*** sapcc-bot has joined #openstack-nova04:23
*** sapcc-bot has quit IRC04:23
*** sapcc-bot has joined #openstack-nova04:24
*** sapcc-bot has quit IRC04:24
*** markvoelker has joined #openstack-nova04:30
*** rajathagasthya has quit IRC04:38
openstackgerritsudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid()  https://review.openstack.org/48100704:40
*** kristian__ has quit IRC04:41
*** sapcc-bot has joined #openstack-nova04:42
*** sapcc-bot has quit IRC04:42
*** Dinesh_Bhor has joined #openstack-nova04:46
*** sapcc-bot has joined #openstack-nova04:47
*** sapcc-bot has quit IRC04:47
*** Sukhdev has joined #openstack-nova04:47
*** kristian__ has joined #openstack-nova04:52
*** sapcc-bot has joined #openstack-nova04:52
*** sapcc-bot has quit IRC04:52
*** sapcc-bot has joined #openstack-nova04:53
*** sapcc-bot has quit IRC04:53
*** kristian__ has quit IRC04:54
*** kristian__ has joined #openstack-nova05:00
*** kristian__ has quit IRC05:00
*** sapcc-bot has joined #openstack-nova05:00
*** sapcc-bot has quit IRC05:00
*** kristian__ has joined #openstack-nova05:00
*** thorst has joined #openstack-nova05:01
*** zhurong has quit IRC05:04
*** markvoelker has quit IRC05:04
*** kristian__ has quit IRC05:05
*** sapcc-bot has joined #openstack-nova05:05
*** sapcc-bot has quit IRC05:05
*** thorst has quit IRC05:06
*** jjung has joined #openstack-nova05:13
*** ekuris_ has joined #openstack-nova05:13
*** ratailor has joined #openstack-nova05:16
*** trinaths has joined #openstack-nova05:22
*** trinaths1 has joined #openstack-nova05:22
*** armax has joined #openstack-nova05:23
*** trinaths has quit IRC05:26
*** sapcc-bot has joined #openstack-nova05:27
*** sapcc-bot has quit IRC05:28
*** nicolasbock has joined #openstack-nova05:28
*** zhurong has joined #openstack-nova05:29
*** armax has quit IRC05:30
*** armax has joined #openstack-nova05:30
*** armax has quit IRC05:31
*** armax has joined #openstack-nova05:31
*** armax has quit IRC05:32
*** armax has joined #openstack-nova05:32
*** armax has quit IRC05:32
*** armax has joined #openstack-nova05:33
*** armax has quit IRC05:33
*** sapcc-bot has joined #openstack-nova05:33
*** sapcc-bot has quit IRC05:33
*** armax has joined #openstack-nova05:34
*** sbezverk has joined #openstack-nova05:34
*** armax has quit IRC05:34
*** armax has joined #openstack-nova05:35
*** armax has quit IRC05:35
*** avolkov has joined #openstack-nova05:39
*** sapcc-bot has joined #openstack-nova05:40
*** sapcc-bot has quit IRC05:40
*** sridharg has joined #openstack-nova05:40
*** kiennt has quit IRC05:45
*** vvargaszte has joined #openstack-nova05:56
*** sapcc-bot has joined #openstack-nova05:56
*** sapcc-bot has quit IRC05:56
*** Sara has quit IRC05:58
*** rcernin_ has joined #openstack-nova06:00
*** sapcc-bot has joined #openstack-nova06:03
*** sapcc-bot has quit IRC06:03
*** kiennt has joined #openstack-nova06:03
*** prateek has joined #openstack-nova06:03
*** Oku_OS-away is now known as Oku_OS06:06
*** sapcc-bot has joined #openstack-nova06:08
*** nicolasbock has quit IRC06:13
*** nicolasbock has joined #openstack-nova06:15
*** kristian__ has joined #openstack-nova06:16
*** kiennt_ has joined #openstack-nova06:17
*** kiennt has quit IRC06:17
*** sapcc-bot has joined #openstack-nova06:20
*** sapcc-bot has quit IRC06:20
*** kristian__ has quit IRC06:20
*** andreas_s has joined #openstack-nova06:23
*** mikal has joined #openstack-nova06:24
*** vvargaszte1 has joined #openstack-nova06:24
*** sapcc-bot has joined #openstack-nova06:25
*** sapcc-bot has quit IRC06:26
*** vvargaszte has quit IRC06:26
*** vvargaszte1 is now known as vvargaszte06:26
*** prateek has quit IRC06:30
*** sapcc-bot has joined #openstack-nova06:31
*** sapcc-bot has quit IRC06:31
*** sapcc-bot has joined #openstack-nova06:32
*** sapcc-bot has quit IRC06:32
*** sapcc-bot has joined #openstack-nova06:33
*** sapcc-bot has quit IRC06:33
*** edmondsw has joined #openstack-nova06:39
*** sapcc-bot has joined #openstack-nova06:39
*** sapcc-bot has quit IRC06:39
*** kristian__ has joined #openstack-nova06:40
*** kristian__ has quit IRC06:40
*** kristian__ has joined #openstack-nova06:41
*** kristian__ has quit IRC06:41
*** kristian__ has joined #openstack-nova06:41
*** kristian__ has quit IRC06:42
*** kristian__ has joined #openstack-nova06:43
*** edmondsw has quit IRC06:43
*** rdopiera has joined #openstack-nova06:44
*** kristia__ has joined #openstack-nova06:46
*** kristian__ has quit IRC06:47
*** rdopiera has left #openstack-nova06:47
*** sapcc-bot has joined #openstack-nova06:49
*** sapcc-bot has quit IRC06:49
*** mlakat has joined #openstack-nova06:49
*** Sukhdev has quit IRC06:50
*** sapcc-bot has joined #openstack-nova06:56
*** sapcc-bot has quit IRC06:56
*** sshwarts has joined #openstack-nova06:58
*** itlinux has quit IRC06:59
*** sapcc-bot has joined #openstack-nova07:00
*** itlinux has joined #openstack-nova07:00
*** sapcc-bot has quit IRC07:00
*** markvoelker has joined #openstack-nova07:01
*** huanxie has quit IRC07:01
*** thorst has joined #openstack-nova07:02
*** thorst has quit IRC07:07
*** itlinux has quit IRC07:08
*** damien_r has joined #openstack-nova07:12
alex_xusdague: if you are around this week, there are something need your help on review https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/api-no-more-extensions-pike, just need one more +2. Thanks in advance!07:14
*** karthiks_afk is now known as karthiks07:16
*** sapcc-bot has joined #openstack-nova07:21
*** sapcc-bot has quit IRC07:21
*** damien_r has quit IRC07:22
*** alexchadin has joined #openstack-nova07:22
*** damien_r has joined #openstack-nova07:22
*** sapcc-bot has joined #openstack-nova07:23
*** sapcc-bot has quit IRC07:23
*** alexchad_ has joined #openstack-nova07:28
*** sapcc-bot has joined #openstack-nova07:29
*** sapcc-bot has quit IRC07:29
*** alexcha__ has joined #openstack-nova07:29
*** alexchadin has quit IRC07:30
*** sapcc-bot has joined #openstack-nova07:31
*** sapcc-bot has quit IRC07:31
*** tesseract has joined #openstack-nova07:32
*** alexchad_ has quit IRC07:32
*** alexchadin has joined #openstack-nova07:33
*** alexcha__ has quit IRC07:34
*** kuzko has quit IRC07:34
*** markvoelker has quit IRC07:35
*** alexchad_ has joined #openstack-nova07:36
*** kuzko has joined #openstack-nova07:37
*** alexcha__ has joined #openstack-nova07:37
*** prateek has joined #openstack-nova07:37
*** alexchadin has quit IRC07:38
*** alexchad_ has quit IRC07:40
*** ralonsoh has joined #openstack-nova07:42
*** alexcha__ is now known as alexchadin07:42
*** sapcc-bot has joined #openstack-nova07:47
*** sapcc-bot has quit IRC07:47
*** huanxie has joined #openstack-nova07:51
openstackgerritZhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4)  https://review.openstack.org/46980007:52
*** sapcc-bot has joined #openstack-nova07:53
*** sapcc-bot has quit IRC07:53
*** acormier has joined #openstack-nova07:53
*** acormier has quit IRC07:57
*** sapcc-bot has joined #openstack-nova07:59
*** sapcc-bot has quit IRC07:59
openstackgerritManas Mandlekar proposed openstack/nova master: Instance remains in migrating state forever  https://review.openstack.org/48391108:09
*** aarefiev has joined #openstack-nova08:10
*** sapcc-bot has joined #openstack-nova08:11
*** sapcc-bot has quit IRC08:11
*** gjayavelu has joined #openstack-nova08:15
*** sapcc-bot has joined #openstack-nova08:22
*** sapcc-bot has quit IRC08:22
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.live_migration_force_complete notification  https://review.openstack.org/48056108:23
*** thorst has joined #openstack-nova08:27
*** sridharg has quit IRC08:27
*** sapcc-bot has joined #openstack-nova08:29
*** jangutter has joined #openstack-nova08:29
*** sapcc-bot has quit IRC08:29
*** gjayavelu has quit IRC08:30
*** markvoelker has joined #openstack-nova08:31
*** thorst has quit IRC08:32
*** lucas-afk is now known as lucasagomes08:32
openstackgerritBalazs Gibizer proposed openstack/nova master: fix test_volume_swap_server instability  https://review.openstack.org/48428808:38
*** sapcc-bot has joined #openstack-nova08:41
*** sapcc-bot has quit IRC08:41
*** tesseract has quit IRC08:50
openstackgerritMerged openstack/nova master: conf: fix netconf, my_ip and host are unclear  https://review.openstack.org/44121008:51
*** sapcc-bot has joined #openstack-nova08:52
*** sapcc-bot has quit IRC08:52
*** tesseract has joined #openstack-nova08:53
*** mamandle has joined #openstack-nova08:54
*** cdent has joined #openstack-nova08:56
*** phuongnh has quit IRC08:58
*** sapcc-bot has joined #openstack-nova08:58
*** sapcc-bot has quit IRC08:58
mamandle@alex_xu I responded to your comments on https://review.openstack.org/#/c/483911/2, can you please check? Thanks.09:03
*** sapcc-bot has joined #openstack-nova09:04
*** sapcc-bot has quit IRC09:04
*** markvoelker has quit IRC09:05
*** dixiaoli has joined #openstack-nova09:07
*** dixiaoli has quit IRC09:07
*** yamamoto has joined #openstack-nova09:13
*** yamamoto has quit IRC09:13
*** yamamoto has joined #openstack-nova09:14
*** yamamoto has quit IRC09:16
*** yamamoto has joined #openstack-nova09:17
openstackgerritManas Mandlekar proposed openstack/nova master: Instance remains in migrating state forever  https://review.openstack.org/48391109:20
*** yamamoto has quit IRC09:21
*** yamamoto has joined #openstack-nova09:21
*** derekh has joined #openstack-nova09:22
*** sambetts|afk is now known as sambetts09:25
*** yamamoto has quit IRC09:26
*** sapcc-bot has joined #openstack-nova09:28
*** sapcc-bot has quit IRC09:28
*** tuanluong has joined #openstack-nova09:28
*** sapcc-bot has joined #openstack-nova09:29
*** sapcc-bot has quit IRC09:29
*** yamamoto has joined #openstack-nova09:30
*** sapcc-bot has joined #openstack-nova09:34
*** sapcc-bot has quit IRC09:34
openstackgerritBalazs Gibizer proposed openstack/nova master: explain payload inheritance in notification devref  https://review.openstack.org/45366709:34
openstackgerritBalazs Gibizer proposed openstack/nova master: Add snapshot id to the snapshot notifications  https://review.openstack.org/45307709:34
openstackgerritMerged openstack/python-novaclient master: Microversion 2.50 - fix quota class sets resource usage  https://review.openstack.org/48306009:38
*** yamamoto has quit IRC09:38
*** sapcc-bot has joined #openstack-nova09:40
*** sapcc-bot has quit IRC09:40
*** lpetrut has joined #openstack-nova09:40
*** takedakn has joined #openstack-nova09:41
*** yufei has quit IRC09:45
*** mdnadeem has quit IRC09:47
*** sapcc-bot has joined #openstack-nova09:52
*** sapcc-bot has quit IRC09:52
openstackgerritMerged openstack/python-novaclient master: Add support for the 2.51 microversion  https://review.openstack.org/48314909:57
*** sapcc-bot has joined #openstack-nova09:58
*** sapcc-bot has quit IRC09:58
openstackgerritZhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4)  https://review.openstack.org/46980010:01
*** markvoelker has joined #openstack-nova10:02
*** sapcc-bot has joined #openstack-nova10:05
*** sapcc-bot has quit IRC10:05
*** gcb has quit IRC10:06
*** trungnv has quit IRC10:06
*** liujiong has quit IRC10:06
*** jistr is now known as jistr|afk10:09
*** sapcc-bot has joined #openstack-nova10:10
*** sapcc-bot has quit IRC10:10
*** sapcc-bot has joined #openstack-nova10:10
*** sapcc-bot has quit IRC10:10
*** udesale__ has joined #openstack-nova10:12
*** udesale has quit IRC10:15
*** edmondsw has joined #openstack-nova10:15
*** sapcc-bot has joined #openstack-nova10:16
*** sapcc-bot has quit IRC10:16
*** sapcc-bot has joined #openstack-nova10:16
*** sapcc-bot has quit IRC10:16
openstackgerritBalazs Gibizer proposed openstack/nova master: Send soft_delete from context manager  https://review.openstack.org/47645910:18
openstackgerritBalazs Gibizer proposed openstack/nova master: use context mgr in instance.delete  https://review.openstack.org/44376410:18
openstackgerritBalazs Gibizer proposed openstack/nova master: Transform missing delete notifications  https://review.openstack.org/41029710:18
*** edmondsw has quit IRC10:19
*** sapcc-bot has joined #openstack-nova10:21
*** sapcc-bot has quit IRC10:21
*** sapcc-bot has joined #openstack-nova10:22
*** sapcc-bot has quit IRC10:22
*** trungnv has joined #openstack-nova10:23
*** ecelik has joined #openstack-nova10:24
*** thorst has joined #openstack-nova10:28
*** thorst has quit IRC10:32
*** mvk has quit IRC10:33
*** ecelik has left #openstack-nova10:33
*** cdent has quit IRC10:33
*** sapcc-bot has joined #openstack-nova10:34
*** sapcc-bot has quit IRC10:34
*** markvoelker has quit IRC10:35
*** aloga has quit IRC10:36
*** aloga has joined #openstack-nova10:36
*** yamamoto has joined #openstack-nova10:37
*** kiennt_ has quit IRC10:37
*** tuanluong has quit IRC10:38
*** udesale__ has quit IRC10:39
*** udesale has joined #openstack-nova10:40
*** owalsh has quit IRC10:40
*** owalsh has joined #openstack-nova10:42
*** tbachman has quit IRC10:46
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.live_migration_force_complete notification  https://review.openstack.org/48056110:46
*** lucasagomes is now known as lucas-hungry10:49
*** sridharg has joined #openstack-nova10:50
*** tbachman has joined #openstack-nova10:50
*** fragatina has joined #openstack-nova10:57
*** lpetrut_ has joined #openstack-nova10:57
*** sapcc-bot has joined #openstack-nova10:57
*** sapcc-bot has quit IRC10:57
*** fragatina has quit IRC10:57
*** sapcc-bot has joined #openstack-nova10:58
*** sapcc-bot has quit IRC10:58
*** fragatina has joined #openstack-nova10:58
*** lpetrut has quit IRC11:00
*** mvk has joined #openstack-nova11:00
openstackgerritBalazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (3)  https://review.openstack.org/45282011:01
openstackgerritBalazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data  https://review.openstack.org/45281811:01
openstackgerritBalazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (2)  https://review.openstack.org/45281911:01
openstackgerritBalazs Gibizer proposed openstack/nova master: Refactor instance.power-off notification samples  https://review.openstack.org/47586011:01
*** alexchadin has quit IRC11:04
*** Guest55861 has left #openstack-nova11:06
*** thorst has joined #openstack-nova11:08
*** thorst has quit IRC11:09
*** sapcc-bot has joined #openstack-nova11:09
*** sapcc-bot has quit IRC11:09
*** cdent has joined #openstack-nova11:14
*** sapcc-bot has joined #openstack-nova11:15
*** sapcc-bot has quit IRC11:15
openstackgerritBalazs Gibizer proposed openstack/nova master: Add sample test for instance audit  https://review.openstack.org/48095511:15
*** sapcc-bot has joined #openstack-nova11:16
*** thorst has joined #openstack-nova11:16
*** mamandle has quit IRC11:23
*** hoonetorg has joined #openstack-nova11:32
*** markvoelker has joined #openstack-nova11:32
*** trinaths1 has quit IRC11:33
*** ircuser-1 has quit IRC11:35
openstackgerritMerged openstack/nova master: trivial: Remove unnecessary function  https://review.openstack.org/48343311:35
*** gszasz has joined #openstack-nova11:37
*** udesale has quit IRC11:37
*** sapcc-bot has joined #openstack-nova11:39
*** sapcc-bot has quit IRC11:39
*** ociuhandu has quit IRC11:46
*** jjung has quit IRC11:47
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/47709111:48
*** ratailor has quit IRC11:50
*** sapcc-bot has joined #openstack-nova11:51
*** sapcc-bot has quit IRC11:51
*** litao__ has quit IRC11:52
*** mkucia has joined #openstack-nova11:55
cdentedleafe: yeah, I got pretty confused on what the right thing to do throughout 500 fix. It made think: here be edges and confusion11:55
*** smatzek has joined #openstack-nova11:56
*** markvoelker has quit IRC11:56
*** markvoelker has joined #openstack-nova11:57
*** sapcc-bot has joined #openstack-nova12:02
*** sapcc-bot has quit IRC12:02
*** takedakn has quit IRC12:03
*** fragatina has quit IRC12:05
*** vladikr has joined #openstack-nova12:06
*** ociuhandu has joined #openstack-nova12:10
*** tylerderosagrund has joined #openstack-nova12:12
*** ircuser-1 has joined #openstack-nova12:15
*** jistr|afk is now known as jistr12:17
*** tylerderosagrund has quit IRC12:24
*** tylerderosagrund has joined #openstack-nova12:24
*** ekuris_ has quit IRC12:25
*** ekuris has joined #openstack-nova12:26
*** MVenesio has joined #openstack-nova12:28
*** sapcc-bot has joined #openstack-nova12:30
*** sapcc-bot has quit IRC12:30
*** dtantsur has joined #openstack-nova12:33
*** jaypipes has joined #openstack-nova12:33
*** prateek has quit IRC12:33
*** edmondsw has joined #openstack-nova12:34
*** edmondsw has quit IRC12:34
*** edmondsw has joined #openstack-nova12:35
*** sapcc-bot has joined #openstack-nova12:36
*** sapcc-bot has quit IRC12:36
*** jjung has joined #openstack-nova12:37
*** kbaegis has joined #openstack-nova12:39
openstackgerritChris Dent proposed openstack/nova master: [placement] Update allocation-candidates.yaml for gabbi 1.35  https://review.openstack.org/48435612:39
cdentedleafe, jaypipes ^12:39
cdentmay be of interest12:39
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.rebuild_scheduled notification  https://review.openstack.org/47392912:42
*** mamandle has joined #openstack-nova12:42
*** jjung has quit IRC12:44
openstackgerritDmitri Plakhov proposed openstack/nova master: Added Oberbooking quota driver  https://review.openstack.org/48435912:44
*** lyan has joined #openstack-nova12:45
*** sapcc-bot has joined #openstack-nova12:47
*** sapcc-bot has quit IRC12:47
*** jjung has joined #openstack-nova12:47
*** BobBall has joined #openstack-nova12:47
*** efoley has joined #openstack-nova12:48
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.resize_prep notification  https://review.openstack.org/46508112:49
*** catintheroof has joined #openstack-nova12:51
*** udesale has joined #openstack-nova12:52
*** Jack_Iv has joined #openstack-nova12:52
*** prateek has joined #openstack-nova12:53
*** pchavva has joined #openstack-nova12:53
Jack_IvHi! could anyone suggest something? https://bugs.launchpad.net/nova/+bug/170478812:53
openstackLaunchpad bug 1704788 in OpenStack Compute (nova) "Hardcoded choices for nova scheduler driver" [Undecided,New]12:53
*** mtanino_ has joined #openstack-nova12:56
*** alexchadin has joined #openstack-nova12:57
*** Sara_ has joined #openstack-nova12:59
Sara_Hello all... I have booted a VM using "nova boot".. After the Vm is in active state.. I have done the following:13:00
Sara_virsh destroy <instance>13:00
Sara_virsh undefine "<instance>"13:00
*** gouthamr has joined #openstack-nova13:01
Sara_In nova list, I could see "VM in Shutoff" state13:01
Sara_The VM is not deleted from nova database and resources are not released..13:02
Sara_Is this the expected behaviour?13:02
bauzasSara_: it's not accepted13:02
*** fried_rice is now known as efried13:02
bauzasSara_: Nova doesn't support operators killing VMs directly13:03
bauzasinstead, you need to "nova delete" your instance13:03
*** edmondsw_ has joined #openstack-nova13:04
*** sapcc-bot has joined #openstack-nova13:04
*** sapcc-bot has quit IRC13:04
openstackgerritChris Dent proposed openstack/nova master: [placement] Update allocation-candidates.yaml for gabbi 1.35  https://review.openstack.org/48435613:04
Sara_bauzas_:  But what happens in the scenario where compute host is removed from the openstack setup and rejoined ?13:05
bauzasSara_: you can still delete the instances13:05
bauzasSara_: even if the compute is not there13:05
*** jwcroppe has quit IRC13:05
Sara_Does libvirt update the "undefine" event to nova?13:06
bauzasSara_: when the compute will be back, it will call the nova DB to know which instances are deleted, and then will destroy the libvirt VMs13:06
*** gcb has joined #openstack-nova13:06
efriedAnyone savvy on nova microversion support timeframes?  In particular, is pre-2.27 still supported?13:07
openstackgerritGábor Antal proposed openstack/nova master: Transform HostAPI-set_enabled notifications  https://review.openstack.org/48262213:07
*** edmondsw has quit IRC13:07
Sara_bauzas_: Does libvirt update the "undefine" event to nova? I want to get an update from libvirt to nova when the domain gets undefined..Is it supported in Mitaka?13:08
cdentefried: the removal of nova-net kind of broke the microversion contract (which is effectively that they are around for-ever)13:09
*** lucas-hungry is now known as lucasagomes13:09
*** baoli has joined #openstack-nova13:10
efriedcdent Okay, so we actually need to handle https://review.openstack.org/#/c/482746/10/keystoneauth1/session.py@539 this?13:10
cdenturgh. I don’t know, that’s a good question. Some combination of mordred, mriedem and sdague may have ideas13:11
*** acormier has joined #openstack-nova13:12
openstackgerritChris Dent proposed openstack/nova master: [placement] fix 500 error when allocating to bad class  https://review.openstack.org/48416213:13
sdagueefried: honestly, there probably should be special case code in there to handle the compute case13:13
sdagueif the point is to support what's out there13:13
efriedsdague And if I understand the docs right, we can just send both headers down and nova will pick the new one if it's post 2.27, yes?13:14
*** baoli has quit IRC13:14
sdagueefried: yep13:15
efriedk13:15
*** imacdonn has quit IRC13:16
*** sree has joined #openstack-nova13:16
*** sapcc-bot has joined #openstack-nova13:16
*** sapcc-bot has quit IRC13:16
*** mmehan has joined #openstack-nova13:16
openstackgerritMerged openstack/nova master: fix unshelve notification test instability  https://review.openstack.org/48398613:18
*** ekuris has quit IRC13:18
*** jwcroppe has joined #openstack-nova13:20
*** sapcc-bot has joined #openstack-nova13:21
*** sapcc-bot has quit IRC13:21
*** acormier has quit IRC13:21
*** zhurong has quit IRC13:22
*** scottda has joined #openstack-nova13:23
bauzasstephenfin: not sure you're there, or if someone can answer me this question, but where can I find upstream docs about PCI passthru and SR-IOV ?13:24
jaypipescdent: nice! +2 from me. :)13:25
cdentcoo, thanks13:25
*** lbragstad has quit IRC13:26
*** baoli has joined #openstack-nova13:26
*** udesale has quit IRC13:27
mordredcdent: reading13:28
*** gcb has quit IRC13:28
cdentmordred: I think sdague pretty much nailed it: compute needs a special case. Unclear about other services.13:28
efriedcdent mordred Comments in the review.13:29
mordredyah. agree - I don't think any other services made it out the door with that form, did they?13:29
bauzasSara_: like I said earlier, if you undefine a libvirt VM, Nova can just say that the instance is shutdown, because it can't find it13:30
bauzasSara_: there is no 'callback' from a specific virt driver for updating some internal Nova status about instances13:31
sdaguemordred: the only other one that might have was ironic13:31
bauzasSara_: that's just nova which periodically monitors host states wrt instances13:31
sdagueI think we got standardization before anything else adopted it13:31
*** sapcc-bot has joined #openstack-nova13:32
*** sapcc-bot has quit IRC13:32
bauzascdent: is there a dependent change for updating requirements to gabbi>=1.35 ?13:33
*** superdan is now known as dansmith13:33
cdentbauzas: it’s already merged13:33
bauzascdent: talking of https://review.openstack.org/#/c/484356/2/test-requirements.txt13:33
*** markvoelker_ has joined #openstack-nova13:33
cdenthttps://review.openstack.org/#/c/481665/13:33
*** mriedem has joined #openstack-nova13:33
*** markvoelker has quit IRC13:33
*** ekuris has joined #openstack-nova13:34
bauzascdent: oh ok, so I guess gabbi-1.35 supports legacy behaviour13:34
*** edmondsw has joined #openstack-nova13:34
cdentbauzas: yes, it follows semantic versioning13:34
bauzascdent: WAT? impossible :p13:35
cdentikr!13:35
bauzashehe13:35
bauzascdent: anyway, the gate works quite good today, so I guess you did your job correctly :p13:36
*** edmondsw_ has quit IRC13:36
*** ociuhandu has quit IRC13:36
*** ociuhandu has joined #openstack-nova13:36
cdentgabbi tests itself on gnocchi’s use of gabbi, I could probably add the same for nova’s use of gabbi too13:37
* cdent makes a note of it13:37
*** links has quit IRC13:37
*** Kris has joined #openstack-nova13:38
*** Kris has left #openstack-nova13:38
*** sapcc-bot has joined #openstack-nova13:38
*** sapcc-bot has quit IRC13:38
*** markvoelker_ has quit IRC13:38
*** markvoelker has joined #openstack-nova13:39
edleafeScheduler subteam meeting in 20 minutes in #openstack-meeting-alt13:40
efriedmordred Much as I hate to suggest further dirtying up sta, perhaps it should have legacy-microversion-header and legacy-microversion-header-before-version fields.13:42
mriedemgmann: alex_xu: was anyone going to post a change for fixing the docs issues in https://review.openstack.org/#/c/467999/ ? if not i could work on that today13:43
*** sapcc-bot has joined #openstack-nova13:43
*** sapcc-bot has quit IRC13:43
*** gcb has joined #openstack-nova13:44
*** udesale has joined #openstack-nova13:45
*** cdent_ has joined #openstack-nova13:45
efriedsdague mordred I can't even find anything about ironic that suggests microversion support *other* than via X-OpenStack-Ironic-API-Version13:46
efriedWas looking for a statement like this one that nova has: https://developer.openstack.org/api-ref/compute/#api-versions13:47
*** cdent has quit IRC13:47
*** cdent_ is now known as cdent13:47
sdagueefried: oh, maybe they predated and never fixed it13:47
*** xyang1 has joined #openstack-nova13:47
*** sapcc-bot has joined #openstack-nova13:48
*** sapcc-bot has quit IRC13:48
*** beekneemech is now known as bnemec13:48
*** gcb has quit IRC13:48
*** gcb has joined #openstack-nova13:49
*** yamamoto has quit IRC13:49
*** sree has quit IRC13:50
*** esberglu has joined #openstack-nova13:50
*** kylek3h has joined #openstack-nova13:50
*** ekuris has quit IRC13:50
*** felipemonteiro has joined #openstack-nova13:51
*** sbezverk has quit IRC13:51
alex_xumriedem: here is https://review.openstack.org/48317013:51
alex_xusorry, I forget to tell you the link13:51
*** sbezverk has joined #openstack-nova13:51
*** felipemonteiro__ has joined #openstack-nova13:53
*** stvnoyes has quit IRC13:53
*** sapcc-bot has joined #openstack-nova13:53
*** sapcc-bot has quit IRC13:53
*** crushil has joined #openstack-nova13:53
mordredefried: ugh13:53
efriedagree13:54
*** sapcc-bot has joined #openstack-nova13:54
*** sapcc-bot has quit IRC13:54
*** awaugama has joined #openstack-nova13:54
*** stvnoyes has joined #openstack-nova13:54
cdentmordred, efried: sorry not sorry13:55
efriedcdent I think you're off the hook - it was cmurphy brought it up.13:56
openstackgerritSteven Webster proposed openstack/nova master: Fix instance evacuation with PCI devices  https://review.openstack.org/48438113:56
*** damien_r has left #openstack-nova13:56
mordredefried: another option is to not support legacy headers in the ksa microversion helper method, since sending this at all is a new ksa feature in the first place - and we can have python-novaclient and python-ironicclient also add the old headers if they want to13:56
cdentefried: I was the person who proposed the new form13:56
*** felipemonteiro has quit IRC13:56
efriedcdent Oh, the generic OpenStack-API-Version ?13:56
cdentmordred: I’d like that, but is that fair?13:56
mordredI'm not crazy about either option tbh - just bringing it up13:56
cdentefried: yes\13:56
mordredwell - my concern is that there's no way for the user to know then that get('/servers', microversion='2.13') isn't going to work13:57
efriedcdent Yeah, it's the better way of doing it.  You just got there too late.13:57
jangutterbauzas: do you still need docs about PCI passthrough and SR-IOV?13:57
cdentstory of my life13:57
efriedmordred Right, python-*client isn't necessarily the only API consumer, right?13:57
mordredcdent: the original form is basically unworkable in general - so thank you very much for making it the change13:57
mordredefried: exactly13:58
efriedmordred So we hardcode it in ksa... or make it part of sta.13:58
efriedCause I really don't like that other option - making "consumers" know about this quirk.13:58
*** gbarros has quit IRC13:58
*** zhurong has joined #openstack-nova13:59
bauzasjangutter: I'd appreciate, yes13:59
*** zhurong has quit IRC13:59
bauzasjangutter: I mean, I can find other docs, and I know most of that13:59
bauzasjangutter: but we had some upstream docs that give me now 404s and I'd like to make sure we still have those somewhere14:00
jangutterbauzas: I always refer back to some very old IOMMU/VFIO/PCI-ASSIGN presentation waay back when.14:00
bauzasI know we had some admin-guide doc14:01
*** gbarros has joined #openstack-nova14:01
bauzasbut those are giving me 404s because of the doc migration14:01
mordredefried: yah. me either. although maybe the answer for today is to not add the helper method in this release since this is getting in to a kind-of ugly place and needs some thought/design14:01
jangutterbauzas: aah, end-user stuff.14:01
*** sree has joined #openstack-nova14:01
edleafecdent: bauzas: scheduler meeting now running14:01
bauzasedleafe: snap, caught by the police :p14:02
mordredefried: or - we make sure that the "add microversion helper parameter" bit to the end of the stack so that if we can't all agree on the approach we don't block anything else14:02
jangutterbauzas: https://wiki.openstack.org/wiki/SR-IOV-Passthrough-For-Networking <--- very stale.14:02
efriedmordred Probably at least that last thing.14:03
bauzasasettle: hola, unsure if you can answer my question14:03
efriedmordred Cause I imagine it may take a little while to figure out all the special cases, whatever approach we land on.14:04
jangutterbauzas: and the one for Ocata: https://docs.openstack.org/ocata/networking-guide/config-sriov.html <--- something like this, or do you want more low-level?14:04
bauzasasettle: but I can see a 404 on http://docs.openstack.org/admin-guide/compute-pci-passthrough.html14:04
bauzasjangutter: yeah, that will work out14:04
mordredefried: yes - and I think we need to get a ksa release cut that hasall the other things this week so that it makes the cutoff and you can use it in nova14:05
bauzasjangutter: it's not really for my knowledge, it's rather more about providing guidance to others14:05
efriedmordred Yes, and I'm a tad worried about that.  Working hard on getting https://review.openstack.org/#/c/483604/ polished off, cause that affects the initial APIs of the new methods.14:05
efriedmordred Also concerned about getting that rename (min/max_version => min/max_endpoint_version) in place for same reason.14:06
*** lucasxu has joined #openstack-nova14:06
mordredefried: ++14:06
efriedmordred Prolly should move this conversation to -dev/-sdks14:06
*** sree_ has joined #openstack-nova14:07
*** sree_ is now known as Guest3563614:07
*** yamamoto has joined #openstack-nova14:09
*** yamamoto has quit IRC14:09
*** rnoriega_ has joined #openstack-nova14:09
*** sree has quit IRC14:10
*** sapcc-bot has joined #openstack-nova14:10
*** sapcc-bot has quit IRC14:10
*** hongbin has joined #openstack-nova14:11
*** alexchadin has quit IRC14:11
openstackgerritJay Pipes proposed openstack/nova master: claim resources in placement API during schedule()  https://review.openstack.org/48356614:12
openstackgerritJay Pipes proposed openstack/nova master: add dict of allocation requests to select_dests()  https://review.openstack.org/48356514:12
*** adisky__ has quit IRC14:12
jaypipesedleafe: done ^14:12
*** Jack_Iv has quit IRC14:12
edleafejaypipes: gracias14:12
openstackgerritJan Gutter proposed openstack/nova master: Netronome SmartNIC Enablement  https://review.openstack.org/48345914:13
*** rnoriega_ has quit IRC14:16
*** burt has joined #openstack-nova14:19
*** acormier has joined #openstack-nova14:23
*** udesale has quit IRC14:24
*** jwcroppe has quit IRC14:28
asettlebauzas: uno momento, in a meeting :) will look shortly14:28
asettlebauzas: admin guide has been migrated14:29
bauzask14:29
asettleY'all need to migrate the Comptue section of the admin guide into the nova repo14:29
*** jwcroppe has joined #openstack-nova14:29
*** lbragstad has joined #openstack-nova14:29
*** marst has quit IRC14:30
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Migration from 'ip' commands to pyroute2  https://review.openstack.org/48438614:31
*** sapcc-bot has joined #openstack-nova14:33
*** sapcc-bot has quit IRC14:33
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Migration from 'ip' commands to pyroute2  https://review.openstack.org/48438614:34
ralonsohstephenfin: hi! A present: https://review.openstack.org/#/c/484386/14:35
ralonsohstephenfin: BTW. I'm running out of time. Can you take a look at https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/enable-sriov-nic-features?14:36
mdrabedansmith: I think https://review.openstack.org/#/c/482620 should be bueno14:36
*** gyee has joined #openstack-nova14:37
dansmithmdrabe: indeed, thanks14:37
openstackgerritArtem Tiumentcev proposed openstack/nova master: Add root and ephemeral disk quota  https://review.openstack.org/48439014:39
*** esberglu has quit IRC14:40
*** yamamoto has joined #openstack-nova14:40
*** mtanino_ has quit IRC14:41
openstackgerritArtem Tiumentcev proposed openstack/python-novaclient master: Add 'local_gb' disk quota support  https://review.openstack.org/48439114:43
*** yamamoto has quit IRC14:46
*** mtanino_ has joined #openstack-nova14:46
*** marst has joined #openstack-nova14:46
*** psachin has quit IRC14:48
*** chyka has joined #openstack-nova14:52
*** armax has joined #openstack-nova14:53
*** mamandle has quit IRC14:53
*** esberglu has joined #openstack-nova14:55
*** sapcc-bot has joined #openstack-nova14:56
*** sapcc-bot has quit IRC14:56
*** esberglu has quit IRC14:56
*** esberglu has joined #openstack-nova14:57
*** jwcroppe has quit IRC14:58
*** jjung has quit IRC14:58
*** armax has quit IRC14:58
*** yamahata has joined #openstack-nova14:59
*** Guest35636 has quit IRC14:59
*** lucasxu has quit IRC15:00
*** cdent has quit IRC15:02
openstackgerritSteve Noyes proposed openstack/nova master: update live migration to use v3 cinder api  https://review.openstack.org/46398715:02
*** sapcc-bot has joined #openstack-nova15:03
*** armax has joined #openstack-nova15:06
*** gyee has quit IRC15:06
*** gyee has joined #openstack-nova15:08
mriedemsdague: can you go through this new style cinder attachments swap volume change? https://review.openstack.org/#/c/456971/ - i'm good with it and jay is +2,15:09
mriedemsdague: we really need another core to help review the new style cinder attachments stuff, so if you could help here this week it'd be really good15:09
*** andreas_s has quit IRC15:10
mriedemkeep in mind, everything is keyed off the bdm.attachment_id field being set - which doesn't happen until the very last patch where we turn this all on - which only happens when (1) cinder is new enough to provide 3.27 and (2) all computes are upgraded15:10
*** Oku_OS is now known as Oku_OS-away15:11
gibimriedem: hi! I have a fix up for the volume_swap notification test instability https://review.openstack.org/#/c/484288/15:11
*** mamandle has joined #openstack-nova15:11
mriedemgibi: thanks, +215:13
*** ociuhandu has quit IRC15:14
*** trinaths has joined #openstack-nova15:20
gibimriedem: thanks15:21
openstackgerritArtem Tiumentcev proposed openstack/python-novaclient master: Add 'local_gb' disk quota support  https://review.openstack.org/48439115:23
mriedemsfinucan: i'm debugging a fun issue with NUMATopologyLimits not getting serialized in the scheduler with a new oslo.serialization change15:25
mriedemhttp://logs.openstack.org/87/482287/6/check/gate-cross-nova-func-ubuntu-xenial/a645ce8/console.html#_2017-07-16_14_02_02_64938415:25
mriedemthat's due to this change in oslo.serialization https://review.openstack.org/#/c/327564/15:25
mriedemwhich is failing here: https://review.openstack.org/#/c/482287/15:26
*** lucasxu has joined #openstack-nova15:27
mriedemi'm trying to sort out where this is getting serializaed15:27
mriedem*serialized15:27
mriedemobviously the select_destinations call15:27
*** stvnoyes has quit IRC15:28
mriedemoh i think i see,15:28
mriedemthe request spec is serialized, and contains numa_topology which comes from the HostState object:15:28
mriedemhost_state.limits['numa_topology'] = limits15:29
mriedemset via the NUMATopologyFilter15:29
mriedemstill not sure why that would fail to serialize15:31
*** annegentle has joined #openstack-nova15:31
jangutternewbie question about the gate: is there a way to inform it that the unit tests require an updated os-vif?15:32
mriedemjangutter: depends-on15:33
mriedembuth,15:33
mriedem*but,15:33
*** sree has joined #openstack-nova15:33
mriedemyou have to get the changes merged and released in os-vif first15:33
mriedemyou can't depends-on a source change in a library w/o it being released first in most jobs15:33
janguttermriedem: yah, thought as much.15:33
*** ociuhandu has joined #openstack-nova15:35
*** jdillaman has joined #openstack-nova15:35
janguttermriedem: funnily enough, the non-test code seem to load the patched os-vif.15:36
*** annegentle has quit IRC15:36
*** trinaths has quit IRC15:36
janguttermriedem: I presume it's because tox creates a sandbox.15:36
*** jwcroppe has joined #openstack-nova15:37
*** sree has quit IRC15:37
*** annegentle has joined #openstack-nova15:38
mriedemi'm not really understanding what you're saying15:40
*** david-lyle has joined #openstack-nova15:40
mriedemit's all going to run in a tox venv with dependencies using the versions from upper-constraints15:40
mriedemhttps://github.com/openstack/requirements/blob/master/upper-constraints.txt15:40
*** stvnoyes has joined #openstack-nova15:40
mriedemso until the change in os-vif is released and in upper-constraints, then the tox venv isn't using it15:41
mriedemat least in the CI system15:41
mriedemso either your local venv is hacked or it's not working the way you think15:41
*** MVenesio has quit IRC15:42
janguttermriedem: Yeah, I hacked my local venv (by running install on os-vif), but only for the unit tests.15:42
*** ckopper has joined #openstack-nova15:42
mriedemsfinucan: this is what we get right before select_destinations blows up http://paste.openstack.org/show/615631/15:43
sdaguemriedem: will look15:44
mriedemsdague: thanks, poke me with questions15:44
*** aloga_ has joined #openstack-nova15:46
bauzasjaypipes: dansmith: mriedem: sorry, was off last friday, but just saw https://review.openstack.org/#/c/483564/215:46
bauzasjaypipes: dansmith: mriedem: so we're now passing a full list of weighed hosts to _schedule() even if don't really use those15:47
bauzasjaypipes: dansmith: mriedem: I get of course why we need to do such in later changes but I do wonder of the performance penalty15:47
mriedemhuh?15:48
bauzasjaypipes: dansmith: mriedem: the performance penalty being for example a list of 1000 nodes15:48
mriedemwhat's getting passed to _schedule that you are worried about?15:48
bauzasgiven we don't pass an iterator but a full ist15:48
bauzaslist15:48
bauzasmriedem: https://review.openstack.org/#/c/483564/2/nova/scheduler/filter_scheduler.py@15015:48
bauzasthat's a list, not an iterator15:48
mriedemyou mean not what's passed to _schedule, you mean what's passed from _get_sorted_hosts and used in _schedule15:49
bauzasso we literally build in memory an object that could be large just for the purpose of selecting only the first item15:49
bauzasyup, that15:49
bauzasI'd frankly be more happy if that was an iterator15:49
bauzasbut I could be picky15:50
bauzasjust raising my concern out loud15:50
bauzasnot worth a revert of course15:50
mriedemso push a change to do that15:50
mriedemor leave a comment in the patch15:50
bauzasmriedem: I can workout a patch15:50
*** ociuhandu has quit IRC15:50
*** ociuhandu has joined #openstack-nova15:52
bauzasjaypipes: mriedem: oh looks like https://review.openstack.org/#/c/483564/2 will not land because of a functional test issue15:54
mriedemi see the "# Note: remember, we are using an iterator here." on L93 but which variable is the iterator? is it 'hosts' returned from _get_all_host_states ?15:54
mriedembecause i think that returns a tuple15:54
*** aloga_ has quit IRC15:54
mriedemHostManager.get_host_states_by_uuids calls _get_host_states which returns a tuple15:55
*** mkucia has quit IRC15:55
mriedemI think get_filtered_hosts is what converts to an iterator15:56
mriedemhosts = six.itervalues(name_to_cls_map)15:56
*** aloga_ has joined #openstack-nova15:56
bauzasmriedem: the whole filters/weighers accepts an iterator15:56
bauzasAFAIR15:56
mriedemand passes that to filter_handler.get_filtered_objects15:56
bauzasmriedem: yup15:56
mriedemand ^ calls the filters15:57
bauzasmriedem: but the thing is that we build a list once we get the weighed hosts15:57
*** hoonetorg has quit IRC15:57
mriedemdoesn't get_weighed_hosts return a sorted list?15:58
*** rajathagasthya has joined #openstack-nova15:59
mriedemew now i see that WeighedObject is a wrapper around a HostState - wonder why someone used a wrapper rather than subclass HostState with a weight field?15:59
mriedemWeighedHostState(HostState)15:59
mriedemthat's why we have that ugly host.obj.foo all over15:59
*** fragatina has joined #openstack-nova16:00
mriedemanyway, i'm a bit lost in here wrt the iterator16:00
bauzasyeah, seems like it's uglier than I thought16:01
mriedembauzas: the new _get_sorted_hosts method is still calling host_manager.get_filtered_hosts which still returns an iterator16:01
bauzasmriedem: I was remembering we were using iterators, but looks like our filtering/weighing helpers work with lists...16:01
mriedemand that iterator is still passed to host_manager.get_weighed_hosts16:01
mriedemand weighed_hosts comes back as a sorted list16:01
mriedemnot an iterator16:02
mriedemfrom what i can tell16:02
bauzasyup16:02
mriedemand by the time you get to chosen_host = hosts[0] it's really no different from before16:02
mriedemso seems like something that doesn't matter to this change16:02
bauzasI was just concerned by the new code taking more memory than previously, but it's just honestly not the case16:02
bauzasyup16:02
bauzasbecause we did work with iterators but once you enter filtering/weighting you're back to full lists of hosts anyway16:03
* bauzas shrugs16:03
*** armax has quit IRC16:03
bauzasanyway, I'm now looking at why that patch failed16:03
bauzasmmmmpf http://logs.openstack.org/64/483564/2/gate/gate-nova-tox-functional-ubuntu-xenial/0f4534d/console.html#_2017-07-17_15_20_35_76989816:05
*** rajathagasthya has quit IRC16:06
*** rajathagasthya has joined #openstack-nova16:07
*** cdent has joined #openstack-nova16:07
*** tylerderosagrund has quit IRC16:08
*** tylerderosagrund has joined #openstack-nova16:08
mriedemshouldn't that also be running through other filters?16:09
jaypipesmriedem, bauzas: that whole comment about it being an iterator is nonsense. As soon as it enters the filter host manager, it converts it all to a list: https://github.com/openstack/nova/blob/master/nova/filters.py#L6916:09
mriedemlike ServerGroupAntiAffinityFilter16:09
mordredmriedem, sdague: if a client sends the legacy microversion header to a new nova, that's not an error, right? and if it sends both, that's also acceptable?16:09
jaypipesmriedem: it was always trying to be too clever for its own good, IMHO.16:09
bauzasjaypipes: yup, just saw that, see above the IRC discussion with matt16:10
bauzasjaypipes: honestly, I'm not surprised16:10
sdaguemordred: correct, either are accepted16:10
jaypipesmriedem: btw, I was trying to get rid of the host.obj foo nonsense.16:10
sdagueas are both, I think when both we take the standard one as gospel16:10
mordredsdague: awesome. I believe I have a general solution to our earlier problem16:10
bauzasjaypipes: someone probably did care of passing iterators, but someone else probably messed it up by just using lists afterwards16:11
sdaguecdent: I think you wrote the acceptor code, right?16:11
mriedemjaypipes: yeah i noted that in the commit message16:11
cdentsdague: yeah, it’s happy with both, prefers the newer one16:11
mriedemless turducken is good16:11
mordredcdent: sweet16:11
bauzasjaypipes: anyway, it's not super critical, but something is : your patch is getting -1 because http://logs.openstack.org/64/483564/2/gate/gate-nova-tox-functional-ubuntu-xenial/0f4534d/console.html#_2017-07-17_15_20_35_76989816:11
*** lyan has quit IRC16:11
cdentit always sends both back in the response too16:11
jaypipesbauzas: I can't reproduce that locally :(16:12
openstackgerritzhang zhenzhong proposed openstack/nova master: fix rp registration failed while placement endpoint created after compute start  https://review.openstack.org/48346016:12
bauzasjaypipes: I was just about to run it :)16:13
mriedemhttp://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22%2C%20in%20test_boot_servers_with_soft_anti_affinity_one_available_host%5C%22%20AND%20tags%3A%5C%22console%5C%22&from=7d16:13
bauzasmy child at home is killing my work because of netflix eating all the poor internet bandwidth FWIW16:13
bauzasplex FTW16:14
bauzasgit pull is taking more than 1 min16:14
*** prateek has quit IRC16:14
mriedemfwiw, that test also failed in https://review.openstack.org/#/c/482288/16:14
mriedemwhich would be totally unrelated16:14
*** aloga_ has quit IRC16:15
bauzasjaypipes: did you run the whole servergroup functional test group, or just the one failing?16:15
jaypipesbauzas: both16:15
bauzask16:15
bauzaslast time I had such problem, it was because of the fixture16:16
bauzasbut I was able to reproduce it16:16
*** cdent has quit IRC16:17
*** Apoorva has joined #openstack-nova16:18
*** rcernin_ has quit IRC16:19
mriedemso the test creates a soft-anti-affinity server group and kills the 2nd compute service so there should only be 116:19
mriedemsince the 'soft' means it will go to the same host if there is only 1 available16:19
sdaguemriedem: you have an opinion on this - https://review.openstack.org/#/c/215385/21/nova/virt/ironic/client_wrapper.py - I thought we wanted N-1 compat16:19
*** fragatina has quit IRC16:20
mriedemsdague: that's basically how it's always worked with nova + ironic as far as i know; yes it's not what we do with anything else16:21
sdaguemriedem: ok, well we definitely need upgrade stanza in the release notes then16:21
mriedemi raised a similar concern in another nova ironic change,16:22
mriedemhttps://review.openstack.org/#/c/419975/11/releasenotes/notes/add-interface-attach-detach-support-in-ironic-cb2bf11f3875350a.yaml@416:22
mriedembasically they implemented something in a driver and then started claiming support for hot plugging interfaces in nova16:23
*** derekh has quit IRC16:23
mriedemmy argument was, what if my ironic is from ocata and i upgrade nova, and was told, that doesn't happen because everyone knows to upgrade ironic first16:23
sdaguemriedem: so... I'm extremely suspicious of "everybody knows"16:23
sdagueI'm fine with it being policy, but we still need upgrade notes16:24
mriedemyeah they added one later here https://review.openstack.org/#/c/419975/17/releasenotes/notes/add-interface-attach-detach-support-in-ironic-cb2bf11f3875350a.yaml16:24
sdagueI'd be more happy with an N-1 policy, but we do need to communicate it every time it would trip people up16:25
*** kfarr has joined #openstack-nova16:26
mriedemi don't disagree16:26
*** tylerder_ has joined #openstack-nova16:28
mriedemjaypipes: bauzas: i have an idea on that failure16:29
mriedemthe functional test failure that is16:29
jaypipesyeah?16:30
bauzas?16:30
mriedemthe test setup creates 2 compute services/nodes/hosts16:30
mriedemyou can see the 2nd host CN created here http://logs.openstack.org/64/483564/2/gate/gate-nova-tox-functional-ubuntu-xenial/0f4534d/console.html#_2017-07-17_15_20_35_76800016:30
mriedemwith uuid a945d18d-e6b6-4fd5-8b3b-e50c973a1ff916:30
mriedemand it's posted into placement16:30
mriedemnow the purpose of the test is to restrict to 1 host16:30
mriedemso16:30
mriedemself.compute2.kill()16:30
mriedemHOWEVER16:31
mriedemfilter scheduler is getting it's hosties from placement now16:31
mriedemso that RP is still floating around in placement16:31
*** ralonsoh has quit IRC16:31
mriedemyou can see the scheduler do it's thing here16:31
mriedemGET /allocation_candidates?resources=DISK_GB%3A20%2CMEMORY_MB%3A2048%2CVCPU%3A1 HTTP/1.1" status: 200 len: 992 time: 0.052165016:31
mriedemand then16:32
mriedemNo compute service record found for host host216:32
mriedembecause the test killed it16:32
mriedemFilter RetryFilter returned 0 hosts16:32
*** tylerderosagrund has quit IRC16:32
mriedemwhat i don't understand is why it doesn't try the other compute host16:32
mriedemso one easy hack is deleting the host2 RP from test_boot_servers_with_soft_anti_affinity_one_available_host - but that's pretty hacky, it seems odd that the filtering wouldn't check the other host16:34
jaypipesmriedem: it's also odd that this test fails randomly.16:35
*** annegentle has quit IRC16:37
*** gszasz has quit IRC16:38
mriedemyeah so we get this https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L67416:39
mriedemso _get_host_states should return 1 host16:39
*** lyan has joined #openstack-nova16:39
*** swamireddy has quit IRC16:40
*** knikolla has quit IRC16:40
*** hoonetorg has joined #openstack-nova16:41
*** sridharg has quit IRC16:41
*** annegentle has joined #openstack-nova16:41
*** spotz has quit IRC16:42
*** rhagarty has quit IRC16:43
*** spotz has joined #openstack-nova16:45
mriedemmy only guess is for whatever reason the retry is disabled when the test runs16:45
mriedemnvm that doesn't make sense16:46
jaypipesjust failed that test again in the isolate get_sorted_hosts() patch :(16:46
*** yamahata has quit IRC16:46
mriedemjaypipes: yeah that's the one bauzas was talking about16:46
mriedemit failed in the gate16:46
*** rhagarty has joined #openstack-nova16:47
*** lucasagomes is now known as lucas-afk16:47
mriedemman you really need debug output when this fails16:48
mriedemand why wouldn't we see this in the logs? https://github.com/openstack/nova/blob/master/nova/scheduler/filters/retry_filter.py#L4716:49
jaypipesmriedem: anything in the scheduler needs DEBUG output because of the policy of only doing DEBUG level logging :(16:49
mriedemjaypipes: heh yeah i know16:49
jaypipesmriedem: because it was never passed any hosts.16:49
jaypipesmriedem: if you look at the log failures, you see REtryFilter(0 hosts)16:50
mriedemRetryFilter: (start: 0, end: 0) ?16:50
jaypipesyeah16:50
*** vladikr has quit IRC16:50
mriedemok, so the first compute node never came back from GET /allocation_candidates probably16:51
mriedemhttp://logs.openstack.org/64/483564/2/gate/gate-nova-tox-functional-ubuntu-xenial/0f4534d/console.html#_2017-07-17_15_20_35_76579116:53
mriedemthat's when the first compute host starts up16:53
*** ircuser-1 has quit IRC16:53
mriedemand then "Compute_service record created for compute:fake-mini"16:53
mriedemAnother thread already created a resource provider with the UUID 3694a10a-e435-4400-83aa-7d6ade1fc866. Grabbing that record from the placement API.16:54
mriedemi don't know what 3694a10a-e435-4400-83aa-7d6ade1fc866 is16:54
mriedemi thought the first compute RP should be e9e74898-4332-4e50-a4c6-1ad75dd295e516:54
*** hoonetorg has quit IRC16:54
mriedemhmm wtf GET /resource_providers/3694a10a-e435-4400-83aa-7d6ade1fc866" status: 40416:55
mriedemUnable to refresh my resource provider record16:55
*** fragatina has joined #openstack-nova16:56
mriedemso whatever happened, it never posted inventory for the first compute host,16:56
mriedemwhich is why it didn't come back in GET /allocation_candidates16:56
*** ociuhandu has quit IRC16:56
mriedemseems like another case of the wsgi eventlet weirdness16:56
mriedemcrossing streams16:56
mriedemon a side note, it's silly that we go through all the work of calling down to get_filtered_objects when the original list we got back from GET /allocation_candidates is empty to begin with16:58
mriedemah yes jay's todo: "Really, we should raise NoValidHosts here"16:59
jaypipestada.16:59
*** vladikr has joined #openstack-nova16:59
mriedemso i think the failure is totally unrelated16:59
*** mtanino_ has quit IRC17:00
*** ircuser-1 has joined #openstack-nova17:02
*** hoonetorg has joined #openstack-nova17:03
mriedemi rechecked and left a big ass summary of the failure17:03
*** fragatina has quit IRC17:04
*** tbachman has quit IRC17:05
*** mvk has quit IRC17:06
*** tbachman has joined #openstack-nova17:09
mriedemaha 10 hours later i think i figured out the ValueError: Cannot convert NUMATopologyLimits(cpu_allocation_ratio=16.0,ram_allocation_ratio=1.5) to primitive17:10
mriedemand is a real bug it looks like17:10
*** armax has joined #openstack-nova17:12
*** yamahata has joined #openstack-nova17:12
mriedemoooo and one mr jay introduced :)17:13
mriedemi guess our numa 3rd party ci isn't running?17:13
jaypipesmriedem: in, like, how long? :)17:13
mriedemhttps://review.openstack.org/#/c/478291/1/nova/scheduler/manager.py17:13
mriedemL4817:13
mriedemthat's not a primitive17:13
mriedemso when passing to jsonutils.to_primitive17:14
mriedemkablammo17:14
mriedemin the latest oslo.serialization17:14
mriedemhttp://logs.openstack.org/87/482287/6/check/gate-cross-nova-func-ubuntu-xenial/a645ce8/console.html#_2017-07-16_14_02_02_64938417:14
mriedembut not to worry,17:14
jaypipesmmm17:14
mriedemi've wrangled the oslo team to blacklist that version because it was breaking us, because of the bug :)17:14
mriedemi'll push a fix17:14
jaypipesk, thanks matt17:15
*** tylerder_ has quit IRC17:15
*** tylerderosagrund has joined #openstack-nova17:16
*** chyka has quit IRC17:16
*** chyka has joined #openstack-nova17:17
*** harlowja has joined #openstack-nova17:17
mriedemsean-k-mooney: do you know if anyone is running the intel NUMA 3rd party ci anymore?17:17
*** imacdonn has joined #openstack-nova17:21
*** imacdonn_ has joined #openstack-nova17:21
mriedemfwiw this could have always been busted https://review.openstack.org/#/c/478291/1/nova/scheduler/filter_scheduler.py17:21
mriedemi don't see any fancy conversion before17:21
*** imacdonn_ has quit IRC17:22
*** fragatina has joined #openstack-nova17:23
*** sshwarts has quit IRC17:26
openstackgerritMerged openstack/nova master: Fix the releasenote and api-ref for quota-class API  https://review.openstack.org/48317017:28
*** tylerderosagrund has quit IRC17:30
*** tbachman has quit IRC17:31
jaypipesalex_xu: https://review.openstack.org/#/c/480379/ reviewed. couple tiny nits. if you can fix those up and re-push, I'm +2.17:31
*** tbachman has joined #openstack-nova17:31
*** yamahata has quit IRC17:33
*** hoonetorg has quit IRC17:34
*** yamahata has joined #openstack-nova17:34
*** prateek has joined #openstack-nova17:34
*** mvk has joined #openstack-nova17:37
*** sambetts is now known as sambetts|afk17:38
*** efoley has quit IRC17:38
openstackgerritGábor Antal proposed openstack/nova master: Transform servergroup.create notification  https://review.openstack.org/45402317:39
openstackgerritGábor Antal proposed openstack/nova master: Transform servergroup.delete notification  https://review.openstack.org/48390217:39
*** MVenesio has joined #openstack-nova17:39
*** tylerderosagrund has joined #openstack-nova17:44
*** kristia__ has quit IRC17:49
*** kristian__ has joined #openstack-nova17:50
*** tbachman has quit IRC17:51
*** tbachman has joined #openstack-nova17:52
*** hoonetorg has joined #openstack-nova17:54
openstackgerritGábor Antal proposed openstack/nova master: Transform servergroup.delete notification  https://review.openstack.org/48390217:54
*** kristian__ has quit IRC17:54
*** jangutter has quit IRC17:55
*** jangutter has joined #openstack-nova17:55
openstackgerritGábor Antal proposed openstack/nova master: Transform servergroup.create notification  https://review.openstack.org/45402317:59
*** hoonetorg has quit IRC17:59
openstackgerritGábor Antal proposed openstack/nova master: Transform servergroup.delete notification  https://review.openstack.org/48390217:59
*** lpetrut_ has quit IRC18:00
*** tesseract has quit IRC18:01
openstackgerritMatt Riedemann proposed openstack/nova master: Convert HostState.limits objcets to primitives  https://review.openstack.org/48443918:03
mriedemdansmith: sfinucan: ^ i'm going to need someone more familiar with the numa limits stuff to look at that18:03
mriedemlike sahid or vladikr?18:03
mriedemi don't know how this was actually working before18:03
openstackgerritMatt Riedemann proposed openstack/nova master: Convert HostState.limits objects to primitives  https://review.openstack.org/48443918:04
mriedemwas conductor serializing that response before for us so the NUMATopologyLimits was getting serialized by conductor?18:05
*** dtantsur is now known as dtantsur|afk18:07
*** gbarros has quit IRC18:07
*** felipemonteiro__ has quit IRC18:07
oomichialex_xu: most patches are approved on https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/api-no-more-extensions-pike18:07
oomichialex_xu: need to update the remaining for conflicts18:08
*** gbarros has joined #openstack-nova18:09
*** chyka has quit IRC18:15
*** chyka has joined #openstack-nova18:16
*** Sukhdev has joined #openstack-nova18:24
*** rcernin has joined #openstack-nova18:26
*** fragatina has quit IRC18:28
*** fragatina has joined #openstack-nova18:28
openstackgerritMatt Riedemann proposed openstack/nova master: Log compute node uuid when the record is created  https://review.openstack.org/48444718:31
*** kristian__ has joined #openstack-nova18:37
*** rnoriega has quit IRC18:38
*** vishwanathj has quit IRC18:38
*** vishwanathj has joined #openstack-nova18:39
*** ociuhandu has joined #openstack-nova18:39
*** felipemonteiro has joined #openstack-nova18:39
*** felipemonteiro_ has joined #openstack-nova18:40
*** lpetrut_ has joined #openstack-nova18:41
*** kristian__ has quit IRC18:42
*** felipemonteiro has quit IRC18:44
*** jbernard has quit IRC18:45
*** jbernard has joined #openstack-nova18:45
*** lpetrut_ has quit IRC18:47
*** gjayavelu has joined #openstack-nova18:47
*** ftersin has quit IRC18:49
*** rnoriega has joined #openstack-nova18:49
mriedemdansmith: sfinucan: nvm i think i know how that worked before - jsonutils.to_primitive() wouldn't error out, but wouldn't convert the NUMATopologyLimits either, so it would go through NovaObjectSerializer to do the serialize/deserialize work on the object18:52
mriedemthe new change in to_primitive to raise ValueError is what blew it all up because it would try to serialize before hitting the NovaObjectSerializer18:53
dansmithsorry I missed the above ping,18:53
dansmithbut I assume I can ignore now18:53
mriedemvalidation of my understanding, and feelings, is always appreciated, but yeah you can ignore probably18:53
dansmithjsonutils.to_primitive used to fail with "recursion error" or something like that when it hit a non-serializeable object18:54
dansmithmriedem: what change introduced this?18:56
dansmithoh I see, in oslo18:56
mriedemyeah that's what the oslo.serialization change was trying to prevent, the infinite recursion18:58
mriedemby just failing fast18:58
dansmithmriedem: so the thing I'm concerned about is: are we returning something that is only using jsonutils for serialization?18:59
dansmithour object serializer doesn't go hunting in an iterable for any instance of an object to serialize18:59
*** avolkov has quit IRC19:00
dansmithdoesn't go arbitrarily deep, I mean19:00
mriedemhmm yeah because that's what i was looking at with https://github.com/openstack/nova/blob/master/nova/objects/base.py#L24419:01
mriedemso it goes 1 level?19:01
mriedemwell, actually,19:01
mriedemso this would be a like {'limits': {'numa_topology': object}}19:01
mriedemso that should hit it ^19:01
dansmithwhere is this exactly?19:02
dansmithis this in a returned value?19:02
mriedemyeah19:02
mriedemhttps://review.openstack.org/#/c/484439/2/nova/scheduler/manager.py@14019:02
*** nicolasbock has quit IRC19:03
mriedemcreates a list of these dicts, and each dict will have a 'limits' key that maps to a dict, and that thing has the numa_topology_limits=NUMATopologyLimits object19:03
mriedemmap19:03
openstackgerritMatt Riedemann proposed openstack/nova master: Optimize creating servers with tags  https://review.openstack.org/48417019:05
dansmithI guess it will go pretty deep19:05
dansmithhad to re-read that a couple times19:05
mriedemyeah so i think that's how it wasn't blowing up before19:05
mriedemwell, and was actually serializing the thing19:05
*** nicolasbock has joined #openstack-nova19:05
dansmithyeah19:06
mriedemthe fact jsonutils.to_primitive ignored it was to our advantage19:06
*** tylerder_ has joined #openstack-nova19:06
dansmithyeah19:06
mriedemthat change is also being reverted https://review.openstack.org/#/c/484396/19:06
mriedemb/c i pointed out that releasing that under a patch release versoin w/o any prior warning <2 weeks to FF is likely to break other projects19:07
dansmithheh19:07
dansmithyeah I guess so19:07
mriedemwe only had prior warning b/c the requirements changes gate on nova unit/functional tests19:07
*** tylerderosagrund has quit IRC19:09
*** tylerder_ has quit IRC19:10
*** lpetrut_ has joined #openstack-nova19:11
*** mtanino_ has joined #openstack-nova19:13
*** knikolla has joined #openstack-nova19:14
*** kristian__ has joined #openstack-nova19:25
mriedemoomichi: i'm +2 again on https://review.openstack.org/#/c/479774/ after your comments19:25
*** kbaegis has quit IRC19:26
*** harlowja has quit IRC19:32
*** smatzek has quit IRC19:35
*** Ctesias has joined #openstack-nova19:39
Ctesiashi.  I'm digging around in nova trying to figure out why my nova_override.conf file isn't overriding nova defaults and I'm curious where `import nova.conf` and `CONF = nova.conf.CONF` are coming from because I don't see those being defined anywhere19:39
mriedemCtesias: for nova-compute the options are loaded up here https://github.com/openstack/nova/blob/master/nova/cmd/compute.py#L4219:41
mriedemsimilar pattern for the other service entry points19:41
mriedemhttps://github.com/openstack/nova/blob/master/nova/config.py#L4919:41
CtesiasThanks for the pointer.19:43
CtesiasIs putting an override_nova.conf file in the config directory the right way to get overrides into the nova config?19:43
mriedemi've never used nova_override.conf19:44
mriedemso if it works, it's some magic i'm not aware of19:44
*** dave-mccowan has joined #openstack-nova19:44
Ctesiaswell... It doesn't work :(19:45
CtesiasAny tips on how I would go about overriding defaults in nova.conf the right way?19:45
mriedemcan't you pass overrides on the --config-file cli option19:45
mriedemi'm not sure though19:46
*** sree has joined #openstack-nova19:46
mriedemthere could be something in https://docs.openstack.org/oslo.config/latest/19:46
*** smatzek has joined #openstack-nova19:47
sdagueCtesias: can you explain what you are trying to do with overrides besides modifying the nova.conf directly? Maybe we're just not understanding the question19:47
*** boris-42__ has joined #openstack-nova19:48
CtesiasI was under the impression modifying nova.conf directly was no bueno.  But I'm trying to enable a barbican with Nova, so I need to set the api_class setting in config file under [keymgr]19:49
CtesiasI created nova_override.conf in the same directory where nova.conf was and put [keymgr] and api_class = <the barbican manager> in it19:50
Ctesiasrestarted nova19:50
Ctesiasbut nova had the defaults when it came back19:50
*** sree has quit IRC19:51
mriedemjust modify nova.conf19:51
mriedemnova.conf can be totally blank and it won't override any defaults - the defaults are all in code19:52
*** tbachman_ has joined #openstack-nova19:52
mriedemso if you want to override any defaults, it happens in nova.conf19:52
mriedemnova_override.conf is probably some package-specific trick19:52
Ctesias.....Oh.19:52
*** mgiles has joined #openstack-nova19:52
CtesiasWell awesome.  Thanks for your help, really19:52
mriedemyw19:52
CtesiasI've been pulling my hair out for days19:52
*** tbachman has quit IRC19:54
*** mgiles has quit IRC19:56
*** mgiles has joined #openstack-nova19:57
*** nicolasbock has quit IRC19:58
*** tbachman_ has quit IRC19:59
openstackgerritMatt Riedemann proposed openstack/nova master: Support tag instances when boot(4/4)  https://review.openstack.org/46980020:00
mriedemKevin_Zheng: ^ fixed up the functional test failures; it was pretty simple but not obvious until i injected some debug code and ran the tests, and then also had to tweak the tests a bit20:00
*** brault has joined #openstack-nova20:01
*** tbachman has joined #openstack-nova20:02
mriedemsdague: can you take a gander at ^ ? it's pretty straight-forward and well tested20:03
*** cdent has joined #openstack-nova20:03
*** brault has quit IRC20:05
openstackgerritMatt Riedemann proposed openstack/nova master: Optimize creating servers with tags  https://review.openstack.org/48417020:05
sdaguemriedem: I'll add it to the morning list20:06
mriedemthanks - alex_xu might get it overnight then, he was reviewing it before too20:06
*** annegentle has quit IRC20:07
oomichimriedem: thanks, +220:08
*** lpetrut_ has quit IRC20:10
CtesiasHi again.  I'm getting the same behavior with the changes directly in my nova.conf file.  I've tried pointing at the directory with --config-dir and directly at the nova.conf file with --config-file with no change in behavior20:10
Ctesiasthe api_class value is still the default20:10
mriedemCtesias: which service?20:11
Ctesiasnova service20:11
mriedemwhich of the nova services, there are several20:11
mriedemnova-osapi_compute?20:11
*** owalsh has quit IRC20:11
mriedemnova-compute, etc20:11
Ctesiasnova-compute20:12
*** dklyle has joined #openstack-nova20:12
*** david-lyle has quit IRC20:13
mriedemwhen you restart nova-compute, do you have debug logging enabled so you can see it dump which config files are being used to and it will dump the options20:13
*** pchavva has quit IRC20:16
Ctesiasthe file is pretty long, there something I can grep for to get the file it's using?20:16
*** dave-mccowan has quit IRC20:17
mriedemyeah20:17
mriedem"command line args"20:17
mriedem"config files"20:18
mriedemlike here http://logs.openstack.org/47/484447/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/b2fbd0c/logs/screen-n-cpu.txt.gz#_Jul_17_18_47_11_34578820:18
mriedemand then it dumps the options it's using20:18
mriedemCtesias: you're using the wrong group20:19
mriedemit's not [keymgr]20:19
mriedemit's [key_manager]20:19
mriedemhttp://logs.openstack.org/47/484447/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/b2fbd0c/logs/screen-n-cpu.txt.gz#_Jul_17_18_47_11_67012520:19
Ctesiasyeah, that was a typo earlier.  I've got [key_manager] in the file20:19
CtesiasIf I put a breakpoint in nova/keymgr/conf_key_mgr.py I can go to the line after it grabs the nova configuration and see it's the same20:21
Ctesias(Pdb) CONF['key_manager']['api_class'] 'nova.keymgr.conf_key_mgr.ConfKeyManager'20:21
*** zhenq has joined #openstack-nova20:21
mriedemand the debug log output when you restart nova-compute shows the same when it dumps the options?20:23
*** mamandle has quit IRC20:23
Ctesiasyes20:23
Ctesias2017-07-17 16:20:13.483 DEBUG oslo_service.service [req-ece128f3-fb7c-4da3-af58-78c39fca910c None None] key_manager.api_class          = nova.keymgr.conf_key_mgr.ConfKeyManager from (pid=14641) log_opt_values /opt/pf9/venv/lib/python2.7/site-packages/oslo_config/cfg.py:2626-07-17 16:20:13.485 DEBUG oslo_service.service [req-ece128f3-fb7c-4da3-af58-78c39fca910c None None] osapi_v21.project_id_regex     = None from (pid=14641)20:24
mriedemhmm, api_class comes from castellan20:25
mriedemwhich version of that library do you have installed?20:25
Ctesias0.4.020:26
mriedemkfarr: ^ have you heard of anything like this?20:26
*** david-lyle has joined #openstack-nova20:26
mriedemCtesias: which version of nova? master (pike)? or ocata?20:26
*** randomhack has joined #openstack-nova20:26
mriedembecause that version is old for master https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L22720:26
CtesiasNewton, actually20:27
mriedemew20:27
Ctesiasspeed of business, man20:27
*** dklyle has quit IRC20:27
mriedemso things were different in newton https://github.com/openstack/nova/blob/stable/newton/nova/conf/key_manager.py20:28
mriedemlooks like it should still set those options20:28
*** kristian__ has quit IRC20:29
*** kristian__ has joined #openstack-nova20:29
mriedemCtesias: this might be your problem https://github.com/openstack/nova/blob/stable/newton/nova/keymgr/__init__.py#L2920:30
mriedemnot totally sure20:30
mriedemneed to probably talk to kfarr20:31
CtesiasI don't see api_class as an option in /nova/conf/key_manager.py20:32
mriedemthat's not the options registry,20:32
mriedemthat's the default impl20:32
mriedemthe options are registered here https://github.com/openstack/nova/blob/stable/newton/nova/conf/key_manager.py20:32
mriedemand then there is some backward compat magic in https://github.com/openstack/nova/blob/stable/newton/nova/keymgr/__init__.py20:33
*** prateek has quit IRC20:33
*** kristian__ has quit IRC20:34
Ctesiasyeah commenting out the lines in __init__.py fixes it20:37
CtesiasIs this behavior fixed upstream from Newton anywhere yet?20:38
*** mgiles has quit IRC20:40
mriedemi think that stuff in init was all removed in ocata20:41
*** crushil has quit IRC20:41
mriedemoh i guess not ocata https://github.com/openstack/nova/blob/stable/ocata/nova/keymgr/__init__.py20:41
mriedemor master https://github.com/openstack/nova/blob/master/nova/keymgr/__init__.py20:41
mriedemdamn20:41
mriedemso the fact it's setting that b/c of devstack is a total hack20:42
mriedemlet me see if that's at least set in devstack20:42
*** vishwana_ has joined #openstack-nova20:43
mriedemCtesias: https://review.openstack.org/#/c/439855/20:43
*** burt has quit IRC20:43
Ctesiascool.  Thanks20:44
mriedemCtesias: so did you just remove this line? https://github.com/openstack/nova/blob/stable/newton/nova/keymgr/__init__.py#L29 ?20:45
mriedemor all of the stuff in there20:45
*** vishwanathj has quit IRC20:46
CtesiasI only commented out the lines that re-set the api_class value20:46
*** damien_r has joined #openstack-nova20:47
Ctesiasbut if you're curious I can try to get rid of all of it and see what happens20:47
mriedemi'm curious if just commenting out https://github.com/openstack/nova/blob/stable/newton/nova/keymgr/__init__.py#L29 resolves your issue20:47
mriedembecause that would be easy for me to change on master and backport through to newton20:47
CtesiasYes.20:47
mriedemok20:47
CtesiasIf you would be so kind as to post that change once it comes in so I can follow it back and get upgraded to it that would be excellent.20:48
mriedemi can mess around with pushing that change later then - i can't say i totally understand *why* it would cause an issue since it should just set a default, not override your value20:48
mriedemunless it thinks that your value was the default or something?20:48
Ctesiasbeats the hell out of me20:48
mriedemi'll just remove it on the grounds that we have the value set in devstack so removing the hack is good enough20:49
*** vishwana_ has quit IRC20:50
*** crushil has joined #openstack-nova20:50
mriedemCtesias: reporting a bug against nova would also help with the backports20:50
mriedemhttps://bugs.launchpad.net/nova/+filebug20:50
Ctesiaswill do20:52
*** harlowja has joined #openstack-nova20:56
*** damien_r has quit IRC20:59
*** baoli has quit IRC20:59
*** lucasxu has quit IRC21:01
*** Ctesias has quit IRC21:01
*** thorst has quit IRC21:10
*** randomhack has quit IRC21:11
*** MVenesio has quit IRC21:12
openstackgerritMatt Riedemann proposed openstack/nova master: Update policy descriptions for base.  https://review.openstack.org/47262221:13
*** smatzek has quit IRC21:13
openstackgerritMatt Riedemann proposed openstack/nova master: Consistent policies  https://review.openstack.org/47977421:13
openstackgerritMatt Riedemann proposed openstack/nova master: Update policy descriptions for base.  https://review.openstack.org/47262221:14
mriedemoomichi: another easy one and the final patch for the policy docs bp https://review.openstack.org/#/c/472622/21:15
*** awaugama has quit IRC21:19
*** owalsh has joined #openstack-nova21:21
*** Sukhdev has quit IRC21:26
mriedemwho here feels like adding back in the ability to load up out of tree scheduler drivers? https://bugs.launchpad.net/nova/+bug/170478821:27
openstackLaunchpad bug 1704788 in OpenStack Compute (nova) "Hardcoded choices for nova scheduler driver" [Undecided,Confirmed]21:27
dansmithheh21:30
mriedembauzas: since you were +W on that thing, how do you feel about ^21:30
mriedemanswer tomorrow when you're awake21:30
mriedembecause we really probably should have made that (1) a deprecation cycle and (2) ffs at least reno it21:30
openstackgerritMerged openstack/nova master: Handle Cinder 3.27 style attachments in swap_volume  https://review.openstack.org/45697121:33
*** rcernin has quit IRC21:33
openstackgerritMerged openstack/nova master: [placement] Update allocation-candidates.yaml for gabbi 1.35  https://review.openstack.org/48435621:34
cdent\o/21:34
*** dave-mccowan has joined #openstack-nova21:34
mriedemcdent: btw, just so you know,21:34
mriedemwe went hiking, toured a historic village, and went swimming yesterday21:34
mriedemso it wasn't all work21:35
cdentah well, I am so relieved21:35
*** sree_ has joined #openstack-nova21:36
*** sree_ is now known as Guest5047621:36
melwittmriedem: replied to your latest comments. the things that seem up in the air are 1) whether to add vm_states.SOFT_DELETED + task_states.RESTORING instances to the count and recheck in restore21:36
melwitt2) whether to check quota on resize confirm21:36
mriedemok21:38
mriedemoh dan https://review.openstack.org/#/c/472622/ <321:38
melwittmriedem: I'm not sure what I think other than, if I do both 1) and 2) I guess that eliminates the TODOs I had written bc checking quota at resize confirm sort of serves as a recheck for resize (as I described in the review comment)21:40
*** Guest50476 has quit IRC21:40
*** annegentle has joined #openstack-nova21:40
melwittso maybe that one should be guarded by "if CONF.quota.recheck_quota"21:40
melwittcorrection, the revert_resize TODO would remain21:44
dansmithmriedem: we might want to get this in before the quota-ectomy so we can backport: https://review.openstack.org/#/c/462521/4/nova/compute/manager.py21:44
mriedemdansmith: that's 3 changes in a series21:45
mriedemthe 2nd change has no tests21:45
dansmithah, yup21:46
dansmiththe bottom one doesn't either and it seems like they'd be trivial21:49
*** itlinux_ has joined #openstack-nova21:52
*** cdent has quit IRC21:52
*** annegentle has quit IRC21:53
openstackgerritMatt Riedemann proposed openstack/nova master: Remove key_manager.api_class hack  https://review.openstack.org/48450121:58
*** kylek3h has quit IRC22:02
*** felipemonteiro_ has quit IRC22:03
*** lpetrut_ has joined #openstack-nova22:04
*** esberglu has quit IRC22:05
*** edmondsw has quit IRC22:10
*** thorst has joined #openstack-nova22:10
*** mriedem has quit IRC22:10
*** gouthamr has quit IRC22:12
*** gouthamr has joined #openstack-nova22:14
*** thorst has quit IRC22:15
openstackgerritMatt Riedemann proposed openstack/nova master: Log compute node uuid when the record is created  https://review.openstack.org/48444722:15
*** mriedem has joined #openstack-nova22:16
*** mmehan has quit IRC22:17
*** lpetrut_ has quit IRC22:19
*** esberglu has joined #openstack-nova22:20
*** esberglu has quit IRC22:25
*** baoli has joined #openstack-nova22:25
*** baoli has quit IRC22:26
*** edmondsw has joined #openstack-nova22:26
*** xyang1 has quit IRC22:29
openstackgerritMatt Riedemann proposed openstack/nova master: Convert HostState.limits['numa_topology'] to primitive  https://review.openstack.org/48443922:29
mriedemdansmith: ^ is that what you were looking for wrt being specific?22:29
dansmithmriedem: well kinda, although I don't see why you need to check for if it's an object22:30
dansmiththat's just too much ambiguity unless it's really needed22:30
dansmithif it's set, it's that object, right?22:30
mriedemhonestly i'd rather not even do this specific check at all22:30
mriedemand just handle it generically22:30
*** edmondsw has quit IRC22:31
mriedemi don't like the tight coupling between the filters and the scheduler manager here22:31
*** zhenq has quit IRC22:32
dansmithidk, I just think more magic object conversions hide where this is happening22:32
mriedemtrue22:33
mriedemi don't like any of it really so i don't care too much :)22:33
*** acormier has quit IRC22:33
*** acormier has joined #openstack-nova22:34
dansmiththey're reverting the jsonutils change, but they plan to put it back in I guess?22:35
mriedemthey plan to put it back in with a UserWarning that the thing passed in can't be turned into a primitive22:35
mriedemand then drop that at some point with a major version bump i guess22:36
dansmithokay22:36
mriedemsince it's an api change in the library22:36
mriedembut 6 months from now who knows if anyone will remember any of this22:36
dansmithwell, I dunno, we shouldn't really be doing this in the first place for an unversioned return value of an rpc call, IMHO22:36
dansmithbut IMHO, it's best to be very explicit and targeted if we're going to convert it22:37
dansmithi.e. "if there is a numa limit, it is a NumaWhateverObject and we shall always convert it to primitive"22:37
*** marst has quit IRC22:37
mriedemok making that change22:38
*** acormier has quit IRC22:38
*** thorst has joined #openstack-nova22:41
*** itlinux_ has quit IRC22:42
*** thorst has quit IRC22:43
*** ckopper has quit IRC22:46
*** armax has quit IRC22:47
openstackgerritMatt Riedemann proposed openstack/nova master: Convert HostState.limits['numa_topology'] to primitive  https://review.openstack.org/48443922:48
*** mriedem has quit IRC22:54
*** Apoorva has quit IRC22:55
*** Apoorva has joined #openstack-nova22:56
*** edmondsw has joined #openstack-nova22:57
*** baoli has joined #openstack-nova23:02
*** baoli has quit IRC23:02
*** edmondsw has quit IRC23:02
*** Apoorva_ has joined #openstack-nova23:04
*** Apoorva has quit IRC23:07
*** catintheroof has quit IRC23:07
*** mriedem has joined #openstack-nova23:08
*** lyan has quit IRC23:10
mriedemefried: is https://review.openstack.org/#/c/458257/ the last thing for that bp?23:11
mriedemi see the dependent change is in ksa 2.2023:11
efriedmriedem No, that one had a couple of the clienty changes; plan was to follow up with others.23:12
efriedmriedem Did ksa 2.20 happen??23:12
efriedmriedem Or maybe 2.20 isn't the one I thought I was trying to squeeze stuff into right now.23:12
mriedemhttps://review.openstack.org/#/c/482742/ ?23:13
mriedemwhich change in ksa specifically do we need?23:14
mriedemor all over https://review.openstack.org/#/q/topic:version-discovery ?23:14
mriedem*all ov23:14
mriedem*of23:14
efriedmriedem That was the hope.  Specifically I wanted to fix the APIs in discover.py - https://review.openstack.org/#/c/483604/23:15
efried...so we didn't release them with unnecessary params and have to support those (and their [really complicated] implications) forever.23:15
mriedemok so realizing that ksa boat of changes isn't going to land by thursday probably,23:17
*** ckopper has joined #openstack-nova23:17
mriedemdo we have something pragmatic to land in nova to complete this for pike and then improve it later?23:17
mriedemsfinucan: you took over as author on this https://review.openstack.org/#/c/428241/23:18
mriedemi think that git review / rebase bug where it changes author is hitting more than just me23:19
efriedmriedem If you're assuming the boat won't land... I would have to work something up that's better than https://review.openstack.org/#/c/458257/ but not the whole ksa hog.23:19
mriedemefried: i'm assuming it won't land by thursday, no23:19
efriedBut I've been operating under the hope that it's not too late.  mordred Pipe dream?23:19
mriedemand if it's rushed, it'll likely be buggy23:19
mordredI don't think it's a pipe-dream23:21
mordredthe stuff we're working on getting in are mostly cleanups, better docs and better tests - the hard logic is in23:21
mordredmriedem: I'm currently still pushing towards getting it out this week - I don't think it's unachievable23:22
efriedmordred You and I should probably take a hard look at whether https://review.openstack.org/#/c/483604/ (or some variant thereof) is within that achievable umbrella.23:22
efriedCause that's hard logic.23:23
efriedGood news is, the test cases are robust to a point where, if we can get 'em passing, I'll have confidence in the code.23:23
*** sree_ has joined #openstack-nova23:24
*** sree_ is now known as Guest120323:24
mriedemmordred: ok the release freeze is thursday23:25
mordredefried: yes. I agree with that23:25
mordredmriedem: yup23:25
mriedem </project management status update check>23:25
mordredefried: (I agree we need to take a hard look at it - and I agree that the tests are robust and trustable)23:25
*** ckopper has quit IRC23:27
*** hongbin has quit IRC23:28
*** Guest1203 has quit IRC23:28
*** acormier has joined #openstack-nova23:29
*** ckopper has joined #openstack-nova23:30
*** gouthamr has quit IRC23:32
*** takashin has joined #openstack-nova23:32
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Microversion 2.52 - List/Show all server migration types  https://review.openstack.org/43083923:33
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Microversion 2.52 - Enable cold migration with target host  https://review.openstack.org/40670723:33
*** acormier has quit IRC23:33
*** ckopper has quit IRC23:34
mriedemTheJulia: question in https://review.openstack.org/#/c/419975/23:48
mriedemnot sure who on the ironic team has the setup to test this,23:48
mriedembut when plugging the vif, neutron should eventually recognize a state change in the port and send an event back to nova to refresh the cache23:48
mriedemthen we wouldn't need all that gorpy refresh cache code within the driver23:48
*** jwcroppe has quit IRC23:49
*** gouthamr has joined #openstack-nova23:50
*** acormier has joined #openstack-nova23:52
*** gouthamr has quit IRC23:53
*** gouthamr has joined #openstack-nova23:54
*** chyka has quit IRC23:57

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