| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-14 | |||
| 10:21:21 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test allocation candidates: multiple aggregates https://review.openstack.org/518633 | |
| 10:21:22 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add aggregates check in allocation candidates https://review.openstack.org/522407 | |
| 10:21:22 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Fix accumulated nits in refactor series https://review.openstack.org/521189 | |
| 10:21:23 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: placement: func tests for multiple shared RPs https://review.openstack.org/498737 | |
| 10:21:23 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Fix missing rps in allocation candidates https://review.openstack.org/522409 | |
| 10:23:41 | gibi | jaypipes: ^^ rebased the series as master already has the fix for the frequently failing functional test that prevented these to be merged | |
| 10:26:48 | openstackgerrit | rahul bardia proposed openstack/python-novaclient master: nova limits ERROR (Exception): Field names must be unique This happens when 'Others' field is occuring more than once which is not handled in code causing multiple columns having same name. Which throws above exception and does not pretty prints absolute https://review.openstack.org/527922 | |
| 11:11:50 | kashyap | stephenfin: Hi there, when you get a moment, you know why is Zuul complaning here - https://review.openstack.org/#/c/527691/ | |
| 11:21:35 | stephenfin | kashyap: Looking | |
| 11:22:45 | stephenfin | kashyap: That job's been failing a lot lately, but I don't know why. It's unrelated to this, in any case | |
| 11:23:03 | stephenfin | We should probably investigate that, actually | |
| 11:24:38 | stephenfin | bauzas: Ta for the review :) | |
| 11:25:32 | kashyap | stephenfin: Yeah, noted; right now I need to run an errand. | |
| 11:34:45 | openstackgerrit | Merged openstack/nova stable/ocata: Add regression test for rebuilding a volume-backed server https://review.openstack.org/525664 | |
| 11:41:42 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407 https://review.openstack.org/526095 | |
| 11:41:43 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild https://review.openstack.org/525242 | |
| 11:41:43 | openstack | bug 1735407 in OpenStack Compute (nova) "[Nova] Evacuation doesn't respect anti-affinity rules" [Medium,In progress] https://launchpad.net/bugs/1735407 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 11:51:35 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297 | |
| 11:51:36 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send soft_delete from context manager https://review.openstack.org/476459 | |
| 12:01:57 | gibi | kashyap, stephenfin: the test_live_migration_actions functional test failure has already been fixed on master so you just have to rebase | |
| 12:02:22 | stephenfin | gibi: Aha. What patch fixed it, out of curiosity? | |
| 12:02:23 | gibi | kashyap, stephenfin: this was the fix https://review.openstack.org/#/c/527440/ | |
| 12:02:32 | stephenfin | jinx :)_ | |
| 12:02:54 | stephenfin | Oh, nice | |
| 12:03:01 | gibi | stephenfin: there is one more fix in the pipe that fixes another failure mode of that test here https://review.openstack.org/#/c/527444/ | |
| 12:04:12 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Extract instance allocation removal code https://review.openstack.org/513041 | |
| 12:35:32 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Enable limiting GET /allocation_candidates https://review.openstack.org/513526 | |
| 12:40:44 | openstackgerrit | Chason Chan proposed openstack/nova master: Fix the bug report link of API Guide https://review.openstack.org/527660 | |
| 12:43:31 | openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Modify signature of _filter_non_requested_pfs https://review.openstack.org/527473 | |
| 12:43:31 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add PCI NUMA policies https://review.openstack.org/527472 | |
| 12:44:30 | efried_cya_jan | stephenfin gibi Thanks for looking after those patches! | |
| 12:45:26 | cdent | efried_cya_jan: since you're here (naughty!) I should report that I've not had a chance to do anything with your stuff: I've got a nasty cold so mostly done nothing for the past couple of days. But when I surface, I will. | |
| 12:46:17 | efried_cya_jan | cdent I somehow caught your cold, which is a sucky way to start a vacation. It looks like stephenfin has been addressing review comments and he & gibi have been rebasing stuff, so I think you're in the clear... for now :) | |
| 12:46:35 | cdent | internet germs are the worst | |
| 12:46:49 | gibi | efried_cya_jan: o/ | |
| 12:48:19 | openstackgerrit | Raghad Qutteneh proposed openstack/nova master: Allow force-delete to delete instance in task_state block_device_mapping https://review.openstack.org/527951 | |
| 12:53:13 | gibi | efried_cya_jan: if you are here I have a question. Do I understand correctly that the GET allocation_candidate API does not handle nested RPs yet? | |
| 12:53:57 | efried_cya_jan | gibi That's correct. That's a major piece we need before we can slot in the granular resource requests work. | |
| 12:54:16 | gibi | efried_cya_jan: OK, cool | |
| 12:54:23 | efried_cya_jan | gibi: jaypipes is on the hook for that. | |
| 12:55:13 | gibi | OK | |
| 13:48:55 | kashyap | gibi: Thanks for the (non-null) pointer :-) | |
| 13:52:27 | mriedem | oh crap today is thursday huh | |
| 13:52:32 | mriedem | gibi: can you run the meeting? | |
| 13:53:12 | gibi | mriedem: sure | |
| 13:54:04 | mriedem | thanks. i'll update the agenda quick. | |
| 13:54:09 | gibi | mriedem: thanks, that helps | |
| 13:54:17 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Enable limiting GET /allocation_candidates https://review.openstack.org/513526 | |
| 13:57:46 | mriedem | done https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 13:58:09 | gibi | cool | |
| 14:00:17 | gibi | nova meeting is about to start on @openstack-meeting | |
| 14:00:29 | cdent | mriedem: just wanted to say: thanks for trying to keep it real in the cycle length thread. | |
| 14:00:43 | mriedem | np | |
| 14:01:05 | cdent | I'm trying to gather my thoughts, but not having much success | |
| 14:02:16 | mriedem | i think i will have to move on today from that thread | |
| 14:02:42 | cdent | that would be a bummer, you are uniquely positioned to, uh, keep it real | |
| 14:04:12 | mriedem | i don't have the energy to fight that battle, and it sounds like the majority of people (ops, users?) seem to like the idea. i think it will turn out to be more than what they bargained for (some things they'll lose, some things they won't get out of it) | |
| 14:05:44 | mriedem | we'll do what we need to do for nova, if that means intermediate releases just to give us a deadline to work toward even if no one ever consumes it | |
| 14:06:56 | cdent | i definitely think that if it happens hard interediate deadlines will be _very_ important | |
| 14:07:40 | edleafe | so if you don't meet a deadline, would that mean waiting a year? Or what? | |
| 14:09:58 | edleafe | just wondering how "dead" such a deadline would be | |
| 14:10:33 | cdent | as you know, I think all deadlines are made up and thus fungible, but they are useful encouragement tools? | |
| 14:11:14 | edleafe | IMO, deadlines work as sticks, not carrots. | |
| 14:12:02 | cdent | maybe call them boundaries? | |
| 14:13:19 | mriedem | jaypipes: i put this up for that dbdeadlock issue https://review.openstack.org/#/c/527836/ - not sure that's best, but it's a thing | |
| 14:13:40 | mriedem | edleafe: i think we'd do intermediate releases so we could have more than 1 freeze in a year | |
| 14:14:19 | mriedem | so either 1 or 2 intermediate releases (6 month or 4 month between them), and do our normal spec review, spec freeze, code churn, feature freeze, stability minute, then open up for new specs again | |
| 14:14:31 | mriedem | i don't think we can freeze out specs for an entire year, | |
| 14:14:45 | mriedem | and i don't think we can leave spec reviews open for an entire year and focus on actually getting things done | |
| 14:14:52 | edleafe | mriedem: so would they be Rocky I, Rocky II, ... | |
| 14:15:11 | mriedem | rocky IV is where we fight the mirantis guys | |
| 14:15:18 | mriedem | us guys, | |
| 14:15:19 | mriedem | and yous guys | |
| 14:16:37 | mriedem | having said that, i don't know what we'd do for upgrade testing with intermediate releases | |
| 14:16:53 | mriedem | do you do queens->rocky.1, queens->rocky.2, or queens->rocky.1->rocky.2? | |
| 14:17:09 | mriedem | today it would be the former because of how the CI system works on branches | |
| 14:17:31 | mriedem | i assume we would use minor version bumps for intermediate releases | |
| 14:17:34 | edleafe | yuck | |
| 14:17:46 | cdent | blargh | |
| 14:17:49 | mriedem | so rocky 1 would be 17.1.0 | |
| 14:17:53 | mriedem | if rocky GA was 18.0.0 | |
| 14:17:56 | mriedem | but, | |
| 14:18:08 | mriedem | that gets tricky if we actually need to backport something to stable/pike which would normally be a minor update | |
| 14:18:12 | mriedem | like a schema migration | |
| 14:19:09 | mriedem | likely a question i'll need to ask about in "the thread" | |
| 14:20:18 | stephenfin | efried_cya_jan: See your comments now. I'll address them shortly | |
| 14:20:30 | stephenfin | Fwiw, I'll keep trying to do the minimum possible amount to keep everyone happy and get it in (ignored most of my own comments/preferences for this reason) | |
| 14:20:57 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Refactor encryptor attach and detach calls https://review.openstack.org/460243 | |
| 14:20:58 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Use QEMU's native LUKS support https://review.openstack.org/523958 | |
| 14:20:58 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Introduce disk encryption config classes https://review.openstack.org/464008 | |
| 14:24:22 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Use QEMU's native LUKS support https://review.openstack.org/523958 | |
| 14:26:08 | openstackgerrit | Merged openstack/nova master: [placement] add name to resource provider create error https://review.openstack.org/526710 | |
| 14:26:14 | openstackgerrit | Merged openstack/nova master: [placement] Add 'Location' parameters in API ref https://review.openstack.org/521541 | |
| 14:26:20 | openstackgerrit | Merged openstack/nova master: doc: link in some Sydney summit content https://review.openstack.org/526587 | |
| 14:27:09 | dansmith | bauzas: need you to look at https://review.openstack.org/#/c/527799/2 | |
| 14:30:01 | takashin | mriedem: The compute API microversion 2.57 was added in "Deprecate file injection" https://review.openstack.org/#/c/522027/ . | |
| 14:30:06 | takashin | mriedem: But I cannot find a patch for microversion 2.57 in python-novaclient. Would you submit the patch? | |
| 14:31:11 | takashin | mriedem: Are you around? | |
| 14:33:22 | jaypipes | mriedem: ok, looking at it now. | |
| 14:37:04 | mriedem | takashin: i plan on working on that novaclient patch today | |