*** munimeha1 has quit IRC | 00:00 | |
openstackgerrit | Hongbin Lu proposed openstack/nova-specs master: Scheduling support for SR-IOV bond https://review.openstack.org/626055 | 00:03 |
---|---|---|
*** rcernin has quit IRC | 00:07 | |
*** rcernin has joined #openstack-nova | 00:09 | |
*** slaweq has quit IRC | 00:18 | |
*** artom has joined #openstack-nova | 00:23 | |
*** artom has quit IRC | 00:23 | |
*** macza has quit IRC | 00:28 | |
*** owalsh_ has joined #openstack-nova | 00:33 | |
*** mlavalle has quit IRC | 00:34 | |
*** slaweq has joined #openstack-nova | 00:35 | |
*** owalsh has quit IRC | 00:36 | |
openstackgerrit | Merged openstack/nova stable/rocky: Fail to live migration if instance has a NUMA topology https://review.openstack.org/625880 | 00:36 |
*** liuyulong has quit IRC | 00:39 | |
*** Swami has quit IRC | 00:46 | |
*** slaweq has quit IRC | 00:47 | |
*** ileixe has joined #openstack-nova | 00:50 | |
*** ileixe has quit IRC | 00:51 | |
*** tetsuro has joined #openstack-nova | 00:53 | |
*** ileixe has joined #openstack-nova | 01:01 | |
*** itlinux has joined #openstack-nova | 01:03 | |
*** erlon has joined #openstack-nova | 01:15 | |
*** slaweq has joined #openstack-nova | 01:18 | |
*** itlinux has quit IRC | 01:19 | |
*** tiendc has joined #openstack-nova | 01:20 | |
*** erlon has quit IRC | 01:22 | |
*** owalsh_ is now known as owalsh | 01:23 | |
*** slaweq has quit IRC | 01:23 | |
*** Kevin_Zheng has joined #openstack-nova | 01:30 | |
*** tetsuro has quit IRC | 01:41 | |
*** hongbin_ has joined #openstack-nova | 01:48 | |
*** xek has quit IRC | 01:51 | |
*** macza has joined #openstack-nova | 01:53 | |
*** macza has quit IRC | 01:57 | |
openstackgerrit | Merged openstack/nova stable/rocky: Imported Translations from Zanata https://review.openstack.org/628109 | 02:06 |
openstackgerrit | Merged openstack/nova stable/rocky: Fix a broken link https://review.openstack.org/626798 | 02:06 |
openstackgerrit | Merged openstack/nova stable/rocky: Removing pip-missing-reqs from default tox jobs https://review.openstack.org/628952 | 02:06 |
openstackgerrit | Merged openstack/nova stable/rocky: Fix destination_type attribute in the bdm_v2 documentation https://review.openstack.org/626874 | 02:06 |
*** igordc has quit IRC | 02:06 | |
*** igordc has joined #openstack-nova | 02:06 | |
*** igordc has quit IRC | 02:08 | |
*** efried has quit IRC | 02:09 | |
*** igordc has joined #openstack-nova | 02:10 | |
*** xek has joined #openstack-nova | 02:12 | |
*** itlinux has joined #openstack-nova | 02:12 | |
*** xek has quit IRC | 02:12 | |
*** igordc has quit IRC | 02:12 | |
*** xek has joined #openstack-nova | 02:13 | |
*** sapd1_x has joined #openstack-nova | 02:13 | |
*** itlinux has quit IRC | 02:13 | |
takashin | 02:25 | |
*** psachin has joined #openstack-nova | 02:48 | |
*** wolverineav has quit IRC | 02:48 | |
*** wolverineav has joined #openstack-nova | 02:48 | |
*** whoami-rajat has joined #openstack-nova | 02:52 | |
*** wolverineav has quit IRC | 02:54 | |
*** wolverineav has joined #openstack-nova | 02:55 | |
*** wolverineav has quit IRC | 02:57 | |
*** mhen has quit IRC | 02:58 | |
*** wolverineav has joined #openstack-nova | 02:59 | |
*** mhen has joined #openstack-nova | 02:59 | |
*** wolverineav has quit IRC | 03:03 | |
*** brinzhang has joined #openstack-nova | 03:05 | |
*** gyee has quit IRC | 03:10 | |
*** markvoelker has joined #openstack-nova | 03:11 | |
*** markvoelker has quit IRC | 03:12 | |
*** brinzhang has quit IRC | 03:31 | |
*** itlinux has joined #openstack-nova | 03:33 | |
*** tbachman has quit IRC | 03:37 | |
*** janki has joined #openstack-nova | 03:45 | |
*** spsurya has joined #openstack-nova | 03:46 | |
*** Dinesh_Bhor has joined #openstack-nova | 03:51 | |
*** Dinesh_Bhor has quit IRC | 03:59 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova stable/queens: Add description of custom resource classes https://review.openstack.org/619125 | 04:07 |
openstackgerrit | Takashi NATSUME proposed openstack/nova stable/queens: Add description of custom resource classes https://review.openstack.org/619125 | 04:07 |
*** macza has joined #openstack-nova | 04:16 | |
*** hongbin_ has quit IRC | 04:24 | |
*** sridharg has joined #openstack-nova | 04:27 | |
*** udesale has joined #openstack-nova | 04:27 | |
*** ileixe has quit IRC | 04:31 | |
*** markvoelker has joined #openstack-nova | 04:37 | |
*** markvoelker has quit IRC | 04:42 | |
*** takashin has quit IRC | 04:45 | |
*** wolverineav has joined #openstack-nova | 04:45 | |
*** wwriverrat has joined #openstack-nova | 04:52 | |
*** lifeless_ has joined #openstack-nova | 04:57 | |
*** wolverineav has quit IRC | 05:00 | |
*** ileixe has joined #openstack-nova | 05:01 | |
*** sridharg has quit IRC | 05:02 | |
*** psachin has quit IRC | 05:02 | |
*** sapd1_x has quit IRC | 05:02 | |
*** sapd1_ has quit IRC | 05:02 | |
*** chason has quit IRC | 05:02 | |
*** lifeless has quit IRC | 05:02 | |
*** Dinesh_Bhor has joined #openstack-nova | 05:04 | |
*** sapd1_x has joined #openstack-nova | 05:08 | |
*** sridharg has joined #openstack-nova | 05:08 | |
*** sapd1_ has joined #openstack-nova | 05:10 | |
*** Dinesh_Bhor has quit IRC | 05:12 | |
*** wolverineav has joined #openstack-nova | 05:15 | |
*** med_ has quit IRC | 05:23 | |
*** takashin has joined #openstack-nova | 05:28 | |
*** Dinesh_Bhor has joined #openstack-nova | 05:30 | |
*** takashin has left #openstack-nova | 05:30 | |
*** udesale has quit IRC | 05:32 | |
*** udesale has joined #openstack-nova | 05:40 | |
*** ratailor has joined #openstack-nova | 06:01 | |
*** markvoelker has joined #openstack-nova | 06:04 | |
*** tbachman has joined #openstack-nova | 06:08 | |
*** tbachman_ has joined #openstack-nova | 06:14 | |
*** tbachman has quit IRC | 06:15 | |
*** tbachman_ is now known as tbachman | 06:15 | |
*** macza has quit IRC | 06:28 | |
*** tbachman has quit IRC | 06:29 | |
*** bhagyashris_ has joined #openstack-nova | 06:40 | |
*** bhagyashris_ has quit IRC | 06:49 | |
*** Luzi has joined #openstack-nova | 06:56 | |
*** wolverineav has quit IRC | 07:04 | |
*** wolverineav has joined #openstack-nova | 07:05 | |
*** pcaruana has joined #openstack-nova | 07:06 | |
*** wolverineav has quit IRC | 07:09 | |
*** moshele has joined #openstack-nova | 07:10 | |
*** udesale has quit IRC | 07:17 | |
*** udesale has joined #openstack-nova | 07:22 | |
*** rcernin has quit IRC | 07:26 | |
*** bhagyashris__ has joined #openstack-nova | 07:27 | |
*** udesale has quit IRC | 07:27 | |
*** udesale has joined #openstack-nova | 07:29 | |
*** sapd1_x has quit IRC | 07:37 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Amend the detach-boot-volume design https://review.openstack.org/619161 | 07:38 |
*** Dinesh_Bhor has quit IRC | 07:38 | |
*** Dinesh_Bhor has joined #openstack-nova | 07:39 | |
*** itlinux has quit IRC | 07:48 | |
*** sapd1_x has joined #openstack-nova | 07:54 | |
*** lpetrut has joined #openstack-nova | 07:55 | |
*** Dinesh_Bhor has quit IRC | 07:59 | |
*** maciejjozefczyk has joined #openstack-nova | 08:04 | |
*** slaweq has joined #openstack-nova | 08:13 | |
*** rha has quit IRC | 08:16 | |
*** hongbin has quit IRC | 08:23 | |
*** sapd1_x has quit IRC | 08:24 | |
*** sapd__x has joined #openstack-nova | 08:24 | |
*** helenafm has joined #openstack-nova | 08:25 | |
*** ralonsoh has joined #openstack-nova | 08:25 | |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Per aggregate scheduling weight https://review.openstack.org/628163 | 08:33 |
*** avolkov has joined #openstack-nova | 08:46 | |
gibi | stephenfin: left a question in https://review.openstack.org/#/c/629281 | 08:58 |
*** rpittau has joined #openstack-nova | 09:06 | |
*** ccamacho has joined #openstack-nova | 09:16 | |
*** Dinesh_Bhor has joined #openstack-nova | 09:21 | |
sean-k-mooney | gibi: 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 subsribed | 09:23 |
sean-k-mooney | the 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 size | 09:24 |
*** moshele has quit IRC | 09:24 | |
*** markvoelker has quit IRC | 09:33 | |
*** moshele has joined #openstack-nova | 09:37 | |
*** derekh has joined #openstack-nova | 09:38 | |
openstackgerrit | Maciej Jozefczyk proposed openstack/nova master: Add fill_virtual_interface_list online_data_migration script https://review.openstack.org/614167 | 09:40 |
openstackgerrit | Maciej Jozefczyk proposed openstack/nova master: Force refresh instance info_cache during heal https://review.openstack.org/591607 | 09:40 |
*** yan0s has joined #openstack-nova | 09:41 | |
maciejjozefczyk | sean-k-mooney: I found a bug in https://review.openstack.org/#/c/614167 | 09:41 |
gibi | sean-k-mooney: thanks for the explanation | 09:42 |
*** bhagyashris__ has quit IRC | 09:42 | |
openstackgerrit | Jan Gutter proposed openstack/nova master: Convert vrouter legacy plugging to os-vif https://review.openstack.org/571325 | 09:48 |
kashyap | Is 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 |
kashyap | stephenfin: ^ | 09:53 |
kashyap | (Or anyone interested in wordsmithing) | 09:54 |
gibi | kashyap: I like this message but I'm not a native English speaker | 09:55 |
kashyap | I consider myself an "English geek" (although it's my 3rd language). I went for maximum clarity for non-native speakers. | 09:56 |
kashyap | Because: "explicit is better than implicit". | 09:56 |
kashyap | And 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 |
kashyap | I'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-mooney | kashyap: it reads fine to me. its hard to be too verbose when explaining that you have conflicting config options set | 09:59 |
sean-k-mooney | you even provided a recommendation on what option to prefer so i looks good to me too | 09:59 |
kashyap | Okido; thanks for the review. | 09:59 |
sean-k-mooney | maciejjozefczyk: was it related to the socket timeout or unrelated. | 09:59 |
kashyap | As I noted, my bar for error messages is: "give me something actionable" | 10:00 |
kashyap | It is really frustrating to just see "error: please figure it out" | 10:00 |
sean-k-mooney | you mean like "no valid host" | 10:00 |
openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Support native TLS for migration and disks over NBD https://review.openstack.org/625216 | 10:00 |
kashyap | sean-k-mooney: Heh | 10:00 |
sean-k-mooney | its my favorite nova error message | 10:01 |
kashyap | Yeah, it's the most awful message. | 10:01 |
kashyap | One day, if I feel angry enough, I might clean them all up | 10:01 |
sean-k-mooney | well 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 logs | 10:03 |
kashyap | Yeah, I recall that it's "best we could do" | 10:03 |
*** dtantsur|afk is now known as dtantsur | 10:10 | |
*** Dinesh_Bhor has quit IRC | 10:17 | |
*** panda|off is now known as panda | 10:18 | |
*** jaosorior has quit IRC | 10:22 | |
*** Dinesh_Bhor has joined #openstack-nova | 10:29 | |
*** Dinesh_Bhor has quit IRC | 10:31 | |
maciejjozefczyk | sean-k-mooney: nope, different topic | 10:35 |
maciejjozefczyk | already fixed in new patch | 10:35 |
sean-k-mooney | cool i had only tried the followon patch as i was testing on master so did not need the db change | 10:36 |
maciejjozefczyk | for me I needed to update about 7k vifs only for one db | 10:38 |
*** moshele has quit IRC | 10:57 | |
*** jaosorior has joined #openstack-nova | 10:59 | |
*** nehaalhat has joined #openstack-nova | 11:07 | |
nehaalhat | gibi: Hi, in case of functional test cases, how nova sync with placement service? Do you any idea about this? | 11:08 |
gibi | nehaalhat: in functional test nova imports placement code via https://github.com/openstack/nova/blob/e3eb5f916580a9bab8f67b0fd685c6b3b23a97b7/nova/tests/functional/fixtures.py#L41 | 11:11 |
*** maciejjozefczyk has quit IRC | 11:12 | |
nehaalhat | like aggregates created in nova, how they are synced in placement service? | 11:12 |
*** maciejjozefczyk has joined #openstack-nova | 11:13 | |
nehaalhat | gibi: 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#L385 | 11:22 |
*** sapd__x has quit IRC | 11:22 | |
*** udesale has quit IRC | 11:24 | |
*** dpawlik has joined #openstack-nova | 11:26 | |
*** dpawlik has left #openstack-nova | 11:26 | |
gibi | nehaalhat: do you have a patch with a functional test which doesn't work without mocking something in placement? | 11:31 |
gibi | nehaalhat: in general mocking in functional test is discuraged | 11:32 |
gibi | nehaalhat: but I can look at your patch to understand more | 11:32 |
*** erlon has joined #openstack-nova | 11:34 | |
*** moshele has joined #openstack-nova | 11:34 | |
nehaalhat | gibi: No, i dont have a patch on community gerrit right now | 11:34 |
gibi | nehaalhat: let me try to dig out where nova sync the aggregates to placement... | 11:35 |
nehaalhat | ok | 11:35 |
gibi | nehaalhat: there is a nova-manage CLI command for aggregates sync https://github.com/openstack/nova/blob/da17e0ed8a9b9ddc79b6d2004e6fb941c6163fbf/nova/cmd/manage.py#L2158 | 11:36 |
gibi | nehaalhat: and this is the patch that introduced the automatic syncing https://review.openstack.org/#/c/553597/ | 11:37 |
nehaalhat | gibi: 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 FT | 11:37 |
gibi | nehaalhat: 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@5147 | 11:38 |
gibi | nehaalhat: 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 weel | 11:38 |
gibi | well | 11:38 |
nehaalhat | gibi: ok thanks | 11:39 |
gibi | nehaalhat: but if you push a wip patch to gerrit then I can help making it work | 11:40 |
*** rtjure has quit IRC | 11:40 | |
*** erlon has quit IRC | 11:40 | |
nehaalhat | gibi: 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 | 11:41 |
nehaalhat | and I want to mock this method in my FT : https://github.com/openstack/placement/blob/master/placement/util.py#L236 | 11:41 |
nehaalhat | actually I cant push on community gerrit without internal approval | 11:42 |
gibi | nehaalhat: why do you need to mock that? | 11:42 |
openstackgerrit | Jan Gutter proposed openstack/nova master: Add support for vrouter HW datapath offloads https://review.openstack.org/572082 | 11:42 |
gibi | nehaalhat: what is the use case you want to tests? | 11:43 |
*** ygk_12345 has joined #openstack-nova | 11:46 | |
ygk_12345 | hi all | 11:46 |
ygk_12345 | I am seeing this error on the compute node | 11:46 |
ygk_12345 | HypervisorUnavailable: Connection to the hypervisor is broken on host: compute-7 | 11:46 |
*** ratailor has quit IRC | 11:46 | |
ygk_12345 | i cant do a virsh list command | 11:47 |
nehaalhat | gibi: working on this feature: https://review.openstack.org/#/c/609960/3/specs/stein/approved/placement-req-filter-forbidden-aggregates.rst | 11:47 |
sean-k-mooney | ygk_12345: that usually means the libvirtd daemon has exited | 11:47 |
ygk_12345 | sean-k-mooney: what does that mean ? | 11:47 |
*** dpawlik has joined #openstack-nova | 11:47 | |
sean-k-mooney | the libvirtd service has crashed/stopped on the compute node | 11:47 |
jangutter | ygk_12345 what distro and which version are you running? | 11:48 |
ygk_12345 | jangutter: Ubuntu 16.04.3 LTS \n \l | 11:48 |
sean-k-mooney | ygk_12345: try systemctl status libvirtd | 11:48 |
ygk_12345 | sean-k-mooney: I see this | 11:49 |
ygk_12345 | sean-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 unavailable | 11:49 |
ygk_12345 | sean-k-mooney: i did libvirtd -l | 11:49 |
sean-k-mooney | ygk_12345: that will try to start a new libvirtd | 11:49 |
ygk_12345 | sean-k-mooney: Failed to open file '/proc/28288/stat': No such file or directory | 11:49 |
sean-k-mooney | that is not the same thing as runing status | 11:49 |
ygk_12345 | sean-k-mooney: the libvirtd is running finr though | 11:50 |
ygk_12345 | sean-k-mooney: failed to connect to monitor socket: No such process | 11:50 |
sean-k-mooney | you said you cant run virsh command so its obviouly in an error state | 11:50 |
nehaalhat | gibi: and this spec is dependent on https://review.openstack.org/#/c/603352/4/specs/stein/approved/negative-aggregate-membership.rst | 11:50 |
ygk_12345 | sean-k-mooney: virsh list command is hanging | 11:50 |
ygk_12345 | sean-k-mooney: no output | 11:50 |
gibi | nehaalhat: 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-mooney | yes because the libvirt daemon is not responding to requests | 11:51 |
nehaalhat | gibi: yes | 11:51 |
*** f10d4 has joined #openstack-nova | 11:51 | |
f10d4 | hi | 11:51 |
ygk_12345 | sean-k-mooney: now after sometime the vrish command is working but the flaoting IP of the instances is not pingable | 11:51 |
ygk_12345 | sean-k-mooney: it seems tot fix itself after sometime | 11:52 |
ygk_12345 | sean-k-mooney: bit strange | 11:52 |
sean-k-mooney | virsh is just a client that publishes request to the libvirt daemon to execute commands via dbus | 11:52 |
jangutter | ygk_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 |
gibi | nehaalhat: 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 test | 11:52 |
sean-k-mooney | jangutter: well the process could still be running but not respoinding on the libvirt contol socket also | 11:53 |
gibi | nehaalhat: 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 accordinlgy | 11:53 |
ygk_12345 | jangutter: i see some unable to connect to hyoervisor meaages from the nova in the syslog | 11:54 |
sean-k-mooney | ygk_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 router | 11:54 |
sean-k-mooney | if that works then you shoudl check the fip namespace and see if the dnat rules for the vm are present | 11:54 |
nehaalhat | gibi: placement change yet to implemenetd by community members: https://review.openstack.org/#/c/603352/4/specs/stein/approved/negative-aggregate-membership.rst | 11:54 |
jangutter | sean-k-mooney: yeah, it could be many fun things. Hopefully there's logs. | 11:54 |
ygk_12345 | sean-k-mooney: jangutter usually rebooting the compute node fixes this issue. but that is not a viable solution always | 11:55 |
jangutter | ygk_12345: the nova message is not the root cause, it's a consequence of something else bad happening. | 11:55 |
ygk_12345 | jangutter: yes true | 11:55 |
gibi | nehaalhat: I'm affraid that without the placement feature exists you cannot make the final functional test in nova | 11:55 |
ygk_12345 | jangutter: unable to pinpoint it in the libvirtd | 11:55 |
nehaalhat | gibi: ya that will not be final FT but for my patch I want to add FT and for that need to mock placement method | 11:57 |
sean-k-mooney | ygk_12345: a less invasive way to correct it when it is broken is to restart libvirtd service and then restart the nova-compute agent | 11:57 |
gibi | nehaalhat: 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#L87 | 11:57 |
gibi | nehaalhat: temporarly you can try to mock sure | 11:57 |
*** maciejjozefczyk has quit IRC | 11:57 | |
sean-k-mooney | still if you can find the actully issue with libvirtd that is obviously better | 11:57 |
gibi | nehaalhat: I think the normal mock.patch() call should work in the functional tests | 11:57 |
ygk_12345 | sean-k-mooney: but even the restart of libvirts is not fixing the problem | 11:57 |
*** maciejjozefczyk has joined #openstack-nova | 11:57 | |
jangutter | ygk_12345: you also should restart nova-compute | 11:57 |
jangutter | ygk_12345: after libvirtd is restarted | 11:58 |
nehaalhat | gibi: 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-mooney | ygk_12345: nova-compute could still have open file handels to the old libvirt socket so restarting nova-compute forces it to reopen them | 11:58 |
ygk_12345 | sean-k-mooney: ok | 11:59 |
*** maciejjozefczyk has quit IRC | 11:59 | |
gibi | nehaalhat: give me a couple of minutes, I will try to mock it mysef. | 11:59 |
nehaalhat | gibi: ok thank you | 11:59 |
*** rtjure has joined #openstack-nova | 12:00 | |
jangutter | Weird 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-mooney | which revierw | 12:03 |
*** maciejjozefczyk has joined #openstack-nova | 12:03 | |
*** ileixe has quit IRC | 12:04 | |
jangutter | It 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,z | 12:04 |
jangutter | triggered by: https://review.openstack.org/571325 | 12:05 |
sean-k-mooney | i assume you used git review to submit it | 12:06 |
sean-k-mooney | what is your local branch name | 12:06 |
sean-k-mooney | that is where it pulls the topic from if you dont pass -t | 12:07 |
alex_xu | sean-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@104 | 12:07 |
sean-k-mooney | alex_xu: no other virt dirive supports it | 12:07 |
jaypipes | alex_xu: it isn't. | 12:07 |
jaypipes | alex_xu: the one I mentioned was emulator_threads_policy. | 12:08 |
alex_xu | no other virt driver support it, it doesn't mean libvirt specific, right? | 12:08 |
*** ygk_12345 has left #openstack-nova | 12:08 | |
sean-k-mooney | jaypipes: cpu_thread_policy is only supported by libvirt also | 12:08 |
alex_xu | sean-k-mooney: not all the virt driver implement numa affinity also | 12:08 |
sean-k-mooney | alex_xu: yes but at least 2 implement it | 12:08 |
jaypipes | alex_xu: "emulator" specifically refers to a libvirt/QEMU implementation detail :) | 12:08 |
jangutter | sean-k-mooney: local branch name is: review/jan_gutter/bp/vrouter-os-vif-conversion | 12:08 |
sean-k-mooney | hyperv and libvirt | 12:08 |
alex_xu | jangutter: yea, I agree with you that | 12:08 |
alex_xu | jangutter: sorry, ping the wrong name... | 12:09 |
sean-k-mooney | jangutter: yep that is why it create that topic upstream :) | 12:09 |
alex_xu | hah | 12:09 |
jangutter | sean-k-mooney: aaaah.... sorry, I pasted the wrong link in.... let me repaste the correct link... | 12:10 |
*** erlon has joined #openstack-nova | 12:10 | |
jangutter | It 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,z | 12:10 |
sean-k-mooney | jangutter: hum weird | 12:11 |
jangutter | sean-k-mooney: it picked the correct topic, but it generated a URL where "#q," should be "#/q/" | 12:11 |
jangutter | sean-k-mooney: maybe the link was valid on an older version of gerrit? | 12:12 |
alex_xu | jaypipes: we also should put it into the same level of 'nodes' field, right? sean-k-mooney mention that also | 12:12 |
alex_xu | we can't set cpu thread policy for each node | 12:12 |
jangutter | sean-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-nova | 12:13 | |
sean-k-mooney | jangutter: hehe dont worry about it but they will likely have a better answer | 12:13 |
jangutter | alex_xu: heh, no worries, I was confused for a bit, then I realised something autocompleted. | 12:13 |
ygk_12345 | sean-k-mooney: how to enable logging in libvirtd ? | 12:14 |
jaypipes | alex_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_xu | jangutter: yea, I type tab too early :) | 12:14 |
*** wolverineav has joined #openstack-nova | 12:14 | |
alex_xu | jaypipes: interesting...we put that attribute into cell obj | 12:15 |
alex_xu | jaypipes: but we only have one extra spec to set cpu policy for the whole guest | 12:15 |
sean-k-mooney | jaypipes well there is no fundemental resone we could not do this per cell but we only support setting for the entire instnace | 12:15 |
jaypipes | sean-k-mooney: overkill? :) | 12:16 |
sean-k-mooney | totally | 12:16 |
sean-k-mooney | i mean at that point we might as well let people chose the cpus | 12:17 |
sean-k-mooney | i mean that is what they really wanted all along... | 12:17 |
sean-k-mooney | jaypipes: 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 that | 12:18 |
alex_xu | actually those policy can easy to get from the flavor extra spec | 12:18 |
sean-k-mooney | alex_xu: yes | 12:19 |
*** wolverineav has quit IRC | 12:19 | |
sean-k-mooney | alex_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 admin | 12:19 |
sean-k-mooney | *wasn't | 12:20 |
*** rpittau is now known as rpittau|lunch | 12:20 | |
alex_xu | at least I don't want to the api user to parse the extra spec, and copy the parse rule which inside nova | 12:20 |
alex_xu | and yes, we have image meta for those policy also | 12:20 |
jaypipes | alex_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-mooney | jaypipes: sure that sounds resonable | 12:21 |
alex_xu | jaypipes: cool | 12:22 |
sean-k-mooney | jaypipes: alex_xu what is the process for specless blueprint by the way | 12:27 |
sean-k-mooney | jaypipes: alex_xu specifcally for https://blueprints.launchpad.net/nova/+spec/libvirt-disable-pmu-for-realtime-guests | 12:27 |
*** tiendc has quit IRC | 12:27 | |
alex_xu | sean-k-mooney: get an approve in nova meeting? I guess | 12:28 |
jangutter | sean-k-mooney: Last time I proposed one, it went up as an agenda item on the Nova meeting. | 12:28 |
sean-k-mooney | ya i was going to add it for this weeks | 12:28 |
*** udesale has joined #openstack-nova | 12:28 | |
sean-k-mooney | if 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 feature | 12:29 |
*** cdent has joined #openstack-nova | 12:35 | |
*** erlon has quit IRC | 12:37 | |
openstackgerrit | Merged openstack/nova-specs master: Fix format flavor-extra-spec-image-property-validation https://review.openstack.org/629311 | 12:41 |
gibi | nehaalhat: I was able to mock the placement function you need http://paste.openstack.org/show/740847/ | 12:41 |
*** sapd__x has joined #openstack-nova | 12:42 | |
nehaalhat | gibi: ok | 12:44 |
kashyap | gibi: 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 |
openstackgerrit | Chris Dent proposed openstack/nova master: Remove placement from nova install docs https://review.openstack.org/628184 | 12:47 |
openstackgerrit | Chris Dent proposed openstack/nova master: Remove link to placement configuration from nova config docs https://review.openstack.org/628185 | 12:47 |
openstackgerrit | Chris Dent proposed openstack/nova master: Remove placement from contributor doc https://review.openstack.org/628186 | 12:47 |
kashyap | gibi: The test results posted by Martin's matches my own (which I also posted in the review). | 12:47 |
*** sapd__x has quit IRC | 12:48 | |
kashyap | Thanks for reviewing! | 12:48 |
gibi | kashyap: ohh, then I have to read a bit more carfeully :) | 12:48 |
kashyap | Heh, 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 review | 12:49 |
kashyap | I 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 |
gibi | kashyap: that context helps :) I'm +2 on that patch | 12:50 |
*** erlon has joined #openstack-nova | 12:52 | |
kashyap | Thanks a ton! | 12:55 |
kashyap | gibi: I'm getting the follow-up docs patch; just need to rST-ify my .txt file | 12:55 |
gibi | kashyap: I thank you for the effort | 12:55 |
kashyap | gibi: 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-migration | 12:56 |
gibi | kashyap: could you please add it to the nova meeting agenda? that way I can make sure we won't forget it | 12:57 |
kashyap | gibi: Definitely; let me go find the agenda page | 12:57 |
gibi | kashyap: https://wiki.openstack.org/wiki/Meetings/Nova | 12:58 |
kashyap | Meanwhile, I added it to also to: https://etherpad.openstack.org/p/nova-stein-blueprint-status | 12:58 |
kashyap | Thanks! | 12:58 |
gibi | kashyap: cool | 13:01 |
kashyap | Alright, added it to the agenda (as part of: "Open discussion"). | 13:04 |
gibi | thanks | 13:07 |
*** erlon has quit IRC | 13:09 | |
*** dave-mccowan has joined #openstack-nova | 13:19 | |
*** sridharg has quit IRC | 13:20 | |
*** rpittau|lunch is now known as rpittau | 13:21 | |
*** erlon has joined #openstack-nova | 13:25 | |
openstackgerrit | Paul-Emile Element proposed openstack/os-traits master: Add new traits for TPM support https://review.openstack.org/629299 | 13:28 |
*** f10d4 has left #openstack-nova | 13:32 | |
*** rha has joined #openstack-nova | 13:47 | |
*** whoami-rajat has quit IRC | 13:52 | |
*** ygk_12345 has left #openstack-nova | 13:52 | |
*** moshele has quit IRC | 13:57 | |
*** maciejjozefczyk has left #openstack-nova | 13:58 | |
*** sapd__x has joined #openstack-nova | 14:03 | |
*** munimeha1 has joined #openstack-nova | 14:13 | |
openstackgerrit | Yongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology" https://review.openstack.org/612256 | 14:18 |
yonglihe | Jay, 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 IRC | 14:25 | |
*** janki has quit IRC | 14:26 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/nova master: Rename Ironic jobs https://review.openstack.org/629504 | 14:26 |
sean-k-mooney | two nits inline but i think the current version is largely fine. | 14:27 |
sean-k-mooney | yonglihe: ^% | 14:27 |
*** janki has joined #openstack-nova | 14:27 | |
*** whoami-rajat has joined #openstack-nova | 14:31 | |
*** burt has joined #openstack-nova | 14:35 | |
jaypipes | yonglihe: done | 14:40 |
yonglihe | got, working on that. Jay, thanks. | 14:43 |
openstackgerrit | Yongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology" https://review.openstack.org/612256 | 14:43 |
yonglihe | sean, updated. thanks. | 14:43 |
*** janki has quit IRC | 14:47 | |
*** janki has joined #openstack-nova | 14:47 | |
*** sara_nierodz_ has joined #openstack-nova | 14:51 | |
*** janki has quit IRC | 14:51 | |
*** janki has joined #openstack-nova | 14:52 | |
*** sara_nierodz_ has left #openstack-nova | 14:53 | |
*** janki has quit IRC | 14:53 | |
*** janki has joined #openstack-nova | 14:53 | |
*** janki has quit IRC | 14:54 | |
*** janki has joined #openstack-nova | 14:55 | |
*** janki has quit IRC | 14:57 | |
*** janki has joined #openstack-nova | 14:57 | |
openstackgerrit | Yongli He proposed openstack/nova-specs master: add 'show-server-group' spec https://review.openstack.org/612255 | 14:59 |
*** janki has quit IRC | 14:59 | |
yonglihe | Jay Pipes, done | 15:00 |
dpawlik | Hello, could you review: https://review.openstack.org/#/c/628956/ and https://review.openstack.org/#/c/628128/ ? thanks | 15:02 |
*** awaugama has joined #openstack-nova | 15:03 | |
sean-k-mooney | dpawlik: does are queens stable backprot lyarwood ^ that might be something you might be interested in | 15:04 |
*** chason_ has joined #openstack-nova | 15:09 | |
*** tbachman has joined #openstack-nova | 15:18 | |
*** Luzi has quit IRC | 15:19 | |
*** hongbin has joined #openstack-nova | 15:22 | |
*** mlavalle has joined #openstack-nova | 15:31 | |
*** udesale has quit IRC | 15:34 | |
*** dpawlik has quit IRC | 15:35 | |
*** lpetrut has quit IRC | 15:37 | |
*** wwriverrat has quit IRC | 15:38 | |
*** purplerbot has quit IRC | 15:38 | |
*** med_ has joined #openstack-nova | 15:39 | |
*** purplerbot has joined #openstack-nova | 15:46 | |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Per aggregate scheduling weight https://review.openstack.org/628163 | 15:50 |
*** wolverineav has joined #openstack-nova | 15:51 | |
*** wolverineav has quit IRC | 15:55 | |
*** purplerbot has quit IRC | 15:58 | |
*** purplerbot has joined #openstack-nova | 15:59 | |
*** hongbin has quit IRC | 16:03 | |
*** pcaruana has quit IRC | 16:03 | |
*** gyee has joined #openstack-nova | 16:18 | |
*** cfriesen has joined #openstack-nova | 16:24 | |
melwitt | jaypipes: 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.limit | 16:24 |
*** artom has joined #openstack-nova | 16:24 | |
*** moshele has joined #openstack-nova | 16:24 | |
jaypipes | melwitt: 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 |
melwitt | jaypipes: 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 |
cfriesen | so 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 |
jaypipes | melwitt: ++ | 16:32 |
*** moshele has quit IRC | 16:32 | |
jaypipes | cfriesen: yes. | 16:32 |
jaypipes | cfriesen: and an updated requirements.txt in nova. | 16:32 |
cfriesen | right. so what's the process for triggering the new package version? is that something we need to request formally somewhere? | 16:33 |
melwitt | cfriesen: you can request a release of the package via the openstack/releases repo, like this https://review.openstack.org/#/c/612843/ | 16:34 |
gibi | melwitt: 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 ready | 16:35 |
*** wwriverrat has joined #openstack-nova | 16:36 | |
melwitt | gibi: 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 |
gibi | melwitt: kashyap prepares a followup patch top of the implementation that adds documentation how to set up the compute hosts for tls | 16:37 |
*** helenafm has quit IRC | 16:38 | |
melwitt | kashyap 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 specless | 16:38 |
*** rnoriega has quit IRC | 16:38 | |
*** macza has joined #openstack-nova | 16:38 | |
*** radez has quit IRC | 16:38 | |
dansmith | where is the docs patch? it doesn't seem to be in the same series | 16:38 |
gibi | melwitt: OK, ACKing on the nova meeting works for me | 16:38 |
*** kashyap` has joined #openstack-nova | 16:38 | |
gibi | dansmith: 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.txt | 16:38 |
kashyap` | gibi: Actually it is 95% complete doc :-). Converting to rST as we speak... | 16:39 |
*** wznoinsk has quit IRC | 16:39 | |
dansmith | gibi: 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 |
dansmith | kashyap`: ^ | 16:39 |
*** med_ has quit IRC | 16:39 | |
kashyap` | dansmith: Hiya, yeah. Follow-up patch indeed | 16:39 |
kashyap` | Not going to smush it into the first patch | 16:39 |
*** awaugama has quit IRC | 16:40 | |
*** kashyap has quit IRC | 16:40 | |
dansmith | kashyap: 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 |
gibi | dansmith: sure. I just didn't thought that having the doc patch up is a prerequisit to approve the bp | 16:40 |
kashyap` | Yeap | 16:40 |
dansmith | gibi: that was the original answer, IIRC :) | 16:40 |
dansmith | note that tomorrow is the early meeting so I won't be around anyway | 16:40 |
kashyap` | dansmith: If it soothes your nerves, FWIW, Martin Schuppert from TripleO independently tested this patch and showed me evidence | 16:41 |
kashyap` | (Which matches mine.) | 16:41 |
dansmith | kashyap`: nope, all I want to see is docs :) | 16:41 |
gibi | dansmith: :) | 16:42 |
kashyap` | dansmith: Sure. It's already there :-) Getting it polished a bit | 16:42 |
*** rnoriega has joined #openstack-nova | 16:43 | |
melwitt | ah right, tomorrow's meeting is at 6am for us so I won't be around yet either | 16: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 |
dansmith | melwitt: yeah, so let's just see the docs rendered in a patch, and then we can sign off on the blueprint, independent of the meeting | 16:44 |
melwitt | dansmith: sure, sounds like a plan | 16:44 |
melwitt | gibi ^ | 16:45 |
melwitt | thanks for helping to get this sorted out | 16:46 |
gibi | melwitt: works for me | 16:46 |
gibi | melwitt: I've reviewed the implementation too and looks good to me | 16:46 |
melwitt | great, thanks for that | 16:46 |
melwitt | gibi: 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 possible | 16:49 |
gibi | melwitt: sure, I can move that to Tuesday if that is better | 16:50 |
gibi | melwitt: what do you think, 17:00 UTC would work for the team? | 16:50 |
*** imacdonn has quit IRC | 16:51 | |
*** imacdonn has joined #openstack-nova | 16:51 | |
melwitt | gibi: he didn't mention specific favorable days, but I expect tuesday should be better. and I expect 1700 UTC should work | 16:51 |
gibi | melwitt: thanks, I will send and update to the ML | 16:52 |
melwitt | ok, thanks | 16: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 setup | 16:54 |
melwitt | ok, thanks kashyap` | 16:54 |
*** kashyap` is now known as kashyap | 16:54 | |
*** med_ has joined #openstack-nova | 17:00 | |
*** itlinux has joined #openstack-nova | 17:06 | |
*** _alastor1 is now known as _alastor_ | 17:07 | |
*** rpittau has quit IRC | 17:08 | |
*** dtantsur is now known as dtantsur|afk | 17:11 | |
*** moshele has joined #openstack-nova | 17:19 | |
*** moshele has quit IRC | 17:28 | |
openstackgerrit | sean mooney proposed openstack/nova master: [WIP] libvirt: auto detach/attach sriov ports on migration https://review.openstack.org/629589 | 17:31 |
openstackgerrit | Stephen Finucane proposed openstack/nova stable/queens: Fail to live migration if instance has a NUMA topology https://review.openstack.org/629597 | 17:41 |
*** wolverineav has joined #openstack-nova | 17:44 | |
*** wolverineav has quit IRC | 17:49 | |
*** igordc has joined #openstack-nova | 17:56 | |
*** derekh has quit IRC | 17:56 | |
sean-k-mooney | adrianc: by the way this https://review.openstack.org/#/c/609166/4 is not needed an i can abandon it right | 18:11 |
adrianc | sean-k-mooney: yep, you can abandon | 18:14 |
sean-k-mooney | cool its replaced by the code in https://review.openstack.org/#/c/619929/7 | 18:14 |
*** awaugama has joined #openstack-nova | 18:15 | |
kashyap | Any rST gurus here, what is wrong here? | 18:15 |
kashyap | Assuming 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-mooney | kashyap: i just ask stephenfin | 18:16 |
sean-k-mooney | kashyap: im not sure if you need to use a relivie path | 18:16 |
stephenfin | kashyap: the path is relative, so if it's in the same directory, yes | 18:16 |
stephenfin | otherwise get those dots out | 18:16 |
adrianc | sean-k-mooney: yes, also noticed you added a commit for the direct mode, ill take a look tomorrow | 18:16 |
*** dpawlik has joined #openstack-nova | 18:16 | |
stephenfin | adrianc: been reviewing all those. On the last one | 18:16 |
stephenfin | adrianc: Mostly +2 except for minor changes, mostly nits | 18:17 |
stephenfin | just fyi :) | 18:17 |
kashyap | stephenfin: You mean get those colons out, if it's (it is) in the same dir? | 18:17 |
sean-k-mooney | adrianc: ya its still a work in progress ill need to test it properly tomorrow and then add unit tests and a release note | 18:17 |
adrianc | stephenfin: :) great! | 18:17 |
kashyap | Is this the `diff`? | 18:17 |
kashyap | - Refer to the :doc:`live-migration-usage` | 18:17 |
stephenfin | kashyap: 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 correct | 18:18 |
kashyap | It _is_ in the same dir | 18:18 |
stephenfin | then you're good | 18:18 |
kashyap | stephenfin: Hmm, but look: | 18:18 |
kashyap | $> rst2html secure-live-migration-with-qemu-native-tls.rst foo.html | 18:18 |
kashyap | secure-live-migration-with-qemu-native-tls.rst:138: (ERROR/3) Unknown interpreted text role "doc". | 18:18 |
kashyap | That's the _only_ error between me and dinner. | 18:19 |
stephenfin | doc8? | 18:19 |
*** yan0s has quit IRC | 18:19 | |
stephenfin | Ignore it, if so | 18:19 |
stephenfin | doc8 uses docutils, which doesn't know about doctrees and therefore doesn't have a doc role | 18:19 |
stephenfin | doctrees are a purely Sphinx thing | 18:19 |
kashyap | Aah, so it should be fine. | 18:20 |
stephenfin | yup, all good | 18:20 |
kashyap | Thanks! | 18:21 |
*** wolverineav has joined #openstack-nova | 18:27 | |
*** wolverineav has quit IRC | 18:30 | |
*** wolverineav has joined #openstack-nova | 18:31 | |
*** lpetrut has joined #openstack-nova | 18:33 | |
*** itlinux has quit IRC | 18:36 | |
*** lpetrut has quit IRC | 18:37 | |
*** itlinux has joined #openstack-nova | 18:38 | |
openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: docs: Secure live migration with QEMU-native TLS https://review.openstack.org/629627 | 18:43 |
*** macza has quit IRC | 18:44 | |
*** macza has joined #openstack-nova | 18:45 | |
kashyap | melwitt: dansmith ^ First cut | 18:45 |
* kashyap gets dinner now | 18:45 | |
dansmith | cool | 18:45 |
melwitt | kashyap: thanks | 18:46 |
dansmith | I expect a review from stephenfin on that too | 18:46 |
dansmith | which will yield a $number_of_lines count of nit changes | 18:46 |
dansmith | I found an issue in the first line already :P | 18:47 |
kashyap | Haha, yeah. Happy to get stephenfin's rST eyes | 18:47 |
kashyap | dansmith: What is it? | 18:47 |
dansmith | kashyap: you have to wait :) | 18:47 |
kashyap | Nerves are wracking, dammit. I go better eat | 18:48 |
*** awaugama_ has joined #openstack-nova | 18:58 | |
*** awaugama has quit IRC | 19:00 | |
*** cdent has quit IRC | 19:09 | |
*** panda has quit IRC | 19:19 | |
*** panda has joined #openstack-nova | 19:21 | |
openstackgerrit | Inbar Stolberg proposed openstack/nova master: Author: Inbar Stolberg https://review.openstack.org/629646 | 19:23 |
*** awaugama_ has quit IRC | 19:29 | |
*** awaugama has joined #openstack-nova | 19:30 | |
*** ralonsoh has quit IRC | 19:40 | |
dansmith | kashyap fails the build | 19:42 |
dansmith | some index thing, I'm sure stephenfin knows | 19:43 |
*** moshele has joined #openstack-nova | 19:50 | |
melwitt | sean-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-guests | 19:54 |
melwitt | any other opinions? | 19:54 |
*** efried has joined #openstack-nova | 20:06 | |
*** dpawlik has quit IRC | 20:08 | |
*** munimeha1 has quit IRC | 20:34 | |
*** slaweq has quit IRC | 20:37 | |
*** awaugama has quit IRC | 20:39 | |
*** efried has quit IRC | 20:40 | |
*** hongbin has joined #openstack-nova | 20:42 | |
*** markvoelker has joined #openstack-nova | 20:45 | |
*** slaweq has joined #openstack-nova | 20:53 | |
*** ociuhandu has joined #openstack-nova | 21:13 | |
*** moshele has quit IRC | 21:29 | |
*** ociuhandu has quit IRC | 21:39 | |
*** takashin has joined #openstack-nova | 21:53 | |
openstackgerrit | Inbar Stolberg proposed openstack/nova master: Author: Inbar Stolberg https://review.openstack.org/629646 | 21:56 |
*** ccamacho has quit IRC | 22:08 | |
*** ccamacho has joined #openstack-nova | 22:27 | |
*** ccamacho has quit IRC | 22:28 | |
*** tbachman has quit IRC | 22:30 | |
openstackgerrit | Jack Ding proposed openstack/nova master: [WIP] Flavor extra spec and image properties validation https://review.openstack.org/620706 | 22:33 |
*** tbachman has joined #openstack-nova | 22:35 | |
*** hongbin has quit IRC | 22:40 | |
openstackgerrit | Yongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology" https://review.openstack.org/612256 | 22:41 |
*** rcernin has joined #openstack-nova | 22:52 | |
*** hongbin has joined #openstack-nova | 22:52 | |
*** hongbin has quit IRC | 22:52 | |
*** hongbin has joined #openstack-nova | 22:54 | |
*** markvoelker has quit IRC | 22:57 | |
*** markvoelker has joined #openstack-nova | 22:57 | |
*** macza has quit IRC | 22:59 | |
*** macza has joined #openstack-nova | 22:59 | |
openstackgerrit | Jack Ding proposed openstack/nova master: [WIP] Flavor extra spec and image properties validation https://review.openstack.org/620706 | 22:59 |
*** markvoelker has quit IRC | 23:02 | |
openstackgerrit | Inbar Stolberg proposed openstack/nova master: Fixes server rebuild if NumaTopologyFilter is used with CPU pinning https://review.openstack.org/629646 | 23:03 |
openstackgerrit | Jack Ding proposed openstack/nova master: [WIP] Flavor extra spec and image properties validation https://review.openstack.org/620706 | 23:03 |
*** whoami-rajat has quit IRC | 23:04 | |
*** itlinux has quit IRC | 23:06 | |
*** erlon has quit IRC | 23:06 | |
*** mlavalle has quit IRC | 23:16 | |
*** hongbin has quit IRC | 23:52 | |
*** _alastor_ has quit IRC | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!