Wednesday, 2017-07-12

*** dtp has quit IRC00:04
*** abalutoiu has quit IRC00:05
*** gmann has quit IRC00:11
*** gmann has joined #openstack-nova00:11
*** thorst has quit IRC00:17
*** rajathagasthya has quit IRC00:19
*** Swami has quit IRC00:22
*** sree has joined #openstack-nova00:23
mriedemdansmith: video ram isn't tracked for placement :) it's checked on server create and decremented from ram quota on server delete, and checked in the libvirt driver, but not part of the resource claim tracking :)00:24
mriedemof course00:24
openstackgerritMerged openstack/nova master: Pre-load instance.device_metadata in InstanceMetadata  https://review.openstack.org/48123500:26
openstackgerritMerged openstack/nova master: hyperv: Fixes log message in livemigrationops  https://review.openstack.org/47924500:27
*** sree has quit IRC00:27
*** vishwanathj has quit IRC00:30
*** vishwanathj has joined #openstack-nova00:30
*** xyang1 has quit IRC00:35
*** zhurong has joined #openstack-nova00:42
*** fragatin_ has joined #openstack-nova00:43
*** slaweq has joined #openstack-nova00:43
*** trungnv has quit IRC00:43
*** hieulq has quit IRC00:44
mriedemalex_xu: i replied to your comments in https://review.openstack.org/#/c/464280/ - i probably won't be able to attend the api meeting this week, so i might post some thoughts in the ML for discussion, but i'm struggling with deciding if we should allow the same behavior as before regarding trying to disable an already disabled service and failing or ignoring it, same for enabling an enabled service since that's not a failure t00:45
mriedembut that's essentially how i wrote that change - it won't fail if you try to enable and enabled service and vice versa for disabled00:45
mriedem*an enabled service00:46
*** fragatina has quit IRC00:46
*** fragatin_ has quit IRC00:47
*** Apoorva has quit IRC00:48
*** fragatina has joined #openstack-nova00:48
*** slaweq has quit IRC00:48
*** Shunli has joined #openstack-nova00:48
*** baoli has joined #openstack-nova00:49
*** vishwanathj has quit IRC00:53
*** vishwanathj has joined #openstack-nova00:54
*** liujiong has joined #openstack-nova00:56
*** gcb has quit IRC00:57
*** abalutoiu has joined #openstack-nova00:58
*** huanxie has joined #openstack-nova00:58
*** acormier has joined #openstack-nova00:58
*** trungnv has joined #openstack-nova00:58
*** hieulq has joined #openstack-nova00:59
*** Nel1x has quit IRC00:59
*** vladikr has quit IRC01:02
*** phuongnh has joined #openstack-nova01:02
*** gouthamr has quit IRC01:06
*** dixiaoli has joined #openstack-nova01:09
*** yufei has joined #openstack-nova01:10
*** acormier has quit IRC01:12
*** acormier has joined #openstack-nova01:12
*** trungnv has quit IRC01:14
*** masuberu has joined #openstack-nova01:14
*** itlinux has joined #openstack-nova01:15
*** phuongnh has quit IRC01:16
*** hieulq has quit IRC01:16
*** vishwanathj has quit IRC01:16
*** vishwanathj has joined #openstack-nova01:17
*** vishwanathj has quit IRC01:18
*** masber has quit IRC01:18
*** vishwanathj has joined #openstack-nova01:18
*** vvargaszte has quit IRC01:20
*** acormier has quit IRC01:26
*** acormier has joined #openstack-nova01:26
*** mriedem has quit IRC01:33
*** awaugama has quit IRC01:38
*** slaweq has joined #openstack-nova01:45
*** slaweq has quit IRC01:49
*** masber has joined #openstack-nova01:57
*** yushb has joined #openstack-nova01:57
*** annegentle has joined #openstack-nova01:58
*** masuberu has quit IRC02:00
*** yamahata has quit IRC02:00
*** smatzek has joined #openstack-nova02:03
*** trungnv has joined #openstack-nova02:07
*** hieulq has joined #openstack-nova02:07
*** acormier has quit IRC02:08
*** acormier has joined #openstack-nova02:08
*** hongbin has joined #openstack-nova02:09
*** kiennt has joined #openstack-nova02:10
*** Vek has quit IRC02:10
*** huanxie has quit IRC02:18
*** fragatin_ has joined #openstack-nova02:18
*** coreywright has quit IRC02:20
*** litao__ has joined #openstack-nova02:22
*** trungnv has quit IRC02:22
*** fragatina has quit IRC02:22
*** gouthamr has joined #openstack-nova02:25
*** annegentle has quit IRC02:29
*** itlinux has quit IRC02:31
*** fragatin_ has quit IRC02:33
*** coreywright has joined #openstack-nova02:34
*** fragatina has joined #openstack-nova02:34
*** fragatina has quit IRC02:34
*** trungnv has joined #openstack-nova02:35
*** fragatina has joined #openstack-nova02:35
*** thorst has joined #openstack-nova02:39
*** hieulq has quit IRC02:39
*** kiennt has quit IRC02:40
*** trungnv has quit IRC02:40
*** thorst has quit IRC02:44
*** slaweq has joined #openstack-nova02:45
*** yamahata has joined #openstack-nova02:49
*** slaweq has quit IRC02:50
*** jmlowe has joined #openstack-nova02:50
*** zzzeek_ has quit IRC02:52
*** smatzek has quit IRC03:00
*** rajathagasthya has joined #openstack-nova03:05
*** rajathagasthya has quit IRC03:05
*** zhurong has quit IRC03:08
*** itlinux has joined #openstack-nova03:09
*** acormier has quit IRC03:10
*** acormier has joined #openstack-nova03:10
*** acormier has quit IRC03:15
*** trungnv has joined #openstack-nova03:16
*** brad[] has quit IRC03:16
*** hieulq has joined #openstack-nova03:16
*** tbachman has joined #openstack-nova03:17
*** dave-mccowan has joined #openstack-nova03:19
*** baoli has quit IRC03:22
*** itlinux has quit IRC03:23
openstackgerritzhangdaolong proposed openstack/nova master: Fix appropriate parameters name for unit test  https://review.openstack.org/48278203:23
*** huanxie has joined #openstack-nova03:24
*** baoli has joined #openstack-nova03:26
*** gbarros has quit IRC03:28
*** baoli has quit IRC03:31
*** gcb has joined #openstack-nova03:36
*** yufei has quit IRC03:40
*** dave-mccowan has quit IRC03:40
*** udesale has joined #openstack-nova03:44
*** zhurong has joined #openstack-nova03:44
*** kiennt has joined #openstack-nova03:45
*** gouthamr has quit IRC03:46
*** slaweq has joined #openstack-nova03:46
*** slaweq has quit IRC03:51
*** huanxie has quit IRC03:54
*** itlinux has joined #openstack-nova03:54
openstackgerritZhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4)  https://review.openstack.org/46980003:55
*** tonyb has quit IRC04:05
*** NostawRm has quit IRC04:08
*** Vek has joined #openstack-nova04:12
*** mdnadeem has joined #openstack-nova04:13
*** NostawRm has joined #openstack-nova04:13
*** phuongnh has joined #openstack-nova04:15
*** adisky__ has joined #openstack-nova04:21
*** phuongnh has quit IRC04:27
*** kiennt has quit IRC04:27
*** cNilesh has joined #openstack-nova04:27
*** tonyb has joined #openstack-nova04:28
*** swebster has joined #openstack-nova04:28
*** swebster_ has quit IRC04:29
*** trungnv has quit IRC04:29
*** hieulq has quit IRC04:30
*** psachin has joined #openstack-nova04:30
*** fragatin_ has joined #openstack-nova04:31
*** yushb has quit IRC04:32
*** yingjun has joined #openstack-nova04:32
*** yamamoto has joined #openstack-nova04:33
*** fragatina has quit IRC04:34
*** hongbin has quit IRC04:35
*** fragatin_ has quit IRC04:35
*** gongysh has joined #openstack-nova04:35
*** tbh__ has joined #openstack-nova04:38
*** tbh__ is now known as tbh_04:38
*** thorst has joined #openstack-nova04:40
*** pcaruana has joined #openstack-nova04:41
*** fragatina has joined #openstack-nova04:44
*** trinaths has joined #openstack-nova04:44
*** thorst has quit IRC04:45
*** kiennt has joined #openstack-nova04:45
*** trungnv has joined #openstack-nova04:46
*** hieulq has joined #openstack-nova04:47
*** fragatina has quit IRC04:48
*** trinaths1 has joined #openstack-nova04:53
*** jmlowe has quit IRC04:56
*** trinaths has quit IRC04:57
*** pcaruana has quit IRC04:58
*** trinaths has joined #openstack-nova04:59
*** trinaths1 has quit IRC05:00
*** huanxie has joined #openstack-nova05:01
*** yufei has joined #openstack-nova05:04
*** slaweq has joined #openstack-nova05:05
*** sree has joined #openstack-nova05:05
*** slaweq has quit IRC05:05
*** fragatina has joined #openstack-nova05:14
*** Shunli has quit IRC05:14
*** armax has quit IRC05:14
*** armax has joined #openstack-nova05:14
*** armax has quit IRC05:15
*** Shunli has joined #openstack-nova05:15
*** armax has joined #openstack-nova05:15
*** armax has quit IRC05:16
*** NostawRm has quit IRC05:16
*** NostawRm has joined #openstack-nova05:16
*** armax has joined #openstack-nova05:16
*** armax has quit IRC05:16
*** tbachman has quit IRC05:17
*** armax has joined #openstack-nova05:17
*** armax has quit IRC05:17
*** krtaylor has quit IRC05:17
*** armax has joined #openstack-nova05:17
*** armax has quit IRC05:18
*** fragatina has quit IRC05:18
*** tbachman has joined #openstack-nova05:18
*** armax has joined #openstack-nova05:18
*** armax has quit IRC05:19
*** armax has joined #openstack-nova05:19
*** armax has quit IRC05:20
*** baoli has joined #openstack-nova05:22
*** baoli has quit IRC05:27
*** hieulq has quit IRC05:27
*** trungnv has quit IRC05:28
*** kiennt has quit IRC05:29
*** rcernin has joined #openstack-nova05:33
*** ratailor has joined #openstack-nova05:36
*** gcb has quit IRC05:37
*** nicolasbock has joined #openstack-nova05:38
*** diga has joined #openstack-nova05:38
*** moshele has joined #openstack-nova05:44
*** trungnv has joined #openstack-nova05:44
*** hieulq has joined #openstack-nova05:45
*** lpetrut has joined #openstack-nova05:46
*** itlinux has quit IRC05:52
*** gyee has quit IRC05:52
openstackgerritMerged openstack/nova master: Fix quota class set APIs  https://review.openstack.org/46799905:56
*** ekuris has joined #openstack-nova05:57
*** prateek has joined #openstack-nova05:58
*** huanxie has quit IRC05:59
*** yushb has joined #openstack-nova06:00
*** yassine has quit IRC06:01
*** slaweq has joined #openstack-nova06:02
*** slaweq_ has joined #openstack-nova06:06
*** slaweq has quit IRC06:06
*** slaweq has joined #openstack-nova06:07
*** kiennt has joined #openstack-nova06:08
*** tjones has quit IRC06:09
*** slaweq_ has quit IRC06:12
*** crushil has quit IRC06:14
*** crushil has joined #openstack-nova06:15
*** ekuris has quit IRC06:15
*** liujiong has quit IRC06:19
*** Oku_OS-away is now known as Oku_OS06:19
*** liujiong has joined #openstack-nova06:19
*** Shunli has quit IRC06:20
*** Shunli has joined #openstack-nova06:21
*** kiennt_ has joined #openstack-nova06:21
*** kiennt has quit IRC06:22
*** phuongnh has joined #openstack-nova06:27
*** crushil_ has joined #openstack-nova06:28
*** Shunli has quit IRC06:30
*** ekuris has joined #openstack-nova06:31
*** sshwarts has joined #openstack-nova06:32
openstackgerritBéla Vancsics proposed openstack/nova master: Modify the value of instance's root_gb when flavor's root_gb is 0  https://review.openstack.org/39889106:34
*** gongysh has quit IRC06:34
openstackgerritZhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4)  https://review.openstack.org/46980006:36
*** gjayavelu has joined #openstack-nova06:37
*** smcginnis has quit IRC06:39
*** lpetrut has quit IRC06:41
*** dixiaoli has quit IRC06:41
*** thorst has joined #openstack-nova06:41
*** phuongnh has quit IRC06:43
*** kiennt_ has quit IRC06:43
*** smcginnis has joined #openstack-nova06:43
*** andreas_s has joined #openstack-nova06:44
*** kiennt has joined #openstack-nova06:44
*** phuongnh has joined #openstack-nova06:45
*** thorst has quit IRC06:46
*** belmoreira has joined #openstack-nova06:47
*** openstackgerrit has quit IRC06:48
*** liujiong_lj has joined #openstack-nova06:50
-openstackstatus- NOTICE: FYI, ask.openstack.org is down, review.o.o is slow - please have patience until this is fixed06:52
*** phuongnh has quit IRC06:53
*** kiennt has quit IRC06:53
*** kman has joined #openstack-nova06:53
*** liujiong has quit IRC06:54
*** phuongnh has joined #openstack-nova06:54
*** kiennt has joined #openstack-nova06:55
*** aloga has quit IRC06:55
*** zhurong has quit IRC06:55
*** trinaths is now known as trinaths_lunch06:59
*** phuongnh has quit IRC07:03
*** udesale has quit IRC07:03
*** kiennt has quit IRC07:03
*** phuongnh has joined #openstack-nova07:05
*** kiennt has joined #openstack-nova07:05
*** slaweq_ has joined #openstack-nova07:07
*** damien_r has joined #openstack-nova07:10
*** gjayavelu has quit IRC07:11
*** tesseract has joined #openstack-nova07:11
*** slaweq_ has quit IRC07:12
*** aloga has joined #openstack-nova07:14
*** gjayavelu has joined #openstack-nova07:15
*** kman has quit IRC07:17
*** yassine has joined #openstack-nova07:18
*** hieulq has quit IRC07:19
*** trungnv has quit IRC07:20
*** kiennt has quit IRC07:22
*** phuongnh has quit IRC07:22
*** edmondsw has joined #openstack-nova07:24
*** phuongnh has joined #openstack-nova07:25
*** kiennt has joined #openstack-nova07:25
*** crushil_ has quit IRC07:27
*** edmondsw has quit IRC07:29
*** alexchadin has joined #openstack-nova07:30
*** pcaruana has joined #openstack-nova07:31
*** zhurong has joined #openstack-nova07:31
*** aarefiev_afk is now known as aarefiev07:32
*** trungnv has joined #openstack-nova07:34
*** hieulq has joined #openstack-nova07:35
*** tuanluong has joined #openstack-nova07:36
*** ygl has joined #openstack-nova07:36
*** ygl has left #openstack-nova07:37
*** roeyc has joined #openstack-nova07:38
*** openstackgerrit has joined #openstack-nova07:38
openstackgerritjichenjc proposed openstack/nova master: Enhance support matrix document  https://review.openstack.org/48201307:38
*** claudiub has joined #openstack-nova07:39
*** pcaruana has quit IRC07:41
*** ekuris has quit IRC07:41
danpawlikGood morning07:42
danpawlikDoes someone know, if its possible to change keyboard layout in novnc proxy without changing instnce  xml file?07:43
*** hferenc has quit IRC07:43
*** kristian__ has joined #openstack-nova07:44
*** amotoki_away is now known as amotoki07:45
*** kristian__ has quit IRC07:45
*** pcaruana has joined #openstack-nova07:45
*** hferenc has joined #openstack-nova07:45
openstackgerritjichenjc proposed openstack/nova master: Add quiesce and unquiesce in support matrix  https://review.openstack.org/48239007:45
openstackgerritzhangdaolong proposed openstack/nova master: Fix wrong log parm.  https://review.openstack.org/48283207:47
*** vvargaszte has joined #openstack-nova07:48
-openstackstatus- NOTICE: Gerrit is going to be restarted, due to low performance07:49
*** sree has quit IRC07:54
*** ekuris has joined #openstack-nova07:55
-openstackstatus- NOTICE: Gerrit has been successfully restarted07:57
openstackgerritjichenjc proposed openstack/nova master: Enhance support matrix document  https://review.openstack.org/48202007:57
*** sree_ has joined #openstack-nova07:58
*** sree_ is now known as Guest5897607:58
*** zzzeek has quit IRC08:00
*** kristian__ has joined #openstack-nova08:00
*** zzzeek has joined #openstack-nova08:00
*** gouthamr has joined #openstack-nova08:02
*** liujiong_lj is now known as liujiong08:02
*** Guest58976 has quit IRC08:03
*** slaweq_ has joined #openstack-nova08:08
*** ekuris_ has joined #openstack-nova08:09
*** ekuris has quit IRC08:09
*** dr_gogeta86 has joined #openstack-nova08:13
*** slaweq_ has quit IRC08:13
*** trinaths_lunch is now known as trinaths08:13
*** dr_gogeta86 has quit IRC08:13
*** dr_gogeta86 has joined #openstack-nova08:13
*** ralonsoh has joined #openstack-nova08:17
*** efoley has joined #openstack-nova08:17
openstackgerritZhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4)  https://review.openstack.org/46980008:20
*** gouthamr has quit IRC08:21
*** tojuvone has quit IRC08:26
*** kristian__ has quit IRC08:26
*** tojuvone has joined #openstack-nova08:26
*** yushb has quit IRC08:30
*** yushb has joined #openstack-nova08:30
*** lucas-afk is now known as lucasagomes08:32
*** tojuvone has quit IRC08:33
*** pcaruana has quit IRC08:33
*** tojuvone has joined #openstack-nova08:34
bauzasp!spy08:36
*** thorst has joined #openstack-nova08:42
*** dklyle has joined #openstack-nova08:45
*** david-lyle has quit IRC08:45
openstackgerritMerged openstack/os-vif master: Enable some off-by-default checks  https://review.openstack.org/48238408:46
*** pcaruana has joined #openstack-nova08:46
*** karimb has joined #openstack-nova08:46
*** thorst has quit IRC08:47
*** vvargaszte1 has joined #openstack-nova08:47
*** liujiong has quit IRC08:47
*** liujiong has joined #openstack-nova08:47
*** tbh_ has quit IRC08:48
ildikovbauzas: hi :)08:49
bauzasildikov: hey08:49
openstackgerritMerged openstack/os-vif master: set mtu on all code paths  https://review.openstack.org/46995108:49
* bauzas in meeting08:49
ildikovbauzas: I have a review with one +2 on it already: https://review.openstack.org/#/c/456971/08:49
ildikovbauzas: was just wondering if you could take a look when you have a little time :)08:49
ildikovbauzas: pretty please :)08:49
bauzasildikov: sure, I can08:49
ildikovbauzas: awesome, thank you!08:50
*** vvargaszte has quit IRC08:50
*** vvargaszte1 is now known as vvargaszte08:50
openstackgerritBalazs Gibizer proposed openstack/nova master: Add BDM to InstancePayload  https://review.openstack.org/44877908:50
openstackgerritsudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid()  https://review.openstack.org/48100708:51
*** karimb has quit IRC08:56
*** yushb has quit IRC08:58
*** mkucia has joined #openstack-nova08:59
*** udesale has joined #openstack-nova09:02
openstackgerritAlex Szarka proposed openstack/nova master: Changed maximum code complexity to 30  https://review.openstack.org/35117609:02
openstackgerritiswarya vakati proposed openstack/nova master: Fixed wrap from taking negative values  https://review.openstack.org/48146509:04
*** karimb has joined #openstack-nova09:07
*** ygl has joined #openstack-nova09:09
*** slaweq_ has joined #openstack-nova09:09
*** claudiub|2 has joined #openstack-nova09:11
*** edmondsw has joined #openstack-nova09:12
*** claudiub has quit IRC09:14
*** slaweq_ has quit IRC09:14
*** liujiong has quit IRC09:15
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-baremetal-nodes endpoint instead of stevedore  https://review.openstack.org/46148009:15
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-security-groups instead of stevedore  https://review.openstack.org/47890109:15
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-security-group-rules instead of stevedore  https://review.openstack.org/47890309:15
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for server-security-groups instead of stevedore  https://review.openstack.org/47890209:15
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-networks instead of stevedore  https://review.openstack.org/47446309:15
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for image-metadata instead of stevedore  https://review.openstack.org/47849909:15
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-volumes instead of stevedore  https://review.openstack.org/47949709:15
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for images instead of stevedore  https://review.openstack.org/47849809:15
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-security-group-default-rules instead of stevedore  https://review.openstack.org/47890409:15
*** liujiong has joined #openstack-nova09:15
*** claudiub has joined #openstack-nova09:15
*** edmondsw has quit IRC09:16
*** claudiub|2 has quit IRC09:18
*** yushb has joined #openstack-nova09:21
*** lpetrut has joined #openstack-nova09:27
*** huanxie has joined #openstack-nova09:31
*** pcaruana has quit IRC09:32
*** pcaruana has joined #openstack-nova09:32
*** yamahata has quit IRC09:34
*** sambetts|afk is now known as sambetts09:40
*** yingjun has quit IRC09:40
*** thorst has joined #openstack-nova09:43
*** mvk has joined #openstack-nova09:44
*** thorst has quit IRC09:47
openstackgerritjichenjc proposed openstack/nova master: Add description for tag support in attach  https://review.openstack.org/48147909:48
openstackgerritjichenjc proposed openstack/nova master: Document device tags in support matrix  https://review.openstack.org/48147809:48
*** kristian__ has joined #openstack-nova09:49
*** karimb has quit IRC09:50
openstackgerrityuanyue proposed openstack/nova master: Recreate root volume for instance rebuild  https://review.openstack.org/47833009:51
*** brad[] has joined #openstack-nova09:57
*** cdent has joined #openstack-nova09:57
*** karimb has joined #openstack-nova09:58
*** yufei has quit IRC09:58
*** cnf has joined #openstack-nova10:01
*** cnf has left #openstack-nova10:01
*** ygl has quit IRC10:02
*** gjayavelu has quit IRC10:08
*** slaweq_ has joined #openstack-nova10:10
*** kiennt has quit IRC10:12
*** phuongnh has quit IRC10:14
*** slaweq_ has quit IRC10:14
*** slaweq_ has joined #openstack-nova10:14
*** slaweq_ has quit IRC10:14
*** tuanluong has quit IRC10:14
*** hieulq has quit IRC10:16
*** trungnv has quit IRC10:18
*** sridharg has quit IRC10:21
*** liujiong has quit IRC10:23
*** udesale__ has joined #openstack-nova10:24
openstackgerrityuanyue proposed openstack/nova master: Recreate root volume for instance rebuild  https://review.openstack.org/47833010:27
*** udesale has quit IRC10:27
*** yushb has quit IRC10:27
*** dklyle has quit IRC10:28
*** david-lyle has joined #openstack-nova10:28
*** alexchadin has quit IRC10:28
*** sridharg has joined #openstack-nova10:30
*** udesale has joined #openstack-nova10:37
*** vvargaszte has quit IRC10:37
*** vvargaszte has joined #openstack-nova10:38
*** udesale__ has quit IRC10:38
*** vvargaszte has quit IRC10:38
*** vvargaszte has joined #openstack-nova10:39
*** zhurong has quit IRC10:41
*** vvargaszte has quit IRC10:41
*** vvargaszte has joined #openstack-nova10:41
*** tbachman_ has joined #openstack-nova10:42
*** vvargaszte has quit IRC10:42
*** vvargaszte has joined #openstack-nova10:42
*** tbachman has quit IRC10:44
*** tbachman_ is now known as tbachman10:44
*** udesale has quit IRC10:45
*** xinliang has quit IRC10:48
*** hferenc has quit IRC10:57
*** xinliang has joined #openstack-nova11:01
*** trungnv has joined #openstack-nova11:04
*** hieulq has joined #openstack-nova11:04
*** hieulq has quit IRC11:08
*** markvoelker has quit IRC11:09
*** lucasagomes is now known as lucas-hungry11:10
*** markvoelker has joined #openstack-nova11:10
*** markvoelker has quit IRC11:14
*** slaweq_ has joined #openstack-nova11:15
*** trungnv has quit IRC11:16
*** smatzek has joined #openstack-nova11:18
*** dave-mccowan has joined #openstack-nova11:19
*** slaweq_ has quit IRC11:20
*** gszasz has joined #openstack-nova11:21
*** ratailor has quit IRC11:27
*** acormier has joined #openstack-nova11:29
*** liverpooler has quit IRC11:31
*** lyan has joined #openstack-nova11:34
*** hferenc has joined #openstack-nova11:36
*** acormier has quit IRC11:37
*** acormier has joined #openstack-nova11:38
*** thorst has joined #openstack-nova11:38
*** huanxie has quit IRC11:38
*** acormier has quit IRC11:42
*** hieulq has joined #openstack-nova11:44
*** kristia__ has joined #openstack-nova11:45
*** trungnv has joined #openstack-nova11:45
*** ecelik has joined #openstack-nova11:47
*** ecelik has left #openstack-nova11:47
*** kristian__ has quit IRC11:47
*** smatzek has quit IRC11:50
*** karimb has quit IRC11:52
*** efried_zzz is now known as efried11:52
*** smatzek has joined #openstack-nova11:53
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocation_candidates  https://review.openstack.org/48111211:53
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for usages  https://review.openstack.org/48056311:53
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Make placement_api_docs.py failing  https://review.openstack.org/48092411:53
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP usages  https://review.openstack.org/45010511:53
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP allocations  https://review.openstack.org/45055011:53
*** markvoelker has joined #openstack-nova11:55
*** litao__ has quit IRC11:56
*** alexchadin has joined #openstack-nova12:00
*** READ10 has quit IRC12:01
*** mkucia has quit IRC12:03
*** sridharg has quit IRC12:07
*** lucas-hungry is now known as lucasagomes12:09
openstackgerritEric Fried proposed openstack/nova master: Document device tags in support matrix  https://review.openstack.org/48147812:10
openstackgerritEric Fried proposed openstack/nova master: Document device tags in support matrix  https://review.openstack.org/48147812:11
*** pchavva has joined #openstack-nova12:13
*** slaweq_ has joined #openstack-nova12:16
*** udesale has joined #openstack-nova12:16
*** MVenesio has joined #openstack-nova12:18
*** lyan has quit IRC12:19
*** slaweq_ has quit IRC12:21
*** acormier has joined #openstack-nova12:21
*** jamesden_ has joined #openstack-nova12:21
*** acormier has quit IRC12:22
*** acormier has joined #openstack-nova12:22
*** krtaylor has joined #openstack-nova12:23
*** jamesdenton has quit IRC12:24
*** sridharg has joined #openstack-nova12:24
*** diga has quit IRC12:24
*** acormier has quit IRC12:27
*** crushil_ has joined #openstack-nova12:30
*** yushb has joined #openstack-nova12:32
*** zzzeek_ has joined #openstack-nova12:32
*** crushil is now known as rushil12:33
*** crushil_ is now known as crushil12:33
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-baremetal-nodes endpoint instead of stevedore  https://review.openstack.org/46148012:37
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-security-groups instead of stevedore  https://review.openstack.org/47890112:37
openstackgerritAlex Xu proposed openstack/nova master: Remove the test for the route '/resources.:(format)'  https://review.openstack.org/47890012:37
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-security-group-rules instead of stevedore  https://review.openstack.org/47890312:37
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for server-security-groups instead of stevedore  https://review.openstack.org/47890212:37
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for image-metadata instead of stevedore  https://review.openstack.org/47849912:37
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-volumes instead of stevedore  https://review.openstack.org/47949712:37
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for images instead of stevedore  https://review.openstack.org/47849812:37
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-security-group-default-rules instead of stevedore  https://review.openstack.org/47890412:37
*** wasmum has quit IRC12:40
*** icey_ has joined #openstack-nova12:40
*** sballe__ has joined #openstack-nova12:40
*** zhenguo_ has joined #openstack-nova12:40
*** fmccrthy_ has joined #openstack-nova12:40
*** szaher has quit IRC12:40
*** cNilesh has quit IRC12:41
*** efried_ has joined #openstack-nova12:41
*** yamamoto has quit IRC12:41
*** fmccrthy has quit IRC12:41
*** ujjain- has quit IRC12:41
*** DinaBelova has quit IRC12:41
*** rfolco has quit IRC12:41
*** icey has quit IRC12:41
*** vipul has quit IRC12:41
*** zhenguo has quit IRC12:41
*** sballe_ has quit IRC12:41
*** efried has quit IRC12:41
*** melwitt has quit IRC12:41
*** rhagarty has quit IRC12:41
*** fmccrthy_ is now known as fmccrthy12:41
*** icey_ is now known as icey12:41
*** zhenguo_ is now known as zhenguo12:41
*** sballe__ is now known as sballe_12:41
*** melwitt has joined #openstack-nova12:41
*** vipul has joined #openstack-nova12:41
*** cNilesh has joined #openstack-nova12:41
*** ujjain has joined #openstack-nova12:41
*** ujjain has quit IRC12:41
*** ujjain has joined #openstack-nova12:41
*** wasmum has joined #openstack-nova12:41
*** melwitt is now known as Guest8620812:41
*** DinaBelova has joined #openstack-nova12:42
*** rfolco has joined #openstack-nova12:42
*** rhagarty has joined #openstack-nova12:42
*** yushb has quit IRC12:43
*** szaher has joined #openstack-nova12:44
*** dgonzalez has quit IRC12:44
alex_xunova api meeting is in 15 mins at #openstack-meeting-412:45
*** dgonzalez has joined #openstack-nova12:46
*** eharney has joined #openstack-nova12:47
*** liverpooler has joined #openstack-nova12:49
claudiubmoshele: hello. around? :)12:50
*** edmondsw has joined #openstack-nova12:50
mosheleclaudiub: hi12:51
claudiubmoshele: great. :) If you have a couple of minutes, I have a couple of questions regarding sr-iov12:51
mosheleclaudiub: ok12:51
claudiuband more explicitly, some implementation details.12:51
*** edmondsw_ has joined #openstack-nova12:51
*** edmondsw has quit IRC12:52
claudiubmoshele: so, whatever guides i've found online are kind of outdated, or do not contain all the information i need. :)12:52
*** jaypipes has joined #openstack-nova12:53
claudiubmoshele: well, first of all: let's say I have a NIC with SR-IOV capabilities, and I have a couple of free VFs. I'm already reporting them to nova in the virt driver as PCI devices with the type type-VF.12:53
mosheleclaudiub: yes if they are in the whitelist12:54
claudiuband they're whitelisted in the compute node's conf and nova-api's conf12:54
claudiubup to this point, that should be fine IMO.12:54
moshelewhitelisted in the compute conf not need in the nova api12:55
claudiubmoshele: aliased, sorry. :)12:55
claudiubaliased in the api conf file12:55
*** karimb has joined #openstack-nova12:55
moshelethe alise should be in nova-api and nova-cmpute12:55
claudiuboh, ok. i only knew about the alias in the nova-api. how does the alias help in the compute node?12:56
*** xinliang has quit IRC12:56
claudiubnow... I was thinking that I could claim those VFs the same way I would claim other PCI devices: through flavor extra_specs pci_passthrough:alias="the_alias:1"12:56
mosheleclaudiub: for resize and migration12:56
*** xinliang has joined #openstack-nova12:57
claudiubmoshele: hmm, meaning that the nova compute manager will check if the destination has those aliases as well?12:57
mosheleclaudiub: yes12:58
claudiubmoshele: hm, interesting. i did not know that. :) good to know.12:58
*** ralonsoh has quit IRC12:58
*** ralonsoh_ has joined #openstack-nova12:58
*** crushil has quit IRC12:59
*** sridharg has quit IRC12:59
*** gmann_ has joined #openstack-nova12:59
*** sridharg has joined #openstack-nova13:00
claudiubanyways, continueing my idea. so, I would end up spawning an instance, and that instance would have in the instance.pci_requests.requests an VF. Now, I've been told before that all I have to do is just plug that VF into the VM and that's it13:00
*** ralonsoh_ is now known as ralonsoh13:00
*** crushil has joined #openstack-nova13:01
claudiubbut that VF would not be tied to any neutron port, meaning that it there won't be normally any dhcp server serving it. I've been told that "the users would have to take care of that". is that ok?13:02
claudiubnor would it have security groups applied to it.13:02
*** sridharg has quit IRC13:02
openstackgerritRawan Herzallah proposed openstack/nova master: Adding NVMEoF for libvirt driver  https://review.openstack.org/48264013:07
*** yamamoto has joined #openstack-nova13:08
*** mdrabe has joined #openstack-nova13:10
claudiubmoshele: also, the alias is quite limiting, IMO. for example, I have 2 identical SR-IOV NICs, which means same vendor id and same product id, but they're actually connected to different networks. the alias config option only allows the following fields: vendor_id, product_id, dev_type, name (alias). which means that I don't have any way to specify which network I actually want. :)13:10
mosheleclaudiub: see this https://docs.openstack.org/mitaka/networking-guide/config-sriov.html13:10
*** zzzeek_ has quit IRC13:12
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'configuration' section  https://review.openstack.org/47847313:12
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'user' section  https://review.openstack.org/47847513:13
openstackgerritStephen Finucane proposed openstack/nova master: doc: Switch to openstackdocstheme  https://review.openstack.org/47775113:13
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'contributor' section  https://review.openstack.org/47847413:13
openstackgerritStephen Finucane proposed openstack/nova master: doc: Enable pep8 on doc generation code  https://review.openstack.org/48046913:13
openstackgerritStephen Finucane proposed openstack/nova master: doc: Rework README to reflect new doc URLs  https://review.openstack.org/48007413:13
openstackgerritStephen Finucane proposed openstack/nova master: doc: Start using oslo_policy.sphinxext  https://review.openstack.org/47935813:13
openstackgerritStephen Finucane proposed openstack/nova master: policies: Fix Sphinx issues  https://review.openstack.org/48051613:13
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'reference' section  https://review.openstack.org/47848413:13
openstackgerritStephen Finucane proposed openstack/nova master: doc: Remove dead plugin  https://review.openstack.org/47846913:13
openstackgerritStephen Finucane proposed openstack/nova master: doc: Rework index page per new sections  https://review.openstack.org/47848513:13
openstackgerritStephen Finucane proposed openstack/nova master: doc: Remove cruft from conf.py  https://review.openstack.org/47846813:13
openstackgerritStephen Finucane proposed openstack/nova master: doc: Remove dead files  https://review.openstack.org/47847013:13
openstackgerritStephen Finucane proposed openstack/nova master: doc: Use consistent author, section for man pages  https://review.openstack.org/48296013:13
openstackgerritStephen Finucane proposed openstack/nova master: doc: Start using oslo_config.sphinxext  https://review.openstack.org/48296113:13
*** zzzeek_ has joined #openstack-nova13:13
*** yamamoto has quit IRC13:13
mosheleclaudiub: there is SR-IOV for networking solution is work with the sriovnicswitch mechanism driver which is linux ip link based so maybe you need to add support there to windows13:13
*** zhurong has joined #openstack-nova13:14
*** efried_ is now known as efried13:14
cdentavolkov`++13:14
mosheleclaudiub:  what you talking is pci passthrough when neutron is not aware of the SR-IOV network interface13:14
stephenfinjaypipes: IF you get a chance to look through that doc-migration series again, I'd be mighty appreciative. Fixed the issue efried pointed out13:14
stephenfinSorry - gibi13:15
jaypipesstephenfin: you mean the one you just rebased and pushed above?13:15
*** cNilesh has quit IRC13:15
claudiubmoshele: are both cases viable?13:15
*** kylek3h has joined #openstack-nova13:15
*** mdnadeem has quit IRC13:16
stephenfinjaypipes: Yup https://review.openstack.org/#/q/topic:doc-migration+project:openstack/nova+status:open13:16
jaypipesstephenfin: sure thing :)13:16
efriedstephenfin So that wasn't intended to be to my attn?13:17
mosheleclaudiub: yes both of them should work13:17
stephenfinefried: Nope, as always you can ignore me :)13:17
*** slaweq_ has joined #openstack-nova13:17
claudiubmoshele: aham. interesting. well, i'll look into the neutron side as well then. :) thanks for the help. :)13:18
mosheleclaudiub: np :)13:18
*** moshele has quit IRC13:21
bauzasjaypipes: dansmith: sorry folks, was (again) on PTO yesterday13:21
*** trinaths has left #openstack-nova13:21
avolkov`stephenfin: placement-api-ref should also be moved to doc/source/api, I'll prepare patch when it's accomplished13:22
*** slaweq_ has quit IRC13:22
stephenfinavolkov`: I thought we weren't moving that yet?13:22
stephenfinDid plans change?13:22
cdentstephenfin: dhellman wants that to happen asap13:22
cdentso that we don't have to have a situation where there are publishing jobs that need to get changed13:22
cdent(there's a placement-api-ref publishing job pending)13:23
stephenfincdent: Ah, hadn't heard that update13:23
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'configuration' section  https://review.openstack.org/47847313:23
stephenfinSounds fair13:23
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'cli' section  https://review.openstack.org/47847213:23
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'user' section  https://review.openstack.org/47847513:23
openstackgerritStephen Finucane proposed openstack/nova master: doc: Switch to openstackdocstheme  https://review.openstack.org/47775113:23
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'contributor' section  https://review.openstack.org/47847413:23
openstackgerritStephen Finucane proposed openstack/nova master: doc: Enable pep8 on doc generation code  https://review.openstack.org/48046913:23
openstackgerritStephen Finucane proposed openstack/nova master: doc: Start using oslo_config.sphinxext  https://review.openstack.org/48296113:23
openstackgerritStephen Finucane proposed openstack/nova master: doc: Rework README to reflect new doc URLs  https://review.openstack.org/48007413:23
openstackgerritStephen Finucane proposed openstack/nova master: doc: Start using oslo_policy.sphinxext  https://review.openstack.org/47935813:23
openstackgerritStephen Finucane proposed openstack/nova master: policies: Fix Sphinx issues  https://review.openstack.org/48051613:23
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'reference' section  https://review.openstack.org/47848413:23
openstackgerritStephen Finucane proposed openstack/nova master: doc: Rework index page per new sections  https://review.openstack.org/47848513:23
openstackgerritStephen Finucane proposed openstack/nova master: doc: Remove dead files  https://review.openstack.org/47847013:23
jaypipesefried: stephenfin just wanted to wake you out of your slumber.13:23
* stephenfin accidentally squashed two patches. Sorry, gate13:23
*** rushil has quit IRC13:24
jaypipesralonsoh: I'm really liking the direction of your native OVSDB os-vif work.13:24
*** baoli has joined #openstack-nova13:28
stephenfinbauzas: Could I ask you to look at the doc-migration series starting with https://review.openstack.org/#/c/478468/ too?13:29
*** zzzeek_ has quit IRC13:29
bauzasstephenfin: roger this.13:29
stephenfinShouldn't take long, and it's easy re-arrange stuff later if we don't like the end result13:29
stephenfinbauzas: Ta :)13:29
ralonsohjaypipes: thanks! Also I added functional tests. YOu only need to have a live OVS running in the system to execute them13:30
jaypipesralonsoh: I saw those. ++13:30
*** fragatina has joined #openstack-nova13:31
*** frickler has quit IRC13:33
*** crushil has quit IRC13:34
*** zzzeek_ has joined #openstack-nova13:34
*** crushil has joined #openstack-nova13:35
openstackgerritStephen Finucane proposed openstack/nova master: Always set MTU when plugging devices  https://review.openstack.org/40878113:36
*** tbachman has quit IRC13:36
*** fragatina has quit IRC13:36
*** armax has joined #openstack-nova13:39
gibistephenfin: thanks for the rebase13:41
*** frickler has joined #openstack-nova13:41
*** gmann_ has quit IRC13:42
openstackgerritChason Chan proposed openstack/python-novaclient master: switch to openstackdocstheme  https://review.openstack.org/47775713:46
*** mriedem has joined #openstack-nova13:46
mriedemo/13:48
jaypipescdent: thx for your comments on https://review.openstack.org/#/c/482383/1. Answered. Please have a gander at the latest revision.13:49
*** yamamoto has joined #openstack-nova13:51
gibimriedem: good morning! I fixed the last outstanding issue with the BDM in notification patch https://review.openstack.org/#/c/448779/13:51
mriedemjaypipes: should i just remove this sentence? https://review.openstack.org/#/c/454322/35/nova/api/openstack/compute/rest_api_version_history.rst@59813:51
mriedemjaypipes: i think that would probably clear up the confusion in that paragraph13:51
jaypipesmriedem: yeah13:52
jaypipesmriedem: I'd appreciate that. sorry for not yet responding to you.13:52
*** kristia__ has quit IRC13:52
mriedemgibi: cool13:53
mriedemjaypipes: np13:53
mriedemgibi: i'll go through that today13:53
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/47709113:54
gibimriedem: thanks!13:55
*** xyang1 has joined #openstack-nova13:56
*** catintheroof has joined #openstack-nova13:57
*** frickler has quit IRC13:57
*** ZZelle has quit IRC13:57
*** frickler has joined #openstack-nova14:03
*** vladikr has joined #openstack-nova14:03
*** gcb has joined #openstack-nova14:04
*** kristian__ has joined #openstack-nova14:04
*** awaugama has joined #openstack-nova14:05
*** hongbin has joined #openstack-nova14:06
*** jangutter has quit IRC14:06
*** tylerderosagrund has joined #openstack-nova14:08
*** jmlowe has joined #openstack-nova14:09
*** kristian__ has quit IRC14:11
*** kristian__ has joined #openstack-nova14:12
*** ekuris_ has quit IRC14:13
openstackgerritStephen Finucane proposed openstack/python-novaclient master: doc: Switch from oslosphinx to openstackdocstheme  https://review.openstack.org/47775714:13
openstackgerritStephen Finucane proposed openstack/python-novaclient master: doc: Create directory structure for docs migration  https://review.openstack.org/48013514:13
openstackgerritStephen Finucane proposed openstack/python-novaclient master: Remove custom autodoc implementation  https://review.openstack.org/48026214:13
mriedemalex_xu: i understand your comments on the os-services change now; fwiw i care less about the wasted db call - this api won't be used often and i'd rather keep the complexity in here to a minimum14:16
*** coreywright has quit IRC14:16
*** acormier has joined #openstack-nova14:16
mriedembut i can investigate it, and add validation for the other error cases you pointed out,14:17
mriedemlike passing status=enabled and disabled_reason14:17
*** tbachman has joined #openstack-nova14:18
*** slaweq_ has joined #openstack-nova14:18
*** catintheroof has quit IRC14:20
*** tjones has joined #openstack-nova14:20
*** slaweq_ has quit IRC14:23
*** zzzeek has quit IRC14:24
*** gbarros has joined #openstack-nova14:25
*** moshele has joined #openstack-nova14:27
*** tbachman has quit IRC14:27
*** tbachman has joined #openstack-nova14:31
*** jangutter has joined #openstack-nova14:34
*** gouthamr has joined #openstack-nova14:35
*** itlinux has joined #openstack-nova14:37
*** coreywright has joined #openstack-nova14:37
*** crushil_ has joined #openstack-nova14:38
*** udesale has quit IRC14:38
*** jdillaman has quit IRC14:38
mriedemgibi: if you'll be around for 5 minutes i'll have something for you14:39
mriedemgibi: https://review.openstack.org/#/c/448779/14:40
mriedemquestions in there about why we aren't using the existing in scope BDMs when building the payload, like during instance.create14:40
mriedemseems that could be optimized to use the bdms we already have when we notify, rather than always look them up from the db - but maybe that's a later optimization14:40
gibimriedem: looking...14:42
*** mkucia has joined #openstack-nova14:43
gibimriedem: I think we can do some optimization on the loading of BDM. But I feel that it would be better to do that separately14:44
mriedemheh, of course you would :)14:45
mriedemalso https://review.openstack.org/#/c/448779/38/nova/tests/fixtures.py14:45
mriedemgibi: it's not even 5pm yet! :)14:45
stephenfinsean-k-mooney, moshele, jaypipes: What on earth is IVS, in the context of network interfaces?14:45
gibimriedem: true :)14:46
jaypipesstephenfin: it's Indigo (plumbgrid) stuff14:46
jaypipesplumgrid14:46
mriedemgibi: fwiw i'm ok with deferring the optimization part since loading bdms is disabled by default14:46
mriedembut i wanted to make sure we discussed it first14:46
jaypipesstephenfin: NIH OVS for NFV? :)14:47
gibimriedem: let's see if I can make the optimization this week then we can squash the two patches but if I run out of time then it would be nice to merge at least what we have already14:47
stephenfinjaypipes: So not an "initialization vector"? :) It's rather un-Googleable14:47
mriedemgibi: how about you put a TODO in the code in the InstanceAction init where it calls BlockDevicePayload.from_instance to investigate optimizing that by passing in any existing in-scope BDMs, rather than pull from the db14:47
jaypipesstephenfin: no. I mean, there's the ivshmem thing but that's DPDK-related.14:47
mriedemgibi: i'll +2 with the todo14:48
mriedemcheck out my question in the cinder fixture also14:48
jaypipesstephenfin: depends on the context I guess.14:48
gibimriedem: sure, I can do that til 5 pm :)14:48
mriedem\o/14:48
* stephenfin is back to figuring out where the nova os-vif boundary lies in nova/network/*.py files, and failing yet again14:48
stephenfinjaypipes: Cool cool. TA14:48
stephenfin*Ta14:48
*** itlinux has quit IRC14:49
*** nicolasbock has quit IRC14:50
jaypipesmriedem, dansmith, edleafe, bauzas: so do you think we should be returning alternate *destination hosts* or do you think we should be returning alternate *allocation requests*? I'm questioning my thoughts here. :(14:50
dansmithjaypipes: I thought the whole point was to store alternate allocation requests14:50
dansmithjaypipes: otherwise cell conductor has to formulate such a thing, without the details the scheduler has, and I don't think that's going to work14:51
dansmiths/store/return/14:51
dansmithso conductor can almost blindly throw the allocation request at placement, and if it sticks, send it along14:51
cdentI agree with dansmith. I seem to recall that's why we ended up down the allocation_candidates path in the first place?14:51
dansmithone of yeah14:51
jaypipesdansmith: yeah, that was my initial thought. I guess I've gotten bent out of shape on this latest patch series. :(14:52
jaypipesI think I need a braincation.14:52
*** zhurong has quit IRC14:53
cdentdansmith: since you're here, I was reading some cells docs and code: Is the main reason that discover_hosts doesn't do map_instances so that you can have granularity of timing and volume of change? Are there other reasons?14:53
dansmithcdent: we only needed map_instances to bridge between like mitaka and newton14:53
dansmithcdent: now instance maps are created when we create instances14:54
*** psachin has quit IRC14:54
dansmithcdent: and we would need it if we were grafting a cell from one deployment onto another or something like that, so I don't know that we should remove it necessarily14:54
cdentah, right, I see. thanks.14:54
cdentyeah, keeping it around seems a handy thing14:55
*** dtantsur is now known as dtantsur|afk14:55
openstackgerritBalazs Gibizer proposed openstack/nova master: Add BDM to InstancePayload  https://review.openstack.org/44877914:56
*** crushil has quit IRC14:56
*** vvargaszte has quit IRC14:57
*** crushil_ is now known as crushil14:57
openstackgerritGhanshyam Mann proposed openstack/nova master: Fix api-ref for instance action events's attribute as optional  https://review.openstack.org/48299414:58
*** david-lyle has quit IRC14:58
*** david-lyle has joined #openstack-nova14:58
moshelejaypipes: hi can you send an invite to my gmaiil moshe010@gmail.com ?14:58
ralonsohjaypipes: and rodolfo.ah@gmail.com14:58
bauzasjaypipes: allocation requests looks good to me14:59
*** prateek has quit IRC14:59
bauzasjaypipes: but I just think we need to make sure those allocation requests would be for RPs in the same cell, nope ?14:59
jaypipesmoshele, ralonsoh: done14:59
dansmithbauzas: yes, and he noted that15:00
jaypipesbauzas: yes, I agree with that. but I need to rework these patches. I got off track returning alternate host states instead of a dict of allocation requests.15:00
*** marst has joined #openstack-nova15:00
bauzasjaypipes: btw. I just left a question in https://review.openstack.org/#/c/482382/315:00
bauzasjaypipes: about why we would not use the current existing method15:00
jangutterjaypipes: If my work hangouts doesn't work, please yell. I can also do personal.15:01
bauzasjaypipes: dansmith: btw. sorry about me not reviewing the patches previously, I was (again) in PTO15:01
dansmithbauzas: I just assume you're on PTO unless you say otherwise15:01
*** rcernin has quit IRC15:01
bauzas.... and Friday is an holiday day :(15:01
bauzasso terrible15:02
bauzasI should be done with all my PTOs for last year FWIW15:02
*** zzzeek_ has quit IRC15:02
bauzasI sincerely apologize for that15:02
*** catintheroof has joined #openstack-nova15:03
*** zzzeek has joined #openstack-nova15:04
gibimriedem: added the TODO and fixed / answered the fixture comments in https://review.openstack.org/#/c/44877915:05
ftersinmdbooth: hi, thank you for reviewing. If you have the opportunity, pls, look at my comments there, i just asked some details for the reworking15:06
*** burt has joined #openstack-nova15:06
mriedemgibi: is the indent intentional here? https://review.openstack.org/#/c/448779/39/nova/tests/fixtures.py@127015:07
*** felipemonteiro has joined #openstack-nova15:07
gibimriedem: yes, I tried to explain the reason of that in my response to your comment15:08
*** felipemonteiro_ has joined #openstack-nova15:08
gibimriedem: either need the explicity detach in the test or I break the swap volume tests.15:08
mriedemok just checking15:09
gibimriedem: will propose a clean up on this fixture at some point15:09
*** nicolasbock has joined #openstack-nova15:10
mriedemok, +215:10
mriedemi left a comment for other reviewers too15:10
gibimriedem: thanks15:10
mriedema todo or note in there would also be good,15:10
mriedembut only if you respin15:10
gibimriedem: OK15:11
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2)  https://review.openstack.org/40895515:11
*** roeyc has quit IRC15:12
*** felipemonteiro has quit IRC15:12
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2)  https://review.openstack.org/43060815:12
*** karimb has quit IRC15:12
bauzasgibi: https://review.openstack.org/#/c/482497/1 is needed for what ?15:12
bauzasjust a clean-up right?15:13
gibibauzas: splited out from the BDM patch on top of it15:13
gibibauzas: it basically optimize out some data generation if the notification will not be sent out at the end15:13
bauzasgibi: oh nevermind, it was just something missed from https://review.openstack.org/#/c/428260/ ?15:14
gibibauzas: yes, from that and from other patches later on15:14
bauzasgibi: all good15:15
gibibauzas: thanks for the review15:15
*** yassine has quit IRC15:16
*** slaweq_ has joined #openstack-nova15:19
*** fragatina has joined #openstack-nova15:19
*** gbarros has quit IRC15:19
ildikovgibi: that fixture is a mess :(15:20
*** fragatina has quit IRC15:20
*** prateek has joined #openstack-nova15:20
*** fragatina has joined #openstack-nova15:20
*** gbarros has joined #openstack-nova15:21
*** tbachman has quit IRC15:21
gibiildikov: I have my sticker to clean up it, but first I will optimize out some db query related to the instance.create15:22
*** chyka has joined #openstack-nova15:22
*** lyan has joined #openstack-nova15:22
gibiildikov: or do you have some time to work on it? ;]15:23
ildikovgibi: I have a patch touching it with a note to figure something out there :)15:23
ildikovgibi: well, let's see who gets there first ;)15:23
gibiildikov: well, that will be a real fight for that juicy commit15:24
*** slaweq_ has quit IRC15:24
ildikovgibi: lol :)15:24
*** alexchadin has quit IRC15:25
mriedemgmann: i'm not sure i agree with this https://review.openstack.org/#/c/482994/15:26
mriedemit gets really confusing which is why i left it out of my patch15:26
*** chyka_ has joined #openstack-nova15:26
mriedemto mark 'events' as optional15:26
*** Oku_OS is now known as Oku_OS-away15:28
*** gszasz has quit IRC15:28
*** chyka has quit IRC15:29
*** tbachman has joined #openstack-nova15:29
bauzasgibi: may I ask a silly question about https://review.openstack.org/#/c/448779/39 ?15:30
*** belmoreira has quit IRC15:31
bauzasgibi: why some notifications samples provide an empty list for bdm notifications objects while others provide it?15:31
gibibauzas: sure15:31
gibibauzas: in some cases the bdm is overridden during the test15:32
gibibauzas: like test_volume_swap_server_with_error15:32
*** READ10 has joined #openstack-nova15:32
gibibauzas: or test_create_delete_server_with_instance_update15:33
*** mdrabe has quit IRC15:33
gibibauzas: in test_create_delete_server_with_instance_update the bdm appears only after the instance goes through the block device mapping step during the create15:33
*** tylerderosagrund has quit IRC15:34
gibibauzas: so the early instance-updates has no bdm but at that point it starts appearing15:34
gibibauzas: and I don't want to store different sample files for each steps15:34
*** edmondsw_ is now known as edmondsw15:34
*** Sukhdev has joined #openstack-nova15:34
*** yamahata has joined #openstack-nova15:35
*** gszasz has joined #openstack-nova15:36
*** andreas_s has quit IRC15:38
*** mdrabe has joined #openstack-nova15:38
*** mvk has quit IRC15:40
*** cdent has quit IRC15:40
gibibauzas: specially in the instance.create case we only attach the volume after the boot not during it15:40
*** kaisers_ has joined #openstack-nova15:41
*** liusheng has quit IRC15:42
mriedemVek: can you take a look at this? https://review.openstack.org/#/c/480392/15:42
mriedemor andreykurilin ^15:42
gibibauzas: disregard my statement about test_create_delete_server_with_instance_update15:43
gibibauzas: in that case the bdm is empty as there the volume also attached after the create15:43
*** liusheng has joined #openstack-nova15:43
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2)  https://review.openstack.org/40896415:44
andreykurilinlooking15:45
*** fragatina has quit IRC15:47
*** crushil has quit IRC15:47
*** sshwarts has quit IRC15:47
*** yamamoto has quit IRC15:48
openstackgerritMatt Riedemann proposed openstack/nova master: api-ref: mark instance action events parameter as optional  https://review.openstack.org/48079215:49
openstackgerritMatt Riedemann proposed openstack/nova master: Add ability to signal and perform online volume size change  https://review.openstack.org/45432215:49
*** yamamoto has joined #openstack-nova15:49
*** crushil has joined #openstack-nova15:49
*** yamamoto has quit IRC15:51
*** lucasagomes is now known as lucas-afk15:52
*** damien_r has quit IRC15:53
openstackgerritMatt Riedemann proposed openstack/nova master: Add ability to signal and perform online volume size change  https://review.openstack.org/45432215:53
*** jmlowe has quit IRC15:54
*** itlinux has joined #openstack-nova15:54
*** amotoki is now known as amotoki_away15:56
*** rajathagasthya has joined #openstack-nova15:56
*** cdent has joined #openstack-nova15:57
*** amotoki_away is now known as amotoki15:59
*** tbachman has quit IRC15:59
*** amotoki is now known as amotoki_away15:59
*** jamesden_ has quit IRC15:59
bauzasgibi: okay, that's reasonable, those notifications mutated ?16:00
gibibauzas: the sample is mutated during the test16:01
*** jamesdenton has joined #openstack-nova16:01
gibibauzas: but I'm not sure this is what you are after16:02
mriedemsean-k-mooney: ralonsoh: i wouldn't block on that out of band port update thingy, as i said, but i am not really an authority on that entire series of changes either as i haven't gone into it in detail,16:02
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004216:03
mriedemand my understanding of a lot of the pci stuff in nova is very limited16:03
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004216:03
bauzasgibi: it was just for my understanding16:03
bauzasgibi: I have all I need, I was just in a short meeting beforehand16:03
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004216:03
ralonsohmriedem: thanks for your comments!16:04
*** esberglu has quit IRC16:04
gibibauzas: OK. I will leave soon for today but just ping me tomorrow if you have questions16:04
bauzasgibi: that's not necessary :)16:04
*** tbachman has joined #openstack-nova16:05
*** tesseract has quit IRC16:05
openstackgerritTakashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host  https://review.openstack.org/41492616:05
openstackgerritTakashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host  https://review.openstack.org/41492616:05
gibibauzas: there are patches up to help keeping the style of the sample files consistent. I left the links under your comment16:07
bauzas++16:08
bauzasnot really super important, but for review time, it helps16:08
*** gcb has quit IRC16:09
gibibauzas: I agree16:10
*** gszasz has quit IRC16:11
*** crushil has quit IRC16:11
*** dtp has joined #openstack-nova16:13
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2)  https://review.openstack.org/45948316:14
*** annegentle has joined #openstack-nova16:14
*** roeyc has joined #openstack-nova16:15
*** esberglu has joined #openstack-nova16:18
*** damien_r has joined #openstack-nova16:18
*** damien_r1 has joined #openstack-nova16:19
*** jjung has joined #openstack-nova16:19
*** slaweq_ has joined #openstack-nova16:21
*** fragatina has joined #openstack-nova16:21
*** esberglu has quit IRC16:22
*** gszasz has joined #openstack-nova16:22
*** damien_r has quit IRC16:22
*** jjung has quit IRC16:24
*** slaweq_ has quit IRC16:25
*** esberglu has joined #openstack-nova16:27
mriedemjaypipes: given sdague was +2 on https://review.openstack.org/#/c/454322/ and is out for the rest of this week, if you're cool with it, want to just fast approve that one?16:28
jaypipesmriedem: done16:28
mriedemthanks16:28
*** kristian__ has quit IRC16:29
*** trinaths has joined #openstack-nova16:30
*** catintheroof has quit IRC16:30
*** catintheroof has joined #openstack-nova16:31
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request  https://review.openstack.org/45177716:32
*** ralonsoh has quit IRC16:32
*** annegentle has quit IRC16:33
*** annegentle has joined #openstack-nova16:34
*** damien_r1 has quit IRC16:34
*** damien_r has joined #openstack-nova16:35
*** lpetrut has quit IRC16:35
openstackgerritStephen Finucane proposed openstack/nova master: Remove plug_ovs_hybrid, unplug_ovs_hybrid  https://review.openstack.org/48303016:36
openstackgerritStephen Finucane proposed openstack/nova master: tests: Remove useless test  https://review.openstack.org/48303116:36
*** Guest86208 is now known as melwitt16:36
edleafe1/away16:38
edleafegeez16:38
*** mdrabe has quit IRC16:39
*** mdrabe_ has joined #openstack-nova16:39
*** Sukhdev has quit IRC16:41
*** mvk has joined #openstack-nova16:42
bauzasedleafe: 2/there16:43
edleafebauzas: :)16:43
* edleafe has fat fingers16:43
*** claudiub has quit IRC16:43
*** fragatina has quit IRC16:43
*** fragatina has joined #openstack-nova16:44
*** mdrabe_ has quit IRC16:46
*** mdrabe_ has joined #openstack-nova16:46
*** damien_r has quit IRC16:47
*** damien_r has joined #openstack-nova16:47
*** jamesden_ has joined #openstack-nova16:48
*** gjayavelu has joined #openstack-nova16:50
*** jamesdenton has quit IRC16:50
*** roeyc has quit IRC16:51
*** damien_r has quit IRC16:51
*** yamamoto has joined #openstack-nova16:51
*** pcaruana has quit IRC16:56
*** moshele has quit IRC16:56
openstackgerritJay Pipes proposed openstack/nova master: placement: add claim_resources() to report client  https://review.openstack.org/48238216:57
openstackgerritJay Pipes proposed openstack/nova master: placement: claim resources in filter scheduler  https://review.openstack.org/48238316:57
jaypipesedleafe, mriedem, bauzas, dansmith: ^^ squashed the "alternate destinations" patch into the claim resources in filter scheduler patch and re-ordered the patches so the add claim_resources() to report client can be reviewed ahead of the claim in scheduler patch.16:58
jaypipesstill need to handle the cells thing.16:58
*** jmlowe has joined #openstack-nova16:59
*** kristian__ has joined #openstack-nova16:59
*** fragatin_ has joined #openstack-nova17:01
mriedemok17:01
*** yamamoto has quit IRC17:02
*** fragatina has quit IRC17:04
*** gbarros has quit IRC17:08
*** eharney has quit IRC17:12
*** eharney has joined #openstack-nova17:13
mriedemgmann: https://review.openstack.org/#/c/467999/ needs some work17:17
mriedemit's merged but the docs are lacking17:17
*** weshay is now known as weshay|lunch17:20
*** weshay|lunch is now known as weshay17:20
edleafejaypipes: regarding filtering alternates by cell, feel free to take as much of this as works: https://review.openstack.org/#/c/465768/8/nova/scheduler/filter_scheduler.py@18117:21
*** slaweq_ has joined #openstack-nova17:21
*** tylerderosagrund has joined #openstack-nova17:21
*** claudiub has joined #openstack-nova17:22
*** lpetrut has joined #openstack-nova17:22
*** moshele has joined #openstack-nova17:23
*** Apoorva has joined #openstack-nova17:25
*** yamahata has quit IRC17:25
*** tylerderosagrund has quit IRC17:25
*** slaweq_ has quit IRC17:26
openstackgerritMerged openstack/python-novaclient master: Microversion 2.49 - Virt device tagged attach  https://review.openstack.org/48039217:27
*** jmlowe has quit IRC17:30
*** trinaths has left #openstack-nova17:31
*** kristian__ has quit IRC17:32
*** sambetts is now known as sambetts|afk17:32
*** tylerderosagrund has joined #openstack-nova17:43
*** rajathagasthya has quit IRC17:45
*** rajathagasthya has joined #openstack-nova17:46
*** gbarros has joined #openstack-nova17:46
*** fragatin_ has quit IRC17:47
*** jjung has joined #openstack-nova17:47
*** annegentle has quit IRC17:47
*** fragatina has joined #openstack-nova17:47
*** mkucia has quit IRC17:48
*** lpetrut has quit IRC17:49
*** gyee has joined #openstack-nova17:53
*** yamahata has joined #openstack-nova17:54
*** hoonetorg has quit IRC17:54
*** moshele has quit IRC17:56
*** jamesden_ has quit IRC18:00
*** jamesdenton has joined #openstack-nova18:01
*** yamamoto has joined #openstack-nova18:04
*** annegentle has joined #openstack-nova18:08
*** hoonetorg has joined #openstack-nova18:10
*** lpetrut has joined #openstack-nova18:10
*** READ10 has quit IRC18:15
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: WIP: Microversion 2.50 - fix quota class sets resource usage  https://review.openstack.org/48306018:16
*** yamamoto has quit IRC18:20
*** yamamoto has joined #openstack-nova18:21
*** gbarros has quit IRC18:22
*** slaweq_ has joined #openstack-nova18:22
*** gbarros has joined #openstack-nova18:22
*** yamamoto has quit IRC18:26
*** slaweq_ has quit IRC18:27
*** gbarros has quit IRC18:27
*** MVenesio has quit IRC18:32
cdentedleafe: filtering rps by traits: https://review.openstack.org/#/c/474602/ still alive?18:36
gheorghe_is there something wrong with migrations on ocata? my VMs are on the same host after migration...always...18:39
mriedemgheorghe_: are there errors in the logs?18:40
mriedemare you doing cold migration, live migration, or resize?18:40
*** efoley has quit IRC18:40
mriedemthere are no known issues, upstream integration test for migrations is fine18:40
*** lpetrut has quit IRC18:41
rm_workHey, I'm about to look into making keypairs be project-level scope instead of user+project18:43
rm_workJust wondering if there is some easy way to just enable this18:43
rm_workor if someone has already done this before18:43
rm_work(not necessarily to push upstream, we just need this in our cloud)18:43
rm_work^^ before anyone freaks out18:44
mriedemrm_work: ask melwitt - quotas could be a problem18:44
rm_workwe currently have quotas as -1 for keypairs18:44
rm_workso i'm not especially concerned18:44
rm_workbut i'll take that note down18:44
*** tylerderosagrund has quit IRC18:44
*** tylerderosagrund has joined #openstack-nova18:45
gheorghe_mriedem i am doing live migration, with block and without block (i also have some VMs on iscsi target) and in both cases the migration is not reporting any errors, but the VM stays on the same host18:45
*** Sukhdev has joined #openstack-nova18:45
mriedemgheorghe_: then something is failing18:45
rm_workmelwitt: have you worked on keypairs per project?18:45
mriedemand it's either rolling back to the source host, or it's never even getting to the destination host18:45
melwittrm_work: what do you mean by make it project scope instead of project+user? do you mean set a limit across a project and count allocated keypairs only across a project?18:46
gheorghe_mriedem, i don't see anything in /var/log/libvirt/qemu# so i suspect something is failing on nova level18:46
rm_workmelwitt: I mean, if user A on project X uploads a keypair, we want user B on project X to also see that keypair in that project18:47
rm_workand be able to use it18:47
melwittoh, I see18:47
rm_workin most other clouds, this is the way key auth works18:48
*** lpetrut has joined #openstack-nova18:48
rm_workand users are incredibly confused when they set up keypairs and then their service account or whatever goes to create a VM and errors out because the keypair isn't there (but THEY see it)18:48
rm_workand rather than try to educate/correct an ongoing set of users who have experience with other clouds or make an expectation based on how *everything else* works, we'd rather just ... enable the functionality they expect, because I can't see a good reason not to :P18:49
rm_workit's just public keys...18:49
*** jmlowe has joined #openstack-nova18:49
rm_workmy public key is posted on my website...18:49
mriedemgheorghe_: sure, and that should be in the /var/log/nova logs18:50
*** Swami has joined #openstack-nova18:50
rm_workmelwitt: have you worked on anything like this?18:50
melwittrm_work: yeah, use case makes sense. I haven't heard of anyone doing it before though. I'm not sure what it would take, whether it would be hard. AFAIK it can't just be easily enabled18:50
rm_workkk18:50
gheorghe_mriedem i get only info messages within /var/log/nova ... if you want, i can try i migration now and give you the nova logs. do you want to see the ones from controller or the ones from kvm?18:50
*** rajathagasthya has quit IRC18:50
mriedemgheorghe_: i don't want to debug your logs no18:51
edleafecdent: yes, the filtering by traits is still alive. It can also be adapted for filtering the new allocation candidates method.18:51
mriedemgheorghe_: run it with debug=True in the nova.conf on all nodes, computes and controller18:51
mriedemand then i'd trace the request id through the nova-api, conductor and compute logs18:52
gheorghe_request id returns only an info message18:52
mriedemthe request id is logged with all messages for the given request context18:52
mriedemso if you have debug logging enabled, you should be able to see it there also18:52
mriedeme.g. http://logs.openstack.org/90/481290/3/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/fcc4b2f/logs/screen-n-cpu.txt.gz#_Jul_08_18_07_57_38461718:53
gheorghe_omg mariedem i actually found something18:53
mriedemreq-df8aac96-7d4f-4652-9237-d6da65164ee5 is the request id18:53
gheorghe_https://paste.gnome.org/pg7hkufiz18:54
gheorghe_i will troubleshoot this, brb18:54
*** raunak has joined #openstack-nova18:54
mriedemyup that would make it fail18:54
*** mdrabe_ has quit IRC18:54
*** gbarros has joined #openstack-nova18:55
*** mdrabe has joined #openstack-nova18:55
gheorghe_kvm-04 is not even listening on 1650918:56
*** annegentle has quit IRC18:57
rm_workmelwitt: ah, I was wrong about keypairs already being associated with a project18:59
rm_workthey are purely user associated at the moment :(18:59
gheorghe_mriedem does it work only with insecure conn? https://paste.gnome.org/pfdfgbkaj18:59
rm_workso yeah pretty big change I think T_T18:59
*** baoli has quit IRC18:59
*** baoli has joined #openstack-nova19:00
*** baoli has quit IRC19:00
*** baoli has joined #openstack-nova19:01
*** annegentle has joined #openstack-nova19:02
*** tylerder_ has joined #openstack-nova19:03
melwittrm_work: *looks* yeah, I didn't know project_id isn't even part of the data model. so yeah, much bigger change than I was thinking too19:03
mriedemgheorghe_: there are lots of different ways to configure it i think,19:03
mriedemdevstack uses ssh19:03
mriedemyou could also run in tunneled mode but that slows it down19:04
gheorghe_mriedem this is not devstack, this is the tutorial with 1 controller and 4 kvm hosts19:04
mriedemsure19:05
mriedemthat's not my point though19:05
mriedemalso, which tutorial?19:05
mriedemand https://docs.openstack.org/security-guide/instance-management/security-services-for-instances.html#instance-migrations19:05
*** annegentle has quit IRC19:06
*** tylerderosagrund has quit IRC19:06
mriedemalso https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux_openstack_platform/7/html-single/migrating_instances/19:06
jangutterjaypipes: so I got succesful plugging with an external os-vif + neutron plug-in.19:08
jaypipesjangutter: \0/19:08
jangutterjaypipes: Yep. Nova changes were tiny. There are two hitches though.19:08
jangutterjaypipes: one is that stashed the PCI address inside the vhost-user socket path.19:09
jangutterjaypipes: and the second is that there's no filtering for the Agilio NICs. it's basically picking anything that says it's an ethernet VF.19:09
jangutterjaypipes: the second is the one we know about.19:09
jaypipesjangutter: put the Netronome vendor_id and product_id in the pci_passthrough_whitelist.19:09
jangutterjaypipes: yep, that I did. And it's working as expected.19:10
jaypipesjangutter: I don't understand... you said it's not filtering?19:10
jangutterOh, I mean that if you have two sets of NICs in the same system, things will go... unexpected.19:11
*** jmlowe has quit IRC19:11
jangutterjaypipes: this is probably good enough (tm) for now in any case, if stashing the PCI address inside a socket path is acceptable.19:12
jaypipesjangutter: you mean if there's, say, a mlnx nic and a netronome nic, and you allow guests to use either of those, there's no way to select which NIC for a flavor, right?19:12
*** raunak has quit IRC19:13
jangutterjaypipes: correct.19:13
jaypipesjangutter: if both are connected to the same physical network, yes, unfortunately that's the case right now. until we land ralonsoh's patch for NIC capabilities.19:13
jangutterjaypipes: yep, so I'm happy to live with that for a first stab.19:14
gheorghe_mriedem i am using ubuntu, not redhat19:14
*** moshele has joined #openstack-nova19:15
jaypipesjangutter: k, good with me.19:16
jaypipesjangutter: happy to hear you've made some good progress!19:16
*** yamamoto has joined #openstack-nova19:16
*** nicolasbock has quit IRC19:16
*** harlowja has quit IRC19:17
gheorghe_mriedem i got libvirt to listen but none of the tutorials online helped :D i just had to dig through config files19:17
*** david-lyle has quit IRC19:17
*** david-lyle has joined #openstack-nova19:17
*** abalutoiu_ has joined #openstack-nova19:18
jangutterjaypipes: If I can get sean-k-mooney's way to set bridge name added, that means we only add the VIF_TYPE to nova.network.model, the associated _nova_to_osvif_vif_agilio_ovs() to os_vif_util19:18
jangutterjaypipes: (and moshel's _set_config_VIFHostDevice) of course.19:19
*** kristian__ has joined #openstack-nova19:19
jaypipesjangutter: right. the fewer changes, the better :)19:20
*** adisky__ has quit IRC19:20
jangutterjaypipes: anyway, I'll do my best to get the code uploaded ASAP.19:20
jaypipesjangutter: awesome, thanks!19:21
*** abalutoiu has quit IRC19:22
*** slaweq_ has joined #openstack-nova19:23
*** slaweq_ has quit IRC19:26
*** slaweq_ has joined #openstack-nova19:26
*** tbachman has quit IRC19:27
*** crushil has joined #openstack-nova19:28
*** crushil has quit IRC19:28
*** annegentle has joined #openstack-nova19:30
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: WIP: Microversion 2.50 - fix quota class sets resource usage  https://review.openstack.org/48306019:30
gheorghe_mriedem it works now19:32
*** crushil has joined #openstack-nova19:39
*** awaugama has quit IRC19:40
mriedemgheorghe_: cool - if you have something to fix up the config options help text within nova, that would be appreciated19:43
*** slaweq_ has quit IRC19:43
*** jjung has quit IRC19:43
mriedemor was it libvirt config that you changed?19:43
gheorghe_i will show you my config for deploying hosts19:43
*** slaweq_ has joined #openstack-nova19:43
gheorghe_it configs nova AND libvirt for migration19:43
gheorghe_wait a sec19:43
gheorghe_check at the bottom: https://paste.gnome.org/p9vruj0zw ... as you will see here, this is the unsecure resolution. but it will enable you to migrate. also, it greps IP from a subnet deployed within my test environment19:44
gheorghe_i just this on every kvm node and it enables nova-compute and migration for kvm19:45
gheorghe_*i just RUN this19:45
*** slaweq_ has quit IRC19:48
*** slaweq_ has joined #openstack-nova19:49
*** crushil has quit IRC19:51
*** lucasxu has joined #openstack-nova19:51
*** tblakes has joined #openstack-nova19:51
*** mdrabe_ has joined #openstack-nova19:53
*** gszasz has quit IRC19:53
*** mdrabe_ has left #openstack-nova19:53
*** moshele has quit IRC19:54
*** yamamoto has quit IRC19:54
*** crushil has joined #openstack-nova19:56
*** baoli has quit IRC19:56
*** tblakes has quit IRC19:59
*** FL1SK has joined #openstack-nova20:00
*** baoli has joined #openstack-nova20:00
*** baoli has quit IRC20:00
*** harlowja has joined #openstack-nova20:02
*** moshele has joined #openstack-nova20:04
*** baoli has joined #openstack-nova20:07
*** lucasxu has quit IRC20:08
*** liverpooler has quit IRC20:10
*** cdent has quit IRC20:10
*** jdandrea has joined #openstack-nova20:11
*** tbachman has joined #openstack-nova20:13
jdandreaFor "nova boot" how does one specify metadata with a dictionary value?20:14
jdandreaTrying --meta foo='{"bar": ["baz"]]}}' and it seems to arrive in the scheduler plugin's instance properties as a dict of the form {'foo': '{bar:[baz]}'} instead.20:14
jdandreaWhich is not valid JSON nor YAML. Maybe I'm doing something wrong on the cmd line?20:14
jdandrea(I should say the value for foo isn't valid JSON nor YAML.)20:15
openstackgerritMerged openstack/nova master: doc: Remove cruft from conf.py  https://review.openstack.org/47846820:15
*** crushil has quit IRC20:16
*** rajathagasthya has joined #openstack-nova20:17
*** tylerder_ has quit IRC20:18
mriedemjdandrea: i think you specify --meta multiple times20:19
jdandreamriedem - how do you do that to reference a nested key though?20:19
jdandreaSo I'd say --meta level_1_key=... something and then somehow refer to a level below level_1_key?20:20
mriedemoh, no20:20
*** abalutoiu_ has quit IRC20:20
jdandreaI want to provide a value that is itself a dict. So I figure I'd send in a JSON string enclosed in single quotes, but that doesn't quite work.20:20
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: WIP: Microversion 2.50 - fix quota class sets resource usage  https://review.openstack.org/48306020:20
jdandreaAnd I can't remove the single quotes without it going sideways.20:20
jdandreaMy problem is what I get on the inside doesn't appear to be valid JSON or YAML, so I can't even string load it.20:21
mriedemthe CLI isn't expect the value to be itself a dict, so it does a single split and assumes it's a string, which it is - and it's not going to unpack that20:23
mriedemreturn dict(v.split('=', 1) for v in metadata)20:23
mriedemis what the cli code is doing20:23
jaypipesjdandrea: who let you in here!? :P20:24
jdandreajaypipes - STOP FOLLOWING ME. :p20:24
jaypipeshehe20:24
jdandrealol20:24
*** tylerderosagrund has joined #openstack-nova20:24
jdandreamriedem Right, that makes sense. So the trick is to encode it such that it lands in the plugin as something that can be converted sanely.20:25
openstackgerritMerged openstack/nova master: doc: Use consistent author, section for man pages  https://review.openstack.org/48296020:25
openstackgerritMerged openstack/nova master: doc: Remove dead plugin  https://review.openstack.org/47846920:25
*** tylerderosagrund has quit IRC20:26
*** tylerderosagrund has joined #openstack-nova20:26
mriedemjdandrea: hmm, do you need to encode it somehow?20:26
mriedemi'm not sure what's goofing it up when it gets over into the API20:26
jdandreamriedem Yeah, I'm not either. Pondering.20:27
mriedemurlencode?20:27
jdandreaMayyyyybe.20:27
jdandreaPlausible.20:27
mriedemi'm totally guessing20:27
openstackgerritMerged openstack/nova master: api-ref: mark instance action events parameter as optional  https://review.openstack.org/48079220:27
jdandreaBut messy, I admit.20:27
jdandreaGonna hack on this a li'l bit ...20:27
jdandreabiab20:27
* jdandrea scurries away again :)20:28
melwittguh, more random "OpenStackApiAuthenticationException: Authentication error" functional test fails20:29
melwittgonna have to figure that out soon20:29
melwittin the gate20:32
mriedemeventlet20:32
* melwitt shakes fist20:32
mriedemit's something with the compute api and placement api client helper things in the functional tests getting screwed up over eventlet, i think20:33
mriedemcdent had a patch at one point for a related issue where he put a yield in there20:33
melwittoh, that sounds familiar20:33
mriedemi'm sure i can find it20:33
mriedemlike a hog hunting for truffles20:33
* mriedem makes hog sounds20:33
melwittlol20:33
*** moshele has quit IRC20:36
*** raunak has joined #openstack-nova20:37
mriedemhttps://review.openstack.org/#/c/459735/ is what we went with20:40
mriedemhttps://review.openstack.org/#/c/459733/ was cdents20:40
melwitthm20:41
mriedemmaybe we should set that globally,20:41
mriedemif it's failing on other tests that aren't using the integrated helpers stuf20:42
mriedem*stuff20:42
melwittoh yeah, lemme see what this one was20:42
mriedemi had one earlier here http://logs.openstack.org/22/454322/37/check/gate-nova-tox-functional-ubuntu-xenial/76cbda6/console.html#_2017-07-12_17_44_14_51488420:42
melwittapi_sample_tests, thought those were using integrated_helpers http://logs.openstack.org/74/467774/7/gate/gate-nova-tox-functional-ubuntu-xenial/4a00841/testr_results.html.gz20:42
mriedemapi sample tests are yeah20:42
*** crushil has joined #openstack-nova20:43
mriedemmine was something else20:43
mriedemNoValidHost20:44
mriedemi think because the compute node never checked in20:44
mriedembecause of some eventlet timeout20:44
*** pchavva has quit IRC20:44
* jdandrea wanders in20:48
jaypipesmriedem, dansmith, edleafe: did we want to return a list of alternates *for each instance in num_instances*? of a single list of alternate allocation requests for the entire request?20:48
jaypipess/of a single/or a single/g20:48
jdandreamriedem jaypipes So what I did was look at the value and if it is a string, I try to json.loads the thing. If that fails, I leave it as is. Mischief managed.20:48
dansmithjaypipes: well, if all of them can be rescheduled max_attempts time, then it needs to be num_instances * max_attempts20:49
dansmithjaypipes: but we could also take some percentage or something, and provide some number of bet-hedging alternates, but not enough for all of them to be rescheduled max times20:49
*** smatzek has quit IRC20:49
jaypipesdansmith: no, sorry, I meant should I return a single list of alternate allocation requests for the entire call to select_destinations() or should I return a list of alternate allocation requests for each host returned by select_destinations()?20:50
dansmithjaypipes: independent of the number?20:51
dansmithjaypipes: an alternate has to go with a given primary host, because it has to be the same cell,20:51
dansmithso a single list of num*max alternates won't help us once we lose context of which host is in which cell20:51
jaypipesright20:51
jaypipesok, that answers that20:51
melwittcells meeting in 8 minutes20:52
dansmithmelwitt: correction20:52
dansmithmelwitt: short-ass cells meetinglet in 8 minutes20:52
*** penick has joined #openstack-nova20:54
melwittmeetinglet, I like it20:54
*** yamamoto has joined #openstack-nova20:54
edleafejaypipes: yeah, I think that a single list per instance is best, where each list is the selected (claimed) host followed by the alternates.20:56
efriedIn Portuguese, meetazinho.  In Spanish, meetito.  In French, meetette.  In German, meetchen.20:56
dansmithooh, meetito is great20:57
dansmithlike dorito, but less yummy20:57
efriedNothing is less yummy than a cooler ranch dorito20:57
jaypipesedleafe: I'm going for a dict, keyed by instance_uuid, of two-tuples where the tuple is (WeighedHost(claimed_host), list(alternate allocation requests))20:57
jaypipesedleafe, dansmith: ^^ make sense to you?20:57
dansmithsure20:58
efriedjaypipes Is this yet another thing we're going to regret not being an object later?20:59
melwittefried: phew, that was close. I misread at first that you were saying cooler ranch was the best dorito20:59
efriedmelwitt Rest easy.20:59
edleafejaypipes: this is what scheduler is returning to the conductor?21:00
jaypipesedleafe: well, not *yet*... I'm currently referring to what filter scheduler _schedule() returns to filter scheduler's select_destinations() call.21:00
jaypipesedleafe: still need to figure out what's best for the RPC changes.21:00
jaypipesedleafe: it *may* be that structure...21:01
jaypipeshaven't fully thought the RPC layer changes through yet21:01
*** yamamoto has quit IRC21:01
edleafejaypipes: I'm still not clear why the selected host has to be in a separate structure than the alternates21:01
jaypipesedleafe: the alternates are meant to be opaque structures. the selected host is the host identifier. the allocation requests are alternate things that can be passed to placement to claim resources if the original destination host fails to fully launch the instance.21:03
jaypipesedleafe: or are you asking to just have a list where the first element of the list is a WeighedHost and the other elements of the list are allocation request blobs?21:04
*** Sukhdev has quit IRC21:04
edleafejaypipes: the distinction seems unnecessary. Why not send a list of (host, alloc) for each instance, where the first item in the list is the selected host? This way, if the build fails, the cell conductor can easily unclaim the resources already claimed21:05
jaypipesdansmith: thoughts? ^21:06
dansmithjaypipes: we're in the cells meeting right now21:06
jaypipesoh, sorry.21:06
edleafejaypipes: passing a heterogenous structure seems like it will lead to a lot of conditional blocks to handle the different cases21:06
jaypipesedleafe: I don't disagree with you on that21:07
*** dave-mccowan has quit IRC21:09
*** dgonzalez_ has joined #openstack-nova21:10
*** dgonzalez_ has quit IRC21:12
*** jmlowe has joined #openstack-nova21:12
*** edmondsw has quit IRC21:12
*** edmondsw has joined #openstack-nova21:13
*** edmondsw_ has joined #openstack-nova21:14
*** edmondsw has quit IRC21:14
*** tylerderosagrund has quit IRC21:14
*** edmondsw_ has quit IRC21:15
*** lennyb has quit IRC21:15
*** edmondsw has joined #openstack-nova21:15
*** kylek3h has quit IRC21:19
*** edmondsw has quit IRC21:20
dansmithjaypipes: I would expect the compute to un-claim before passing the reschedule attempt to conductor, since it will un-claim always on failed boot, and then punt to conductor if it thinks it should be retried21:22
mriedemdansmith: dtp: melwitt: https://etherpad.openstack.org/p/nova-pike-cells-v2-todos21:23
mriedemi'll start dumping21:23
melwittcool, thanks21:23
dansmithjaypipes: so I'm not sure that we need anything too complicated, and was kinda expecting the conductor to do the next claim in order, and then take the would-be-versioned _response_ from placement as an indication of which compute node to use21:23
*** thorst has quit IRC21:23
dansmithjaypipes: however, if passing (hostname, allocation) around is more convenient or symmetric then that's fine21:24
dansmithjaypipes: your select-destinations can't return a HostState outside of the scheduler though21:24
jaypipesdansmith: yeah, I think edleafe's suggestion is a good one.21:24
dansmithjaypipes: I'm not sure what he's suggesting, other than just making it a list where the implication is that we try the first one as the "selected host"21:25
dansmithI assumed your example of the weighedhost as the selected one was for efficiency of the next step in the process21:25
*** efried has quit IRC21:25
edleafedansmith: yeah, that's pretty much it :)21:25
jaypipesdansmith: he's suggesting returning a list of (host, allocation_request) tuples for each instance_uuid in the request21:26
dansmithobviously that's the most you can return from the rpc call, if that's what you're talking about, but you said WeighedHost() above, so I figured you were talking internal still21:26
*** lennyb has joined #openstack-nova21:27
dansmithbut yeah, I think I said before that just returning a list of destinations per instance, and processing them in order in each phase is the way I think of it21:27
dansmithlike, I would expect each round to just pop the first one off the list,21:27
dansmithtry it, and move on21:27
dansmithdoing claim in conductor made that super symmetric21:27
dansmithdoing it in scheduler means it's pre-claimed before we return, which is what I was arguing against in the first place :)21:28
jaypipesdansmith: but we can't claim in the conductor when we have num_instances > 0 unfortunately. see: instance groups, affinity, and similar issues.21:30
*** lpetrut has quit IRC21:30
edleafejaypipes: so assuming a retry is necessary, how will the cell conductor deal with those things?21:31
jaypipesedleafe: unfortunately, the only way to deal with those things on retry is the last-minute check on the alternate target host :(21:32
dansmithjaypipes: I don't think I understand why that would be, but you guys steamrolled past that anyway, so it doesn't matter21:32
jaypipesdansmith: because of this line: https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L127-L12821:33
jaypipesdansmith: those little lines are the bane of my existence.21:33
dansmithI don't see how that affects claiming in the conductor, but whatever, not important21:34
*** efried has joined #openstack-nova21:38
*** awaugama has joined #openstack-nova21:40
*** lyan has quit IRC21:40
*** zzzeek_ has joined #openstack-nova21:42
*** edmondsw has joined #openstack-nova21:46
*** catintheroof has quit IRC21:46
*** edmondsw has quit IRC21:50
*** armax has quit IRC21:51
*** felipemonteiro_ has quit IRC21:52
*** armax has joined #openstack-nova21:52
mriedemmelwitt: so i didn't get to the resize stuff yet https://review.openstack.org/#/c/416521/ but with fresh eyes i think i've found 2 issues already discussed about over quota on multiple create not cleaning up, and restore21:52
*** eharney has quit IRC21:54
*** jmlowe has quit IRC21:54
*** armax has quit IRC21:56
*** tylerderosagrund has joined #openstack-nova21:57
*** Apoorva has quit IRC21:58
melwittmriedem: hm, okay. will look at that22:04
*** Apoorva has joined #openstack-nova22:04
*** Apoorva has quit IRC22:04
*** burt has quit IRC22:08
*** Apoorva has joined #openstack-nova22:10
*** esberglu has quit IRC22:11
*** jamesdenton has quit IRC22:14
*** lyan has joined #openstack-nova22:14
*** jamesdenton has joined #openstack-nova22:14
*** zzzeek_ has quit IRC22:15
*** tylerderosagrund has quit IRC22:17
*** zzzeek_ has joined #openstack-nova22:17
*** zzzeek has quit IRC22:18
*** raunak has quit IRC22:19
*** jamesdenton has quit IRC22:19
*** penick has quit IRC22:19
melwittmriedem: replied about restore. reading through multi create22:21
*** slaweq_ has quit IRC22:23
*** kristian__ has quit IRC22:26
*** kristian__ has joined #openstack-nova22:26
melwittmriedem: though now I see the change to state "soft-deleted" doesn't happen until after an rpc call to compute, so I think that's why I didn't bother with recheck_quota there22:28
melwittmaybe I should change it to set the state right away before the cast. not sure if that would be okay22:28
*** tylerderosagrund has joined #openstack-nova22:30
*** sapcc-bot3 has joined #openstack-nova22:31
*** carthaca_1 has joined #openstack-nova22:31
*** d34dh0r53 has quit IRC22:32
*** liangy has joined #openstack-nova22:32
*** sapcc-bot3 has quit IRC22:32
*** carthaca_1 has quit IRC22:32
*** artom has quit IRC22:32
*** sapcc-bot has quit IRC22:32
*** carthaca_ has quit IRC22:32
*** kristian__ has quit IRC22:32
*** carthaca_2 has joined #openstack-nova22:32
*** sapcc-bot4 has joined #openstack-nova22:32
*** d34dh0r53 has joined #openstack-nova22:32
*** rm_work has quit IRC22:32
*** lyan has quit IRC22:32
*** rm_work has joined #openstack-nova22:32
*** artom has joined #openstack-nova22:33
*** anish has quit IRC22:34
*** rhagarty has quit IRC22:35
*** raginbajin has quit IRC22:35
*** csuttles has quit IRC22:35
*** anish has joined #openstack-nova22:35
*** tylerderosagrund has quit IRC22:35
*** slagle has quit IRC22:35
*** csuttles has joined #openstack-nova22:35
*** slagle has joined #openstack-nova22:35
*** raunak has joined #openstack-nova22:35
*** sambetts|afk has quit IRC22:36
*** szaher has quit IRC22:37
*** szaher has joined #openstack-nova22:37
mriedemok still going through some other parts of this22:37
*** rhagarty has joined #openstack-nova22:37
*** sambetts_ has joined #openstack-nova22:37
openstackgerritMaciej Kucia proposed openstack/nova-specs master: Extended PCI alias  https://review.openstack.org/47421822:38
*** claudiub has quit IRC22:38
*** raginbajin has joined #openstack-nova22:42
mriedemok reading your replies now22:42
*** thorst has joined #openstack-nova22:45
*** thorst has quit IRC22:45
mriedemhmm, so when you soft delete an instance,22:45
mriedemthe deleted_at field is set on the instance22:45
mriedembut when you restore it, i don't see the deleted_at field set to None22:46
mriedemso it's both ACTIVE and deleted_at22:46
melwitt*thumbs up*22:46
mriedemunless i'm missing something22:46
melwittI doubt it22:46
melwitteither it was missed or someone thought it helpful to let it show that it had been deleted in the past22:47
mriedemi don't have a devstack handy22:47
mriedemyeah....but instance actions...22:47
mriedemif you want an audit record22:47
melwittyeah22:47
openstackgerritVipin Balachandran proposed openstack/nova master: VMware:Use power_state const for power state check  https://review.openstack.org/48312222:49
*** tylerderosagrund has joined #openstack-nova22:50
*** yamamoto has joined #openstack-nova22:52
mriedemmelwitt: ok, replies are all in - still haven't gotten t oresize22:53
mriedem*to resize22:53
mriedemto summarize, i think we're ok with restore, except maybe that we don't have a recheck deltas in there?22:53
mriedemand i also got a bit hung up on the fact we aren't accounting for the hw video ram when counting existing usage22:53
mriedemas dansmith pointed out22:53
mriedemfor some reason i was ok with that last night, but now i'm not so sure22:53
melwittmriedem: yeah, I think restore is fine. we might probably want to move the vm_state change to compute/api and do a recheck yeah22:54
mriedemoh i see,22:55
melwittmriedem: yeah, for the majority of the time, I think we were counting video ram on allocate and not on deallocate. someone recently fixed a bug to add subtraction of video ram usage upon deallocate22:55
mriedembecause today it's (1) check, (2) change state (api or compute), and rechecking at 1.5 wouldn't do anything since the state didn't change to affect the count,22:55
mriedemso to recheck, we'd have to (1) check, (2) change state, (3) recheck, (4) cast22:55
melwittyes22:55
mriedemand older computes are setting the vm state to ACTIVE but that shouldn't cause a problem22:56
mriedemif the api did it i mean22:56
mriedemit's a bit of a lie though...for example, if we set the state to ACTIVE, counted, and then cast to compute, but the rpc message never got there22:56
mriedemwe'd say the instance is active even though the guest is dead22:57
melwittoh yeah. maybe it could be a problem if an old compute goes to set SOFT_DELETED and it's already SOFT_DELETED (if it expected a different vm_state)22:57
mriedemi think vm_state is ok,22:57
mriedemyou can set that to ACTIVE twice and it's fine22:58
mriedemthe problem is task_state22:58
mriedeminstance.save(expected_task_state=[task_states.SOFT_DELETING])22:58
mriedemand setting vm_state=ACTIVE while task_state is SOFT_DELETING is a problem,22:58
melwittoh, guh sorry getting it backward. restore sets to active22:58
mriedembecause when an instance is ACTIVE it's task_state is supposed to be None22:58
*** armax has joined #openstack-nova22:58
mriedemright22:58
*** acormier has quit IRC22:58
mriedemyou would almost have to temporarily change the state to active for the recheck, and then set the vm_state back to SOFT_DELETED for the cast to the compute - which makes me barf a little22:59
melwittyeah. that would suck to call it active if RPC message lost22:59
*** acormier has joined #openstack-nova22:59
*** tylerder_ has joined #openstack-nova22:59
melwittlemme see22:59
mriedemmelwitt: btw, let me hand you a power pack of good will for sticking with this, because this makes my brain bleed23:00
melwittokay yeah there's no in between vm_state of RESTORING. that was a task_state23:00
*** awaugama has quit IRC23:00
*** annegentle has quit IRC23:01
melwittmriedem: heh ... thanks. I felt like a burned out husk for quite a time. props to you and those who have braved reviewing it. I kind of cringed every time I asked someone to review it23:02
*** tylerderosagrund has quit IRC23:02
*** jmlowe has joined #openstack-nova23:02
*** acormier has quit IRC23:03
*** crushil has quit IRC23:04
melwittmriedem: these state changes in the API change deleted_at to None. and I guess I had been thinking to just punt this and assume parallel restores are not really a thing in real life23:04
melwittmaybe I could add a filter to count RESTORING as usage too and do handle the recheck that way. I dunno23:06
*** itlinux has quit IRC23:06
*** tylerderosagrund has joined #openstack-nova23:07
mriedemha23:07
mriedeminstance.deleted_at = None23:07
mriedemright in front of my gd eyes23:07
melwittmine too. and I still didn't see it until like the fourth time I looked at it23:08
*** tylerder_ has quit IRC23:08
*** jmlowe has quit IRC23:09
*** tylerder_ has joined #openstack-nova23:12
*** tylerderosagrund has quit IRC23:14
*** tylerder_ has quit IRC23:14
*** tylerderosagrund has joined #openstack-nova23:14
*** tylerder_ has joined #openstack-nova23:15
*** tylerderosagrund has quit IRC23:15
mriedemmelwitt: so,23:15
mriedemre the hw video ram thing,23:15
mriedemwe might want to just burn that turd out of the api before your series23:15
dansmith+10,000,00023:16
melwittlol23:16
*** baoli_ has joined #openstack-nova23:16
mriedemit was added back in icehouse https://review.openstack.org/#/q/topic:bp/libvirt-video-driver-selection23:16
dansmithgoing over that just now, that's nutsypants mcgee23:16
mriedemand is the only special virt-driver specific extra spec accounted against quota in the api23:16
melwittwell, that makes me feel bad for the person who has been trying to fix those video ram quota bugs23:16
melwittbut yeah, I was thinking to at least start off with no video ram23:16
mriedemwell, they fixed something that was screwing up their quota23:16
melwittyes23:17
mriedemif we don't count it against usage in the first place, that's not a problem23:17
*** tylerder_ has quit IRC23:17
*** dtp has quit IRC23:17
melwittyeah, that's what I was thinking23:17
melwittif anything they'll get more out of their quota if we don't count it :P23:17
mriedemwell, it's not like there aren't other things we don't account for in quota, which are calculated as overheads in the virt driver23:18
mriedemthis could potentially move there for the RT claim stuff23:18
mriedemor wherever23:18
melwittyeah23:18
*** baoli has quit IRC23:18
melwittfine by me. makes my life easier too23:18
mriedemi'll probably poke the ops list first to be safe23:19
mriedemdansmith: agree i assume? ^23:19
dansmithmriedem: I'd rather you not but whatever23:19
mriedemso sneaky remove?23:19
dansmithbugfix, imho23:20
melwittyeah. the bug where video ram usage wasn't ever decremented, that was a bug from whenever the feature was added to ... like last month23:20
melwittso I'm guessing around 5 years at least23:20
dansmithdat bug yo23:20
melwittI assume people can't have cared about it too much23:21
dansmithonly people with video ram called out too23:21
mriedemhttps://review.openstack.org/#/c/456050/23:22
mriedemwas the recent fix23:23
*** melwitt has left #openstack-nova23:23
*** melwitt has joined #openstack-nova23:23
dansmithrage quit!23:23
*** slaweq_ has joined #openstack-nova23:23
melwittlol23:24
mriedemok so maybe i just remove it and mark it related to that bug, and bug 1704012 and the counting quotas series23:24
openstackbug 1704012 in OpenStack Compute (nova) "hw_video:ram_max_mb is tracked for quota but not compute node resource provider allocations" [Undecided,New] https://launchpad.net/bugs/170401223:24
melwittI do that so often. open a link from the chat window and try to close the browser tab and close the channel tab23:24
melwitt(with ctrl w)23:24
dansmithmelwitt: it wasn't even mentioned in the commit message, nor was the quota bit tested specifically, AFAICT23:25
dansmither, mriedem ^23:25
*** baoli_ has quit IRC23:26
mriedemso,23:27
mriedemcan my commit message to remove it just say, "hey, what's that behind you!"23:27
melwittlol23:27
mriedemmelwitt: you want me to do it and you rebase on top or you just handle it?23:28
melwittmriedem: yeah, you can do it23:28
*** slaweq_ has quit IRC23:28
mriedemok23:29
dansmithmriedem: yes, yes it can23:29
melwittyou can have that glory23:29
melwittI'm still trying to think about multi create23:30
*** penick has joined #openstack-nova23:33
*** tbachman has quit IRC23:33
*** tbachman has joined #openstack-nova23:36
melwittmriedem: yeah, I think you're right about the multi create thing. in compute/api, we're in the place where we haven't yet returned anything to the user. so if we bail with OverQuota, there should be no instances left that were created as a result of the request23:39
melwittit should be as if it never happend23:39
mriedemslide of hand23:39
mriedem*slight23:39
melwittlooks like would have to clean up request specs and build requests too.23:40
mriedemyeah23:40
mriedemsimilar issue in conductor23:40
*** hongbin has quit IRC23:40
melwittyeah23:40
*** chyka_ has quit IRC23:40
*** chyka has joined #openstack-nova23:41
*** marst has quit IRC23:41
*** chyka has quit IRC23:45
*** Swami has quit IRC23:54
openstackgerritMatt Riedemann proposed openstack/nova master: Stop counting hw_video:ram_max_mb against quota  https://review.openstack.org/48313323:56
mriedemdansmith: melwitt: ^23:56
*** moshele has joined #openstack-nova23:57
melwittmriedem: cool23:59
melwittalso, I think I finished replying to your other comments23:59

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