Tuesday, 2016-06-14

*** xarses has quit IRC00:05
openstackgerritDan Radez proposed openstack/puppet-tacker: Commiting a cookie cutter module sync set of files  https://review.openstack.org/32923000:12
openstackgerritDan Radez proposed openstack/puppet-tacker: Commiting a cookie cutter module sync set of files  https://review.openstack.org/32923000:17
*** saneax is now known as saneax_AFK00:41
*** rwsu has quit IRC01:30
*** ilbot3 has quit IRC01:46
*** ilbot3 has joined #puppet-openstack01:47
*** xarses has joined #puppet-openstack01:57
*** mfisch has quit IRC02:10
EmilienMmwhahaha: should we merge https://review.openstack.org/#/c/234798/9?02:33
EmilienMI'm about to -2 it02:33
mwhahahaProbably should stop that02:41
mwhahahaI blocked it, I'll look into it more in the morning02:44
iurygregory_lol i have recived the email with +2 from jenkis .-.02:55
iurygregory_but tks god we don't have the commit in the repo =D02:56
*** richm has quit IRC02:59
*** openstackgerrit has quit IRC03:11
*** openstackgerrit has joined #puppet-openstack03:11
openstackgerritMatt Fischer proposed openstack/puppet-openstack-guide: Document Emilien's tips for new modules  https://review.openstack.org/32925303:15
*** iurygregory_ has quit IRC03:33
*** ducttape_ has quit IRC03:50
*** ayoung has quit IRC03:56
*** xarses_ has joined #puppet-openstack04:14
*** saneax_AFK is now known as saneax04:16
*** markvoelker has quit IRC04:16
*** andreww has joined #puppet-openstack04:17
*** xarses has quit IRC04:17
*** andreww has quit IRC04:18
*** andreww has joined #puppet-openstack04:18
*** xarses_ has quit IRC04:20
*** jaosorior has joined #puppet-openstack05:11
*** markvoelker has joined #puppet-openstack05:17
openstackgerritDan Radez proposed openstack/puppet-tacker: Commiting a cookie cutter module sync set of files  https://review.openstack.org/32923005:22
*** markvoelker has quit IRC05:22
openstackgerritDan Radez proposed openstack/puppet-tacker: Commiting a cookie cutter module sync set of files  https://review.openstack.org/32923005:23
openstackgerritDan Radez proposed openstack/puppet-congress: Commiting a cookie cutter module sync set of files  https://review.openstack.org/32832205:23
*** jmikolajczak has joined #puppet-openstack05:45
*** rcernin has joined #puppet-openstack06:11
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/32927506:11
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/32859106:12
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova vncproxy profile  https://review.openstack.org/32613006:28
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova Consoleauth profile  https://review.openstack.org/32611206:28
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova Compute profile  https://review.openstack.org/32723606:28
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova Scheduler profile  https://review.openstack.org/32609206:28
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova API profile  https://review.openstack.org/32593706:28
*** dgurtner has joined #puppet-openstack06:31
*** dgurtner has quit IRC06:31
*** dgurtner has joined #puppet-openstack06:31
*** hjensas__ has quit IRC06:44
*** openstackgerrit has quit IRC06:48
*** openstackgerrit has joined #puppet-openstack06:48
*** ducttape_ has joined #puppet-openstack06:51
openstackgerritMerged openstack/puppet-magnum: Fix magnum-db-sync orchestration  https://review.openstack.org/32901806:54
*** tesseract has joined #puppet-openstack06:56
*** ducttape_ has quit IRC06:56
*** amoralej|off is now known as amoralej06:58
*** leanderthal|afk is now known as leanderthal07:01
openstackgerritMerged openstack/puppet-openstack-integration: Ensure $GEM_BIN_DIR is in $PATH  https://review.openstack.org/32922207:13
*** xarses_ has joined #puppet-openstack07:14
*** andreww has quit IRC07:17
*** markvoelker has joined #puppet-openstack07:18
*** markvoelker has quit IRC07:22
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add fact to get the fqdn for a host in the different networks  https://review.openstack.org/32929907:23
*** dgurtner has quit IRC07:25
*** jpena|off is now known as jpena07:26
*** karimb has joined #puppet-openstack07:30
*** hjensas__ has joined #puppet-openstack07:31
*** karimb has quit IRC07:31
*** dmk0202 has joined #puppet-openstack07:32
*** karimb has joined #puppet-openstack07:32
*** ifarkas has joined #puppet-openstack07:32
*** jaosorior is now known as jaosorior_brb07:35
openstackgerritMerged openstack/puppet-tripleo: Add mongodb profiles  https://review.openstack.org/30478007:55
*** dtantsur|afk is now known as dtantsur08:04
*** shardy has joined #puppet-openstack08:04
*** danpawlik has joined #puppet-openstack08:09
*** queretaro has joined #puppet-openstack08:16
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Add gnocchi profiles  https://review.openstack.org/31552708:23
*** chem`` has joined #puppet-openstack08:31
*** chem` has quit IRC08:32
*** jaosorior_brb has quit IRC08:39
*** jaosorior_brb has joined #puppet-openstack08:40
*** derekh has joined #puppet-openstack08:40
*** ducttape_ has joined #puppet-openstack08:52
*** jaosorior_brb is now known as jaosorior08:53
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova vncproxy profile  https://review.openstack.org/32613008:56
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova Consoleauth profile  https://review.openstack.org/32611208:56
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova Compute profile  https://review.openstack.org/32723608:56
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova Scheduler profile  https://review.openstack.org/32609208:56
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova API profile  https://review.openstack.org/32593708:56
*** ducttape_ has quit IRC08:58
*** dgurtner has joined #puppet-openstack09:08
*** rochaporto has quit IRC09:23
*** rochaporto has joined #puppet-openstack09:24
openstackgerritMerged openstack/puppet-keystone: Add keystone::wsgi::apache::admin_bind_host param  https://review.openstack.org/32911409:31
*** foxhunt_away is now known as foxhunt7209:32
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Composable roles within services - NTP  https://review.openstack.org/31072509:38
openstackgerritOleksiy Molchanov proposed openstack/puppet-ceph: Fix activate check  https://review.openstack.org/31050709:39
*** _degorenko|afk is now known as degorenko09:39
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Add ceilometer profiles  https://review.openstack.org/30514309:45
openstackgerritDenis Egorenko proposed openstack/puppet-murano: Use regexp by whole name for murano_application  https://review.openstack.org/32934309:46
dtantsurdegorenko, morning! could you take a look at https://review.openstack.org/#/c/329087/ please? it blocks some tripleo work.09:54
degorenkodtantsur, morning, sure09:55
degorenkodtantsur, lgtm :)09:57
dtantsurthanks!09:57
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker.  https://review.openstack.org/30906910:04
*** dmk0202 has quit IRC10:08
*** xb_ has joined #puppet-openstack10:12
*** dgurtner has quit IRC10:16
openstackgerritMerged openstack/puppet-neutron: beaker: fix resource ordering  https://review.openstack.org/32903310:16
openstackgerritMerged openstack/puppet-rally: Manage SQLite database directory  https://review.openstack.org/31793910:27
openstackgerritBoris Kreitchman proposed openstack/puppet-tripleo: Add a Cinder backup and a Cinder backup Ceph profiles.  https://review.openstack.org/31121810:28
openstackgerritMerged openstack/puppet-ironic: stop managing /etc/ironic and ironic.conf  https://review.openstack.org/32908710:38
openstackgerritMerged openstack/puppet-octavia: Implement api.pp  https://review.openstack.org/32614210:42
*** dtantsur is now known as dtantsur|brb10:45
openstackgerritMerged openstack/puppet-octavia: Update db_sync command  https://review.openstack.org/32865910:46
*** jpena is now known as jpena|off10:48
*** jpena|off is now known as jpena10:48
openstackgerritIvan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/32767810:53
*** ducttape_ has joined #puppet-openstack10:54
jaosoriordegorenko: Hello; quick question, why did you try a re-check in https://review.openstack.org/#/c/321818/ ?10:57
degorenkojaosorior, hey, is your tripleo patch merged? Or not yet? i lost link :D10:58
jaosoriordegorenko: It can't merge because it depends on the puppet-nova patch10:58
degorenkojaosorior, ah, got it now10:58
*** ducttape_ has quit IRC10:59
jaosoriordegorenko: haha yeah, it can be a bit confusing. But yeah, that puppet-nova patch should be ready to merge now.10:59
degorenkojaosorior, ok, +2'ed :)10:59
jaosoriordegorenko: Awesome, thanks man10:59
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add fact to get the fqdn for a host in the different networks  https://review.openstack.org/32929911:02
openstackgerritXingchao Yu proposed openstack/puppet-rally: Add rally::db::sync  https://review.openstack.org/31797011:05
*** permalac has quit IRC11:09
*** queretaro has quit IRC11:19
*** kullmar has joined #puppet-openstack11:32
*** karimb has quit IRC11:36
openstackgerritAlexey Deryugin proposed openstack/puppet-openstack-integration: Add tempest-horizon plugin installation  https://review.openstack.org/32938811:38
openstackgerritIvan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/32767811:40
openstackgerritAlexey Deryugin proposed openstack/puppet-openstack-integration: Add tempest-horizon plugin installation  https://review.openstack.org/32938811:42
*** kullmar has quit IRC11:45
EmilienMhello11:46
iberezovskiy|offhi11:46
*** iberezovskiy|off is now known as iberezovskiy11:46
*** jpena is now known as jpena|lunch11:52
openstackgerritAlexey Deryugin proposed openstack/puppet-openstack-integration: scenario003: enable application catalog service  https://review.openstack.org/25043611:52
*** ducttape_ has joined #puppet-openstack11:54
openstackgerritXingchao Yu proposed openstack/puppet-rally: Add rally::db::sync  https://review.openstack.org/31797011:55
*** dtantsur|brb is now known as dtantsur11:58
*** ducttape_ has quit IRC12:03
openstackgerritMerged openstack/puppet-aodh: beaker: fix ordering for RabbitMQ  https://review.openstack.org/32903712:04
openstackgerritMerged openstack/puppet-trove: beaker: disable taskmanager on Xenial  https://review.openstack.org/32902012:05
openstackgerritIvan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/32767812:07
EmilienMiberezovskiy: still debugging? lol12:10
iberezovskiyit was a bad idea to debug it at sunday midnight12:11
iberezovskiywe used wrong dir name12:11
iberezovskiyit should be siteS-*, not site-*12:11
EmilienMah :(12:12
EmilienMdegorenko: no need to do recheck https://review.openstack.org/#/c/328591/12:12
*** markvoelker has joined #puppet-openstack12:13
EmilienMwe need to inspect logs and find what's wrong12:13
*** paramite has joined #puppet-openstack12:20
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/32859112:24
iberezovskiyEmilienM, take a look please on the latest patch set, looks like it works but tempest has failed :D12:25
openstackgerritMerged openstack/puppet-nova: providers: add hack to require nova providers outside puppet-nova  https://review.openstack.org/32852512:25
iberezovskiyhttp://logs.openstack.org/78/327678/25/check/gate-puppet-openstack-integration-3-scenario001-tempest-ubuntu-xenial/d6c6085/console.html#_2016-06-14_12_19_44_70112:25
* iberezovskiy bbs ~30 min12:28
*** fwdit has joined #puppet-openstack12:30
iurygregorymorning people =)12:30
*** trown|outtypewww is now known as trown12:37
EmilienMiberezovskiy: where should I look?12:40
EmilienMah12:40
EmilienMI missed the link, sorry12:40
EmilienMit's in keystone12:41
EmilienMhttp://codesearch.openstack.org/?q=keystone_tempest_plugin&i=nope&files=&repos=12:41
*** johnmilton has joined #puppet-openstack12:41
EmilienMmaybe we need to unpin tempest12:41
*** jayg|g0n3 is now known as jayg12:43
openstackgerritBrad P. Crochet proposed openstack/puppet-tripleo: Split Heat pacemaker roles into separate services  https://review.openstack.org/32770812:45
*** queretaro has joined #puppet-openstack12:47
*** amoralej is now known as amoralej|lunch12:48
openstackgerritAlexey Deryugin proposed openstack/puppet-openstack-integration: Add tempest-horizon plugin installation  https://review.openstack.org/32938812:49
*** jpena|lunch is now known as jpena12:51
openstackgerritMarcellin Fom Tchassem proposed openstack/puppet-swift: Change default service_name to 'swift'  https://review.openstack.org/32725212:51
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Add transport_url and rpc_response_timeout for oslo.messaging  https://review.openstack.org/32181812:58
EmilienMjaosorior: it was failing Ci, rebasing and re approving ^12:59
jaosoriorEmilienM: alright12:59
openstackgerritMarcellin Fom Tchassem proposed openstack/puppet-swift: Change default service_name to 'swift'  https://review.openstack.org/32725213:00
*** hjensas__ has quit IRC13:02
iberezovskiyEmilienM, I'll check locally first13:07
*** dgurtner has joined #puppet-openstack13:07
EmilienMiberezovskiy: see on #ubuntu-server13:07
EmilienMiberezovskiy: can you take care of it? i'm trying to make progress on RDO promotion today, we didn't promote since Friday and I see some problems too13:08
iberezovskiyok, I'll work on this13:08
mnasermorning, i see log_facility is deprecated in neutron but not in nova.  was it forgotten in nova or is it actually not deprecated in nova?13:09
EmilienMiberezovskiy: do you want to cleanup https://review.openstack.org/#/c/327678/25/copy_logs.sh ?13:21
iberezovskiyEmilienM, yeah, sure13:21
EmilienMiberezovskiy: otherwise your patch looks fine.13:22
openstackgerritDan Radez proposed openstack/puppet-tacker: Commiting a cookie cutter module sync set of files  https://review.openstack.org/32923013:22
*** fwdit has quit IRC13:24
openstackgerritIvan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/32767813:24
iurygregoryEmilienM, do you know Dan Radez irc? O.o13:27
*** ericksonsantos has joined #puppet-openstack13:29
*** radez has joined #puppet-openstack13:31
EmilienMiurygregory: ^13:31
iurygregoryLOL13:32
iurygregorytks EmilienM13:32
radezhey iurygregory, thx for your reviews :)13:32
iurygregoryradez, o/ np tks for your work and patience13:33
radezsure thing13:33
openstackgerritMerged openstack/puppet-nova: beaker: don't deploy virtlock and virtlog  https://review.openstack.org/32880913:33
* iurygregory brb13:34
openstackgerritAlex Schultz proposed openstack/puppet-keystone: Preparation for default endpoint names rename  https://review.openstack.org/23479813:36
*** richm has joined #puppet-openstack13:36
*** amoralej|lunch is now known as amoralej13:37
*** foxhunt72 is now known as foxhunt_away13:38
*** dfisher has joined #puppet-openstack13:41
openstackgerritBrad P. Crochet proposed openstack/puppet-tripleo: Add Mistral profiles  https://review.openstack.org/32343113:42
*** xarses_ has quit IRC13:45
*** xarses_ has joined #puppet-openstack13:45
*** foxhunt_away is now known as foxhunt7213:50
openstackgerritMerged openstack/puppet-rally: Add rally::db::sync  https://review.openstack.org/31797013:54
*** cdelatte has joined #puppet-openstack13:56
*** xarses_ has quit IRC14:00
*** dhollinger has joined #puppet-openstack14:01
dfisherso, for the record, configuring Neutron with:  an external vlan net, an internal vxlan net, a router and all the in-betweens (router interfaces, gateways, etc.) via Puppet is hard and frustrating.14:05
dfisher</whine>14:05
*** jaosorior has quit IRC14:05
EmilienMmwhahaha: fyi /me afk after our weekly meeting until tomorrow morning or late tonight14:06
mwhahahak14:06
*** xarses_ has joined #puppet-openstack14:08
*** xarses_ has quit IRC14:08
*** xarses_ has joined #puppet-openstack14:09
*** ducttape_ has joined #puppet-openstack14:12
*** ducttape_ has quit IRC14:12
*** ducttape_ has joined #puppet-openstack14:13
*** foxhunt72 is now known as foxhunt_away14:13
openstackgerritDan Radez proposed openstack/puppet-congress: Commiting a cookie cutter module sync set of files  https://review.openstack.org/32832214:17
openstackgerritDan Radez proposed openstack/puppet-tacker: Commiting a cookie cutter module sync set of files  https://review.openstack.org/32923014:19
*** osp has joined #puppet-openstack14:19
*** xarses_ has quit IRC14:21
openstackgerritMerged openstack/puppet-nova: Add transport_url and rpc_response_timeout for oslo.messaging  https://review.openstack.org/32181814:29
ospCan anyone recommend a resource detailing how to apply a custom puppet manifest prior to overcloud deployment? I have tried with ComputeExtraConfigPre but seem to run into multiple issue.14:31
openstackgerritMerged openstack/puppet-tripleo: Implement Nova API profile  https://review.openstack.org/32593714:31
*** Lokesh_Jain has joined #puppet-openstack14:35
openstackgerritIvan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/32767814:35
iberezovskiyEmilienM, while we are waiting, just try for alternative solution ^14:35
EmilienMosp: you would be luckier on #tripleo14:36
*** ayoung has joined #puppet-openstack14:36
jmiuosp: you on #tripleo?14:43
*** xarses_ has joined #puppet-openstack14:44
ospEmilienM: thanks i will ask on there.14:48
openstackgerritMerged openstack/puppet-tripleo: Implement Nova Scheduler profile  https://review.openstack.org/32609214:48
openstackgerritMerged openstack/puppet-tripleo: Implement Nova Consoleauth profile  https://review.openstack.org/32611214:48
ospjmiu: yes I am, will ask on there. Thanks14:48
*** xarses_ is now known as xarses14:50
openstackgerritMerged openstack/puppet-tripleo: Implement Nova vncproxy profile  https://review.openstack.org/32613014:51
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Deploy nova::db classes in api profile  https://review.openstack.org/32602714:55
iurygregoryAlmost meeting time people, #openstack-meeting-414:56
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: profile/base/nova: declare nova class properly  https://review.openstack.org/32834714:57
*** _ody has joined #puppet-openstack14:58
openstackgerritMerged openstack/puppet-watcher: This commit adds the initial project files.  https://review.openstack.org/32659314:58
iurygregorydanpawlik, ^ congrats14:59
danpawlikiurygregory: yay :D14:59
*** _stowa has joined #puppet-openstack15:00
radezdegorenko: thx for putting my modules on the meeting adgenda today15:05
degorenkoradez, please join on #openstack-meeting-4 :)15:06
radezdegorenko: I'm in there15:06
openstackgerritIvan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/32767815:07
*** rcernin has quit IRC15:11
*** _stowa has quit IRC15:12
radezdegorenko: the module is coming from some where, so I'm not sure that we're brakeing anyone15:13
radezthey can just used the previous location until the official location is ready for use.15:13
*** saneax is now known as saneax_AFK15:13
EmilienMvinsh: you're missing our meeting15:14
EmilienMand we talk about you15:14
radezdegorenko: I agree with you that it doesn't make sense to pull code out just to add it back, but I can see how deleteing ansd starting from scratch of cc/msync would fit more in line with the idea15:14
vinshDanggit15:14
radezthat EmilienM is suggesting that modules should be started in puppet-openstack's forge15:14
openstackgerritMerged openstack/puppet-murano: Use regexp by whole name for murano_application  https://review.openstack.org/32934315:16
degorenkoradez, wait please for end of meeting :D15:16
vinsh_ody: stating around puppet 4.1~  this problem occurs: http://logs.openstack.org/96/328796/3/experimental/gate-puppet-swift-puppet-beaker-rspec-ubuntu-xenial-nv/3211d51/console.html#_2016-06-13_16_40_02_775  It is triggered here: https://github.com/openstack/puppet-swift/blob/master/lib/puppet/provider/service/swiftinit.rb#L84-L104   This does not occur on puppet <4.115:17
vinshI can't figure out what is going on... where is it getting a value of "24" for an md5sum?15:18
_odyOk. Got them up in browser.15:18
vinshI have tried many things.. even setting hash check to "none".. the problem is.. you can't specify file contents with "none" set for hashtype.15:20
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-openstack-guide: Update meetings 85  https://review.openstack.org/32951915:23
iurygregoryEmilienM,15:23
degorenkoradez, well, ok, if there are no any problem with removing old stuff :)15:23
* radez shrugs15:24
radezI see valid arguments for both ways15:24
radezbut since the modules are out of the norm of the operations of establishing them15:24
radezI suppose starting with a blank canvas but preserving the history could make up for that somewhat15:25
radezdegorenko: ^15:25
degorenkoradez, btw, why you have different commit message from cookiecutter commit message template? :)15:25
*** xarses_ has joined #puppet-openstack15:25
radezdegorenko: not sure I understand that you're asking15:25
EmilienMiurygregory: etherpad link is wrong15:25
*** xarses has quit IRC15:26
radezdegorenko: oh you mean I'm not using the cookie cutter git history?15:26
degorenkoradez, i meant, that your commit message is not from https://github.com/openstack/puppet-openstack-cookiecutter/blob/master/contrib/bootstrap.sh#L13815:26
radezdegorenko: I just copied the content into the existing repo's git hitory and made a new commit to preserve the old history15:26
degorenkoradez, did you run this script?15:26
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-openstack-guide: Update meetings 85  https://review.openstack.org/32951915:27
radezI did, run it, but copied the content into the existing repo I had to preseve the histsory of where it came from degorenko15:27
degorenkohm, ok then15:27
radezdegorenko: would you prefer a different way?15:28
radezI"m not sure that from the import it gerrit would allow it a different way15:28
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Add gnocchi profiles  https://review.openstack.org/31552715:28
degorenkoradez, well, i'm ok with current way. How can guaranty that we don't miss something? :)15:28
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Add ceilometer profiles  https://review.openstack.org/30514315:29
radezwell in the patches surrent states they match the content from cc/msync15:29
radez*current15:29
iurygregoryEmilienM, fixed =D15:29
degorenkoradez, yes, i'm about next patches15:29
degorenkohave you prepared local copy of current repo? :D15:29
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Composable roles within services - NTP  https://review.openstack.org/31072515:29
radezdegorenko: I figure we'll take the content from the history and start to overlay it over the cc/msync patch15:30
radezso that the development process looks as if we started from scratch15:30
_odyvinsh: Ok.  Doing a fresh beaker run locally so I can log in and figure out what is going on.15:30
radezdegorenko: then the previous history is just there for preservation sake15:30
degorenkoradez, ok to me15:30
vinsh_ody: ACK15:31
radezkk, so in that case I believe the current state of the patches are ready for commit and I"ll draft new patches that start to incorporate the preexisting content back over the cc/msync content15:31
radeziurygregory: degorenko: ^^ agreed15:31
radez?15:31
iurygregoryradez, thinking ...15:32
radez:)15:32
openstackgerritIvan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/32767815:32
degorenkoradez, i'm reviewing your current patches :D15:32
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: nova/api: include ::nova::network::neutron  https://review.openstack.org/32952915:32
* radez nods15:32
* radez brb15:32
*** leanderthal is now known as leanderthal|afk15:33
iurygregoryradez, i think i'm ok =)15:33
degorenkoradez, one minor comment :)15:34
*** jtomasek_ has joined #puppet-openstack15:34
iurygregorydegorenko, he is following cc =P15:34
degorenkoiurygregory, well, then we must note in docs - that users should read doc sections before they did git review15:35
degorenkoinit commit should not contain any pattern stuff from msycn and cookie cutter, imho15:36
radezwould it make more sense to delete the repos and recreate them empty15:36
iurygregorydegorenko, +1 from me and also i cant find some files in the commit .-.15:36
radezand throw away the previous history?15:36
degorenkoiurygregory, probably they are already created15:36
degorenkoEmilienM, thoughts about removing and re-creating repo ?15:36
iurygregoryradez, not throw away! (you should make a copy locally)15:37
radezwell yes, I have a local copy15:37
EmilienMdegorenko: really, no15:37
EmilienMI mean15:37
EmilienMdo not patch infra15:37
iurygregorydegorenko, i think this is like my idea =P15:37
EmilienMif you want to rewrite the module in repo, ok15:37
iurygregorycreate a commit removing files..15:37
degorenkothat's can be a solution15:37
iurygregoryand another adding cc/mysnc15:37
iurygregory=)15:37
radezbut will that pass ci?15:38
iurygregorywow good question LOL15:38
EmilienMI haven't followed all discussion but it's too hard to rewrite it?15:38
radezwe're at gerrit's mercy15:38
*** permalac has joined #puppet-openstack15:38
iurygregoryi think it will pass15:38
radezwe could try15:38
degorenkoradez, well, we can leave it just with rewritting with common cookie stuff15:38
radezI think either way will accomplish the same thing15:39
degorenkocurrent commits are good15:39
radezas long as we have a commit that is pure cc/msync15:39
radezwe're "starting" from a clean slate15:39
degorenkoradez you can change your title to smthg like : switch to openstack sytle or whatever15:39
degorenkostyle*15:39
degorenkosome title, that user can understand why we are rewritting module15:40
degorenkoi guess it will be enough15:40
radezsure, would you like me to copy the commit message from contrib.sh?15:40
radezoh I see what you're saying15:40
degorenkoradez, that's not important though, just asking :)15:40
radezyea I'll be more verbose in the commit msg15:40
degorenkoyes15:40
iurygregorytks radez15:40
degorenkoand clean up unneeded doc in init.pp :D15:41
* iurygregory brb lunch time 15:41
radezkk15:41
radezthx degorenko iurygregory15:41
radezI'll get a new patch set up for both modules15:42
degorenkosure, np15:42
*** delattec has joined #puppet-openstack15:44
openstackgerritDan Radez proposed openstack/puppet-tacker: Commiting a cookie cutter module sync set of files  https://review.openstack.org/32923015:47
*** cdelatte has quit IRC15:47
openstackgerritDan Radez proposed openstack/puppet-congress: Commiting a cookie cutter module sync set of files  https://review.openstack.org/32832215:47
*** danpawlik has quit IRC15:47
*** delattec has quit IRC15:51
*** delattec has joined #puppet-openstack15:53
*** jtomasek_ has quit IRC15:54
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Move enable_sig and release to ceph::params  https://review.openstack.org/32954415:55
*** dhollinger has quit IRC16:02
EmilienMmwhahaha: flight cancelled, I'll be here :(16:07
mwhahaha:(16:07
EmilienMtoo much wind16:08
iurygregoryEmilienM, you're at the airport? O.o16:10
EmilienMiurygregory: no, but I had a pilot lesson this afternoon, it's cancelled :(16:10
iurygregoryEmilienM, wow very cool :D16:12
openstackgerritDmitry Ilyin proposed openstack/puppet-ec2api: Add all other classes  https://review.openstack.org/31459816:12
*** permalac has quit IRC16:16
*** ifarkas has quit IRC16:19
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker.  https://review.openstack.org/30906916:24
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Move enable_sig and release to ceph::params  https://review.openstack.org/32954416:25
*** rwsu has joined #puppet-openstack16:27
_odyvinsh: sigh...I got a whole slew of different errors when I ran the beaker tests outside of OpenStack CI.16:36
vinshI can probably help get you going.. I'm running on just a stand alone vm.16:37
vinshI also find that version 4.2.3 works16:37
vinshthen 4.3.0 fails on concat errors I am tracking down16:37
vinshI'm moving past the concat error to see where this specific md5sum error comes in on 4.3 branch16:38
_odyvinsh: I thought we only ran beaker against puppet 4 latest.16:40
vinshThat might be true... i'm just trying to find out which version of puppet this issue starts in... so we can see what feature/change might have caused it.16:40
_odyah.16:40
EmilienMvinsh: we run beaker against 316:41
EmilienMerr _ody ^16:41
EmilienM_ody: but with xenial jobs, we run beaker against 4 (puppetlabs.com repo)16:41
_odyEmilienM: Yeah. I just meant that when we run beaker with puppet 4 it is against the latest release of puppet 4.16:41
EmilienMcool16:42
*** tesseract has quit IRC16:43
vinsh_ody: EmilienM Ok.. I have narrowed it down.. this error of " change from false to true failed: File written to disk did not match checksum; discarding changes (24 vs {md5}d41d8cd98f00b204e9800998ecf8427e) "16:45
EmilienMBTW I reproduced it in our CI16:46
vinshDOES NOT occur with puppet 4.3.2.  It first occurs with puppet 4.4.0 and later16:46
EmilienMif you want to see logs16:46
EmilienMvinsh: very nice catch16:46
openstackgerritMerged openstack/puppet-openstack-guide: Update meetings 85  https://review.openstack.org/32951916:46
openstackgerritIvan Berezovskiy proposed openstack/puppet-keystone: [Test] Keystone doesn't have service script anymore  https://review.openstack.org/32956716:55
iberezovskiyEmilienM, jfyi ^16:55
iberezovskiykeystone now is under apache by default in UCA16:55
EmilienMexcellent16:56
EmilienMiberezovskiy: also, can you maybe move our workaround to puppet-keystone?16:56
iberezovskiyI'll update commit later with case for OS, I just want to continue testing16:56
EmilienMok cool16:57
EmilienMthanks16:57
iberezovskiylet's make it work first, then decide where to move16:57
EmilienMI like your plan you know that?16:57
iberezovskiy?16:59
*** acabot has quit IRC16:59
iberezovskiy:)16:59
*** trown is now known as trown|lunch17:01
openstackgerritMerged openstack/puppet-tripleo: Implement Nova Compute profile  https://review.openstack.org/32723617:02
*** acabot has joined #puppet-openstack17:03
vinshopenstackgerrit: https://docs.puppet.com/puppet/4.4/reference/release_notes.html  search that page for "PUP-5695"17:07
vinshthat has got to be it17:07
vinsh_ody: ^17:08
vinshlol.. gerrit wonders why I'm talking to it.17:08
_odyHow on earth would be we passing a "24" to that new parameter?17:09
vinshI wish I knew!17:09
vinshpuppet/type/file.rb" 985L17:10
vinshis where the sum magic happens. I"m adding some debug to that to see what gives.17:11
EmilienMvinsh: nice catch17:12
vinshIt's the right area.. but unsure exactly whats busted still.17:12
openstackgerritIvan Berezovskiy proposed openstack/puppet-keystone: [WIP] Keystone doesn't have service script anymore in UCA  https://review.openstack.org/32956717:14
openstackgerritAdam Vinsh proposed openstack/puppet-swift: DNM TEST  https://review.openstack.org/32957917:15
openstackgerritAdam Vinsh proposed openstack/puppet-swift: DNM TEST  https://review.openstack.org/32957917:16
vinshhttps://review.openstack.org/#/c/329579/1..2/file.rb17:16
vinshdiff between 4.3.2 and 4.4.0 of the file type.17:16
*** ducttape_ has quit IRC17:17
EmilienMvinsh: do you know https://www.diffchecker.com ? lol17:18
vinshohhhh17:18
vinsheven easier17:18
_odyHah.  That is exactly what I was doing before you found that is was checksum_value.17:20
_odyTheir is a lost of code to wade through.17:20
*** dtantsur is now known as dtantsur|afk17:21
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Add ceilometer profiles  https://review.openstack.org/30514317:22
EmilienMbrb lunch17:22
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Add gnocchi profiles  https://review.openstack.org/31552717:23
*** ducttape_ has joined #puppet-openstack17:28
vinshopenstackgerrit: Line 857 of https://review.openstack.org/#/c/329579/1..2/file.rb17:31
vinshI added the old one back in  "write_content".. and added the write_content method back in17:31
vinshand it all works17:31
vinshusing property.write(file) in 4.4.0+ seems to be the breaking spot.17:32
openstackgerritIvan Berezovskiy proposed openstack/puppet-keystone: [WIP] Keystone doesn't have service script anymore in UCA  https://review.openstack.org/32956717:36
vinshInfact "write_content" returns a real md5sum. and the new "property.write()"  in 4.4.0+ returns "24"17:37
vinshI don't exactly understand what "property.write()" is..  is that the method at line 847 that is calling itself?17:39
vinshyes, it is.17:46
*** jesusaur has quit IRC17:48
*** jesusaur has joined #puppet-openstack17:49
*** jpena is now known as jpena|off17:51
iberezovskiyEmilienM, so probably this https://review.openstack.org/#/c/329567/ will help to fix scenarios for 4 puppet as well. I'll update it tomorrow. and of course we are waiting for keystone tempest package17:51
*** iberezovskiy is now known as iberezovskiy|off17:52
iberezovskiy|offI'm off for today17:52
EmilienMiberezovskiy|off: ack18:05
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker.  https://review.openstack.org/30906918:05
*** dmk0202 has joined #puppet-openstack18:09
*** richm has quit IRC18:10
*** trown|lunch is now known as trown18:11
openstackgerritMarcellin Fom Tchassem proposed openstack/puppet-neutron: Change device_driver to the neutron LBaasV2 version  https://review.openstack.org/32101818:23
*** priteau has joined #puppet-openstack18:24
*** priteau has quit IRC18:25
*** priteau has joined #puppet-openstack18:25
*** dgurtner has quit IRC18:26
*** richm has joined #puppet-openstack18:28
*** dmk0202 has quit IRC18:30
EmilienMmwhahaha: https://review.openstack.org/234798 is it really consistent with what we recently did in other modules?18:31
mwhahahano18:31
mwhahahait's not18:31
mwhahahathats why i blocked it18:31
mwhahahaand actually you reverted the glance one18:31
mwhahahacause it broke stuff18:32
mwhahahaso that one kinda conflicts with https://bugs.launchpad.net/puppet-nova/+bug/159004018:34
openstackLaunchpad bug 1590040 in puppet-manila "Modules *::keystone::auth should always pass auth_name to keystone::resource::service_identity and have a consistent service_name" [Medium,In progress] - Assigned to zhongshengping (chdzsp)18:34
mwhahahaso we went through and made them all match the name of the service18:34
mwhahahakeystone wasn't on that list18:34
*** johnmilton has quit IRC18:34
mwhahahaso techincally for #1590040 it should be 'keystone'18:34
EmilienMmwhahaha: I did https://github.com/openstack/puppet-glance/commit/80faf9a017dd7f4169867fbdae021866e4e3d5b618:35
mwhahahanow if in the long term we need to be renaming them to something else then we should do it all18:35
EmilienMand yeah it's conflicting18:35
mwhahahahttps://review.openstack.org/#/c/266024/18:35
mwhahaharevert18:35
EmilienMoh cinder18:35
openstackgerritMarcellin Fom Tchassem proposed openstack/puppet-neutron: Change device_driver to the neutron LBaasV2 version  https://review.openstack.org/32101818:36
mwhahahaso we should complete Bug 1590040 first, then figure out what we want to do for Bug 150606118:36
openstackbug 1590040 in puppet-manila "Modules *::keystone::auth should always pass auth_name to keystone::resource::service_identity and have a consistent service_name" [Medium,In progress] https://launchpad.net/bugs/1590040 - Assigned to zhongshengping (chdzsp)18:36
openstackbug 1506061 in puppet-nova "Modules should use *::kestone::auth default values for service names from Keystone's default catalog" [High,In progress] https://launchpad.net/bugs/1506061 - Assigned to Martin Mágr (mmagr)18:36
openstackgerritMerged openstack/puppet-congress: Commiting a cookie cutter module sync set of files  https://review.openstack.org/32832218:36
EmilienMradez: ^18:36
EmilienMmwhahaha: agree18:37
radezEmilienM: awesome, that good progress :)18:38
iurygregoryradez, congrats :D18:39
radeziurygregory: like you said before... lots of work to do now :)18:39
iurygregoryi'll do my best to review o/18:39
radezthx iurygregory and degorenko for working through getting these established18:39
radezand reviewed in the future :)18:40
iurygregoryyou're welcome =D18:40
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Move enable_sig and release to ceph::params  https://review.openstack.org/32954418:52
*** amoralej is now known as amoralej|off18:57
*** dmk0202 has joined #puppet-openstack19:05
*** xb_ has quit IRC19:09
*** xarses_ has quit IRC19:16
*** ayoung_ has joined #puppet-openstack19:34
vinsh_ody: I am so stumped on this.19:36
*** delattec has quit IRC19:45
*** derekh has quit IRC19:46
*** rwsu_ has joined #puppet-openstack19:58
*** rwsu has quit IRC20:00
*** rwsu_ is now known as rwsu20:04
*** ducttape_ has quit IRC20:07
*** cdelatte has joined #puppet-openstack20:12
colonwqI am having a slow brain moment today. Can any tell me why the new line 61 does not work? The value comes out as '' instead of 'jewel' https://review.openstack.org/#/c/329544/3/manifests/repo.pp,unified20:12
*** ducttape_ has joined #puppet-openstack20:16
*** ducttape_ has quit IRC20:16
*** ducttape_ has joined #puppet-openstack20:16
iurygregorylol i think it should work20:18
*** xarses_ has joined #puppet-openstack20:19
dfisherok, i'm back asking dumb questions about neutron, neutron_l3_agent_config, neutron_router and a name -> id function … somewhere.20:20
iurygregorycolonwq, try to put  inherits ceph::params20:20
dfisherneutron_router { 'name':20:20
dfisher    ensure => 'present',20:20
dfisher}20:20
dfisherworks fine but I need the damn ID from that to populate l3_agent.ini20:20
dfisherEmilienM: pointed me at https://github.com/openstack/puppet-tempest/blob/master/lib/puppet/provider/tempest_neutron_net_id_setter/openstack.rb last week20:21
colonwqiurygregory, that would go inside the class definition or before class(){?20:21
dfisherand 28:42 shows it, mostly.20:21
dfisherbut the neutron_l3_agent_config provider is just a subclass of ini_file20:21
dfisherand trivially simple20:21
dfisherso I don't know where to put a similar function20:22
iurygregorycolonwq, http://paste.openstack.org/show/516058/20:22
dfisherobviously, I need authentication to query Neutron20:22
openstackgerritAdam Vinsh proposed openstack/puppet-swift: swiftinit provider write file directly  https://review.openstack.org/32966620:22
vinsh@EmilienM _ody ^^^ Should do the trick for the swift issue.20:22
dfisherso I have to inherit from provider/neutron.rb to get auth_neutron()20:23
vinshI don't think it's worth fighting the file resource the way I was.20:23
colonwqiurygregory, I see. That is very c++ of it. I'll give it a try.20:23
EmilienMvinsh: ack, running experimental job to see if it works20:23
EmilienMvinsh: see my comment20:23
*** xarses_ has quit IRC20:24
*** xarses_ has joined #puppet-openstack20:24
iurygregorycolonwq, i hope it helps =)20:24
openstackgerritAdam Vinsh proposed openstack/puppet-swift: swiftinit provider write file directly  https://review.openstack.org/32966620:24
vinshjust rebased on top of 32907420:25
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Move enable_sig and release to ceph::params  https://review.openstack.org/32954420:25
*** dmk0202 has quit IRC20:26
colonwqiurygregory, we will find out in a while.20:26
*** dmk0202 has joined #puppet-openstack20:26
*** xarses_ has quit IRC20:27
*** xarses_ has joined #puppet-openstack20:27
*** paramite has quit IRC20:28
*** dmk0202 has quit IRC20:33
EmilienMvinsh: looks like it works20:36
EmilienMCI job is still running (looking at zuul)20:37
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: opencontrail: update keystone section  https://review.openstack.org/32907920:39
vinshEmilienM: Good!  Something about expecting puppet resources to compile correctly from within a provider... is probably a bad idea anyway.20:41
vinshI now know WAY too much about the puppet file type/provider innards.20:41
*** saju_m has joined #puppet-openstack20:43
*** saju_m has quit IRC20:44
*** saju_m has joined #puppet-openstack20:44
*** saju_m has quit IRC20:45
*** saju_m has joined #puppet-openstack20:46
*** saneax_AFK is now known as saneax20:46
*** saju_m has quit IRC20:46
*** jayg is now known as jayg|g0n320:47
EmilienMvinsh: it failed20:47
EmilienMhttp://logs.openstack.org/66/329666/2/experimental/gate-puppet-swift-puppet-beaker-rspec-ubuntu-xenial-nv/1d964ce/console.html#_2016-06-14_20_45_17_53220:47
*** saju_m has joined #puppet-openstack20:48
*** saju_m has quit IRC20:49
EmilienMvinsh: it sounds like idempotency?20:50
*** saju_m has joined #puppet-openstack20:50
*** saju_m has quit IRC20:50
*** yolanda has quit IRC20:52
*** shardy has quit IRC20:53
_odyvinsh: The place that puppet is checking for the systemd unit file in enabled? is different than where enable is placing the file. I think line 88 is just a typo, looking at the diff.20:56
*** saju_m has joined #puppet-openstack20:56
*** ayoung_ has quit IRC20:57
*** ayoung has quit IRC20:57
*** saju_m has quit IRC20:57
*** saju_m has joined #puppet-openstack20:58
*** saju_m has quit IRC20:58
*** trown is now known as trown|outtypewww20:59
*** fc___ has joined #puppet-openstack20:59
*** fc__ has quit IRC21:02
*** fc___ is now known as fc__21:02
zigoEmilienM: I have finished all of Newton.21:05
iurygregorycolonwq, \o/ it passed but with failure in non-voting for beaker =X21:05
zigoNeutron's unit tests are completely broken, but it's looking like it's only unit tests.21:06
zigoEverything else should be just fine.21:06
zigoSo, where can I modify the CI to test all of that? :P21:06
iurygregoryzigo, puppet-openstack-integration =D21:07
zigoOk.21:07
openstackgerritAdam Vinsh proposed openstack/puppet-swift: swiftinit provider write file directly  https://review.openstack.org/32966621:07
iurygregoryyou need to update the repo right?21:07
vinsh_ody: EmilienM I was delirious and did typo.21:07
colonwqiurygregory, Cool. Thanks for the help.21:11
* colonwq is gone for a few hours21:11
iurygregorycolonwq, np =D21:11
dfisheranybody have any ideas about my router name -> id -> l3-agent.ini issue? :(21:12
iurygregorydfisher, i don't have sorry =(21:13
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Implement Libvirt profile  https://review.openstack.org/32968221:22
*** haaj has joined #puppet-openstack21:38
*** cdelatte has quit IRC21:39
openstackgerritBradley Klein proposed openstack/puppet-monasca: Ensure all servers are listed (beyond osapi_max_limit)  https://review.openstack.org/32969321:57
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Enable compute node micro-services  https://review.openstack.org/32969422:07
*** priteau has quit IRC22:17
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Enable compute node micro-services  https://review.openstack.org/32969422:22
*** openstackstatus has quit IRC22:25
*** openstack has joined #puppet-openstack22:27
*** openstackstatus has joined #puppet-openstack22:27
*** ChanServ sets mode: +v openstackstatus22:27
*** rcernin has joined #puppet-openstack22:28
*** xarses has joined #puppet-openstack22:30
*** xarses_ has quit IRC22:30
*** furlongm_ has joined #puppet-openstack22:37
*** furlongm has quit IRC22:37
*** iurygregory_ has joined #puppet-openstack22:40
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Enable compute node micro-services  https://review.openstack.org/32969422:40
*** ducttape_ has quit IRC22:43
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Enable compute node micro-services  https://review.openstack.org/32969422:45
*** dfisher has quit IRC22:58
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Enable compute node micro-services  https://review.openstack.org/32969423:08
*** ayoung has joined #puppet-openstack23:17
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Create libvirt micro-service  https://review.openstack.org/32971423:21
*** xarses has quit IRC23:37
*** rcernin has quit IRC23:40

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