| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-19 | |||
| 14:44:22 | openstackgerrit | Chris Dent proposed openstack/nova master: Move placement exceptions into the placement package https://review.openstack.org/549862 | |
| 14:44:44 | kashyap | melwitt: mriedem: As a last point, as dansmith also said in different words on Friday here — | |
| 14:44:55 | kashyap | It is not sensible to create an intentionally incompatible syntax in the Nova config file when the actual Nova code to process the config options is largely identical in both cases. | |
| 14:45:34 | openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | |
| 14:47:16 | openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | |
| 14:47:29 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857 | |
| 14:51:12 | openstackgerrit | Lenny Verkhovsky proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640 | |
| 15:01:41 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Address review comments from afdc828db3c9d0205b6ded268db24f5cdf857fa6 https://review.openstack.org/554251 | |
| 15:04:32 | openstackgerrit | Silvan Kaiser proposed openstack/nova master: Exec systemd-run with privileges in Quobyte driver https://review.openstack.org/554195 | |
| 15:06:19 | kaisers1 | efried: fyi https://review.openstack.org/#/c/554195/2 | |
| 15:06:30 | openstackgerrit | Eric Fried proposed openstack/nova master: Report client: Remove version discovery comment https://review.openstack.org/554253 | |
| 15:06:41 | efried | jaypipes, mriedem, cdent ^ | |
| 15:06:45 | efried | kaisers1: ack | |
| 15:06:55 | efried | edleafe: also --^ | |
| 15:06:57 | cfriesen | is github.com enforcing ssh versions now? I just got a "Peer reports incompatible or unsupported protocol version." | |
| 15:07:48 | efried | Could that be what kashyap was running into earlier? | |
| 15:08:14 | kashyap | efried: Probably; it's gone now. I'm able to fetch 'https' from git.o.o. | |
| 15:08:42 | kashyap | cfriesen: I also had troubles this morning w/ GitHub as my 'origin' for Nova | |
| 15:08:57 | cfriesen | looks like they're "weak crypto" versions of https | |
| 15:09:07 | cfriesen | bah, they're *disabling* | |
| 15:09:26 | kashyap | Yeah, apparently | |
| 15:09:58 | efried | Cause it's crucial that the content of a public repo be strongly encrypted. | |
| 15:10:14 | efried | Is that, what, so we don't get MITMs injecting viruses into the codebase for us? | |
| 15:11:03 | efried | tox -e py27 -- trojan_horse ? | |
| 15:11:30 | jaypipes | edleafe, cdent: do we have a bug in placement that diga might be able to pick up? he's looking for some work in placement/scheduler. | |
| 15:12:14 | cfriesen | is it possible to use SSH URLs with github? | |
| 15:12:27 | cfriesen | I don't see any on the nova page | |
| 15:12:55 | jroll | cfriesen: yep, git@github.com:openstack/nova | |
| 15:13:03 | jroll | need to add your public key in settings | |
| 15:13:08 | edleafe | jaypipes: diga: there's always https://bugs.launchpad.net/nova/+bugs?field.tag=placement as a starting point | |
| 15:13:20 | cfriesen | jroll: thanks | |
| 15:13:25 | jroll | np | |
| 15:13:48 | diga | edleafe: Sure, will go through it | |
| 15:14:49 | diga | jaypipes: Thank you for your help | |
| 15:16:43 | jaypipes | diga: if I come across something for you, I will let you know. | |
| 15:17:01 | diga | jaypipes: Sure | |
| 15:21:56 | diga | jaypipes: I also know NFV technologies like hugepages, numa, dpdk very well. If you have something on that, let me know | |
| 15:23:57 | jaypipes | diga: cool. can you review this spec please? https://review.openstack.org/#/c/552924/ | |
| 15:24:17 | diga | jaypipes: sure | |
| 15:28:12 | Spaz-Home | Morning folks | |
| 15:29:39 | openstackgerrit | Chris Dent proposed openstack/nova master: Move placement exceptions into the placement package https://review.openstack.org/549862 | |
| 15:29:39 | openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | |
| 15:29:40 | openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | |
| 15:29:40 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857 | |
| 15:32:13 | Spaz-Home | Actually taking a relax weekend.. no coding.. just boiling 55 Habaneros, 2 Ghost Peppers, and trying to make a hot sauce without killing myself. Enjoy your day Cloud Creators :p | |
| 15:38:03 | cdent | jaypipes: what's the status of your "standardized provider descriptor file" spec/idea? Where on the scale or need<->experimental fishing is that? | |
| 15:38:23 | jaypipes | cdent: it's dead. | |
| 15:38:29 | jaypipes | cdent: I'll abandon it. | |
| 15:38:32 | cdent | dead fish | |
| 15:38:40 | jaypipes | cdent: smelly dead fish. | |
| 15:39:05 | jaypipes | cdent: I did too :) but I understand why dansmith doesn't. | |
| 15:42:25 | mriedem | sahid: there are a couple of issues in the tests for https://review.openstack.org/#/c/497457/2 | |
| 15:42:27 | mriedem | https://review.openstack.org/#/c/497457/ | |
| 15:42:39 | mriedem | sahid: specifically self.assertEqual(0, guest.migrate_configure_max_speed.called) | |
| 15:42:44 | mriedem | .called is a boolean | |
| 15:43:05 | mriedem | it's probably better to use guest.migrate_configure_max_speed.assert_not_called() | |
| 15:44:04 | mriedem | if you're about done for the day i could clean those up and then just approve | |
| 15:44:49 | sahid | oh i was sure of that "called" returns a number | |
| 15:44:52 | sahid | no no, i will | |
| 15:44:59 | mriedem | call_count returns a number | |
| 15:45:03 | sahid | but thanks for asking :) | |
| 15:45:12 | sahid | ack | |
| 15:45:29 | mriedem | thanks for finding that issue in the multinode job config | |
| 15:46:31 | sahid | mriedem: yeah, it's that g-api service which looked very strange | |
| 15:47:06 | openstackgerrit | Jay Pipes proposed openstack/nova master: validate provider summary structure https://review.openstack.org/536516 | |
| 15:47:07 | openstackgerrit | Jay Pipes proposed openstack/nova master: WIP - populate alloc cands for trees https://review.openstack.org/534968 | |
| 15:47:39 | kashyap | sahid: One more nit there: s/TAPs devices/TAP devices/ :-) | |
| 15:47:46 | kashyap | (In driver.py) | |
| 15:48:49 | cfriesen | has anyone run into problems with setuptools recently? I got a report that 39.0.1 didn't work with our version of pike, wondering if it's a general problem. | |
| 15:49:05 | sahid | kashyap: :) sure i will address that too | |
| 15:49:41 | kashyap | Thx | |
| 15:50:42 | dansmith | cfriesen: I had an issue downstream last week where the parse_version was returning a tuple instead of a string | |
| 15:50:56 | dansmith | cfriesen: migi was looking at it, in case you or he know anything else | |
| 15:53:34 | cfriesen | dansmith: we ended up pinning it to 38.5.1 for now, which seems to work okay | |
| 16:00:41 | mriedem | jaypipes: on that vmware resource pools spec, https://review.openstack.org/#/c/549067/6/specs/rocky/approved/vmware-resource-pool-placement.rst@210 | |
| 16:01:05 | mriedem | jaypipes: finally just realized that this is proposing that the admin changes things *in vcenter*, not placement directly, and that's all proxied through the virt driver | |
| 16:01:24 | mriedem | which kind of goes against something we've always said we woudn't do, which is support out of band changes on the hypervisor | |
| 16:02:27 | cdent | mriedem: I think those sorts of changes are pretty much why upt exists? | |
| 16:02:42 | cdent | the hardware underlaying the hypervisor changes in some fashion | |
| 16:02:43 | jaypipes | mriedem: vcenter virt driver is virtually identical (pun intended) to how ironic virt driver wants to do things. i.e. the virt driver (not the compute service worker ala the resource tracker) creates the compute node resource provider records. | |
| 16:02:45 | cdent | upt notices | |
| 16:03:10 | cdent | efried: powervm is much the same, yeah? ^ | |
| 16:03:38 | mriedem | jaypipes: this isn't creating the compute node resource provider record, | |
| 16:03:59 | mriedem | it's seeing, oh the admin created some resource pools in this cluster in vcenter directly, and now the virt driver needs to mirror that into placement, | |
| 16:04:11 | mriedem | and what happens when the admin decides to delete those pools in vcenter? nova would also need to mirror that | |
| 16:04:43 | dansmith | cdent: powervm only has one node | |
| 16:05:19 | mriedem | powervm wants to manage shared storage pool aggregates from what i remember | |
| 16:05:19 | dansmith | jaypipes: I read that spec as not exposing more nodes, but exposing a cluster as a very NUMA single node | |
| 16:05:28 | cdent | dansmith: not talking about the node here, just what the node is aware of: efried has mentioned that the powervm driver wants to ... yeah what mriedem said | |
| 16:05:36 | dansmith | mriedem: right, but not multiple nodes from one service as it stands in tree right now | |
| 16:05:50 | mriedem | these would be nested RPs | |
| 16:06:02 | dansmith | yeah, I think jaypipes is saying multiple nodes a-la ironic | |
| 16:06:07 | dansmith | and I don't think that's on the table for either | |
| 16:06:16 | mriedem | compute node RP (created by nova) -> nested tenant-specific resource pool providers, created by the virt driver, proxied from vcenter | |
| 16:06:30 | mriedem | the 1:M compute node thing isn't in this | |
| 16:06:30 | dansmith | yep | |
| 16:06:34 | dansmith | right | |
| 16:06:39 | cfriesen | mriedem: is there anything in particular blocking https://review.openstack.org/#/c/465787/ ? I notice it hasn't really gotten any reviews other than me. | |
| 16:06:42 | dansmith | I think jaypipes was asserting that it was | |
| 16:07:23 | jaypipes | dansmith: sorry, on a video call for 45 mintues | |