Thursday, 2016-03-24

« Wednesday, 2016-03-23 Index Friday, 2016-03-25 »
*** ansmith has joined #puppet-openstack00:01
*** xarses has quit IRC00:06
*** ansmith has quit IRC00:07
openstackgerritMerged openstack/puppet-glance: Configure endpoint for Glance Glare  https://review.openstack.org/29665600:11
openstackgerritMerged openstack/puppet-ceph: Refactor ceph::profile::rgw  https://review.openstack.org/29605000:11
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge  https://review.openstack.org/29610200:11
*** ducttape_ has joined #puppet-openstack00:12
*** ducttape_ has quit IRC00:17
*** ansmith has joined #puppet-openstack00:17
*** aimon has joined #puppet-openstack00:25
*** xarses has joined #puppet-openstack00:40
*** ducttape_ has joined #puppet-openstack00:42
*** ansmith has quit IRC00:44
*** ansmith has joined #puppet-openstack00:53
*** ansmith has quit IRC01:02
*** ducttape_ has quit IRC01:02
*** ducttape_ has joined #puppet-openstack01:03
*** angdraug has quit IRC01:13
*** baker has joined #puppet-openstack01:14
*** brandon has quit IRC01:15
*** ansmith has joined #puppet-openstack01:22
*** ansmith has quit IRC01:37
*** ducttape_ has quit IRC01:57
*** mtreinish has quit IRC01:59
*** mtreinish has joined #puppet-openstack02:01
*** ilbot3 has quit IRC02:45
*** ilbot3 has joined #puppet-openstack02:47
*** baker_ has joined #puppet-openstack02:51
*** baker has quit IRC02:54
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge  https://review.openstack.org/29610203:06
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge  https://review.openstack.org/29610203:10
*** fedexo has joined #puppet-openstack03:29
*** richm has quit IRC03:31
*** baker_ has quit IRC03:58
*** fedexo has quit IRC04:44
*** saneax is now known as saneax_AFK05:04
*** saneax_AFK is now known as saneax05:06
*** markvoelker has quit IRC05:28
*** jaosorior has joined #puppet-openstack06:15
*** jaosorior has quit IRC06:17
*** jaosorior has joined #puppet-openstack06:18
*** brandon has joined #puppet-openstack06:20
*** aimon has quit IRC06:20
*** rcernin has joined #puppet-openstack06:23
*** brandon has quit IRC06:23
*** markvoelker has joined #puppet-openstack06:29
*** markvoelker has quit IRC06:34
*** finchd has quit IRC06:49
*** aimon has joined #puppet-openstack06:52
openstackgerritMatthew J Black proposed openstack/puppet-keystone: keystone user provider fix for v3 calls  https://review.openstack.org/28999506:57
*** aimon has quit IRC07:00
*** jmikolajczak has joined #puppet-openstack07:13
*** tesseract has joined #puppet-openstack07:41
*** tesseract is now known as Guest2798207:42
openstackgerritSaverio Proto proposed openstack/puppet-cinder: Add osapi_max_limit parameter  https://review.openstack.org/29693107:50
*** markvoelker has joined #puppet-openstack07:56
*** finchd has joined #puppet-openstack07:58
*** markvoelker has quit IRC08:02
openstackgerritSaverio Proto proposed openstack/puppet-cinder: Add osapi_max_limit parameter  https://review.openstack.org/29693108:28
*** foxhunt_away is now known as foxhunt7208:38
*** aimon has joined #puppet-openstack08:41
*** aimon has quit IRC08:45
*** higgins has quit IRC08:51
*** foxhunt72 is now known as foxhunt_away08:54
*** foxhunt_away is now known as foxhunt7209:01
*** chem has joined #puppet-openstack09:07
openstackgerritRicardo Rocha proposed openstack/puppet-magnum: fix incorrect rabbit_userid parameter  https://review.openstack.org/29565209:08
openstackgerritRicardo Rocha proposed openstack/puppet-magnum: Add conductor install and configuration.  https://review.openstack.org/27522309:08
openstackgerritRicardo Rocha proposed openstack/puppet-magnum: minor doc fixes  https://review.openstack.org/29381409:08
*** social has joined #puppet-openstack09:20
*** igajsin1 has joined #puppet-openstack09:22
*** igajsin1 has left #puppet-openstack09:23
*** aimon has joined #puppet-openstack09:35
*** aimon has quit IRC09:40
*** _degorenko|afk is now known as degorenko09:51
*** markvoelker has joined #puppet-openstack09:58
*** higgins has joined #puppet-openstack10:00
*** markvoelker has quit IRC10:03
*** imcsk8 has quit IRC10:14
*** imcsk8 has joined #puppet-openstack10:14
*** higgins has quit IRC10:20
openstackgerritSaverio Proto proposed openstack/puppet-cinder: Add osapi_max_limit parameter  https://review.openstack.org/29693110:26
*** aimon has joined #puppet-openstack10:29
*** saneax is now known as saneax_AFK10:30
*** aimon has quit IRC10:34
*** jaosorior has quit IRC10:34
*** higgins has joined #puppet-openstack10:35
*** jaosorior has joined #puppet-openstack10:43
*** saneax_AFK is now known as saneax10:45
openstackgerritDenis Egorenko proposed openstack/puppet-glance: Add missed glare_config options for glance::config class  https://review.openstack.org/29639010:48
*** ansmith has joined #puppet-openstack10:48
openstackgerritMichael Polenchuk proposed openstack/puppet-nova: Autorequire nova security group  https://review.openstack.org/29654811:04
openstackgerritJerzy Mikolajczak proposed openstack/puppet-neutron: Fix useless tenant_name parameter being set  https://review.openstack.org/29702811:08
*** ansmith has quit IRC11:08
*** brad[] has quit IRC11:08
*** brad[] has joined #puppet-openstack11:09
EmilienMo/11:37
degorenkomorning :)11:37
*** trown|outtypewww is now known as trown11:58
*** dgurtner has joined #puppet-openstack11:58
*** markvoelker has joined #puppet-openstack11:59
*** morazi has joined #puppet-openstack11:59
openstackgerritIvan Berezovskiy proposed openstack/puppet-oslo: Fix values for rabbit_hosts  https://review.openstack.org/29396412:01
*** jayg|g0n3 is now known as jayg12:13
*** alee has quit IRC12:24
openstackgerritDmitry Burmistrov proposed openstack/puppet-heat: Support preferred Keystone auth methods  https://review.openstack.org/26132612:30
*** ansmith has joined #puppet-openstack12:32
*** dgurtner has quit IRC12:33
iurygregorymorning12:33
*** foxhunt72 is now known as foxhunt_away12:43
*** johnmilton has joined #puppet-openstack13:00
openstackgerritJerzy Mikolajczak proposed openstack/puppet-neutron: Fix useless tenant_name parameter being set  https://review.openstack.org/29702813:08
openstackgerritGonéri Le Bouder proposed openstack/puppet-ironic: expose the ipxe_enabled parameter  https://review.openstack.org/29677413:12
*** gfidente has joined #puppet-openstack13:13
*** ducttape_ has joined #puppet-openstack13:13
*** ducttape_ has quit IRC13:16
*** samuelBartel has joined #puppet-openstack13:19
*** ansmith has quit IRC13:25
*** dgurtner has joined #puppet-openstack13:32
*** baker has joined #puppet-openstack13:38
*** alee has joined #puppet-openstack13:40
*** ansmith has joined #puppet-openstack13:40
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-openstack-integration: Fix bug and non-idiom in bash scripts.  https://review.openstack.org/29711413:41
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Allow the Redis specific monitor to use authentication  https://review.openstack.org/29711513:41
*** baker has quit IRC13:42
*** foxhunt_away is now known as foxhunt7213:43
openstackgerritMerged openstack/puppet-tripleo: Allow the Redis specific monitor to use authentication  https://review.openstack.org/29593713:44
*** _skolekonov is now known as skolekonov13:50
*** ansmith has quit IRC13:55
*** ducttape_ has joined #puppet-openstack13:56
degorenkomwhahaha, EmilienM can we guys land it: https://review.openstack.org/#/c/296390/ ?13:57
mwhahahamaybe :D13:58
EmilienMI was about to branch13:58
EmilienMgrrrr13:58
*** xarses has quit IRC14:01
*** ducttape_ has quit IRC14:01
EmilienMskolekonov: if you have the chance to help, I'm still fighting with https://review.openstack.org/#/c/296102/14:03
degorenkoEmilienM, mwhahaha thanks :)14:06
EmilienMok I'm branching except for puppet-heat14:08
degorenkoEmilienM, patch is ready https://review.openstack.org/26132614:08
degorenkoall ci's green14:08
EmilienMdegorenko: I'll -114:08
EmilienMthe release note is not enough14:08
EmilienMcan you add a deprecation section, you're dropping deprecated parameters, no?14:09
degorenkoEmilienM, sure14:09
EmilienMsorry14:09
degorenkodmburmistrov, ^^^^14:09
*** ansmith has joined #puppet-openstack14:09
*** ducttape_ has joined #puppet-openstack14:09
EmilienMit will help people reading release notes, so they understand they need to update their manifests. Thanks14:10
dmburmistrovEmilienM, ok, no problem, just give me a few minutes :)14:10
EmilienMsure, don't worry I will wait to have it in stable/mitaka14:12
*** haplo37 has joined #puppet-openstack14:12
degorenkoEmilienM, thank you a lot for this!14:12
EmilienMmwhahaha, mfisch, degorenko: do you have anything else?14:12
degorenkoi guess no14:12
EmilienMI'm doing branching right now14:12
degorenkoEmilienM, just wait please https://review.openstack.org/296390 for glance it is on gate now14:12
degorenkoEmilienM, do you need any help? :)14:13
*** hjensas has quit IRC14:13
EmilienMdegorenko: sure, of course14:13
EmilienMdegorenko: for release, that's fine. SHould be quick14:13
EmilienMfor https://review.openstack.org/#/c/296102/ yes I need help but I can't work on it later14:13
EmilienMdegorenko: does fuel use linuxbridge?14:13
degorenkoEmilienM, no14:13
EmilienMok14:14
degorenkoEmilienM, skolekonov looking at that 29610214:14
EmilienMok we're having an infra change14:17
EmilienMand we'll need https://review.openstack.org/#/q/status:open+branch:master+topic:nodesets/trusty14:17
EmilienMI'm doing branching to make progress and will backport the nodesets things, when JJB is updated14:18
EmilienMwell, I can wait also14:19
EmilienMI'm waiting for branching, let's deal with CI now14:19
dmburmistrovEmilienM, here is current RN: http://paste.openstack.org/show/491720/  -- is it okay?14:19
EmilienMdmburmistrov: I see more the last note in deprecation rather upgrade14:20
dmburmistrovEmilienM, I did it according to this docs http://docs.openstack.org/developer/reno/usage.html - look at the "deprecations" description14:21
EmilienMdmburmistrov: you're right then, go ahead14:21
dmburmistrovI think I can add a few words in "deprecations" (about usage non-auth_plugin methods)14:22
dmburmistrovthanks for thought!14:22
openstackgerritDmitry Burmistrov proposed openstack/puppet-heat: Support preferred Keystone auth methods  https://review.openstack.org/26132614:26
skolekonovEmilienM, let's wait please for https://review.openstack.org/#/c/297028 before creating a branch14:26
michchap_EmilienM: jwt 1.5.3 doesn't exist?14:26
degorenkoEmilienM, ci page is updated for fuel ci :) https://wiki.openstack.org/wiki/Puppet/CI14:27
*** baker has joined #puppet-openstack14:27
openstackgerritGonéri Le Bouder proposed openstack/puppet-ironic: expose the ipxe_enabled parameter  https://review.openstack.org/29677414:28
skolekonovor we can just backport it14:28
EmilienMmichchap_: jwt?14:30
EmilienMdegorenko: thx14:30
michchap_EmilienM: http://logs.openstack.org/12/292712/14/gate/gate-puppet-tripleo-puppet-unit-4.0-dsvm-centos7/96d178f/console.html14:30
michchap_EmilienM: bundle install is failing in CI because it's looking for a version 1.5.3 of jwt gem which doesn't exist afaict14:30
michchap_EmilienM: (I have no idea what jwt is)14:31
EmilienMmichchap_: it's a random error, isn't?14:31
EmilienM2016-03-24 14:17:51.658 | Gem::RemoteFetcher::FetchError: bad response Not Found 404 (https://rubygems.global.ssl.fastly.net/gems/jwt-1.5.3.gem)14:31
michchap_EmilienM: I get the same on local box14:31
EmilienMor not14:31
michchap_https://rubygems.org/gems/jwt/versions/1.5.314:31
EmilienMa new broken requirement?14:32
EmilienMmichchap_: who requires it?14:32
EmilienMskolekonov: we'll backport it I guess14:33
michchap_EmilienM: looking14:33
mwhahahaboo people who yank versions14:34
michchap_mwhahaha: topical :D14:34
EmilienMif someone breaks our CI during release14:35
EmilienMwell, I don't tell.14:36
*** jaosorior has quit IRC14:37
*** baker has quit IRC14:38
michchap_EmilienM: I'm not sure..I don't see it in the list if I do gem dependency14:38
*** xarses has joined #puppet-openstack14:39
*** _ducttape_ has joined #puppet-openstack14:44
*** ducttape_ has quit IRC14:47
openstackgerritEmilien Macchi proposed openstack/puppet-vswitch: Switch to ubuntu-trusty DIBS  https://review.openstack.org/29581914:49
mwhahahahttps://github.com/jwt/ruby-jwt/issues/133 :/15:02
EmilienMmichchap_: does it break our CI? or only puppet-tripleO?15:03
mwhahahaall the puppet unit jobs15:03
mwhahahaall the fuel ones are broken as well15:04
EmilienMdamn15:04
mwhahahawhat does NOT_REGISTERED mean for the CI again?15:11
EmilienMmwhahaha: that means the Jenkins job is not registered with Gearman15:12
degorenko^ yep15:12
EmilienMit happens when zuul is restarted and a layout changed15:12
mwhahahaso we just have to wait?15:12
EmilienMyes15:12
degorenkorecheck and wiat :)15:12
degorenkowait*15:12
EmilienMit happenned because pabelanger is changing the trusty nodes for puppet CI15:12
mwhahahak15:12
EmilienMwe'll use ubuntu DIBS15:12
EmilienMit's very cool15:12
*** mkarpin has quit IRC15:13
*** dmburmistrov has quit IRC15:13
*** clayton has quit IRC15:13
*** dmsimard has quit IRC15:13
*** Mosibi_ has quit IRC15:13
mwhahahato unblock fuel-library, we're trying to pin to 1.5.2 https://review.openstack.org/#/c/297193/15:13
EmilienMyeah but... it requires to patch all our modules :(15:14
mwhahahayea hopefully they can publish 1.5.4 or something15:15
*** Guest27982 has quit IRC15:17
*** rcernin has quit IRC15:17
*** fvollero has quit IRC15:17
*** fvollero has joined #puppet-openstack15:18
*** dmsimard has joined #puppet-openstack15:25
*** baker has joined #puppet-openstack15:25
openstackgerritRicardo Rocha proposed openstack/puppet-magnum: Add conductor install and configuration.  https://review.openstack.org/27522315:27
*** imcsk8 has quit IRC15:28
*** imcsk8_ has joined #puppet-openstack15:28
*** dgurtner has quit IRC15:29
*** emccormickva has joined #puppet-openstack15:29
*** dmburmistrov has joined #puppet-openstack15:29
*** mkarpin has joined #puppet-openstack15:29
*** clayton has joined #puppet-openstack15:29
*** Mosibi_ has joined #puppet-openstack15:29
*** wolfe.freenode.net sets mode: +o clayton15:29
*** Guest27982 has joined #puppet-openstack15:29
*** rcernin has joined #puppet-openstack15:29
EmilienMmwhahaha: pin worked?15:29
*** emccormick has quit IRC15:32
*** emccormick__C7 has joined #puppet-openstack15:33
EmilienMmwhahaha: w00t, they released 1.5.415:34
mwhahahayay15:34
mwhahahathe pin didn't work because of the NOT_REGISTERED stuff15:34
EmilienMk15:34
*** emccormickva has quit IRC15:36
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Do not merge: Test Ceph with Infernalis  https://review.openstack.org/29722615:39
mwhahahathe recheck came in, yea the pin worked as an alternative. glad to see they release 1.5.4 tho15:43
*** aimon has joined #puppet-openstack15:44
EmilienMI'm going to play with Gemfile, I'm not super familiar yet15:46
openstackgerritEmilien Macchi proposed openstack/puppet-nova: CI test - will fail  https://review.openstack.org/29723115:49
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper: Test puppet-nova Gemfile change  https://review.openstack.org/29723315:50
EmilienMmwhahaha: ^ wdyt?15:50
*** angdraug has joined #puppet-openstack15:51
mwhahahalet me see15:51
EmilienMthough I don't know how to deal with group :development, :test now15:52
mwhahahathat's an interesting way to do it, if it works it still allows for per-module additions but we could get the core dependencies into one place15:52
EmilienMright15:52
EmilienMI see a regression though, it's the groups we have in Gemfile15:53
EmilienMwhich means, we'll install all gems, everytime15:53
EmilienMto me, it's not a blocker but I might miss something15:53
mwhahahawell it depends on how we target it15:53
EmilienMok openstack infra is having serious issues with jenkins (oom, etc)15:53
mwhahahaspec helper would only cover those int he :Development, :test groups15:53
EmilienMoh right15:54
EmilienMbut I would expect for doing it simple to use puppet-openstack_spec_helper in all groups15:54
mwhahahawe'd just have to figure out how to split apart the system test group from the others15:55
mwhahahabut i think a good step 1 would be just to handle the development, test groups and leave teh system test group in there15:55
EmilienMok15:56
EmilienMmwhahaha: I would like a same system as other python project, with openstack/requirement15:56
EmilienMit would make our life so easier15:56
xarsesdmsimard: maybe we should re-implement loic's multi version ceph testing?15:57
EmilienMxarses: I disagree, we had the same challenge in Puppet OpenStack, and we said each puppet module release would support the project module. Example: puppet-glance/mitaka would support Mitaka only.15:59
*** jmikolajczak has quit IRC15:59
EmilienMwe have a blueprint about it, http://specs.openstack.org/openstack/puppet-openstack-specs/specs/kilo/master-policy.html15:59
EmilienMthat's why we need to release more often the puppet-ceph module.16:01
dmsimardxarses: not sure what that statement is about16:02
dmsimardor rather, what it is referring to16:02
dmsimardcontext ?16:02
EmilienMI think xarses is wondering if puppet-ceph functional tests should try different version of ceph16:03
dmsimardis that NOT_REGISTERED stuff fixed ?16:03
EmilienMand I gave my opinion on that16:03
EmilienMdmsimard: not yet AFIK...16:04
dmsimardEmilienM: well, puppet-ceph should definitely test the versions it claims to support16:04
*** emccormickva has joined #puppet-openstack16:04
dmsimardif puppet-ceph says it supports, say, firefly hammer and infernalis16:05
dmsimardand we don't test that it actually works, we're not doing a very good job16:05
EmilienMI think what you need is stable branches and a backport policy16:05
EmilienMbut testing more version does not scale, you need to limit that16:05
dmsimardthere was at most 3 versions tested concurrently I think, not any worse than 3 scenarios and two distros :p16:06
dmsimardstable releases could be dedicated to a ceph version, perhaps, yes16:06
dmsimardor stable branches, rather16:07
EmilienMthat's my point16:07
*** emccormick__C7 has quit IRC16:07
xarseswe had initially wanted to support the two LTS releases of ceph16:08
xarsesand whatever is inbetween16:08
EmilienMdon't run recheck if you see NOT_REGISTERED, infra is dealing with JJB updates at this time.16:08
*** dgurtner has joined #puppet-openstack16:08
*** trown is now known as trown|lunch16:09
*** zino has quit IRC16:12
*** imcsk8_ is now known as imcsk816:27
EmilienMmwhahaha: though I'm wondering if puppet-openstack_spec_helper.gemspec is actually used, since we don't use rubygems.org for puppet-openstack_spec_helper but github16:27
EmilienMif you see what I mean16:27
mwhahahai thought the default source for gems is rubygems16:27
mwhahahalike that's just automagically used for dependency16:28
* mwhahaha shrugs16:28
skolekonovEmilienM, looks like a found a potential problem: /sbin/brctl show br-ex | /bin/grep "No such device" - this will return 1 if br-ex doesn't exist, so it won't be created16:34
EmilienMit's in onlyiff16:34
EmilienMI'm busy on infra issues atm, feel free to take over the patch if you have a fix16:35
skolekonovok. I'll try16:36
EmilienMdo not expect our CI jobs running fine at this time16:37
*** saneax is now known as saneax_AFK16:37
*** EmilienM changes topic to "CI is currently down - do not recheck / Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet"16:37
EmilienMmwhahaha: we might not need https://review.openstack.org/#/q/branch:master+topic:nodesets/trusty16:39
EmilienMplease do not merge the patches16:39
mwhahahak16:42
EmilienMand gerrit is now down16:42
*** Jtsigma has joined #puppet-openstack16:42
EmilienMwhat a nice day to release our modules!16:43
xarsesok, glad that wasn't just me16:43
EmilienMI should have staid in bed16:43
xarsessad that it isn't just me16:43
mfischEmilienM: mfisch CI found a bug in treove16:45
mfischactually cdelatte CI found the bug, mfisch is going to fix it16:45
EmilienMcool :)16:46
EmilienMmfisch: why our CI did not catch it?16:46
openstackgerritSergey Kolekonov proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge  https://review.openstack.org/29610216:46
mfischyou dont set rabbit_hosts I guess?16:46
mfischif our CI was perfect I'd just retire16:46
EmilienMwe set rabbit_host16:46
EmilienMmaybe should we use rabbit_hosts ?16:47
mfischthis might be an ordering issue in how the modules are included16:48
mfischstill looking16:48
mfischEmilienM: our CI isnt including trove::guestagent16:49
mfischunless I dont see it16:49
EmilienMright16:50
EmilienMI'm not sure our Trove testing is really extended16:50
*** samuelBartel has quit IRC16:54
*** openstackgerrit has quit IRC17:01
*** openstackgerrit has joined #puppet-openstack17:02
EmilienMskolekonov: /sbin/brctl show | /bin/grep -w "br-ex17:04
EmilienMif br-ex does not exist, the show will still show br-ex string17:04
EmilienMbrctl show br-ex17:04
EmilienMbridge name bridge id  STP enabled interfaces17:04
EmilienMbr-ex  can't get info No such device17:04
EmilienMso grep will return 017:04
EmilienMand the bridge will never be created17:05
EmilienMthis is what I fixed in latest patchset I believe17:05
skolekonov/sbin/brctl show | /bin/grep -w "br-ex -> this will return 0 if bridge exists, and we have unless17:05
skolekonovthe problem is that 'br-ex  can't get info No such device' is written to stderr17:05
skolekonovso grep ignores it if you don't add 2>&117:06
skolekonov/sbin/brctl show - I just list all bridges, if a bridge doesn't exist, grep will return 1, so it will be created17:07
EmilienMoh, excellent17:07
EmilienMindeed :-)17:07
*** foxhunt72 is now known as foxhunt_away17:10
openstackgerritEmilien Macchi proposed openstack/puppet-nova: CI test - never merge  https://review.openstack.org/29726517:19
*** trown|lunch is now known as trown17:32
*** skolekonov is now known as _skolekonov17:33
*** degorenko is now known as _degorenko|afk17:35
mfischEmilienM: found the issue17:35
mfischhttps://bugs.launchpad.net/puppet-trove/+bug/156165117:35
openstackLaunchpad bug 1561651 in puppet-trove "guestagent.conf template doesn't handle rabbit_hosts" [Undecided,New] - Assigned to Matt Fischer (mfisch)17:35
mfischwill fix17:35
EmilienMI've seen that before17:35
mfischanother module?17:36
mfischif so I will steal the fix17:36
*** rcernin has quit IRC17:36
EmilienMmfisch: no, in trove17:36
EmilienMnevermind17:36
*** ansmith has quit IRC17:41
*** angdraug has quit IRC17:47
mfischEmilienM: can I get your opinion on something17:55
mfischactually let me propose this patch first17:55
EmilienMmfisch: go ahead. I'm afk a bit for lunch17:56
EmilienMI'll catch-up after17:56
*** ansmith has joined #puppet-openstack17:57
mwhahahaEmilienM: 2016-03-24 17:21:46.473 | /home/jenkins/workspace/gate-puppet-glance-puppet-beaker-rspec-ubuntu-trusty/.bundled_gems/gems/beaker-2.37.0/lib/beaker/options/hosts_file_parser.rb:23:in `parse_hosts_file': Host file '/home/jenkins/workspace/gate-puppet-glance-puppet-beaker-rspec-ubuntu-trusty/spec/acceptance/nodesets/nodepool-[trusty].yml' does not18:05
mwhahahaexist! (ArgumentError)18:05
mwhahahahttps://review.openstack.org/#/c/296390/18:05
EmilienMright18:05
EmilienMwe need https://review.openstack.org/297253 to land18:05
EmilienMand be applied on jenkins nodes18:05
EmilienMbut right now zuul is processing the patch in the gate queue18:06
mwhahahak18:06
mfischDo we have any tests that check values from templated files?18:08
mfischI guess contain_file is what I need18:09
dmsimardmfisch: I guess https://github.com/openstack/puppet-nova/blob/master/spec/classes/nova_compute_rbd_spec.rb#L81-L8918:10
mfischdoing something like this18:11
mfisch      should contain_file('/etc/logrotate.d/nginx') \18:11
mfisch        .with_content(/^\s*compress$/)18:11
openstackgerritMatt Fischer proposed openstack/puppet-trove: guestagent.conf template should use rabbit_hosts  https://review.openstack.org/29729318:21
mfischcdelatte: ^18:22
cdelattemfisch: cool.  will take a look shortly18:22
openstackgerritMatt Fischer proposed openstack/puppet-trove: guestagent.conf template should use rabbit_hosts  https://review.openstack.org/29729318:23
mfischcdelatte: we're going to have to rework all of that in order to have it on a separate partition18:23
mfischor we can work around it in cirrus18:23
*** brandon has joined #puppet-openstack18:26
*** cgfbee has quit IRC18:27
*** cgfbee has joined #puppet-openstack18:29
EmilienMok breaker jobs should be back18:41
EmilienMmwhahaha: fyi18:41
mwhahahak18:41
*** ansmith has quit IRC18:44
Jtsigmawhich puppet module installs the openstack cli?18:45
*** dhollinger has joined #puppet-openstack18:45
*** Guest27982 has quit IRC18:46
EmilienMJtsigma: puppet-openstacklib18:46
EmilienMhttps://github.com/openstack/puppet-openstacklib/blob/master/manifests/openstackclient.pp18:47
*** cgfbee has quit IRC18:48
*** cgfbee has joined #puppet-openstack18:50
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper: Test puppet-nova Gemfile change  https://review.openstack.org/29723318:50
EmilienMmwhahaha: I think my stuff with gemfile works18:54
EmilienM\o/18:54
EmilienMhttps://jenkins05.openstack.org/job/gate-puppet-openstack-spec-helper-unit-devstack-centos7/1/console18:54
EmilienMI try to pin rspec-puppet to 2.3.018:55
EmilienMwell > 2.3.018:55
EmilienMand I got 2.3.218:55
EmilienMbefore I had 2.2.018:55
mwhahahacool18:55
*** ansmith has joined #puppet-openstack18:55
JtsigmaEmilienM,  thanks18:55
EmilienMI don't think crinkle is around but I want her thoughts on that18:55
EmilienMJtsigma: anytime18:56
Jtsigmaah yes i see now https://github.com/openstack/puppet-openstacklib/blob/master/manifests/openstackclient.pp18:56
EmilienMJtsigma: yeah and some of our modules include it18:56
EmilienMJtsigma: you can use the class in your composition layer, it should not break your catalog, we use ensure_resource18:56
EmilienMbut I'm not sure you would need to18:56
Jtsigmacool!18:59
mfischEmilienM: okay ready for some Trove talk19:03
EmilienMmfisch: shoot19:03
mfischI dont understand why we have the templated config19:04
mfischthere are 2 issues with it19:04
EmilienMI agree19:04
mfisch1) its a fork of the class19:04
mfisch2) it means that we can't set the guestagent to use different rabbit creds than our control plane19:04
mfischI think if you want guestagent.conf and not the service, you just the dont manage service and packages flags and just use the config19:05
*** dgurtner has quit IRC19:05
mfischEmilienM: I'll start a ML topic19:06
mfischbut this wont be backportable so I'd like to land my other change19:06
EmilienMi'm cutting the branches this afternoon19:07
EmilienMwhat patch do you need?19:07
mfischhttps://review.openstack.org/#/c/297293/219:07
mfischI'll just do a BP when its ready, no rush19:07
*** gfidente has quit IRC19:09
*** angdraug has joined #puppet-openstack19:21
*** Jtsigma has quit IRC19:31
*** Jtsigma has joined #puppet-openstack19:32
EmilienMmfisch: +219:38
*** ddmitriev has joined #puppet-openstack19:38
EmilienMmwhahaha: can you review https://review.openstack.org/#/c/297293/ please? so we can have it in mitaka without backport, if the patch looks good to you19:39
mwhahahain a sec19:41
EmilienMsure, i'm starting branching from A->Z so trove will be later19:41
EmilienMI don't branch puppet-heat yet, to let a chance to https://review.openstack.org/#/c/261326/19:49
openstackgerritMatt Fischer proposed openstack/puppet-trove: trivial comment paste-o fix  https://review.openstack.org/29734020:00
openstackgerritAlex Krzos proposed openstack/puppet-keystone: Tune number of apache processes/threads for Keystone in apache.  https://review.openstack.org/29734220:02
*** cdelatte has quit IRC20:02
crinkleEmilienM: what's up?20:05
EmilienMcrinkle: o/20:05
EmilienMcrinkle: we were looking for a way to have a single place for Gem dependencies20:05
crinkleEmilienM: you could just have all the gems in puppet-openstack_spec_helper, then installing that installs all the dependencies20:06
EmilienMand I thought about using puppet-openstack_spec_helper puppet-openstack_spec_helper.gemspec20:06
EmilienMright20:06
EmilienMbut the thing is, we would loose the ability to choose which gems we want for beaker/ unit20:06
EmilienMthe groups20:06
*** johnmilton has quit IRC20:06
crinklei don't think you even need the .gemspec, i think you can put them in the Gemfile20:07
EmilienMI don't mind about that, it's just a notice20:07
crinklethen you still have the groups20:07
crinklei think the groups were necessary when we were testing on ruby 1.8.7, it might not be necessary any more20:07
EmilienMok so either I create Gemfile or I use .gemspec20:07
EmilienMI tested patching .gemspec and it worked20:08
EmilienMI did not test with a Gemfile20:08
EmilienMcrinkle: what do you prefer?20:08
crinkleif the Gemfile works i'd prefer that20:08
EmilienMok20:08
EmilienMI'll do that, thanks20:08
crinkle.gemspec is for when you're publishing the gem on rubygems and i think we're always going to be installing this from git20:09
EmilienMright20:09
EmilienMso if our CI breaks again because a dep, we can fix it very quickly20:09
*** johnmilton has joined #puppet-openstack20:17
*** simonmcc has quit IRC20:19
*** simonmcc has joined #puppet-openstack20:20
*** johnmilton has quit IRC20:28
openstackgerritJerzy Mikolajczak proposed openstack/puppet-neutron: Fix useless tenant_name parameter being set  https://review.openstack.org/29702820:29
*** haplo37 has quit IRC20:31
EmilienMok everything is branched stable/mitaka except puppet-heat and puppet-trove20:32
openstackgerritMerged openstack/puppet-tripleo: Allow the Redis specific monitor to use authentication  https://review.openstack.org/29711520:32
iurygregorycongrats EmilienM o/20:32
EmilienMiurygregory: thx, we're almost there!20:33
iurygregoryI hope to be more active in the Newton release = ' (some internal problems here20:34
EmilienMiurygregory: we're looking forward!20:35
iurygregoryEmilienM,  i'll join the sessions  in Austin =D20:36
EmilienMiurygregory: awesome!20:36
iurygregoryyour talk was accepted?20:37
EmilienMiurygregory: nope20:39
EmilienMbut it's okay, I'll focus on dev sessions :-)20:39
iurygregorygood luck o/20:40
*** trown is now known as trown|outtypewww20:41
openstackgerritAlex Krzos proposed openstack/puppet-keystone: Tune number of apache processes/threads for Keystone in apache.  https://review.openstack.org/29734220:42
iurygregory^ EmilienM, tune these values ​​will make the CI spend more resources to run tests?20:47
*** ansmith has quit IRC20:47
*** openstackgerrit has quit IRC20:48
*** openstackgerrit has joined #puppet-openstack20:49
*** mdorman has joined #puppet-openstack20:55
EmilienMiurygregory: ayoung and I think it's too much21:01
*** aimon has quit IRC21:01
EmilienMadam reviewed and I agree21:01
EmilienMthis is not sane defaults21:01
ayoungiurygregory, I like the limiting of the threads part21:01
ayoung but on an all-in-one install on a 96 CPU machine, if everything did that, you would flood the machine.21:02
*** aimon has joined #puppet-openstack21:02
dmsimardoh, not just that -- each one of the these threads keeps a connection with mysql, right ? :)21:08
*** johnmilton has joined #puppet-openstack21:10
*** zino has joined #puppet-openstack21:15
openstackgerritMerged openstack/puppet-glance: Add missed glare_config options for glance::config class  https://review.openstack.org/29639021:19
*** jayg is now known as jayg|g0n321:27
*** dhollinger has quit IRC21:30
*** higgins has quit IRC21:33
*** mwhahaha_ has joined #puppet-openstack21:34
*** patrickeast has quit IRC21:34
*** mwhahaha has quit IRC21:34
*** ayoung has quit IRC21:34
*** mwhahaha_ is now known as mwhahaha21:35
*** patrickeast has joined #puppet-openstack21:37
*** openstack has joined #puppet-openstack21:50
*** dmburmistrov has joined #puppet-openstack21:56
*** aimon has quit IRC21:59
openstackgerritMatthew J Black proposed openstack/puppet-keystone: keystone user provider fix for v3 calls  https://review.openstack.org/28999521:59
*** aimon has joined #puppet-openstack21:59
*** mdorman has joined #puppet-openstack22:00
*** mkarpin has joined #puppet-openstack22:00
*** clayton has joined #puppet-openstack22:00
*** Mosibi_ has joined #puppet-openstack22:00
*** wolfe.freenode.net sets mode: +o clayton22:00
mjblackEmilienM: hey, link the the meeting was posted here with regards to my change, https://review.openstack.org/#/c/289995/22:05
mjblackshould I abandon the change?22:05
*** johnmilton has quit IRC22:08
*** ddmitriev has quit IRC22:16
*** _ducttape_ has quit IRC22:17
EmilienMmwhahaha: https://review.openstack.org/#/c/261326/22:20
EmilienMfuel CI failing, no problem?22:20
mwhahahayea there's an issue with dpkg getting locked, unrelated22:21
EmilienMk22:21
openstackgerritEmilien Macchi proposed openstack/puppet-ceph: repo: allow to disable EPEL repository  https://review.openstack.org/29739322:31
EmilienMdmsimard: what I just said on #rdo ^22:31
EmilienMmwhahaha: can you drop +A on https://review.openstack.org/#/c/297293/ please? We need to trigger zuul again... I don't know why22:36
*** alee has quit IRC22:37
mwhahahaweird22:38
EmilienMok now it works22:39
EmilienMit goes in gate22:39
EmilienMthx22:39
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: Updates for stable/mitaka  https://review.openstack.org/29740122:49
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: Updates for stable/mitaka  https://review.openstack.org/29740222:49
openstackgerritEmilien Macchi proposed openstack/puppet-designate: Updates for stable/mitaka  https://review.openstack.org/29740322:49
openstackgerritEmilien Macchi proposed openstack/puppet-glance: Updates for stable/mitaka  https://review.openstack.org/29740422:49
openstackgerritEmilien Macchi proposed openstack/puppet-gnocchi: Updates for stable/mitaka  https://review.openstack.org/29740522:49
openstackgerritEmilien Macchi proposed openstack/puppet-horizon: Updates for stable/mitaka  https://review.openstack.org/29740622:49
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Updates for stable/mitaka  https://review.openstack.org/29740722:49
openstackgerritEmilien Macchi proposed openstack/puppet-manila: Updates for stable/mitaka  https://review.openstack.org/29740822:49
openstackgerritEmilien Macchi proposed openstack/puppet-mistral: Updates for stable/mitaka  https://review.openstack.org/29740922:49
openstackgerritEmilien Macchi proposed openstack/puppet-murano: Updates for stable/mitaka  https://review.openstack.org/29741022:50
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: Updates for stable/mitaka  https://review.openstack.org/29741122:50
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Updates for stable/mitaka  https://review.openstack.org/29741222:50
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_extras: Updates for stable/mitaka  https://review.openstack.org/29741322:50
openstackgerritEmilien Macchi proposed openstack/puppet-openstacklib: Updates for stable/mitaka  https://review.openstack.org/29741422:50
openstackgerritEmilien Macchi proposed openstack/puppet-sahara: Updates for stable/mitaka  https://review.openstack.org/29741522:50
openstackgerritEmilien Macchi proposed openstack/puppet-swift: Updates for stable/mitaka  https://review.openstack.org/29741622:50
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: Updates for stable/mitaka  https://review.openstack.org/29741722:50
openstackgerritEmilien Macchi proposed openstack/puppet-zaqar: Updates for stable/mitaka  https://review.openstack.org/29741822:50
EmilienMmjblack: not sure you need to abandon it22:51
EmilienMmjblack: I would ask deni22:51
EmilienMdenisù22:51
openstackgerritMerged openstack/puppet-heat: Support preferred Keystone auth methods  https://review.openstack.org/26132622:51
EmilienMI would ask Denis if whether or not you need to abandon it22:52
EmilienMbut I don't think so22:52
mjblackEmilienM: all I know is that right now with domain configs on, keystone fails big time for me22:52
EmilienMthat's unfortunate :(22:53
mjblackI can use a cherry-pick commit but did want to see that get into master22:54
EmilienMpuppet-heat branched, now waiting for trove and all will be branched tonight22:55
chemmjblack: (I shouldn't be here...) bear with me for one week at most, this should be fixed soon, I have this https://review.openstack.org/296370 test working (multi-domain with ldap deployment), so the fix will come afterward by removing prefetch as discussed on the mailing list22:55
chemEmilienM: ^22:55
EmilienMchem: excellent!22:55
chemmjblack: Emilien didn't push it as it's too late :) but tomorrow then the start on the removal of prefetch will start22:56
*** johnmilton has joined #puppet-openstack22:56
openstackgerritEmilien Macchi proposed openstack/puppet-heat: Updates for stable/mitaka  https://review.openstack.org/29742122:56
EmilienMchem: fyi tomorrow is off in US & monday too in Canada :-P22:56
chemmjblack: don't look at the code right now, it's only a placeholer22:57
mjblackchem: I'm fine with holding off for about a week ;)22:57
chemEmilienM: lazy North-America22:57
EmilienMchem: yea22:57
chemmjblack: cool :)22:58
EmilienMfor reviewers: more patches on https://review.openstack.org/#/q/status:open+branch:stable/mitaka+topic:puppet/mitaka22:58
EmilienMand last serie is coming22:58
chemmjblack: I'll keep you informed when the patch is there, review will be much welcome (I'm not a ldap wizard ....)23:00
*** saneax_AFK is now known as saneax23:00
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: Import release notes from stable/mitaka  https://review.openstack.org/29742523:02
EmilienMok last serie of spams23:02
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: Import release notes from stable/mitaka  https://review.openstack.org/29742623:02
openstackgerritEmilien Macchi proposed openstack/puppet-designate: Import release notes from stable/mitaka  https://review.openstack.org/29742723:03
openstackgerritEmilien Macchi proposed openstack/puppet-glance: Import release notes from stable/mitaka  https://review.openstack.org/29742823:03
openstackgerritEmilien Macchi proposed openstack/puppet-gnocchi: Import release notes from stable/mitaka  https://review.openstack.org/29742923:03
openstackgerritEmilien Macchi proposed openstack/puppet-horizon: Import release notes from stable/mitaka  https://review.openstack.org/29743023:03
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Import release notes from stable/mitaka  https://review.openstack.org/29743223:03
openstackgerritEmilien Macchi proposed openstack/puppet-manila: Import release notes from stable/mitaka  https://review.openstack.org/29743323:03
openstackgerritEmilien Macchi proposed openstack/puppet-mistral: Import release notes from stable/mitaka  https://review.openstack.org/29743523:03
openstackgerritEmilien Macchi proposed openstack/puppet-murano: Import release notes from stable/mitaka  https://review.openstack.org/29743623:03
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: Import release notes from stable/mitaka  https://review.openstack.org/29743723:03
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Import release notes from stable/mitaka  https://review.openstack.org/29743823:04
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_extras: Import release notes from stable/mitaka  https://review.openstack.org/29743923:04
openstackgerritEmilien Macchi proposed openstack/puppet-openstacklib: Import release notes from stable/mitaka  https://review.openstack.org/29744023:04
openstackgerritEmilien Macchi proposed openstack/puppet-sahara: Import release notes from stable/mitaka  https://review.openstack.org/29744123:04
openstackgerritEmilien Macchi proposed openstack/puppet-swift: Import release notes from stable/mitaka  https://review.openstack.org/29744223:04
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: Import release notes from stable/mitaka  https://review.openstack.org/29744323:04
openstackgerritEmilien Macchi proposed openstack/puppet-zaqar: Import release notes from stable/mitaka  https://review.openstack.org/29744423:04
xarsescommit-oholic23:06
openstackgerritMerged openstack/puppet-trove: guestagent.conf template should use rabbit_hosts  https://review.openstack.org/29729323:09
EmilienMok all is branched23:09
openstackgerritEmilien Macchi proposed openstack/puppet-trove: Updates for stable/mitaka  https://review.openstack.org/29744723:10
*** ducttape_ has joined #puppet-openstack23:11
openstackgerritEmilien Macchi proposed openstack/puppet-heat: Import release notes from stable/mitaka  https://review.openstack.org/29744823:12
openstackgerritEmilien Macchi proposed openstack/puppet-trove: Import release notes from stable/mitaka  https://review.openstack.org/29744923:12
EmilienMok all patches are sent23:13
EmilienMfeel free to review it23:13
EmilienMif all is merged by monday, i'll tag23:13
*** ducttape_ has quit IRC23:25
*** alee has joined #puppet-openstack23:26
*** ducttape_ has joined #puppet-openstack23:26
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: bump RDO Mitaka to March 22th  https://review.openstack.org/29745523:33
*** alee has quit IRC23:33
*** ducttape_ has quit IRC23:33
*** ducttape_ has joined #puppet-openstack23:34
*** ducttape_ has quit IRC23:34
*** ansmith has joined #puppet-openstack23:34
*** _ducttape_ has joined #puppet-openstack23:40
*** xarses has quit IRC23:51
*** zino has quit IRC23:58
« Wednesday, 2016-03-23 Index Friday, 2016-03-25 »

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