Tuesday, 2015-08-04

« Monday, 2015-08-03 Index Wednesday, 2015-08-05 »
*** xingchao has quit IRC00:05
*** gfidente|afk has quit IRC00:08
*** bklei has quit IRC00:29
*** bklei has joined #puppet-openstack00:30
*** linkedinyou has quit IRC00:40
*** saneax has quit IRC00:46
*** gildub has quit IRC00:46
*** tiswanso has joined #puppet-openstack00:55
*** tiswanso has quit IRC00:58
*** tiswanso has joined #puppet-openstack00:58
*** xarses has quit IRC01:01
*** xingchao has joined #puppet-openstack01:04
*** xingchao has quit IRC01:13
*** gildub has joined #puppet-openstack01:20
*** gildub has joined #puppet-openstack01:20
*** angdraug has quit IRC01:25
*** xarses has joined #puppet-openstack01:36
*** rcallawa has joined #puppet-openstack01:41
*** rcallawa has quit IRC01:42
*** rcallawa has joined #puppet-openstack01:43
*** rcallawa_ has joined #puppet-openstack01:51
*** rcallawa has quit IRC01:53
*** degorenko has quit IRC01:54
*** bklei has quit IRC02:01
*** xingchao has joined #puppet-openstack02:16
openstackgerritDan Prince proposed openstack/puppet-tripleo: Add pre_create_files resource and helper class  https://review.openstack.org/20877402:27
*** mesamoo has joined #puppet-openstack02:41
*** rcallawa_ has quit IRC03:32
*** rcallawa has joined #puppet-openstack03:33
*** tiswanso has quit IRC03:55
*** gildub has quit IRC03:59
*** Madkiss has quit IRC04:06
*** Madkiss has joined #puppet-openstack04:06
*** Madkiss has joined #puppet-openstack04:06
*** xingchao has quit IRC04:21
*** richm has quit IRC04:32
*** linkedinyou has joined #puppet-openstack04:48
*** Madkiss has quit IRC05:07
*** Madkiss has joined #puppet-openstack05:08
*** Madkiss has joined #puppet-openstack05:08
*** saneax has joined #puppet-openstack05:27
*** bogdando has quit IRC05:36
*** bogdando has joined #puppet-openstack05:37
*** xingchao has joined #puppet-openstack05:41
*** jtomasek has joined #puppet-openstack06:06
*** xingchao has quit IRC06:53
*** xingchao has joined #puppet-openstack06:54
*** dgurtner has joined #puppet-openstack06:57
*** dgurtner has quit IRC07:01
*** dgurtner has joined #puppet-openstack07:01
*** degorenko has joined #puppet-openstack07:08
openstackgerritRicardo Carrillo Cruz proposed openstack/puppet-ironic: Add bifrost manifest  https://review.openstack.org/20202507:44
*** gildub has joined #puppet-openstack07:51
openstackgerritRicardo Carrillo Cruz proposed openstack/puppet-ironic: Add bifrost manifest  https://review.openstack.org/20202508:02
*** jistr has joined #puppet-openstack08:08
*** paramite has joined #puppet-openstack08:08
*** chem has joined #puppet-openstack08:13
openstackgerritMartin Mágr proposed openstack/puppet-nova: Allow to change archive destination  https://review.openstack.org/20791508:23
*** paramite is now known as paramite|afk08:26
*** paramite|afk is now known as paramite08:28
*** dgurtner has quit IRC08:35
*** bogdando has quit IRC08:35
*** Madkiss has quit IRC08:35
*** hogepodge has quit IRC08:35
*** dachary has quit IRC08:35
*** ianw has quit IRC08:35
*** colby_ has quit IRC08:35
*** tdb has quit IRC08:35
*** mattymo has quit IRC08:35
*** crinkle has quit IRC08:35
*** zino_ has quit IRC08:35
*** ric` has quit IRC08:35
*** joshsamuelson has quit IRC08:35
*** dachary1 has joined #puppet-openstack08:37
*** dgurtner has joined #puppet-openstack08:37
*** bogdando has joined #puppet-openstack08:37
*** Madkiss has joined #puppet-openstack08:37
*** hogepodge has joined #puppet-openstack08:37
*** dachary has joined #puppet-openstack08:37
*** ianw has joined #puppet-openstack08:37
*** colby_ has joined #puppet-openstack08:37
*** tdb has joined #puppet-openstack08:37
*** zino_ has joined #puppet-openstack08:37
*** mattymo has joined #puppet-openstack08:37
*** crinkle has joined #puppet-openstack08:37
*** ric` has joined #puppet-openstack08:37
*** joshsamuelson has joined #puppet-openstack08:37
*** cameron.freenode.net sets mode: +o crinkle08:37
*** dachary1 has quit IRC08:42
*** dachary1 has joined #puppet-openstack08:45
openstackgerritMerged openstack/puppet-cinder: Update default notification_driver for ceilometer  https://review.openstack.org/20480808:48
openstackgerritMerged openstack/puppet-trove: Introduce trove:config to manage custom options  https://review.openstack.org/20782308:49
openstackgerritIvan Berezovskiy proposed openstack/puppet-ceilometer: Add ability to specify ttl and timeout parameters  https://review.openstack.org/20789808:50
*** gfidente has joined #puppet-openstack08:58
openstackgerritMartin Mágr proposed openstack/puppet-cinder: DB purge cron job  https://review.openstack.org/20893909:33
*** saneax has quit IRC09:38
*** openstackgerrit_ has joined #puppet-openstack09:53
*** sac has quit IRC09:58
*** sac has joined #puppet-openstack09:59
*** gildub has quit IRC10:06
openstackgerritLimor Stotland proposed openstack/puppet-mistral: puppet-mistral: initial commit  https://review.openstack.org/20845710:07
*** saneax has joined #puppet-openstack10:08
openstackgerritLimor Stotland proposed openstack/puppet-mistral: puppet-mistral: initial commit  https://review.openstack.org/20845710:13
*** gildub has joined #puppet-openstack10:23
*** xingchao has quit IRC10:24
*** xingchao has joined #puppet-openstack10:41
*** ToMiles has quit IRC10:44
*** xingchao has quit IRC10:56
openstackgerritLimor Stotland proposed openstack/puppet-mistral: puppet-mistral: initial commit  https://review.openstack.org/20845710:59
*** xingchao has joined #puppet-openstack11:01
*** xingchao has quit IRC11:05
*** degorenko has quit IRC11:06
*** degorenko has joined #puppet-openstack11:08
openstackgerritMartin Mágr proposed openstack/puppet-cinder: DB purge cron job  https://review.openstack.org/20893911:08
EmilienMgood morning11:19
*** ChanServ sets mode: +o EmilienM11:20
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: acceptance: test OVS bridge listing  https://review.openstack.org/19217611:21
*** rcallawa has quit IRC11:22
*** rcarrillocruz has joined #puppet-openstack11:26
rcarrillocruzEmilienM , crinkle: hiya, i amended https://review.openstack.org/#/c/202025/11:28
EmilienMrcarrillocruz: ok !11:28
*** openstackgerrit has quit IRC11:31
*** openstackgerrit_ is now known as openstackgerrit11:31
*** openstackgerrit has quit IRC11:32
*** gsilvis has quit IRC11:32
*** openstackgerrit_ has joined #puppet-openstack11:32
*** openstackgerrit_ is now known as openstackgerrit11:32
*** openstackgerrit_ has joined #puppet-openstack11:33
*** dprince has joined #puppet-openstack11:39
*** gildub has quit IRC11:50
EmilienMmgagne: have you filed a bug for your issue with glance stores?12:07
*** openstackgerrit has quit IRC12:16
*** openstackgerrit has joined #puppet-openstack12:17
*** rcallawa has joined #puppet-openstack12:18
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/20268912:19
*** jayg|g0n3 is now known as jayg12:21
*** morazi has quit IRC12:22
sbadiahi!12:24
sbadia_ody_: here?12:24
openstackgerritDan Prince proposed openstack/puppet-tripleo: Add pre_create_files resource and helper class  https://review.openstack.org/20877412:27
openstackgerritDan Prince proposed openstack/puppet-tripleo: tripleo::noop: add support for more resources  https://review.openstack.org/20903012:27
EmilienMsbadia: early for him I guess :)12:29
sbadiayep :/12:30
openstackgerritIvan Berezovskiy proposed openstack/puppet-nova: Add nameservers (dns) parameters  https://review.openstack.org/20278812:31
openstackgerritMerged openstack/puppet-glance: Add swift_store_region parameter to glance::backend::swift  https://review.openstack.org/20865412:33
openstackgerritAndrey Brito proposed openstack/puppet-murano: Update Murano README  https://review.openstack.org/20763912:35
openstackgerritMerged openstack/puppet-nova: Support allow_start and allow_end parameters  https://review.openstack.org/20609912:36
openstackgerritMerged openstack/puppet-neutron: N1Kv: Adding nexus1000v puppet class  https://review.openstack.org/20479012:37
openstackgerritMerged openstack/puppet-designate: acceptance: Remove dirty fix for empty values (fixed in inifile module)  https://review.openstack.org/19500312:42
degorenkoEmilienM, hey :) Can you review my patch? https://review.openstack.org/205987 :-) Also, is ready to merge next: https://review.openstack.org/20698112:43
*** dachary1 has quit IRC12:48
openstackgerritAndrey Brito proposed openstack/puppet-murano: Update Murano README  https://review.openstack.org/20763912:50
openstackgerritMerged openstack/puppet-keystone: WSGI: use real service name in restart_keystone Exec  https://review.openstack.org/20801912:51
openstackgerritMerged openstack/puppet-keystone: admin_user_domain and admin_project_domain can now be equal  https://review.openstack.org/20743712:51
openstackgerritMerged openstack/puppet-nova: Add ability to set default baremetal filters  https://review.openstack.org/20221512:51
openstackgerritMerged openstack/puppet-trove: Fix catalog compilation when not configuring endpoint  https://review.openstack.org/20743512:51
crinklercarrillocruz: can you clarify your comment on patchset 7 of https://review.openstack.org/#/c/202025 ? I believe ensure => latest on vcsrepo will cause puppet to try to update to the latest commit on each run, which may or may not be what someone wants, which is why i think it should be a class parameter and not hard-coded to latest or present12:51
crinklercarrillocruz: do you have a response to my other non-inline comment? i'm more concerned about that12:52
openstackgerritYanis Guenane proposed openstack/puppet-sahara: Creation of sahara::db::sync  https://review.openstack.org/19668312:52
*** rcallawa_ has joined #puppet-openstack12:58
openstackgerritLukas Bezdicka proposed openstack/puppet-keystone: beaker: test RDO/liberty  https://review.openstack.org/20904412:59
*** rcallawa has quit IRC13:01
rcarrillocruzcrinkle: i can parameterize sure, but having it at latest was a bug. If you have it set to present the resource will clone it and put whichever revision you pass as an arg (a commit, a branch, a tag, etc)13:07
rcarrillocruzso having as before as 'ensure=>latest' forced to have it to latest commit from the branch specified13:07
rcarrillocruzmakes sense to have it as default 'present' and leave the operator the change to put it as absent tho13:07
rcarrillocruzi will push in a bit13:07
crinklercarrillocruz: cool13:08
crinklercarrillocruz: I'm find with keeping ensure => present hardcoded for now, too13:08
*** skolekonov has joined #puppet-openstack13:09
crinklercarrillocruz: as it is, if it's set to ensure => latest then it might cause merge conflicts with the baremetal.json and the group_vars_all13:09
*** morazi has joined #puppet-openstack13:10
*** dfisher has joined #puppet-openstack13:10
*** dachary1 has joined #puppet-openstack13:11
rcarrillocruzbut i can't restrict a given resource to just present/absent, can I?13:11
rcarrillocruzunless i do some dirty if comparison within the manifest13:11
crinkleyeah that isn't ideal13:12
crinklercarrillocruz: is there some way this could be done without modifying the repo?13:12
rcarrillocruzyou mean referencing the conf files in a folder that's outside of the cloned repo, right13:13
rcarrillocruz?13:13
crinklercarrillocruz: right13:13
rcarrillocruzbaremetal.json can be done13:13
rcarrillocruzhowever13:13
rcarrillocruzthe way ansible works13:13
rcarrillocruzit just expects a certain layout13:13
rcarrillocruzand in this case, group_vars/all13:13
rcarrillocruzit's expected at the top level13:14
rcarrillocruznot sure if that can be tweaked with ansibe.cfg or some flag13:14
EmilienMcrinkle, sbadia, mfisch, clayton, spredzy: do you have something to add in our agenda (pretty empty now) ?13:14
openstackgerritMartin Mágr proposed openstack/puppet-cinder: DB purge cron job  https://review.openstack.org/20893913:14
crinklercarrillocruz: hrm okay13:14
crinklercarrillocruz: I can +2 for now and we can iterate on it maybe13:15
claytonnothing here, and mfisch is on vacation until next week13:15
crinkleEmilienM: nothing for me13:15
rcarrillocruzthat's one of the issues for mixing conf + code, no hiera goodness here :/13:15
*** tiswanso has joined #puppet-openstack13:15
spredzyEmilienM, not this week13:15
dgurtnerdmsimard: xarses dachary1 ping re: https://review.openstack.org/#/c/117422/13:16
rcarrillocruzcrinkle: k, let me push another change for the ensure param, then we land13:16
rcarrillocruzi'll think something and iterate later13:16
rcarrillocruzthx13:16
dgurtnerdmsimard: xarses dachary1 consequently we should also get https://review.openstack.org/#/c/153783/ merged otherwise it's actually broken13:17
EmilienMpabelanger, crinkle: having our jobs (syntax + unit) on centos7 is much faster13:19
crinklenice13:19
EmilienMpabelanger, crinkle: maybe this feedback is a bit early13:19
EmilienMpabelanger, crinkle: but I noticed that this morning13:19
EmilienMthe jobs start immediately or almost13:19
sbadianothing here too :/13:19
openstackgerritMatthew Mosesohn proposed openstack/puppet-keystone: Update get_endpoint to use credentials auth_url  https://review.openstack.org/20789013:19
EmilienMcrinkle: we should run the trusty jobs on centos7, using a trusty container :P13:19
*** xingchao has joined #puppet-openstack13:20
*** dachary2 has joined #puppet-openstack13:23
*** xingchao has quit IRC13:23
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Add basic manifests  https://review.openstack.org/19549313:23
*** dachary1 has quit IRC13:25
*** dachary2 has quit IRC13:27
*** dachary1 has joined #puppet-openstack13:27
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Sanitize array parameters  https://review.openstack.org/20906313:30
socialEmilienM: https://jenkins02.openstack.org/job/gate-puppet-keystone-puppet-beaker-rspec-dsvm-centos7/144/console13:30
EmilienMwoot13:31
EmilienMsocial: how many beers I owe you?13:31
social013:32
pabelangerEmilienM: Yup, that's was the goal. devstack-trusty gets hammered with devstack-gates13:32
dfisheri'll take them.13:32
socialEmilienM: simple quotes13:32
pabelangergoing to checkout other jobs today13:32
EmilienMpabelanger: cool, I suggest we wait a bit for the rest (lint?) now13:32
socialEmilienM: false without quotes just made it to ignore the gpg option13:32
EmilienMsocial: ok... good13:32
EmilienMsocial: that means we can go ahead maybe13:32
EmilienMsocial: thanks *a lot*13:33
openstackgerritYanis Guenane proposed openstack/puppet-heat: Creation of heat::db::sync  https://review.openstack.org/19664613:33
EmilienMdfisher: no way13:33
pabelangerEmilienM: Ya, lint going to take some looking into since it is more complicated.  But plan to spend some time on it today13:33
socialyeah, I do want to move ahead with liberty13:33
EmilienMpabelanger: can you attend our weekly meeting today?13:33
EmilienMpabelanger: we make a CI status13:34
pabelangerEmilienM: yes, in calendar13:34
openstackgerritDrew Fisher proposed openstack/puppet-neutron: Remove POSIX users, groups, and file modes.  https://review.openstack.org/20906813:34
EmilienMpabelanger: I was thinking, we should install modules in /etc/puppet/modules13:35
EmilienMfirst of all, because this is the place to be, and also because my work on logs will list modules from there13:35
EmilienMhttps://review.openstack.org/20669313:35
pabelangerEmilienM: What benifit does installing them into /etc/puppet give us?  Over using the --modulepath to local directory?13:37
pabelangerlooking at your patchset13:37
EmilienMpabelanger: because this is where beaker install modules now13:37
EmilienMand also because this is where other infra modules are installed13:37
EmilienMI think it makes just sense to keep them in the right place13:37
EmilienMpabelanger: yeah, look at the line where I list OpenStack project using metadata.json13:38
pabelangerEmilienM: cannot speak for beaker, but we are actually going to move them from /etc/puppet/modules to some other place in -infra.13:38
socialEmilienM: one offtopic note, on centos I got facter installed from package and facter istalled from gem by bundle install, which kinda broke the spec test a lot13:38
EmilienMsocial: never seen that before13:38
EmilienMpabelanger: why? where?13:38
pabelangerEmilienM: see clark comments: https://review.openstack.org/#/c/208088/13:39
pabelangerEmilienM: Ya, I can go both ways but prefer installing modules in local directory, since we don't need sudo access to install them.13:40
pabelangerhowever, we do need sudo to run puppet, so there is that13:40
EmilienMpabelanger: this one is in my review list for today13:40
socialEmilienM: btw I'd love to have this brought up https://review.openstack.org/#/c/201229/13:43
EmilienMpabelanger: his comment makes sense though13:43
EmilienMsocial: adding it in my review list13:43
pabelangeranother one is how we install beaker.  I don't think we should be doing sudo gem install beaker, personally I don't see the reason for this.  Aside from it gets installed into /usr/local/bin path.  Was going to review the changes, and just install beaker into local GEM_HOME path and update jobs.  For me, removing sudo access when ever possible just makes things better for local sandbox testing (when13:44
pabelangerpossible)13:44
pabelangerI'm working on an email to -infra about some of this stuff around sudo access for installation of packages, it revolves around the ability to reuse a node, if possible, to help reduce the stress on the gate13:45
crinklewhich patch are we talking about? why are we installing beaker with sudo?13:45
*** ferest has joined #puppet-openstack13:45
pabelangercrinkle: I don't know why, I can only guess to put beaker in the $PATH variable13:45
pabelangerfor our integration job, I've changed it to use GEM_HOME for beaker install13:46
pabelangerIs is wrong, to me: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/puppet-module-jobs.yaml#L913:47
*** saneax has quit IRC13:47
pabelangerthis is correct: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/puppet-module-jobs.yaml#L20113:47
*** tiswanso has quit IRC13:47
openstackgerritIvan Berezovskiy proposed openstack/puppet-ceilometer: Add ability to specify ttl and timeout parameters  https://review.openstack.org/20789813:48
pabelangerNeed some coffee, and review emails from holiday13:48
*** tiswanso has joined #puppet-openstack13:48
openstackgerritIgor Gajsin proposed openstack/puppet-cinder: set parameter host in vmdk.pp  https://review.openstack.org/15700413:48
crinkleoh installing bundler, not beaker13:50
pabelangerdon't ya, my bad (lack of coffee)13:50
pabelangerand I keep swapping dem names in my head13:50
*** dachary1 has quit IRC13:51
*** dachary1 has joined #puppet-openstack13:51
*** dachary1 has quit IRC13:52
*** dachary1 has joined #puppet-openstack13:52
EmilienMspredzy: re - ::db classes - I think it's cool13:54
EmilienMit's consistent with other modules13:55
*** bklei has joined #puppet-openstack13:55
*** bklei_ has joined #puppet-openstack13:56
*** bklei has quit IRC13:56
spredzyEmilienM, cool. I'll send an email later either today or tomorrow to explain the intent.13:56
EmilienMspredzy, crinkle: I was wondering your feelings on https://review.openstack.org/#/c/201229/13:56
*** mdorman has joined #puppet-openstack13:56
EmilienMfor me, it's really specific to be in our puppet modules13:57
*** dachary1 has quit IRC13:57
crinkleI have no idea what best practice is there13:57
crinkleI haven't seen any of our other modules do that13:57
EmilienMme neither13:57
EmilienMcrinkle: to me, it depends on which package release you use, etc13:58
EmilienMalso, openstack-selinux should take care of it13:59
EmilienMat least it is partially taking care of this atm13:59
*** ferest has quit IRC13:59
pabelangerEmilienM: I would agree. If there is a selinux issue, seems selinux should be fixed outside the puppet modules. Or inside vendor puppet modules13:59
EmilienMpabelanger: social is working on packstack and his use case is fair13:59
*** dachary1 has joined #puppet-openstack13:59
spredzyEmilienM, mainly agree with pabelanger, also in order to be able to by passe packaging pontential issue, maybe adding a parameter in puppetlabs/apache that can take a list of boolean to enable and enable them14:01
EmilienMw00t14:01
EmilienMthat's also a great idea I guess14:01
spredzyIf we start having selinux bool enablement within the code, it might clutter it quite easily14:01
EmilienMand add the interface in puppet-horizon14:01
spredzyhttpd::enabled_selinux: ['httpd_can_network_connect_db', 'httpd_use_openstack']14:03
EmilienMspredzy: you can give your proposal in gerrit14:03
*** dachary1 has quit IRC14:04
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: Enable https and GPG check for EPEL  https://review.openstack.org/20908114:08
*** ferest has joined #puppet-openstack14:18
*** dachary1 has joined #puppet-openstack14:20
*** ferest has quit IRC14:21
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: acceptance: bump to Liberty release  https://review.openstack.org/20909714:31
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: acceptance: bump to Liberty release  https://review.openstack.org/20909714:33
openstackgerritKuo-Le.Mei proposed openstack/puppet-nova: Fixed value `rbd.keyring` not update  https://review.openstack.org/20390914:33
*** xingchao has joined #puppet-openstack14:34
*** dachary1 has quit IRC14:37
EmilienMour weekly meeting will start in 20min on #openstack-meeting-4 - our agenda is here: https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-2015080414:40
spredzyRspec question if anyone knows: In this review https://review.openstack.org/#/c/202574/3/lib/puppet_x/openstack/util/ini_file.rb,cm I require a file that is outside the scope of my modules. This causes any rspec test to fail due to cannot load such file -- /home/spredzy/Projects/openstack/puppet-openstack-modules/inifile/lib/puppet/util/ini_file (Puppet::Error). Since I don't have the inifile there during my test if tails to load it.14:41
spredzyWould anyone have an idea on how I can do/solve that ?14:41
spredzyso far all the test file is : http://pastebin.com/iVMX6C8k14:42
EmilienMin spec_helper.rb:$LOAD_PATH.push(File.join(File.dirname(__FILE__), 'fixtures', 'modules', 'openstacklib', 'lib'))14:45
EmilienMspredzy: ^14:45
EmilienMmaybe you are missing that14:45
EmilienMspredzy: look in puppet-keystone how it does with openstack provider14:45
EmilienMit's importing it from openstacklib14:46
spredzyEmilienM, ack checking14:46
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Update parameters for Sahara  https://review.openstack.org/20598714:47
*** vsaienko has joined #puppet-openstack14:50
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: acceptance: bump to Liberty release  https://review.openstack.org/20909714:51
EmilienMpabelanger: how did you make it work on trusty ????? https://review.openstack.org/#/c/207078/14:53
EmilienMpabelanger: it works now \o/14:53
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Add basic manifests  https://review.openstack.org/19549314:55
*** social has quit IRC14:56
*** angdraug has joined #puppet-openstack14:56
*** social has joined #puppet-openstack14:58
mdormanfyi i will be late to the meeting14:59
sbadiamdorman: ack, np14:59
*** tiswanso has quit IRC14:59
*** tiswanso has joined #puppet-openstack15:01
guimalufwhich channel is the meeting?15:03
guimaluffound it15:03
pabelangerEmilienM: I didn't. There is a race / timeout condition on apt-get I think15:05
openstackgerritColleen Murphy proposed openstack/puppet-openstack_extras: Sync Gemfile  https://review.openstack.org/20911615:11
openstackgerritColleen Murphy proposed openstack/puppet-openstack_extras: Sync Gemfile  https://review.openstack.org/20911615:11
openstackgerritColleen Murphy proposed openstack/puppet-openstack_extras: Add acceptance tests for repo::* classes  https://review.openstack.org/20718015:11
openstackgerritColleen Murphy proposed openstack/puppet-openstack_extras: Add auth_file acceptance test  https://review.openstack.org/20717915:11
openstackgerritRicardo Carrillo Cruz proposed openstack/puppet-ironic: Add bifrost manifest  https://review.openstack.org/20202515:13
sbadiasorry for the spam :)15:15
openstackgerritSebastien Badia proposed openstack/puppet-ceilometer: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/20912115:15
openstackgerritSebastien Badia proposed openstack/puppet-designate: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18986415:15
openstackgerritColleen Murphy proposed openstack/puppet-openstack_extras: Sync Gemfile  https://review.openstack.org/20911615:15
openstackgerritColleen Murphy proposed openstack/puppet-openstack_extras: Add acceptance tests for repo::* classes  https://review.openstack.org/20718015:15
openstackgerritColleen Murphy proposed openstack/puppet-openstack_extras: Add auth_file acceptance test  https://review.openstack.org/20717915:15
openstackgerritSebastien Badia proposed openstack/puppet-gnocchi: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18986615:15
openstackgerritSebastien Badia proposed openstack/puppet-horizon: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18986915:15
openstackgerritSebastien Badia proposed openstack/puppet-keystone: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987115:15
openstackgerritSebastien Badia proposed openstack/puppet-neutron: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/20912215:15
openstackgerritSebastien Badia proposed openstack/puppet-openstack_extras: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987615:15
openstackgerritSebastien Badia proposed openstack/puppet-sahara: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987715:16
openstackgerritSebastien Badia proposed openstack/puppet-tempest: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987915:16
openstackgerritSebastien Badia proposed openstack/puppet-trove: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18988115:16
openstackgerritSebastien Badia proposed openstack/puppet-vswitch: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18988315:16
openstackgerritSebastien Badia proposed openstack/puppet-cinder: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18986315:16
openstackgerritSebastien Badia proposed openstack/puppet-glance: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18986515:16
openstackgerritSebastien Badia proposed openstack/puppet-heat: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18986815:16
openstackgerritSebastien Badia proposed openstack/puppet-ironic: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987015:16
openstackgerritSebastien Badia proposed openstack/puppet-manila: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987215:16
openstackgerritSebastien Badia proposed openstack/puppet-nova: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987415:16
openstackgerritSebastien Badia proposed openstack/puppet-openstacklib: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987515:16
openstackgerritSebastien Badia proposed openstack/puppet-swift: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987815:16
openstackgerritSebastien Badia proposed openstack/puppet-tripleo: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18988015:16
openstackgerritSebastien Badia proposed openstack/puppet-tuskar: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18988215:16
paramiteoh boy15:17
sbadiamsync run: https://review.openstack.org/#/q/status:open+branch:master+topic:msync_rspec_version,n,z15:17
iurygregoryomg o.o15:18
*** rcallawa_ has quit IRC15:21
*** rcallawa has joined #puppet-openstack15:21
*** IBerezovskiy has joined #puppet-openstack15:23
openstackgerritSebastien Badia proposed openstack/puppet-modulesync-configs: spec: Fix nodesets errors (x64 for el and amd64 for debian)  https://review.openstack.org/20912715:27
openstackgerritSebastien Badia proposed openstack/puppet-modulesync-configs: sample config (update namespace)  https://review.openstack.org/20912815:27
openstackgerritMartin Mágr proposed openstack/puppet-keystone: Allow to change archive destination  https://review.openstack.org/20912915:27
*** cdelatte has joined #puppet-openstack15:29
rcarrillocruzcrinkle: thx for the +2 , just saw15:31
EmilienMrcarrillocruz: will approve when CI is green15:34
rcarrillocruz\o/15:34
rcarrillocruzthx EmilienM15:34
EmilienMcrinkle: are you going to patch project-config to run acceptance on _extras ?15:35
*** skolekonov has quit IRC15:35
crinkleEmilienM: ah i think i was and then forgot to submit it15:36
*** IBerezovskiy has quit IRC15:36
*** IBerezovskiy has joined #puppet-openstack15:37
*** dachary1 has joined #puppet-openstack15:40
guimalufpabelanger, EmilienM you said that I should rerun the CI tests. I do this by submiting an empty patch set, or there is another way?15:40
EmilienMguimaluf: go in your review, and drop a comment 'recheck'15:41
guimalufgreat! :)15:41
EmilienMguimaluf: it will re-trigger zuul and Jenkins jobs15:41
guimalufvery nice! thanks :)15:42
EmilienMcrinkle: we might some reviews on the integration serie of patches, starting from https://review.openstack.org/#/c/203742/15:42
pabelangeryup 'recheck'15:45
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: acceptance: bump to Liberty release  https://review.openstack.org/20909715:47
*** paramite is now known as paramite|afk15:49
*** paramite|afk is now known as paramite15:49
openstackgerritMerged openstack/puppet-glance: Update README.md for Glance  https://review.openstack.org/20780515:53
*** paramite has quit IRC15:54
*** tiswanso has quit IRC15:56
*** tiswanso has joined #puppet-openstack15:56
*** tiswanso has quit IRC15:57
EmilienMHunner, crinkle: I found a blocker for having OpenStack Liberty working on centos7: https://github.com/puppetlabs/puppetlabs-mysql/pull/74116:07
*** jistr has quit IRC16:09
HunnerEmilienM: for approved modules, do you know if the nova feedback went in? Should we check it again? Also, which one should be next?16:12
EmilienMHunner: no, I did not spend time on it yet16:13
HunnerI'll look at the mysql PR; mysql has a release next week16:13
EmilienMoh nice16:13
EmilienMHunner: if you remember, I did the same kind of patch for rabbitmq16:13
EmilienMHunner: 0227f762070ffbbea3c28d6a60174de98fa4cc1c16:13
HunnerSo I think the nova review was mostly about docs; any module we could review that wouldn't have the same issue?16:13
EmilienMHunner: they probably all have the same issue16:14
EmilienMwe need more doc16:14
HunnerGood for anyone wanting ATC ;)16:14
*** darkxploit has joined #puppet-openstack16:15
EmilienMcrinkle: I have another blocker for ubuntu/liberty but it's packaging issue in openstackclient, I'm digging16:16
EmilienMcrinkle: after that, i have both centos7 & trusty working16:16
degorenkofolks, can anybody merge this https://review.openstack.org/#/c/206981/ ?16:22
pabelangerEmilienM: so, undecided if I should move the beaker-trusty node onto devstack-centos7 base.16:23
pabelangerdoing so, means we loose functional testing of launching beaker on ubuntu16:23
pabelangerhowever, not sure we really care about that16:23
pabelangercrinkle: thoughts? ^16:24
EmilienMpabelanger: no we won't loose that.16:24
EmilienMwe will keep trusty & centos716:24
EmilienMdropping trusty is even not an option16:24
* pabelanger confused16:24
pabelangerdropping trusty where?16:24
EmilienM<pabelanger> doing so, means we loose functional testing of launching beaker on ubuntu16:25
EmilienMwe don't want that16:25
EmilienMwe want to keep testing on ubuntu & centos16:25
pabelangerOkay, so you care which OS we launch beaker from.... that is what I am understanding16:26
pabelangeractually16:26
pabelangerthis is what I don't understand16:27
pabelangerwhen we do beaker testing, is beaker actually launching some image? Or does it use the local node to provision?16:27
EmilienMah16:27
EmilienMpabelanger: sorry I was confused16:28
EmilienMno I care about the nodepool instance that'll run Puppet16:28
EmilienMit has to be a trusty image16:28
crinklebeaker "launches" localhost16:28
EmilienMyeah that's why I think I correctly understood16:28
openstackgerritAndrey Brito proposed openstack/puppet-murano: Update Murano README  https://review.openstack.org/20763916:28
EmilienMso we don't need to change that now I guess16:28
crinkleso we need to have it on both trusty and centos716:28
*** openstackgerrit_ has quit IRC16:29
pabelangerokay. starting to understand16:29
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: Enable GPG for EPEL  https://review.openstack.org/20908116:33
*** tiswanso has joined #puppet-openstack16:34
EmilienMcrinkle: I talked to jamespage, and he found the issue in osclient/liberty - he's fixing it so maybe we can have liberty working asap16:37
*** sergmelikyan has joined #puppet-openstack16:45
_ody_sbadia: Is there any reason to keep around16:47
_ody_spec/acceptance/nodesets/nodepool.yml16:47
_ody_spec/acceptance/nodesets/default.yml too I suppose.16:48
crinklenodepool.yml should go away, default.yml should be changed to a vagrant nodeset16:48
_ody_crinkle: there are vagrant nodesets for ubuntu and centos already in msync.16:50
crinkle_ody_: in the puppetlabs modules we usually have the default be a copy of one of the others16:51
_ody_crinkle: If it is a standard then ok.16:52
*** _ody_ is now known as _ody16:52
openstackgerritTimothy Swanson proposed openstack/puppet-neutron: Update Cisco ML2 classes and templates  https://review.openstack.org/19718116:54
EmilienMHunner: I guess that will be 3.5.1 ?17:03
HunnerContext?17:04
EmilienMHunner: sorry, puppetlabs-mysql release17:07
EmilienMbecause it will block our beaker jobs until we pin another release17:07
EmilienMI guess we'll need to bump17:08
_odySlow CI today?17:08
HunnerEmilienM: I want to get https://tickets.puppetlabs.com/browse/MODULES-2321 in too, so probably 3.6.017:10
HunnerEmilienM: Also, please don't push PR branches to puppetlabs github; use a fork :)17:10
Hunneroh, that's postgres. nm17:12
HunnerEmilienM: Well, we already merged code to decouple the mysql_* resources from mysql::server, so that is a feature bump17:12
*** chem has quit IRC17:19
*** gfidente has quit IRC17:24
*** tiswanso has quit IRC17:29
*** tiswanso has joined #puppet-openstack17:30
openstackgerritRyan Hefner proposed openstack/puppet-tripleo: Enable Manila Service  https://review.openstack.org/20424917:30
EmilienMHunner: sorry for the branch, I won't do that next time17:32
HunnerEmilienM: No worries. I blame github UX ;)17:32
guimalufEmilienM, pabelanger it seems the 'recheck' comment doesn't work...17:33
pabelangerguimaluf: review?17:35
guimalufpabelanger, https://review.openstack.org/#/c/207161/17:35
pabelangerguimaluf: it is running on status.openstack.org/zuul17:36
pabelangerstill in the gate17:36
pabelangers/gate/check queue/17:37
*** dprince has quit IRC17:37
guimalufpabelanger, oh! I got it... I tought it would clear gate status like when a new patch set is push17:38
guimalufpabelanger, thanks! sorry for bothering you17:38
EmilienMHunner: https://github.com/puppetlabs/puppetlabs-mysql/pull/74217:39
pabelangerguimaluf: no problems17:39
*** xarses has quit IRC17:52
*** dprince has joined #puppet-openstack17:57
*** dprince has quit IRC17:57
*** tiswanso has quit IRC17:58
*** tiswanso has joined #puppet-openstack18:01
*** xingchao has quit IRC18:02
HunnerEmilienM: Thanks ^_^18:04
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: Fix rgw keystone to use correct URL/API string  https://review.openstack.org/15378318:06
EmilienMHunner: can we backport it?18:06
EmilienMHunner: to 3.5.x ?18:06
HunnerEmilienM: sadface18:06
EmilienMHunner: I'm just looking for a sane way to have this patch in our CI18:07
EmilienMwithout checkouting master18:07
HunnerHmm. I wonder if I can spin mysql 3.5.1 instead of 3.6.0...18:07
EmilienMcurrently, our CI is on 3.4.118:07
EmilienMbut we can go to 3.5.x18:07
HunnerEmilienM: oh is that what you want. I can push 3.6.x... but first, do you think https://github.com/puppetlabs/puppetlabs-mysql/compare/3.5.0...master is worth a feature release?18:08
EmilienMHunner: let me some time to see18:08
EmilienMHunner: no18:08
EmilienMI would say 'push 3.5.1'18:08
HunnerI *could* say that decoupling mysql_* from mysql::server is a bugfix18:09
EmilienMyeah but other than that18:09
EmilienMboth ways work for me18:10
EmilienMHunner: ^18:10
openstackgerritMerged openstack/puppet-ironic: Add bifrost manifest  https://review.openstack.org/20202518:11
HunnerEmilienM: Okay, I think it really is a feature, so 3.6.0. Anything else you want in before I push 3.6.x?18:14
EmilienMHunner: no18:14
EmilienMHunner: go ahead! boom18:14
Hunner3.6.x pushed and .0 should be released around 10th-12th18:18
*** dgurtner has quit IRC18:19
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Puppetfile: bump MySQL module to 3.6.x  https://review.openstack.org/20920918:21
EmilienMcrinkle, Hunner ^18:21
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: acceptance: bump to Liberty release  https://review.openstack.org/20909718:21
openstackgerritCody Herriges proposed openstack/puppet-designate: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18986418:28
openstackgerritCody Herriges proposed openstack/puppet-gnocchi: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18986618:30
openstackgerritCody Herriges proposed openstack/puppet-sahara: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987718:32
openstackgerritAlex Schultz proposed openstack/puppet-horizon: Make redirect type configurable  https://review.openstack.org/20846218:33
openstackgerritCody Herriges proposed openstack/puppet-trove: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18988118:34
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP: Introduce swift_service class  https://review.openstack.org/20322018:34
openstackgerritCody Herriges proposed openstack/puppet-cinder: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18986318:36
openstackgerritCody Herriges proposed openstack/puppet-glance: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18986518:37
openstackgerritCody Herriges proposed openstack/puppet-ironic: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987018:38
openstackgerritCody Herriges proposed openstack/puppet-manila: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987218:40
openstackgerritCody Herriges proposed openstack/puppet-nova: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987418:41
openstackgerritCody Herriges proposed openstack/puppet-openstacklib: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987518:42
EmilienMcrinkle: it seems like my attempt to use Depends-On does not work on https://review.openstack.org/209097 - any idea,18:43
openstackgerritCody Herriges proposed openstack/puppet-heat: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18986818:43
openstackgerritCody Herriges proposed openstack/puppet-tuskar: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18988218:45
openstackgerritAlex Schultz proposed openstack/puppet-horizon: Make redirect type configurable  https://review.openstack.org/20846218:46
crinkleEmilienM: looks like it's working to me?18:47
EmilienMcrinkle: it's pulling 3.5.x18:47
EmilienMbut I wanted 3.6.x18:47
EmilienMI mean, the Depends-On works18:47
crinkleEmilienM: the 3.6.x branch still has 3.5.0 as the metadata versionb https://github.com/puppetlabs/puppetlabs-mysql/blob/3.6.x/metadata.json18:47
EmilienMbut it's not pulling the right version of MySQL module18:48
EmilienMah18:48
EmilienMgood catch18:48
EmilienMcrinkle: sorry for noise18:48
crinkle:)18:48
openstackgerritCody Herriges proposed openstack/puppet-horizon: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18986918:48
openstackgerritCody Herriges proposed openstack/puppet-keystone: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987118:50
EmilienMcrinkle: we might need to review patches in puppet openstack integration because no patch will be merged until basic stricture is in place18:50
EmilienMlike https://review.openstack.org/20920918:51
openstackgerritAlex Schultz proposed openstack/puppet-horizon: Make redirect type configurable  https://review.openstack.org/20846218:51
openstackgerritCody Herriges proposed openstack/puppet-vswitch: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18988318:52
openstackgerritCody Herriges proposed openstack/puppet-swift: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987818:53
*** xarses has joined #puppet-openstack18:53
*** xarses has quit IRC18:53
openstackgerritCody Herriges proposed openstack/puppet-tripleo: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18988018:53
*** xarses has joined #puppet-openstack18:54
openstackgerritClayton O'Neill proposed openstack/puppet-cinder: Add lock_path to cinder config  https://review.openstack.org/20921618:55
openstackgerritCody Herriges proposed openstack/puppet-openstack_extras: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987618:58
openstackgerritClayton O'Neill proposed openstack/puppet-cinder: Add lock_path to cinder config  https://review.openstack.org/20921618:58
*** paramite has joined #puppet-openstack19:00
*** jasondotstar has joined #puppet-openstack19:01
EmilienMiurygregory: reviewed19:03
iurygregorythanks EmilienM, after my coffee i'll take a look ^^19:04
EmilienMiurygregory: take 2 coffees19:05
openstackgerritCody Herriges proposed openstack/puppet-openstack_extras: Fixing lint errors  https://review.openstack.org/20921919:05
*** angdraug has quit IRC19:05
*** xingchao has joined #puppet-openstack19:06
openstackgerritCody Herriges proposed openstack/puppet-openstack_extras: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987619:08
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP: Introduce swift_service class  https://review.openstack.org/20322019:09
openstackgerritCody Herriges proposed openstack/puppet-tempest: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987919:11
*** dprince has joined #puppet-openstack19:14
vinshIs their something stopping the puppet-lint jobs from running today?19:18
mgagneEmilienM: now yes: bug #148146019:21
openstackbug 1481460 in puppet-glance "Cannot configure multiple Glance stores" [Undecided,New] https://launchpad.net/bugs/148146019:21
EmilienMmgagne: this one is high or critical I guess19:22
mgagneEmilienM: I guess so. it looks like glance changed behavior. I used to be able to upload from an http source even when only the swift store was configured. Now I can't upload without http store configured19:22
*** dgurtner has joined #puppet-openstack19:29
*** angdraug has joined #puppet-openstack19:30
EmilienMcan someone review https://review.openstack.org/192176 and https://review.openstack.org/190016 please ?19:32
openstackgerritCody Herriges proposed openstack/puppet-tempest: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987919:33
crinkleEmilienM: why does the upgrade test fail on 190016? it looks potentially related19:35
*** tiswanso has quit IRC19:36
EmilienMcrinkle: might be related to https://review.openstack.org/#/c/208019/19:38
EmilienMor not19:38
*** tiswanso has joined #puppet-openstack19:39
EmilienMcrinkle: upgrade jobs are broken now19:39
EmilienMbut not the same issue19:39
*** jasondotstar is now known as jasondotstar|afk19:39
EmilienMcrinkle: so there  is some in my patch that breaks upgrade?19:40
EmilienMI'm rebasing in case of19:40
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: Validate service_identity resources  https://review.openstack.org/19001619:40
crinkleEmilienM: i'm not looking that closely but the change is about the service_identity resource and the errors are about keystone resources so it seems related19:41
EmilienMcrinkle: my patch just show warnings or errors19:41
EmilienMcrinkle: it's a sanity patch19:41
iurygregoryHey EmilienM, you think is more valid have one class for service provider instead of three right?20:00
EmilienMyeah20:00
EmilienMbecause they share parameters, right?20:00
*** Fin1te has joined #puppet-openstack20:02
*** paramite has quit IRC20:03
iurygregorymellon and shibboleth yes, but openid connector don't use some parameters and have others20:04
openstackgerritDan Prince proposed openstack/puppet-tripleo: Add pre_create_files resource and helper class  https://review.openstack.org/20877420:05
openstackgerritDan Prince proposed openstack/puppet-tripleo: tripleo::noop: add support for more resources  https://review.openstack.org/20903020:05
*** dprince has quit IRC20:07
iurygregoryabout the functional testing with acceptance,  i need describe the test in the spec or just say that functional test will be written?20:19
*** xingchao has quit IRC20:26
*** sergmelikyan has quit IRC20:30
*** sergmelikyan has joined #puppet-openstack20:37
*** arnaud_orange1 has quit IRC20:45
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: Fix a race condition in mon creation  https://review.openstack.org/20925020:48
*** sergmelikyan has quit IRC20:48
*** sergmelikyan has joined #puppet-openstack20:51
*** angdraug has quit IRC21:02
*** rcallawa_ has joined #puppet-openstack21:07
*** jasondotstar|afk is now known as jasondotstar21:07
*** rcallawa has quit IRC21:11
*** rcallawa_ has quit IRC21:11
*** sergmelikyan has quit IRC21:15
openstackgerritVasyl Saienko proposed openstack/puppet-horizon: allow to pass more params to 'horizon::wsgi::apache'  https://review.openstack.org/20925621:17
*** Fin1te has quit IRC21:20
*** dmsimard has quit IRC21:21
*** dmsimard has joined #puppet-openstack21:40
*** jayg is now known as jayg|g0n321:43
*** dachary1 has quit IRC21:57
*** dachary1 has joined #puppet-openstack21:57
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: Fix rgw keystone  https://review.openstack.org/15378322:00
*** dachary1 has quit IRC22:14
*** bklei_ has quit IRC22:29
openstackgerritDan Prince proposed openstack/puppet-tripleo: Add pre_create_files resource and helper class  https://review.openstack.org/20877422:35
openstackgerritDan Prince proposed openstack/puppet-tripleo: tripleo::noop: add support for more resources  https://review.openstack.org/20903022:35
*** jasondotstar has quit IRC22:36
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: Fix rgw keystone  https://review.openstack.org/15378322:42
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: Fix rgw keystone  https://review.openstack.org/15378322:43
*** rcallawa_ has joined #puppet-openstack23:00
*** rcallawa_ has joined #puppet-openstack23:01
*** morazi has quit IRC23:06
*** gildub has joined #puppet-openstack23:07
*** dfisher has quit IRC23:07
EmilienMcrinkle: we would need https://review.openstack.org/#/c/203742/ merged so we can have https://review.openstack.org/#/c/209209/ passing and eventually have Liberty support https://review.openstack.org/#/c/209097/23:12
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: Fix rgw keystone  https://review.openstack.org/15378323:14
*** tiswanso has joined #puppet-openstack23:19
*** jasondotstar has joined #puppet-openstack23:20
openstackgerritMerged stackforge/puppet-ceph: Enable GPG for EPEL  https://review.openstack.org/20908123:25
*** dgurtner has quit IRC23:26
*** cdelatte has quit IRC23:28
*** tiswanso has quit IRC23:30
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Initial import to gate on  https://review.openstack.org/20374223:37
EmilienMcrinkle: ^23:38
EmilienMcrinkle: is the commit message good?23:39
crinklepabelanger: ^ can you clarify what the difference we between patchset 5 and 6 and if the weird rakefile was needed?23:39
crinkleEmilienM: lgtm23:39
EmilienMcrinkle: thx23:40
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Puppetfile: bump MySQL module to 3.6.x  https://review.openstack.org/20920923:41
EmilienMcrinkle: I rebased https://review.openstack.org/#/c/190016/ and upgrade fails for the same reason as other patches so not related.23:43
crinkleEmilienM: ok23:44
EmilienMcrinkle: I'll work on upgrade jobs when integration will be in place23:44
EmilienMcrinkle: when we have logs + integration in place, I focus on upgrade23:44
EmilienMbut I can't work // on that, it's too dependent I think23:44
EmilienMcrinkle: my last request for today: https://review.openstack.org/#/c/192176/ also23:44
EmilienMthanks23:44
*** delattec has joined #puppet-openstack23:51
« Monday, 2015-08-03 Index Wednesday, 2015-08-05 »

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