Friday, 2016-09-30

« Thursday, 2016-09-29 Index Saturday, 2016-10-01 »
*** ducttape_ has joined #puppet-openstack00:08
openstackgerritAdam Vinsh proposed openstack/puppet-swift: Replace proxy concat/template with ini provider  https://review.openstack.org/37895000:12
*** ducttape_ has quit IRC00:20
*** markvoelker has joined #puppet-openstack00:28
*** xarses has joined #puppet-openstack00:36
*** leanderthal|afk has quit IRC00:36
*** saneax is now known as saneax-_-|AFK00:41
*** rain has joined #puppet-openstack00:50
*** rain is now known as Guest2977600:51
*** ducttape_ has joined #puppet-openstack01:05
*** fultonj has joined #puppet-openstack01:25
*** chem has quit IRC01:26
*** ducttape_ has quit IRC01:27
*** iurygregory_ has quit IRC01:35
*** ducttape_ has joined #puppet-openstack01:36
*** ilbot3 has quit IRC01:45
*** ilbot3 has joined #puppet-openstack01:48
*** fultonj has quit IRC01:48
*** ducttape_ has quit IRC01:50
openstackgerritMerged openstack/puppet-openstack-integration: Add Designate test coverage to scenario003  https://review.openstack.org/37899502:21
openstackgerritMerged openstack/puppet-openstack-integration: Enable Volume encryption testing again  https://review.openstack.org/36314202:21
*** ducttape_ has joined #puppet-openstack02:51
*** ducttape_ has quit IRC02:56
*** priteau has joined #puppet-openstack03:16
*** priteau has quit IRC03:21
*** ducttape_ has joined #puppet-openstack03:52
*** ducttape_ has quit IRC03:57
*** markvoelker has quit IRC04:01
*** ducttape_ has joined #puppet-openstack04:53
*** ducttape_ has quit IRC04:58
*** markvoelker has joined #puppet-openstack05:02
*** markvoelker has quit IRC05:07
*** skramaja has joined #puppet-openstack05:23
*** richm has quit IRC05:39
*** jaosorior has joined #puppet-openstack05:40
openstackgerritzhongshengping proposed openstack/puppet-cinder: Deprecate backup_topic option  https://review.openstack.org/37620705:50
*** rcernin has joined #puppet-openstack06:07
openstackgerritSharat Sharma proposed openstack/puppet-designate: Added the options to be configured for designate-central service  https://review.openstack.org/37995006:15
*** jaosorior has quit IRC06:40
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Remove unused pacemaker profiles  https://review.openstack.org/37995706:41
openstackgerritgecong proposed openstack/puppet-tripleo: Fix a typo in haproxy.pp  https://review.openstack.org/37995806:41
openstackgerritSharat Sharma proposed openstack/puppet-designate: Added rabbit_ha_queues option  https://review.openstack.org/37996106:54
*** markvoelker has joined #puppet-openstack07:03
*** markvoelker has quit IRC07:07
*** jpena|off is now known as jpena07:24
*** rwsu has quit IRC07:27
*** ayoung has quit IRC07:47
*** jpich has joined #puppet-openstack07:53
*** ayoung has joined #puppet-openstack07:53
*** rwsu has joined #puppet-openstack07:54
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/37998307:55
*** ayoung has quit IRC08:00
*** saneax-_-|AFK is now known as saneax08:02
*** dgurtner has joined #puppet-openstack08:05
*** dgurtner has joined #puppet-openstack08:05
*** dtantsur has joined #puppet-openstack08:11
*** shardy has joined #puppet-openstack08:16
*** dgurtner has quit IRC08:19
*** priteau has joined #puppet-openstack08:20
*** derekh has joined #puppet-openstack08:20
*** dgurtner has joined #puppet-openstack08:25
*** dgurtner has quit IRC08:25
*** dgurtner has joined #puppet-openstack08:25
*** ayoung has joined #puppet-openstack08:28
*** morazi has joined #puppet-openstack08:34
*** hjensas has joined #puppet-openstack08:35
openstackgerritPeter Zhurba proposed openstack/puppet-glare: Add posibility install glare from pip  https://review.openstack.org/37380508:56
openstackgerritPeter Zhurba proposed openstack/puppet-glare: Add hooks for external install & svc management  https://review.openstack.org/38002608:56
*** paramite has joined #puppet-openstack08:58
*** _degorenko is now known as degorenko09:01
*** markvoelker has joined #puppet-openstack09:04
*** chem has joined #puppet-openstack09:05
*** markvoelker has quit IRC09:08
openstackgerritPeter Zhurba proposed openstack/puppet-glare: Add posibility install glare from pip  https://review.openstack.org/37380509:20
*** chem has quit IRC09:55
*** chem has joined #puppet-openstack09:55
*** richm has joined #puppet-openstack10:08
*** Guest29776 is now known as leanderthal10:33
*** leanderthal is now known as leanderthal|afk10:33
openstackgerritSharat Sharma proposed openstack/puppet-designate: Added the options to be configured for designate-central service  https://review.openstack.org/37995010:35
*** delattec has quit IRC10:36
*** gfidente|afk is now known as gfidente10:37
openstackgerritPeter Zhurba proposed openstack/puppet-glare: Add posibility install glare from pip  https://review.openstack.org/37380510:43
openstackgerritJulie Pichon proposed openstack/puppet-tripleo: Clean out UI httpd configuration file  https://review.openstack.org/38015210:49
*** ducttape_ has joined #puppet-openstack10:58
*** shardy is now known as shardy_lunch10:59
*** ducttape_ has quit IRC11:02
*** johnmilton has quit IRC11:09
*** lucas-afk is now known as lucasagomes11:31
openstackgerritMykyta Karpin proposed openstack/puppet-nova: [WIP] Add class for migration options configuration in qemu.conf  https://review.openstack.org/37875411:36
*** dprince has joined #puppet-openstack11:40
*** ccamacho has quit IRC11:40
*** shardy_lunch is now known as shardy11:42
*** tlbr has quit IRC11:43
*** jtriley has joined #puppet-openstack11:45
*** jpena is now known as jpena|lunch11:46
*** johnmilton has joined #puppet-openstack11:47
*** tlbr has joined #puppet-openstack11:50
*** ccamacho has joined #puppet-openstack11:51
*** cdelatte has joined #puppet-openstack11:56
*** ducttape_ has joined #puppet-openstack11:58
*** ducttape_ has quit IRC12:03
colonwqdmsimard, if I add VolumesBackupsAdminV2Test to the whitelist.txt, will it impact/fail the other scenarios who do not have cinder backup configured?12:09
*** jayg|g0n3 is now known as jayg12:10
dmsimardcolonwq: no because the test looks for the toggle to "really" activate itself12:11
dmsimardThe backup available config from tempest.conf.12:11
*** trown|outtypewww is now known as trown12:12
*** ducttape_ has joined #puppet-openstack12:12
colonwqdmsimard, ok thanks.12:14
*** amoralej is now known as amoralej|lunch12:14
openstackgerritKeith Schincke proposed openstack/puppet-openstack-integration: Add cinder::backup test to scenario0001  https://review.openstack.org/37715812:16
*** jtriley has quit IRC12:17
*** gfidente has quit IRC12:20
*** jtriley has joined #puppet-openstack12:20
*** fultonj has joined #puppet-openstack12:21
*** morazi has quit IRC12:23
*** ccamacho is now known as ccamacho|lunch12:25
*** jtriley has quit IRC12:26
*** ducttape_ has quit IRC12:27
iurygregorygood morning12:30
*** gfidente has joined #puppet-openstack12:30
*** skramaja has quit IRC12:33
*** ayoung has quit IRC12:43
*** ayoung has joined #puppet-openstack12:44
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_extras: repos: configure RDO Virtualization repository  https://review.openstack.org/37898612:45
EmilienMdegorenko: I missed it :( ^12:45
EmilienMthanks12:45
*** jpena|lunch is now known as jpena12:47
*** markvoelker has joined #puppet-openstack12:52
*** hjensas has quit IRC12:52
*** jtriley has joined #puppet-openstack12:56
*** ayoung has quit IRC13:02
EmilienMcolonwq, dmsimard: http://logs.openstack.org/58/377158/7/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/300fb64/console.html#_2016-09-30_12_48_41_69954313:02
openstackgerritMerged openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/37998313:04
EmilienMhttp://logs.openstack.org/58/377158/7/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/300fb64/logs/ps.txt.gz13:04
EmilienMthere is no cinder-backup process13:04
*** imcsk8 has quit IRC13:06
*** imcsk8 has joined #puppet-openstack13:08
*** jperry has joined #puppet-openstack13:11
degorenkoEmilienM, bad rebase :)13:12
openstackgerritEmilien Macchi proposed openstack/puppet-gnocchi: Default metricd workers to a safer value  https://review.openstack.org/38027813:13
*** jperry has quit IRC13:13
EmilienMdegorenko: yeah that's what happens when i do rebase very late :(13:13
*** jperry has joined #puppet-openstack13:13
*** jtriley has quit IRC13:13
degorenkoEmilienM, that happens sometime with all of us :D13:16
EmilienM:)13:16
openstackgerritBrent Eagles proposed openstack/puppet-neutron: Change worker defaults to ::os_workers  https://review.openstack.org/38028613:20
*** ccamacho|lunch is now known as ccamacho13:21
*** ayoung has joined #puppet-openstack13:22
*** amoralej|lunch is now known as amoralej13:22
*** ducttape_ has joined #puppet-openstack13:28
*** danpawlik has quit IRC13:30
*** jtriley has joined #puppet-openstack13:30
*** danpawlik has joined #puppet-openstack13:32
*** fultonj has quit IRC13:32
*** fultonj has joined #puppet-openstack13:34
*** ducttape_ has quit IRC13:35
*** _nick is now known as yankcrime13:39
*** jtriley has quit IRC13:42
openstackgerritPeter Zhurba proposed openstack/puppet-glare: Add posibility install glare from pip  https://review.openstack.org/37380513:43
*** jtriley has joined #puppet-openstack13:43
dmsimardThat guy is not giving up on pip.13:45
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: telemetry: normalize coordination_url  https://review.openstack.org/38030613:47
*** jtriley has quit IRC13:48
openstackgerritMykyta Karpin proposed openstack/puppet-nova: [WIP] Add class for migration options configuration in qemu.conf  https://review.openstack.org/37875413:49
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Make configure_user and configure_user_role configurable  https://review.openstack.org/38031213:51
*** morazi has joined #puppet-openstack13:56
EmilienMdmsimard: can you check if you can now +2 poi?13:56
dmsimardEmilienM: looking13:57
openstackgerritDan Radez proposed openstack/puppet-congress: Populating init.pp and parameters  https://review.openstack.org/33019513:57
dmsimardEmilienM: works.13:58
EmilienMcool13:58
dmsimardthanks for having me :)13:58
EmilienMthanks for your consistent work13:58
*** danpawlik has quit IRC13:59
iurygregorycongrats dmsimard :D13:59
*** ducttape_ has joined #puppet-openstack14:04
*** saneax is now known as saneax-_-|AFK14:06
mwhahahaEmilienM or iurygregory can you please review https://review.openstack.org/#/c/379796/ we will need to backport this for newton14:10
iurygregorymwhahaha, doing14:10
EmilienMsure14:10
EmilienMlgtm, nice catch14:11
iurygregoryEmilienM is fast =P14:11
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/38033114:14
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Enable Volume encryption testing again  https://review.openstack.org/38033314:15
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Re-enable trove testing on Ubuntu  https://review.openstack.org/38033414:15
beaglesmwhahaha: nice one on the workers thing btw, I was planning on bringing it up for ocata .. either introduce capped values or let the services decide (I noticed neutron is doing this)14:15
mwhahahayea unfortunately when you let the services decide it's not much better :D14:16
mwhahahaguess that's what you get when everyone devs on 2cpu vms14:16
beaglesmwhahaha: :)14:17
beaglesmwhahaha: I can very strongly recall the first time I was looking at live deployment with 40+ CPUs and a large handful of services were all using processorcount14:18
beaglesmwhahaha: talk about your "facepalm moments"14:18
mwhahaha:o14:18
*** jtriley has joined #puppet-openstack14:20
openstackgerritMerged openstack/puppet-openstack_extras: repos: configure RDO Virtualization repository  https://review.openstack.org/37898614:20
iurygregoryEmilienM, now you can cherry pick to newton ^14:21
EmilienMiurygregory: thx, /me in release meeting14:21
iurygregoryEmilienM, sorry ^^14:21
EmilienMi'll do it after14:21
openstackgerritMerged openstack/puppet-congress: Populating init.pp and parameters  https://review.openstack.org/33019514:22
*** jtriley has quit IRC14:25
*** pma_ has quit IRC14:27
*** lucasagomes is now known as lucas-afk14:31
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_extras: repos: configure RDO Virtualization repository  https://review.openstack.org/38034714:36
openstackgerritAlex Schultz proposed openstack/puppet-tripleo: Add aodh profile rspec testing  https://review.openstack.org/37440214:36
EmilienMiurygregory: done ^14:36
iurygregoryack14:36
iurygregoryi'll not say anything about the topic lol14:36
*** morazi has quit IRC14:45
openstackgerritJulie Pichon proposed openstack/puppet-tripleo: Clean out UI httpd configuration file  https://review.openstack.org/38015214:46
openstackgerritAlex Schultz proposed openstack/puppet-tripleo: Add aodh profile rspec testing  https://review.openstack.org/37440214:46
*** jtriley has joined #puppet-openstack14:47
*** jtriley has quit IRC14:53
openstackgerritMerged openstack/puppet-swift: Fixed swift::proxy::cache requirement on memcached  https://review.openstack.org/37979614:59
openstackgerritAlex Schultz proposed openstack/puppet-swift: Fixed swift::proxy::cache requirement on memcached  https://review.openstack.org/38037015:03
dmsimardwth ? http://logs.openstack.org/33/380333/1/check/gate-puppet-openstack-integration-4-scenario001-tempest-ubuntu-xenial/5939aac/console.html#_2016-09-30_14_33_22_28744015:09
dmsimardnon idempotent puppet stops aodh api ?15:09
dmsimardUbuntu tests are you drunk15:10
*** jtriley has joined #puppet-openstack15:10
EmilienMaodh-api fails to start15:11
EmilienMit's in syslog15:11
*** pepijn has joined #puppet-openstack15:11
EmilienMSep 30 14:23:12 ubuntu-xenial-rax-ord-4669736 aodh-api[20959]: aodh-api: error: unrecognized arguments: --config-file=/etc/aodh/aodh.conf --log-file=/var/log/aodh/aodh-api.log15:11
EmilienMSep 30 14:23:13 ubuntu-xenial-rax-ord-4669736 systemd[1]: aodh-api.service: Main process exited, code=exited, status=2/INVALIDARGUMENT15:11
EmilienMSep 30 14:23:12 ubuntu-xenial-rax-ord-4669736 aodh-api[20959]: usage: aodh-api [-h] [--port PORT]15:11
EmilienMthey probably introduced a regression15:11
EmilienM#ubuntu-server15:12
EmilienMI reported it, let's see how it goes15:12
EmilienMis it blocking all jobs? or ephemeral?15:13
dmsimardEmilienM: it's a newton backport15:13
dmsimardEmilienM: https://review.openstack.org/#/c/380333/15:13
EmilienMso https://review.openstack.org/#/c/363142/ was merged15:15
EmilienMI fail to find encryption volume tempest tests in http://logs.openstack.org/42/363142/11/gate/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/17b2215/console.html15:15
EmilienMiurygregory, mwhahaha ^ did you see it ?15:15
dmsimardEmilienM: ah https://review.openstack.org/#/c/363142/11/run_tests.sh is no good15:15
dmsimardEmilienM: I just rebased your old patch15:15
dmsimardEmilienM: it needs to add the test to the whitelist15:16
EmilienMyes15:16
EmilienMwhen enabling services or tests, please look tempest logs15:16
EmilienMdmsimard: I'm fixing it15:16
*** jtriley has quit IRC15:17
dmsimardEmilienM: ack15:17
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Enable TestEncryptedCinderVolumes tests (for real)  https://review.openstack.org/38037415:18
EmilienMdmsimard: i'm pretty sure it will fail15:19
EmilienMhttps://review.openstack.org/#/c/366750/ was merged in cinder/master15:19
EmilienMbut not backported in stable/newton15:19
EmilienMand our CI is AFIK deploying stable/newton15:19
dmsimardEmilienM: did that third patch merge? I just wanted to rebase15:19
dmsimardokà15:19
*** bnemec is now known as beekneemech15:20
EmilienMdmsimard: don't backport this thing until cinder patch is backported15:20
EmilienMCI won't pass anyway.15:20
EmilienMdmsimard: why do you manually promote?15:20
dmsimardstable/newton15:20
dmsimardthe proposal bot updates master15:21
dmsimardnot stable/newton15:21
dmsimardand stable/newton is 3 days old now15:21
EmilienMah ok15:22
EmilienMapproved15:22
EmilienMthey don't test Aodh, sigh15:22
EmilienMit's friday, can I troll?15:23
*** pepijn has quit IRC15:28
*** ccamacho has quit IRC15:29
EmilienMok the aodh thing was indeed a packaging issue15:31
dmsimardEmilienM: I poked openstack-cinder about the backport15:32
EmilienMcool15:32
EmilienMit's a bit late though, rc2 is out15:32
dmsimard¯\_(ツ)_/¯15:32
dmsimardIf it merges and doesn't make rc2, I'll arrange to carry it in RDO15:33
dmsimarddoesn't make rc3/release*15:33
EmilienMmhh15:33
EmilienMok I need to align stars15:36
EmilienMdmsimard: do you see now why I don't want gnocchi on ubuntu jobs?15:37
dmsimardEmilienM: so we install aodh on ubuntu jobs but it's not tested and it's causing problems ?15:37
EmilienMyes15:37
EmilienMsame for ceilo15:37
EmilienMsame for gnocchi15:37
EmilienMetc15:37
dmsimardzigo: ^ take notes, we need tempest plugins, tempest plugins everywhere15:38
EmilienMIRC bot didn't work but : https://review.openstack.org/38039015:42
dmsimardEmilienM: so we need to backport that to stable/newton? what a mess :(15:43
EmilienMyes15:43
EmilienMwelcome to Friday15:44
*** dtantsur is now known as dtantsur|afk15:50
Vinshdegorenko: Thank you for the thorough review earlier, I'll address those items now.15:51
openstackgerritAlex Schultz proposed openstack/puppet-tripleo: Add ceilometer profile rspec testing  https://review.openstack.org/38040615:53
* mjblack yawns 15:53
VinshIt's a yawn'y morning here, overcast in Boulder, CO.15:54
mjblack:D15:54
degorenkoVinsh, you're welcome :)15:54
openstackgerritAlex Schultz proposed openstack/puppet-tripleo: Only run ceilometer::db::sync on bootstrap node  https://review.openstack.org/38041415:57
mwhahahatypical boulder people showing up at 10am :D15:57
Vinsh:p15:58
*** xarses has quit IRC15:58
dmsimardEmilienM: we frankly need a core in p-o-i that cares about Ubuntu15:59
EmilienMwhat?15:59
dmsimardnot that we don't care -- but I mean maintains the kind of stuff we're struggling with right now15:59
*** ayoung has quit IRC16:00
mjblackdmsimard: what stuff are we struggling with?16:00
mwhahahathe lack of tempest plugin availability with UCA16:00
mwhahahaand the sluggish packages in UCA16:00
dmsimardmjblack: without wanting to toot my horn, parity with RDO16:00
*** b00tcat has quit IRC16:01
EmilienMit's Friday, #topic open discussion16:01
dmsimardRDO has tempest plugin packaged and we have the ability and tooling to fix issues ourselves16:01
*** ayoung has joined #puppet-openstack16:01
dmsimardFor UCA it's ... harder16:01
mjblackright16:01
mjblackUCA is a bit slower on that16:01
openstackgerritBrent Eagles proposed openstack/puppet-neutron: Change worker defaults to ::os_workers  https://review.openstack.org/38028616:02
dmsimardBasically I can't go and submit a review/patch to UCA to fix an issue I found (assuming I had the time to do that in the first place)16:02
dmsimardhopefully zigo's upstream work can really shine here16:03
mjblackI know what you mean, I'm struggling too16:03
*** ccamacho has joined #puppet-openstack16:03
dmsimardso basically what I meant by my comment is that it would be great to have someone that keeps pursuing wider and better testing coverage for UCA like we do in RDO16:03
dmsimardEmilienM and I are ... zealous (?) about ever increasing the quality of coverage16:04
dmsimardbut UCA isn't able to keep up right now16:04
*** rcernin has quit IRC16:08
mwhahahadmsimard right now, https://s-media-cache-ak0.pinimg.com/564x/80/0e/59/800e5924af658bd48819ed6929583942.jpg16:09
*** jtriley has joined #puppet-openstack16:10
dmsimardmwhahaha: It's hard to publicize how I objectively believe RDO is better than UCA without sounding super biased about it because I'm at Red Hat and I'm a RDO maintainer :(16:11
mwhahaha:D it's all good16:12
dmsimardbut I mean it's not like Mirantis did a talk at the OpenStack summit about Fuel vs TripleO or anything16:12
dmsimard:P16:12
mwhahahayea but mirantis doesn't use UCA16:12
mwhahahai wedged in UCA support into fuel, but that included a bunch of package overrides cause stuff didn't work right16:13
dmsimardWell I was more referring to a company saying their product is better than it's competitor's -- it's hard to come out like that and sound like you're not biased16:13
Vinsh*cough* docker *cough* https://bfd-github.os.cloud.twc.net/gerrit-mirrors/puppet-os_docker16:14
mwhahahadmsimard: yea i know16:14
dmsimardVinsh: not sure what that is but that doesn't work for me16:15
VinshOh, we got sick of UCA.. so we use puppet and that module to deploy the services in containers16:15
mwhahahahttps://github.com/twc-openstack/puppet-os_docker16:15
VinshYeah, sorry bad link16:15
dmsimardVinsh: ah, from source ?16:15
VinshYeah16:16
Vinshthat's what all those big "external hooks" patches we have put up are for16:16
dmsimardVinsh: so not unlike, say, Kolla or OpenStack-Ansible16:16
VinshRight, very similar16:16
VinshWe use the puppet openstack modules.. but with containers instead of UCA packages which are always too far behind/awful.16:17
dmsimardVinsh: so basically you configure the services in the images with puppet but without installing the actual binary packages ?16:17
VinshI would say the puppet-os_docker module is fine tuned for puppet-openstack use.16:17
zigodmsimard: What tempest plugin is that? I believe I package them all but maybe I forgot some...16:17
dmsimardzigo: all of them :D16:17
zigo:)16:17
dmsimardzigo: Hard to tell if you're missing any as we're not testing deb packages yet16:17
zigodmsimard: Be specific and I'll fix...16:18
mwhahahais there a repo that we could try out a debian deploy with?16:18
dmsimardzigo: UCA is missing a couple, designate gnocchi and aodh come to mind16:18
Vinshdmsimard: We install the services from source into a container.. then use os_docker to wrap that container in a normal /usr/bin command.. then we let puppet-openstack manage the service as usual.16:18
zigodmsimard: for Ocata I'll do Ubuntu at the same time as Xenial.16:18
dmsimardVinsh: so the containers aren't read-only ?16:18
zigoToo late to add it for Newton, and it was hard enough to move to os-infra. :(16:19
Vinshdmsimard: the containers mount things like /etc/swift read only.. but mount /var/log rw16:19
Vinshfor example16:19
dmsimardVinsh: ok, interesting :16:19
dmsimard:)16:19
VinshTake advantage of the puppet modules which kick ass.. but not be bound by UCA limits.16:20
Vinshdmsimard: https://bfd-github.os.cloud.twc.net/gerrit-mirrors/puppet-os_docker/blob/master/manifests/ironic.pp#L7216:21
VinshThose rely on the hooks built into the modules..16:21
EmilienMbfd-github.os.cloud.twc.net is internal dude16:21
Vinshso set packages to ensure:absent.. then container fits in16:22
Vinshlol sorry!16:22
Vinshstill have that link up16:22
Vinshhttps://github.com/twc-openstack/puppet-os_docker/blob/master/manifests/ironic.pp#L7216:22
*** jtriley has quit IRC16:23
EmilienMdmsimard: see how it fails nicely https://review.openstack.org/#/c/380374/16:23
EmilienMso we can likely put this patch on stale16:24
EmilienMsince ubuntu packages won't contain the cinder fix because a while16:24
* EmilienM afk lunch16:25
*** priteau has quit IRC16:26
*** priteau has joined #puppet-openstack16:26
iurygregoryEmilienM, D: i didn't saw16:27
iurygregoryi was in my lunch time16:27
*** shardy has quit IRC16:28
iurygregorydmsimard,  you saw ? http://logs.openstack.org/74/380374/1/check/gate-puppet-openstack-integration-4-scenario002-tempest-centos-7/f1839d2/console.html#_2016-09-30_16_08_24_78479416:34
iurygregoryi really hate timeouts urg -.-16:34
dmsimardiurygregory: yeah16:34
dmsimardiurygregory: turns out the patch hasn't been backported to newton in time16:34
dmsimardiurygregory: it probably won't even be in the first release16:34
iurygregoryoh =(16:34
Vinshdegorenko: BTW for these https://github.com/openstack/puppet-swift/blob/master/templates/proxy-server.conf.erb#L27-L31 .. there was no config for them in the proxy class all along so I dropped them16:35
*** jpena is now known as jpena|off16:35
degorenkoVinsh, so, they are outdated in templates?16:37
VinshYeah16:37
degorenkook16:37
*** jpich has quit IRC16:40
openstackgerritAlex Schultz proposed openstack/puppet-tripleo: Only run ceilometer::db::sync on bootstrap node  https://review.openstack.org/38041416:41
openstackgerritMerged openstack/puppet-openstack-integration: Re-enable trove testing on Ubuntu  https://review.openstack.org/38033416:58
*** derekh has quit IRC16:59
*** jtriley has joined #puppet-openstack17:03
*** ducttape_ has quit IRC17:09
*** dgurtner has quit IRC17:12
*** jtriley has quit IRC17:13
*** jtriley has joined #puppet-openstack17:17
Vinshdegorenko: Hey.. consider this : https://review.openstack.org/#/c/378950/8/manifests/proxy/tempurl.pp17:17
VinshIf a user removes the tempurl class from their manifest.. the tempurl section will still be there in the config file.17:17
*** jperry has quit IRC17:18
VinshPretty much same for any of the proxy filters.17:18
degorenkoright17:18
degorenkoold configs will be kept to17:18
degorenkotoo17:18
*** morazi has joined #puppet-openstack17:18
degorenkoVinsh, so, you want just leave it as it now?17:19
VinshHow can we handle this?  Or do users just wipe away the config file and let it re-populate?17:19
Vinshall of the filters in this review would stay in the config file if their class is removed17:19
VinshI don't know of a more graceful method with ini provider17:20
amoralejmjblack, i've been testing lbaas-dashboard with newton rc2 and it really needs to be fixed17:20
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: WIP: Add magnum testing support to scenario002  https://review.openstack.org/36701217:20
amoraleji've created some bugs in LP, i'm going to wait before adding it in packstack17:20
degorenkoVinsh, hm, as one option, probably, we can add ensure parameter, for each filter, tag configs inside each filter and if absent - just remove them17:21
Vinshdegorenko: Maybe a section in proxy.pp that runs like this:  has a list of all proxy classes.. checks to see if a class is not included currently.. and if not sets an absent on them.17:21
degorenkoVinsh, not sure how you can just set absent on class. Will it apply absent on all resource? i'm not sure that it will work17:22
dmsimardcolonwq: an idea for improvement in scenario004 eventually: manila with cephfs backend ;)17:22
Vinshdegorenko: I mean it would set the swift_proxy_config "filter:/* " values to "::os:service:default"17:23
Vinshso that they are wiped out17:23
Vinsha hash of class names+filter section header names17:23
degorenkoVinsh, how you will filter needed swift_proxy_config resources?17:24
Vinshiterate on them setting to osservicedefault if not part of "pipeline" in proxy.pp17:24
openstackgerritDavid Moreau Simard proposed openstack/puppet-tempest: WIP: Add support for testing magnum  https://review.openstack.org/36700917:24
degorenkoto difficult i think17:24
degorenkotoo*17:24
VinshWe require in proxy.pp that all included classes be part of $pipeline17:24
degorenkoyeah17:24
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: WIP: Add magnum testing support to scenario002  https://review.openstack.org/36701217:24
degorenkobut you need to filter proper config resources17:24
colonwqdmsimard, ok. That is two more things I need to start figuring out.17:25
Vinshdegorenko: Yeah, I get what you mean.. i'm going to hack something up and test.  How do other modules handle this?17:25
*** ducttape_ has joined #puppet-openstack17:25
Vinshor do they all only use "DEFAULT" so are easier to manage?17:25
dmsimardEmilienM: thanks for reminding me about magnum... I would've surely forgotten about those local branches and lost a lot of work17:26
dmsimardEmilienM: I'll come back to it in a few weeks probably :p17:26
degorenkoVinsh, other modules don't have such bunch of optional 'proxies' :)17:26
EmilienMdmsimard: yeah, let's wait to stabilize things a little bit17:26
VinshO_o17:26
dmsimardEmilienM: I have no time for it right now anyway17:26
degorenkoi mean, other modules have some count of configs, which mostly all reflected in parameters17:26
degorenkoso, if user wants - he can just switch to a new value17:27
degorenkoor in some other cases - uses collectors with absent i think17:27
Vinsh#swiftproblems17:27
degorenkobut in swift case - there are a lot of config options :D17:27
degorenkowell, such change is really downside on this page17:27
Vinshwell, this is a good item to solve for this review. good for you to bring up17:28
degorenkoold format everytime creates a new config with up to date parameters17:28
*** paramite has quit IRC17:28
degorenkoVinsh, may be other guys have other ideas :)17:28
VinshDo you know of an example on the collector method?17:28
degorenkoVinsh, collect ?17:29
degorenkolike this http://ruby-doc.org/core-2.2.0/Array.html#method-i-collect ?17:29
Vinshfor _config resources and absent17:29
VinshI mean it's use in this way in a module already17:29
degorenkoaaah17:29
degorenkosoo17:30
VinshI don't know how I would collect and not affect resources we want to keep17:30
degorenkoVinsh, http://paste.openstack.org/show/583651/17:30
degorenkotry this17:30
Vinshmmm, so set the tag different for included clases17:31
*** amoralej is now known as amoralej|off17:31
degorenkoVinsh, so, the idea is tag every filter config option with uniq tag :)17:31
VinshBUT, if a class isn't included.. how will it even be collected?17:31
degorenkoVinsh, no where :(17:31
Vinshheh :)17:31
degorenkoand it is downside17:32
degorenkowhich i told above :)17:32
degorenkoand i think we can't do anthing in such case17:32
Vinshok, so then just go through a list of filter sections that are not part of $pipeline and set to absent17:32
degorenkoit like, include in first run authtoken or rabbitmq - and then include some other amqp or auth without disabling old17:32
VinshTime to hack/test.17:33
degorenkoVinsh, yeah :)17:33
*** jperry has joined #puppet-openstack17:33
EmilienMmwhahaha, degorenko: can we land https://review.openstack.org/#/c/380390/ ?17:35
EmilienMI'll revert it during the weekend17:35
EmilienMpackage fix is in their pipe now17:35
degorenko+217:35
EmilienMiurygregory: also if you're around ^17:36
*** kylo has joined #puppet-openstack17:36
iurygregoryi'm17:36
iurygregorybut is already +A17:37
iurygregoryXD17:37
degorenkoyou're late17:37
degorenko:)17:37
iurygregoryi'm slow lol17:37
EmilienMthanks folks17:38
degorenkomwhahaha, is there around some plan for ocata ? may be :D17:38
EmilienMI'm even going to backport since it also breaks stable/newton CI17:38
iurygregorydegorenko, did you saw the spreedsheat?17:39
degorenkoiurygregory, nope17:39
degorenkowhere is it?17:39
iurygregorydegorenko, https://docs.google.com/spreadsheets/d/1GV4v-T8E7Sq6GDE12pcTk_i0hFaVrDGnIDDVikXrQ34/edit#gid=017:39
degorenkooh, nice, thanks iurygregory17:39
mwhahahayea the plan is mostly cleanup17:39
mwhahahaand getting everything consistent17:39
degorenkosounds good :)17:40
mwhahahaalso giving some attention to some of the other modules/projects and seeing where they are in packaging, etc17:40
degorenkoyeah we had many new modules during newton17:41
EmilienMsounds good too :)17:41
mwhahahaanyway afk lunch17:41
mwhahahabrb in a bit17:41
openstackgerritMerged openstack/puppet-swift: Fixed swift::proxy::cache requirement on memcached  https://review.openstack.org/38037017:42
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Disable Telemetry testing on Ubuntu  https://review.openstack.org/38045817:43
degorenkoi'm off :) have a nice weekend guys!17:43
*** degorenko is now known as _degorenko|afk17:43
VinshParty on!17:43
*** trown is now known as trown|lunch17:43
iurygregorybeer on!17:43
EmilienMwell, I'm also off for 2h. ttyl17:43
iurygregoryflight leasons ^17:44
openstackgerritMerged openstack/puppet-tripleo: Update .gitreview for stable/newton  https://review.openstack.org/37959917:45
EmilienMiurygregory: no, just walking in fresh air17:46
iurygregorygood for you \o/17:47
*** ayoung has quit IRC17:54
*** ayoung has joined #puppet-openstack18:01
openstackgerritKeith Schincke proposed openstack/puppet-openstack-integration: Add cinder::backup test to scenario0001  https://review.openstack.org/37715818:04
*** ccamacho has quit IRC18:08
dmsimardcolonwq: that should do it !18:09
colonwqdmsimard, I hope. we will see. CI is the final authority and she is fickle.18:11
openstackgerritMerged openstack/puppet-openstack-integration: Disable Telemetry testing on Ubuntu  https://review.openstack.org/38039018:19
openstackgerritMerged openstack/puppet-openstack_extras: repos: configure RDO Virtualization repository  https://review.openstack.org/38034718:19
*** imcsk8 has quit IRC18:34
*** trown|lunch is now known as trown18:47
openstackgerritAlex Schultz proposed openstack/puppet-modulesync-configs: Use openstack spec helper facts by default  https://review.openstack.org/38048818:53
openstackgerritayoung proposed openstack/puppet-keystone: Tune number of apache processes/threads for Keystone in apache.  https://review.openstack.org/29734218:54
openstackgerritMerged openstack/puppet-tripleo: telemetry: normalize coordination_url  https://review.openstack.org/38030618:54
openstackgerritMerged openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/38033118:55
openstackgerritAlex Schultz proposed openstack/puppet-modulesync-configs: Update managed module list  https://review.openstack.org/38049218:58
*** johnmilton has quit IRC18:59
Vinsh_degorenko|afk: if you look back here later.. I found 3 lines that solve our problem.19:01
Vinshresources { 'swift_proxy_config':19:01
Vinsh  purge => true,19:01
Vinsh}19:01
VinshWill purge the sections under any unmanaged resource in the config file.19:02
mwhahahawe usually make that configurable19:07
VinshOk, good call. I'll make it an option to the proxy class19:11
VinshMakes sense, maybe a user has some add-ins that are custom they don't want wiped out.19:11
iurygregorymwhahaha, in 380488 we can really remove puppet-openstack_spec_helper/defaults ?19:11
mwhahahayea19:11
mwhahahait's included in /facts19:11
iurygregoryoh nice =)19:12
mwhahahahttps://github.com/openstack/puppet-openstack_spec_helper/blob/master/lib/puppet-openstack_spec_helper/facts.rb#L219:12
iurygregorytks19:13
openstackgerritAlex Schultz proposed openstack/puppet-barbican: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38050719:16
openstackgerritAlex Schultz proposed openstack/puppet-ceilometer: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38050819:16
* iurygregory hides spam time19:16
mwhahaha:D19:16
openstackgerritAlex Schultz proposed openstack/puppet-cinder: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38051019:17
openstackgerritAlex Schultz proposed openstack/puppet-congress: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38051119:17
openstackgerritAlex Schultz proposed openstack/puppet-designate: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38051219:17
*** johnmilton has joined #puppet-openstack19:17
openstackgerritAlex Schultz proposed openstack/puppet-ec2api: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38051319:18
*** openstackgerrit has quit IRC19:18
*** openstackgerrit has joined #puppet-openstack19:19
openstackgerritAlex Schultz proposed openstack/puppet-gnocchi: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38051619:19
openstackgerritAlex Schultz proposed openstack/puppet-heat: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38051719:19
mwhahahahmm glance/glare got dropped19:19
iurygregoryo.o19:20
openstackgerritAlex Schultz proposed openstack/puppet-horizon: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38051919:20
openstackgerritAlex Schultz proposed openstack/puppet-ironic: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38052019:20
openstackgerritAlex Schultz proposed openstack/puppet-keystone: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38052219:21
*** cdelatte has quit IRC19:22
openstackgerritAlex Schultz proposed openstack/puppet-magnum: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38052319:22
openstackgerritAlex Schultz proposed openstack/puppet-manila: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38052419:22
*** ayoung has quit IRC19:22
openstackgerritAlex Schultz proposed openstack/puppet-mistral: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38052519:23
openstackgerritAlex Schultz proposed openstack/puppet-murano: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38052619:23
openstackgerritAlex Schultz proposed openstack/puppet-neutron: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38052719:23
openstackgerritAlex Schultz proposed openstack/puppet-nova: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38052819:24
openstackgerritAlex Schultz proposed openstack/puppet-octavia: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38052919:24
openstackgerritAlex Schultz proposed openstack/puppet-openstack_extras: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38053019:24
openstackgerritAlex Schultz proposed openstack/puppet-openstacklib: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38053119:25
openstackgerritAlex Schultz proposed openstack/puppet-oslo: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38053219:27
openstackgerritAlex Schultz proposed openstack/puppet-ovn: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38053319:28
openstackgerritAlex Schultz proposed openstack/puppet-panko: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38053519:28
openstackgerritAlex Schultz proposed openstack/puppet-rally: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38053619:29
openstackgerritAlex Schultz proposed openstack/puppet-rally: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38053619:29
openstackgerritAlex Schultz proposed openstack/puppet-sahara: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38053719:30
openstackgerritAlex Schultz proposed openstack/puppet-swift: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38053819:31
*** cdelatte has joined #puppet-openstack19:32
openstackgerritAlex Schultz proposed openstack/puppet-tacker: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38053919:32
openstackgerritAlex Schultz proposed openstack/puppet-tempest: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38054019:32
openstackgerritAlex Schultz proposed openstack/puppet-trove: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38054119:33
*** ayoung has joined #puppet-openstack19:33
openstackgerritAlex Schultz proposed openstack/puppet-vitrage: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38054419:34
openstackgerritAlex Schultz proposed openstack/puppet-vswitch: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38054519:34
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: telemetry: normalize coordination_url  https://review.openstack.org/38054719:34
openstackgerritAlex Schultz proposed openstack/puppet-watcher: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38054819:34
EmilienMfresh air was awesome19:35
*** dfisher has joined #puppet-openstack19:35
openstackgerritAlex Schultz proposed openstack/puppet-zaqar: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38054919:35
openstackgerritAlex Schultz proposed openstack/puppet-openstack_spec_helper: Fix @default_facts typo  https://review.openstack.org/38055019:40
openstackgerritAlex Schultz proposed openstack/puppet-openstack_spec_helper: Fix @default_facts typo  https://review.openstack.org/38055019:40
dmsimardEmilienM: Montreal has none of that :)19:47
dmsimardEmilienM: Gaspésie on the other hand... so awesome19:47
*** paramite has joined #puppet-openstack19:47
EmilienMdmsimard: Quebec City is not too bad19:52
openstackgerritMerged openstack/puppet-aodh: Move rspec-puppet-facts to spec helper  https://review.openstack.org/37516120:03
openstackgerritAlex Schultz proposed openstack/puppet-nova: Move rspec-puppet-facts to spec helper  https://review.openstack.org/38052820:03
openstackgerritMerged openstack/puppet-openstack-integration: Disable Telemetry testing on Ubuntu  https://review.openstack.org/38045820:06
* iurygregory back end of spam time =P20:10
mwhahahafor now :D20:10
* iurygregory hides again hahaha20:11
mwhahahai'm done for today :D need https://review.openstack.org/#/c/380550/ first anyway20:11
iurygregoryjust waiting for CI20:11
* iurygregory watching zuul20:12
mwhahahathe one bad thing about using the rspec helper like this is that it doesn't get tested on a depends on20:12
iurygregoryoh =( i was wondering why all other patches does not have the Depends On20:13
*** dprince has quit IRC20:13
openstackgerritMerged openstack/puppet-modulesync-configs: Use openstack spec helper facts by default  https://review.openstack.org/38048820:23
EmilienMmwhahaha: +A to save time20:25
EmilienMyeah gemfile can't git checkout using zuul cloner20:25
iurygregoryzuul "0 min" and jenkis does not give +1 lol20:26
*** xarses has joined #puppet-openstack20:28
*** xarses has quit IRC20:29
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Revert "Disable Telemetry testing on Ubuntu"  https://review.openstack.org/38063820:29
*** xarses has joined #puppet-openstack20:29
iurygregorywhat? '-'20:29
iurygregoryit's working now?20:29
*** jtriley has quit IRC20:30
*** jayg is now known as jayg|g0n320:32
EmilienMit should20:32
iurygregoryawesome =)20:33
openstackgerritMerged openstack/puppet-openstack_spec_helper: Fix @default_facts typo  https://review.openstack.org/38055020:51
*** fultonj has quit IRC20:59
*** cdelatte has quit IRC21:04
openstackgerritAlex Schultz proposed openstack/puppet-tripleo: Add aodh profile rspec testing  https://review.openstack.org/37440221:12
*** trown is now known as trown|outtypewww21:12
openstackgerritAlex Schultz proposed openstack/puppet-tripleo: Add ceilometer profile rspec testing  https://review.openstack.org/38040621:12
openstackgerritAlex Schultz proposed openstack/puppet-tripleo: Only run ceilometer::db::sync on bootstrap node  https://review.openstack.org/38041421:12
mwhahahaoh look when you don't have typos it works :D21:18
dmsimardor invisible breaking whitespaces21:25
* dmsimard hates invisible breaking whitespace21:25
*** ducttape_ has quit IRC21:36
*** jtomasek has quit IRC21:50
*** priteau has quit IRC21:51
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Remove unused pacemaker profiles  https://review.openstack.org/37995721:57
*** jperry has quit IRC22:01
*** morazi has quit IRC22:05
openstackgerritMerged openstack/puppet-cinder: Deprecate backup_topic option  https://review.openstack.org/37620722:13
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Fix the timeout for pacemaker systemd resources  https://review.openstack.org/38066522:18
*** dfisher has quit IRC22:24
openstackgerritMerged openstack/puppet-keystone: Resolve OpenID Connect Integration issues  https://review.openstack.org/37199422:31
*** ducttape_ has joined #puppet-openstack22:42
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Change rabbitmq queues HA mode from ha-all to ha-exactly  https://review.openstack.org/37958622:52
*** iurygregory_ has joined #puppet-openstack23:31
*** xarses has quit IRC23:49
*** furlongm has quit IRC23:50
*** furlongm has joined #puppet-openstack23:52
« Thursday, 2016-09-29 Index Saturday, 2016-10-01 »

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