Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-08-04

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

All times shown according to UTC.

Time Nick Message
00:30 bklei joined #puppet-openstack
00:55 tiswanso joined #puppet-openstack
00:58 tiswanso joined #puppet-openstack
01:04 xingchao joined #puppet-openstack
01:20 gildub joined #puppet-openstack
01:20 gildub joined #puppet-openstack
01:36 xarses joined #puppet-openstack
01:41 rcallawa joined #puppet-openstack
01:46 rcallawa joined #puppet-openstack
01:51 rcallawa_ joined #puppet-openstack
02:16 xingchao joined #puppet-openstack
02:27 openstackgerrit Dan Prince proposed openstack/puppet-tripleo: Add pre_create_files resource and helper class  https://review.openstack.org/208774
02:41 mesamoo joined #puppet-openstack
03:33 rcallawa joined #puppet-openstack
04:06 Madkiss joined #puppet-openstack
04:06 Madkiss joined #puppet-openstack
04:48 linkedinyou joined #puppet-openstack
05:08 Madkiss joined #puppet-openstack
05:08 Madkiss joined #puppet-openstack
05:27 saneax joined #puppet-openstack
05:37 bogdando joined #puppet-openstack
05:41 xingchao joined #puppet-openstack
06:06 jtomasek joined #puppet-openstack
06:54 xingchao joined #puppet-openstack
06:57 dgurtner joined #puppet-openstack
07:01 dgurtner joined #puppet-openstack
07:08 degorenko joined #puppet-openstack
07:44 openstackgerrit Ricardo Carrillo Cruz proposed openstack/puppet-ironic: Add bifrost manifest  https://review.openstack.org/202025
07:51 gildub joined #puppet-openstack
08:02 openstackgerrit Ricardo Carrillo Cruz proposed openstack/puppet-ironic: Add bifrost manifest  https://review.openstack.org/202025
08:08 jistr joined #puppet-openstack
08:08 paramite joined #puppet-openstack
08:13 chem joined #puppet-openstack
08:23 openstackgerrit Martin Mágr proposed openstack/puppet-nova: Allow to change archive destination  https://review.openstack.org/207915
08:37 dachary1 joined #puppet-openstack
08:37 dgurtner joined #puppet-openstack
08:37 bogdando joined #puppet-openstack
08:37 Madkiss joined #puppet-openstack
08:37 hogepodge joined #puppet-openstack
08:37 dachary joined #puppet-openstack
08:37 ianw joined #puppet-openstack
08:37 colby_ joined #puppet-openstack
08:37 tdb joined #puppet-openstack
08:37 zino_ joined #puppet-openstack
08:37 mattymo joined #puppet-openstack
08:37 crinkle joined #puppet-openstack
08:37 ric` joined #puppet-openstack
08:37 joshsamuelson joined #puppet-openstack
08:45 dachary1 joined #puppet-openstack
08:48 openstackgerrit Merged openstack/puppet-cinder: Update default notification_driver for ceilometer  https://review.openstack.org/204808
08:49 openstackgerrit Merged openstack/puppet-trove: Introduce trove:config to manage custom options  https://review.openstack.org/207823
08:50 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-ceilometer: Add ability to specify ttl and timeout parameters  https://review.openstack.org/207898
08:58 gfidente joined #puppet-openstack
09:33 openstackgerrit Martin Mágr proposed openstack/puppet-cinder: DB purge cron job  https://review.openstack.org/208939
09:53 openstackgerrit_ joined #puppet-openstack
09:59 sac joined #puppet-openstack
10:07 openstackgerrit Limor Stotland proposed openstack/puppet-mistral: puppet-mistral: initial commit  https://review.openstack.org/208457
10:08 saneax joined #puppet-openstack
10:13 openstackgerrit Limor Stotland proposed openstack/puppet-mistral: puppet-mistral: initial commit  https://review.openstack.org/208457
10:23 gildub joined #puppet-openstack
10:41 xingchao joined #puppet-openstack
10:59 openstackgerrit Limor Stotland proposed openstack/puppet-mistral: puppet-mistral: initial commit  https://review.openstack.org/208457
11:01 xingchao joined #puppet-openstack
11:08 degorenko joined #puppet-openstack
11:08 openstackgerrit Martin Mágr proposed openstack/puppet-cinder: DB purge cron job  https://review.openstack.org/208939
11:19 EmilienM good morning
11:21 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: acceptance: test OVS bridge listing  https://review.openstack.org/192176
11:26 rcarrillocruz joined #puppet-openstack
11:28 rcarrillocruz EmilienM , crinkle: hiya, i amended https://review.openstack.org/#/c/202025/
11:28 EmilienM rcarrillocruz: ok !
11:32 openstackgerrit_ joined #puppet-openstack
11:33 openstackgerrit_ joined #puppet-openstack
11:39 dprince joined #puppet-openstack
12:07 EmilienM mgagne: have you filed a bug for your issue with glance stores?
12:17 openstackgerrit joined #puppet-openstack
12:18 rcallawa joined #puppet-openstack
12:19 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/202689
12:24 sbadia hi!
12:24 sbadia _ody_: here?
12:27 openstackgerrit Dan Prince proposed openstack/puppet-tripleo: Add pre_create_files resource and helper class  https://review.openstack.org/208774
12:27 openstackgerrit Dan Prince proposed openstack/puppet-tripleo: tripleo::noop: add support for more resources  https://review.openstack.org/209030
12:29 EmilienM sbadia: early for him I guess :)
12:30 sbadia yep :/
12:31 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-nova: Add nameservers (dns) parameters  https://review.openstack.org/202788
12:33 openstackgerrit Merged openstack/puppet-glance: Add swift_store_region parameter to glance::backend::swift  https://review.openstack.org/208654
12:35 openstackgerrit Andrey Brito proposed openstack/puppet-murano: Update Murano README  https://review.openstack.org/207639
12:36 openstackgerrit Merged openstack/puppet-nova: Support allow_start and allow_end parameters  https://review.openstack.org/206099
12:37 openstackgerrit Merged openstack/puppet-neutron: N1Kv: Adding nexus1000v puppet class  https://review.openstack.org/204790
12:42 openstackgerrit Merged openstack/puppet-designate: acceptance: Remove dirty fix for empty values (fixed in inifile module)  https://review.openstack.org/195003
12:43 degorenko EmilienM, hey :) Can you review my patch? https://review.openstack.org/205987 :-) Also, is ready to merge next: https://review.openstack.org/206981
12:50 openstackgerrit Andrey Brito proposed openstack/puppet-murano: Update Murano README  https://review.openstack.org/207639
12:50 openstackgerrit Merged openstack/puppet-keystone: WSGI: use real service name in restart_keystone Exec  https://review.openstack.org/208019
12:51 openstackgerrit Merged openstack/puppet-keystone: admin_user_domain and admin_project_domain can now be equal  https://review.openstack.org/207437
12:51 openstackgerrit Merged openstack/puppet-nova: Add ability to set default baremetal filters  https://review.openstack.org/202215
12:51 openstackgerrit Merged openstack/puppet-trove: Fix catalog compilation when not configuring endpoint  https://review.openstack.org/207435
12:51 crinkle rcarrillocruz: 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 present
12:51 crinkle rcarrillocruz: do you have a response to my other non-inline comment? i'm more concerned about that
12:52 openstackgerrit Yanis Guenane proposed openstack/puppet-sahara: Creation of sahara::db::sync  https://review.openstack.org/196683
12:58 rcallawa_ joined #puppet-openstack
12:59 openstackgerrit Lukas Bezdicka proposed openstack/puppet-keystone: beaker: test RDO/liberty  https://review.openstack.org/209044
13:07 rcarrillocruz crinkle: 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 rcarrillocruz so having as before as 'ensure=>latest' forced to have it to latest commit from the branch specified
13:07 rcarrillocruz makes sense to have it as default 'present' and leave the operator the change to put it as absent tho
13:07 rcarrillocruz i will push in a bit
13:07 crinkle rcarrillocruz: cool
13:08 crinkle rcarrillocruz: I'm find with keeping ensure => present hardcoded for now, too
13:09 skolekonov joined #puppet-openstack
13:09 crinkle rcarrillocruz: as it is, if it's set to ensure => latest then it might cause merge conflicts with the baremetal.json and the group_vars_all
13:10 morazi joined #puppet-openstack
13:10 dfisher joined #puppet-openstack
13:11 dachary1 joined #puppet-openstack
13:11 rcarrillocruz but i can't restrict a given resource to just present/absent, can I?
13:11 rcarrillocruz unless i do some dirty if comparison within the manifest
13:12 crinkle yeah that isn't ideal
13:12 crinkle rcarrillocruz: is there some way this could be done without modifying the repo?
13:13 rcarrillocruz you mean referencing the conf files in a folder that's outside of the cloned repo, right
13:13 rcarrillocruz ?
13:13 crinkle rcarrillocruz: right
13:13 rcarrillocruz baremetal.json can be done
13:13 rcarrillocruz however
13:13 rcarrillocruz the way ansible works
13:13 rcarrillocruz it just expects a certain layout
13:13 rcarrillocruz and in this case, group_vars/all
13:14 rcarrillocruz it's expected at the top level
13:14 rcarrillocruz not sure if that can be tweaked with ansibe.cfg or some flag
13:14 EmilienM crinkle, sbadia, mfisch, clayton, spredzy: do you have something to add in our agenda (pretty empty now) ?
13:14 openstackgerrit Martin Mágr proposed openstack/puppet-cinder: DB purge cron job  https://review.openstack.org/208939
13:14 crinkle rcarrillocruz: hrm okay
13:15 crinkle rcarrillocruz: I can +2 for now and we can iterate on it maybe
13:15 clayton nothing here, and mfisch is on vacation until next week
13:15 crinkle EmilienM: nothing for me
13:15 rcarrillocruz that's one of the issues for mixing conf + code, no hiera goodness here :/
13:15 tiswanso joined #puppet-openstack
13:15 spredzy EmilienM, not this week
13:16 dgurtner dmsimard: xarses dachary1 ping re: https://review.openstack.org/#/c/117422/
13:16 rcarrillocruz crinkle: k, let me push another change for the ensure param, then we land
13:16 rcarrillocruz i'll think something and iterate later
13:16 rcarrillocruz thx
13:17 dgurtner dmsimard: xarses dachary1 consequently we should also get https://review.openstack.org/#/c/153783/ merged otherwise it's actually broken
13:18 EmilienM pabelanger, crinkle: having our jobs (syntax + unit) on centos7 is much faster
13:19 crinkle nice
13:19 EmilienM pabelanger, crinkle: maybe this feedback is a bit early
13:19 EmilienM pabelanger, crinkle: but I noticed that this morning
13:19 EmilienM the jobs start immediately or almost
13:19 sbadia nothing here too :/
13:19 openstackgerrit Matthew Mosesohn proposed openstack/puppet-keystone: Update get_endpoint to use credentials auth_url  https://review.openstack.org/207890
13:19 EmilienM crinkle: we should run the trusty jobs on centos7, using a trusty container :P
13:20 xingchao joined #puppet-openstack
13:23 dachary2 joined #puppet-openstack
13:23 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Add basic manifests  https://review.openstack.org/195493
13:27 dachary1 joined #puppet-openstack
13:30 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Sanitize array parameters  https://review.openstack.org/209063
13:30 social EmilienM: https://jenkins02.openstack.org/jo​b/gate-puppet-keystone-puppet-beak​er-rspec-dsvm-centos7/144/console
13:31 EmilienM woot
13:31 EmilienM social: how many beers I owe you?
13:32 social 0
13:32 pabelanger EmilienM: Yup, that's was the goal. devstack-trusty gets hammered with devstack-gates
13:32 dfisher i'll take them.
13:32 social EmilienM: simple quotes
13:32 pabelanger going to checkout other jobs today
13:32 EmilienM pabelanger: cool, I suggest we wait a bit for the rest (lint?) now
13:32 social EmilienM: false without quotes just made it to ignore the gpg option
13:32 EmilienM social: ok... good
13:32 EmilienM social: that means we can go ahead maybe
13:33 EmilienM social: thanks *a lot*
13:33 openstackgerrit Yanis Guenane proposed openstack/puppet-heat: Creation of heat::db::sync  https://review.openstack.org/196646
13:33 EmilienM dfisher: no way
13:33 pabelanger EmilienM: Ya, lint going to take some looking into since it is more complicated.  But plan to spend some time on it today
13:33 social yeah, I do want to move ahead with liberty
13:33 EmilienM pabelanger: can you attend our weekly meeting today?
13:34 EmilienM pabelanger: we make a CI status
13:34 pabelanger EmilienM: yes, in calendar
13:34 openstackgerrit Drew Fisher proposed openstack/puppet-neutron: Remove POSIX users, groups, and file modes.  https://review.openstack.org/209068
13:35 EmilienM pabelanger: I was thinking, we should install modules in /etc/puppet/modules
13:35 EmilienM first of all, because this is the place to be, and also because my work on logs will list modules from there
13:35 EmilienM https://review.openstack.org/206693
13:37 pabelanger EmilienM: What benifit does installing them into /etc/puppet give us?  Over using the --modulepath to local directory?
13:37 pabelanger looking at your patchset
13:37 EmilienM pabelanger: because this is where beaker install modules now
13:37 EmilienM and also because this is where other infra modules are installed
13:37 EmilienM I think it makes just sense to keep them in the right place
13:38 EmilienM pabelanger: yeah, look at the line where I list OpenStack project using metadata.json
13:38 pabelanger EmilienM: cannot speak for beaker, but we are actually going to move them from /etc/puppet/modules to some other place in -infra.
13:38 social EmilienM: 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 lot
13:38 EmilienM social: never seen that before
13:38 EmilienM pabelanger: why? where?
13:39 pabelanger EmilienM: see clark comments: https://review.openstack.org/#/c/208088/
13:40 pabelanger EmilienM: 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 pabelanger however, we do need sudo to run puppet, so there is that
13:40 EmilienM pabelanger: this one is in my review list for today
13:43 social EmilienM: btw I'd love to have this brought up https://review.openstack.org/#/c/201229/
13:43 EmilienM pabelanger: his comment makes sense though
13:43 EmilienM social: adding it in my review list
13:44 pabelanger another 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 (when
13:44 pabelanger possible)
13:45 pabelanger I'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 gate
13:45 crinkle which patch are we talking about? why are we installing beaker with sudo?
13:45 ferest joined #puppet-openstack
13:45 pabelanger crinkle: I don't know why, I can only guess to put beaker in the $PATH variable
13:46 pabelanger for our integration job, I've changed it to use GEM_HOME for beaker install
13:47 pabelanger Is is wrong, to me: https://github.com/openstack-infra​/project-config/blob/master/jenkin​s/jobs/puppet-module-jobs.yaml#L9
13:47 pabelanger this is correct: https://github.com/openstack-infra/​project-config/blob/master/jenkins/​jobs/puppet-module-jobs.yaml#L201
13:48 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-ceilometer: Add ability to specify ttl and timeout parameters  https://review.openstack.org/207898
13:48 pabelanger Need some coffee, and review emails from holiday
13:48 tiswanso joined #puppet-openstack
13:48 openstackgerrit Igor Gajsin proposed openstack/puppet-cinder: set parameter host in vmdk.pp  https://review.openstack.org/157004
13:50 crinkle oh installing bundler, not beaker
13:50 pabelanger don't ya, my bad (lack of coffee)
13:50 pabelanger and I keep swapping dem names in my head
13:51 dachary1 joined #puppet-openstack
13:52 dachary1 joined #puppet-openstack
13:54 EmilienM spredzy: re - ::db classes - I think it's cool
13:55 EmilienM it's consistent with other modules
13:55 bklei joined #puppet-openstack
13:56 bklei_ joined #puppet-openstack
13:56 spredzy EmilienM, cool. I'll send an email later either today or tomorrow to explain the intent.
13:56 EmilienM spredzy, crinkle: I was wondering your feelings on https://review.openstack.org/#/c/201229/
13:56 mdorman joined #puppet-openstack
13:57 EmilienM for me, it's really specific to be in our puppet modules
13:57 crinkle I have no idea what best practice is there
13:57 crinkle I haven't seen any of our other modules do that
13:57 EmilienM me neither
13:58 EmilienM crinkle: to me, it depends on which package release you use, etc
13:59 EmilienM also, openstack-selinux should take care of it
13:59 EmilienM at least it is partially taking care of this atm
13:59 pabelanger EmilienM: I would agree. If there is a selinux issue, seems selinux should be fixed outside the puppet modules. Or inside vendor puppet modules
13:59 EmilienM pabelanger: social is working on packstack and his use case is fair
13:59 dachary1 joined #puppet-openstack
14:01 spredzy EmilienM, 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 them
14:01 EmilienM w00t
14:01 EmilienM that's also a great idea I guess
14:01 spredzy If we start having selinux bool enablement within the code, it might clutter it quite easily
14:01 EmilienM and add the interface in puppet-horizon
14:03 spredzy httpd::enabled_selinux: ['httpd_can_network_connect_db', 'httpd_use_openstack']
14:03 EmilienM spredzy: you can give your proposal in gerrit
14:08 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Enable https and GPG check for EPEL  https://review.openstack.org/209081
14:18 ferest joined #puppet-openstack
14:20 dachary1 joined #puppet-openstack
14:31 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: acceptance: bump to Liberty release  https://review.openstack.org/209097
14:33 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: acceptance: bump to Liberty release  https://review.openstack.org/209097
14:33 openstackgerrit Kuo-Le.Mei proposed openstack/puppet-nova: Fixed value `rbd.keyring` not update  https://review.openstack.org/203909
14:34 xingchao joined #puppet-openstack
14:40 EmilienM our weekly meeting will start in 20min on #openstack-meeting-4 - our agenda is here: https://etherpad.openstack.org/p/pupp​et-openstack-weekly-meeting-20150804
14:41 spredzy Rspec 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-opens​tack-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 spredzy Would anyone have an idea on how I can do/solve that ?
14:42 spredzy so far all the test file is : http://pastebin.com/iVMX6C8k
14:45 EmilienM in spec_helper.rb:$LOAD_PATH.push(F​ile.join(File.dirname(__FILE__), 'fixtures', 'modules', 'openstacklib', 'lib'))
14:45 EmilienM spredzy: ^
14:45 EmilienM maybe you are missing that
14:45 EmilienM spredzy: look in puppet-keystone how it does with openstack provider
14:46 EmilienM it's importing it from openstacklib
14:46 spredzy EmilienM, ack checking
14:47 openstackgerrit Denis Egorenko proposed openstack/puppet-sahara: Update parameters for Sahara  https://review.openstack.org/205987
14:50 vsaienko joined #puppet-openstack
14:51 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: acceptance: bump to Liberty release  https://review.openstack.org/209097
14:53 EmilienM pabelanger: how did you make it work on trusty ????? https://review.openstack.org/#/c/207078/
14:53 EmilienM pabelanger: it works now \o/
14:55 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Add basic manifests  https://review.openstack.org/195493
14:56 angdraug joined #puppet-openstack
14:57 social joined #puppet-openstack
14:59 mdorman fyi i will be late to the meeting
14:59 sbadia mdorman: ack, np
15:01 tiswanso joined #puppet-openstack
15:03 guimaluf which channel is the meeting?
15:03 guimaluf found it
15:05 pabelanger EmilienM: I didn't. There is a race / timeout condition on apt-get I think
15:11 openstackgerrit Colleen Murphy proposed openstack/puppet-openstack_extras: Sync Gemfile  https://review.openstack.org/209116
15:11 openstackgerrit Colleen Murphy proposed openstack/puppet-openstack_extras: Sync Gemfile  https://review.openstack.org/209116
15:11 openstackgerrit Colleen Murphy proposed openstack/puppet-openstack_extras: Add acceptance tests for repo::* classes  https://review.openstack.org/207180
15:11 openstackgerrit Colleen Murphy proposed openstack/puppet-openstack_extras: Add auth_file acceptance test  https://review.openstack.org/207179
15:13 openstackgerrit Ricardo Carrillo Cruz proposed openstack/puppet-ironic: Add bifrost manifest  https://review.openstack.org/202025
15:15 sbadia sorry for the spam :)
15:15 openstackgerrit Sebastien Badia proposed openstack/puppet-ceilometer: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/209121
15:15 openstackgerrit Sebastien Badia proposed openstack/puppet-designate: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189864
15:15 openstackgerrit Colleen Murphy proposed openstack/puppet-openstack_extras: Sync Gemfile  https://review.openstack.org/209116
15:15 openstackgerrit Colleen Murphy proposed openstack/puppet-openstack_extras: Add acceptance tests for repo::* classes  https://review.openstack.org/207180
15:15 openstackgerrit Colleen Murphy proposed openstack/puppet-openstack_extras: Add auth_file acceptance test  https://review.openstack.org/207179
15:15 openstackgerrit Sebastien Badia proposed openstack/puppet-gnocchi: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189866
15:15 openstackgerrit Sebastien Badia proposed openstack/puppet-horizon: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189869
15:15 openstackgerrit Sebastien Badia proposed openstack/puppet-keystone: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189871
15:15 openstackgerrit Sebastien Badia proposed openstack/puppet-neutron: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/209122
15:15 openstackgerrit Sebastien Badia proposed openstack/puppet-openstack_extras: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189876
15:16 openstackgerrit Sebastien Badia proposed openstack/puppet-sahara: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189877
15:16 openstackgerrit Sebastien Badia proposed openstack/puppet-tempest: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189879
15:16 openstackgerrit Sebastien Badia proposed openstack/puppet-trove: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189881
15:16 openstackgerrit Sebastien Badia proposed openstack/puppet-vswitch: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189883
15:16 openstackgerrit Sebastien Badia proposed openstack/puppet-cinder: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189863
15:16 openstackgerrit Sebastien Badia proposed openstack/puppet-glance: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189865
15:16 openstackgerrit Sebastien Badia proposed openstack/puppet-heat: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189868
15:16 openstackgerrit Sebastien Badia proposed openstack/puppet-ironic: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189870
15:16 openstackgerrit Sebastien Badia proposed openstack/puppet-manila: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189872
15:16 openstackgerrit Sebastien Badia proposed openstack/puppet-nova: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189874
15:16 openstackgerrit Sebastien Badia proposed openstack/puppet-openstacklib: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189875
15:16 openstackgerrit Sebastien Badia proposed openstack/puppet-swift: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189878
15:16 openstackgerrit Sebastien Badia proposed openstack/puppet-tripleo: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189880
15:16 openstackgerrit Sebastien Badia proposed openstack/puppet-tuskar: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189882
15:17 paramite oh boy
15:17 sbadia msync run: https://review.openstack.org/#/q/status:op​en+branch:master+topic:msync_rspec_version,n,z
15:18 iurygregory omg o.o
15:21 rcallawa joined #puppet-openstack
15:23 IBerezovskiy joined #puppet-openstack
15:27 openstackgerrit Sebastien Badia proposed openstack/puppet-modulesync-configs: spec: Fix nodesets errors (x64 for el and amd64 for debian)  https://review.openstack.org/209127
15:27 openstackgerrit Sebastien Badia proposed openstack/puppet-modulesync-configs: sample config (update namespace)  https://review.openstack.org/209128
15:27 openstackgerrit Martin Mágr proposed openstack/puppet-keystone: Allow to change archive destination  https://review.openstack.org/209129
15:29 cdelatte joined #puppet-openstack
15:31 rcarrillocruz crinkle: thx for the +2 , just saw
15:34 EmilienM rcarrillocruz: will approve when CI is green
15:34 rcarrillocruz \o/
15:34 rcarrillocruz thx EmilienM
15:35 EmilienM crinkle: are you going to patch project-config to run acceptance on _extras ?
15:36 crinkle EmilienM: ah i think i was and then forgot to submit it
15:37 IBerezovskiy joined #puppet-openstack
15:40 dachary1 joined #puppet-openstack
15:40 guimaluf pabelanger, 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:41 EmilienM guimaluf: go in your review, and drop a comment 'recheck'
15:41 guimaluf great! :)
15:41 EmilienM guimaluf: it will re-trigger zuul and Jenkins jobs
15:42 guimaluf very nice! thanks :)
15:42 EmilienM crinkle: we might some reviews on the integration serie of patches, starting from https://review.openstack.org/#/c/203742/
15:45 pabelanger yup 'recheck'
15:47 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: acceptance: bump to Liberty release  https://review.openstack.org/209097
15:53 openstackgerrit Merged openstack/puppet-glance: Update README.md for Glance  https://review.openstack.org/207805
15:56 tiswanso joined #puppet-openstack
16:07 EmilienM Hunner, crinkle: I found a blocker for having OpenStack Liberty working on centos7: https://github.com/puppetlab​s/puppetlabs-mysql/pull/741
16:12 Hunner EmilienM: for approved modules, do you know if the nova feedback went in? Should we check it again? Also, which one should be next?
16:13 EmilienM Hunner: no, I did not spend time on it yet
16:13 Hunner I'll look at the mysql PR; mysql has a release next week
16:13 EmilienM oh nice
16:13 EmilienM Hunner: if you remember, I did the same kind of patch for rabbitmq
16:13 EmilienM Hunner: 0227f762070ffbbea3c28d6a60174de98fa4cc1c
16:13 Hunner So I think the nova review was mostly about docs; any module we could review that wouldn't have the same issue?
16:14 EmilienM Hunner: they probably all have the same issue
16:14 EmilienM we need more doc
16:14 Hunner Good for anyone wanting ATC ;)
16:15 darkxploit joined #puppet-openstack
16:16 EmilienM crinkle: I have another blocker for ubuntu/liberty but it's packaging issue in openstackclient, I'm digging
16:16 EmilienM crinkle: after that, i have both centos7 & trusty working
16:22 degorenko folks, can anybody merge this https://review.openstack.org/#/c/206981/ ?
16:23 pabelanger EmilienM: so, undecided if I should move the beaker-trusty node onto devstack-centos7 base.
16:23 pabelanger doing so, means we loose functional testing of launching beaker on ubuntu
16:23 pabelanger however, not sure we really care about that
16:24 pabelanger crinkle: thoughts? ^
16:24 EmilienM pabelanger: no we won't loose that.
16:24 EmilienM we will keep trusty & centos7
16:24 EmilienM dropping trusty is even not an option
16:24 * pabelanger confused
16:24 pabelanger dropping trusty where?
16:25 EmilienM <pabelanger> doing so, means we loose functional testing of launching beaker on ubuntu
16:25 EmilienM we don't want that
16:25 EmilienM we want to keep testing on ubuntu & centos
16:26 pabelanger Okay, so you care which OS we launch beaker from.... that is what I am understanding
16:26 pabelanger actually
16:27 pabelanger this is what I don't understand
16:27 pabelanger when we do beaker testing, is beaker actually launching some image? Or does it use the local node to provision?
16:27 EmilienM ah
16:28 EmilienM pabelanger: sorry I was confused
16:28 EmilienM no I care about the nodepool instance that'll run Puppet
16:28 EmilienM it has to be a trusty image
16:28 crinkle beaker "launches" localhost
16:28 EmilienM yeah that's why I think I correctly understood
16:28 openstackgerrit Andrey Brito proposed openstack/puppet-murano: Update Murano README  https://review.openstack.org/207639
16:28 EmilienM so we don't need to change that now I guess
16:28 crinkle so we need to have it on both trusty and centos7
16:29 pabelanger okay. starting to understand
16:33 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Enable GPG for EPEL  https://review.openstack.org/209081
16:34 tiswanso joined #puppet-openstack
16:37 EmilienM crinkle: I talked to jamespage, and he found the issue in osclient/liberty - he's fixing it so maybe we can have liberty working asap
16:45 sergmelikyan joined #puppet-openstack
16:47 _ody_ sbadia: Is there any reason to keep around
16:47 _ody_ spec/acceptance/nodesets/nodepool.yml
16:48 _ody_ spec/acceptance/nodesets/default.yml too I suppose.
16:48 crinkle nodepool.yml should go away, default.yml should be changed to a vagrant nodeset
16:50 _ody_ crinkle: there are vagrant nodesets for ubuntu and centos already in msync.
16:51 crinkle _ody_: in the puppetlabs modules we usually have the default be a copy of one of the others
16:52 _ody_ crinkle: If it is a standard then ok.
16:54 openstackgerrit Timothy Swanson proposed openstack/puppet-neutron: Update Cisco ML2 classes and templates  https://review.openstack.org/197181
17:03 EmilienM Hunner: I guess that will be 3.5.1 ?
17:04 Hunner Context?
17:07 EmilienM Hunner: sorry, puppetlabs-mysql release
17:07 EmilienM because it will block our beaker jobs until we pin another release
17:08 EmilienM I guess we'll need to bump
17:08 _ody Slow CI today?
17:10 Hunner EmilienM: I want to get https://tickets.puppetlabs.com/browse/MODULES-2321 in too, so probably 3.6.0
17:10 Hunner EmilienM: Also, please don't push PR branches to puppetlabs github; use a fork :)
17:12 Hunner oh, that's postgres. nm
17:12 Hunner EmilienM: Well, we already merged code to decouple the mysql_* resources from mysql::server, so that is a feature bump
17:30 tiswanso joined #puppet-openstack
17:30 openstackgerrit Ryan Hefner proposed openstack/puppet-tripleo: Enable Manila Service  https://review.openstack.org/204249
17:32 EmilienM Hunner: sorry for the branch, I won't do that next time
17:32 Hunner EmilienM: No worries. I blame github UX ;)
17:33 guimaluf EmilienM, pabelanger it seems the 'recheck' comment doesn't work...
17:35 pabelanger guimaluf: review?
17:35 guimaluf pabelanger, https://review.openstack.org/#/c/207161/
17:36 pabelanger guimaluf: it is running on status.openstack.org/zuul
17:36 pabelanger still in the gate
17:37 pabelanger s/gate/check queue/
17:38 guimaluf pabelanger, oh! I got it... I tought it would clear gate status like when a new patch set is push
17:38 guimaluf pabelanger, thanks! sorry for bothering you
17:39 EmilienM Hunner: https://github.com/puppetlab​s/puppetlabs-mysql/pull/742
17:39 pabelanger guimaluf: no problems
17:57 dprince joined #puppet-openstack
18:01 tiswanso joined #puppet-openstack
18:04 Hunner EmilienM: Thanks ^_^
18:06 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Fix rgw keystone to use correct URL/API string  https://review.openstack.org/153783
18:06 EmilienM Hunner: can we backport it?
18:06 EmilienM Hunner: to 3.5.x ?
18:06 Hunner EmilienM: sadface
18:07 EmilienM Hunner: I'm just looking for a sane way to have this patch in our CI
18:07 EmilienM without checkouting master
18:07 Hunner Hmm. I wonder if I can spin mysql 3.5.1 instead of 3.6.0...
18:07 EmilienM currently, our CI is on 3.4.1
18:07 EmilienM but we can go to 3.5.x
18:08 Hunner EmilienM: oh is that what you want. I can push 3.6.x... but first, do you think https://github.com/puppetlabs/puppe​tlabs-mysql/compare/3.5.0...master is worth a feature release?
18:08 EmilienM Hunner: let me some time to see
18:08 EmilienM Hunner: no
18:08 EmilienM I would say 'push 3.5.1'
18:09 Hunner I *could* say that decoupling mysql_* from mysql::server is a bugfix
18:09 EmilienM yeah but other than that
18:10 EmilienM both ways work for me
18:10 EmilienM Hunner: ^
18:11 openstackgerrit Merged openstack/puppet-ironic: Add bifrost manifest  https://review.openstack.org/202025
18:14 Hunner EmilienM: 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 EmilienM Hunner: no
18:14 EmilienM Hunner: go ahead! boom
18:18 Hunner 3.6.x pushed and .0 should be released around 10th-12th
18:21 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Puppetfile: bump MySQL module to 3.6.x  https://review.openstack.org/209209
18:21 EmilienM crinkle, Hunner ^
18:21 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: acceptance: bump to Liberty release  https://review.openstack.org/209097
18:28 openstackgerrit Cody Herriges proposed openstack/puppet-designate: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189864
18:30 openstackgerrit Cody Herriges proposed openstack/puppet-gnocchi: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189866
18:32 openstackgerrit Cody Herriges proposed openstack/puppet-sahara: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189877
18:33 openstackgerrit Alex Schultz proposed openstack/puppet-horizon: Make redirect type configurable  https://review.openstack.org/208462
18:34 openstackgerrit Cody Herriges proposed openstack/puppet-trove: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189881
18:34 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: WIP: Introduce swift_service class  https://review.openstack.org/203220
18:36 openstackgerrit Cody Herriges proposed openstack/puppet-cinder: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189863
18:37 openstackgerrit Cody Herriges proposed openstack/puppet-glance: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189865
18:38 openstackgerrit Cody Herriges proposed openstack/puppet-ironic: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189870
18:40 openstackgerrit Cody Herriges proposed openstack/puppet-manila: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189872
18:41 openstackgerrit Cody Herriges proposed openstack/puppet-nova: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189874
18:42 openstackgerrit Cody Herriges proposed openstack/puppet-openstacklib: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189875
18:43 EmilienM crinkle: it seems like my attempt to use Depends-On does not work on https://review.openstack.org/209097 - any idea,
18:43 openstackgerrit Cody Herriges proposed openstack/puppet-heat: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189868
18:45 openstackgerrit Cody Herriges proposed openstack/puppet-tuskar: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189882
18:46 openstackgerrit Alex Schultz proposed openstack/puppet-horizon: Make redirect type configurable  https://review.openstack.org/208462
18:47 crinkle EmilienM: looks like it's working to me?
18:47 EmilienM crinkle: it's pulling 3.5.x
18:47 EmilienM but I wanted 3.6.x
18:47 EmilienM I mean, the Depends-On works
18:47 crinkle EmilienM: the 3.6.x branch still has 3.5.0 as the metadata versionb https://github.com/puppetlabs/puppet​labs-mysql/blob/3.6.x/metadata.json
18:48 EmilienM but it's not pulling the right version of MySQL module
18:48 EmilienM ah
18:48 EmilienM good catch
18:48 EmilienM crinkle: sorry for noise
18:48 crinkle :)
18:48 openstackgerrit Cody Herriges proposed openstack/puppet-horizon: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189869
18:50 openstackgerrit Cody Herriges proposed openstack/puppet-keystone: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189871
18:50 EmilienM crinkle: we might need to review patches in puppet openstack integration because no patch will be merged until basic stricture is in place
18:51 EmilienM like https://review.openstack.org/209209
18:51 openstackgerrit Alex Schultz proposed openstack/puppet-horizon: Make redirect type configurable  https://review.openstack.org/208462
18:52 openstackgerrit Cody Herriges proposed openstack/puppet-vswitch: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189883
18:53 openstackgerrit Cody Herriges proposed openstack/puppet-swift: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189878
18:53 xarses joined #puppet-openstack
18:53 openstackgerrit Cody Herriges proposed openstack/puppet-tripleo: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189880
18:54 xarses joined #puppet-openstack
18:55 openstackgerrit Clayton O'Neill proposed openstack/puppet-cinder: Add lock_path to cinder config  https://review.openstack.org/209216
18:58 openstackgerrit Cody Herriges proposed openstack/puppet-openstack_extras: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189876
18:58 openstackgerrit Clayton O'Neill proposed openstack/puppet-cinder: Add lock_path to cinder config  https://review.openstack.org/209216
19:00 paramite joined #puppet-openstack
19:01 jasondotstar joined #puppet-openstack
19:03 EmilienM iurygregory: reviewed
19:04 iurygregory thanks EmilienM, after my coffee i'll take a look ^^
19:05 EmilienM iurygregory: take 2 coffees
19:05 openstackgerrit Cody Herriges proposed openstack/puppet-openstack_extras: Fixing lint errors  https://review.openstack.org/209219
19:06 xingchao joined #puppet-openstack
19:08 openstackgerrit Cody Herriges proposed openstack/puppet-openstack_extras: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189876
19:09 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: WIP: Introduce swift_service class  https://review.openstack.org/203220
19:11 openstackgerrit Cody Herriges proposed openstack/puppet-tempest: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189879
19:14 dprince joined #puppet-openstack
19:18 vinsh Is their something stopping the puppet-lint jobs from running today?
19:21 mgagne EmilienM: now yes: bug #1481460
19:21 openstack bug 1481460 in puppet-glance "Cannot configure multiple Glance stores" [Undecided,New] https://launchpad.net/bugs/1481460
19:22 EmilienM mgagne: this one is high or critical I guess
19:22 mgagne EmilienM: 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 configured
19:29 dgurtner joined #puppet-openstack
19:30 angdraug joined #puppet-openstack
19:32 EmilienM can someone review https://review.openstack.org/192176 and https://review.openstack.org/190016 please ?
19:33 openstackgerrit Cody Herriges proposed openstack/puppet-tempest: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189879
19:35 crinkle EmilienM: why does the upgrade test fail on 190016? it looks potentially related
19:38 EmilienM crinkle: might be related to https://review.openstack.org/#/c/208019/
19:38 EmilienM or not
19:39 tiswanso joined #puppet-openstack
19:39 EmilienM crinkle: upgrade jobs are broken now
19:39 EmilienM but not the same issue
19:40 EmilienM crinkle: so there  is some in my patch that breaks upgrade?
19:40 EmilienM I'm rebasing in case of
19:40 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: Validate service_identity resources  https://review.openstack.org/190016
19:41 crinkle EmilienM: 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 related
19:41 EmilienM crinkle: my patch just show warnings or errors
19:41 EmilienM crinkle: it's a sanity patch
20:00 iurygregory Hey EmilienM, you think is more valid have one class for service provider instead of three right?
20:00 EmilienM yeah
20:00 EmilienM because they share parameters, right?
20:02 Fin1te joined #puppet-openstack
20:04 iurygregory mellon and shibboleth yes, but openid connector don't use some parameters and have others
20:05 openstackgerrit Dan Prince proposed openstack/puppet-tripleo: Add pre_create_files resource and helper class  https://review.openstack.org/208774
20:05 openstackgerrit Dan Prince proposed openstack/puppet-tripleo: tripleo::noop: add support for more resources  https://review.openstack.org/209030
20:19 iurygregory about the functional testing with acceptance,  i need describe the test in the spec or just say that functional test will be written?
20:37 sergmelikyan joined #puppet-openstack
20:48 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Fix a race condition in mon creation  https://review.openstack.org/209250
20:51 sergmelikyan joined #puppet-openstack
21:07 rcallawa_ joined #puppet-openstack
21:17 openstackgerrit Vasyl Saienko proposed openstack/puppet-horizon: allow to pass more params to 'horizon::wsgi::apache'  https://review.openstack.org/209256
21:40 dmsimard joined #puppet-openstack
21:57 dachary1 joined #puppet-openstack
22:00 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Fix rgw keystone  https://review.openstack.org/153783
22:35 openstackgerrit Dan Prince proposed openstack/puppet-tripleo: Add pre_create_files resource and helper class  https://review.openstack.org/208774
22:35 openstackgerrit Dan Prince proposed openstack/puppet-tripleo: tripleo::noop: add support for more resources  https://review.openstack.org/209030
22:42 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Fix rgw keystone  https://review.openstack.org/153783
22:43 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Fix rgw keystone  https://review.openstack.org/153783
23:00 rcallawa_ joined #puppet-openstack
23:01 rcallawa_ joined #puppet-openstack
23:07 gildub joined #puppet-openstack
23:12 EmilienM crinkle: 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:14 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Fix rgw keystone  https://review.openstack.org/153783
23:19 tiswanso joined #puppet-openstack
23:20 jasondotstar joined #puppet-openstack
23:25 openstackgerrit Merged stackforge/puppet-ceph: Enable GPG for EPEL  https://review.openstack.org/209081
23:37 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Initial import to gate on  https://review.openstack.org/203742
23:38 EmilienM crinkle: ^
23:39 EmilienM crinkle: is the commit message good?
23:39 crinkle pabelanger: ^ can you clarify what the difference we between patchset 5 and 6 and if the weird rakefile was needed?
23:39 crinkle EmilienM: lgtm
23:40 EmilienM crinkle: thx
23:41 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Puppetfile: bump MySQL module to 3.6.x  https://review.openstack.org/209209
23:43 EmilienM crinkle: I rebased https://review.openstack.org/#/c/190016/ and upgrade fails for the same reason as other patches so not related.
23:44 crinkle EmilienM: ok
23:44 EmilienM crinkle: I'll work on upgrade jobs when integration will be in place
23:44 EmilienM crinkle: when we have logs + integration in place, I focus on upgrade
23:44 EmilienM but I can't work // on that, it's too dependent I think
23:44 EmilienM crinkle: my last request for today: https://review.openstack.org/#/c/192176/ also
23:44 EmilienM thanks
23:51 delattec joined #puppet-openstack

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