| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-07 | |||
| 11:35:02 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Clarify 'cells-count-resources-to-check-quota-in-api' https://review.openstack.org/471729 | |
| 11:36:40 | openstackgerrit | Merged openstack/nova master: Completely remove mox from unit/network/test_linux_net.py https://review.openstack.org/445504 | |
| 11:40:31 | mriedem | sfinucan: done https://review.openstack.org/#/c/379524/ | |
| 11:42:41 | sfinucan | mriedem: Am I correct in saying that scheduler filters run before the weighers? | |
| 11:43:32 | mriedem | yes | |
| 11:43:42 | mriedem | FilterScheduler._schedule method | |
| 11:51:32 | openstackgerrit | Merged openstack/nova master: Use plain routes list for hosts endpoint instead of stevedore https://review.openstack.org/459231 | |
| 11:53:12 | mriedem | sean-k-mooney: would you agree with this? https://review.openstack.org/#/c/441183/15/nova/network/model.py@87 | |
| 11:53:16 | mriedem | s/would/do/ | |
| 11:59:58 | openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/469581 | |
| 12:11:37 | openstackgerrit | Michael Henkel proposed openstack/nova master: contrail: add vrouter VIF plugin type support https://review.openstack.org/441183 | |
| 12:19:07 | openstackgerrit | Matt Riedemann proposed openstack/nova master: contrail: add vrouter VIF plugin type support https://review.openstack.org/441183 | |
| 12:21:45 | bauzas | mriedem: I tend to think we should be supporting unicode | |
| 12:22:02 | bauzas | mriedem: I'm trying to verify that | |
| 12:22:16 | mriedem | bauzas: for the db connection url? otherwise that's a very random statement | |
| 12:22:21 | bauzas | yup | |
| 12:22:28 | bauzas | like " | |
| 12:22:30 | bauzas | oops | |
| 12:22:31 | bauzas | like é | |
| 12:22:45 | bauzas | I'm just verifying this | |
| 12:22:45 | mriedem | do people use unicode username/password/db names? | |
| 12:22:51 | bauzas | mriedem: in France, yes | |
| 12:23:02 | mriedem | oh france | |
| 12:23:41 | openstackgerrit | Merged openstack/nova master: Add InstanceGroupList.get_counts() https://review.openstack.org/457857 | |
| 12:23:42 | bauzas | like logins with "hervé" | |
| 12:24:19 | openstackgerrit | Merged openstack/nova master: Fix default_availability_zone docs https://review.openstack.org/438887 | |
| 12:25:07 | openstackgerrit | Merged openstack/nova master: Fix live migration devstack hook for multicell environment https://review.openstack.org/458537 | |
| 12:25:50 | openstackgerrit | Merged openstack/nova master: Move notifications/objects/test_base.py https://review.openstack.org/471425 | |
| 12:27:09 | mriedem | johnthetubaguy: sfinucan: bauzas: melwitt: am i crazy or did this move an option to another group and it's not tested? https://review.openstack.org/438887 | |
| 12:27:13 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add PCIWeigher https://review.openstack.org/379524 | |
| 12:27:14 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Prefer non-PCI host nodes for non-PCI instances https://review.openstack.org/379625 | |
| 12:27:51 | bauzas | mriedem: nope, it's the global | |
| 12:27:58 | bauzas | I verified it | |
| 12:28:02 | bauzas | see my comment | |
| 12:28:30 | mriedem | oh i see, nvm | |
| 12:28:33 | mriedem | confusing | |
| 12:28:37 | mriedem | compute_group vs compute_group_opts | |
| 12:30:02 | sfinucan | bauzas, mriedem: were ye so inclined, I did modify that again here https://review.openstack.org/#/c/462469/ | |
| 12:36:46 | mriedem | sfinucan: what happens with the PCIWeigher if you don't have NUMATopologyFilter enabled? | |
| 12:36:52 | mriedem | which isn't enabled by default, | |
| 12:36:59 | mriedem | but all weighers are loaded and run automatically i believe | |
| 12:37:34 | bauzas | holy fuck, how can I pdb nova-manage commands ? | |
| 12:37:46 | bauzas | looks like my raw_input is crapped | |
| 12:38:17 | bauzas | I just wanna know which kind of string we're getting from the conf object | |
| 12:38:32 | bauzas | in a py2 world | |
| 12:38:42 | bauzas | unless someone can answer this ^ | |
| 12:39:58 | mriedem | raise Exception | |
| 12:40:06 | mriedem | debug like a pro | |
| 12:41:31 | sfinucan | https://sd.keepcalm-o-matic.co.uk/i/keep-calm-and-printf-debugging.png | |
| 12:42:34 | bauzas | please no | |
| 12:43:29 | artom | If that fails because stdout gets lost somwhere, open a temp file and write to it | |
| 12:44:12 | bauzas | mriedem: do you have a pattern ? I can raise some exception, but that's not helping me | |
| 12:44:41 | artom | Well damn, looks like my idea is actually useful :D | |
| 12:44:55 | sfinucan | mriedem: RE: what happens with the PCIWeigher if you don't have NUMATopologyFilter enabled? | |
| 12:45:47 | sfinucan | mriedem: It should work the exact same. The PCI-wanting instances will fail to schedule (because that filter is disabled) and the PCI-hating instances will get the hosts with the least free PCI devices | |
| 12:47:01 | sfinucan | I'm not sure if you'd need to note that the NUMATopologyFilter must be enabled because you'd never even get to the weighting stage without it | |
| 12:47:38 | sfinucan | ...so it's a bit like noting that you must have OpenStack deployed :) | |
| 12:51:19 | bauzas | nvm, found where we unset the output stream | |
| 12:54:10 | sean-k-mooney | mriedem: regarding https://review.openstack.org/#/c/441183/15/nova/network/model.py@87 i dont have a problem with the current name given that it is already used in there nuetron plugin. in future it would be good to consider amending as you suggested but for pike i think its fine as is. | |
| 12:54:30 | sean-k-mooney | mriedem: ill review the latest version of the patch fully now | |
| 12:58:39 | sean-k-mooney | mriedem: yes looking at https://review.openstack.org/#/c/441183/17 i think its in a good state now | |
| 13:00:32 | alex_xu | nova api meeting is running at #openstack-meeting-4 | |
| 13:03:41 | mriedem | sfinucan: fair enough | |
| 13:03:46 | alex_xu | sdague: mriedem the spec for quota-classes bug fix is waiting for feedkback https://review.openstack.org/468619 | |
| 13:04:48 | mriedem | sfinucan: ok so i think if we can fix the typo in the config option and link to https://docs.openstack.org/developer/nova/filter_scheduler.html i'll be ok | |
| 13:05:12 | mriedem | alex_xu: ok | |
| 13:05:44 | stvnoyes | mriedem: I'm working on not using bdm to store the old attachment_id. I think I can add this info to migrate_data.old_attachments as a dict ({bdm_id: old_attachment_id} during pre-migrate, and I don't think it ever needs to be persisted (it would not be in Migrate object). Do you know if there's a reason it needs to be persisted in the db? or will this be ok? | |
| 13:07:53 | mriedem | stvnoyes: stashing it in the MigrateData object is probably good enough if that works for us | |
| 13:07:53 | alex_xu | dansmith: mriedem can we change the field type like this https://review.openstack.org/#/c/430608/24/nova/objects/migration.py@56, I remember the review guide said we can't change the field type | |
| 13:09:45 | stvnoyes | ok cool... what's the reason for persisting that Migration object in the db. Some kind of migration recovery if the control node goes down? that seems challenging. | |
| 13:10:09 | mriedem | stvnoyes: for one thing to list active migrations out of the REST API | |
| 13:10:40 | stvnoyes | mriedem: ah ok that makes sense. | |
| 13:10:47 | mriedem | alex_xu: we've done it before https://github.com/openstack/nova/commit/c37af848ceb7bd0509a8839fb6bc89af8d3a3c5b | |
| 13:11:45 | alex_xu | mriedem: ah..I see now | |
| 13:12:00 | mriedem | stvnoyes: but it's also used for things like resize where there are multiple operations, i.e. (1) and then (2) confirm/reject resize | |
| 13:12:06 | mriedem | (1) resize | |
| 13:12:38 | stvnoyes | got it, thx | |
| 13:12:51 | mriedem | stvnoyes: LiveMigrateData is the right place for temporary data that's only needed during a live migration, | |
| 13:13:05 | mriedem | so if we don't to ever get that information after the live migration, we should be ok | |
| 13:13:15 | mriedem | *don't need | |
| 13:13:51 | stvnoyes | it should be ok for this case, the old attach ids are not relevant after the migration | |
| 13:19:15 | dansmith | alex_xu: if it's entirely compatible with what we have, then we can | |
| 13:19:28 | dansmith | alex_xu: not sure how I feel about that status enum change, but I'll take a look | |
| 13:20:27 | alex_xu | dansmith: thanks | |
| 13:20:33 | mriedem | alex_xu: dansmith: at some point i should go over those again too, because in an earlier revision on that series i reviewed i was worried about the migration status enums being biased toward being able to abort a cold migration | |
| 13:20:47 | mriedem | it all seemed very tightly coupled to the api | |
| 13:21:27 | mriedem | https://review.openstack.org/#/c/430608/18/nova/objects/fields.py@1055 | |
| 13:22:38 | openstackgerrit | Merged openstack/nova master: Remove cells topic configuration option https://review.openstack.org/463218 | |
| 13:23:05 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add PCIWeigher https://review.openstack.org/379524 | |
| 13:23:08 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Prefer non-PCI host nodes for non-PCI instances https://review.openstack.org/379625 | |
| 13:23:18 | mriedem | given we have 20 different migration statuses and 4 different types of migration, getting this all right is a mess | |
| 13:23:38 | alex_xu | mriedem: I remember we said the reverting and confirming didn't have actualy data copy, and it can't be abort, so we didn't count those status as in-progress | |
| 13:23:40 | mriedem | plus when it's baked into the api it's going to be even more of a mess if we screw it up, because contracts | |
| 13:24:04 | mriedem | alex_xu: i think what's considered an "in-progress" migration shouldn't be coupled to what you can or can't abort | |
| 13:24:11 | mriedem | that's a higher-level whitelist in the API imo | |
| 13:24:27 | mriedem | so there might be 10 different in-progress states for a resize, but you can only abort 5 of those | |
| 13:24:30 | alex_xu | mriedem: oh, i probably get your point | |
| 13:24:47 | mriedem | the object layer should report the real in-progress states, the api should control which you can operate on | |
| 13:25:13 | openstackgerrit | Matthew Edmonds proposed openstack/nova master: [WIP] Deactivate powervm compute until ready https://review.openstack.org/471773 | |
| 13:25:58 | alex_xu | mriedem: but we list in-progress migrations from the object layer, it still couple together? | |