Monday, 2015-10-05

« Sunday, 2015-10-04 Index Tuesday, 2015-10-06 »
*** mwoodson has quit IRC00:14
*** mwoodson has joined #puppet-openstack00:14
*** guessi has joined #puppet-openstack00:19
*** sanjayu has quit IRC00:53
*** guessi has left #puppet-openstack01:06
*** imcsk8 has joined #puppet-openstack01:10
*** imcsk8_ has quit IRC01:10
*** ducttape_ has joined #puppet-openstack02:14
*** guessi has joined #puppet-openstack02:37
*** ducttape_ has quit IRC02:40
*** ducttape_ has joined #puppet-openstack02:47
*** ducttape_ has quit IRC03:08
*** _ody has quit IRC03:16
*** guessi has left #puppet-openstack03:37
*** _ody has joined #puppet-openstack03:44
*** lavaboy has joined #puppet-openstack04:00
*** sergmelikyan has joined #puppet-openstack04:07
*** sergmelikyan has quit IRC04:11
*** sanjayu has joined #puppet-openstack05:27
*** lavaboy has quit IRC05:33
*** lavaboy has joined #puppet-openstack05:34
*** saravanankr has joined #puppet-openstack05:36
*** guessi has joined #puppet-openstack05:42
*** saravanankr has quit IRC05:52
*** guessi has left #puppet-openstack05:52
*** darkxploit has quit IRC05:55
*** saravanankr has joined #puppet-openstack06:09
*** darkxploit has joined #puppet-openstack06:11
*** aarefiev_ has quit IRC06:28
*** leanderthal has joined #puppet-openstack06:28
*** jtomasek has joined #puppet-openstack06:28
*** aarefiev has joined #puppet-openstack06:30
*** khushbu has joined #puppet-openstack06:42
*** arnaud_orange has joined #puppet-openstack06:42
*** arnaud_orange has quit IRC06:49
*** arnaud_orange has joined #puppet-openstack07:04
*** jpena has joined #puppet-openstack07:30
*** sergmelikyan has joined #puppet-openstack07:32
*** sergmelikyan has quit IRC07:34
*** dgurtner has joined #puppet-openstack07:46
*** sergmelikyan has joined #puppet-openstack08:03
*** paramite has joined #puppet-openstack08:10
*** khushbu has quit IRC08:14
*** gfidente has joined #puppet-openstack08:16
*** gfidente has quit IRC08:16
*** gfidente has joined #puppet-openstack08:16
*** sergmelikyan has quit IRC08:28
*** mattymo has quit IRC08:29
*** mattymo has joined #puppet-openstack08:29
*** sergmelikyan has joined #puppet-openstack08:38
*** jistr has joined #puppet-openstack08:38
*** paramite is now known as paramite|afk08:47
*** sergmelikyan has quit IRC08:59
*** jistr has quit IRC09:00
*** paramite|afk is now known as paramite09:00
*** jistr has joined #puppet-openstack09:01
*** paramite is now known as paramite|afk09:02
*** markvoelker has joined #puppet-openstack09:05
*** markvoelker_ has joined #puppet-openstack09:06
*** jmccrory has quit IRC09:07
*** jmccrory has joined #puppet-openstack09:08
*** markvoelker has quit IRC09:09
*** sergmelikyan has joined #puppet-openstack09:11
*** sergmelikyan has quit IRC09:12
*** sergmelikyan has joined #puppet-openstack09:14
*** paramite|afk is now known as paramite09:18
*** rochaporto has joined #puppet-openstack09:18
openstackgerritChristian Schwede proposed openstack/puppet-swift: Define ${name}-auditors in storage/generic.pp  https://review.openstack.org/23091409:19
*** markvoelker_ has quit IRC09:20
*** paramite is now known as paramite|afk09:23
myatsenkomfisch: hello Matt! You wrote you faced with some problem,09:26
myatsenkois it related to this patch - https://review.openstack.org/#/c/209412/ ?09:26
*** sergmelikyan has quit IRC09:27
*** sergmelikyan has joined #puppet-openstack09:29
*** paramite|afk is now known as paramite09:31
*** sergmelikyan has quit IRC09:40
openstackgerritChristian Schwede proposed openstack/puppet-swift: Define ${name}-auditors in storage/generic.pp  https://review.openstack.org/23091409:43
openstackgerritYanis Guenane proposed openstack/puppet-cinder: Specify a dedicated keystone user for cinderv2  https://review.openstack.org/23092209:52
*** degorenko has joined #puppet-openstack09:54
*** IBerezovskiy has joined #puppet-openstack09:58
*** iberezovskiy has joined #puppet-openstack09:58
*** sergmelikyan has joined #puppet-openstack10:00
*** sergmelikyan has quit IRC10:24
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Add semantic title for keystone_tenant.  https://review.openstack.org/22691910:35
*** sergmelikyan has joined #puppet-openstack10:41
*** paramite has quit IRC10:46
*** paramite has joined #puppet-openstack10:46
*** _skolekonov is now known as skolekonov10:51
*** igajsin has joined #puppet-openstack10:55
*** lavaboy has quit IRC11:00
openstackgerritYanis Guenane proposed openstack/puppet-horizon: Create horizon::keystone::auth  https://review.openstack.org/23094911:06
openstackgerritYanis Guenane proposed openstack/puppet-horizon: Create horizon::keystone::auth  https://review.openstack.org/23094911:08
*** igajsin has left #puppet-openstack11:08
*** sergmelikyan has quit IRC11:19
*** iurygregory has joined #puppet-openstack11:30
iurygregorygood morning people :D11:31
EmilienMgood morning11:39
degorenkomorning o/11:42
iurygregory\o11:43
openstackgerritDenis Egorenko proposed openstack/puppet-nova: Use OpenstackClient for nova providers auth  https://review.openstack.org/22686211:44
openstackgerritDenis Egorenko proposed openstack/puppet-nova: Use OpenstackClient for nova providers auth  https://review.openstack.org/22686211:48
*** darkxploit has quit IRC11:52
*** jayg|g0n3 is now known as jayg11:54
openstackgerritIvan Berezovskiy proposed openstack/puppet-ceilometer: Add ability to override class parameters  https://review.openstack.org/22991812:00
*** trown|outttypeww is now known as trown12:00
openstackgerritIvan Berezovskiy proposed openstack/puppet-ceilometer: Add ability to override class parameters  https://review.openstack.org/22991812:02
*** v1k0d3n has joined #puppet-openstack12:04
*** darkxploit has joined #puppet-openstack12:04
*** sergmelikyan has joined #puppet-openstack12:09
*** ducttape_ has joined #puppet-openstack12:14
*** saravanankr has quit IRC12:23
*** dprince has joined #puppet-openstack12:23
*** ducttape_ has quit IRC12:27
openstackgerritMerged openstack/puppet-openstacklib: Add os_service_default fact  https://review.openstack.org/22905012:34
EmilienMmwhahaha: it seems your patch is merged ^ - could you clean-up anything useless in openstacklib from previous proposals?12:37
EmilienMmwhahaha: or pobably just adjusting, like the is_service_default function12:38
mwhahahaok12:39
openstackgerritAlex Schultz proposed openstack/puppet-cinder: Switch cinder to use os_service_default fact  https://review.openstack.org/22909512:40
*** jpena is now known as jpena|lunch12:40
*** sergmelikyan has quit IRC12:47
*** sergmelikyan has joined #puppet-openstack12:53
degorenkoguys, take a look please: https://review.openstack.org/#/q/status:open+project:openstack/puppet-nova+topic:bp/use-openstackclient-in-module-resources,n,z12:54
*** sanjayu has quit IRC12:57
*** sanjayu has joined #puppet-openstack12:57
*** sanjayu has quit IRC12:58
*** sanjayu has joined #puppet-openstack12:58
*** sanjayu has quit IRC12:59
*** sanjayu has joined #puppet-openstack12:59
openstackgerritMerged openstack/puppet-glance: file backend: do not inherit from glance::api  https://review.openstack.org/23055613:05
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Application resource implementation  https://review.openstack.org/22023813:15
*** rochaporto has quit IRC13:18
*** sergmelikyan has quit IRC13:19
*** blentz_ is now known as blentz13:20
openstackgerritJohn Trowbridge proposed openstack/puppet-ironic: Add ironic-inspector support  https://review.openstack.org/22369013:23
*** sergmelikyan has joined #puppet-openstack13:25
openstackgerritLi Chengyuan proposed stackforge/puppet-ceph: Change keyring file mode default value to 0644 instead of 0600  https://review.openstack.org/23099313:29
*** morazi has joined #puppet-openstack13:32
*** jasondotstar|afk is now known as jasondotstar13:32
*** dfisher has joined #puppet-openstack13:42
*** mattymo has quit IRC13:46
*** mattymo has joined #puppet-openstack13:47
*** mattymo has quit IRC13:47
*** jpena|lunch is now known as jpena13:48
*** mattymo__ has joined #puppet-openstack13:49
*** mattymo__ has quit IRC13:49
*** tiswanso has joined #puppet-openstack13:50
*** mattymo has joined #puppet-openstack13:51
*** mattymo has quit IRC13:55
*** ferest has joined #puppet-openstack13:55
*** mattymo__ has joined #puppet-openstack13:55
*** mattymo__ has quit IRC13:55
*** mattymo1 has joined #puppet-openstack13:55
*** mattymo has joined #puppet-openstack13:57
*** dfisher has quit IRC13:57
*** dfisher has joined #puppet-openstack13:58
*** sanjayu has quit IRC13:58
*** ducttape_ has joined #puppet-openstack14:00
openstackgerritDrew Fisher proposed openstack/puppet-glance: Remove POSIX users, groups, and file modes.  https://review.openstack.org/21159314:11
*** ferest has quit IRC14:11
dfisherlet's see if CI likes me this morning14:11
*** richm has joined #puppet-openstack14:14
*** Guest38101 is now known as zeus14:14
*** zeus has quit IRC14:14
*** zeus has joined #puppet-openstack14:14
*** paramite is now known as paramite|afk14:24
openstackgerritLi Chengyuan proposed stackforge/puppet-ceph: Change keyring file mode default value to 0644 instead of 0600  https://review.openstack.org/23099314:27
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Client and dashboard class implementation  https://review.openstack.org/21833314:28
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Application resource implementation  https://review.openstack.org/22023814:28
openstackgerritAlex Schultz proposed openstack/puppet-cinder: Switch cinder to use os_service_default fact  https://review.openstack.org/22909514:28
*** paramite|afk is now known as paramite14:36
*** ChanServ sets mode: -o EmilienM14:40
*** devnull0 has joined #puppet-openstack14:48
*** mdorman has joined #puppet-openstack14:52
*** EmilienM has quit IRC14:55
*** EmilienM has joined #puppet-openstack14:55
*** paramite has quit IRC14:58
*** ChanServ sets mode: +o EmilienM14:59
*** tiswanso has quit IRC15:00
*** _ducttape_ has joined #puppet-openstack15:36
*** _ducttape_ has quit IRC15:37
*** _ducttape_ has joined #puppet-openstack15:38
*** ducttape_ has quit IRC15:39
EmilienMrubygems servers have issue today, expect some CI issues15:48
*** angdraug has joined #puppet-openstack15:48
*** sergmelikyan has quit IRC15:51
*** richm has quit IRC15:59
openstackgerritClayton O'Neill proposed openstack/puppet-cinder: Allow RBD host to be configurable  https://review.openstack.org/23106815:59
mfischmyatsenko: yes, it broke us16:04
mfischmyatsenko: also it should be backend_host, host is deprecated16:04
*** fedexo has joined #puppet-openstack16:06
*** bnemec has joined #puppet-openstack16:11
claytonmyatsenko: https://review.openstack.org/231068 is my fix for the issue16:12
*** imcsk8 is now known as imcsk8|afk16:17
mfischclayton: do we really need the absent stuff? I thought cinder supported service_default16:21
aderyugincrinkle, mfish, mgagne, sbadia, spredzy: review https://review.openstack.org/#/c/216682/ and merge if possible. Already got +2 from EmilienM.16:23
*** trown is now known as trown|lunch16:24
*** gfidente is now known as gfidente|afk16:26
*** jistr has quit IRC16:26
mfischclayton: nm on my comment16:31
*** jpena has quit IRC16:40
*** ducttape_ has joined #puppet-openstack16:40
*** skolekonov is now known as _skolekonov16:43
*** _ducttape_ has quit IRC16:43
*** dgurtner has quit IRC16:50
*** richm has joined #puppet-openstack16:50
*** trown|lunch is now known as trown16:57
*** iberezovskiy has quit IRC16:58
*** fedexo has quit IRC17:01
*** iurygregory has quit IRC17:09
trownEmilienM: should I recheck https://review.openstack.org/#/c/223690/ only failures are upgrade and gem install failures on a couple of the unit tests17:21
EmilienMtrown: yes17:22
dfisherEmilienM is rubygems back to life?17:22
*** xarses has quit IRC17:23
EmilienMsounds unstable17:24
*** richm has quit IRC17:26
EmilienMmfisch, clayton, mgagne, crinkle: when you got time, I would like more eyes on https://review.openstack.org/#/c/226624/ - it's some "not backward compatibible" stuff for liberty.17:28
EmilienMsince domains were experimental for us, I guess such a design change is acceptable17:30
*** openstackgerrit has quit IRC17:31
*** openstackgerrit has joined #puppet-openstack17:32
*** rahulait has joined #puppet-openstack17:40
*** fedexo has joined #puppet-openstack17:41
*** xarses has joined #puppet-openstack17:44
*** sergmelikyan has joined #puppet-openstack17:48
*** fedexo has quit IRC17:48
*** angdraug has quit IRC17:50
*** iurygregory has joined #puppet-openstack17:50
EmilienMmfisch: could you justify your +1 on https://review.openstack.org/#/c/231068/ ? (why not +2)17:51
EmilienMI'm always curious why core folks use +1 without comments.17:51
mfischdidn't want to +2 something from someone from my company with whom I've discussed this17:51
claytonI assumed it's because he avoiding looking biased.17:51
EmilienMclayton: lol17:51
mfischI'd be happy to be the 2nd +217:51
EmilienMwe already had this discussion with spredzy and sbadia17:52
mfischthis isnt Horizon right ducttape_ ?  We expect other companies to +2 our work ;)17:52
EmilienMas soon the review & discussions remains open, I don't think we should have the "company barrier"17:52
mfischI'm +2 on it for sure ;)17:52
ducttape_no comment mfisch17:52
mfischI'd like to get some feedback from the original author as well17:53
EmilienMwhat I mean is, if the patch is good, and we're not doing ninja reviews (ie, not let some time to review it), it's fine17:53
EmilienMclayton: why do you drop the test on https://review.openstack.org/#/c/231068/1/spec/classes/cinder_volume_rbd_spec.rb,cm ?17:54
claytonbecause it should have never been in that file17:54
claytonthat's a volume test file, but the feature being tested is a backend feature.17:54
EmilienMclayton: ok. I've +2-ed17:54
EmilienMclayton, mfisch: I would be happy if you could look https://review.openstack.org/#/c/226624/ asap17:55
*** fedexo has joined #puppet-openstack17:58
*** openstackgerrit has quit IRC18:01
*** openstackgerrit has joined #puppet-openstack18:02
openstackgerritMerged openstack/puppet-keystone: wsgi: make sure keystone service is stopped before starting httpd  https://review.openstack.org/23062618:04
mfischEmilienM: I have no idea from the commit message what this is really supposed to do18:07
EmilienMwell, there is a thread about that, also a bug report and read the very last comment18:07
EmilienMmfisch: ^18:08
beddarigenerally this project isn't the greatest ever at commit messages ;)18:14
*** fedexo has quit IRC18:15
*** sergmelikyan has quit IRC18:17
*** sergmelikyan has joined #puppet-openstack18:20
*** gchamoul has quit IRC18:21
dmsimardEmilienM: FYI spotted failing CI in https://review.openstack.org/#/c/230626/, checking18:31
EmilienMdmsimard: :(18:31
EmilienMour CI is blocked again...18:31
EmilienMdmsimard: http://logs.openstack.org/34/230634/1/gate/gate-puppet-ceilometer-puppet-beaker-rspec-dsvm-centos7/ade4768/console.html#_2015-10-05_17_57_27_87918:32
dmsimardEmilienM: I didn't see that one, I was looking elsehwere, i.e http://logs.openstack.org/26/230626/1/check/gate-puppet-openstack-integration-scenario001-dsvm-centos7/89d9527/console.html#_2015-10-05_17_40_22_95518:36
EmilienMdmsimard: don't look at that18:36
dmsimardwhy not18:37
EmilienMeverything is fixed by https://review.openstack.org/23043118:37
EmilienMafter https://review.openstack.org/230431 - integration job should be more stable18:37
dmsimardEmilienM: Ok so you got a problem with just that package install failure then ?18:37
dmsimardThat looks like a general failure, even epel fails right below18:38
dmsimardCan you re-run it ?18:38
EmilienMdmsimard: I did recheck18:39
EmilienMlet's see if it works now18:39
trownby the grace of the rubygems gods https://review.openstack.org/#/c/223690/ has passed all but the centos upgrade test... is that one expected to fail?18:40
*** imcsk8|afk is now known as imcsk818:43
*** bklei has joined #puppet-openstack18:44
*** sergmelikyan has quit IRC19:03
*** rgowrishankar has joined #puppet-openstack19:12
*** rahulait has quit IRC19:14
*** sanjayu has joined #puppet-openstack19:19
*** sanjayu has quit IRC19:27
dmsimardEmilienM: did that CI come back clean ?19:30
EmilienMdmsimard: looks like19:31
dmsimard\o/19:31
dmsimardtrown: don't think that's a normal failure: http://logs.openstack.org/90/223690/26/check/gate-puppet-ironic-puppet-beaker-rspec-upgrade-dsvm-centos7/cf3be0b/console.html#_2015-10-05_18_34_31_50719:34
dmsimard(checking if it's our fault)19:34
dmsimardtrown: does that ring you a bell? http://logs.openstack.org/90/223690/26/check/gate-puppet-ironic-puppet-beaker-rspec-upgrade-dsvm-centos7/cf3be0b/console.html#_2015-10-05_18_34_26_28419:37
trowndmsimard: hmm... that would seem related to the change at least...19:39
trownEmilienM: what is the difference between the upgrade and non-upgrade acceptance jobs?19:39
EmilienMtrown: https://wiki.openstack.org/wiki/Puppet/CI19:40
trownthanks19:41
trowndmsimard: EmilienM I do not really get how the upgrade test would fail that way19:45
EmilienMtrown: have you look logs?19:46
trownfor inspector the upgrade test should be pretty much equivalent to the non-upgrade test, since there is nothing inspector related on master19:46
trownEmilienM: http://logs.openstack.org/90/223690/26/check/gate-puppet-ironic-puppet-beaker-rspec-upgrade-dsvm-centos7/cf3be0b/console.html#_2015-10-05_18_34_26_28419:46
openstackgerritMerged openstack/puppet-ceilometer: auth: drop service dependency for Keystone_user_role  https://review.openstack.org/23063419:46
trownEmilienM: it is clearly failing running ironic-inspector-dbsync... but why would it fail there and not in the non-upgrade case19:46
*** gfidente|afk has quit IRC19:46
EmilienMtrown: it's sounds like packaging stuff, isn't?19:47
trownEmilienM: but `ironic-inspector-dbsync --config-file /etc/ironic-inspector/inspector.conf upgrade` is run exactly once in both cases, why does it fail on one and not the other19:47
EmilienMtrown: I have no idea, I'm not Ironic expert19:48
EmilienMwhat I understand is, doin db_sync after a first puppet run fails19:49
EmilienMmaybe it's not your patch19:49
EmilienMhave you looked if the job is green on other patches?19:49
EmilienMwhat about ubuntu?19:49
trownubuntu does not have inspector, so it passes19:49
trownbecause the patch is a noop there19:49
openstackgerritMerged openstack/puppet-openstack-integration: Split scenarios001 in two scenarios  https://review.openstack.org/23043119:53
EmilienMcrinkle: do you see anything else I can do for https://review.openstack.org/226830 or is it fine now?19:53
*** iurygregory has left #puppet-openstack19:58
*** iurygregory has joined #puppet-openstack19:58
crinkleEmilienM: it's still rm -rf'ing stuff when it shouldn't be20:08
*** dprince has quit IRC20:09
crinkleEmilienM: hmm though i don't see that part in your patch20:09
openstackgerritClayton O'Neill proposed openstack/puppet-neutron: Set new ml2::ovs parameters to use SERVICE DEFAULT  https://review.openstack.org/23053620:10
crinkleEmilienM: it rm -rf's here http://git.openstack.org/cgit/openstack/puppet-openstack-integration/tree/install_modules.sh#n4120:11
openstackgerritClayton O'Neill proposed openstack/puppet-neutron: Set new ml2::ovs parameters to use service default  https://review.openstack.org/23053620:11
*** markvoelker has joined #puppet-openstack20:12
EmilienMcrinkle: when using fixtures, if the test pass, spec/fixtures/modules is cleaup20:15
EmilienMwith my patch, it will also be cleanup20:15
crinkleEmilienM: but if the tests don't pass they should not be cleaned up20:15
crinkleEmilienM: with your patch they are always re-cloned even if tests failed20:16
EmilienMcrinkle: spec_clean is run only if test pass, or?20:16
crinkleEmilienM: no install_modules.sh always rm -rf's everything no matter what20:16
crinkleit doesn't matter if spec_clean is run20:16
EmilienMcrinkle: yes, but we are not talking about the same rm_rf20:17
EmilienMI'm talking post-tests20:17
crinklei'm talking about this one http://git.openstack.org/cgit/openstack/puppet-openstack-integration/tree/install_modules.sh#n4120:17
openstackgerritClayton O'Neill proposed openstack/puppet-neutron: dbsync should only notify services that use the db  https://review.openstack.org/23116120:17
EmilienMinstall_modules.sh is for pre-tests20:17
openstackgerritIvan Chavero proposed openstack/puppet-neutron: Split SR-IOV configuration file into driver and agent pieces  https://review.openstack.org/22199120:17
crinkleif i run "bundle exec rake spec" it will still run spec_prep and still run install_modules.sh20:17
EmilienMcrinkle: should not we cleanup the dir *after* tests?20:18
crinkleEmilienM: it should not clean up if tests fail20:18
EmilienMcrinkle: how can we do that?20:19
crinkleand this has nothing to do with after the tests, i'm saying before tests it is running rm -rf20:19
crinkleEmilienM: i think we should not be using install_modules.sh or we should clean up that script so it doesn't mess with the unit tests20:19
crinkle1) the rm -rf is not good, 2) the puppet module list is not good, 3) the gem install r10k is not good20:20
crinklethis script isn't general enough for us to be using it in unit tests20:20
EmilienMcrinkle: we could create a new script20:21
EmilienMcrinkle: would that work for you?20:21
crinklewe should extract the common parts into a functions file20:21
EmilienMok20:21
crinkleso install_external(), install_openstack(), install_all(), and the part that does the csplit on the puppetfile would be useful in both cases20:21
EmilienMcrinkle: oki20:22
EmilienMcrinkle: i do it20:22
*** rahulait has joined #puppet-openstack20:29
*** jayg is now known as jayg|g0n320:30
mwhahahaanyone know why the beaker tests are blowing up on the <SERVICE DEFAULT> stuff when it gets moved to a fact? https://review.openstack.org/#/c/229095/ it's making it to the config as <SERVICE DEFAULT> for whatever reason20:35
mwhahahahttp://logs.openstack.org/95/229095/4/check/gate-puppet-cinder-puppet-beaker-rspec-dsvm-centos7/c6d4407/logs/cinder/api.txt.gz#_2015-10-05_15_09_36_22920:35
EmilienMoh nice20:36
mwhahahaor did we not merge that part yet20:38
EmilienMmwhahaha: https://github.com/openstack/puppet-openstacklib/commits/master20:38
mwhahahano i mean the part that handles <SERVICE DEFAULT> was that in oslo config or within openstack_config provider?20:39
mwhahahacause i'm not seeing anything in the openstack_config that specifically deals with <SERVICE DEFAULT>20:40
EmilienMmwhahaha: it does20:40
EmilienMmwhahaha: https://github.com/openstack/puppet-cinder/search?utf8=%E2%9C%93&q=%3CSERVICE+DEFAULT%3E&type=Code20:41
mwhahahai know, thats why i'm so confused20:41
mwhahahai know it's in puppet-cinder20:41
mwhahahabut i'm not seeing anything that fixes it in the confnigs20:42
mwhahahaconfigs20:42
mwhahahawhere is the part that handles the string of '<SERVICE DEFAULT>' when it gets written, was that in oslo?20:42
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Create functions for common bits  https://review.openstack.org/23117120:42
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Create install_modules_unit.sh  https://review.openstack.org/23117520:45
EmilienMcrinkle: if I drop the rm -rf from the bash script, can we keep it in Rakefile?20:46
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/22683020:48
crinkleEmilienM: then it won't work properly for the beaker tests20:48
EmilienMcrinkle: no, in Rakefile20:48
EmilienMif I keep Rakefile doing the rm_rf, but not in install_modules_unit.sh20:48
EmilienMinstall_modules.sh will be for beaker jobs20:49
EmilienMinstall_modules_unit.sh will be for unit jobs20:49
crinklethe rakefile should only be rm_rf'ing in spec_clean20:49
EmilienMok20:49
EmilienMcrinkle: but last time, I told you it will fail if you run tests and stop20:50
EmilienMif you run rake and then stop, you have to cleanup the dir20:50
EmilienMbecause nothing will do it for you otherwise20:50
crinkleEmilienM: it should be doing this: https://github.com/puppetlabs/puppetlabs_spec_helper/blob/master/lib/puppetlabs_spec_helper/rake_tasks.rb#L12320:50
EmilienMcrinkle: I don't understand what's wrong with this rm_rf20:52
EmilienMcrinkle: why do you want to git clone again?20:52
EmilienMwe had this discussion last week20:52
EmilienMand we admitted doing a ln_s was fine20:53
crinkleEmilienM: okay which rm_rf are we talking about now20:53
crinkleEmilienM: what i don't want is rm -rf modules/*20:53
EmilienMcrinkle: L7120:53
crinkleof what20:53
EmilienMrm_rf("spec/fixtures/modules/#{modname}")20:53
EmilienMof https://review.openstack.org/22683020:53
crinklei'm fine with that20:53
EmilienMcrinkle: so I guess https://review.openstack.org/226830 is fine for you now20:54
EmilienMif it pass CI and if you're fine with the deps20:54
crinklei think so20:54
EmilienMhmm wait20:54
*** angdraug has joined #puppet-openstack20:55
EmilienMcrinkle: who is in charge of cleaning spec/fixtures/modules if test pass?20:55
crinkleEmilienM: spec_clean20:55
EmilienMI dropped that code20:55
crinklewhy?20:55
EmilienMI don't know20:56
EmilienMit's a mistake of me20:56
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/22683020:57
EmilienMok, let's wait and see if CI pass20:57
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/22683021:01
mwhahahaoh i think i figured it out21:02
openstackgerritAlex Schultz proposed openstack/puppet-cinder: Switch cinder to use os_service_default fact  https://review.openstack.org/22909521:04
mwhahahalets see if that works21:04
EmilienMyeah :D21:04
openstackgerritAlex Schultz proposed openstack/puppet-cinder: Switch cinder to use os_service_default fact  https://review.openstack.org/22909521:04
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Fix README table markdown syntax  https://review.openstack.org/23118821:05
openstackgerritAlex Schultz proposed openstack/puppet-cinder: Switch cinder to use os_service_default fact  https://review.openstack.org/22909521:06
*** trown is now known as trown|outttypeww21:08
*** zigo has quit IRC21:09
*** spredzy has quit IRC21:09
*** zigo has joined #puppet-openstack21:10
openstackgerritClayton O'Neill proposed openstack/puppet-neutron: dbsync should only notify services that use the db  https://review.openstack.org/23116121:10
*** spredzy has joined #puppet-openstack21:11
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test Trove  https://review.openstack.org/22945021:14
EmilienMthere is a nice condition in puppet-cinder21:26
EmilienMcinder type-create randomly fails: http://logs.openstack.org/71/231171/1/check/gate-puppet-openstack-integration-scenario001-dsvm-trusty/58abf5c/console.html#_2015-10-05_21_06_12_88221:27
EmilienMlook at the timestamp, and look cinder api logs: http://logs.openstack.org/71/231171/1/check/gate-puppet-openstack-integration-scenario001-dsvm-trusty/58abf5c/logs/cinder/cinder-api.txt.gz#_2015-10-05_21_06_12_88221:27
*** dfisher has quit IRC21:27
EmilienMcinder-api is not finished to start but puppet runs the Exec21:28
EmilienMcrinkle: like you said, testing is useful :)21:28
*** linkedinyou has quit IRC21:30
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: type: Add retry to Exec resource  https://review.openstack.org/23119721:34
EmilienMcrinkle: https://review.openstack.org/#/c/226830/ and its deps are ready for review21:35
*** devnull0 has quit IRC21:36
*** angdraug has quit IRC22:09
*** morazi has quit IRC22:10
*** angdraug has joined #puppet-openstack22:11
*** EmilienM sets mode: +o spredzy22:14
*** openstackgerrit has quit IRC22:16
*** rcarrillocruz has quit IRC22:17
*** openstackgerrit has joined #puppet-openstack22:17
mfischEmilienM: lets talk about https://review.openstack.org/#/c/23119722:18
mfischwe should be using the healthcheck stuff here right?22:18
*** rcarrillocruz has joined #puppet-openstack22:20
*** bklei has quit IRC22:20
EmilienMmfisch: how that?22:20
mfischI think Yanis worked on this22:20
mfischhttps://github.com/puppet-community/puppet-healthcheck22:20
mfischyou can put a dep on cinder being functional before those providers run22:21
EmilienMwow, that's a new dependency22:21
mfisch(also in our talk next week)22:21
EmilienMindeed22:21
EmilienMthis is something that we could investigate22:21
EmilienMfor long term22:21
nibalizerwell note that healthcheck is super experimental22:21
EmilienMyeah, that's why I said "wow"22:22
mfischwe've been using it in prod for some time22:22
mfischour own version22:22
nibalizerbut I think openstack-puppet is on the cutting edge so if yall want to push this stuff I'm down22:22
mfischswitched to that one recently22:22
nibalizerwow22:22
mfischcome to our talk on Thursday!22:22
nibalizerso it was functionaly the same?22:22
mfischyeah22:22
nibalizeroh right are yall here for puppetconf?22:22
nibalizerwhen is openstackbeers22:22
mfischThu?22:23
mfischI'm not in until Wed22:23
EmilienMevery night?22:23
nibalizerEmilienM: my man!22:23
*** mdorman has quit IRC22:23
EmilienMmfisch: for short term, we should go ahead with https://review.openstack.org/23119722:23
mfischsure22:23
EmilienMmfisch: during the summit, we can talk about healthcheck. Can you create a topic?22:23
EmilienMhttps://etherpad.openstack.org/p/HND-puppet22:24
mfischyeah where's the link... jinx22:24
mfischnibalizer: must remember rain jacket!22:24
mfischI forget it rains in other places sometimes22:24
EmilienMis it raining in PDX ?22:24
mfischI assume always22:24
EmilienMlol22:24
nibalizerits actually super warm22:25
mfisch65 then22:25
mfischlol22:25
EmilienMI'm off for tonight, cheers22:30
bkeroactually 8122:30
bkero(27)22:30
*** angdraug has quit IRC22:44
openstackgerritIvan Chavero proposed openstack/puppet-neutron: Split SR-IOV configuration file into driver and agent pieces  https://review.openstack.org/22199122:45
*** rgowrishankar has quit IRC22:57
*** ducttape_ has quit IRC23:13
*** ducttape_ has joined #puppet-openstack23:15
*** jtomasek has quit IRC23:15
*** ducttape_ has quit IRC23:15
*** rahulait has quit IRC23:18
*** pradk has quit IRC23:30
*** angdraug has joined #puppet-openstack23:53
« Sunday, 2015-10-04 Index Tuesday, 2015-10-06 »

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