Tuesday, 2016-04-19

*** karimb has quit IRC00:05
*** xarses has quit IRC00:14
EmilienMmwhahaha: priorites are set by queue, like gate > experimental, et00:30
EmilienMrecheck just trigger jenkins jobs, except if thoses are already running00:31
EmilienMthe queue is going down atm, but it's still ver high00:31
mwhahahaYea I didn't know if they gave rechecks a lower priority. But I guess it makes sense that it wouldn't00:33
EmilienMmwhahaha: https://github.com/openstack-infra/project-config/blob/master/zuul/layout.yaml#L4000:35
EmilienMthis is how Zuul decides who's is important00:35
EmilienMthat's why post queue is soooooooo high00:35
EmilienMand gate is so low00:35
openstackgerritEmilien Macchi proposed openstack/puppet-trove: Add other-requirements.txt file  https://review.openstack.org/30752800:37
openstackgerritEmilien Macchi proposed openstack/puppet-trove: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30721500:38
EmilienMfor core reviewers, please review https://review.openstack.org/#/q/topic:gemfile/liberty+status:open00:38
*** xarses has joined #puppet-openstack00:43
openstackgerritMerged openstack/puppet-murano: Totally drop Qpid support  https://review.openstack.org/30708401:03
openstackgerritMerged openstack/puppet-designate: Totally drop Qpid support  https://review.openstack.org/30709101:07
openstackgerritMerged openstack/puppet-heat: Totally drop Qpid support  https://review.openstack.org/30710301:10
*** ayoung has joined #puppet-openstack01:12
openstackgerritMerged openstack/puppet-manila: Totally drop Qpid support  https://review.openstack.org/30710001:12
openstackgerritMerged openstack/puppet-neutron: Totally drop Qpid support  https://review.openstack.org/30708801:27
openstackgerritMerged openstack/puppet-trove: Totally drop Qpid support  https://review.openstack.org/30710701:35
*** ducttape_ has joined #puppet-openstack01:37
openstackgerritMerged openstack/puppet-zaqar: Totally drop Qpid support  https://review.openstack.org/30711201:44
openstackgerritMerged openstack/puppet-gnocchi: Totally drop Qpid support  https://review.openstack.org/30708501:44
openstackgerritMerged openstack/puppet-sahara: Totally drop Qpid support  https://review.openstack.org/30709701:44
openstackgerritMerged openstack/puppet-keystone: Totally drop Qpid support  https://review.openstack.org/30709801:44
openstackgerritMerged openstack/puppet-cinder: Totally drop Qpid support  https://review.openstack.org/30708101:44
openstackgerritMerged openstack/puppet-nova: Totally drop Qpid support  https://review.openstack.org/30708001:53
*** darkxploit has quit IRC02:02
*** haplo37 has joined #puppet-openstack02:05
*** haplo37 has quit IRC02:15
*** ducttape_ has quit IRC02:19
*** haplo37 has joined #puppet-openstack02:20
openstackgerritMatt Fischer proposed openstack/puppet-nova: Remove deprecated qpid config/classes  https://review.openstack.org/30387703:09
*** haplo37_ has joined #puppet-openstack03:23
*** haplo37_ has quit IRC03:23
*** ntpttr has quit IRC03:34
openstackgerritMark Kirkwood proposed openstack/puppet-swift: Allow a replication network to be specified  https://review.openstack.org/30248303:37
*** richm has quit IRC03:56
*** ayoung has quit IRC04:11
*** saneax is now known as saneax_AFK04:12
*** ducttape_ has joined #puppet-openstack04:20
*** ducttape_ has quit IRC04:25
*** haplo37 has quit IRC04:31
*** haplo37 has joined #puppet-openstack04:32
*** rcernin has joined #puppet-openstack05:10
*** saneax_AFK is now known as saneax05:23
*** chandankumar has joined #puppet-openstack05:30
*** ntpttr has joined #puppet-openstack05:43
*** haplo37 has quit IRC05:45
*** haplo37 has joined #puppet-openstack05:46
*** leanderthal|afk is now known as leanderthal06:17
*** shardy has joined #puppet-openstack06:20
*** haplo37 has quit IRC06:20
*** ducttape_ has joined #puppet-openstack06:21
*** magicboiz has joined #puppet-openstack06:22
*** magicboiz has quit IRC06:23
*** ducttape_ has quit IRC06:25
*** jaosorior has joined #puppet-openstack06:37
*** maestro1 has joined #puppet-openstack06:47
*** fawadkhaliq has joined #puppet-openstack06:49
*** hjensas has quit IRC06:50
*** dalees` has quit IRC06:58
*** dalees` has joined #puppet-openstack06:59
*** dgurtner has joined #puppet-openstack07:00
*** dgurtner has quit IRC07:00
*** dgurtner has joined #puppet-openstack07:00
*** chem has quit IRC07:01
*** amoralej|off is now known as amoralej07:01
*** jmikolajczak has joined #puppet-openstack07:05
*** fawadkhaliq has quit IRC07:12
*** tesseract has joined #puppet-openstack07:24
*** paramite has joined #puppet-openstack07:24
*** tesseract is now known as Guest2294507:24
*** jaosorior has quit IRC07:27
*** social has quit IRC07:28
*** foxhunt_away is now known as foxhunt7207:34
*** jpena|off is now known as jpena07:37
*** paramite is now known as paramite|afk07:43
*** hjensas has joined #puppet-openstack07:45
*** jaosorior has joined #puppet-openstack07:55
*** paramite|afk is now known as paramite07:58
*** mkarpin1 has quit IRC07:59
*** cwolferh has quit IRC08:07
*** mkarpin has joined #puppet-openstack08:10
*** jistr has joined #puppet-openstack08:16
*** ducttape_ has joined #puppet-openstack08:22
*** maestro1 has quit IRC08:22
*** maestro1 has joined #puppet-openstack08:23
*** ducttape_ has quit IRC08:26
*** skolekonov_away is now known as skolekonov08:30
*** maestro2 has joined #puppet-openstack08:34
*** maestro1 has quit IRC08:37
openstackgerritMykyta Karpin proposed openstack/puppet-ironic: Switch ironic to use puppet-oslo module  https://review.openstack.org/30529908:38
*** mgould has joined #puppet-openstack08:47
*** derekh has joined #puppet-openstack08:49
*** maestro1 has joined #puppet-openstack08:54
*** iberezovskiy has quit IRC08:56
*** maestro2 has quit IRC08:56
*** iberezovskiy has joined #puppet-openstack08:58
*** jaosorior has quit IRC09:07
*** karimb has joined #puppet-openstack09:11
*** mattymo` has joined #puppet-openstack09:22
*** jgrassler has joined #puppet-openstack09:23
*** hogepodge has joined #puppet-openstack09:23
*** imcsk8 is now known as imcsk8|zZz09:25
*** paramite is now known as paramite|afk09:27
*** maestro1 has quit IRC09:38
*** dtantsur|afk is now known as dtantsur09:40
*** paramite|afk is now known as paramite09:44
*** johnmilton has joined #puppet-openstack09:47
*** paramite is now known as paramite|afk09:51
*** johnmilton has quit IRC09:51
*** _degorenko|afk is now known as degorenko10:08
mflobodegorenko, hi o/ so, about this https://review.openstack.org/#/c/276103/ you would propose to split the commit in 2 separate commits, right?10:16
*** chandankumar has quit IRC10:21
*** ducttape_ has joined #puppet-openstack10:23
*** mgould has quit IRC10:24
degorenkomflobo, hey10:24
degorenkomflobo, yeah10:25
degorenkoso one patch is adding new services and another one fixing wrong parameters and so on10:25
*** ducttape_ has quit IRC10:27
*** chandankumar has joined #puppet-openstack10:28
*** mgould has joined #puppet-openstack10:29
*** markvoelker has joined #puppet-openstack10:37
*** markvoelker has quit IRC10:42
*** jaosorior has joined #puppet-openstack10:42
*** johnmilton has joined #puppet-openstack10:59
*** shardy is now known as shardy_lunch11:03
*** paramite|afk is now known as paramite11:04
*** paramite is now known as paramite|afk11:05
*** paramite|afk is now known as paramite11:07
openstackgerritMerged openstack/puppet-cinder: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30719811:15
openstackgerritMerged openstack/puppet-vswitch: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30721611:21
openstackgerritMerged openstack/puppet-murano: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30720611:26
openstackgerritMerged openstack/puppet-manila: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30720511:27
*** shardy_lunch is now known as shardy11:27
openstackgerritMerged openstack/puppet-swift: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30721311:29
openstackgerritMerged openstack/puppet-sahara: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30721211:29
openstackgerritMerged openstack/puppet-openstacklib: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30721111:30
openstackgerritMerged openstack/puppet-openstack_extras: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30721011:33
openstackgerritMerged openstack/puppet-designate: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30719911:37
*** markvoelker has joined #puppet-openstack11:38
*** markvoelker has quit IRC11:42
*** saneax is now known as saneax_AFK11:45
*** paramite is now known as paramite|afk11:48
*** karimb has quit IRC11:49
*** karimb has joined #puppet-openstack11:49
openstackgerritMerged openstack/puppet-nova: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30720911:50
openstackgerritMerged openstack/puppet-neutron: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30720811:50
*** paramite|afk is now known as paramite11:50
openstackgerritMerged openstack/puppet-neutron: Add ml2 package requirement to neutron::plugins::ml2::mech_driver  https://review.openstack.org/30601711:51
openstackgerritMerged openstack/puppet-neutron: Support the LibreSwan driver in Red Hat platforms  https://review.openstack.org/30631411:51
openstackgerritMerged openstack/puppet-ironic: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30720411:51
openstackgerritMerged openstack/puppet-horizon: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30720311:51
openstackgerritMerged openstack/puppet-heat: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30720211:51
*** karimb_ has joined #puppet-openstack11:52
*** mgould has quit IRC11:53
*** Guest22945 is now known as tesseract11:54
*** karimb has quit IRC11:54
*** tesseract is now known as Guest7339711:54
*** karimb_ has quit IRC11:54
openstackgerritMerged openstack/puppet-glance: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30720011:55
*** dprince has joined #puppet-openstack11:55
openstackgerritMerged openstack/puppet-aodh: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30714611:55
openstackgerritMerged openstack/puppet-horizon: Added ensure_resource for python-memcache package.  https://review.openstack.org/30525011:55
*** ansmith has joined #puppet-openstack11:56
*** paramite is now known as paramite|afk12:10
openstackgerritMerged openstack/puppet-tempest: deprecate verbose parameter  https://review.openstack.org/30744612:12
*** trown|outtypewww is now known as trown12:12
*** paramite|afk is now known as paramite12:13
openstackgerritMatthew J Black proposed openstack/puppet-neutron: WIP Arista ml2 plugin  https://review.openstack.org/30741912:13
*** ducttape_ has joined #puppet-openstack12:14
*** amoralej is now known as amoralej|lunch12:16
*** mtreinish has quit IRC12:18
*** markvoelker has joined #puppet-openstack12:19
*** ntaylor1781 has quit IRC12:19
*** mtreinish has joined #puppet-openstack12:21
*** chem has joined #puppet-openstack12:21
*** paramite has quit IRC12:25
*** gfidente has joined #puppet-openstack12:26
*** ducttape_ has quit IRC12:31
*** chem has quit IRC12:33
*** fwdit has joined #puppet-openstack12:33
iurygregorymorning12:35
mjblackmorning12:37
*** fwdit has quit IRC12:42
*** jpena is now known as jpena|lunch12:43
EmilienMmwhahaha, degorenko: small typo on https://review.openstack.org/#/c/306677/7/releasenotes/notes/virtlock-virtlog-services-40ab5eae62817df7.yaml -- I'll push over and +2 +A12:48
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Add other virt services  https://review.openstack.org/30667712:49
degorenkoEmilienM, ok for me12:49
EmilienMdegorenko: ^ I'll let you +A12:49
degorenkoEmilienM, done12:50
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Add support for creating and managing Sahara templates  https://review.openstack.org/30779612:57
*** gfidente has quit IRC12:57
EmilienMso OVH mirrors are back again, infra is enabling OVH cloud again, so the long queue might be fixed by today12:58
*** EmilienM changes topic to "CI status: green http://tinyurl.com/puppet-ci - Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet"13:03
*** EmilienM sets mode: +o mfisch13:03
*** richm has joined #puppet-openstack13:08
*** ducttape_ has joined #puppet-openstack13:11
openstackgerritMatthew J Black proposed openstack/puppet-openstack-integration: Filter debug messages.  https://review.openstack.org/30479713:12
*** morazi has joined #puppet-openstack13:13
*** amoralej|lunch is now known as amoralej13:15
*** ducttape_ has quit IRC13:16
*** mgould has joined #puppet-openstack13:18
*** karimb has joined #puppet-openstack13:24
*** hjensas has quit IRC13:25
*** trown is now known as trown|brb13:25
*** chandankumar has quit IRC13:28
*** xarses has quit IRC13:31
openstackgerritEmilien Macchi proposed openstack/puppet-vswitch: add missing dev keyword to ip link command  https://review.openstack.org/30393613:32
*** xarses has joined #puppet-openstack13:32
*** trown|brb is now known as trown13:34
*** dfisher has joined #puppet-openstack13:34
*** openstackstatus has joined #puppet-openstack13:38
*** ChanServ sets mode: +v openstackstatus13:38
-openstackstatus- NOTICE: We have recovered one of our cloud providers, but there is a huge backlog of jobs to process. Please have patience until your jobs are processed13:41
*** twm2016 has joined #puppet-openstack13:46
iurygregoryEmilienM, https://github.com/openstack/puppet-monasca/blob/master/Gemfile this should be update? like in keystone etc ?13:48
EmilienMI don't patch puppet-monasca13:48
iurygregoryyeah i know, but the change should match all modules upstream or not? ^^13:49
*** saneax_AFK is now known as saneax13:50
*** ayoung has joined #puppet-openstack13:50
EmilienMsure thing13:50
iurygregoryok ^^13:51
*** jpena|lunch is now known as jpena13:51
EmilienMI did a mistake with https://review.openstack.org/#/c/307142/13:54
*** xarses has quit IRC14:00
EmilienMI'm submitting a fix https://review.openstack.org/30783614:00
*** dhollinger has joined #puppet-openstack14:02
*** haplo37 has joined #puppet-openstack14:03
mwhahahalooks like zuul is still backed up today14:05
*** hjensas has joined #puppet-openstack14:07
*** haplo37 has quit IRC14:08
*** jprs is now known as samuelson14:10
*** aimon has joined #puppet-openstack14:19
*** haplo37 has joined #puppet-openstack14:21
*** gfidente has joined #puppet-openstack14:26
*** gfidente has quit IRC14:26
*** gfidente has joined #puppet-openstack14:26
*** ducttape_ has joined #puppet-openstack14:29
openstackgerritDenis Egorenko proposed openstack/puppet-oslo: Totally drop Qpid support  https://review.openstack.org/30709414:31
*** ducttape_ has quit IRC14:31
*** ducttape_ has joined #puppet-openstack14:31
openstackgerritDenis Egorenko proposed openstack/puppet-barbican: Totally drop Qpid support  https://review.openstack.org/30711014:31
*** ducttape_ has quit IRC14:32
openstackgerritDenis Egorenko proposed openstack/puppet-octavia: Totally drop Qpid support  https://review.openstack.org/30718214:32
*** ducttape_ has joined #puppet-openstack14:33
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: IPv6 dual-stack support  https://review.openstack.org/28634414:36
*** xarses has joined #puppet-openstack14:39
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: venv: create venv & run pip in exec  https://review.openstack.org/30785314:45
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: venv: create venv & run pip in exec  https://review.openstack.org/30785414:45
*** gfidente has quit IRC14:45
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: venv: create venv & run pip in exec  https://review.openstack.org/30785414:47
EmilienMweekly meeting in 5 min ! https://wiki.openstack.org/wiki/Meetings/Puppet14:56
*** chandankumar has joined #puppet-openstack14:57
*** mdorman has joined #puppet-openstack14:58
*** dtantsur is now known as dtantsur|brb14:59
*** jaosorior has quit IRC15:00
*** mattymo` has quit IRC15:03
*** karimb has quit IRC15:04
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Correct typo resulting in a hardcoded paramter used incorrectly  https://review.openstack.org/29961515:06
*** trown is now known as trown|afk15:09
openstackgerritMichael Chapman proposed openstack/puppet-tripleo: Add steps to database profiles  https://review.openstack.org/30786315:10
*** francois has quit IRC15:14
*** mattymo` has joined #puppet-openstack15:15
*** francois1 has joined #puppet-openstack15:15
*** leanderthal is now known as leanderthal|afk15:17
*** saneax is now known as saneax_AFK15:19
*** ayoung has quit IRC15:20
mjblackapachectl on centos 7 appears to be a wrapper around systemctl15:29
openstackgerritMatt Jarvis proposed openstack/puppet-neutron: Don't declare the service when service_enable is false  https://review.openstack.org/30787915:30
*** karimb has joined #puppet-openstack15:31
*** mmosesohn_ has joined #puppet-openstack15:33
EmilienMmjblack: good news, so we can use it15:35
mjblackEmilienM: well it appears to work :D15:36
*** mattymo` has quit IRC15:37
*** ducttape_ has quit IRC15:37
mjblackI can pop the manifest change into test and see if it still works15:37
*** dtantsur|brb is now known as dtantsur15:37
*** foxhunt72 is now known as foxhunt_away15:38
*** dprince has quit IRC15:40
EmilienMmjblack: yeah, just as WIP15:41
EmilienMmjblack, mwhahaha: nex time you guys see the bug, please update the LP with link15:42
mwhahahaEmilienM & degorenko: can we merge https://review.openstack.org/#/c/305505/ as the depends-on change as now be merged15:42
degorenkomwhahaha, done15:42
mwhahahawow my english is terrible today15:42
mwhahahathe depends-on change is now merged15:42
EmilienMmwhahaha: well, don't look my english then.15:43
mwhahahayea but i'm not bilingual :D15:43
*** cwolferh has joined #puppet-openstack15:43
EmilienMmwhahaha: I'm frenglish, I mix both lol15:44
*** ducttape_ has joined #puppet-openstack15:44
openstackgerritMatt Jarvis proposed openstack/puppet-neutron: Don't declare the service when service_enable is false  https://review.openstack.org/30787915:48
EmilienMdfisher: quick update about tarballs. It's still in my list of things to check if it works, but CI has got many issues lately, it might be in place later this week.15:48
dfisherok.15:49
dfisherthanks for the update15:49
openstackgerritAlex Schultz proposed openstack/puppet-nova: Rewrite nova_security_group to use osclient  https://review.openstack.org/30678715:49
openstackgerritAlex Schultz proposed openstack/puppet-nova: Rewrite nova_security_rule to use osclient  https://review.openstack.org/30788815:49
*** dgurtner has quit IRC15:49
openstackgerritMerged openstack/puppet-openstack_spec_helper: Use master branch for local tests  https://review.openstack.org/30731415:51
*** dgurtner has joined #puppet-openstack15:51
openstackgerritNathan Taylor proposed openstack/puppet-cinder: Add purge_config option for the cinder config  https://review.openstack.org/30788915:54
*** ntaylor1781 has joined #puppet-openstack15:55
mnaserpuppet-ceph master seems to be failing me at deploying ceph at the moment... everything is mostly setup as listed on the USECASES.md file.. everything completes but it looks like the client.admin doesnt get setup16:00
mnasertherefore ceph -s results in "librados: client.admin authentication error (1) Operation not permitted"16:00
*** Guest73397 has quit IRC16:00
EmilienMmnaser: oops - any logs?16:00
mnaserEmilienM: logs look like it did it right, let me show puppet apply logs16:01
mwhahahathat's what we were getting in ci recently16:01
mwhahahait was an ordering thing that i thought was addressed16:01
mnasermwhahaha / EmilienM: http://paste.openstack.org/show/kxYmbm9HRDK7aLy4vG4z/16:01
mwhahahahttps://review.openstack.org/#/c/304378/16:01
mnaseri'm doing: service ceph stop; rm -rfv /var/lib/ceph/mon/* /etc/ceph/ceph.c* -- before rerunning16:02
mnaserand i can see the ceph_config happening before the monitor starts16:03
mwhahahathat doesn't have an error in it?16:03
mnaserwell, there's no errors, except i can't access anything like ceph -s16:03
mnaser2016-04-19 12:03:54.098263 7fba759e9700  0 librados: client.admin authentication error (1) Operation not permitted16:03
mnasercephx server client.admin: couldn't find entity name: client.admin16:04
mnaserin the ceph mon log16:04
mnaserwhich seems to imply that the client.admin key does not get created, but we have it defined and you see it get created there16:05
EmilienMmnaser: you running master, right?16:06
mnaseroui16:06
mnaseralso, storage SIG packages in here16:06
mnaser(unsure if that makes much of a difference)16:07
EmilienMmnaser: we're deploying our integration jobs with SIG packaging16:07
mnaserthe monitor itself seems to be setup.. logs show this: mon.ceph-1-ca-ymq-2@0(leader).data_health(1) update_stats avail 98% total 111 GB, used 1492 MB, avail 110 GB16:07
EmilienMgate-puppet-openstack-integration-3-scenario001-tempest-devstack-centos7 deploys ceph from SIG16:07
mnaserbut it seems like the user was not setup properly... unsure why :(16:07
EmilienMmhh16:08
EmilienMlet's see if we can reproduce it in our CI16:08
mnaseri can see that it's going through the right steps..16:08
EmilienMmnaser: can you give me path of ceph command please?16:09
EmilienMis it in /usr/bin? or /usr/sbin?16:09
mnaser /usr/bin/ceph16:10
EmilienMthx16:10
*** dtantsur is now known as dtantsur|brb16:10
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: run_tests.sh: run ceph -s at the end of CI job  https://review.openstack.org/30790016:11
EmilienMmnaser: ok this is a test to see how it works in our CI ^16:11
mnaseryeah i think that's good idea16:11
EmilienMmnaser: we're running a single node deployment though, but using SIG packaging16:11
*** ayoung has joined #puppet-openstack16:11
mnaserEmilienM: cool, i'll keep looking around in the meantime16:12
dmsimardmnaser: are you using the profile classes ?16:13
dmsimardfrom puppet-ceph16:13
openstackgerritMatt Fischer proposed openstack/puppet-trove: WIP deprecate templated guestagent config  https://review.openstack.org/30790316:13
mnaserno actually,  i just saw them16:13
mnaser(right as you mentioned it)16:13
dmsimardmnaser: let me give you a snippet of what I had for testing purposes for the profiles, see if it can help get you started16:14
dmsimardthe ::profile classes is essentially a built-in composition layer16:14
mnaserim going through them, they are basically what we do16:14
mnaseryeah, we use role/profile (as you can see from the logs pasted above)16:14
mnaserim going to have to submit a small change to allow for using SIG in them16:14
mnaserhmm, there's manage_repo .. do you guys see value in adding an option to use the storage SIG packages in the profiles or let users just manage_repo = false and use ::ceph::repo ?16:19
openstackgerritAlexey Deryugin proposed openstack/puppet-trove: Use oslo module for messaging (rabbit) configuration  https://review.openstack.org/30790716:19
*** dgurtner has quit IRC16:21
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Enable Sahara integration job on Ubuntu  https://review.openstack.org/30791116:24
openstackgerritMatthew J Black proposed openstack/puppet-keystone: WIP Add delay to restart with apache  https://review.openstack.org/30791216:26
EmilienMmnaser: this is how we do: https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/ceph.pp#L1316:26
EmilienMand https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/repos.pp#L5516:26
EmilienMbut yeah, you can add the feature to the base profile, it would make sense16:26
*** imcsk8|zZz is now known as imcsk816:27
*** karimb has quit IRC16:29
*** colonwq is now known as colonwq_lunch16:33
*** jistr has quit IRC16:39
*** iberezovskiy_ has joined #puppet-openstack16:40
*** iberezovskiy has quit IRC16:40
openstackgerritMatthew J Black proposed openstack/puppet-openstack-integration: Filter debug messages.  https://review.openstack.org/30479716:42
*** iberezovskiy_ has left #puppet-openstack16:43
*** iberezovskiy_ has joined #puppet-openstack16:43
mnaserwell uh, i just rewrote our stuff based on the profiles, and it works now... so...16:43
*** iberezovskiy_ has left #puppet-openstack16:44
*** fawadkhaliq has joined #puppet-openstack16:45
*** iberezovskiy_ has joined #puppet-openstack16:46
openstackgerritEmilien Macchi proposed openstack/puppet-ovn: Implement basic beakers tests  https://review.openstack.org/30792016:51
*** degorenko_ has joined #puppet-openstack16:53
*** degorenko_ has left #puppet-openstack16:54
mgouldEmilienM, is there any chance that a backport of https://review.openstack.org/#/c/302376 and https://review.openstack.org/#/c/302390 would be accepted into stable/mitaka or stable/liberty?16:54
EmilienMmgould: yes16:56
mgouldEmilienM, excellent, thanks16:56
iurygregoryEmilienM, sorry i was in a internal meeting =) i will not forget the chocolate o/17:00
EmilienMiurygregory: I hope not :P17:00
*** derekh has quit IRC17:01
*** dprince has joined #puppet-openstack17:02
*** rcernin has quit IRC17:04
*** dtantsur|brb is now known as dtantsur17:06
EmilienMdfisher: this is the last blocker to have all tarballs available: https://review.openstack.org/30793117:07
dfishercool!17:08
EmilienMdfisher: once it's in place (should be later this week, with all CI issues), you'll be able to download our modules using this url: https://tarballs.openstack.org/puppet-<modulename>17:09
dfisheroutstanding17:09
EmilienMdfisher: for now, we only support master builds17:09
* dfisher nods17:09
EmilienMdfisher: but once all of this is in place, I'll work on building modules for stable/branches17:09
dfisherok17:10
EmilienMdfisher: the magic is here: https://github.com/openstack-infra/project-config/blob/master/jenkins/scripts/run-tarball.sh#L19-L2617:10
EmilienMbut we'll figure that out later17:11
*** foxhunt_away is now known as foxhunt7217:12
*** jpena is now known as jpena|off17:12
mjblackEmilienM: with puppet-ovn, I was going to add the debian package names into params, you ok with me fixing the formatting of the classes too in the same review?17:12
EmilienMmjblack: please submit it as a separated patch and rebase my beaker patch on top of yours17:13
EmilienMmjblack: ovn is packaged on ubuntu?17:13
mjblackyup17:13
EmilienMexcellent news whouhhhhoooo17:14
EmilienMsorry17:14
mjblackI'll add the params for it17:14
iurygregorywut? O.o17:14
iurygregoryCI is not slow? =P this is excellent news XD17:14
openstackgerritAlex Schultz proposed openstack/puppet-nova: Add other virt services  https://review.openstack.org/30667717:17
mwhahahaEmilienM: will you +A -^ again? it was failing in the gate so rather than continue to wait for it to error i just rebased it17:18
mjblackEmilienM: so you still want me to rebase your change onto mine?17:18
EmilienMmwhahaha: done17:18
mwhahahathanks17:18
EmilienMmjblack: yes please17:18
EmilienMmjblack: submit your patch to openstack/puppet-ovn and then rebase https://review.openstack.org/307920 on top of your patch17:19
EmilienMGerrit has a nice button to rebase very quickly17:19
* EmilienM out for lunch17:19
mjblackEmilienM: yeah I'm just making sure we have specs for debian ;)17:19
*** mmosesohn_ is now known as mattymo`17:25
openstackgerritAlexey Deryugin proposed openstack/puppet-trove: Use trove module for messaging (rabbit) configuration  https://review.openstack.org/30790717:26
openstackgerritAlexey Deryugin proposed openstack/puppet-trove: Use oslo module for messaging (rabbit) configuration  https://review.openstack.org/30790717:28
*** aimon has quit IRC17:29
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-monasca: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30794017:32
*** colonwq_lunch is now known as colonwq17:33
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-manila: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30794217:35
openstackgerritMatthew J Black proposed openstack/puppet-ovn: Added params for debian packages  https://review.openstack.org/30794317:38
openstackgerritMatthew J Black proposed openstack/puppet-ovn: Implement basic beakers tests  https://review.openstack.org/30792017:39
openstackgerritMiles Gould proposed openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI  https://review.openstack.org/30794717:40
openstackgerritMatthew J Black proposed openstack/puppet-ovn: Added params for debian packages  https://review.openstack.org/30794317:41
dmsimardmnaser: cool :)17:42
openstackgerritMiles Gould proposed openstack/puppet-ironic: Copy iPXE ROMs into /tftpboot  https://review.openstack.org/30794917:44
openstackgerritMatthew J Black proposed openstack/puppet-ovn: Added params for debian packages  https://review.openstack.org/30794317:47
*** gchamoul- has quit IRC17:57
*** gchamoul has joined #puppet-openstack17:58
*** mattymo` has quit IRC18:01
*** foxhunt72 is now known as foxhunt_away18:02
*** mgould has quit IRC18:02
*** mtreinish has quit IRC18:03
*** dgurtner has joined #puppet-openstack18:07
*** dgurtner has quit IRC18:07
*** dgurtner has joined #puppet-openstack18:07
*** mtreinish has joined #puppet-openstack18:08
EmilienMmjblack: awesome, thanks man18:11
mjblackEmilienM: trying to convert the spec to on_supported_os format18:11
EmilienMmnaser: let us know any feedback :)18:11
mnaserEmilienM: will do... even if im pretty much setup now18:12
dmsimardEmilienM: any progress on newton ?18:12
EmilienMmjblack: https://github.com/openstack/puppet-cinder/blob/master/spec/classes/cinder_wsgi_apache_spec.rb18:12
EmilienMdmsimard: yes, I'm waiting for https://review.openstack.org/30748218:13
dmsimardEmilienM: oh, is that what was causing the jobs to timeout then ?18:14
EmilienMyes18:15
dmsimardbecause even with 90 minutes they don't finish on ci.centos18:15
EmilienMdmsimard: using neutron plugin was adding > 250 tests18:15
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Enable Sahara testing on Ubuntu  https://review.openstack.org/30796818:16
*** amoralej is now known as amoralej|off18:17
EmilienMdegorenko: I was about doing it too ^ :P18:17
degorenkoEmilienM, i saw your cherry pick for integration :D i forgot to cherry pick it :(18:18
*** dtantsur is now known as dtantsur|afk18:18
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: test telemetry on Ubuntu again  https://review.openstack.org/30797218:21
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: enable zaqar on Ubuntu  https://review.openstack.org/30797618:25
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: test ironic on Ubuntu again  https://review.openstack.org/30797718:26
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: test ironic on Ubuntu again  https://review.openstack.org/30797718:27
mjblackEmilienM: I'm getting errors with spec with the platform being set incorrectly18:31
mjblackovn::northd on fedora-22-x86_6418:31
openstackgerritMatthew J Black proposed openstack/puppet-ovn: Added params for debian packages  https://review.openstack.org/30794318:32
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/30588618:32
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Revert "run_tests: stop mistral testing"  https://review.openstack.org/30556118:32
*** chandankumar has quit IRC18:33
*** chandankumar has joined #puppet-openstack18:33
EmilienMmjblack: what error? logs?18:34
mjblackwith the spec18:34
mjblackwhen I run locally the platforms are "fedora-22-x86_64"18:34
mjblackfor example18:34
*** 7F1AAV9VC has joined #puppet-openstack18:34
*** degorenko is now known as _degorenko|afk18:36
*** chandankumar has quit IRC18:37
EmilienMmjblack: any pastebin?18:38
mwhahahaEmilienM: http://logs.openstack.org/65/307465/2/gate/gate-puppet-openstack-integration-3-scenario003-tempest-ubuntu-trusty/abd75bf/console.html#_2016-04-19_17_50_06_28018:38
EmilienMboom18:38
EmilienMkeystone issue here it is18:38
mwhahahayea now we've got a log to go look at :D18:39
mjblackEmilienM: http://paste.openstack.org/show/494703/18:39
EmilienMI'm looking at logs now18:39
mwhahahait's also on trusty18:39
EmilienMmwhahaha: is it only on trusty?18:39
mwhahahawhich is what we ran into where mod_wsgi was having issues, i think we ended up patching it on trusty but that wouldn't be a solution for us18:39
mwhahahai think so, not completely sure we'd have to keep track of the failures and see if it pops up on centos18:40
EmilienMmwhahaha: http://logs.openstack.org/65/307465/2/gate/gate-puppet-openstack-integration-3-scenario003-tempest-ubuntu-trusty/abd75bf/logs/keystone/keystone.txt.gz#_2016-04-19_17_49_00_32118:40
EmilienMsomething (packaging probably) is starting eventlet18:40
mwhahahaah so we don't run into that on fuel because we disable autostarting18:41
EmilienMhow?18:41
EmilienMwith puppet?18:41
mwhahahahttps://github.com/openstack/fuel-library/blob/master/deployment/puppet/tweaks/manifests/ubuntu_service_override.pp18:41
EmilienMouch18:41
mwhahahayea :(18:41
mwhahahai hate the autostarting18:42
* mwhahaha throws things at zigo18:42
mjblacklol me too18:42
mwhahahawhich is also why we needed the virtlock/virtlog managed in puppet-nova because for us it never started18:42
*** stewie925_ has joined #puppet-openstack18:42
*** chandankumar has joined #puppet-openstack18:43
EmilienMI'm reporting the bug to UCA18:43
EmilienMare you sure it's on centos7 too?18:43
*** trown|afk is now known as trown18:43
mwhahahano18:43
EmilienMwe have a keystone-ninja (aka ayoung) around, let's poke him, he deserves it.18:44
ayoungI am no Ninja18:44
ayoungI am SAMURAI18:44
EmilienMman we need you18:44
ayoungEmilienM, Keystone meeting, but I'll try to multitask.18:45
EmilienMayoung: can you confirm keystone package in RDO does not start eventlet process automatically?18:45
EmilienMno you can come back later18:45
ayoungit better not18:45
EmilienMayoung: well, ubuntu packaging does18:45
ayoungEmilienM, I lie about many things, though18:45
EmilienMayoung: come back after meeting.18:45
ayoungEmilienM, that is a major difference between Debian style packaging nad RPM18:45
ayoungRPM does not even configure18:45
ayoungDeb trys to give you a running app18:45
ayoungits philisophical18:46
mwhahahait's rage enducing18:46
EmilienMright, that's why people move to containers18:46
mwhahahainducing rather18:46
EmilienManyway. We need to workaround that in our CI18:47
mjblackEmilienM: isnt this RDO? https://github.com/openstack-packages/keystone/blob/rpm-master/openstack-keystone.spec18:47
EmilienMmjblack: it is18:47
mwhahahai guess for wsgi configuration should we explictly stop/disable the eventlet prior to configuring wsgi?18:47
mjblackthen it doesnt look like it enables the service18:47
EmilienMmwhahaha: can you give me link of the patch where it failed please?18:47
mwhahahahttps://review.openstack.org/#/c/307465/18:47
EmilienMI have a serious doubt that centos is also aaffected18:48
EmilienMI'll be very surprised18:48
mjblackEmilienM: what if we tried to include what fuel does?18:48
mwhahahai wouldn't18:48
7F1AAV9VChello all, just submitted a patch for a bug report I raised, would be good to get some feedback18:48
EmilienM7F1AAV9VC: sure, which patch?18:48
mwhahahawe should investigating explictly stopping the eventlet service prior to configuring httpd18:48
EmilienMmjblack: I disagree18:48
EmilienMmjblack: what does Fuel in this cas is (I'm sorry...) an hack18:49
EmilienMwe have to fix it in packaging18:49
mwhahahawell it won't be18:49
EmilienMsad panda18:49
mwhahahabut we should make sure the eventlet service is stopped if we configure with httpd18:49
EmilienMmwhahaha: see https://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L92618:49
EmilienMI did that in the past to make sure we first stop eventlet then start httpd18:50
mwhahahahmmm18:50
EmilienMmaybe we need more18:50
mjblackEmilienM: what if we add a conditional for ubuntu to make sure to stop eventlet before starting apache? :D18:50
*** 7F1AAV9VC is now known as dc_mattj18:50
EmilienMmjblack: look my link.18:50
mwhahahayou know i bet we need to wait18:50
EmilienMI already fixed it (or thought I did)18:50
dc_mattjEmilienM: https://review.openstack.org/#/c/307879/18:50
mwhahahaService['keystone'] ~> sleep -> Service[httpd']18:51
mwhahahahttp://logs.openstack.org/65/307465/2/gate/gate-puppet-openstack-integration-3-scenario003-tempest-ubuntu-trusty/abd75bf/logs/keystone/keystone.txt.gz#_2016-04-19_17_49_00_321 took 37 sec to stop18:51
dc_mattjprobably a bit of an edge case, given we use neutron-von-agent, so feel free to flame18:51
dc_mattjvpn ;)18:51
EmilienMdc_mattj: sure, we'll review it as soon other CI jobs are run. The CI is very slow lately, because of some infra stuffs.18:51
dc_mattjah ok, no worries18:51
EmilienMdc_mattj: but your patch looks good18:51
mjblackEmilienM,mwhahaha: are we sure the exec to restart apache is happening after the service call?18:51
EmilienMoh dear18:52
dc_mattjfirst patch I've submitted so wasn't sure if I had jumped through all the right hoops ;)18:52
mwhahahayea the service stopps at 17:49:00->17:39:3718:52
mjblackhttps://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L97218:52
mwhahahabut the start of http is ~17:49:44.42118:53
ayoungEventlet in Keystone is dead.  DEAD I TELL YOU.....WHY WON"T YOU DIE!  DIE EVENTLET DIE!18:53
mwhahahaservice_restart doesn't happen until after httpd is started http://logs.openstack.org/65/307465/2/gate/gate-puppet-openstack-integration-3-scenario003-tempest-ubuntu-trusty/abd75bf/console.html#_2016-04-19_18_04_05_59718:54
EmilienMok I reported https://bugs.launchpad.net/cloud-archive/+bug/157228318:55
openstackLaunchpad bug 1572283 in Ubuntu Cloud Archive "keystone eventlet should never be started" [Undecided,New]18:55
EmilienMUCA folks are on it18:55
EmilienMdc_mattj: excellent, welcome here!18:55
mjblackEmilienM,mwhahaha: I'm going to abandon my patch on keystone18:56
EmilienMmjblack: which patch? provide link please..18:56
mjblackEmilienM: it was from the meeting today18:56
mjblackit sounds like the change wouldnt have worked with ubuntu anyways18:56
EmilienMmjblack: I'm reviewing so much patches per day, a link would be very useful18:57
*** dgurtner has quit IRC18:57
mjblackEmilienM: https://review.openstack.org/#/c/307912/18:58
EmilienMaccording to mwhahaha , the error is before that Exec18:58
mwhahahayea it's on the start, ours was a restart delay18:59
EmilienMmwhahaha: do we see the service keystone stop somewhere in logs?19:00
*** fawadkhaliq has quit IRC19:00
mwhahahahmm maybe not, i saw the wsgi lines19:00
mwhahahai'll look more in a bit19:01
mwhahahai thought http://logs.openstack.org/65/307465/2/gate/gate-puppet-openstack-integration-3-scenario003-tempest-ubuntu-trusty/abd75bf/logs/keystone/keystone.txt.gz#_2016-04-19_17_49_37_351 was the service stopping19:01
mwhahahabecause http doesn't get started until after that time19:01
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: centos: deploy cinder in SSL  https://review.openstack.org/30800119:02
*** chandankumar has quit IRC19:02
EmilienMit's weird, keystone should have 2 workers, not 419:03
EmilienMreading logs, it sounds like both process concur19:03
openstackgerritMerged openstack/puppet-ironic: Totally drop Qpid support  https://review.openstack.org/30710619:04
EmilienMerr wrong verb19:04
EmilienMthey compete19:04
EmilienMhttp://logs.openstack.org/65/307465/2/gate/gate-puppet-openstack-integration-3-scenario003-tempest-ubuntu-trusty/abd75bf/console.html#_2016-04-19_17_49_37_45419:05
EmilienM2016-04-19 17:49:37.454 | Notice: /Stage[main]/Keystone::Service/Service[keystone]/ensure: ensure changed 'running' to 'stopped'19:05
EmilienMbut the error when httpd can't start is at 2016-04-19 17:50:06.28019:05
ayoungEmilienM, you good?  Still need me?19:05
openstackgerritTrevor McCasland proposed openstack/puppet-nova: WIP: Add nova flavor provider  https://review.openstack.org/30546319:06
EmilienMso the only thing I see now is, keystone/eventlet takes more than 29 seconds to stop19:06
EmilienMayoung: can you quadruple check that keystone package in RDO does not have eventlet service and does not even try to start if it has it?19:06
EmilienMayoung: we have a bug in ubuntu. Just need to make sure we don't have it in RDO19:07
EmilienMmwhahaha: I'm spawing a VM and try to reproduce it now.19:07
mwhahahak19:07
mwhahahayea i'm pretty sure rdo does not suffer from the same problem19:07
mjblackwell19:08
mjblacklets find out19:08
mjblackvagrant is awesome for these things19:08
ayoungEmilienM, look at the commit message for https://github.com/openstack-packages/keystone/blob/rpm-master/openstack-keystone.spec19:09
EmilienMhttps://github.com/openstack-packages/keystone/commit/9dbd2556545afff77ec1d33449dd159eeb34be5919:09
ayounghttps://github.com/openstack-packages/keystone/commit/9dbd2556545afff77ec1d33449dd159eeb34be5919:09
EmilienM Remove eventlet support19:10
ayoungremoves the eventlet dependencies19:10
EmilienMok excellent. But I guess the service was not started automatically before, right?19:10
ayoungremoves systemd setup (cuz apache)19:10
ayoungum...should not have been.  It might have been stopped and restarted in an upgrade, though19:10
EmilienMayoung: ok, so no.19:10
EmilienMayoung: thx19:10
ayoungEmilienM, https://github.com/openstack-packages/keystone/blob/rpm-liberty/openstack-keystone.spec#L148  this was the liberty install section19:11
ayoungand then post was19:11
ayounghttps://github.com/openstack-packages/keystone/blob/rpm-liberty/openstack-keystone.spec#L19619:11
ayounglet me see what that does19:12
EmilienMayoung: can it be possible that the eventlet service would take more than 30s to stop?19:12
mjblackEmilienM: this is what systemd reports from centos 719:12
mjblack   Loaded: loaded (/usr/lib/systemd/system/openstack-keystone.service; disabled)19:12
mjblack   Active: inactive (dead)19:12
ayoungEmilienM, only in an error case,  I would think19:12
EmilienMmjblack: we want to test ubuntu I think, and see if package is starting the service19:12
mjblackit is19:12
EmilienMalso test a deploying and look how much time it takes to stop the service.19:13
ayoungEmilienM, https://fedoraproject.org/wiki/Packaging:Scriptlets  SEARCH FOR sYSTEM D AND YOU WILL SEE:19:14
ayoungOn upgrade, a package may only restart a service if it is running; it may not start it if it is off.19:14
mjblackayoung: what if we dont want to look for systemd?19:15
mjblackI still havent drank the koolaid19:15
ayoungI don't think LKeystone meets the criteria here https://fedoraproject.org/wiki/Packaging:DefaultServices?rd=Starting_services_by_default19:15
* EmilienM goes on https://en.wikipedia.org/wiki/Drinking_the_Kool-Aid19:15
ayoungmjblack, this is RDO specific19:15
mjblackI was making a systemd joke :)19:16
ayoungmjblack, so systemd is a done deal, for other distros do what you want19:16
EmilienMmjblack: sorry I screwed it.19:16
ayoungmjblack, I have no sense of humor.19:16
mjblackEmilienM: hmmm?19:16
EmilienMayoung: no you don't. :P19:16
mwhahahai have sense of bitterness, does that count?19:16
*** mattymo` has joined #puppet-openstack19:17
mjblackmwhahaha: you're just a normal person working in IT though19:17
EmilienMdamn I need to deploy ubuntu because of you19:18
* mwhahaha points and laughs19:18
mjblackEmilienM: what? I'm testing it19:18
mwhahahatouch the deb19:18
EmilienMyou make me deploying ubuntu, what a sad day.19:18
mjblackEmilienM: give me a minute, about to do it install19:18
EmilienMmjblack: oh thanks19:18
mjblack:D19:18
lluais there a reason why ceph::mds in puppet-ceph only add ceph.conf settings and not manage starting the service like ceph::mon or ceph::osd?19:18
*** dc_mattj has quit IRC19:19
openstackgerritMerged openstack/puppet-glance: Totally drop Qpid support  https://review.openstack.org/30710819:19
mjblackEmilienM: yup, ubuntu autostarts it... keystone start/running, process 753919:19
mwhahahaif you stop it what do the logs look like?19:19
EmilienMmjblack: can you run it with time?19:19
mjblackwould log output be fine?19:20
*** chandankumar has joined #puppet-openstack19:20
mjblackhttp://paste.openstack.org/show/494710/19:20
mwhahaha31sec to stop19:21
EmilienMlol19:21
EmilienMnice catch19:21
mjblackmwhahaha: no, its not 31 seconds to stop19:21
mjblackit stops within a second19:22
mwhahahaoh yea19:22
EmilienMi'm testing with some data19:22
EmilienMI'm actually running p-o-i on my vagrant19:22
mwhahahai wonder if one of the children doesn't stop19:23
mjblackEmilienM: openstack ci jobs run on VMs?19:23
mwhahahayea19:23
EmilienMit does19:23
openstackgerritMerged openstack/puppet-swift: Replace File with Concat in dependencies  https://review.openstack.org/30732119:23
mjblackI'm wondering if the VM slows it down, with all other things going on in the background with the scenarios19:23
mwhahahai'm sure it does19:23
mwhahahait's not a constant failure19:23
mjblackright19:23
EmilienMI suggest we patch puppet-keystone with a custom stop command for ubuntu19:24
EmilienMuntil packaging is fixed19:24
mjblackEmilienM: we could do that but another alternative that might do it19:24
EmilienMmjblack: which one?19:25
mjblackwe add in a sleep between Service[keystone] and Service[httpd]19:25
mjblackonly for ubuntu19:25
EmilienMa sleep is very racy19:25
EmilienMI was about kill -9 the process :P19:25
mjblackService[keystone] -> Exec[ubuntu_sleep_cause_it_dont_die_fast] -> Service[httpd] :D19:25
mjblackyeah but then you have to either find the pid or do a pkill19:26
EmilienMyeah, that could work.19:26
EmilienMmjblack: pkill ftw19:26
mjblacklol19:26
dfisherew.19:26
dfisherarbitrary csleeps19:26
dfishersleeps*19:26
mjblackEmilienM: well there is a good way of finding all pids...19:27
mwhahahasleep driven development, welcome to orchestration :D19:27
*** twm2016 has quit IRC19:27
dfishergod, no shit.19:27
EmilienMno, welcome to debian packaging.19:27
EmilienMI don't want packaging starting services19:27
dfisher^19:28
EmilienMI wonder how people doing ubuntu + containers do, they have to manually stop services? lol.19:29
mwhahahaeh they just restart it after they config it19:29
mjblackEmilienM: you know my pain now19:29
mjblackEmilienM: try uninstalling openstack-dashboard after puppet configured it19:30
mjblackmakes me wish for noscripts19:30
*** chandankumar has quit IRC19:30
mwhahahathe only problem here is that the service can be configured in two ways and we're using both (one because of packages and one because we want apache)19:30
mwhahahaopenstack-dashboard is it's own fiasco19:30
mwhahahacollectstatic can DIAF19:30
mjblackmwhahaha: yes but you cant ever apt-get purge it after its been installed and configured by puppet without zero'ing out a python file and deleting the vhosts19:31
mjblackwhoever did the packaging for it apparently tries to compress the assets on uninstall19:32
* mwhahaha points to zigo19:33
EmilienMmjblack: did you have data when deploying keystone 5 min ago?19:34
EmilienMI'm testing now, on ubuntu, with p-o-i scenario19:34
*** dgurtner has joined #puppet-openstack19:34
*** dgurtner has quit IRC19:34
*** dgurtner has joined #puppet-openstack19:34
mjblackEmilienM: btw19:34
mjblackyou need to do the sleep delay19:34
mjblackkeystone is an upstart script19:34
openstackgerritMerged openstack/puppet-neutron: Remove the spaces in the file server.pp  https://review.openstack.org/30688019:35
mjblackEmilienM: it has to be stopped or upstart will restart it19:36
EmilienMmwhahaha, mjblack  on#openstack-dev with UCA folks19:36
EmilienMthey don't want to fix it19:36
EmilienMnot for Mitaka19:36
mwhahahaworkaround time :(19:37
EmilienMif it was me I would disable testing on ubuntu19:38
EmilienMI'm so tired of their packages19:38
mjblackEmilienM: dont do that to me man...I know it sucks but I'm still on it :P19:38
mjblackEmilienM: maybe we should write an ensure_stopped type :D19:39
mwhahahaneed a wait until port is available provider or something19:39
mjblackEmilienM,mwhahaha: looks like I'm reviving my review with a different change19:40
EmilienMwait, I haven't said my last word19:40
*** mtreinish has quit IRC19:41
*** mtreinish has joined #puppet-openstack19:42
*** dc_mattj has joined #puppet-openstack19:43
EmilienMmjblack: he does not want to make the change19:43
EmilienMI give up19:43
EmilienMdo what you like in pupept-keystone to fix it, and if it's failing too much we'll disable voting for ubuntu jobs19:44
openstackgerritMatthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/30801219:45
mjblackEmilienM: ^19:45
mwhahahawonder if we need 35 seconds or if something like 5 is sufficient, anyway we'll see19:46
mwhahahamaybe tomorrow at the rate the CI is going19:46
openstackgerritMatthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/30801219:46
*** shardy has quit IRC19:46
EmilienMmjblack: commented19:46
mjblackmwhahaha: we cant control exactly the flow within puppet, 35 seconds is a good "even" number19:46
iurygregorymwhahaha, if you have time : https://review.openstack.org/#/c/307940/  https://review.openstack.org/#/c/307942/19:48
mjblackEmilienM: I can move it within that block but no need to add an else19:48
EmilienMmjblack: for centos, isn't?19:48
mjblackwe dont need it for centos19:48
*** chandankumar has joined #puppet-openstack19:48
*** mtreinish has quit IRC19:49
EmilienMmjblack: ok19:49
EmilienMworks for me.19:49
mjblackEmilienM: all the flow is saying is that the keystone service is refreshing the exec and we want the exec to happen before apache19:49
EmilienMexcellent19:50
openstackgerritMatthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/30801219:50
*** chandankumar has quit IRC19:50
mjblackEmilienM: do we really want to do 50? :D19:51
mjblack35 should be enough especially with the chaining the resources19:52
EmilienMwell, let's see if 35 worked19:52
EmilienMthe thing is, we want it backported to stable/mitaka and maybe stable/liberty19:52
EmilienMwe might want to keep our stable CIs *stable*19:52
EmilienMsomething that apparently packaging people don't get19:53
mwhahahait is a functional change19:53
mwhahahastill very annoying19:54
EmilienMmwhahaha: good news is, you can drop fuel workaround :P19:54
mwhahahanaw we use that all over the place19:54
EmilienMmwhahaha: puppet already disable the upstart service19:54
mwhahahamysql is another one that also does stupid stuff19:54
EmilienMmwhahaha: I don't recall, do you have plans to deploy with RDO packaging? or only debian systems?19:55
mwhahahawe've asked for help19:55
mwhahahawith RDO19:55
mwhahahaUCA is 9.0, RDO is next up19:55
EmilienMyou use UCA in 9.0 ?19:55
mwhahahabut it's alot more work because we need to be able to provision centos19:55
mwhahahawe can19:55
mwhahahathat's part of what that virtlock/virtlog thing was for19:56
mwhahahaUCA is broken because virtlogd doesn't get autostarted on install (because, you know we don't want that)19:56
*** dc_mattj has quit IRC19:56
mwhahahaonce that gets merged and i can get it in stable/mitaka then UCA will work with fuel again19:56
EmilienMnice19:56
mjblackEmilienM: next question is...when will CI be working again? :D19:57
mjblackor rather how long will we have to wait before it gets to this change :D19:57
mwhahahanext week when everyone is at the summit and no commits ;)19:57
EmilienMmjblack: don't yell at me! http://status.openstack.org/zuul/19:57
mjblackEmilienM: cant we just go on teh backend and like cancel everyone elses jobs?19:57
EmilienM650 jobs in check and 500 in post, you can go sleep and come back tomorrow, zuul will still be busy19:57
*** shardy has joined #puppet-openstack19:58
* mwhahaha goes back to integrating upstream haproxy19:58
iurygregorywatching zuul ( https://openstackreactions.wordpress.com/2013/03/07/watching-the-zuul-queue-testing-my-change/ )19:59
mjblackiurygregory: more like https://cdn.meme.am/images/8373718.jpg20:00
iurygregorymjblack, +120:01
mjblackor this one https://imgflip.com/readImage?iid=4815670620:01
EmilienMmjblack: commented (again)20:02
mjblackEmilienM: heh I have that change queued up, I just dont want to stress zuul out more20:03
openstackgerritMatthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/30801220:03
mjblackbah, waiting another 3 days wont make a difference20:04
*** mattymo` has quit IRC20:04
mjblackEmilienM: ah ok, yeah for your other comment...you're right we dont need to check if ubuntu20:05
EmilienMmjblack: just drop L909, 910 and 911 and we're good I think20:06
mjblackyeah I'm removing the conditional too...if its not running to begin with it wont refresh it20:06
openstackgerritMatthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/30801220:06
mjblackEmilienM: ^ amended the comment too20:07
openstackgerritMichael Glaser proposed openstack/puppet-keystone: Add Unit Tests for keystone_config type  https://review.openstack.org/30664420:10
*** fawadkhaliq has joined #puppet-openstack20:14
*** johnmilton has quit IRC20:16
*** dc_mattj has joined #puppet-openstack20:19
*** fawadkhaliq has quit IRC20:22
*** fawadkhaliq has joined #puppet-openstack20:22
*** delattec has quit IRC20:34
*** ducttape_ has quit IRC20:36
*** ducttape_ has joined #puppet-openstack20:36
*** dc_mattj has quit IRC20:37
*** ducttape_ has quit IRC20:38
*** ducttape_ has joined #puppet-openstack20:39
*** stewie925_ has quit IRC20:40
*** dprince has quit IRC20:40
*** dc_mattj has joined #puppet-openstack20:41
*** iurygregory has quit IRC20:42
*** dc_mattj has left #puppet-openstack20:46
*** dc_mattj has joined #puppet-openstack20:46
*** ansmith has quit IRC20:47
*** ducttape_ has quit IRC20:52
*** dgurtner has quit IRC20:52
*** ducttape_ has joined #puppet-openstack20:52
*** stewie925 has joined #puppet-openstack20:52
*** tristanC_ is now known as tristanC20:54
*** shardy has quit IRC20:55
*** stewie925 has quit IRC20:56
*** delattec has joined #puppet-openstack20:57
mjblackEmilienM: been seeing this in my puppet-openstack-integration failures, /tmp/hudson7590797017510595100.sh: line 2: ZUUL_REF=${{ZUUL_REF:-None}}: bad substitution20:57
mjblackfull log at http://logs.openstack.org/19/307419/5/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/56b198c/console.html20:58
EmilienMyeah run recheck20:58
EmilienMI fixed it this morning20:58
*** foxhunt_away is now known as foxhunt7220:59
*** haplo37 has quit IRC21:03
*** dc_mattj has quit IRC21:09
*** trown is now known as trown|outtypewww21:12
mjblackEmilienM: yeah but all the ubuntu runs are delayed significantly21:12
EmilienMyeah21:12
mjblackEmilienM: well it works on centos so that should be good enough for workflow :D21:15
EmilienMit's fun, in my terrible VM with low resources, keystone eventlet took 1 sec to stop. I'm wondering why it takes so much time in our CI21:15
mjblackEmilienM: is it the only thing going on in the vm?21:16
mjblackduring the scenarios, you have a lot more going on21:18
EmilienMno I run scenario-aio21:18
*** fawadkhaliq has quit IRC21:23
*** fawadkhaliq has joined #puppet-openstack21:24
*** ducttape_ has quit IRC21:24
*** ducttape_ has joined #puppet-openstack21:25
*** ducttape_ has quit IRC21:27
*** fawadkhaliq has quit IRC21:28
*** iberezovskiy_ has quit IRC21:34
*** iberezovskiy has joined #puppet-openstack21:35
*** mdorman has quit IRC21:47
zigomwhahaha: FYI, I've stopped working on the Horizon package, now it's Ivan Udovichenko that does.21:53
zigoThere's too many politics involved around using collectstatic, so I've given up.21:54
mwhahahagood to know21:54
zigoI am still convinced that we should *not* use collectstatic at all...21:54
mwhahahawe shouldn't21:54
zigoBut I'm the only one thinking this way.21:54
zigoAh ok, so we're 2 ... :P21:54
mwhahahadoing something like that is not how you deploy software21:55
mwhahaha:D21:55
zigoExactly.21:55
zigoThat's for debugging, not for production.21:55
zigoThough Horizon devs believe otherwise, and some of Mirantis customers as well...21:55
zigoI've been told that if I didn't do collecstatic in Debian, then MOS and Debian packages would diverge.21:55
zigoSo I gave up...21:55
mwhahahathat's a recurring developer issue21:56
*** johnmilton has joined #puppet-openstack21:58
mwhahahaOooO my script headers are pretty when you manually run beaker as well22:03
*** xarses has quit IRC22:12
*** fawadkhaliq has joined #puppet-openstack22:14
*** foxhunt72 is now known as foxhunt_away22:15
*** mtreinish has joined #puppet-openstack22:16
*** ayoung has quit IRC22:16
*** furlongm has joined #puppet-openstack22:21
*** dhollinger has quit IRC22:21
EmilienMmwhahaha: you like yourself? lol22:26
mwhahahasometimes :D22:26
*** morazi has quit IRC22:30
*** jmikolajczak has quit IRC22:33
_odyWhat's the thing called where we dedicate to releasing the modules on time along with OpenStack every cycle?22:34
EmilienM_ody: https://review.openstack.org/#/c/297382/22:36
EmilienM_ody: is that helping to answer your question?22:37
_odyEmilienM: Yeah.  That is what I was the review I was thinking of.22:37
* _ody working on presentation outline22:37
_odyYes I plugging the community during my talk too.22:38
EmilienMcool!22:38
*** xarses has joined #puppet-openstack22:38
EmilienMsomeone somewhere did not want us to work this week22:45
EmilienMstill 600 jobs in check queue, and 530 in post22:45
*** johnmilton has quit IRC22:54
_odyEmilienM: Thank for doing the release tag stuff.  Goes along nicely with the project being the first automation tool under the big tent.22:56
*** saneax_AFK is now known as saneax23:00
*** dfisher has quit IRC23:00
mwhahahawish my nova change would land23:10
* mwhahaha throws things at the CI23:10
*** ianw has joined #puppet-openstack23:13
*** johnmilton has joined #puppet-openstack23:18
*** fawadkhaliq has quit IRC23:19
*** fawadkhaliq has joined #puppet-openstack23:19
*** ducttape_ has joined #puppet-openstack23:22
*** fawadkhaliq has quit IRC23:24
*** ducttape_ has quit IRC23:25
*** ducttape_ has joined #puppet-openstack23:28
*** Matum has joined #puppet-openstack23:30
*** pleia2 has quit IRC23:43
*** pleia2 has joined #puppet-openstack23:43
*** ducttape_ has quit IRC23:59

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