| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-13 | |||
| 20:38:08 | jaypipes | efried: we can't programmatically determine resource ownership and policy for the various virt drivers and potential external agents ahead of time. all we can do is docstring the heck out of update_provider_tree() and explain the ramifications of calling ProviderTree.remove(). | |
| 20:38:12 | efried | But yeah. That's my point. Whoever writes & reviews that test, and the code it operates on, has to know which bits are managed by virt and which by external agents. | |
| 20:38:38 | efried | jaypipes: ++ I need to add words to the docstring for this. | |
| 20:39:00 | efried | I think there are words there, but they don't really talk about the implications, just about the effects. | |
| 20:39:50 | jaypipes | efried: for instance, the PowerVM virt driver may just as well WANT to call ProviderTree.remove() on child nodes if the PowerVM virt driver is fully managing things like NICs and disk subsystems, etc.. | |
| 20:39:56 | jaypipes | just no way to know ahead of time. | |
| 20:40:02 | efried | exactly so. | |
| 20:41:15 | jaypipes | efried: and, furthermore, the virt driver's update_provider_tree() method is pretty much exactly what I'd like us to propose to external agents and systems like Cinder/Neutron to use as their "interface" into the resource provider modeling of Placement's API. | |
| 20:41:38 | efried | ++ | |
| 20:42:07 | openstackgerrit | Jeremy Stanley proposed openstack/nova master: Include PCRE headers in bindep manifest https://review.openstack.org/544071 | |
| 20:42:58 | fungi | just ran across that ^ due to https://ask.openstack.org/question/112467 | |
| 20:43:12 | jaypipes | edleafe: unless they're being blocked as spam, I don't think the email notifications on your blog comments are working :( | |
| 20:50:09 | edleafe | jaypipes: arrrrggghh | |
| 20:50:20 | edleafe | recent update must have changed the settings | |
| 20:50:34 | edleafe | lemme investigate | |
| 20:53:56 | jaypipes | sean-k-mooney: so... | |
| 20:54:14 | jaypipes | sean-k-mooney: are you or are you not in support of this spec? https://review.openstack.org/#/c/533077/3/specs/rocky/approved/libvirt-qemu-numa-support.rst | |
| 20:56:47 | openstackgerrit | melissaml proposed openstack/nova-specs master: Replace Chinese quotes to English quotes https://review.openstack.org/544076 | |
| 20:57:48 | openstackgerrit | Merged openstack/python-novaclient stable/ocata: Zuul: Remove project name https://review.openstack.org/544018 | |
| 20:58:35 | mriedem | fungi: https://review.openstack.org/#/c/542340/ | |
| 21:00:39 | fungi | mriedem: excellent! i'll abandon mine. thanks | |
| 21:02:10 | openstackgerrit | Nicolas Bock proposed openstack/nova master: install-guide: Make formatting of console consistent https://review.openstack.org/544081 | |
| 21:03:58 | dansmith | efried: maybe you can answer this: https://review.openstack.org/#/c/540447/3 | |
| 21:04:00 | dansmith | or jaypipes ^ | |
| 21:04:28 | efried | on it | |
| 21:05:38 | edleafe | jaypipes: should be working now. LMK if you don't get emails for future comments | |
| 21:06:04 | efried | dansmith: Done. | |
| 21:08:18 | dansmith | efried: replied, thanks | |
| 21:09:07 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: Update Queens NRP spec to reflect reality https://review.openstack.org/544086 | |
| 21:09:20 | efried | mriedem, jaypipes, dansmith: There's that edit we talked about yesterday ^ | |
| 21:10:29 | efried | edleafe: ^ | |
| 21:12:26 | jaypipes | efried: cool, thanks | |
| 21:13:20 | efried | jaypipes: In case you didn't read that backscroll (link in the commit message of ^) the plan is to propose a more or less new Rocky spec to talk about NRP-in-GET-alloc-cands stuff, rather than "reproposing" the NRP spec. | |
| 21:14:12 | mriedem | i would recommend not reading the backscroll... | |
| 21:14:47 | efried | Basically, the Q one lays the NRP groundwork, and the stuff coming off of that spiders in multiple directions - update_provider_tree, NRP-in-alloc-cands, wtf we're going to do about NUMA, etc. - each of which ought to have its own spec. | |
| 21:14:54 | mriedem | but if you do, your reward is some nice simpsons clip | |
| 21:16:33 | jaypipes | efried: ack | |
| 21:17:27 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add admin guide doc on volume multiattach support https://review.openstack.org/544090 | |
| 21:17:30 | openstackgerrit | Merged openstack/nova-specs master: Add generation support in aggregate association https://review.openstack.org/540447 | |
| 21:19:48 | edleafe | efried: will review shortly | |
| 21:20:24 | efried | d'oh, I forgot to tag the commit message with the bp. Editing... | |
| 21:20:51 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: Update Queens NRP spec to reflect reality https://review.openstack.org/544086 | |
| 21:20:53 | efried | done ^ | |
| 21:21:57 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: Clarify old microversion behavior https://review.openstack.org/544092 | |
| 21:22:02 | efried | dansmith: There's that followup ^ | |
| 21:24:06 | dansmith | efried: I sent it to its maker, thanks | |
| 21:29:50 | openstackgerrit | Nicolas Bock proposed openstack/nova master: install-guide: Wrap long console command https://review.openstack.org/544093 | |
| 21:30:39 | openstackgerrit | Merged openstack/nova master: doc: merge numa.rst to cpu-topologies.rst https://review.openstack.org/540585 | |
| 21:38:19 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Update Queens NRP spec to reflect reality https://review.openstack.org/544086 | |
| 21:39:52 | mriedem | efried: lgtm, thanks | |
| 21:42:43 | openstackgerrit | Merged openstack/nova master: install-guide: Make formatting of console consistent https://review.openstack.org/544081 | |
| 21:43:08 | openstackgerrit | Merged openstack/nova-specs master: Clarify old microversion behavior https://review.openstack.org/544092 | |
| 21:45:46 | mriedem | dansmith: easy peasy https://review.openstack.org/#/c/543571/ | |
| 21:45:50 | mriedem | likely RC2 fodder | |
| 21:47:26 | dansmith | whoopsie | |
| 21:54:00 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Bindep does not catch missing libpcre3-dev on Ubuntu https://review.openstack.org/544108 | |
| 21:54:05 | mriedem | esberglu: want to post a backport to queens for https://review.openstack.org/#/c/543571/ ? | |
| 21:54:37 | openstackgerrit | Eric Berglund proposed openstack/nova stable/queens: Use correct arguments in task inits https://review.openstack.org/544109 | |
| 21:54:54 | mriedem | with the -x option on git cherry-pick please | |
| 22:03:22 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: Update Provider Tree https://review.openstack.org/540111 | |
| 22:03:37 | efried | dansmith, mriedem, jaypipes, edmondsw, edleafe: ^ | |
| 22:05:39 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Use correct arguments in task inits https://review.openstack.org/544109 | |
| 22:05:47 | mriedem | esberglu: like that ^ | |
| 22:06:20 | esberglu | mriedem: Was one command slower :) | |
| 22:06:36 | esberglu | I just used the gerrit UI, noted for the future | |
| 22:06:41 | mriedem | cherry picking from gerrit UI doesn't give you the -x thing until the change is merged | |
| 22:06:48 | mriedem | so you have to resort to the CLI | |
| 22:16:29 | melwitt | mriedem: about https://review.openstack.org/#/c/539003, we (and everyone) need that, else we have the same exposure from CVE-2017-16239 if a "bad rebuild" is attempted twice | |
| 22:22:16 | melwitt | mriedem: amoralej|off was asking in #rdo earlier today, which novaclient version should be used in u-c? he said he saw this change with a -2 on it and wanted to know if 9.1.1 is going to be the official version for queens | |
| 22:22:32 | melwitt | https://review.openstack.org/#/c/538070 | |
| 22:40:52 | mlavalle | melwitt: hi. I am starting to put together the PTG agenda for the Neutron team. We have some topics that I think we should discuss with the Nova team. Can we have a cross team session? Maybe Thursday afternoon? | |
| 22:43:04 | melwitt | mlavalle: yes, we have some topics we need to discuss with you all too (L145 in https://etherpad.openstack.org/p/nova-ptg-rocky). I think thursday afternoon should probably work | |
| 22:46:07 | mlavalle | melwitt: can I add to that section of your etherpad the items I have in my radar screen? I'll ping you again tomorrow afternoon, once you've had a chance to look at them | |
| 22:46:33 | melwitt | mlavalle: sure, please do | |
| 22:46:39 | mlavalle | melwitt: :-) | |
| 22:55:44 | mriedem | melwitt: yes on novaclient because 10.0.0 breaks horizon in queens | |
| 22:55:48 | mriedem | and openstackclient | |
| 22:56:15 | melwitt | okay, cool. thanks | |
| 22:57:59 | mriedem | melwitt: ok for https://review.openstack.org/#/q/I3399a66fe9b1297cd6b0dca440145393ceaef41f - half the stable core team is from red hat so you'll have to raise the alarm there | |
| 22:58:18 | mriedem | or just ping mtreinish | |
| 22:58:20 | mriedem | he loves stable | |
| 22:59:14 | melwitt | okay, will do. thanks for the heads up | |
| 23:03:18 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Update Queens NRP spec to reflect reality https://review.openstack.org/544086 | |
| 23:08:10 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Check for leaked server resource allocations in post_test_hook https://review.openstack.org/538510 | |
| 23:08:26 | mtreinish | mriedem: Need me to look at some backports? | |
| 23:09:55 | mriedem | https://review.openstack.org/#/q/I3399a66fe9b1297cd6b0dca440145393ceaef41f | |
| 23:12:17 | mtreinish | mriedem: when does ocata enter phase 3? | |
| 23:12:56 | mriedem | LTS baby | |
| 23:13:25 | mriedem | mtreinish: feb 26 i guess? https://releases.openstack.org/ | |
| 23:13:28 | mriedem | this is related to a cve | |
| 23:17:43 | openstackgerrit | Merged openstack/nova master: Use correct arguments in task inits https://review.openstack.org/543571 | |
| 23:22:41 | openstackgerrit | Nicolas Bock proposed openstack/nova master: install-guide: Wrap long console command https://review.openstack.org/544093 | |
| 23:25:50 | openstackgerrit | Merged openstack/nova master: Ensure resource classes correctly https://review.openstack.org/539738 | |
| 23:30:57 | openstackgerrit | Nicolas Bock proposed openstack/nova stable/pike: Fix SUSE Install Guide: Placement port https://review.openstack.org/544131 | |
| 23:41:58 | openstackgerrit | Eric Fried proposed openstack/nova master: Avoid inventory DELETE API (no conflict detection) https://review.openstack.org/539712 | |
| 23:41:59 | efried | jaypipes: Yet another manual rebase +2 please ^ | |
| #openstack-nova - 2018-02-14 | |||
| 00:09:23 | openstackgerrit | Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: Snapshot https://review.openstack.org/543023 | |
| 00:59:32 | gmann_ | mriedem_afk: gibi stephenfin as we faced the nova jobs irrelevant_files and branch restriction issues and have to fix on project-config side, i am thinking to move pipeline list jobs on project side. | |
| 01:00:18 | gmann_ | mriedem_afk: gibi stephenfin this will give nova a full control on those bits. i started a generic discussion on ML- http://lists.openstack.org/pipermail/openstack-dev/2018-February/127320.html | |
| 01:00:58 | gmann_ | please check and if that looks ok/not ok from nova perspective. | |