| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-05 | |||
| 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... | |
| 19:55:44 | efried | jaypipes But the alternative is making them separate resource classes. Which doesn't seem right at all. | |
| 19:55:58 | jaypipes | efried: no, that doesn't work either... | |
| 19:56:12 | jaypipes | efried: because their not separate resource classes. they're separate providers. | |
| 19:56:35 | jaypipes | efried: just imagine for a second how one might ask AWS for such a thing. | |
| 19:56:55 | jaypipes | efried: you can't. | |
| 19:57:06 | efried | uhm | |
| 19:57:10 | jaypipes | efried: you need a more complicated DSL like CloudFormation or TOSCA | |
| 19:57:32 | efried | So how are we planning to model e.g. asking for more than one cinder volume? | |
| 19:57:36 | jaypipes | efried: and an orchestrator that would take one of those request templates and produce multiple requests to EC2 APIs. | |
| 19:58:09 | jaypipes | efried: we don't. that's the very reason why we've pushed back on having the compute API be an orchestration system. | |
| 19:58:35 | mriedem | prometheanfire: i think i've got these xenapi unit tests fixed | |
| 19:58:38 | mriedem | should have a patch up shortly | |
| 19:58:43 | prometheanfire | cool | |
| 19:58:44 | jaypipes | efried: you want >1 cinder volumes? then create them in cinder and attach them to your compute instance. | |
| 19:58:59 | prometheanfire | if that merges today then the xenapi change can go in tomorrow via bot update | |
| 19:59:04 | jaypipes | efried: there is absolutely no "atomically create 10 volumes in Cinder" command. | |
| 19:59:38 | mriedem | there isn't?! | |
| 19:59:40 | jaypipes | efried: and these kinds of requests that the VNFM would be making are essentially that: requests for a set of related resources to be created as an atomic unit. | |
| 20:00:14 | efried | jaypipes Seems like there should be a way to specify multiple `resources=` keys in a single request, kind of thing. | |
| 20:00:18 | mriedem | how about i request 10 VMs with block_device_mapping_v2 in the request, of size 10, for each entry with different device names so nova creates the volumes! | |
| 20:00:29 | jaypipes | incidentally, this is why there's no atomicity guarantees that *any* orchestrator provides. not k8s, not heat, not IBM "smart" cloud orchestrator. not any of them. | |
| 20:00:47 | mriedem | does sco still exist? | |
| 20:00:56 | jaypipes | heh, no idea. | |
| 20:01:03 | smcginnis | mriedem: I hear they own the copyright to OpenStack. | |
| 20:01:04 | efried | only as a lawsuit troller | |
| 20:01:18 | mriedem | they do have a snapshot api for vmware instances | |
| 20:01:22 | mriedem | which is different from os-createImage | |