| *** diogogmt has quit IRC | 00:01 | |
| *** ctrath has quit IRC | 00:02 | |
| *** kaisers has quit IRC | 00:05 | |
| *** thorst has joined #openstack-nova | 00:05 | |
| *** ssurana has quit IRC | 00:06 | |
| *** doug-fish has joined #openstack-nova | 00:09 | |
| *** pratikmallya has quit IRC | 00:10 | |
| *** doug-fish has quit IRC | 00:14 | |
| *** markvoelker has quit IRC | 00:14 | |
| *** rk4n has quit IRC | 00:15 | |
| *** markvoelker has joined #openstack-nova | 00:17 | |
| auggy | i triaged a bug earlier which i marked as invalid, but i wanted to make sure i did the right thing - https://bugs.launchpad.net/nova/+bug/1535570 | 00:17 |
|---|---|---|
| openstack | Launchpad bug 1535570 in OpenStack Compute (nova) "Unexpected API Error in XenServer CI, test_get_console_output_with_unlimited_size" [Undecided,Invalid] | 00:17 |
| auggy | i'm looking at the ci dashboard and i'm seeing a few xen ci failures on the same test and i want to make sure there isn't a known issue or if there's something further i should be doing with this issue | 00:18 |
| auggy | the test is I believe in tempest and it's test_get_console_output_with_unlimited_size | 00:18 |
| auggy | and i've just seen a second failure on the same test for another commit - http://ci-watch.tintri.com/project?project=nova | 00:19 |
| auggy | this is the latest failure - https://review.openstack.org/#/c/264271/8 | 00:20 |
| auggy | but it's passing otherwise, so looks intermittent | 00:20 |
| auggy | i just want to make sure i don't need to reopen this bug and maybe redirect it to another project | 00:20 |
| *** kaisers has joined #openstack-nova | 00:21 | |
| *** n0ano has quit IRC | 00:21 | |
| *** baoli has joined #openstack-nova | 00:23 | |
| *** prometheanfire has quit IRC | 00:28 | |
| *** [1]Thelo has joined #openstack-nova | 00:31 | |
| *** baoli has quit IRC | 00:31 | |
| *** Thelo has quit IRC | 00:33 | |
| *** [1]Thelo is now known as Thelo | 00:33 | |
| *** baoli has joined #openstack-nova | 00:44 | |
| *** ZZelle_ has quit IRC | 00:44 | |
| *** Sukhdev has quit IRC | 00:44 | |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: Add test coverage to functional api tests _compare_result method https://review.openstack.org/263440 | 00:46 |
| openstackgerrit | Chuck Carmack proposed openstack/nova: REST API changes for user settable server description https://review.openstack.org/254950 | 00:46 |
| *** eil397 has quit IRC | 00:52 | |
| *** baoli has quit IRC | 00:53 | |
| *** gzhai has joined #openstack-nova | 00:54 | |
| *** abitha has quit IRC | 00:59 | |
| *** tjones has quit IRC | 01:01 | |
| *** yonglihe has quit IRC | 01:01 | |
| *** jinxing has joined #openstack-nova | 01:05 | |
| *** garyk has quit IRC | 01:05 | |
| *** unicell1 has joined #openstack-nova | 01:05 | |
| *** baoli has joined #openstack-nova | 01:05 | |
| *** jacky-zhang has joined #openstack-nova | 01:06 | |
| *** psargent has left #openstack-nova | 01:06 | |
| *** unicell has quit IRC | 01:07 | |
| *** vivekd has joined #openstack-nova | 01:09 | |
| *** jdurgin1 has quit IRC | 01:09 | |
| *** jacky-zhang_ has joined #openstack-nova | 01:09 | |
| *** jacky-zhang has quit IRC | 01:10 | |
| *** jacky-zhang_ is now known as jacky-zhang | 01:11 | |
| *** salv-orlando has quit IRC | 01:12 | |
| *** ivase has joined #openstack-nova | 01:14 | |
| *** baoli has quit IRC | 01:15 | |
| *** lewo has quit IRC | 01:17 | |
| *** EinstCrazy has joined #openstack-nova | 01:17 | |
| *** Sukhdev has joined #openstack-nova | 01:17 | |
| eliqiao | pkoniszewski: hi there? | 01:17 |
| *** ivase has quit IRC | 01:18 | |
| eliqiao | pkoniszewski: thanks for the explainaination on review #254101, I am very appreciate for you comments :) | 01:18 |
| *** kaisers1 has joined #openstack-nova | 01:19 | |
| *** gonzalo2kx has quit IRC | 01:19 | |
| *** Sukhdev has quit IRC | 01:19 | |
| *** Sukhdev has joined #openstack-nova | 01:19 | |
| *** ljxiash has joined #openstack-nova | 01:20 | |
| *** kaisers has quit IRC | 01:21 | |
| *** Sukhdev_ has joined #openstack-nova | 01:22 | |
| *** Sukhdev has quit IRC | 01:22 | |
| *** lewo has joined #openstack-nova | 01:23 | |
| eliqiao | Kevin_Zheng: hi are you there, do you have time to discuss about your patch #245005 | 01:23 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 01:24 |
| *** zhangjn has joined #openstack-nova | 01:24 | |
| *** su_zhang has quit IRC | 01:24 | |
| *** zz_dimtruck is now known as dimtruck | 01:24 | |
| *** ljxiash has quit IRC | 01:24 | |
| Kevin_Zheng | eliqiao: sure | 01:25 |
| *** Sukhdev_ has quit IRC | 01:25 | |
| *** mtanino has quit IRC | 01:26 | |
| *** tangchen__ has quit IRC | 01:29 | |
| *** akshai has quit IRC | 01:29 | |
| *** harlowja has quit IRC | 01:30 | |
| *** harlowja has joined #openstack-nova | 01:30 | |
| *** david-lyle has quit IRC | 01:31 | |
| *** Jeffrey4l has joined #openstack-nova | 01:31 | |
| eliqiao | Kevin_Zheng: sorry, just back to seat. | 01:32 |
| Kevin_Zheng | NP | 01:32 |
| eliqiao | Kevin_Zheng: for the live_migration_transport_target in CONF. I still don't think it's a good idea to pick it without any validation. | 01:33 |
| eliqiao | Kevin_Zheng: We can do this in 2 place, 1) in pre_live_migation (dest side), line 6411, do the validation and set it to None if it's not a valid ip address or hostname. | 01:34 |
| *** nic has quit IRC | 01:35 | |
| eliqiao | 2) catch exception when do live_migration operation if we catch up special exeption (for ex: host is not valid, not sure if libvirt will raise it out) and try to use dest host name again. | 01:35 |
| Kevin_Zheng | eliqiao: hmm, how can we check it is correct? | 01:35 |
| *** browne has joined #openstack-nova | 01:36 | |
| eliqiao | Since we(or admin) can not very sure about the ip/host name is valid(maybe mistake) for each computenode. | 01:36 |
| Kevin_Zheng | would you like to help on it? | 01:37 |
| eliqiao | Kevin_Zheng: simple way it to make sure it is ip address like or hostname like, this may not help since we don't know it can be accessed when we try to do an connect | 01:37 |
| *** dimtruck is now known as zz_dimtruck | 01:37 | |
| Kevin_Zheng | eliqiao: ip address is easy to check, Hostname is just a string, it will be harder | 01:38 |
| *** jinxing has quit IRC | 01:38 | |
| eliqiao | Kevin_Zheng: sure, I see. | 01:38 |
| eliqiao | Kevin_Zheng: as I side, we can not make sure it is a valid host/ip before we do a connect to that host. | 01:39 |
| eliqiao | Kevin_Zheng: we can search for some oslo utils of lib to see if there is any method to do such connection testing. | 01:39 |
| eliqiao | Kevin_Zheng: beside, | 01:39 |
| Kevin_Zheng | Hm, maybe | 01:40 |
| *** rk4n has joined #openstack-nova | 01:40 | |
| *** jinxing has joined #openstack-nova | 01:40 | |
| eliqiao | Kevin_Zheng: beside, API layer has a validation checking method "hostname_or_ip_address" | 01:40 |
| eliqiao | Kevin_Zheng: I know it may not help too much, but it's better to have this checking. | 01:40 |
| Kevin_Zheng | Hm, I will check that, | 01:41 |
| *** EinstCrazy has quit IRC | 01:41 | |
| eliqiao | Kevin_Zheng: the best scenario is admin configure a bad host/ip name, he can not do the live-migration on that node without any notification | 01:41 |
| *** kfarr has quit IRC | 01:41 | |
| eliqiao | Kevin_Zheng: I'd suggest that we can do simply testing use "hostname_or_ip_address" in check_can_live_migration_dest (it is early than pre_live_migration and we don) | 01:42 |
| eliqiao | sorry... :( | 01:42 |
| eliqiao | and we don't do any roll back. | 01:42 |
| *** haomaiwang has quit IRC | 01:42 | |
| Kevin_Zheng | yes, we can just use dest | 01:43 |
| Kevin_Zheng | to do the rest work | 01:43 |
| Kevin_Zheng | I will check it | 01:43 |
| *** rk4n_ has joined #openstack-nova | 01:43 | |
| eliqiao | Kevin_Zheng: okay, thanks for your response, if anything I can help, feel free to ping me :) | 01:43 |
| Kevin_Zheng | or do you want to help? I have to work on another one today, because it is non priority | 01:43 |
| Kevin_Zheng | it will be freezed tomorrow, I have to make changes to the codes | 01:44 |
| eliqiao | Kevin_Zheng: hmm.. do you mean #245005 will be freezed tomorrow ? or you are talking about your priority task ? | 01:45 |
| eliqiao | Kevin_Zheng: sure I can help | 01:45 |
| *** rk4n has quit IRC | 01:46 | |
| *** otter768 has joined #openstack-nova | 01:46 | |
| eliqiao | Kevin_Zheng: To my understanding, #245005 is LM related work, so it's wont be freezed until the end of Fed (see http://docs.openstack.org/releases/schedules/mitaka.html) | 01:47 |
| Kevin_Zheng | eliqiao: no #245005 is in the mitaka priority since is about live-migration, I have another task of expose quisce/unquiesce API this one is non-priority | 01:47 |
| Kevin_Zheng | yes | 01:47 |
| *** hemna has quit IRC | 01:47 | |
| eliqiao | Kevin_Zheng: Okay, then take your time, I will see if I can help on your patch. | 01:47 |
| Kevin_Zheng | eliqiao: Thanks, I will join you after I done the codes update | 01:48 |
| eliqiao | Kevin_Zheng: We still have time to discuss about your patch on LM meeting (maybe next Tuesday) | 01:48 |
| eliqiao | Kevin_Zheng: Okay, no worry. | 01:48 |
| *** Sukhdev has joined #openstack-nova | 01:49 | |
| *** mlavalle has quit IRC | 01:50 | |
| *** EinstCrazy has joined #openstack-nova | 01:50 | |
| *** Sukhdev has quit IRC | 01:50 | |
| *** armax has quit IRC | 01:51 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Split-network-plane-for-live-migration https://review.openstack.org/245005 | 01:52 |
| *** tjones has joined #openstack-nova | 01:52 | |
| Kevin_Zheng | eliqiao: you can add you as co-author if you want :) | 01:52 |
| *** armax has joined #openstack-nova | 01:54 | |
| *** EinstCrazy has quit IRC | 01:55 | |
| *** EinstCrazy has joined #openstack-nova | 01:55 | |
| *** EinstCra_ has joined #openstack-nova | 02:00 | |
| openstackgerrit | wuhao proposed openstack/nova: Add 'all_state' value to 'notify_on_state_change' option https://review.openstack.org/269720 | 02:00 |
| *** akshai has joined #openstack-nova | 02:01 | |
| *** nelsnelson has quit IRC | 02:03 | |
| *** EinstCrazy has quit IRC | 02:03 | |
| *** gonzalo2kx has joined #openstack-nova | 02:05 | |
| openstackgerrit | Vilobh Meshram proposed openstack/nova: ServiceGroup Refactoring: Add is_up to Service Obj https://review.openstack.org/269929 | 02:05 |
| *** jacky-zhang has quit IRC | 02:05 | |
| *** rk4n_ has quit IRC | 02:06 | |
| *** armax has quit IRC | 02:12 | |
| *** yonglihe has joined #openstack-nova | 02:14 | |
| *** nelsnelson has joined #openstack-nova | 02:21 | |
| *** armax has joined #openstack-nova | 02:22 | |
| *** ssurana has joined #openstack-nova | 02:23 | |
| *** dave-mccowan has joined #openstack-nova | 02:23 | |
| *** jacky-zhang has joined #openstack-nova | 02:24 | |
| *** zz_dimtruck is now known as dimtruck | 02:27 | |
| *** tjones has quit IRC | 02:28 | |
| *** diana_clarke has quit IRC | 02:29 | |
| *** browne has quit IRC | 02:30 | |
| *** baoli has joined #openstack-nova | 02:32 | |
| *** edmondsw has quit IRC | 02:37 | |
| *** tjones has joined #openstack-nova | 02:39 | |
| *** penick has quit IRC | 02:39 | |
| *** doug-fish has joined #openstack-nova | 02:39 | |
| *** tjones has quit IRC | 02:40 | |
| *** davideagnello has quit IRC | 02:40 | |
| *** davideagnello has joined #openstack-nova | 02:41 | |
| *** dansmith has quit IRC | 02:42 | |
| *** markvoelker has quit IRC | 02:42 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add swap volume notifications https://review.openstack.org/250283 | 02:42 |
| *** vilobhmm11 has quit IRC | 02:43 | |
| *** annegentle has quit IRC | 02:43 | |
| *** vilobhmm11 has joined #openstack-nova | 02:43 | |
| *** doug-fish has quit IRC | 02:44 | |
| *** dimtruck is now known as zz_dimtruck | 02:44 | |
| *** vishwanathj has joined #openstack-nova | 02:45 | |
| *** vishwanathj has quit IRC | 02:45 | |
| *** ljxiash has joined #openstack-nova | 02:45 | |
| *** vilobhmm11 has quit IRC | 02:45 | |
| *** tjones has joined #openstack-nova | 02:46 | |
| *** ljxiash_ has joined #openstack-nova | 02:46 | |
| *** gongysh has joined #openstack-nova | 02:47 | |
| *** PsionTheory has quit IRC | 02:47 | |
| *** dansmith has joined #openstack-nova | 02:48 | |
| *** dansmith is now known as Guest38462 | 02:48 | |
| *** ljxiash has quit IRC | 02:49 | |
| *** zhangjn has quit IRC | 02:53 | |
| *** jaypipes has quit IRC | 02:55 | |
| *** mdnadeem has joined #openstack-nova | 02:55 | |
| *** zz_dimtruck is now known as dimtruck | 02:55 | |
| *** baoli has quit IRC | 02:57 | |
| *** ducttape_ has joined #openstack-nova | 02:58 | |
| *** unicell1 has quit IRC | 03:03 | |
| *** haomaiwa_ has joined #openstack-nova | 03:04 | |
| *** houming has joined #openstack-nova | 03:05 | |
| *** browne has joined #openstack-nova | 03:06 | |
| *** baoli has joined #openstack-nova | 03:08 | |
| *** thorst has quit IRC | 03:09 | |
| *** apoorvad has quit IRC | 03:13 | |
| *** dimtruck is now known as zz_dimtruck | 03:15 | |
| *** su_zhang has joined #openstack-nova | 03:16 | |
| *** gongysh has quit IRC | 03:16 | |
| *** achanda has joined #openstack-nova | 03:17 | |
| *** tangchen has joined #openstack-nova | 03:19 | |
| *** coolsvap|away is now known as coolsvap | 03:20 | |
| *** yamahata has quit IRC | 03:20 | |
| *** jianghuaw_ has quit IRC | 03:20 | |
| tangchen | Hi alex_xu, oomichi, I'm so sorry that my patch was merge conflict again... And I have fixed it and passed all the CI. Please help to review again. https://review.openstack.org/#/c/202617/ | 03:21 |
| tangchen | alex_xu, oomichi: Thanks for your patient. :) | 03:21 |
| alex_xu | tangchen: will try afternoon | 03:21 |
| alex_xu | tangchen: np :) | 03:21 |
| *** zz_dimtruck is now known as dimtruck | 03:24 | |
| *** ducttape_ has quit IRC | 03:29 | |
| *** achanda has quit IRC | 03:32 | |
| *** vilobhmm11 has joined #openstack-nova | 03:32 | |
| *** houming has quit IRC | 03:35 | |
| *** ljxiash_ has quit IRC | 03:36 | |
| *** ljxiash has joined #openstack-nova | 03:37 | |
| *** achanda has joined #openstack-nova | 03:38 | |
| *** dimtruck is now known as zz_dimtruck | 03:39 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 03:40 |
| *** shyama has joined #openstack-nova | 03:41 | |
| *** ljxiash has quit IRC | 03:41 | |
| *** vivekd has quit IRC | 03:42 | |
| *** zhangjn has joined #openstack-nova | 03:42 | |
| *** houming has joined #openstack-nova | 03:43 | |
| *** takashin has joined #openstack-nova | 03:43 | |
| *** Marga_ has quit IRC | 03:44 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 03:44 |
| *** vivekd has joined #openstack-nova | 03:44 | |
| *** gyee has quit IRC | 03:44 | |
| *** EinstCrazy has joined #openstack-nova | 03:44 | |
| *** zz_dimtruck is now known as dimtruck | 03:44 | |
| *** Marga_ has joined #openstack-nova | 03:47 | |
| *** EinstCra_ has quit IRC | 03:47 | |
| *** akshai has quit IRC | 03:48 | |
| *** mragupat has joined #openstack-nova | 03:48 | |
| *** Marga_ has quit IRC | 03:51 | |
| *** dimtruck is now known as zz_dimtruck | 03:52 | |
| *** mragupat has quit IRC | 03:53 | |
| *** mragupat has joined #openstack-nova | 03:53 | |
| *** achanda_ has joined #openstack-nova | 03:54 | |
| *** dims has quit IRC | 03:54 | |
| *** achanda_ has quit IRC | 03:56 | |
| *** achanda has quit IRC | 03:56 | |
| *** akshai has joined #openstack-nova | 03:57 | |
| *** jacky-zhang has quit IRC | 03:58 | |
| *** akshai_ has joined #openstack-nova | 03:59 | |
| *** dims has joined #openstack-nova | 03:59 | |
| openstackgerrit | Vilobh Meshram proposed openstack/nova: ServiceGroup Refactoring: Add is_up to Service Obj https://review.openstack.org/269929 | 04:00 |
| *** akshai has quit IRC | 04:02 | |
| *** gonzalo2kx has quit IRC | 04:04 | |
| *** links has joined #openstack-nova | 04:06 | |
| *** annegentle has joined #openstack-nova | 04:06 | |
| *** akshai_ has quit IRC | 04:06 | |
| *** akshai has joined #openstack-nova | 04:07 | |
| *** Jeffrey4l has quit IRC | 04:08 | |
| *** otter768 has quit IRC | 04:10 | |
| *** annegentle has quit IRC | 04:10 | |
| *** dave-mccowan has quit IRC | 04:12 | |
| *** akshai has quit IRC | 04:12 | |
| *** akshai has joined #openstack-nova | 04:13 | |
| *** dims has quit IRC | 04:13 | |
| *** markvoelker has joined #openstack-nova | 04:14 | |
| *** ljxiash has joined #openstack-nova | 04:15 | |
| *** akshai has quit IRC | 04:18 | |
| *** mragupat has quit IRC | 04:21 | |
| *** vilobhmm11 has quit IRC | 04:21 | |
| *** zhangjn has quit IRC | 04:21 | |
| *** tjones has quit IRC | 04:25 | |
| *** baoli has quit IRC | 04:25 | |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: Base enablement for memory bandwidth monitor https://review.openstack.org/266773 | 04:28 |
| *** jacky-zhang has joined #openstack-nova | 04:29 | |
| *** Marga_ has joined #openstack-nova | 04:32 | |
| *** vilobhmm11 has joined #openstack-nova | 04:33 | |
| *** Marga_ has quit IRC | 04:36 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 04:39 |
| *** doug-fish has joined #openstack-nova | 04:42 | |
| *** su_zhang has quit IRC | 04:43 | |
| *** nelsnelson has quit IRC | 04:43 | |
| *** haomaiwa_ has quit IRC | 04:43 | |
| *** david-lyle has joined #openstack-nova | 04:44 | |
| *** haomaiwa_ has joined #openstack-nova | 04:46 | |
| *** Jeffrey4l has joined #openstack-nova | 04:46 | |
| *** Marga_ has joined #openstack-nova | 04:46 | |
| *** doug-fish has quit IRC | 04:46 | |
| *** Jeffrey4l has quit IRC | 04:47 | |
| *** Jeffrey4l has joined #openstack-nova | 04:47 | |
| *** doug-fish has joined #openstack-nova | 04:49 | |
| *** doug-fish has quit IRC | 04:53 | |
| openstackgerrit | Yingxin Cheng proposed openstack/nova: Refactor claim code to eliminate duplication https://review.openstack.org/262939 | 04:54 |
| openstackgerrit | Yingxin Cheng proposed openstack/nova: Scheduler use Claim to check resource consumption https://review.openstack.org/262938 | 04:55 |
| openstackgerrit | Vilobh Meshram proposed openstack/nova: Servicegroup Foundational Refactoring https://review.openstack.org/235637 | 05:02 |
| *** josecastroleon has quit IRC | 05:03 | |
| *** oomichi_away has quit IRC | 05:04 | |
| *** jaosorior has joined #openstack-nova | 05:06 | |
| *** su_zhang has joined #openstack-nova | 05:07 | |
| openstackgerrit | Alex Xu proposed openstack/nova: doc: add version discovery guideline in api concept doc https://review.openstack.org/264044 | 05:07 |
| openstackgerrit | Alex Xu proposed openstack/nova: doc: add client interactive guideline for microversions https://review.openstack.org/264045 | 05:07 |
| openstackgerrit | Alex Xu proposed openstack/nova: doc: completes microversion use-cases in api concept doc https://review.openstack.org/263220 | 05:07 |
| *** armax has quit IRC | 05:09 | |
| *** oomichi has joined #openstack-nova | 05:10 | |
| *** Sukhdev has joined #openstack-nova | 05:11 | |
| *** nelsnelson has joined #openstack-nova | 05:13 | |
| *** jacky-zhang has quit IRC | 05:14 | |
| openstackgerrit | Vilobh Meshram proposed openstack/nova: ServiceGroup Refactoring: Add is_up to Service Obj https://review.openstack.org/269929 | 05:14 |
| *** armax has joined #openstack-nova | 05:17 | |
| *** irenab has joined #openstack-nova | 05:18 | |
| *** vivekd_ has joined #openstack-nova | 05:23 | |
| *** vivekd has quit IRC | 05:23 | |
| *** vivekd_ is now known as vivekd | 05:24 | |
| openstackgerrit | Abhishek Kekane proposed openstack/nova: Python3: Replace dict.iteritems with six.iteritems https://review.openstack.org/268018 | 05:28 |
| *** jacky-zhang has joined #openstack-nova | 05:28 | |
| *** rk4n has joined #openstack-nova | 05:29 | |
| *** chhavi has joined #openstack-nova | 05:31 | |
| *** teju has joined #openstack-nova | 05:31 | |
| *** vivekd_ has joined #openstack-nova | 05:33 | |
| openstackgerrit | Anand Shanmugam proposed openstack/nova: Handle Instance Exceptions in add_fixed_ip and remove_fixed_ip https://review.openstack.org/266675 | 05:33 |
| *** vivekd has quit IRC | 05:35 | |
| *** vivekd_ is now known as vivekd | 05:35 | |
| *** achanda has joined #openstack-nova | 05:36 | |
| *** gcb has quit IRC | 05:47 | |
| *** gjayavelu has joined #openstack-nova | 05:47 | |
| *** Jeffrey4l has quit IRC | 05:47 | |
| *** salv-orlando has joined #openstack-nova | 05:49 | |
| *** yamahata has joined #openstack-nova | 05:53 | |
| *** gongysh has joined #openstack-nova | 05:54 | |
| *** jwcroppe_ has joined #openstack-nova | 06:00 | |
| *** tobasco has quit IRC | 06:00 | |
| *** tobasco has joined #openstack-nova | 06:02 | |
| *** fawadkhaliq has joined #openstack-nova | 06:02 | |
| *** jwcroppe has quit IRC | 06:02 | |
| *** nelsnelson has quit IRC | 06:03 | |
| *** Sukhdev has quit IRC | 06:03 | |
| *** paul-carlton2 has joined #openstack-nova | 06:04 | |
| *** unicell has joined #openstack-nova | 06:05 | |
| *** ljxiash has quit IRC | 06:06 | |
| *** annegentle has joined #openstack-nova | 06:06 | |
| *** ljxiash has joined #openstack-nova | 06:06 | |
| *** gcb has joined #openstack-nova | 06:07 | |
| *** Jeffrey4l has joined #openstack-nova | 06:07 | |
| *** ljxiash has quit IRC | 06:10 | |
| *** ljxiash has joined #openstack-nova | 06:10 | |
| *** otter768 has joined #openstack-nova | 06:11 | |
| *** annegentle has quit IRC | 06:11 | |
| *** vivekd has quit IRC | 06:11 | |
| *** lpetrut has joined #openstack-nova | 06:11 | |
| *** gongysh has quit IRC | 06:13 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 06:15 |
| *** otter768 has quit IRC | 06:15 | |
| *** Jeffrey4l has quit IRC | 06:18 | |
| *** lyanchih has joined #openstack-nova | 06:19 | |
| *** Lane_ has joined #openstack-nova | 06:19 | |
| *** lyanchih has quit IRC | 06:19 | |
| *** lyanchih has joined #openstack-nova | 06:20 | |
| *** Jeffrey4l has joined #openstack-nova | 06:21 | |
| *** achanda has quit IRC | 06:25 | |
| *** zhangjn has joined #openstack-nova | 06:26 | |
| *** Jeffrey4l has quit IRC | 06:26 | |
| *** achanda has joined #openstack-nova | 06:28 | |
| *** irenab has quit IRC | 06:31 | |
| *** ijw_ has joined #openstack-nova | 06:31 | |
| *** achanda has quit IRC | 06:32 | |
| *** achanda has joined #openstack-nova | 06:32 | |
| *** cfriesen has quit IRC | 06:33 | |
| *** ijw has quit IRC | 06:34 | |
| *** unicell has quit IRC | 06:34 | |
| *** ijw__ has joined #openstack-nova | 06:34 | |
| *** gjayavelu has quit IRC | 06:34 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata https://review.openstack.org/265601 | 06:35 |
| *** unicell has joined #openstack-nova | 06:36 | |
| *** wuhao has joined #openstack-nova | 06:37 | |
| *** ijw_ has quit IRC | 06:37 | |
| *** achanda has quit IRC | 06:38 | |
| *** jamielennox is now known as jamielennox|away | 06:40 | |
| *** rk4n_ has joined #openstack-nova | 06:40 | |
| *** ijw has joined #openstack-nova | 06:42 | |
| *** unicell1 has joined #openstack-nova | 06:42 | |
| *** fawadk has joined #openstack-nova | 06:43 | |
| *** unicell has quit IRC | 06:43 | |
| *** shihanzhang has quit IRC | 06:44 | |
| *** rk4n has quit IRC | 06:44 | |
| *** shihanzhang has joined #openstack-nova | 06:44 | |
| *** takashin has quit IRC | 06:44 | |
| *** takashin has joined #openstack-nova | 06:45 | |
| *** bradjones has quit IRC | 06:45 | |
| *** bradjones has joined #openstack-nova | 06:45 | |
| *** bradjones has quit IRC | 06:45 | |
| *** bradjones has joined #openstack-nova | 06:45 | |
| *** liverpooler has quit IRC | 06:45 | |
| *** ijw__ has quit IRC | 06:46 | |
| *** Jeffrey4l has joined #openstack-nova | 06:47 | |
| *** ankit_ag has joined #openstack-nova | 06:47 | |
| *** nkrinner has joined #openstack-nova | 06:47 | |
| *** fawadkhaliq has quit IRC | 06:47 | |
| *** codebauss has quit IRC | 06:48 | |
| *** lpetrut has quit IRC | 06:49 | |
| ankit_ag | ndipanov: hi | 06:50 |
| ankit_ag | ndipanov: I have addressed all your review comments on https://review.openstack.org/#/c/221162/ could you please take a look. | 06:50 |
| ankit_ag | Thanks | 06:50 |
| *** SridharG has joined #openstack-nova | 06:52 | |
| *** pbrooko has joined #openstack-nova | 06:54 | |
| *** jaosorior has quit IRC | 06:54 | |
| *** wuhao has quit IRC | 06:55 | |
| *** wuhao has joined #openstack-nova | 06:55 | |
| *** ajayaa has joined #openstack-nova | 06:55 | |
| *** ssurana has quit IRC | 06:57 | |
| *** liusheng has joined #openstack-nova | 06:57 | |
| *** haomai___ has joined #openstack-nova | 06:58 | |
| *** haomaiwa_ has quit IRC | 06:58 | |
| *** haomai___ has quit IRC | 07:01 | |
| *** wuhao has quit IRC | 07:01 | |
| *** 6A4ABQMYH has joined #openstack-nova | 07:01 | |
| *** vivekd has joined #openstack-nova | 07:01 | |
| *** rcernin has joined #openstack-nova | 07:02 | |
| *** browne has quit IRC | 07:02 | |
| *** wuhao has joined #openstack-nova | 07:02 | |
| *** wuhao has quit IRC | 07:03 | |
| *** wuhao has joined #openstack-nova | 07:03 | |
| *** _diana_ has quit IRC | 07:05 | |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: Base enablement for memory bandwidth monitor https://review.openstack.org/266773 | 07:06 |
| *** mjura_ has joined #openstack-nova | 07:08 | |
| *** mjura has joined #openstack-nova | 07:08 | |
| *** _diana_ has joined #openstack-nova | 07:09 | |
| *** chhavi has quit IRC | 07:11 | |
| *** markvoelker has quit IRC | 07:11 | |
| *** markvoelker has joined #openstack-nova | 07:12 | |
| *** codebauss has joined #openstack-nova | 07:15 | |
| *** markvoelker_ has joined #openstack-nova | 07:18 | |
| *** zhangjn has quit IRC | 07:18 | |
| *** scheuran has joined #openstack-nova | 07:19 | |
| *** markvoelker has quit IRC | 07:20 | |
| *** su_zhang has quit IRC | 07:20 | |
| *** kaisers1 has quit IRC | 07:20 | |
| *** zhangjn has joined #openstack-nova | 07:22 | |
| *** su_zhang has joined #openstack-nova | 07:24 | |
| *** gjayavelu has joined #openstack-nova | 07:25 | |
| *** tdurakov has joined #openstack-nova | 07:25 | |
| *** cfriesen has joined #openstack-nova | 07:26 | |
| tdurakov | morning everyone | 07:26 |
| *** chhavi has joined #openstack-nova | 07:28 | |
| *** kaisers has joined #openstack-nova | 07:29 | |
| *** gjayavelu has quit IRC | 07:30 | |
| *** vilobhmm11 has quit IRC | 07:32 | |
| *** rk4n_ has quit IRC | 07:32 | |
| *** zhangjn has quit IRC | 07:33 | |
| *** zenoway has joined #openstack-nova | 07:34 | |
| *** tdurakov has quit IRC | 07:34 | |
| *** tdurakov has joined #openstack-nova | 07:36 | |
| *** paul-carlton2 has quit IRC | 07:36 | |
| *** irenab has joined #openstack-nova | 07:37 | |
| *** cfriesen has quit IRC | 07:37 | |
| *** gszasz has joined #openstack-nova | 07:39 | |
| *** belmoreira has joined #openstack-nova | 07:40 | |
| *** su_zhang has quit IRC | 07:42 | |
| *** markvoel_ has joined #openstack-nova | 07:42 | |
| *** markvoelker_ has quit IRC | 07:43 | |
| *** Kennan has quit IRC | 07:45 | |
| *** tdurakov has quit IRC | 07:47 | |
| *** salv-orlando has quit IRC | 07:49 | |
| *** Kennan has joined #openstack-nova | 07:50 | |
| chhavi | ildikov: did u came across this DB error "AttributeError: '_TransactionContextManager' object has no attribute 'allow_async'" with the recent build | 07:53 |
| *** slunkad has quit IRC | 07:53 | |
| ildikov | chhavi: I haven't seen this | 07:54 |
| ildikov | chhavi: is this in your local env or in the Jenkins logs? | 07:54 |
| chhavi | there is an open bug for the same as well: https://bugs.launchpad.net/nova/+bug/1531930 | 07:54 |
| openstack | Launchpad bug 1531930 in OpenStack Compute (nova) "SQLalchemy API crashes executing migration_get_unconfirmed_by_dest_compute" [Undecided,New] | 07:54 |
| chhavi | facing the same issue in my env, during nova-compute restart | 07:55 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 07:55 |
| ildikov | I haven't touched migration, but I remember that I read activity around this or smth close to this in the last few days | 07:56 |
| chhavi | ok | 07:57 |
| *** otter768 has joined #openstack-nova | 08:00 | |
| *** 6A4ABQMYH has quit IRC | 08:01 | |
| *** haomaiwa_ has joined #openstack-nova | 08:01 | |
| *** mdnadeem has quit IRC | 08:01 | |
| *** otter768 has quit IRC | 08:02 | |
| *** otter768_ has joined #openstack-nova | 08:02 | |
| *** takashin has left #openstack-nova | 08:02 | |
| *** otter768 has joined #openstack-nova | 08:04 | |
| *** otter768_ has quit IRC | 08:04 | |
| *** slunkad has joined #openstack-nova | 08:05 | |
| *** otter768 has quit IRC | 08:05 | |
| *** markus_z has joined #openstack-nova | 08:06 | |
| *** otter768 has joined #openstack-nova | 08:06 | |
| *** annegentle has joined #openstack-nova | 08:07 | |
| *** rotbeard has joined #openstack-nova | 08:07 | |
| *** otter768_ has joined #openstack-nova | 08:08 | |
| *** Jeffrey4l has quit IRC | 08:08 | |
| *** otter76__ has joined #openstack-nova | 08:09 | |
| *** otter768_ has quit IRC | 08:10 | |
| *** otter768 has quit IRC | 08:10 | |
| *** rmart04 has joined #openstack-nova | 08:10 | |
| *** szhsong has joined #openstack-nova | 08:11 | |
| *** otter76__ has quit IRC | 08:11 | |
| *** annegentle has quit IRC | 08:12 | |
| *** gongysh has joined #openstack-nova | 08:13 | |
| *** Jeffrey4l has joined #openstack-nova | 08:14 | |
| *** mdnadeem has joined #openstack-nova | 08:15 | |
| *** rdopiera has joined #openstack-nova | 08:16 | |
| *** matrohon has joined #openstack-nova | 08:16 | |
| *** vivekd_ has joined #openstack-nova | 08:16 | |
| *** ihrachys has joined #openstack-nova | 08:18 | |
| *** vivekd has quit IRC | 08:19 | |
| *** vivekd_ is now known as vivekd | 08:19 | |
| *** gongysh has quit IRC | 08:21 | |
| *** jichen has joined #openstack-nova | 08:26 | |
| *** zakora has joined #openstack-nova | 08:28 | |
| *** fawadk has quit IRC | 08:29 | |
| *** liverpooler has joined #openstack-nova | 08:30 | |
| *** gongysh has joined #openstack-nova | 08:31 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 08:33 |
| *** liverpooler has quit IRC | 08:34 | |
| *** liverpooler has joined #openstack-nova | 08:35 | |
| *** gongysh has quit IRC | 08:36 | |
| *** tdurakov has joined #openstack-nova | 08:37 | |
| *** pbrooko_ has joined #openstack-nova | 08:42 | |
| *** coolsvap is now known as coolsvap|away | 08:43 | |
| *** jlanoux has joined #openstack-nova | 08:44 | |
| *** pbrooko has quit IRC | 08:45 | |
| *** derekh has joined #openstack-nova | 08:47 | |
| *** fawadkhaliq has joined #openstack-nova | 08:51 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova: XenAPI: create vdistore to stream vhd files https://review.openstack.org/266283 | 08:56 |
| *** fawadkhaliq has quit IRC | 08:56 | |
| openstackgerrit | Abhishek Kekane proposed openstack/nova: Python3: Replace dict.iteritems with six.iteritems https://review.openstack.org/268018 | 08:59 |
| *** haomaiwa_ has quit IRC | 09:01 | |
| *** haomaiwang has joined #openstack-nova | 09:01 | |
| *** tdurakov has quit IRC | 09:01 | |
| *** yamahata has quit IRC | 09:01 | |
| *** tdurakov has joined #openstack-nova | 09:02 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova: Added new scheduler filter: AggregateTypeExtraSpecsAffinityFilter https://review.openstack.org/189279 | 09:05 |
| *** ociuhandu has joined #openstack-nova | 09:05 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 09:07 |
| *** sahid has joined #openstack-nova | 09:08 | |
| *** pbrooko_ has quit IRC | 09:10 | |
| *** dedery has joined #openstack-nova | 09:11 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: Calculate block_migration in virt drivers if it's None https://review.openstack.org/254101 | 09:13 |
| openstackgerrit | Eli Qiao proposed openstack/nova: Return host and block_migration in task_api._execute() https://review.openstack.org/254102 | 09:13 |
| openstackgerrit | Jianghua Wang proposed openstack/nova: XenAPI: create vdistore to stream vhd files https://review.openstack.org/266283 | 09:15 |
| *** jistr has joined #openstack-nova | 09:16 | |
| *** dedery has quit IRC | 09:17 | |
| *** danpb has joined #openstack-nova | 09:20 | |
| *** ndipanov has quit IRC | 09:20 | |
| *** yassine__ has joined #openstack-nova | 09:22 | |
| *** pbrooko has joined #openstack-nova | 09:24 | |
| *** pbrooko has quit IRC | 09:24 | |
| *** fawadkhaliq has joined #openstack-nova | 09:26 | |
| alex_xu | johnthetubaguy: this trivial amendment need your help https://review.openstack.org/#/c/269664/, appreciate if you can take a look at it! | 09:27 |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: add memory bandwidth base class https://review.openstack.org/270079 | 09:29 |
| *** pbrooko has joined #openstack-nova | 09:29 | |
| *** BobBall_AWOL is now known as BobBall | 09:30 | |
| *** esracelik has joined #openstack-nova | 09:31 | |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: add memory bandwidth monitor base class https://review.openstack.org/270079 | 09:32 |
| *** Jeffrey4l has quit IRC | 09:34 | |
| johnthetubaguy | alex_xu: ah, yes, thats cool, add rebuild | 09:37 |
| alex_xu | johnthetubaguy: thanks :) | 09:37 |
| *** zenoway has quit IRC | 09:37 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Add new DB API method to retrieve migration for instance https://review.openstack.org/257270 | 09:39 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Add new API to force live migration to complete https://review.openstack.org/245921 | 09:39 |
| ildikov | johnthetubaguy: hi | 09:40 |
| ildikov | johnthetubaguy: I have a +2 on this one: https://review.openstack.org/#/c/193134/ | 09:40 |
| ildikov | johnthetubaguy: could you please take a look and hopefully approve it? :) | 09:41 |
| *** melwitt_ has joined #openstack-nova | 09:42 | |
| *** melwitt_ has quit IRC | 09:43 | |
| *** houming has quit IRC | 09:45 | |
| *** tdurakov has quit IRC | 09:45 | |
| *** jacky-zhang has quit IRC | 09:45 | |
| *** zenoway has joined #openstack-nova | 09:46 | |
| *** tdurakov has joined #openstack-nova | 09:46 | |
| johnthetubaguy | ildikov: I am unsure when I will get to that I am afraid | 09:47 |
| johnthetubaguy | ildikov: I will try to hit those | 09:47 |
| BobBall | johnthetubaguy: FYI updated https://etherpad.openstack.org/p/mitaka-nova-priorities-tracking with XenAPI VIF fix | 09:48 |
| *** omnipresent has joined #openstack-nova | 09:48 | |
| ildikov | johnthetubaguy: the one what I linked would be very important | 09:48 |
| *** pbrooko has quit IRC | 09:48 | |
| ildikov | johnthetubaguy: the next patch on top of that is a bug fix and the rest goes to Newton | 09:48 |
| openstackgerrit | Wen Zhi Yu proposed openstack/nova: Make sure destination will rollback if live migration fails https://review.openstack.org/270093 | 09:48 |
| ildikov | johnthetubaguy: I would appreciate if you can give the final push for that puppy | 09:49 |
| *** pbrooko has joined #openstack-nova | 09:50 | |
| tdurakov | johnthetubaguy, hi, as patch to project config is merged, job run is now available: https://review.openstack.org/#/c/252283/ | 09:50 |
| *** zenoway has quit IRC | 09:50 | |
| *** lpetrut has joined #openstack-nova | 09:51 | |
| *** efoley has joined #openstack-nova | 09:52 | |
| *** vivekd_ has joined #openstack-nova | 09:52 | |
| *** vivekd has quit IRC | 09:54 | |
| *** vivekd_ is now known as vivekd | 09:54 | |
| *** irenab has quit IRC | 09:56 | |
| *** FL1SK has quit IRC | 09:56 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova: Added new scheduler filter: AggregateTypeExtraSpecsAffinityFilter https://review.openstack.org/189279 | 09:56 |
| *** aix has joined #openstack-nova | 09:56 | |
| *** ljxiash has quit IRC | 09:57 | |
| johnthetubaguy | tdurakov: great to see the test result | 10:00 |
| *** haomaiwang has quit IRC | 10:01 | |
| *** EinstCra_ has joined #openstack-nova | 10:01 | |
| johnthetubaguy | tdurakov: just looking at that current sh file, is there any chance we can make that easier to read, mostly that bit at the bottom? it feels like there is a lot of cut and past, but slightly different code in there? | 10:01 |
| *** haomaiwang has joined #openstack-nova | 10:01 | |
| tdurakov | johnthetubaguy, will try to make it easier to read | 10:02 |
| *** jaosorior has joined #openstack-nova | 10:02 | |
| *** irenab has joined #openstack-nova | 10:03 | |
| *** jaosorior has quit IRC | 10:04 | |
| *** jaosorior has joined #openstack-nova | 10:04 | |
| *** EinstCrazy has quit IRC | 10:04 | |
| *** EinstCra_ has quit IRC | 10:06 | |
| johnthetubaguy | tdurakov: added some idea, I think creating a ceph.sh and nfs.sh and utils.sh that all contain a bunch of function would work well | 10:06 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: WIP: os-vif: convert libvirt driver to use os-vif library https://review.openstack.org/269672 | 10:06 |
| tdurakov | johnthetubaguy, yep, that what I'm working on right now | 10:08 |
| *** jinxing has quit IRC | 10:08 | |
| *** annegentle has joined #openstack-nova | 10:08 | |
| johnthetubaguy | tdurakov: awesome thank you | 10:08 |
| *** jaosorior_ has joined #openstack-nova | 10:09 | |
| *** jaosorior has quit IRC | 10:11 | |
| *** annegentle has quit IRC | 10:13 | |
| *** mariusv has quit IRC | 10:13 | |
| *** dedery has joined #openstack-nova | 10:14 | |
| markus_z | mikal: In case mriedem didn't already review it, could you have a look at https://review.openstack.org/#/q/topic:bp/libvirt-aio-mode ? danpb already +2'ed it | 10:14 |
| *** dedery_ has joined #openstack-nova | 10:15 | |
| openstackgerrit | Ryan Moore proposed openstack/nova: Add better help to rdp options https://review.openstack.org/253556 | 10:18 |
| openstackgerrit | Ryan Moore proposed openstack/nova: config options: centralize section: "rdp" https://review.openstack.org/252389 | 10:18 |
| *** dedery has quit IRC | 10:18 | |
| mikal | markus_z: sure, give me a sec to work out who I am first | 10:19 |
| markus_z | mikal: uh, I thought you are in the Australian timezone? | 10:20 |
| *** irenab_ has joined #openstack-nova | 10:20 | |
| *** irenab has quit IRC | 10:20 | |
| *** irenab_ is now known as irenab | 10:20 | |
| mikal | markus_z: nope, in the uk this week getting used to this time zone for the mid-cycle next week | 10:21 |
| mikal | markus_z: which means currently a little bit jet lagged | 10:21 |
| markus_z | mikal: ah, cool you're here. yeah, that's a mean distance for jet lags. Just ping me if you have questions to that patch sets. | 10:22 |
| *** shyama has quit IRC | 10:23 | |
| *** e0ne has joined #openstack-nova | 10:25 | |
| *** fawadkhaliq has quit IRC | 10:26 | |
| *** fawadkhaliq has joined #openstack-nova | 10:26 | |
| gibi | mikal: hi! thanks for started looking at the versioned notification patch chain. I fixed your comments in the first patch | 10:28 |
| *** links has quit IRC | 10:28 | |
| mikal | gibi: oh cool. I was planning on just doing the fix for you, but then johnthetubaguy snuck up behind me and distracted me | 10:28 |
| mikal | gibi: thanks for dealing with the nits | 10:29 |
| *** [1]Thelo has joined #openstack-nova | 10:31 | |
| gibi | mikal: any further comments or +2s are really appreciated. ;) | 10:31 |
| *** moorryan has joined #openstack-nova | 10:33 | |
| *** Thelo has quit IRC | 10:33 | |
| *** [1]Thelo is now known as Thelo | 10:33 | |
| *** irenab has quit IRC | 10:36 | |
| *** irenab has joined #openstack-nova | 10:36 | |
| *** jaosorior_ is now known as jaosorior | 10:39 | |
| *** lyanchih has quit IRC | 10:40 | |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova: VMware: Factor out relocate_vm() https://review.openstack.org/270115 | 10:43 |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova: VMware: Live migration of instances https://review.openstack.org/270116 | 10:43 |
| * johnthetubaguy shouts boo at mikal, although not from london this time | 10:44 | |
| johnthetubaguy | mikal: gibi: which patch was that again? | 10:45 |
| gibi | johnthetubaguy: https://review.openstack.org/#/c/247024/ | 10:45 |
| gibi | johnthetubaguy: the first patch of the versioned notification chain ^^ | 10:46 |
| * gibi goes for some lunch | 10:47 | |
| *** rfolco has joined #openstack-nova | 10:51 | |
| *** links has joined #openstack-nova | 10:51 | |
| *** efoley has quit IRC | 10:52 | |
| *** efoley has joined #openstack-nova | 10:55 | |
| *** markd_ has joined #openstack-nova | 10:57 | |
| *** haomaiwang has quit IRC | 11:01 | |
| *** haomaiwang has joined #openstack-nova | 11:01 | |
| *** gongysh has joined #openstack-nova | 11:04 | |
| *** rcernin has quit IRC | 11:05 | |
| *** cdent has joined #openstack-nova | 11:05 | |
| *** links has quit IRC | 11:05 | |
| *** rcernin has joined #openstack-nova | 11:07 | |
| *** sfinucan has joined #openstack-nova | 11:07 | |
| *** rcernin is now known as rcernin|lunch | 11:13 | |
| *** Marga_ has quit IRC | 11:19 | |
| ccarmack | all: is something wrong with jenkins? I'm getting this error: http://logs.openstack.org/64/269664/3/gate/gate-nova-specs-docs/d695eb0/console.html#_2016-01-20_09_39_48_425 | 11:19 |
| *** aysyd has joined #openstack-nova | 11:20 | |
| sdague | hmmmm large-ops fail rate way up in the last 24 hours http://tinyurl.com/zyqz8mo | 11:21 |
| *** daemontool has joined #openstack-nova | 11:21 | |
| johnthetubaguy | ccarmack: that does look like an infra related issue, maybe try folks on the infra channel? I see some folks discussing issues on there | 11:21 |
| johnthetubaguy | sulo: un-oh | 11:22 |
| johnthetubaguy | uh^ | 11:22 |
| sdague | ccarmack: that's the pip8 issue | 11:22 |
| sdague | it's on the mailing list | 11:22 |
| ccarmack | sdague, johnthetubaguy: ok, thanks | 11:22 |
| *** aysyd has quit IRC | 11:23 | |
| *** aysyd has joined #openstack-nova | 11:24 | |
| *** EinstCrazy has joined #openstack-nova | 11:31 | |
| *** links has joined #openstack-nova | 11:32 | |
| *** tdurakov has quit IRC | 11:35 | |
| cdent | I wish we could just abandon python libs from packages entirely | 11:35 |
| *** links has quit IRC | 11:38 | |
| *** ndipanov has joined #openstack-nova | 11:39 | |
| sdague | cdent: yeh, well its normally not our issue | 11:40 |
| cdent | sdague: When I say "we" I pretty much mean everyone, not just openstack. | 11:41 |
| cdent | everytime I've tried^wbeen forced to mix deb,rpm,brew python libs with pip libs I end up pissed off | 11:42 |
| *** openstackgerrit has quit IRC | 11:43 | |
| cdent | But then again I'm one of those bastards who thinks distros are where the puck was, not is and certainly not will be. | 11:43 |
| -openstackstatus- NOTICE: review.openstack.org is being restarted to apply patches | 11:43 | |
| *** ChanServ changes topic to "review.openstack.org is being restarted to apply patches" | 11:43 | |
| sdague | I still wonder if the lang env inherently supported versioning in libraries it would mitigate that | 11:43 |
| *** openstackgerrit has joined #openstack-nova | 11:44 | |
| cdent | I can see how that would have value but ugh, more complexity... | 11:45 |
| *** gongysh has quit IRC | 11:46 | |
| * cdent is a grape in the path of the steamroller of progress | 11:47 | |
| markus_z | ccarmack: I guess its related to this ML post: http://lists.openstack.org/pipermail/openstack-dev/2016-January/084475.html | 11:47 |
| BobBall | I'm sure grapes can be quite slippy under a steamroller... Enough grapes and it's sure to slide off course! | 11:47 |
| *** smatzek has joined #openstack-nova | 11:49 | |
| *** jacky-zhang has joined #openstack-nova | 11:50 | |
| cdent | BobBall: I'll have to find some comrades | 11:50 |
| *** ahelkhou has joined #openstack-nova | 11:52 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: Open specs for Neutron https://review.openstack.org/270148 | 11:53 |
| *** ChanServ changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack. Please see: https://wiki.openstack.org/wiki/Nova/Mitaka_Release_Schedule" | 11:54 | |
| -openstackstatus- NOTICE: Restart done, review.openstack.org is available | 11:54 | |
| cdent | johnthetubaguy: bit of a typo there ^ I think you meant newton? | 11:54 |
| johnthetubaguy | heh | 11:54 |
| mikal | johnthetubaguy: is it poorly socialized of me to insist that new config options be added straight into the new shiney central location? | 11:54 |
| cdent | I'm sure _tons_ of people are going to be making exactly the same error over and over and over | 11:55 |
| *** gongysh has joined #openstack-nova | 11:55 | |
| johnthetubaguy | mikal: good question, I have only being insisting for the stuff that has already moved | 11:55 |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: Open specs for Newton https://review.openstack.org/270148 | 11:55 |
| mikal | johnthetubaguy: yeah, I'm thinking totally new things like this versioned notification stuff | 11:56 |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: Open specs for Newton https://review.openstack.org/270148 | 11:56 |
| markus_z | ccarmack: http://grafana.openstack.org/dashboard/db/tempest-failure-rate?from=now-7d shows nicely when things go crazy. | 11:56 |
| johnthetubaguy | mikal: yeah, I haven't been force those either, but maybe thats bad, it would stop us creating work for ourselves | 11:57 |
| mikal | johnthetubaguy: its already oin patchset 28 though, so I think I'd be a terrible person to -1 for that | 11:57 |
| ccarmack | markus_z: thanks for the info | 11:57 |
| mikal | johnthetubaguy: moving them is realtively easy, do meh? | 11:57 |
| johnthetubaguy | mikal: well, there is that, maybe a follow on patch? | 11:57 |
| mikal | so meh even | 11:57 |
| johnthetubaguy | mikal: yeah, that | 11:57 |
| mikal | johnthetubaguy: yeah, I think do that | 11:57 |
| mikal | Also, I can't type on this crazy UK keyboard | 11:57 |
| mikal | Get QWERTY people! | 11:58 |
| *** aix has quit IRC | 11:59 | |
| markus_z | QWERTZ is the real deal | 12:00 |
| johnthetubaguy | lol | 12:00 |
| *** haomaiwang has quit IRC | 12:01 | |
| *** haomaiwang has joined #openstack-nova | 12:01 | |
| *** fawadkhaliq has quit IRC | 12:02 | |
| *** amotoki has joined #openstack-nova | 12:04 | |
| *** gongysh has quit IRC | 12:07 | |
| *** annegentle has joined #openstack-nova | 12:09 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: Open specs for Newton https://review.openstack.org/270148 | 12:09 |
| ccarmack | alex_xu: are you there? | 12:12 |
| *** annegentle has quit IRC | 12:13 | |
| *** boris-42 has quit IRC | 12:13 | |
| *** mjura has quit IRC | 12:15 | |
| openstackgerrit | Pavel Kholkin proposed openstack/python-novaclient: Support to boot a VM with network name https://review.openstack.org/225326 | 12:17 |
| *** vishwanathj has joined #openstack-nova | 12:18 | |
| *** achanda has joined #openstack-nova | 12:19 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: Reorder the priorities section https://review.openstack.org/270164 | 12:19 |
| *** Jeffrey4l has joined #openstack-nova | 12:21 | |
| openstackgerrit | Chuck Carmack proposed openstack/nova: REST API changes for user settable server description https://review.openstack.org/254950 | 12:21 |
| *** jaypipes has joined #openstack-nova | 12:23 | |
| *** efoley has quit IRC | 12:24 | |
| ccarmack | alex_xu ^ | 12:25 |
| *** doug-fish has joined #openstack-nova | 12:25 | |
| gibi | mikal, johnthetubaguy: thank for the +W on the first versioned notification patch :) | 12:30 |
| *** otter768 has joined #openstack-nova | 12:31 | |
| gibi | mikal: could you check the other patches in the series? john already +2d them as well | 12:33 |
| *** aix has joined #openstack-nova | 12:33 | |
| ccarmack | all: I'm looking for a 2nd core to review https://review.openstack.org/#/c/254950/12 | 12:36 |
| *** sahid has quit IRC | 12:36 | |
| *** otter768 has quit IRC | 12:36 | |
| *** claudiub has joined #openstack-nova | 12:39 | |
| *** tpatzig has joined #openstack-nova | 12:41 | |
| *** atuvenie has joined #openstack-nova | 12:42 | |
| *** thorst has joined #openstack-nova | 12:46 | |
| *** baoli has joined #openstack-nova | 12:47 | |
| *** dims has joined #openstack-nova | 12:49 | |
| *** vishalcdac07 has quit IRC | 12:49 | |
| cdent | sdague: what words are missing from "There is no one active on them, and the Rally team is." ? | 12:49 |
| sdague | the rally team is active | 12:50 |
| sdague | yeh, that's coffee #1 english for me .... sorry | 12:50 |
| *** sahid has joined #openstack-nova | 12:50 | |
| cdent | you mean in terms of performance investigation in general, so these tests are redundant? | 12:50 |
| *** ducttape_ has joined #openstack-nova | 12:50 | |
| sdague | I do not know if they are actually catching these kinds of issues, but I know they are an active team | 12:51 |
| *** baoli_ has joined #openstack-nova | 12:52 | |
| *** sahid has quit IRC | 12:52 | |
| *** sahid has joined #openstack-nova | 12:53 | |
| *** achanda has quit IRC | 12:54 | |
| *** efoley has joined #openstack-nova | 12:55 | |
| *** baoli has quit IRC | 12:55 | |
| *** ankit_ag has quit IRC | 12:56 | |
| cdent | I hope it warms up, upcountry, for next week. | 12:57 |
| *** ljxiash has joined #openstack-nova | 12:58 | |
| *** mdnadeem has quit IRC | 12:58 | |
| *** Jeffrey4l has quit IRC | 12:59 | |
| *** davidhadas has joined #openstack-nova | 12:59 | |
| *** hop has joined #openstack-nova | 13:00 | |
| *** haomaiwang has quit IRC | 13:01 | |
| *** haomaiwa_ has joined #openstack-nova | 13:01 | |
| *** ljxiash has quit IRC | 13:02 | |
| *** takedakn has joined #openstack-nova | 13:02 | |
| *** takedakn has quit IRC | 13:02 | |
| *** dedery has joined #openstack-nova | 13:03 | |
| openstackgerrit | Dinesh Bhor proposed openstack/nova: Fix invalid import order https://review.openstack.org/270179 | 13:04 |
| *** matrohon has quit IRC | 13:04 | |
| *** matrohon has joined #openstack-nova | 13:05 | |
| alex_xu | ccarmack: will try to review again | 13:05 |
| ccarmack | alex_xu: thanks | 13:06 |
| alex_xu | ccarmack: np | 13:06 |
| *** dedery_ has quit IRC | 13:06 | |
| *** rcernin|lunch is now known as rcernin | 13:07 | |
| *** haomaiwa_ has quit IRC | 13:07 | |
| *** mdnadeem has joined #openstack-nova | 13:07 | |
| ccarmack | alex_xu: do you have a suggestion for a 2nd core to review it? | 13:08 |
| openstackgerrit | Wen Zhi Yu proposed openstack/nova: Make sure destination will rollback if live migration fails https://review.openstack.org/270093 | 13:13 |
| *** ducttape_ has quit IRC | 13:14 | |
| *** doug-fish has quit IRC | 13:14 | |
| *** doug-fish has joined #openstack-nova | 13:16 | |
| *** jacky-zhang has quit IRC | 13:17 | |
| *** Jeffrey4l has joined #openstack-nova | 13:17 | |
| *** daemontool has quit IRC | 13:18 | |
| openstackgerrit | dstepanenko proposed openstack/nova: Go away from using PciDeviceStats consume_requests https://review.openstack.org/243131 | 13:19 |
| openstackgerrit | dstepanenko proposed openstack/nova: Added support_requests method to PciDevicePoolList https://review.openstack.org/242087 | 13:19 |
| *** belmoreira has quit IRC | 13:19 | |
| *** efoley has quit IRC | 13:19 | |
| alex_xu | ccarmack: good to have oomichi to take a look at it also | 13:22 |
| *** pumaranikar has joined #openstack-nova | 13:23 | |
| *** dguitarbite has quit IRC | 13:23 | |
| *** rfolco has quit IRC | 13:25 | |
| *** rfolco has joined #openstack-nova | 13:25 | |
| ccarmack | alex_xu: ok... oomichi, can you take a look at https://review.openstack.org/#/c/254950/ ? | 13:25 |
| *** dguitarbite has joined #openstack-nova | 13:28 | |
| *** eglynn has joined #openstack-nova | 13:28 | |
| *** jacky-zhang has joined #openstack-nova | 13:29 | |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'block_device_mapping' https://review.openstack.org/262023 | 13:30 |
| ansiwen | is `python setup.py compile_catalog` supposed to work, or is nobody caring about internationalization? | 13:31 |
| *** vladikr has joined #openstack-nova | 13:32 | |
| *** kluka_ has joined #openstack-nova | 13:32 | |
| *** ljxiash has joined #openstack-nova | 13:33 | |
| kluka_ | Hi, I have a question about "nova image-create". Is it possible to force the snapshot format? Like qcow2 for example? | 13:34 |
| *** dslev has joined #openstack-nova | 13:34 | |
| sdague | ansiwen: AJaeger in #openstack-infra is probably the right person to ask on i18n bits | 13:34 |
| kluka_ | On test OS setup I get QCOW2 format snapshot, but second OS setup I always get RAW format... | 13:35 |
| kluka_ | Thanks in advance. | 13:35 |
| *** haomaiwa_ has joined #openstack-nova | 13:35 | |
| lyarwood | does anyone know if stable/liberty tarballs will be uploaded to launchpad as with previous stable releases? | 13:39 |
| *** eharney has quit IRC | 13:39 | |
| sdague | ansiwen: we seem to be missing a test and there are a few errors in the es translation | 13:40 |
| sdague | AJaeger is looking to fix it now | 13:40 |
| *** FL1SK has joined #openstack-nova | 13:40 | |
| openstackgerrit | Andreas Jaeger proposed openstack/nova: Validate translations https://review.openstack.org/270190 | 13:42 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Add metadata objects for device tagging https://review.openstack.org/264015 | 13:43 |
| *** dslev has quit IRC | 13:43 | |
| *** aysyd has quit IRC | 13:43 | |
| *** annegentle has joined #openstack-nova | 13:44 | |
| *** MCoLo has quit IRC | 13:44 | |
| *** takedakn has joined #openstack-nova | 13:45 | |
| *** MCoLo has joined #openstack-nova | 13:45 | |
| *** aysyd has joined #openstack-nova | 13:45 | |
| *** jwang_ has joined #openstack-nova | 13:46 | |
| *** smatzek_ has joined #openstack-nova | 13:46 | |
| *** zenoway has joined #openstack-nova | 13:47 | |
| *** ihrachys_ has joined #openstack-nova | 13:47 | |
| *** rmart04_ has joined #openstack-nova | 13:48 | |
| *** hemna_ has joined #openstack-nova | 13:48 | |
| *** ociuhandu_ has joined #openstack-nova | 13:49 | |
| *** phschwartz_ has joined #openstack-nova | 13:49 | |
| *** teju has left #openstack-nova | 13:50 | |
| *** crinkle_ has joined #openstack-nova | 13:50 | |
| mikal | gibi: I'm reviewing the entire series nowish, just slowly | 13:50 |
| *** sshen_ has joined #openstack-nova | 13:50 | |
| *** mdbooth_ has joined #openstack-nova | 13:50 | |
| *** FL1SK has quit IRC | 13:50 | |
| *** hwoarang_ has joined #openstack-nova | 13:50 | |
| *** ansiwen_ has joined #openstack-nova | 13:50 | |
| *** baffle__ has joined #openstack-nova | 13:51 | |
| *** dosaboy_ has joined #openstack-nova | 13:51 | |
| *** liverpoo1er has joined #openstack-nova | 13:51 | |
| *** tcammann1 has joined #openstack-nova | 13:51 | |
| *** lifeless_ has joined #openstack-nova | 13:51 | |
| *** frickler_ has joined #openstack-nova | 13:51 | |
| *** tonyb_ has joined #openstack-nova | 13:51 | |
| *** clduser_ has joined #openstack-nova | 13:51 | |
| *** gerhardq1x has joined #openstack-nova | 13:51 | |
| *** j_king_ has joined #openstack-nova | 13:51 | |
| *** kfox1111_ has joined #openstack-nova | 13:52 | |
| sdague | ansiwen: I think things are fixed in the upstream translation engine now, we'll get a code import overnight, that should make it work again. Then https://review.openstack.org/#/c/270190 will make sure we don't import any more breaks. | 13:53 |
| *** MCoLo has left #openstack-nova | 13:53 | |
| *** lee_ has joined #openstack-nova | 13:53 | |
| *** lee_ is now known as lyarwood_ | 13:53 | |
| sdague | mikal / alex_xu / johnthetubaguy - can I pre get a second +2 on this - https://review.openstack.org/#/c/270190 - it will fail jenkins today, but will pass tomorrow once the translations are fixed | 13:54 |
| openstackgerrit | Andreas Jaeger proposed openstack/nova: Validate translations https://review.openstack.org/270190 | 13:54 |
| *** rodrigod` has joined #openstack-nova | 13:54 | |
| *** coreycb` has joined #openstack-nova | 13:54 | |
| *** raildo` has joined #openstack-nova | 13:54 | |
| *** mkoderer_ has joined #openstack-nova | 13:54 | |
| *** mc_nair_ has joined #openstack-nova | 13:54 | |
| *** raginbaj- has joined #openstack-nova | 13:55 | |
| *** keekz_ has joined #openstack-nova | 13:55 | |
| *** htruta` has joined #openstack-nova | 13:55 | |
| *** toabctl_ has joined #openstack-nova | 13:55 | |
| *** sb- has joined #openstack-nova | 13:55 | |
| *** krotscheck_ has joined #openstack-nova | 13:55 | |
| *** markmc` has joined #openstack-nova | 13:55 | |
| *** bradjones_ has joined #openstack-nova | 13:55 | |
| *** leseb- has joined #openstack-nova | 13:55 | |
| *** cfarquhar_ has joined #openstack-nova | 13:55 | |
| *** bradjones_ is now known as Guest7791 | 13:55 | |
| mikal | sdague: I'll look | 13:55 |
| *** lyarwood_ has left #openstack-nova | 13:55 | |
| *** smatzek has quit IRC | 13:56 | |
| *** omnipresent has quit IRC | 13:56 | |
| *** ociuhandu has quit IRC | 13:56 | |
| *** ihrachys has quit IRC | 13:56 | |
| *** chhavi has quit IRC | 13:56 | |
| *** notmorgan has quit IRC | 13:56 | |
| *** ababich has quit IRC | 13:56 | |
| *** sshen has quit IRC | 13:56 | |
| *** mdbooth has quit IRC | 13:56 | |
| *** zzzeek has quit IRC | 13:56 | |
| *** ansiwen has quit IRC | 13:56 | |
| *** markmc has quit IRC | 13:56 | |
| *** hwoarang has quit IRC | 13:56 | |
| *** mnaser has quit IRC | 13:56 | |
| *** dosaboy has quit IRC | 13:56 | |
| *** raginbajin has quit IRC | 13:56 | |
| *** _gryf has quit IRC | 13:56 | |
| *** leseb_ has quit IRC | 13:56 | |
| *** phschwartz has quit IRC | 13:56 | |
| *** anteaya has quit IRC | 13:56 | |
| *** rmstar has quit IRC | 13:56 | |
| *** tcammann has quit IRC | 13:56 | |
| *** frickler has quit IRC | 13:56 | |
| *** kfox1111 has quit IRC | 13:56 | |
| *** gerhardqux has quit IRC | 13:56 | |
| *** raildo has quit IRC | 13:56 | |
| *** coreycb has quit IRC | 13:56 | |
| *** htruta has quit IRC | 13:56 | |
| *** rodrigods has quit IRC | 13:56 | |
| *** liverpooler has quit IRC | 13:56 | |
| *** rmart04 has quit IRC | 13:56 | |
| *** liusheng has quit IRC | 13:56 | |
| *** SridharG has quit IRC | 13:56 | |
| *** bradjones has quit IRC | 13:56 | |
| *** cfarquhar has quit IRC | 13:56 | |
| *** keekz has quit IRC | 13:56 | |
| *** mc_nair has quit IRC | 13:56 | |
| *** toabctl has quit IRC | 13:56 | |
| *** jwang has quit IRC | 13:56 | |
| *** mkoderer has quit IRC | 13:56 | |
| *** dulek has quit IRC | 13:56 | |
| *** crinkle has quit IRC | 13:56 | |
| *** BobBall has quit IRC | 13:56 | |
| *** lbeliveau has quit IRC | 13:56 | |
| *** tonyb has quit IRC | 13:56 | |
| *** andreykurilin has quit IRC | 13:56 | |
| *** hemnafk has quit IRC | 13:56 | |
| *** lifeless has quit IRC | 13:56 | |
| *** lyarwood has quit IRC | 13:56 | |
| *** krotscheck has quit IRC | 13:56 | |
| *** j_king has quit IRC | 13:56 | |
| *** q3k has quit IRC | 13:56 | |
| *** baffle has quit IRC | 13:56 | |
| *** strictlyb has quit IRC | 13:56 | |
| *** clduser has quit IRC | 13:56 | |
| *** ociuhandu_ is now known as ociuhandu | 13:56 | |
| *** krotscheck_ is now known as krotscheck | 13:56 | |
| *** rmart04_ is now known as rmart04 | 13:56 | |
| *** liusheng has joined #openstack-nova | 13:56 | |
| *** rmstar has joined #openstack-nova | 13:56 | |
| *** zzzeek has joined #openstack-nova | 13:56 | |
| *** dulek has joined #openstack-nova | 13:56 | |
| *** raginbaj- is now known as raginbajin | 13:56 | |
| *** mdbooth_ is now known as mdbooth | 13:57 | |
| *** mc_nair_ is now known as mc_nair | 13:57 | |
| mikal | johnthetubaguy: you still around? | 13:57 |
| *** toabctl_ is now known as toabctl | 13:57 | |
| *** BobBall has joined #openstack-nova | 13:58 | |
| *** BobBall is now known as Guest45232 | 13:59 | |
| *** migi has quit IRC | 13:59 | |
| johnthetubaguy | mikal: I am here | 13:59 |
| * johnthetubaguy he hopes | 13:59 | |
| mikal | johnthetubaguy: so, I'm a bit concerned about the syntax in one of these notification changes, and I am unsure if you are unconcerned, or didn't notice when you did your review | 14:00 |
| mikal | We have: | 14:00 |
| johnthetubaguy | mikal: which bit is that? | 14:00 |
| mikal | if ({'disabled', 'disabled_reason', 'forced_down'} & | 14:00 |
| mikal | set(updates.keys())): | 14:00 |
| *** toabctl is now known as toabctl_ | 14:00 | |
| mikal | Which I had to go lookup, as syntax I've never seen before | 14:00 |
| *** migi has joined #openstack-nova | 14:00 | |
| *** takedakn has quit IRC | 14:00 | |
| *** dave-mccowan has joined #openstack-nova | 14:00 | |
| *** ansiwen_ is now known as ansiwen | 14:00 | |
| mikal | Which worries me, because its a bit non-intuituive | 14:00 |
| johnthetubaguy | mikal: I certainly did a double take | 14:00 |
| *** xyang1 has joined #openstack-nova | 14:00 | |
| *** atuvenie has quit IRC | 14:01 | |
| *** haomaiwa_ has quit IRC | 14:01 | |
| jaypipes | HenryG: on hangout... :) | 14:01 |
| *** jwcroppe_ is now known as jwcroppe | 14:01 | |
| *** haomaiwang has joined #openstack-nova | 14:01 | |
| mikal | So, we can do this instead: | 14:01 |
| mikal | set(keys).intersection({'disabled', 'disabled_reason', 'forced_down'}) | 14:01 |
| mikal | Edit | 14:01 |
| mikal | 14:01 | |
| mikal | Which is at least self documenting | 14:01 |
| *** eharney has joined #openstack-nova | 14:01 | |
| johnthetubaguy | mikal: yeah, thats an easier read | 14:01 |
| *** Jeffrey4l has quit IRC | 14:01 | |
| mikal | I'll comment as such i suppose | 14:02 |
| *** chhavi has joined #openstack-nova | 14:02 | |
| mikal | I just wasn't sure if you had had similar concerns | 14:02 |
| *** PsionTheory has joined #openstack-nova | 14:02 | |
| *** eharney is now known as Guest983 | 14:02 | |
| *** psargent has joined #openstack-nova | 14:02 | |
| *** q3k has joined #openstack-nova | 14:02 | |
| *** mnaser has joined #openstack-nova | 14:02 | |
| *** jinxing has joined #openstack-nova | 14:02 | |
| *** rlrossit has joined #openstack-nova | 14:03 | |
| *** _gryf has joined #openstack-nova | 14:03 | |
| *** anteaya has joined #openstack-nova | 14:03 | |
| *** tonyb_ is now known as tonyb | 14:03 | |
| *** ababich has joined #openstack-nova | 14:04 | |
| *** andreykurilin has joined #openstack-nova | 14:04 | |
| *** raildo` is now known as raildo | 14:04 | |
| edleafe | mikal: (reading scrollback) If the config options haven't been moved yet, it would make a mess to put some in one place and others somewhere else. | 14:04 |
| mikal | edleafe: this was a new config group though, so I don't think that applies here | 14:05 |
| edleafe | mikal: we might insist on following the more verbose, readable help text, thoguh | 14:05 |
| ansiwen | sdague: thanks. I just wanted to fix our packaging, and realized that compile_catalog only compiles some of the .po files... So I wondered if this is a waste of efford, since it it's not taken care of upstream, so I might jst delete the .po files... | 14:05 |
| *** SridharG has joined #openstack-nova | 14:05 | |
| edleafe | mikal: ah. Then yes, anything new should be added to nova/conf | 14:05 |
| *** notmorgan has joined #openstack-nova | 14:06 | |
| mikal | edleafe: well, given it was on patchset 28 I gave up and approved, it was just one option, so its a relatively easy fix later | 14:06 |
| edleafe | mikal: oh, totally agree on that bit. I meant more of a general rule | 14:06 |
| mikal | edleafe: ahh, I see. Agreed | 14:07 |
| *** ctrath has joined #openstack-nova | 14:07 | |
| *** jacky-zhang has quit IRC | 14:08 | |
| *** tdurakov has joined #openstack-nova | 14:08 | |
| *** thorst has quit IRC | 14:09 | |
| *** AJaeger has joined #openstack-nova | 14:09 | |
| *** thorst has joined #openstack-nova | 14:09 | |
| AJaeger | ansiwen, sdague : REgarding compile_catalog only compiling some files: That's a problem with the setup ;( | 14:10 |
| AJaeger | Let me explain: The babel setup only handles *one* po file. | 14:10 |
| *** Guest983 is now known as eharney | 14:10 | |
| *** eharney has quit IRC | 14:10 | |
| *** eharney has joined #openstack-nova | 14:10 | |
| *** mdrabe has joined #openstack-nova | 14:11 | |
| mordred | mikal: wow - bitwise-and works as an intersection operator between sets? | 14:11 |
| AJaeger | So, it will only translate the nova.pot file | 14:11 |
| AJaeger | For translation of the warning etc files, you need to run commands like | 14:12 |
| cdent | mordred: yup, I actually find that form more readable than the suggested alternative, but I guess that's just from whatever you're used to | 14:12 |
| AJaeger | msgfmt on all of the warning/log/error pot files and for all languages. | 14:12 |
| mordred | cdent: fascinating. well, I guess today is a success - I have learned something new | 14:13 |
| mordred | I can go ahead and start drinking now | 14:13 |
| *** ducttape_ has joined #openstack-nova | 14:13 | |
| cdent | #success modred learned something new | 14:13 |
| openstackstatus | cdent: Added success to Success page | 14:13 |
| cdent | dammit! | 14:13 |
| ansiwen | AJaeger: so, you cannot teach setup to take more than one file? | 14:13 |
| AJaeger | ;) | 14:13 |
| cdent | that was not a success | 14:13 |
| *** efoley_ has joined #openstack-nova | 14:13 | |
| * cdent needs to learn to type | 14:13 | |
| *** efoley_ is now known as efoley | 14:13 | |
| AJaeger | ansiwen: not that I'm aware off ;( | 14:14 |
| *** jacky-zhang has joined #openstack-nova | 14:14 | |
| *** shyama has joined #openstack-nova | 14:14 | |
| sdague | ansiwen: it got missed, master will be fixed tomorrow | 14:14 |
| ansiwen | AJaeger: http://babel.pocoo.org/en/latest/setup.html there is some stuff about method mapping... | 14:14 |
| * cdent fixed it | 14:14 | |
| sdague | AJaeger: can you backport that to liberty / kilo as well ? | 14:14 |
| *** tdurakov has quit IRC | 14:15 | |
| AJaeger | sdague: the tox change? Sure... | 14:15 |
| ansiwen | sdague: how will it be fixed? with using msgfmt? | 14:15 |
| sdague | ansiwen: well that will ensure we don't have any bad messages | 14:15 |
| sdague | is there another issue? | 14:15 |
| ansiwen | sdague: oh, yes | 14:15 |
| *** dslevin has joined #openstack-nova | 14:15 | |
| *** dslevin has quit IRC | 14:16 | |
| ansiwen | only some .po files are compiled | 14:16 |
| *** lyarwood has joined #openstack-nova | 14:16 | |
| sdague | oh, this is because of the translation domains? | 14:16 |
| ansiwen | setup.py only looks at novy.po files, all aother po files are ignored | 14:16 |
| *** dslevin has joined #openstack-nova | 14:16 | |
| sdague | sigh | 14:16 |
| ansiwen | sdague: so, if we can't teach setup.py to also look at other files, we would need to use msgfmt and do it on our own | 14:17 |
| gibi | mikal: I appreciate your effort! | 14:17 |
| sdague | AJaeger: we may need to revisit the translation domains then, if it's breaking the output / packaging | 14:17 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Add tag column to vifs and bdm https://review.openstack.org/264014 | 14:17 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Add metadata objects for device tagging https://review.openstack.org/264015 | 14:17 |
| rlrossit | mordred: It gets more fun too! | does union, - does difference, and ^ does xor | 14:17 |
| AJaeger | sdague: for extracting, we run: python setup.py $QUIET extract_messages --no-default-keywords --keyword _LE --output-file ${POT} | 14:18 |
| rlrossit | cdent: I like these ^ better for syntax because it's more math-like | 14:18 |
| AJaeger | something similar shoudl work for compile_catalog | 14:18 |
| *** tdurakov has joined #openstack-nova | 14:18 | |
| sdague | AJaeger: ok, but is this breaking normal packaging flow? | 14:18 |
| *** busterswt has joined #openstack-nova | 14:19 | |
| AJaeger | sdague: let me put this on my translation list - to find a solution | 14:19 |
| sdague | AJaeger: yeh, that would be great | 14:19 |
| sdague | thanks | 14:19 |
| AJaeger | sdague: it just means that the _LE files are most likely not translated currently by packages. | 14:19 |
| AJaeger | The main files get compiled, just the _LE/_LW/... not | 14:19 |
| psargent | hello! Does anyone know if it's possible to create a VM with CLI commands (nova boot or other nova command?) without actually booting the VM? Thanks! | 14:20 |
| *** speller has joined #openstack-nova | 14:21 | |
| *** dane-fichter has joined #openstack-nova | 14:21 | |
| *** ducttape_ has quit IRC | 14:22 | |
| cdent | rlrossit: I think I got hooked on them with ^ | 14:22 |
| wznoinsk | claudiub, hi | 14:22 |
| claudiub | wznoinsk: hello | 14:22 |
| ansiwen | AJaeger: so the splitting in several .po files per language is a recent change? | 14:22 |
| wznoinsk | I was wondering how can I set a os_live_resize Set in glance's image properties on the command line for instance live resize to work, it's passed as unicode atm | 14:23 |
| *** tdurakov has quit IRC | 14:23 | |
| wznoinsk | <- claudiub, | 14:23 |
| AJaeger | ansiwen: two years ago I think... | 14:25 |
| wznoinsk | claudiub, for the moment I changed it to be a regular list to get going but was wondering how it's done properly, the tests are passing Set no problem, the command line is one I have problemw ith | 14:25 |
| sdague | AJaeger: right, but that seems like quite an issue, given that the work people do in touching those files is largely wasted | 14:25 |
| ansiwen | AJaeger: oh, so it's broken since then? obviously nobody is missing the translations... ;-) | 14:26 |
| AJaeger | ansiwen: first import for nova in June 2014 | 14:26 |
| sdague | ansiwen: yeh this translation domain issue goes way back | 14:26 |
| speller | Has the ability to create a VM shutdown, rather than powered up been discussed? | 14:26 |
| psargent | speller: don't think so | 14:27 |
| claudiub | wznoinsk: you mean glance image-update <image_id> --property os_live_resize="disk,memory" ? | 14:27 |
| AJaeger | sdague, ansiwen: I implemented at that time the *infra* side of this as suggested by dhellmann, let me check with him whether anything in projects wsa forgotten/overseen... | 14:27 |
| *** ivase has joined #openstack-nova | 14:27 | |
| wznoinsk | didn't work for me | 14:27 |
| openstackgerrit | Sean Dague proposed openstack/nova: cleanup: remove wsgi serialize/deserialize decorators https://review.openstack.org/270215 | 14:28 |
| wznoinsk | claudiub, although I've tried with only one value, let me recheck | 14:28 |
| ansiwen | AJaeger, sdague: if nobody complained since then, is the internationalization worth it then? (besides frontends like horizon of course) | 14:28 |
| claudiub | wznoinsk: hmm, wondering if it's missing the patch that adds the property to the ImageMetaProps object... | 14:28 |
| AJaeger | ansiwen, sdague : Looking at oslo.i18n, there's everything prepared, just the compile part seems missing | 14:28 |
| *** irenab has quit IRC | 14:29 | |
| speller | @psargent: Ok thnx. Should I request it on launchpad or mailing list? | 14:29 |
| AJaeger | ansiwen: 90 % of the strings are in nova.pot and those get compiled. The main focus of translation team is indeed on horizon and friends | 14:30 |
| AJaeger | ansiwen: feel free to ask/comment on openstack-i18n mailing list | 14:30 |
| *** irenab has joined #openstack-nova | 14:30 | |
| psargent | speller: the mailing list would be great! | 14:30 |
| speller | @psargent: Amazing thank you | 14:30 |
| ansiwen | AJaeger: is babel compiling the po files on the fly if the mo files are missing? | 14:31 |
| AJaeger | ansiwen: I doubt it - but don't know | 14:31 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added db API layer to add instance tag-list filtering support https://review.openstack.org/248805 | 14:31 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added server tags controller https://review.openstack.org/128940 | 14:31 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added server tags support in nova-api https://review.openstack.org/268932 | 14:31 |
| *** smatzek_ has quit IRC | 14:31 | |
| *** mriedem has joined #openstack-nova | 14:31 | |
| ansiwen | AJaeger: because I found this issue, because somebody complained that we have the .po files still in the packages... but maybe that's the reason why nobody noticed, because they were created during runtime | 14:32 |
| *** otter768 has joined #openstack-nova | 14:32 | |
| AJaeger | sdague: I'm not backport to kilo - there are no more translation imports to kilo. Will do liberty now | 14:32 |
| *** thorst has quit IRC | 14:32 | |
| sdague | AJaeger: ok | 14:32 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: doc: add devref about versioned notifications https://review.openstack.org/263788 | 14:32 |
| sdague | AJaeger: do we know if kilo works? | 14:33 |
| *** edmondsw has joined #openstack-nova | 14:33 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Add service status notification https://review.openstack.org/245678 | 14:33 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Generate doc for versioned notifications https://review.openstack.org/265151 | 14:33 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Adds json sample for the versioned notifications https://review.openstack.org/248167 | 14:33 |
| gibi | mikal: fixed your comment in https://review.openstack.org/#/c/245678/ | 14:34 |
| AJaeger | sdague: I can check - but first let me fix the liberty problems... | 14:34 |
| sdague | AJaeger: there are 3 fatal errors in kilo | 14:34 |
| *** w_verdugo has quit IRC | 14:34 | |
| sdague | it's actually the same issue in the es catalog | 14:34 |
| sdague | about placeholders | 14:35 |
| AJaeger | That old ;( So nobody ever used ;( That's sad ;( | 14:35 |
| mikal | gibi: thanks | 14:35 |
| *** dedery has quit IRC | 14:35 | |
| *** dedery has joined #openstack-nova | 14:36 | |
| *** liverpoo1er has quit IRC | 14:36 | |
| sdague | AJaeger: yeh, well ... I don't know | 14:36 |
| *** mtanino has joined #openstack-nova | 14:36 | |
| sdague | but it seems like being broken isn't good | 14:36 |
| *** otter768 has quit IRC | 14:36 | |
| *** w_verdugo has joined #openstack-nova | 14:39 | |
| wznoinsk | claudiub, nova.api.openstack.extensions ValueError: A set is required in field os_live_resize, not a unicode - that's after setting exactly how you suggested | 14:40 |
| wznoinsk | claudiub, can't find anything relevant in gerrit, could you help? | 14:40 |
| *** bklei has joined #openstack-nova | 14:40 | |
| *** tpatzig has quit IRC | 14:40 | |
| speller | @psargent: Is that the general of the dev mailing list? | 14:41 |
| *** ivase has quit IRC | 14:41 | |
| *** ivase has joined #openstack-nova | 14:41 | |
| mikal | johnthetubaguy: that entire versioned notification review series is now approved and awaiting merge | 14:42 |
| johnthetubaguy | mikal: awesomeness | 14:42 |
| gibi | mikal: thanks a lot! | 14:42 |
| mikal | johnthetubaguy: I did a think and fast pathed +W for the last one where there was a one line syntax change compared to what you'd +2'ed | 14:43 |
| mikal | thing even | 14:43 |
| *** aysyd has quit IRC | 14:43 | |
| johnthetubaguy | cool | 14:43 |
| *** aysyd has joined #openstack-nova | 14:43 | |
| openstackgerrit | Sean Dague proposed openstack/nova: cleanup: collapse wsgi serializer test hierarchy https://review.openstack.org/270227 | 14:44 |
| AJaeger | sdague: liberty patch: https://review.openstack.org/270223 | 14:45 |
| sdague | AJaeger: +2 | 14:46 |
| *** ivase has quit IRC | 14:46 | |
| *** knikolla has joined #openstack-nova | 14:47 | |
| *** ivase has joined #openstack-nova | 14:47 | |
| sdague | it is amazing how much of nova/api/openstack/wsgi.py is vestigial now with the xml drop, I'm going to just keep hacking bits out of it | 14:48 |
| mikal | sdague: murderer | 14:48 |
| mriedem | AJaeger: sdague: can someone explain what's up with the pot files patches? | 14:49 |
| mriedem | there is no bug, just a bunch of patches | 14:49 |
| *** ahelkhou has quit IRC | 14:49 | |
| sdague | mriedem: url? | 14:50 |
| mriedem | https://review.openstack.org/#/q/Ic1f106684270a8f2197e0fb2a67d6fc00535ee65,n,z | 14:50 |
| mnaser | maybe someone can enlighten me about this but didnt nova have a TRACE debug level which tracebacks were printed inside? | 14:50 |
| mriedem | mnaser: that would be in oslo.log | 14:50 |
| mnaser | tracebacks are coming in ERROR which is making the logging a bit more messy than it has to be | 14:51 |
| *** keekz_ is now known as keekz | 14:52 | |
| mriedem | mnaser: the difference is the pid is in the traceback lines http://logs.openstack.org/67/269867/2/check/gate-tempest-dsvm-neutron-src-os-brick/bef7c8e/logs/screen-q-svc.txt.gz?level=TRACE#_2016-01-19_22_50_02_398 | 14:52 |
| sdague | mriedem: t? means 0 or 1 t | 14:52 |
| AJaeger | mriedem: we noticed that compile_catalog fails with current pot files and put a tox change in place - like it's done for keystone - so that imports from the translation bot will not be able to pass | 14:52 |
| sdague | AJaeger: it could use a bug | 14:52 |
| sdague | honestly, we could use 2 bugs, one for this, and one for the missing catalogs | 14:52 |
| AJaeger | on my way... | 14:53 |
| mriedem | ok, i just have like 0 context and there wasn't much in the commit message, but i'm expecting stable branch changes | 14:53 |
| mriedem | and the tox.ini change says to check po files but the regex is only checking pot files | 14:53 |
| sdague | mriedem: right, stable branch is also impacted | 14:53 |
| mriedem | i'm not sure if the comment is wrong or the regex | 14:53 |
| sdague | mriedem: no, it is not | 14:53 |
| mnaser | mriedem: oh i see, there isn't a "TRACE" log level? I guess we'd have to filter those out to have more concise log filtering that indicates the problem clearly | 14:53 |
| sdague | you are just reading the regex wrong | 14:53 |
| sdague | '.*\.pot?' | 14:53 |
| mnaser | ex: ideally we'd like to be alerted on this only: "2016-01-19 23:04:01.898 ERROR oslo_db.api [req-0ad2d235-5995-4a19-bc92-00010da2cde0 tempest-NegativeSecGroupTest-628194806 tempest-NegativeSecGroupTest-2239149] DB error." | 14:53 |
| AJaeger | mriedem: run the find command locally, it works for me ;) | 14:53 |
| sdague | match anything, then a dot, then po, then 0 or 1 t | 14:53 |
| mriedem | oh | 14:54 |
| mriedem | sneaky regex | 14:54 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [python3] byte/string conversions and enable more tests https://review.openstack.org/261045 | 14:54 |
| mriedem | so the change on master fails, i'm assuming that has to wait for the next translation dump? | 14:54 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [python3] Port more code to python3 and enable tests https://review.openstack.org/262083 | 14:54 |
| sdague | mriedem: yes | 14:54 |
| sdague | upstream translations fixed | 14:54 |
| sdague | for master | 14:54 |
| *** jacky-zhang has quit IRC | 14:55 | |
| wznoinsk | claudiub, I presume you have this patch in the works so I'll work with lists for the moment to work on virt implementation | 14:55 |
| mriedem | which should be https://review.openstack.org/#/c/265601/ | 14:55 |
| sdague | that should come in tomorrow, we can then fast approve this after so we don't regress | 14:55 |
| mriedem | oh, ok | 14:55 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'ec2_instance' and 'instance_fault' https://review.openstack.org/261979 | 14:55 |
| *** cfriesen has joined #openstack-nova | 14:55 | |
| mriedem | mnaser: you'd probably have to ask dims about the TRACE log level, i thought there was one | 14:56 |
| sdague | mriedem: yeh that's what will be updated | 14:56 |
| mriedem | mnaser: but for our logstash filtering we have to key off the pid for multiline trace | 14:56 |
| mnaser | mriedem: gotcha, do you know where i can find that filter you're using, might be more useful than me writing it out :) | 14:57 |
| AJaeger | mriedem, sdague: is https://bugs.launchpad.net/nova/+bug/1536214 detailed enough? | 14:57 |
| openstack | Launchpad bug 1536214 in OpenStack Compute (nova) "PO files broken" [Undecided,New] | 14:57 |
| claudiub | wznoinsk: yeah, list is fine as well. I have to check out what happens, worse case scenario, it will be a list instead of a set. Thanks for letting me know. :) | 14:57 |
| mriedem | mnaser: i think the openstack-infra/system-config repo | 14:57 |
| *** ducttape_ has joined #openstack-nova | 14:57 | |
| mnaser | i will be on the lookup, thanks mriedem | 14:57 |
| *** tdurakov has joined #openstack-nova | 14:57 | |
| AJaeger | mriedem: I fixed upstream translations for master and liberty today - so tomorrow'S import should be fine, and then the change I did should pass | 14:57 |
| AJaeger | mriedem: currently the change will fail - and that shows it does the right thing ;) | 14:57 |
| mriedem | mnaser: https://github.com/openstack-infra/system-config/blob/master/modules/openstack_project/templates/logstash/indexer.conf.erb | 14:59 |
| mriedem | https://github.com/openstack-infra/system-config/commit/686886c8d6df70fe4fc54c8e54e16ac1cd52126c | 14:59 |
| *** smatzek_ has joined #openstack-nova | 15:00 | |
| mnaser | that's exactly what i need, thank you so much mriedem | 15:00 |
| *** haomaiwang has quit IRC | 15:01 | |
| *** mgoddard__ has joined #openstack-nova | 15:01 | |
| pkholkin | alaski: Andrew, many many thanks for your reviews of bp/new-oslodb-enginefacade patches!!! Answered to all your comments in all patches :) | 15:01 |
| *** haomaiwa_ has joined #openstack-nova | 15:01 | |
| ansiwen | AJaeger, sdague: if I can help on the other issue (missing catalogs), I'm happy to do so, because I'm also interested in this | 15:01 |
| *** su_zhang has joined #openstack-nova | 15:02 | |
| *** ahelkhou has joined #openstack-nova | 15:02 | |
| *** daemontool has joined #openstack-nova | 15:02 | |
| mriedem | AJaeger: the bug looks fine, please add that to the patches | 15:02 |
| *** rodrigod` is now known as rodrigods | 15:02 | |
| *** tdurakov has quit IRC | 15:03 | |
| ildikov | mriedem: hi. could you please give this puppy the final push: https://review.openstack.org/#/c/193134 ? | 15:03 |
| mriedem | AJaeger: so i suppose we just released nova liberty 12.0.01 with busted translations :( | 15:03 |
| ildikov | mriedem: already +2d by jaypipes | 15:03 |
| mriedem | ildikov: yeah, it's still early, on my list | 15:04 |
| *** mgoddard_ has quit IRC | 15:04 | |
| AJaeger | mriedem: bug added | 15:04 |
| mriedem | on https://review.openstack.org/#/c/270190/ too please | 15:04 |
| AJaeger | mriedem: those busted translations are in for quite some time ;(((( | 15:04 |
| mriedem | i'll quick re-approve | 15:04 |
| *** josecastroleon has joined #openstack-nova | 15:04 | |
| ildikov | mriedem: cool, tnx, I just wouldn't want to miss the deadline after all the misery with it | 15:04 |
| AJaeger | mriedem: is on both - and first we need to import translations (tomorrow 6:00 UTC) | 15:05 |
| mriedem | AJaeger: ? the bug ref isn't in the commit message for the master branch change https://review.openstack.org/#/c/270190/ | 15:05 |
| *** jaosorior has quit IRC | 15:06 | |
| *** tdurakov has joined #openstack-nova | 15:06 | |
| *** zz_dimtruck is now known as dimtruck | 15:06 | |
| *** jschlueter has quit IRC | 15:06 | |
| openstackgerrit | Andreas Jaeger proposed openstack/nova: Validate translations https://review.openstack.org/270190 | 15:07 |
| AJaeger | mriedem: I forgot to publish ;( Thanks for double checking! | 15:07 |
| *** daemontool has quit IRC | 15:07 | |
| mriedem | thanks | 15:07 |
| mriedem | +W | 15:07 |
| *** daemontool has joined #openstack-nova | 15:07 | |
| ansiwen | AJaeger: if you didn't do so I will create a bug for the missing catalogs... | 15:07 |
| AJaeger | thanks, mriedem | 15:07 |
| AJaeger | ansiwen: just finished with https://bugs.launchpad.net/openstack-i18n/+bug/1536226 - please enhance | 15:08 |
| openstack | Launchpad bug 1536226 in openstack i18n "Not all pot files compiled" [Undecided,New] | 15:08 |
| *** mrkz has joined #openstack-nova | 15:08 | |
| *** thorst has joined #openstack-nova | 15:08 | |
| *** thorst has quit IRC | 15:08 | |
| *** thorst has joined #openstack-nova | 15:08 | |
| ansiwen | AJaeger: ok | 15:08 |
| *** pratikmallya has joined #openstack-nova | 15:09 | |
| AJaeger | mriedem: I 'recheck' both changes once translations are imported - and will ping tomorrow for translation import | 15:09 |
| mriedem | AJaeger: ok | 15:09 |
| *** pratikma_ has joined #openstack-nova | 15:10 | |
| mriedem | ildikov: i see that i have bugs in my tempest tests for multiattach, so i also have to get that updated | 15:11 |
| openstackgerrit | dstepanenko proposed openstack/nova: Add PciDevicePoolList.get_by_compute_node method https://review.openstack.org/247692 | 15:12 |
| ildikov | mriedem: you mean issues with the multiattach implementation or with this first patch? | 15:12 |
| openstackgerrit | dstepanenko proposed openstack/nova: Add PciDevicePoolList.get_all method https://review.openstack.org/250757 | 15:12 |
| mriedem | ildikov: i mean my tempest change to add the multiattach test has bugs, | 15:13 |
| mriedem | but i wanted to see that passing | 15:13 |
| *** daemontool has quit IRC | 15:13 | |
| *** su_zhang has quit IRC | 15:13 | |
| *** ahelkhou has quit IRC | 15:13 | |
| mriedem | http://logs.openstack.org/33/266633/2/check/gate-tempest-dsvm-full/8795802/console.html#_2016-01-19_21_37_56_913 | 15:13 |
| ildikov | mriedem: a-ha, ok, I wasn't sure as that depends on my changes | 15:13 |
| *** pratikmallya has quit IRC | 15:13 | |
| ildikov | mriedem: I'll take a look | 15:14 |
| mriedem | ildikov: so on https://review.openstack.org/#/c/193134/79/nova/api/openstack/compute/legacy_v2/contrib/volumes.py | 15:14 |
| mriedem | you added a comment but ndipanov and i asked that you don't assume it's an OrderedDict and just pull the first entry from the dict and use that | 15:14 |
| alaski | pkholkin: thanks, will go them again in a bit | 15:14 |
| *** burt has joined #openstack-nova | 15:15 | |
| pkholkin | alaski: happy to hear) | 15:15 |
| ildikov | mriedem: sorry, the last comment from ndipanov was to add a comment and I assumed that's the verdict | 15:16 |
| openstackgerrit | Andrea Rosa proposed openstack/nova: Adding volume operations for shelved instances https://review.openstack.org/259528 | 15:16 |
| *** boris-42 has joined #openstack-nova | 15:16 | |
| mriedem | "We really should not rely on that - it seems like a hack and whether that's an ordered dict or not is squarely an internal implementation detail of the cinder module - no way to test for breakage in the future :/" | 15:16 |
| ildikov | but it's not coming from Cinder | 15:17 |
| mriedem | he meant the cinder volume api in nova | 15:17 |
| mriedem | but i tend to agree that the rest api shouldn't rely on the volume api using an OrderedDict | 15:17 |
| ildikov | ah, ok | 15:17 |
| mriedem | that's gross tight coupling | 15:18 |
| mriedem | let me go through the rest of the changes before you update | 15:18 |
| ildikov | it should not be there for long | 15:18 |
| ildikov | and if someone is not hacking directly in Cinder all the volumes will have one attachment | 15:18 |
| ndipanov | mriedem, ildikov yeah I am not super happy with that assumpiton - can't we just make it a list | 15:18 |
| ndipanov | how many entries do we realistically expect there? | 15:18 |
| ildikov | ndipanov: johnthetubaguy asked for this change to make the checks easier for multiattach | 15:19 |
| mriedem | i'm fine if it's a dict, | 15:19 |
| mriedem | i just don't like relying on it being an OrderedDict | 15:19 |
| ildikov | ndipanov: when we update the api to return all the volume attachments it will not matter whether it's a dict or list | 15:19 |
| ndipanov | ildikov, you mean the cinder API... | 15:20 |
| *** szhsong has quit IRC | 15:20 | |
| johnthetubaguy | ildikov: which bit is this, pass the attachement_ids down? | 15:21 |
| ildikov | johnthetubaguy: no, to transform the attachments list to a dict with instance_uuid as key | 15:22 |
| *** jinxing has quit IRC | 15:22 | |
| *** andymaier has joined #openstack-nova | 15:22 | |
| ildikov | ndipanov: I mean the Nova REST API, basicly that part of the code that we're talking about now | 15:22 |
| *** andymaier has quit IRC | 15:23 | |
| ildikov | ndipanov: well, not the legacy part though as I guess we cannot change that anyway | 15:23 |
| ndipanov | yeah... | 15:23 |
| ndipanov | well | 15:23 |
| *** andymaier has joined #openstack-nova | 15:23 | |
| *** EinstCrazy has quit IRC | 15:24 | |
| *** phschwartz_ is now known as phschwartz | 15:25 | |
| *** bauzas has quit IRC | 15:25 | |
| *** markvoel_ has quit IRC | 15:25 | |
| johnthetubaguy | ildikov: it just looked like that would make it less messy, but yeah not an ordereddict, that doesn't feel right | 15:25 |
| ansiwen | AJaeger: so, you see any other option than to merge the files again? | 15:26 |
| mriedem | ftr, i don't care if the cinder volume_api code is using an ordereddict or not, i just don't want the rest api relying on that | 15:26 |
| ildikov | johnthetubaguy: currently we can return only one attachment on the API | 15:26 |
| ildikov | as it currently returns the first attachment in the list that Cinder sends | 15:27 |
| johnthetubaguy | ildikov: which API is that? | 15:27 |
| ildikov | johnthetubaguy: we do the translation to the internal format here: https://review.openstack.org/#/c/193134/79/nova/volume/cinder.py | 15:28 |
| ildikov | line 165, where we get the first attachment in the list on the left | 15:28 |
| AJaeger | ansiwen: proper documentation on how to build them | 15:28 |
| *** mdnadeem has quit IRC | 15:28 | |
| ildikov | johnthetubaguy: this is the api where I use it currently: https://review.openstack.org/#/c/193134/79/nova/api/openstack/compute/legacy_v2/contrib/volumes.py | 15:28 |
| ildikov | johnthetubaguy: same for the not legacy one | 15:28 |
| ansiwen | AJaeger: you mean with a find ... -exec msgfmt ... ? | 15:29 |
| *** bauzas has joined #openstack-nova | 15:30 | |
| AJaeger | or something similar... | 15:30 |
| *** yamahata has joined #openstack-nova | 15:30 | |
| johnthetubaguy | ildikov: seems like the API allows multiple attachements, but just doesn't show more than one right now? | 15:31 |
| *** tonytan4ever has joined #openstack-nova | 15:32 | |
| johnthetubaguy | ildikov: when I look at this: http://developer.openstack.org/api-ref-compute-v2.1.html#listVolumesDetailed | 15:32 |
| *** mragupat has joined #openstack-nova | 15:32 | |
| *** cdelatte has joined #openstack-nova | 15:32 | |
| ildikov | johnthetubaguy: you mean the current Nova REST API? | 15:32 |
| mriedem | johnthetubaguy: the rest api can't change the response w/o a microversion to show multiple attachments | 15:32 |
| mriedem | johnthetubaguy: which happens in newton | 15:32 |
| mriedem | and you can't have multiple attachments today anyway w/o the libvirt changes later in the series | 15:32 |
| johnthetubaguy | mriedem: but the current API seems to imply multiple attachments are possible, its just not used yet | 15:33 |
| ansiwen | AJaeger: well... is that feasible? I'm quite new to python, but I guess people have strict expectations what setup.py should do... | 15:33 |
| ildikov | johnthetubaguy: yeah, it's a list already | 15:33 |
| *** diogogmt has joined #openstack-nova | 15:33 | |
| johnthetubaguy | np | 15:34 |
| johnthetubaguy | oops | 15:34 |
| mriedem | johnthetubaguy: honestly i'm not really sure why the rest api changes need to be in here, i think they are just done because the cinder volume_api is changing the translation format | 15:34 |
| mriedem | to do it as a dict rather than a list | 15:34 |
| johnthetubaguy | mriedem: yeah, true | 15:34 |
| AJaeger | ansiwen: let's figure it out ... | 15:34 |
| mriedem | the goal for mitaka is really just to have the compute handling things | 15:34 |
| mriedem | which means updating detach in the compute manager and volume_api | 15:34 |
| ildikov | but it's really just an internal format | 15:35 |
| mriedem | so the compute rpcapi is updated | 15:35 |
| mriedem | which means updating the compute api to pass the attachment_id | 15:35 |
| ildikov | and the translation function has to be changed anyway | 15:35 |
| mriedem | but in reality i don't think the compute api needs to be passing the attachment_id in mitaka | 15:35 |
| *** bdrich has joined #openstack-nova | 15:35 | |
| mriedem | it doesn't hurt....but it won't be used | 15:36 |
| ildikov | but in a mixed env the mitaka compute will not pass the attachment_id and detach will not work | 15:36 |
| *** mdrabe has quit IRC | 15:37 | |
| mriedem | ildikov: i agree the compute manager changes are necessary here | 15:37 |
| mriedem | and the compute rpcapi changes | 15:37 |
| mriedem | but the only compute API that would be sending the attachment_id is really from newton right? | 15:37 |
| *** delattec has joined #openstack-nova | 15:37 | |
| mriedem | newton api talking to mitaka compute | 15:37 |
| *** tdurakov has quit IRC | 15:37 | |
| *** ssurana has joined #openstack-nova | 15:37 | |
| ildikov | why would that only be Newton? | 15:39 |
| *** frickler_ is now known as frickler | 15:39 | |
| *** cdelatte has quit IRC | 15:39 | |
| ildikov | johnthetubaguy: as the attachments field is already a list can we actually use it here? | 15:40 |
| *** diogogmt has quit IRC | 15:40 | |
| *** pushkaru has joined #openstack-nova | 15:40 | |
| mriedem | ildikov: i thought we weren't enabling multiattac in the compute api in mitaka? | 15:40 |
| mriedem | *multiattach | 15:40 |
| ildikov | the attach part | 15:41 |
| *** thangp has joined #openstack-nova | 15:41 | |
| mriedem | in nova.virt.block_device | 15:41 |
| mriedem | which is actually compute | 15:41 |
| ildikov | and prepare Mitaka Nova for detach and saving round trips with figuring out the attachment_id, where we have the volume info | 15:41 |
| mriedem | per the ML, i thought we were leaning toward requiring a microversion in newton to support multiattach | 15:41 |
| ildikov | mriedem: there's nothing in this patch that would require a microversion | 15:42 |
| mriedem | ildikov: but you shouldn't hit https://review.openstack.org/#/c/193134/79/nova/compute/api.py in mitaka | 15:42 |
| ildikov | even if I return all the attachments, which in normal cases will be one in the attachments list on the rest api | 15:42 |
| ildikov | mriedem: but that's the detach flow | 15:43 |
| *** tdurakov has joined #openstack-nova | 15:43 | |
| mriedem | technically we don't ever need a microversion for this to functionally work, but there was talk in the ML about adding a microversion to advertise the capability and the change in the rest api response | 15:43 |
| mriedem | when you do have multiple attachments | 15:43 |
| *** shyama has quit IRC | 15:44 | |
| ildikov | but this patch contains only the detach flow | 15:45 |
| *** devananda has quit IRC | 15:45 | |
| mriedem | yeah i get that | 15:45 |
| mriedem | and i get that we need the changes for detach on the compute service | 15:45 |
| ildikov | so the only case when you have multiple attachments if you attach a volume to an instance in Nova and then you go to Cinder and attach it outside of Nova | 15:45 |
| mriedem | but i'm not seeing the case that the changes to the compute API are needed | 15:45 |
| ildikov | which sounds weird though, but I think it's possible | 15:46 |
| mriedem | so, | 15:46 |
| mriedem | i think we should table the multiattach stuff until next week when the nova team can go over the details and plan at the midcycle, and then grant a 1 week extension/exception for the bp after we figure out the scenarios we need to support for mitaka | 15:46 |
| *** markvoelker_ has joined #openstack-nova | 15:46 | |
| ildikov | the compute api changes are for saving the round trip to Cinder and provide all the info to detach to retrieve the attachment_id, if it's not passed | 15:46 |
| mriedem | which i think is just the compute node changes | 15:47 |
| mriedem | ildikov: yeah, but we don't need those in mitaka | 15:47 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: WIP: bug process doc https://review.openstack.org/266453 | 15:47 |
| mriedem | we need those in newton | 15:47 |
| mriedem | i'd really like to get myself, ndipanov, johnthetubaguy, sdague, alaski, and dansmith in the same room to hash this out | 15:47 |
| mriedem | rather than rush this in b/c of the 1/21 deadline | 15:48 |
| ildikov | we need it in Mitaka for the same reason as in Newton | 15:48 |
| alaski | mriedem: +1 | 15:48 |
| ildikov | they don't do any harm as the rpc api is implemented that way | 15:48 |
| johnthetubaguy | mriedem: +1 | 15:48 |
| *** ivase has quit IRC | 15:48 | |
| openstackgerrit | Hemanth Makkapati proposed openstack/nova: config options: centralize s3 opts https://review.openstack.org/270263 | 15:48 |
| alaski | we should talk this out next week and get everyone on the same page | 15:48 |
| mriedem | ildikov: in newton, the compute api gets the attachment_id (your change here) and then passes that to the compute rpcapi via version 4.7 | 15:48 |
| johnthetubaguy | I am happy giving this an FFE so we can get folks in the same room | 15:48 |
| ildikov | sorry guys, we are talking about it so long and that patch is quite neat | 15:49 |
| *** ivase has joined #openstack-nova | 15:49 | |
| ndipanov | are we talking about just that one patch | 15:49 |
| mriedem | yeah i know, i'm torn between wanting to just stop talking about this forever and merge it | 15:49 |
| ndipanov | or is there more stuff that should land for m | 15:49 |
| ndipanov | ? | 15:49 |
| *** ivase has quit IRC | 15:49 | |
| mriedem | ndipanov: there are at least 2 more | 15:49 |
| *** ivase has joined #openstack-nova | 15:50 | |
| ildikov | my nerves are out of my head, I haven't slept for almost weeks now and that patch is really over tested and super safe | 15:50 |
| mriedem | ndipanov: i think we have to get to https://review.openstack.org/#/c/193135/ in mitaka | 15:50 |
| mriedem | to enable multiattach on the compute in mitaka | 15:50 |
| *** devananda has joined #openstack-nova | 15:50 | |
| sdague | mriedem: kilo is also busted | 15:50 |
| *** FL1SK has joined #openstack-nova | 15:50 | |
| *** Sukhdev has joined #openstack-nova | 15:50 | |
| sdague | on translations | 15:50 |
| ildikov | mriedem: we will not enable that in Mitaka | 15:50 |
| sdague | yay for no integration tests on i18n ... | 15:51 |
| ildikov | mriedem: or that was my understanding | 15:51 |
| ndipanov | ildikov, so which patches do we want in? | 15:51 |
| ildikov | mriedem: otherwise you need a config option in the computes to tell when to enable the second attachment | 15:51 |
| mriedem | ildikov: i thought we needed libvirt on the compute in mitaka to be able to multiattach for when the newton API sends a request there? | 15:51 |
| ildikov | mriedem: so I thought that we make detach safe and working and have all the other changes in Newton along with the microversion | 15:52 |
| mriedem | it would probably help if we just documented some use cases | 15:52 |
| ildikov | mriedem: how to disable multiattach on the compute for Mitaka but enable it, when we have Newton and a mixed env? | 15:52 |
| *** ivase has quit IRC | 15:53 | |
| *** matrohon has quit IRC | 15:53 | |
| *** vivekd has quit IRC | 15:53 | |
| *** ivase has joined #openstack-nova | 15:53 | |
| ildikov | mriedem: we said we don't enable multiattach now because of upgrade path issues and the BDM issues, which Dan fixed | 15:53 |
| mriedem | ildikov: in the API | 15:53 |
| ildikov | mriedem: there's no in the API | 15:54 |
| ildikov | mriedem: everything happens on the compute | 15:54 |
| mriedem | i guess 'enable' in this case you mean the support in the libvirt driver | 15:54 |
| ildikov | and the compute manager to allow the second attachment | 15:54 |
| ildikov | or no, wait not that one | 15:54 |
| mriedem | which is this https://review.openstack.org/#/c/267587/11/nova/volume/cinder.py | 15:55 |
| *** SridharG has quit IRC | 15:55 | |
| tonyb | sdague: do you have a kilo review to fix/verify the translations? | 15:55 |
| ildikov | so this one: https://review.openstack.org/#/c/267587/11/nova/compute/manager.py | 15:55 |
| *** armax has left #openstack-nova | 15:55 | |
| AJaeger | mriedem, sdague: What do you think of this for kilo: https://review.openstack.org/270268 ? | 15:55 |
| AJaeger | tonyb: ^ | 15:55 |
| ildikov | mriedem: and here https://review.openstack.org/#/c/193135/89/nova/virt/block_device.py | 15:56 |
| AJaeger | sdague, mriedem : I can add the tox.ini change if you prefer | 15:56 |
| ildikov | mriedem: plus the shareable flag for the disk in libvirt | 15:56 |
| mriedem | ildikov: right, but https://review.openstack.org/#/c/267587/11/nova/volume/cinder.py is really what enables it in the API | 15:56 |
| mriedem | b/c the compute API calls check_attach | 15:56 |
| mriedem | before casting off to the compute node | 15:56 |
| ildikov | a-ha, ok I see the check now | 15:56 |
| mriedem | ildikov: and what i was saying in the ML is rather than just have https://review.openstack.org/#/c/267587/11/nova/volume/cinder.py - we actually do that with a microversion in the REST API | 15:57 |
| *** mdrabe has joined #openstack-nova | 15:57 | |
| mriedem | so if you're requesting a 2nd attach on a multiattach volume, and are below the microversion, we fail | 15:57 |
| ildikov | mriedem: is that check_attach called in every case from the API? | 15:57 |
| tonyb | AJaeger: We're not impiorting transaltion fixes in kilo anymore? | 15:57 |
| AJaeger | tonyb: correct | 15:58 |
| AJaeger | liberty is the first release where we branched and imported to a stable branch | 15:58 |
| mriedem | ildikov: yes https://github.com/openstack/nova/blob/4f75ba6729366f87624554e7732ef122c6ebac4b/nova/compute/api.py#L2987 | 15:58 |
| ildikov | mriedem: I mean every possible case when a volume is attached? | 15:58 |
| AJaeger | Before that we only imported for master - and that's the default for most projects | 15:58 |
| AJaeger | tonyb: only a few have liberty branches | 15:58 |
| tonyb | AJaeger: Huh well there you go :) | 15:58 |
| mriedem | ildikov: yes there are 4 calls to check_attach here: https://github.com/openstack/nova/blob/4f75ba6729366f87624554e7732ef122c6ebac4b/nova/compute/api.py | 15:58 |
| ildikov | mriedem: I think I got confused because of the do_check_attach flag in the compute manager | 16:00 |
| mriedem | ildikov: yeah, i don't think that's actually used.... | 16:00 |
| mriedem | https://github.com/openstack/nova/blob/4f75ba6729366f87624554e7732ef122c6ebac4b/nova/compute/manager.py#L4678 | 16:00 |
| mriedem | https://github.com/openstack/nova/blob/4f75ba6729366f87624554e7732ef122c6ebac4b/nova/compute/manager.py#L4327 | 16:00 |
| *** haomaiwa_ has quit IRC | 16:01 | |
| *** bklei has quit IRC | 16:01 | |
| mriedem | b/c it's done in the compute API already | 16:01 |
| *** haomaiwa_ has joined #openstack-nova | 16:01 | |
| *** mgoddard__ has quit IRC | 16:02 | |
| *** ivase has quit IRC | 16:02 | |
| tonyb | AJaeger: that review looks okay to me. Just trying to understand the background. | 16:02 |
| *** mgoddard has joined #openstack-nova | 16:02 | |
| *** ivase has joined #openstack-nova | 16:02 | |
| mriedem | what is our mixed compute use case here? create a multiattach volume in mitaka, attach to an instance in mitaka, and then attach to another instance in newton? then detach from the mitaka instance? | 16:02 |
| mriedem | ildikov: ^ | 16:03 |
| ildikov | mriedem: so that flag always makes think that the check_attach is not called when it should have been called... | 16:04 |
| ildikov | mriedem: as the patches didn't progress that well, I shot for the bare minimum for Mitaka which is making detach safe | 16:05 |
| *** mriedem has quit IRC | 16:05 | |
| ildikov | mriedem: so having a Mitaka compute in a Newton env even if it's not allowed to do the second attach it should be able to detach a multiattached volume | 16:05 |
| sdague | AJaeger: the pot files could be fixed manually for kilo, right? | 16:06 |
| *** jaypipes has quit IRC | 16:06 | |
| ildikov | ndipanov: the first patch has to make it for sure | 16:06 |
| ildikov | ndipanov: but if we can agree that relying on check_attach is safe we have more to get in | 16:07 |
| *** jaosorior has joined #openstack-nova | 16:08 | |
| *** mriedem has joined #openstack-nova | 16:08 | |
| mriedem | ildikov: yeah, i know, and i'm not totally opposed to landing https://review.openstack.org/#/c/193134/ with the unusable compute api changes | 16:08 |
| mriedem | since we don't have the actual attach support in mitaka | 16:09 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: WIP - image_meta: Add hw_rescue_device and hw_rescue_bus https://review.openstack.org/270285 | 16:09 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: WIP - libvirt: Add boot ordering to individual devices https://review.openstack.org/270286 | 16:09 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: WIP - virt: Provide the bdm of an instance during rescue https://review.openstack.org/270287 | 16:09 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: WIP - compute: Allow for the rescue of volume backed instances https://review.openstack.org/270288 | 16:09 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: WIP - libvirt: stable device layout when rescuing instances https://review.openstack.org/270289 | 16:09 |
| ndipanov | ildikov, safe in what sense? | 16:09 |
| mriedem | ndipanov: in that we don't land the actual changes to allow attaching a volume to a 2nd instance in mitaka | 16:10 |
| ildikov | mriedem: if we can rely on check_attach, then I can move those cahnges to the REST API patch and we can get more patches in with an exception | 16:10 |
| *** ssurana has quit IRC | 16:10 | |
| ildikov | mriedem: would be better from the point of view of a homogen environment, I mean don't rely on whether we have Mitaka computes in the system or not | 16:11 |
| mriedem | sure, an easy button would be nice | 16:11 |
| *** jaypipes has joined #openstack-nova | 16:11 | |
| mriedem | but nova supports rolling upgrades | 16:11 |
| openstackgerrit | Cale Rath proposed openstack/nova: Purge soft-deleted instances cmd https://review.openstack.org/203751 | 16:11 |
| *** jaosorior has quit IRC | 16:11 | |
| ildikov | mriedem: if check_attach is surely called from the API, then we're good | 16:11 |
| ildikov | mriedem: as the API has to be upgraded first | 16:12 |
| *** cdent has quit IRC | 16:12 | |
| ndipanov | ildikov, that is no longer the case I'm afraid iiu dansmith correctly | 16:12 |
| ildikov | mriedem: so I mean that we can prepare the computes now for attach too and don't make the changes in check_attach so it's not "enabled" | 16:12 |
| ndipanov | hmm or actually maybe it is | 16:12 |
| ndipanov | it's quite confusing | 16:13 |
| ildikov | does sound odd to have the other way in place | 16:14 |
| mriedem | well i kind of thought that was the plan, so you can enable multiattach in the newton api, and it can attach to mitaka and newton computes | 16:14 |
| ndipanov | I think the idea is that only conductors have to go ahead first, but everything else can be upgraded on their own | 16:14 |
| mriedem | so i thought we had to get all of that infrastructure in place in the compute in mitaka | 16:15 |
| ildikov | mriedem: I gave up that plan a few days ago as I did not see a way it's going to make it before the deadline | 16:15 |
| ildikov | mriedem: I still support this way, if we can agree on discussing it on the mid-cycle and have FFE for the attach prepare patches I'm totally fine with that | 16:16 |
| mriedem | i kind of think we have to discuss this in person at the midcycle and then figure out what our use cases are and what we're going to support, then get that in the week after the midcycle (crap, which i'll be on vacation that week :( ) | 16:17 |
| *** ivase has quit IRC | 16:17 | |
| ildikov | mriedem: according to the current plans I will be there only via Hangouts | 16:18 |
| mriedem | yeah, i know | 16:18 |
| mriedem | i want to talk with the nova brain trust in person | 16:18 |
| mriedem | (tm) | 16:18 |
| *** edtubill has joined #openstack-nova | 16:18 | |
| *** ivase_ has joined #openstack-nova | 16:19 | |
| ildikov | well, it would be useful for me too to lurk on that chat | 16:19 |
| kashyap | I'm trying to run upstream-Kilo unit tests on Fedora-23, and notice this: | 16:19 |
| ildikov | or do some meeting minutes for me :) | 16:19 |
| kashyap | from nova.api.ec2 import apirequest | 16:19 |
| kashyap | ImportError: cannot import name apirequest | 16:20 |
| kashyap | Any hints as to how can I fix that? | 16:20 |
| mriedem | ildikov: sure | 16:20 |
| kashyap | (My test invocation is: $ .venv/bin/python -m testtools.run nova.tests.unit.virt.libvirt.test_driver.LibvirtDriverTestCase.test_public_api_signatures) | 16:21 |
| ndipanov | ildikov, we'll call you for sure - it's the same tz at least this time :) | 16:22 |
| ildikov | yeah, true :) | 16:22 |
| openstackgerrit | Tracy Jones proposed openstack/nova: remove mox from vmwareapi/test_driver_api.py https://review.openstack.org/270303 | 16:23 |
| *** Marga_ has joined #openstack-nova | 16:24 | |
| ildikov | mriedem: ndipanov: I might give it a try with the managers, but I would not count on that much... | 16:24 |
| *** Jeffrey4l has joined #openstack-nova | 16:24 | |
| ildikov | johnthetubaguy: mriedem: ndipanov: what is the strategy for now? | 16:24 |
| *** rcernin has quit IRC | 16:25 | |
| ildikov | I mean wait with all the patches? | 16:25 |
| *** Jeffrey4l has quit IRC | 16:25 | |
| *** hemna_ is now known as hemna | 16:25 | |
| *** Jeffrey4l has joined #openstack-nova | 16:25 | |
| ildikov | or if I use the first item of that dict and fix the test issues we're good with that and have an excpetion for the others? | 16:26 |
| hemna | mornin | 16:26 |
| ndipanov | ildikov, tbh I don't care about that dict too much really ... | 16:26 |
| ndipanov | I'd be happy with a comment | 16:26 |
| ildikov | ndipanov: I added a comment | 16:27 |
| mriedem | i'd like to not rely on an OrderedDict, i have other comments in https://review.openstack.org/#/c/193134/ | 16:27 |
| mriedem | nothing major | 16:27 |
| mriedem | the only open questions i have are if we should put the compute api changes in mitaka | 16:27 |
| mriedem | when i don't think they will be used there | 16:27 |
| ildikov | mriedem: they will save one round trip still | 16:28 |
| mriedem | these changes i mean https://review.openstack.org/#/c/193134/79/nova/compute/api.py | 16:28 |
| ildikov | the instance_uuid is needed in compute anyway | 16:28 |
| ndipanov | mriedem, imho it doesn't really matter - for it to actually work across versions all of the logic needs to be there in mitaka right? | 16:28 |
| *** krtaylor has quit IRC | 16:29 | |
| ndipanov | since we're not going to backport any of that realistically | 16:29 |
| *** markstur has joined #openstack-nova | 16:29 | |
| ildikov | if we want to send attachment_id to Cinder it's in the volume info | 16:29 |
| ildikov | which we need to ask Cinder for | 16:29 |
| *** ivase_ has quit IRC | 16:29 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Move wsgi options into a group https://review.openstack.org/263812 | 16:29 |
| *** ivase has joined #openstack-nova | 16:30 | |
| ildikov | that change retrieves it from the volume info as if the detach is request through the REST API we ask for the volume info from Cinder | 16:30 |
| *** salv-orlando has joined #openstack-nova | 16:30 | |
| ildikov | so in that case we can save having another round trip from the compute, when we have all the info in the API | 16:30 |
| ildikov | so for detach we need it for Mitaka just as much as for Newton | 16:30 |
| *** jdurgin1 has joined #openstack-nova | 16:30 | |
| *** nelsnelson has joined #openstack-nova | 16:32 | |
| ildikov | mriedem: ^ | 16:33 |
| *** otter768 has joined #openstack-nova | 16:33 | |
| ildikov | hemna: afternoon :) | 16:33 |
| hemna | ildikov, :) | 16:33 |
| openstackgerrit | David Edery proposed openstack/nova: Fix create snapshot failure on VMs with SRIOV https://review.openstack.org/262341 | 16:34 |
| *** cdent has joined #openstack-nova | 16:34 | |
| tonyb | AJaeger: why has this translations thing become a problem now? | 16:34 |
| *** davidhadas has quit IRC | 16:35 | |
| tonyb | sdague: I assume we need the kilo translations fox because of grenade running on liberty? | 16:35 |
| *** nkrinner has quit IRC | 16:36 | |
| *** otter768 has quit IRC | 16:37 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: tests_fields: Remove most enum-type field tests https://review.openstack.org/244370 | 16:38 |
| *** tdurakov has quit IRC | 16:38 | |
| *** dedery_ has joined #openstack-nova | 16:39 | |
| *** fawadkhaliq has joined #openstack-nova | 16:39 | |
| *** ihrachys_ has quit IRC | 16:40 | |
| *** davidhadas_ has joined #openstack-nova | 16:40 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: test_fields: Remove all 'Enum' subclass tests https://review.openstack.org/244372 | 16:40 |
| *** krtaylor has joined #openstack-nova | 16:40 | |
| *** unicell1 has quit IRC | 16:41 | |
| *** paul-carlton2 has joined #openstack-nova | 16:41 | |
| *** dedery has quit IRC | 16:42 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: releasenotes: Note on CPU thread pinning support https://review.openstack.org/264693 | 16:42 |
| *** gcb has quit IRC | 16:43 | |
| *** su_zhang has joined #openstack-nova | 16:43 | |
| *** dedery_ has quit IRC | 16:43 | |
| *** rmart04 has quit IRC | 16:44 | |
| alaski | bauzas: Can you look at https://review.openstack.org/#/c/259470/3 and comment on if that code can be removed now, or when it can if not now? | 16:44 |
| *** gszasz has quit IRC | 16:45 | |
| *** gcb has joined #openstack-nova | 16:45 | |
| *** unicell has joined #openstack-nova | 16:46 | |
| *** mjura_ has quit IRC | 16:46 | |
| *** annegentle has quit IRC | 16:46 | |
| *** Sukhdev has quit IRC | 16:47 | |
| *** annegentle has joined #openstack-nova | 16:47 | |
| *** tdurakov has joined #openstack-nova | 16:47 | |
| *** fawadkhaliq has quit IRC | 16:48 | |
| *** speller has quit IRC | 16:48 | |
| *** fawadkhaliq has joined #openstack-nova | 16:48 | |
| *** irenab has quit IRC | 16:48 | |
| *** tdurakov has quit IRC | 16:49 | |
| *** fawadkhaliq has quit IRC | 16:50 | |
| *** efoley has quit IRC | 16:51 | |
| *** apoorvad has joined #openstack-nova | 16:53 | |
| *** andymaier has quit IRC | 16:53 | |
| *** zakora has quit IRC | 16:54 | |
| *** andymaier has joined #openstack-nova | 16:54 | |
| *** nic has joined #openstack-nova | 16:54 | |
| *** melwitt_ has joined #openstack-nova | 16:54 | |
| *** irenab has joined #openstack-nova | 16:55 | |
| *** irenab has quit IRC | 16:56 | |
| mriedem | ndipanov: "imho it doesn't really matter - for it to actually work across versions all of the logic needs to be there in mitaka right?" - i'm still not sure why https://review.openstack.org/#/c/193134/79/nova/compute/api.py would be needed in mitaka | 16:57 |
| mriedem | ndipanov: assuming you are upgrading API to get the microversion to support multiattach | 16:57 |
| mriedem | in newton | 16:57 |
| mriedem | it'd be the newton api talking to the mitaka compute | 16:57 |
| mriedem | johnthetubaguy: ^ am i missing something there? | 16:58 |
| *** irenab has joined #openstack-nova | 16:58 | |
| mriedem | yikes cells v2 meeting in 2 minutes | 16:58 |
| *** melwitt_ has quit IRC | 16:59 | |
| *** belmoreira has joined #openstack-nova | 16:59 | |
| ndipanov | mriedem, my point is that even if you make the call to cinder work in mitaka on compute nodes, that's still not the whole logic that will eventually be there (I think) | 17:00 |
| *** tdurakov has joined #openstack-nova | 17:00 | |
| ndipanov | mriedem, and without it it won't be safe to just assume it will work on older nodes... especially the detach part (again - I think!) | 17:00 |
| *** irenab has quit IRC | 17:00 | |
| *** delattec has quit IRC | 17:00 | |
| *** browne has joined #openstack-nova | 17:01 | |
| *** haomaiwa_ has quit IRC | 17:01 | |
| *** irenab has joined #openstack-nova | 17:01 | |
| johnthetubaguy | mriedem: sorry, just looking | 17:01 |
| *** achanda has joined #openstack-nova | 17:01 | |
| *** melwitt_ has joined #openstack-nova | 17:01 | |
| *** armax has joined #openstack-nova | 17:01 | |
| mriedem | ndipanov: i agree we want this in mitaka https://review.openstack.org/#/c/193134/79/nova/compute/manager.py | 17:02 |
| mriedem | i could just be overthinking this, which is why i wanted to discuss in person | 17:02 |
| *** delattec has joined #openstack-nova | 17:03 | |
| *** mgoddard_ has joined #openstack-nova | 17:03 | |
| *** mgoddard has quit IRC | 17:03 | |
| johnthetubaguy | mriedem: I think we need to pass the instance_uuid, so we can be sure we are "safe" once we get multiples about the place | 17:04 |
| *** yassine__ has quit IRC | 17:04 | |
| *** jistr has quit IRC | 17:04 | |
| johnthetubaguy | mriedem: but I might be overthinking it at this point too | 17:04 |
| *** ssurana has joined #openstack-nova | 17:05 | |
| mriedem | johnthetubaguy: sure, but that's passing from newton api right? | 17:05 |
| *** READ10 has joined #openstack-nova | 17:05 | |
| *** haomaiwa_ has joined #openstack-nova | 17:05 | |
| mriedem | or can we not rely on that b/c of load balancers? | 17:06 |
| johnthetubaguy | mriedem: so we need N and N+1 APIs at the same time now, now we have service version to stop the data migrations | 17:06 |
| ildikov | mriedem: so when we release Mitaka the computes start passing attachment_id | 17:06 |
| johnthetubaguy | yeah, I am going around in circles here | 17:07 |
| * mriedem misses dansmith | 17:07 | |
| ildikov | mriedem: so without that changes in the compute API we don't have neither attachment_id from the volume info nor the instance_uuid passed without we can't retrieve at all even with calling out to Cinder | 17:07 |
| *** ociuhandu has quit IRC | 17:07 | |
| johnthetubaguy | so this is mitkata computes with newton computes | 17:07 |
| openstackgerrit | Samuel Matzek proposed openstack/nova: glance: secure https when getting client version https://review.openstack.org/254944 | 17:07 |
| johnthetubaguy | the newton API gets a detach | 17:07 |
| johnthetubaguy | it goes to an old compute | 17:08 |
| johnthetubaguy | but the new compute has already done multi-attach | 17:08 |
| ildikov | johnthetubaguy: mriedem: the compute api change is even with only Mitaka computes | 17:08 |
| johnthetubaguy | and then the old compute gets confused about which volume its detaching | 17:08 |
| johnthetubaguy | oh, wait, thats non-sense | 17:08 |
| *** irenab has quit IRC | 17:08 | |
| mriedem | johnthetubaguy: no because the newton api is passing the attachment id | 17:08 |
| mriedem | to the mitaka compute | 17:08 |
| ildikov | or if it's not tha Mitaka compute tries to get it | 17:09 |
| johnthetubaguy | mriedem: but mitaka compute is on the old interface, which doesn't take the id? | 17:09 |
| *** jlanoux has quit IRC | 17:09 | |
| mriedem | johnthetubaguy: no i think we land the compute rpcapi and manager changes for that in mitaka | 17:09 |
| mriedem | and use them in newton | 17:09 |
| johnthetubaguy | mriedem: oh, I see what you mean now | 17:09 |
| mriedem | at least i thought that was the plan | 17:09 |
| johnthetubaguy | mriedem: but we may was well use the new bits now, to save any confusion? | 17:09 |
| mriedem | nova compute api passing the attachment id explicitly in mitaka today is really just doing cinder's job for it | 17:10 |
| mriedem | cinder will already find the attachment id if there is only one today | 17:10 |
| ildikov | johnthetubaguy: if we send attachment_id in Mitaka then we would want to save round trip already in Mitaka | 17:10 |
| mriedem | which would be the case b/c we don't support multiattach in mitaka | 17:10 |
| *** yamahata has quit IRC | 17:10 | |
| mriedem | ok, so i think i'm getting ildikov's point, | 17:10 |
| johnthetubaguy | mriedem: but we do with mixed, so we would have to double check | 17:10 |
| mriedem | we need the compute manager changes, which means we need the volume_api changes, which calls to cinder if attachment id isn't provided | 17:10 |
| ildikov | yeah | 17:11 |
| johnthetubaguy | ah, yeah, thats more logical that my nonsense | 17:11 |
| mriedem | so if we don't land the compute api changes in mitaka, even though not really necessary, we'd be adding a 2nd call to cinder api in mitaka that is unnecessary | 17:11 |
| mriedem | not related to mixed env | 17:11 |
| mriedem | not related to multiattach | 17:11 |
| ildikov | yeap, exactly | 17:11 |
| mriedem | ildikov: why didn't you say so?! :P | 17:12 |
| ildikov | this is what I tried to decribe, but my native English still needs improvements :) | 17:12 |
| *** melwitt__ has joined #openstack-nova | 17:12 | |
| ildikov | mriedem: :P | 17:12 |
| johnthetubaguy | well, the original call is needed because of mixed | 17:12 |
| johnthetubaguy | but lets not confuse things | 17:12 |
| mriedem | yeah | 17:12 |
| mriedem | ok, so i think i'm on the same page as ildikov now | 17:13 |
| ildikov | mriedem: \o/ :) | 17:13 |
| mriedem | ildikov: so address the comments in https://review.openstack.org/#/c/193134/ and then we'll go from there | 17:13 |
| *** e0ne has quit IRC | 17:13 | |
| ildikov | mriedem: from what you wrote I can confirm | 17:13 |
| ildikov | mriedem: tests are running now | 17:13 |
| mriedem | b/c i'm still not sure what our support case is for mixed comptues in newton | 17:13 |
| *** scheuran has quit IRC | 17:14 | |
| ildikov | mriedem: I think assert will need None as it is different if something is not provided or None is provided and attachment_id is optional in the whole chain of calls | 17:14 |
| ildikov | mriedem: for the first patch it's kinda doesn't matter it's needed in all cases if we want multiattach in Newton | 17:15 |
| *** melwitt_ has quit IRC | 17:15 | |
| *** zenoway has quit IRC | 17:15 | |
| *** zenoway has joined #openstack-nova | 17:16 | |
| mriedem | ildikov: for the mox calls, yeah, i figured, because mox is picky | 17:16 |
| ildikov | for the rest, as I said I got confused with check_attach, but if that can be used to "disabke" multiattach we can add the required changes and enable multiattach along with the microversion | 17:16 |
| *** dims has quit IRC | 17:16 | |
| *** zenoway has quit IRC | 17:16 | |
| ndipanov | I still don't think that unless we have all the code in mitaka compute nodes - meaning the code that checks if it's safe to disconnect volume from the compute node - it's still not going to be safe to call it in a mitaka compute node from any API | 17:16 |
| *** zenoway has joined #openstack-nova | 17:16 | |
| ndipanov | service version | 17:16 |
| ildikov | ndipanov: you mean the attach part, right? | 17:17 |
| ndipanov | detach as well | 17:17 |
| *** zenoway has quit IRC | 17:18 | |
| ndipanov | because it's not only about how we call out to cinder | 17:18 |
| ndipanov | it's also about what we do on the compute host with the volume drivers no? | 17:19 |
| mriedem | ndipanov: yeah so this https://review.openstack.org/#/c/193135/ | 17:19 |
| mriedem | that's why i thought we needed all of the compute changes in mitaka | 17:19 |
| mriedem | and then we'd allow check_attach to support multiattach in newton with a microversion | 17:20 |
| ndipanov | mriedem, yeah I think taht if we want to support this working in a mix cloud - there's really no other option | 17:20 |
| *** ahelkhou1 has joined #openstack-nova | 17:20 | |
| mriedem | ok, yar har | 17:21 |
| *** ahelkhou2 has joined #openstack-nova | 17:21 | |
| mriedem | "working" | 17:21 |
| ildikov | other option than land these patches and enable in Newton? | 17:21 |
| *** rmart04 has joined #openstack-nova | 17:21 | |
| mriedem | this will never be working for rax | 17:22 |
| mriedem | because xen | 17:22 |
| mriedem | so rax will use a policy rule to disable it | 17:22 |
| ildikov | detach should not do anything with volume drivers | 17:22 |
| ndipanov | ildikov, oh? | 17:22 |
| ildikov | mriedem: as the capability is added to the virt drivers from Nova part it will fail as planned | 17:23 |
| ildikov | ndipanov: it passes an id to Cinder and it does with it whatever it wants, hopefully using it for identifying the right attachment :) | 17:24 |
| *** melwitt__ has quit IRC | 17:24 | |
| ndipanov | ildikov, so what I am thinking about is that we currently disconnect the volume on any detach | 17:24 |
| ndipanov | and this is not what we want to do once we have multi-attach | 17:24 |
| *** ahelkhou1 has quit IRC | 17:25 | |
| *** tonytan4ever has quit IRC | 17:25 | |
| ildikov | ndipanov: it's on Cinder also | 17:25 |
| ildikov | ndipanov: this is why we have the follow up patch with passing the host_name as well | 17:25 |
| ildikov | ndipanov: the lvm driver has issues with two attahcments on one host | 17:25 |
| *** Marga_ has quit IRC | 17:25 | |
| ildikov | ndipanov: 3PAR is fine from that PoV | 17:25 |
| *** Marga_ has joined #openstack-nova | 17:26 | |
| ildikov | ndipanov: passing the host_name is problematic too though, we need to figure out smth for live migration | 17:26 |
| mriedem | ndipanov: yeah i think https://review.openstack.org/#/c/256273/ is for that | 17:26 |
| *** dims has joined #openstack-nova | 17:26 | |
| *** kfarr has joined #openstack-nova | 17:26 | |
| ildikov | if I know correctly jgriffith has ideas on what to do with this issue | 17:26 |
| mriedem | hrm | 17:27 |
| ndipanov | hmmm | 17:27 |
| ildikov | I don't know about the progress atm | 17:27 |
| smcginnis | Some cinder drivers care, other's don't. | 17:27 |
| mriedem | if you can't live migrate with a multiattach volume, then we should be blocking that in the code | 17:27 |
| *** rmart04 has quit IRC | 17:27 | |
| smcginnis | It will be up to us on the cinder side to make sure we detach the right things. | 17:27 |
| *** edtubill has quit IRC | 17:27 | |
| ildikov | smcginnis: yeah, with hemna we thought passing the host_name to Cinder will solve this | 17:28 |
| *** markus_z has quit IRC | 17:28 | |
| ildikov | smcginnis: but Nova does not call attach again in case of live migration | 17:28 |
| smcginnis | ildikov: Yeah, I believe that should cover it. | 17:28 |
| ildikov | it calls only initiate_connection | 17:28 |
| *** unicell has quit IRC | 17:28 | |
| smcginnis | ildikov: He's definitely dug deeper than me, but from what I've looked in to, that should be enough for those drivers that need it. | 17:28 |
| ildikov | I hope I remember the name correctly | 17:29 |
| smcginnis | ildikov: You got it. ;) | 17:29 |
| ildikov | so we cannot pass the host name change | 17:29 |
| hemna | mriedem, at this point I don't think many folks have been able to test multiattachj | 17:29 |
| hemna | so, live migration may not work for anyone | 17:29 |
| hemna | I really don't know | 17:29 |
| mriedem | that inspires confidence :) | 17:29 |
| hemna | live migration has not gotten a lot of attention from Cinder folks | 17:29 |
| AJaeger | tonyb: somebody noticed today about the problem... | 17:29 |
| ildikov | smcginnis: cool, I'm learning then :) | 17:30 |
| hemna | a few of us have looked at it hard and tested it | 17:30 |
| *** knikolla has quit IRC | 17:31 | |
| AJaeger | sdague: fixing pot files manually for kilo is what I did in 270268 | 17:31 |
| hemna | but ildikov is right, having the host_name in there with the attachment will help cinder try and make a decision as to what to do with the exported target | 17:31 |
| tonyb | AJaeger: okay. sdague implied is was causing gate failures which confusd me. | 17:31 |
| openstackgerrit | Dimitry Ushakov proposed openstack/nova: (lxc) Updated regex to ignore failing tests https://review.openstack.org/267724 | 17:32 |
| AJaeger | tonyb: I'm not aware of gate failures. My proposed changes for liberty and master will cause gate failures whenever an invlaid translation file gets proposed by the infra scripts to avoid importing broken files | 17:32 |
| *** rmart04 has joined #openstack-nova | 17:32 | |
| ndipanov | ok so here's the problem I am thinking of and folks please correct me if I am wrong: unless there is specific code in Nova that checks that 2 instances that attach the same volume are on the same host - regardless of what cinder does, nova compute is going to plough through detach_volume code which ultimately disconnects the volume regardless of what cinder did | 17:33 |
| tonyb | AJaeger: okay thanks for clearing that up. | 17:33 |
| AJaeger | tonyb: you're welcome | 17:33 |
| *** daemontool has joined #openstack-nova | 17:34 | |
| hemna | ndipanov, I think you may be right. nova shouldn't call disconnect_volume if that same volume path is used by another instance. | 17:34 |
| *** diana_clarke has joined #openstack-nova | 17:34 | |
| hemna | meeting...... | 17:34 |
| *** hemna is now known as hemnafk | 17:34 | |
| ndipanov | attach is similar but slightly less bad - we don't want to connect it - I assume those checks would be part of the multi-attach work... | 17:35 |
| *** yamahata has joined #openstack-nova | 17:37 | |
| *** yamahata has quit IRC | 17:37 | |
| *** yamahata has joined #openstack-nova | 17:37 | |
| *** mragupat has quit IRC | 17:37 | |
| *** su_zhang has quit IRC | 17:38 | |
| *** Jeffrey4l has quit IRC | 17:40 | |
| jgriffith | hemnafk: ndipanov I'd still like to change that so they each have their own unique path | 17:40 |
| nic | mriedem: I'm of the belief that https://review.openstack.org/187395 and https://review.openstack.org/205282 are in a state that should have all of your concerns addressed | 17:40 |
| *** lpetrut has quit IRC | 17:41 | |
| *** ccarmack has left #openstack-nova | 17:41 | |
| ndipanov | jgriffith, sure what I'm saying is without having that code (however it's structured) in both m and n we can't transparently expect stuff to work once we enable it in the API in n | 17:43 |
| *** aghadge has joined #openstack-nova | 17:43 | |
| ndipanov | mriedem, ^ | 17:43 |
| jgriffith | ndipanov: understood | 17:43 |
| *** belmoreira has quit IRC | 17:43 | |
| *** aghadge has left #openstack-nova | 17:44 | |
| *** daemontool has quit IRC | 17:44 | |
| mriedem | ndipanov: yeah, can you note that in https://review.openstack.org/#/c/193135/ / | 17:45 |
| mriedem | nic: i plan on getting back on those after lunch | 17:45 |
| *** pratikma_ has quit IRC | 17:46 | |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: Allow monitor plugins to set own metric object https://review.openstack.org/270341 | 17:47 |
| *** slDabbler has joined #openstack-nova | 17:47 | |
| ildikov | ndipanov: you mean don't call out to os-brick, if we have two attachments on the same host? | 17:48 |
| ildikov | sorry, on a meeting in parallel | 17:48 |
| openstackgerrit | Gábor Antal proposed openstack/nova: Added super() call in some of the Model's child https://review.openstack.org/270343 | 17:48 |
| ndipanov | ildikov, yeah basically (may be more to it than that but that's the point) | 17:51 |
| ndipanov | nova code around detaching is not well structured | 17:51 |
| ndipanov | so there might be subtle details too | 17:52 |
| ildikov | ndipanov: ok, then it needs to be addressed now too | 17:52 |
| ildikov | yeah, I realized that :( | 17:52 |
| *** rotbeard has quit IRC | 17:53 | |
| *** slunkad_ has joined #openstack-nova | 17:54 | |
| ildikov | ndipanov: I don't know os-brick that much, could that be the component to decide, whether to disconnect or not or is it Nova, who should be responsible for that? | 17:55 |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: WIP: define NUMA memory bw monitor https://review.openstack.org/270344 | 17:56 |
| *** tdurakov has quit IRC | 17:57 | |
| *** knikolla has joined #openstack-nova | 17:57 | |
| *** mragupat has joined #openstack-nova | 17:58 | |
| ndipanov | ildikov, well the API there is mostly connect/disconnect so just from that POV it seems like it's up to Nova... | 17:58 |
| *** mragupat has quit IRC | 17:59 | |
| ildikov | ndipanov: hmm, not good news :( | 17:59 |
| *** danpb has quit IRC | 17:59 | |
| mriedem | i suppose the tempest test will hit this when i have it working | 18:00 |
| mriedem | since it's attaching 2 instances on the same host to the same volume, and then detaching them in order | 18:00 |
| ndipanov | mriedem, I would assume so | 18:00 |
| mriedem | which is why we like tempest testing for new complicated features :) | 18:00 |
| *** haomaiwa_ has quit IRC | 18:01 | |
| *** sahid has quit IRC | 18:01 | |
| *** sfinucan has quit IRC | 18:01 | |
| ildikov | mriedem: yeah, I owe you beers for that test :) | 18:01 |
| mriedem | once it's working... | 18:01 |
| *** haomaiwang has joined #openstack-nova | 18:01 | |
| *** tonytan4ever has joined #openstack-nova | 18:01 | |
| mriedem | gotta review nic's rbd stuff first | 18:02 |
| nic | Will it help the review if I tell you it's been running in production for 18 months with no issues? | 18:02 |
| mriedem | nope | 18:03 |
| *** crinkle_ is now known as crinkle | 18:04 | |
| *** cdent has quit IRC | 18:05 | |
| *** dane-fichter has quit IRC | 18:05 | |
| *** browne has quit IRC | 18:05 | |
| *** rmart04 has quit IRC | 18:07 | |
| *** unicell has joined #openstack-nova | 18:08 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: libvirt: Allow multiple volume attachments https://review.openstack.org/193135 | 18:09 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: Pass attachment_id to Cinder when detach a volume https://review.openstack.org/193134 | 18:09 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: comp: Allow volume multiattach https://review.openstack.org/267587 | 18:09 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach https://review.openstack.org/153038 | 18:09 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: Pass host when call attach to Cinder https://review.openstack.org/256273 | 18:09 |
| *** derekh has quit IRC | 18:10 | |
| ildikov | mriedem: ndipanov: new version of the detach patch is up ^ | 18:11 |
| ndipanov | wow that's fast | 18:11 |
| *** lifeless_ is now known as lifeless | 18:12 | |
| *** markd_ has quit IRC | 18:12 | |
| *** jwang__ has joined #openstack-nova | 18:12 | |
| *** ociuhandu has joined #openstack-nova | 18:12 | |
| *** mnestratov|2 has joined #openstack-nova | 18:13 | |
| ildikov | ndipanov: it's the fix for mriedem's comments, I haven't solved the disconnect_volume issue yet :) | 18:13 |
| ildikov | ndipanov: that will be another patch though anyway | 18:13 |
| *** aysyd_ has joined #openstack-nova | 18:14 | |
| *** leseb_ has joined #openstack-nova | 18:14 | |
| *** xek_ has joined #openstack-nova | 18:14 | |
| *** eglynn_ has joined #openstack-nova | 18:15 | |
| *** jschlueter has joined #openstack-nova | 18:15 | |
| *** burt1 has joined #openstack-nova | 18:15 | |
| *** _xin1 has joined #openstack-nova | 18:16 | |
| *** migi_ has joined #openstack-nova | 18:16 | |
| *** kfox1111 has joined #openstack-nova | 18:17 | |
| *** ahelkhou2 has left #openstack-nova | 18:17 | |
| *** smcginni1 has joined #openstack-nova | 18:17 | |
| *** kragniz_ has joined #openstack-nova | 18:17 | |
| *** hemnafk is now known as hemna | 18:18 | |
| *** e0ne has joined #openstack-nova | 18:18 | |
| *** ccarmack has joined #openstack-nova | 18:18 | |
| *** inara` has joined #openstack-nova | 18:18 | |
| hemna | jgriffith, I agree. I'm hoping you can change LVM to do that. But in the case of the other cinder backends that can't, we still probably need some checks in place on the nova side | 18:19 |
| *** wasmum- has joined #openstack-nova | 18:19 | |
| *** mriedem has quit IRC | 18:19 | |
| *** su_zhang has joined #openstack-nova | 18:20 | |
| *** tjones has joined #openstack-nova | 18:20 | |
| *** isq_ has joined #openstack-nova | 18:20 | |
| *** aignatov2 has joined #openstack-nova | 18:21 | |
| *** ajo_ has joined #openstack-nova | 18:21 | |
| openstackgerrit | Andreas Jaeger proposed openstack/python-novaclient: Remove argparse from requirements https://review.openstack.org/270389 | 18:21 |
| *** slunkad_ has quit IRC | 18:22 | |
| *** annegentle has quit IRC | 18:22 | |
| *** paul-carlton2 has quit IRC | 18:22 | |
| *** burt has quit IRC | 18:22 | |
| *** thorst has quit IRC | 18:22 | |
| *** aysyd has quit IRC | 18:22 | |
| *** busterswt has quit IRC | 18:22 | |
| *** lyarwood has quit IRC | 18:22 | |
| *** ababich has quit IRC | 18:22 | |
| *** _gryf has quit IRC | 18:22 | |
| *** migi has quit IRC | 18:22 | |
| *** leseb- has quit IRC | 18:22 | |
| *** kfox1111_ has quit IRC | 18:22 | |
| *** jwang_ has quit IRC | 18:22 | |
| *** eglynn has quit IRC | 18:22 | |
| *** claudiub has quit IRC | 18:22 | |
| *** ndipanov has quit IRC | 18:22 | |
| *** esracelik has quit IRC | 18:22 | |
| *** med_ has quit IRC | 18:22 | |
| *** arif-ali has quit IRC | 18:22 | |
| *** xek has quit IRC | 18:22 | |
| *** ajo has quit IRC | 18:22 | |
| *** mnestratov has quit IRC | 18:22 | |
| *** wasmum has quit IRC | 18:22 | |
| *** inara has quit IRC | 18:22 | |
| *** mrsoul` has quit IRC | 18:22 | |
| *** isq has quit IRC | 18:22 | |
| *** hamzy has quit IRC | 18:22 | |
| *** _xin has quit IRC | 18:22 | |
| *** aignatov has quit IRC | 18:22 | |
| *** kragniz has quit IRC | 18:22 | |
| *** smcginnis has quit IRC | 18:22 | |
| *** aysyd_ is now known as aysyd | 18:22 | |
| *** aignatov2 is now known as aignatov | 18:22 | |
| *** med_ has joined #openstack-nova | 18:22 | |
| *** davidhadas_ has quit IRC | 18:23 | |
| *** med_ is now known as Guest65103 | 18:23 | |
| *** arif-ali has joined #openstack-nova | 18:23 | |
| *** mriedem has joined #openstack-nova | 18:24 | |
| *** lyarwood has joined #openstack-nova | 18:24 | |
| *** ndipanov has joined #openstack-nova | 18:24 | |
| *** slunkad_ has joined #openstack-nova | 18:24 | |
| *** annegentle has joined #openstack-nova | 18:24 | |
| *** mrsoul` has joined #openstack-nova | 18:24 | |
| jgriffith | hemna: I looked at it again last night, not sure how easy it will be. Regardless, my concern is that it's getting kinda convoluted and brittle. But we'll see what we can do. Maybe talk through some things next week | 18:25 |
| hemna | jgriffith, no problemo. | 18:25 |
| *** delattec has quit IRC | 18:26 | |
| *** delattec has joined #openstack-nova | 18:27 | |
| *** penick has joined #openstack-nova | 18:27 | |
| *** ducttape_ has quit IRC | 18:27 | |
| *** _ducttape_ has joined #openstack-nova | 18:27 | |
| vladikr | jaypipes, could you please take a look the libosinfo series again, when you'll have a moment I've addressed your comments, thanks | 18:27 |
| *** ccarmack has quit IRC | 18:27 | |
| vladikr | https://review.openstack.org/#/q/topic:bp/libvirt-hardware-policy-from-libosinfo | 18:27 |
| *** paul-carlton2 has joined #openstack-nova | 18:27 | |
| *** chhavi has quit IRC | 18:28 | |
| ildikov | jgriffith: thanks for the effort on this | 18:28 |
| ildikov | jgriffith: also would be nice to touch every sensitive/fragile topic next week we can | 18:28 |
| *** _gryf has joined #openstack-nova | 18:28 | |
| *** thorst has joined #openstack-nova | 18:29 | |
| *** baoli_ has quit IRC | 18:30 | |
| *** ccarmack has joined #openstack-nova | 18:30 | |
| jaypipes | vladikr: yes, will do. | 18:30 |
| *** esracelik has joined #openstack-nova | 18:31 | |
| *** baoli has joined #openstack-nova | 18:31 | |
| vladikr | jaypipes, thanks! | 18:31 |
| *** ababich has joined #openstack-nova | 18:31 | |
| *** hamzy has joined #openstack-nova | 18:31 | |
| *** ctrath has quit IRC | 18:34 | |
| *** otter768 has joined #openstack-nova | 18:34 | |
| *** jdurgin1 has quit IRC | 18:34 | |
| *** cdent has joined #openstack-nova | 18:35 | |
| *** lyarwood has quit IRC | 18:35 | |
| *** ndipanov has quit IRC | 18:35 | |
| *** slunkad_ has quit IRC | 18:35 | |
| *** annegentle has quit IRC | 18:35 | |
| *** mrsoul` has quit IRC | 18:35 | |
| *** Sukhdev has joined #openstack-nova | 18:36 | |
| *** _ducttape_ has quit IRC | 18:36 | |
| *** baoli_ has joined #openstack-nova | 18:36 | |
| *** lyarwood has joined #openstack-nova | 18:37 | |
| *** ndipanov has joined #openstack-nova | 18:37 | |
| *** slunkad_ has joined #openstack-nova | 18:37 | |
| *** annegentle has joined #openstack-nova | 18:37 | |
| *** mrsoul` has joined #openstack-nova | 18:37 | |
| *** otter768 has quit IRC | 18:38 | |
| *** baoli has quit IRC | 18:39 | |
| jgriffith | ildikov: we only have a week :) | 18:40 |
| *** ctrath has joined #openstack-nova | 18:40 | |
| ildikov | jgriffith: what to say, I don't know how longer I can go without sleeping :) | 18:41 |
| ildikov | *how much | 18:41 |
| jgriffith | ildikov: don't do that to yourself! :) | 18:41 |
| *** ndipanov has quit IRC | 18:42 | |
| *** ndipanov has joined #openstack-nova | 18:43 | |
| ildikov | jgriffith: I have no time schedule skills, I'm trying to improve there day by day :) | 18:43 |
| jgriffith | ildikov: I hear ya, and suffer the same curse | 18:44 |
| ildikov | I think many of us do here | 18:44 |
| ildikov | ... in some extent at least | 18:44 |
| *** zenoway has joined #openstack-nova | 18:45 | |
| melwitt | ccarmack: I just checked a recent cells tempest job run and it looks like all of the settings I was thinking of are already set the way we want by default http://logs.openstack.org/40/128940/107/check/gate-tempest-dsvm-cells/6ae292e/logs/tempest_conf.txt.gz | 18:47 |
| *** READ10 has quit IRC | 18:47 | |
| ccarmack | melwitt: looking | 18:47 |
| melwitt | ccarmack: that is, the compute.ssh_connect_method = fixed, validation.connect_method = fixed, validation.run_validation = False | 18:48 |
| *** ndipanov has quit IRC | 18:48 | |
| *** lyarwood has quit IRC | 18:48 | |
| *** slunkad_ has quit IRC | 18:48 | |
| *** annegentle has quit IRC | 18:48 | |
| *** mrsoul` has quit IRC | 18:48 | |
| *** ndipanov has joined #openstack-nova | 18:48 | |
| *** lyarwood has joined #openstack-nova | 18:48 | |
| *** slunkad_ has joined #openstack-nova | 18:48 | |
| *** annegentle has joined #openstack-nova | 18:48 | |
| *** mrsoul` has joined #openstack-nova | 18:48 | |
| *** lpetrut has joined #openstack-nova | 18:48 | |
| ccarmack | melwitt: the run_validation = false is what prevents the verify ssh in test_server_basic_ops | 18:49 |
| *** rdopiera has quit IRC | 18:49 | |
| *** eglynn_ has quit IRC | 18:49 | |
| melwitt | ccarmack: oh, hmm | 18:49 |
| *** doug-fish has quit IRC | 18:49 | |
| ccarmack | https://github.com/openstack/tempest/blob/master/tempest/scenario/test_server_basic_ops.py#L64 | 18:50 |
| *** claudiub has joined #openstack-nova | 18:50 | |
| ccarmack | melwitt: I was going to ask mtreinish about it in the qa channel.... why it is false, and can I add a test just for cells that does the ssh | 18:52 |
| ccarmack | melwitt: because I removed the blacklist for test_server_basic_ops in my nova patchset | 18:53 |
| *** browne has joined #openstack-nova | 18:53 | |
| *** doug-fish has joined #openstack-nova | 18:53 | |
| *** tjones1 has joined #openstack-nova | 18:54 | |
| *** flaper87 has quit IRC | 18:54 | |
| *** flaper87 has joined #openstack-nova | 18:54 | |
| *** luis_ has joined #openstack-nova | 18:54 | |
| ccarmack | melwitt: I checked the other scenario tests doing ssh, and they are all skipped for some reason -- nova rc blacklists them for floating ip, or a skip for a bug | 18:54 |
| *** slDabbler has quit IRC | 18:55 | |
| melwitt | ccarmack: right. I had missed the run_validation = False preventing all the ssh, that is strange | 18:55 |
| *** rcrit has joined #openstack-nova | 18:56 | |
| *** hblixt has joined #openstack-nova | 18:56 | |
| *** mdbooth has quit IRC | 18:56 | |
| *** mdbooth has joined #openstack-nova | 18:56 | |
| *** ansiwen has quit IRC | 18:56 | |
| *** ansiwen has joined #openstack-nova | 18:56 | |
| *** Marga_ has quit IRC | 18:56 | |
| *** mc_nair has quit IRC | 18:57 | |
| *** mc_nair has joined #openstack-nova | 18:57 | |
| *** markmc` has quit IRC | 18:57 | |
| *** markmc` has joined #openstack-nova | 18:57 | |
| *** eharney has quit IRC | 18:57 | |
| *** eharney has joined #openstack-nova | 18:57 | |
| *** rlrossit has quit IRC | 18:57 | |
| *** rlrossit has joined #openstack-nova | 18:57 | |
| *** 77CAALV3Z has joined #openstack-nova | 18:57 | |
| *** luis_ has quit IRC | 18:57 | |
| *** edmondsw has quit IRC | 18:57 | |
| *** edmondsw has joined #openstack-nova | 18:57 | |
| *** mtanino has quit IRC | 18:57 | |
| *** mtanino has joined #openstack-nova | 18:57 | |
| *** smatzek_ has quit IRC | 18:57 | |
| *** smatzek_ has joined #openstack-nova | 18:57 | |
| *** pratikmallya has joined #openstack-nova | 18:57 | |
| *** rfolco has quit IRC | 18:57 | |
| *** boris-42 has quit IRC | 18:57 | |
| *** boris-42 has joined #openstack-nova | 18:57 | |
| *** andymaier has quit IRC | 18:58 | |
| *** andymaier has joined #openstack-nova | 18:58 | |
| *** doug-fish has quit IRC | 18:58 | |
| *** kfarr has quit IRC | 18:58 | |
| *** kfarr has joined #openstack-nova | 18:58 | |
| *** pratikma_ has joined #openstack-nova | 18:58 | |
| *** Sukhdev has quit IRC | 18:58 | |
| *** _xin1 has quit IRC | 18:59 | |
| *** _xin1 has joined #openstack-nova | 18:59 | |
| *** migi_ has quit IRC | 18:59 | |
| *** migi_ has joined #openstack-nova | 18:59 | |
| *** 77CAALV3Z has quit IRC | 18:59 | |
| *** ndipanov has quit IRC | 18:59 | |
| *** lyarwood has quit IRC | 18:59 | |
| *** slunkad_ has quit IRC | 18:59 | |
| *** annegentle has quit IRC | 18:59 | |
| *** mrsoul` has quit IRC | 18:59 | |
| *** wasmum- has quit IRC | 18:59 | |
| *** wasmum- has joined #openstack-nova | 18:59 | |
| *** ajo_ has quit IRC | 18:59 | |
| *** ajo_ has joined #openstack-nova | 18:59 | |
| *** mriedem has quit IRC | 18:59 | |
| *** mriedem has joined #openstack-nova | 18:59 | |
| *** penick has quit IRC | 18:59 | |
| *** penick has joined #openstack-nova | 18:59 | |
| *** ccarmack has quit IRC | 18:59 | |
| *** ccarmack has joined #openstack-nova | 18:59 | |
| *** baoli_ has quit IRC | 18:59 | |
| *** baoli_ has joined #openstack-nova | 18:59 | |
| *** ctrath has quit IRC | 18:59 | |
| *** ctrath has joined #openstack-nova | 18:59 | |
| *** mriedem has quit IRC | 19:00 | |
| melwitt | ccarmack: looks like devstack defaults it to false if not set in the env https://github.com/openstack-dev/devstack/blob/2d9f777cc076df863af659caeddc9583fba1e1e7/lib/tempest#L450 | 19:00 |
| *** pratikma_ has quit IRC | 19:00 | |
| *** pratikma_ has joined #openstack-nova | 19:00 | |
| melwitt | ccarmack: and I only see that getting set to true in project-config for neutron related jobs | 19:00 |
| *** knikolla has quit IRC | 19:00 | |
| *** haomaiwang has quit IRC | 19:01 | |
| *** 17WABLM3R has joined #openstack-nova | 19:01 | |
| *** browne has quit IRC | 19:01 | |
| *** mragupat has joined #openstack-nova | 19:02 | |
| *** mragupat has quit IRC | 19:02 | |
| ccarmack | melwitt: yea, I need to talk to mtreinish | 19:02 |
| *** andymaier has quit IRC | 19:02 | |
| *** pratikmallya has quit IRC | 19:02 | |
| *** mragupat has joined #openstack-nova | 19:02 | |
| *** browne has joined #openstack-nova | 19:02 | |
| *** mragupat has quit IRC | 19:03 | |
| *** mragupat has joined #openstack-nova | 19:03 | |
| *** cfarquhar_ has quit IRC | 19:03 | |
| *** tjones has quit IRC | 19:04 | |
| melwitt | cool | 19:04 |
| *** liverpooler has joined #openstack-nova | 19:05 | |
| *** mriedem has joined #openstack-nova | 19:05 | |
| *** cfarquhar has joined #openstack-nova | 19:07 | |
| *** cfarquhar has quit IRC | 19:07 | |
| *** cfarquhar has joined #openstack-nova | 19:07 | |
| *** RichardRaseley has joined #openstack-nova | 19:08 | |
| *** Marga_ has joined #openstack-nova | 19:09 | |
| *** doug-fish has joined #openstack-nova | 19:09 | |
| *** eharney has quit IRC | 19:10 | |
| *** doug-fis_ has joined #openstack-nova | 19:10 | |
| *** unicell has left #openstack-nova | 19:11 | |
| *** tjones has joined #openstack-nova | 19:11 | |
| *** knikolla has joined #openstack-nova | 19:13 | |
| *** doug-fish has quit IRC | 19:14 | |
| *** vilobhmm11 has joined #openstack-nova | 19:16 | |
| *** Sukhdev has joined #openstack-nova | 19:16 | |
| *** aix has quit IRC | 19:16 | |
| *** bnemec has quit IRC | 19:17 | |
| *** ijw_ has joined #openstack-nova | 19:17 | |
| *** vilobhmm111 has joined #openstack-nova | 19:17 | |
| *** ZZelle_ has joined #openstack-nova | 19:19 | |
| *** ducttape_ has joined #openstack-nova | 19:19 | |
| *** knikolla has quit IRC | 19:20 | |
| *** vilobhmm11 has quit IRC | 19:20 | |
| *** knikolla has joined #openstack-nova | 19:20 | |
| *** ociuhandu has quit IRC | 19:20 | |
| *** ijw has quit IRC | 19:20 | |
| *** achanda has quit IRC | 19:21 | |
| *** tdurakov has joined #openstack-nova | 19:24 | |
| *** Marga_ has quit IRC | 19:25 | |
| *** bnemec has joined #openstack-nova | 19:25 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Move config options from nova/network/linux_net.py https://review.openstack.org/270441 | 19:26 |
| *** eharney has joined #openstack-nova | 19:26 | |
| ccarmack | melwitt: I'm chatting with mtreinish in the qa channel if you are interested | 19:26 |
| *** busterswt has joined #openstack-nova | 19:26 | |
| *** eharney is now known as Guest80152 | 19:26 | |
| *** diogogmt has joined #openstack-nova | 19:28 | |
| mriedem | nic: so close https://review.openstack.org/#/c/187395/44/nova/tests/unit/virt/libvirt/test_driver.py | 19:29 |
| *** tonytan4ever has quit IRC | 19:30 | |
| nic | mriedem: "Shouldn't you be asserting that rollback_to_snap and remove_snap is called in here somewhere?" | 19:31 |
| nic | You mean like I do on lines 13758-13759? | 19:31 |
| mriedem | heh blarg | 19:31 |
| mriedem | ok that's fine then, but the next test | 19:32 |
| nic | New revision incoming | 19:32 |
| mriedem | asserts that the mock is raising the side_effect | 19:32 |
| *** bdrich has quit IRC | 19:32 | |
| mriedem | then it's +2 time big boy | 19:32 |
| nic | Yay! I'm a big boy today! | 19:33 |
| *** Guest80152 is now known as eharney | 19:33 | |
| *** eharney has quit IRC | 19:33 | |
| *** eharney has joined #openstack-nova | 19:33 | |
| mriedem | https://www.youtube.com/watch?v=4q4fZxQmKGU | 19:34 |
| mriedem | yes you are | 19:34 |
| ildikov | mriedem: I wonder when it will be +2 time for big girls ;) | 19:34 |
| mriedem | i don't have a clip for that | 19:34 |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: Fix/implement revert-resize for RBD-backed images https://review.openstack.org/187395 | 19:34 |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: make snapshot use RBD snapshot/clone when available https://review.openstack.org/205282 | 19:34 |
| ildikov | mriedem: yeah, I just checked it :D | 19:35 |
| mriedem | probably delayed b/c now i'm distracted with the simpsons | 19:35 |
| ildikov | :) | 19:35 |
| *** smcginni1 is now known as smcginnis | 19:38 | |
| *** slDabbler has joined #openstack-nova | 19:38 | |
| mriedem | uh oh | 19:45 |
| *** mtanino has quit IRC | 19:45 | |
| *** tjones has quit IRC | 19:50 | |
| *** tjones has joined #openstack-nova | 19:51 | |
| *** achanda has joined #openstack-nova | 19:55 | |
| *** Marga_ has joined #openstack-nova | 19:55 | |
| *** paul-carlton2 has quit IRC | 19:56 | |
| ildikov | mriedem: what's the 'uh oh' is for? :) | 19:56 |
| *** pushkaru has quit IRC | 19:58 | |
| *** hemna is now known as hemnafk | 19:58 | |
| mriedem | don't worry, it's nic's patch | 19:59 |
| *** ctrath has quit IRC | 19:59 | |
| nic | ? | 19:59 |
| ildikov | mriedem: ok :) | 20:00 |
| mriedem | nic: http://logs.openstack.org/82/205282/39/check/gate-tempest-dsvm-full/06003c5/logs/screen-n-cpu.txt.gz?level=TRACE#_2016-01-20_16_37_12_966 | 20:00 |
| *** ducttape_ has quit IRC | 20:00 | |
| mriedem | notice for non-rbd backends, "Performing standard snapshot because direct snapshot failed" | 20:00 |
| *** ctrath has joined #openstack-nova | 20:00 | |
| mriedem | we shouldn't be warning for NotImplementedError | 20:00 |
| *** 17WABLM3R has quit IRC | 20:01 | |
| nic | DEBUG? | 20:01 |
| *** haomaiwang has joined #openstack-nova | 20:01 | |
| nic | Or nothing at all? | 20:01 |
| *** tjones1 has quit IRC | 20:02 | |
| mriedem | well, we want a warning in the case that the direct snapshot on the rbd backend fails | 20:02 |
| mriedem | but we don't want a warning if it's just that the imagebackend we're using doesn't implement that direct snapshot method | 20:02 |
| mriedem | i'm commenting inline | 20:02 |
| nic | I pretty much parroted how commit 112b140 did things… | 20:04 |
| nic | Trying to play nice with extant code and all | 20:04 |
| *** su_zhang has quit IRC | 20:04 | |
| *** knikolla has quit IRC | 20:07 | |
| *** mragupat_ has joined #openstack-nova | 20:08 | |
| *** paul-carlton2 has joined #openstack-nova | 20:09 | |
| *** rlrossit has quit IRC | 20:11 | |
| *** mragupat has quit IRC | 20:11 | |
| bauzas | alaski: heya, sorry was afk today because of an OpenStack class teaching | 20:12 |
| alaski | bauzas: no worries, I just wasn't sure about when some backwards compat code can be removed. I didn't notice a comment providing that info | 20:14 |
| bauzas | alaski: tbc, I'm okay with removing that since it was about Kilo computes - but I think the commit msg should say why | 20:15 |
| alaski | bauzas: agreed | 20:16 |
| tjones | anyone else having hyper-v failures in CI (ssh timeout)? | 20:17 |
| tjones | http://64.119.130.115/nova/266060/4/results.html.gz | 20:17 |
| *** salv-orlando has quit IRC | 20:18 | |
| *** annegentle has joined #openstack-nova | 20:18 | |
| *** matrohon has joined #openstack-nova | 20:18 | |
| *** ndipanov has joined #openstack-nova | 20:18 | |
| *** lyarwood has joined #openstack-nova | 20:18 | |
| *** mrsoul` has joined #openstack-nova | 20:18 | |
| bauzas | alaski: I'm adding https://review.openstack.org/#/c/126372/19/nova/compute/resource_tracker.py to the comment | 20:18 |
| *** gongysh has joined #openstack-nova | 20:18 | |
| *** davidlenwell has quit IRC | 20:18 | |
| *** changbl has quit IRC | 20:19 | |
| *** davidlenwell has joined #openstack-nova | 20:20 | |
| mriedem | nic: comments published | 20:21 |
| *** matrohon has quit IRC | 20:21 | |
| *** matrohon has joined #openstack-nova | 20:22 | |
| bauzas | alaski: so I missed again one cells v2 meeting, right ? ;( | 20:22 |
| bauzas | :( | 20:22 |
| *** gongysh has quit IRC | 20:23 | |
| bauzas | meh, is it me or the gate is bad ? | 20:23 |
| melwitt | alaski: I noticed the CellMapping.create doesn't generate a uuid for the cell mapping. is that a bug? | 20:24 |
| mriedem | bauzas: yes things are bad | 20:24 |
| bauzas | :( | 20:24 |
| * bauzas looking at the mailing | 20:25 | |
| mriedem | i think it's all under control, | 20:25 |
| mriedem | it's just the matter of getting everything caught back up after like 2-3 days of resets | 20:25 |
| *** ljxiash has quit IRC | 20:27 | |
| *** mtanino has joined #openstack-nova | 20:27 | |
| alaski | melwitt: it's not a bug, I was expecting the creator to assign a uuid. I could be convinced that it should set it's own uuid though. | 20:27 |
| alaski | melwitt: although cell0 creation might need to pass in a custom uuid | 20:27 |
| mriedem | mikal: would be nice for you to review this one since it's holding up a bp for feature freeze, has a +2, pretty straight forward https://review.openstack.org/#/c/187395/ | 20:27 |
| *** ljxiash has joined #openstack-nova | 20:27 | |
| mriedem | mikal: plus related to imagebackend stuff, your old favorite | 20:27 |
| *** apoorvad has quit IRC | 20:27 | |
| melwitt | alaski: oh, okay. I'll add --uuid as a param then. I was used to some other things auto generating uuid | 20:27 |
| melwitt | I suppose the nova-manage command could generate one if one isn't provided | 20:28 |
| *** mtanino has quit IRC | 20:28 | |
| *** matrohon has quit IRC | 20:28 | |
| *** annegentle has quit IRC | 20:28 | |
| *** ndipanov has quit IRC | 20:28 | |
| *** lyarwood has quit IRC | 20:28 | |
| *** mrsoul` has quit IRC | 20:28 | |
| alaski | bauzas: yep, missed another meeting. I didn't ping you since you didn't answer my question asked around the same time, figured you were out | 20:28 |
| alaski | melwitt: yeah, I think nova-manage should generate one. | 20:29 |
| melwitt | alaski: cool, thanks | 20:29 |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Ceph for live-migration job https://review.openstack.org/252283 | 20:29 |
| bauzas | alaski: my bad :( | 20:30 |
| nic | Oh cool, stack.sh is broken | 20:31 |
| alaski | bauzas: no worries | 20:31 |
| *** n0ano has joined #openstack-nova | 20:31 | |
| *** ljxiash has quit IRC | 20:31 | |
| *** Thelo has quit IRC | 20:33 | |
| *** bdrich has joined #openstack-nova | 20:33 | |
| *** otter768 has joined #openstack-nova | 20:35 | |
| *** diana_clarke has quit IRC | 20:37 | |
| *** otter768 has quit IRC | 20:39 | |
| *** nic has quit IRC | 20:40 | |
| *** nic has joined #openstack-nova | 20:42 | |
| *** ociuhandu has joined #openstack-nova | 20:42 | |
| *** ijw has joined #openstack-nova | 20:42 | |
| *** Thelo has joined #openstack-nova | 20:42 | |
| *** mtanino has joined #openstack-nova | 20:42 | |
| *** matrohon has joined #openstack-nova | 20:42 | |
| *** annegentle has joined #openstack-nova | 20:42 | |
| *** ndipanov has joined #openstack-nova | 20:42 | |
| *** lyarwood has joined #openstack-nova | 20:42 | |
| *** mrsoul` has joined #openstack-nova | 20:42 | |
| *** mtanino has quit IRC | 20:42 | |
| *** mtanino has joined #openstack-nova | 20:42 | |
| *** boris-42 has quit IRC | 20:43 | |
| *** ijw_ has quit IRC | 20:45 | |
| *** hemnafk is now known as hemna | 20:45 | |
| *** pratikma_ has quit IRC | 20:48 | |
| *** rlrossit_ has joined #openstack-nova | 20:50 | |
| *** ducttape_ has joined #openstack-nova | 20:52 | |
| *** ijw has quit IRC | 20:53 | |
| *** Thelo has quit IRC | 20:53 | |
| *** mtanino has quit IRC | 20:53 | |
| *** matrohon has quit IRC | 20:53 | |
| *** annegentle has quit IRC | 20:53 | |
| *** ndipanov has quit IRC | 20:53 | |
| *** lyarwood has quit IRC | 20:53 | |
| *** mrsoul` has quit IRC | 20:53 | |
| mriedem | ildikov: still awake? | 20:53 |
| ildikov | mriedem: yes | 20:53 |
| mriedem | https://review.openstack.org/#/c/193134/80/nova/volume/cinder.py | 20:54 |
| *** changbl has joined #openstack-nova | 20:54 | |
| mriedem | clean that up and i'm +2 | 20:54 |
| ildikov | mriedem: cool, sorry for all these roukie mistakes | 20:55 |
| mriedem | want to fix the typo in the commit message here when you rebase? https://review.openstack.org/#/c/256273/ | 20:56 |
| sdague | anyone have time for a story time / question time? | 20:57 |
| *** tonytan4ever has joined #openstack-nova | 20:57 | |
| *** tdurakov has quit IRC | 20:58 | |
| sdague | today in nova if you pass an invalid content-type we swallow that fact early and zero out the body - https://github.com/openstack/nova/blob/f7adbfdb9e163bb9369b26fe30bb353204adb8be/nova/api/openstack/wsgi.py#L619 | 20:58 |
| sdague | and then it tumbles in the stack... | 20:59 |
| sdague | probably triggering an invalid content type exception later by accident, though maybe not | 21:00 |
| *** jschlueter has quit IRC | 21:00 | |
| *** haomaiwang has quit IRC | 21:01 | |
| mriedem | sdague: see topic | 21:01 |
| sdague | ... ? | 21:01 |
| mriedem | sdague: boilerplate response to questions no one knows the answer to or wants to investigate :) | 21:02 |
| sdague | heh | 21:02 |
| mriedem | git tells me that Vek wrote all of that code back in essex | 21:02 |
| sdague | yeh, well, I'm trying to delete most of it | 21:02 |
| sdague | piece by peice | 21:03 |
| sdague | I just hit a behavior here that is a bit undefined | 21:03 |
| sdague | or at least hard to characterize | 21:03 |
| *** ihrachys has joined #openstack-nova | 21:03 | |
| sdague | because we end up turning bad content types into None, which is the same as no content type being passed, which we can't distinguish later | 21:03 |
| sdague | it seems like we should throw the bad content type early | 21:04 |
| cdent | sdague: yes, yes we should | 21:04 |
| mriedem | sdague: yeah, seems like you'd eventually get here https://github.com/openstack/nova/blob/f7adbfdb9e163bb9369b26fe30bb353204adb8be/nova/api/openstack/wsgi.py#L773-L776 | 21:05 |
| mriedem | if something was validating the content-type, but if it's None you don't know | 21:05 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: libvirt: Allow multiple volume attachments https://review.openstack.org/193135 | 21:05 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: Pass attachment_id to Cinder when detach a volume https://review.openstack.org/193134 | 21:05 |
| sdague | mriedem: yeh, except I'm trying to delete that code | 21:05 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: comp: Allow volume multiattach https://review.openstack.org/267587 | 21:05 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach https://review.openstack.org/153038 | 21:05 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: Pass host when call attach to Cinder https://review.openstack.org/256273 | 21:05 |
| sdague | because that was all about deserializers | 21:05 |
| sdague | which we only have 1 | 21:05 |
| mriedem | but rax has many? | 21:05 |
| mriedem | or i guess we jus tused to have xml | 21:05 |
| sdague | https://review.openstack.org/#/c/270215/ | 21:06 |
| mriedem | and this was part of that | 21:06 |
| sdague | yes | 21:06 |
| sdague | it was the code to generalize for xml | 21:06 |
| ildikov | mriedem: fixed the commit message too ^^ thanks for catching | 21:06 |
| mriedem | sdague: heh, well Vek is +2 on your change so you could ask him if he remembers anything about that case | 21:06 |
| mriedem | but it seems reasonable to fail early there | 21:06 |
| *** eglynn_ has joined #openstack-nova | 21:07 | |
| sdague | yeh, I think a lot of unit tests might need to update for it | 21:07 |
| openstackgerrit | Tobias Urdin proposed openstack/nova: Move is_volume_backed_instance to compute.utils https://review.openstack.org/270482 | 21:08 |
| cdent | man, nova's little wsgi framework is...special | 21:09 |
| sdague | yeh, well, it grew up to do some things | 21:10 |
| sdague | and we deleted a bunch of it | 21:10 |
| sdague | but the generics to support that stuff was left behind | 21:11 |
| * cdent nods | 21:11 | |
| *** tdurakov_ has joined #openstack-nova | 21:11 | |
| nic | mriedem: you mean to tell me you want me to remove those preposterous RBD mocks in the test_driver stuff? THIS MAKES ME SO HAPPY | 21:11 |
| nic | I'll get a new revision pushed in an hour or so | 21:12 |
| mriedem | nic: heh, i thought you wanted that | 21:12 |
| mriedem | A->B->C, testing A shouldn't mock C | 21:12 |
| mriedem | ildikov: +2 on https://review.openstack.org/#/c/193134/ despite some cleanup if you respin | 21:12 |
| nic | It seemed like a good idea at the time? | 21:12 |
| cburgess | mriedem So no to stir the pot here but.... given the deadline is rapidly approaching, how are things looking with this RBD feature? | 21:13 |
| mriedem | nic: sounds like me in college | 21:13 |
| mriedem | nic: so like i said in the review, you could cleanup the test_driver mock code in a follow up change, | 21:14 |
| cburgess | mriedem nic I say that about some of the code I wrote in the early days of MC all the time. | 21:14 |
| mriedem | nic: i was really just -1 on the gratuitious warnings when not using the rbd image backend and doing the snapshot | 21:14 |
| *** raildo is now known as raildo-afk | 21:14 | |
| mriedem | cburgess: i'm +2 on the dependent change | 21:15 |
| nic | mriedem: I've already got those fixed, is fixing the RBD tests in test_driver a hard requirement for you, too? | 21:15 |
| mriedem | and after the warnings for direct-snapshot not implemented are only logged when using rbd in the top change, i'll be +2 on that - with a promise from nic to cleanup the libvirt test_driver mocking | 21:15 |
| mriedem | nic: not for this change, no, that could wait for a follow on change | 21:15 |
| nic | OK, I'll hurry, then | 21:16 |
| *** Sukhdev has quit IRC | 21:16 | |
| mriedem | cburgess: after that, i'll need to lobby mikal for a review tonight | 21:16 |
| cburgess | mriedem OK just starting to get worried. Really don't want to see this feature miss another release. At this point there are several people who have carried this feature locally for a while and would be great to finally get this in. | 21:16 |
| mriedem | s/i'll/you'll/ | 21:16 |
| mriedem | cburgess: yeah i don't want to see it slip either | 21:17 |
| mriedem | just b/c i <3 me some MC guys | 21:17 |
| cburgess | mriedem Oh you want me to? OK. I'm not above bribing mikal. :) Might cost me in the future. | 21:17 |
| cfriesen | I think we've stumbled over a potential problem with the confirmation of a resize/migration. If we call confirm_resize() that sets migration.status to 'confirming' and sends an RPC cast to the compute node. If there's a glitch and that cast is never processed, there's no way to either confirm or revert the resize since both APIs look for migrations with a status of "finished". | 21:18 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: libvirt: Allow multiple volume attachments https://review.openstack.org/193135 | 21:18 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: Pass attachment_id to Cinder when detach a volume https://review.openstack.org/193134 | 21:18 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: comp: Allow volume multiattach https://review.openstack.org/267587 | 21:18 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach https://review.openstack.org/153038 | 21:18 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: Pass host when call attach to Cinder https://review.openstack.org/256273 | 21:18 |
| *** smatzek_ has quit IRC | 21:18 | |
| *** salv-orlando has joined #openstack-nova | 21:18 | |
| mriedem | ildikov: lovely | 21:19 |
| ildikov | mriedem: fixed it now | 21:19 |
| *** salv-orlando has quit IRC | 21:19 | |
| *** salv-orlando has joined #openstack-nova | 21:19 | |
| ildikov | mriedem: I promise I go and get some sleep, that should help with my reading ability too :) | 21:20 |
| ildikov | mriedem: thanks much | 21:20 |
| mriedem | later | 21:20 |
| *** eglynn_ has quit IRC | 21:22 | |
| ildikov | laters | 21:22 |
| *** eglynn_ has joined #openstack-nova | 21:22 | |
| edleafe | mriedem: with He Who Shall Not Be Named out, can you weigh in on marcus_z's question on https://review.openstack.org/#/c/268186/1/nova/conf/cert.py | 21:25 |
| sdague | cdent: but, having been deep into some of the routes work here to get past project_id, I figured it was time to just rip some of these bits out | 21:25 |
| *** achanda has quit IRC | 21:26 | |
| *** delattec has quit IRC | 21:28 | |
| *** apoorvad has joined #openstack-nova | 21:28 | |
| *** mdnadeem has joined #openstack-nova | 21:30 | |
| *** thelo_ has joined #openstack-nova | 21:31 | |
| mriedem | edleafe: he's right on the 2nd one about the version aliases | 21:31 |
| mriedem | edleafe: i'm not sure about the first one, the wording is odd, i'd ask dansmith, he's the rpc compat wizard | 21:31 |
| edleafe | mriedem: yeah, already noted that | 21:31 |
| mriedem | he's also off irc so we can call him out now | 21:32 |
| edleafe | mriedem: AHHHH YOU SAID THE NAME! | 21:32 |
| *** apoorvad has quit IRC | 21:33 | |
| *** apoorvad has joined #openstack-nova | 21:34 | |
| thelo_ | mriedem: Hi regarding this patch https://review.openstack.org/#/c/253353/ we are still waiting for the os-brick library to be added into the openstack requirements ( so far its been queue for > 24h). Is there any chance that we can merge it before the deadline or add it after? | 21:35 |
| mriedem | jroll: so i guess this isn't happening for mitaka https://review.openstack.org/#/c/206163/ | 21:36 |
| mriedem | thelo_: sorry, forgot all about that one | 21:36 |
| jroll | mriedem: I'd like to, I've been slacking on getting back to it while we finish the ironic side. I actually poked john about this earlier but he was already afk | 21:36 |
| thelo_ | mriedem: My bad, I didnt remind you because we went through several issues while trying to issue a new os-brick release | 21:37 |
| mriedem | thelo_: ok, yeah, i see https://review.openstack.org/#/c/267725/ is in the gate | 21:38 |
| *** mdnadeem has quit IRC | 21:38 | |
| *** htruta` is now known as htruta | 21:38 | |
| mriedem | jroll: yeah, but that patch is nowhere near ready and tomorrow is FF | 21:39 |
| jroll | mriedem: I understand, my goal was to update it today and apply for FFE | 21:39 |
| jroll | because I'm a terrible person, mostly | 21:39 |
| mriedem | i didn't want to say it | 21:39 |
| mriedem | :) | 21:39 |
| jroll | mriedem: a huge amount of code is landing in ironic this cycle, that patch is the last nova thing to make the feature complete, so I'd be really sad to not make mitaka | 21:40 |
| jroll | and I know that's also a crappy argument | 21:40 |
| jroll | but that's all I've really got right now | 21:40 |
| mriedem | let's blame alaski | 21:40 |
| jroll | sounds fun | 21:41 |
| * alaski slowly walks away | 21:41 | |
| alaski | I did approve a patch in that series. So yay, I helped! | 21:43 |
| *** pratikmallya has joined #openstack-nova | 21:45 | |
| *** ihrachys has quit IRC | 21:45 | |
| mriedem | high five | 21:45 |
| mriedem | we both did | 21:45 |
| mriedem | well, +2'ed one anyway | 21:46 |
| mriedem | thelo_: do you have 3rd party ci for this in cinder? | 21:46 |
| *** penick has quit IRC | 21:46 | |
| thelo_ | mriedem: Yes for both osbrick and cinder | 21:46 |
| mriedem | thelo_: and i'm assuming the 3rd party ci is running with the nova patch atm? | 21:47 |
| thelo_ | yes | 21:47 |
| *** mrsoul has joined #openstack-nova | 21:47 | |
| mriedem | http://128.199.204.149/52/253352/10/check/ITRI_DISCO_os-brick/8941576/ | 21:47 |
| openstackgerrit | Sean Dague proposed openstack/nova: cleanup: remove infrastructure for content/type deserializer https://review.openstack.org/270497 | 21:47 |
| *** cdelatte has joined #openstack-nova | 21:47 | |
| *** haomaiwa_ has joined #openstack-nova | 21:47 | |
| *** ijw has joined #openstack-nova | 21:47 | |
| *** su_zhang has joined #openstack-nova | 21:47 | |
| *** Thelo has joined #openstack-nova | 21:48 | |
| mriedem | thelo_: heh, it's using the lvm driver http://128.199.204.149/52/253352/10/check/ITRI_DISCO_os-brick/8941576/logs/etc/cinder/cinder.conf.txt.gz | 21:48 |
| mriedem | not disco | 21:48 |
| *** thangp has left #openstack-nova | 21:49 | |
| *** cdent has quit IRC | 21:49 | |
| thelo_ | mriedem: true for os brick it is, for cinder it uses the disco driver | 21:49 |
| mriedem | hmmm | 21:49 |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: make snapshot use RBD snapshot/clone when available https://review.openstack.org/205282 | 21:49 |
| thelo_ | mriedem/ hemna told me to do this until the atch is accepted then to move to DISCO driver | 21:49 |
| mriedem | which patch? the nova one or the os-brick one? | 21:50 |
| *** davidlenwell has quit IRC | 21:52 | |
| *** davidlenwell has joined #openstack-nova | 21:54 | |
| *** omnipresent has joined #openstack-nova | 21:54 | |
| thelo_ | mriedem: for nova,os-brick and cinder. right now the patches for os-brick and cinder have been merged. the CI for cinder is using the patches for nova, os-brick and cinder, while the osbrick CI does not use any patch at the moment and its configured fr LVM. | 21:55 |
| *** changbl has quit IRC | 21:55 | |
| *** cdelatte has quit IRC | 21:56 | |
| mriedem | thelo_: ok, but the ci for os-brick should be using the disco backend i'd think | 21:57 |
| mriedem | thelo_: anyway, comments in https://review.openstack.org/#/c/253353/ | 21:57 |
| mriedem | it's pretty clean, just small issues | 21:57 |
| mriedem | thelo_: get that cleaned up today/tonight yet and i'll +2 | 21:57 |
| thelo_ | mriedem: sure I will change this when I go to work | 21:58 |
| mriedem | what tz are you in? | 21:58 |
| *** zenoway has quit IRC | 21:58 | |
| thelo_ | mriedem: asia pacific - Taipei | 21:58 |
| mriedem | ah | 21:58 |
| mriedem | ok | 21:58 |
| *** cdelatte has joined #openstack-nova | 21:58 | |
| *** eglynn_ has quit IRC | 21:59 | |
| thelo_ | mriedem: so tonight is basically in 14hours for me. | 22:00 |
| *** davidlenwell has quit IRC | 22:00 | |
| *** haomaiwa_ has quit IRC | 22:01 | |
| mriedem | sure, so if you can get that updated this morning for your moring, that'd be good | 22:01 |
| mriedem | b/c it's 4pm for me | 22:01 |
| *** lyarwood has joined #openstack-nova | 22:01 | |
| *** ndipanov has joined #openstack-nova | 22:01 | |
| *** annegentle has joined #openstack-nova | 22:01 | |
| *** haomaiwang has joined #openstack-nova | 22:01 | |
| thelo_ | mriedem: thanks I do it now then | 22:01 |
| mriedem | and i'll be working from home for a few hours tonight, but that's it - the family folk demand my attention | 22:01 |
| *** davidlenwell has joined #openstack-nova | 22:02 | |
| nic | mriedem: new revision pushed. Ta-da! | 22:02 |
| *** cdelatte has quit IRC | 22:02 | |
| mriedem | gerritbot doesn't think so | 22:04 |
| *** annegent_ has joined #openstack-nova | 22:04 | |
| *** annegentle has quit IRC | 22:04 | |
| mriedem | i see it | 22:04 |
| nic | Patchset #41 | 22:04 |
| *** annegentle has joined #openstack-nova | 22:04 | |
| *** penick has joined #openstack-nova | 22:05 | |
| mriedem | nic: i'm pretty sure that test_raw_with_rbd_clone_everything_fails isn't testing that generic exception block in the driver | 22:06 |
| *** cdelatte has joined #openstack-nova | 22:06 | |
| mriedem | test_raw_with_rbd_clone_everything_fails tests that direct_snapshot raises Forbidden which gets you into the fallback block, and then the test raises IOError from that | 22:06 |
| mriedem | which isn't the same block that's handling the generic exception that's on the same level as the Forbidden/ImageUnacceptable handling | 22:06 |
| openstackgerrit | Tobias Urdin proposed openstack/nova: Move is_volume_backed_instance to compute.utils https://review.openstack.org/270482 | 22:07 |
| hemna | thelo_, looks like you are getting some CI failures in os-brick (timeouts) | 22:07 |
| hemna | thelo_, http://128.199.204.149/81/268481/2/check/ITRI_DISCO_os-brick/c5d7de6/console.html | 22:08 |
| hemna | FYI | 22:08 |
| *** annegent_ has quit IRC | 22:08 | |
| thelo_ | hemna: Ok, I ll take a look at this today | 22:08 |
| nic | mriedem: it was according to the debugger I attached | 22:09 |
| mriedem | nic: hrm, ok... | 22:09 |
| *** lpetrut has quit IRC | 22:09 | |
| mriedem | still reviewing,i'll check again | 22:09 |
| *** annegentle has quit IRC | 22:11 | |
| *** lyarwood has quit IRC | 22:11 | |
| *** ndipanov has quit IRC | 22:11 | |
| *** annegentle has joined #openstack-nova | 22:11 | |
| *** lyarwood has joined #openstack-nova | 22:11 | |
| *** ndipanov has joined #openstack-nova | 22:11 | |
| *** kragniz_ is now known as kragniz | 22:12 | |
| mriedem | nic: you're mocking out this: | 22:13 |
| mriedem | with libvirt_utils.file_open(out_path) as image_file: | 22:13 |
| *** thelo_ has quit IRC | 22:13 | |
| mriedem | nic: which is down a level from where except Exception: is handled on line 1566 | 22:13 |
| *** achanda has joined #openstack-nova | 22:14 | |
| *** thorst has quit IRC | 22:14 | |
| nic | Huh. Yeah. WTF | 22:14 |
| nic | One more revision incoming | 22:14 |
| *** thorst has joined #openstack-nova | 22:15 | |
| *** su_zhang has quit IRC | 22:15 | |
| *** su_zhang has joined #openstack-nova | 22:15 | |
| mriedem | see other comments | 22:16 |
| *** w_verdugo has quit IRC | 22:16 | |
| openstackgerrit | Tobias Urdin proposed openstack/nova: Fix evaluation of host disk usage by volume-backed instances https://review.openstack.org/200870 | 22:18 |
| *** thorst has quit IRC | 22:19 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: WIP - remove mox from vmwareapi/test_driver_api.py https://review.openstack.org/270303 | 22:19 |
| *** edmondsw has quit IRC | 22:21 | |
| *** annegentle has quit IRC | 22:22 | |
| *** lyarwood has quit IRC | 22:22 | |
| *** ndipanov has quit IRC | 22:22 | |
| *** boris-42 has joined #openstack-nova | 22:23 | |
| *** dims has quit IRC | 22:24 | |
| *** kfarr has quit IRC | 22:25 | |
| *** dims has joined #openstack-nova | 22:30 | |
| * mriedem starts listening to danzig | 22:32 | |
| nic | My day: http://i.imgur.com/u3lWw.gif | 22:33 |
| cburgess | Hmm... we might need to send you on vacation. | 22:33 |
| nic | I know, I know, do it right the first time, and you don't have these kinds of problems... | 22:33 |
| cburgess | Well there is that. | 22:33 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the cert options https://review.openstack.org/268186 | 22:34 |
| nic | Action Item: Suck less | 22:34 |
| mriedem | nic: queue up some danzig | 22:36 |
| mriedem | it helps | 22:36 |
| *** changbl has joined #openstack-nova | 22:36 | |
| nic | I'll opt for Terrorfakt, but yes | 22:36 |
| mriedem | or the humpty dance | 22:36 |
| mriedem | b/c it came out before rlrossit_ was born | 22:36 |
| mriedem | so we had to listen to it in our conf room | 22:36 |
| nic | Join me: https://youtu.be/arD_7wcD4Eo | 22:37 |
| nic | OK, WTF, now I can't install deps to run unit tests | 22:38 |
| nic | "Detected a distutils installed project ('argparse') which we cannot uninstall. The metadata provided by distutils does not contain a list of files which have been installed, so pip does not know which files to uninstall." | 22:38 |
| mriedem | nic: pip 8? | 22:39 |
| mriedem | you need pip<8 i think | 22:39 |
| nic | $ pip --version | 22:39 |
| nic | pip 1.5.6 from /usr/local/lib/python2.7/dist-packages (python 2.7) | 22:39 |
| *** ljxiash has joined #openstack-nova | 22:39 | |
| mriedem | hrm, idk then | 22:39 |
| nic | Oh, crap. It's installing 8.0.0 in the venv | 22:39 |
| nic | WTF | 22:39 |
| nic | Easy enough to fix | 22:40 |
| *** thorst has joined #openstack-nova | 22:40 | |
| openstackgerrit | Tobias Urdin proposed openstack/nova: Move is_volume_backed_instance to compute.utils https://review.openstack.org/270482 | 22:40 |
| mriedem | :) | 22:42 |
| *** burgerk_ has quit IRC | 22:42 | |
| *** ljxiash has quit IRC | 22:43 | |
| *** thorst has quit IRC | 22:44 | |
| *** tonytan4ever has quit IRC | 22:45 | |
| *** doug-fis_ has quit IRC | 22:46 | |
| *** jamielennox|away is now known as jamielennox | 22:47 | |
| *** jacky-zhang has joined #openstack-nova | 22:48 | |
| *** jichen has quit IRC | 22:49 | |
| mriedem | nic: gotta pick up my kid, will be online later to check that update | 22:50 |
| *** diogogmt has quit IRC | 22:52 | |
| nic | Thanks | 22:53 |
| *** devananda has quit IRC | 22:54 | |
| *** bdrich has quit IRC | 22:55 | |
| *** achanda has quit IRC | 22:55 | |
| *** mriedem has quit IRC | 22:55 | |
| *** dimtruck is now known as zz_dimtruck | 22:57 | |
| vilobhmm111 | mriedem : ping | 22:57 |
| vilobhmm111 | have couple of min | 22:57 |
| *** devananda has joined #openstack-nova | 22:58 | |
| *** achanda has joined #openstack-nova | 22:58 | |
| *** tdurakov_ has quit IRC | 22:59 | |
| *** ccarmack has quit IRC | 23:00 | |
| *** haomaiwang has quit IRC | 23:01 | |
| *** haomaiwa_ has joined #openstack-nova | 23:01 | |
| *** rlrossit_ has quit IRC | 23:01 | |
| *** nelsnels_ has joined #openstack-nova | 23:01 | |
| *** busterswt has quit IRC | 23:02 | |
| *** andrearo1 has joined #openstack-nova | 23:03 | |
| *** doug-fish has joined #openstack-nova | 23:03 | |
| *** ctrath has quit IRC | 23:03 | |
| *** ekhugen_alt has quit IRC | 23:03 | |
| *** kmartin_ has joined #openstack-nova | 23:03 | |
| *** ekhugen has joined #openstack-nova | 23:03 | |
| *** andrearosa has quit IRC | 23:04 | |
| *** asselin_ has quit IRC | 23:04 | |
| *** asselin has joined #openstack-nova | 23:04 | |
| *** ccarmack has joined #openstack-nova | 23:04 | |
| *** nelsnelson has quit IRC | 23:04 | |
| *** vladikr has quit IRC | 23:04 | |
| *** kmartin has quit IRC | 23:04 | |
| *** mordred has quit IRC | 23:04 | |
| *** mordred has joined #openstack-nova | 23:05 | |
| *** mragupat_ has quit IRC | 23:05 | |
| *** zhiyan has quit IRC | 23:06 | |
| *** irenab has joined #openstack-nova | 23:06 | |
| *** mdrabe has quit IRC | 23:07 | |
| *** e0ne has quit IRC | 23:07 | |
| *** dims has quit IRC | 23:07 | |
| *** ctrath has joined #openstack-nova | 23:08 | |
| *** doug-fish has quit IRC | 23:08 | |
| *** zhiyan has joined #openstack-nova | 23:08 | |
| *** ccarmack has quit IRC | 23:10 | |
| *** achanda has quit IRC | 23:11 | |
| *** johnthetubaguy has quit IRC | 23:13 | |
| *** rushiagr has quit IRC | 23:13 | |
| *** pratikmallya has quit IRC | 23:16 | |
| *** johnthetubaguy has joined #openstack-nova | 23:16 | |
| *** su_zhang has quit IRC | 23:16 | |
| *** burt1 has quit IRC | 23:17 | |
| *** rushiagr has joined #openstack-nova | 23:19 | |
| *** su_zhang has joined #openstack-nova | 23:21 | |
| *** ivase has quit IRC | 23:23 | |
| *** ivase has joined #openstack-nova | 23:24 | |
| *** omnipresent has quit IRC | 23:25 | |
| *** zenoway has joined #openstack-nova | 23:26 | |
| *** eil397 has joined #openstack-nova | 23:26 | |
| *** hblixt has quit IRC | 23:27 | |
| tjones | anyone here a hyper-v person? | 23:27 |
| notmorgan | hm. i probably need to get here earlier in the day to bug the cores so i can get something off my plate. | 23:28 |
| * notmorgan puts on a todo for tomorrow | 23:28 | |
| *** zenoway has quit IRC | 23:30 | |
| openstackgerrit | Cale Rath proposed openstack/nova: Purge soft-deleted instances cmd https://review.openstack.org/203751 | 23:31 |
| *** cdelatte has quit IRC | 23:32 | |
| *** dims has joined #openstack-nova | 23:32 | |
| *** jaypipes has quit IRC | 23:32 | |
| *** dave-mccowan has quit IRC | 23:33 | |
| *** xyang1 has quit IRC | 23:37 | |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: make snapshot use RBD snapshot/clone when available https://review.openstack.org/205282 | 23:38 |
| *** ajayaa has quit IRC | 23:39 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Fix indents of servers-detail-resp.json https://review.openstack.org/270536 | 23:39 |
| oomichi | tojuvone: can you take a look at https://review.openstack.org/270536 ? | 23:40 |
| *** ducttape_ has quit IRC | 23:40 | |
| *** achanda has joined #openstack-nova | 23:40 | |
| *** irenab has quit IRC | 23:42 | |
| *** ivase has quit IRC | 23:46 | |
| openstackgerrit | Thelo Gaultier proposed openstack/nova: Add ITRI DISCO os-brick connector for libvirt https://review.openstack.org/253353 | 23:46 |
| oomichi | alex_xu: morning | 23:47 |
| oomichi | alex_xu: on your https://review.openstack.org/#/c/264045/ , it is possible to approve it after fixing kevin's comments. | 23:47 |
| *** aysyd has quit IRC | 23:47 | |
| *** mriedem has joined #openstack-nova | 23:47 | |
| *** Sukhdev has joined #openstack-nova | 23:48 | |
| tjones | i have a patch with 2 +2 and it is stuck on hyper-v tests that have nothing to do with the patch (unit test patch). any one know how i can get out of this stuck spot? https://review.openstack.org/#/c/266060/ | 23:54 |
| *** jacky-zhang has quit IRC | 23:54 | |
| *** mrkz has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
