Thursday, 2018-07-12

*** TxGirlGeek has joined #openstack-nova00:01
*** shaohe_feng has quit IRC00:07
*** shaohe_feng has joined #openstack-nova00:07
openstackgerritmelanie witt proposed openstack/nova-specs master: Add a script for counting blueprints  https://review.openstack.org/58191400:11
*** felipemonteiro has joined #openstack-nova00:15
*** shaohe_feng has quit IRC00:17
*** shaohe_feng has joined #openstack-nova00:18
*** phuongnh has joined #openstack-nova00:21
anupnHi mriedem: I have a problem while launching instance on my baremetal instance inside Devstack.00:22
anupnmriedem: I see a error message of "Failure prepping block device"00:22
*** linkmark has quit IRC00:23
*** felipemonteiro_ has joined #openstack-nova00:23
anupnwhen I ran "openstack server show". Any idea, what this could be?00:23
anupnI checked few things like 1. inside n-sch logs is the respective host getting selected (it is). 2. hypervisor stats show, show correct information 3. resource provider are matching00:24
*** felipemonteiro has quit IRC00:26
*** shaohe_feng has quit IRC00:27
*** shaohe_feng has joined #openstack-nova00:28
*** shaohe_feng has quit IRC00:37
*** shaohe_feng has joined #openstack-nova00:40
*** phuongnh has quit IRC00:40
*** phuongnh has joined #openstack-nova00:40
mriedemattaching volumes in compute failed, check cinder and nova-compute logs00:42
*** edmondsw has joined #openstack-nova00:43
*** hongbin has joined #openstack-nova00:46
*** shaohe_feng has quit IRC00:48
gmannmriedem: dansmith alex_xu yikun I agree with point of keeping consistency between API, Object layer. I am ok with 'policy' & 'rules' also. My vote it - "policy_*" is better. "policy & rules" are good (not bad)00:48
*** shaohe_feng has joined #openstack-nova00:48
*** edmondsw has quit IRC00:48
*** yamamoto has joined #openstack-nova00:49
*** chyka has joined #openstack-nova00:51
*** yamamoto has quit IRC00:54
*** TxGirlGeek has quit IRC00:54
*** chyka has quit IRC00:56
*** shaohe_feng has quit IRC00:58
*** shaohe_feng has joined #openstack-nova01:00
*** phuongnh has quit IRC01:01
*** phuongnh has joined #openstack-nova01:02
*** shaohe_feng has quit IRC01:08
*** shaohe_feng has joined #openstack-nova01:09
arvindn05mriedem: I updated the glance documentation to reflect the glance-image-traits spec functionality https://review.openstack.org/#/c/581920/01:12
arvindn05i will try to get the glance core's to approve it...but wanted to make sure you are fine with the amount of detail01:13
*** shaohe_feng has quit IRC01:18
*** shaohe_feng has joined #openstack-nova01:20
*** jichen has joined #openstack-nova01:26
*** mrsoul has joined #openstack-nova01:29
*** shaohe_feng has quit IRC01:29
*** mrsoul` has joined #openstack-nova01:29
*** mschuppert has quit IRC01:31
*** shaohe_feng has joined #openstack-nova01:32
*** mrsoul`` has quit IRC01:32
*** itlinux has joined #openstack-nova01:36
*** tiendc has joined #openstack-nova01:36
*** shaohe_feng has quit IRC01:39
*** med_ has quit IRC01:39
*** shaohe_feng has joined #openstack-nova01:42
*** yamahata has quit IRC01:44
mriedemgmann: arvindn05: ack, thanks01:45
*** mriedem is now known as mriedem_away01:45
*** mlavalle has quit IRC01:45
*** shaohe_feng has quit IRC01:49
*** trungnv has quit IRC01:50
*** shaohe_feng has joined #openstack-nova01:50
*** trungnv has joined #openstack-nova01:50
*** yamamoto has joined #openstack-nova01:50
*** med_ has joined #openstack-nova01:53
*** yamamoto has quit IRC01:55
*** blkart has quit IRC01:57
*** blkart has joined #openstack-nova01:57
*** shaohe_feng has quit IRC01:59
*** shaohe_feng has joined #openstack-nova02:00
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object  https://review.openstack.org/56337502:00
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object  https://review.openstack.org/56340102:00
yikungmann: OK, ack~02:01
*** blkart has quit IRC02:01
*** itlinux has quit IRC02:02
*** antosh has joined #openstack-nova02:03
*** yamamoto has joined #openstack-nova02:03
openstackgerritlei zhang proposed openstack/nova master: Replace ZeroMQ URL with Qpid URL  https://review.openstack.org/58192602:05
*** shaohe_feng has quit IRC02:10
*** blkart has joined #openstack-nova02:10
*** shaohe_feng has joined #openstack-nova02:12
*** tetsuro has joined #openstack-nova02:12
*** shaohe_feng has quit IRC02:20
*** shaohe_feng has joined #openstack-nova02:21
*** huanhongda has joined #openstack-nova02:23
*** antosh has quit IRC02:26
*** gongysh has joined #openstack-nova02:29
*** shaohe_feng has quit IRC02:30
*** oomichi has quit IRC02:31
*** shaohe_feng has joined #openstack-nova02:31
*** edmondsw has joined #openstack-nova02:32
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Change the ServerGroupAntiAffinityFilter to adapt to new policy  https://review.openstack.org/57116602:34
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Adapt _validate_instance_group_policy to new policy model  https://review.openstack.org/57146502:34
*** jmlowe has quit IRC02:35
*** edmondsw has quit IRC02:36
*** hongbin has quit IRC02:38
*** hongbin has joined #openstack-nova02:38
*** shaohe_feng has quit IRC02:40
*** shaohe_feng has joined #openstack-nova02:41
*** antosh has joined #openstack-nova02:42
*** psachin has joined #openstack-nova02:44
*** Guest44289 has quit IRC02:45
*** shaohe_feng has quit IRC02:51
*** Guest44289 has joined #openstack-nova02:51
*** shaohe_feng has joined #openstack-nova02:51
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (8)  https://review.openstack.org/57531102:59
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (9)  https://review.openstack.org/57558102:59
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (10)  https://review.openstack.org/57601702:59
*** shaohe_feng has quit IRC03:01
*** shaohe_feng has joined #openstack-nova03:02
*** hongbin has quit IRC03:08
*** brault has joined #openstack-nova03:09
openstackgerritTakashi NATSUME proposed openstack/nova master: Transform libvirt.error notification  https://review.openstack.org/48485103:11
*** shaohe_feng has quit IRC03:11
*** hongbin has joined #openstack-nova03:12
*** shaohe_feng has joined #openstack-nova03:13
*** brault has quit IRC03:14
openstackgerritTakashi NATSUME proposed openstack/nova master: Transform libvirt.error notification  https://review.openstack.org/48485103:14
*** shaohe_feng has quit IRC03:21
*** shaohe_feng has joined #openstack-nova03:22
*** gongysh has quit IRC03:26
*** med_ has quit IRC03:29
*** shaohe_feng has quit IRC03:32
*** spsurya_ has joined #openstack-nova03:32
*** shaohe_feng has joined #openstack-nova03:32
*** shaohe_feng has quit IRC03:42
*** shaohe_feng has joined #openstack-nova03:42
*** gongysh has joined #openstack-nova03:50
*** shaohe_feng has quit IRC03:52
*** shaohe_feng has joined #openstack-nova03:53
*** udesale has joined #openstack-nova03:55
openstackgerritChen proposed openstack/nova master: Clean up group members in db when delete  https://review.openstack.org/58140303:55
*** myrat has joined #openstack-nova03:58
*** shaohe_feng has quit IRC04:02
*** shaohe_feng has joined #openstack-nova04:04
*** gongysh has quit IRC04:05
*** sridharg has joined #openstack-nova04:06
*** shaohe_feng has quit IRC04:13
*** mdnadeem has joined #openstack-nova04:14
*** shaohe_feng has joined #openstack-nova04:17
*** chyka has joined #openstack-nova04:18
*** hongbin has quit IRC04:19
*** edmondsw has joined #openstack-nova04:20
*** antosh has quit IRC04:22
*** shaohe_feng has quit IRC04:23
*** chyka has quit IRC04:23
*** edmondsw has quit IRC04:24
*** shaohe_feng has joined #openstack-nova04:26
*** agrebennikov has joined #openstack-nova04:26
*** shaohe_feng has quit IRC04:33
*** shaohe_feng has joined #openstack-nova04:34
*** vivsoni_ has joined #openstack-nova04:38
*** vivsoni has quit IRC04:38
*** vivsoni has joined #openstack-nova04:41
*** shaohe_feng has quit IRC04:43
*** vivsoni_ has quit IRC04:43
*** shaohe_feng has joined #openstack-nova04:44
*** abhishekk has joined #openstack-nova04:52
*** shaohe_feng has quit IRC04:54
*** shaohe_feng has joined #openstack-nova04:54
*** Eran_Kuris has joined #openstack-nova04:56
*** janki has joined #openstack-nova04:57
*** felipemonteiro_ has quit IRC04:58
*** flwang1 has quit IRC04:59
*** links has joined #openstack-nova05:03
*** shaohe_feng has quit IRC05:04
*** shaohe_feng has joined #openstack-nova05:05
*** shaohe_feng has quit IRC05:14
*** shaohe_feng has joined #openstack-nova05:15
*** ratailor has joined #openstack-nova05:20
*** jdillaman has quit IRC05:23
*** dillaman has quit IRC05:23
*** shaohe_feng has quit IRC05:24
*** shaohe_feng has joined #openstack-nova05:25
*** dr_gogeta86 has quit IRC05:26
*** huanhongda has quit IRC05:27
*** strigazi_ has joined #openstack-nova05:29
*** pmannidi has joined #openstack-nova05:30
*** strigazi has quit IRC05:32
*** strigazi has joined #openstack-nova05:34
*** shaohe_feng has quit IRC05:35
*** jdillaman has joined #openstack-nova05:35
*** dillaman has joined #openstack-nova05:35
*** strigazi_ has quit IRC05:36
*** shaohe_feng has joined #openstack-nova05:38
*** shaohe_feng has quit IRC05:45
*** shaohe_feng has joined #openstack-nova05:46
*** shaohe_feng has quit IRC05:55
*** shaohe_feng has joined #openstack-nova05:57
*** gongysh has joined #openstack-nova06:04
*** shaohe_feng has quit IRC06:05
*** shaohe_feng has joined #openstack-nova06:07
*** Guest44289 has quit IRC06:09
*** hshiina has joined #openstack-nova06:14
*** shaohe_feng has quit IRC06:16
*** shaohe_feng has joined #openstack-nova06:17
*** armaan has joined #openstack-nova06:18
*** maciejjozefczyk has joined #openstack-nova06:19
*** maciejjozefczyk has quit IRC06:19
*** belmoreira has joined #openstack-nova06:19
*** maciejjozefczyk has joined #openstack-nova06:24
*** mrsoul is now known as mschuppert06:24
*** Guest44289 has joined #openstack-nova06:24
*** shaohe_feng has quit IRC06:26
*** belmore__ has joined #openstack-nova06:26
*** shaohe_feng has joined #openstack-nova06:27
*** belmoreira has quit IRC06:28
*** bkopilov has quit IRC06:30
*** jangutter has quit IRC06:31
*** cfriesen has quit IRC06:32
*** ispp has joined #openstack-nova06:32
*** shaohe_feng has quit IRC06:36
*** shaohe_feng has joined #openstack-nova06:38
*** gongysh has quit IRC06:38
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver  https://review.openstack.org/52338706:43
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver  https://review.openstack.org/52765806:43
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add snapshot function  https://review.openstack.org/53424006:43
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add power actions  https://review.openstack.org/54334006:43
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add get console output  https://review.openstack.org/54334406:43
openstackgerritTakashi NATSUME proposed openstack/nova master: Transform volume.usage notification  https://review.openstack.org/58034506:46
*** shaohe_feng has quit IRC06:46
*** shaohe_feng has joined #openstack-nova06:47
*** unclok has joined #openstack-nova06:48
*** ykarel has joined #openstack-nova06:48
*** moshele has joined #openstack-nova06:49
unclokhello, can I ask a question?06:49
unclokI'm installing 1 controller + 2 computes node openstack using packstack, from centos-openstack-queens repo06:50
unclokbut after I finished the installation, my nova service can't find hypervisors06:51
unclokand I'm getting this error from nova-compute.log06:51
unclok2018-07-12 06:46:57.580 6044 ERROR nova.compute.manager [req-cc06eb6a-0d3e-4224-a82c-2affb187d6b7 - - - - -] No compute node record for host compute1.packstack.spark: ComputeHostNotFound_Remote: Compute host compute1.packstack.spark could not be found.06:52
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/57801906:52
unclokit's a whole nova-compute.log file06:52
unclokhttps://files.fm/f/83ypewsn#_06:52
*** sahid has joined #openstack-nova06:53
unclokI can find my nova cell from "nova-manage cell_v2 list_cells" command, and I can find it's configuration is alright06:55
unclok|   Name  |                 UUID                 |              Transport URL              |                Database Connection                 |06:55
unclok|  cell0  | 00000000-0000-0000-0000-000000000000 |                  none:/                 | mysql+pymysql://nova:****@172.17.17.226/nova_cell0 |06:55
unclok| default | 87da9bc1-34d0-4a99-ae1e-d60a7b606df7 | rabbit://guest:****@172.17.17.226:5672/ |    mysql+pymysql://nova:****@172.17.17.226/nova    |06:55
unclok"default" cell is an auto-created cell by packstack06:56
unclokeven I tried to remove default cell,06:56
*** shaohe_feng has quit IRC06:57
unclokand modified cell0 configurations using right transport_url and database_connection settings06:57
*** shaohe_feng has joined #openstack-nova06:57
*** bkopilov has joined #openstack-nova06:58
*** vivsoni_ has joined #openstack-nova06:58
unclokbut I can't still find my hypervisors from "openstack hypervisor list" or "nova hypervisor-list"06:58
*** vivsoni has quit IRC06:59
unclokI'm just using 1 nic for each nodes, using openvswitch and vxlan for tunneling07:00
*** vivsoni has joined #openstack-nova07:02
*** amoralej|off is now known as amoralej07:02
*** belmoreira has joined #openstack-nova07:04
*** vivsoni_ has quit IRC07:04
*** jangutter has joined #openstack-nova07:05
*** belmore__ has quit IRC07:05
*** shaohe_feng has quit IRC07:07
*** shaohe_feng has joined #openstack-nova07:08
*** mgoddard has joined #openstack-nova07:08
*** aloga has joined #openstack-nova07:09
*** tssurya has joined #openstack-nova07:11
openstackgerritGhanshyam Mann proposed openstack/nova master: Complete the api-ref of security group rule  https://review.openstack.org/58010907:13
openstackgerritChen proposed openstack/nova master: Clean up group members in db when delete  https://review.openstack.org/58140307:14
*** shaohe_feng has quit IRC07:17
*** stakeda has quit IRC07:18
*** vladikr has joined #openstack-nova07:18
*** shaohe_feng has joined #openstack-nova07:18
*** rcernin has quit IRC07:20
*** ispp has quit IRC07:21
*** flwang1 has joined #openstack-nova07:22
*** janki has quit IRC07:22
*** andreas_s has joined #openstack-nova07:22
*** janki has joined #openstack-nova07:23
*** gongysh has joined #openstack-nova07:24
*** peereb has joined #openstack-nova07:25
*** liuyulong has joined #openstack-nova07:25
*** shaohe_feng has quit IRC07:27
*** gongysh has quit IRC07:27
*** shaohe_feng has joined #openstack-nova07:28
*** ispp has joined #openstack-nova07:30
*** unclok has quit IRC07:30
*** mgoddard has quit IRC07:30
*** jangutter_ has joined #openstack-nova07:31
*** unclok has joined #openstack-nova07:31
*** gongysh has joined #openstack-nova07:32
*** alexchadin has joined #openstack-nova07:32
*** jangutter has quit IRC07:32
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (11)  https://review.openstack.org/57601807:36
*** mgoddard has joined #openstack-nova07:36
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (12)  https://review.openstack.org/57601907:36
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (13)  https://review.openstack.org/57602007:36
*** shaohe_feng has quit IRC07:38
*** shaohe_feng has joined #openstack-nova07:39
*** janki has quit IRC07:39
*** mgoddard has quit IRC07:40
*** belmoreira has quit IRC07:40
*** janki has joined #openstack-nova07:40
*** ttsiouts has joined #openstack-nova07:41
*** Guest44289 has quit IRC07:45
*** shaohe_feng has quit IRC07:48
*** yamamoto has quit IRC07:48
*** ispp has quit IRC07:50
*** shaohe_feng has joined #openstack-nova07:51
*** ispp has joined #openstack-nova07:51
*** ttsiouts has quit IRC07:52
*** mago_ has joined #openstack-nova07:53
*** chyka has joined #openstack-nova07:54
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Microversion 2.64 - Use new format policy in server group  https://review.openstack.org/56753407:56
*** stakeda has joined #openstack-nova07:56
*** belmoreira has joined #openstack-nova07:56
*** shaohe_feng has quit IRC07:58
*** chyka has quit IRC07:59
*** andreas_s has quit IRC07:59
*** andreas_s has joined #openstack-nova08:00
*** shaohe_feng has joined #openstack-nova08:00
openstackgerritTakashi NATSUME proposed openstack/nova master: Transform libvirt.error notification  https://review.openstack.org/48485108:04
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (14)  https://review.openstack.org/57602708:04
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15)  https://review.openstack.org/57603108:05
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16)  https://review.openstack.org/57629908:05
*** gongysh has quit IRC08:05
*** yassine has joined #openstack-nova08:06
*** yassine is now known as Guest9068108:06
*** ispp has quit IRC08:08
*** shaohe_feng has quit IRC08:08
*** Guest90681 has quit IRC08:09
*** shaohe_feng has joined #openstack-nova08:09
*** yassine_ has joined #openstack-nova08:10
*** tssurya has left #openstack-nova08:13
*** ttsiouts has joined #openstack-nova08:14
openstackgerrithuanhongda proposed openstack/nova-specs master: Add ram/cpu/disk allocation ratio to Hypervisor show  https://review.openstack.org/58208908:15
*** itlinux has joined #openstack-nova08:16
*** ccamacho has joined #openstack-nova08:17
*** vabada has joined #openstack-nova08:17
*** vabada has quit IRC08:18
*** vabada has joined #openstack-nova08:19
*** shaohe_feng has quit IRC08:19
*** mriedem_away has quit IRC08:20
openstackgerritVu Cong Tuan proposed openstack/python-novaclient master: Switch to stestr  https://review.openstack.org/58209008:21
*** shaohe_feng has joined #openstack-nova08:21
openstackgerrithuanhongda proposed openstack/nova stable/queens: [Stable Only] Remove soft-deleted instances from quota_usages  https://review.openstack.org/57909308:23
*** links has quit IRC08:24
openstackgerritTuan Do Anh proposed openstack/nova-specs master: Update pypi url to new url  https://review.openstack.org/58209208:26
*** Shilpa has joined #openstack-nova08:26
*** cdent has joined #openstack-nova08:26
*** links has joined #openstack-nova08:26
*** shaohe_feng has quit IRC08:29
*** shaohe_feng has joined #openstack-nova08:30
*** mvk_ has quit IRC08:30
*** ispp has joined #openstack-nova08:31
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17)  https://review.openstack.org/57634408:35
*** sapd has quit IRC08:35
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18)  https://review.openstack.org/57667308:35
openstackgerrithuanhongda proposed openstack/nova-specs master: Add ram/cpu/disk allocation ratio to Hypervisor show  https://review.openstack.org/58208908:35
*** sapd has joined #openstack-nova08:35
*** tssurya has joined #openstack-nova08:36
*** tesseract has joined #openstack-nova08:37
*** tssurya has left #openstack-nova08:37
*** shaohe_feng has quit IRC08:39
*** tssurya has joined #openstack-nova08:40
openstackgerritYikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.64 - Use new format policy in server group  https://review.openstack.org/57826108:40
*** shaohe_feng has joined #openstack-nova08:41
openstackgerrithuanhongda proposed openstack/nova-specs master: Return "deleted" time of instance when vm_state is SOFT-DELETED  https://review.openstack.org/58163808:41
*** derekh has joined #openstack-nova08:43
*** yamamoto has joined #openstack-nova08:44
*** ispp has quit IRC08:48
*** shaohe_feng has quit IRC08:49
*** yamamoto has quit IRC08:50
*** shaohe_feng has joined #openstack-nova08:50
*** ispp has joined #openstack-nova08:53
*** mvk_ has joined #openstack-nova08:54
vabadaCtrl-B $08:55
vabada08:55
vabadaArne Wiebalck08:55
vabada10:55 AM08:55
vabadasorry08:55
*** sambetts|afk is now known as sambetts08:57
*** tetsuro has quit IRC08:58
*** shaohe_feng has quit IRC09:00
*** agrebennikov has quit IRC09:01
*** derekh has quit IRC09:04
*** derekh has joined #openstack-nova09:04
*** vladikr has quit IRC09:05
*** shaohe_feng has joined #openstack-nova09:06
*** shaohe_feng has quit IRC09:10
*** jangutter_ has quit IRC09:10
*** links has quit IRC09:10
*** shaohe_feng has joined #openstack-nova09:11
*** links has joined #openstack-nova09:12
*** andreas_s has quit IRC09:12
*** andreas_s has joined #openstack-nova09:13
*** andreas_s has quit IRC09:16
*** mvk_ has quit IRC09:16
*** andreas_s has joined #openstack-nova09:16
openstackgerritVu Cong Tuan proposed openstack/python-novaclient master: Switch to stestr  https://review.openstack.org/58209009:16
*** jangutter has joined #openstack-nova09:17
*** shaohe_feng has quit IRC09:20
*** shaohe_feng has joined #openstack-nova09:21
*** vabada has quit IRC09:21
*** vabada has joined #openstack-nova09:22
*** UdayTK has joined #openstack-nova09:22
*** hoonetorg has quit IRC09:27
*** mvk_ has joined #openstack-nova09:28
*** shaohe_feng has quit IRC09:30
*** belmoreira has quit IRC09:31
*** UdayTK has quit IRC09:31
*** shaohe_feng has joined #openstack-nova09:32
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Microversion 2.64 - Use new format policy in server group  https://review.openstack.org/56753409:33
*** pmannidi has quit IRC09:36
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19)  https://review.openstack.org/57667609:36
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20)  https://review.openstack.org/57668909:36
*** hoonetorg has joined #openstack-nova09:40
*** shaohe_feng has quit IRC09:41
*** stakeda has quit IRC09:41
*** shaohe_feng has joined #openstack-nova09:42
*** BrinZhang has joined #openstack-nova09:42
*** mgoddard has joined #openstack-nova09:43
BrinZhangdansmith: Hi, can you review this patch?09:43
BrinZhangdansmith: https://review.openstack.org/#/c/579747/09:44
*** yamamoto has joined #openstack-nova09:46
BrinZhangmelwitt: Do you have time ?09:46
BrinZhangThank you :)09:46
*** shaohe_feng has quit IRC09:51
*** yamamoto has quit IRC09:51
*** alexchadin has quit IRC09:51
*** shaohe_feng has joined #openstack-nova09:53
*** alexchadin has joined #openstack-nova09:59
*** shaohe_feng has quit IRC10:01
*** shaohe_feng has joined #openstack-nova10:02
*** yamamoto has joined #openstack-nova10:03
*** takashin_ has joined #openstack-nova10:04
*** takashin has quit IRC10:06
*** mgoddard has quit IRC10:06
*** jichen has quit IRC10:08
*** rodolof has joined #openstack-nova10:09
*** shaohe_feng has quit IRC10:11
*** shaohe_feng has joined #openstack-nova10:12
*** phuongnh has quit IRC10:14
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21)  https://review.openstack.org/57670910:19
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22)  https://review.openstack.org/57671210:19
*** belmoreira has joined #openstack-nova10:19
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py  https://review.openstack.org/56615310:20
*** liuyulong has quit IRC10:20
openstackgerritTakashi NATSUME proposed openstack/nova master: Add the guideline to write API reference  https://review.openstack.org/56905810:21
kashyapBrinZhang: Hi, it is middle of the night for both of them.  They'll take notice when they come online.10:21
*** shaohe_feng has quit IRC10:22
BrinZhangkashyap: That's it, thank you :)10:22
*** shaohe_feng has joined #openstack-nova10:23
*** Eran_Kuris has quit IRC10:26
*** Eran_Kuris has joined #openstack-nova10:26
*** abhishekk has quit IRC10:28
*** shaohe_feng has quit IRC10:32
*** shaohe_feng has joined #openstack-nova10:33
*** vladikr has joined #openstack-nova10:34
*** udesale has quit IRC10:35
*** shaohe_feng has quit IRC10:42
*** ttsiouts has quit IRC10:42
*** shaohe_feng has joined #openstack-nova10:43
*** mdbooth has joined #openstack-nova10:45
*** BrinZhang has quit IRC10:50
*** BrinZhang has joined #openstack-nova10:50
*** shaohe_feng has quit IRC10:52
*** shaohe_feng has joined #openstack-nova10:53
*** cdent has quit IRC10:54
*** myrat has quit IRC10:57
*** mgoddard has joined #openstack-nova10:57
*** takashin_ has quit IRC11:00
*** takashin has joined #openstack-nova11:02
*** shaohe_feng has quit IRC11:03
*** shaohe_feng has joined #openstack-nova11:03
*** mago_ has quit IRC11:09
*** shaohe_feng has quit IRC11:13
*** shaohe_feng has joined #openstack-nova11:14
openstackgerritzhaolihui proposed openstack/python-novaclient master: Fix flavor keyerror when nova boot  https://review.openstack.org/58214711:14
*** tianhui has joined #openstack-nova11:14
*** lyan has joined #openstack-nova11:16
*** lyan is now known as Guest8785811:17
gmannmelwitt: i have updated nova meeting agenda to include the API subteam status. I might not be able to attend today but ll be there from next week(on 14.00 UTC slot).  for this week, you can refer the mail i shared for API highlights and next week i can start summarize in meeting. - http://lists.openstack.org/pipermail/openstack-dev/2018-July/132148.html11:17
*** amoralej is now known as amoralej|lunch11:18
openstackgerritkarim proposed openstack/nova master: Handle rebuild of instances with image traits  https://review.openstack.org/56949811:19
*** jaypipes has joined #openstack-nova11:22
*** shaohe_feng has quit IRC11:23
*** med_ has joined #openstack-nova11:24
*** shaohe_feng has joined #openstack-nova11:24
*** chyka has joined #openstack-nova11:30
*** shaohe_feng has quit IRC11:33
*** shaohe_feng has joined #openstack-nova11:34
*** chyka has quit IRC11:36
*** abhishekk has joined #openstack-nova11:36
*** ttsiouts has joined #openstack-nova11:36
*** mago_ has joined #openstack-nova11:42
*** mgoddard has quit IRC11:43
*** shaohe_feng has quit IRC11:44
*** shaohe_feng has joined #openstack-nova11:44
*** hshiina has quit IRC11:46
*** yikun has quit IRC11:46
openstackgerritRadoslav Gerganov proposed openstack/nova master: VMware: Live migration of instances  https://review.openstack.org/27011611:46
*** armaan has quit IRC11:48
*** armaan has joined #openstack-nova11:49
*** belmorei_ has joined #openstack-nova11:52
*** shaohe_feng has quit IRC11:54
*** belmoreira has quit IRC11:55
*** shaohe_feng has joined #openstack-nova11:55
*** udesale has joined #openstack-nova11:57
*** mdbooth has quit IRC12:01
*** itlinux has quit IRC12:01
openstackgerritRadoslav Gerganov proposed openstack/nova master: VMware: ensure that live migration attaches to correct interface  https://review.openstack.org/47879712:01
*** mriedem has joined #openstack-nova12:01
*** shaohe_feng has quit IRC12:04
*** shaohe_feng has joined #openstack-nova12:05
*** tssurya has quit IRC12:08
*** edmondsw has joined #openstack-nova12:09
*** belmorei_ has quit IRC12:09
*** belmorei_ has joined #openstack-nova12:11
*** ratailor has quit IRC12:11
*** BrinZhang has quit IRC12:12
*** tiendc has quit IRC12:13
*** shaohe_feng has quit IRC12:14
*** damien_r1 has quit IRC12:14
*** ispp has quit IRC12:16
*** damien_r has joined #openstack-nova12:16
*** shaohe_feng has joined #openstack-nova12:17
*** amoralej|lunch is now known as amoralej12:20
*** ispp has joined #openstack-nova12:22
*** yassine_ has quit IRC12:23
*** yassine_ has joined #openstack-nova12:24
*** shaohe_feng has quit IRC12:25
*** ykarel is now known as ykarel|afk12:25
*** shaohe_feng has joined #openstack-nova12:26
*** yikun has joined #openstack-nova12:27
*** mdnadeem has quit IRC12:29
*** ykarel|afk has quit IRC12:30
*** belmorei_ has quit IRC12:32
mriedemarvindn05: done https://review.openstack.org/#/c/581920/12:32
*** shaohe_feng has quit IRC12:35
*** shaohe_feng has joined #openstack-nova12:37
*** ykarel|afk has joined #openstack-nova12:38
*** efried_off is now known as efried12:41
*** armaan has quit IRC12:41
*** mdbooth has joined #openstack-nova12:41
*** armaan has joined #openstack-nova12:41
*** felipemonteiro has joined #openstack-nova12:43
*** shaohe_feng has quit IRC12:45
openstackgerritChen proposed openstack/nova master: fix cellv2 delete_host  https://review.openstack.org/58217112:47
*** shaohe_feng has joined #openstack-nova12:47
*** neha_alhat_ has quit IRC12:52
*** neha_alhat has joined #openstack-nova12:53
*** ykarel|afk is now known as ykarel12:54
neha_alhatjaypipes: Hi, Regarding this patch: https://review.openstack.org/#/c/581218/12:54
*** mchlumsky has joined #openstack-nova12:55
*** shaohe_feng has quit IRC12:55
*** mdnadeem has joined #openstack-nova12:55
neha_alhatjaypipes: Actually no need to remove logger, we can keep it as for future use(if any other client passed logger object while creating novaclient object) if not pass it will be None12:56
*** shaohe_feng has joined #openstack-nova12:56
*** pchavva has joined #openstack-nova12:56
neha_alhatjaypipes: and there is no bug reported for this12:56
jaypipesneha_alhat: but aren't you removing all use of self.logger in the client?12:56
jaypipesneha_alhat: please see Kevin Mitchell's comment on the patch. It seems like there should be a bug for this and at a minimum a test case that demonstrates how you are seeing what you are seeing.12:57
openstackgerritMatt Riedemann proposed openstack/nova master: Mention osc-placement for managing traits in docs  https://review.openstack.org/58217312:57
*** pchavva has quit IRC12:58
neha_alhatjaypipes: can you please this patch: https://review.openstack.org/#/c/578045/12:58
neha_alhatjaypipes: please refer*12:58
*** pchavva has joined #openstack-nova12:59
*** tssurya has joined #openstack-nova12:59
neha_alhatjaypipes: Ya I am removing use of self.logger12:59
*** yamamoto has quit IRC13:01
*** vladikr has quit IRC13:01
*** shaohe_feng has quit IRC13:06
*** shaohe_feng has joined #openstack-nova13:08
*** belmoreira has joined #openstack-nova13:08
*** jchhatbar has joined #openstack-nova13:09
*** mdnadeem has quit IRC13:12
*** janki has quit IRC13:13
*** dklyle has quit IRC13:14
*** udesale has quit IRC13:15
*** shaohe_feng has quit IRC13:16
*** shaohe_feng has joined #openstack-nova13:17
jaypipesneha_alhat: I'll chat with sean mcginnis about your patch.13:17
neha_alhatjaypipes: ok, sure13:18
*** priteau has joined #openstack-nova13:24
*** shaohe_feng has quit IRC13:26
*** shaohe_feng has joined #openstack-nova13:27
*** yamamoto has joined #openstack-nova13:28
*** yamamoto has quit IRC13:29
efriedalex_xu: Are you putting eyes on the rest of the zvm series too?  https://review.openstack.org/#/c/523387/  I'm +2 all the way up.  When we have a second +2 on all the patches, we can ask dansmith to remove his -2 and merge.13:31
dansmithI'm going to have to drop my -2 as I'm out next week13:31
dansmithplease don't let me come back to half that series merged :)13:31
*** yamamoto has joined #openstack-nova13:34
*** shaohe_feng has quit IRC13:36
*** shaohe_feng has joined #openstack-nova13:37
*** felipemonteiro_ has joined #openstack-nova13:38
mriedemsob http://paste.openstack.org/show/725704/13:38
mriedembfv with cross_az_attach=False, but i guess that's known13:38
mriedemwas hoping it was fixed by somethign else13:38
*** cnf has left #openstack-nova13:39
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Microversion 2.64 - Use new format policy in server group  https://review.openstack.org/56753413:39
alex_xuefried: yea, I'm working ont he review ont hat series13:40
efriedcool13:40
*** yamamoto_ has joined #openstack-nova13:41
*** yamamoto has quit IRC13:41
*** felipemonteiro has quit IRC13:42
efriedmriedem: https://review.openstack.org/#/c/569498/ looks like karimull nailed it. I'm +2 again.13:46
*** psachin has quit IRC13:46
stephenfindansmith: In brief, could you tell me what's the different between setting an o.vo attribute via the initializer and manually setting it after?13:47
*** shaohe_feng has quit IRC13:47
stephenfine.g. NUMACell(network_metadata=xyz) vs. numa_cell = NUMACell(); numa_cell.network_metadata = xyz13:47
dansmithstephenfin: meaning the difference between Obj(foo=bar) and obj = pbj(); obj.foo = bar?13:47
stephenfinyup13:48
dansmithshould be nothing13:48
stephenfinThat's what I'd thought13:48
*** Eran_Kuris has quit IRC13:48
dansmith...but...?13:49
stephenfinbut I've spent the last two days on a case where doing that resulted in the attribute apparently not being set13:49
*** shaohe_feng has joined #openstack-nova13:49
stephenfinLemme finish making sure this works and I'll push it up with a comment on the offending line, in case you're curious13:49
dansmithokay13:50
stephenfinProbably PEBKAC but we'll see :)13:50
*** links has quit IRC13:50
dansmithhopefully PEBKAC :)13:50
stephenfinmriedem: After that fun, I totally agree on getting functional tests for numa-aware-vswitches. I'm going to need a little guidance/existing examples on how to actually approach it, if you have anything13:51
*** psachin has joined #openstack-nova13:51
stephenfinI have half-written Tempest tests but they won't be able to run upstream13:52
gibistephenfin: which part of the functional test you feel problematic? I might be able to help13:55
*** awaugama has joined #openstack-nova13:55
stephenfinAlso, forget about PCI, how we track all things NUMA is weird and I dislike it13:55
melwittnova meeting in 5 min13:55
alex_xuefried: yea, +2 also, leave the +w to mriedem13:55
efriedThanks alex_xu13:56
stephenfingibi: tbh, most of it. I guess I need to model the whole flow from API call to instance creation. I'm not sure how much of that functional tests case do for us13:56
stephenfinAlso, simple things like faking nova.conf13:56
melwittgmann: ack, thanks13:56
*** andreas_s has quit IRC13:57
*** shaohe_feng has quit IRC13:57
*** andreas_s has joined #openstack-nova13:57
mriedemefried: alex_xu: thanks will check after the meeting13:58
*** shaohe_feng has joined #openstack-nova13:58
gibistephenfin: functional test is fairly easy if the code under test is not in (or triggered by) the virt driver13:58
*** belmore__ has joined #openstack-nova13:58
gibistephenfin: there is a lot of server create and move operation tests in https://github.com/openstack/nova/blob/master/nova/tests/functional/test_servers.py13:59
mriedemstephenfin: you'll need a fixture for the host inventory stuff for numa13:59
*** alexchad_ has joined #openstack-nova13:59
*** mlavalle has joined #openstack-nova13:59
mriedemprobably some kind of simple fixture data struct that you can insert inventory into and then run scenarios against13:59
*** belmoreira has quit IRC14:00
*** andreas_s has quit IRC14:00
stephenfinmriedem, gibi: OK, cool. The only functional tests I'd worked14:00
*** andreas_s has joined #openstack-nova14:00
stephenfin*I've worked on before are the PCI ones but they don't seem to be E2E tests. This will be fun, heh https://github.com/openstack/nova/blob/master/nova/tests/functional/libvirt/test_pci_sriov_servers.py14:01
*** munimeha1 has joined #openstack-nova14:01
*** alexchadin has quit IRC14:01
*** armaan_ has joined #openstack-nova14:01
*** armaan has quit IRC14:02
mriedemso start simple with writing a fixture that can model some numa cell hardware on a host with no physnets or tunneled set,14:02
mriedemrun a server create with numa requirements against that and make sure it's ok,14:03
*** linkmark has joined #openstack-nova14:03
mriedemthen start folding in wrinkles like phsynet inventory and requirements, tunneled, etc14:03
gibistephenfin: I don't know about functional tests that are end2end in a sense that it trigger something on the API and uses real libvirt driver code (by mocking the libvirt interface)14:03
mriedemyou can create the numa topology for the host in the db14:03
mriedemfor a single compute node created in the test14:03
*** rodolof has quit IRC14:04
*** psachin has quit IRC14:05
gibimriedem, stephenfin: I wouldn't touch db form a functional test, maybe a FakeVirtDriver that reports numa topology14:05
mriedemdepends on how complicated you want to start with,14:06
mriedemcreating stuff in the db to start might be easiest and then later convert to a fake virt driver14:06
mriedemonce you know all the scenarios14:07
*** shaohe_feng has quit IRC14:07
*** shaohe_feng has joined #openstack-nova14:08
openstackgerritRadoslav Gerganov proposed openstack/nova master: VMware: Live migration of instances  https://review.openstack.org/27011614:08
openstackgerritRadoslav Gerganov proposed openstack/nova master: VMware: ensure that live migration attaches to correct interface  https://review.openstack.org/47879714:08
gibimriedem: sure that could work14:08
*** janki has joined #openstack-nova14:09
*** jchhatbar has quit IRC14:09
stephenfinmriedem: Sounds good. Lemme give that a shot14:10
*** eharney has joined #openstack-nova14:12
*** cfriesen has joined #openstack-nova14:13
*** ccamacho has quit IRC14:16
*** moshele has quit IRC14:16
*** tidwellr has joined #openstack-nova14:17
*** shaohe_feng has quit IRC14:17
*** shaohe_feng has joined #openstack-nova14:20
*** ccamacho has joined #openstack-nova14:21
*** dpawlik has quit IRC14:25
*** ykarel is now known as ykarel|away14:27
*** shaohe_feng has quit IRC14:28
*** shaohe_feng has joined #openstack-nova14:29
*** abhishekk has quit IRC14:29
*** antosh has joined #openstack-nova14:30
openstackgerritMerged openstack/nova master: Add policy to InstanceGroup object  https://review.openstack.org/56337514:34
*** GonZo2000 has joined #openstack-nova14:34
*** GonZo2000 has quit IRC14:34
*** GonZo2000 has joined #openstack-nova14:34
*** belmore__ has quit IRC14:36
*** hongbin has joined #openstack-nova14:37
*** shaohe_feng has quit IRC14:38
*** shaohe_feng has joined #openstack-nova14:39
*** ttsiouts has quit IRC14:39
*** ttsiouts has joined #openstack-nova14:40
*** ttsiouts has quit IRC14:44
stephenfinmriedem: RE: [1], NUMACell.network_metadata is not nullable. However, that means I need to update everything that uses NUMACell to set something for that field, right? [1] https://review.openstack.org/#/c/564439/13/nova/objects/numa.py@5814:45
stephenfinThat's a _lot_ of changes, if so14:46
dansmithstephenfin: it can be unset regardless of the nullability14:47
dansmithI think his point was that you should just not set it instead of always forcing it to None14:47
*** tssurya has quit IRC14:48
*** shaohe_feng has quit IRC14:48
*** dpawlik has joined #openstack-nova14:48
*** TxGirlGeek has joined #openstack-nova14:48
*** tssurya has joined #openstack-nova14:48
stephenfindansmith: That makes sense, but I found a piece I'd missed here https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L1692-L169514:49
*** _pewp_ has quit IRC14:49
stephenfin(that was where I was seeing the "setting in the initializer vs. setting as an attribute issue, btw)14:49
stephenfinIf I'm assuming network_metadata is always set in production and it's a bug if not, then I need to update any unit test which calls the function14:50
*** _pewp_ has joined #openstack-nova14:50
dansmithI'm not sure what you mean14:50
stephenfindansmith: I need http://paste.openstack.org/show/725721/14:51
*** felipemonteiro_ has quit IRC14:51
*** felipemonteiro__ has joined #openstack-nova14:51
dansmithsounds like you need to conditionally set it14:52
dansmithlike:14:52
dansmithif "network_metadata" in hostcell: newcell.network_metadata = hostcell.network_metadata14:52
dansmithright?14:52
*** alexchadin has joined #openstack-nova14:53
stephenfinIndeed, and that's what I had done but I saw the aforementioned issue14:53
*** shaohe_feng has joined #openstack-nova14:53
dansmithyou never really told me what you saw.. I'm assuming you meant that setting it after initialization seemed not to stick?14:53
dansmiththat would, of course, be strange behavior we'd want to nail down14:53
dansmithbecause _obviously_ you should be able to do that r:)14:54
mriedemstephenfin: dansmith: the issue i pointed out later in the series once i got to the api part, was that if the user explicitly specifies networks='none' it means network_metadata will be None,14:56
*** alexchad_ has quit IRC14:56
mriedemso we can't blindly set that on the RequestSpec in the api because it will blow up if the field is not nullable,14:56
mriedemso we just need to be conscious of that,14:56
stephenfinYeah, I was just concerned that even with that, we'd have the potential to hide some bugs so it wouldn't be acceptable14:56
stephenfini.e. if, for some reason, we weren't setting the attribute somewhere when we should be14:56
mriedembut it also means the consuming code in the scheduler filter (and hardware.py) needs to be aware the field might not be set14:56
mriedemwhich hardware.py is already doing i believe14:56
dansmithmriedem: if there is a valid case where it should be None, then it should be nullable=true, however, we shouldn't ever set it to None when we don't know the value14:56
stephenfinmriedem: I don't think that's an issue - we always set NetworkMetadata but the physnets/tunneled attributes might not be set14:57
mriedemnetworks='none' is the case where we know it should be None14:57
mriedemstephenfin: bzzt,14:57
mriedemlet me link you14:57
mriedem(to your code)14:57
dansmithmriedem: should that be network_metadata=None, or just physnets=[],tunneled=False ?14:57
mriedemhttps://review.openstack.org/#/c/564444/14/nova/network/neutronv2/api.py@163314:58
mriedemthat depends on ^14:58
mriedemi have a whole buttload of comments in https://review.openstack.org/#/c/564452/ related to this14:58
*** shaohe_feng has quit IRC14:58
mriedemhttps://review.openstack.org/#/c/564452/17/nova/compute/api.py@88914:58
stephenfinmriedem: Sorry, you said RequestSpec and I read NUMACell https://review.openstack.org/#/c/564441/21/nova/virt/libvirt/driver.py@633814:59
*** shaohe_feng has joined #openstack-nova14:59
*** _pewp_ has quit IRC14:59
stephenfinmriedem: Yeah, I should just be setting that to NetworkMetadata(physnets=set(), tunneled=False)15:00
*** _pewp_ has joined #openstack-nova15:00
mriedemwell also see https://review.openstack.org/#/c/564444/14/nova/tests/unit/network/test_neutronv2.py@496315:01
mriedemsimilar concern15:01
*** armaan_ has quit IRC15:01
mriedemanyway, you have options15:01
*** jmlowe has joined #openstack-nova15:02
*** ratailor has joined #openstack-nova15:02
mriedemeither use an 'empty' NetworkMetadata object, or support None, or handle unset15:02
mriedemat this point i kind of think nullable=True makes sense for when we know we don't have networking15:02
mriedembut it's like a 5% "makes sense"15:03
stephenfinheh15:04
stephenfinOK, I'll explore that15:04
*** rmart04 has joined #openstack-nova15:04
stephenfinand try to push something to demonstrate this possible o.vo bug15:04
*** takashin has left #openstack-nova15:04
*** _pewp_ has quit IRC15:04
*** _pewp_ has joined #openstack-nova15:06
*** shaohe_feng has quit IRC15:09
*** shaohe_feng has joined #openstack-nova15:09
mriedemanyone have any idea why we wouldn't cache the host az here? https://github.com/openstack/nova/blob/da16690f4db6172659ffa4f804296100b5ba24d6/nova/availability_zones.py#L9315:09
mriedemwe do it when getting the az for an instance that has a host set https://github.com/openstack/nova/blob/da16690f4db6172659ffa4f804296100b5ba24d6/nova/availability_zones.py#L19415:10
*** _pewp_ has quit IRC15:10
*** ttsiouts has joined #openstack-nova15:11
*** _pewp_ has joined #openstack-nova15:11
*** ispp has quit IRC15:12
mriedemdansmith: i think i have a fix for the cinder/cross_az_attach=False up-call issue15:12
mriedem1 line fix15:12
*** ispp has joined #openstack-nova15:13
jmlowedansmith: do you happen to know off the top of your head where the ceph free space is queried?15:13
mriedemjmlowe: should be down in libvirt driver update_available_resource15:13
mriedemget_available_resource?15:13
jmlowethose are grepable, thanks15:13
mriedemyeah _get_local_gb_info15:13
mriedemelif CONF.libvirt.images_type == 'rbd':15:13
mriedem            info = LibvirtDriver._get_rbd_driver().get_pool_info()15:13
*** rmart04_ has joined #openstack-nova15:15
*** rmart04_ has left #openstack-nova15:16
*** _pewp_ has quit IRC15:16
*** ttsiouts has quit IRC15:16
*** _pewp_ has joined #openstack-nova15:16
*** rmart04 has quit IRC15:17
*** shaohe_feng has quit IRC15:19
mriedemhttps://bugs.launchpad.net/nova/+bug/178142115:20
openstackLaunchpad bug 1781421 in OpenStack Compute (nova) "CantStartEngineError due to host aggregate up-call when boot from volume and [cinder]/cross_az_attach=False" [Medium,Triaged]15:20
mriedem-    if not host:15:20
mriedem+    if not host or (host and instance.availability_zone):15:20
mriedemfixed!15:20
*** shaohe_feng has joined #openstack-nova15:20
*** yamamoto_ has quit IRC15:24
openstackgerritMatt Riedemann proposed openstack/nova master: Mention osc-placement for managing traits in docs  https://review.openstack.org/58217315:25
*** ccamacho has quit IRC15:26
*** felipemonteiro_ has joined #openstack-nova15:28
melwittjmlowe: and here's a proposal for another way to query free space that I've not fixed up with tests yet https://review.openstack.org/#/c/556692/1/nova/virt/libvirt/storage/rbd_utils.py15:28
*** shaohe_feng has quit IRC15:29
*** shaohe_feng has joined #openstack-nova15:30
*** gyee has joined #openstack-nova15:30
*** felipemonteiro_ has quit IRC15:31
dansmithnote that in placement, we don't report free space, but infer it from total - (allocated + reserved)15:31
*** felipemonteiro__ has quit IRC15:31
*** ispp has quit IRC15:32
melwittoh, okay15:32
*** ykarel|away has quit IRC15:32
mriedemmelwitt: you were +2 on this before jaypipes pointed out that we couldn't actually update consumer project/user info in placement, but that bug is fixed now and his -2 is gone, and i've added a functional test so can you review this again? https://review.openstack.org/#/c/574488/ - it's the heal_allocations thing for healing incomplete consumers15:32
mriedemand would close out that bp15:33
*** ttsiouts has joined #openstack-nova15:33
melwittmriedem: ah yup, thanks for the heads up15:33
melwittwill revisit that15:33
*** gyee has quit IRC15:34
*** moshele has joined #openstack-nova15:38
*** ttsiouts has quit IRC15:39
*** shaohe_feng has quit IRC15:39
stephenfinPro tip #1: 'tox -e py27 -- --failing' runs all the tests that failed on a previous iteration15:39
stephenfinPro tip #2: 'tox -e py27 -- -n nova/tests/unit/virt/test_hardware.py' let's you run tests without needing to convert to a Python path15:40
*** ttsiouts has joined #openstack-nova15:40
*** shaohe_feng has joined #openstack-nova15:40
* stephenfin has been reading the stestr docs15:40
mriedemefried: alex_xu: arvindn05: karimull: on its way to the gate of horrors https://review.openstack.org/#/c/569498/ - i'll do a follow up for my comments15:42
efriedmriedem: cool mon15:42
*** felipemonteiro has joined #openstack-nova15:42
mriedempass the dutchie15:42
mriedemmelwitt: ^ means another runway slot will be open today15:43
melwittthat means we get to close out a bp right? woo15:43
melwittnoyce. good news15:43
melwittstephenfin: protip #2 I didn't know about, thanks15:44
*** ttsiouts has quit IRC15:44
*** wwriverr_ has joined #openstack-nova15:45
*** peereb has quit IRC15:45
efriedI tend to use stestr run directly from the venv so I don't have to wait an extra 45 seconds for tox to do... whatever it does in "preparation".15:46
melwittI do too when I'm iterating on running tests15:47
*** TxGirlGeek has quit IRC15:47
stephenfinefried: I think we discussed this before, but doesn't 'touch nova.egg_info' fix that for you?15:47
*** jmlowe has quit IRC15:47
*** agrebennikov has joined #openstack-nova15:48
efriedstephenfin: That does sound familiar. Not sure. But I know this works :)15:48
*** wwriverrat has quit IRC15:48
stephenfinFair :)15:49
efriedstephenfin: I've got it in a bash function so I can kick it off easily15:49
stephenfinI really need to figure out what's going on with tox there. It seems pretty useless, given how often I need to recreate stuff when dependencies change15:50
*** shaohe_feng has quit IRC15:50
*** pchavva has quit IRC15:50
*** shaohe_feng has joined #openstack-nova15:50
efriedi.e. very rarely, yah?15:51
*** felipemonteiro has quit IRC15:51
stephenfinEver :) If it's doing a dependency check based on the timestamp of .egg_info, that check should be failing because dependencies have changed15:52
efriedAssuming it's dependency checks that are slow, if there was a tox --no-dep-check flag that let you bypass all that stuff, that would be coolness.15:52
stephenfinIf it's not failing, you'd have to wonder what the "dependency check" _is_ doing15:52
efriedoh, or that, if it's actually a bug that should just be fixed :)15:52
efriedwell, occasionally tox does rebuild my venv.  I assume it does that based on figuring out that something is out of date.15:52
stephenfinYeah, I'd rather some way to just disable it, personally. The gate will catch the rare case where a local version mismatch causes issues15:53
stephenfinThe only time I see a full rebuild is when tox.ini changes15:53
melwittmine never auto rebuilds the venv for me. I do it manually if I hit "no module X found" errors when things change15:53
stephenfin(assuming you mean recreate)15:53
stephenfinmelwitt: Yeah, same here15:53
openstackgerritEric Fried proposed openstack/nova master: WIP: Compute: Handle reshaped provider trees  https://review.openstack.org/57623615:54
stephenfinHence, what _is_ tox doing :)15:54
melwittyeah...15:54
efriedIf I know a dep is out of date, I'll enter the venv and pip install/upgrade the sucker manually.15:54
melwittI'd like to know too but I've no idea how to find out15:55
efriedTakes way less time than rebuilding the whole damn venv.15:55
melwittheh, I've done that in the past but nowadays I'm like meh15:55
stephenfinefried: That. I personally let it rebuild but Real World Devs (TM) often fix this stuff themselves. I think a flag would be a good idea15:56
efriedUsually when I'm running tox it's because I actually want to run tests, not because I want to kick off a command and go eat lunch and come back to see if it has finished yet.15:56
stephenfin...as soon as we figure out what is taking that time when it's not rebuilding (the thing 'touch nova.*' solves)15:56
efriedCool mon, let me know if you want me to try anything out.15:56
stephenfinAck. I'll try take a look tomorrow once I get somewhere on these functional tests for numa-aware-vswitch15:57
efriedThat's not using nrp, is it?15:57
efried(I haven't read the spec in yonks, have totally forgotten what it actually does)15:58
stephenfinNope, nothing to do with placement15:58
stephenfinNot now, anyway. A fancier version of it might use it in the future but that's 2-3 cycles down the road, I'd imagine15:59
*** shaohe_feng has quit IRC16:00
*** andreas_s has quit IRC16:01
*** shaohe_feng has joined #openstack-nova16:01
*** dklyle has joined #openstack-nova16:02
mriedemTheJulia: do you know if there are any ironic jobs that can be used to test this rescue patch? https://review.openstack.org/#/c/416487/316:03
mriedemhttps://review.openstack.org/#/c/416487/16:03
mriedemi know tempest tests rescue, we'd just have to flip the switch in some ironic job config16:03
TheJuliamriedem: we've not merged them yet because of the nova patch not merging, looking it up16:03
mriedemTheJulia: ok but we can see a depends-on patch passing tests right?16:03
TheJuliathat being said, the logs show it working in the proposed tempest patch16:03
mriedemah https://review.openstack.org/#/c/528699/16:04
TheJuliamriedem: already did if my memory is correct16:04
*** hamzy_ has quit IRC16:04
TheJuliayup, thats the one16:04
*** andreas_s has joined #openstack-nova16:04
*** hamzy_ has joined #openstack-nova16:04
TheJuliahttps://review.openstack.org/#/c/528704/16:05
mriedemok that depends on the plugin change which depends on the nova change, i see16:06
TheJuliaand I see you already rechecked it16:06
TheJuliathanks!16:06
openstackgerritMultipleCrashes proposed openstack/nova master: Retry decorator fix for instances which go into ERROR state during bulk delete  https://review.openstack.org/56341816:07
*** gyee has joined #openstack-nova16:08
*** shaohe_feng has quit IRC16:10
*** shaohe_feng has joined #openstack-nova16:11
*** ykarel|away has joined #openstack-nova16:12
openstackgerritMatt Riedemann proposed openstack/nova master: Follow up for Ie49d605c66062d2548241d7e04f5a2a6b98c011e  https://review.openstack.org/58229216:12
*** yamamoto has joined #openstack-nova16:12
kashyapmriedem: The above commit message is really odd16:13
*** alexchadin has quit IRC16:13
kashyap(It's logically correct.  But it can be turned into a sentence w/o a commit has :-))16:13
*** mriedem is now known as mriedem_away16:16
* kashyap must also admit the positive -- mriedem consistently writes some of the detailed commit messages with rarely any errors16:17
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Resource provider examples  https://review.openstack.org/55346116:17
*** shaohe_feng has quit IRC16:20
*** shaohe_feng has joined #openstack-nova16:21
*** armaan has joined #openstack-nova16:22
*** ratailor has quit IRC16:23
*** andreas_s has quit IRC16:23
*** itlinux has joined #openstack-nova16:26
*** spsurya_ has quit IRC16:26
*** sahid has quit IRC16:26
openstackgerritStephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata  https://review.openstack.org/56445216:27
openstackgerritStephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options  https://review.openstack.org/56444016:27
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_metadata field  https://review.openstack.org/56444116:27
anupnmriedem_away: Yes n-cpu showed the sec group used was wrong, and after changing that instance was launched :)16:30
*** shaohe_feng has quit IRC16:31
*** ykarel|away has quit IRC16:32
*** felipemonteiro has joined #openstack-nova16:32
*** itlinux has quit IRC16:32
*** TxGirlGeek has joined #openstack-nova16:33
*** shaohe_feng has joined #openstack-nova16:33
*** munimeha1 has quit IRC16:34
*** TxGirlGeek has quit IRC16:35
*** tssurya has quit IRC16:39
*** shaohe_feng has quit IRC16:41
*** yamamoto has quit IRC16:42
stephenfindansmith: PEBCAK http://paste.openstack.org/show/725735/ (line 14)16:42
dansmithstephenfin: heh, good16:43
stephenfinTwo days.16:44
* stephenfin goes to hit a wall or something 😅16:44
*** shaohe_feng has joined #openstack-nova16:44
*** yassine_ has quit IRC16:46
melwittI know the feeling all too well16:46
*** yassine_ has joined #openstack-nova16:47
dansmithpebkac is way better than some obscure bug16:47
dansmithfeels stupid, but it's almost always better16:48
melwitttrue that16:49
*** shaohe_feng has quit IRC16:51
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_metadata field  https://review.openstack.org/56444116:51
stephenfinAgreed16:52
stephenfinand with that, I'm outta here16:52
*** shaohe_feng has joined #openstack-nova16:53
*** derekh has quit IRC17:01
*** shaohe_feng has quit IRC17:01
*** GonZo2000 has quit IRC17:02
*** shaohe_feng has joined #openstack-nova17:02
*** sambetts is now known as sambetts|afk17:03
*** armaan has quit IRC17:03
*** imacdonn has quit IRC17:03
*** imacdonn has joined #openstack-nova17:03
*** sridharg has quit IRC17:04
*** hemna_ has joined #openstack-nova17:04
*** janki has quit IRC17:06
*** med_ has quit IRC17:07
*** munimeha1 has joined #openstack-nova17:08
*** TxGirlGeek has joined #openstack-nova17:09
*** shaohe_feng has quit IRC17:12
*** shaohe_feng has joined #openstack-nova17:13
openstackgerritEric Fried proposed openstack/nova master: WIP: Compute: Handle reshaped provider trees  https://review.openstack.org/57623617:15
*** tesseract has quit IRC17:16
*** shaohe_feng has quit IRC17:22
*** shaohe_feng has joined #openstack-nova17:23
*** abhishekk has joined #openstack-nova17:25
openstackgerritmelanie witt proposed openstack/nova master: Update process doc to be more generic about point of contact  https://review.openstack.org/58231217:25
*** shaohe_feng has quit IRC17:32
*** shaohe_feng has joined #openstack-nova17:33
melwittdoes anyone know if/how we can identify ARM64 platform in the libvirt driver? would it just be fields.Architecture.AARCH64? asking in the context of this bug I'm triaging https://bugs.launchpad.net/nova/+bug/174828017:34
openstackLaunchpad bug 1748280 in nova (Ubuntu) "'nova reboot' on arm64 is just 'nova reboot --hard' but slower" [Undecided,Confirmed]17:34
*** mriedem_away is now known as mriedem17:37
mriedemthe host capabilities should have the arch17:39
melwittyeah, wasn't sure if it's ARMV6 or ARMV7 or ARMV7B or AARCH64 or all of them. commenting on the bug17:40
*** shaohe_feng has quit IRC17:42
*** yamamoto has joined #openstack-nova17:43
*** shaohe_feng has joined #openstack-nova17:43
melwittthis is a troubling one from pike https://bugs.launchpad.net/nova/+bug/178136817:44
openstackLaunchpad bug 1781368 in OpenStack Compute (nova) "KeyError: 'flavor' when nova boot" [Undecided,New] - Assigned to zhaolihui (zhaolh)17:44
*** yamahata has joined #openstack-nova17:44
melwittactually, I can check some stable/pike gate runs to see if that's in the log17:44
dansmiththat trace shows novaclient code17:46
dansmithnot on the server side17:46
melwittoh, guh17:46
melwittbefore that, the server returns a 404 which seems wrong though. hm.17:48
*** priteau has quit IRC17:48
*** shaohe_feng has quit IRC17:53
*** shaohe_feng has joined #openstack-nova17:54
dansmithyeah which might be related, but also,17:54
dansmithisn't 2.60 much later than pike?17:55
dansmith2.53 is pike17:55
melwittoh yeah, queens. 17.0.5 is queens, sorry17:55
dansmithokay17:56
*** mago_ has quit IRC17:56
*** wwriverr_ has quit IRC17:56
*** TxGirlGeek has quit IRC18:00
*** yamamoto has quit IRC18:00
melwittthis is a weird one that I've noticed myself when trying to trace requests during debugging, periodic tasks sharing request_id with actual requests. the weird thing is I swear this did not use to happen in the past, but AFAIK nothing has changed around the periodic task context use https://bugs.launchpad.net/nova/+bug/177310218:03
openstackLaunchpad bug 1773102 in OpenStack Compute (nova) "Abnormal request id in logs" [Undecided,New]18:03
*** felipemonteiro has quit IRC18:03
*** shaohe_feng has quit IRC18:03
melwittI don't know why this has started happening in the last couple of cycles (not sure when it started)18:03
*** wwriverrat has joined #openstack-nova18:04
melwittmaybe cell targeting is messing with it somehow18:04
efriedmelwitt: Nothing to do with the global request ID work?18:05
melwittefried: I don't think so. the global request ID is a separate request ID IIUC18:05
*** shaohe_feng has joined #openstack-nova18:05
*** felipemonteiro has joined #openstack-nova18:12
*** shaohe_feng has quit IRC18:13
*** shaohe_feng has joined #openstack-nova18:14
*** amoralej is now known as amoralej|off18:17
*** hamzy_ has quit IRC18:18
*** bkopilov has quit IRC18:18
artom*sigh*18:19
artomHow do I file a bug in Storyboard? Is it just a Story?18:19
*** hamzy has joined #openstack-nova18:20
*** shaohe_feng has quit IRC18:23
*** armaan has joined #openstack-nova18:24
*** jmlowe has joined #openstack-nova18:24
*** harlowja has joined #openstack-nova18:24
*** shaohe_feng has joined #openstack-nova18:24
melwittartom: I don't know, but what are you looking to file a bug for? I didn't think we use storyboard in nova18:24
artommelwitt, osc18:24
melwittah18:24
artomAm I an old man now? Grumbling about change for change's sake?18:25
arvindn05mriedem: thanks for the review on https://review.openstack.org/#/c/581920/ - Update glance documentation for trait support. I've addressed the comments. I asked the glance core folks to take a look as well, they should put in their comments/approvals early next week18:25
*** armaan has quit IRC18:28
*** armaan has joined #openstack-nova18:29
mriedemartom: yes it's a story18:29
*** TxGirlGeek has joined #openstack-nova18:29
openstackgerritmelanie witt proposed openstack/nova master: DNM: Don't overwrite greenthread-local context during target_cell  https://review.openstack.org/58233218:29
artommriedem, thanks!18:30
*** ykarel|away has joined #openstack-nova18:32
*** moshele has quit IRC18:33
*** armaan has quit IRC18:33
*** shaohe_feng has quit IRC18:34
*** shaohe_feng has joined #openstack-nova18:34
*** munimeha1 has quit IRC18:36
mriedemoopsydoodle http://logs.openstack.org/04/528704/33/check/ironic-tempest-dsvm-ipa-partition-pxe_ipmitool-tinyipa-python3/ec5520f/logs/screen-n-cpu.txt.gz#_Jul_12_18_10_59_14334418:39
melwitthm, looks like that's coming from ironicclient18:41
openstackgerritMerged openstack/nova master: Revert "docs: Disable smartquotes"  https://review.openstack.org/57884118:43
*** shaohe_feng has quit IRC18:44
*** shaohe_feng has joined #openstack-nova18:45
*** felipemonteiro has quit IRC18:46
melwittfrom distutils.version import StrictVersion and the compare operator for StrictVersion is the thing that accesses the 'version' attribute18:48
melwittso something about statements like this blows up https://github.com/openstack/python-ironicclient/blob/master/ironicclient/common/http.py#L16118:48
openstackgerritMerged openstack/nova master: Mention osc-placement for managing traits in docs  https://review.openstack.org/58217318:49
*** jistr has quit IRC18:50
*** felipemonteiro has joined #openstack-nova18:53
*** shaohe_feng has quit IRC18:54
mriedemi can't get hits on that in logstash, not sure if the ironic job logs are indexed18:54
*** shaohe_feng has joined #openstack-nova18:55
efriedIs the gate particularly slow at the moment?  The check queue seems unusually deep and not moving very fast.18:55
mriedemhttp://grafana.openstack.org/dashboard/db/zuul-status18:56
mriedemlooks like executors have dropped down18:56
efriedyou must be seeing something I'm not. That dashboard looks pretty healthy to me.18:59
efriedor at least pretty stable for the past 8h.19:00
efried'cept for the spike in "Node Requests", whatever that is.19:00
*** shaohe_feng has quit IRC19:04
*** shaohe_feng has joined #openstack-nova19:06
openstackgerritMatt Riedemann proposed openstack/nova master: Avoid up-call in get_instance_availability_zone  https://review.openstack.org/58234219:09
mriedemfix for the cross_az_attach=False up-call issue with cells v2 ^19:09
mriedemkashyap: ^ is that commit message more to your liking?19:09
melwittreally wish we still had mirroring of release tags to github *tear*19:12
*** abhishekk has quit IRC19:12
mriedemwe do, but sometimes you have to put them in yourself19:14
*** shaohe_feng has quit IRC19:15
mriedemhttps://github.com/openstack/nova/tree/17.0.519:15
mriedemi don't know why they don't show up in the dropdown19:15
melwittoh, weird. I didn't know that hack19:15
*** shaohe_feng has joined #openstack-nova19:15
melwittyass19:16
*** ykarel|away has quit IRC19:17
*** shaohe_feng has quit IRC19:25
*** jistr has joined #openstack-nova19:25
*** shaohe_feng has joined #openstack-nova19:26
*** mvk_ has quit IRC19:27
*** mriedem has quit IRC19:32
openstackgerritEric Fried proposed openstack/nova-specs master: Make reshape-provider-tree reflect reality  https://review.openstack.org/58235019:33
*** shaohe_feng has quit IRC19:35
*** moshele has joined #openstack-nova19:39
*** shaohe_feng has joined #openstack-nova19:39
*** shaohe_feng has quit IRC19:45
*** eharney has quit IRC19:46
*** shaohe_feng has joined #openstack-nova19:46
*** flwang1 has quit IRC19:47
*** awaugama has quit IRC19:50
*** TxGirlGeek has quit IRC19:52
*** agrebennikov_ has joined #openstack-nova19:54
*** shaohe_feng has quit IRC19:56
*** mvk_ has joined #openstack-nova19:56
*** shaohe_feng has joined #openstack-nova19:56
*** TxGirlGeek has joined #openstack-nova19:58
*** mriedem has joined #openstack-nova20:00
mriedemTheJulia: just a few small things in the ironic rescue patch https://review.openstack.org/#/c/416487/ - i'd like to see the ironicclient 2.2.0 version negotiation stuff split out as a dependent change in the series if possible,20:01
mriedembut i'm not married to that if it's more pain than it's worth20:01
TheJuliaI guess my real concern with retooling that is that we're less likely to get any traction on negotiation changes, and we'll end up shooting ourselves... again... with a static pin update.20:02
TheJuliathen again, I guess it could be before, but then what is the point if we're not actually upping the version in that single patch.20:03
* TheJulia waits for webpage to load to lookup version information20:05
*** shaohe_feng has quit IRC20:06
mriedemthe point is it's laying the groundwork for a feature that's going to use it20:07
*** shaohe_feng has joined #openstack-nova20:07
mriedemagain, i'm not a hard stop on splitting it out, just seems we could/should20:07
mriedemalso, seeing issues in n-cpu logs with StrictVersion usage in there20:07
mriedemhttp://logs.openstack.org/04/528704/33/check/ironic-tempest-dsvm-ipa-partition-pxe_ipmitool-tinyipa-python3/ec5520f/logs/screen-n-cpu.txt.gz#_Jul_12_18_10_59_14334420:07
mriedemwe must be passing in StrictVersion(None) at some point20:09
TheJuliaInteresting20:10
mriedemanywho, gotta drop for a bit20:10
TheJuliaGreat point on the logging, I think. It has been a long time since I looked at this patch20:10
*** mriedem has quit IRC20:11
TheJuliaoh!20:11
TheJuliai know how :\20:11
*** shaohe_feng has quit IRC20:16
*** shaohe_feng has joined #openstack-nova20:18
*** tssurya has joined #openstack-nova20:20
*** unclok has quit IRC20:21
openstackgerritEric Fried proposed openstack/nova master: WIP: Compute: Handle reshaped provider trees  https://review.openstack.org/57623620:21
* TheJulia changes gears to the rescue patch20:22
*** owalsh_ has joined #openstack-nova20:24
*** shaohe_feng has quit IRC20:26
*** owalsh has quit IRC20:27
*** shaohe_feng has joined #openstack-nova20:28
*** owalsh_ has quit IRC20:29
*** belmoreira has joined #openstack-nova20:30
*** owalsh has joined #openstack-nova20:30
openstackgerritEric Fried proposed openstack/nova master: WIP: Compute: Handle reshaped provider trees  https://review.openstack.org/57623620:30
openstackgerritEric Fried proposed openstack/nova-specs master: Make reshape-provider-tree reflect reality  https://review.openstack.org/58235020:31
openstackgerritEric Fried proposed openstack/nova-specs master: Make reshape-provider-tree spec reflect reality  https://review.openstack.org/58235020:32
*** moshele has quit IRC20:35
*** belmoreira has quit IRC20:36
*** shaohe_feng has quit IRC20:37
*** rodolof has joined #openstack-nova20:37
*** shaohe_feng has joined #openstack-nova20:37
*** rodolof has quit IRC20:38
melwitthas anyone gotten this error trying to run functional tests? tox.ConfigError: ConfigError: substitution key 'posargs' not found20:43
*** med_ has joined #openstack-nova20:46
*** med_ has quit IRC20:46
*** med_ has joined #openstack-nova20:46
*** shaohe_feng has quit IRC20:47
*** shaohe_feng has joined #openstack-nova20:48
efriedmelwitt: Paste me your [testenv:functional] paragraph from tox.ini20:54
* efried scrambles around for stephenfin's cape20:54
melwittI just pulled it from master to see if it changed anything and it didn't. I wonder if my tox version is too old or something20:54
efriedthat would have been my first guess, yeah.20:55
*** mriedem has joined #openstack-nova20:55
efriedmelwitt: It's near the top of my todo list to bump our min required tox version to take advantage of the basepython fix anyway...20:55
efriedmelwitt: What is your tox version?20:57
melwitt2.3.120:57
*** shaohe_feng has quit IRC20:57
efriedthere ya go.20:57
efriedhttps://github.com/tox-dev/tox/issues/27920:57
melwittfrom whatever's for xenial6420:57
openstackgerritJay Pipes proposed openstack/nova master: do not assume 1 consumer in AllocList.delete_all()  https://review.openstack.org/58238220:57
openstackgerritJay Pipes proposed openstack/nova master: perform reshaper operations in single transaction  https://review.openstack.org/58238320:57
*** shaohe_feng has joined #openstack-nova20:58
melwittthanx efried20:58
efriedmelwitt: Be interesting to see if it goes away immediately by upgrading to 2.3.220:58
efriedrather than jumping straight to latest or whatever.20:59
efriedmake sure it's the same bug and all.20:59
efriedbut whatevs.20:59
melwittyeah, lemme see20:59
*** TxGirlGeek has quit IRC21:01
melwittyup, 2.3.2 works21:01
melwittyey21:01
efriedsweet21:01
*** TxGirlGeek has joined #openstack-nova21:02
efriedI'm going to work on that patch to bump the minimum to 3.1.1 and see if it flies.21:02
*** flwang1 has joined #openstack-nova21:03
*** mago_ has joined #openstack-nova21:03
*** shaohe_feng has quit IRC21:07
*** shaohe_feng has joined #openstack-nova21:09
melwitt2.3.2 forever21:09
* TheJulia wonders if that is a commitment to always use 2.3.2... and if so she is curious why21:10
melwittjust hassling efried21:11
*** mago_ has quit IRC21:13
* TheJulia opens a beer and resumes hacking on versioning code21:15
efriedTheJulia: Slightly lower down on my to-do list was adding os-ironic-api-version kwargs to more methods in ironicclient.  That's not what you're working on, is it?21:15
TheJuliaefried: my brain kind of segfaulted upon all of the everyone implement x api features and bump their major versions discussion at the ptg.. and has not revisited versioning since.21:16
TheJuliaerr21:16
TheJulias/ptg/summit/21:16
efriedTheJulia: ooo, that's also not at all what I'm referring to.21:17
TheJuliaoh, I know what your referring to, I'm not sure it is really actually needed for most usage patterns, at least that is what I was thinking about when I added it to a few more21:18
*** shaohe_feng has quit IRC21:18
TheJuliaI'm just stating, my brain broke later on, and has not revisited the subject.21:18
*** shaohe_feng has joined #openstack-nova21:18
efriedTheJulia: Okay. IIRC it was only needed in a small handful of nova use cases, like half a dozen.21:18
TheJuliaI thought it was... 3 or 421:19
openstackgerritEric Fried proposed openstack/nova master: Use tox 3.1.1 and basepython fix  https://review.openstack.org/58239221:19
TheJuliaanyway, back to splitting it apart which actually makes sense given the fix necessary21:19
efriedmelwitt: ^21:19
efriedstephenfin: https://review.openstack.org/58239221:20
*** antosh has quit IRC21:26
*** rmart04 has joined #openstack-nova21:28
*** rmart04 has quit IRC21:28
*** shaohe_feng has quit IRC21:28
*** shaohe_feng has joined #openstack-nova21:30
*** tidwellr has quit IRC21:31
openstackgerritmelanie witt proposed openstack/nova master: DNM: Don't overwrite greenthread-local context in target_cell  https://review.openstack.org/58233221:32
*** edmondsw has quit IRC21:34
*** felipemonteiro has quit IRC21:37
*** shaohe_feng has quit IRC21:38
*** edmondsw has joined #openstack-nova21:40
*** shaohe_feng has joined #openstack-nova21:40
*** TxGirlGeek has quit IRC21:43
*** edmondsw has quit IRC21:44
*** TxGirlGeek has joined #openstack-nova21:44
*** tssurya has quit IRC21:45
*** TxGirlGeek has quit IRC21:46
*** shaohe_feng has quit IRC21:48
*** shaohe_feng has joined #openstack-nova21:49
*** TxGirlGeek has joined #openstack-nova21:50
*** tssurya has joined #openstack-nova21:54
*** rcernin has joined #openstack-nova21:58
*** shaohe_feng has quit IRC21:59
*** shaohe_feng has joined #openstack-nova21:59
openstackgerritmelanie witt proposed openstack/nova master: Add functional regression test for bug 1778305  https://review.openstack.org/58240722:05
openstackbug 1778305 in OpenStack Compute (nova) "Nova may erronously look up service version of a deleted service, when hostname have been reused" [Undecided,New] https://launchpad.net/bugs/177830522:05
openstackgerritmelanie witt proposed openstack/nova master: Don't generate service UUID for deleted services  https://review.openstack.org/58240822:05
*** boris_42_ has joined #openstack-nova22:09
*** shaohe_feng has quit IRC22:09
*** shaohe_feng has joined #openstack-nova22:10
*** dklyle has quit IRC22:16
*** dklyle has joined #openstack-nova22:16
mriedemfinally re-remembered something i had to learn 9 months ago22:17
mriedemyay22:17
mriedem*re-learned22:17
*** shaohe_feng has quit IRC22:19
*** shaohe_feng has joined #openstack-nova22:20
*** TxGirlGeek has quit IRC22:23
*** tssurya has quit IRC22:26
openstackgerritMatt Riedemann proposed openstack/nova master: Add note about reschedules and num_attempts in filter_properties  https://review.openstack.org/58241222:28
*** shaohe_feng has quit IRC22:29
*** TxGirlGeek has joined #openstack-nova22:31
*** shaohe_feng has joined #openstack-nova22:32
*** shaohe_feng has quit IRC22:40
*** shaohe_feng has joined #openstack-nova22:41
openstackgerritmelanie witt proposed openstack/nova stable/ocata: [stable only] Always commit quota decrement if build request found  https://review.openstack.org/58241322:49
*** shaohe_feng has quit IRC22:50
*** shaohe_feng has joined #openstack-nova22:52
melwittmriedem: I'd appreciate a sanity check from you at your convenience ^ before I open a bug. this came from a customer seeing behavior where they delete an instance and don't see quota usage decrease (in the logs I saw racing creates and deletes)22:53
melwittI think I've convinced myself that it makes sense to commit the quotas whether we succeed in looking up the instance or not but it would be good to know if you think that's legit too22:54
mriedemseems not crazy, but i'd have to load up all of that terrible context from conductor and such into my brain22:55
mriedemi wish we could remove a lot of that "in the future" and "soon this will happen" stuff in api and conductor22:56
melwittyeah :( it was painful for me to re-learn that stuff too22:56
melwittyeah, same. for all I know, we can. honestly don't know where we're at on those22:57
*** shaohe_feng has quit IRC23:00
*** yassine_ has quit IRC23:01
*** shaohe_feng has joined #openstack-nova23:01
cfriesenre: the issue with service UUIDs and deleted services...you might want to take a look at https://bugs.launchpad.net/nova/+bug/176455623:02
openstackLaunchpad bug 1764556 in OpenStack Compute (nova) ""nova list" fails with exception.ServiceNotFound if service is deleted and has no UUID" [Medium,In progress]23:02
*** TxGirlGeek has quit IRC23:04
cfriesennot sure if it'll cause problems or not, but it's mucking around in the same area23:04
melwittyeah, thanks for the heads up. reading through it23:04
openstackgerritJulia Kreger proposed openstack/nova master: virt/ironic: Implement rescue and unrescue  https://review.openstack.org/41648723:05
openstackgerritJulia Kreger proposed openstack/nova master: ironic: provide facilies to gracefully navigate versions  https://review.openstack.org/58241623:05
mriedemmelwitt: i've had these forever https://review.openstack.org/#/q/topic:cleanup-update-instance+(status:open+OR+status:merged)23:05
mriedemnot in the same stuff you're removing, bu related idea on cleaning up transition code23:05
TheJuliamriedem: ^^^23:05
*** edleafe- has joined #openstack-nova23:05
mriedemTheJulia: ack23:05
*** edleafe has quit IRC23:05
*** edleafe- is now known as edleafe23:05
TheJuliamriedem: I'm going to go kick the two jobs to recheck23:05
mriedemgood idea23:06
melwittmriedem: gah, I didn't know about those. I've starred them to remind myself to review23:06
*** bnemec has quit IRC23:07
TheJuliaheh, need to rebase one anyway23:08
* TheJulia does that23:08
*** med_ has quit IRC23:08
melwittcfriesen: unless I'm all mixed up, it sounds like the patch I've proposed would fix that bug too23:09
*** shaohe_feng has quit IRC23:10
*** shaohe_feng has joined #openstack-nova23:11
* TheJulia goes and finds dinner23:13
*** yassine_ has joined #openstack-nova23:13
cfriesenmelwitt: because it will no longer try to generate the new UUID?23:14
melwittyes23:14
melwittI'm assuming there's no reason to generate a new UUID for a deleted service, but maybe I'm wrong23:15
melwittif we want to do that, we'd have to add some conditionals in the Service.save() method to skip things like the version check if self.deleted and do something to handle ServiceNotFound23:16
melwitt*if we want to generate UUID for deleted service23:16
openstackgerritMatt Riedemann proposed openstack/nova master: Pass request_spec from compute to cell conductor on reschedule  https://review.openstack.org/58241723:16
cfriesenmelwitt: seems plausible that it'd fix the problem.23:16
melwittI'm gonna add another functional regression test to the stack for the bug you linked to verify it23:17
melwittmaybe we'll have to do a different approach to fix things but the functional tests should be useful either way23:18
*** hongbin has quit IRC23:19
*** shaohe_feng has quit IRC23:21
*** shaohe_feng has joined #openstack-nova23:22
mriedemTheJulia: couple of small things in https://review.openstack.org/#/c/582416/23:27
*** mlavalle has quit IRC23:28
TheJuliaI can revise once the ironic jobs finish, otherwise zuul will flag everything as mergefail and then even more fun :(23:29
mriedemyeah23:29
mriedem+2 on the rescue driver change itself now at least23:31
*** shaohe_feng has quit IRC23:31
TheJuliamriedem: awesome! Thanks!23:32
*** shaohe_feng has joined #openstack-nova23:32
*** unclok has joined #openstack-nova23:35
*** med__ has quit IRC23:37
*** med_ has joined #openstack-nova23:39
*** med_ has quit IRC23:39
*** med_ has joined #openstack-nova23:39
*** Guest87858 has quit IRC23:40
*** shaohe_feng has quit IRC23:41
*** shaohe_feng has joined #openstack-nova23:42
*** med_ has quit IRC23:43
*** med_ has joined #openstack-nova23:46
*** med_ has quit IRC23:46
*** med_ has joined #openstack-nova23:46
*** yassine_ has quit IRC23:51
*** shaohe_feng has quit IRC23:51
*** shaohe_feng has joined #openstack-nova23:53
*** yassine_ has joined #openstack-nova23:57
*** markvoelker has quit IRC23:59

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