| *** owalsh has joined #openstack-nova | 00:01 | |
| *** thorst has joined #openstack-nova | 00:02 | |
| *** thorst has quit IRC | 00:08 | |
| *** ircuser-1 has quit IRC | 00:08 | |
| *** gmann has quit IRC | 00:11 | |
| *** gmann has joined #openstack-nova | 00:11 | |
| *** iceyao has joined #openstack-nova | 00:19 | |
| *** fragatina has joined #openstack-nova | 00:19 | |
| *** iceyao has quit IRC | 00:23 | |
| *** fragatina has quit IRC | 00:24 | |
| *** thorst has joined #openstack-nova | 00:36 | |
| *** thorst has quit IRC | 00:36 | |
| *** mriedem has quit IRC | 00:42 | |
| *** abalutoiu_ has quit IRC | 00:43 | |
| *** zhurong has joined #openstack-nova | 00:47 | |
| *** trungnv has quit IRC | 00:53 | |
| *** iceyao has joined #openstack-nova | 00:57 | |
| *** trungnv has joined #openstack-nova | 00:57 | |
| *** ijw has joined #openstack-nova | 00:59 | |
| *** iceyao has quit IRC | 01:02 | |
| *** dixiaoli has joined #openstack-nova | 01:04 | |
| *** liusheng has quit IRC | 01:04 | |
| *** liusheng has joined #openstack-nova | 01:09 | |
| *** iceyao has joined #openstack-nova | 01:12 | |
| *** gouthamr has quit IRC | 01:13 | |
| *** fragatina has joined #openstack-nova | 01:20 | |
| *** fragatina has quit IRC | 01:25 | |
| *** ijw_ has joined #openstack-nova | 01:28 | |
| *** ijw has quit IRC | 01:31 | |
| *** ijw has joined #openstack-nova | 01:32 | |
| *** dave-mccowan has joined #openstack-nova | 01:33 | |
| *** ijw_ has quit IRC | 01:36 | |
| *** hongbin has joined #openstack-nova | 01:36 | |
| *** yushb has joined #openstack-nova | 01:40 | |
| *** gbarros_ has quit IRC | 01:46 | |
| *** thorst has joined #openstack-nova | 01:52 | |
| *** thorst has quit IRC | 01:56 | |
| *** iceyao has quit IRC | 01:56 | |
| *** gbarros has joined #openstack-nova | 02:03 | |
| *** xiefp has joined #openstack-nova | 02:06 | |
| *** gbarros has quit IRC | 02:08 | |
| *** lbragstad has joined #openstack-nova | 02:10 | |
| *** ijw has quit IRC | 02:12 | |
| *** baoli has joined #openstack-nova | 02:21 | |
| *** baoli has quit IRC | 02:28 | |
| *** zzzeek_ has joined #openstack-nova | 02:30 | |
| *** ijw has joined #openstack-nova | 02:36 | |
| *** ijw_ has joined #openstack-nova | 02:37 | |
| *** ijw has quit IRC | 02:41 | |
| *** ijw__ has joined #openstack-nova | 02:41 | |
| *** zhurong has quit IRC | 02:43 | |
| *** ijw_ has quit IRC | 02:44 | |
| *** ijw__ has quit IRC | 02:46 | |
| *** ijw has joined #openstack-nova | 02:56 | |
| *** fragatina has joined #openstack-nova | 02:58 | |
| *** ijw has quit IRC | 03:01 | |
| *** fragatina has quit IRC | 03:03 | |
| *** stvnoyes has quit IRC | 03:04 | |
| *** fragatina has joined #openstack-nova | 03:09 | |
| *** gcb has joined #openstack-nova | 03:12 | |
| *** fragatina has quit IRC | 03:13 | |
| *** reedip has quit IRC | 03:13 | |
| *** reedip has joined #openstack-nova | 03:25 | |
| *** burgerk has joined #openstack-nova | 03:33 | |
| *** udesale has joined #openstack-nova | 03:36 | |
| *** ijw has joined #openstack-nova | 03:41 | |
| *** gbarros has joined #openstack-nova | 03:45 | |
| *** yamamoto has joined #openstack-nova | 03:47 | |
| *** slaweq has joined #openstack-nova | 03:47 | |
| *** Dinesh_Bhor has joined #openstack-nova | 03:52 | |
| *** slaweq has quit IRC | 03:52 | |
| *** thorst has joined #openstack-nova | 03:53 | |
| *** psachin has joined #openstack-nova | 03:56 | |
| *** thorst has quit IRC | 03:57 | |
| *** hongbin has quit IRC | 03:58 | |
| *** armax has joined #openstack-nova | 04:01 | |
| *** armax has quit IRC | 04:03 | |
| *** lbragstad has quit IRC | 04:04 | |
| *** dave-mccowan has quit IRC | 04:05 | |
| *** iceyao has joined #openstack-nova | 04:07 | |
| *** adisky__ has joined #openstack-nova | 04:07 | |
| *** iceyao has quit IRC | 04:12 | |
| *** ijw has quit IRC | 04:20 | |
| *** yushb has quit IRC | 04:20 | |
| *** sridharg has joined #openstack-nova | 04:22 | |
| *** gbarros has quit IRC | 04:24 | |
| *** ekuris has quit IRC | 04:24 | |
| *** mdnadeem has joined #openstack-nova | 04:28 | |
| *** gcb has quit IRC | 04:30 | |
| *** burgerk has quit IRC | 04:36 | |
| *** yushb has joined #openstack-nova | 04:37 | |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: remove extension pop https://review.openstack.org/481491 | 04:42 |
|---|---|---|
| *** zhurong has joined #openstack-nova | 04:47 | |
| *** trinaths has joined #openstack-nova | 04:48 | |
| *** yushb has quit IRC | 04:54 | |
| *** trinaths has quit IRC | 05:04 | |
| masber | good afternoon, I am trying to create a server but apparently I am exceeding the quota? how is it possible? http://paste.openstack.org/raw/614863/ | 05:07 |
| *** trinaths has joined #openstack-nova | 05:08 | |
| *** mlakat has quit IRC | 05:10 | |
| *** mlakat has joined #openstack-nova | 05:10 | |
| *** fragatina has joined #openstack-nova | 05:10 | |
| *** moshele has joined #openstack-nova | 05:15 | |
| *** fragatina has quit IRC | 05:16 | |
| *** hshiina has joined #openstack-nova | 05:16 | |
| *** moshele has quit IRC | 05:18 | |
| *** ekuris has joined #openstack-nova | 05:23 | |
| *** bhagyashris has quit IRC | 05:24 | |
| *** prateek has joined #openstack-nova | 05:36 | |
| *** vvargaszte has quit IRC | 05:45 | |
| *** ratailor has joined #openstack-nova | 05:46 | |
| *** vvargaszte has joined #openstack-nova | 05:52 | |
| *** thorst has joined #openstack-nova | 05:54 | |
| *** phuongnh has joined #openstack-nova | 05:57 | |
| *** thorst has quit IRC | 05:59 | |
| *** gbarros has joined #openstack-nova | 06:01 | |
| *** yushb has joined #openstack-nova | 06:05 | |
| openstackgerrit | Shunli Zhou proposed openstack/nova master: Transform servergroup.create notification https://review.openstack.org/454023 | 06:06 |
| *** gbarros has quit IRC | 06:07 | |
| *** blaisebool has joined #openstack-nova | 06:10 | |
| *** bhagyashris has joined #openstack-nova | 06:10 | |
| *** Oku_OS-away is now known as Oku_OS | 06:10 | |
| *** blaisebool is now known as Guest67678 | 06:10 | |
| *** dixiaoli has quit IRC | 06:18 | |
| *** dixiaoli has joined #openstack-nova | 06:18 | |
| *** vvargaszte1 has joined #openstack-nova | 06:19 | |
| *** andreas_s has joined #openstack-nova | 06:20 | |
| *** vvargaszte has quit IRC | 06:21 | |
| *** vvargaszte1 is now known as vvargaszte | 06:21 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova master: Ironic: Support boot from Cinder volume https://review.openstack.org/215385 | 06:22 |
| *** bkopilov has joined #openstack-nova | 06:25 | |
| openstackgerrit | jichenjc proposed openstack/nova master: remove extension param and usage https://review.openstack.org/481491 | 06:26 |
| *** ociuhandu has quit IRC | 06:26 | |
| *** ociuhandu has joined #openstack-nova | 06:27 | |
| *** alexchadin has joined #openstack-nova | 06:27 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Enhance support matrix document https://review.openstack.org/482013 | 06:32 |
| openstackgerrit | bhavani proposed openstack/nova master: Stop using deprecated 'message' attribute in Exception https://review.openstack.org/482014 | 06:34 |
| *** lpetrut has joined #openstack-nova | 06:37 | |
| *** iceyao has joined #openstack-nova | 06:39 | |
| *** claudiub has joined #openstack-nova | 06:39 | |
| *** moshele has joined #openstack-nova | 06:43 | |
| *** lpetrut has quit IRC | 06:43 | |
| *** iceyao has quit IRC | 06:43 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Enhance support matrix document https://review.openstack.org/482020 | 06:44 |
| *** rcernin has joined #openstack-nova | 06:47 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Modify some comments for virt driver https://review.openstack.org/482022 | 06:50 |
| openstackgerrit | jichenjc proposed openstack/nova master: Enhance support matrix document https://review.openstack.org/482020 | 06:51 |
| *** belmoreira has joined #openstack-nova | 06:54 | |
| *** Shunli has joined #openstack-nova | 06:59 | |
| *** iceyao has joined #openstack-nova | 07:00 | |
| *** abalutoiu_ has joined #openstack-nova | 07:04 | |
| openstackgerrit | bhavani proposed openstack/nova master: Stop using deprecated 'message' attribute in Exception https://review.openstack.org/482014 | 07:04 |
| *** iceyao has quit IRC | 07:05 | |
| *** amotoki_away is now known as amotoki | 07:11 | |
| *** harlowja has quit IRC | 07:11 | |
| *** gbarros has joined #openstack-nova | 07:12 | |
| *** fragatina has joined #openstack-nova | 07:13 | |
| *** zsli_ has joined #openstack-nova | 07:14 | |
| *** roeyc has joined #openstack-nova | 07:15 | |
| *** Shunli has quit IRC | 07:17 | |
| *** fragatina has quit IRC | 07:17 | |
| *** iceyao has joined #openstack-nova | 07:20 | |
| *** sshwarts has joined #openstack-nova | 07:22 | |
| *** litao__ has joined #openstack-nova | 07:25 | |
| *** iceyao has quit IRC | 07:25 | |
| *** damien_r has joined #openstack-nova | 07:25 | |
| *** kiennt has joined #openstack-nova | 07:26 | |
| *** ecelik has joined #openstack-nova | 07:27 | |
| *** ecelik has left #openstack-nova | 07:27 | |
| *** gbarros has quit IRC | 07:29 | |
| *** damien_r has quit IRC | 07:29 | |
| *** damien_r has joined #openstack-nova | 07:30 | |
| kiennt | hi, i saw Nova already had assert:supports-rolling-upgrade tag. How you test rolling upgrade in gate job? I am woking on Heat rolling upgrade and have to setup the CI to test this. | 07:31 |
| *** tesseract has joined #openstack-nova | 07:36 | |
| *** trungnv has quit IRC | 07:36 | |
| *** nicolasbock has joined #openstack-nova | 07:37 | |
| *** trungnv has joined #openstack-nova | 07:39 | |
| *** masber has quit IRC | 07:40 | |
| *** iceyao has joined #openstack-nova | 07:41 | |
| *** dixiaoli has quit IRC | 07:45 | |
| *** iceyao has quit IRC | 07:46 | |
| *** dixiaoli has joined #openstack-nova | 07:46 | |
| *** trungnv has quit IRC | 07:47 | |
| *** sambetts is now known as sambetts|afk | 07:47 | |
| *** mkucia has joined #openstack-nova | 07:50 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix quota class set APIs https://review.openstack.org/467999 | 07:51 |
| *** masber has joined #openstack-nova | 07:53 | |
| *** thorst has joined #openstack-nova | 07:54 | |
| *** iceyao has joined #openstack-nova | 07:56 | |
| *** sambetts|afk is now known as sambetts | 07:57 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** thorst has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-nova | 08:02 | |
| *** gszasz has joined #openstack-nova | 08:03 | |
| *** phuongnh has quit IRC | 08:07 | |
| *** phuongnh has joined #openstack-nova | 08:07 | |
| *** _pewp_ has quit IRC | 08:10 | |
| *** fragatina has joined #openstack-nova | 08:13 | |
| *** _pewp_ has joined #openstack-nova | 08:14 | |
| *** fragatina has quit IRC | 08:18 | |
| *** Guest67678 has quit IRC | 08:25 | |
| *** jistr|off is now known as jistr | 08:25 | |
| *** blaisebool has joined #openstack-nova | 08:28 | |
| *** blaisebool is now known as Guest55861 | 08:29 | |
| *** efoley has joined #openstack-nova | 08:31 | |
| *** afazekas|away is now known as afazekas | 08:34 | |
| *** jgrassler has quit IRC | 08:39 | |
| *** kiennt has quit IRC | 08:39 | |
| *** hieulq has quit IRC | 08:39 | |
| *** phuongnh has quit IRC | 08:40 | |
| *** vipul has joined #openstack-nova | 08:47 | |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova master: Ensure that periodic reclaim cleans DB deleted instances https://review.openstack.org/323250 | 08:49 |
| moshele | sean-k-mooney: hi | 08:50 |
| openstackgerrit | sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007 | 08:50 |
| moshele | sean-k-mooney: can you review my respond on https://review.openstack.org/#/c/398265/ ? | 08:50 |
| *** ralonsoh has joined #openstack-nova | 08:51 | |
| *** klipka has joined #openstack-nova | 08:52 | |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova master: Ensure that periodic reclaim cleans DB deleted instances https://review.openstack.org/323250 | 08:53 |
| *** felipemonteiro has joined #openstack-nova | 08:56 | |
| *** slunkad has joined #openstack-nova | 08:59 | |
| *** felipemonteiro has quit IRC | 09:01 | |
| *** cdent has joined #openstack-nova | 09:02 | |
| *** mdavidson has joined #openstack-nova | 09:04 | |
| *** lpetrut has joined #openstack-nova | 09:07 | |
| openstackgerrit | bhavani proposed openstack/nova master: Stop using deprecated 'message' attribute in Exception https://review.openstack.org/482014 | 09:14 |
| *** cdent has quit IRC | 09:14 | |
| *** trungnv has joined #openstack-nova | 09:14 | |
| *** kiennt has joined #openstack-nova | 09:15 | |
| *** hieulq has joined #openstack-nova | 09:15 | |
| openstackgerrit | Béla Vancsics proposed openstack/nova master: Transform instance-live_migration_pre notification https://review.openstack.org/480119 | 09:19 |
| ildikov | stephenfin: hi :) | 09:20 |
| stephenfin | ildikov: Morning :) | 09:20 |
| ildikov | stephenfin: just wanted to give you a heads up that there are a few positive review comments on the swap_volume change: https://review.openstack.org/#/c/456971/ | 09:21 |
| ildikov | stephenfin: in case that would help with reviewing it :) | 09:21 |
| *** mvk has joined #openstack-nova | 09:21 | |
| *** thorst has joined #openstack-nova | 09:23 | |
| stephenfin | ildikov: Can't do any harm :) I'll take a shot again today, but I can't make any promises | 09:24 |
| stephenfin | (just setting expectations) | 09:24 |
| ildikov | stephenfin: thanks | 09:25 |
| ildikov | stephenfin: Matt is basically fine with the patch; and it's also not that easy to find people in the core team who're familiar with this part of the code, so I thought to ping people first who reviewed related code changes earlier | 09:27 |
| *** ociuhandu has quit IRC | 09:27 | |
| ildikov | stephenfin: so no pressure, but it would be a big help :) | 09:27 |
| *** gbarros has joined #openstack-nova | 09:27 | |
| *** zsli_ has quit IRC | 09:27 | |
| stephenfin | ildikov: I must be missing something, but why didn't he +2 it himself? | 09:27 |
| *** thorst has quit IRC | 09:28 | |
| ildikov | stephenfin: he uploaded the last patch set with some more tests | 09:28 |
| openstackgerrit | Béla Vancsics proposed openstack/nova master: Transform instance-live_migration_pre notification https://review.openstack.org/482070 | 09:28 |
| gmann | alex_xu, can you check this -https://review.openstack.org/#/c/467999/9 | 09:32 |
| alex_xu | gmann: yea, I will review before the end of day | 09:32 |
| gmann | alex_xu, thanks | 09:33 |
| alex_xu | gmann: np, thanks for the patch | 09:33 |
| *** cdent has joined #openstack-nova | 09:34 | |
| *** karthiks has joined #openstack-nova | 09:38 | |
| *** vipul has quit IRC | 09:38 | |
| *** slaweq has joined #openstack-nova | 09:39 | |
| *** vancsics has joined #openstack-nova | 09:42 | |
| openstackgerrit | Béla Vancsics proposed openstack/nova master: Transform instance-live_migration_post notification https://review.openstack.org/480119 | 09:42 |
| *** vipul has joined #openstack-nova | 09:47 | |
| *** iceyao has quit IRC | 09:48 | |
| *** gbarros has quit IRC | 09:50 | |
| openstackgerrit | bhavani proposed openstack/nova master: Stop using deprecated 'message' attribute in Exception https://review.openstack.org/482014 | 09:55 |
| *** dtantsur is now known as dtantsur|bbl | 09:59 | |
| *** kiennt has quit IRC | 10:03 | |
| *** fragatina has joined #openstack-nova | 10:16 | |
| *** fragatina has quit IRC | 10:20 | |
| *** sdague has joined #openstack-nova | 10:21 | |
| *** yushb has quit IRC | 10:23 | |
| *** moshele has quit IRC | 10:25 | |
| *** moshele has joined #openstack-nova | 10:28 | |
| *** sridharg has quit IRC | 10:34 | |
| *** udesale has quit IRC | 10:44 | |
| *** xinliang has quit IRC | 10:53 | |
| *** gbarros has joined #openstack-nova | 10:55 | |
| *** hieulq has quit IRC | 10:57 | |
| *** dtantsur|bbl is now known as dtantsur | 10:57 | |
| *** dixiaoli has quit IRC | 11:00 | |
| *** lucasagomes is now known as lucas-hungry | 11:03 | |
| *** xinliang has joined #openstack-nova | 11:06 | |
| *** gbarros has quit IRC | 11:10 | |
| *** alexchadin has quit IRC | 11:10 | |
| *** hieulq has joined #openstack-nova | 11:12 | |
| andreykurilin | sdague: hi! Have a minute? Can you check one small patch to novaclient which fixes discoverign versionsand unblocks gates? (It is already have one +2) | 11:18 |
| sdague | sure, url? | 11:19 |
| andreykurilin | https://review.openstack.org/#/c/479818/ | 11:19 |
| *** edmondsw has joined #openstack-nova | 11:25 | |
| *** edmondsw has quit IRC | 11:25 | |
| *** edmondsw has joined #openstack-nova | 11:25 | |
| *** jwcroppe_ has quit IRC | 11:28 | |
| *** jwcroppe has joined #openstack-nova | 11:29 | |
| sean-k-mooney | moshele: hi moshele i just responded in the review but yes using something like openstack port create --binding-profile | 11:35 |
| sean-k-mooney | '{"capabilities": ["hw-tc-offload"]}' --vnic_type direct --network private port1. | 11:35 |
| sean-k-mooney | works for me | 11:35 |
| moshele | sean-k-mooney: you want me to add a TODO comment in the code for now? | 11:37 |
| *** trinaths has left #openstack-nova | 11:40 | |
| sean-k-mooney | moshele: https://review.openstack.org/#/c/398265/ should probably be rebased ontop of https://review.openstack.org/#/c/466051 | 11:41 |
| sean-k-mooney | moshele: if we open a todo comment we should also log a bug to track its completeion | 11:42 |
| sean-k-mooney | moshele: does libvirt/ethtool currently expose the hw-tc-offload or switchdev feature flag | 11:44 |
| *** danpawlik has quit IRC | 11:48 | |
| *** thorst has joined #openstack-nova | 11:53 | |
| *** danpawlik has joined #openstack-nova | 11:54 | |
| *** dave-mccowan has joined #openstack-nova | 11:55 | |
| moshele | sean-k-mooney: ok bye the way we have CI ready http://52.169.200.208/13/480213/3/check-os-vif/OVS_HW_offload/08d10ee/ | 11:57 |
| moshele | s/bye/by | 11:57 |
| *** udesale has joined #openstack-nova | 11:58 | |
| sean-k-mooney | moshele: oh cool and you started with the 2 most value test sets +1 | 11:58 |
| *** litao__ has quit IRC | 11:58 | |
| sean-k-mooney | moshele: will you be adding a full temest compute or networking run or leaving it at the scenario tests? | 11:59 |
| *** cdent has quit IRC | 11:59 | |
| moshele | sean-k-mooney: I think the scenario covers all we need for now | 12:00 |
| sean-k-mooney | moshele: yep they should we only ran the other tests as nova wanted that before giveing voteing rights | 12:01 |
| bhagyashris | cdent: Hi, Thanks for new release of gabbi. Actually I am just trying to chcek response object with sample.json (expected response) for the placement functional gabbi test "confirm the correct post" but i am getting the error as "Unable to match $ as <@sample.json, got" after running the test. | 12:03 |
| *** smatzek has joined #openstack-nova | 12:03 | |
| *** trungnv has quit IRC | 12:04 | |
| *** hieulq has quit IRC | 12:04 | |
| bhagyashris | cdent: I am checking like: response_json_paths: | 12:04 |
| bhagyashris | $: <@sample.json | 12:04 |
| *** ratailor has quit IRC | 12:04 | |
| *** lucas-hungry is now known as lucasagomes | 12:08 | |
| *** mkucia has quit IRC | 12:09 | |
| *** brad[] has quit IRC | 12:16 | |
| *** fragatina has joined #openstack-nova | 12:17 | |
| *** brad[] has joined #openstack-nova | 12:18 | |
| *** krtaylor_ has joined #openstack-nova | 12:19 | |
| *** trungnv has joined #openstack-nova | 12:19 | |
| *** hieulq has joined #openstack-nova | 12:19 | |
| *** krtaylor has quit IRC | 12:20 | |
| *** fragatina has quit IRC | 12:22 | |
| *** liverpooler has joined #openstack-nova | 12:23 | |
| *** mkucia has joined #openstack-nova | 12:24 | |
| *** xiefp has quit IRC | 12:25 | |
| *** baoli has joined #openstack-nova | 12:25 | |
| *** alexchadin has joined #openstack-nova | 12:25 | |
| *** baoli has quit IRC | 12:27 | |
| *** fried_rice is now known as efried | 12:27 | |
| *** acormier has joined #openstack-nova | 12:27 | |
| *** baoli has joined #openstack-nova | 12:28 | |
| *** acormier has quit IRC | 12:28 | |
| *** acormier has joined #openstack-nova | 12:28 | |
| *** ekuris has quit IRC | 12:29 | |
| *** mdrabe has joined #openstack-nova | 12:29 | |
| *** vvargaszte has quit IRC | 12:29 | |
| *** vvargaszte has joined #openstack-nova | 12:30 | |
| *** zhurong has quit IRC | 12:36 | |
| *** sambetts has quit IRC | 12:38 | |
| *** jmlowe has quit IRC | 12:40 | |
| *** slaweq has quit IRC | 12:41 | |
| *** slaweq has joined #openstack-nova | 12:42 | |
| *** pchavva has joined #openstack-nova | 12:42 | |
| *** sambetts has joined #openstack-nova | 12:42 | |
| *** ekuris has joined #openstack-nova | 12:43 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send soft_delete from context manager https://review.openstack.org/476459 | 12:44 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: use context mgr in instance.delete https://review.openstack.org/443764 | 12:44 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297 | 12:44 |
| *** lyan has joined #openstack-nova | 12:44 | |
| *** vancsics has quit IRC | 12:45 | |
| *** jaypipes has joined #openstack-nova | 12:46 | |
| *** acormier has quit IRC | 12:47 | |
| *** Guest9640 is now known as beagles | 12:47 | |
| *** cdent has joined #openstack-nova | 12:49 | |
| *** sridharg has joined #openstack-nova | 12:51 | |
| bhagyashris | cdent: Hi, Thanks for new release of gabbi. Actually I am just trying to chcek response object with sample.json (expected response) for the placement functional gabbi test "confirm the correct post" but i am getting the error as "Unable to match $ as <@sample.json, got" after running the test. | 12:57 |
| bhagyashris | cdent: I am checking like: response_json_paths: | 12:58 |
| bhagyashris | <bhagyashris> $: <@sample.json | 12:58 |
| cdent | the file sample.json needs to be in the same directory as the yaml file, and it of course needs to have the correct content in it | 12:59 |
| cdent | can you paste your yaml file and sample.json somewhere? | 12:59 |
| *** mdnadeem has quit IRC | 12:59 | |
| bhagyashris | yamal | 12:59 |
| bhagyashris | - name: confirm the correct post | 12:59 |
| bhagyashris | GET: /resource_providers/$ENVIRON['RP_UUID'] | 12:59 |
| bhagyashris | request_headers: | 12:59 |
| bhagyashris | content-type: application/json | 12:59 |
| bhagyashris | response_json_paths: | 12:59 |
| bhagyashris | $.: <@sample.json | 12:59 |
| *** baoli has quit IRC | 13:00 | |
| bhagyashris | cdent: sample.json http://paste.openstack.org/show/614909/ | 13:01 |
| *** hshiina has quit IRC | 13:02 | |
| cdent | bhagyashris you’ll want to change sample.json so that name and uuid are “$ENVIRON[‘RP_UUID’]” and $ENVIRON[‘RP_NAME’] and also use the environ in the url paths too? | 13:03 |
| *** catintheroof has joined #openstack-nova | 13:04 | |
| cdent | bhagyashris: but also, based on the error message it looks like you don’t have the latest gabbi installed | 13:04 |
| bhagyashris | cdent: I have pulled new gabbi test and install on my machine | 13:05 |
| *** masber has quit IRC | 13:06 | |
| cdent | bhagyashris: are you running the tests with tox? if so, it will be using the gabbi in the virtualenv | 13:06 |
| bhagyashris | cdent: and regarding the name and uuid temporarily I have hard coded the name and uuid in the fixtures.py | 13:07 |
| cdent | ah, okay | 13:07 |
| bhagyashris | cdent: yeah like tox -e functional | 13:08 |
| *** gbarros has joined #openstack-nova | 13:08 | |
| *** lbragstad has joined #openstack-nova | 13:08 | |
| cdent | then in that case you need to install the new gabbi with '.tox/functional/bin/pip install -U gabbi’ | 13:08 |
| *** esberglu has joined #openstack-nova | 13:09 | |
| *** kristian__ has joined #openstack-nova | 13:09 | |
| bhagyashris | cdent: ok. trying | 13:10 |
| bhagyashris | cdent: got error: raise AssertionError('unable to extract JSON from test results') | 13:11 |
| bhagyashris | AssertionError: unable to extract JSON from test results | 13:11 |
| cdent | bhagyashris: instead of $. you $ | 13:13 |
| bhagyashris | cdent: working :) | 13:14 |
| bhagyashris | cdent: yeah | 13:14 |
| cdent | :) | 13:14 |
| bhagyashris | cdent: I just one question How should I access name and uuid "$ENVIRON[‘RP_UUID’]" and $ENVIRON[‘RP_NAME’] in sample.json | 13:15 |
| *** eharney has joined #openstack-nova | 13:15 | |
| cdent | bhagyashris: I’ll change your paste to show you, one sec | 13:15 |
| *** acormier has joined #openstack-nova | 13:15 | |
| bhagyashris | cdent: ok | 13:15 |
| *** trungnv has quit IRC | 13:16 | |
| *** hieulq has quit IRC | 13:16 | |
| *** udesale has quit IRC | 13:17 | |
| *** gbarros has quit IRC | 13:18 | |
| cdent | bhagyashris: http://paste.openstack.org/show/614916/ | 13:20 |
| *** mmehan has joined #openstack-nova | 13:20 | |
| bhagyashris | cdent: Thanks a lot. :) | 13:22 |
| cdent | bhagyashris: keep me posted on how it’s going, will be interesting to see how it works out | 13:23 |
| *** stvnoyes has joined #openstack-nova | 13:26 | |
| bhagyashris | cdent: ok. and one more thing I guess keeping the sample.json in other folder will cause problem right so is it ok that response.json in same folder? | 13:28 |
| cdent | the way the file reading works is that it needs to be in the same directory as the yaml file or any directory within that directory | 13:29 |
| *** sdague has quit IRC | 13:30 | |
| *** gcb has joined #openstack-nova | 13:31 | |
| bhagyashris | cdent: ok. Thanks for info. | 13:31 |
| *** gbarros has joined #openstack-nova | 13:31 | |
| *** belmorei_ has joined #openstack-nova | 13:32 | |
| *** belmoreira has quit IRC | 13:32 | |
| *** sdague has joined #openstack-nova | 13:33 | |
| *** felipemonteiro has joined #openstack-nova | 13:34 | |
| *** zzzeek_ has quit IRC | 13:34 | |
| *** xyang1 has joined #openstack-nova | 13:34 | |
| *** felipemonteiro_ has joined #openstack-nova | 13:36 | |
| andreykurilin | sdague: thanks | 13:36 |
| *** hieulq has joined #openstack-nova | 13:36 | |
| *** trungnv has joined #openstack-nova | 13:36 | |
| *** vladikr has joined #openstack-nova | 13:37 | |
| sdague | andreykurilin: no prob, thanks for building the patch | 13:38 |
| *** yushb has joined #openstack-nova | 13:38 | |
| *** ekuris has quit IRC | 13:39 | |
| *** felipemonteiro has quit IRC | 13:39 | |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-evacuate notification https://review.openstack.org/480119 | 13:42 |
| *** sridharg has quit IRC | 13:42 | |
| *** roeyc has quit IRC | 13:44 | |
| *** sridharg has joined #openstack-nova | 13:44 | |
| *** sridharg has quit IRC | 13:44 | |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-evacuate notification https://review.openstack.org/482148 | 13:44 |
| *** acormier has quit IRC | 13:45 | |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-live_migration_post notification https://review.openstack.org/480119 | 13:45 |
| *** awaugama has joined #openstack-nova | 13:46 | |
| *** sridharg has joined #openstack-nova | 13:46 | |
| *** sridharg has quit IRC | 13:47 | |
| *** sridharg has joined #openstack-nova | 13:49 | |
| *** sridharg has quit IRC | 13:49 | |
| *** sridharg has joined #openstack-nova | 13:50 | |
| *** sridharg has quit IRC | 13:50 | |
| *** burgerk has joined #openstack-nova | 13:51 | |
| *** sridharg has joined #openstack-nova | 13:51 | |
| *** ekuris has joined #openstack-nova | 13:51 | |
| *** sridharg has quit IRC | 13:52 | |
| *** sridharg has joined #openstack-nova | 13:54 | |
| edleafe | Scheduler subteam meeting in 5 minutes in #openstack-meeting-alt | 13:55 |
| *** jwcroppe has quit IRC | 13:59 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Stop using mox stubs in cast_as_call.py https://review.openstack.org/477730 | 14:01 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Stop using mox stubs in test_remote_consoles.py https://review.openstack.org/478423 | 14:01 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Verify parameters in os-migrations.inc https://review.openstack.org/474879 | 14:01 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Use PCIAddressField in oslo.versionedobjects https://review.openstack.org/466201 | 14:01 |
| *** esberglu has quit IRC | 14:02 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters in server-security-groups https://review.openstack.org/476434 | 14:02 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add missing parameters in limits.inc https://review.openstack.org/475640 | 14:02 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix a missing classifier https://review.openstack.org/478733 | 14:02 |
| *** superdan is now known as dansmith | 14:02 | |
| *** esberglu has joined #openstack-nova | 14:02 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add X-Openstack-Request-Id description https://review.openstack.org/474847 | 14:03 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Stop using mox stubs in test_console_auth_tokens.py https://review.openstack.org/477374 | 14:03 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add 'updated_at' field to InstancePayload in notifications https://review.openstack.org/475276 | 14:03 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Use URIOpt https://review.openstack.org/479215 | 14:03 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Amend the "list-show-all-server-migration-types" spec https://review.openstack.org/460847 | 14:04 |
| *** baoli has joined #openstack-nova | 14:05 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix quota class set APIs https://review.openstack.org/467999 | 14:05 |
| *** slaweq has quit IRC | 14:07 | |
| *** gyee has joined #openstack-nova | 14:07 | |
| *** gouthamr has joined #openstack-nova | 14:10 | |
| *** hongbin has joined #openstack-nova | 14:14 | |
| *** jdillaman has joined #openstack-nova | 14:15 | |
| *** mriedem has joined #openstack-nova | 14:15 | |
| *** jwcroppe has joined #openstack-nova | 14:15 | |
| mriedem | o/ | 14:16 |
| *** fragatina has joined #openstack-nova | 14:19 | |
| openstackgerrit | sean mooney proposed openstack/os-vif master: set mtu on all code paths https://review.openstack.org/469951 | 14:21 |
| *** beekneemech is now known as bnemec | 14:22 | |
| *** fragatina has quit IRC | 14:24 | |
| *** fragatina has joined #openstack-nova | 14:24 | |
| *** alexchadin has quit IRC | 14:25 | |
| *** tjones has joined #openstack-nova | 14:27 | |
| *** zzzeek_ has joined #openstack-nova | 14:28 | |
| *** psachin has quit IRC | 14:28 | |
| *** zhurong has joined #openstack-nova | 14:30 | |
| *** armax has joined #openstack-nova | 14:31 | |
| dansmith | jaypipes: can you legitimify my comment here? https://review.openstack.org/#/c/476631/19 | 14:31 |
| mriedem | dansmith: we're talking about it in the scheduler meeting | 14:32 |
| mriedem | #openstack-meeting-alt | 14:32 |
| dansmith | oh | 14:32 |
| *** dixiaoli has joined #openstack-nova | 14:33 | |
| *** dixiaoli has quit IRC | 14:33 | |
| *** fragatina has quit IRC | 14:36 | |
| *** chenying_ has quit IRC | 14:40 | |
| *** belmorei_ has quit IRC | 14:41 | |
| *** Apoorva has joined #openstack-nova | 14:43 | |
| *** annegentle has joined #openstack-nova | 14:45 | |
| *** chyka has joined #openstack-nova | 14:48 | |
| *** priteau has joined #openstack-nova | 14:48 | |
| *** chyka has quit IRC | 14:48 | |
| *** chyka has joined #openstack-nova | 14:49 | |
| *** hieulq has quit IRC | 14:51 | |
| *** trungnv has quit IRC | 14:51 | |
| *** ratailor has joined #openstack-nova | 14:51 | |
| *** roeyc has joined #openstack-nova | 14:53 | |
| openstackgerrit | Merged openstack/python-novaclient master: Fix cropping the endpoint url https://review.openstack.org/479818 | 14:54 |
| *** zhurong has quit IRC | 14:56 | |
| *** rcernin has quit IRC | 14:57 | |
| *** udesale has joined #openstack-nova | 14:57 | |
| *** armax has quit IRC | 14:58 | |
| artom | mriedem, could you remove the procedural -2 on https://review.openstack.org/#/c/467321/? | 14:59 |
| *** yamahata has quit IRC | 14:59 | |
| *** yamahata has joined #openstack-nova | 15:00 | |
| mriedem | done, but i haven't really looked at everything else going on there | 15:00 |
| * cdent goes home | 15:00 | |
| *** armax has joined #openstack-nova | 15:00 | |
| artom | mriedem, cheers! | 15:00 |
| artom | mriedem, bah, that can be done in its own time | 15:00 |
| *** cdent has quit IRC | 15:00 | |
| artom | Getting the -2 off is a good first step ;) | 15:00 |
| *** udesale has quit IRC | 15:02 | |
| mriedem | i was a -W but who's counting | 15:03 |
| mriedem | artom: not sure if you saw before going on PTO, but i opened a docs bug for putting something in the feature support matrix for device tags, | 15:03 |
| mriedem | is that something you can work on? | 15:03 |
| artom | mriedem, I saw that, yeah | 15:04 |
| artom | mriedem, didn't someone already submit a patch? | 15:04 |
| mriedem | ah yes https://review.openstack.org/#/c/481478/ | 15:05 |
| artom | mriedem, err, twice, even | 15:05 |
| artom | https://review.openstack.org/#/c/481479/ as well | 15:05 |
| *** ircuser-1 has joined #openstack-nova | 15:05 | |
| artom | Or are those different? | 15:05 |
| mriedem | once is for the server create, the other is attach | 15:05 |
| artom | Aha! I'll put those on my to-review list, in a call right this instant | 15:06 |
| *** coreywright has quit IRC | 15:07 | |
| openstackgerrit | KaiLi proposed openstack/nova master: Add "default" rule when generate policy.yaml.sample https://review.openstack.org/481443 | 15:08 |
| *** trungnv has joined #openstack-nova | 15:08 | |
| *** hieulq has joined #openstack-nova | 15:09 | |
| *** roeyc has quit IRC | 15:11 | |
| *** moshele has quit IRC | 15:13 | |
| mkucia | Hi, I have question for qeens as I believe this is beyond pike. Is there some work planned that is related to extracting ResourceProvider handling from nova's driver code? For example when libvirt xml is being assembled there are multiple interations over different kinds of resources. In the RPs perspective it seems for me that those resources can be generalized. | 15:14 |
| openstackgerrit | sean mooney proposed openstack/os-vif master: add host port profile info class https://review.openstack.org/441590 | 15:15 |
| *** Oku_OS is now known as Oku_OS-away | 15:16 | |
| mriedem | mkucia: there are no plans for that | 15:18 |
| mriedem | mkucia: there are only plans to extract the placement service/api/data model into a separate project | 15:18 |
| mkucia | Ok. Thanks! | 15:19 |
| *** coreywright has joined #openstack-nova | 15:20 | |
| *** liusheng has quit IRC | 15:21 | |
| *** liusheng has joined #openstack-nova | 15:22 | |
| *** gbarros has quit IRC | 15:25 | |
| *** gbarros has joined #openstack-nova | 15:26 | |
| *** fragatina has joined #openstack-nova | 15:26 | |
| openstackgerrit | sean mooney proposed openstack/os-vif master: set mtu on all code paths https://review.openstack.org/469951 | 15:27 |
| bauzas | dumb question, but can someone point me if/where we have privsep commands in our codebase? | 15:27 |
| *** trungnv has quit IRC | 15:28 | |
| *** hieulq has quit IRC | 15:28 | |
| mriedem | we don't | 15:29 |
| bauzas | nevermind, got the fully monty https://blueprints.launchpad.net/nova/+spec/privsep | 15:29 |
| mriedem | os-brick and os-vif use privsep | 15:29 |
| bauzas | full* | 15:29 |
| bauzas | mriedem: that's what I discovered | 15:29 |
| mriedem | you can also check out https://review.openstack.org/#/c/459166/ | 15:30 |
| bauzas | the whiteboard is pretty explicit | 15:30 |
| bauzas | yeah | 15:30 |
| bauzas | finding https://blueprints.launchpad.net/nova/+spec/privsep was helpful | 15:30 |
| *** chenying_ has joined #openstack-nova | 15:40 | |
| *** hieulq has joined #openstack-nova | 15:43 | |
| *** trungnv has joined #openstack-nova | 15:43 | |
| mriedem | sdague: are you back around this week? | 15:45 |
| openstackgerrit | sean mooney proposed openstack/os-vif master: set mtu on all code paths https://review.openstack.org/469951 | 15:46 |
| *** damien_r has quit IRC | 15:47 | |
| mgagne | mriedem: sorry about that (overriding your changes) | 15:49 |
| mriedem | mgagne: no problem, got everything updated last night, should be ready for core review | 15:50 |
| *** roeyc has joined #openstack-nova | 15:50 | |
| mriedem | oomichi: can you take a look at this new microversion change when you get a chance? https://review.openstack.org/#/c/454322/ | 15:51 |
| *** gongysh has joined #openstack-nova | 15:51 | |
| mriedem | i've gone through it and would +2 but i added the os-instance-actions changes | 15:51 |
| *** Apoorva has quit IRC | 15:55 | |
| *** baoli has quit IRC | 15:55 | |
| *** Apoorva has joined #openstack-nova | 15:56 | |
| *** baoli has joined #openstack-nova | 15:56 | |
| *** kristian__ has quit IRC | 15:57 | |
| *** kristian__ has joined #openstack-nova | 15:58 | |
| *** yushb has quit IRC | 16:00 | |
| *** Apoorva has quit IRC | 16:01 | |
| openstackgerrit | dane-fichter proposed openstack/nova master: Mask password when logging connection_info during swap volume https://review.openstack.org/466533 | 16:04 |
| *** Apoorva has joined #openstack-nova | 16:08 | |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Raise a real Exception instead of direct Exception call https://review.openstack.org/482200 | 16:09 |
| sean-k-mooney | jaypipes: 30 second rereivew https://review.openstack.org/#/c/469951/ i move some comments that did not make sense after the refactor but otherwise identical to the version you just reviewed. sorry for the churn | 16:10 |
| *** Apoorva has quit IRC | 16:10 | |
| *** Apoorva has joined #openstack-nova | 16:10 | |
| *** cdent has joined #openstack-nova | 16:10 | |
| *** ijw has joined #openstack-nova | 16:10 | |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Raise a real Exception instead of direct Exception call https://review.openstack.org/482200 | 16:10 |
| *** Apoorva_ has joined #openstack-nova | 16:11 | |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Raise a real Exception instead of direct Exception call https://review.openstack.org/482200 | 16:11 |
| *** annegentle has quit IRC | 16:12 | |
| jaypipes | sean-k-mooney: cool +2 | 16:13 |
| *** Apoorva has quit IRC | 16:14 | |
| *** moshele has joined #openstack-nova | 16:14 | |
| *** gongysh has quit IRC | 16:15 | |
| *** ijw has quit IRC | 16:15 | |
| *** rajathagasthya has joined #openstack-nova | 16:20 | |
| *** ratailor has quit IRC | 16:21 | |
| *** fragatina has quit IRC | 16:21 | |
| *** baoli has quit IRC | 16:23 | |
| kashyap | Any Ceph people, the RBD cache is enabled by default - isn't it? | 16:24 |
| *** moshele has quit IRC | 16:25 | |
| kashyap | Yep, it is - http://docs.ceph.com/docs/kraken/rbd/rbd-config-ref/ | 16:25 |
| *** rajathag_ has joined #openstack-nova | 16:25 | |
| *** rajathagasthya has quit IRC | 16:26 | |
| sdague | mriedem: very ish... I'm traveling out to openwest tomorrow and there for the rest of the week | 16:29 |
| sdague | mriedem: if you have a few priority things for me to look at I can, but won't have a ton of bw this week | 16:29 |
| mriedem | sdague: ok, https://review.openstack.org/#/c/454322/ would be good | 16:30 |
| mriedem | there is a cinder change dependent on it, and i've pushed devstack and tempest changes to test it all out - flushed a few bugs last week, test is green now | 16:31 |
| *** sshwarts has quit IRC | 16:32 | |
| *** annegentle has joined #openstack-nova | 16:34 | |
| *** hieulq has quit IRC | 16:36 | |
| *** trungnv has quit IRC | 16:36 | |
| *** harlowja has joined #openstack-nova | 16:37 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: claim resources in scheduler https://review.openstack.org/476632 | 16:38 |
| jaypipes | mriedem, dansmith: ^^ | 16:38 |
| *** jwcroppe has quit IRC | 16:39 | |
| *** fragatina has joined #openstack-nova | 16:42 | |
| mriedem | yar matey | 16:44 |
| *** jwcroppe has joined #openstack-nova | 16:47 | |
| *** slaweq has joined #openstack-nova | 16:48 | |
| *** jwcroppe has quit IRC | 16:49 | |
| *** fragatina has quit IRC | 16:49 | |
| *** trungnv has joined #openstack-nova | 16:51 | |
| *** hieulq has joined #openstack-nova | 16:51 | |
| *** slaweq has quit IRC | 16:54 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Mark LXC as missing for swap volume support https://review.openstack.org/482216 | 16:59 |
| *** armax has quit IRC | 16:59 | |
| *** xyang1 has quit IRC | 16:59 | |
| *** hieulq has quit IRC | 16:59 | |
| *** trungnv has quit IRC | 17:00 | |
| *** tesseract has quit IRC | 17:01 | |
| *** roeyc has quit IRC | 17:04 | |
| *** acormier has joined #openstack-nova | 17:05 | |
| *** lucasagomes is now known as lucas-afk | 17:06 | |
| *** xyang1 has joined #openstack-nova | 17:09 | |
| *** acormier has quit IRC | 17:14 | |
| dansmith | jaypipes: remind me.. we create allocations but the computes won't delete them yet because they're not instance.host=$me, right? | 17:14 |
| dansmith | jaypipes: you're not very defensive about your allocation delete, and I was wondering if we could race to delete those with anything making that risky | 17:14 |
| *** acormier has joined #openstack-nova | 17:14 | |
| *** moshele has joined #openstack-nova | 17:14 | |
| *** fragatina has joined #openstack-nova | 17:15 | |
| *** fragatina has quit IRC | 17:15 | |
| jaypipes | dansmith: in answer to your first question, yes, we don't delete the allocations because of this code: https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L1055-L1061 | 17:15 |
| jaypipes | dansmith: w.r.t. not being very defensive about allocation delete, I'm not following you. I'm deleting allocations that were successfully claimed in a multi-instance launch request where one of the claims failed but others succeeded. | 17:16 |
| *** acormier has quit IRC | 17:17 | |
| *** moshele has quit IRC | 17:17 | |
| dansmith | jaypipes: right, but if one delete fails we won't continue to delete the rest (or try at least) | 17:17 |
| *** fragatina has joined #openstack-nova | 17:17 | |
| artom | What's our policy on English mistakes in things like commit messages and comments? | 17:18 |
| jaypipes | dansmith: if one delete fails, that's correct, we just ignore the failure and attempt deleting the others that were successfully allocated. | 17:18 |
| * artom always feels bad correcting folks who are non-native speakers. | 17:18 | |
| *** roeyc has joined #openstack-nova | 17:18 | |
| dansmith | jaypipes: I was asserting that you wouldn't, but it looks like delete_allocation_for_instance() won't ever raise/fail | 17:18 |
| jaypipes | artom: I always point out the mistakes | 17:18 |
| dansmith | artom: +1 for correcting | 17:18 |
| dansmith | me too | 17:18 |
| jaypipes | dansmith: right. | 17:18 |
| *** fragatina has quit IRC | 17:18 | |
| *** fragatina has joined #openstack-nova | 17:18 | |
| artom | dansmith, jaypipes, cheers! | 17:19 |
| jaypipes | artom: I speak good. | 17:19 |
| artom | jaypipes, only because you not fail english. | 17:19 |
| *** yamamoto has quit IRC | 17:19 | |
| jaypipes | heh :) | 17:19 |
| *** slaweq has joined #openstack-nova | 17:20 | |
| artom | Aww, I was hoping someone would come in with the unpossible reference :( | 17:20 |
| *** sridharg has quit IRC | 17:22 | |
| kashyap | artom: It's the way you phrase it, right? | 17:22 |
| kashyap | artom: About correcting non-native English | 17:22 |
| kashyap | If one talks like a moron, it'll come off bad | 17:23 |
| kashyap | A good community example for this is - Eric Blake from QEMU | 17:23 |
| artom | kashyap, you mean me correcting English mistakes and coming off as a pedantic twat? | 17:23 |
| kashyap | He has this amazing knack to give feedback without sounding like a ****. And he's a stickler for English :-) | 17:23 |
| kashyap | (So am I!) | 17:23 |
| kashyap | aries: Yeah | 17:23 |
| kashyap | Err, it was for artom | 17:23 |
| *** slaweq has quit IRC | 17:24 | |
| artom | kashyap, thanks, I'll try to see if I can distill his technique into usable dos and donts | 17:24 |
| cdent | dansmith: is this functional test still something you hope to make happen, or has it been superseded ? https://review.openstack.org/#/c/452006/ | 17:24 |
| *** slaweq has joined #openstack-nova | 17:24 | |
| artom | kashyap, it's still over text though - I can have the best intentions in the world, it's hard to know how your comments will be received | 17:24 |
| kashyap | artom: Meanwhile, look at this feedback I got about how I over-use "comma", a developer (I know) labelled me: "You've gone comma-wild"! | 17:25 |
| dansmith | cdent: yeah I guess | 17:26 |
| kashyap | artom: `grep` for the terms: "conversation" and "wild" here :-) https://lists.nongnu.org/archive/html/qemu-devel/2017-07/msg01455.html | 17:26 |
| artom | kashyap, I am disappoint | 17:26 |
| artom | They completely missed the "you've made me comma-tose" pun | 17:26 |
| *** lpetrut has quit IRC | 17:26 | |
| cdent | dansmith: yeah, you’re still working on it, or yeah, it’s dead, or maybe “yeah, it should exist but I’ve not got the time”? | 17:26 |
| kashyap | artom: Ah, good one :-) | 17:26 |
| dansmith | cdent: mostly the latter | 17:26 |
| dansmith | cdent: why, you want to work on it? | 17:27 |
| cdent | dansmith: cool, thanks | 17:27 |
| cdent | dansmith: well, perhaps, but the immediate issue was that it’s in my to do list as a recurring thing to review, and I’ve been defering it over increasing long periods | 17:28 |
| dansmith | okay | 17:28 |
| cdent | so thought perhaps it was dead and I can get it off my radar. i’ll keep it there for now. | 17:28 |
| cdent | I don’t have a clear understanding on what time or attention I’ve got… | 17:29 |
| *** roeyc has quit IRC | 17:29 | |
| *** slaweq has quit IRC | 17:29 | |
| *** efoley has quit IRC | 17:30 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: [WIP] Add native implementation OVSDB API https://review.openstack.org/482226 | 17:33 |
| *** Kevin_Zheng has quit IRC | 17:33 | |
| artom | kashyap, since you nibbled at my bait | 17:35 |
| artom | kashyap, https://review.openstack.org/#/c/481478/1//COMMIT_MSG | 17:35 |
| artom | There's nothing *wrong* per se, it just reads weird | 17:35 |
| artom | (Also, I think "taking the bait" is the correct expression, nibbling makes it sound, err, yeah) | 17:36 |
| artom | Surely I can't just come in and "nothing's wrong here, but it's weird, here's what I'd write:" ? | 17:37 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Read datapath_type from VIF object https://review.openstack.org/474914 | 17:38 |
| edleafe | artom: I usually acknowledge that while I understood the message, a native English speaker would phrase it differently | 17:38 |
| kashyap | artom: Need to step out for a few; bbiab. Sorry | 17:39 |
| kashyap | I'll look when I'm back | 17:39 |
| *** trungnv has joined #openstack-nova | 17:40 | |
| artom | edleafe, ah, good technique :) | 17:40 |
| artom | kashyap, sure, thanks :) | 17:40 |
| *** hieulq has joined #openstack-nova | 17:40 | |
| *** armax has joined #openstack-nova | 17:41 | |
| *** trungnv has quit IRC | 17:43 | |
| *** hieulq has quit IRC | 17:43 | |
| * artom has a realization | 17:43 | |
| artom | It comes down to reviewing something a person can't help | 17:43 |
| artom | While we can hold pretty much everyone to the same coding standard, it's unreasonable to hold everyone to the same English standard | 17:44 |
| *** rajathag_ has quit IRC | 17:44 | |
| *** ralonsoh has quit IRC | 17:44 | |
| *** MVenesio has joined #openstack-nova | 17:47 | |
| *** mkucia has quit IRC | 17:48 | |
| mriedem | https://www.youtube.com/watch?v=KzyCi1BFATA | 17:48 |
| *** yamahata has quit IRC | 17:49 | |
| *** acormier has joined #openstack-nova | 17:50 | |
| *** slaweq has joined #openstack-nova | 17:51 | |
| *** acormier has joined #openstack-nova | 17:51 | |
| *** kristia__ has joined #openstack-nova | 17:52 | |
| artom | mriedem, better late than never ;) | 17:54 |
| mriedem | gibi: was it intentional to not include the bdm.device_name in the versioned notifications? | 17:56 |
| *** annegentle has quit IRC | 17:56 | |
| *** kristian__ has quit IRC | 17:56 | |
| *** rajathagasthya has joined #openstack-nova | 17:57 | |
| *** jmlowe has joined #openstack-nova | 17:58 | |
| *** trungnv has joined #openstack-nova | 17:58 | |
| *** hieulq has joined #openstack-nova | 17:58 | |
| *** annegentle has joined #openstack-nova | 17:59 | |
| *** dtantsur is now known as dtantsur|afk | 18:00 | |
| *** jamesdenton has quit IRC | 18:01 | |
| *** dmellado has quit IRC | 18:02 | |
| *** dmellado has joined #openstack-nova | 18:02 | |
| *** jamesdenton has joined #openstack-nova | 18:03 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: claim resources in scheduler https://review.openstack.org/476632 | 18:03 |
| jaypipes | dansmith: corrected. ^ | 18:03 |
| *** kristia__ has quit IRC | 18:03 | |
| * dansmith re-inks his red pen | 18:04 | |
| *** sambetts is now known as sambetts|afk | 18:05 | |
| jaypipes | dansmith: isn't it a quill, you old fart? | 18:07 |
| *** moshele has joined #openstack-nova | 18:07 | |
| dansmith | jaypipes: of course | 18:08 |
| mriedem | if a bdm is created with destination_type='local', does it get a volume_id? if that tree falls in the bdm woods, does it make a sound? | 18:08 |
| dansmith | if a bear drops a bdm in the woods, does it smell? | 18:09 |
| mriedem | i'm sure it does | 18:09 |
| mriedem | <sniff> <sniff> "smells like....bdm" | 18:09 |
| *** kylek3h has joined #openstack-nova | 18:11 | |
| mriedem | so you can have an image/local bdm, which is the boot device on local storage for the image the server is created with? | 18:12 |
| mriedem | which is not the same as boot from volume, since there is no volume? | 18:13 |
| mriedem | and blank/local means swap or ephemeral disk, which is local, which is not a volume | 18:13 |
| mriedem | yeah?! | 18:13 |
| *** Swami has joined #openstack-nova | 18:14 | |
| *** kristian__ has joined #openstack-nova | 18:15 | |
| *** kristian__ has quit IRC | 18:15 | |
| *** lpetrut has joined #openstack-nova | 18:15 | |
| *** yamamoto has joined #openstack-nova | 18:19 | |
| *** abalutoiu_ has quit IRC | 18:20 | |
| *** READ10 has joined #openstack-nova | 18:20 | |
| *** Sukhdev has joined #openstack-nova | 18:24 | |
| *** yamahata has joined #openstack-nova | 18:24 | |
| *** yamamoto has quit IRC | 18:27 | |
| *** anish_ is now known as anish | 18:30 | |
| *** baoli has joined #openstack-nova | 18:31 | |
| *** moshele has quit IRC | 18:31 | |
| *** gjayavelu has joined #openstack-nova | 18:33 | |
| *** lpetrut has quit IRC | 18:35 | |
| *** lpetrut has joined #openstack-nova | 18:42 | |
| *** ckopper has joined #openstack-nova | 18:57 | |
| *** ckopper has quit IRC | 19:00 | |
| mriedem | ha, the instance.update notification doesn't actually included the updated_at field | 19:02 |
| *** annegentle has quit IRC | 19:03 | |
| *** liverpooler has quit IRC | 19:06 | |
| dansmith | jaypipes: I would +2 your top patch but mriedem usually sees that as an opportunity to screw up my stats, and I don't want to inflict that on you, so I'll circle back when he's had a look :) | 19:07 |
| mriedem | gdi | 19:07 |
| mriedem | you know i won't review until you +2 | 19:07 |
| dansmith | well, then we have a problem here | 19:07 |
| dansmith | I propose a dance-off | 19:08 |
| mriedem | dansmith: how about i review that claims shenanigans and you see if the updates to https://review.openstack.org/#/c/416521/ make you happy | 19:08 |
| dansmith | ugh | 19:08 |
| mriedem | ha | 19:08 |
| mriedem | or, see if my hypervisors/services uuid api changes make you happy | 19:08 |
| mriedem | or any of the cinder v3 attachments patches | 19:09 |
| mriedem | there are plenty to choose from | 19:09 |
| ildikov | the Cinder v3 ones are very joyful :) | 19:09 |
| ildikov | and if that statement would ruin my chances to get them reviewed please just ignore it... | 19:10 |
| smcginnis | :) | 19:11 |
| *** liverpooler has joined #openstack-nova | 19:11 | |
| ildikov | although this one is really close: https://review.openstack.org/#/c/456971/ | 19:11 |
| * ildikov remains quiet now | 19:12 | |
| dansmith | mriedem: I thought I was all +2 on your services ones | 19:13 |
| * dansmith checks | 19:13 | |
| mriedem | you said "when the time comes" | 19:13 |
| *** jbernard has quit IRC | 19:13 | |
| *** jbernard has joined #openstack-nova | 19:13 | |
| *** jmlowe has quit IRC | 19:14 | |
| dansmith | oh I see I had a -1 recently on the top one | 19:14 |
| mriedem | yeah i addressed your issues i think | 19:14 |
| mriedem | and alex's | 19:14 |
| dansmith | well, not the bit you said you didn't understand | 19:17 |
| dansmith | does this help? https://review.openstack.org/#/c/471899/24/nova/api/openstack/compute/hypervisors.py | 19:17 |
| mriedem | no | 19:18 |
| mriedem | you want me to rename the parameter? | 19:18 |
| mriedem | see https://review.openstack.org/#/c/471899/24/nova/api/openstack/compute/hypervisors.py@381 | 19:19 |
| dansmith | I know, | 19:23 |
| *** rajathagasthya has quit IRC | 19:23 | |
| dansmith | I realize this is existing, I'm just saying it's super confusing/wrong to pass instances to a thing, calling them servers | 19:23 |
| *** acormier has quit IRC | 19:24 | |
| dansmith | just thought maybe we should clean that up since you're touching this code | 19:24 |
| mriedem | so you want me to rename the formal parameter | 19:24 |
| mriedem | i need you to tell me what you want me to do | 19:24 |
| mriedem | :) | 19:24 |
| mriedem | jaypipes: https://review.openstack.org/#/c/476632/ | 19:24 |
| *** rajathagasthya has joined #openstack-nova | 19:25 | |
| mriedem | i don't personally see how instances and servers is confusing, since instances feed the 'servers' resource in the API | 19:25 |
| mriedem | are you thinking servers == hosts? | 19:25 |
| dansmith | oh, I see, heh | 19:26 |
| dansmith | yes | 19:26 |
| mriedem | :/ | 19:27 |
| dansmith | and also servers=services | 19:27 |
| dansmith | more the latter I think | 19:27 |
| dansmith | because I was already in the services frame of mind | 19:27 |
| edleafe | ugh - I remember this naming confusion back from the early days with RAX | 19:27 |
| mriedem | oh yeah | 19:27 |
| mriedem | https://www.youtube.com/watch?v=D4O94hw3qt0 | 19:27 |
| mriedem | that's the kind of mind i'm in | 19:27 |
| edleafe | since they called their things "Cloud Servers", they insisted on calling an instance a "server" | 19:27 |
| dansmith | servers is a stupid name for it, | 19:28 |
| edleafe | dansmith: just one of the many arguments I lost in those days | 19:28 |
| dansmith | but the ship has sailed, I just don't spend enough time in the api to remember all the time and the patch below is about _services_ | 19:28 |
| mriedem | dansmith: if you're also good with the os-services change, can you +2 that so we can queue things up? | 19:31 |
| mriedem | if i can get alex_xu to also be +2 on the series then we'll push it through | 19:31 |
| dansmith | done | 19:31 |
| mriedem | yay | 19:32 |
| *** kristian__ has joined #openstack-nova | 19:34 | |
| mriedem | now i can't get that song out of my head | 19:36 |
| *** READ10 has quit IRC | 19:36 | |
| dansmith | mriedem: tbh I feel like I have a bit of review fatigue on this quota stuff | 19:37 |
| *** prateek has quit IRC | 19:38 | |
| *** trungnv has quit IRC | 19:38 | |
| *** hieulq has quit IRC | 19:38 | |
| dansmith | not sure I've seen an actual review from you on this one | 19:38 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: claim resources in scheduler https://review.openstack.org/476632 | 19:38 |
| mriedem | you haven't | 19:38 |
| jaypipes | mriedem: fixed :( ^ | 19:38 |
| dansmith | I guess we're not likely to see one from sdague in the near future either, eh? | 19:38 |
| *** kristian__ has quit IRC | 19:39 | |
| mriedem | probably not. i plan on reviewing that quotas one. | 19:39 |
| jaypipes | mriedem: I will put a patch up renaming delete_allocation_for_instance() to delete_allocations_for_consumer() to remedy my need for grammatical correctness. | 19:39 |
| *** eharney has quit IRC | 19:39 | |
| *** rajathagasthya has quit IRC | 19:41 | |
| *** annegentle has joined #openstack-nova | 19:42 | |
| *** moshele has joined #openstack-nova | 19:43 | |
| *** moshele has quit IRC | 19:43 | |
| sdague | dansmith: yeh, not this week | 19:43 |
| *** rajathagasthya has joined #openstack-nova | 19:43 | |
| sdague | I can get my head back into it next week, but I don't know which dates we're racing against now | 19:44 |
| dansmith | yeah, need it before next week I think | 19:44 |
| *** kristian__ has joined #openstack-nova | 19:45 | |
| *** kristian__ has quit IRC | 19:52 | |
| mriedem | sdague: feature freeze is july 27 | 19:52 |
| mriedem | so we're less than 3 weeks from FF | 19:52 |
| mriedem | jaypipes: a couple of comments in https://review.openstack.org/#/c/476632/ | 19:53 |
| mriedem | jaypipes: dansmith: in a nutshell on that one, | 19:53 |
| mriedem | i think it means we potentially have problems for bursts of instances in the same request with that change until we start doing retries in the scheduler | 19:53 |
| *** edmondsw_ has joined #openstack-nova | 19:53 | |
| mriedem | and/or we start randomizing which allocation request match we pick for a given compute node | 19:54 |
| *** eharney has joined #openstack-nova | 19:54 | |
| jaypipes | mriedem: do you want me to do retries in this patch? | 19:55 |
| jaypipes | mriedem: I was working on that in a separate followup. | 19:55 |
| jaypipes | mriedem: but if you're concerned about this, I suppose I could put it all in one patch. | 19:56 |
| jaypipes | mriedem: was trying to keep the LOCC low. | 19:56 |
| mriedem | a single change makes it harder to review for sure | 19:56 |
| mriedem | yeah | 19:56 |
| *** edmondsw has quit IRC | 19:56 | |
| mriedem | i guess we assume no one is doing CD | 19:56 |
| *** edmondsw has joined #openstack-nova | 19:57 | |
| mriedem | jaypipes: how about line up the retries change on top, | 19:57 |
| jaypipes | mriedem: answered you. | 19:57 |
| mriedem | and then we'll get them in order | 19:57 |
| mriedem | so there isn't like a week between changes | 19:57 |
| jaypipes | mriedem: sure thing. | 19:57 |
| jaypipes | mriedem: gimme about an hour. | 19:57 |
| *** edmondsw_ has quit IRC | 19:59 | |
| cdent | is this canon: “i guess we assume no one is doing CD” ? If so, life just got a lot easier. | 20:00 |
| mriedem | no, it's not | 20:00 |
| mriedem | dims is supposed to make huawei public cloud doing CD in the next couple of weeks :) | 20:01 |
| cdent | do you take bribes? | 20:01 |
| *** kristian__ has joined #openstack-nova | 20:01 | |
| *** gbarros has quit IRC | 20:01 | |
| mriedem | https://docs.openstack.org/nova/latest/project_scope.html#upgrade-expectations | 20:02 |
| mriedem | "In addition, we also support users upgrading from the master branch, technically, between any two commits within the same release cycle." | 20:03 |
| cdent | oh yeah, I know, I was just thinking maybe the world had collapsed | 20:03 |
| *** liverpooler has quit IRC | 20:04 | |
| *** pchavva has quit IRC | 20:06 | |
| *** lpetrut has quit IRC | 20:17 | |
| jaypipes | mriedem: so... problem. | 20:30 |
| jaypipes | mriedem: I need to do the alternatives stuff before the retries. | 20:30 |
| jaypipes | mriedem: because the retries need to loop over the alternatives. | 20:30 |
| mriedem | right, b/c we need the alternatives to do retries | 20:30 |
| mriedem | yup | 20:30 |
| jaypipes | mriedem: I don't necessarily need to *return* (over RPC) the alternative yet though. | 20:30 |
| *** acormier has joined #openstack-nova | 20:33 | |
| mriedem | correctamundo | 20:34 |
| * jaypipes gets scalpel | 20:34 | |
| mriedem | so it's like 3 changes on top of this, maybe 2 | 20:34 |
| mriedem | 1. determine alternatives, | 20:34 |
| mriedem | 2. retry on alternatives in scheduler | 20:34 |
| *** nicolasbock has quit IRC | 20:34 | |
| mriedem | 3. pass alternatives to conductor | 20:34 |
| *** awaugama has quit IRC | 20:34 | |
| mriedem | and then 4,5,6,7,8 do things with those in the cell | 20:35 |
| jaypipes | yeah | 20:35 |
| *** acormier has quit IRC | 20:37 | |
| *** gbarros has joined #openstack-nova | 20:37 | |
| jaypipes | mriedem, dansmith: did we ever decide how many alternatives we wanted to return? | 20:38 |
| jaypipes | mriedem, dansmith: limit to 10 per instance UUID maybe? | 20:39 |
| dansmith | jaypipes: CONF.num_retries | 20:39 |
| jaypipes | ah | 20:39 |
| dansmith | times num_instances | 20:39 |
| jaypipes | gotcha. | 20:39 |
| *** lpetrut has joined #openstack-nova | 20:40 | |
| edleafe | jaypipes: this was the consensus a couple of months ago: https://review.openstack.org/#/c/465767/ | 20:41 |
| edleafe | default to CONF.num_retries | 20:41 |
| edleafe | also, won't the order need to be different? | 20:42 |
| *** thorst has quit IRC | 20:42 | |
| edleafe | IOW, claim fails on "best" host, so you try claiming next one. Once you succeed, you select N alternates from the next N hosts in the weighed list that are in the same cell as the successful claimed host | 20:43 |
| jaypipes | edleafe: the consensus was to create a new CONF option build_max_attempts? | 20:43 |
| dansmith | that sounds like a terrible idea | 20:44 |
| dansmith | num_retries and build_max_attempts is highly confusing | 20:44 |
| *** thorst has joined #openstack-nova | 20:45 | |
| dansmith | we have the former, I see no reason for a billionth conf var | 20:45 |
| edleafe | jaypipes: sorry, it was supposed to be 'max_attempts' | 20:45 |
| *** lpetrut has quit IRC | 20:47 | |
| mriedem | plus host_subset_size | 20:47 |
| openstackgerrit | Merged openstack/nova master: placement: scheduler uses allocation candidates https://review.openstack.org/476631 | 20:48 |
| mriedem | we're also going to have to build in the filtering for the alternatives by cell | 20:48 |
| mriedem | i'm guessing we're going to want to cache that as well | 20:48 |
| edleafe | mriedem: host_subset_size only comes into play in the scheduler. The conductor wouldn't use it | 20:48 |
| mriedem | so we don't lookup hostmappings every time we schedule an instance | 20:48 |
| mriedem | edleafe: i was just commenting on the number of existing options | 20:48 |
| dansmith | mriedem: we discussed that yeah, bauzas had a plan for just filtering the alternatives with matching cells | 20:49 |
| *** thorst has quit IRC | 20:49 | |
| dansmith | it was done in his set I think | 20:49 |
| edleafe | it was also done here: https://review.openstack.org/#/c/465768/8/nova/scheduler/filter_scheduler.py | 20:49 |
| mriedem | something else nagging me a bit here, | 20:49 |
| edleafe | (that was before moving to scheduler manager) | 20:49 |
| jaypipes | fuck the instance group shit. | 20:50 |
| mriedem | say it's a create request with 100 servers | 20:50 |
| mriedem | we get our allocation candidates and make claims on them | 20:50 |
| smcginnis | jaypipes: How do you really feel? | 20:50 |
| mriedem | then we're going to send some alternatives down, but couldn't those alternatives be full by the time we're doing with the claims? | 20:50 |
| jaypipes | smcginnis: sorry, that wasn't meant for public consumption. | 20:50 |
| mriedem | as the alternatives aren't really a fresh set? | 20:50 |
| smcginnis | :) | 20:50 |
| mriedem | s/doing/done/ | 20:51 |
| edleafe | mriedem: there could be dupes in the alternatives for each of the 100, too | 20:51 |
| jaypipes | mriedem: yes, you're absolutely correct. this is the problem with num_instances > 1 :( | 20:51 |
| edleafe | mriedem: just like the selected host could be the same for some of the 100 | 20:51 |
| dansmith | jaypipes: fwiw, I was just (minutes ago) helping a real nova user debug a problem with --num_instances=100 :/ | 20:51 |
| mriedem | edleafe: well filtering dupes from the list of alternatives seems trivial, just do a set based on rp uuid | 20:52 |
| edleafe | mriedem: why? | 20:52 |
| mriedem | i get the feeling that huawei's users are also lovers of multiple create | 20:52 |
| edleafe | with the scheduler claiming, the only reason a build will fail will be a host issue | 20:52 |
| jaypipes | it's an NFV thing... | 20:52 |
| mriedem | b/c i've seen bug reports from them about it | 20:52 |
| edleafe | so most will succeed, and we throw away the alternatives | 20:52 |
| mriedem | edleafe: i assume you and i are talking about different things then | 20:52 |
| dansmith | while googling for something, I also found a bug filed against another platform with "nova has --num_instances and I use it a lot, can't use your platform without it...." | 20:52 |
| mriedem | ha, that's a first | 20:53 |
| mriedem | 'every other cloud has spot instances but nova *doesn't* have it' | 20:53 |
| *** catintheroof has quit IRC | 20:53 | |
| edleafe | mriedem: what are you referring to as "alternatives"? | 20:53 |
| *** MVenesio has quit IRC | 20:53 | |
| *** lpetrut has joined #openstack-nova | 20:54 | |
| jaypipes | so here's the thing... I could change the scheduler manager to get rid of all the "consume this stuff within the _schedule() filter scheduler loop" and call the placement API for allocation candidates once for each instance in num_instances. | 20:54 |
| jaypipes | the problem with that is the "allocate 99 instances and the last one fails" problem that large Heat stacks have. | 20:54 |
| edleafe | mriedem: I mean the set of N potential hosts we return with the selected host to the conductor. That's passed to the cell, and if the build fails, the cell conductor does retries with the alternatives | 20:54 |
| mriedem | edleafe: yes, but what are you saying about duplicates? | 20:55 |
| mriedem | wrt alternatives sent to the cell? | 20:55 |
| mriedem | do you mean like 2 alternatives pointing at the same compute node for ram/cpu but different storage providers? (one local, one shared)? | 20:56 |
| mriedem | jaypipes: also, if we call placement for allocation candidates for each instance, that would greatly slow things down wouldn't it? | 20:56 |
| dansmith | that would suck for 100-instance batches, yeah | 20:57 |
| mriedem | or is that a loop of (1) get allocatoin candidates, (2) filter/weigh (3) claim, (4) repeat | 20:57 |
| jaypipes | mriedem: that scenario is actually *exactly* what I benchmarked here: https://github.com/jaypipes/placement-bench/ | 20:57 |
| edleafe | mriedem: if we're returning 100 sets of (selected host, alt1, alt2, ...altN) to the conductor, the same host could appear in multiple alternate lists | 20:57 |
| jaypipes | mriedem: and I was able to achieve >200 claimed requests *per second*. | 20:58 |
| mriedem | jaypipes: what is a 'claimed request'? | 20:58 |
| jaypipes | mriedem: request for resources, get list of providers, claim for an instance against a selected provider. | 20:58 |
| mriedem | that's the full thing through select_destinations to placement to filter/weigh to claim? | 20:58 |
| jaypipes | right. | 20:58 |
| mriedem | edleafe: yes agree | 20:59 |
| mriedem | edleafe: by default it's 3 | 20:59 |
| mriedem | but yes | 20:59 |
| jaypipes | mriedem: I'm think maybe it would be better to scrap trying to make this square peg fit the filter scheduler round hole. | 20:59 |
| *** thorst has joined #openstack-nova | 21:00 | |
| mriedem | what is the square peg in this analogy? | 21:00 |
| mriedem | multiple create? | 21:00 |
| dansmith | edleafe | 21:00 |
| mriedem | ha | 21:00 |
| dansmith | oh did I say that? | 21:00 |
| jaypipes | mriedem: and let me move the _schedule() internals into the caching scheduler and make the filter scheduler call placement each iteration of the num_instances loop... | 21:00 |
| mriedem | he's no square daddy-o | 21:01 |
| *** annegentle has quit IRC | 21:01 | |
| *** gbarros has quit IRC | 21:01 | |
| dansmith | I dunno, I'm highly skeptical of the efficiency of doing one call per instance with 100 instances | 21:01 |
| *** annegentle has joined #openstack-nova | 21:01 | |
| edleafe | dansmith: I sure feel like it | 21:01 |
| jaypipes | dansmith: well, actually.. | 21:01 |
| jaypipes | dansmith: I *am* returning the provider_summaries, which has the remaining capacity for various resources in it. | 21:02 |
| dansmith | even if we *can* do it, that still might be a lot of network, a lot of CPU, etc, and if there's any latency between placement and scheduler... | 21:02 |
| dansmith | jaypipes: right, see, what I would do is, | 21:02 |
| dansmith | ask for some batch N, and consume from there until a claim fails, and then do another batch until you've processed num_instances | 21:02 |
| jaypipes | deduct from the capacity on each loop... | 21:02 |
| jaypipes | which is pretty much what the _schedule() method is doing right now for the host state objects. | 21:03 |
| mriedem | that's "chosen_host.obj.consume_from_request(spec_obj)" right? | 21:03 |
| dansmith | well, if you try to predict capacity, you might be missing some things that placement knows about, and definitely other schedulers are claiming | 21:03 |
| dims | mriedem : LOL | 21:03 |
| dansmith | so if you just claim until you get a failure and then ask for a new batch I think you're safer | 21:03 |
| jaypipes | mriedem: yeah, kinda. | 21:03 |
| dansmith | that could be every request if things are chaotic, but if not, then you've saved a bunch of work I think | 21:04 |
| jaypipes | dansmith: that's my problem with doing that... we get the cache invalidation problem :( | 21:04 |
| dansmith | right, so you claim until fail, then get another batch of candidates, IMHO | 21:05 |
| dansmith | jaypipes: the other thing that sucks about --min, which is what I was debugging just now, | 21:05 |
| dansmith | is that --min only works for quota, | 21:05 |
| dansmith | and if your max is 100 and you can only fit 93, you fail the whole deal, | 21:05 |
| dansmith | even if --min=1 | 21:05 |
| dansmith | which is pretty stupid and not what the user expects | 21:05 |
| dansmith | so it would be nice to fix that in the process of doing this, if we could | 21:06 |
| jaypipes | this whole num_instances thing is stupid to begin with. | 21:06 |
| dansmith | except for all the users that actively use it | 21:06 |
| dansmith | https://bugs.launchpad.net/nova/+bug/1703457 | 21:06 |
| openstack | Launchpad bug 1703457 in OpenStack Compute (nova) "RFE: More intuitive usage of --min and --max arguments in openstack server create" [Undecided,New] | 21:06 |
| dansmith | not from an nfv workload ^ btw | 21:06 |
| jaypipes | yeah, but it should be an orchestration layer that handles all the crazy. not the Nova scheduler. | 21:07 |
| jaypipes | like I don't know.. Heat with convergence? :( | 21:07 |
| dansmith | well, you can say that for sure. they're not using heat or anything else, and we have it in our api... | 21:07 |
| dansmith | given the number of people that seem to want it, | 21:08 |
| dansmith | I think we either keep supporting it, or we're going to be asked to do reservations for that orchestrator thing | 21:08 |
| *** smatzek has quit IRC | 21:09 | |
| * jaypipes thinks.. | 21:10 | |
| dansmith | at a minimum, we should either fix (or allow fixes) for the above kind of stupid thing, or we should cut the whole bit of it out of the api with a microversion | 21:10 |
| melwitt | yeah, cfriesen opened a bug about failing a request where >= min_count instances were successful but it fails bc it couldn't satisfy max_count, two years ago https://bugs.launchpad.net/nova/+bug/1458122 | 21:10 |
| openstack | Launchpad bug 1458122 in OpenStack Compute (nova) "nova shouldn't error if we can't schedule all of max_count instances at boot time" [Wishlist,Opinion] - Assigned to Chris Friesen (cbf123) | 21:10 |
| mriedem | dansmith: even if you cut it out of the api with a microversion, there are all of the microversion before it where it lives | 21:10 |
| dansmith | but for a cloud that supports an application and doesn't use quota at all, --min-instances is literally ineffectual | 21:10 |
| dansmith | mriedem: well, I know, and I think we should just keep supporting it given the popularity, but if we're going to let it rot, we should signal that we're not going to fix things | 21:11 |
| dansmith | melwitt: ah yep | 21:11 |
| melwitt | I know they use the "request multiple instances" a lot at yahoo too | 21:12 |
| dansmith | anyway, I gotta run let some creeper feel my eyeballs | 21:12 |
| cdent | can somebody summarize/recapitulate the problem; i’ve been following along but it is coming in bits and pieces I’m not sure I get it. | 21:12 |
| jaypipes | alright, I think I can do this with the provider_summaries thing. | 21:12 |
| *** eharney has quit IRC | 21:13 | |
| mriedem | cdent: we aren't accounting for allocations already claimed when processing a list of instances in the request | 21:14 |
| mriedem | so we could essentially send alternatives to the cell that are already full | 21:14 |
| melwitt | taking a step back, I think the usefulness of multi boot is with the legacy scheduler behavior that would race and pick full hosts because of claiming in compute. so with multi boot, you get the advantage of not having a lot of your requests fail out because of races | 21:16 |
| *** gszasz has quit IRC | 21:17 | |
| cdent | melwitt: is that more important than “it’s just convenient to do it one step”? | 21:18 |
| *** dklyle has joined #openstack-nova | 21:18 | |
| melwitt | aside from that, people do like the ease of saying "I want N instances" but I don't think that's the main reason people like it | 21:18 |
| cdent | jinxish | 21:18 |
| melwitt | (I could be wrong ... disclaimer) | 21:18 |
| dansmith | melwitt: well, fwiw, I've never heard that use case for it | 21:18 |
| dansmith | but maybe so | 21:18 |
| *** fragatina has quit IRC | 21:18 | |
| melwitt | at yahoo I think that's the use case | 21:18 |
| melwitt | when people do a for-loop of 100 instances, lots of them fail bc races | 21:19 |
| *** david-lyle has quit IRC | 21:19 | |
| dansmith | so you ask for 30 because you actually want 20? | 21:19 |
| *** fragatina has joined #openstack-nova | 21:19 | |
| melwitt | hm, I don't think so. I haven't heard of that. they just want like 100 at a time and if they do it one-by-one a lot fail, if they do multi boot they get almost all | 21:20 |
| *** john51 has quit IRC | 21:20 | |
| *** fragatin_ has joined #openstack-nova | 21:21 | |
| melwitt | well, also, in VM clusters at yahoo retries are disabled. so that's why so many fail due to race | 21:21 |
| openstackgerrit | Maciej Kucia proposed openstack/nova master: SR-IOV passthrough: Check PF only if VF is enabled https://review.openstack.org/476642 | 21:21 |
| *** john51 has joined #openstack-nova | 21:21 | |
| melwitt | (retries are enabled for baremetal clusters) | 21:22 |
| mriedem | wouldn't you also do multiple create to get the instances in the same affinity group? | 21:23 |
| mriedem | at once | 21:23 |
| mriedem | like, i need 10 instances and need them all on the same host | 21:23 |
| *** john51 has quit IRC | 21:23 | |
| melwitt | you could. I think you could also pass the same group for each of 10 too right | 21:24 |
| mriedem | yeah, but you might land 5 of 10 on that same host and then the 6th fails | 21:24 |
| melwitt | yeah, true | 21:24 |
| mriedem | so what do you do? delete them all and try again? | 21:24 |
| *** fragatina has quit IRC | 21:25 | |
| *** fragatin_ has quit IRC | 21:25 | |
| mriedem | i believe that's why the soft affinity weigher was added | 21:25 |
| mriedem | best effort to place the group on the same host, but not fail if you can't | 21:25 |
| *** fragatina has joined #openstack-nova | 21:25 | |
| melwitt | I see | 21:26 |
| *** john51 has joined #openstack-nova | 21:26 | |
| *** oanson has quit IRC | 21:28 | |
| *** priteau has quit IRC | 21:28 | |
| *** oanson has joined #openstack-nova | 21:29 | |
| *** acormier has joined #openstack-nova | 21:30 | |
| *** penick has joined #openstack-nova | 21:30 | |
| *** mmehan has quit IRC | 21:35 | |
| *** baoli has quit IRC | 21:36 | |
| *** zzzeek_ has quit IRC | 21:37 | |
| *** baoli has joined #openstack-nova | 21:37 | |
| mdrabe | Any cores willing to look at https://review.openstack.org/#/c/467774/5? Small bug fix that's been chillin | 21:38 |
| *** acormier has quit IRC | 21:38 | |
| *** kristian__ has quit IRC | 21:38 | |
| *** acormier has joined #openstack-nova | 21:38 | |
| *** zzzeek_ has joined #openstack-nova | 21:40 | |
| *** priteau has joined #openstack-nova | 21:41 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix quota class set APIs https://review.openstack.org/467999 | 21:44 |
| *** priteau has quit IRC | 21:47 | |
| mriedem | mdrabe: done | 21:47 |
| *** gongysh has joined #openstack-nova | 21:47 | |
| mdrabe | thnx | 21:48 |
| *** esberglu has quit IRC | 21:49 | |
| *** vishwanathj has joined #openstack-nova | 21:52 | |
| *** armax has quit IRC | 21:52 | |
| *** trungnv has joined #openstack-nova | 21:53 | |
| *** hieulq has joined #openstack-nova | 21:54 | |
| *** slaweq has quit IRC | 21:55 | |
| *** slaweq has joined #openstack-nova | 21:55 | |
| mriedem | overall i don't know how great of an idea this is in general, or what side effects we might have, would like to see dansmith look at it but he's out for the rest of the day | 21:56 |
| *** lpetrut has quit IRC | 21:58 | |
| *** slaweq has quit IRC | 21:59 | |
| *** kylek3h has quit IRC | 22:00 | |
| *** slaweq has joined #openstack-nova | 22:00 | |
| melwitt | mriedem: I wondered similar, like how/why aren't those resources cleaned up by local delete + init_host. but I'm not well-versed in what's special about evacuation resources | 22:01 |
| *** thorst has quit IRC | 22:02 | |
| *** xyang1 has quit IRC | 22:07 | |
| *** acormier has quit IRC | 22:10 | |
| *** burgerk has quit IRC | 22:11 | |
| *** trungnv has quit IRC | 22:12 | |
| *** hieulq has quit IRC | 22:12 | |
| *** deep-book-gk_ has joined #openstack-nova | 22:14 | |
| *** dillaman has joined #openstack-nova | 22:14 | |
| *** slaweq has quit IRC | 22:16 | |
| *** deep-book-gk_ has left #openstack-nova | 22:17 | |
| openstackgerrit | Chris Dent proposed openstack/nova-specs master: Provide detailed error information for placement API https://review.openstack.org/418393 | 22:17 |
| *** abalutoiu_ has joined #openstack-nova | 22:18 | |
| *** acormier has joined #openstack-nova | 22:19 | |
| *** dillaman has quit IRC | 22:19 | |
| *** lyan has quit IRC | 22:20 | |
| mdrabe | mriedem melwitt: It'll make init_host a little slower, but I don't know where else the source host instance deletion should be handled | 22:21 |
| *** felipemonteiro_ has quit IRC | 22:22 | |
| *** trungnv has joined #openstack-nova | 22:27 | |
| *** hieulq has joined #openstack-nova | 22:27 | |
| *** zzzeek_ has quit IRC | 22:32 | |
| cdent | goodnight all | 22:32 |
| *** cdent has quit IRC | 22:32 | |
| *** gszasz has joined #openstack-nova | 22:32 | |
| melwitt | mdrabe: I think init_host is the right place for doing that. I had thought they'd be deleted by the "_cleanup_running_deleted_instances" periodic task. dansmith knows a lot about evacuate, so hopefully tomorrow he can look at the patch and let us know if there are any gotchas there | 22:34 |
| *** penick has quit IRC | 22:37 | |
| *** slaweq has joined #openstack-nova | 22:37 | |
| mriedem | melwitt: _cleanup_running_deleted_instances gets the instances on that host, | 22:39 |
| mriedem | so that won't work if the instance was evacuated and it's instance.host field was changed | 22:39 |
| melwitt | ah, okay. that's what I was missing | 22:40 |
| mriedem | that's why mdrabe's change is doing it via the migrations | 22:40 |
| *** slaweq has quit IRC | 22:41 | |
| *** catintheroof has joined #openstack-nova | 22:43 | |
| *** armax has joined #openstack-nova | 22:44 | |
| *** zzzeek_ has joined #openstack-nova | 22:50 | |
| openstackgerrit | Matt Rabe proposed openstack/nova master: Query deleted instance records during _destroy_evacuated_instances https://review.openstack.org/467774 | 22:51 |
| *** priteau has joined #openstack-nova | 22:54 | |
| *** priteau has quit IRC | 22:58 | |
| *** tjones has quit IRC | 23:05 | |
| *** acormier has quit IRC | 23:06 | |
| *** acormier has joined #openstack-nova | 23:07 | |
| *** fragatin_ has joined #openstack-nova | 23:07 | |
| *** takedakn has joined #openstack-nova | 23:07 | |
| *** fragatina has quit IRC | 23:10 | |
| *** claudiub has quit IRC | 23:11 | |
| *** acormier has quit IRC | 23:11 | |
| *** fragatin_ has quit IRC | 23:14 | |
| *** fragatina has joined #openstack-nova | 23:15 | |
| *** thorst has joined #openstack-nova | 23:17 | |
| rm_work | I've noticed today that in a new devstack with cirros-0.3.5-x86_64-disk (and supposedly 0.3.4 too according to someone else who is seeing the same thing) the ssh keypair I'm configuring isn't ending up on the VM, there's no ~cirros/.ssh at all, so I have to log in with the default password | 23:18 |
| rm_work | anyone else able to replicate that? | 23:18 |
| openstackgerrit | melanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990 | 23:20 |
| *** thorst has quit IRC | 23:21 | |
| *** acormier has joined #openstack-nova | 23:30 | |
| clarkb | rm_work: I'm pretty sure the gate tests ssh connectivity using ssh keys | 23:34 |
| *** Swami has quit IRC | 23:34 | |
| clarkb | rm_work: but maybe different versions of cirros? | 23:35 |
| rm_work | hmmm | 23:35 |
| rm_work | so, something about our config then? | 23:35 |
| rm_work | do you know what version the gate uses? | 23:35 |
| clarkb | maybe? is metadata service enabled? and if not are you bootign with config drive? | 23:35 |
| rm_work | i know 0.3.4 and 0.3.5 are exhibiting this issue | 23:35 |
| rm_work | i am not sure, my nova knowledge is almost zero, I just run devstack and expect it to work :( | 23:36 |
| clarkb | is your neutron metadata proxy enabled? | 23:36 |
| rm_work | can I paste you the config? | 23:36 |
| rm_work | I haven't changed anything for nova or neutron from devstack defaults | 23:36 |
| clarkb | rm_work: why don't you try with a working local.conf from the gate see if that works, then compare? | 23:36 |
| rm_work | k | 23:36 |
| clarkb | but I can take a look at your config too | 23:36 |
| rm_work | i mean my config is just devstack default | 23:37 |
| rm_work | but I can pastebin it | 23:37 |
| clarkb | so you don't have anything in your local.conf? | 23:37 |
| clarkb | you just manually enter in the passwords when asked? | 23:37 |
| rm_work | nothing for nova or neutron | 23:37 |
| rm_work | this is my local.conf: http://paste.openstack.org/show/614970/ | 23:38 |
| rm_work | maybe i'm missing a new nova service or something | 23:39 |
| *** edmondsw has quit IRC | 23:40 | |
| clarkb | looks like gate runs n-api-meta too | 23:41 |
| rm_work | yeah I appear to be missing some | 23:41 |
| rm_work | gate has n-api,n-api-meta,n-cauth,n-cond,n-cpu,n-novnc,n-obj,n-sch,peakmem_tracker,placement-api | 23:41 |
| rm_work | I am missing ... | 23:41 |
| rm_work | at least that | 23:41 |
| clarkb | rm_work: I would try booting an instance with config drive (assuming cirros supports config drive) as a step to rule metadata in our out | 23:41 |
| clarkb | s/our/or/ | 23:41 |
| *** hongbin has quit IRC | 23:42 | |
| rm_work | well first I will prolly try with an updated local.conf to include the missing service(s) | 23:42 |
| rm_work | because I'm missing n-api-meta,n-cauth,n-novnc | 23:43 |
| rm_work | I assume I don't need n-novnc | 23:43 |
| rm_work | i don't know what n-cauth does | 23:43 |
| rm_work | but n-api-meta seems relevant? | 23:43 |
| clarkb | both should be optional iirc and cauth is going away? but ya n-api-meta is a metadata api server iirc | 23:43 |
| *** amotoki is now known as amotoki_away | 23:43 | |
| clarkb | I thought n-api would also service as that but maybe devstack doesn't configure it that wayZ? | 23:44 |
| rm_work | dunno | 23:44 |
| rm_work | will unstack/restack with n-api-meta | 23:44 |
| rm_work | and see if it helps | 23:44 |
| *** annegentle has quit IRC | 23:45 | |
| *** rajathagasthya has quit IRC | 23:47 | |
| *** chyka has quit IRC | 23:54 | |
| *** chyka has joined #openstack-nova | 23:55 | |
| *** takedakn has quit IRC | 23:55 | |
| *** thorst has joined #openstack-nova | 23:57 | |
| *** bkopilov has quit IRC | 23:57 | |
| *** oanson has quit IRC | 23:59 | |
| *** catintheroof has quit IRC | 23:59 | |
| *** chyka has quit IRC | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!