*** mvk_ has quit IRC | 00:03 | |
*** pvaneck has quit IRC | 00:12 | |
*** diogogmt has joined #openstack-rally | 00:22 | |
*** davideag_ has quit IRC | 00:23 | |
*** rpromyshlennikov has quit IRC | 00:23 | |
*** rpromyshlennikov has joined #openstack-rally | 00:35 | |
*** mvk_ has joined #openstack-rally | 00:37 | |
*** omar_rally_noob has joined #openstack-rally | 00:39 | |
*** mkrcmari__ has quit IRC | 00:40 | |
*** adiantum has quit IRC | 00:43 | |
omar_rally_noob | how do you know exactly what a task is doing for example https://github.com/openstack/rally/blob/master/samples/tasks/scenarios/authenticate/keystone.json I understand the task is going to create 3 tenants with 50 users in each tenant and auth 100 times with 5db transactions at a time. | 00:43 |
---|---|---|
omar_rally_noob | does the task produce logs or detailed output, all i get are results from running these tasks | 00:45 |
*** yingjun has joined #openstack-rally | 00:49 | |
*** stpierre has joined #openstack-rally | 00:54 | |
*** stpierre has quit IRC | 00:55 | |
boris-42 | omar_rally_noob: hi | 01:06 |
*** baohua has joined #openstack-rally | 01:07 | |
boris-42 | omar_rally_noob: you can run "rally task results" to get all data in JSON or "rally task report" to get HTML reports | 01:07 |
boris-42 | omar_rally_noob: as well you can run rally -v task start ... to get logs of running rally | 01:07 |
boris-42 | or -vd to get debug | 01:07 |
*** omar_rally_noob has quit IRC | 01:07 | |
*** akrzos has quit IRC | 01:41 | |
*** ilbot3 has quit IRC | 01:45 | |
*** yanyanhu has joined #openstack-rally | 01:47 | |
*** ilbot3 has joined #openstack-rally | 01:48 | |
*** yanyanhu has quit IRC | 01:57 | |
*** yanyanhu has joined #openstack-rally | 01:58 | |
*** harshs has quit IRC | 02:02 | |
*** akrzos has joined #openstack-rally | 02:07 | |
openstackgerrit | Yanyan Hu proposed openstack/rally: Add basic support for Senlin service https://review.openstack.org/298109 | 02:32 |
openstackgerrit | Merged openstack/rally: Change deployment model schema https://review.openstack.org/276821 | 02:57 |
*** yanyanhu has quit IRC | 03:05 | |
*** yanyanhu has joined #openstack-rally | 03:06 | |
*** ajo_ has joined #openstack-rally | 03:27 | |
*** rpromysh1 has joined #openstack-rally | 03:30 | |
*** dmellado_ has joined #openstack-rally | 03:31 | |
*** ajo has quit IRC | 03:32 | |
*** rpromyshlennikov has quit IRC | 03:32 | |
*** dmellado has quit IRC | 03:32 | |
*** serverascode has quit IRC | 03:33 | |
*** serverascode has joined #openstack-rally | 03:39 | |
*** akrzos has quit IRC | 04:06 | |
*** dratushnyy has joined #openstack-rally | 04:15 | |
*** ushkalim has quit IRC | 04:31 | |
*** rdas has joined #openstack-rally | 04:39 | |
*** arnoldje has quit IRC | 04:49 | |
*** shausy has joined #openstack-rally | 04:59 | |
*** rcernin has joined #openstack-rally | 04:59 | |
*** dratushnyy has quit IRC | 05:09 | |
*** dratushnyy has joined #openstack-rally | 05:09 | |
*** dratushnyy has quit IRC | 05:10 | |
*** sverma has joined #openstack-rally | 05:28 | |
*** shausy has quit IRC | 05:39 | |
*** shausy has joined #openstack-rally | 05:39 | |
*** dratushnyy has joined #openstack-rally | 05:43 | |
*** lazy_prince has quit IRC | 05:44 | |
*** pcaruana has quit IRC | 05:49 | |
*** lazy_prince has joined #openstack-rally | 05:53 | |
*** e0ne has joined #openstack-rally | 05:56 | |
*** rdas has quit IRC | 05:56 | |
openstackgerrit | Boris Pavlovic proposed openstack/rally: Reduce default speed of users creation in users context https://review.openstack.org/299757 | 05:57 |
*** rdas has joined #openstack-rally | 05:57 | |
*** ushkalim has joined #openstack-rally | 06:09 | |
*** lazy_prince has quit IRC | 06:15 | |
*** e0ne has quit IRC | 06:24 | |
*** lazy_prince has joined #openstack-rally | 06:24 | |
*** bkopilov has quit IRC | 06:24 | |
*** winston-d_ has joined #openstack-rally | 06:32 | |
*** bkopilov has joined #openstack-rally | 06:36 | |
*** tesseract has joined #openstack-rally | 06:49 | |
*** tesseract is now known as Guest11037 | 06:49 | |
*** akamyshnikova has joined #openstack-rally | 06:51 | |
*** e0ne has joined #openstack-rally | 07:15 | |
*** e0ne has quit IRC | 07:18 | |
*** abehl has joined #openstack-rally | 07:24 | |
*** arxcruz has joined #openstack-rally | 07:30 | |
*** pcaruana has joined #openstack-rally | 07:41 | |
*** permalac has joined #openstack-rally | 07:48 | |
openstackgerrit | Boris Pavlovic proposed openstack/rally: Calculate duration of load more accurate https://review.openstack.org/298590 | 07:48 |
*** rdas has quit IRC | 07:54 | |
openstackgerrit | Srinivas Sakhamuri proposed openstack/rally: Use glance wrapper for nova and cinder https://review.openstack.org/298787 | 07:55 |
*** klindgren has quit IRC | 07:56 | |
*** klindgren has joined #openstack-rally | 07:57 | |
openstackgerrit | Merged openstack/rally: Reduce default speed of users creation in users context https://review.openstack.org/299757 | 07:57 |
openstackgerrit | Merged openstack/rally: Actually delete result_batch arrays https://review.openstack.org/298636 | 07:59 |
*** amaretskiy has joined #openstack-rally | 08:02 | |
*** permalac has quit IRC | 08:06 | |
*** permalac has joined #openstack-rally | 08:25 | |
*** rpromysh1 has quit IRC | 08:27 | |
*** rpromyshlennikov has joined #openstack-rally | 08:27 | |
*** e0ne has joined #openstack-rally | 08:28 | |
*** e0ne has quit IRC | 08:29 | |
openstackgerrit | Merged openstack/rally: [Reports] Various fixes for load profile chart https://review.openstack.org/298622 | 08:32 |
*** e0ne has joined #openstack-rally | 08:34 | |
*** e0ne has quit IRC | 08:35 | |
*** adiantum has joined #openstack-rally | 08:38 | |
openstackgerrit | Rodion Promyshlennikov proposed openstack/rally: Move Credentials initialization to OSClients https://review.openstack.org/294520 | 08:43 |
openstackgerrit | Rodion Promyshlennikov proposed openstack/rally: Make Credential class plugin based https://review.openstack.org/294521 | 08:43 |
*** adiantum has quit IRC | 08:43 | |
*** rdas has joined #openstack-rally | 08:44 | |
openstackgerrit | Rodion Promyshlennikov proposed openstack/rally: Add support for multi-credentials deployment https://review.openstack.org/297202 | 08:44 |
*** adiantum has joined #openstack-rally | 08:48 | |
openstackgerrit | Andrey Kurilin proposed openstack/rally: DO NOT MERGE: print more debug messages for neutron scenario https://review.openstack.org/288522 | 09:20 |
*** yingjun has quit IRC | 09:24 | |
*** e0ne has joined #openstack-rally | 09:30 | |
*** baohua has quit IRC | 09:35 | |
*** e0ne has quit IRC | 09:37 | |
openstackgerrit | Merged openstack/rally: Calculate duration of load more accurate https://review.openstack.org/298590 | 09:40 |
*** sverma has quit IRC | 09:41 | |
*** e0ne has joined #openstack-rally | 09:42 | |
*** mkrcmari__ has joined #openstack-rally | 09:44 | |
*** tosky has joined #openstack-rally | 09:45 | |
openstackgerrit | Boris Pavlovic proposed openstack/rally: [WIP] OMG performance what are you doing? stohp please!! https://review.openstack.org/298591 | 09:46 |
openstackgerrit | Boris Pavlovic proposed openstack/rally: [Reports] Fix calc of running iterations in parallel https://review.openstack.org/299833 | 09:46 |
*** mvk_ has quit IRC | 09:47 | |
*** mvk_ has joined #openstack-rally | 09:52 | |
*** yanyanhu has quit IRC | 09:54 | |
*** yanyanhu has joined #openstack-rally | 09:55 | |
*** yanyanhu has quit IRC | 09:55 | |
*** mkrcmari__ has quit IRC | 09:55 | |
*** rook-lappy has quit IRC | 10:05 | |
*** mkrcmari__ has joined #openstack-rally | 10:13 | |
*** e0ne has quit IRC | 10:15 | |
*** mvk_ has quit IRC | 10:17 | |
*** e0ne has joined #openstack-rally | 10:17 | |
*** klindgren has quit IRC | 10:20 | |
*** e0ne has quit IRC | 10:20 | |
*** klindgren has joined #openstack-rally | 10:20 | |
*** e0ne has joined #openstack-rally | 10:21 | |
*** e0ne has quit IRC | 10:22 | |
*** e0ne has joined #openstack-rally | 10:23 | |
*** e0ne has quit IRC | 10:24 | |
*** mvk_ has joined #openstack-rally | 10:28 | |
*** mkrcmari__ has quit IRC | 10:32 | |
*** winston-d_ has quit IRC | 10:42 | |
*** permalac has quit IRC | 10:44 | |
*** dmellado_ is now known as dmellado | 10:52 | |
*** rdas has quit IRC | 10:59 | |
*** e0ne has joined #openstack-rally | 11:17 | |
*** e0ne has quit IRC | 11:18 | |
*** bigjools has quit IRC | 11:22 | |
*** e0ne has joined #openstack-rally | 11:23 | |
*** bigjools has joined #openstack-rally | 11:24 | |
*** bigjools has quit IRC | 11:24 | |
*** bigjools has joined #openstack-rally | 11:24 | |
*** e0ne has quit IRC | 11:26 | |
*** sverma has joined #openstack-rally | 11:26 | |
*** sverma has quit IRC | 11:28 | |
*** sverma has joined #openstack-rally | 11:33 | |
openstackgerrit | Merged openstack/rally: devstack: support to be enabled with different plugin name https://review.openstack.org/296239 | 11:33 |
*** e0ne has joined #openstack-rally | 11:35 | |
openstackgerrit | Piyush Raman Srivastava proposed openstack/rally: Show Missing Argument Error raised by Argparse https://review.openstack.org/298349 | 11:38 |
openstackgerrit | Dmitry Ratushnyy proposed openstack/rally: [CLI] Display task errors in human-friendly form https://review.openstack.org/294727 | 11:40 |
*** adiantum has quit IRC | 11:40 | |
openstackgerrit | Chris St. Pierre proposed openstack/rally: Add Cinder volume_types context https://review.openstack.org/295403 | 11:41 |
*** e0ne has quit IRC | 11:41 | |
*** akrzos has joined #openstack-rally | 11:54 | |
*** yingjun has joined #openstack-rally | 11:57 | |
*** e0ne has joined #openstack-rally | 11:59 | |
*** e0ne has quit IRC | 12:00 | |
*** adiantum has joined #openstack-rally | 12:05 | |
*** e0ne has joined #openstack-rally | 12:11 | |
*** e0ne has quit IRC | 12:13 | |
*** kbaikov has quit IRC | 12:14 | |
*** akrzos has quit IRC | 12:26 | |
*** akrzos has joined #openstack-rally | 12:27 | |
*** moravec has quit IRC | 12:30 | |
*** MaxPC has joined #openstack-rally | 12:44 | |
openstackgerrit | Illia Khudoshyn proposed openstack/rally: Refactor DB shema for task results https://review.openstack.org/297020 | 12:45 |
openstackgerrit | Alexander Maretskiy proposed openstack/rally: [Reports] Fix calc of running iterations in parallel https://review.openstack.org/299833 | 12:48 |
*** shausy has quit IRC | 12:49 | |
*** e0ne has joined #openstack-rally | 12:50 | |
*** dmellado is now known as dmellado|lunch | 12:50 | |
*** e0ne has quit IRC | 12:51 | |
openstackgerrit | Alexander Maretskiy proposed openstack/rally: [WIP] OMG performance what are you doing? stohp please!! https://review.openstack.org/298591 | 12:51 |
openstackgerrit | Merged openstack/rally: [Reports] Refactor processing.plot.plot https://review.openstack.org/294119 | 13:02 |
openstackgerrit | Merged openstack/rally: [Reports] Move angular directive to separate file https://review.openstack.org/298597 | 13:02 |
openstackgerrit | Andrey Kurilin proposed openstack/rally: Use existing_users in TestTaskSamples https://review.openstack.org/299919 | 13:04 |
*** mvk_ has quit IRC | 13:06 | |
*** MaxPC has quit IRC | 13:15 | |
*** yingjun has quit IRC | 13:19 | |
*** yingjun has joined #openstack-rally | 13:20 | |
*** baohua has joined #openstack-rally | 13:22 | |
*** stpierre has joined #openstack-rally | 13:24 | |
*** yingjun has quit IRC | 13:24 | |
*** MaxPC has joined #openstack-rally | 13:29 | |
*** diogogmt has quit IRC | 13:30 | |
*** e0ne has joined #openstack-rally | 13:32 | |
*** dratushnyy has quit IRC | 13:34 | |
*** diogogmt has joined #openstack-rally | 13:35 | |
*** dpaterson has joined #openstack-rally | 13:38 | |
*** moravec has joined #openstack-rally | 13:40 | |
openstackgerrit | Yanyan Hu proposed openstack/rally: Add basic support for Senlin service https://review.openstack.org/298109 | 13:41 |
*** permalac has joined #openstack-rally | 13:45 | |
*** klkumar has joined #openstack-rally | 13:46 | |
*** arnoldje has joined #openstack-rally | 13:47 | |
*** kbaikov has joined #openstack-rally | 13:47 | |
*** e0ne has quit IRC | 13:52 | |
*** permalac has quit IRC | 13:53 | |
*** baohua has quit IRC | 13:53 | |
*** diogogmt has quit IRC | 13:54 | |
*** e0ne has joined #openstack-rally | 13:54 | |
*** dmellado|lunch is now known as dmellado | 13:54 | |
*** baohua has joined #openstack-rally | 13:54 | |
*** e0ne has quit IRC | 13:56 | |
*** pkarikh has joined #openstack-rally | 14:01 | |
*** pkarikh has left #openstack-rally | 14:02 | |
*** rook-lappy has joined #openstack-rally | 14:05 | |
*** ushkalim has quit IRC | 14:06 | |
openstackgerrit | Andrey Kurilin proposed openstack/rally: Use existing_users in TestTaskSamples https://review.openstack.org/299919 | 14:06 |
*** baohua has quit IRC | 14:07 | |
*** baohua has joined #openstack-rally | 14:08 | |
*** ushkalim has joined #openstack-rally | 14:17 | |
*** baohua_ has joined #openstack-rally | 14:18 | |
*** baohua has quit IRC | 14:21 | |
*** rook-lappy has quit IRC | 14:22 | |
*** e0ne has joined #openstack-rally | 14:29 | |
*** e0ne has quit IRC | 14:29 | |
openstackgerrit | Roman Vasilets proposed openstack/rally: Add exact error message into runcommand_heat scenario https://review.openstack.org/299999 | 14:38 |
openstackgerrit | Roman Vasilets proposed openstack/rally: [Docs] Anniversary patch https://review.openstack.org/300003 | 14:42 |
*** mvk_ has joined #openstack-rally | 14:42 | |
*** e0ne has joined #openstack-rally | 14:45 | |
openstackgerrit | Srinivas Sakhamuri proposed openstack/rally: Make glance web uploads streamable https://review.openstack.org/297808 | 14:46 |
*** zhenq has joined #openstack-rally | 14:51 | |
*** yingjun has joined #openstack-rally | 15:00 | |
*** ushkalim has quit IRC | 15:01 | |
*** e0ne has quit IRC | 15:01 | |
openstackgerrit | Andrey Kurilin proposed openstack/rally: Use existing_users in TestTaskSamples https://review.openstack.org/299919 | 15:07 |
*** abehl has quit IRC | 15:08 | |
*** ushkalim has joined #openstack-rally | 15:13 | |
*** klkumar has quit IRC | 15:17 | |
*** baohua_ has quit IRC | 15:22 | |
*** arxcruz has quit IRC | 15:25 | |
*** baohua has joined #openstack-rally | 15:27 | |
*** e0ne has joined #openstack-rally | 15:37 | |
*** tosky has quit IRC | 15:38 | |
*** e0ne has quit IRC | 15:40 | |
*** adiantum has quit IRC | 15:43 | |
*** e0ne has joined #openstack-rally | 15:44 | |
*** e0ne has quit IRC | 15:46 | |
*** e0ne has joined #openstack-rally | 15:47 | |
*** baohua has quit IRC | 15:50 | |
*** klkumar has joined #openstack-rally | 15:53 | |
*** harshs has joined #openstack-rally | 15:53 | |
*** harshs has quit IRC | 15:53 | |
*** jaypipes has joined #openstack-rally | 15:55 | |
*** e0ne has quit IRC | 15:58 | |
*** yingjun has quit IRC | 16:01 | |
*** pcaruana has quit IRC | 16:08 | |
*** e0ne has joined #openstack-rally | 16:11 | |
*** amaretskiy has quit IRC | 16:22 | |
*** arnoldje has quit IRC | 16:26 | |
*** davideagnello has joined #openstack-rally | 16:30 | |
*** e0ne has quit IRC | 16:32 | |
*** pvaneck has joined #openstack-rally | 16:42 | |
*** e0ne has joined #openstack-rally | 16:47 | |
*** e0ne has quit IRC | 16:50 | |
*** diogogmt has joined #openstack-rally | 17:01 | |
*** Guest11037 has quit IRC | 17:05 | |
*** tosky has joined #openstack-rally | 17:05 | |
*** rcernin has quit IRC | 17:06 | |
*** lezbar has quit IRC | 17:13 | |
openstackgerrit | Merged openstack/rally: Fix validate_args function in cliutils https://review.openstack.org/293569 | 17:28 |
*** dratushnyy has joined #openstack-rally | 17:34 | |
boris-42 | @stpierre hi | 17:37 |
boris-42 | @stpierre http://logs.openstack.org/33/299833/2/check/gate-rally-dsvm-neutron-rally/c064f03/rally-plot/resources_diff.txt.gz | 17:37 |
boris-42 | stpierre: ^ this looks bad | 17:37 |
stpierre | i don't think that's a new problem | 17:40 |
stpierre | looks like it may have somehow been caused by https://review.openstack.org/#/c/272638 | 17:43 |
stpierre | yep, loads of tracebacks here: http://logs.openstack.org/38/272638/14/check/gate-rally-dsvm-neutron-rally/40927c3/console.html.gz | 17:45 |
*** ushkalim has quit IRC | 17:46 | |
boris-42 | stpierre: please be very accurate when you are working on such things like cleanup | 17:46 |
boris-42 | it's very easy to mess up everything | 17:46 |
stpierre | we need to get the resource checks to influence the success of the job | 17:47 |
stpierre | that review was +2'd by four separate cores | 17:47 |
stpierre | if five of us could overlook the same thing, that's a problem with the gate IMO | 17:48 |
-openstackstatus- NOTICE: Job failures for missing traceroute packages are in the process of being fixed now, ETA 30 minutes to effectiveness for new jobs | 17:49 | |
boris-42 | stpierre: the problem is that core team started thinking that if it pass gates patch is OK and works for any cases | 17:50 |
stpierre | we should strive to make that the case | 17:50 |
stpierre | people are really good at screwing up, and they always will be. if there's an unexpected increase in resources, that should fail the job. | 17:50 |
boris-42 | stpierre: it's imposible, there will be always ways to find how to break it without braking gates | 17:50 |
stpierre | that doesn't mean we should just give up | 17:51 |
stpierre | we should get as close as we can | 17:51 |
boris-42 | stpierre: so I agree that there is room for improvement for Ci | 17:51 |
boris-42 | stpierre: so the problem here is that WE (as a core team) KNOW that we are NOT testing cleanup very well | 17:51 |
stpierre | well, i'm making a concrete suggestion for how to do it | 17:51 |
boris-42 | in gates | 17:51 |
stpierre | so let's test it better. we've got the data, we just need to make it fail the job | 17:52 |
boris-42 | stpierre: yep we can hardocde the amount of possible failuers | 17:52 |
boris-42 | stpierre: however it won't be ideal | 17:52 |
stpierre | don't let the perfect be the enemy of the good. it'd be more ideal than it is right now | 17:52 |
boris-42 | can you try to analyze the reason of failuers? | 17:53 |
stpierre | it's pretty obvious | 17:54 |
*** klkumar has quit IRC | 17:54 | |
stpierre | and is hopefully a one-line fix | 17:54 |
boris-42 | stpierre: ok I will wait for patch | 17:55 |
boris-42 | stpierre: btw | 17:55 |
boris-42 | https://github.com/openstack/rally/blob/master/tests/ci/osresources.py#L298 | 17:55 |
boris-42 | stpierre: ^ | 17:55 |
boris-42 | stpierre: so seems like we should hardcode here len of changes that we are allowing | 17:56 |
boris-42 | stpierre: or even do the better job and just fail if there are any changes | 17:56 |
openstackgerrit | Chris St. Pierre proposed openstack/rally: Cleanup: Fix getting name from neutron resources https://review.openstack.org/300121 | 17:57 |
stpierre | we need to make it a bit smarter, since there are some resources we expect -- 'default' security groups and the _member_ role at a minimum, it looks like | 17:58 |
stpierre | but i think we can probably filter those out, and then check for any additional changes | 17:58 |
stpierre | that'd be better than just failing on > 3 changes | 17:58 |
stpierre | i'll try to get that done this afternoon | 17:58 |
boris-42 | stpierre: thanks | 18:14 |
*** diogogmt_ has joined #openstack-rally | 18:16 | |
*** harshs has joined #openstack-rally | 18:16 | |
*** diogogmt has quit IRC | 18:17 | |
*** diogogmt_ is now known as diogogmt | 18:17 | |
*** shausy has joined #openstack-rally | 18:17 | |
*** shausy has quit IRC | 18:28 | |
*** mvk_ has quit IRC | 18:36 | |
*** openstackgerrit has quit IRC | 19:03 | |
*** openstackgerrit has joined #openstack-rally | 19:04 | |
openstackgerrit | Chris St. Pierre proposed openstack/rally: Test: don't stringify cloud resources until printing https://review.openstack.org/300157 | 19:04 |
openstackgerrit | Chris St. Pierre proposed openstack/rally: Test: Distinguish expected added resources https://review.openstack.org/300158 | 19:04 |
openstackgerrit | Chris St. Pierre proposed openstack/rally: Test: Fail jobs if resources are added or removed https://review.openstack.org/300159 | 19:04 |
openstackgerrit | Kevin Benton proposed openstack/rally: Allow max_avg_sla per atomic actions https://review.openstack.org/298462 | 19:26 |
boris-42 | stpierre: ping | 19:28 |
stpierre | pong | 19:28 |
boris-42 | stpierre: I talk to kevinbenton | 19:28 |
boris-42 | stpierre: he said that it is known issue that you can't delete security group in neutron | 19:28 |
boris-42 | stpierre: so he will try to fix this on neutron side | 19:29 |
stpierre | right | 19:29 |
stpierre | same is true in nova-net | 19:29 |
boris-42 | stpierre: btw you patch works | 19:29 |
boris-42 | stpierre: going to merge it | 19:29 |
boris-42 | stpierre: so he checked he is saying that we need to delete security groups as a admin | 19:29 |
boris-42 | stpierre: so this seems like not a hard to fix stuff | 19:29 |
stpierre | ok | 19:30 |
boris-42 | stpierre: here is the patch https://review.openstack.org/#/c/296402/2 | 19:31 |
stpierre | it looks like we set admin_required=True on the base NeutronMixin, but that isn't inherited | 19:31 |
boris-42 | stpierre: however it is not properly written | 19:31 |
boris-42 | stpierre: so actually it's a bit hard to fix bug | 19:32 |
boris-42 | stpierre: because if we are running with existing users and without admin (which will be possible in future) we shouldn't even try to delete default secruity group | 19:32 |
boris-42 | stpierre: seems like some refactoring will be required here | 19:33 |
stpierre | i dunno, lots of the cleanup resources require admin. it seems reasonable to just set admin_required on NeutronSecurityGroup for now. admittedly it's not perfectly forward-looking, but it should be a one-line fix for an actual bug, and it will be just one of a large list of things that require refactoring later. | 19:35 |
stpierre | i.e., we don't seem to lose anything by fixing it now | 19:35 |
boris-42 | stpierre: nope it is not | 19:36 |
boris-42 | stpierre: please let not make our life harder and harder | 19:37 |
stpierre | that doesn't make it any harder than it already is | 19:37 |
boris-42 | stpierre: each hack that we are adding is making rally less maintable | 19:37 |
boris-42 | stpierre: it makes it harder to make non-admin mode | 19:37 |
boris-42 | stpierre: I have an idea how to fix it | 19:37 |
stpierre | it fixes a real actual bug in one line, and there are twelve other places that already ahve admin_required | 19:37 |
boris-42 | stpierre: it creates another one | 19:38 |
stpierre | we can't just refuse to fix minor, simple bugs any time they conflict with some future idea that isn't even a spec yet | 19:38 |
boris-42 | stpierre: actually it creates 2 bugs | 19:38 |
boris-42 | 1) if you are using existing users, you will delete default security group of it (which is bad idea) | 19:39 |
boris-42 | 2) scenarios that don't require actually admin will start requiring it | 19:39 |
stpierre | that second one isn't a bug, because if they need to clean up security groups then they do actually require admin | 19:39 |
boris-42 | stpierre: sorry but I am already working 2 years for making non admin | 19:40 |
boris-42 | stpierre: and had to refactor whole Rally to achieve that | 19:40 |
boris-42 | stpierre: even plugin base was related to this task | 19:40 |
boris-42 | stpierre: and creating rally context* | 19:40 |
stpierre | i've gotta say, the amount of time we're spending spinning our wheels and not merging real fixes to actual bugs, or important features that are desperately needed, because of these distant pipe dream features is starting to get quite frustrating. | 19:40 |
boris-42 | stpierre: and current cleanups are trying to achieve this | 19:41 |
boris-42 | stpierre: so there is way to fix this | 19:41 |
boris-42 | stpierre: btw you don't need to be a admin to cleanup security groups | 19:41 |
boris-42 | stpierre: just for default security group | 19:41 |
stpierre | right, but if we want to do a complete job of cleanup, that's part of it. | 19:41 |
boris-42 | stpierre: so | 19:42 |
boris-42 | stpierre: there is another way | 19:42 |
stpierre | ok | 19:42 |
boris-42 | stpierre: btw | 19:42 |
boris-42 | stpierre: the code is rally | 19:43 |
boris-42 | stpierre: why this doesn't cleanup default security groups? | 19:43 |
boris-42 | https://github.com/openstack/rally/blob/master/rally/plugins/openstack/context/keystone/users.py#L124-L139 | 19:43 |
stpierre | that is an excellent question | 19:44 |
stpierre | i suspect that *is* cleaning up most of them, since we're only seeing a single default security group in the osresources output | 19:44 |
stpierre | so the question is where that one stray secgroup is coming from | 19:44 |
stpierre | which might have a much simpler answer | 19:44 |
boris-42 | stpierre: so | 19:47 |
boris-42 | stpierre: it's interesting stuff | 19:48 |
stpierre | looks like both of the 'default' security groups are for the 'service' tenant | 19:48 |
stpierre | which is created by devstack | 19:48 |
stpierre | so it should show up in the pre-rally inventory AFAICT | 19:48 |
-openstackstatus- NOTICE: Any jobs which erroneously failed on missing traceroute packages should be safe to recheck now | 19:55 | |
*** arnoldje has joined #openstack-rally | 19:56 | |
*** dratushnyy has quit IRC | 20:02 | |
boris-42 | stpierre: btw could you take a look at this patch https://review.openstack.org/#/c/299833/2 | 20:08 |
boris-42 | stpierre: it's like quite bad bug | 20:09 |
openstackgerrit | Merged openstack/rally: Cleanup: Fix getting name from neutron resources https://review.openstack.org/300121 | 20:16 |
*** moravec has quit IRC | 20:38 | |
openstackgerrit | Chris St. Pierre proposed openstack/rally: Test: Distinguish expected added resources https://review.openstack.org/300158 | 20:46 |
openstackgerrit | Chris St. Pierre proposed openstack/rally: Test: Fail jobs if resources are added or removed https://review.openstack.org/300159 | 20:46 |
openstackgerrit | Chris St. Pierre proposed openstack/rally: Test: don't stringify cloud resources until printing https://review.openstack.org/300157 | 20:46 |
*** harshs has quit IRC | 21:11 | |
*** stpierre has quit IRC | 21:21 | |
*** luzC has left #openstack-rally | 21:25 | |
*** dpaterson has quit IRC | 21:28 | |
*** harshs has joined #openstack-rally | 21:44 | |
*** MaxPC has quit IRC | 22:07 | |
openstackgerrit | Merged openstack/rally: Allow max_avg_sla per atomic actions https://review.openstack.org/298462 | 22:12 |
*** arnoldje has quit IRC | 22:38 | |
*** harshs has quit IRC | 23:20 | |
*** tosky has quit IRC | 23:32 | |
*** moravec has joined #openstack-rally | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!