| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-08 | |||
| 14:18:15 | dtantsur | sdague: it's a bug, and I thought dansmith fixed it some time ago.. | |
| 14:18:34 | stephenfin | johnthetubaguy: Unfortunately a lot of it is wrong :( especially the stuff around extra_specs | |
| 14:18:49 | johnthetubaguy | stephenfin: joys | |
| 14:19:12 | stephenfin | I've trying to clean both that, and the specs I'm having to review for background info, this week | |
| 14:19:14 | sdague | dtantsur: you able to ask enough questions on that bug to figure out what is going on? | |
| 14:19:15 | dansmith | dtantsur: sdague: It's working as designed, we can't put compute node records in without anything reported from ironic.. that's a fundamental thing | |
| 14:19:18 | stephenfin | Extra specs suuuuuuuck | |
| 14:19:24 | sdague | dansmith: ok | |
| 14:19:37 | dtantsur | dansmith: except that it creates a horrible user experience | |
| 14:19:55 | dansmith | sdague: we write 1:1 ironic node reported from ironic into the compute nodes table.. if zero are reported (or we can't do the list) then we have nothing to write | |
| 14:20:26 | dansmith | sdague: there is a periodic task you can enable on scheduler that will run discovery every once in a while, which is a workaround | |
| 14:20:40 | dansmith | dtantsur: I dunno what to say.. I can't report things I don't know are there | |
| 14:20:44 | sdague | dansmith: ok, cool, I'll close as working as designed | |
| 14:20:48 | dtantsur | then we have to document this periodic task a mandatory for ironic.. | |
| 14:20:51 | dtantsur | s/a/as/ | |
| 14:22:57 | dtantsur | dansmith: https://github.com/openstack/nova/commit/62878ef5a6c9f6a2c727c91397830387425c4d37 | |
| 14:23:25 | dtantsur | isn't it fixing precisely that? | |
| 14:24:22 | dtantsur | hmm, maybe not actually | |
| 14:24:30 | openstackgerrit | Elod Illes proposed openstack/nova master: Transform aggregate.remove_host notification https://review.openstack.org/396211 | |
| 14:25:47 | dtantsur | dansmith: this means that we should either document running nova-manage after adding every ironic node/bulk of nodes or document using this periodic task, right? | |
| 14:25:56 | mriedem | stephenfin: "What flavors are why they are used."? | |
| 14:27:07 | stephenfin | damn | |
| 14:27:07 | stephenfin | *and | |
| 14:27:22 | stephenfin | *What are flavors and why are they used | |
| 14:28:10 | openstackgerrit | Merged openstack/nova master: Typo error about help resource_classes.inc https://review.openstack.org/501930 | |
| 14:29:43 | mriedem | ^ | |
| 14:29:43 | mriedem | inspur is going on a spell check tear i've noticed recently | |
| 14:29:51 | mriedem | that's the 2nd patch i've seen from the same company for the same typo | |
| 14:30:01 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Split flavors docs into admin and user guides https://review.openstack.org/501342 | |
| 14:31:21 | mriedem | who wants to get these other two stable/pike high severity fixes? https://review.openstack.org/#/q/topic:bug/1715462+branch:stable/pike | |
| 14:31:46 | sdague | looking | |
| 14:32:23 | sdague | mriedem: +2 on both of those | |
| 14:32:28 | mriedem | thanks | |
| 14:32:55 | mriedem | gibi: i think https://review.openstack.org/#/c/500521/ might be better in some kind of 'limitations' doc | |
| 14:33:18 | mriedem | gibi: the ironic and fgpa thing was noted in the spec as an example, but so was shared IP allocation pools in the original resource providers spec back in i think mitaka | |
| 14:33:30 | mriedem | those are just examples, not declarations of what we support today | |
| 14:34:24 | gibi | mriedem: but we don't have limitation doc I assume | |
| 14:35:32 | mriedem | gibi: no, but that's easy to change | |
| 14:35:33 | gibi | mriedem: but I understand that release notes are not the good place either | |
| 14:35:40 | mriedem | gibi: i left some comments in the patch with more detailed thoughts | |
| 14:35:56 | mriedem | i think we could fit something in here https://docs.openstack.org/nova/latest/user/placement.html | |
| 14:36:24 | mriedem | we have overview, deployment, upgrade notes (per release), and REST API microversion history, we could also have a Limitations section | |
| 14:36:51 | gibi | mriedem: I like this idea | |
| 14:36:57 | mriedem | i like that you like this idea | |
| 14:37:34 | gibi | mriedem: I will try type up a patch during my flight tomorrow | |
| 14:38:01 | mriedem | cool, thanks | |
| 14:38:36 | gibi | thanks for the feedback | |
| 14:40:39 | gibi | mriedem: also thanks for the ML reply on my notification question, elod will start working on adding the new interface_attach notification next week | |
| 14:43:04 | mriedem | \o/ | |
| 14:44:05 | gibi | I'm logging off for the weekend. See you in Denver | |
| 14:44:10 | mriedem | have a good one | |
| 14:44:16 | gibi | thanks | |
| 14:58:55 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Move hash ring initialization to init_host() for ironic https://review.openstack.org/502082 | |
| 15:03:54 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Fix AZ related API docs https://review.openstack.org/456655 | |
| 15:11:22 | sdague | stephenfin: I think you dropped a word - https://review.openstack.org/#/c/456655 | |
| 15:11:30 | sdague | after that, +2 | |
| 15:12:16 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Fix AZ related API docs https://review.openstack.org/456655 | |
| 15:12:31 | stephenfin | sdague: Second time I've done that today :( Done | |
| 15:12:46 | sdague | +2 | |
| 15:12:49 | stephenfin | ta | |
| 15:16:44 | mriedem | stephenfin: couple nits inline there | |
| 15:17:17 | mriedem | "Typically, you use availability zones to arrange OpenStack compute hosts into logical groups." | |
| 15:17:27 | mriedem | you (the end user?) doesn't actually arrange the compute hosts | |
| 15:17:31 | mriedem | the admin does that with host aggregates | |
| 15:17:50 | mriedem | and labels them with AZs that the end user (non-admin) can then use for isolation | |
| 15:18:07 | mriedem | but maybe i'm splitting hairs | |
| 15:22:25 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Fix AZ related API docs https://review.openstack.org/456655 | |
| 15:22:45 | stephenfin | mriedem: I agree, and I think I've addressed it | |
| 15:27:04 | mriedem | cool, +2 | |
| 15:28:00 | mikal | sdague: can I get a +W on https://review.openstack.org/#/c/472229/ please? It has two +2's but the +W fell off when we went around a corner too fast. | |
| 15:31:40 | sdague | mikal: done | |
| 15:31:47 | mikal | Ta | |
| 15:32:18 | sdague | mikal: so, is there actually a privsep discussion area somewhere carved out yet for PTG? | |
| 15:36:12 | mriedem | it's in the etherpad | |
| 15:36:22 | mriedem | https://etherpad.openstack.org/p/nova-ptg-queens | |
| 15:36:32 | mriedem | i don't have a specific time slotted for that | |
| 15:36:56 | sdague | mriedem: right, it's on the big list, but not with a timeslot | |
| 15:37:10 | sdague | I was also wondering if it was in the class of things to have mon/tues time somewhere | |
| 15:37:18 | sdague | mostly to get folks more introduced to the idea | |
| 15:37:26 | mriedem | it's not in the mon/tues xp stuff as far as i know | |
| 15:37:30 | mriedem | unless mikal is driving that | |
| 15:37:38 | mriedem | friday is the default timeslot for 'everything else' | |
| 15:44:47 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rework man pages https://review.openstack.org/502105 | |
| 15:50:42 | openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP! doc: Add contents page https://review.openstack.org/498820 | |
| 15:50:42 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Rename cellsv2_layout -> cellsv2-layout https://review.openstack.org/498821 | |
| 15:50:43 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Cleanup of existing index pages https://review.openstack.org/498819 | |
| 15:57:43 | mikal | dansmith: https://etherpad.openstack.org/p/opendev-sf-use-cases | |
| 15:58:20 | mikal | mriedem / sdague: I had expected it to be scheduled in the nova thing, but that doesn't seem to have happened | |
| 15:59:00 | mikal | I kind of feel like we should do our thing this release, and then push it out to others next release | |
| 16:01:02 | mriedem | ok i still need to carve up the placement stuff into slots | |
| 16:01:03 | sdague | mikal: that's fine, it does seem like if there were interested parties on Tues afternoon or something to gather and do some bootstrapping | |
| 16:01:32 | sdague | it feels like one of those things that's enough of a shift some bootstrapping on what's going on, and good patterns, would be useful | |
| 16:02:06 | mriedem | i have moved some of the placement stuff to it's own etherpad https://etherpad.openstack.org/p/nova-ptg-queens-placement | |
| 16:02:20 | mriedem | i think we're going to do the high priority stuff in the 2 hour slot on wed afternoon | |
| 16:02:31 | mriedem | and then bump the other new stuff for placement to thursday afternoon after neutron | |
| 16:02:37 | mriedem | plus whatever else to friday | |
| 16:05:26 | mriedem | mikal: maybe we do privsep at 4 on wednesday | |
| 16:05:37 | mriedem | after everyone is exhausted | |
| 16:06:54 | sdague | less exhausted than friday | |
| 16:07:03 | sdague | we might be able to complete sentences | |