Tuesday, 2016-11-08

*** jtriley has joined #puppet-openstack00:01
*** ducttape_ has joined #puppet-openstack00:11
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP Add support for swift storage policies  https://review.openstack.org/32729800:12
*** imcsk8 has joined #puppet-openstack00:18
*** ducttape_ has quit IRC00:22
*** dfisher has quit IRC00:22
openstackgerritAlex Schultz proposed openstack/puppet-tripleo: Standardize httpchk for http services  https://review.openstack.org/39468600:23
*** hecliunyx has joined #puppet-openstack00:25
*** agrebennikov has quit IRC00:35
*** markvoelker has quit IRC00:39
*** radez has joined #puppet-openstack00:46
radeziurygregory: ping00:46
radeziurygregory: I don't think I'm understanding os_service_default, my tests are now putting <SERVICE DEFAULT> all over my conf files00:47
radezI thought I understood that ::os_service_default was equavalent to the value not being set in the conf file. Is that not correct?00:48
*** toMeloos has quit IRC00:57
*** evidex has joined #puppet-openstack01:00
*** evidex has quit IRC01:06
*** fultonj has quit IRC01:09
*** jtriley has quit IRC01:09
*** ayoung has joined #puppet-openstack01:20
*** saneax is now known as saneax-_-|AFK01:25
*** ducttape_ has joined #puppet-openstack01:27
openstackgerritzhongshengping proposed openstack/puppet-barbican: Fix common options position in params.pp  https://review.openstack.org/39469001:35
openstackgerritzhongshengping proposed openstack/puppet-ceilometer: Fix common options position in params.pp  https://review.openstack.org/39469101:37
*** markvoelker has joined #puppet-openstack01:40
openstackgerritXiaohua Yuan proposed openstack/puppet-nova: remove osapi_v3 option  https://review.openstack.org/39424001:41
openstackgerritzhongshengping proposed openstack/puppet-cinder: Fix common options position in params.pp  https://review.openstack.org/39469201:42
*** markvoelker has quit IRC01:45
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty: Fix common options position in params.pp  https://review.openstack.org/39469401:45
openstackgerritzhongshengping proposed openstack/puppet-congress: Fix common options position in params.pp  https://review.openstack.org/39469701:54
*** ayoung has quit IRC02:02
openstackgerritzhongshengping proposed openstack/puppet-gnocchi: Fix common options position in params.pp  https://review.openstack.org/39469802:02
*** jtriley has joined #puppet-openstack02:05
openstackgerritzhongshengping proposed openstack/puppet-heat: Fix common options position in params.pp  https://review.openstack.org/39469902:06
openstackgerritzhongshengping proposed openstack/puppet-ironic: Fix common options position in params.pp  https://review.openstack.org/39470002:09
*** jtriley has quit IRC02:10
openstackgerritzhongshengping proposed openstack/puppet-magnum: include ::openstacklib::defaults in params.pp  https://review.openstack.org/39470202:17
openstackgerritzhongshengping proposed openstack/puppet-manila: Fix common options position in params.pp  https://review.openstack.org/39470402:20
*** michapma_alt has joined #puppet-openstack02:23
*** ducttape_ has quit IRC02:25
*** imcsk8 has quit IRC02:44
*** imcsk8 has joined #puppet-openstack02:45
openstackgerritzhongshengping proposed openstack/puppet-manila: Fix common options position in params.pp  https://review.openstack.org/39470403:03
openstackgerritzhongshengping proposed openstack/puppet-murano: Fix common options position in params.pp  https://review.openstack.org/39471703:10
openstackgerritzhongshengping proposed openstack/puppet-oslo: Fix common options position in params.pp  https://review.openstack.org/39472203:23
*** ducttape_ has joined #puppet-openstack03:27
openstackgerritzhongshengping proposed openstack/puppet-rally: include ::openstacklib::defaults in params.pp  https://review.openstack.org/39472403:28
*** ducttape_ has quit IRC03:30
*** _ducttape_ has joined #puppet-openstack03:30
openstackgerritzhongshengping proposed openstack/puppet-swift: Fix common options position in params.pp  https://review.openstack.org/39472803:33
*** ilbot3 has quit IRC03:34
*** ayoung has joined #puppet-openstack03:37
openstackgerritzhongshengping proposed openstack/puppet-trove: Fix common options position in params.pp  https://review.openstack.org/39472903:40
*** markvoelker has joined #puppet-openstack03:41
openstackgerritzhongshengping proposed openstack/puppet-vitrage: include ::openstacklib::defaults in params.pp  https://review.openstack.org/39473103:41
*** markvoelker has quit IRC03:46
*** ayoung has quit IRC03:54
*** ayoung has joined #puppet-openstack03:56
*** jtriley has joined #puppet-openstack03:59
*** jtriley has quit IRC04:04
*** ayoung has quit IRC04:06
openstackgerritBrent Eagles proposed openstack/puppet-tripleo: WIP: Call VF configuration from udev rules  https://review.openstack.org/39460404:09
*** ccamacho has quit IRC04:13
*** ccamacho has joined #puppet-openstack04:13
*** jtriley has joined #puppet-openstack04:26
*** jtriley has quit IRC04:32
*** jtriley has joined #puppet-openstack04:36
openstackgerritBrent Eagles proposed openstack/puppet-tripleo: Add proper handling of IPv6 addresses for rabbit host/port handling  https://review.openstack.org/39474204:39
*** jtriley has quit IRC04:40
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-ceilometer: Remove old authtoken options  https://review.openstack.org/39474604:48
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-cinder: Remove old authtoken options  https://review.openstack.org/39474704:48
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-designate: Remove old authtoken options  https://review.openstack.org/39474804:48
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-glance: Remove old authtoken options  https://review.openstack.org/39474904:48
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-heat: Remove old authtoken options  https://review.openstack.org/39475004:49
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-ironic: Remove old authtoken options  https://review.openstack.org/39475104:49
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-manila: Remove old authtoken options  https://review.openstack.org/39475204:49
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-nova: Remove old authtoken options  https://review.openstack.org/39475304:49
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-trove: Remove old authtoken options  https://review.openstack.org/39475404:49
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-zaqar: Remove old authtoken options  https://review.openstack.org/39475504:49
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-neutron: Remove old authtoken options  https://review.openstack.org/39475604:50
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-gnocchi: Remove old authtoken options  https://review.openstack.org/39475704:51
*** evidex has joined #puppet-openstack05:01
*** evidex has quit IRC05:06
*** dencaval_ has joined #puppet-openstack05:14
*** dencaval has quit IRC05:17
*** dencaval_ is now known as dencaval05:17
openstackgerritBrent Eagles proposed openstack/puppet-tripleo: Add proper handling of IPv6 addresses for rabbit host/port handling  https://review.openstack.org/39474205:21
*** _ducttape_ has quit IRC05:24
*** ducttape_ has joined #puppet-openstack05:25
*** ducttape_ has quit IRC05:30
*** jtriley has joined #puppet-openstack05:36
*** jtriley has quit IRC05:41
*** jperry has joined #puppet-openstack05:42
*** rcernin has joined #puppet-openstack05:54
*** saneax-_-|AFK is now known as saneax06:19
*** jaosorior has joined #puppet-openstack06:20
*** jaosorior has quit IRC06:32
*** jaosorior has joined #puppet-openstack06:33
*** jtriley has joined #puppet-openstack06:37
*** richm has quit IRC06:41
*** markvoelker has joined #puppet-openstack06:43
*** jtriley has quit IRC06:43
*** mkoderer has joined #puppet-openstack06:44
*** saneax is now known as saneax-_-|AFK06:45
*** acabot has quit IRC06:47
*** markvoelker has quit IRC06:47
*** saneax-_-|AFK is now known as saneax06:59
*** snarwade has quit IRC06:59
*** evidex has joined #puppet-openstack07:02
*** amorin has left #puppet-openstack07:02
*** tesseract has joined #puppet-openstack07:03
*** tesseract is now known as Guest657007:03
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Include local CA in haproxy PEM  https://review.openstack.org/39479307:06
*** evidex has quit IRC07:07
*** snarwade has joined #puppet-openstack07:11
*** evidex has joined #puppet-openstack07:12
*** ebalduf has quit IRC07:23
*** jtriley has joined #puppet-openstack07:39
*** jtriley has quit IRC07:44
*** hjensas has joined #puppet-openstack07:47
*** hjensas has quit IRC07:47
*** hjensas has joined #puppet-openstack07:47
*** evidex has quit IRC07:51
*** ducttape_ has joined #puppet-openstack07:57
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/39244607:59
*** shardy has joined #puppet-openstack08:01
*** ducttape_ has quit IRC08:01
*** chem has quit IRC08:12
*** chem has joined #puppet-openstack08:12
*** jperry has quit IRC08:21
*** paramite has joined #puppet-openstack08:22
*** jpich has joined #puppet-openstack08:34
*** jpena|off is now known as jpena08:38
*** amoralej|off is now known as amoralej08:39
*** jtriley has joined #puppet-openstack08:40
*** markvoelker has joined #puppet-openstack08:43
*** jtriley has quit IRC08:45
*** markvoelker has quit IRC08:48
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-heat: Add manifests to deploy APIs over httpd  https://review.openstack.org/39483509:09
*** zhongshengping has joined #puppet-openstack09:12
*** lucas-afk is now known as lucasagomes09:14
openstackgerritXingchao Yu proposed openstack/puppet-cloudkitty: Introduce cloudkitty::deps and cloudkitty::storage  https://review.openstack.org/39401809:17
*** zhongshengping has quit IRC09:20
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: Specify tripleo::haproxy for keepalived dependency.  https://review.openstack.org/39484409:25
*** ducttape_ has joined #puppet-openstack09:27
*** ducttape_ has quit IRC09:32
*** jtriley has joined #puppet-openstack09:41
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-heat: Add manifests to deploy APIs over httpd  https://review.openstack.org/39483509:41
*** jtriley has quit IRC09:46
*** derekh has joined #puppet-openstack09:49
openstackgerritJulie Pichon proposed openstack/puppet-tripleo: Enable communication between UI and the Undercloud by making HAProxy proxy for the UI  https://review.openstack.org/39339010:01
openstackgerritMerged openstack/puppet-openstack_spec_helper: Pin addressable rubygem  https://review.openstack.org/39462410:04
jaosoriorjpich: is there an LP bug for this? https://review.openstack.org/#/c/393390/10:06
jpichjaosorior: It seems like it was associated with https://blueprints.launchpad.net/tripleo/+spec/enable-communication-ui-undercloud , whether one considers it a bug or a bp is another question...10:06
jpichs/bp/feature/10:07
openstackgerritJulie Pichon proposed openstack/puppet-tripleo: Enable communication between UI and the Undercloud by making HAProxy proxy for the UI  https://review.openstack.org/39339010:10
openstackgerritMarios Andreou proposed openstack/puppet-tripleo: Specify tripleo::haproxy for keepalived dependency.  https://review.openstack.org/39484410:10
*** gfidente has joined #puppet-openstack10:21
*** gfidente has quit IRC10:21
*** gfidente has joined #puppet-openstack10:21
*** dtantsur|afk is now known as dtantsur10:31
*** jperry has joined #puppet-openstack10:32
*** _degorenko|afk is now known as degorenko10:32
openstackgerritXingchao Yu proposed openstack/puppet-cloudkitty: Add support for configuring common component  https://review.openstack.org/39402810:33
*** maestropandy has joined #puppet-openstack10:38
*** maestropandy has left #puppet-openstack10:41
*** jtriley has joined #puppet-openstack10:42
openstackgerritChris Jones proposed openstack/puppet-tripleo: Improve failed mysql node removal time in HA deploys.  https://review.openstack.org/39367310:45
*** jtriley has quit IRC10:47
*** richm has joined #puppet-openstack11:10
*** joelio has joined #puppet-openstack11:21
joelioHi, could you point me to documentation on how to correctly add 'releasenotes' to a puppet module? Failing to find it on the site11:22
*** saneax is now known as saneax-_-|AFK11:23
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Pass X-Forwarded-Proto for missing services  https://review.openstack.org/39491011:23
*** dprince has joined #puppet-openstack11:24
*** ntpttr has quit IRC11:30
*** ntpttr has joined #puppet-openstack11:31
dtantsurjoelio, http://docs.openstack.org/developer/reno/usage.html#creating-new-release-notes11:41
joeliomany thanks dtantsur11:41
dtantsurnp11:42
*** jtriley has joined #puppet-openstack11:43
*** jtriley has quit IRC11:47
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-cinder: Add preload wsgi script option to apache's configuration  https://review.openstack.org/39378811:50
iurygregorygood morning11:54
*** jpena is now known as jpena|lunch11:59
*** amoralej is now known as amoralej|lunch12:00
openstackgerritJoel Merrick proposed openstack/puppet-horizon: Add support for 'Retrieve Password' via Horizon UI  https://review.openstack.org/39453412:07
*** paramite has quit IRC12:08
openstackgerritJoel Merrick proposed openstack/puppet-horizon: Add support for 'Retrieve Password' via Horizon UI  https://review.openstack.org/39453412:09
joelioignore that, test still failing12:13
*** paramite has joined #puppet-openstack12:27
*** toMeloos has joined #puppet-openstack12:31
*** dtantsur is now known as dtantsur|brb12:34
*** mkoderer has quit IRC12:42
joelioI'm running the acceptance tests on a vanilla clone of puppet-horizon and I get failures out of the box, is this correct?12:45
joelio"rspec ./spec/acceptance/horizon_with_apache_spec.rb:7 # horizon class default parameters should work with no errors"12:45
*** markvoelker has joined #puppet-openstack12:45
iurygregorydo you have the logs in CI?12:45
joelioI'm running locally12:46
joelioI did push a review before actually, perhaps it's worked it's way through12:46
iurygregorycan you point the link?12:46
joelioassumed it was me that broke it with my change, but it's not relevant to it12:46
iurygregory=)12:46
joeliosure, one mo12:46
joeliohttps://review.openstack.org/394534 is the review12:47
joelioapologies if something's obviously wrong, it's my first time ;)12:48
iurygregoryrlx =)12:48
*** lucasagomes is now known as lucas-hungry12:49
iurygregorywell patch #1,2,3 have +1 from jenkis12:49
iurygregoryi'll wait for the result in patch 512:49
iurygregorylocally i run acceptance with vagrant =D12:49
*** markvoelker has quit IRC12:50
joelioyea, the beaker tests spin up in vagrant, this is the thing that has failed for me12:50
joeliothe spec was added in patch 4 fwiw12:51
iurygregorybut you didnt touch acceptance in spec12:52
iurygregoryto run spec for classes you just need to http://docs.openstack.org/developer/puppet-openstack-guide/testing.html#running-rspec12:52
joeliosure, I read that.. last line in rspec commands.......12:53
joeliothat's the failing one, if I don't need to run that then cool :)12:53
joelioeverything else passes12:53
iurygregoryO.o12:53
iurygregorydid you install everything?12:54
joelioyup :)12:54
joelioI'll wait for this patch set to work it's way through, see what that says for verificatiomn12:54
joelioit fires up vagrant fine, churns away for a while but breaks on the line I listed above (which ironically isn't touched by my pr)12:55
*** johnmilton has joined #puppet-openstack12:56
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: Specify tripleo::haproxy for keepalived dependency.  https://review.openstack.org/39484412:57
*** ducttape_ has joined #puppet-openstack12:58
*** fultonj has joined #puppet-openstack13:00
*** snarwade has quit IRC13:00
*** trown|outtypewww is now known as trown13:01
*** pradk has joined #puppet-openstack13:05
*** jayg|g0n3 is now known as jayg13:05
*** ilbot3 has joined #puppet-openstack13:06
*** jperry has quit IRC13:09
*** moravec has joined #puppet-openstack13:13
*** markvoelker has joined #puppet-openstack13:14
openstackgerritMerged openstack/puppet-openstacklib: Add region support to credentials  https://review.openstack.org/39454213:22
openstackgerritMerged openstack/puppet-openstacklib: Add region support to credentials  https://review.openstack.org/39454413:22
*** jpena|lunch is now known as jpena13:22
*** moravec has left #puppet-openstack13:22
*** fultonj_ has joined #puppet-openstack13:29
*** fultonj_ has quit IRC13:29
*** amoralej|lunch is now known as amoralej13:34
openstackgerritSven Anderson proposed openstack/puppet-tripleo: Implement Nova ec2api profile  https://review.openstack.org/33656213:36
*** jtriley has joined #puppet-openstack13:45
*** ducttape_ has quit IRC13:47
*** morazi has joined #puppet-openstack13:48
*** openstackgerrit has quit IRC13:48
*** openstackgerrit has joined #puppet-openstack13:49
*** lucas-hungry is now known as lucasagomes13:59
radeziurygregory: ping, hey did you happen to see my ping from yesterday?14:00
iurygregoryradez, no i didn't14:00
iurygregoryo.o14:00
iurygregorysorry can you repeat?14:00
*** ebalduf has joined #puppet-openstack14:03
radeziurygregory: sure, np14:03
radezI'm having trouble with the os_service_default14:03
iurygregoryok14:03
radezI'm trying to test the congress puppet module14:03
radezand in the congress.conf file14:03
radezall my vaules are coming in with the literal string14:03
radez<SERVICE DEFAULT>14:03
iurygregorywow14:03
iurygregoryo.o14:04
radezhave I done something wrong?14:04
mwhahahathe provider needs to support os_service_default14:04
*** StraubTW has joined #puppet-openstack14:04
mwhahahathere's a specific thing that has to be done, let me see if i can dig it up14:04
radezI thought it did, I used the generated skeleton code14:04
iurygregoryi never saw that happen o.o14:04
radezI'll see if I can pull up the code that I'm using on git too mwhahaha to verify14:04
iurygregorycongress have acceptance ?14:05
radezhttps://github.com/openstack/puppet-congress/blob/master/lib/puppet/type/congress_config.rb14:05
radeziurygregory: not yet becasue we're waiting for the rpm package to be fnished14:05
iurygregoryradez, ack14:06
radezmwhahaha: there's what I thought handled it mwhahaha14:06
radez^^^14:06
mwhahaharadez: hmm the provider looks correct, i'm not sure off the top of my head.14:06
radezhm :/14:06
mwhahahayou do n't have an old version of openstacklib in place do you?14:07
mwhahahaare you running into it just when doing testing or what14:07
radeznot that I know of14:07
radezyes when I test14:07
radezI'm running puppet apply by hand14:07
iurygregorycan you check the versions you're using?14:07
radezthe other modules seem to have worked properly14:07
radezopenstack-puppet-modules-9.3.0-0.20161030235934.8c758d6.el7.centos.noarch14:08
radezI realize that may not be terribly helpful14:08
radezbut that deployment was run in the past day of two with the newton code base14:08
radezand OPM included in it14:08
mwhahahathat's relatively new14:09
mwhahahawe added os_service_default back in like the end of liberty or beginning of mitaka14:09
radezI I dropped congress into /etc/puppet/modules with the rest of them14:10
radezand created a .pp file in root's home and ran puppet apply on the pp file14:10
radezit just has a congress class in it14:10
radeznothing fancy14:10
*** jtriley has quit IRC14:10
mwhahahahttps://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack_config/ini_setting.rb#L6-L1114:10
mwhahahathat's specifically what handles it that should cause it to be removed14:11
mwhahahayou have to ensure you have ensure_absent_val set on the type which it looks like it was14:11
mwhahahaso i'll have to poke at it some more after this mornings irc meetings14:11
radezso that checks if the value it defined in the conf file14:11
radezand sets it absent if it's not there so that it doesn't get defined correct?14:11
mwhahahahttps://github.com/openstack/puppet-congress/blob/master/lib/puppet/type/congress_config.rb#L44-L4714:11
mwhahahaif it comes across the ensure_absent_val it sets it to absent instead of writing out the value14:12
mwhahahaor at least it's supposed to14:12
*** jtriley has joined #puppet-openstack14:12
radezmwhahaha: and the last link to the congress_config.rb file is correct?14:12
mwhahahayea14:12
radezack14:12
iurygregorymaybe we missed something i don't know14:13
iurygregoryi'll try to look at new modules like ec2api and watcher..14:13
openstackgerritJoel Merrick proposed openstack/puppet-horizon: Add support for 'Retrieve Password' via Horizon UI  https://review.openstack.org/39453414:13
radezk, thx for your help iurygregory and mwhahaha, I'll keep poking a bit too and see if I can figure out anything14:14
iurygregorylet us know if you find something =)14:14
*** ducttape_ has joined #puppet-openstack14:14
radezyea sure will14:20
*** agrebennikov has joined #puppet-openstack14:22
*** jperry has joined #puppet-openstack14:23
*** jayg is now known as jayg|g0n314:23
*** jayg|g0n3 is now known as jayg14:25
*** dtantsur|brb is now known as dtantsur14:27
*** jaosorior has quit IRC14:27
iurygregorymwhahaha, just a fyi authtoken https://review.openstack.org/#/q/status:open+projects:openstack/puppet+branch:master+topic:old_authtoken14:28
iurygregorywhen you have time =)14:28
mwhahahalooks like fuel isn't ready :D but i'll take a look14:28
iurygregorysome patches are fine haha XD14:29
iurygregoryothers don't :P14:29
*** ducttape_ has quit IRC14:32
*** rmart04 has joined #puppet-openstack14:34
*** jayg is now known as jayg|g0n314:42
*** agrebennikov has quit IRC14:46
*** gfidente has quit IRC14:50
*** fultonj has quit IRC14:56
*** zhongshengping has joined #puppet-openstack14:56
*** dfisher has joined #puppet-openstack14:57
iurygregorymeeting?14:58
mwhahahayea14:58
*** gfidente has joined #puppet-openstack14:59
openstackgerritJulie Pichon proposed openstack/puppet-tripleo: Persist the UI config file  https://review.openstack.org/38859215:01
*** jayg|g0n3 is now known as jayg15:03
openstackgerritCarlos Camacho proposed openstack/puppet-swift: Remove trailing whitespace  https://review.openstack.org/39501615:06
openstackgerritCarlos Camacho proposed openstack/puppet-swift: Remove trailing whitespace  https://review.openstack.org/39501615:08
*** ducttape_ has joined #puppet-openstack15:10
*** ducttape_ has quit IRC15:11
*** ducttape_ has joined #puppet-openstack15:11
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-openstack-guide: Add meeting #96  https://review.openstack.org/39501915:11
*** agrebennikov has joined #puppet-openstack15:15
*** zhongshengping has quit IRC15:21
*** fultonj has joined #puppet-openstack15:28
*** michapma_alt has quit IRC15:29
openstackgerritMichael Glaser proposed openstack/puppet-tripleo: Wrong default in docstring  https://review.openstack.org/39504015:35
openstackgerritMichael Glaser proposed openstack/puppet-tripleo: Wrong default in docstring  https://review.openstack.org/39504015:39
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: Ensure keepalived is restarted when necessary.  https://review.openstack.org/39505315:50
*** ayoung has joined #puppet-openstack15:52
mfischhey guys sorry about that15:56
mfischstupid daylight savings time15:56
mfischI was here ready to go15:56
openstackgerritMerged openstack/puppet-tripleo: Increase haproxy timeouts  https://review.openstack.org/39437816:00
*** Guest6570 has quit IRC16:01
mwhahahachem: we need to tag puppet-pacemaker, https://review.openstack.org/39506116:06
chemmwhahaha: ack16:06
chemmwhahaha: done16:07
mwhahahaty i'll push the tag later16:07
*** jtriley has quit IRC16:09
*** jtriley has joined #puppet-openstack16:09
mfischdmsimard: yt?16:12
dmsimardmfisch: yeah16:12
openstackgerritMerged openstack/puppet-tripleo: Improve failed mysql node removal time in HA deploys.  https://review.openstack.org/39367316:12
mfischdmsimard: I'd like to go through how we can undo the giant disaster that is trove-guestagent and taskmanager16:13
mfischI thnk taskmanager including it is a disaster so I tried to fix16:14
mfischbut now CI fails and acceptance too16:14
dmsimardmfisch: is this something new in ocata ?16:16
mfischnot really, it was done in a terrible way to start with16:16
mfischguestagent is weird because you need the config but you dont run the service on your controllers16:16
mfischthe service runs in guests16:16
mfischthis is the topic: https://review.openstack.org/#/c/394218/16:17
mfischand https://review.openstack.org/#/c/39295716:17
dmsimardright, the agent is in the trove image, not on the controller16:18
dmsimardhuh16:18
openstackgerritMerged openstack/puppet-tripleo: Enable TLS in the internal network for gnocchi  https://review.openstack.org/37764516:19
*** rcernin has quit IRC16:20
mfischdmsimard: so I'm looking for ideas on fixing that CI job first16:21
dmsimarddefine fix, because it was green until you submitted something16:21
dmsimard(trying to understand)16:21
* dmsimard not super familiar with trove to begin with16:22
mfischdmsimard: trove-guestagent needs to be includeable stand-alone16:27
mfischtask-manager shouldnt be including it that was my original change16:27
mfischdmsimard: what did your last comment on 394218 mean?16:29
dmsimardmfisch: you're adding just class { '::trove::guestagent': }16:30
dmsimardbut we need to pass some actual parameters16:30
dmsimardmfisch: like https://github.com/openstack/puppet-trove/blob/master/manifests/guestagent.pp#L10916:30
mfischwhats wrong with the defaults? why does that break stuff?16:31
dmsimardthe default is wrong, it needs to be along to lines of $::openstack_integration::config::keystone_admin_uri for example16:31
dmsimardsame for swift parameters16:31
mfischoh yeah thats clearly wrong16:31
mfischbut this class isnt really going to run anything since this service doesnt really run16:32
dmsimardmfisch: you can see here that it's setting some garbage parameters http://logs.openstack.org/18/394218/1/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/3f2c1d4/logs/puppet.txt.gz#_2016-11-07_01_41_2616:32
*** rmart04 has quit IRC16:32
mfischreading16:32
mfischhow did you highlight that section?  #magic16:32
mfischI can try that16:33
dmsimardmfisch: click on the timestamps :)16:34
mfischI had no idea16:34
*** iurygregory has left #puppet-openstack16:35
*** iurygregory has joined #puppet-openstack16:35
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: Ensure keepalived is restarted when necessary.  https://review.openstack.org/39505316:38
dmsimardmfisch: so, the guestagent class doesn't actually run anything but it does quite a bit of config stuff16:44
mfischright16:44
mfischI cant think of any reason why it would ever run on the box16:45
mfischthe guest agent sits on the trove instance and talks back to Trove via rabbit16:45
openstackgerritMatt Fischer proposed openstack/puppet-openstack-integration: Remove deprecated trove references  https://review.openstack.org/39421816:45
mfischlets see if that helps16:45
dmsimardmfisch: also, does that change depend on the puppet-trove change ?16:49
dmsimardor should it work on it's own ?16:49
mfischit should work on its own16:53
mfischits a pre-req b/c the other one wont pass until I fix this16:53
*** rcernin has joined #puppet-openstack16:55
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: Better way to ensure keepalived before haproxy.  https://review.openstack.org/39484416:57
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: Ensure keepalived is restarted when necessary.  https://review.openstack.org/39505316:57
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: Ensure keepalived is restarted when necessary.  https://review.openstack.org/39505317:00
*** dencaval has quit IRC17:02
*** paramite has quit IRC17:04
*** fultonj has quit IRC17:11
*** mesamoo has quit IRC17:12
*** dtantsur is now known as dtantsur|afk17:16
*** johnmilton has quit IRC17:16
openstackgerritMerged openstack/puppet-openstack-guide: Add meeting #96  https://review.openstack.org/39501917:19
*** paramite has joined #puppet-openstack17:20
*** rcernin has quit IRC17:29
*** bogdando has joined #puppet-openstack17:33
*** jpich has quit IRC17:34
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: Better way to ensure keepalived before haproxy.  https://review.openstack.org/39484417:36
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: Ensure keepalived is restarted when necessary.  https://review.openstack.org/39505317:36
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: Ensure keepalived is restarted when necessary.  https://review.openstack.org/39505317:38
*** trown is now known as trown|lunchnvote17:43
openstackgerritMatt Fischer proposed openstack/puppet-openstack-integration: Remove deprecated trove references  https://review.openstack.org/39421817:46
*** derekh has quit IRC17:51
*** shardy has quit IRC17:54
openstackgerritMerged openstack/puppet-tripleo: Add proper handling of IPv6 addresses for rabbit host/port handling  https://review.openstack.org/39474218:00
*** jperry has quit IRC18:00
*** lucasagomes is now known as lucas-afk18:07
*** gfidente has quit IRC18:09
*** jperry has joined #puppet-openstack18:10
openstackgerritBrent Eagles proposed openstack/puppet-tripleo: Add proper handling of IPv6 addresses for rabbit host/port handling  https://review.openstack.org/39510418:11
mwhahahadmsimard: is there an rdo marathon package yet?18:29
dmsimardmwhahaha: there should be, unless apevec forgot to pull it.. let me see18:29
*** jpena has left #puppet-openstack18:29
dmsimardmwhahaha: nope, it's there https://cbs.centos.org/koji/packageinfo?packageID=501018:30
dmsimardmwhahaha: I believe amoralej also added it as a requirement to the spec, let me check18:30
mwhahahak i shall recheck magnum and see where we're at18:30
dmsimardmwhahaha: https://review.rdoproject.org/r/#/c/3548/ yup should be good18:30
amoralejyes, but not promoted yet18:30
amoralejdmsimard ^18:30
*** jpena has joined #puppet-openstack18:30
amoralejas we are inconsistent in master since last week18:30
dmsimardahhhhh18:31
dmsimardso that means we don't have the designate fix either18:31
*** jpena is now known as jpena|off18:31
dmsimardyeah last repo goes back to nov 3rd https://trunk.rdoproject.org/centos7-master/puppet-passed-ci/?C=M;O=D18:32
openstackgerritAdam Vinsh proposed openstack/puppet-swift: Add support for swift storage policies  https://review.openstack.org/32729818:33
vinshwoohoo. Finally ready ^18:33
vinshLots of release nodes/readme updates added.18:34
*** johnmilton has joined #puppet-openstack18:35
*** jtriley has quit IRC18:35
openstackgerritAdam Vinsh proposed openstack/puppet-swift: Add support for swift storage policies  https://review.openstack.org/32729818:35
*** fultonj has joined #puppet-openstack18:40
*** jtriley has joined #puppet-openstack18:43
*** agrebennikov has quit IRC18:44
openstackgerritAdam Vinsh proposed openstack/puppet-swift: Add support for swift storage policies  https://review.openstack.org/32729818:49
mfischdmsimard: its better now but I think rdo packages are busted18:50
dmsimard /maybe/18:50
dmsimardyou have a link ?18:51
openstackgerritMatt Fischer proposed openstack/puppet-trove: Ocata deprecation cleanup (partial)  https://review.openstack.org/39295718:54
*** ebalduf has quit IRC18:55
*** ccamacho has quit IRC19:08
*** agrebennikov has joined #puppet-openstack19:11
*** amoralej is now known as amoralej|off19:11
*** ayoung has quit IRC19:16
*** ayoung has joined #puppet-openstack19:17
openstackgerritAlex Schultz proposed openstack/puppet-tripleo: Standardize httpchk for http services  https://review.openstack.org/39468619:23
openstackgerritAlex Schultz proposed openstack/puppet-tripleo: Standardize httpchk for http services  https://review.openstack.org/39468619:30
*** paramite has quit IRC19:30
*** trown|lunchnvote is now known as trown19:35
openstackgerritAlex Schultz proposed openstack/puppet-glance: Deprecate rabbitmq connection parameters  https://review.openstack.org/39513319:43
openstackgerritAlex Schultz proposed openstack/puppet-heat: Deprecate rabbitmq connection parameters  https://review.openstack.org/39513419:47
openstackgerritAlex Schultz proposed openstack/puppet-ironic: Deprecate rabbitmq connection parameters  https://review.openstack.org/39513719:49
openstackgerritMerged openstack/puppet-congress: Deprecate rabbitmq connection parameters  https://review.openstack.org/39197019:53
openstackgerritAlex Schultz proposed openstack/puppet-keystone: Deprecate rabbitmq connection parameters  https://review.openstack.org/39514219:57
*** iurygregory has quit IRC19:58
*** ebalduf has joined #puppet-openstack20:01
openstackgerritAlex Schultz proposed openstack/puppet-magnum: Deprecate rabbitmq connection parameters  https://review.openstack.org/39514320:04
*** iurygregory has joined #puppet-openstack20:22
openstackgerritAlex Schultz proposed openstack/puppet-manila: Deprecate rabbitmq connection parameters  https://review.openstack.org/39515120:24
openstackgerritAlex Schultz proposed openstack/puppet-mistral: Deprecate rabbitmq connection parameters  https://review.openstack.org/39515520:30
openstackgerritMerged openstack/puppet-horizon: Add support for 'Retrieve Password' via Horizon UI  https://review.openstack.org/39453420:32
openstackgerritMerged openstack/puppet-barbican: Deprecate rabbitmq connection parameters  https://review.openstack.org/39193520:32
*** jtriley has quit IRC20:38
*** iurygregory has quit IRC20:41
*** ayoung has quit IRC20:44
*** jtriley has joined #puppet-openstack20:47
*** ducttape_ has quit IRC20:52
*** rcernin has joined #puppet-openstack20:59
*** agrebennikov has quit IRC21:03
*** ducttape_ has joined #puppet-openstack21:08
*** jayg is now known as jayg|g0n321:09
*** toMeloos has quit IRC21:11
*** dprince has quit IRC21:18
*** mfisch has quit IRC21:21
*** mfisch has joined #puppet-openstack21:21
*** mfisch has quit IRC21:21
*** mfisch has joined #puppet-openstack21:21
*** johnmilton has quit IRC21:28
openstackgerritAlex Schultz proposed openstack/puppet-mistral: Deprecate rabbitmq connection parameters  https://review.openstack.org/39515521:43
*** chem has quit IRC21:44
*** iurygregory has joined #puppet-openstack21:52
*** agrebennikov has joined #puppet-openstack21:53
*** trown is now known as trown|outtypewww21:59
openstackgerritAlex Schultz proposed openstack/puppet-murano: Deprecate rabbitmq connection parameters  https://review.openstack.org/39521322:01
openstackgerritAlex Schultz proposed openstack/puppet-neutron: Deprecate rabbitmq connection parameters  https://review.openstack.org/39521522:07
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-nova: Remove old authtoken options  https://review.openstack.org/39475322:10
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-zaqar: Remove old authtoken options  https://review.openstack.org/39475522:11
mwhahahaugh nova cells and rabbit configuration :(22:12
*** jtriley has quit IRC22:13
openstackgerritAlex Schultz proposed openstack/puppet-nova: Deprecate rabbitmq connection parameters  https://review.openstack.org/39521922:14
openstackgerritAlex Schultz proposed openstack/puppet-octavia: Deprecate rabbitmq connection parameters  https://review.openstack.org/39522022:17
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-ironic: Remove old authtoken options  https://review.openstack.org/39475122:24
openstackgerritAlex Schultz proposed openstack/puppet-sahara: Deprecate rabbitmq connection parameters  https://review.openstack.org/39522422:24
openstackgerritLokesh Jain proposed openstack/puppet-neutron: Updated Nuage package name for Mitaka release  https://review.openstack.org/39522522:26
openstackgerritLokesh Jain proposed openstack/puppet-neutron: Updated Nuage package name for Mitaka release  https://review.openstack.org/39522722:30
openstackgerritAlex Schultz proposed openstack/puppet-swift: Deprecate rabbitmq connection parameters  https://review.openstack.org/39522922:34
openstackgerritAlex Schultz proposed openstack/puppet-tacker: Deprecate rabbitmq connection parameters  https://review.openstack.org/39523322:39
*** pradk has quit IRC22:40
mwhahahamfisch: since you use trove, do you know how the oslo messaging default/transport url works for the taskmanager/conductor/guestagent?22:40
mwhahahai guess they all get put in different config files22:41
mwhahahaso many rabbitmq deprecations :(22:41
dfisherwhat's the reason for rabbitmq deprecations?22:42
dfisheri feel like I've missed a memo :)22:42
mwhahahaoslo is switching to DEFAULT/transport_url22:42
dfisherin Ocata?22:42
mfischmwhahaha: I can look22:43
mwhahahaall the rabbit_{host,port,user,pass,virtual_host} have been nuked22:43
mwhahahanewton22:43
mfischwhats it now? 1 URL?22:43
mwhahahayea22:43
mfischlike rabbit://foo:567322:43
mwhahahayea22:43
dfisherah, cool22:43
mwhahahaDEFAULT/transport_url is the new config setting22:43
*** agrebennikov has quit IRC22:43
mwhahahai've never liked urls because i feel it limits password options22:43
dfisherthis right here is why OpenStack needs an architecture committee with the ability to tell people "no"22:43
dfisherthis and about a billion other things :)22:44
mwhahahapretty much22:44
mwhahahaso much effort to just shuffle configs around22:44
dfisherso, random shot in the dark but can anybody explain Matt R's somewhat cryptic note about upgrading to Ocata Nova with this "placement" service?22:45
mwhahahathere's a new placement service22:45
mwhahaha:D22:45
mfisch"This too shall be deprecated"22:45
* dfisher sighs22:45
mwhahahawe don't currently support configuring it22:45
dfisherok22:45
mwhahahasoooOooOo if you want to add that22:45
mwhahahathere's a bug somewhere22:45
mwhahahaEmilienM was going to take a look at it but he didn't get a chance and nows he's on vaca22:46
dfishersorry, a billion things to do on my end (like make sed -i work in Solaris)22:46
mwhahahamaybe after i finish deprecating all these rabbit configs i'll take a look at it22:47
mwhahahai'm almost done22:47
dfisherso, we skip every other release in Solaris.  We'll do Mitaka -> Ocata and I'm trying to learn what I can about this new service that does … stuff …. to things …. in Nova.22:47
dfisherand the Nova guys don't like us :)22:48
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-barbican: Remove old authtoken options  https://review.openstack.org/39523822:51
*** jperry has quit IRC22:57
openstackgerritAlex Schultz proposed openstack/puppet-trove: Deprecate rabbitmq connection parameters  https://review.openstack.org/39524223:03
openstackgerritAlex Schultz proposed openstack/puppet-vitrage: Deprecate rabbitmq connection parameters  https://review.openstack.org/39524323:06
openstackgerritAlex Schultz proposed openstack/puppet-watcher: Deprecate rabbitmq connection parameters  https://review.openstack.org/39524423:12
openstackgerritAlex Schultz proposed openstack/puppet-watcher: Deprecate rabbitmq connection parameters  https://review.openstack.org/39524423:17
*** rcernin has quit IRC23:18
*** StraubTW has quit IRC23:23
*** ducttape_ has quit IRC23:27
mwhahahathis designate module makes me sad23:27
*** michapma_alt has joined #puppet-openstack23:34
iurygregoryin new modules we should deprecate or just remove?23:40
mwhahahai'd rather not break people who are probably using the old rabbit_* params23:42
mwhahahaother might say too bad, but i think for this one we probably should have a warning for a while23:45
iurygregoryok =D23:47
*** ayoung has joined #puppet-openstack23:52
openstackgerritAlex Schultz proposed openstack/puppet-designate: Deprecate rabbitmq connection parameters  https://review.openstack.org/39526023:55
mwhahaha(╯°□°)╯︵ ƃuıƃɐssǝɯ˙oʃso23:56
iurygregoryahsuahsuashaush23:58
iurygregorybest meme23:59

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