*** r-daneel has quit IRC | 00:01 | |
*** itlinux has quit IRC | 00:03 | |
*** gcb has quit IRC | 00:05 | |
*** hongbin has quit IRC | 00:06 | |
mriedem | i'm +2 on the OSC CLI change for show/set/delete allocations https://review.openstack.org/#/c/457534/ - i think that's a good one to get in before the final client release freeze this week (same day as FF) | 00:07 |
---|---|---|
mriedem | best way to review these i've found is pull them down to your devstack and kick the tires | 00:08 |
*** yasemin has joined #openstack-nova | 00:08 | |
melwitt | mriedem: is there something special that has to be done to use the osc-placement plugin with OSC? I tried to use it last week and failed to figure it out. there were no 'openstack placement' or 'openstack resource' commands available | 00:10 |
mriedem | melwitt: git clone the repo, | 00:10 |
mriedem | then pip install it | 00:10 |
mriedem | then just: openstack resource provider -? | 00:11 |
mriedem | for the list of commands | 00:11 |
melwitt | okay. in my devstack it was installed and I even tried pip removing and pip installing it again with no luck. I'll try it again | 00:11 |
melwitt | that is, the osc-placement package got installed by devstack by itself (I didn't do anything to install it manually) | 00:12 |
mriedem | hmm, did you have the latest version of the osc-placement repo? | 00:12 |
*** Hazelesque has quit IRC | 00:12 | |
mriedem | oh... | 00:12 |
melwitt | no, it took whatever was on pypi | 00:12 |
mriedem | i always git clone it into /opt/stack/ | 00:12 |
*** robcresswell has quit IRC | 00:12 | |
melwitt | good to know | 00:12 |
mriedem | the pypi version doesn't have any actual CLIs yet | 00:12 |
melwitt | oh, no wonder. hah | 00:12 |
mriedem | i'm waiting to get at least this last allocations one merged before we release for queens | 00:13 |
mriedem | ideally we could get to https://review.openstack.org/#/c/511182/ yet for queens | 00:14 |
mriedem | because not being able to work with resource classes in the CLI makes the other existing CLIs can't of hard | 00:14 |
mriedem | *kind of hard | 00:14 |
*** chyka_ has joined #openstack-nova | 00:14 | |
melwitt | ack | 00:14 |
*** Hazelesque has joined #openstack-nova | 00:15 | |
*** sdague has quit IRC | 00:16 | |
*** AlexeyAbashkin has joined #openstack-nova | 00:16 | |
*** chyka has quit IRC | 00:18 | |
*** AlexeyAbashkin has quit IRC | 00:21 | |
*** yangyapeng has quit IRC | 00:26 | |
*** gongysh has joined #openstack-nova | 00:35 | |
*** dikonoor has quit IRC | 00:38 | |
*** chyka_ has quit IRC | 00:40 | |
*** Tom-Tom has joined #openstack-nova | 00:43 | |
openstackgerrit | Gao Fei proposed openstack/nova master: Redirects to a valid https link from the old one https://review.openstack.org/535244 | 00:45 |
*** tetsuro_ has joined #openstack-nova | 00:47 | |
*** Tom-Tom has quit IRC | 00:47 | |
*** zhurong has joined #openstack-nova | 00:48 | |
*** sambetts|afk has quit IRC | 00:54 | |
*** sambetts_ has joined #openstack-nova | 00:54 | |
*** Dinesh_Bhor has joined #openstack-nova | 00:57 | |
*** lbragstad has quit IRC | 00:57 | |
*** Dinesh_Bhor has quit IRC | 00:59 | |
*** crushil has joined #openstack-nova | 01:01 | |
*** zhaochao has joined #openstack-nova | 01:05 | |
*** hiro-kobayashi has joined #openstack-nova | 01:06 | |
*** phuongnh has joined #openstack-nova | 01:06 | |
*** mdnadeem has joined #openstack-nova | 01:06 | |
*** mdnadeem has quit IRC | 01:07 | |
*** mdnadeem has joined #openstack-nova | 01:07 | |
*** sree has joined #openstack-nova | 01:11 | |
*** Tom-Tom has joined #openstack-nova | 01:12 | |
*** zhurong has quit IRC | 01:13 | |
*** yangyapeng has joined #openstack-nova | 01:13 | |
*** sree has quit IRC | 01:15 | |
*** tiendc has joined #openstack-nova | 01:17 | |
*** gongysh has quit IRC | 01:21 | |
*** liverpooler has quit IRC | 01:23 | |
*** Dinesh_Bhor has joined #openstack-nova | 01:23 | |
*** sree has joined #openstack-nova | 01:27 | |
*** sree has quit IRC | 01:31 | |
*** Yikun has quit IRC | 01:36 | |
*** Yikun has joined #openstack-nova | 01:37 | |
*** sree has joined #openstack-nova | 01:40 | |
*** pooja has quit IRC | 01:45 | |
*** sree has quit IRC | 01:45 | |
*** hshiina has joined #openstack-nova | 01:46 | |
*** chyka has joined #openstack-nova | 01:48 | |
*** sree has joined #openstack-nova | 01:53 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix 'all_tenants' & 'all_projects' type in api-ref https://review.openstack.org/522918 | 01:54 |
*** sree has quit IRC | 02:02 | |
*** sree has joined #openstack-nova | 02:03 | |
*** chyka has quit IRC | 02:06 | |
*** efried_back_wed has quit IRC | 02:06 | |
*** hongbin has joined #openstack-nova | 02:08 | |
*** sree has quit IRC | 02:08 | |
*** yangyapeng has quit IRC | 02:08 | |
*** robcresswell has joined #openstack-nova | 02:09 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use neutron port_list when filtering instance by ip https://review.openstack.org/525505 | 02:12 |
*** yangyapeng has joined #openstack-nova | 02:13 | |
*** lbragstad has joined #openstack-nova | 02:14 | |
*** gongysh has joined #openstack-nova | 02:15 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | 02:15 |
*** efried_back_wed has joined #openstack-nova | 02:19 | |
Kevin_Zheng | mriedem https://review.openstack.org/525505 <- rebased due to conflict in test_compute_api.py | 02:19 |
*** liverpooler has joined #openstack-nova | 02:24 | |
openstackgerrit | Merged openstack/osc-placement master: CLI for allocations https://review.openstack.org/457534 | 02:24 |
*** sree has joined #openstack-nova | 02:26 | |
*** harlowja has quit IRC | 02:27 | |
*** annp has joined #openstack-nova | 02:28 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix 'all_tenants' & 'all_projects' type in api-ref https://review.openstack.org/522918 | 02:30 |
*** sree has quit IRC | 02:31 | |
openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.61 - List/Show all server migration types https://review.openstack.org/430839 | 02:32 |
*** suresh12 has quit IRC | 02:39 | |
*** yingjun has joined #openstack-nova | 02:41 | |
*** tbachman has quit IRC | 02:41 | |
*** suresh12 has joined #openstack-nova | 02:42 | |
*** tianhui has quit IRC | 02:45 | |
*** suresh12 has quit IRC | 02:47 | |
*** bkopilov has quit IRC | 02:53 | |
openstackgerrit | Merged openstack/nova master: Fix update_cell to ignore existing identical cells https://review.openstack.org/536546 | 02:59 |
*** vladikr has quit IRC | 02:59 | |
openstackgerrit | Merged openstack/nova master: Start moving users of parted to privsep. https://review.openstack.org/519011 | 03:00 |
*** vladikr has joined #openstack-nova | 03:00 | |
*** dikonoor has joined #openstack-nova | 03:03 | |
*** vladikr has quit IRC | 03:05 | |
*** hieulq has joined #openstack-nova | 03:09 | |
*** smatzek has joined #openstack-nova | 03:11 | |
*** dave-mcc_ has quit IRC | 03:16 | |
*** gongysh has quit IRC | 03:18 | |
*** hiro-kobayashi has quit IRC | 03:24 | |
*** gongysh has joined #openstack-nova | 03:24 | |
*** mgariepy has quit IRC | 03:24 | |
*** fragatina has quit IRC | 03:25 | |
*** hieulq_ has joined #openstack-nova | 03:25 | |
*** sree has joined #openstack-nova | 03:26 | |
*** phuongnh has quit IRC | 03:27 | |
*** tiendc has quit IRC | 03:27 | |
*** mgariepy has joined #openstack-nova | 03:27 | |
*** hieulq has quit IRC | 03:28 | |
*** annp has quit IRC | 03:29 | |
*** sree has quit IRC | 03:31 | |
*** tiendc has joined #openstack-nova | 03:32 | |
*** phuongnh has joined #openstack-nova | 03:32 | |
*** abhishekk has joined #openstack-nova | 03:34 | |
*** smatzek has quit IRC | 03:34 | |
*** amodi has quit IRC | 03:35 | |
*** suresh12 has joined #openstack-nova | 03:37 | |
*** suresh12 has quit IRC | 03:42 | |
*** hshiina has quit IRC | 03:43 | |
*** annp has joined #openstack-nova | 03:47 | |
*** hshiina has joined #openstack-nova | 03:47 | |
*** armax has quit IRC | 03:48 | |
*** suresh12 has joined #openstack-nova | 03:49 | |
*** armax has joined #openstack-nova | 03:55 | |
*** gongysh has quit IRC | 03:55 | |
*** liverpooler has quit IRC | 03:55 | |
*** sree has joined #openstack-nova | 03:57 | |
*** yamahata has joined #openstack-nova | 03:57 | |
openstackgerrit | Merged openstack/nova master: Remove redundant swap_volume tests https://review.openstack.org/531179 | 03:59 |
*** fragatina has joined #openstack-nova | 04:01 | |
*** sapd_ has quit IRC | 04:02 | |
*** sapd_ has joined #openstack-nova | 04:02 | |
*** suresh12 has quit IRC | 04:02 | |
*** links has joined #openstack-nova | 04:02 | |
*** fragatina has quit IRC | 04:05 | |
*** Tom-Tom has quit IRC | 04:07 | |
*** links has quit IRC | 04:07 | |
*** Tom-Tom has joined #openstack-nova | 04:07 | |
*** Tom-Tom has quit IRC | 04:07 | |
*** Tom-Tom_ has joined #openstack-nova | 04:07 | |
*** mlavalle has joined #openstack-nova | 04:08 | |
*** fragatina has joined #openstack-nova | 04:10 | |
*** moshele has joined #openstack-nova | 04:11 | |
*** diga has joined #openstack-nova | 04:11 | |
*** fragatina has quit IRC | 04:11 | |
*** suresh12 has joined #openstack-nova | 04:13 | |
*** Tom-Tom_ has quit IRC | 04:18 | |
*** Tom-Tom has joined #openstack-nova | 04:19 | |
*** links has joined #openstack-nova | 04:19 | |
*** moshele has quit IRC | 04:20 | |
*** bkopilov has joined #openstack-nova | 04:24 | |
*** gongysh has joined #openstack-nova | 04:24 | |
*** annp has quit IRC | 04:28 | |
*** tiendc has quit IRC | 04:28 | |
*** annp has joined #openstack-nova | 04:28 | |
*** tiendc has joined #openstack-nova | 04:28 | |
*** david-lyle has quit IRC | 04:30 | |
*** dklyle has joined #openstack-nova | 04:31 | |
*** dikonoor has quit IRC | 04:33 | |
*** suresh12 has quit IRC | 04:36 | |
*** hongbin has quit IRC | 04:37 | |
*** Tom-Tom has quit IRC | 04:38 | |
*** bhujay has joined #openstack-nova | 04:41 | |
*** psachin has joined #openstack-nova | 04:42 | |
*** sree has quit IRC | 04:42 | |
*** sree has joined #openstack-nova | 04:42 | |
*** Tom-Tom has joined #openstack-nova | 04:44 | |
*** fragatina has joined #openstack-nova | 04:45 | |
*** gongysh has quit IRC | 04:47 | |
*** Tom-Tom has quit IRC | 04:48 | |
*** armax_ has joined #openstack-nova | 04:49 | |
*** armax has quit IRC | 04:49 | |
*** armax_ has quit IRC | 04:53 | |
*** armax has joined #openstack-nova | 04:56 | |
*** ratailor has joined #openstack-nova | 05:01 | |
*** dikonoor has joined #openstack-nova | 05:02 | |
*** vish_18 has left #openstack-nova | 05:02 | |
*** armax has quit IRC | 05:03 | |
*** links has quit IRC | 05:06 | |
*** links has joined #openstack-nova | 05:08 | |
*** suresh12 has joined #openstack-nova | 05:11 | |
*** claudiub has joined #openstack-nova | 05:11 | |
*** harlowja has joined #openstack-nova | 05:12 | |
*** harlowja has quit IRC | 05:13 | |
*** moshele has joined #openstack-nova | 05:16 | |
takashin | 05:19 | |
*** crushil has quit IRC | 05:21 | |
*** pcaruana has joined #openstack-nova | 05:24 | |
*** trinaths has joined #openstack-nova | 05:25 | |
*** ratailor has quit IRC | 05:25 | |
*** ratailor has joined #openstack-nova | 05:27 | |
*** zhurong has joined #openstack-nova | 05:29 | |
*** ratailor has quit IRC | 05:34 | |
*** smatzek has joined #openstack-nova | 05:35 | |
*** pcaruana has quit IRC | 05:36 | |
*** sidx64 has joined #openstack-nova | 05:37 | |
*** sidx64 has quit IRC | 05:38 | |
*** smatzek has quit IRC | 05:39 | |
*** sidx64 has joined #openstack-nova | 05:40 | |
*** gongysh has joined #openstack-nova | 05:40 | |
*** ratailor has joined #openstack-nova | 05:41 | |
*** ratailor_ has joined #openstack-nova | 05:45 | |
*** dikonoor has quit IRC | 05:46 | |
*** ratailor has quit IRC | 05:47 | |
*** artom has quit IRC | 05:51 | |
*** artom has joined #openstack-nova | 05:51 | |
fanzhang | hi anyone here | 05:52 |
fanzhang | I hit a H202 error while running tox pep8 check, it's because my unit test using something like self.assertRaises(Exception,... I was wondering is H202 a mandatory rule or not? Please, if anyone knows about that, please do me a favor :) | 05:56 |
*** jaosorior has quit IRC | 05:57 | |
*** jaosorior has joined #openstack-nova | 05:57 | |
*** dikonoor has joined #openstack-nova | 05:58 | |
*** mlavalle has quit IRC | 06:00 | |
jianghuaw | fanzhang, it's mandatory I think. You should define a specific exception instead of using the general Exception. | 06:02 |
*** mlavalle has joined #openstack-nova | 06:03 | |
*** mlavalle has quit IRC | 06:03 | |
fanzhang | cool, thanks a lot. jianghuaw :) | 06:03 |
*** amotoki has quit IRC | 06:03 | |
jianghuaw | np | 06:03 |
*** amotoki has joined #openstack-nova | 06:04 | |
*** diga has quit IRC | 06:05 | |
*** Dinesh_Bhor has quit IRC | 06:06 | |
*** Dinesh_Bhor has joined #openstack-nova | 06:06 | |
*** liuzz has joined #openstack-nova | 06:08 | |
*** janki has joined #openstack-nova | 06:13 | |
*** Eran_Kuris has joined #openstack-nova | 06:14 | |
*** yamahata has quit IRC | 06:15 | |
*** stakeda has joined #openstack-nova | 06:17 | |
*** mdnadeem has quit IRC | 06:17 | |
*** lpetrut has joined #openstack-nova | 06:21 | |
*** Tom-Tom has joined #openstack-nova | 06:24 | |
*** hshiina has quit IRC | 06:25 | |
*** sidx64 has quit IRC | 06:27 | |
*** trinaths has left #openstack-nova | 06:29 | |
*** Yikun has quit IRC | 06:30 | |
Spazmotic | You're actually here jianghuaw? :D Didn't think you hung out on IRC | 06:30 |
Spazmotic | I saw you put some XenAPI notes on the PTG, Did you plan on attending Dublin? | 06:31 |
*** Yikun has joined #openstack-nova | 06:31 | |
Spazmotic | Hoping to get Corporate approval to attend, if so, would be nice to talk to another person interested in the XenAPI drivers | 06:31 |
jianghuaw | Spazmotic, yes. I'm planning to attend the PTG. | 06:31 |
jianghuaw | Spazmotic, thanks. | 06:32 |
*** mdnadeem has joined #openstack-nova | 06:32 | |
jianghuaw | Spazmotic, hope I can see you in Dublin. | 06:32 |
Spazmotic | Well wonderful to meet you. If you ever want to talk i'm around this time of night. I remain on US Night shift / Seoul time :p | 06:32 |
*** sidx64 has joined #openstack-nova | 06:33 | |
jianghuaw | nice to know:-) | 06:33 |
*** ratailor__ has joined #openstack-nova | 06:36 | |
*** salv-orlando has joined #openstack-nova | 06:36 | |
*** ratailor_ has quit IRC | 06:38 | |
*** sridharg has joined #openstack-nova | 06:41 | |
*** damien_r has quit IRC | 06:47 | |
*** sidx64 has quit IRC | 06:53 | |
*** sidx64 has joined #openstack-nova | 06:55 | |
*** itlinux has joined #openstack-nova | 06:56 | |
*** psachin has quit IRC | 06:56 | |
Spazmotic | Some Tuesday morning KPOP before we start waking up and working on actual serious things. Enjoy your days friends, i'll be around. https://www.youtube.com/watch?v=Ri6wvGjuoOg | 06:59 |
*** itlinux has quit IRC | 06:59 | |
*** sidx64 has quit IRC | 07:00 | |
*** tuanla____ has joined #openstack-nova | 07:01 | |
*** trinaths has joined #openstack-nova | 07:03 | |
*** sidx64 has joined #openstack-nova | 07:03 | |
*** tovin07 has joined #openstack-nova | 07:04 | |
*** gszasz has quit IRC | 07:05 | |
*** sidx64 has quit IRC | 07:06 | |
*** sidx64 has joined #openstack-nova | 07:07 | |
*** sidx64_ has joined #openstack-nova | 07:12 | |
*** sidx64 has quit IRC | 07:14 | |
*** sidx64 has joined #openstack-nova | 07:15 | |
*** sidx64_ has quit IRC | 07:17 | |
*** fragatina has quit IRC | 07:18 | |
*** itlinux has joined #openstack-nova | 07:27 | |
*** edand has joined #openstack-nova | 07:28 | |
*** pcaruana has joined #openstack-nova | 07:33 | |
*** lpetrut has quit IRC | 07:33 | |
*** vipul has joined #openstack-nova | 07:35 | |
*** slaweq has joined #openstack-nova | 07:38 | |
*** andreas_s has joined #openstack-nova | 07:38 | |
*** lpetrut has joined #openstack-nova | 07:41 | |
*** rha has quit IRC | 07:44 | |
*** suresh12 has quit IRC | 07:47 | |
*** andreas_s has quit IRC | 07:49 | |
*** tbachman has joined #openstack-nova | 07:50 | |
*** suresh12 has joined #openstack-nova | 07:51 | |
ameeda | Morning | 07:51 |
*** ralonsoh has joined #openstack-nova | 07:51 | |
ameeda | what can I do with this ERROR log "http://paste.openstack.org/show/650903/" | 07:52 |
*** psachin has joined #openstack-nova | 07:53 | |
*** suresh12 has quit IRC | 07:56 | |
*** itlinux has quit IRC | 07:58 | |
*** tbachman has quit IRC | 07:58 | |
*** pcaruana has quit IRC | 07:58 | |
*** rcernin has quit IRC | 08:00 | |
*** sidx64 has quit IRC | 08:02 | |
*** rushiagr has quit IRC | 08:04 | |
*** AlexeyAbashkin has joined #openstack-nova | 08:05 | |
*** sahid has joined #openstack-nova | 08:05 | |
*** tesseract has joined #openstack-nova | 08:09 | |
*** matrohon has joined #openstack-nova | 08:09 | |
*** rushiagr has joined #openstack-nova | 08:10 | |
*** pcaruana has joined #openstack-nova | 08:13 | |
*** damien_r has joined #openstack-nova | 08:14 | |
fanzhang | hi ameeda, I met this error some days earlier, updating your sqlite to sqlite-3.11.0 and your sqlite-libs to sqlite-libs-3.11.0 fixed my problem | 08:15 |
fanzhang | I mean it could help :) | 08:15 |
*** sapd_ has quit IRC | 08:16 | |
*** alexchadin has joined #openstack-nova | 08:16 | |
*** sapd_ has joined #openstack-nova | 08:16 | |
*** takashin has left #openstack-nova | 08:16 | |
*** lajoskatona has joined #openstack-nova | 08:16 | |
*** hiro-kobayashi has joined #openstack-nova | 08:19 | |
*** lpetrut has quit IRC | 08:21 | |
*** sahid has quit IRC | 08:23 | |
*** sahid has joined #openstack-nova | 08:26 | |
*** salv-orlando has quit IRC | 08:29 | |
*** salv-orlando has joined #openstack-nova | 08:29 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: tests for alloc candidates with nested and traits https://review.openstack.org/531899 | 08:31 |
*** Eran_Kuris has quit IRC | 08:33 | |
*** bkopilov has quit IRC | 08:33 | |
*** Eran_Kuris has joined #openstack-nova | 08:34 | |
*** salv-orlando has quit IRC | 08:34 | |
*** gszasz has joined #openstack-nova | 08:34 | |
*** annp has quit IRC | 08:38 | |
*** tiendc has quit IRC | 08:38 | |
*** annp has joined #openstack-nova | 08:38 | |
*** tuanla____ has quit IRC | 08:38 | |
*** tovin07 has quit IRC | 08:38 | |
*** phuongnh has quit IRC | 08:38 | |
*** tiendc has joined #openstack-nova | 08:38 | |
*** phuongnh has joined #openstack-nova | 08:39 | |
*** tuanla____ has joined #openstack-nova | 08:39 | |
*** tovin07 has joined #openstack-nova | 08:39 | |
ameeda | fanzhang: thanks , but how can I update that ? | 08:39 |
*** mdnadeem has quit IRC | 08:42 | |
*** gouthamr has joined #openstack-nova | 08:44 | |
*** jafeha__ is now known as jafeha | 08:44 | |
*** bkopilov has joined #openstack-nova | 08:46 | |
*** itlinux has joined #openstack-nova | 08:46 | |
fanzhang | ameeda: this part is where you should google it. :) I'll suggest that you get the right rpm packages and just install them. | 08:46 |
*** jpena|off is now known as jpena | 08:49 | |
*** cdent has joined #openstack-nova | 08:52 | |
*** Tom-Tom_ has joined #openstack-nova | 08:57 | |
ameeda | fanzhang: hehe, ok thank you very much ! | 08:58 |
*** gouthamr has quit IRC | 08:58 | |
fanzhang | ameeda: np | 08:59 |
ameeda | fanzhang: but you know, when I add my code the error appears, is that caused by my code ? | 08:59 |
*** salv-orlando has joined #openstack-nova | 08:59 | |
fanzhang | ameeda: I'm not sure. Have you tested before adding your codes? I don't know, you can surely try to find out. :) | 09:00 |
*** Tom-Tom has quit IRC | 09:01 | |
*** mdnadeem has joined #openstack-nova | 09:01 | |
bauzas | good morning Novza | 09:01 |
ameeda | yes I test it without my code, I didn't see the error. | 09:01 |
*** Eran_Kuris has quit IRC | 09:01 | |
*** bkopilov has quit IRC | 09:02 | |
*** amorin[away] is now known as amorin | 09:02 | |
*** priteau has joined #openstack-nova | 09:02 | |
openstackgerrit | Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353 | 09:04 |
*** gouthamr has joined #openstack-nova | 09:06 | |
fanzhang | ameeda: well, actually, I never test it before addming my codes, but updating sqlite related rpms solved my problem. | 09:06 |
ameeda | fanzhang: so let me check that, thanks ! | 09:06 |
fanzhang | ameeda: :) | 09:07 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Allow abort live migrations in queued status https://review.openstack.org/536722 | 09:08 |
*** sidx64 has joined #openstack-nova | 09:11 | |
*** avolkov has joined #openstack-nova | 09:12 | |
*** itlinux has quit IRC | 09:12 | |
Spazmotic | Hmm I like that | 09:13 |
*** avolkov has quit IRC | 09:13 | |
Spazmotic | Would help with scheduler failures as well. | 09:13 |
*** avolkov has joined #openstack-nova | 09:13 | |
*** Eran_Kuris has joined #openstack-nova | 09:14 | |
*** bkopilov has joined #openstack-nova | 09:15 | |
ameeda | jaypipes: are you around? | 09:20 |
*** artom has quit IRC | 09:20 | |
*** claudiub|2 has joined #openstack-nova | 09:21 | |
*** artom has joined #openstack-nova | 09:21 | |
openstackgerrit | Gao Fei proposed openstack/nova master: Replace Chinese punctuation with English punctuation https://review.openstack.org/536727 | 09:21 |
ameeda | any work-flow can check this for me ? "https://review.openstack.org/#/c/526900/" | 09:22 |
*** claudiub has quit IRC | 09:24 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Replace curly quotes with straight quotes https://review.openstack.org/536727 | 09:25 |
*** mvk has quit IRC | 09:26 | |
openstackgerrit | Radoslav Gerganov proposed openstack/nova master: WIP: VMware: implement update_provider_tree() https://review.openstack.org/536348 | 09:27 |
*** dikonoo has joined #openstack-nova | 09:31 | |
bauzas | jianghuaw: around ? | 09:33 |
*** vivsoni has quit IRC | 09:33 | |
bauzas | jianghuaw: I don't know if you tested your vGPU patches with all the instance actions | 09:33 |
*** vivsoni has joined #openstack-nova | 09:33 | |
*** tesseract-RH has joined #openstack-nova | 09:33 | |
*** xinliang has quit IRC | 09:34 | |
bauzas | jianghuaw: but see my relnote for libvirt https://review.openstack.org/#/c/533818/5/releasenotes/notes/add-support-for-vgpu-libvirt-91d2983e643f5ff1.yaml | 09:34 |
*** tesseract-RH has quit IRC | 09:34 | |
*** dikonoor has quit IRC | 09:34 | |
bauzas | jianghuaw: IMHO, you should be impacted by possibly the same problems | 09:34 |
bauzas | like for migrating | 09:34 |
bauzas | or resizing | 09:34 |
*** tesseract-RH has joined #openstack-nova | 09:35 | |
gibi | esberglu_: thanks for the update, I'm +2 on the SEA patch | 09:35 |
*** tesseract has quit IRC | 09:37 | |
openstackgerrit | Alex Xu proposed openstack/nova master: placement: using the dict format for the allocations https://review.openstack.org/536083 | 09:37 |
openstackgerrit | Alex Xu proposed openstack/nova master: placement: enable required traits from the flavor extra specs https://review.openstack.org/536085 | 09:37 |
*** yingjun has quit IRC | 09:37 | |
*** derekh has joined #openstack-nova | 09:39 | |
Spazmotic | Morning UK Nova | 09:40 |
Spazmotic | good to see you all :) | 09:40 |
*** alexchadin has quit IRC | 09:40 | |
alex_xu | gibi: bauzas ^, I just fix the functional tests | 09:40 |
Spazmotic | jianghuaw had some meetings, fyi bauzas | 09:40 |
alex_xu | bauzas: also there need one more +2 https://review.openstack.org/#/c/535642/4 :) appreciate you can help it | 09:41 |
*** alexchadin has joined #openstack-nova | 09:41 | |
*** alexchadin has quit IRC | 09:41 | |
*** alexchadin has joined #openstack-nova | 09:41 | |
*** alexchadin has quit IRC | 09:42 | |
Spazmotic | If anyone gets a chance with XenAPI knowledge, would love a look at https://review.openstack.org/#/c/533168/. Been sitting idle for about 12 days | 09:42 |
*** alexchadin has joined #openstack-nova | 09:42 | |
*** alexchadin has quit IRC | 09:42 | |
*** alexchadin has joined #openstack-nova | 09:43 | |
*** alexchadin has quit IRC | 09:43 | |
*** alexchadin has joined #openstack-nova | 09:44 | |
*** alexchadin has quit IRC | 09:44 | |
*** alexchadin has joined #openstack-nova | 09:44 | |
*** alexchadin has quit IRC | 09:45 | |
bauzas | Spazmotic: ok, thanks | 09:46 |
bauzas | Spazmotic: you working on Xen ? | 09:46 |
bauzas | alex_xu: looking | 09:46 |
alex_xu | bauzas: thanks! | 09:46 |
*** xinliang has joined #openstack-nova | 09:46 | |
maciejjozefczyk | Hello everyone! | 09:47 |
bauzas | alex_xu: I promised today to be a review day | 09:47 |
bauzas | unfortunately, I have some nits to modify and then I'm looking | 09:47 |
bauzas | the placement API for traits is something really crucial so yeah | 09:48 |
alex_xu | bauzas: got it, appreciate for that! | 09:48 |
*** Dinesh_Bhor has quit IRC | 09:48 | |
*** Dinesh_Bhor has joined #openstack-nova | 09:49 | |
*** Dinesh_Bhor has quit IRC | 09:49 | |
Spazmotic | yes sir bauzas | 09:51 |
gibi | alex_xu: looking as well | 09:51 |
*** amoralej has joined #openstack-nova | 09:52 | |
Spazmotic | Back and forth planting my pepper seeds for the season, sorry :) | 09:53 |
*** tuanla____ has quit IRC | 09:55 | |
*** tovin07 has quit IRC | 09:55 | |
*** phuongnh has quit IRC | 09:55 | |
*** jaosorior has quit IRC | 09:55 | |
*** phuongnh has joined #openstack-nova | 09:56 | |
*** tuanla____ has joined #openstack-nova | 09:56 | |
*** tovin07 has joined #openstack-nova | 09:56 | |
Spazmotic | Superhot Peppers and XenAPI.. essentially the same thing when caring for them. | 09:56 |
cdent | handle with care | 09:56 |
Spazmotic | Indeedie heh | 09:57 |
*** dtantsur|afk is now known as dtantsur | 09:58 | |
*** markvoelker has quit IRC | 09:59 | |
*** mvk has joined #openstack-nova | 09:59 | |
amoralej | melwitt, we are hitting a problem in last periodic in a tempest test https://bugs.launchpad.net/tripleo/+bug/1744907 | 10:01 |
openstack | Launchpad bug 1744907 in tripleo "Tempest test: "test_create_second_image_when_first_image_is_being_saved" failing in featureset020 periodic job" [Critical,New] | 10:01 |
amoralej | i'm not sure if it's nova or glance related | 10:01 |
amoralej | any idea? | 10:02 |
*** jcosmao has joined #openstack-nova | 10:02 | |
amoralej | second image is being properly created when error is expected | 10:02 |
alex_xu | gibi: thanks | 10:04 |
*** Eran_Kuris has quit IRC | 10:04 | |
*** sree has quit IRC | 10:06 | |
mdbooth | lyarwood: Have you seen melwitt 's review of the luks patch? | 10:06 |
*** Eran_Kuris has joined #openstack-nova | 10:06 | |
*** sree has joined #openstack-nova | 10:06 | |
mdbooth | Spazmotic: Expect severe pain the morning after use? | 10:07 |
mdbooth | lyarwood: I think you're still good, but you are missing 2 tests. | 10:07 |
jianghuaw | bauzas, sorry for late response. Just ended a meeting. thanks Spazmotic:-) | 10:09 |
bauzas | no worries, see my earlier pings | 10:09 |
jianghuaw | bauzas, yes. XenAPI doesn't support suspend also. | 10:09 |
lyarwood | mdbooth: yup working through it now | 10:09 |
jianghuaw | XenAPI has precheck on it. | 10:09 |
bauzas | jianghuaw: and resize I guess ? | 10:09 |
jianghuaw | I've not verified resizing. | 10:10 |
mdbooth | lyarwood: I've replied to most things if you refresh (and you're interested) | 10:10 |
jianghuaw | The suspend will be rejected by XenServer XAPI if there is vGPU attached in the VM. | 10:11 |
*** Eran_Kuris has quit IRC | 10:11 | |
*** zhurong has quit IRC | 10:11 | |
*** sree has quit IRC | 10:11 | |
*** sahid has quit IRC | 10:12 | |
jianghuaw | bauzas, resizing works well from my side. | 10:14 |
*** lpetrut has joined #openstack-nova | 10:14 | |
jianghuaw | XAPI handles more things beyond the virt driver than libvirt. | 10:15 |
*** itlinux has joined #openstack-nova | 10:15 | |
bauzas | jianghuaw: so XenAPI persists which vGPUs are used ? | 10:16 |
jianghuaw | bauzas, XAPI handle it. it has records in Db. | 10:16 |
jianghuaw | So nova needn't worry about status. | 10:17 |
jianghuaw | :-) | 10:17 |
bauzas | k | 10:17 |
*** alexchadin has joined #openstack-nova | 10:19 | |
jianghuaw | bauzas, actually you need handle more things which may be more suitable be handled by hypervisor. | 10:21 |
*** sdague has joined #openstack-nova | 10:21 | |
jianghuaw | Anyway you're super bauzas:-) | 10:21 |
openstackgerrit | Merged openstack/osc-placement master: CLI for usages https://review.openstack.org/457535 | 10:23 |
*** Eran_Kuris has joined #openstack-nova | 10:23 | |
Spazmotic | sweet sweet state.db | 10:24 |
Spazmotic | peppers all planted.. hopefully none of them will kill me | 10:24 |
*** dikonoo has quit IRC | 10:26 | |
Spazmotic | Thanks bob, if you're in here ;) | 10:29 |
*** zhurong has joined #openstack-nova | 10:30 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: pass the mdevs when rebooting the guest https://review.openstack.org/533818 | 10:30 |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: Avoid suspending guest with attached vGPUs https://review.openstack.org/535693 | 10:30 |
*** ralonsoh_ has joined #openstack-nova | 10:31 | |
*** stakeda has quit IRC | 10:31 | |
bauzas | jianghuaw: just provided all the comments in the relnote $ | 10:31 |
bauzas | ^ | 10:31 |
*** sambetts_ is now known as sambetts | 10:32 | |
*** abhishekk has quit IRC | 10:32 | |
bauzas | gibi: you were +2 on https://review.openstack.org/533818 | 10:32 |
*** Tom-Tom_ has quit IRC | 10:32 | |
bauzas | gibi: I actually just modified the relnote+ rebased | 10:33 |
kashyap | mdbooth: I'm still duking aroud the XML generation locally. As there 4 different classes I have to navigate: | 10:33 |
kashyap | - LibvirtConfigCPUFeature - base class for defining CPU features | 10:33 |
kashyap | - LibvirtConfigCPU - base class for defining CPU models | 10:33 |
kashyap | - LibvirtConfigGuestCPUFeature - extension for setting the guest | 10:33 |
kashyap | specific feature policy | 10:33 |
kashyap | - LibvirtConfigGuestCPU - extension for setting the guest specific | 10:33 |
kashyap | match policy, and allowing use of host CPU model passthrough | 10:33 |
* kashyap just wanted to note that he hasn't dropped working on that. | 10:33 | |
jianghuaw | bauzas, sure. I will look at the patches. | 10:33 |
kashyap | Context: It was in reference to https://review.openstack.org/#/c/534384/ | 10:34 |
mdbooth | kashyap: Yep, guessed. | 10:34 |
*** ralonsoh has quit IRC | 10:34 | |
mdbooth | kashyap: Incidentally, it sounded from that ml discussion that we might want to propose adding PCID automatically in Nova if it's not present in the explicit cpu model? | 10:35 |
*** lucas-afk is now known as lucasagomes | 10:35 | |
*** diga has joined #openstack-nova | 10:36 | |
mdbooth | Or were we still erring towards triple-o? | 10:36 |
kashyap | mdbooth: It's the latter. | 10:36 |
mdbooth | I guess the advantage of doing it in Nova is that you can query libvirt to expand the cpu flags from the specified model | 10:36 |
kashyap | mdbooth: Because, adding it automatically indeed is a bit too magic | 10:36 |
mdbooth | i.e. You don't need to duplicate the list | 10:37 |
* mdbooth wonders if ibrs is around for the long haul | 10:37 | |
gibi | bauzas: I put back my +2 | 10:37 |
kashyap | Yes, it should be. | 10:37 |
*** gouthamr_ has joined #openstack-nova | 10:38 | |
*** sidx64 has quit IRC | 10:39 | |
kashyap | mdbooth: Do you have a few more minutes to confirm I'm reading code correctly? /me begins to type | 10:39 |
*** mdnadeem has quit IRC | 10:39 | |
bauzas | gibi: <3 | 10:39 |
kashyap | mdbooth: So if you look at the class LibvirtConfigGuestCPUFeature() in libvirt/config.py. The infrastrucutre already _exists_ | 10:40 |
kashyap | To generate something like: | 10:40 |
kashyap | <cpu match='exact'> | 10:40 |
kashyap | <model fallback='forbid'>IvyBridge</model> | 10:40 |
kashyap | <vendor>Intel</vendor> | 10:40 |
kashyap | <feature policy='require' name='pcid'/> | 10:40 |
kashyap | </cpu> | 10:40 |
kashyap | mdbooth: Also see corresponding test: LibvirtConfigGuestCPUFeatureTest() in test_libvirt_config.py. | 10:41 |
*** gouthamr has quit IRC | 10:41 | |
kashyap | So since I'm introducing a config variable, I need to generate the objects for LibvirtConfigGuestCPUFeature() -- for every CPU flag configured in nova.conf | 10:42 |
* kashyap goes back to tinkering | 10:42 | |
mdbooth | kashyap: Yep | 10:43 |
ameeda | WARNING: when I pull the last version of nova "DevStack" , the system doesn't work anymore . | 10:45 |
kashyap | mdbooth: Okido. So my change here is wrong -- https://review.openstack.org/#/c/534384/2/nova/virt/libvirt/config.py. Because the root element 'feature' is already handled in the existing class LibvirtConfigCPUFeature() | 10:45 |
mdbooth | kashyap: Yeah, that whole bit of your patch looked confused, but as it was a WIP and you hadn't written any tests yet I figured you'd get there on your own. | 10:45 |
* kashyap nods; sorry if I wasted review time | 10:46 | |
mdbooth | Not at all. I've learned loads. | 10:46 |
*** itlinux has quit IRC | 10:47 | |
mdbooth | The motivation is much more interesting than some libvirt config wrangling code. | 10:47 |
*** sidx64 has joined #openstack-nova | 10:49 | |
kashyap | mdbooth: Hmm. I'm wondering if I should make a single class that will handle the complete root element: | 10:51 |
kashyap | <feature policy='require' name='pcid'/> | 10:51 |
kashyap | <feature policy='require' name='mtrr'/> | 10:51 |
kashyap | ... | 10:51 |
kashyap | Because, for now, the 'name' attribute is handled in the class LibvirtConfigCPUFeature() | 10:52 |
kashyap | And the 'policy' in LibvirtConfigGuestCPUFeature(). | 10:52 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: QEMU native LUKS decryption for encrypted volumes https://review.openstack.org/523958 | 10:52 |
mdbooth | kashyap: LibvirtConfigGuestCPUFeature already does that, no? | 10:52 |
mdbooth | lyarwood: Looking :) | 10:53 |
kashyap | mdbooth: This is what it does currently: | 10:53 |
kashyap | 486 class LibvirtConfigGuestCPUFeature(LibvirtConfigCPUFeature): | 10:53 |
kashyap | 487 | 10:53 |
kashyap | 488 def __init__(self, name=None, **kwargs): | 10:53 |
kashyap | 489 super(LibvirtConfigGuestCPUFeature, self).__init__(name, **kwargs) | 10:53 |
kashyap | 490 | 10:53 |
*** itlinux has joined #openstack-nova | 10:53 | |
kashyap | 491 self.policy = "require" | 10:53 |
*** jaosorior has joined #openstack-nova | 10:53 | |
kashyap | 492 | 10:53 |
kashyap | 493 def format_dom(self): | 10:53 |
kashyap | 494 ft = super(LibvirtConfigGuestCPUFeature, self).format_dom() | 10:53 |
kashyap | 495 | 10:53 |
kashyap | 496 ft.set("policy", self.policy) | 10:53 |
kashyap | 497 | 10:53 |
kashyap | 498 return ft | 10:53 |
kashyap | Okay, it inherits from the LibvirtConfigCPUFeature()... | 10:54 |
*** ralonsoh__ has joined #openstack-nova | 10:58 | |
*** gszasz has quit IRC | 10:59 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Add request_id field to migrations object https://review.openstack.org/536765 | 11:01 |
*** ralonsoh_ has quit IRC | 11:01 | |
Spazmotic | I need to maek my first nova-spec.. need to read up on how to do that. | 11:02 |
*** mvk has quit IRC | 11:02 | |
*** annp has quit IRC | 11:03 | |
*** AlexeyAbashkin has quit IRC | 11:06 | |
Spazmotic | sounds like a job for tomorrowwwwwman | 11:06 |
*** itlinux has quit IRC | 11:07 | |
*** gouthamr_ has quit IRC | 11:09 | |
openstackgerrit | Lee Yarwood proposed openstack/nova master: DNM: Test LM with encrypted volumes https://review.openstack.org/536350 | 11:09 |
cdent | gibi, bauzas: this would be a useful debugging improvement to have for the post-freeze cleanup: https://review.openstack.org/#/c/519462/ Kenichi has a concern about a private member, but I think that's just the way it goes for now. Already has +2 from stephenfin | 11:12 |
*** bkopilov has quit IRC | 11:12 | |
*** tuanla____ has quit IRC | 11:12 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Allow abort live migrations in queued status https://review.openstack.org/536722 | 11:14 |
*** mvk has joined #openstack-nova | 11:15 | |
kashyap | stephenfin or anyone: Do we have a precendence in Nova where we allow comma-separated values for a config attribute? | 11:17 |
stephenfin | kashyap: Look at oslo_config.types | 11:17 |
stephenfin | I'm guessing ListOpt is what you want | 11:17 |
*** dikonoo has joined #openstack-nova | 11:18 | |
stephenfin | I _think_ ListOpt uses CSV, but it should be easy enough to verify (look for existing usages in the nova tree) | 11:18 |
kashyap | stephenfin: I see, will look once I make quick lunch | 11:19 |
kashyap | stephenfin: So there is some precedence: | 11:19 |
kashyap | $> git grep ListOpt nova/conf/libvirt.py | 11:19 |
kashyap | nova/conf/libvirt.py: cfg.ListOpt('disk_cachemodes', | 11:19 |
kashyap | nova/conf/libvirt.py: cfg.ListOpt('hw_machine_type', | 11:19 |
kashyap | nova/conf/libvirt.py: cfg.ListOpt('uid_maps', | 11:19 |
kashyap | nova/conf/libvirt.py: cfg.ListOpt('gid_maps', | 11:19 |
kashyap | nova/conf/libvirt.py: cfg.ListOpt('enabled_perf_events', | 11:19 |
kashyap | nova/conf/libvirt.py: cfg.ListOpt('vzstorage_mount_opts', | 11:19 |
stephenfin | kashyap: Then you're golden. That's the approach I'd take | 11:20 |
stephenfin | Offloads all the hard work (parsing) to oslo.config | 11:21 |
*** hiro-kobayashi has quit IRC | 11:22 | |
kashyap | I see, noted. | 11:22 |
kashyap | stephenfin: So, what I'm looking to do is something like: | 11:22 |
kashyap | If you specify: | 11:23 |
kashyap | [libvirt] | 11:23 |
kashyap | cpu_model=IvyBridge | 11:23 |
kashyap | cpu_model_extra_flags="+pcid,-mtrr,pdpe1gb" | 11:23 |
kashyap | Which means: Add the PCID feature and 'pdpel1gb' feature (assume '+' is | 11:23 |
kashyap | given if it's not specified), and *remove* the 'mtrr' feature. | 11:23 |
kashyap | Then Nova should be able to generate an XML something like: | 11:23 |
*** AlexeyAbashkin has joined #openstack-nova | 11:23 | |
kashyap | <cpu match='exact'> | 11:23 |
kashyap | <model fallback='forbid'>IvyBridge</model> | 11:23 |
kashyap | <vendor>Intel</vendor> | 11:23 |
kashyap | <feature policy='require' name='pcid'/> | 11:23 |
kashyap | <feature policy='require' name='pdpe1gb'/> | 11:23 |
kashyap | </cpu> | 11:23 |
kashyap | stephenfin: Makes sense? | 11:23 |
stephenfin | Sounds sane. Why would you remove something though? | 11:24 |
kashyap | The +/- in nova.conf will be optional. If nothing is specified, assume '+'. | 11:24 |
stephenfin | I assume 'mtrr' would be added by default if you didn't explicitly add '-mtrr'? | 11:24 |
kashyap | stephenfin: If an advanced operator wants to generate a baseline CPU config, and a certain CPU flag is causing performance issue or other nuisance | 11:25 |
kashyap | And if she isolated the problem to _that_ specific CPU flag | 11:25 |
kashyap | Thens she may want to remove it. | 11:25 |
kashyap | Reasonable? | 11:25 |
kashyap | stephenfin: Yep, your assumption about 'mtrr' is true. | 11:26 |
*** phuongnh has quit IRC | 11:26 | |
stephenfin | Yum, that sounds reasonable to me | 11:26 |
stephenfin | *Yup | 11:27 |
stephenfin | :) | 11:27 |
kashyap | I.e. if you specified: 'mtrr', then it will be '+mtrr'. (Additional note: some named libvirt CPU models _already_ included 'mtrr') | 11:27 |
kashyap | Heh, noted. | 11:27 |
kashyap | Thanks | 11:27 |
kashyap | Now to figure out the XML spaghetti | 11:27 |
*** tiendc has quit IRC | 11:31 | |
*** sidx64 has quit IRC | 11:32 | |
*** gongysh has quit IRC | 11:36 | |
*** diga has quit IRC | 11:37 | |
*** sahid has joined #openstack-nova | 11:38 | |
*** trinaths has left #openstack-nova | 11:43 | |
*** ratailor__ has quit IRC | 11:44 | |
*** itlinux has joined #openstack-nova | 11:45 | |
*** salv-orlando has quit IRC | 11:48 | |
*** sree has joined #openstack-nova | 11:48 | |
*** salv-orlando has joined #openstack-nova | 11:48 | |
*** gouthamr has joined #openstack-nova | 11:49 | |
*** Guest78703 has quit IRC | 11:49 | |
*** dtantsur is now known as dtantsur|brb | 11:50 | |
*** tesseract-RH has quit IRC | 11:52 | |
*** gouthamr_ has joined #openstack-nova | 11:52 | |
*** salv-orlando has quit IRC | 11:53 | |
*** mvenesio has joined #openstack-nova | 11:54 | |
*** gouthamr has quit IRC | 11:55 | |
*** jcosmao has left #openstack-nova | 11:57 | |
*** jcosmao has joined #openstack-nova | 11:58 | |
*** takedakn has joined #openstack-nova | 11:59 | |
*** markvoelker has joined #openstack-nova | 12:00 | |
*** alexchadin has quit IRC | 12:00 | |
openstackgerrit | Chris Dent proposed openstack/nova master: Fix nits in allocation candidate limit handling https://review.openstack.org/536784 | 12:00 |
cdent | stephenfin: that ^ gets some cleanups you asked for | 12:00 |
ameeda | alex_xu : can you please check this for me ? https://review.openstack.org/#/c/526900/ | 12:02 |
*** liuzz has quit IRC | 12:03 | |
*** gszasz has joined #openstack-nova | 12:05 | |
*** zhurong has quit IRC | 12:05 | |
*** takedakn has quit IRC | 12:07 | |
*** jaypipes has joined #openstack-nova | 12:14 | |
*** alexchadin has joined #openstack-nova | 12:16 | |
*** smatzek has joined #openstack-nova | 12:19 | |
*** lpetrut has quit IRC | 12:19 | |
*** gouthamr_ has quit IRC | 12:19 | |
*** nicolasbock has joined #openstack-nova | 12:22 | |
*** dtantsur|brb is now known as dtantsur | 12:23 | |
*** tetsuro_ has quit IRC | 12:29 | |
*** jpena is now known as jpena|lunch | 12:33 | |
*** markvoelker has quit IRC | 12:33 | |
*** yamahata has joined #openstack-nova | 12:34 | |
*** alexchadin has quit IRC | 12:36 | |
*** alexchadin has joined #openstack-nova | 12:36 | |
openstackgerrit | int32bit proposed openstack/nova master: Set server status to ERROR if rebuild failed https://review.openstack.org/536268 | 12:39 |
openstackgerrit | int32bit proposed openstack/nova master: Refix disk size during live migration with disk over-commit https://review.openstack.org/536351 | 12:43 |
*** kwathore__ has joined #openstack-nova | 12:43 | |
*** kwathore_ has quit IRC | 12:46 | |
*** alexchadin has quit IRC | 12:46 | |
*** tesseract-RH has joined #openstack-nova | 12:47 | |
*** salv-orlando has joined #openstack-nova | 12:49 | |
ilyashakhat | hi! may I have cores attention to https://review.openstack.org/#/c/519664/ ? the patch is approved, but Zuul did not merge it | 12:49 |
ilyashakhat | most probably needs another +W to trigger the gate | 12:49 |
*** yangyapeng has quit IRC | 12:50 | |
*** yangyapeng has joined #openstack-nova | 12:51 | |
alex_xu | ilyashakhat: done | 12:51 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: DNM: Test LM with encrypted volumes https://review.openstack.org/536350 | 12:51 |
ilyashakhat | alex_xu: thanks! | 12:51 |
*** psachin has quit IRC | 12:51 | |
*** alexchadin has joined #openstack-nova | 12:52 | |
alex_xu | ameeda: sorry, really busy on freeze, will get chance after freeze | 12:52 |
alex_xu | ilyashakhat: np | 12:52 |
*** salv-orlando has quit IRC | 12:54 | |
ameeda | alex_xu: np :) | 12:54 |
*** tesseract-RH has quit IRC | 12:54 | |
*** alexchadin has quit IRC | 12:54 | |
*** yangyapeng has quit IRC | 12:55 | |
*** tesseract has joined #openstack-nova | 12:55 | |
*** ralonsoh__ is now known as ralonsoh | 12:55 | |
*** yassine has joined #openstack-nova | 12:57 | |
*** zhurong has joined #openstack-nova | 12:57 | |
*** yassine is now known as Guest87011 | 12:57 | |
*** sidx64 has joined #openstack-nova | 12:58 | |
*** alexchadin has joined #openstack-nova | 13:00 | |
*** sree has quit IRC | 13:04 | |
*** yamahata has quit IRC | 13:04 | |
*** sree has joined #openstack-nova | 13:04 | |
*** bkopilov has joined #openstack-nova | 13:08 | |
*** sree has quit IRC | 13:09 | |
*** eharney has joined #openstack-nova | 13:11 | |
*** sdague has quit IRC | 13:11 | |
*** zhurong has quit IRC | 13:14 | |
*** bhujay has quit IRC | 13:16 | |
*** amoralej is now known as amoralej|lunch | 13:16 | |
*** psachin has joined #openstack-nova | 13:17 | |
openstackgerrit | sahid proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997 | 13:17 |
*** liverpooler has joined #openstack-nova | 13:19 | |
*** Tom-Tom has joined #openstack-nova | 13:19 | |
*** sidx64 has quit IRC | 13:20 | |
*** gouthamr has joined #openstack-nova | 13:21 | |
*** sidx64 has joined #openstack-nova | 13:21 | |
*** dims has quit IRC | 13:22 | |
*** pchavva has joined #openstack-nova | 13:24 | |
*** alex_xu has quit IRC | 13:24 | |
*** dims has joined #openstack-nova | 13:25 | |
*** yangyapeng has joined #openstack-nova | 13:26 | |
*** alex_xu has joined #openstack-nova | 13:27 | |
*** sree has joined #openstack-nova | 13:28 | |
lyarwood | mdbooth: *facepalm* how did I screw up that if volume_id block *sigh* | 13:29 |
*** jpena|lunch is now known as jpena | 13:29 | |
mdbooth | lyarwood: Hehe, it totally doesn't matter. | 13:29 |
*** markvoelker has joined #openstack-nova | 13:31 | |
*** gouthamr has quit IRC | 13:35 | |
*** sree has quit IRC | 13:36 | |
*** kwathore_ has joined #openstack-nova | 13:38 | |
*** esberglu_ has quit IRC | 13:39 | |
* mdbooth sighs. *Another* merge conflict in my local disk serial numbers series :( | 13:40 | |
mdbooth | lyarwood: Bet ^^^ is your fault :P | 13:41 |
*** kwathore__ has quit IRC | 13:42 | |
lyarwood | mdbooth: you're welcome! :) | 13:42 |
lyarwood | mdbooth: pretty sure your swap volume test removal patch also caused a conflict this morning | 13:42 |
lyarwood | mdbooth: so it works both ways :) | 13:42 |
Spazmotic | This I will say is the one benefit from working the XenAPI drivers | 13:43 |
mdbooth | Indeed. | 13:43 |
Spazmotic | No merge conflicts :p | 13:43 |
mdbooth | Spazmotic: And also the curse :) How do you find getting core reviews on it? | 13:43 |
Spazmotic | Not sure if that burn was incidental or on purpose hehehehe | 13:43 |
* Spazmotic needs aloe | 13:43 | |
* mdbooth used to have terrible trouble in the vmware driver. | 13:44 | |
Spazmotic | My last and first commit ac tually went through pretty quick, but that was back in Liberty | 13:44 |
Spazmotic | Now that i'm trying to get more active, struggling ot get any love with Xenapi, hence why i'm trying to learn placement | 13:44 |
bauzas | cdent: all cool | 13:45 |
bauzas | reviewing https://review.openstack.org/#/c/519462/ | 13:45 |
*** jaypipes has quit IRC | 13:46 | |
mdbooth | Spazmotic: It's a perennial problem that it's harder to get core reviews in areas of the codebase cores are less familiar with. Doesn't even help to become a core, because you can't core review your own stuff :/ | 13:47 |
Spazmotic | Hahah nah I certainly understand.. I know XenAPI isn't the hotness right now, so i try to be patient and wait :) | 13:47 |
*** edmondsw has joined #openstack-nova | 13:49 | |
cdent | efried_back_wed: you're not supposed to be here. I haven't rechecked the things you just rechecked yet to try to make less noise while the stuff before it is still being chewed on | 13:49 |
*** salv-orlando has joined #openstack-nova | 13:50 | |
efried_back_wed | cdent It's a toss-up whether it'll be slower to check them serially (and have to wait for N to merge before rechecking N+1) or keep 'em all in the pipe in parallel so that maybe, just maybe, more than one will merge at once. | 13:50 |
efried_back_wed | cdent And I promise, I'll bugger off to my appointment in a few minutes. | 13:51 |
cdent | efried_back_wed: given that most of the failures are timeouts I tend to not want to parellelize | 13:51 |
cdent | which is probably illogical since something else will just take the spot, but meh | 13:52 |
efried_back_wed | cdent It had occurred to me to ask the infra folks whether we could increase whatever timeouts those are. I imagine they've already weighed the pros and cons of that, though. | 13:52 |
cdent | 'spect so | 13:52 |
cdent | this is going to merge very soon: https://review.openstack.org/#/c/526540/ | 13:52 |
*** salv-orlando has quit IRC | 13:55 | |
efried_back_wed | Now you've jinxed it | 13:57 |
*** esberglu has joined #openstack-nova | 13:58 | |
Spazmotic | Feels good to have a +1 on my code finally, and from someone at citrix with that.. makes me breathe easier hehe | 14:02 |
*** markvoelker has quit IRC | 14:04 | |
*** kwathore_ has quit IRC | 14:05 | |
*** links has quit IRC | 14:05 | |
ildikov | #success OpenStack now supports to attach a Cinder volume to multiple VM instances managed by Nova. | 14:05 |
openstackstatus | ildikov: Added success to Success page | 14:05 |
ildikov | Miracles exist when we have amazing and dedicated people to make them happen. :) | 14:05 |
*** lyan has joined #openstack-nova | 14:05 | |
gibi | ildikov: \o/ | 14:06 |
rgerganov | efried_back_wed, no luck trying to register a nested RP with the provider_tree | 14:06 |
ildikov | gibi: :) | 14:06 |
efried_back_wed | rgerganov Okay; leave some words on the patch, or send me an email, and I can dig in some more tomorrow (or maybe for a few minutes tonight) | 14:07 |
*** kwathore has joined #openstack-nova | 14:07 | |
rgerganov | efried_back_wed, sure, I am digging into this right now, will post all findings that I have at the end of the day | 14:07 |
efried_back_wed | rgerganov ++ | 14:07 |
openstackgerrit | Radoslav Gerganov proposed openstack/nova master: WIP: VMware: implement update_provider_tree() https://review.openstack.org/536348 | 14:08 |
*** abhishekk has joined #openstack-nova | 14:08 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: Avoid suspending guest with attached vGPUs https://review.openstack.org/535693 | 14:09 |
*** psachin has quit IRC | 14:09 | |
mriedem | andreykurilin: can you take a look at the novaclient change to support 2.59 https://review.openstack.org/#/c/516168/ and the small patch after it for 2.60? | 14:10 |
andreykurilin | looking | 14:11 |
*** dave-mcc_ has joined #openstack-nova | 14:12 | |
*** melwitt has quit IRC | 14:12 | |
cdent | rgerganov: is your code that's not working viewable somewhere? | 14:12 |
rgerganov | cdent: https://review.openstack.org/#/c/536348/3 | 14:13 |
cdent | oh never mind, actually read the log | 14:13 |
cdent | thanks | 14:13 |
mriedem | avolkov: i had some questions in the osc-placement patch for aggregates, which is the first one to add microversion support https://review.openstack.org/#/c/505643/ | 14:14 |
*** melwitt has joined #openstack-nova | 14:14 | |
mriedem | avolkov: i'm not sure how to expose in the help output that --os-placement-api-version 1.1 needs to be used | 14:14 |
mriedem | besides just adding it directly into the help | 14:14 |
*** lucasagomes is now known as lucas-hungry | 14:15 | |
*** moshele has quit IRC | 14:17 | |
Roamer` | mriedem, sorry if I'm starting to annoy you, I guess you didn't have time yesterday to take a look at 140733 - the StorPool driver? It's just that it seems that there might not be much time for us to make changes if something should appear to be not quite right... and yes, I do realize that you have a lot to look at, and an exception for a new driver is kind of low priority, I know | 14:18 |
*** crushil has joined #openstack-nova | 14:18 | |
jroll | mriedem: you didn't secretly fix the segfault thing yesterday, did you? seems to be passing now. | 14:19 |
*** smatzek has quit IRC | 14:20 | |
openstackgerrit | Alex Xu proposed openstack/nova master: placement: enable required traits from the flavor extra specs https://review.openstack.org/536085 | 14:21 |
cdent | rgerganov: two things: a) what kind of error are you getting with regard to "no luck", b) if you're putting some vcpus (and other inventory classes) in resource pools child providers, then they can't also be in the parent. it appears that's what your code is currently doing? | 14:22 |
alex_xu | gibi: I added the release note, thanks for the review | 14:22 |
*** salv-orlando has joined #openstack-nova | 14:22 | |
alex_xu | bauzas: efried_back_wed I will address the nits tommorow, thanks for the review | 14:22 |
*** dklyle has quit IRC | 14:22 | |
alex_xu | gibi: also the functional test! | 14:22 |
gibi | alex_xu: I'm totally OK to have the functional test in a separate followup | 14:23 |
rgerganov | cdent, you mean that we can't have VCPU in both parent and childs? | 14:23 |
mriedem | jroll: nope | 14:23 |
alex_xu | gibi: thanks, will try to submit a separate followup tomorrow | 14:24 |
jroll | yay | 14:24 |
cdent | rgerganov: it can be in both places, but they cannot overlap. Any inventory of the same type associated with the child provider needs to be subtracted from the parent. | 14:24 |
mriedem | Roamer`: i still have the tab open | 14:25 |
rgerganov | cdent, this is not the problem that I have now | 14:25 |
rgerganov | cdent, when you start nova-compute udpate_provider_tree is being called twice | 14:25 |
cdent | rgerganov: yeah, I get that, was just noting it for future reference | 14:25 |
cdent | first in init and then in the periodic job? | 14:26 |
*** edand has quit IRC | 14:26 | |
rgerganov | cdent, both calls come from _update_available_resource in the RT | 14:26 |
*** rgerganov has left #openstack-nova | 14:27 | |
* cdent nods | 14:27 | |
*** rgerganov has joined #openstack-nova | 14:27 | |
* cdent nods again | 14:27 | |
rgerganov | cdent, on the first call the provider tree seems to be correct and both parent and child RP are updated in placement | 14:28 |
rgerganov | cdent, on the second call I get 409 from placement | 14:28 |
gibi | alex_xu: I'm +2 on the last patch too, thanks for the reno | 14:28 |
alex_xu | gibi: thanks for the review! | 14:28 |
rgerganov | cdent, because it tries to create the nested RP again | 14:29 |
cdent | rgerganov: how close together are those jobs, by timestamp? | 14:29 |
rgerganov | cdent, they are far away because I am tracing the code | 14:30 |
rgerganov | cdent, like couple of seconds away | 14:30 |
cdent | It seems like the provider tree isn't getting a chance to update its cache of the rp generation | 14:30 |
rgerganov | cdent, correct | 14:31 |
cdent | It behaves the same way with the debugger not involved? | 14:31 |
rgerganov | yes | 14:31 |
rgerganov | on the first call the provider tree has both parent and child and the call to placement succeds | 14:32 |
*** vladikr has joined #openstack-nova | 14:32 | |
rgerganov | on the second call it has only the parent and when the driver adds the child and placement is called, we get 409 | 14:32 |
*** sidx64 has quit IRC | 14:33 | |
cdent | rgerganov: and the message with the 409 is what? | 14:34 |
*** sidx64 has joined #openstack-nova | 14:35 | |
*** sidx64 has quit IRC | 14:35 | |
rgerganov | cdent, Failed to create resource provider record in placement API for UUID 4be176f1-b09f-48dd-82d7-d9c1c86fb68b. Got 409: {"errors": [{"status": 409, "request_id": "req-69d31b34-bc50-4178-a569-ebaef4734cbd", "detail": "There was a conflict when trying to complete your request.\n\n Conflicting resource provider name: vdc already exists. ", "title": "Conflict"}]}. | 14:35 |
cdent | rgerganov: thanks, looking at the calling code now | 14:37 |
*** mvk has quit IRC | 14:37 | |
*** eharney has quit IRC | 14:37 | |
*** crushil has quit IRC | 14:38 | |
*** markvoelker has joined #openstack-nova | 14:41 | |
*** edand has joined #openstack-nova | 14:42 | |
*** cleong has joined #openstack-nova | 14:42 | |
*** bhujay has joined #openstack-nova | 14:43 | |
edmondsw | gibi tx for reviewing the powervm SEA patch! Are you planning to look at the powervm vscsi patch as well? | 14:44 |
cdent | rgerganov: on the second request, when the provider tree doesnt have the child, have you traced what's happening in _ensure_resource_provider. It seems like in there is where it could be going awry and not creating a full tree | 14:45 |
openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408 | 14:45 |
*** tbachman has joined #openstack-nova | 14:45 | |
gibi | edmondsw: I cannot promise that it will fit before the feature freeze | 14:46 |
*** alexchadin has quit IRC | 14:46 | |
edmondsw | gibi ok. If you get to it, thanks. If not, maybe someone else will | 14:47 |
*** fragatina has joined #openstack-nova | 14:47 | |
gibi | edmondsw: please ping me daily, that will help me not to forget | 14:48 |
*** ralonsoh_ has joined #openstack-nova | 14:48 | |
edmondsw | gibi you asked for it... ;) | 14:49 |
edmondsw | thanks | 14:49 |
*** bhujay has quit IRC | 14:49 | |
edmondsw | stephenfin do you also want daily reminders? ;) | 14:50 |
gibi | :) | 14:50 |
*** ralonsoh has quit IRC | 14:51 | |
stephenfin | edmondsw: Nope, it'll be done :) | 14:51 |
* stephenfin is working through this https://bugzilla.redhat.com/show_bug.cgi?id=1534669 | 14:51 | |
openstack | bugzilla.redhat.com bug 1534669 in openstack-nova "emulator_threads_policy needs improvement when hyper threading is enabled" [Medium,New] - Assigned to sfinucan | 14:51 |
edmondsw | stephenfin thanks! | 14:51 |
*** bhujay has joined #openstack-nova | 14:51 | |
* stephenfin should probably create a Launchpad bug for that | 14:51 | |
rgerganov | cdent, correct, this method doesn't fetch the full tree on the second call | 14:51 |
rgerganov | cdent, self._refresh_associations doesn't do what we expect | 14:52 |
*** hongbin has joined #openstack-nova | 14:52 | |
*** rgerganov is now known as rgerganov|away | 14:53 | |
*** dave-mcc_ has quit IRC | 14:55 | |
*** amoralej|lunch is now known as amoralej | 14:55 | |
edmondsw | bauzas johnthetubaguy any chance you could look at https://review.openstack.org/#/c/526094/ adding powervm cinder volume support? | 14:58 |
*** matrohon has quit IRC | 15:01 | |
openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 15:02 |
mriedem | melwitt: want to hit this 3 LOC patch for 2.60 support in novaclient? https://review.openstack.org/#/c/536621/ | 15:03 |
*** claudiub has joined #openstack-nova | 15:05 | |
*** dave-mccowan has joined #openstack-nova | 15:05 | |
*** slaweq has quit IRC | 15:06 | |
*** slaweq has joined #openstack-nova | 15:06 | |
*** mdnadeem has joined #openstack-nova | 15:06 | |
*** smatzek has joined #openstack-nova | 15:07 | |
*** awaugama has joined #openstack-nova | 15:07 | |
*** bauzas_ has joined #openstack-nova | 15:08 | |
*** claudiub|2 has quit IRC | 15:08 | |
*** amodi has joined #openstack-nova | 15:09 | |
*** bauzas_ has quit IRC | 15:09 | |
melwitt | mriedem: got it | 15:09 |
bauzas | mriedem: I can look | 15:09 |
bauzas | heh, snap | 15:09 |
mriedem | thanks, that'll likely be it for queens | 15:10 |
*** slaweq has quit IRC | 15:10 | |
*** gaoyan has joined #openstack-nova | 15:13 | |
*** jackie-truong has joined #openstack-nova | 15:13 | |
*** janki has quit IRC | 15:14 | |
*** gaoyan has quit IRC | 15:14 | |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Usage docs and initial release note for osc-placement https://review.openstack.org/536858 | 15:14 |
openstackgerrit | int32bit proposed openstack/nova master: Set server status to ERROR if rebuild failed https://review.openstack.org/536268 | 15:16 |
*** ralonsoh__ has joined #openstack-nova | 15:17 | |
*** moshele has joined #openstack-nova | 15:18 | |
*** bauzas_ has joined #openstack-nova | 15:18 | |
*** itlinux has quit IRC | 15:19 | |
*** mlavalle has joined #openstack-nova | 15:20 | |
*** dikonoo has quit IRC | 15:20 | |
*** lucas-hungry is now known as lucasagomes | 15:20 | |
*** ralonsoh_ has quit IRC | 15:20 | |
*** links has joined #openstack-nova | 15:21 | |
*** bauzas_ has quit IRC | 15:22 | |
openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.61 - Add trusted_image_certificates https://review.openstack.org/500396 | 15:22 |
*** bauzas has quit IRC | 15:23 | |
*** esberglu_ has joined #openstack-nova | 15:23 | |
*** bauzas has joined #openstack-nova | 15:24 | |
*** esbergl__ has joined #openstack-nova | 15:24 | |
*** moshele has quit IRC | 15:24 | |
*** armax has joined #openstack-nova | 15:25 | |
*** felipemonteiro has joined #openstack-nova | 15:25 | |
alex_xu | mriedem: in case you have time https://review.openstack.org/#/q/topic:bp/request-traits-in-nova+(status:open+OR+status:merged), I have last two patches waiting for one more +2, appreciate if you can help it | 15:25 |
*** esberglu has quit IRC | 15:26 | |
*** lajoskatona has quit IRC | 15:27 | |
*** eharney has joined #openstack-nova | 15:27 | |
mriedem | alex_xu: ok, also looks like https://review.openstack.org/#/c/536083/4/nova/scheduler/client/report.py overlaps with something dansmith was working on | 15:27 |
mriedem | to use the 1.12 allocation candidates format for the move operations in the report client | 15:28 |
*** esberglu_ has quit IRC | 15:28 | |
dansmith | mriedem: I think that patch just has two remaining failures to look at but I've been kinda absorbed with other things lately | 15:29 |
melwitt | mriedem: on the set status to ERROR upon rebuild failure patch, do you think it would be worth adding a fault message to the instance along with the ERROR state? I was thinking since the instance is sitting with the wrong imageref, it might be helpful to give some guidance in the fault message in case someone doesn't know how to recover https://review.openstack.org/#/c/536268/ | 15:31 |
*** esberglu has joined #openstack-nova | 15:31 | |
*** tidwellr has quit IRC | 15:32 | |
sean-k-mooney | stephenfin: for the record that refactor is annoying. i have all but 1 test passing now howerver the pci request spec object has like 8 required field not just the 4 in the alias schema definition | 15:32 |
mriedem | yeah a fault is probably necessary, i thought we'd have one on there but apparently we only use wrap_instance_fault in the compute manager | 15:32 |
stephenfin | sean-k-mooney: Really? What are the other ones? | 15:33 |
*** suresh12 has joined #openstack-nova | 15:33 | |
dansmith | melwitt: so can you push up a change to add a fault so we can get moving on that? | 15:33 |
sean-k-mooney | stephenfin: http://paste.openstack.org/show/650978/ | 15:34 |
*** corvus has quit IRC | 15:34 | |
melwitt | dansmith: yep, let me do that | 15:34 |
sean-k-mooney | stephenfin: the commented out ones should be deleted | 15:34 |
*** esbergl__ has quit IRC | 15:35 | |
dansmith | melwitt: cool | 15:35 |
sean-k-mooney | stephenfin: also i found that some test use device_id instead of product id so i have to support both and while the alias uses device_type the db uses dev_type so i have to convert when i create the spec | 15:35 |
*** damini has joined #openstack-nova | 15:36 | |
stephenfin | sean-k-mooney: You can probably drop capability_type | 15:37 |
sean-k-mooney | stephenfin: anyway when i get the last test passing ill push up the new revision. | 15:37 |
stephenfin | Per my comment in nova/pci/request.py | 15:37 |
stephenfin | But otherwise, yeah, I figured it wouldn't be super easy but it is a much better design long-term, IMO | 15:37 |
sean-k-mooney | stephenfin: nope you will get an invalid key exception unless i specificlally add code to drop it | 15:37 |
stephenfin | sean-k-mooney: I'm working through the CPU pinning code again. Every time I go through this, I have to relearn it | 15:38 |
stephenfin | sean-k-mooney: But nothing except tests use it | 15:38 |
stephenfin | *set it | 15:38 |
stephenfin | and we could just filter it out of any '[pci]spec' options with a warning that it must be manually removed | 15:39 |
damini | 111Pls can anyone tell me how to write tempest test for shelve instance and unshelve .I am new to openstack. | 15:39 |
sean-k-mooney | haha well now you mention it the code that is failing is in consume_requests a pop from an empty list. | 15:39 |
sean-k-mooney | stephenfin: ya its easy to ignore it and log a message the way i have it set up instead of a exception | 15:40 |
*** david-lyle has joined #openstack-nova | 15:40 | |
*** slaweq has joined #openstack-nova | 15:40 | |
stephenfin | damini: Tempest has a helper function for that. You might want to start there https://github.com/openstack/tempest/blob/master/tempest/common/compute.py#L263 | 15:41 |
*** jeblair has joined #openstack-nova | 15:41 | |
*** jeblair is now known as corvus | 15:42 | |
stephenfin | sean-k-mooney: (y) that would be my recommendation | 15:42 |
alex_xu | dansmith: do you have the link? I didn't found in the gerrit | 15:42 |
stephenfin | No point adding something we're going to immediately deprecate | 15:42 |
dansmith | alex_xu: link for what? | 15:42 |
alex_xu | dansmith: the 1.12 which mriedem mentioned, I guess you hit 1.12 by this patch https://review.openstack.org/#/c/531517/2 | 15:43 |
mriedem | yeah that's the one | 15:43 |
*** damini has quit IRC | 15:44 | |
*** slaweq has quit IRC | 15:44 | |
alex_xu | mriedem: ah, I got it, dansmith already said he is working on other things. sorry, i misunderstand few words previously | 15:46 |
mriedem | np, i'm just pointing out you were both working on part of the same problem i think | 15:46 |
*** nicolasbock has quit IRC | 15:47 | |
dansmith | alex_xu: why are you only bumping to 1.12 instead of to current? | 15:47 |
dansmith | alex_xu: because we're just going to have to bump right after this one | 15:47 |
alex_xu | dansmith: I have followup patch to support traits https://review.openstack.org/536085, which is 1.17 | 15:47 |
dansmith | alex_xu: okay seems like you could just start calling at 1.17 in the first patch, but okay | 15:48 |
dansmith | I guess I'll rebase mine on your last change | 15:49 |
*** gaoyan has joined #openstack-nova | 15:50 | |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Address comments from original inventory patch https://review.openstack.org/521578 | 15:50 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Usage docs and initial release note for osc-placement https://review.openstack.org/536858 | 15:50 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Add missing runtime requirements https://review.openstack.org/536870 | 15:50 |
alex_xu | dansmith: ok, thanks | 15:50 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Address review comments from allocations patch https://review.openstack.org/536871 | 15:50 |
*** gaoyan has quit IRC | 15:52 | |
*** slaweq has joined #openstack-nova | 15:53 | |
*** suresh12 has quit IRC | 15:55 | |
*** Eran_Kuris has quit IRC | 15:55 | |
*** gszasz has quit IRC | 15:58 | |
*** r-daneel has joined #openstack-nova | 15:59 | |
openstackgerrit | Merged openstack/nova master: Add reference to policy sample https://review.openstack.org/531713 | 16:00 |
*** abhishekk has quit IRC | 16:00 | |
openstackgerrit | Merged openstack/nova master: Raise on API errors getting aggregates/traits https://review.openstack.org/526540 | 16:00 |
openstackgerrit | Merged openstack/python-novaclient master: Microversion 2.59 - Migrations list pagination https://review.openstack.org/516168 | 16:01 |
openstackgerrit | melanie witt proposed openstack/nova master: Set server status to ERROR if rebuild failed https://review.openstack.org/536268 | 16:02 |
*** openstackgerrit has quit IRC | 16:03 | |
*** openstackgerrit has joined #openstack-nova | 16:04 | |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: CLI for aggregates (v1.1) https://review.openstack.org/505643 | 16:04 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: CLI for resource classes (v1.2) https://review.openstack.org/511182 | 16:04 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: RP list: member_of and resources parameters (v1.3, v1.4) https://review.openstack.org/511183 | 16:04 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: RP delete inventories (v1.5) https://review.openstack.org/514642 | 16:04 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: CLI for traits (v1.6) https://review.openstack.org/514643 | 16:04 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644 | 16:04 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646 | 16:04 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647 | 16:04 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: [WIP] Get resource provider by uuid or name https://review.openstack.org/527791 | 16:04 |
*** janki has joined #openstack-nova | 16:05 | |
bauzas | sorry folks, I just migrated my IRC bouncer | 16:07 |
bauzas | in case some pinged me, lemme know | 16:08 |
openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.61 - Add trusted_image_certificates https://review.openstack.org/500396 | 16:09 |
*** zhaochao has quit IRC | 16:10 | |
openstackgerrit | Matthew Booth proposed openstack/nova master: Give volume DriverBlockDevice classes a common prefix https://review.openstack.org/526346 | 16:10 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice https://review.openstack.org/526347 | 16:10 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Expose driver_block_device fields consistently https://review.openstack.org/528362 | 16:10 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Add local_root to block_device_info https://review.openstack.org/529029 | 16:10 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Pass DriverBlockDevice to driver.attach_volume https://review.openstack.org/528363 | 16:10 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Fix libvirt volume tests passing invalid disk_info https://review.openstack.org/529328 | 16:10 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Pass disk_info dict to libvirt_info https://review.openstack.org/529329 | 16:10 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Expose volume host type and path independent of libvirt config https://review.openstack.org/530786 | 16:10 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Don't generate fake disk_info in swap_volume https://review.openstack.org/530787 | 16:10 |
*** tssurya has quit IRC | 16:10 | |
openstackgerrit | Matthew Booth proposed openstack/nova master: Local disk serial numbers for the libvirt driver https://review.openstack.org/529380 | 16:10 |
openstackgerrit | melanie witt proposed openstack/nova master: Set server status to ERROR if rebuild failed https://review.openstack.org/536268 | 16:16 |
mriedem | melwitt: suggestion in https://review.openstack.org/#/c/536268/ | 16:16 |
mriedem | right before you pushed | 16:16 |
mriedem | we can assert the fault is set in the functional test that is also being updated | 16:17 |
melwitt | I think maybe I did what you asked | 16:17 |
*** tidwellr has joined #openstack-nova | 16:17 | |
*** suresh12 has joined #openstack-nova | 16:17 | |
melwitt | oh, nope, different func test | 16:17 |
*** edand_ has joined #openstack-nova | 16:17 | |
mriedem | ah that works too | 16:17 |
*** gszasz has joined #openstack-nova | 16:17 | |
melwitt | k, lemme add that | 16:18 |
mriedem | that's fine, | 16:18 |
mriedem | this is better actually because a rebuild can be done for a non-admin and we want to make sure the non-admin can see the fault | 16:18 |
mriedem | i don't know if we want to say something specific in the fault message about attempting to rebuild with a different image, that could be a slippery slope of assumptions about why we failed | 16:18 |
melwitt | okay, cool | 16:18 |
mriedem | like, image might be fine, but some other filter failed because a resource is unavailable or something | 16:19 |
melwitt | yeah, I was thinking about that too | 16:19 |
mriedem | +2 | 16:19 |
mriedem | heh stable/newton is still around | 16:20 |
*** edand has quit IRC | 16:20 | |
melwitt | :) | 16:21 |
*** tssurya has joined #openstack-nova | 16:22 | |
*** tesseract has quit IRC | 16:22 | |
*** suresh12 has quit IRC | 16:22 | |
mriedem | this is probably not worth yet another newton release before eol though | 16:22 |
mriedem | tonyb: when you gonna pull that newton-eol trigger? | 16:22 |
melwitt | wah wah | 16:23 |
*** sahid has quit IRC | 16:27 | |
*** edand__ has joined #openstack-nova | 16:29 | |
*** penick has joined #openstack-nova | 16:29 | |
*** tidwellr has quit IRC | 16:29 | |
bauzas | wait, Newton is not EOL'd yet ? | 16:30 |
bauzas | woah | 16:30 |
mriedem | melwitt: want to start on the backports? | 16:30 |
melwitt | mriedem: yes | 16:30 |
bauzas | melwitt: mriedem: dansmith: I think we should document a relnote with https://review.openstack.org/#/c/536268/6 | 16:30 |
bauzas | I was about to say that we will provide a new behaviour | 16:31 |
dansmith | bauzas: it's really the behavior that every other action has though | 16:31 |
bauzas | I'm not asking for a microversion, but we need to explain for operators I think | 16:31 |
*** tidwellr has joined #openstack-nova | 16:31 | |
bauzas | dansmith: I know | 16:31 |
bauzas | dansmith: I'm just thinking that it's a game changer | 16:31 |
bauzas | and documenting it would be IMHO a small thing for a better point | 16:31 |
mriedem | *shrug* i thought about a "fixes" release note just to point out that if scheduling fails during rebuild the instance will go to ERROR state and a fault will be recorded | 16:32 |
bauzas | I just don't want to see rants here :p | 16:32 |
bauzas | mriedem: exactly my point | 16:32 |
bauzas | not more than that | 16:32 |
mriedem | i'd be fine with that | 16:32 |
bauzas | so we can point out the relnote if someone is aking that | 16:32 |
dansmith | meh. | 16:32 |
*** edand_ has quit IRC | 16:32 | |
bauzas | backporting that means also that we will change the logic | 16:33 |
bauzas | hopefully the stable version will be not a .z one | 16:33 |
*** sdague has joined #openstack-nova | 16:33 | |
mriedem | umm | 16:33 |
mriedem | it's not a new feature | 16:34 |
mriedem | it's a bug fix | 16:34 |
bauzas | sure, but we're changing the API right? | 16:34 |
mriedem | we didn't bump the .y for the change that made the API fail fast if you tried to rebuild a volume-backed instance and change the image | 16:34 |
bauzas | again, I'm not asking for a microversion, please | 16:34 |
bauzas | okay | 16:34 |
bauzas | whatever the semver will be, I just think we need to explain that to ops | 16:34 |
bauzas | os-instance-actions API is admin by default ? | 16:35 |
mriedem | no | 16:35 |
*** bhujay has quit IRC | 16:35 | |
bauzas | I don't remember, I need to look at the default policy | 16:35 |
bauzas | ok, cool then | 16:35 |
mriedem | os-instance-actions events with the traceback is admin by default | 16:35 |
*** matrohon has joined #openstack-nova | 16:35 | |
bauzas | hah, right | 16:36 |
mriedem | i think you can't see the events before some microversion if you're not an admin either | 16:36 |
bauzas | I can test on my box | 16:36 |
mriedem | don't bother: https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id46 | 16:36 |
bauzas | cool cool | 16:37 |
*** moshele has joined #openstack-nova | 16:37 | |
openstackgerrit | Merged openstack/python-novaclient master: Add support for microversion 2.60 - volume multiattach https://review.openstack.org/536621 | 16:38 |
bauzas | mriedem: dansmith: melwitt: looks like my proposal to add a relnote didn't get a huge consensus, still okay if I'm providing it ? | 16:39 |
melwitt | mriedem, bauzas: so, should I add a fixes relnote to the patch then? | 16:39 |
bauzas | hah, jinxed | 16:40 |
mriedem | incoming | 16:40 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Set server status to ERROR if rebuild failed https://review.openstack.org/536268 | 16:40 |
bauzas | who's doing that ? I can take my pen if nobody does | 16:40 |
bauzas | oh, a new rev | 16:40 |
bauzas | mriedem: on it | 16:40 |
bauzas | mriedem: +2 | 16:41 |
*** moshele has quit IRC | 16:41 | |
bauzas | I can fast-approve or dansmith can +W | 16:41 |
*** pcaruana has quit IRC | 16:41 | |
mriedem | stvnoyes: ildikov: first victim of the multiattach swap volume test http://logs.openstack.org/89/532689/5/check/nova-multiattach/094e46b/job-output.txt.gz#_2018-01-23_12_43_58_790340 | 16:43 |
*** dmsimard has joined #openstack-nova | 16:44 | |
mriedem | this is where the swap starts http://logs.openstack.org/89/532689/5/check/nova-multiattach/094e46b/job-output.txt.gz#_2018-01-23_12_43_59_312226 | 16:44 |
openstackgerrit | melanie witt proposed openstack/nova stable/pike: Set server status to ERROR if rebuild failed https://review.openstack.org/536897 | 16:45 |
dmsimard | mriedem, kashyap: hi o/ It would be nice if we could land these in time for queens: https://review.openstack.org/#/c/524748/ && https://review.openstack.org/#/c/526748/ (and understand why the guestfs.set_backend_settings patch is breaking things when testing it here https://review.openstack.org/#/c/524750/ ) | 16:45 |
dmsimard | I suppose these are stable backport candidates as well | 16:46 |
*** wxy has quit IRC | 16:46 | |
mriedem | ildikov: stvnoyes: this is on a rax-dfw node and those have been really slow lately, so maybe it's just related to a slow node timeout since the swap volume tests do take awhile | 16:47 |
mriedem | dmsimard: i can't really say why things don't work with that stuff, | 16:48 |
mriedem | need some more libguestfs/qemu people | 16:48 |
dmsimard | mriedem: I don't know these people, I'm a mere mortal :( | 16:48 |
mriedem | dmsimard: i think https://review.openstack.org/#/c/526748/ is pretty straight forward though - don't do logging in a thread | 16:49 |
mriedem | b/c eventlet can explode | 16:49 |
*** avolkov has quit IRC | 16:50 | |
stvnoyes | mriedem, ildikov, I finally got my new env working but I am seeing some failures in the 'normal' tempest multiattach tests. I am looking to see why that is. It is likely something unique to my env. that's what I'm doing atm. | 16:51 |
openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 16:52 |
mriedem | it's doing the swap here http://logs.openstack.org/89/532689/5/check/nova-multiattach/094e46b/logs/screen-n-cpu.txt.gz#_Jan_23_11_16_02_654135 | 16:55 |
mriedem | sdague: do you ever remember anything about losing a requestid when using an elevated context? or people talking about resolving that? | 16:56 |
mriedem | i was tracing a swap volume operation through the logs and lose some of the logging once the context gets elevated | 16:57 |
openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.61 - Add trusted_image_certificates https://review.openstack.org/500396 | 16:58 |
openstackgerrit | Radoslav Gerganov proposed openstack/nova master: fix _ensure_resource_provider() https://review.openstack.org/536902 | 16:58 |
*** sahid has joined #openstack-nova | 16:59 | |
*** Eran_Kuris has joined #openstack-nova | 17:01 | |
openstackgerrit | Radoslav Gerganov proposed openstack/nova master: WIP: VMware: implement update_provider_tree() https://review.openstack.org/536348 | 17:01 |
*** tidwellr has quit IRC | 17:02 | |
*** AlexeyAbashkin has quit IRC | 17:02 | |
*** tidwellr has joined #openstack-nova | 17:02 | |
openstackgerrit | melanie witt proposed openstack/nova stable/ocata: Set server status to ERROR if rebuild failed https://review.openstack.org/536904 | 17:03 |
*** mvenesio has quit IRC | 17:06 | |
*** mvenesio has joined #openstack-nova | 17:07 | |
*** chyka has joined #openstack-nova | 17:07 | |
*** Tom-Tom has quit IRC | 17:10 | |
openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Refactor WSGI apps and utils to limit imports https://review.openstack.org/533797 | 17:13 |
*** Guest87011 has quit IRC | 17:13 | |
*** matrohon has quit IRC | 17:14 | |
*** mvenesio_ has joined #openstack-nova | 17:14 | |
*** armax has quit IRC | 17:14 | |
*** moshele has joined #openstack-nova | 17:14 | |
*** jackie-truong has quit IRC | 17:14 | |
gibi | mriedem: btw, do you need me to run the weekly nova meeting on Thrusday? | 17:15 |
mriedem | gibi: maybe... | 17:16 |
mriedem | i'll let you know in a bit | 17:16 |
gibi | mriedem: OK, I can do that if needed | 17:16 |
mriedem | gibi: i should be able to do it, assuming i can remember :) | 17:17 |
*** mvenesio has quit IRC | 17:17 | |
gibi | mriedem: OK | 17:17 |
*** felipemonteiro has quit IRC | 17:25 | |
*** gszasz has quit IRC | 17:29 | |
*** salv-orlando has quit IRC | 17:32 | |
*** penick has quit IRC | 17:32 | |
*** salv-orlando has joined #openstack-nova | 17:32 | |
*** armax has joined #openstack-nova | 17:33 | |
*** smatzek_ has joined #openstack-nova | 17:34 | |
*** penick has joined #openstack-nova | 17:36 | |
artom | melwitt, no newton backport for the rebuild imageref fix? There was some confusion whether newton is EOL or not | 17:36 |
*** salv-orlando has quit IRC | 17:37 | |
dansmith | artom: it's EOL, just not deleted yet | 17:38 |
melwitt | artom: indeed the stable/newton branch is still around coincidentally. but earlier mriedem said it's probably not worth another newton release for this | 17:38 |
dansmith | I agree | 17:39 |
artom | Sure, not gonna argue with that | 17:40 |
artom | (I mean, I probably would, in theory) | 17:40 |
artom | But... I'm also realistic | 17:40 |
*** salv-orlando has joined #openstack-nova | 17:45 | |
*** Eran_Kuris has quit IRC | 17:46 | |
*** sahid has quit IRC | 17:47 | |
*** gyee has joined #openstack-nova | 17:47 | |
lyarwood | melwitt: thanks for your review of https://review.openstack.org/#/c/523958/ btw, are you going to be able to hit it again today? | 17:55 |
*** sridharg has quit IRC | 17:56 | |
melwitt | lyarwood: yes, will re-review today | 17:56 |
lyarwood | melwitt: cool, thanks | 17:57 |
*** derekh has quit IRC | 18:01 | |
*** gouthamr has joined #openstack-nova | 18:01 | |
*** ralonsoh__ has quit IRC | 18:03 | |
*** edand__ has quit IRC | 18:03 | |
*** david-lyle has quit IRC | 18:07 | |
*** jackie-truong has joined #openstack-nova | 18:07 | |
openstackgerrit | melanie witt proposed openstack/nova master: libvirt: pass the mdevs when rebooting the guest https://review.openstack.org/533818 | 18:08 |
*** tidwellr_ has joined #openstack-nova | 18:13 | |
*** tidwellr has quit IRC | 18:13 | |
Spazmotic | Whew busy day | 18:22 |
Spazmotic | If any core's are able to spare some cycles today for a XenAPI review, would love it. I am getting ready to head to bed but let me know if you have any questions. https://review.openstack.org/#/c/533168/ | 18:25 |
*** janki has quit IRC | 18:26 | |
*** links has quit IRC | 18:26 | |
*** smatzek_ has quit IRC | 18:27 | |
*** AlexeyAbashkin has joined #openstack-nova | 18:31 | |
*** dtruong has quit IRC | 18:34 | |
*** lucasagomes is now known as lucas-afk | 18:35 | |
*** AlexeyAbashkin has quit IRC | 18:35 | |
*** mdnadeem has quit IRC | 18:39 | |
*** links has joined #openstack-nova | 18:40 | |
ildikov | mriedem: stvnoyes: sorry, partially off today | 18:41 |
openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.61 - Add trusted_image_certificates https://review.openstack.org/500396 | 18:47 |
*** fragatina has quit IRC | 18:48 | |
*** jpena is now known as jpena|off | 18:48 | |
*** dtantsur is now known as dtantsur|afk | 18:48 | |
*** fragatina has joined #openstack-nova | 18:49 | |
*** gszasz has joined #openstack-nova | 18:50 | |
*** smatzek has quit IRC | 18:54 | |
mriedem | ildikov: np, i reported a bug against cinder, seems to be intermittent timing issue anyway | 18:54 |
*** cdent has quit IRC | 18:55 | |
*** smatzek has joined #openstack-nova | 18:56 | |
*** smatzek_ has joined #openstack-nova | 18:58 | |
ildikov | mriedem: that does not sound good | 19:00 |
*** smatzek has quit IRC | 19:00 | |
ildikov | mriedem: the timing issue I mean | 19:00 |
ildikov | mriedem: Tnx for looking into it and reporting the big | 19:01 |
ildikov | *bug | 19:01 |
*** eharney has quit IRC | 19:03 | |
*** david-lyle has joined #openstack-nova | 19:04 | |
*** jackie-truong has quit IRC | 19:05 | |
*** smatzek_ has quit IRC | 19:09 | |
*** moshele has quit IRC | 19:10 | |
*** acormier has joined #openstack-nova | 19:12 | |
*** liusheng has quit IRC | 19:14 | |
*** liusheng has joined #openstack-nova | 19:15 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Make sure that functional test triggered on sample changes https://review.openstack.org/536934 | 19:16 |
*** links has quit IRC | 19:19 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Make sure that functional test triggered on sample changes https://review.openstack.org/536936 | 19:19 |
*** felipemonteiro has joined #openstack-nova | 19:19 | |
*** smatzek has joined #openstack-nova | 19:23 | |
*** smatzek has quit IRC | 19:29 | |
*** penick has quit IRC | 19:29 | |
*** gszasz has quit IRC | 19:31 | |
*** gszasz has joined #openstack-nova | 19:31 | |
*** jackie-truong has joined #openstack-nova | 19:33 | |
*** gouthamr_ has joined #openstack-nova | 19:34 | |
*** harlowja has joined #openstack-nova | 19:34 | |
*** gouthamr has quit IRC | 19:36 | |
mriedem | alex_xu: remember to update the placement API reference docs with the new 'required' parameter to GET /allocation_candidates https://review.openstack.org/#/c/535642/ | 19:36 |
melwitt | I was looking at this bug and it looks like the auto-disabling of compute can be used to DoS the cloud https://bugs.launchpad.net/nova/+bug/1742102 | 19:36 |
openstack | Launchpad bug 1742102 in OpenStack Compute (nova) "Simple user can disable compute" [Undecided,Confirmed] | 19:36 |
mriedem | melwitt: dansmith and i talked with the person that reported it a couple of weeks ago | 19:37 |
mriedem | problem is they are getting a super generic error on the nova side (VirtualInterfaceException due to the vif plugging failing) | 19:37 |
*** smatzek has joined #openstack-nova | 19:37 | |
mriedem | all that nova gets back from neutron is the vif plug failed event, with no details about why it failed | 19:37 |
mriedem | so in that case we don't know if it's specific to that host or what | 19:37 |
dansmith | which is normally a thing worth disabling the compute over | 19:38 |
mriedem | we could whitelist a set of exceptions that we ignore for the auto-disable, but that gets tricky | 19:38 |
melwitt | the latest comment demonstrates it can be done via OverQuota, which is something I think we could avoid | 19:38 |
*** fragatina has quit IRC | 19:38 | |
mriedem | i think they said it wasn't a quota issue, | 19:38 |
mriedem | because we check port quota in the api before casting to compute to build the instance and the port, | 19:38 |
*** amoralej is now known as amoralej|off | 19:38 | |
mriedem | the issue was someting to do with no more fixed IPs in the pool for the network backend | 19:39 |
mriedem | if there is a way to preemptively know about that fixed IP thing, yeah we could check for that ahead of time | 19:39 |
melwitt | agreed it's tricky. the latest comment is an example involving volume quota, if the volume quota is 10 and they try to create 100 instances, 90 fail and they got 9 computes disabled | 19:40 |
dansmith | we should totally be able to distinguish something like that and not count that against the compute | 19:41 |
mriedem | yeah i know we definitely don't check volume quota, even in the api | 19:41 |
mriedem | https://review.openstack.org/#/c/520122/ | 19:41 |
melwitt | it feels like maybe auto-disable shouldn't be the default, it looks like this is easy to exploit | 19:41 |
*** smatzek has quit IRC | 19:42 | |
melwitt | I'm thinking of if a public cloud is running openstack, someone could do this and get all the computes disabled. unless I'm missing something | 19:42 |
dansmith | in a public cloud it's less likely | 19:42 |
dansmith | because a single successful boot will reset the counter | 19:42 |
dansmith | I mean less likely IMHO | 19:42 |
*** smatzek has joined #openstack-nova | 19:43 | |
mriedem | the fact we have no limit on the multiple instance create is a problem since you can just say give me 1000 instances, which can cause other issues like the rpc call to select_destinations to timeout | 19:43 |
mriedem | but much harder to mess with that when i spec'ed up some ideas | 19:43 |
mriedem | https://review.openstack.org/#/c/510235/ | 19:44 |
dansmith | we could do something like record the user-id of the last failure and not count subsequent fails by a single user, although that would fail for single-tenant clouds (i.e. NFV) | 19:44 |
melwitt | thinking ... I think all you'd need is a burst of requests to pretty well ensure the counter won't reset because usually the scheduler will pack. but yeah, maybe it's not as easy to exploit in a public cloud as I was thinking | 19:46 |
dansmith | I'm definitely in favor of catching and excluding quota-type failures if and where we can | 19:47 |
*** smatzek has quit IRC | 19:47 | |
*** Guest87011 has joined #openstack-nova | 19:47 | |
melwitt | yeah, that one is at least obvious. so any OverQuota that came come from compute (neutron/cinder) | 19:48 |
*** smatzek has joined #openstack-nova | 19:49 | |
*** salv-orlando has quit IRC | 19:51 | |
*** salv-orlando has joined #openstack-nova | 19:51 | |
*** smatzek has quit IRC | 19:53 | |
*** smatzek has joined #openstack-nova | 19:55 | |
*** gszasz has quit IRC | 19:55 | |
*** Guest87011 has quit IRC | 19:56 | |
*** salv-orlando has quit IRC | 19:56 | |
*** smatzek has quit IRC | 19:59 | |
*** jackie-truong_ has joined #openstack-nova | 20:01 | |
*** smatzek has joined #openstack-nova | 20:01 | |
*** jackie-truong has quit IRC | 20:01 | |
*** eharney has joined #openstack-nova | 20:02 | |
*** smatzek has quit IRC | 20:06 | |
*** sambetts is now known as sambetts|afk | 20:06 | |
*** itlinux has joined #openstack-nova | 20:07 | |
*** eharney_ has joined #openstack-nova | 20:07 | |
*** eharney_ has quit IRC | 20:07 | |
mriedem | cellsv2 map_instances usage bug thingy https://bugs.launchpad.net/nova/+bug/1742649 | 20:08 |
openstack | Launchpad bug 1742649 in nova (Ubuntu) "map_instances default batch size is too small." [Undecided,New] | 20:08 |
mriedem | mapping 200K instances took a long time | 20:08 |
*** edmondsw has quit IRC | 20:08 | |
*** penick has joined #openstack-nova | 20:08 | |
*** edmondsw has joined #openstack-nova | 20:09 | |
melwitt | that's a lot of instances | 20:10 |
mriedem | because it maps all of them in a single shot but in batches of 50 | 20:10 |
*** smatzek has joined #openstack-nova | 20:13 | |
melwitt | urgh. so really it would be best to have it similar to what we have in archive_deleted_rows now where you can give the max batch size but also say --until-complete | 20:13 |
*** edmondsw has quit IRC | 20:13 | |
mriedem | that's what he asked for in there yeah | 20:14 |
mriedem | "Actually, it would be useful to be able to say map all the instances using this batch size, eg: nova-manage --config-file /etc/nova/nova.conf cell_v2 map_instances --cell_uuid <uuid> --max_count all --batch_size 50000 " | 20:14 |
melwitt | yeah, I was thinking that sounded familiar and saw we have it for archive_deleted_rows | 20:14 |
mriedem | the thing about archive_deleted_rows is you're going to run that many times throughout the life of the deployment | 20:15 |
mriedem | map_instances is a done time deal when you're upgrading to cells v2 | 20:15 |
mriedem | *one time | 20:15 |
melwitt | yeah | 20:15 |
melwitt | I dunno, it could be a low-hanging-fruit thing to do | 20:15 |
*** edmondsw has joined #openstack-nova | 20:15 | |
mriedem | i guess | 20:16 |
mriedem | i'm not sure i'd want to review it though | 20:16 |
mriedem | high hanging review fruit | 20:16 |
melwitt | I wasn't thinking it would be hard to review. maybe I'm underestimating it | 20:17 |
*** smatzek has quit IRC | 20:17 | |
mriedem | the only thing you're underestimating is my laziness at the moment | 20:18 |
*** smatzek has joined #openstack-nova | 20:19 | |
melwitt | heh. fair enough :) | 20:19 |
mriedem | i'll put something in the docs and someone else can wizbang up a fancy new batch-size option or whatever | 20:19 |
*** edmondsw has quit IRC | 20:19 | |
*** amodi has quit IRC | 20:20 | |
melwitt | okay. I'll mention it at the cells meeting and see if tssurya might be interested | 20:21 |
mriedem | alright. already threw it on the meeting agenda | 20:21 |
mriedem | in the bugs list | 20:21 |
melwitt | coolness | 20:21 |
mriedem | btw, have we heard any updates from belmiro on the great cellsv2ification at cern? | 20:21 |
mriedem | and how it's going? | 20:22 |
melwitt | I haven't heard anything yet. just implied from the fixes they've been doing for other nova-manage cell_v2 commands, that they've been working on it | 20:22 |
melwitt | I think they'll need an improvement to map_instances themselves lest it take hours to run, heh | 20:23 |
*** smatzek has quit IRC | 20:23 | |
melwitt | if they haven't already gone through it | 20:23 |
*** smatzek has joined #openstack-nova | 20:24 | |
*** Jeffrey4l has quit IRC | 20:25 | |
*** Jeffrey4l has joined #openstack-nova | 20:25 | |
*** edmondsw has joined #openstack-nova | 20:26 | |
openstackgerrit | Hongbin Lu proposed openstack/nova master: Handle RPC timeout on get_console_output https://review.openstack.org/525335 | 20:27 |
*** smatzek has quit IRC | 20:29 | |
*** amodi has joined #openstack-nova | 20:32 | |
*** smatzek has joined #openstack-nova | 20:32 | |
*** mvenesio_ has quit IRC | 20:32 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Expand on when you might want to set --max-count for map_instances https://review.openstack.org/536959 | 20:33 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Expand on when you might want to set --max-count for map_instances https://review.openstack.org/536959 | 20:34 |
*** awaugama has quit IRC | 20:35 | |
*** penick has quit IRC | 20:36 | |
*** penick has joined #openstack-nova | 20:36 | |
*** mvenesio has joined #openstack-nova | 20:39 | |
*** salv-orlando has joined #openstack-nova | 20:39 | |
*** mvenesio has quit IRC | 20:43 | |
*** stvnoyes1 has quit IRC | 20:48 | |
*** stvnoyes has quit IRC | 20:48 | |
*** jackie-truong_ has quit IRC | 20:50 | |
melwitt | what is with "libvirtError: Cannot recv data: Connection reset by peer" on stable, it's been flipping out, and specifically on the cells v1 job | 20:50 |
*** jackie-truong has joined #openstack-nova | 20:51 | |
mriedem | is libvirt crashing? | 20:52 |
mriedem | check the libvirtd logs? | 20:52 |
mriedem | Roamer`: done https://review.openstack.org/#/c/140733/ | 20:52 |
* melwitt looks | 20:53 | |
*** moshele has joined #openstack-nova | 20:54 | |
Roamer` | mriedem, just got the e-mail, thanks! Looking at it... | 20:58 |
*** itlinux has quit IRC | 20:59 | |
*** nicolasbock has joined #openstack-nova | 20:59 | |
*** moshele has quit IRC | 20:59 | |
tonyb | mriedem: my plan is to do the double checking today and then ask infra. | 21:01 |
tonyb | mriedem: (re tagging nova newton-eol) | 21:01 |
melwitt | doesn't look like a crash but I'm not too familiar with libvirtd logging. I see a message '"kvm" accelerator not found' several times http://logs.openstack.org/97/536897/1/check/legacy-tempest-dsvm-cells/1f42e60/logs/libvirt/libvirtd.txt.gz#_2018-01-23_18_37_23_873 | 21:01 |
*** moshele has joined #openstack-nova | 21:01 | |
*** sdague has quit IRC | 21:03 | |
*** pramodrj07 has joined #openstack-nova | 21:03 | |
*** mvk has joined #openstack-nova | 21:04 | |
*** damien_r has quit IRC | 21:05 | |
*** damien_r has joined #openstack-nova | 21:05 | |
*** lyan has quit IRC | 21:05 | |
mriedem | doesn't look like it | 21:06 |
mriedem | tonyb: ok | 21:06 |
melwitt | comparing libvirtd.log with master change | 21:07 |
tonyb | melwitt: I wouldn't think that was a problem in a dsvm test | 21:08 |
*** lyan has joined #openstack-nova | 21:08 | |
melwitt | I don't find the kvm accelerator message in the master change libvirtd log | 21:08 |
*** moshele has quit IRC | 21:09 | |
*** penick has quit IRC | 21:09 | |
mriedem | melwitt: what's the virt_type in nova.conf? | 21:09 |
mriedem | should be qemu | 21:09 |
mriedem | note that the version of libvirt is probably different between stable and master job runs | 21:10 |
mriedem | we use the pike UCA in master | 21:10 |
melwitt | huh, the nova-cpu.conf is very empty. http://logs.openstack.org/97/536897/1/check/legacy-tempest-dsvm-cells/1f42e60/logs/etc/nova/nova-cpu.conf.txt.gz | 21:10 |
melwitt | but in nova.conf virt_type = qemu | 21:10 |
*** penick has joined #openstack-nova | 21:11 | |
*** smatzek_ has joined #openstack-nova | 21:15 | |
*** smatzek_ has quit IRC | 21:15 | |
*** gszasz has joined #openstack-nova | 21:16 | |
*** penick has quit IRC | 21:16 | |
melwitt | tonyb: what do you mean? the kvm accelerator thing shouldn't hurt? | 21:22 |
mriedem | melwitt: the cellsv1 job is probably not running in superconductor mode, | 21:23 |
mriedem | so doesn't use nova-cpu.conf | 21:23 |
melwitt | oh, okay | 21:23 |
*** smatzek has quit IRC | 21:23 | |
tonyb | melwitt: just that in dsvm I'd expect kvm accelerator to me missing (as we should be using qemu), but it it strange that it's even looking for it | 21:24 |
melwitt | bah, there's a legit fail in the functional test_server_group tests on stable too. working on that | 21:24 |
*** penick has joined #openstack-nova | 21:24 | |
melwitt | tonyb: oh, I see | 21:24 |
*** moshele has joined #openstack-nova | 21:27 | |
*** penick has quit IRC | 21:27 | |
*** slaweq has quit IRC | 21:28 | |
*** penick has joined #openstack-nova | 21:29 | |
openstackgerrit | Merged openstack/nova master: ProviderTree.populate_from_iterable https://review.openstack.org/520756 | 21:30 |
*** jackie-truong has quit IRC | 21:30 | |
*** slaweq has joined #openstack-nova | 21:31 | |
*** jackie-truong has joined #openstack-nova | 21:32 | |
*** edmondsw has quit IRC | 21:33 | |
openstackgerrit | Merged openstack/nova master: Replace curly quotes with straight quotes https://review.openstack.org/536727 | 21:33 |
*** edmondsw has joined #openstack-nova | 21:33 | |
melwitt | curly quotes eh? | 21:34 |
openstackgerrit | Hongbin Lu proposed openstack/nova master: Handle RPC timeout on get_console_output https://review.openstack.org/525335 | 21:35 |
jackie-truong | If any core reviewer has a chance to take a look at this patch, it would be very much appreciated: https://review.openstack.org/#/c/486204/ | 21:35 |
*** cleong has quit IRC | 21:35 | |
*** slaweq has quit IRC | 21:36 | |
jackie-truong | jaypipes was looking at the patch series, but he's out at a work conference at the moment. He had taken away +2 from its dependent patch (https://review.openstack.org/#/c/489408/) due to the patch series not being complete. But patch series is complete now :-) | 21:37 |
jackie-truong | Usage and testing instructions can be found here: https://etherpad.openstack.org/p/queens-nova-certificate-validation | 21:38 |
*** edmondsw has quit IRC | 21:38 | |
jackie-truong | And we also wrote a barbican-tempest-plugin test for it: https://review.openstack.org/#/c/515210/ | 21:38 |
*** penick has quit IRC | 21:41 | |
*** gouthamr_ has quit IRC | 21:42 | |
*** dave-mccowan has quit IRC | 21:43 | |
*** penick has joined #openstack-nova | 21:43 | |
openstackgerrit | melanie witt proposed openstack/nova stable/pike: Set server status to ERROR if rebuild failed https://review.openstack.org/536897 | 21:46 |
openstackgerrit | melanie witt proposed openstack/nova stable/pike: Fix false positive server group functional tests https://review.openstack.org/536981 | 21:46 |
*** fragatina has joined #openstack-nova | 21:47 | |
*** tidwellr_ has quit IRC | 21:54 | |
*** pchavva has quit IRC | 21:57 | |
*** felipemonteiro_ has joined #openstack-nova | 22:01 | |
*** dave-mccowan has joined #openstack-nova | 22:03 | |
openstackgerrit | Peter Penchev proposed openstack/nova master: Add the Nova libvirt StorPool attachment driver. https://review.openstack.org/140733 | 22:03 |
*** slaweq has joined #openstack-nova | 22:03 | |
*** priteau has quit IRC | 22:03 | |
*** slaweq has quit IRC | 22:04 | |
mriedem | dansmith: on alex_xu's 1.12 allocation candidate change, i assume you ran into the same sticky part here https://review.openstack.org/#/c/536083/4/nova/scheduler/client/report.py@1153 | 22:05 |
*** felipemonteiro has quit IRC | 22:05 | |
mriedem | with assuming that the client is getting a 1.12 format request, | 22:05 |
mriedem | we could be in the middle of a server create / cold migrate where the format is actually 1.10, you upgrade and restart conductor, and now the client code assumes it's going to get a 1.12 but it's 1.10 and kablammo | 22:06 |
mriedem | via reschedule from the compute | 22:06 |
mriedem | because we don't have a versioned object facade for this payload | 22:06 |
*** amodi has quit IRC | 22:09 | |
Roamer` | mriedem, new patchset up; thanks a lot for the very useful comments! | 22:10 |
Roamer` | (I guess now we wait for 12 hours or so for the checks to pass...) | 22:10 |
*** rcernin has joined #openstack-nova | 22:15 | |
*** moshele has quit IRC | 22:16 | |
*** penick has quit IRC | 22:16 | |
mriedem | Roamer`: do you have a 3rd party CI that will eventually comment on this change to say if it's working? | 22:17 |
Roamer` | mriedem, actually, yes, we do; the logs are at http://logs.ci-openstack.storpool.com/ but the run it's doing right now is not with this change yet, the next one will be | 22:19 |
Roamer` | so I guess in about two hours or so there'll be a run with this one | 22:19 |
Roamer` | (yeah, I should really look into speeding the CI runs up a bit) | 22:19 |
Roamer` | it's just Cinder though, not Nova | 22:20 |
Roamer` | I mean, it's the Tempest volume tests, not all the Nova-related tests | 22:20 |
mriedem | but at some point it's going to at least run a tempest compute api test to attach and detach a volume to a server instance yeah? | 22:20 |
Roamer` | yes, it will | 22:20 |
mriedem | ok | 22:21 |
Roamer` | I may not be awake when it does (just after midnight here), but I'll post a link in about seven hours | 22:21 |
Roamer` | then you won't be awake :) | 22:21 |
*** penick has joined #openstack-nova | 22:21 | |
mriedem | oh i'll be awake | 22:21 |
mriedem | sitting by the phone | 22:21 |
mriedem | anyone know when jaypipes is going to be back? | 22:24 |
jackie-truong | mriedem: All I heard is "at a work conference this week" | 22:24 |
mriedem | alright | 22:25 |
Roamer` | hopefully you won't have to rescue the mailman again... here in Sofia we have a suburb up in the mountain where I used to live for like 20 years, and the snow is a bit of a problem sometimes, like public transport going "nuh-uh, not today" | 22:25 |
mriedem | i was unable to rescue the mailman - he had to get a tow truck | 22:25 |
mriedem | rear-wheel drive with go-kart tires on an inline in a foot of snow isn't going to cut it | 22:26 |
mriedem | *incline | 22:26 |
Roamer` | oh, I hear you... there was this New Year's Eve when a neighbor had a party, he specifically warned people not to drive if they don't have the tires, and as a result there were three taxi cabs stuck in the snow on our street | 22:27 |
mriedem | i did enjoy watching the neighbor across the street, who is new to snow, try to drive his bmw sedan over the 3 foot wall of ice that the plow left in front of his driveway, and of course get it stuck halfway | 22:27 |
*** yamahata has joined #openstack-nova | 22:28 | |
melwitt | I have read that the most important thing is to have winter tires. and that 4x4 isn't necessarily needed? | 22:28 |
mriedem | i've never had winter tires | 22:28 |
mriedem | or 4x3 | 22:28 |
mriedem | *4x4 | 22:28 |
mriedem | just don't be a jackhole | 22:28 |
mriedem | and put a shovel in the trunk | 22:28 |
melwitt | heh, good to know | 22:29 |
mriedem | laura's got the 4x4 in our family | 22:29 |
jroll | melwitt: 4wd helps you go, winter tires help you stop :P | 22:29 |
*** penick has quit IRC | 22:29 | |
mriedem | pump those brakes | 22:29 |
openstackgerrit | melanie witt proposed openstack/nova stable/ocata: Set server status to ERROR if rebuild failed https://review.openstack.org/536904 | 22:29 |
openstackgerrit | melanie witt proposed openstack/nova stable/ocata: Fix false positive server group functional tests https://review.openstack.org/536995 | 22:29 |
jroll | don't be a jackhole is definitely the best advice though | 22:30 |
* melwitt records info for if she's ever driving in snow | 22:30 | |
mriedem | ha, that reminds me - saw a minivan racing down the hill towing like 6 kids on a sled yesterday, | 22:30 |
mriedem | if they would have had to stop short, all of those kids could have been destoryed | 22:30 |
mriedem | *destroyed | 22:30 |
Roamer` | well, they *might* have been able to roll to the side, fall off the sled, but still... no, not a good image at all | 22:31 |
*** jackie-truong has quit IRC | 22:31 | |
mriedem | the ones that are destined to pass on their genes, yes they might have had the instinct | 22:31 |
*** Guest87011 has joined #openstack-nova | 22:31 | |
mriedem | or, be in the back | 22:32 |
*** felipemonteiro_ has quit IRC | 22:32 | |
melwitt | *facepalm* | 22:32 |
dansmith | mriedem: hadn't even gotten that far actually, but yeah, newer code will probably need to handle the case and transform older ones to newer ones | 22:32 |
dansmith | mriedem: and yes, that's related enough to what I was saying about passing this dict of allocations/resources around to need some wrapage | 22:33 |
mriedem | i guess you could determine format based on whether or not the 'allocations' entry within is a list or dict, and swaparoo if it's not a dict | 22:34 |
*** acormier has quit IRC | 22:34 | |
*** acormier has joined #openstack-nova | 22:34 | |
dansmith | for this case I think it can be done, yeah, but it's not a good precedent | 22:35 |
*** acormier has joined #openstack-nova | 22:35 | |
mriedem | agree | 22:36 |
openstackgerrit | melanie witt proposed openstack/nova stable/ocata: Fix false positive server group functional tests https://review.openstack.org/536995 | 22:36 |
openstackgerrit | melanie witt proposed openstack/nova stable/ocata: Set server status to ERROR if rebuild failed https://review.openstack.org/536904 | 22:36 |
*** dave-mccowan has quit IRC | 22:37 | |
*** acormier has quit IRC | 22:37 | |
*** acormier has joined #openstack-nova | 22:37 | |
*** esberglu has quit IRC | 22:38 | |
*** gszasz_ has joined #openstack-nova | 22:38 | |
*** gszasz has quit IRC | 22:40 | |
*** fragatina has quit IRC | 22:50 | |
*** fragatina has joined #openstack-nova | 22:50 | |
*** cburgess has quit IRC | 23:00 | |
*** cburgess has joined #openstack-nova | 23:01 | |
*** mvk has quit IRC | 23:02 | |
*** artom has quit IRC | 23:03 | |
openstackgerrit | Hongbin Lu proposed openstack/nova master: [WIP] Use revision_number on adding/removing security group https://review.openstack.org/535510 | 23:10 |
*** acormier has quit IRC | 23:12 | |
*** amodi has joined #openstack-nova | 23:12 | |
*** acormier has joined #openstack-nova | 23:13 | |
*** salv-orlando has quit IRC | 23:13 | |
*** masber has quit IRC | 23:14 | |
*** salv-orlando has joined #openstack-nova | 23:14 | |
*** mvk has joined #openstack-nova | 23:17 | |
*** acormier has quit IRC | 23:18 | |
*** salv-orlando has quit IRC | 23:19 | |
*** hongbin has quit IRC | 23:25 | |
*** acormier has joined #openstack-nova | 23:26 | |
*** acormier has quit IRC | 23:30 | |
Roamer` | mriedem, FTR, the StorPool CI is now doing a run with the latest patchset, and it just passed some tempest.api.compute.* tests; I'm going to sleep now, but I'll send you a link to the logs when I wake up, or you could check http://logs.ci-openstack.storpool.com/ for 534854/4 in an hour - hour and a half | 23:41 |
*** oanson has quit IRC | 23:42 | |
*** jackie-truong has joined #openstack-nova | 23:42 | |
*** oanson has joined #openstack-nova | 23:43 | |
*** dave-mccowan has joined #openstack-nova | 23:51 | |
*** takashin has joined #openstack-nova | 23:52 | |
*** smatzek has joined #openstack-nova | 23:53 | |
*** gszasz_ has quit IRC | 23:55 | |
*** smatzek has quit IRC | 23:55 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!