*** ducttape_ has quit IRC | 00:06 | |
*** ducttape_ has joined #openstack-nova | 00:09 | |
*** gmann_ has joined #openstack-nova | 00:24 | |
*** stewie925_ has joined #openstack-nova | 00:30 | |
*** stewie925_ is now known as stewie925 | 00:30 | |
*** thorst_ has joined #openstack-nova | 00:38 | |
*** yamahata has joined #openstack-nova | 00:40 | |
*** thorst_ has quit IRC | 00:43 | |
*** ducttape_ has quit IRC | 00:44 | |
openstackgerrit | Huan Xie proposed openstack/nova: [DO NOT MERGE] Use os-xenapi repo https://review.openstack.org/393103 | 00:45 |
---|---|---|
*** stewie925 has quit IRC | 00:53 | |
*** tuanluong has joined #openstack-nova | 00:56 | |
*** ducttape_ has joined #openstack-nova | 01:00 | |
*** tovin07 has joined #openstack-nova | 01:00 | |
*** gmann_ has quit IRC | 01:06 | |
openstackgerrit | Rong Han proposed openstack/nova: migration.source_compute should be unchanged after finish_revert_resize. https://review.openstack.org/335270 | 01:08 |
*** ducttape_ has quit IRC | 01:13 | |
*** hoangcx has joined #openstack-nova | 01:18 | |
*** amotoki has joined #openstack-nova | 01:25 | |
*** hoangcx has quit IRC | 01:32 | |
*** Daisy has joined #openstack-nova | 01:32 | |
*** amotoki has quit IRC | 01:34 | |
*** hoangcx has joined #openstack-nova | 01:34 | |
*** amotoki has joined #openstack-nova | 01:35 | |
*** Daisy has quit IRC | 01:37 | |
*** Daisy has joined #openstack-nova | 01:37 | |
*** amotoki has quit IRC | 01:47 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Send out notification about server group changes when delete instances https://review.openstack.org/393649 | 01:47 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Remove mox in tests/unit/objects/test_security_group https://review.openstack.org/307135 | 01:47 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Remove mox in tests/unit/objects/test_instance_info_cache https://review.openstack.org/307155 | 01:47 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Remove mox in tests/unit/objects/test_pci_device https://review.openstack.org/307615 | 01:48 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Remove mox in tests/unit/objects/test_compute_node https://review.openstack.org/307625 | 01:49 |
*** amotoki has joined #openstack-nova | 01:51 | |
*** Daisy has quit IRC | 01:52 | |
*** Daisy has joined #openstack-nova | 01:53 | |
*** amotoki has quit IRC | 01:53 | |
*** amotoki has joined #openstack-nova | 01:55 | |
*** trisq has joined #openstack-nova | 01:57 | |
*** Daisy has quit IRC | 01:58 | |
*** annp has joined #openstack-nova | 02:00 | |
*** fabian4 has joined #openstack-nova | 02:00 | |
*** amotoki has quit IRC | 02:03 | |
*** Daisy has joined #openstack-nova | 02:05 | |
*** sdake has quit IRC | 02:08 | |
*** tovin07 is now known as tovin07_ | 02:08 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/python-novaclient: Replace oslo_utils.timeutils.isotime https://review.openstack.org/394228 | 02:09 |
*** dikonoo has joined #openstack-nova | 02:09 | |
*** tovin07_ has quit IRC | 02:10 | |
*** Daisy has quit IRC | 02:10 | |
*** Daisy has joined #openstack-nova | 02:11 | |
*** tovin07_ has joined #openstack-nova | 02:11 | |
*** sdake has joined #openstack-nova | 02:12 | |
*** thorst_ has joined #openstack-nova | 02:12 | |
*** gongysh has joined #openstack-nova | 02:12 | |
*** tovin07_ has quit IRC | 02:13 | |
*** tovin07__ has joined #openstack-nova | 02:13 | |
*** tovin07__ has quit IRC | 02:13 | |
*** tovin07_ has joined #openstack-nova | 02:14 | |
*** tovin07_ has quit IRC | 02:15 | |
*** tovin07_ has joined #openstack-nova | 02:16 | |
*** Daisy has quit IRC | 02:16 | |
*** tovin07_ is now known as tovin07 | 02:16 | |
*** thorst_ has quit IRC | 02:17 | |
*** tovin07 is now known as tovin07_ | 02:17 | |
*** Daisy has joined #openstack-nova | 02:17 | |
*** thorst_ has joined #openstack-nova | 02:17 | |
*** tovin07_ is now known as tovin07 | 02:18 | |
*** tovin07 has quit IRC | 02:18 | |
*** tovin07 has joined #openstack-nova | 02:18 | |
*** Daisy has quit IRC | 02:22 | |
*** shuquan has joined #openstack-nova | 02:22 | |
*** amotoki has joined #openstack-nova | 02:23 | |
*** Daisy has joined #openstack-nova | 02:23 | |
*** baoli has quit IRC | 02:25 | |
*** baoli has joined #openstack-nova | 02:26 | |
*** trananhkma has joined #openstack-nova | 02:27 | |
*** thorst_ has quit IRC | 02:28 | |
*** baoli has quit IRC | 02:28 | |
*** Daisy has quit IRC | 02:28 | |
*** baoli has joined #openstack-nova | 02:28 | |
*** thorst_ has joined #openstack-nova | 02:28 | |
*** armax has quit IRC | 02:33 | |
*** Daisy has joined #openstack-nova | 02:35 | |
*** baoli has quit IRC | 02:37 | |
*** thorst_ has quit IRC | 02:37 | |
*** baoli has joined #openstack-nova | 02:39 | |
*** Daisy has quit IRC | 02:40 | |
*** Daisy has joined #openstack-nova | 02:42 | |
*** Daisy has quit IRC | 02:46 | |
openstackgerrit | Xuanzhou Perry Dong proposed openstack/nova: Avoid multiple initializations of Host class https://review.openstack.org/339341 | 02:46 |
*** Daisy has joined #openstack-nova | 02:48 | |
*** armax has joined #openstack-nova | 02:49 | |
*** Daisy has quit IRC | 02:52 | |
*** Daisy has joined #openstack-nova | 02:55 | |
*** armax has quit IRC | 02:55 | |
*** Daisy has quit IRC | 02:56 | |
*** Daisy has joined #openstack-nova | 02:56 | |
*** bkopilov has quit IRC | 02:57 | |
*** baoli has quit IRC | 03:01 | |
*** shuquan__ has joined #openstack-nova | 03:06 | |
*** shuquan has quit IRC | 03:07 | |
*** dave-mccowan has quit IRC | 03:09 | |
*** ducttape_ has joined #openstack-nova | 03:14 | |
*** gcb has joined #openstack-nova | 03:18 | |
*** ducttape_ has quit IRC | 03:19 | |
*** namnh has joined #openstack-nova | 03:19 | |
*** eandersson_ has joined #openstack-nova | 03:22 | |
*** eandersson has quit IRC | 03:24 | |
*** thorst_ has joined #openstack-nova | 03:26 | |
*** dikonoo has quit IRC | 03:26 | |
*** dikonoo has joined #openstack-nova | 03:27 | |
*** thorst_ has quit IRC | 03:28 | |
*** thorst_ has joined #openstack-nova | 03:28 | |
*** amotoki has quit IRC | 03:30 | |
*** gongysh has quit IRC | 03:36 | |
*** thorst_ has quit IRC | 03:37 | |
*** amotoki has joined #openstack-nova | 03:40 | |
openstackgerrit | Duong Ha-Quang proposed openstack/nova: Remove mox in nova/tests/unit/compute/test_shelve.py (2) https://review.openstack.org/342593 | 03:54 |
*** links has joined #openstack-nova | 03:55 | |
*** Daisy has quit IRC | 03:56 | |
*** trananhkma has quit IRC | 03:57 | |
*** coolsvap has joined #openstack-nova | 04:10 | |
*** cfriesen__ has quit IRC | 04:10 | |
*** cfriesen has joined #openstack-nova | 04:10 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova: Add functional tests for cold migration with target https://review.openstack.org/368557 | 04:11 |
*** klkumar has joined #openstack-nova | 04:12 | |
*** trananhkma has joined #openstack-nova | 04:15 | |
*** bks has joined #openstack-nova | 04:17 | |
*** bkopilov has joined #openstack-nova | 04:17 | |
*** sdake has quit IRC | 04:21 | |
*** ayoung has quit IRC | 04:25 | |
*** baoli has joined #openstack-nova | 04:25 | |
*** baoli has quit IRC | 04:30 | |
*** raunak has joined #openstack-nova | 04:33 | |
*** thorst_ has joined #openstack-nova | 04:35 | |
*** bks has quit IRC | 04:38 | |
*** raunak has quit IRC | 04:40 | |
*** thorst_ has quit IRC | 04:42 | |
*** prateek has joined #openstack-nova | 04:54 | |
*** prateek has quit IRC | 04:54 | |
*** Daisy has joined #openstack-nova | 04:57 | |
*** Daisy has quit IRC | 05:01 | |
*** Daisy has joined #openstack-nova | 05:02 | |
*** GB21 has joined #openstack-nova | 05:03 | |
*** sudipto has joined #openstack-nova | 05:05 | |
*** sudipto_ has joined #openstack-nova | 05:05 | |
*** klkumar has quit IRC | 05:11 | |
*** klkumar has joined #openstack-nova | 05:15 | |
*** ducttape_ has joined #openstack-nova | 05:15 | |
*** hshiina has joined #openstack-nova | 05:17 | |
*** ducttape_ has quit IRC | 05:20 | |
*** manasm has joined #openstack-nova | 05:20 | |
*** Jeffrey4l has joined #openstack-nova | 05:21 | |
*** Daisy has quit IRC | 05:25 | |
*** GB21 has quit IRC | 05:25 | |
*** Daisy has joined #openstack-nova | 05:28 | |
*** fandi has joined #openstack-nova | 05:28 | |
*** sahid has joined #openstack-nova | 05:33 | |
*** GB21 has joined #openstack-nova | 05:37 | |
*** thorst_ has joined #openstack-nova | 05:40 | |
*** sridharg has joined #openstack-nova | 05:40 | |
*** shuquan__ has quit IRC | 05:47 | |
*** pbandark has joined #openstack-nova | 05:47 | |
*** thorst_ has quit IRC | 05:47 | |
*** dikonoor has joined #openstack-nova | 05:48 | |
*** dikonoo has quit IRC | 05:48 | |
*** shuquan has joined #openstack-nova | 05:49 | |
*** sudswas__ has joined #openstack-nova | 05:52 | |
*** armax has joined #openstack-nova | 05:53 | |
*** ratailor has joined #openstack-nova | 05:54 | |
*** sudipto_ has quit IRC | 05:55 | |
*** sudipto has quit IRC | 05:55 | |
*** sudipto has joined #openstack-nova | 05:56 | |
*** sandanar has joined #openstack-nova | 05:58 | |
*** armax has quit IRC | 05:58 | |
*** rcernin has joined #openstack-nova | 06:03 | |
*** amotoki has quit IRC | 06:03 | |
*** ecelik has joined #openstack-nova | 06:04 | |
*** ecelik has left #openstack-nova | 06:04 | |
*** duonghq has joined #openstack-nova | 06:10 | |
*** sahid has quit IRC | 06:11 | |
duonghq | hi all, anybody can review my patchset: https://review.openstack.org/#/c/342043/ | 06:12 |
*** Daisy has quit IRC | 06:15 | |
*** Daisy has joined #openstack-nova | 06:17 | |
*** lpetrut has joined #openstack-nova | 06:25 | |
*** gmann_ has joined #openstack-nova | 06:35 | |
*** nherciu has joined #openstack-nova | 06:36 | |
*** ekuris has joined #openstack-nova | 06:37 | |
*** Daisy has quit IRC | 06:38 | |
*** rdo has quit IRC | 06:39 | |
*** Daisy has joined #openstack-nova | 06:39 | |
*** dimtruck is now known as zz_dimtruck | 06:46 | |
*** thorst_ has joined #openstack-nova | 06:46 | |
*** prashkre has joined #openstack-nova | 06:48 | |
openstackgerrit | Xuanzhou Perry Dong proposed openstack/nova: Avoid multiple initializations of Host class https://review.openstack.org/339341 | 06:49 |
*** thorst_ has quit IRC | 06:53 | |
*** amotoki has joined #openstack-nova | 06:53 | |
*** armax has joined #openstack-nova | 06:55 | |
*** klkumar has quit IRC | 06:56 | |
*** kaisers1 has quit IRC | 06:56 | |
*** klkumar has joined #openstack-nova | 06:56 | |
*** armax has quit IRC | 07:00 | |
*** namnh_ has joined #openstack-nova | 07:00 | |
*** namnh has quit IRC | 07:02 | |
*** shuquan__ has joined #openstack-nova | 07:04 | |
*** shuquan has quit IRC | 07:04 | |
*** tesseract has joined #openstack-nova | 07:06 | |
*** tesseract is now known as Guest71310 | 07:06 | |
*** amotoki has quit IRC | 07:08 | |
*** gongysh has joined #openstack-nova | 07:12 | |
*** rdo has joined #openstack-nova | 07:14 | |
*** ducttape_ has joined #openstack-nova | 07:16 | |
*** pcaruana has joined #openstack-nova | 07:17 | |
*** ljxiash has joined #openstack-nova | 07:17 | |
*** Jeffrey4l has quit IRC | 07:18 | |
*** Oku_OS-away is now known as Oku_OS | 07:19 | |
*** ducttape_ has quit IRC | 07:20 | |
*** amotoki has joined #openstack-nova | 07:21 | |
*** amotoki has quit IRC | 07:22 | |
*** amotoki has joined #openstack-nova | 07:22 | |
*** andreas_s has joined #openstack-nova | 07:25 | |
*** gmann__ has joined #openstack-nova | 07:27 | |
*** gmann_ has quit IRC | 07:29 | |
*** avolkov has quit IRC | 07:30 | |
*** khamtamtun has joined #openstack-nova | 07:41 | |
*** lpetrut has quit IRC | 07:42 | |
*** khamtamtun has quit IRC | 07:43 | |
*** gszasz has joined #openstack-nova | 07:43 | |
*** klkumar has quit IRC | 07:45 | |
*** GB21 has quit IRC | 07:45 | |
*** belmoreira has joined #openstack-nova | 07:46 | |
*** manasm has quit IRC | 07:47 | |
*** thorst_ has joined #openstack-nova | 07:50 | |
*** lpetrut has joined #openstack-nova | 07:51 | |
*** moshele has joined #openstack-nova | 07:51 | |
*** manasm has joined #openstack-nova | 07:54 | |
*** armax has joined #openstack-nova | 07:56 | |
*** thorst_ has quit IRC | 07:57 | |
*** markus_z has joined #openstack-nova | 07:59 | |
*** lpetrut has quit IRC | 08:00 | |
*** armax has quit IRC | 08:00 | |
*** garyk1 has joined #openstack-nova | 08:01 | |
*** ociuhandu has joined #openstack-nova | 08:09 | |
*** gmann__ has quit IRC | 08:13 | |
*** Jeffrey4l has joined #openstack-nova | 08:17 | |
*** baoli has joined #openstack-nova | 08:18 | |
*** xinliang has quit IRC | 08:19 | |
*** matrohon has joined #openstack-nova | 08:22 | |
*** baoli has quit IRC | 08:23 | |
*** shuquan_1 has joined #openstack-nova | 08:24 | |
*** shuquan__ has quit IRC | 08:24 | |
openstackgerrit | Andrey Volkov proposed openstack/nova: PoC: add hierarchy quota driver https://review.openstack.org/391072 | 08:27 |
*** namnh has joined #openstack-nova | 08:29 | |
*** shaohe_feng has quit IRC | 08:29 | |
*** shaohe_feng has joined #openstack-nova | 08:30 | |
*** alex_xu has quit IRC | 08:30 | |
*** liverpooler has joined #openstack-nova | 08:31 | |
*** mkoderer has joined #openstack-nova | 08:31 | |
*** namnh_ has quit IRC | 08:32 | |
*** xinliang has joined #openstack-nova | 08:33 | |
*** betherly has joined #openstack-nova | 08:35 | |
*** alex_xu has joined #openstack-nova | 08:37 | |
*** oanson has joined #openstack-nova | 08:41 | |
*** nherciu has quit IRC | 08:46 | |
-openstackstatus- NOTICE: Gerrit is going to be restarted due to slowness and proxy errors | 08:46 | |
*** openstackgerrit has quit IRC | 08:48 | |
*** openstackgerrit has joined #openstack-nova | 08:48 | |
*** GB21 has joined #openstack-nova | 08:53 | |
*** thorst_ has joined #openstack-nova | 08:55 | |
*** klkumar has joined #openstack-nova | 08:56 | |
*** armax has joined #openstack-nova | 08:57 | |
*** danpb has joined #openstack-nova | 08:58 | |
*** ihrachys has joined #openstack-nova | 08:58 | |
*** zenoway has joined #openstack-nova | 08:59 | |
*** zzzeek has quit IRC | 09:00 | |
*** zzzeek has joined #openstack-nova | 09:00 | |
*** ljxiash has quit IRC | 09:01 | |
*** ljxiash has joined #openstack-nova | 09:01 | |
*** armax has quit IRC | 09:02 | |
*** thorst_ has quit IRC | 09:02 | |
*** roeyc has joined #openstack-nova | 09:05 | |
*** ljxiash has quit IRC | 09:06 | |
openstackgerrit | Jakub Pavlik proposed openstack/nova: Fix qemu-img convert image incompatability in alpine linux https://review.openstack.org/387485 | 09:07 |
moshele | danpb: hi, I updated User-controlled SR-IOV ports spec allocation the https://review.openstack.org/#/c/182242/ according to your comments | 09:07 |
*** pilgrimstack has joined #openstack-nova | 09:11 | |
*** Daisy has quit IRC | 09:13 | |
openstackgerrit | Esha Seth proposed openstack/nova: Add a no-op wait method to NetworkInfo https://review.openstack.org/393669 | 09:14 |
fabian4 | Hi, i have a question, nova delete chinise name apeer 'ascii' codec can't encode characters in position 0-4: ordinal not in range(128) , how can i change it? | 09:15 |
*** ralonsoh has joined #openstack-nova | 09:15 | |
openstackgerrit | Choe, Cheng-Dae proposed openstack/nova: Query availability zone in a single query https://review.openstack.org/389759 | 09:16 |
*** ducttape_ has joined #openstack-nova | 09:17 | |
*** lucas-afk is now known as lucasagomes | 09:18 | |
*** narayrak has joined #openstack-nova | 09:21 | |
*** ducttape_ has quit IRC | 09:22 | |
*** narayrak has quit IRC | 09:23 | |
*** sree has joined #openstack-nova | 09:24 | |
*** narayrak has joined #openstack-nova | 09:24 | |
*** BobBall_AWOL is now known as BobBall | 09:25 | |
*** Oku_OS is now known as Oku_OS-away | 09:25 | |
*** yassine has joined #openstack-nova | 09:29 | |
*** karthiks has joined #openstack-nova | 09:29 | |
*** yassine is now known as Guest90996 | 09:29 | |
*** mnestratov has joined #openstack-nova | 09:30 | |
*** markvoelker has quit IRC | 09:32 | |
*** xek has joined #openstack-nova | 09:34 | |
*** ihrachys has quit IRC | 09:36 | |
openstackgerrit | Andrey Volkov proposed openstack/nova: PoC: add hierarchy quota driver https://review.openstack.org/391072 | 09:36 |
*** lpetrut has joined #openstack-nova | 09:37 | |
*** mgoddard has joined #openstack-nova | 09:39 | |
*** Murali has joined #openstack-nova | 09:40 | |
Murali | We are facing a issue with openstack setup | 09:41 |
Murali | unable to launch the instance | 09:41 |
Murali | Someone have a look at this: https://ask.openstack.org/en/question/98662/connection-to-libvirt-failed/ | 09:41 |
*** Oku_OS-away is now known as Oku_OS | 09:43 | |
*** betherly has quit IRC | 09:44 | |
*** derekh has joined #openstack-nova | 09:44 | |
*** tovin07 has quit IRC | 09:51 | |
*** sdake has joined #openstack-nova | 09:55 | |
Murali | Any help or idea guys? | 09:56 |
*** armax has joined #openstack-nova | 09:57 | |
*** thorst_ has joined #openstack-nova | 10:00 | |
*** sdake_ has joined #openstack-nova | 10:00 | |
*** brault has quit IRC | 10:01 | |
*** brault has joined #openstack-nova | 10:02 | |
*** armax has quit IRC | 10:02 | |
*** sdake has quit IRC | 10:03 | |
*** roeyc has quit IRC | 10:03 | |
*** bauwser is now known as bauzas | 10:03 | |
*** sahid has joined #openstack-nova | 10:03 | |
*** thorst_ has quit IRC | 10:07 | |
*** annp has quit IRC | 10:11 | |
*** yamahata has quit IRC | 10:11 | |
*** sambetts|afk is now known as sambetts | 10:12 | |
*** gongysh has quit IRC | 10:13 | |
*** hoangcx has quit IRC | 10:15 | |
*** al_indigo has quit IRC | 10:16 | |
*** duonghq has quit IRC | 10:18 | |
*** sree has quit IRC | 10:18 | |
*** paul-carlton2 has joined #openstack-nova | 10:24 | |
*** sdake_ is now known as sdake | 10:25 | |
*** eglynn has joined #openstack-nova | 10:29 | |
*** avolkov has joined #openstack-nova | 10:30 | |
*** ecelik has joined #openstack-nova | 10:30 | |
*** ecelik has left #openstack-nova | 10:31 | |
tdurakov | wznoinsk: hi, are you around? | 10:34 |
*** takashin has joined #openstack-nova | 10:35 | |
*** jaosorior has joined #openstack-nova | 10:38 | |
*** ihrachys has joined #openstack-nova | 10:39 | |
*** cdent has joined #openstack-nova | 10:41 | |
*** tuanluong has quit IRC | 10:42 | |
*** bks has joined #openstack-nova | 10:42 | |
openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Call host connection callbacks asynchronously https://review.openstack.org/393869 | 10:46 |
wznoinsk | tdurakov, hi | 10:49 |
tdurakov | wznoinsk: during the previous l-m meeting we've discussed that you will start ml thread about ci and volunteers to move tests to upstream tempest | 10:51 |
tdurakov | wznoinsk: could you please send a mail to openstack-dev today? | 10:51 |
wznoinsk | tdurakov, I had a quick chat with sfinucan, I need to follow up on that with him | 10:51 |
tdurakov | is he planning to work on that? | 10:52 |
tdurakov | wznoinsk: ^ | 10:52 |
wznoinsk | tdurakov, he's interested I didn't ask for details yet, sfinucan ping? | 10:52 |
sfinucan | wznoinsk, tdurakov: That depends on what you're planning to do :) | 10:53 |
wznoinsk | sfinucan, I guess make sure cpu pinning/numa/hugepages tests we have in intel-nfv-ci-tests repo upstream tempest compliant and have them running by gate | 10:54 |
sfinucan | wznoinsk: Ok, but how do you plan to do that without the relevant hardware | 10:55 |
wznoinsk | numa probably not easy but is CPU pinning not possible on any VM from any clouds I wonder? | 10:55 |
wznoinsk | sfinucan, ^ | 10:56 |
*** rfolco has joined #openstack-nova | 10:56 | |
wznoinsk | obviously that would cover one numa tests only, not sure that's enough either | 10:57 |
tdurakov | wznoinsk, sfinucan, I'd expect first we need to move tests from github ci repo to upstream tempest? | 10:57 |
sfinucan | wznoinsk: the biggest issue I know of is the lack of nested virt. afaik, this isn't an option for Rackspace-provided infra | 10:57 |
sfinucan | tdurakov: We can't move them into the upstream tempest if they can't be run in infra. There was talk about feature flags in upstream Tempest a few cycles ago but it was dismissed | 10:58 |
sfinucan | that might not be the case still (only infra could say for sure) but it was the policy last time I checked | 10:59 |
openstackgerrit | Manas Mandlekar proposed openstack/nova: Don't destroy evacuated instance if host didn't change https://review.openstack.org/392620 | 10:59 |
tdurakov | sfinucan: I think we can, just need to add several params in tempest conf | 10:59 |
sfinucan | tdurakov: We should confirm that before doing anything else, I guess | 10:59 |
tdurakov | wznoinsk: link to ci tests on github please? | 11:00 |
wznoinsk | sfinucan, would you have time to submit a PS upstream for one of them (CPU pinning?) and see whether there's anything we can do to make it working? | 11:00 |
sfinucan | fwiw, the existing NFV tests are structured as a Tempest plugin so anyone should be able to run them as is | 11:00 |
wznoinsk | https://github.com/stackforge/intel-nfv-ci-tests | 11:00 |
tdurakov | sfinucan: the reason I'd prefer openstack-dev ml for that discussion | 11:00 |
sfinucan | tdurakov: github.com/openstack/intel-nfv-ci-tests | 11:00 |
tdurakov | thanks! | 11:01 |
*** pc_m has left #openstack-nova | 11:01 | |
tdurakov | wznoinsk, could you please start a thread anyway, so we could ask qa folks about moving that to tempest | 11:01 |
sfinucan | wznoinsk: Finding out if it's even an option (via the mailing list) would be a better move first, I'd say | 11:02 |
wznoinsk | sfinucan, having it as plugin is giving a good starting point, I think they may need a bit of tweaking so they're written the best possible way before they will be accepted into upstream tempest | 11:02 |
wznoinsk | roger that, will start ML thread today | 11:02 |
sfinucan | tdurakov wznoinsk: Out of curiosity, what's the rationale for putting them into upstream Tempest vs a plugin? | 11:02 |
tdurakov | sfinucan: will be reused by other folks, improve coverage | 11:03 |
sfinucan | tdurakov: and that can't be done via the current Tempest plugin structure? | 11:03 |
tdurakov | sfinucan: only procs from my point of view | 11:03 |
tdurakov | sfinucan: I'm not familiar with current tempest plugins structure | 11:04 |
sfinucan | Could I suggest an alternative | 11:04 |
sfinucan | We document a list of available Tempest plugins in Tempest, but keep them separate | 11:04 |
*** riccardo_ has joined #openstack-nova | 11:04 | |
tdurakov | need to drop now, could we continue in an hour? | 11:04 |
sfinucan | Sure | 11:04 |
wznoinsk | tdurakov, in short, you install a tmpest plugin via pip install git+http://githublinkhere and run tempest with -eall-plugin | 11:04 |
*** thorst_ has joined #openstack-nova | 11:05 | |
sfinucan | wznoinsk, tdurakov: http://docs.openstack.org/developer/tempest/plugin-registry.html | 11:07 |
sfinucan | already beaten to the punch, it seems | 11:07 |
manasm | @bauzas review https://review.openstack.org/#/c/392620/ has been updated, please take a look. | 11:07 |
*** erlon-airlong has joined #openstack-nova | 11:08 | |
*** gszasz has quit IRC | 11:12 | |
*** thorst_ has quit IRC | 11:12 | |
*** haplo37 has quit IRC | 11:13 | |
*** bks has quit IRC | 11:14 | |
*** haplo37 has joined #openstack-nova | 11:15 | |
openstackgerrit | Pawel Koniszewski proposed openstack/nova: live-mig: Add claims and proper resource tracking https://review.openstack.org/244489 | 11:15 |
openstackgerrit | sahid proposed openstack/nova-specs: Libvirt driver emulator threads placement policy https://review.openstack.org/284094 | 11:24 |
*** trisq has quit IRC | 11:28 | |
*** bhagyashris has quit IRC | 11:28 | |
*** takashin has quit IRC | 11:29 | |
*** markvoelker has joined #openstack-nova | 11:32 | |
*** ociuhandu has quit IRC | 11:33 | |
*** roeyc has joined #openstack-nova | 11:34 | |
openstackgerrit | Vasyl Saienko proposed openstack/nova-specs: Proposal for ironic static portgroups support https://review.openstack.org/387534 | 11:34 |
*** rwmjones has quit IRC | 11:35 | |
*** sudswas__ has quit IRC | 11:35 | |
*** sudipto has quit IRC | 11:35 | |
*** markvoelker has quit IRC | 11:38 | |
*** rwmjones has joined #openstack-nova | 11:38 | |
*** gszasz has joined #openstack-nova | 11:43 | |
*** tbachman has quit IRC | 11:45 | |
*** nherciu has joined #openstack-nova | 11:49 | |
*** takedakn has joined #openstack-nova | 11:55 | |
*** ociuhandu has joined #openstack-nova | 11:56 | |
tdurakov | sfinucan, wznoinsk I'm back, so plugin looks good either, it's ok for me to do it that way | 11:57 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Provide an object facade for a subset of RPs https://review.openstack.org/386242 | 11:58 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Expose a REST API method for a specific list of RPs https://review.openstack.org/392569 | 11:58 |
*** dencaval has quit IRC | 11:58 | |
*** dencaval has joined #openstack-nova | 11:58 | |
*** pradk has joined #openstack-nova | 11:59 | |
wznoinsk | tdurakov, sfinucan get these tests working as plugin so running intel-nfv-ci-tests is now as simple as these two commands I listed above, obviously other nova/other projects CIs would not be doing this - it would extend these epa features test coverage if these tests would be just there in tempest | 12:00 |
*** shuquan_1 has quit IRC | 12:03 | |
*** sandanar has quit IRC | 12:04 | |
sahid | johnthetubaguy: hello, i tried to address all of view questions, not sure if some of them needed actions or not, when you have a moment feel free to let me know if you need me to update the spec | 12:05 |
sahid | s/view/your | 12:05 |
*** shuquan_ has joined #openstack-nova | 12:05 | |
*** takedakn has quit IRC | 12:06 | |
tdurakov | wznoinsk: if we try to merge tests in tempest master instead, it will introduce new flags in tempest.conf, smth like `enable_nfv_testing`, which is almost the same as having a plugin for that(you need to enable/disable that per job basis) | 12:08 |
*** rtheis has joined #openstack-nova | 12:09 | |
openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Covert StrOpt -> PortOpt https://review.openstack.org/394392 | 12:09 |
openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Deprecate 'remap_vbd_dev' option https://review.openstack.org/394393 | 12:09 |
*** thorst_ has joined #openstack-nova | 12:10 | |
*** ratailor has quit IRC | 12:12 | |
*** shuquan_ has quit IRC | 12:15 | |
*** lucasagomes is now known as lucas-hungry | 12:15 | |
*** thorst_ has quit IRC | 12:17 | |
wznoinsk | tdurakov, good point, and that won't be set to 'true' by default as the features are mostly relevant to real time workloads - is my understanding on the right track? | 12:17 |
tdurakov | wznoinsk: yeah, disabled by default | 12:20 |
*** bkopilov has quit IRC | 12:21 | |
*** manasm has quit IRC | 12:23 | |
*** namnh has quit IRC | 12:27 | |
wznoinsk | tdurakov, would it be better to have it enabled but rely on the hsot capabilities (discovery) to run theses tests or not, similar to "SKIPPED: Less than 2 compute nodes, skipping multinode tests." | 12:29 |
tdurakov | wznoinsk: it will be difficult, especially if you have a large cloud | 12:30 |
*** riccardo_ has quit IRC | 12:30 | |
wznoinsk | hm, yes, unless you have these resource tracked | 12:31 |
wznoinsk | also, it would be tempest specific default 'enable_nfv_testing=true' configuration for the gate testing so no huge cloud tests there | 12:33 |
*** bvanhav_ has joined #openstack-nova | 12:33 | |
wznoinsk | I mean, CI testing (1st or 3rd party) is _usually_ not on clouds with many compute node setups | 12:33 |
wznoinsk | tdurakov, ^ | 12:34 |
*** raunak has joined #openstack-nova | 12:34 | |
tdurakov | wznoinsk: right, but tempest and it's plugins are not for ci only:) | 12:34 |
tdurakov | imo | 12:34 |
*** maticue has joined #openstack-nova | 12:35 | |
*** hoangcx has joined #openstack-nova | 12:39 | |
*** sdake has quit IRC | 12:40 | |
*** sdake has joined #openstack-nova | 12:40 | |
*** hoangcx has quit IRC | 12:43 | |
*** rcernin has quit IRC | 12:43 | |
*** rcernin has joined #openstack-nova | 12:44 | |
*** jaypipes has joined #openstack-nova | 12:46 | |
*** GB21 has quit IRC | 12:46 | |
*** Kevin_Zheng has quit IRC | 12:47 | |
*** Kevin_Zheng has joined #openstack-nova | 12:48 | |
*** thorst_ has joined #openstack-nova | 12:50 | |
*** sudipto has joined #openstack-nova | 12:50 | |
*** thorst_ has quit IRC | 12:50 | |
*** sudswas__ has joined #openstack-nova | 12:50 | |
*** thorst_ has joined #openstack-nova | 12:52 | |
sahid | jaypipes: do you have a moment to review that serie https://review.openstack.org/#/c/389687/ | 12:56 |
sahid | the work has been started long time ago | 12:56 |
sahid | it's basically to claim resources according the numa instance topoloy | 12:56 |
sahid | then to update the domain XML of the live migrated instance | 12:57 |
sahid | it's something necessary, really and we already have missed 2 releases :/ | 12:57 |
*** kylek3h has quit IRC | 12:57 | |
*** tblakes has joined #openstack-nova | 13:02 | |
*** aloga_ has joined #openstack-nova | 13:03 | |
*** eharney has quit IRC | 13:05 | |
*** tbachman has joined #openstack-nova | 13:05 | |
*** gcb has quit IRC | 13:07 | |
*** lamt has joined #openstack-nova | 13:07 | |
openstackgerrit | edan david proposed openstack/nova: Use available port binding constants https://review.openstack.org/387830 | 13:09 |
*** ducttape_ has joined #openstack-nova | 13:10 | |
*** tbachman_ has joined #openstack-nova | 13:11 | |
*** lamt has quit IRC | 13:11 | |
*** tbachman has quit IRC | 13:12 | |
*** tbachman_ is now known as tbachman | 13:12 | |
*** edmondsw has joined #openstack-nova | 13:17 | |
openstackgerrit | Andrey Volkov proposed openstack/nova-specs: WIP Add hierarchy quota driver specification https://review.openstack.org/394422 | 13:18 |
openstackgerrit | Andrey Volkov proposed openstack/nova-specs: WIP Add hierarchy quota driver specification https://review.openstack.org/394422 | 13:19 |
*** markvoelker has joined #openstack-nova | 13:21 | |
*** lucas-hungry is now known as lucasagomes | 13:21 | |
*** jheroux has joined #openstack-nova | 13:22 | |
*** ljxiash has joined #openstack-nova | 13:22 | |
openstackgerrit | Arne Recknagel proposed openstack/nova: Increase driver dict generation robustness https://review.openstack.org/394425 | 13:22 |
*** dave-mccowan has joined #openstack-nova | 13:23 | |
*** rodrigods has quit IRC | 13:25 | |
*** rodrigods has joined #openstack-nova | 13:25 | |
jaypipes | sahid: I will in about 30 minutes after meetings :) | 13:27 |
*** tlian has quit IRC | 13:29 | |
*** jwcroppe has quit IRC | 13:30 | |
*** kylek3h has joined #openstack-nova | 13:30 | |
*** kylek3h has quit IRC | 13:30 | |
*** jwcroppe has joined #openstack-nova | 13:30 | |
*** kylek3h has joined #openstack-nova | 13:30 | |
openstackgerrit | edan david proposed openstack/nova: User-controlled SR-IOV ports allocation https://review.openstack.org/374151 | 13:30 |
sahid | jaypipes: thanks a lot jay | 13:32 |
*** jwcroppe has quit IRC | 13:35 | |
*** ducttape_ has quit IRC | 13:36 | |
*** pradk has quit IRC | 13:37 | |
*** lamt has joined #openstack-nova | 13:38 | |
*** mvk has quit IRC | 13:38 | |
*** tlian has joined #openstack-nova | 13:39 | |
*** links has quit IRC | 13:40 | |
gibi_ | nova experts, is it a safe assumption that if a nova ovo has a remotable method then the body of that method will always run in the nova-conductor service?\ | 13:40 |
*** riccardo has joined #openstack-nova | 13:40 | |
gibi_ | or it is depends on the fact that the ovo is stored in the normal db or in the api_db? | 13:43 |
bauzas | gibi_: you mean the decorator ? | 13:46 |
*** mdrabe has joined #openstack-nova | 13:46 | |
bauzas | gibi_: if so, that depends on a conf | 13:46 |
*** danpb has quit IRC | 13:46 | |
bauzas | gibi_: I meant https://github.com/openstack/nova/blob/master/nova/cmd/compute.py#L53 | 13:47 |
*** sdake has quit IRC | 13:48 | |
gibi_ | bauzas: yes. for example the nova.objects.aggregate.Aggregate#create has @base.remotable decorator | 13:48 |
gibi_ | bauzas: and the question is where the code of the create runs | 13:48 |
bauzas | gibi_: where https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/base.py#L243 and https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/base.py#L172 | 13:48 |
*** pilgrimstack has quit IRC | 13:49 | |
bauzas | gibi_: to answer your question, yes, all the body will be run by the conductor in case the indirection API is set to it | 13:49 |
*** hshiina has quit IRC | 13:49 | |
openstackgerrit | edan david proposed openstack/nova: Use utils ANY constant https://review.openstack.org/387908 | 13:49 |
bauzas | gibi_: ie. only by the computes | 13:49 |
*** pilgrimstack has joined #openstack-nova | 13:50 | |
*** garyk1 has quit IRC | 13:50 | |
edleafe | Scheduler subteam meeting in 10 minutes in #openstack-meeting-alt | 13:50 |
*** takashin has joined #openstack-nova | 13:50 | |
bauzas | gibi_: https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L96 | 13:51 |
openstackgerrit | edan david proposed openstack/nova: Use utils ANY constant https://review.openstack.org/387908 | 13:51 |
gibi_ | bauzas: so if the nova-api service calls the Aggregate#create then the code will run in the nova-api service? | 13:52 |
bauzas | gibi_: yup | 13:52 |
*** bkopilov has joined #openstack-nova | 13:52 | |
bauzas | gibi_: https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/base.py#L1010-L1014 | 13:52 |
*** trisq has joined #openstack-nova | 13:53 | |
gibi_ | bauzas: thanks a lot! I see now. | 13:54 |
*** jwcroppe has joined #openstack-nova | 13:55 | |
bauzas | gibi_: tbc, the RPC call is made there https://github.com/openstack/nova/blob/master/nova/conductor/rpcapi.py#L230-L237 and the server-side is https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L91 | 13:56 |
*** claudiub has joined #openstack-nova | 13:56 | |
*** edand has joined #openstack-nova | 13:57 | |
bauzas | edleafe: due to daylight shift, I'll have to bail out early by every meeting :( | 13:57 |
bauzas | edleafe: around 1425UTC | 13:57 |
bauzas | edleafe: I need to pick my daughter at school :/ | 13:57 |
*** amotoki has quit IRC | 13:58 | |
edleafe | bauzas: ok. I almost missed the meeting because I had to drop my daughter off at school | 13:58 |
*** Guest71310 has quit IRC | 13:58 | |
bauzas | :) | 13:58 |
*** danpb has joined #openstack-nova | 13:58 | |
edleafe | bauzas: of course, she drives, but she doesn't have her own car :) | 13:58 |
bauzas | edleafe: well, mine is 6 so hopefully I'd have to wait a couple of more years than you :p | 13:59 |
edleafe | Yup | 13:59 |
*** brault has quit IRC | 13:59 | |
bauzas | my kids can be growing fast, but I'd be scared if they'd drive by 6 :p | 13:59 |
*** brault has joined #openstack-nova | 13:59 | |
*** narayrak has quit IRC | 14:00 | |
*** vladikr has joined #openstack-nova | 14:00 | |
*** shewless_ has quit IRC | 14:01 | |
*** baoli has joined #openstack-nova | 14:03 | |
*** oanson has quit IRC | 14:06 | |
*** fandi has quit IRC | 14:06 | |
*** cleong has joined #openstack-nova | 14:07 | |
*** shewless has quit IRC | 14:08 | |
*** bvanhav_ has quit IRC | 14:08 | |
*** amotoki has joined #openstack-nova | 14:09 | |
*** beekneemech is now known as bnemec | 14:10 | |
*** READ10 has joined #openstack-nova | 14:10 | |
*** READ10 has quit IRC | 14:10 | |
*** READ10 has joined #openstack-nova | 14:10 | |
*** mvk has joined #openstack-nova | 14:11 | |
*** Guest71310 has joined #openstack-nova | 14:12 | |
*** amotoki has quit IRC | 14:13 | |
*** ducttape_ has joined #openstack-nova | 14:13 | |
openstackgerrit | Andrey Volkov proposed openstack/nova-specs: WIP Add hierarchy quota driver specification https://review.openstack.org/394422 | 14:15 |
sambetts | Hi nova team, I have a question about your OSC plugin, is anyone involved with that around? | 14:16 |
ildikov | johnthetubaguy: hi | 14:17 |
*** ducttape_ has quit IRC | 14:18 | |
johnthetubaguy | ildikov: hey | 14:18 |
*** Guest71310 has quit IRC | 14:18 | |
ildikov | johnthetubaguy: I wanted to ask whether you're around for the today's Cinder-Nova meeting? | 14:18 |
*** klkumar has quit IRC | 14:19 | |
*** tesseract has joined #openstack-nova | 14:19 | |
johnthetubaguy | ildikov: I should be OK, 17:00 UTC right? | 14:19 |
sambetts | Is there an OSC equivolent for nova interface-attach/detach ?/ | 14:19 |
*** tesseract is now known as Guest41117 | 14:19 | |
johnthetubaguy | so an hour earlier than last time, for most folks | 14:19 |
ildikov | johnthetubaguy: I need to double check it with the other John too and I would like to try to turn it into a recorded Hangouts call | 14:20 |
johnthetubaguy | ildikov: that should help the discussion | 14:20 |
ildikov | johnthetubaguy: yeap, same UTC slot, which means an hour earlier due to lovely daylight saving changes | 14:20 |
*** paul-carlton2 has quit IRC | 14:20 | |
*** mriedem has joined #openstack-nova | 14:20 | |
ildikov | johnthetubaguy: I think by now the US folks switched too | 14:20 |
johnthetubaguy | ildikov: yep, I think its change time for both of us now | 14:21 |
*** mgoddard has quit IRC | 14:22 | |
*** Guest41117 has quit IRC | 14:22 | |
*** raunak has quit IRC | 14:24 | |
*** bvanhav_ has joined #openstack-nova | 14:25 | |
*** mgoddard has joined #openstack-nova | 14:27 | |
*** tesseract- has joined #openstack-nova | 14:27 | |
*** eharney has joined #openstack-nova | 14:28 | |
ildikov | johnthetubaguy: cool, I always loose track on this... | 14:30 |
*** ljxiash has quit IRC | 14:30 | |
*** prashkre has quit IRC | 14:30 | |
*** prashkre has joined #openstack-nova | 14:31 | |
johnthetubaguy | ildikov: I defaulted outlook to book my meetings in UTC, to make me choose! | 14:31 |
*** xyang1 has joined #openstack-nova | 14:32 | |
*** jaosorior has quit IRC | 14:32 | |
*** jaosorior has joined #openstack-nova | 14:32 | |
*** raunak has joined #openstack-nova | 14:34 | |
*** victorhe has joined #openstack-nova | 14:37 | |
*** mlavalle has joined #openstack-nova | 14:41 | |
ildikov | johnthetubaguy: haha, nice one! | 14:41 |
*** jamesdenton has joined #openstack-nova | 14:41 | |
*** raunak has quit IRC | 14:43 | |
*** sree has joined #openstack-nova | 14:43 | |
*** jlvill-travel is now known as jlvillal | 14:43 | |
*** riccardo has quit IRC | 14:46 | |
*** paul-carlton2 has joined #openstack-nova | 14:49 | |
*** paw has quit IRC | 14:49 | |
*** esberglu has joined #openstack-nova | 14:49 | |
*** trisq has quit IRC | 14:50 | |
*** trisq has joined #openstack-nova | 14:51 | |
openstackgerrit | Maciej Szankin proposed openstack/nova: conf: Move api options to a group https://review.openstack.org/384490 | 14:52 |
openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.create notification https://review.openstack.org/384621 | 14:53 |
openstackgerrit | Maciej Szankin proposed openstack/nova: conf: Move api options to a group https://review.openstack.org/384490 | 14:56 |
*** paul-carlton2 has quit IRC | 14:56 | |
*** ducttape_ has joined #openstack-nova | 14:59 | |
dansmith | mriedem: so, what should we do about this? https://review.openstack.org/#/c/393441 | 15:00 |
mriedem | b/c of cells v1? | 15:00 |
mriedem | or b/c there are no QA cores? | 15:00 |
mriedem | you just actually made me think that ^ probably shouldn't be completely unconditional | 15:00 |
*** sudswas__ has quit IRC | 15:01 | |
*** sudipto has quit IRC | 15:01 | |
openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Call host connection callbacks asynchronously https://review.openstack.org/393869 | 15:01 |
mriedem | i.e. we probably don't enable cells v2 if devstack is configured to setup cells v1 | 15:01 |
mriedem | i'm not sure what kind of wonkiness happens there | 15:01 |
openstackgerrit | Sergey Nikitin proposed openstack/nova: Modify fields of 'Diagnostics' object https://review.openstack.org/394480 | 15:01 |
mriedem | dansmith: i'm going to run the cells v1 job on https://review.openstack.org/#/c/392289/ quick | 15:02 |
mriedem | should be ok, but not really sure | 15:02 |
dansmith | mriedem: I meant in terms of waiting until sdague is back to opine or going forward | 15:02 |
*** mvk has quit IRC | 15:02 | |
dansmith | mriedem: we don't grenade on cellsv1 do we? | 15:02 |
mriedem | no | 15:02 |
dansmith | oh you meant for the devstack change? | 15:03 |
mriedem | right | 15:03 |
mriedem | cellsv1 is in the experimental queue for devstack so i need to run that job first | 15:03 |
mriedem | i think it should be ok, it'll just setup cell0 and the mappings which won't be used | 15:03 |
dansmith | okay, but we'll have to set it up for cellsv1 too else I can't merge that blocker migration | 15:03 |
mriedem | right | 15:03 |
mriedem | did you see my comments on the db migration change? | 15:03 |
jaypipes | cdent, bauzas: doh, sorry, I was hit by daylight savings :( Didn't realize sched meeting was at 9am not 10am EST. | 15:04 |
dansmith | no, but you said: <mriedem>i.e. we probably don't enable cells v2 if devstack is configured to setup cells v1 | 15:04 |
dansmith | mriedem: but I see you meant we probably don't, but should.. got it | 15:04 |
mriedem | double speak | 15:04 |
cdent | jaypipes: was okay, pretty laid back today, most getting people on same page | 15:04 |
mriedem | i was really saying, we maybe shouldn't enable cells v2 if configured for cells v1, but that won't work | 15:04 |
mriedem | as you noted | 15:04 |
mriedem | i also don't think it will hurt aything, but if it does it's a bug probably and we just have to handle it | 15:05 |
mriedem | b/c for some amount of time cells v1 people are going to be running both | 15:05 |
mriedem | and migrating | 15:05 |
mriedem | at least i think | 15:05 |
dansmith | right | 15:05 |
mriedem | why do i think about jesse ventura when i think about an http response body | 15:07 |
bauzas | jaypipes: no worries, I had the same issue with last week's nova meeting :) | 15:07 |
bauzas | I should propose to have all our meetings by the CET timezone :p | 15:08 |
*** nherciu has quit IRC | 15:08 | |
*** paul-carlton2 has joined #openstack-nova | 15:08 | |
*** jaosorior is now known as jaosorior_sick | 15:09 | |
*** danpb has quit IRC | 15:09 | |
*** amotoki has joined #openstack-nova | 15:10 | |
jaypipes | bauzas: any chance you or mriedem can +W this? https://review.openstack.org/#/c/386843/ You've both +2'd it. | 15:11 |
bauzas | jaypipes: like I said in a comment and during the meeting, I had a question for you before +Wing | 15:12 |
bauzas | jaypipes: not sure you saw it | 15:12 |
*** sridharg has quit IRC | 15:12 | |
*** danpb has joined #openstack-nova | 15:12 | |
bauzas | jaypipes: but like I said, it's something we can discuss later, just wanted to make sure you knew it before I was +Wing it | 15:13 |
openstackgerrit | Paul Carlton proposed openstack/nova: Enabling detach root volume for shelved_offloaded https://review.openstack.org/326899 | 15:13 |
openstackgerrit | Paul Carlton proposed openstack/nova: Attach/detach root volume for shelved_offloaded https://review.openstack.org/317446 | 15:13 |
openstackgerrit | Paul Carlton proposed openstack/nova: Adding device_present field to bdm table https://review.openstack.org/268053 | 15:13 |
bauzas | jaypipes: ie. https://review.openstack.org/#/c/386843/11/nova/objects/resource_provider.py@1048 | 15:13 |
openstackgerrit | Dan Smith proposed openstack/nova: Require cellsv2 setup before migrating to Ocata https://review.openstack.org/392227 | 15:15 |
mriedem | alex_xu: i'm writing a functional api samples test for addFloatingIp, it's getting pretty hairy though | 15:15 |
*** cdent has quit IRC | 15:15 | |
mriedem | more like a tempest test at this point | 15:15 |
*** gouthamr has joined #openstack-nova | 15:15 | |
*** sree has quit IRC | 15:15 | |
mriedem | btw, when we drop nova-network, our functional tests are going to be a mess to cleanup | 15:15 |
openstackgerrit | Roman Podoliaka proposed openstack/nova: tests: stub out libvirt in a fixture https://review.openstack.org/394487 | 15:16 |
*** mvk has joined #openstack-nova | 15:18 | |
*** paw has joined #openstack-nova | 15:18 | |
dansmith | yup | 15:19 |
*** oanson has joined #openstack-nova | 15:19 | |
mriedem | we do have the network service stub....but untangling that mess is going to be not fun | 15:20 |
*** roeyc has quit IRC | 15:21 | |
*** paul-carlton2 has quit IRC | 15:21 | |
*** manasm has joined #openstack-nova | 15:22 | |
dansmith | so last week, I learned that I have no idea how security groups work at boot if we're using neutron | 15:22 |
dansmith | they seem to disappear on one side of a tunnel and re-appear later | 15:23 |
dansmith | so, that's cool | 15:23 |
mriedem | we don't store them in the nova db right? | 15:23 |
mriedem | the security groups are tied to the ports on the instance | 15:24 |
mriedem | and each port has a default security group in neutron if port security is enabled | 15:24 |
*** pbandark has quit IRC | 15:24 | |
mriedem | meeting, gotta run | 15:24 |
dansmith | mriedem: yeah, but we don't alloc until we get to the compute node, so where we keep them between api and there is what I'm missing | 15:25 |
dansmith | alaski: did you say you were around this week? | 15:25 |
*** hongbin has joined #openstack-nova | 15:27 | |
*** mriedem1 has joined #openstack-nova | 15:28 | |
mriedem1 | dansmith: request sepc? | 15:28 |
mriedem1 | *spec | 15:28 |
dansmith | mriedem1: we don't use that anywhere in the compute/neutronapi bits that I see though | 15:28 |
mriedem1 | i thought the list of requested security groups just got passed from api to conductor, and from conductor to compute | 15:28 |
*** pradk has joined #openstack-nova | 15:28 | |
*** mriedem has quit IRC | 15:29 | |
dansmith | yeah, maybe it's just in an rpc arg | 15:29 |
*** trisq has quit IRC | 15:31 | |
*** trisq has joined #openstack-nova | 15:32 | |
dansmith | ah, I think I see.. we provision_security_groups() in the api, which overwrites our list, and set that on the instance. in neutron, that's an empty list | 15:32 |
dansmith | but our caller still has the list of names, and passes that along as an rpc arg | 15:32 |
*** aleph1 has joined #openstack-nova | 15:32 | |
dansmith | because we eclipse our argument in that inner _provision_instances() call | 15:33 |
dansmith | that means we're not storing the real list of requested secgroups in the ReqSpec either | 15:33 |
*** mriedem1 is now known as mriedem | 15:33 | |
*** yamahata has joined #openstack-nova | 15:33 | |
mriedem | we also lie in the response | 15:34 |
*** aleph1 is now known as agarner | 15:34 | |
mriedem | which i had a spec to change, but dropped it in favor of john's spec to redo how we handle security groups in the server create request | 15:34 |
*** raunak has joined #openstack-nova | 15:34 | |
mriedem | which we aren't going to do in ocata | 15:34 |
*** owalsh_ has joined #openstack-nova | 15:34 | |
*** links has joined #openstack-nova | 15:34 | |
mriedem | but if we need to store the security groups from the request in the request spec for cells v2 that's something we should be able to add | 15:35 |
dansmith | well, that will be hard to support for older computes | 15:35 |
dansmith | although it seems like the point of the ReqSpec is to be able to recreate the original request | 15:35 |
dansmith | and I don't think we can do that for neutron | 15:36 |
*** raunak has quit IRC | 15:36 | |
cfriesen | any chance of a core taking a look at https://review.openstack.org/#/c/346263/ ? dpb and mdbooth have both approved. | 15:36 |
*** knasim-wrs has joined #openstack-nova | 15:36 | |
*** paul-carlton2 has joined #openstack-nova | 15:37 | |
mriedem | cfriesen: so at the summit we talked a little bit about testing migration/resize with flavors that have ephemeral/swap devices, | 15:37 |
mriedem | did you catch any of that discussion? | 15:37 |
mriedem | diana_clarke had worked on a test that did something like that, but the test itself was creating the flavor, which the tempest folks didn't want, | 15:38 |
cfriesen | mriedem: no, think I missed that. | 15:38 |
mriedem | so at the summit we talked about changing the flavors that devstack sets in tempest.conf for use in the tests | 15:38 |
mriedem | so those flavors would have ephemeral/swap | 15:38 |
openstackgerrit | Lee Yarwood proposed openstack/nova: WIP encryptors: Introduce EncryptionProvider constants https://review.openstack.org/393901 | 15:38 |
*** owalsh has quit IRC | 15:38 | |
cfriesen | mriedem: right, that makes sense. | 15:39 |
mriedem | and then a test could check the flavor if it has ephemeral/swap and skip if it doesn't - assuming we had a specific test for that | 15:39 |
knasim-wrs | good day folks, my CI has been open for a while (https://review.openstack.org/#/c/300616/) and I had to rebase it a few times just to get all the voting CIs to come clean. Would any of the cores be able to look at this please? | 15:39 |
mriedem | i'm not sure if we need a specific test for it though, unless the test ssh's into the guest and verifies the disk was resized | 15:39 |
dansmith | mriedem: this is where we lose secgroups on the cellsv2 stack: https://review.openstack.org/#/c/367557/15/nova/conductor/api.py@133 | 15:40 |
*** zz_dimtruck is now known as dimtruck | 15:40 | |
diana_clarke | cfriesen: FYI - here's that tempest test mriedem mentioned: https://review.openstack.org/#/c/338411/ | 15:40 |
*** mlavalle has quit IRC | 15:40 | |
mriedem | dansmith: ok, so bauzas is asking that we just pass the request spec right? | 15:40 |
dansmith | mriedem: we are passing the reqspec, but it's not in there either | 15:41 |
*** rrecio has joined #openstack-nova | 15:41 | |
dansmith | mriedem: I think what bauzas is asking for is to pass an array of reqspecs, one per instance | 15:41 |
*** harlowja_at_home has joined #openstack-nova | 15:41 | |
*** erlon-airlong has quit IRC | 15:41 | |
*** serverascode has quit IRC | 15:41 | |
cfriesen | mriedem: okay, so are we looking to improve the testing before merging changes in this area? | 15:41 |
*** coolsvap has quit IRC | 15:42 | |
*** carl_baldwin has quit IRC | 15:42 | |
*** johnsom has quit IRC | 15:42 | |
*** auggy has quit IRC | 15:42 | |
*** vdrok has quit IRC | 15:42 | |
*** csatari has quit IRC | 15:42 | |
*** rmcadams has quit IRC | 15:42 | |
*** cargonza has quit IRC | 15:42 | |
*** nikhil has quit IRC | 15:42 | |
*** fmccrthy has quit IRC | 15:42 | |
*** dougwig has quit IRC | 15:42 | |
*** zhiyan has quit IRC | 15:42 | |
mriedem | cfriesen: well, i'm just personally not comfortable merging it right now w/o that, | 15:42 |
mriedem | i'd have to recreate the bug and pull the fix down to test it out myself | 15:42 |
*** Kevin_Zheng has quit IRC | 15:42 | |
*** zhenguo has quit IRC | 15:42 | |
*** raginbaj- is now known as raginbajin | 15:42 | |
*** mkoderer has quit IRC | 15:43 | |
*** scottda has quit IRC | 15:43 | |
*** nkrinner_afk has quit IRC | 15:43 | |
*** ameade has quit IRC | 15:43 | |
*** raorn has quit IRC | 15:43 | |
*** clif_h has quit IRC | 15:43 | |
*** andrewbogott has quit IRC | 15:43 | |
mriedem | cfriesen: because i asked a bunch of questions at one point in there and then there was much discussion and now there is another ask to look at it, and i haven't seen all of the discussion in between, | 15:43 |
mriedem | but before i asked all of those questions people were saying, 'hey this is ready to go, please look' | 15:43 |
*** fungi has quit IRC | 15:43 | |
*** mrhillsman has quit IRC | 15:43 | |
*** odyssey4me has quit IRC | 15:43 | |
*** FrankZhang has quit IRC | 15:43 | |
*** toan has quit IRC | 15:43 | |
mriedem | makes me a bit trigger shy | 15:43 |
cfriesen | mriedem: true enough. the bugfix itself is easy. making sure it doesn't break other stuff.... | 15:44 |
mriedem | dansmith: ok - and if we store the security groups in the request spec we should be good there right? | 15:44 |
dansmith | mriedem: yeah | 15:44 |
mriedem | cool | 15:44 |
dansmith | mriedem: you already have a patch or should I just do one up? should be not much trouble | 15:44 |
*** rrecio_ has joined #openstack-nova | 15:44 | |
*** mlavalle has joined #openstack-nova | 15:45 | |
*** zhiyan has joined #openstack-nova | 15:45 | |
*** raorn has joined #openstack-nova | 15:45 | |
*** Kevin_Zheng has joined #openstack-nova | 15:45 | |
*** scottda has joined #openstack-nova | 15:45 | |
cfriesen | mriedem: on the other hand, it's painful to have to maintain out-of-tree patches for issues that we know about, because a separate testing project isn't up to par. | 15:45 |
*** rmcadams has joined #openstack-nova | 15:45 | |
*** fmccrthy has joined #openstack-nova | 15:45 | |
*** auggy has joined #openstack-nova | 15:45 | |
*** nikhil has joined #openstack-nova | 15:45 | |
*** coolsvap has joined #openstack-nova | 15:45 | |
*** rcarrillocruz has quit IRC | 15:45 | |
*** dougwig has joined #openstack-nova | 15:45 | |
*** csatari has joined #openstack-nova | 15:45 | |
mriedem | dansmith: i don't | 15:45 |
dansmith | okay I'll cook it up | 15:46 |
*** rrecio has quit IRC | 15:46 | |
mriedem | cfriesen: well i don't expect the tempest / QA people to know we have this gap | 15:46 |
*** knasim-wrs has quit IRC | 15:46 | |
mriedem | it's on us as a nova team to fill that gap | 15:46 |
*** zhenguo has joined #openstack-nova | 15:46 | |
*** vdrok has joined #openstack-nova | 15:46 | |
*** johnsom has joined #openstack-nova | 15:46 | |
*** cargonza has joined #openstack-nova | 15:46 | |
*** andrewbogott has joined #openstack-nova | 15:46 | |
*** knasim-wrs has joined #openstack-nova | 15:46 | |
paw | Can a core look at https://review.openstack.org/#/c/388970/? | 15:46 |
*** toan has joined #openstack-nova | 15:47 | |
*** gcb has joined #openstack-nova | 15:47 | |
*** ameade has joined #openstack-nova | 15:47 | |
*** andrewbogott has quit IRC | 15:48 | |
*** andrewbogott has joined #openstack-nova | 15:48 | |
*** serverascode has joined #openstack-nova | 15:48 | |
*** carl_baldwin has joined #openstack-nova | 15:48 | |
*** odyssey4me has joined #openstack-nova | 15:48 | |
mriedem | pkoniszewski: where are you on ^ ? | 15:48 |
mriedem | pkoniszewski: do you need to see the bug fixed first? | 15:48 |
cfriesen | mriedem: mtreinish was recommending a functional test for the resize case (in the discussion around that tempest test). Would that be sufficient or do we really want a tempest test? | 15:49 |
*** FrankZhang has joined #openstack-nova | 15:49 | |
*** knasim-wrs has quit IRC | 15:49 | |
mriedem | cfriesen: we don't have a functional job that runs with an actual live libvirt | 15:49 |
mriedem | it's a thing we've talked about wanting, but needs people to work on it | 15:50 |
mriedem | something like a nova-dsvm-integration job | 15:50 |
cfriesen | mriedem: do we need actual live libvirt for resize though? that just tests ssh and scp | 15:50 |
cfriesen | mriedem: at least for the disk resizing case | 15:50 |
mriedem | cfriesen: if you can write a functional test that doesn't require live libvirt, then maybe we can use that | 15:50 |
openstackgerrit | prashkre proposed openstack/nova: Separate CRUD policy for server_groups https://review.openstack.org/391113 | 15:51 |
mriedem | there is a nova/tests/functional/libvirt set of testes | 15:51 |
mriedem | *tests | 15:51 |
*** rcarrillocruz has joined #openstack-nova | 15:52 | |
*** erlon-airlong has joined #openstack-nova | 15:52 | |
bauzas | dansmith: mriedem: sorry, was afk for family business | 15:52 |
bauzas | dansmith: mriedem: so I tend to think we should call the RPC by each instance, not passing a list | 15:53 |
*** dave-mccowan has quit IRC | 15:53 | |
cfriesen | mriedem: okay, thanks | 15:53 |
cfriesen | diana_clarke: did you ever propose the new flavors in devstack? | 15:53 |
bauzas | dansmith: mriedem: because I agree with alaski, we should maybe later discussing about deprecating the possibility to call the API for more than 1 instance | 15:53 |
dansmith | bauzas: why does that make sense? | 15:53 |
cfriesen | diana_clarke: or the modified flavors | 15:54 |
*** nkrinner_afk has joined #openstack-nova | 15:54 | |
*** rama_y has joined #openstack-nova | 15:54 | |
bauzas | dansmith: because I would appreciate to have the behaviour exactly like the current conductor, | 15:54 |
*** mrhillsman has joined #openstack-nova | 15:54 | |
bauzas | dansmith: ie. being cast'd for each instance | 15:54 |
*** clif_h has joined #openstack-nova | 15:54 | |
*** mkoderer has joined #openstack-nova | 15:54 | |
*** rcernin has quit IRC | 15:55 | |
bauzas | dansmith: and only the API being able to have a multiple instance method | 15:55 |
*** takashin has left #openstack-nova | 15:55 | |
*** fungi has joined #openstack-nova | 15:55 | |
dansmith | bauzas: we make one call to conductor today though, right? | 15:56 |
mriedem | cfriesen: i'll start a dev list thread to try and dump status on that so we can hopefully all get on the same page and maybe move forward | 15:56 |
mriedem | i'm a bit split brain atm | 15:56 |
bauzas | dansmith: nope | 15:56 |
dansmith | bauzas: build_instances() | 15:56 |
bauzas | dansmith: we make one cast per instance | 15:56 |
cfriesen | mriedem: sounds good | 15:56 |
dansmith | bauzas: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1167 | 15:56 |
dansmith | bauzas: takes a list of instances | 15:57 |
diana_clarke | cfriesen: no, I didn't because I didn't think they would get accepted | 15:57 |
bauzas | oh sec, you're right? | 15:57 |
*** hshiina has joined #openstack-nova | 15:57 | |
bauzas | wtf ? | 15:57 |
cfriesen | mriedem: would this one fall under the same sort of case (needing better test coverage first)? https://review.openstack.org/#/c/300616/ | 15:58 |
bauzas | I remember us casting each instance | 15:58 |
dansmith | bauzas: we cast one per instance to compute of course | 15:58 |
*** Sukhdev has joined #openstack-nova | 15:59 | |
dansmith | bauzas: making one per instance to the conductor will end up with a bunch of scheduling calls too, instead of one scheduling call with N instances, which is how the code is (and was) designed | 15:59 |
bauzas | dansmith: mmmm, sec, verifying something | 16:00 |
*** Kimmo__ has quit IRC | 16:00 | |
bauzas | dansmith: nevermind then my opinion | 16:01 |
dansmith | okay | 16:01 |
*** mgoddard_ has joined #openstack-nova | 16:01 | |
bauzas | dansmith: that said, we still need to pass a list of Spec objects IHMO | 16:02 |
*** zenoway has quit IRC | 16:02 | |
* bauzas reviewing fast the series | 16:02 | |
dansmith | okay, got a meeting now, but we can discuss after | 16:02 |
*** zenoway has joined #openstack-nova | 16:02 | |
mriedem | cfriesen: for me probably yes, or a manual recreate and verify of the fix | 16:04 |
mriedem | i can't in good conscience +2 something like that by just eyeballing it | 16:04 |
*** jdurgin1 has joined #openstack-nova | 16:04 | |
*** edand has quit IRC | 16:04 | |
*** mgoddard has quit IRC | 16:05 | |
cfriesen | mriedem: about what I expected given the other one. :) | 16:05 |
openstackgerrit | prashkre proposed openstack/nova: Separate CRUD policy for server_groups https://review.openstack.org/391113 | 16:05 |
*** yamahata has quit IRC | 16:06 | |
*** dave-mccowan has joined #openstack-nova | 16:06 | |
*** zenoway has quit IRC | 16:06 | |
*** owalsh_ is now known as owalsh | 16:08 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform aggregate.create notification https://review.openstack.org/394511 | 16:09 |
openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform aggregate.delete notification https://review.openstack.org/394512 | 16:09 |
*** tbachman has quit IRC | 16:09 | |
*** sree has joined #openstack-nova | 16:10 | |
*** hughsaunders has joined #openstack-nova | 16:10 | |
*** moshele has quit IRC | 16:11 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform aggregate.create notification https://review.openstack.org/394511 | 16:11 |
openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform aggregate.delete notification https://review.openstack.org/394512 | 16:11 |
*** prashkre has quit IRC | 16:11 | |
*** ducttape_ has quit IRC | 16:12 | |
*** pcaruana has quit IRC | 16:12 | |
*** sean-k-mooney has joined #openstack-nova | 16:14 | |
*** sree has quit IRC | 16:14 | |
*** sree has joined #openstack-nova | 16:14 | |
cfriesen | diana_clarke: for qcow images at least it shouldn't actually take up any more space to have swap/ephemeral, right? as long as nobody actually writes to them they'll be minimal size. | 16:18 |
*** sree has quit IRC | 16:19 | |
*** sree has joined #openstack-nova | 16:19 | |
*** sree has quit IRC | 16:19 | |
diana_clarke | cfriesen: I'm not 100% sure, but mdbooth understands this stuff way better than me. | 16:20 |
*** sree has joined #openstack-nova | 16:20 | |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add PCIWeigher https://review.openstack.org/379524 | 16:20 |
openstackgerrit | Stephen Finucane proposed openstack/nova: hardware: Flatten functions https://review.openstack.org/379624 | 16:20 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Prefer non-PCI host nodes for non-PCI instances https://review.openstack.org/379625 | 16:20 |
danpb | cfriesen: hmm, nova by defaults pre-allocates files, even qcow2 files | 16:20 |
*** manasm has quit IRC | 16:21 | |
thorst_ | mriedem mikal: Updated the powervm blueprint per your comments. When able, can you take another look? https://review.openstack.org/#/c/381772/ | 16:21 |
cfriesen | danpb: preallocate_images defaults to "none" | 16:22 |
*** manasm has joined #openstack-nova | 16:22 | |
mriedem | thorst_: yeah been meaning to get back to that | 16:22 |
kashyap | cfriesen: Indeed, I was just looking at code | 16:22 |
thorst_ | mriedem: thx. | 16:22 |
danpb | cfriesen: oh, really, that's probably a mistake in general :-) | 16:22 |
danpb | poor_performance=True out of the box :-( | 16:22 |
*** raunak has joined #openstack-nova | 16:22 | |
cfriesen | danpb: and I'm not sure it applies to anything that isn't an image (like swap/ephemeral) | 16:22 |
kashyap | 'preallocate_images', | 16:23 |
kashyap | default='none', | 16:23 |
kashyap | choices=('none', 'space'), | 16:23 |
mdbooth | danpb: It's not *that* poor any more, right? | 16:23 |
mdbooth | It used to be shocking (like 100x hit), but last time I tested it it wasn't completely awful. | 16:23 |
danpb | mdbooth: you get a I/O barrier any time a write causes a new sector to be allocated in qcow2 | 16:23 |
*** stewie925 has joined #openstack-nova | 16:23 | |
danpb | its not completely awful, but its certainly notably slower | 16:23 |
mdbooth | danpb: We hit this in v2v and had a workaround for it, but a year or so later we took it out because it was no longer as important. | 16:24 |
cfriesen | danpb: when enabling pre-allocation, does it also zero out the allocated space? | 16:24 |
kashyap | cfriesen: Unless I'm wrong, it only allocates qcow2 metadata | 16:25 |
*** sdake has joined #openstack-nova | 16:25 | |
*** sree has quit IRC | 16:25 | |
kashyap | Maybe not, it also does "fallocate". /me checks... | 16:25 |
danpb | cfriesen: yes, its all zeros | 16:25 |
kashyap | Yep | 16:26 |
danpb | mdbooth: alos if you don't preallocate, then you have to use aio=threads, instead of aio=native, which is a worse performing I/o model for qemu | 16:27 |
mdbooth | cfriesen: What's the context of this discussion, btw? | 16:27 |
*** mlavalle has quit IRC | 16:28 | |
mdbooth | Are we discussing ephemerals? | 16:28 |
openstackgerrit | Arne Recknagel proposed openstack/nova: Increase driver dict generation robustness https://review.openstack.org/394425 | 16:28 |
kashyap | mdbooth: Yeah: "cfriesen> for qcow images at least it shouldn't actually take up any more space to have swap/ephemeral, right?" | 16:28 |
cfriesen | mdbooth: there are a couple of patches related to ephemeral/swap over resize/migration, and mriedem is understandably reluctant to merge them without test coverage | 16:28 |
*** ekuris has quit IRC | 16:28 | |
mdbooth | cfriesen: Links? | 16:28 |
mdbooth | Have I reviewed them already, btw? | 16:29 |
cfriesen | mdbooth: https://review.openstack.org/#/c/300616/ and https://review.openstack.org/#/c/346263/ (you've reviewed the latter) | 16:29 |
cfriesen | mdbooth: mriedem just posted to the dev list about improving the testing...I was just wondering how much of an impact it would really have to add ephemeral/swap to the default devstack flavors | 16:30 |
*** belmoreira has quit IRC | 16:30 | |
mdbooth | Ah... gotcha | 16:30 |
mdbooth | So, is thick vs thin alloc something operators might want to differentiate per instance, btw? | 16:31 |
*** tesseract- has quit IRC | 16:31 | |
mdbooth | For eg, pricing. | 16:31 |
openstackgerrit | Gábor Antal proposed openstack/nova: Fix a TypeError in notification_sample_base.py https://review.openstack.org/394516 | 16:31 |
cfriesen | mdbooth: are you talking qcow or LVM? | 16:32 |
mdbooth | In general | 16:32 |
*** manasm has quit IRC | 16:32 | |
*** nherciu has joined #openstack-nova | 16:33 | |
*** krtaylor has quit IRC | 16:33 | |
mdbooth | Although I don't think we do thin lvm yet, do we? Certainly not lvm snapshots. | 16:33 |
cfriesen | mdbooth: we did thin in house because the performance hit for zeroing out thick was so huge, especially with multiple simultaneous instance deletions on compute nodes with crappy IO subsystems. | 16:34 |
*** paul-carlton2 has quit IRC | 16:35 | |
mdbooth | cfriesen: Cool. Do you still have patches? | 16:35 |
cfriesen | mdbooth: I mentioned it at the time but nobody was really interested. I could see about posting them. | 16:35 |
*** gouthamr has quit IRC | 16:35 | |
cfriesen | mdbooth: basically you take the zeroing-out hit on first write rather than all at once. | 16:36 |
*** tbachman has joined #openstack-nova | 16:36 | |
*** bvanhav_ has quit IRC | 16:36 | |
*** mlavalle has joined #openstack-nova | 16:37 | |
*** nherciu_ has joined #openstack-nova | 16:39 | |
*** nherciu has quit IRC | 16:39 | |
*** vishwanathj has joined #openstack-nova | 16:40 | |
kashyap | cfriesen: On Preallocation, I wonder if you tested with all three options: | 16:40 |
kashyap | "Preallocation mode (allowed values: "off", "falloc", "full"). "falloc" mode preallocates space for image by calling posix_fallocate(). "full" mode preallocates space for image by writing zeros to underlying storage." | 16:40 |
kashyap | Currently, Nova does "falloc", if you see the commit that introduced it 24f6c62ad776dc0fc85ce29eb34e7e0a1f270d07 | 16:41 |
cfriesen | kashyap: yeah, I wonder if fallocate is a security risk? | 16:41 |
kashyap | cfriesen: Why would it be? | 16:41 |
cfriesen | kashyap: we haven't done any of the preallocation stuff with qcow2. If it allocates space but doesn't cause that space to get zeroed at some point, you could end up reading someone else's data. | 16:42 |
*** raunak has quit IRC | 16:43 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: api-ref: fix addFloatingIp action docs https://review.openstack.org/393951 | 16:43 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Add functional api_samples test for addFloatingIp action https://review.openstack.org/394524 | 16:43 |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: virtlogd: use virtlogd for char devices https://review.openstack.org/323765 | 16:44 |
cfriesen | kashyap: where are you seeing those preallocation modes? | 16:44 |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: remove py26 compat code in "get_console_output" https://review.openstack.org/334480 | 16:44 |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: read rotated "console.log" files https://review.openstack.org/385366 | 16:44 |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: refactor console device creation methods https://review.openstack.org/385367 | 16:44 |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: change get_console_output as prep work for bp/libvirt-virtlogd https://review.openstack.org/394525 | 16:44 |
kashyap | cfriesen: I think you're referring to https://lwn.net/Articles/492959/ | 16:44 |
kashyap | cfriesen: I was looking at qemu-img man page. Nova currently does fallocate (if you see the imagebackend.py) | 16:44 |
*** trisq has quit IRC | 16:45 | |
mdbooth | kashyap: It currently does it in cache(), which is the wrong place :( | 16:45 |
cfriesen | kashyap: okay, good. fallocate() is safe from the kernel. | 16:45 |
mriedem | alex_xu: all of your api sample functional test dreams have come true https://review.openstack.org/#/c/394524/1 | 16:46 |
kashyap | mdbooth: Indeed it's in cache() method, I didn't dig why it's wrong place, but I know you're meditating on imagebackend | 16:47 |
*** paul-carlton2 has joined #openstack-nova | 16:47 | |
*** annegentle has joined #openstack-nova | 16:47 | |
*** annegentle has quit IRC | 16:47 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform aggregate.create notification https://review.openstack.org/394511 | 16:47 |
openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform aggregate.delete notification https://review.openstack.org/394512 | 16:47 |
*** annegentle has joined #openstack-nova | 16:47 | |
*** shaohe_feng_ has joined #openstack-nova | 16:48 | |
mdbooth | kashyap: Because not everything is a file, and fallocate on a qcow2 is not correct. | 16:48 |
mdbooth | It won't corrupt it, but it won't be the right size, either. | 16:48 |
*** shaohe_f- has joined #openstack-nova | 16:48 | |
kashyap | mdbooth: Why is it not correct? I'm curious. Many moons ago I learnt that from a QEMU dev - https://kashyapc.com/2011/12/02/little-more-disk-io-perf-improvement-with-fallocateing-a-qcow2-disk/ | 16:49 |
mdbooth | kashyap: It doesn't take into account the metadata overhead, for eg. | 16:49 |
mdbooth | It won't be the right size. | 16:49 |
kashyap | (Not that I'm saying it is correct. And, since then, that 'falloc' option is now baked into 'qemu-img' as an option) | 16:49 |
kashyap | mdbooth: Ah, right...The qcow2 metadata | 16:49 |
mdbooth | Yeah | 16:50 |
*** shaohe_feng has quit IRC | 16:50 | |
kashyap | mdbooth: But I still have a question | 16:51 |
*** ducttape_ has joined #openstack-nova | 16:51 | |
* kashyap brings a URL | 16:51 | |
*** shaohe_feng has joined #openstack-nova | 16:52 | |
*** shaohe_feng_ has quit IRC | 16:52 | |
kashyap | mdbooth: Then, why would QEMU upstream merge this https://lists.gnu.org/archive/html/qemu-devel/2014-07/msg00363.html | 16:52 |
*** shaohe_f- has quit IRC | 16:52 | |
mdbooth | kashyap: Presumably that knows how large to make the file. | 16:53 |
*** mriedem1 has joined #openstack-nova | 16:53 | |
mdbooth | As I say, it won't corrupt the data, it just won't be correct. | 16:53 |
mdbooth | What error do you get when you write to a file and allocation fails? | 16:53 |
mdbooth | Ah.. it is ENOSPC, I thought it might have been something else. | 16:54 |
kashyap | By your first comment, that patch is allowing posix_fallocate(3) for a qcow2 file | 16:54 |
*** Daisy_ has joined #openstack-nova | 16:54 | |
mdbooth | Anyway, you aren't 100% protecting yourself against that | 16:54 |
*** mriedem has quit IRC | 16:55 | |
dirk | dear nova cores, please take one minute to review https://review.openstack.org/#/c/389042/ - this is a very trivial changeset, it already had one +2, please review and merge. it blocks oslotest update and is an obstacle for the g-r team. thanks a lot! | 16:55 |
*** sdake has quit IRC | 16:56 | |
*** bvanhav_ has joined #openstack-nova | 16:57 | |
kashyap | And yes, when allocation fails -ENOSPC. | 16:58 |
*** Daisy_ has quit IRC | 16:59 | |
*** fragatina has joined #openstack-nova | 16:59 | |
* mdbooth was getting confused with memory overcommit, where you get a BUS error iirc | 16:59 | |
*** mriedem1 is now known as mriedem | 17:00 | |
*** shaohe_feng has quit IRC | 17:00 | |
*** harlowja_at_home has quit IRC | 17:00 | |
*** mgoddard_ has quit IRC | 17:00 | |
kashyap | mdbooth: posix_fallocate(3) is equivalent to metadata preallocation on the FS-level. But you of course know this | 17:00 |
*** mgoddard has joined #openstack-nova | 17:00 | |
*** fragatina has quit IRC | 17:01 | |
*** shaohe_feng has joined #openstack-nova | 17:01 | |
*** markus_z has quit IRC | 17:01 | |
*** fragatina has joined #openstack-nova | 17:01 | |
mdbooth | kashyap: Presumably only if you know the total size? | 17:01 |
*** shaohe_f- has joined #openstack-nova | 17:02 | |
*** diogogmt has joined #openstack-nova | 17:02 | |
kashyap | mdbooth: Hmm, I honestly don't know. I'll stop hand-waving here. I recall a large discussion on qemu-devel list when that patch was proposed | 17:02 |
*** shaohe_f| has joined #openstack-nova | 17:02 | |
*** vishwanathj has quit IRC | 17:03 | |
kashyap | mdbooth: Oh, I do know - yes, you should know the total size | 17:03 |
*** krtaylor has joined #openstack-nova | 17:03 | |
* kashyap will blame the chilly wind here | 17:03 | |
*** shaohe_f- has quit IRC | 17:04 | |
*** krtaylor has quit IRC | 17:05 | |
mdbooth | $ ls -l test-falloc.qcow2 test.raw | 17:05 |
mdbooth | -rw-r--r--. 1 mbooth mbooth 1074135040 Nov 7 17:04 test-falloc.qcow2 | 17:05 |
mdbooth | -rw-rw-r--. 1 mbooth mbooth 1073741824 Nov 7 17:05 test.raw | 17:05 |
*** shaohe_feng has quit IRC | 17:06 | |
mdbooth | kashyap: Notice how the qcow2 file is 393216 bytes bigger | 17:06 |
openstackgerrit | Lee Yarwood proposed openstack/nova: WIP encryptors: Introduce EncryptionProvider constants https://review.openstack.org/393901 | 17:06 |
*** shaohe_f| has quit IRC | 17:06 | |
mdbooth | That's been pre-allocated with qemu-img -o preallocation=falloc | 17:06 |
mdbooth | That means that if you just falloc the qcow2 to 1G, it's still 393216 too small | 17:07 |
kashyap | Hmm. mdbooth - And, with preallocation=full? | 17:07 |
mdbooth | Which means that, although it's unlikely, you can still get ENOSPC when writing to it | 17:07 |
*** owalsh has quit IRC | 17:08 | |
mdbooth | kashyap: They're both the same size. | 17:08 |
mdbooth | But full is obviously much slower to create | 17:08 |
*** gcb has quit IRC | 17:08 | |
*** matrohon has quit IRC | 17:09 | |
kashyap | Yeah | 17:09 |
kashyap | (As it's more performant.) But we don't expect users of Nova to know these kind of details, do we..."Just give me a performant image", and Nova should do the Right Thing I suppose | 17:10 |
openstackgerrit | Sergey Nikitin proposed openstack/nova: Modify fields of 'Diagnostics' object https://review.openstack.org/394480 | 17:10 |
*** mvk has quit IRC | 17:10 | |
*** paul-carlton2 has quit IRC | 17:11 | |
*** krtaylor has joined #openstack-nova | 17:11 | |
*** smatzek has joined #openstack-nova | 17:13 | |
*** paul-carlton2 has joined #openstack-nova | 17:13 | |
*** shaohe_feng has joined #openstack-nova | 17:13 | |
jaypipes | bauzas: thx for +W'ing that one patch. sorry, went out to grab some lunch and didn't see your notice until just now. | 17:15 |
bauzas | jaypipes: np, didn't want to block | 17:16 |
jaypipes | bauzas: if you're looking for a relatively easy patch to review, cdent's microversion decorator patch is looking good: https://review.openstack.org/#/c/388115/ | 17:16 |
bauzas | jaypipes: roger. | 17:16 |
jaypipes | bauzas: 10-4 good buddy. | 17:16 |
* jaypipes can see dansmith rolling his eyes from here... | 17:17 | |
bauzas | jaypipes: FWIW, I nacked the patch you mentioned me about the AZ cache performance improvement, because I honestly think it would be better to cut the AZ cache and fix the root cause rather than just do the hack | 17:17 |
jaypipes | bauzas: yeah, I saw your review comment on that. I'm a bit torn on it, frankly, but I think this is an easy performance win and doesn't make things any more broken than they currently are. | 17:17 |
bauzas | that AZ cache is causing too much trouble, and us trying to workaround it is just piling a new stack of code against something hugely already wrong | 17:18 |
*** shaohe_feng has quit IRC | 17:18 | |
bauzas | jaypipes: so, here are my thoughts | 17:18 |
jaypipes | bauzas: I don't necessarily disagree with you on that point :) | 17:18 |
*** owalsh has joined #openstack-nova | 17:18 | |
bauzas | jaypipes: if we have the promise that this code is about to be killed later in that cycle because we're gonna kill that silly cache and do the Real Work (c), then I'm +2 | 17:19 |
bauzas | but I don't want to keep that rustine for a long time | 17:19 |
bauzas | oops | 17:19 |
* bauzas google translates "rustine" | 17:19 | |
*** andreas_s has quit IRC | 17:19 | |
bauzas | rubber patch | 17:19 |
*** nic has joined #openstack-nova | 17:19 | |
*** links has quit IRC | 17:20 | |
*** slaweq_ has joined #openstack-nova | 17:20 | |
bauzas | jaypipes: heh, missed your point https://en.wikipedia.org/wiki/List_of_CB_slang | 17:21 |
openstackgerrit | Lee Yarwood proposed openstack/nova: WIP encryptors: Introduce EncryptionProvider constants https://review.openstack.org/393901 | 17:21 |
*** Apoorva has joined #openstack-nova | 17:22 | |
*** mvk has joined #openstack-nova | 17:22 | |
*** shaohe_feng has joined #openstack-nova | 17:22 | |
*** gouthamr has joined #openstack-nova | 17:23 | |
*** unicell1 has quit IRC | 17:23 | |
*** shaohe_feng_ has joined #openstack-nova | 17:25 | |
*** shaohe_feng has quit IRC | 17:26 | |
*** shaohe_feng has joined #openstack-nova | 17:26 | |
*** fragatina has quit IRC | 17:28 | |
openstackgerrit | Lee Yarwood proposed openstack/nova: WIP encryptors: Introduce EncryptionProvider constants https://review.openstack.org/393901 | 17:29 |
openstackgerrit | Roman Podoliaka proposed openstack/nova: WIP: Integrate OSprofiler and Nova https://review.openstack.org/254703 | 17:29 |
*** shaohe_feng_ has quit IRC | 17:30 | |
*** Sukhdev has quit IRC | 17:30 | |
*** shaohe_feng has quit IRC | 17:30 | |
*** sudipto has joined #openstack-nova | 17:31 | |
*** sudipto_ has joined #openstack-nova | 17:31 | |
*** diogogmt has quit IRC | 17:33 | |
*** nherciu_ has quit IRC | 17:36 | |
cfriesen | danpb: qemu question for you...if I do pci-passthrough and in the guest I try to reassign the interrupt affinity: 1) will it work?, and 2) will it affect the interrupt affinity on the host? | 17:36 |
*** shaohe_feng_ has joined #openstack-nova | 17:37 | |
*** lamt has quit IRC | 17:38 | |
*** lamt has joined #openstack-nova | 17:38 | |
*** lpetrut has quit IRC | 17:39 | |
*** ircuser-1 has joined #openstack-nova | 17:40 | |
*** shaohe_feng_ has quit IRC | 17:40 | |
danpb | cfriesen: interupt affinity is nothing todo with qemu - its purely a guest os concept | 17:41 |
*** shaohe_feng has joined #openstack-nova | 17:42 | |
*** diogogmt has joined #openstack-nova | 17:43 | |
*** lamt has quit IRC | 17:43 | |
openstackgerrit | Gábor Antal proposed openstack/nova: Readibility enhancements in notification_sample_tests/ https://review.openstack.org/385884 | 17:44 |
*** jdurgin1 has quit IRC | 17:45 | |
*** sahid has quit IRC | 17:45 | |
*** lucasagomes is now known as lucas-afk | 17:45 | |
*** shaohe_feng_ has joined #openstack-nova | 17:46 | |
aunnam | johnthetubaguy, Can you review this https://review.openstack.org/#/c/393931/ when you have time. | 17:47 |
*** shaohe_feng_ has joined #openstack-nova | 17:47 | |
*** shaohe_feng has quit IRC | 17:49 | |
cfriesen | danpb: so if I want to ensure that all work for a pci-passthrough device stays on the guest CPUs (assuming dedicated CPUs) I'd have to separately set irq affinity in the host and in the guest? | 17:50 |
*** shaohe_feng_ has quit IRC | 17:51 | |
danpb | cfriesen: yes | 17:51 |
cfriesen | thanks | 17:51 |
*** shaohe_feng has joined #openstack-nova | 17:51 | |
*** lpetrut has joined #openstack-nova | 17:51 | |
danpb | cfriesen: there's actually a gap in nova right now | 17:52 |
danpb | cfriesen: because we don't tell QEMU what *guest* NUMA node its assigned PCI device is associated with | 17:52 |
danpb | cfriesen: so if the guest has > 1 numa node, there's no way to know which guest CPUs to use for the PCI device to ensure optimal performance | 17:52 |
danpb | cfriesen: we're just about at the point where we can fix that now with libvirt+QEMU by adding extra PCI buses for each guest NUMA node | 17:53 |
*** shaohe_feng_ has joined #openstack-nova | 17:53 | |
*** cdelatte has joined #openstack-nova | 17:54 | |
*** rama_y has quit IRC | 17:54 | |
*** shaohe_f- has joined #openstack-nova | 17:55 | |
*** slaweq__ has joined #openstack-nova | 17:56 | |
*** shaohe_feng has quit IRC | 17:56 | |
*** slaweq__ has quit IRC | 17:56 | |
*** slaweq_ has quit IRC | 17:56 | |
*** shuquan has joined #openstack-nova | 17:57 | |
*** shaohe_feng has joined #openstack-nova | 17:57 | |
*** shaohe_feng_ has quit IRC | 17:58 | |
*** ralonsoh has quit IRC | 17:58 | |
*** Kimmo__ has joined #openstack-nova | 17:58 | |
*** paul-carlton2 has quit IRC | 17:59 | |
openstackgerrit | Julian proposed openstack/nova: Transform instance.snapshot notifications https://review.openstack.org/330128 | 17:59 |
*** shaohe_f- has quit IRC | 17:59 | |
*** shaohe_feng has quit IRC | 18:00 | |
*** jwcroppe has quit IRC | 18:00 | |
*** slaweq__ has joined #openstack-nova | 18:00 | |
*** jwcroppe has joined #openstack-nova | 18:01 | |
*** shuquan has quit IRC | 18:01 | |
*** mvk has quit IRC | 18:02 | |
*** derekh has quit IRC | 18:03 | |
*** fragatina has joined #openstack-nova | 18:04 | |
*** jwcroppe_ has joined #openstack-nova | 18:04 | |
*** unicell has joined #openstack-nova | 18:05 | |
*** jwcroppe has quit IRC | 18:05 | |
*** dave-mccowan has quit IRC | 18:06 | |
*** yamahata has joined #openstack-nova | 18:06 | |
*** jwcroppe_ has quit IRC | 18:06 | |
*** rama_y has joined #openstack-nova | 18:06 | |
openstackgerrit | Jay Pipes proposed openstack/nova: placement: adds REST API for nested providers https://review.openstack.org/384807 | 18:06 |
openstackgerrit | Jay Pipes proposed openstack/nova: placement: add get_all_by_root_provider_uuid() https://review.openstack.org/377215 | 18:06 |
openstackgerrit | Jay Pipes proposed openstack/nova: placement: update client to set parent provider https://review.openstack.org/385693 | 18:06 |
openstackgerrit | Jay Pipes proposed openstack/nova: placement: add nested resource providers https://review.openstack.org/377138 | 18:06 |
*** jwcroppe has joined #openstack-nova | 18:07 | |
*** sach-m has joined #openstack-nova | 18:08 | |
*** danpb has quit IRC | 18:09 | |
*** annegentle has quit IRC | 18:10 | |
*** dave-mccowan has joined #openstack-nova | 18:11 | |
*** annegentle has joined #openstack-nova | 18:11 | |
*** jwcroppe has quit IRC | 18:11 | |
*** shaohe_feng has joined #openstack-nova | 18:12 | |
*** bvanhav__ has joined #openstack-nova | 18:13 | |
*** bvanhav_ has quit IRC | 18:13 | |
*** jaosorior_sick has quit IRC | 18:14 | |
*** shaohe_feng has quit IRC | 18:14 | |
*** shaohe_feng has joined #openstack-nova | 18:14 | |
*** gszasz has quit IRC | 18:15 | |
*** hshiina has quit IRC | 18:15 | |
*** lpetrut has quit IRC | 18:17 | |
*** shaohe_feng_ has joined #openstack-nova | 18:18 | |
*** shaohe_feng has quit IRC | 18:18 | |
*** shaohe_feng_ is now known as shaohe_feng | 18:18 | |
*** yamahata has quit IRC | 18:19 | |
*** dikonoor has quit IRC | 18:21 | |
*** yamahata has joined #openstack-nova | 18:22 | |
*** ociuhandu has quit IRC | 18:24 | |
*** liverpooler has quit IRC | 18:24 | |
*** tyrola has joined #openstack-nova | 18:26 | |
*** sach-m has quit IRC | 18:28 | |
*** sudipto has quit IRC | 18:30 | |
*** sudipto_ has quit IRC | 18:30 | |
cfriesen | jaypipes: could you look at https://review.openstack.org/#/c/339715 ? we talked about it at the summit, your earlier comments have been addressed. | 18:30 |
*** fragatina has quit IRC | 18:30 | |
*** fragatina has joined #openstack-nova | 18:31 | |
*** yamahata has quit IRC | 18:31 | |
*** annegentle has quit IRC | 18:31 | |
*** annegentle has joined #openstack-nova | 18:32 | |
*** mkoderer has quit IRC | 18:32 | |
*** ihrachys has quit IRC | 18:37 | |
*** aloga_ has quit IRC | 18:38 | |
*** armax has joined #openstack-nova | 18:39 | |
jaypipes | cfriesen: done. | 18:41 |
*** prashkre has joined #openstack-nova | 18:42 | |
*** jwcroppe has joined #openstack-nova | 18:42 | |
*** paw has quit IRC | 18:45 | |
*** jwcroppe has quit IRC | 18:46 | |
*** jwcroppe has joined #openstack-nova | 18:46 | |
*** prashkre_ has joined #openstack-nova | 18:50 | |
*** edand has joined #openstack-nova | 18:51 | |
*** jwcroppe has quit IRC | 18:51 | |
*** prashkre_ has quit IRC | 18:52 | |
*** sach-m has joined #openstack-nova | 18:53 | |
*** prashkre has quit IRC | 18:53 | |
*** lamt has joined #openstack-nova | 18:53 | |
*** markvoelker_ has joined #openstack-nova | 18:54 | |
*** Sukhdev has joined #openstack-nova | 18:55 | |
*** Zer0Byte__ has joined #openstack-nova | 18:56 | |
*** markvoelker has quit IRC | 18:56 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/394584 | 18:56 |
*** sach-m has quit IRC | 18:57 | |
*** tbachman has quit IRC | 18:57 | |
*** markvoelker has joined #openstack-nova | 18:57 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Fix race in test_volume_swap_server_with_error https://review.openstack.org/394585 | 18:58 |
mriedem | ^ fixes a race in our functional tests | 18:59 |
*** markvoelker_ has quit IRC | 18:59 | |
*** sambetts is now known as sambetts|afk | 18:59 | |
ildikov | mriedem: can you take a look at this at some point: https://review.openstack.org/#/c/335358/ ? | 19:00 |
mriedem | yeah. cburgess potentially had an issue with that, but he might have talked himself out of it. | 19:00 |
cburgess | mriedem What did I do? | 19:00 |
cburgess | Oh well... | 19:00 |
ildikov | mriedem: I updated a few things in it and I added some follow up questions to your earlier comments | 19:00 |
cburgess | Its not that I have issue one way or the other per se. | 19:00 |
ildikov | mriedem: I hope it's getting there to become something sane now... | 19:01 |
cburgess | mriedem My point was *if* we intend to remove it then we need to implement a different "fix" to the lack of volume type support. | 19:01 |
cburgess | mriedem A more complicated fix I should add but if its the direction we want to go then fine. | 19:01 |
*** ayoung has joined #openstack-nova | 19:06 | |
*** prashkre has joined #openstack-nova | 19:10 | |
*** slaweq__ has quit IRC | 19:10 | |
openstackgerrit | Jay Pipes proposed openstack/nova: placement: REST API for resource classes https://review.openstack.org/391959 | 19:10 |
*** sach-m has joined #openstack-nova | 19:13 | |
mriedem | cburgess: i can't even keep it straight in my head anymore on what you needed - i thought you wanted the wait for available thing in the compute manager, that waits for the volume that nova creates during BFV, to be less restrictive | 19:14 |
*** mnestratov has quit IRC | 19:15 | |
mriedem | or was your case a BFV with an existing volume, that was not yet 'available' | 19:15 |
mriedem | i thought i was maybe the latter b/c then you can create the volume with the type you want | 19:15 |
mriedem | but don't want to have to wait for it to be available | 19:15 |
*** slaweq_ has joined #openstack-nova | 19:18 | |
*** slaweq_ has quit IRC | 19:20 | |
openstackgerrit | prashkre proposed openstack/nova: Separate CRUD policy for server_groups https://review.openstack.org/391113 | 19:21 |
mriedem | thorst_: efried: word to the wise, you're going to want to use test ids here rather than test names https://github.com/powervm/powervm-ci/blob/master/tempest/skip_tests.txt | 19:21 |
*** slaweq_ has joined #openstack-nova | 19:21 | |
mriedem | b/c the test names/paths can change, the idempotent id won't | 19:21 |
cburgess | mriedem Well what I *want* is to pass volume-type on the BFV call. But that was shot down. So the alternate solution was create a volume first then issue a BFV call on that existing volume. But right now the code will error if the volume isn't | 19:21 |
cburgess | Isn't available | 19:21 |
efried | esberglu ^-3^ | 19:22 |
mriedem | cburgess: right so you were going to update the check_attach method right? | 19:22 |
cburgess | So the proposal was to change check-attach to allow downloading as a valid status and pass it onto the compute node which will use its eixsting wait to become available code. | 19:22 |
cburgess | mriedem Correct that was the original idea. | 19:22 |
mriedem | ok | 19:22 |
*** paw has joined #openstack-nova | 19:22 | |
cburgess | mriedem But if check_attach goes away... then we have to update the reserve call on cinder to be smarter since that does all the heavy lifting now in the check_attach-less world. | 19:22 |
cburgess | mriedem So its not that I object to removing check_attach per se other then it makes our job harder since we now have to land something in cinder, then in nova. | 19:23 |
cburgess | Vs it being purely a nova problem. | 19:23 |
mriedem | right, because os-reserve expects a volume in status 'available' | 19:23 |
mriedem | if it's not a multiattach volume | 19:23 |
cburgess | Right | 19:23 |
cburgess | The real issue is that the reserve call on the cinder side is just written wrong. | 19:24 |
cburgess | Its going to run into issues with multi-attach eventually too I suspect | 19:24 |
mriedem | huh, so with BFV today with a provided volume, we never call os-reserve on it? | 19:24 |
mriedem | i guess not | 19:24 |
cburgess | Nope | 19:25 |
cburgess | We just make sure its "available" | 19:25 |
cburgess | ildikov patch changes/fixes that | 19:25 |
mriedem | yup | 19:25 |
mriedem | which i tend to agree with | 19:25 |
mriedem | i could BFV 2 instances with the same volume, they'd both pass the api but one would fail in the compute | 19:25 |
ildikov | cburgess: reserve is already smart enough in Cinder | 19:25 |
cburgess | mriedem So the question on Friday in BCN was... do we plan on landing ildikov patch. If so we have to take a different route to fix our issue. | 19:25 |
mriedem | ildikov: cburgess wants os-reserve to allow 'available' or 'downloading' status for the volume | 19:26 |
mriedem | to reserve it | 19:26 |
mriedem | s/reserve/lock/ | 19:26 |
ildikov | cburgess: and I also updated the BFV call which was missing the reserve_volume call | 19:26 |
cburgess | mriedem ildikov Which won't work today because reserve changes the volume status to "attaching" which is wrong because its not attaching. Reserve should have used a new status field. | 19:26 |
mriedem | ildikov: chet doesn't actually want nova calling os-reserve on the BFV volume in nova-api as it makes his job harder | 19:26 |
mriedem | cburgess: well, that might also be fixed with the new create_attachment API stuff that john griffith is working on | 19:27 |
cburgess | mriedem Thats a strong statement. Like I said I don't have strong feelings either way other then it makes my job harder. | 19:27 |
mriedem | which decouples the status on the volume from the status on the attachment | 19:27 |
ildikov | mriedem: cburgess bah, ok, I'm getting there | 19:27 |
ildikov | cburgess: reserve is smart enough to handle multi-attach, just like the new calls will be that we're planning | 19:28 |
cburgess | mriedem ildikov nic and I already worked out how to do this on the cinder and nova side using reserve. We just need to write a cinder spec for the changes and work on getting it landed. | 19:28 |
mriedem | ildikov: but not for a volume that is still being created | 19:28 |
mriedem | which is what chet cares about | 19:28 |
ildikov | cburgess: besides that we only considered the current behavior | 19:28 |
cburgess | ildikov The problem with reserve as written today is that it is monotomic. So with multiattach it supports a multiattach volume but *not* if its already been reserved. So you can only issue reserve once on a volume and then you have to wait until its no longer "attaching". | 19:29 |
cburgess | ildikov At least thats what I seem to recall from reading the code a few weeks ago, maybe I'm wrong. | 19:29 |
dansmith | bauzas: to resume, you think that even with one call, we should pass a tuple of (reqspec, instance) to the build call, right? | 19:29 |
mriedem | cburgess: that's correct | 19:30 |
cburgess | mriedem ildikov So the TL;DR version is.. I don't have an objection to the patch set to remove check-attach and use reserve. Just makes my life harder, but its OpenStack, I'm used to that. | 19:30 |
prashkre | mriedem: Hi. Pls review https://review.openstack.org/391113 when u get sometime. Thank you! | 19:30 |
*** ociuhandu has joined #openstack-nova | 19:30 | |
cburgess | mriedem Which feels weird to me to say this thing can support multi-attach but only monotomically. | 19:30 |
ildikov | cburgess: I think it would worth a chat on the Cinder channel regarding doing those attaches in parallel | 19:31 |
mriedem | cburgess: yeah that came up in the nova/cinder call this morning | 19:31 |
mriedem | cburgess: we might change that over time | 19:31 |
cburgess | mriedem ildikov fair enough | 19:31 |
mriedem | but after we actually get the simple multiattach flow working first | 19:31 |
cburgess | Its another one I don't have a strong opinion on other then "Huh thats weird" | 19:31 |
ildikov | mriedem: +1 | 19:31 |
bauzas | dansmith: well, I'm not really opiniated, I just wanted to make sure we pass a Requestspec per instance, not only one for a multiple instances call | 19:31 |
harlowja | does the following ring-a-bell for anyone http://logs.openstack.org/periodic/periodic-nova-py27-with-oslo-master/7f6fac2/console.html#_2016-11-07_06_14_19_790374 ? | 19:31 |
cburgess | mriedem I'm still hoping I can change your mind so I can just do it in the API, like it should be done. :P | 19:31 |
bauzas | dansmith: so I thought about something like a tuple for each instance, but I leave you for the kind of format | 19:32 |
dansmith | bauzas: but, right now, we do one reqspec per instance, right? | 19:32 |
harlowja | oslo.policy doesn't seem to have changed, or oslo.middleware so wondering if its a known bug | 19:32 |
dansmith | bauzas: wait | 19:32 |
mriedem | harlowja: nope | 19:32 |
dansmith | bauzas: we do one for all I mean | 19:32 |
bauzas | dansmith: no, we have a Requestspec for each instance | 19:32 |
bauzas | even if two instances are called at the same time | 19:33 |
dansmith | bauzas: ah, right I see now | 19:33 |
bauzas | dansmith: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L921-L931 yup | 19:34 |
mriedem | harlowja: according to logstash that only ever fails in the periodic job | 19:34 |
harlowja | mriedem ya, which is odd | 19:34 |
*** ociuhandu has quit IRC | 19:35 | |
mriedem | harlowja: tox -v -epy27-oslo-master - what is py27-oslo-master ? where is that defined? | 19:37 |
harlowja | mriedem its dynamic :-/ | 19:37 |
harlowja | https://github.com/openstack-infra/project-config/blob/master/jenkins/scripts/run-tox-with-oslo-master.sh | 19:38 |
*** edand has quit IRC | 19:38 | |
*** thingee_ has quit IRC | 19:38 | |
*** edand has joined #openstack-nova | 19:38 | |
*** thingee has joined #openstack-nova | 19:39 | |
harlowja | http://logs.openstack.org/periodic/periodic-nova-py27-with-oslo-master/6235dad/console.html#_2016-11-06_06_03_45_855922 (installing all the oslo stuff from source) | 19:39 |
*** yamahata has joined #openstack-nova | 19:39 | |
*** dave-mccowan has quit IRC | 19:39 | |
dansmith | bauzas: you mentioned deleting the requestspec further down, but shouldn't we do it at https://review.openstack.org/#/c/374267/9/nova/compute/api.py@992 also? | 19:39 |
*** ociuhandu has joined #openstack-nova | 19:42 | |
harlowja | mriedem wondering if it was https://github.com/openstack/oslo.context/commit/fd224586199f99affc5d37993907024bbf3cd00f | 19:42 |
harlowja | hmmm | 19:42 |
* harlowja will try that in a few | 19:43 | |
mriedem | harlowja: fwiw that error seems to have shown up on 11/2 | 19:44 |
mriedem | fd224586199f99affc5d37993907024bbf3cd00f merged in august | 19:44 |
*** armax has quit IRC | 19:44 | |
bauzas | dansmith: correct, I missed that one | 19:45 |
dansmith | bauzas: okay just wanted to make sure | 19:46 |
bauzas | thanks for that, I still need to update my series for deleting the spec when calling the instance destroy() method | 19:46 |
*** ociuhandu has quit IRC | 19:46 | |
harlowja | mriedem https://github.com/openstack/oslo.context/commit/e52b6f790f17edc321e9a1a3c41f945ed86a71f4 (11 days) | 19:46 |
harlowja | its like the only change not yet released, ha | 19:47 |
harlowja | (probably proposed in aug) | 19:47 |
dansmith | bauzas: there also appears to be a *lot* of residue still of the legacy requestspec components :/ | 19:47 |
mriedem | harlowja: yeah probably good point - just revert and run the tests and see if it works | 19:48 |
harlowja | mriedem i'll run locally before that full on revert, ha | 19:49 |
bauzas | dansmith: yup, I need to write a blueprint and ask during the next nova meeting about that being a specless one so we could clean that up | 19:49 |
*** nherciu has joined #openstack-nova | 19:50 | |
bauzas | just had no time to do it yet | 19:50 |
thorst_ | mriedem: sorry for the delay. Thx for the guidance on test ids | 19:50 |
mriedem | bauzas: we probably don't need a spec to cleanup compat code | 19:51 |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Update hacking version https://review.openstack.org/389338 | 19:52 |
bauzas | mriedem: <3 | 19:52 |
*** dave-mccowan has joined #openstack-nova | 19:53 | |
bauzas | mriedem: yeah, was my thought, just wanted to make it clear :) | 19:53 |
openstackgerrit | Lee Yarwood proposed openstack/nova: WIP Compute: Reset bdm connection_info during LM rollback https://review.openstack.org/338929 | 19:53 |
*** yamahata has quit IRC | 19:54 | |
*** woodster_ has joined #openstack-nova | 19:56 | |
*** ociuhandu has joined #openstack-nova | 19:57 | |
*** ociuhandu has quit IRC | 20:02 | |
*** shuquan has joined #openstack-nova | 20:11 | |
*** ociuhandu has joined #openstack-nova | 20:12 | |
mriedem | thorst_: +2 on https://review.openstack.org/#/c/381772/ | 20:14 |
mriedem | mikal is out this week on PTO | 20:14 |
mriedem | so you'll have to grease another nova-specs core | 20:14 |
cburgess | mriedem 2 weeks I think | 20:14 |
mriedem | oh right | 20:14 |
mriedem | yeah | 20:14 |
mriedem | i even gave him shit for that | 20:14 |
*** shuquan has quit IRC | 20:16 | |
*** prashkre has quit IRC | 20:16 | |
*** ociuhandu has quit IRC | 20:17 | |
*** edand has quit IRC | 20:21 | |
thorst_ | mriedem: thx. We'll ask in the nova IRC meeting coming up. We'll work on things in background until then | 20:23 |
*** jlk has quit IRC | 20:24 | |
*** jlk has joined #openstack-nova | 20:25 | |
*** jlk has quit IRC | 20:25 | |
*** jlk has joined #openstack-nova | 20:25 | |
cfriesen | jaypipes: thanks | 20:26 |
*** ociuhandu has joined #openstack-nova | 20:27 | |
*** jwcroppe has joined #openstack-nova | 20:27 | |
*** tomhambleton has joined #openstack-nova | 20:30 | |
bauzas | mriedem: maybe we should plan a spec review day ? | 20:30 |
mriedem | bauzas: mayhap | 20:32 |
*** ociuhandu has quit IRC | 20:32 | |
mriedem | spec freeze is 11/17 | 20:32 |
mriedem | so maybe next monday | 20:32 |
mriedem | that doesn't give a lot of time for responses though | 20:32 |
bauzas | mriedem: yup, just thought about this possiblity given folks could help us | 20:33 |
mriedem | could do it this wednesday but then we don't get to announce it in the nova meeting, just in the ML | 20:33 |
bauzas | Friday maybe ? | 20:33 |
mriedem | bauzas: i assume you mean non-core reviewers? | 20:33 |
bauzas | mriedem: yup | 20:33 |
mriedem | non-core specs reviewers can help us regardless... | 20:33 |
mriedem | i could do friday | 20:34 |
bauzas | mriedem: a review day is good for me because I'll leave other reviews and do the needed ones | 20:34 |
*** bvanhav__ has quit IRC | 20:34 | |
bauzas | mriedem: and of course, it could also be maybe explaining to the owners that they need to look at their specs by this day | 20:35 |
mriedem | friday in general just kind of sucks b/c it's a low energy day for the most part | 20:35 |
* bauzas nods | 20:38 | |
*** oanson has quit IRC | 20:41 | |
*** READ10 has quit IRC | 20:41 | |
*** matrohon has joined #openstack-nova | 20:41 | |
*** armax has joined #openstack-nova | 20:47 | |
*** tyrola has quit IRC | 20:50 | |
*** lpetrut has joined #openstack-nova | 20:53 | |
*** lpetrut has quit IRC | 20:57 | |
dansmith | bauzas: so we have to pass a (legacy) requestspec to the _schedule_instances() method, so if I pass one per instance, I have to just grab the first one and use that.. is that what you want? | 20:57 |
dansmith | I mean that's effectively what was going on before, so I guess yes, | 20:58 |
dansmith | and we just get them all over RPC that way | 20:58 |
dansmith | in case they later diverge in some way | 20:58 |
*** dimtruck is now known as zz_dimtruck | 20:59 | |
bauzas | dansmith: so in general, I was passing the object in the RPC cast, and then using the primitiving method for getting the legacy dict | 20:59 |
dansmith | I know, that's what is happening now | 20:59 |
dansmith | that's not really what I'm asking | 20:59 |
bauzas | sorry then | 20:59 |
bauzas | I probably misunderstood your question | 20:59 |
bauzas | you're asking if we should only pass one object? | 21:00 |
dansmith | no | 21:00 |
dansmith | nevermind | 21:00 |
dansmith | I think I know what to do, you can review it when it's up | 21:00 |
bauzas | okay, sorry :( | 21:00 |
dansmith | it's okay | 21:01 |
dansmith | saying it out loud made it semi-obvious to me | 21:02 |
*** kfarr has joined #openstack-nova | 21:03 | |
*** annegentle has quit IRC | 21:03 | |
*** ayoung has quit IRC | 21:04 | |
*** annegentle has joined #openstack-nova | 21:04 | |
*** Zer0Byte__ has quit IRC | 21:05 | |
*** slaweq_ has quit IRC | 21:06 | |
*** Zer0Byte__ has joined #openstack-nova | 21:06 | |
*** slaweq_ has joined #openstack-nova | 21:06 | |
*** ijw has joined #openstack-nova | 21:08 | |
mriedem | jaypipes: you should take a look at this spec https://review.openstack.org/#/c/182242/ | 21:08 |
mriedem | jaypipes: really worries me about how much you have to thread that needle to even get it to work correctly | 21:08 |
mriedem | and there is a whole crazy assumption about how the admin has configured the cloud which the user needs to know about to do this properly | 21:08 |
mriedem | i guess that just all gets hidden in vendor docs | 21:08 |
mriedem | dansmith: you too ^ | 21:08 |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Update hacking version https://review.openstack.org/389338 | 21:09 |
*** moshele has joined #openstack-nova | 21:10 | |
dansmith | murgh. | 21:10 |
dansmith | bauzas: if we get filter_properties from the reqspec, can we just remove passing that through the new rpc method altogether? seems like we must be able to, but you didn't say that in the review | 21:12 |
bauzas | dansmith: yup, I thought I commented that | 21:13 |
bauzas | dansmith: yes, please, the filter_props dict is actually merged with the other legacy request_spec dict into the object | 21:13 |
dansmith | bauzas: well, you said something that implied it I guess, just wanted to be sure | 21:14 |
bauzas | dansmith: sorry it was in the other change https://review.openstack.org/#/c/367557/15/nova/conductor/api.py | 21:15 |
dansmith | bauzas: ah, sure enough :) | 21:15 |
bauzas | that's why I need a specless BP for the follow-ups about the ReqSpec objects, because I'll need to touch the RPC API to deprecate those old params | 21:16 |
*** Apoorva has quit IRC | 21:17 | |
*** moshele has quit IRC | 21:17 | |
*** jamielennox is now known as jamielennox|away | 21:19 | |
*** smatzek has quit IRC | 21:23 | |
*** ayoung has joined #openstack-nova | 21:27 | |
dansmith | bauzas: http://pastebin.com/trSgkNy7 \o/ | 21:27 |
bauzas | dansmith: LGTM :) | 21:29 |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Update hacking version https://review.openstack.org/389338 | 21:33 |
*** eharney has quit IRC | 21:34 | |
*** Jeffrey4l has quit IRC | 21:35 | |
*** Zer0Byte__ has quit IRC | 21:36 | |
*** tblakes has quit IRC | 21:37 | |
*** cleong has quit IRC | 21:38 | |
*** ijw has quit IRC | 21:38 | |
*** lpetrut has joined #openstack-nova | 21:38 | |
*** tblakes has joined #openstack-nova | 21:38 | |
*** yamahata has joined #openstack-nova | 21:41 | |
*** Apoorva has joined #openstack-nova | 21:43 | |
*** jamesdenton has quit IRC | 21:44 | |
*** baoli has quit IRC | 21:45 | |
*** rtheis has quit IRC | 21:51 | |
*** tbachman has joined #openstack-nova | 21:52 | |
*** sree has joined #openstack-nova | 21:53 | |
openstackgerrit | Diana Clarke proposed openstack/python-novaclient: Microversion 2.39 - Simple tenant usage pagination https://review.openstack.org/394653 | 21:55 |
*** jheroux has quit IRC | 21:55 | |
openstackgerrit | Diana Clarke proposed openstack/python-novaclient: Microversion 2.39 - Simple tenant usage pagination https://review.openstack.org/394653 | 21:56 |
openstackgerrit | edan david proposed openstack/nova: User-controlled SR-IOV ports allocation https://review.openstack.org/374151 | 21:57 |
*** sree has quit IRC | 21:58 | |
*** ijw has joined #openstack-nova | 21:58 | |
*** lpetrut has quit IRC | 22:00 | |
*** shuquan_ has joined #openstack-nova | 22:01 | |
*** armax has quit IRC | 22:01 | |
*** gouthamr has quit IRC | 22:01 | |
*** jamielennox|away is now known as jamielennox | 22:02 | |
*** ijw has quit IRC | 22:03 | |
*** shuquan_ has quit IRC | 22:05 | |
*** Zer0Byte__ has joined #openstack-nova | 22:05 | |
*** yamahata has quit IRC | 22:06 | |
*** Zer0Byte__ has quit IRC | 22:07 | |
*** annegentle has quit IRC | 22:10 | |
*** thorst_ has quit IRC | 22:10 | |
*** annegentle has joined #openstack-nova | 22:12 | |
openstackgerrit | Dan Smith proposed openstack/nova: Return build_requests instead of instances https://review.openstack.org/374267 | 22:13 |
openstackgerrit | Dan Smith proposed openstack/nova: Add schedule_and_build_instances conductor method https://review.openstack.org/367557 | 22:13 |
openstackgerrit | Dan Smith proposed openstack/nova: Refactor two nearly useless secgroup tests https://review.openstack.org/393438 | 22:13 |
openstackgerrit | Dan Smith proposed openstack/nova: Add CellDatabase fixture https://review.openstack.org/392925 | 22:13 |
openstackgerrit | Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | 22:13 |
openstackgerrit | Dan Smith proposed openstack/nova: Store security groups in RequestSpec https://review.openstack.org/394656 | 22:13 |
*** edmondsw has quit IRC | 22:15 | |
*** ijw has joined #openstack-nova | 22:20 | |
*** krtaylor has quit IRC | 22:22 | |
*** nherciu has quit IRC | 22:24 | |
*** ijw has quit IRC | 22:24 | |
*** tblakes has quit IRC | 22:28 | |
*** mvk has joined #openstack-nova | 22:29 | |
*** matrohon has quit IRC | 22:31 | |
*** zz_dimtruck is now known as dimtruck | 22:33 | |
*** annegentle has quit IRC | 22:34 | |
*** victorhe_ has joined #openstack-nova | 22:36 | |
*** fragatin_ has joined #openstack-nova | 22:37 | |
*** eglynn has quit IRC | 22:38 | |
*** victorhe has quit IRC | 22:39 | |
*** fragatina has quit IRC | 22:40 | |
*** ijw has joined #openstack-nova | 22:40 | |
*** victorhe_ has quit IRC | 22:41 | |
*** mdrabe has quit IRC | 22:42 | |
*** claudiub has quit IRC | 22:44 | |
*** ijw has quit IRC | 22:48 | |
*** victorhe has joined #openstack-nova | 22:48 | |
*** mlavalle has quit IRC | 22:50 | |
openstackgerrit | John Garbutt proposed openstack/nova-specs: Add spec to use Cinder's new attachment APIs https://review.openstack.org/373203 | 22:50 |
*** kylek3h has quit IRC | 22:51 | |
*** victorhe has quit IRC | 22:52 | |
*** mlavalle has joined #openstack-nova | 22:53 | |
mriedem | johnthetubaguy: no you weren't supposed to do it during a meeting and late at night!!! | 22:56 |
*** mriedem has quit IRC | 22:57 | |
*** oomichi_ has quit IRC | 22:58 | |
*** oomichi_afk has joined #openstack-nova | 22:58 | |
*** rrecio_ has quit IRC | 23:01 | |
*** rrecio has joined #openstack-nova | 23:01 | |
openstackgerrit | Kevin Benton proposed openstack/nova: Pass MTU into os-vif Network object https://review.openstack.org/370679 | 23:02 |
openstackgerrit | Kevin Benton proposed openstack/nova: Revert MTU hacks for bug 1623876 https://review.openstack.org/394669 | 23:02 |
openstack | bug 1623876 in OpenStack Compute (nova) "nova is not setting the MTU provided by Neutron" [High,Fix released] https://launchpad.net/bugs/1623876 - Assigned to Kevin Benton (kevinbenton) | 23:02 |
*** lamt has quit IRC | 23:10 | |
*** baoli has joined #openstack-nova | 23:11 | |
*** ijw has joined #openstack-nova | 23:12 | |
*** esberglu has quit IRC | 23:12 | |
*** esberglu has joined #openstack-nova | 23:13 | |
*** esberglu has quit IRC | 23:18 | |
*** tbachman has quit IRC | 23:18 | |
*** ijw has quit IRC | 23:19 | |
*** ijw has joined #openstack-nova | 23:19 | |
*** ducttape_ has quit IRC | 23:23 | |
*** pradk has quit IRC | 23:25 | |
*** esberglu has joined #openstack-nova | 23:26 | |
*** esberglu has quit IRC | 23:30 | |
*** ayoung has quit IRC | 23:31 | |
*** Jeffrey4l has joined #openstack-nova | 23:31 | |
*** cdelatte has quit IRC | 23:33 | |
*** Guest90996 has quit IRC | 23:34 | |
*** ayoung has joined #openstack-nova | 23:34 | |
*** armax has joined #openstack-nova | 23:36 | |
*** tbachman has joined #openstack-nova | 23:39 | |
*** xyang1 has quit IRC | 23:43 | |
*** mlavalle has quit IRC | 23:43 | |
*** victorhe has joined #openstack-nova | 23:44 | |
*** rrecio has quit IRC | 23:46 | |
*** thorst_ has joined #openstack-nova | 23:47 | |
*** victorhe has quit IRC | 23:49 | |
*** esberglu has joined #openstack-nova | 23:49 | |
*** ayoung has quit IRC | 23:49 | |
*** gibi_ has quit IRC | 23:56 | |
*** gibi has joined #openstack-nova | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!