| *** evgenyf has joined #openstack-qa | 00:01 | |
| mtreinish | mkoderer: ^^^ sorry it looks like that bounced one of your patches... | 00:01 |
|---|---|---|
| *** rhsu has joined #openstack-qa | 00:03 | |
| *** cpallares has quit IRC | 00:03 | |
| openstackgerrit | A change was merged to openstack/tempest: Enable E128 ignore E129 https://review.openstack.org/98909 | 00:03 |
| *** cpallares has joined #openstack-qa | 00:04 | |
| *** yamahata has joined #openstack-qa | 00:05 | |
| *** yamahata has quit IRC | 00:05 | |
| *** yamahata has joined #openstack-qa | 00:05 | |
| openstackgerrit | Lakshmi N Sampath proposed a change to openstack/tempest: Glance Metadata Definitions Catalog - Tempest tests https://review.openstack.org/113632 | 00:15 |
| *** aimon has quit IRC | 00:19 | |
| *** atiwari has quit IRC | 00:19 | |
| *** cpallares has quit IRC | 00:26 | |
| openstackgerrit | Lakshmi N Sampath proposed a change to openstack/tempest: Glance Metadata Definitions Catalog - Tempest tests https://review.openstack.org/113632 | 00:27 |
| *** evgenyf has quit IRC | 00:43 | |
| *** lordd_ has quit IRC | 00:44 | |
| *** lordd_ has joined #openstack-qa | 00:45 | |
| *** suneelb has quit IRC | 00:47 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Modify `glance image-create` commands to use openstackclient https://review.openstack.org/111538 | 00:50 |
| *** flwang_ has joined #openstack-qa | 00:55 | |
| *** flwang_ has quit IRC | 00:59 | |
| *** rhsu has quit IRC | 01:05 | |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack-dev/devstack: Allow devstack to build ironic agent ramdisk https://review.openstack.org/114408 | 01:09 |
| *** salv-orlando has quit IRC | 01:13 | |
| *** jyuso has joined #openstack-qa | 01:14 | |
| *** nosnos has joined #openstack-qa | 01:15 | |
| *** dims has quit IRC | 01:22 | |
| *** nosnos has quit IRC | 01:22 | |
| *** dims has joined #openstack-qa | 01:22 | |
| *** nosnos has joined #openstack-qa | 01:23 | |
| *** dims_ has joined #openstack-qa | 01:25 | |
| *** dims has quit IRC | 01:25 | |
| *** zz_dimtruck is now known as dimtruck | 01:35 | |
| *** Sukhdev has quit IRC | 01:36 | |
| *** Sukhdev has joined #openstack-qa | 01:36 | |
| *** Sukhdev_ has joined #openstack-qa | 01:39 | |
| *** jaypipes has quit IRC | 01:42 | |
| *** Sukhdev has quit IRC | 01:42 | |
| *** jaypipes has joined #openstack-qa | 01:43 | |
| *** ChanServ sets mode: +o jaypipes | 01:43 | |
| *** marun has quit IRC | 01:44 | |
| *** Sukhdev_ has quit IRC | 01:44 | |
| *** marun has joined #openstack-qa | 01:44 | |
| *** aimon has joined #openstack-qa | 01:45 | |
| *** krtaylor has joined #openstack-qa | 01:54 | |
| *** jaypipes has quit IRC | 01:55 | |
| *** dims_ has quit IRC | 02:04 | |
| *** dims has joined #openstack-qa | 02:05 | |
| *** jaypipes has joined #openstack-qa | 02:06 | |
| *** ChanServ sets mode: +o jaypipes | 02:06 | |
| *** Krast has joined #openstack-qa | 02:33 | |
| *** dustins has joined #openstack-qa | 02:43 | |
| *** dustins has quit IRC | 02:48 | |
| *** dustins has joined #openstack-qa | 02:48 | |
| *** dims has quit IRC | 02:53 | |
| *** flwang_ has joined #openstack-qa | 02:56 | |
| *** flwang_ has quit IRC | 03:00 | |
| *** annegentle_ has quit IRC | 03:02 | |
| *** Krast has quit IRC | 03:10 | |
| *** Krast has joined #openstack-qa | 03:10 | |
| *** adalbas has quit IRC | 03:13 | |
| *** nosnos has quit IRC | 03:28 | |
| jroll | ianw: around? | 03:29 |
| ianw | jroll: hi | 03:29 |
| jroll | ianw: hey, I saw you left a -1 on 114408, not sure what you mean by "pkg install fns" but assuming that's a devstack-ism | 03:31 |
| jroll | better link, https://review.openstack.org/#/c/114408/1/lib/ironic | 03:31 |
| *** yfried__ has quit IRC | 03:31 | |
| ianw | jroll: there's install_package functions, that are platform independent | 03:31 |
| jroll | ianw: since I finally caught you, can I ask you to review the patches that depends on? https://review.openstack.org/112095 and https://review.openstack.org/108457 | 03:32 |
| jroll | ^ they've been hanging for a wihle | 03:32 |
| ianw | but there's also files/apts/ironic which may be appropriate | 03:32 |
| jroll | ianw: ok, cool, I assumed there was something, that one is somewhat of a wip | 03:32 |
| jroll | yeah | 03:32 |
| jroll | I don't know devstack too well, I got that working and put it up to make sure I didn't lose it :P | 03:33 |
| *** dimtruck is now known as zz_dimtruck | 03:34 | |
| ianw | jroll: can you give me the 2-second version of why https://review.openstack.org/#/c/112095/3 is needed? | 03:36 |
| ianw | jroll: it looks ok, but i have no context | 03:36 |
| jroll | ianw: the other patch relies on swift temp urls being a thing | 03:37 |
| jroll | we don't want any credentials being passed to the deploy ramdisk (aka agent) | 03:37 |
| jroll | this allows that ramdisk to download images without creds | 03:37 |
| ianw | jroll: right, so that would have been a better description :) | 03:39 |
| jroll | ianw: I mean, it's a supported feature in swift, so chancs are someone might use it with devstack :P | 03:40 |
| *** nosnos has joined #openstack-qa | 03:41 | |
| *** harlowja is now known as harlowja_away | 03:42 | |
| jroll | ianw: to be honest, I've just had a really hard time getting a hold of devstack cores, which is why I'm bugging you. if you don't like anything about these patches, I'm happy to do someback-and-forth as long as I get your +2 with it :) | 03:44 |
| ianw | jroll: i assume you're ok if I put you down against ironic in MAINTAINERS.rst? | 03:44 |
| jroll | lol, no please don't credit me | 03:44 |
| jroll | :P | 03:44 |
| jroll | that's fine with me | 03:45 |
| jroll | I'm an ironic core, please hold me accountable if needed | 03:45 |
| *** Sukhdev has joined #openstack-qa | 03:46 | |
| jroll | ianw: I'm also happy to throw others under that bus as needed :) | 03:47 |
| jroll | ugh | 03:52 |
| jroll | ianw: thank you for the +A's, apparently there is a merge conflict | 03:52 |
| ianw | jroll: ok, thanks ... nothing jumps out at me worth holding those up for. i might have jumped on them earlier if the change descriptions were more in depth. personally i feel you can not have too much info in there | 03:53 |
| ianw | i have no special knowledge of swift/ironic, so details really help! :) | 03:53 |
| *** chandankumar has joined #openstack-qa | 03:54 | |
| ianw | jroll: bummer, yeah i guess you have to rebase | 03:54 |
| jroll | ianw: yeah, I hear you on the commit messages | 03:55 |
| jroll | will ping you if I rebase tonight, may wait until morning | 03:55 |
| ianw | jroll: ok, i'm in sydney/austraila tz | 03:56 |
| *** amotoki has joined #openstack-qa | 03:56 | |
| *** dustins has quit IRC | 04:02 | |
| jroll | ianw: aha, everything makes so much more sense :P | 04:03 |
| *** pcrews has quit IRC | 04:04 | |
| *** chandankumar has quit IRC | 04:06 | |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack-dev/devstack: Add ironic-python-agent support to devstack https://review.openstack.org/108457 | 04:08 |
| jroll | ianw: ^^ theoretically that should work? | 04:08 |
| jroll | I'm much more confident than that ? seems :P | 04:09 |
| jroll | ianw: and if I just get your +2 that's fine... I can catch dtroyer in the morning (pacific time) | 04:09 |
| *** amotoki has quit IRC | 04:10 | |
| * jroll will be back tomorroww | 04:13 | |
| jroll | thanks ianw | 04:13 |
| ianw | jroll: ok, i'll keep an eye on jenkins | 04:15 |
| *** amotoki has joined #openstack-qa | 04:33 | |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: A service to install diskimage-builder https://review.openstack.org/92256 | 04:47 |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: WIP enable building test image https://review.openstack.org/95617 | 04:47 |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Utility functions for building test images https://review.openstack.org/95616 | 04:47 |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Build an image for heat functional tests https://review.openstack.org/110819 | 04:47 |
| *** ildikov has quit IRC | 04:51 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Add swift tempurl support to devstack https://review.openstack.org/112095 | 04:51 |
| *** rfolco has quit IRC | 04:54 | |
| *** flwang_ has joined #openstack-qa | 04:57 | |
| *** yfried__ has joined #openstack-qa | 04:59 | |
| *** flwang_ has quit IRC | 05:01 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Work around pip installed pkgs on Rackspace image https://review.openstack.org/111646 | 05:07 |
| *** Krast has quit IRC | 05:31 | |
| *** zz_dimtruck is now known as dimtruck | 05:34 | |
| *** jamielennox is now known as jamielennox|away | 05:35 | |
| *** rfolco has joined #openstack-qa | 05:38 | |
| *** Longgeek has joined #openstack-qa | 05:40 | |
| *** dimtruck is now known as zz_dimtruck | 05:44 | |
| *** Sukhdev has quit IRC | 05:46 | |
| *** ildikov has joined #openstack-qa | 05:53 | |
| *** dims has joined #openstack-qa | 05:54 | |
| *** dims has quit IRC | 05:58 | |
| *** Krast has joined #openstack-qa | 06:09 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Add keepalived, conntrackd as dependencies https://review.openstack.org/110679 | 06:16 |
| *** marun has quit IRC | 06:20 | |
| *** marun has joined #openstack-qa | 06:20 | |
| *** pcrews has joined #openstack-qa | 06:40 | |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Allow dict's as schema definitions https://review.openstack.org/112566 | 06:43 |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Convert request schema (flavors) https://review.openstack.org/112570 | 06:46 |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Convert request schema (flavors_admin) https://review.openstack.org/112783 | 06:46 |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Convert request schema (servers) https://review.openstack.org/113787 | 06:46 |
| mkoderer | andreaf: good morning.. did you ever tried the rebase button in gerrit? | 06:47 |
| *** salv-orlando has joined #openstack-qa | 06:56 | |
| *** salv-orlando has quit IRC | 06:57 | |
| *** flwang_ has joined #openstack-qa | 06:58 | |
| *** salv-orlando has joined #openstack-qa | 07:02 | |
| *** flwang_ has quit IRC | 07:02 | |
| *** salv-orlando has quit IRC | 07:03 | |
| *** marun has quit IRC | 07:03 | |
| *** wuhg has joined #openstack-qa | 07:03 | |
| *** markus_z has joined #openstack-qa | 07:12 | |
| *** salv-orlando has joined #openstack-qa | 07:13 | |
| *** salv-orlando has quit IRC | 07:13 | |
| *** salv-orlando has joined #openstack-qa | 07:14 | |
| *** salv-orlando has quit IRC | 07:16 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Hide output when configuring apt retry https://review.openstack.org/103539 | 07:16 |
| *** tkammer has joined #openstack-qa | 07:16 | |
| *** gszasz has joined #openstack-qa | 07:26 | |
| *** gszasz has quit IRC | 07:29 | |
| *** gszasz has joined #openstack-qa | 07:29 | |
| andreaf | mkoderer: I did some time ago - and it seemed to work, but I usually prefer to do rebase from my workstation | 07:31 |
| *** zz_dimtruck is now known as dimtruck | 07:34 | |
| *** andreaf_ has joined #openstack-qa | 07:35 | |
| *** jlanoux has joined #openstack-qa | 07:35 | |
| *** andreaf_ has quit IRC | 07:36 | |
| *** Longgeek has quit IRC | 07:37 | |
| *** Longgeek has joined #openstack-qa | 07:37 | |
| *** andreaf_ has joined #openstack-qa | 07:38 | |
| *** andreaf has quit IRC | 07:38 | |
| *** andreaf_ is now known as andreaf | 07:38 | |
| *** andreaf has quit IRC | 07:39 | |
| *** andreaf has joined #openstack-qa | 07:39 | |
| *** Longgeek has quit IRC | 07:42 | |
| *** dimtruck is now known as zz_dimtruck | 07:44 | |
| *** markus_z has quit IRC | 07:47 | |
| *** Longgeek has joined #openstack-qa | 07:51 | |
| *** jpich has joined #openstack-qa | 08:00 | |
| *** qba73 has joined #openstack-qa | 08:10 | |
| *** jlibosva has joined #openstack-qa | 08:22 | |
| openstackgerrit | Lakshmi N Sampath proposed a change to openstack/tempest: Glance Metadata Definitions Catalog - Tempest tests https://review.openstack.org/113632 | 08:26 |
| *** yamahata has quit IRC | 08:43 | |
| mkoderer | andreaf: ok so I had to rebase https://review.openstack.org/#/c/112566/ | 08:43 |
| mkoderer | jenkins is already through | 08:43 |
| *** Longgeek has quit IRC | 08:48 | |
| *** Dafna has quit IRC | 08:49 | |
| *** Longgeek has joined #openstack-qa | 08:50 | |
| *** Longgeek has quit IRC | 08:52 | |
| *** dmellado has quit IRC | 08:52 | |
| *** Longgeek has joined #openstack-qa | 08:53 | |
| *** Dafna has joined #openstack-qa | 08:59 | |
| *** flwang_ has joined #openstack-qa | 08:59 | |
| *** flwang_ has quit IRC | 09:03 | |
| shardy | mtreinish: I gave up on https://review.openstack.org/#/c/90143 because of slow reviews, the fact that it kept breaking between constant rebases, and the fact that it should have an easier and less frustrating path via stevebaker's proposal for in-tree functional tests | 09:10 |
| shardy | I have to say, it's been a hugely demotivating process with that particular patch | 09:10 |
| mkoderer | shardy: did we discuss about this patch in a qa meeting before? | 09:14 |
| shardy | mkoderer: I guess not, I didn't know I had to show up to the meeting to get reviews :\ | 09:14 |
| shardy | Anyway, IMO the move to scenario tests in project trees will work out better for everyone :) | 09:15 |
| mkoderer | shardy: so usualy you don't need to be in the qa meeting but if you have such patches it's might the fastest way | 09:15 |
| mkoderer | since many cores are around during that time | 09:15 |
| shardy | mkoderer: Ok, useful info, thanks | 09:16 |
| mkoderer | shardy: https://wiki.openstack.org/wiki/Meetings/QATeamMeeting | 09:16 |
| mkoderer | we usualy have a topic called "critical reviews" | 09:16 |
| mkoderer | shardy: sry that the review time is sometimes quite high.. | 09:18 |
| *** jyuso has quit IRC | 09:18 | |
| shardy | mkoderer: No worries, we have the same issue with Heat review backlog, so I do understand | 09:18 |
| shardy | My frustration with that patch is that it was designed to catch a real regression which happened, which then happened again | 09:19 |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: Adds status check for FloatingIP in scenarios https://review.openstack.org/102700 | 09:22 |
| yfried__ | andreaf: do you have a better idea instead of copy-pasting those waiters https://review.openstack.org/#/c/102700/9/tempest/common/waiters.py,cm ? | 09:23 |
| yfried__ | andreaf: or is this outside of the scope for now? | 09:23 |
| *** Longgeek has quit IRC | 09:35 | |
| *** gmann has quit IRC | 09:37 | |
| openstackgerrit | Christian Berendt proposed a change to openstack-dev/devstack: Remove code tagged as "can be removed after Icehouse" https://review.openstack.org/111703 | 09:44 |
| *** Longgeek has joined #openstack-qa | 09:47 | |
| *** cdent has joined #openstack-qa | 09:48 | |
| *** tosky has joined #openstack-qa | 09:51 | |
| *** Krast has quit IRC | 09:56 | |
| *** salv-orlando has joined #openstack-qa | 10:17 | |
| *** dims has joined #openstack-qa | 10:25 | |
| *** jhenner has joined #openstack-qa | 10:27 | |
| *** Longgeek has quit IRC | 10:30 | |
| *** dims has quit IRC | 10:32 | |
| *** asselin has quit IRC | 10:34 | |
| *** kmartin has quit IRC | 10:41 | |
| coolsvap | yfried__, hi u thr? | 10:50 |
| yfried__ | coolsvap: for now | 10:51 |
| coolsvap | yfried__, just need a min regarding your comments for cinder qos patch | 10:51 |
| coolsvap | yfried__, the expected status comment | 10:51 |
| *** dmellado has joined #openstack-qa | 10:51 | |
| coolsvap | yfried__, i checked the patch before updating the client | 10:52 |
| coolsvap | yfried__, https://review.openstack.org/#/c/101310/3/tempest/services/image/v1/json/image_client.py | 10:52 |
| yfried__ | coolsvap: yeah? | 10:52 |
| yfried__ | coolsvap: I'm sorry | 10:53 |
| yfried__ | coolsvap: I talked with dkranz and he said they are ok with how you did it (for now) so ignore that one | 10:53 |
| openstackgerrit | Swapnil Kulkarni proposed a change to openstack/tempest: Adds the clients and tests for CINDER QoS V1 & V2 APIs https://review.openstack.org/90306 | 10:54 |
| *** jlanoux has quit IRC | 10:55 | |
| coolsvap | yfried__, is the dvsm-large-ops issue sorted out? | 10:56 |
| coolsvap | couple of my patches failed due to not able to download packages on node | 10:57 |
| coolsvap | yfried__, do you have any update | 10:57 |
| *** gszasz has quit IRC | 10:57 | |
| *** flwang_ has joined #openstack-qa | 11:00 | |
| *** Longgeek has joined #openstack-qa | 11:00 | |
| *** flwang_ has quit IRC | 11:04 | |
| *** dims has joined #openstack-qa | 11:06 | |
| *** dims has quit IRC | 11:12 | |
| *** dims has joined #openstack-qa | 11:12 | |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack-dev/devstack: Install sqlalchemy-migrate from master if requested https://review.openstack.org/114224 | 11:22 |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack-dev/devstack: (DO NOT MERGE) test run using mysqlconnector https://review.openstack.org/114207 | 11:22 |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack-dev/devstack: Add ability to switch database driver https://review.openstack.org/105209 | 11:22 |
| *** SridharG has joined #openstack-qa | 11:27 | |
| *** lordd_ has quit IRC | 11:27 | |
| *** Longgeek_ has joined #openstack-qa | 11:34 | |
| *** Longgeek has quit IRC | 11:37 | |
| *** SridharG has quit IRC | 11:37 | |
| *** aix has joined #openstack-qa | 11:38 | |
| *** aix has quit IRC | 11:40 | |
| *** aix has joined #openstack-qa | 11:40 | |
| openstackgerrit | Daniel Berrange proposed a change to openstack-dev/devstack: libvirt: display warning message if firewalld is enabled https://review.openstack.org/114513 | 11:56 |
| *** ildikov has quit IRC | 11:56 | |
| *** ildikov has joined #openstack-qa | 11:58 | |
| *** nmagnezi has joined #openstack-qa | 12:05 | |
| *** yfried__ has quit IRC | 12:17 | |
| *** amotoki has quit IRC | 12:17 | |
| *** afazekas has joined #openstack-qa | 12:19 | |
| *** rfolco has quit IRC | 12:26 | |
| *** rfolco has joined #openstack-qa | 12:26 | |
| *** adalbas has joined #openstack-qa | 12:26 | |
| *** dkranz has joined #openstack-qa | 12:31 | |
| *** nosnos has quit IRC | 12:37 | |
| ianw | jroll: looks like that merge failed, i'll try to check on it tomorrow if you get it past jenkins | 12:39 |
| *** gszasz has joined #openstack-qa | 12:45 | |
| *** aysyd has joined #openstack-qa | 12:51 | |
| *** aix has quit IRC | 12:55 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Add Credentials Provider factory https://review.openstack.org/107685 | 12:56 |
| *** aix has joined #openstack-qa | 12:57 | |
| *** nmagnezi has quit IRC | 13:00 | |
| *** andreaf_ has joined #openstack-qa | 13:01 | |
| *** flwang_ has joined #openstack-qa | 13:01 | |
| *** jaypipes is now known as leakypipes | 13:03 | |
| *** andreaf has quit IRC | 13:03 | |
| *** dpaterson has joined #openstack-qa | 13:03 | |
| *** dustins has joined #openstack-qa | 13:04 | |
| *** timello_ has joined #openstack-qa | 13:05 | |
| *** flwang_ has quit IRC | 13:05 | |
| *** andreaf_ has quit IRC | 13:06 | |
| *** andreaf_ has joined #openstack-qa | 13:06 | |
| *** wuhg has quit IRC | 13:08 | |
| *** johnthetubaguy has quit IRC | 13:09 | |
| *** wuhg has joined #openstack-qa | 13:10 | |
| *** dims_ has joined #openstack-qa | 13:13 | |
| *** pcrews_ has joined #openstack-qa | 13:15 | |
| openstackgerrit | gordon chung proposed a change to openstack-dev/devstack: enable multiple collector workers https://review.openstack.org/109058 | 13:18 |
| *** dims has quit IRC | 13:20 | |
| *** jpich has quit IRC | 13:20 | |
| *** pcrews has quit IRC | 13:20 | |
| *** mriedem has joined #openstack-qa | 13:23 | |
| *** zz_johnthetubagu has joined #openstack-qa | 13:25 | |
| *** zz_johnthetubagu is now known as johnthetubaguy | 13:25 | |
| *** jpich has joined #openstack-qa | 13:26 | |
| *** dansmith is now known as superdan | 13:31 | |
| *** timello_ has quit IRC | 13:32 | |
| *** timello_ has joined #openstack-qa | 13:33 | |
| *** jlanoux has joined #openstack-qa | 13:35 | |
| *** mkollaro has joined #openstack-qa | 13:37 | |
| *** andreaf_ is now known as andreaf | 13:42 | |
| *** tkammer has quit IRC | 13:42 | |
| andreaf | salv-orlando: hi - have you seen this failure on neutron full before? http://paste.openstack.org/show/95572/ | 13:43 |
| *** cdent_ has joined #openstack-qa | 13:43 | |
| *** cdent has quit IRC | 13:45 | |
| *** cdent_ is now known as cdent | 13:45 | |
| salv-orlando | andreaf: yes it was fixed on Aug 13th -> https://review.openstack.org/#/c/113554/ | 13:47 |
| andreaf | salv-orlando: ok thanks - so on rebase it should go away from here: https://review.openstack.org/#/c/101381/8 | 13:48 |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack-dev/devstack: Install sqlalchemy-migrate from master if requested https://review.openstack.org/114224 | 13:48 |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack-dev/devstack: (DO NOT MERGE) test run using mysqlconnector https://review.openstack.org/114207 | 13:48 |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack-dev/devstack: Add ability to switch database driver https://review.openstack.org/105209 | 13:48 |
| andreaf | salv-orlando: I was about to approve the change when I saw the failure and I was worried - it should be fine then, thank you | 13:48 |
| *** rbak has joined #openstack-qa | 13:53 | |
| openstackgerrit | Choonho Son proposed a change to openstack/tempest: Fix QuotaClassesAdminTest to call V3 API https://review.openstack.org/114537 | 13:59 |
| *** cpallares has joined #openstack-qa | 14:00 | |
| *** jgrimm has joined #openstack-qa | 14:01 | |
| *** markmcclain has joined #openstack-qa | 14:03 | |
| *** yfried__ has joined #openstack-qa | 14:07 | |
| yfried__ | coolsvap: no idea what you were talking about | 14:07 |
| yfried__ | salv-orlando: don't bother with the gate on the Floating IP status. it's based on a few unstable patches that need to be merged first | 14:08 |
| *** dpaterson has quit IRC | 14:11 | |
| *** markmcclain1 has joined #openstack-qa | 14:13 | |
| salv-orlando | yfried__: it was not about the gate, but about minesweeper CI. I found a bug related to that -> https://review.openstack.org/#/c/114504/ | 14:13 |
| salv-orlando | once that merges, vmware CI should pass. | 14:13 |
| *** zz_dimtruck is now known as dimtruck | 14:14 | |
| *** markmcclain has quit IRC | 14:15 | |
| *** jecarey has joined #openstack-qa | 14:26 | |
| *** mitz has quit IRC | 14:27 | |
| *** tkammer has joined #openstack-qa | 14:30 | |
| andreaf | mtreinish: https://review.openstack.org/#/c/107685/ for test-accounts is now passing CI | 14:32 |
| *** dimtruck is now known as zz_dimtruck | 14:32 | |
| mtreinish | andreaf: ok I'll take a look | 14:33 |
| andreaf | mtreinish: thanks | 14:34 |
| mtreinish | I'm thinking it might be worthwhile to put a hardcoded test patch on top of it just to see how if it will work at all | 14:34 |
| andreaf | mtreinish: uhm right with a yaml file and config settings? | 14:34 |
| mtreinish | yeah something like that, it might be too tricky todo | 14:35 |
| mtreinish | I'm just thinking we've written all this code | 14:35 |
| andreaf | mtreinish: config settings won't be enough as devstack sets allow_tenant_isolation true, but I can change the code if allow_tenant_isolation true use the new approach | 14:35 |
| mtreinish | and only unit tested it | 14:35 |
| mtreinish | andreaf: yeah that, or you could just inline setting the value of allow_tenant_isolation to false | 14:36 |
| openstackgerrit | afazekas proposed a change to openstack-infra/devstack-gate: Fix ssh key specification in the README.rst https://review.openstack.org/114553 | 14:36 |
| andreaf | mtreinish: ok - good idea | 14:36 |
| andreaf | mtreinish: is the periodic job running with tenant_isolation off? | 14:37 |
| mtreinish | andreaf: I believe we have one or 2 periodic jobs that run without tenant isolation | 14:38 |
| *** dims_ is now known as dimsum | 14:40 | |
| *** dimsum is now known as dimsum_ | 14:40 | |
| openstackgerrit | Christian Berendt proposed a change to openstack-dev/devstack: Remove code tagged as "can be removed after Icehouse" https://review.openstack.org/111703 | 14:40 |
| andreaf | mtreinish: so I would need to create some more accounts on the fly - else it won't work very well | 14:42 |
| *** ildikov has quit IRC | 14:43 | |
| Shrews | Any possibility of getting one more tempest core to review https://review.openstack.org/94439 today? andreaf? dkranz? | 14:45 |
| Shrews | Would love to see that merged before vacation next week :) | 14:45 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack-dev/devstack: Add ironic-python-agent support to devstack https://review.openstack.org/108457 | 14:45 |
| mtreinish | andreaf: yeah that's what I figured which is why I thought it might be too complicated to do in a test patch | 14:45 |
| jroll | dtroyer, ianw, ^^ not sure how I messed up that rebase last night :P | 14:45 |
| mtreinish | too many other pieces are missing | 14:45 |
| andreaf | Shrews: nice test - I'll take a look later today | 14:46 |
| Shrews | andreaf: thank you very much | 14:46 |
| *** rbak has quit IRC | 14:47 | |
| openstackgerrit | gordon chung proposed a change to openstack-dev/devstack: enable multiple collector workers https://review.openstack.org/109058 | 14:51 |
| *** rbak has joined #openstack-qa | 14:52 | |
| *** zhiyan has quit IRC | 14:53 | |
| *** zhiyan has joined #openstack-qa | 14:54 | |
| openstackgerrit | A change was merged to openstack/tempest: Allow dict's as schema definitions https://review.openstack.org/112566 | 14:54 |
| *** atiwari has joined #openstack-qa | 14:54 | |
| *** zz_dimtruck is now known as dimtruck | 14:58 | |
| *** flwang_ has joined #openstack-qa | 15:02 | |
| *** ildikov has joined #openstack-qa | 15:06 | |
| *** atiwari has quit IRC | 15:06 | |
| *** flwang_ has quit IRC | 15:06 | |
| *** mestery is now known as mestery_afk | 15:10 | |
| dkranz | Shrews: I found two small things to fix and will then +2 | 15:13 |
| openstackgerrit | Dolph Mathews proposed a change to openstack-infra/elastic-recheck: ade query for nova quota reservation deadlock https://review.openstack.org/114566 | 15:14 |
| Shrews | dkranz: ok, thx | 15:14 |
| openstackgerrit | Dolph Mathews proposed a change to openstack-infra/elastic-recheck: add query for bug 1316926 https://review.openstack.org/102932 | 15:16 |
| coolsvap | dkranz, hi | 15:18 |
| dkranz | coolsvap: hi | 15:19 |
| coolsvap | dkranz, can you have a look at https://review.openstack.org/#/c/90306/ | 15:19 |
| *** mkollaro has quit IRC | 15:19 | |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack-dev/devstack: (DO NOT MERGE) test run using mysqlconnector https://review.openstack.org/114207 | 15:19 |
| dkranz | coolsvap: ok | 15:20 |
| openstackgerrit | Dolph Mathews proposed a change to openstack-infra/elastic-recheck: Add query for bin/pip suddenly missing https://review.openstack.org/111361 | 15:23 |
| *** atiwari has joined #openstack-qa | 15:27 | |
| *** yfried__ has quit IRC | 15:31 | |
| Shrews | dkranz: good comments. however, can i get you to clarify "parameter default should not be mutable" ? | 15:32 |
| mtreinish | Shrews: I think he was referring to: https://review.openstack.org/#/c/100132/ | 15:33 |
| dkranz | Shrews: I only learned this recently, but apparently when you say x={} in a function signature, Python treats the {} as a module global | 15:33 |
| mtreinish | or something like that | 15:33 |
| Shrews | ah | 15:34 |
| dkranz | Shrews: so if any code that receives this value were to modify it, it would change the meaning of the signature | 15:34 |
| dkranz | Shrews: I consider this to be a bug in Python | 15:34 |
| dkranz | Shrews: BUt others obviously differ :) | 15:34 |
| *** andreaf has quit IRC | 15:35 | |
| *** andreaf has joined #openstack-qa | 15:36 | |
| *** andreaf has quit IRC | 15:36 | |
| *** andreaf has joined #openstack-qa | 15:37 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Force use of Accounts for test purposes https://review.openstack.org/114578 | 15:39 |
| *** mriedem is now known as mattykins | 15:41 | |
| openstackgerrit | David Shrewsbury proposed a change to openstack/tempest: Add ironic instance rebuild test https://review.openstack.org/94439 | 15:41 |
| andreaf | mtreinish: first attempt https://review.openstack.org/#/c/114578 | 15:41 |
| *** mattykins is now known as matty_kins | 15:41 | |
| *** jpich has quit IRC | 15:42 | |
| Shrews | dkranz: mtreinish: ^^^ that should address dkranz's comments on 94439. thanks! | 15:43 |
| *** jpich has joined #openstack-qa | 15:43 | |
| mtreinish | andreaf: heh, that's a clever approach if the yaml doesn't exist generate one with 8 accounts using tenant isolation | 15:44 |
| mtreinish | I like that | 15:44 |
| *** dpaterson has joined #openstack-qa | 15:45 | |
| andreaf | mtreinish: hopefully it works :) | 15:46 |
| *** jlanoux has quit IRC | 15:47 | |
| *** markmcclain1 has quit IRC | 15:48 | |
| andreaf | dkranz, are you already on https://review.openstack.org/94439 ? | 15:49 |
| *** aix has quit IRC | 15:49 | |
| dkranz | andreaf: Yes, I found a few issues and there is a new patch up. | 15:49 |
| dkranz | andreaf: I'm looking at it now | 15:49 |
| andreaf | dkranz, ok thanks | 15:50 |
| *** wuhg has quit IRC | 15:52 | |
| *** dustins has quit IRC | 15:57 | |
| dkranz | andreaf: I am also reviewing https://review.openstack.org/#/c/90306/27/tempest/api/volume/base.py,unified which will collide with the multi-user auth stuff. Should this patch go in before yours? You will have to change the new Base class in this file. | 15:59 |
| *** dustins has joined #openstack-qa | 15:59 | |
| *** asselin has joined #openstack-qa | 16:00 | |
| *** bnemec is now known as beekneemech | 16:01 | |
| *** aix has joined #openstack-qa | 16:02 | |
| andreaf | dkranz: yes I have seen this one but could not complete a full review yet :) I think it can go first - only I don't like the fact that it reads CONF for account details, it shouldn't do that anyways - but I suppose we could leave that as it is, I will have to change it anyways with the test-accounts stuff | 16:04 |
| dkranz | andreaf: Agreed | 16:05 |
| dkranz | andreaf: Are you going to finish reviewing it? | 16:05 |
| dkranz | andreaf: A cinder domain-expert review might be useful | 16:06 |
| andreaf | dkranz: I reviewed it in the past, I'm going to have a look at the new patchset - I would not define myself a cinder expert though | 16:08 |
| *** jlibosva has quit IRC | 16:09 | |
| *** leakypipes is now known as leakypipes-codin | 16:14 | |
| *** leakypipes-codin is now known as leakypipes | 16:15 | |
| coolsvap | dkranz, I am picking up the client status changes for volume clients, I will push some patches over the weekend | 16:17 |
| coolsvap | s/patches/reviews | 16:17 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Remove not used server creation from test_server_rescue https://review.openstack.org/114592 | 16:20 |
| *** markmcclain has joined #openstack-qa | 16:22 | |
| *** comstud is now known as bearhands | 16:26 | |
| *** markmcclain has quit IRC | 16:27 | |
| *** jogo is now known as flashgordon | 16:28 | |
| *** aimon has quit IRC | 16:32 | |
| *** jpich has quit IRC | 16:37 | |
| *** dimtruck is now known as zz_dimtruck | 16:40 | |
| *** aix has quit IRC | 16:41 | |
| *** zz_dimtruck is now known as dimtruck | 16:42 | |
| *** markmcclain has joined #openstack-qa | 16:44 | |
| *** markmcclain has quit IRC | 16:47 | |
| *** markmcclain has joined #openstack-qa | 16:47 | |
| *** markmcclain has quit IRC | 16:48 | |
| *** markmcclain has joined #openstack-qa | 16:49 | |
| *** aimon has joined #openstack-qa | 16:49 | |
| *** markmcclain has quit IRC | 16:51 | |
| *** dimsum_ has quit IRC | 16:55 | |
| *** aix has joined #openstack-qa | 16:56 | |
| andreaf | mtreinish: something's wrong with lock_path | 16:56 |
| andreaf | mtreinish: http://logs.openstack.org/78/114578/1/check/check-tempest-dsvm-full/df8d8e6/logs/testr_results.html.gz | 16:57 |
| andreaf | mtreinish: until now lock_path has only been used in unit tests, and there a default value is set in fake_config | 16:57 |
| *** dims has joined #openstack-qa | 16:58 | |
| *** openstackgerrit has quit IRC | 17:02 | |
| *** openstackgerrit has joined #openstack-qa | 17:02 | |
| *** flwang_ has joined #openstack-qa | 17:03 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Add Credentials Provider factory https://review.openstack.org/107685 | 17:03 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Force use of Accounts for test purposes https://review.openstack.org/114578 | 17:03 |
| *** harlowja_away is now known as harlowja | 17:03 | |
| andreaf | mtreinish: addressed your comments and fixed pep8 in 114578 - but until the lock_path issue is solved the test change won't tell us much more | 17:04 |
| *** suneelb has joined #openstack-qa | 17:05 | |
| *** flwang_ has quit IRC | 17:07 | |
| *** dustins has quit IRC | 17:09 | |
| *** gszasz has quit IRC | 17:14 | |
| *** dustins has joined #openstack-qa | 17:15 | |
| *** dustins has quit IRC | 17:16 | |
| *** dustins has joined #openstack-qa | 17:17 | |
| *** dustins has quit IRC | 17:22 | |
| *** qba73 has quit IRC | 17:22 | |
| *** qba73 has joined #openstack-qa | 17:23 | |
| *** aix has quit IRC | 17:23 | |
| andreaf | mtreinish: I think it's related to the config proxy - as far as I can tell the additional lock options are only loaded when the lock utils are imported - but they are not registered in the TempestConfigPrivate object | 17:24 |
| *** qba73 has quit IRC | 17:25 | |
| *** qba73 has joined #openstack-qa | 17:28 | |
| *** qba73 has quit IRC | 17:28 | |
| *** matty_kins is now known as mriedem | 17:32 | |
| *** dustins has joined #openstack-qa | 17:34 | |
| *** dustins has quit IRC | 17:34 | |
| *** aix has joined #openstack-qa | 17:36 | |
| mtreinish | andreaf: oh, yeah it's the proxy | 17:37 |
| mtreinish | we've never accessed lock_path directly through the proxy before | 17:37 |
| andreaf | mtreinish: so I am preparing a one line change that adds that to Private config | 17:37 |
| mtreinish | oslo lock uses it, but it doesn't go through the proxy | 17:38 |
| mtreinish | andreaf: ok cool | 17:38 |
| andreaf | but I wonder why the fixture works in aggregate tests | 17:38 |
| andreaf | ah ok you answered already | 17:38 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Force use of Accounts for test purposes https://review.openstack.org/114578 | 17:40 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Add lock_path to TempestPrivateConfig https://review.openstack.org/114609 | 17:40 |
| andreaf | mtreinish: ^^^ | 17:40 |
| openstackgerrit | afazekas proposed a change to openstack-infra/elastic-recheck: Add query for 1357476 https://review.openstack.org/114610 | 17:40 |
| andreaf | mtreinish: actually I should have put the lock one on the bottom of the chain so it can merge alone but hopefully the whole thing will merge soon | 17:41 |
| andreaf | mtreinish: so the test change was useful already | 17:42 |
| mtreinish | yep, and actually writing unit tests caught that | 17:42 |
| mtreinish | but I hacked around it | 17:42 |
| mtreinish | andreaf: I'm wondering if it would be better to have a getattr fallback to the cfg.CONF object if the proxy returns AttributeError | 17:45 |
| mtreinish | because the next time we need to access an option in the default group this will pop up again | 17:45 |
| *** krtaylor is now known as krtaylor_away | 17:46 | |
| *** dustins has joined #openstack-qa | 17:47 | |
| andreaf | mtreinish: yep makes sense I was a bit lazy wanted to see if it would work | 17:50 |
| mtreinish | that's fine, but I'lll probably -1 it when the time for review comes :) | 17:51 |
| mtreinish | but for testing in the meantime it's fine | 17:51 |
| *** dims is now known as dimsum_ | 17:52 | |
| *** dpaterson has quit IRC | 17:53 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Add Credentials Provider factory https://review.openstack.org/107685 | 17:59 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Force use of Accounts for test purposes https://review.openstack.org/114578 | 17:59 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Supports DEFAULT group opts in TempestPrivateConfig https://review.openstack.org/114609 | 17:59 |
| andreaf | mtreinish: ok I saw your message too late - I made a new patch-set | 18:00 |
| openstackgerrit | gordon chung proposed a change to openstack-dev/devstack: enable multiple collector workers https://review.openstack.org/109058 | 18:00 |
| *** catohornet has joined #openstack-qa | 18:00 | |
| *** dustins has quit IRC | 18:02 | |
| jroll | so, a devstack patch broke ironic's gate, I'm pushing a fix now but just a heads up that it needs to be fast tracked: https://bugs.launchpad.net/devstack/+bug/1357495 | 18:03 |
| *** catohornet has quit IRC | 18:03 | |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack-dev/devstack: Fix image create functions for Ironic https://review.openstack.org/114619 | 18:05 |
| jroll | ^ there's the fix | 18:05 |
| *** aix has quit IRC | 18:05 | |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack-dev/devstack: Add ironic-python-agent support to devstack https://review.openstack.org/108457 | 18:09 |
| *** Longgeek_ has quit IRC | 18:12 | |
| *** marun has joined #openstack-qa | 18:15 | |
| *** dimtruck is now known as zz_dimtruck | 18:15 | |
| adam_g | did gate-tempest-pep8 only just recently start enforcing H305? | 18:16 |
| *** marun has quit IRC | 18:17 | |
| mtreinish | adam_g: yeah, sort of | 18:17 |
| mtreinish | wait which one is h305 | 18:17 |
| adam_g | import grouping (stdlib, thirdparty) | 18:18 |
| *** marun has joined #openstack-qa | 18:18 | |
| mtreinish | yeah, that one was baked into a larger fix I had re-enabling a bunch of rules we turned off after the hacking 0.9 update | 18:18 |
| adam_g | ah | 18:18 |
| adam_g | mtreinish, any idea why i wouldnt be able to catch it locally? | 18:18 |
| mtreinish | I'm thinking I need to push through a patch to disable it | 18:18 |
| mtreinish | python3 vs 2.7 in the gate? | 18:18 |
| mtreinish | the patch also just landed yesterday | 18:19 |
| mtreinish | so you might not have had it in your tree | 18:19 |
| adam_g | dah | 18:21 |
| adam_g | rebase fail | 18:21 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Disable hacking rule H305 https://review.openstack.org/114622 | 18:23 |
| mtreinish | adam_g, dkranz: ^^^ | 18:23 |
| openstackgerrit | Adam Gandelman proposed a change to openstack/tempest: Add new snapshot compute feature flag https://review.openstack.org/101381 | 18:23 |
| adam_g | dah! | 18:23 |
| adam_g | mtreinish, fixes H305 failures that just showed up ^ :) | 18:23 |
| adam_g | andreaf, ^ | 18:24 |
| *** andreaf_ has joined #openstack-qa | 18:27 | |
| andreaf_ | adam_g: +A | 18:29 |
| adam_g | andreaf, gratzi | 18:29 |
| *** andreaf has quit IRC | 18:30 | |
| *** dustins has joined #openstack-qa | 18:30 | |
| adam_g | dkranz, dtroyer anyone around to help fix a gate breaker? https://review.openstack.org/#/c/114619/ | 18:34 |
| dkranz | adam_g: Sorry I don't have +2 on devstack | 18:35 |
| adam_g | dkranz, oh, sorry, thought you did | 18:36 |
| dtroyer | adam_g: +2 | 18:37 |
| dkranz | mtreinish: Gave +2. I guess the theoretical turned practical pretty fast... | 18:38 |
| mtreinish | heh, well I've see 3 things bounce off the gate because of it already | 18:38 |
| dkranz | mtreinish: ouch | 18:38 |
| dkranz | mtreinish, adam_g : I actually have to leave the keyboard now so will not be able to zap anything else today | 18:39 |
| mtreinish | dkranz: no worries, I'll also have to call it an early day myself shortly | 18:40 |
| Shrews | mtreinish: one more look at 94439 before you leave today? the ironic job is failing due to the broken gate (passed in previous runs) | 18:41 |
| mtreinish | Shrews: heh, sure | 18:42 |
| Shrews | mtreinish: thx! simple change from previous patchset | 18:42 |
| mtreinish | Shrews: +A | 18:43 |
| Shrews | mtreinish: awesome! may your weekend be filled with rainbows and bunnies :) | 18:44 |
| Shrews | heck, even unicorns | 18:44 |
| mtreinish | thanks | 18:44 |
| andreaf_ | mtreinish: question re extensions discovery vs. feature flags | 18:45 |
| andreaf_ | mtreinish: e.g. https://review.openstack.org/#/c/98235 or https://review.openstack.org/#/c/112474/ | 18:46 |
| andreaf_ | mtreinish: tests may check if an extension is loaded and skip if not - or we may rely on extension discovery - what's your opinion on this? | 18:47 |
| mtreinish | andreaf_: we should never poll the api to make a decision regarding skipping a test | 18:48 |
| mtreinish | we should use the config file to specify whether the extension exists or not | 18:48 |
| andreaf_ | mtreinish: what is the rationale? | 18:49 |
| andreaf_ | for instance in https://review.openstack.org/#/c/112474/ it's not about skipping the whole test, it's about allowing additional quota items | 18:49 |
| andreaf_ | while in https://review.openstack.org/#/c/98235 it's about skipping the test | 18:50 |
| andreaf_ | are you worried about the discovery being broken? | 18:50 |
| mtreinish | the api for feature discovery could have bugs (it's happened several times before) in which case we skip when we didn't mean to | 18:50 |
| mtreinish | andreaf_: yeah, it used to happen more than we realized | 18:50 |
| mtreinish | and we were just skipping tests and didn't realize it | 18:50 |
| mtreinish | and bugs slipped through | 18:50 |
| andreaf_ | ok thanks | 18:51 |
| mtreinish | actually I need to add this point to https://review.openstack.org/#/c/99451/2/REVIEWING.rst | 18:51 |
| andreaf_ | mtreinish: yes please | 18:52 |
| andreaf_ | mtreinish: btw, another issue found by the test change for test-accounts http://logs.openstack.org/78/114578/4/check/check-tempest-dsvm-full/0d168ac/logs/testr_results.html.gz | 18:52 |
| mtreinish | doh, s/self.credentials/self.isolated_creds | 18:54 |
| mtreinish | this stack of fixes is getting larger and larger | 18:54 |
| mtreinish | andreaf_: oops you beat me to the comment on 98235 | 18:55 |
| *** aysyd has quit IRC | 18:56 | |
| andreaf_ | mtreinish: I was almost done reviewing it anyways | 18:58 |
| andreaf_ | mtreinish: week-end time for me | 18:58 |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for test_postgresql_opportunistically bug 1355929 https://review.openstack.org/113581 | 18:58 |
| andreaf_ | mtreinish: if you have time for test-accounts for scenario migration patches reviews it would be great | 18:58 |
| andreaf_ | mtreinish: have a great weekend | 18:58 |
| mtreinish | sure, I'll try to take a look | 18:59 |
| mtreinish | yeah you too, enjoy | 18:59 |
| *** cdent has quit IRC | 19:00 | |
| *** jhenner has quit IRC | 19:02 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add REVIEWING.rst https://review.openstack.org/99451 | 19:03 |
| mtreinish | andreaf_: ^^^ | 19:03 |
| *** flwang_ has joined #openstack-qa | 19:04 | |
| *** zz_dimtruck is now known as dimtruck | 19:05 | |
| *** tosky has quit IRC | 19:07 | |
| *** tkammer has quit IRC | 19:07 | |
| *** flwang_ has quit IRC | 19:08 | |
| *** jhenner has joined #openstack-qa | 19:09 | |
| *** jhenner has quit IRC | 19:12 | |
| openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Remove firewalld on Fedora 20 by default https://review.openstack.org/113856 | 19:22 |
| openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Source fixup_stuff... https://review.openstack.org/114635 | 19:22 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add Credentials Provider factory https://review.openstack.org/107685 | 19:28 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Force use of Accounts for test purposes https://review.openstack.org/114578 | 19:28 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Fix use of nonexistent class variable in accounts.py https://review.openstack.org/114636 | 19:28 |
| openstackgerrit | Lakshmi N Sampath proposed a change to openstack/tempest: Glance Metadata Definitions Catalog - Tempest tests https://review.openstack.org/113632 | 19:45 |
| *** mlavalle has joined #openstack-qa | 19:45 | |
| mriedem | afazekas: https://review.openstack.org/#/c/114610/ needs a small tweak, then i'll approve | 19:46 |
| *** SergeyLukjanov has quit IRC | 19:52 | |
| *** DinaBelova has quit IRC | 19:52 | |
| openstackgerrit | afazekas proposed a change to openstack-infra/elastic-recheck: Add query for 1357476 https://review.openstack.org/114610 | 19:52 |
| afazekas | mriedem: ok | 19:52 |
| mriedem | afazekas: thanks | 19:53 |
| *** danfinch has quit IRC | 19:53 | |
| *** chmouel has quit IRC | 19:57 | |
| *** DinaBelova has joined #openstack-qa | 19:58 | |
| *** chmouel has joined #openstack-qa | 19:58 | |
| *** SergeyLukjanov has joined #openstack-qa | 20:00 | |
| *** DinaBelova has quit IRC | 20:07 | |
| *** DinaBelova has joined #openstack-qa | 20:09 | |
| *** notmyname has quit IRC | 20:10 | |
| *** chmouel has quit IRC | 20:10 | |
| *** mriedem1 has joined #openstack-qa | 20:11 | |
| *** notmyname has joined #openstack-qa | 20:11 | |
| *** mriedem1 has quit IRC | 20:11 | |
| *** mriedem1 has joined #openstack-qa | 20:12 | |
| *** mriedem has quit IRC | 20:13 | |
| *** chmouel has joined #openstack-qa | 20:15 | |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Add query for bin/pip suddenly missing https://review.openstack.org/111361 | 20:20 |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for 1357476 https://review.openstack.org/114610 | 20:21 |
| *** cpallares has quit IRC | 20:21 | |
| flashgordon | mtreinish: https://review.openstack.org/#/c/109144/ want to an e-r review? | 20:22 |
| Shrews | jeblair: around? | 20:26 |
| Shrews | or ianw? | 20:27 |
| Shrews | or chmouel? :) we need a devstack change approved to fix ironic gate: https://review.openstack.org/114619 | 20:27 |
| jeblair | Shrews: hullo | 20:28 |
| Shrews | howdy | 20:28 |
| Shrews | jeblair: tl;dr ... a recent devstack change broke us (never passed ironic test). that change above fixes us | 20:29 |
| jeblair | Shrews: can you point me to the breaking devstack change? | 20:29 |
| Shrews | certainly. 1 sec | 20:29 |
| Shrews | jeblair: https://review.openstack.org/#/c/111538/ | 20:30 |
| jroll | we need voting jobs already :( | 20:30 |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for neutron subnet delete race bug 1357055 https://review.openstack.org/114359 | 20:30 |
| *** danfinch has joined #openstack-qa | 20:30 | |
| jeblair | Shrews: so there's a behavior change between glance image-create and openstack image create? | 20:31 |
| Shrews | jeblair: apparently :( | 20:31 |
| jroll | jeblair: afaict, glance client will guess the container format, openstack client will not | 20:31 |
| *** dustins has quit IRC | 20:32 | |
| jeblair | Shrews, jroll: why did the ironic-nv and ironic-postgres-nv jobs pass? | 20:32 |
| chmouel | Shrews: will look | 20:32 |
| Shrews | jeblair: iirc, they only test ironic API behavior while still using libvirt | 20:33 |
| Shrews | jeblair: we are merging these tests, fwiw | 20:33 |
| chmouel | i was indeed thinkin the same question as jeblair | 20:33 |
| adam_g | https://review.openstack.org/#/c/112134/ should clarify all that | 20:34 |
| Shrews | thx adam_g | 20:34 |
| jeblair | oh neat | 20:34 |
| chmouel | cool adam_g | 20:34 |
| chmouel | Shrews: +2A | 20:35 |
| Shrews | awesome-sauce. muchas gracias | 20:35 |
| jroll | thanks chmouel and jeblair :) | 20:35 |
| jeblair | np | 20:36 |
| jeblair | ianw, dtroyer: probably worth drawing your attention to https://review.openstack.org/#/c/111538/1 which had an actual failure on a non-voting job | 20:36 |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for bin/pip suddenly missing https://review.openstack.org/111361 | 20:37 |
| jroll | jeblair: that made me especially sad since it was lib/ironic being changed there | 20:40 |
| jroll | :( | 20:40 |
| dtroyer | jroll, jeblair: yes, very unfortunate, I should have caught that. especially since the behaviour change is in OSC where it sets default values for —container-format and —disk-format… my apologies | 20:42 |
| jroll | it happens, luckily the ironic team is super quiet today :P | 20:43 |
| openstackgerrit | Lakshmi N Sampath proposed a change to openstack/tempest: Glance Metadata Definitions Catalog - Tempest tests https://review.openstack.org/113632 | 20:43 |
| jroll | dtroyer: mind re-reviewing this while you're around? had to rebase https://review.openstack.org/#/c/108457/ | 20:43 |
| dtroyer | jroll: sure... | 20:45 |
| jroll | nice, thank you! :) | 20:46 |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for a rescue related failure https://review.openstack.org/112549 | 20:47 |
| *** rwsu has quit IRC | 20:51 | |
| openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Fix missing --container-format options https://review.openstack.org/114658 | 20:55 |
| dtroyer | chmouel: if you're still around, https://review.openstack.org/114658 completes the fix for the image create command changes for baremetal | 20:57 |
| *** flwang_ has joined #openstack-qa | 21:05 | |
| *** flwang_ has quit IRC | 21:09 | |
| *** leakypipes has quit IRC | 21:15 | |
| *** atiwari has quit IRC | 21:16 | |
| andreaf_ | mtreinish: ping - are you still around? I'm back online for a couple of minutes - I left a comment on https://review.openstack.org/#/c/114636 with another fix to be added | 21:27 |
| *** andreaf_ has quit IRC | 21:37 | |
| *** jecarey has quit IRC | 21:38 | |
| openstackgerrit | Clark Boylan proposed a change to openstack-infra/devstack-gate: Allow for grenade upgrades within a release https://review.openstack.org/111094 | 21:39 |
| *** mriedem1 has quit IRC | 21:41 | |
| *** catohornet has joined #openstack-qa | 21:43 | |
| *** arnaud__ has joined #openstack-qa | 21:45 | |
| *** coolsvap has quit IRC | 21:50 | |
| openstackgerrit | Danny Al-Gaaf proposed a change to openstack/tempest: Use python abc in StressAction class https://review.openstack.org/113820 | 21:54 |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: add query for lbaas create member fail AttributeError https://review.openstack.org/114325 | 22:02 |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Remove fingerprint for bug 1333410 https://review.openstack.org/114674 | 22:03 |
| flashgordon | mtreinish: ^ | 22:03 |
| *** coolsvap has joined #openstack-qa | 22:05 | |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Remove queries for fixed bugs https://review.openstack.org/114676 | 22:08 |
| openstackgerrit | Danny Al-Gaaf proposed a change to openstack/tempest: Use python abc in DeletableResource class https://review.openstack.org/113850 | 22:10 |
| *** dimsum_ has quit IRC | 22:16 | |
| *** dimsum_ has joined #openstack-qa | 22:16 | |
| *** markmcclain has joined #openstack-qa | 22:19 | |
| *** dimsum_ has quit IRC | 22:20 | |
| *** dimsum_ has joined #openstack-qa | 22:26 | |
| *** dimsum_ has quit IRC | 22:26 | |
| *** dimsum_ has joined #openstack-qa | 22:26 | |
| *** dimsum_ has quit IRC | 22:28 | |
| *** dimsum_ has joined #openstack-qa | 22:28 | |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Add fingerprint for nova unit test bug 1357578 https://review.openstack.org/114680 | 22:30 |
| *** dimsum_ has quit IRC | 22:30 | |
| *** dimsum_ has joined #openstack-qa | 22:30 | |
| openstackgerrit | A change was merged to openstack/tempest: Add ironic instance rebuild test https://review.openstack.org/94439 | 22:33 |
| *** dimsum_ has quit IRC | 22:46 | |
| *** dimsum_ has joined #openstack-qa | 22:47 | |
| *** dimsum_ has quit IRC | 22:51 | |
| *** Sukhdev has joined #openstack-qa | 22:57 | |
| *** dimsum_ has joined #openstack-qa | 23:04 | |
| *** flwang_ has joined #openstack-qa | 23:05 | |
| *** rwsu has joined #openstack-qa | 23:06 | |
| *** morganfainberg is now known as morganfainberg_Z | 23:07 | |
| *** dimsum_ has quit IRC | 23:10 | |
| *** dimsum_ has joined #openstack-qa | 23:10 | |
| *** flwang_ has quit IRC | 23:10 | |
| *** tkammer has joined #openstack-qa | 23:12 | |
| ianw | jeblair: mea culpa on that, i didn't notice that failure sorry | 23:13 |
| *** dimsum_ has quit IRC | 23:15 | |
| openstackgerrit | Ramy Asselin proposed a change to openstack-dev/devstack: Allow cinder types to be created without be file https://review.openstack.org/114693 | 23:22 |
| *** dimtruck is now known as zz_dimtruck | 23:33 | |
| *** markmcclain has quit IRC | 23:35 | |
| *** rwsu has quit IRC | 23:35 | |
| *** tkammer has quit IRC | 23:50 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!