| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-01-29 | |||
| 19:35:04 | efried | right, that was my gripe. We now have same_subtree to dictate which can be shared and which can be spread. Because same_subtree also devolves to same rp in relevant cases. | |
| 19:35:34 | sean-k-mooney | ya | |
| 19:36:12 | sean-k-mooney | provide we model nics with an RP per PF same_subtree or the negation should hanel that too | |
| 19:36:37 | sean-k-mooney | so we might be able to remvoe the group polices thing at some point | |
| 19:37:04 | sean-k-mooney | anyway if this does not require it to make it work it makes me happy | |
| 19:40:15 | sean-k-mooney | i have saved a copy of that etherpad locally just in case by the way. | |
| 19:41:07 | efried | cool. I've got my comment all composed, will post it when I'm done reviewing the rest. Hopefully bauzas will parlay the etherpad into the doc before the etherpad goes kablooey, as it inevitably will. | |
| 19:41:38 | sean-k-mooney | well we nerver delete them | |
| 19:41:48 | sean-k-mooney | the only go away if the db gets currpted | |
| 19:42:04 | sean-k-mooney | so you would be suprised how long they survie | |
| 19:42:20 | sean-k-mooney | that said we have been bitten enough times at ptgs that i make backups | |
| 19:42:22 | efried | I've seen enough corrupted | |
| 19:42:23 | efried | yeah. | |
| 20:15:25 | artom | dansmith, around? So, are you completely opposed to https://review.opendev.org/#/c/672595/61/nova/tests/unit/virt/libvirt/fakelibvirt.py@489 ? | |
| 20:16:06 | artom | Because I've been trying to write something better, and I'm not sure what I have is worth it. Maybe I just suck, or am overthinking it? | |
| 20:16:30 | dansmith | Why is [1] harder than 1? Aside from all the test change | |
| 20:17:08 | dansmith | or maybe you're stuck on the fact that it really should be named cpu_socket_map or something? | |
| 20:17:27 | dansmith | I do think what you have is pretty ugly and would prefer a mutually exclusive additional arg to what you have It hink | |
| 20:17:52 | artom | dansmith, well, the way I have it now is a full-on separate class | |
| 20:18:01 | artom | Well, classes, because it trickles up and down | |
| 20:18:06 | artom | Now = locally, not pushed yet | |
| 20:18:53 | dansmith | and that's really just for tests? | |
| 20:19:24 | artom | The objective being, allow tests that don't need the sockets maps complexity to use the existing class, and have a new one for the tests that need it | |
| 20:19:33 | artom | Instead of imposing sockets map on all tests | |
| 20:20:03 | artom | You mean the HostInfo and NUMATopology classes? Yeah, just for tests | |
| 20:20:08 | dansmith | Right, point is you've created a new class to wrap hostinfo just for tests | |
| 20:20:19 | dansmith | seems kinda smelly since they're not doing what the production code does | |
| 20:20:28 | dansmith | at least make that a fixture or utility thing in tests/ | |
| 20:21:45 | artom | HostInfo already exists just for tests | |
| 20:22:43 | dansmith | ohhh, sorry I maybe was thinking too fast.. doesn't it mirror something else in the main code? | |
| 20:24:04 | artom | dansmith, no worries - I've been struggling with this for way too much time, too | |
| 20:25:04 | artom | dansmith, so, the HostInfo is eventually passed down to the fake_connection here: https://github.com/openstack/nova/blob/master/nova/tests/functional/libvirt/base.py#L111 | |
| 20:25:23 | artom | dansmith, it's essentially our libvirt service stub | |
| 20:25:47 | dansmith | a wrapper class for ease makes perfect sense I think | |
| 20:25:47 | dansmith | it doesn't inherit from something real, so ignore my previous comments | |
| 20:25:57 | dansmith | yup I'm caught up now | |
| 20:27:20 | artom | Ohhhh | |
| 20:27:25 | artom | I think I just got it | |
| 20:27:40 | artom | Make the "internal machinery" use sockets map | |
| 20:27:50 | artom | And leave HostInfo on top as a compatibility shim | |
| 20:28:29 | artom | And for tests that need it, they can use the "internal machinery" class directly | |
| 20:32:01 | dansmith | yeah, i thought that was what you were suggesting | |
| 20:32:47 | artom | No, I had 2 classes in parallel | |
| 21:24:19 | efried | bauzas, sean-k-mooney, stephenfin: Finished that review of the numa topo spec. | |
| 21:34:23 | sean-k-mooney | efried: cool i will try to catch up with the review tommorow | |
| #openstack-nova - 2020-01-30 | |||
| 00:11:49 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Functional tests for NUMA live migration https://review.opendev.org/672595 | |
| 00:11:51 | openstack | bug 1845146 in OpenStack Compute (nova) train "NUMA aware live migration failed when vCPU pin set" [High,Fix committed] https://launchpad.net/bugs/1845146 - Assigned to Dan Smith (danms) | |
| 00:11:51 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: NUMA LM: Add func test for bug 1845146 https://review.opendev.org/687404 | |
| 01:48:54 | openstackgerrit | Brin Zhang proposed openstack/nova master: Follow-up: Add delete_on_termination to volume-attach API https://review.opendev.org/680655 | |
| 04:14:25 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Pass accelerator requests to each virt driver from compute manager. https://review.opendev.org/698581 | |
| 04:14:25 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Create and bind Cyborg ARQs. https://review.opendev.org/631244 | |
| 04:14:26 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735 | |
| 04:14:26 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Compose accelerator PCI devices into domain XML in libvirt driver. https://review.opendev.org/631245 | |
| 04:14:27 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Enable start/stop of instances with accelerators. https://review.opendev.org/699553 | |
| 04:14:27 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Enable hard/soft reboot with accelerators. https://review.opendev.org/697940 | |
| 04:14:28 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Bump compute rpcapi version and reduce Cyborg calls. https://review.opendev.org/704227 | |
| 04:14:28 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Enable and use COMPUTE_ACCELERATORS trait. https://review.opendev.org/699554 | |
| 04:14:29 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Add cyborg tempest job. https://review.opendev.org/670999 | |
| 06:29:14 | hariii2503 | Hi Guys . I am writing a python script to get some data from nova python client . | |
| 06:29:46 | hariii2503 | I m getting authentication error , when I used below commands | |
| 06:30:01 | hariii2503 | from novaclient import client | |
| 06:30:03 | hariii2503 | nova = client.Client(VERSION, USERNAME, PASSWORD, PROJECT_ID, AUTH_URL) | |
| 06:30:15 | hariii2503 | nova.servers.list() | |
| 06:30:28 | hariii2503 | Please help me in this | |
| 06:30:37 | hariii2503 | openstack version is stein | |
| 07:45:47 | openstackgerrit | Victor Coutellier proposed openstack/nova-specs master: Non-admin user can filter their instances by AZs https://review.opendev.org/701763 | |
| 08:04:22 | openstackgerrit | Victor Coutellier proposed openstack/nova-specs master: Non-admin user can filter their instances by AZs https://review.opendev.org/701763 | |
| 09:54:50 | openstackgerrit | Victor Coutellier proposed openstack/nova-specs master: Non-Admin user can filter their instances by more filters https://review.opendev.org/701763 | |
| 10:05:11 | stephenfin | bauzas: Could you hit https://review.opendev.org/#/c/695380/ ? | |
| 10:05:47 | stephenfin | Appears to be the only remaining issue we have for the py3 goal | |
| 10:51:24 | jawad_axd | Hi all ! Question is, from instances, download/upload speed is around 10MB/s while from underlying host is 80MB/s. How can I increase/tune bandwidth parameters? I am using Stein,linux bridge,vxlans setup.Any pointers? | |
| 11:22:04 | donnyd | @jawad_axd you need to look for your bottleneck. Download speeds can depend on a lot of different things | |
| 11:22:56 | donnyd | maybe start with some simple iperf testing in and out of the tenant space. | |
| 11:23:29 | donnyd | What are you using for a storage backend on your instances - that is the second place i would look | |
| 11:24:44 | donnyd | maybe also try some fio disk benchmarking to ensure your instances can write to disk at faster than 10MB/s | |
| 11:29:32 | jawad_axd | @donnyd I am using ceph as storage backend. | |
| 11:30:01 | donnyd | I would start with a disk benchmark then | |
| 11:30:46 | jawad_axd | Okay..thats good pointer. I will look into it. Thanks | |
| 11:30:59 | donnyd | also maybe reachout to #openstack-operators | |
| 11:31:19 | jawad_axd | Sure. | |
| 11:31:49 | donnyd | that is where all the operators hang out and you may get to an end a little sooner there | |
| 11:32:12 | donnyd | np @jawad_axd | |
| 11:57:57 | openstackgerrit | Vladyslav Drok proposed openstack/nova master: Fix volume attachment rollback https://review.opendev.org/704847 | |
| 12:09:56 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Re-propose the flavor extra spec validation spec https://review.opendev.org/682655 | |
| 12:19:52 | openstackgerrit | sean mooney proposed openstack/os-vif master: [OVS] VLAN tag should be set in the Port register https://review.opendev.org/703373 | |
| 12:32:46 | gibi | stephenfin: there are two additional places that uses USE_PYTHON3 in our tree. Can we get rid of those as well now? https://review.opendev.org/#/c/695380 | |
| 12:35:03 | sean-k-mooney | gibi: you mean in the legacy jobs | |
| 12:35:53 | sean-k-mooney | gibi: we could remove them there at this point too however i htink we might need to keep it for grenade | |
| 12:36:35 | gibi | sean-k-mooney: do we have grenade job where the from state is py27 and the dest state is py36? | |
| 12:36:45 | sean-k-mooney | no | |
| 12:36:51 | sean-k-mooney | at least not intentionally | |
| 12:37:05 | sean-k-mooney | we are having issue related to that however | |
| 12:37:32 | sean-k-mooney | specificly because on train devstack install under py2 and py3 on train when you enable py3 | |
| 12:37:52 | gibi | I see. | |
| 12:38:07 | gibi | my knowledge is limited here so I follow your lead | |
| 12:38:10 | sean-k-mooney | and that causees issues with osc and other console script still being the python 2 version in the py3 only ussuri | |
| 12:38:52 | sean-k-mooney | gibi: i belive its still one of the issue tosky is trying to fix in the new zuul v3 native grenade job to get plugins to work | |
| 12:39:11 | gibi | sean-k-mooney: OK, thanks | |
| 12:39:19 | gibi | stephenfin: +A-d https://review.opendev.org/#/c/695380/ | |
| 12:40:00 | sean-k-mooney | gibi: did you have a chance to review the numa proposal following the conversation efried and i had yesterday | |