Tuesday, 2015-07-14

*** aimon has quit IRC00:01
*** ducttape_ has joined #puppet-openstack00:05
*** gildub has quit IRC00:06
*** ducttape_ has quit IRC00:08
*** xingchao has quit IRC00:14
*** xingchao has joined #puppet-openstack00:15
*** ducttape_ has joined #puppet-openstack00:21
*** xingchao has quit IRC00:25
*** topshare has joined #puppet-openstack00:33
*** gildub has joined #puppet-openstack00:35
*** ducttape_ has quit IRC00:42
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: acceptance: run keystone in a WSGI server  https://review.openstack.org/20082700:42
EmilienMcrinkle: I updated ^00:43
EmilienMnot tested though00:44
*** aimon has joined #puppet-openstack00:45
richmEmilienM: works for me00:59
EmilienMrichm: nice00:59
EmilienMthanks for testing00:59
*** ducttape_ has joined #puppet-openstack01:03
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: acceptance: run keystone in a WSGI server  https://review.openstack.org/20082701:05
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: acceptance: run keystone in a WSGI server  https://review.openstack.org/20082701:06
EmilienMrichm: just updated the test title ^01:06
*** xingchao has joined #puppet-openstack01:20
*** ducttape_ has quit IRC01:25
*** ducttape_ has joined #puppet-openstack01:25
*** ducttape_ has quit IRC01:27
*** iurygregory has left #puppet-openstack01:40
openstackgerritMatt Fischer proposed openstack/puppet-ceilometer: fix module install reference  https://review.openstack.org/19980601:43
openstackgerritMatt Fischer proposed openstack/puppet-cinder: fix module install reference  https://review.openstack.org/19980701:43
openstackgerritMatt Fischer proposed openstack/puppet-glance: fix module install reference  https://review.openstack.org/19980801:43
openstackgerritMatt Fischer proposed openstack/puppet-horizon: fix module install reference  https://review.openstack.org/19980901:43
openstackgerritMatt Fischer proposed openstack/puppet-keystone: fix module install reference  https://review.openstack.org/19981001:43
openstackgerritMatt Fischer proposed openstack/puppet-manila: fix module install reference  https://review.openstack.org/19981101:43
openstackgerritMatt Fischer proposed openstack/puppet-neutron: fix module install reference  https://review.openstack.org/19981201:43
openstackgerritMatt Fischer proposed openstack/puppet-nova: fix module install reference  https://review.openstack.org/19981301:43
openstackgerritMatt Fischer proposed openstack/puppet-openstacklib: fix module install reference  https://review.openstack.org/19981401:43
openstackgerritMatt Fischer proposed openstack/puppet-swift: fix module install reference  https://review.openstack.org/19981501:43
*** markvoelker has quit IRC01:44
*** ducttape_ has joined #puppet-openstack01:48
*** bklei_ has quit IRC01:49
*** topshare_ has joined #puppet-openstack02:10
*** topshare has quit IRC02:11
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: acceptance: validate Keystone with Tempest  https://review.openstack.org/19856102:16
*** richm has quit IRC02:17
*** xingchao has quit IRC02:18
EmilienMclayton: this is really awesome: https://github.com/twc-openstack/puppet-designate_ext02:19
claytonthanks :)02:19
EmilienMI don't ask if it works02:19
claytonwriting an email for the list about it now02:19
claytonthe virtualenv support works, we're going to product with it this week or next02:19
EmilienMclayton: your patch on designate shows that it has a few impact to make it possible02:19
EmilienMclayton: and docker?02:19
claytonthe docker support appears to work, but I have to put in place other pieces in our CI & deployment pipelines to allow pushing it to production02:20
EmilienMclayton: I'm highly interested to help you02:20
EmilienMclayton: and move this work under our community02:20
claytonthe docker support requires two patches to the puppet-docker module.  one of those has been merged and a PR is outstanding for the second one (as of yesterday)02:20
claytonEmilienM: that's pretty much my hope :)02:21
claytonI think heat, keystone, designate, horizon are probably the easy services I can think of to do02:21
claytonstuff like neutron probably requires refinements to the approach02:22
EmilienMclayton: do you think we would need a spec for that?02:22
claytonI think we'd probably need one WIP at least before starting it02:22
EmilienMclayton: I have a question tho02:22
EmilienMwith your module, that means your designated.conf is shared across all dockers, right?02:23
claytonfor example, I think the designate approach is too naive in places, because it's notifying all services when for example, the db changes, when you could be more granular02:23
EmilienMyeah02:23
claytonyes, it's a read only volume mount of /etc/designate02:23
EmilienMok so this is not good enough too02:23
claytonthe /var/log/designate directroy is mounted rw02:23
EmilienMwe need to be able to have multiple config files on one host02:23
claytonthe existing inifile providers would require changes, or you'd have to replace them with something else02:24
claytonsince the paths are hard coded.02:24
EmilienMclayton: actually, we would just need to changte the path02:24
EmilienMyeah02:24
EmilienMthat's all about the path, so no big deal here02:24
claytonwhy multiple config files on one host?02:25
EmilienMclayton: well, why not?02:25
EmilienMclayton: it's important to support multiple configs02:25
EmilienMlike02:25
claytonwell, it's work and well, I don't know why anyone would want it :)02:26
EmilienMif heat engine and heat api are running on the same node02:26
EmilienMthey don't have the exact same config02:26
EmilienMclayton: anyway, time to stop :) - it's good to know we're making progress on this topic too02:27
*** xingchao has joined #puppet-openstack02:28
EmilienMclayton: I'm looking forward to reading you on the ML02:28
*** xingchao has quit IRC02:28
claytonhoping to get the email sent and go to bed :)02:28
*** gildub has quit IRC02:41
*** gildub has joined #puppet-openstack02:41
xavpaiceanother use case for multiple configs - having one API endpoint that allows admin actions via policy.json, one that doesn't (and the non admin one exposed to Internet)02:44
openstackgerritMerged openstack/puppet-keystone: Use an Anchor when service is managed  https://review.openstack.org/20130002:44
xavpaice(sorry to butt in there)02:44
*** markvoelker has joined #puppet-openstack02:54
*** markvoelker has quit IRC02:55
*** markvoelker has joined #puppet-openstack02:55
openstackgerritMerged openstack/puppet-keystone: acceptance: run keystone in a WSGI server  https://review.openstack.org/20082702:59
*** ducttape_ has quit IRC03:08
*** nigelb has joined #puppet-openstack03:09
*** ducttape_ has joined #puppet-openstack03:42
*** ducttape_ has quit IRC03:56
*** markvoelker_ has joined #puppet-openstack04:48
*** markvoelker has quit IRC04:50
*** markvoelker_ has quit IRC04:52
*** markvoelker has joined #puppet-openstack04:57
*** markvoelker has quit IRC05:03
*** markvoelker has joined #puppet-openstack05:08
*** markvoelker has quit IRC05:16
*** markvoelker has joined #puppet-openstack05:22
*** markvoelker has quit IRC05:28
*** dalgaaf has joined #puppet-openstack06:06
*** sergmelikyan has joined #puppet-openstack06:27
*** dgurtner has joined #puppet-openstack06:58
*** dgurtner has joined #puppet-openstack06:58
*** jpena has joined #puppet-openstack07:01
*** _stowa has joined #puppet-openstack07:11
*** _stowa has quit IRC07:17
*** _stowa has joined #puppet-openstack07:24
*** paramite has joined #puppet-openstack07:26
*** jistr has joined #puppet-openstack07:42
*** paramite_ has joined #puppet-openstack07:44
*** paramite_ is now known as paramite_|afk07:53
*** paramite_|afk is now known as paramite_08:00
*** dgurtner has quit IRC08:15
*** gildub has quit IRC08:23
*** derekh has joined #puppet-openstack08:30
*** saneax has joined #puppet-openstack08:42
*** xingchao has joined #puppet-openstack08:49
*** imcsk8 is now known as imcsk8|zZz08:51
*** topshare_ has quit IRC09:01
*** topshare has joined #puppet-openstack09:08
*** paramite_ has quit IRC09:15
*** xingchao has quit IRC09:15
*** dgurtner has joined #puppet-openstack09:20
*** dgurtner has quit IRC09:20
*** dgurtner has joined #puppet-openstack09:20
*** sergmelikyan has quit IRC09:36
*** dalgaaf has quit IRC09:48
*** xingchao has joined #puppet-openstack10:15
*** xingchao has quit IRC10:15
*** rcallawa_ has quit IRC10:28
*** rcallawa has joined #puppet-openstack10:29
*** dweaver has quit IRC10:33
*** dweaver has joined #puppet-openstack10:33
*** _para has joined #puppet-openstack10:44
*** gfidente has joined #puppet-openstack10:45
*** dalgaaf has joined #puppet-openstack10:49
*** sergmelikyan has joined #puppet-openstack11:00
*** topshare has quit IRC11:04
*** openstackgerrit has quit IRC11:09
*** openstackgerrit has joined #puppet-openstack11:10
*** paramite is now known as crazy-panda11:38
*** sergmelikyan has quit IRC11:58
*** ducttape_ has joined #puppet-openstack11:58
*** topshare has joined #puppet-openstack12:03
*** topshare has quit IRC12:04
*** topshare has joined #puppet-openstack12:04
*** topshare has quit IRC12:04
*** rcallawa has quit IRC12:05
*** topshare has joined #puppet-openstack12:05
*** topshare has quit IRC12:05
*** topshare has joined #puppet-openstack12:06
*** crazy-panda is now known as paramite12:08
*** ducttape_ has quit IRC12:10
*** openstack has joined #puppet-openstack12:13
*** jpena is now known as jpena|lunch12:13
*** dprince has joined #puppet-openstack12:18
*** morazi has quit IRC12:20
*** topshare has quit IRC12:27
*** topshare has joined #puppet-openstack12:29
*** sergmelikyan has joined #puppet-openstack12:29
*** linkedinyou has joined #puppet-openstack12:32
EmilienMhello12:32
*** jayg|g0n3 is now known as jayg12:37
*** iurygregory has joined #puppet-openstack12:42
*** markvoelker has joined #puppet-openstack12:43
*** paramite is now known as paramite|afk12:43
*** paramite has joined #puppet-openstack12:46
*** paramite is now known as Guest235412:46
*** Guest2354 has quit IRC12:47
*** markvoelker has quit IRC12:47
*** _para has quit IRC12:47
*** rcallawa has joined #puppet-openstack12:51
*** markvoelker has joined #puppet-openstack12:57
*** markvoelker has quit IRC13:01
*** rcallawa_ has joined #puppet-openstack13:01
*** markvoelker has joined #puppet-openstack13:01
*** rcallawa has quit IRC13:03
*** markvoelker has quit IRC13:06
*** morazi has joined #puppet-openstack13:08
*** markvoelker has joined #puppet-openstack13:10
*** sergmelikyan has quit IRC13:12
dprinceHunner: hi, question for you. How do I get a handle to the catalog in a customer provider/type... as opposed to a custom function like this MEETING TIME: Every other Tuesday at 19:00 UTC in #openstack-meeting-alt.13:15
dprinceHunner: oops. I meant this: https://review.openstack.org/#/c/194342/2/lib/puppet/parser/functions/write_package_names.rb,cm13:15
*** jpena|lunch is now known as jpen13:18
*** jpen is now known as jpena13:18
*** prad has joined #puppet-openstack13:29
*** dfisher has joined #puppet-openstack13:36
*** sergmelikyan has joined #puppet-openstack13:40
*** IBerezovskiy has joined #puppet-openstack13:42
*** bklei has joined #puppet-openstack13:45
*** bklei has quit IRC13:46
*** bklei has joined #puppet-openstack13:47
*** bklei has quit IRC13:47
*** rcallawa_ has quit IRC13:47
*** sergmelikyan has quit IRC13:49
*** xingchao has joined #puppet-openstack13:53
*** richm has joined #puppet-openstack13:56
*** sergmelikyan has joined #puppet-openstack13:57
*** markvoelker_ has joined #puppet-openstack14:00
*** paramite|afk is now known as paramite14:01
*** markvoelker has quit IRC14:04
openstackgerritPaul Belanger proposed openstack/puppet-openstack-integration: [WIP] Initial commit for keystone  https://review.openstack.org/20129814:08
openstackgerritPaul Belanger proposed openstack/puppet-openstack-integration: [WIP] Initial commit for keystone  https://review.openstack.org/20129814:09
*** ducttape_ has joined #puppet-openstack14:11
*** saneax has quit IRC14:14
*** xingchao has quit IRC14:18
*** rcallawa has joined #puppet-openstack14:22
*** stamak has joined #puppet-openstack14:23
*** kbyrne has quit IRC14:23
*** xingchao has joined #puppet-openstack14:24
*** rcallawa_ has joined #puppet-openstack14:27
*** rcallawa has quit IRC14:29
EmilienMrichm: I found a bug in tempest, that's why puppet-keystone/tempest-v3 does not work14:33
richmEmilienM: ack14:33
EmilienMrichm: https://bugs.launchpad.net/tempest/+bug/147419314:33
openstackLaunchpad bug 1474193 in tempest "Isolated credentials with v3 auth creates unusable users" [Undecided,In progress] - Assigned to yong sheng gong (gongysh)14:33
iurygregoryhey people i'm trying to install keystone with puppet (site.pp code http://paste.openstack.org/show/374524/) but i'm getting this errors http://paste.openstack.org/raw/374523/ =/14:36
*** sergmelikyan has quit IRC14:37
EmilienMiurygregory: have you tried master?14:39
EmilienMiurygregory: you're missing keystone::endpoints14:40
iurygregoryEmilienM, thanks i'll try ;)14:41
EmilienMiurygregory: wait14:41
iurygregoryok14:41
EmilienMiurygregory: https://github.com/openstack/puppet-keystone/blob/master/spec/acceptance/keystone_wsgi_apache_spec.rb#L9-L6114:41
iurygregorythe puppet-keystone code is stable/juno14:41
EmilienMyou should try this manifest, which is the latest tested one14:41
EmilienMoh14:41
EmilienMstill you should use keystone::endpoint14:41
EmilienMyou're missing that14:41
EmilienMthat's why you have an error14:42
iurygregoryok14:42
iurygregory=)14:42
iurygregorythe code keystone::resource::service_identity should be removed?14:42
EmilienMyes14:43
*** sergmelikyan has joined #puppet-openstack14:43
EmilienMit's consumed by other modules to create resources14:43
*** kbyrne has joined #puppet-openstack14:43
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: Use an Anchor when service is managed  https://review.openstack.org/20159714:43
iurygregoryok EmilienM thanks14:44
*** sergmelikyan has quit IRC14:48
*** markvoelker_ has quit IRC14:49
*** markvoelker has joined #puppet-openstack14:50
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: Use an Anchor when service is managed  https://review.openstack.org/20160514:51
EmilienMrichm: backported to stable/kilo & juno ^14:51
crinklemorning14:51
EmilienMcrinkle: o/14:51
EmilienMour weekly meeting is starting in 5 min14:54
*** markvoelker has quit IRC14:55
*** mdorman has joined #puppet-openstack14:56
*** rwsu has joined #puppet-openstack14:56
*** rwsu has quit IRC14:58
*** rwsu has joined #puppet-openstack14:58
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Add basic manifests  https://review.openstack.org/19549315:07
*** vinsh_ is now known as Vinsh15:09
*** cwolferh has quit IRC15:17
*** ferest has joined #puppet-openstack15:17
*** ferest has quit IRC15:21
*** topshare has quit IRC15:23
*** paramite has quit IRC15:24
*** markvoelker has joined #puppet-openstack15:26
*** topshare has joined #puppet-openstack15:27
*** topshare has quit IRC15:27
*** topshare has joined #puppet-openstack15:28
Hunnerdprince: I think it's self.catalog from a resource and resource.catalog from a provider15:43
dprinceHunner: okay, will try it again shortly. If that works I'd be much happier...15:44
dprinceHunner: any imports required though?15:45
HunnerNope15:45
*** sergmelikyan has joined #puppet-openstack15:47
dprinceHunner: thanks :)15:47
*** dprince has quit IRC15:48
*** saneax has joined #puppet-openstack15:49
*** sergmelikyan has quit IRC15:51
*** cwolferh has joined #puppet-openstack15:59
EmilienMcrinkle: where is the patch for https://bugs.launchpad.net/puppet-horizon/+bug/1470993 ?16:01
openstackLaunchpad bug 1470993 in puppet-horizon "if compress_static=true failed to apply horizon puppets" [Undecided,In progress] - Assigned to Vasyl Saienko (vsaienko)16:01
crinkleEmilienM: https://review.openstack.org/#/c/198119/16:01
EmilienMoh that one :)16:02
crinklere https://review.openstack.org/#/c/193328/ I can +2 but would like to see one more perspective16:03
*** aimon has quit IRC16:04
*** dprince has joined #puppet-openstack16:08
mfischcrinkle: EmilienM that bug was my fault, source is right16:12
mfischhttps://bugs.launchpad.net/puppet-glance/+bug/147136416:12
openstackLaunchpad bug 1471364 in puppet-glance "glance_image keeps trying to update an image from location causing an error" [Undecided,Invalid]16:12
EmilienMcool16:12
EmilienMmfisch: invalid then16:13
mfischyep and marked as such16:13
EmilienMcool16:13
EmilienMmfisch, crinkle: no bug in glance_image -> https://goo.gl/SCxx3c16:15
crinkleso what is location for then?16:16
_odycrinkle: re https://review.openstack.org/#/c/193328...for example.  I am in direct disagreement with the premise of the patch.  I am a fan of fewer parameters.16:16
socialI have question about https://review.openstack.org/#/c/200877/ I don't like neutron::db::sync checking for variable from neutron class but I can't come up with better solution, any ideas?16:16
EmilienMcrinkle: location is used by Glance to download an image the first time I think16:16
EmilienMthat's why it's always updated16:17
crinkleEmilienM: should we even have it as a parameter? has it ever worked?16:17
crinkle_ody: would you mention that in the patch?16:17
_odyDoing so now.16:18
EmilienMcrinkle: we should not support it imho16:18
_odyI'll ellaborate too16:18
openstackgerritMerged openstack/puppet-keystone: Use an Anchor when service is managed  https://review.openstack.org/20159716:19
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Add basic manifests  https://review.openstack.org/19549316:20
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Add basic manifests  https://review.openstack.org/19549316:20
socialshould I add service_plugins parameter to db sync and to neutron server or really just take it from the ::neutron class?16:21
openstackgerritAndrew Boik proposed openstack/puppet-neutron: Update Cisco ML2 classes and templates  https://review.openstack.org/19718116:26
crinkleso source and location work interchangeably for me16:34
*** derekh has quit IRC16:34
crinklewhen using http://download.cirros-cloud.net/0.3.1/cirros-0.3.1-x86_64-disk.img as the url16:34
crinklemfisch: EmilienM ^16:35
*** dgurtner has quit IRC16:35
EmilienMcrinkle: ok16:36
crinkleand when running the example from the bug16:36
openstackgerritLukas Bezdicka proposed openstack/puppet-neutron: Run neutron db sync also for each neutron module  https://review.openstack.org/20087716:39
claytoncrinkle: do those translate to --copy-from and --location?16:40
claytonwith --copy-from it will pull in the image and store it inside glance, whereas --location will just store the URL and download it on demand16:40
claytonah, so source translates to either --file or --copy-from16:41
claytonand location always translates to --location16:41
crinkleyes16:41
*** aimon has joined #puppet-openstack16:46
degorenkoHi folks, please, take a look  https://review.openstack.org/198744  https://review.openstack.org/19272116:46
*** jpena is now known as jpena|PTO16:51
*** sergmelikyan has joined #puppet-openstack16:51
iurygregoryhey EmilienM , can you add me to the trello (#link https://trello.com/b/4X3zxWRZ/on-going-effort) my profile is iuryferreira16:56
EmilienMsure thing16:57
EmilienMiurygregory: done16:57
iurygregorythanks16:57
openstackgerritMerged openstack/puppet-tripleo: Remove mode tcp enforcement where unneeded, we default to mode tcp  https://review.openstack.org/19950016:59
*** mgarza_ has joined #puppet-openstack17:00
iurygregorywith the 6.0 release, i need to move my spec  https://review.openstack.org/#/c/190361/ to Liberty folder?  (there is already one?)17:00
EmilienMiurygregory: yes you have to17:04
EmilienMiurygregory: create one, and update the index also17:04
openstackgerritMerged openstack/puppet-tripleo: Add missing options to Ceilometer/Ironic/Horizon  https://review.openstack.org/19950717:10
iurygregoryok EmilienM ;)17:13
*** jistr has quit IRC17:14
EmilienMpabelanger: we also use trello in our upstream work. I created a card for our work: https://trello.com/c/rqJQL0yU/56-implement-openstack-integration-ci-job17:16
pabelangerEmilienM, sounds good. Will review it17:18
*** imcsk8|zZz is now known as imcsk817:19
EmilienMpabelanger: what do you think if we do a brainstorm on an etherpad?17:23
EmilienMthat would help us to write a BP17:24
pabelangerEmilienM, works for me17:25
pabelangerif we get it started today, I have help add stuff.  having meetings the rest of the afternoon in an hour or tow17:26
pabelangertwo*17:26
EmilienMpabelanger: let's just brainstorm a bit17:27
EmilienMpabelanger, crinkle, nibalizer: https://etherpad.openstack.org/p/puppet-openstack-integration17:27
crinkleEmilienM: i'm in-and-out again this afternoon and for the rest of the week but i'll help where i can17:29
EmilienMcrinkle: no prob17:29
EmilienMjust fuy17:30
*** ianw has quit IRC17:31
nibalizerEmilienM: neat17:31
EmilienMnibalizer: I saw it ! lol17:36
EmilienMnibalizer, pabelange, crinkle : not sure we want to write a composition layer17:44
EmilienMie: https://github.com/kickstandproject-dev/grunt/blob/master/modules/kickstandproject/manifests/grunt17:44
EmilienMI would rather use Hiera directly17:44
EmilienMand compose our scenarios17:44
pabelangerRight, I decided not to use heira, so my existing code doesn't fit well into that17:45
pabelangerto be honest, my hiera is weak17:45
EmilienMpabelanger: I would rather not follow grunt design17:47
EmilienMbecause you're putting some logic we want to have in our modules17:48
EmilienMnot in the composition layer17:48
EmilienMimho, we should rather work on Hiera scenarios of deployments17:48
pabelangerRight, not advocating that model. Just showing what I did in the past17:49
EmilienMoh it's a cool thing17:49
pabelangerEmilienM, do you have a URL showing how that would look in hiera?17:49
EmilienMbut there is a planty of composition layers on the market (I wrote one also, https://github.com/stackforge/puppet-openstack-cloud )17:49
EmilienMwe would have YAML files defining our classes and parameters17:50
EmilienMnibalizer, crinkle: wdyt?17:50
nibalizeri would hope you could do it all /most in yaml17:50
nibalizerbut i've not actually tried to use the openstack/puppet-*17:50
nibalizermodules17:50
crinklei don't know why we need hiera for this17:51
EmilienM?17:51
pabelangernibalizer, what does that look like? Do you have an existing example to point too?17:51
crinklewe could do it all in puppet, makes it easier to debug17:51
EmilienMcrinkle: so you want a composition layer?17:51
nibalizerya17:52
EmilienMI prefer having YAML here17:52
nibalizerI kinda prefer what crinkle is saying17:52
crinkleEmilienM: i mean it's a composition layer in puppet or a composition layer in hiera17:52
EmilienMnibalizer: well, you just said opposite17:52
EmilienMoh ok17:52
pabelangeronly issue I have with going all in hiera, is are we limiting the amount of people that will contribute to it? Like I said, I haven't played much with hiera17:52
nibalizerEmilienM: the debug argument is what gets me17:52
EmilienMwe basically need .pp files with : include ::keystone17:52
crinkleyeah this doesn't need to be elegant and beautiful, it needs to be super readable17:53
nibalizercomplex parameters with data bindings overriding defaults coded into modules is hard17:53
nibalizerya devstack is supposed to be readable17:53
EmilienMspredzy  has something17:53
EmilienMawesome17:53
* EmilienM search the URL17:53
EmilienMhttps://github.com/Mylezeem/puppeels17:53
nibalizeri've got to step out for a while, will circle back in a couple hours17:53
nibalizerEmilienM: or whomever I was chatting with, got a link to the test that alphabetizes the Puppetfile?17:54
EmilienMnibalizer: wasn't me17:54
crinklethe puppetfile is in two sections, i would think alphabetizing would break it17:54
EmilienMcrinkle: see https://github.com/Mylezeem/puppeels/tree/master/manifests/openstack17:54
nibalizercrinkle: ya a section based alphabetizing I meant17:55
pabelangernibalizer, it was me. I might have confused the work you did with your recent alphabetizing work17:55
EmilienMthis is exactly what we need here, cc spredzy17:55
crinkleno i disagree17:55
crinklehttps://github.com/Mylezeem/puppeels/blob/master/manifests/openstack/compute.pp doesn't say anything about how nova is configured17:55
crinklei have to look somewhere else to find it17:56
EmilienMcrinkle: this is done by hiera17:56
crinkleit's very difficult17:56
crinklei know17:56
crinkleand i'm saying i don't want it done with hiera17:56
crinklei want the parameters in the manifest17:56
*** openstackgerrit has quit IRC17:56
crinkleat least for an integration test17:56
EmilienMok so this is definitely a module that will be used *only* for Puppet OpenStack gating in OpenStack Infra CI, right?17:57
crinkleyes absolutely17:57
*** openstackgerrit has joined #puppet-openstack17:57
pabelangerthat is how I se it17:57
pabelangersee*17:57
EmilienMcrinkle: but why not Hiera? because it's too hard to debug?17:57
crinkleEmilienM: yes17:57
pabelangerwell, 3rd party CI could use it as well17:57
EmilienMcrinkle: and how will we do when in one year we will want more integration jobs, with multiple scenarios17:58
crinklei have to 1) know that hiera is a thing, 2) look up the hierarchy, 3) find the right value in the yaml file17:58
crinkleEmilienM: add another manifest17:58
EmilienMvery overloaded17:58
EmilienMkeystone.pp17:58
EmilienMkeystone_ha.pp17:58
EmilienMetc?17:58
crinkleit makes it super obvious what's going on17:58
crinklewhich is needed when all we have to debug are jenkins logs17:58
EmilienMdo we want obvious manifest or effective Puppet code?17:58
*** sergmelikyan has quit IRC18:00
EmilienMcrinkle: so we are going to write a single manifest, with some parameters, right? Just asking18:00
EmilienMlike openstack_aio.pp18:01
EmilienMand put all our classes / parameters inside18:01
EmilienMto call puppet-* modules18:01
crinkleEmilienM: i don't know, maybe18:01
* crinkle afk18:01
EmilienMthis sounds good if we don't want to scale the module later18:02
pabelangerFor me, I was envisioning single.pp, which would mean a single node.  Then, inside single.pp, we include each element require /keystone / database / nova, which could be toggled via hiera18:03
pabelangeror, more generic like controller.pp18:03
EmilienMno because this is not a controller18:03
pabelangerthen single.pp will include controller.pp and compute.pp18:03
pabelangerjust brain dumping18:04
EmilienMthe problem I have with all of that, is there is a barrier between flexibility & simplicity18:04
pabelangerright18:04
EmilienMand I already made the mistake to write a composition layer in 2 installers I wrote for my companies18:04
pabelangerso, if we want ultimate flexibility I think single.pp works, then we include keystone.pp18:04
EmilienMI agree Hiera is not easy to debug18:04
pabelangerbut, maybe not18:04
pabelanger*shurgs*18:04
EmilienMI would suggest to iterate18:05
pabelangerya, haven;t debugged hiera much, but sounds like a pain18:05
EmilienMstart with a single.pp and see how it behaves18:05
EmilienMif we think at multi node later, we will be able to split the code18:05
crinklei think it would be different manifests for different scenarios, eg scenario1.pp, scenario2.pp, no parameters18:05
EmilienMie: keystone.pp taken from single.pp18:05
crinklemakes it obvious, makes it unmistakable for a composition layer18:06
EmilienMcrinkle: ok so the manifest would not have any parameter18:06
pabelangercrinkle, right, that works. Just means more code in the repo to manage18:06
EmilienMok18:06
EmilienMI got your idea now crinkle18:06
crinklepabelanger: i'm fine with that18:06
crinkleit's not a module18:06
crinkleit's tests18:06
pabelangerright, that is how I was envisioning it too.18:06
crinklecool18:07
pabelangerI was just calling it single.pp18:07
EmilienMfine for me too now..18:07
crinklepabelanger: ah i see18:07
pabelangerso, scenario001.pp is a single node setup.18:07
EmilienMwhatever name, it will be documented to explain what we install18:08
crinkleyes18:08
pabelangercrinkle, how would you see different database support for scenario001.pp? Is not parameter is passed in to toggle mysql or postgres?18:08
crinklepabelanger: no i see scen1_mysql.pp and scen2_pg.pp18:09
EmilienMwe will gate mysql18:09
EmilienMcrinkle: I disagree18:09
EmilienMcrinkle: because I can ask the same for the neutron plugin18:09
EmilienMand for keystone backends18:10
EmilienMand so on18:10
EmilienMwe need to agree on a scenario18:10
EmilienMthat is the most common18:10
EmilienMor multiple scenarios18:10
pabelangerright, I see a fair bit of duplicate code is that configuration. just to toggle something18:10
EmilienMbut keep naming consistent18:10
crinkleEmilienM: the acceptance tests should be testing the minutia18:10
crinklethe integration test should test the larger deploument scenarios18:10
* EmilienM googling minutia18:10
crinklelike mysql vs pg18:11
*** xarses has quit IRC18:11
crinkleEmilienM: details18:11
EmilienMgot it18:11
EmilienMpabelanger: I sent my comments on https://review.openstack.org/#/c/201584/18:12
EmilienMcrinkle: you might also want to see ^18:12
*** sergmelikyan has joined #puppet-openstack18:14
EmilienMpabelanger: you might want to see https://github.com/openstack/puppet-keystone/blob/master/spec/acceptance/keystone_wsgi_apache_spec.rb#L9-L61 for the initial manifest18:19
EmilienMit installs RDO/UCA, MySQL, Apache, Keystone18:19
openstackgerritMerged openstack/puppet-ironic: Allow customization of force_power_state_during_sync  https://review.openstack.org/19702218:21
*** pleia2 has quit IRC18:25
*** duritong_ has joined #puppet-openstack18:25
*** timfreun1 has joined #puppet-openstack18:25
*** saneax_ has joined #puppet-openstack18:26
*** pleia2 has joined #puppet-openstack18:27
*** xarses has joined #puppet-openstack18:27
*** dweaver` has joined #puppet-openstack18:28
*** tlbr_ has joined #puppet-openstack18:29
*** clayton_ has joined #puppet-openstack18:29
*** mgagne_ has joined #puppet-openstack18:29
*** ken_barber1 has joined #puppet-openstack18:30
*** saneax has quit IRC18:30
*** dweaver has quit IRC18:30
*** khappone has quit IRC18:30
*** tlbr has quit IRC18:30
*** duritong has quit IRC18:30
*** timfreund has quit IRC18:30
*** rhefner has quit IRC18:30
*** tchaypo has quit IRC18:30
*** ken_barber has quit IRC18:30
*** clayton has quit IRC18:30
*** mgagne has quit IRC18:30
*** ken_barber1 is now known as ken_barber18:30
*** clayton_ is now known as clayton18:30
*** xingchao has quit IRC18:33
*** mgagne_ has left #puppet-openstack18:33
*** rcallawa has joined #puppet-openstack18:33
*** dboik has joined #puppet-openstack18:34
*** rcallawa_ has quit IRC18:35
*** mgagne_ has joined #puppet-openstack18:40
*** openstackgerrit has quit IRC18:56
*** openstackgerrit has joined #puppet-openstack18:56
*** ianw has joined #puppet-openstack19:01
*** timfreun1 is now known as timfreund19:04
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: Add rspec beaker boxes for centos7 and trusy  https://review.openstack.org/19892519:10
openstackgerritMykola Golub proposed stackforge/puppet-ceph: osd: fix pattern when matching ${data}  https://review.openstack.org/20173019:14
openstackgerritAndrew Boik proposed openstack/puppet-neutron: Update Cisco ML2 classes and templates  https://review.openstack.org/19718119:17
openstackgerritAndrew Boik proposed openstack/puppet-neutron: Update Cisco ML2 classes and templates  https://review.openstack.org/19718119:19
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: Add rspec beaker boxes for centos7 and trusy  https://review.openstack.org/19892519:25
*** xingchao has joined #puppet-openstack19:33
*** mgarza_ has quit IRC19:37
*** gfidente has quit IRC19:38
*** mgarza_ has joined #puppet-openstack19:38
openstackgerritMerged openstack/puppet-tuskar: keystone/auth: make service description configurable  https://review.openstack.org/19515819:40
*** paramite has joined #puppet-openstack19:45
*** cwolferh has quit IRC19:48
*** mgarza_ has quit IRC19:49
*** mgarza_ has joined #puppet-openstack19:49
openstackgerritMatt Fischer proposed openstack/puppet-manila: Note that manila is on Puppet Forge  https://review.openstack.org/20175119:52
*** rhefner has joined #puppet-openstack19:52
mfischcrinkle: per your comment  https://review.openstack.org/20175119:53
crinklemfisch: sweet19:54
mfischcrinkle: I'm crushing it with these patches lately!19:54
EmilienMcrinkle: https://review.openstack.org/#/c/199676/ is really for backward compatibility19:55
EmilienMcrinkle: people are using this resource, because it's easier to manage admin resources like this19:56
EmilienMcrinkle: and having this patch allows a smooth migration to keystone v3 admin resources19:56
*** mgarza_ has quit IRC19:57
*** mgarza_ has joined #puppet-openstack19:57
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: acceptance: validate Keystone with Tempest  https://review.openstack.org/19856119:57
openstackgerritMatt Fischer proposed openstack/puppet-ceilometer: fix module install reference  https://review.openstack.org/19980619:59
openstackgerritMatt Fischer proposed openstack/puppet-ceilometer: Enable support for memcached_servers  https://review.openstack.org/19983519:59
*** paramite has quit IRC20:00
*** tchaypo has joined #puppet-openstack20:01
openstackgerritBen Kero proposed openstack/puppet-swift: Make object storage server parameters match Swift defaults  https://review.openstack.org/20175520:02
*** sergmelikyan has quit IRC20:06
*** sergmelikyan has joined #puppet-openstack20:07
*** rcallawa has quit IRC20:09
*** rcallawa has joined #puppet-openstack20:09
*** sergmelikyan has quit IRC20:12
openstackgerritMerged openstack/puppet-openstacklib: fix module install reference  https://review.openstack.org/19981420:13
openstackgerritMerged openstack/puppet-cinder: fix module install reference  https://review.openstack.org/19980720:19
openstackgerritMerged openstack/puppet-glance: fix module install reference  https://review.openstack.org/19980820:19
openstackgerritMerged openstack/puppet-horizon: fix module install reference  https://review.openstack.org/19980920:19
openstackgerritMerged openstack/puppet-manila: fix module install reference  https://review.openstack.org/19981120:19
openstackgerritMerged openstack/puppet-swift: fix module install reference  https://review.openstack.org/19981520:19
openstackgerritMerged openstack/puppet-neutron: fix module install reference  https://review.openstack.org/19981220:20
openstackgerritMathieu Gagné proposed openstack/puppet-nova: Remove very old xenserver configs  https://review.openstack.org/20176020:24
*** dprince has quit IRC20:30
*** linkedinyou has quit IRC20:30
openstackgerritMerged openstack/puppet-nova: fix module install reference  https://review.openstack.org/19981320:31
*** mgarza_ has quit IRC20:38
*** mgarza has joined #puppet-openstack20:39
*** mgarza has quit IRC20:42
*** cwolferh has joined #puppet-openstack20:43
*** jpena|PTO has quit IRC20:50
*** mgagne_ is now known as mgagne20:50
*** jayg is now known as jayg|g0n320:53
*** aimon has quit IRC20:54
*** stamak has quit IRC20:56
*** rcallawa_ has joined #puppet-openstack20:59
*** aimon has joined #puppet-openstack20:59
*** rcallawa has quit IRC21:03
*** rcallawa_ has quit IRC21:03
*** ericpeterson has joined #puppet-openstack21:06
*** ducttape_ has quit IRC21:09
*** ducttape_ has joined #puppet-openstack21:22
*** ericpeterson has quit IRC21:26
*** gfidente has joined #puppet-openstack21:27
*** gfidente has quit IRC21:27
*** gfidente has joined #puppet-openstack21:27
*** rcarrillocruz has joined #puppet-openstack21:33
*** dboik has quit IRC21:37
*** prad has quit IRC21:39
*** delattec has quit IRC21:52
*** rcallawa has joined #puppet-openstack21:53
*** morazi has quit IRC21:55
*** rcallawa_ has joined #puppet-openstack21:58
*** rcallawa has quit IRC22:02
mgagneso this one bit me today: https://github.com/openstack/puppet-ceilometer/blob/master/manifests/init.pp#L201-L205 vs https://github.com/openstack/puppet-nova/blob/master/manifests/init.pp#L530-L53822:09
mgagneEmilienM: ^ I think that one is a good candidate for openstacklib22:11
mgagnenow that oslo_messaging is (should) be used everywhere22:12
EmilienMlol are you kidding me22:12
EmilienMwe talked so much long time during the summit22:12
mgagneI'm mentally ready for that one :D22:12
EmilienMmgagne: you even did not show up to our session22:13
EmilienMand now you're asking for changing that22:13
EmilienMlol22:13
EmilienMmgagne: you can go ahead if you do the patches22:13
*** delattec has joined #puppet-openstack22:14
mgagneEmilienM: yea, sorry about my absence. I've been far from puppet land those last months but now I'm back into it with kilo testing :-/22:16
EmilienMgood to know22:17
EmilienMjust saying we *already* talked about this. So if you want a change now, it's a DIY22:17
EmilienMbecause we have so much in our plate now22:17
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: acceptance: validate Keystone with Tempest  https://review.openstack.org/19856122:22
openstackgerritMerged openstack/puppet-keystone: Use an Anchor when service is managed  https://review.openstack.org/20160522:23
openstackgerritMerged openstack/puppet-nova: Remove very old xenserver configs  https://review.openstack.org/20176022:24
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: acceptance: validate Keystone with Tempest  https://review.openstack.org/19856122:39
*** dfisher has quit IRC22:49
*** cwolferh has quit IRC23:06
*** cwolferh has joined #puppet-openstack23:06
*** aimon has quit IRC23:14
*** dprince has joined #puppet-openstack23:24
*** gildub has joined #puppet-openstack23:26
*** ducttape_ has quit IRC23:27
*** gfidente has quit IRC23:28
*** delattec has quit IRC23:29
*** mdorman has quit IRC23:30
*** bitblt has joined #puppet-openstack23:31
*** LimorStotland has quit IRC23:34
*** blkperl has quit IRC23:38
*** LimorStotland has joined #puppet-openstack23:38
*** blkperl has joined #puppet-openstack23:38
*** cdelatte has joined #puppet-openstack23:47
*** cdelatte has quit IRC23:47

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