*** yamahata has joined #openstack-nova | 00:01 | |
*** fragatina has joined #openstack-nova | 00:06 | |
*** masber has joined #openstack-nova | 00:07 | |
*** gouthamr has quit IRC | 00:09 | |
*** fragatina has quit IRC | 00:10 | |
*** fragatina has joined #openstack-nova | 00:11 | |
*** fragatina has quit IRC | 00:12 | |
*** yangyapeng has joined #openstack-nova | 00:19 | |
*** yangyapeng has quit IRC | 00:24 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete https://review.openstack.org/525231 | 00:29 |
---|---|---|
*** brault has joined #openstack-nova | 00:31 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: disable numa feature when virt_type is not kvm https://review.openstack.org/465160 | 00:32 |
*** yamamoto has joined #openstack-nova | 00:32 | |
*** takashin has joined #openstack-nova | 00:33 | |
*** brault has quit IRC | 00:36 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (2) https://review.openstack.org/526263 | 00:36 |
*** yamamoto has quit IRC | 00:40 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (3) https://review.openstack.org/526557 | 00:40 |
*** sbezverk has quit IRC | 00:40 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (3) https://review.openstack.org/526557 | 00:41 |
*** salv-orlando has joined #openstack-nova | 00:41 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: disable numa feature when virt_type is not kvm https://review.openstack.org/465160 | 00:44 |
*** sean-k-m1 has quit IRC | 00:45 | |
*** tetsuro_ has joined #openstack-nova | 00:45 | |
*** Dinesh_Bhor has joined #openstack-nova | 00:46 | |
*** salv-orlando has quit IRC | 00:46 | |
*** sean-k-m1 has joined #openstack-nova | 00:49 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in post https://review.openstack.org/526823 | 00:50 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in post https://review.openstack.org/526823 | 00: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-nova | 00:51 | |
*** Dinesh_Bhor has quit IRC | 00:52 | |
*** Dinesh_Bhor has joined #openstack-nova | 00:52 | |
*** hoangcx has joined #openstack-nova | 00:54 | |
*** Apoorva has quit IRC | 00:58 | |
*** linkmark has quit IRC | 00:59 | |
*** salv-orlando has joined #openstack-nova | 01:00 | |
*** tetsuro_ has quit IRC | 01:00 | |
*** liusheng has quit IRC | 01:02 | |
*** salv-orlando has quit IRC | 01:05 | |
*** yangyapeng has joined #openstack-nova | 01:07 | |
*** gouthamr has joined #openstack-nova | 01:08 | |
*** phuongnh has joined #openstack-nova | 01:08 | |
*** huanxie has quit IRC | 01:10 | |
openstackgerrit | zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526 | 01:11 |
*** r-daneel has joined #openstack-nova | 01:21 | |
*** liuzz has joined #openstack-nova | 01:35 | |
*** huanxie has joined #openstack-nova | 01:37 | |
*** hui has joined #openstack-nova | 01:40 | |
*** adreznec has joined #openstack-nova | 01:43 | |
*** asettle_ is now known as asettle | 01:45 | |
*** tetsuro has joined #openstack-nova | 01:46 | |
*** sapd_ has joined #openstack-nova | 01:50 | |
*** sapd__ has quit IRC | 01:50 | |
*** Apoorva has joined #openstack-nova | 01:59 | |
*** vladikr has joined #openstack-nova | 02:10 | |
*** markvoelker has quit IRC | 02:13 | |
*** dave-mccowan has joined #openstack-nova | 02:13 | |
*** annp has joined #openstack-nova | 02:15 | |
*** dave-mcc_ has joined #openstack-nova | 02:15 | |
*** dave-mccowan has quit IRC | 02:19 | |
*** oanson has quit IRC | 02:19 | |
*** oanson has joined #openstack-nova | 02:21 | |
*** felipemonteiro_ has quit IRC | 02:22 | |
*** Tom-Tom has joined #openstack-nova | 02:25 | |
*** dave-mccowan has joined #openstack-nova | 02:33 | |
*** dave-mcc_ has quit IRC | 02:35 | |
*** hieulq has quit IRC | 02:40 | |
*** hieulq has joined #openstack-nova | 02:43 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Fix API reference for microversion 1.14 https://review.openstack.org/526973 | 02:43 |
*** Tom-Tom has quit IRC | 02:45 | |
*** psachin has quit IRC | 02:49 | |
*** Tom-Tom has joined #openstack-nova | 02:50 | |
*** zhurong has joined #openstack-nova | 02:53 | |
*** vladikr has quit IRC | 02:54 | |
openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.57 - List/Show all server migration types https://review.openstack.org/430839 | 02:57 |
*** namnh has joined #openstack-nova | 03:02 | |
*** psachin has joined #openstack-nova | 03:03 | |
*** liverpooler has joined #openstack-nova | 03:03 | |
*** Tom-Tom has quit IRC | 03:09 | |
*** Tom-Tom has joined #openstack-nova | 03:10 | |
*** Tom-Tom has quit IRC | 03:10 | |
*** Tom-Tom has joined #openstack-nova | 03:11 | |
*** bkopilov has quit IRC | 03:14 | |
*** yamamoto has joined #openstack-nova | 03:14 | |
*** Tom-Tom has quit IRC | 03:16 | |
*** Apoorva has quit IRC | 03:18 | |
*** takashin has quit IRC | 03:23 | |
*** Tom-Tom has joined #openstack-nova | 03:25 | |
*** jichen has joined #openstack-nova | 03:31 | |
*** brault has joined #openstack-nova | 03:31 | |
*** baoli has quit IRC | 03:34 | |
*** brault has quit IRC | 03:36 | |
*** vivsoni has quit IRC | 03:36 | |
*** david-lyle has joined #openstack-nova | 03:38 | |
*** dave-mccowan has quit IRC | 03:41 | |
*** liverpooler has quit IRC | 03:41 | |
*** abhishekk has joined #openstack-nova | 03:42 | |
*** gouthamr has quit IRC | 03:45 | |
*** david-lyle has quit IRC | 03:45 | |
*** vivsoni has joined #openstack-nova | 03:52 | |
*** sean-k-m1 has quit IRC | 03:52 | |
*** Tom-Tom has quit IRC | 03:54 | |
*** Tom-Tom has joined #openstack-nova | 03:55 | |
*** Tom-Tom has quit IRC | 03:59 | |
*** sree has joined #openstack-nova | 04:01 | |
*** rmcall has quit IRC | 04:06 | |
*** liverpooler has joined #openstack-nova | 04:06 | |
*** Tom-Tom has joined #openstack-nova | 04:07 | |
*** links has joined #openstack-nova | 04:07 | |
*** links has quit IRC | 04:07 | |
*** sean-k-m1 has joined #openstack-nova | 04:11 | |
*** rmcall has joined #openstack-nova | 04:13 | |
*** markvoelker has joined #openstack-nova | 04:13 | |
openstackgerrit | Chen Hanxiao proposed openstack/nova master: libvirt: guest: introduce blockStats instead of domain.blockStats https://review.openstack.org/526833 | 04:19 |
*** zhurong has quit IRC | 04:26 | |
*** udesale has joined #openstack-nova | 04:27 | |
*** masuberu has joined #openstack-nova | 04:30 | |
*** masber has quit IRC | 04:33 | |
*** jmlowe has quit IRC | 04:33 | |
*** masuberu has quit IRC | 04:36 | |
*** armax has joined #openstack-nova | 04:36 | |
*** armax has quit IRC | 04:41 | |
*** markvoelker has quit IRC | 04:48 | |
*** bkopilov has joined #openstack-nova | 04:58 | |
*** trungnv has joined #openstack-nova | 05:01 | |
*** Dinesh_Bhor has quit IRC | 05:01 | |
*** Alex_Staf has joined #openstack-nova | 05:04 | |
*** fragatina has joined #openstack-nova | 05:13 | |
*** gbarros has quit IRC | 05:14 | |
*** takashin has joined #openstack-nova | 05:20 | |
*** liverpooler has quit IRC | 05:24 | |
*** liverpooler has joined #openstack-nova | 05:25 | |
*** zhurong has joined #openstack-nova | 05:32 | |
*** udesale__ has joined #openstack-nova | 05:34 | |
openstackgerrit | Chen Hanxiao proposed openstack/nova master: libvirt: throw NotImplementedError if qga is not responsive when setting password https://review.openstack.org/523828 | 05:34 |
*** ratailor has joined #openstack-nova | 05:35 | |
*** udesale has quit IRC | 05:37 | |
*** sridharg has joined #openstack-nova | 05:38 | |
*** masber has joined #openstack-nova | 05:42 | |
*** markvoelker has joined #openstack-nova | 05:45 | |
*** Dinesh_Bhor has joined #openstack-nova | 05:48 | |
*** fragatin_ has joined #openstack-nova | 05:49 | |
*** fragatina has quit IRC | 05:50 | |
*** diga has joined #openstack-nova | 05:50 | |
*** yamamoto has quit IRC | 06:08 | |
*** janki has joined #openstack-nova | 06:12 | |
*** karthiks has joined #openstack-nova | 06:15 | |
*** threestrands has quit IRC | 06:17 | |
*** bhagyashri_s is now known as bhagyashris | 06:17 | |
*** markvoelker has quit IRC | 06:19 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/524795 | 06:22 |
*** yamamoto has joined #openstack-nova | 06:22 | |
*** sapcc-bot3 has joined #openstack-nova | 06:22 | |
*** sapcc-bot has quit IRC | 06:22 | |
*** liusheng has joined #openstack-nova | 06:28 | |
*** diga has quit IRC | 06:32 | |
*** yamamoto has quit IRC | 06:32 | |
*** toabctl has joined #openstack-nova | 06:41 | |
*** ircuser-1 has joined #openstack-nova | 06:42 | |
*** pooja_jadhav has joined #openstack-nova | 06:43 | |
*** pooja-jadhav has joined #openstack-nova | 06:43 | |
*** pooja-jadhav has quit IRC | 06:43 | |
*** hshiina has joined #openstack-nova | 06:44 | |
*** bkopilov has quit IRC | 06:45 | |
*** udesale has joined #openstack-nova | 06:46 | |
*** udesale__ has quit IRC | 06:48 | |
*** zhurong has quit IRC | 06:50 | |
*** moshele has joined #openstack-nova | 06:52 | |
*** bkopilov has joined #openstack-nova | 07:04 | |
*** salv-orlando has joined #openstack-nova | 07:08 | |
*** josecastroleon has joined #openstack-nova | 07:10 | |
*** vivsoni has quit IRC | 07:10 | |
*** Tom-Tom has quit IRC | 07:12 | |
*** Tom-Tom has joined #openstack-nova | 07:13 | |
*** edand has joined #openstack-nova | 07:15 | |
*** Alex_Staf has quit IRC | 07:15 | |
*** markvoelker has joined #openstack-nova | 07:16 | |
*** hoonetorg has quit IRC | 07:18 | |
*** Tom-Tom_ has joined #openstack-nova | 07:20 | |
*** adisky_ has joined #openstack-nova | 07:20 | |
*** Tom-Tom has quit IRC | 07:22 | |
*** Tom-Tom__ has joined #openstack-nova | 07:22 | |
*** Tom-Tom_ has quit IRC | 07:25 | |
*** namnh has quit IRC | 07:29 | |
*** namnh_ has joined #openstack-nova | 07:29 | |
*** hoonetorg has joined #openstack-nova | 07:31 | |
*** yamamoto has joined #openstack-nova | 07:33 | |
*** diga has joined #openstack-nova | 07:36 | |
*** yamamoto has quit IRC | 07:38 | |
*** yamamoto has joined #openstack-nova | 07:38 | |
*** lajoskatona has joined #openstack-nova | 07:39 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for snapshot instances https://review.openstack.org/525921 | 07:41 |
*** sshwarts has joined #openstack-nova | 07:42 | |
*** rcernin has quit IRC | 07:43 | |
*** salv-orlando has quit IRC | 07:44 | |
*** salv-orlando has joined #openstack-nova | 07:44 | |
*** AlexeyAbashkin has joined #openstack-nova | 07:46 | |
*** Rambo has joined #openstack-nova | 07:48 | |
*** salv-orlando has quit IRC | 07:48 | |
*** markvoelker has quit IRC | 07:49 | |
*** moshele has quit IRC | 07:50 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for snapshot instances https://review.openstack.org/525921 | 07:50 |
*** yamamoto has quit IRC | 07:52 | |
*** rmcall has quit IRC | 07:54 | |
*** salv-orlando has joined #openstack-nova | 07:56 | |
*** sahid has joined #openstack-nova | 07:57 | |
*** ralonsoh has joined #openstack-nova | 07:59 | |
*** takashin has left #openstack-nova | 08:02 | |
openstackgerrit | Silvan Kaiser proposed openstack/nova master: Added mount fstype based validation of Quobyte mounts https://review.openstack.org/522245 | 08:05 |
*** yamamoto has joined #openstack-nova | 08:06 | |
*** andreas_s has joined #openstack-nova | 08:06 | |
*** vivsoni has joined #openstack-nova | 08:12 | |
*** ttsiouts has quit IRC | 08:12 | |
*** andreas_s has quit IRC | 08:13 | |
*** andreas_s has joined #openstack-nova | 08:13 | |
*** phuongnh has quit IRC | 08:13 | |
*** andreas_s has quit IRC | 08:13 | |
*** andreas_s has joined #openstack-nova | 08:14 | |
*** damien_r has joined #openstack-nova | 08:14 | |
*** andreas__ has joined #openstack-nova | 08:15 | |
*** giblet is now known as gibi | 08:17 | |
gibi | good morning nova | 08:17 |
*** andreas_s has quit IRC | 08:18 | |
*** alexchadin has joined #openstack-nova | 08:19 | |
*** andreas__ has quit IRC | 08:19 | |
*** damien_r has left #openstack-nova | 08:19 | |
*** andreas_s has joined #openstack-nova | 08:26 | |
openstackgerrit | jichenjc proposed openstack/nova master: Add quiesce and unquiesce in support matrix https://review.openstack.org/482390 | 08:27 |
*** damien_r has joined #openstack-nova | 08:28 | |
openstackgerrit | jichenjc proposed openstack/nova master: deprecate fping_path config option https://review.openstack.org/526602 | 08:29 |
*** andreas_s has quit IRC | 08:30 | |
*** andreas_s has joined #openstack-nova | 08:30 | |
gmann | alex_xu: ping | 08:30 |
gmann | gibi: good morning | 08:31 |
gmann | alex_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.html | 08:31 |
gibi | gmann: I assume you are already close to the end your Monday. How lucky you are. :) | 08:32 |
gmann | alex_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/1 | 08:32 |
gmann | gibi: yea, heh but unlucky too to get monday 1 day before you guys :) | 08:33 |
gmann | alex_xu: it got triggered while reviewing this -https://review.openstack.org/#/c/526558/1 | 08:33 |
gmann | alex_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 Rocky | 08:34 |
gmann | so that people does not get confused with new policy rules for all_tenants mentioned in spec | 08:35 |
*** ludovic has joined #openstack-nova | 08:38 | |
*** salv-orlando has quit IRC | 08:39 | |
ludovic | Hello all, maybe someone can help to determine what i have a lot of urllib3.connectionpool full in nova.conf ? | 08:41 |
*** hoangcx has quit IRC | 08:41 | |
ludovic | When i use nova host-evacuate , a lot of error VMs are in Error state with Gateway timeout Error | 08:42 |
ludovic | So i really need help | 08:42 |
*** vivsoni has quit IRC | 08:43 | |
*** vivsoni has joined #openstack-nova | 08:43 | |
*** andreas_s has quit IRC | 08:45 | |
*** jpena|off is now known as jpena | 08:45 | |
*** Alex_Staf has joined #openstack-nova | 08:45 | |
*** andreas_s has joined #openstack-nova | 08:45 | |
*** markvoelker has joined #openstack-nova | 08:46 | |
*** hoangcx has joined #openstack-nova | 08:46 | |
*** rcernin has joined #openstack-nova | 08:47 | |
*** andreas_s has quit IRC | 08:50 | |
*** vivsoni has quit IRC | 08:50 | |
*** brault has joined #openstack-nova | 08:53 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs master: Few correction in the server filter/sort spec https://review.openstack.org/527019 | 08:55 |
gmann | alex_xu: ^^ | 08:56 |
*** andreas_s has joined #openstack-nova | 08:56 | |
*** hshiina has quit IRC | 08:57 | |
*** brault has quit IRC | 08:57 | |
*** andreas__ has joined #openstack-nova | 08:58 | |
*** brault has joined #openstack-nova | 08:59 | |
*** andreas_s has quit IRC | 09:01 | |
*** brault has quit IRC | 09:01 | |
*** salv-orlando has joined #openstack-nova | 09:01 | |
*** brault has joined #openstack-nova | 09:01 | |
*** andreas_s has joined #openstack-nova | 09:01 | |
openstackgerrit | jichenjc proposed openstack/nova master: [Placement] Add create inventories doc for placement https://review.openstack.org/512215 | 09:02 |
*** andreas__ has quit IRC | 09:03 | |
*** andreas_s has quit IRC | 09:04 | |
*** andreas_s has joined #openstack-nova | 09:04 | |
*** andreas_s has quit IRC | 09:04 | |
*** Jack_Iv has joined #openstack-nova | 09:07 | |
*** sean-k-m1 has quit IRC | 09:09 | |
*** damien_r has quit IRC | 09:10 | |
openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova master: Ensure that periodic reclaim cleans DB deleted instances https://review.openstack.org/323250 | 09:11 |
*** damien_r has joined #openstack-nova | 09:11 | |
*** vivsoni has joined #openstack-nova | 09:12 | |
*** sean-k-m1 has joined #openstack-nova | 09:12 | |
*** markvoelker has quit IRC | 09:20 | |
openstackgerrit | Merged openstack/nova stable/pike: Fix binary name https://review.openstack.org/524531 | 09:20 |
*** brault has quit IRC | 09:26 | |
*** andreas_s has joined #openstack-nova | 09:27 | |
openstackgerrit | sahid proposed openstack/nova master: libvirt: disconnect volume from host during detach https://review.openstack.org/515008 | 09:27 |
alex_xu | gmann: thanks, I will check it | 09:29 |
alex_xu | gmann: btw, I'm on PTO, will back to work on Thursday | 09:30 |
openstackgerrit | jichenjc proposed openstack/nova master: deprecate fping_path config option https://review.openstack.org/526602 | 09:31 |
*** andreas_s has quit IRC | 09:32 | |
*** yamamoto has quit IRC | 09:33 | |
*** andreas_s has joined #openstack-nova | 09:33 | |
*** lucas-afk is now known as lucasagomes | 09:33 | |
*** andreas_s has quit IRC | 09:33 | |
*** yamamoto has joined #openstack-nova | 09:34 | |
*** andreas_s has joined #openstack-nova | 09:37 | |
gmann | alex_xu: oh, sure no hurry. enjoy | 09:41 |
*** andreas_s has quit IRC | 09:42 | |
ludovic | someone is familiar with networking issue like urllib3 connection pool full in nova or cinder ? | 09:43 |
*** Dinesh_Bhor has quit IRC | 09:43 | |
*** andreas_s has joined #openstack-nova | 09:45 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for snapshot instances https://review.openstack.org/525921 | 09:49 |
*** finucannot is now known as stephenfin | 09:52 | |
*** sree_ has joined #openstack-nova | 09:53 | |
*** sree_ is now known as Guest28012 | 09:53 | |
*** jichen has quit IRC | 09:54 | |
*** derekh has joined #openstack-nova | 09:55 | |
*** gszasz has joined #openstack-nova | 09:56 | |
*** sree has quit IRC | 09:56 | |
*** trungnv has quit IRC | 09:56 | |
*** brault has joined #openstack-nova | 09:58 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add server filters whitelist in server api-ref https://review.openstack.org/527042 | 09:58 |
*** gmann is now known as gmann_afk | 09:58 | |
*** Guest28012 has quit IRC | 09:59 | |
*** andreas_s has quit IRC | 10:04 | |
*** brault has quit IRC | 10:04 | |
*** ttsiouts has joined #openstack-nova | 10:08 | |
*** ludovic has quit IRC | 10:08 | |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: [DNM] test https://review.openstack.org/527045 | 10:10 |
*** tetsuro has quit IRC | 10:12 | |
openstackgerrit | Lee Yarwood proposed openstack/nova master: conf: Do not inherit image signature props with snapshots https://review.openstack.org/527046 | 10:12 |
*** andreas_s has joined #openstack-nova | 10:12 | |
*** brault has joined #openstack-nova | 10:13 | |
*** tetsuro has joined #openstack-nova | 10:14 | |
*** andreas__ has joined #openstack-nova | 10:14 | |
*** andreas_s has quit IRC | 10:17 | |
*** brault has quit IRC | 10:18 | |
*** andreas__ has quit IRC | 10:19 | |
*** andreas_s has joined #openstack-nova | 10:23 | |
*** andreas_s has quit IRC | 10:32 | |
*** cdent has joined #openstack-nova | 10:35 | |
*** annp has quit IRC | 10:35 | |
*** sambetts_ is now known as sambetts | 10:36 | |
*** jmlowe has joined #openstack-nova | 10:38 | |
*** abhishekk has quit IRC | 10:39 | |
*** adisky_ has quit IRC | 10:40 | |
*** alexchadin has quit IRC | 10:42 | |
*** andreas_s has joined #openstack-nova | 10:43 | |
*** yamamoto has quit IRC | 10:47 | |
*** yamamoto has joined #openstack-nova | 10:47 | |
*** andreas_s has quit IRC | 10:47 | |
*** erlon has joined #openstack-nova | 10:48 | |
*** yamamoto has quit IRC | 10:52 | |
*** alexchadin has joined #openstack-nova | 10:52 | |
*** sdague has joined #openstack-nova | 10:53 | |
*** alexchadin has quit IRC | 10:56 | |
*** alexchadin has joined #openstack-nova | 10:57 | |
*** mdnadeem has joined #openstack-nova | 10:57 | |
*** abhishekk has joined #openstack-nova | 10:59 | |
*** abhishekk has quit IRC | 10:59 | |
*** namnh_ has quit IRC | 11:03 | |
*** moshele has joined #openstack-nova | 11:04 | |
*** coreywright has quit IRC | 11:06 | |
*** andreas_s has joined #openstack-nova | 11:08 | |
*** Jack_Iv has quit IRC | 11:08 | |
*** andreas_s has quit IRC | 11:12 | |
*** edand has quit IRC | 11:16 | |
*** markvoelker has joined #openstack-nova | 11:17 | |
*** Jack_Iv has joined #openstack-nova | 11:18 | |
*** edand has joined #openstack-nova | 11:18 | |
*** Guest56845 has quit IRC | 11:19 | |
*** udesale has quit IRC | 11:19 | |
*** andreas_s has joined #openstack-nova | 11:19 | |
*** sree has joined #openstack-nova | 11:22 | |
*** Jack_Iv has quit IRC | 11:23 | |
*** andreas_s has quit IRC | 11:23 | |
*** AlexeyAbashkin has quit IRC | 11:25 | |
*** sree has quit IRC | 11:26 | |
*** salv-orlando has quit IRC | 11:26 | |
*** andreas_s has joined #openstack-nova | 11:27 | |
*** sree has joined #openstack-nova | 11:27 | |
*** salv-orlando has joined #openstack-nova | 11:27 | |
*** sree has quit IRC | 11:32 | |
*** andreas_s has quit IRC | 11:35 | |
*** ratailor has quit IRC | 11:41 | |
*** dtantsur|afk is now known as dtantsur | 11:43 | |
*** andreas_s has joined #openstack-nova | 11:45 | |
*** diga has quit IRC | 11:45 | |
*** AlexeyAbashkin has joined #openstack-nova | 11:45 | |
*** sree has joined #openstack-nova | 11:47 | |
*** liverpooler has quit IRC | 11:48 | |
*** linkmark has joined #openstack-nova | 11:49 | |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: [DNM] test https://review.openstack.org/527045 | 11:50 |
*** markvoelker has quit IRC | 11:50 | |
dtantsur | morning, compute overlords and overladies | 11:51 |
*** sree has quit IRC | 11:51 | |
dtantsur | we're experiencing a new nova-related failure in the ironic multinode job: https://bugs.launchpad.net/ironic/+bug/1737395 | 11:51 |
openstack | Launchpad bug 1737395 in Ironic "n-cpu fails to start in the multinode job: Conflicting resource provider name: <uuid> already exists" [High,Confirmed] | 11:51 |
dtantsur | do you think any recent merge could cause it? | 11:51 |
*** rodolof has joined #openstack-nova | 11:54 | |
*** janki has quit IRC | 11:56 | |
*** lpetrut has joined #openstack-nova | 11:57 | |
*** andreas_s has quit IRC | 11:58 | |
*** andreas_s has joined #openstack-nova | 11:59 | |
*** gszasz has quit IRC | 12:00 | |
*** gszasz has joined #openstack-nova | 12:00 | |
cdent | dtantsur: might be https://review.openstack.org/#/c/524263/ ? | 12:00 |
cdent | when did it start? | 12:00 |
dtantsur | cdent: over the weekend, maybe Friday evening | 12:00 |
cdent | hmmm, that’s a bit older than that | 12:01 |
*** Tom-Tom__ has quit IRC | 12:01 | |
*** Tom-Tom has joined #openstack-nova | 12:02 | |
*** yamamoto has joined #openstack-nova | 12:02 | |
cdent | dtantsur: is this the thing that https://review.openstack.org/#/c/508555/ is supposed to fix? | 12:02 |
*** Rambo has quit IRC | 12:03 | |
*** Tom-Tom_ has joined #openstack-nova | 12:03 | |
*** andreas_s has quit IRC | 12:04 | |
dtantsur | cdent: it looks so, but it's the first time I see this patch | 12:04 |
dtantsur | johnthetubaguy: around? re ^^^ | 12:04 |
*** sahid has quit IRC | 12:04 | |
cdent | It 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-nova | 12:05 | |
*** Tom-Tom has quit IRC | 12:06 | |
*** dave-mccowan has joined #openstack-nova | 12:06 | |
*** mdnadeem has quit IRC | 12:06 | |
dtantsur | (moving conversation from #openstack-ironic) | 12:07 |
dtantsur | sambetts: so yeah, seems like we need to rebase and merge this.. | 12:07 |
dtantsur | cdent: do you know if johnthetubaguy is around nowadays? | 12:07 |
*** sree has joined #openstack-nova | 12:07 | |
*** Tom-Tom_ has quit IRC | 12:08 | |
*** Jack_Iv has joined #openstack-nova | 12:08 | |
dtantsur | one of us can rebase the patch, but I don't want to step on his shoes | 12:08 |
cdent | I haven’t seen him much lately, so I would think it would be okay to adopt | 12:08 |
dtantsur | sambetts or vdrok, wanna do ^^^ or should I? | 12:09 |
*** andreas_s has joined #openstack-nova | 12:10 | |
*** sree has quit IRC | 12:11 | |
* sambetts is currently fighting third party CI fires so don't have the time right now :( | 12:12 | |
*** sahid has joined #openstack-nova | 12:12 | |
*** ijw has joined #openstack-nova | 12:12 | |
*** yassine has joined #openstack-nova | 12:13 | |
*** dave-mcc_ has joined #openstack-nova | 12:14 | |
*** yassine is now known as Guest52604 | 12:14 | |
dtantsur | okie | 12:14 |
*** ralonsoh has quit IRC | 12:15 | |
*** dave-mccowan has quit IRC | 12:16 | |
*** ralonsoh has joined #openstack-nova | 12:16 | |
*** BryanS68 has joined #openstack-nova | 12:17 | |
*** Tom-Tom has joined #openstack-nova | 12:18 | |
*** andreas_s has quit IRC | 12:19 | |
*** lucasagomes is now known as lucas-hungry | 12:19 | |
*** Tom-Tom has quit IRC | 12:22 | |
*** Jack_Iv_ has joined #openstack-nova | 12:23 | |
*** jpena is now known as jpena|lunch | 12:25 | |
*** andreas_s has joined #openstack-nova | 12:25 | |
*** cdent has quit IRC | 12:27 | |
*** alexchadin has quit IRC | 12:27 | |
*** Jack_Iv has quit IRC | 12:27 | |
openstackgerrit | Dmitry Tantsur proposed openstack/nova master: Re-use existing ComputeNode on ironic rebalance https://review.openstack.org/508555 | 12:27 |
dtantsur | I'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-nova | 12:27 | |
*** ralonsoh has quit IRC | 12:28 | |
*** ralonsoh has joined #openstack-nova | 12:28 | |
*** jaypipes has joined #openstack-nova | 12:31 | |
*** andreas_s has quit IRC | 12:34 | |
*** gszasz has quit IRC | 12:35 | |
*** andreas_s has joined #openstack-nova | 12:35 | |
*** Roamer` has joined #openstack-nova | 12:35 | |
*** andreas_s has quit IRC | 12:37 | |
*** salv-orlando has quit IRC | 12:38 | |
*** andreas_s has joined #openstack-nova | 12:38 | |
*** salv-orlando has joined #openstack-nova | 12:40 | |
*** andreas_s has quit IRC | 12:42 | |
openstackgerrit | Merged openstack/nova stable/pike: Document the real behavior of notify_on_state_change https://review.openstack.org/521870 | 12:42 |
*** tetsuro has quit IRC | 12:43 | |
*** smatzek has joined #openstack-nova | 12:45 | |
*** markvoelker has joined #openstack-nova | 12:47 | |
*** sahid has quit IRC | 12:48 | |
*** dtantsur is now known as dtantsur|brb | 12:50 | |
*** psachin has quit IRC | 12:53 | |
*** andreas_s has joined #openstack-nova | 12:57 | |
*** BryanS68 has quit IRC | 12:58 | |
*** gszasz has joined #openstack-nova | 12:59 | |
*** lennyb has quit IRC | 13:00 | |
*** andreas_s has quit IRC | 13:01 | |
*** rodolof has quit IRC | 13:02 | |
*** rodolof has joined #openstack-nova | 13:02 | |
*** lennyb has joined #openstack-nova | 13:02 | |
*** udesale has joined #openstack-nova | 13:03 | |
*** jangutter has joined #openstack-nova | 13:06 | |
*** sree has joined #openstack-nova | 13:07 | |
*** ChanServ has quit IRC | 13:08 | |
*** vladikr has joined #openstack-nova | 13:09 | |
*** baoli has joined #openstack-nova | 13:10 | |
*** tssurya has joined #openstack-nova | 13:10 | |
*** andreas_s has joined #openstack-nova | 13:11 | |
*** sree has quit IRC | 13:11 | |
*** andreas_s has quit IRC | 13:15 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660 | 13:17 |
*** ChanServ has joined #openstack-nova | 13:18 | |
*** barjavel.freenode.net sets mode: +o ChanServ | 13:18 | |
*** yangyapeng has quit IRC | 13:19 | |
*** yangyapeng has joined #openstack-nova | 13:20 | |
*** janki has joined #openstack-nova | 13:20 | |
*** janki has quit IRC | 13:20 | |
*** janki has joined #openstack-nova | 13:20 | |
*** markvoelker has quit IRC | 13:21 | |
*** andreas_s has joined #openstack-nova | 13:22 | |
*** rcernin has quit IRC | 13:23 | |
*** sahid has joined #openstack-nova | 13:24 | |
*** lucas-hungry is now known as lucasagomes | 13:25 | |
*** yangyapeng has quit IRC | 13:25 | |
*** andreas_s has quit IRC | 13:26 | |
*** jpena|lunch is now known as jpena | 13:27 | |
*** moshele has quit IRC | 13:28 | |
*** moshele has joined #openstack-nova | 13:29 | |
*** cdent has joined #openstack-nova | 13:31 | |
*** markvoelker has joined #openstack-nova | 13:38 | |
*** sahid_ has joined #openstack-nova | 13:38 | |
*** pchavva has joined #openstack-nova | 13:39 | |
*** andreas_s has joined #openstack-nova | 13:39 | |
*** andreas_s has quit IRC | 13:40 | |
*** andreas_s has joined #openstack-nova | 13:40 | |
*** sahid has quit IRC | 13:41 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add server filters whitelist in server api-ref https://review.openstack.org/527042 | 13:42 |
jaypipes | happy Monday morning, supernovas! Who's still working this week? | 13:42 |
gibi | jaypipes: good morning! I'm still here until 20th of Dec | 13:43 |
*** baoli has quit IRC | 13:43 | |
jaypipes | gibi: cool :) | 13:43 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Add uuid column to BlockDeviceMapping https://review.openstack.org/242602 | 13:43 |
edleafe | jaypipes: unfortunately, me | 13:43 |
jaypipes | glad to hear it! | 13:43 |
jaypipes | edleafe: :) | 13:43 |
*** Tom-Tom has joined #openstack-nova | 13:46 | |
openstackgerrit | Matthew Booth proposed openstack/nova master: Make BlockDeviceMapping object support uuid https://review.openstack.org/242603 | 13:46 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Add an online migration for BDM.uuid https://review.openstack.org/525599 | 13:46 |
cdent | jaypipes: I’ll be around until… uh 22nd I guess | 13:46 |
*** andreas_s has quit IRC | 13:47 | |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: [DNM] test https://review.openstack.org/527045 | 13:47 |
jaypipes | cdent: cool beans. | 13:48 |
*** dtantsur|brb is now known as dtantsur | 13:49 | |
edleafe | For those around, nova scheduler subteam meeting in 10 minutes in #openstack-meeting-alt | 13:50 |
*** sree has joined #openstack-nova | 13:50 | |
*** liverpooler has joined #openstack-nova | 13:51 | |
* mdbooth is around for the long haul | 13:51 | |
*** sree_ has joined #openstack-nova | 13:51 | |
*** andreas_s has joined #openstack-nova | 13:51 | |
*** sree_ is now known as Guest52880 | 13:51 | |
mdbooth | jaypipes: 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 |
mdbooth | Would also appreciate a second opinion on the subject of unique constraints on nullable columns, but I did research that one carefully. | 13:54 |
mdbooth | In short, I'm very much in favour of them. | 13:54 |
*** sree has quit IRC | 13:54 | |
*** mriedem has joined #openstack-nova | 13:55 | |
*** yangyapeng has joined #openstack-nova | 13:57 | |
*** andreas_s has quit IRC | 13:57 | |
jaypipes | mdbooth: will re-review shortly. | 13:57 |
mdbooth | jaypipes: Thanks | 13:57 |
*** sbezverk has joined #openstack-nova | 13:59 | |
*** gszasz has quit IRC | 14:03 | |
*** andreas__ has joined #openstack-nova | 14:05 | |
*** udesale has quit IRC | 14:05 | |
*** gszasz has joined #openstack-nova | 14:06 | |
mriedem | jaypipes: 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 IRC | 14:10 | |
mriedem | like if we're updating multiple tables in a single method | 14:10 |
*** Tom-Tom has quit IRC | 14:11 | |
*** purplerbot has joined #openstack-nova | 14:12 | |
*** esberglu has joined #openstack-nova | 14:12 | |
*** baoli has joined #openstack-nova | 14:12 | |
cdent | mriedem: or flush is callewd | 14:13 |
*** weshay|MOD is now known as weshay|sickday | 14:13 | |
cdent | (which sometimes happens to find out an id() | 14:13 |
cdent | keyboards in the cold suck, especially when you already suck at typing | 14:14 |
mriedem | trying 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 decorator | 14:14 |
mriedem | cdent: radiant space heater for xmas | 14:14 |
*** cdent has quit IRC | 14:15 | |
*** cdent has joined #openstack-nova | 14:15 | |
*** Tom-Tom has joined #openstack-nova | 14:15 | |
*** huanxie has quit IRC | 14:16 | |
*** alexchadin has quit IRC | 14:16 | |
*** andreas_s has joined #openstack-nova | 14:16 | |
*** alexchadin has joined #openstack-nova | 14:16 | |
*** alexchadin has quit IRC | 14:17 | |
*** alexchadin has joined #openstack-nova | 14:17 | |
*** alexchadin has quit IRC | 14:17 | |
jaypipes | mriedem: that is correct. | 14:18 |
*** huanxie has joined #openstack-nova | 14:19 | |
mriedem | jaypipes: ok - so i guess we just have ot use the deadlock decorator in ^ then | 14:20 |
mriedem | unless anyone else has bright ideas | 14:20 |
jaypipes | mriedem: looking at it now. | 14:20 |
*** andreas_s has quit IRC | 14:20 | |
*** takashin has joined #openstack-nova | 14:21 | |
*** diga has joined #openstack-nova | 14:21 | |
*** tssurya has quit IRC | 14:23 | |
*** yangyapeng has quit IRC | 14:24 | |
*** gcb has joined #openstack-nova | 14:24 | |
*** yangyapeng has joined #openstack-nova | 14:25 | |
*** cleong has joined #openstack-nova | 14:25 | |
dtantsur | folks, can someone please take a look at https://review.openstack.org/#/c/508555/ ? it fixes our CI apparently | 14:26 |
dtantsur | mriedem, jaypipes ^^^ | 14:26 |
*** gouthamr has joined #openstack-nova | 14:26 | |
jaypipes | dtantsur: yes, will do. | 14:26 |
cdent | dtantsur: so that was it? I’m surprised it didn’t show up a bit sooner. | 14:27 |
dtantsur | cdent: all I can say that a patch with Depends-On it has passed the CI :) | 14:27 |
cdent | well it presumably needs to happen anyway, so motion in the right direction | 14:28 |
*** lajoskatona has quit IRC | 14:28 | |
dtantsur | yep | 14:28 |
openstackgerrit | Merged openstack/python-novaclient master: Remove deprecated command in nova.rst https://review.openstack.org/526569 | 14:28 |
jaypipes | mriedem: 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 |
jaypipes | mriedem: which doesn't seem like it should happen, no? | 14:29 |
*** josecastroleon1 has joined #openstack-nova | 14:29 | |
*** josecastroleon has quit IRC | 14:29 | |
jaypipes | mriedem: 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-nova | 14:29 | |
*** smatzek has quit IRC | 14:30 | |
jaypipes | mriedem: unless request ID is not a UUID? | 14:30 |
jaypipes | mriedem: if request ID is an auto-inc, it could happen. | 14:30 |
*** smatzek has joined #openstack-nova | 14:30 | |
*** smatzek has quit IRC | 14:30 | |
mriedem | request ID should be unique per user request | 14:30 |
*** edand has quit IRC | 14:31 | |
mriedem | generated by some keystone or oslo middleware i tihnk | 14:31 |
jaypipes | mriedem: hmm, not necessarily... | 14:31 |
mriedem | https://github.com/openstack/oslo.middleware/blob/master/oslo_middleware/request_id.py | 14:31 |
mriedem | the failures all seem to happen on a resize test | 14:31 |
jaypipes | mriedem: see the comment and code on line 6302-6310 | 14:32 |
diga | jaypipes: Hi | 14:32 |
jaypipes | mriedem: seems that on a service restart, the action is retrieved by getting the "last created by instance UUID" | 14:32 |
jaypipes | mriedem: I'm wondering if that is returning an action ID that another greenthread might be operating on? | 14:32 |
mriedem | jaypipes: maybe, | 14:33 |
jaypipes | diga: hey. just saw your email. will email you an answer, ok? | 14:33 |
mriedem | we wouldn't have a compute restart in the middle of this tempest run | 14:33 |
diga | jaypipes: I have my setup ready but how to reproduce the issue is the big changel I am facing | 14:33 |
diga | jaypipes: sure | 14:33 |
mriedem | jaypipes: but we probably shouldn't update those kinds of actions anyway | 14:33 |
mriedem | so good spot | 14:33 |
mriedem | jaypipes: this is failing resize testing where we're rpc'ing back and forth with the same user context, | 14:34 |
mriedem | to decorated methods like prep_resize, resize_instance, finish_resize, etc | 14:34 |
mriedem | so it's possible we are hitting this quickly | 14:34 |
mriedem | jaypipes: yeah i think that's probably what's happening here | 14:35 |
mriedem | prep_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 again | 14:36 |
mriedem | 3 updates within a short window | 14:36 |
*** sahid_ has quit IRC | 14:37 | |
*** erlon has quit IRC | 14:38 | |
mriedem | jaypipes: left comments, thanks for the help | 14:39 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Change RPC for select_destinations() https://review.openstack.org/516707 | 14:40 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils https://review.openstack.org/511357 | 14:40 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Make conductor pass and use host_lists https://review.openstack.org/511358 | 14:40 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize https://review.openstack.org/526436 | 14:40 |
edleafe | mriedem: ^^ rebased | 14:40 |
mriedem | yar | 14:40 |
* edleafe goes to make more coffee | 14:40 | |
jaypipes | mriedem: not sure I really helped, but ok :) | 14:40 |
mriedem | you greased my brain gears | 14:41 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638 | 14:42 |
superdan | mriedem: that xen thing merged and this is fixed up ^ | 14:42 |
superdan | whoa | 14:42 |
*** superdan is now known as dansmith | 14:42 | |
*** brault has joined #openstack-nova | 14:42 | |
*** peter-hamilton has joined #openstack-nova | 14:45 | |
*** alexchadin has joined #openstack-nova | 14:45 | |
*** brault has quit IRC | 14:47 | |
*** tssurya has joined #openstack-nova | 14:47 | |
*** alexchadin has quit IRC | 14:48 | |
*** burt has joined #openstack-nova | 14:49 | |
*** huanxie has quit IRC | 14:50 | |
*** sahid has joined #openstack-nova | 14:50 | |
*** huanxie has joined #openstack-nova | 14:50 | |
*** sahid has quit IRC | 14:51 | |
*** david-lyle has joined #openstack-nova | 14:51 | |
mriedem | dansmith: service version needs to be fixed | 14:51 |
mriedem | otherwise lgtm | 14:51 |
*** sahid has joined #openstack-nova | 14:51 | |
dansmith | eh I bumped it? | 14:51 |
mriedem | rebase fail | 14:52 |
dansmith | oh doh | 14:52 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638 | 14:52 |
dansmith | dammit, hang on | 14:52 |
dansmith | sheesh | 14:53 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638 | 14:53 |
dansmith | did half of that on friday and then completed before max caffeine absorption = fail | 14:53 |
*** yamamoto has quit IRC | 14:53 | |
*** felipemonteiro_ has joined #openstack-nova | 14:55 | |
*** felipemonteiro__ has joined #openstack-nova | 14:56 | |
*** andreas_s has joined #openstack-nova | 14:56 | |
mriedem | +2 | 14:57 |
*** baoli has quit IRC | 14:57 | |
mriedem | gibi: you've been on this series before https://review.openstack.org/#/c/507638/ | 14:58 |
mriedem | ^ is the last patch | 14:58 |
*** shaner has quit IRC | 14:59 | |
*** baoli has joined #openstack-nova | 15:00 | |
*** baoli has quit IRC | 15:00 | |
*** felipemonteiro_ has quit IRC | 15:00 | |
jaypipes | mriedem: easy one. https://review.openstack.org/#/c/526973/ | 15:00 |
*** andreas_s has quit IRC | 15:01 | |
*** artom has joined #openstack-nova | 15:01 | |
*** lyan has joined #openstack-nova | 15:02 | |
*** ralonsoh has quit IRC | 15:02 | |
*** baoli has joined #openstack-nova | 15:04 | |
mriedem | jaypipes: this must be a response sample? https://review.openstack.org/#/c/526973/1/placement-api-ref/source/update-resource_provider.json | 15:05 |
*** cdent has quit IRC | 15:05 | |
*** shaner has joined #openstack-nova | 15:05 | |
mriedem | yeah i guess http://logs.openstack.org/73/526973/1/check/build-placement-api-ref/cbc7192/html/#id11 | 15:05 |
*** andreas_s has joined #openstack-nova | 15:06 | |
jaypipes | yessir | 15:06 |
*** smatzek has joined #openstack-nova | 15:06 | |
mriedem | it's weird that we have 2 source files for resource providers for api-ref | 15:06 |
mriedem | once has create/list, the other has show/update | 15:06 |
mriedem | *and delete | 15:06 |
*** yamamoto has joined #openstack-nova | 15:06 | |
*** peter-hamilton has quit IRC | 15:08 | |
mriedem | anyway something takashin could fix up later, | 15:08 |
*** pchavva has quit IRC | 15:08 | |
mriedem | +W | 15:08 |
*** stelucz has quit IRC | 15:08 | |
mriedem | lyarwood: did you say you're back now? | 15:09 |
*** sean-k-m1 has quit IRC | 15:09 | |
lyarwood | mriedem: I am indeed | 15:09 |
mriedem | lyarwood: 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/pike | 15:10 |
mriedem | if you would | 15:10 |
mriedem | i need to do a release by end of day | 15:10 |
lyarwood | mriedem: ack, did a few at the end of the list earlier, I'll start on these now | 15:10 |
mriedem | thanks | 15:10 |
lyarwood | mriedem: 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 |
lyarwood | mriedem: I'd rather not tbh, just wondering if that's now policy on stable or something | 15:11 |
*** yamamoto has quit IRC | 15:12 | |
mriedem | lyarwood: 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 now | 15:12 |
mriedem | dansmith did a bunch of those +2s already | 15:12 |
lyarwood | mriedem: kk cool | 15:13 |
mriedem | andreas_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.gz | 15:14 |
mriedem | 2017-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 directory | 15:14 |
andreas_s | mriedem: 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 |
mriedem | ok | 15:15 |
*** brault has joined #openstack-nova | 15:15 | |
*** alexchadin has joined #openstack-nova | 15:17 | |
*** brault has quit IRC | 15:17 | |
*** brault has joined #openstack-nova | 15:18 | |
*** r-daneel has quit IRC | 15:18 | |
*** Alex_Staf has quit IRC | 15:19 | |
*** moshele has quit IRC | 15:21 | |
*** alexchadin has quit IRC | 15:21 | |
*** sahid has quit IRC | 15:22 | |
*** brault has quit IRC | 15:22 | |
*** andreas_s has quit IRC | 15:23 | |
*** andreas_s has joined #openstack-nova | 15:23 | |
*** mlavalle has joined #openstack-nova | 15:23 | |
*** Guest52880 has quit IRC | 15:24 | |
*** pchavva has joined #openstack-nova | 15:24 | |
*** janki has quit IRC | 15:24 | |
*** sree has joined #openstack-nova | 15:24 | |
*** vivsoni has quit IRC | 15:25 | |
*** sean-k-m1 has joined #openstack-nova | 15:26 | |
*** sean-k-m1 has quit IRC | 15:26 | |
*** rodolof has quit IRC | 15:26 | |
*** yamahata has quit IRC | 15:26 | |
*** rodolof has joined #openstack-nova | 15:27 | |
*** andreas_s has quit IRC | 15:27 | |
*** sree has quit IRC | 15:28 | |
*** moshele has joined #openstack-nova | 15:28 | |
*** sree has joined #openstack-nova | 15:28 | |
*** rodolof has quit IRC | 15:29 | |
*** rodolof has joined #openstack-nova | 15:30 | |
*** tidwellr has joined #openstack-nova | 15:32 | |
*** moshele has quit IRC | 15:32 | |
*** andreas_s has joined #openstack-nova | 15:33 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce framework for RFB authentication https://review.openstack.org/345397 | 15:34 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce the VeNCrypt RFB authentication scheme https://review.openstack.org/345398 | 15:34 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: console: Provide an RFB security proxy implementation https://review.openstack.org/345399 | 15:34 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Document TLS security setup for noVNC proxy https://review.openstack.org/500544 | 15:35 |
openstackgerrit | Jay Pipes proposed openstack/nova master: handle traits with sharing providers https://review.openstack.org/517119 | 15:35 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Test alloc_cands with non overlapping sharing RPs https://review.openstack.org/519380 | 15:35 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Test alloc_cands with one RP shared between two RPs https://review.openstack.org/519617 | 15:35 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Test allocation candidates: multiple aggregates https://review.openstack.org/518633 | 15:35 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Test helper: validate provider summaries https://review.openstack.org/518982 | 15:35 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Fix accumulated nits in refactor series https://review.openstack.org/521189 | 15:35 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Add aggregates check in allocation candidates https://review.openstack.org/522407 | 15:35 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Fix missing rps in allocation candidates https://review.openstack.org/522409 | 15:35 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: func tests for multiple shared RPs https://review.openstack.org/498737 | 15:35 |
*** sree has quit IRC | 15:37 | |
*** bnemec has joined #openstack-nova | 15:40 | |
*** alexchadin has joined #openstack-nova | 15:40 | |
*** cdent has joined #openstack-nova | 15:41 | |
*** alexchadin has quit IRC | 15:44 | |
*** gcb has quit IRC | 15:44 | |
*** baoli has quit IRC | 15:45 | |
*** baoli has joined #openstack-nova | 15:45 | |
*** josecastroleon1 has quit IRC | 15:45 | |
*** mvk has quit IRC | 15:46 | |
*** yangyapeng has quit IRC | 15:49 | |
*** baoli has quit IRC | 15:50 | |
*** josecastroleon has joined #openstack-nova | 15:51 | |
*** andreas_s has quit IRC | 15:51 | |
*** ralonsoh has joined #openstack-nova | 15:52 | |
*** yamamoto has joined #openstack-nova | 15:52 | |
gibi | mriedem: went through https://review.openstack.org/#/c/507638/ and +W-d it | 15:52 |
mriedem | gibi: thanks | 15:52 |
*** huanxie has quit IRC | 15:53 | |
*** yamamoto has quit IRC | 15:54 | |
*** ralonsoh has quit IRC | 15:54 | |
*** ralonsoh_ has joined #openstack-nova | 15:54 | |
*** yamamoto has joined #openstack-nova | 15:54 | |
*** yamamoto has quit IRC | 15:54 | |
*** huanxie has joined #openstack-nova | 15:54 | |
*** alexchadin has joined #openstack-nova | 15:56 | |
*** armax has joined #openstack-nova | 15:56 | |
*** r-daneel has joined #openstack-nova | 15:56 | |
*** sridharg has quit IRC | 15:57 | |
*** baoli has joined #openstack-nova | 15:57 | |
*** _ix has joined #openstack-nova | 15:59 | |
*** alexchadin has quit IRC | 15:59 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project https://review.openstack.org/514309 | 16:01 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: zuul: Resolve review comments from I41b03a347 https://review.openstack.org/525255 | 16:01 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: zuul: Remove WSGI functional tests https://review.openstack.org/527140 | 16:01 |
*** gouthamr has quit IRC | 16:01 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Update Instance action's updated_at when action event updated. https://review.openstack.org/507473 | 16:02 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add instance action db and obj pagination support. https://review.openstack.org/526422 | 16:02 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326 | 16:02 |
openstackgerrit | Merged openstack/python-novaclient stable/pike: Move zuulv3 jobs to project repo https://review.openstack.org/522207 | 16:02 |
openstackgerrit | Merged openstack/python-novaclient stable/ocata: Move zuulv3 jobs to project repo https://review.openstack.org/522208 | 16:02 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project https://review.openstack.org/514309 | 16:03 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: zuul: Remove WSGI functional tests https://review.openstack.org/527140 | 16:03 |
*** sileht has left #openstack-nova | 16:05 | |
*** andreas_s has joined #openstack-nova | 16:06 | |
mriedem | first 3 patches in this series are ready to go i think https://review.openstack.org/#/c/524047/ | 16:07 |
*** diga has quit IRC | 16:07 | |
*** josecastroleon has quit IRC | 16:09 | |
*** vipul has quit IRC | 16:10 | |
*** vipul has joined #openstack-nova | 16:10 | |
*** dave-mcc_ is now known as dave-mccowan | 16:11 | |
*** andreas_s has quit IRC | 16:12 | |
*** Apoorva has joined #openstack-nova | 16:12 | |
*** andreas_s has joined #openstack-nova | 16:13 | |
*** andreas__ has joined #openstack-nova | 16:14 | |
*** baoli has quit IRC | 16:15 | |
*** andrea___ has joined #openstack-nova | 16:15 | |
*** bkopilov has quit IRC | 16:17 | |
*** andreas_s has quit IRC | 16:18 | |
*** chyka has joined #openstack-nova | 16:18 | |
*** andreas__ has quit IRC | 16:19 | |
*** andrea___ has quit IRC | 16:20 | |
openstackgerrit | Ed Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize https://review.openstack.org/526436 | 16:20 |
*** andreas_s has joined #openstack-nova | 16:20 | |
*** chyka has quit IRC | 16:22 | |
*** hongbin has joined #openstack-nova | 16:22 | |
*** chyka has joined #openstack-nova | 16:22 | |
mriedem | edleafe: i think my only hangup on https://review.openstack.org/#/c/516707/22 is the limits object<>dict conversion | 16:25 |
mriedem | i think we can hide all of that in the compute rpcapi client code | 16:25 |
*** andreas_s has quit IRC | 16:25 | |
mriedem | for build_and_run_instance, prep_resize, unshelve, etc | 16:25 |
mriedem | then conductor and compute managers don't need to check that | 16:25 |
*** huanxie has quit IRC | 16:26 | |
mriedem | if/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 dict | 16:26 |
mriedem | but that's not necessary for this series | 16:26 |
mriedem | dansmith: ^ agree? or am i missing something? | 16:27 |
*** baoli has joined #openstack-nova | 16:27 | |
*** ralonsoh_ is now known as ralonsoh | 16:27 | |
*** sree has joined #openstack-nova | 16:27 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Deprecate file injection https://review.openstack.org/522027 | 16:27 |
*** huanxie has joined #openstack-nova | 16:28 | |
*** baoli has quit IRC | 16:29 | |
*** sshwarts has quit IRC | 16:29 | |
mriedem | sdague: you want to take a pass through https://review.openstack.org/#/c/522027/ ? | 16:29 |
*** baoli has joined #openstack-nova | 16:29 | |
*** hemna_ has joined #openstack-nova | 16:31 | |
*** alexchadin has joined #openstack-nova | 16:32 | |
*** tidwellr has quit IRC | 16:32 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project https://review.openstack.org/514309 | 16:32 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: zuul: Remove WSGI functional tests https://review.openstack.org/527140 | 16:32 |
dansmith | mriedem: 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-nova | 16:32 | |
*** sree has quit IRC | 16:32 | |
dansmith | mriedem: that is fine as long as there isn | 16:32 |
*** tssurya has quit IRC | 16:32 | |
dansmith | isn'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 data | 16:32 |
mriedem | dansmith: the latter yes, like he was doing in an older patch set https://review.openstack.org/#/c/516707/17/nova/compute/rpcapi.py | 16:33 |
mriedem | at 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 object | 16:34 |
dansmith | you mean we always do the dict conversion at the moment, until we have a compute rpc bump to handle it, yes? | 16:35 |
mriedem | yes | 16:35 |
*** alexchadin has quit IRC | 16:36 | |
dansmith | yep makes sense to me | 16:37 |
edleafe | mriedem: ok, so just convert in the rpcapi layer, and assume dicts throughout compute | 16:38 |
mriedem | yup | 16:38 |
edleafe | ok, gimme a few... | 16:38 |
mriedem | edleafe: i.e. we shouldn't have any changes to compute in this patch | 16:38 |
mriedem | nova/compute/manager.py i mean | 16:38 |
edleafe | mriedem: gotcha. Most of those changes were defensive anyway. You know, "just in case" | 16:39 |
mriedem | sure, 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 IRC | 16:40 | |
edleafe | mriedem: it probably was happening in the unit tests | 16:40 |
*** jpena is now known as jpena|brb | 16:41 | |
edleafe | I probably created SchedulerLimits objects everywhere. I'll make sure those are cleaned up, too | 16:41 |
*** Apoorva has quit IRC | 16:42 | |
*** AlexeyAbashkin has quit IRC | 16:43 | |
*** sree has joined #openstack-nova | 16:43 | |
*** gbarros has joined #openstack-nova | 16:44 | |
*** burt has quit IRC | 16:45 | |
cdent | edleafe: 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 |
mriedem | i think we need a nova-status check first at least | 16:49 |
mriedem | and probably backport that nova-status check to pike | 16:49 |
*** damien_r has quit IRC | 16:50 | |
cdent | gibi: you still thinking about https://review.openstack.org/#/c/501252/ | 16:50 |
*** gyee has joined #openstack-nova | 16:51 | |
*** lpetrut has quit IRC | 16:51 | |
dtantsur | mriedem, dansmith, sorry, could you please check https://review.openstack.org/#/c/508555/ ? we're waiting for it to unblock our CI | 16:51 |
dtantsur | others too ^^' | 16:51 |
edleafe | cdent: I'm still not clear why it should stay | 16:52 |
*** sahid has joined #openstack-nova | 16:52 | |
edleafe | cdent: By Queens, the flavors must all be migrated | 16:52 |
*** moshele has joined #openstack-nova | 16:52 | |
*** sree has quit IRC | 16:52 | |
cdent | edleafe: I have no skin, I was fine with it moving, so it is mriedem and dtantsur you could be talking to | 16:52 |
*** penick has joined #openstack-nova | 16:53 | |
edleafe | ok | 16:53 |
mriedem | i'm personally a bit uncomfortable at the moment with operators adoption / understanding of the new thing yet | 16:53 |
mriedem | especially if we don't have traits via flavors yet | 16:53 |
mriedem | to replace the capabilities filter | 16:53 |
gibi | cdent: soon we will have n-rp that also makes it possible to use CUSTOM resource so I think I will abandon those patches soon | 16:54 |
*** Jack_Iv_ has quit IRC | 16:54 | |
*** yamamoto has joined #openstack-nova | 16:54 | |
mriedem | i'm thinking of https://specs.openstack.org/openstack/nova-specs/specs/queens/approved/ironic-driver-traits.html | 16:54 |
cdent | gibi: cool, just checking, i’m passing through the list I created on friday | 16:55 |
*** Jack_Iv has joined #openstack-nova | 16:55 | |
gibi | cdent: thanks for reminding me about this patch :) | 16:55 |
edleafe | mriedem: I get that that spec isn't completed. I'm just not clear how that impacts resources in extra_specs | 16:56 |
*** gbarros has quit IRC | 16:57 | |
jaypipes | dtantsur: done | 16:58 |
jroll | edleafe: seems like the suggestion is to give operators more time to move to resource classes | 16:58 |
mriedem | holy crap | 16:59 |
jroll | I'm not sure the traits thing is relevant as people can still run the compute capabilities filter | 16:59 |
* jroll hides | 16:59 | |
jroll | :P | 16:59 |
*** yangyapeng has joined #openstack-nova | 16:59 | |
mriedem | maybe traits isn't relevant, i just don't have a good feeling about how much operators understand what's being asked of them to upgrade | 16:59 |
*** moshele has quit IRC | 17:00 | |
mriedem | and 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 IRC | 17:00 | |
edleafe | jroll: hey stranger! | 17:00 |
jroll | hi :) | 17:00 |
* cdent waves at jroll | 17:01 | |
edleafe | ok, if the idea is to not make resource classes required in queens, that's fine | 17:01 |
jroll | mriedem: I don't have enough recent context to agree, but totally see where you and dmitry are coming from | 17:01 |
edleafe | I hadn't heard about that change | 17:01 |
mriedem | edleafe: that's not what i'm saying | 17:01 |
mriedem | but before we make something required, i think we need a bit of work to help people know when their migration is done | 17:02 |
mriedem | like we do, for example, with schema blocker migrations at times | 17:02 |
edleafe | mriedem: that's fair. IMO, we should add that, but keeping the pike migration code around doesn't feel like it would help that | 17:03 |
mriedem | personally my rear end gets sore hearing about how hard upgrades are and how complicated nova makes everything | 17:03 |
*** yamamoto has quit IRC | 17:03 | |
mriedem | edleafe: keeping it around helps motivate writing the nova-status change :) | 17:03 |
*** yangyapeng has quit IRC | 17:03 | |
mriedem | removing code is the carrot | 17:04 |
*** salv-orlando has quit IRC | 17:04 | |
mriedem | dtantsur: jaypipes: if https://review.openstack.org/#/c/508555/ regresses something, does it only impact the ironic driver? | 17:05 |
mriedem | doesn't look like it https://review.openstack.org/#/c/508555/6/nova/compute/resource_tracker.py | 17:05 |
* dtantsur is in a meeting, sorry | 17:05 | |
*** jmlowe has quit IRC | 17:05 | |
*** sahid has quit IRC | 17:05 | |
dtantsur | it's hard to tell for me, unfortunately | 17:05 |
dansmith | mriedem: I'm about to comment on the error message in that way | 17:05 |
*** edand has joined #openstack-nova | 17:06 | |
*** Jack_Iv_ has joined #openstack-nova | 17:06 | |
jroll | seems like it's unlikely to affect non-ironic, but it could, in the drivers where it makes sense for host != nodename | 17:06 |
jroll | that's normal for vmware or something right? | 17:07 |
*** alexchadin has joined #openstack-nova | 17:07 | |
jaypipes | jroll: I don't think so any more... | 17:07 |
mriedem | dansmith: yeah just did | 17:07 |
mriedem | about the rebalance thing? | 17:07 |
dansmith | mriedem: yeah | 17:07 |
*** josecastroleon has joined #openstack-nova | 17:07 | |
mriedem | because people eff up their compute nodes tables and service host records i'm sure | 17:08 |
jroll | jaypipes: ah, cool | 17:08 |
*** penick has quit IRC | 17:08 | |
*** jmlowe has joined #openstack-nova | 17:08 | |
* dtantsur will fix later on | 17:08 | |
dansmith | or they change a hypervisor's name, | 17:09 |
dansmith | or have an accidental name overlap | 17:09 |
*** Jack_Iv has quit IRC | 17:09 | |
dansmith | or they point the compute service at the wrong hypervisor | 17:09 |
mriedem | or it's tuesday? | 17:09 |
*** alexchadin has quit IRC | 17:09 | |
dtantsur | wut? :D | 17:09 |
*** fragatin_ has quit IRC | 17:10 | |
dansmith | since our UC is over (host,hypervisor_hostname) each hypervisor_hostname could be the same today technically | 17:10 |
mriedem | dtantsur: tuesday is random break your resource tracker day | 17:10 |
dtantsur | any suggestions for a flag name? allow_nodes_migration (hmm, "migrate" may be confusing) | 17:10 |
dtantsur | mriedem: good to know :) | 17:10 |
*** fragatina has joined #openstack-nova | 17:10 | |
*** fragatina has quit IRC | 17:10 | |
sean-k-mooney | dansmith: the UC is over (host,hypervisor_hostname) today for ironic correct? | 17:11 |
*** fragatina has joined #openstack-nova | 17:11 | |
*** gbarros has joined #openstack-nova | 17:12 | |
*** lucasagomes is now known as lucas-afk | 17:12 | |
*** penick has joined #openstack-nova | 17:12 | |
dansmith | sean-k-mooney: for everything | 17:13 |
sean-k-mooney | dansmith: 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 db | 17:14 |
dansmith | but we track nodes for every virt driver as well | 17:14 |
dansmith | they're just all 1:1 | 17:14 |
dtantsur | so, any ideas better than Driver.allow_nodes_migration? | 17:16 |
jroll | dtantsur: what about just if_driver_is_ironic() ? | 17:17 |
jroll | or however you might actually write that | 17:17 |
dtantsur | if driver.has_bears() :) | 17:17 |
dansmith | are you talking about checking a flag on the driver in RT? | 17:17 |
dtantsur | yep | 17:17 |
dansmith | I don't love that, but I'd vote for the generic-er flag name, with a comment above of "ironic strikes again" or something | 17:18 |
* dansmith jumps on a call | 17:18 | |
dtantsur | I was thinking about ^^^ as well | 17:19 |
*** edand has quit IRC | 17:20 | |
mriedem | driver.rebalances_nodes? | 17:20 |
mriedem | that's specific to the problem here | 17:21 |
dtantsur | sounds good | 17:21 |
mriedem | doesn't talk about migration | 17:21 |
mriedem | and is obviously false for everything else | 17:21 |
dansmith | let's hope no other drivers try to use this to implement HA | 17:21 |
dtantsur | people never take a random feature and try to misuse it, right? | 17:22 |
dansmith | never | 17:22 |
*** alexchadin has joined #openstack-nova | 17:23 | |
*** jpena|brb is now known as jpena | 17:24 | |
*** alexchadin has quit IRC | 17:25 | |
*** sree has joined #openstack-nova | 17:26 | |
*** salv-orlando has joined #openstack-nova | 17:27 | |
*** huanxie has quit IRC | 17:27 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Handle images with no data https://review.openstack.org/526329 | 17:28 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Use correct response type in tests https://review.openstack.org/527182 | 17:28 |
sean-k-mooney | dtantsur: 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 |
dtantsur | good plan, go for it | 17:29 |
openstackgerrit | Chris Dent proposed openstack/nova master: placement: skip authentication on root URI https://review.openstack.org/522002 | 17:29 |
*** huanxie has joined #openstack-nova | 17:30 | |
*** sree has quit IRC | 17:31 | |
stephenfin | mriedem: 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/526329 | 17:33 |
* stephenfin is going to fix the glanceclient API docs now | 17:33 | |
mriedem | cdent: https://review.openstack.org/#/c/521640/ | 17:33 |
cdent | ack | 17:34 |
*** andreas_s has joined #openstack-nova | 17:36 | |
sean-k-mooney | stephenfin: 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-nova | 17:39 | |
*** alexchadin has quit IRC | 17:42 | |
*** xyang1 has joined #openstack-nova | 17:43 | |
openstackgerrit | Merged openstack/nova stable/pike: placement: avoid returning duplicated alloc_reqs when no sharing rp https://review.openstack.org/508885 | 17:43 |
*** damien_r has joined #openstack-nova | 17:44 | |
*** Apoorva has joined #openstack-nova | 17:45 | |
*** Apoorva has quit IRC | 17:45 | |
*** Apoorva has joined #openstack-nova | 17:46 | |
*** sree has joined #openstack-nova | 17:47 | |
*** yangyapeng has joined #openstack-nova | 17:49 | |
*** derekh has quit IRC | 17:50 | |
*** penick has quit IRC | 17:50 | |
*** sree has quit IRC | 17:51 | |
*** ijw has quit IRC | 17:51 | |
*** yangyapeng has quit IRC | 17:53 | |
cdent | mriedem: 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 contrary | 17:55 |
*** brault has joined #openstack-nova | 17:59 | |
*** andreas_s has quit IRC | 18:00 | |
*** andreas_s has joined #openstack-nova | 18:01 | |
*** huanxie has quit IRC | 18:01 | |
*** baoli has quit IRC | 18:01 | |
*** ralonsoh has quit IRC | 18:02 | |
*** huanxie has joined #openstack-nova | 18:02 | |
*** andreas_s has quit IRC | 18:03 | |
*** andreas_s has joined #openstack-nova | 18:03 | |
*** brault has quit IRC | 18:03 | |
*** baoli_ has joined #openstack-nova | 18:04 | |
*** bkopilov has joined #openstack-nova | 18:04 | |
*** andreas_s has quit IRC | 18:05 | |
*** andreas_s has joined #openstack-nova | 18:05 | |
*** sree has joined #openstack-nova | 18:07 | |
*** sambetts is now known as sambetts|afk | 18:08 | |
*** andreas_s has quit IRC | 18:10 | |
*** sree has quit IRC | 18:11 | |
*** Alex_Staf has joined #openstack-nova | 18:12 | |
*** andreas_s has joined #openstack-nova | 18:16 | |
*** ijw has joined #openstack-nova | 18:17 | |
*** jpena is now known as jpena|brb | 18:20 | |
*** tbachman has joined #openstack-nova | 18:20 | |
*** gbarros has quit IRC | 18:20 | |
*** burt has joined #openstack-nova | 18:21 | |
*** ijw has quit IRC | 18:21 | |
*** andreas__ has joined #openstack-nova | 18:22 | |
*** ijw has joined #openstack-nova | 18:22 | |
*** gbarros has joined #openstack-nova | 18:24 | |
*** andreas_s has quit IRC | 18:25 | |
*** salv-orlando has quit IRC | 18:26 | |
*** salv-orlando has joined #openstack-nova | 18:27 | |
*** ijw has quit IRC | 18:27 | |
*** sree has joined #openstack-nova | 18:27 | |
*** AlexeyAbashkin has joined #openstack-nova | 18:28 | |
mriedem | cdent: i think we should probably do that for at least the requests that can generate a lot of results in a big cloud | 18:28 |
*** Jack_Iv_ has quit IRC | 18:28 | |
mriedem | so GET /traits and GET /resource_providers | 18:28 |
mriedem | GET /traits will always be at least reasonable size because of the library | 18:29 |
cdent | mriedem: i’ve done it for allocations, traits, resource classes, resource providers, but not inventories | 18:29 |
dtantsur | mriedem: re that patch: should I keep the functional tests changes, given that the behavior is no longer the default? | 18:29 |
mriedem | cdent: allocations are per provider or consumer right? so i didn't think that one was as big a deal | 18:30 |
mriedem | but if you're going to establish a pattern... | 18:30 |
*** ijw has joined #openstack-nova | 18:30 | |
mriedem | i just tend to think of allocations like inventories - they are mapped to a single provider | 18:30 |
cdent | mriedem: I was imagining something other than libvirt, where maybe the compute-mode is ginormous | 18:31 |
mriedem | dtantsur: i probably wouldn't | 18:31 |
cdent | or a shared disk | 18:31 |
dtantsur | k | 18:31 |
mriedem | cdent: or vmware :) | 18:31 |
*** salv-orlando has quit IRC | 18:31 | |
mriedem | *vcenter | 18:31 |
* cdent knows nothing of this vcenter people speak of | 18:31 | |
cdent | i was mostly thinking of my.petapbytediskfarm.rp-5 | 18:32 |
*** ijw has quit IRC | 18:32 | |
*** sree has quit IRC | 18:32 | |
mriedem | but then the inventory is per resource class per provider isn't it? | 18:32 |
*** AlexeyAbashkin has quit IRC | 18:32 | |
mriedem | so you'd only have a lot of inventory entries for a provider if you have a lot of resource classes on that provider | 18:32 |
_ix | Hello friends. I'm having some trouble getting a compute node to show up in the hosts list. | 18:34 |
cdent | mriedem: right, which isn’t predicted, so that one is left off | 18:34 |
cdent | but when it turns into a problem, is fixable | 18:35 |
mriedem | stephenfin: question in https://review.openstack.org/#/c/526329/ | 18:35 |
mriedem | _ix: you have to run the discover_hosts CLI | 18:35 |
mriedem | nova-manage cell_v2 discover_hosts --verbose | 18:35 |
_ix | I have. It says there aren't any unmapped hosts. | 18:35 |
mriedem | what does nova service-list show? | 18:36 |
mriedem | does the entry show up there? | 18:36 |
_ix | For nova compute? Yes. | 18:36 |
_ix | This is just 1/5 of the hosts not showing up. | 18:37 |
_ix | But, I might be misleading a bit. I see there's an rpc problem on the host. | 18:37 |
mriedem | what does nova hypervisor-list show? | 18:37 |
mriedem | nova host-list is deprecated since pike btw | 18:37 |
smatzek | The 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 |
openstackgerrit | Dmitry Tantsur proposed openstack/nova master: Re-use existing ComputeNode on ironic rebalance https://review.openstack.org/508555 | 18:38 |
dtantsur | mriedem: I hope I did not mess up anything ^^^ | 18:38 |
mriedem | smatzek: yes for resize with volume attached - pretty sure anyway | 18:38 |
mriedem | smatzek: not sure about rebuild with a volume attached (in tempest anyway) | 18:38 |
*** Jack_Iv has joined #openstack-nova | 18:38 | |
_ix | mriedem: I generally use the openstack client. Is `openstack host list` also deprecated? | 18:38 |
mriedem | smatzek: test_resize_volume_backed_server_confirm | 18:39 |
mriedem | in tempest | 18:39 |
smatzek | mriedem: 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_755007 | 18:40 |
mriedem | _ix: i deprecated nova host-list and didn't update openstack client so it's probably not deprecated | 18:40 |
*** jpena|brb is now known as jpena | 18:41 | |
_ix | Cool. 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 |
smatzek | rebuild, 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 |
mriedem | nova-compute will auto-disable the host if it's had 10 failed build attempts | 18:41 |
mriedem | in a row | 18:41 |
_ix | Oh, hey. That's a fine tip. Thanks again! I'm sure that's what's going on here. | 18:42 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: CLI for allocations https://review.openstack.org/457534 | 18:42 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: CLI for usages https://review.openstack.org/457535 | 18:42 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: CLI for aggregates (v1.1) https://review.openstack.org/505643 | 18:42 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: CLI for resource classes (v1.2) https://review.openstack.org/511182 | 18:42 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: RP list: member_of and resources parameters (v1.3, v1.4) https://review.openstack.org/511183 | 18:42 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: RP delete inventories (v1.5) https://review.openstack.org/514642 | 18:43 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: CLI for traits (v1.6) https://review.openstack.org/514643 | 18:43 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644 | 18:43 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646 | 18:43 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647 | 18:43 |
_ix | Also, 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 that | 18:43 |
mriedem | but yeah, ^ is why it's on by default | 18:43 |
_ix | I don't. I think that's great. | 18:43 |
*** Jack_Iv has quit IRC | 18:43 | |
mriedem | dansmith: ^ high praise | 18:43 |
smatzek | mriedem: 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 |
mriedem | smatzek: 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_499795 | 18: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 |
_ix | mriedem: 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: nope | 18:45 |
mriedem | smatzek: that empty connection_info is presumably the problem | 18:45 |
mriedem | because nova pulls the driver_volume_type out of the connection_info on the attachment | 18:45 |
mriedem | smatzek: although, | 18:46 |
*** baoli has joined #openstack-nova | 18:46 | |
mriedem | i think that's normal for a 'reserved' attachment maybe? we'd have to ask jgriffith | 18:46 |
mriedem | until nova updates the attachment with the host connector, the attachment doesn't have a connection_info | 18:46 |
*** sree has joined #openstack-nova | 18:46 | |
_ix | Will the openstack client one day be the only client? | 18:46 |
smatzek | yep, 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 |
ildikov | mriedem: if it's only reserved connection_info is empty | 18:47 |
_ix | (That is, the openstack cli client) | 18:47 |
mriedem | _ix: at this rate i doubt it | 18:47 |
_ix | That's a sad story. | 18:47 |
mriedem | _ix: been talking about that for years now | 18:47 |
smatzek | To 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 IRC | 18:47 | |
mriedem | _ix: would need to get feature parity in osc | 18:48 |
mriedem | _ix: and no one is working on that | 18:48 |
dansmith | mriedem: nice | 18:48 |
_ix | Forgive me, what is osc? | 18:48 |
mriedem | _ix: we could make that a priority, but that means de-prioritizing other work | 18:48 |
mriedem | openstack client | 18:48 |
mriedem | OpenStackClient | 18:48 |
_ix | Oh, I see. | 18:48 |
*** yangyapeng has joined #openstack-nova | 18:49 | |
ildikov | smatzek: I need to jump on a call, but will look into the logs once I got off the phone | 18:49 |
ildikov | smatzek: 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 IRC | 18:50 | |
mriedem | hmm | 18: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 IRC | 18:51 | |
*** damien_r has quit IRC | 18:53 | |
mriedem | smatzek: can you open a bug, the logstash query does show this starting on the 9th | 18:53 |
mriedem | http://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=7d | 18:53 |
mriedem | *8th actually | 18:54 |
*** yangyapeng has quit IRC | 18:54 | |
smatzek | mriedem: 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 |
ildikov | I don't recall seeing this issue earlier | 18:57 |
*** MasterOfBugs has joined #openstack-nova | 18:57 | |
*** pramodrj07 has joined #openstack-nova | 18:57 | |
*** gbarros has quit IRC | 18:58 | |
ildikov | we should have connection_info by the time we get to the _connect_volume call, I need to look into what went wrong there | 18:58 |
*** Apoorva has quit IRC | 18:58 | |
*** jmlowe has joined #openstack-nova | 18:58 | |
*** moshele has joined #openstack-nova | 19:00 | |
*** jpena is now known as jpena|away | 19:01 | |
smatzek | mriedem: https://bugs.launchpad.net/nova/+bug/1737599 | 19:02 |
openstack | Launchpad bug 1737599 in OpenStack Compute (nova) "Instance resize with attach volume fails" [Undecided,New] | 19:02 |
*** AlexeyAbashkin has joined #openstack-nova | 19:03 | |
mriedem | i can see what's different about this flow from the bfv one | 19:03 |
*** huanxie has quit IRC | 19:04 | |
mriedem | we seem to lose the connection_info during some of the much complicated bdm object -> driver bdm -> dict block_device_info conversions... | 19:04 |
mriedem | or, it was just never set in the first place | 19:04 |
mriedem | my guess is the latter | 19:04 |
mriedem | i can definitely see where it blows up though yeah | 19:05 |
*** Jack_Iv has joined #openstack-nova | 19:06 | |
*** andreas__ has quit IRC | 19:06 | |
*** andreas_s has joined #openstack-nova | 19:06 | |
*** huanxie has joined #openstack-nova | 19:06 | |
*** AlexeyAbashkin has quit IRC | 19:07 | |
*** MasterOfBugs has quit IRC | 19:07 | |
*** pramodrj07 has quit IRC | 19:07 | |
*** josecastroleon has quit IRC | 19:08 | |
*** ijw has joined #openstack-nova | 19:10 | |
openstackgerrit | Ed Leafe proposed openstack/nova master: Change RPC for select_destinations() https://review.openstack.org/516707 | 19:10 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils https://review.openstack.org/511357 | 19:10 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Make conductor pass and use host_lists https://review.openstack.org/511358 | 19:10 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize https://review.openstack.org/526436 | 19:10 |
*** andreas_s has quit IRC | 19:11 | |
edleafe | mriedem: cleaned up ^^ | 19:11 |
*** penick has joined #openstack-nova | 19:12 | |
*** ijw has quit IRC | 19:17 | |
*** itlinux has joined #openstack-nova | 19:17 | |
*** jroll has quit IRC | 19:18 | |
openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Add cache headers to placement api requests https://review.openstack.org/521640 | 19:18 |
openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Add info about last-modified to contrib docs https://review.openstack.org/526084 | 19:18 |
*** jroll has joined #openstack-nova | 19:19 | |
cdent | mriedem, edleafe ^ that fixes the several suggestions matt had | 19:20 |
mriedem | smatzek: ildikov: i think i have the resize thing figured out | 19:20 |
*** itlinux has quit IRC | 19:22 | |
*** chyka has quit IRC | 19:23 | |
smatzek | mriedem: great, thanks | 19:23 |
mriedem | i'll post a patch and you can push a trove depends-on to see if it works | 19:24 |
*** dtantsur is now known as dtantsur|afk | 19:24 | |
*** andreas_s has joined #openstack-nova | 19:26 | |
*** sree has joined #openstack-nova | 19:26 | |
*** moshele has quit IRC | 19:27 | |
*** itlinux has joined #openstack-nova | 19:27 | |
*** salv-orlando has joined #openstack-nova | 19:27 | |
*** fragatina has quit IRC | 19:28 | |
*** Alex_Staf has quit IRC | 19:28 | |
smatzek | would the thing you found possibly cause the mountpoint to be lost in Cinder's attachments during rebuild? | 19:28 |
smatzek | mriedem: ^ | 19:29 |
*** itlinux has quit IRC | 19:29 | |
ildikov | mriedem: cool, that was quick! | 19:29 |
mriedem | smatzek: don't know, one problem at a time | 19:30 |
mriedem | smatzek: but maybe | 19:31 |
*** sree has quit IRC | 19:31 | |
openstackgerrit | Murali Annamneni proposed openstack/nova master: Enables MySQL Cluster Support for Nova https://review.openstack.org/446643 | 19:31 |
mriedem | smatzek: there is a good chance it is though | 19:31 |
*** salv-orlando has quit IRC | 19:32 | |
*** pramodrj07 has joined #openstack-nova | 19:35 | |
*** MasterOfBugs has joined #openstack-nova | 19:35 | |
*** huanxie has quit IRC | 19:37 | |
*** huanxie has joined #openstack-nova | 19:39 | |
*** Apoorva has joined #openstack-nova | 19:41 | |
ildikov | smatzek: that log indicates that there might be a step missing from rebuild | 19:41 |
ildikov | smatzek: will check what mriedem has as a fix and then we can compare whether we covered everything or not | 19:42 |
ericyoung | Any idea when https://review.openstack.org/#/c/407440/ will get some eyes on it? | 19:42 |
ericyoung | It is the patch for ScaleIO ephemeral/image volumes | 19:43 |
*** Apoorva has quit IRC | 19:45 | |
*** MasterOfBugs has quit IRC | 19:46 | |
*** pramodrj07 has quit IRC | 19:46 | |
*** MasterOfBugs has joined #openstack-nova | 19:46 | |
*** pramodrj07 has joined #openstack-nova | 19:46 | |
*** sree has joined #openstack-nova | 19:47 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Save driver_volume_type when refreshing attachment connection_info https://review.openstack.org/527228 | 19:48 |
mriedem | smatzek: ildikov: ^ see if that fixes the resize issue in trove; needs unit tests | 19:48 |
*** andreas_s has quit IRC | 19:49 | |
*** ijw has joined #openstack-nova | 19:50 | |
*** andreas_s has joined #openstack-nova | 19:50 | |
mriedem | actually, i think that will fix the VolumeDriverNotFound but we'll still fail to connect the volume on the destination host | 19:51 |
*** andreas_s has quit IRC | 19:51 | |
mriedem | because the new attachment connection_info doesn't have 'data' set | 19:51 |
*** sree has quit IRC | 19:51 | |
*** andreas_s has joined #openstack-nova | 19:51 | |
smatzek | so should I hold off on pushing a depends-on test to Trove? | 19:52 |
mriedem | just start that depends-on to make sure it blows up where i think it is now | 19:52 |
mriedem | and we'll need it anyway to verify the eventual fix | 19:52 |
smatzek | mriedem: 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 |
smatzek | https://review.openstack.org/#/c/527230 | 19:54 |
*** Apoorva has joined #openstack-nova | 19:54 | |
mriedem | yes that should work | 19:54 |
*** ijw has quit IRC | 19:54 | |
*** moshele has joined #openstack-nova | 19:54 | |
*** Jack_Iv has quit IRC | 19:59 | |
mriedem | ok this is a bit shittier than i thought, but i think i know of another relatively painless fix | 20:06 |
mriedem | to avoid doing crazy crap in the virt driver(s) or nova/virt/block_device.py code | 20:06 |
ildikov | mriedem: hmm, we are explicitly transforming 'driver_volume_type' so it is supposed to come from Cinder already | 20:07 |
mriedem | ildikov: it's pulled out of the connection_info from the attachment, | 20:07 |
mriedem | which is empty for the new attachment which is not yet connected on the dest host | 20:07 |
mriedem | i.e. we don't call attachment_update in the cold migrate flow | 20:07 |
mriedem | for cold migration, the drivers just do all the work on the hosts with connecting the volume | 20:08 |
mriedem | and the old init/terminate_connection calls | 20:08 |
mriedem | on the volume | 20:08 |
ildikov | hmm, but shouldn't we call attachment_update with the new flow now? | 20:08 |
mriedem | yes - but we aren't | 20:08 |
mriedem | so that's what i think i need to add | 20:08 |
ildikov | sorry, I just got confused with the changes in your patch in mind | 20:09 |
ildikov | as the log looks like we have a missing attahcment_update call somewhere | 20:09 |
ildikov | and then we need to check whether attachment_complete happens there or not | 20:09 |
mriedem | yes my patch does'nt fix it | 20:10 |
mriedem | i think i know what needs to happen | 20:10 |
mriedem | but making coffee first | 20:10 |
ildikov | fair enough | 20:13 |
*** moshele has quit IRC | 20:15 | |
*** nsingh has joined #openstack-nova | 20:16 | |
*** bhagyashri_s has joined #openstack-nova | 20:16 | |
*** pooja-jadhav has joined #openstack-nova | 20:16 | |
*** andreas_s has quit IRC | 20:18 | |
*** pooja_jadhav has quit IRC | 20:18 | |
*** andreas_s has joined #openstack-nova | 20:18 | |
*** niraj_singh has quit IRC | 20:18 | |
*** bhagyashris has quit IRC | 20:18 | |
*** Swami has joined #openstack-nova | 20:19 | |
*** fragatina has joined #openstack-nova | 20:22 | |
*** fragatina has quit IRC | 20:23 | |
*** andreas_s has quit IRC | 20:23 | |
*** fragatina has joined #openstack-nova | 20:23 | |
*** fragatina has quit IRC | 20:23 | |
*** fragatina has joined #openstack-nova | 20:24 | |
*** marst has joined #openstack-nova | 20:25 | |
*** salv-orlando has joined #openstack-nova | 20:28 | |
openstackgerrit | Ed Leafe proposed openstack/nova master: Make conductor pass and use host_lists https://review.openstack.org/511358 | 20:31 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize https://review.openstack.org/526436 | 20:31 |
*** salv-orlando has quit IRC | 20:33 | |
*** penick has quit IRC | 20:35 | |
*** shaner has quit IRC | 20:38 | |
_ix | More 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 |
_ix | Thank you! | 20:39 |
*** shaner has joined #openstack-nova | 20:39 | |
*** huanxie has quit IRC | 20:39 | |
*** Guest52604 has joined #openstack-nova | 20:40 | |
mriedem | thanks for the feedback | 20:41 |
mriedem | always nice to know when people actually "like" something | 20:41 |
*** moshele has joined #openstack-nova | 20:41 | |
*** liverpooler has quit IRC | 20:42 | |
cdent | 👍 | 20:42 |
*** huanxie has joined #openstack-nova | 20:42 | |
mriedem | dtantsur|afk: do you have a run on https://review.openstack.org/#/c/508555/ yet? | 20:43 |
mriedem | TheJulia: do you know where ^ is being tested to see if it unblocks the ironic multinode job? | 20:44 |
mriedem | dtantsur|afk: TheJulia: nevermind, found it https://review.openstack.org/#/c/527075/ | 20:44 |
TheJulia | :) | 20:45 |
mriedem | is ironic-tempest-dsvm-ipa-wholedisk-agent_ipmitool-tinyipa-multinode the job to care about? | 20:45 |
*** tidwellr_ has joined #openstack-nova | 20:49 | |
*** tidwellr has quit IRC | 20:49 | |
* TheJulia double checks | 20:51 | |
TheJulia | mriedem: yes | 20:51 |
*** nsingh has quit IRC | 20:54 | |
*** felipemonteiro_ has joined #openstack-nova | 20:54 | |
*** moshele has quit IRC | 20:55 | |
*** nsingh has joined #openstack-nova | 20:55 | |
mriedem | dansmith: 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 there | 20:56 |
*** felipemonteiro__ has quit IRC | 20:57 | |
*** tidwellr_ has quit IRC | 20:59 | |
dansmith | mriedem: yep, looks right | 21:00 |
*** chyka has joined #openstack-nova | 21:01 | |
*** tidwellr has joined #openstack-nova | 21:03 | |
*** cdent has quit IRC | 21:04 | |
*** chyka has quit IRC | 21:05 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Update and complete volume attachments during resize https://review.openstack.org/527228 | 21:06 |
mriedem | smatzek: ildikov: ^ should do it for resize | 21:06 |
*** sree has joined #openstack-nova | 21:07 | |
mriedem | smatzek: the rebuild fix will be different, so do you have a separate bug for that? | 21:08 |
mriedem | with links to the logs | 21:08 |
smatzek | I don't, but I'll open one. | 21:08 |
mriedem | dansmith: 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 paternity | 21:09 |
mriedem | maybe it's fine to wait another day assuming this has been broken since pike | 21:09 |
dansmith | yeah I think he is.. up to you, I'm around to +2 if you want, or I can do it | 21:10 |
mriedem | TheJulia: is this a latent bug or regression in pike? | 21:10 |
dansmith | if you update I think jaypipes and I can get it | 21:10 |
mriedem | dansmith: ok i'll update quick | 21:10 |
*** Apoorva has quit IRC | 21:11 | |
*** sree has quit IRC | 21:11 | |
TheJulia | mriedem: don't have enough context to say either way | 21:12 |
ildikov | mriedem: so resize is using a different flow and not calling bdm.attach(), right? | 21:13 |
mriedem | ildikov: correct | 21:13 |
*** huanxie has quit IRC | 21:13 | |
*** david-lyle has quit IRC | 21:13 | |
mriedem | doing 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 resize | 21:14 |
mriedem | since in the old flow, we never needed to call bdm.attach because the volume was already attached | 21:14 |
mriedem | the new flow is all about the attachment though, so we have to do things differently | 21:14 |
mriedem | we 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 host | 21:15 |
*** huanxie has joined #openstack-nova | 21:15 | |
smatzek | mriedem: 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 IRC | 21:19 | |
smatzek | Trove 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-L1371 | 21:20 |
smatzek | this 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 |
ildikov | mriedem: 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 |
mriedem | smatzek: ok i'll leave rebuild up to you | 21:22 |
smatzek | mriedem: 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 |
mriedem | ildikov: we definitely have a todo to add a test to tempest for resize with a volume attached (non-bfv) | 21:22 |
*** smatzek has quit IRC | 21:23 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Re-use existing ComputeNode on ironic rebalance https://review.openstack.org/508555 | 21:24 |
mriedem | dansmith: ^ i'll recheck the ironic canary patch | 21:24 |
openstackgerrit | Merged openstack/nova master: Address nits from service create/destroy notification review https://review.openstack.org/523162 | 21:24 |
openstackgerrit | Merged openstack/nova master: [placement] Fix API reference for microversion 1.14 https://review.openstack.org/526973 | 21:24 |
openstackgerrit | Merged openstack/nova stable/pike: Mention API behavior change when over quota limit https://review.openstack.org/525511 | 21:24 |
openstackgerrit | Merged openstack/nova stable/pike: Downgrade log for keystone verify client fail https://review.openstack.org/525475 | 21:24 |
mriedem | all he wanted was a pepsi https://www.youtube.com/watch?v=aYItTxqTc38 | 21:25 |
*** jmlowe has quit IRC | 21:27 | |
ildikov | just another regular Monday :) | 21:28 |
_ix | Quick question ... can I assume that that ports should be bound to the host that... hosts the instance? | 21:28 |
openstackgerrit | Merged openstack/nova stable/pike: Fix ValueError if invalid max_rows passed to db purge https://review.openstack.org/525628 | 21:28 |
mriedem | _ix: unless it's migrating | 21:28 |
*** tidwellr has joined #openstack-nova | 21:28 | |
*** jmlowe has joined #openstack-nova | 21:28 | |
*** chyka has joined #openstack-nova | 21:29 | |
_ix | mriedem: Where should I be looking for problems if this isn't necessarily the case? | 21:29 |
*** salv-orlando has joined #openstack-nova | 21:29 | |
_ix | That is, I just instantiated, and they seem to be bound incorrectly. | 21:29 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Make conductor pass and use host_lists https://review.openstack.org/511358 | 21:29 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize https://review.openstack.org/526436 | 21:29 |
dansmith | mriedem: cool | 21:30 |
mriedem | _ix: nova-compute logs | 21: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 properly | 21:30 |
_ix | I'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 |
_ix | No 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 host | 21:31 |
_ix | ! | 21:32 |
mriedem | if the instance built on the next host, it would be using the 2nd port but we orphaned the 1st port | 21:32 |
_ix | I just saw that error today. | 21:32 |
_ix | Cool. | 21:32 |
*** burt has quit IRC | 21:32 | |
*** salv-orlando has quit IRC | 21:34 | |
mriedem | i've removed zkvm from nova-ci for now | 21:39 |
*** threestrands has joined #openstack-nova | 21:41 | |
*** r-daneel has quit IRC | 21:43 | |
*** r-daneel has joined #openstack-nova | 21:43 | |
*** pchavva has quit IRC | 21:46 | |
*** huanxie has quit IRC | 21:46 | |
*** pooja_jadhav has joined #openstack-nova | 21:46 | |
*** bhagyashris_ has joined #openstack-nova | 21:47 | |
*** salv-orlando has joined #openstack-nova | 21:47 | |
*** sree has joined #openstack-nova | 21:47 | |
*** bfife_ has joined #openstack-nova | 21:48 | |
*** niraj_singh has joined #openstack-nova | 21:49 | |
*** huanxie has joined #openstack-nova | 21:49 | |
*** cfriesen__ has joined #openstack-nova | 21:50 | |
*** tidwellr has quit IRC | 21:50 | |
*** migi_ has joined #openstack-nova | 21:50 | |
*** tonyb_ has joined #openstack-nova | 21:51 | |
*** takashin has joined #openstack-nova | 21:52 | |
*** sree has quit IRC | 21:53 | |
*** gbarros has joined #openstack-nova | 21:54 | |
*** andreykurilin_ has joined #openstack-nova | 21:55 | |
*** markmc` has joined #openstack-nova | 21:55 | |
*** rmk` has joined #openstack-nova | 21:55 | |
*** stephenfin_ has joined #openstack-nova | 21:55 | |
*** EmilienM_ has joined #openstack-nova | 21:55 | |
*** aignatov2 has joined #openstack-nova | 21:55 | |
*** DinaBelova2 has joined #openstack-nova | 21:56 | |
*** nsingh has quit IRC | 21:56 | |
*** pooja-jadhav has quit IRC | 21:56 | |
*** bhagyashri_s has quit IRC | 21:56 | |
*** lennyb has quit IRC | 21:56 | |
*** ircuser-1 has quit IRC | 21:56 | |
*** trozet has quit IRC | 21:56 | |
*** cfriesen_ has quit IRC | 21:56 | |
*** aloga has quit IRC | 21:56 | |
*** EmilienM has quit IRC | 21:56 | |
*** migi has quit IRC | 21:56 | |
*** jdillaman has quit IRC | 21:56 | |
*** tonyb has quit IRC | 21:56 | |
*** rmk has quit IRC | 21:56 | |
*** markmc has quit IRC | 21:56 | |
*** bfife has quit IRC | 21:56 | |
*** mdbooth has quit IRC | 21:56 | |
*** ansiwen has quit IRC | 21:56 | |
*** brad[] has quit IRC | 21:56 | |
*** stephenfin has quit IRC | 21:56 | |
*** rybridges has quit IRC | 21:56 | |
*** andreykurilin has quit IRC | 21:56 | |
*** DinaBelova has quit IRC | 21:56 | |
*** aignatov has quit IRC | 21:56 | |
*** rmk` is now known as rmk | 21:56 | |
*** aignatov2 is now known as aignatov | 21:56 | |
*** rmk has quit IRC | 21:56 | |
*** rmk has joined #openstack-nova | 21:56 | |
*** DinaBelova2 is now known as DinaBelova | 21:56 | |
*** EmilienM_ is now known as EmilienM | 21:56 | |
*** EmilienM has quit IRC | 21:56 | |
*** EmilienM has joined #openstack-nova | 21:56 | |
mriedem | never 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.gz | 21:57 |
*** Apoorva has joined #openstack-nova | 21:57 | |
*** tlbr has quit IRC | 21:57 | |
*** cleong has quit IRC | 21:58 | |
*** ircuser-1 has joined #openstack-nova | 21:58 | |
*** mdbooth has joined #openstack-nova | 21:58 | |
*** trozet has joined #openstack-nova | 21:59 | |
*** aloga has joined #openstack-nova | 21:59 | |
*** PramodJ has joined #openstack-nova | 22:00 | |
*** lennyb has joined #openstack-nova | 22:00 | |
*** rybridges has joined #openstack-nova | 22:00 | |
*** ansiwen has joined #openstack-nova | 22:00 | |
*** Pramod has joined #openstack-nova | 22:00 | |
*** gouthamr has joined #openstack-nova | 22:00 | |
*** jdillaman has joined #openstack-nova | 22:00 | |
*** PramodJ has quit IRC | 22:01 | |
*** Pramod has quit IRC | 22:01 | |
*** pramodrj07 has quit IRC | 22:01 | |
*** MasterOfBugs has quit IRC | 22:01 | |
*** tlbr has joined #openstack-nova | 22:02 | |
mriedem | edleafe: 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 |
openstackgerrit | Merged openstack/nova stable/pike: Vzstorage: synchronize volume connect https://review.openstack.org/524688 | 22:05 |
*** sree has joined #openstack-nova | 22:07 | |
*** brad[] has joined #openstack-nova | 22:08 | |
*** yangyapeng has joined #openstack-nova | 22:09 | |
mriedem | edleafe: 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-nova | 22:10 | |
*** sree has quit IRC | 22:11 | |
edleafe | mriedem: I can do it | 22:12 |
edleafe | mriedem: re: the filter properties dict key, I'm tracing the logic backwards | 22:12 |
*** yangyapeng has quit IRC | 22:13 | |
edleafe | mriedem: ok, yeah, I don't see any paths that the filter_properties could contain a SchedulerLimits object | 22:15 |
mriedem | yeah i think it's handled in the 4 cases that matter | 22:16 |
mriedem | it's just different in two of them which is why i had to dig | 22:16 |
*** ijw has joined #openstack-nova | 22:17 | |
*** kumarmn has joined #openstack-nova | 22:18 | |
*** rcernin has joined #openstack-nova | 22:19 | |
*** ijw has quit IRC | 22:21 | |
*** penick has joined #openstack-nova | 22:22 | |
*** huanxie has quit IRC | 22:23 | |
*** huanxie has joined #openstack-nova | 22:24 | |
*** baoli has quit IRC | 22:28 | |
*** baoli has joined #openstack-nova | 22:29 | |
*** lyan has quit IRC | 22:30 | |
openstackgerrit | Ed Leafe proposed openstack/nova master: Change RPC for select_destinations() https://review.openstack.org/516707 | 22:33 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils https://review.openstack.org/511357 | 22:33 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Make conductor pass and use host_lists https://review.openstack.org/511358 | 22:33 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize https://review.openstack.org/526436 | 22:33 |
edleafe | mriedem: addressed your comments ^^ | 22:33 |
mriedem | on it | 22:33 |
*** baoli has quit IRC | 22:33 | |
*** xyang1 has quit IRC | 22:34 | |
*** dave-mccowan has quit IRC | 22:34 | |
*** gouthamr has quit IRC | 22:40 | |
openstackgerrit | Merged openstack/nova stable/pike: Add regression test for rebuilding a volume-backed server https://review.openstack.org/525632 | 22:41 |
mriedem | edleafe: +2 on that bottom patch | 22:42 |
mriedem | thanks | 22:42 |
mriedem | and the one after it | 22:44 |
*** hongbin has quit IRC | 22:44 | |
mriedem | jaypipes: dansmith: https://review.openstack.org/#/c/516707/ and the one after it have my +2 | 22:44 |
*** larainema has quit IRC | 22:45 | |
edleafe | mriedem: kewl | 22:46 |
mriedem | +2 on cdent's cache headers patch too https://review.openstack.org/#/c/521640/ | 22:46 |
*** kumarmn has quit IRC | 22:47 | |
*** jogo has quit IRC | 22:48 | |
jaypipes | mriedem: cool. | 22:48 |
*** rodolof has quit IRC | 22:53 | |
ildikov | mriedem: BTW as for resize, do we need to do the same magic with finish_revert_resize too? | 22:53 |
mriedem | ildikov: maybe | 22:55 |
mriedem | likely need a test for confirm and revert a resize with an instance that has a volume attached | 22:55 |
ildikov | mriedem: as we're calling _terminate_volume_connections in revert_resize too which basically that same story just backwards | 22:56 |
ildikov | *is basically the | 22:56 |
ildikov | I mean backwards from one host to the other | 22:57 |
ildikov | anyway, I think you got what I wanted to say :) | 22:57 |
openstackgerrit | Sam Morrison proposed openstack/nova master: Allow ability for non admin users to use all filters on server list. https://review.openstack.org/526558 | 22:57 |
*** yangyapeng has joined #openstack-nova | 22:58 | |
*** sree has joined #openstack-nova | 23:02 | |
*** yangyapeng has quit IRC | 23:03 | |
*** threestrands_ has joined #openstack-nova | 23:04 | |
*** threestrands_ has quit IRC | 23:04 | |
*** threestrands_ has joined #openstack-nova | 23:04 | |
*** ijw has joined #openstack-nova | 23:05 | |
*** kumarmn has joined #openstack-nova | 23:06 | |
*** sree has quit IRC | 23:06 | |
*** threestrands has quit IRC | 23:06 | |
*** sree has joined #openstack-nova | 23:07 | |
*** ijw has quit IRC | 23:09 | |
*** gbarros has quit IRC | 23:09 | |
*** kumarmn has quit IRC | 23:11 | |
*** sree has quit IRC | 23:11 | |
*** felipemonteiro_ has quit IRC | 23:16 | |
*** huanxie has quit IRC | 23:20 | |
*** jappleii__ has joined #openstack-nova | 23:20 | |
*** felipemonteiro_ has joined #openstack-nova | 23:20 | |
*** huanxie has joined #openstack-nova | 23:21 | |
*** jappleii__ has quit IRC | 23:21 | |
*** jappleii__ has joined #openstack-nova | 23:22 | |
*** gmann_afk is now known as gmann | 23:22 | |
*** threestrands_ has quit IRC | 23:22 | |
gmann | mriedem: 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-servers | 23:23 |
gmann | patch - https://review.openstack.org/#/c/527042/ | 23:23 |
mriedem | gmann: that's pretty hard to read | 23:24 |
gmann | i tried to make it list but that's huge | 23:24 |
*** marst has quit IRC | 23:24 | |
mriedem | gmann: it would probably be better to create a page in the server concepts guide, with a sorted list that we can link to | 23:25 |
mriedem | that page can also point out which microversions added any filter item | 23:25 |
gmann | mriedem: ok, that also good, ll do. | 23:26 |
*** mlavalle has quit IRC | 23:27 | |
mriedem | jaypipes: ironic multinode job passed on https://review.openstack.org/#/c/508555/ if you want to send that in | 23:27 |
*** jpena|away is now known as jpena|off | 23:28 | |
gmann | mriedem: 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 list | 23:29 |
mriedem | gmann: that could work since the non-admin whitelist is much smaller | 23:30 |
gmann | yea | 23:30 |
gmann | let me push and if that is clear enough | 23:31 |
gmann | mriedem: thanks. | 23:32 |
mriedem | yw | 23:32 |
* gmann hurry for bank | 23:32 | |
*** gmann is now known as gmann_afk | 23:32 | |
*** Tom-Tom has quit IRC | 23:37 | |
*** penick has quit IRC | 23:40 | |
*** sree has joined #openstack-nova | 23:42 | |
*** _ix has quit IRC | 23:43 | |
*** felipemonteiro_ has quit IRC | 23:44 | |
*** jogo has joined #openstack-nova | 23:45 | |
*** baoli has joined #openstack-nova | 23:46 | |
*** sree has quit IRC | 23:46 | |
*** linkmark has quit IRC | 23:47 | |
*** Apoorva has quit IRC | 23:48 | |
*** Apoorva has joined #openstack-nova | 23:48 | |
*** baoli_ has joined #openstack-nova | 23:50 | |
*** baoli has quit IRC | 23:50 | |
*** huanxie has quit IRC | 23:51 | |
*** yangyapeng has joined #openstack-nova | 23:52 | |
*** huanxie has joined #openstack-nova | 23:54 | |
*** wxy has quit IRC | 23:55 | |
*** karlamrhein has quit IRC | 23:55 | |
*** DuncanT has quit IRC | 23:55 | |
*** salv-orlando has quit IRC | 23:55 | |
*** wxy has joined #openstack-nova | 23:56 | |
*** johnsom has quit IRC | 23:56 | |
openstackgerrit | Merged openstack/nova stable/pike: Only query BDMs once in API during rebuild https://review.openstack.org/525633 | 23:56 |
*** yangyapeng has quit IRC | 23:56 | |
*** TheJulia has quit IRC | 23:56 | |
*** melwitt has quit IRC | 23:57 | |
*** Kvisle has quit IRC | 23:57 | |
*** McNinja has quit IRC | 23:57 | |
*** betherly has quit IRC | 23:57 | |
*** ameade has quit IRC | 23:57 | |
*** tinwood has quit IRC | 23:57 | |
*** mwhahaha has quit IRC | 23:57 | |
*** toan has quit IRC | 23:57 | |
*** karlamrhein has joined #openstack-nova | 23:58 | |
*** TheJulia has joined #openstack-nova | 23:58 | |
*** AlexeyAbashkin has joined #openstack-nova | 23:58 | |
*** tinwood has joined #openstack-nova | 23:58 | |
*** johnsom has joined #openstack-nova | 23:58 | |
openstackgerrit | Merged openstack/nova stable/pike: Get original image_id from volume for volume-backed instance rebuild https://review.openstack.org/525634 | 23:59 |
openstackgerrit | Merged openstack/nova stable/pike: Fix 'force' parameter in os-quota-sets PUT schema https://review.openstack.org/526419 | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!