| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-01-29 | |||
| 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 | it doesn't inherit from something real, so ignore my previous comments | |
| 20:25:47 | dansmith | a wrapper class for ease makes perfect sense I think | |
| 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 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: NUMA LM: Add func test for bug 1845146 https://review.opendev.org/687404 | |
| 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) | |
| 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: Create and bind Cyborg ARQs. https://review.opendev.org/631244 | |
| 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: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: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:27 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Enable hard/soft reboot with accelerators. https://review.opendev.org/697940 | |
| 04:14:27 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Enable start/stop of instances with accelerators. https://review.opendev.org/699553 | |
| 04:14:28 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Enable and use COMPUTE_ACCELERATORS trait. https://review.opendev.org/699554 | |
| 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: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 | |
| 12:40:28 | stephenfin | gibi: Thanks. Looks like sean-k-mooney addressed your earlier question :) | |
| 12:40:53 | sean-k-mooney | stephenfin: acully ^ goes for you and bauzas | |
| 12:42:11 | stephenfin | I saw the etherpad, yup. LGTM | |
| 12:42:22 | sean-k-mooney | stephenfin: bauzas gibi https://review.opendev.org/#/c/552924/15/specs/ussuri/approved/numa-topology-with-rps.rst@126 is the main comment. | |
| 12:43:30 | sean-k-mooney | stephenfin: cool | |
| 12:43:35 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Re-propose the flavor extra spec validation spec https://review.opendev.org/682655 | |
| 12:43:48 | sean-k-mooney | hehe i was just reviewing that | |
| 12:43:57 | sean-k-mooney | although its small so ill start again | |
| 12:45:52 | gibi | sean-k-mooney: I read the scrollback but did not read the etherpad and the review yet. stay tuned | |
| 12:46:08 | sean-k-mooney | will do | |
| 12:46:31 | sean-k-mooney | the etherpad has a diagram which hopefully makes the design clear | |
| 12:46:54 | sean-k-mooney | when it comes to placement pictures help a lot | |
| 12:47:36 | gibi | totally agree ^^ | |
| 12:55:45 | sean-k-mooney | stephenfin: im happy with the latest version of the extra spec validation | |
| 12:58:52 | gibi | stephenfin: I have one, hopefully the last question in https://review.opendev.org/#/c/682655/5/specs/ussuri/approved/flavor-extra-spec-validators.rst@152 | |
| 13:02:30 | openstackgerrit | Merged openstack/nova-specs master: Non-Admin user can filter their instances by more filters https://review.opendev.org/701763 | |