| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-14 | |||
| 21:51:52 | dansmith | efried: required https://hangouts.google.com/call/vebCoylDfu_0tvlCDPwmAAEE, edleafe mriedem jaypipes optional | |
| 21:52:04 | mriedem | pants required? | |
| 21:52:04 | mnaser | i guess i'll look into making it create bdms in cell0 | |
| 21:52:11 | dansmith | mriedem: yes | |
| 21:52:28 | mriedem | mnaser: that or we unreserve the volumes for the bdms right there in conductor... | |
| 21:52:35 | mriedem | so that the api doesn't have to delete them locally | |
| 21:52:59 | mriedem | but then we get the "how many different nova services does it take to detach a volume" problem | |
| 21:53:01 | mnaser | that means that the volume will become available right away | |
| 21:53:06 | mnaser | that | |
| 21:53:06 | mnaser | but yeah | |
| 21:56:03 | mriedem | mnaser: melwitt: left notes on https://review.openstack.org/#/c/340614/ - thanks for the testing | |
| 22:01:21 | openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.61 - Add trusted_image_certificates https://review.openstack.org/500396 | |
| 22:01:56 | mnaser | mriedem: melwitt i can do both (though storing bdms seems easier).. i'll leave the decision up to you | |
| 22:11:48 | mriedem | before all of this we'd have the bdms created in the api along with the instance (before cellsv2 happened), so might as well just go with that for starters and see how it goes | |
| 22:12:11 | mriedem | not api, the 'nova' db, but the nova-api service would create them in the nova db | |
| 22:13:22 | melwitt | mriedem: sure, I can take the meeting baton tomorrow. let me know if I need to update the agenda or if you already did | |
| 22:13:32 | mriedem | i have not updated the agenda | |
| 22:13:46 | melwitt | k. I shall agenda then | |
| 22:15:22 | melwitt | and that sucks about the bdms thing. | |
| 22:15:39 | mriedem | right so we'll likely need to stagger these patches for sanity | |
| 22:15:54 | openstackgerrit | Dan Smith proposed openstack/nova-specs master: Add placement-req-filter spec https://review.openstack.org/544585 | |
| 22:15:56 | dansmith | efried: ^ | |
| 22:16:00 | mriedem | regression test, do something about bdms in conductor on scheduling failure, then the local delete api patch | |
| 22:16:03 | efried | dansmith: ack | |
| 22:16:04 | melwitt | yeah, I was about to say ... should mnaser take that issue as a separate patch? | |
| 22:16:10 | mriedem | melwitt: i think so | |
| 22:16:19 | mriedem | we're going to want to backport all of this to ocata too | |
| 22:16:26 | melwitt | good times | |
| 22:16:40 | mriedem | this is the most fun i've had all week | |
| 22:16:47 | melwitt | SAME | |
| 22:18:26 | mnaser | I have 3 patches right now, fixing fixtures, functional tests and melwitt final patch on top. I’ll add the bdm creation in the stack. | |
| 22:18:34 | mnaser | I’ll make sure to keep them | |
| 22:18:40 | mnaser | Under the same topic | |
| 22:18:48 | mnaser | (But first supper) | |
| 22:18:48 | mriedem | mnaser: thanks | |
| 22:20:48 | mlavalle | melwitt: I added some topics to the Nova PTG etherpad. With those topics, I figured 2 hours. Right now I have the Neutron / Nova session from 3 to 5 on Thursday. When you have a chance let me know what you think. No need to rush | |
| 22:21:17 | mlavalle | fro 3 to 5 in the Neutron etherpad^^^^ | |
| 22:21:43 | mlavalle | but I am keeping that afternoon open until you and I firm up the times | |
| 22:23:48 | melwitt | mlavalle: k, probably will be fine. just have to sync up with the ironic team. I'll let you know | |
| 22:31:11 | mlavalle | melwitt: :-) | |
| 22:40:26 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Scheduler multiple workers support https://review.openstack.org/159382 | |
| 22:53:20 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Check for leaked server resource allocations in post_test_hook https://review.openstack.org/538510 | |
| 23:04:00 | openstackgerrit | Dan Smith proposed openstack/nova master: Remove deprecated aggregate DB compatibility https://review.openstack.org/544698 | |
| 23:04:01 | openstackgerrit | Dan Smith proposed openstack/nova master: Add aggregates list to Destination object https://review.openstack.org/544729 | |
| 23:04:01 | openstackgerrit | Dan Smith proposed openstack/nova master: Add AggregateList.get_by_metadata_value() query method https://review.openstack.org/544728 | |
| 23:04:02 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Add request filter functionality and first filter https://review.openstack.org/544730 | |
| 23:12:13 | melwitt | mlavalle: I've confirmed that 3-5 on thursday works for nova/neutron. thanks | |
| 23:12:30 | mlavalle | melwitt: Thank you! | |
| 23:26:40 | mnaser | yay adding bdms to cell0 fixed it | |
| 23:35:30 | melwitt | tonyb: this says ocata enters phase 3 support on 2/26 but also says 2/26 is the EOL date https://releases.openstack.org/ does that make sense to you? | |
| 23:39:18 | mriedem | melwitt: do you want/plan on doing a queens retrospective at the ptg? | |
| 23:41:16 | melwitt | mriedem: I'm not opposed to having one but I hadn't thought about it because no one had mentioned it (whereas we have had obvious interest in retro in the past). do you think we should just plan for one? | |
| 23:42:34 | melwitt | I also might have missed it if anyone has expressed interest in having a queens retro | |
| 23:45:57 | mriedem | it hasn't come up, but i assume we should at least make it an option | |
| 23:46:01 | mriedem | if no one has anything, it's a short thing | |
| 23:46:12 | melwitt | yeah, makes sense | |
| 23:46:29 | mnaser | i have 4 patches right, is the following order ok or should i reorder: fix cinder fixtures -> add func regr. tests -> add bdms to cell0 -> patch to remove ports and bdms | |
| 23:46:56 | melwitt | mriedem: maybe we could start off with that on wednesday since any takeaways could apply to later discussions | |
| 23:47:02 | mriedem | https://etherpad.openstack.org/p/nova-queens-retrospective | |
| 23:47:14 | mriedem | mnaser: sounds correct | |
| 23:47:54 | mnaser | ok awesome, i have to fix a few unit tests because of the changes in _bury_in_cell0 which now create bdms and should be ready for review soon | |
| 23:53:10 | openstackgerrit | Murali Annamneni proposed openstack/nova master: Enables MySQL Cluster Support for Nova https://review.openstack.org/446643 | |
| #openstack-nova - 2018-02-15 | |||
| 00:02:15 | mriedem | melwitt: this is what i wanted the retrospective link for http://lists.openstack.org/pipermail/openstack-dev/2018-February/127402.html | |
| 00:02:51 | melwitt | mriedem: a-ha, nice. I was just looking for that email to link in the agenda for tomorrow's meeting | |
| 00:06:32 | mnaser | here goes nothing :> | |
| 00:08:30 | openstackgerrit | Mohammed Naser proposed openstack/nova master: Clean up ports and volumes when deleting ERROR instance https://review.openstack.org/340614 | |
| 00:08:30 | openstackgerrit | Mohammed Naser proposed openstack/nova master: Drop extra loop which modifies Cinder volume status https://review.openstack.org/539758 | |
| 00:08:31 | openstackgerrit | Mohammed Naser proposed openstack/nova master: Store block device mappings in cell0 https://review.openstack.org/544748 | |
| 00:08:31 | openstackgerrit | Mohammed Naser proposed openstack/nova master: Add functional tests to ensure BDM removal on delete https://review.openstack.org/544747 | |
| 00:14:16 | mnaser | i guess at some point we'll have to squash the func. tests and clean up commit to get them to merge | |
| 00:15:06 | melwitt | mriedem: just updated the agenda for tomorrow's meeting https://wiki.openstack.org/wiki/Meetings/Nova | |
| 00:15:53 | mriedem | mnaser: what we usually do, | |
| 00:16:01 | mriedem | is land the regression test that asserts the failre, | |
| 00:16:03 | melwitt | mnaser: why? the way we do that is the func test is introduced with the after-bug-fix asserts commented out with a note (and I'll let mriedem finish) | |
| 00:16:11 | mriedem | then the patch that fixes it changes the functional test to show it's passing | |
| 00:17:02 | mnaser | ok i see so then my functional test is wrong | |
| 00:17:14 | mnaser | because it is a failing one (which will pass in the patch above of it) | |
| 00:18:19 | mnaser | so i will switch the best to make it assert the failure and then change the fixing patch as explained | |
| 00:19:53 | melwitt | mnaser: that's nearly there, just have to comment out the failing assert and add TODO(mnaser) above it to explain it should be uncommented in the same patch as the fix, then add an assert for the wrong (but expected) thing with a TODO to remove it in the fix patch | |
| 00:26:17 | openstackgerrit | Mohammed Naser proposed openstack/nova master: Store block device mappings in cell0 https://review.openstack.org/544748 | |
| 00:26:18 | openstackgerrit | Mohammed Naser proposed openstack/nova master: Clean up ports and volumes when deleting ERROR instance https://review.openstack.org/340614 | |
| 00:26:18 | openstackgerrit | Mohammed Naser proposed openstack/nova master: Add functional tests to ensure BDM removal on delete https://review.openstack.org/544747 | |
| 00:26:28 | mnaser | that should do it | |
| 00:27:46 | openstackgerrit | Mohammed Naser proposed openstack/nova master: Drop extra loop which modifies Cinder volume status https://review.openstack.org/539758 | |
| 00:27:47 | openstackgerrit | Mohammed Naser proposed openstack/nova master: Add functional tests to ensure BDM removal on delete https://review.openstack.org/544747 | |
| 00:27:47 | openstackgerrit | Mohammed Naser proposed openstack/nova master: Store block device mappings in cell0 https://review.openstack.org/544748 | |
| 00:27:48 | openstackgerrit | Mohammed Naser proposed openstack/nova master: Clean up ports and volumes when deleting ERROR instance https://review.openstack.org/340614 | |
| 00:36:32 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Enable NUMA Features for Libvirt/QEMU Driver https://review.openstack.org/533077 | |
| 00:39:44 | mriedem | mnaser: melwitt: ok i've gone through the first 3, left some nits but nothing major | |
| 00:40:01 | mriedem | i can get the final big patch in the morning | |
| 00:42:04 | melwitt | thanks | |
| 00:43:15 | mnaser | mriedem: thanks, im addressing them now and rebasing | |
| 00:53:41 | tonyb | melwitt: It is indeed confusing. The "next pahse" shoudl just go strainght to EOL but there is a window in there where it's in Phase III while we do the EOLing ... and then we went and confused ourselves with the Long lived stable branches so I don't know what'll happen | |
| 00:54:14 | tonyb | melwitt: but there's a session at the PTG to discuss it so hopefully shortly after that I can at least make that table look more sane | |
| 00:54:45 | melwitt | tonyb: thanks for the glimpse at what it means :) | |
| 00:55:08 | tonyb | melwitt: ;P I guess you have to care more about that stuff now:) | |
| 00:55:29 | melwitt | nah, total coincidence | |
| 00:55:31 | tonyb | melwitt: anytime I can help you know where I am | |
| 00:55:35 | melwitt | :) thanks | |
| 00:55:39 | tonyb | melwitt: ohh my bad ;P | |