*** hemna is now known as hemnafk | 00:03 | |
*** rhsu has joined #openstack-qa | 00:07 | |
openstackgerrit | A change was merged to openstack-infra/devstack-gate: Use Jenkins credentials store if specified https://review.openstack.org/40310 | 00:25 |
---|---|---|
*** rhsu has quit IRC | 00:45 | |
*** sthaha has joined #openstack-qa | 00:54 | |
*** sthaha has joined #openstack-qa | 00:54 | |
*** prekarat has quit IRC | 02:07 | |
openstackgerrit | A change was merged to openstack-dev/devstack: Update projects to global requirements before setup https://review.openstack.org/35705 | 02:15 |
*** lifeless has quit IRC | 02:16 | |
*** adalbas has quit IRC | 02:47 | |
*** jhenner has quit IRC | 02:48 | |
*** adalbas has joined #openstack-qa | 02:57 | |
*** prekarat has joined #openstack-qa | 03:00 | |
*** lifeless has joined #openstack-qa | 03:00 | |
*** prekarat has quit IRC | 03:04 | |
*** prekarat has joined #openstack-qa | 03:06 | |
openstackgerrit | Prem Karat proposed a change to openstack/tempest: Add nova v3 tests for test_servers https://review.openstack.org/40147 | 03:45 |
*** afazekas has quit IRC | 03:47 | |
*** SergeyLukjanov has joined #openstack-qa | 04:03 | |
openstackgerrit | anju Tiwari proposed a change to openstack/tempest: Added a test for stop and start a server https://review.openstack.org/40191 | 04:05 |
*** SergeyLukjanov has quit IRC | 04:30 | |
*** SergeyLukjanov has joined #openstack-qa | 04:31 | |
openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Generate heat images for tempest tests https://review.openstack.org/40341 | 05:04 |
openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Colorize heat engine log https://review.openstack.org/40342 | 05:04 |
openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Baremetal uses packaged diskimage-builder https://review.openstack.org/40343 | 05:04 |
*** prekarat has quit IRC | 05:17 | |
*** SergeyLukjanov has quit IRC | 05:24 | |
*** prekarat has joined #openstack-qa | 05:31 | |
openstackgerrit | anju Tiwari proposed a change to openstack/tempest: Added a test for stop and start a server https://review.openstack.org/40191 | 05:33 |
openstackgerrit | afazekas proposed a change to openstack-dev/devstack: .conf suffix for the horizon config on Fedora https://review.openstack.org/40352 | 06:03 |
openstackgerrit | afazekas proposed a change to openstack-dev/devstack: .conf suffix for the horizon config on Fedora https://review.openstack.org/40352 | 06:11 |
openstackgerrit | Prem Karat proposed a change to openstack/tempest: Add nova-v3-api tests for flavors https://review.openstack.org/40166 | 06:26 |
*** sthaha has quit IRC | 06:36 | |
*** prekarat has quit IRC | 06:56 | |
*** jhenner has joined #openstack-qa | 07:01 | |
*** prekarat has joined #openstack-qa | 07:11 | |
*** jhenner has quit IRC | 07:23 | |
*** tkammer has joined #openstack-qa | 07:28 | |
*** fbo_away is now known as fbo | 07:39 | |
openstackgerrit | anju Tiwari proposed a change to openstack/tempest: A volume upload test added for xml https://review.openstack.org/39664 | 07:41 |
openstackgerrit | anju Tiwari proposed a change to openstack/tempest: A volume upload test added for xml https://review.openstack.org/39664 | 07:44 |
*** dosaboy_alt has joined #openstack-qa | 08:08 | |
*** SergeyLukjanov has joined #openstack-qa | 08:08 | |
*** jhenner has joined #openstack-qa | 08:22 | |
*** jhenner has quit IRC | 08:28 | |
*** jhenner has joined #openstack-qa | 08:37 | |
*** jhenner has quit IRC | 08:42 | |
*** woodspa has joined #openstack-qa | 09:24 | |
*** woodspa has quit IRC | 09:24 | |
openstackgerrit | Salvatore Orlando proposed a change to openstack-dev/devstack: Run Neutron migrations after creating schema https://review.openstack.org/40243 | 09:40 |
*** giulivo has joined #openstack-qa | 09:41 | |
openstackgerrit | Aditi Raveesh proposed a change to openstack/tempest: Adding extra_specs to flavor format https://review.openstack.org/40384 | 10:12 |
openstackgerrit | Giulio Fidente proposed a change to openstack/tempest: scenario test involving glance, cinder and nova functionalities https://review.openstack.org/38943 | 10:19 |
openstackgerrit | Nejc Saje proposed a change to openstack/tempest: Adds client and CRUD tests for Ceilometer alarms https://review.openstack.org/39237 | 10:51 |
giulivo | ping sdague | 10:52 |
*** sthaha has joined #openstack-qa | 11:01 | |
*** prekarat has quit IRC | 11:07 | |
*** mkollaro has joined #openstack-qa | 11:07 | |
*** key has quit IRC | 11:14 | |
*** woodspa has joined #openstack-qa | 11:27 | |
*** prekarat has joined #openstack-qa | 11:28 | |
mkoderer | giulivo: pong (from yesterday) | 11:32 |
*** raies has quit IRC | 11:46 | |
*** jasondotstar has quit IRC | 11:47 | |
*** weshay has joined #openstack-qa | 11:52 | |
openstackgerrit | Nejc Saje proposed a change to openstack/tempest: Adds client and CRUD tests for Ceilometer alarms https://review.openstack.org/39237 | 11:56 |
*** dosaboy_alt has quit IRC | 12:00 | |
*** dosaboy_alt has joined #openstack-qa | 12:01 | |
openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: test_servers_whitebox - using official client https://review.openstack.org/40396 | 12:08 |
*** dkranz has joined #openstack-qa | 12:09 | |
giulivo | hi mkoderer I think I forgot what it was about :( | 12:16 |
openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: test_servers_whitebox - using official client https://review.openstack.org/40396 | 12:16 |
mkoderer | giulivo: np ;) | 12:17 |
*** prekarat has quit IRC | 12:18 | |
giulivo | mkoderer, I don't have such a long scrollback , was it before or after we discussed the decorator? | 12:18 |
mkoderer | after | 12:18 |
mkoderer | you should try irccloud.com | 12:19 |
giulivo | mkoderer, only thing I can think about is this https://review.openstack.org/#/c/40019/ | 12:26 |
mkoderer | giulivo: ok what goes on there? | 12:27 |
giulivo | just wanted to ask if you faced issues around the dependencies installation | 12:28 |
giulivo | and if you had a "preferred" way to do it | 12:28 |
mkoderer | giulivo: dependency issues in devstack? | 12:30 |
giulivo | mkoderer, with devstack yes | 12:31 |
mkoderer | giulivo: never had.. but I didn't update devstack for a while now | 12:32 |
*** prekarat has joined #openstack-qa | 12:33 | |
*** sthaha has quit IRC | 12:34 | |
*** jasondotstar has joined #openstack-qa | 12:35 | |
openstackgerrit | nayna-patel proposed a change to openstack/tempest: Added 3 Routers related testcases for Neutron API https://review.openstack.org/36629 | 12:40 |
*** psedlak has joined #openstack-qa | 12:42 | |
*** psedlak has quit IRC | 12:44 | |
*** whayutin_ has joined #openstack-qa | 12:57 | |
*** weshay has quit IRC | 12:57 | |
*** anteaya has joined #openstack-qa | 12:59 | |
*** fbo is now known as fbo_away | 13:11 | |
*** woodspa_ has joined #openstack-qa | 13:13 | |
*** psedlak has joined #openstack-qa | 13:14 | |
*** woodspa has quit IRC | 13:16 | |
*** sthaha has joined #openstack-qa | 13:17 | |
*** sthaha has joined #openstack-qa | 13:17 | |
*** sthaha has quit IRC | 13:17 | |
rahmu | hello. I have submitted a patch last week here (https://review.openstack.org/#/c/39659/) and it still hasn't been reviewed. Am I missing something? What's the correct procedure to get the reviewing started? | 13:19 |
*** fbo_away is now known as fbo | 13:20 | |
*** mriedem has joined #openstack-qa | 13:22 | |
openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: Add NOVA_DEFAULT_LOG_LEVELS setting https://review.openstack.org/40411 | 13:27 |
*** sthaha has joined #openstack-qa | 13:28 | |
openstackgerrit | afazekas proposed a change to openstack-dev/devstack: Show ip address before associating address https://review.openstack.org/40412 | 13:34 |
*** krtaylor has quit IRC | 13:36 | |
openstackgerrit | Fabien Boucher proposed a change to openstack/tempest: Add tests for swift container listing filters https://review.openstack.org/40416 | 13:39 |
openstackgerrit | Giulio Fidente proposed a change to openstack/tempest: scenario test involving glance, cinder and nova functionalities https://review.openstack.org/38943 | 13:41 |
openstackgerrit | Monty Taylor proposed a change to openstack-dev/devstack: Re-enble pip -e in setup_develop https://review.openstack.org/40417 | 13:42 |
openstackgerrit | Monty Taylor proposed a change to openstack-dev/devstack: Protect local dev changes from the update logic https://review.openstack.org/40418 | 13:42 |
openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: Some bash versions don't support negative lengths https://review.openstack.org/39236 | 13:42 |
*** prekarat has quit IRC | 13:42 | |
*** woodspa__ has joined #openstack-qa | 13:49 | |
openstackgerrit | A change was merged to openstack/tempest: Sync up the default value for the network_for_ssh option https://review.openstack.org/39577 | 13:51 |
*** woodspa_ has quit IRC | 13:52 | |
mkoderer | rahmu: it's already started | 13:55 |
mkoderer | rahmu: usually you have to wait until 2 core reviewers have the time | 13:56 |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Make test_admin_timeout CLI test configurable https://review.openstack.org/40419 | 13:57 |
openstackgerrit | A change was merged to openstack-dev/devstack: Don't add bulk middleware. https://review.openstack.org/39826 | 13:59 |
openstackgerrit | A change was merged to openstack-dev/devstack: Ensures cross compatibility for zsh and bash. https://review.openstack.org/39971 | 13:59 |
*** whayutin_ has quit IRC | 14:05 | |
*** dkranz has quit IRC | 14:05 | |
openstackgerrit | Fabien Boucher proposed a change to openstack/tempest: Add tests for swift container listing filters https://review.openstack.org/40416 | 14:06 |
*** weshay has joined #openstack-qa | 14:10 | |
*** sthaha has joined #openstack-qa | 14:11 | |
rahmu | mkoderer: thanks for your answer, I just got worried when I didn't see any activity after a few days | 14:12 |
*** psedlak has quit IRC | 14:15 | |
*** sthaha has joined #openstack-qa | 14:16 | |
*** sthaha has joined #openstack-qa | 14:16 | |
*** sthaha has quit IRC | 14:16 | |
*** krtaylor has joined #openstack-qa | 14:17 | |
*** sthaha has joined #openstack-qa | 14:19 | |
*** sthaha has joined #openstack-qa | 14:19 | |
*** sthaha has quit IRC | 14:19 | |
*** sthaha has joined #openstack-qa | 14:19 | |
*** sthaha has joined #openstack-qa | 14:19 | |
*** sthaha has quit IRC | 14:19 | |
*** sthaha has joined #openstack-qa | 14:21 | |
*** sthaha has joined #openstack-qa | 14:21 | |
*** burt has joined #openstack-qa | 14:24 | |
mriedem | mkoderer: ping | 14:27 |
mkoderer | mriedem: hey matt | 14:27 |
mriedem | mkoderer: hey, wanted to discuss a global cli timeout for https://review.openstack.org/#/c/40419/ | 14:28 |
mkoderer | mriedem: ok | 14:28 |
mriedem | in case anyone else had an opinion | 14:28 |
mriedem | mkoderer: so the fact that nova's timeout is 2 seconds and keystones is 15, if i made a global option i'd have to default to 15 for keystone, | 14:28 |
mriedem | but bumping the nova timeout to 15 seconds from 2 seems odd | 14:28 |
mkoderer | mriedem: yes thats also true | 14:29 |
mkoderer | mriedem: but having a parameter just for one test case... | 14:29 |
mriedem | mkoderer: yeah, i know | 14:29 |
mriedem | mkoderer: i don't have a great answer either way, so bringing to IRC | 14:29 |
mkoderer | and glance does have the timeout parameter as well | 14:29 |
mkoderer | but it isn't tested | 14:30 |
mkoderer | mriedem: ^^ | 14:30 |
mkoderer | mriedem: I mean we could have a global parameter cli.timeout and cli.compute_timeout or cli.keystone_timeout | 14:32 |
mkoderer | so if one of those specific parameters isn't set it uses the global one | 14:32 |
mkoderer | mriedem: but I think this is a bit too much just for this simple problem | 14:33 |
mriedem | mkoderer: yeah, agree that's a bit much | 14:34 |
mriedem | sdague: any opinion on this? https://review.openstack.org/#/c/40419/ | 14:37 |
mriedem | mtreinish: ^ | 14:37 |
mkoderer | mriedem: but what happens if you switch it to 15 secs on nova | 14:37 |
*** Anju_ has joined #openstack-qa | 14:37 | |
mkoderer | just in case something is broken you need to wait 15 secs | 14:38 |
mriedem | mkoderer: nothing, 15 seconds would be fine for nova - in fact 3 seconds worked for me but 2 seconds didn't | 14:38 |
mkoderer | mriedem: I see :) | 14:38 |
mriedem | mkoderer: basically i wasn't sure if someone thought nova list should be super fast or what, | 14:38 |
mriedem | i'm inclined to think that if we want to avoid a project-specific timeout for CLI, i'm ok with a global that defaults to 15 to cover keystone | 14:39 |
mriedem | if nova list times out at 15 seconds, it's probably busted anyway :) | 14:39 |
mriedem | mkoderer: as long as we can get rid of the hard-coding in the test, i'm happy | 14:39 |
mkoderer | mriedem: is there any reason why keystone is set to 15? | 14:42 |
mriedem | mkoderer: no idea | 14:42 |
mriedem | mkoderer: i guess someone thought it was super slow? | 14:42 |
mkoderer | mriedem: I have the feeling we can put the default to 5 | 14:42 |
mkoderer | mriedem: you have such a slow system? maybe just test it with 5 ;) | 14:42 |
sdague | mtreinish: this is the patch - https://review.openstack.org/#/c/40417/ | 14:42 |
mriedem | mkoderer: let me find the commit that added the keystone test to see if there is any justification for the big timeout | 14:43 |
mkoderer | mriedem: thats a plan | 14:43 |
mriedem | looks like jog0 added those | 14:44 |
mriedem | and i think he's here | 14:44 |
mriedem | https://review.openstack.org/#/c/31541/ | 14:44 |
mriedem | mkoderer: so it was defaulted to 2 seconds, afazekas asked him to change to 5 seconds, and he changed to 15 seconds to be safe | 14:45 |
openstackgerrit | Monty Taylor proposed a change to openstack-dev/devstack: Don't uninstall packages when on xenserver https://review.openstack.org/40431 | 14:46 |
mriedem | mkoderer: given that, i think it's probably safe to use the same logic for the nova timeout and make a global cli timeout option that defaults to 15 | 14:46 |
mriedem | mkoderer: sound good? | 14:46 |
mkoderer | mriedem: great - thanks for clarification | 14:47 |
*** mlavalle has joined #openstack-qa | 14:49 | |
Anju_ | mkoderer:https://review.openstack.org/#/c/40191/ please have a look | 14:52 |
openstackgerrit | Fabien Boucher proposed a change to openstack/tempest: Add test for swift ACLs https://review.openstack.org/38823 | 14:54 |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Make CLI timeout tests configurable https://review.openstack.org/40419 | 15:00 |
Anju_ | mkoderer:means there should bei only one line | 15:02 |
mkoderer | Anju_: yes but in general you could improve the commit message | 15:04 |
mriedem | Anju_: there were no previous tests for stopping and starting an instance? | 15:04 |
Anju_ | noo | 15:04 |
Anju_ | i check that | 15:04 |
mkoderer | mriedem: yes I was surprised about that | 15:05 |
mkoderer | didn't find any .. | 15:06 |
mriedem | Anju_: mkoderer: me neither, i think i was thinking of test_create_image_from_stopped_server which is an images test | 15:06 |
mkoderer | but anyway strange since this is such a basic and important test | 15:07 |
Anju_ | yes it is .but for server like pause,unpause, | 15:07 |
mriedem | pause and unpause are non-core APIs though | 15:07 |
Anju_ | is available | 15:07 |
mriedem | so i'm not really surprised by that | 15:07 |
mriedem | stop/start are core APIS | 15:07 |
mkoderer | mriedem: ok I see | 15:07 |
Anju_ | mriedem:so do u think that it should so important | 15:09 |
Anju_ | i mean that it is not imp | 15:09 |
mriedem | Anju_: yes, stop/start coverage for nova is important :) | 15:09 |
openstackgerrit | Mike Perez proposed a change to openstack-dev/devstack: Start using cinder rest api v2 by default https://review.openstack.org/22489 | 15:09 |
*** ianw has quit IRC | 15:10 | |
*** sthaha has quit IRC | 15:11 | |
Anju_ | mriedem :thanks for reviewing :) | 15:14 |
mriedem | Anju_: no problem, a pretty simple patch | 15:14 |
mkoderer | Anju_: simple but important ;) | 15:15 |
Anju_ | mkoderer::) | 15:15 |
mriedem | testing core APIs important? no way. | 15:16 |
openstackgerrit | Fabien Boucher proposed a change to openstack/tempest: Add tests for swift container listing filters https://review.openstack.org/40416 | 15:16 |
Anju_ | i started from the Api document :mkoderer: u have a good idea about patches | 15:16 |
mriedem | the problem i've had with nova is knowing what's a core api | 15:16 |
mtreinish | sdague, mordred: with https://review.openstack.org/#/c/40417/ I'm hitting on my local environment: http://paste.openstack.org/show/43320/ and nova-api doesn't start | 15:17 |
*** dkranz has joined #openstack-qa | 15:17 | |
clarkb | mtreinish: I think you want 40019 | 15:18 |
clarkb | mtreinish: that at least gives more background on the problem and a potential fix | 15:19 |
mtreinish | clarkb: ok, yeah that seems like it would at least change things. I'll give it a try | 15:19 |
mordred | mtreinish: I think before we land 40019, we might want to pull a few specific known problem children out | 15:20 |
*** psedlak has joined #openstack-qa | 15:34 | |
*** prekarat has joined #openstack-qa | 15:35 | |
*** rhsu has joined #openstack-qa | 15:49 | |
*** prekarat has quit IRC | 15:51 | |
*** prekarat has joined #openstack-qa | 15:51 | |
sdague | actually 40019 is a sledge hammer | 15:53 |
sdague | mtreinish: that paramiko thing should be fixed by the pip install | 15:53 |
sdague | if not, we need to figure out a way to make it so | 15:54 |
openstackgerrit | A change was merged to openstack-dev/devstack: Re-enble pip -e in setup_develop https://review.openstack.org/40417 | 15:56 |
*** dkranz has quit IRC | 16:22 | |
*** tkammer has quit IRC | 16:24 | |
*** dkranz has joined #openstack-qa | 16:35 | |
*** fbo is now known as fbo_away | 16:43 | |
*** krtaylor has quit IRC | 16:53 | |
openstackgerrit | A change was merged to openstack-dev/devstack: Update tempest test sample in devstack/README.md https://review.openstack.org/39820 | 17:00 |
*** dkranz has quit IRC | 17:00 | |
*** dosaboy_alt has quit IRC | 17:07 | |
openstackgerrit | A change was merged to openstack-dev/devstack: Run Neutron migrations after creating schema https://review.openstack.org/40243 | 17:14 |
*** mlavalle has quit IRC | 17:15 | |
*** dkranz has joined #openstack-qa | 17:16 | |
*** afazekas has joined #openstack-qa | 17:20 | |
*** krtaylor has joined #openstack-qa | 17:31 | |
*** SergeyLukjanov has quit IRC | 17:32 | |
*** torandu_ has quit IRC | 17:33 | |
*** SergeyLukjanov has joined #openstack-qa | 17:33 | |
*** torandu has joined #openstack-qa | 17:33 | |
*** SergeyLukjanov has quit IRC | 17:34 | |
openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Add tools/install_pip.sh https://review.openstack.org/39827 | 17:48 |
*** dkranz has quit IRC | 17:57 | |
*** mlavalle has joined #openstack-qa | 17:58 | |
*** Anju_ has quit IRC | 18:01 | |
*** afazekas has quit IRC | 18:02 | |
*** dkranz has joined #openstack-qa | 18:10 | |
*** jhenner has joined #openstack-qa | 18:16 | |
*** SergeyLukjanov has joined #openstack-qa | 18:16 | |
*** jhenner has quit IRC | 18:25 | |
*** jhenner has joined #openstack-qa | 18:25 | |
*** jhenner has quit IRC | 18:32 | |
*** harlowja_ has quit IRC | 18:32 | |
*** jhenner has joined #openstack-qa | 18:34 | |
*** harlowja has joined #openstack-qa | 18:35 | |
*** jhenner has quit IRC | 18:38 | |
jog0 | mtreinish: ping | 18:52 |
*** krtaylor has quit IRC | 19:01 | |
jog0 | mtreinish: I wanted to follow up an the aggregates bug | 19:02 |
mtreinish | jog0: sorry was out getting coffee | 19:14 |
mtreinish | jog0: which bug? There have been a few | 19:15 |
jog0 | mtreinish: the one related to https://review.openstack.org/#/c/36651/19 | 19:17 |
mtreinish | jog0: I pushed a fix for those bugs: https://review.openstack.org/#/c/40424/ I've got to add tests for it though. It's on my list for today | 19:18 |
*** anteaya has quit IRC | 19:19 | |
jog0 | cool | 19:19 |
jog0 | have to step out myself but thanks | 19:19 |
mtreinish | jog0: sure np | 19:19 |
*** rhsu has quit IRC | 19:22 | |
*** krtaylor has joined #openstack-qa | 19:27 | |
*** anteaya has joined #openstack-qa | 19:34 | |
openstackgerrit | A change was merged to openstack-dev/devstack: Some bash versions don't support negative lengths https://review.openstack.org/39236 | 19:36 |
*** krtaylor has quit IRC | 20:10 | |
openstackgerrit | A change was merged to openstack-dev/devstack: Don't uninstall packages when on xenserver https://review.openstack.org/40431 | 20:11 |
*** psedlak has quit IRC | 20:16 | |
*** dosaboy_ has joined #openstack-qa | 20:21 | |
*** dosaboy has quit IRC | 20:24 | |
*** krtaylor has joined #openstack-qa | 21:01 | |
*** SergeyLukjanov has quit IRC | 21:08 | |
*** dosaboy has joined #openstack-qa | 21:27 | |
*** dosaboy_ has quit IRC | 21:28 | |
*** jasondotstar has quit IRC | 21:30 | |
*** woodspa__ has quit IRC | 21:31 | |
*** mriedem has quit IRC | 21:37 | |
*** dosaboy has quit IRC | 21:53 | |
*** mkollaro has quit IRC | 21:58 | |
*** dosaboy has joined #openstack-qa | 22:01 | |
*** giulivo has quit IRC | 22:03 | |
*** burt has quit IRC | 22:10 | |
*** weshay has quit IRC | 22:22 | |
openstackgerrit | A change was merged to openstack/tempest: Added a test for stop and start a server https://review.openstack.org/40191 | 22:26 |
openstackgerrit | A change was merged to openstack/tempest: Make CLI timeout tests configurable https://review.openstack.org/40419 | 22:31 |
*** ianw has joined #openstack-qa | 22:47 | |
stevebaker | hey, are there any reviews yet for the slow-heat gate job? | 22:55 |
*** mlavalle has quit IRC | 23:18 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!