*** salv-orlando has quit IRC | 00:03 | |
*** markvoelker has quit IRC | 00:04 | |
*** ArxCruz has quit IRC | 00:04 | |
*** tochi has quit IRC | 00:09 | |
*** tochi has joined #openstack-qa | 00:13 | |
*** hogepodge has quit IRC | 00:16 | |
*** ngoracke has quit IRC | 00:19 | |
*** hogepodge has joined #openstack-qa | 00:22 | |
*** nicholas_ has joined #openstack-qa | 00:23 | |
*** nicholas_ has quit IRC | 00:25 | |
*** hogepodge has quit IRC | 00:27 | |
*** r-mibu has joined #openstack-qa | 00:27 | |
*** nicholas_ has joined #openstack-qa | 00:29 | |
*** salv-orl_ has quit IRC | 00:36 | |
*** nicholas_ has quit IRC | 00:37 | |
*** ngoracke has joined #openstack-qa | 00:40 | |
*** hogepodge has joined #openstack-qa | 00:43 | |
*** dims has joined #openstack-qa | 00:43 | |
*** ngoracke has quit IRC | 00:44 | |
*** ngoracke has joined #openstack-qa | 00:49 | |
*** ngoracke has quit IRC | 00:53 | |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Properly display missing data in run graphs https://review.openstack.org/259646 | 00:55 |
---|---|---|
*** dims has quit IRC | 01:02 | |
*** ngoracke has joined #openstack-qa | 01:03 | |
*** markvoelker has joined #openstack-qa | 01:05 | |
*** hoangcx has joined #openstack-qa | 01:09 | |
*** hoangcx has left #openstack-qa | 01:09 | |
*** markvoelker has quit IRC | 01:10 | |
*** armax has quit IRC | 01:15 | |
*** ngoracke has quit IRC | 01:18 | |
*** ngoracke has joined #openstack-qa | 01:19 | |
*** ngoracke has quit IRC | 01:20 | |
openstackrecheck | Console logs not available after 13:23s for gate-cinder-python27 244328,11,68d7b3c | 01:32 |
*** salv-orlando has joined #openstack-qa | 01:37 | |
*** nicholas_ has joined #openstack-qa | 01:43 | |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Add logic to select the request microversion https://review.openstack.org/244439 | 01:46 |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Add set microversion in base compute service client https://review.openstack.org/258868 | 01:48 |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Add support of schema versioning for microversion https://review.openstack.org/244974 | 01:48 |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Add tests for Nova microversion v2.2 https://review.openstack.org/244996 | 01:48 |
*** panbalag has quit IRC | 01:52 | |
*** dims has joined #openstack-qa | 02:07 | |
*** jasonsb has joined #openstack-qa | 02:07 | |
*** jasonsb has quit IRC | 02:07 | |
*** jasonsb has joined #openstack-qa | 02:09 | |
*** nicholas_ has quit IRC | 02:10 | |
*** salv-orlando has quit IRC | 02:12 | |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Make volume volume_client to use **kwargs-part1 https://review.openstack.org/256271 | 02:16 |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Make volume volume_client to use **kwargs-part2 https://review.openstack.org/259249 | 02:16 |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Make volume snapshots_client to use **kwargs https://review.openstack.org/256227 | 02:21 |
r-mibu | gmann, ping | 02:22 |
gmann | r-mibu: hi | 02:23 |
r-mibu | hi | 02:23 |
r-mibu | could you review my ptach? | 02:23 |
r-mibu | https://review.openstack.org/#/c/255161/ | 02:23 |
r-mibu | i guess you already awer this | 02:23 |
r-mibu | s/awer/aware/ | 02:24 |
gmann | r-mibu: sure, i will check in afternoon | 02:24 |
r-mibu | thanks | 02:24 |
gmann | r-mibu: just question, which project you are trying the plugin? | 02:25 |
r-mibu | ceilometer | 02:25 |
r-mibu | and aodh | 02:25 |
openstackgerrit | ZongKai LI proposed openstack-dev/grenade: Upgrade all rootwrap filters for Neutron https://review.openstack.org/258758 | 02:25 |
r-mibu | aodh is alarming function split from ceilometer in liberty | 02:25 |
gmann | mtreinish: quick one for lib version - https://review.openstack.org/#/c/259269/ | 02:25 |
gmann | r-mibu: cool. do you have patches up for them or need to submit? | 02:26 |
gmann | r-mibu: i can have look in case anything needed there | 02:26 |
r-mibu | gmann: already in gerrit | 02:26 |
gmann | r-mibu: ok, great. i will check.Thanks | 02:26 |
r-mibu | link https://review.openstack.org/#/q/topic:bp/tempest-plugin,n,z | 02:27 |
r-mibu | gmann: thanks in advance! | 02:27 |
gmann | r-mibu: np! | 02:28 |
*** ngoracke has joined #openstack-qa | 02:31 | |
*** markvoelker has joined #openstack-qa | 02:36 | |
*** woodster_ has quit IRC | 02:36 | |
*** markvoelker has quit IRC | 02:40 | |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Update global-requirement for tempest-lib https://review.openstack.org/259269 | 02:47 |
*** dave-mccowan has quit IRC | 02:51 | |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Use floating_ips_client from tempest-lib https://review.openstack.org/259826 | 02:51 |
*** jasonsb has quit IRC | 03:01 | |
*** mfisher_ora has quit IRC | 03:04 | |
*** mfisher_ora has joined #openstack-qa | 03:04 | |
*** sree has joined #openstack-qa | 03:12 | |
*** nicholas_ has joined #openstack-qa | 03:13 | |
*** zz_dimtruck is now known as dimtruck | 03:13 | |
*** ngoracke has quit IRC | 03:14 | |
*** nicholas_ has quit IRC | 03:18 | |
*** ngoracke has joined #openstack-qa | 03:19 | |
*** coolsvap|away is now known as coolsvap | 03:23 | |
*** ngoracke has quit IRC | 03:26 | |
*** nicholas_ has joined #openstack-qa | 03:26 | |
*** ngoracke has joined #openstack-qa | 03:29 | |
*** nicholas_ has quit IRC | 03:31 | |
*** nicholas_ has joined #openstack-qa | 03:31 | |
*** dims has quit IRC | 03:32 | |
*** coolsvap is now known as coolsvap|away | 03:33 | |
*** ngoracke has quit IRC | 03:33 | |
*** ngoracke has joined #openstack-qa | 03:33 | |
*** nicholas_ has quit IRC | 03:35 | |
*** nicholas_ has joined #openstack-qa | 03:36 | |
*** ngoracke has quit IRC | 03:38 | |
*** sree_ has joined #openstack-qa | 03:40 | |
*** coolsvap|away is now known as coolsvap | 03:40 | |
*** sree_ is now known as Guest2822 | 03:40 | |
*** sree has quit IRC | 03:43 | |
*** ngoracke has joined #openstack-qa | 03:44 | |
*** nicholas_ has quit IRC | 03:44 | |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Use Tempest-lib's compute security_group_rules_client https://review.openstack.org/259833 | 03:48 |
*** ngoracke has quit IRC | 03:49 | |
*** tochi has quit IRC | 03:51 | |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Use Tempest-lib's compute server_groups_client https://review.openstack.org/259836 | 04:01 |
*** agireud has quit IRC | 04:03 | |
*** tochi has joined #openstack-qa | 04:05 | |
*** markvoelker has joined #openstack-qa | 04:37 | |
*** dimtruck is now known as zz_dimtruck | 04:40 | |
*** markvoelker has quit IRC | 04:41 | |
*** zz_dimtruck is now known as dimtruck | 04:46 | |
*** Poornima has joined #openstack-qa | 04:57 | |
*** markvoelker has joined #openstack-qa | 05:10 | |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Use servers_client from tempest-lib https://review.openstack.org/259841 | 05:21 |
*** amotoki has joined #openstack-qa | 05:21 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack-dev/devstack: Add FILTERS for customizing filters https://review.openstack.org/258882 | 05:29 |
*** shausy has joined #openstack-qa | 05:34 | |
*** gamado has joined #openstack-qa | 05:49 | |
*** gamado_ has joined #openstack-qa | 05:49 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack-infra/devstack-gate: Enable SameHostFilter and DifferentHostFilter https://review.openstack.org/259843 | 05:53 |
*** Guest2822 has quit IRC | 05:57 | |
*** Thelo has quit IRC | 06:05 | |
*** oomichi has joined #openstack-qa | 06:06 | |
*** yfried has quit IRC | 06:11 | |
*** rossella_s has quit IRC | 06:21 | |
*** rossella_s has joined #openstack-qa | 06:22 | |
*** itzikb has joined #openstack-qa | 06:27 | |
*** oomichi has quit IRC | 06:30 | |
*** SridharG has joined #openstack-qa | 06:31 | |
*** dimtruck is now known as zz_dimtruck | 06:32 | |
*** aopincar has joined #openstack-qa | 06:34 | |
*** anshul has joined #openstack-qa | 06:38 | |
*** felix_li has joined #openstack-qa | 06:42 | |
*** _nadya_ has joined #openstack-qa | 06:49 | |
*** oomichi has joined #openstack-qa | 06:53 | |
*** amotoki has quit IRC | 06:59 | |
openstackgerrit | Anusha Ramineni proposed openstack/tempest: create_server: pass arguments as part of kwargs https://review.openstack.org/259859 | 06:59 |
*** _nadya_ has quit IRC | 07:00 | |
openstackgerrit | FelixLi proposed openstack/tempest: Splitting clients into client groups in manager.py https://review.openstack.org/254160 | 07:01 |
*** anshul has quit IRC | 07:05 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Split out Neutron agents client https://review.openstack.org/257764 | 07:05 |
andreaf | mkoderer: I started working on moving clients data to a singleton, so that it can be loaded once, together with test discovery | 07:10 |
andreaf | mkoderer: however there some metadata I need for the client group, and I wonder where should I get that from | 07:10 |
andreaf | mkoderer: perhaps we could add a static dict to the __init__ of the client group module | 07:11 |
*** anshul has joined #openstack-qa | 07:20 | |
*** markvoelker has quit IRC | 07:21 | |
*** sree has joined #openstack-qa | 07:22 | |
andreaf | mkoderer: and I also wonder where that info should be static or discovered | 07:24 |
*** markvoelker has joined #openstack-qa | 07:25 | |
andreaf | mkoderer: the group name is easy, the list of service clients could be discovered I guess, and I still wonder whether it should be really modules as now, or strings with the name of modules - using a string would delay the import as well, but I'm not sure there's much value in that, unless service clients do wrong things in the class definition part | 07:27 |
*** _nadya_ has joined #openstack-qa | 07:28 | |
*** _nadya_ has quit IRC | 07:28 | |
*** gfidente has joined #openstack-qa | 07:29 | |
*** gfidente has quit IRC | 07:29 | |
*** gfidente has joined #openstack-qa | 07:29 | |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Make volume volume_client to use **kwargs-part1 https://review.openstack.org/256271 | 07:29 |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Make volume volume_client to use **kwargs-part2 https://review.openstack.org/259249 | 07:33 |
*** lezbar has joined #openstack-qa | 07:34 | |
gmann | andreaf: need to look into refactoring thing but 1 question | 07:35 |
gmann | andreaf: same instance of clients will be shared among tests? | 07:36 |
*** neeti has joined #openstack-qa | 07:37 | |
*** lezbar has quit IRC | 07:39 | |
dmellado | oomichi: aware of that + I need to quickly change some stuff too | 07:41 |
dmellado | will be submitting a PS within some minutes | 07:41 |
oomichi | dmellado: sorry, I cannot follow you. what mean? | 07:41 |
dmellado | on your -1 in the users_client | 07:41 |
dmellado | https://review.openstack.org/#/c/255385/ | 07:42 |
oomichi | dmellado: my latest review? | 07:42 |
dmellado | yep please don't merge until I upload the next PS will be also modifying a behavior of the class there | 07:42 |
*** lezbar__ has joined #openstack-qa | 07:43 | |
*** _nadya_ has joined #openstack-qa | 07:45 | |
oomichi | dmellado: ok, I am waiting for the next patch | 07:48 |
dmellado | thanks oomichi, + I'll also have to rebase the removal from botoclients | 07:48 |
*** anshul has quit IRC | 07:49 | |
*** gamado has quit IRC | 07:50 | |
*** anshul has joined #openstack-qa | 08:02 | |
openstackgerrit | Anusha Ramineni proposed openstack/tempest: create_server: pass arguments as part of kwargs https://review.openstack.org/259859 | 08:05 |
openstackgerrit | Benny Kopilov proposed openstack/tempest: This flag was not supported by ice-house . https://review.openstack.org/244057 | 08:11 |
*** salv-orlando has joined #openstack-qa | 08:11 | |
openstackgerrit | Benny Kopilov proposed openstack/tempest: Config: change cinder bootable flag to True. https://review.openstack.org/244057 | 08:13 |
andreaf | gmann: no, the instancee of clients is different | 08:13 |
andreaf | gmann: the client manager stays one instance per set of credentials | 08:13 |
andreaf | gmann: so if a test uses 3 credentials, there are three client manager instances | 08:14 |
*** salv-orlando has quit IRC | 08:14 | |
andreaf | gmann: but the initial list of available clients, we discussed with mkoderer that it should be a singleton | 08:14 |
*** ihrachys has joined #openstack-qa | 08:14 | |
*** hoangcx has joined #openstack-qa | 08:15 | |
andreaf | gmann: because that list may include clients coming from plugins - so we want to generate that list once and then re-use it in all instances of client managers | 08:15 |
*** hoangcx has left #openstack-qa | 08:15 | |
andreaf | gmann: I'm reworking the manager refactor to include support for this | 08:15 |
gmann | andreaf: initial list you mean just static data(client name etc) but those will be instantiated separately per each Manager right? | 08:16 |
andreaf | gmann: right | 08:16 |
gmann | andreaf: ok. | 08:17 |
gmann | andreaf: i will check by tomorrow | 08:17 |
openstackgerrit | Daniel Mellado proposed openstack/tempest: Split users_client from keystone client v2. https://review.openstack.org/255385 | 08:17 |
gmann | andreaf: btw i updated microversion patches - https://review.openstack.org/#/q/topic:bp/api-microversions-testing-support+status:open | 08:18 |
andreaf | gmann: cool thanks - anyways I'm doing some rework now, so I'll remove the WIP when it's ready | 08:18 |
andreaf | gmann: thanks I will review them | 08:18 |
gmann | andreaf: sure | 08:18 |
gmann | andreaf: Thanks | 08:18 |
gmann | andreaf: you are putting all under topic "manager_split" right? | 08:19 |
gmann | andreaf: with refactor spec update also | 08:19 |
andreaf | gmann: yes | 08:19 |
gmann | andreaf: ok | 08:20 |
*** grafuls has joined #openstack-qa | 08:28 | |
felix_li | #openstack-nova | 08:29 |
*** shardy has joined #openstack-qa | 08:37 | |
*** bochi-michael has joined #openstack-qa | 08:42 | |
*** jacky-zhang has joined #openstack-qa | 08:43 | |
jacky-zhang | hi bochi-michael | 08:44 |
bochi-michael | jacky-zhang: hi | 08:45 |
*** ifarkas has joined #openstack-qa | 08:45 | |
openstackgerrit | Marian Horban proposed openstack/tempest-lib: Added processing /compute URL for getting list of versions https://review.openstack.org/245815 | 08:54 |
openstackgerrit | Merged openstack/tempest: 'List volume type' comment has fixed https://review.openstack.org/258114 | 08:54 |
openstackgerrit | Merged openstack/tempest: Drop lock_path section from config guide https://review.openstack.org/177978 | 08:55 |
*** asselin__ has joined #openstack-qa | 08:55 | |
*** asselin_ has quit IRC | 08:57 | |
*** tochi has quit IRC | 09:02 | |
*** yamamoto has joined #openstack-qa | 09:05 | |
*** safchain has joined #openstack-qa | 09:12 | |
*** drptbl has joined #openstack-qa | 09:13 | |
*** drptbl has quit IRC | 09:13 | |
*** drptbl has joined #openstack-qa | 09:18 | |
*** felix_li has quit IRC | 09:18 | |
*** maht has joined #openstack-qa | 09:19 | |
mkoderer | andreaf: hi | 09:22 |
mkoderer | andreaf: I think module name should be fine | 09:23 |
mkoderer | andreaf: what other metadata do you have in mind | 09:23 |
andreaf | mkoderer: I'm changing things a bit to move aliases to the consumer side, and have require client group wide unique names only | 09:24 |
andreaf | mkoderer: because we cannot ask a plugin to define a client alias that is universally unique | 09:24 |
andreaf | mkoderer: but if I use names like manilla_shares_client, it's enough for shares_client to be unique within manila | 09:25 |
*** markvoelker has quit IRC | 09:25 | |
andreaf | mkoderer: afk for 10min - sorry | 09:25 |
mkoderer | andreaf: I guess we can asume that plugin_name + client_name is unique enought | 09:27 |
mkoderer | andreaf: no worries.. my calender is also packed today :) | 09:27 |
openstackgerrit | Marc Koderer proposed openstack/tempest: WIP: Introduce data driven testing https://review.openstack.org/223953 | 09:28 |
andreaf | mkoderer: yes that was my assumption | 09:31 |
andreaf | mkoderer: the other bit of metadata I need is which parameters are required to initialise clients in the client group (or plugin if you will) | 09:32 |
andreaf | mkoderer: my assumption is that there is a set of parameters that are common to the entire group, and some parameters that are specific to one client | 09:33 |
mkoderer | andreaf: so you just give the chance to add some additional kwargs for each client | 09:34 |
*** salv-orlando has joined #openstack-qa | 09:34 | |
mkoderer | andreaf: I guess we should add a function here: https://github.com/openstack/tempest/blob/master/tempest/test_discover/plugins.py#L27 | 09:35 |
andreaf | mkoderer: like in compute the servers_client has an extra attribute enable_instance_password | 09:35 |
mkoderer | andreaf: to have a hook for the plugins to call your new class in order to register the client | 09:36 |
*** yamamoto has quit IRC | 09:36 | |
mkoderer | andreaf: yep that's fine | 09:37 |
andreaf | mkoderer: oh, right I guess we could do it like that, I can have two register functions anyways, one for internal modules and one for plugins | 09:37 |
*** jacky-zhang has quit IRC | 09:38 | |
*** ansiwen has joined #openstack-qa | 09:38 | |
*** ansiwen has quit IRC | 09:38 | |
*** ansiwen has joined #openstack-qa | 09:39 | |
*** openstack has joined #openstack-qa | 15:35 | |
*** lucas-hungry is now known as lucasagomes | 15:35 | |
*** openstackstatus has joined #openstack-qa | 15:36 | |
*** ChanServ sets mode: +v openstackstatus | 15:36 | |
*** sree has joined #openstack-qa | 15:40 | |
openstackrecheck | Console logs not available after 13:22s for gate-devstack-plugin-ceph-bashate 258602,5,7a1bedd | 15:43 |
*** aopincar has quit IRC | 15:45 | |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Fix mean runtime value on jobs page https://review.openstack.org/256112 | 15:47 |
*** yamamoto has joined #openstack-qa | 15:50 | |
*** dave-mccowan has quit IRC | 15:53 | |
*** yamamoto has quit IRC | 15:54 | |
*** mtanino has joined #openstack-qa | 15:54 | |
*** yamamoto has joined #openstack-qa | 15:54 | |
*** artom has joined #openstack-qa | 15:55 | |
openstackrecheck | Console logs not available after 13:24s for gate-horizon-npm-run-test 249981,4,f6bc5c6 | 15:56 |
*** vgridnev has quit IRC | 15:56 | |
openstackrecheck | openstack/requirements change: https://review.openstack.org/259741 failed gate-grenade-dsvm-multinode in the gate queue with an unrecognized error | 15:57 |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Add skips data to the jobs page https://review.openstack.org/256113 | 15:58 |
*** vgridnev has joined #openstack-qa | 16:00 | |
*** yamamoto has quit IRC | 16:02 | |
*** yamamoto has joined #openstack-qa | 16:02 | |
*** yamamoto has quit IRC | 16:03 | |
*** itzikb has quit IRC | 16:04 | |
*** vgridnev has quit IRC | 16:05 | |
*** dave-mccowan has joined #openstack-qa | 16:07 | |
openstackgerrit | Andrea Frittoli proposed openstack/tempest: Split the client manager in two parts. https://review.openstack.org/258566 | 16:10 |
openstackgerrit | Andrea Frittoli proposed openstack/tempest: Add a service client data store https://review.openstack.org/260076 | 16:10 |
openstackgerrit | Andrea Frittoli proposed openstack/tempest: Migrate compute client group to the new manager https://review.openstack.org/260077 | 16:10 |
*** sree has quit IRC | 16:12 | |
*** sree has joined #openstack-qa | 16:12 | |
*** _hop_ has quit IRC | 16:15 | |
*** sree has quit IRC | 16:17 | |
*** lutostag has quit IRC | 16:18 | |
*** lutostag has joined #openstack-qa | 16:21 | |
openstackgerrit | Dean Troyer proposed openstack-dev/devstack: docs: Add additional configuration in Neutron guide https://review.openstack.org/237780 | 16:26 |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Add skips data to the jobs page https://review.openstack.org/256113 | 16:27 |
*** stevemar_znc is now known as stevemar | 16:28 | |
*** ccneill has joined #openstack-qa | 16:30 | |
*** ifarkas has quit IRC | 16:34 | |
*** yamamoto has joined #openstack-qa | 16:34 | |
*** yamamoto has quit IRC | 16:38 | |
*** markvoelker has joined #openstack-qa | 16:43 | |
*** dimtruck is now known as zz_dimtruck | 16:45 | |
*** maht has quit IRC | 16:45 | |
mtreinish | sc68cal: on: https://bugs.launchpad.net/tempest/+bug/1498981 is that really a tempest bug? | 16:46 |
openstack | Launchpad bug 1498981 in tempest "Tempest connects using 127.0.0.1 when SERVICE_IP_VERSION=6" [Undecided,New] - Assigned to Shuquan Huang (shuquan) | 16:46 |
mtreinish | it looks like you want to change how things get configured when ipv6 is set in d-g | 16:46 |
*** markvoelker has quit IRC | 16:48 | |
*** vgridnev has joined #openstack-qa | 16:50 | |
*** ihrachys has quit IRC | 16:52 | |
*** grafuls has quit IRC | 16:54 | |
*** armax has joined #openstack-qa | 16:55 | |
*** cdent has quit IRC | 17:00 | |
*** zackf is now known as jrcloud | 17:02 | |
*** jrcloud is now known as zackf | 17:02 | |
openstackgerrit | Vasyl Saienko proposed openstack-dev/devstack: Introduce iniget_sections function https://review.openstack.org/256294 | 17:03 |
*** _nadya_ has quit IRC | 17:06 | |
*** devananda has quit IRC | 17:08 | |
*** yfried_ has quit IRC | 17:10 | |
*** devananda has joined #openstack-qa | 17:15 | |
openstackgerrit | Randall Burt proposed openstack-infra/elastic-recheck: Add query for bug 1527483 https://review.openstack.org/260119 | 17:15 |
openstack | bug 1527483 in neutron "VPNaaS - No providers specified for 'VPN' service" [Critical,Confirmed] https://launchpad.net/bugs/1527483 - Assigned to Martin Hickey (martin-hickey) | 17:15 |
*** drptbl has quit IRC | 17:16 | |
*** woodster_ has quit IRC | 17:16 | |
*** maht has joined #openstack-qa | 17:18 | |
*** _nadya_ has joined #openstack-qa | 17:20 | |
openstackgerrit | Dmitry Tantsur proposed openstack/tempest: Make rebuild test optional in ironic https://review.openstack.org/260121 | 17:24 |
dtantsur | jroll, ^^ | 17:24 |
*** gszasz has quit IRC | 17:25 | |
jroll | whee, thanks | 17:25 |
*** maht has quit IRC | 17:28 | |
sc68cal | mtreinish: no, it's a devstack-gate bug | 17:31 |
openstackgerrit | Sean M. Collins proposed openstack-infra/devstack-gate: Stop hard coding SERVICE_HOST https://review.openstack.org/229111 | 17:32 |
sc68cal | mtreinish: sorry, when I pushed a patch to fix I was dumb and didn't link to the bug in the commit message. | 17:34 |
mtreinish | sc68cal: heh, no worries. jeepyb probably won't pick it up anyway because d-g doesn't map back to tempest | 17:40 |
*** markvoelker has joined #openstack-qa | 17:44 | |
*** zz_dimtruck is now known as dimtruck | 17:46 | |
*** mhickey has quit IRC | 17:47 | |
*** e0ne has quit IRC | 17:49 | |
*** markvoelker has quit IRC | 17:49 | |
openstackgerrit | Dmitry Tantsur proposed openstack/tempest: Stop testing rebuild for ironic https://review.openstack.org/260121 | 17:51 |
dtantsur | jroll, mtreinish, I decided it's easier to remove the rebuild test for now ^^ and bring it back after we have an in-tree plugin | 17:51 |
jroll | ++ | 17:51 |
*** cdent has joined #openstack-qa | 17:55 | |
*** cdent has quit IRC | 17:58 | |
*** aix has quit IRC | 17:58 | |
*** cdent has joined #openstack-qa | 17:59 | |
*** Swami has joined #openstack-qa | 18:00 | |
*** salv-orl_ has joined #openstack-qa | 18:00 | |
*** maht has joined #openstack-qa | 18:00 | |
*** drptbl has joined #openstack-qa | 18:01 | |
*** salv-orlando has quit IRC | 18:03 | |
*** dtantsur is now known as dtantsur|afk | 18:06 | |
*** yamahata has joined #openstack-qa | 18:08 | |
openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/211808 failed gate-devstack-bashate in the gate queue with an unrecognized error | 18:08 |
openstackgerrit | Merged openstack/openstack-health: Cut the date window for test pagess in half https://review.openstack.org/256742 | 18:09 |
*** drptbl has quit IRC | 18:12 | |
*** gfidente is now known as gfidente|afk | 18:14 | |
*** Guest95009 is now known as jgriffith | 18:15 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/tempest: Updated from global requirements https://review.openstack.org/256534 | 18:15 |
*** yamahata has quit IRC | 18:16 | |
*** yamahata has joined #openstack-qa | 18:16 | |
*** hemna__ is now known as hemna | 18:19 | |
*** rossella_s has quit IRC | 18:21 | |
*** electrichead is now known as redrobot | 18:21 | |
*** jrcloud has joined #openstack-qa | 18:21 | |
*** rossella_s has joined #openstack-qa | 18:22 | |
*** zackf has quit IRC | 18:24 | |
*** woodster_ has joined #openstack-qa | 18:30 | |
*** jrcloud is now known as zackf | 18:31 | |
openstackrecheck | openstack/keystone change: https://review.openstack.org/230181 failed gate-tempest-dsvm-neutron-full in the gate queue with an unrecognized error | 18:33 |
openstackgerrit | Yaroslav Lobankov proposed openstack/tempest: Fixing two tests when nova-net is enabled and run_validation = True https://review.openstack.org/260154 | 18:34 |
sc68cal | mtreinish: ack. Yeah I need to change the project the bug targets | 18:34 |
*** e0ne has joined #openstack-qa | 18:34 | |
*** yamamoto has joined #openstack-qa | 18:37 | |
openstackgerrit | Yaroslav Lobankov proposed openstack/tempest: [V3] Renaming region_client into regions_client https://review.openstack.org/259331 | 18:39 |
*** yamamoto has quit IRC | 18:42 | |
ylobankov | mtreinish: Hi. Could you please take a look at https://review.openstack.org/#/c/260154/ ? It looks like your changes broke two tests :) | 18:43 |
mtreinish | ylobankov: heh, I could have sworn that we tested it before we merged it | 18:45 |
mtreinish | oh, the job with validation turned on is neutron only | 18:45 |
*** zackf has quit IRC | 18:53 | |
*** zackf has joined #openstack-qa | 18:55 | |
*** shardy is now known as shardy_afk | 18:58 | |
ylobankov | mtreinish: yep. It is only for Neutron | 18:59 |
*** pvaneck has joined #openstack-qa | 19:01 | |
*** safchain has quit IRC | 19:02 | |
*** ccneill has quit IRC | 19:03 | |
*** weshay_pto has quit IRC | 19:05 | |
*** agireud has joined #openstack-qa | 19:09 | |
*** ccneill has joined #openstack-qa | 19:15 | |
*** _nadya_ has quit IRC | 19:16 | |
dims_ | sc68cal : ianw : can one of you please take a look at https://review.openstack.org/#/c/260009/ (tox 2.3.1 breaking gate) | 19:16 |
sc68cal | dims_: cool | 19:17 |
sc68cal | dims_: I can +W that | 19:17 |
dims_ | sc68cal : thanks sean! | 19:18 |
sc68cal | dims_: thanks for fixing | 19:18 |
dims_ | i should thank for the quick review :) | 19:19 |
*** moravec1 has quit IRC | 19:31 | |
*** moravec has joined #openstack-qa | 19:34 | |
*** maht has quit IRC | 19:34 | |
*** diegows has quit IRC | 19:38 | |
openstackgerrit | Ramy Asselin proposed openstack-infra/elastic-recheck: Include query results in graph https://review.openstack.org/260188 | 19:42 |
asselin | mtreinish, another interesting one ^^ I think it'll need some UI refinement, but otherwise should be useful | 19:43 |
*** markvoelker has joined #openstack-qa | 19:45 | |
*** markvoelker has quit IRC | 19:50 | |
*** diegows has joined #openstack-qa | 19:50 | |
mriedem | mtreinish: i'm doing my part with the deprecation warnings... https://review.openstack.org/#/c/260186/ | 19:51 |
mriedem | i'll see you at the manila mid-cycle now that i'm an ATC | 19:52 |
*** diegows has quit IRC | 19:52 | |
*** lucasagomes is now known as lucas-dinner | 19:53 | |
*** lezbar has joined #openstack-qa | 19:56 | |
mtreinish | mriedem: heh, Im not going to the manila midcycle | 19:58 |
*** lezbar__ has quit IRC | 19:58 | |
mkoderer | mtreinish: if you have some secs for reviews: https://review.openstack.org/#/c/258873/, https://review.openstack.org/#/c/259321/, https://review.openstack.org/#/c/259925/ | 19:59 |
mkoderer | just some small ones :) | 20:00 |
mkoderer | mtreinish: ah and I added a spec about data driven testing | 20:00 |
*** lezbar has quit IRC | 20:00 | |
mtreinish | mkoderer: sure, I'll take a look | 20:01 |
mkoderer | thx | 20:02 |
mtreinish | mkoderer: also, if you get a sec: https://review.openstack.org/#/c/216303/4 is something thats been sitting for a while | 20:02 |
mtreinish | and related to an open bp of yours from forever ago :) | 20:02 |
openstackgerrit | Ramy Asselin proposed openstack-infra/elastic-recheck: Include query results in graph https://review.openstack.org/260188 | 20:06 |
*** hemna is now known as hemnalunch | 20:08 | |
*** moravec has quit IRC | 20:10 | |
*** dkranz has quit IRC | 20:15 | |
*** Swami has quit IRC | 20:16 | |
*** _nadya_ has joined #openstack-qa | 20:21 | |
*** gamado_ has quit IRC | 20:22 | |
*** artom has quit IRC | 20:22 | |
*** dkranz has joined #openstack-qa | 20:23 | |
*** _nadya_ has quit IRC | 20:25 | |
*** e0ne has quit IRC | 20:25 | |
*** moravec has joined #openstack-qa | 20:29 | |
*** e0ne has joined #openstack-qa | 20:29 | |
*** e0ne has quit IRC | 20:34 | |
*** _nadya_ has joined #openstack-qa | 20:37 | |
*** e0ne has joined #openstack-qa | 20:38 | |
*** zackf has quit IRC | 20:41 | |
*** e0ne has quit IRC | 20:45 | |
*** _nadya_ has quit IRC | 20:46 | |
*** e0ne has joined #openstack-qa | 20:50 | |
*** hichihara has joined #openstack-qa | 20:51 | |
mriedem | oh wow the manila tests are not fast http://logs.openstack.org/86/260186/1/check/gate-manila-tempest-dsvm-neutron-scenario-multibackend/b942989/console.html#_2015-12-21_20_15_28_537 | 20:55 |
mtreinish | mriedem: heh, yeah those are slow | 20:56 |
*** moravec has quit IRC | 20:59 | |
*** moravec has joined #openstack-qa | 21:03 | |
*** hemnalunch is now known as hemna | 21:06 | |
*** ansiwen has quit IRC | 21:06 | |
*** ansiwen has joined #openstack-qa | 21:08 | |
ianw | sc68cal: i think it's time to move that into a separate script | 21:08 |
sc68cal | ianw: move what into a separate script | 21:09 |
ianw | sc68cal: sorry, just reading scrollback - the bashate invocation | 21:09 |
*** weshay_xchat has joined #openstack-qa | 21:09 | |
sc68cal | ianw: ah. yeah probably. you'd get my +2 on that, but I didn't want the good to be the enemy of the perfect | 21:10 |
*** nicholasgoracke has quit IRC | 21:11 | |
*** Swami has joined #openstack-qa | 21:12 | |
openstackgerrit | Merged openstack/tempest: Fix telemetry tests to use v1 create image correctly https://review.openstack.org/258271 | 21:17 |
*** shardy_afk is now known as shardy | 21:19 | |
*** nicholasgoracke has joined #openstack-qa | 21:21 | |
*** bnemec has quit IRC | 21:26 | |
*** bnemec has joined #openstack-qa | 21:28 | |
openstackgerrit | Merged openstack-dev/devstack: Do not fail with Tox 2.3.1 https://review.openstack.org/260009 | 21:30 |
*** markvoelker has joined #openstack-qa | 21:30 | |
openstackgerrit | Merged openstack/tempest: Add description for stress in tempest cli https://review.openstack.org/258873 | 21:31 |
*** dims_ has quit IRC | 21:33 | |
*** dims has joined #openstack-qa | 21:33 | |
*** markvoelker has quit IRC | 21:36 | |
*** vgridnev has quit IRC | 21:42 | |
*** dims has quit IRC | 21:43 | |
sc68cal | mtreinish: ianw: can you guys weigh in on https://review.openstack.org/#/c/259385/ ? | 21:47 |
sc68cal | the issue is we have a couple jobs that want to use the q-vpn service work in DevStack, which is currently a problem due to https://launchpad.net/bugs/1527483 | 21:48 |
openstack | Launchpad bug 1527483 in neutron "VPNaaS - No providers specified for 'VPN' service" [Critical,Confirmed] - Assigned to Martin Hickey (martin-hickey) | 21:48 |
sc68cal | it's basically heat and sahara | 21:48 |
sc68cal | http://paste.openstack.org/show/482455/ | 21:48 |
*** zackf has joined #openstack-qa | 21:48 | |
sc68cal | I'd rather see them move to using the devstack plugin for neutron-vpnaas | 21:48 |
*** dims has joined #openstack-qa | 21:49 | |
openstackgerrit | Ramy Asselin proposed openstack-infra/elastic-recheck: Include query results in graph https://review.openstack.org/260188 | 21:51 |
ianw | sc68cal: i think while we've got q-vpn in there, we need to make sure it works | 21:51 |
ianw | looks like it's just fall-out from that autogen patch? | 21:51 |
sc68cal | ianw: yep | 21:52 |
sc68cal | my concern is we're already broken at the gate. so do we take the time to fix it by having them use the devstack plugin for vpnaas, or do we do it in a follow up | 21:53 |
sc68cal | or just do a temporary fix | 21:53 |
sc68cal | my thinking was take the pain now and fix it, but it is the gate so maybe that's not reasonable | 21:54 |
sc68cal | *fix it - meaning have those jobs start using the plugin | 21:54 |
sc68cal | ianw: changed my vote and +A'd it | 21:56 |
openstackgerrit | Merged openstack-infra/elastic-recheck: Add query for bug 1527483 https://review.openstack.org/260119 | 21:57 |
openstack | bug 1527483 in neutron "VPNaaS - No providers specified for 'VPN' service" [Critical,Confirmed] https://launchpad.net/bugs/1527483 - Assigned to Martin Hickey (martin-hickey) | 21:57 |
ianw | sc68cal: i wouldn't take much to retire it, but we should at least try and do it gracefully, especially when devstack kind of broke it unintentionally | 21:58 |
sc68cal | ianw: ack. I'll assign myself a bug for it since I opined about it | 21:59 |
*** eil397 has joined #openstack-qa | 22:05 | |
*** cdent has quit IRC | 22:06 | |
*** dims_ has joined #openstack-qa | 22:08 | |
*** openstackrecheck has quit IRC | 22:08 | |
*** dims has quit IRC | 22:08 | |
*** openstackrecheck has joined #openstack-qa | 22:08 | |
*** shardy has quit IRC | 22:12 | |
*** dims has joined #openstack-qa | 22:12 | |
*** dims_ has quit IRC | 22:13 | |
jbernard | mriedem: for the nova/ceph tempest failures | 22:15 |
mriedem | like which ones? | 22:15 |
mriedem | the weighter thing? | 22:15 |
jbernard | mriedem: this one for instance http://logs.openstack.org/14/258314/2/check/gate-tempest-dsvm-full-ceph/1222c6a/ | 22:15 |
jbernard | mriedem: that one is ebs_image_and_check_boot | 22:15 |
jbernard | mriedem: which fails on occasion | 22:15 |
jbernard | mriedem: i believe this occurs because https://github.com/openstack/tempest/blob/master/tempest/scenario/test_volume_boot_pattern.py#L167 | 22:16 |
mriedem | jbernard: that's a stable/liberty change | 22:16 |
mriedem | jbernard: let me introduce you to https://review.openstack.org/#/c/258695/ | 22:16 |
jbernard | mriedem: it is, but it also occurs on master | 22:16 |
mriedem | it should be less frequent on master | 22:16 |
mriedem | since https://review.openstack.org/#/c/258118/ | 22:16 |
jbernard | mriedem: it is better, i just wanted to make sure my findings today are the same as what you're seeing | 22:16 |
mriedem | logstash also shows it dropped off | 22:17 |
mriedem | http://status.openstack.org/elastic-recheck/index.html#1464259 | 22:17 |
*** dkranz has quit IRC | 22:17 | |
mriedem | nova still has problems in this code (races i mean), that's in the ML | 22:17 |
mriedem | and on my todo list | 22:17 |
jbernard | interesting | 22:17 |
mriedem | namely, nova isn't waiting for detach to complete before it tries to delete the volume | 22:17 |
jbernard | i still see it every once and awhile on my local env | 22:18 |
mriedem | but that wouldn't cause that ebs test to fail | 22:18 |
mriedem | it's just a problem with nova orphaning volumes | 22:18 |
jbernard | so at https://github.com/openstack/tempest/blob/master/tempest/scenario/test_volume_boot_pattern.py#L167 | 22:18 |
jbernard | i believe the delete occurs while the instance is being created | 22:18 |
mriedem | there is definitely still a race in the test | 22:18 |
mriedem | yes, | 22:18 |
jbernard | and a voluem is left orphaned | 22:18 |
mriedem | nova is trying to delete the volume at the same time that tempest is telling cinder to delete the snapshot | 22:18 |
mriedem | if nova gets that done fast enough, great | 22:19 |
jbernard | ok, so this is not a different issue | 22:19 |
mriedem | no, there is still a chance for a race here | 22:19 |
jbernard | ok, that's what i wanted to check | 22:19 |
mriedem | but from logstash above, it looks like things are at least calming down on master | 22:19 |
jbernard | i think we're getting lucky there, but ill take it ;) | 22:20 |
mriedem | now if nova starts waiting for volume detach to complete before doing volume delete, we might have problems | 22:20 |
mriedem | yeah, me too | 22:20 |
jbernard | mriedem: wrt liberty, is there anything i can do to help there? | 22:20 |
mriedem | to remove the race, tempest would really have to add a waiter on the volume being deleted before deleting the snapshot | 22:20 |
mriedem | jbernard: just review the backport | 22:20 |
jbernard | this one: https://review.openstack.org/#/c/258695/ | 22:20 |
jbernard | mriedem: ok, will do | 22:21 |
jbernard | mriedem: thanks | 22:21 |
openstackgerrit | Tim Buckley proposed openstack/openstack-health: Add global resolution, grouping key, and filter options. https://review.openstack.org/238563 | 22:21 |
mriedem | i plan on looking at the orphaned volumes issue after the holidays | 22:22 |
*** dims has quit IRC | 22:29 | |
*** mriedem is now known as mriedem_away | 22:30 | |
*** mfisher_ora has quit IRC | 22:30 | |
*** zackf has quit IRC | 22:34 | |
*** e0ne has quit IRC | 22:35 | |
*** dimtruck is now known as zz_dimtruck | 22:42 | |
*** markvoelker has joined #openstack-qa | 22:46 | |
openstackgerrit | Merged openstack/openstack-health: Fix mean runtime value on jobs page https://review.openstack.org/256112 | 22:46 |
*** moravec has quit IRC | 22:48 | |
mtreinish | timothyb89, jhesketh: another fast o-h patch: https://review.openstack.org/#/c/256113/ to add the skip data to the jobs page | 22:49 |
timothyb89 | mtreinish: right, was just looking at that one | 22:49 |
mtreinish | timothyb89: cool | 22:49 |
*** nicholasgoracke has quit IRC | 22:49 | |
mtreinish | my only concern with it whether violet was a good color choice for the 3rd line | 22:50 |
mtreinish | it's what I used on the stacked area graph on the per test page | 22:50 |
mtreinish | but lines are a little different | 22:50 |
*** pvaneck has quit IRC | 22:51 | |
*** markvoelker has quit IRC | 22:51 | |
*** nicholasgoracke has joined #openstack-qa | 22:52 | |
mtreinish | mriedem_away: so we just dropped the boto tests from tempest. Would you prefer a tag as the commit right before the removal or the removal patch? | 22:53 |
timothyb89 | mtreinish: that color seems fine to me | 22:57 |
*** ajmiller has quit IRC | 22:58 | |
*** ajmiller_ has joined #openstack-qa | 22:58 | |
*** mtanino has quit IRC | 23:03 | |
*** woodster_ has quit IRC | 23:06 | |
*** moravec has joined #openstack-qa | 23:09 | |
*** nicholasgoracke has quit IRC | 23:19 | |
*** dims has joined #openstack-qa | 23:21 | |
*** asselin_ has joined #openstack-qa | 23:26 | |
*** asselin__ has quit IRC | 23:27 | |
jhesketh | mtreinish: lookgin | 23:28 |
tonyb | mtreinish: ping? | 23:29 |
*** dims has quit IRC | 23:30 | |
mtreinish | tonyb: umm, is that a question? | 23:30 |
tonyb | mtreinish: yes it's "are you available?" | 23:31 |
mtreinish | heh, ok | 23:31 |
tonyb | mtreinish: http://git.openstack.org/cgit/openstack/python-barbicanclient/tree/functionaltests/run_tests.sh?h=stable/kilo | 23:31 |
*** ccneill has quit IRC | 23:31 | |
tonyb | what is the *correct* way to run tempest in a venv so that I don't blow up in stable/kilo | 23:31 |
mtreinish | oh, fun they use nose | 23:31 |
mtreinish | that won't work | 23:31 |
tonyb | http://logs.openstack.org/98/201198/4/check/gate-python-barbicanclient-devstack-dsvm-oldstable/b7a89b0/console.html#_2015-12-21_22_19_12_234 | 23:32 |
mtreinish | well it might for their hacked together plugin tests, but tempest doesn't | 23:32 |
*** sabeen has joined #openstack-qa | 23:32 | |
*** krtaylor has quit IRC | 23:32 | |
mtreinish | tonyb: thats because they're using oslo config >=2.0 with old code | 23:32 |
*** dims has joined #openstack-qa | 23:32 | |
mtreinish | post namespace removal | 23:32 |
tonyb | mtreinish: right g-r caps it correctly when you install barbicanclient but then they install tempest outside a venv and get the master caps | 23:33 |
mtreinish | tonyb: tempest in a venv isn't any different than anything else. In the gate we use tox to do it, just for convience | 23:34 |
tonyb | mtreinish: okay. I'll run up a kilo devstack to workout why they do what they're doing. | 23:36 |
gmann | mtreinish: Thanks for reviewing service client patches. | 23:36 |
gmann | mtreinish: microversion one are ready to review -https://review.openstack.org/#/q/topic:bp/api-microversions-testing-support+status:open | 23:36 |
openstackgerrit | Merged openstack-dev/devstack: Neutron VPNaaS: Set default service provider https://review.openstack.org/259385 | 23:36 |
mtreinish | tonyb: I'm kinda confused by what those tests are importing tempest for. It looks like they're reusing the config file from tempest | 23:36 |
gmann | mtreinish: have a look in your appropriate time | 23:36 |
mtreinish | but so far I haven't found any options theyre using except for the auth url and keystone version | 23:37 |
openstackgerrit | Merged openstack/tempest: Update global-requirement for tempest-lib https://review.openstack.org/259269 | 23:37 |
mtreinish | which seems like a pretty poor reason to use the config file from tempest (which has a couple hundred options iirc) | 23:38 |
tonyb | mtreinish: yeah I have to admit I'm confused. but that just means I have an oppotunity to learn something so that's good. | 23:40 |
mtreinish | tonyb: well, I mean that assumes there's something to learn, instead of just unwrapping a tangled ball of yarn | 23:41 |
tonyb | mtreinish: yeah ok. fair call. | 23:43 |
mtreinish | mordred: just a friendly reminder I did eventually review: https://review.openstack.org/#/c/236715 | 23:44 |
mtreinish | it might have gotten lost in the noise (my slow turnaround didn't help) | 23:44 |
mtreinish | gmann: ok, I'll try to take a look | 23:44 |
*** krtaylor has joined #openstack-qa | 23:44 | |
gmann | mtreinish: Thanks | 23:45 |
*** zz_dimtruck is now known as dimtruck | 23:46 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/tempest: Updated from global requirements https://review.openstack.org/256534 | 23:46 |
*** tosky has quit IRC | 23:48 | |
*** jecarey has quit IRC | 23:55 | |
*** dimtruck is now known as zz_dimtruck | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!