Tuesday, 2018-03-13

*** liusheng has joined #openstack-nova00:00
*** awaugama has quit IRC00:04
*** Sukhdev has joined #openstack-nova00:04
taskerI just got through looking at my history via stackalytics and it has me associated with "*independent". the "how to change?" pop-out explains the rules of determining my association, but not really how to change it. on my launchpad.net profile, I don't see any field to change my company. am I missing something?00:11
*** itlinux has quit IRC00:14
*** yangyapeng has joined #openstack-nova00:21
openstackgerritEric M Gonzalez (tasker) proposed openstack/nova master: unquiesce instance after quiesce failure  https://review.openstack.org/55086500:22
*** xinliang has quit IRC00:22
*** tbachman has joined #openstack-nova00:24
*** yangyapeng has quit IRC00:26
taskerI guess I need to add my info to https://github.com/openstack/stackalytics/blob/master/etc/default_data.json and submit for review?00:27
*** tetsuro has joined #openstack-nova00:28
*** salv-orlando has joined #openstack-nova00:29
*** stakeda has joined #openstack-nova00:31
*** chyka has joined #openstack-nova00:34
*** salv-orlando has quit IRC00:34
*** mriedem has quit IRC00:36
*** yamahata_ has quit IRC00:37
*** chyka has quit IRC00:38
*** xinliang has joined #openstack-nova00:39
openstackgerritGiridhar Jayavelu proposed openstack/nova-specs master: VMware: map instance to DRS vm group  https://review.openstack.org/55219000:42
*** Swami has quit IRC00:45
*** zhurong has joined #openstack-nova00:47
*** liuzz_ has joined #openstack-nova00:49
openstackgerritNakanishi Tomotaka proposed openstack/nova master: Use Placement API to check resource usage  https://review.openstack.org/52895300:50
openstackgerritGiridhar Jayavelu proposed openstack/nova-specs master: VMware: map instance to DRS vm group  https://review.openstack.org/55219000:51
*** liuzz has quit IRC00:51
*** Dinesh_Bhor has joined #openstack-nova00:52
*** gjayavelu has quit IRC00:56
*** liuzz_ has quit IRC00:57
*** liuzz has joined #openstack-nova00:57
*** claudiub|2 has quit IRC00:58
*** tbachman has quit IRC00:58
*** kaisers1 has joined #openstack-nova01:06
*** kaisers has quit IRC01:07
*** tiendc has joined #openstack-nova01:08
*** yassine has quit IRC01:09
*** phuongnh has joined #openstack-nova01:09
*** tiendc has quit IRC01:10
*** harlowja has quit IRC01:10
*** jichen has joined #openstack-nova01:12
*** yassine has joined #openstack-nova01:14
*** germs has joined #openstack-nova01:15
*** tiendc has joined #openstack-nova01:18
*** yangyapeng has joined #openstack-nova01:19
*** germs has quit IRC01:19
*** hongbin has joined #openstack-nova01:24
SpazmoticHmm not too sure how that will work tasker,  stackalytics associates by company based on email address as far as I know and a list of known companies01:30
*** salv-orlando has joined #openstack-nova01:31
Spazmoticjson looks like a decentp lace to change it if it's there though ;P01:31
*** zhaochao has joined #openstack-nova01:33
*** amodi has joined #openstack-nova01:35
*** salv-orlando has quit IRC01:35
*** Sukhdev has quit IRC01:36
*** lei-zh has joined #openstack-nova01:38
*** nicolasbock has quit IRC01:38
*** namnh has joined #openstack-nova01:39
*** nicolasbock has joined #openstack-nova01:40
*** tidwellr has joined #openstack-nova01:40
*** tbachman has joined #openstack-nova01:42
*** zhurong has quit IRC01:45
*** gongysh has joined #openstack-nova01:45
*** wolverineav has quit IRC01:51
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver  https://review.openstack.org/52338701:51
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver  https://review.openstack.org/52765801:51
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add snapshot function  https://review.openstack.org/53424001:51
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add power actions  https://review.openstack.org/54334001:51
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add get console output  https://review.openstack.org/54334401:51
*** yamahata_ has joined #openstack-nova01:51
*** wolverineav has joined #openstack-nova01:51
jichenmelwitt: hi, could you please let me know where's the etherpad you mentioned that we track previously approved low priority specs?01:52
openstackgerritMichael Still proposed openstack/nova master: Move configurable mkfs to privsep.  https://review.openstack.org/55192101:52
openstackgerritMichael Still proposed openstack/nova master: Move xenapi xenstore_read's to privsep.  https://review.openstack.org/55224101:52
openstackgerritMichael Still proposed openstack/nova master: Move xenapi disk resizing to privsep.  https://review.openstack.org/55224201:52
jichenmelwitt: we have a set of patches ready but lack of review since queens......thanks https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/add-zvm-driver-rocky01:52
*** wolverineav has quit IRC01:55
*** wolverineav has joined #openstack-nova01:57
*** tuanla____ has joined #openstack-nova02:00
*** wolverineav has quit IRC02:01
*** r-daneel has quit IRC02:03
*** rcernin has quit IRC02:04
*** amodi has quit IRC02:09
*** r-daneel has joined #openstack-nova02:11
*** annp has joined #openstack-nova02:13
*** suresh12_ has quit IRC02:20
*** suresh12 has joined #openstack-nova02:22
*** zhurong has joined #openstack-nova02:26
*** suresh12 has quit IRC02:27
*** itlinux has joined #openstack-nova02:30
*** wolverineav has joined #openstack-nova02:31
*** salv-orlando has joined #openstack-nova02:31
*** nicolasbock has quit IRC02:32
SpazmoticMichael Still if you're here i'll put those in the Xenapi Subteam etherpad02:34
*** wolverineav has quit IRC02:35
*** salv-orlando has quit IRC02:36
*** yassine has quit IRC02:45
*** psachin has joined #openstack-nova02:47
*** suresh12 has joined #openstack-nova02:48
*** yassine has joined #openstack-nova02:49
*** suresh12 has quit IRC02:52
*** tidwellr has quit IRC02:55
*** amodi has joined #openstack-nova02:57
*** gouthamr has quit IRC03:00
*** stakeda has quit IRC03:00
*** bkopilov has quit IRC03:05
*** wolverineav has joined #openstack-nova03:06
*** rcernin has joined #openstack-nova03:07
*** wolverineav has quit IRC03:11
*** germs has joined #openstack-nova03:15
*** germs has quit IRC03:15
*** germs has joined #openstack-nova03:15
*** edmondsw has joined #openstack-nova03:18
*** germs has quit IRC03:20
*** lei-zh has quit IRC03:22
*** edmondsw has quit IRC03:22
*** Zames has joined #openstack-nova03:22
*** lei-zh has joined #openstack-nova03:22
*** Zames has quit IRC03:26
*** deepak_ has quit IRC03:27
*** Zames has joined #openstack-nova03:28
*** salv-orlando has joined #openstack-nova03:32
*** salv-orlando has quit IRC03:36
*** amodi has quit IRC03:37
*** Tom-Tom has joined #openstack-nova03:39
*** deepak_ has joined #openstack-nova03:39
*** Zames has quit IRC03:44
*** wolverineav has joined #openstack-nova03:45
*** gongysh has quit IRC03:47
*** harlowja has joined #openstack-nova03:48
*** Tom-Tom has quit IRC03:48
*** Tom-Tom has joined #openstack-nova03:48
*** links has joined #openstack-nova03:50
*** hongbin has quit IRC03:51
*** fragatina has quit IRC03:52
*** Spaz-Home has joined #openstack-nova03:53
*** Tom-Tom has quit IRC03:53
*** chyka has joined #openstack-nova03:53
*** amodi has joined #openstack-nova03:54
*** Tom-Tom has joined #openstack-nova03:55
*** dave-mccowan has quit IRC03:56
*** Spazmotic has quit IRC03:57
*** chyka has quit IRC03:58
*** gyee has quit IRC03:59
*** udesale has joined #openstack-nova04:03
*** shaohe_feng has quit IRC04:06
*** gongysh has joined #openstack-nova04:07
*** gongysh has quit IRC04:08
*** Tom-Tom has quit IRC04:08
*** vladikr has quit IRC04:12
*** fragatina has joined #openstack-nova04:14
*** fragatin_ has joined #openstack-nova04:17
*** fragatina has quit IRC04:18
*** gongysh has joined #openstack-nova04:20
*** gongysh has quit IRC04:21
*** gongysh has joined #openstack-nova04:21
*** gongysh has quit IRC04:22
*** gongysh has joined #openstack-nova04:22
*** gongysh has quit IRC04:25
*** suresh12 has joined #openstack-nova04:27
*** yassine has quit IRC04:28
*** sridharg has joined #openstack-nova04:32
*** salv-orlando has joined #openstack-nova04:33
*** bkopilov has joined #openstack-nova04:36
*** salv-orlando has quit IRC04:38
*** lpetrut has joined #openstack-nova04:38
*** yassine has joined #openstack-nova04:39
*** fragatin_ has quit IRC04:50
*** fragatina has joined #openstack-nova04:50
*** moshele has joined #openstack-nova04:51
*** zhurong has quit IRC04:53
*** gjayavelu has joined #openstack-nova04:56
*** ratailor has joined #openstack-nova04:59
*** suresh12 has quit IRC04:59
*** ratailor has quit IRC04:59
*** Dinesh_Bhor has quit IRC05:00
*** suresh12 has joined #openstack-nova05:00
*** Dinesh_Bhor has joined #openstack-nova05:01
*** lpetrut has quit IRC05:05
*** edmondsw has joined #openstack-nova05:06
*** lei-zh1 has joined #openstack-nova05:06
*** ratailor has joined #openstack-nova05:08
*** lei-zh has quit IRC05:08
*** edmondsw has quit IRC05:10
*** Dinesh_Bhor has quit IRC05:11
*** lpetrut has joined #openstack-nova05:11
*** liverpooler has quit IRC05:12
*** salv-orlando has joined #openstack-nova05:14
*** lei-zh1 has quit IRC05:16
*** germs has joined #openstack-nova05:16
*** germs has quit IRC05:16
*** germs has joined #openstack-nova05:16
*** lei-zh has joined #openstack-nova05:16
*** lei-zh1 has joined #openstack-nova05:17
*** lei-zh has quit IRC05:17
*** yassine has quit IRC05:18
*** Dinesh_Bhor has joined #openstack-nova05:19
*** germs has quit IRC05:21
*** jaosorior has quit IRC05:22
openstackgerritwangqi proposed openstack/nova-specs master: Clean imports in code  https://review.openstack.org/55229405:23
*** lpetrut has quit IRC05:23
*** sidx64 has joined #openstack-nova05:26
*** itlinux has quit IRC05:26
*** mdnadeem has joined #openstack-nova05:26
*** wolverineav has quit IRC05:26
*** wolverineav has joined #openstack-nova05:27
*** yangyapeng has quit IRC05:30
*** janki has joined #openstack-nova05:30
*** yangyapeng has joined #openstack-nova05:30
*** wolverineav has quit IRC05:32
*** yassine has joined #openstack-nova05:32
*** oanson has quit IRC05:37
*** zhurong has joined #openstack-nova05:42
*** claudiub|2 has joined #openstack-nova05:44
*** harlowja has quit IRC05:59
*** amodi has quit IRC05:59
*** suresh12 has quit IRC05:59
*** vks1 has joined #openstack-nova05:59
*** gongysh has joined #openstack-nova06:02
*** david-lyle has quit IRC06:08
*** ccamacho has quit IRC06:15
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/54877206:16
*** hiro-kobayashi has joined #openstack-nova06:20
*** Dinesh_Bhor has quit IRC06:21
*** vks1 has quit IRC06:21
*** Dinesh_Bhor has joined #openstack-nova06:23
*** germs has joined #openstack-nova06:29
*** germs has quit IRC06:29
*** germs has joined #openstack-nova06:29
*** suresh12 has joined #openstack-nova06:30
*** Eran_Kuris has joined #openstack-nova06:31
*** sridharg has quit IRC06:32
*** hiro-kobayashi has quit IRC06:34
*** suresh12 has quit IRC06:35
*** oanson has joined #openstack-nova06:38
*** oanson has quit IRC06:38
*** oanson has joined #openstack-nova06:39
*** vks1 has joined #openstack-nova06:44
*** claudiub|2 has quit IRC06:44
*** sahid has joined #openstack-nova06:44
*** liusheng has quit IRC06:45
openstackgerritJianle He proposed openstack/nova master: fix bug/1619606 on https://bugs.launchpad.net/nova/+bug/1619606  https://review.openstack.org/55231006:48
openstackLaunchpad bug 1619606 in OpenStack Compute (nova) "snapshot_volume_backed races, could result in data corruption" [Low,Confirmed] - Assigned to Jianle He (hejianle1989)06:48
*** liusheng has joined #openstack-nova06:48
*** lei-zh1 has quit IRC06:50
*** kholkina has joined #openstack-nova06:51
openstackgerritPranab proposed openstack/os-vif master: Add abstract OVSDB API  https://review.openstack.org/47661206:52
*** germs has quit IRC06:52
openstackgerritYikun Jiang (Kero) proposed openstack/nova-specs master: Add host info to instance action events  https://review.openstack.org/54327706:54
openstackgerritMerged openstack/nova master: remove unnecessary conf imports  https://review.openstack.org/53931406:54
*** edmondsw has joined #openstack-nova06:54
*** lei-zh has joined #openstack-nova06:55
*** zhurong has quit IRC06:55
*** edmondsw has quit IRC06:59
openstackgerritDao Cong Tien proposed openstack/nova master: Stop using mox in virt/xenapi/image/test_vdi_through_dev.py  https://review.openstack.org/34268207:01
*** gjayavelu has quit IRC07:01
*** suresh12 has joined #openstack-nova07:01
*** jaosorior has joined #openstack-nova07:02
*** kholkina has quit IRC07:04
*** hiro-kobayashi has joined #openstack-nova07:04
*** sridharg has joined #openstack-nova07:06
*** suresh12 has quit IRC07:06
*** liuyulong has joined #openstack-nova07:07
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/55236307:09
openstackgerritPranab proposed openstack/os-vif master: Add abstract OVSDB API  https://review.openstack.org/47661207:09
*** zhaochao has quit IRC07:09
openstackgerritYikun Jiang (Kero) proposed openstack/nova-specs master: Add host info to instance action events  https://review.openstack.org/54327707:10
*** zhaochao has joined #openstack-nova07:11
*** gjayavelu has joined #openstack-nova07:11
*** fragatina has quit IRC07:13
*** fragatina has joined #openstack-nova07:14
*** sidx64 has quit IRC07:14
*** gjayavelu has quit IRC07:16
openstackgerritPranab proposed openstack/os-vif master: Add native implementation OVSDB API  https://review.openstack.org/48222607:18
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/53391807:20
*** sar has joined #openstack-nova07:20
*** lpetrut has joined #openstack-nova07:20
*** ttsiouts has quit IRC07:22
*** ttsiouts has joined #openstack-nova07:22
*** sidx64 has joined #openstack-nova07:23
*** tojuvone has quit IRC07:23
*** rcernin has quit IRC07:24
*** mdnadeem has quit IRC07:25
openstackgerritOpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements  https://review.openstack.org/55238407:27
*** salv-orlando has quit IRC07:29
*** chyka has joined #openstack-nova07:29
*** vks1 has quit IRC07:31
*** salv-orlando has joined #openstack-nova07:34
*** chyka has quit IRC07:34
*** suresh12 has joined #openstack-nova07:34
*** mdnadeem has joined #openstack-nova07:35
*** suresh12 has quit IRC07:39
*** pcaruana has joined #openstack-nova07:42
*** lei-zh has quit IRC07:44
*** andreas_s has joined #openstack-nova07:44
*** lei-zh has joined #openstack-nova07:44
*** sidx64 has quit IRC07:47
*** maciejjozefczyk has joined #openstack-nova07:47
*** vks1 has joined #openstack-nova07:50
*** pasuder has joined #openstack-nova07:50
*** AlexeyAbashkin has joined #openstack-nova07:50
*** links has quit IRC07:52
*** sidx64 has joined #openstack-nova07:54
*** danpawlik has joined #openstack-nova07:57
*** lajoskatona has joined #openstack-nova07:58
*** links has joined #openstack-nova07:59
openstackgerritMerged openstack/nova master: conf: Remove '[conductor] topic' opt  https://review.openstack.org/49917908:00
*** ccamacho has joined #openstack-nova08:05
*** ccamacho has joined #openstack-nova08:06
*** suresh12 has joined #openstack-nova08:07
*** suresh12 has quit IRC08:11
*** ameeda has joined #openstack-nova08:12
*** suresh12 has joined #openstack-nova08:15
*** jpena|off is now known as jpena08:16
*** suresh12 has quit IRC08:19
*** tesseract has joined #openstack-nova08:19
*** tesseract-RH has joined #openstack-nova08:20
*** tesseract-RH has quit IRC08:20
*** hiro-kobayashi has quit IRC08:21
*** vks1 has quit IRC08:22
*** salv-orlando has quit IRC08:23
*** tssurya has joined #openstack-nova08:23
*** pmannidi has joined #openstack-nova08:34
*** pmannidi has quit IRC08:34
*** sticker has joined #openstack-nova08:34
*** vks1 has joined #openstack-nova08:34
*** sidx64 has quit IRC08:39
*** shaohe_feng has joined #openstack-nova08:40
*** r-daneel has quit IRC08:41
*** edmondsw has joined #openstack-nova08:42
*** damien_r has joined #openstack-nova08:43
*** suresh12 has joined #openstack-nova08:43
*** kholkina has joined #openstack-nova08:46
*** cdent has joined #openstack-nova08:46
*** edmondsw has quit IRC08:46
*** alexchadin has joined #openstack-nova08:46
*** suresh12 has quit IRC08:48
*** r-daneel has joined #openstack-nova08:48
*** abhishekk has joined #openstack-nova08:49
*** abhishekk is now known as akekane|wfh08:50
*** akekane|wfh is now known as abhishekk08:50
*** trinaths has joined #openstack-nova08:52
*** hiro-kobayashi has joined #openstack-nova08:56
*** salv-orlando has joined #openstack-nova08:57
*** suresh12 has joined #openstack-nova08:57
*** priteau has joined #openstack-nova08:59
*** ralonsoh has joined #openstack-nova08:59
*** alexchadin has quit IRC08:59
*** lucas-afk is now known as lucasagomes08:59
*** amoralej|off is now known as amoralej09:01
*** suresh12 has quit IRC09:02
openstackgerritChris Dent proposed openstack/nova master: Move resource class fields  https://review.openstack.org/54004909:03
openstackgerritChris Dent proposed openstack/nova master: Move resource provider objects into placement hierarchy  https://review.openstack.org/55152809:03
openstackgerritChris Dent proposed openstack/nova master: Reparent placement objects to oslo_versionedobjects  https://review.openstack.org/55152909:03
openstackgerritChris Dent proposed openstack/nova master: Optional separate database for placement API  https://review.openstack.org/36276609:03
openstackgerritChris Dent proposed openstack/nova master: Isolate placement database config  https://review.openstack.org/54143509:03
openstackgerritChris Dent proposed openstack/nova master: Move placement exceptions into the placement package  https://review.openstack.org/54986209:03
*** ccamacho has quit IRC09:04
openstackgerritChris Dent proposed openstack/nova master: Use nova.db.api directly  https://review.openstack.org/54326209:09
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver  https://review.openstack.org/52338709:12
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver  https://review.openstack.org/52765809:12
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add snapshot function  https://review.openstack.org/53424009:12
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add power actions  https://review.openstack.org/54334009:12
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add get console output  https://review.openstack.org/54334409:12
openstackgerritChris Dent proposed openstack/nova master: DNM: Demo code for microversion parse extraction  https://review.openstack.org/55026509:12
*** ccamacho has joined #openstack-nova09:21
*** sridharg has quit IRC09:24
*** sridharg has joined #openstack-nova09:24
*** sidx64 has joined #openstack-nova09:26
*** derekh has joined #openstack-nova09:26
*** sidx64 has quit IRC09:26
bauzasmorning folks09:30
cdentmornin09:31
*** ragiman has joined #openstack-nova09:31
*** mvk has quit IRC09:32
*** Dinesh_Bhor has quit IRC09:34
*** yangyapeng has quit IRC09:35
*** alexchadin has joined #openstack-nova09:36
*** yangyapeng has joined #openstack-nova09:37
*** FL1SK has quit IRC09:37
*** ratailor is now known as rtailor09:39
*** sdague has joined #openstack-nova09:50
*** suresh12 has joined #openstack-nova09:51
*** claudiub|2 has joined #openstack-nova09:52
*** sidx64 has joined #openstack-nova09:54
*** suresh12 has quit IRC09:55
*** yamahata_ has quit IRC09:56
*** vks1 has quit IRC09:57
*** namnh has quit IRC09:58
*** dtantsur|afk is now known as dtantsur09:59
*** lei-zh has quit IRC10:03
*** suresh12 has joined #openstack-nova10:03
*** sidx64 has quit IRC10:08
*** suresh12 has quit IRC10:08
*** hiro-kobayashi has quit IRC10:09
*** ameeda has quit IRC10:09
*** sidx64 has joined #openstack-nova10:11
*** sidx64 has quit IRC10:13
gibilate morning10:15
*** vks1 has joined #openstack-nova10:16
*** pooja-jadhav is now known as pooja_jadhav10:16
* cdent waves at gibi 10:17
*** mgoddard_ has joined #openstack-nova10:17
*** jichen has quit IRC10:18
*** sidx64 has joined #openstack-nova10:18
*** sidx64 has quit IRC10:19
* gibi waves back10:20
*** rtailor has quit IRC10:22
*** brault_ has quit IRC10:23
*** ratailor has joined #openstack-nova10:24
*** sidx64 has joined #openstack-nova10:24
*** edmondsw has joined #openstack-nova10:30
*** edmondsw has quit IRC10:35
sardoes anyone know where in the database this info is stored: os-extended-volumes:volumes_attached10:35
sarand which database. nova or cinder10:35
sarspecifically where delete_on_termination can be found10:36
*** sidx64 has quit IRC10:38
kashyapmdbooth: That's the redundant Cinder call, is it?10:40
kashyap        bdms = objects.BlockDeviceMappingList.get_by_instance_uuid(10:40
kashyap                ctxt, instance.uuid)10:40
kashyapIn nova/compute/manager.py, _post_live_migration()10:40
mdboothkashyap: No, the redundant cinder call is the second call to initialize_connection10:40
mdboothTo fetch the connection_info which we had before, but overwrote during pre_live_migration10:41
kashyapErr, the yeah.  /me reads closer10:42
mdboothkashyap: With my patch we're still fetching those BDMs, we just do it earlier, before calling pre_live_migration10:42
kashyapYeah, re-reading your commit10:42
mdboothThat means we've stored the 'old' values, which is also why I've explicitly named them source_bdms.10:43
kashyapYeah, noticed the 'source_bdms' parameter, and the accompanying code comment10:44
*** sidx64 has joined #openstack-nova10:45
mdboothkashyap: I'd be glad of a critical eye on that, btw. Substantive -1s very much appreciated!10:45
*** pcaruana has quit IRC10:45
kashyapmdbooth: Understood.  I don't claim expertise in the BDM area, first educating myself about the code area.10:46
kashyapmdbooth: Also sounds like you want to see it exercise in a functional test by "someone"?10:46
kashyap(Besides the 'mocks' in the unit tests)10:47
*** phuongnh has quit IRC10:47
*** brault has joined #openstack-nova10:47
mdboothWell testing is something I'm not sure of. I have tested it manually (well, the equivalent backported to Newton), but ideally it's not doing anything it wasn't doing before, just in a different order.10:48
mdboothSo if you've got an idea for a useful additional test that would also be good.10:48
mdboothstephenfin: ^^^10:48
kashyapAh, okay.  I see you're more looking for any potential races10:48
mdboothThere shouldn't be an issue with races because the call to pre_live_migration is synchronous, but again please verify.10:49
* kashyap nods10:50
*** brault has quit IRC10:52
*** alexchadin has quit IRC10:56
kashyap`git review -d` for a tiny change is horribly slow for me; wonder if it's just me10:56
kashyap(It's _still_ "downloading", for Pete's sake.)10:57
*** mvk has joined #openstack-nova11:00
*** suresh12 has joined #openstack-nova11:00
*** vivsoni_ has quit IRC11:00
*** RaoulHC has joined #openstack-nova11:01
*** rcernin has joined #openstack-nova11:02
*** brault has joined #openstack-nova11:03
kashyapDone; /me should've timed it11:03
*** ratailor has quit IRC11:04
*** suresh12 has quit IRC11:04
*** ratailor has joined #openstack-nova11:05
openstackgerritBence Romsics proposed openstack/osc-placement master: Do not depend on jenkins user in devstack gate  https://review.openstack.org/55247611:05
*** chyka has joined #openstack-nova11:06
*** nicolasbock has joined #openstack-nova11:06
*** alexchadin has joined #openstack-nova11:07
*** sidx64 has quit IRC11:07
*** sidx64 has joined #openstack-nova11:08
*** brault has quit IRC11:10
*** chyka has quit IRC11:10
*** tbachman has quit IRC11:10
openstackgerritStephen Finucane proposed openstack/nova master: Transform live_migration.post.dest notifications  https://review.openstack.org/46978411:19
*** vks1 has quit IRC11:23
*** tiendc has quit IRC11:25
*** bkopilov has quit IRC11:30
*** suresh12 has joined #openstack-nova11:38
*** takedakn has joined #openstack-nova11:39
*** vivsoni has joined #openstack-nova11:39
openstackgerritRaoul Hidalgo Charman proposed openstack/nova master: Expose shutdown retry interval as config setting  https://review.openstack.org/55248311:39
*** udesale_ has joined #openstack-nova11:39
*** dave-mccowan has joined #openstack-nova11:42
*** udesale has quit IRC11:42
*** suresh12 has quit IRC11:43
*** gongysh has quit IRC11:43
*** udesale_ has quit IRC11:44
*** sidx64 has quit IRC11:45
*** edmondsw has joined #openstack-nova11:45
*** yamamoto has quit IRC11:46
*** brault has joined #openstack-nova11:48
*** FL1SK has joined #openstack-nova11:48
*** sidx64 has joined #openstack-nova11:49
*** sidx64 has quit IRC11:49
*** yamamoto has joined #openstack-nova11:50
*** trinaths has quit IRC11:52
*** sidx64 has joined #openstack-nova11:53
*** sidx64 has quit IRC11:54
*** sidx64 has joined #openstack-nova11:55
*** salv-orlando has quit IRC11:58
*** danpawlik has quit IRC12:00
*** rcernin has quit IRC12:00
*** ratailor has quit IRC12:00
*** yamamoto has quit IRC12:01
*** yamamoto has joined #openstack-nova12:01
*** mikal_ has joined #openstack-nova12:02
*** sambetts|afk is now known as sambetts12:02
*** liverpooler has joined #openstack-nova12:04
*** mikal has quit IRC12:04
*** yamamoto has quit IRC12:06
*** annp has quit IRC12:07
*** amoralej is now known as amoralej|lunch12:09
*** abhishekk has quit IRC12:09
*** jpena is now known as jpena|lunch12:10
*** lucasagomes is now known as lucas-hungry12:10
*** tuanla____ has quit IRC12:11
*** suresh12 has joined #openstack-nova12:12
*** takedakn has quit IRC12:12
*** liuzz has quit IRC12:12
*** tbachman has joined #openstack-nova12:13
*** tbachman has quit IRC12:15
*** suresh12 has quit IRC12:17
kashyapmdbooth: Should the comment here in post_live_migration() be updated, in light of your change: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L7716,#L772912:18
kashyapTell me if I'm talking nonsense...12:18
*** takedakn has joined #openstack-nova12:23
mdboothkashyap: Um... no you're not. I've lost the libvirt driver change in the forward port :/12:25
kashyapmdbooth: Wanted to check here before I added a comment in the review;12:25
*** janki has quit IRC12:25
*** mgoddard_ has quit IRC12:30
*** yangyapeng has quit IRC12:30
*** yangyapeng has joined #openstack-nova12:30
*** mgoddard_ has joined #openstack-nova12:31
cdentedleafe: do you have an opinion on whether we need both 'any' and 'all' on member_of? (I think you answered everything else I mentioned on that review)12:31
*** vks1 has joined #openstack-nova12:32
*** vladikr has joined #openstack-nova12:33
openstackgerritRaoul Hidalgo Charman proposed openstack/nova master: Expose shutdown retry interval as config setting  https://review.openstack.org/55248312:34
*** Eran_Kuris has quit IRC12:34
*** zhurong has joined #openstack-nova12:35
edleafecdent: the only use cases ever discussed were for the 'any' case12:35
edleafecdent: I can't think of an 'all' case, but I think that would be appropriate for an iterative approach if it ever does arise12:36
* cdent snickers12:36
cdentmy brain struggles to articulate the reason for the 'any' case. It's the 'all' case that seems obvious to me but I suspect I'm not understanding the path of things in nova-scheduler12:37
*** pchavva has joined #openstack-nova12:38
edleafeRight now we get back all RPs. This is simply a way to begin to limit that through aggs12:38
edleafeIIRC, the initial approach was a single agg, but later discussions felt that was a bit too restrictive (don't remember details), so multiple agg candidates were proposed12:39
*** Zames has joined #openstack-nova12:39
openstackgerritSurya Seetharaman proposed openstack/nova master: [WIP] Add disabled field to CellMapping object  https://review.openstack.org/55009012:40
openstackgerritSurya Seetharaman proposed openstack/nova master: [WIP] Add CellMappingList.get_all_enabled() query method  https://review.openstack.org/55018812:40
openstackgerritSurya Seetharaman proposed openstack/nova master: [WIP] Allow scheduling only to enabled cells (Filter Scheduler)  https://review.openstack.org/55052712:40
openstackgerritSurya Seetharaman proposed openstack/nova master: [WIP] Add disabled column to cell_mappings table.  https://review.openstack.org/55250512:40
*** odyssey4me has quit IRC12:41
*** odyssey4me has joined #openstack-nova12:41
*** _pewp_ has quit IRC12:41
*** zhurong has quit IRC12:42
openstackgerritSurya Seetharaman proposed openstack/nova master: [WIP] Bump versions of objects dependent on CellMapping  https://review.openstack.org/55043212:43
*** _pewp_ has joined #openstack-nova12:43
*** takedakn has quit IRC12:43
*** Zames has quit IRC12:43
*** zhurong has joined #openstack-nova12:45
*** eharney has quit IRC12:45
*** Eran_Kuris has joined #openstack-nova12:48
*** takedakn has joined #openstack-nova12:49
*** Zames has joined #openstack-nova12:49
*** mriedem has joined #openstack-nova12:51
openstackgerritBence Romsics proposed openstack/osc-placement master: WIP Do not depend on jenkins user in devstack gate  https://review.openstack.org/55247612:52
*** takedakn has quit IRC12:53
openstackgerritSurya Seetharaman proposed openstack/nova-specs master: Support disabling a cell  https://review.openstack.org/54668412:57
*** udesale has joined #openstack-nova12:57
*** salv-orlando has joined #openstack-nova12:59
*** Zames has quit IRC12:59
*** jpena|lunch is now known as jpena13:02
*** vks1 has quit IRC13:02
*** ameeda has joined #openstack-nova13:03
*** salv-orlando has quit IRC13:04
*** lyan has joined #openstack-nova13:05
*** lyan is now known as Guest9094613:05
*** psachin has quit IRC13:07
*** HW-Peter has joined #openstack-nova13:08
*** vks1 has joined #openstack-nova13:08
*** mgoddard_ has quit IRC13:10
*** danpawlik has joined #openstack-nova13:10
*** HW-Peter has quit IRC13:10
openstackgerritsahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready  https://review.openstack.org/49745713:10
*** HW-Peter has joined #openstack-nova13:11
HW-Peterhey13:11
*** lucas-hungry is now known as lucasagomes13:11
*** ameeda has quit IRC13:11
HW-Peteris there a way to regenerate the libvirt config file for an active instance using the API, without recreating the instnace?13:12
openstackgerritMerged openstack/nova master: Fix N358 hacking check  https://review.openstack.org/54767013:12
openstackgerritMerged openstack/nova master: Remove old flavor_access_get_by_flavor_id db api method  https://review.openstack.org/53970013:12
openstackgerritMerged openstack/nova master: Make the InstanceMapping marker UUID-like  https://review.openstack.org/53932313:13
*** zhurong has quit IRC13:15
*** Eran_Kuris has quit IRC13:16
*** Eran_Kuris has joined #openstack-nova13:18
*** jaosorior has quit IRC13:19
*** nclNGJPQ8 has joined #openstack-nova13:23
*** nclNGJPQ8 has quit IRC13:25
edleafecdent: my gabbi-fu is weak these days. Is there a way to use a UUID defined in the fixture in a URL call? E.g.: GET /allocation_candidates?member_of=$ENVIRON['AGG1_UUID']13:26
cdentyeah, pretty much just like that13:26
cdentthe fixture needs to set the environ (it does a few of those already)13:27
edleafecdent: ok, wasn't sure where the substitutions were allowed13:27
*** vks1 has quit IRC13:27
cdentpretty much anywhere on the right hand side13:27
sean-k-mooneycdent: no mather how many times i see it in opnestack code it still amuses me we use environ for things like this13:27
*** gongysh has joined #openstack-nova13:28
cdentsean-k-mooney: it comes out of the fact that gabbi is capable of being used for running tests from any language (using gabbi-runner, the cli) and the environ is sort of the universal thing13:28
*** yangyapeng has quit IRC13:29
cdentedleafe: you can do left hand subs in response_json_paths13:29
cdenthttps://gabbi.readthedocs.io/en/latest/format.html#substitution13:29
*** yangyapeng has joined #openstack-nova13:29
*** danpawlik has quit IRC13:29
openstackgerritBence Romsics proposed openstack/osc-placement master: WIP Do not depend on jenkins user in devstack gate  https://review.openstack.org/55247613:29
*** yangyapeng has quit IRC13:30
HW-Peterif an instance had it's libvirt xml file deleted, is there a way to regenerate it?13:30
*** tblakes has joined #openstack-nova13:30
*** vks1 has joined #openstack-nova13:30
openstackgerritMerged openstack/nova master: Transform servergroup.addmember notification  https://review.openstack.org/54110113:32
openstackgerritMerged openstack/nova master: conf: Remove deprecated 'allow_instance_snapshots' opt  https://review.openstack.org/49962113:32
*** amoralej|lunch is now known as amoralej13:32
*** yamamoto has joined #openstack-nova13:34
*** suresh12 has joined #openstack-nova13:35
edleafecdent: thanks; I got that part. I wasn't sure if putting them in the URL was allowed13:36
*** eharney has joined #openstack-nova13:37
cdentyeah, from gabbi's perspective (at that stage) it is just a string13:37
*** suresh12 has quit IRC13:39
*** _pewp_ has quit IRC13:39
*** yamamoto has quit IRC13:39
*** _pewp_ has joined #openstack-nova13:40
*** jroll has quit IRC13:41
*** jroll has joined #openstack-nova13:42
*** bkopilov has joined #openstack-nova13:42
*** dtantsur is now known as dtantsur|brb13:45
*** lpetrut has quit IRC13:45
*** mlavalle has joined #openstack-nova13:47
tblakesgibi: In the case where a deploy fails with NoValidHost, do you know where the notification is sent out that the deploy failed with that exception? Was that changed at all in the queens release?13:47
*** yamamoto has joined #openstack-nova13:53
*** esberglu has joined #openstack-nova13:53
*** takedakn has joined #openstack-nova13:53
*** awaugama has joined #openstack-nova13:54
openstackgerritMatthew Booth proposed openstack/nova master: Avoid redundant initialize_connection on source post live migration  https://review.openstack.org/55130213:54
openstackgerritMatthew Booth proposed openstack/nova master: Avoid redundant initialize_connection on source post live migration  https://review.openstack.org/55130213:55
*** yamamoto has quit IRC13:55
*** jaosorior has joined #openstack-nova13:55
openstackgerritBence Romsics proposed openstack/osc-placement master: Do not depend on jenkins user in devstack gate  https://review.openstack.org/55247613:56
stephenfingibi: Fancy sending these on their merry way? https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:crypto-cleanup13:58
openstackgerritSurya Seetharaman proposed openstack/nova master: Marker reset option for nova-manage map_instances  https://review.openstack.org/53950113:58
*** burt has joined #openstack-nova13:59
gibistephenfin: I will do that shortly13:59
gibitblakes: let me dig out13:59
*** tidwellr has joined #openstack-nova14:00
*** salv-orlando has joined #openstack-nova14:00
edleafecdent: no, I guess you can't put an ENVIRON value in the URL. I tried, and the call was rejected as invalid, with this output: u'member_of': u'$ENVIRON[AGGA_UUID]'14:00
*** gouthamr has joined #openstack-nova14:00
edleafecdent: Any other ideas?14:00
*** takedakn has quit IRC14:00
edleafecdent: I'd really rather not hardcode UUIDs into the tests14:01
cdentyou can do it, so there's something else wrong14:01
cdentare you working in a new file, or an existing one?14:01
*** mgoddard_ has joined #openstack-nova14:01
gibitblakes: is this about the preemptible instances feature?14:01
edleafeexisting14:01
cdentwhich fixture did you set your eviron in? is it liste in fixtures: at the top of the yaml file?14:01
gibitblakes: if yes then I summarized what we have today in notifications in the nova rocky etherpad https://etherpad.openstack.org/p/nova-ptg-rocky starting at L47014:02
lennybHi, I have nova instance stuck in 'deleting' after trying to deploy it with ironic. nova force-delete is not working as well. I am working with master branches. Any ideas how to solve this?14:02
edleafeyeah - I'm adding tests to the allocation-candidates.yaml GABBIT14:02
edleafeeek! capslock!14:02
*** mingyu has joined #openstack-nova14:04
cdentedleafe: It's hard for me to guess, if you wanna push or paste I can look14:04
*** salv-orlando has quit IRC14:04
edleafecdent: ok, gimme a few14:04
tblakesgibi: No this isn't about preemptible instances. I'm just wondering about the basic case where a VM deploy fails with NoValidHost. I haven't been able to find where that failure notification should get sent out.14:05
*** pasuder has left #openstack-nova14:05
*** danpawlik has joined #openstack-nova14:07
*** amodi has joined #openstack-nova14:08
gibitblakes: searching the code...14:09
edleafecdent: ok, flase alarm. A missing quote mark was the culprit. The output made me think that gabbi didn't substitute the value14:14
edleafefalse alarm, even14:14
*** alexchadin has quit IRC14:14
gibitblakes: I think this is the place https://github.com/openstack/nova/blob/107ad403d46c9d6961c5b15c68df0fa24014a832/nova/conductor/manager.py#L58814:14
*** sidx64 has quit IRC14:14
gibitblakes: this emits the legacy compute_task.build_instances notification14:15
gibitblakes: example of the payload is here http://paste.openstack.org/show/688874/14:15
cdentedleafe: if you see the original string in the error output it can only be two thing: the sub couldn't happen because it wasn't able to be parsed as a sub (your case?) or it was parseable but the info couldn't be retrieved (the environ is not set)14:15
gibitblakes: transformation of that legacy notification is still ongoing in https://review.openstack.org/#/c/482629/14:16
cdentin the former case we can't error that the sub didn't happen because gabbi doesn't know it was supposed to be a sub14:16
gibitblakes: I don't know about any change around these code path in Queens14:16
*** sidx64 has joined #openstack-nova14:16
*** gbarros has joined #openstack-nova14:17
gibitblakes: at least not for the notification14:17
edleafecdent: makes sense14:20
*** alexchadin has joined #openstack-nova14:20
edleafecdent: now I'm down to debugging sqla failures. Joy!14:20
*** alexchadin has quit IRC14:20
tblakesgibi: Thanks for pointing me to the code in build_instances!14:21
gibistephenfin: about the crypto removal patches, is this a followup of the bp remove-nova-cert14:21
gibistephenfin: or how we ended up with so many dead code?14:22
gibitblakes: no problem, I'm happy to help14:22
*** nicolasbock has quit IRC14:22
*** dtantsur|brb is now known as dtantsur14:24
kashyapjohnthetubaguy: Thanks for review here: https://review.openstack.org/#/c/534384/.  Just reading through14:26
kashyapjohnthetubaguy: A quick note on the name: `cpu_model_extra_flags`.  I used the term 'flags' there because, all these extra features are enumerated under 'flags' when you run `cat /proc/cpuinfo`14:27
kashyapBut anyway; that's relatively minor :-).  I'll get to the rest of the core changes first14:28
johnthetubaguykashyap: ah, got you, I could go either way14:28
johnthetubaguycool14:28
*** eharney has quit IRC14:28
kashyapThanks14:29
*** felipemonteiro has joined #openstack-nova14:29
*** eharney has joined #openstack-nova14:30
*** hongbin has joined #openstack-nova14:31
openstackgerritEric Fried proposed openstack/nova-specs master: Allow for merging traits and aggregates  https://review.openstack.org/55212214:32
*** nicolasbock has joined #openstack-nova14:32
*** suresh12 has joined #openstack-nova14:32
SamYaplei cant seem to find this information online, but assuming i have multiple cellv2 cells, can an instance migrate (or preferably live-migration) between the cells?14:32
*** janki has joined #openstack-nova14:33
stephenfingibi: I'm not really sure. I spotted it by chance and realized I could go delete it all14:36
gibistephenfin: meanwhile I checked and I think nova-cert was the last user14:36
gibistephenfin: anyhow seems safe to remove14:37
stephenfingibi: Good to hear :)14:37
*** suresh12 has quit IRC14:37
*** sidx64 has quit IRC14:39
*** nicolasbock has quit IRC14:41
*** chyka has joined #openstack-nova14:42
gibistephenfin: I've sent all 3 patches to the gate14:43
stephenfingibi: Excellent. Thank you :)14:43
gibistephenfin: going back to reading your mypy example :)14:43
*** nicolasbock has joined #openstack-nova14:43
*** sidx64 has joined #openstack-nova14:44
*** yasemin has quit IRC14:44
*** gokhan_ has quit IRC14:45
*** suresh12 has joined #openstack-nova14:49
openstackgerritNguyen Hai proposed openstack/nova-specs master: Enhance nova-specs repo and webpage  https://review.openstack.org/55180214:49
*** alexchadin has joined #openstack-nova14:51
jaypipesstephenfin: will be 4 mins late to nmeetubg14:51
jaypipesmeeting14:51
*** alexchadin has quit IRC14:53
openstackgerritEd Leafe proposed openstack/nova master: Add 'member_of' param to GET /allocation_candidates  https://review.openstack.org/55209814:54
*** diga has joined #openstack-nova14:54
dansmithedleafe: oh that reminds me you pinged me yesterday during a meeting and I forgot14:55
*** suresh12 has quit IRC14:55
edleafedansmith: if your sqla-fu is strong, I need to find out why the added join isn't working14:55
edleafejaypipes ^^14:56
*** burt has quit IRC14:56
dansmithit's weak, but let me look over it14:56
*** idlemind has quit IRC14:56
edleafeSpeaking of meetings... off I go into that black hole..14:56
dansmithedleafe: I don't think you need to make this patch depend on any of mine14:56
dansmithit can go in straight away14:56
*** gongysh has quit IRC14:57
edleafedansmith: sure, I can switch that14:58
*** burt has joined #openstack-nova14:58
tssuryadansmith : mriedem gave his blessings on https://review.openstack.org/#/c/546684/ ; whenever you have the time could you please circle back for your re-approval ?14:58
dansmithtssurya: I saw, it's in my queue14:59
tssuryadansmith: thank you14:59
*** kholkina has quit IRC14:59
dansmithedleafe: yeah that join is over my head for sure15:00
*** salv-orlando has joined #openstack-nova15:00
*** gongysh has joined #openstack-nova15:01
*** gongysh has quit IRC15:02
edleafedansmith: I was trying to follow the "join chain" logic in the rest of the code, but sqla doesn't like the "in" clause, it seems15:03
dansmithedleafe: and member_of is a list of strings yeah?15:03
*** alexchadin has joined #openstack-nova15:04
edleafedansmith: yep15:04
dansmithedleafe: we should try query.summon(guru=jaypipes).execute()15:04
*** munimeha1 has joined #openstack-nova15:04
edleafedansmith: that reads like we're going to execute Jay15:05
*** gjayavelu has joined #openstack-nova15:05
dansmithheh15:05
*** salv-orlando has quit IRC15:05
*** Eran_Kuris has quit IRC15:05
*** sar has quit IRC15:07
*** zhaochao has quit IRC15:10
Spaz-HomeMan didn't deny it15:10
*** tbachman has joined #openstack-nova15:11
*** tbachman has quit IRC15:15
*** artom has joined #openstack-nova15:16
cdentedleafe: you getting an error or bad results on that join chain? have you debugged to look at the generated table? is the join between the rp_tbl and rp_agg_tbl happening somewhere else already or do you need to hork that in there somewhere too? looks like that may be missing15:16
*** mdnadeem has quit IRC15:16
edleafecdent: getting an error from sqla on the join clause itself.15:17
*** sticker has quit IRC15:17
cdentit looks like it is an orphan15:17
*** diga has quit IRC15:18
*** suresh12 has joined #openstack-nova15:18
edleafecdent: lemme paste the error15:19
cdent15:19
edleafecd http://paste.openstack.org/show/699991/15:21
edleafecdent: ^^15:22
* cdent looks15:22
*** antonym has quit IRC15:23
*** antonym has joined #openstack-nova15:24
cdentedleafe: the third argument to join is the onclause, and you've got a table15:25
*** david-lyle has joined #openstack-nova15:25
cdentso it trying to write a query but a table has no boolean characteristic15:26
cdenthttp://docs.sqlalchemy.org/en/rel_1_1/core/selectable.html?highlight=sa%20join#sqlalchemy.sql.expression.join15:26
*** jmlowe has quit IRC15:27
*** wolverineav has joined #openstack-nova15:27
*** jmlowe has joined #openstack-nova15:27
edleafecdent: ah, ok. I was trying to copy Jay's existing join chain logic15:28
*** yamamoto has joined #openstack-nova15:28
edleafelet me try breaking that up into two steps15:28
*** salv-orlando has joined #openstack-nova15:29
edleafe...after meetings are done, of course!15:29
cdentedleafe: I think basically you need more than one join15:30
edmondswmelwitt should I be doing anything to get eyes on the PowerVM virt driver spec for Rocky besides where I added to https://etherpad.openstack.org/p/rocky-nova-priorities-tracking?15:31
*** wolverineav has quit IRC15:32
*** tidwellr has quit IRC15:32
*** salv-orlando has quit IRC15:33
*** jaosorior has quit IRC15:34
*** jackie-truong has joined #openstack-nova15:34
mriedemsahid: let me know if you have any questions about my comments in https://review.openstack.org/#/c/485522/15:34
*** tidwellr has joined #openstack-nova15:36
*** beagles is now known as beagles_food15:36
*** yamamoto has quit IRC15:38
*** suresh12 has quit IRC15:39
jaypipesedleafe: on a call with stephenfin and gibi. gimme about 15 mins?15:40
*** wolverineav has joined #openstack-nova15:43
sahidmriedem: ack, will have a look and let you know15:43
edleafejaypipes: cdent may have pointed me in the right direction. Wait until my next revision15:44
*** sar has joined #openstack-nova15:45
*** itlinux has joined #openstack-nova15:45
*** harlowja has joined #openstack-nova15:46
*** yamahata_ has joined #openstack-nova15:47
sahidmriedem: in that meantime if i can ask you to have a look at this: https://review.openstack.org/#/c/539605/ i think we are all agree15:47
*** alexchadin has quit IRC15:47
mriedemhmm15:48
*** moshele has quit IRC15:48
*** jackie-truong has quit IRC15:50
sahiddoes that "hum" means you are not agree... :)15:50
*** tblakes has quit IRC15:50
*** gyee has joined #openstack-nova15:51
jaypipesstephenfin, gibi: https://etherpad.openstack.org/p/nfDwesfaMs15:53
*** yangyapeng has joined #openstack-nova15:53
*** links has quit IRC15:55
*** david-lyle has quit IRC15:57
*** dklyle has joined #openstack-nova15:57
*** lajoskatona has quit IRC15:58
*** tidwellr has quit IRC15:59
*** yangyapeng has quit IRC15:59
*** brault has quit IRC16:04
*** brault has joined #openstack-nova16:04
*** danpawlik has quit IRC16:05
*** harlowja has quit IRC16:06
*** moshele has joined #openstack-nova16:07
*** suresh12 has joined #openstack-nova16:07
openstackgerritMerged openstack/python-novaclient master: Updated from global requirements  https://review.openstack.org/55238416:07
*** links has joined #openstack-nova16:08
openstackgerritMerged openstack/nova master: Transform live_migration.post.dest notifications  https://review.openstack.org/46978416:10
mriedemsahid: comments inline16:11
*** moshele has quit IRC16:11
*** suresh12 has quit IRC16:12
*** wolverineav has quit IRC16:13
*** wolverineav has joined #openstack-nova16:13
mriedemsahid: i won't claim to know much about virtio-net multiqueue16:13
mriedemso my comments and questions are mostly about the big picture and for understanding16:14
*** gjayavelu has quit IRC16:15
dansmithmriedem: do you care to scrutinize the os-vif and networky bits of this? https://review.openstack.org/#/c/489451/3016:15
mriedemit's not using os-vif is it?16:16
mriedemalso https://blueprints.launchpad.net/nova/+spec/live-migration-in-xapi-pool isn't approved16:16
*** beagles_food is now known as beagles16:16
mriedemis this the thing to drop the aggregate up-call for xen live migration?16:17
*** ragiman has quit IRC16:17
*** wolverineav has quit IRC16:18
Spaz-HomeOh there's a blueprint for that16:19
Spaz-Home?16:19
Spaz-Homethat makes me happy if so16:19
*** sidx64 has quit IRC16:19
dansmithmriedem: yeah, we need to approve it specless I think, we discussed that at PTG16:19
Spaz-HomeI think i'd like to talk to Niachuan about that..16:20
dansmithmriedem: what I meant about os-vif is using your os-vif awareness to review all the networky bits of that16:20
*** udesale has quit IRC16:21
*** yangyapeng has joined #openstack-nova16:21
*** suresh12 has joined #openstack-nova16:21
mriedemdansmith: comments inline16:21
mriedemthat patch doesn't remove the aggregate up-call at all16:21
mriedemalso, i don't have os-vif awareness :)16:21
dansmithI know, see my comment16:21
mriedemexcept to say i'm aware it exists16:22
dansmiththis patch allows live migration through a different pool mechanism, other than aggregates16:22
*** gbarros has quit IRC16:22
dansmithwhich we discussed on friday at PTG,16:22
dansmithis a precursor to removing the aggregate stuff16:22
Spaz-HomeI'd honestly like there to be a switch in there to allow for neither of those to be used.16:22
Spaz-HomeFor example just using Cells and not HA pools16:23
dansmithum16:23
dansmithcells has nothing to do with this, but maybe you mean "just assume you can live migrate anywhere" ?16:23
openstackgerritStephen Finucane proposed openstack/osc-placement master: Resolve nits from I552688b9ee32b719a576a7a9ed5e4d5aa31d7b3f  https://review.openstack.org/53797116:23
Spaz-HomeI mean that don't assume that it needs to be aggregated to live migrate.  Cells does have to do with it when it can use Cells to know what to migrate to.16:23
Spaz-Homeor pooled as well really16:24
dansmiththis change has an "assume on the same network" flag, which I would assume is that?16:24
dansmithbut still, I don't see how "it can use cells to know"16:24
mriedemso in other words, this is routed networks for xenapi live migration?16:25
dansmiththe xen driver has always had a mechanism to determine if two hosts can live migrate between each other (aggregates),16:25
dansmiththis is using a different mechanism for that now,16:25
Spaz-HomeIt's also been pretty inconsistent.  block-migration skips it completely16:25
*** yangyapeng has quit IRC16:25
*** suresh12 has quit IRC16:26
dansmithright16:26
Spaz-HomeBut yeah I can agree.. so long as it's consistent16:26
*** gjayavelu has joined #openstack-nova16:27
openstackgerritMerged openstack/nova master: Updated from global requirements  https://review.openstack.org/55236316:27
*** salv-orlando has joined #openstack-nova16:28
dansmithSpaz-Home: are you going to comment on that patch or what? I can't tell if you have feedback or concerns16:29
Spaz-HomeIt may be worth saving that for a separate patch though to bring block-migration into proper parity as it is not the only issue that it is missing.16:29
Spaz-HomeStill just reading it and thinking, sir.16:29
melwittjichen: we're tracking them on the priorities etherpad at L95. I've added your patches there https://etherpad.openstack.org/p/rocky-nova-priorities-tracking16:29
*** links has quit IRC16:30
Spaz-HomeJust figured i'd talk it out with people as I assume the nova development  IRC room was for that..16:30
Spaz-HomeBut if not forgive me.16:30
mriedemthis room is only for kitten jokes16:31
dansmithSpaz-Home: okay I'm just trying to make sure you comment on there if you have opinions one way or the other16:31
Spaz-HomeRight now I don't, just thinking about it.. but the more I think it's the same behavior it exhibits now just in a different way, and I cannot oppose that on concept alone16:32
mriedemSpaz-Home: in other words, xenapi users / operators are actually pretty far and few between, so people that actually know how it works and can provide feedback on the review is very helpful16:32
Spaz-HomeI think maybe a follow-up commit to bring -block-migration into parity is a better method.16:32
mriedemSpaz-Home: b/c i have no authority on xenapi virt driver knowledge16:32
Spaz-HomeAnd the cells thing is more my jenky environment running that i'm looking to preserve, so I will drop that argument for it :D16:32
jaypipessean-k-mooney: "deleting all the functional tests is not a solution to getting them running". <-- classic.16:33
melwittedmondsw: I think that should be sufficient. I'm going to be sending periodic reminders to the dev ML and mention in nova meetings to help people remember to look at the priorities etherpad16:33
Spaz-HomeI'll talk with the citrix folks on it tonight though and comment on it a bit later after some rest.16:34
dansmithSpaz-Home: can you make sure someone shows up to the nova meeting to make sure the bp gets approved?16:34
dansmithI'll add it to the agenda now before I forget16:34
Spaz-HomeYeah i'll ping them all tonight to remind them.16:35
Spaz-HomeI work their hours so we see each other more than you guys do I Think16:35
*** gbarros has joined #openstack-nova16:35
*** andreas_s_ has joined #openstack-nova16:36
*** andreas_s has quit IRC16:38
*** yangyapeng has joined #openstack-nova16:39
Spaz-Homemelwitt, if you got my message while you were on vacation, I finished that script to pull metrics.  If the numbers are something you're interested in let me know and i'll send them your way.16:40
Spaz-HomeNeed to pull them for the previous 10 releases or so tonight,  Will send you over a CSV.16:40
melwittSpaz-Home: not yet, sorry. I haven't yet gone through all the pings yet16:42
*** r-daneel has quit IRC16:42
*** yangyapeng has quit IRC16:44
Spaz-HomeNo worries at all, just figured i'd send a follow up for ya :)16:44
Spaz-HomeAbsolutely bottom priority16:44
*** fragatina has quit IRC16:44
*** andreas_s_ has quit IRC16:45
edmondswmelwitt thanks16:45
melwittefried: on the runways thing, it will be at least linked to from the priorities etherpad. this week I'm writing up an etherpad on how I think it will work and I'll send it to the dev ML for people to add their input, then we'll finalize how it will work and whether it gets its own etherpad16:46
*** gouthamr has quit IRC16:47
efriedmelwitt: Cool beans, thanks.  I just didn't know if it was already "launched" and I just didn't know where.16:47
*** eharney_ has joined #openstack-nova16:47
melwittefried: ah, okay. sorry bout that, I was on PTO last week so this week I'm doing the things like writing up PTG summaries for the non-placement stuff and writing up something for runways, etc16:49
efriedNo worries.  I also have been on PTO.  Enjoy catching up :P16:49
*** andreas_s has joined #openstack-nova16:49
melwitt:)16:50
Spaz-HomeGood to see the nova channel back in full gear though :)16:50
*** vks1 has quit IRC16:51
*** eharney has quit IRC16:51
*** karimull has quit IRC16:55
*** pchavva has quit IRC16:55
*** wolverineav has joined #openstack-nova16:57
*** karimull has joined #openstack-nova16:58
*** eharney_ has quit IRC16:58
*** andreas_s has quit IRC16:59
*** wolverineav has quit IRC17:00
*** wolverineav has joined #openstack-nova17:00
*** r-daneel has joined #openstack-nova17:01
openstackgerritEd Leafe proposed openstack/nova master: Add 'member_of' param to GET /allocation_candidates  https://review.openstack.org/55209817:03
*** eharney_ has joined #openstack-nova17:03
*** eharney_ is now known as eharney17:03
edleafejaypipes: dansmith: cdent: ^^ Tests should be working now17:03
openstackgerritMerged openstack/nova-specs master: Support disabling a cell  https://review.openstack.org/54668417:03
edleafeStill need to add a reno17:03
dansmithsweet17:03
mdboothkashyap: stephenfin The tests that I care about passed: https://review.openstack.org/#/c/551302/17:04
mdboothMind taking another look? Now with libvirt driver changes :/17:05
*** gjayavelu has quit IRC17:05
*** yangyapeng has joined #openstack-nova17:05
*** Swami has joined #openstack-nova17:06
*** fragatina has joined #openstack-nova17:06
*** gouthamr has joined #openstack-nova17:06
*** pchavva has joined #openstack-nova17:07
*** pchavva has quit IRC17:07
*** andreas_s has joined #openstack-nova17:08
*** jackie-truong has joined #openstack-nova17:09
bauzasman, that NUMA spec makes me mad17:09
*** lucasagomes is now known as lucas-afk17:09
*** yangyapeng has quit IRC17:10
cdentedleafe: yeah that double join makes far more sense17:10
cdentotherwise there's no attachment to the rp table17:11
*** pchavva has joined #openstack-nova17:11
openstackgerritMatt Riedemann proposed openstack/nova master: Make nova-cells-v1 run with neutron  https://review.openstack.org/54978917:18
*** dtantsur is now known as dtantsur|afk17:18
*** damien_r has quit IRC17:23
*** yangyapeng has joined #openstack-nova17:24
bauzasstephenfin: sahid: others: do you know if Nova supports ccNUMA ?17:24
stephenfinbauzas: ccNUMA?17:26
dansmithedleafe: do we need a reno for a placement api change?17:26
bauzasI mean, my question is: do we have NUMA nodes having different cache numbers for CPUs ?17:26
stephenfinbauzas: Also, which spec is upsetting you? :)17:26
*** RaoulHC has quit IRC17:26
stephenfinbauzas: I hadn't heard of it before so I would assume not17:26
bauzasstephenfin: https://en.wikipedia.org/wiki/Non-uniform_memory_access#/media/File:Hwloc.png17:26
openstackgerritMerged openstack/nova-specs master: Allow for merging traits and aggregates  https://review.openstack.org/55212217:27
bauzasstephenfin: ccNUMA means that you can have multiple cache levels17:27
*** andreas_s has quit IRC17:27
bauzasI mean, for the same NUMA node17:27
stephenfinbauzas: Oh, that sounds a bit like the cluster-on-die thing sean-k-mooney has been on about for years17:27
bauzasno worries17:28
*** yangyapeng has quit IRC17:28
stephenfinWe just use whatever libvirt is reporting to us. Based on that hwloc output, we'd see four NUMA nodes/sockets for that system17:28
bauzasso, the only resouces per NUMA node are just memory and devices, right?17:28
stephenfinHugepages and PCI devices, yup17:28
bauzasI mean, the ones we know17:28
stephenfinBe they PCI passthrough or SR-IOV17:29
bauzask17:29
stephenfinI'm trying to add PCI devices used by vSwitches and I guess vGPU will use that at some point too17:29
*** felipemonteiro_ has joined #openstack-nova17:29
bauzasstephenfin: yeah, hence why I'm writing a spec17:30
bauzasfor NUMA affinity for vGPUs17:30
bauzasat least, that spec is just about providing a RP tree for NUMA nodes17:30
stephenfinGotcha. Stick me on that once you've something drafted. Would be interested in seeing what it looks like17:30
*** suresh12 has joined #openstack-nova17:30
bauzasif we have PCI devices attached to a specific NUMA node, we could imagine those being nested RPs17:30
stephenfinYup, that's the plan17:31
stephenfina cycle or two from now though17:31
*** felipemonteiro has quit IRC17:31
cdentjaypipes: if you can put your +2 back on here https://review.openstack.org/#/c/540049/ then 3 things can merge and there will much rejoicing17:31
stephenfinat least, afaik17:31
bauzasbut the memory would just be a resource class17:31
bauzaseven for hugepages17:31
stephenfinNot nested under NUMA nodes?17:31
bauzasthat was my question about L2 caches17:31
bauzasyeah17:31
stephenfinHmm, that seems...odd17:31
bauzaseach NUMA node has some memory, right?17:32
stephenfinThen again, so does two views into what are essentially the same thing17:32
stephenfinThey should17:32
*** andreas_s has joined #openstack-nova17:32
*** Eran_Kuris has joined #openstack-nova17:32
bauzasokay, lemme draft the spec then17:32
stephenfinThough I can't remember what happens if you leave some sockets on the motherboard empty17:32
bauzasbecause IMHO, unless some cache memory isn't shared across the NUMA node, I see that as just a resource class for the NUMA node RP17:33
stephenfinYup17:33
bauzaslike CPUs17:33
bauzasie. I have a 2-sockets with 4 cores each, HT17:33
stephenfinBut remember, hugepages are essentially normal memory17:33
stephenfinYou can have 4k huge pages17:33
*** yangyapeng has joined #openstack-nova17:33
stephenfin*"huge pages"17:34
stephenfinWe currently have open bugs around not tracking that stuff correctly17:34
bauzasin that case, I'd have 2 NUMA nodes with 8 CPUs each17:34
stephenfinYup17:34
*** AlexeyAbashkin has quit IRC17:34
openstackgerritMerged openstack/nova master: crypto: Remove unused functions  https://review.openstack.org/55077217:34
bauzasie. 2 child RPs, with each of those having some memory and a VCPU RC of 817:34
*** gjayavelu has joined #openstack-nova17:34
bauzasok, I'm going to model this17:35
bauzasstephenfin: hugepages sound a different resource class, right?17:35
*** suresh12 has quit IRC17:36
bauzasstephenfin: https://blueprints.launchpad.net/nova/+spec/numa-topology-with-rps is the boilerplate17:36
*** suresh12 has joined #openstack-nova17:36
stephenfinbauzas: Yup, but I don't know if we have any way to identify 4k "huge pages" from normal, non huge-page memory17:36
stephenfinI need to root out those bugs to get the full context again17:37
bauzasright17:37
melwittmriedem: on https://blueprints.launchpad.net/nova/+spec/add-server-group-remove-member-notifications, I'm not sure we want to notify about server group member removal upon overquota. the "remove server group member" is not a public API and the overquota in this case would be the one that occurs during a quota recheck, and I didn't think we'd want to expose that to an end user17:37
*** yangyapeng has quit IRC17:37
*** r-daneel has quit IRC17:38
*** suresh12 has quit IRC17:38
*** suresh12 has joined #openstack-nova17:38
*** jpena is now known as jpena|off17:38
stephenfinbauzas: Enjoy drafting that. I'm off for a run o/17:38
melwittmriedem: guess I'll comment on the bp whiteboard17:38
openstackgerritMerged openstack/nova master: ca: Remove 'nova/CA' directory  https://review.openstack.org/55077317:39
openstackgerritMerged openstack/nova master: conf: Remove 'nova.crypto' opts  https://review.openstack.org/55077417:40
jaypipescdent: done17:43
*** mvk has quit IRC17:43
lyarwoodstephenfin: https://docs.openstack.org/nova/latest/configuration/config.html - under passthrough_whitelist - are the quotes all rendering as unicode left “ and right ” quotation marks for you? Have a guess what this does if you copy things into nova.conf ^_^17:43
*** imacdonn has quit IRC17:44
*** imacdonn has joined #openstack-nova17:44
*** ccamacho has quit IRC17:44
efriedI have never found smart quotes to be as advertised.17:45
cdentthanks jaypipes17:45
*** _pewp_ has quit IRC17:45
*** andreas_s has quit IRC17:46
openstackgerritElod Illes proposed openstack/nova master: WIP: Transform scheduler.select_destinations notification  https://review.openstack.org/50850617:46
*** yangyapeng has joined #openstack-nova17:46
*** yangyapeng has quit IRC17:51
*** danpawlik has joined #openstack-nova17:51
*** Eran_Kuris has quit IRC17:51
openstackgerritSurya Seetharaman proposed openstack/nova master: Make archive_deleted_rows handle a missing CONF.api_database.connection  https://review.openstack.org/55096717:54
*** fragatina has quit IRC17:57
*** fragatina has joined #openstack-nova17:57
*** _pewp_ has joined #openstack-nova17:58
*** sahid has quit IRC17:59
jaypipesdansmith: not sure I follow you with "dynamic section like cinder does for backends". could you clue me in on that with a link to what you're referring to?17:59
dansmithjaypipes: do you remember the discussion we had on thursday in the lunch room?18:00
*** wolverineav has quit IRC18:01
dansmithjaypipes: cinder has a thing where you define backend names, and then it assumes you have created a section in config named "backend_%(name)s", with keys in it18:01
jaypipesdansmith: I don't believe I was there, no. (I don't remember sharing lunch with you on any of the days)18:01
*** derekh has quit IRC18:01
dansmithwhich is a way to encode dicts in ini format in less nasty form18:01
*** mgoddard_ has quit IRC18:01
dansmithjaypipes: I was speaking directly to you, you were there :)18:01
*** wolverineav has joined #openstack-nova18:01
jaypipesdansmith: which soup was it? :)18:01
cdentthat was the twin18:01
dansmiththis was when we relocated to the lunch room of the hotel on thursday afternoon for the neutron discussion18:02
dansmithjaypipes: no soup, it was not over lunch18:02
*** sambetts is now known as sambetts|afk18:02
*** wolverineav has quit IRC18:02
*** wolverineav has joined #openstack-nova18:02
jaypipesk. gibi had indeed brought up the option of doing something like physnet_mapping_%(physnetname)s instead of a single config option. I wasn't keen on that.18:03
*** gjayavelu has quit IRC18:03
jaypipesdansmith: sorry, I just don't remember that conversation :(18:03
dansmithI _hate_ encoding json in our config files18:04
dansmithI think it's a pain we should not inflict on humans18:04
dansmiththere's prior art for this in at least cinder18:04
*** gbarros has quit IRC18:04
dansmithwe previously broke operators with the pci whitelist when we accidentally changed the format from a list to a dict, because it was json and thus obscure that anything was changing18:05
dansmiththat caused people a lot of pain, on top of the pain required to view/edit it in general18:05
jaypipesdansmith: so we'd have multiple options like this? physnet_mapping_physnet0_name = "tenant_data_0"; physnet_mapping_physnet0_numa_nodes = 0,1;18:05
jaypipesdansmith: personally I find encoded that to be less nice than just putting it in a JSON blob.18:05
*** felipemonteiro_ has quit IRC18:05
*** yangyapeng has joined #openstack-nova18:06
dansmithno18:06
mriedemmelwitt: notifications are on the rpc queue, so the user wouldn't see them...18:06
dansmithjaypipes: https://pastebin.com/zP7eyAZw18:07
jaypipesdansmith: mind etherpadding up your proposed conf option format?18:07
jaypipeslol18:07
jaypipesjinx18:07
mriedemunless you mean user = admin/operator looking at notifications; as for the actual usefulness of that specific case, i don't think it's really useful either since it's normal during a quota recheck18:07
jaypipesalways ten steps ahead of me :)18:07
mriedemi.e. we don't send a versioned notification when server create fails due to overquota on ram/cpu18:07
jaypipesdansmith: heh, ok. sure, I can go for that.18:07
jaypipesdansmith: excellent use of the words "bloopy" and "poopy"18:07
*** amodi has quit IRC18:08
dansmithum, excuse me.18:08
dansmithit's "boopy"18:08
jaypipesoh, my mistake. yes, boopy.18:08
mriedemlyarwood: i don't see the problem, clear as day https://docs.openstack.org/nova/latest/configuration/config.html#pci.passthrough_whitelist18:09
jaypipesdansmith: k. added a link on the spec and mentioned I could go for that.18:09
dansmithack18:09
*** yangyapeng has quit IRC18:10
*** _pewp_ has quit IRC18:12
*** _pewp_ has joined #openstack-nova18:12
*** jackie-truong has quit IRC18:14
lyarwoodmriedem: so for me that renders with unicode quotes that when copied into nova.conf makes n-cpu bork18:14
*** andreas_s has joined #openstack-nova18:14
mriedemlyarwood: i was joking, of course it does18:16
*** r-daneel has joined #openstack-nova18:16
lyarwoodwunderbar18:17
mriedemhttps://github.com/openstack/nova/blob/master/nova/conf/pci.py#L66 all looks pretty normal to me though18:17
edleafedansmith: I assume that all microversion changes, placement or not, require a reno18:19
*** yamahata_ has quit IRC18:19
dansmithedleafe: okay I thought just a spec, but I don't really review that many api changes18:19
*** pchavva has quit IRC18:19
mriedemarvindn05: comments on https://review.openstack.org/#/c/541507/18:23
mriedemarvindn05: i don't think we can simply pass through "trait:*" to the ImageMetaProps object,18:23
mriedemif you haven't yet noticed, the fields in that object are strictly defined18:23
openstackgerritEric Fried proposed openstack/nova master: placement: generation in provider aggregate APIs  https://review.openstack.org/54824918:24
openstackgerritEric Fried proposed openstack/nova master: placement: Return new provider from POST /rps  https://review.openstack.org/54893418:24
openstackgerritEric Fried proposed openstack/nova master: Stop assuming initial provider generation is 0  https://review.openstack.org/54897518:24
mriedemtherefore "trait:CUSTOM_FOOBARS" is going to explode if you try to hydrate it into that object18:24
mriedemso traits in flavors is != traits in image properties because of the wild west vs strict nature of the two18:24
*** r-daneel_ has joined #openstack-nova18:25
*** jaosorior has joined #openstack-nova18:25
*** r-daneel has quit IRC18:26
*** r-daneel_ is now known as r-daneel18:26
*** yangyapeng has joined #openstack-nova18:27
*** amodi has joined #openstack-nova18:27
mriedemedleafe: this isn't pointing at the correct blueprint https://review.openstack.org/#/c/552098/18:27
*** andreas_s has quit IRC18:28
*** mvk has joined #openstack-nova18:28
*** harlowja has joined #openstack-nova18:30
*** harlowja has quit IRC18:30
*** ralonsoh has quit IRC18:30
openstackgerritSurya Seetharaman proposed openstack/nova master: Marker reset option for nova-manage map_instances  https://review.openstack.org/53950118:31
*** yangyapeng has quit IRC18:31
*** andreas_s has joined #openstack-nova18:33
*** gouthamr has quit IRC18:33
*** jackie-truong has joined #openstack-nova18:34
dansmithmriedem: it's not?18:34
dansmithoh, I see18:35
dansmithit's for mine, but we did a different spec for it because api18:35
*** wolverineav has quit IRC18:36
*** wolverineav has joined #openstack-nova18:37
edleafedansmith: I'm confused. I thought this was on the same BP as yours18:39
mriedemit's a dependency18:39
dansmithedleafe: we did a spec for it specifically because of the api change, which is technically a different blueprint in launchpad18:39
dansmithedleafe: but I know, it seemed right to me as well18:39
*** danpawlik has quit IRC18:40
*** wolverineav has quit IRC18:41
edleafedansmith: ok, found it18:42
openstackgerritEd Leafe proposed openstack/nova master: Add 'member_of' param to GET /allocation_candidates  https://review.openstack.org/55209818:42
edleafepushed a new rev, with correct BP and shiny new reno18:42
*** andreas_s has quit IRC18:42
*** tssurya has quit IRC18:42
*** wolverineav has joined #openstack-nova18:45
*** wolverineav has quit IRC18:50
*** wolverineav has joined #openstack-nova18:51
*** felipemonteiro has joined #openstack-nova18:51
*** andreas_s has joined #openstack-nova18:52
*** yamahata_ has joined #openstack-nova18:53
melwittmriedem: yeah, I was thinking the quota recheck stuff is supposed to be transparent, though the more I thought about it I realized I think some behavior changed, for example, with multi-create, where overquota during recheck results in ERROR instances where it didn't before. but agreed we don't send server delete notifications for a server create fail for overquota during a recheck18:54
*** gjayavelu has joined #openstack-nova18:55
*** felipemonteiro_ has joined #openstack-nova18:55
*** wolverineav has quit IRC18:55
mriedemmelwitt: "with multi-create, where overquota during recheck results in ERROR instances where it didn't before" - Kevin_Zheng brought that up in denver at the queens ptg18:55
mriedemand was mostly written off18:55
mriedemjust reno'ed for now https://docs.openstack.org/releasenotes/nova/pike.html#known-issues18:56
melwittyeah, I remember. I thought it was worth handling (to not change the behavior) but I think I was not in the majority opinion about it18:56
*** amoralej is now known as amoralej|off18:56
melwittI was aware of putting the instances in ERROR state but for whatever reason at the time, I didn't realize that meant things used to fail in the API before18:57
mriedemwe fail either way,18:57
mriedemthe difference is before you didn't have 50 instances in ERROR state that you had to manually clean up18:58
*** r-daneel_ has joined #openstack-nova18:58
*** r-daneel has quit IRC18:58
*** r-daneel_ is now known as r-daneel18:58
*** AlexeyAbashkin has joined #openstack-nova18:58
*** felipemonteiro has quit IRC18:59
*** itlinux has quit IRC18:59
*** eharney has quit IRC19:00
melwittyeah, that's what I meant, fail in the API without creating instances. and we can't just delete them instead of putting them into ERROR state because we have no other way to signal the create failed after the API returned a 20219:00
*** wolverineav has joined #openstack-nova19:01
*** wolverineav has quit IRC19:01
*** wolverineav has joined #openstack-nova19:01
melwittso then we were talking about counting build requests along with instances with de-duping I think. I think it would be worth it to avoid manual cleanup for users. but I think the other argument was that operators could turn off quota recheck19:01
mriedemthat and it would look janky if you're polling and watching the number of instances returned increase, then start automatically decreasing19:01
mriedemi don't know who is going to ever turn off that quota recheck flag19:02
mriedembut the build request count could also be based on that flag i guess19:02
melwittyeah19:02
melwitttrue19:02
*** AlexeyAbashkin has quit IRC19:03
*** itlinux has joined #openstack-nova19:04
*** mgoddard_ has joined #openstack-nova19:04
*** eharney has joined #openstack-nova19:05
*** andreas_s has quit IRC19:06
openstackgerritChris Dent proposed openstack/nova master: Use nova.db.api directly  https://review.openstack.org/54326219:10
*** andreas_s has joined #openstack-nova19:11
*** tesseract has quit IRC19:11
*** moshele has joined #openstack-nova19:11
*** jackie-truong has quit IRC19:11
openstackgerritMerged openstack/nova master: Move resource class fields  https://review.openstack.org/54004919:13
openstackgerritMerged openstack/nova master: Move resource provider objects into placement hierarchy  https://review.openstack.org/55152819:14
*** sridharg has quit IRC19:15
*** danpawlik has joined #openstack-nova19:16
cfriesenmaybe a weird question, but is the metadata API accessible from all instances within the guest?  Or would some like PCI VF/PF not necessarily support it?   Just wondering how you're supposed to access device tagging if you don't know which NICs are which.  Is a config drive recommended in this case?19:18
cfriesens/instances/interfaces19:18
dansmithcfriesen: yeah, metadata access is generally handled by overriding routing on the host, or in the neutron netns19:20
dansmithcfriesen: assuming conventional networking to cloud instances19:20
dansmithif you have only direct network access,19:20
dansmiththen it's going to be hard19:20
dansmithconfigdrive being a better option for that (and many cases)19:20
*** suresh12 has quit IRC19:23
*** suresh12 has joined #openstack-nova19:23
artomKeep in mind: config drive is write-once. Tagged (at|de)tachments are not reflected.19:24
*** andreas_s has quit IRC19:25
artomHrmm, that reminds me...19:25
*** gbarros has joined #openstack-nova19:26
artomDidn't we have a conversation at some point about making the metadata API easier to consume?19:26
* artom goes digging.19:26
*** damien_r has joined #openstack-nova19:26
*** esberglu has quit IRC19:27
*** suresh12 has quit IRC19:28
artomAhh, virtio-sock: http://lists.openstack.org/pipermail/openstack-dev/2017-February/112602.html19:29
openstackgerritDan Smith proposed openstack/nova master: Add AggregateList.get_by_metadata() query method  https://review.openstack.org/54472819:29
openstackgerritDan Smith proposed openstack/nova master: Add aggregates list to Destination object  https://review.openstack.org/54472919:29
openstackgerritDan Smith proposed openstack/nova master: Add request filter functionality to scheduler  https://review.openstack.org/54473019:29
openstackgerritDan Smith proposed openstack/nova master: Make get_allocation_candidates() honor aggregate restrictions  https://review.openstack.org/54799019:29
openstackgerritDan Smith proposed openstack/nova master: Add require_tenant_aggregate request filter  https://review.openstack.org/54500219:29
openstackgerritDan Smith proposed openstack/nova master: WIP: Honor availability_zone hint via placement  https://review.openstack.org/54628219:29
dansmithedleafe: rebased on top of your patch ^19:29
artomWonder how evil it would be support the metadata API with virtio-sock in libvirt only...19:30
*** andreas_s has joined #openstack-nova19:30
*** suresh12 has joined #openstack-nova19:31
*** danpawlik has quit IRC19:32
*** gouthamr has joined #openstack-nova19:33
*** andreas_s has quit IRC19:34
dansmithartom: it would be fully contained in the libvirt driver I think.. not sure that'd be a problem19:34
artomdansmith, evil in the sense that we'd be leaving other virt drivers in a "we don't support the metadata API as well as libvirt does" state19:35
dansmithI dunno how addressing works on vsock, and if you'd have to come up with a well-known address or port for it, but other than that it'd just be a proxy to the http interface19:35
artomThere'd also be the question of do we let the user decide whether they want a vsock or not19:36
dansmithif it's a thing cloud-init looks for and supports, then it'd try that, like it tries http, and like it looks for a configdrive19:36
artomI should really be thinking about other stuff, but this is more fun :/19:36
dansmithno, I'd say you just expose it if libvirt can do it, maybe optional via config,19:36
dansmithbut don't make it a user thing19:36
*** esberglu has joined #openstack-nova19:37
dansmithI'm not sure it's a huge problem in need of a solution yeah :)19:37
dansmithworst case, give them one vif on a private network that can get to metadata :)19:37
dansmithah, ports are strings, so "org.openstack.nova.metadata-api" or something19:38
artomWell, as we use the metadata for more stuff (what did cfriesen want to use it for at the PTG? Something to do with pinning?), the fact that operators don't like deploying it (which I gather is a thing) is becoming a problem19:38
*** damien_r has quit IRC19:38
cfriesenYeah, I use it for CPU scaling since our qemu is too old for proper CPU hotplug...the metadata tells the guest at guest kernel boot time which vCPUs should be "offline".19:39
cfriesenBut we also have a terrible terrible feature that lets you specify the desired virtual PCI address for your virtual NICs.  This was added for a customer that had legacy apps that had hardcoded PCI addresses for things.19:40
artomcfriesen, hah, we had a customer that wanted to do that19:41
artomI *think* we convinced them that device tagging was the way to go19:41
cfriesenyeah...but until Pike you couldn't attach/detach tagged devices19:41
*** moshele has quit IRC19:42
artomIndeed, and PFs still can't be tagged, it turns out19:42
artom*nudge nudge* https://review.openstack.org/#/c/533805/ ;)19:42
cfriesenhmm, that's kind of lame19:42
cfriesenI'll see if I can get our PCI guy to look at it.19:43
* artom is now completely distracted from what he was originally doing19:43
artomcfriesen, thanks :)19:43
*** mgoddard_ has quit IRC19:47
cdentsorry, not sorry, I've given lots of people a merge conflict19:58
mikal_artom: I don't have a particular problem with the vsock thing if it "tastes like" normal network access inside the instance.20:00
mikal_artom: in fact, it would be nice for such a thing to "sign" the request so we strongly know what instance made it.20:00
mikal_artom: ala AWS20:00
artommikal_, hrmm, that's actually a good question - how does the host know what instance a request came from over vsock?20:01
mikal_artom: its not one vsock per instance?20:02
*** mikal_ is now known as mikal20:02
*** moshele has joined #openstack-nova20:02
artommikal, it is, but...20:02
artomThis is where what little knowledge I have of this thing breaks down20:02
mikalSo nova knows when it sets the vsock the instance its going to wire it into...20:02
artomBut if there's only one service listening on the host side, even if there's one vsock per guest...20:03
*** suresh12 has quit IRC20:03
*** suresh12 has joined #openstack-nova20:03
artomPresumably they speak TCP over that vsock, so unless every instance has its own "vsock IP" that nova knows about...20:04
artomIf berrange was here he'd probably laugh at my ignorance :/20:04
*** suresh12 has quit IRC20:07
*** suresh12 has joined #openstack-nova20:08
*** tssurya has joined #openstack-nova20:09
dansmithvsock works like AF_INET or AF_UNIX20:09
dansmithit's a socket protocol20:09
dansmithso no TCP or IP20:09
dansmithit's 1:1 and uses strings as the addressing on either end20:09
dansmithso it wouldn't "taste like normal networking" in that sense.. cloud-init would have to open an AF_VSOCK socket to some known port name and then try to speak HTTP over it or something,20:10
dansmithwhich something on the host could proxy to the regular metadata API, or just emulate the metadata server20:10
dansmithmikal: artom ^20:11
artomdansmith, heh, dunno if that was off the top of your head or you did the research, but thanks regardless :)20:11
cfriesenartom: https://wiki.qemu.org/Features/VirtioVsock20:12
* artom will try to propose that next cycle I guess? And maybe properly version the metadata API while we're at it :D20:12
*** moshele has quit IRC20:12
dansmithartom: I had to refresh my memory on the port bit, but yeah, the rest is pretty straightforward20:15
*** tssurya has quit IRC20:15
*** tssurya has joined #openstack-nova20:15
dansmithit's been discussed a couple times at least20:15
mikaldansmith: it doesn't feel hard to get changes into cloud-init, but it would take a long time to propogate20:15
dansmithmikal: yep, and we kinda have to do it first20:15
dansmithmikal: I think it'd just be one of those "use it if it's there" sort of things anyway, so it wouldn't be too bad20:15
mikalI've had a long term dream of getting something a bit like AWS IAMS into nova, but I don't think this is it on reflection20:15
mikaldansmith: yeah, part of the discovery process at boot20:16
dansmithbut someone would have to write up both sides and think through a bunch of it20:16
mikalIAMS is more like a transparent http proxy embedded into neutron20:16
mikalSo what is the advantage of vsock over the current ways? It would need to be large to be bothered making the change.20:16
dansmithit has no addressing, and requires no networking20:17
dansmithit's an agent transport basically,20:17
dansmithbut done in a generic POSIX-compatible way20:17
artomSecurity is a big advantage, I'd htink20:18
artomYou're not sending potentially sensitive metadata over potentially unsecure proxies20:19
dansmithI don't know20:19
*** mgoddard_ has joined #openstack-nova20:19
artomIn a potentially sniffable by other guests way20:19
dansmithit feels more secure but I don't think it's substantially so20:19
* artom needs to fetch kids and relocate20:19
dansmithno, I don't think this is less sniffable20:19
dansmithless prone to bugs which might leak sniffability, maybe :)20:20
mikalAnd instances need networking to be useful, so I guess we're talking about a before-networking-configured case here20:21
mikalThe benefits still feel marginal compared to the amount of mucking around to get them20:21
dansmithwell, this came up mostly because you need to know some things to plumb yourself for certain types of networking20:21
dansmithchicken and egg like20:21
mikalconfigdrive!20:22
dansmithyeah, but you can't update that at runtime20:22
mikal(Which cracks me up because configdrive was meant to be long dead by now)20:22
dansmithwhich is why we landed here :)20:22
mikalSo for these network types, you need to have attached the _first_ interface before the config drive is generated?20:22
mikalBut the later ones you could use metadata server for?20:23
dansmithif you boot with two SRIOV nics and you need to know which one is public, and you can't get to metadata over them (yet), then you could use configdrive,20:23
dansmithbut then if you attach something else later,20:23
dansmithyou'd need to know that even though at boot there was no metadata server, now there is and you should use that instead20:24
*** artom has quit IRC20:24
dansmithit's doable of course20:24
openstackgerritEd Leafe proposed openstack/nova master: Add 'member_of' param to GET /allocation_candidates  https://review.openstack.org/55209820:26
*** yangyapeng has joined #openstack-nova20:30
*** yangyapeng has quit IRC20:35
*** harlowja has joined #openstack-nova20:37
*** cdent has quit IRC20:44
*** sidx64 has joined #openstack-nova20:52
*** pchavva has joined #openstack-nova20:54
*** brault has quit IRC20:56
*** mingyu has quit IRC20:58
*** mingyu has joined #openstack-nova21:00
*** mgoddard_ has quit IRC21:03
cfriesenjust noticed something odd...the docs at https://docs.openstack.org/nova/pike/configuration/config.html specify that ram_weight_multiplier/disk_weight_multiplier/etc are under DEFAULT, but that generates a warning log since they're actually under "filter_scheduler" as far as I can tell.21:07
*** burt has quit IRC21:07
tssuryadansmith : do we still plan to keep this NewtonCellsCheck Test in Rocky ? -> https://github.com/openstack/nova/blob/master/nova/tests/unit/db/test_sqlalchemy_migration.py#L356 ; since it uses a particularly older DBVersion it fails for my new DB version; not sure how to handle it because it is wrong to just update it to the new version I guess.21:07
mriedemcfriesen: https://docs.openstack.org/nova/latest/configuration/config.html says the are under the [cells] section21:09
mriedemoh heh there are two options with the same name21:11
mriedemnice21:11
mriedemcfriesen: it says it's in the [filter_scheduler] group https://docs.openstack.org/nova/latest/configuration/config.html#filter-scheduler21:12
mriedemthe 'group' you're looking at is the 'Deprecated Variations¶'21:12
mriedemb/c deprecated_group="DEFAULT",21:13
*** mingyu has quit IRC21:13
mriedemwe can probably start cleaning up all of those aliases21:13
mriedemthey were made in 201621:13
*** mingyu has joined #openstack-nova21:13
openstackgerritMatt Riedemann proposed openstack/nova master: Add check if neutron "binding-extended" extension is available  https://review.openstack.org/52354821:16
openstackgerritMatt Riedemann proposed openstack/nova master: Add code to bind a port against a dest host during live migration  https://review.openstack.org/52360421:16
openstackgerritMatt Riedemann proposed openstack/nova master: Add VIFMigrateData object for live migration  https://review.openstack.org/51542321:16
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: libvirt: use dest host vif migrate details for live migration  https://review.openstack.org/55137021:16
openstackgerritMatt Riedemann proposed openstack/nova master: Port binding based on events during live migration  https://review.openstack.org/43487021:16
openstackgerritMatt Riedemann proposed openstack/nova master: Add "delete_port_binding" network API method  https://review.openstack.org/55217021:16
openstackgerritMatt Riedemann proposed openstack/nova master: compute: use port binding extended API during live migration  https://review.openstack.org/55137121:16
openstackgerritMatt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate  https://review.openstack.org/52253721:16
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Turn on new port binding extended live migrate flow  https://review.openstack.org/55217321:16
*** pchavva has quit IRC21:16
cfriesenmriedem: yeah, I got messed up by the deprecated variations bit. my bad21:20
*** gouthamr has quit IRC21:22
*** gouthamr has joined #openstack-nova21:23
*** burt has joined #openstack-nova21:25
*** sar has quit IRC21:25
*** claudiub has joined #openstack-nova21:27
*** bhagyashris_ has joined #openstack-nova21:28
openstackgerritMatt Riedemann proposed openstack/nova master: Make nova-cells-v1 run with neutron  https://review.openstack.org/54978921:28
*** gouthamr has quit IRC21:29
*** janki has quit IRC21:30
*** claudiub|2 has quit IRC21:30
*** esberglu has quit IRC21:30
*** andreas_s has joined #openstack-nova21:30
*** bhagyashri_s has quit IRC21:31
*** esberglu has joined #openstack-nova21:33
*** eharney has quit IRC21:33
*** sidx64 has quit IRC21:33
*** pchavva has joined #openstack-nova21:34
*** pchavva has quit IRC21:34
*** andreas_s has quit IRC21:35
*** AlexeyAbashkin has joined #openstack-nova21:36
*** tssurya has quit IRC21:37
*** esberglu has quit IRC21:37
*** pchavva has joined #openstack-nova21:38
*** pchavva has quit IRC21:38
*** AlexeyAbashkin has quit IRC21:41
*** awaugama has quit IRC21:41
openstackgerritDan Smith proposed openstack/nova master: [WIP] Add --by-service to discover_hosts  https://review.openstack.org/55269121:43
openstackgerritMerged openstack/nova master: conf: Fix indentation of database options  https://review.openstack.org/44309721:44
*** salv-orlando has quit IRC21:44
openstackgerritMerged openstack/nova master: Re-work the metadata service docs  https://review.openstack.org/55025021:44
*** salv-orlando has joined #openstack-nova21:53
*** danpawlik has joined #openstack-nova21:53
*** suresh12 has quit IRC21:53
*** mriedem has quit IRC21:55
*** pooja_jadhav has quit IRC21:55
*** pooja_jadhav has joined #openstack-nova21:56
*** rcernin has joined #openstack-nova21:59
*** burt has quit IRC22:00
*** danpawlik has quit IRC22:02
*** esberglu has joined #openstack-nova22:02
*** amodi has quit IRC22:04
*** suresh12 has joined #openstack-nova22:08
*** ayoung has joined #openstack-nova22:10
*** suresh12 has quit IRC22:17
*** wolverineav has quit IRC22:18
*** wolverineav has joined #openstack-nova22:19
*** yangyapeng has joined #openstack-nova22:20
*** suresh12 has joined #openstack-nova22:21
*** edmondsw has quit IRC22:21
*** Guest90946 has quit IRC22:23
*** yangyapeng has quit IRC22:24
*** yangyapeng has joined #openstack-nova22:40
*** priteau has quit IRC22:40
*** chyka has quit IRC22:43
*** chyka has joined #openstack-nova22:43
*** yangyapeng has quit IRC22:44
*** munimeha1 has quit IRC22:51
*** liverpooler has quit IRC22:53
*** hongbin has quit IRC22:59
*** yangyapeng has joined #openstack-nova23:04
*** hoangcx has quit IRC23:07
*** hoangcx has joined #openstack-nova23:07
*** yamamoto has joined #openstack-nova23:08
*** yangyapeng has quit IRC23:08
*** salv-orlando has quit IRC23:15
*** felipemonteiro_ has quit IRC23:16
*** artom has joined #openstack-nova23:20
*** yangyapeng has joined #openstack-nova23:23
*** gongysh has joined #openstack-nova23:25
*** gongysh has quit IRC23:26
*** gongysh has joined #openstack-nova23:27
*** yangyapeng has quit IRC23:28
*** AlexeyAbashkin has joined #openstack-nova23:36
*** edmondsw has joined #openstack-nova23:36
*** yangyapeng has joined #openstack-nova23:38
*** mlavalle has quit IRC23:40
*** AlexeyAbashkin has quit IRC23:41
*** wolverineav has quit IRC23:41
*** edmondsw has quit IRC23:41
*** wolverineav has joined #openstack-nova23:42
*** yangyapeng has quit IRC23:42
*** germs has joined #openstack-nova23:55
*** mriedem has joined #openstack-nova23:56
mriedemso uh, getting the cells v1 job to pass with neutron was actually pretty damn easy https://review.openstack.org/#/c/549789/23:56
*** Swami has quit IRC23:56
*** dklyle has quit IRC23:57

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