Wednesday, 2014-05-28

*** matsuhashi has joined #openstack-qa00:01
*** mrodden has quit IRC00:09
*** sthaha has joined #openstack-qa00:22
*** aimon has joined #openstack-qa00:22
*** jchai has joined #openstack-qa00:24
openstackgerritGabriel Assis Bezerra proposed a change to openstack-dev/devstack: Fix the way Apache site configuration files are used, to improve OS portability  https://review.openstack.org/9597700:25
*** dmorita has joined #openstack-qa00:36
*** Sukhdev has quit IRC00:38
*** yamahata has joined #openstack-qa00:39
gabriel-bezerraHi guys, might you review my patch, please? ^00:39
openstackgerritGabriel Assis Bezerra proposed a change to openstack-dev/devstack: Fix the way Apache site configuration files are used, to improve OS portability  https://review.openstack.org/9597700:53
openstackgerritGabriel Assis Bezerra proposed a change to openstack-dev/devstack: Fix the way Apache site configuration files are used, to improve OS portability  https://review.openstack.org/9597700:54
*** mlavalle has quit IRC01:02
openstackgerritGabriel Assis Bezerra proposed a change to openstack-dev/devstack: Fix the way Apache site configuration files are used, to improve OS portability  https://review.openstack.org/9597701:09
openstackgerritGabriel Assis Bezerra proposed a change to openstack-dev/devstack: Fix the way Apache site configuration files are used, to improve OS portability  https://review.openstack.org/9597701:13
*** tiantian has joined #openstack-qa01:13
*** weshay has quit IRC01:29
openstackrecheckopenstack/glance change: https://review.openstack.org/91143 failed gate-glance-python26 in the gate queue with an unrecognized error01:30
*** weshay has joined #openstack-qa01:37
*** jchai is now known as jchai_afk01:46
*** nosnos has joined #openstack-qa01:48
*** jchai_afk is now known as jchai02:05
jchaiHello. Could I get another review for https://review.openstack.org/87762 ? It has a +2 . Thanks.02:09
*** gabriel-bezerra has quit IRC02:09
*** nati_uen_ has quit IRC02:13
openstackrecheckConsole logs not available after 13:23s for gate-tempest-dsvm-postgres-full 85366,7,2bd2f9102:15
openstackgerritAaron Rosen proposed a change to openstack/tempest: Upload public image requires admin role by default  https://review.openstack.org/9443502:16
*** weshay has quit IRC02:24
openstackrecheckConsole logs not available after 13:29s for check-grenade-dsvm-neutron 67225,4,16a75ce02:28
*** asselin_ has joined #openstack-qa02:32
*** lbragstad has quit IRC02:33
*** asselin has quit IRC02:33
*** lbragstad has joined #openstack-qa02:33
*** torandu has left #openstack-qa02:35
openstackrecheckConsole logs not available after 13:22s for check-grenade-dsvm-neutron 95904,1,97f604b02:42
openstackrecheckConsole logs not available after 13:25s for check-grenade-dsvm-neutron 94047,15,f0319d402:55
*** dims has quit IRC02:57
*** jchai has quit IRC03:00
openstackrecheckConsole logs not available after 13:24s for check-tempest-dsvm-full-apache-services 95896,1,72a173003:09
*** gabriel-bezerra has joined #openstack-qa03:10
*** harlowja_ is now known as harlowja_away03:16
openstackrecheckConsole logs not available after 13:25s for check-grenade-dsvm-neutron 95467,4,ec245ac03:23
openstackrecheckopenstack/nova change: https://review.openstack.org/86983 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/130277403:23
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add query for bug 1308715  https://review.openstack.org/9591803:23
*** openstackrecheck has quit IRC03:26
*** openstackrecheck has joined #openstack-qa03:26
openstackgerritZhi Kun Liu proposed a change to openstack/tempest: Sharing codes for cinder v1 and v2 tests  https://review.openstack.org/9562903:38
openstackgerritZhi Kun Liu proposed a change to openstack/tempest: Sharing codes for cinder v1 and v2 tests  https://review.openstack.org/9562903:41
openstackrecheckConsole logs not available after 13:26s for check-tempest-dsvm-neutron-heat-slow 64429,20,5ee27b903:42
*** nosnos has quit IRC03:47
*** yfried has joined #openstack-qa03:50
openstackrecheckConsole logs not available after 13:23s for check-tempest-dsvm-full 94945,3,e8cc98e03:56
openstackgerritJamie Lennox proposed a change to openstack-dev/devstack: Use identity_uri instead of auth fragments  https://review.openstack.org/9454504:06
openstackrecheckConsole logs not available after 13:24s for check-tempest-dsvm-neutron 95977,5,99e7b6f04:09
*** afazekas has quit IRC04:12
openstackgerritHaiyang DING proposed a change to openstack/tempest: Add verification to name property in cinder volume  https://review.openstack.org/9571404:12
*** matsuhashi has quit IRC04:15
*** yfried has quit IRC04:16
*** psharma has joined #openstack-qa04:22
openstackrecheckConsole logs not available after 13:26s for check-grenade-dsvm-neutron 92390,5,e4caf3704:23
*** matsuhashi has joined #openstack-qa04:26
*** nosnos has joined #openstack-qa04:27
*** bkopilov has quit IRC04:29
*** k4n0 has joined #openstack-qa04:30
openstackrecheckConsole logs not available after 13:26s for check-grenade-dsvm 89715,2,bba325504:36
*** GMann has joined #openstack-qa04:38
*** afazekas has joined #openstack-qa04:40
*** thomasbiege has joined #openstack-qa04:51
*** yfried has joined #openstack-qa04:51
*** yfried has quit IRC04:53
*** ashishsnigam has joined #openstack-qa04:59
openstackrecheckConsole logs not available after 13:27s for check-tempest-dsvm-neutron-heat-slow 95994,1,312053a05:01
*** ashishsnigam has quit IRC05:07
*** thomasbiege has quit IRC05:09
*** ashishsnigam has joined #openstack-qa05:11
*** ashishsnigam has quit IRC05:12
*** thomasbiege has joined #openstack-qa05:12
*** ashishsnigam has joined #openstack-qa05:12
*** krtaylor has quit IRC05:15
*** thomasbiege has quit IRC05:17
*** ashishsnigam has quit IRC05:17
*** ashishsnigam has joined #openstack-qa05:18
*** Madan has joined #openstack-qa05:20
*** tiantian has quit IRC05:21
*** ashishsnigam has quit IRC05:23
openstackrecheckConsole logs not available after 13:34s for check-grenade-dsvm-neutron 96000,1,6d881b705:24
openstackrecheckConsole logs not available after 13:23s for check-tempest-dsvm-neutron-full 81707,4,387998805:39
*** Madan has quit IRC05:42
*** Madan has joined #openstack-qa05:46
*** markmcclain has joined #openstack-qa05:52
*** markmcclain1 has joined #openstack-qa05:54
*** markmcclain has quit IRC05:57
*** markmcclain1 has quit IRC06:01
*** ashishsnigam has joined #openstack-qa06:03
openstackrecheckConsole logs not available after 13:33s for check-swift-dsvm-functional 96030,1,fed175c06:07
*** tiantian has joined #openstack-qa06:19
openstackrecheckConsole logs not available after 13:24s for check-grenade-dsvm-neutron 96026,1,aa2b33206:20
*** tkammer has joined #openstack-qa06:21
*** SridharG has joined #openstack-qa06:32
*** sthaha has quit IRC06:32
openstackrecheckopenstack/heat change: https://review.openstack.org/94329 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1254890, gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/122641206:34
openstackgerritajay proposed a change to openstack/tempest: Added cinder cli tests  https://review.openstack.org/9607206:38
openstackgerritAshish Nigam proposed a change to openstack/tempest: Add basic read only tests for trove cli  https://review.openstack.org/9459906:43
*** ajay_ has joined #openstack-qa06:44
*** boris-42 has quit IRC06:45
*** yfried has joined #openstack-qa06:48
*** boris-42 has joined #openstack-qa06:49
*** yfried has quit IRC06:49
*** yfried has joined #openstack-qa06:49
openstackrecheckConsole logs not available after 13:38s for gate-python-keystoneclient-python27 74908,10,9cd604d06:50
*** yfried_ has joined #openstack-qa06:50
*** yfried has quit IRC06:54
*** yfried_ is now known as yfried06:57
*** evgenyf has joined #openstack-qa06:58
*** tkammer has quit IRC07:01
openstackrecheckopenstack/sahara change: https://review.openstack.org/95861 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/125487207:08
*** yrabl has joined #openstack-qa07:14
*** yrabl has quit IRC07:19
*** Albert_13 has joined #openstack-qa07:20
*** jlibosva has joined #openstack-qa07:25
*** yrabl has joined #openstack-qa07:26
*** gszasz has joined #openstack-qa07:28
*** bkopilov has joined #openstack-qa07:29
*** giulivo has joined #openstack-qa07:38
*** yfried has quit IRC07:53
openstackgerritafazekas proposed a change to openstack-dev/devstack: Faster nova fixed key generation  https://review.openstack.org/9609807:54
*** markmcclain has joined #openstack-qa07:55
openstackgerritafazekas proposed a change to openstack-infra/elastic-recheck: Add query for 1320617  https://review.openstack.org/9408007:58
*** jpich has joined #openstack-qa07:58
*** markmcclain has quit IRC08:00
*** tkammer has joined #openstack-qa08:02
*** jordanP has joined #openstack-qa08:03
*** yfried has joined #openstack-qa08:04
*** yfried has quit IRC08:09
*** coolsvap|afk is now known as coolsvap08:11
*** matsuhashi has quit IRC08:14
*** matsuhashi has joined #openstack-qa08:15
*** matsuhashi has quit IRC08:17
*** matsuhashi has joined #openstack-qa08:17
*** andreaf has joined #openstack-qa08:18
*** yfried has joined #openstack-qa08:22
*** sthaha has joined #openstack-qa08:22
*** mmaglana has quit IRC08:27
*** amotoki has joined #openstack-qa08:30
*** markmcclain has joined #openstack-qa08:50
*** eezhova has joined #openstack-qa08:51
*** tosky has joined #openstack-qa08:52
*** GMann has quit IRC09:09
*** sthaha has quit IRC09:16
*** yamahata has quit IRC09:22
*** thomasbiege has joined #openstack-qa09:24
*** markmcclain has quit IRC09:24
*** gszasz has quit IRC09:37
*** thomasbiege has quit IRC09:37
*** ashishsnigam has quit IRC09:43
*** boris-42 has quit IRC09:52
*** k4n0 has quit IRC09:54
*** markmcclain has joined #openstack-qa09:54
*** thomasbiege has joined #openstack-qa09:55
*** thomasbiege has quit IRC09:57
*** mkollaro has joined #openstack-qa10:01
openstackgerritnayna-patel proposed a change to openstack/tempest: Adds v2 keystone service test to service v3  https://review.openstack.org/9037010:06
*** k4n0 has joined #openstack-qa10:07
*** boris-42 has joined #openstack-qa10:14
*** jchai has joined #openstack-qa10:17
openstackgerritnayna-patel proposed a change to openstack/tempest: Adds v2 keystone service test to service v3  https://review.openstack.org/9037010:17
*** gszasz has joined #openstack-qa10:22
*** mmaglana has joined #openstack-qa10:28
*** k4n0 has quit IRC10:30
*** mmaglana has quit IRC10:33
openstackgerritVictor Stinner proposed a change to openstack/tempest: Fix nova unit tests to support multiple networks  https://review.openstack.org/9613110:37
*** k4n0 has joined #openstack-qa10:37
*** thomasbiege has joined #openstack-qa10:41
openstackgerritVictor Stinner proposed a change to openstack/tempest: Fix nova unit tests to support multiple networks  https://review.openstack.org/9613110:44
*** matsuhashi has quit IRC10:48
*** jchai is now known as jchai_afk10:48
*** thomasbiege has quit IRC10:49
*** markmcclain has quit IRC10:50
*** luqas has joined #openstack-qa10:52
openstackgerritVictor Stinner proposed a change to openstack/tempest: Fix nova unit tests attaching an interface  https://review.openstack.org/9613510:53
openstackgerritYaroslav Lobankov proposed a change to openstack/tempest: Sahara: add changes to cli tests  https://review.openstack.org/9513410:58
*** malini_afk is now known as malini11:02
*** ylobankov has joined #openstack-qa11:04
*** dmorita has quit IRC11:11
*** jchai_afk is now known as jchai11:11
*** jchai has quit IRC11:16
*** malini is now known as malini_afk11:19
*** thomasbiege has joined #openstack-qa11:22
*** samuelbercovici has joined #openstack-qa11:23
*** samuelbercovici1 has joined #openstack-qa11:26
*** samuelbercovici has quit IRC11:29
*** samuelbercovici1 is now known as samuelbercovici11:29
*** samuelbercovici1 has joined #openstack-qa11:32
*** coolsvap is now known as coolsvap|afk11:33
*** samuelbercovici has quit IRC11:35
*** samuelbercovici1 is now known as samuelbercovici11:35
openstackgerritSean Dague proposed a change to openstack-dev/devstack: remove the build timeouts  https://review.openstack.org/9614311:42
*** sthaha has joined #openstack-qa11:43
*** dims has joined #openstack-qa11:45
*** samuelbercovici1 has joined #openstack-qa11:46
openstackgerritGabriel Assis Bezerra proposed a change to openstack-dev/devstack: Fix the way Apache site configuration files are used, to improve OS portability  https://review.openstack.org/9597711:47
*** samuelbercovici has quit IRC11:49
*** samuelbercovici1 is now known as samuelbercovici11:49
*** thomasbiege has quit IRC11:51
*** samuelbercovici1 has joined #openstack-qa11:54
*** samuelbercovici has quit IRC11:57
*** samuelbercovici1 is now known as samuelbercovici11:57
openstackgerritArtur Svechnikov proposed a change to openstack/tempest: Add Ceilometer client for scenarios tests  https://review.openstack.org/9210812:01
*** markmcclain has joined #openstack-qa12:02
*** markmcclain1 has joined #openstack-qa12:04
*** markmcclain1 has quit IRC12:04
*** markmcclain1 has joined #openstack-qa12:04
*** markmcclain has quit IRC12:06
*** k4n0 has quit IRC12:13
*** psharma has quit IRC12:15
*** aysyd has joined #openstack-qa12:18
*** krtaylor has joined #openstack-qa12:19
*** thomasbiege has joined #openstack-qa12:20
*** enikanorov_ has quit IRC12:21
*** weshay has joined #openstack-qa12:23
*** sthaha has quit IRC12:24
*** sthaha has joined #openstack-qa12:24
*** sthaha has quit IRC12:25
*** sthaha has joined #openstack-qa12:25
*** mmaglana has joined #openstack-qa12:29
*** mmaglana has quit IRC12:33
*** markmcclain1 has quit IRC12:36
*** jchai has joined #openstack-qa12:39
*** dims has quit IRC12:39
*** andreaf has quit IRC12:41
*** thomasbiege has quit IRC12:44
*** ekhugen has joined #openstack-qa12:49
*** nosnos has quit IRC12:50
*** yfried has quit IRC13:10
*** mriedem has joined #openstack-qa13:10
*** Madan has quit IRC13:11
openstackgerritVadim Rovachev proposed a change to openstack/tempest: Add compute notifications tests for ceilometer  https://review.openstack.org/6413613:17
*** bkopilov has quit IRC13:18
*** bkopilov has joined #openstack-qa13:19
*** jaypipes has quit IRC13:20
*** Obulpathi has joined #openstack-qa13:22
*** jaypipes has joined #openstack-qa13:23
*** ChanServ sets mode: +o jaypipes13:23
*** malini_afk is now known as malini13:24
*** Obulpathi has quit IRC13:26
openstackgerritMasayuki Igawa proposed a change to openstack/qa-specs: WIP: Add tempest server/client/GUI spec  https://review.openstack.org/9616313:29
*** cdent has joined #openstack-qa13:29
openstackgerritMasayuki Igawa proposed a change to openstack/qa-specs: WIP: Add tempest server/client/GUI spec  https://review.openstack.org/9616313:31
*** tkammer has quit IRC13:31
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Add a hacking check to block scenario manager in api tests  https://review.openstack.org/9616413:32
*** dkorn has joined #openstack-qa13:32
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Add a hacking check to block scenario manager in api tests  https://review.openstack.org/9616413:33
openstackgerritJordan Pittier proposed a change to openstack/tempest: Scenario : add shelve/unshelve in TestSnapshotPattern  https://review.openstack.org/9616513:36
*** samuelbercovici1 has joined #openstack-qa13:39
*** jordanP has quit IRC13:39
*** samuelbercovici has quit IRC13:42
*** samuelbercovici1 is now known as samuelbercovici13:42
*** tkammer has joined #openstack-qa13:45
*** samuelbercovici1 has joined #openstack-qa13:47
*** Obulpathi has joined #openstack-qa13:48
*** samuelbercovici has quit IRC13:50
*** samuelbercovici1 is now known as samuelbercovici13:50
openstackgerritMarc Koderer proposed a change to openstack/tempest: Add decorator for safe tearDown  https://review.openstack.org/8464513:51
*** andreaf has joined #openstack-qa13:53
*** rbak has joined #openstack-qa13:59
*** yfried has joined #openstack-qa14:00
*** amotoki has quit IRC14:00
andreafmtreinish, sdague: ping14:03
mtreinishandreaf: what's up?14:04
*** ekhugen has quit IRC14:04
andreafmtreinish: re the use of official clients vs. tempest ones14:04
*** enikanorov_ has joined #openstack-qa14:04
mtreinishandreaf: heh, did you see my hacking patch?14:05
andreafmtreinish: what do you think about tempest.common14:05
andreafmtreinish: yes14:05
mtreinishandreaf: so it has to be the tempest clients for the api tests. There really isn't an exception to that14:05
*** ajay_ has quit IRC14:05
andreafmtreinish: and I'm reviewing https://review.openstack.org/#/c/96131 which uses official clients for translating network name to network id14:05
mtreinishandreaf: yeah that's what spawned my hacking check14:06
andreafok there are things in common which can be used by both api and scenario - should they behave differently depending on the caller?14:06
andreafthis is what I did in auth.py, but common is in a mixed state now14:07
mtreinishandreaf: so that's the tricky bit, the official clients behave differently. We say the scenario tests use the official clients, but we're not as tough on that. (Although I don't think it's ever come up)14:07
*** tkammer has quit IRC14:07
mtreinishideally you'd change hwat's used depending on the caller14:07
mtreinishI did that for tenant isolation too14:07
andreafmtreinish: so we should have tempest.common.api and tempest.common.scenario?14:08
mtreinishandreaf: I just did if checks...14:09
mtreinishand had an optional param to set which client was being used14:09
andreafmtreinish: tenant isolation works with official clients only (just an example)14:09
mtreinishyeah, if you set tempest_client=False when you call init: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/common/isolated_creds.py#n3014:11
mtreinishit will only use the official clients14:11
andreafmtreinish: what about some adaptation layer for common classes?14:11
mtreinishI'm sure there are better ways to do it :)14:11
andreafmtreinish: oh so it actually works with both kind of clients... good :)14:12
andreafmtreinish: ok anyways I just wanted to clarify the policy for the review14:13
*** markmcclain has joined #openstack-qa14:13
andreafmtreinish: thanks14:13
mtreinishsure, np14:14
mtreinishandreaf: I always like to use that file as an example of why I hate the official clients. Everything is so different between all the clients14:15
mtreinisheverything is always different with them14:15
openstackrecheckopenstack/requirements change: https://review.openstack.org/94722 failed because of: gate-tempest-dsvm-postgres-full: unrecognized error, gate-requirements-pypy: https://bugs.launchpad.net/bugs/131546814:16
sdagueso I actually think we should try to build some tempest client isolation into a tempest.client namespace14:18
sdaguebecause I think people would actually like to consume our client in other test environments, and I think that's a sane idea14:18
*** ekhugen has joined #openstack-qa14:19
*** yamahata has joined #openstack-qa14:20
mtreinishI'm not opposed to that, but what exactly would that entail. A wrapper around tempest.serverices to make a more friendly interface to us externally?14:21
*** tkammer has joined #openstack-qa14:22
mtreinishsdague: haven't you been fighting how closely coupled things are to the tempest tree though with the javelin stuff. I'm not sure how feasible it'll be to make the clients work on their own.14:24
*** samuelbercovici1 has joined #openstack-qa14:25
*** ekhugen has quit IRC14:26
*** samuelbercovici has quit IRC14:27
*** samuelbercovici1 is now known as samuelbercovici14:27
*** markmcclain has quit IRC14:29
*** mmaglana has joined #openstack-qa14:30
gabriel-bezerraHi guys. I can' t have my patch to devstack properly tested because Jenkins only applies that after a stable branch is used. Can you help me? https://review.openstack.org/9597714:32
gabriel-bezerraIt changes the way apache services are configured.14:33
*** markmcclain has joined #openstack-qa14:33
*** mmaglana has quit IRC14:35
*** mlavalle has joined #openstack-qa14:35
sdaguemtreinish: so actually, I think we could decouple14:39
*** SridharG has quit IRC14:40
sdaguethe coupling is mostly eggregious at this point14:40
*** mrodden has joined #openstack-qa14:41
gabriel-bezerraHow are upgrades managed in devstack? Is there any place to put upgrading code?14:43
sdaguegabriel-bezerra: grenade uses the icehouse config for the juno setup. So any method of adding things to devstack should be backwards compatible there.14:45
sdaguedtroyer: https://review.openstack.org/#/c/95934/ - you good with this before taking off?14:45
dtroyeryes…did I really do that? ugh...14:47
dtroyerBTW, I'm trying to get through ceph/sheepdog this morning yet…14:48
sdagueno, I did it14:48
sdagueI was trying to add an exit hook for debug, and completely flubbed that bit14:49
dtroyer<phew>  now I can sleep better tonight driving across western Kansas, aka, Nothingville14:50
*** yrabl has quit IRC14:51
*** jgrimm has joined #openstack-qa14:52
sdague:)14:53
*** tkammer has quit IRC14:55
*** markmcclain has quit IRC14:55
openstackgerritafazekas proposed a change to openstack-dev/devstack: soft-qemu handling with F20 could image  https://review.openstack.org/9618014:59
*** afazekas has quit IRC15:10
*** kgriffs|afk is now known as kgriffs15:13
*** ekhugen has joined #openstack-qa15:16
*** bkopilov has quit IRC15:21
*** rdekel has joined #openstack-qa15:23
sdaguequestion... I'd really like to use the normal asserts for validating the javelin resources, however those are all hidden under the test classes. I wonder if there is a sane way to do that.15:24
openstackrecheckopenstack/requirements change: https://review.openstack.org/95974 failed gate-tempest-dsvm-postgres-full in the gate queue with an unrecognized error15:28
*** rdekel has quit IRC15:30
*** mmaglana has joined #openstack-qa15:31
*** samuelbercovici1 has joined #openstack-qa15:35
*** eglynn has joined #openstack-qa15:37
*** samuelbercovici has quit IRC15:38
*** samuelbercovici1 is now known as samuelbercovici15:38
eglynnhi folks15:40
eglynnanyone got time for a quick question on the implications of "branchless tempest"?15:40
*** Sukhdev has joined #openstack-qa15:40
eglynnvrovachev and I were just discussing this on the #openstack-ceilometer channel15:40
vrovachevsdague, mtreinish - yes15:41
vrovachevwe have problems with tempest + ceilo integration15:41
gabriel-bezerrasdague: My build is breaking with an error on upgrade-glance, it is not running with my new lib/apache version, so the new lib/keystone breaks. Do you have any idea whether this is my fault or grenade's?15:41
eglynnTBH I'm a bit confused on the notion of tempest:master running against stable/icehouse also15:41
vrovachevsdague, mtreinish, eglynn - our tests won't work with icehouse -> as there is previous cycle job in tempest...15:42
vrovachevwe have no many choices here15:42
eglynn... to my naive understanding, that seems to block progress on landing tempest coverage of stuff that's known to work on master15:42
gabriel-bezerrasdague: should lib/keystone source lib/apache (it currently doesn't do that)?15:42
sdagueeglynn: the openstack API is the openstack API15:42
morganfainberggabriel-bezerrab, it doesn't? how does it do the apache setup then?15:43
sdagueit shouldn't change between versions15:43
vrovachevit's not about api15:43
eglynnsdague: ... this isn't an API compatability issue15:43
vrovachevit's about the performance15:43
eglynnsdague: ... rather a quality-of-implementation issue15:43
morganfainberggabriel-bezerra, *is suddently confused* i need to reread those scripts.15:43
vrovachevneeded change might be possible only in new juno features - we can't backport these things to icehouse ceilo15:44
sdagueeglynn: is there a specific instance we can discuss?15:44
eglynnsdague: ... backporting the improvements to the ceilo sql-a driver from master to stable/icehouse is not going to viable in all cases15:44
openstackgerritDavid Paterson proposed a change to openstack/qa-specs: Add spec for bp:post-run-cleanup  https://review.openstack.org/9172515:44
sdagueeglynn: ok, but point at a specific test that's proposed that's problematic here15:44
eglynnsdague: ... for example, we've gotten a big performance improvement from the schema simplifications possible after removing support for the ceilometer v1 API15:45
sdagueok15:45
eglynnsdague: here's a test https://review.openstack.org/#/c/6413615:45
eglynnsdague: ... that v1 removal change cannot be backported15:45
eglynnsdague: ... as that would break policy around the deprecation path15:46
eglynnsdague: ... and would require that schema migrations are backported (also problematic)15:46
eglynnsdague: ... I mean "the deprecation path" for the v1 API15:46
sdagueso the issue is that these tests can't run in < 60s in icehouse?15:46
eglynnsdague: ... those tests couldn't be landed during icehouse due to the performance issue that we're now making progress on solving15:47
*** torandu has joined #openstack-qa15:47
vrovachevsdague - here all ceilo tests https://review.openstack.org/#/q/owner:vrovachev+project:openstack/tempest+status:open,n,z15:47
eglynnsdague: ... but of course those performance issues still exist on stable/icehouse15:47
vrovachevsdague - 60s improvement won't help here - we'll need at least 18min range..15:48
vrovachevfor every test!15:48
vrovachevit'll be endless15:48
eglynnsdague: ... so we've tests that were completely unviable to run in the gate for the reasons vrovachev points out15:49
eglynnsdague: ... they are now becoming viable, but only against master15:49
sdagueso the way we've thought about this so far is there needs to be some end user signal that these tests can possibly run. Because how do you decide what to run against a cloud in the wild.15:52
sdaguewhat would be some good signaling there?15:52
eglynnsdague: "some end user signal" ==> as in test.attr tagging or something of that ilk encoded in the testcase itself?15:53
sdaguehow would a user know whether or not they should expect these tests to succeed?15:54
vrovachevwe were thinking about this job disabling... it simply can't work... dunno how this is possible15:54
*** afazekas has joined #openstack-qa15:54
*** mmaglana has quit IRC15:55
*** mkollaro has quit IRC15:56
*** mmaglana has joined #openstack-qa15:56
eglynnvrovachev: ... would it suffice to disable selected test cases with a test.attr(type='only_works_on_master_so_skip_on_stable_*') ?15:58
gabriel-bezerramorganfainberg: I don't know how that gets in the environment variables, but they are there and work when one ./stack.sh s15:58
*** tkammer has joined #openstack-qa15:59
eglynn... where 'only_works_on_master_so_skip_on_stable_*' is replaced by something a bit more punchy ;)15:59
vrovacheveglynn, possibly - if it's there I can try to15:59
morganfainberggabriel-bezerra, ah i think stack.sh sources it early15:59
vrovachevor do I need to impl this?15:59
eglynnsdague: ... is that kind of tagging approach suggested above ^^^ feasible in the gate?15:59
gabriel-bezerrain grenade it doesn't work though15:59
vrovachevsdague - is there this type for the test.attr?16:00
sdagueeglynn: and what value would you set for testing hp public cloud?16:00
sdagueor rax, or cloudwatt16:00
*** mmaglana has quit IRC16:00
gabriel-bezerramorganfainberg: grenade does "source $TARGET_DEVSTACK_DIR/lib/keystone" but doesn't do the same with apache16:01
gabriel-bezerrain upgrade-glance16:01
eglynnsdague: ... are you thinking in terms of Tempest==DefCore, as opposed to Tempest==upsteam gate?16:01
sdagueeglynn: I'm thinking as Tempest test suite16:01
jvrbanacHowdy all! I'm just getting started with putting together tests that work with Tempest for the Barbican project. Is there a good example or boilerplate for me to get started. I want to make sure I'm doing this right. I was looking through the docs and didn't see anything regarding starting from scratch.16:02
sdagueTempest is designed to work on any openstack cloud16:02
sdagueit's not just the gate16:02
sdaguepeople run it on their clouds regularly for monitoring16:02
eglynnsdague: "designed to work on any openstack cloud" == "designed to run against any openstack cloud *of any version*"?16:03
sdagueeglynn: that's the point of branchless tempest16:03
sdagueyes, of any version, as long as the API versions are supported16:03
*** ildikov has joined #openstack-qa16:03
sdagueNova v2 API should be the same, regardless of release version. And when it isn't that's a bug in Nova.16:04
*** Albert_13 has quit IRC16:04
sdagueanyway, have to run off to lunch now. Can chat about this more once I get back.16:04
eglynnsdague: ok, thanks ... ping me when you're available again16:05
*** aimon has quit IRC16:05
*** atiwari has joined #openstack-qa16:06
gabriel-bezerramorganfainberg: Should it source lib/apache then?16:08
*** SridharG has joined #openstack-qa16:19
openstackrecheckopenstack/nova change: https://review.openstack.org/86983 failed gate-nova-pep8 in the gate queue with an unrecognized error16:25
openstackrecheckopenstack/cinder change: https://review.openstack.org/95333 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/130277416:26
gabriel-bezerrasdague, morganfainberg ^16:27
morganfainberggabriel-bezerra, oh. i see uhm...16:27
morganfainberggabriel-bezerra, if it legitimately relies on apache, probably.16:28
*** bearhands is now known as comstud16:32
*** dkorn has quit IRC16:32
*** evgenyf has quit IRC16:34
*** jpich has quit IRC16:34
gabriel-bezerramorganfainberg: It is not a must, but Keystone may be deployed on Apache and devstack supports that. When that happens, lib/keystone depends on lib/apache.16:35
morganfainberggabriel-bezerra, ok for future proofing, it should source lib/apache for keystone purposes (we are moving towards keystone being deployed under apache as the default)16:37
morganfainberggabriel-bezerra, i'm not as familiar with genade as i should be tbh.16:37
morganfainberg:(16:37
gabriel-bezerramorganfainberg: is there any problem in sourcing lib/apache multiple times in a stack/grenade run?16:38
morganfainberggabriel-bezerra, there might be. like i said, not as familair as i should be with it16:39
gabriel-bezerraI can't see any by looking at lib/apache code16:39
gabriel-bezerrabut I'd like to know from you whether it should be avoided16:40
gabriel-bezerraor should grenade source that instead of lib/keystone16:40
gabriel-bezerra?16:40
gabriel-bezerrasdague ^16:41
morganfainberggabriel-bezerra, ok, looking at lib/apache, it should eb safe to source multiple times.16:43
morganfainberggabriel-bezerra, (have it open now)16:43
*** eglynn-afk has joined #openstack-qa16:43
morganfainberggabriel-bezerra, the convention appears that lib/XXX does not source lib/apache but lib/apache is sources before lib/XXX (e.g. XXX= keystone)16:44
SergeyLukjanovsdague, mtreinish, please, take a look at https://review.openstack.org/#/c/94311/16:44
morganfainberggabriel-bezerra, so that if you need lib/apache it's available16:44
gabriel-bezerrawell.. I'll do that then. Hope it to be clarified in the review, should it be a wrong step.16:44
morganfainberggabriel-bezerra, i would say that sourcing lib/apache prior to lib/keystone e.g. how stack.sh works https://github.com/openstack-dev/devstack/blob/master/stack.sh#L336 would be correct16:45
morganfainberggabriel-bezerra, sorry was juggling a couple things at once and needed to switch to looking at devstack to give you a good answer.16:46
*** DinaBelova has joined #openstack-qa16:46
*** jlibosva has quit IRC16:46
*** mmaglana has joined #openstack-qa16:47
*** tkammer has quit IRC16:49
*** luqas has quit IRC16:51
*** mlavalle has quit IRC16:51
*** samuelbercovici has quit IRC16:52
eglynnsdague: I need to run shortly so can I just throw an idea out on the table?16:54
eglynnsdague: so I was just reading the branchless tempest BP spec https://github.com/openstack/qa-specs/blob/master/specs/branchless-tempest.rst16:54
eglynnsdague: and here's an idea ...16:55
*** aimon has joined #openstack-qa16:55
eglynnsdague: ... what if we treat these problematic ceilometer tests as falling under "Scenario 1: New Tests for new features"?16:55
eglynnsdague: ... the new feature being: "Run fast enough for gate testing to be viable"16:55
eglynnsdague: ... effectively treating the ceilometer sqlalchemy driver in all versions prior to Juno as being essentially an experimental/PoC thing16:55
DinaBelovaeglynn, absolutely ++ - if that's possible to be used here - it'll be cool16:55
eglynnsdague: now the branchless tempest BP isn't specific as to mechanics of the skipping for new feature tests on stable/icehouse16:56
eglynnsdague: it just states ... "this test will need to be behind a feature flag (as the feature wasn't available in Icehouse)."16:56
eglynnsdague: ... if that "feature flag" maps on a test.attr decorator or some-such, then this may be workable?16:56
openstackgerritA change was merged to openstack-dev/devstack: chown stack/data/nova/networks to $STACK_USER  https://review.openstack.org/9591416:57
*** nati_ueno has joined #openstack-qa16:57
openstackrecheckopenstack/nova change: https://review.openstack.org/72375 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/122641216:57
eglynnsdague: I need dash off now, ~6pm in my TZ (... I'll check in on the eavesdrop logs when I get back online)16:59
*** eglynn has quit IRC17:01
*** yamahata has quit IRC17:03
*** andreaf has quit IRC17:06
*** ildikov has quit IRC17:09
*** andreaf has joined #openstack-qa17:10
*** andreaf has quit IRC17:10
*** andreaf has joined #openstack-qa17:10
*** andreaf has quit IRC17:11
*** andreaf has joined #openstack-qa17:12
*** Obulpathi has quit IRC17:13
*** harlowja_away is now known as harlowja_17:16
*** Obulpathi has joined #openstack-qa17:17
openstackgerritMonty Taylor proposed a change to openstack-dev/devstack: Update get-pip url  https://review.openstack.org/9621117:20
*** vponomaryov has quit IRC17:24
*** vponomaryov has joined #openstack-qa17:25
mtreinisheglynn-afk: I think that's probably the right approach. BUt, you'll need to document that somewhere on the ceilometer side, about it being a poc prior to juno.17:25
mtreinisheglynn-afk: the way the we do the skips based on feature flag is pretty simple, there is no need to add a separate tag for it17:26
*** bkopilov has joined #openstack-qa17:26
mtreinisheglynn-afk: we normally just use a skip decorator that checks the config flag, or we raise a skip exception from an if block17:27
mtreinishthere are a bunch of examples of both in the tree17:27
*** levi_b has joined #openstack-qa17:27
sdagueeglynn-afk: right. I guess the real question is what we call the flag, and how someone would know to set it. Is it "ceilometer_not_too_slow" or something. Because it's not really clear how an end user would know if their cloud did this or not.17:27
sdagueI think that's a challenge here in that while the surface didn't change, the semantics did. Being 2 orders of magnitude faster is different.17:28
sdagueand if someone wrote an application that worked, and depending on those timings, and then pointed it at a icehouse cloud, which in theory has all the same surfaces, how would they know that their app was going to not work because it would timeout17:29
*** bkopilov has quit IRC17:35
*** Sukhdev has quit IRC17:40
*** gszasz has quit IRC17:40
*** tkammer has joined #openstack-qa17:43
*** ildikov has joined #openstack-qa17:44
*** bkopilov has joined #openstack-qa17:49
openstackrecheckopenstack/nova change: https://review.openstack.org/95206 failed gate-tempest-dsvm-postgres-full in the gate queue with an unrecognized error17:50
gabriel-bezerramorganfainberg: so that should be done on grenade/upgrade-glance, not in lib/keystone. Right?17:51
*** kgriffs is now known as kgriffs|afk17:54
openstackgerritSean Dague proposed a change to openstack/tempest: wip: javelin 2  https://review.openstack.org/9514817:59
openstackgerritRob Crittenden proposed a change to openstack-dev/devstack: Configure heat keystone client for SSL when keystone uses SSL  https://review.openstack.org/9621618:10
openstackgerritafazekas proposed a change to openstack/qa-specs: bp/net-debug DRAFT  https://review.openstack.org/9579418:14
*** jamielennox is now known as jamielennox|away18:16
*** luqas has joined #openstack-qa18:17
*** nati_ueno has quit IRC18:24
*** SridharG has quit IRC18:24
*** tkammer has quit IRC18:24
*** SridharG has joined #openstack-qa18:25
*** nati_ueno has joined #openstack-qa18:30
gabriel-bezerradtroyer, what do you mean by "It feels like the use of APACHE_SITE_DIR and APACHE_NAME in apache_site_config_for() should be noted somewhere." in https://review.openstack.org/95977 ?18:40
dtroyerThey are globals that are assumed to be declared for that function to work.  We've tried to mark those somewhere, maybe just note them in the function comments?18:41
sdaguemtreinish: the image clients in tempest are confusing me a bit. It looks like there is no way to set properties in v2, but there is also no explicit store call in v118:44
mtreinishsdague: hmm, let me take a look18:45
*** dkorn has joined #openstack-qa18:45
sdaguebasically creating an ami 3 part image via tempest clients that actually works... is challenging18:45
openstackgerritafazekas proposed a change to openstack-dev/devstack: soft-qemu handling with F20 could image  https://review.openstack.org/9618018:46
*** gabriel-bezerra has quit IRC18:47
mtreinishsdague: yeah it looks like the v2 client doesn't support any optional metadata being added to the create call (except visibility)18:49
sdaguemtreinish: if I understood the v2 update call on setting properties18:49
mtreinishbut the update call does for v218:49
sdagueany idea what that namespace is supposed to look like?18:49
mtreinishon v1 there is no separate store call you just pass it the image data for create or update18:49
*** gabriel-bezerra has joined #openstack-qa18:50
gabriel-bezerradtroyer: ok, I'll put that18:51
dtroyergabriel-bezerra: thanks18:52
mtreinishsdague: this is the only v2 update I see in tree: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/image/v2/test_images.py#n11718:52
mtreinishwhich matches up with: http://docs.openstack.org/api/openstack-image-service/2.0/content/restricted-json-pointers.html18:52
mtreinishand http://docs.openstack.org/api/openstack-image-service/2.0/content/update-an-image.html18:53
*** mlavalle has joined #openstack-qa18:53
mtreinishsdague: although loosening the v2 create call is probably a fair bug fix18:53
sdaguemtreinish: yeh, I found finding the same issue18:53
mtreinishbecause only using visibility from kwargs seems a bit restrictive18:54
sdaguemaybe, though this is properties in a different namespace IIRC18:55
sdagueI could revert to v118:55
mtreinishsdague: which properties do you need to set for the ami image parts though? It should just be format and container right?18:57
mtreinishoh, nm you need the metadata to link them18:57
*** shardy is now known as shardy_afk18:57
mtreinishthe various ids of the different parts18:57
*** jhenner has quit IRC18:57
sdagueyeh, I need the ramdisk_id and kernel_id set as properties18:58
*** jhenner has joined #openstack-qa18:58
*** levi_b_ has joined #openstack-qa18:58
*** eglynn has joined #openstack-qa18:58
*** levi_b_ has quit IRC18:58
eglynnsdague: just catching up with backlog, thanks for the response!18:59
*** levi_b_ has joined #openstack-qa18:59
eglynnsdague: cool, sounds like we've a basis to move forward ...18:59
eglynnsdague: ... I'll familiarize myself with those examples in-tree of skipping new features18:59
eglynnsdague: ... on the point about an existing application depending on the previous slowness19:00
eglynnsdague: ... note that much of the speed has been gained on the "write side"19:00
mtreinishsdague: yeah I would just change the loop in the v2 create honestly. So it loops over kwargs instead of a preset list of know opts19:00
eglynnsdague: ... i.e. in the pipeline for collecting and persisting metering data19:00
sdagueeglynn: ok19:01
eglynnsdague: (... which means that samples appear in the metering store faster than they would done previously, as opposed to an API-call-timing-out situation)19:01
eglynnsdague: ... though such timeouts could have happened in past too :(19:02
mtreinishsdague: it doesn't look like v1 has support for those params either...19:02
mtreinishit does the same hard coded list thing19:02
*** levi_b has quit IRC19:02
sdaguemtreinish: yeh, but I could to the properties update19:02
*** atiwari has quit IRC19:02
sdaguecould do image_update on 'properties'19:02
openstackgerritMatt Riedemann proposed a change to openstack/tempest: Consolidate _check_tenant_network_connectivity in NetworkScenarioTest  https://review.openstack.org/9584419:03
openstackgerritMatt Riedemann proposed a change to openstack/tempest: Consolidate _check_public_network_connectivity into NetworkScenarioTest  https://review.openstack.org/9583119:03
*** levi_b_ has quit IRC19:03
mtreinishsdague: yeah give that a try to start I guess.19:03
*** Sukhdev has joined #openstack-qa19:04
sdagueso I'm going to change create, because I have no idea how to do the update correctly in v219:06
*** levi_b has joined #openstack-qa19:06
*** dims has joined #openstack-qa19:07
mtreinishsdague: heh, you could always ask in -glance, I'm sure they'd know off the top of their head...19:08
sdaguehmmm that didn't do the right thing19:08
sdagueI asked in -dev19:08
sdagueI honestly don't think it's reasonable to expect for anyone to figure out anything they have to be in 30 openstack channels :)19:09
mtreinishheh, I'm only up to r on my client (which I think is 14)19:10
sdagueok, I give up, there is actually no public docs on v2.19:15
sdagueI actually think that should be a criteria for landing tests in tempest, that there are API docs that can be pointed at.19:15
*** mkollaro has joined #openstack-qa19:15
mtreinishsdague: is the wip you posted a little while ago up to date? It doesn't look too confusing from the api-ref docs I'd play with it for a little bit19:21
sdaguemtreinish: are the image parts in there?19:22
sdaguemtreinish: they aren't yet, I can push with those19:22
mtreinishsdague: yeah go ahead, I'll spin up a devstack node to test it on19:23
gabriel-bezerramorganfainberg: You mean that I should fix grenade/upgrade-glance, instead of sourcing lib/apache in lib/keystone?19:24
openstackgerritSean Dague proposed a change to openstack/tempest: wip: javelin 2  https://review.openstack.org/9514819:24
sdaguemtreinish: so the issue is the properties aren't set in that call19:25
morganfainberggabriel-bezerra, nope i would source lib/apache lib/keystone (lib/apache first)19:25
morganfainberggabriel-bezerra, but sdague and mtreinish might tell you to do it the other way19:25
morganfainberggabriel-bezerra, i'd go with what they say over what i say, but without their input, i'd start with sourcing in the lib/<thing> files19:26
morganfainbergsdague, mtreinish, ^ long going convo about grenade - might be easier to discuss via gerrit review at this point.19:27
gabriel-bezerramorganfainberg: I'll source them, but I'd just like to know where to do it. It doesn't seem good to me to do it in lib/keystone as it now assumes lib/apache has been previously sourced by stack.sh.19:27
morganfainberggabriel-bezerra, right so before sourcing lib/keystone, source lib/apache19:28
morganfainbergwherever you do source them.19:28
morganfainbergand lib/apache should be 100% safe to source multiple times if needed19:28
gabriel-bezerramorganfainberg: I didn't touch anything that sources them.19:28
*** levi_b has quit IRC19:29
*** SridharG has quit IRC19:29
gabriel-bezerraand I would no like to indroduce a pattern/guideline breakage if there is a proper way to do it19:29
gabriel-bezerrawould not*19:29
*** levi_b has joined #openstack-qa19:29
morganfainberggabriel-bezerra, *nod* ok so, if sourcing in lib/keystone somewhere, you can just add a source of lib/apache right above it and it should solve the dep issue. I think i'm missing context here at this point - sorry don't mean to be sending you in circles19:29
sdaguemorganfainberg: sure, I'm just trying to push through on this redo of the javelin code so I've not had the time to dive in on this one yet19:31
gabriel-bezerramorganfainberg: no worries. The context is: it works when I use stack.sh, but grenade tests break because it doesn't seem to either use stack.sh when upgrading or source lib/apache in the right place19:32
morganfainbergsdague, ack - i'll work w/ gabriel-bezerra to try and solve the questions and we can discuss in the review then. shouldn't take you away from javalin19:32
*** nati_ueno has quit IRC19:32
morganfainberggabriel-bezerra, got it.19:32
morganfainberggabriel-bezerra, and it sources in lib/keystone in some places - just not lib/apache19:33
morganfainberggabriel-bezerra, ?19:33
gabriel-bezerrayes, in upgrade-glance: https://github.com/openstack-dev/grenade/blob/master/upgrade-glance#L7119:33
sdaguemtreinish: so if I'm reading this latest explosion on my side, tempest's v1 image client can't upload binary content, only ascii content.19:33
morganfainberggabriel-bezerra, ok perfect, just add the source of lib/apache above the lib/keystone source19:33
*** levi_b has quit IRC19:34
gabriel-bezerramorganfainberg: well, I'll make a patch here, can you review that, please?19:34
morganfainberggabriel-bezerra, sure!19:34
morganfainberggabriel-bezerra, and this also tells me i need to get more familiar with grenade on the whole.19:35
openstackgerritJulien Vey proposed a change to openstack-dev/devstack: Add an environment variable to enable extensions in keystone  https://review.openstack.org/9577819:35
mtreinishsdague: maybe, it uses a separate httplib to do the chunked upload, but I think we only ever test it with stringio in tree19:36
mtreinishits not like we have actual image data to upload19:37
mtreinishsdague: paste of the explosion?19:37
sdaguemtreinish: except we totally do19:37
sdaguebecause we specify a compute image19:37
sdaguemtreinish: http://paste.openstack.org/show/81919/19:38
*** malini is now known as malini_afk19:38
sdagueso I can upload a binary image fine with the v2 client, I just can't add properties19:38
sdagueor, more importantly, I don't know how to19:39
sdagueeglynn: you're still a glance core, right? :)19:39
sdaguemight you know how to add properties to a v2 image via the API?19:39
openstackgerritGabriel Assis Bezerra proposed a change to openstack-dev/grenade: Source lib/apache before lib/keystone in upgrade-glance  https://review.openstack.org/9624419:43
openstackgerritGabriel Assis Bezerra proposed a change to openstack-dev/grenade: Source lib/apache before lib/keystone in upgrade-glance  https://review.openstack.org/9624419:44
gabriel-bezerramorganfainberg: https://review.openstack.org/9624419:44
eglynnsdague: yes, for my sins ..19:44
eglynnsdague: use PATCH19:44
sdagueeglynn: what's the properties namespace?19:45
eglynnsdague: with something like [{"path": "/foo", "value": "bar", "op": "add"}] to add new property foo=bar19:45
eglynnsdague: ^^^ i.e. as the PATCH entity body19:45
gabriel-bezerramorganfainberg: just wait a minute, I'll check the other files for the same bug.19:45
morganfainbergi commented on it, +1 for approach19:45
morganfainbergin either case. (even if other files require the same thing) my comment will apply19:46
eglynnsdague: gotta run again, sorry ... back online later19:46
sdagueno prob19:46
gabriel-bezerramorganfainberg: thanks :)19:48
*** aimon has quit IRC19:49
openstackgerritGabriel Assis Bezerra proposed a change to openstack-dev/grenade: Source lib/apache before lib/keystone  https://review.openstack.org/9624419:52
openstackgerritMatthew Treinish proposed a change to openstack/tempest: wip: javelin 2  https://review.openstack.org/9514819:52
mtreinishsdague: ^^^ http://paste.openstack.org/show/81926/19:52
mtreinishI didn't test a boot though19:53
sdaguemtreinish: well that's further than I got, thanks19:54
*** eglynn has quit IRC19:55
mtreinishsdague: the trick was to look at the jsonschema for an image that glance returned from a getschema call19:55
*** bkopilov has quit IRC19:55
mtreinishit showed additionalProperties which is probably where things belonged19:55
gabriel-bezerramorganfainberg: just submitted the patch with the other files https://review.openstack.org/9624419:58
gabriel-bezerrasdague: ^19:58
*** shardy_afk is now known as shardy19:58
morganfainberggabriel-bezerra, nod. my comment stands19:58
mtreinishsdague: it went to active state fine when I booted from the uec image19:58
morganfainberggabriel-bezerra, it would be good to fix in devstack, but that has (potentially) larger implications19:58
gabriel-bezerramorganfainberg: pardon me19:59
morganfainberggabriel-bezerra, :) ya did it right.19:59
gabriel-bezerramorganfainberg: devstack already sources lib/apache before lib/keystone19:59
mtreinishsdague: yeah it works fine with that update call19:59
mtreinishI was able to login into it19:59
gabriel-bezerramorganfainberg: you mean make each lib/XXX source its dependencies instead of the global runners?20:00
morganfainberggabriel-bezerra, yep. but i think this fix to grenade is the right one to start with.20:00
morganfainberggabriel-bezerra, there might be bigger pitfalls to fixing devstack to source it's deps in like that20:01
gabriel-bezerramorganfainberg: I agree with you.20:01
morganfainberggabriel-bezerra, :)20:01
gabriel-bezerramorganfainberg: and I hope this fix to be enough to make my patch to devstack pass the tests.20:01
morganfainberggabriel-bezerra, we can hope!20:02
gabriel-bezerra:)20:02
*** mlavalle has quit IRC20:02
*** levi_b has joined #openstack-qa20:02
*** luqas has quit IRC20:02
*** mlavalle has joined #openstack-qa20:05
*** nati_ueno has joined #openstack-qa20:06
*** jaypipes has quit IRC20:08
*** nati_ueno has quit IRC20:11
*** cdent_ has joined #openstack-qa20:12
mtreinishsdague: oh, I didn't realize you had patched create in that commit. I think that'll work too if you change properties to additionalProperties20:12
*** cdent has quit IRC20:14
*** cdent_ is now known as cdent20:14
*** bkopilov has joined #openstack-qa20:14
sdagueok, maybe I'll try that path, because this isn't working as expected20:17
sdague2014-05-28 16:16:24.611 8010 CRITICAL tempest [-] Bad request20:17
sdagueDetails: 400 Bad Request20:17
sdagueOperation objects must contain exactly one member named "add", "remove", or "replace".20:17
*** mriedem has quit IRC20:17
*** mriedem has joined #openstack-qa20:18
mtreinishsdague: wait on the rev I uploaded?20:21
sdagueno, on mine20:21
*** cdent has quit IRC20:26
openstackgerritSean Dague proposed a change to openstack/tempest: wip: javelin 2  https://review.openstack.org/9514820:26
sdaguemtreinish: ok, so the following gives me20:26
sdaguehttp://paste.openstack.org/show/81930/20:27
*** levi_b has quit IRC20:30
sdaguegah, ok, I have it20:30
*** levi_b has joined #openstack-qa20:30
mtreinishsdague: hmm, yeah it defines additionalProperties as a string type: http://docs.openstack.org/api/openstack-image-service/2.0/content/get-image-schema.html20:30
mtreinishok cool20:30
openstackgerritSean Dague proposed a change to openstack/tempest: wip: javelin 2  https://review.openstack.org/9514820:30
*** salv-orlando has joined #openstack-qa20:30
sdagueyou actually just specify as top level20:31
sdagueand it bins it right on it's side20:31
mtreinishok cool20:31
mtreinishwant to spin the image client part off as a separate patch? With tests that use it...20:32
sdagueso probably yes, but I want to actually get the rest of this working first20:32
*** andreaf has quit IRC20:32
*** levi_b has quit IRC20:34
mtreinishfair enough20:35
*** mestery has quit IRC20:39
*** mestery has joined #openstack-qa20:40
*** nati_ueno has joined #openstack-qa20:50
openstackgerritA change was merged to openstack-dev/grenade: don't corrupt grenade exit value  https://review.openstack.org/9593420:50
*** levi_b has joined #openstack-qa20:54
openstackgerritOpenStack Proposal Bot proposed a change to openstack/tempest: Updated from global requirements  https://review.openstack.org/9627421:01
*** jchai has quit IRC21:02
*** Sukhdev has quit IRC21:04
*** aysyd has quit IRC21:06
openstackgerritJulien Vey proposed a change to openstack-dev/devstack: Add an environment variable to enable extensions in keystone  https://review.openstack.org/9577821:06
*** levi_b has quit IRC21:07
*** levi_b has joined #openstack-qa21:07
*** levi_b has quit IRC21:12
*** Sukhdev has joined #openstack-qa21:15
gabriel-bezerramtreinish: might you take a look at https://review.openstack.org/96244 ?21:24
gabriel-bezerraI couldn't find you with Gerrit's reviewer search21:24
mtreinishgabriel-bezerra: sure I can, but I'm not core on grenade.21:27
*** ekhugen has quit IRC21:28
gabriel-bezerramtreinish: that's fine21:30
gabriel-bezerrano problems21:30
openstackgerritSean Dague proposed a change to openstack-dev/devstack: be a little more clever about interface default  https://review.openstack.org/9628821:32
mtreinishsdague: ^^^ what about wireless or bonded interfaces...21:38
sdagueyou can't bridget wireless21:38
sdaguebridge21:38
sdaguein the base case21:38
sdagueand, bonded people can override :)21:39
sdaguethe point being that eth0 is now not only wrong, but impossibly wrong on new Linux21:39
sdagueso we could at least do only wrong21:39
mtreinishheh, ok fair enough. I was just trolling because of device names on systemd that don't start with e or p21:39
mtreinishI think those are the only 221:40
mtreinishor at least the only ones I've encountered21:40
sdaguemtreinish: what do systemd devices start as?21:41
mtreinishno that matches systemd, AFAIC there's no p with systemd it's all en, wl, and ww21:42
mtreinishexcept on my boxes with bonded controllers21:42
mtreinishthen its bond021:42
sdagueww ?21:42
mtreinishWWAN21:42
mtreinishI just read the man page21:42
sdagueI thought p1s5 was the new hotness for pci21:42
mtreinishI've never seen it21:42
mtreinishthat's the ubuntu (or fedora) thing, I've never seen it on arch.21:43
sdagueyeh, well arch doesn't count for devstack :)21:43
*** TravT has joined #openstack-qa21:43
sdaguegabriel-bezerra: if the grenade tests pass on your last patch I'll approve it through21:44
mtreinishone day, just wait...21:44
sdaguejust want to see the test results roll in first21:44
mroddensomeone is changing from the ethN convention?21:44
sdaguemrodden: all of them are21:44
mroddenwtf?21:44
mtreinishmrodden: you have to stop using rhel...21:44
mtreinish(and windows)21:44
mroddengentoo21:44
mtreinishmrodden: oh sorry mriedem is the windows, rhel py26 guy21:45
mtreinishyeah I guess on gentoo systemd is optional21:45
mroddeni thought we were done with the interface device naming problem21:45
mroddengentoo forked udev so they didnt' have to deal with systemd21:45
mtreinishmrodden: http://www.freedesktop.org/wiki/Software/systemd/PredictableNetworkInterfaceNames/21:46
mroddeneudev isn't so hot though21:46
mroddeni hate systemd21:46
gabriel-bezerrasdague: thank you :)21:46
mroddenen is AIX21:47
mroddennaming schema21:47
mrodden /barf21:47
mtreinishsdague: for your regex: http://cgit.freedesktop.org/systemd/systemd/tree/src/udev/udev-builtin-net_id.c#n2921:48
* mriedem goes back to watching tox21:48
*** nati_ueno has quit IRC21:49
mtreinishsdague: oh, the example for ww is a 3g modem. That seems like something you want to bridge on for devstack :)21:49
mroddenubuntu isn't using systemd are they?21:49
mtreinishmrodden: not yet, but they have a similar naming scheme thing going on21:50
mroddenugh....21:50
mtreinishor I think they did on the last ubuntu box I ran which was a while ago21:50
sdaguemrodden: they will in 14.1021:52
sdaguedebian decided to go systemd21:52
* mrodden goes back to slackware21:53
mroddenactually i did have a question21:54
mroddenwould anyone feel strongly about me self-merging some of my patches to bash8 when i'm doing cleanup and adding (badly needed) tests?21:55
sdaguemrodden: give me a review url and I'll go through them quickly21:56
mtreinishmrodden: you're the ptl, you get to set the tone of the project :)21:56
mtreinishmrodden: btw, who is the core group on bash8?21:56
mroddensdague, dtroyer and myself i think21:56
sdagueI though chmouel as well21:56
mroddenlet me check21:56
sdagueI suggested it was devstack-core + mrodden when it was built21:57
sdaguewhich might also still mean jeblair21:57
mtreinishok, that makes sense. I was just curious21:57
mroddenhttps://review.openstack.org/#/admin/groups/333,members21:57
*** dhellmann is now known as dhellmann_21:58
mroddeni can add chmouel21:58
mroddeni was wondering if he wanted to be core since he was doing reviews21:58
*** mriedem has quit IRC21:58
mtreinishmrodden: just add the devstack-core group. I think that makes sense21:58
*** dhellmann_ is now known as dhellmann21:58
mroddenok21:58
*** dhellmann is now known as dhellmann_21:58
mroddendone21:59
mroddenhah sdague already found my two patches21:59
mroddenwas about to ask21:59
mroddenthanks21:59
sdagueI'm pretty sure chmouel has some patches for function as well. He had proposed them to devstack right as you were working on the extraction, and I told him to hold22:00
mroddenoh ok22:00
mroddenyeah there was a double bracket check on22:01
mroddenone*22:01
mroddenit was failing jenkins checks last time i looked at it22:01
sdaguedoes glance not handle simultaneous connects well?22:02
sdague2014-05-28 17:57:31.124 TRACE nova.compute.manager [instance: 9f2bfde4-1b56-424e-85f6-0486aa027c29] GlanceConnectionFailed: Connection to glance host 10.42.0.50:9292 failed: Error communicating with http://10.42.0.50:9292 [Errno 101] ENETUNREACH22:03
sdaguethat's a new one to me22:03
mroddenunreachable22:03
sdagueoh... son of a...22:03
sdagueI bet there is a window when the first bridge gets set up22:04
sdaguewhere the network goes funny for a sec22:04
*** atiwari has joined #openstack-qa22:05
*** yamahata has joined #openstack-qa22:06
*** nati_ueno has joined #openstack-qa22:07
mroddenmoving the IP from the interface to the bridge would upset your routing table for bit22:07
mroddencould result in unreachable22:08
*** gabriel-bezerra has quit IRC22:13
*** jgrimm has quit IRC22:13
*** gabriel-bezerra has joined #openstack-qa22:14
sdagueyeh, that's what I'm guessing22:20
sdaguewhat's problematic is that it's a deep nova-compute failure22:20
sdaguewith no retry22:21
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add query for 1320617  https://review.openstack.org/9408022:23
openstackgerritSean Dague proposed a change to openstack/tempest: wip: javelin 2  https://review.openstack.org/9514822:25
*** openstackrecheck has quit IRC22:25
*** openstackrecheck has joined #openstack-qa22:25
jogoheh the week of the summit we only merged 100 patches22:28
*** weshay has quit IRC22:29
*** atiwari has quit IRC22:32
*** levi_b has joined #openstack-qa22:32
*** mrodden has quit IRC22:34
*** levi_b has quit IRC22:36
*** dims has quit IRC22:51
*** dims has joined #openstack-qa22:52
*** rbak has quit IRC22:57
*** bkopilov has quit IRC22:59
*** yamahata has quit IRC23:09
*** Obulpathi has quit IRC23:10
*** bkopilov has joined #openstack-qa23:13
*** Sukhdev has quit IRC23:15
*** amotoki has joined #openstack-qa23:20
*** salv-orlando has quit IRC23:24
*** salv-orlando has joined #openstack-qa23:29
*** mmaglana has quit IRC23:40
*** tosky has quit IRC23:40
*** mkollaro has quit IRC23:45
*** oomichi has joined #openstack-qa23:46
*** salv-orlando has quit IRC23:52
*** jaypipes has joined #openstack-qa23:54
*** ChanServ sets mode: +o jaypipes23:54
openstackgerritA change was merged to openstack-dev/grenade: Source lib/apache before lib/keystone  https://review.openstack.org/9624423:55
*** bkopilov has quit IRC23:57

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!