openstackgerrit | Keith Schincke proposed openstack/puppet-ceph: Add support for civetweb https://review.openstack.org/254548 | 00:04 |
---|---|---|
*** ducttape_ has quit IRC | 00:12 | |
*** markvoelker has quit IRC | 00:15 | |
*** markvoelker has joined #puppet-openstack | 00:46 | |
*** xingchao has joined #puppet-openstack | 00:48 | |
*** xingchao has quit IRC | 00:53 | |
*** saneax is now known as saneax_AFK | 00:56 | |
*** ducttape_ has joined #puppet-openstack | 00:59 | |
openstackgerrit | Merged openstack/puppet-oslo: Add define oslo::messaging::amqp https://review.openstack.org/274764 | 01:06 |
*** imcsk8_ has joined #puppet-openstack | 01:26 | |
openstackgerrit | Matt Fischer proposed openstack/puppet-keystone: WIP Keystone hooks support https://review.openstack.org/283898 | 01:32 |
*** v1k0d3n_ has joined #puppet-openstack | 01:35 | |
*** v1k0d3n__ has joined #puppet-openstack | 01:35 | |
*** v1k0d3n has quit IRC | 01:37 | |
*** severion has quit IRC | 01:37 | |
*** xarses has joined #puppet-openstack | 01:39 | |
*** xarses has quit IRC | 01:39 | |
*** xarses has joined #puppet-openstack | 01:40 | |
*** ducttape_ has quit IRC | 01:42 | |
*** xingchao has joined #puppet-openstack | 01:49 | |
*** baker has joined #puppet-openstack | 01:54 | |
*** xingchao has quit IRC | 01:55 | |
*** xingchao has joined #puppet-openstack | 02:01 | |
*** brandon has quit IRC | 02:05 | |
*** xingchao has quit IRC | 02:17 | |
*** xingchao has joined #puppet-openstack | 02:20 | |
*** xingchao has quit IRC | 02:25 | |
*** baker has quit IRC | 02:31 | |
*** baker has joined #puppet-openstack | 02:40 | |
*** baker has quit IRC | 02:43 | |
*** ilbot3 has quit IRC | 02:46 | |
*** ilbot3 has joined #puppet-openstack | 02:47 | |
openstackgerrit | Michael Chapman proposed openstack/puppet-ceilometer: Add event_pipeline.yaml management https://review.openstack.org/283912 | 02:48 |
*** gildub has joined #puppet-openstack | 02:49 | |
*** xingchao has joined #puppet-openstack | 02:54 | |
*** xingchao has quit IRC | 02:54 | |
*** xingchao has joined #puppet-openstack | 02:55 | |
*** baker has joined #puppet-openstack | 03:03 | |
*** furlongm has quit IRC | 03:20 | |
*** mdorman has quit IRC | 03:26 | |
*** gildub has quit IRC | 03:30 | |
*** gildub has joined #puppet-openstack | 03:30 | |
*** davideagle has quit IRC | 03:35 | |
openstackgerrit | Merged openstack/puppet-keystone: Drop User/group resource dependency https://review.openstack.org/195684 | 03:36 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: tempest: run tox with -eall-plugin https://review.openstack.org/283786 | 03:41 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: neutron: enable fwaas https://review.openstack.org/283920 | 03:41 |
openstackgerrit | Matt Fischer proposed openstack/puppet-keystone: WIP Keystone hooks support https://review.openstack.org/283898 | 03:55 |
*** ducttape_ has joined #puppet-openstack | 03:58 | |
openstackgerrit | Matt Fischer proposed openstack/puppet-keystone: stop managing files that the package makes https://review.openstack.org/283308 | 03:59 |
*** furlongm has joined #puppet-openstack | 04:02 | |
openstackgerrit | Michael Chapman proposed openstack/puppet-ceilometer: Add event_pipeline.yaml management https://review.openstack.org/283912 | 04:08 |
michchap | EmilienM: fyi you'll need that event_pipeline.yaml thing for aodh in tripleo | 04:08 |
michchap | EmilienM: we have it working in apex. Here's the 3 patches to enable: https://github.com/trozet/opnfv-tht/commit/ae0c6365123fb47d243096aaf102ac985fe59fac, https://github.com/trozet/opnfv-tht/commit/ae0c6365123fb47d243096aaf102ac985fe59fac, https://github.com/trozet/opnfv-tht/commit/dc6bbb7b3d46096925b1fb2ea65be7ab2114d714 | 04:09 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: run dstat before r10k'ing modules https://review.openstack.org/283926 | 04:10 |
EmilienM | michchap: hey, what is apex? | 04:11 |
EmilienM | michchap: I don't like to manage files with puppet openstack modules -- imho it should be managed by packaging. | 04:11 |
michchap | EmilienM: it's a downstream of RDO for installing OPNFV, which is basically openstack with some more networking options | 04:11 |
michchap | EmilienM: yeah I just removed the user/group, I'm not sure how often that file needs to be changed by an operator tbh | 04:12 |
EmilienM | michchap: I'm not sure why you need to enable it by default. | 04:12 |
EmilienM | our CI is running with aodh without that file. | 04:12 |
michchap | does aodh actually work? | 04:12 |
EmilienM | yep. | 04:12 |
EmilienM | we gate it with tempest | 04:12 |
EmilienM | it's even documented: https://github.com/openstack/puppet-openstack-integration#description | 04:13 |
michchap | perhaps it's a recent change | 04:13 |
EmilienM | https://github.com/openstack/puppet-openstack-integration/blob/master/run_tests.sh#L102 | 04:13 |
EmilienM | we run Aodh since ~3 months | 04:13 |
EmilienM | and we run OpenStack from trunk | 04:13 |
EmilienM | anyway. If we need this yaml file, IMHO we should manage it in packaging and config it with puppet | 04:13 |
EmilienM | if that's not possible, np but don't hardcode user/group. | 04:14 |
michchap | interesting. We have frozen Liberty packages from december so perhaps there was something merged to make it work after that | 04:14 |
EmilienM | and IMHO, no need to create the file by default. | 04:14 |
EmilienM | we run Aodh on Liberty & Mitaka iirc | 04:14 |
michchap | yeah the manage option is false by default | 04:14 |
EmilienM | michchap: the doc is wrong though, irrc | 04:15 |
openstackgerrit | Merged openstack/puppet-tripleo: Override X-Forwarded-Proto header https://review.openstack.org/283074 | 04:15 |
EmilienM | iirc* | 04:15 |
michchap | EmilienM: yeah I just fixed it | 04:15 |
*** ducttape_ has quit IRC | 04:15 | |
EmilienM | mfisch: I've noticed r10k randomly take HUGE time to be installed | 04:15 |
EmilienM | that's why our CI jobs sometimes timout | 04:15 |
EmilienM | I'm doing that : https://review.openstack.org/283926 to help in debug this issue | 04:16 |
*** richm has quit IRC | 04:20 | |
colonwq | EmilienM, your advice helped! My patch passed all voting tests this evening. | 04:22 |
*** yolanda has joined #puppet-openstack | 04:24 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: generate testrepository.subunit all the time https://review.openstack.org/283930 | 04:24 |
EmilienM | colonwq: sorry for lag, it was a busy day | 04:25 |
EmilienM | colonwq: though I think your patch is not passing puppet-latest while it was working before | 04:26 |
EmilienM | have you investigated that? | 04:26 |
colonwq | EmilienM, Understandable. | 04:26 |
colonwq | EmilienM, Ill look at it tonight. | 04:26 |
EmilienM | http://logs.openstack.org/48/254548/59/check/gate-puppet-ceph-puppet-unit-latest-dsvm-centos7/0863b85/console.html#_2016-02-24_00_35_54_867 | 04:27 |
EmilienM | I think it's just a fact missing in the tests | 04:27 |
colonwq | yeah. I have seen multiple places where versioncmp is a puppet error. I am probably doing the same thing in multiple places. | 04:29 |
openstackgerrit | Merged openstack/puppet-zaqar: Switch zaqar to use rspec-puppet-facts https://review.openstack.org/282165 | 04:32 |
*** fedexo has joined #puppet-openstack | 04:37 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: CI run https://review.openstack.org/283931 | 04:39 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: CI run https://review.openstack.org/283932 | 04:40 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: CI run https://review.openstack.org/283933 | 04:40 |
EmilienM | ok trying to run multiple CI runs and see how often we have timeouts | 04:41 |
*** xingchao has quit IRC | 04:46 | |
openstackgerrit | Merged openstack/puppet-cinder: Set default branch to stable/liberty in .gitreview https://review.openstack.org/281997 | 04:49 |
*** baker has quit IRC | 04:55 | |
*** baker has joined #puppet-openstack | 04:55 | |
*** baker has quit IRC | 05:15 | |
openstackgerrit | Merged openstack/puppet-cinder: Allow customization of DEFAULT/host https://review.openstack.org/281998 | 05:21 |
openstackgerrit | Merged openstack/puppet-openstack-integration: keystone: manage admin user role before openrc https://review.openstack.org/283734 | 05:21 |
*** fedexo has quit IRC | 05:23 | |
openstackgerrit | Keith Schincke proposed openstack/puppet-ceph: Add support for civetweb https://review.openstack.org/254548 | 05:28 |
* colonwq drops the mic until tomorrow. | 05:28 | |
*** vinsh has quit IRC | 05:41 | |
*** v1k0d3n_ has quit IRC | 05:53 | |
*** v1k0d3n has joined #puppet-openstack | 05:53 | |
*** v1k0d3n__ has quit IRC | 05:56 | |
*** severion has joined #puppet-openstack | 05:56 | |
*** v1k0d3n_ has joined #puppet-openstack | 06:06 | |
*** v1k0d3n has quit IRC | 06:07 | |
*** saneax_AFK is now known as saneax | 06:16 | |
*** severion has quit IRC | 06:20 | |
*** v1k0d3n_ has quit IRC | 06:20 | |
*** xingchao has joined #puppet-openstack | 06:21 | |
*** openstack has joined #puppet-openstack | 13:22 | |
*** openstackstatus has joined #puppet-openstack | 13:22 | |
*** ChanServ sets mode: +v openstackstatus | 13:22 | |
openstackgerrit | Nikolay Starodubtsev proposed openstack/puppet-murano: Switch murano to use keystone API v3 by default https://review.openstack.org/279444 | 13:30 |
*** 32NAAD4T7 has quit IRC | 13:31 | |
*** jpena is now known as jpena|lunch | 13:46 | |
openstackgerrit | Nikolay Starodubtsev proposed openstack/puppet-murano: Switch murano to use keystone API v3 by default https://review.openstack.org/279444 | 13:51 |
*** xingchao has joined #puppet-openstack | 13:54 | |
openstackgerrit | Merged openstack/puppet-nova: Remove installation of pm-utils https://review.openstack.org/284032 | 13:59 |
*** saneax_AFK is now known as saneax | 14:02 | |
*** ayoung has quit IRC | 14:04 | |
openstackgerrit | Denis Egorenko proposed openstack/puppet-heat: Use identity_uri and auth_uri by default https://review.openstack.org/261326 | 14:13 |
*** jaosorior has quit IRC | 14:18 | |
*** jaosorior has joined #puppet-openstack | 14:19 | |
*** bklei has joined #puppet-openstack | 14:19 | |
*** severion has quit IRC | 14:22 | |
*** severion has joined #puppet-openstack | 14:22 | |
openstackgerrit | Nikolay Starodubtsev proposed openstack/puppet-murano: Switch murano to use keystone API v3 by default https://review.openstack.org/279444 | 14:23 |
*** severion has quit IRC | 14:23 | |
*** v1k0d3n has quit IRC | 14:24 | |
*** v1k0d3n has joined #puppet-openstack | 14:24 | |
openstackgerrit | Gael Chamoulaud proposed openstack/puppet-heat: Revise keystone/trusts config to avoid deprecated options https://review.openstack.org/265865 | 14:33 |
*** saneax is now known as saneax_AFK | 14:37 | |
*** mdorman has joined #puppet-openstack | 14:40 | |
*** richm has joined #puppet-openstack | 14:44 | |
*** ayoung has joined #puppet-openstack | 14:46 | |
EmilienM | degorenko: no need to do recheck on my patches | 14:48 |
EmilienM | they fail for a reason, if you look at logs | 14:48 |
*** karimb has quit IRC | 14:49 | |
degorenko | EmilienM, here are a couple of patches, which was failed because of some problem on hosts | 14:49 |
degorenko | like not enough RAM or disk space | 14:49 |
degorenko | may be i missed somewhere this | 14:49 |
EmilienM | indeed, I noticed CI issues yesterday | 14:50 |
EmilienM | it's probably due to my recent work on getting more testing | 14:50 |
*** foxhunt72 is now known as foxhunt_away | 14:52 | |
*** jpena|lunch is now known as jpena | 14:55 | |
EmilienM | mwhahaha: if you around can you please review https://review.openstack.org/#/c/283926/ and https://review.openstack.org/283930 ? it will help to find CI issues | 14:55 |
EmilienM | degorenko: I've noticed sometimes, nodes running on internap cloud do timeouts | 14:56 |
*** xarses has quit IRC | 14:57 | |
*** dfisher has joined #puppet-openstack | 14:58 | |
openstackgerrit | Denis Egorenko proposed openstack/puppet-cinder: Replace defines for managing cinder types with providers https://review.openstack.org/273513 | 14:59 |
*** ayoung has quit IRC | 14:59 | |
*** ducttape_ has joined #puppet-openstack | 15:01 | |
*** dhollinger has joined #puppet-openstack | 15:09 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: neutron: enable fwaas https://review.openstack.org/283920 | 15:10 |
openstackgerrit | Matt Fischer proposed openstack/puppet-cinder: module to set quotas per volume type https://review.openstack.org/197572 | 15:17 |
*** xarses has joined #puppet-openstack | 15:19 | |
openstackgerrit | Matt Fischer proposed openstack/puppet-keystone: stop managing files that the package makes https://review.openstack.org/283308 | 15:20 |
openstackgerrit | Merged openstack/puppet-designate: Set correct options path for bind9 agent https://review.openstack.org/267856 | 15:22 |
*** yolanda has quit IRC | 15:24 | |
*** baker has joined #puppet-openstack | 15:25 | |
*** ddmitriev has joined #puppet-openstack | 15:26 | |
*** dhollinger has quit IRC | 15:31 | |
*** paramite is now known as paramite|afk | 15:32 | |
*** paramite|afk has quit IRC | 15:38 | |
EmilienM | degorenko: look at https://review.openstack.org/#/c/283933/ | 15:40 |
degorenko | ack | 15:40 |
EmilienM | the failing job | 15:40 |
EmilienM | we have this randomly | 15:40 |
EmilienM | and it's running on internap | 15:41 |
EmilienM | degorenko: going on #openstack-infra | 15:41 |
degorenko | yeah, looks like internap slaves are broken | 15:41 |
EmilienM | degorenko: http://logs.openstack.org/33/283933/1/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/58804b9/console.html#_2016-02-24_04_50_39_450 | 15:42 |
*** johnmilton has quit IRC | 15:42 | |
degorenko | EmilienM, some problems with gem installation? | 15:43 |
* degorenko afk on few minutes | 15:43 | |
EmilienM | I have no idea | 15:44 |
*** dhollinger has joined #puppet-openstack | 15:47 | |
*** haplo37 has joined #puppet-openstack | 15:52 | |
*** paramite|afk has joined #puppet-openstack | 15:53 | |
*** jaosorior has quit IRC | 15:55 | |
*** johnmilton has joined #puppet-openstack | 15:58 | |
*** paramite|afk has quit IRC | 15:58 | |
degorenko | EmilienM, no, gem install same | 16:01 |
degorenko | for other slaves | 16:01 |
EmilienM | degorenko: what? | 16:01 |
degorenko | i'm about gem installing | 16:02 |
EmilienM | degorenko: you say it also happens on ovh & bluebox? | 16:02 |
degorenko | EmilienM, http://logs.openstack.org/33/283933/1/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-centos7/0a60c3d/console.html#_2016-02-24_04_56_41_813 | 16:02 |
degorenko | yes | 16:02 |
EmilienM | degorenko: no | 16:03 |
EmilienM | it takes 15 seconds | 16:03 |
EmilienM | that's fine | 16:03 |
EmilienM | I'm taking about 15 min here, on internap. | 16:03 |
degorenko | щр нуфр | 16:04 |
degorenko | :D | 16:04 |
EmilienM | ? | 16:04 |
degorenko | oh yeah, you're right | 16:04 |
*** alee has quit IRC | 16:04 | |
*** alee has joined #puppet-openstack | 16:05 | |
degorenko | gchamoul, ping | 16:08 |
*** hjensas has quit IRC | 16:10 | |
gchamoul | degorenko: yes sir? | 16:13 |
*** dhollinger has quit IRC | 16:13 | |
degorenko | gchamoul, hey, i saw that your patch for heat trusts passed ci | 16:13 |
degorenko | and mine is not | 16:13 |
gchamoul | yes just saw that! | 16:14 |
*** leanderthal is now known as leanderthal|afk | 16:14 | |
degorenko | so, i guess, there are some values that help you did that | 16:14 |
degorenko | probably you don't need my patch, or we should mix our patches in one | 16:14 |
gchamoul | degorenko: let me do our daily and I will try to give you a better answer in ~20 min | 16:15 |
degorenko | gchamoul, ack, will wait | 16:15 |
*** ayoung has joined #puppet-openstack | 16:20 | |
*** dhollinger has joined #puppet-openstack | 16:26 | |
openstackgerrit | Bradley Klein proposed openstack/puppet-monasca: Support dynamic mysql port https://review.openstack.org/284240 | 16:34 |
*** xingchao has quit IRC | 16:35 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: install_modules.sh: run gem install with --debug https://review.openstack.org/283696 | 16:36 |
gchamoul | degorenko: well, I was just needing the new auth_uri parameter! | 16:38 |
gchamoul | degorenko: but yes ... I think we can mix our patches in one | 16:39 |
*** samuelBartel has joined #puppet-openstack | 16:39 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: install_modules.sh: run gem install with --verbose https://review.openstack.org/283696 | 16:41 |
*** ducttape_ has quit IRC | 16:42 | |
*** ducttape_ has joined #puppet-openstack | 16:43 | |
degorenko | gchamoul, so, issue for my patch is in client_keystone/auth_uri you think? | 16:43 |
*** ducttape_ has quit IRC | 16:45 | |
*** jistr has quit IRC | 16:45 | |
*** ducttape_ has joined #puppet-openstack | 16:45 | |
degorenko | gchamoul, i think, that problem is in username and password for trustee section | 16:47 |
gchamoul | degorenko: not sure .. but you started to add trustee_domain stuff | 16:47 |
gchamoul | yes I remember you added them in your PS#6 or 7 | 16:48 |
openstackgerrit | ayoung proposed openstack/puppet-keystone: Notify Apache HTTPD as a class for changes to Keystone https://review.openstack.org/276285 | 16:49 |
degorenko | i didn't, only auth_uri | 16:49 |
ayoung | i AM TOTALLY cargo CULTING ALL MY pUPPeT WORk | 16:49 |
ayoung | ^^ is for Tripleo. And I have no idea if it makes sense or not. | 16:50 |
gchamoul | degorenko: in the patchset8 https://review.openstack.org/#/c/261326/8/manifests/init.pp | 16:50 |
degorenko | gchamoul, well, anyway, we have working v3 auth for our pathces, but we also must check v2.0 | 16:50 |
degorenko | gchamoul, oh yeah, domains | 16:50 |
degorenko | you're right | 16:50 |
degorenko | gchamoul, i will update my change with trustee username password - will check how it will wokr | 16:52 |
*** chem has quit IRC | 16:52 | |
gchamoul | ack | 16:52 |
degorenko | please, do not change your patch :) we must be sure, that it will work | 16:52 |
EmilienM | ayoung: I'm commenting. | 16:53 |
ayoung | EmilienM, do you understand all this? | 16:53 |
ayoung | I mean, I get the theory, but why do the services even depend on each other? | 16:54 |
ayoung | Who cares if I restart Keystone? Keystone should not trigger restartes in other services, no matter where it is run | 16:54 |
EmilienM | ayoung: I do. | 16:54 |
EmilienM | ayoung: and I have a solution | 16:55 |
ayoung | Excellent | 16:55 |
EmilienM | we already work on a solution | 16:55 |
EmilienM | let me comment. | 16:55 |
ayoung | EmilienM, I can abandon that review if it is in the way | 16:56 |
Guest51435 | EmilienM: http://paste.openstack.org/show/488050/ | 16:56 |
*** Guest51435 is now known as mgagne | 16:56 | |
*** mgagne has quit IRC | 16:56 | |
*** mgagne has joined #puppet-openstack | 16:56 | |
ayoung | I just need to make sure I don't turn around and find out that Tripleo has reverted. | 16:57 |
openstackgerrit | Denis Egorenko proposed openstack/puppet-heat: Use identity_uri and auth_uri by default https://review.openstack.org/261326 | 16:57 |
mgagne | EmilienM: that was me ^ | 16:57 |
mgagne | I'm not sure what I'm supposed to look for. I installed all the packages installed by the job and then installed r10k, no issue there | 16:57 |
EmilienM | ayoung: commented. | 16:58 |
mgagne | could it be related to the image used in CI? Or intermittent network issues? | 16:58 |
mgagne | moving to infra =) | 16:58 |
EmilienM | mgagne: I trust you but why do we have problem? | 16:58 |
mgagne | EmilienM: I don't know =) | 16:58 |
ayoung | EmilienM, ok, abandoned it again. | 16:58 |
mgagne | no issue found in rubygems status page | 16:58 |
*** samuelBartel has quit IRC | 17:06 | |
EmilienM | I'm trying to create a scenario003: https://review.openstack.org/284271 | 17:21 |
EmilienM | I think our scenarios are currently strugling in resources. | 17:21 |
openstackgerrit | Jason Guiditta proposed openstack/puppet-tripleo: loadbalancer: fix MySQL timeout HAproxy config https://review.openstack.org/284274 | 17:26 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: neutron: enable fwaas https://review.openstack.org/283920 | 17:27 |
dmsimard | dgurtner: so | 17:28 |
dmsimard | dgurtner: we have a merge conflict waiting to happen | 17:29 |
dmsimard | https://review.openstack.org/#/c/266027/ vs https://review.openstack.org/#/c/254548/ | 17:29 |
dmsimard | EmilienM: wow r10k took 12 minutes to run ? http://logs.openstack.org/48/254548/60/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-centos7/5600d56/console.html#_2016-02-24_05_46_53_907 | 17:31 |
dmsimard | the job timed out, in part because of that | 17:31 |
*** samuelBartel has joined #puppet-openstack | 17:31 | |
EmilienM | dmsimard: I sent an email to os-infra | 17:32 |
dmsimard | oh ? | 17:32 |
EmilienM | and it happens only on internap cloud | 17:32 |
EmilienM | I pinged mgagne already | 17:32 |
EmilienM | and we're investigating | 17:32 |
dmsimard | ok | 17:32 |
EmilienM | I added verbose option to gem https://review.openstack.org/283696 | 17:32 |
EmilienM | so I'm on it | 17:32 |
EmilienM | in the meantime: https://review.openstack.org/284271 | 17:32 |
dmsimard | EmilienM: so you should rename the current "aio" scenario003 to scenario-aio or something | 17:34 |
*** rahulait has joined #puppet-openstack | 17:34 | |
EmilienM | yeah I'll do that | 17:34 |
dmsimard | but ok | 17:34 |
EmilienM | getting upstream CI back to a stable shape is my priority #1 today | 17:34 |
EmilienM | we are testing too much | 17:35 |
dmsimard | lol | 17:36 |
dmsimard | you're saying that like it's a bad thing | 17:36 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Reduce number of workers to 2 when possible https://review.openstack.org/284289 | 17:37 |
EmilienM | it's a good thing | 17:37 |
EmilienM | but we need to scale our CI arch now | 17:37 |
EmilienM | because we have more testing coming: zaqar, murano, etc etc | 17:37 |
dmsimard | EmilienM: for swift you also have these https://github.com/openstack/puppet-swift/blob/2ffbf4527773f94425bde2e9df12a18fe95fef4c/manifests/storage/server.pp#L126-L128 | 17:40 |
dmsimard | re: workers | 17:40 |
dmsimard | they're not exposed by swift::storage::all, though, so perhaps you can set the define defaults or something | 17:41 |
EmilienM | dmsimard: https://bugs.launchpad.net/puppet-tempest/+bug/1549366/comments/1 | 17:41 |
openstack | Launchpad bug 1549366 in puppet-tempest "puppet-tempest should be able to install from binary" [Undecided,New] | 17:41 |
EmilienM | dmsimard: ok thx for swift | 17:42 |
EmilienM | default is 1, not too bad (swift storage worker) | 17:43 |
dmsimard | EmilienM: other puppet modules ensure the package is there | 17:43 |
dmsimard | EmilienM: but I think your comment is a good workaround | 17:43 |
EmilienM | dmsimard: wait I don't say puppet-tempest is like other for packaging. I just say, out of the box you can use it with your own tempest packaging | 17:43 |
dmsimard | yeah. | 17:44 |
EmilienM | I did it already, back in time I worked for enovance | 17:44 |
dmsimard | It's not a bug so much as it is a RFE for adding support for binary install from within puppet-tempest | 17:44 |
dmsimard | I mean, otherwise I can monkey patch everything in my composition layer because support isn't upstream :p | 17:44 |
EmilienM | no you're right, let's add it in puppet-tempest | 17:44 |
dmsimard | It's not a priority or anything, just thought I'd formalize the need for it | 17:45 |
EmilienM | nice | 17:45 |
EmilienM | I updated the bug | 17:45 |
dgurtner | dmsimard: tell me? | 17:49 |
dgurtner | dmsimard: agreed, but tbh. it's not too difficult to rebase either or the other | 17:50 |
dgurtner | dmsimard: while I feel bad for Keith after the tens of revisions he already went through, Oleksiy's patch is passing tests since a while and is right now while Keiths isn't | 17:51 |
dmsimard | EmilienM: added a comment with a couple more worker tweak opportunities | 17:55 |
EmilienM | dmsimard: all ::api are managed by apache | 17:55 |
EmilienM | dmsimard: so look at the params in the wsgi class | 17:55 |
dmsimard | dgurtner: Keith's patch is not passing by a third party test (puppet-openstack) and the test failed due to an issue installing r10k (>12 minutes) so it timed out | 17:56 |
dgurtner | dmsimard: yup, just saw it | 17:57 |
dgurtner | dmsimard: soo... what's your suggestions? | 17:58 |
dgurtner | dmsimard: ask them who's willing to rebase on top of the other? | 17:58 |
dmsimard | It's hard to do a decision without making it look like it was biased by affiliation | 17:58 |
dgurtner | dmsimard: ;) | 17:58 |
dgurtner | dmsimard: tbh. I can do the rebase work if it's only about the extra work | 17:59 |
EmilienM | dmsimard: commented | 17:59 |
dmsimard | degorenko: is Oleksiy Molchanov on IRC ? | 17:59 |
degorenko | dmsimard, well, i don't know actually | 18:00 |
degorenko | let me ask him | 18:00 |
dmsimard | degorenko: We feel bad about https://review.openstack.org/#/c/266027/ vs https://review.openstack.org/#/c/254548/ - two large patches that will conflict when we land either | 18:01 |
dmsimard | and they are both ready to merge | 18:01 |
dmsimard | colonwq, xarses ^ | 18:01 |
*** derekh has quit IRC | 18:01 | |
degorenko | dmsimard, he is offline now, but as i see in fuel library channel, he has nick - omolchanov. | 18:02 |
degorenko | dmsimard, as i see second change has -1 from ci | 18:03 |
dmsimard | degorenko: yeah, copy pasting due to laziness: Keith's patch is not passing by a third party test (puppet-openstack) and the test failed due to an issue installing r10k (>12 minutes) so it timed out | 18:04 |
degorenko | dmsimard, oh yeah, the same problem as for all another modules, got it | 18:04 |
dmsimard | it's third party ci job provided by puppet-openstack | 18:04 |
degorenko | i know :) | 18:05 |
xarses | dmsimard: ya, I've been on the fence on who to merge first | 18:06 |
degorenko | let's merge first who has green tests ;) | 18:06 |
dmsimard | let's merge both simultaneously and let the god of the gate decide which goes in first | 18:07 |
dmsimard | no worse than a coin toss :P | 18:07 |
xarses | they are aligned so, we can fix whichever ends in conflict =/ | 18:08 |
dmsimard | colonwq: are you around ? | 18:08 |
degorenko | second one has -1 from check gate, it will not be merge, will not it? | 18:08 |
xarses | I cant remember if thats voting or not | 18:08 |
dmsimard | wait, puppet-openstack-scenario001 is voting ? | 18:10 |
xarses | EmilienM: is gate-puppet-openstack-integration-scenario001-tempest-dsvm-centos7 still flaky? | 18:10 |
dmsimard | that's a mistake, is it ? | 18:10 |
dmsimard | EmilienM: we need to make puppet-openstack jobs non-voting on puppet-ceph, they're there for feedback | 18:10 |
xarses | dmsimard: I think they are non-voiting on gate | 18:11 |
dmsimard | but they're voting on check | 18:11 |
xarses | yes | 18:11 |
dmsimard | so jenkins will -1 if check fails | 18:11 |
*** degorenko is now known as _degorenko|afk | 18:14 | |
*** gfidente has quit IRC | 18:14 | |
EmilienM | xarses: gate-puppet-openstack-integration-scenario00*-* are all flicky now | 18:14 |
EmilienM | I'm working on it today | 18:15 |
EmilienM | we are having lot of timeouts | 18:15 |
EmilienM | but they are supposed to be stable, that's why they vote. | 18:15 |
EmilienM | I don't see a reason to stop making them vote today, they fail ~20% and we are fixing it. | 18:15 |
EmilienM | dmsimard: yes it's voting for puppet-ceph and they are not only here for feedback IMHO. | 18:16 |
EmilienM | TripleO job is here for feedback, but IMHO, you should not disabling voting for the puppet-openstack-integration on puppet-ceph | 18:16 |
EmilienM | they will bring to puppet-ceph very useful feedback, we run a full scenario with RBD backend | 18:16 |
EmilienM | something that puppet-ceph tests do not provide. | 18:17 |
*** jtomasek has quit IRC | 18:21 | |
colonwq | dmsimard, I am back. | 18:28 |
*** jpena has quit IRC | 18:29 | |
dmsimard | colonwq: are you okay with rebasing your patch on https://review.openstack.org/#/c/266027/ ? Both are essentially ready for merge and we need one of the two patches to rebase on top of the other | 18:32 |
dmsimard | It's going to conflict. | 18:32 |
colonwq | dmsimard, I can see the conflict. merge 266027 and I'll rebase and change the rgw_frontend_type to frontend_type | 18:33 |
dmsimard | colonwq: okay, thanks for your understanding | 18:33 |
colonwq | mine does have a higher number set count though. :) | 18:33 |
dmsimard | meanwhile, don't worry about the current failed test, it's a gate environment problem that is being addressed | 18:33 |
dmsimard | yes, we recognize the amount of work you've put into it | 18:34 |
*** samuelBartel has quit IRC | 18:34 | |
colonwq | I have a flat spot on my head now and less hair. | 18:34 |
dmsimard | ask EmilienM and dgurtner how I've praised your perseverence and asked for help :p | 18:36 |
colonwq | you were all very patient with me learning this process. Now I need to make more. | 18:38 |
*** dgurtner has quit IRC | 18:38 | |
*** petems has quit IRC | 18:48 | |
*** petems has joined #puppet-openstack | 18:48 | |
*** paramite|afk has joined #puppet-openstack | 18:51 | |
openstackgerrit | Miles Gould proposed openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI https://review.openstack.org/276187 | 18:55 |
*** paramite|afk has quit IRC | 19:09 | |
xarses | colonwq: I have some nits otherwise this looks awsome | 19:10 |
xarses | thanks for sticking it out | 19:10 |
*** dgurtner has joined #puppet-openstack | 19:23 | |
*** dgurtner has quit IRC | 19:23 | |
*** dgurtner has joined #puppet-openstack | 19:23 | |
colonwq | xarses, I saw. I'll review the code and see what can be done. Thanks. | 19:25 |
*** rahulait has quit IRC | 19:25 | |
*** rahulait has joined #puppet-openstack | 19:26 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: tempest: run tox with -eall-plugin https://review.openstack.org/283786 | 19:28 |
xarses | The way I see it, in the end we should have the user call ::ceph::rgw::apache if they want apache, or ::ceph:rgw::civetweb if they want cvietweb that way, we can reduce need to `if` in as each module comes up | 19:28 |
xarses | colonwq: we can leave it as is for now and get this in first | 19:29 |
colonwq | xarses, not a problem. | 19:33 |
*** IvanBerezovskiy has left #puppet-openstack | 19:37 | |
*** ducttape_ has quit IRC | 19:38 | |
colonwq | xarses, about the controller vs node question: I am looking at is looking at using rgw on the osp controller. | 19:47 |
EmilienM | ok we should have scenario003 in place in a few time | 19:48 |
EmilienM | I'm working on make it working now. | 19:49 |
EmilienM | hopefully will get stable by tomorrow. | 19:49 |
*** ducttape_ has joined #puppet-openstack | 19:49 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: rename scenario003 to scenario-aio https://review.openstack.org/284380 | 19:52 |
dmsimard | EmilienM: have you seen this error before ? libvirtError: internal error: process exited while connecting to monitor: Failed to open module: /usr/lib/x86_64-linux-gnu/qemu/block-dmg.so: failed to map segment from shared object: Permission denied | 19:52 |
dmsimard | http://logs.openstack.org/27/266027/25/gate/gate-puppet-openstack-integration-scenario001-tempest-dsvm-trusty/fe3bb49/logs/nova/nova-compute.txt.gz#_2016-02-24_19_09_38_550 | 19:52 |
EmilienM | no | 19:53 |
bkero | Smells like selinux | 19:53 |
EmilienM | but AFIK jamespage updated Ubuntu repo today. We should not take ceph packaging from there, but maybe a dep? | 19:53 |
EmilienM | bkero: on Ubuntu? Hum. :-) | 19:53 |
dmsimard | bkero: selinux on trusty? :P | 19:53 |
bkero | Yeah, maybe not | 19:53 |
bkero | apparmor? | 19:53 |
EmilienM | we don't enable it. | 19:53 |
*** bklei has quit IRC | 19:54 | |
bkero | Huh | 19:54 |
dmsimard | looks like it could be related to nested virtualization | 19:54 |
dmsimard | might be another specific thing from one of the -infra providers | 19:54 |
dmsimard | trying to figure out | 19:54 |
bkero | Do you know what hypervisor/host os you're on? | 19:55 |
bkero | Hm, can find evidence on the internet associating the problem with both apparmor and selinux. Although they're disabled inside the VM. It'd be really weird for the host OS to control that stuff via paravirt | 19:57 |
EmilienM | bkero: it depends on providers | 19:57 |
EmilienM | bkero: OS infra relies on OVH, bluebox, internap, rackspace, and infra cloud (soon) | 19:57 |
dmsimard | I'll try a recheck for the time being | 19:58 |
bkero | Yeah. I'm sure they all use different hypervisors and host OSes | 19:58 |
dmsimard | EmilienM: mtreinish doesn't have an opinion on tox vs run_tempest :P | 20:00 |
dmsimard | they use tox in the gate but run_tempest is there for folks to use | 20:01 |
dmsimard | I guess I'll patch packstack to use tox instead just so we're all using the same thing | 20:01 |
dmsimard | to be like the other cool kids | 20:02 |
mtreinish | dmsimard: fwiw, I dislike run_tempest personally and have tried to remove it many times | 20:03 |
mtreinish | but everytime I do people complain | 20:03 |
mtreinish | so I've just given up on that fight | 20:03 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: create scenario003 to relax other scenarios https://review.openstack.org/284388 | 20:08 |
EmilienM | ok I hope it will help ^ | 20:08 |
dmsimard | EmilienM: okay let me know when it merges ? I'll implement it in weirdo | 20:21 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: create scenario003 to relax other scenarios https://review.openstack.org/284388 | 20:21 |
EmilienM | dmsimard: sure | 20:21 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: install_modules.sh: run gem install with --verbose https://review.openstack.org/283696 | 20:22 |
openstackgerrit | Nikolay Starodubtsev proposed openstack/puppet-murano: Switch murano to use keystone API v3 by default https://review.openstack.org/279444 | 20:23 |
*** brandon has joined #puppet-openstack | 20:36 | |
EmilienM | dmsimard: watching CI jobs. I don't want to say victory but I noticed jobs are much faster | 20:47 |
dmsimard | with what, the worker patch and the split ? | 20:48 |
EmilienM | the split | 20:48 |
EmilienM | worker was just to gain a little :-) | 20:48 |
*** dgurtner has quit IRC | 20:51 | |
EmilienM | dmsimard: scenario001: 35 min now :-) | 21:01 |
EmilienM | versus 55 | 21:01 |
EmilienM | and all 3 scenarios working on first shop | 21:01 |
EmilienM | shot* | 21:01 |
bkero | Wow, that's a good improvement | 21:02 |
EmilienM | let's see how it works | 21:04 |
*** ducttape_ has quit IRC | 21:04 | |
*** ducttape_ has joined #puppet-openstack | 21:04 | |
_ody | mfisch: I had just started working through what it takes to do beaker tests on our OpenStack cloud for Puppet OpenStack stuff. You can probably ignore me on "Deprecate python.pp and fix client tags" if you just find someone to merge. Else. I can take another look in a minute. | 21:05 |
_ody | I think my setup is working now so I can run the tests again to see if they were erroneous. | 21:05 |
EmilienM | _ody: what do you mean by running beaker stuff on puppetlabs cloud? | 21:06 |
EmilienM | _ody: you should probably poke nibalizer about that | 21:07 |
*** baker has quit IRC | 21:12 | |
_ody | EmilienM: I was working making sure I could run beaker tests against our internal cloud yesterday. That is when I had the issue running test that I was pinging mfish about. | 21:14 |
EmilienM | _ody: oh ok ! | 21:14 |
EmilienM | _ody: if it's an openstack cloud, it should work. I run my beaker against trystack | 21:14 |
_ody | Yeah. I think I have it working now. | 21:15 |
mfisch | _ody: ok | 21:18 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: create scenario003 to relax other scenarios https://review.openstack.org/284388 | 21:18 |
*** baker has joined #puppet-openstack | 21:19 | |
*** dgurtner has joined #puppet-openstack | 21:19 | |
*** dgurtner has quit IRC | 21:19 | |
*** dgurtner has joined #puppet-openstack | 21:19 | |
*** brandon has quit IRC | 21:22 | |
*** johnmilton has quit IRC | 21:25 | |
_ody | EmilienM: I generalized your blog on running beaker on OpenStack and already have other people using it to test other modules. | 21:25 |
EmilienM | coool | 21:27 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: create scenario003 to relax other scenarios https://review.openstack.org/284388 | 21:41 |
*** brandon has joined #puppet-openstack | 21:41 | |
*** _ducttape_ has joined #puppet-openstack | 21:50 | |
*** rahulait has quit IRC | 21:53 | |
*** ducttape_ has quit IRC | 21:54 | |
*** baker has quit IRC | 22:02 | |
*** baker has joined #puppet-openstack | 22:03 | |
*** brandon has quit IRC | 22:06 | |
*** jayg is now known as jayg|g0n3 | 22:09 | |
openstackgerrit | Cody Herriges proposed openstack/puppet-horizon: Making root_url configurable https://review.openstack.org/283827 | 22:13 |
*** _ducttape_ has quit IRC | 22:14 | |
*** ducttape_ has joined #puppet-openstack | 22:15 | |
*** _ducttape_ has joined #puppet-openstack | 22:25 | |
*** dprince has quit IRC | 22:26 | |
*** mdorman has quit IRC | 22:28 | |
*** ducttape_ has quit IRC | 22:28 | |
*** mdorman has joined #puppet-openstack | 22:28 | |
*** haplo37 has quit IRC | 22:29 | |
*** delattec has joined #puppet-openstack | 22:34 | |
*** cdelatte has quit IRC | 22:37 | |
mfisch | EmilienM: whats going on with the beaker test? | 22:52 |
mfisch | sorry ive been away from this room most of the day | 22:52 |
*** brandon has joined #puppet-openstack | 22:57 | |
*** baker has quit IRC | 23:09 | |
*** dhollinger has quit IRC | 23:10 | |
*** _ducttape_ has quit IRC | 23:15 | |
openstackgerrit | Cody Herriges proposed openstack/puppet-horizon: Making root_url configurable https://review.openstack.org/283827 | 23:15 |
*** saneax_AFK is now known as saneax | 23:21 | |
*** dgurtner has quit IRC | 23:24 | |
mwhahaha | did the neutron package rename get pushed for ubuntu? http://logs.openstack.org/88/284388/4/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-trusty/a56c730/console.html#_2016-02-24_22_51_47_429 | 23:53 |
mwhahaha | I rechecked https://review.openstack.org/#/c/283714/ to see if it passes now | 23:54 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!