| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-01 | |||
| 11:31:36 | openstackgerrit | Stephen Finucane proposed openstack/nova: rbd: Remove unnecessary 'encode' calls https://review.openstack.org/412356 | |
| 11:37:19 | sfinucan | johnthetubaguy: True, but it's still a new feature rather than a regression fix. Definitely happy to let someone else make the decision for me :) | |
| 11:37:33 | johnthetubaguy | sfinucan: its totally a feature | |
| 11:47:01 | cdent | jd__: that's probably this: http://lists.openstack.org/pipermail/openstack-dev/2017-January/111295.html | |
| 11:59:15 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova: Optimize time during PCI alias parsing process https://review.openstack.org/427145 | |
| 12:00:30 | openstackgerrit | John Garbutt proposed openstack/nova: Adopts keystoneauth with glance client. https://review.openstack.org/412634 | |
| 12:03:59 | openstackgerrit | John Garbutt proposed openstack/nova: Add service_token for nova-glance interaction https://review.openstack.org/419662 | |
| 12:06:49 | openstackgerrit | John Garbutt proposed openstack/nova: Add service_token for nova-glance interaction https://review.openstack.org/419662 | |
| 12:06:49 | openstackgerrit | John Garbutt proposed openstack/nova: Adopts keystoneauth with glance client. https://review.openstack.org/412634 | |
| 12:09:09 | openstackgerrit | John Garbutt proposed openstack/nova: DNM: Test service token in Glance https://review.openstack.org/419666 | |
| 12:10:18 | johnthetubaguy | sfinucan: I fixed up my nits on those patches ^ | |
| 12:11:08 | openstackgerrit | Maciej Szankin proposed openstack/nova: config options: remove libvirt imagecache https://review.openstack.org/378497 | |
| 12:12:02 | sfinucan | johnthetubaguy: I'll take a look now | |
| 12:14:33 | openstackgerrit | Lee Yarwood proposed openstack/nova: WIP - libvirt: Remove redundant bdm serial mangling and saving during swap_volume https://review.openstack.org/427364 | |
| 12:21:52 | sileht | cdent, we use the default devstack setup with additional plugin, we don't override the default service list, does the placement-api is not enabled by default ? | |
| 12:22:44 | cdent | sileht: it is enabled by default, but I thought most dsvm jobs didn't do anything by default? | |
| 12:23:50 | sileht | cdent, the failing dsvm job run the default | |
| 12:24:29 | sileht | cdent, looks like a real postgres issue: http://logs.openstack.org/82/405682/8/check/gate-ceilometer-dsvm-tempest-plugin-postgresql-ubuntu-xenial/02f896f/logs/apache/placement-api.txt.gz?level=ERROR | |
| 12:24:50 | cdent | feh | |
| 12:25:04 | sileht | cdent, this error comes when you miss some field in the groupby | |
| 12:25:22 | cdent | that's a great catch, I don't think nova tests anything in anything but mysql | |
| 12:25:23 | sileht | cdent, when mysql adds them automaticaly | |
| 12:26:07 | cdent | i'll make a bug | |
| 12:26:15 | sileht | cdent, even for mysql this is better to explicit everything for groupby | |
| 12:26:44 | cdent | the sql being used there is...insane (to me) | |
| 12:31:52 | openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/nova: TEST https://review.openstack.org/427667 | |
| 12:36:37 | openstack | Launchpad bug 1660959 in OpenStack Compute (nova) "placement resource provider filtering does not work with postgres" [Undecided,Triaged] | |
| 12:36:37 | cdent | sileht, mriedem : https://bugs.launchpad.net/nova/+bug/1660959 | |
| 12:39:20 | kashyap | lyarwood: Do you have DevStack local.conf with multipath config somewhere? | |
| 12:40:11 | lyarwood | kashyap: nope sorry | |
| 12:41:57 | kashyap | lyarwood: You only test with RPM-based setups? | |
| 12:43:01 | lyarwood | kashyap: yeah, almost all of the multipath issues I've had to work on recently are for Kilo envs so the volume code doesn't even apply to upstream anymore | |
| 12:43:21 | lyarwood | kashyap: well, not since liberty and the split into os-brick | |
| 12:43:31 | kashyap | lyarwood: Hmm, okay. Yeah, it's just a pity that it's such a pain in the neck to figure out the right configs to even test with upstream | |
| 12:44:02 | kashyap | Seems like nobody bothers with it much | |
| 13:02:53 | openstackgerrit | Stephen Finucane proposed openstack/nova: console: Move proxies to 'console/proxy' https://review.openstack.org/408192 | |
| 13:08:33 | wiggin15 | Hi everyone. Something changed in Nova that fails the Cinder third party CI systems. We're getting "No hosts found to map to cell, exiting.". It started yesterday | |
| 13:08:35 | openstackgerrit | Stephen Finucane proposed openstack/nova: tests: Validate huge pages https://review.openstack.org/399653 | |
| 13:08:36 | wiggin15 | Here's Netapp's log: http://dcf901611175aa43f968-c54047c910227e27e1d6f03bb1796fd7.r95.cf5.rackcdn.com/56/427356/1/check/cinder-7mode-iSCSI/0699f1b/devstacklog.txt | |
| 13:08:36 | openstackgerrit | Stephen Finucane proposed openstack/nova: tests: Don't reinvent __init__ https://review.openstack.org/402560 | |
| 13:08:36 | openstackgerrit | Stephen Finucane proposed openstack/nova: tests: Combine multiple NUMA-generation functions https://review.openstack.org/402561 | |
| 13:08:37 | openstackgerrit | Stephen Finucane proposed openstack/nova: tests: Remove duplicate NumaHostInfo https://review.openstack.org/402562 | |
| 13:08:37 | openstackgerrit | Stephen Finucane proposed openstack/nova: libvirt: create functional test base class https://review.openstack.org/407055 | |
| 13:08:52 | wiggin15 | Here's virtuozzo: http://openstack-3rd-party-virtuozzo-ci-logs.virtuozzo.com/71/416371/9/check/check-dsvm-tempest-vz7-exe-minimal/0854760/logs/devstacklog.txt.gz | |
| 13:08:56 | wiggin15 | others are failing too | |
| 13:09:30 | wiggin15 | Does anyone know if something changed in the past 24 hours that can cause this? | |
| 13:10:34 | cdent | wiggin15: sounds like it could be something related to cellsv2 stuff that recently merged | |
| 13:11:48 | cdent | wiggin15: there was some discussion about some changes that need to happen, last night, but I'm not sure of the outcome or timing. when mriedem comes around he'll probably know more. but also check the os-dev list for messages from him, he's being sending gate updates there | |
| 13:16:22 | macsz | was the Nova API meeting canceled for today or is my calendar messed up? | |
| 13:17:09 | openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/nova: WIP: placement-api: fix ResourceProviderList query https://review.openstack.org/427667 | |
| 13:20:10 | johnthetubaguy | macsz: it was cancelled today | |
| 13:20:54 | macsz | johnthetubaguy: ah, ok | |
| 13:28:09 | cdent | rpodolyaka: on group by thing, are you saying: "we're not using c.consumer_id, so just leave it out of the select"? | |
| 13:28:52 | rpodolyaka | cdent: yeah, my understanding is that we calculate total usage values per RP, not "per RP per consumer" | |
| 13:29:08 | rpodolyaka | and this total actually includes multiple consumers | |
| 13:29:28 | rpodolyaka | so consumer_id in select does not make sense and that's what postgresql is unhappy about | |
| 13:30:14 | cdent | makes sense | |
| 13:30:18 | smcginnis | wiggin15: Sounds like you, like me, ignored this in the ML and now has stacking failures: http://lists.openstack.org/pipermail/openstack-dev/2017-January/111295.html | |
| 13:30:33 | rpodolyaka | we *could* group by consumer_id, but my understanding is that it's not what we really want | |
| 13:30:48 | cdent | rpodolyaka: I think you're right | |
| 13:31:03 | rpodolyaka | but I'm not really familiar with this code, so jaypipes should know better :P | |
| 13:31:32 | cdent | rpodolyaka: but the second group_by change is probably right? | |
| 13:31:36 | cdent | (for postgres?) | |
| 13:32:55 | cdent | sileht: you want to carry on with that stuff. If not I can take it over. | |
| 13:33:17 | openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/nova: placement-api: fix ResourceProviderList query https://review.openstack.org/427667 | |
| 13:33:38 | sileht | rpodolyaka, the functional tests have catched your thought :) | |
| 13:35:31 | rpodolyaka | sileht: yay! :) | |
| 13:36:42 | openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/nova: placement-api: fix ResourceProviderList query https://review.openstack.org/427667 | |
| 13:36:44 | cdent | sileht: being very nit picky on the review in the hope that by the time some core comes along it is nice and happy and we get this merged quick | |
| 13:36:56 | sileht | cdent, | |
| 13:37:11 | sileht | cdent, it's not really nit picky when feedback are instant :) | |
| 13:37:23 | rpodolyaka | cdent: hmm, the second group by is done on a unique column - it must be ok, as it uniquely identifies a particular table row. I'd expect "select rp.* ..." work fine in this case | |
| 13:37:58 | rpodolyaka | tbh it's the first time I've seen a sqalclhemy model passed to group_by() | |
| 13:38:17 | rpodolyaka | my understanding is that it's equivalent of grouping by all columns of a table | |
| 13:38:19 | sileht | rpodolyaka, with postgresql you must have everything you put in the select | |
| 13:38:35 | sileht | rpodolyaka, yes because the select does 'select rp.*' | |
| 13:38:52 | sileht | rpodolyaka, so even if rp.uuid is enought, postgresql don't want to guess | |
| 13:40:01 | sileht | rpodolyaka, I think the second groupby is here just to remove duplicate rows ? | |
| 13:40:22 | mriedem | wiggin15: you need to have the placement-api service enabled, in ENABLED_SERVICES | |
| 13:40:36 | mriedem | wiggin15: if you're overriding ENABLED_SERVICES that is - there is a note in the dev list about it | |
| 13:41:53 | sileht | rpodolyaka, the orignal error we have: http://logs.openstack.org/82/405682/8/check/gate-ceilometer-dsvm-tempest-plugin-postgresql-ubuntu-xenial/02f896f/logs/apache/placement-api.txt.gz?level=ERROR | |
| 13:42:10 | sileht | rpodolyaka, the error when I have fixed the first groupby: http://logs.openstack.org/68/427668/1/check/gate-ceilometer-dsvm-tempest-plugin-postgresql-ubuntu-xenial/372f489/logs/apache/placement-api.txt.gz?level=ERROR | |
| 13:42:22 | mriedem | johnthetubaguy: sfinucan: your call on https://review.openstack.org/#/c/412634/ since it's disabled by default an experimental now anyway | |
| 13:42:36 | mriedem | it was already approved once, just had a busted unit test | |
| 13:43:05 | rpodolyaka | sileht: hmm, interesting | |
| 13:43:08 | openstack | Launchpad bug 1660959 in OpenStack Compute (nova) "placement resource provider filtering does not work with postgres" [Undecided,In progress] - Assigned to Mehdi Abaakouk (sileht) | |
| 13:43:08 | cdent | mriedem: I tagged this https://bugs.launchpad.net/nova/+bug/1660959 as rc-potential because postgres support used to work in placement api. it's what sileht and rpodolyaka are talking over now. | |
| 13:43:15 | mriedem | cdent: yup saw it, thanks | |
| 13:43:17 | mriedem | silly pg | |
| 13:43:24 | wiggin15 | mriedem we have "export OVERRIDE_ENABLED_SERVICES" with services like "c-api", "q-metering" etc., is this it? | |
| 13:43:32 | mriedem | wiggin15: i think so yes | |
| 13:43:35 | mriedem | you need placement-api in that list | |
| 13:43:38 | wiggin15 | mriedem thanks | |
| 13:43:40 | sileht | cdent, and our (future) stable/ocata is already broken too :p | |
| 13:43:40 | sfinucan | mriedem: I'm happy to let it in, in that case. It's technically a feature but it seems low risk to me. Can't speak for johnthetubaguy, mind | |
| 13:43:54 | mriedem | i'm heading into the office, bbiab | |
| 13:44:23 | cdent | sileht: I think the whole world's future is broken lately. | |
| 13:46:29 | rpodolyaka | sileht: heh, looks like it only allows this for PKs, but not for UNIQUEs - http://paste.openstack.org/show/597180/ | |
| 13:48:53 | sileht | rpodolyaka, do we have another column that can do it (instead of putting all cols) ? | |
| 13:51:16 | sileht | rpodolyaka, I will use id, then | |