| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-01 | |||
| 10:53:44 | mdbooth | Just death by 1,000 cuts. | |
| 10:53:55 | johnthetubaguy | it breaks a pretty fundamental API feature, that seemed fairly important to be from a parity point of view, but I duno really. | |
| 10:56:54 | mdbooth | johnthetubaguy: Yeah, I know. But it's been broken a long time and I'd have preferred to see an attempt to fix it without adding tech debt first. | |
| 10:57:34 | johnthetubaguy | mdbooth: that my preference too, but its a balancing act | |
| 10:58:11 | johnthetubaguy | I was very boarderline on -1 +2 myself | |
| 10:58:14 | mdbooth | If it's important to somebody, they'll fix it. You even suggested where to look, and I agreed. | |
| 10:58:48 | bauzas | lyarwood: nice catch, +2d https://review.openstack.org/#/c/427267/2 | |
| 10:59:13 | sfinucan | bauzas: Could you take a look at https://review.openstack.org/#/c/392643/, please? :) | |
| 10:59:14 | bauzas | johnthetubaguy: in case you wanna fix a nasty bug with an easy peasy fix ^ | |
| 10:59:35 | bauzas | sfinucan: roger | |
| 10:59:39 | mdbooth | johnthetubaguy: Now ^^^ is a nasty bug :) | |
| 10:59:56 | mdbooth | Ah, not that one. | |
| 11:00:13 | mdbooth | The one before which bauzas mentioned. | |
| 11:02:03 | kashyap | sfinucan: Your friendly monthly reminder: If you put commit IDs in single quotes, then 'clickability" is lost :-) | |
| 11:02:10 | sfinucan | lyarwood: Finally done, though it seems bauzas pipped me to the mark :) | |
| 11:02:23 | sfinucan | nasty old bug indeed | |
| 11:02:24 | kashyap | sfinucan: Just reminding for next time. | |
| 11:02:26 | lyarwood | bauzas: thanks! | |
| 11:02:33 | sfinucan | sorry, kashyap :'( | |
| 11:02:33 | lyarwood | sfinucan: cheers, np :) | |
| 11:11:21 | openstackgerrit | Sergey Nikitin proposed openstack/nova-specs: Restore V3 Diagnostics info https://review.openstack.org/427637 | |
| 11:15:59 | sfinucan | johnthetubaguy: What are we doing about https://review.openstack.org/#/c/357190 ? | |
| 11:16:41 | johnthetubaguy | sfinucan: not sure what you mean? | |
| 11:16:53 | johnthetubaguy | oh, its that damm mox things | |
| 11:17:01 | sfinucan | aye :) | |
| 11:18:24 | johnthetubaguy | sfinucan: these bits hurt my eyes a little "mock_domain.undefineFlags.assert_has_calls([mock.call(1)])" | |
| 11:18:45 | johnthetubaguy | assert_called_once_with(1) would be much better | |
| 11:19:13 | sfinucan | derekh - you around? | |
| 11:19:21 | derekh | sfinucan: yup | |
| 11:19:57 | sfinucan | See John's comments above - could you respin https://review.openstack.org/#/c/417227 accordingly | |
| 11:20:08 | johnthetubaguy | its OK, we can do a follow on patch | |
| 11:20:26 | sfinucan | johnthetubaguy: Won't argue with that | |
| 11:20:27 | johnthetubaguy | derekh: its these bits, I think we can make it a lot simpler: https://review.openstack.org/#/c/417227/2/nova/tests/unit/virt/libvirt/test_driver.py@11908 | |
| 11:20:45 | johnthetubaguy | derekh: I wouldn't complain normally, but folks might start copying that pattern | |
| 11:21:12 | derekh | johnthetubaguy: ack, I'll push a follow up this morning | |
| 11:21:21 | johnthetubaguy | derekh: awesome, thanks | |
| 11:21:35 | derekh | johnthetubaguy: np | |
| 11:21:44 | derekh | johnthetubaguy: sfinucan thanks both | |
| 11:21:45 | johnthetubaguy | sfinucan: my browsers got out of hand, so I gave up this morning and closed everything! | |
| 11:21:49 | sfinucan | (y) | |
| 11:22:57 | sfinucan | johnthetubaguy: Also, are we proceeding with this change or punting to Pike? https://review.openstack.org/#/c/412634/ | |
| 11:23:17 | sfinucan | mriedem's etherpad sugggests the former, but it seems a little late in the day https://etherpad.openstack.org/p/nova-ocata-feature-freeze | |
| 11:24:00 | johnthetubaguy | sfinucan: so we have merged half of it, but its tempting to wait until friday, i.e. pike | |
| 11:24:23 | sfinucan | that would be my gut feeling - it's not critical | |
| 11:25:17 | jd__ | hi there | |
| 11:25:33 | jd__ | could someone give me a hint on what this failure means http://logs.openstack.org/82/405682/8/check/gate-ceilometer-dsvm-tempest-plugin-postgresql-ubuntu-xenial/02f896f/logs/screen-n-sch.txt.gz#_2017-02-01_10_20_33_556 ? | |
| 11:25:56 | johnthetubaguy | sfinucan: so it does stop people from taking large snapshots | |
| 11:26:50 | johnthetubaguy | sfinucan: I am tempted to ask mriedem to make the call | |
| 11:28:10 | sfinucan | johnthetubaguy: sounds fine by me - he's clearly thinking about it anyway, given the Etherpad | |
| 11:28:24 | openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Resolve TODOs in 'database' https://review.openstack.org/393695 | |
| 11:30:05 | johnthetubaguy | sfinucan: completeness wise, we have merged half of it already | |
| 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? | |