| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-05 | |||
| 18:26:28 | efried | First of all, is this along the lines of what you've been thinking? | |
| 18:27:05 | efried | Doesn't have to be percentage; could be bps or whatever; point is the number of VFs and the total bandwidth are separate but parallel resource classes in the RP. | |
| 18:27:33 | cdent | “parallel”? | |
| 18:27:59 | efried | just meaning that they're RC inventories on the same RP | |
| 18:28:37 | cdent | isn’t that what we do around here? | |
| 18:28:45 | efried | Some of us shouldn't. | |
| 18:29:25 | jaypipes | efried: yep. that's pretty close. I don't like the pct versus using bytes_per_sec but yeah. | |
| 18:29:39 | jaypipes | efried: in fact, that's pretty close to what I had written on the ML, no? | |
| 18:29:47 | efried | jaypipes okay, so help me fill in the gaps here. | |
| 18:30:16 | efried | IRL I'll have multiple PF-y RPs, and let's say they each have both of those RC inventories defined. | |
| 18:30:30 | jaypipes | k | |
| 18:31:14 | efried | When I make a request for, say, SRIOV_VFS=1,EGRESS_BW_PCT=20 -- what's to stop placement from allocating the VF inventory from one RP and the egress bandwidth inventory from another?? | |
| 18:32:47 | openstackgerrit | Mateusz Kowalski proposed openstack/nova stable/pike: Handle keypair not found from metadata server using cells https://review.openstack.org/500953 | |
| 18:32:59 | openstackgerrit | Mateusz Kowalski proposed openstack/nova stable/ocata: Handle keypair not found from metadata server using cells https://review.openstack.org/500954 | |
| 18:36:59 | jaypipes | efried: actually currently stressing out trying to figure out how to deal with this hurricane that is going to hit us. | |
| 18:37:13 | efried | jaypipes Been there, done that. Where you at? | |
| 18:37:26 | jaypipes | efried: and getting my girls to safety and my wife to her plane flight to Italy on Sunday :( | |
| 18:37:33 | jaypipes | efried: Sarasota | |
| 18:38:29 | efried | Totally counts as (b) | |
| 18:40:28 | jaypipes | efried: the answer to your question is: nothing would stop placement from doing so. | |
| 18:40:52 | efried | jaypipes Then... "eek". | |
| 18:40:59 | jaypipes | efried: yeah | |
| 18:41:37 | efried | The followup, of course, is what about SRIOV_VFS=4,EGRESS_BW_PCT=20 ? | |
| 18:42:11 | jaypipes | efried: we would need to add some mechanism to the request that says "make sure the thing providing EGRESS_BW_PCT and SRIOV_VF is the same provider. | |
| 18:42:44 | efried | jaypipes Rojah dat. Any thoughts been assayed in that regard to this point? | |
| 18:43:39 | melwitt | mriedem: I'm gonna sign up for that nova project update redhat interview thing at the PTG | |
| 18:44:05 | jaypipes | efried: resource_constraints=same_provider:SRIOV_NET_VF,NET_EGRESS_BYTES_SEC&resources=SRIOV_NET_VF:1&NET_EGRESS_BYTES_SEC:20000 | |
| 18:44:06 | mriedem | melwitt: awesome | |
| 18:44:53 | openstackgerrit | Merged openstack/nova master: Replace dd with shred for zeroing lvm volumes. https://review.openstack.org/495532 | |
| 18:46:35 | efried | jaypipes What stage is that semantic in? Just popped out of your head, in a spec somewhere, implemented, released? | |
| 18:48:20 | jaypipes | efried: popped off the top of my thick, oh so thick, head of hair. | |
| 18:48:30 | melwitt | mriedem: let me know if there's anything else you'd like me to highlight other than cells and placement. I was gonna go through the rel notes and dev ML announcements to see what else to mention | |
| 18:49:10 | melwitt | and then I'll use the nova PTG etherpad to summarize what's coming up in queens | |
| 18:49:10 | efried | jaypipes Cool, will add it to discussion points. | |
| 18:49:46 | efried | I still need to complete my reading, but presumably there's a syntax for requesting multiple different resources of the same class with different traits? | |
| 18:51:26 | efried | jaypipes So like, pursuant to the anti-affinity grouping conversation avolkov and I were having earlier, I want to be able to say gimme (SRIOV_NET_VF:1&NET_EGRESS_BYTES_SET:2000&traits=CUSTOM_PF_HA_GROUP_1; SRIOV_NET_VF:1&NET_EGRESS_BYTES_SET:2000&traits=CUSTOM_PF_HA_GROUP_2) | |
| 18:51:36 | mriedem | melwitt: i can send you my list that i sent internally | |
| 18:52:15 | melwitt | mriedem: cool, I think that would be helpful. thanks | |
| 18:54:41 | jaypipes | efried: lemme pastebin something for that | |
| 18:54:43 | prometheanfire | this look like an error with the os-xenapi bump? http://logs.openstack.org/70/500770/3/check/gate-cross-nova-python27-ubuntu-xenial/1305547/testr_results.html.gz | |
| 18:54:50 | prometheanfire | from https://review.openstack.org/500770 | |
| 18:55:44 | mriedem | prometheanfire: looking | |
| 18:56:20 | openstackgerrit | Michael Still proposed openstack/nova master: Move nbd commands to privsep. https://review.openstack.org/500351 | |
| 18:56:21 | openstackgerrit | Michael Still proposed openstack/nova master: Move lvm handling to privsep. https://review.openstack.org/495516 | |
| 18:56:21 | openstackgerrit | Michael Still proposed openstack/nova master: Move xend existence probes to privsep. https://review.openstack.org/495538 | |
| 18:56:22 | openstackgerrit | Michael Still proposed openstack/nova master: Move shred to privsep. https://review.openstack.org/495537 | |
| 18:56:22 | openstackgerrit | Michael Still proposed openstack/nova master: Cleanup mount / umount and associated rmdir calls https://review.openstack.org/494423 | |
| 18:56:23 | openstackgerrit | Michael Still proposed openstack/nova master: WIP / Aspirational: we don't need rootwrap any more. https://review.openstack.org/495542 | |
| 18:56:23 | openstackgerrit | Michael Still proposed openstack/nova master: Move loopback setup and removal to privsep. https://review.openstack.org/495664 | |
| 18:56:26 | openstackgerrit | Michael Still proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972 | |
| 18:56:27 | openstackgerrit | Michael Still proposed openstack/nova master: Don't shell out to mkdir, use ensure_tree() https://review.openstack.org/492326 | |
| 18:56:28 | openstackgerrit | Michael Still proposed openstack/nova master: Move the idmapshift binary into privsep. https://review.openstack.org/495541 | |
| 18:56:28 | openstackgerrit | Michael Still proposed openstack/nova master: Move ploop commands to privsep. https://review.openstack.org/492325 | |
| 18:56:30 | openstackgerrit | Michael Still proposed openstack/nova master: Read from console ptys using privsep. https://review.openstack.org/489486 | |
| 18:56:31 | openstackgerrit | Michael Still proposed openstack/nova master: Move kpartx calls to privsep. https://review.openstack.org/500354 | |
| 18:56:35 | openstackgerrit | Michael Still proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229 | |
| 18:56:36 | openstackgerrit | Michael Still proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166 | |
| 18:56:38 | openstackgerrit | Michael Still proposed openstack/nova master: Move execs of touch to privsep. https://review.openstack.org/489190 | |
| 18:56:38 | prometheanfire | mriedem: that's a quick git-review you have there (or rebase maybe) | |
| 18:56:39 | openstackgerrit | Michael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep. https://review.openstack.org/490737 | |
| 18:56:39 | openstackgerrit | Michael Still proposed openstack/nova master: Move blkid calls to privsep. https://review.openstack.org/500398 | |
| 18:56:41 | openstackgerrit | Michael Still proposed openstack/nova master: Move execs of tee to privsep. https://review.openstack.org/489438 | |
| 18:56:53 | prometheanfire | bah, meant that for michael | |
| 18:57:41 | openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/500011 | |
| 18:57:46 | efried | prometheanfire git restack, yo. | |
| 18:58:09 | prometheanfire | what that come from? | |
| 18:58:22 | efried | It's a thing, hold on... | |
| 18:59:19 | efried | prometheanfire I think you install it with `pip install git-restack` | |
| 18:59:44 | efried | It's a git plugin maintained by stackers | |
| 19:00:49 | mriedem | prometheanfire: looks like this is the regression https://github.com/openstack/os-xenapi/commit/755876c19f7c47eaa773f835a68dcf93c3d6b50d | |
| 19:01:14 | efried | prometheanfire You say `git restack <commit-before-the-bottom-of-the-pile>`. Then you can choose to edit whichever ones in the chain via an edit file that looks kinda like when you rebase -i. It auto rebases everything in the pile. Then when you're done, you can `git review` and it pushes the whole pile (anything that has been changed, including rebases) | |
| 19:01:55 | prometheanfire | mriedem: k, so nova needs to update it's tests? | |
| 19:02:08 | prometheanfire | efried: I'll probably just stick to the classic rebase then | |
| 19:02:55 | mriedem | prometheanfire: not sure if it's that or if os-xenapi is regressed | |
| 19:03:02 | mriedem | i can start with reporting a bug | |
| 19:03:27 | prometheanfire | k | |
| 19:03:35 | prometheanfire | I'll hold that back for now then | |
| 19:03:41 | mriedem | thanks | |
| 19:05:09 | efried | jaypipes I'll have the same question about the "multiple resources, same class, different traits" syntax (in what state of maturity is this idea?) | |
| 19:08:24 | mriedem | prometheanfire: https://bugs.launchpad.net/nova/+bug/1715217 | |
| 19:08:24 | openstack | Launchpad bug 1715217 in OpenStack Compute (nova) "nova xenapi unit tests fail with os-xenapi 0.3.0" [Undecided,New] | |
| 19:09:17 | prometheanfire | thanks | |
| 19:13:26 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs to instance_extra https://review.openstack.org/457711 | |
| 19:17:28 | artom | How are our man pages generated? Am I understanding correctly that https://review.openstack.org/#/c/475810/ has nothing to do with man pages? | |
| 19:17:36 | artom | stephenfin ^^, since you're the patch author :) | |
| 19:24:10 | mriedem | artom: https://github.com/openstack/nova/blob/master/doc/source/conf.py#L113 | |
| 19:24:14 | mriedem | part of the sphinx build | |
| 19:24:41 | mriedem | and yes https://review.openstack.org/#/c/475810/ has nothing to do with man pages | |
| 19:24:57 | artom | mriedem, and those all come from the doc/source/cli directory? | |
| 19:25:13 | mriedem | yeah | |
| 19:25:23 | artom | Thanks :) | |
| 19:25:31 | mriedem | did i win? | |
| 19:25:46 | artom | My eternal indifference | |
| 19:26:06 | artom | (It's 2 steps up from seething hatred) | |
| 19:49:28 | openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: Libvirt: Native LUKS file and host device decryption by QEMU https://review.openstack.org/490824 | |
| 19:53:53 | jaypipes | efried: I don't know how to structure a request that would impart the required information for this query :( | |
| 19:54:27 | efried | jaypipes Well, shit, if YOU can't do it.... | |
| 19:54:32 | efried | Adding to list. | |
| 19:54:42 | jaypipes | efried: we would essentially need to replicate child-parent information in the request for resources. in other words, the requestor must know that a VF is a child of a PF is a child of a compute host. | |
| 19:55:06 | jaypipes | efried: and we can't ask the user to know that relationship... | |