Monday, 2016-03-28

openstackgerritEmilien Macchi proposed openstack/puppet-nova: CI test - will fail  https://review.openstack.org/29723100:09
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper: Test puppet-nova Gemfile change  https://review.openstack.org/29723300:09
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper: Test puppet-nova Gemfile change  https://review.openstack.org/29723300:16
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Add Glance profiles  https://review.openstack.org/29607600:20
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: revert temporary workaround on centos  https://review.openstack.org/29246000:21
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: CI test - dnm  https://review.openstack.org/29806700:23
*** cwolferh has quit IRC00:24
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: IPv6: dual-stack support for Keystone  https://review.openstack.org/28634400:25
*** saneax is now known as saneax_AFK00:39
openstackgerritMerged openstack/puppet-tempest: Updates for stable/mitaka  https://review.openstack.org/29741700:59
*** ducttape_ has joined #puppet-openstack01:07
*** ducttape_ has quit IRC01:16
*** ilbot3 has quit IRC01:45
*** ilbot3 has joined #puppet-openstack01:48
*** moravec has joined #puppet-openstack02:10
*** rmallah has joined #puppet-openstack02:23
openstackgerritguessi proposed openstack/puppet-openstack-integration: Fix package version on Fedora  https://review.openstack.org/28861902:24
rmallahin a new keystone deployment using puppet after initial db and keystone setup ,         keystone_domain { 'domain_one': ensure => present }02:25
rmallahis not working02:25
rmallahError: Could not prefetch keystone_domain provider 'openstack': Execution of '/usr/bin/openstack domain list --quiet --format csv' returned 1: ERROR: openstack Authorization failed: The resource could not be found. (HTTP 404)02:26
rmallahhow to create initial projects , domain , user , roles ?02:26
mfischEmilienM: do you have permissions to close https://bugs.launchpad.net/puppet-openstack/+bug/129038503:54
openstackLaunchpad bug 1290385 in puppet-openstack "$openstack::compute::neutron_auth_url does not propagate to nova.conf" [Undecided,New]03:54
mfischcrinkle: can you add me to this team?  https://launchpad.net/~puppet-openstack03:55
*** saneax_AFK is now known as saneax03:56
crinklemfisch: done03:58
mfischty03:58
*** dgurtner has joined #puppet-openstack04:10
*** dgurtner has quit IRC04:10
*** dgurtner has joined #puppet-openstack04:10
*** dgurtner has quit IRC04:19
*** cdelatte has joined #puppet-openstack04:26
*** aimon has quit IRC04:28
*** delattec has joined #puppet-openstack04:28
*** cdelatte has quit IRC04:31
*** rmallah has quit IRC04:49
*** rmallah has joined #puppet-openstack04:53
*** aimon has joined #puppet-openstack05:14
*** aimon has quit IRC05:19
*** delatte has joined #puppet-openstack05:32
*** delattec has quit IRC05:36
*** fedexo has joined #puppet-openstack05:41
*** rmallah has quit IRC05:47
*** rmallah has joined #puppet-openstack05:52
*** aimon has joined #puppet-openstack06:08
*** aimon has quit IRC06:13
*** fedexo has quit IRC06:41
rmallahcan anyone please help me with : Error: Could not prefetch keystone_tenant provider 'openstack': Execution of '/usr/bin/openstack project list --quiet --format csv --long' returned 1: The resource could not be found. (HTTP 404)07:04
rmallahi get this when i run puppet agent -t -v on controller node07:05
openstackgerritLiao Penghui proposed openstack/puppet-keystone: Fix bugs for puppet fail when missing endpoint  https://review.openstack.org/29450607:18
Mosibi_rmallah: look in de keystone logfile for errors..07:54
*** Mosibi_ is now known as Mosibi07:54
*** aimon has joined #puppet-openstack07:57
*** aimon has quit IRC08:01
rmallahthere are 404s for requests for domains.08:14
rmallaheg:GET /v2.0/domains?name=Default HTTP/1.1" 404 318 0.00116008:15
rmallahdoes openstack/puppet-*  manage apt sources to get compatible openstack versions ?08:21
rmallahon the node in which puppet agent is run08:21
openstackgerritMichael Polenchuk proposed openstack/puppet-nova: Autorequire nova security group  https://review.openstack.org/29654808:49
*** Tesrte has joined #puppet-openstack09:12
skolekonov|awayrmallah, hi! Can you try to set auth urls without versions? Keystone should discover them09:19
*** skolekonov|away is now known as skolekonov09:19
*** Tesrte has quit IRC09:30
*** rmallah has quit IRC09:36
*** igajsin1 has joined #puppet-openstack09:44
*** aimon has joined #puppet-openstack09:45
*** igajsin1 has left #puppet-openstack09:45
*** aimon has quit IRC09:49
*** rmallah has joined #puppet-openstack09:51
*** rmallah has quit IRC09:52
openstackgerritOleksiy Molchanov proposed openstack/puppet-keystone: Fix for pki_setup  https://review.openstack.org/29818309:58
*** trown|outtypewww is now known as trown|PTO11:22
*** aimon has joined #puppet-openstack11:33
*** aimon has quit IRC11:38
*** foxhunt_away is now known as foxhunt7212:03
*** ducttape_ has joined #puppet-openstack12:10
*** jayg|g0n3 is now known as jayg12:19
openstackgerritOleksiy Molchanov proposed openstack/puppet-keystone: Fix for pki_setup  https://review.openstack.org/29818312:22
*** dprince has joined #puppet-openstack12:24
*** ducttape_ has quit IRC12:24
openstackgerritMerged openstack/puppet-openstack-integration: Fix bug and non-idiom in bash scripts.  https://review.openstack.org/29711412:24
*** _degorenko is now known as degorenko12:32
degorenkozigo, ping12:33
*** johnmilton has joined #puppet-openstack13:12
openstackgerritDenis Egorenko proposed openstack/puppet-glance: Add posibility to configure Glance backend settings for Glare service  https://review.openstack.org/29779513:17
*** aimon has joined #puppet-openstack13:21
*** ayoung has joined #puppet-openstack13:22
*** aimon has quit IRC13:26
*** vinsh has quit IRC13:28
*** vinsh has joined #puppet-openstack13:29
*** vinsh_ has joined #puppet-openstack13:30
*** vinsh has quit IRC13:30
*** openstackgerrit has quit IRC13:48
*** openstackgerrit has joined #puppet-openstack13:48
*** richm has joined #puppet-openstack13:50
*** ducttape_ has joined #puppet-openstack13:52
*** dfisher has joined #puppet-openstack14:05
*** andreww has quit IRC14:10
*** aimon has joined #puppet-openstack14:15
openstackgerritDavid Moreau Simard proposed openstack/puppet-neutron: Fix LinuxBridge Agent configuration file name  https://review.openstack.org/29825414:25
openstackgerritDavid Moreau Simard proposed openstack/puppet-neutron: Fix LinuxBridge Agent configuration file name  https://review.openstack.org/29825414:29
openstackgerritDavid Moreau Simard proposed openstack/puppet-neutron: Set default branch to stable/liberty  https://review.openstack.org/29826014:31
*** andreww has joined #puppet-openstack14:37
*** markvoelker has joined #puppet-openstack14:38
aleedegorenko, morning14:45
*** dhollinger has joined #puppet-openstack14:45
aleedegorenko, which parameters are you talking about here? https://review.openstack.org/#/c/296159/2/spec/classes/barbican_api_spec.rb14:45
degorenkoalee, good day :)14:46
aleedegorenko, just going through your comments .. hopefully we can get all these ironed out today ..14:46
degorenkoalee, for passed for example rabbit_host14:47
degorenkoor whatever else14:47
degorenkoyou have a bunch of parameters, which can be passed to puppet manifest14:47
degorenkonow your test cover only default values14:47
aleedegorenko, yeah ok14:48
aleedegorenko, in https://review.openstack.org/#/c/296162/2/manifests/api.pp , I'll go ahead and implement the new parameters14:49
openstackgerritDmitry Burmistrov proposed openstack/puppet-oslo: Rework oslo-cache class  https://review.openstack.org/29827314:50
aleedegorenko, to do so though - I need to know how to implement multicfg options ..14:50
openstackgerritMerged openstack/puppet-openstacklib: acceptance: do not enable EPEL  https://review.openstack.org/29797814:50
aleedegorenko, do you happen to have a link or example of how I can do that?  I know they were recently added ..14:50
aleeEmilienM, ^^14:51
degorenkoalee, it's bad practice when you're trying to implement some feature that's not implemented in code yet14:51
degorenkoalee, these parameters already added?14:51
aleedegorenko, sure  -- the plan was to add a patch right after this set of patches14:51
aleedegorenko, these parameters are already in the underlying barbican patches14:52
aleeI mean barbican package14:52
*** foxhunt72 is now known as foxhunt_away14:52
aleeI just did not - at the time - have the ability to enable them in the pupet modules14:52
aleebecvause they are oslo multicfg options14:52
aleeie.  I can define the same option multiple times in the config file14:53
degorenkoalee, can you please provide me example? like, part of config file with these parameters, i didn't get it now , sorry14:53
aleeso for intance ..14:54
aleeif I have the parameter .. enabled_plugins = ['foo', 'bar']14:54
aleethis will actually show up in the config file as ..14:54
aleeenabled_plugin = foo14:55
degorenkoalee no :)14:55
aleeenabled_plugin = bar14:55
degorenkoif you need to have enabled_plugin=foo,bar14:55
degorenkoit is can be easily done by having array for parameter enabled_plugin - as in your example14:55
aleeie. the same directive shows up twice .. with different values14:55
degorenkoand14:55
degorenkoenabled_plugin = join($enabled_plugin, ',')14:56
degorenkoit will be written to config as enabled_plugin=foo,bar14:56
aleeyeah - thats not how it needs to show up14:56
degorenkoah14:56
degorenkoi see14:57
degorenkoyou need 2 different options14:57
degorenkoon by one right?14:57
degorenkoone*14:57
aleeright - it shows up as14:58
degorenkoi got it now14:58
degorenkowell, it will be a little bit tricky then14:58
aleeso for instance -- option = foo, bar  will be  two lines : option = foo  and option == bar14:58
degorenkoyep14:58
aleeits something called an oslo Multicfg I think -- looking for link14:59
aleedegorenko, EmilienM mentioned that it was being implemented in puppet ..14:59
degorenkoso, for barbican_config provider the problem is that it can manage only one property for each section, we can't do this like you want through standard config provider14:59
degorenkohm14:59
degorenkoreally?14:59
degorenkolet me check14:59
aleedegorenko, anyways thats the reason the options are not implemented in puppet yet -- they are in the barbican package, but I did not know how to implement in puppet.15:01
openstackgerritKyrylo Galanov proposed openstack/puppet-swift: Add rmq credentials to each host in proxy  https://review.openstack.org/29761815:02
aleedegorenko, http://docs.openstack.org/developer/oslo.config/opts.html?highlight=multi#oslo_config.cfg.MultiOpt15:02
degorenkoalee, i don't see any patches for implementing multiple same parameters in config files15:02
*** darkxploit has joined #puppet-openstack15:03
degorenkoalee, no, i'm wrong, we have some patch with mentioned by you doc15:04
degorenkohttps://github.com/openstack/puppet-openstacklib/commit/029c6a74cce2dceff5ae78c138a050d9eb7a6f9b15:04
aleedegorenko, ah good15:04
degorenkoalee, but, i don't know how it works, i didn't see that before15:04
aleedegorenko, cool - I'll try to tease out the needed syntax from the spec tests ..15:06
degorenkoalee, i don't see unit tests for this.... :(15:07
degorenkooh, now15:07
degorenkohttps://github.com/openstack/puppet-openstacklib/blob/master/spec/unit/puppet/util/openstackconfig_spec.rb15:07
degorenkolooks like we need that ^15:07
aleedegorenko, right15:07
degorenkoalee, go ahead then :)15:09
aleedegorenko, yup - fun fun!  I'm going to try to get that in today.15:09
alee(assuming I dont get pulled off on something else)15:10
openstackgerritMerged openstack/puppet-trove: trivial comment paste-o fix  https://review.openstack.org/29734015:11
*** saneax is now known as saneax_AFK15:15
*** michchap_ has quit IRC15:18
Anticimexhm15:19
Anticimexpuppet-ceph claims support for the ceph infernalis release, but there is no systemd code in it (ceph infernalis moved start scripts to systemd)15:19
-openstackstatus- NOTICE: Gerrit is restarting on review.openstack.org in an attempt to address an issue reading an object from the ec2-api repository15:22
EmilienMhello15:27
degorenkohey o/15:28
*** michchap has joined #puppet-openstack15:28
*** EmilienM sets mode: +o degorenko15:28
EmilienMdegorenko: CI was in a bad shape this week end15:30
EmilienMfor different reasons15:30
EmilienMonly one is remaining AFIK: sahara looks broken on Ubuntu15:30
degorenkoEmilienM, also i found a new problem with sahara package :(15:30
EmilienMyeah, we should ping zigo maybe15:30
degorenkoi tried to ping him today15:31
degorenkobut no responce15:31
EmilienMyeah it's PTO for many of us (me included)15:31
degorenkooh, good to know15:31
*** mdorman has joined #puppet-openstack15:39
*** aimon has quit IRC15:43
*** aimon has joined #puppet-openstack15:43
EmilienMdegorenko: do you know if service notify would restart a service if the resource is set to ensure => stopped?15:50
EmilienMwe might have found a bug but I'm not sure15:50
degorenkowell, we have a new state, i think yes15:50
degorenkoso, you have one service which was stopped and it notified another service?15:51
EmilienMdegorenko: https://bugs.launchpad.net/puppet-neutron/+bug/156292215:51
openstackLaunchpad bug 1562922 in puppet-neutron "neutron-server service get notify even when we don't manage service" [Undecided,New]15:51
* degorenko looking15:52
degorenkohmm15:53
degorenkoi'm not sure too15:55
degorenkobut it should not15:55
EmilienMdegorenko: it should not restart?15:57
EmilienMbecause it does for sure15:57
EmilienMwe have the bug in tripleo, when upgrading, jayg can confirm15:57
jaygyes, the tripleo folks are definitely see this behavior even with manager_service and enabled set to false15:58
skolekonovsystemd doesn't start services automatically. right?15:59
EmilienMno, but I think the ~> runs a restart15:59
degorenko~> also means hard dependency16:00
degorenkoso in one catalog we will have config changes at all16:00
degorenkoand only then we will manage service16:00
EmilienMI haven't read puppet core code yet but we might need to check16:01
skolekonovI'll check16:01
skolekonovmanaged should be set to true also as I remember16:01
skolekonovand enabled => false16:01
EmilienMright, the systemd service should be disabled16:02
*** cwolferh has joined #puppet-openstack16:04
skolekonovI checked on Ubuntu. The service (neutron-server) was disabled before Puppet run, then I applied a manifest and it was not restarted. My example http://paste.openstack.org/show/492099/16:07
*** andreww is now known as xarses16:08
EmilienMskolekonov: right16:08
EmilienMskolekonov: can you run puppet again to make sure it's idempotent?16:09
jaygyes, but the service _is_ running in this case, just not managed by puppet16:09
EmilienMjayg: the service is managed by pacemaker16:09
jaygyes, that is what I am meaning16:09
EmilienMjayg: but in your case, the service should be systemd-disabled16:10
jaygso it probably gets the status correctly as running and tries to restart16:10
jaygyou don't think it would report it as running?  I seem to remember it doing so16:10
jaygeven though pacemaker is the one controlling it16:10
jaygmaybe I am remembering wrong, it was a while ago for me16:11
skolekonovEmilienM, as it's an exec it's executed every time, but service is stopped16:12
EmilienMskolekonov: right. I wonder if pacemaker enables it in systemd16:13
skolekonovYou mean it can be connected with pacemaker provider?16:14
EmilienMskolekonov: we don't have pacemaker provider16:14
EmilienMthat manage the service16:14
EmilienMwe're working on it16:14
openstackgerritMerged openstack/puppet-keystone: Fix for pki_setup  https://review.openstack.org/29818316:14
EmilienMjayg: I'm looking at this code now https://github.com/openstack/puppet-pacemaker/blob/master/manifests/resource/service.pp16:15
* jayg looks16:16
EmilienMit actually create a systemd resource in pacemaker with https://github.com/openstack/puppet-pacemaker/blob/master/manifests/resource/systemd.pp16:16
EmilienMso my guess is that it re-creates what puppet disabled16:17
EmilienMI'm not familiar enough with pacemaker though16:17
jaygyes, as I recall, pacemaker uses the systemd unit files, but it supposed to pass in some kind of extra flag to tell systemd to not manage it16:17
jaygthis is part of the resource-agents magic iirc16:18
EmilienMjayg: where is code of that?16:18
jayglet me see if I have a checkout, I am not sure if I ever got the code other than via rpm when we were doing foreman installs16:18
jaygEmilienM: https://github.com/ClusterLabs/resource-agents looks like it, though I am not seeing a systemd one right off, looking more now..16:20
EmilienMjayg: looks like https://github.com/ClusterLabs/resource-agents/blob/01352b5a28c3da04436fd19d886dacdce0a8c1a8/ldirectord/systemd/ldirectord.service.in16:22
EmilienMor not16:22
EmilienManyway, it does not sounds like something in puppet-neutron16:22
EmilienMskolekonov, degorenko: thx guys16:22
skolekonovnp16:23
degorenkoyw :)16:23
*** darkxploit has quit IRC16:26
*** fedexo has joined #puppet-openstack16:27
*** darkxploit has joined #puppet-openstack16:38
*** dfisher has quit IRC16:42
*** dfisher has joined #puppet-openstack16:42
*** fedexo has quit IRC16:49
*** skolekonov is now known as skolekonov|away16:53
xarses dmsimard, please review https://review.openstack.org/#/c/297393/16:58
dmsimardxarses: EmilienM told me about it last week16:58
dmsimardxarses: I've actually only ever installed Ceph on top of Ubuntu16:59
dmsimardI don't know why EPEL would be necessary at all or why it would be installed by default16:59
dmsimarddgurtner or loic might know16:59
xarsesit used to be necessary, as all of the deps where not present16:59
dmsimardthe patch itself is fine and backwards compatible, I'm just questioning whether the entire block should not be removed17:00
xarsesalot has changed since RH acquired Inktank17:00
dmsimardxarses: the dependencies might not have been self contained in the ceph repo, you mean ?17:00
xarsescorrect, they where not in either mainline or ceph repo17:01
dmsimardxarses: eh, I can submit a dummy patch to see what happens if I tear that out17:01
openstackgerritDavid Moreau Simard proposed openstack/puppet-ceph: Do not merge: Test without EPEL  https://review.openstack.org/29833217:04
dmsimardxarses: ^17:04
*** darkxploit has quit IRC17:09
*** foxhunt_away is now known as foxhunt7217:22
EmilienMdegorenko: https://review.openstack.org/#/c/292460/2 seems ok now, it's passing on https://review.openstack.org/#/c/298067/17:27
degorenkonice17:28
EmilienMdegorenko: I replied on https://review.openstack.org/#/c/298047/17:29
EmilienMdegorenko: I copied pasted the file from other projects17:30
degorenkoEmilienM, ack17:30
degorenkodo we need to update README.rst with proper command?17:31
degorenkoor just base commit only17:31
degorenko?17:31
EmilienMdegorenko: commend is good no?17:31
degorenkois it just tox?17:32
degorenkowithout any build_sphinx?17:32
degorenkoi'm not familiar with approach of building docs for repos17:33
degorenkojust some experience from package building, i may wrong17:33
degorenkomay be*17:33
EmilienMdegorenko: try it17:35
EmilienMdegorenko: run tox, and it builds everything17:35
degorenkoit builds env17:35
degorenkook, i trust you :)17:36
EmilienMyeah17:36
EmilienMand run "firefox doc/build/html/index.html"17:36
EmilienMyou can see the output17:36
degorenkoi will try, give me a moment :)17:36
degorenkoEmilienM, cool, works :)17:39
EmilienMcool17:39
EmilienMdmsimard: it fails without epel I think17:39
EmilienM2016-03-28 17:13:40.086 |   Error: Execution of '/usr/bin/yum -d 0 -e 0 -y install ceph' returned 1: Error: Package: 1:ceph-0.94.6-0.el7.x86_64 (ext-ceph-hammer)17:39
EmilienM2016-03-28 17:13:40.087 |              Requires: libleveldb.so.1()(64bit)17:39
EmilienM2016-03-28 17:13:40.087 |    You could try using --skip-broken to work around the problem17:39
EmilienMhttps://jenkins02.openstack.org/job/gate-puppet-ceph-puppet-beaker-rspec-devstack-centos7/1/consoleFull17:39
*** dtantsur is now known as dtantsur|afk17:39
dmsimard... ew17:43
bkerowat17:48
dmsimardI'm asking about it to the Ceph guys17:49
xarsesEmilienM: if it's still broken, why do you want to remove the repo then?17:54
EmilienMxarses: I want to be allowed to remove the repo17:54
EmilienMwhich is why our modules are flexible17:54
EmilienMthe fact it does not work without EPEL is a separated issue17:54
xarsesthat's what I figured17:54
EmilienMmy patch is backward compatible, since it enables it by default like before.17:55
xarsesyep, I'm still fine with it thanks17:57
degorenkoEmilienM, mwhahaha can you guys, please, take a look on this: https://review.openstack.org/#/c/297795/ :)17:57
EmilienMcrinkle: hi, I don't have strong opinion on https://review.openstack.org/#/c/297382/2/reference/projects.yaml but even if we don't release it, it has stable branches every 6 months, so I think it applies to the tags, isn't?17:58
EmilienMdegorenko: I'll look later17:59
degorenkoыгку17:59
degorenkooops17:59
degorenkosure :D17:59
*** ericksonsantos has quit IRC18:07
*** degorenko is now known as _degorenko|afk18:15
crinkleEmilienM: it has stable branches?18:23
EmilienMcrinkle: yes18:23
EmilienMcrinkle: https://github.com/openstack/puppet-openstack_spec_helper/branches18:24
EmilienMbecause the lib needs to know what repos to checkout18:24
crinkleEmilienM: okay i see why18:25
crinkleEmilienM: +118:25
EmilienMcrinkle: thx!18:26
openstackgerritMerged openstack/puppet-openstack-guide: First commit for Puppet OpenStack Guide  https://review.openstack.org/29804718:28
dmsimardEmilienM, xarses: re: EPEL requirement18:30
* dmsimard is tired from all the digging18:30
dmsimardThis is indeed an issue only with the official upstream repositories (which is kind of sad, actually)18:30
dmsimardCentOS is working towards providing repositories for Ceph through it's storage SIG initiative (https://wiki.centos.org/SpecialInterestGroup/Storage/) and this repository has the leveldb package baked in (ex: http://cbs.centos.org/repos/storage7-ceph-hammer-candidate/x86_64/os/Packages/)18:31
openstackgerritCody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/29655718:33
EmilienMhttp://docs.openstack.org/developer/puppet-openstack-guide/ is alive !18:34
openstackgerritCody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/29655718:34
*** foxhunt72 is now known as foxhunt_away18:36
dmsimardxarses: I documented the issue here https://bugs.launchpad.net/puppet-ceph/+bug/156299818:38
openstackLaunchpad bug 1562998 in puppet-ceph "EPEL requirement on CentOS is only because of leveldb" [Undecided,New]18:38
openstackgerritCody Herriges proposed openstack/puppet-keystone: Ensure endpoints created before admin_token_auth  https://review.openstack.org/29837318:50
openstackgerritCody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/29655718:51
openstackgerritCody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/29655718:51
openstackgerritCody Herriges proposed openstack/puppet-keystone: Ensure endpoints created before admin_token_auth  https://review.openstack.org/29837318:56
*** brandon has joined #puppet-openstack19:02
*** aimon has quit IRC19:18
*** aimon has joined #puppet-openstack19:19
*** jtsigma has joined #puppet-openstack19:22
*** angdraug has joined #puppet-openstack19:22
jtsigmacan someone explain to me where in the puppet-ceilometer code (as an example), does it call in the mysql manifest?:  https://github.com/openstack/puppet-ceilometer/blob/master/manifests/db/mysql.pp19:23
jtsigmai tried reading through the code, and don't see where that particular manifest gets executed19:23
EmilienMjtsigma: it is not, you have to declare it in your composition layer19:24
jtsigmaoooh,  ok that makes sense19:24
jtsigmathanks EmilienM19:24
EmilienMnp!19:25
*** foxhunt_away is now known as foxhunt7219:30
openstackgerritCody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint  https://review.openstack.org/29839719:32
openstackgerritCody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/29655719:34
*** tlbr has quit IRC19:53
openstackgerritCody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint  https://review.openstack.org/29839719:59
*** foxhunt72 is now known as foxhunt_away20:00
openstackgerritMerged openstack/puppet-ceph: repo: allow to disable EPEL repository  https://review.openstack.org/29739320:04
*** johnmilton has quit IRC20:09
*** dprince has quit IRC20:17
*** jayg is now known as jayg|g0n320:28
openstackgerritMerged openstack/puppet-oslo: Fix values for rabbit_hosts  https://review.openstack.org/29396420:30
_odyIs it just me or is CI really slow today?20:36
xarsesseems slow20:36
EmilienM_ody: what is slow?20:38
EmilienMgetting a node?20:38
_odyEmilienM: Running the actual tests.20:39
_odyI keep walking around the office thinking it is my wifi.20:39
EmilienM_ody: I haven't noticed long runs20:39
EmilienM_ody: do you have logs handy to see it's long?20:39
EmilienMwhere do you see it's long?20:39
_odyEmilienM: I am just surprised my last change to puppet-cinder has been running tests for over 40 minutes.20:41
EmilienM_ody: https://review.openstack.org/#/c/298397/ ?20:41
EmilienMit's almost finished: https://jenkins07.openstack.org/job/gate-puppet-openstack-integration-3-scenario002-tempest-devstack-centos7/42/console20:42
EmilienM35 min20:42
EmilienMit's average20:42
EmilienMit usually takes ~33 min20:42
_odyEmilienM:How did you figure out which jenkins that was running on?20:43
EmilienMhttp://status.openstack.org/zuul/20:43
_odyo.O20:43
_odyI knew that existed just never noticed how to get to jenkins from it.20:44
EmilienM_ody: "TIL" :-)20:45
_odyoh.  I just didn't click it the right place it would seem.20:46
mwhahahahttp://cdn.meme.am/instances/66957857.jpg20:50
EmilienMmwhahaha: I use this picture when my friends (working on windows) complain about their bugs20:52
mwhahaha:D20:52
EmilienMmwhahaha: also this one: https://goo.gl/Gt7F2l20:53
*** ericksonsantos has joined #puppet-openstack21:02
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge  https://review.openstack.org/29610221:02
_odymaybe I'll just need to finish up the puppet4 stuff tomorrow.  It's taking me 15 minutes to clone a single repo from git.openstack.org.21:07
openstackgerritMerged openstack/puppet-glance: Add posibility to configure Glance backend settings for Glare service  https://review.openstack.org/29779521:07
EmilienM_ody: have you talked to #openstack-infra folks?21:07
* _ody is hung on running "git review" now21:07
_odyEmilienM: Was reading through scroll and and asking folks.  Haven't broadcasted the issue yet.21:08
mwhahahado we have an example of the glance_image provider usage anywhere21:12
*** johnmilton has joined #puppet-openstack21:13
mwhahahanm it's in the acceptance scripts21:14
EmilienMmwhahaha: https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/provision.pp#L67-L8721:14
mwhahahagracias21:15
openstackgerritMerged openstack/puppet-tripleo: Redirect to https for horizon  https://review.openstack.org/29445621:16
*** johnmilton has quit IRC21:16
*** johnmilton has joined #puppet-openstack21:17
*** tlbr has joined #puppet-openstack21:18
*** johnmilton has quit IRC21:21
*** johnmilton has joined #puppet-openstack21:23
*** aimon has quit IRC21:29
*** aimon has joined #puppet-openstack21:29
openstackgerritMerged openstack/puppet-openstack-integration: revert temporary workaround on centos  https://review.openstack.org/29246021:33
*** johnmilton has quit IRC21:42
openstackgerritMerged openstack/puppet-swift: Add rmq credentials to each host in proxy  https://review.openstack.org/29761821:47
*** johnmilton has joined #puppet-openstack21:55
EmilienMcrinkle: when you have some time, please look https://review.openstack.org/#/c/296102/ - I remember you deployed linuxbridge for infra cloud, thanks!21:55
openstackgerritMerged openstack/puppet-cinder: Add osapi_max_limit parameter  https://review.openstack.org/29693121:57
crinkleEmilienM: +221:59
openstackgerritAlex Krzos proposed openstack/puppet-keystone: Tune number of apache processes/threads for Keystone in apache.  https://review.openstack.org/29734221:59
openstackgerritAlex Krzos proposed openstack/puppet-keystone: Tune number of apache processes/threads for Keystone in apache.  https://review.openstack.org/29734222:01
*** johnmilton has quit IRC22:07
openstackgerritCody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint  https://review.openstack.org/29839722:11
*** dhollinger has quit IRC22:13
*** ducttape_ has quit IRC22:19
*** mdorman has quit IRC22:20
EmilienMcrinkle: thx22:20
*** markvoelker has quit IRC22:29
openstackgerritCody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint  https://review.openstack.org/29839722:31
openstackgerritCody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint  https://review.openstack.org/29839722:43
openstackgerritCody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint  https://review.openstack.org/29839723:04
EmilienMcrinkle: do you know of Gemfile groups are ignored on the dependencies? example: if the group is in puppet-openstack_spec_helper/Gemfile, does puppet-nova will install the right deps?23:05
EmilienMlast week you mentionned we still could use a gemfile but I tested it and it does not work, maybe I did wrong23:05
crinkleEmilienM: i don't know :/23:05
* crinkle will poke it23:06
EmilienMok np23:06
* EmilienM reading the fm :)23:06
colonwqEmilienM, When you get a chance, can you look at 296002 and 289027? I think I have polished this mud balls. They deploy correctly in my local environment.23:11
crinkleEmilienM: you're right, adding it to the gemfile totally doesn't work23:11
crinkleEmilienM: but adding it to the gemspec does23:11
crinkleEmilienM: so i'm not sure how to get groups to work in that case23:12
EmilienMthere are groups in gemspec, but not all23:12
EmilienMhttp://guides.rubygems.org/specification-reference/#add_development_dependency23:12
EmilienMdevelopment and runtime deps only23:13
crinkleright...that's not really what we want23:13
EmilienMotherwise, we stop creating groups :(23:13
EmilienMcolonwq: I'll review it, sure23:13
colonwqEmilienM, Thanks.23:14
EmilienMcolonwq: 296002 has 6123:14
EmilienM-1*23:14
openstackgerritMerged openstack/puppet-keystone: Ensure endpoints created before admin_token_auth  https://review.openstack.org/29837323:15
openstackgerritMerged openstack/puppet-nova: Autorequire nova security group  https://review.openstack.org/29654823:15
EmilienMcrinkle: what is your opinion on whether or not using gem groups?23:16
colonwqEmilienM, dprince thinks I have my depends-on backwards with the two patches.23:17
*** saneax_AFK is now known as saneax23:17
crinkleEmilienM: so the reasons for having groups were 1) beaker stopped supporting ruby 1.8.7 and separating the groups allowed us to still run unit tests on 1.8.7 (i think this affected the puppetlabs modules more than us) and 2) faster installs since not all the jobs needed all the gems23:17
*** ducttape_ has joined #puppet-openstack23:17
crinkleEmilienM: I think 1) isn't an issue any more and we can live with 2)23:18
crinkleer, we can live with slower installs23:18
EmilienMI agree23:18
colonwqEmilienM, I reversed the Depends-On direction of the patches.23:25
EmilienMcolonwq: ok I'll look thx23:25
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper: gemspec: add runtime dependencies for modules  https://review.openstack.org/29846523:29
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/29846723:30
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-cookiecutter: metadata.json: bump to Mitaka  https://review.openstack.org/29600523:31
*** ducttape_ has quit IRC23:43
*** ducttape_ has joined #puppet-openstack23:46
*** xarses has quit IRC23:49

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