*** masayukig has quit IRC | 00:01 | |
*** masayukig has joined #openstack-nova | 00:02 | |
*** ljxiash_ has joined #openstack-nova | 00:04 | |
*** Swami_ has joined #openstack-nova | 00:07 | |
*** ljxiash has quit IRC | 00:08 | |
*** dave-mccowan has quit IRC | 00:08 | |
*** Swami has quit IRC | 00:11 | |
*** thorst has joined #openstack-nova | 00:12 | |
*** hongbin has quit IRC | 00:14 | |
*** thorst has quit IRC | 00:15 | |
*** thorst has joined #openstack-nova | 00:16 | |
*** ducttape_ has joined #openstack-nova | 00:20 | |
*** thorst has quit IRC | 00:25 | |
*** catintheroof has quit IRC | 00:25 | |
*** mfisch has quit IRC | 00:28 | |
*** mfisch has joined #openstack-nova | 00:29 | |
*** mfisch has quit IRC | 00:29 | |
*** mfisch has joined #openstack-nova | 00:29 | |
*** yingjun has joined #openstack-nova | 00:29 | |
*** penick has quit IRC | 00:31 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova-specs: Abort Cold Migration https://review.openstack.org/334732 | 00:34 |
---|---|---|
*** tbachman has quit IRC | 00:37 | |
melwitt | dansmith, mriedem: I had been wondering if it would be worth trying to make the CellDatabases stuff smarter with how it picks a context manager for get_context_manager. maybe nah for now? | 00:42 |
mriedem | i get the feeling dan wants to move on | 00:43 |
melwitt | :) | 00:43 |
mriedem | as in, let's not sweat the small things right now and try to get things working, | 00:43 |
mriedem | optimize later | 00:43 |
mriedem | and i'm totally fine with that | 00:43 |
melwitt | I think it's cool to move on for now. I've been experimenting with some things and realized the main weirdness is that the API cell has a main database too, which is something I've been not thinking about until I tried to make _last_ctxt_manager smarter | 00:44 |
*** ljxiash_ has quit IRC | 00:46 | |
*** b_rake has quit IRC | 00:46 | |
dansmith | mriedem: melwitt I don't want to change anything until that set gets a little smaller | 00:47 |
*** b_rake has joined #openstack-nova | 00:47 | |
dansmith | melwitt: did you see the one change I did make though? | 00:47 |
dansmith | I was able to fix a ton of stuff by restoring a default database consistently every time | 00:47 |
melwitt | dansmith: yeah, I saw. I don't completely understand it yet but I'm sure it's related to what I ran into in the end, where there's more than just the compute cell main database and the cell0 main database. there's also the API cell main database and I think that wasn't accounted for until your change | 00:49 |
mriedem | i'd like to get a little smarter too but that's probably not going to happen | 00:49 |
dansmith | mriedem: I guess we let in another db migration that isn't first in ocata | 00:49 |
mriedem | dansmith: i'm fixing that right now | 00:49 |
mriedem | should be up in minutes | 00:49 |
dansmith | okay | 00:49 |
*** mingyu has joined #openstack-nova | 00:50 | |
mriedem | did someone remove testscenarios from test-requirements.txt? | 00:50 |
mriedem | oh need to rebase | 00:51 |
*** b_rake has quit IRC | 00:51 | |
melwitt | dansmith: and I think with the default set to cell1, that folds the API cell main database and the compute cell database together for the test | 00:52 |
dansmith | melwitt: it folds anything that isn't targeting a cell into the default, yeah | 00:52 |
*** hfu has joined #openstack-nova | 00:52 | |
dansmith | that's the point | 00:52 |
*** gjayavelu has quit IRC | 00:52 | |
*** tovin07 has joined #openstack-nova | 00:54 | |
*** tovin07_ has joined #openstack-nova | 00:54 | |
*** mingyu has quit IRC | 00:55 | |
*** diogogmt has quit IRC | 00:55 | |
mriedem | fun fact, if you run nova/tests/unit/db/test_sqlalchemy_migration.py in isolation it fails | 00:55 |
*** penick has joined #openstack-nova | 00:56 | |
melwitt | dansmith: yeah, I was noting that's not accurate as I think there are some non targeted calls that are supposed to go to the API cell's main database, not the compute cell's main database. example, flavor read from main database during ensure_migrated | 00:56 |
melwitt | not always accurate, rather | 00:56 |
dansmith | melwitt: well, that one needs to go away anyway, and also no longer makes sense if we can have two cells | 00:57 |
melwitt | dansmith: yeah, that is true | 00:57 |
*** yingjun has quit IRC | 00:57 | |
dansmith | but you could configure your api's main connection to be whichever of the cells used to be your main database and it would still be okay | 00:57 |
dansmith | melwitt: but if you look at my set now, there are several things yet to fix -- I fixed instance.fault today, which is loaded later and has to be targetd | 00:57 |
dansmith | so when we get rid of any and all of those legacy things, then we can remove the defaulting thing and maybe even make a non-targeted access fail so we can identify any leaks | 00:58 |
*** penick has quit IRC | 00:59 | |
melwitt | dansmith: okay, cool. I'm glad to be convinced what's there now is more reasonable than I started worrying about | 00:59 |
dansmith | well, there's plenty left to do, but yeah | 00:59 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Require cellsv2 setup before migrating to Ocata https://review.openstack.org/392227 | 01:00 |
*** penick has joined #openstack-nova | 01:01 | |
*** rcernin has quit IRC | 01:01 | |
melwitt | I was trying something like associating each service with a host and patching the rpc receive to stash a cell_host in the RequestContext so it could pick the right one associated with the service making the access. similar for init and periodic task. but then I ran into the flavor read thing, so not everything was covered | 01:01 |
*** fragatin_ has quit IRC | 01:02 | |
*** fragatina has joined #openstack-nova | 01:03 | |
*** tovin07_ has quit IRC | 01:04 | |
*** rama_y has quit IRC | 01:04 | |
*** ljxiash has joined #openstack-nova | 01:08 | |
*** ljxiash has quit IRC | 01:08 | |
*** thorst has joined #openstack-nova | 01:09 | |
*** thorst has quit IRC | 01:09 | |
*** thorst has joined #openstack-nova | 01:10 | |
*** tbachman has joined #openstack-nova | 01:10 | |
*** thorst has quit IRC | 01:14 | |
*** thorst has joined #openstack-nova | 01:23 | |
*** thorst has quit IRC | 01:24 | |
*** unicell1 has joined #openstack-nova | 01:26 | |
openstackgerrit | Anusha Unnam proposed openstack/nova: Add more field's in InstancePayload https://review.openstack.org/407128 | 01:26 |
openstackgerrit | zhangyanxian proposed openstack/nova: Fix import method to follow community guideline https://review.openstack.org/405703 | 01:26 |
*** rfolco has quit IRC | 01:26 | |
*** unicell has quit IRC | 01:27 | |
*** yingjun has joined #openstack-nova | 01:30 | |
*** gjayavelu has joined #openstack-nova | 01:30 | |
*** Apoorva has quit IRC | 01:31 | |
*** zhangjl has joined #openstack-nova | 01:33 | |
*** yuntongjin has joined #openstack-nova | 01:35 | |
*** unicell has joined #openstack-nova | 01:36 | |
*** unicell1 has quit IRC | 01:38 | |
*** gjayavelu has quit IRC | 01:38 | |
*** kevinz has joined #openstack-nova | 01:40 | |
*** esberglu has joined #openstack-nova | 01:47 | |
*** hongbin has joined #openstack-nova | 01:47 | |
*** sam_nowitzki has joined #openstack-nova | 01:50 | |
*** esberglu has quit IRC | 01:51 | |
*** esberglu has joined #openstack-nova | 01:53 | |
*** Swami_ has quit IRC | 01:53 | |
*** nic has quit IRC | 01:54 | |
*** esberglu has quit IRC | 01:58 | |
*** penick has quit IRC | 01:59 | |
*** yufei has joined #openstack-nova | 02:01 | |
*** esberglu has joined #openstack-nova | 02:04 | |
*** thorst has joined #openstack-nova | 02:05 | |
*** thorst has quit IRC | 02:05 | |
*** thorst has joined #openstack-nova | 02:06 | |
*** esberglu has quit IRC | 02:08 | |
*** magic has joined #openstack-nova | 02:12 | |
*** int32bit has joined #openstack-nova | 02:14 | |
*** thorst has quit IRC | 02:15 | |
*** hfu has quit IRC | 02:15 | |
*** xiaoyang has quit IRC | 02:16 | |
int32bit | I run test on my host and the result is ok, but Jenkins seems fail for 'oslo_config.cfg.NoSuchOptError: no such option osapi_max_limit in group [DEFAULT]', the log: http://logs.openstack.org/71/405171/3/check/gate-nova-python27-db-ubuntu-xenial/28e7f6c/console.html | 02:17 |
*** mtanino has quit IRC | 02:19 | |
melwitt | int32bit: maybe you need to rebase? (to get the same behavior as jenkins) | 02:19 |
int32bit | OK, I will try it, thanks melwitt. | 02:21 |
melwitt | if a config options move change has landed since you started your branch, you might be out of date on that | 02:22 |
mriedem | int32bit: yeah you need to rebase. those options moved last week. | 02:24 |
*** browne has quit IRC | 02:28 | |
*** yuntongjin has quit IRC | 02:29 | |
int32bit | My work is started from latest master branch, so I have little prospect of moving change the config. | 02:29 |
*** gcb has joined #openstack-nova | 02:31 | |
*** tbachman has quit IRC | 02:31 | |
*** yuntongjin has joined #openstack-nova | 02:32 | |
diana_clarke | int32bit: I think you'll need to change "max_limit = CONF.osapi_max_limit" to something like "max_limit = CONF.api.max_limit" in your test_compute.py | 02:34 |
openstackgerrit | Huan Xie proposed openstack/nova: WIP: XenAPI Use os-xenapi lib for nova https://review.openstack.org/406059 | 02:34 |
diana_clarke | int32bit: And also in api.py | 02:35 |
openstackgerrit | Huan Xie proposed openstack/nova: XenAPI Use os-xenapi lib for nova https://review.openstack.org/406059 | 02:35 |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: conf: Remove config option compute_ manager https://review.openstack.org/405921 | 02:35 |
*** yuntongjin1 has joined #openstack-nova | 02:44 | |
*** hfu has joined #openstack-nova | 02:44 | |
*** yuntongjin has quit IRC | 02:45 | |
*** takashin has joined #openstack-nova | 02:45 | |
*** hongbin has quit IRC | 02:46 | |
*** yuntongjin1 has quit IRC | 02:48 | |
*** woodster_ has quit IRC | 02:56 | |
*** yuntongjin has joined #openstack-nova | 02:56 | |
*** kaisers__ has joined #openstack-nova | 02:58 | |
*** sudipto has joined #openstack-nova | 03:00 | |
*** sudipto_ has joined #openstack-nova | 03:00 | |
*** kaisers_ has quit IRC | 03:01 | |
*** kristian__ has joined #openstack-nova | 03:02 | |
*** brault has quit IRC | 03:04 | |
*** kristian__ has quit IRC | 03:06 | |
*** thorst has joined #openstack-nova | 03:12 | |
*** bkopilov has quit IRC | 03:13 | |
*** yuntongjin has quit IRC | 03:17 | |
*** thorst has quit IRC | 03:20 | |
*** stewie925 has quit IRC | 03:20 | |
*** kevinz has quit IRC | 03:30 | |
*** kevinz has joined #openstack-nova | 03:31 | |
*** hfu has quit IRC | 03:31 | |
*** baoli has joined #openstack-nova | 03:33 | |
*** Amit_M has joined #openstack-nova | 03:34 | |
*** hfu has joined #openstack-nova | 03:36 | |
*** armax has quit IRC | 03:36 | |
*** hfu has quit IRC | 03:40 | |
*** hfu has joined #openstack-nova | 03:40 | |
*** code-R has joined #openstack-nova | 03:42 | |
*** CristinaPauna has quit IRC | 03:43 | |
*** hfu has quit IRC | 03:44 | |
*** hfu has joined #openstack-nova | 03:45 | |
*** code-R_ has joined #openstack-nova | 03:45 | |
*** armax has joined #openstack-nova | 03:45 | |
*** hfu has quit IRC | 03:47 | |
*** code-R has quit IRC | 03:48 | |
*** ducnc has joined #openstack-nova | 03:48 | |
*** efried has quit IRC | 03:48 | |
*** efried has joined #openstack-nova | 03:50 | |
*** hfu has joined #openstack-nova | 03:50 | |
*** links has joined #openstack-nova | 03:52 | |
*** hfu has quit IRC | 03:54 | |
*** mriedem has quit IRC | 03:56 | |
*** code-R_ has quit IRC | 03:59 | |
*** code-R has joined #openstack-nova | 03:59 | |
*** zhangjl has quit IRC | 03:59 | |
*** zhangjl has joined #openstack-nova | 04:00 | |
*** baoli has quit IRC | 04:03 | |
*** psachin has joined #openstack-nova | 04:03 | |
*** esberglu has joined #openstack-nova | 04:05 | |
*** masber has joined #openstack-nova | 04:07 | |
*** hfu has joined #openstack-nova | 04:08 | |
*** esberglu has quit IRC | 04:10 | |
openstackgerrit | Jay Pipes proposed openstack/nova: placement: correct improper test case inheritance https://review.openstack.org/406363 | 04:10 |
openstackgerrit | Jay Pipes proposed openstack/nova: placement: create aggregate map in report client https://review.openstack.org/407309 | 04:10 |
*** esberglu has joined #openstack-nova | 04:11 | |
*** armax has quit IRC | 04:12 | |
*** armax has joined #openstack-nova | 04:13 | |
*** armax has quit IRC | 04:13 | |
*** hfu has quit IRC | 04:15 | |
*** esberglu has quit IRC | 04:16 | |
*** thorst has joined #openstack-nova | 04:18 | |
*** hfu has joined #openstack-nova | 04:20 | |
*** int32bit has quit IRC | 04:20 | |
*** GB21 has joined #openstack-nova | 04:21 | |
*** mdnadeem has joined #openstack-nova | 04:21 | |
*** thorst has quit IRC | 04:25 | |
*** sudipto_ has quit IRC | 04:25 | |
*** sudipto has quit IRC | 04:25 | |
*** mnaser has quit IRC | 04:25 | |
*** mordred has quit IRC | 04:26 | |
*** greghaynes has quit IRC | 04:26 | |
*** afazekas has quit IRC | 04:26 | |
*** sreeram_v has joined #openstack-nova | 04:26 | |
*** hfu has quit IRC | 04:27 | |
*** GB21 has quit IRC | 04:33 | |
*** sridharg has joined #openstack-nova | 04:34 | |
*** cfriesen has quit IRC | 04:35 | |
*** CristinaPauna has joined #openstack-nova | 04:35 | |
*** bkopilov has joined #openstack-nova | 04:41 | |
*** nicolasbock has quit IRC | 04:48 | |
*** ratailor has joined #openstack-nova | 04:48 | |
*** GB21 has joined #openstack-nova | 04:49 | |
*** alex_xu has joined #openstack-nova | 04:49 | |
*** alex_xu has quit IRC | 04:49 | |
*** alex_xu has joined #openstack-nova | 04:51 | |
openstackgerrit | Diana Clarke proposed openstack/python-novaclient: Revert "Microversion 2.39 - Simple tenant usage pagination" https://review.openstack.org/406360 | 04:52 |
*** afazekas has joined #openstack-nova | 04:58 | |
*** greghaynes has joined #openstack-nova | 05:03 | |
*** mordred has joined #openstack-nova | 05:04 | |
openstackgerrit | Diana Clarke proposed openstack/python-novaclient: Revert "Microversion 2.39 - Simple tenant usage pagination" https://review.openstack.org/406360 | 05:06 |
*** zhangguoqing has joined #openstack-nova | 05:07 | |
*** nandal has joined #openstack-nova | 05:08 | |
nandal | Hi, Is there a way to update some info in release notes? | 05:09 |
*** mnaser has joined #openstack-nova | 05:11 | |
*** yufei has quit IRC | 05:13 | |
*** prateek_ has joined #openstack-nova | 05:13 | |
*** ayogi has joined #openstack-nova | 05:14 | |
*** carthaca_1 has joined #openstack-nova | 05:15 | |
*** mfisch has quit IRC | 05:16 | |
*** anteaya has quit IRC | 05:16 | |
*** mordred has quit IRC | 05:18 | |
*** mnaser has quit IRC | 05:18 | |
*** sudipto has joined #openstack-nova | 05:18 | |
*** sudipto_ has joined #openstack-nova | 05:18 | |
*** carthaca_ has quit IRC | 05:19 | |
*** mat128 has quit IRC | 05:19 | |
*** mingyu has joined #openstack-nova | 05:21 | |
*** cardeois has quit IRC | 05:22 | |
*** mfisch has joined #openstack-nova | 05:23 | |
*** mat128 has joined #openstack-nova | 05:23 | |
*** mfisch is now known as Guest16197 | 05:23 | |
*** code-R has quit IRC | 05:23 | |
*** coreywright has quit IRC | 05:24 | |
*** thorst has joined #openstack-nova | 05:24 | |
*** cardeois has joined #openstack-nova | 05:24 | |
*** coreywright has joined #openstack-nova | 05:24 | |
*** mingyu has quit IRC | 05:26 | |
*** mordred has joined #openstack-nova | 05:26 | |
*** anteaya has joined #openstack-nova | 05:29 | |
gcb | nadal, show you know how to use reno to generate releasenotes, just want update info ,right ? | 05:29 |
*** code-R has joined #openstack-nova | 05:29 | |
*** thorst has quit IRC | 05:30 | |
gcb | nandal ^ | 05:30 |
*** shangxdy has joined #openstack-nova | 05:35 | |
*** hfu has joined #openstack-nova | 05:37 | |
nandal | gcb, thank you for reply | 05:38 |
nandal | i was away | 05:38 |
nandal | ¨Now VMwareVCDriver will set disk.EnableUUID=True by default in all guest VM configuration file. To enable udev to generate /dev/disk/by-id¨ | 05:39 |
nandal | The second sentence of this line is due to a typing mistake | 05:39 |
nandal | which is confusing | 05:40 |
*** hfu has quit IRC | 05:40 | |
nandal | It should be :- | 05:40 |
nandal | ¨Now VMwareVCDriver is set disk.EnableUUID=True by default in all guest VM configuration file to enable udev to generate /dev/disk/by-id¨ | 05:41 |
gcb | Is there a link ? | 05:42 |
nandal | http://docs.openstack.org/releasenotes/nova/newton.html | 05:42 |
*** hfu has joined #openstack-nova | 05:42 | |
*** mnaser has joined #openstack-nova | 05:42 | |
*** mdnadeem has quit IRC | 05:44 | |
gcb | nandal , just update in https://github.com/openstack/nova/blob/master/releasenotes/notes/vmware_disk_enableuuid_true-99b88e00fc168dd3.yaml . But this is for newton one, hope the doc can be generated again | 05:45 |
*** tlian has quit IRC | 05:46 | |
*** hfu has quit IRC | 05:47 | |
*** hfu has joined #openstack-nova | 05:47 | |
nandal | gcb, thank you | 05:47 |
gcb | nandal, np | 05:48 |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: conf: Remove config option compute_ manager https://review.openstack.org/405921 | 05:49 |
*** pbandark has joined #openstack-nova | 05:50 | |
*** penick has joined #openstack-nova | 05:50 | |
openstackgerrit | Alex Xu proposed openstack/nova: api-ref: Complete all the verifications of remote consoles https://review.openstack.org/338051 | 05:50 |
*** mdnadeem has joined #openstack-nova | 05:52 | |
*** kiran-r has joined #openstack-nova | 05:57 | |
*** kristian__ has joined #openstack-nova | 06:01 | |
*** int32bit has joined #openstack-nova | 06:01 | |
*** GB21 has quit IRC | 06:01 | |
*** kiran-r has quit IRC | 06:05 | |
*** kristian__ has quit IRC | 06:05 | |
*** hfu has quit IRC | 06:06 | |
openstackgerrit | int32bit proposed openstack/nova: Page the db query when filter instances with IP https://review.openstack.org/405171 | 06:09 |
*** GB21 has joined #openstack-nova | 06:14 | |
*** hfu has joined #openstack-nova | 06:15 | |
*** esberglu has joined #openstack-nova | 06:18 | |
*** esberglu has quit IRC | 06:23 | |
*** oanson has joined #openstack-nova | 06:25 | |
*** hfu has quit IRC | 06:27 | |
*** sudipto_ has quit IRC | 06:28 | |
*** sudipto has quit IRC | 06:28 | |
*** thorst has joined #openstack-nova | 06:29 | |
*** gmann has quit IRC | 06:30 | |
*** hfu has joined #openstack-nova | 06:32 | |
*** rcernin has joined #openstack-nova | 06:34 | |
*** thorst has quit IRC | 06:34 | |
*** hfu has quit IRC | 06:36 | |
*** yuntongjin has joined #openstack-nova | 06:38 | |
*** code-R has quit IRC | 06:38 | |
*** kaisers__ has quit IRC | 06:43 | |
*** hfu has joined #openstack-nova | 06:46 | |
*** Cristina_ has joined #openstack-nova | 06:47 | |
*** hfu has quit IRC | 06:48 | |
openstackgerrit | Hussain Chachuliya proposed openstack/nova: Deny volume swapping with non-bootable volume https://review.openstack.org/407346 | 06:49 |
*** kristian__ has joined #openstack-nova | 06:49 | |
*** CristinaPauna has quit IRC | 06:50 | |
*** sudipto has joined #openstack-nova | 06:51 | |
*** sudipto_ has joined #openstack-nova | 06:52 | |
*** hfu has joined #openstack-nova | 06:53 | |
*** kristian__ has quit IRC | 06:54 | |
*** code-R has joined #openstack-nova | 06:55 | |
*** hfu has quit IRC | 06:57 | |
*** hfu has joined #openstack-nova | 06:57 | |
*** code-R has quit IRC | 07:00 | |
*** markus_z has joined #openstack-nova | 07:04 | |
*** mjura has joined #openstack-nova | 07:06 | |
*** rcernin has quit IRC | 07:12 | |
*** mjura has quit IRC | 07:13 | |
*** mjura has joined #openstack-nova | 07:14 | |
*** code-R has joined #openstack-nova | 07:14 | |
*** kristian__ has joined #openstack-nova | 07:15 | |
*** nkrinner_afk is now known as nkrinner | 07:18 | |
*** andreas_s has joined #openstack-nova | 07:21 | |
*** mat128 has quit IRC | 07:22 | |
*** voelzmo has joined #openstack-nova | 07:22 | |
*** voelzmo has quit IRC | 07:23 | |
*** voelzmo has joined #openstack-nova | 07:23 | |
*** vladikr has quit IRC | 07:23 | |
*** mat128 has joined #openstack-nova | 07:23 | |
*** shz has quit IRC | 07:24 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Check destination_type when booting with bdm provided https://review.openstack.org/402372 | 07:24 |
openstackgerrit | Takashi NATSUME proposed openstack/nova-specs: Fix formatting errors and warnings https://review.openstack.org/407364 | 07:25 |
*** mordred has quit IRC | 07:25 | |
*** bhagyashris has joined #openstack-nova | 07:27 | |
*** Guest66666 has quit IRC | 07:27 | |
*** kaisers_ has joined #openstack-nova | 07:28 | |
*** cardeois has quit IRC | 07:29 | |
*** Guest66666 has joined #openstack-nova | 07:29 | |
*** alexpilo_ has joined #openstack-nova | 07:29 | |
*** takashin has left #openstack-nova | 07:29 | |
*** alexpilotti has quit IRC | 07:30 | |
*** mnaser has quit IRC | 07:30 | |
*** mrsoul has quit IRC | 07:32 | |
*** kristia__ has joined #openstack-nova | 07:32 | |
*** thorst has joined #openstack-nova | 07:32 | |
*** mordred has joined #openstack-nova | 07:34 | |
*** kristi___ has joined #openstack-nova | 07:34 | |
*** rcernin has joined #openstack-nova | 07:34 | |
*** cardeois has joined #openstack-nova | 07:34 | |
*** mrsoul has joined #openstack-nova | 07:35 | |
*** kristian__ has quit IRC | 07:35 | |
*** Jeffrey4l has quit IRC | 07:36 | |
*** kaisers_ has quit IRC | 07:36 | |
*** vladikr has joined #openstack-nova | 07:37 | |
*** kristia__ has quit IRC | 07:38 | |
*** kristi___ has quit IRC | 07:39 | |
*** GB21 has quit IRC | 07:39 | |
*** thorst has quit IRC | 07:40 | |
*** pbandark has quit IRC | 07:41 | |
*** pbandark has joined #openstack-nova | 07:41 | |
*** pcaruana has joined #openstack-nova | 07:42 | |
*** pbandark has quit IRC | 07:42 | |
*** kristian__ has joined #openstack-nova | 07:42 | |
*** shangxdy has quit IRC | 07:44 | |
openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Check source_type and destination_type when booting with bdm provided https://review.openstack.org/402364 | 07:44 |
*** yamahata has quit IRC | 07:44 | |
*** pbandark has joined #openstack-nova | 07:46 | |
*** kristian__ has quit IRC | 07:47 | |
*** code-R has quit IRC | 07:47 | |
markus_z | pkoniszewski: I'm not sure how to resolve your -1 on https://review.openstack.org/#/c/347471/ Are the backports to newton a pre-condition? | 07:48 |
*** hfu has quit IRC | 07:48 | |
*** hfu has joined #openstack-nova | 07:49 | |
*** mnaser has joined #openstack-nova | 07:50 | |
*** tuanluong has joined #openstack-nova | 07:51 | |
*** moshele has joined #openstack-nova | 07:51 | |
*** Jeffrey4l has joined #openstack-nova | 07:52 | |
*** hfu has quit IRC | 07:53 | |
*** fragatina has quit IRC | 07:53 | |
*** hfu has joined #openstack-nova | 07:53 | |
*** fragatina has joined #openstack-nova | 07:54 | |
openstackgerrit | int32bit proposed openstack/nova: Page the db query when filter instances with IP https://review.openstack.org/405171 | 07:55 |
*** fragatina has quit IRC | 07:59 | |
*** b_rake has joined #openstack-nova | 07:59 | |
*** b_rake has quit IRC | 07:59 | |
*** b_rake has joined #openstack-nova | 08:00 | |
*** mvk has quit IRC | 08:01 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Don't delete neutron port when attach failed https://review.openstack.org/403568 | 08:02 |
*** GB21 has joined #openstack-nova | 08:02 | |
*** matrohon has joined #openstack-nova | 08:02 | |
*** ihrachys has joined #openstack-nova | 08:02 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Remove mox in tests/unit/objects/test_security_group https://review.openstack.org/307135 | 08:03 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Remove mox in tests/unit/objects/test_instance_info_cache https://review.openstack.org/307155 | 08:03 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Remove mox in tests/unit/objects/test_compute_node https://review.openstack.org/307625 | 08:03 |
*** FL1SK has quit IRC | 08:04 | |
*** b_rake has quit IRC | 08:04 | |
bauzas | p!spy | 08:07 |
*** shaohe_feng has joined #openstack-nova | 08:07 | |
*** nandal has quit IRC | 08:08 | |
openstackgerrit | Huan Xie proposed openstack/nova: XenAPI Use os-xenapi lib for nova https://review.openstack.org/406059 | 08:08 |
*** gszasz has joined #openstack-nova | 08:11 | |
*** lpetrut has joined #openstack-nova | 08:13 | |
*** brault has joined #openstack-nova | 08:13 | |
*** mnaser has quit IRC | 08:15 | |
*** slaweq_ has quit IRC | 08:17 | |
*** shaohe_feng has quit IRC | 08:17 | |
*** shangxdy has joined #openstack-nova | 08:17 | |
*** shaohe_feng has joined #openstack-nova | 08:18 | |
*** ralonsoh has joined #openstack-nova | 08:18 | |
*** code-R has joined #openstack-nova | 08:18 | |
*** ljxiash has joined #openstack-nova | 08:20 | |
*** code-R_ has joined #openstack-nova | 08:21 | |
*** yuntongjin has quit IRC | 08:21 | |
*** slaweq_ has joined #openstack-nova | 08:22 | |
*** code-R has quit IRC | 08:24 | |
openstackgerrit | wanghao proposed openstack/nova: Set min_disk in the image meta based on the root volume's size https://review.openstack.org/407397 | 08:24 |
*** ljxiash has quit IRC | 08:24 | |
*** ccard_ has quit IRC | 08:25 | |
*** shaohe_feng has quit IRC | 08:27 | |
*** tuanluong has quit IRC | 08:29 | |
*** shaohe_feng has joined #openstack-nova | 08:30 | |
openstackgerrit | Pavel Kholkin proposed openstack/nova: [proxy-api] microversion 2.39 deprecates image-metadata proxy API https://review.openstack.org/405461 | 08:32 |
pkholkin | alex_xu: hello and thanks a lot for review! I've added two notes, please take a look https://review.openstack.org/#/c/405461 | 08:32 |
*** mvk has joined #openstack-nova | 08:33 | |
*** int32bit has quit IRC | 08:35 | |
*** mnaser has joined #openstack-nova | 08:35 | |
alex_xu | pkholkin: thanks, will check it asap | 08:35 |
*** shaohe_feng has quit IRC | 08:37 | |
*** thorst has joined #openstack-nova | 08:38 | |
*** shaohe_feng has joined #openstack-nova | 08:39 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: conf: Remove config option compute_ manager https://review.openstack.org/405921 | 08:42 |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: create consoles in an understandable/extensible way https://review.openstack.org/407120 | 08:42 |
openstackgerrit | Maho Koshiya proposed openstack/nova: Add metadata functional negative tests https://review.openstack.org/391744 | 08:43 |
openstackgerrit | Gábor Antal proposed openstack/python-novaclient: Use more specific asserts in tests https://review.openstack.org/365667 | 08:43 |
*** zenoway has joined #openstack-nova | 08:43 | |
*** thorst has quit IRC | 08:44 | |
*** mkoderer has joined #openstack-nova | 08:46 | |
*** diga has joined #openstack-nova | 08:46 | |
*** zhangguoqing has quit IRC | 08:47 | |
*** shaohe_feng has quit IRC | 08:47 | |
*** shaohe_feng has joined #openstack-nova | 08:50 | |
*** andrey-mp has joined #openstack-nova | 08:50 | |
openstackgerrit | tianmaofu proposed openstack/nova: Unity the comparison of hw_qemu_guest_agent https://review.openstack.org/407409 | 08:51 |
*** mdnadeem has quit IRC | 08:52 | |
*** ociuhandu has quit IRC | 08:53 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Don't apply multi-queue to SRIOV ports https://review.openstack.org/397545 | 08:54 |
openstackgerrit | Moshe Levi proposed openstack/os-vif: [WIP] add OVS acceleration https://review.openstack.org/398277 | 08:55 |
*** shaohe_feng has quit IRC | 08:56 | |
*** tuanluong has joined #openstack-nova | 08:57 | |
*** shaohe_feng has joined #openstack-nova | 08:58 | |
*** zzzeek has quit IRC | 09:00 | |
*** zzzeek has joined #openstack-nova | 09:00 | |
*** roeyc has joined #openstack-nova | 09:02 | |
*** psachin has quit IRC | 09:04 | |
*** mdnadeem has joined #openstack-nova | 09:04 | |
*** GB21 has quit IRC | 09:05 | |
*** ociuhandu has joined #openstack-nova | 09:08 | |
*** larainema has quit IRC | 09:14 | |
*** larainema_ has joined #openstack-nova | 09:14 | |
*** gmann has joined #openstack-nova | 09:15 | |
openstackgerrit | Oleg Bondarev proposed openstack/nova: Live migration: wait for vif-plugged event on pre live migration https://review.openstack.org/246910 | 09:15 |
*** psachin has joined #openstack-nova | 09:17 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Check destination_type when booting with bdm provided https://review.openstack.org/402372 | 09:18 |
*** GB21 has joined #openstack-nova | 09:18 | |
*** ftersin has joined #openstack-nova | 09:21 | |
*** thorst has joined #openstack-nova | 09:24 | |
*** acabot_ has joined #openstack-nova | 09:27 | |
*** rmart04 has joined #openstack-nova | 09:27 | |
*** kristian__ has joined #openstack-nova | 09:27 | |
*** kristian__ has joined #openstack-nova | 09:27 | |
*** sreeram_v has quit IRC | 09:29 | |
*** sreeram_v has joined #openstack-nova | 09:30 | |
*** thorst has quit IRC | 09:31 | |
*** sreeram_v has quit IRC | 09:34 | |
*** sreeram_v has joined #openstack-nova | 09:34 | |
*** yassine has joined #openstack-nova | 09:34 | |
*** paul-carlton1 has joined #openstack-nova | 09:34 | |
*** yassine is now known as Guest17450 | 09:35 | |
*** daemontool has joined #openstack-nova | 09:35 | |
*** lucas-afk is now known as lucasagomes | 09:35 | |
*** paul-carlton2 has joined #openstack-nova | 09:35 | |
*** paul-carlton2 has quit IRC | 09:37 | |
*** paul-carlton1 has quit IRC | 09:37 | |
*** paul-carlton2 has joined #openstack-nova | 09:38 | |
*** acabot__ has joined #openstack-nova | 09:40 | |
*** yingjun has quit IRC | 09:40 | |
*** derekh has joined #openstack-nova | 09:40 | |
*** kristian__ has quit IRC | 09:40 | |
gibi | johnthetubaguy, bauzas: hi! there is a pretty simple test-only change that has +2 from jaypipes already. Could you check it please? https://review.openstack.org/#/c/407033/ | 09:42 |
bauzas | gibi: oh yeah, saw the change | 09:45 |
bauzas | gibi: +Wed | 09:45 |
gibi | bauzas: thanks a lot! | 09:46 |
*** sudipto_ has quit IRC | 09:47 | |
*** sudipto has quit IRC | 09:47 | |
openstackgerrit | Timofey Durakov proposed openstack/nova: fix for auth during live-migration https://review.openstack.org/407147 | 09:53 |
pkoniszewski | markus_z: can we conditionally start those serial console tests? I mean, can we add a condition so that if this is a grenade job and branch is stable/newton it won't start serial console tests? | 09:54 |
*** gmann has quit IRC | 09:58 | |
*** gmann has joined #openstack-nova | 09:58 | |
*** huhaoran has joined #openstack-nova | 10:01 | |
*** acabot_ has quit IRC | 10:03 | |
*** acabot__ is now known as acabot_ | 10:03 | |
*** voelzmo has quit IRC | 10:03 | |
*** FL1SK has joined #openstack-nova | 10:05 | |
*** nkrinner has quit IRC | 10:05 | |
*** nkrinner has joined #openstack-nova | 10:06 | |
*** kristian__ has joined #openstack-nova | 10:06 | |
*** sahid has joined #openstack-nova | 10:06 | |
*** tovin07 has quit IRC | 10:09 | |
*** kristian__ has quit IRC | 10:11 | |
*** voelzmo has joined #openstack-nova | 10:11 | |
openstackgerrit | Feodor Tersin proposed openstack/nova: Implement ScaleIO image backend https://review.openstack.org/407440 | 10:11 |
*** shangxdy has quit IRC | 10:13 | |
*** oanson has quit IRC | 10:15 | |
*** eandersson has joined #openstack-nova | 10:15 | |
*** nkrinner has quit IRC | 10:15 | |
*** aweeks has quit IRC | 10:16 | |
*** shaohe_feng has quit IRC | 10:17 | |
*** nkrinner has joined #openstack-nova | 10:17 | |
*** shaohe_feng has joined #openstack-nova | 10:17 | |
*** esberglu has joined #openstack-nova | 10:20 | |
*** sam_nowitzki has quit IRC | 10:23 | |
*** daemontool has quit IRC | 10:23 | |
*** baoli has joined #openstack-nova | 10:24 | |
*** JoseMello has joined #openstack-nova | 10:24 | |
*** esberglu has quit IRC | 10:24 | |
*** thorst has joined #openstack-nova | 10:28 | |
*** baoli has quit IRC | 10:28 | |
*** cdent has joined #openstack-nova | 10:29 | |
*** kristian__ has joined #openstack-nova | 10:30 | |
*** voelzmo has quit IRC | 10:30 | |
openstackgerrit | Chris Dent proposed openstack/nova: Update cors-to-versions-pipeline release note https://review.openstack.org/407021 | 10:31 |
*** voelzmo has joined #openstack-nova | 10:32 | |
*** kristia__ has joined #openstack-nova | 10:32 | |
*** kristia__ has quit IRC | 10:33 | |
*** kristia__ has joined #openstack-nova | 10:34 | |
*** kristian__ has quit IRC | 10:34 | |
*** kristia__ has quit IRC | 10:35 | |
*** kristian__ has joined #openstack-nova | 10:35 | |
*** thorst has quit IRC | 10:36 | |
*** voelzmo has quit IRC | 10:36 | |
*** kristia__ has joined #openstack-nova | 10:40 | |
*** kristian__ has quit IRC | 10:40 | |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: virtlogd: use virtlogd for char devices https://review.openstack.org/407450 | 10:41 |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: virtlogd: use virtlogd for char devices https://review.openstack.org/407450 | 10:42 |
*** ducnc has quit IRC | 10:43 | |
*** sreeram_v has quit IRC | 10:43 | |
*** sreeram_v has joined #openstack-nova | 10:44 | |
markus_z | pkoniszewski: good question, I'll investigate that and let you know at the l-m meeting today. | 10:45 |
*** huhaoran has quit IRC | 10:46 | |
sfinucan | :q | 10:47 |
*** huhaoran has joined #openstack-nova | 10:47 | |
*** nandal has joined #openstack-nova | 10:49 | |
*** rfolco has joined #openstack-nova | 10:49 | |
*** kevinz has quit IRC | 10:50 | |
*** huhaoran has quit IRC | 10:51 | |
*** huhaoran has joined #openstack-nova | 10:51 | |
*** tbachman has joined #openstack-nova | 10:53 | |
*** hfu has quit IRC | 10:58 | |
*** kristian__ has joined #openstack-nova | 10:59 | |
*** kristia__ has quit IRC | 11:02 | |
*** sdague has joined #openstack-nova | 11:02 | |
*** kristia__ has joined #openstack-nova | 11:04 | |
*** yingjun has joined #openstack-nova | 11:05 | |
*** kristian__ has quit IRC | 11:08 | |
*** zhangjl has quit IRC | 11:11 | |
*** daemontool has joined #openstack-nova | 11:11 | |
openstackgerrit | sahid proposed openstack/nova: objects: add new field cpu_emulator_threads_policy https://review.openstack.org/385022 | 11:12 |
openstackgerrit | sahid proposed openstack/nova: numa: add numa constraints for emulator threads policy https://review.openstack.org/385023 | 11:12 |
openstackgerrit | sahid proposed openstack/nova: libvirt: return a CPU overhead if isolate emulator threads requested https://review.openstack.org/385364 | 11:12 |
openstackgerrit | sahid proposed openstack/nova: objects: add new field cpus_overhead in NUMACell https://review.openstack.org/400874 | 11:12 |
openstackgerrit | sahid proposed openstack/nova: numa: fit instance toplogy with cpu overhead onto cores https://review.openstack.org/400875 | 11:12 |
openstackgerrit | sahid proposed openstack/nova: virt: update overhead to take into account vCPUs https://review.openstack.org/385024 | 11:12 |
openstackgerrit | sahid proposed openstack/nova: libvirt: make emulator threads to run on the reserved pCPU https://review.openstack.org/400878 | 11:12 |
openstackgerrit | sahid proposed openstack/nova: numa: take into account cpus overhead https://review.openstack.org/400876 | 11:12 |
openstackgerrit | sahid proposed openstack/nova: numa: update numa usage incording the cpu overhead https://review.openstack.org/400877 | 11:12 |
*** dimtruck is now known as zz_dimtruck | 11:13 | |
*** owalsh has joined #openstack-nova | 11:13 | |
*** yingjun has quit IRC | 11:13 | |
*** hshiina has joined #openstack-nova | 11:15 | |
*** claudiub|2 has joined #openstack-nova | 11:22 | |
*** kristia__ has quit IRC | 11:23 | |
*** kevinz has joined #openstack-nova | 11:23 | |
*** andrey-mp has left #openstack-nova | 11:23 | |
*** kristian__ has joined #openstack-nova | 11:25 | |
*** sreeram_v has quit IRC | 11:25 | |
*** kristia__ has joined #openstack-nova | 11:28 | |
*** kristian__ has quit IRC | 11:28 | |
*** kristia__ has quit IRC | 11:29 | |
*** yingjun has joined #openstack-nova | 11:30 | |
openstackgerrit | Pawel Koniszewski proposed openstack/nova: live-mig: Add claims and proper resource tracking https://review.openstack.org/244489 | 11:31 |
*** thorst has joined #openstack-nova | 11:34 | |
*** voelzmo has joined #openstack-nova | 11:36 | |
*** yingjun has quit IRC | 11:36 | |
*** alexpilo_ has quit IRC | 11:36 | |
*** yingjun has joined #openstack-nova | 11:36 | |
*** alexpilotti has joined #openstack-nova | 11:37 | |
*** mdnadeem has quit IRC | 11:37 | |
*** ociuhandu has quit IRC | 11:37 | |
*** nicolasbock has joined #openstack-nova | 11:38 | |
*** GB21 has quit IRC | 11:38 | |
openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Add comments in _hard_reboot https://review.openstack.org/407470 | 11:39 |
*** ihrachys has quit IRC | 11:39 | |
mdbooth | johnthetubaguy: ^^^ Would take approximately 2 seconds of your time :) It just adds a couple of comments. | 11:40 |
*** voelzmo has quit IRC | 11:40 | |
*** thorst has quit IRC | 11:41 | |
*** yingjun has quit IRC | 11:41 | |
*** alexpilotti has quit IRC | 11:41 | |
*** lpetrut has quit IRC | 11:44 | |
*** jamielennox is now known as jamielennox|away | 11:44 | |
*** prateek_ has quit IRC | 11:45 | |
*** cdelatte has joined #openstack-nova | 11:46 | |
markus_z | sfinucan: congrats on the new role! You've earned it. | 11:46 |
sfinucan | markus_z: Well it's not confirmed yet, but thanks :) | 11:46 |
*** alexpilotti has joined #openstack-nova | 11:47 | |
mdbooth | sfinucan: woot, congrats! | 11:47 |
*** ratailor has quit IRC | 11:48 | |
*** mdnadeem has joined #openstack-nova | 11:49 | |
*** tbachman has quit IRC | 11:49 | |
*** nkrinner has quit IRC | 11:50 | |
sfinucan | mdbooth: Cheers :) | 11:51 |
*** alexpilotti has quit IRC | 11:51 | |
*** GB21 has joined #openstack-nova | 11:55 | |
*** voelzmo has joined #openstack-nova | 11:56 | |
*** nkrinner has joined #openstack-nova | 11:58 | |
*** smatzek has joined #openstack-nova | 12:00 | |
*** nkrinner has quit IRC | 12:06 | |
*** nkrinner has joined #openstack-nova | 12:06 | |
openstackgerrit | Feodor Tersin proposed openstack/nova: Implement ScaleIO image backend https://review.openstack.org/407440 | 12:07 |
*** GB21 has quit IRC | 12:07 | |
*** tuanluong has quit IRC | 12:08 | |
ftersin | mdbooth: hi. have you seen my comments on https://review.openstack.org/#/c/333244 and the next patch? | 12:09 |
*** yingjun has joined #openstack-nova | 12:10 | |
*** portdirect__ has joined #openstack-nova | 12:10 | |
*** hfu has joined #openstack-nova | 12:11 | |
*** lpetrut has joined #openstack-nova | 12:11 | |
*** GB21 has joined #openstack-nova | 12:16 | |
*** kristian__ has joined #openstack-nova | 12:16 | |
*** kristian__ has quit IRC | 12:17 | |
*** kristian__ has joined #openstack-nova | 12:17 | |
*** rtheis has joined #openstack-nova | 12:17 | |
*** esberglu has joined #openstack-nova | 12:21 | |
*** kristian__ has quit IRC | 12:22 | |
*** kristian__ has joined #openstack-nova | 12:23 | |
*** ihrachys has joined #openstack-nova | 12:24 | |
*** shangxdy has joined #openstack-nova | 12:25 | |
*** esberglu has quit IRC | 12:25 | |
*** bvanhav has joined #openstack-nova | 12:27 | |
*** lpetrut has quit IRC | 12:27 | |
*** kristian__ has quit IRC | 12:28 | |
*** thorst has joined #openstack-nova | 12:30 | |
*** code-R_ has quit IRC | 12:30 | |
*** lucasagomes is now known as lucas-hungry | 12:30 | |
*** databus23_ has joined #openstack-nova | 12:31 | |
*** dgonzalez_ has joined #openstack-nova | 12:31 | |
*** tommy_ has joined #openstack-nova | 12:31 | |
*** david_1 has joined #openstack-nova | 12:31 | |
*** tpatzig_ has joined #openstack-nova | 12:31 | |
*** code-R has joined #openstack-nova | 12:31 | |
*** dgonzalez_ has quit IRC | 12:31 | |
*** sapcc-bot has quit IRC | 12:31 | |
*** databus23_ has quit IRC | 12:31 | |
*** tommy_ has quit IRC | 12:31 | |
*** tpatzig_ has quit IRC | 12:31 | |
*** carthaca_1 has quit IRC | 12:31 | |
*** david_1 has quit IRC | 12:31 | |
*** sreeram_v has joined #openstack-nova | 12:35 | |
sdague | markus_z: once jaypipes gets up, we should get him to look at the alternative virtlogd stack | 12:37 |
markus_z | sdague: yep, will keep that in mind and ping him later | 12:37 |
sdague | because it feels a lot easier to understand in the final code (even though that first patch is a bit complex to get there) | 12:37 |
sdague | I +2ed the alternative stack | 12:37 |
cdent | bauzas, jaypipes, edleafe: could one of you sanity check/triage this bug please if you have a chance: https://bugs.launchpad.net/nova/+bug/1647697 ? I discovered it might be the case while editing the grp spec and don't want to lose my motivation on that. | 12:38 |
openstack | Launchpad bug 1647697 in OpenStack Compute (nova) "When deleting a resource provider in the placement api the rp's associated aggregates are not cleaned up" [Undecided,New] | 12:38 |
*** hferenc has joined #openstack-nova | 12:38 | |
openstackgerrit | sahid proposed openstack/nova: libvirt: return a CPU overhead if isolate emulator threads requested https://review.openstack.org/385364 | 12:38 |
openstackgerrit | sahid proposed openstack/nova: libvirt: make emulator threads to run on the reserved pCPU https://review.openstack.org/400878 | 12:38 |
markus_z | sdague: Thanks a lot. FWIW, if it weren't for the deadline next week I would have done it differently. | 12:39 |
sdague | markus_z: yep, no worries | 12:39 |
*** bkopilov has quit IRC | 12:39 | |
*** sreeram_v has quit IRC | 12:40 | |
*** ociuhandu has joined #openstack-nova | 12:41 | |
openstackgerrit | zhaolihui proposed openstack/nova: Fix keypair quota error https://review.openstack.org/403529 | 12:44 |
*** larainema_ has quit IRC | 12:46 | |
*** larainema_ has joined #openstack-nova | 12:46 | |
*** Kevin_Zheng has quit IRC | 12:46 | |
*** Kevin_Zheng has joined #openstack-nova | 12:47 | |
*** cdelatte has quit IRC | 12:51 | |
*** GB21 has quit IRC | 12:54 | |
*** RuiChen has quit IRC | 12:56 | |
*** belmoreira has joined #openstack-nova | 12:56 | |
*** RuiChen has joined #openstack-nova | 12:57 | |
bauzas | cdent: looks correct | 12:57 |
bauzas | I mean, looks reasonable | 12:57 |
cdent | thanks bauzas | 12:57 |
*** cdelatte has joined #openstack-nova | 12:58 | |
openstackgerrit | jichenjc proposed openstack/nova: Remove AdminRequired usage in flavor https://review.openstack.org/406580 | 12:58 |
*** ayogi has quit IRC | 12:59 | |
*** nkrinner has quit IRC | 13:01 | |
*** nkrinner has joined #openstack-nova | 13:02 | |
openstackgerrit | Oleg Bondarev proposed openstack/nova: Live migration: wait for vif-plugged event on pre live migration https://review.openstack.org/246910 | 13:02 |
*** kristian__ has joined #openstack-nova | 13:04 | |
openstackgerrit | jichenjc proposed openstack/nova: Add more log when delete orphan node https://review.openstack.org/401274 | 13:06 |
*** tbachman has joined #openstack-nova | 13:06 | |
openstackgerrit | jichenjc proposed openstack/nova: Refactored query user_id code for quota-sets https://review.openstack.org/280165 | 13:06 |
jaypipes | sdague: I'm up! :) | 13:07 |
*** hfu has quit IRC | 13:07 | |
jaypipes | sdague: link for me? | 13:07 |
openstackgerrit | Hussain Chachuliya proposed openstack/nova: Deny volume swapping with non-bootable volume https://review.openstack.org/407346 | 13:08 |
*** kristian__ has quit IRC | 13:08 | |
jaypipes | cdent: yup, that's def a bug | 13:08 |
* cdent nods at jaypipes | 13:08 | |
cdent | thanks for confirming | 13:08 |
*** Amit_M has quit IRC | 13:09 | |
sdague | jaypipes: https://review.openstack.org/#/c/407450/ | 13:11 |
*** jheroux has joined #openstack-nova | 13:11 | |
*** tbachman has quit IRC | 13:11 | |
sdague | I asked markus_z to restructure the virtlogd patch so it's easier to understand in final form (though the 2 patches to get there are a little more confusing because the base code is so odd) | 13:12 |
sdague | you have a +2 on the other patch, so wanted to see if you are up for this one instead | 13:12 |
*** gouthamr has joined #openstack-nova | 13:12 | |
*** tbachman has joined #openstack-nova | 13:12 | |
*** edmondsw_ has joined #openstack-nova | 13:13 | |
*** fragatina has joined #openstack-nova | 13:15 | |
*** fragatin_ has joined #openstack-nova | 13:15 | |
wznoinsk | sfinucan, congrats | 13:16 |
*** edmondsw_ has quit IRC | 13:17 | |
openstackgerrit | jichenjc proposed openstack/nova: Enhancement comments on CountableResource https://review.openstack.org/402051 | 13:18 |
*** david_1 has joined #openstack-nova | 13:18 | |
*** sapcc-bot has joined #openstack-nova | 13:18 | |
*** tpatzig_ has joined #openstack-nova | 13:18 | |
*** dgonzalez_ has joined #openstack-nova | 13:18 | |
*** databus23_ has joined #openstack-nova | 13:18 | |
*** carthaca_ has joined #openstack-nova | 13:18 | |
*** fragatina has quit IRC | 13:19 | |
*** dave-mccowan has joined #openstack-nova | 13:19 | |
*** lamt has joined #openstack-nova | 13:19 | |
mdbooth | ftersin: I haven't read them yet because I'm still working on something else, but I will definitely address them before re-pushing. | 13:20 |
*** tpatzig_ has quit IRC | 13:20 | |
*** databus23_ has quit IRC | 13:20 | |
*** dgonzalez_ has quit IRC | 13:20 | |
*** carthaca_ has quit IRC | 13:20 | |
*** david_1 has quit IRC | 13:20 | |
*** shangxdy has quit IRC | 13:21 | |
mdbooth | ftersin: Ah, yes. I have read those, actually. I thought I'd responded, but apparently it got lost. I'm going to have to look at that closely. | 13:22 |
*** pradk has joined #openstack-nova | 13:22 | |
*** hfu has joined #openstack-nova | 13:22 | |
mdbooth | I have a sneaking suspicion that fallback is there because we start using it when we ditch the legacy cache() implementation. | 13:23 |
mdbooth | But I'll look in detail and update it if necessary before I re-push. | 13:24 |
*** huhaoran has quit IRC | 13:25 | |
*** yingjun has quit IRC | 13:25 | |
* mdbooth was trying to avoid pulling in the changes to remove create_images_and_backing in this series, but I realised I couldn't. | 13:25 | |
mdbooth | Could I have a really quick couple of +2 on this patch which adds 4 lines of comments to the libvirt driver: https://review.openstack.org/#/c/407470/ | 13:26 |
*** hamzy has quit IRC | 13:26 | |
*** mdnadeem has quit IRC | 13:27 | |
*** hfu has quit IRC | 13:29 | |
*** hfu has joined #openstack-nova | 13:29 | |
*** kristian__ has joined #openstack-nova | 13:32 | |
*** nkrinner has quit IRC | 13:33 | |
jaypipes | sdague, markus_z: done. | 13:33 |
*** code-R_ has joined #openstack-nova | 13:33 | |
tdurakov | mdbooth: hi, got question on that one | 13:34 |
markus_z | jaypipes: Thanks a ton! | 13:34 |
tdurakov | mdbooth: what is the reason we need auth_token? | 13:34 |
mdbooth | tdurakov: It's in the comment! | 13:34 |
*** kristian__ has quit IRC | 13:34 | |
mdbooth | Ah, you mean what's it subsequently used for? | 13:35 |
*** kristian__ has joined #openstack-nova | 13:35 | |
tdurakov | yes | 13:35 |
mdbooth | Fetching from glance | 13:35 |
tdurakov | mdbooth: could you please point me to? | 13:35 |
mdbooth | I could draw you a map | 13:35 |
tdurakov | lol | 13:35 |
mdbooth | However, it would take me a really long time :) | 13:35 |
markus_z | jaypipes: If you could check https://review.openstack.org/#/c/407120/ too, then bp/libvirt-virtlogd would be done. :) | 13:35 |
mdbooth | tdurakov: Sec... | 13:36 |
*** nkrinner has joined #openstack-nova | 13:36 | |
*** code-R has quit IRC | 13:36 | |
tdurakov | mdbooth: it sounds similar to https://review.openstack.org/#/c/407147/ so, I wonder could we fix nova glance communication instead | 13:37 |
*** acabot__ has joined #openstack-nova | 13:37 | |
mdbooth | tdurakov: Have a look in _create_images_and_backing | 13:37 |
*** hfu has quit IRC | 13:38 | |
mdbooth | If it has a local root disk, we fetch its backing file from glance | 13:38 |
mdbooth | _try_fetch_image_cache | 13:38 |
mdbooth | That needs auth_token | 13:38 |
mdbooth | But as this code isn't strictly necessary at all in _hard_reboot, it doesn't really matter if we don't do it | 13:39 |
mdbooth | Hence the if is ok | 13:39 |
mdbooth | But.... my patch just explains why auth_token might not be set, and therefore why that if statement is there | 13:39 |
* mdbooth had to dig that out of git, and having done that I thought it would be useful to put the information in a comment instead. | 13:40 | |
*** nkrinner has quit IRC | 13:43 | |
*** catintheroof has joined #openstack-nova | 13:43 | |
tdurakov | pkoniszewski: hi, are you around? | 13:43 |
pkoniszewski | tdurakov: yes | 13:43 |
*** swamireddy has quit IRC | 13:43 | |
tdurakov | pkoniszewski: don't we just wait for backports? | 13:43 |
tdurakov | for serial console hook | 13:44 |
pkoniszewski | that's an option | 13:44 |
*** tlian has joined #openstack-nova | 13:44 | |
markus_z | pkoniszewski: tdurakov: I would show my proposals on that during the l-m meeting. jroll pointed me to a promising direction: http://eavesdrop.openstack.org/irclogs/%23openstack-qa/%23openstack-qa.2016-12-06.log.html#t2016-12-06T12:44:16 | 13:46 |
*** huhaoran has joined #openstack-nova | 13:47 | |
openstackgerrit | Feodor Tersin proposed openstack/nova: Implement ScaleIO image backend https://review.openstack.org/407440 | 13:48 |
openstackgerrit | jichenjc proposed openstack/nova: Catch ProjectQuotaNotFound when do update_limit https://review.openstack.org/288153 | 13:48 |
tdurakov | pkoniszewski: for your proposal, we could check BRANCH_OVERRIDE DEVSTACK_GATE_GRENADE variables | 13:48 |
openstackgerrit | Feodor Tersin proposed openstack/nova: Implement ScaleIO image backend https://review.openstack.org/407440 | 13:49 |
tdurakov | markus_z: acked, let's discuss it on a meeting | 13:50 |
*** lucas-hungry is now known as lucasagomes | 13:50 | |
*** palexster has quit IRC | 13:51 | |
*** bvanhav_ has joined #openstack-nova | 13:51 | |
tdurakov | markus_z: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/devstack-gate.yaml#L1453 | 13:51 |
tdurakov | it won't work | 13:51 |
tdurakov | we explicilty set it to false for live-migration grenade job | 13:52 |
*** fragatin_ has quit IRC | 13:52 | |
*** bvanhav has quit IRC | 13:52 | |
markus_z | tdurakov: no worries, I have a list of proposal which don't have a dependency to the smoke tests. | 13:52 |
*** fragatina has joined #openstack-nova | 13:52 | |
tdurakov | markus_z: ok) | 13:52 |
*** hfu has joined #openstack-nova | 13:53 | |
*** nkrinner has joined #openstack-nova | 13:55 | |
*** baoli has joined #openstack-nova | 13:56 | |
openstackgerrit | Hussain Chachuliya proposed openstack/nova: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514 | 13:57 |
*** yingjun has joined #openstack-nova | 13:57 | |
*** hfu has quit IRC | 13:58 | |
*** baoli has quit IRC | 13:58 | |
*** baoli has joined #openstack-nova | 13:58 | |
*** swamireddy has joined #openstack-nova | 13:58 | |
*** esberglu has joined #openstack-nova | 13:58 | |
*** baoli has quit IRC | 13:59 | |
tdurakov | PaulMurray paul-carlton2 eliqiao shaohe_feng pkoniszewski yuntongjin mdbooth kashyap kmartin claudiub tobasco andreas_s abhishekk mrhillsman raj_singh wznoinsk siva_krishnan live migration meeting in 2 minutes on #openstack-meeting-3 | 13:59 |
*** baoli has joined #openstack-nova | 13:59 | |
*** palexster has joined #openstack-nova | 13:59 | |
openstackgerrit | Pawel Koniszewski proposed openstack/nova: Move migration_downtime_steps to libvirt/migration https://review.openstack.org/407517 | 14:00 |
openstackgerrit | Pawel Koniszewski proposed openstack/nova: Change exponential function to linear https://review.openstack.org/407518 | 14:00 |
*** prateek has joined #openstack-nova | 14:00 | |
*** lamt has quit IRC | 14:01 | |
*** lamt has joined #openstack-nova | 14:02 | |
*** Guest16197 is now known as mfisch | 14:03 | |
*** mfisch is now known as Guest73037 | 14:04 | |
*** daemontool has quit IRC | 14:04 | |
*** mdrabe has joined #openstack-nova | 14:07 | |
*** hfu has joined #openstack-nova | 14:08 | |
*** oanson has joined #openstack-nova | 14:08 | |
*** prateek has quit IRC | 14:09 | |
*** ihrachys has quit IRC | 14:10 | |
*** READ10 has joined #openstack-nova | 14:13 | |
*** ihrachys has joined #openstack-nova | 14:13 | |
*** ducttape_ has quit IRC | 14:15 | |
*** Guest73037 is now known as mfisch | 14:15 | |
*** prateek has joined #openstack-nova | 14:15 | |
*** mfisch is now known as Guest13281 | 14:16 | |
*** Guest13281 is now known as mfisch | 14:16 | |
*** mriedem has joined #openstack-nova | 14:16 | |
*** mfisch is now known as Guest40664 | 14:17 | |
*** Guest40664 is now known as mfisch | 14:17 | |
*** mfisch has quit IRC | 14:17 | |
*** mfisch has joined #openstack-nova | 14:17 | |
*** swamireddy has quit IRC | 14:17 | |
diana_clarke | mriedem, andreykurilin: It took me a while, but I'm pretty certain the python-novaclient gate failures are cells v2 related. If I revert the devstack change "Always setup cells v2", the gate is green again. | 14:19 |
andreykurilin | hm | 14:19 |
mriedem | diana_clarke: yeah i suspected that, i think the marker stuff might be messed up | 14:19 |
mriedem | i posted some patches to debug it, still need to get into those | 14:19 |
mriedem | https://review.openstack.org/#/c/407204/ | 14:20 |
diana_clarke | mriedem: yeah, I was following along looking at similar stuff at the same time | 14:20 |
mriedem | oops actually https://review.openstack.org/#/c/407205/ | 14:20 |
jaypipes | markus_z: reviewed https://review.openstack.org/#/c/407120. +0... I have a question in there. | 14:21 |
openstackgerrit | Timofey Durakov proposed openstack/nova: dnm to identify grenade job https://review.openstack.org/407532 | 14:21 |
markus_z | jaypipes: ok, will answer in a minute | 14:22 |
*** tbachman has quit IRC | 14:22 | |
*** tbachman_ has joined #openstack-nova | 14:22 | |
*** jamesdenton has joined #openstack-nova | 14:24 | |
openstackgerrit | jichenjc proposed openstack/nova: Add description for 2.9 microversion https://review.openstack.org/371481 | 14:25 |
markus_z | jaypipes: crap, that was a mistake, I'll push a new ps. thanks for spotting this! | 14:25 |
mriedem | jaypipes: while you're feeling saucy, can you also get this? https://review.openstack.org/#/c/402097/ it's the bottom change on the vhostuser series, which you were +W on the top | 14:27 |
*** hfu has quit IRC | 14:28 | |
*** hfu has joined #openstack-nova | 14:29 | |
*** tblakes has joined #openstack-nova | 14:30 | |
*** hshiina has quit IRC | 14:30 | |
mriedem | diana_clarke: hmm, so here is the marker not found with extra debug http://logs.openstack.org/05/407205/1/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/70528ca/console.html#_2016-12-06_00_42_18_938368 | 14:31 |
cdent | sfinucan: woot | 14:31 |
mriedem | ERROR (BadRequest): marker [dc89a764-5d12-4d17-8379-ccd110c81335] not found; current instance UUID: ['f4cc5ce3-cfcb-40bd-981c-6b1b4901c697', '6b2aae34-0308-4b92-af93-dd26d3569760', 'dc89a764-5d12-4d17-8379-ccd110c81335'] (HTTP 400) (Request-ID: req-2a81fd48-517b-4577-885a-0ad5a6dc8dec) | 14:31 |
mriedem | the marker uuid is in the list of available instances | 14:31 |
diana_clarke | mriedem: yup | 14:31 |
bauzas | cdent: quick question, can I GET /RPs?uuid=<foo>&uuid=<bar> ? | 14:31 |
bauzas | cdent: per the code, it seems no | 14:31 |
cdent | bauzas: looking, one sec | 14:31 |
diana_clarke | mriedem: strange, huh. perhaps a cells canary that something isn't quite right... | 14:32 |
*** swamireddy has joined #openstack-nova | 14:32 | |
jaypipes | mriedem: done. | 14:32 |
mriedem | diana_clarke: at first i suspected something was up with a race to query the instances table before the instance existed, but the build request would have existed, but this test is waiting for all 3 instances to be ACTIVE before it does the query | 14:32 |
mriedem | jaypipes: thanks | 14:32 |
mriedem | so if the instances are ACTIVE then they are in the DB | 14:33 |
jaypipes | markus_z: wait... so I spotted something sdague missed?! today is a good day. | 14:33 |
markus_z | jaypipes: I'm shocked too ;P | 14:33 |
cdent | bauzas: unclear, looking at master, it depends on the behavior of sd.query.filter(foo == bar) when bar is a list | 14:33 |
*** smatzek has quit IRC | 14:33 | |
jaypipes | markus_z: :) | 14:33 |
bauzas | cdent: well, I'm looking at the API | 14:34 |
sfinucan | cdent: I feel like I should have prepared a speech or something | 14:34 |
bauzas | cdent: given we're looping over a list of allowed filter keys | 14:34 |
jaypipes | sfinucan: lol | 14:34 |
jaypipes | sfinucan: just need to get your casual friday nick in order. | 14:34 |
sfinucan | jaypipes: heh. Working on it | 14:35 |
cdent | bauzas: it also depends on whether req.GET['uuid'] returns a scalar or a list | 14:35 |
cdent | bauzas: looks like it needs a test | 14:35 |
mriedem | diana_clarke: the other wrinkle is that all 3 instances have the same name, and we're filtering on name | 14:35 |
*** hfu has quit IRC | 14:35 | |
bauzas | cdent: ah good point, we don't check the scalar | 14:36 |
bauzas | cdent: I was supposing a scalar for that key, not a list | 14:36 |
mriedem | so filtering on name and passing a marker | 14:36 |
cdent | bauzas: the uuidutils.is_uuid_like call may cause it to bail out | 14:36 |
bauzas | cdent: like uuid=foo,bar | 14:36 |
bauzas | yeah :( | 14:36 |
cdent | so seems a few different bugs there? | 14:36 |
cdent | or at least unexpecte behaviors | 14:37 |
bauzas | not really, just that I'd love to see a way to ask for more than one uuid | 14:37 |
bauzas | that's not a bug honestly | 14:37 |
*** tbachman_ has quit IRC | 14:37 | |
bauzas | or the attribute could be named uuid*s* | 14:37 |
cdent | well I agree that it should be possible to do uuid=something&uuid=somethingelse | 14:37 |
cdent | that would be the normal way to do a list of the same query | 14:37 |
bauzas | that's what I'd prefer than just a uuid=<list> | 14:38 |
bauzas | exactly | 14:38 |
*** hfu has joined #openstack-nova | 14:38 | |
cdent | it should be doable, and I can't think of a reason why not to do it | 14:38 |
bauzas | for me, uuid is expecting a scalar, not a list but if you want more than one, then just add a new qs | 14:38 |
bauzas | okay | 14:38 |
tdurakov | pkoniszewski: for grenade job we tests only newton and master compatibility, right? | 14:39 |
cdent | bauzas: in related news, jaypipes remembered in the last 24 hours that we need a similar filter on the same url for aggregates | 14:39 |
pkoniszewski | tdurakov: it also tests mitaka<->newton | 14:40 |
cdent | one that states: resource providers that are in these aggregates (is that <- the proper translation jaypipes ? ) | 14:40 |
jaypipes | bauzas: right, otherwise we don't have a way of finding providers of shared resources. | 14:40 |
mriedem | 2016-12-06 00:42:17.880 27703 DEBUG nova.compute.api [req-2a81fd48-517b-4577-885a-0ad5a6dc8dec admin admin] Searching by: {'deleted': False, 'project_id': u'2ba71d0343c74b2aa8b70fcead4ad407', u'name': u'365f4533-46fd-4879-b75b-0d34c6d66585'} get_all /opt/stack/new/nova/nova/compute/api.py:2315 | 14:40 |
*** hfu has quit IRC | 14:40 | |
*** links has quit IRC | 14:40 | |
pkoniszewski | tdurakov: i mean, it is supposed to test this, but it fails due to wrong version of a script | 14:40 |
bauzas | jaypipes: cdent: I'm about to propose my REST API patch very soon, I'm just blocked by the above | 14:40 |
bauzas | I hope my proposal will be loved by all of you, guys | 14:41 |
bauzas | kind of a trade-off :) | 14:41 |
tdurakov | pkoniszewski: so, right now, I'd prefer to skip serial console for all grenade, not just for newton -> master | 14:41 |
jaypipes | bauzas: uuid=<uuid1>&uuid=<uuid2> is perfect. go with that. | 14:41 |
diana_clarke | mriedem: It's failing before the sorting and filtering though. It's unable to do a simple get by id here, I think: https://github.com/openstack/nova/blob/f61db221f31d9ba86f61c13a7d1c5a951654fdc0/nova/db/sqlalchemy/api.py#L2240 | 14:41 |
*** ducttape_ has joined #openstack-nova | 14:41 | |
bauzas | jaypipes: okay, I'll put in the same change for reviews, but I could split that | 14:41 |
tdurakov | pkoniszewski: smth like http://xsnippet.org/362141/ I've prepared patch, but want to be sure that it will work, so wating for https://review.openstack.org/#/c/407532/ | 14:42 |
mdbooth | Peeps, I use this kind of pattern in a bunch of places, and mostly it works really well: | 14:42 |
mdbooth | for name in ('kernel', 'ramdisk', 'disk', 'disk.eph0', 'disk.swap'): | 14:42 |
mdbooth | policyfixture.disks[name].create.assert_called_once_with() | 14:42 |
pkoniszewski | tdurakov: but will this change affect mitaka<->newton tests? I mean, we would need to backport hooks, wouldn't we? | 14:42 |
mdbooth | Problem is, when it fails you've got to annotate it to work out what it failed on | 14:42 |
mdbooth | Does anybody have a better pattern? | 14:43 |
jaypipes | bauzas: whatever you want, lemme know a link when you're ready for review. | 14:43 |
*** tbachman has joined #openstack-nova | 14:43 | |
*** kaisers has quit IRC | 14:43 | |
bauzas | jaypipes: hopefully in the next hour | 14:43 |
dansmith | mdbooth: assert_has_calls | 14:43 |
dansmith | vladikr: any progress on that failure yesterday? | 14:43 |
mdbooth | dansmith: It's multiple different objects | 14:43 |
dansmith | mdbooth: oh I see yeah | 14:44 |
tdurakov | pkoniszewski: will check that | 14:44 |
mdbooth | dansmith: Unless assert_has_calls lets you add additional output? | 14:44 |
dansmith | mdbooth: no, I just didn't read the example very clearly | 14:44 |
tdurakov | as for now stable branches tests only limited scenarios for live-migration job, simple block migration only | 14:44 |
lbeliveau_ | tdurakov: pkoniszewski: I'm getting serial console issues with devstack, is this related to the issue you are mentionning above http://paste.openstack.org/show/591544/ | 14:45 |
mriedem | mdbooth: you can name a mock | 14:45 |
mriedem | would that help with the failure output? | 14:45 |
mdbooth | mriedem: Yeah, if the name ended up in the output | 14:45 |
mriedem | mock.Mock(..., name='disks.kernel') | 14:45 |
pkoniszewski | lbeliveau_: no, it's unrelated to config options | 14:45 |
* mdbooth looks that up | 14:45 | |
vladikr | dansmith, not much, just got deeper into the mess.. on some tests , at some point the instance disappears right after one of the instance.save() | 14:46 |
mdbooth | mriedem: ooh. Trying it now. | 14:46 |
pkoniszewski | lbeliveau_: the issue we talk about is about acquiring/releasing serial console ports | 14:46 |
mriedem | "name: If the mock has a name then it will be used in the repr of the mock. This can be useful for debugging. The name is propagated to child mocks." | 14:46 |
dansmith | vladikr: hmm | 14:46 |
*** burt has joined #openstack-nova | 14:46 | |
vladikr | dansmith, on other tests it saves an empty pci device object, while I see that the constructed devices are correct | 14:46 |
*** lpetrut has joined #openstack-nova | 14:47 | |
*** kaisers has joined #openstack-nova | 14:47 | |
tdurakov | mriedem: hi, what is the resolution on this one https://review.openstack.org/#/c/407147/ ? | 14:48 |
mriedem | tdurakov: i don't have one yet | 14:48 |
mriedem | i'm busy with something else atm | 14:48 |
tdurakov | mriedem: acked | 14:48 |
vladikr | dansmith, also, some tests that failed in the ci are passing locally, but others are failing :/ | 14:48 |
edleafe | bauzas: cdent: If you ran GET /RPs?uuid=<foo>&uuid=<bar> it ANDs the filters together. It should be GET /RPs?uuid=in:foo,bar | 14:49 |
dansmith | vladikr: really.. just in that one test module or elsewhere? | 14:49 |
dansmith | vladikr: because I've since pushed up another revision where those are the only failures | 14:49 |
vladikr | dansmith, I've tested this module only | 14:49 |
cdent | edleafe: excellent point, good catch | 14:49 |
bauzas | edleafe: from an API perspective, duplicating the key is fine by me | 14:49 |
cdent | bauzas: edleafe's point is that api-wg guidelines have it that duplicated keys are ands | 14:49 |
tdurakov | pkoniszewski: are there any other issues with grenade job? | 14:50 |
cdent | so to avoid that you need to use a different form | 14:50 |
* bauzas facepalms | 14:50 | |
edleafe | bauzas: but two filters are ANDed together, so that will always return nothing | 14:50 |
pkoniszewski | tdurakov: haven't spotted any other issue | 14:50 |
mdbooth | mriedem: Unfortunate that doesn't make it into the output of an assertion failure :( | 14:50 |
tdurakov | pkoniszewski: https://review.openstack.org/#/c/407147/ | 14:50 |
mdbooth | Meh, nm | 14:50 |
bauzas | so, say I'm asking for fruit=apple&fruit=banana, it will AND ? | 14:50 |
bauzas | oh man | 14:50 |
* mdbooth makes coffee | 14:51 | |
mriedem | mdbooth: so it worked? | 14:51 |
mriedem | diana_clarke: ok i'm going to just push up another nova change with more debug logging | 14:51 |
bauzas | edleafe: is that a RFC or HTTP argument ? | 14:51 |
jaypipes | bauzas: edleafe is absolutely correct. that is what the API WG placed in the guidelines. sorry, man. | 14:51 |
diana_clarke | mriedem: I think it's cells & _build_instance_get query related... but I'm not yet sure how | 14:51 |
pkoniszewski | tdurakov: it looks like a rabbit issue http://logs.openstack.org/47/407147/2/check/gate-grenade-dsvm-neutron-multinode-live-migration-nv/a6a9a8b/logs/subnode-2/old/screen-n-cpu.txt.gz?level=TRACE | 14:51 |
cdent | bauzas: it's in here: http://specs.openstack.org/openstack/api-wg/guidelines/pagination_filter_sort.html | 14:51 |
bauzas | I always expected as a customer that I'm providing two attributes with the same name, it will OR them | 14:51 |
edleafe | bauzas: cdent: https://github.com/openstack/api-wg/blob/master/guidelines/pagination_filter_sort.rst | 14:52 |
cdent | edleafe: buy me a coke | 14:52 |
edleafe | "Multiple filters result in an implicit AND, so in our example /app/items?foo=buzz&baz=quux would provide no results." | 14:52 |
* bauzas is desesperated by that pile of news | 14:52 | |
tdurakov | pkoniszewski: the reason I'm asking: https://review.openstack.org/#/c/400878/ https://review.openstack.org/#/c/407120/ | 14:52 |
cdent | bauzas: it may be worthwhile to note that the precedent for this is coming out of nova, that guideline is one of the things that was copied in from the existing apis | 14:53 |
edleafe | bauzas: is there any reason that you can't use in:? | 14:53 |
*** kashyap has quit IRC | 14:53 | |
bauzas | edleafe: that looks pretty special, but okay | 14:53 |
*** pbandark has quit IRC | 14:53 | |
bauzas | I can do that to respect the WG :-) | 14:53 |
jaypipes | edleafe: I can understand where bauzas is coming from actually. there's really no point in the ability to say "uuid = A AND uuid = B". it's an unresolvable condition. | 14:54 |
pkoniszewski | tdurakov: that's weird | 14:54 |
tdurakov | jaypipes: hi, if you have some time, could you please take a look https://review.openstack.org/#/c/407147/ | 14:54 |
pkoniszewski | tdurakov: http://logs.openstack.org/78/400878/7/check/gate-grenade-dsvm-neutron-multinode-live-migration-nv/272a2c8/logs/subnode-2/old/screen-n-cpu.txt.gz?level=TRACE | 14:54 |
edleafe | bauzas: it's not respecting the WG as much as much as respecting OpenStack | 14:54 |
*** lpetrut has quit IRC | 14:54 | |
bauzas | I'm always having hard time to respect myself even | 14:55 |
tdurakov | pkoniszewski: yeah, looks like constant issue( | 14:55 |
pkoniszewski | tdurakov: neutron agent is also failing on rabbit http://logs.openstack.org/78/400878/7/check/gate-grenade-dsvm-neutron-multinode-live-migration-nv/272a2c8/logs/subnode-2/old/screen-q-agt.txt.gz?level=TRACE | 14:55 |
tdurakov | need to figure out | 14:55 |
bauzas | but I'm missing snow, that's probably why | 14:55 |
edleafe | jaypipes: well, yeah, that would make no sense, so use the in: operator for that case | 14:55 |
cdent | bauza, edleafe, yeah, in fact if we were writing the filtering, sorting and pagination guidelines from scratch, rather than precedent they would likely be _much_ different | 14:55 |
edleafe | jaypipes: What makes no sense at all is having some APIs where filters are ANDed, and some where filters are ORed. | 14:55 |
pkoniszewski | tdurakov: and always subnode-2 is affected | 14:55 |
bauzas | cdent: edleafe: that's where I think we name that 'guidelines' and not rules :) | 14:56 |
jaypipes | edleafe: there's never a case where an AND'd condition on a single attribute would ever make sense. But, hey, the main point is for consistency here. | 14:56 |
bauzas | anyway, let's put in: | 14:56 |
dansmith | vladikr: could it be related to the issue I fixed where the compute service was being restarted? | 14:56 |
dansmith | vladikr: like maybe that was covering something up in the test that was broken? | 14:57 |
edleafe | bauzas: sure. The idea was that no one should have to change existing code, but all new development should follow the guidelines | 14:57 |
bauzas | honestly, I don't want to discuss yet again 4 weeks for that point :) | 14:57 |
jaypipes | agreed. | 14:57 |
bauzas | so, given the API WG is saying "in:", let's write that "in:" | 14:57 |
*** tbachman has quit IRC | 14:57 | |
tdurakov | pkoniszewski: https://review.openstack.org/#/c/407120/ - there is a similar job that passes gate-grenade-dsvm-neutron-multinode-ubuntu-xenial | 14:58 |
jaypipes | bauzas: ++ | 14:58 |
bauzas | anyway, edleafe, thanks for pointing that out now :) | 14:58 |
vladikr | dansmith, it definitely feels like a failure that is being masked.. | 14:58 |
edleafe | bauzas: :) | 14:59 |
tdurakov | pkoniszewski: and both of them suffer from messaging timeout btw | 14:59 |
*** sudipto has joined #openstack-nova | 14:59 | |
*** sudipto_ has joined #openstack-nova | 14:59 | |
vladikr | dansmith, maybe something related to saving pci devices.. as these are the only tests that are failing .. | 14:59 |
*** psachin has quit IRC | 15:00 | |
pkoniszewski | tdurakov: yeah, http://logs.openstack.org/20/407120/2/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/7b7054a/logs/subnode-2/old/screen-q-agt.txt.gz?level=TRACE | 15:00 |
pkoniszewski | tdurakov: so not only lm grenade job is affected | 15:00 |
*** cdelatte has quit IRC | 15:00 | |
jaypipes | tdurakov: done | 15:00 |
dansmith | vladikr: so, if I apply the start-compute-only-once change to master, I get a failure | 15:00 |
*** siva_krish has joined #openstack-nova | 15:00 | |
*** anush has joined #openstack-nova | 15:00 | |
dansmith | vladikr: which I'm pretty sure isn't expected, right? | 15:00 |
dansmith | vladikr: because it's supposed to be testing with one compute service to make sure that the second instance doesn't get VFs from a used PF right? | 15:01 |
*** danpawlik has quit IRC | 15:01 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: DNM: debug novaclient limit/marker failure https://review.openstack.org/407204 | 15:02 |
*** mlavalle has joined #openstack-nova | 15:02 | |
*** smatzek has joined #openstack-nova | 15:03 | |
*** cdelatte has joined #openstack-nova | 15:03 | |
vladikr | dansmith, yea, there shouldn't be two instances running. are you referring to "if not self.compute_started:" that you've added in the test? | 15:04 |
*** hfu has joined #openstack-nova | 15:04 | |
*** hamzy has joined #openstack-nova | 15:04 | |
dansmith | vladikr: yep | 15:04 |
dansmith | vladikr: without that the test calls start_service(compute) more than once | 15:04 |
dansmith | which it should not do | 15:04 |
dansmith | regardless of the base test changes | 15:04 |
vladikr | dansmith, yes | 15:05 |
mriedem | diana_clarke: what's confusing is the same request is hitting ComputeAPI.get_all() twice it looks like | 15:06 |
mriedem | http://logs.openstack.org/05/407205/1/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/70528ca/logs/screen-n-api.txt.gz#_2016-12-06_00_42_17_880 | 15:06 |
mriedem | http://logs.openstack.org/05/407205/1/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/70528ca/logs/screen-n-api.txt.gz#_2016-12-06_00_42_17_955 | 15:06 |
* diana_clarke looks | 15:06 | |
mriedem | same request id | 15:06 |
mriedem | same 'searching by' log line | 15:06 |
mriedem | but that should only be once per request | 15:06 |
*** penick has quit IRC | 15:07 | |
*** tbachman has joined #openstack-nova | 15:07 | |
mriedem | i guess the 2nd query could be because of my debug call w/o the marker | 15:08 |
mriedem | but the order seems messed up | 15:08 |
mriedem | i.e. i have a 2nd get_all query here https://review.openstack.org/#/c/407204/3/nova/api/openstack/compute/servers.py | 15:08 |
mriedem | actually nvm i'm just dumb | 15:08 |
mriedem | that's the right order | 15:08 |
mriedem | my new patch set should make this more clear which query has the marker | 15:08 |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: virtlogd: use virtlogd for char devices https://review.openstack.org/407450 | 15:09 |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: create consoles in an understandable/extensible way https://review.openstack.org/407120 | 15:09 |
markus_z | sdague: jaypipes: ^ If you could have a look again please? | 15:10 |
lbeliveau_ | guys (and gals), I'm running into this issue with latest devstack (nova-serialproxy), I'll investigate it, but was wondering if anybody was working on it or seen it http://paste.openstack.org/show/591544/ | 15:11 |
*** larainema_ is now known as larainema | 15:11 | |
*** lbeliveau_ is now known as lbeliveau | 15:11 | |
sdague | markus_z: lgtm. And good catches by jay on those conditionals | 15:12 |
*** mjura_ has joined #openstack-nova | 15:13 | |
*** nandal has quit IRC | 15:13 | |
*** cfriesen has joined #openstack-nova | 15:13 | |
*** prateek has quit IRC | 15:14 | |
vladikr | dansmith, one of the test is failing because it's trying to save an empty pci device, however, the allocated pci object (on a second line) is complete https://paste.fedoraproject.org/500472/14810371/ | 15:14 |
*** mjura has quit IRC | 15:15 | |
dansmith | vladikr: which test is that? | 15:15 |
tdurakov | jaypipes 2c about mox, there are 2 classes: TestNeutronv2 and TestNeutronv2WithMock, so fix is merged, I'd be happy to rewrite these to mock as well | 15:15 |
*** yingjun has quit IRC | 15:15 | |
vladikr | dansmith, nova.tests.functional.libvirt.test_pci_sriov_servers.SRIOVServersTest.test_create_server_with_pci_dev_and_numa | 15:15 |
*** yingjun has joined #openstack-nova | 15:16 | |
dansmith | vladikr: that one passes for me | 15:16 |
vladikr | dansmith, weird .. it failed in the CI | 15:17 |
vladikr | 2016-12-05 23:56:01.553491 | {7} nova.tests.functional.libvirt.test_pci_sriov_servers.SRIOVServersTest.test_create_server_with_pci_dev_and_numa [4.205866s] ... FAILED | 15:17 |
dansmith | vladikr: strange | 15:18 |
dansmith | vladikr: I wonder if it might be better to fix the test before even trying to make it work on top of this base test change/ | 15:18 |
mriedem | tdurakov: unwinding the mox in TestNeutronv2 is probably not the best use of your time or reviewer time atm | 15:18 |
*** tbachman has quit IRC | 15:18 | |
mriedem | as it's a terrible monster | 15:18 |
jaypipes | tdurakov: nah, it's cool :) | 15:18 |
*** tbachman has joined #openstack-nova | 15:19 | |
tdurakov | mriedem: right, it's a monster | 15:19 |
*** erlon-airlong has quit IRC | 15:20 | |
openstackgerrit | Chris Dent proposed openstack/nova-specs: Update the generic resource pools to reflect reality https://review.openstack.org/407562 | 15:20 |
dansmith | vladikr: it didn't fail with that constraint issue though right? | 15:20 |
*** yingjun has quit IRC | 15:20 | |
*** erlon-airlong has joined #openstack-nova | 15:20 | |
*** eharney has joined #openstack-nova | 15:20 | |
jaypipes | markus_z: done. | 15:21 |
*** mjura_ has quit IRC | 15:21 | |
vladikr | dansmith, no, and the test was working fine before the cells change .. | 15:21 |
*** jwcroppe has joined #openstack-nova | 15:21 | |
dansmith | vladikr: sure, but the test is doing bad stuff already (starting two computes with the same hostname) and fails if you don't do that even on master | 15:22 |
dansmith | vladikr: which makes me suspect its correctness :) | 15:22 |
markus_z | jaypipes: awesome, thanks again! | 15:23 |
openstackgerrit | Oleg Bondarev proposed openstack/nova: Live migration: wait for vif-plugged event on pre live migration https://review.openstack.org/246910 | 15:23 |
vladikr | dansmith, well, I can push a quick fix for the two computes issue, but I doubt it was related to this issue.. the test it self doesn't do anything different from the other functional tests | 15:24 |
mriedem | jaypipes: forgot to +W https://review.openstack.org/#/c/407120/ | 15:24 |
vladikr | dansmith, but sure, I'll submit a patch is a sec | 15:24 |
dansmith | vladikr: you should push that fix, yeah | 15:24 |
dansmith | vladikr: other functional tests don't start two computes with the same hostname :) | 15:25 |
jaypipes | mriedem: sdorry, done. | 15:26 |
*** diga has quit IRC | 15:27 | |
*** rama_y has joined #openstack-nova | 15:27 | |
openstackgerrit | Julian proposed openstack/nova: Transform instance.snapshot notifications https://review.openstack.org/330128 | 15:29 |
*** mjura_ has joined #openstack-nova | 15:33 | |
*** kfarr has joined #openstack-nova | 15:35 | |
*** anush has quit IRC | 15:37 | |
*** anush has joined #openstack-nova | 15:38 | |
tdurakov | markus_z: ping | 15:39 |
markus_z | tdurakov: pong | 15:39 |
*** hongbin has joined #openstack-nova | 15:39 | |
tdurakov | markus_z: I couldn't see any output for grenade job at all http://logs.openstack.org/32/407532/1/check/gate-grenade-dsvm-neutron-multinode-live-migration-nv/33c4319/console.html#_2016-12-06_14_57_06_602346 | 15:39 |
tdurakov | am I missing smth? | 15:39 |
markus_z | tdurakov: hm, I only looked at the top: http://logs.openstack.org/32/407532/1/check/gate-grenade-dsvm-neutron-multinode-live-migration-nv/33c4319/console.html#_2016-12-06_14_22_13_270934 | 15:41 |
*** tblakes has quit IRC | 15:41 | |
tdurakov | markus_z: to compare http://logs.openstack.org/32/407532/1/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/9ab7d67/console.html#_2016-12-06_14_46_39_775842 | 15:41 |
markus_z | tdurakov: Is it possible that empty lines get captured and not printed? | 15:42 |
tdurakov | for l-m job it's outputs smth | 15:42 |
markus_z | ah, yeah, forget what I said, there is the empty line too | 15:42 |
*** kiran-r has joined #openstack-nova | 15:43 | |
*** rama_y has quit IRC | 15:43 | |
* markus_z tries to figure that out | 15:44 | |
*** mkoderer has quit IRC | 15:46 | |
*** mkoderer has joined #openstack-nova | 15:47 | |
markus_z | tdurakov: mind if I add a patchset to that? | 15:48 |
tdurakov | sure | 15:49 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: tests: avoid starting compute service twice in sriov functional test https://review.openstack.org/407572 | 15:50 |
dansmith | vladikr: are those two changes related? | 15:52 |
*** fragatina has quit IRC | 15:53 | |
dansmith | vladikr: also, that patch for me: {0} nova.tests.functional.libvirt.test_pci_sriov_servers.SRIOVServersTest.test_create_server_with_PF_no_VF [5.241744s] ... FAILED | 15:53 |
vladikr | dansmith, no, just wanted to get this test to be as similar as the other numa test. the other thing is very trivial | 15:53 |
vladikr | dansmith, I see.. this one fails for me, as in the CI.. | 15:54 |
*** mdnadeem has joined #openstack-nova | 15:54 | |
dansmith | vladikr: that's what I was trying to say before.. just making that change causes a fail on master | 15:54 |
vladikr | dansmith, does anything fails for you in that SRIOVServersTest test? | 15:54 |
dansmith | vladikr: on the first run that's the only fail I saw, let me try in parallel | 15:55 |
dansmith | vladikr: ah, they pass in parallel, so something else is going on :) | 15:55 |
vladikr | yea.. | 15:56 |
*** armax has joined #openstack-nova | 15:56 | |
sfinucan | vladikr: Missing context, but assuming you're working on functional tests, have you seen this? https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:sriov_numa_func_test | 15:56 |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: dnm to identify grenade job https://review.openstack.org/407532 | 15:56 |
*** mtanino has joined #openstack-nova | 15:58 | |
vladikr | sfinucan, sure, I've commented on https://review.openstack.org/#/c/399653 :) | 15:58 |
sfinucan | vladikr: Cool - just checking :) | 15:59 |
vladikr | sfinucan, I'll review it again today | 15:59 |
vladikr | :) | 15:59 |
sfinucan | ...so as to avoid overlap etc. | 15:59 |
*** rama_y has joined #openstack-nova | 16:01 | |
dansmith | sfinucan: you're distracting vladikr from caring about *my* problem :) | 16:01 |
ildikov | Hi Nova cores! If any of you would have some time and check my attempt to remove 'check_attach' from the code base I very much appreciated it: https://review.openstack.org/#/c/335358/ :) | 16:02 |
sfinucan | dansmith: Barely in the door and already messing with other cores. Excellent :) | 16:02 |
vladikr | dansmith, I think all of the tests are working on master. It only fails in https://review.openstack.org/#/c/406380 | 16:03 |
dansmith | vladikr: but not with your compute-once fix | 16:03 |
dansmith | vladikr: but even still, the base test change really shouldn't be affecting those tests ... | 16:04 |
*** rama_y has quit IRC | 16:04 | |
*** ayoung-afk is now known as ayoung | 16:05 | |
*** HenryG has left #openstack-nova | 16:06 | |
*** tblakes has joined #openstack-nova | 16:06 | |
vladikr | dansmith, locally all tests are passing with the compute-once fix - will wait for the CI, and will look at the instance.save() with the /406380 change | 16:06 |
dansmith | vladikr: have you tried running with --concurrency=1? | 16:07 |
dansmith | vladikr: if it fails with some test arrangements and not others, that means there are interactions and it's a timebomb in CI that will eventually go off | 16:08 |
markus_z | For awareness: https://bugs.launchpad.net/neutron/+bug/1647431 | 16:08 |
openstack | Launchpad bug 1647431 in neutron "grenade job times out on Xenial" [Critical,Confirmed] | 16:08 |
vladikr | dansmith, I see.. I think I did earlier.. let me try again now | 16:08 |
markus_z | Seems to hit "gate-grenade-dsvm-neutron-multinode-ubuntu-xenial" | 16:08 |
dansmith | vladikr: fails 100% of the time for me with --concurrency=1 | 16:08 |
*** sridharg has quit IRC | 16:08 | |
dansmith | vladikr: and the worrisome part is that it fails because it succeeds in starting the instance that should be failing | 16:09 |
dansmith | vladikr: it also fails if you run that test by itself | 16:09 |
vladikr | dansmith, got it.. this is the test_create_server_with_VF_no_PF test? | 16:10 |
dansmith | vladikr: yeah | 16:10 |
vladikr | ok | 16:10 |
mriedem | markus_z: hmm, not terrible deterministic http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22timeout%2030%20sh%20-c%20'while%20openstack%20server%20show%20cinder_server1%5C%22%20AND%20tags%3A%5C%22grenade.sh.txt%5C%22&from=7d | 16:11 |
mriedem | *terribly | 16:11 |
*** ihrachys has quit IRC | 16:11 | |
markus_z | mriedem: yeah, the bug reporter noticed that too | 16:12 |
openstackgerrit | Ed Leafe proposed openstack/nova: Corrects the type of a base64 encoded string https://review.openstack.org/385686 | 16:12 |
macsz | ls | 16:12 |
*** kevinz has quit IRC | 16:12 | |
macsz | oops, sorry :D | 16:12 |
macsz | lucky it was just ``ls`` :D | 16:13 |
mriedem | looks like it's mostly only on multinode jobs | 16:13 |
*** JoseMello has quit IRC | 16:13 | |
markus_z | mriedem: looks like this started on Nov 27th | 16:14 |
*** catinthe_ has joined #openstack-nova | 16:14 | |
*** yamahata has joined #openstack-nova | 16:14 | |
*** owalsh has quit IRC | 16:14 | |
markus_z | maybe grenade jobs only? | 16:14 |
markus_z | whops, nevermind, you had that in the ELK query | 16:15 |
mriedem | markus_z: hmm, this looks pretty gd awful http://logs.openstack.org/40/402140/7/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/ad0cf41/logs/new/screen-c-vol.txt.gz?level=TRACE | 16:15 |
*** bkopilov has joined #openstack-nova | 16:16 | |
markus_z | Everything red and orange, it's not *that* bad ... :/ | 16:16 |
*** daemontool has joined #openstack-nova | 16:16 | |
*** catintheroof has quit IRC | 16:16 | |
*** code-R_ has quit IRC | 16:17 | |
mriedem | that started around 12/2 | 16:17 |
*** rcernin has quit IRC | 16:17 | |
*** catinthe_ has quit IRC | 16:18 | |
*** annegentle has joined #openstack-nova | 16:18 | |
*** nkrinner is now known as nkrinner_afk | 16:18 | |
*** pcaruana has quit IRC | 16:19 | |
*** rmart04 has quit IRC | 16:19 | |
*** gszasz has quit IRC | 16:19 | |
* mdbooth would like to re-pimp this change, which simply adds 4 lines of comments to the libvirt driver: https://review.openstack.org/#/c/407470/ | 16:20 | |
*** acabot_ has quit IRC | 16:20 | |
*** acabot__ has quit IRC | 16:20 | |
*** moshele has quit IRC | 16:22 | |
*** oanson has quit IRC | 16:22 | |
*** cdent has quit IRC | 16:22 | |
*** zul has quit IRC | 16:22 | |
*** sapcc-bot1 has joined #openstack-nova | 16:25 | |
*** zul has joined #openstack-nova | 16:25 | |
*** sapcc-bot has quit IRC | 16:25 | |
*** mvk has quit IRC | 16:26 | |
*** cdent has joined #openstack-nova | 16:26 | |
vladikr | dansmith, that's crazy.. it all passes, checked on two different envs. https://paste.fedoraproject.org/500517/10413071/ | 16:26 |
*** bvanhav_ has quit IRC | 16:27 | |
dansmith | vladikr: wrong test | 16:28 |
dansmith | PF_no_VF | 16:28 |
dansmith | sorry I guess I confirmed the opposite above, but I pasted the full name of the test at some point | 16:28 |
vladikr | dansmith, I've tested all of them.. all seems to pass, will double check in a sec | 16:29 |
vladikr | dansmith, https://paste.fedoraproject.org/500521/41859148/ | 16:31 |
markus_z | mriedem: Version 3.1 is already 2 months old: https://github.com/openstack/cinder/commit/154404f5ddfe73f843e75196ed8e950d9edf9ba3 | 16:32 |
*** baoli has quit IRC | 16:32 | |
markus_z | I'm wondering if it is something outside of cinder | 16:32 |
mriedem | markus_z: https://bugs.launchpad.net/cinder/+bug/1647789 | 16:33 |
openstack | Launchpad bug 1647789 in Cinder "RPCVersionCapError: Requested message version, 3.1 is incompatible. It needs to be equal in major version and less than or equal in minor version as the specified version cap 3.0. in c-vol in grenade jobs" [Undecided,New] | 16:33 |
mriedem | red herring | 16:33 |
markus_z | ah crap, I looked at the wrong place | 16:35 |
openstackgerrit | Christopher Martin proposed openstack/nova: Bugfix: Changes to Instance Soft Delete https://review.openstack.org/386756 | 16:38 |
*** unicell has quit IRC | 16:39 | |
dansmith | vladikr: try this: .tox/functional/bin/python -m subunit.run /home/dan/nova/nova/tests/functional/libvirt/test_pci_sriov_servers.py | /home/dan/nova/.tox/py27/bin/subunit-trace | 16:39 |
vladikr | it finally failed :) | 16:39 |
dansmith | vladikr: with your home dir | 16:39 |
dansmith | ah, okay | 16:39 |
dansmith | vladikr: it's a similar failure to what I see with the test base, but on more tests, which is why I think it might be related | 16:40 |
*** kiran-r has quit IRC | 16:40 | |
vladikr | dansmith, got it, ok, looking into it | 16:40 |
dansmith | vladikr: thanks :) | 16:40 |
mriedem | markus_z: it's weird, the server delete should be starting here http://logs.openstack.org/40/402140/7/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/ad0cf41/logs/grenade.sh.txt.gz#_2016-12-05_09_07_08_778 | 16:41 |
mriedem | the server uuid is 7e8aad9d-50e1-4b6c-8418-7395a035256f | 16:41 |
mriedem | the server delete should be around 09:07:08.778 | 16:41 |
mriedem | but i don't see any DELETE request for that server around that time in the api logs http://logs.openstack.org/40/402140/7/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/ad0cf41/logs/new/screen-n-api.txt.gz | 16:42 |
mriedem | there is a POST from neutron for an external network changed event because the floating IP was removed from the server | 16:42 |
mriedem | ah here we go http://logs.openstack.org/40/402140/7/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/ad0cf41/logs/new/screen-n-api.txt.gz#_2016-12-05_09_07_10_734 | 16:44 |
*** bvanhav_ has joined #openstack-nova | 16:44 | |
*** zenoway has quit IRC | 16:45 | |
*** kaisers_ has joined #openstack-nova | 16:45 | |
openstackgerrit | Ludovic Beliveau proposed openstack/nova: Fix nova-serialproxy when registering cli options https://review.openstack.org/407602 | 16:45 |
openstackgerrit | melanie witt proposed openstack/nova: Pass context as kwarg instead of positional arg to get_engine https://review.openstack.org/407603 | 16:45 |
*** zenoway has joined #openstack-nova | 16:46 | |
lbeliveau | sfinucan: ^ can you please have a look since you are well versed with config options :) | 16:46 |
*** belmoreira has quit IRC | 16:47 | |
mriedem | no failures in the n-cpu logs | 16:47 |
*** stvnoyes has joined #openstack-nova | 16:48 | |
*** ihrachys has joined #openstack-nova | 16:48 | |
*** tbachman has quit IRC | 16:48 | |
mriedem | markus_z: last thing n-cpu does is post to cinder http://logs.openstack.org/40/402140/7/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/ad0cf41/logs/new/screen-n-cpu.txt.gz#_2016-12-05_09_07_13_263 | 16:49 |
mriedem | for os-terminate_connection | 16:49 |
tdurakov | markus_z, mriedem what's the problem you are working on? | 16:50 |
*** tjones has joined #openstack-nova | 16:50 | |
*** zenoway has quit IRC | 16:50 | |
mriedem | tdurakov: https://bugs.launchpad.net/neutron/+bug/1647431 | 16:50 |
openstack | Launchpad bug 1647431 in neutron "multinode neutron grenade job times out on Xenial" [Critical,Confirmed] | 16:50 |
*** kaisers_ has quit IRC | 16:50 | |
tdurakov | mriedem: acked, will take a look too | 16:51 |
markus_z | mriedem: I don't get why the cinder-scheduler automatically chooses version cap 3.0 | 16:51 |
*** rama_y has joined #openstack-nova | 16:53 | |
mriedem | markus_z: dulek is looking into that too | 16:53 |
markus_z | OK, cool, then I call it a day | 16:53 |
markus_z | It's getting late here | 16:53 |
mriedem | o7 | 16:53 |
*** sahid has quit IRC | 16:54 | |
*** markus_z has quit IRC | 16:54 | |
mriedem | weird, so the volume terminate_connection rpc call to c-vol happens from c-api here http://logs.openstack.org/40/402140/7/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/ad0cf41/logs/new/screen-c-api.txt.gz#_2016-12-05_09_07_13_391 | 16:54 |
mriedem | that request never shows up in c-vol | 16:55 |
mriedem | req-42b76e49-9bec-4ac5-b457-938bc82b3649 | 16:55 |
mriedem | there is a crazy amount of logging in c-vol | 16:57 |
*** mjura_ has quit IRC | 17:00 | |
*** STEWIE925 has joined #openstack-nova | 17:01 | |
bauzas | edleafe: I'm getting mad at writing the regex that would match in:foo,bar :p | 17:01 |
mriedem | tdurakov: at this point the instance is stopped http://logs.openstack.org/40/402140/7/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/ad0cf41/logs/new/screen-n-cpu.txt.gz#_2016-12-05_09_07_25_978 | 17:01 |
sdague | bauzas: I expect you can't do that with just a regex | 17:02 |
*** roeyc has quit IRC | 17:02 | |
bauzas | sdague: supposingly, I wrote re.findall('^in:(?:([^,]+),?)+, mystr) | 17:02 |
bauzas | but anyway, I'll just split the string without using re... | 17:03 |
bauzas | that's just sad | 17:03 |
mriedem | tdurakov: this is really odd, we seem to just stop in _shutdown_instance in the compute manager | 17:03 |
mriedem | we post the os-terminate_connection to cinder | 17:03 |
sdague | yeh, I would just do that. Regex complexity for that just gets weird | 17:03 |
mriedem | but we never make the os-detach call to cinder | 17:03 |
*** efoley has joined #openstack-nova | 17:03 | |
sdague | whereas if you do a couple of splits first, you can then easily validate the components | 17:03 |
dulek | mriedem: Be advised that you have two c-vols - on primary and subnode. | 17:03 |
*** catintheroof has joined #openstack-nova | 17:03 | |
*** catintheroof has quit IRC | 17:03 | |
*** catintheroof has joined #openstack-nova | 17:04 | |
mriedem | ah | 17:04 |
edleafe | bauzas: I'm knee-deep in some other stuff, but if you send me what you have I can take a look at it | 17:04 |
bauzas | edleafe: sdague: nah, no worries, I'll just write a .split() stuff :( | 17:04 |
tdurakov | mriedem: have you tried to find a root cause patch? | 17:05 |
mriedem | dulek: the volume, 6645dbcf-3a6e-4444-8afc-16df7fd70267, is on the 'new' c-vol though, it doesn't show up in the logs for the subnode | 17:05 |
dulek | mriedem: Hm… | 17:05 |
*** eharney has quit IRC | 17:05 | |
dulek | mriedem: So is this even related to the error I'm fixing? | 17:06 |
mriedem | tdurakov: no because i don't know what's causing the server to not get deleted | 17:06 |
mriedem | dulek: no | 17:06 |
mriedem | tdurakov: we seem to just deadlock or something | 17:06 |
*** voelzmo has quit IRC | 17:07 | |
tdurakov | right, otherwise it will be 100% reproducible | 17:07 |
sfinucan | lbeliveau: Sure. Checking | 17:07 |
dulek | mriedem: I've observed something like that when I've messed up with oslo.messaging topics in my patch. | 17:07 |
mriedem | we're this far in n-cpu to call os-terminate_connection in cinder http://logs.openstack.org/40/402140/7/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/ad0cf41/logs/new/screen-n-cpu.txt.gz#_2016-12-05_09_07_13_263 | 17:08 |
dulek | mriedem: Actually exactly like that, it was terminate_connection. | 17:08 |
mriedem | i can see c-api call c-vol | 17:08 |
mriedem | right here http://logs.openstack.org/40/402140/7/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/ad0cf41/logs/new/screen-c-api.txt.gz#_2016-12-05_09_07_13_391 | 17:08 |
mriedem | then we hang | 17:08 |
mriedem | and die | 17:08 |
dulek | I'll look into it once I'll push patch for bug 1647789. | 17:09 |
openstack | bug 1647789 in Cinder "RPCVersionCapError: Requested message version, 3.1 is incompatible. It needs to be equal in major version and less than or equal in minor version as the specified version cap 3.0. in c-vol in grenade jobs" [High,In progress] https://launchpad.net/bugs/1647789 - Assigned to Michal Dulko (michal-dulko-f) | 17:09 |
*** andreas_s has quit IRC | 17:09 | |
*** gjayavelu has joined #openstack-nova | 17:10 | |
*** Swami has joined #openstack-nova | 17:10 | |
*** dinobot_ has joined #openstack-nova | 17:10 | |
lbeliveau | sfinucan: btw congrats on your "promotion" to core, well deserved :) | 17:11 |
*** rama_y has quit IRC | 17:11 | |
openstackgerrit | Ed Leafe proposed openstack/nova: Do not post allocations that are zero https://review.openstack.org/407180 | 17:11 |
mriedem | so the c-api rpc call to c-vol for terminate_connection uses msg id dd4e70fdda9a4234a4377adbb24f9831 | 17:11 |
mriedem | so i guess we can check the rabbit logs for that? | 17:11 |
dulek | mriedem: Not much of them in what's published. | 17:12 |
*** gjayavelu has quit IRC | 17:12 | |
*** gjayavelu has joined #openstack-nova | 17:12 | |
*** anush has quit IRC | 17:13 | |
*** matrohon has quit IRC | 17:14 | |
mriedem | dulek: yeah there is nothing in the rabbitmq logs | 17:14 |
dulek | It looks like there's a lot of guru meditation reports in new primary c-vol logs? | 17:15 |
*** anush has joined #openstack-nova | 17:15 | |
*** prateek has joined #openstack-nova | 17:15 | |
dulek | RPCVersionCapError is probably uncatched and kills the threads. | 17:16 |
mriedem | dulek: https://github.com/openstack-dev/devstack/commit/1f739c867a7fe6d54ca061c6352eed12873db5cc | 17:16 |
dulek | But we still shouldn't lose messages, right? | 17:16 |
mriedem | idk | 17:16 |
dulek | mriedem: Oh, okay, I see now. | 17:17 |
dulek | mriedem: c-vol has this notify_service_capabilities in initialization. So it fails to spawn *any* thread that could be able to process *any* message. | 17:18 |
*** tbachman has joined #openstack-nova | 17:18 | |
dulek | mriedem: So this terminate_connection is first message that gets to new c-vol after upgrade. It fails, so everything fails. | 17:18 |
dulek | mriedem: This would mean my commit (https://review.openstack.org/407612) fixes that. | 17:19 |
*** eharney has joined #openstack-nova | 17:20 | |
*** irenab_ has joined #openstack-nova | 17:21 | |
*** irenab has quit IRC | 17:23 | |
*** irenab_ is now known as irenab | 17:23 | |
mdbooth | https://review.openstack.org/#/c/407470/ <--- Any chance we can knock that on the head? It's trivial and it will save me some time. | 17:24 |
*** annegentle has quit IRC | 17:25 | |
sdague | mdbooth: +2 from me | 17:25 |
mdbooth | sdague: Thanks | 17:25 |
* mdbooth is just trying to minimise rebases :) | 17:25 | |
sdague | yep | 17:26 |
*** nic has joined #openstack-nova | 17:26 | |
*** rcernin has joined #openstack-nova | 17:26 | |
*** mdrabe has quit IRC | 17:27 | |
*** bvanhav__ has joined #openstack-nova | 17:28 | |
*** bvanhav_ has quit IRC | 17:29 | |
*** eharney has quit IRC | 17:30 | |
*** mlavalle has quit IRC | 17:31 | |
*** diogogmt has joined #openstack-nova | 17:32 | |
*** mdrabe has joined #openstack-nova | 17:32 | |
*** sudipto_ has quit IRC | 17:32 | |
*** sudipto has quit IRC | 17:32 | |
*** lucasagomes is now known as lucas-afk | 17:33 | |
*** code-R has joined #openstack-nova | 17:33 | |
*** mdnadeem has quit IRC | 17:34 | |
mriedem | now back to the other super fun gate failure for novaclient... | 17:35 |
*** aweeks has joined #openstack-nova | 17:35 | |
*** baoli has joined #openstack-nova | 17:36 | |
*** code-R_ has joined #openstack-nova | 17:36 | |
*** code-R has quit IRC | 17:38 | |
diana_clarke | mriedem: yeah... I'm still stumped. Aside from knowing the cells v2 devstack change triggered it. | 17:39 |
sdague | mriedem: what failure are we looking at? | 17:40 |
mriedem | sdague: https://review.openstack.org/#/c/407205/ | 17:40 |
mriedem | sdague: http://logs.openstack.org/05/407205/1/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/02d19a4/console.html#_2016-12-06_15_24_58_720026 | 17:41 |
mriedem | getting a marker not found from a test in the client | 17:41 |
mriedem | creates 3 servers with the same name, waits for them to be active, then does a nova list --limit=1 --name <name> | 17:41 |
*** tblakes has quit IRC | 17:41 | |
mriedem | the --limit=1 makes novaclient loop and pass in a marker | 17:41 |
diana_clarke | sdague: And reverting this makes it green again: https://review.openstack.org/#/c/407319/ | 17:41 |
mriedem | diana_clarke: i'm not even sure why the limit=-1 logic makes it make a 2nd call | 17:42 |
mriedem | because i thought limit=-1 meant, only make another call if there are more results to get | 17:42 |
mriedem | ? | 17:42 |
diana_clarke | mriedem: something like that, I think. It's in while True loop, IIRC | 17:42 |
mriedem | yeah https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/servers.py#L841 | 17:43 |
sdague | what is limit -1 supposed to mean semantically? | 17:44 |
mriedem | sdague: see ^ | 17:44 |
mriedem | means page until there are no more results | 17:45 |
mriedem | in the client | 17:45 |
*** eharney has joined #openstack-nova | 17:45 | |
mriedem | we have 3 servers, so in the first query i expect it returns those 3, then since limit=-1 it takes the last one as the marker, which is 212edd36-6dc0-4c6f-a4fb-69e08e1c2335 | 17:45 |
mriedem | then makes a 2nd query with marker=212edd36-6dc0-4c6f-a4fb-69e08e1c2335 | 17:46 |
mriedem | and that fails with a marker not found, which is confusing me | 17:46 |
mriedem | here is where we call with the marker set http://logs.openstack.org/05/407205/1/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/02d19a4/logs/screen-n-api.txt.gz#_2016-12-06_15_24_57_835 | 17:47 |
*** yamahata has quit IRC | 17:47 | |
*** fragatina has joined #openstack-nova | 17:48 | |
*** rmart04 has joined #openstack-nova | 17:48 | |
mriedem | i dumped the sql query object in there | 17:48 |
mriedem | and i don't actually see the marker in the query | 17:48 |
mriedem | right here https://review.openstack.org/#/c/407204/3/nova/db/sqlalchemy/api.py@1891 | 17:49 |
mriedem | is this http://logs.openstack.org/05/407205/1/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/02d19a4/logs/screen-n-api.txt.gz#_2016-12-06_15_24_57_851 | 17:49 |
*** sreeram_v has joined #openstack-nova | 17:50 | |
*** tblakes has joined #openstack-nova | 17:50 | |
mriedem | WHERE instances.uuid = :uuid_1 | 17:51 |
mriedem | wtf is that | 17:51 |
mriedem | ? | 17:51 |
diana_clarke | mriedem: the marker clause is there, just indented left: "WHERE instances.uuid = :uuid_1" I think anyway | 17:51 |
sdague | yeh, so this is an off by one error right? | 17:51 |
*** Zer0Byte__ has joined #openstack-nova | 17:51 | |
mriedem | that should be WHERE instances.uuid = 212edd36-6dc0-4c6f-a4fb-69e08e1c2335 | 17:51 |
sdague | http://logs.openstack.org/05/407205/1/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/02d19a4/logs/screen-n-api.txt.gz#_2016-12-06_15_24_57_996 looks like this is a 400 on the API side | 17:51 |
*** bkopilov has quit IRC | 17:51 | |
*** abalutoiu_ has joined #openstack-nova | 17:51 | |
*** abalutoiu_ has quit IRC | 17:51 | |
sdague | the client is just translating it as such | 17:52 |
sdague | the client is right, the server is wrong | 17:52 |
mriedem | right but clearly we have an instance with that uuid | 17:52 |
mriedem | ERROR (BadRequest): marker [212edd36-6dc0-4c6f-a4fb-69e08e1c2335] not found; current instance UUIDs: ['66e7d166-2a6d-4810-9a70-5743cb2159e5', 'ef018cbe-a878-4ebc-b519-7fed72fd014e', '212edd36-6dc0-4c6f-a4fb-69e08e1c2335'] (HTTP 400) (Request-ID: req-042909e7-45e1-4485-b72d-c62391eeff29) | 17:52 |
mriedem | b/c it's in that list | 17:52 |
mriedem | i make the same query after the marker not found to list the same instances but w/o the marker | 17:53 |
sdague | mriedem: there is a name param on the filter as well | 17:53 |
sdague | "GET /v2.1/servers/detail?marker=212edd36-6dc0-4c6f-a4fb-69e08e1c2335&name=95979b2e-599b-4d05-aa66-14d718c23747 HTTP/1.1" | 17:53 |
mriedem | yeah, we don't get to the name filter i don't think in the db api | 17:53 |
mriedem | b/c we're first looking up the instance by marker | 17:54 |
*** rama_y has joined #openstack-nova | 17:54 | |
*** sreeram_v has quit IRC | 17:54 | |
mriedem | right here https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L2235 | 17:54 |
mriedem | that's before we apply the display_name filter in sqlalchemyutils.paginate_query | 17:55 |
dansmith | vladikr: I might have figured out the other problem with those tests that show up after this fixture changes | 17:55 |
*** browne has joined #openstack-nova | 17:55 | |
vladikr | dansmith, great, what is it? | 17:55 |
dansmith | vladikr: some interleaving with the mucking of global db state it's doing | 17:56 |
dansmith | vladikr: so with that in place, the only fail I see is that one PF_no_VF failure thing | 17:56 |
*** baoli has quit IRC | 17:56 | |
vladikr | dansmith, ok, yea, I'm still debugging it - hope to finish soon | 17:57 |
dansmith | vladikr: coolt hanks | 17:57 |
*** baoli has joined #openstack-nova | 17:57 | |
mriedem | sdague: diana_clarke: https://github.com/openstack/nova/commit/9ffa622b6ff062379e489809c0ed5a1b61881916 - i wonder if after we turned cells v2 on, if the query is going to the API DB or something? | 17:57 |
*** weshay_ has quit IRC | 17:58 | |
*** bkopilov has joined #openstack-nova | 17:58 | |
diana_clarke | mriedem: But those tables don't exist in the API database, correct? I would expect an exception in that case. I was thinking that perhaps one of the joins are failing on get because perhaps in the cells v2 case something isn't created that's joined on. | 17:58 |
*** rmart04 has quit IRC | 17:59 | |
diana_clarke | But honestly, I don't know... that was my only remaining hunch. | 17:59 |
*** weshay has joined #openstack-nova | 17:59 | |
mriedem | melwitt: do we lose anything needed for picking the proper cell when we elevate a context? | 18:00 |
*** mvk has joined #openstack-nova | 18:00 | |
*** rmart04 has joined #openstack-nova | 18:00 | |
melwitt | mriedem: let me check. it will be a matter of whether the db_connection attr is preserved during the clone | 18:00 |
dansmith | mriedem: it does a deepcopy | 18:00 |
dansmith | so I think it should be fine | 18:00 |
*** Guest17450 has quit IRC | 18:01 | |
*** tbachman has quit IRC | 18:01 | |
*** alexpilotti has joined #openstack-nova | 18:02 | |
cfriesen | jaypipes: are we expecting the resource provider work to solve https://bugs.launchpad.net/nova/+bug/1469179 in Ocata? Or would that not land till Pike? | 18:02 |
openstack | Launchpad bug 1469179 in OpenStack Compute (nova) "instance.root_gb should be 0 for volume-backed instances" [Medium,In progress] - Assigned to melanie witt (melwitt) | 18:02 |
mriedem | i guess i'm still confused about "WHERE instances.uuid = :uuid_1" | 18:02 |
mriedem | and how to actually get the full sql we'd see when it's executed | 18:02 |
diana_clarke | mriedem: Put another way, perhaps the instance create path differs subtly in the cells v2 case, making the get not work later. | 18:02 |
*** jwcroppe has quit IRC | 18:03 | |
sdague | mriedem: why do you think it's not https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L2242 when it blows up? | 18:03 |
*** tbachman has joined #openstack-nova | 18:03 | |
mriedem | sdague: we don't get that far | 18:03 |
*** syjulian_ has joined #openstack-nova | 18:03 | |
mriedem | sdague: we get here https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L2240 | 18:03 |
mriedem | and raise that up to the rest api | 18:03 |
melwitt | that means the uuid is missing right i.e. nothing was passed? that marker was None? | 18:03 |
diana_clarke | sdague: It raises a MarkerNotFound exception a few lines before that | 18:03 |
mriedem | melwitt: that wouldn't happen https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L2235 | 18:04 |
mriedem | melwitt: and: 2016-12-06 15:24:57.835 27484 DEBUG nova.compute.api [req-042909e7-45e1-4485-b72d-c62391eeff29 admin admin] Searching with limit '1000' and marker '212edd36-6dc0-4c6f-a4fb-69e08e1c2335' get_all /opt/stack/new/nova/nova/compute/api.py:2316 | 18:04 |
dansmith | vladikr: although a bunch now fail in new and different ways with the top patch :( | 18:04 |
*** siva_krish has quit IRC | 18:04 | |
melwitt | okay | 18:04 |
openstackgerrit | Chris Dent proposed openstack/nova: Support filtering resource providers by aggregates https://review.openstack.org/407629 | 18:04 |
dansmith | vladikr: I dunno why these are so different from everything else | 18:04 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Extend get_all_by_filters to support resource criteria https://review.openstack.org/386242 | 18:05 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Expose a REST API for a specific list of RPs https://review.openstack.org/392569 | 18:05 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Support multiple uuids and names for RP list https://review.openstack.org/407630 | 18:05 |
sdague | diana_clarke: our MarkerNotFound error messages are different though, right? I guess I was trying to figure out where it was translating | 18:05 |
*** derekh has quit IRC | 18:05 | |
vladikr | dansmith, are these only the sriov tests? | 18:05 |
mriedem | sdague: in the rest api, we throw away the MarkerNotFound message | 18:05 |
dansmith | vladikr: yeah | 18:05 |
mriedem | sdague: https://review.openstack.org/#/c/407204/3/nova/api/openstack/compute/servers.py@358 | 18:05 |
mriedem | maybe i should run with CONF.database.connection_trace = True | 18:05 |
bauzas | jaypipes: dansmith: cdent: edleafe: my pleasure to tell to you that I wrote a new REST API for the RP list https://review.openstack.org/392569 | 18:06 |
sdague | mriedem: ah, I see now | 18:06 |
*** siva_krish has joined #openstack-nova | 18:06 | |
*** ralonsoh has quit IRC | 18:06 | |
bauzas | jaypipes: dansmith: cdent: edleafe: I'll be writing the gabbi tests tonighty | 18:06 |
*** code-R_ has quit IRC | 18:06 | |
*** alexpilotti has quit IRC | 18:06 | |
*** huhaoran has quit IRC | 18:07 | |
*** code-R has joined #openstack-nova | 18:07 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Expose a REST API for a specific list of RPs https://review.openstack.org/392569 | 18:09 |
diana_clarke | mriedem: To answer your earlier question, this is 100% normal: " :uuid_1". That's what a compiled query looks like before the bind parameters are applied. | 18:09 |
mriedem | diana_clarke: is there a way to see what it looks like after the parameters are applied? | 18:10 |
*** eandersson has quit IRC | 18:10 | |
*** kristian__ has quit IRC | 18:10 | |
mriedem | btw | 18:11 |
mriedem | oslo.db 4.15.0 on 12/1 https://review.openstack.org/#/c/405491/ | 18:11 |
diana_clarke | mriedem: you can increase the connection_debug logging in the nova conf (which is what I do when I play with devstack) | 18:11 |
mriedem | maybe there is something coincidental in there with the cells v2 stuff | 18:11 |
sdague | or turn on slow query on the mysql side to 0, and have it dump all the queries | 18:11 |
sdague | it might be easier to read than the sqlalchemy versions of it | 18:12 |
mriedem | diana_clarke: heh | 18:12 |
mriedem | # Verbosity of SQL debugging information: 0=None, 100=Everything. (integer # value) # Minimum value: 0 # Maximum value: 100 # Deprecated group/name - [DEFAULT]/sql_connection_debug #connection_debug = 0 | 18:12 |
diana_clarke | mriedem: I think you can also .compile().params | 18:12 |
mriedem | maybe 75 has what i want, maybe 25 does, who knows?! | 18:13 |
diana_clarke | on the query, but it's been a long time since I last tried. edleafe was asking the same thing a few days ago | 18:13 |
* mriedem summons zzzeek | 18:13 | |
diana_clarke | mriedem: greater than 50 is what you want, I think | 18:13 |
zzzeek | hey, got about 10 minutes here | 18:13 |
mriedem | zzzeek: https://review.openstack.org/#/c/407204/3/nova/db/sqlalchemy/api.py | 18:13 |
mriedem | i want to get the actual compiled params from that query before executing it | 18:14 |
mriedem | and dump the straight up sql that's going to be executed | 18:14 |
mriedem | is there an easy way? | 18:14 |
sdague | so, a little bit of a distraction, but we're doing an aweful lot of work in getting a full instance (with join tables) when we litterally only need an object with .project_id and .uuid | 18:14 |
diana_clarke | mriedem: http://git.openstack.org/cgit/openstack/rack/tree/rack/openstack/common/db/sqlalchemy/session.py#n600 | 18:14 |
zzzeek | mriedem: it's not "easy" but it is: compiled = query.statement.compile(); compiled.params | 18:14 |
mriedem | sdague: yes i noticed that too | 18:14 |
mriedem | diana_clarke: ah nice, we should maybe update the help text for that config option | 18:15 |
sdague | it might be easier to see what is going on if we did that optimization there first | 18:15 |
mriedem | zzzeek: and compiled.params is the sql string? | 18:15 |
zzzeek | mriedem: that's a dictionary of the parameters themselves | 18:15 |
*** anush has quit IRC | 18:15 | |
zzzeek | str(compiled) is the SQL string | 18:15 |
mriedem | ok that's what i want, thanks | 18:15 |
zzzeek | mriedem: if you want it to be the MySQL compiled, statemnet.compile(dialect=my_engine.dialect) | 18:16 |
diana_clarke | mriedem: this example might help visualize it: http://docs.sqlalchemy.org/en/latest/core/tutorial.html#insert-expressions | 18:17 |
openstackgerrit | Dan Smith proposed openstack/nova: Make servers api view load instance fault from proper cell https://review.openstack.org/407238 | 18:17 |
openstackgerrit | Dan Smith proposed openstack/nova: Add schedule_and_build_instances conductor method https://review.openstack.org/367557 | 18:17 |
openstackgerrit | Dan Smith proposed openstack/nova: WIP Multicell support for instance listing https://review.openstack.org/396775 | 18:17 |
openstackgerrit | Dan Smith proposed openstack/nova: Make RPCFixture support multiple connections https://review.openstack.org/396417 | 18:17 |
openstackgerrit | Dan Smith proposed openstack/nova: Setup CellsV2 environment in base test https://review.openstack.org/406380 | 18:17 |
openstackgerrit | Dan Smith proposed openstack/nova: Add SecurityGroup.identifier to prefer uuid over name https://review.openstack.org/406260 | 18:17 |
openstackgerrit | Dan Smith proposed openstack/nova: Add SingleCellSimple fixture https://review.openstack.org/405684 | 18:17 |
openstackgerrit | Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | 18:17 |
openstackgerrit | Dan Smith proposed openstack/nova: Fix up non-cells-aware context managers in test_db_api https://review.openstack.org/406379 | 18:17 |
openstackgerrit | Dan Smith proposed openstack/nova: Add a CellDatabases test fixture https://review.openstack.org/399710 | 18:17 |
dansmith | melwitt: so I updated the fixture patch.. I think we *have* to lock before we muck with global db state ^ | 18:18 |
dansmith | melwitt: basically, what was happening in one case was that a test created an instance, which was in the middle of going through the build process in cell1, | 18:18 |
openstackgerrit | Sarafraj Singh proposed openstack/nova: conf: remove deprecated barbican options https://review.openstack.org/397774 | 18:19 |
dansmith | melwitt: and that test would poll the api, which would switch between cell1 and cell0 to compile the full list, and occasionally overlap with the build process, causing it to fail because suddenly the compute manager methods were hitting cell0 | 18:19 |
melwitt | dansmith: ugh, okay | 18:19 |
*** tbachman has quit IRC | 18:19 | |
dansmith | melwitt: the fix isn't as ugly as I thought it was going to be, luckily | 18:19 |
dansmith | and it makes sense | 18:20 |
openstackgerrit | Matt Riedemann proposed openstack/nova: DNM: debug novaclient limit/marker failure https://review.openstack.org/407204 | 18:20 |
melwitt | dansmith: yeah. I'm not surprised there were more issues with _last_ctxt_mgr. I just didn't know what it was going to be | 18:21 |
melwitt | thanks for fixing that :) | 18:22 |
dansmith | yup | 18:22 |
dansmith | narrowing the problems I think | 18:22 |
dansmith | I think there might be one more with cell0 stuff, on top of what vladikr is debugging right now, but hoping the light I see is the end of the tunnel | 18:22 |
dansmith | and not the one you see right before death | 18:22 |
melwitt | lol | 18:23 |
melwitt | that's a good one | 18:23 |
*** efoley has quit IRC | 18:23 | |
* dansmith notes that his potential demise is funny to melwitt | 18:23 | |
openstackgerrit | Sarafraj Singh proposed openstack/nova: conf: remove deprecated barbican options https://review.openstack.org/397774 | 18:25 |
diana_clarke | mriedem: Either way, I don't think seeing the query will help. I suspect it's passing the correct marker, and that a join is failing. | 18:25 |
vladikr | dansmith, I think I think I know what the problem is with the test.. the parent addresses of the vf are not generated in fakelibvirt - it doesn't matter for the rest of the tests, but this one is completely wrong - fixing it now | 18:25 |
melwitt | heh | 18:25 |
dansmith | vladikr: sweet | 18:25 |
edleafe | bauzas: sweet | 18:25 |
*** rama_y has quit IRC | 18:26 | |
*** rfolco has quit IRC | 18:26 | |
*** yamahata has joined #openstack-nova | 18:26 | |
*** krtaylor has quit IRC | 18:28 | |
openstackgerrit | melanie witt proposed openstack/nova: Handle ImageNotFound exception during instance backup https://review.openstack.org/388519 | 18:29 |
*** alexpilotti has joined #openstack-nova | 18:31 | |
mriedem | sdague: going back to your question, it looks like paginate_query in oslo.db only uses the marker for pulling sort keys, which by default are https://review.openstack.org/#/c/407204/3/nova/db/sqlalchemy/api.py | 18:31 |
mriedem | oops | 18:31 |
*** kaisers_ has joined #openstack-nova | 18:31 | |
mriedem | default_keys=['created_at', 'id'] | 18:31 |
*** tbachman has joined #openstack-nova | 18:31 | |
*** jwcroppe has joined #openstack-nova | 18:32 | |
mriedem | and as we shouldn't sort on joined tables, yeah we probably don't need those | 18:32 |
mriedem | even though today you can sort on joined tables, but we're fixing that | 18:32 |
*** kaisers_ has quit IRC | 18:33 | |
*** alexpilotti has quit IRC | 18:33 | |
*** kaisers_ has joined #openstack-nova | 18:33 | |
*** alexpilotti has joined #openstack-nova | 18:33 | |
*** Apoorva has joined #openstack-nova | 18:33 | |
edleafe | bauzas: oh, wait - you're doing the claim logic? I thought that wasn't on the table for Ocata | 18:34 |
*** penick has joined #openstack-nova | 18:35 | |
*** dharinic is now known as dharinic|lunch | 18:35 | |
*** penick_ has joined #openstack-nova | 18:36 | |
melwitt | diana_clarke, mriedem: did anyone try not elevating the context and seeing if it succeeds otherwise? I noticed elevated only deep copies self.roles and shallow copies everything else, but I didn't think that should matter | 18:36 |
mriedem | haven't tried it | 18:36 |
*** mkoderer has quit IRC | 18:36 | |
mriedem | i also want to try not joining on all of these other tables | 18:36 |
melwitt | okay | 18:37 |
*** rama_y has joined #openstack-nova | 18:37 | |
*** b_rake has joined #openstack-nova | 18:37 | |
*** b_rake has quit IRC | 18:38 | |
mriedem | melwitt: that would still copy db_connection right? | 18:38 |
*** tbachman has quit IRC | 18:38 | |
melwitt | mriedem: shallow copy, yeah | 18:38 |
*** b_rake has joined #openstack-nova | 18:39 | |
*** penick has quit IRC | 18:39 | |
*** penick_ is now known as penick | 18:39 | |
tdurakov | sdague: hi, what will you recommend me on this one: https://review.openstack.org/#/c/389767/ if it's should be tempest plugin, should I exclude all live-migration related tests from tempest as well? | 18:39 |
*** tbachman has joined #openstack-nova | 18:42 | |
*** b_rake has quit IRC | 18:43 | |
*** rfolco has joined #openstack-nova | 18:44 | |
*** tbachman has quit IRC | 18:44 | |
*** lpetrut has joined #openstack-nova | 18:46 | |
*** anush has joined #openstack-nova | 18:47 | |
*** penick has quit IRC | 18:48 | |
jroll | jaypipes: jfyi, started on the CI stuff for the resource class things. gonna be a couple days (given project-config is frozen) but I think I've got the right direction here: https://review.openstack.org/#/q/topic:bug/1647813 | 18:49 |
*** lpetrut has quit IRC | 18:50 | |
jaypipes | jroll: rock on brother. | 18:53 |
jaypipes | bauzas: reviewing now. | 18:54 |
*** krtaylor has joined #openstack-nova | 18:54 | |
jaypipes | cfriesen: dansmith already solved that for the resource providers inventory/allocation records: https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L103-L107 | 18:55 |
*** weshay has quit IRC | 18:55 | |
*** rmart04 has quit IRC | 18:58 | |
*** weshay has joined #openstack-nova | 18:58 | |
*** tbachman has joined #openstack-nova | 19:02 | |
stvnoyes | hi, i'm having trouble using a remote debugger (pdev/eclipse) with nova-compute in devstack. I started nova-compute with --remote_debug-host 192.168.9.94 --remote_debug-port 5678 , and I hit the breakpoint at the log msg warning about eventlet. But when I resume execution, nova-compute hangs after printing "Connecting to libvirt: qemu:///system" in the log. Is this familiar to anyone? | 19:02 |
mriedem | bauzas: jaypipes: could use your input on the last comment from me here https://review.openstack.org/#/c/406627/ | 19:04 |
mriedem | about the proper order and way handle deleting a compute service record and the compute node | 19:04 |
mriedem | *way of handling | 19:04 |
*** huhaoran has joined #openstack-nova | 19:07 | |
*** jamielennox|away is now known as jamielennox | 19:07 | |
*** nic has quit IRC | 19:08 | |
*** nic has joined #openstack-nova | 19:09 | |
mriedem | jaypipes: also, can't we get rid of create/update_inventory methods in the ComptueNode object now? | 19:09 |
mriedem | as that's done in the RT | 19:10 |
mriedem | i thought that stuff was all reverted out in newton | 19:10 |
jaypipes | mriedem: yes we absolutely can now. | 19:11 |
mriedem | aye aye | 19:11 |
mriedem | sdague: you could drop the glance v1 code in nova now too if you wanted :) | 19:11 |
mriedem | i don't think anyone has claimed that yet | 19:11 |
jaypipes | mriedem: b/c Mitaka was the last code that called that, IIRC. dansmith pls confirm? | 19:12 |
*** huhaoran has quit IRC | 19:12 | |
dansmith | jaypipes: mriedem they're not remotable so they have nothing to do with when they were last called | 19:12 |
openstackgerrit | Chris Dent proposed openstack/nova: Fix a typo in a comment in microversion history https://review.openstack.org/407664 | 19:12 |
*** Apoorva has quit IRC | 19:12 | |
dansmith | although they are still using remotable methods on the RP objects | 19:13 |
cdent | sdague: ^ that's in response to something you noticed | 19:13 |
dansmith | which has impacts for cdent's removal of those | 19:13 |
dansmith | but I assume we're just breaking all the rules for that anyway | 19:13 |
*** Apoorva has joined #openstack-nova | 19:13 | |
cdent | @I might as well begin to put some action in my life@ | 19:13 |
*** catinthe_ has joined #openstack-nova | 19:14 | |
mriedem | dansmith: idk anything about that, i just don't see anything calling create/update_inventory on the ComputeNode object anywhere | 19:14 |
mriedem | so assume it's dead code | 19:14 |
*** bapalm_ has quit IRC | 19:14 | |
dansmith | mriedem: I assume it is as well, if you have confirmed, then they can come out yeah | 19:14 |
mriedem | ok | 19:14 |
*** catintheroof has quit IRC | 19:16 | |
*** liverpooler has joined #openstack-nova | 19:16 | |
*** bapalm has joined #openstack-nova | 19:18 | |
*** code-R has quit IRC | 19:20 | |
jaypipes | mriedem: done. | 19:20 |
sdague | cdent: thanks | 19:21 |
mriedem | jaypipes: reading, thanks | 19:21 |
sdague | cdent: +A | 19:21 |
jaypipes | bauzas: you may want to weigh in on https://review.openstack.org/#/c/406627 as well since it was you and I who removed the linkage between compute_nodes and services. | 19:21 |
sdague | mriedem: sure, I could probably cycle around on that | 19:21 |
mriedem | jaypipes: i see some RT/computenode/servicegroup love of yours in here :) | 19:21 |
jaypipes | mriedem: :) | 19:21 |
jaypipes | mriedem: bad memories. | 19:21 |
*** fragatina has quit IRC | 19:21 | |
mriedem | it's over now jay | 19:22 |
mriedem | well, not really | 19:22 |
jaypipes | mriedem: :) | 19:22 |
mriedem | but something something PTSD comforting | 19:22 |
tdurakov | melwitt: are you around? | 19:22 |
melwitt | tdurakov: yes | 19:22 |
tdurakov | melwitt: quick question on https://review.openstack.org/#/c/407603/ | 19:23 |
*** Sukhdev has joined #openstack-nova | 19:23 | |
melwitt | okay | 19:24 |
tdurakov | melwitt: how will this affect execution flow, if main_context_manager will be used instead of db_connection, that is stored in context? | 19:24 |
* tdurakov trying to figure out whether this one should be backported or not | 19:25 | |
*** dharinic|lunch is now known as dharinic | 19:27 | |
mriedem | smcginnis: was the bug smash last week advertised all that well? i wasn't even aware of it happening. | 19:27 |
mriedem | smcginnis: or did you just direct them away from all of the nova bugs and make them focus on cinder bugs | 19:28 |
melwitt | tdurakov: I'm not sure I follow. the way it is currently, the context passed in is never being used because it's assigned to use_slave | 19:28 |
*** annegentle has joined #openstack-nova | 19:31 | |
*** Sukhdev has quit IRC | 19:31 | |
*** claudiub has joined #openstack-nova | 19:32 | |
*** Sukhdev has joined #openstack-nova | 19:32 | |
*** McNinja has quit IRC | 19:33 | |
tdurakov | melwitt: yes, but I'm still not sure, do we have bugs because of that or not | 19:33 |
*** morgabra has quit IRC | 19:34 | |
*** morgabra has joined #openstack-nova | 19:34 | |
*** morgabra has joined #openstack-nova | 19:34 | |
*** claudiub|2 has quit IRC | 19:35 | |
*** slunkad has quit IRC | 19:35 | |
*** andreaf has quit IRC | 19:35 | |
*** toabctl has quit IRC | 19:35 | |
melwitt | tdurakov: oh, I see what you mean. I think not yet because we don't have multiple cells v2 cells yet | 19:35 |
*** McNinja has joined #openstack-nova | 19:36 | |
*** toabctl has joined #openstack-nova | 19:36 | |
*** slunkad has joined #openstack-nova | 19:36 | |
*** andreaf has joined #openstack-nova | 19:37 | |
tdurakov | melwitt: acked, thanks | 19:38 |
*** Jeffrey4l has quit IRC | 19:42 | |
*** Jeffrey4l has joined #openstack-nova | 19:42 | |
*** kristian__ has joined #openstack-nova | 19:43 | |
*** dinobot_ has quit IRC | 19:44 | |
*** siva_krish has quit IRC | 19:48 | |
*** ijw has quit IRC | 19:49 | |
*** claudiub|2 has joined #openstack-nova | 19:50 | |
dansmith | melwitt: mriedem: okay so the bottom few patches of this series are still good to go: https://review.openstack.org/#/c/399710/5 | 19:52 |
dansmith | I have -Wd the one where we need to hold off until vladikr fixes the sriov tests | 19:52 |
dansmith | but that series is getting a little big for its britches, so would like to see some progress there | 19:52 |
*** claudiub has quit IRC | 19:53 | |
melwitt | k, will look today | 19:53 |
dansmith | thanks | 19:53 |
dansmith | we need someone else for the fixture anyway | 19:53 |
melwitt | yes | 19:54 |
*** jwcroppe has quit IRC | 19:54 | |
*** WillUK2010 has joined #openstack-nova | 19:56 | |
*** moshele has joined #openstack-nova | 19:59 | |
*** rfolco has quit IRC | 20:01 | |
*** catintheroof has joined #openstack-nova | 20:02 | |
mriedem | i was +2 on the fixture last night | 20:04 |
*** catinthe_ has quit IRC | 20:04 | |
*** kristian__ has quit IRC | 20:04 | |
mriedem | but i see it's all been drastically changed | 20:04 |
*** portdirect__ is now known as portdirect_away | 20:04 | |
dansmith | mriedem: drastically fixed you mean | 20:05 |
smcginnis | mriedem: No and yes. ;) | 20:05 |
mriedem | drastically awesomatized yes | 20:05 |
smcginnis | mriedem: I only recall seeing one mention in the ML that the bugsmash was coming up. | 20:05 |
*** siva_krish has joined #openstack-nova | 20:06 | |
smcginnis | mriedem: The only reason I even knew about it this time was I had missed the last two and kind of wanted to participate, especially since I made Ocata mostly bugfixes for Cinder. | 20:06 |
smcginnis | mriedem: I'm hoping by that posting it can get better communicated out for next time. | 20:06 |
smcginnis | mriedem: There was a group working on Nova bugs, but I didn't spend any time at that table. | 20:07 |
smcginnis | mriedem: At a minimum, I think it would be good to alert the cores of the projects that have attendees working on bugs so they can watch out for and review their patches while we can take advantage of their focused attention. | 20:08 |
mriedem | smcginnis: they got the timing right this time around re: the release schedule | 20:08 |
mriedem | so it's not in the 3rd milestone or later | 20:08 |
smcginnis | mriedem: Right! | 20:08 |
mriedem | i just hadn't heard about it happening until you said you were there last week | 20:08 |
*** kristian__ has joined #openstack-nova | 20:08 | |
*** portdirect_away is now known as portdirect | 20:10 | |
*** kaisers_ has quit IRC | 20:11 | |
melwitt | diana_clarke, mriedem: I think the problem with the limit/marker thing might be that the servers are BuildRequests and when the 2nd query is made with the marker, the instances query raises MarkerNotFound instead of just returning an empty list? as there are no instances in the instances table, they are build_requests | 20:12 |
mriedem | melwitt: it's not that | 20:12 |
*** hfu has quit IRC | 20:12 | |
mriedem | melwitt: we still create the instances in the db from the api right now | 20:12 |
mriedem | that hasn't changed | 20:13 |
mriedem | and this test is waiting for the servers to be active before doing the query | 20:13 |
*** kaisers_ has joined #openstack-nova | 20:13 | |
mriedem | melwitt: but that's what i was thinking of yesterday, that we were actually querying against build requests and there was a bug in that, but i don't think so | 20:13 |
melwitt | hmm | 20:13 |
mriedem | plus my debug logging in the recreate is showing we're getting down to the point of querying the instances table | 20:13 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Removed unused ComputeNode create/update_inventory methods https://review.openstack.org/407692 | 20:15 |
openstackgerrit | Ludovic Beliveau proposed openstack/nova: Resource tracker doesn't free resources on confirm resize https://review.openstack.org/405575 | 20:16 |
lbeliveau | jaypipes: ^ I have a fix for this RT bug I mentionned to you | 20:16 |
diana_clarke | melwitt: I think mriedem added an if statement on build_requests, but I never saw it appear in the logs, so i don't think so... | 20:16 |
openstackgerrit | Vasyl Saienko proposed openstack/nova: DO NOT REVIEW https://review.openstack.org/407693 | 20:16 |
jaypipes | lbeliveau: rock on brother. reviewing now. | 20:17 |
mriedem | do i smell some hulkamaniacs? | 20:17 |
jaypipes | step in to a slim jim. | 20:17 |
mriedem | oooo yyyyeeeaaahhhh | 20:17 |
lbeliveau | :) | 20:18 |
*** kaisers_ has quit IRC | 20:18 | |
mriedem | dansmith: on https://review.openstack.org/#/c/396417/20/nova/rpc.py did oslo.messaging get a release out with the fix for that hack? | 20:20 |
mriedem | diana_clarke: bleh still "WHERE instances.uuid = :uuid_1" | 20:23 |
mriedem | http://logs.openstack.org/05/407205/1/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/0c20b61/logs/screen-n-api.txt.gz#_2016-12-06_19_03_40_212 | 20:23 |
diana_clarke | mriedem: yeah, you didn't print .params ;( | 20:23 |
mriedem | interesting though | 20:24 |
mriedem | http://logs.openstack.org/05/407205/1/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/0c20b61/logs/screen-n-api.txt.gz#_2016-12-06_19_03_40_333 | 20:24 |
mriedem | ^ is one the query with marker=None | 20:24 |
mriedem | the 2nd query | 20:24 |
mriedem | that doesn't show up before the first one | 20:24 |
mriedem | which is https://review.openstack.org/#/c/407204/4/nova/compute/api.py@2410 | 20:25 |
mriedem | wait a minute | 20:25 |
mriedem | are we querying cell0? https://review.openstack.org/#/c/407204/4/nova/compute/api.py@2399 | 20:25 |
*** jwcroppe has joined #openstack-nova | 20:25 | |
mriedem | i bet we are | 20:26 |
mriedem | updating that patch to see | 20:26 |
*** portdirect is now known as portdirect_away | 20:26 | |
*** bvanhav__ has quit IRC | 20:27 | |
*** portdirect_away has quit IRC | 20:27 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Removed unused ComputeNode create/update_inventory methods https://review.openstack.org/407692 | 20:29 |
*** slaweq_ has quit IRC | 20:30 | |
*** sapcc-bot1 has quit IRC | 20:32 | |
*** sapcc-bot has joined #openstack-nova | 20:32 | |
*** syjulian_ has quit IRC | 20:33 | |
*** moshele has quit IRC | 20:36 | |
vladikr | dansmith, the test is passing now. going to pickup the kids ans will send the patch. | 20:36 |
openstackgerrit | Matt Riedemann proposed openstack/nova: DNM: debug novaclient limit/marker failure https://review.openstack.org/407204 | 20:37 |
mriedem | diana_clarke: melwitt: sdague: i think ^ is going to show it | 20:37 |
mriedem | i think we're getting a MarkerNotFound from cell0 | 20:37 |
*** slaweq_ has joined #openstack-nova | 20:37 | |
mriedem | then when i query the 2nd time, the query to cell0 returns nothing, and then we hit the actual instances table | 20:38 |
dansmith | vladikr: thanks | 20:38 |
dansmith | mriedem: ah, forgot about that hack, damn | 20:38 |
dansmith | mriedem: they merged, but no release It hink | 20:39 |
*** ijw_ has joined #openstack-nova | 20:39 | |
*** ijw_ has quit IRC | 20:39 | |
*** ijw_ has joined #openstack-nova | 20:39 | |
*** gouthamr has quit IRC | 20:39 | |
jaypipes | lbeliveau: done | 20:40 |
*** prateek has quit IRC | 20:40 | |
*** Sukhdev has quit IRC | 20:41 | |
lbeliveau | jaypipes: thanks ! will work on it | 20:41 |
*** kaisers_ has joined #openstack-nova | 20:42 | |
jaypipes | lbeliveau: no, thank YOU! :) | 20:42 |
mriedem | dansmith: how much do you want to wait for oslo.messaging release? or just move forward | 20:42 |
*** kashyap has joined #openstack-nova | 20:42 | |
*** kashyap has quit IRC | 20:43 | |
*** kashyap has joined #openstack-nova | 20:43 | |
*** daemontool_ has joined #openstack-nova | 20:44 | |
mriedem | nvm i guess harlow is doing one now | 20:46 |
*** daemontool has quit IRC | 20:47 | |
*** diogogmt has quit IRC | 20:48 | |
*** matrohon has joined #openstack-nova | 20:48 | |
*** siva_krish has quit IRC | 20:51 | |
*** david-lyle has quit IRC | 20:55 | |
dansmith | mriedem: there is a slightly less eye-killing hack I can do I think, but if there is a release coming then that's probably better | 20:56 |
*** xyang1 has joined #openstack-nova | 20:56 | |
*** slaweq_ has quit IRC | 21:01 | |
mriedem | dansmith: question in https://review.openstack.org/#/c/396417/20/nova/tests/fixtures.py | 21:02 |
*** xinliang has quit IRC | 21:06 | |
*** catintheroof has quit IRC | 21:06 | |
openstackgerrit | Ed Leafe proposed openstack/nova: WIP - Cascade deletes of RP aggregate associations https://review.openstack.org/407707 | 21:06 |
*** slaweq_ has joined #openstack-nova | 21:07 | |
*** toure is now known as toure|biab | 21:07 | |
*** ociuhandu has quit IRC | 21:07 | |
*** fragatina has joined #openstack-nova | 21:09 | |
*** huhaoran has joined #openstack-nova | 21:09 | |
*** dave-mccowan has quit IRC | 21:10 | |
*** cmartin has joined #openstack-nova | 21:14 | |
*** huhaoran has quit IRC | 21:14 | |
dansmith | mriedem: replied (in case you didn't see already) | 21:14 |
cmartin | mdbooth, Hey if you are there I'd like to have a discussion about this bug https://bugs.launchpad.net/nova/+bug/1501808 | 21:15 |
openstack | Launchpad bug 1501808 in OpenStack Compute (nova) "Enabling soft-deletes opens a DOS on compute hosts" [High,In progress] - Assigned to Chris Martin (cm876n) | 21:15 |
mriedem | dansmith: yeah i think a fixme is more clear | 21:15 |
*** xinliang has joined #openstack-nova | 21:18 | |
*** cdelatte has quit IRC | 21:18 | |
*** kristia__ has joined #openstack-nova | 21:22 | |
*** kristian__ has quit IRC | 21:22 | |
mriedem | diana_clarke: ha success http://logs.openstack.org/05/407205/1/check/gate-novaclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/61e9a05/console.html | 21:27 |
mriedem | http://logs.openstack.org/05/407205/1/check/gate-novaclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/61e9a05/logs/screen-n-api.txt.gz#_2016-12-06_20_58_54_417 | 21:27 |
mriedem | it was cell0 | 21:27 |
mriedem | dirty old cell0 | 21:27 |
openstackgerrit | Chris Dent proposed openstack/nova: Support filtering resource providers by aggregates https://review.openstack.org/407629 | 21:29 |
mriedem | melwitt: dansmith: do you know if the functional tests setup cell0 for us automatically? | 21:30 |
mriedem | i'd rather fix this with a functional test than a comptue api unit test using mock | 21:31 |
*** Jeffrey4l has quit IRC | 21:32 | |
*** Jeffrey4l has joined #openstack-nova | 21:34 | |
mriedem | looks like the database fixture only handles main and api dbs, but i'd think that could be changed to be main or cell0 | 21:36 |
mriedem | or api | 21:36 |
mriedem | b/c main and cell0 setup the same schema | 21:36 |
*** ihrachys has quit IRC | 21:36 | |
dansmith | mriedem: they do as of my base setup patch | 21:37 |
dansmith | mriedem: otherwise no | 21:37 |
mriedem | so something like this: | 21:38 |
mriedem | fix = nova_fixtures.CellDatabases() | 21:38 |
mriedem | fix.add_cell_database('main') | 21:38 |
mriedem | fix.add_cell_database('cell0') | 21:39 |
mriedem | self.useFixture(fix) | 21:39 |
mriedem | ? | 21:39 |
*** READ10 has quit IRC | 21:39 | |
mriedem | and then create the cell0 mapping in the api db i think | 21:39 |
dansmith | mriedem: that won't make cell0 work, that will give you two cell databases | 21:39 |
dansmith | yeah | 21:39 |
*** hamzy has quit IRC | 21:39 | |
dansmith | or just merge my patch that does this for any db-using test :) | 21:39 |
mriedem | dude | 21:39 |
dansmith | well, once we get vladikr's fix | 21:39 |
mriedem | i +2d the bottom fixture patch | 21:39 |
dansmith | I know, we can't even merge the test base patch until the rpc one, which depends on o.msg, and vladikr's fix :) | 21:40 |
dansmith | but.. UH | 21:40 |
mriedem | why can't we merge the bottom change yet? | 21:40 |
mriedem | nothing is using it yet | 21:40 |
dansmith | the bottom one we can yeah | 21:41 |
dansmith | let me clarify with links | 21:41 |
dansmith | mriedem: this is what I meant: https://review.openstack.org/#/c/406380/ | 21:42 |
mriedem | oh yeah that's what i want | 21:43 |
mriedem | that's what i NEED | 21:43 |
dansmith | you know I'll give you what you need | 21:43 |
dansmith | you KNOW it | 21:43 |
mriedem | well in the interest of time i'm probably just going to unit test this with a mock | 21:44 |
mriedem | b/c novaclient is blocked | 21:44 |
dansmith | sure | 21:46 |
diana_clarke | mriedem: nice work! It's still unclear to me which tables to expect in the cell0 database vs the others, but that can be my homework ;) | 21:48 |
mriedem | diana_clarke: cell0 schema is the same as the normal nova db schema | 21:48 |
mriedem | the non-api db | 21:49 |
dansmith | right | 21:49 |
diana_clarke | mriedem: ah... thanks! | 21:49 |
mriedem | we just don't populate cell0 yet | 21:49 |
mriedem | cell0 is the place where failed instance builds and their records go to die | 21:49 |
dansmith | kind of like florida for old people | 21:49 |
* mriedem thinks how to make a jay joke w/o offending him | 21:50 | |
*** alexpilo_ has joined #openstack-nova | 21:51 | |
*** smatzek has quit IRC | 21:52 | |
*** rtheis has quit IRC | 21:53 | |
*** whenry has quit IRC | 21:53 | |
*** alexpilotti has quit IRC | 21:54 | |
*** daemontool_ has quit IRC | 21:55 | |
*** daemontool_ has joined #openstack-nova | 21:55 | |
*** matrohon has quit IRC | 22:02 | |
mriedem | dansmith: why do we need cell0 again if we know which cell an instance failed to build in? | 22:03 |
dansmith | mriedem: for instances that failed to schedule | 22:03 |
mriedem | so it doesn't land in a cell | 22:04 |
mriedem | NoValidHost? | 22:04 |
mriedem | i.e. all scheduler filters rejected the build request | 22:04 |
mriedem | / request spec | 22:04 |
mriedem | ok | 22:04 |
mriedem | i need to be reminded quarterly | 22:04 |
dansmith | yeah, and for things that we discover are hosed after we return from the api call, | 22:04 |
dansmith | which could be things other than scheduling, but mostly scheduling | 22:04 |
*** whenry has joined #openstack-nova | 22:06 | |
*** krtaylor has quit IRC | 22:07 | |
jaypipes | mriedem: nothing you could say about FL would offend me. :) | 22:07 |
*** thorst has quit IRC | 22:09 | |
*** thorst has joined #openstack-nova | 22:09 | |
*** WillUK2010 has quit IRC | 22:14 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Handle MarkerNotFound from cell0 database https://review.openstack.org/407204 | 22:14 |
mriedem | melwitt: diana_clarke: andreykurilin: tdurakov: ^ | 22:14 |
mriedem | ooo i think alaski is still core :) | 22:15 |
* mriedem fishes | 22:15 | |
*** Sukhdev has joined #openstack-nova | 22:15 | |
dansmith | especially since that's his code, IIRC | 22:16 |
openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [DO NOT MERGE] Check nova's revert https://review.openstack.org/406388 | 22:16 |
mriedem | i'm pretty sure i was +2 or +W on the change that added that code to the compute api though | 22:16 |
mriedem | andreykurilin: you don't need that anymore | 22:17 |
dansmith | yeah, probably me too :) | 22:17 |
mriedem | andreykurilin: https://review.openstack.org/407204 is the fix | 22:17 |
andreykurilin | )) | 22:17 |
mriedem | andreykurilin: see https://review.openstack.org/#/c/407205/ | 22:17 |
mriedem | tests passed | 22:17 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: tests: avoid starting compute service twice in sriov functional test https://review.openstack.org/407572 | 22:17 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: tests: generate correct pci addresses for fake pci devices https://review.openstack.org/407730 | 22:17 |
andreykurilin | oh | 22:18 |
andreykurilin | ok, will abandon my patch) | 22:18 |
*** thorst has quit IRC | 22:18 | |
*** whenry has quit IRC | 22:18 | |
dansmith | vladikr: do we have tests for fakelibvirt? | 22:19 |
dansmith | vladikr: nova/tests/unit/virt/libvirt/test_fakelibvirt.py | 22:19 |
vladikr | dansmith, ah.. ok, no idea what does that do.. I was thinking about the functional tests as the test for the fakelibvirt | 22:20 |
vladikr | dansmith, I can add tests to the test_fakelibvirt.py - it's just a lot of things to add | 22:21 |
dansmith | vladikr: looks like that file has some basic tests, probably from good-intentioned original authors | 22:21 |
*** kristia__ has quit IRC | 22:21 | |
dansmith | vladikr: might be good just to make sure there's a little coverage of this path at least | 22:21 |
*** kristian__ has joined #openstack-nova | 22:21 | |
vladikr | dansmith, I'll look into it.. | 22:22 |
vladikr | dansmith, have to drive the kids to a class, will be online in 20 mins | 22:22 |
cfriesen | mriedem: do you have a proposal for https://bugs.launchpad.net/nova/+bug/1527623 ? Without the fix we can end up orphaning cinder volumes, which will require manual cleanup. | 22:22 |
openstack | Launchpad bug 1527623 in OpenStack Compute (nova) "Nova might orphan volumes when it's racing to delete a volume-backed instance" [Medium,In progress] - Assigned to ChangBo Guo(gcb) (glongwave) | 22:22 |
tdurakov | mriedem: question on that fix | 22:22 |
tdurakov | mriedem: is it a rootcause https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L2240? | 22:22 |
dansmith | vladikr: okay, it's late there anyway, not a huge rush, but would be cool to get these merged tomorrow | 22:22 |
tdurakov | if it is, why not to fix that instead? | 22:23 |
vladikr | dansmith, sure | 22:23 |
tdurakov | dansmith: could you take a look also? | 22:23 |
dansmith | vladikr: thanks! | 22:23 |
dansmith | tdurakov: we have to raise that exception somehow | 22:24 |
mriedem | tdurakov: cell0 is the same behavior as a normal cell database, | 22:24 |
*** david-lyle has joined #openstack-nova | 22:24 | |
mriedem | the bug is we're looking for an instance in there by marker which doesn't exist, when listing instances | 22:24 |
mriedem | if it's not in cell0, it's in the main cell db | 22:25 |
andreykurilin | mriedem: btw, rally gates has a lot of servers in "ERROR" state now. I think server creation start failure at the end of last week. Rally did not change anything in own code. I did not noticed failures since rally gates were broken by keystone... | 22:25 |
mriedem | we know things might not exist in cell0 | 22:25 |
dansmith | tdurakov: go -1 my patch that looks in more than one cell for not handling that properly | 22:25 |
dansmith | tdurakov: but in the current master code, what mriedem has is right | 22:25 |
mriedem | we could move cell0 queries until after the main cell db, | 22:25 |
dansmith | tdurakov: https://review.openstack.org/#/c/396775/9/nova/compute/api.py@2502 | 22:26 |
mriedem | i'm not entirely sure why laski had the order this way, except that once we have multiple cells that mult-cell listing gets more expensive | 22:26 |
dansmith | mriedem: no we can't | 22:26 |
mriedem | but i think we also agreed this order at the newton midcycle | 22:26 |
*** kristian__ has quit IRC | 22:26 | |
dansmith | mriedem: we decided that sorting would always be build requests, cell0, then cells | 22:26 |
dansmith | right | 22:26 |
mriedem | yeah | 22:26 |
mriedem | i just didn't want to go to the trouble of looking that up in the etherpad or ML if challenged | 22:27 |
mriedem | :P | 22:27 |
dansmith | heh | 22:27 |
mriedem | dansmith: i left a comment in there as a reminder | 22:28 |
dansmith | stolen from the hands of tdurakov ! | 22:28 |
mriedem | i didn't score | 22:28 |
mriedem | he can -1 it | 22:28 |
mriedem | pulled a classic alaski no vote | 22:29 |
dansmith | tdurakov: I'd be pissed if I were you | 22:29 |
bauzas | jaypipes: okay for your comment about using POST /resource_provider | 22:29 |
bauzas | I just want to not use GET | 22:29 |
mriedem | cfriesen: did you look at https://review.openstack.org/#/c/260339/ ? | 22:29 |
mriedem | cfriesen: also that was a year ago so i've lost a lot of context on that race bug since then | 22:30 |
*** jheroux has quit IRC | 22:30 | |
tdurakov | dansmith: that's why I'm not going to vote on that:p what's the point if you are already know about that | 22:30 |
mriedem | would have to dig into the bug report details to remember | 22:30 |
*** lpetrut has joined #openstack-nova | 22:30 | |
mriedem | tdurakov: dansmith needs -1s to remember | 22:30 |
mriedem | and he enjoys the pain | 22:30 |
dansmith | tdurakov: as a reminder! :) | 22:30 |
dansmith | right ^ | 22:30 |
dansmith | tdurakov: my wife has to -1 me to remind me to take out the trash and clean the dishes | 22:30 |
tdurakov | lol | 22:31 |
*** eharney has quit IRC | 22:31 | |
tdurakov | dansmith: done, is that good? | 22:31 |
*** dave-mccowan has joined #openstack-nova | 22:32 | |
dansmith | tdurakov: ahhhhhh, ohhh yeahhh.. | 22:32 |
mriedem | haha | 22:32 |
mriedem | cleanup in aisle dan | 22:32 |
dansmith | hah | 22:32 |
*** daemontool_ has quit IRC | 22:33 | |
*** daemontool_ has joined #openstack-nova | 22:34 | |
*** dave-mcc_ has joined #openstack-nova | 22:34 | |
tdurakov | <dansmith> tdurakov: ahhhhhh, ohhh yeahhh.. - is it common reaction for -1?) | 22:36 |
dansmith | tdurakov: common reaction for a masochist | 22:36 |
*** ijw_ has quit IRC | 22:37 | |
*** dave-mccowan has quit IRC | 22:37 | |
*** ijw has joined #openstack-nova | 22:37 | |
*** esberglu has quit IRC | 22:38 | |
*** thorst_ has joined #openstack-nova | 22:38 | |
*** thorst_ has quit IRC | 22:38 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Expose a REST API for a specific list of RPs https://review.openstack.org/392569 | 22:40 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Support multiple uuids and names for RP list https://review.openstack.org/407630 | 22:40 |
bauzas | jaypipes, edleafe: amended my patch to use POST /requests instead of claims :) | 22:40 |
openstackgerrit | Chris Dent proposed openstack/nova: rt: pass the nodename to public methods https://review.openstack.org/398470 | 22:40 |
openstackgerrit | Chris Dent proposed openstack/nova: rt: remove fluff from test_resource_tracker https://review.openstack.org/398471 | 22:40 |
openstackgerrit | Chris Dent proposed openstack/nova: rt: pass nodename to internal methods https://review.openstack.org/398469 | 22:40 |
openstackgerrit | Chris Dent proposed openstack/nova: placement: create aggregate map in report client https://review.openstack.org/407309 | 22:40 |
openstackgerrit | Chris Dent proposed openstack/nova: Support filtering resource providers by aggregates https://review.openstack.org/407629 | 22:40 |
openstackgerrit | Chris Dent proposed openstack/nova: placement: correct improper test case inheritance https://review.openstack.org/406363 | 22:40 |
openstackgerrit | Chris Dent proposed openstack/nova: rt: explicitly pass compute node to _update() https://review.openstack.org/398472 | 22:40 |
openstackgerrit | Chris Dent proposed openstack/nova: rt: use a single ResourceTracker object instance https://review.openstack.org/398473 | 22:40 |
openstackgerrit | Chris Dent proposed openstack/nova: HTTP interface for resource providers by aggregates https://review.openstack.org/407742 | 22:40 |
*** david-lyle has quit IRC | 22:41 | |
*** david-lyle has joined #openstack-nova | 22:41 | |
*** ijw has quit IRC | 22:42 | |
*** harlowja has quit IRC | 22:43 | |
*** mriedem has quit IRC | 22:43 | |
cdent | jaypipes: that stack ^ got rebased that stack to pull in the 1.2 microversion (for resource classes) because I was making a 1.3 microversion | 22:43 |
*** harlowja has joined #openstack-nova | 22:43 | |
*** cmartin has quit IRC | 22:45 | |
*** david-lyle has quit IRC | 22:46 | |
*** cdent has quit IRC | 22:53 | |
andreykurilin | mriedem: http://logs.openstack.org/13/407713/1/check/gate-rally-dsvm-rally-cinder/6225c7a/logs/screen-n-cpu.txt.gz?level=ERROR it appeared last week | 22:54 |
*** tblakes has quit IRC | 22:55 | |
STEWIE925 | hi guys, has anyone installed devstack with ubuntu 16.04? | 22:56 |
*** Swami has quit IRC | 22:57 | |
STEWIE925 | its my first with Ubuntu 16.04, and I am getting "could not determine host ip address. See local.conf for suggestions on setting HOST_IP | 22:57 |
STEWIE925 | I copied the local.conf from my working devstack - the local.conf file didnt have the host_ip line but its stacking fine. | 22:58 |
andreykurilin | mriedem: also, "marker" error presents in rally gates too... | 22:58 |
STEWIE925 | oh, and I am installing the new devstack instance in virtualbox | 22:59 |
*** ijw has joined #openstack-nova | 23:01 | |
*** jamesdenton has quit IRC | 23:02 | |
*** rcernin has quit IRC | 23:02 | |
*** mdrabe has quit IRC | 23:04 | |
*** jamielennox is now known as jamielennox|away | 23:05 | |
*** jamielennox|away is now known as jamielennox | 23:06 | |
*** nic has left #openstack-nova | 23:07 | |
*** nic has joined #openstack-nova | 23:07 | |
*** dave-mcc_ has quit IRC | 23:08 | |
*** sdague has quit IRC | 23:11 | |
*** xyang1 has quit IRC | 23:13 | |
*** ducttape_ has quit IRC | 23:14 | |
*** jwcroppe has quit IRC | 23:26 | |
*** kfarr has quit IRC | 23:26 | |
*** jwcroppe has joined #openstack-nova | 23:26 | |
*** jklare has quit IRC | 23:27 | |
*** krtaylor has joined #openstack-nova | 23:30 | |
*** jklare has joined #openstack-nova | 23:30 | |
*** jwcroppe has quit IRC | 23:31 | |
dansmith | tonyb: ohai. can haz? https://review.openstack.org/#/c/407750/ | 23:35 |
tonyb | dansmith: pass jenkins it must hmmm then look good it does | 23:36 |
* dansmith runs that through google translate | 23:37 | |
dansmith | oh, wait for jenkins? | 23:37 |
*** masber has quit IRC | 23:37 | |
tonyb | dansmith: I'm sad that my Yoda impression didn't work :( | 23:37 |
dansmith | tonyb: can haz when 'murica sleeps and jenkins is +1? | 23:37 |
tonyb | dansmith: Yup. | 23:38 |
dansmith | tonyb: thanks | 23:38 |
*** thorst_ has joined #openstack-nova | 23:39 | |
*** lpetrut has quit IRC | 23:42 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Extend get_all_by_filters to support resource criteria https://review.openstack.org/386242 | 23:46 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Expose a REST API for a specific list of RPs https://review.openstack.org/392569 | 23:46 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Support multiple uuids and names for RP list https://review.openstack.org/407630 | 23:46 |
bauzas | dansmith: jaypipes: edleafe: last call for https://review.openstack.org/392569, that's no longer a WIP | 23:47 |
bauzas | I wrote the gabbi tests | 23:47 |
*** thorst_ has quit IRC | 23:48 | |
*** ducttape_ has joined #openstack-nova | 23:51 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Expose a REST API for a specific list of RPs https://review.openstack.org/392569 | 23:58 |
*** catintheroof has joined #openstack-nova | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!