Wednesday, 2015-07-22

*** jasondotstar has quit IRC00:01
*** sergmelikyan has quit IRC00:02
*** bklei has joined #puppet-openstack00:02
*** sergmelikyan has joined #puppet-openstack00:04
*** richm has quit IRC00:05
*** jasondotstar has joined #puppet-openstack00:16
*** jasondotstar has joined #puppet-openstack00:16
*** linkedinyou has quit IRC00:17
openstackgerritGilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_trust for Keystone V3 API  https://review.openstack.org/20099600:32
*** topshare has quit IRC00:34
*** topshare has joined #puppet-openstack00:34
*** topshare has quit IRC00:36
*** sergmelikyan has quit IRC00:37
*** bklei has quit IRC00:37
*** bklei has joined #puppet-openstack00:38
*** topshare has joined #puppet-openstack00:38
*** sergmelikyan has joined #puppet-openstack00:46
*** sergmelikyan has quit IRC00:47
*** xingchao has joined #puppet-openstack00:50
*** rcallawa_ has joined #puppet-openstack00:54
*** aimon has joined #puppet-openstack00:55
*** sergmelikyan has joined #puppet-openstack00:55
*** jasondotstar has quit IRC00:56
*** sergmelikyan has quit IRC00:56
*** rcallawa has quit IRC00:58
*** serg_melikyan has joined #puppet-openstack01:00
*** xingchao has quit IRC01:02
*** bklei has quit IRC01:06
*** topshare has quit IRC01:09
*** topshare has joined #puppet-openstack01:10
*** xingchao has joined #puppet-openstack01:13
*** serg_melikyan has quit IRC01:15
*** sergmelikyan has joined #puppet-openstack01:18
*** sergmelikyan has quit IRC01:23
*** xingchao has quit IRC01:29
*** xiaohua has joined #puppet-openstack01:39
*** rcallawa_ has quit IRC01:39
*** rcallawa has joined #puppet-openstack01:39
*** rcallawa_ has joined #puppet-openstack01:42
*** sergmelikyan has joined #puppet-openstack01:44
*** rcallawa has quit IRC01:44
*** sergmelikyan has quit IRC01:46
*** rcallawa has joined #puppet-openstack01:47
*** bklei has joined #puppet-openstack01:51
*** rcallawa_ has quit IRC01:51
*** sergmelikyan has joined #puppet-openstack01:54
*** rcallawa has quit IRC01:54
*** rcallawa has joined #puppet-openstack01:55
*** sergmelikyan has quit IRC01:55
*** sergmelikyan has joined #puppet-openstack01:56
*** sergmelikyan has quit IRC01:57
*** xingchao has joined #puppet-openstack02:01
*** sergmelikyan has joined #puppet-openstack02:02
*** sergmelikyan has quit IRC02:07
*** ducttape_ has joined #puppet-openstack02:17
*** ducttape_ has quit IRC02:18
*** ericpeterson has joined #puppet-openstack02:18
*** linkedinyou has joined #puppet-openstack02:25
*** xiaohua has quit IRC02:33
*** xiaohua has joined #puppet-openstack02:40
*** bklei has quit IRC02:44
*** xiaohua has quit IRC02:47
*** saneax has quit IRC02:55
*** xiaohua has joined #puppet-openstack02:55
*** linkedinyou|2 has joined #puppet-openstack03:09
*** xingchao has quit IRC03:11
*** linkedinyou|2 has quit IRC03:11
*** linkedinyou has quit IRC03:12
*** linkedinyou has joined #puppet-openstack03:31
*** ericpeterson has quit IRC03:32
*** ducttape_ has joined #puppet-openstack03:33
*** ducttape_ has quit IRC03:33
*** saneax has joined #puppet-openstack03:34
*** gildub has quit IRC03:57
*** saneax has quit IRC04:08
*** xingchao has joined #puppet-openstack04:10
*** gildub has joined #puppet-openstack04:19
*** linkedinyou has quit IRC04:22
*** zacdev has joined #puppet-openstack04:27
*** saneax has joined #puppet-openstack04:31
*** tlbr_ has quit IRC04:31
*** tlbr has joined #puppet-openstack04:32
*** stamak has joined #puppet-openstack04:35
*** xiaohua has quit IRC04:40
*** xingchao has quit IRC04:41
*** stamak has quit IRC04:49
*** xiaohua has joined #puppet-openstack04:57
*** dalgaaf has quit IRC04:58
*** xiaohua has quit IRC05:01
*** xiaohua has joined #puppet-openstack05:13
*** xiaohua has quit IRC05:18
*** stamak has joined #puppet-openstack05:30
*** xiaohua has joined #puppet-openstack05:44
*** topshare has quit IRC05:51
*** jtomasek has joined #puppet-openstack06:12
*** xiaohua has quit IRC06:13
*** topshare has joined #puppet-openstack06:14
*** xiaohua has joined #puppet-openstack06:15
*** stamak has quit IRC06:25
*** topshare has quit IRC06:36
*** magicboiz has quit IRC06:57
*** LimorStotland has joined #puppet-openstack06:57
*** paramite has joined #puppet-openstack07:05
*** xingchao has joined #puppet-openstack07:08
*** sergmelikyan has joined #puppet-openstack07:09
openstackgerritsac proposed openstack/puppet-manila: Add support to configure GlusterFS drivers with Manila shares.  https://review.openstack.org/20081107:13
*** dgurtner has joined #puppet-openstack07:29
*** dgurtner has quit IRC07:29
*** dgurtner has joined #puppet-openstack07:29
*** xingchao has quit IRC07:29
*** xiaohua has quit IRC07:34
*** xiaohua has joined #puppet-openstack07:38
*** gfidente has joined #puppet-openstack07:42
*** topshare has joined #puppet-openstack07:47
*** topshare has quit IRC07:52
*** jistr has joined #puppet-openstack08:03
*** gildub has quit IRC08:03
*** dweaver has quit IRC08:06
*** dweaver has joined #puppet-openstack08:10
*** stamak has joined #puppet-openstack08:13
*** derekh has joined #puppet-openstack08:17
openstackgerritYanis Guenane proposed openstack/puppet-glance: Add 'glance' tag to package and services resources  https://review.openstack.org/20448908:20
*** guimaluf has quit IRC08:21
*** guimaluf has joined #puppet-openstack08:22
openstackgerritYanis Guenane proposed openstack/puppet-glance: Creation of glance::db::sync  https://review.openstack.org/19662908:26
*** stamak has quit IRC08:34
openstackgerritYanis Guenane proposed openstack/puppet-heat: Add 'heat' tag to package and service resources  https://review.openstack.org/20449708:42
openstackgerritYanis Guenane proposed openstack/puppet-heat: Creation of heat::db::sync  https://review.openstack.org/19664608:45
*** stamak has joined #puppet-openstack08:49
openstackgerritYanis Guenane proposed openstack/puppet-heat: Creation of heat::db::sync  https://review.openstack.org/19664608:49
*** xiaohua has quit IRC08:51
*** xiaohua has joined #puppet-openstack08:51
openstackgerritYanis Guenane proposed openstack/puppet-glance: Add 'glance' tag to package and service resources  https://review.openstack.org/20448908:53
*** paramite is now known as paramite|afk09:11
openstackgerritYanis Guenane proposed openstack/puppet-ironic: Add 'ironic' tag to package and service resources  https://review.openstack.org/20450909:16
*** zacdev has quit IRC09:16
openstackgerritYanis Guenane proposed openstack/puppet-ironic: Creation of ironic::db::sync  https://review.openstack.org/19665709:18
openstackgerritYanis Guenane proposed openstack/puppet-heat: Add 'heat' tag to package and service resources  https://review.openstack.org/20449709:28
*** jasondotstar has joined #puppet-openstack09:30
*** rcallawa_ has joined #puppet-openstack09:33
openstackgerritYanis Guenane proposed openstack/puppet-sahara: Add 'sahara' tag to package and service resources  https://review.openstack.org/20451709:34
*** jasondotstar has quit IRC09:35
*** mattymo_ has joined #puppet-openstack09:36
*** rcallawa has quit IRC09:36
*** xingchao has joined #puppet-openstack09:42
*** rcallawa_ has quit IRC09:46
*** rcallawa has joined #puppet-openstack09:46
openstackgerritYanis Guenane proposed openstack/puppet-sahara: Creation of sahara::db::sync  https://review.openstack.org/19668309:48
openstackgerritBartłomiej Piotrowski proposed openstack/puppet-glance: Add swift_store_region parameter in glance::backend::swift  https://review.openstack.org/19866909:49
openstackgerritYanis Guenane proposed openstack/puppet-cinder: Add 'cinder' tag to package and service resources  https://review.openstack.org/20453210:01
openstackgerritYanis Guenane proposed openstack/puppet-cinder: Sync cinder::db::sync with new standard  https://review.openstack.org/19669310:05
*** gfidente is now known as gfidente|afk10:08
*** xiaohua has quit IRC10:09
*** xiaohua has joined #puppet-openstack10:10
*** xingchao has quit IRC10:15
*** paramite|afk is now known as paramite10:18
*** xiaohua has quit IRC10:19
*** xiaohua has joined #puppet-openstack10:22
*** dgurtner has quit IRC10:43
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/20268910:43
openstackgerritMartin Mágr proposed openstack/puppet-heat: Use resource for domain  https://review.openstack.org/20454110:44
*** jasondotstar has joined #puppet-openstack10:48
*** gildub has joined #puppet-openstack11:04
*** paramite is now known as paramite|afk11:06
*** paramite|afk is now known as paramite11:09
*** dgurtner has joined #puppet-openstack11:12
*** xiaohua has quit IRC11:21
*** xiaohua has joined #puppet-openstack11:21
*** LimorStotland has quit IRC11:26
*** gildub has quit IRC11:27
*** rcallawa has quit IRC11:29
*** xiaohua has quit IRC11:30
*** LimorStotland has joined #puppet-openstack11:31
*** xiaohua has joined #puppet-openstack11:38
*** gildub has joined #puppet-openstack11:43
*** xiaohua has quit IRC11:46
*** xiaohua has joined #puppet-openstack11:46
*** dgurtner has quit IRC11:55
*** dprince has joined #puppet-openstack11:59
*** ducttape_ has joined #puppet-openstack12:12
*** rcallawa has joined #puppet-openstack12:12
*** stamak has quit IRC12:17
openstackgerritDenis Egorenko proposed openstack/puppet-glance: Add swift_store_region parameter in glance::backend::swift  https://review.openstack.org/19866912:28
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/20268912:30
*** jayg|g0n3 is now known as jayg12:34
*** prad has joined #puppet-openstack12:42
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/20268912:43
*** xiaohua has quit IRC12:45
*** LimorStotland_ has joined #puppet-openstack12:50
Mosibi_hi, ceph question: is commit b76719ef98cfec1cc17ceec68773d7a7a0079dc7 (osd.pp) tested with the 'normal' situation?12:50
Mosibi_ceph-disk list | grep ' */dev/sdk .*ceph data, active'12:51
Mosibi_this regex does not return data, the previous code, without the extra space after the disk, does work12:52
Mosibi_ ceph-disk list | grep ' */dev/sdk.*ceph data, active'12:52
Mosibi_ /dev/sdk1 ceph data, active, cluster ceph, osd.4, journal /dev/sdk212:52
*** Mosibi_ is now known as Mosibi12:52
*** LimorStotland has quit IRC12:52
openstackgerritYanis Guenane proposed openstack/puppet-ceilometer: Add 'ceilometer' tag to package and service resources  https://review.openstack.org/20459212:54
*** paramite is now known as paramite|afk12:56
*** rcallawa_ has joined #puppet-openstack12:57
*** paramite|afk is now known as paramite12:58
*** rcallawa has quit IRC12:59
*** ducttape_ has quit IRC13:00
*** stamak has joined #puppet-openstack13:00
openstackgerritYanis Guenane proposed openstack/puppet-ceilometer: Creation of ceilometer::db::sync  https://review.openstack.org/19660713:02
*** morazi has joined #puppet-openstack13:06
*** gildub has quit IRC13:08
openstackgerritYanis Guenane proposed openstack/puppet-keystone: Add 'keystone' tag to package and service resources  https://review.openstack.org/20459813:11
*** xiaohua has joined #puppet-openstack13:18
*** xingchao has joined #puppet-openstack13:21
*** saneax has quit IRC13:22
*** xiaohua has quit IRC13:22
*** xiaohua has joined #puppet-openstack13:23
*** gildub has joined #puppet-openstack13:29
*** gildub has quit IRC13:35
*** ducttape_ has joined #puppet-openstack13:40
*** ducttape_ has quit IRC13:42
*** bklei has joined #puppet-openstack13:46
*** bklei has quit IRC13:48
*** dgurtner has joined #puppet-openstack13:52
*** dgurtner has quit IRC13:52
*** dgurtner has joined #puppet-openstack13:52
*** ducttape_ has joined #puppet-openstack13:55
*** ducttape_ has quit IRC13:56
*** ducttape_ has joined #puppet-openstack13:58
*** ducttape_ has quit IRC14:00
*** dfisher has joined #puppet-openstack14:02
*** ducttape_ has joined #puppet-openstack14:04
*** ducttape_ has quit IRC14:08
*** ducttape_ has joined #puppet-openstack14:09
openstackgerritDrew Fisher proposed openstack/puppet-glance: Remove POSIX users, groups, and file modes.  https://review.openstack.org/19639014:15
*** richm has joined #puppet-openstack14:17
gchamoulsbadia: you there?14:25
*** bklei has joined #puppet-openstack14:26
mtreinishrichm: I saw your comment on https://review.openstack.org/#/c/198561/ so it's an ordering thing right?14:26
mtreinishrichm: would you be able to push up a new rev to fix things so they run in the right order?14:27
*** bklei has quit IRC14:27
*** bklei has joined #puppet-openstack14:28
gchamoulparamite: done14:28
*** saneax has joined #puppet-openstack14:32
richmmtreinish: I need help from puppet gurus14:32
sbadiagchamoul: yes!14:32
sbadia(hi here)14:32
mtreinishrichm: heh, ok. That was my feeling too, which is why I asked you :)14:32
mtreinishrichm: let's just point crinkle at it :)14:33
richmor EmilienM|off14:33
gchamoulsbadia: what do you want about https://review.openstack.org/#/c/204025/1/metadata.json,cm ?14:33
gchamoulsbadia: puppet-monasca is using the last release of jdowning/influxdb ...14:34
sbadiagchamoul: agree with matt14:34
sbadiagchamoul: just an interval (>= 1.x.x < 2.x.x) no?14:35
gchamoulsbadia: ok14:35
gchamoulsbadia: I could do that if you prefer14:36
openstackgerritYanis Guenane proposed openstack/puppet-horizon: Add 'horizon' tag to package and service resources  https://review.openstack.org/20464014:36
sbadiagchamoul: we have this « pattern » in all our module, so it's maybe a good idea to use it also here, but as you want :)14:37
gchamoulsbadia: no I will respect the convention! :-)14:38
sbadiagchamoul: héhé, thx :)14:39
openstackgerritGael Chamoulaud proposed openstack/puppet-monasca: Puppet-Monasca Project Namespace Change  https://review.openstack.org/20403214:40
openstackgerritGael Chamoulaud proposed openstack/puppet-monasca: Fix metadata.json lint issues  https://review.openstack.org/20402514:40
gchamoulsbadia: done ^^14:41
sbadiathx!!14:41
openstackgerritYanis Guenane proposed openstack/puppet-gnocchi: Add 'gnocchi' tag to package and service resources  https://review.openstack.org/20464414:41
_nickhi gchamoul - can you take a look at my PR https://review.openstack.org/#/c/202065/14:43
gchamoul_nick: sure thing!14:43
_nickyou mentioned it in another https://review.openstack.org/#/c/200877/ - the developer of that particular change has suggested rebasing on top of my proposed fix for the package names14:44
gchamoul_nick: oh yes I remember now!14:45
*** mdorman has joined #puppet-openstack14:46
_nickgchamoul: this is my first time round the whole process so not sure of the best way to proceed really - would appreciate any guidance or if there's anything in particular i should do (or not!)14:46
gchamoul_nick: sure!14:48
gchamoul_nick: well I think we don't need to check the operatingsystemrelease cause we won't deliver kilo or liberty on RHEL614:51
openstackgerritYanis Guenane proposed openstack/puppet-manila: Add 'manila' tag to package and service resources  https://review.openstack.org/20465014:52
iurygregoryhello people o/14:52
_nickgchamoul: i did wonder about that, but thought i'd play it safe with the check anyway - i'll move it out of that conditional though if that's the case14:52
gchamoul_nick: ack .. will rebase the other change on top of your once you will provide the patchset 214:54
_nickgchamoul: righto, thanks!14:54
crinklerichm: mtreinish i'm not sure it's an ordering problem, it looks like the issue is with the adminv3 user but that user has the right relationship with the admin_domain which is supposed to be default http://logs.openstack.org/61/198561/31/check/gate-puppet-keystone-puppet-beaker-rspec-dsvm-trusty/69828b4/console.html#_2015-07-20_21_18_23_03914:56
crinklerichm: mtreinish you could try printing out the keystone.conf and seeing if the default domain is what we expect14:56
iurygregoryhey sbadia do you think this is ok? (https://review.openstack.org/#/c/204252/) i'm updating the README in puppet projects with little changes14:56
mtreinishcrinkle: that's a good idea, I'll cp the /etc/keystone.conf to the logs dir at the end14:57
sbadiaiurygregory: yep, perfect :)14:58
mtreinishcrinkle: oh wait it should be logged by oslo.log because we have debug on14:58
iurygregoryit's right to use openstack/module instand of puppetlabs/module for all puppet-openstack modules?14:59
mtreinishcrinkle: yeah it looks like it's not getting set correctly: http://paste.openstack.org/show/400323/14:59
crinklehmm richm ^15:00
iurygregorysbadia,  and some projects are missing the topic Beginning with *Module" in Section Setup, but i don't know examples to put there =/15:00
sbadiaiurygregory: all 6.x modules are now on openstack namespace yes15:02
iurygregoryok sbadia , can i upgrade all README ? :P15:02
sbadiafor beginning section, it's not really an issue no?15:02
iurygregoryupdate*15:02
sbadiait's almost done by matt no?15:03
iurygregorynops its just to look fine for begginers i think15:03
sbadiaok, as you want15:03
crinklemtreinish: richm puppet claims to have changed it http://logs.openstack.org/61/198561/31/check/gate-puppet-keystone-puppet-beaker-rspec-dsvm-trusty/69828b4/console.html#_2015-07-20_21_20_03_43715:03
iurygregorythanks sbadia o/15:03
sbadianp :)15:03
crinklemtreinish: richm maybe the service isn't getting restarted15:04
mtreinishcrinkle: hmm, is keystone started at that point?15:04
mtreinishcrinkle: heh, yeah that's what I'm thinking15:04
sbadiaoh ♥ we have the EmilienM's face for the openstack namespace on the puppet forge15:04
mtreinishcrinkle: in http://logs.openstack.org/61/198561/31/check/gate-puppet-keystone-puppet-beaker-rspec-dsvm-trusty/69828b4/logs/keystone.log I only see the config file parsed once, so we're probably not restarting it15:05
iurygregorycrinkle, sorry for bother you but if you have time you can take a look at the spec Enabling Federation https://review.openstack.org/#/c/190361/ ^^15:05
*** xiaohua has quit IRC15:06
openstackgerritYanis Guenane proposed openstack/puppet-tuskar: Add 'tuskar' tag to package and service resources  https://review.openstack.org/20466015:07
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/20268915:07
crinklemtreinish: richm looks like our logic to restart the service when the  default domain changes is only in the keystone class http://git.openstack.org/cgit/openstack/puppet-keystone/tree/manifests/init.pp#n97415:07
crinklemtreinish: richm so i think you'd have to set default_domain => 'admin_domain' in the class { 'keystone': ... too15:08
crinkleor instead15:08
crinkleiurygregory: i will put it on my list for today15:09
iurygregorythanks crinkle :D15:09
openstackgerritMerged openstack/puppet-cinder: Fix rspec 3.x syntax  https://review.openstack.org/20364415:10
*** dprince has quit IRC15:12
mtreinishcrinkle: hmm I'll give that a shot, one sec15:12
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/20268915:12
openstackgerritMatthew Treinish proposed openstack/puppet-keystone: acceptance: validate Keystone with Tempest  https://review.openstack.org/19856115:16
richmcrinkle: I tried that - same result - projects/users created before default domain was set15:16
mtreinishcrinkle, richm: ^^^15:16
*** xingchao has quit IRC15:17
richmmtreinish: I think there is some confusion about the 'default_domain' parameter15:17
richmmtreinish: in class keystone, it means "this is the default domain for all v2.0 clients to use, and all v3 clients that do not specify a domain"15:18
richmmtreinish: elsewhere, it means "if I don't specify an explicit user or project domain, use this domain for both the user and the project"15:18
*** xarses_ has joined #puppet-openstack15:18
richmmtreinish: with keystone::endpoint it is the latter15:18
*** xarses_ has quit IRC15:19
richmI regret that I caused this confusion, as this probably won't be the last time we see this15:19
richmhowever, I do not think this is the cause of the original problem15:19
*** xarses_ has joined #puppet-openstack15:19
mtreinishrichm: well it seems related because the v2 thing is the original problem15:20
richmmtreinish: I would expect this to fail - class keystone will attempt to ensure the 'admin_domain' which will conflict with the definition at line 6715:20
*** xarses has quit IRC15:21
richmThe problem is that the code does not ensure that all domain scoped resources depend on the default domain being set first15:24
openstackgerritGael Chamoulaud proposed openstack/puppet-murano: Puppet-Murano Project Namespace Change  https://review.openstack.org/20466415:24
richmi.e. projects/users are allowed to be created before the default domain is set15:25
*** xarses_ is now known as xarses15:26
richmtrying with this15:27
richm    Exec['restart_keystone'] -> Keystone_user<||>15:27
richm    Exec['restart_keystone'] -> Keystone_tenant<||>15:27
richmwhen the default domain is set15:27
openstackgerritNick Jones proposed openstack/puppet-neutron: Accommodate for VPNaaS package name change in RDO  https://review.openstack.org/20206515:31
crinklerichm: maybe in addition to autorequiring Anchor['keystone_started'] the keystone_user and keystone_tenant resources should autorequire that exec http://git.openstack.org/cgit/openstack/puppet-keystone/tree/lib/puppet/type/keystone_user.rb#n10515:32
crinklei was thinking the anchor would take care of it but it probably doesn't15:32
openstackgerritYanis Guenane proposed openstack/puppet-swift: Add 'swift' tag to package and service resources  https://review.openstack.org/20466815:32
crinkleI still think default_domain should be set in the keystone class, just remove the extra keystone_domain resource from the test15:32
spredzyclayton, ping https://review.openstack.org/#/q/status:open+branch:master+topic:tag_package_and_service,n,z15:32
spredzyif I cant get your feedbacks on those, it is a first step toward what you're trying to implement in designate, without the anchor yet but at lease the tagging15:33
spredzythe db_sync reviews have been resubmitted to rely on those changes15:33
openstackgerritGael Chamoulaud proposed openstack/puppet-murano: Fix rspec 3.x syntax  https://review.openstack.org/20467115:33
*** spredzy is now known as spredzy|afk15:34
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/20268915:36
openstackgerritGael Chamoulaud proposed openstack/puppet-tuskar: Fix rspec 3.x syntax  https://review.openstack.org/20467215:37
openstackgerritNick Jones proposed openstack/puppet-neutron: Accommodate for VPNaaS package name change in RDO  https://review.openstack.org/20206515:39
*** sergmelikyan has quit IRC15:40
gchamoul_nick: cook thanks! ^^15:45
_nickgchamoul: no worries :)15:46
*** ducttape_ has quit IRC15:49
*** rcallawa_ has quit IRC15:51
*** Mosibi has quit IRC15:55
*** Mosibi has joined #puppet-openstack15:56
*** rcallawa has joined #puppet-openstack15:56
*** paramite has quit IRC15:57
*** xarses has quit IRC16:03
openstackgerritGael Chamoulaud proposed openstack/puppet-neutron: Run neutron db sync also for each neutron module  https://review.openstack.org/20087716:07
clayton@spredzy|afk: auto tagging will add the tags you've added by hand, but that's part of the problem.  You need to add a tag that autotagging cannot add, such as something with a hyphen in it.  I used designate-package for that purpose16:08
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP: Introduce swift_service type  https://review.openstack.org/20322016:09
richmcrinkle: the tempest-test patch was not rebased on the patch that added anchors16:11
richmcrinkle: mtreinish: I'm trying it with a rebase now16:11
*** aimon has quit IRC16:14
*** ducttape_ has joined #puppet-openstack16:14
*** jistr has quit IRC16:19
*** xarses has joined #puppet-openstack16:20
*** xingchao has joined #puppet-openstack16:24
*** dprince has joined #puppet-openstack16:24
*** ferest has joined #puppet-openstack16:29
Hunner"The winner of the French Scrabble championship does not speak French." http://qz.com/459634/a-non-french-speaker-just-won-the-french-scrabble-championship/16:32
dfisherto be fair, that guy wins just about every scrabble tourney16:32
*** ferest has quit IRC16:33
dfisherawesome.  i have kilo puppet configuring kilo solaris16:34
dfisherkilo all the things!16:34
dfisherman, when this stuff all works, it's amazing.16:34
*** ducttape_ has quit IRC16:35
*** ducttape_ has joined #puppet-openstack16:37
*** ducttape_ has quit IRC16:38
*** sergmelikyan has joined #puppet-openstack16:38
*** ducttape_ has joined #puppet-openstack16:39
*** sergmelikyan has quit IRC16:43
*** stamak has quit IRC16:43
*** xingchao has quit IRC16:45
*** aimon has joined #puppet-openstack16:46
*** jasondotstar has quit IRC16:50
*** spredzy|afk is now known as spredzy16:54
spredzyclayton, ack I understand better now why you had -package and -service16:55
spredzyIt seemed redundant a first, but it does make sense now16:55
*** rcallawa_ has joined #puppet-openstack16:55
*** angdraug has joined #puppet-openstack16:57
*** rcallawa has quit IRC16:58
*** derekh has quit IRC17:04
dgurtnermgagne: crinkle gchamoul EmilienM|off can I get some reviews on https://review.openstack.org/#/c/171877/ it's been around for a long time...17:11
*** jasondotstar has joined #puppet-openstack17:13
*** xiaohua has joined #puppet-openstack17:14
crinkledgurtner: +217:17
crinkledgurtner: we should see if we can come to an agreement with bogdando on that17:17
*** xiaohua has quit IRC17:19
openstackgerritMerged stackforge/puppet-ceph: Fix the two centos70 nodeset to install correct puppet  https://review.openstack.org/20402017:19
dgurtnercrinkle: thank you :)17:28
dgurtnercrinkle: sure, maybe bogdando can review current discussion and give us a +1/0/-1 based on that17:32
dgurtnercrinkle: but the concerns mentioned by mgagne on the first patchset and bogdando on the last one are basically mutually exclusive17:33
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: osd: fix pattern when matching ${data}  https://review.openstack.org/20173017:34
*** ericksonsantos has quit IRC17:42
*** dgurtner has quit IRC17:45
*** ducttape_ has quit IRC17:46
*** sergmelikyan has joined #puppet-openstack17:48
*** mdorman has quit IRC17:48
*** linkedinyou has joined #puppet-openstack17:53
*** jasondotstar has quit IRC17:57
*** sergmelikyan has quit IRC17:59
*** sergmelikyan has joined #puppet-openstack18:01
openstackgerritYanis Guenane proposed openstack/puppet-glance: Add tag to package and service resources  https://review.openstack.org/20448918:06
*** rcallawa_ has quit IRC18:06
*** rcallawa has joined #puppet-openstack18:07
*** ducttape_ has joined #puppet-openstack18:15
openstackgerritYanis Guenane proposed openstack/puppet-heat: Add tag to package and service resources  https://review.openstack.org/20449718:23
*** sergmelikyan has quit IRC18:27
openstackgerritYanis Guenane proposed openstack/puppet-ironic: Add tag to package and service resources  https://review.openstack.org/20450918:28
*** sergmelikyan has joined #puppet-openstack18:28
*** sergmelikyan has quit IRC18:29
openstackgerritYanis Guenane proposed openstack/puppet-sahara: Add tag to package and service resources  https://review.openstack.org/20451718:30
*** ericksonsantos has joined #puppet-openstack18:31
*** dgurtner has joined #puppet-openstack18:31
*** dgurtner has quit IRC18:31
*** dgurtner has joined #puppet-openstack18:31
*** jasondotstar has joined #puppet-openstack18:36
openstackgerritYanis Guenane proposed openstack/puppet-cinder: Add tag to package and service resources  https://review.openstack.org/20453218:37
*** sergmelikyan has joined #puppet-openstack18:42
richmcrinkle: mtreinish: rebasing on top of the anchor code did not help18:52
richmI got similar errors due to keystone_user and keystone_tenant resources being created before the default domain + keystone restart18:52
richmI added this to class keystone in the default domain clause18:52
richm+    Exec['restart_keystone'] -> Keystone_user<||>18:53
richm+    Exec['restart_keystone'] -> Keystone_tenant<||>18:53
crinklerichm: i think an autorequire needs to be added for the exec, the anchor doesn't relate to the exec18:53
richmand now I get a completely different error18:53
richmhttp://paste.fedoraproject.org/247106/14375912/18:55
richmIt's trying to start keystone, but keystone has already been started18:56
richmseems like it should be doing a restart18:56
richmcrinkle: an autorequire in keystone_user.rb and keystone_tenant.rb, instead of the Exec['restart_keystone'] -> Keystone_user<||> in class keystone?18:58
crinklerichm: yes, it would accomplish the same thing while avoiding collectors18:59
crinklerichm: is there more context around that error?19:00
richmlooking19:00
nibalizercould people familiar with beaker review this: https://review.openstack.org/#/c/184905/21 we're trying to get our beaker up in -infra but we're also not the best at ruby19:00
openstackgerritYanis Guenane proposed openstack/puppet-ceilometer: Add tag to package and service resources  https://review.openstack.org/20459219:01
richmcrinkle: here is the full output - https://rmeggins.fedorapeople.org/puppet-keystone-tempest-1.txt19:02
iurygregorycrinkle, do you think i need to clarify about the apache? maybe add a note?19:02
crinkleiurygregory: i would like to see more information about whether/how the puppetlabs-apache module will be used there19:04
crinklerichm: that looks like it's happening before even the package is installed?19:06
richmcrinkle: does the anchor need a dependency?19:07
iurygregorycrinkle: the keystone need to be running over apache, this is the only condition for Federation to work and some configurations need to added to keystone.conf in apache/sites-available19:07
openstackgerritYanis Guenane proposed openstack/puppet-keystone: Add tag to package and service resources  https://review.openstack.org/20459819:08
iurygregoryi'll be in a meeting right now, tonight i'll send a patch ^^thanks for the review crinkle19:09
crinkleiurygregory: i understand that, we use the puppetlabs-apache module to configure apache vhosts and mods and i'm wondering about those details19:09
richmcrinkle: iurygregory: in my poc code, I used the standard puppetlabs-apache vhosts, apache::mod, etc.19:10
richmno changes required19:10
crinklerichm: uh EmilienM|off wrote that, i'm pretty sure the require in the anchor is critical19:10
richmcrinkle: yeah, I meant perhaps an additional require is needed?19:10
crinklerichm: on the exec? maybe19:11
crinklerichm: i don't see why that service resource is getting executed first19:11
*** rcallawa_ has joined #puppet-openstack19:12
openstackgerritYanis Guenane proposed openstack/puppet-horizon: Add tag to package resource  https://review.openstack.org/20464019:14
*** rcallawa has quit IRC19:16
*** rcallawa_ has quit IRC19:17
richmcrinkle: I'm using puppet version 3.8.119:17
richmfyi19:17
openstackgerritYanis Guenane proposed openstack/puppet-gnocchi: Add tag to package and service resources  https://review.openstack.org/20464419:20
crinklerichm: http://git.openstack.org/cgit/openstack/puppet-keystone/tree/manifests/init.pp#n529 should make it impossible to execute that service resource as the first thing on a clean system19:22
crinkleso i'm out of ideas19:22
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP: Introduce swift_service class  https://review.openstack.org/20322019:25
*** sergmelikyan has quit IRC19:25
richmcrinkle: ok - I'm inclined to wait until EmilienM|off is back19:28
openstackgerritYanis Guenane proposed openstack/puppet-manila: Add tag to package and service resources  https://review.openstack.org/20465019:29
*** jasondotstar has quit IRC19:36
*** stamak has joined #puppet-openstack19:36
openstackgerritYanis Guenane proposed openstack/puppet-trove: Add tag to package and service resources  https://review.openstack.org/20465619:36
*** sergmelikyan has joined #puppet-openstack19:37
*** sergmelikyan has quit IRC19:39
*** sergmelikyan has joined #puppet-openstack19:41
openstackgerritYanis Guenane proposed openstack/puppet-tuskar: Add tag to package and service resources  https://review.openstack.org/20466019:42
openstackgerritYanis Guenane proposed openstack/puppet-swift: Add tag to package and service resources  https://review.openstack.org/20466819:52
*** sergmelikyan has quit IRC19:53
*** stamak has quit IRC20:03
gchamouldgurtner: will!20:07
*** jasondotstar has joined #puppet-openstack20:12
*** xiaohua has joined #puppet-openstack20:16
openstackgerritYanis Guenane proposed openstack/puppet-monasca: Add tag to package and service resources  https://review.openstack.org/20474620:19
*** xiaohua has quit IRC20:20
*** rcallawa has joined #puppet-openstack20:21
*** rcallawa_ has joined #puppet-openstack20:22
*** LimorStotland_ has quit IRC20:24
*** rcallawa has quit IRC20:26
*** gildub has joined #puppet-openstack20:30
*** aimon has quit IRC20:32
*** jasondotstar has quit IRC20:38
*** jayg is now known as jayg|g0n320:39
*** ducttape_ has quit IRC20:39
*** gildub has quit IRC20:40
*** jeblair is now known as CarolChanning20:45
*** CarolChanning is now known as jeblair20:45
*** linkedinyou has quit IRC20:45
*** linkedinyou has joined #puppet-openstack20:46
*** ducttape_ has joined #puppet-openstack20:46
openstackgerritYanis Guenane proposed openstack/puppet-nova: Add tag to package and service resources  https://review.openstack.org/20475620:48
spredzyclayton, around ?20:52
claytonfor a bit20:52
spredzyI had a question about neutron and tagging20:52
spredzyshould only the neutron and neutron-server be tagged20:53
*** linkedinyou has quit IRC20:53
spredzyor should also the agent and plugins packages be tagged ?20:53
claytonI think neutron-common should be, and anything that requires neutron-common20:53
spredzyclayton, ack20:54
claytonon ubuntu the migrations are contained in python-neutron, but nothing requires that directly, all the packages depend on neutron-common which pulls in python-neutron20:54
claytonI'm not sure about redhat20:55
spredzyCan I ask you to look at any of https://review.openstack.org/#/q/status:open+branch:master+topic:tag_package_and_service,n,z and let me know if the pattern looks good to you now20:55
claytoncan do20:55
spredzythanks20:56
*** sergmelikyan has joined #puppet-openstack20:56
*** bklei_ has joined #puppet-openstack20:56
*** dfisher has quit IRC20:58
claytonlooks good, going through them21:00
*** bklei has quit IRC21:00
spredzyclayton, ack thanks.21:00
claytonthanks for the patches :)21:00
spredzyyw, it was blocking my db_sync reviews :p21:00
*** sergmelikyan has quit IRC21:04
*** aimon has joined #puppet-openstack21:05
*** sergmelikyan has joined #puppet-openstack21:06
dgurtnerspredzy: on rhel openstack-neutron pulls in python-neutron (which contains the migrations, same as ubuntu)21:08
openstackgerritYanis Guenane proposed openstack/puppet-neutron: Add tag to package and service resources  https://review.openstack.org/20476421:09
mfischgchamoul: you here?21:09
mfischvous et ici maintenant?21:10
mfischor is it vous est21:10
spredzydgurtner, thanks for the clarification.21:10
spredzymfisch, Etes vous ici ?21:10
mfischetes!21:10
mfischthanks21:10
mfischshould stick with es tu21:10
spredzyhehe21:10
dgurtnerspredzy: yw21:11
spredzyyeah vous is too formal21:11
mfischsbadia: a quick question on https://review.openstack.org/#/c/204025 if you are around, where did the version requirement maximums come from, do you know?21:11
mfischspredzy: better to be formal than piss someone off ;)21:11
spredzymfisch, true. But its impossible to piss gchamoul off so its riskfree here :D21:12
mfischIts not like I could make you guys think Americans were any less cultured anyway21:12
spredzymfisch, ahaha we have no judgement over Americans :)21:16
openstackgerritMerged openstack/puppet-tuskar: Fix rspec 3.x syntax  https://review.openstack.org/20467221:19
spredzyOk I am out. Ttyl people21:20
*** ericpeterson has joined #puppet-openstack21:22
*** ducttape_ has quit IRC21:25
*** ericpeterson has quit IRC21:27
*** aimon has quit IRC21:29
*** linkedinyou has joined #puppet-openstack21:30
openstackgerritYanis Guenane proposed openstack/puppet-swift: Add tag to package and service resources  https://review.openstack.org/20466821:32
*** xarses has quit IRC21:32
*** sergmelikyan has quit IRC21:39
*** aimon has joined #puppet-openstack21:47
*** sergmelikyan has joined #puppet-openstack21:53
*** jasondotstar has joined #puppet-openstack22:01
*** sergmelikyan has quit IRC22:07
*** sergmelikyan has joined #puppet-openstack22:10
*** jasondotstar has quit IRC22:18
*** jasondotstar has joined #puppet-openstack22:27
*** aimon has quit IRC22:30
*** dprince has quit IRC22:31
*** morazi has quit IRC22:33
*** jasondotstar has quit IRC22:33
*** sergmelikyan has quit IRC22:43
*** sergmelikyan has joined #puppet-openstack22:51
*** dgurtner has quit IRC22:52
*** prad has quit IRC22:53
*** xiaohua has joined #puppet-openstack22:56
openstackgerritSergii Golovatiuk proposed openstack/puppet-glance: Add swift_store_region parameter to glance::backend::swift  https://review.openstack.org/19866922:59
*** xiaohua has quit IRC23:01
*** sergmelikyan has quit IRC23:08
*** linkedinyou has quit IRC23:10
*** xingchao has joined #puppet-openstack23:16
*** bklei_ has quit IRC23:20
*** bklei has joined #puppet-openstack23:21
*** bklei has quit IRC23:24
*** jasondotstar has joined #puppet-openstack23:25
openstackgerritMerged openstack/puppet-murano: Puppet-Murano Project Namespace Change  https://review.openstack.org/20466423:26
*** sergmelikyan has joined #puppet-openstack23:28
*** sergmelikyan has quit IRC23:30
*** gildub has joined #puppet-openstack23:35
openstackgerritSaksham Varma proposed openstack/puppet-neutron: N1Kv: Adding nexus1000v puppet class  https://review.openstack.org/20479023:35
*** gfidente|afk has quit IRC23:37
*** aimon has joined #puppet-openstack23:40
*** xarses has joined #puppet-openstack23:45
*** gildub has quit IRC23:54
*** sergmelikyan has joined #puppet-openstack23:58

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