Monday, 2017-12-11

*** yamahata has joined #openstack-nova00:01
*** fragatina has joined #openstack-nova00:06
*** masber has joined #openstack-nova00:07
*** gouthamr has quit IRC00:09
*** fragatina has quit IRC00:10
*** fragatina has joined #openstack-nova00:11
*** fragatina has quit IRC00:12
*** yangyapeng has joined #openstack-nova00:19
*** yangyapeng has quit IRC00:24
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete  https://review.openstack.org/52523100:29
*** brault has joined #openstack-nova00:31
openstackgerritTetsuro Nakamura proposed openstack/nova master: disable numa feature when virt_type is not kvm  https://review.openstack.org/46516000:32
*** yamamoto has joined #openstack-nova00:32
*** takashin has joined #openstack-nova00:33
*** brault has quit IRC00:36
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (2)  https://review.openstack.org/52626300:36
*** yamamoto has quit IRC00:40
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (3)  https://review.openstack.org/52655700:40
*** sbezverk has quit IRC00:40
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (3)  https://review.openstack.org/52655700:41
*** salv-orlando has joined #openstack-nova00:41
openstackgerritTetsuro Nakamura proposed openstack/nova master: disable numa feature when virt_type is not kvm  https://review.openstack.org/46516000:44
*** sean-k-m1 has quit IRC00:45
*** tetsuro_ has joined #openstack-nova00:45
*** Dinesh_Bhor has joined #openstack-nova00:46
*** salv-orlando has quit IRC00:46
*** sean-k-m1 has joined #openstack-nova00:49
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in post  https://review.openstack.org/52682300:50
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in post  https://review.openstack.org/52682300:50
tetsuro_efried, jaypipes: Because of "Reformat _get_all_with_shared" https://review.openstack.org/#/c/512799/ brought MergeConflict in top of the 2 patches in refactor-alloc-candidates branch https://review.openstack.org/#/c/498737/6, I have rebased it.00:50
*** psachin has joined #openstack-nova00:51
*** Dinesh_Bhor has quit IRC00:52
*** Dinesh_Bhor has joined #openstack-nova00:52
*** hoangcx has joined #openstack-nova00:54
*** Apoorva has quit IRC00:58
*** linkmark has quit IRC00:59
*** salv-orlando has joined #openstack-nova01:00
*** tetsuro_ has quit IRC01:00
*** liusheng has quit IRC01:02
*** salv-orlando has quit IRC01:05
*** yangyapeng has joined #openstack-nova01:07
*** gouthamr has joined #openstack-nova01:08
*** phuongnh has joined #openstack-nova01:08
*** huanxie has quit IRC01:10
openstackgerritzhangyanxian proposed openstack/nova-specs master: Sriov-bond  https://review.openstack.org/46352601:11
*** r-daneel has joined #openstack-nova01:21
*** liuzz has joined #openstack-nova01:35
*** huanxie has joined #openstack-nova01:37
*** hui has joined #openstack-nova01:40
*** adreznec has joined #openstack-nova01:43
*** asettle_ is now known as asettle01:45
*** tetsuro has joined #openstack-nova01:46
*** sapd_ has joined #openstack-nova01:50
*** sapd__ has quit IRC01:50
*** Apoorva has joined #openstack-nova01:59
*** vladikr has joined #openstack-nova02:10
*** markvoelker has quit IRC02:13
*** dave-mccowan has joined #openstack-nova02:13
*** annp has joined #openstack-nova02:15
*** dave-mcc_ has joined #openstack-nova02:15
*** dave-mccowan has quit IRC02:19
*** oanson has quit IRC02:19
*** oanson has joined #openstack-nova02:21
*** felipemonteiro_ has quit IRC02:22
*** Tom-Tom has joined #openstack-nova02:25
*** dave-mccowan has joined #openstack-nova02:33
*** dave-mcc_ has quit IRC02:35
*** hieulq has quit IRC02:40
*** hieulq has joined #openstack-nova02:43
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Fix API reference for microversion 1.14  https://review.openstack.org/52697302:43
*** Tom-Tom has quit IRC02:45
*** psachin has quit IRC02:49
*** Tom-Tom has joined #openstack-nova02:50
*** zhurong has joined #openstack-nova02:53
*** vladikr has quit IRC02:54
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Microversion 2.57 - List/Show all server migration types  https://review.openstack.org/43083902:57
*** namnh has joined #openstack-nova03:02
*** psachin has joined #openstack-nova03:03
*** liverpooler has joined #openstack-nova03:03
*** Tom-Tom has quit IRC03:09
*** Tom-Tom has joined #openstack-nova03:10
*** Tom-Tom has quit IRC03:10
*** Tom-Tom has joined #openstack-nova03:11
*** bkopilov has quit IRC03:14
*** yamamoto has joined #openstack-nova03:14
*** Tom-Tom has quit IRC03:16
*** Apoorva has quit IRC03:18
*** takashin has quit IRC03:23
*** Tom-Tom has joined #openstack-nova03:25
*** jichen has joined #openstack-nova03:31
*** brault has joined #openstack-nova03:31
*** baoli has quit IRC03:34
*** brault has quit IRC03:36
*** vivsoni has quit IRC03:36
*** david-lyle has joined #openstack-nova03:38
*** dave-mccowan has quit IRC03:41
*** liverpooler has quit IRC03:41
*** abhishekk has joined #openstack-nova03:42
*** gouthamr has quit IRC03:45
*** david-lyle has quit IRC03:45
*** vivsoni has joined #openstack-nova03:52
*** sean-k-m1 has quit IRC03:52
*** Tom-Tom has quit IRC03:54
*** Tom-Tom has joined #openstack-nova03:55
*** Tom-Tom has quit IRC03:59
*** sree has joined #openstack-nova04:01
*** rmcall has quit IRC04:06
*** liverpooler has joined #openstack-nova04:06
*** Tom-Tom has joined #openstack-nova04:07
*** links has joined #openstack-nova04:07
*** links has quit IRC04:07
*** sean-k-m1 has joined #openstack-nova04:11
*** rmcall has joined #openstack-nova04:13
*** markvoelker has joined #openstack-nova04:13
openstackgerritChen Hanxiao proposed openstack/nova master: libvirt: guest: introduce blockStats instead of domain.blockStats  https://review.openstack.org/52683304:19
*** zhurong has quit IRC04:26
*** udesale has joined #openstack-nova04:27
*** masuberu has joined #openstack-nova04:30
*** masber has quit IRC04:33
*** jmlowe has quit IRC04:33
*** masuberu has quit IRC04:36
*** armax has joined #openstack-nova04:36
*** armax has quit IRC04:41
*** markvoelker has quit IRC04:48
*** bkopilov has joined #openstack-nova04:58
*** trungnv has joined #openstack-nova05:01
*** Dinesh_Bhor has quit IRC05:01
*** Alex_Staf has joined #openstack-nova05:04
*** fragatina has joined #openstack-nova05:13
*** gbarros has quit IRC05:14
*** takashin has joined #openstack-nova05:20
*** liverpooler has quit IRC05:24
*** liverpooler has joined #openstack-nova05:25
*** zhurong has joined #openstack-nova05:32
*** udesale__ has joined #openstack-nova05:34
openstackgerritChen Hanxiao proposed openstack/nova master: libvirt: throw NotImplementedError if qga is not responsive when setting password  https://review.openstack.org/52382805:34
*** ratailor has joined #openstack-nova05:35
*** udesale has quit IRC05:37
*** sridharg has joined #openstack-nova05:38
*** masber has joined #openstack-nova05:42
*** markvoelker has joined #openstack-nova05:45
*** Dinesh_Bhor has joined #openstack-nova05:48
*** fragatin_ has joined #openstack-nova05:49
*** fragatina has quit IRC05:50
*** diga has joined #openstack-nova05:50
*** yamamoto has quit IRC06:08
*** janki has joined #openstack-nova06:12
*** karthiks has joined #openstack-nova06:15
*** threestrands has quit IRC06:17
*** bhagyashri_s is now known as bhagyashris06:17
*** markvoelker has quit IRC06:19
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/52479506:22
*** yamamoto has joined #openstack-nova06:22
*** sapcc-bot3 has joined #openstack-nova06:22
*** sapcc-bot has quit IRC06:22
*** liusheng has joined #openstack-nova06:28
*** diga has quit IRC06:32
*** yamamoto has quit IRC06:32
*** toabctl has joined #openstack-nova06:41
*** ircuser-1 has joined #openstack-nova06:42
*** pooja_jadhav has joined #openstack-nova06:43
*** pooja-jadhav has joined #openstack-nova06:43
*** pooja-jadhav has quit IRC06:43
*** hshiina has joined #openstack-nova06:44
*** bkopilov has quit IRC06:45
*** udesale has joined #openstack-nova06:46
*** udesale__ has quit IRC06:48
*** zhurong has quit IRC06:50
*** moshele has joined #openstack-nova06:52
*** bkopilov has joined #openstack-nova07:04
*** salv-orlando has joined #openstack-nova07:08
*** josecastroleon has joined #openstack-nova07:10
*** vivsoni has quit IRC07:10
*** Tom-Tom has quit IRC07:12
*** Tom-Tom has joined #openstack-nova07:13
*** edand has joined #openstack-nova07:15
*** Alex_Staf has quit IRC07:15
*** markvoelker has joined #openstack-nova07:16
*** hoonetorg has quit IRC07:18
*** Tom-Tom_ has joined #openstack-nova07:20
*** adisky_ has joined #openstack-nova07:20
*** Tom-Tom has quit IRC07:22
*** Tom-Tom__ has joined #openstack-nova07:22
*** Tom-Tom_ has quit IRC07:25
*** namnh has quit IRC07:29
*** namnh_ has joined #openstack-nova07:29
*** hoonetorg has joined #openstack-nova07:31
*** yamamoto has joined #openstack-nova07:33
*** diga has joined #openstack-nova07:36
*** yamamoto has quit IRC07:38
*** yamamoto has joined #openstack-nova07:38
*** lajoskatona has joined #openstack-nova07:39
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for snapshot instances  https://review.openstack.org/52592107:41
*** sshwarts has joined #openstack-nova07:42
*** rcernin has quit IRC07:43
*** salv-orlando has quit IRC07:44
*** salv-orlando has joined #openstack-nova07:44
*** AlexeyAbashkin has joined #openstack-nova07:46
*** Rambo has joined #openstack-nova07:48
*** salv-orlando has quit IRC07:48
*** markvoelker has quit IRC07:49
*** moshele has quit IRC07:50
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for snapshot instances  https://review.openstack.org/52592107:50
*** yamamoto has quit IRC07:52
*** rmcall has quit IRC07:54
*** salv-orlando has joined #openstack-nova07:56
*** sahid has joined #openstack-nova07:57
*** ralonsoh has joined #openstack-nova07:59
*** takashin has left #openstack-nova08:02
openstackgerritSilvan Kaiser proposed openstack/nova master: Added mount fstype based validation of Quobyte mounts  https://review.openstack.org/52224508:05
*** yamamoto has joined #openstack-nova08:06
*** andreas_s has joined #openstack-nova08:06
*** vivsoni has joined #openstack-nova08:12
*** ttsiouts has quit IRC08:12
*** andreas_s has quit IRC08:13
*** andreas_s has joined #openstack-nova08:13
*** phuongnh has quit IRC08:13
*** andreas_s has quit IRC08:13
*** andreas_s has joined #openstack-nova08:14
*** damien_r has joined #openstack-nova08:14
*** andreas__ has joined #openstack-nova08:15
*** giblet is now known as gibi08:17
gibigood morning nova08:17
*** andreas_s has quit IRC08:18
*** alexchadin has joined #openstack-nova08:19
*** andreas__ has quit IRC08:19
*** damien_r has left #openstack-nova08:19
*** andreas_s has joined #openstack-nova08:26
openstackgerritjichenjc proposed openstack/nova master: Add quiesce and unquiesce in support matrix  https://review.openstack.org/48239008:27
*** damien_r has joined #openstack-nova08:28
openstackgerritjichenjc proposed openstack/nova master: deprecate fping_path config option  https://review.openstack.org/52660208:29
*** andreas_s has quit IRC08:30
*** andreas_s has joined #openstack-nova08:30
gmannalex_xu: ping08:30
gmanngibi: good morning08:31
gmannalex_xu: we have all_tenants policy things not done which we mentioned in this ocata spec - https://specs.openstack.org/openstack/nova-specs/specs/ocata/implemented/add-whitelist-for-server-list-filter-sort-parameters.html08:31
gibigmann: I assume you are already close to the end your Monday. How lucky you are. :)08:32
gmannalex_xu: you remember we tried to do that with microversion but did not happen - https://review.openstack.org/#/c/425533/3  https://review.openstack.org/#/c/426128/108:32
gmanngibi: yea, heh but unlucky too to get monday 1 day before you guys :)08:33
gmannalex_xu: it got triggered while reviewing this -https://review.openstack.org/#/c/526558/108:33
gmannalex_xu: now i am thinking to remove all_tenants policies things mentioned in ocata implemented spec which actually did not get implemented and start new policy rule things as fresh in Rocky08:34
gmannso that people does not get confused with new policy rules for all_tenants mentioned in spec08:35
*** ludovic has joined #openstack-nova08:38
*** salv-orlando has quit IRC08:39
ludovicHello all, maybe someone can help to determine what i have a lot of urllib3.connectionpool full in nova.conf ?08:41
*** hoangcx has quit IRC08:41
ludovicWhen i use nova host-evacuate , a lot of error VMs are in Error state with Gateway timeout Error08:42
ludovicSo i really need help08:42
*** vivsoni has quit IRC08:43
*** vivsoni has joined #openstack-nova08:43
*** andreas_s has quit IRC08:45
*** jpena|off is now known as jpena08:45
*** Alex_Staf has joined #openstack-nova08:45
*** andreas_s has joined #openstack-nova08:45
*** markvoelker has joined #openstack-nova08:46
*** hoangcx has joined #openstack-nova08:46
*** rcernin has joined #openstack-nova08:47
*** andreas_s has quit IRC08:50
*** vivsoni has quit IRC08:50
*** brault has joined #openstack-nova08:53
openstackgerritGhanshyam Mann proposed openstack/nova-specs master: Few correction in the server filter/sort spec  https://review.openstack.org/52701908:55
gmannalex_xu: ^^08:56
*** andreas_s has joined #openstack-nova08:56
*** hshiina has quit IRC08:57
*** brault has quit IRC08:57
*** andreas__ has joined #openstack-nova08:58
*** brault has joined #openstack-nova08:59
*** andreas_s has quit IRC09:01
*** brault has quit IRC09:01
*** salv-orlando has joined #openstack-nova09:01
*** brault has joined #openstack-nova09:01
*** andreas_s has joined #openstack-nova09:01
openstackgerritjichenjc proposed openstack/nova master: [Placement] Add create inventories doc for placement  https://review.openstack.org/51221509:02
*** andreas__ has quit IRC09:03
*** andreas_s has quit IRC09:04
*** andreas_s has joined #openstack-nova09:04
*** andreas_s has quit IRC09:04
*** Jack_Iv has joined #openstack-nova09:07
*** sean-k-m1 has quit IRC09:09
*** damien_r has quit IRC09:10
openstackgerritAlvaro Lopez Garcia proposed openstack/nova master: Ensure that periodic reclaim cleans DB deleted instances  https://review.openstack.org/32325009:11
*** damien_r has joined #openstack-nova09:11
*** vivsoni has joined #openstack-nova09:12
*** sean-k-m1 has joined #openstack-nova09:12
*** markvoelker has quit IRC09:20
openstackgerritMerged openstack/nova stable/pike: Fix binary name  https://review.openstack.org/52453109:20
*** brault has quit IRC09:26
*** andreas_s has joined #openstack-nova09:27
openstackgerritsahid proposed openstack/nova master: libvirt: disconnect volume from host during detach  https://review.openstack.org/51500809:27
alex_xugmann: thanks, I will check it09:29
alex_xugmann: btw, I'm on PTO, will back to work on Thursday09:30
openstackgerritjichenjc proposed openstack/nova master: deprecate fping_path config option  https://review.openstack.org/52660209:31
*** andreas_s has quit IRC09:32
*** yamamoto has quit IRC09:33
*** andreas_s has joined #openstack-nova09:33
*** lucas-afk is now known as lucasagomes09:33
*** andreas_s has quit IRC09:33
*** yamamoto has joined #openstack-nova09:34
*** andreas_s has joined #openstack-nova09:37
gmannalex_xu: oh, sure no hurry.  enjoy09:41
*** andreas_s has quit IRC09:42
ludovicsomeone is familiar with networking issue like urllib3 connection pool full in nova or cinder ?09:43
*** Dinesh_Bhor has quit IRC09:43
*** andreas_s has joined #openstack-nova09:45
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for snapshot instances  https://review.openstack.org/52592109:49
*** finucannot is now known as stephenfin09:52
*** sree_ has joined #openstack-nova09:53
*** sree_ is now known as Guest2801209:53
*** jichen has quit IRC09:54
*** derekh has joined #openstack-nova09:55
*** gszasz has joined #openstack-nova09:56
*** sree has quit IRC09:56
*** trungnv has quit IRC09:56
*** brault has joined #openstack-nova09:58
openstackgerritGhanshyam Mann proposed openstack/nova master: Add server filters whitelist in server api-ref  https://review.openstack.org/52704209:58
*** gmann is now known as gmann_afk09:58
*** Guest28012 has quit IRC09:59
*** andreas_s has quit IRC10:04
*** brault has quit IRC10:04
*** ttsiouts has joined #openstack-nova10:08
*** ludovic has quit IRC10:08
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: [DNM] test  https://review.openstack.org/52704510:10
*** tetsuro has quit IRC10:12
openstackgerritLee Yarwood proposed openstack/nova master: conf: Do not inherit image signature props with snapshots  https://review.openstack.org/52704610:12
*** andreas_s has joined #openstack-nova10:12
*** brault has joined #openstack-nova10:13
*** tetsuro has joined #openstack-nova10:14
*** andreas__ has joined #openstack-nova10:14
*** andreas_s has quit IRC10:17
*** brault has quit IRC10:18
*** andreas__ has quit IRC10:19
*** andreas_s has joined #openstack-nova10:23
*** andreas_s has quit IRC10:32
*** cdent has joined #openstack-nova10:35
*** annp has quit IRC10:35
*** sambetts_ is now known as sambetts10:36
*** jmlowe has joined #openstack-nova10:38
*** abhishekk has quit IRC10:39
*** adisky_ has quit IRC10:40
*** alexchadin has quit IRC10:42
*** andreas_s has joined #openstack-nova10:43
*** yamamoto has quit IRC10:47
*** yamamoto has joined #openstack-nova10:47
*** andreas_s has quit IRC10:47
*** erlon has joined #openstack-nova10:48
*** yamamoto has quit IRC10:52
*** alexchadin has joined #openstack-nova10:52
*** sdague has joined #openstack-nova10:53
*** alexchadin has quit IRC10:56
*** alexchadin has joined #openstack-nova10:57
*** mdnadeem has joined #openstack-nova10:57
*** abhishekk has joined #openstack-nova10:59
*** abhishekk has quit IRC10:59
*** namnh_ has quit IRC11:03
*** moshele has joined #openstack-nova11:04
*** coreywright has quit IRC11:06
*** andreas_s has joined #openstack-nova11:08
*** Jack_Iv has quit IRC11:08
*** andreas_s has quit IRC11:12
*** edand has quit IRC11:16
*** markvoelker has joined #openstack-nova11:17
*** Jack_Iv has joined #openstack-nova11:18
*** edand has joined #openstack-nova11:18
*** Guest56845 has quit IRC11:19
*** udesale has quit IRC11:19
*** andreas_s has joined #openstack-nova11:19
*** sree has joined #openstack-nova11:22
*** Jack_Iv has quit IRC11:23
*** andreas_s has quit IRC11:23
*** AlexeyAbashkin has quit IRC11:25
*** sree has quit IRC11:26
*** salv-orlando has quit IRC11:26
*** andreas_s has joined #openstack-nova11:27
*** sree has joined #openstack-nova11:27
*** salv-orlando has joined #openstack-nova11:27
*** sree has quit IRC11:32
*** andreas_s has quit IRC11:35
*** ratailor has quit IRC11:41
*** dtantsur|afk is now known as dtantsur11:43
*** andreas_s has joined #openstack-nova11:45
*** diga has quit IRC11:45
*** AlexeyAbashkin has joined #openstack-nova11:45
*** sree has joined #openstack-nova11:47
*** liverpooler has quit IRC11:48
*** linkmark has joined #openstack-nova11:49
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: [DNM] test  https://review.openstack.org/52704511:50
*** markvoelker has quit IRC11:50
dtantsurmorning, compute overlords and overladies11:51
*** sree has quit IRC11:51
dtantsurwe're experiencing a new nova-related failure in the ironic multinode job: https://bugs.launchpad.net/ironic/+bug/173739511:51
openstackLaunchpad bug 1737395 in Ironic "n-cpu fails to start in the multinode job: Conflicting resource provider name: <uuid> already exists" [High,Confirmed]11:51
dtantsurdo you think any recent merge could cause it?11:51
*** rodolof has joined #openstack-nova11:54
*** janki has quit IRC11:56
*** lpetrut has joined #openstack-nova11:57
*** andreas_s has quit IRC11:58
*** andreas_s has joined #openstack-nova11:59
*** gszasz has quit IRC12:00
*** gszasz has joined #openstack-nova12:00
cdentdtantsur: might be https://review.openstack.org/#/c/524263/ ?12:00
cdentwhen did it start?12:00
dtantsurcdent: over the weekend, maybe Friday evening12:00
cdenthmmm, that’s a bit older than that12:01
*** Tom-Tom__ has quit IRC12:01
*** Tom-Tom has joined #openstack-nova12:02
*** yamamoto has joined #openstack-nova12:02
cdentdtantsur: is this the thing that https://review.openstack.org/#/c/508555/ is supposed to fix?12:02
*** Rambo has quit IRC12:03
*** Tom-Tom_ has joined #openstack-nova12:03
*** andreas_s has quit IRC12:04
dtantsurcdent: it looks so, but it's the first time I see this patch12:04
dtantsurjohnthetubaguy: around? re ^^^12:04
*** sahid has quit IRC12:04
cdentIt might be that the first patch I linked makes the problem addressed by the second more noticeable. Because in the past a 409 would return a None, masking that anthing went wrong.12:05
*** nicolasbock has joined #openstack-nova12:05
*** Tom-Tom has quit IRC12:06
*** dave-mccowan has joined #openstack-nova12:06
*** mdnadeem has quit IRC12:06
dtantsur(moving conversation from #openstack-ironic)12:07
dtantsursambetts: so yeah, seems like we need to rebase and merge this..12:07
dtantsurcdent: do you know if johnthetubaguy is around nowadays?12:07
*** sree has joined #openstack-nova12:07
*** Tom-Tom_ has quit IRC12:08
*** Jack_Iv has joined #openstack-nova12:08
dtantsurone of us can rebase the patch, but I don't want to step on his shoes12:08
cdentI haven’t seen him much lately, so I would think it would be okay to adopt12:08
dtantsursambetts or vdrok, wanna do ^^^ or should I?12:09
*** andreas_s has joined #openstack-nova12:10
*** sree has quit IRC12:11
* sambetts is currently fighting third party CI fires so don't have the time right now :( 12:12
*** sahid has joined #openstack-nova12:12
*** ijw has joined #openstack-nova12:12
*** yassine has joined #openstack-nova12:13
*** dave-mcc_ has joined #openstack-nova12:14
*** yassine is now known as Guest5260412:14
dtantsurokie12:14
*** ralonsoh has quit IRC12:15
*** dave-mccowan has quit IRC12:16
*** ralonsoh has joined #openstack-nova12:16
*** BryanS68 has joined #openstack-nova12:17
*** Tom-Tom has joined #openstack-nova12:18
*** andreas_s has quit IRC12:19
*** lucasagomes is now known as lucas-hungry12:19
*** Tom-Tom has quit IRC12:22
*** Jack_Iv_ has joined #openstack-nova12:23
*** jpena is now known as jpena|lunch12:25
*** andreas_s has joined #openstack-nova12:25
*** cdent has quit IRC12:27
*** alexchadin has quit IRC12:27
*** Jack_Iv has quit IRC12:27
openstackgerritDmitry Tantsur proposed openstack/nova master: Re-use existing ComputeNode on ironic rebalance  https://review.openstack.org/50855512:27
dtantsurI'm not sure I fully understand what is going on in the functional tests, but here is my attempt ^^^12:27
*** alexchadin has joined #openstack-nova12:27
*** ralonsoh has quit IRC12:28
*** ralonsoh has joined #openstack-nova12:28
*** jaypipes has joined #openstack-nova12:31
*** andreas_s has quit IRC12:34
*** gszasz has quit IRC12:35
*** andreas_s has joined #openstack-nova12:35
*** Roamer` has joined #openstack-nova12:35
*** andreas_s has quit IRC12:37
*** salv-orlando has quit IRC12:38
*** andreas_s has joined #openstack-nova12:38
*** salv-orlando has joined #openstack-nova12:40
*** andreas_s has quit IRC12:42
openstackgerritMerged openstack/nova stable/pike: Document the real behavior of notify_on_state_change  https://review.openstack.org/52187012:42
*** tetsuro has quit IRC12:43
*** smatzek has joined #openstack-nova12:45
*** markvoelker has joined #openstack-nova12:47
*** sahid has quit IRC12:48
*** dtantsur is now known as dtantsur|brb12:50
*** psachin has quit IRC12:53
*** andreas_s has joined #openstack-nova12:57
*** BryanS68 has quit IRC12:58
*** gszasz has joined #openstack-nova12:59
*** lennyb has quit IRC13:00
*** andreas_s has quit IRC13:01
*** rodolof has quit IRC13:02
*** rodolof has joined #openstack-nova13:02
*** lennyb has joined #openstack-nova13:02
*** udesale has joined #openstack-nova13:03
*** jangutter has joined #openstack-nova13:06
*** sree has joined #openstack-nova13:07
*** ChanServ has quit IRC13:08
*** vladikr has joined #openstack-nova13:09
*** baoli has joined #openstack-nova13:10
*** tssurya has joined #openstack-nova13:10
*** andreas_s has joined #openstack-nova13:11
*** sree has quit IRC13:11
*** andreas_s has quit IRC13:15
openstackgerritBalazs Gibizer proposed openstack/nova master: Transform instance.exists notification  https://review.openstack.org/40366013:17
*** ChanServ has joined #openstack-nova13:18
*** barjavel.freenode.net sets mode: +o ChanServ13:18
*** yangyapeng has quit IRC13:19
*** yangyapeng has joined #openstack-nova13:20
*** janki has joined #openstack-nova13:20
*** janki has quit IRC13:20
*** janki has joined #openstack-nova13:20
*** markvoelker has quit IRC13:21
*** andreas_s has joined #openstack-nova13:22
*** rcernin has quit IRC13:23
*** sahid has joined #openstack-nova13:24
*** lucas-hungry is now known as lucasagomes13:25
*** yangyapeng has quit IRC13:25
*** andreas_s has quit IRC13:26
*** jpena|lunch is now known as jpena13:27
*** moshele has quit IRC13:28
*** moshele has joined #openstack-nova13:29
*** cdent has joined #openstack-nova13:31
*** markvoelker has joined #openstack-nova13:38
*** sahid_ has joined #openstack-nova13:38
*** pchavva has joined #openstack-nova13:39
*** andreas_s has joined #openstack-nova13:39
*** andreas_s has quit IRC13:40
*** andreas_s has joined #openstack-nova13:40
*** sahid has quit IRC13:41
openstackgerritGhanshyam Mann proposed openstack/nova master: Add server filters whitelist in server api-ref  https://review.openstack.org/52704213:42
jaypipeshappy Monday morning, supernovas! Who's still working this week?13:42
gibijaypipes: good morning! I'm still here until 20th of Dec13:43
*** baoli has quit IRC13:43
jaypipesgibi: cool :)13:43
openstackgerritMatthew Booth proposed openstack/nova master: Add uuid column to BlockDeviceMapping  https://review.openstack.org/24260213:43
edleafejaypipes: unfortunately, me13:43
jaypipesglad to hear it!13:43
jaypipesedleafe: :)13:43
*** Tom-Tom has joined #openstack-nova13:46
openstackgerritMatthew Booth proposed openstack/nova master: Make BlockDeviceMapping object support uuid  https://review.openstack.org/24260313:46
openstackgerritMatthew Booth proposed openstack/nova master: Add an online migration for BDM.uuid  https://review.openstack.org/52559913:46
cdentjaypipes: I’ll be around until… uh 22nd I guess13:46
*** andreas_s has quit IRC13:47
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: [DNM] test  https://review.openstack.org/52704513:47
jaypipescdent: cool beans.13:48
*** dtantsur|brb is now known as dtantsur13:49
edleafeFor those around, nova scheduler subteam meeting in 10 minutes in #openstack-meeting-alt13:50
*** sree has joined #openstack-nova13:50
*** liverpooler has joined #openstack-nova13:51
* mdbooth is around for the long haul13:51
*** sree_ has joined #openstack-nova13:51
*** andreas_s has joined #openstack-nova13:51
*** sree_ is now known as Guest5288013:51
mdboothjaypipes: Speaking of which I just respun https://review.openstack.org/#/c/242602/25 following comments from mriedem (is he around this week?). I just added comments, but I fixed your nits from the previous patch as well as rebasing for a merge conflict later in the series.13:53
mdboothWould also appreciate a second opinion on the subject of unique constraints on nullable columns, but I did research that one carefully.13:54
mdboothIn short, I'm very much in favour of them.13:54
*** sree has quit IRC13:54
*** mriedem has joined #openstack-nova13:55
*** yangyapeng has joined #openstack-nova13:57
*** andreas_s has quit IRC13:57
jaypipesmdbooth: will re-review shortly.13:57
mdboothjaypipes: Thanks13:57
*** sbezverk has joined #openstack-nova13:59
*** gszasz has quit IRC14:03
*** andreas__ has joined #openstack-nova14:05
*** udesale has quit IRC14:05
*** gszasz has joined #openstack-nova14:06
mriedemjaypipes: is it correct to say that nothing done in a db api method is committed until we start to exit a pick_context_manager_writer context?14:10
*** andreas__ has quit IRC14:10
mriedemlike if we're updating multiple tables in a single method14:10
*** Tom-Tom has quit IRC14:11
*** purplerbot has joined #openstack-nova14:12
*** esberglu has joined #openstack-nova14:12
*** baoli has joined #openstack-nova14:12
cdentmriedem: or flush is callewd14:13
*** weshay|MOD is now known as weshay|sickday14:13
cdent(which sometimes happens to find out an id()14:13
cdentkeyboards in the cold suck, especially when you already suck at typing14:14
mriedemtrying to figure out if there is a better way to fix the deadlock we're seeing in https://review.openstack.org/#/c/507473/15/nova/db/sqlalchemy/api.py w/o using the retry_on_deadlock decorator14:14
mriedemcdent: radiant space heater for xmas14:14
*** cdent has quit IRC14:15
*** cdent has joined #openstack-nova14:15
*** Tom-Tom has joined #openstack-nova14:15
*** huanxie has quit IRC14:16
*** alexchadin has quit IRC14:16
*** andreas_s has joined #openstack-nova14:16
*** alexchadin has joined #openstack-nova14:16
*** alexchadin has quit IRC14:17
*** alexchadin has joined #openstack-nova14:17
*** alexchadin has quit IRC14:17
jaypipesmriedem: that is correct.14:18
*** huanxie has joined #openstack-nova14:19
mriedemjaypipes: ok - so i guess we just have ot use the deadlock decorator in ^ then14:20
mriedemunless anyone else has bright ideas14:20
jaypipesmriedem: looking at it now.14:20
*** andreas_s has quit IRC14:20
*** takashin has joined #openstack-nova14:21
*** diga has joined #openstack-nova14:21
*** tssurya has quit IRC14:23
*** yangyapeng has quit IRC14:24
*** gcb has joined #openstack-nova14:24
*** yangyapeng has joined #openstack-nova14:25
*** cleong has joined #openstack-nova14:25
dtantsurfolks, can someone please take a look at https://review.openstack.org/#/c/508555/ ? it fixes our CI apparently14:26
dtantsurmriedem, jaypipes ^^^14:26
*** gouthamr has joined #openstack-nova14:26
jaypipesdtantsur: yes, will do.14:26
cdentdtantsur: so that was it? I’m surprised it didn’t show up a bit sooner.14:27
dtantsurcdent: all I can say that a patch with Depends-On it has passed the CI :)14:27
cdentwell it presumably needs to happen anyway, so motion in the right direction14:28
*** lajoskatona has quit IRC14:28
dtantsuryep14:28
openstackgerritMerged openstack/python-novaclient master: Remove deprecated command in nova.rst  https://review.openstack.org/52656914:28
jaypipesmriedem: yeah, that's sort of an odd place for the deadlock to occur. that basically means there are two greenthreads that are attempting to simultaneously (or near-simultaneously) calling action_event_finish() on an action with the *same* request ID.14:29
jaypipesmriedem: which doesn't seem like it should happen, no?14:29
*** josecastroleon1 has joined #openstack-nova14:29
*** josecastroleon has quit IRC14:29
jaypipesmriedem: i.e. there should not be >1 greenthread attempting to "finish" a particular task/event at the same time.14:29
*** takashin has left #openstack-nova14:29
*** smatzek has quit IRC14:30
jaypipesmriedem: unless request ID is not a UUID?14:30
jaypipesmriedem: if request ID is an auto-inc, it could happen.14:30
*** smatzek has joined #openstack-nova14:30
*** smatzek has quit IRC14:30
mriedemrequest ID should be unique per user request14:30
*** edand has quit IRC14:31
mriedemgenerated by some keystone or oslo middleware i tihnk14:31
jaypipesmriedem: hmm, not necessarily...14:31
mriedemhttps://github.com/openstack/oslo.middleware/blob/master/oslo_middleware/request_id.py14:31
mriedemthe failures all seem to happen on a resize test14:31
jaypipesmriedem: see the comment and code on line 6302-631014:32
digajaypipes: Hi14:32
jaypipesmriedem: seems that on a service restart, the action is retrieved by getting the "last created by instance UUID"14:32
jaypipesmriedem: I'm wondering if that is returning an action ID that another greenthread might be operating on?14:32
mriedemjaypipes: maybe,14:33
jaypipesdiga: hey. just saw your email. will email you an answer, ok?14:33
mriedemwe wouldn't have a compute restart in the middle of this tempest run14:33
digajaypipes: I have my setup ready but how to reproduce the issue is the big changel I am facing14:33
digajaypipes: sure14:33
mriedemjaypipes: but we probably shouldn't update those kinds of actions anyway14:33
mriedemso good spot14:33
mriedemjaypipes: this is failing resize testing where we're rpc'ing back and forth with the same user context,14:34
mriedemto decorated methods like prep_resize, resize_instance, finish_resize, etc14:34
mriedemso it's possible we are hitting this quickly14:34
mriedemjaypipes: yeah i think that's probably what's happening here14:35
mriedemprep_resize updates it on the dest, rpc casts to resize_instance on the source which updates it again, which rpc casts back to the dest to finish_resize which updates the action again14:36
mriedem3 updates within a short window14:36
*** sahid_ has quit IRC14:37
*** erlon has quit IRC14:38
mriedemjaypipes: left comments, thanks for the help14:39
openstackgerritEd Leafe proposed openstack/nova master: Change RPC for select_destinations()  https://review.openstack.org/51670714:40
openstackgerritEd Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils  https://review.openstack.org/51135714:40
openstackgerritEd Leafe proposed openstack/nova master: Make conductor pass and use host_lists  https://review.openstack.org/51135814:40
openstackgerritEd Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize  https://review.openstack.org/52643614:40
edleafemriedem: ^^ rebased14:40
mriedemyar14:40
* edleafe goes to make more coffee14:40
jaypipesmriedem: not sure I really helped, but ok :)14:40
mriedemyou greased my brain gears14:41
openstackgerritDan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation  https://review.openstack.org/50763814:42
superdanmriedem: that xen thing merged and this is fixed up ^14:42
superdanwhoa14:42
*** superdan is now known as dansmith14:42
*** brault has joined #openstack-nova14:42
*** peter-hamilton has joined #openstack-nova14:45
*** alexchadin has joined #openstack-nova14:45
*** brault has quit IRC14:47
*** tssurya has joined #openstack-nova14:47
*** alexchadin has quit IRC14:48
*** burt has joined #openstack-nova14:49
*** huanxie has quit IRC14:50
*** sahid has joined #openstack-nova14:50
*** huanxie has joined #openstack-nova14:50
*** sahid has quit IRC14:51
*** david-lyle has joined #openstack-nova14:51
mriedemdansmith: service version needs to be fixed14:51
mriedemotherwise lgtm14:51
*** sahid has joined #openstack-nova14:51
dansmitheh I bumped it?14:51
mriedemrebase fail14:52
dansmithoh doh14:52
openstackgerritDan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation  https://review.openstack.org/50763814:52
dansmithdammit, hang on14:52
dansmithsheesh14:53
openstackgerritDan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation  https://review.openstack.org/50763814:53
dansmithdid half of that on friday and then completed before max caffeine absorption = fail14:53
*** yamamoto has quit IRC14:53
*** felipemonteiro_ has joined #openstack-nova14:55
*** felipemonteiro__ has joined #openstack-nova14:56
*** andreas_s has joined #openstack-nova14:56
mriedem+214:57
*** baoli has quit IRC14:57
mriedemgibi: you've been on this series before https://review.openstack.org/#/c/507638/14:58
mriedem^ is the last patch14:58
*** shaner has quit IRC14:59
*** baoli has joined #openstack-nova15:00
*** baoli has quit IRC15:00
*** felipemonteiro_ has quit IRC15:00
jaypipesmriedem: easy one. https://review.openstack.org/#/c/526973/15:00
*** andreas_s has quit IRC15:01
*** artom has joined #openstack-nova15:01
*** lyan has joined #openstack-nova15:02
*** ralonsoh has quit IRC15:02
*** baoli has joined #openstack-nova15:04
mriedemjaypipes: this must be a response sample? https://review.openstack.org/#/c/526973/1/placement-api-ref/source/update-resource_provider.json15:05
*** cdent has quit IRC15:05
*** shaner has joined #openstack-nova15:05
mriedemyeah i guess http://logs.openstack.org/73/526973/1/check/build-placement-api-ref/cbc7192/html/#id1115:05
*** andreas_s has joined #openstack-nova15:06
jaypipesyessir15:06
*** smatzek has joined #openstack-nova15:06
mriedemit's weird that we have 2 source files for resource providers for api-ref15:06
mriedemonce has create/list, the other has show/update15:06
mriedem*and delete15:06
*** yamamoto has joined #openstack-nova15:06
*** peter-hamilton has quit IRC15:08
mriedemanyway something takashin could fix up later,15:08
*** pchavva has quit IRC15:08
mriedem+W15:08
*** stelucz has quit IRC15:08
mriedemlyarwood: did you say you're back now?15:09
*** sean-k-m1 has quit IRC15:09
lyarwoodmriedem: I am indeed15:09
mriedemlyarwood: ok, we have a bunch of pike backports which need a final look https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/pike15:10
mriedemif you would15:10
mriedemi need to do a release by end of day15:10
lyarwoodmriedem: ack, did a few at the end of the list earlier, I'll start on these now15:10
mriedemthanks15:10
lyarwoodmriedem: should I take anything that you have pushed as already having a +2 btw? Noticed you did that with something of mine the other day..15:11
lyarwoodmriedem: I'd rather not tbh, just wondering if that's now policy on stable or something15:11
*** yamamoto has quit IRC15:12
mriedemlyarwood: no you don't have to do that unless it's trivial - i was just referring to the other +2'ed patches in pike right now15:12
mriedemdansmith did a bunch of those +2s already15:12
lyarwoodmriedem: kk cool15:13
mriedemandreas_s: zkvm ci failed in 15 minutes on a patch on master http://sng01.objectstorage.softlayer.net/v1/AUTH_1940ea10-6e82-4501-b2f9-eb236510e575/ibmzkvmci/production/507638/29/check-tempest-dsvm-neutron-full-ubuntu-xenial-s390x/4e9ba43/logs/devstacklog.txt.gz15:14
mriedem2017-12-11 15:06:27.360 | cp: cannot stat '/opt/stack/new/devstack/files/etcd-v3.1.7-linux-s390x/etcdctl': No such file or directory15:14
andreas_smriedem: hi, saw it earlier the day - a new nodepool image with the updated binary has been build 20 minutes ago, hopefully new test runs will suceed...15:15
mriedemok15:15
*** brault has joined #openstack-nova15:15
*** alexchadin has joined #openstack-nova15:17
*** brault has quit IRC15:17
*** brault has joined #openstack-nova15:18
*** r-daneel has quit IRC15:18
*** Alex_Staf has quit IRC15:19
*** moshele has quit IRC15:21
*** alexchadin has quit IRC15:21
*** sahid has quit IRC15:22
*** brault has quit IRC15:22
*** andreas_s has quit IRC15:23
*** andreas_s has joined #openstack-nova15:23
*** mlavalle has joined #openstack-nova15:23
*** Guest52880 has quit IRC15:24
*** pchavva has joined #openstack-nova15:24
*** janki has quit IRC15:24
*** sree has joined #openstack-nova15:24
*** vivsoni has quit IRC15:25
*** sean-k-m1 has joined #openstack-nova15:26
*** sean-k-m1 has quit IRC15:26
*** rodolof has quit IRC15:26
*** yamahata has quit IRC15:26
*** rodolof has joined #openstack-nova15:27
*** andreas_s has quit IRC15:27
*** sree has quit IRC15:28
*** moshele has joined #openstack-nova15:28
*** sree has joined #openstack-nova15:28
*** rodolof has quit IRC15:29
*** rodolof has joined #openstack-nova15:30
*** tidwellr has joined #openstack-nova15:32
*** moshele has quit IRC15:32
*** andreas_s has joined #openstack-nova15:33
openstackgerritStephen Finucane proposed openstack/nova master: console: introduce framework for RFB authentication  https://review.openstack.org/34539715:34
openstackgerritStephen Finucane proposed openstack/nova master: console: introduce the VeNCrypt RFB authentication scheme  https://review.openstack.org/34539815:34
openstackgerritStephen Finucane proposed openstack/nova master: console: Provide an RFB security proxy implementation  https://review.openstack.org/34539915:34
openstackgerritStephen Finucane proposed openstack/nova master: doc: Document TLS security setup for noVNC proxy  https://review.openstack.org/50054415:35
openstackgerritJay Pipes proposed openstack/nova master: handle traits with sharing providers  https://review.openstack.org/51711915:35
openstackgerritJay Pipes proposed openstack/nova master: Test alloc_cands with non overlapping sharing RPs  https://review.openstack.org/51938015:35
openstackgerritJay Pipes proposed openstack/nova master: Test alloc_cands with one RP shared between two RPs  https://review.openstack.org/51961715:35
openstackgerritJay Pipes proposed openstack/nova master: Test allocation candidates: multiple aggregates  https://review.openstack.org/51863315:35
openstackgerritJay Pipes proposed openstack/nova master: Test helper: validate provider summaries  https://review.openstack.org/51898215:35
openstackgerritJay Pipes proposed openstack/nova master: Fix accumulated nits in refactor series  https://review.openstack.org/52118915:35
openstackgerritJay Pipes proposed openstack/nova master: Add aggregates check in allocation candidates  https://review.openstack.org/52240715:35
openstackgerritJay Pipes proposed openstack/nova master: Fix missing rps in allocation candidates  https://review.openstack.org/52240915:35
openstackgerritJay Pipes proposed openstack/nova master: placement: func tests for multiple shared RPs  https://review.openstack.org/49873715:35
*** sree has quit IRC15:37
*** bnemec has joined #openstack-nova15:40
*** alexchadin has joined #openstack-nova15:40
*** cdent has joined #openstack-nova15:41
*** alexchadin has quit IRC15:44
*** gcb has quit IRC15:44
*** baoli has quit IRC15:45
*** baoli has joined #openstack-nova15:45
*** josecastroleon1 has quit IRC15:45
*** mvk has quit IRC15:46
*** yangyapeng has quit IRC15:49
*** baoli has quit IRC15:50
*** josecastroleon has joined #openstack-nova15:51
*** andreas_s has quit IRC15:51
*** ralonsoh has joined #openstack-nova15:52
*** yamamoto has joined #openstack-nova15:52
gibimriedem: went through https://review.openstack.org/#/c/507638/ and +W-d it15:52
mriedemgibi: thanks15:52
*** huanxie has quit IRC15:53
*** yamamoto has quit IRC15:54
*** ralonsoh has quit IRC15:54
*** ralonsoh_ has joined #openstack-nova15:54
*** yamamoto has joined #openstack-nova15:54
*** yamamoto has quit IRC15:54
*** huanxie has joined #openstack-nova15:54
*** alexchadin has joined #openstack-nova15:56
*** armax has joined #openstack-nova15:56
*** r-daneel has joined #openstack-nova15:56
*** sridharg has quit IRC15:57
*** baoli has joined #openstack-nova15:57
*** _ix has joined #openstack-nova15:59
*** alexchadin has quit IRC15:59
openstackgerritStephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project  https://review.openstack.org/51430916:01
openstackgerritStephen Finucane proposed openstack/nova master: zuul: Resolve review comments from I41b03a347  https://review.openstack.org/52525516:01
openstackgerritStephen Finucane proposed openstack/nova master: zuul: Remove WSGI functional tests  https://review.openstack.org/52714016:01
*** gouthamr has quit IRC16:01
openstackgerritMatt Riedemann proposed openstack/nova master: Update Instance action's updated_at when action event updated.  https://review.openstack.org/50747316:02
openstackgerritMatt Riedemann proposed openstack/nova master: Add instance action db and obj pagination support.  https://review.openstack.org/52642216:02
openstackgerritMatt Riedemann proposed openstack/nova master: Add pagination and changes-since for instance-actions  https://review.openstack.org/32632616:02
openstackgerritMerged openstack/python-novaclient stable/pike: Move zuulv3 jobs to project repo  https://review.openstack.org/52220716:02
openstackgerritMerged openstack/python-novaclient stable/ocata: Move zuulv3 jobs to project repo  https://review.openstack.org/52220816:02
openstackgerritStephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project  https://review.openstack.org/51430916:03
openstackgerritStephen Finucane proposed openstack/nova master: zuul: Remove WSGI functional tests  https://review.openstack.org/52714016:03
*** sileht has left #openstack-nova16:05
*** andreas_s has joined #openstack-nova16:06
mriedemfirst 3 patches in this series are ready to go i think https://review.openstack.org/#/c/524047/16:07
*** diga has quit IRC16:07
*** josecastroleon has quit IRC16:09
*** vipul has quit IRC16:10
*** vipul has joined #openstack-nova16:10
*** dave-mcc_ is now known as dave-mccowan16:11
*** andreas_s has quit IRC16:12
*** Apoorva has joined #openstack-nova16:12
*** andreas_s has joined #openstack-nova16:13
*** andreas__ has joined #openstack-nova16:14
*** baoli has quit IRC16:15
*** andrea___ has joined #openstack-nova16:15
*** bkopilov has quit IRC16:17
*** andreas_s has quit IRC16:18
*** chyka has joined #openstack-nova16:18
*** andreas__ has quit IRC16:19
*** andrea___ has quit IRC16:20
openstackgerritEd Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize  https://review.openstack.org/52643616:20
*** andreas_s has joined #openstack-nova16:20
*** chyka has quit IRC16:22
*** hongbin has joined #openstack-nova16:22
*** chyka has joined #openstack-nova16:22
mriedemedleafe: i think my only hangup on https://review.openstack.org/#/c/516707/22 is the limits object<>dict conversion16:25
mriedemi think we can hide all of that in the compute rpcapi client code16:25
*** andreas_s has quit IRC16:25
mriedemfor build_and_run_instance, prep_resize, unshelve, etc16:25
mriedemthen conductor and compute managers don't need to check that16:25
*** huanxie has quit IRC16:26
mriedemif/when we want to send the SchedulerLimits object to compute, we can bump the compute rpc api version and send the object down to compute and the compute will expect the object rather than the dict16:26
mriedembut that's not necessary for this series16:26
mriedemdansmith: ^ agree? or am i missing something?16:27
*** baoli has joined #openstack-nova16:27
*** ralonsoh_ is now known as ralonsoh16:27
*** sree has joined #openstack-nova16:27
openstackgerritMatt Riedemann proposed openstack/nova master: Deprecate file injection  https://review.openstack.org/52202716:27
*** huanxie has joined #openstack-nova16:28
*** baoli has quit IRC16:29
*** sshwarts has quit IRC16:29
mriedemsdague: you want to take a pass through https://review.openstack.org/#/c/522027/ ?16:29
*** baoli has joined #openstack-nova16:29
*** hemna_ has joined #openstack-nova16:31
*** alexchadin has joined #openstack-nova16:32
*** tidwellr has quit IRC16:32
openstackgerritStephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project  https://review.openstack.org/51430916:32
openstackgerritStephen Finucane proposed openstack/nova master: zuul: Remove WSGI functional tests  https://review.openstack.org/52714016:32
dansmithmriedem: you're proposing making the conductor and scheduler not check service version and do the dict conversion, but instead doing it in compute rpc based on rpc version?16:32
*** tidwellr has joined #openstack-nova16:32
*** sree has quit IRC16:32
dansmithmriedem: that is fine as long as there isn16:32
*** tssurya has quit IRC16:32
dansmithisn't some other semantic detail that needs to happen.. like conductor needing to know if compute is new or old so it can do something different with some other data16:32
mriedemdansmith: the latter yes, like he was doing in an older patch set https://review.openstack.org/#/c/516707/17/nova/compute/rpcapi.py16:33
mriedemat this point i don't think we need to check the compute rpc version to convert to a primitive as we know no computes at this point can handle getting an object16:34
dansmithyou mean we always do the dict conversion at the moment, until we have a compute rpc bump to handle it, yes?16:35
mriedemyes16:35
*** alexchadin has quit IRC16:36
dansmithyep makes sense to me16:37
edleafemriedem: ok, so just convert in the rpcapi layer, and assume dicts throughout compute16:38
mriedemyup16:38
edleafeok, gimme a few...16:38
mriedemedleafe: i.e. we shouldn't have any changes to compute in this patch16:38
mriedemnova/compute/manager.py i mean16:38
edleafemriedem: gotcha. Most of those changes were defensive anyway. You know, "just in case"16:39
mriedemsure, it's just that kind of stuff gets lost in the sands of time and a year from now i'll be looking at the compute code saying, "why is this here? it can never happen..."16:40
*** Guest52604 has quit IRC16:40
edleafemriedem: it probably was happening in the unit tests16:40
*** jpena is now known as jpena|brb16:41
edleafeI probably created SchedulerLimits objects everywhere. I'll make sure those are cleaned up, too16:41
*** Apoorva has quit IRC16:42
*** AlexeyAbashkin has quit IRC16:43
*** sree has joined #openstack-nova16:43
*** gbarros has joined #openstack-nova16:44
*** burt has quit IRC16:45
cdentedleafe: you happy to abandon this for later recovery (to get it off the radar), given neither dtantsur nor mriedem seem to want it now? https://review.openstack.org/#/c/494206/16:47
mriedemi think we need a nova-status check first at least16:49
mriedemand probably backport that nova-status check to pike16:49
*** damien_r has quit IRC16:50
cdentgibi: you still thinking about https://review.openstack.org/#/c/501252/16:50
*** gyee has joined #openstack-nova16:51
*** lpetrut has quit IRC16:51
dtantsurmriedem, dansmith, sorry, could you please check https://review.openstack.org/#/c/508555/ ? we're waiting for it to unblock our CI16:51
dtantsurothers too ^^'16:51
edleafecdent: I'm still not clear why it should stay16:52
*** sahid has joined #openstack-nova16:52
edleafecdent: By Queens, the flavors must all be migrated16:52
*** moshele has joined #openstack-nova16:52
*** sree has quit IRC16:52
cdentedleafe: I have no skin, I was fine with it moving, so it is mriedem and dtantsur you could be talking to16:52
*** penick has joined #openstack-nova16:53
edleafeok16:53
mriedemi'm personally a bit uncomfortable at the moment with operators adoption / understanding of the new thing yet16:53
mriedemespecially if we don't have traits via flavors yet16:53
mriedemto replace the capabilities filter16:53
gibicdent: soon we will have n-rp that also makes it possible to use CUSTOM resource so I think I will abandon those patches soon16:54
*** Jack_Iv_ has quit IRC16:54
*** yamamoto has joined #openstack-nova16:54
mriedemi'm thinking of https://specs.openstack.org/openstack/nova-specs/specs/queens/approved/ironic-driver-traits.html16:54
cdentgibi: cool, just checking, i’m passing through the list I created on friday16:55
*** Jack_Iv has joined #openstack-nova16:55
gibicdent: thanks for reminding me about this patch :)16:55
edleafemriedem: I get that that spec isn't completed. I'm just not clear how that impacts resources in extra_specs16:56
*** gbarros has quit IRC16:57
jaypipesdtantsur: done16:58
jrolledleafe: seems like the suggestion is to give operators more time to move to resource classes16:58
mriedemholy crap16:59
jrollI'm not sure the traits thing is relevant as people can still run the compute capabilities filter16:59
* jroll hides16:59
jroll:P16:59
*** yangyapeng has joined #openstack-nova16:59
mriedemmaybe traits isn't relevant, i just don't have a good feeling about how much operators understand what's being asked of them to upgrade16:59
*** moshele has quit IRC17:00
mriedemand like i said, we should at least have a nova-status check for this to say, 'yup, you're good to upgrade'17:00
*** gszasz has quit IRC17:00
edleafejroll: hey stranger!17:00
jrollhi :)17:00
* cdent waves at jroll 17:01
edleafeok, if the idea is to not make resource classes required in queens, that's fine17:01
jrollmriedem: I don't have enough recent context to agree, but totally see where you and dmitry are coming from17:01
edleafeI hadn't heard about that change17:01
mriedemedleafe: that's not what i'm saying17:01
mriedembut before we make something required, i think we need a bit of work to help people know when their migration is done17:02
mriedemlike we do, for example, with schema blocker migrations at times17:02
edleafemriedem: that's fair. IMO, we should add that, but keeping the pike migration code around doesn't feel like it would help that17:03
mriedempersonally my rear end gets sore hearing about how hard upgrades are and how complicated nova makes everything17:03
*** yamamoto has quit IRC17:03
mriedemedleafe: keeping it around helps motivate writing the nova-status change :)17:03
*** yangyapeng has quit IRC17:03
mriedemremoving code is the carrot17:04
*** salv-orlando has quit IRC17:04
mriedemdtantsur: jaypipes: if https://review.openstack.org/#/c/508555/ regresses something, does it only impact the ironic driver?17:05
mriedemdoesn't look like it https://review.openstack.org/#/c/508555/6/nova/compute/resource_tracker.py17:05
* dtantsur is in a meeting, sorry17:05
*** jmlowe has quit IRC17:05
*** sahid has quit IRC17:05
dtantsurit's hard to tell for me, unfortunately17:05
dansmithmriedem: I'm about to comment on the error message in that way17:05
*** edand has joined #openstack-nova17:06
*** Jack_Iv_ has joined #openstack-nova17:06
jrollseems like it's unlikely to affect non-ironic, but it could, in the drivers where it makes sense for host != nodename17:06
jrollthat's normal for vmware or something right?17:07
*** alexchadin has joined #openstack-nova17:07
jaypipesjroll: I don't think so any more...17:07
mriedemdansmith: yeah just did17:07
mriedemabout the rebalance thing?17:07
dansmithmriedem: yeah17:07
*** josecastroleon has joined #openstack-nova17:07
mriedembecause people eff up their compute nodes tables and service host records i'm sure17:08
jrolljaypipes: ah, cool17:08
*** penick has quit IRC17:08
*** jmlowe has joined #openstack-nova17:08
* dtantsur will fix later on17:08
dansmithor they change a hypervisor's name,17:09
dansmithor have an accidental name overlap17:09
*** Jack_Iv has quit IRC17:09
dansmithor they point the compute service at the wrong hypervisor17:09
mriedemor it's tuesday?17:09
*** alexchadin has quit IRC17:09
dtantsurwut? :D17:09
*** fragatin_ has quit IRC17:10
dansmithsince our UC is over (host,hypervisor_hostname) each hypervisor_hostname could be the same today technically17:10
mriedemdtantsur: tuesday is random break your resource tracker day17:10
dtantsurany suggestions for a flag name? allow_nodes_migration (hmm, "migrate" may be confusing)17:10
dtantsurmriedem: good to know :)17:10
*** fragatina has joined #openstack-nova17:10
*** fragatina has quit IRC17:10
sean-k-mooneydansmith:  the UC is over (host,hypervisor_hostname) today for ironic correct?17:11
*** fragatina has joined #openstack-nova17:11
*** gbarros has joined #openstack-nova17:12
*** lucasagomes is now known as lucas-afk17:12
*** penick has joined #openstack-nova17:12
dansmithsean-k-mooney: for everything17:13
sean-k-mooneydansmith: well what i mean is the resone the unique constratint is (host,hypervisor_hostname) instaed of just (host) or (hypervisor_hostname) is because of how we track ironic nodes in the nova db17:14
dansmithbut we track nodes for every virt driver as well17:14
dansmiththey're just all 1:117:14
dtantsurso, any ideas better than Driver.allow_nodes_migration?17:16
jrolldtantsur: what about just if_driver_is_ironic() ?17:17
jrollor however you might actually write that17:17
dtantsurif driver.has_bears() :)17:17
dansmithare you talking about checking a flag on the driver in RT?17:17
dtantsuryep17:17
dansmithI don't love that, but I'd vote for the generic-er flag name, with a comment above of "ironic strikes again" or something17:18
* dansmith jumps on a call17:18
dtantsurI was thinking about ^^^ as well17:19
*** edand has quit IRC17:20
mriedemdriver.rebalances_nodes?17:20
mriedemthat's specific to the problem here17:21
dtantsursounds good17:21
mriedemdoesn't talk about migration17:21
mriedemand is obviously false for everything else17:21
dansmithlet's hope no other drivers try to use this to implement HA17:21
dtantsurpeople never take a random feature and try to misuse it, right?17:22
dansmithnever17:22
*** alexchadin has joined #openstack-nova17:23
*** jpena|brb is now known as jpena17:24
*** alexchadin has quit IRC17:25
*** sree has joined #openstack-nova17:26
*** salv-orlando has joined #openstack-nova17:27
*** huanxie has quit IRC17:27
openstackgerritStephen Finucane proposed openstack/nova master: Handle images with no data  https://review.openstack.org/52632917:28
openstackgerritStephen Finucane proposed openstack/nova master: tests: Use correct response type in tests  https://review.openstack.org/52718217:28
sean-k-mooneydtantsur: i mean if i can observer a specific semantic behavior or side effect  of a pulic api/config option that is not documented anyware it was totally part of the public api contract that i can build my feature on correct?17:28
dtantsurgood plan, go for it17:29
openstackgerritChris Dent proposed openstack/nova master: placement: skip authentication on root URI  https://review.openstack.org/52200217:29
*** huanxie has joined #openstack-nova17:30
*** sree has quit IRC17:31
stephenfinmriedem: I'd to rework that patch. Turns out the glance API docs are wrong. Would appreciate a second look at it https://review.openstack.org/52632917:33
* stephenfin is going to fix the glanceclient API docs now17:33
mriedemcdent: https://review.openstack.org/#/c/521640/17:33
cdentack17:34
*** andreas_s has joined #openstack-nova17:36
sean-k-mooneystephenfin: you have a seperate bug open for the itorator to list conversion on line 179 of glance.py correct?17:38
*** alexchadin has joined #openstack-nova17:39
*** alexchadin has quit IRC17:42
*** xyang1 has joined #openstack-nova17:43
openstackgerritMerged openstack/nova stable/pike: placement: avoid returning duplicated alloc_reqs when no sharing rp  https://review.openstack.org/50888517:43
*** damien_r has joined #openstack-nova17:44
*** Apoorva has joined #openstack-nova17:45
*** Apoorva has quit IRC17:45
*** Apoorva has joined #openstack-nova17:46
*** sree has joined #openstack-nova17:47
*** yangyapeng has joined #openstack-nova17:49
*** derekh has quit IRC17:50
*** penick has quit IRC17:50
*** sree has quit IRC17:51
*** ijw has quit IRC17:51
*** yangyapeng has quit IRC17:53
cdentmriedem: is your opinion we should go ahead with the want_version thing inside the serialization loop for the last-modified stuff. I read your first comment as maybe we should consider, and then as you progressed is became more certain. So I’ll take it as “doooo iiiiit” unless you feel like saying something to the contrary17:55
*** brault has joined #openstack-nova17:59
*** andreas_s has quit IRC18:00
*** andreas_s has joined #openstack-nova18:01
*** huanxie has quit IRC18:01
*** baoli has quit IRC18:01
*** ralonsoh has quit IRC18:02
*** huanxie has joined #openstack-nova18:02
*** andreas_s has quit IRC18:03
*** andreas_s has joined #openstack-nova18:03
*** brault has quit IRC18:03
*** baoli_ has joined #openstack-nova18:04
*** bkopilov has joined #openstack-nova18:04
*** andreas_s has quit IRC18:05
*** andreas_s has joined #openstack-nova18:05
*** sree has joined #openstack-nova18:07
*** sambetts is now known as sambetts|afk18:08
*** andreas_s has quit IRC18:10
*** sree has quit IRC18:11
*** Alex_Staf has joined #openstack-nova18:12
*** andreas_s has joined #openstack-nova18:16
*** ijw has joined #openstack-nova18:17
*** jpena is now known as jpena|brb18:20
*** tbachman has joined #openstack-nova18:20
*** gbarros has quit IRC18:20
*** burt has joined #openstack-nova18:21
*** ijw has quit IRC18:21
*** andreas__ has joined #openstack-nova18:22
*** ijw has joined #openstack-nova18:22
*** gbarros has joined #openstack-nova18:24
*** andreas_s has quit IRC18:25
*** salv-orlando has quit IRC18:26
*** salv-orlando has joined #openstack-nova18:27
*** ijw has quit IRC18:27
*** sree has joined #openstack-nova18:27
*** AlexeyAbashkin has joined #openstack-nova18:28
mriedemcdent: i think we should probably do that for at least the requests that can generate a lot of results in a big cloud18:28
*** Jack_Iv_ has quit IRC18:28
mriedemso GET /traits and GET /resource_providers18:28
mriedemGET /traits will always be at least reasonable size because of the library18:29
cdentmriedem: i’ve done it for allocations, traits, resource classes, resource providers, but not inventories18:29
dtantsurmriedem: re that patch: should I keep the functional tests changes, given that the behavior is no longer the default?18:29
mriedemcdent: allocations are per provider or consumer right? so i didn't think that one was as big a deal18:30
mriedembut if you're going to establish a pattern...18:30
*** ijw has joined #openstack-nova18:30
mriedemi just tend to think of allocations like inventories - they are mapped to a single provider18:30
cdentmriedem: I was imagining something other than libvirt, where maybe the compute-mode is ginormous18:31
mriedemdtantsur: i probably wouldn't18:31
cdentor a shared disk18:31
dtantsurk18:31
mriedemcdent: or vmware :)18:31
*** salv-orlando has quit IRC18:31
mriedem*vcenter18:31
* cdent knows nothing of this vcenter people speak of18:31
cdenti was mostly thinking of my.petapbytediskfarm.rp-518:32
*** ijw has quit IRC18:32
*** sree has quit IRC18:32
mriedembut then the inventory is per resource class per provider isn't it?18:32
*** AlexeyAbashkin has quit IRC18:32
mriedemso you'd only have a lot of inventory entries for a provider if you have a lot of resource classes on that provider18:32
_ixHello friends. I'm having some trouble getting a compute node to show up in the hosts list.18:34
cdentmriedem: right, which isn’t predicted, so that one is left off18:34
cdentbut when it turns into a problem, is fixable18:35
mriedemstephenfin: question in https://review.openstack.org/#/c/526329/18:35
mriedem_ix: you have to run the discover_hosts CLI18:35
mriedemnova-manage cell_v2 discover_hosts --verbose18:35
_ixI have. It says there aren't any unmapped hosts.18:35
mriedemwhat does nova service-list show?18:36
mriedemdoes the entry show up there?18:36
_ixFor nova compute? Yes.18:36
_ixThis is just 1/5 of the hosts not showing up.18:37
_ixBut, I might be misleading a bit. I see there's an rpc problem on the host.18:37
mriedemwhat does nova hypervisor-list show?18:37
mriedemnova host-list is deprecated since pike btw18:37
smatzekThe Trove gates started failing on Friday or Saturday, right about the time the new attach flows landed.  In particular they are failing when resizing or rebuilding instances with attached volumes.  Does Nova have gate tests that do this?18:37
openstackgerritDmitry Tantsur proposed openstack/nova master: Re-use existing ComputeNode on ironic rebalance  https://review.openstack.org/50855518:38
dtantsurmriedem: I hope I did not mess up anything ^^^18:38
mriedemsmatzek: yes for resize with volume attached - pretty sure anyway18:38
mriedemsmatzek: not sure about rebuild with a volume attached (in tempest anyway)18:38
*** Jack_Iv has joined #openstack-nova18:38
_ixmriedem: I generally use the openstack client. Is `openstack host list` also deprecated?18:38
mriedemsmatzek: test_resize_volume_backed_server_confirm18:39
mriedemin tempest18:39
smatzekmriedem: resize is failing like this, which in looking at the code means that connection_info is missing the driver type, and in looking at the previous log entries I can see that connection_info is indeed missing it:  http://logs.openstack.org/39/524539/1/check/legacy-trove-functional-dsvm-mysql/3ef681d/logs/screen-n-cpu.txt.gz#_Dec_11_03_03_28_75500718:40
mriedem_ix: i deprecated nova host-list and didn't update openstack client so it's probably not deprecated18:40
*** jpena|brb is now known as jpena18:41
_ixCool. I see that the host is disabled in the `nova hypervisor-list` Thanks for the help. Odd.18:41
mriedem_ix: starting in pike,18:41
smatzekrebuild, which I haven't dug as deep into today, is failing a bit more nuanced way.  The mount point it is getting from either cinder or nova is giving /dev/na, as in "not applicable" which is a default for some fields in Cinder.18:41
mriedemnova-compute will auto-disable the host if it's had 10 failed build attempts18:41
mriedemin a row18:41
_ixOh, hey. That's a fine tip. Thanks again! I'm sure that's what's going on here.18:42
openstackgerritAndrey Volkov proposed openstack/osc-placement master: CLI for allocations  https://review.openstack.org/45753418:42
openstackgerritAndrey Volkov proposed openstack/osc-placement master: CLI for usages  https://review.openstack.org/45753518:42
openstackgerritAndrey Volkov proposed openstack/osc-placement master: CLI for aggregates (v1.1)  https://review.openstack.org/50564318:42
openstackgerritAndrey Volkov proposed openstack/osc-placement master: CLI for resource classes (v1.2)  https://review.openstack.org/51118218:42
openstackgerritAndrey Volkov proposed openstack/osc-placement master: RP list: member_of and resources parameters (v1.3, v1.4)  https://review.openstack.org/51118318:42
openstackgerritAndrey Volkov proposed openstack/osc-placement master: RP delete inventories (v1.5)  https://review.openstack.org/51464218:43
openstackgerritAndrey Volkov proposed openstack/osc-placement master: CLI for traits (v1.6)  https://review.openstack.org/51464318:43
openstackgerritAndrey Volkov proposed openstack/osc-placement master: Resource class set (v1.7)  https://review.openstack.org/51464418:43
openstackgerritAndrey Volkov proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9)  https://review.openstack.org/51464618:43
openstackgerritAndrey Volkov proposed openstack/osc-placement master: CLI allocation candidates (v1.10)  https://review.openstack.org/51464718:43
_ixAlso, a side note. That's a super handy feature from my previous deployments.18:43
mriedem_ix: there is a config option if you want to change that18:43
mriedembut yeah, ^ is why it's on by default18:43
_ixI don't. I think that's great.18:43
*** Jack_Iv has quit IRC18:43
mriedemdansmith: ^ high praise18:43
smatzekmriedem: the tempest test test_resize_volume_backed_server_confirm is doing resize with a volume backed / BFV instance.  The trove test is different, the instance has ephemeral disk and a volume attached.18:44
mriedemsmatzek: hmm http://logs.openstack.org/39/524539/1/check/legacy-trove-functional-dsvm-mysql/3ef681d/logs/screen-n-cpu.txt.gz#_Dec_11_03_03_28_49979518:44
mriedem{"attachment": {"status": "reserved", "detached_at": "", "connection_info": {}, "attached_at": "", "attach_mode": null, "instance": "85cdb482-63a5-487a-b103-95b9383ffcc7", "volume_id": "b3c82217-38f4-4c1f-b0f0-486a4fda6dda", "id": "848654a4-9647-46b8-9485-b3647fe87cfb"}}18:45
_ixmriedem: Can you offer any conjecture about kernel versions, etc? We're on Centos 7.4, and upgraded to kernel 4.14. It looks like the rpcbind.socket is failing us.18:45
mriedem_ix: nope18:45
mriedemsmatzek: that empty connection_info is presumably the problem18:45
mriedembecause nova pulls the driver_volume_type out of the connection_info on the attachment18:45
mriedemsmatzek: although,18:46
*** baoli has joined #openstack-nova18:46
mriedemi think that's normal for a 'reserved' attachment maybe? we'd have to ask jgriffith18:46
mriedemuntil nova updates the attachment with the host connector, the attachment doesn't have a connection_info18:46
*** sree has joined #openstack-nova18:46
_ixWill the openstack client one day be the only client?18:46
smatzekyep, I came to the same conclusion, that the empty connection_info coming back from Cinder is part of the issue, but was drawing a blank on if it was suppose to be empty there or not.18:46
ildikovmriedem: if it's only reserved connection_info is empty18:47
_ix(That is, the openstack cli client)18:47
mriedem_ix: at this rate i doubt it18:47
_ixThat's a sad story.18:47
mriedem_ix: been talking about that for years now18:47
smatzekTo my knowledge Trove hasn't had changes here and it just started about when the Nova code landed so I thought I'd check.  Also, it's not "mainline".  Regular attach works and only resize and rebuild are having issues afaik.18:47
*** baoli_ has quit IRC18:47
mriedem_ix: would need to get feature parity in osc18:48
mriedem_ix: and no one is working on that18:48
dansmithmriedem: nice18:48
_ixForgive me, what is osc?18:48
mriedem_ix: we could make that a priority, but that means de-prioritizing other work18:48
mriedemopenstack client18:48
mriedemOpenStackClient18:48
_ixOh, I see.18:48
*** yangyapeng has joined #openstack-nova18:49
ildikovsmatzek: I need to jump on a call, but will look into the logs once I got off the phone18:49
ildikovsmatzek: the new flow shouldn't blow those calls up, but tempest might lack some coverage, I don't think we did a thorough manual testing :(18:50
*** sree has quit IRC18:50
mriedemhmm18:51
mriedem'block_device_mapping': [{'guest_format': None, 'boot_index': -1, 'mount_device': u'/dev/vdb', 'connection_info': {'serial': u'b3c82217-38f4-4c1f-b0f0-486a4fda6dda'}, 'disk_bus': u'virtio', 'device_type': u'disk', 'attachment_id': '848654a4-9647-46b8-9485-b3647fe87cfb', 'delete_on_termination': True}]18:51
*** jmlowe has quit IRC18:51
*** damien_r has quit IRC18:53
mriedemsmatzek: can you open a bug, the logstash query does show this starting on the 9th18:53
mriedemhttp://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22VolumeDriverNotFound%3A%20Could%20not%20find%20a%20handler%20for%20None%20volume.%5C%22%20AND%20tags%3A%5C%22screen-n-cpu.txt%5C%22&from=7d18:53
mriedem*8th actually18:54
*** yangyapeng has quit IRC18:54
smatzekmriedem:  will do, and then I'm going to look into the Trove upgrade failure, which uses Nova rebuild a bit more. The stack/failure there is in Trove code but it has that bad /dev/na mountpoint which it fetched from an API (I don't recall which without re-digging)18:56
ildikovI don't recall seeing this issue earlier18:57
*** MasterOfBugs has joined #openstack-nova18:57
*** pramodrj07 has joined #openstack-nova18:57
*** gbarros has quit IRC18:58
ildikovwe should have connection_info by the time we get to the _connect_volume call, I need to look into what went wrong there18:58
*** Apoorva has quit IRC18:58
*** jmlowe has joined #openstack-nova18:58
*** moshele has joined #openstack-nova19:00
*** jpena is now known as jpena|away19:01
smatzekmriedem: https://bugs.launchpad.net/nova/+bug/173759919:02
openstackLaunchpad bug 1737599 in OpenStack Compute (nova) "Instance resize with attach volume fails" [Undecided,New]19:02
*** AlexeyAbashkin has joined #openstack-nova19:03
mriedemi can see what's different about this flow from the bfv one19:03
*** huanxie has quit IRC19:04
mriedemwe seem to lose the connection_info during some of the much complicated bdm object -> driver bdm -> dict block_device_info conversions...19:04
mriedemor, it was just never set in the first place19:04
mriedemmy guess is the latter19:04
mriedemi can definitely see where it blows up though yeah19:05
*** Jack_Iv has joined #openstack-nova19:06
*** andreas__ has quit IRC19:06
*** andreas_s has joined #openstack-nova19:06
*** huanxie has joined #openstack-nova19:06
*** AlexeyAbashkin has quit IRC19:07
*** MasterOfBugs has quit IRC19:07
*** pramodrj07 has quit IRC19:07
*** josecastroleon has quit IRC19:08
*** ijw has joined #openstack-nova19:10
openstackgerritEd Leafe proposed openstack/nova master: Change RPC for select_destinations()  https://review.openstack.org/51670719:10
openstackgerritEd Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils  https://review.openstack.org/51135719:10
openstackgerritEd Leafe proposed openstack/nova master: Make conductor pass and use host_lists  https://review.openstack.org/51135819:10
openstackgerritEd Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize  https://review.openstack.org/52643619:10
*** andreas_s has quit IRC19:11
edleafemriedem: cleaned up ^^19:11
*** penick has joined #openstack-nova19:12
*** ijw has quit IRC19:17
*** itlinux has joined #openstack-nova19:17
*** jroll has quit IRC19:18
openstackgerritChris Dent proposed openstack/nova master: [placement] Add cache headers to placement api requests  https://review.openstack.org/52164019:18
openstackgerritChris Dent proposed openstack/nova master: [placement] Add info about last-modified to contrib docs  https://review.openstack.org/52608419:18
*** jroll has joined #openstack-nova19:19
cdentmriedem, edleafe ^ that fixes the several suggestions matt had19:20
mriedemsmatzek: ildikov: i think i have the resize thing figured out19:20
*** itlinux has quit IRC19:22
*** chyka has quit IRC19:23
smatzekmriedem: great, thanks19:23
mriedemi'll post a patch and you can push a trove depends-on to see if it works19:24
*** dtantsur is now known as dtantsur|afk19:24
*** andreas_s has joined #openstack-nova19:26
*** sree has joined #openstack-nova19:26
*** moshele has quit IRC19:27
*** itlinux has joined #openstack-nova19:27
*** salv-orlando has joined #openstack-nova19:27
*** fragatina has quit IRC19:28
*** Alex_Staf has quit IRC19:28
smatzekwould the thing you found possibly cause the mountpoint to be lost in Cinder's attachments during rebuild?19:28
smatzekmriedem: ^19:29
*** itlinux has quit IRC19:29
ildikovmriedem: cool, that was quick!19:29
mriedemsmatzek: don't know, one problem at a time19:30
mriedemsmatzek: but maybe19:31
*** sree has quit IRC19:31
openstackgerritMurali Annamneni proposed openstack/nova master: Enables MySQL Cluster Support for Nova  https://review.openstack.org/44664319:31
mriedemsmatzek: there is a good chance it is though19:31
*** salv-orlando has quit IRC19:32
*** pramodrj07 has joined #openstack-nova19:35
*** MasterOfBugs has joined #openstack-nova19:35
*** huanxie has quit IRC19:37
*** huanxie has joined #openstack-nova19:39
*** Apoorva has joined #openstack-nova19:41
ildikovsmatzek: that log indicates that there might be a step missing from rebuild19:41
ildikovsmatzek: will check what mriedem has as a fix and then we can compare whether we covered everything or not19:42
ericyoungAny idea when https://review.openstack.org/#/c/407440/ will get some eyes on it?19:42
ericyoungIt is the patch for ScaleIO ephemeral/image volumes19:43
*** Apoorva has quit IRC19:45
*** MasterOfBugs has quit IRC19:46
*** pramodrj07 has quit IRC19:46
*** MasterOfBugs has joined #openstack-nova19:46
*** pramodrj07 has joined #openstack-nova19:46
*** sree has joined #openstack-nova19:47
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Save driver_volume_type when refreshing attachment connection_info  https://review.openstack.org/52722819:48
mriedemsmatzek: ildikov: ^ see if that fixes the resize issue in trove; needs unit tests19:48
*** andreas_s has quit IRC19:49
*** ijw has joined #openstack-nova19:50
*** andreas_s has joined #openstack-nova19:50
mriedemactually, i think that will fix the VolumeDriverNotFound but we'll still fail to connect the volume on the destination host19:51
*** andreas_s has quit IRC19:51
mriedembecause the new attachment connection_info doesn't have 'data' set19:51
*** sree has quit IRC19:51
*** andreas_s has joined #openstack-nova19:51
smatzekso should I hold off on pushing a depends-on test to Trove?19:52
mriedemjust start that depends-on to make sure it blows up where i think it is now19:52
mriedemand we'll need it anyway to verify the eventual fix19:52
smatzekmriedem:  https://review.openstack.org/#/c/527230. It looks like the depends-on linked correctly.  To be honest I've never done one cross-projects so hopefully this is right.19:54
smatzekhttps://review.openstack.org/#/c/52723019:54
*** Apoorva has joined #openstack-nova19:54
mriedemyes that should work19:54
*** ijw has quit IRC19:54
*** moshele has joined #openstack-nova19:54
*** Jack_Iv has quit IRC19:59
mriedemok this is a bit shittier than i thought, but i think i know of another relatively painless fix20:06
mriedemto avoid doing crazy crap in the virt driver(s) or nova/virt/block_device.py code20:06
ildikovmriedem: hmm, we are explicitly transforming 'driver_volume_type' so it is supposed to come from Cinder already20:07
mriedemildikov: it's pulled out of the connection_info from the attachment,20:07
mriedemwhich is empty for the new attachment which is not yet connected on the dest host20:07
mriedemi.e. we don't call attachment_update in the cold migrate flow20:07
mriedemfor cold migration, the drivers just do all the work on the hosts with connecting the volume20:08
mriedemand the old init/terminate_connection calls20:08
mriedemon the volume20:08
ildikovhmm, but shouldn't we call attachment_update with the new flow now?20:08
mriedemyes - but we aren't20:08
mriedemso that's what i think i need to add20:08
ildikovsorry, I just got confused with the changes in your patch in mind20:09
ildikovas the log looks like we have a missing attahcment_update call somewhere20:09
ildikovand then we need to check whether attachment_complete happens there or not20:09
mriedemyes my patch does'nt fix it20:10
mriedemi think i know what needs to happen20:10
mriedembut making coffee first20:10
ildikovfair enough20:13
*** moshele has quit IRC20:15
*** nsingh has joined #openstack-nova20:16
*** bhagyashri_s has joined #openstack-nova20:16
*** pooja-jadhav has joined #openstack-nova20:16
*** andreas_s has quit IRC20:18
*** pooja_jadhav has quit IRC20:18
*** andreas_s has joined #openstack-nova20:18
*** niraj_singh has quit IRC20:18
*** bhagyashris has quit IRC20:18
*** Swami has joined #openstack-nova20:19
*** fragatina has joined #openstack-nova20:22
*** fragatina has quit IRC20:23
*** andreas_s has quit IRC20:23
*** fragatina has joined #openstack-nova20:23
*** fragatina has quit IRC20:23
*** fragatina has joined #openstack-nova20:24
*** marst has joined #openstack-nova20:25
*** salv-orlando has joined #openstack-nova20:28
openstackgerritEd Leafe proposed openstack/nova master: Make conductor pass and use host_lists  https://review.openstack.org/51135820:31
openstackgerritEd Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize  https://review.openstack.org/52643620:31
*** salv-orlando has quit IRC20:33
*** penick has quit IRC20:35
*** shaner has quit IRC20:38
_ixMore high praise if you're interested. I think that newish Pike feature in disabling nodes saved me about half a day of troubleshooting.20:38
_ixThank you!20:39
*** shaner has joined #openstack-nova20:39
*** huanxie has quit IRC20:39
*** Guest52604 has joined #openstack-nova20:40
mriedemthanks for the feedback20:41
mriedemalways nice to know when people actually "like" something20:41
*** moshele has joined #openstack-nova20:41
*** liverpooler has quit IRC20:42
cdent👍20:42
*** huanxie has joined #openstack-nova20:42
mriedemdtantsur|afk: do you have a run on https://review.openstack.org/#/c/508555/ yet?20:43
mriedemTheJulia: do you know where ^ is being tested to see if it unblocks the ironic multinode job?20:44
mriedemdtantsur|afk: TheJulia: nevermind, found it https://review.openstack.org/#/c/527075/20:44
TheJulia:)20:45
mriedemis ironic-tempest-dsvm-ipa-wholedisk-agent_ipmitool-tinyipa-multinode the job to care about?20:45
*** tidwellr_ has joined #openstack-nova20:49
*** tidwellr has quit IRC20:49
* TheJulia double checks20:51
TheJuliamriedem: yes20:51
*** nsingh has quit IRC20:54
*** felipemonteiro_ has joined #openstack-nova20:54
*** moshele has quit IRC20:55
*** nsingh has joined #openstack-nova20:55
mriedemdansmith: comments in john's rebalance patch - https://review.openstack.org/#/c/508555/ - i think it looks better, but we could maybe drop the new object query method too - i think there is an existing one that could be re-used in there20:56
*** felipemonteiro__ has quit IRC20:57
*** tidwellr_ has quit IRC20:59
dansmithmriedem: yep, looks right21:00
*** chyka has joined #openstack-nova21:01
*** tidwellr has joined #openstack-nova21:03
*** cdent has quit IRC21:04
*** chyka has quit IRC21:05
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Update and complete volume attachments during resize  https://review.openstack.org/52722821:06
mriedemsmatzek: ildikov: ^ should do it for resize21:06
*** sree has joined #openstack-nova21:07
mriedemsmatzek: the rebuild fix will be different, so do you have a separate bug for that?21:08
mriedemwith links to the logs21:08
smatzekI don't, but I'll open one.21:08
mriedemdansmith: ok, ironic multinode job passed and given dmitry and john are gone, i could update quick and get another run, or just wait for john to do it - but i think he's probably on paternity21:09
mriedemmaybe it's fine to wait another day assuming this has been broken since pike21:09
dansmithyeah I think he is.. up to you, I'm around to +2 if you want, or I can do it21:10
mriedemTheJulia: is this a latent bug or regression in pike?21:10
dansmithif you update I think jaypipes and I can get it21:10
mriedemdansmith: ok i'll update quick21:10
*** Apoorva has quit IRC21:11
*** sree has quit IRC21:11
TheJuliamriedem: don't have enough context to say either way21:12
ildikovmriedem: so resize is using a different flow and not calling bdm.attach(), right?21:13
mriedemildikov: correct21:13
*** huanxie has quit IRC21:13
*** david-lyle has quit IRC21:13
mriedemdoing it the way i'm doing it is a lot easier than trying to re-plumb how things work for connecting the volume during a resize21:14
mriedemsince in the old flow, we never needed to call bdm.attach because the volume was already attached21:14
mriedemthe new flow is all about the attachment though, so we have to do things differently21:14
mriedemwe could call bdm.attach and pass do_driver_attach=False, but then we have to do that before calling driver.finish_migration and it seems wrong to 'complete' the attachment before we've actually connected it on the host21:15
*** huanxie has joined #openstack-nova21:15
smatzekmriedem: I can open a bug for the rebuild one if you want.  If you know there are changes needed there and want a bug to work them. However, I'm not convinced there is a bug in rebuild itself after digging through the code this afternoon.21:16
*** tidwellr has quit IRC21:19
smatzekTrove does this with cinderclient before calling nova rebuild, and at this point the device/mountpoint is /dev/na, which means it was lost previous to rebuild.  https://github.com/openstack/trove/blob/master/trove/taskmanager/models.py#L1369-L137121:20
smatzekthis probably needs more digging on my part to see what was done in the test suite to that instance before getting to this point.21:20
ildikovmriedem: ok, that makes sense, I don't have a Devstack running at the moment, will set up one and check these flows as I always have to re-understand the whole thing if I just try to follow this code by reading...21:21
mriedemsmatzek: ok i'll leave rebuild up to you21:22
smatzekmriedem: I need to take off to get the kids from school and run a few errands so I'll dig into it later.21:22
mriedemildikov: we definitely have a todo to add a test to tempest for resize with a volume attached (non-bfv)21:22
*** smatzek has quit IRC21:23
openstackgerritMatt Riedemann proposed openstack/nova master: Re-use existing ComputeNode on ironic rebalance  https://review.openstack.org/50855521:24
mriedemdansmith: ^ i'll recheck the ironic canary patch21:24
openstackgerritMerged openstack/nova master: Address nits from service create/destroy notification review  https://review.openstack.org/52316221:24
openstackgerritMerged openstack/nova master: [placement] Fix API reference for microversion 1.14  https://review.openstack.org/52697321:24
openstackgerritMerged openstack/nova stable/pike: Mention API behavior change when over quota limit  https://review.openstack.org/52551121:24
openstackgerritMerged openstack/nova stable/pike: Downgrade log for keystone verify client fail  https://review.openstack.org/52547521:24
mriedemall he wanted was a pepsi https://www.youtube.com/watch?v=aYItTxqTc3821:25
*** jmlowe has quit IRC21:27
ildikovjust another regular Monday :)21:28
_ixQuick question ... can I assume that that ports should be bound to the host that... hosts the instance?21:28
openstackgerritMerged openstack/nova stable/pike: Fix ValueError if invalid max_rows passed to db purge  https://review.openstack.org/52562821:28
mriedem_ix: unless it's migrating21:28
*** tidwellr has joined #openstack-nova21:28
*** jmlowe has joined #openstack-nova21:28
*** chyka has joined #openstack-nova21:29
_ixmriedem: Where should I be looking for problems if this isn't necessarily the case?21:29
*** salv-orlando has joined #openstack-nova21:29
_ixThat is, I just instantiated, and they seem to be bound incorrectly.21:29
openstackgerritEd Leafe proposed openstack/nova master: Make conductor pass and use host_lists  https://review.openstack.org/51135821:29
openstackgerritEd Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize  https://review.openstack.org/52643621:29
dansmithmriedem: cool21:30
mriedem_ix: nova-compute logs21:30
mriedem?21:30
mriedem_ix: maybe check to see if the instance was rescheduled during the create to another host and the ports weren't cleaned up properly21:30
_ixI've been seeing a bit of that, so that might make sense.21:30
mriedem_ix: maybe check https://review.openstack.org/#/c/520248/21:31
_ixNo ports appear to be working on this host that it's supposed to be living on, so, there are a litany of issues. Thanks.21:31
mriedem_ix: that likely means we hit that compute, created a port that couldn't be bound, and we rescheduled to another compute host21:31
_ix!21:32
mriedemif the instance built on the next host, it would be using the 2nd port but we orphaned the 1st port21:32
_ixI just saw that error today.21:32
_ixCool.21:32
*** burt has quit IRC21:32
*** salv-orlando has quit IRC21:34
mriedemi've removed zkvm from nova-ci for now21:39
*** threestrands has joined #openstack-nova21:41
*** r-daneel has quit IRC21:43
*** r-daneel has joined #openstack-nova21:43
*** pchavva has quit IRC21:46
*** huanxie has quit IRC21:46
*** pooja_jadhav has joined #openstack-nova21:46
*** bhagyashris_ has joined #openstack-nova21:47
*** salv-orlando has joined #openstack-nova21:47
*** sree has joined #openstack-nova21:47
*** bfife_ has joined #openstack-nova21:48
*** niraj_singh has joined #openstack-nova21:49
*** huanxie has joined #openstack-nova21:49
*** cfriesen__ has joined #openstack-nova21:50
*** tidwellr has quit IRC21:50
*** migi_ has joined #openstack-nova21:50
*** tonyb_ has joined #openstack-nova21:51
*** takashin has joined #openstack-nova21:52
*** sree has quit IRC21:53
*** gbarros has joined #openstack-nova21:54
*** andreykurilin_ has joined #openstack-nova21:55
*** markmc` has joined #openstack-nova21:55
*** rmk` has joined #openstack-nova21:55
*** stephenfin_ has joined #openstack-nova21:55
*** EmilienM_ has joined #openstack-nova21:55
*** aignatov2 has joined #openstack-nova21:55
*** DinaBelova2 has joined #openstack-nova21:56
*** nsingh has quit IRC21:56
*** pooja-jadhav has quit IRC21:56
*** bhagyashri_s has quit IRC21:56
*** lennyb has quit IRC21:56
*** ircuser-1 has quit IRC21:56
*** trozet has quit IRC21:56
*** cfriesen_ has quit IRC21:56
*** aloga has quit IRC21:56
*** EmilienM has quit IRC21:56
*** migi has quit IRC21:56
*** jdillaman has quit IRC21:56
*** tonyb has quit IRC21:56
*** rmk has quit IRC21:56
*** markmc has quit IRC21:56
*** bfife has quit IRC21:56
*** mdbooth has quit IRC21:56
*** ansiwen has quit IRC21:56
*** brad[] has quit IRC21:56
*** stephenfin has quit IRC21:56
*** rybridges has quit IRC21:56
*** andreykurilin has quit IRC21:56
*** DinaBelova has quit IRC21:56
*** aignatov has quit IRC21:56
*** rmk` is now known as rmk21:56
*** aignatov2 is now known as aignatov21:56
*** rmk has quit IRC21:56
*** rmk has joined #openstack-nova21:56
*** DinaBelova2 is now known as DinaBelova21:56
*** EmilienM_ is now known as EmilienM21:56
*** EmilienM has quit IRC21:56
*** EmilienM has joined #openstack-nova21:56
mriedemnever seen a live migration failure like this before http://logs.openstack.org/07/516707/23/check/legacy-tempest-dsvm-multinode-live-migration/d9089b2/logs/libvirt/qemu/instance-0000000a.txt.gz21:57
*** Apoorva has joined #openstack-nova21:57
*** tlbr has quit IRC21:57
*** cleong has quit IRC21:58
*** ircuser-1 has joined #openstack-nova21:58
*** mdbooth has joined #openstack-nova21:58
*** trozet has joined #openstack-nova21:59
*** aloga has joined #openstack-nova21:59
*** PramodJ has joined #openstack-nova22:00
*** lennyb has joined #openstack-nova22:00
*** rybridges has joined #openstack-nova22:00
*** ansiwen has joined #openstack-nova22:00
*** Pramod has joined #openstack-nova22:00
*** gouthamr has joined #openstack-nova22:00
*** jdillaman has joined #openstack-nova22:00
*** PramodJ has quit IRC22:01
*** Pramod has quit IRC22:01
*** pramodrj07 has quit IRC22:01
*** MasterOfBugs has quit IRC22:01
*** tlbr has joined #openstack-nova22:02
mriedemedleafe: so for prep_resize and unshelve_instance, https://review.openstack.org/#/c/516707/23/nova/compute/rpcapi.py - the filter_properties dict contains a limits key, is that getting converted to a dict via scheduler utils?22:03
openstackgerritMerged openstack/nova stable/pike: Vzstorage: synchronize volume connect  https://review.openstack.org/52468822:05
*** sree has joined #openstack-nova22:07
*** brad[] has joined #openstack-nova22:08
*** yangyapeng has joined #openstack-nova22:09
mriedemedleafe: ok so i think the only remaining thing is there are some unnecessary conversions for the limits object in conductor manager in https://review.openstack.org/#/c/516707/ still - do you want to remove those or should i just do that quick?22:09
*** mvk has joined #openstack-nova22:10
*** sree has quit IRC22:11
edleafemriedem: I can do it22:12
edleafemriedem: re: the filter properties dict key, I'm tracing the logic backwards22:12
*** yangyapeng has quit IRC22:13
edleafemriedem: ok, yeah, I don't see any paths that the filter_properties could contain a SchedulerLimits object22:15
mriedemyeah i think it's handled in the 4 cases that matter22:16
mriedemit's just different in two of them which is why i had to dig22:16
*** ijw has joined #openstack-nova22:17
*** kumarmn has joined #openstack-nova22:18
*** rcernin has joined #openstack-nova22:19
*** ijw has quit IRC22:21
*** penick has joined #openstack-nova22:22
*** huanxie has quit IRC22:23
*** huanxie has joined #openstack-nova22:24
*** baoli has quit IRC22:28
*** baoli has joined #openstack-nova22:29
*** lyan has quit IRC22:30
openstackgerritEd Leafe proposed openstack/nova master: Change RPC for select_destinations()  https://review.openstack.org/51670722:33
openstackgerritEd Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils  https://review.openstack.org/51135722:33
openstackgerritEd Leafe proposed openstack/nova master: Make conductor pass and use host_lists  https://review.openstack.org/51135822:33
openstackgerritEd Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize  https://review.openstack.org/52643622:33
edleafemriedem: addressed your comments ^^22:33
mriedemon it22:33
*** baoli has quit IRC22:33
*** xyang1 has quit IRC22:34
*** dave-mccowan has quit IRC22:34
*** gouthamr has quit IRC22:40
openstackgerritMerged openstack/nova stable/pike: Add regression test for rebuilding a volume-backed server  https://review.openstack.org/52563222:41
mriedemedleafe: +2 on that bottom patch22:42
mriedemthanks22:42
mriedemand the one after it22:44
*** hongbin has quit IRC22:44
mriedemjaypipes: dansmith: https://review.openstack.org/#/c/516707/ and the one after it have my +222:44
*** larainema has quit IRC22:45
edleafemriedem: kewl22:46
mriedem+2 on cdent's cache headers patch too https://review.openstack.org/#/c/521640/22:46
*** kumarmn has quit IRC22:47
*** jogo has quit IRC22:48
jaypipesmriedem: cool.22:48
*** rodolof has quit IRC22:53
ildikovmriedem: BTW as for resize, do we need to do the same magic with finish_revert_resize too?22:53
mriedemildikov: maybe22:55
mriedemlikely need a test for confirm and revert a resize with an instance that has a volume attached22:55
ildikovmriedem: as we're calling _terminate_volume_connections in revert_resize too which basically that same story just backwards22:56
ildikov*is basically the22:56
ildikovI mean backwards from one host to the other22:57
ildikovanyway, I think you got what I wanted to say :)22:57
openstackgerritSam Morrison proposed openstack/nova master: Allow ability for non admin users to use all filters on server list.  https://review.openstack.org/52655822:57
*** yangyapeng has joined #openstack-nova22:58
*** sree has joined #openstack-nova23:02
*** yangyapeng has quit IRC23:03
*** threestrands_ has joined #openstack-nova23:04
*** threestrands_ has quit IRC23:04
*** threestrands_ has joined #openstack-nova23:04
*** ijw has joined #openstack-nova23:05
*** kumarmn has joined #openstack-nova23:06
*** sree has quit IRC23:06
*** threestrands has quit IRC23:06
*** sree has joined #openstack-nova23:07
*** ijw has quit IRC23:09
*** gbarros has quit IRC23:09
*** kumarmn has quit IRC23:11
*** sree has quit IRC23:11
*** felipemonteiro_ has quit IRC23:16
*** huanxie has quit IRC23:20
*** jappleii__ has joined #openstack-nova23:20
*** felipemonteiro_ has joined #openstack-nova23:20
*** huanxie has joined #openstack-nova23:21
*** jappleii__ has quit IRC23:21
*** jappleii__ has joined #openstack-nova23:22
*** gmann_afk is now known as gmann23:22
*** threestrands_ has quit IRC23:22
gmannmriedem: is it fine like this (server filter whitelist in api-ref) - http://logs.openstack.org/42/527042/2/check/build-openstack-api-ref/b3307d7/html/#list-servers23:23
gmannpatch - https://review.openstack.org/#/c/527042/23:23
mriedemgmann: that's pretty hard to read23:24
gmanni tried to make it list but that's huge23:24
*** marst has quit IRC23:24
mriedemgmann: it would probably be better to create a page in the server concepts guide, with a sorted list that we can link to23:25
mriedemthat page can also point out which microversions added any filter item23:25
gmannmriedem: ok, that also good, ll do.23:26
*** mlavalle has quit IRC23:27
mriedemjaypipes: ironic multinode job passed on https://review.openstack.org/#/c/508555/ if you want to send that in23:27
*** jpena|away is now known as jpena|off23:28
gmannmriedem: other idea i was thinking- for admin just mention that whitelist is all keys mentioned in request section, and for non-admin we mention the whitelist  as list23:29
mriedemgmann: that could work since the non-admin whitelist is much smaller23:30
gmannyea23:30
gmannlet me push and  if that is clear enough23:31
gmannmriedem: thanks.23:32
mriedemyw23:32
* gmann hurry for bank 23:32
*** gmann is now known as gmann_afk23:32
*** Tom-Tom has quit IRC23:37
*** penick has quit IRC23:40
*** sree has joined #openstack-nova23:42
*** _ix has quit IRC23:43
*** felipemonteiro_ has quit IRC23:44
*** jogo has joined #openstack-nova23:45
*** baoli has joined #openstack-nova23:46
*** sree has quit IRC23:46
*** linkmark has quit IRC23:47
*** Apoorva has quit IRC23:48
*** Apoorva has joined #openstack-nova23:48
*** baoli_ has joined #openstack-nova23:50
*** baoli has quit IRC23:50
*** huanxie has quit IRC23:51
*** yangyapeng has joined #openstack-nova23:52
*** huanxie has joined #openstack-nova23:54
*** wxy has quit IRC23:55
*** karlamrhein has quit IRC23:55
*** DuncanT has quit IRC23:55
*** salv-orlando has quit IRC23:55
*** wxy has joined #openstack-nova23:56
*** johnsom has quit IRC23:56
openstackgerritMerged openstack/nova stable/pike: Only query BDMs once in API during rebuild  https://review.openstack.org/52563323:56
*** yangyapeng has quit IRC23:56
*** TheJulia has quit IRC23:56
*** melwitt has quit IRC23:57
*** Kvisle has quit IRC23:57
*** McNinja has quit IRC23:57
*** betherly has quit IRC23:57
*** ameade has quit IRC23:57
*** tinwood has quit IRC23:57
*** mwhahaha has quit IRC23:57
*** toan has quit IRC23:57
*** karlamrhein has joined #openstack-nova23:58
*** TheJulia has joined #openstack-nova23:58
*** AlexeyAbashkin has joined #openstack-nova23:58
*** tinwood has joined #openstack-nova23:58
*** johnsom has joined #openstack-nova23:58
openstackgerritMerged openstack/nova stable/pike: Get original image_id from volume for volume-backed instance rebuild  https://review.openstack.org/52563423:59
openstackgerritMerged openstack/nova stable/pike: Fix 'force' parameter in os-quota-sets PUT schema  https://review.openstack.org/52641923:59

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