Wednesday, 2019-01-09

« Tuesday, 2019-01-08 Index Thursday, 2019-01-10 »
*** munimeha1 has quit IRC00:00
openstackgerritHongbin Lu proposed openstack/nova-specs master: Scheduling support for SR-IOV bond  https://review.openstack.org/62605500:03
*** rcernin has quit IRC00:07
*** rcernin has joined #openstack-nova00:09
*** slaweq has quit IRC00:18
*** artom has joined #openstack-nova00:23
*** artom has quit IRC00:23
*** macza has quit IRC00:28
*** owalsh_ has joined #openstack-nova00:33
*** mlavalle has quit IRC00:34
*** slaweq has joined #openstack-nova00:35
*** owalsh has quit IRC00:36
openstackgerritMerged openstack/nova stable/rocky: Fail to live migration if instance has a NUMA topology  https://review.openstack.org/62588000:36
*** liuyulong has quit IRC00:39
*** Swami has quit IRC00:46
*** slaweq has quit IRC00:47
*** ileixe has joined #openstack-nova00:50
*** ileixe has quit IRC00:51
*** tetsuro has joined #openstack-nova00:53
*** ileixe has joined #openstack-nova01:01
*** itlinux has joined #openstack-nova01:03
*** erlon has joined #openstack-nova01:15
*** slaweq has joined #openstack-nova01:18
*** itlinux has quit IRC01:19
*** tiendc has joined #openstack-nova01:20
*** erlon has quit IRC01:22
*** owalsh_ is now known as owalsh01:23
*** slaweq has quit IRC01:23
*** Kevin_Zheng has joined #openstack-nova01:30
*** tetsuro has quit IRC01:41
*** hongbin_ has joined #openstack-nova01:48
*** xek has quit IRC01:51
*** macza has joined #openstack-nova01:53
*** macza has quit IRC01:57
openstackgerritMerged openstack/nova stable/rocky: Imported Translations from Zanata  https://review.openstack.org/62810902:06
openstackgerritMerged openstack/nova stable/rocky: Fix a broken link  https://review.openstack.org/62679802:06
openstackgerritMerged openstack/nova stable/rocky: Removing pip-missing-reqs from default tox jobs  https://review.openstack.org/62895202:06
openstackgerritMerged openstack/nova stable/rocky: Fix destination_type attribute in the bdm_v2 documentation  https://review.openstack.org/62687402:06
*** igordc has quit IRC02:06
*** igordc has joined #openstack-nova02:06
*** igordc has quit IRC02:08
*** efried has quit IRC02:09
*** igordc has joined #openstack-nova02:10
*** xek has joined #openstack-nova02:12
*** itlinux has joined #openstack-nova02:12
*** xek has quit IRC02:12
*** igordc has quit IRC02:12
*** xek has joined #openstack-nova02:13
*** sapd1_x has joined #openstack-nova02:13
*** itlinux has quit IRC02:13
takashin02:25
*** psachin has joined #openstack-nova02:48
*** wolverineav has quit IRC02:48
*** wolverineav has joined #openstack-nova02:48
*** whoami-rajat has joined #openstack-nova02:52
*** wolverineav has quit IRC02:54
*** wolverineav has joined #openstack-nova02:55
*** wolverineav has quit IRC02:57
*** mhen has quit IRC02:58
*** wolverineav has joined #openstack-nova02:59
*** mhen has joined #openstack-nova02:59
*** wolverineav has quit IRC03:03
*** brinzhang has joined #openstack-nova03:05
*** gyee has quit IRC03:10
*** markvoelker has joined #openstack-nova03:11
*** markvoelker has quit IRC03:12
*** brinzhang has quit IRC03:31
*** itlinux has joined #openstack-nova03:33
*** tbachman has quit IRC03:37
*** janki has joined #openstack-nova03:45
*** spsurya has joined #openstack-nova03:46
*** Dinesh_Bhor has joined #openstack-nova03:51
*** Dinesh_Bhor has quit IRC03:59
openstackgerritTakashi NATSUME proposed openstack/nova stable/queens: Add description of custom resource classes  https://review.openstack.org/61912504:07
openstackgerritTakashi NATSUME proposed openstack/nova stable/queens: Add description of custom resource classes  https://review.openstack.org/61912504:07
*** macza has joined #openstack-nova04:16
*** hongbin_ has quit IRC04:24
*** sridharg has joined #openstack-nova04:27
*** udesale has joined #openstack-nova04:27
*** ileixe has quit IRC04:31
*** markvoelker has joined #openstack-nova04:37
*** markvoelker has quit IRC04:42
*** takashin has quit IRC04:45
*** wolverineav has joined #openstack-nova04:45
*** wwriverrat has joined #openstack-nova04:52
*** lifeless_ has joined #openstack-nova04:57
*** wolverineav has quit IRC05:00
*** ileixe has joined #openstack-nova05:01
*** sridharg has quit IRC05:02
*** psachin has quit IRC05:02
*** sapd1_x has quit IRC05:02
*** sapd1_ has quit IRC05:02
*** chason has quit IRC05:02
*** lifeless has quit IRC05:02
*** Dinesh_Bhor has joined #openstack-nova05:04
*** sapd1_x has joined #openstack-nova05:08
*** sridharg has joined #openstack-nova05:08
*** sapd1_ has joined #openstack-nova05:10
*** Dinesh_Bhor has quit IRC05:12
*** wolverineav has joined #openstack-nova05:15
*** med_ has quit IRC05:23
*** takashin has joined #openstack-nova05:28
*** Dinesh_Bhor has joined #openstack-nova05:30
*** takashin has left #openstack-nova05:30
*** udesale has quit IRC05:32
*** udesale has joined #openstack-nova05:40
*** ratailor has joined #openstack-nova06:01
*** markvoelker has joined #openstack-nova06:04
*** tbachman has joined #openstack-nova06:08
*** tbachman_ has joined #openstack-nova06:14
*** tbachman has quit IRC06:15
*** tbachman_ is now known as tbachman06:15
*** macza has quit IRC06:28
*** tbachman has quit IRC06:29
*** bhagyashris_ has joined #openstack-nova06:40
*** bhagyashris_ has quit IRC06:49
*** Luzi has joined #openstack-nova06:56
*** wolverineav has quit IRC07:04
*** wolverineav has joined #openstack-nova07:05
*** pcaruana has joined #openstack-nova07:06
*** wolverineav has quit IRC07:09
*** moshele has joined #openstack-nova07:10
*** udesale has quit IRC07:17
*** udesale has joined #openstack-nova07:22
*** rcernin has quit IRC07:26
*** bhagyashris__ has joined #openstack-nova07:27
*** udesale has quit IRC07:27
*** udesale has joined #openstack-nova07:29
*** sapd1_x has quit IRC07:37
openstackgerritZhenyu Zheng proposed openstack/nova-specs master: Amend the detach-boot-volume design  https://review.openstack.org/61916107:38
*** Dinesh_Bhor has quit IRC07:38
*** Dinesh_Bhor has joined #openstack-nova07:39
*** itlinux has quit IRC07:48
*** sapd1_x has joined #openstack-nova07:54
*** lpetrut has joined #openstack-nova07:55
*** Dinesh_Bhor has quit IRC07:59
*** maciejjozefczyk has joined #openstack-nova08:04
*** slaweq has joined #openstack-nova08:13
*** rha has quit IRC08:16
*** hongbin has quit IRC08:23
*** sapd1_x has quit IRC08:24
*** sapd__x has joined #openstack-nova08:24
*** helenafm has joined #openstack-nova08:25
*** ralonsoh has joined #openstack-nova08:25
openstackgerritYikun Jiang proposed openstack/nova master: Per aggregate scheduling weight  https://review.openstack.org/62816308:33
*** avolkov has joined #openstack-nova08:46
gibistephenfin: left a question in https://review.openstack.org/#/c/62928108:58
*** rpittau has joined #openstack-nova09:06
*** ccamacho has joined #openstack-nova09:16
*** Dinesh_Bhor has joined #openstack-nova09:21
sean-k-mooneygibi: i replied to your question. i think is a safe assumtion to make if they have enabled ram oversubcription that the smallest pagesize on the host is swappable and therefor can be over subsribed09:23
sean-k-mooneythe linux kernel does not allow you to reserve all memory on a node for hugepages so the smallest pagesize will always be the system default small page size09:24
*** moshele has quit IRC09:24
*** markvoelker has quit IRC09:33
*** moshele has joined #openstack-nova09:37
*** derekh has joined #openstack-nova09:38
openstackgerritMaciej Jozefczyk proposed openstack/nova master: Add fill_virtual_interface_list online_data_migration script  https://review.openstack.org/61416709:40
openstackgerritMaciej Jozefczyk proposed openstack/nova master: Force refresh instance info_cache during heal  https://review.openstack.org/59160709:40
*** yan0s has joined #openstack-nova09:41
maciejjozefczyksean-k-mooney: I found a bug in https://review.openstack.org/#/c/61416709:41
gibisean-k-mooney: thanks for the explanation09:42
*** bhagyashris__ has quit IRC09:42
openstackgerritJan Gutter proposed openstack/nova master: Convert vrouter legacy plugging to os-vif  https://review.openstack.org/57132509:48
kashyapIs this too verbose a message?09:51
kashyap                msg = _("Setting both 'live_migration_tunnelled' and"09:51
kashyap                        "'live_migration_with_native_tls' at the same"09:51
kashyap                        "time is invalid.  If you have the relevant"09:51
kashyap                        "libvirt and QEMU versions, and have TLS"09:51
kashyap                        "configured in your environment, pick"09:51
kashyap                        "'live_migration_with_native_tls'.")09:51
kashyapstephenfin: ^09:53
kashyap(Or anyone interested in wordsmithing)09:54
gibikashyap: I like this message but I'm not a native English speaker09:55
kashyapI consider myself an "English geek" (although it's my 3rd language).  I went for maximum clarity for non-native speakers.09:56
kashyapBecause: "explicit is better than implicit".09:56
kashyapAnd I don't like error messages that tells me what is not valid (that I already see from the error); give me something actionable.09:56
kashyapI'll go with the above (with the second occurrence of "have" removed), and will see if anyone can come up w/ something better.09:57
sean-k-mooneykashyap: it reads fine to me. its hard to be too verbose when explaining that you have conflicting config options set09:59
sean-k-mooneyyou even provided a recommendation on what option to prefer so i looks good to me too09:59
kashyapOkido; thanks for the review.09:59
sean-k-mooneymaciejjozefczyk: was it related to the socket timeout or unrelated.09:59
kashyapAs I noted, my bar for error messages is: "give me something actionable"10:00
kashyapIt is really frustrating to just see "error: please figure it out"10:00
sean-k-mooneyyou mean like "no valid host"10:00
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Support native TLS for migration and disks over NBD  https://review.openstack.org/62521610:00
kashyapsean-k-mooney: Heh10:00
sean-k-mooneyits my favorite nova error message10:01
kashyapYeah, it's the most awful message.10:01
kashyapOne day, if I feel angry enough, I might clean them all up10:01
sean-k-mooneywell its availd error message but the issue is 90% of the time it because you hit the retry filter limit and you hit the retry filter because or a different error which was lost in the logs10:03
kashyapYeah, I recall that it's "best we could do"10:03
*** dtantsur|afk is now known as dtantsur10:10
*** Dinesh_Bhor has quit IRC10:17
*** panda|off is now known as panda10:18
*** jaosorior has quit IRC10:22
*** Dinesh_Bhor has joined #openstack-nova10:29
*** Dinesh_Bhor has quit IRC10:31
maciejjozefczyksean-k-mooney: nope, different topic10:35
maciejjozefczykalready fixed in new patch10:35
sean-k-mooneycool i had only tried the followon patch as i was testing on master so did not need the db change10:36
maciejjozefczykfor me I needed to update about 7k vifs only for one db10:38
*** moshele has quit IRC10:57
*** jaosorior has joined #openstack-nova10:59
*** nehaalhat has joined #openstack-nova11:07
nehaalhatgibi: Hi, in case of functional test cases, how nova sync with placement service? Do you any idea about this?11:08
gibinehaalhat: in functional test nova imports placement code via https://github.com/openstack/nova/blob/e3eb5f916580a9bab8f67b0fd685c6b3b23a97b7/nova/tests/functional/fixtures.py#L4111:11
*** maciejjozefczyk has quit IRC11:12
nehaalhatlike aggregates created in nova, how they are synced in placement service?11:12
*** maciejjozefczyk has joined #openstack-nova11:13
nehaalhatgibi: and do you have idea how to mock  a method from placement in nova functional test cases where placementfixture is used like: https://github.com/openstack/nova/blob/e3eb5f916580a9bab8f67b0fd685c6b3b23a97b7/nova/tests/functional/test_aggregates.py#L38511:22
*** sapd__x has quit IRC11:22
*** udesale has quit IRC11:24
*** dpawlik has joined #openstack-nova11:26
*** dpawlik has left #openstack-nova11:26
gibinehaalhat: do you have a patch with a functional test which doesn't work without mocking something in placement?11:31
gibinehaalhat: in general mocking in functional test is discuraged11:32
gibinehaalhat: but I can look at your patch to understand more11:32
*** erlon has joined #openstack-nova11:34
*** moshele has joined #openstack-nova11:34
nehaalhatgibi: No, i dont have a patch on community gerrit right now11:34
gibinehaalhat: let me try to dig out where nova sync the aggregates to placement...11:35
nehaalhatok11:35
gibinehaalhat: there is a nova-manage CLI command for aggregates sync https://github.com/openstack/nova/blob/da17e0ed8a9b9ddc79b6d2004e6fb941c6163fbf/nova/cmd/manage.py#L215811:36
gibinehaalhat: and this is the patch that introduced the automatic syncing https://review.openstack.org/#/c/553597/11:37
nehaalhatgibi: actually I want to add a FT under: https://github.com/openstack/nova/blob/e3eb5f916580a9bab8f67b0fd685c6b3b23a97b7/nova/tests/functional/test_aggregates.py#L385 where placement fixture is used, and I want to mock this method in my FT11:37
gibinehaalhat: and this is the point where nova adds a host to a placement aggregate https://review.openstack.org/#/c/553597/15/nova/compute/api.py@514711:38
gibinehaalhat: so I think if your functional test adds a host to an aggregate then nova will automatically add that host to the placement aggregate as weel11:38
gibiwell11:38
nehaalhatgibi: ok thanks11:39
gibinehaalhat: but if you push a wip patch to gerrit then I can help making it work11:40
*** rtjure has quit IRC11:40
*** erlon has quit IRC11:40
nehaalhatgibi: I want to add a FT under: https://github.com/openstack/nova/blob/e3eb5f916580a9bab8f67b0fd685c6b3b23a97b7/nova/tests/functional/test_aggregates.py#L385 where placement fixture is used11:41
nehaalhatand I want to mock this method in my FT : https://github.com/openstack/placement/blob/master/placement/util.py#L23611:41
nehaalhatactually I cant push on community gerrit without internal approval11:42
gibinehaalhat: why do you need to mock that?11:42
openstackgerritJan Gutter proposed openstack/nova master: Add support for vrouter HW datapath offloads  https://review.openstack.org/57208211:42
gibinehaalhat: what is the use case you want to tests?11:43
*** ygk_12345 has joined #openstack-nova11:46
ygk_12345hi all11:46
ygk_12345I am seeing this error on the compute node11:46
ygk_12345HypervisorUnavailable: Connection to the hypervisor is broken on host: compute-711:46
*** ratailor has quit IRC11:46
ygk_12345i cant do a virsh list command11:47
nehaalhatgibi: working on this feature: https://review.openstack.org/#/c/609960/3/specs/stein/approved/placement-req-filter-forbidden-aggregates.rst11:47
sean-k-mooneyygk_12345: that usually means the libvirtd daemon has exited11:47
ygk_12345sean-k-mooney: what does that mean ?11:47
*** dpawlik has joined #openstack-nova11:47
sean-k-mooneythe libvirtd service has crashed/stopped on the compute node11:47
jangutterygk_12345 what distro and which version are you running?11:48
ygk_12345jangutter: Ubuntu 16.04.3 LTS \n \l11:48
sean-k-mooneyygk_12345: try systemctl status libvirtd11:48
ygk_12345sean-k-mooney: I see this11:49
ygk_12345sean-k-mooney: 2019-01-09 11:48:44.365+0000: 16119: error : virPidFileAcquirePath:422 : Failed to acquire pid file '/var/run/libvirtd.pid': Resource temporarily unavailable11:49
ygk_12345sean-k-mooney: i did libvirtd -l11:49
sean-k-mooneyygk_12345: that will try to start a new libvirtd11:49
ygk_12345sean-k-mooney: Failed to open file '/proc/28288/stat': No such file or directory11:49
sean-k-mooneythat is not the same thing as runing status11:49
ygk_12345sean-k-mooney: the libvirtd is running finr though11:50
ygk_12345sean-k-mooney: failed to connect to monitor socket: No such process11:50
sean-k-mooneyyou said you cant run virsh command so its obviouly in an error state11:50
nehaalhatgibi: and this spec is dependent on https://review.openstack.org/#/c/603352/4/specs/stein/approved/negative-aggregate-membership.rst11:50
ygk_12345sean-k-mooney: virsh list command is hanging11:50
ygk_12345sean-k-mooney: no output11:50
gibinehaalhat: OK. Do you need to mock something from placement because the placement change that supports your feature is not merged yet?11:51
sean-k-mooneyyes because the libvirt daemon is not responding to requests11:51
nehaalhatgibi: yes11:51
*** f10d4 has joined #openstack-nova11:51
f10d4hi11:51
ygk_12345sean-k-mooney: now after sometime the vrish command is working but the flaoting IP of the instances is not pingable11:51
ygk_12345sean-k-mooney: it seems tot fix itself after sometime11:52
ygk_12345sean-k-mooney: bit strange11:52
sean-k-mooneyvirsh is just a client that publishes request to the libvirt daemon to execute commands via dbus11:52
jangutterygk_12345: Ubuntu 16.04 doesn't always use systemd: you might have to look at libvirt's logs directly. Check your system logs to see if everything's healthy or something isn't restarting all the time.11:52
gibinehaalhat: if you have both the placement change an the nova change locally then you have to instruct the nova test to use your local placement change for the functional test11:52
sean-k-mooneyjangutter: well the process could still be running but not respoinding on the libvirt contol socket also11:53
gibinehaalhat: when you will upload that to gerrit you can use a Depends-On tag in the commit message of the nova change that point to the placement change and the CI will act accordinlgy11:53
ygk_12345jangutter: i see some unable to connect to hyoervisor meaages from the nova in the syslog11:54
sean-k-mooneyygk_12345: your floating ip issue coudl be several things the first thing i would check is if the vm actully has a privte ip and can ping its router11:54
sean-k-mooneyif that works then you shoudl check the fip namespace and see if the dnat rules for the vm are present11:54
nehaalhatgibi: placement change yet to implemenetd by community members: https://review.openstack.org/#/c/603352/4/specs/stein/approved/negative-aggregate-membership.rst11:54
janguttersean-k-mooney: yeah, it could be many fun things. Hopefully there's logs.11:54
ygk_12345sean-k-mooney: jangutter usually rebooting the compute node fixes this issue. but that is not a viable solution always11:55
jangutterygk_12345: the nova message is not the root cause, it's a consequence of something else bad happening.11:55
ygk_12345jangutter: yes true11:55
gibinehaalhat: I'm affraid that without the placement feature exists you cannot make the final functional test in nova11:55
ygk_12345jangutter: unable to pinpoint it in the libvirtd11:55
nehaalhatgibi: ya that will not be final FT but for my patch I want to add FT and for that need to mock placement method11:57
sean-k-mooneyygk_12345: a less invasive way to correct it when it is broken is to restart libvirtd service and then restart the nova-compute agent11:57
gibinehaalhat: to run the nova functional test with local placement version, you can change tox.ini here https://github.com/openstack/nova/blob/master/tox.ini#L8711:57
gibinehaalhat: temporarly you can try to mock sure11:57
*** maciejjozefczyk has quit IRC11:57
sean-k-mooneystill if you can find the actully issue with libvirtd that is obviously better11:57
gibinehaalhat: I think the normal mock.patch() call should work in the functional tests11:57
ygk_12345sean-k-mooney: but even the restart of libvirts is not fixing the problem11:57
*** maciejjozefczyk has joined #openstack-nova11:57
jangutterygk_12345: you also should restart nova-compute11:57
jangutterygk_12345: after libvirtd is restarted11:58
nehaalhatgibi: yes I used normal mock.patch() but still the call goes to method, like: @mock.patch('placement.util.normalize_member_of_qs_param')11:58
sean-k-mooneyygk_12345: nova-compute could still have open file handels to the old libvirt socket so restarting nova-compute forces it to reopen them11:58
ygk_12345sean-k-mooney: ok11:59
*** maciejjozefczyk has quit IRC11:59
gibinehaalhat: give me a couple of minutes, I will try to mock it mysef.11:59
nehaalhatgibi: ok thank you11:59
*** rtjure has joined #openstack-nova12:00
jangutterWeird question: I submitted a review, and openstack-infra updated the link to the review with a broken link on a blueprint. Did I do something stupid?12:02
sean-k-mooneywhich revierw12:03
*** maciejjozefczyk has joined #openstack-nova12:03
*** ileixe has quit IRC12:04
jangutterIt added: Gerrit topic: https://review.openstack.org/#q,topic:bp/vrouter-os-vif-conversion,n,z where it should have added https://review.openstack.org/#/q/topic:bp/vrouter-hw-offloads,n,z12:04
janguttertriggered by: https://review.openstack.org/57132512:05
sean-k-mooneyi assume you used git review to submit it12:06
sean-k-mooneywhat is your local branch name12:06
sean-k-mooneythat is where it pulls the topic from if you dont pass -t12:07
alex_xusean-k-mooney: jaypipes, yonglihe why cpu_thread_policy' is libvirt specific? re:https://review.openstack.org/#/c/612256/20/specs/stein/approved/show-server-numa-topology.rst@10412:07
sean-k-mooneyalex_xu: no other virt dirive supports it12:07
jaypipesalex_xu: it isn't.12:07
jaypipesalex_xu: the one I mentioned was emulator_threads_policy.12:08
alex_xuno other virt driver support it, it doesn't mean libvirt specific, right?12:08
*** ygk_12345 has left #openstack-nova12:08
sean-k-mooneyjaypipes: cpu_thread_policy is only supported by libvirt also12:08
alex_xusean-k-mooney: not all the virt driver implement numa affinity also12:08
sean-k-mooneyalex_xu: yes but at least 2 implement it12:08
jaypipesalex_xu: "emulator" specifically refers to a libvirt/QEMU implementation detail :)12:08
janguttersean-k-mooney: local branch name is: review/jan_gutter/bp/vrouter-os-vif-conversion12:08
sean-k-mooneyhyperv and libvirt12:08
alex_xujangutter: yea, I agree with you that12:08
alex_xujangutter: sorry, ping the wrong name...12:09
sean-k-mooneyjangutter: yep that is why it create that topic upstream :)12:09
alex_xuhah12:09
janguttersean-k-mooney: aaaah.... sorry, I pasted the wrong link in.... let me repaste the correct link...12:10
*** erlon has joined #openstack-nova12:10
jangutterIt added: Gerrit topic: https://review.openstack.org/#q,topic:bp/vrouter-os-vif-conversion,n,z where it should have added https://review.openstack.org/#/q/topic:bp/vrouter-os-vif-conversion,n,z12:10
sean-k-mooneyjangutter: hum weird12:11
janguttersean-k-mooney: it picked the correct topic, but it generated a URL where "#q," should be "#/q/"12:11
janguttersean-k-mooney: maybe the link was valid on an older version of gerrit?12:12
alex_xujaypipes: we also should put it into the same level of 'nodes' field, right? sean-k-mooney mention that also12:12
alex_xuwe can't set cpu thread policy for each node12:12
janguttersean-k-mooney: sorry, just realised, I can probably ask this on #openstack-infra (facepalm).... it's been a looooong Monday.12:13
*** ygk_12345 has joined #openstack-nova12:13
sean-k-mooneyjangutter: hehe dont worry about it but they will likely have a better answer12:13
jangutteralex_xu: heh, no worries, I was confused for a bit, then I realised something autocompleted.12:13
ygk_12345sean-k-mooney: how to enable logging in libvirtd ?12:14
jaypipesalex_xu: is that a fundamental problem with the underlying modeling? in other words, is this: https://github.com/openstack/nova/blob/master/nova/objects/instance_numa_topology.py#L56 incorrectly placed on the NUMA cell instead of the InstanceNUMATopology object? :)12:14
alex_xujangutter: yea, I type tab too early :)12:14
*** wolverineav has joined #openstack-nova12:14
alex_xujaypipes: interesting...we put that attribute into cell obj12:15
alex_xujaypipes: but we only have one extra spec to set cpu policy for the whole guest12:15
sean-k-mooneyjaypipes well there is no fundemental resone we could not do this per cell but we only support setting for the entire instnace12:15
jaypipessean-k-mooney: overkill? :)12:16
sean-k-mooneytotally12:16
sean-k-mooneyi mean at that point we might as well let people chose the cpus12:17
sean-k-mooneyi mean that is what they really wanted all along...12:17
sean-k-mooneyjaypipes: my guess would be we store it in the instance cell to not need to pass it as a paramater to fuctions later when calulating the pinning or some other reason liek that12:18
alex_xuactually those policy can easy to get from the flavor extra spec12:18
sean-k-mooneyalex_xu: yes12:19
*** wolverineav has quit IRC12:19
sean-k-mooneyalex_xu: wasing the main point of this spec that it would be reading the numa/cpu related extra specs and image metadata and presenting the compined view to the admin12:19
sean-k-mooney*wasn't12:20
*** rpittau is now known as rpittau|lunch12:20
alex_xuat least I don't want to the api user to parse the extra spec, and copy the parse rule which inside nova12:20
alex_xuand yes, we have image meta for those policy also12:20
jaypipesalex_xu, sean-k-mooney, yonglihe: easy to get sidetracked here. my bottom line recommendation: a) don't put emulator_threads_policy in the HTTP response AND b) put the cpu_threads_policy as an attribute at the same level in the HTTP response as "nodes" (like sean-k-mooney and alex_xu suggest)12:21
sean-k-mooneyjaypipes: sure that sounds resonable12:21
alex_xujaypipes: cool12:22
sean-k-mooneyjaypipes: alex_xu what is the process for specless blueprint by the way12:27
sean-k-mooneyjaypipes: alex_xu specifcally for https://blueprints.launchpad.net/nova/+spec/libvirt-disable-pmu-for-realtime-guests12:27
*** tiendc has quit IRC12:27
alex_xusean-k-mooney: get an approve in nova meeting? I guess12:28
janguttersean-k-mooney: Last time I proposed one, it went up as an agenda item on the Nova meeting.12:28
sean-k-mooneyya i was going to add it for this weeks12:28
*** udesale has joined #openstack-nova12:28
sean-k-mooneyif it gets punted to next cycle it wont kill me but it was a simple addtion i was going to work on after m2 as a 1 patch feature12:29
*** cdent has joined #openstack-nova12:35
*** erlon has quit IRC12:37
openstackgerritMerged openstack/nova-specs master: Fix format flavor-extra-spec-image-property-validation  https://review.openstack.org/62931112:41
gibinehaalhat: I was able to mock the placement function you need http://paste.openstack.org/show/740847/12:41
*** sapd__x has joined #openstack-nova12:42
nehaalhatgibi: ok12:44
kashyapgibi: Hi there; when you get a moment: I've answered your question on the review.  What Martin posted on the review is the successful test evidence.12:46
openstackgerritChris Dent proposed openstack/nova master: Remove placement from nova install docs  https://review.openstack.org/62818412:47
openstackgerritChris Dent proposed openstack/nova master: Remove link to placement configuration from nova config docs  https://review.openstack.org/62818512:47
openstackgerritChris Dent proposed openstack/nova master: Remove placement from contributor doc  https://review.openstack.org/62818612:47
kashyapgibi: The test results posted by Martin's matches my own (which I also posted in the review).12:47
*** sapd__x has quit IRC12:48
kashyapThanks for reviewing!12:48
gibikashyap: ohh, then I have to read a bit more carfeully :)12:48
kashyapHeh, no problem at all.  It is low-level QMP log; so not everyone can make sense of it.  I just added context as to what it means in the bug comment and the review12:49
kashyapI didn't spell out my own results in the review explicitly; I merely _linked_ to several files.  And I did the comparison of Martin's test with mine manually.12:50
gibikashyap: that context helps :) I'm +2 on that patch12:50
*** erlon has joined #openstack-nova12:52
kashyapThanks a ton!12:55
kashyapgibi: I'm getting the follow-up docs patch; just need to rST-ify my .txt file12:55
gibikashyap: I thank you for the effort12:55
kashyapgibi: I'm hoping melwitt or someone can ACK the blueprint today / this week -- https://blueprints.launchpad.net/nova/+spec/support-qemu-native-tls-for-live-migration12:56
gibikashyap: could you please add it to the nova meeting agenda? that way I can make sure we won't forget it12:57
kashyapgibi: Definitely; let me go find the agenda page12:57
gibikashyap: https://wiki.openstack.org/wiki/Meetings/Nova12:58
kashyapMeanwhile, I added it to also to: https://etherpad.openstack.org/p/nova-stein-blueprint-status12:58
kashyapThanks!12:58
gibikashyap: cool13:01
kashyapAlright, added it to the agenda (as part of: "Open discussion").13:04
gibithanks13:07
*** erlon has quit IRC13:09
*** dave-mccowan has joined #openstack-nova13:19
*** sridharg has quit IRC13:20
*** rpittau|lunch is now known as rpittau13:21
*** erlon has joined #openstack-nova13:25
openstackgerritPaul-Emile Element proposed openstack/os-traits master: Add new traits for TPM support  https://review.openstack.org/62929913:28
*** f10d4 has left #openstack-nova13:32
*** rha has joined #openstack-nova13:47
*** whoami-rajat has quit IRC13:52
*** ygk_12345 has left #openstack-nova13:52
*** moshele has quit IRC13:57
*** maciejjozefczyk has left #openstack-nova13:58
*** sapd__x has joined #openstack-nova14:03
*** munimeha1 has joined #openstack-nova14:13
openstackgerritYongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology"  https://review.openstack.org/61225614:18
yongliheJay, alex_xu, sean, numa info spec revised. thank you, guys.14:22
yonglihe'show-server-group' spec, https://review.openstack.org/#/c/612255/11,  this is quite small, hope it could make it.14:23
*** avolkov has quit IRC14:25
*** janki has quit IRC14:26
openstackgerritIury Gregory Melo Ferreira proposed openstack/nova master: Rename Ironic jobs  https://review.openstack.org/62950414:26
sean-k-mooneytwo nits inline but i think the current version is largely fine.14:27
sean-k-mooneyyonglihe: ^%14:27
*** janki has joined #openstack-nova14:27
*** whoami-rajat has joined #openstack-nova14:31
*** burt has joined #openstack-nova14:35
jaypipesyonglihe: done14:40
yonglihegot, working on that. Jay, thanks.14:43
openstackgerritYongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology"  https://review.openstack.org/61225614:43
yonglihesean, updated. thanks.14:43
*** janki has quit IRC14:47
*** janki has joined #openstack-nova14:47
*** sara_nierodz_ has joined #openstack-nova14:51
*** janki has quit IRC14:51
*** janki has joined #openstack-nova14:52
*** sara_nierodz_ has left #openstack-nova14:53
*** janki has quit IRC14:53
*** janki has joined #openstack-nova14:53
*** janki has quit IRC14:54
*** janki has joined #openstack-nova14:55
*** janki has quit IRC14:57
*** janki has joined #openstack-nova14:57
openstackgerritYongli He proposed openstack/nova-specs master: add 'show-server-group' spec  https://review.openstack.org/61225514:59
*** janki has quit IRC14:59
yongliheJay Pipes, done15:00
dpawlikHello, could you review: https://review.openstack.org/#/c/628956/ and https://review.openstack.org/#/c/628128/ ? thanks15:02
*** awaugama has joined #openstack-nova15:03
sean-k-mooneydpawlik: does are queens stable backprot lyarwood ^ that might be something you might be interested in15:04
*** chason_ has joined #openstack-nova15:09
*** tbachman has joined #openstack-nova15:18
*** Luzi has quit IRC15:19
*** hongbin has joined #openstack-nova15:22
*** mlavalle has joined #openstack-nova15:31
*** udesale has quit IRC15:34
*** dpawlik has quit IRC15:35
*** lpetrut has quit IRC15:37
*** wwriverrat has quit IRC15:38
*** purplerbot has quit IRC15:38
*** med_ has joined #openstack-nova15:39
*** purplerbot has joined #openstack-nova15:46
openstackgerritYikun Jiang proposed openstack/nova master: Per aggregate scheduling weight  https://review.openstack.org/62816315:50
*** wolverineav has joined #openstack-nova15:51
*** wolverineav has quit IRC15:55
*** purplerbot has quit IRC15:58
*** purplerbot has joined #openstack-nova15:59
*** hongbin has quit IRC16:03
*** pcaruana has quit IRC16:03
*** gyee has joined #openstack-nova16:18
*** cfriesen has joined #openstack-nova16:24
melwittjaypipes: hey, I dunno if you saw my reply on the counting quotas from placement spec https://review.openstack.org/#/c/509042/ but I think we can't consider removing user-specific quotas until we migrate to keystone limits + oslo.limit16:24
*** artom has joined #openstack-nova16:24
*** moshele has joined #openstack-nova16:24
jaypipesmelwitt: ack, and I wasn't suggesting removing them yet, just that we should plan to just use keystone and the hierarchical projects stuff in future.16:25
melwittjaypipes: ah, ok. yeah, that is the plan when we move to keystone limits + oslo.limits user-specific quotas will be dropped as part of that. described in johnthetubaguy's spec if you're interested https://review.openstack.org/#/c/602201/16:28
cfriesenso there's a review up for the os-traits portion of the emulated TPM stuff.  Am I correct that we'd need a new version of os-traits to pick that up in nova?16:31
jaypipesmelwitt: ++16:32
*** moshele has quit IRC16:32
jaypipescfriesen: yes.16:32
jaypipescfriesen: and an updated requirements.txt in nova.16:32
cfriesenright.  so what's the process for triggering the new package version?  is that something we need to request formally somewhere?16:33
melwittcfriesen: you can request a release of the package via the openstack/releases repo, like this https://review.openstack.org/#/c/612843/16:34
gibimelwitt: hi! kashyap has a bp https://blueprints.launchpad.net/nova/+spec/support-qemu-native-tls-for-live-migration that hasn't been approved but it seems it was discussed before. Do you remember if anything blocks that bp? The implementation seems to be ready16:35
*** wwriverrat has joined #openstack-nova16:36
melwittgibi: yeah, I think everyone is OK with the feature but the discussion was around whether it needed a spec or not. IIRC dansmith suggested the docs and references related to the feature be proposed along with the implementation, which I thought was a good idea,16:37
gibimelwitt: kashyap prepares a followup patch top of the implementation that adds documentation how to set up the compute hosts for tls16:37
*** helenafm has quit IRC16:38
melwittkashyap said he's done that and the bp is on the agenda for the nova meeting tomorrow, where we can get additional ACKs for approving it as specless16:38
*** rnoriega has quit IRC16:38
*** macza has joined #openstack-nova16:38
*** radez has quit IRC16:38
dansmithwhere is the docs patch? it doesn't seem to be in the same series16:38
gibimelwitt: OK, ACKing on the nova meeting works for me16:38
*** kashyap` has joined #openstack-nova16:38
gibidansmith: patch is not up yet, a perliminary doc is here https://kashyapc.fedorapeople.org/Native-TLS/Setup-for-NBD-and-migration-streams-over-TLS.rst.txt16:38
kashyap`gibi: Actually it is 95% complete doc :-).  Converting to rST as we speak...16:39
*** wznoinsk has quit IRC16:39
dansmithgibi: erm, okay, not sure why we wouldn't just have that in a follow-on patch so we can see what that's going to look like :)16:39
dansmithkashyap`: ^16:39
*** med_ has quit IRC16:39
kashyap`dansmith: Hiya, yeah.  Follow-up patch indeed16:39
kashyap`Not going to smush it into the first patch16:39
*** awaugama has quit IRC16:40
*** kashyap has quit IRC16:40
dansmithkashyap: can you have that in gerrit today so we can see it rendered and stuff ahead of the meeting tomorrow?16:40
kashyap`dansmith: Actually gibi said it as much -- a follow-on -- on the review.16:40
gibidansmith: sure. I just didn't thought that having the doc patch up is a prerequisit to approve the bp16:40
kashyap`Yeap16:40
dansmithgibi: that was the original answer, IIRC :)16:40
dansmithnote that tomorrow is the early meeting so I won't be around anyway16:40
kashyap`dansmith: If it soothes your nerves, FWIW, Martin Schuppert from TripleO independently tested this patch and showed me evidence16:41
kashyap`(Which matches mine.)16:41
dansmithkashyap`: nope, all I want to see is docs :)16:41
gibidansmith: :)16:42
kashyap`dansmith: Sure.  It's already there :-)  Getting it polished a bit16:42
*** rnoriega has joined #openstack-nova16:43
melwittah right, tomorrow's meeting is at 6am for us so I won't be around yet either16:43
kashyap`The high-level thing is: (a) Installer needs to setup all the TLS env; (b) Only _then_ this feature makes sense.16:43
dansmithmelwitt: yeah, so let's just see the docs rendered in a patch, and then we can sign off on the blueprint, independent of the meeting16:44
melwittdansmith: sure, sounds like a plan16:44
melwittgibi ^16:45
melwittthanks for helping to get this sorted out16:46
gibimelwitt: works for me16:46
gibimelwitt: I've reviewed the implementation too and looks good to me16:46
melwittgreat, thanks for that16:46
melwittgibi: a note on the bandwidth aware scheduling hangout meeting, I think mriedem had said he'd rather not have the meeting on the 14th, his first day back, if that is possible16:49
gibimelwitt: sure, I can move that to Tuesday if that is better16:50
gibimelwitt: what do you think, 17:00 UTC would work for the team?16:50
*** imacdonn has quit IRC16:51
*** imacdonn has joined #openstack-nova16:51
melwittgibi: he didn't mention specific favorable days, but I expect tuesday should be better. and I expect 1700 UTC should work16:51
gibimelwitt: thanks, I will send and update to the ML16:52
melwittok, thanks16:52
kashyap`Thanks, folks!16:53
kashyap`I'll get some dinner, and then get the rendering going.  I'm writing some Nova-specific context.  Because most people would use installer tools to take care of all the CA setup16:54
melwittok, thanks kashyap`16:54
*** kashyap` is now known as kashyap16:54
*** med_ has joined #openstack-nova17:00
*** itlinux has joined #openstack-nova17:06
*** _alastor1 is now known as _alastor_17:07
*** rpittau has quit IRC17:08
*** dtantsur is now known as dtantsur|afk17:11
*** moshele has joined #openstack-nova17:19
*** moshele has quit IRC17:28
openstackgerritsean mooney proposed openstack/nova master: [WIP] libvirt: auto detach/attach sriov ports on migration  https://review.openstack.org/62958917:31
openstackgerritStephen Finucane proposed openstack/nova stable/queens: Fail to live migration if instance has a NUMA topology  https://review.openstack.org/62959717:41
*** wolverineav has joined #openstack-nova17:44
*** wolverineav has quit IRC17:49
*** igordc has joined #openstack-nova17:56
*** derekh has quit IRC17:56
sean-k-mooneyadrianc: by the way this https://review.openstack.org/#/c/609166/4 is not needed an i can abandon it right18:11
adriancsean-k-mooney: yep, you can abandon18:14
sean-k-mooneycool its replaced by the code in https://review.openstack.org/#/c/619929/718:14
*** awaugama has joined #openstack-nova18:15
kashyapAny rST gurus here, what is wrong here?18:15
kashyapAssuming a doc called 'live-migration-usage.rst' exists, if I do: "Refer to the :doc:`live-migration-usage`"  from another .rst doc, it should Just Work?18:15
sean-k-mooneykashyap: i just ask stephenfin18:16
sean-k-mooneykashyap: im not sure if you need to use a relivie path18:16
stephenfinkashyap: the path is relative, so if it's in the same directory, yes18:16
stephenfinotherwise get those dots out18:16
adriancsean-k-mooney: yes, also noticed you added a commit for the direct mode, ill take a look tomorrow18:16
*** dpawlik has joined #openstack-nova18:16
stephenfinadrianc: been reviewing all those. On the last one18:16
stephenfinadrianc: Mostly +2 except for minor changes, mostly nits18:17
stephenfinjust fyi :)18:17
kashyapstephenfin: You mean get those colons out, if it's (it is) in the same dir?18:17
sean-k-mooneyadrianc: ya its still a work in progress ill need to test it properly tomorrow and then add unit tests and a release note18:17
adriancstephenfin: :) great!18:17
kashyapIs this the `diff`?18:17
kashyap- Refer to the :doc:`live-migration-usage`18:17
stephenfinkashyap: heh, no. :doc:`../live-migration-usage` (if not same directory)18:17
kashyap+ Refer to the doc`live-migration-usage`18:17
stephenfin:doc:`path` is correct18:18
kashyapIt _is_ in the same dir18:18
stephenfinthen you're good18:18
kashyapstephenfin: Hmm, but look:18:18
kashyap$> rst2html secure-live-migration-with-qemu-native-tls.rst foo.html18:18
kashyapsecure-live-migration-with-qemu-native-tls.rst:138: (ERROR/3) Unknown interpreted text role "doc".18:18
kashyapThat's the _only_ error between me and dinner.18:19
stephenfindoc8?18:19
*** yan0s has quit IRC18:19
stephenfinIgnore it, if so18:19
stephenfindoc8 uses docutils, which doesn't know about doctrees and therefore doesn't have a doc role18:19
stephenfindoctrees are a purely Sphinx thing18:19
kashyapAah, so it should be fine.18:20
stephenfinyup, all good18:20
kashyapThanks!18:21
*** wolverineav has joined #openstack-nova18:27
*** wolverineav has quit IRC18:30
*** wolverineav has joined #openstack-nova18:31
*** lpetrut has joined #openstack-nova18:33
*** itlinux has quit IRC18:36
*** lpetrut has quit IRC18:37
*** itlinux has joined #openstack-nova18:38
openstackgerritKashyap Chamarthy proposed openstack/nova master: docs: Secure live migration with QEMU-native TLS  https://review.openstack.org/62962718:43
*** macza has quit IRC18:44
*** macza has joined #openstack-nova18:45
kashyapmelwitt: dansmith ^ First cut18:45
* kashyap gets dinner now18:45
dansmithcool18:45
melwittkashyap: thanks18:46
dansmithI expect a review from stephenfin on that too18:46
dansmithwhich will yield a $number_of_lines count of nit changes18:46
dansmithI found an issue in the first line already :P18:47
kashyapHaha, yeah.  Happy to get stephenfin's rST eyes18:47
kashyapdansmith: What is it?18:47
dansmithkashyap: you have to wait :)18:47
kashyapNerves are wracking, dammit.  I go better eat18:48
*** awaugama_ has joined #openstack-nova18:58
*** awaugama has quit IRC19:00
*** cdent has quit IRC19:09
*** panda has quit IRC19:19
*** panda has joined #openstack-nova19:21
openstackgerritInbar Stolberg proposed openstack/nova master: Author: Inbar Stolberg  https://review.openstack.org/62964619:23
*** awaugama_ has quit IRC19:29
*** awaugama has joined #openstack-nova19:30
*** ralonsoh has quit IRC19:40
dansmith kashyap fails the build19:42
dansmithsome index thing, I'm sure stephenfin knows19:43
*** moshele has joined #openstack-nova19:50
melwittsean-k-mooney: I'm pretty sure anything that needs to add a flavor extra spec needs a spec https://blueprints.launchpad.net/nova/+spec/libvirt-disable-pmu-for-realtime-guests19:54
melwittany other opinions?19:54
*** efried has joined #openstack-nova20:06
*** dpawlik has quit IRC20:08
*** munimeha1 has quit IRC20:34
*** slaweq has quit IRC20:37
*** awaugama has quit IRC20:39
*** efried has quit IRC20:40
*** hongbin has joined #openstack-nova20:42
*** markvoelker has joined #openstack-nova20:45
*** slaweq has joined #openstack-nova20:53
*** ociuhandu has joined #openstack-nova21:13
*** moshele has quit IRC21:29
*** ociuhandu has quit IRC21:39
*** takashin has joined #openstack-nova21:53
openstackgerritInbar Stolberg proposed openstack/nova master: Author: Inbar Stolberg  https://review.openstack.org/62964621:56
*** ccamacho has quit IRC22:08
*** ccamacho has joined #openstack-nova22:27
*** ccamacho has quit IRC22:28
*** tbachman has quit IRC22:30
openstackgerritJack Ding proposed openstack/nova master: [WIP] Flavor extra spec and image properties validation  https://review.openstack.org/62070622:33
*** tbachman has joined #openstack-nova22:35
*** hongbin has quit IRC22:40
openstackgerritYongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology"  https://review.openstack.org/61225622:41
*** rcernin has joined #openstack-nova22:52
*** hongbin has joined #openstack-nova22:52
*** hongbin has quit IRC22:52
*** hongbin has joined #openstack-nova22:54
*** markvoelker has quit IRC22:57
*** markvoelker has joined #openstack-nova22:57
*** macza has quit IRC22:59
*** macza has joined #openstack-nova22:59
openstackgerritJack Ding proposed openstack/nova master: [WIP] Flavor extra spec and image properties validation  https://review.openstack.org/62070622:59
*** markvoelker has quit IRC23:02
openstackgerritInbar Stolberg proposed openstack/nova master: Fixes server rebuild if NumaTopologyFilter is used with CPU pinning  https://review.openstack.org/62964623:03
openstackgerritJack Ding proposed openstack/nova master: [WIP] Flavor extra spec and image properties validation  https://review.openstack.org/62070623:03
*** whoami-rajat has quit IRC23:04
*** itlinux has quit IRC23:06
*** erlon has quit IRC23:06
*** mlavalle has quit IRC23:16
*** hongbin has quit IRC23:52
*** _alastor_ has quit IRC23:59
« Tuesday, 2019-01-08 Index Thursday, 2019-01-10 »

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