| *** sdague has quit IRC | 00:01 | |
| *** claudiub has quit IRC | 00:10 | |
| *** markvoelker has joined #openstack-nova | 00:11 | |
| *** itlinux has joined #openstack-nova | 00:14 | |
| *** itlinux has quit IRC | 00:15 | |
| *** awaugama has joined #openstack-nova | 00:19 | |
| *** bigdogstl has joined #openstack-nova | 00:19 | |
| *** itlinux has joined #openstack-nova | 00:21 | |
| *** bigdogstl has quit IRC | 00:25 | |
| *** yangyapeng has quit IRC | 00:27 | |
| *** fragatina has joined #openstack-nova | 00:31 | |
| *** hiro-kobayashi has joined #openstack-nova | 00:35 | |
| *** armax has joined #openstack-nova | 00:36 | |
| *** yamamoto has joined #openstack-nova | 00:36 | |
| *** fragatina has quit IRC | 00:37 | |
| *** Dinesh_Bhor has joined #openstack-nova | 00:39 | |
| *** READ10 has quit IRC | 00:40 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Stop globally caching host states in scheduler HostManager https://review.openstack.org/533371 | 00:40 |
|---|---|---|
| *** fragatina has joined #openstack-nova | 00:40 | |
| *** Swami_ has quit IRC | 00:40 | |
| *** Dinesh_Bhor has quit IRC | 00:41 | |
| *** tbachman has joined #openstack-nova | 00:41 | |
| *** Dinesh_Bhor has joined #openstack-nova | 00:43 | |
| *** bigdogstl has joined #openstack-nova | 00:51 | |
| *** Dinesh_Bhor has quit IRC | 00:51 | |
| *** fragatina has quit IRC | 00:52 | |
| *** Dinesh_Bhor has joined #openstack-nova | 00:52 | |
| *** Swami_ has joined #openstack-nova | 00:54 | |
| tonyb | bauzas: Oh then you need a verion of noce that works with old and new versions of os-vif | 00:57 |
| *** awaugama has quit IRC | 01:00 | |
| *** yangyapeng has joined #openstack-nova | 01:00 | |
| *** phuongnh has joined #openstack-nova | 01:03 | |
| *** zhaochao has joined #openstack-nova | 01:03 | |
| *** liuzz has joined #openstack-nova | 01:03 | |
| *** jichen has joined #openstack-nova | 01:04 | |
| *** yangyapeng has quit IRC | 01:04 | |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949 | 01:05 |
| *** bigdogstl has quit IRC | 01:07 | |
| *** mvk has quit IRC | 01:10 | |
| *** liverpooler has joined #openstack-nova | 01:13 | |
| *** yangyapeng has joined #openstack-nova | 01:14 | |
| *** mdnadeem has joined #openstack-nova | 01:22 | |
| openstackgerrit | Chen Hanxiao proposed openstack/nova master: libvirt: guest: introduce blockStats instead of domain.blockStats https://review.openstack.org/526833 | 01:24 |
| *** mvk has joined #openstack-nova | 01:25 | |
| *** pmannidi has joined #openstack-nova | 01:29 | |
| *** Swami_ has quit IRC | 01:30 | |
| *** hiro-kobayashi has quit IRC | 01:33 | |
| *** sapd__ has joined #openstack-nova | 01:43 | |
| *** sapd_ has quit IRC | 01:43 | |
| openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.60 - Add trusted_image_certificates https://review.openstack.org/500396 | 01:46 |
| *** zhurong has joined #openstack-nova | 01:51 | |
| *** hongbin has joined #openstack-nova | 01:56 | |
| *** masber has joined #openstack-nova | 02:00 | |
| *** bigdogstl has joined #openstack-nova | 02:00 | |
| *** threestrands_ has joined #openstack-nova | 02:02 | |
| *** masuberu has quit IRC | 02:02 | |
| *** jappleii__ has quit IRC | 02:04 | |
| *** bigdogstl has quit IRC | 02:05 | |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.60 - List/Show all server migration types https://review.openstack.org/430839 | 02:06 |
| *** annp has joined #openstack-nova | 02:07 | |
| *** bigdogstl has joined #openstack-nova | 02:08 | |
| *** bigdogstl has quit IRC | 02:17 | |
| *** suresh12 has quit IRC | 02:18 | |
| *** chyka_ has quit IRC | 02:25 | |
| *** edmondsw has quit IRC | 02:34 | |
| *** phuongnh has quit IRC | 02:45 | |
| *** phuongnh has joined #openstack-nova | 02:45 | |
| *** phuongnh has quit IRC | 02:50 | |
| *** phuongnh has joined #openstack-nova | 02:51 | |
| *** bigdogstl has joined #openstack-nova | 02:52 | |
| *** Dinesh_Bhor has quit IRC | 02:53 | |
| *** hongbin has quit IRC | 02:53 | |
| *** hongbin has joined #openstack-nova | 02:54 | |
| *** Dinesh_Bhor has joined #openstack-nova | 02:55 | |
| *** suresh12 has joined #openstack-nova | 02:58 | |
| *** bkopilov has quit IRC | 02:58 | |
| *** bigdogstl has quit IRC | 02:58 | |
| *** Dinesh_Bhor has quit IRC | 03:00 | |
| *** suresh12 has quit IRC | 03:02 | |
| *** bigdogstl has joined #openstack-nova | 03:04 | |
| *** markvoelker has quit IRC | 03:13 | |
| *** nicolasbock has quit IRC | 03:15 | |
| *** sree has joined #openstack-nova | 03:15 | |
| *** hongbin has quit IRC | 03:19 | |
| *** bigdogstl has quit IRC | 03:19 | |
| *** hongbin has joined #openstack-nova | 03:20 | |
| *** sree has quit IRC | 03:29 | |
| *** sree has joined #openstack-nova | 03:29 | |
| *** sree has quit IRC | 03:30 | |
| *** sree has joined #openstack-nova | 03:30 | |
| *** takashin has quit IRC | 03:31 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/533046 | 03:36 |
| *** yamamoto_ has joined #openstack-nova | 03:39 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/533918 | 03:43 |
| *** yamamoto has quit IRC | 03:43 | |
| *** jmlowe has quit IRC | 03:44 | |
| *** bigdogstl has joined #openstack-nova | 03:46 | |
| *** penick has joined #openstack-nova | 03:47 | |
| *** jaianshu has joined #openstack-nova | 03:47 | |
| *** edmondsw has joined #openstack-nova | 03:47 | |
| *** hoangcx has quit IRC | 03:48 | |
| *** mlavalle has quit IRC | 03:48 | |
| *** hoangcx has joined #openstack-nova | 03:48 | |
| *** bigdogstl has quit IRC | 03:51 | |
| *** namnh has joined #openstack-nova | 03:52 | |
| *** edmondsw has quit IRC | 03:52 | |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 03:55 |
| *** abhishekk has joined #openstack-nova | 03:59 | |
| *** penick has quit IRC | 04:05 | |
| *** tanee has quit IRC | 04:05 | |
| *** links has joined #openstack-nova | 04:06 | |
| *** dave-mccowan has quit IRC | 04:15 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform instance.resize_confirm notification https://review.openstack.org/482557 | 04:19 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/533046 | 04:20 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/533046 | 04:20 |
| *** elmaciej has quit IRC | 04:20 | |
| *** zhurong has quit IRC | 04:22 | |
| *** owalsh_ has joined #openstack-nova | 04:24 | |
| *** SamYaple has quit IRC | 04:24 | |
| *** SamYaple has joined #openstack-nova | 04:24 | |
| *** bkopilov has joined #openstack-nova | 04:25 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/533994 | 04:25 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/533918 | 04:25 |
| *** bigdogstl has joined #openstack-nova | 04:26 | |
| *** owalsh has quit IRC | 04:27 | |
| *** rano has joined #openstack-nova | 04:32 | |
| rano | hi.....how can i check for nova logs | 04:35 |
| rano | ? | 04:35 |
| *** andreas_s has joined #openstack-nova | 04:35 | |
| *** vishu1810 has joined #openstack-nova | 04:35 | |
| *** karthiks has joined #openstack-nova | 04:36 | |
| *** bigdogstl has quit IRC | 04:37 | |
| *** penick has joined #openstack-nova | 04:37 | |
| rano | andreas_s: where i can see the nova logs ? | 04:38 |
| rano | can you help me out | 04:38 |
| rano | ? | 04:38 |
| rano | while using nova cli | 04:39 |
| *** andreas_s has quit IRC | 04:40 | |
| *** bzhao__ has quit IRC | 04:40 | |
| *** vishu1810 has quit IRC | 04:41 | |
| *** gyee has quit IRC | 04:42 | |
| jichen | use nova --debug xxx to print more logs and go to host running nova service, check logs ,sometimes it locates at /var/log/nova/* | 04:44 |
| *** bigdogstl has joined #openstack-nova | 04:46 | |
| *** Damini has joined #openstack-nova | 04:50 | |
| *** bigdogstl has quit IRC | 04:51 | |
| *** penick has quit IRC | 04:53 | |
| *** ratailor has joined #openstack-nova | 04:54 | |
| *** whoami-rajat has joined #openstack-nova | 04:56 | |
| *** whoami-rajat has left #openstack-nova | 04:56 | |
| *** bigdogstl has joined #openstack-nova | 04:58 | |
| Spazmotic | Morning... evening..whatever | 04:58 |
| rano | Spazmotic: hi morning | 05:00 |
| rano | Spazmotic: can you help me in find the nova logs | 05:01 |
| rano | ? | 05:01 |
| rano | i am new to nova | 05:01 |
| *** Dinesh_Bhor has joined #openstack-nova | 05:01 | |
| Spazmotic | Like running nova services in an env, or while doing development? | 05:02 |
| Spazmotic | Like jichen mentioned, depending on the service, they'll usually be located in /var/log/nova, or you can run the --debugs | 05:02 |
| rano | Spazmotic: AIO devstack | 05:02 |
| *** cburgess has quit IRC | 05:02 | |
| *** bigdogstl has quit IRC | 05:03 | |
| rano | can you help me with CLI that can be use to see | 05:03 |
| Spazmotic | the nova client CLI? | 05:03 |
| Spazmotic | What do you need to do? | 05:03 |
| *** Yikun has quit IRC | 05:04 | |
| *** hongbin has quit IRC | 05:04 | |
| *** Yikun has joined #openstack-nova | 05:04 | |
| rano | Spazmotic: I am running the `openstack aggregate list` after installation of devstack | 05:04 |
| rano | and want to check the logs of fired CLI | 05:04 |
| rano | that is what i am trying to do | 05:05 |
| Spazmotic | So you're running aggregate list and want to see it running that action directly in the logs, i guess for more understanding of what all it hits? | 05:05 |
| rano | yes exactly | 05:05 |
| rano | Spazmotic: | 05:05 |
| *** Yikun has quit IRC | 05:06 | |
| Spazmotic | so if you run the --debug command with the openstack client or just use nova, it will show you a pretty nice breakdown of what it calls.. but if you want to get more granular than that you will need a sharper understanding of nova itself | 05:06 |
| Spazmotic | Somoene here may be more familiar with the aggregate code, so i'm not 100% sure what all it hits.. the logs you would check would vary depending. | 05:07 |
| *** Yikun has joined #openstack-nova | 05:07 | |
| *** Dinesh_Bhor has quit IRC | 05:07 | |
| Spazmotic | A good place to start would be nova api code for that action and see where it bounces off to. | 05:07 |
| *** Dinesh_Bhor has joined #openstack-nova | 05:07 | |
| *** cburgess has joined #openstack-nova | 05:08 | |
| *** janki has joined #openstack-nova | 05:10 | |
| *** cburgess has quit IRC | 05:12 | |
| *** suresh12 has joined #openstack-nova | 05:12 | |
| *** cburgess has joined #openstack-nova | 05:13 | |
| *** markvoelker has joined #openstack-nova | 05:14 | |
| Spazmotic | rano: Just a quick look through nova API and into compute API.. it pushes the get_aggregatre_list through here and then moves into objects -- https://github.com/openstack/nova/blob/master/nova/compute/api.py#L4815-L4817 | 05:15 |
| Spazmotic | Be careful moving through conductor you'll get black holed.. if it sends it to conductor generally just look in nova.compute.manager.py | 05:15 |
| Spazmotic | good luck :) | 05:15 |
| Spazmotic | or honestly just run the command then "ls -lstrh" on the log dir to see which ones change and peruse them | 05:17 |
| *** markvoelker has quit IRC | 05:18 | |
| *** sree_ has joined #openstack-nova | 05:20 | |
| *** sree_ has quit IRC | 05:20 | |
| *** trinaths has joined #openstack-nova | 05:21 | |
| *** namnh_ has joined #openstack-nova | 05:22 | |
| rano | Spazmotic: thanks a lot for help | 05:22 |
| *** namnh_ has quit IRC | 05:22 | |
| rano | but i am unable to find the log directory | 05:22 |
| rano | :( | 05:22 |
| *** sree has quit IRC | 05:23 | |
| *** bigdogstl has joined #openstack-nova | 05:23 | |
| *** hoangcx has quit IRC | 05:25 | |
| *** namnh has quit IRC | 05:25 | |
| *** psachin has joined #openstack-nova | 05:26 | |
| *** bigdogstl has quit IRC | 05:30 | |
| *** claudiub has joined #openstack-nova | 05:30 | |
| *** bigdogstl has joined #openstack-nova | 05:42 | |
| *** markvoelker has joined #openstack-nova | 05:44 | |
| Spazmotic | gotta do some googling buddy. https://ask.openstack.org/en/question/13014/where-to-find-log-files-in-devstack-installation/ | 05:45 |
| *** markvoelker has quit IRC | 05:49 | |
| *** hieulq has quit IRC | 05:50 | |
| *** bigdogstl has quit IRC | 05:55 | |
| *** hoangcx has joined #openstack-nova | 06:00 | |
| *** lpetrut has joined #openstack-nova | 06:04 | |
| *** Dinesh_Bhor has quit IRC | 06:06 | |
| *** rmk has quit IRC | 06:06 | |
| *** armax has quit IRC | 06:08 | |
| ameeda | Morning :) | 06:08 |
| ameeda | gmann: please review the code, there is a reviewer give me -1, it seems that he doesn't understand the bug clearly. I don't change anything on metadata | 06:09 |
| gmann | ameeda: sure,i might check it tomorrow or in night. | 06:10 |
| ameeda | Thank you very much, so I don't need to change anything as jay said ? | 06:11 |
| *** hemna_ has quit IRC | 06:11 | |
| *** bigdogstl has joined #openstack-nova | 06:11 | |
| *** suresh12 has quit IRC | 06:13 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353 | 06:13 |
| *** moshele has joined #openstack-nova | 06:14 | |
| *** sridharg has joined #openstack-nova | 06:14 | |
| gmann | you should :), if you have any query on review comment you can reply. his comments seems valid. | 06:16 |
| *** Dinesh_Bhor has joined #openstack-nova | 06:17 | |
| gmann | ameeda: image service proxy API are now deprecated. i think if you fix the comment and test as he mentioned should be ok | 06:18 |
| gmann | ameeda: because that bug is little half scenario kind of, where nova store server metadata which is restricted to 255 by API layer but internally image might have >255 metadata value which are being stored in instance_system_metadata table | 06:19 |
| gmann | i all also check the review but later | 06:20 |
| *** whoami-rajat has joined #openstack-nova | 06:20 | |
| *** bigdogstl has quit IRC | 06:20 | |
| *** Tom-Tom has joined #openstack-nova | 06:21 | |
| *** Dinesh_Bhor has quit IRC | 06:22 | |
| *** Dinesh_Bhor has joined #openstack-nova | 06:22 | |
| *** harsha has joined #openstack-nova | 06:23 | |
| *** lajoskatona has joined #openstack-nova | 06:24 | |
| *** bigdogstl has joined #openstack-nova | 06:26 | |
| openstackgerrit | Nakanishi Tomotaka proposed openstack/nova master: Test Compute API in multiple cell https://review.openstack.org/534116 | 06:27 |
| ameeda | gmann: Thank you, I will fix that | 06:29 |
| *** mgoddard has quit IRC | 06:36 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353 | 06:41 |
| *** bigdogstl has quit IRC | 06:41 | |
| *** mgoddard has joined #openstack-nova | 06:42 | |
| *** sree has joined #openstack-nova | 06:55 | |
| alex_xu | ls | 07:05 |
| *** links has quit IRC | 07:06 | |
| *** lpetrut has quit IRC | 07:07 | |
| *** ragiman has joined #openstack-nova | 07:07 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector from ports/portgroups https://review.openstack.org/517322 | 07:09 |
| ratailor | alex_xu, could you pl review, https://review.openstack.org/#/c/498334/ | 07:10 |
| ratailor | alex_xu, it gets stuck in merge conflict every now and than. | 07:11 |
| *** lpetrut has joined #openstack-nova | 07:12 | |
| *** rcernin has quit IRC | 07:14 | |
| *** bigdogstl has joined #openstack-nova | 07:15 | |
| *** threestrands_ has quit IRC | 07:18 | |
| *** links has joined #openstack-nova | 07:22 | |
| *** bigdogstl has quit IRC | 07:23 | |
| *** edmondsw has joined #openstack-nova | 07:24 | |
| *** bigdogstl has joined #openstack-nova | 07:26 | |
| *** andreas_s has joined #openstack-nova | 07:27 | |
| *** edmondsw has quit IRC | 07:28 | |
| *** bigdogstl has quit IRC | 07:31 | |
| *** takashin has joined #openstack-nova | 07:33 | |
| *** slaweq has joined #openstack-nova | 07:36 | |
| *** slaweq has joined #openstack-nova | 07:36 | |
| *** janki has quit IRC | 07:39 | |
| *** links has quit IRC | 07:39 | |
| *** MarginHu has joined #openstack-nova | 07:40 | |
| *** markvoelker has joined #openstack-nova | 07:46 | |
| *** kwathore has joined #openstack-nova | 07:47 | |
| *** lpetrut has quit IRC | 07:49 | |
| *** sree has quit IRC | 07:56 | |
| *** dikonoor has joined #openstack-nova | 08:00 | |
| *** tesseract has joined #openstack-nova | 08:05 | |
| *** avolkov has joined #openstack-nova | 08:06 | |
| *** sree has joined #openstack-nova | 08:08 | |
| *** damien_r has joined #openstack-nova | 08:11 | |
| *** slaweq has quit IRC | 08:13 | |
| *** slaweq has joined #openstack-nova | 08:13 | |
| *** AlexeyAbashkin has joined #openstack-nova | 08:15 | |
| *** links has joined #openstack-nova | 08:15 | |
| *** kholkina has joined #openstack-nova | 08:16 | |
| *** bigdogstl has joined #openstack-nova | 08:18 | |
| *** slaweq has quit IRC | 08:18 | |
| *** markvoelker has quit IRC | 08:19 | |
| *** sahid has joined #openstack-nova | 08:21 | |
| *** matrohon has joined #openstack-nova | 08:22 | |
| *** sree_ has joined #openstack-nova | 08:22 | |
| *** sree_ is now known as Guest72548 | 08:22 | |
| *** sree has quit IRC | 08:25 | |
| *** Guest72548 has quit IRC | 08:25 | |
| *** sree has joined #openstack-nova | 08:26 | |
| *** TTris has quit IRC | 08:28 | |
| *** bigdogstl has quit IRC | 08:29 | |
| *** sree has quit IRC | 08:32 | |
| *** slaweq has joined #openstack-nova | 08:33 | |
| *** sree has joined #openstack-nova | 08:34 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: reno for notification-transformation-queens https://review.openstack.org/518018 | 08:35 |
| *** tuanla____ has joined #openstack-nova | 08:38 | |
| *** sree has quit IRC | 08:39 | |
| *** sree has joined #openstack-nova | 08:40 | |
| *** owalsh_ is now known as owalsh | 08:43 | |
| *** sree has quit IRC | 08:45 | |
| *** janki has joined #openstack-nova | 08:46 | |
| *** jpena|off is now known as jpena | 08:46 | |
| *** janki has quit IRC | 08:46 | |
| *** janki has joined #openstack-nova | 08:47 | |
| gibi | cores, I think this is a fix for a problematic master stability issue, could you check it? https://review.openstack.org/#/c/533210/ | 08:48 |
| *** moshele has left #openstack-nova | 08:49 | |
| *** moshele has joined #openstack-nova | 08:49 | |
| *** sree has joined #openstack-nova | 08:49 | |
| *** cdent has joined #openstack-nova | 08:49 | |
| *** bigdogstl has joined #openstack-nova | 08:51 | |
| *** sree has quit IRC | 08:52 | |
| *** sree has joined #openstack-nova | 08:53 | |
| *** sree has quit IRC | 08:53 | |
| gmann | gibi: +1, if you want you can optimism it more. | 08:55 |
| *** Drankis has joined #openstack-nova | 08:55 | |
| ratailor | Can anybody provide steps to create live-migration setup ? | 08:56 |
| *** bigdogstl has quit IRC | 08:56 | |
| ratailor | I have created with one all-in-one controller and 1 compute node, and setup ssh keys as mentioned here. https://docs.openstack.org/devstack/latest/guides/multinode-lab.html#live-migration | 08:56 |
| ratailor | But I couldn't live-migrate vm from one node to other. | 08:57 |
| openstackgerrit | Merged openstack/nova master: Transform rescue/unrescue instance notifications https://review.openstack.org/385644 | 08:58 |
| *** priteau has joined #openstack-nova | 08:59 | |
| openstackgerrit | Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900 | 09:00 |
| *** ttsiouts has quit IRC | 09:03 | |
| *** ttsiouts has joined #openstack-nova | 09:03 | |
| gibi | gmann: thanks, I replied inline | 09:03 |
| *** sree has joined #openstack-nova | 09:04 | |
| *** hieulq has joined #openstack-nova | 09:08 | |
| gmann | gibi: is it? irrelevant-files override should work in inherited jobs. | 09:08 |
| gibi | gmann: it doesn't work. Let me dig out the infra log | 09:09 |
| gmann | i am trying to push irrelevant-files on base jobs like legacy dvsm etc which can be issue then | 09:09 |
| gibi | gmann: discussion starts here http://eavesdrop.openstack.org/irclogs/%23openstack-infra/%23openstack-infra.2018-01-12.log.html#t2018-01-12T14:50:01 | 09:10 |
| gmann | i have not tired the overridden scenario but with discussion with clark long back it should. | 09:10 |
| *** sree has quit IRC | 09:12 | |
| *** edmondsw has joined #openstack-nova | 09:12 | |
| ratailor | I am getting below error in n-cpu logs: | 09:13 |
| ratailor | Jan 16 07:42:16 devstack-compute nova-compute: File "/usr/lib64/python2.7/site-packages/libvirt.py", line 1702, in migrateToURI3 | 09:13 |
| ratailor | Jan 16 07:42:16 devstack-compute nova-compute: if ret == -1: raise libvirtError ('virDomainMigrateToURI3() failed', dom=self) | 09:13 |
| ratailor | Jan 16 07:42:16 devstack-compute nova-compute: libvirtError: internal error: unable to execute QEMU command 'drive-mirror': Failed to connect socket: Connection timed out | 09:13 |
| kashyap | ratailor: Can you successfully repeat it? | 09:14 |
| kashyap | Is it with today's Git master? | 09:14 |
| kashyap | ratailor: That error means, QEMU has quit. | 09:14 |
| *** sree has joined #openstack-nova | 09:15 | |
| gmann | gibi: i think that issue was different where auto converted job definition caused that bug. irrelevant-files is defined as regex text in zuulv3 and it override among job inheritance. | 09:16 |
| *** matrohon has quit IRC | 09:17 | |
| *** edmondsw has quit IRC | 09:17 | |
| openstackgerrit | Michał Dulko proposed openstack/os-vif master: Revert "Move 'ips' field from Subnet object to VIF object" https://review.openstack.org/534206 | 09:18 |
| kashyap | ratailor: You can see what happens in the libvirtd.log | 09:18 |
| gibi | gmann: OK, I will try what you suggest to see if in the in tree definition the override works | 09:18 |
| *** bigdogstl has joined #openstack-nova | 09:20 | |
| *** mvk has quit IRC | 09:20 | |
| gmann | gibi: thanks, i am trying to fill base job with basic irrelevant-files so that they can be overridden if needed like this- https://review.openstack.org/#/c/531194/ | 09:20 |
| *** ralonsoh has joined #openstack-nova | 09:20 | |
| *** sree has quit IRC | 09:21 | |
| *** Dinesh_Bhor has quit IRC | 09:24 | |
| ratailor | kashyap, let me try it again. | 09:25 |
| *** Dinesh_Bhor has joined #openstack-nova | 09:25 | |
| *** takashin has left #openstack-nova | 09:30 | |
| *** bigdogstl has quit IRC | 09:31 | |
| ildikov | johnthetubaguy: hi | 09:32 |
| ildikov | johnthetubaguy: if you could go through the multi-attach series rather the first half of this week that would be great so I can fix things before the gate gets impossible | 09:32 |
| ildikov | johnthetubaguy: series starts here: https://review.openstack.org/#/c/267587/ | 09:33 |
| ildikov | johnthetubaguy: experimental job: https://review.openstack.org/#/c/532689/ | 09:33 |
| ildikov | johnthetubaguy: the multi-attach tests ran fine | 09:33 |
| *** slaweq_ has joined #openstack-nova | 09:35 | |
| *** lucas-afk is now known as lucasagomes | 09:36 | |
| *** slunkad has quit IRC | 09:36 | |
| ildikov | gibi: same pretty please goes to you for this one: https://review.openstack.org/#/c/271047/ :) | 09:37 |
| kashyap | mdbooth: I don't suppose this ever happened, did it? http://lists.openstack.org/pipermail/openstack-operators/2017-January/012540.html | 09:38 |
| kashyap | mdbooth: It is your proposal to remove "support for sparse LVM volumes from Nova" | 09:39 |
| mdbooth | kashyap: I think that was in the context of work which didn't land, so no | 09:39 |
| kashyap | I see. | 09:39 |
| *** slaweq_ has quit IRC | 09:41 | |
| *** derekh has joined #openstack-nova | 09:44 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: tests: fixes mock autospec usage https://review.openstack.org/447505 | 09:51 |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: tests: autospecs all the mock.patch usages https://review.openstack.org/470775 | 09:51 |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: hyper-v: autospec classes before they are instantiated https://review.openstack.org/342211 | 09:51 |
| *** ebbex has joined #openstack-nova | 09:53 | |
| *** mvk has joined #openstack-nova | 09:53 | |
| *** danpawlik has quit IRC | 09:57 | |
| *** MarginHu has quit IRC | 09:57 | |
| *** sahid has quit IRC | 09:57 | |
| *** sahid has joined #openstack-nova | 09:57 | |
| *** danpawlik has joined #openstack-nova | 09:57 | |
| *** slaweq_ has joined #openstack-nova | 09:58 | |
| *** Dinesh_Bhor has quit IRC | 09:58 | |
| *** Dinesh_Bhor has joined #openstack-nova | 09:58 | |
| *** bigdogstl has joined #openstack-nova | 10:00 | |
| *** Tom-Tom has quit IRC | 10:00 | |
| *** Damini has quit IRC | 10:02 | |
| *** matrohon has joined #openstack-nova | 10:02 | |
| *** Dinesh_Bhor has quit IRC | 10:03 | |
| *** slaweq_ has quit IRC | 10:03 | |
| *** elmaciej has joined #openstack-nova | 10:03 | |
| *** slunkad has joined #openstack-nova | 10:03 | |
| *** yangyapeng has quit IRC | 10:04 | |
| *** yangyapeng has joined #openstack-nova | 10:04 | |
| *** bhujay has joined #openstack-nova | 10:05 | |
| bhujay | Hello Everybody , any idea how to configure nova libvirt section for a FC shared storage instead of rbd | 10:05 |
| *** bigdogstl has quit IRC | 10:11 | |
| *** priteau has quit IRC | 10:12 | |
| openstackgerrit | Jacek Tomasiak proposed openstack/python-novaclient master: Fix listing of instances above API max_limit https://review.openstack.org/534222 | 10:13 |
| openstackgerrit | Jacek Tomasiak proposed openstack/python-novaclient master: Fix listing of instances above API max_limit https://review.openstack.org/534222 | 10:14 |
| *** markvoelker has joined #openstack-nova | 10:16 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Enable py36 unit tests in tox https://review.openstack.org/534223 | 10:17 |
| *** Tom-Tom has joined #openstack-nova | 10:20 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] use simple FaultWrapper https://review.openstack.org/533752 | 10:20 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Refactor WSGI apps and utils to limit imports https://review.openstack.org/533797 | 10:20 |
| *** Tom-Tom has quit IRC | 10:21 | |
| *** Tom-Tom has joined #openstack-nova | 10:21 | |
| *** ratailor has quit IRC | 10:21 | |
| *** matrohon has quit IRC | 10:23 | |
| *** AlexeyAbashkin has quit IRC | 10:23 | |
| *** ratailor has joined #openstack-nova | 10:23 | |
| *** Tom-Tom has quit IRC | 10:24 | |
| *** matrohon has joined #openstack-nova | 10:24 | |
| *** AlexeyAbashkin has joined #openstack-nova | 10:26 | |
| cdent | gibi: if you're around and have a moment, I could do with your input on https://review.openstack.org/#/c/533797/ It's passing all functional tests except for 8 in 'notification_sample_tests' and the cause is not immediately obvious (the zuul runs are out of date) | 10:27 |
| *** liuzz has quit IRC | 10:29 | |
| *** sree has joined #openstack-nova | 10:29 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: pass the mdevs when rebooting the guest https://review.openstack.org/533818 | 10:32 |
| *** Tom-Tom has joined #openstack-nova | 10:33 | |
| *** psachin has quit IRC | 10:37 | |
| *** andreas_s has quit IRC | 10:37 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: pass the mdevs when rebooting the guest https://review.openstack.org/533818 | 10:37 |
| sean-k-mooney | stephenfin: you about? | 10:38 |
| *** Tom-Tom has quit IRC | 10:38 | |
| *** andreas_s has joined #openstack-nova | 10:42 | |
| *** psachin has joined #openstack-nova | 10:43 | |
| *** gcb has joined #openstack-nova | 10:46 | |
| *** alexchadin has joined #openstack-nova | 10:47 | |
| *** dikonoor has quit IRC | 10:49 | |
| *** alexchadin has quit IRC | 10:49 | |
| *** alexchadin has joined #openstack-nova | 10:49 | |
| *** markvoelker has quit IRC | 10:50 | |
| *** chyka has joined #openstack-nova | 10:50 | |
| *** abhishekk has quit IRC | 10:51 | |
| *** andreas_s has quit IRC | 10:51 | |
| *** andreas_s has joined #openstack-nova | 10:53 | |
| *** chyka has quit IRC | 10:55 | |
| *** ratailor_ has joined #openstack-nova | 10:57 | |
| *** mvk has quit IRC | 10:58 | |
| *** ratailor has quit IRC | 10:59 | |
| *** dtantsur|afk is now known as dtantsur | 11:00 | |
| *** edmondsw has joined #openstack-nova | 11:00 | |
| *** andreas_s has quit IRC | 11:02 | |
| *** psachin has quit IRC | 11:04 | |
| *** harsha has quit IRC | 11:04 | |
| ameeda | Hi, Regarding this gerrit "https://review.openstack.org/#/c/528385/", when I run the functional test i got this kind of failurs "http://paste.openstack.org/show/645697/" , can you please guess me what I can do with ? | 11:05 |
| *** sdague has joined #openstack-nova | 11:05 | |
| *** edmondsw has quit IRC | 11:05 | |
| stephenfin | sean-k-mooney: I am | 11:05 |
| *** Guest55219 has quit IRC | 11:07 | |
| sean-k-mooney | stephenfin: i pushed a patch for the nova unit tests but i looks like pyrote2 may leak on python2 based on matthew thode's comment on https://review.openstack.org/#/c/532987/ | 11:07 |
| sean-k-mooney | https://github.com/svinota/pyroute2/pull/443 | 11:07 |
| openstackgerrit | nafeiyang proposed openstack/nova master: z/VM driver: add snapshot function https://review.openstack.org/534240 | 11:08 |
| stephenfin | sean-k-mooney: So that explains our "too many open files issue", I guess | 11:08 |
| sean-k-mooney | since rodolfo like to test thing when he wrote the pyroute 2 implementation he almost wrote a ip tools impl in the functional tests https://github.com/openstack/os-vif/blob/master/os_vif/tests/functional/internal/command/ip/test_impl_pyroute2.py#L28-L94 | 11:09 |
| *** bkopilov has quit IRC | 11:09 | |
| stephenfin | sean-k-mooney: So I guess we can revert/disable the pyroute2 path until a new version is released | 11:09 |
| sean-k-mooney | stephenfin: what i was thing was i could make that an actully conforming implementation to the api and we could merge and make it the defalut | 11:09 |
| stephenfin | We can't rely on that being released before the deadline | 11:09 |
| sean-k-mooney | well i was going to do it now. if i dont have it finished and up for review by this evening ill submit a reviert for rodolfos patch | 11:10 |
| stephenfin | sean-k-mooney: But why not keep what we had, in that case? The reason to switch to pyroute2 was that it used the netlink protocol in Python, rather than wrapping existing command-line tools | 11:11 |
| stephenfin | Before integrating pyroute2, we _were_ using subprocess | 11:11 |
| stephenfin | Well, processutils | 11:11 |
| openstackgerrit | Merged openstack/nova master: update_cell allows more than once cell to have the same db/transport url https://review.openstack.org/518556 | 11:12 |
| *** bigdogstl has joined #openstack-nova | 11:12 | |
| sean-k-mooney | yes though really the api is 3 fucntion add device delete device and set properties on device so its really small | 11:12 |
| *** mvk has joined #openstack-nova | 11:13 | |
| stephenfin | sean-k-mooney: Cool. If you get it finished then let me know and I'll review | 11:13 |
| *** psachin has joined #openstack-nova | 11:13 | |
| stephenfin | We can mark the processutils variant as immediately deprecated and switch to pyroute2 once a new version is released | 11:14 |
| sean-k-mooney | give me an hour or so and i should have a second driver working and a config option that default to iptools instead of pyroute 2 done. ill ping you the review when its up | 11:14 |
| gibi | cdent: I will check it later today | 11:14 |
| sean-k-mooney | stephenfin: ya that was my reasoning too | 11:15 |
| cdent | thanks gibi | 11:15 |
| cdent | If I have it figured out before then, I'll let you know | 11:15 |
| *** Brin has joined #openstack-nova | 11:15 | |
| *** Tom-Tom has joined #openstack-nova | 11:15 | |
| *** andreas_s has joined #openstack-nova | 11:16 | |
| *** sree has quit IRC | 11:18 | |
| *** bigdogstl has quit IRC | 11:19 | |
| *** Tom-Tom has quit IRC | 11:19 | |
| openstackgerrit | Jiri Suchomel proposed openstack/nova master: Retry few times when creating the initial connection to libvirtd. https://review.openstack.org/416928 | 11:20 |
| *** sree has joined #openstack-nova | 11:21 | |
| cdent | stephenfin: see my second set of comments on https://review.openstack.org/#/c/534223/ I don't think your suggestion works (at least not for me) | 11:23 |
| stephenfin | Oh, maybe not | 11:25 |
| * stephenfin checks | 11:25 | |
| *** sree has quit IRC | 11:25 | |
| *** sree has joined #openstack-nova | 11:26 | |
| *** TTris has joined #openstack-nova | 11:27 | |
| *** jichen has quit IRC | 11:27 | |
| *** yamamoto_ has quit IRC | 11:27 | |
| stephenfin | cdent: Right you are. I retract my -1 | 11:27 |
| *** matrohon has quit IRC | 11:28 | |
| cdent | stephenfin: do you still think I should change the default list? | 11:28 |
| stephenfin | cdent: I don't think so. We haven't switched to py36 in CIs yet (I think?), so it doesn't need to run by default | 11:28 |
| *** TTris has left #openstack-nova | 11:28 | |
| *** mgoddard_ has joined #openstack-nova | 11:34 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] use simple FaultWrapper https://review.openstack.org/533752 | 11:34 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Refactor WSGI apps and utils to limit imports https://review.openstack.org/533797 | 11:34 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: QEMU native LUKS decryption for volumes https://review.openstack.org/523958 | 11:36 |
| mdbooth | stephenfin: Please could you kick this back into the gate for me? https://review.openstack.org/#/c/529028/ Previously failed, then rebased. | 11:38 |
| stephenfin | mdbooth: Done | 11:38 |
| *** READ10 has joined #openstack-nova | 11:39 | |
| mdbooth | stephenfin: Thanks | 11:39 |
| mdbooth | Hmm, that patch series is starting to descend into merge conflict hell. | 11:40 |
| *** MarginHu has joined #openstack-nova | 11:43 | |
| *** READ10 has quit IRC | 11:44 | |
| *** mdnadeem has quit IRC | 11:48 | |
| *** oanson has quit IRC | 11:48 | |
| *** ratailor_ has quit IRC | 11:49 | |
| *** tbachman has quit IRC | 11:49 | |
| *** damien_r has quit IRC | 11:50 | |
| *** damien_r1 has joined #openstack-nova | 11:50 | |
| *** oanson has joined #openstack-nova | 11:50 | |
| *** ratailor has joined #openstack-nova | 11:55 | |
| *** phuongnh has quit IRC | 11:56 | |
| *** ratailor has quit IRC | 11:56 | |
| *** liverpooler has quit IRC | 11:57 | |
| *** edmondsw has joined #openstack-nova | 12:03 | |
| *** sree has quit IRC | 12:05 | |
| *** sree has joined #openstack-nova | 12:06 | |
| *** sambetts|afk is now known as sambetts | 12:06 | |
| *** mgoddard_ has quit IRC | 12:09 | |
| *** pmannidi has quit IRC | 12:09 | |
| *** annp has quit IRC | 12:10 | |
| *** oanson has quit IRC | 12:10 | |
| *** tuanla____ has quit IRC | 12:12 | |
| *** oanson has joined #openstack-nova | 12:12 | |
| *** HW-Peter has joined #openstack-nova | 12:12 | |
| * mdbooth sent a long message: mdbooth_2018-01-16_12:13:39.txt <https://matrix.devlab.redhat.com/_matrix/media/v1/download/redhat.com/ZOkiHajvlKvFxQUcknOXyfhJ> | 12:13 | |
| mdbooth | How do I hate thee? Let me count the ways. | 12:13 |
| mdbooth | Actually, I'm out of fingers :/ | 12:14 |
| *** yassine has joined #openstack-nova | 12:14 | |
| * mdbooth wonders if riot posted a link there... probably | 12:15 | |
| lyarwood | mdbooth: it did | 12:15 |
| lyarwood | * mdbooth sent a long message: mdbooth_2018-01-16_12:13:39.txt | 12:15 |
| lyarwood | <https://matrix.devlab.redhat.com/_matrix/media/v1/download/redhat.com/ZOkiHajvlKvFxQUcknOXyfhJ> | 12:15 |
| *** yassine is now known as Guest58384 | 12:15 | |
| mdbooth | lyarwood: Hehe, yeah. | 12:15 |
| * mdbooth will get a better one, cos it's a doozy | 12:15 | |
| *** bigdogstl has joined #openstack-nova | 12:16 | |
| *** HW-Peter has quit IRC | 12:16 | |
| mdbooth | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L7519 | 12:17 |
| *** HW-Peter has joined #openstack-nova | 12:17 | |
| mdbooth | There down to... | 12:17 |
| mdbooth | except Exception: pass | 12:17 |
| mdbooth | Yes, really | 12:17 |
| mdbooth | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L7519-L7530 | 12:17 |
| *** HWPeter has joined #openstack-nova | 12:17 | |
| *** Peter-HW has joined #openstack-nova | 12:18 | |
| *** Brin has quit IRC | 12:18 | |
| *** Peter-HW has quit IRC | 12:21 | |
| openstackgerrit | Lajos Katona proposed openstack/nova master: Deduplicate aggregate notification samples https://review.openstack.org/531162 | 12:21 |
| *** Peter-HW has joined #openstack-nova | 12:21 | |
| *** nicolasbock has joined #openstack-nova | 12:22 | |
| *** Peter-HW has quit IRC | 12:22 | |
| *** bigdogstl has quit IRC | 12:23 | |
| *** Peter-HW has joined #openstack-nova | 12:23 | |
| *** Peter-HW has quit IRC | 12:25 | |
| *** Peter-HW has joined #openstack-nova | 12:25 | |
| *** yamamoto has joined #openstack-nova | 12:26 | |
| *** Peter-HW has quit IRC | 12:26 | |
| *** Peter-HW has joined #openstack-nova | 12:27 | |
| *** Peter-HW has quit IRC | 12:29 | |
| *** lpetrut has joined #openstack-nova | 12:29 | |
| *** pmannidi has joined #openstack-nova | 12:32 | |
| *** smatzek has joined #openstack-nova | 12:32 | |
| *** lucasagomes is now known as lucas-hungry | 12:32 | |
| *** pcaruana has joined #openstack-nova | 12:33 | |
| *** jpena is now known as jpena|lunch | 12:35 | |
| cdent | Does anyone recall if nova/objects/ec2.py is used any more. It seems only in tests | 12:36 |
| *** Peter-HW has joined #openstack-nova | 12:36 | |
| cdent | hard to be clear | 12:37 |
| *** bigdogstl has joined #openstack-nova | 12:38 | |
| *** HW_Peter has joined #openstack-nova | 12:39 | |
| cdent | (╯°□°）╯︵ ┻━┻ | 12:40 |
| *** trinaths has quit IRC | 12:43 | |
| *** bigdogstl has quit IRC | 12:43 | |
| *** Tom-Tom has joined #openstack-nova | 12:43 | |
| *** kolesovdv has joined #openstack-nova | 12:45 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/533918 | 12:45 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/533918 | 12:45 |
| *** markvoelker has joined #openstack-nova | 12:47 | |
| alex_xu | cdent: just found there still have a lot of returns by searching 'ec2' in the source code | 12:49 |
| cdent | alex_xu: yes, but at least some of that is dead (only some) | 12:49 |
| cdent | and it is hard to tell which | 12:49 |
| cdent | alex_xu: I'm trying to work on the "less imports" goal implied by this bug https://bugs.launchpad.net/nova/+bug/1743120 and it is ... messy | 12:50 |
| openstack | Launchpad bug 1743120 in OpenStack Compute (nova) "placement inadvertently imports many python modules it does not need" [Low,In progress] - Assigned to Chris Dent (cdent) | 12:50 |
| *** sree_ has joined #openstack-nova | 12:50 | |
| *** sree_ is now known as Guest80016 | 12:51 | |
| *** sree has quit IRC | 12:51 | |
| *** elmaciej has quit IRC | 12:51 | |
| alex_xu | cdent: yea, I guess nova/api/openstack/__init__.py is source of evil :) | 12:51 |
| cdent | i was able to get some of it: https://review.openstack.org/#/c/533797/ | 12:52 |
| cdent | registering objects is also a culprint | 12:52 |
| *** bigdogstl has joined #openstack-nova | 12:52 | |
| *** vladikr has quit IRC | 12:53 | |
| alex_xu | interesting, put it in my review list, I still have circle import in the nova api, hope to get some clue | 12:54 |
| *** yangyapeng has quit IRC | 12:56 | |
| *** tbachman has joined #openstack-nova | 12:56 | |
| cdent | alex_xu: another source of trouble is conf, it tends to imports lots of stuff that isn't always needed. but I reckon the main culprit is code in __init__.py being common | 12:56 |
| gibi | cdent: you have to rebase your patch https://review.openstack.org/#/c/533797 to have this revert in your history https://review.openstack.org/#/c/533190/ | 12:57 |
| alex_xu | cdent: yea...__init__.py... | 12:57 |
| cdent | gibi: ah cool, thanks | 12:58 |
| *** bigdogstl has quit IRC | 12:58 | |
| gibi | cdent: this was last | 12:58 |
| gibi | cdent: this was last Friday's fun | 12:58 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] use simple FaultWrapper https://review.openstack.org/533752 | 12:58 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Refactor WSGI apps and utils to limit imports https://review.openstack.org/533797 | 12:58 |
| *** Peter-HW has quit IRC | 12:59 | |
| *** HW_Peter has quit IRC | 12:59 | |
| *** HWPeter has quit IRC | 12:59 | |
| *** HW-Peter has quit IRC | 12:59 | |
| cdent | ah | 12:59 |
| *** Drankis has quit IRC | 12:59 | |
| *** HW-Peter has joined #openstack-nova | 12:59 | |
| *** weshay_PTO is now known as weshay | 13:01 | |
| cdent | yup, that fixed it gibi, thanks | 13:03 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Refactor WSGI apps and utils to limit imports https://review.openstack.org/533797 | 13:04 |
| cdent | I guess I started this branch longer ago than I thought | 13:04 |
| *** pmannidi has quit IRC | 13:05 | |
| *** vladikr has joined #openstack-nova | 13:06 | |
| *** mgoddard_ has joined #openstack-nova | 13:08 | |
| *** aloga has quit IRC | 13:09 | |
| *** aloga has joined #openstack-nova | 13:10 | |
| zioproto | sdague: ping, are you around ? | 13:15 |
| *** dtantsur is now known as dtantsur|brb | 13:17 | |
| *** lucas-hungry is now known as lucasagomes | 13:20 | |
| *** markvoelker has quit IRC | 13:20 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Make sure that functional test triggered on sample changes https://review.openstack.org/533210 | 13:21 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Deduplicate aggregate notification samples https://review.openstack.org/531162 | 13:21 |
| *** pchavva has joined #openstack-nova | 13:21 | |
| bauzas | folks loving to see how vGPUs are possible with libvirt, the series is up for reviews https://review.openstack.org/#/c/528832/10 | 13:21 |
| *** yangyapeng has joined #openstack-nova | 13:22 | |
| bauzas | there are still some corner cases (at least for suspend and migrations), but that's something I'll provide by next changes | 13:22 |
| *** sdague has quit IRC | 13:22 | |
| *** sdague has joined #openstack-nova | 13:22 | |
| bauzas | gibi: stephenfin: in case you're interested in ^ | 13:22 |
| *** bigdogstl has joined #openstack-nova | 13:23 | |
| sdague | zioproto: for a little bit, what's up? | 13:23 |
| openstackgerrit | Merged openstack/os-vif master: Git ignore .stestr https://review.openstack.org/533699 | 13:23 |
| *** liverpooler has joined #openstack-nova | 13:23 | |
| *** elmaciej has joined #openstack-nova | 13:26 | |
| *** lpetrut_ has joined #openstack-nova | 13:26 | |
| *** elmaciej has quit IRC | 13:26 | |
| *** bigdogstl has quit IRC | 13:28 | |
| *** lpetrut has quit IRC | 13:28 | |
| *** xnox has quit IRC | 13:28 | |
| *** mvenesio has joined #openstack-nova | 13:30 | |
| *** bhujay has quit IRC | 13:32 | |
| *** yamamoto_ has joined #openstack-nova | 13:32 | |
| zioproto | sdague: hey, you have been metioned on openstack-dev, make sure you look at the thread [oslo][oslo.log] JSON logs are missing the request ID | 13:32 |
| *** bigdogstl has joined #openstack-nova | 13:33 | |
| zioproto | sdague: please read the thread (5 emails so far) and tell me if you have some important info for me :) | 13:33 |
| zioproto | sdague: thanks ! | 13:33 |
| *** markvoelker has joined #openstack-nova | 13:33 | |
| mvenesio | Hi guys, i'm trying to configure nova and rabbitmq to use SSL with trusted certificates, but doing the configuration as is in the ssl security guide im getting the "[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed" error when nova tries to connect with rabbitmq, any idea about what can be wrong ? | 13:34 |
| openstackgerrit | sean mooney proposed openstack/os-vif master: adds iptools driver for ip commands https://review.openstack.org/534307 | 13:34 |
| *** matrohon has joined #openstack-nova | 13:35 | |
| *** yamamoto_ has quit IRC | 13:35 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Make sure that functional test triggered on sample changes https://review.openstack.org/533210 | 13:35 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Deduplicate aggregate notification samples https://review.openstack.org/531162 | 13:35 |
| *** yamamoto has quit IRC | 13:35 | |
| sean-k-mooney | stephenfin: ^ that is not done yet but need to run in to town for a bit. ill fix it up when i get back. if i dont have it working by this evening ill revert the original pyroute2 patch. | 13:36 |
| sean-k-mooney | stephenfin: actully ill propose the revert now just ot have it in the gate but ill -1 it until this evening | 13:36 |
| openstackgerrit | sean mooney proposed openstack/os-vif master: Revert "Migrate from 'ip' commands to 'pyroute2'" https://review.openstack.org/534308 | 13:37 |
| *** jpena|lunch is now known as jpena | 13:39 | |
| *** liverpooler has quit IRC | 13:39 | |
| *** liverpooler has joined #openstack-nova | 13:40 | |
| *** dave-mccowan has joined #openstack-nova | 13:40 | |
| sdague | zioproto: I didn't really touch the json log formatter, though it wouldn't be too hard to add I don't think | 13:43 |
| *** bigdogstl has quit IRC | 13:43 | |
| *** jaianshu_ has joined #openstack-nova | 13:46 | |
| *** yamamoto has joined #openstack-nova | 13:47 | |
| *** dave-mccowan has quit IRC | 13:47 | |
| openstackgerrit | sahid proposed openstack/nova master: hardware: only take into account small pages https://review.openstack.org/532168 | 13:47 |
| *** jaianshu has quit IRC | 13:49 | |
| *** jaianshu_ has quit IRC | 13:50 | |
| *** yamamoto has quit IRC | 13:52 | |
| *** cleong has joined #openstack-nova | 13:52 | |
| Spazmotic | Poor XenAPI drivers, no love hehe | 13:55 |
| zioproto | sdague: should I open a bug then ? | 13:55 |
| openstackgerrit | Matthew Edmonds proposed openstack/nova master: Add taskflow to requirements https://review.openstack.org/533779 | 13:57 |
| *** Eran_Kuris has quit IRC | 13:57 | |
| sdague | zioproto: are you sure that request_id isn't in there? | 13:57 |
| sdague | because in looking at the code, it looks like it should be a key under the 'context' key, if it exists | 13:57 |
| *** dave-mccowan has joined #openstack-nova | 13:58 | |
| zioproto | it is not there | 13:58 |
| zioproto | I even shared my all config in the thread | 13:58 |
| zioproto | I see the same problem in neutron-server and nova-api | 13:58 |
| gibi | gmann: I did what you suggested in https://review.openstack.org/#/c/533210/7 plus I made an override for irrelevant files to see if it works or not in this context. Based on zuul status the override does not work in this context either | 13:59 |
| zioproto | I did not test other services so far | 13:59 |
| *** esberglu has joined #openstack-nova | 13:59 | |
| sdague | zioproto: you have a json dump of what you see, I don't see that in the email thread | 14:00 |
| *** mlavalle has joined #openstack-nova | 14:00 | |
| *** eharney has joined #openstack-nova | 14:00 | |
| *** Spazmotic is now known as BKaminski | 14:00 | |
| *** xnox has joined #openstack-nova | 14:01 | |
| *** xnox has quit IRC | 14:01 | |
| *** xnox has joined #openstack-nova | 14:01 | |
| *** fragatina has joined #openstack-nova | 14:02 | |
| *** fragatina has quit IRC | 14:02 | |
| *** fragatina has joined #openstack-nova | 14:03 | |
| efried | gibi You got the PowerVM OVS patch on your radar again today? https://review.openstack.org/#/c/422512/ | 14:04 |
| *** links has quit IRC | 14:05 | |
| *** damini has joined #openstack-nova | 14:05 | |
| *** matrohon has quit IRC | 14:05 | |
| efried | stephenfin I remember you saying you were really looking forward to that one ^ :) | 14:06 |
| stephenfin | Ha! | 14:06 |
| stephenfin | I'll look at it today :) | 14:06 |
| efried | Thank you sir. | 14:07 |
| *** ralonsoh_ has joined #openstack-nova | 14:08 | |
| *** ralonsoh has quit IRC | 14:08 | |
| *** lyan has joined #openstack-nova | 14:09 | |
| *** psachin has quit IRC | 14:12 | |
| *** Eran_Kuris has joined #openstack-nova | 14:13 | |
| *** yamamoto has joined #openstack-nova | 14:17 | |
| *** dtantsur|brb is now known as dtantsur | 14:18 | |
| *** andreas_s has quit IRC | 14:19 | |
| *** danpawlik has quit IRC | 14:19 | |
| *** andreas_s has joined #openstack-nova | 14:19 | |
| *** xnox has quit IRC | 14:20 | |
| *** matrohon has joined #openstack-nova | 14:20 | |
| *** sapd has joined #openstack-nova | 14:21 | |
| *** MarginHu has quit IRC | 14:21 | |
| *** yamamoto has quit IRC | 14:22 | |
| *** kolesovdv has quit IRC | 14:23 | |
| *** Eran_Kuris has quit IRC | 14:24 | |
| *** andreas_s has quit IRC | 14:26 | |
| *** abhishekk has joined #openstack-nova | 14:26 | |
| *** andreas_s has joined #openstack-nova | 14:27 | |
| *** jmlowe has joined #openstack-nova | 14:27 | |
| bauzas | mdbooth: I think I addressed most of your concerns in https://review.openstack.org/#/c/528832/ and later patches | 14:28 |
| bauzas | mdbooth: there are still some open issues (at least for suspend) that I'm working on, but after testing my series with a specific node, it works quite fine | 14:29 |
| *** yamahata has quit IRC | 14:30 | |
| mdbooth | bauzas: Thanks. wrt side effects in _get_guest_xml, I wonder if I'll have to suck that up. Perhaps we should rename it. | 14:31 |
| bauzas | renaming _get_guest_xml ? well, yes it's not a "get" | 14:31 |
| bauzas | but anyway | 14:31 |
| *** Spazmotic has joined #openstack-nova | 14:32 | |
| *** xnox has joined #openstack-nova | 14:33 | |
| *** xnox has quit IRC | 14:33 | |
| *** xnox has joined #openstack-nova | 14:33 | |
| *** david-lyle has quit IRC | 14:33 | |
| efried | jaypipes Given mgoddard's feedback, I'm thinking I do need to go to the trouble of reordering ProviderTree.snapshot ahead of set_traits_for_provider so we can get rid of that generation arg right away and have it be useful to him. Unless you see getting this series merged pretty quickly... | 14:34 |
| *** gszasz has joined #openstack-nova | 14:34 | |
| *** crushil has joined #openstack-nova | 14:34 | |
| jaypipes | efried: go for it. I'm workin' on the traits stuff for alloc cands | 14:34 |
| jaypipes | efried: i.e. I won't get to reviews on that until this afternoon | 14:35 |
| efried | ight | 14:35 |
| *** Spaz has joined #openstack-nova | 14:36 | |
| *** archit has joined #openstack-nova | 14:36 | |
| *** Eran_Kuris has joined #openstack-nova | 14:36 | |
| *** Spazmotic has quit IRC | 14:37 | |
| *** hongbin has joined #openstack-nova | 14:39 | |
| *** Spaz has quit IRC | 14:39 | |
| *** abhishekk has quit IRC | 14:39 | |
| *** tbachman has quit IRC | 14:42 | |
| jaypipes | sean-k-mooney: reverted that one patch in os-vif. you feel like cutting the 1.8.1 release? | 14:42 |
| sean-k-mooney | jaypipes: which one kuryr? | 14:42 |
| jaypipes | ya | 14:42 |
| *** burt has joined #openstack-nova | 14:42 | |
| BKaminski | johnthetubaguy, You around today by chance sir? | 14:43 |
| sean-k-mooney | we have another one | 14:43 |
| jaypipes | oh>? | 14:43 |
| sean-k-mooney | pyroute2 has a filehandel leak | 14:43 |
| sean-k-mooney | im implentint an iptools driver until its patched | 14:43 |
| sean-k-mooney | jaypipes: https://review.openstack.org/#/c/534307/ | 14:43 |
| jaypipes | looking... | 14:44 |
| sean-k-mooney | im gixing some tests currently | 14:44 |
| sean-k-mooney | fixing | 14:44 |
| jaypipes | hehe | 14:44 |
| jaypipes | sean-k-mooney: have fun gixing the implentint | 14:44 |
| sean-k-mooney | i have a revert open also https://review.openstack.org/#/c/512396/ so if i dont have https://review.openstack.org/#/c/534307 ready be this evening we can just straight revert | 14:44 |
| jaypipes | yup, saw that | 14:45 |
| sean-k-mooney | jaypipes: hehe im fixing pep8 first then ill fix spelling and add unit tests | 14:45 |
| *** bhujay has joined #openstack-nova | 14:46 | |
| openstackgerrit | Lajos Katona proposed openstack/nova master: Deduplicate service status notification samples https://review.openstack.org/534335 | 14:49 |
| *** danpawlik has joined #openstack-nova | 14:50 | |
| damini | Hi while installing newton in devstack enviournment i am getting error of | 14:51 |
| damini | Command "/usr/bin/python -u -c "import setuptools, tokenize;__file__='/tmp/pip-build-EW0AN9/PyECLib/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record /tmp/pip-iZslP6-record/install-record.txt --single-version-externally-managed --compile" failed with error code 255 in /tmp/pip-build-EW0AN9/PyECLib/ | 14:51 |
| damini | Can anyone tell me what the problem is ..i am new to this | 14:51 |
| *** awaugama has joined #openstack-nova | 14:51 | |
| openstackgerrit | Merged openstack/os-vif master: Revert "Move 'ips' field from Subnet object to VIF object" https://review.openstack.org/534206 | 14:52 |
| zioproto | sdague: taking a random line of the file, it looks like this https://pastebin.com/CcQ0bwi8 | 14:52 |
| zioproto | sdague: this is from nova-api | 14:52 |
| *** felipemonteiro has joined #openstack-nova | 14:53 | |
| *** felipemonteiro_ has joined #openstack-nova | 14:54 | |
| *** sree has joined #openstack-nova | 14:56 | |
| *** felipemonteiro has quit IRC | 14:58 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Make sure that functional test triggered on sample changes https://review.openstack.org/533210 | 14:59 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Deduplicate aggregate notification samples https://review.openstack.org/531162 | 14:59 |
| *** Guest80016 has quit IRC | 14:59 | |
| *** bhujay has quit IRC | 15:00 | |
| sdague | zioproto: what version? | 15:00 |
| sdague | because I don't see how that could come from current code, there should be a context key | 15:01 |
| sdague | even if it's an empty array | 15:01 |
| sdague | https://github.com/openstack/oslo.log/blob/e29dc73e9eaf009e7fccd17dcb96abdaf787efe7/oslo_log/formatters.py#L246-L251 | 15:02 |
| *** yamamoto has joined #openstack-nova | 15:03 | |
| *** yamamoto has quit IRC | 15:03 | |
| *** armax has joined #openstack-nova | 15:04 | |
| *** felipemonteiro_ has quit IRC | 15:04 | |
| *** alexchadin has quit IRC | 15:07 | |
| *** damini has quit IRC | 15:07 | |
| *** sapd has quit IRC | 15:09 | |
| *** tidwellr has joined #openstack-nova | 15:11 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: func tests for nested providers in alloc candidate https://review.openstack.org/531443 | 15:11 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: add _has_provider_trees() utility function https://review.openstack.org/531474 | 15:11 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: _get_trees_matching_all() https://review.openstack.org/531512 | 15:11 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: add tests for _get_trees_matching_all() with trait https://review.openstack.org/531899 | 15:11 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: add test for scenario with sum of child resources https://review.openstack.org/534339 | 15:11 |
| *** yamamoto has joined #openstack-nova | 15:12 | |
| jaypipes | gibi, efried, edleafe, cdent: ready for your signoff on ^^. gibi, the last patch contains the additional test case you'd asked for (which exposed buggy behaviour, btw) | 15:12 |
| *** bigdogstl has joined #openstack-nova | 15:13 | |
| efried | ack | 15:13 |
| edleafe | jaypipes: in meeting hell this morning. Will check it out later | 15:13 |
| jaypipes | edleafe: np | 15:13 |
| *** yamamoto has quit IRC | 15:17 | |
| *** zhaochao has quit IRC | 15:18 | |
| *** moshele has quit IRC | 15:18 | |
| *** kholkina has quit IRC | 15:20 | |
| *** mvenesio has quit IRC | 15:22 | |
| *** bigdogstl has quit IRC | 15:22 | |
| *** mvenesio has joined #openstack-nova | 15:23 | |
| lyarwood | mdbooth: https://review.openstack.org/#/c/523958/ - do you have time to go over the LUKS change again, I've had to introduce the _format logic again for block device volumes for now | 15:26 |
| *** MarginHu has joined #openstack-nova | 15:26 | |
| mdbooth | lyarwood: Yep. Looking now. | 15:27 |
| *** mvenesio has quit IRC | 15:27 | |
| *** lajoskatona has left #openstack-nova | 15:28 | |
| gibi | jaypipes: the whole series looks good to me. Thanks for the extra test case | 15:28 |
| *** links has joined #openstack-nova | 15:29 | |
| mdbooth | lyarwood: How does that work for rbd volumes? | 15:30 |
| lyarwood | mdbooth: with just this patch it doesn't, still waiting on a c-vol patch that formats and empty rbd volume with LUKS based on my image to volume example - https://review.openstack.org/#/c/527979/ | 15:31 |
| lyarwood | an* | 15:31 |
| *** bhujay has joined #openstack-nova | 15:31 | |
| *** sree_ has joined #openstack-nova | 15:32 | |
| mdbooth | lyarwood: Why are we only expecting c-vol to format rbd volumes, not all of them? | 15:32 |
| *** sree_ is now known as Guest40434 | 15:32 | |
| *** esberglu has quit IRC | 15:32 | |
| lyarwood | mdbooth: it's a different flow, you can format anything providing a local block device easily | 15:33 |
| lyarwood | mdbooth: with rbd at the moment you need to recreate the device | 15:33 |
| *** sree has quit IRC | 15:33 | |
| lyarwood | mdbooth: image even | 15:33 |
| mdbooth | lyarwood: Right, from Nova's side. But why would we expect different contracts from different volume types provided by c-vol? | 15:33 |
| mdbooth | lyarwood: I mean, if we're waiting for c-vol to do rbd, why not just wait for all volume types? | 15:34 |
| *** READ10 has joined #openstack-nova | 15:34 | |
| lyarwood | mdbooth: if this was earlier in the cycle sure but with limited time before M3 I assume we would only have time to get the rbd flow in | 15:35 |
| lyarwood | mdbooth: and then follow up with others in R, removing this call to format volumes then | 15:35 |
| lyarwood | mdbooth: as it wouldn't run anyway if the volumes already contained a LUKS header | 15:35 |
| *** hemna_ has joined #openstack-nova | 15:37 | |
| *** bhujay has quit IRC | 15:40 | |
| *** itlinux has quit IRC | 15:42 | |
| *** fragatina has quit IRC | 15:43 | |
| *** fragatina has joined #openstack-nova | 15:44 | |
| mdbooth | lyarwood: What's the thinking behind the volume_use_native_luks tunable? Why would you turn that off? | 15:46 |
| *** esberglu has joined #openstack-nova | 15:48 | |
| lyarwood | mdbooth: TBH that's only there to allow easier upgrade testing without needing to jump between releases downstream | 15:48 |
| *** david-lyle has joined #openstack-nova | 15:49 | |
| lyarwood | mdbooth: moving it to False once encrypted volumes are attached would cause failures, maybe it's best to drop it? | 15:49 |
| mdbooth | lyarwood: Gotcha. That's really useful info. Where can we stash that such that future generations will find it? | 15:49 |
| *** bigdogstl has joined #openstack-nova | 15:50 | |
| mdbooth | I mean, ideally we'd drop an unnecessary tunable. Is there any other way to do this for upgrades testing? | 15:50 |
| *** fragatina has quit IRC | 15:51 | |
| *** links has quit IRC | 15:51 | |
| *** fragatina has joined #openstack-nova | 15:51 | |
| jaypipes | gibi: thanks for asking for that test case! it highlighted a breakdown of the method in question :) | 15:51 |
| mdbooth | lyarwood: Thinking about upgrades... do we support upgrading nova-compute without rebooting the host? | 15:52 |
| jaypipes | gibi: you're so good at highlighting cracks in my code :) and alex_xu too. he's very good at that. | 15:52 |
| lyarwood | mdbooth: during an actual upgrade in grenade upstream or tripleo-ci downstream | 15:52 |
| *** ragiman has quit IRC | 15:52 | |
| lyarwood | mdbooth: yeah | 15:52 |
| *** dave-mccowan has quit IRC | 15:52 | |
| mdbooth | ... I think that's ok. We might leave orphaned encryptors, though. | 15:53 |
| mdbooth | But that might prevent disconnect_volume? | 15:53 |
| lyarwood | mdbooth: how so? | 15:53 |
| lyarwood | mdbooth: the current logic will still try to detach an encryptor if a secret isn't found for the volume | 15:53 |
| lyarwood | mdbooth: and a secret is only created when connecting the volume | 15:54 |
| mdbooth | ... and host secret is only added during connect | 15:54 |
| mdbooth | Nice | 15:54 |
| mdbooth | lyarwood: Just thinking about grenade, when would we set that to false even during testing? | 15:56 |
| *** lyan has quit IRC | 15:56 | |
| mdbooth | Surely the only interesting upgrade is across the no-native to native boundary | 15:56 |
| mdbooth | And libvirt/qemu version before is irrelevant, as we always used cryptsetup | 15:56 |
| lyarwood | mdbooth: yeah you wouldn't set it to false | 15:56 |
| mdbooth | Ok. Assume it's no big deal to remove the config option? | 15:57 |
| lyarwood | mdbooth: not at all | 15:57 |
| lyarwood | mdbooth: feel free to -1 on that and I'll respin shortly, about to jump on yet another call | 15:58 |
| *** yamamoto has joined #openstack-nova | 15:58 | |
| *** lyan has joined #openstack-nova | 15:59 | |
| gibi | jaypipes: I like this community. People here does not get offended when I find a crack in a proposed patch | 16:01 |
| *** yamamoto has quit IRC | 16:01 | |
| jaypipes | gibi: oh hell no, I love it. | 16:01 |
| *** yamamoto has joined #openstack-nova | 16:01 | |
| *** yamamoto has quit IRC | 16:01 | |
| jaypipes | gibi: much rather find it now than in production! | 16:01 |
| *** bigdogstl has quit IRC | 16:02 | |
| *** jackie-truong has joined #openstack-nova | 16:02 | |
| gibi | jaypipes: yeah, totally agree | 16:03 |
| *** Tom-Tom_ has joined #openstack-nova | 16:03 | |
| *** Tom-Tom has quit IRC | 16:06 | |
| *** yamahata has joined #openstack-nova | 16:07 | |
| *** bkopilov has joined #openstack-nova | 16:08 | |
| *** crushil has quit IRC | 16:09 | |
| *** gouthamr has joined #openstack-nova | 16:10 | |
| *** suresh12 has joined #openstack-nova | 16:11 | |
| jaypipes | efried: ah, cool, didn't catch there was an _allocate_from_provider helper | 16:14 |
| efried | Mr. Makestuffeasytoread, that's me. | 16:14 |
| gibi | efried: I'm +2 on network part of the PowerVM driver but the rest needs an other day | 16:15 |
| efried | gibi Thank you very much for the review! esberglu ^^ | 16:15 |
| esberglu | gibi: Thanks! | 16:16 |
| *** tbachman has joined #openstack-nova | 16:20 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Document TLS security setup for noVNC proxy https://review.openstack.org/500544 | 16:20 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Fix accumulated nits https://review.openstack.org/534368 | 16:20 |
| edmondsw | gibi thanks! the SEA patch is also networking, so it should be easier having built off the ovs patch. When you can get to it :) | 16:21 |
| stephenfin | mriedem, melwitt: Cleanups from the websocket-proxy-to-host-security series ^^^ | 16:21 |
| *** dave-mccowan has joined #openstack-nova | 16:21 | |
| *** links has joined #openstack-nova | 16:25 | |
| openstackgerrit | Aditya Vaja proposed openstack/nova master: remove IVS plug/unplug as they're moved to separate plugin https://review.openstack.org/534371 | 16:25 |
| *** janki has quit IRC | 16:29 | |
| stephenfin | sean-k-mooney: Hurrah! ^ | 16:30 |
| stephenfin | jaypipes too ^ | 16:30 |
| sean-k-mooney | oh cool | 16:30 |
| melwitt | stephenfin: ack | 16:31 |
| *** carl_baldwin has quit IRC | 16:32 | |
| *** tidwellr has quit IRC | 16:32 | |
| stephenfin | melwitt: Might be interested in this, from waaaaay back :) https://github.com/tox-dev/tox/issues/744 | 16:33 |
| * stephenfin got to sleuthing over his lunch | 16:33 | |
| melwitt | hah, I remember that! | 16:33 |
| *** tidwellr has joined #openstack-nova | 16:34 | |
| *** krtaylor has quit IRC | 16:36 | |
| *** jpena is now known as jpena|off | 16:37 | |
| *** gouthamr has quit IRC | 16:38 | |
| *** gyee has joined #openstack-nova | 16:41 | |
| openstackgerrit | Mark Goddard proposed openstack/nova master: WIP: Send traits to ironic on server boot https://review.openstack.org/508116 | 16:41 |
| openstackgerrit | Mark Goddard proposed openstack/nova master: Add get_traits() method to ComputeDriver https://review.openstack.org/532287 | 16:41 |
| openstackgerrit | Mark Goddard proposed openstack/nova master: Call get_traits() in the resource tracker https://review.openstack.org/532290 | 16:41 |
| openstackgerrit | Mark Goddard proposed openstack/nova master: Implement get_traits() for the ironic virt driver https://review.openstack.org/532288 | 16:41 |
| openstackgerrit | Mark Goddard proposed openstack/nova master: DNM Remove generation from set_traits_for_provider https://review.openstack.org/534375 | 16:41 |
| *** yamamoto has joined #openstack-nova | 16:41 | |
| *** jpena|off is now known as jpena | 16:41 | |
| *** Guest58384 has quit IRC | 16:42 | |
| *** itlinux has joined #openstack-nova | 16:44 | |
| bauzas | just in case people want some karma and have some review cycles, need a +W https://review.openstack.org/#/c/528832/ | 16:44 |
| *** yamamoto has quit IRC | 16:46 | |
| *** andreas_s has quit IRC | 16:46 | |
| *** bigdogstl has joined #openstack-nova | 16:47 | |
| *** andreas_s has joined #openstack-nova | 16:47 | |
| *** itlinux has quit IRC | 16:49 | |
| *** rha has joined #openstack-nova | 16:49 | |
| *** bigdogstl has quit IRC | 16:51 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] use simple FaultWrapper https://review.openstack.org/533752 | 16:52 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Refactor WSGI apps and utils to limit imports https://review.openstack.org/533797 | 16:52 |
| openstackgerrit | Merged openstack/nova master: Increase notification wait timeout in functional tests https://review.openstack.org/532816 | 16:52 |
| *** chyka has joined #openstack-nova | 16:53 | |
| *** bigdogstl has joined #openstack-nova | 16:54 | |
| *** mvk has quit IRC | 16:54 | |
| *** links has quit IRC | 16:54 | |
| *** moshele has joined #openstack-nova | 16:56 | |
| *** rano has quit IRC | 16:57 | |
| *** matrohon has quit IRC | 16:58 | |
| *** bigdogstl has quit IRC | 16:59 | |
| *** Guest40434 has quit IRC | 17:00 | |
| *** tidwellr has quit IRC | 17:00 | |
| *** gouthamr has joined #openstack-nova | 17:00 | |
| *** sree has joined #openstack-nova | 17:00 | |
| *** andreas_s has quit IRC | 17:01 | |
| openstackgerrit | Jacek Tomasiak proposed openstack/python-novaclient master: Fix listing of instances above API max_limit https://review.openstack.org/534222 | 17:01 |
| *** tidwellr has joined #openstack-nova | 17:02 | |
| *** yamahata has quit IRC | 17:02 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Cleanup tox.ini https://review.openstack.org/534382 | 17:04 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Remove 'tools/releasenotes_tox.sh' https://review.openstack.org/534383 | 17:04 |
| stephenfin | cdent: Might be interested in ^ | 17:05 |
| *** sree has quit IRC | 17:05 | |
| cdent | I get excited whenever I see the word "Cleanup" | 17:05 |
| *** sudipto_ has joined #openstack-nova | 17:06 | |
| *** sudipto has joined #openstack-nova | 17:06 | |
| edleafe | cdent: Cleanup in aisle 5! | 17:08 |
| openstackgerrit | sean mooney proposed openstack/os-vif master: adds iptools driver for ip commands https://review.openstack.org/534307 | 17:09 |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: [WIP] libvirt: Allow to specify granular CPU feature flags https://review.openstack.org/534384 | 17:10 |
| sean-k-mooney | stephenfin: ^ this should work i think. i just finished the fuctional tests and pep8 | 17:10 |
| *** AlexeyAbashkin has quit IRC | 17:11 | |
| lyarwood | melwitt: any objections to a plain recheck on https://review.openstack.org/#/c/531407/ given the failure looks unrelated? http://logs.openstack.org/07/531407/3/gate/legacy-tempest-dsvm-cells/4480311/job-output.txt.gz#_2018-01-16_04_53_50_389275 | 17:16 |
| melwitt | lyarwood: yeah that's fine | 17:17 |
| *** lucasagomes is now known as lucas-pto | 17:17 | |
| jaypipes | stephenfin: questions for you on that VNC security doc patch... | 17:18 |
| stephenfin | hit me | 17:18 |
| jaypipes | stephenfin: no, I left them in th ereview :) | 17:19 |
| stephenfin | Ah, gotcha :) Finishing review of efried's OVS patch, but I'll tackle after that | 17:19 |
| stephenfin | By which I mean, I'll go ask danpb what he meant | 17:19 |
| *** tesseract has quit IRC | 17:21 | |
| *** damien_r1 has quit IRC | 17:24 | |
| stephenfin | efried: This taskflow integration is pretty neat... | 17:28 |
| *** bigdogstl has joined #openstack-nova | 17:28 | |
| efried | stephenfin Thanks, yeah, it's been super useful for rollbacks and batching transactions. | 17:28 |
| efried | thorst gets most of the credit for that work | 17:28 |
| efried | or possibly kylekh | 17:29 |
| *** suresh12 has quit IRC | 17:29 | |
| mdbooth | lyarwood: If you're still around... I'm not 100% sure of this yet but sharing anyway. | 17:31 |
| lyarwood | mdbooth: still here | 17:32 |
| mdbooth | If we live migrate from a host without native luks to a host with native luks | 17:32 |
| stephenfin | efried: A'ight, two small questions left | 17:32 |
| efried | esberglu ^ | 17:32 |
| stephenfin | Other than that, it _looks_ fine. I can't test it though, obv | 17:32 |
| stephenfin | Relying on the third-party CI to do that | 17:32 |
| efried | stephenfin That's okay, our CI does. | 17:32 |
| mdbooth | I think that on the destination we will not have disk/encryptor | 17:32 |
| efried | yup | 17:32 |
| stephenfin | Spot on | 17:32 |
| mdbooth | But we *will* have the secret | 17:32 |
| stephenfin | Just those two so and I'm good | 17:32 |
| * stephenfin disappears for 20 minutes | 17:33 | |
| efried | stephenfin We can sell you a POWER9 system to test on. | 17:33 |
| lyarwood | mdbooth: I think nova/virt/libvirt/migration.py would take care of that with the XML update | 17:33 |
| mdbooth | Looking | 17:33 |
| lyarwood | mdbooth: we get_volume_config on the dest and then update the volume XML with anything new | 17:33 |
| *** zigo has quit IRC | 17:33 | |
| *** openstackgerrit has quit IRC | 17:33 | |
| lyarwood | mdbooth: I had to look into that for the native to native case | 17:34 |
| lyarwood | mdbooth: didn't think of the encryptor to native case but that *should* work | 17:34 |
| mdbooth | Reading... thanks. | 17:34 |
| *** mvenesio has joined #openstack-nova | 17:34 | |
| *** yamahata has joined #openstack-nova | 17:36 | |
| *** zigo has joined #openstack-nova | 17:37 | |
| *** openstackgerrit has joined #openstack-nova | 17:38 | |
| openstackgerrit | Merged openstack/nova master: conf: Do not inherit image signature props with snapshots https://review.openstack.org/527046 | 17:38 |
| *** HW_Peter has joined #openstack-nova | 17:38 | |
| *** bigdogstl has quit IRC | 17:38 | |
| BKaminski | Time for my third shift self to head to sleep. If anyone who is familiar with the XenAPI drivers get a chance, would love if you could take a look at my small commit :) https://review.openstack.org/#/c/533168/ Have a great day evreyone. | 17:39 |
| *** sridharg has quit IRC | 17:39 | |
| mdbooth | lyarwood: Time for another sanity check: I think that might not work, because get_updated_guest_xml() runs on the source, not the dest. | 17:40 |
| lyarwood | mdbooth: odd, that wouldn't make sense | 17:40 |
| lyarwood | mdbooth: well, to me at least :) | 17:41 |
| mdbooth | It's used to generate the destination xml to pass to the destination in initiating the migration | 17:41 |
| lyarwood | mdbooth: I assumed it wanted the updated volume XML from the dest POV | 17:41 |
| *** sudipto_ has quit IRC | 17:41 | |
| *** sudipto has quit IRC | 17:41 | |
| mdbooth | It runs from _live_migration_operation, which I'm pretty sure is on the source | 17:41 |
| mdbooth | But we do call _connect_volume() on the destination | 17:42 |
| mdbooth | in pre_live_migration | 17:42 |
| *** yamamoto has joined #openstack-nova | 17:44 | |
| lyarwood | right but the volume secret lookup would return the src hosts secret UUID | 17:44 |
| mdbooth | And use_native_luks will be true on the destination, so we'll create the host secret | 17:44 |
| lyarwood | that's pretty useless, updating instance XML using the src host | 17:44 |
| mdbooth | No, it's the other way round | 17:44 |
| mdbooth | When we call _attach_encryptor during pre_live_migration on the destination | 17:45 |
| mdbooth | We will create the host secret on the destination | 17:45 |
| mdbooth | Also... we won't actually attach the encryptor | 17:46 |
| lyarwood | right, but _get_volume_config is being called on the src, that does the secret lookup | 17:46 |
| mdbooth | No it doesn't, because the source hasn't been upgraded | 17:46 |
| *** andreas_s has joined #openstack-nova | 17:47 | |
| lyarwood | ah right, in that old to native use case, you're correct | 17:47 |
| mdbooth | I think we're uncovering a design defect in live migration here, tbh | 17:47 |
| mdbooth | But it is what it is | 17:47 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.snapshot => ProviderSnapshot https://review.openstack.org/533244 | 17:47 |
| openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.set_traits_for_provider https://review.openstack.org/532564 | 17:47 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Track associated sharing RPs in report client https://review.openstack.org/526539 | 17:47 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Raise on API errors getting aggregates/traits https://review.openstack.org/526540 | 17:47 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.populate_from_iterable https://review.openstack.org/520756 | 17:47 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Track tree-associated providers in report client https://review.openstack.org/526541 | 17:47 |
| openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.set_aggregates_for_provider https://review.openstack.org/533808 | 17:47 |
| openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.get_provider_tree https://review.openstack.org/521098 | 17:47 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ComputeDriver.update_provider_tree() https://review.openstack.org/521187 | 17:47 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821 | 17:47 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | 17:47 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260 | 17:47 |
| efried | mgoddard This whole thing has been rearranged and rebased ^ | 17:47 |
| efried | mgoddard I noticed you put some stuff on top (or in the middle, or something) - it'll need to be rebased accordingly. | 17:48 |
| mdbooth | efried: Zuul can take another one for the team ;) | 17:48 |
| mdbooth | lyarwood: So I'm not 100% sure what the state would be on the destination | 17:48 |
| efried | mdbooth The way zuul has been treating me lately, I don't have too much sympathy. | 17:48 |
| mdbooth | efried: Hehe | 17:49 |
| efried | Course, zuul is in the position of power, there. | 17:49 |
| efried | jaypipes See above: snapshot comes first now, followed by set_traits_for_provider so mgoddard can use it for the ironic traits bp. | 17:49 |
| *** fragatina has quit IRC | 17:49 | |
| mdbooth | lyarwood: Given that this would be a hopefully ephemeral situation, I wonder if we can detect this in pre_live_migration and fail early. | 17:49 |
| mdbooth | lyarwood: i.e. the fix is to not allow live migration host->native encryption | 17:50 |
| lyarwood | mdbooth: well, this breaks native to native as well right? | 17:51 |
| mdbooth | It goes away when they finish the upgrade | 17:51 |
| lyarwood | mdbooth: if we can't lookup the UUID of the new secret on the dest and udpate the XML we can't migrate | 17:51 |
| mdbooth | lyarwood: Does it? | 17:51 |
| * mdbooth ponders | 17:51 | |
| lyarwood | mdbooth: we'd just end up pointing to the src secret UUID that doesn't exist on the dest | 17:51 |
| *** jackie-truong has quit IRC | 17:51 | |
| mdbooth | lyarwood: I thought we were using the volume id as the secret uuid, no? | 17:51 |
| *** mlavalle has quit IRC | 17:52 | |
| lyarwood | mdbooth: no, the lookup code uses the volume_id, the actual secret has a seperate UUID you need to provide in the XML | 17:52 |
| efried | oh gdi. Stupid pep8 mistakes. And on the bottom patch, of course. | 17:52 |
| mdbooth | efried: LOL. FWIW: 'git rebase -i master -x "tox -e pep8 -- -HEAD"' is your friend | 17:53 |
| * mdbooth uses that all the time | 17:53 | |
| efried | ooo | 17:53 |
| mdbooth | Will run pep8 against each of your patches individually | 17:53 |
| jaypipes | efried: acl | 17:53 |
| efried | That's slightly easier than the way I was gonna do it. | 17:53 |
| jaypipes | ack | 17:53 |
| *** penick has joined #openstack-nova | 17:53 | |
| *** mlavalle has joined #openstack-nova | 17:54 | |
| *** derekh has quit IRC | 17:55 | |
| mdbooth | lyarwood: Well, without checking the libvirt api specifically, sounds like that's broken :( | 17:55 |
| mdbooth | This must have come up in a different context. | 17:55 |
| lyarwood | mdbooth: I wonder if we can create a secret with the same UUID on the dest? | 17:56 |
| esberglu | stephenfin: Responded to your OVS questions | 17:57 |
| mdbooth | lyarwood: This works currently, right? | 17:57 |
| mdbooth | i.e. We can't just not support it, because that would be a regression. | 17:57 |
| lyarwood | mdbooth: LM with encrypted volumes? Yeah AFAIK it does. | 17:58 |
| mdbooth | lyarwood: You'd need a config variable to switch it off ;) You should add one of those. | 17:58 |
| *** yamamoto has quit IRC | 17:58 | |
| lyarwood | \o/ | 17:58 |
| lyarwood | mdbooth: brb | 17:58 |
| mdbooth | I have to run, but I don't believe this is beyond the wit of man. | 17:59 |
| mdbooth | lyarwood: Catch up tomorrow afternoon? | 17:59 |
| openstackgerrit | Mark Goddard proposed openstack/nova master: WIP: Send traits to ironic on server boot https://review.openstack.org/508116 | 17:59 |
| openstackgerrit | Mark Goddard proposed openstack/nova master: Add get_traits() method to ComputeDriver https://review.openstack.org/532287 | 17:59 |
| openstackgerrit | Mark Goddard proposed openstack/nova master: Call get_traits() in the resource tracker https://review.openstack.org/532290 | 17:59 |
| openstackgerrit | Mark Goddard proposed openstack/nova master: Implement get_traits() for the ironic virt driver https://review.openstack.org/532288 | 17:59 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.snapshot => ProviderSnapshot https://review.openstack.org/533244 | 17:59 |
| openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.set_traits_for_provider https://review.openstack.org/532564 | 17:59 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Track associated sharing RPs in report client https://review.openstack.org/526539 | 17:59 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Raise on API errors getting aggregates/traits https://review.openstack.org/526540 | 17:59 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.populate_from_iterable https://review.openstack.org/520756 | 17:59 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Track tree-associated providers in report client https://review.openstack.org/526541 | 17:59 |
| openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.set_aggregates_for_provider https://review.openstack.org/533808 | 17:59 |
| openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.get_provider_tree https://review.openstack.org/521098 | 17:59 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ComputeDriver.update_provider_tree() https://review.openstack.org/521187 | 17:59 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821 | 17:59 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | 17:59 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260 | 17:59 |
| efried | There, ffs. mgoddard Sorry dude | 17:59 |
| lyarwood | mdbooth: sure, thanks again | 18:00 |
| melwitt | jaypipes: scheduler question for you ... when the filter scheduler does the compute node prune based on placement before going through the scheduler filters, does it take into consideration overcommit values set by aggregate? | 18:00 |
| jaypipes | melwitt: no | 18:01 |
| jaypipes | melwitt: placement has no concept of "aggregate overcommit" | 18:01 |
| melwitt | jaypipes: k, that's what I was thinking | 18:03 |
| melwitt | jaypipes: we're seeing issues where all of the overcommit was set via aggregates, so placement is filtering out very many nodes that can actually serve requests given overcommit | 18:05 |
| *** andreas_s has quit IRC | 18:05 | |
| *** dtantsur is now known as dtantsur|afk | 18:05 | |
| melwitt | as a workaround I was thinking, have to set overcommit per compute node in nova.conf to get around this | 18:05 |
| *** bigdogstl has joined #openstack-nova | 18:06 | |
| jaypipes | melwitt: I'll be honest, I didn't know about setting overcommit via aggregate. | 18:06 |
| jaypipes | melwitt: sounds like a half (quarter?)-baked feature | 18:07 |
| melwitt | heh. well, it's the AggregateCoreFilter and friends | 18:07 |
| jaypipes | melwitt: and as we've discussed before, aggregates and Ironic don't mix. at all... | 18:07 |
| jaypipes | melwitt: or is this not an ironic thing? | 18:08 |
| melwitt | this is not an ironic thing | 18:08 |
| jaypipes | kk | 18:08 |
| melwitt | since they're doing overcommit via aggregate, placement is returning only like 6 nodes when 20+ can fulfill given overcommit | 18:08 |
| jaypipes | melwitt: still sounds like a half-baked "feature" to me... | 18:08 |
| *** david-lyle has quit IRC | 18:08 | |
| *** mgoddard_ has quit IRC | 18:09 | |
| melwitt | meaning, you expect setting overcommit per compute node should be the fully baked way? | 18:09 |
| melwitt | just trying to understand how they should proceed going forward | 18:09 |
| jaypipes | melwitt: yes. it's the *only* way. | 18:09 |
| *** bigdogstl has quit IRC | 18:11 | |
| melwitt | k. I think the concern there is it hurts usability of aggregates. setting them per aggregate (a few times) vs setting them per compute node (a lot of times) | 18:11 |
| cdent | overcommit by aggregate. interesting. yet another thing I've never heard of before. I can see the appeal | 18:11 |
| melwitt | since aggregate means "bucket of like compute nodes" I can see how one would want to set overcommit same per bucket | 18:12 |
| melwitt | *since aggregate usually means | 18:12 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: add _has_provider_trees() utility function https://review.openstack.org/531474 | 18:12 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: _get_trees_matching_all() https://review.openstack.org/531512 | 18:12 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: add tests for _get_trees_matching_all() with trait https://review.openstack.org/531899 | 18:12 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: add test for scenario with sum of child resources https://review.openstack.org/534339 | 18:12 |
| jaypipes | melwitt: an aggregate in placement doesn't have any metadata associated with it at all. it's a pure grouping mechanism. | 18:13 |
| *** lpetrut_ has quit IRC | 18:14 | |
| efried | stephenfin Not sure which two comments were the ones holding you back. Let us know if we can clarify anything further. Thanks for the review! | 18:14 |
| melwitt | jaypipes: yeah, I understand. just pointing out the high level effect on the usability in this case | 18:14 |
| jaypipes | melwitt: the problem with the aggregate overcommit thing in Nova is what happens when the compute node's allocation ratio is != the aggregate's allocation ratio? also, what happens when a compute node is in multiple aggregates? which allocation ratio should be used? all these things are not issues in placement because we don't store metadata for aggregates. | 18:14 |
| melwitt | jaypipes: good points | 18:15 |
| melwitt | especially the multiple aggregates | 18:15 |
| *** sahid has quit IRC | 18:16 | |
| jaypipes | melwitt: and I'm not trying to be argumentative. just pointing out that there are definitely some things we won't be porting over to placement land... | 18:16 |
| *** sahid has joined #openstack-nova | 18:16 | |
| *** sahid has quit IRC | 18:16 | |
| *** links has joined #openstack-nova | 18:17 | |
| melwitt | jaypipes: understood. I'm not trying to be argumentative either. just looking to understand and be able to explain it to ppl :) | 18:17 |
| melwitt | so thanks | 18:17 |
| *** sahid has joined #openstack-nova | 18:18 | |
| jaypipes | melwitt: when discussing a similar thing with operators (having to set traits on compute node resource provider records individually instead of relying on agg metadata) I had suggested a simple CLI tool that would essentially apply a particular trait to all providers associated with an aggregate. they seemed responsive to that idea. | 18:18 |
| *** avolkov has quit IRC | 18:19 | |
| *** bigdogstl has joined #openstack-nova | 18:19 | |
| *** bigdogstl has quit IRC | 18:24 | |
| *** armax has quit IRC | 18:25 | |
| *** bigdogstl has joined #openstack-nova | 18:26 | |
| efried | jaypipes edleafe cdent gibi For all the hubbub, we may still want to implement an accessor for provider generation. Given the way I'm using it e.g. in https://review.openstack.org/#/c/532564/9/nova/scheduler/client/report.py@1046 | 18:26 |
| melwitt | jaypipes: k, I'll mention that to them. we'll be looking at how to change over from the Aggregate<resource> filters and configure the deployment to set it per compute node | 18:26 |
| *** armax has joined #openstack-nova | 18:26 | |
| melwitt | s/it/overcommit/ | 18:27 |
| edleafe | efried: what's the reason you can't just use generation = self._provider_tree.generation? | 18:27 |
| *** moshele has quit IRC | 18:28 | |
| efried | edleafe ProviderTree doesn't have a generation. A single provider does. | 18:28 |
| efried | edleafe Back to the thing where individual providers are hidden in the ProviderTree. | 18:28 |
| edleafe | efried: ok, I thought there was something I was missing | 18:29 |
| efried | ProviderTree.snapshot(rp_uuid).generation *works* - it's just wildly inefficient. | 18:29 |
| edleafe | efried: are there other atts besides generation that you have a need to access? | 18:30 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: rbd: flatten images when creating/unshelving an instance https://review.openstack.org/457886 | 18:31 |
| efried | edleafe Yes, the complex ones (aggs, traits, inventory) which we *must* copy out. Primitives, though... maybe parent_uuid? Eventually name? | 18:31 |
| *** moshele has joined #openstack-nova | 18:31 | |
| efried | edleafe More will shake out once drivers start implementing update_provider_tree. | 18:32 |
| *** cleong has quit IRC | 18:33 | |
| edleafe | efried: so are you thinking something like: ptree.getattr(rp_uuid, attname) ? | 18:33 |
| *** links has quit IRC | 18:34 | |
| efried | edleafe Initially I was just thinking about ptree.generation_for(rp_uuid), qua https://review.openstack.org/532922 | 18:34 |
| edleafe | efried: well, that's why I asked if you will need this for more than just generation | 18:35 |
| *** bigdogstl has quit IRC | 18:35 | |
| efried | edleafe Surely. Even so, the idea of N accessors was discussed on Monday and jaypipes expressed a preference for doing it this way. | 18:35 |
| edleafe | efried: it would suck to have ptree.name_for(rp_uuid), ptree.parent_uuid_for(rp_uuid), etc. | 18:35 |
| efried | agreeeeed. | 18:35 |
| efried | edleafe I think what I'm actually coming around to is that we shouldn't do the deep copy of the nested children in .snapshot() - at least not by default. | 18:36 |
| *** links has joined #openstack-nova | 18:36 | |
| efried | edleafe Then at least we're only throwing away one provider's worth of stuff we copied for no reason. | 18:36 |
| *** penick has quit IRC | 18:37 | |
| sean-k-mooney | stephenfin: jaypipes i ran all the tempest smoke test + senario test + tox -e py35,py27,functional,pep8,docs locally on https://review.openstack.org/#/c/534307/2 here http://paste.openstack.org/show/645744/ all passed so just waiting on the gate to do the same. | 18:38 |
| edleafe | or have snapshot() take an optional rp_uuid | 18:38 |
| edleafe | and only return that provider w/o children | 18:38 |
| efried | edleafe It already does take rp_uuid; that's how you tell it which provider you want to snapshot. | 18:38 |
| * edleafe should have the code open when commenting | 18:39 | |
| *** penick has joined #openstack-nova | 18:40 | |
| esberglu | stephenfin: Asked around and it turns out that nova-networking isn't supported. You okay with leaving bit in for now and removing it in a follow up? | 18:41 |
| esberglu | Trying to avoid respinning the series if possible | 18:41 |
| sean-k-mooney | esberglu: nova-networking has not been removed yet. it will be in early rocky | 18:42 |
| esberglu | sean-k-mooney: This is in regards to powervm virt driver support | 18:42 |
| esberglu | It only supports neutron | 18:42 |
| sean-k-mooney | esberglu: oh well if povervm never supported nova-net before or you have already dropped(or never supported) cells v1 then you dont need nova-networks support | 18:43 |
| *** AlexeyAbashkin has joined #openstack-nova | 18:43 | |
| efried | esberglu fyi stephenfin is on UK time, so may be gone for the evening. | 18:44 |
| esberglu | efried: ahh thanks for the info | 18:44 |
| *** rmcall_ has joined #openstack-nova | 18:45 | |
| *** jpena is now known as jpena|off | 18:45 | |
| efried | esberglu And gibi is also euro. So you might as well respin the series and they should be able to nail 'em quick in the morning (like 4am for us). | 18:45 |
| sean-k-mooney | efried: are you east coast or west? | 18:46 |
| *** ralonsoh_ has quit IRC | 18:47 | |
| efried | sean-k-mooney Central (Texas) | 18:47 |
| *** lbragstad_ has joined #openstack-nova | 18:47 | |
| *** gszasz has quit IRC | 18:47 | |
| sean-k-mooney | ah ok | 18:47 |
| efried | I think esberglu is either Central or East | 18:47 |
| esberglu | central | 18:47 |
| *** AlexeyAbashkin has quit IRC | 18:48 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094 | 18:48 |
| sean-k-mooney | ya unlike me stephen starts early and finishes early. i generally stay later but start later to give more us overlap in my day. also i hate have 2 7 oclocks in my day unless the frist on is 7 pm | 18:49 |
| *** lbragstad has quit IRC | 18:49 | |
| *** lbragstad_ is now known as lbragstad | 18:50 | |
| efried | sean-k-mooney Sounds like you coordinate way more than we do :) | 18:50 |
| efried | As far as meetings go, I consider a successful week to be one with three IRC meetings and zero in-person or phone meetings. | 18:51 |
| *** suresh12 has joined #openstack-nova | 18:51 | |
| efried | (at least, zero in-person meetings that don't involve choking the persons involved) | 18:52 |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: tests: fixes mock autospec usage https://review.openstack.org/447505 | 18:52 |
| sean-k-mooney | efried: haha i would love that. lately i have not had much internal meeting but in the past i had. i like beeing on irc when most of the nova,neutron and kolla core team are about too so haveing a slightly later day helps | 18:53 |
| efried | Mm. I believe cdent is of similar philosophy. | 18:53 |
| edleafe | efried: I have 3 phone meetings *today* | 18:54 |
| cdent | ... that's more side-effect than goal | 18:54 |
| *** efried is now known as efried_nomnom | 18:57 | |
| *** tidwellr has quit IRC | 18:58 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM driver: ovs vif https://review.openstack.org/422512 | 18:58 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: SEA https://review.openstack.org/523216 | 18:58 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094 | 18:58 |
| *** sambetts is now known as sambetts|afk | 18:58 | |
| *** tidwellr has joined #openstack-nova | 18:59 | |
| *** david-lyle has joined #openstack-nova | 18:59 | |
| sean-k-mooney | efried_nomnom: esberglu is there a reason you are not using and os-vif plugin intead of https://review.openstack.org/#/c/422512/40/nova/virt/powervm/vif.py | 19:01 |
| efried_nomnom | sean-k-mooney The main reason is probably because os-vif didn't exist when we wrote this stuff, or even when we first started porting it in tree. | 19:02 |
| sean-k-mooney | efried_nomnom: os-vif has been around for 3 maybe 4 cycles now | 19:02 |
| sean-k-mooney | efried_nomnom: it would be nice to convert in rocky | 19:03 |
| efried_nomnom | Duly noted. esberglu ^ for the to-do list? | 19:03 |
| efried_nomnom | Also sean-k-mooney we wouldn't know how to do that offhand. But since you've volunteered to guide us.... :) | 19:03 |
| cfriesen | I'm trying to figure out if a bug is in nova or libvirt/qemu. Do we have any functional/tempest tests of live migration with a config drive and an attached cinder volume? | 19:04 |
| Roamer` | hmm, if mriedem is really on vacation, could I interest any of you in taking a quick look at https://review.openstack.org/140733/ - the StorPool libvirt volume attachment driver? :) The Cinder and os-brick support is in, and Nova's requirements.txt has a dependency on the new version of os-brick now. | 19:04 |
| *** mgoddard_ has joined #openstack-nova | 19:05 | |
| Roamer` | of course, "leave it till he gets back" might also be kind of reasonable... although it may be cutting it a bit too close to the January 25th deadline | 19:05 |
| *** gouthamr has quit IRC | 19:06 | |
| sean-k-mooney | efried_nomnom: sure im happy to help. its not a huge amount of work to do but it means you dont have to submit patches to nova to take advantage of things like ovs now support jumbo frames with datapath x or other network backend specific features | 19:06 |
| efried_nomnom | sean-k-mooney Thanks in advance | 19:07 |
| *** links has quit IRC | 19:07 | |
| sean-k-mooney | efried_nomnom: other then requireing a powerpc system is there any other requirement to be able to deploy powervm. e.g. is it opensouce? | 19:08 |
| efried_nomnom | sean-k-mooney As long as "having a POWER system" includes relevant licensing (e.g. for VIOS), no other requirements I believe. Everything else is free (though not all of it is open source). | 19:10 |
| sean-k-mooney | efried_nomnom: hum ok i was wondering if i could emulate PPC under kvm and in stall powervm hyperviors under something like ubunut but i guess not | 19:11 |
| cfriesen | melwitt: do you know what the fix was for https://bugzilla.redhat.com/show_bug.cgi?id=1394964 (I'm guessing "mwitt@redhat.com" is you...) | 19:11 |
| openstack | bugzilla.redhat.com bug 1394964 in openstack-nova "Live migration with config-drive fails with InvalidSharedStorage error" [High,Closed: errata] - Assigned to mwitt | 19:11 |
| efried_nomnom | sean-k-mooney No, fraid not. | 19:11 |
| efried_nomnom | sean-k-mooney I mean, if you *could* actually emulate POWER (which isn't PowerPC, btw) then I imagine you could do that. | 19:12 |
| efried_nomnom | sean-k-mooney But emulating POWER is very, very hard. There is/was a project around here to do it, but I don't think it's up to date; and it's dogggggg slow. | 19:13 |
| sean-k-mooney | oh i did not know they were different. PPC is ibm right is POWER openPOWER | 19:13 |
| *** andreas_s has joined #openstack-nova | 19:13 | |
| efried_nomnom | I'm not sure what the status of ppc is actually. | 19:13 |
| efried_nomnom | And POWER is *almost* openPOWER. | 19:13 |
| melwitt | cfriesen: yes, that is me. looking ... | 19:14 |
| efried_nomnom | I think openpower is a way for not-IBM to build POWER-compatible systems. | 19:14 |
| efried_nomnom | But I'm totally not a terminology/marketing guy. I just try to write as much code as possible. | 19:15 |
| * efried_nomnom is really going to go eat now. | 19:15 | |
| *** fragatina has joined #openstack-nova | 19:15 | |
| cfriesen | melwitt: maybe bc4b9980f ? | 19:16 |
| sean-k-mooney | til well i shoudl be able to help in anycase. there is a lot you can do with unitest to fake things out + there is the IBM PowerVM CI so if you remind me/ i get time i might take a go at it but not before the PTG/end of Q1 | 19:16 |
| *** andreas_s has quit IRC | 19:17 | |
| *** andreas_s has joined #openstack-nova | 19:18 | |
| melwitt | cfriesen: so it was fixed by having https://review.openstack.org/#/c/227278 and removing VIR_MIGRATE_TUNNELLED from the block_migration_flag in nova.conf | 19:19 |
| melwitt | and I think config drive has to be vfat also | 19:19 |
| *** itlinux has joined #openstack-nova | 19:20 | |
| cfriesen | melwitt: okay, was wondering about that last part. we're still (in Pike) hitting issues with iso9660 config drives and boot-from-volume on a compute node with local storage. | 19:20 |
| cfriesen | melwitt: specifically with block-live-migrating such instances | 19:21 |
| cfriesen | I'm trying to see if it's something we broke or an upstream issue. | 19:22 |
| melwitt | cfriesen: yeah, I can't remember the context now but if you dig into the libvirt driver code there are a lot of conditions that have to be met before live block migration will work, and if you have a config drive, that's local storage so has to be a live BLOCK migration | 19:22 |
| melwitt | you can't have the tunneling but I don't remember why | 19:22 |
| cfriesen | melwitt: that part I know...tunnelling prevents the "block migrate some disks but not others" behaviour in libvirt | 19:23 |
| melwitt | oh, okay. cool | 19:23 |
| melwitt | now I know then :) | 19:24 |
| *** cdent has quit IRC | 19:24 | |
| melwitt | cfriesen: I know that in the past iso9660 config drives couldn't be live migrated, I don't know when/if that was fixed. based on the absence of 'vfat' in the pike code I'm guessing it was fixed | 19:29 |
| melwitt | I just updated the BZ to make a lot of the comments public that didn't need to be private | 19:30 |
| *** READ10 has quit IRC | 19:30 | |
| melwitt | in case that might help ppl that happen upon it | 19:30 |
| *** andreas_s has quit IRC | 19:35 | |
| melwitt | looks like iso9660 migration was enabled by https://review.openstack.org/#/c/234659 | 19:36 |
| *** bigdogstl has joined #openstack-nova | 19:38 | |
| *** lpetrut has joined #openstack-nova | 19:40 | |
| *** lpetrut has quit IRC | 19:40 | |
| *** andreas_s has joined #openstack-nova | 19:42 | |
| *** smatzek has quit IRC | 19:42 | |
| *** slaweq_ has joined #openstack-nova | 19:43 | |
| *** smatzek has joined #openstack-nova | 19:44 | |
| *** harlowja has joined #openstack-nova | 19:44 | |
| *** slaweq_ has quit IRC | 19:46 | |
| *** slaweq_ has joined #openstack-nova | 19:47 | |
| *** bigdogstl has quit IRC | 19:48 | |
| *** itlinux has quit IRC | 19:50 | |
| *** pcaruana has quit IRC | 19:50 | |
| *** andreas_s has quit IRC | 19:51 | |
| *** slaweq_ has quit IRC | 19:52 | |
| *** gouthamr has joined #openstack-nova | 19:52 | |
| *** moshele has quit IRC | 19:55 | |
| *** david-lyle has quit IRC | 19:57 | |
| *** david-lyle has joined #openstack-nova | 19:57 | |
| *** mvk has joined #openstack-nova | 19:59 | |
| *** itlinux has joined #openstack-nova | 20:05 | |
| *** smatzek has quit IRC | 20:06 | |
| *** matrohon has joined #openstack-nova | 20:06 | |
| *** smatzek has joined #openstack-nova | 20:06 | |
| *** smatzek has quit IRC | 20:08 | |
| *** smatzek has joined #openstack-nova | 20:09 | |
| openstackgerrit | Merged openstack/nova master: Rename block_device_info_get_root https://review.openstack.org/529028 | 20:09 |
| *** bigdogstl has joined #openstack-nova | 20:09 | |
| *** dave-mccowan has quit IRC | 20:12 | |
| *** slaweq_ has joined #openstack-nova | 20:13 | |
| *** smatzek has quit IRC | 20:13 | |
| *** bigdogstl has quit IRC | 20:14 | |
| *** penick has quit IRC | 20:16 | |
| *** mvk has quit IRC | 20:19 | |
| sean-k-mooney | jaypipes: looks like the gate is happy with it. https://review.openstack.org/#/c/534307/2 once that is merged im happy to go with a 1.8.1 release. | 20:21 |
| jaypipes | sean-k-mooney: question... why don't we just straight revert the pyroute patch from rodolfo>? | 20:24 |
| *** tbachman has quit IRC | 20:26 | |
| *** bigdogstl has joined #openstack-nova | 20:28 | |
| *** penick has joined #openstack-nova | 20:29 | |
| *** dave-mccowan has joined #openstack-nova | 20:33 | |
| *** bigdogstl has quit IRC | 20:38 | |
| *** efried_nomnom is now known as efried | 20:42 | |
| *** sean-k-mooney2 has joined #openstack-nova | 20:43 | |
| *** rmcall_ has quit IRC | 20:45 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM driver: ovs vif https://review.openstack.org/422512 | 20:46 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: SEA https://review.openstack.org/523216 | 20:46 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094 | 20:47 |
| sean-k-mooney2 | jaypipes rodolfo patch also introduces the infra for functional testing and deduplicates the logic from linux bridge plugin and ovs | 20:47 |
| sean-k-mooney2 | So i would like to keep that and just delay moving to pyroute2 until a new release is out | 20:48 |
| *** rmcall_ has joined #openstack-nova | 20:49 | |
| jaypipes | sean-k-mooney2: ok... | 20:49 |
| *** artom has quit IRC | 20:52 | |
| *** sean-k-mooney2 has quit IRC | 20:55 | |
| *** sean-k-mooney2 has joined #openstack-nova | 20:58 | |
| *** sean-k-mooney2 has quit IRC | 21:00 | |
| *** moshele has joined #openstack-nova | 21:01 | |
| *** artom has joined #openstack-nova | 21:02 | |
| *** tbachman has joined #openstack-nova | 21:06 | |
| *** suresh12 has quit IRC | 21:06 | |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: [WIP] libvirt: Allow to specify granular CPU feature flags https://review.openstack.org/534384 | 21:07 |
| *** suresh12 has joined #openstack-nova | 21:07 | |
| *** armax has quit IRC | 21:10 | |
| *** sdague has quit IRC | 21:11 | |
| *** tbachman_ has joined #openstack-nova | 21:12 | |
| *** liverpooler has quit IRC | 21:12 | |
| *** david-lyle has quit IRC | 21:12 | |
| *** tbachman has quit IRC | 21:13 | |
| *** tbachman_ is now known as tbachman | 21:13 | |
| *** bigdogstl has joined #openstack-nova | 21:15 | |
| *** threestrands_ has joined #openstack-nova | 21:18 | |
| *** sahid has quit IRC | 21:20 | |
| *** moshele has quit IRC | 21:20 | |
| *** tbachman has quit IRC | 21:21 | |
| *** david-lyle has joined #openstack-nova | 21:23 | |
| *** mgoddard_ has quit IRC | 21:25 | |
| *** felipemonteiro has joined #openstack-nova | 21:35 | |
| *** archit has quit IRC | 21:35 | |
| *** esberglu has quit IRC | 21:37 | |
| *** HW-Peter has quit IRC | 21:38 | |
| matrohon | sean-k-mooney: hi | 21:42 |
| *** bigdogstl has quit IRC | 21:45 | |
| cfriesen | melwitt: mdbooth: it turns out that if you have an instance with an iso9660 config drive and a ceph-backed attached cinder volume, then block-live-migration fails in libvirt with "qemuMigrationPrecreateStorage:463 : Operation not supported: pre-creation of storage targets for incremental storage migration is not supported". With iscsi-backed cinder volumes it works fine. | 21:45 |
| cfriesen | At least in our code....still trying to track down whether we broke something or if it's upstream. | 21:46 |
| *** pchavva has quit IRC | 21:48 | |
| *** archit has joined #openstack-nova | 21:50 | |
| *** esberglu has joined #openstack-nova | 21:50 | |
| *** armax has joined #openstack-nova | 21:51 | |
| *** slaweq_ has quit IRC | 21:51 | |
| *** bigdogstl has joined #openstack-nova | 21:51 | |
| *** rmk has joined #openstack-nova | 21:52 | |
| *** rmk has quit IRC | 21:52 | |
| *** rmk has joined #openstack-nova | 21:52 | |
| *** rmcall_ has quit IRC | 21:55 | |
| *** eharney has quit IRC | 21:55 | |
| *** itlinux has quit IRC | 21:58 | |
| *** moshele has joined #openstack-nova | 22:00 | |
| *** mchiappero has quit IRC | 22:00 | |
| *** moshele has quit IRC | 22:02 | |
| *** bigdogstl has quit IRC | 22:03 | |
| *** bigdogstl has joined #openstack-nova | 22:05 | |
| *** lyan has quit IRC | 22:08 | |
| *** archit has quit IRC | 22:18 | |
| *** lyan has joined #openstack-nova | 22:21 | |
| *** bigdogstl has quit IRC | 22:22 | |
| *** gouthamr has quit IRC | 22:27 | |
| *** suresh12 has quit IRC | 22:27 | |
| *** suresh12 has joined #openstack-nova | 22:28 | |
| *** rcernin has joined #openstack-nova | 22:31 | |
| *** tidwellr has quit IRC | 22:33 | |
| *** pchavva has joined #openstack-nova | 22:33 | |
| *** tidwellr has joined #openstack-nova | 22:35 | |
| *** itlinux has joined #openstack-nova | 22:39 | |
| *** tidwellr has quit IRC | 22:39 | |
| *** mvenesio has quit IRC | 22:41 | |
| *** itlinux has quit IRC | 22:41 | |
| *** felipemonteiro has quit IRC | 22:42 | |
| *** vladikr has quit IRC | 22:42 | |
| *** itlinux has joined #openstack-nova | 22:42 | |
| *** vladikr has joined #openstack-nova | 22:42 | |
| *** matrohon has quit IRC | 22:45 | |
| *** lyan has quit IRC | 22:45 | |
| *** itlinux has quit IRC | 22:47 | |
| *** liverpooler has joined #openstack-nova | 22:48 | |
| *** tbachman has joined #openstack-nova | 22:53 | |
| *** jroll has quit IRC | 22:53 | |
| openstackgerrit | Jacek Tomasiak proposed openstack/python-novaclient master: Fix listing of instances above API max_limit https://review.openstack.org/534222 | 22:53 |
| *** edmondsw has quit IRC | 22:55 | |
| *** edmondsw has joined #openstack-nova | 22:56 | |
| *** tbachman_ has joined #openstack-nova | 22:56 | |
| *** burt has quit IRC | 22:56 | |
| *** liverpooler has quit IRC | 22:56 | |
| *** dave-mccowan has quit IRC | 22:56 | |
| *** tbachman has quit IRC | 22:57 | |
| *** tbachman_ is now known as tbachman | 22:57 | |
| *** jroll has joined #openstack-nova | 22:58 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (resource_provider) https://review.openstack.org/528629 | 22:59 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova stable/pike: Add 'delete_host' command in 'nova-manage cell_v2' https://review.openstack.org/513717 | 22:59 |
| *** edmondsw has quit IRC | 23:00 | |
| *** bigdogstl has joined #openstack-nova | 23:00 | |
| *** jmlowe_ has joined #openstack-nova | 23:04 | |
| *** jmlowe has quit IRC | 23:05 | |
| *** jackie-truong has joined #openstack-nova | 23:08 | |
| *** bigdogstl has quit IRC | 23:08 | |
| *** bigdogstl has joined #openstack-nova | 23:16 | |
| *** bigdogstl has quit IRC | 23:22 | |
| *** claudiub has quit IRC | 23:31 | |
| *** bigdogstl has joined #openstack-nova | 23:33 | |
| *** MarginHu has quit IRC | 23:34 | |
| *** penick has quit IRC | 23:40 | |
| *** bigdogstl has quit IRC | 23:45 | |
| *** andreas_s has joined #openstack-nova | 23:49 | |
| *** andreas_s has quit IRC | 23:53 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!