*** matsuhashi has joined #openstack-qa | 00:01 | |
*** mrodden has quit IRC | 00:09 | |
*** sthaha has joined #openstack-qa | 00:22 | |
*** aimon has joined #openstack-qa | 00:22 | |
*** jchai has joined #openstack-qa | 00:24 | |
openstackgerrit | Gabriel 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/95977 | 00:25 |
---|---|---|
*** dmorita has joined #openstack-qa | 00:36 | |
*** Sukhdev has quit IRC | 00:38 | |
*** yamahata has joined #openstack-qa | 00:39 | |
gabriel-bezerra | Hi guys, might you review my patch, please? ^ | 00:39 |
openstackgerrit | Gabriel 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/95977 | 00:53 |
openstackgerrit | Gabriel 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/95977 | 00:54 |
*** mlavalle has quit IRC | 01:02 | |
openstackgerrit | Gabriel 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/95977 | 01:09 |
openstackgerrit | Gabriel 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/95977 | 01:13 |
*** tiantian has joined #openstack-qa | 01:13 | |
*** weshay has quit IRC | 01:29 | |
openstackrecheck | openstack/glance change: https://review.openstack.org/91143 failed gate-glance-python26 in the gate queue with an unrecognized error | 01:30 |
*** weshay has joined #openstack-qa | 01:37 | |
*** jchai is now known as jchai_afk | 01:46 | |
*** nosnos has joined #openstack-qa | 01:48 | |
*** jchai_afk is now known as jchai | 02:05 | |
jchai | Hello. Could I get another review for https://review.openstack.org/87762 ? It has a +2 . Thanks. | 02:09 |
*** gabriel-bezerra has quit IRC | 02:09 | |
*** nati_uen_ has quit IRC | 02:13 | |
openstackrecheck | Console logs not available after 13:23s for gate-tempest-dsvm-postgres-full 85366,7,2bd2f91 | 02:15 |
openstackgerrit | Aaron Rosen proposed a change to openstack/tempest: Upload public image requires admin role by default https://review.openstack.org/94435 | 02:16 |
*** weshay has quit IRC | 02:24 | |
openstackrecheck | Console logs not available after 13:29s for check-grenade-dsvm-neutron 67225,4,16a75ce | 02:28 |
*** asselin_ has joined #openstack-qa | 02:32 | |
*** lbragstad has quit IRC | 02:33 | |
*** asselin has quit IRC | 02:33 | |
*** lbragstad has joined #openstack-qa | 02:33 | |
*** torandu has left #openstack-qa | 02:35 | |
openstackrecheck | Console logs not available after 13:22s for check-grenade-dsvm-neutron 95904,1,97f604b | 02:42 |
openstackrecheck | Console logs not available after 13:25s for check-grenade-dsvm-neutron 94047,15,f0319d4 | 02:55 |
*** dims has quit IRC | 02:57 | |
*** jchai has quit IRC | 03:00 | |
openstackrecheck | Console logs not available after 13:24s for check-tempest-dsvm-full-apache-services 95896,1,72a1730 | 03:09 |
*** gabriel-bezerra has joined #openstack-qa | 03:10 | |
*** harlowja_ is now known as harlowja_away | 03:16 | |
openstackrecheck | Console logs not available after 13:25s for check-grenade-dsvm-neutron 95467,4,ec245ac | 03:23 |
openstackrecheck | openstack/nova change: https://review.openstack.org/86983 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1302774 | 03:23 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for bug 1308715 https://review.openstack.org/95918 | 03:23 |
*** openstackrecheck has quit IRC | 03:26 | |
*** openstackrecheck has joined #openstack-qa | 03:26 | |
openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: Sharing codes for cinder v1 and v2 tests https://review.openstack.org/95629 | 03:38 |
openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: Sharing codes for cinder v1 and v2 tests https://review.openstack.org/95629 | 03:41 |
openstackrecheck | Console logs not available after 13:26s for check-tempest-dsvm-neutron-heat-slow 64429,20,5ee27b9 | 03:42 |
*** nosnos has quit IRC | 03:47 | |
*** yfried has joined #openstack-qa | 03:50 | |
openstackrecheck | Console logs not available after 13:23s for check-tempest-dsvm-full 94945,3,e8cc98e | 03:56 |
openstackgerrit | Jamie Lennox proposed a change to openstack-dev/devstack: Use identity_uri instead of auth fragments https://review.openstack.org/94545 | 04:06 |
openstackrecheck | Console logs not available after 13:24s for check-tempest-dsvm-neutron 95977,5,99e7b6f | 04:09 |
*** afazekas has quit IRC | 04:12 | |
openstackgerrit | Haiyang DING proposed a change to openstack/tempest: Add verification to name property in cinder volume https://review.openstack.org/95714 | 04:12 |
*** matsuhashi has quit IRC | 04:15 | |
*** yfried has quit IRC | 04:16 | |
*** psharma has joined #openstack-qa | 04:22 | |
openstackrecheck | Console logs not available after 13:26s for check-grenade-dsvm-neutron 92390,5,e4caf37 | 04:23 |
*** matsuhashi has joined #openstack-qa | 04:26 | |
*** nosnos has joined #openstack-qa | 04:27 | |
*** bkopilov has quit IRC | 04:29 | |
*** k4n0 has joined #openstack-qa | 04:30 | |
openstackrecheck | Console logs not available after 13:26s for check-grenade-dsvm 89715,2,bba3255 | 04:36 |
*** GMann has joined #openstack-qa | 04:38 | |
*** afazekas has joined #openstack-qa | 04:40 | |
*** thomasbiege has joined #openstack-qa | 04:51 | |
*** yfried has joined #openstack-qa | 04:51 | |
*** yfried has quit IRC | 04:53 | |
*** ashishsnigam has joined #openstack-qa | 04:59 | |
openstackrecheck | Console logs not available after 13:27s for check-tempest-dsvm-neutron-heat-slow 95994,1,312053a | 05:01 |
*** ashishsnigam has quit IRC | 05:07 | |
*** thomasbiege has quit IRC | 05:09 | |
*** ashishsnigam has joined #openstack-qa | 05:11 | |
*** ashishsnigam has quit IRC | 05:12 | |
*** thomasbiege has joined #openstack-qa | 05:12 | |
*** ashishsnigam has joined #openstack-qa | 05:12 | |
*** krtaylor has quit IRC | 05:15 | |
*** thomasbiege has quit IRC | 05:17 | |
*** ashishsnigam has quit IRC | 05:17 | |
*** ashishsnigam has joined #openstack-qa | 05:18 | |
*** Madan has joined #openstack-qa | 05:20 | |
*** tiantian has quit IRC | 05:21 | |
*** ashishsnigam has quit IRC | 05:23 | |
openstackrecheck | Console logs not available after 13:34s for check-grenade-dsvm-neutron 96000,1,6d881b7 | 05:24 |
openstackrecheck | Console logs not available after 13:23s for check-tempest-dsvm-neutron-full 81707,4,3879988 | 05:39 |
*** Madan has quit IRC | 05:42 | |
*** Madan has joined #openstack-qa | 05:46 | |
*** markmcclain has joined #openstack-qa | 05:52 | |
*** markmcclain1 has joined #openstack-qa | 05:54 | |
*** markmcclain has quit IRC | 05:57 | |
*** markmcclain1 has quit IRC | 06:01 | |
*** ashishsnigam has joined #openstack-qa | 06:03 | |
openstackrecheck | Console logs not available after 13:33s for check-swift-dsvm-functional 96030,1,fed175c | 06:07 |
*** tiantian has joined #openstack-qa | 06:19 | |
openstackrecheck | Console logs not available after 13:24s for check-grenade-dsvm-neutron 96026,1,aa2b332 | 06:20 |
*** tkammer has joined #openstack-qa | 06:21 | |
*** SridharG has joined #openstack-qa | 06:32 | |
*** sthaha has quit IRC | 06:32 | |
openstackrecheck | openstack/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/1226412 | 06:34 |
openstackgerrit | ajay proposed a change to openstack/tempest: Added cinder cli tests https://review.openstack.org/96072 | 06:38 |
openstackgerrit | Ashish Nigam proposed a change to openstack/tempest: Add basic read only tests for trove cli https://review.openstack.org/94599 | 06:43 |
*** ajay_ has joined #openstack-qa | 06:44 | |
*** boris-42 has quit IRC | 06:45 | |
*** yfried has joined #openstack-qa | 06:48 | |
*** boris-42 has joined #openstack-qa | 06:49 | |
*** yfried has quit IRC | 06:49 | |
*** yfried has joined #openstack-qa | 06:49 | |
openstackrecheck | Console logs not available after 13:38s for gate-python-keystoneclient-python27 74908,10,9cd604d | 06:50 |
*** yfried_ has joined #openstack-qa | 06:50 | |
*** yfried has quit IRC | 06:54 | |
*** yfried_ is now known as yfried | 06:57 | |
*** evgenyf has joined #openstack-qa | 06:58 | |
*** tkammer has quit IRC | 07:01 | |
openstackrecheck | openstack/sahara change: https://review.openstack.org/95861 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1254872 | 07:08 |
*** yrabl has joined #openstack-qa | 07:14 | |
*** yrabl has quit IRC | 07:19 | |
*** Albert_13 has joined #openstack-qa | 07:20 | |
*** jlibosva has joined #openstack-qa | 07:25 | |
*** yrabl has joined #openstack-qa | 07:26 | |
*** gszasz has joined #openstack-qa | 07:28 | |
*** bkopilov has joined #openstack-qa | 07:29 | |
*** giulivo has joined #openstack-qa | 07:38 | |
*** yfried has quit IRC | 07:53 | |
openstackgerrit | afazekas proposed a change to openstack-dev/devstack: Faster nova fixed key generation https://review.openstack.org/96098 | 07:54 |
*** markmcclain has joined #openstack-qa | 07:55 | |
openstackgerrit | afazekas proposed a change to openstack-infra/elastic-recheck: Add query for 1320617 https://review.openstack.org/94080 | 07:58 |
*** jpich has joined #openstack-qa | 07:58 | |
*** markmcclain has quit IRC | 08:00 | |
*** tkammer has joined #openstack-qa | 08:02 | |
*** jordanP has joined #openstack-qa | 08:03 | |
*** yfried has joined #openstack-qa | 08:04 | |
*** yfried has quit IRC | 08:09 | |
*** coolsvap|afk is now known as coolsvap | 08:11 | |
*** matsuhashi has quit IRC | 08:14 | |
*** matsuhashi has joined #openstack-qa | 08:15 | |
*** matsuhashi has quit IRC | 08:17 | |
*** matsuhashi has joined #openstack-qa | 08:17 | |
*** andreaf has joined #openstack-qa | 08:18 | |
*** yfried has joined #openstack-qa | 08:22 | |
*** sthaha has joined #openstack-qa | 08:22 | |
*** mmaglana has quit IRC | 08:27 | |
*** amotoki has joined #openstack-qa | 08:30 | |
*** markmcclain has joined #openstack-qa | 08:50 | |
*** eezhova has joined #openstack-qa | 08:51 | |
*** tosky has joined #openstack-qa | 08:52 | |
*** GMann has quit IRC | 09:09 | |
*** sthaha has quit IRC | 09:16 | |
*** yamahata has quit IRC | 09:22 | |
*** thomasbiege has joined #openstack-qa | 09:24 | |
*** markmcclain has quit IRC | 09:24 | |
*** gszasz has quit IRC | 09:37 | |
*** thomasbiege has quit IRC | 09:37 | |
*** ashishsnigam has quit IRC | 09:43 | |
*** boris-42 has quit IRC | 09:52 | |
*** k4n0 has quit IRC | 09:54 | |
*** markmcclain has joined #openstack-qa | 09:54 | |
*** thomasbiege has joined #openstack-qa | 09:55 | |
*** thomasbiege has quit IRC | 09:57 | |
*** mkollaro has joined #openstack-qa | 10:01 | |
openstackgerrit | nayna-patel proposed a change to openstack/tempest: Adds v2 keystone service test to service v3 https://review.openstack.org/90370 | 10:06 |
*** k4n0 has joined #openstack-qa | 10:07 | |
*** boris-42 has joined #openstack-qa | 10:14 | |
*** jchai has joined #openstack-qa | 10:17 | |
openstackgerrit | nayna-patel proposed a change to openstack/tempest: Adds v2 keystone service test to service v3 https://review.openstack.org/90370 | 10:17 |
*** gszasz has joined #openstack-qa | 10:22 | |
*** mmaglana has joined #openstack-qa | 10:28 | |
*** k4n0 has quit IRC | 10:30 | |
*** mmaglana has quit IRC | 10:33 | |
openstackgerrit | Victor Stinner proposed a change to openstack/tempest: Fix nova unit tests to support multiple networks https://review.openstack.org/96131 | 10:37 |
*** k4n0 has joined #openstack-qa | 10:37 | |
*** thomasbiege has joined #openstack-qa | 10:41 | |
openstackgerrit | Victor Stinner proposed a change to openstack/tempest: Fix nova unit tests to support multiple networks https://review.openstack.org/96131 | 10:44 |
*** matsuhashi has quit IRC | 10:48 | |
*** jchai is now known as jchai_afk | 10:48 | |
*** thomasbiege has quit IRC | 10:49 | |
*** markmcclain has quit IRC | 10:50 | |
*** luqas has joined #openstack-qa | 10:52 | |
openstackgerrit | Victor Stinner proposed a change to openstack/tempest: Fix nova unit tests attaching an interface https://review.openstack.org/96135 | 10:53 |
openstackgerrit | Yaroslav Lobankov proposed a change to openstack/tempest: Sahara: add changes to cli tests https://review.openstack.org/95134 | 10:58 |
*** malini_afk is now known as malini | 11:02 | |
*** ylobankov has joined #openstack-qa | 11:04 | |
*** dmorita has quit IRC | 11:11 | |
*** jchai_afk is now known as jchai | 11:11 | |
*** jchai has quit IRC | 11:16 | |
*** malini is now known as malini_afk | 11:19 | |
*** thomasbiege has joined #openstack-qa | 11:22 | |
*** samuelbercovici has joined #openstack-qa | 11:23 | |
*** samuelbercovici1 has joined #openstack-qa | 11:26 | |
*** samuelbercovici has quit IRC | 11:29 | |
*** samuelbercovici1 is now known as samuelbercovici | 11:29 | |
*** samuelbercovici1 has joined #openstack-qa | 11:32 | |
*** coolsvap is now known as coolsvap|afk | 11:33 | |
*** samuelbercovici has quit IRC | 11:35 | |
*** samuelbercovici1 is now known as samuelbercovici | 11:35 | |
openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: remove the build timeouts https://review.openstack.org/96143 | 11:42 |
*** sthaha has joined #openstack-qa | 11:43 | |
*** dims has joined #openstack-qa | 11:45 | |
*** samuelbercovici1 has joined #openstack-qa | 11:46 | |
openstackgerrit | Gabriel 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/95977 | 11:47 |
*** samuelbercovici has quit IRC | 11:49 | |
*** samuelbercovici1 is now known as samuelbercovici | 11:49 | |
*** thomasbiege has quit IRC | 11:51 | |
*** samuelbercovici1 has joined #openstack-qa | 11:54 | |
*** samuelbercovici has quit IRC | 11:57 | |
*** samuelbercovici1 is now known as samuelbercovici | 11:57 | |
openstackgerrit | Artur Svechnikov proposed a change to openstack/tempest: Add Ceilometer client for scenarios tests https://review.openstack.org/92108 | 12:01 |
*** markmcclain has joined #openstack-qa | 12:02 | |
*** markmcclain1 has joined #openstack-qa | 12:04 | |
*** markmcclain1 has quit IRC | 12:04 | |
*** markmcclain1 has joined #openstack-qa | 12:04 | |
*** markmcclain has quit IRC | 12:06 | |
*** k4n0 has quit IRC | 12:13 | |
*** psharma has quit IRC | 12:15 | |
*** aysyd has joined #openstack-qa | 12:18 | |
*** krtaylor has joined #openstack-qa | 12:19 | |
*** thomasbiege has joined #openstack-qa | 12:20 | |
*** enikanorov_ has quit IRC | 12:21 | |
*** weshay has joined #openstack-qa | 12:23 | |
*** sthaha has quit IRC | 12:24 | |
*** sthaha has joined #openstack-qa | 12:24 | |
*** sthaha has quit IRC | 12:25 | |
*** sthaha has joined #openstack-qa | 12:25 | |
*** mmaglana has joined #openstack-qa | 12:29 | |
*** mmaglana has quit IRC | 12:33 | |
*** markmcclain1 has quit IRC | 12:36 | |
*** jchai has joined #openstack-qa | 12:39 | |
*** dims has quit IRC | 12:39 | |
*** andreaf has quit IRC | 12:41 | |
*** thomasbiege has quit IRC | 12:44 | |
*** ekhugen has joined #openstack-qa | 12:49 | |
*** nosnos has quit IRC | 12:50 | |
*** yfried has quit IRC | 13:10 | |
*** mriedem has joined #openstack-qa | 13:10 | |
*** Madan has quit IRC | 13:11 | |
openstackgerrit | Vadim Rovachev proposed a change to openstack/tempest: Add compute notifications tests for ceilometer https://review.openstack.org/64136 | 13:17 |
*** bkopilov has quit IRC | 13:18 | |
*** bkopilov has joined #openstack-qa | 13:19 | |
*** jaypipes has quit IRC | 13:20 | |
*** Obulpathi has joined #openstack-qa | 13:22 | |
*** jaypipes has joined #openstack-qa | 13:23 | |
*** ChanServ sets mode: +o jaypipes | 13:23 | |
*** malini_afk is now known as malini | 13:24 | |
*** Obulpathi has quit IRC | 13:26 | |
openstackgerrit | Masayuki Igawa proposed a change to openstack/qa-specs: WIP: Add tempest server/client/GUI spec https://review.openstack.org/96163 | 13:29 |
*** cdent has joined #openstack-qa | 13:29 | |
openstackgerrit | Masayuki Igawa proposed a change to openstack/qa-specs: WIP: Add tempest server/client/GUI spec https://review.openstack.org/96163 | 13:31 |
*** tkammer has quit IRC | 13:31 | |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add a hacking check to block scenario manager in api tests https://review.openstack.org/96164 | 13:32 |
*** dkorn has joined #openstack-qa | 13:32 | |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add a hacking check to block scenario manager in api tests https://review.openstack.org/96164 | 13:33 |
openstackgerrit | Jordan Pittier proposed a change to openstack/tempest: Scenario : add shelve/unshelve in TestSnapshotPattern https://review.openstack.org/96165 | 13:36 |
*** samuelbercovici1 has joined #openstack-qa | 13:39 | |
*** jordanP has quit IRC | 13:39 | |
*** samuelbercovici has quit IRC | 13:42 | |
*** samuelbercovici1 is now known as samuelbercovici | 13:42 | |
*** tkammer has joined #openstack-qa | 13:45 | |
*** samuelbercovici1 has joined #openstack-qa | 13:47 | |
*** Obulpathi has joined #openstack-qa | 13:48 | |
*** samuelbercovici has quit IRC | 13:50 | |
*** samuelbercovici1 is now known as samuelbercovici | 13:50 | |
openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Add decorator for safe tearDown https://review.openstack.org/84645 | 13:51 |
*** andreaf has joined #openstack-qa | 13:53 | |
*** rbak has joined #openstack-qa | 13:59 | |
*** yfried has joined #openstack-qa | 14:00 | |
*** amotoki has quit IRC | 14:00 | |
andreaf | mtreinish, sdague: ping | 14:03 |
mtreinish | andreaf: what's up? | 14:04 |
*** ekhugen has quit IRC | 14:04 | |
andreaf | mtreinish: re the use of official clients vs. tempest ones | 14:04 |
*** enikanorov_ has joined #openstack-qa | 14:04 | |
mtreinish | andreaf: heh, did you see my hacking patch? | 14:05 |
andreaf | mtreinish: what do you think about tempest.common | 14:05 |
andreaf | mtreinish: yes | 14:05 |
mtreinish | andreaf: so it has to be the tempest clients for the api tests. There really isn't an exception to that | 14:05 |
*** ajay_ has quit IRC | 14:05 | |
andreaf | mtreinish: and I'm reviewing https://review.openstack.org/#/c/96131 which uses official clients for translating network name to network id | 14:05 |
mtreinish | andreaf: yeah that's what spawned my hacking check | 14:06 |
andreaf | ok there are things in common which can be used by both api and scenario - should they behave differently depending on the caller? | 14:06 |
andreaf | this is what I did in auth.py, but common is in a mixed state now | 14:07 |
mtreinish | andreaf: 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 IRC | 14:07 | |
mtreinish | ideally you'd change hwat's used depending on the caller | 14:07 |
mtreinish | I did that for tenant isolation too | 14:07 |
andreaf | mtreinish: so we should have tempest.common.api and tempest.common.scenario? | 14:08 |
mtreinish | andreaf: I just did if checks... | 14:09 |
mtreinish | and had an optional param to set which client was being used | 14:09 |
andreaf | mtreinish: tenant isolation works with official clients only (just an example) | 14:09 |
mtreinish | yeah, if you set tempest_client=False when you call init: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/common/isolated_creds.py#n30 | 14:11 |
mtreinish | it will only use the official clients | 14:11 |
andreaf | mtreinish: what about some adaptation layer for common classes? | 14:11 |
mtreinish | I'm sure there are better ways to do it :) | 14:11 |
andreaf | mtreinish: oh so it actually works with both kind of clients... good :) | 14:12 |
andreaf | mtreinish: ok anyways I just wanted to clarify the policy for the review | 14:13 |
*** markmcclain has joined #openstack-qa | 14:13 | |
andreaf | mtreinish: thanks | 14:13 |
mtreinish | sure, np | 14:14 |
mtreinish | andreaf: I always like to use that file as an example of why I hate the official clients. Everything is so different between all the clients | 14:15 |
mtreinish | everything is always different with them | 14:15 |
openstackrecheck | openstack/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/1315468 | 14:16 |
sdague | so I actually think we should try to build some tempest client isolation into a tempest.client namespace | 14:18 |
sdague | because I think people would actually like to consume our client in other test environments, and I think that's a sane idea | 14:18 |
*** ekhugen has joined #openstack-qa | 14:19 | |
*** yamahata has joined #openstack-qa | 14:20 | |
mtreinish | I'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-qa | 14:22 | |
mtreinish | sdague: 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-qa | 14:25 | |
*** ekhugen has quit IRC | 14:26 | |
*** samuelbercovici has quit IRC | 14:27 | |
*** samuelbercovici1 is now known as samuelbercovici | 14:27 | |
*** markmcclain has quit IRC | 14:29 | |
*** mmaglana has joined #openstack-qa | 14:30 | |
gabriel-bezerra | Hi 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/95977 | 14:32 |
gabriel-bezerra | It changes the way apache services are configured. | 14:33 |
*** markmcclain has joined #openstack-qa | 14:33 | |
*** mmaglana has quit IRC | 14:35 | |
*** mlavalle has joined #openstack-qa | 14:35 | |
sdague | mtreinish: so actually, I think we could decouple | 14:39 |
*** SridharG has quit IRC | 14:40 | |
sdague | the coupling is mostly eggregious at this point | 14:40 |
*** mrodden has joined #openstack-qa | 14:41 | |
gabriel-bezerra | How are upgrades managed in devstack? Is there any place to put upgrading code? | 14:43 |
sdague | gabriel-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 |
sdague | dtroyer: https://review.openstack.org/#/c/95934/ - you good with this before taking off? | 14:45 |
dtroyer | yes…did I really do that? ugh... | 14:47 |
dtroyer | BTW, I'm trying to get through ceph/sheepdog this morning yet… | 14:48 |
sdague | no, I did it | 14:48 |
sdague | I was trying to add an exit hook for debug, and completely flubbed that bit | 14:49 |
dtroyer | <phew> now I can sleep better tonight driving across western Kansas, aka, Nothingville | 14:50 |
*** yrabl has quit IRC | 14:51 | |
*** jgrimm has joined #openstack-qa | 14:52 | |
sdague | :) | 14:53 |
*** tkammer has quit IRC | 14:55 | |
*** markmcclain has quit IRC | 14:55 | |
openstackgerrit | afazekas proposed a change to openstack-dev/devstack: soft-qemu handling with F20 could image https://review.openstack.org/96180 | 14:59 |
*** afazekas has quit IRC | 15:10 | |
*** kgriffs|afk is now known as kgriffs | 15:13 | |
*** ekhugen has joined #openstack-qa | 15:16 | |
*** bkopilov has quit IRC | 15:21 | |
*** rdekel has joined #openstack-qa | 15:23 | |
sdague | question... 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 |
openstackrecheck | openstack/requirements change: https://review.openstack.org/95974 failed gate-tempest-dsvm-postgres-full in the gate queue with an unrecognized error | 15:28 |
*** rdekel has quit IRC | 15:30 | |
*** mmaglana has joined #openstack-qa | 15:31 | |
*** samuelbercovici1 has joined #openstack-qa | 15:35 | |
*** eglynn has joined #openstack-qa | 15:37 | |
*** samuelbercovici has quit IRC | 15:38 | |
*** samuelbercovici1 is now known as samuelbercovici | 15:38 | |
eglynn | hi folks | 15:40 |
eglynn | anyone got time for a quick question on the implications of "branchless tempest"? | 15:40 |
*** Sukhdev has joined #openstack-qa | 15:40 | |
eglynn | vrovachev and I were just discussing this on the #openstack-ceilometer channel | 15:40 |
vrovachev | sdague, mtreinish - yes | 15:41 |
vrovachev | we have problems with tempest + ceilo integration | 15:41 |
gabriel-bezerra | sdague: 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 |
eglynn | TBH I'm a bit confused on the notion of tempest:master running against stable/icehouse also | 15:41 |
vrovachev | sdague, mtreinish, eglynn - our tests won't work with icehouse -> as there is previous cycle job in tempest... | 15:42 |
vrovachev | we have no many choices here | 15:42 |
eglynn | ... to my naive understanding, that seems to block progress on landing tempest coverage of stuff that's known to work on master | 15:42 |
gabriel-bezerra | sdague: should lib/keystone source lib/apache (it currently doesn't do that)? | 15:42 |
sdague | eglynn: the openstack API is the openstack API | 15:42 |
morganfainberg | gabriel-bezerrab, it doesn't? how does it do the apache setup then? | 15:43 |
sdague | it shouldn't change between versions | 15:43 |
vrovachev | it's not about api | 15:43 |
eglynn | sdague: ... this isn't an API compatability issue | 15:43 |
vrovachev | it's about the performance | 15:43 |
eglynn | sdague: ... rather a quality-of-implementation issue | 15:43 |
morganfainberg | gabriel-bezerra, *is suddently confused* i need to reread those scripts. | 15:43 |
vrovachev | needed change might be possible only in new juno features - we can't backport these things to icehouse ceilo | 15:44 |
sdague | eglynn: is there a specific instance we can discuss? | 15:44 |
eglynn | sdague: ... backporting the improvements to the ceilo sql-a driver from master to stable/icehouse is not going to viable in all cases | 15:44 |
openstackgerrit | David Paterson proposed a change to openstack/qa-specs: Add spec for bp:post-run-cleanup https://review.openstack.org/91725 | 15:44 |
sdague | eglynn: ok, but point at a specific test that's proposed that's problematic here | 15:44 |
eglynn | sdague: ... for example, we've gotten a big performance improvement from the schema simplifications possible after removing support for the ceilometer v1 API | 15:45 |
sdague | ok | 15:45 |
eglynn | sdague: here's a test https://review.openstack.org/#/c/64136 | 15:45 |
eglynn | sdague: ... that v1 removal change cannot be backported | 15:45 |
eglynn | sdague: ... as that would break policy around the deprecation path | 15:46 |
eglynn | sdague: ... and would require that schema migrations are backported (also problematic) | 15:46 |
eglynn | sdague: ... I mean "the deprecation path" for the v1 API | 15:46 |
sdague | so the issue is that these tests can't run in < 60s in icehouse? | 15:46 |
eglynn | sdague: ... those tests couldn't be landed during icehouse due to the performance issue that we're now making progress on solving | 15:47 |
*** torandu has joined #openstack-qa | 15:47 | |
vrovachev | sdague - here all ceilo tests https://review.openstack.org/#/q/owner:vrovachev+project:openstack/tempest+status:open,n,z | 15:47 |
eglynn | sdague: ... but of course those performance issues still exist on stable/icehouse | 15:47 |
vrovachev | sdague - 60s improvement won't help here - we'll need at least 18min range.. | 15:48 |
vrovachev | for every test! | 15:48 |
vrovachev | it'll be endless | 15:48 |
eglynn | sdague: ... so we've tests that were completely unviable to run in the gate for the reasons vrovachev points out | 15:49 |
eglynn | sdague: ... they are now becoming viable, but only against master | 15:49 |
sdague | so 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 |
sdague | what would be some good signaling there? | 15:52 |
eglynn | sdague: "some end user signal" ==> as in test.attr tagging or something of that ilk encoded in the testcase itself? | 15:53 |
sdague | how would a user know whether or not they should expect these tests to succeed? | 15:54 |
vrovachev | we were thinking about this job disabling... it simply can't work... dunno how this is possible | 15:54 |
*** afazekas has joined #openstack-qa | 15:54 | |
*** mmaglana has quit IRC | 15:55 | |
*** mkollaro has quit IRC | 15:56 | |
*** mmaglana has joined #openstack-qa | 15:56 | |
eglynn | vrovachev: ... would it suffice to disable selected test cases with a test.attr(type='only_works_on_master_so_skip_on_stable_*') ? | 15:58 |
gabriel-bezerra | morganfainberg: I don't know how that gets in the environment variables, but they are there and work when one ./stack.sh s | 15:58 |
*** tkammer has joined #openstack-qa | 15:59 | |
eglynn | ... where 'only_works_on_master_so_skip_on_stable_*' is replaced by something a bit more punchy ;) | 15:59 |
vrovachev | eglynn, possibly - if it's there I can try to | 15:59 |
morganfainberg | gabriel-bezerra, ah i think stack.sh sources it early | 15:59 |
vrovachev | or do I need to impl this? | 15:59 |
eglynn | sdague: ... is that kind of tagging approach suggested above ^^^ feasible in the gate? | 15:59 |
gabriel-bezerra | in grenade it doesn't work though | 15:59 |
vrovachev | sdague - is there this type for the test.attr? | 16:00 |
sdague | eglynn: and what value would you set for testing hp public cloud? | 16:00 |
sdague | or rax, or cloudwatt | 16:00 |
*** mmaglana has quit IRC | 16:00 | |
gabriel-bezerra | morganfainberg: grenade does "source $TARGET_DEVSTACK_DIR/lib/keystone" but doesn't do the same with apache | 16:01 |
gabriel-bezerra | in upgrade-glance | 16:01 |
eglynn | sdague: ... are you thinking in terms of Tempest==DefCore, as opposed to Tempest==upsteam gate? | 16:01 |
sdague | eglynn: I'm thinking as Tempest test suite | 16:01 |
jvrbanac | Howdy 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 |
sdague | Tempest is designed to work on any openstack cloud | 16:02 |
sdague | it's not just the gate | 16:02 |
sdague | people run it on their clouds regularly for monitoring | 16:02 |
eglynn | sdague: "designed to work on any openstack cloud" == "designed to run against any openstack cloud *of any version*"? | 16:03 |
sdague | eglynn: that's the point of branchless tempest | 16:03 |
sdague | yes, of any version, as long as the API versions are supported | 16:03 |
*** ildikov has joined #openstack-qa | 16:03 | |
sdague | Nova 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 IRC | 16:04 | |
sdague | anyway, have to run off to lunch now. Can chat about this more once I get back. | 16:04 |
eglynn | sdague: ok, thanks ... ping me when you're available again | 16:05 |
*** aimon has quit IRC | 16:05 | |
*** atiwari has joined #openstack-qa | 16:06 | |
gabriel-bezerra | morganfainberg: Should it source lib/apache then? | 16:08 |
*** SridharG has joined #openstack-qa | 16:19 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/86983 failed gate-nova-pep8 in the gate queue with an unrecognized error | 16:25 |
openstackrecheck | openstack/cinder change: https://review.openstack.org/95333 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1302774 | 16:26 |
gabriel-bezerra | sdague, morganfainberg ^ | 16:27 |
morganfainberg | gabriel-bezerra, oh. i see uhm... | 16:27 |
morganfainberg | gabriel-bezerra, if it legitimately relies on apache, probably. | 16:28 |
*** bearhands is now known as comstud | 16:32 | |
*** dkorn has quit IRC | 16:32 | |
*** evgenyf has quit IRC | 16:34 | |
*** jpich has quit IRC | 16:34 | |
gabriel-bezerra | morganfainberg: 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 |
morganfainberg | gabriel-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 |
morganfainberg | gabriel-bezerra, i'm not as familiar with genade as i should be tbh. | 16:37 |
morganfainberg | :( | 16:37 |
gabriel-bezerra | morganfainberg: is there any problem in sourcing lib/apache multiple times in a stack/grenade run? | 16:38 |
morganfainberg | gabriel-bezerra, there might be. like i said, not as familair as i should be with it | 16:39 |
gabriel-bezerra | I can't see any by looking at lib/apache code | 16:39 |
gabriel-bezerra | but I'd like to know from you whether it should be avoided | 16:40 |
gabriel-bezerra | or should grenade source that instead of lib/keystone | 16:40 |
gabriel-bezerra | ? | 16:40 |
gabriel-bezerra | sdague ^ | 16:41 |
morganfainberg | gabriel-bezerra, ok, looking at lib/apache, it should eb safe to source multiple times. | 16:43 |
morganfainberg | gabriel-bezerra, (have it open now) | 16:43 |
*** eglynn-afk has joined #openstack-qa | 16:43 | |
morganfainberg | gabriel-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 |
SergeyLukjanov | sdague, mtreinish, please, take a look at https://review.openstack.org/#/c/94311/ | 16:44 |
morganfainberg | gabriel-bezerra, so that if you need lib/apache it's available | 16:44 |
gabriel-bezerra | well.. I'll do that then. Hope it to be clarified in the review, should it be a wrong step. | 16:44 |
morganfainberg | gabriel-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 correct | 16:45 |
morganfainberg | gabriel-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-qa | 16:46 | |
*** jlibosva has quit IRC | 16:46 | |
*** mmaglana has joined #openstack-qa | 16:47 | |
*** tkammer has quit IRC | 16:49 | |
*** luqas has quit IRC | 16:51 | |
*** mlavalle has quit IRC | 16:51 | |
*** samuelbercovici has quit IRC | 16:52 | |
eglynn | sdague: I need to run shortly so can I just throw an idea out on the table? | 16:54 |
eglynn | sdague: so I was just reading the branchless tempest BP spec https://github.com/openstack/qa-specs/blob/master/specs/branchless-tempest.rst | 16:54 |
eglynn | sdague: and here's an idea ... | 16:55 |
*** aimon has joined #openstack-qa | 16:55 | |
eglynn | sdague: ... what if we treat these problematic ceilometer tests as falling under "Scenario 1: New Tests for new features"? | 16:55 |
eglynn | sdague: ... the new feature being: "Run fast enough for gate testing to be viable" | 16:55 |
eglynn | sdague: ... effectively treating the ceilometer sqlalchemy driver in all versions prior to Juno as being essentially an experimental/PoC thing | 16:55 |
DinaBelova | eglynn, absolutely ++ - if that's possible to be used here - it'll be cool | 16:55 |
eglynn | sdague: now the branchless tempest BP isn't specific as to mechanics of the skipping for new feature tests on stable/icehouse | 16:56 |
eglynn | sdague: it just states ... "this test will need to be behind a feature flag (as the feature wasn't available in Icehouse)." | 16:56 |
eglynn | sdague: ... if that "feature flag" maps on a test.attr decorator or some-such, then this may be workable? | 16:56 |
openstackgerrit | A change was merged to openstack-dev/devstack: chown stack/data/nova/networks to $STACK_USER https://review.openstack.org/95914 | 16:57 |
*** nati_ueno has joined #openstack-qa | 16:57 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/72375 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1226412 | 16:57 |
eglynn | sdague: 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 IRC | 17:01 | |
*** yamahata has quit IRC | 17:03 | |
*** andreaf has quit IRC | 17:06 | |
*** ildikov has quit IRC | 17:09 | |
*** andreaf has joined #openstack-qa | 17:10 | |
*** andreaf has quit IRC | 17:10 | |
*** andreaf has joined #openstack-qa | 17:10 | |
*** andreaf has quit IRC | 17:11 | |
*** andreaf has joined #openstack-qa | 17:12 | |
*** Obulpathi has quit IRC | 17:13 | |
*** harlowja_away is now known as harlowja_ | 17:16 | |
*** Obulpathi has joined #openstack-qa | 17:17 | |
openstackgerrit | Monty Taylor proposed a change to openstack-dev/devstack: Update get-pip url https://review.openstack.org/96211 | 17:20 |
*** vponomaryov has quit IRC | 17:24 | |
*** vponomaryov has joined #openstack-qa | 17:25 | |
mtreinish | eglynn-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 |
mtreinish | eglynn-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 it | 17:26 |
*** bkopilov has joined #openstack-qa | 17:26 | |
mtreinish | eglynn-afk: we normally just use a skip decorator that checks the config flag, or we raise a skip exception from an if block | 17:27 |
mtreinish | there are a bunch of examples of both in the tree | 17:27 |
*** levi_b has joined #openstack-qa | 17:27 | |
sdague | eglynn-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 |
sdague | I 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 |
sdague | and 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 timeout | 17:29 |
*** bkopilov has quit IRC | 17:35 | |
*** Sukhdev has quit IRC | 17:40 | |
*** gszasz has quit IRC | 17:40 | |
*** tkammer has joined #openstack-qa | 17:43 | |
*** ildikov has joined #openstack-qa | 17:44 | |
*** bkopilov has joined #openstack-qa | 17:49 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/95206 failed gate-tempest-dsvm-postgres-full in the gate queue with an unrecognized error | 17:50 |
gabriel-bezerra | morganfainberg: so that should be done on grenade/upgrade-glance, not in lib/keystone. Right? | 17:51 |
*** kgriffs is now known as kgriffs|afk | 17:54 | |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: wip: javelin 2 https://review.openstack.org/95148 | 17:59 |
openstackgerrit | Rob Crittenden proposed a change to openstack-dev/devstack: Configure heat keystone client for SSL when keystone uses SSL https://review.openstack.org/96216 | 18:10 |
openstackgerrit | afazekas proposed a change to openstack/qa-specs: bp/net-debug DRAFT https://review.openstack.org/95794 | 18:14 |
*** jamielennox is now known as jamielennox|away | 18:16 | |
*** luqas has joined #openstack-qa | 18:17 | |
*** nati_ueno has quit IRC | 18:24 | |
*** SridharG has quit IRC | 18:24 | |
*** tkammer has quit IRC | 18:24 | |
*** SridharG has joined #openstack-qa | 18:25 | |
*** nati_ueno has joined #openstack-qa | 18:30 | |
gabriel-bezerra | dtroyer, 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 |
dtroyer | They 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 |
sdague | mtreinish: 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 v1 | 18:44 |
mtreinish | sdague: hmm, let me take a look | 18:45 |
*** dkorn has joined #openstack-qa | 18:45 | |
sdague | basically creating an ami 3 part image via tempest clients that actually works... is challenging | 18:45 |
openstackgerrit | afazekas proposed a change to openstack-dev/devstack: soft-qemu handling with F20 could image https://review.openstack.org/96180 | 18:46 |
*** gabriel-bezerra has quit IRC | 18:47 | |
mtreinish | sdague: yeah it looks like the v2 client doesn't support any optional metadata being added to the create call (except visibility) | 18:49 |
sdague | mtreinish: if I understood the v2 update call on setting properties | 18:49 |
mtreinish | but the update call does for v2 | 18:49 |
sdague | any idea what that namespace is supposed to look like? | 18:49 |
mtreinish | on v1 there is no separate store call you just pass it the image data for create or update | 18:49 |
*** gabriel-bezerra has joined #openstack-qa | 18:50 | |
gabriel-bezerra | dtroyer: ok, I'll put that | 18:51 |
dtroyer | gabriel-bezerra: thanks | 18:52 |
mtreinish | sdague: 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#n117 | 18:52 |
mtreinish | which matches up with: http://docs.openstack.org/api/openstack-image-service/2.0/content/restricted-json-pointers.html | 18:52 |
mtreinish | and http://docs.openstack.org/api/openstack-image-service/2.0/content/update-an-image.html | 18:53 |
*** mlavalle has joined #openstack-qa | 18:53 | |
mtreinish | sdague: although loosening the v2 create call is probably a fair bug fix | 18:53 |
sdague | mtreinish: yeh, I found finding the same issue | 18:53 |
mtreinish | because only using visibility from kwargs seems a bit restrictive | 18:54 |
sdague | maybe, though this is properties in a different namespace IIRC | 18:55 |
sdague | I could revert to v1 | 18:55 |
mtreinish | sdague: which properties do you need to set for the ami image parts though? It should just be format and container right? | 18:57 |
mtreinish | oh, nm you need the metadata to link them | 18:57 |
*** shardy is now known as shardy_afk | 18:57 | |
mtreinish | the various ids of the different parts | 18:57 |
*** jhenner has quit IRC | 18:57 | |
sdague | yeh, I need the ramdisk_id and kernel_id set as properties | 18:58 |
*** jhenner has joined #openstack-qa | 18:58 | |
*** levi_b_ has joined #openstack-qa | 18:58 | |
*** eglynn has joined #openstack-qa | 18:58 | |
*** levi_b_ has quit IRC | 18:58 | |
eglynn | sdague: just catching up with backlog, thanks for the response! | 18:59 |
*** levi_b_ has joined #openstack-qa | 18:59 | |
eglynn | sdague: cool, sounds like we've a basis to move forward ... | 18:59 |
eglynn | sdague: ... I'll familiarize myself with those examples in-tree of skipping new features | 18:59 |
eglynn | sdague: ... on the point about an existing application depending on the previous slowness | 19:00 |
eglynn | sdague: ... note that much of the speed has been gained on the "write side" | 19:00 |
mtreinish | sdague: yeah I would just change the loop in the v2 create honestly. So it loops over kwargs instead of a preset list of know opts | 19:00 |
eglynn | sdague: ... i.e. in the pipeline for collecting and persisting metering data | 19:00 |
sdague | eglynn: ok | 19:01 |
eglynn | sdague: (... 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 |
eglynn | sdague: ... though such timeouts could have happened in past too :( | 19:02 |
mtreinish | sdague: it doesn't look like v1 has support for those params either... | 19:02 |
mtreinish | it does the same hard coded list thing | 19:02 |
*** levi_b has quit IRC | 19:02 | |
sdague | mtreinish: yeh, but I could to the properties update | 19:02 |
*** atiwari has quit IRC | 19:02 | |
sdague | could do image_update on 'properties' | 19:02 |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Consolidate _check_tenant_network_connectivity in NetworkScenarioTest https://review.openstack.org/95844 | 19:03 |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Consolidate _check_public_network_connectivity into NetworkScenarioTest https://review.openstack.org/95831 | 19:03 |
*** levi_b_ has quit IRC | 19:03 | |
mtreinish | sdague: yeah give that a try to start I guess. | 19:03 |
*** Sukhdev has joined #openstack-qa | 19:04 | |
sdague | so I'm going to change create, because I have no idea how to do the update correctly in v2 | 19:06 |
*** levi_b has joined #openstack-qa | 19:06 | |
*** dims has joined #openstack-qa | 19:07 | |
mtreinish | sdague: heh, you could always ask in -glance, I'm sure they'd know off the top of their head... | 19:08 |
sdague | hmmm that didn't do the right thing | 19:08 |
sdague | I asked in -dev | 19:08 |
sdague | I 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 |
mtreinish | heh, I'm only up to r on my client (which I think is 14) | 19:10 |
sdague | ok, I give up, there is actually no public docs on v2. | 19:15 |
sdague | I 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-qa | 19:15 | |
mtreinish | sdague: 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 bit | 19:21 |
sdague | mtreinish: are the image parts in there? | 19:22 |
sdague | mtreinish: they aren't yet, I can push with those | 19:22 |
mtreinish | sdague: yeah go ahead, I'll spin up a devstack node to test it on | 19:23 |
gabriel-bezerra | morganfainberg: You mean that I should fix grenade/upgrade-glance, instead of sourcing lib/apache in lib/keystone? | 19:24 |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: wip: javelin 2 https://review.openstack.org/95148 | 19:24 |
sdague | mtreinish: so the issue is the properties aren't set in that call | 19:25 |
morganfainberg | gabriel-bezerra, nope i would source lib/apache lib/keystone (lib/apache first) | 19:25 |
morganfainberg | gabriel-bezerra, but sdague and mtreinish might tell you to do it the other way | 19:25 |
morganfainberg | gabriel-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> files | 19:26 |
morganfainberg | sdague, mtreinish, ^ long going convo about grenade - might be easier to discuss via gerrit review at this point. | 19:27 |
gabriel-bezerra | morganfainberg: 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 |
morganfainberg | gabriel-bezerra, right so before sourcing lib/keystone, source lib/apache | 19:28 |
morganfainberg | wherever you do source them. | 19:28 |
morganfainberg | and lib/apache should be 100% safe to source multiple times if needed | 19:28 |
gabriel-bezerra | morganfainberg: I didn't touch anything that sources them. | 19:28 |
*** levi_b has quit IRC | 19:29 | |
*** SridharG has quit IRC | 19:29 | |
gabriel-bezerra | and I would no like to indroduce a pattern/guideline breakage if there is a proper way to do it | 19:29 |
gabriel-bezerra | would not* | 19:29 |
*** levi_b has joined #openstack-qa | 19:29 | |
morganfainberg | gabriel-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 circles | 19:29 |
sdague | morganfainberg: 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 yet | 19:31 |
gabriel-bezerra | morganfainberg: 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 place | 19:32 |
morganfainberg | sdague, 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 javalin | 19:32 |
*** nati_ueno has quit IRC | 19:32 | |
morganfainberg | gabriel-bezerra, got it. | 19:32 |
morganfainberg | gabriel-bezerra, and it sources in lib/keystone in some places - just not lib/apache | 19:33 |
morganfainberg | gabriel-bezerra, ? | 19:33 |
gabriel-bezerra | yes, in upgrade-glance: https://github.com/openstack-dev/grenade/blob/master/upgrade-glance#L71 | 19:33 |
sdague | mtreinish: 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 |
morganfainberg | gabriel-bezerra, ok perfect, just add the source of lib/apache above the lib/keystone source | 19:33 |
*** levi_b has quit IRC | 19:34 | |
gabriel-bezerra | morganfainberg: well, I'll make a patch here, can you review that, please? | 19:34 |
morganfainberg | gabriel-bezerra, sure! | 19:34 |
morganfainberg | gabriel-bezerra, and this also tells me i need to get more familiar with grenade on the whole. | 19:35 |
openstackgerrit | Julien Vey proposed a change to openstack-dev/devstack: Add an environment variable to enable extensions in keystone https://review.openstack.org/95778 | 19:35 |
mtreinish | sdague: maybe, it uses a separate httplib to do the chunked upload, but I think we only ever test it with stringio in tree | 19:36 |
mtreinish | its not like we have actual image data to upload | 19:37 |
mtreinish | sdague: paste of the explosion? | 19:37 |
sdague | mtreinish: except we totally do | 19:37 |
sdague | because we specify a compute image | 19:37 |
sdague | mtreinish: http://paste.openstack.org/show/81919/ | 19:38 |
*** malini is now known as malini_afk | 19:38 | |
sdague | so I can upload a binary image fine with the v2 client, I just can't add properties | 19:38 |
sdague | or, more importantly, I don't know how to | 19:39 |
sdague | eglynn: you're still a glance core, right? :) | 19:39 |
sdague | might you know how to add properties to a v2 image via the API? | 19:39 |
openstackgerrit | Gabriel Assis Bezerra proposed a change to openstack-dev/grenade: Source lib/apache before lib/keystone in upgrade-glance https://review.openstack.org/96244 | 19:43 |
openstackgerrit | Gabriel Assis Bezerra proposed a change to openstack-dev/grenade: Source lib/apache before lib/keystone in upgrade-glance https://review.openstack.org/96244 | 19:44 |
gabriel-bezerra | morganfainberg: https://review.openstack.org/96244 | 19:44 |
eglynn | sdague: yes, for my sins .. | 19:44 |
eglynn | sdague: use PATCH | 19:44 |
sdague | eglynn: what's the properties namespace? | 19:45 |
eglynn | sdague: with something like [{"path": "/foo", "value": "bar", "op": "add"}] to add new property foo=bar | 19:45 |
eglynn | sdague: ^^^ i.e. as the PATCH entity body | 19:45 |
gabriel-bezerra | morganfainberg: just wait a minute, I'll check the other files for the same bug. | 19:45 |
morganfainberg | i commented on it, +1 for approach | 19:45 |
morganfainberg | in either case. (even if other files require the same thing) my comment will apply | 19:46 |
eglynn | sdague: gotta run again, sorry ... back online later | 19:46 |
sdague | no prob | 19:46 |
gabriel-bezerra | morganfainberg: thanks :) | 19:48 |
*** aimon has quit IRC | 19:49 | |
openstackgerrit | Gabriel Assis Bezerra proposed a change to openstack-dev/grenade: Source lib/apache before lib/keystone https://review.openstack.org/96244 | 19:52 |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: wip: javelin 2 https://review.openstack.org/95148 | 19:52 |
mtreinish | sdague: ^^^ http://paste.openstack.org/show/81926/ | 19:52 |
mtreinish | I didn't test a boot though | 19:53 |
sdague | mtreinish: well that's further than I got, thanks | 19:54 |
*** eglynn has quit IRC | 19:55 | |
mtreinish | sdague: the trick was to look at the jsonschema for an image that glance returned from a getschema call | 19:55 |
*** bkopilov has quit IRC | 19:55 | |
mtreinish | it showed additionalProperties which is probably where things belonged | 19:55 |
gabriel-bezerra | morganfainberg: just submitted the patch with the other files https://review.openstack.org/96244 | 19:58 |
gabriel-bezerra | sdague: ^ | 19:58 |
*** shardy_afk is now known as shardy | 19:58 | |
morganfainberg | gabriel-bezerra, nod. my comment stands | 19:58 |
mtreinish | sdague: it went to active state fine when I booted from the uec image | 19:58 |
morganfainberg | gabriel-bezerra, it would be good to fix in devstack, but that has (potentially) larger implications | 19:58 |
gabriel-bezerra | morganfainberg: pardon me | 19:59 |
morganfainberg | gabriel-bezerra, :) ya did it right. | 19:59 |
gabriel-bezerra | morganfainberg: devstack already sources lib/apache before lib/keystone | 19:59 |
mtreinish | sdague: yeah it works fine with that update call | 19:59 |
mtreinish | I was able to login into it | 19:59 |
gabriel-bezerra | morganfainberg: you mean make each lib/XXX source its dependencies instead of the global runners? | 20:00 |
morganfainberg | gabriel-bezerra, yep. but i think this fix to grenade is the right one to start with. | 20:00 |
morganfainberg | gabriel-bezerra, there might be bigger pitfalls to fixing devstack to source it's deps in like that | 20:01 |
gabriel-bezerra | morganfainberg: I agree with you. | 20:01 |
morganfainberg | gabriel-bezerra, :) | 20:01 |
gabriel-bezerra | morganfainberg: and I hope this fix to be enough to make my patch to devstack pass the tests. | 20:01 |
morganfainberg | gabriel-bezerra, we can hope! | 20:02 |
gabriel-bezerra | :) | 20:02 |
*** mlavalle has quit IRC | 20:02 | |
*** levi_b has joined #openstack-qa | 20:02 | |
*** luqas has quit IRC | 20:02 | |
*** mlavalle has joined #openstack-qa | 20:05 | |
*** nati_ueno has joined #openstack-qa | 20:06 | |
*** jaypipes has quit IRC | 20:08 | |
*** nati_ueno has quit IRC | 20:11 | |
*** cdent_ has joined #openstack-qa | 20:12 | |
mtreinish | sdague: oh, I didn't realize you had patched create in that commit. I think that'll work too if you change properties to additionalProperties | 20:12 |
*** cdent has quit IRC | 20:14 | |
*** cdent_ is now known as cdent | 20:14 | |
*** bkopilov has joined #openstack-qa | 20:14 | |
sdague | ok, maybe I'll try that path, because this isn't working as expected | 20:17 |
sdague | 2014-05-28 16:16:24.611 8010 CRITICAL tempest [-] Bad request | 20:17 |
sdague | Details: 400 Bad Request | 20:17 |
sdague | Operation objects must contain exactly one member named "add", "remove", or "replace". | 20:17 |
*** mriedem has quit IRC | 20:17 | |
*** mriedem has joined #openstack-qa | 20:18 | |
mtreinish | sdague: wait on the rev I uploaded? | 20:21 |
sdague | no, on mine | 20:21 |
*** cdent has quit IRC | 20:26 | |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: wip: javelin 2 https://review.openstack.org/95148 | 20:26 |
sdague | mtreinish: ok, so the following gives me | 20:26 |
sdague | http://paste.openstack.org/show/81930/ | 20:27 |
*** levi_b has quit IRC | 20:30 | |
sdague | gah, ok, I have it | 20:30 |
*** levi_b has joined #openstack-qa | 20:30 | |
mtreinish | sdague: hmm, yeah it defines additionalProperties as a string type: http://docs.openstack.org/api/openstack-image-service/2.0/content/get-image-schema.html | 20:30 |
mtreinish | ok cool | 20:30 |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: wip: javelin 2 https://review.openstack.org/95148 | 20:30 |
*** salv-orlando has joined #openstack-qa | 20:30 | |
sdague | you actually just specify as top level | 20:31 |
sdague | and it bins it right on it's side | 20:31 |
mtreinish | ok cool | 20:31 |
mtreinish | want to spin the image client part off as a separate patch? With tests that use it... | 20:32 |
sdague | so probably yes, but I want to actually get the rest of this working first | 20:32 |
*** andreaf has quit IRC | 20:32 | |
*** levi_b has quit IRC | 20:34 | |
mtreinish | fair enough | 20:35 |
*** mestery has quit IRC | 20:39 | |
*** mestery has joined #openstack-qa | 20:40 | |
*** nati_ueno has joined #openstack-qa | 20:50 | |
openstackgerrit | A change was merged to openstack-dev/grenade: don't corrupt grenade exit value https://review.openstack.org/95934 | 20:50 |
*** levi_b has joined #openstack-qa | 20:54 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/tempest: Updated from global requirements https://review.openstack.org/96274 | 21:01 |
*** jchai has quit IRC | 21:02 | |
*** Sukhdev has quit IRC | 21:04 | |
*** aysyd has quit IRC | 21:06 | |
openstackgerrit | Julien Vey proposed a change to openstack-dev/devstack: Add an environment variable to enable extensions in keystone https://review.openstack.org/95778 | 21:06 |
*** levi_b has quit IRC | 21:07 | |
*** levi_b has joined #openstack-qa | 21:07 | |
*** levi_b has quit IRC | 21:12 | |
*** Sukhdev has joined #openstack-qa | 21:15 | |
gabriel-bezerra | mtreinish: might you take a look at https://review.openstack.org/96244 ? | 21:24 |
gabriel-bezerra | I couldn't find you with Gerrit's reviewer search | 21:24 |
mtreinish | gabriel-bezerra: sure I can, but I'm not core on grenade. | 21:27 |
*** ekhugen has quit IRC | 21:28 | |
gabriel-bezerra | mtreinish: that's fine | 21:30 |
gabriel-bezerra | no problems | 21:30 |
openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: be a little more clever about interface default https://review.openstack.org/96288 | 21:32 |
mtreinish | sdague: ^^^ what about wireless or bonded interfaces... | 21:38 |
sdague | you can't bridget wireless | 21:38 |
sdague | bridge | 21:38 |
sdague | in the base case | 21:38 |
sdague | and, bonded people can override :) | 21:39 |
sdague | the point being that eth0 is now not only wrong, but impossibly wrong on new Linux | 21:39 |
sdague | so we could at least do only wrong | 21:39 |
mtreinish | heh, ok fair enough. I was just trolling because of device names on systemd that don't start with e or p | 21:39 |
mtreinish | I think those are the only 2 | 21:40 |
mtreinish | or at least the only ones I've encountered | 21:40 |
sdague | mtreinish: what do systemd devices start as? | 21:41 |
mtreinish | no that matches systemd, AFAIC there's no p with systemd it's all en, wl, and ww | 21:42 |
mtreinish | except on my boxes with bonded controllers | 21:42 |
mtreinish | then its bond0 | 21:42 |
sdague | ww ? | 21:42 |
mtreinish | WWAN | 21:42 |
mtreinish | I just read the man page | 21:42 |
sdague | I thought p1s5 was the new hotness for pci | 21:42 |
mtreinish | I've never seen it | 21:42 |
mtreinish | that's the ubuntu (or fedora) thing, I've never seen it on arch. | 21:43 |
sdague | yeh, well arch doesn't count for devstack :) | 21:43 |
*** TravT has joined #openstack-qa | 21:43 | |
sdague | gabriel-bezerra: if the grenade tests pass on your last patch I'll approve it through | 21:44 |
mtreinish | one day, just wait... | 21:44 |
sdague | just want to see the test results roll in first | 21:44 |
mrodden | someone is changing from the ethN convention? | 21:44 |
sdague | mrodden: all of them are | 21:44 |
mrodden | wtf? | 21:44 |
mtreinish | mrodden: you have to stop using rhel... | 21:44 |
mtreinish | (and windows) | 21:44 |
mrodden | gentoo | 21:44 |
mtreinish | mrodden: oh sorry mriedem is the windows, rhel py26 guy | 21:45 |
mtreinish | yeah I guess on gentoo systemd is optional | 21:45 |
mrodden | i thought we were done with the interface device naming problem | 21:45 |
mrodden | gentoo forked udev so they didnt' have to deal with systemd | 21:45 |
mtreinish | mrodden: http://www.freedesktop.org/wiki/Software/systemd/PredictableNetworkInterfaceNames/ | 21:46 |
mrodden | eudev isn't so hot though | 21:46 |
mrodden | i hate systemd | 21:46 |
gabriel-bezerra | sdague: thank you :) | 21:46 |
mrodden | en is AIX | 21:47 |
mrodden | naming schema | 21:47 |
mrodden | /barf | 21:47 |
mtreinish | sdague: for your regex: http://cgit.freedesktop.org/systemd/systemd/tree/src/udev/udev-builtin-net_id.c#n29 | 21:48 |
* mriedem goes back to watching tox | 21:48 | |
*** nati_ueno has quit IRC | 21:49 | |
mtreinish | sdague: oh, the example for ww is a 3g modem. That seems like something you want to bridge on for devstack :) | 21:49 |
mrodden | ubuntu isn't using systemd are they? | 21:49 |
mtreinish | mrodden: not yet, but they have a similar naming scheme thing going on | 21:50 |
mrodden | ugh.... | 21:50 |
mtreinish | or I think they did on the last ubuntu box I ran which was a while ago | 21:50 |
sdague | mrodden: they will in 14.10 | 21:52 |
sdague | debian decided to go systemd | 21:52 |
* mrodden goes back to slackware | 21:53 | |
mrodden | actually i did have a question | 21:54 |
mrodden | would 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 |
sdague | mrodden: give me a review url and I'll go through them quickly | 21:56 |
mtreinish | mrodden: you're the ptl, you get to set the tone of the project :) | 21:56 |
mtreinish | mrodden: btw, who is the core group on bash8? | 21:56 |
mrodden | sdague, dtroyer and myself i think | 21:56 |
sdague | I though chmouel as well | 21:56 |
mrodden | let me check | 21:56 |
sdague | I suggested it was devstack-core + mrodden when it was built | 21:57 |
sdague | which might also still mean jeblair | 21:57 |
mtreinish | ok, that makes sense. I was just curious | 21:57 |
mrodden | https://review.openstack.org/#/admin/groups/333,members | 21:57 |
*** dhellmann is now known as dhellmann_ | 21:58 | |
mrodden | i can add chmouel | 21:58 |
mrodden | i was wondering if he wanted to be core since he was doing reviews | 21:58 |
*** mriedem has quit IRC | 21:58 | |
mtreinish | mrodden: just add the devstack-core group. I think that makes sense | 21:58 |
*** dhellmann_ is now known as dhellmann | 21:58 | |
mrodden | ok | 21:58 |
*** dhellmann is now known as dhellmann_ | 21:58 | |
mrodden | done | 21:59 |
mrodden | hah sdague already found my two patches | 21:59 |
mrodden | was about to ask | 21:59 |
mrodden | thanks | 21:59 |
sdague | I'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 hold | 22:00 |
mrodden | oh ok | 22:00 |
mrodden | yeah there was a double bracket check on | 22:01 |
mrodden | one* | 22:01 |
mrodden | it was failing jenkins checks last time i looked at it | 22:01 |
sdague | does glance not handle simultaneous connects well? | 22:02 |
sdague | 2014-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] ENETUNREACH | 22:03 |
sdague | that's a new one to me | 22:03 |
mrodden | unreachable | 22:03 |
sdague | oh... son of a... | 22:03 |
sdague | I bet there is a window when the first bridge gets set up | 22:04 |
sdague | where the network goes funny for a sec | 22:04 |
*** atiwari has joined #openstack-qa | 22:05 | |
*** yamahata has joined #openstack-qa | 22:06 | |
*** nati_ueno has joined #openstack-qa | 22:07 | |
mrodden | moving the IP from the interface to the bridge would upset your routing table for bit | 22:07 |
mrodden | could result in unreachable | 22:08 |
*** gabriel-bezerra has quit IRC | 22:13 | |
*** jgrimm has quit IRC | 22:13 | |
*** gabriel-bezerra has joined #openstack-qa | 22:14 | |
sdague | yeh, that's what I'm guessing | 22:20 |
sdague | what's problematic is that it's a deep nova-compute failure | 22:20 |
sdague | with no retry | 22:21 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for 1320617 https://review.openstack.org/94080 | 22:23 |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: wip: javelin 2 https://review.openstack.org/95148 | 22:25 |
*** openstackrecheck has quit IRC | 22:25 | |
*** openstackrecheck has joined #openstack-qa | 22:25 | |
jogo | heh the week of the summit we only merged 100 patches | 22:28 |
*** weshay has quit IRC | 22:29 | |
*** atiwari has quit IRC | 22:32 | |
*** levi_b has joined #openstack-qa | 22:32 | |
*** mrodden has quit IRC | 22:34 | |
*** levi_b has quit IRC | 22:36 | |
*** dims has quit IRC | 22:51 | |
*** dims has joined #openstack-qa | 22:52 | |
*** rbak has quit IRC | 22:57 | |
*** bkopilov has quit IRC | 22:59 | |
*** yamahata has quit IRC | 23:09 | |
*** Obulpathi has quit IRC | 23:10 | |
*** bkopilov has joined #openstack-qa | 23:13 | |
*** Sukhdev has quit IRC | 23:15 | |
*** amotoki has joined #openstack-qa | 23:20 | |
*** salv-orlando has quit IRC | 23:24 | |
*** salv-orlando has joined #openstack-qa | 23:29 | |
*** mmaglana has quit IRC | 23:40 | |
*** tosky has quit IRC | 23:40 | |
*** mkollaro has quit IRC | 23:45 | |
*** oomichi has joined #openstack-qa | 23:46 | |
*** salv-orlando has quit IRC | 23:52 | |
*** jaypipes has joined #openstack-qa | 23:54 | |
*** ChanServ sets mode: +o jaypipes | 23:54 | |
openstackgerrit | A change was merged to openstack-dev/grenade: Source lib/apache before lib/keystone https://review.openstack.org/96244 | 23:55 |
*** bkopilov has quit IRC | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!