Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2016-06-14

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:12 openstackgerrit Dan Radez proposed openstack/puppet-tacker: Commiting a cookie cutter module sync set of files  https://review.openstack.org/329230
00:17 openstackgerrit Dan Radez proposed openstack/puppet-tacker: Commiting a cookie cutter module sync set of files  https://review.openstack.org/329230
01:47 ilbot3 joined #puppet-openstack
01:47 Topic for #puppet-openstack is now CI status: green - Puppet OpenStack modules - http://docs.openstack.org/developer/puppet-openstack-guide/
01:57 xarses joined #puppet-openstack
02:33 EmilienM mwhahaha: should we merge https://review.openstack.org/#/c/234798/9?
02:33 EmilienM I'm about to -2 it
02:41 mwhahaha Probably should stop that
02:44 mwhahaha I blocked it, I'll look into it more in the morning
02:55 iurygregory_ lol i have recived the email with +2 from jenkis .-.
02:56 iurygregory_ but tks god we don't have the commit in the repo =D
03:11 openstackgerrit joined #puppet-openstack
03:15 openstackgerrit Matt Fischer proposed openstack/puppet-openstack-guide: Document Emilien's tips for new modules  https://review.openstack.org/329253
04:14 xarses_ joined #puppet-openstack
04:17 andreww joined #puppet-openstack
04:18 andreww joined #puppet-openstack
05:11 jaosorior joined #puppet-openstack
05:17 markvoelker joined #puppet-openstack
05:22 openstackgerrit Dan Radez proposed openstack/puppet-tacker: Commiting a cookie cutter module sync set of files  https://review.openstack.org/329230
05:23 openstackgerrit Dan Radez proposed openstack/puppet-tacker: Commiting a cookie cutter module sync set of files  https://review.openstack.org/329230
05:23 openstackgerrit Dan Radez proposed openstack/puppet-congress: Commiting a cookie cutter module sync set of files  https://review.openstack.org/328322
05:45 jmikolajczak joined #puppet-openstack
06:11 rcernin joined #puppet-openstack
06:11 openstackgerrit OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/329275
06:12 openstackgerrit OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/328591
06:28 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova vncproxy profile  https://review.openstack.org/326130
06:28 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova Consoleauth profile  https://review.openstack.org/326112
06:28 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova Compute profile  https://review.openstack.org/327236
06:28 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova Scheduler profile  https://review.openstack.org/326092
06:28 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova API profile  https://review.openstack.org/325937
06:31 dgurtner joined #puppet-openstack
06:31 dgurtner joined #puppet-openstack
06:48 openstackgerrit joined #puppet-openstack
06:51 ducttape_ joined #puppet-openstack
06:54 openstackgerrit Merged openstack/puppet-magnum: Fix magnum-db-sync orchestration  https://review.openstack.org/329018
06:56 tesseract joined #puppet-openstack
07:13 openstackgerrit Merged openstack/puppet-openstack-integration: Ensure $GEM_BIN_DIR is in $PATH  https://review.openstack.org/329222
07:14 xarses_ joined #puppet-openstack
07:18 markvoelker joined #puppet-openstack
07:23 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add fact to get the fqdn for a host in the different networks  https://review.openstack.org/329299
07:30 karimb joined #puppet-openstack
07:31 hjensas__ joined #puppet-openstack
07:32 dmk0202 joined #puppet-openstack
07:32 karimb joined #puppet-openstack
07:32 ifarkas joined #puppet-openstack
07:55 openstackgerrit Merged openstack/puppet-tripleo: Add mongodb profiles  https://review.openstack.org/304780
08:04 shardy joined #puppet-openstack
08:09 danpawlik joined #puppet-openstack
08:16 queretaro joined #puppet-openstack
08:23 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Add gnocchi profiles  https://review.openstack.org/315527
08:31 chem`` joined #puppet-openstack
08:40 jaosorior_brb joined #puppet-openstack
08:40 derekh joined #puppet-openstack
08:52 ducttape_ joined #puppet-openstack
08:56 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova vncproxy profile  https://review.openstack.org/326130
08:56 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova Consoleauth profile  https://review.openstack.org/326112
08:56 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova Compute profile  https://review.openstack.org/327236
08:56 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova Scheduler profile  https://review.openstack.org/326092
08:56 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Implement Nova API profile  https://review.openstack.org/325937
09:08 dgurtner joined #puppet-openstack
09:24 rochaporto joined #puppet-openstack
09:31 openstackgerrit Merged openstack/puppet-keystone: Add keystone::wsgi::apache::admin_bind_host param  https://review.openstack.org/329114
09:38 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Composable roles within services - NTP  https://review.openstack.org/310725
09:39 openstackgerrit Oleksiy Molchanov proposed openstack/puppet-ceph: Fix activate check  https://review.openstack.org/310507
09:45 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Add ceilometer profiles  https://review.openstack.org/305143
09:46 openstackgerrit Denis Egorenko proposed openstack/puppet-murano: Use regexp by whole name for murano_application  https://review.openstack.org/329343
09:54 dtantsur degorenko, morning! could you take a look at https://review.openstack.org/#/c/329087/ please? it blocks some tripleo work.
09:55 degorenko dtantsur, morning, sure
09:57 degorenko dtantsur, lgtm :)
09:57 dtantsur thanks!
10:04 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker.  https://review.openstack.org/309069
10:12 xb_ joined #puppet-openstack
10:16 openstackgerrit Merged openstack/puppet-neutron: beaker: fix resource ordering  https://review.openstack.org/329033
10:27 openstackgerrit Merged openstack/puppet-rally: Manage SQLite database directory  https://review.openstack.org/317939
10:28 openstackgerrit Boris Kreitchman proposed openstack/puppet-tripleo: Add a Cinder backup and a Cinder backup Ceph profiles.  https://review.openstack.org/311218
10:38 openstackgerrit Merged openstack/puppet-ironic: stop managing /etc/ironic and ironic.conf  https://review.openstack.org/329087
10:42 openstackgerrit Merged openstack/puppet-octavia: Implement api.pp  https://review.openstack.org/326142
10:46 openstackgerrit Merged openstack/puppet-octavia: Update db_sync command  https://review.openstack.org/328659
10:53 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/327678
10:54 ducttape_ joined #puppet-openstack
10:57 jaosorior degorenko: Hello; quick question, why did you try a re-check in https://review.openstack.org/#/c/321818/ ?
10:58 degorenko jaosorior, hey, is your tripleo patch merged? Or not yet? i lost link :D
10:58 jaosorior degorenko: It can't merge because it depends on the puppet-nova patch
10:58 degorenko jaosorior, ah, got it now
10:59 jaosorior degorenko: haha yeah, it can be a bit confusing. But yeah, that puppet-nova patch should be ready to merge now.
10:59 degorenko jaosorior, ok, +2'ed :)
10:59 jaosorior degorenko: Awesome, thanks man
11:02 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add fact to get the fqdn for a host in the different networks  https://review.openstack.org/329299
11:05 openstackgerrit Xingchao Yu proposed openstack/puppet-rally: Add rally::db::sync  https://review.openstack.org/317970
11:32 kullmar joined #puppet-openstack
11:38 openstackgerrit Alexey Deryugin proposed openstack/puppet-openstack-integration: Add tempest-horizon plugin installation  https://review.openstack.org/329388
11:40 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/327678
11:42 openstackgerrit Alexey Deryugin proposed openstack/puppet-openstack-integration: Add tempest-horizon plugin installation  https://review.openstack.org/329388
11:46 EmilienM hello
11:46 iberezovskiy|off hi
11:52 openstackgerrit Alexey Deryugin proposed openstack/puppet-openstack-integration: scenario003: enable application catalog service  https://review.openstack.org/250436
11:54 ducttape_ joined #puppet-openstack
11:55 openstackgerrit Xingchao Yu proposed openstack/puppet-rally: Add rally::db::sync  https://review.openstack.org/317970
12:03 openstackgerrit Merged openstack/puppet-aodh: beaker: fix ordering for RabbitMQ  https://review.openstack.org/329037
12:05 openstackgerrit Merged openstack/puppet-trove: beaker: disable taskmanager on Xenial  https://review.openstack.org/329020
12:07 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/327678
12:10 EmilienM iberezovskiy: still debugging? lol
12:11 iberezovskiy it was a bad idea to debug it at sunday midnight
12:11 iberezovskiy we used wrong dir name
12:11 iberezovskiy it should be siteS-*, not site-*
12:12 EmilienM ah :(
12:12 EmilienM degorenko: no need to do recheck https://review.openstack.org/#/c/328591/
12:12 markvoelker joined #puppet-openstack
12:13 EmilienM we need to inspect logs and find what's wrong
12:20 paramite joined #puppet-openstack
12:24 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/328591
12:25 iberezovskiy EmilienM, take a look please on the latest patch set, looks like it works but tempest has failed :D
12:25 openstackgerrit Merged openstack/puppet-nova: providers: add hack to require nova providers outside puppet-nova  https://review.openstack.org/328525
12:25 iberezovskiy http://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_701
12:28 * iberezovskiy bbs ~30 min
12:30 fwdit joined #puppet-openstack
12:30 iurygregory morning people =)
12:40 EmilienM iberezovskiy: where should I look?
12:40 EmilienM ah
12:40 EmilienM I missed the link, sorry
12:41 EmilienM it's in keystone
12:41 EmilienM http://codesearch.openstack.org/?q=keystone_tempest_plugin&i=nope&files=&repos=
12:41 johnmilton joined #puppet-openstack
12:41 EmilienM maybe we need to unpin tempest
12:45 openstackgerrit Brad P. Crochet proposed openstack/puppet-tripleo: Split Heat pacemaker roles into separate services  https://review.openstack.org/327708
12:47 queretaro joined #puppet-openstack
12:49 openstackgerrit Alexey Deryugin proposed openstack/puppet-openstack-integration: Add tempest-horizon plugin installation  https://review.openstack.org/329388
12:51 openstackgerrit Marcellin Fom Tchassem proposed openstack/puppet-swift: Change default service_name to 'swift'  https://review.openstack.org/327252
12:58 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Add transport_url and rpc_response_timeout for oslo.messaging  https://review.openstack.org/321818
12:59 EmilienM jaosorior: it was failing Ci, rebasing and re approving ^
12:59 jaosorior EmilienM: alright
13:00 openstackgerrit Marcellin Fom Tchassem proposed openstack/puppet-swift: Change default service_name to 'swift'  https://review.openstack.org/327252
13:07 iberezovskiy EmilienM, I'll check locally first
13:07 dgurtner joined #puppet-openstack
13:07 EmilienM iberezovskiy: see on #ubuntu-server
13:08 EmilienM iberezovskiy: 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 too
13:08 iberezovskiy ok, I'll work on this
13:09 mnaser morning, 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:21 EmilienM iberezovskiy: do you want to cleanup https://review.openstack.org/#/c/327678/25/copy_logs.sh ?
13:21 iberezovskiy EmilienM, yeah, sure
13:22 EmilienM iberezovskiy: otherwise your patch looks fine.
13:22 openstackgerrit Dan Radez proposed openstack/puppet-tacker: Commiting a cookie cutter module sync set of files  https://review.openstack.org/329230
13:24 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/327678
13:27 iurygregory EmilienM, do you know Dan Radez irc? O.o
13:29 ericksonsantos joined #puppet-openstack
13:31 radez joined #puppet-openstack
13:31 EmilienM iurygregory: ^
13:32 iurygregory LOL
13:32 iurygregory tks EmilienM
13:32 radez hey iurygregory, thx for your reviews :)
13:33 iurygregory radez, o/ np tks for your work and patience
13:33 radez sure thing
13:33 openstackgerrit Merged openstack/puppet-nova: beaker: don't deploy virtlock and virtlog  https://review.openstack.org/328809
13:34 * iurygregory brb
13:36 openstackgerrit Alex Schultz proposed openstack/puppet-keystone: Preparation for default endpoint names rename  https://review.openstack.org/234798
13:36 richm joined #puppet-openstack
13:41 dfisher joined #puppet-openstack
13:42 openstackgerrit Brad P. Crochet proposed openstack/puppet-tripleo: Add Mistral profiles  https://review.openstack.org/323431
13:45 xarses_ joined #puppet-openstack
13:54 openstackgerrit Merged openstack/puppet-rally: Add rally::db::sync  https://review.openstack.org/317970
13:56 cdelatte joined #puppet-openstack
14:01 dhollinger joined #puppet-openstack
14:05 dfisher so, 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:06 EmilienM mwhahaha: fyi /me afk after our weekly meeting until tomorrow morning or late tonight
14:06 mwhahaha k
14:08 xarses_ joined #puppet-openstack
14:09 xarses_ joined #puppet-openstack
14:12 ducttape_ joined #puppet-openstack
14:13 ducttape_ joined #puppet-openstack
14:17 openstackgerrit Dan Radez proposed openstack/puppet-congress: Commiting a cookie cutter module sync set of files  https://review.openstack.org/328322
14:19 openstackgerrit Dan Radez proposed openstack/puppet-tacker: Commiting a cookie cutter module sync set of files  https://review.openstack.org/329230
14:19 osp joined #puppet-openstack
14:29 openstackgerrit Merged openstack/puppet-nova: Add transport_url and rpc_response_timeout for oslo.messaging  https://review.openstack.org/321818
14:31 osp Can 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 openstackgerrit Merged openstack/puppet-tripleo: Implement Nova API profile  https://review.openstack.org/325937
14:34 Lokesh_Jain joined #puppet-openstack
14:35 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/327678
14:35 iberezovskiy EmilienM, while we are waiting, just try for alternative solution ^
14:36 EmilienM osp: you would be luckier on #tripleo
14:36 ayoung joined #puppet-openstack
14:43 jmiu osp: you on #tripleo?
14:44 xarses_ joined #puppet-openstack
14:48 osp EmilienM: thanks i will ask on there.
14:48 openstackgerrit Merged openstack/puppet-tripleo: Implement Nova Scheduler profile  https://review.openstack.org/326092
14:48 openstackgerrit Merged openstack/puppet-tripleo: Implement Nova Consoleauth profile  https://review.openstack.org/326112
14:48 osp jmiu: yes I am, will ask on there. Thanks
14:51 openstackgerrit Merged openstack/puppet-tripleo: Implement Nova vncproxy profile  https://review.openstack.org/326130
14:55 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Deploy nova::db classes in api profile  https://review.openstack.org/326027
14:56 iurygregory Almost meeting time people, #openstack-meeting-4
14:57 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: profile/base/nova: declare nova class properly  https://review.openstack.org/328347
14:58 _ody joined #puppet-openstack
14:58 openstackgerrit Merged openstack/puppet-watcher: This commit adds the initial project files.  https://review.openstack.org/326593
14:59 iurygregory danpawlik, ^ congrats
14:59 danpawlik iurygregory: yay :D
15:00 _stowa joined #puppet-openstack
15:05 radez degorenko: thx for putting my modules on the meeting adgenda today
15:06 degorenko radez, please join on #openstack-meeting-4 :)
15:06 radez degorenko: I'm in there
15:07 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/327678
15:13 radez degorenko: the module is coming from some where, so I'm not sure that we're brakeing anyone
15:13 radez they can just used the previous location until the official location is ready for use.
15:14 EmilienM vinsh: you're missing our meeting
15:14 EmilienM and we talk about you
15:14 radez degorenko: 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 idea
15:14 vinsh Danggit
15:14 radez that EmilienM is suggesting that modules should be started in puppet-openstack's forge
15:16 openstackgerrit Merged openstack/puppet-murano: Use regexp by whole name for murano_application  https://review.openstack.org/329343
15:16 degorenko radez, wait please for end of meeting :D
15:17 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.1
15:18 vinsh I can't figure out what is going on... where is it getting a value of "24" for an md5sum?
15:18 _ody Ok. Got them up in browser.
15:20 vinsh I 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:23 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-guide: Update meetings 85  https://review.openstack.org/329519
15:23 iurygregory EmilienM,
15:23 degorenko radez, well, ok, if there are no any problem with removing old stuff :)
15:24 * radez shrugs
15:24 radez I see valid arguments for both ways
15:24 radez but since the modules are out of the norm of the operations of establishing them
15:24 radez I suppose starting with a blank canvas but preserving the history could make up for that somewhat
15:25 radez degorenko: ^
15:25 degorenko radez, btw, why you have different commit message from cookiecutter commit message template? :)
15:25 xarses_ joined #puppet-openstack
15:25 radez degorenko: not sure I understand that you're asking
15:25 EmilienM iurygregory: etherpad link is wrong
15:26 radez degorenko: oh you mean I'm not using the cookie cutter git history?
15:26 degorenko radez, i meant, that your commit message is not from https://github.com/openstack/puppet-openstack-cookiecutter/blob/master/contrib/bootstrap.sh#L138
15:26 radez degorenko: I just copied the content into the existing repo's git hitory and made a new commit to preserve the old history
15:26 degorenko radez, did you run this script?
15:27 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-guide: Update meetings 85  https://review.openstack.org/329519
15:27 radez I did, run it, but copied the content into the existing repo I had to preseve the histsory of where it came from degorenko
15:27 degorenko hm, ok then
15:28 radez degorenko: would you prefer a different way?
15:28 radez I"m not sure that from the import it gerrit would allow it a different way
15:28 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Add gnocchi profiles  https://review.openstack.org/315527
15:28 degorenko radez, well, i'm ok with current way. How can guaranty that we don't miss something? :)
15:29 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Add ceilometer profiles  https://review.openstack.org/305143
15:29 radez well in the patches surrent states they match the content from cc/msync
15:29 radez *current
15:29 iurygregory EmilienM, fixed =D
15:29 degorenko radez, yes, i'm about next patches
15:29 degorenko have you prepared local copy of current repo? :D
15:29 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Composable roles within services - NTP  https://review.openstack.org/310725
15:30 radez degorenko: I figure we'll take the content from the history and start to overlay it over the cc/msync patch
15:30 radez so that the development process looks as if we started from scratch
15:30 _ody vinsh: Ok.  Doing a fresh beaker run locally so I can log in and figure out what is going on.
15:30 radez degorenko: then the previous history is just there for preservation sake
15:30 degorenko radez, ok to me
15:31 vinsh _ody: ACK
15:31 radez kk, 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 content
15:31 radez iurygregory: degorenko: ^^ agreed
15:31 radez ?
15:32 iurygregory radez, thinking ...
15:32 radez :)
15:32 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/327678
15:32 degorenko radez, i'm reviewing your current patches :D
15:32 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: nova/api: include ::nova::network::neutron  https://review.openstack.org/329529
15:32 * radez nods
15:32 * radez brb
15:33 iurygregory radez, i think i'm ok =)
15:34 degorenko radez, one minor comment :)
15:34 jtomasek_ joined #puppet-openstack
15:34 iurygregory degorenko, he is following cc =P
15:35 degorenko iurygregory, well, then we must note in docs - that users should read doc sections before they did git review
15:36 degorenko init commit should not contain any pattern stuff from msycn and cookie cutter, imho
15:36 radez would it make more sense to delete the repos and recreate them empty
15:36 iurygregory degorenko, +1 from me and also i cant find some files in the commit .-.
15:36 radez and throw away the previous history?
15:36 degorenko iurygregory, probably they are already created
15:36 degorenko EmilienM, thoughts about removing and re-creating repo ?
15:37 iurygregory radez, not throw away! (you should make a copy locally)
15:37 radez well yes, I have a local copy
15:37 EmilienM degorenko: really, no
15:37 EmilienM I mean
15:37 EmilienM do not patch infra
15:37 iurygregory degorenko, i think this is like my idea =P
15:37 EmilienM if you want to rewrite the module in repo, ok
15:37 iurygregory create a commit removing files..
15:37 degorenko that's can be a solution
15:37 iurygregory and another adding cc/mysnc
15:37 iurygregory =)
15:38 radez but will that pass ci?
15:38 iurygregory wow good question LOL
15:38 EmilienM I haven't followed all discussion but it's too hard to rewrite it?
15:38 radez we're at gerrit's mercy
15:38 permalac joined #puppet-openstack
15:38 iurygregory i think it will pass
15:38 radez we could try
15:38 degorenko radez, well, we can leave it just with rewritting with common cookie stuff
15:39 radez I think either way will accomplish the same thing
15:39 degorenko current commits are good
15:39 radez as long as we have a commit that is pure cc/msync
15:39 radez we're "starting" from a clean slate
15:39 degorenko radez you can change your title to smthg like : switch to openstack sytle or whatever
15:39 degorenko style*
15:40 degorenko some title, that user can understand why we are rewritting module
15:40 degorenko i guess it will be enough
15:40 radez sure, would you like me to copy the commit message from contrib.sh?
15:40 radez oh I see what you're saying
15:40 degorenko radez, that's not important though, just asking :)
15:40 radez yea I'll be more verbose in the commit msg
15:40 degorenko yes
15:40 iurygregory tks radez
15:40 degorenko and clean up unneeded doc in init.pp :D
15:41 * iurygregory brb lunch time
15:41 radez kk
15:41 radez thx degorenko iurygregory
15:42 radez I'll get a new patch set up for both modules
15:42 degorenko sure, np
15:44 delattec joined #puppet-openstack
15:46 openstackgerrit Dan Radez proposed openstack/puppet-tacker: Commiting a cookie cutter module sync set of files  https://review.openstack.org/329230
15:47 openstackgerrit Dan Radez proposed openstack/puppet-congress: Commiting a cookie cutter module sync set of files  https://review.openstack.org/328322
15:53 delattec joined #puppet-openstack
15:55 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Move enable_sig and release to ceph::params  https://review.openstack.org/329544
16:07 EmilienM mwhahaha: flight cancelled, I'll be here :(
16:07 mwhahaha :(
16:08 EmilienM too much wind
16:10 iurygregory EmilienM, you're at the airport? O.o
16:10 EmilienM iurygregory: no, but I had a pilot lesson this afternoon, it's cancelled :(
16:12 iurygregory EmilienM, wow very cool :D
16:12 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add all other classes  https://review.openstack.org/314598
16:24 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker.  https://review.openstack.org/309069
16:25 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Move enable_sig and release to ceph::params  https://review.openstack.org/329544
16:26 rwsu joined #puppet-openstack
16:36 _ody vinsh: sigh...I got a whole slew of different errors when I ran the beaker tests outside of OpenStack CI.
16:37 vinsh I can probably help get you going.. I'm running on just a stand alone vm.
16:37 vinsh I also find that version 4.2.3 works
16:37 vinsh then 4.3.0 fails on concat errors I am tracking down
16:38 vinsh I'm moving past the concat error to see where this specific md5sum error comes in on 4.3 branch
16:40 _ody vinsh: I thought we only ran beaker against puppet 4 latest.
16:40 vinsh That 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 _ody ah.
16:41 EmilienM vinsh: we run beaker against 3
16:41 EmilienM err _ody ^
16:41 EmilienM _ody: but with xenial jobs, we run beaker against 4 (puppetlabs.com repo)
16:41 _ody EmilienM: Yeah. I just meant that when we run beaker with puppet 4 it is against the latest release of puppet 4.
16:42 EmilienM cool
16:45 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:46 EmilienM BTW I reproduced it in our CI
16:46 vinsh DOES NOT occur with puppet 4.3.2.  It first occurs with puppet 4.4.0 and later
16:46 EmilienM if you want to see logs
16:46 EmilienM vinsh: very nice catch
16:46 openstackgerrit Merged openstack/puppet-openstack-guide: Update meetings 85  https://review.openstack.org/329519
16:55 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-keystone: [Test] Keystone doesn't have service script anymore  https://review.openstack.org/329567
16:55 iberezovskiy EmilienM, jfyi ^
16:55 iberezovskiy keystone now is under apache by default in UCA
16:56 EmilienM excellent
16:56 EmilienM iberezovskiy: also, can you maybe move our workaround to puppet-keystone?
16:56 iberezovskiy I'll update commit later with case for OS, I just want to continue testing
16:57 EmilienM ok cool
16:57 EmilienM thanks
16:57 iberezovskiy let's make it work first, then decide where to move
16:57 EmilienM I like your plan you know that?
16:59 iberezovskiy ?
16:59 iberezovskiy :)
17:02 openstackgerrit Merged openstack/puppet-tripleo: Implement Nova Compute profile  https://review.openstack.org/327236
17:03 acabot joined #puppet-openstack
17:07 vinsh openstackgerrit: https://docs.puppet.com/puppet/4.4/reference/release_notes.html  search that page for "PUP-5695"
17:07 vinsh that has got to be it
17:08 vinsh _ody: ^
17:08 vinsh lol.. gerrit wonders why I'm talking to it.
17:09 _ody How on earth would be we passing a "24" to that new parameter?
17:09 vinsh I wish I knew!
17:10 vinsh puppet/type/file.rb" 985L
17:11 vinsh is where the sum magic happens. I"m adding some debug to that to see what gives.
17:12 EmilienM vinsh: nice catch
17:12 vinsh It's the right area.. but unsure exactly whats busted still.
17:14 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-keystone: [WIP] Keystone doesn't have service script anymore in UCA  https://review.openstack.org/329567
17:15 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: DNM TEST  https://review.openstack.org/329579
17:16 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: DNM TEST  https://review.openstack.org/329579
17:16 vinsh https://review.openstack.org/#/c/329579/1..2/file.rb
17:16 vinsh diff between 4.3.2 and 4.4.0 of the file type.
17:18 EmilienM vinsh: do you know https://www.diffchecker.com ? lol
17:18 vinsh ohhhh
17:18 vinsh even easier
17:19 _ody Hah.  That is exactly what I was doing before you found that is was checksum_value.
17:20 _ody Their is a lost of code to wade through.
17:22 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Add ceilometer profiles  https://review.openstack.org/305143
17:22 EmilienM brb lunch
17:23 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Add gnocchi profiles  https://review.openstack.org/315527
17:28 ducttape_ joined #puppet-openstack
17:31 vinsh openstackgerrit: Line 857 of https://review.openstack.org/#/c/329579/1..2/file.rb
17:31 vinsh I added the old one back in  "write_content".. and added the write_content method back in
17:31 vinsh and it all works
17:32 vinsh using property.write(file) in 4.4.0+ seems to be the breaking spot.
17:36 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-keystone: [WIP] Keystone doesn't have service script anymore in UCA  https://review.openstack.org/329567
17:37 vinsh Infact "write_content" returns a real md5sum. and the new "property.write()"  in 4.4.0+ returns "24"
17:39 vinsh I don't exactly understand what "property.write()" is..  is that the method at line 847 that is calling itself?
17:46 vinsh yes, it is.
17:49 jesusaur joined #puppet-openstack
17:51 iberezovskiy EmilienM, 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 package
17:52 iberezovskiy|off I'm off for today
18:05 EmilienM iberezovskiy|off: ack
18:05 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker.  https://review.openstack.org/309069
18:09 dmk0202 joined #puppet-openstack
18:23 openstackgerrit Marcellin Fom Tchassem proposed openstack/puppet-neutron: Change device_driver to the neutron LBaasV2 version  https://review.openstack.org/321018
18:24 priteau joined #puppet-openstack
18:25 priteau joined #puppet-openstack
18:28 richm joined #puppet-openstack
18:31 EmilienM mwhahaha: https://review.openstack.org/234798 is it really consistent with what we recently did in other modules?
18:31 mwhahaha no
18:31 mwhahaha it's not
18:31 mwhahaha thats why i blocked it
18:31 mwhahaha and actually you reverted the glance one
18:32 mwhahaha cause it broke stuff
18:33 mwhahaha so that one kinda conflicts with https://bugs.launchpad.net/puppet-nova/+bug/1590040
18:34 openstack Launchpad 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 mwhahaha so we went through and made them all match the name of the service
18:34 mwhahaha keystone wasn't on that list
18:34 mwhahaha so techincally for #1590040 it should be 'keystone'
18:35 EmilienM mwhahaha: I did https://github.com/openstack/puppet-glance/commit/80faf9a017dd7f4169867fbdae021866e4e3d5b6
18:35 mwhahaha now if in the long term we need to be renaming them to something else then we should do it all
18:35 EmilienM and yeah it's conflicting
18:35 mwhahaha https://review.openstack.org/#/c/266024/
18:35 mwhahaha revert
18:35 EmilienM oh cinder
18:36 openstackgerrit Marcellin Fom Tchassem proposed openstack/puppet-neutron: Change device_driver to the neutron LBaasV2 version  https://review.openstack.org/321018
18:36 mwhahaha so we should complete Bug 1590040 first, then figure out what we want to do for Bug 1506061
18:36 openstack 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] https://launchpad.net/bugs/1590040 - Assigned to zhongshengping (chdzsp)
18:36 openstack bug 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 openstackgerrit Merged openstack/puppet-congress: Commiting a cookie cutter module sync set of files  https://review.openstack.org/328322
18:36 EmilienM radez: ^
18:37 EmilienM mwhahaha: agree
18:38 radez EmilienM: awesome, that good progress :)
18:39 iurygregory radez, congrats :D
18:39 radez iurygregory: like you said before... lots of work to do now :)
18:39 iurygregory i'll do my best to review o/
18:39 radez thx iurygregory and degorenko for working through getting these established
18:40 radez and reviewed in the future :)
18:40 iurygregory you're welcome =D
18:52 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Move enable_sig and release to ceph::params  https://review.openstack.org/329544
19:05 dmk0202 joined #puppet-openstack
19:34 ayoung_ joined #puppet-openstack
19:36 vinsh _ody: I am so stumped on this.
19:58 rwsu_ joined #puppet-openstack
20:12 cdelatte joined #puppet-openstack
20:12 colonwq I 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,unified
20:16 ducttape_ joined #puppet-openstack
20:16 ducttape_ joined #puppet-openstack
20:18 iurygregory lol i think it should work
20:19 xarses_ joined #puppet-openstack
20:20 dfisher ok, i'm back asking dumb questions about neutron, neutron_l3_agent_config, neutron_router and a name -> id function … somewhere.
20:20 iurygregory colonwq, try to put  inherits ceph::params
20:20 dfisher neutron_router { 'name':
20:20 dfisher ensure => 'present',
20:20 dfisher }
20:20 dfisher works fine but I need the damn ID from that to populate l3_agent.ini
20:21 dfisher EmilienM: pointed me at https://github.com/openstack/puppet-tempest/blob/master/lib/puppet/provider/tempest_neutron_net_id_setter/openstack.rb last week
20:21 colonwq iurygregory, that would go inside the class definition or before class(){?
20:21 dfisher and 28:42 shows it, mostly.
20:21 dfisher but the neutron_l3_agent_config provider is just a subclass of ini_file
20:21 dfisher and trivially simple
20:22 dfisher so I don't know where to put a similar function
20:22 iurygregory colonwq, http://paste.openstack.org/show/516058/
20:22 dfisher obviously, I need authentication to query Neutron
20:22 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: swiftinit provider write file directly  https://review.openstack.org/329666
20:22 vinsh @EmilienM _ody ^^^ Should do the trick for the swift issue.
20:23 dfisher so I have to inherit from provider/neutron.rb to get auth_neutron()
20:23 vinsh I don't think it's worth fighting the file resource the way I was.
20:23 colonwq iurygregory, I see. That is very c++ of it. I'll give it a try.
20:23 EmilienM vinsh: ack, running experimental job to see if it works
20:23 EmilienM vinsh: see my comment
20:24 xarses_ joined #puppet-openstack
20:24 iurygregory colonwq, i hope it helps =)
20:24 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: swiftinit provider write file directly  https://review.openstack.org/329666
20:25 vinsh just rebased on top of 329074
20:25 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Move enable_sig and release to ceph::params  https://review.openstack.org/329544
20:26 colonwq iurygregory, we will find out in a while.
20:26 dmk0202 joined #puppet-openstack
20:27 xarses_ joined #puppet-openstack
20:36 EmilienM vinsh: looks like it works
20:37 EmilienM CI job is still running (looking at zuul)
20:39 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: opencontrail: update keystone section  https://review.openstack.org/329079
20:41 vinsh EmilienM: Good!  Something about expecting puppet resources to compile correctly from within a provider... is probably a bad idea anyway.
20:41 vinsh I now know WAY too much about the puppet file type/provider innards.
20:43 saju_m joined #puppet-openstack
20:44 saju_m joined #puppet-openstack
20:46 saju_m joined #puppet-openstack
20:47 EmilienM vinsh: it failed
20:47 EmilienM http://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_532
20:48 saju_m joined #puppet-openstack
20:50 EmilienM vinsh: it sounds like idempotency?
20:50 saju_m joined #puppet-openstack
20:56 _ody vinsh: 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 joined #puppet-openstack
20:58 saju_m joined #puppet-openstack
20:59 fc___ joined #puppet-openstack
21:05 zigo EmilienM: I have finished all of Newton.
21:05 iurygregory colonwq, \o/ it passed but with failure in non-voting for beaker =X
21:06 zigo Neutron's unit tests are completely broken, but it's looking like it's only unit tests.
21:06 zigo Everything else should be just fine.
21:06 zigo So, where can I modify the CI to test all of that? :P
21:07 iurygregory zigo, puppet-openstack-integration =D
21:07 zigo Ok.
21:07 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: swiftinit provider write file directly  https://review.openstack.org/329666
21:07 iurygregory you need to update the repo right?
21:07 vinsh _ody: EmilienM I was delirious and did typo.
21:11 colonwq iurygregory, Cool. Thanks for the help.
21:11 * colonwq is gone for a few hours
21:11 iurygregory colonwq, np =D
21:12 dfisher anybody have any ideas about my router name -> id -> l3-agent.ini issue? :(
21:13 iurygregory dfisher, i don't have sorry =(
21:22 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Implement Libvirt profile  https://review.openstack.org/329682
21:38 haaj joined #puppet-openstack
21:57 openstackgerrit Bradley Klein proposed openstack/puppet-monasca: Ensure all servers are listed (beyond osapi_max_limit)  https://review.openstack.org/329693
22:07 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Enable compute node micro-services  https://review.openstack.org/329694
22:22 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Enable compute node micro-services  https://review.openstack.org/329694
22:27 openstack joined #puppet-openstack
22:27 openstackstatus joined #puppet-openstack
22:28 rcernin joined #puppet-openstack
22:30 xarses joined #puppet-openstack
22:37 furlongm_ joined #puppet-openstack
22:40 iurygregory_ joined #puppet-openstack
22:40 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Enable compute node micro-services  https://review.openstack.org/329694
22:45 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Enable compute node micro-services  https://review.openstack.org/329694
23:08 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Enable compute node micro-services  https://review.openstack.org/329694
23:17 ayoung joined #puppet-openstack
23:21 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Create libvirt micro-service  https://review.openstack.org/329714

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary