Tuesday, 2016-08-02

*** citizen482 has joined #puppet-openstack00:14
*** foxhunt72 is now known as foxhunt_away00:16
*** jesusaur has joined #puppet-openstack00:16
*** citizen482 has left #puppet-openstack00:18
*** cdelatte has quit IRC00:23
*** jesusaur has quit IRC00:23
*** xarses has joined #puppet-openstack00:27
*** fxpester has joined #puppet-openstack00:30
*** saneax is now known as saneax_AFK00:31
*** jesusaur has joined #puppet-openstack00:50
*** jesusaur has quit IRC00:57
*** jesusaur has joined #puppet-openstack01:00
*** fxpester has quit IRC01:04
*** chem has quit IRC01:19
*** chem has joined #puppet-openstack01:19
*** fultonj has quit IRC01:23
*** iurygregory_ has joined #puppet-openstack01:28
*** dhollinger has quit IRC01:28
*** dhollinger has joined #puppet-openstack01:39
*** ilbot3 has quit IRC01:46
*** ilbot3 has joined #puppet-openstack01:48
*** chem has quit IRC01:52
*** chem has joined #puppet-openstack01:53
*** johnmilton has quit IRC01:57
*** johnmilton has joined #puppet-openstack02:12
openstackgerritKeith Schincke proposed openstack/puppet-openstack-integration: WIP - Introduce scenario004  https://review.openstack.org/33910602:13
*** markvoelker has joined #puppet-openstack02:25
*** foxhunt_away is now known as foxhunt7202:29
*** markvoelker_ has joined #puppet-openstack02:30
*** markvoelker has quit IRC02:30
*** foxhunt72 is now known as foxhunt_away02:47
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: Configure keystone authtoken options  https://review.openstack.org/34290503:09
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: neutron: add granularity for aas services  https://review.openstack.org/34976503:14
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: tempest: only deploy neutron fwaas and lbaas  https://review.openstack.org/34976603:16
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/34915503:17
*** iurygregory_ has quit IRC03:19
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-nova: Move nova to authtoken  https://review.openstack.org/33738903:34
*** ducttape_ has joined #puppet-openstack03:56
*** ducttape_ has quit IRC04:10
*** yottatsa has joined #puppet-openstack04:15
*** markvoelker_ has quit IRC04:15
*** saneax_AFK is now known as saneax04:28
*** crinkle has quit IRC04:35
*** crinkle has joined #puppet-openstack04:35
*** yottatsa has quit IRC05:07
*** markvoelker has joined #puppet-openstack05:10
*** ducttape_ has joined #puppet-openstack05:11
*** yottatsa has joined #puppet-openstack05:11
*** yottatsa_ has joined #puppet-openstack05:14
*** ducttape_ has quit IRC05:16
*** markvoelker has quit IRC05:16
*** yottatsa has quit IRC05:17
*** jaosorior has joined #puppet-openstack05:19
*** chem has quit IRC05:35
*** richm has quit IRC05:40
*** maestropandy has joined #puppet-openstack05:53
openstackgerritguessi proposed openstack/puppet-nova: Update virsh secret if rbd_keyring or libvirt_rbd_secret_uuid change  https://review.openstack.org/23236905:57
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Remove unnecessary certmonger manifest  https://review.openstack.org/34981206:06
*** markvoelker has joined #puppet-openstack06:06
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/34915506:11
*** ducttape_ has joined #puppet-openstack06:12
*** markvoelker has quit IRC06:13
*** ducttape_ has quit IRC06:17
*** fvollero has quit IRC06:34
*** fvollero has joined #puppet-openstack06:34
*** moravec has quit IRC06:42
*** tesseract- has joined #puppet-openstack06:43
*** leanderthal|afk is now known as leanderthal06:47
*** yottatsa_ has quit IRC06:57
*** markvoelker has joined #puppet-openstack07:02
*** jpena|off is now known as jpena07:07
*** markvoelker has quit IRC07:08
*** fxpester has joined #puppet-openstack07:11
*** ducttape_ has joined #puppet-openstack07:13
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Fix mistral profile steps  https://review.openstack.org/34334607:16
*** ducttape_ has quit IRC07:18
*** shardy has joined #puppet-openstack07:30
*** aarefiev has joined #puppet-openstack07:43
*** markvoelker has joined #puppet-openstack07:58
*** dgurtner has joined #puppet-openstack08:04
*** markvoelker has quit IRC08:04
*** hecliunyx has quit IRC08:07
*** foxhunt_away is now known as foxhunt7208:08
*** hecliunyx has joined #puppet-openstack08:09
*** jaosorior has quit IRC08:11
*** jaosorior has joined #puppet-openstack08:12
*** vinsh has quit IRC08:14
*** vinsh has joined #puppet-openstack08:15
*** kgalanov has quit IRC08:16
*** jmiu_ is now known as jmiu08:16
*** v1k0d3n has quit IRC08:17
*** mfisch has quit IRC08:17
*** jaosorior_ has joined #puppet-openstack08:18
*** v1k0d3n has joined #puppet-openstack08:18
*** kgalanov has joined #puppet-openstack08:18
*** chem has joined #puppet-openstack08:18
*** karthiks has quit IRC08:19
*** karthiks has joined #puppet-openstack08:20
*** jaosorior has quit IRC08:21
*** mfisch has joined #puppet-openstack08:22
*** mfisch has quit IRC08:22
*** mfisch has joined #puppet-openstack08:22
*** dtantsur|afk is now known as dtantsur08:22
*** danpawlik has joined #puppet-openstack08:29
*** ntpttr has quit IRC08:38
*** ntpttr has joined #puppet-openstack08:38
openstackgerritKarthik S proposed openstack/puppet-neutron: Configure the max_vfs for SRIOV interfaces  https://review.openstack.org/34491808:50
*** gfidente has joined #puppet-openstack08:53
*** gfidente has quit IRC08:53
*** gfidente has joined #puppet-openstack08:53
*** markvoelker has joined #puppet-openstack08:54
*** snecklifter has quit IRC08:55
*** skolekonov has joined #puppet-openstack08:59
*** markvoelker has quit IRC09:00
*** yottatsa has joined #puppet-openstack09:06
*** gchamoul has joined #puppet-openstack09:14
*** ducttape_ has joined #puppet-openstack09:15
*** jaosorior_ is now known as jaosorior09:16
*** ducttape_ has quit IRC09:19
openstackgerritCarlos Camacho proposed openstack/puppet-glance: Add missing db class to configure glance-manage.conf  https://review.openstack.org/34946309:24
*** aderyugin has joined #puppet-openstack09:28
*** paramite has joined #puppet-openstack09:33
*** markvoelker has joined #puppet-openstack09:49
*** markvoelker has quit IRC09:56
*** richm has joined #puppet-openstack10:08
*** chem has quit IRC10:20
*** _degorenko|afk is now known as degorenko10:20
*** cgfbee has joined #puppet-openstack10:22
*** dtantsur is now known as dtantsur|lunch10:28
*** ntpttr- has quit IRC10:30
*** johnmilton has quit IRC10:33
*** fxpester has quit IRC10:38
*** ntpttr- has joined #puppet-openstack10:39
*** markvoelker has joined #puppet-openstack10:45
*** johnmilton has joined #puppet-openstack10:47
*** markvoelker has quit IRC10:51
*** fxpester has joined #puppet-openstack10:52
*** dtantsur|lunch is now known as dtantsur11:10
*** shardy is now known as shardy_lunch11:14
*** ducttape_ has joined #puppet-openstack11:16
*** ducttape_ has quit IRC11:21
*** guessi has quit IRC11:23
openstackgerritDenis Egorenko proposed openstack/puppet-glance: Move Glance to new authtoken scheme  https://review.openstack.org/34882611:37
*** markvoelker has joined #puppet-openstack11:39
*** paramite is now known as paramite|afk11:40
*** johnmilton has quit IRC11:41
*** bklei has quit IRC11:41
*** markvoelker_ has joined #puppet-openstack11:44
*** markvoelker has quit IRC11:44
EmilienMhello11:48
degorenkohi11:49
*** dprince has joined #puppet-openstack11:55
*** fultonj has joined #puppet-openstack12:02
iurygregorymorning people12:04
*** jpena is now known as jpena|lunch12:04
*** shardy_lunch is now known as shardy12:05
*** ducttape_ has joined #puppet-openstack12:08
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: neutron: add granularity for aas services  https://review.openstack.org/34976512:12
*** paramite|afk is now known as paramite12:13
skramajaEmilienM: gate-puppet-vswitch-puppet-unit-3.8-centos-7 is not able to pull the kmod changes where as other gates are able to pull - https://review.openstack.org/#/c/347464/12:15
skramajais there any issue on CI?12:16
EmilienMwhat says logs?12:16
skramaja2016-08-01 16:59:42.840422 |      Puppet::Error:12:16
skramaja2016-08-01 16:59:42.840513 |        Puppet::Parser::AST::Resource failed with error ArgumentError: Invalid resource type kmod::load at /home/jenkins/workspace/gate-puppet-vswitch-puppet-unit-3.8-centos-7/spec/fixtures/modules/vswitch/manifests/dpdk.pp:38 on node centos-7-rax-dfw-306972612:16
skramaja2016-08-01 16:59:42.840556 |      Shared Example Group: "vswitch dpdk" called from ./spec/classes/vswitch_dpdk_spec.rb:10412:16
skramajaEmilienM: the new module is not loaded12:16
*** fxpester|2 has joined #puppet-openstack12:21
*** fxpester has quit IRC12:25
*** johnmilton has joined #puppet-openstack12:25
*** ducttape_ has quit IRC12:26
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/34915512:29
*** jayg|g0n3 is now known as jayg12:46
iurygregoryrichm, you around?12:50
*** maestropandy has left #puppet-openstack12:53
*** dmsimard|afk is now known as dmsimard12:55
EmilienMdegorenko: I think scenario002 is broken12:57
EmilienMthere is a regression in Zaqar12:57
degorenkolooks like yeah12:57
*** morazi has joined #puppet-openstack12:58
EmilienMI reported it on zaqar channel12:58
*** jpena|lunch is now known as jpena13:00
skramajaEmilienM: looks like the error is " Invalid resource type kmod::load" but it happens only for 3.8 .. other gates are able to load this new module successfully..13:00
EmilienMdegorenko: maybe in tempest, https://github.com/openstack/tempest/commit/11d8cd0fbc0aafc4fc410887f8f3e55c799e2a3513:01
EmilienMI'm investigating13:01
EmilienMit will be fixed by https://review.openstack.org/#/c/349809/13:01
EmilienM:)13:01
EmilienMskramaja: I'll look after my current task13:01
degorenkoEmilienM, good to hear that :)13:02
EmilienMdegorenko: FYI https://review.rdoproject.org/r/#/c/1775/ is merged13:02
EmilienMso we'll be able to start murano testing soon13:02
degorenko\o/13:03
degorenkogood news13:03
*** dgurtner has quit IRC13:05
*** ericksonsantos has quit IRC13:09
*** iurygregory has quit IRC13:09
*** saneax is now known as saneax_AFK13:09
*** mkoderer_cloud has joined #puppet-openstack13:11
*** iurygregory has joined #puppet-openstack13:11
*** ericksonsantos has joined #puppet-openstack13:12
*** ducttape_ has joined #puppet-openstack13:12
*** bklei has joined #puppet-openstack13:21
*** markvoelker_ has quit IRC13:23
*** ducttape_ has quit IRC13:29
*** morazi has quit IRC13:31
*** morazi has joined #puppet-openstack13:33
*** dgurtner has joined #puppet-openstack13:34
*** dgurtner has quit IRC13:34
*** dgurtner has joined #puppet-openstack13:34
*** vinsh has quit IRC13:34
richmiurygregory: yes13:40
*** cdelatte has joined #puppet-openstack13:43
*** fultonj has quit IRC13:48
*** fultonj has joined #puppet-openstack13:49
*** fultonj has quit IRC13:50
iurygregoryrichm, maybe the problem http://logs.openstack.org/89/337389/23/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/b454803/logs/keystone/keystone.txt.gz#_2016-08-02_03_56_15_928 is related to nova be using v2 in https://github.com/openstack/puppet-nova/search?utf8=%E2%9C%93&q=Puppet%3A%3AProvider%3A%3AOpenstack%3A%3ACredentialsV2_0.new ?13:53
*** fultonj has joined #puppet-openstack13:54
*** fultonj has quit IRC13:56
*** ducttape_ has joined #puppet-openstack13:56
*** fultonj has joined #puppet-openstack13:56
*** fultonj has quit IRC13:56
*** fultonj has joined #puppet-openstack13:56
*** xarses has quit IRC13:56
richmiurygregory: maybe, but afaik keystone should be able to handle both v2 and v3 authentication - you use the v2 endpoint with v2 tokens, and use the v3 endpoint with v3 tokens13:57
*** ebalduf has joined #puppet-openstack13:58
iurygregoryyeah i know .-.13:58
*** dtantsur is now known as dtantsur|mtg13:58
iurygregoryI'm out of ideas of how to solve the problem .-. damm it13:58
*** saneax_AFK is now known as saneax13:59
gfidenteguys, anybody interested in ceph willing to review https://review.openstack.org/#/c/347860/ and https://review.openstack.org/#/c/232369/ ?14:00
richmiurygregory: we need help from someone with more recent keystone experience who can figure out how to enable logging of the url and token contents14:00
iurygregoryhaha yeah XD14:01
gfidentethose are meant to ensure we refresh the keyrings when the cephx keys or key caps are updated14:01
*** markvoelker has joined #puppet-openstack14:02
richmiurygregory: For starters, we need to know 1) what is the URL of the request that is causing the error 2) what is the contents of the value that is causing the ValueError 3) what is the IP address or hostname of the peer14:03
richmiurygregory: looks like keystone is already running with DEBUG so I'm not sure how to get that information14:04
richmiurygregory: in addition - the stack trace appears truncated - if this is coming from keystone I would expect to see a much larger stack trace with many more frames14:05
iurygregoryrichm, ack i'll try to look14:05
*** dfisher has joined #puppet-openstack14:15
*** vinsh has joined #puppet-openstack14:19
*** richm has quit IRC14:19
openstackgerritcraig delatte proposed openstack/puppet-cinder: Cinder hooks support  https://review.openstack.org/34446414:19
*** saneax is now known as saneax_AFK14:31
*** dgurtner has quit IRC14:34
EmilienMdegorenko: when you ready https://review.openstack.org/34290514:36
*** yottatsa has quit IRC14:37
degorenkoEmilienM, boom14:39
EmilienMcool14:39
degorenkoEmilienM, can you take a look: https://review.openstack.org/348826 ?14:39
EmilienMsure14:39
degorenkomay be i missed something14:39
*** xarses has joined #puppet-openstack14:39
EmilienMok It's big, i'll look today14:40
degorenkoEmilienM, yeah :( 3 services here14:40
degorenkoiurygregory, can you take a look too? :) ^^^14:40
iurygregorydegorenko, ack14:40
mwhahahajust an fyi, i'm on pto starting tomorrow for a week. so on the off chance anyone cares, i won't be around :D14:40
EmilienMmwhahaha: you lucky14:40
iurygregorydegorenko, EmilienM we should revert ec2api, designate and zaqar or just patch again?14:41
mwhahahajust patch again14:41
degorenkomwhahaha, if you will be on pto, who is going to +2 me on fuel now? :(14:41
degorenko:D14:41
EmilienMyeah patch again14:41
iurygregoryack ;)14:41
mwhahahadegorenko: no one else evidently :D14:41
iurygregoryanyone can help me with nova? D:14:41
mwhahahasoo many open patches14:42
EmilienMwe should take PTO all in same time14:42
EmilienMmy next PTO are November, 2 weeks :P14:42
*** saneax_AFK is now known as saneax14:50
colonwqmy next PTO is tomorrow. :)14:51
dfisherwhat's PTO?14:53
*** jmiu has quit IRC14:53
dfisher:(14:53
mwhahahathat thing we so rarely take :(14:53
*** jmiu has joined #puppet-openstack14:53
* dfisher sad high-five14:53
iurygregorydegorenko, it's not possible to have glance::keystone::authtoken and define the 3 resources inside ?14:53
*** hjensas has quit IRC14:54
mwhahahayou could but is there ever a case where you'd want to configure the 3 services differently?14:54
iurygregorymwhahaha, make sense =)14:55
iurygregorybut i don't know if anyone have this use case XD14:55
mwhahahayea me neither. i think it might be better to do all three at once then you're less likely to have accidentally misconfigured services14:57
EmilienMthere is no item for today's agenda https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-2016080214:57
EmilienMfeel free to add some or I'll cancel the meeting and let people working14:58
* iurygregory waits before send the meeting patch14:58
iurygregorymwhahaha, yeah the user can make more errors while need to set 3 classes instead of one..14:59
iurygregorydegorenko, EmilienM what you guys think?14:59
* iurygregory lunch time in 2 hours i'm back15:04
EmilienMdegorenko: reviewed15:05
*** richm has joined #puppet-openstack15:05
EmilienMiurygregory: you can cancel it15:05
mfischEmilienM: will you or other puppeteers be at the operators meetup?15:06
EmilienMmfisch: I won't be15:06
degorenkoiurygregory, in case of using one class, we need to manage all deprecated parameters and services in one class. Is it good?15:06
EmilienMmfisch: and I don't know for others15:07
EmilienMmfisch: in BNC?15:07
EmilienMerr, BCN15:07
mfischNYC15:07
EmilienMnope15:07
mfischlast week of Aug15:07
mfischI won't be in Barcelona, nobody from our team is going15:07
*** dtantsur|mtg is now known as dtantsur15:08
EmilienMwhen i read this thing it's always the same topics15:08
EmilienM"Config management - puppet, chef"15:08
EmilienMblabla15:08
mfischlike arguing with a spouse15:08
EmilienM"How to lay this out? What do we want exactly?"15:08
mfischexcept in NYC and with more alcohol15:08
mfischwhether or not anything productive comes out of the sessions is another question15:09
bkeroand probably less regrets15:09
mfischbkero: lol15:09
EmilienMmfisch: I dropped a comment in the etherpad15:09
EmilienMand I can't go in NYC for sure, I'll burn somewhere in US on a secrete beach15:10
mfischwe can get you anywhere, is there oil near you possibly?15:10
EmilienMlol15:10
mfischwe can help democracy there if there is oil15:10
mfischwhere is your comment?15:11
EmilienMmfisch: L4615:13
mfischyep15:13
mfischgood answer15:13
mfischfollowup15:14
mfischL4715:14
*** ebalduf has quit IRC15:15
EmilienMmwhahaha: https://review.openstack.org/#/c/346494/15:17
EmilienMmwhahaha: the first time I see that15:17
EmilienMmfisch: good15:17
*** moravec has joined #puppet-openstack15:18
*** mkoderer_cloud has quit IRC15:18
*** ebalduf has joined #puppet-openstack15:20
karthiksEmilienM,  for review  https://review.openstack.org/#/c/330544,  can I keep the deprecation message like15:34
karthiks# [*sriov_agent_required*]15:34
karthiks#   (optional) Deprecated. Its enabled when mechanism_drivers has sriovnicswitch15:34
karthiks#   Defaults to undef.15:34
karthiks#15:34
*** foxhunt72 is now known as foxhunt_away15:35
EmilienMmwhahaha, degorenko: our CI is blocked by zaqar, i'm going to disable it, promote and re-enable it eventually15:36
EmilienMkarthiks: did you see my comment?15:38
EmilienMjust copy paste15:38
karthiksYes .... Thanks15:38
openstackgerritKarthik S proposed openstack/puppet-neutron: Cleanup for sriov-agent sriov-agent-required  https://review.openstack.org/33054415:39
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: disable zaqar  https://review.openstack.org/35011815:40
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: neutron: add granularity for aas services  https://review.openstack.org/34976515:41
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: tempest: only deploy neutron fwaas and lbaas  https://review.openstack.org/34976615:42
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/34915515:42
*** karthiks is now known as karthiks|afk15:43
iurygregorydegorenko, well i think is better than 3 new class .-.15:43
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/34915515:43
iurygregoryEmilienM, what do think about the idea in glance to just have one class and configure the 3 services?15:44
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Revert "scenario002: disable zaqar"  https://review.openstack.org/35011915:44
*** dtantsur is now known as dtantsur|afk15:45
*** danpawlik has quit IRC15:50
*** bklei_ has joined #puppet-openstack15:51
*** bklei has quit IRC15:52
*** jaosorior has quit IRC15:52
*** leanderthal is now known as leanderthal|afk15:53
*** jaosorior has joined #puppet-openstack15:54
*** jaosorior has quit IRC16:06
colonwqEmilienM, I have 347956 working with Keystone V3. I have 339106 working upto the glance integration part. Both need a little polishing.16:07
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-barbican: Move barbican to authtoken  https://review.openstack.org/34710516:20
iurygregorymwhahaha, you will have time to patch designate again?16:27
*** kragniz has joined #puppet-openstack16:28
openstackgerritPradeep Kilambi proposed openstack/puppet-tripleo: Implement aodh profiles  https://review.openstack.org/33285416:28
*** rski has joined #puppet-openstack16:29
mwhahahaiurygregory: yea i can16:42
EmilienMiurygregory: yeah maybe for glance16:43
iurygregorymwhahaha, ack16:43
iurygregoryEmilienM, ok o/16:44
_odyI think p-o-i or really anything that uses r10k on EL is going to break.16:46
_odyhttps://github.com/grosser/fast_gettext/commits/master16:46
*** fxpester|2 has quit IRC16:46
* EmilienM lunch16:47
*** saneax is now known as saneax_AFK16:49
*** bklei_ has quit IRC16:50
*** guimaluf has quit IRC16:51
*** tesseract- has quit IRC16:51
*** bklei has joined #puppet-openstack16:52
*** jpena is now known as jpena|off16:59
*** paramite has quit IRC17:08
*** skolekonov has quit IRC17:09
*** crinkle has quit IRC17:17
*** foxhunt_away is now known as foxhunt7217:18
*** gfidente has quit IRC17:21
openstackgerritCody Herriges proposed openstack/puppet-openstack-integration: scenario002: enable Barbican and Volume Encryption testing  https://review.openstack.org/33980617:31
EmilienM_ody: why?17:33
*** maestropandy has joined #puppet-openstack17:36
_odyIt was premature, I've gotten confused trying to track down a parameter issue.17:38
*** maestropandy has left #puppet-openstack17:43
_odyEmilienM: Feeling just a bit crazy.  In the change here, https://review.openstack.org/#/c/339806/46/manifests/cinder.pp the cinder::api class is declared with parameters I don't see available in the dependent cinder change, https://review.openstack.org/#/c/342905/17/manifests/api.pp.17:44
EmilienM_ody: right, I was about to update it, the puppet-cinder patch was WIP17:47
_odyok.  I trying to track down this change's puppet 4 issue you pinged me about the other day.  I'll hold off then.17:48
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: enable Barbican and Volume Encryption testing  https://review.openstack.org/33980617:50
iurygregoryCI is crazy? O.o17:51
EmilienMiurygregory: why?17:51
iurygregoryhttp://zuul.openstack.org/ 34710517:52
iurygregoryi've run everything locally O.o17:52
bkeroiurygregory: I think he's talking about https://s-media-cache-ak0.pinimg.com/236x/54/5c/d9/545cd91d77536585081d03d606b54fc4.jpg17:52
iurygregorybkero, lol17:53
iurygregoryEmilienM, i hope is just could not find repo or something17:54
EmilienM_ody: you are right, something breaks17:58
EmilienM_ody: dmsimard found something in RDO CI17:59
_odyEmilienM: you mean r10k?17:59
dmsimard_ody: yeah we started seeing failures to install r10k today in our CI17:59
EmilienM_ody: yes17:59
dmsimardhttp://paste.openstack.org/show/545707/17:59
dmsimardWe did not have this error yesterday17:59
_odyTemporarily we can do a "gem install fast_gettext -v 1.1.0" before install r10k.18:00
*** crinkle has joined #puppet-openstack18:00
*** johnmilton has quit IRC18:00
*** EmilienM sets mode: +o crinkle18:00
*** EmilienM sets mode: +o degorenko18:00
_odyI'll dig into ticket trackers.18:00
*** EmilienM sets mode: +o iberezovskiy18:00
*** iberezovskiy has quit IRC18:00
*** EmilienM sets mode: +o _ody18:00
*** EmilienM sets mode: +o mfisch18:00
dmsimard_ody: I'll try and reproduce upstream with and without fast_gettext18:00
EmilienMI haven't seen it in our CI yet18:01
openstackgerritMerged openstack/puppet-openstack-guide: Move #89 to August 9  https://review.openstack.org/35008318:01
dmsimardEmilienM: maybe the version of ruby is more up to date or something ?18:02
bkeroor deep deps changed18:02
bkero(changed constraints)18:02
_odyEither infra updates ruby on EL images or gems are now mirrored and mirror isn't updated yet.18:03
EmilienMdmsimard: you running centos7, right?18:03
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Test r10k installation without fast_gettext  https://review.openstack.org/35018918:03
dmsimardEmilienM: yeah18:03
_odydmsimard: installing fast_gettext is how I am testing locally today.  I use the vanilla upstream CentOS 7 cloud images.18:03
dmsimardruby-2.0.0.598-25.el7_1.x86_6418:04
bkero_ody: I don't believe the ruby mirror is in use yet18:04
dmsimardupstream p-o-i has ruby-2.0.0.598-25.el7_1.x86_64 as well18:05
dmsimardEmilienM: the issue just appeared today, like in the past couple hours18:05
dmsimardEmilienM: I guess we will see with https://review.openstack.org/#/c/350189/18:06
mwhahahauh that's commented out18:06
mwhahahaso it's not going to work18:07
iurygregoryyou are not going to install XD18:07
dmsimardmwhahaha, iurygregory: that's on purpose18:07
dmsimardI want to see if it reproduces18:08
iurygregorydmsimard, oh18:08
dmsimardthen I'll uncomment with a new patchset18:08
_odyIt seems that infra EL 7 does use ruby 2.0.0.18:08
_odyOk.  I think I see what is happening.18:09
mwhahahawell it reproduced, http://logs.openstack.org/89/350189/1/check/gate-puppet-openstack-integration-puppet-syntax-3-centos-7/959157b/console.html#_2016-08-02_18_05_47_84413318:09
mwhahahasooo remove the #? :D18:09
_odyah. nvm.  you found one.18:09
dmsimardmwhahaha: cool, removing :D18:09
dmsimardlet me file a bug for this.. hang on18:09
*** shardy has quit IRC18:10
_odyIn test where we don't us bundler I don't see the failure because it looks like fast_gettext is already part of the image so r10k is not updating it.18:10
mwhahahai want to punch the entire ruby gem community in the head whenever they go from no ruby requirement for a gem to something above the currently provided via main distros18:10
_odyhttp://logs.openstack.org/67/329767/1/check/gate-puppet-openstack-integration-3-scenario003-tempest-centos-7/8e9d3db/console.html.gz#_2016-06-15_05_57_30_39118:11
dmsimard_ody: that's not the same review though18:11
bkero_ody: so distro package already installed?18:11
dmsimard_ody: also that's from june :P18:12
_odydmsimard: I know.  I had already picked a random different one to look at.18:12
_odydmsimard: I was just illustrating that in the list of gems installed it doesn't list fast_gettext.18:12
mwhahahagettext-setup requires it18:12
mwhahahawhich puppet_forge requires18:12
mwhahahaso that's where it comes from18:12
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Pre-install fast_gettext before r10k  https://review.openstack.org/35018918:13
dmsimarduncommented ^18:13
bkeromwhahaha: shouldn't contraint managing just use an older version of fast_gettext that is compatible with ruby 2.0.0 though?18:14
mwhahahait doesn't take the ruby version into consideration18:14
mwhahahanever does18:14
bkeroThat's naive :/18:14
mwhahahathat's ruby18:14
* mwhahaha gets stabby18:14
dmsimardI hate ruby as well :( I feel pain whenever I have to do ruby.18:15
mwhahahawell it's not just ruby, it's developers18:15
bkerocan a conditional be put in a gemspec for that?18:15
mwhahahaoh it comes from a gemspec, it's just taken into consideration during the dependency calculations18:15
mwhahahahttps://github.com/grosser/fast_gettext/blob/master/fast_gettext.gemspec#L1118:16
bkerohttps://github.com/grosser/fast_gettext/commit/ccb620fde33bf900a4490564a49a5696ce81158b18:17
dmsimard_ody: it doesn't look like my installation of fast-gettext is working http://logs.openstack.org/89/350189/1/check/gate-puppet-openstack-integration-puppet-syntax-3-centos-7/959157b/console.html#_2016-08-02_18_07_24_52603718:17
bkero"modernize."18:17
bkeronice commit message18:17
mwhahahaimho ruby version requirement changes on gems should be a full version bump because it always ends up breaking something somewhere18:17
_odyI think dmsimard fix is only going to affect scenarios, Don't we needs to update it somewhere else for things that use bundler?18:17
mwhahahait's just an r10k requirement, so we need to trackdown that gemfile somewhere18:17
*** degorenko is now known as _degorenko|afk18:18
_odyProbably here, https://github.com/openstack/puppet-openstack_spec_helper/blob/master/puppet-openstack_spec_helper.gemspec.18:18
dmsimard_ody: hm, nevermind my last statement, that's from the first run18:18
_odydmsimard: Still, I think my statement stands.  We need to update spec_helper's gemspec.18:18
bkerodmsimard: I think that might need to be ran beforehand, and bundler is sourcing in gettext too18:18
_odydepends on the job, p-o-i vs. syntax and rspec jobs.18:19
mwhahahait needs to get fixed in that gemspec18:19
dmsimard_ody: gemspec probably needs to be fixed because that's what the beaker jobs uses, no ?18:19
mwhahahaand unit tests18:19
dmsimardright18:19
bkerospec.add_dependency 'fast-gettext', ['~> 1.1.0'] or some such18:19
bkero1.1.0 is the latest tag that doesn't include the commit that requires ruby 2.1.0 (per https://github.com/grosser/fast_gettext/commits/master )18:20
_odydmsimard: Yep. Beaker, unit, and syntax.18:20
dmsimardok I'll retry with 1.1.0 instead18:21
dmsimardwait no that's what I'm already trying18:21
bkeroneed it in the gemspec18:21
_odydmsimard: It's the right fix for one problem but we have another.18:21
_odyI'll grab it.18:21
bkero'bundle install' is happening before gem install r10k18:21
dmsimardbkero: yeah you're right18:22
dmsimard_ody: you're taking care of the right fix ?18:22
_odydmsimard: They are both correct fixes.18:22
_odyYes.  I am getting the second one.18:22
dmsimardbkero: huh, do you know where the bundle install happens ?18:23
bkerodmsimard: http://logs.openstack.org/89/350189/1/check/gate-puppet-openstack-integration-puppet-syntax-3-centos-7/959157b/console.html#_2016-08-02_18_05_07_73417118:23
dmsimardbkero: I meant technically18:24
bkeroah18:24
dmsimardbkero: but I found it in the meantime18:24
dmsimardit's in the jobs themselves, not in run_tests.sh or what have you18:24
bkeroYeah, it's part of the job builder18:24
dmsimardi.e, https://github.com/openstack-infra/project-config/blob/35ddbb318d25bd918caf0ef6d0ad8a70186fd8f6/jenkins/jobs/puppet-module-jobs.yaml#L3818:24
mwhahahabut it's pulling the requirements from the spec helper gemspec18:25
bkeroSo once we have to start putting fixes into other repos around this problem18:25
mwhahahaso just update that18:25
bkeroI wonder where the proper place for a fix to this is18:25
dmsimard_ody: so I confirm that the gem install in p-o-i fixes the issue18:25
_odycool.18:26
_odySubmitting the other now.18:26
dmsimard_ody: the integration jobs are running right now. Would a depends-on work to resolve the syntax job failures ?18:26
_odyYeah.18:26
openstackgerritCody Herriges proposed openstack/puppet-openstack_spec_helper: Lock fast_gettext to something that isn't 1.2.0.  https://review.openstack.org/35021718:27
dmsimard_ody: ok, will try that. Can you also link https://bugs.launchpad.net/puppet-openstack-integration/+bug/1609081 ?18:27
openstackLaunchpad bug 1609081 in puppet-openstack-integration "r10k installation fails with fast_gettext requires Ruby version >= 2.1.0" [Undecided,New]18:27
_odysure18:27
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Pre-install fast_gettext before r10k  https://review.openstack.org/35018918:27
dmsimardadded Depends-On.18:28
openstackgerritCody Herriges proposed openstack/puppet-openstack_spec_helper: Lock fast_gettext to something that isn't 1.2.0.  https://review.openstack.org/35021718:28
bkeroi think a combination of those two would fix it18:29
bkerowhee 'This change depends on a change that failed to merge.'18:30
*** johnmilton has joined #puppet-openstack18:31
dmsimardhuh18:31
dmsimard_ody: is your commit rebased ?18:31
bkeroI'm trying to look at zuul for ody's commit18:31
bkerolooks like they're running18:32
bkero2016-08-02 18:33:15.348711 | Installing fast_gettext 1.1.018:33
* dmsimard shrugs18:33
bkero\o/18:33
_odyGood enough for me.  Going to head off to lunch with wife.  Be back in about 1.5 ish.18:34
*** johnmilton has quit IRC18:35
* bkero waves18:35
*** guimaluf has joined #puppet-openstack18:35
dmsimardI guess I will wait for the spechelper merge and remove the dependson18:36
dmsimardlooks like there's an installation elsewhere too http://logs.openstack.org/17/350217/2/check/gate-puppet-openstack-libraries-puppet-beaker-rspec-centos-7/7bc18c3/console.html#_2016-08-02_18_40_14_91515918:43
dmsimarddoh, it uses install_modules from p-o-i18:43
dmsimardfuck how are we going to merge this18:43
dmsimardboth depend on each other18:43
dmsimardEmilienM: yo18:43
* mwhahaha facepalms18:44
bkerohahaha18:44
dmsimardEmilienM: the bug affects spec helper and p-o-i, spec helper uses p-o-i's install_modules and p-o-i runs syntax jobs which requires the fix from spechelper18:44
EmilienMI know18:44
EmilienMI wrote that18:44
dmsimardEmilienM: should we temporarily make the jobs non-voting or something18:44
EmilienMlol18:44
EmilienMno need of non voting18:45
dmsimardDepends-On on both ends doesn't work, does it ?18:45
EmilienMjust exit 0 in the run_*.sh18:45
dmsimardugh18:45
dmsimardthat's ugly18:45
bkerodirtydirty18:45
bkerobut hey18:45
dmsimardbut okay18:45
EmilienMwait a sec18:47
EmilienMbetr18:47
EmilienMbetter18:47
EmilienMyou can run gem install fast_gettext -v 1.1.0 in functions18:47
EmilienMin spec_helper18:47
EmilienMand you'll remove it later18:47
dmsimardEmilienM: hm, maybe18:48
*** johnmilton has joined #puppet-openstack18:48
EmilienMyeah do that18:49
EmilienMwe can survive with it18:49
dmsimardEmilienM: I'm not sure functions is in the code path for this particular error, wouldn't it be in the beaker spec helper right before the install_modules.sh from p-o-i gets called %?18:51
dmsimardhere: https://github.com/openstack/puppet-openstack_spec_helper/blob/master/lib/puppet-openstack_spec_helper/beaker_spec_helper.rb#L5618:52
*** guimaluf has quit IRC18:56
EmilienMdmsimard: no18:56
EmilienMso18:56
EmilienM1) patch functions to make the tests working and 2) patch puppet-openstack_spec_helper.gemspec with the pin18:57
EmilienMonce we land this patch, patch p-o-i with your change18:57
EmilienMand then 3), we'll remove the workaround in functions of spec_helper18:57
EmilienMso we keep testing things18:57
*** maestropandy1 has joined #puppet-openstack18:58
*** maestropandy1 has left #puppet-openstack18:59
EmilienMdon't we do bundle install? a sec18:59
dmsimardEmilienM: I'm failing to understand what you mean, sorry ... the r10k install in http://logs.openstack.org/17/350217/2/check/gate-puppet-openstack-libraries-puppet-beaker-rspec-centos-7/7bc18c3/console.html#_2016-08-02_18_40_14_915159 works (the one from "functions"18:59
dmsimardEmilienM: it's the one on the beaker machine that doesn't18:59
dmsimardEmilienM: because the beaker machine uses install_modules.sh from p-o-i which isn't patched18:59
EmilienMah https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/puppet-module-jobs.yaml#L22118:59
EmilienMwe don't run bundle install19:00
EmilienMhmm, this thing is weird19:00
dmsimardEmilienM: you see this works: http://logs.openstack.org/17/350217/2/check/gate-puppet-openstack-libraries-puppet-beaker-rspec-centos-7/7bc18c3/console.html#_2016-08-02_18_39_46_312074 (since the gemspec update)19:00
EmilienMis our CI broken because of that right now?19:00
dmsimardEmilienM: yeah, everything is broken19:00
dfisheri can't get any of my patches to pass CI :(19:00
EmilienMdmsimard: did we file a bug?19:00
dmsimardEmilienM: https://bugs.launchpad.net/puppet-openstack-integration/+bug/160908119:01
openstackLaunchpad bug 1609081 in puppet-openstack-spec-helper "r10k installation fails with fast_gettext requires Ruby version >= 2.1.0" [Undecided,New] - Assigned to Cody Herriges (ody-cat)19:01
EmilienMon github I mean19:01
dmsimardEmilienM: _ody mentioned he was going to check around I believe19:01
EmilienMdmsimard: ok, if you want jobs disabled, just exit 0 in the bash scripts in spec_helper19:01
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack_spec_helper: Lock fast_gettext to something that isn't 1.2.0.  https://review.openstack.org/35021719:02
dmsimardEmilienM: ^ should work19:03
EmilienMI'm not sure19:03
dmsimardI am, because doing exactly that in p-o-i fixed it19:03
EmilienMyou need GEM_HOME when installing a gem, no?19:03
*** foxhunt72 is now known as foxhunt_away19:04
dmsimardah..19:04
* dmsimard sighs19:04
dmsimardok, this will be ugly19:04
_odyI was looking into Puppet trackers, not GitHub. Had to meet wife for lunch. Will be back in about an hour19:05
*** foxhunt_away is now known as foxhunt7219:06
openstackgerritMerged openstack/puppet-tripleo: New composable service: FluentD  https://review.openstack.org/34354619:07
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack_spec_helper: Lock fast_gettext to something that isn't 1.2.0.  https://review.openstack.org/35021719:12
*** dhollinger has quit IRC19:12
EmilienMdmsimard: so smart19:13
dmsimardI screwed up though19:13
dmsimardneed to escape quotes19:13
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack_spec_helper: Lock fast_gettext to something that isn't 1.2.0.  https://review.openstack.org/35021719:14
dmsimardthis will /probably/ work but I'm not proud19:14
EmilienMdmsimard: please add the lp bug https://review.openstack.org/#/c/350217/5/puppet-openstack_spec_helper.gemspec19:14
EmilienMso we can track it19:14
EmilienM_ody: can you report the issue on github when you're back? I even don't know what to report19:15
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack_spec_helper: Lock fast_gettext to something that isn't 1.2.0.  https://review.openstack.org/35021719:15
_odyYep.19:15
EmilienMthx19:15
*** maestropandy has joined #puppet-openstack19:19
*** maestropandy has left #puppet-openstack19:19
*** foxhunt72 is now known as foxhunt_away19:23
dmsimardneed to escape more characters, bleh19:29
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack_spec_helper: Lock fast_gettext to something that isn't 1.2.0.  https://review.openstack.org/35021719:32
dmsimardevery time I need to double backslash I die a little inside19:32
colonwqleaning toothpicks.19:33
*** maestropandy1 has joined #puppet-openstack19:33
*** maestropandy1 has left #puppet-openstack19:33
dfisheroh yeech.  inline sed.19:33
dmsimarddfisher: does solaris have sed ? jk. jk.19:34
dfishernope.  sure doesn't.19:34
dmsimardreally, now ?19:35
dfisherof course it has sed...19:35
dfishertwo different incompatible ones!19:35
* dfisher sobs19:35
dmsimardyou almost scared me19:35
dmsimardkinda like tar vs gnu tar ?19:35
dfisherexactly like that.19:36
bkeroBSD sed?19:36
dfisherdunno.  whichever is the "shitty one".19:36
mwhahahabsd sed19:36
mwhahaha:D19:36
* dfisher sighs19:36
bkero/cry19:36
dfisher /cry19:36
* mwhahaha cries onto his osx bsd sed19:36
dfisheri want sed -i for inline replace19:36
dmsimardoh what is that http://stackoverflow.com/questions/3576380/sed-i-what-the-same-option-in-solaris19:36
dfisheri don't get sed -i19:36
bkeromwhahaha: how old is that sed?19:36
*** degorenko has joined #puppet-openstack19:36
dfisheryes, see?  that.19:36
dmsimardsed '$ s/OLD/NEW/g' test > test.tmp && mv test.tmp test19:37
dmsimardjesus19:37
* dfisher sobs19:37
dfisheri promise, the bug has been filed against it.19:37
bkeroand sat there for a decade?19:37
mwhahahai have no idea what version of sed comes on OSX but the man page is from 200519:37
colonwqdmsimard, have you though about awk? '{ if( $0 contains gem install) {print "gem install fast_gettest"} print $0}'19:37
dfisheri'm looking sec :)19:37
bkeroD:19:38
*** iberezovskiy has joined #puppet-openstack19:38
dmsimardcolonwq: I'm not super knowledgeable with awk, but that hack is going away as soon as we land another patch in another repo19:38
dmsimardcolonwq: I have great respect for people that are good with awk :)19:38
colonwqdmsimard, Ah ok. I am ok with awk. It helps make quick reports and stuff.19:39
*** dhollinger has joined #puppet-openstack19:39
dmsimardI love using awk instead of grep sometimes and using different delimitors, there's a lot of overlap in my awk usage versus grep/cut .. I think it depends on my mood19:40
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3  https://review.openstack.org/34795619:41
EmilienMdmsimard: you'll need to squash poi with https://review.openstack.org/#/c/350118/19:41
EmilienMreally broken CI19:41
*** maestropandy has joined #puppet-openstack19:42
bkeroawk feels really weird when you override IFS19:42
dfisherbkero: the specific bug for -i support in solaris sed was filed on july 26th, 201119:42
dfisherso not THAT old but ffs.19:42
dmsimardbkero: bah, awk -F, '{print $1}' vs cut -f1 -d , ..19:42
bkeroright19:42
bkeroIFS=, echo $2 ;)19:43
dmsimardEmilienM: ok19:43
*** fxpester|2 has joined #puppet-openstack19:43
EmilienMdmsimard: thank you for helping on this thing19:43
*** maestropandy has left #puppet-openstack19:47
dmsimardEmilienM: it's hard to get that sed working within beaker19:47
dmsimardEmilienM: did you have another idea19:47
*** fxpester|2 has quit IRC19:49
bkerodmsimard: could you prepend it to the entire file?19:51
*** dprince has quit IRC19:51
dmsimardbkero: it needs to be done after the stuff like GEM_HOME has been set and stuff19:51
dmsimardhttps://github.com/openstack/puppet-openstack-integration/blob/master/install_modules.sh19:51
dmsimardthe sed I did adds it right before the gem install r10k19:52
bkeroright19:53
dmsimardI mean, instead of doing that, I *could* checkout the p-o-i review19:53
dmsimardwhich would be even dirtier19:53
dmsimardI just can't put a depends-on in zuul proper because it won't merge the spec_helper review19:53
bkerosed 's/^gem install r10k/gem install fast_gettext -v 1.1.0\rgem install r10k' ?19:55
bkerothat's arguably cleaner, but still bad :/19:55
bkeroless double backslashes though19:56
*** _ducttape_ has joined #puppet-openstack19:56
mwhahahawhen all else fails, bring up sed so people can argue about regexs for the next 6 hours19:56
dmsimard:D19:57
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Pre-install fast_gettext before r10k  https://review.openstack.org/35018919:57
colonwqThe great regex wars. I am glad the editor wars are over. Vi was the obvious winner.19:58
*** jesusaur has quit IRC19:58
bkeroMx-Mc-insult19:58
colonwq:)19:59
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Pre-install fast_gettext before r10k, disable Zaqar  https://review.openstack.org/35018919:59
*** ducttape_ has quit IRC20:00
*** jesusaur has joined #puppet-openstack20:00
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Pre-install fast_gettext before r10k, disable Zaqar  https://review.openstack.org/35018920:00
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack_spec_helper: Lock fast_gettext to something that isn't 1.2.0.  https://review.openstack.org/35021720:03
dmsimardI am not proud of this ^20:03
mwhahahaha, sounds about right20:03
EmilienMwow20:03
mwhahahajust about every time CI breaks it ends up being an adventure in awfulness for the workaround20:04
EmilienM319 jobs in check pipeline20:04
EmilienMmwhahaha: lol yeah20:04
dmsimardwell inter-dependent changes like that really sucks20:05
dmsimardit's in those times it's useful to be able to +5 a review20:05
dmsimardnot many people can do that upstream, though, I believe20:06
mwhahahawell we should probably get it so our job configurations/scripts all come from a single repo20:06
mwhahahathis one could probably be avoided if we moved the gem install from out of the project config and leveraged a single script in the spec helper repo20:06
mwhahahathou shall only use a single method for installation!20:07
dmsimardit's unclear to me because I'm not super familiar with it20:07
dmsimardbut sometimes things are installed directly by the job, other times by install_modules, other times by a bundle install20:08
*** _ducttape_ has quit IRC20:09
*** ducttape_ has joined #puppet-openstack20:09
* dmsimard maniacal laughter20:16
dmsimardthe git checkout hack works20:17
EmilienMI've +2 https://review.openstack.org/#/c/350217/20:18
*** ducttape_ has quit IRC20:24
*** Ahharu has joined #puppet-openstack20:26
Ahharuhello20:27
*** ducttape_ has joined #puppet-openstack20:27
Ahharuany ideas on this error? 2016-08-02 15:44:11.583034 | An error occurred while installing fast_gettext (1.2.0), and Bundler cannot20:27
Ahharuit is affecting lots of openstack projects20:27
openstackgerritAlex Schultz proposed openstack/puppet-designate: Update keystone authtoken configuration pattern  https://review.openstack.org/35025520:27
EmilienMyes we are working on it20:27
Ahharuupdating my gemfile to force the version would work?20:27
Ahharuor not since its installed by packae rubygems?20:28
mwhahahawe're working on pinning it for the puppet openstack modules, https://review.openstack.org/#/c/350217/20:29
openstackgerritcraig delatte proposed openstack/puppet-cinder: Cinder hooks support  https://review.openstack.org/34446420:29
_odydmsimard: Both changes look good.  From reading your node on puppet-openstack_spec_helper though I suspect that shouldn't be merged through until p-o-i fix is?20:33
dmsimard_ody: the spec_helper patch pulls the p-o-i review20:34
dmsimardit's a dirty hack to work around the depends-on from zuul which requires the parent patch to merge for a child patch to merge20:34
*** Ahharu has quit IRC20:34
dmsimardbecause the p-o-i patch needs the spec_helper patch and back and forth20:34
*** openstackgerrit_ has joined #puppet-openstack20:35
dmsimardonce both are merged we need to remove the git checkout from spec_helper asap20:36
*** openstackgerrit_ has quit IRC20:36
*** degorenko has quit IRC20:36
*** iberezovskiy has quit IRC20:36
_odyoh right.  It has to be ther to merge because zuul will run the pipeline again.20:37
_odyok.  I'll just wait on tests to finish then.20:37
_odydmsimard, EmilienM: the r10k issue is already fixed upstream. https://github.com/puppetlabs/gettext-setup-gem/commit/6a6a305053f42bbb3cf835454d5d4ca9f2e28c5820:43
dmsimardwell yay, we can abandon everything then20:43
*** dgurtner has joined #puppet-openstack20:43
mwhahahaha the commit message20:43
iurygregory\o/20:44
dmsimardabandon ship, abandon ship20:45
dmsimard_ody: I can't abandon your change20:45
_odydmsimard: Ok.20:46
dmsimardEmilienM: I did a recheck on the disable zaqar patch.20:47
*** bklei has quit IRC20:48
EmilienM_ody: is it released?20:48
*** bklei has joined #puppet-openstack20:48
_odyEmilienM: gettext-setup is.  Double checking if released r10k deplosy the right hting.20:48
mwhahahahttps://rubygems.org/gems/gettext-setup/versions/0.5 yea it is20:49
_odyYep.20:49
_odyI think we are good.20:49
_odyIt installed properly here.20:49
EmilienMok20:49
EmilienMdmsimard: sorry20:50
EmilienMI'm doing recheck on https://review.openstack.org/#/c/350118/20:50
EmilienMto bring our CI back20:50
dmsimardEmilienM: oops I did a recheck on the wrong patch https://review.openstack.org/#/c/350119/120:50
EmilienMdmsimard: it's ok20:51
openstackgerritEmilien Macchi proposed openstack/puppet-gnocchi: TripleO Overcloud CI test - do not merge  https://review.openstack.org/35026521:00
*** foxhunt_away is now known as foxhunt7221:00
*** moravec has quit IRC21:03
*** jmiu has quit IRC21:06
*** jmiu has joined #puppet-openstack21:06
*** jmiu has quit IRC21:06
*** fultonj has quit IRC21:07
*** jmiu_ has joined #puppet-openstack21:07
*** bklei_ has joined #puppet-openstack21:15
*** bklei has quit IRC21:18
*** bklei_ has quit IRC21:24
*** bklei has joined #puppet-openstack21:25
*** foxhunt72 is now known as foxhunt_away21:26
*** jayg is now known as jayg|g0n321:26
*** bklei has quit IRC21:35
*** bklei has joined #puppet-openstack21:36
EmilienMmwhahaha: I need to go now /me back later - can you look https://review.openstack.org/#/c/349765/ when you have time? It will help to promote our CI, thanks21:38
openstackgerritBradley Klein proposed openstack/puppet-monasca: Add two additional projections for new api queries  https://review.openstack.org/35028021:46
openstackgerritMerged openstack/puppet-ceph: Refresh keys if caps are changed  https://review.openstack.org/34786021:51
*** markvoelker has quit IRC21:54
*** bklei has quit IRC22:06
*** dgurtner has quit IRC22:24
*** guimaluf has joined #puppet-openstack22:38
*** _ducttape_ has joined #puppet-openstack22:44
*** guimaluf has quit IRC22:44
*** guimaluf has joined #puppet-openstack22:45
*** ducttape_ has quit IRC22:47
*** delattec has joined #puppet-openstack22:48
*** _ducttape_ has quit IRC22:48
*** cdelatte has quit IRC22:51
*** guimaluf has quit IRC22:51
*** saneax_AFK is now known as saneax23:00
*** gfidente has joined #puppet-openstack23:01
*** vinsh has quit IRC23:05
*** dfisher has quit IRC23:06
*** xarses has quit IRC23:09
*** guessi has joined #puppet-openstack23:45
openstackgerritSam Morrison proposed openstack/puppet-neutron: Add support for rabbit_transient_queues_ttl oslo_messaging config option  https://review.openstack.org/34931223:48
*** morazi has quit IRC23:49
*** bklei has joined #puppet-openstack23:55
*** dprince has joined #puppet-openstack23:56

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