| *** yamamoto has quit IRC | 00:00 | |
| mriedem | q3 tag is up https://review.openstack.org/538072 | 00:00 |
|---|---|---|
| *** jmlowe has joined #openstack-nova | 00:00 | |
| *** liusheng has joined #openstack-nova | 00:01 | |
| *** yamamoto has joined #openstack-nova | 00:01 | |
| *** yamamoto has quit IRC | 00:01 | |
| *** yamamoto has joined #openstack-nova | 00:01 | |
| *** hshiina has joined #openstack-nova | 00:04 | |
| *** hrw has quit IRC | 00:05 | |
| *** yamamoto has quit IRC | 00:06 | |
| *** yamamoto has joined #openstack-nova | 00:07 | |
| jroll | mriedem: nice one, thanks | 00:07 |
| SpazZz | Will be heading on vacation for about 18 days here soon, any chance that I can get some core love on this sometime soon so it doesn't hit the wayside? https://review.openstack.org/#/c/533168/ | 00:09 |
| *** tetsuro has joined #openstack-nova | 00:12 | |
| tetsuro | is efried around? | 00:12 |
| tetsuro | good morning | 00:13 |
| *** chyka_ has quit IRC | 00:13 | |
| gmann | stephenfin: thanks for adding in ur review list | 00:14 |
| gmann | mriedem: thanks for tempest job, my only left question is can we enable multi-attach in any of the existing job? - https://review.openstack.org/#/c/532689/ | 00:15 |
| gmann | bcz new job you are adding as voting | 00:16 |
| *** gongysh has quit IRC | 00:16 | |
| *** jafeha__ has joined #openstack-nova | 00:17 | |
| gmann | ameeda: sorry i forgot to re-review, i will check today for sure | 00:18 |
| *** armax has quit IRC | 00:19 | |
| *** jafeha has quit IRC | 00:21 | |
| *** markvoelker has quit IRC | 00:22 | |
| gmann | mriedem: just read your cmt msg for devstack patch about Pike UCA pckg and qemu,l ibvirt version dependency, i think thats why you added separate new job ? | 00:23 |
| efried | tetsuro Hi, briefly around. What's up? | 00:29 |
| efried | mriedem Any point rechecking stuff now, or will it surely fail without the cinder thingy? | 00:31 |
| openstackgerrit | Eric Fried proposed openstack/nova master: New-style _set_inventory_for_provider https://review.openstack.org/537648 | 00:34 |
| openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821 | 00:34 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | 00:34 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260 | 00:34 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517 | 00:34 |
| efried | tetsuro Bailing out now. If you have the time and inclination, please feel free to recheck stuff all day :P | 00:36 |
| *** chyka has joined #openstack-nova | 00:36 | |
| *** rloo has quit IRC | 00:37 | |
| *** mlavalle has quit IRC | 00:39 | |
| efried | By the way tetsuro-さん, ありがとうございました for all of your excellent and detailed reviews. | 00:40 |
| tetsuro | efried: thank you so much for so kindful and gread explanation! | 00:40 |
| *** mlavalle has joined #openstack-nova | 00:40 | |
| tetsuro | どういたしまして、こちらこそありがとう！笑 | 00:40 |
| *** chyka has quit IRC | 00:40 | |
| efried | tetsuro You did point out a bad copy/paste, which I'll fix in the next rev. | 00:41 |
| efried | I'm really leaving now. See y'all tomorrow. | 00:41 |
| tetsuro | ack. | 00:41 |
| *** mlavalle has quit IRC | 00:41 | |
| tetsuro | have a good night! | 00:41 |
| *** tovin07 has quit IRC | 00:46 | |
| *** tovin07 has joined #openstack-nova | 00:47 | |
| *** zhaochao has joined #openstack-nova | 00:50 | |
| *** zhurong has joined #openstack-nova | 00:51 | |
| *** zhurong_ has joined #openstack-nova | 00:52 | |
| *** phuongnh has joined #openstack-nova | 01:01 | |
| *** mdnadeem has joined #openstack-nova | 01:07 | |
| *** hieulq__ has joined #openstack-nova | 01:07 | |
| *** armax has joined #openstack-nova | 01:09 | |
| *** hieulq_ has quit IRC | 01:10 | |
| *** stakeda has joined #openstack-nova | 01:10 | |
| *** kevzha01 has joined #openstack-nova | 01:17 | |
| gmann | mriedem: +1 on nova multiattach job. tempest one need rebase. rechecked job for testing the latest version of tempest test . ll review them accordingly | 01:18 |
| *** tiendc has joined #openstack-nova | 01:18 | |
| *** liuzz has joined #openstack-nova | 01:19 | |
| *** liuzz_ has joined #openstack-nova | 01:22 | |
| *** liuzz has quit IRC | 01:23 | |
| *** rmcall has joined #openstack-nova | 01:24 | |
| *** gongysh has joined #openstack-nova | 01:24 | |
| *** mdnadeem has quit IRC | 01:29 | |
| *** hrw has joined #openstack-nova | 01:32 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Log the events we timed out waiting for while plugging vifs https://review.openstack.org/538103 | 01:33 |
| *** annp has joined #openstack-nova | 01:40 | |
| *** markvoelker has joined #openstack-nova | 01:41 | |
| *** mdnadeem has joined #openstack-nova | 01:42 | |
| *** dave-mccowan has joined #openstack-nova | 01:46 | |
| *** Supun has joined #openstack-nova | 01:48 | |
| *** huanxie has joined #openstack-nova | 01:52 | |
| *** hongbin has quit IRC | 01:52 | |
| *** armax has quit IRC | 01:53 | |
| *** salv-orl_ has joined #openstack-nova | 02:02 | |
| *** fragatina has quit IRC | 02:05 | |
| *** salv-orlando has quit IRC | 02:05 | |
| *** tbachman has quit IRC | 02:10 | |
| *** karimull has joined #openstack-nova | 02:14 | |
| *** karimull has left #openstack-nova | 02:15 | |
| *** harlowja has quit IRC | 02:16 | |
| *** karimull has joined #openstack-nova | 02:16 | |
| *** huanxie has quit IRC | 02:17 | |
| *** huanxie has joined #openstack-nova | 02:18 | |
| *** huanxie has quit IRC | 02:18 | |
| openstackgerrit | Chen Hanxiao proposed openstack/nova master: test: Use assertEqual() instead of assertXXXEqual() https://review.openstack.org/533493 | 02:19 |
| *** huanxie has joined #openstack-nova | 02:22 | |
| *** Supun has quit IRC | 02:24 | |
| *** zhurong has quit IRC | 02:24 | |
| *** chyka has joined #openstack-nova | 02:26 | |
| SpazZz | Morning huanxie, i'm heading into work in about an hour, would love to get a chance to talk to you today :) | 02:29 |
| SpazZz | And thanks for the code review :D | 02:29 |
| huanxie | @SpazZz, sure | 02:30 |
| *** chyka has quit IRC | 02:31 | |
| *** swamireddy has joined #openstack-nova | 02:31 | |
| huanxie | There is a ticket about live migration with XenAPI driver https://bugs.launchpad.net/nova/+bug/1704071 and I have made a draft patch fixing this problem, we can have a discuss about this | 02:33 |
| openstack | Launchpad bug 1704071 in OpenStack Compute (nova) "XenAPI: volume VM live migration failed with VDI_NOT_IN_MAP" [Low,Confirmed] | 02:33 |
| SpazZz | Yes ma'am, that's exactly what i'd like to talk about | 02:33 |
| *** Supun has joined #openstack-nova | 02:35 | |
| *** jackie-truong has joined #openstack-nova | 02:35 | |
| huanxie | SpazZa, wait a moment, let me find some thing for you | 02:36 |
| *** bhujay has joined #openstack-nova | 02:39 | |
| *** mdnadeem_ has joined #openstack-nova | 02:41 | |
| SpazZz | hey huanxie I gotta get reayd to head to work. If you find something send it ot the Spazmotic nick.. that's my work IRC :) | 02:41 |
| SpazZz | Be back in just al ittle bit. | 02:41 |
| huanxie | Sure, can leave you message | 02:42 |
| *** mdnadeem has quit IRC | 02:44 | |
| gmann | mriedem: saw mail for q-3 tag. will we defer this to Rocky https://blueprints.launchpad.net/nova/+spec/api-extensions-merge-queens | 02:44 |
| mriedem | the remaining parts yeah | 02:45 |
| gmann | k | 02:46 |
| *** chohoor has joined #openstack-nova | 02:48 | |
| *** Supun has quit IRC | 02:50 | |
| jackie-truong | dansmith: are you still around? | 02:51 |
| *** chohoor has quit IRC | 02:51 | |
| *** amodi has quit IRC | 02:57 | |
| openstackgerrit | shuangyang.qian proposed openstack/nova master: Fix the metadata re to match the unicode https://review.openstack.org/536236 | 03:03 |
| *** mdnadeem_ has quit IRC | 03:04 | |
| openstackgerrit | Merged openstack/os-traits master: Update reno for stable/queens https://review.openstack.org/537512 | 03:14 |
| openstackgerrit | Merged openstack/os-vif master: Update reno for stable/queens https://review.openstack.org/537514 | 03:14 |
| *** mdnadeem_ has joined #openstack-nova | 03:16 | |
| *** crushil has joined #openstack-nova | 03:19 | |
| *** tuanla____ has joined #openstack-nova | 03:22 | |
| *** bhujay has quit IRC | 03:27 | |
| *** hongbin has joined #openstack-nova | 03:27 | |
| *** crushil has quit IRC | 03:32 | |
| *** bhujay has joined #openstack-nova | 03:35 | |
| *** chyka has joined #openstack-nova | 03:35 | |
| *** chyka has quit IRC | 03:41 | |
| *** pramodrj07 has quit IRC | 03:43 | |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949 | 03:47 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 03:47 |
| *** dave-mccowan has quit IRC | 03:49 | |
| *** bhujay has quit IRC | 03:53 | |
| *** huanxie has quit IRC | 03:53 | |
| *** rmcall has quit IRC | 03:53 | |
| *** Spazmotic has quit IRC | 04:00 | |
| *** Spazmotic-Work has joined #openstack-nova | 04:00 | |
| Spazmotic-Work | Aigoo~ stupid VPN | 04:00 |
| *** bhujay has joined #openstack-nova | 04:01 | |
| *** Spazmotic-Work is now known as Spazmotic | 04:01 | |
| openstackgerrit | Merged openstack/osc-placement master: Update reno for stable/queens https://review.openstack.org/537698 | 04:03 |
| *** hoangcx has quit IRC | 04:05 | |
| *** gongysh has quit IRC | 04:07 | |
| *** huanxie has joined #openstack-nova | 04:14 | |
| *** huanxie has quit IRC | 04:15 | |
| *** jackie-truong has quit IRC | 04:19 | |
| *** mriedem has quit IRC | 04:23 | |
| *** hshiina has quit IRC | 04:24 | |
| *** andreas_s has joined #openstack-nova | 04:25 | |
| *** andreas_s has quit IRC | 04:30 | |
| *** zhurong_ has quit IRC | 04:39 | |
| *** huanxie has joined #openstack-nova | 04:44 | |
| *** harlowja has joined #openstack-nova | 04:51 | |
| Spazmotic | Welcome back huanxie | 04:52 |
| Spazmotic | Got some time? | 04:52 |
| Spazmotic | Will put some KPOP and try to remember what all I had to ask you if you do hehe | 04:52 |
| *** openstackstatus has quit IRC | 04:56 | |
| *** openstackstatus has joined #openstack-nova | 04:58 | |
| *** ChanServ sets mode: +v openstackstatus | 04:58 | |
| *** chyka has joined #openstack-nova | 04:59 | |
| *** chyka has quit IRC | 05:03 | |
| *** gongysh has joined #openstack-nova | 05:11 | |
| *** hamzy has joined #openstack-nova | 05:12 | |
| *** gongysh has quit IRC | 05:16 | |
| *** huanxie has quit IRC | 05:17 | |
| *** zhurong has joined #openstack-nova | 05:18 | |
| *** huanxie has joined #openstack-nova | 05:20 | |
| *** claudiub has joined #openstack-nova | 05:23 | |
| *** fragatina has joined #openstack-nova | 05:31 | |
| *** fragatina has quit IRC | 05:32 | |
| *** fragatina has joined #openstack-nova | 05:32 | |
| openstackgerrit | Merged openstack/python-novaclient master: Update reno for stable/queens https://review.openstack.org/538061 | 05:36 |
| *** izombie has joined #openstack-nova | 05:38 | |
| *** izombie has left #openstack-nova | 05:38 | |
| *** hamzy has left #openstack-nova | 05:39 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: tests: fixes mock autospec usage https://review.openstack.org/447505 | 05:47 |
| *** links has joined #openstack-nova | 05:58 | |
| *** huanxie has quit IRC | 05:59 | |
| *** hongbin has quit IRC | 05:59 | |
| *** huanxie has joined #openstack-nova | 06:01 | |
| *** brault_ has quit IRC | 06:09 | |
| *** brault has joined #openstack-nova | 06:19 | |
| *** lpetrut has joined #openstack-nova | 06:25 | |
| *** takashin has left #openstack-nova | 06:33 | |
| *** psachin has joined #openstack-nova | 06:39 | |
| *** belmoreira has joined #openstack-nova | 06:42 | |
| *** gouthamr has joined #openstack-nova | 06:48 | |
| *** chyka has joined #openstack-nova | 06:48 | |
| *** zhurong has quit IRC | 06:52 | |
| *** chyka has quit IRC | 06:53 | |
| *** fragatina has quit IRC | 06:54 | |
| *** bhujay has quit IRC | 06:59 | |
| *** bhujay has joined #openstack-nova | 07:03 | |
| *** namnh has joined #openstack-nova | 07:03 | |
| *** andreas_s has joined #openstack-nova | 07:06 | |
| openstackgerrit | Merged openstack/nova master: PowerVM Driver: SEA https://review.openstack.org/523216 | 07:07 |
| openstackgerrit | Andreas Jaeger proposed openstack/nova master: Fix SUSE Install Guide: Placement port https://review.openstack.org/538163 | 07:08 |
| *** izombie has joined #openstack-nova | 07:11 | |
| *** lajoskatona has joined #openstack-nova | 07:17 | |
| *** tbachman has joined #openstack-nova | 07:21 | |
| *** sahid has joined #openstack-nova | 07:22 | |
| *** harlowja has quit IRC | 07:23 | |
| *** tbachman has quit IRC | 07:25 | |
| *** slaweq has joined #openstack-nova | 07:27 | |
| *** lpetrut has quit IRC | 07:27 | |
| *** tetsuro has quit IRC | 07:28 | |
| *** hrw has quit IRC | 07:29 | |
| *** slaweq has quit IRC | 07:31 | |
| *** slaweq has joined #openstack-nova | 07:33 | |
| *** slaweq has joined #openstack-nova | 07:35 | |
| *** lpetrut has joined #openstack-nova | 07:41 | |
| openstackgerrit | Ameed Ashour proposed openstack/nova master: detaching volumes when deleted VM is in error state https://review.openstack.org/528069 | 07:43 |
| openstackgerrit | Ameed Ashour proposed openstack/nova master: detach instance volumes when VM creation fails https://review.openstack.org/528385 | 07:43 |
| *** psachin has quit IRC | 07:46 | |
| *** izombie has quit IRC | 07:46 | |
| *** chyka has joined #openstack-nova | 07:48 | |
| *** gszasz has joined #openstack-nova | 07:50 | |
| *** chyka has quit IRC | 07:52 | |
| *** hoonetorg has quit IRC | 07:55 | |
| *** jafeha__ is now known as jafeha | 08:06 | |
| *** matrohon has joined #openstack-nova | 08:07 | |
| *** avolkov has joined #openstack-nova | 08:14 | |
| *** lpetrut has quit IRC | 08:16 | |
| *** jafeha has quit IRC | 08:21 | |
| *** jangutter has joined #openstack-nova | 08:22 | |
| *** jafeha has joined #openstack-nova | 08:24 | |
| *** amoralej|off is now known as amoralej | 08:24 | |
| *** stakeda has quit IRC | 08:24 | |
| *** hoonetorg has joined #openstack-nova | 08:25 | |
| *** bhujay has quit IRC | 08:26 | |
| Spazmotic | Andreas you in here? Any reason why you zvkm rechecked? | 08:31 |
| *** alexchadin has joined #openstack-nova | 08:33 | |
| Spazmotic | You making IBM hate me again hehe | 08:34 |
| *** gouthamr_ has joined #openstack-nova | 08:34 | |
| *** yamahata has joined #openstack-nova | 08:35 | |
| *** ralonsoh has joined #openstack-nova | 08:35 | |
| *** jafeha__ has joined #openstack-nova | 08:36 | |
| *** gouthamr has quit IRC | 08:36 | |
| *** jafeha has quit IRC | 08:38 | |
| *** AlexeyAbashkin has joined #openstack-nova | 08:39 | |
| *** gouthamr has joined #openstack-nova | 08:39 | |
| *** damien_r has joined #openstack-nova | 08:40 | |
| *** gouthamr_ has quit IRC | 08:41 | |
| *** alexchadin has quit IRC | 08:41 | |
| *** alexchadin has joined #openstack-nova | 08:42 | |
| *** jpena|off is now known as jpena | 08:43 | |
| *** liuyulong has joined #openstack-nova | 08:45 | |
| *** ttsiouts has quit IRC | 08:46 | |
| *** lpetrut has joined #openstack-nova | 08:46 | |
| *** liuyulong has quit IRC | 08:53 | |
| *** ttsiouts has joined #openstack-nova | 08:54 | |
| *** lpetrut has quit IRC | 08:55 | |
| *** zhaochao has quit IRC | 08:57 | |
| *** blkart has quit IRC | 08:58 | |
| *** lpetrut has joined #openstack-nova | 09:02 | |
| *** sahid has quit IRC | 09:05 | |
| *** sahid has joined #openstack-nova | 09:05 | |
| *** matrohon has quit IRC | 09:07 | |
| *** salv-orl_ has quit IRC | 09:07 | |
| *** lpetrut has quit IRC | 09:09 | |
| *** gszasz has quit IRC | 09:09 | |
| *** sahid has quit IRC | 09:10 | |
| *** sahid has joined #openstack-nova | 09:10 | |
| *** salv-orlando has joined #openstack-nova | 09:11 | |
| *** huanxie has quit IRC | 09:13 | |
| *** huanxie has joined #openstack-nova | 09:14 | |
| *** gszasz has joined #openstack-nova | 09:15 | |
| *** gszasz has quit IRC | 09:15 | |
| *** liusheng has quit IRC | 09:19 | |
| *** liusheng has joined #openstack-nova | 09:19 | |
| *** mgoddard_ has joined #openstack-nova | 09:20 | |
| *** liuyulong has joined #openstack-nova | 09:24 | |
| Spazmotic | man the tempest zuul and IBM failures plaguing me with snapshots failing to create is driving me into insanity | 09:26 |
| *** sambetts|afk is now known as sambetts | 09:27 | |
| *** yangyapeng has quit IRC | 09:32 | |
| *** yangyapeng has joined #openstack-nova | 09:35 | |
| *** gszasz has joined #openstack-nova | 09:36 | |
| *** chyka has joined #openstack-nova | 09:37 | |
| *** elmaciej has joined #openstack-nova | 09:38 | |
| *** derekh has joined #openstack-nova | 09:38 | |
| gmann | alex_xu: gibi stephenfin : any other optimized way to get system metadata ?- https://review.openstack.org/#/c/526900/22/nova/tests/functional/regressions/test_bug_1117923.py@124 | 09:41 |
| *** chyka has quit IRC | 09:42 | |
| *** chyka has joined #openstack-nova | 09:42 | |
| *** chyka has quit IRC | 09:42 | |
| *** bhujay has joined #openstack-nova | 09:44 | |
| gmann | ameade: ^^ let's see if other folk can tell better way | 09:45 |
| *** avolkov has quit IRC | 09:45 | |
| gibi | gmann: as far as I see there is no way to directly get the instace system_metadata via the nova REST_API | 09:50 |
| *** avolkov has joined #openstack-nova | 09:53 | |
| openstackgerrit | Merged openstack/nova master: Use neutron port_list when filtering instance by ip https://review.openstack.org/525505 | 09:53 |
| openstackgerrit | Merged openstack/nova master: Modify the test case of get_disk_mapping_rescue_with_config https://review.openstack.org/494156 | 09:53 |
| Spazmotic | nova hides that junk pretty well unfortunately from what i've tested in the past | 09:55 |
| *** alexchadin has quit IRC | 09:58 | |
| Spazmotic | Any UK cores here today? :) | 10:01 |
| *** lpetrut has joined #openstack-nova | 10:01 | |
| Spazmotic | Would love if one had some free cycles for a review. https://review.openstack.org/#/c/533168/4 . Zuul freaking out over some volume stuff but should go back to +1 here soon. Heading on vacation in about 9 days and don't want it to sit idle for 2 weeks | 10:02 |
| *** tuanla____ has quit IRC | 10:02 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Add 'move-nova-cmds-to-cliff' spec https://review.openstack.org/433603 | 10:03 |
| *** lpetrut has quit IRC | 10:04 | |
| openstackgerrit | lei zhang proposed openstack/nova master: Fix the incorrect RST convention https://review.openstack.org/538208 | 10:06 |
| *** lucas-afk is now known as lucasagomes | 10:07 | |
| *** mdnadeem_ has quit IRC | 10:08 | |
| *** namnh has quit IRC | 10:08 | |
| *** tovin07 has quit IRC | 10:09 | |
| openstackgerrit | Zengzhi Wang proposed openstack/nova stable/pike: Make eventlet hub use a monotonic clock https://review.openstack.org/537811 | 10:12 |
| jianghuaw | Spazmotic, I guess the zvkm rechecks were done automatically as there was an error in upstream package which caused some CI job failed. | 10:13 |
| Spazmotic | Maybe maybe.. now I Just gotta fight zuul hehe | 10:14 |
| *** liuzz_ has quit IRC | 10:15 | |
| jianghuaw | hehe. Good luck. | 10:15 |
| *** kevzha01 has quit IRC | 10:16 | |
| *** jangutter has quit IRC | 10:22 | |
| *** jangutter has joined #openstack-nova | 10:22 | |
| *** gouthamr has quit IRC | 10:25 | |
| *** gszasz has quit IRC | 10:26 | |
| Roamer` | Spazmotic, if you're seeing Zuul failures in volume backup tests, there's little point in rechecking until https://review.openstack.org/#/c/538027/ merges | 10:26 |
| Spazmotic | ahh something regress? Alrighty roamer, thanks | 10:27 |
| *** jangutter has quit IRC | 10:27 | |
| Roamer` | Spazmotic, yep, sporadic, not always, but quite annoying the last couple of days | 10:28 |
| *** yamamoto has quit IRC | 10:30 | |
| *** avolkov has quit IRC | 10:32 | |
| Roamer` | of course, once it merges, you'd wish you had rechecked early enough so that your patch is close to the head of the queue :P | 10:34 |
| *** jangutter has joined #openstack-nova | 10:39 | |
| *** bhujay has quit IRC | 10:39 | |
| *** bhujay has joined #openstack-nova | 10:39 | |
| *** rm_work has quit IRC | 10:42 | |
| *** yamahata has quit IRC | 10:43 | |
| *** itlinux has joined #openstack-nova | 10:46 | |
| *** rm_work has joined #openstack-nova | 10:46 | |
| *** yamamoto has joined #openstack-nova | 10:53 | |
| *** yamamoto has quit IRC | 10:58 | |
| *** huanxie has quit IRC | 11:01 | |
| *** yamamoto has joined #openstack-nova | 11:01 | |
| *** yamamoto has quit IRC | 11:01 | |
| *** phuongnh has quit IRC | 11:02 | |
| *** blkart has joined #openstack-nova | 11:02 | |
| *** fragatina has joined #openstack-nova | 11:15 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Integrate Mypy Type Checking https://review.openstack.org/538217 | 11:17 |
| stephenfin | cdent, mdbooth, sean-k-mooney: Would be interested in your thoughts on this. PoC changes to follow shortly https://review.openstack.org/538217 | 11:21 |
| *** itlinux has quit IRC | 11:21 | |
| *** johnthetubaguy has quit IRC | 11:26 | |
| *** johnthetubaguy has joined #openstack-nova | 11:28 | |
| *** gszasz has joined #openstack-nova | 11:41 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: tox: Add mypy target https://review.openstack.org/538221 | 11:41 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: tox: Store list of converted files https://review.openstack.org/538222 | 11:41 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Move __init__ function https://review.openstack.org/538223 | 11:41 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: mypy: Add type annotations to 'nova.pci' https://review.openstack.org/538224 | 11:41 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: mypy: Add 'types' module https://review.openstack.org/538225 | 11:41 |
| stephenfin | cdent, mdbooth, sean-k-mooney: Ta-da! ^ That was my evening's work 🙈 | 11:42 |
| Spazmotic | You did all that and i'm just over here smashing my face into xenapi live migrate | 11:43 |
| *** liuyulong has quit IRC | 11:44 | |
| johnthetubaguy | Spazmotic: xenapi live-migrate... whats up there? | 11:50 |
| Spazmotic | man john i've been crying your name for days :p | 11:50 |
| johnthetubaguy | Spazmotic: your welcome | 11:50 |
| Spazmotic | hehehehe | 11:50 |
| Spazmotic | Looks like it's not working in XS7+ due to the SR not existing during the assert_can_live_migrate for src -> destand XS7 being more insistent that it does. | 11:51 |
| johnthetubaguy | oh, they "fixed" that check | 11:51 |
| Spazmotic | Yep, so it would seem. It actually insists now and throws MAP errors | 11:52 |
| johnthetubaguy | yeah, we don't pass the SR map to the first check call, as it never used to do anything with it | 11:52 |
| Spazmotic | johnthetubaguy: by the by I manually added you to a review with Xenapi the other week, I hope you don't mind. | 11:52 |
| johnthetubaguy | that is no problem, although you might have to ping me to get to that, hoping to ramp my reviewing back up now things are more settled over here | 11:53 |
| Spazmotic | Well in this case it's for our iSCSI volumes only causing an issue since they don't exist yet on the dest during its first pass call through that method | 11:53 |
| Spazmotic | yeah I understand, I mentioned you a few times but figured yo uwere just busy | 11:53 |
| johnthetubaguy | yeah, but I think you will need to send the map for it to find them | 11:53 |
| Spazmotic | IT's pretty simple code but it directly involves things I know yo uworked on, so wanted your advice | 11:53 |
| Spazmotic | I agree, my only real thought here is how to handle it. My best thought now is to generate the SR a little earlier in the code line | 11:54 |
| johnthetubaguy | its a nasty one honestly, as the work isn't done until much later to create the volumes on the dest, as I am sure you saw | 11:54 |
| johnthetubaguy | the problem is Cinder really, that extra connection has tracking info that doesn't get updated till after the check phase | 11:54 |
| johnthetubaguy | the other (very unfortunate but quick) approach is to move the check later, before you call the live-migration | 11:55 |
| Spazmotic | Without a major overhaul, the only thing I was able to think of that wasn't too gross was to use connect_to_volume provider at the destination check. IT would require that code pulling BDMS though.. which doesn't feel right. | 11:55 |
| Spazmotic | at least that way the SR exists, but doesn't need to attach the volume yet..and it generates the map without error. | 11:55 |
| johnthetubaguy | yeah, those BDMs are wrong at that point | 11:55 |
| johnthetubaguy | you need the specific connection params for the destination hypervisor, which could be different credentails, in the general case | 11:56 |
| Spazmotic | It would literally just be a throw away of the variable afterwards, which is a waste | 11:56 |
| Spazmotic | That's true that data is stored in the SR isn't it. | 11:57 |
| Spazmotic | I guess it's still workable since bdms pulls conn_data anyway | 11:58 |
| Spazmotic | But still not right | 11:58 |
| *** andreas_s has quit IRC | 11:58 | |
| johnthetubaguy | hmm, not sure I understand what you mean, I thought each SR would have different passwords in conn_data, potentially, not sure we ever tested that mind | 11:58 |
| *** andreas_s has joined #openstack-nova | 11:58 | |
| johnthetubaguy | Spazmotic: basically, you need this code to run *before* you attach the SR from the bdm: https://github.com/openstack/nova/blob/d0d0262379a57230e546cddf233da8012d3fe5b1/nova/compute/manager.py#L5889 | 12:00 |
| *** elmaciej has quit IRC | 12:00 | |
| johnthetubaguy | That means due to the common code the earliest you can attach the SRs on the destination is this path: https://github.com/openstack/nova/blob/d0d0262379a57230e546cddf233da8012d3fe5b1/nova/compute/manager.py#L5944 | 12:01 |
| Spazmotic | Yeah I saw that and agree as well unless I duplicated the bdms pull, which was why I say it's wrong | 12:01 |
| Spazmotic | re-pulling bdms in the check call to be able to call the attachments.. which sucks | 12:01 |
| johnthetubaguy | yeah, the bdms just don't have the info in them you think they do, if its done before that call | 12:02 |
| *** yamamoto has joined #openstack-nova | 12:02 | |
| Spazmotic | Ah no? are they modified before then? Didn't think that would change. | 12:02 |
| Spazmotic | Ahhhh i see | 12:02 |
| Spazmotic | I didn't read the loop close enough.. damn | 12:02 |
| johnthetubaguy | they can be modified by this call: https://github.com/openstack/nova/blob/d0d0262379a57230e546cddf233da8012d3fe5b1/nova/compute/manager.py#L5905 | 12:03 |
| johnthetubaguy | cool | 12:03 |
| *** andreas_s has quit IRC | 12:03 | |
| Spazmotic | The other option is to clean up sort out Huan Xie's code.. which just eats the exception and forms the map manually | 12:04 |
| Spazmotic | So.. lots to mush face against on it | 12:04 |
| Spazmotic | nothing really graceful showing up in my mind yet | 12:05 |
| Spazmotic | Could push the bdm attachment setting into its own method to call twice.. but that's just adjusting the code to fit this specific situation. | 12:05 |
| johnthetubaguy | its more we can't call that till the pre-live-migrate stage, else all the error handling is screwed up | 12:06 |
| Spazmotic | mmhmm.. I can see that | 12:07 |
| Spazmotic | Just needs more thought then I think | 12:07 |
| *** bkopilov has quit IRC | 12:08 | |
| openstackgerrit | Ameed Ashour proposed openstack/nova master: detaching volumes when deleted VM is in error state https://review.openstack.org/528069 | 12:09 |
| openstackgerrit | Ameed Ashour proposed openstack/nova master: detach instance volumes when VM creation fails https://review.openstack.org/528385 | 12:09 |
| openstackgerrit | Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900 | 12:09 |
| mdbooth | stephenfin: You can't spit-shine a turd ;) | 12:09 |
| mdbooth | stephenfin: Thanks for trying, though. | 12:09 |
| johnthetubaguy | Spazmotic: so the quickest fix is to skip the check_can_live-migrate for XS 7+ | 12:09 |
| Spazmotic | I was wondering if we could do that but I could not think of a way | 12:10 |
| johnthetubaguy | Spazmotic: that might not be as bad as it first sounds, given how the API has changed | 12:10 |
| Spazmotic | We still do need the data from the source check, but if we could skip the assertation that would be ideal. | 12:10 |
| Spazmotic | I guess I could use the XenAPI host calls to make a switch | 12:11 |
| *** chyka has joined #openstack-nova | 12:11 | |
| johnthetubaguy | Spazmotic: so you can call it right before where we call live_migrate, if the data is missing in migrate data? | 12:11 |
| johnthetubaguy | Spazmotic: downside is failed live-migrations will take much longer to fail, and involve more work to rollback (disconnect the volumes) | 12:11 |
| johnthetubaguy | but given you need to connect the volume first now anyway, its maybe not a big deal | 12:12 |
| ameeda | gmann: gibi: is this ready for merge ? https://review.openstack.org/#/c/526900/ | 12:12 |
| johnthetubaguy | Spazmotic: so I think I am on watch the baby duties in a few mins, over lunch, so maybe catch you later on | 12:12 |
| Spazmotic | So we could potentially still generate the map and just have it just not run an assertation if the XS version is high.. but yea it's not a bad idea | 12:13 |
| Spazmotic | worth looking into for usre.. I appreciate the brain dump man | 12:13 |
| Spazmotic | and good luck with the baby :) | 12:13 |
| *** blkart has quit IRC | 12:13 | |
| johnthetubaguy | I may need it :) | 12:13 |
| *** yamamoto has quit IRC | 12:13 | |
| Spazmotic | Make your baby review my code.. will keep htem busy :p | 12:14 |
| johnthetubaguy | Spazmotic: yeah, that works, generate the map, but skip the check check call based on version (there is some structures in place for version skips already) | 12:14 |
| *** andreas_s has joined #openstack-nova | 12:14 | |
| *** dave-mccowan has joined #openstack-nova | 12:14 | |
| Spazmotic | Awesome.. maybe i'll look how those are done nad see if i can keep it standard | 12:15 |
| Spazmotic | If you happen to know one feel free to let me know when you get a cahnce so I can take a look | 12:15 |
| *** chyka has quit IRC | 12:15 | |
| *** annp has quit IRC | 12:21 | |
| Spazmotic | Ah and johnthetubaguy since my irc nme is wierd.. the commit i added you to is https://review.openstack.org/#/c/533168/4.. Ignore zuul.. we fighting right now :P | 12:22 |
| *** salv-orlando has quit IRC | 12:26 | |
| *** salv-orlando has joined #openstack-nova | 12:26 | |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova master: WIP: VMware: implement update_provider_tree() https://review.openstack.org/536348 | 12:27 |
| *** matrohon has joined #openstack-nova | 12:29 | |
| *** salv-orlando has quit IRC | 12:31 | |
| *** rgerganov has joined #openstack-nova | 12:33 | |
| rgerganov | efried, is it safe to assume that when update_provider_tree is called, provider_tree.roots[0] points to the root RP of the compute node? | 12:35 |
| *** salv-orlando has joined #openstack-nova | 12:38 | |
| *** jpena is now known as jpena|lunch | 12:39 | |
| *** sahid has quit IRC | 12:42 | |
| *** kholkina has joined #openstack-nova | 12:44 | |
| Spazmotic | Nice.. host_rec['software_version']['product_version'].. super easy | 12:45 |
| *** hamzy has joined #openstack-nova | 12:46 | |
| *** matrohon has quit IRC | 12:47 | |
| openstackgerrit | Merged openstack/nova master: Add the Nova libvirt StorPool attachment driver. https://review.openstack.org/140733 | 12:47 |
| kholkina | Hi. I use this patch https://review.openstack.org/#/c/530224/ to update user-data. It works fine on 169.254.169.254/2009-04-04/user-data, but doesn't work when I use config drive. What should I do to update it on config-drive too? | 12:48 |
| openstackgerrit | Merged openstack/nova master: Transform instance-evacuate notification https://review.openstack.org/482148 | 12:48 |
| *** sahid has joined #openstack-nova | 12:49 | |
| *** sahid has quit IRC | 12:50 | |
| *** sahid has joined #openstack-nova | 12:51 | |
| *** gszasz has quit IRC | 12:51 | |
| *** gszasz has joined #openstack-nova | 12:51 | |
| *** janki has joined #openstack-nova | 12:51 | |
| artom | kholkina, the config drive is essentially write-once, at instance creation. | 12:52 |
| artom | kholkina, whatever metadata the instance was created with, that's what's on the config drive, and it can't be updated | 12:52 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: DNM: test if tempest-full is triggered or not https://review.openstack.org/537936 | 13:01 |
| *** gcb has quit IRC | 13:02 | |
| kholkina | artom, ok, thanks | 13:03 |
| *** bkero- has joined #openstack-nova | 13:04 | |
| bkero- | Hi there, does anybody know if it's possible to get the nova-cli to output in a different format? Sort of like OSC's -f and -c options | 13:05 |
| artom | bkero-, if there is, nova --help should tell you :) | 13:06 |
| bkero- | I did not see it in there, which is why I came here to ask | 13:07 |
| *** jackie-truong has joined #openstack-nova | 13:07 | |
| openstackgerrit | Merged openstack/nova master: tests: Use correct response type in tests https://review.openstack.org/527182 | 13:08 |
| *** tiendc has quit IRC | 13:09 | |
| artom | bkero-, then we don't do it :) Were you hoping for some secret options we only tell to the select few? ;) | 13:12 |
| *** edmondsw has joined #openstack-nova | 13:13 | |
| bkero- | pretty much | 13:14 |
| *** jackie-truong has quit IRC | 13:14 | |
| bkero- | some environment variable or undocumented flag | 13:14 |
| bkero- | Or "oh, we use this library for our cli binary, and you'd be setting this env var/flag according to it" | 13:15 |
| artom | bkero-, hehe :) | 13:15 |
| artom | bkero-, I'm not the ultimate expert on this, but I've never come across something like that | 13:16 |
| Spazmotic | Heading out for the day. Will be on at home for a bit, hit me up on SpazZz if you need anything | 13:21 |
| *** liverpooler has joined #openstack-nova | 13:24 | |
| *** tssurya has quit IRC | 13:25 | |
| *** matrohon has joined #openstack-nova | 13:26 | |
| *** gszasz has quit IRC | 13:26 | |
| *** gszasz has joined #openstack-nova | 13:29 | |
| *** gszasz has quit IRC | 13:30 | |
| *** gszasz has joined #openstack-nova | 13:31 | |
| *** gszasz has quit IRC | 13:33 | |
| *** gszasz has joined #openstack-nova | 13:33 | |
| *** ameeda has quit IRC | 13:33 | |
| *** liusheng has quit IRC | 13:40 | |
| *** gszasz has quit IRC | 13:43 | |
| *** itlinux has joined #openstack-nova | 13:45 | |
| *** jpena|lunch is now known as jpena | 13:45 | |
| *** fragatina has quit IRC | 13:47 | |
| *** bkopilov has joined #openstack-nova | 13:51 | |
| bkero- | Hm, is there any way to clear out 'nova migration-list'? | 13:51 |
| *** Tom-Tom has joined #openstack-nova | 13:56 | |
| *** edleafe is now known as figleaf | 14:01 | |
| *** mriedem has joined #openstack-nova | 14:02 | |
| *** stephenfin is now known as finucannot | 14:05 | |
| gibi | mriedem: FYI I'm getting lost in zuul job definitions: http://lists.openstack.org/pipermail/openstack-dev/2018-January/126616.html | 14:07 |
| mriedem | crack that nut | 14:09 |
| *** amoralej is now known as amoralej|lunch | 14:09 | |
| mriedem | gibi: this is the last versioned notification transformation patch i'm tracking for queens https://review.openstack.org/#/c/465081/ so you could update https://review.openstack.org/#/c/518018/ based on that | 14:13 |
| *** lyan has joined #openstack-nova | 14:13 | |
| sean-k-mooney | finucannot: qq i dont have time to read mypy stuff right now (tyring to do the refactor before i the end of the day) but is it related in any way to https://www.python.org/dev/peps/pep-0484/ | 14:14 |
| finucannot | sean-k-mooney: One and the same | 14:14 |
| finucannot | sean-k-mooney: Only I'm using Python 2-compatible comments for now. It could be converted once Python 2 support is dropped, but that's a minimum of two years off | 14:15 |
| sean-k-mooney | finucannot: from what i saw in your pathcs however it looked like mypy did it via comments not fuction changes | 14:15 |
| sean-k-mooney | ah ok so mypy support both | 14:15 |
| finucannot | Correct http://mypy.readthedocs.io/en/latest/python2.html | 14:16 |
| sean-k-mooney | comments for backwards compatiblity and greeting(name: str) -> str: for native python 3 | 14:16 |
| *** rmcall has joined #openstack-nova | 14:16 | |
| finucannot | sean-k-mooney: Python 3.4+ (I think) | 14:16 |
| finucannot | Thought 3.3 is EOL now so that doesn't matter | 14:16 |
| finucannot | *though | 14:16 |
| sean-k-mooney | ok cool i did like that pep but did i did not know there was a python2 compaible way e.g. i taught we would have to wait for openstack to become python3 only. | 14:17 |
| finucannot | sean-k-mooney: It's pretty neat. I've seen it used in Sphinx and was thinking it would help reviewing ralonsoh's PCI spec object patch, so I tried it out | 14:17 |
| sean-k-mooney | ill add them all to my review list as ya this is somthing i would be in favor of. | 14:17 |
| *** tssurya has joined #openstack-nova | 14:17 | |
| *** hoonetorg has quit IRC | 14:18 | |
| *** esberglu has joined #openstack-nova | 14:18 | |
| sean-k-mooney | im also wondering is there a way we could integrate mypy with ovo but that for another day | 14:18 |
| finucannot | sean-k-mooney: Cool :) There's a spec there too because I think this would take a while to do (not that we need to check everything - only the most egregious stuff) so that's probably worth a look at | 14:19 |
| efried | rgerganov No | 14:19 |
| *** bhujay has quit IRC | 14:19 | |
| efried | rgerganov Also, we really don't want you to be accessing _Provider members directly. | 14:19 |
| sean-k-mooney | ya i was going to start with the spec then read the ptaches you wrote once i had the context for why you were doing it :) | 14:20 |
| finucannot | sean-k-mooney: And o.v.o uses a lot of magic and can definitely wait for a while :) I have started looking at oslo.config though | 14:20 |
| finucannot | Mostly to test out pyannotate, mind | 14:20 |
| efried | rgerganov I think the best way to find the root of the compute node tree is to use the nodename parameter we pass you in update_provider_tree. | 14:21 |
| efried | rgerganov ...as a parm to .exists, .data, .has_*_changed, .update_*, etc. | 14:21 |
| rgerganov | efried, then I think we need a find() method in the provider tree | 14:22 |
| efried | rgerganov You can't have one :) | 14:22 |
| efried | rgerganov We used to have it, but we took it out. Tell me what you are trying to do, what you need it for. | 14:22 |
| rgerganov | efried, how do I find the provider for the compute node? iterate over all roots? | 14:23 |
| efried | rgerganov From update_provider_tree? We pass you a nodename parameter. That ought to do the trick. | 14:23 |
| efried | rgerganov All the ProviderTree methods accept a name_or_uuid to identify the provider you're looking for. | 14:24 |
| rgerganov | efried, no it does not | 14:24 |
| efried | rgerganov Can you tell me a little more about what you're trying to do with the root provider record? | 14:25 |
| rgerganov | efried, first of all I want to remove all existing childs and add them again to avoid complicated synchronizations | 14:25 |
| rgerganov | efried, https://review.openstack.org/#/c/536348/4/nova/virt/vmwareapi/driver.py | 14:25 |
| rgerganov | efried, sorry, this is the latest one https://review.openstack.org/#/c/536348/5/nova/virt/vmwareapi/driver.py | 14:25 |
| *** lucasagomes is now known as lucas-hungry | 14:26 | |
| efried | rgerganov And this doesn't do the right thing? ==> provider_tree.update_inventory(nodename, root_rp_inv, None) | 14:26 |
| rgerganov | efried, but how do I add a new child? | 14:26 |
| rgerganov | I need the uuid of the parent | 14:27 |
| rgerganov | not the name | 14:27 |
| efried | oic, yeah. Probably wouldn't be a horrible idea for us to change that to name_or_uuid. Let me check if there's any reason we can't do that... | 14:27 |
| *** bhujay has joined #openstack-nova | 14:27 | |
| efried | rgerganov But in the meantime, you can use provider_tree.data(nodename).uuid | 14:27 |
| rgerganov | efried, ok, what about removing all the childs without using _Provider methods | 14:28 |
| rgerganov | is this OK? root_rp.children.clear() | 14:28 |
| efried | no, you'll have to do something like: | 14:29 |
| efried | descendants = set(provider_tree.get_provider_uuids(nodename)) - set([provider_tree.data(nodename).uuid]); for desc_uuid in reversed(descendants): provider_tree.remove(desc_uuid) | 14:30 |
| rgerganov | why so complicated? | 14:31 |
| *** itlinux has quit IRC | 14:31 | |
| *** Supun has joined #openstack-nova | 14:31 | |
| *** hoonetorg has joined #openstack-nova | 14:32 | |
| *** itlinux has joined #openstack-nova | 14:32 | |
| *** izombie has joined #openstack-nova | 14:32 | |
| izombie | artom: are you around? | 14:33 |
| efried | rgerganov Well, to be perfectly honest, we didn't anticipate that you would want to prune all descendants of the root. I'm still not convinced it's really necessary, but if it is, there's a way to do it. | 14:33 |
| efried | rgerganov As far as why we don't let you have access to the internals, it's for thread safety. | 14:33 |
| rgerganov | efried, but the virt driver get a copy of the tree, right? | 14:34 |
| efried | rgerganov Keep in mind that we've designed this thing with a certain set of use cases vaguely in mind. As real consumers start really consuming it, and we encounter real use cases that run up against awkwardness, we can sure look into adding convenience methods for oft-seen usages. | 14:35 |
| *** david-lyle has quit IRC | 14:35 | |
| * rgerganov real consumer :) | 14:35 | |
| efried | rgerganov In update_provider_tree, yes, you don't have to worry about thread safety on the provider_tree parameter. You own it. But we don't have a separate ProviderTreeWhereThreadSafetyIsNotAnIssue class at the moment. | 14:35 |
| rgerganov | ok | 14:36 |
| *** itlinux has quit IRC | 14:36 | |
| artom | izombie, yeah | 14:36 |
| efried | up to the point of that in-flight patch, SchedulerReportClient was the only thing using ProviderTree, and it does need to worry about thread safety (sort of :) | 14:36 |
| izombie | The issue I was speaking of yesterday. | 14:37 |
| izombie | artom: It was with the project-id, it wasn't according to the new regex pattern ;) | 14:37 |
| artom | izombie, err, refresh my memory? | 14:37 |
| izombie | artom: was trying to list nova servers and I was hitting 404.. | 14:37 |
| artom | izombie, oh, right | 14:38 |
| efried | rgerganov Feel like proposing that change for new_child to accept name_or_uuid for the parent? | 14:38 |
| rgerganov | efried, yes | 14:38 |
| *** yamamoto has joined #openstack-nova | 14:38 | |
| rgerganov | efried, I will submit patch for it | 14:38 |
| artom | izombie, err, cool? I'm not sure what you mean by new regex pattern | 14:38 |
| efried | rgerganov You could *almost* cheat and pass in the name as it stands :) But that won't quiiite work all the way. | 14:39 |
| rgerganov | efried, I also think that removing all child providers is pretty valid use case | 14:39 |
| *** r-daneel has joined #openstack-nova | 14:39 | |
| izombie | artom: so project mapper has introduced some change in the way project id's are supposed to be | 14:39 |
| *** yamamoto has quit IRC | 14:39 | |
| efried | rgerganov I would need to understand more about why that's necessary. And in any case you'll have to get it past Jay :) | 14:39 |
| efried | Which reminds me... | 14:39 |
| *** efried is now known as fried_rice | 14:39 | |
| fried_rice | It's FRIDAY! | 14:40 |
| artom | izombie, I'll be honest, I'm not following | 14:40 |
| izombie | artom: the regex expects a hex sort of value, but our's was simple characters. hence the project id was never fetched and directly passed to routes to find a match | 14:40 |
| *** rgerganov is now known as radio_gaga | 14:40 | |
| fried_rice | niiice | 14:40 |
| radio_gaga | fried_rice, you familiar with the vmware driver? | 14:40 |
| fried_rice | radio_gaga Not at all, sorry. | 14:40 |
| radio_gaga | fried_rice, basically one nova-compute manges a compute cluster in vcenter server | 14:41 |
| izombie | artom: okay, this code might help you gain context https://github.com/openstack/nova/blob/master/nova/api/openstack/__init__.py#L156-L158 | 14:41 |
| radio_gaga | fried_rice, a compute cluster may be divided into resource pools | 14:41 |
| *** eharney has joined #openstack-nova | 14:41 | |
| radio_gaga | fried_rice, now we want to create a nested RP for each resource pool under the compute cluster managed by nova | 14:41 |
| fried_rice | radio_gaga And you don't have n-cpu running on those other ("slave"?) nodes in the cluster? | 14:41 |
| *** dansmith is now known as superdan | 14:41 | |
| *** yamamoto has joined #openstack-nova | 14:41 | |
| radio_gaga | fried_rice, no | 14:42 |
| fried_rice | k | 14:42 |
| radio_gaga | fried_rice, the thing is that resource pools can be dynamic | 14:42 |
| fried_rice | what's a "resource pool"? | 14:42 |
| artom | izombie, ah, thanks - first time looking at that bit of the API code :) | 14:42 |
| radio_gaga | fried_rice, it's just a group of compute resources | 14:42 |
| radio_gaga | fried_rice, the point is that those resource pools can by dynamic | 14:42 |
| fried_rice | radio_gaga Okay, so the hierarchy is: compute-host-where-service-is-running => resource-pool => slave-compute-node | 14:43 |
| radio_gaga | fried_rice, and it would be a lot easier if we delete all child RP and create them again instead of trying to synchronize what's in the provider_tree and what's in vcenter | 14:43 |
| izombie | artom: :) Necessity! | 14:44 |
| fried_rice | radio_gaga Well, you could always provider_tree.remove(nodename) | 14:45 |
| fried_rice | and then rebuild the whole shebang including the root. | 14:46 |
| fried_rice | .remove(nodename) will implicitly whack everything under him. | 14:46 |
| radio_gaga | fried_rice, hm, actually that will do the job | 14:46 |
| radio_gaga | fried_rice, cool thanks | 14:46 |
| *** matrohon has quit IRC | 14:47 | |
| radio_gaga | fried_rice, I will try it just in case :) | 14:47 |
| fried_rice | radio_gaga But have you considered having all your compute nodes as roots, and tagging them with traits to identify their resource pool? | 14:47 |
| *** claudiub has quit IRC | 14:47 | |
| *** claudiub|2 has joined #openstack-nova | 14:47 | |
| radio_gaga | fried_rice, having them under one root is a better model because this is the same model in vCenter; you divide one big compute cluster into resource pools | 14:48 |
| *** elmaciej has joined #openstack-nova | 14:49 | |
| *** jaypipes has joined #openstack-nova | 14:49 | |
| *** ctrath has joined #openstack-nova | 14:49 | |
| *** jaypipes is now known as leakypipes | 14:49 | |
| *** tesseract has joined #openstack-nova | 14:49 | |
| fried_rice | radio_gaga As you wish, just another possibility. Or have the slaves all be children directly under the root, and still tag 'em with resource pool trait name. | 14:50 |
| *** lajoskatona has quit IRC | 14:50 | |
| *** ameeda has joined #openstack-nova | 14:50 | |
| radio_gaga | fried_rice, yeah, that's what I am trying to accomplish right now | 14:50 |
| fried_rice | radio_gaga Point is, then you don't need to whack all the pools and rebuild them - you can just walk the children and reassign their traits. | 14:51 |
| leakypipes | radio_gaga: awesome Friday nick :) | 14:51 |
| radio_gaga | leakypipes, thanks :) | 14:51 |
| radio_gaga | leakypipes, we were discussing changing provider_tree.new_child to accept name_or_uuid instead of uuid | 14:52 |
| fried_rice | leakypipes IMO an oversight, easily corrected. Agree? | 14:52 |
| fried_rice | leakypipes correction: parent_name_or_uuid instead of parent_uuid | 14:53 |
| radio_gaga | yeah, right | 14:53 |
| leakypipes | fried_rice: how about just "parent" | 14:54 |
| fried_rice | sure | 14:54 |
| leakypipes | fried_rice: and make it accept either. | 14:54 |
| *** ctrath has left #openstack-nova | 14:54 | |
| ameeda | can you please review this when you available ? https://review.openstack.org/#/c/528385/ | 14:54 |
| fried_rice | radio_gaga is going to propose it | 14:54 |
| *** amoralej|lunch is now known as amoralej | 14:56 | |
| *** r-daneel has quit IRC | 15:01 | |
| *** damien_r has quit IRC | 15:01 | |
| *** izombie has quit IRC | 15:01 | |
| *** awaugama has joined #openstack-nova | 15:02 | |
| tssurya | superdan, mriedem, melwitt : regarding the solution we discussed for nova list displaying only the uuid when a cell DB is down, there is this issue that we end up displaying all the instance uuids in that cell (including the deleted ones)...since we take it from the mappings, :( | 15:02 |
| tssurya | any ideas on how to differentiate the deleted ones using api info ? | 15:03 |
| melwitt | there isn't a way | 15:03 |
| superdan | mm, yep, that's not really avoidable unfortunately | 15:03 |
| leakypipes | ameeda: those are bug fixes. we can get to those as needed over next few weeks. | 15:03 |
| *** salv-orlando has quit IRC | 15:03 | |
| tssurya | melwitt, superdan : ok :( | 15:03 |
| * superdan points to his readonly replica idea under his breath | 15:04 | |
| *** salv-orlando has joined #openstack-nova | 15:04 | |
| *** brault has quit IRC | 15:04 | |
| *** brault has joined #openstack-nova | 15:04 | |
| *** jaosorior has quit IRC | 15:04 | |
| *** janki has quit IRC | 15:05 | |
| leakypipes | JGrg543#1 | 15:06 |
| superdan | nice | 15:06 |
| *** tbachman has joined #openstack-nova | 15:06 | |
| belmoreira | superdan that isn't good | 15:06 |
| leakypipes | oops. now y'all know my verizon laptop password. | 15:06 |
| *** gszasz has joined #openstack-nova | 15:07 | |
| ameeda | leakpipes: if you can review it for me as you can, to let me know if that is good or not. | 15:07 |
| superdan | belmoreira: referring to leakypipes' password or the deleted instances thing? | 15:07 |
| *** leakypipes is now known as leakypassword | 15:07 | |
| *** sahid has quit IRC | 15:07 | |
| SpazZz | Perfect | 15:08 |
| *** sahid has joined #openstack-nova | 15:08 | |
| belmoreira | :) referring to the readonly replica idea | 15:08 |
| radio_gaga | I am going to +2 all my patches with jay's account | 15:08 |
| *** salv-orlando has quit IRC | 15:08 | |
| SpazZz | We handing out +2s? | 15:08 |
| SpazZz | Will pay in baked chicken | 15:08 |
| leakypassword | radio_gaga: good luck :) it's only for my mac laptop (which I hate and don't use for anything) ;) | 15:08 |
| *** gszasz has quit IRC | 15:08 | |
| *** leakypassword is now known as leakypipes | 15:09 | |
| SpazZz | password has been secured. Pipe is again at risk | 15:09 |
| leakypipes | heh | 15:10 |
| *** yamahata has joined #openstack-nova | 15:12 | |
| *** gibi is now known as giblet | 15:12 | |
| leakypipes | superdan, mdbooth, mriedem: if someone is using qcow2 images in glance, with ceph as the backing store, what is the most efficient way to do instance snapshots? is it setting snapshot_image_format=raw or is there some ceph-specific efficient snapshotting method? | 15:13 |
| sean-k-mooney | finucannot: leakypipes superdan just noticed we are at/past feature freeze can https://blueprints.launchpad.net/nova/+spec/enable-sriov-nic-features get a FFE i ame working on the last change finucannot asked for currently so i think this will be done this evening | 15:13 |
| melwitt | leakypipes: you have to configure raw image format in glance | 15:14 |
| *** amodi has joined #openstack-nova | 15:14 | |
| melwitt | http://docs.ceph.com/docs/master/rbd/rbd-openstack/ | 15:15 |
| leakypipes | sean-k-mooney: I don't have a problem with it, but mriedem is the decider-in-chief | 15:16 |
| *** gcb has joined #openstack-nova | 15:17 | |
| *** tbachman has quit IRC | 15:17 | |
| leakypipes | melwitt: lol, I love how the ceph docs refer to a thing called "OpenStack" -- as if it were a single thing. | 15:17 |
| leakypipes | "Install OpenStack" | 15:17 |
| leakypipes | heh | 15:17 |
| melwitt | yeah, totally simple | 15:17 |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova master: ProviderTree.new_child: parent is either uuid or name https://review.openstack.org/538264 | 15:19 |
| jroll | leakypipes: https://i.imgur.com/rCr9A.png | 15:19 |
| sean-k-mooney | leakypipes: :) ok ill send a mail to the list requestion the FFE mriedem i hope you dont mind if i cc you. im currently working on the changes now so the final version should be up before i leave today unless there are new review comments. | 15:19 |
| melwitt | leakypipes: you probably need snapshot_image_format = raw too if that will dictate what format the image ends up in glance. the root point is that the image format in glance must be raw for ceph's native cow clone stuff to work | 15:19 |
| radio_gaga | leakypipes, fried_rice https://review.openstack.org/#/c/538264 | 15:19 |
| superdan | jroll: love it | 15:20 |
| jroll | :D | 15:21 |
| *** tbachman has joined #openstack-nova | 15:21 | |
| *** mlavalle has joined #openstack-nova | 15:21 | |
| kashyap | Can anyone explain what exactly is 'nova live-igration-force-complete' doing under the hood? /me goes to look at source meanwhile | 15:21 |
| *** lpetrut has joined #openstack-nova | 15:21 | |
| *** mdnadeem_ has joined #openstack-nova | 15:21 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: reno for notification-transformation-queens https://review.openstack.org/518018 | 15:22 |
| giblet | mriedem: ^^ | 15:22 |
| leakypipes | jroll: lol, that's awesome | 15:22 |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova master: ProviderTree.new_child: parent is either uuid or name https://review.openstack.org/538264 | 15:22 |
| *** gcb has quit IRC | 15:23 | |
| * kashyap is not the one for GIFs, but this "reaction" captures live migration: http://imagizer.imageshack.us/a/img633/4417/HnWM6A.gif | 15:24 | |
| *** slaweq has quit IRC | 15:24 | |
| fried_rice | radio_gaga Small test addition please. | 15:24 |
| *** slaweq has joined #openstack-nova | 15:25 | |
| mriedem | sean-k-mooney: too late for that one | 15:27 |
| mriedem | leakypipes: there is a direct snapshot thing you can configure with the rbd image backend | 15:27 |
| mriedem | metacloud added it a few years ago | 15:27 |
| *** tidwellr has joined #openstack-nova | 15:28 | |
| mriedem | kashyap: post-copy i think | 15:28 |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova master: ProviderTree.new_child: parent is either uuid or name https://review.openstack.org/538264 | 15:29 |
| kashyap | mriedem: It is related, but not entirely specific to PostCopy, near as I see. | 15:29 |
| *** slaweq has quit IRC | 15:29 | |
| kashyap | I'm looking at this document: doc/source/admin/live-migration-usage.rst | 15:29 |
| melwitt | leakypipes, mriedem: oh yeah. the direct snapshot also involves exposing a direct image URL in glance | 15:29 |
| kashyap | And it says: | 15:29 |
| melwitt | trying to find docs for this | 15:29 |
| mriedem | kashyap: any docs on live migratoin in nova are probably super old | 15:29 |
| *** matrohon has joined #openstack-nova | 15:30 | |
| kashyap | mriedem: Okay, I'll add a TODO to update that, actually | 15:30 |
| mriedem | leakypipes: melwitt: https://specs.openstack.org/openstack/nova-specs/specs/mitaka/implemented/rbd-instance-snapshots.html | 15:30 |
| radio_gaga | fried_rice, done | 15:30 |
| mriedem | the ceph CI job is also configured to make this work ^ | 15:30 |
| *** dave-mccowan has quit IRC | 15:31 | |
| melwitt | mriedem: yeah, just found that. I would have thought there was some kinda operator docs about this somewhere | 15:31 |
| melwitt | apparently not | 15:31 |
| *** lucas-hungry is now known as lucasagomes | 15:31 | |
| melwitt | not finding anything on the googles | 15:31 |
| superdan | kashyap: it pauses the instance to stop the dirtying I think | 15:32 |
| sean-k-mooney | mriedem: ok ill see if i can continue working on it until early rocky | 15:32 |
| mriedem | melwitt: well, if there was, the operator docs were all deleted | 15:32 |
| mriedem | melwitt: but i doubt it | 15:33 |
| kashyap | superdan: At least the existing doc says (12 lines of paste upcoming): | 15:33 |
| mriedem | the direct image / location url stuff is semi-not recommended | 15:33 |
| kashyap | - **Manually force-complete the migration** | 15:33 |
| kashyap | .. code-block:: console | 15:33 |
| kashyap | $ nova live-migration-force-complete INSTANCE_ID MIGRATION_ID | 15:33 |
| kashyap | The instance is paused until memory copy completes. | 15:33 |
| kashyap | .. caution:: | 15:33 |
| kashyap | Since the pause impacts time keeping on the instance and not all | 15:33 |
| mriedem | because it exposes the location of the image | 15:33 |
| kashyap | applications tolerate incorrect time settings, use this approach with | 15:33 |
| kashyap | caution. | 15:33 |
| kashyap | superdan: So your guess seems right | 15:34 |
| superdan | kashyap: it wasn't really a guess, but ... okay :) | 15:35 |
| kashyap | superdan: I like how you're a stickler for words. | 15:35 |
| *** david-lyle has joined #openstack-nova | 15:37 | |
| *** damien_r has joined #openstack-nova | 15:39 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: tests: fixes mock autospec usage https://review.openstack.org/447505 | 15:47 |
| *** Tom-Tom has quit IRC | 15:54 | |
| *** slaweq has joined #openstack-nova | 15:56 | |
| *** Spazmotic has quit IRC | 15:59 | |
| *** slaweq_ has joined #openstack-nova | 16:00 | |
| *** slaweq has quit IRC | 16:00 | |
| *** armax has joined #openstack-nova | 16:00 | |
| *** Spazmotic has joined #openstack-nova | 16:01 | |
| *** links has quit IRC | 16:05 | |
| *** gouthamr has joined #openstack-nova | 16:05 | |
| *** jackie-truong has joined #openstack-nova | 16:06 | |
| *** AlexeyAbashkin has quit IRC | 16:06 | |
| mriedem | figleaf: ooo alternate hosts for resize is finally in the gate queue | 16:06 |
| mriedem | progress | 16:06 |
| lpetrut | mriedem: hi, quick question about nova docs: do you know by any chance if config option tables are still being used? e.g. https://github.com/openstack/openstack-manuals/blob/stable/newton/doc/config-reference/source/tables/nova-vnc.rst | 16:08 |
| mriedem | yes | 16:09 |
| mriedem | https://docs.openstack.org/nova/latest/configuration/sample-config.html | 16:09 |
| mriedem | https://docs.openstack.org/nova/latest/configuration/config.html | 16:09 |
| mriedem | lpetrut: those docs are auto-generated from the config options in code | 16:09 |
| lpetrut | got it, the only "issue" with that is that you no longer have separate tables that you may reference throughout the docs | 16:09 |
| *** kholkina has quit IRC | 16:10 | |
| *** slaweq_ has quit IRC | 16:10 | |
| mriedem | so like linking directly to hyperv options? | 16:10 |
| lpetrut | yeah, for example | 16:10 |
| leakypipes | sahid: I see you are assigned to https://bugs.launchpad.net/nova/+bug/1729584. Did you have in-progress code on that? Seems like a trivial fix (from sileht) | 16:10 |
| openstack | Launchpad bug 1729584 in OpenStack Compute (nova) "boot from volume + configdrive broken" [Undecided,Confirmed] - Assigned to sahid (sahid-ferdjaoui) | 16:10 |
| mriedem | lpetrut: https://docs.openstack.org/nova/latest/configuration/config.html#hyperv | 16:10 |
| mriedem | tada! | 16:10 |
| lpetrut | hmm, I guess that works as well. Previously, you could've embedded the tables on other doc pages, e.g. https://docs.openstack.org/cinder/latest/configuration/block-storage/drivers/lvm-volume-driver.html | 16:12 |
| *** bhujay has quit IRC | 16:12 | |
| *** Yikun has quit IRC | 16:13 | |
| figleaf | mriedem: 11 hours and still just sitting there. Maybe it will get through by the end of Rocky :) | 16:13 |
| *** Yikun has joined #openstack-nova | 16:13 | |
| fried_rice | figleaf Based on the past few days, it'll take another 12h or so to fail. | 16:14 |
| giblet | figleaf: I have patches in gate queue for more than 17 hours without any job being executed on them | 16:14 |
| mriedem | patience, | 16:15 |
| mriedem | at least the volume backup thing is fixed (again) | 16:15 |
| superdan | heh top nova change has been there for 20h | 16:15 |
| fried_rice | figleaf When you're asleep, so it's another 8h before you can recheck if nobody else gets to it. | 16:15 |
| sahid | leakypipes: nothing so far | 16:15 |
| *** belmoreira has quit IRC | 16:17 | |
| leakypipes | sahid: k. mind if I take it? | 16:18 |
| sahid | leakypipes: please do | 16:18 |
| mriedem | that community-wide mox goal will kill us with test_neutronv2.py alone | 16:18 |
| leakypipes | sahid: k, cheers | 16:18 |
| *** radio_gaga has quit IRC | 16:22 | |
| figleaf | fried_rice: oh, no, I'm planning on staying up all night staring at the Zuul status page | 16:22 |
| fried_rice | figleaf Tonight and Saturday night? Thanks! | 16:23 |
| figleaf | fried_rice: All weekend!! | 16:23 |
| *** melwitt is now known as jgwentworth | 16:23 | |
| fried_rice | mriedem So does http://lists.openstack.org/pipermail/openstack-dev/2018-January/126621.html mean we can continue approving new function? | 16:26 |
| mriedem | not in my opinion | 16:26 |
| mriedem | http://lists.openstack.org/pipermail/openstack-dev/2018-January/126600.html | 16:27 |
| mriedem | personally i'm spent, we have a bunch of stuff already approved, | 16:27 |
| mriedem | and 2 weeks to FF | 16:27 |
| mriedem | *RC1 | 16:27 |
| mriedem | so we're going to have bugs creep up from what gets merged over the weekend | 16:27 |
| fried_rice | Swhat I figured, and I probably would have said the same in your position, but it was worth a shot :) | 16:28 |
| *** bhujay has joined #openstack-nova | 16:28 | |
| mriedem | i don't want to be busting ass to merge more stuff that will add more bugs | 16:28 |
| mriedem | ack | 16:28 |
| *** tidwellr has quit IRC | 16:28 | |
| mriedem | i've already deferred several bps this morning anyway | 16:28 |
| *** tidwellr has joined #openstack-nova | 16:28 | |
| *** gouthamr has quit IRC | 16:29 | |
| *** gyee has joined #openstack-nova | 16:31 | |
| *** ttsiouts has quit IRC | 16:34 | |
| *** damien_r has quit IRC | 16:34 | |
| *** andreas_s has quit IRC | 16:34 | |
| *** andreas_s has joined #openstack-nova | 16:35 | |
| *** andreas_s has quit IRC | 16:35 | |
| *** itlinux has joined #openstack-nova | 16:35 | |
| *** andreas_s has joined #openstack-nova | 16:35 | |
| *** gouthamr has joined #openstack-nova | 16:36 | |
| sean-k-mooney | mriedem: oh on that am is it ok to propose the feature bases schduling spec for rocky since its defered or would you like me to hold off until ater the release to re propose the spec | 16:39 |
| mriedem | re-proposing specs for rocky right now is fine | 16:39 |
| *** andreas_s has quit IRC | 16:40 | |
| *** itlinux has quit IRC | 16:42 | |
| *** armax has quit IRC | 16:44 | |
| sean-k-mooney | mriedem: cool will do i know we need to focus on getting queens out the door so dont want to cause un needed churn. ill upoald the latest version fo the code too and set a -1 on it unill master reopens for rocky after the PTG thanks :) | 16:46 |
| *** armax has joined #openstack-nova | 16:46 | |
| *** mdnadeem_ has quit IRC | 16:47 | |
| *** chyka has joined #openstack-nova | 16:48 | |
| *** matrohon has quit IRC | 16:51 | |
| *** gouthamr has quit IRC | 16:51 | |
| *** gouthamr has joined #openstack-nova | 16:55 | |
| *** amodi has quit IRC | 16:55 | |
| *** jackie-truong has quit IRC | 16:57 | |
| *** kukacz has quit IRC | 17:02 | |
| mriedem | python-pcre must be new? | 17:03 |
| *** dave-mccowan has joined #openstack-nova | 17:05 | |
| mriedem | not sure what's pulling that in, but now i need libpcre3-dev installed | 17:06 |
| *** harlowja has joined #openstack-nova | 17:10 | |
| leakypipes | mriedem: just noticed that too... | 17:10 |
| fried_rice | Not something I ever thought I would be happy to say, but my patches are now failing in a couple of hours. | 17:12 |
| *** sahid has quit IRC | 17:12 | |
| figleaf | fried_rice: efficient! | 17:12 |
| *** yamahata has quit IRC | 17:12 | |
| fried_rice | The gate is officially giving me the finger: build-openstack-sphinx-docs finger://ze05.openstack.org/c5b1d14f3fa944388c1e6d0979203466 : POST_FAILURE in 6m 55s | 17:12 |
| *** elmaciej has quit IRC | 17:13 | |
| *** mlavalle has quit IRC | 17:14 | |
| *** amodi has joined #openstack-nova | 17:14 | |
| *** fried_rice is now known as fried_rolls | 17:15 | |
| *** fragatina has joined #openstack-nova | 17:15 | |
| *** liverpooler has quit IRC | 17:16 | |
| *** mlavalle has joined #openstack-nova | 17:16 | |
| *** pchavva has joined #openstack-nova | 17:20 | |
| *** hoonetorg has quit IRC | 17:20 | |
| *** jangutter has quit IRC | 17:21 | |
| *** itlinux has joined #openstack-nova | 17:25 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: only increment disk address unit for scsi devices https://review.openstack.org/538310 | 17:25 |
| *** salv-orlando has joined #openstack-nova | 17:25 | |
| cfriesen | is anyone doing performance testing release-over-release? We just did a test where launching an instance took over 3x as long on Pike as it did on Newton. Still working to characterize. | 17:27 |
| *** Guest14094 has quit IRC | 17:30 | |
| *** jpena is now known as jpena|off | 17:30 | |
| *** amodi has quit IRC | 17:31 | |
| *** david-lyle has quit IRC | 17:32 | |
| leakypipes | cfriesen: nope, not that I know of. | 17:32 |
| leakypipes | cfriesen: just make sure you're accounting for all the many things that may have changed, including versions of libvirt/QEMU, operating system, etvc | 17:33 |
| *** tesseract has quit IRC | 17:35 | |
| *** hoonetorg has joined #openstack-nova | 17:36 | |
| *** pchavva1 has joined #openstack-nova | 17:37 | |
| *** itlinux has quit IRC | 17:38 | |
| *** pchavva has quit IRC | 17:38 | |
| *** fragatina has quit IRC | 17:38 | |
| *** AlexeyAbashkin has joined #openstack-nova | 17:40 | |
| *** AlexeyAbashkin has quit IRC | 17:45 | |
| *** slaweq has joined #openstack-nova | 17:46 | |
| *** gouthamr has quit IRC | 17:47 | |
| *** gouthamr has joined #openstack-nova | 17:49 | |
| *** slaweq_ has joined #openstack-nova | 17:49 | |
| *** slaweq has quit IRC | 17:50 | |
| openstackgerrit | James E. Blair proposed openstack/nova master: DNM: test if tempest-full is triggered or not https://review.openstack.org/537936 | 17:50 |
| *** esberglu has quit IRC | 17:53 | |
| *** Tom-Tom has joined #openstack-nova | 17:55 | |
| *** yamahata has joined #openstack-nova | 17:56 | |
| *** jackie-truong has joined #openstack-nova | 17:57 | |
| *** yamamoto has quit IRC | 17:58 | |
| *** Tom-Tom has quit IRC | 17:59 | |
| openstackgerrit | Sen Yang proposed openstack/python-novaclient master: Implement hypervisor hostname exact patten match for server cold migrate https://review.openstack.org/520187 | 17:59 |
| *** gouthamr_ has joined #openstack-nova | 17:59 | |
| *** rmcall has quit IRC | 18:01 | |
| *** links has joined #openstack-nova | 18:02 | |
| *** gouthamr has quit IRC | 18:02 | |
| *** lpetrut has quit IRC | 18:03 | |
| *** hongbin has joined #openstack-nova | 18:04 | |
| *** itlinux has joined #openstack-nova | 18:05 | |
| *** mgoddard_ has quit IRC | 18:06 | |
| *** AlexeyAbashkin has joined #openstack-nova | 18:09 | |
| *** mriedem has quit IRC | 18:13 | |
| *** AlexeyAbashkin has quit IRC | 18:14 | |
| *** mvenesio has joined #openstack-nova | 18:19 | |
| *** yamamoto has joined #openstack-nova | 18:19 | |
| *** penick has joined #openstack-nova | 18:20 | |
| *** itlinux has quit IRC | 18:21 | |
| *** pramodrj07 has joined #openstack-nova | 18:24 | |
| *** pramodrj07 has quit IRC | 18:24 | |
| *** pramodrj07 has joined #openstack-nova | 18:25 | |
| *** imacdonn has quit IRC | 18:26 | |
| *** imacdonn has joined #openstack-nova | 18:27 | |
| *** jafeha__ has quit IRC | 18:29 | |
| *** jafeha__ has joined #openstack-nova | 18:29 | |
| jgwentworth | leakypipes: we've got a scheduler host manager bug fix up at https://review.openstack.org/#/c/533371 which would much appreciate your review | 18:29 |
| leakypipes | jgwentworth: on it. | 18:30 |
| *** jafeha__ has quit IRC | 18:30 | |
| *** jafeha__ has joined #openstack-nova | 18:30 | |
| leakypipes | jgwentworth: trade ya for https://review.openstack.org/538310 | 18:30 |
| jgwentworth | leakypipes: sure thing | 18:31 |
| *** links has quit IRC | 18:32 | |
| *** jackie-truong is now known as fullmetaljackiet | 18:32 | |
| jgwentworth | leakypipes: I'm wondering if there's more to do on that one, see comment 2 on the bug https://bugs.launchpad.net/nova/+bug/1729584 | 18:33 |
| openstack | Launchpad bug 1729584 in OpenStack Compute (nova) "boot from volume + configdrive broken (regression)" [High,In progress] - Assigned to Jay Pipes (jaypipes) | 18:33 |
| *** tssurya has quit IRC | 18:34 | |
| jgwentworth | I don't quite understand it yet. saying that the secondary volumes can end up as LUN 0 which results in an unbootable instance. IIUC that's even after applying the patch mentioned in comment 0 | 18:34 |
| *** fragatina has joined #openstack-nova | 18:36 | |
| *** jafeha__ has quit IRC | 18:37 | |
| *** jafeha__ has joined #openstack-nova | 18:37 | |
| *** armax_ has joined #openstack-nova | 18:37 | |
| jgwentworth | maybe that's just residual from before the patch was applied | 18:39 |
| jgwentworth | logan-: hey, are you around? | 18:39 |
| *** armax has quit IRC | 18:40 | |
| *** armax_ is now known as armax | 18:40 | |
| *** weshay is now known as weshay|ruck|brb | 18:43 | |
| *** pchavva1 has quit IRC | 18:45 | |
| *** mgoddard_ has joined #openstack-nova | 18:46 | |
| *** esberglu has joined #openstack-nova | 18:48 | |
| *** itlinux has joined #openstack-nova | 18:49 | |
| *** lucasagomes is now known as lucas-afk | 18:49 | |
| *** itlinux has quit IRC | 18:50 | |
| *** yamamoto has quit IRC | 18:52 | |
| *** jafeha has joined #openstack-nova | 18:54 | |
| *** jafeha__ has quit IRC | 18:54 | |
| *** ralonsoh has quit IRC | 18:58 | |
| *** bhujay has quit IRC | 18:58 | |
| *** Supun has quit IRC | 18:59 | |
| *** fullmetaljackiet has quit IRC | 19:04 | |
| *** sambetts is now known as sambetts|afk | 19:04 | |
| *** jafeha has quit IRC | 19:05 | |
| *** mriedem has joined #openstack-nova | 19:05 | |
| *** jafeha has joined #openstack-nova | 19:05 | |
| *** slaweq_ has quit IRC | 19:05 | |
| mriedem | wtf now i'm getting this | 19:05 |
| mriedem | File "/home/user/git/nova/.tox/py27/local/lib/python2.7/site-packages/enum/__init__.py", line 230, in __new__ | 19:05 |
| mriedem | enum_member = __new__(enum_class, *args) | 19:05 |
| mriedem | OverflowError: Python int too large to convert to C long | 19:05 |
| mriedem | with unit tests | 19:05 |
| mriedem | cfriesen: figure out the performance issue? | 19:05 |
| *** yamamoto has joined #openstack-nova | 19:07 | |
| cfriesen | mriedem: not yet. we're going to do a comparison of right before/after the OpenStack version switch to remove as many factors as possible. | 19:08 |
| cfriesen | mriedem: things as basic as nova-compute downloading an image from glance take more than twice as long | 19:08 |
| *** david-lyle has joined #openstack-nova | 19:08 | |
| sean-k-mooney | mriedem: damb you are overflowing a long in a enum that is bizar | 19:09 |
| mriedem | sean-k-mooney: wasn't a problem until i had to install libpcre3-dev | 19:09 |
| *** jafeha has quit IRC | 19:09 | |
| *** jafeha has joined #openstack-nova | 19:09 | |
| *** comstud has quit IRC | 19:10 | |
| sean-k-mooney | maybe they chaged the type form a long long but still why would you ever need an enum that high or with that many values. there must be something else going on | 19:10 |
| mriedem | i haven't done any other apt-get updates either | 19:11 |
| mriedem | so shouldn't be pulling in the new python2.7 for xenial | 19:11 |
| sean-k-mooney | mriedem: you didnt install any new pip pakages or downgrade them did you. pip really does not handel downgrades well so i have seen random issue in the past becase of that | 19:12 |
| mriedem | no, only thing was a tox -r -e py27 | 19:12 |
| mriedem | and then it was trying to install python-pcre | 19:12 |
| *** yamamoto has quit IRC | 19:12 | |
| mriedem | and i had to install libpcre3-dev for that to work | 19:12 |
| *** harlowja has quit IRC | 19:14 | |
| sean-k-mooney | mriedem: were you looking at a partcalar patch. can an try pulling it donw in a vm i dont care too much about and see if i get teh same issue | 19:15 |
| mriedem | Collecting python-pcre===0.7 (from -c https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt (line 440)) | 19:15 |
| sean-k-mooney | that said ill be heading home in 45 mins to an hour so wont be arround too much longer | 19:15 |
| mriedem | sean-k-mooney: no i'm trying to write unit tests for something | 19:15 |
| sean-k-mooney | hum strange looks like there has not been an u-c bump to that for 2 months | 19:16 |
| *** jafeha has quit IRC | 19:17 | |
| mriedem | i don't know what's pulling it in | 19:17 |
| *** jafeha has joined #openstack-nova | 19:17 | |
| mriedem | aha! | 19:19 |
| mriedem | dhellmann's whereto thing | 19:19 |
| mriedem | http://git.openstack.org/cgit/openstack/whereto/tree/requirements.txt#n6 | 19:19 |
| *** penick_ has joined #openstack-nova | 19:20 | |
| sean-k-mooney | so it got pulled in form https://github.com/openstack/nova/commit/cec0faa7111ce93b5f99622b8e6174d66b25b6d0 | 19:20 |
| Roamer` | mriedem, yeah, yesterday something started depending on python-pcre and it caused some problems for most third-party CI systems that didn't have libpcre3-dev installed (although it had been in requirements/bindep.txt for quite some time), but now it's really being used | 19:20 |
| *** links has joined #openstack-nova | 19:21 | |
| mriedem | yes it's that whereto change | 19:21 |
| mriedem | asking doug in -release | 19:21 |
| *** jafeha has quit IRC | 19:21 | |
| Roamer` | right, I saw your links, but I'd already started writing that :) sorry | 19:21 |
| Roamer` | but still your problem looks..... very funny | 19:21 |
| *** jafeha has joined #openstack-nova | 19:22 | |
| *** penick_ has quit IRC | 19:22 | |
| sean-k-mooney | well looking at https://pypi.python.org/pypi/python-pcre it has not had a release since 2015 so it looks pretty dead | 19:22 |
| sean-k-mooney | its also not tested with python 3.5 | 19:23 |
| sean-k-mooney | i belived python had pretty good regex support natively does where to really need PCRE | 19:25 |
| *** READ10 has joined #openstack-nova | 19:26 | |
| jroll | I was wondering that too, but the commit message makes sense: https://github.com/openstack/whereto/commit/797cb67a37d67e41b2caa57aeeb6307b75b66bc8 | 19:26 |
| mriedem | i commented out whereto, rebuilt and still hitting this overflow error... | 19:27 |
| *** weshay|ruck|brb is now known as weshay|ruck | 19:27 | |
| jroll | well, that enum certainly is bigger than a C long (32 bits) https://github.com/openstack/nova/blob/master/nova/console/rfb/auth.py#L40 | 19:28 |
| jroll | but that code is 6 months old | 19:28 |
| sean-k-mooney | jroll: weel a long is not normally 32bit | 19:28 |
| sean-k-mooney | and it is | 19:29 |
| mriedem | jroll: that code just merged within the last 2 weeks, | 19:29 |
| sean-k-mooney | but now a long is typically 128bits | 19:29 |
| mriedem | the commit was written 6 months ago | 19:29 |
| jroll | ah | 19:29 |
| jroll | sean-k-mooney: oh right, 32 bits is the minimum in the spec | 19:29 |
| sean-k-mooney | jroll: on a 32 bit arch yes on 64bit a long is at least the size of an int so minium would be 64bit | 19:30 |
| jroll | nod | 19:31 |
| mriedem | this is a 32 bit vm i guess, so that's my problem | 19:32 |
| *** pramodrj07 has quit IRC | 19:32 | |
| jroll | woo | 19:32 |
| * jroll wonders if anyone runs nova on a 32-bit machine out there | 19:33 | |
| jroll | :P | 19:34 |
| *** lpetrut has joined #openstack-nova | 19:34 | |
| Roamer` | sean-k-mooney, actually it depends on the architecture, there are some really crazy ones, like e.g. x86_64, where a long is 64 bits, but an int is still 32 bits :) | 19:35 |
| Roamer` | (this one really caught me off-guard some years ago when I was writing something quite close to the hardware and didn't even think to check sizeof(int) until a coworker pointed it out to me) | 19:36 |
| *** tidwellr has quit IRC | 19:36 | |
| *** tidwellr_ has joined #openstack-nova | 19:36 | |
| *** jafeha has quit IRC | 19:37 | |
| *** jafeha has joined #openstack-nova | 19:37 | |
| openstackgerrit | Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/537171 | 19:41 |
| *** gouthamr_ has quit IRC | 19:44 | |
| *** jafeha has quit IRC | 19:45 | |
| *** salv-orlando has quit IRC | 19:45 | |
| *** jafeha has joined #openstack-nova | 19:45 | |
| *** salv-orlando has joined #openstack-nova | 19:45 | |
| *** harlowja has joined #openstack-nova | 19:46 | |
| *** jafeha has quit IRC | 19:49 | |
| *** jafeha has joined #openstack-nova | 19:49 | |
| *** pramodrj07 has joined #openstack-nova | 19:50 | |
| *** salv-orlando has quit IRC | 19:50 | |
| *** Pramod has joined #openstack-nova | 19:51 | |
| *** jafeha has quit IRC | 19:53 | |
| *** jafeha has joined #openstack-nova | 19:53 | |
| *** mvk has joined #openstack-nova | 19:53 | |
| *** pramodrj07 has quit IRC | 19:55 | |
| *** penick has quit IRC | 19:56 | |
| *** penick has joined #openstack-nova | 19:57 | |
| *** harlowja_ has joined #openstack-nova | 19:58 | |
| *** fried_rolls is now known as fried_rice | 19:58 | |
| *** gouthamr has joined #openstack-nova | 20:00 | |
| *** harlowja has quit IRC | 20:00 | |
| *** jafeha has quit IRC | 20:01 | |
| *** jafeha has joined #openstack-nova | 20:01 | |
| *** Pramod has quit IRC | 20:02 | |
| *** pramodrj07 has joined #openstack-nova | 20:02 | |
| *** slaweq has joined #openstack-nova | 20:08 | |
| openstackgerrit | Sen Yang proposed openstack/python-novaclient master: Implement hypervisor hostname exact patten match for server cold migrate https://review.openstack.org/520187 | 20:08 |
| *** jafeha has quit IRC | 20:13 | |
| *** jafeha has joined #openstack-nova | 20:13 | |
| *** yamamoto has joined #openstack-nova | 20:15 | |
| *** jafeha has quit IRC | 20:17 | |
| *** jafeha has joined #openstack-nova | 20:17 | |
| *** chris___ has joined #openstack-nova | 20:17 | |
| chris___ | Hey Guys, does anyone know how to disable non-admin users from taking snapshots via horizon in openstack juno? I'm looking in the /etc/nova/policy.json file, but I don't see anything related to compute:snapshots, only compute:volume_snapshot_create | 20:17 |
| *** mvenesio has quit IRC | 20:18 | |
| *** mvenesio has joined #openstack-nova | 20:18 | |
| jgwentworth | chris___: I think it's called create_image | 20:23 |
| *** matrohon has joined #openstack-nova | 20:23 | |
| *** mvenesio has quit IRC | 20:23 | |
| sean-k-mooney | Roamer`: actully yes but x86_64 even depends on the os i was reading this a while back. https://wiki.osdev.org/X86-64#Data_Types so linux should have a 64 bit long on x86_64. if you are on power or arm totally different. that is why c++ 11 and c 11 standards introduced the fixed with integer types http://en.cppreference.com/w/cpp/types/integer | 20:25 |
| jgwentworth | hm, looks like back in juno it wasn't controllable via policy | 20:25 |
| sean-k-mooney | Roamer`: anyway time to head home enjoy your weekend. | 20:26 |
| chris___ | jgwentworth: Thanks for the response, but i'm not seeing that parameter inside of the nova policy.json file | 20:28 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle network-changed event for a specific port https://review.openstack.org/465787 | 20:28 |
| mriedem | cfriesen: finally added tests for this ^ | 20:28 |
| jgwentworth | chris___: yeah, I posted another message saying back in juno it looks like it wasn't controllable via policy. I see it was possible starting in liberty | 20:28 |
| sean-k-mooney | chris___: since it apprently was not supported. if you run nova's api under appache or have a loadplancer in front of it you might need to add a redirect rule that will block the api endpoint form public ips. | 20:30 |
| sean-k-mooney | chris___: you should be able to match on the url and return a 404 or other message at the webserver level before it hits the openstack api as a workaround | 20:31 |
| *** READ10 has quit IRC | 20:31 | |
| *** yamamoto has quit IRC | 20:32 | |
| chris___ | jgwentworth: sean-k-mooney: Thanks for the replies, i might look into that for the time being | 20:34 |
| *** chris___ has quit IRC | 20:41 | |
| *** gouthamr has quit IRC | 20:43 | |
| *** links has quit IRC | 20:43 | |
| jgwentworth | mriedem: I've been noticing frequent timeouts on the functional test job. I was thinking, I wonder if we should split the api samples into their own job? I already feel like they should be removed from a normal 'tox -e functional' run considering they have their own tox target 'api-samples' | 20:44 |
| mriedem | https://review.openstack.org/#/c/537933/ | 20:45 |
| *** jafeha has quit IRC | 20:45 | |
| mriedem | ^ increases functional job timeout, but doesn't split them up | 20:46 |
| *** jafeha has joined #openstack-nova | 20:46 | |
| *** salv-orlando has joined #openstack-nova | 20:46 | |
| *** rmcall has joined #openstack-nova | 20:46 | |
| jgwentworth | mriedem: yeah, we should definitely do that. I was just wondering if it might help to split them up longer term | 20:47 |
| *** salv-orlando has quit IRC | 20:50 | |
| mriedem | yeah maybe, | 20:50 |
| mriedem | i know i don't run the full functional very often b/c of the api-samples tests | 20:50 |
| jgwentworth | same | 20:53 |
| jgwentworth | I was just thinking we could a) reduce developer pain for running non-api-sample functional tests b) keep the timeouts more sane in the future | 20:54 |
| *** matrohon has quit IRC | 20:55 | |
| *** rmcall has quit IRC | 20:59 | |
| *** slaweq has quit IRC | 21:22 | |
| *** slaweq has joined #openstack-nova | 21:22 | |
| *** slaweq has quit IRC | 21:27 | |
| *** sdague has quit IRC | 21:29 | |
| *** Pramod has joined #openstack-nova | 21:31 | |
| *** pramodrj07 has quit IRC | 21:35 | |
| *** dave-mccowan has quit IRC | 21:36 | |
| *** mgoddard_ has quit IRC | 21:37 | |
| *** tianhui_ has joined #openstack-nova | 21:41 | |
| *** david-lyle has quit IRC | 21:42 | |
| *** dklyle has joined #openstack-nova | 21:43 | |
| *** tianhui has quit IRC | 21:43 | |
| *** esberglu has quit IRC | 21:44 | |
| *** PramodJ has joined #openstack-nova | 21:44 | |
| *** PramodJ has quit IRC | 21:44 | |
| *** slaweq has joined #openstack-nova | 21:44 | |
| *** pramodrj07 has joined #openstack-nova | 21:45 | |
| *** Pramod has quit IRC | 21:45 | |
| *** salv-orlando has joined #openstack-nova | 21:46 | |
| *** salv-orlando has quit IRC | 21:51 | |
| *** esberglu has joined #openstack-nova | 22:03 | |
| *** amoralej is now known as amoralej|off | 22:10 | |
| *** amoralej|off is now known as amoralej | 22:11 | |
| *** esberglu has quit IRC | 22:11 | |
| *** elmaciej has joined #openstack-nova | 22:12 | |
| *** salv-orlando has joined #openstack-nova | 22:12 | |
| *** amoralej is now known as amoralej|off | 22:14 | |
| *** ircuser-1 has quit IRC | 22:18 | |
| *** kukacz has joined #openstack-nova | 22:19 | |
| jgwentworth | the 'increase functional test job timeout' change timed out | 22:25 |
| jgwentworth | on the py35 functional test job | 22:25 |
| *** ircuser-1 has joined #openstack-nova | 22:27 | |
| jgwentworth | and there are about a billion policy deprecated messages in the job output | 22:28 |
| jgwentworth | I wonder if the deprecation messages are related to the timeouts | 22:34 |
| *** esberglu has joined #openstack-nova | 22:41 | |
| *** lpetrut has quit IRC | 22:44 | |
| *** esberglu has quit IRC | 22:46 | |
| *** lyan has quit IRC | 22:46 | |
| *** esberglu has joined #openstack-nova | 22:53 | |
| mriedem | superdan has a patch to fix that | 22:53 |
| mriedem | er superlance | 22:53 |
| mriedem | https://review.openstack.org/#/c/537603/ | 22:53 |
| jgwentworth | already sent them toward the gate | 22:54 |
| mriedem | i've been busy getting a new dev vm setup this afternoon... | 22:55 |
| mriedem | after all the 32-bit shame from earlier | 22:55 |
| jgwentworth | haha | 22:55 |
| *** tidwellr_ has quit IRC | 22:56 | |
| *** esberglu has quit IRC | 22:58 | |
| *** burt has quit IRC | 23:01 | |
| *** edmondsw has quit IRC | 23:07 | |
| penick | Someone took my junoman nick, dang | 23:07 |
| *** penick is now known as JunOcataGuy | 23:08 | |
| *** slaweq has quit IRC | 23:09 | |
| *** slaweq has joined #openstack-nova | 23:09 | |
| *** lbragstad is now known as elbragstad | 23:11 | |
| *** esberglu has joined #openstack-nova | 23:13 | |
| *** slaweq has quit IRC | 23:13 | |
| mriedem | welp, looks like most of the approved stuff is all happily waiting or chugging through the gate now | 23:16 |
| *** esberglu has quit IRC | 23:17 | |
| *** figleaf is now known as edleafe | 23:18 | |
| *** mvk has quit IRC | 23:27 | |
| *** jgwentworth is now known as melwitt | 23:28 | |
| *** pchavva1 has joined #openstack-nova | 23:37 | |
| *** pchavva1 has quit IRC | 23:42 | |
| *** esberglu has joined #openstack-nova | 23:43 | |
| *** esberglu has quit IRC | 23:48 | |
| *** tbachman has quit IRC | 23:51 | |
| *** tbachman has joined #openstack-nova | 23:53 | |
| *** hongbin has quit IRC | 23:54 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
