*** artom has joined #openstack-nova | 00:08 | |
openstackgerrit | Merged openstack/nova master: doc: Add links to novaclient contributor guide https://review.opendev.org/669028 | 00:27 |
---|---|---|
*** tbachman has quit IRC | 00:30 | |
*** brinzhang_ has joined #openstack-nova | 00:34 | |
*** brinzhang has quit IRC | 00:37 | |
*** jangutter has quit IRC | 00:42 | |
openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Make the '--config-drive' be a boolean option https://review.opendev.org/653683 | 00:46 |
*** TxGirlGeek has joined #openstack-nova | 00:48 | |
*** tbachman has joined #openstack-nova | 00:52 | |
*** mlavalle has quit IRC | 00:53 | |
*** tbachman_ has joined #openstack-nova | 00:56 | |
*** tbachman has quit IRC | 00:57 | |
*** tbachman_ is now known as tbachman | 00:57 | |
*** igordc has quit IRC | 01:01 | |
*** tbachman has quit IRC | 01:01 | |
*** bbowen has quit IRC | 01:02 | |
*** bbowen has joined #openstack-nova | 01:02 | |
*** gyee has quit IRC | 01:14 | |
*** imacdonn has quit IRC | 01:15 | |
*** imacdonn has joined #openstack-nova | 01:16 | |
*** yedongcan has joined #openstack-nova | 01:18 | |
*** spatel has joined #openstack-nova | 01:27 | |
*** hongda has joined #openstack-nova | 01:59 | |
openstackgerrit | Ming Jin proposed openstack/nova master: Nova handle empty vcpu_pin_set scenario https://review.opendev.org/670935 | 02:23 |
*** tkajinam has quit IRC | 02:23 | |
*** tkajinam has joined #openstack-nova | 02:24 | |
openstackgerrit | Yongli He proposed openstack/python-novaclient master: Microversion 2.75 - show server topology https://review.opendev.org/670790 | 02:25 |
*** hemna has quit IRC | 02:30 | |
*** hongda has quit IRC | 02:38 | |
*** whoami-rajat has joined #openstack-nova | 02:42 | |
*** takashin has joined #openstack-nova | 02:57 | |
*** hongbin has joined #openstack-nova | 03:00 | |
openstackgerrit | Ming Jin proposed openstack/nova master: Nova handle empty vcpu_pin_set scenario https://review.opendev.org/670935 | 03:05 |
*** logan- has quit IRC | 03:11 | |
*** logan- has joined #openstack-nova | 03:14 | |
*** factor has joined #openstack-nova | 03:20 | |
*** psachin has joined #openstack-nova | 03:26 | |
*** hongbin has quit IRC | 03:42 | |
*** TxGirlGeek has quit IRC | 03:59 | |
*** damien_r has joined #openstack-nova | 04:08 | |
*** udesale has joined #openstack-nova | 04:09 | |
*** damien_r has quit IRC | 04:13 | |
*** damien_r has joined #openstack-nova | 04:35 | |
*** damien_r has quit IRC | 04:38 | |
*** damien_r has joined #openstack-nova | 04:41 | |
*** mdbooth_ has joined #openstack-nova | 04:43 | |
*** mdbooth has quit IRC | 04:46 | |
openstackgerrit | Yongli He proposed openstack/python-novaclient master: Microversion 2.75 - show server topology https://review.opendev.org/670790 | 04:46 |
*** igordc has joined #openstack-nova | 05:07 | |
*** pcaruana has joined #openstack-nova | 05:08 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix a unit test for pre_live_migration method https://review.opendev.org/670948 | 05:09 |
*** ricolin_ has joined #openstack-nova | 05:10 | |
*** spatel has quit IRC | 05:28 | |
*** udesale has quit IRC | 05:39 | |
*** udesale has joined #openstack-nova | 05:40 | |
*** udesale has quit IRC | 05:42 | |
*** udesale has joined #openstack-nova | 05:57 | |
*** ratailor has joined #openstack-nova | 06:06 | |
*** jmlowe has quit IRC | 06:07 | |
*** jmlowe has joined #openstack-nova | 06:08 | |
*** belmoreira has joined #openstack-nova | 06:19 | |
openstackgerrit | Alex Xu proposed openstack/nova master: Add the virt driver interface for claim and unclaim the devices https://review.opendev.org/670782 | 06:27 |
openstackgerrit | Alex Xu proposed openstack/nova master: Move the allocation retrieving early https://review.opendev.org/670783 | 06:27 |
openstackgerrit | Alex Xu proposed openstack/nova master: Claim and unclaim deivces when spawn and destory the instance https://review.opendev.org/670784 | 06:27 |
openstackgerrit | Alex Xu proposed openstack/nova master: Claim/unclaim the deivces when rebuilding the instance https://review.opendev.org/670785 | 06:27 |
openstackgerrit | Alex Xu proposed openstack/nova master: Claim/unclaim the devices when resizing instance https://review.opendev.org/670786 | 06:27 |
openstackgerrit | Alex Xu proposed openstack/nova master: Populate the existing mediated devices in the libvirt device manager https://review.opendev.org/670787 | 06:27 |
*** damien_r has quit IRC | 06:31 | |
openstackgerrit | Yongli He proposed openstack/nova master: clean up orphan instances https://review.opendev.org/627765 | 06:38 |
*** dpawlik has joined #openstack-nova | 06:40 | |
*** damien_r has joined #openstack-nova | 06:53 | |
*** rcernin has quit IRC | 07:00 | |
*** maciejjozefczyk has joined #openstack-nova | 07:02 | |
openstackgerrit | melanie witt proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.opendev.org/507486 | 07:06 |
*** rpittau|afk is now known as rpittau | 07:08 | |
*** xek has joined #openstack-nova | 07:09 | |
*** Alon_KS has quit IRC | 07:15 | |
*** igordc has quit IRC | 07:16 | |
*** tssurya has joined #openstack-nova | 07:20 | |
*** spatel has joined #openstack-nova | 07:24 | |
*** spatel has quit IRC | 07:29 | |
*** belmoreira has quit IRC | 07:30 | |
*** jangutter has joined #openstack-nova | 07:32 | |
*** jangutter has quit IRC | 07:33 | |
*** jangutter has joined #openstack-nova | 07:33 | |
*** ricolin_ is now known as ricolin | 07:33 | |
*** ratailor_ has joined #openstack-nova | 07:36 | |
*** ratailor has quit IRC | 07:39 | |
*** ratailor__ has joined #openstack-nova | 07:39 | |
*** helenafm has joined #openstack-nova | 07:40 | |
*** ratailor_ has quit IRC | 07:41 | |
*** slaweq has joined #openstack-nova | 07:43 | |
*** priteau has joined #openstack-nova | 07:43 | |
*** udesale has quit IRC | 07:46 | |
*** udesale has joined #openstack-nova | 07:46 | |
*** igordc has joined #openstack-nova | 07:47 | |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/stein: Add 'path' query parameter to console access url https://review.opendev.org/670972 | 07:51 |
*** cdent has joined #openstack-nova | 07:56 | |
*** ttsiouts has joined #openstack-nova | 08:03 | |
*** belmoreira has joined #openstack-nova | 08:05 | |
*** igordc has quit IRC | 08:06 | |
*** ralonsoh has joined #openstack-nova | 08:13 | |
*** jangutter has quit IRC | 08:15 | |
*** jangutter_ has joined #openstack-nova | 08:15 | |
*** jangutter_ has quit IRC | 08:20 | |
*** jangutter has joined #openstack-nova | 08:24 | |
*** tkajinam has quit IRC | 08:25 | |
tssurya | takashin: are you around ? | 08:28 |
takashin | tssurya: yes. | 08:29 |
tssurya | you review comment here: https://review.opendev.org/#/c/666792/2/novaclient/tests/unit/v2/test_shell.py@4277 | 08:29 |
tssurya | can I ask why 74 was added there in the first place ? | 08:29 |
tssurya | could you respond to the question when you have time ? thanks in advance | 08:30 |
tssurya | your* | 08:30 |
takashin | tssurya: okay | 08:30 |
*** mdbooth has joined #openstack-nova | 08:36 | |
*** mdbooth_ has quit IRC | 08:37 | |
tssurya | @cores: reviews for https://review.opendev.org/#/c/645611/ would be appreciated, its in the runways. Thanks in advance | 08:38 |
*** derekh has joined #openstack-nova | 08:44 | |
*** belmoreira has quit IRC | 08:48 | |
*** panda has quit IRC | 08:54 | |
*** belmoreira has joined #openstack-nova | 08:54 | |
*** panda has joined #openstack-nova | 08:57 | |
*** tetsuro has joined #openstack-nova | 09:00 | |
openstackgerrit | Surya Seetharaman proposed openstack/python-novaclient master: API microversion 2.75: Add 'power-update' external event https://review.opendev.org/666792 | 09:13 |
*** ratailor_ has joined #openstack-nova | 09:14 | |
*** ratailor__ has quit IRC | 09:17 | |
*** ricolin has quit IRC | 09:20 | |
openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: doc: Clarify versioned wrapped method https://review.opendev.org/670989 | 09:25 |
*** ratailor__ has joined #openstack-nova | 09:30 | |
*** takashin has left #openstack-nova | 09:30 | |
*** ratailor_ has quit IRC | 09:34 | |
slaweq | sean-k-mooney: hi, can You help me with one bug? | 09:37 |
slaweq | sean-k-mooney: https://bugs.launchpad.net/neutron/+bug/1836642 | 09:37 |
openstack | Launchpad bug 1836642 in OpenStack Compute (nova) "Metadata responses are very slow sometimes" [Undecided,New] | 09:37 |
*** boxiang has joined #openstack-nova | 09:41 | |
*** belmoreira has quit IRC | 09:44 | |
*** priteau has quit IRC | 10:10 | |
openstackgerrit | YumengBao proposed openstack/nova master: add cyborg tempest job https://review.opendev.org/670999 | 10:15 |
*** tetsuro has quit IRC | 10:16 | |
*** jojoda has joined #openstack-nova | 10:18 | |
*** ttsiouts has quit IRC | 10:18 | |
*** ttsiouts has joined #openstack-nova | 10:19 | |
*** ttsiouts has quit IRC | 10:23 | |
*** udesale has quit IRC | 10:29 | |
*** derekh has quit IRC | 10:29 | |
*** derekh has joined #openstack-nova | 10:38 | |
openstackgerrit | Merged openstack/python-novaclient master: doc: Clarify versioned wrapped method https://review.opendev.org/670989 | 10:47 |
*** jojoda has quit IRC | 10:48 | |
sean-k-mooney | slaweq: its posible that that is just a result of the limited resouces we have in the gate | 10:48 |
slaweq | sean-k-mooney: yes, it's possible but it is causing many failures various jobs :/ | 10:49 |
*** bbowen has quit IRC | 10:49 | |
gibi | tssurya: left comments in https://review.opendev.org/#/c/645611/ | 10:51 |
sean-k-mooney | the first two reuest are long in http://logs.openstack.org/09/666409/7/check/tempest-full/08f4c53/controller/logs/screen-n-api-meta.txt.gz#_Jul_11_23_43_16_704979 | 10:51 |
sean-k-mooney | that one is 16 second and the previous one was 1.4 | 10:52 |
sean-k-mooney | but the request after that are 2ms | 10:52 |
openstackgerrit | Merged openstack/nova master: Add Python 3 Train unit tests https://review.opendev.org/669437 | 10:55 |
sean-k-mooney | slaweq: it look like that was the first request on that worker thread and it had to initalise the db connection andproably some other setup stuff | 10:58 |
*** jojoda has joined #openstack-nova | 10:58 | |
*** brinzhang has joined #openstack-nova | 10:59 | |
sean-k-mooney | slaweq: we can see the request is recived here http://logs.openstack.org/09/666409/7/check/tempest-full/08f4c53/controller/logs/screen-n-api-meta.txt.gz#_Jul_11_23_43_00_523076 | 10:59 |
slaweq | sean-k-mooney: yes, and response is send here: http://logs.openstack.org/09/666409/7/check/tempest-full/08f4c53/controller/logs/screen-n-api-meta.txt.gz#_Jul_11_23_43_16_704979 | 11:00 |
slaweq | it took 16 seconds to send it back | 11:00 |
slaweq | after 10 seconds this "cloud-init" like script in cirros is failing and instance don't have SSH key configured thus test fails :/ | 11:01 |
*** davidsha has joined #openstack-nova | 11:01 | |
sean-k-mooney | yes but if you look at the pids it was setting up the db connection on pid 9232 which is the pid that serviced the request | 11:01 |
slaweq | sean-k-mooney: and what is strange for me, is that only this one request was so slow | 11:02 |
sean-k-mooney | so im guessing it was delayed becasue the api worker hadn to do some intal setup | 11:02 |
slaweq | why all other requests around this are much faster? | 11:02 |
*** brinzhang_ has quit IRC | 11:03 | |
sean-k-mooney | becasue they dont have to inisllise the wsgi app and set up the db connections | 11:03 |
slaweq | sean-k-mooney: but don't You think it is bug if nova needs sometimes 16 seconds to send public-key? | 11:03 |
sean-k-mooney | they can reuse all the work that has been done | 11:03 |
sean-k-mooney | i dont think it actully too 16 second i think it was waiting for the database to return with the reponce | 11:04 |
slaweq | from user (VM) PoV it was 16 seconds | 11:04 |
sean-k-mooney | sure | 11:04 |
sean-k-mooney | but as i said it look like this is because its the frist api request recived by that worker | 11:05 |
sean-k-mooney | 16 seconds is not really that un resonable given the load that the gate vms are under. we can see if there is a way to speed it up but i dont think this would be an issue in production | 11:06 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: Get rid of args to RBDDriver.__init__() https://review.opendev.org/668564 | 11:09 |
slaweq | sean-k-mooney: the problem is that this is an issue in the gate :/ | 11:09 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Move consts from neutronv2/api to constants module https://review.opendev.org/668945 | 11:10 |
sean-k-mooney | slaweq: yep which we may not be able to fix in nova. im goint to look at the other logs and then ill look at the code | 11:10 |
sean-k-mooney | but its like saying the ovs neutorn agenet does not wire up new port imediatly after teh agent is restart | 11:10 |
slaweq | sean-k-mooney: thx | 11:10 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Translatable output strings in heal allocation https://review.opendev.org/668925 | 11:11 |
sean-k-mooney | we know it take a whlie for it to recompute everything because it has a lot of work to do on start up. maybe we tweak something or have devstack ping the metadta api | 11:11 |
sean-k-mooney | e.g. before tempest runs to make sure this setup is done before a test hits it | 11:12 |
slaweq | sean-k-mooney: or other "gate only workaround" could be to patch cirros image and retry those calls if it fails first time | 11:12 |
*** ratailor__ has quit IRC | 11:13 | |
cdent | "gate only workaround"-- | 11:14 |
sean-k-mooney | looking at the logs more closely the reponces that are 2ms are all cached | 11:19 |
sean-k-mooney | uncached responces for new intanstances are between 300-1000ms generally with the 16000 ms and similar long resonces also doing addtional work | 11:21 |
*** ttsiouts has joined #openstack-nova | 11:22 | |
sean-k-mooney | by addtional work i mean it has to aquire a lock and call nova.context.get_or_set_cached_cell_and_set_connections before it then generates the metadata | 11:23 |
sean-k-mooney | or in the case of the first request also set up the db connection | 11:23 |
sean-k-mooney | ok so when its not cached it has to call this | 11:25 |
sean-k-mooney | https://github.com/openstack/nova/blob/af4cd78e8537d8f0933e30598066cedac5844866/nova/context.py#L355-L375 | 11:25 |
*** luksky11 has joined #openstack-nova | 11:26 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use neutron contants in cmd/manage.py https://review.opendev.org/668946 | 11:26 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add 'resource_request' to neutronv2/constants https://review.opendev.org/668947 | 11:26 |
sean-k-mooney | which set up both the db connection and rabbitmq connection if its not already cached | 11:26 |
*** tesseract has joined #openstack-nova | 11:30 | |
sean-k-mooney | ok so that is invoked when teh context target_cell context manager is used to lookup the instace by its uuid here https://github.com/openstack/nova/blob/master/nova/api/metadata/base.py#L680 | 11:35 |
slaweq | sean-k-mooney: thx for looking into this | 11:35 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use the safe get_binding_profile https://review.opendev.org/669817 | 11:37 |
gibi | efried: if you have time, could you look back to https://review.opendev.org/#/c/666857 ? | 11:49 |
*** yedongcan has left #openstack-nova | 11:55 | |
sean-k-mooney | slaweq: im not 100% certin but the metadata api request that took 16 seconds may have been waiting on neutron for 12 second to respond with the security groups for the instance http://logs.openstack.org/09/666409/7/check/tempest-full/08f4c53/controller/logs/screen-q-svc.txt.gz#_Jul_11_23_43_16_660751 | 11:55 |
*** lpetrut has joined #openstack-nova | 11:57 | |
*** bbowen has joined #openstack-nova | 12:00 | |
sean-k-mooney | slaweq: actully the device id of the port matches the instance id of the metadata request so yes its was waiting for neuton to return the security group info | 12:01 |
sean-k-mooney | when we are building the metata data we call neutron to get the security group info here https://github.com/openstack/nova/blob/master/nova/api/metadata/base.py#L145 | 12:02 |
sean-k-mooney | which does this https://github.com/openstack/nova/blob/46a3bcd80b41e99ec4923c7cf3d0f8dd8505e97c/nova/network/security_group/neutron_driver.py#L374-L409 | 12:02 |
sean-k-mooney | we list the ports associated with the server which we can see in the neutorn log here http://logs.openstack.org/09/666409/7/check/tempest-full/08f4c53/controller/logs/screen-q-svc.txt.gz#_Jul_11_23_43_04_463505 | 12:02 |
sean-k-mooney | then we retrive the securtiy group info which takes 12 seconds http://logs.openstack.org/09/666409/7/check/tempest-full/08f4c53/controller/logs/screen-q-svc.txt.gz#_Jul_11_23_43_16_660751 | 12:03 |
sean-k-mooney | it starts here http://logs.openstack.org/09/666409/7/check/tempest-full/08f4c53/controller/logs/screen-q-svc.txt.gz#_Jul_11_23_43_04_575879 | 12:04 |
sean-k-mooney | slaweq: what does "Synchronizing usage tracker for tenant" mean in the context of security groups? | 12:05 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Defaults missing group_policy to 'none' https://review.opendev.org/657796 | 12:07 |
*** boxiang has quit IRC | 12:17 | |
slaweq | sean-k-mooney: let me look at this now | 12:26 |
*** lpetrut has quit IRC | 12:28 | |
kashyap | aspiers: Hi, if you have a SUSE system near by, can you please check if any of the OVMF/EDK2 packages provide this file: /usr/share/OVMF/OVMF_CODE.fd | 12:31 |
* kashyap guesses there is no such file (even as a symlink)as the above in SUSE, except: /usr/share/qemu/ovmf-x86_64-opensuse-code.bin | 12:31 | |
*** lpetrut has joined #openstack-nova | 12:32 | |
*** Luzi has joined #openstack-nova | 12:32 | |
slaweq | sean-k-mooney: it looks that this "Synchronizing usage tracker for tenant.." is done here: https://github.com/openstack/neutron/blob/master/neutron/quota/resource.py#L245 and it counts number of security groups in db and update quota usage in db for this tenant | 12:34 |
slaweq | sean-k-mooney: I will investigate this more, thx a lot for help | 12:34 |
*** ricolin has joined #openstack-nova | 12:35 | |
sean-k-mooney | im updating the bug and tyring to see if its the same cause for other cases | 12:35 |
slaweq | sean-k-mooney: thx | 12:35 |
sean-k-mooney | i am not sure its the same issue in all cases but i did see a another 12 second wait to get netwrok info in another case | 12:35 |
sean-k-mooney | im currently confirming its coming form a metadta build request | 12:36 |
artom | lyarwood, heya, could you hit https://review.opendev.org/#/q/topic:bug/1832028+(status:open+OR+status:merged) ? | 12:37 |
artom | And we'd need to find another stable core, with mriedem having a much earned PTO | 12:37 |
*** mchlumsky has joined #openstack-nova | 12:48 | |
lyarwood | artom: just hit the stable/stein one, quick open question around testing | 12:52 |
*** udesale has joined #openstack-nova | 12:52 | |
*** helenafm has quit IRC | 12:52 | |
lyarwood | artom: I'm not sure if we need another DMN testing change in Neutron on stable/stein for this or if everything in Nova stable/stein is already enough | 12:52 |
*** tesseract has quit IRC | 12:53 | |
*** helenafm has joined #openstack-nova | 12:53 | |
openstackgerrit | Matthew Booth proposed openstack/nova master: Failing unit test for bug 1836212 https://review.opendev.org/671023 | 12:54 |
openstack | bug 1836212 in OpenStack Compute (nova) "libvirt: Failure to recover from failed detach" [Undecided,New] https://launchpad.net/bugs/1836212 | 12:54 |
efried | gibi: on it | 12:55 |
efried | Anyone know where cfriesen has been lately? | 12:55 |
*** tesseract has joined #openstack-nova | 12:55 | |
*** lpetrut has quit IRC | 12:57 | |
*** mchlumsky has quit IRC | 12:58 | |
*** mchlumsky has joined #openstack-nova | 12:59 | |
efried | gibi: are you still here? | 13:12 |
kashyap | efried: I was wondering, too. | 13:17 |
kashyap | Probably buried in downstream stuff | 13:17 |
efried | kashyap: Do you know anything about the libvirt patches to make vTPM more secure? | 13:17 |
kashyap | efried: I do, peripherially. And funny you ask! | 13:18 |
kashyap | Just today I was browsing that vTPM thread on upstream libvirt list | 13:18 |
* kashyap brings the URL | 13:18 | |
*** ricolin has quit IRC | 13:18 | |
kashyap | The patch series is at v5, and almost ready to be merged | 13:19 |
efried | LibVirt RFE: https://bugzilla.redhat.com/show_bug.cgi?id=1728030 | 13:19 |
efried | RH has already written a patch https://www.redhat.com/archives/libvir-list/2019-July/msg00584.html | 13:19 |
openstack | bugzilla.redhat.com bug 1728030 in libvirt "RFE: Add encryption of TPM emulator state" [Unspecified,New] - Assigned to libvirt-maint | 13:19 |
kashyap | efried: https://www.redhat.com/archives/libvir-list/2019-July/msg00854.html | 13:19 |
kashyap | "PATCH v5 00/20] Add support for vTPM state encryption | 13:19 |
kashyap | " | 13:19 |
efried | cool, I don't know how to read any of that, so "almost ready to be merged" is a good bit of info. | 13:19 |
kashyap | It has gone through a good few rounds of feedback from long-time reviewers upstream | 13:20 |
efried | when that happens, will there need to be any special affordance in the nova design to enable it? | 13:20 |
efried | like, do we have to (conditionally based on available libvirt version) do something different to the xml? | 13:20 |
kashyap | efried: Probably the config classes and XML exposure, and the relevant calls in libvirt/driver.py? | 13:20 |
efried | I guess at the very least we would have to expose a trait saying that "more secure vTPM is available here" | 13:20 |
kashyap | Right, on the version conditionals | 13:21 |
efried | But someone needs to be working on the code in the first place. It hasn't been touched since the PTG. | 13:21 |
* kashyap is, meanwhile, shaving unending yaks for the Secure Boot stuff. Now that the Fedora patch is merged (https://src.fedoraproject.org/rpms/edk2/c/674b3c8a27a8), now getting the Debian and Ubuntu folks to follow suit. | 13:22 | |
kashyap | efried: Yeah, if the author has gone AWOL, then probably "punt it to backlog"? | 13:22 |
*** eharney has quit IRC | 13:22 | |
kashyap | When they come back, it can be revived. | 13:22 |
efried | no can do | 13:22 |
kashyap | Sorry, what do you mean? | 13:22 |
efried | more likely "find new owner asap" | 13:22 |
kashyap | Ah | 13:22 |
efried | downstream pressure | 13:23 |
kashyap | Ah-ha, nod. | 13:23 |
kashyap | It is a useful, self-contained solution -- for those who want it. | 13:23 |
*** tbachman has joined #openstack-nova | 13:24 | |
*** tbachman_ has joined #openstack-nova | 13:26 | |
*** hemna has joined #openstack-nova | 13:27 | |
kashyap | efried: You do know that the core vTPM stuff is merged in upstream libvirt, yeah? | 13:27 |
kashyap | (The series linked above is for the vTPM state being ecrypted) | 13:27 |
efried | kashyap: Yeah | 13:27 |
efried | like, the base design (what we have merged) could be completed any time somebody wanted to pick it up and run with it. | 13:27 |
kashyap | Yeah, just double-confirming. Just checked with DanPB, he said he hasn't looked at the v5 libvirt series yet | 13:27 |
artom | lyarwood, yeah, good point, lemme look into that | 13:28 |
*** brinzhang has quit IRC | 13:28 | |
*** tbachman has quit IRC | 13:28 | |
*** tbachman_ is now known as tbachman | 13:28 | |
*** shilpasd has quit IRC | 13:31 | |
sean-k-mooney | kashyap: has it been released in a libvirt version | 13:33 |
*** liuyulong has joined #openstack-nova | 13:33 | |
kashyap | sean-k-mooney: The vTPM state being encrypted? | 13:33 |
kashyap | No, it is still in review upstream. | 13:33 |
kashyap | (As noted above) | 13:33 |
sean-k-mooney | ok then we cant have a feature that depends on that untill its released. not just merged | 13:33 |
sean-k-mooney | or in this case still under review | 13:34 |
*** helenafm has quit IRC | 13:34 | |
sean-k-mooney | but unencrypted vTPM can still proceed | 13:34 |
*** ttsiouts has quit IRC | 13:34 | |
*** ganso has quit IRC | 13:34 | |
kashyap | Right, I know the rule. But as long as it is in an upstream libvirt release, it should damn well be fine to have a feature depend on a brand new upstream release | 13:34 |
sean-k-mooney | it shoudl hopefully be minor to extend unencypted vTPM to encyrpeted once the feature is released | 13:35 |
*** ttsiouts has joined #openstack-nova | 13:35 | |
sean-k-mooney | yep | 13:35 |
sean-k-mooney | just need to be in a livirt releases tar | 13:35 |
sean-k-mooney | it doesnt need to be in a distro | 13:35 |
kashyap | Right. That's good. In the past there was an absolute bonkers discussion that libvirt release should "bake" in a distro for a month | 13:36 |
kashyap | ... and receive some "testing" and some such utter nonsense. | 13:36 |
kashyap | Glad we are doing away with that crap. | 13:36 |
*** helenafm has joined #openstack-nova | 13:36 | |
sean-k-mooney | well the reason for "master is not enough" is like use other porject somtimes merge stuff and revert it before a release | 13:37 |
sean-k-mooney | so it has to be released but it does not need to be packaged | 13:37 |
kashyap | sean-k-mooney: Right, I'm with you on that -- so long as it's in a released tarball, we should be (and are) good. | 13:37 |
*** trident has quit IRC | 13:38 | |
*** ttsiouts has quit IRC | 13:38 | |
*** ttsiouts has joined #openstack-nova | 13:38 | |
*** trident has joined #openstack-nova | 13:39 | |
tssurya | thanks gibi :) | 13:41 |
*** ganso has joined #openstack-nova | 13:43 | |
*** Luzi has quit IRC | 13:43 | |
artom | lyarwood, wrote some words on https://review.opendev.org/#/c/670645/2 | 13:44 |
efried | artom: btw, I did *not* get around to opening that bug yesterday. | 13:44 |
efried | in case you had a yen to do it | 13:44 |
* artom reloads context | 13:44 | |
* artom does the needful | 13:44 | |
artom | lyarwood, if you agree with said words, I shall do the needful for that as well | 13:45 |
* artom pokes sean-k-mooney to look at said words as well | 13:45 | |
* artom feels like sean-k-mooney's bus-factor is a bit too high | 13:46 | |
lyarwood | artom: I am if we can run A against stable/stein, I'm not sure that we can | 13:46 |
* sean-k-mooney sniggers "does the needful" sounds like a euphemism for going to the bathroom | 13:47 | |
lyarwood | artom: A being OVS hybrid same host | 13:47 |
sean-k-mooney | artom: that is the backport | 13:47 |
artom | lyarwood, any reason we can't add it to stein's experimental job? | 13:47 |
artom | sean-k-mooney, yeah, lyarwood is concerned about testing | 13:47 |
artom | (rightfully so) | 13:47 |
lyarwood | artom: yeah I just don't know if we just inherit that all from master or if we need to change stuff | 13:48 |
sean-k-mooney | you should be able to backport the DNM test | 13:48 |
lyarwood | I'll try experimental first and go from there | 13:48 |
* artom does all the things | 13:48 | |
sean-k-mooney | and merge in the neutorn-iptabels job too if needed | 13:48 |
*** TxGirlGeek has joined #openstack-nova | 13:48 | |
artom | And since https://review.opendev.org/#/c/670593/1 popped up when looking through my reviews trying to find the one where efried'd bug manifested itself, it'd be cool if someone could look at that as well, but there's no "natural" person for that area (simple change, but involved understanding SRIOV and device tagging) | 13:49 |
*** tbachman has quit IRC | 13:49 | |
sean-k-mooney | lyarwood: we do not have teh iptables job in the experimental pipline on stable/* | 13:49 |
lyarwood | sean-k-mooney: kk so we need a DNM change right? | 13:50 |
lyarwood | sean-k-mooney: sorry I wasn't sure which part you were talking about above :D | 13:50 |
sean-k-mooney | i have https://review.opendev.org/#/c/664442/ but i need to add the neutron iptables jobs and backport it | 13:51 |
*** luksky11 has quit IRC | 13:51 | |
artom | sean-k-mooney, you have to split the 2 changes as well | 13:51 |
artom | Because we need to test multi-host both without and with hybrid plug | 13:52 |
sean-k-mooney | i can create a version that will drop the unneeded jobs and just have teh two we need and propsoe the backpor if ye want | 13:52 |
*** psachin has quit IRC | 13:52 | |
artom | sean-k-mooney, I'd really appreciated if you handled covering all the test cases, yeah :) | 13:52 |
sean-k-mooney | so we need 3 or 4 test jobs | 13:53 |
artom | sean-k-mooney, 3, for cases A, B, D | 13:54 |
artom | A = neutron-tempest-iptables_hybrid | 13:54 |
*** mlavalle has joined #openstack-nova | 13:54 | |
artom | D = first half of https://review.opendev.org/#/c/664442/ (run revert tests in nova-live-migration) | 13:55 |
sean-k-mooney | b and d is the neutorn multinode job with and without iptables? | 13:55 |
artom | B = second half of https://review.opendev.org/#/c/664442/ (iptables in nova-live-migration/nova-multinode) | 13:55 |
sean-k-mooney | ok i can go do that now | 13:55 |
sean-k-mooney | *nova-multinode which was nova-live-migration | 13:56 |
mdbooth | Folks, I'm undecided on https://review.opendev.org/#/c/671023/ . It looks to me like a customer actually hit this race in practise while running a heat stack. However, in general I suspect it's fairly uncommon. My preference is generally to fix this stuff, but this kind of cleanup also tends to go ignored. Is a fix for this likely to be of interest? | 13:58 |
* mdbooth doesn't want to add to his pile of abandoned code. | 14:00 | |
openstackgerrit | Merged openstack/nova master: Remove Rocky-era min compute trusted certs compat check https://review.opendev.org/669539 | 14:00 |
sean-k-mooney | assuming your runit test actully repoduced the race then it should be fairly staitforwad to show your patch is correct | 14:01 |
sean-k-mooney | so as long as the fix is not too invasive i dont see why this would be contovertial | 14:01 |
sean-k-mooney | but i dont know | 14:01 |
sean-k-mooney | its a race that is proably not hard to debug so if we dont fix it now it will proably bite us later | 14:03 |
*** TxGirlGeek has quit IRC | 14:03 | |
*** TxGirlGeek has joined #openstack-nova | 14:03 | |
mdbooth | sean-k-mooney: It will require a refactor of a method in libvirt.guest. Nobody likes refactors. | 14:04 |
sean-k-mooney | mdbooth: can you do it without changing the api signiture | 14:04 |
mdbooth | Of course, it's internal to the libvirt driver. | 14:04 |
sean-k-mooney | by api signiture i ment the signiture of the funtion | 14:05 |
mdbooth | Yes, probably, although the function signature does also need to be fixed. | 14:05 |
sean-k-mooney | as you said the guest stuff shoudl be internal to the libvirt driver so your e not modifying the virt direver api so it should be fine to backport | 14:06 |
artom | efried, hah, see where else that error popped up: http://logs.openstack.org/09/666409/8/check/tempest-full-py3/38bf84e/job-output.txt#_2019-07-14_17_19_10_677555 | 14:07 |
mdbooth | sean-k-mooney: That's fine. I'm more interested to know if anybody cares. If nobody cares I'm not going to fix it. | 14:07 |
yonglihe | efried: per the run queue, there is 2.75 ahead my patch, seems i better moving to 2.76 in advance? | 14:07 |
efried | artom: okay, so we should sort this out. | 14:07 |
mdbooth | sean-k-mooney: It's bad for my mental health ;) | 14:08 |
sean-k-mooney | mdbooth: well at least we have the bug and a repoducer unit test | 14:08 |
efried | yonglihe: You mean in the runway queue? | 14:08 |
sean-k-mooney | so if you dont fix it someone else can at a later date | 14:08 |
artom | efried, there aren't that many hits, but yeah, our theory from last night is pretty much confirmed | 14:08 |
*** TxGirlGeek has quit IRC | 14:08 | |
yonglihe | efried, yeah. | 14:08 |
yonglihe | 'power-update' might take the 2.75 | 14:09 |
efried | yonglihe: Tough call. Just because it's ahead of you in the runway queue doesn't mean it'll get reviewed first. If you go to 2.76, you'll also need to stack yours on top of the 2.75 one, which is pretty weird since there's no actual dependency. | 14:09 |
efried | probably just best to ride it out. | 14:09 |
yonglihe | ok, let's see what happens then. | 14:09 |
artom | efried, in my experience it was basically a race to see which one lands first, and then the other person is stuck rebasing | 14:10 |
efried | artom: test_delete_server_while_in_building_state sounds like a test designed to expose this bug. | 14:10 |
*** TxGirlGeek has joined #openstack-nova | 14:10 | |
mdbooth | artom: Is that holding up your revert resize change, btw? | 14:10 |
artom | efried, for serious | 14:10 |
artom | mdbooth, what is? | 14:10 |
mdbooth | artom: I don't know, that's what I'm asking. | 14:10 |
sean-k-mooney | mdbooth: the revert resize change has merged | 14:10 |
yonglihe | btw, the corresponding novaclent patch realse released: https://review.opendev.org/#/c/670790/ | 14:10 |
mdbooth | sean-k-mooney artom: \o/ | 14:10 |
artom | mdbooth, no, this is unrelated, something that efried noticed last night | 14:10 |
*** TxGirlGeek has quit IRC | 14:10 | |
sean-k-mooney | mdbooth: on master. im working on creating a DNM patch to test stable | 14:11 |
artom | sean-k-mooney, patch_ES_ :) | 14:11 |
sean-k-mooney | artom has the backport alreay up | 14:11 |
sean-k-mooney | artom: im going to do 1 patch with 3 jobs | 14:11 |
mdbooth | [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: certificate is not yet valid | 14:12 |
sean-k-mooney | its a dnm patch so i have no issue with copy pasting the job definiton and just not seting iptabels in one of them | 14:12 |
mdbooth | I bet I'm the only person to see that regularly :) | 14:12 |
sean-k-mooney | mdbooth: are you doing multi node but have not syncted your clock on each of the hosts | 14:13 |
artom | efried, https://bugs.launchpad.net/nova/+bug/1836754 Should I also propose something for elastic-recheck? | 14:14 |
openstack | Launchpad bug 1836754 in OpenStack Compute (nova) "Conflict when deleting allocations for an instance that hasn't finished building" [Undecided,New] | 14:14 |
*** tbachman has joined #openstack-nova | 14:14 | |
mdbooth | sean-k-mooney: Nah, I'm using Hyper-V, which stops the clock while my laptop is suspended. It was still early last week. | 14:14 |
artom | sean-k-mooney, ack, that works too | 14:14 |
mdbooth | The guest tools are *supposed* to auto sync the time from the HV, but I've never made that work. | 14:14 |
sean-k-mooney | mdbooth: right but do you not have ntpd installed to fix that | 14:14 |
mdbooth | Yeah, I have to fix it with chronyd manually, which is a hack. It has to be manual because it won't auto-fix a jump that large. | 14:15 |
sean-k-mooney | ah i see. i have only ever used hyperv on a desktop or server so never had that issue | 14:16 |
sean-k-mooney | that reminds me i need to order an ssd so i dont need to keep dual booting my workstation | 14:17 |
* sean-k-mooney i think its technical a workstation pc but i guess it could be a tower server... im sure there is a diffence to may wallet but other then that i think they are the same? | 14:19 | |
mdbooth | sean-k-mooney: For an SSD? That seems like a weird distinction, but I'm out of touch with such things. | 14:20 |
sean-k-mooney | no i was wondering if my dell t7610 is technically a server or a workstation | 14:21 |
sean-k-mooney | the ssd boot drive died so im using the one out of my windows gaming pc as a stop gap but that means i have to dual boot it currently | 14:22 |
sean-k-mooney | im planning to get a seccond ssd and install linux then pass the windows ssd into a vm | 14:22 |
artom | ... do I need something special to propose patches to elastic-recheck? Gerrit's refusing me | 14:23 |
efried | artom: elastic-recheck would be nice, though I'm not sure who approves those when mriedem isn't around. | 14:23 |
sean-k-mooney | artom: no you shouldnt | 14:23 |
sean-k-mooney | artom: whats the error? | 14:23 |
*** eharney has joined #openstack-nova | 14:23 | |
artom | oh, no one had updated .gitreview to point to opendev/ instead of openstack-infra | 14:24 |
sean-k-mooney | ah ya that would break it | 14:24 |
sean-k-mooney | i think the redirects dont work proably in all cases | 14:24 |
artom | Oh, I think it's because I cloned from github and no one's re-setup the mirroring | 14:29 |
sean-k-mooney | i dont think the infra projects are going to be mirror going forward | 14:30 |
*** icarusfactor has joined #openstack-nova | 14:32 | |
*** luksky11 has joined #openstack-nova | 14:33 | |
efried | stephenfin: you around to give https://review.opendev.org/#/c/657796/ a bump? | 14:33 |
stephenfin | Looking | 14:34 |
*** factor has quit IRC | 14:35 | |
stephenfin | and done. That was nice and easy | 14:36 |
*** dpawlik has quit IRC | 14:36 | |
gibi | efried, stephenfin: thanks | 14:36 |
efried | gibi: I only regret I can't send "heal port allocations" to the gate. | 14:37 |
efried | half a dozen +As stacked up behind that one. | 14:37 |
gibi | efried: no worries. we will wait for Matt to to +A that after he is back | 14:38 |
gibi | meanwhile I can finally refocus on the server move operations + bandwidth series | 14:38 |
*** jc_ has joined #openstack-nova | 14:56 | |
*** jc_ has left #openstack-nova | 14:57 | |
openstackgerrit | Shilpa Devharakar proposed openstack/nova master: Nova object changes for forbidden aggregates request filter https://review.opendev.org/671072 | 15:01 |
openstackgerrit | Shilpa Devharakar proposed openstack/nova master: DB API changes to get non-matching aggregates from metadata https://review.opendev.org/671074 | 15:02 |
openstackgerrit | Shilpa Devharakar proposed openstack/nova master: Add a new request filter to forbidden aggregates https://review.opendev.org/671075 | 15:03 |
openstackgerrit | Shilpa Devharakar proposed openstack/nova master: Enable request filter forbidden_aggregates https://review.opendev.org/667952 | 15:03 |
*** shilpasd has joined #openstack-nova | 15:05 | |
openstackgerrit | sean mooney proposed openstack/nova stable/stein: [DNM] testing bug/1813789 revert resize events https://review.opendev.org/671079 | 15:22 |
artom | That was fast | 15:22 |
sean-k-mooney | it took longer then i planned but i think ^ is right | 15:23 |
sean-k-mooney | i guess not | 15:25 |
sean-k-mooney | oh i think im missing a new line | 15:26 |
sean-k-mooney | and a space | 15:27 |
openstackgerrit | sean mooney proposed openstack/nova stable/stein: [DNM] testing bug/1813789 revert resize events https://review.opendev.org/671079 | 15:27 |
sean-k-mooney | ok zull accpeted that so it should report back eventually | 15:34 |
*** gyee has joined #openstack-nova | 15:39 | |
*** cdent has quit IRC | 15:45 | |
*** eharney has quit IRC | 15:46 | |
*** eharney has joined #openstack-nova | 15:46 | |
*** igordc has joined #openstack-nova | 15:47 | |
*** ttsiouts has quit IRC | 15:51 | |
*** ttsiouts has joined #openstack-nova | 15:52 | |
*** helenafm has quit IRC | 15:52 | |
*** luksky11 has quit IRC | 15:55 | |
*** tssurya has quit IRC | 15:55 | |
*** ttsiouts has quit IRC | 15:57 | |
*** ricolin has joined #openstack-nova | 16:09 | |
*** hemna has quit IRC | 16:22 | |
*** damien_r has quit IRC | 16:23 | |
*** hamzy has quit IRC | 16:25 | |
*** mmethot has quit IRC | 16:25 | |
*** tesseract has quit IRC | 16:33 | |
efried | shilpasd: still around? | 16:34 |
openstackgerrit | Boris Bobrov proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680 | 16:34 |
openstackgerrit | Boris Bobrov proposed openstack/nova master: Add extra spec parameter and image property for memory encryption https://review.opendev.org/664420 | 16:34 |
openstackgerrit | Boris Bobrov proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334 | 16:34 |
openstackgerrit | Boris Bobrov proposed openstack/nova master: Add <launchSecurity> and <driver iommu='on' /> to config.py https://review.opendev.org/636318 | 16:34 |
openstackgerrit | Boris Bobrov proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565 | 16:34 |
openstackgerrit | Boris Bobrov proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616 | 16:34 |
efried | shilpasd: I'm going to edit the commit message of https://review.opendev.org/#/c/671075 real quick so I can be +2 on the whole series. | 16:35 |
openstackgerrit | Eric Fried proposed openstack/nova master: Add a new request filter to forbidden aggregates https://review.opendev.org/671075 | 16:37 |
openstackgerrit | Eric Fried proposed openstack/nova master: Enable request filter forbidden_aggregates https://review.opendev.org/667952 | 16:37 |
efried | dansmith: I'm +2 all the way up on the forbidden aggs series ^ | 16:38 |
*** rpittau is now known as rpittau|afk | 16:38 | |
efried | the biggish final patch is mostly doc | 16:40 |
*** TxGirlGeek has joined #openstack-nova | 16:40 | |
*** atahardjebbar has joined #openstack-nova | 16:40 | |
openstackgerrit | Raubvogel proposed openstack/nova master: Author: Mauricio Tavares <raubvogel@gmail.com> https://review.opendev.org/671106 | 16:44 |
*** hemna has joined #openstack-nova | 16:44 | |
openstackgerrit | Raubvogel proposed openstack/nova master: Author: Mauricio Tavares <raubvogel@gmail.com> https://review.opendev.org/671106 | 16:45 |
*** mmethot has joined #openstack-nova | 16:47 | |
*** amodi has quit IRC | 16:48 | |
*** hemna has quit IRC | 16:49 | |
*** sapd1_x has joined #openstack-nova | 16:50 | |
*** hamzy has joined #openstack-nova | 16:55 | |
*** derekh has quit IRC | 17:01 | |
openstackgerrit | Boris Bobrov proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565 | 17:03 |
openstackgerrit | Boris Bobrov proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616 | 17:03 |
*** sapd1_x has quit IRC | 17:04 | |
*** ganso has quit IRC | 17:06 | |
*** ganso has joined #openstack-nova | 17:08 | |
*** udesale has quit IRC | 17:09 | |
*** davidsha has quit IRC | 17:11 | |
*** atahardjebbar has quit IRC | 17:13 | |
*** hemna has joined #openstack-nova | 17:14 | |
*** TxGirlGeek has quit IRC | 17:17 | |
*** igordc has quit IRC | 17:17 | |
*** hemna has quit IRC | 17:18 | |
efried | sean-k-mooney: you around? | 17:20 |
sean-k-mooney | kind of | 17:20 |
efried | I try and I try, but I just can't follow this stuff. Would you please review https://review.opendev.org/#/c/670588/ for me? I'll proxy your +1. | 17:20 |
sean-k-mooney | im wait on food to be delivered in 20 mins or so | 17:21 |
efried | I think it was your suggestion anyway from the subsequent patch | 17:21 |
sean-k-mooney | oh right. in the non ovs case we added a noop pluging to os vif to handel generic vhost-user plugging | 17:21 |
sean-k-mooney | and for the ovs case its already handeled by os-vif | 17:22 |
efried | yeah, see, I can read those words, but I've never been able to follow the code around that area | 17:23 |
sean-k-mooney | so those functions should never get called. but ill trace through the code and confim | 17:23 |
efried | thanks | 17:23 |
sean-k-mooney | well it used to do an indrect function call by appending the vif type to a prefix and using gettr to look up the function | 17:24 |
sean-k-mooney | so it was not exactly the simplest code | 17:24 |
sean-k-mooney | that actully predates me working on openstack but i never really understood why we did it that way | 17:25 |
efried | yeah, the getattr removal is in the subsequent patch, and I was able to follow that. But it's also dependent on this being right. | 17:25 |
sean-k-mooney | this code path is for netwroking-vpp or snabb switch | 17:26 |
sean-k-mooney | its very very rarly used but we still dont want to break people that use it | 17:26 |
*** hemna has joined #openstack-nova | 17:34 | |
openstackgerrit | Merged openstack/nova master: Defaults missing group_policy to 'none' https://review.opendev.org/657796 | 17:37 |
*** hemna has quit IRC | 17:38 | |
*** ricolin has quit IRC | 17:40 | |
*** luksky11 has joined #openstack-nova | 17:42 | |
*** igordc has joined #openstack-nova | 17:44 | |
*** TxGirlGeek has joined #openstack-nova | 17:53 | |
sean-k-mooney | efried: done. i have put a walk through of how the dead code would have been called in the past but it is never used after https://review.opendev.org/#/q/Ifab3006454708ab290b93f02d82b794c334c3946 was merged last cycle | 17:53 |
efried | thanks sean-k-mooney | 17:53 |
*** hemna has joined #openstack-nova | 17:54 | |
sean-k-mooney | no worries dinner just arrived so ill be away for a while. i might be back in an hour or two | 17:54 |
*** hemna has quit IRC | 17:59 | |
*** spatel has joined #openstack-nova | 18:01 | |
spatel | sean-k-mooney: yt? | 18:01 |
spatel | virsh list command showing instance-<uuid> is there anyway it can show hostname of instance | 18:01 |
*** eharney has quit IRC | 18:02 | |
spatel | In google people saying "virsh domhostname <domain>" but what is domain going to be? | 18:03 |
spatel | also i don't think kvm support that | 18:03 |
*** eharney has joined #openstack-nova | 18:06 | |
*** dpawlik has joined #openstack-nova | 18:10 | |
*** efried has quit IRC | 18:12 | |
donnyd | Was there still interest in the NUMA for fortnebula? | 18:20 |
donnyd | LOL... interest in NUMA topologies | 18:20 |
*** efried has joined #openstack-nova | 18:21 | |
donnyd | I have things to a more stable state with decent performance, so now I can move forward | 18:21 |
openstackgerrit | Mauricio Tavares proposed openstack/nova master: Increases the level of detail provided by the debug messages in pci_passthrough_filter.py. https://review.opendev.org/671106 | 18:25 |
efried | artom, sean-k-mooney: ---^ ? | 18:30 |
*** hemna has joined #openstack-nova | 18:37 | |
artom | donnyd, yes please, I would like 1 the NUMA ;) | 18:39 |
donnyd | LOL +2 | 18:40 |
artom | For serious though, yeah, flavors with >= 2 NUMA nodes would be awesome | 18:40 |
donnyd | you get 2 numas artom | 18:40 |
donnyd | I am not deep in the zuul or nodepool code, so how does the test work. Can you ask for a specific flavor? | 18:41 |
donnyd | I need a way to stay compatible with the other jobs | 18:41 |
artom | efried, I don't even know where to start with that. I guess look at the logic before drowning them with the formatting and whitespace nits? | 18:41 |
artom | donnyd, that's not my strong suite, I'm afraid | 18:42 |
efried | artom: I was pointing at donnyd, not at Mauricio's patch | 18:42 |
artom | efried, oh, hah | 18:43 |
efried | my ---^ was an attempt to go around the corner | 18:43 |
artom | donnyd, sean-k-mooney would most likely know, though I hate to pile even more things on this plate | 18:44 |
artom | Or ask in -infra? Is that good advice? | 18:44 |
donnyd | 2:44 PM <clarkb> there isnt | 18:46 |
donnyd | 2:44 PM <clarkb> we can do provider specific flavors but generally avoid that to handle cloud outages | 18:46 |
donnyd | artom: I will have to learn how to setup NUMA aware flavors, (unless someone knows off the top of their head) and it looks like we have a path forward | 19:04 |
*** bbowen has quit IRC | 19:05 | |
artom | donnyd, it's kinda tedious, but not super hard: https://docs.openstack.org/nova/latest/user/flavors.html | 19:05 |
artom | Scroll down to CPU topology | 19:06 |
artom | Sorry, NUMA topology | 19:06 |
donnyd | Ok, I will take a look | 19:06 |
artom | Actually both | 19:07 |
artom | hw:cpu_ controls the guest topology, hw:numa_ controls how it maps onto host hardware | 19:07 |
*** eharney has quit IRC | 19:08 | |
openstackgerrit | Merged openstack/nova master: Use Adapter global_request_id kwarg https://review.opendev.org/670907 | 19:11 |
*** maciejjozefczyk has quit IRC | 19:46 | |
*** prometheanfire has joined #openstack-nova | 19:57 | |
prometheanfire | is one of the use cases for nova to run 1-3 compute nodes in 3-4 DCs across the world? (via host aggregates and AZs) | 19:58 |
openstackgerrit | Eric Fried proposed openstack/nova master: Nova object changes for forbidden aggregates request filter https://review.opendev.org/671072 | 19:59 |
openstackgerrit | Eric Fried proposed openstack/nova master: DB API changes to get non-matching aggregates from metadata https://review.opendev.org/671074 | 19:59 |
openstackgerrit | Eric Fried proposed openstack/nova master: Add a new request filter to forbidden aggregates https://review.opendev.org/671075 | 19:59 |
openstackgerrit | Eric Fried proposed openstack/nova master: Enable request filter forbidden_aggregates https://review.opendev.org/667952 | 19:59 |
efried | prometheanfire: What's this in reference to? | 20:00 |
prometheanfire | sometihng I may be doing soon | 20:00 |
prometheanfire | cross continental rabbit could be fun | 20:01 |
prometheanfire | just not sure if it's an intended (supported) use case for nova | 20:01 |
openstackgerrit | Merged openstack/nova master: vif: Remove 'plug_vhostuser', 'unplug_vhostuser' https://review.opendev.org/670588 | 20:03 |
efried | ohh, okay. So you're asking if you can make a setup like that? | 20:06 |
efried | prometheanfire: in a single cell? | 20:06 |
*** eharney has joined #openstack-nova | 20:06 | |
efried | Cause I would imagine you can surely do a multi-cell that way | 20:06 |
efried | donnyd: do you know about this? | 20:06 |
prometheanfire | ya, could do it multi-cell I suppose, single cell may be nice management wise though | 20:07 |
efried | prometheanfire: isn't that how zuul is set up? | 20:07 |
prometheanfire | the main question I have is if I'll need to run non-compute stuff at each location or not (I'd rather it just be nova-compute and neutron agent type stuff) | 20:07 |
donnyd | well if you look at any of the big clouds out there, it would appear they try to stay away from this type of thing | 20:08 |
efried | running a nodepool across a dozen different locations | 20:08 |
donnyd | Control plane / internal stuff usually stays on the same l2//3 networks | 20:08 |
efried | You might have to run a cell conductor at each site. I would think your db performance would be crap if you didn't. | 20:08 |
donnyd | But I am not entirely sure what the ask is | 20:09 |
prometheanfire | how to best build a cloud that spans multiple DCs, but it still small beans (a couple hosts per location) | 20:09 |
donnyd | What is the plan for N/S E/W traffic... that has been my big hurdle in the past... | 20:10 |
artom | prometheanfire, ooooo, are you being edgy? It's all the rage these days ; | 20:10 |
artom | ;) | 20:10 |
donnyd | wanted to host my control plane in someone elses cloud, and then just have hypervisors and storage here | 20:10 |
prometheanfire | artom: kinda, but not quite that scale | 20:11 |
*** bbowen has joined #openstack-nova | 20:11 | |
prometheanfire | donnyd: as far as traffic, we'll have site to sites, non-dedicated. the E/W should be minimal traffic wise (between DCs at least) | 20:12 |
donnyd | I don't see any reason why it cannot be done if you already have S2S connections to secure traffic.. Gonna spend quite a lot of time having fun with rabbit... | 20:15 |
prometheanfire | yep, I dread rabbit | 20:16 |
donnyd | and def be using DVR | 20:17 |
prometheanfire | I'm not sure we'd use more than linuxbridge and maybe gre (or just vlan) | 20:18 |
prometheanfire | kiss | 20:18 |
donnyd | The way its supposed to work is the same l2 between l3 agent and the vm | 20:20 |
prometheanfire | yep | 20:20 |
prometheanfire | that's possible | 20:20 |
donnyd | you would want to use DVR because the L3 agent will be over a wan connection, and all your broadcast traffic would also be doing the same | 20:21 |
prometheanfire | ya, that'd be rough | 20:22 |
prometheanfire | at least this isn't 4 years ago, dvr is better now I hear | 20:22 |
prometheanfire | https://wiki.openstack.org/wiki/Neutron/DVR still mentions the road to juno :D | 20:23 |
prometheanfire | first result when googling for 'neutron dvr' | 20:23 |
donnyd | I am not surprised | 20:25 |
donnyd | juno was only a couple releases back | 20:26 |
prometheanfire | feels like | 20:28 |
prometheanfire | 9 releases? | 20:28 |
*** hamzy has quit IRC | 20:30 | |
*** ralonsoh has quit IRC | 20:35 | |
*** dasp has quit IRC | 20:45 | |
*** dasp has joined #openstack-nova | 20:47 | |
openstackgerrit | Merged openstack/nova master: vif: Stop using getattr for VIF lookups https://review.opendev.org/666630 | 20:47 |
*** pcaruana has quit IRC | 21:01 | |
*** dpawlik has quit IRC | 21:15 | |
*** gokhani has quit IRC | 21:23 | |
*** dpawlik has joined #openstack-nova | 21:27 | |
*** amodi has joined #openstack-nova | 21:27 | |
*** dpawlik has quit IRC | 21:32 | |
*** luksky11 has quit IRC | 21:36 | |
openstackgerrit | melanie witt proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.opendev.org/507486 | 21:47 |
*** spatel has quit IRC | 21:50 | |
*** xek has quit IRC | 22:02 | |
*** betherly has joined #openstack-nova | 22:26 | |
Nick_A | we have snapshots set to qcow2 in nova.conf. I noticed they take 2x disk usage until uploaded to glance. Is it trying to convert qcow2 > qcow2 or something? | 22:26 |
*** betherly has quit IRC | 22:31 | |
*** whoami-rajat has quit IRC | 22:31 | |
*** rcernin has joined #openstack-nova | 22:36 | |
*** tkajinam has joined #openstack-nova | 22:53 | |
*** slaweq has quit IRC | 22:55 | |
*** betherly has joined #openstack-nova | 22:56 | |
*** betherly has quit IRC | 23:01 | |
*** hemna has quit IRC | 23:24 | |
*** hamzy has joined #openstack-nova | 23:27 | |
*** betherly has joined #openstack-nova | 23:29 | |
*** mlavalle has quit IRC | 23:29 | |
*** betherly has quit IRC | 23:34 | |
*** brinzhang has joined #openstack-nova | 23:49 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!