Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-02-06
15:12:45 sean-k-mooney stephenfin: OPN?
15:13:08 sean-k-mooney oh its a vendor id
15:13:12 stephenfin sean-k-mooney: The second column in that firmware download list. I assume it's the part ID
15:13:32 sean-k-mooney ya it like intels etag for there firmware
15:13:47 sean-k-mooney at least it looks like it is anyway
15:14:00 stephenfin sean-k-mooney: I think that's only good to distinguish between e.g. the ConnectX-3 and ConnectX-3 Pro, based on http://www.mellanox.com/page/firmware_HCA_FW_identification
15:14:28 stephenfin and the ID I'm getting, 15b3:1003, doesn't give me any deets on https://www.pcilookup.com/?ven=15b3&dev=1003&action=submit
15:15:44 stephenfin adrianc: I had the PSID and just clicked down through the list of OPNs to find what I was looking for. If there's a better way though, I'd be happy to hear about it
15:15:46 stephenfin sean-k-mooney: ^
15:17:02 openstackgerrit Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268
15:17:02 openstackgerrit Balazs Gibizer proposed openstack/nova master: Extend RequestGroup object for mapping https://review.openstack.org/619527
15:17:03 openstackgerrit Balazs Gibizer proposed openstack/nova master: Calculate RequestGroup resource provider mapping https://review.openstack.org/616239
15:17:03 openstackgerrit Balazs Gibizer proposed openstack/nova master: Fill the RequestGroup mapping during schedule https://review.openstack.org/619528
15:17:04 openstackgerrit Balazs Gibizer proposed openstack/nova master: Pass resource provider mapping to neutronv2 api https://review.openstack.org/616240
15:17:04 openstackgerrit Balazs Gibizer proposed openstack/nova master: Recalculate request group - RP mapping during re-schedule https://review.openstack.org/619529
15:17:05 openstackgerrit Balazs Gibizer proposed openstack/nova master: Send RP uuid in the port binding https://review.openstack.org/569459
15:17:05 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317
15:17:06 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove port allocation during detach https://review.openstack.org/622421
15:17:06 openstackgerrit Balazs Gibizer proposed openstack/nova master: Record requester in the InstancePCIRequest https://review.openstack.org/625310
15:17:07 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add pf_interface_name tag to passthrough_whitelist https://review.openstack.org/625311
15:17:07 openstackgerrit Balazs Gibizer proposed openstack/nova master: Ensure that bandwidth and VF are from the same PF https://review.openstack.org/623543
15:17:40 jaypipes stephenfin: ok, will try. (sorry, been out of IRC contact for 6+ days)
15:19:24 stephenfin jaypipes: It's all good. Thanks for even trying
15:30:27 mriedem why oh why doesn't the set -x here https://review.openstack.org/#/c/634962/2/nova/tests/live_migration/hooks/run_tests.sh@12 enable tracing?
15:30:46 mriedem http://logs.openstack.org/62/634962/2/check/legacy-grenade-dsvm-neutron-multinode-live-migration/f8f245f/logs/devstack-gate-post_test_hook.txt.gz#_2019-02-05_22_35_09_439
15:31:12 mriedem oh i bet something downstream unsets it
15:32:53 mriedem aha http://logs.openstack.org/62/634962/2/check/legacy-grenade-dsvm-neutron-multinode-live-migration/f8f245f/logs/devstack-gate-post_test_hook.txt.gz#_2019-02-05_22_35_09_525
15:36:19 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Fix legacy-grenade-dsvm-neutron-multinode-live-migration https://review.openstack.org/634962
15:36:35 bauzas gibi_off: would you mind if I was rebasing https://review.openstack.org/#/c/631559/ on top of the new revision I'm working for https://review.openstack.org/#/c/599208/
15:36:37 bauzas ?
15:37:33 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Allow [pci]alias to take a list https://review.openstack.org/635191
15:37:51 mriedem bauzas: i'm sure he doesn't mind
15:37:57 bauzas ack
15:46:46 openstackgerrit Jack Ding proposed openstack/nova master: Improve libvirt image and snapshot handling https://review.openstack.org/616692
15:55:08 stephenfin sean-k-mooney: https://that.guru/blog/updating-mellanox-connectx-3/
15:55:27 stephenfin adrianc: ^ if you figure that out, I'll update the above
15:59:58 openstackgerrit sean mooney proposed openstack/nova master: update flavor admin docs https://review.openstack.org/635198
16:04:41 sean-k-mooney stephenfin: i assume this should not happen http://paste.openstack.org/show/744632/
16:05:26 stephenfin sean-k-mooney: You've got left over pyc files
16:05:34 openstackgerrit Merged openstack/nova master: Ignore some PendingDeprecationWarnings for os-vif https://review.openstack.org/634928
16:05:35 stephenfin and the interpreter is trying to read them
16:05:43 sean-k-mooney oh even after -r
16:05:46 stephenfin 'find . -type f -name "*.pyc" -delete'
16:05:58 melwitt o/
16:06:35 mriedem sean-k-mooney: stephenfin: we used to have something in tox.ini to delete those on each run to avoid that but someone had an opinion and removed it
16:07:03 sean-k-mooney mriedem: yes i rememebr i was going to check if it was remvoed
16:07:44 stephenfin mriedem: It's still there and that sounds rather passive aggressive. If that's intentional, stop it please. No need for that
16:07:48 stephenfin We just don't do it for docs
16:07:53 sean-k-mooney stephenfin: anyway that seams to have partly fixed it.
16:08:01 sean-k-mooney i now get dot command 'dot' cannot be run (needed for graphviz output), check the graphviz_dot setting
16:08:38 stephenfin sean-k-mooney: Yup, if you fancy sticking a call to '{[testenv]commands}' in the commands for the 'docs' target, it'd be an easy fix to prevent that recurring
16:08:51 stephenfin sean-k-mooney: Guess you need to install dot?
16:08:51 sean-k-mooney ya i can
16:08:54 mriedem huh?
16:09:00 mriedem we don't delete pyc in the tox docs env
16:09:03 mriedem b/c we override commands
16:09:09 mriedem what am i being passive aggressive about?
16:09:30 mriedem i hit this same thing from time to time and have to remember to delete pycs manually which is annoying
16:10:20 sean-k-mooney ill update the docs tox env but i think we need to add graphviz as a docs dep too
16:10:29 mriedem graphviz is a binary
16:10:32 mriedem it's in bindep
16:10:48 sean-k-mooney you are right but why do we now need it for docs
16:10:53 sean-k-mooney we didnt in the past
16:10:59 mriedem we've needed it for a long time
16:11:02 mriedem are you on a new system?
16:11:18 sean-k-mooney im on a system that has never had devstack run
16:11:20 sean-k-mooney so yes
16:11:28 sean-k-mooney ok ill just install it
16:12:09 jackding mriedem: runway for https://review.openstack.org/#/c/620706/ ends tomorrow. I believe I have addressed your comments. Do you have sometime to revisit?
16:12:12 mriedem stephenfin: this is what i was referring to http://lists.openstack.org/pipermail/openstack-dev/2014-September/045873.html
16:12:38 stephenfin mriedem: "we used to have something in tox.ini to delete those on each run to avoid that but someone had an opinion and removed it"
16:12:44 stephenfin mriedem: I figured that was aimed at me, in which case it seemed like I was being singled out for what was almost certainly a change made for good reasons
16:12:50 stephenfin as opposed to "opinions"
16:13:01 mriedem i was referring to that thread
16:13:13 mriedem which was related to https://review.openstack.org/#/c/338231/
16:13:25 mriedem which was yours, but that was just "don't create pycs rather than delete them" which as you can see i was +2 on
16:13:50 stephenfin Yeah, we reverted that though 233160644f4f0a17177fffe8a1e6db1341e61ad4
16:13:56 mriedem i know
16:14:38 sean-k-mooney anyway ill push a patch to add the delete to the docs job if ye feel like reviewing
16:14:40 stephenfin Cool. So this has just been missed is all. Surprised it didn't come up before now, tbh
16:14:56 mriedem sean-k-mooney: just do what stephenfin asked for
16:15:05 mriedem '{[testenv]commands}'
16:15:17 sean-k-mooney ya that was what i was going to do
16:15:42 sean-k-mooney i was looking up the syntax :)
16:16:37 stephenfin mriedem: But, crucially, I didn't remove it (or anything else, I'd like to think) the first day because of "opinions". I gave sound reasons in that commit message. I just missed the tox issue, which was the reason for the later revert
16:16:39 stephenfin That is all :)
16:16:59 stephenfin Speaking of, as soon as we support a new enough version of tox, we can revert that revert again
16:17:27 stephenfin https://github.com/tox-dev/tox/commit/336f4f6bd8b53223f940fc5cfc43b1bbd78d4699
16:20:05 melwitt gibi_off: thank you for the note! I hope you are having a nice recovery
16:27:07 mriedem stephenfin: question inline https://review.openstack.org/#/c/635170/
16:27:37 stephenfin mriedem: Yeah, I am
16:28:10 stephenfin sean-k-mooney, jangutter: You should probably look at the above too
16:28:44 sean-k-mooney stephenfin: for vhostuser i dont think they work
16:28:52 sean-k-mooney stephenfin: that is why we dont set them today
16:29:14 sean-k-mooney stephenfin: libvirt sets tehm on the tap device but vhost user does not have a tap
16:29:53 jangutter sean-k-mooney: yah vhost-user has a socket.
16:30:56 sean-k-mooney stephenfin: jangutter well the main point is that only way to do ratelimiting with vhost-user that im aware of is via dpdk/ovs
16:31:24 jangutter sean-k-mooney: ya, there's no netdev to set nifty options like rxqueuelen, etc.
16:31:29 sean-k-mooney tc cant do ratelimiting on a unix socket escpially since that socket is part of the contol plane not the data plane
16:31:55 sean-k-mooney jangutter: well you can set the queue lenght in the qemu commandline

Earlier   Later