| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-06-14 | |||
| 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 | dansmith | I don't care | |
| 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: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 | mriedem | https://bugs.launchpad.net/nova/+bug/1776981 | |
| 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: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 | |
| 22:10:04 | mriedem | i'm not sure we have much of that in nova cli | |
| 22:10:22 | mriedem | we definitely could though | |
| 22:10:27 | mriedem | and it would be good to be consistent if we can | |
| 22:11:08 | melwitt | I think there is something like that but can't remember right now | |
| 22:11:22 | melwitt | I'm not sure that users enjoy the repeating args thing | |
| 22:12:49 | mriedem | i think it's a pretty standard pattern in osc | |
| 22:13:09 | mriedem | having to know the delimiter arg for a list isn't fun either | |
| 22:13:53 | mriedem | the --block-device stuff on nova boot is all append | |
| 22:14:01 | mriedem | so we likely should be doing the same thing in nova boot for trusted certs | |
| 22:15:44 | melwitt | fair enuff | |
| #openstack-nova - 2018-06-15 | |||
| 01:16:13 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (9) https://review.openstack.org/575581 | |
| 01:27:37 | openstackgerrit | jiang wei proposed openstack/nova master: Add action initiator attribute to the update payload https://review.openstack.org/536243 | |
| 01:39:21 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add full traceback to ExceptionPayload in versioned notifications https://review.openstack.org/564092 | |
| 02:18:37 | openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387 | |
| 02:18:38 | openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658 | |
| 02:18:39 | openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240 | |
| 02:18:40 | openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340 | |