| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-30 | |||
| 15:17:56 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021 | |
| 15:19:35 | bauzas | mriedem: nevermind, I'm sure it works | |
| 15:19:46 | bauzas | because we decrease the limit | |
| 15:20:44 | mriedem | bauzas: we wouldn't get #2 if the marker is #1 and it's ACTIVE | |
| 15:20:54 | mriedem | because we process the cells with the ACTIVE instances after cell0 with the ERROR instance | |
| 15:21:14 | bauzas | mriedem: so we would return #1 and #3 ? | |
| 15:21:18 | bauzas | only | |
| 15:21:19 | bauzas | ? | |
| 15:21:19 | mriedem | yes | |
| 15:21:27 | bauzas | mmm, that's another bug, right? | |
| 15:21:28 | mriedem | which is probably also a regression | |
| 15:21:46 | bauzas | lemme double-think on it | |
| 15:21:54 | mriedem | it's not going to get fixed here | |
| 15:21:58 | bauzas | sure | |
| 15:22:02 | mriedem | we search in build requests first, then cell0, then the cells | |
| 15:22:04 | ericyoung | mriedem: regarding https://review.openstack.org/#/c/407440/ (ScaleIO ephemeral) we have a couple of changes to our CI environment and then it should pass | |
| 15:22:12 | mriedem | paging gets wonky | |
| 15:22:13 | ericyoung | hopefully in the nxt day or two | |
| 15:22:15 | mriedem | and is why we said searchlight | |
| 15:22:23 | mriedem | ericyoung: ok | |
| 15:23:07 | bauzas | mriedem: well, so we should only set the marker to the last instance UUID we have, which can be either in cell0 or even a BuildRequesrt | |
| 15:23:32 | bauzas | something like, try to get all you can do with the same marker, and then only set the marker to the last ID you got | |
| 15:24:05 | bauzas | mriedem: in that case, I'd recommend to modify https://review.openstack.org/#/c/468559/ to tackle all those issues | |
| 15:24:38 | mriedem | bauzas: if your marker is on #1 then #2 wouldn't be in the page anyway | |
| 15:24:57 | mriedem | because #1 is ACTIVE | |
| 15:24:59 | mriedem | which means it's in a cell | |
| 15:25:25 | mriedem | our pages right now are basically scheduling (build requests), error (cell0) and then everything else | |
| 15:25:35 | bauzas | that I know | |
| 15:25:45 | mriedem | we said that was what we'd do at the newton midcycle at intel | |
| 15:26:10 | mriedem | and long-term a global search backend like searchlight is what you'd use for accurate searches | |
| 15:26:14 | bauzas | sure, simple pagination | |
| 15:26:41 | bauzas | but here, I'm considering a simple pagination with multiple backends (ie. not sorting) | |
| 15:27:00 | bauzas | for that, we're not needing searchlight, right? | |
| 15:27:19 | mriedem | what does multiple backends mean? | |
| 15:37:17 | mordred | efried: I started on the keystoneauth patches to implement all the things we've been poking at over the weekend | |
| 15:37:39 | efried | mordred Cool man. Please add me to the review(s). | |
| 15:38:14 | efried | mordred I still need to get caught up on the... "literature" backlog ;-) | |
| 15:38:19 | mordred | efried: https://review.openstack.org/#/q/status:open+project:openstack/keystoneauth+branch:master+topic:version-discovery | |
| 15:38:55 | efried | mordred Dig. | |
| 15:39:34 | mordred | efried: I still don't have list-of-interface done, nor the service-types aliases - but those should be easy enough to do as a follow on | |
| 15:40:43 | efried | mordred Okay. If you get a chance to comment in https://review.openstack.org/458257 as to how that guy should change as a result, please do. | |
| 15:45:10 | openstackgerrit | John Griffith proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285 | |
| 15:47:04 | sfinucan | jaypipes: Are these scripts auto-generated by sqlmigrate? https://review.openstack.org/#/c/469147/1/nova/db/sqlalchemy/migrate_repo/versions/362_add_pci_devices_uuid.py | |
| 15:47:22 | jaypipes | sfinucan: no. hand written. why? | |
| 15:47:26 | sfinucan | sqlalchemy-migrate | |
| 15:47:33 | mordred | efried: done | |
| 15:47:38 | sfinucan | Just checking - reading up on the syntax now | |
| 15:47:39 | efried | mordred Thanks | |
| 15:47:58 | mordred | efried: it's possible we might be able to clean up more things once we start poking | |
| 15:48:35 | efried | mordred It would be neat if we could do away with the nova.utils bit entirely. | |
| 15:48:52 | efried | mordred But I wouldn't want the consumers to have to do more than a couple of lines. | |
| 15:49:09 | efried | mordred E.g. they shouldn't have to do the looping logic. | |
| 15:49:25 | mordred | yah- well, the looping logic will definitely go away | |
| 15:50:00 | mordred | but even if we can just get it down to 3 lines of load_from_conf_options calls, it's probably somewhat worthwhile to have as a function perhaps | |
| 15:58:59 | jaypipes | melwitt, mriedem: k, quotas base patch reviewed. | |
| 16:00:02 | openstackgerrit | John Griffith proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285 | |
| 16:02:27 | sfinucan | jaypipes: That looks good to me, though I wonder if we need a reno for the online (inline?) migration? | |
| 16:02:34 | sfinucan | If not, I can +2 | |
| 16:04:34 | sfinucan | I'll take a look at nested-resource-providers tomorrow, once I've sahid's trusted-vfs thing done | |
| 16:05:08 | mriedem | trusted-vfs? | |
| 16:07:34 | jaypipes | mriedem: https://review.openstack.org/#/c/458514/ | |
| 16:07:58 | clarkb | mriedem: http://logs.openstack.org/03/415103/4/gate/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/4e7ec35/logs/screen-n-cpu.txt.gz?level=ERROR looking at that failure and trying to dig through the libvirt logs to see if there is any explanation for why that happened and not finding much. But am noticing libvirt complains that dmidecode is missing. Any idea if that is required for live | |
| 16:08:00 | clarkb | migration (so that it can check compatibility of hardware maybe?) | |
| 16:08:29 | mriedem | jaypipes: sfinucan: that's not an approved blueprint | |
| 16:14:20 | jaypipes | mriedem: nested resource providers? | |
| 16:14:45 | clarkb | actually nevermind its explicitly failign to exchange identification implying an auth problem? | |
| 16:19:58 | mriedem | jaypipes: no trusted vfs | |
| 16:20:10 | jaypipes | mriedem: understood. | |
| 16:30:19 | mriedem | clarkb: looks like the problem is "Failed to connect to remote libvirt URI qemu+ssh://stack@ubuntu-xenial-2-node-rax-ord-9016742-596845/system: Cannot recv data: ssh_exchange_identification: Connection closed by remote host: Connection reset by peer" | |
| 16:32:07 | openstackgerrit | Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/468494 | |
| 16:32:31 | mriedem | http://logs.openstack.org/03/415103/4/gate/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/4e7ec35/logs/libvirt/libvirtd.txt.gz#_2017-05-27_00_12_18_968 is the error in the libvirtd logs | |
| 16:34:29 | clarkb | mriedem: yup, I think either we have general network issues between the nodes or ssh isn't configured properly | |
| 16:35:29 | clarkb | mriedem: I'll get a e-r query for it pushed shortly then we can query it and see if its cloud related | |
| 16:51:01 | bauzas | jaypipes: thanks for commenting https://review.openstack.org/#/c/469037/2, I'll provide a new version but see my comments in https://review.openstack.org/#/c/469037/2/nova/conductor/tasks/live_migrate.py | |
| 16:51:17 | bauzas | jaypipes: tl;dr: cellsv1 old instances can possibly have no RequestSpec records | |
| 16:51:44 | bauzas | melwitt: mriedem: FYI ^ given you also worked on https://review.openstack.org/#/c/391060/ | |
| 16:53:52 | openstackgerrit | Rafael Folco proposed openstack/nova master: Delete all inventory has its own method DELETE https://review.openstack.org/460147 | |
| 16:57:41 | jaypipes | bauzas: answered. | |
| 16:57:58 | bauzas | roger. | |
| 16:57:58 | jaypipes | bauzas: I think all that needs is an else: block, no? | |
| 16:58:55 | bauzas | jaypipes: uh no, see, I'm not using *self.request_spec* | |
| 16:59:16 | bauzas | jaypipes: I'm rather passing a newly-created request_spec hydrated | |
| 16:59:58 | bauzas | the conditional in L159 outputs a variable named request_spec which is not an attribute | |
| 17:00:03 | jaypipes | bauzas: ahhh | |
| 17:00:10 | bauzas | and I'm using it L175 | |
| 17:00:16 | jaypipes | bauzas: sorry, that was confusing. I missed the self. vs. not-self. thing. | |
| 17:00:22 | bauzas | jaypipes: np | |
| 17:00:30 | bauzas | and you're right, it's confusing | |
| 17:01:34 | jaypipes | bauzas: k, +2 from me. | |
| 17:01:39 | jaypipes | bauzas: sorry about the confusion. | |
| 17:01:44 | bauzas | jaypipes: anyway, will respin | |
| 17:01:51 | bauzas | in order to cut it from the topic | |
| 17:03:51 | jaypipes | rfolco: one teeny missing 1.7 reference still in there. push a fix and I'm good to +2. | |
| 17:04:36 | rfolco | jaypipes, oh geez, will fix it quick | |
| 17:06:17 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: _schedule_instances() supporting a RequestSpec object https://review.openstack.org/469037 | |
| 17:06:34 | bauzas | jaypipes: https://review.openstack.org/#/c/469037/ rev'd for just amending the bp name and a nit | |
| 17:06:55 | bauzas | jaypipes: will look at the shared RP spec later tonight | |
| 17:07:16 | openstackgerrit | Rafael Folco proposed openstack/nova master: placement: Specific error for inventory in use https://review.openstack.org/460231 | |
| 17:09:07 | openstackgerrit | Rafael Folco proposed openstack/nova master: Delete all inventory has its own method DELETE https://review.openstack.org/460147 | |
| 17:11:11 | rfolco | jaypipes, like your portuguese. Thanks :) | |