| *** sbezverk has joined #openstack-nova | 00:07 | |
| *** vladikr has joined #openstack-nova | 00:16 | |
| *** edmondsw has joined #openstack-nova | 00:17 | |
| *** edmondsw has quit IRC | 00:25 | |
| *** yingjun has joined #openstack-nova | 00:35 | |
| *** yingjun has quit IRC | 00:37 | |
| *** yingjun has joined #openstack-nova | 00:41 | |
| *** yingjun has quit IRC | 00:53 | |
| *** hieulq has joined #openstack-nova | 01:00 | |
| *** Sukhdev has joined #openstack-nova | 01:08 | |
| *** liverpooler has joined #openstack-nova | 01:10 | |
| *** tbachman has joined #openstack-nova | 01:10 | |
| *** zhenguo has joined #openstack-nova | 01:14 | |
| *** litao__ has joined #openstack-nova | 01:22 | |
| *** phuongnh has joined #openstack-nova | 01:26 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: Extract SortContext/SortWrapper of instance_list into utils https://review.openstack.org/518871 | 01:29 |
|---|---|---|
| openstackgerrit | Yikun Jiang proposed openstack/nova master: WIP: Add cross cell sort support for get_migrations https://review.openstack.org/517273 | 01:31 |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: WIP: Add cross cell sort support for get_migrations https://review.openstack.org/517273 | 01:33 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Mention API behavior change when over quota limit https://review.openstack.org/515631 | 01:57 |
| *** armax has joined #openstack-nova | 01:58 | |
| *** fragatina has quit IRC | 01:58 | |
| *** fragatina has joined #openstack-nova | 01:59 | |
| *** liverpooler has quit IRC | 02:04 | |
| *** hongbin has joined #openstack-nova | 02:06 | |
| *** Sukhdev has quit IRC | 02:16 | |
| *** armax has quit IRC | 02:26 | |
| *** Sukhdev has joined #openstack-nova | 02:31 | |
| *** salv-orlando has joined #openstack-nova | 02:48 | |
| *** gcb has joined #openstack-nova | 02:51 | |
| *** salv-orl_ has quit IRC | 02:51 | |
| *** links has joined #openstack-nova | 02:53 | |
| *** hoonetorg has quit IRC | 02:58 | |
| *** Sukhdev has quit IRC | 03:07 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: avoid multiple join in _get_all_with_shared https://review.openstack.org/519081 | 03:09 |
| *** trungnv has joined #openstack-nova | 03:11 | |
| *** hoonetorg has joined #openstack-nova | 03:11 | |
| *** cheneydc has joined #openstack-nova | 03:18 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: avoid multiple join in _get_all_with_shared https://review.openstack.org/519081 | 03:20 |
| *** moshele has joined #openstack-nova | 03:39 | |
| *** TuanLA has joined #openstack-nova | 03:39 | |
| *** moshele has quit IRC | 03:41 | |
| *** sree has joined #openstack-nova | 03:47 | |
| *** sree has quit IRC | 03:48 | |
| *** sree_ has joined #openstack-nova | 03:48 | |
| *** sree_ is now known as Guest7878 | 03:48 | |
| *** namnh has joined #openstack-nova | 03:48 | |
| *** diga has joined #openstack-nova | 03:50 | |
| *** psachin has joined #openstack-nova | 03:56 | |
| *** yamamoto has joined #openstack-nova | 04:01 | |
| *** gbarros has joined #openstack-nova | 04:10 | |
| *** gbarros_ has quit IRC | 04:10 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:14 | |
| *** squid has joined #openstack-nova | 04:15 | |
| *** gbarros has quit IRC | 04:36 | |
| *** udesale__ has joined #openstack-nova | 04:41 | |
| *** phuongnh has quit IRC | 04:53 | |
| *** phuongnh has joined #openstack-nova | 04:54 | |
| *** suresh12 has joined #openstack-nova | 05:06 | |
| *** ratailor has joined #openstack-nova | 05:09 | |
| *** hshiina has joined #openstack-nova | 05:25 | |
| *** hongbin has quit IRC | 05:30 | |
| *** udesale has joined #openstack-nova | 05:32 | |
| *** udesale__ has quit IRC | 05:33 | |
| *** janki has joined #openstack-nova | 05:33 | |
| *** edmondsw has joined #openstack-nova | 05:41 | |
| *** udesale__ has joined #openstack-nova | 05:43 | |
| *** edmondsw has quit IRC | 05:45 | |
| *** udesale has quit IRC | 05:46 | |
| *** lennyb has quit IRC | 05:51 | |
| *** trungnv has quit IRC | 05:55 | |
| *** lennyb has joined #openstack-nova | 05:56 | |
| *** lennyb has quit IRC | 06:03 | |
| *** qsyqian_ has joined #openstack-nova | 06:05 | |
| *** lennyb has joined #openstack-nova | 06:05 | |
| *** suresh12 has quit IRC | 06:06 | |
| *** moshele has joined #openstack-nova | 06:08 | |
| *** swamireddy has quit IRC | 06:11 | |
| *** bswartz has quit IRC | 06:12 | |
| *** trozet has quit IRC | 06:13 | |
| *** mikal_ has joined #openstack-nova | 06:43 | |
| jianghuaw | alex_xu, do you know when OpenStack client will support placement traits? | 06:46 |
| *** mikal has quit IRC | 06:46 | |
| alex_xu | jianghuaw: I don't know that, at least the server side doesn't finish yet | 06:47 |
| jianghuaw | alex_xu, ah. thanks anyway:-) | 06:47 |
| alex_xu | jianghuaw: is there anyone work on the placement support on Openstack Client? | 06:48 |
| *** chialiang has joined #openstack-nova | 06:48 | |
| jianghuaw | alex_xu, I don't know that. | 06:49 |
| alex_xu | jianghuaw: at least I don't know anyone work on that :) | 06:49 |
| alex_xu | jianghuaw: btw, would you go to the bugsmash next week? | 06:49 |
| *** trozet has joined #openstack-nova | 06:50 | |
| jianghuaw | alex_xu, no. I won't attend this time. | 06:50 |
| jianghuaw | I guess you will be there. right? | 06:51 |
| alex_xu | jianghuaw: yea, i will be there | 06:51 |
| jianghuaw | Wish you enjoy the meeting. | 06:52 |
| *** sahid has joined #openstack-nova | 06:53 | |
| *** slaweq has joined #openstack-nova | 06:54 | |
| *** jaosorior has joined #openstack-nova | 06:56 | |
| *** slaweq has quit IRC | 06:59 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | 07:04 |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: placement: Modify scheduler and report API to accept parent uuid https://review.openstack.org/519243 | 07:04 |
| *** sshwarts has joined #openstack-nova | 07:08 | |
| openstackgerrit | Merged openstack/nova master: Refactor test_allocation_candidates https://review.openstack.org/518124 | 07:09 |
| openstackgerrit | Merged openstack/nova master: Test alloc candidates with same RC in cn & shared https://review.openstack.org/513149 | 07:09 |
| *** Oku_OS-away is now known as Oku_OS | 07:12 | |
| *** spectr has joined #openstack-nova | 07:14 | |
| *** spectr has quit IRC | 07:24 | |
| *** qsyqian has joined #openstack-nova | 07:27 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: (WIP)placement: Modify scheduler and report API to accept parent uuid https://review.openstack.org/519243 | 07:27 |
| *** andreas_s has joined #openstack-nova | 07:29 | |
| *** hoonetorg has quit IRC | 07:30 | |
| *** erlon has joined #openstack-nova | 07:30 | |
| *** fragatina has quit IRC | 07:33 | |
| *** qsyqian__ has joined #openstack-nova | 07:35 | |
| *** qsyqian__ has left #openstack-nova | 07:35 | |
| *** qsyqian has quit IRC | 07:36 | |
| *** hoonetorg has joined #openstack-nova | 07:43 | |
| *** TuanLA has quit IRC | 07:43 | |
| *** trungnv has joined #openstack-nova | 07:44 | |
| *** belmoreira has joined #openstack-nova | 07:45 | |
| *** AlexeyAbashkin has joined #openstack-nova | 07:45 | |
| *** snasir has quit IRC | 07:45 | |
| *** qsyqian_ has quit IRC | 07:47 | |
| *** snasir has joined #openstack-nova | 07:47 | |
| *** rcernin has quit IRC | 07:51 | |
| *** slaweq has joined #openstack-nova | 07:52 | |
| *** Eran_Kuris has quit IRC | 07:53 | |
| *** slaweq has quit IRC | 07:56 | |
| *** tianhui has joined #openstack-nova | 07:58 | |
| *** lajoskatona has joined #openstack-nova | 08:01 | |
| *** slaweq has joined #openstack-nova | 08:08 | |
| *** Eran_Kuris has joined #openstack-nova | 08:09 | |
| *** slaweq has quit IRC | 08:10 | |
| *** pcaruana has joined #openstack-nova | 08:10 | |
| *** salv-orlando has quit IRC | 08:11 | |
| *** slaweq has joined #openstack-nova | 08:11 | |
| *** salv-orlando has joined #openstack-nova | 08:11 | |
| *** alexchadin has joined #openstack-nova | 08:13 | |
| *** salv-orlando has quit IRC | 08:15 | |
| *** ralonsoh has joined #openstack-nova | 08:19 | |
| *** tesseract has joined #openstack-nova | 08:27 | |
| *** moshele has quit IRC | 08:31 | |
| *** msno has joined #openstack-nova | 08:35 | |
| msno | hi guyz. | 08:36 |
| *** salv-orlando has joined #openstack-nova | 08:36 | |
| msno | whats is the best method to take backup of a boot from volume instance from a openstack instalation.. and restore it in another openstack installation | 08:37 |
| *** danpawlik has joined #openstack-nova | 08:38 | |
| *** sree has joined #openstack-nova | 08:38 | |
| *** Guest7878 has quit IRC | 08:40 | |
| *** zenirc369 has joined #openstack-nova | 08:40 | |
| *** ragiman has joined #openstack-nova | 08:41 | |
| *** qsyqian_ has joined #openstack-nova | 08:43 | |
| *** qsyqian_ has quit IRC | 08:43 | |
| *** kukacz has quit IRC | 08:46 | |
| *** moshele has joined #openstack-nova | 08:50 | |
| *** jpena|off is now known as jpena | 08:51 | |
| *** dtantsur|afk is now known as dtantsur | 08:55 | |
| *** swamireddy has joined #openstack-nova | 08:55 | |
| *** maciejjozefczyk has joined #openstack-nova | 08:57 | |
| *** akekane has joined #openstack-nova | 08:57 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612 | 08:57 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226 | 08:58 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add support for Windows network commands https://review.openstack.org/487405 | 09:00 |
| *** akekane is now known as abhishekk | 09:03 | |
| *** rcernin has joined #openstack-nova | 09:03 | |
| openstackgerrit | Spencer Yu proposed openstack/nova master: Use safer ast.literal_eval instead of eval https://review.openstack.org/519268 | 09:04 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Change Subnet.dhcp_server to contain multiple IPs https://review.openstack.org/509107 | 09:05 |
| *** strigazi_ is now known as strigazi | 09:11 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach/swap volumes https://review.openstack.org/517205 | 09:14 |
| *** kukacz has joined #openstack-nova | 09:17 | |
| openstackgerrit | licanwei proposed openstack/nova master: cli 'nova-manage db sync' can't upgrage cell1 https://review.openstack.org/519275 | 09:19 |
| *** rodolof has joined #openstack-nova | 09:21 | |
| *** pooja-jadhav is now known as pooja_jadhav | 09:23 | |
| *** derekh has joined #openstack-nova | 09:24 | |
| openstackgerrit | LIU Yulong proposed openstack/nova master: Enable reset keypair while rebuilding instance https://review.openstack.org/379128 | 09:26 |
| *** andreas_s has quit IRC | 09:33 | |
| *** rodolof has quit IRC | 09:34 | |
| *** lucas-afk is now known as lucasagomes | 09:34 | |
| *** diga has quit IRC | 09:39 | |
| *** slunkad has joined #openstack-nova | 09:42 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273 | 09:45 |
| *** takashin has left #openstack-nova | 09:50 | |
| *** alexchadin has quit IRC | 09:51 | |
| *** qsyqian has joined #openstack-nova | 09:51 | |
| *** alexchadin has joined #openstack-nova | 09:53 | |
| *** mvk has quit IRC | 09:54 | |
| *** andreas_s has joined #openstack-nova | 09:56 | |
| *** cheneydc has quit IRC | 10:00 | |
| *** yamamoto has quit IRC | 10:02 | |
| *** s1061123 has joined #openstack-nova | 10:03 | |
| *** s1061123_ has quit IRC | 10:04 | |
| *** lpetrut has joined #openstack-nova | 10:04 | |
| *** rcernin has quit IRC | 10:06 | |
| *** gcb has quit IRC | 10:07 | |
| *** AlexeyAbashkin has quit IRC | 10:08 | |
| openstackgerrit | Spencer Yu proposed openstack/nova master: [WIP]Use defusedxml function instead of lxml.etree.parse https://review.openstack.org/519291 | 10:09 |
| *** diga has joined #openstack-nova | 10:10 | |
| *** andreas_s has quit IRC | 10:12 | |
| *** namnh has quit IRC | 10:13 | |
| *** sree_ has joined #openstack-nova | 10:15 | |
| *** sree_ is now known as Guest84843 | 10:15 | |
| *** sree has quit IRC | 10:18 | |
| *** mvk has joined #openstack-nova | 10:22 | |
| *** lpetrut_ has joined #openstack-nova | 10:22 | |
| *** zenirc369 has quit IRC | 10:23 | |
| *** andreas_s has joined #openstack-nova | 10:24 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/519127 | 10:24 |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273 | 10:24 |
| *** lpetrut has quit IRC | 10:25 | |
| *** vipul has quit IRC | 10:27 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/519165 | 10:28 |
| *** andreas_s has quit IRC | 10:28 | |
| *** andreas_s has joined #openstack-nova | 10:35 | |
| *** vivsoni has joined #openstack-nova | 10:37 | |
| *** andreas_s has quit IRC | 10:40 | |
| *** belmoreira has quit IRC | 10:42 | |
| *** udesale__ has quit IRC | 10:42 | |
| *** rmart04 has joined #openstack-nova | 10:44 | |
| *** Guest84843 has quit IRC | 10:45 | |
| *** vivsoni_ has joined #openstack-nova | 10:51 | |
| *** yamamoto has joined #openstack-nova | 10:52 | |
| *** gszasz has joined #openstack-nova | 10:53 | |
| *** vivsoni has quit IRC | 10:53 | |
| *** Guest29299 has quit IRC | 10:56 | |
| *** yangyapeng has joined #openstack-nova | 10:57 | |
| *** diga has quit IRC | 11:15 | |
| *** sileht has quit IRC | 11:18 | |
| *** yangyapeng has quit IRC | 11:19 | |
| *** yangyapeng has joined #openstack-nova | 11:19 | |
| *** abhishekk has quit IRC | 11:19 | |
| *** sdague has joined #openstack-nova | 11:21 | |
| *** sahid_ has joined #openstack-nova | 11:21 | |
| *** qsyqian has left #openstack-nova | 11:22 | |
| *** AlexeyAbashkin has joined #openstack-nova | 11:23 | |
| *** sileht has joined #openstack-nova | 11:23 | |
| *** yangyapeng has quit IRC | 11:24 | |
| openstackgerrit | Lajos Katona proposed openstack/nova master: Extend ServerMovingTests with custom resources https://review.openstack.org/497399 | 11:24 |
| *** phuongnh has quit IRC | 11:30 | |
| *** gszasz has quit IRC | 11:31 | |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: disconnect volume from host during detach https://review.openstack.org/515008 | 11:35 |
| *** andreas_s has joined #openstack-nova | 11:36 | |
| *** andrea___ has joined #openstack-nova | 11:41 | |
| *** andreas_s has quit IRC | 11:45 | |
| openstackgerrit | Saverio Proto proposed openstack/nova master: Correct log message when removing a security group from a port https://review.openstack.org/519313 | 11:46 |
| *** yamamoto has quit IRC | 11:51 | |
| zioproto | How do I tag this review as low hanging fruit ? | 11:51 |
| zioproto | I mean this one https://bugs.launchpad.net/nova/+bug/1731889 | 11:54 |
| openstack | Launchpad bug 1731889 in OpenStack Compute (nova) "Nova logs adding security group to port when is actually removing a security group from a port" [Undecided,In progress] - Assigned to Saverio Proto (zioproto) | 11:54 |
| *** gszasz has joined #openstack-nova | 12:02 | |
| *** gszasz has quit IRC | 12:03 | |
| *** alexchadin has quit IRC | 12:03 | |
| *** alexchadin has joined #openstack-nova | 12:03 | |
| *** gszasz has joined #openstack-nova | 12:03 | |
| *** udesale has joined #openstack-nova | 12:06 | |
| *** yamamoto has joined #openstack-nova | 12:08 | |
| *** dtantsur is now known as dtantsur|bbl | 12:10 | |
| *** yassine has joined #openstack-nova | 12:13 | |
| *** yassine is now known as Guest68528 | 12:14 | |
| *** belmoreira has joined #openstack-nova | 12:18 | |
| *** udesale has quit IRC | 12:25 | |
| *** smatzek has joined #openstack-nova | 12:26 | |
| openstackgerrit | sean mooney proposed openstack/nova master: add vif info to migration object https://review.openstack.org/515423 | 12:29 |
| *** lucasagomes is now known as lucas-hungry | 12:31 | |
| *** ratailor has quit IRC | 12:32 | |
| *** chialiang has quit IRC | 12:33 | |
| gibi | zioproto: you can tag triaged but unassigned bugs as low hanging with the tag low-hanging-fruit. You cannot really tag a review as low hanging but I will go and look at your patch now | 12:35 |
| *** Jack_Iv has joined #openstack-nova | 12:36 | |
| efried | gibi Greetings! | 12:39 |
| zioproto | gibi: thanks | 12:40 |
| gibi | efried: hi! | 12:41 |
| gibi | zioproto: please fix the commit message then I will +2 your patch | 12:42 |
| openstackgerrit | Saverio Proto proposed openstack/nova master: Correct log message when removing a security group https://review.openstack.org/519313 | 12:45 |
| zioproto | gibi: done ! | 12:45 |
| gibi | zioproto: thanks, +2 | 12:51 |
| *** jpena is now known as jpena|lunch | 12:52 | |
| *** markvoelker has joined #openstack-nova | 12:53 | |
| *** gszasz has quit IRC | 12:57 | |
| *** moshele has quit IRC | 12:59 | |
| *** moshele has joined #openstack-nova | 12:59 | |
| *** sree has joined #openstack-nova | 13:01 | |
| *** kukacz has quit IRC | 13:02 | |
| *** kukacz has joined #openstack-nova | 13:03 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 13:06 |
| *** alexchadin has quit IRC | 13:07 | |
| *** alexchadin has joined #openstack-nova | 13:08 | |
| *** alexchadin has quit IRC | 13:10 | |
| *** alexchadin has joined #openstack-nova | 13:11 | |
| *** udesale has joined #openstack-nova | 13:11 | |
| *** sree_ has joined #openstack-nova | 13:13 | |
| *** sree_ is now known as Guest80344 | 13:13 | |
| *** gcb has joined #openstack-nova | 13:15 | |
| *** sree has quit IRC | 13:16 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Test helper: validate provider summaries https://review.openstack.org/518982 | 13:16 |
| efried | gibi ^ -- I had had some trouble using sets in similar situations in the past because some things aren't hashable. But this seems to have worked out okay. Thanks for the suggestion. | 13:18 |
| gibi | efried: cool, I will check that shortly | 13:19 |
| efried | Thanks! | 13:19 |
| *** edmondsw has joined #openstack-nova | 13:19 | |
| gibi | efried: do you know the reason of the functional test failures in https://review.openstack.org/#/c/385693/ ? | 13:19 |
| efried | ... | 13:19 |
| gibi | efried: I guess it is 'Invalid microversion: Unacceptable version header: 1.12' | 13:19 |
| *** gszasz has joined #openstack-nova | 13:20 | |
| efried | gibi Yeah, looks that way. | 13:20 |
| *** edmondsw has quit IRC | 13:20 | |
| *** edmondsw has joined #openstack-nova | 13:20 | |
| efried | gibi We don't have 1.12 yet, right? | 13:21 |
| gibi | efried: yeah, as far as I see it is not on master | 13:21 |
| gibi | then I understand, thanks :) | 13:21 |
| gibi | sometimes it is hard to follow what is merged and what is not from nrp | 13:22 |
| efried | Now we need to figure out *WHICH* proposed 1.12 this is dependent on. And it should have a Depends-On accordingly. | 13:22 |
| *** ygl has joined #openstack-nova | 13:22 | |
| ygl | hi all | 13:23 |
| gibi | the patch needs to be able to set parent_provider_uuid during RP creation | 13:23 |
| ygl | I want to know the code flow when we excute nova boot command | 13:23 |
| ygl | i have cloned the git for nova | 13:23 |
| ygl | how does the control reaches to nova.api.openstack.compute.servers from the nova api ? | 13:24 |
| gibi | efried: so I guess the patch needs this https://review.openstack.org/#/c/384807/ | 13:24 |
| *** yamamoto has quit IRC | 13:24 | |
| efried | gibi I was half a second behind you :) | 13:24 |
| ygl | can someone help me with the control flow for launching an instance | 13:25 |
| *** pkoniszewski has quit IRC | 13:25 | |
| efried | ygl jaypipes just put out a nice article explaining that flow. Lemme find it for you... | 13:25 |
| ygl | efried: sure thank you | 13:26 |
| *** lucas-hungry is now known as lucasagomes | 13:26 | |
| gibi | efried: I guess I have the advantages having my second caffe already | 13:26 |
| efried | ygl https://github.com/jaypipes/articles/blob/master/openstack/walkthrough-launch-instance-request.md | 13:26 |
| ygl | efried: let me check it once | 13:26 |
| efried | edleafe- Will there be a scheduler meeting this morning? | 13:30 |
| *** moshele has quit IRC | 13:31 | |
| *** moshele has joined #openstack-nova | 13:31 | |
| ygl | efried: thanks for the link. I have a question | 13:33 |
| efried | ygl I'll help if I can. Jay's not on yet. | 13:33 |
| efried | But you clearly read way faster than I do. | 13:33 |
| ygl | efried: after validating the request , it constructs a build request. Which code is responsible in invoking the build request function ? | 13:34 |
| efried | ygl The article links to https://github.com/openstack/nova/blob/stable/pike/nova/compute/api.py#L880 | 13:35 |
| *** shaner has quit IRC | 13:35 | |
| *** shaner has joined #openstack-nova | 13:35 | |
| efried | ygl If you look down that function to L927, it looks like the build request is finally built there. | 13:35 |
| *** bauzas has joined #openstack-nova | 13:38 | |
| *** gbarros has joined #openstack-nova | 13:41 | |
| *** takashin has joined #openstack-nova | 13:46 | |
| ygl | efried: my question is what invokes that build function ? | 13:47 |
| efried | ygl https://github.com/openstack/nova/blob/stable/pike/nova/compute/api.py#L1121 | 13:49 |
| efried | ygl https://github.com/openstack/nova/blob/stable/pike/nova/compute/api.py#L1620 | 13:50 |
| *** edmondsw_ has joined #openstack-nova | 13:53 | |
| ygl | efried: does this function nova.api.openstack.compute.servers.ServersController.create() invoke build request function ? | 13:53 |
| bauzas | f*ck, my IRC bouncer died | 13:53 |
| bauzas | RIP | 13:53 |
| efried | bauzas We still love you. | 13:53 |
| *** edmondsw has quit IRC | 13:53 | |
| *** edmondsw_ is now known as edmondsw | 13:54 | |
| bauzas | that, and my internal DNS server :( | 13:54 |
| efried | ygl That seems like a reasonable assumption. | 13:54 |
| *** cheneydc has joined #openstack-nova | 13:54 | |
| ygl | efried: i am searching for that connection between the two events | 13:55 |
| *** jpena|lunch is now known as jpena | 13:55 | |
| gibi | bauzas: welcome back! | 13:55 |
| bauzas | gibi: well, ty | 13:55 |
| *** cheneydc has quit IRC | 13:55 | |
| efried | ygl Yeah, that's tricky, and I'm not super well-versed on how that works. Perhaps bauzas or stephenfin would know off the top. | 13:55 |
| bauzas | just having a sickness too :p | 13:56 |
| bauzas | sometimes, you love coming back to your house | 13:56 |
| ygl | bauzas: Hi, can you help me with this https://github.com/jaypipes/articles/blob/master/openstack/walkthrough-launch-instance-request.md please ? | 13:56 |
| *** rmart04 has quit IRC | 13:57 | |
| ygl | bauzas: I am trying to understand what actually invokes the build request function ? | 13:57 |
| efried | bauzas ygl is trying to figure out how we get from an API call to boot an instance, down to the create method in here: https://github.com/openstack/nova/blob/stable/pike/nova/compute/api.py#L1620 | 13:58 |
| bauzas | ygl: have you seen the links ?N | 13:59 |
| openstackgerrit | David Wahlstrom proposed openstack/nova master: Fix changes-since search filter https://review.openstack.org/519356 | 13:59 |
| bauzas | https://github.com/jaypipes/articles/blob/master/openstack/walkthrough-launch-instance-request.md#construct-build-request | 13:59 |
| bauzas | linking to https://github.com/openstack/nova/blob/stable/pike/nova/compute/api.py#L880 | 13:59 |
| efried | bauzas Yeah, we walked that back to nova.compute.api.API.create, but don't know how the call into the REST API gets there. | 14:00 |
| *** hshiina has quit IRC | 14:00 | |
| ygl | bauzas: my question is after the http validation by "nova.api.openstack.compute.servers.ServersController.create()" , what actually invookes the build request function ? | 14:01 |
| bauzas | ah | 14:01 |
| bauzas | sec | 14:01 |
| ygl | bauzas: sure | 14:01 |
| *** liverpooler has joined #openstack-nova | 14:01 | |
| bauzas | it calls nova.compute.api.::create() AFAIR | 14:01 |
| bauzas | ie. https://github.com/openstack/nova/blob/stable/pike/nova/compute/api.py#L1584 | 14:02 |
| bauzas | ygl, efried: there, it is https://github.com/openstack/nova/blob/stable/pike/nova/api/openstack/compute/servers.py#L542 | 14:03 |
| bauzas | if cellsv1, it can go to the cells API | 14:03 |
| bauzas | but if not, it directly calls the above method | 14:03 |
| efried | bauzas gibi Any interest in having a scheduler meeting in edleafe-'s absence? | 14:04 |
| bauzas | efried: sorry, I'm a bit sick | 14:04 |
| efried | Sorry to hear that. Rest up! | 14:04 |
| bauzas | honestly, nothing to tell about the Sydney suumit | 14:04 |
| *** gbarros has quit IRC | 14:05 | |
| gibi | efried:If it just the two of us then I think we can skip the meeting | 14:05 |
| efried | wfm | 14:05 |
| ygl | efried: can you find out for me how the control flows after the POST /servers to the create server function ? | 14:05 |
| efried | ygl That sounds like wsgi stuff. Fraid I'm not the guy there. | 14:06 |
| efried | ygl What is it you're actually trying to do here? | 14:06 |
| bauzas | ygl: sec | 14:07 |
| ygl | efried: i am just curious to find it out. I work on openstack | 14:07 |
| bauzas | ygl: you wanna know what calls https://github.com/openstack/nova/blob/stable/pike/nova/api/openstack/compute/servers.py#L453 ? | 14:07 |
| bauzas | I mean, how is it routed ? | 14:07 |
| ygl | bauzas:no, I am following the post https://github.com/jaypipes/articles/blob/master/openstack/walkthrough-launch-instance-request.md#construct-build-request | 14:07 |
| ygl | bauzas: in that I want to know after the "Validate HTTP payload" section, which code actually calls the build reequest function | 14:08 |
| *** mriedem has joined #openstack-nova | 14:08 | |
| *** dtantsur|bbl is now known as dtantsur | 14:09 | |
| bauzas | ygl: I replied to you on that part | 14:09 |
| ygl | efried: bauzas if you find anything please mail the details to ygk.kmr@gmail.com | 14:09 |
| bauzas | (15:02:00) bauzas: ie. https://github.com/openstack/nova/blob/stable/pike/nova/compute/api.py#L1584 | 14:09 |
| bauzas | (15:03:06) bauzas: ygl, efried: there, it is https://github.com/openstack/nova/blob/stable/pike/nova/api/openstack/compute/servers.py#L542 | 14:09 |
| *** andrea___ has quit IRC | 14:09 | |
| ygl | bauzas: I understand you are pointing to the code responsible for creating a server. I am asking how does the control get to that code in the first place ? | 14:10 |
| ygl | bauzas: hope u understand | 14:10 |
| *** lyan has joined #openstack-nova | 14:10 | |
| bauzas | ygl: no sorry I don't get you | 14:11 |
| ygl | efried: hope u understood my question | 14:13 |
| ygl | bauzas: anyway thanks for your help bauzas | 14:13 |
| efried | ygl I'm not sure. I think what you're asking is what handles the actual incoming HTTP request and invokes the code bauzas pointed to above? | 14:13 |
| ygl | efried: exactly | 14:14 |
| *** andreas_s has joined #openstack-nova | 14:14 | |
| efried | ygl Okay, so that's wsgi magic that I don't understand. There's some kind of declarative syntax with decorators registering hooks in the wsgi server or some craziness. sdague is this in your wheelhouse? mtreinish maybe? | 14:15 |
| ygl | efried: if you find anything please let me know on my email at ygk.kmr@gmail.com | 14:16 |
| ygl | efried: thanks so much for your time and help | 14:16 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Fix foreign key constraint error https://review.openstack.org/516880 | 14:16 |
| efried | ygl You're welcome. | 14:16 |
| *** links has quit IRC | 14:18 | |
| efried | mriedem FYI, https://review.openstack.org/#/c/519034/ has passed that test six times in a row now. Which of course proves nothing. | 14:18 |
| bauzas | ygl: magic is there for routing https://github.com/openstack/nova/blob/stable/pike/nova/api/openstack/compute/routes.py | 14:19 |
| bauzas | ygl, efried: in particular for your concern https://github.com/openstack/nova/blob/stable/pike/nova/api/openstack/compute/routes.py#L777 | 14:19 |
| *** edmondsw has quit IRC | 14:20 | |
| bauzas | that's how we map an entrypoint | 14:20 |
| efried | bauzas Beautiful, thanks. | 14:20 |
| *** edmondsw has joined #openstack-nova | 14:20 | |
| *** edmondsw has quit IRC | 14:21 | |
| *** edmondsw has joined #openstack-nova | 14:21 | |
| *** liverpooler has quit IRC | 14:22 | |
| *** liverpooler has joined #openstack-nova | 14:23 | |
| ygl | bauzas: thats awesome | 14:23 |
| bauzas | that's just the routing logic | 14:24 |
| bauzas | not the WSGI call itself | 14:24 |
| *** yamamoto has joined #openstack-nova | 14:25 | |
| ygl | bauzas: ok | 14:26 |
| ygl | bauzas: I think to know how the routing works, we have to learn about wsgi framework in python | 14:27 |
| *** gongysh has joined #openstack-nova | 14:27 | |
| bauzas | ygl: that routing logic is very nova-specific | 14:27 |
| *** gongysh has quit IRC | 14:27 | |
| ygl | bauzas: ok | 14:27 |
| bauzas | ygl: you can find many other python frameworks that do that for you | 14:27 |
| ygl | bauzas: hmm | 14:28 |
| bauzas | like, placement is having a separate routing mechanism AFAIK | 14:28 |
| *** Tom has joined #openstack-nova | 14:28 | |
| *** Tom is now known as Guest36443 | 14:29 | |
| *** andreas_s has quit IRC | 14:29 | |
| bauzas | yeah, it uses wsgify | 14:30 |
| *** belmoreira has quit IRC | 14:30 | |
| *** udesale has quit IRC | 14:30 | |
| ygl | bauzas: thanks for your time | 14:31 |
| bauzas | yeah, placement is totally different https://github.com/openstack/nova/blob/stable/pike/nova/api/openstack/placement/handler.py | 14:31 |
| *** eharney has joined #openstack-nova | 14:31 | |
| bauzas | ygl: so that really depends on your wishes | 14:31 |
| ygl | bauzas: you are the first personto help me with the code flow of nova thanks for that | 14:32 |
| *** belmoreira has joined #openstack-nova | 14:32 | |
| bauzas | np | 14:32 |
| *** tssurya has quit IRC | 14:33 | |
| *** awaugama has joined #openstack-nova | 14:33 | |
| *** smatzek has quit IRC | 14:33 | |
| openstackgerrit | David Wahlstrom proposed openstack/nova master: Fix changes-since search filter https://review.openstack.org/519356 | 14:34 |
| *** smatzek has joined #openstack-nova | 14:34 | |
| *** yamamoto has quit IRC | 14:34 | |
| *** ygl has quit IRC | 14:35 | |
| openstackgerrit | David Wahlstrom proposed openstack/nova master: Fix changes-since search filter https://review.openstack.org/519356 | 14:35 |
| openstackgerrit | David Wahlstrom proposed openstack/nova master: Fix changes-since search filter https://review.openstack.org/519356 | 14:36 |
| *** andreas_s has joined #openstack-nova | 14:37 | |
| *** smatzek has quit IRC | 14:38 | |
| *** edand has joined #openstack-nova | 14:39 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add support for Windows network commands https://review.openstack.org/487405 | 14:40 |
| *** mvk has quit IRC | 14:41 | |
| *** janki has quit IRC | 14:41 | |
| sdague | efried: you good now, or are there more questions? | 14:43 |
| efried | sdague I believe bauzas answered everything, thanks. | 14:43 |
| sdague | ok, great | 14:43 |
| *** lbragstad has joined #openstack-nova | 14:44 | |
| *** amodi has joined #openstack-nova | 14:45 | |
| *** tssurya has joined #openstack-nova | 14:47 | |
| *** lbragstad has quit IRC | 14:48 | |
| jianghuaw | bauzas, dansmith, alex_xu: It's appreciated a lot if you can help to review these patches on vGPU: https://review.openstack.org/#/q/topic:bp/add-support-for-vgpu+(status:open) | 14:49 |
| jianghuaw | With these patches, the simple vGPU feature works well for XenAPI. | 14:49 |
| bauzas | jianghuaw: of course, will do | 14:50 |
| jianghuaw | bauzas, thanks:-) | 14:51 |
| bauzas | jianghuaw: just a bit sick today, and still jetlagging from the summit, so don't expect full steam ahead until tomorrow hopefully :) | 14:51 |
| jianghuaw | bauzas, ah. Sorry for that. Take care and hope you will be well soon. | 14:53 |
| *** READ10 has joined #openstack-nova | 14:53 | |
| *** mriedem has quit IRC | 14:55 | |
| *** mvk has joined #openstack-nova | 14:55 | |
| *** armax has joined #openstack-nova | 14:56 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test alloc_cands with non overlapping sharing RPs https://review.openstack.org/519380 | 14:57 |
| gibi | efried, jaypipes: I found another / simpler case to reproduce bug 1731072 | 14:58 |
| openstack | bug 1731072 in OpenStack Compute (nova) "AllocationCandidates.get_by_filters returns garbage with multiple aggregates" [Undecided,New] https://launchpad.net/bugs/1731072 | 14:58 |
| *** mriedem has joined #openstack-nova | 14:58 | |
| *** edand has quit IRC | 14:58 | |
| efried | gibi Looking... | 14:59 |
| efried | (gibi FYI, Jay's out training today) | 14:59 |
| *** lbragstad has joined #openstack-nova | 14:59 | |
| gibi | efried: basically you don't need overlapping RCs to see the bug | 15:00 |
| *** yamahata has joined #openstack-nova | 15:00 | |
| efried | gibi +1, nice. | 15:01 |
| gibi | efried: I realized that this could be a possible case when I read your regression test. Btw, that looks good so I'm going to +W it now | 15:02 |
| *** mvk has quit IRC | 15:03 | |
| efried | gibi Thanks! | 15:03 |
| gibi | efried: thank your for the bug report and the regression test :) | 15:03 |
| *** amodi has quit IRC | 15:03 | |
| *** takashin has left #openstack-nova | 15:04 | |
| *** hongbin has joined #openstack-nova | 15:05 | |
| *** gongysh has joined #openstack-nova | 15:05 | |
| *** sahid has quit IRC | 15:06 | |
| *** sahid_ has quit IRC | 15:06 | |
| *** sahid has joined #openstack-nova | 15:06 | |
| *** esberglu has joined #openstack-nova | 15:07 | |
| *** smatzek has joined #openstack-nova | 15:07 | |
| *** moshele has quit IRC | 15:07 | |
| *** smatzek has quit IRC | 15:07 | |
| *** smatzek has joined #openstack-nova | 15:07 | |
| *** gbarros has joined #openstack-nova | 15:08 | |
| *** Guest36443 has quit IRC | 15:08 | |
| *** msno has quit IRC | 15:08 | |
| *** mlavalle has joined #openstack-nova | 15:09 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: placement: AllocCands.get_by_{filters => requests} https://review.openstack.org/514092 | 15:09 |
| efried | gibi ^ rebased & fixed up that message, good catch. | 15:09 |
| gibi | efried: looking... | 15:10 |
| efried | gibi Ahdang, missed an unused import. One sec... | 15:10 |
| openstackgerrit | Eric Fried proposed openstack/nova master: placement: AllocCands.get_by_{filters => requests} https://review.openstack.org/514092 | 15:11 |
| efried | gibi Done. | 15:11 |
| gibi | efried: thanks for the update, +2 | 15:13 |
| efried | Thanks! | 15:14 |
| *** Jack_Iv has quit IRC | 15:14 | |
| efried | gibi Don't know how much longer you're around, but if I can help tweak any of Jay's patches to get them moving, let me know. | 15:14 |
| *** salv-orlando has quit IRC | 15:15 | |
| *** mvk has joined #openstack-nova | 15:15 | |
| *** salv-orlando has joined #openstack-nova | 15:15 | |
| *** amodi has joined #openstack-nova | 15:17 | |
| gibi | efried: I think I still have an hour so you can hit me with a patch | 15:18 |
| efried | gibi I was saying if you know of any of Jay's patches that need tweaks, I can get that done so we can keep them moving. | 15:19 |
| efried | At this point I guess everything he has is in merge conflict, so maybe I'll go start rebasing. | 15:19 |
| *** salv-orlando has quit IRC | 15:20 | |
| efried | gibi Meanwhile, if you want some easy +Ws.... | 15:20 |
| *** Guest80344 has quit IRC | 15:21 | |
| efried | gibi https://review.openstack.org/#/c/516783/ and https://review.openstack.org/#/c/517626/ | 15:21 |
| gibi | efried: ahh I underderstand now. I only see merge conflicts and patches waiting for others to look at | 15:21 |
| *** sree has joined #openstack-nova | 15:21 | |
| gibi | efried: looking... | 15:21 |
| efried | gibi There's also one more bug-provin' patch for allocation candidates: https://review.openstack.org/#/c/518633/ | 15:22 |
| efried | gibi ...and the one you looked at earlier which I updated per your comments: https://review.openstack.org/#/c/518982/ | 15:23 |
| *** hemna_ has joined #openstack-nova | 15:23 | |
| *** sree has quit IRC | 15:26 | |
| gibi | efried: yeah, I opened https://review.openstack.org/#/c/518633/ before but I think I need a fresh mind to walk throught he complexity there, so maybe tomorrow :) | 15:27 |
| efried | okay | 15:27 |
| gibi | I looking that the test helper patch now | 15:27 |
| gibi | that looks good too | 15:32 |
| *** gongysh has quit IRC | 15:33 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test alloc_cands with non overlapping sharing RPs https://review.openstack.org/519380 | 15:34 |
| gibi | I think I managed to make it even stranger ^^ | 15:35 |
| gibi | I managed to get the same allocation candidate from placement twice | 15:35 |
| *** liverpooler has quit IRC | 15:36 | |
| *** gouthamr_ has joined #openstack-nova | 15:41 | |
| *** gbarros has quit IRC | 15:41 | |
| *** tasker has joined #openstack-nova | 15:42 | |
| *** nicolasbock has joined #openstack-nova | 15:42 | |
| *** gouthamr_ is now known as gouthamr | 15:42 | |
| tasker | when making a snapshot of a volume-backed instance, the nova driver will queisce the instance's volumes prior to the snapshot. it looks like if the instance fails a snapshot, the exception isn't caught and the instance and its volumes are never thawed: master/nova/compute/api.py#L2738 | 15:44 |
| *** david-lyle has joined #openstack-nova | 15:44 | |
| tasker | am I missing something? I'm still trying to trace through the code to see if the volumes are thawed after and exception. I can't find anything in launchpad regarding this either (could be that I'm not using the right search terms). | 15:45 |
| mriedem | tasker: which release? | 15:48 |
| openstackgerrit | Elod Illes proposed openstack/nova master: Moving more utils to ServerResourceAllocationTestBase https://review.openstack.org/499539 | 15:50 |
| openstackgerrit | Elod Illes proposed openstack/nova master: factor out compute service start in ServerMovingTest https://review.openstack.org/503037 | 15:50 |
| openstackgerrit | Elod Illes proposed openstack/nova master: Change live_migrate tests to use fakedriver https://review.openstack.org/505202 | 15:50 |
| openstackgerrit | Elod Illes proposed openstack/nova master: Use FakeLiveMigrateDriver in notification test https://review.openstack.org/519405 | 15:50 |
| tasker | the above is git master as of a few minutes ago. | 15:50 |
| tasker | commit a4fc1bcd | 15:51 |
| tasker | I have Mitaka in production where I noticed this problem and I see that function unchanged between Mitaka (commit 9825c80) and current master. | 15:52 |
| *** slaweq has quit IRC | 15:52 | |
| *** dave-mccowan has joined #openstack-nova | 15:52 | |
| *** belmoreira has quit IRC | 15:53 | |
| *** andreas_s has quit IRC | 15:56 | |
| *** andreas_s has joined #openstack-nova | 15:57 | |
| *** tssurya has quit IRC | 15:57 | |
| *** andreas_s has quit IRC | 16:00 | |
| *** andreas_s has joined #openstack-nova | 16:00 | |
| *** belmoreira has joined #openstack-nova | 16:01 | |
| *** Apoorva has joined #openstack-nova | 16:01 | |
| tasker | in my case, the snapshot failed due to OverLimit returned by the cinderclient. The exeception bubbles up, and I'm still not seeing anywhere that unquiesces / thaws the instance after the exception. | 16:01 |
| *** jmlowe has joined #openstack-nova | 16:02 | |
| mriedem | ok i remember someone fixing the handling of the OverLimit from cinder, but would have to see if the unquiesce is ever run on failure | 16:05 |
| mriedem | the overlimit used to cause a 500 response from the API | 16:05 |
| *** alexchadin has quit IRC | 16:06 | |
| maciejjozefczyk | mriedem: We talked during summit about bug in calculating used resources in resource_tracker. The bug is there: https://bugs.launchpad.net/nova/+bug/1729621 | 16:07 |
| openstack | Launchpad bug 1729621 in OpenStack Compute (nova) "Inconsistent value for vcpu_used" [Undecided,New] | 16:07 |
| tasker | the traceback from this morning: http://paste.openstack.org/show/626180/ | 16:07 |
| maciejjozefczyk | I mentioned possible solutions (1. option imo seems to be the right one); could you take a look? | 16:08 |
| *** nicolasbock has quit IRC | 16:09 | |
| *** liverpooler has joined #openstack-nova | 16:10 | |
| *** liverpooler has quit IRC | 16:15 | |
| *** salv-orlando has joined #openstack-nova | 16:16 | |
| mriedem | maciejjozefczyk: sure, thanks for pointing out the bu | 16:18 |
| mriedem | *bug | 16:18 |
| *** tssurya has joined #openstack-nova | 16:20 | |
| mriedem | maciejjozefczyk: nice recreate :) | 16:20 |
| *** salv-orlando has quit IRC | 16:21 | |
| *** Oku_OS is now known as Oku_OS-away | 16:22 | |
| *** andreas_s has quit IRC | 16:22 | |
| *** dinesh_ has joined #openstack-nova | 16:24 | |
| *** belmoreira has quit IRC | 16:25 | |
| *** andreas_s has joined #openstack-nova | 16:27 | |
| openstackgerrit | Merged openstack/nova master: Test allocation_candidates with only sharing RPs https://review.openstack.org/518382 | 16:28 |
| tasker | mriedem: I'm not seeing so. the only time I see `unquiesce` called is from compute/api.py#L2807 after the volumes finish snapping. but there is no exception handling between quiesce and unquiesce. if an exception is raised, it goes right past the unuqiesce on its way out the stack. | 16:29 |
| mriedem | tasker: same here | 16:32 |
| mriedem | tasker: want to report a bug? | 16:32 |
| openstackgerrit | Merged openstack/nova master: Update contributor microversion doc for compute https://review.openstack.org/517626 | 16:34 |
| *** andreas_s has quit IRC | 16:36 | |
| efried | mriedem Do you want me to do anything with https://review.openstack.org/#/c/519034/ ? | 16:39 |
| mriedem | efried: not at this moment no | 16:40 |
| mriedem | efried: i'd like jaypipes to also know about this and look at it | 16:40 |
| mriedem | i'm playing catch up on internal stuff today | 16:40 |
| efried | mriedem Okay. Should I bother to keep rechecking it? I'm convinced that it at least avoids the problem, though of course it's probably just masking the root cause. | 16:40 |
| openstackgerrit | Merged openstack/nova master: placement: Contributor doc microversion checklist https://review.openstack.org/516783 | 16:42 |
| mriedem | efried: i'd like others more involved with placement code to look to see if they have any ideas on root cause | 16:42 |
| mriedem | because i'm not sure why we'd be hitting this with a single scheduler CI job | 16:42 |
| efried | rgr | 16:43 |
| mriedem | johnthetubaguy: sdague: want to slam through this bit of spec accounting? https://review.openstack.org/#/c/516033/ | 16:44 |
| *** jpena is now known as jpena|brb | 16:44 | |
| mriedem | stephenfin: you were +W on this before, just needed a rebase https://review.openstack.org/#/c/514782/ | 16:44 |
| *** psachin has quit IRC | 16:46 | |
| *** pcaruana has quit IRC | 16:49 | |
| *** lucasagomes is now known as lucas-afk | 16:53 | |
| *** dinesh_ has quit IRC | 16:53 | |
| dtantsur | folks, who's left the nova_cells meeting running? :) | 17:02 |
| *** ricardoas has joined #openstack-nova | 17:03 | |
| efried | mriedem dansmith Do you have the power to #endmeeting for dtantsur ? | 17:03 |
| dtantsur | I suspect no | 17:04 |
| dtantsur | somebody unrelated has started it, sorry for noise here | 17:04 |
| openstackgerrit | Elod Illes proposed openstack/nova master: Transform instance-live_migration_pre notification https://review.openstack.org/482070 | 17:04 |
| *** sapd_ has joined #openstack-nova | 17:04 | |
| *** sapd__ has quit IRC | 17:05 | |
| *** ragiman has quit IRC | 17:08 | |
| *** salv-orlando has joined #openstack-nova | 17:09 | |
| *** Guest68528 has quit IRC | 17:10 | |
| *** itlinux has joined #openstack-nova | 17:13 | |
| *** jmlowe has quit IRC | 17:17 | |
| *** itlinux has quit IRC | 17:18 | |
| *** fragatina has joined #openstack-nova | 17:20 | |
| *** Apoorva has quit IRC | 17:21 | |
| *** Apoorva has joined #openstack-nova | 17:21 | |
| *** ralonsoh has quit IRC | 17:22 | |
| *** sahid has quit IRC | 17:25 | |
| *** Apoorva has quit IRC | 17:27 | |
| *** jpena|brb is now known as jpena | 17:27 | |
| *** itlinux has joined #openstack-nova | 17:28 | |
| *** sshwarts has quit IRC | 17:30 | |
| openstackgerrit | Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/519165 | 17:31 |
| *** itlinux has quit IRC | 17:32 | |
| *** fragatina has quit IRC | 17:32 | |
| *** moshele has joined #openstack-nova | 17:33 | |
| *** itlinux has joined #openstack-nova | 17:34 | |
| *** tesseract has quit IRC | 17:34 | |
| *** msno has joined #openstack-nova | 17:35 | |
| *** itlinux has quit IRC | 17:37 | |
| *** itlinux has joined #openstack-nova | 17:37 | |
| openstackgerrit | Elod Illes proposed openstack/nova master: Functional test: evacuate with no compute https://review.openstack.org/498482 | 17:38 |
| *** itlinux has quit IRC | 17:41 | |
| mriedem | sdague: i'm +2 on the keypair reset during rebuild change now https://review.openstack.org/#/c/379128/ | 17:42 |
| tasker | mriedem: https://bugs.launchpad.net/nova/+bug/1731986 | 17:45 |
| openstack | Launchpad bug 1731986 in OpenStack Compute (nova) "nova snapshot_volume_backed failure does not thaw filesystems" [Undecided,New] | 17:45 |
| tasker | I'm in the proccess of writing and testing a patch to get my production clusters moving. I"ll hopefully have a review for the patch this afternoon. | 17:46 |
| *** itlinux has joined #openstack-nova | 17:46 | |
| mriedem | tasker: cool, thanks | 17:46 |
| *** Apoorva has joined #openstack-nova | 17:50 | |
| *** lpetrut_ has quit IRC | 17:54 | |
| *** moshele has quit IRC | 17:55 | |
| *** itlinux has quit IRC | 17:55 | |
| *** itlinux has joined #openstack-nova | 17:56 | |
| *** liverpooler has joined #openstack-nova | 17:57 | |
| *** itlinux has quit IRC | 17:59 | |
| *** itlinux has joined #openstack-nova | 18:00 | |
| *** itlinux has quit IRC | 18:03 | |
| openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/519127 | 18:03 |
| *** itlinux has joined #openstack-nova | 18:04 | |
| sdague | mriedem: +A | 18:06 |
| *** derekh has quit IRC | 18:06 | |
| sdague | took me a second to understand the or on the json schema, but I think I get it now | 18:06 |
| mriedem | yeah the oneOf | 18:07 |
| mriedem | we have that in the 'networks' entry in the server create schema too | 18:07 |
| *** dtantsur is now known as dtantsur|afk | 18:09 | |
| *** fragatina has joined #openstack-nova | 18:09 | |
| *** ricardoas1 has joined #openstack-nova | 18:09 | |
| *** ricardoas has quit IRC | 18:12 | |
| *** msno has quit IRC | 18:13 | |
| *** ricardoas has joined #openstack-nova | 18:13 | |
| *** ricardoas1 has quit IRC | 18:14 | |
| *** msno has joined #openstack-nova | 18:17 | |
| *** mvk has quit IRC | 18:17 | |
| *** jpena is now known as jpena|off | 18:17 | |
| *** AlexeyAbashkin has quit IRC | 18:18 | |
| mriedem | hmm, why is it that we don't dump config options when n-api starts up? | 18:19 |
| mriedem | i'm trying to fix the log formatting for placement-api in devstack, and the differences in the logs are a bit weird | 18:20 |
| mriedem | oh i see why | 18:20 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/wsgi.py#L60 | 18:21 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: instance_mapping/request_spec records exist even after the instances are archived https://review.openstack.org/515034 | 18:24 |
| *** tssurya has left #openstack-nova | 18:26 | |
| *** slaweq has joined #openstack-nova | 18:36 | |
| openstackgerrit | Merged openstack/nova master: Fix missing versioned notification sample https://review.openstack.org/516582 | 18:38 |
| *** rmart04 has joined #openstack-nova | 18:38 | |
| *** lpetrut_ has joined #openstack-nova | 18:47 | |
| *** sree has joined #openstack-nova | 18:48 | |
| *** smatzek has quit IRC | 18:52 | |
| *** smatzek has joined #openstack-nova | 18:52 | |
| *** rmart04 has quit IRC | 18:52 | |
| *** sree has quit IRC | 18:53 | |
| *** smatzek_ has joined #openstack-nova | 18:54 | |
| *** lpetrut_ has quit IRC | 18:56 | |
| *** smatzek has quit IRC | 18:56 | |
| *** gszasz has quit IRC | 18:57 | |
| *** Sukhdev has joined #openstack-nova | 18:57 | |
| *** tjones has joined #openstack-nova | 18:58 | |
| *** smatzek_ has quit IRC | 18:59 | |
| *** tjones_ has joined #openstack-nova | 19:01 | |
| *** gyee_ has joined #openstack-nova | 19:01 | |
| *** lpetrut_ has joined #openstack-nova | 19:02 | |
| *** slaweq has quit IRC | 19:03 | |
| *** tjones has quit IRC | 19:03 | |
| *** weshay is now known as weshay_bbiab | 19:03 | |
| *** tjones_ has quit IRC | 19:03 | |
| *** yamahata has quit IRC | 19:09 | |
| *** slaweq has joined #openstack-nova | 19:13 | |
| *** lpetrut_ has quit IRC | 19:13 | |
| *** lpetrut_ has joined #openstack-nova | 19:14 | |
| *** vivsoni__ has joined #openstack-nova | 19:14 | |
| *** moshele has joined #openstack-nova | 19:16 | |
| *** vivsoni_ has quit IRC | 19:17 | |
| *** jmlowe has joined #openstack-nova | 19:18 | |
| *** eharney has quit IRC | 19:22 | |
| *** artom_ has quit IRC | 19:28 | |
| *** eharney has joined #openstack-nova | 19:31 | |
| *** AlexeyAbashkin has joined #openstack-nova | 19:37 | |
| *** sree has joined #openstack-nova | 19:37 | |
| *** AlexeyAbashkin has quit IRC | 19:41 | |
| *** sree has quit IRC | 19:41 | |
| *** smatzek has joined #openstack-nova | 19:41 | |
| *** itlinux has quit IRC | 19:42 | |
| *** moshele has quit IRC | 19:43 | |
| *** xarlos has joined #openstack-nova | 19:48 | |
| xarlos | Hi all. I have just configured a number of nova node in separate region configs. | 19:49 |
| *** esberglu has quit IRC | 19:50 | |
| *** esberglu has joined #openstack-nova | 19:50 | |
| xarlos | However, when I run "openstack hypervisor list" I get 2 listed from one region in this region, and none over in another region. I'm a bit confused. | 19:50 |
| xarlos | I think I failed to add the region statement when I registered them using nova-manage. | 19:51 |
| xarlos | How can I clear this out and get to re-add them? :-/ | 19:51 |
| *** weshay_bbiab is now known as weshay | 19:51 | |
| *** AlexeyAbashkin has joined #openstack-nova | 19:59 | |
| *** moshele has joined #openstack-nova | 20:01 | |
| *** AlexeyAbashkin has quit IRC | 20:06 | |
| *** AlexeyAbashkin has joined #openstack-nova | 20:06 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Log options at debug when starting API services under wsgi https://review.openstack.org/519462 | 20:07 |
| openstackgerrit | Eric M Gonzalez (tasker) proposed openstack/nova master: unquiesce instance on volume snapshot failure https://review.openstack.org/519464 | 20:12 |
| mriedem | tasker: we'll want a test for that | 20:12 |
| tasker | hope I got that submitted all right. | 20:12 |
| mriedem | when you get the time | 20:12 |
| tasker | oh, of course you will. . ) | 20:13 |
| tasker | let me get some lunch. | 20:13 |
| *** mikal_ is now known as mikal | 20:15 | |
| mriedem | tasker: comments inline when you get back | 20:24 |
| mriedem | smcginnis: any idea how graceful cinder is about deleting a snapshot that is being created? concurrent requests i mean. | 20:25 |
| *** Alexey_Abashkin has joined #openstack-nova | 20:27 | |
| *** salv-orlando has quit IRC | 20:28 | |
| *** msno has quit IRC | 20:28 | |
| *** salv-orlando has joined #openstack-nova | 20:28 | |
| *** Alexey_Abashkin1 has joined #openstack-nova | 20:28 | |
| *** AlexeyAbashkin has quit IRC | 20:30 | |
| *** Alexey_Abashkin1 is now known as AlexeyAbashkin | 20:30 | |
| *** dklyle has joined #openstack-nova | 20:31 | |
| *** Alexey_Abashkin has quit IRC | 20:31 | |
| *** david-lyle has quit IRC | 20:31 | |
| *** salv-orlando has quit IRC | 20:33 | |
| tasker | mriedem: regarding your summary comment about splitting it up. after reviewing your comments I agree with you. how do you want me to proceed on that? resubmit with just the unqueisce and revisit the snapshot deletion later (after discussion about race) in another blueprint? | 20:35 |
| *** awaugama has quit IRC | 20:35 | |
| mriedem | tasker: i don't think we need a blueprint, it's just another patch related to the same bug | 20:36 |
| mriedem | tasker: but yeah split them up so quiesce is first and handled separately, we can backport that | 20:36 |
| mriedem | then we'll see about the automatic cleanup of snapshots | 20:36 |
| *** READ10 has quit IRC | 20:37 | |
| tasker | ok. I'll have that and a test (i hope) submitted in a little bit. | 20:37 |
| *** eharney has quit IRC | 20:48 | |
| *** AlexeyAbashkin has quit IRC | 20:53 | |
| openstackgerrit | Merged openstack/nova master: Enable reset keypair while rebuilding instance https://review.openstack.org/379128 | 20:59 |
| *** salv-orlando has joined #openstack-nova | 21:00 | |
| *** yamahata has joined #openstack-nova | 21:02 | |
| *** liverpooler has quit IRC | 21:07 | |
| *** Sukhdev has quit IRC | 21:12 | |
| *** moshele has quit IRC | 21:24 | |
| *** armax has quit IRC | 21:24 | |
| *** smatzek has quit IRC | 21:25 | |
| tasker | this url https://docs.openstack.org/nova/latest/reference/unit_tests.html is no longer valid. does anyone know where it ended up? | 21:26 |
| mriedem | tasker: https://docs.openstack.org/nova/latest/contributor/testing.html#unit-tests ? | 21:31 |
| *** AlexeyAbashkin has joined #openstack-nova | 21:32 | |
| mriedem | that page linking to the hacking repo docs seems like the wrong thing to do... | 21:33 |
| *** sree has joined #openstack-nova | 21:34 | |
| *** AlexeyAbashkin has quit IRC | 21:36 | |
| mriedem | it was probably supposed to link to this https://github.com/openstack/nova/blob/master/HACKING.rst#creating-unit-tests | 21:37 |
| *** amodi has quit IRC | 21:39 | |
| *** sree has quit IRC | 21:39 | |
| tasker | sorry -- got sidetracked because of /. | 21:40 |
| tasker | the latter is where I ended up after some trolling. | 21:41 |
| *** eharney has joined #openstack-nova | 21:45 | |
| *** redondo-mk_ has joined #openstack-nova | 21:46 | |
| *** tommylikehu_ has joined #openstack-nova | 21:46 | |
| *** vdrok_ has joined #openstack-nova | 21:46 | |
| *** fyxim_ has joined #openstack-nova | 21:46 | |
| *** icey_ has joined #openstack-nova | 21:46 | |
| *** johnsom_ has joined #openstack-nova | 21:46 | |
| *** patrickeast_ has joined #openstack-nova | 21:47 | |
| *** Hazelesque_ has joined #openstack-nova | 21:49 | |
| *** raorn_ has joined #openstack-nova | 21:49 | |
| *** liverpooler has joined #openstack-nova | 21:50 | |
| *** _gryf has joined #openstack-nova | 21:51 | |
| *** mwhahaha has quit IRC | 21:51 | |
| *** patrickeast has quit IRC | 21:51 | |
| *** ying_zuo has quit IRC | 21:51 | |
| *** icey has quit IRC | 21:51 | |
| *** johnsom has quit IRC | 21:51 | |
| *** fungi has quit IRC | 21:51 | |
| *** fyxim has quit IRC | 21:51 | |
| *** redondo-mk has quit IRC | 21:51 | |
| *** vdrok has quit IRC | 21:51 | |
| *** MasterofJOKers has quit IRC | 21:51 | |
| *** tommylikehu has quit IRC | 21:51 | |
| *** Hazelesque has quit IRC | 21:51 | |
| *** mgariepy has quit IRC | 21:51 | |
| *** gryf has quit IRC | 21:51 | |
| *** raorn has quit IRC | 21:51 | |
| *** redondo-mk_ is now known as redondo-mk | 21:51 | |
| *** patrickeast_ is now known as patrickeast | 21:51 | |
| *** icey_ is now known as icey | 21:51 | |
| *** tommylikehu_ is now known as tommylikehu | 21:51 | |
| *** fyxim_ is now known as fyxim | 21:51 | |
| *** vdrok_ is now known as vdrok | 21:51 | |
| *** johnsom_ is now known as johnsom | 21:51 | |
| mriedem | i'll fix the link in our docs to point at nova/HACKING.rst, although that is pretty outdated | 21:52 |
| mriedem | 1. use mock, not stubs or mox; 2. easiest to start with an existing test over the same function, copy it and tweak for what you need to test | 21:52 |
| tasker | thanks for the suggestion. I'm doing 2 now and just saw 1 in one of the other tests. | 21:52 |
| mriedem | there might be an existing test for the OverLimit failure, and your change makes it fail if yo'ure not mocking out the unquiesce_instance call you added | 21:52 |
| mriedem | mtreinish: nova uses os-testr which uses stestr now right? | 21:52 |
| mriedem | so nova saying it's using testr isn't really accurate anymore | 21:52 |
| *** MasterofJOKers has joined #openstack-nova | 21:52 | |
| mtreinish | mriedem: actually nova is using stestr directly in most places now | 21:53 |
| *** fungi has joined #openstack-nova | 21:53 | |
| mriedem | ah yeah stestr run '{posargs}' | 21:53 |
| *** rmk has quit IRC | 21:54 | |
| *** dr_gogeta86 has quit IRC | 21:54 | |
| *** mgariepy has joined #openstack-nova | 21:55 | |
| *** dr_gogeta86 has joined #openstack-nova | 21:56 | |
| *** dr_gogeta86 has quit IRC | 21:56 | |
| *** dr_gogeta86 has joined #openstack-nova | 21:56 | |
| *** rmk has joined #openstack-nova | 21:56 | |
| *** rmk has quit IRC | 21:56 | |
| *** rmk has joined #openstack-nova | 21:56 | |
| *** mwhahaha has joined #openstack-nova | 21:56 | |
| mtreinish | mriedem: if you're updating the nova docs, the user manual for stestr is: http://stestr.readthedocs.io/en/latest/MANUAL.html | 21:59 |
| mriedem | yup, just linking to the main index | 21:59 |
| efried | patchbot seems to be down | 22:04 |
| *** rcernin has joined #openstack-nova | 22:09 | |
| *** mvk has joined #openstack-nova | 22:10 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: fix link to creating unit tests in contributor guide https://review.openstack.org/519482 | 22:15 |
| mriedem | efried: looks like it's working for me | 22:15 |
| efried | mriedem Okay. Maybe it just hates me. I just rebased this whole series https://review.openstack.org/#/c/516778/ and nothing popped up. | 22:16 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move remaining uses of parted to privsep. https://review.openstack.org/519483 | 22:20 |
| openstackgerrit | Michael Still proposed openstack/nova master: Convert users of tune2fs to privsep. https://review.openstack.org/519484 | 22:20 |
| *** armax has joined #openstack-nova | 22:22 | |
| *** AlexeyAbashkin has joined #openstack-nova | 22:26 | |
| *** AlexeyAbashkin has quit IRC | 22:31 | |
| *** liverpooler has quit IRC | 22:34 | |
| *** sree has joined #openstack-nova | 22:34 | |
| *** esberglu_ has joined #openstack-nova | 22:35 | |
| *** slaweq has quit IRC | 22:36 | |
| *** esberglu has quit IRC | 22:39 | |
| *** sree has quit IRC | 22:42 | |
| *** esberglu_ has quit IRC | 22:43 | |
| *** esberglu has joined #openstack-nova | 22:44 | |
| *** lyan has quit IRC | 22:44 | |
| *** lpetrut_ has quit IRC | 22:44 | |
| *** esberglu has quit IRC | 22:48 | |
| squid | hi, i seem to sporadically have nova snapshots failing, with rbd as the backend, it throws the error "ERROR oslo_messaging.rpc.server ImageNotAuthorized: Not authorized for image fc378659-9d6f-41d6-8c7f-a7d08c2bdb54" | 22:52 |
| squid | any idea what might be causing this? | 22:52 |
| *** squid is now known as calebb | 22:52 | |
| calebb | this is using the faster rbd backend snapshots as well | 22:54 |
| *** esberglu has joined #openstack-nova | 22:57 | |
| calebb | sorry i'll put this in #openstack | 22:58 |
| tasker | most places I see ImageNotAuthorized being thrown are a synonym to HTTPForbidden. check the OpenStack user that's initiating the call. Also check the glance / rbd authentication and make sure that glance has proper access to the requisite pools. | 22:59 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Test alloc_cands with non overlapping sharing RPs https://review.openstack.org/519380 | 22:59 |
| efried | gibi Hey, I noticed that one of the other tests related to bug 1730730 started passing when incorporated into Jay's series, so I rebased yours on the same. | 23:00 |
| openstack | bug 1730730 in OpenStack Compute (nova) "AllocationCandidates.get_by_filters returns garbage with only sharing providers" [Undecided,New] https://launchpad.net/bugs/1730730 | 23:00 |
| efried | gibi The behavior changed - we're no longer getting the partial candidates (missing some resources). But we are still getting the duplicates. See above. | 23:01 |
| calebb | tasker: thanks, i'll look into those things more, but at first glance (heh) they seem correct | 23:05 |
| openstackgerrit | Eric M Gonzalez (tasker) proposed openstack/nova master: unquiesce instance on volume snapshot failure https://review.openstack.org/519464 | 23:06 |
| *** xarlos has quit IRC | 23:06 | |
| tasker | still not seeing the test in my head yet. I've got one test at nova/tests/functional/regressions/test_bug_1554631.py that deals with cinder OverLimit but doesn't deal with the snapshot_volume_backed function. and then there's nova/tests/unit/compute/test_compute_api.py that has several tests of snapshot_volume_backed. | 23:08 |
| *** imacdonn has joined #openstack-nova | 23:09 | |
| *** gouthamr has quit IRC | 23:09 | |
| tasker | as best I can see it in my head, I want to mock the cinder_client. something like 1) cinder_client = mock({snapshot_volume.side_effect = cinderclient.exceptions.OverLimit}), 2) call snapshot_volume_backed with the mock cinder_client, 3) check that the proper exception was raised, and 4) check that the instance in unquiesecd / thawed. | 23:10 |
| mriedem | tasker: yes, i'd avoid the functional test you referenced, and just add a unit test | 23:11 |
| tasker | ok. | 23:11 |
| mriedem | test_bug_1554631 is dealing with the volume/snapshot proxy apis which are deprecated and not what you're trying to fix | 23:11 |
| mriedem | you'll also need to mock out the quiesce_instance call so that it "passes" | 23:11 |
| mriedem | tasker: looks like you could hack something into test_snapshot_volume_backed_with_quiesce | 23:13 |
| tasker | sorry .. first time using mock. so, this is a mock.patch(compute.api), altering quiesce_instance() and unquiesce_instance()? | 23:13 |
| mriedem | make fake_volume_create_snapshot raise an exception | 23:13 |
| mriedem | i'm not sure that you'd need to mock anything in there | 23:14 |
| *** esberglu has quit IRC | 23:15 | |
| *** mlavalle has quit IRC | 23:15 | |
| tasker | I thought about tweaking compute/test_compute_api.py to fit my needs but thought that might be too much. instead, I've fretted all afternoon. . / | 23:16 |
| tasker | I'll see what I can do with that. | 23:16 |
| mriedem | tasker: i'm thinking something like this http://paste.openstack.org/show/626204/ | 23:17 |
| tasker | yeah .. that's right where I was heading. | 23:18 |
| mriedem | except self.compute_api.snapshot_volume_backed will raise that OverQuota failure, so you'd need to handle that as well | 23:18 |
| tasker | overs="snapshots"? what's 'overs'? | 23:19 |
| mriedem | part of the OverQuota error message | 23:19 |
| mriedem | it's a kwarg | 23:19 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/exception.py#L1054 | 23:20 |
| mriedem | http://paste.openstack.org/show/626205/ | 23:20 |
| tasker | and then use "tox" to test it? | 23:23 |
| tasker | or stestr? | 23:25 |
| *** dikonoor has joined #openstack-nova | 23:28 | |
| mriedem | tox | 23:32 |
| mriedem | tasker: you can just run that one test like so: tox -e py27 -- test_snapshot_volume_backed_with_quiesce_create_snap_fails | 23:32 |
| mriedem | and now i'm afk for awhile | 23:32 |
| *** salv-orlando has quit IRC | 23:37 | |
| *** salv-orlando has joined #openstack-nova | 23:38 | |
| *** salv-orlando has quit IRC | 23:42 | |
| *** ying_zuo has joined #openstack-nova | 23:44 | |
| *** dikonoo has joined #openstack-nova | 23:45 | |
| *** gbarros has joined #openstack-nova | 23:46 | |
| openstackgerrit | Eric M Gonzalez (tasker) proposed openstack/nova master: unquiesce instance on volume snapshot failure https://review.openstack.org/519464 | 23:46 |
| tasker | mriedem: thanks for the help. got the test in place and it's passing. (never sure if that's a full good thing). | 23:47 |
| *** dikonoor has quit IRC | 23:48 | |
| *** gcb has quit IRC | 23:49 | |
| *** gbarros has quit IRC | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!