| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-06-14 | |||
| 19:50:28 | dansmith | we know what host we want to use because we just got granted an allocation for one | |
| 19:50:32 | efried | dansmith: Well, I didn't know this bind step was a thing at all. | |
| 19:50:37 | dansmith | we try to bind(), if that works, then we move on | |
| 19:50:49 | Sundar_ | IRC froze on me. Reconnecting | |
| 19:51:06 | dansmith | bind() tells cyborg that we're consuming a thing, it can adjust its inventory before it says success or failure | |
| 19:51:31 | dansmith | once bind returns true, we know that cyborg is ready to commit that thing to us and we'll next talk to cyborg during the plug() part on the compute | |
| 19:52:08 | tlb | Q: _get_computes_for_cells returning subset of available compute nodes across AZs (wrong subset, as it happens); suggestions in the dbeug log that it's cached somewhere (get_or_set_cached_cell_and_set_connections); any idea how/where? not obvious from the code, or at least, not to me. | |
| 19:52:14 | efried | dansmith: Okay. So the PluginBase should have a bind() method with a default `pass` impl which will probably not be overridden in many cases. The bind() step in cyborg will do stuff that diddles with cyborg's internal db and whatnot, and then call into the plugin's bind() method just in case. | |
| 19:52:18 | tlb | *debug | |
| 19:52:20 | Sundar_ | The difference between Neutron ports and an attach handle is this. A port is a virtual object that needs to be created by CLI. An attach handle (at least in some common cases like PCI VFs) are device properties, which can be discovered. We don't need 'openstack create' for that | |
| 19:52:58 | efried | Sundar_: Discovered... | |
| 19:53:01 | dansmith | efried: anything that has a potentially-varying inventory would need to implement it | |
| 19:53:15 | dansmith | Sundar_: no | |
| 19:53:18 | dansmith | that's not it | |
| 19:53:33 | dansmith | Sundar_: we're discussing an attachment handle as purely a handle to some state on the cyborg side | |
| 19:54:10 | efried | Sundar_: I'm going to tell you what frame rate and display heads and resolution I need from my GPU, and you're going to put that profile into your database and return a UUID to me. | |
| 19:54:18 | dansmith | right | |
| 19:54:24 | dansmith | which may mean you're just going to give me a device, | |
| 19:54:34 | dansmith | or it may mean you'll reconfigure some device to give me a slice of it which will support that | |
| 19:54:37 | efried | This is so that, later on, when nova says "hey, give me the details on this UUID" you can give it back the AccGPU object with those details filled in. | |
| 19:54:37 | dansmith | I don't care | |
| 19:54:46 | dansmith | just give me a uuid | |
| 19:54:51 | dansmith | right | |
| 19:55:02 | efried | It's so we don't have to carry (big random dict) across the nova boot command line and down through the conductor to the scheduler and compute etc. | |
| 19:55:21 | dansmith | which is how it used to work with cinder, and that's why it's been painful every single day since | |
| 19:55:44 | efried | I'm going to work up a seqdiag... | |
| 19:55:47 | Sundar_ | "it may mean you'll reconfigure some device" without knowing which compute node is involved, how do you pick a device to reconfigure? | |
| 19:56:02 | efried | Sundar_: That won't happen until the bind() and/or plug() steps. | |
| 19:56:06 | dansmith | Sundar_: you won't until we bind | |
| 19:56:16 | dansmith | at that point, you get to do that, decide if there's room, etc | |
| 19:56:21 | efried | That's the point - we want to save off configuration details before we know where we're going to land. | |
| 19:57:05 | Sundar_ | "bind() tells cyborg that we're consuming a thing" Is this invoked by n-cpu on os-acc/Cyborg? | |
| 19:57:28 | dansmith | it's likely done by conductor, talking to cyborg, through os-acc | |
| 19:58:28 | dansmith | efried: I have to wander off for a bit, but I think we're on the same page | |
| 19:58:52 | efried | dansmith: ack. I'll make sure you see this diagram :) | |
| 19:59:02 | dansmith | I literally can not friggin wait | |
| 19:59:09 | dansmith | (that's a lie, I can wait) | |
| 19:59:38 | efried | dansmith: If you need something to keep your horses held until then, I still need your comeback on the reshape spec. | |
| 19:59:46 | Sundar_ | efried: You offered to build a flow/sequence diagram. Thanks. Please LMK when it is ready. May be it will make things more concrete. Not sure if we have the same mental model, but let's discuss on an etherpad | |
| 20:00:00 | efried | Sundar_: Roger wilco. | |
| 20:04:29 | melwitt | I started the rocky blueprint status etherpad here, still have some more to go through https://etherpad.openstack.org/p/nova-rocky-blueprint-status | |
| 20:04:49 | mriedem | melwitt: thanks for doing that | |
| 20:04:52 | mriedem | don't you feel better now?! | |
| 20:05:15 | melwitt | sorta | |
| 20:13:09 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | |
| 20:13:10 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add trusted certs to feature support matrix docs https://review.openstack.org/574890 | |
| 20:13:11 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove max_size parameter from fake_libvirt_utils.fetch_*image methods https://review.openstack.org/574911 | |
| 20:13:12 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix nits from trusted certs notification change https://review.openstack.org/575521 | |
| 20:22:09 | mriedem | huh, for some reason the ironic job is running on our docs-only changes https://review.openstack.org/#/c/574890/ | |
| 20:22:12 | mriedem | ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa SUCCESS in 45m 30s (non-voting) | |
| 20:23:04 | mriedem | http://git.openstack.org/cgit/openstack/ironic/tree/zuul.d/legacy-ironic-jobs.yaml#n4 | |
| 20:23:24 | mriedem | ^doc/.*$ should match and skip | |
| 20:23:27 | melwitt | heh. that's odd | |
| 20:23:38 | melwitt | oh, hold the phone | |
| 20:23:59 | mriedem | because they changed how irrelevant-files works? | |
| 20:24:08 | melwitt | maybe, that's what I was wondering http://lists.openstack.org/pipermail/openstack-dev/2018-June/131304.html | |
| 20:24:23 | melwitt | unless it's just always been like that and no one noticed | |
| 20:25:53 | mriedem | i see why | |
| 20:25:57 | mriedem | http://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul.d/projects.yaml#n10496 | |
| 20:27:35 | openstack | Launchpad bug 1776981 in OpenStack Compute (nova) "ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa is running on non-code changes" [Undecided,New] | |
| 20:27:35 | mriedem | https://bugs.launchpad.net/nova/+bug/1776981 | |
| 20:28:01 | mriedem | low-hanging-vegetables | |
| 20:29:26 | melwitt | we could add that to the bug smash list if we have aggregated a list | |
| 20:30:31 | mriedem | done https://etherpad.openstack.org/p/OpenSource-Hackathon-Rocky-Beijing-Bugs-List | |
| 20:30:57 | melwitt | woot thanks | |
| 20:43:05 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add trusted certs to feature support matrix docs https://review.openstack.org/574890 | |
| 20:43:06 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove max_size parameter from fake_libvirt_utils.fetch_*image methods https://review.openstack.org/574911 | |
| 20:43:07 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix nits from trusted certs notification change https://review.openstack.org/575521 | |
| 20:43:20 | mriedem | all ^ should be quick approvals | |
| 20:43:39 | mriedem | which gets all the server side stuff done for trusted certs, and then we just have the client | |
| 20:43:52 | mriedem | https://review.openstack.org/#/c/500396/ | |
| 20:44:58 | mriedem | dansmith: +2 on https://review.openstack.org/#/c/575493/ - easy +W | |
| 20:47:03 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Time how long pre_live_migration() takes https://review.openstack.org/574481 | |
| 20:47:57 | dansmith | woot | |
| 20:48:29 | mriedem | ha, check the file LOC https://review.openstack.org/#/c/560158/ | |
| 20:48:42 | mriedem | the devil's docs! | |
| 20:53:54 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: WIP: Nova-Cyborg workflow https://review.openstack.org/575545 | |
| 20:58:08 | melwitt | spooky | |
| 21:15:49 | tonyb | dansmith, mriedem: There is a special recheck rdo command to do that but it's currently disabled due to overload in RDOCloud | |
| 21:16:27 | tonyb | dansmith, mriedem: You want the periodic-tripleo-ci-centos-7-ovb-1ctlr_1comp-featureset002-queens-upload or similar job | |
| 21:19:28 | dansmith | ah, okay cool | |
| 21:20:10 | mriedem | do i really want it? | |
| 21:20:15 | mriedem | with a name like that, i don't think i do | |
| 21:32:32 | mriedem | efried: nova-status upgrade check for placement finds an OSA victim http://logs.openstack.org/25/575125/5/check/openstack-ansible-upgrade-ubuntu-xenial/222b6ac/job-output.txt.gz#_2018-06-14_21_00_49_582359 | |
| 21:32:45 | mriedem | Details: Placement API version 1.24 needed, you have 1.17. | |
| 21:33:50 | efried | mriedem: Good deal. | |
| 21:36:37 | efried | mriedem: Who else can review https://review.openstack.org/#/c/575140/ ? | |
| 21:53:35 | openstackgerrit | Merged openstack/nova master: [placement] Add status and links fields to version document at / https://review.openstack.org/575117 | |
| 21:55:38 | openstackgerrit | Merged openstack/nova master: Use oslo.messaging per-call monitoring https://review.openstack.org/566696 | |
| 21:58:49 | dansmith | \o/ | |
| 22:01:38 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle broken instance mapping in InstanceLister https://review.openstack.org/575556 | |
| 22:01:39 | mriedem | melwitt: dansmith: for your morrow ^ see the todo question inline about build requests | |
| 22:01:57 | dansmith | okay I need to be gone already | |
| 22:03:45 | openstackgerrit | Merged openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269 | |
| 22:06:04 | melwitt | mriedem: ack | |
| 22:08:03 | mriedem | melwitt: https://review.openstack.org/#/c/574890/ is using OSC | |
| 22:08:32 | melwitt | oh, thanks | |
| 22:08:40 | melwitt | I thought they would use the same args | |
| 22:09:34 | mriedem | osc uses repeating args | |
| 22:09:47 | mriedem | --trusted-image-certificate 1 --trusted-image-certificate 2 | |