Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-07-22

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

All times shown according to UTC.

Time Nick Message
00:02 bklei joined #puppet-openstack
00:04 sergmelikyan joined #puppet-openstack
00:16 jasondotstar joined #puppet-openstack
00:16 jasondotstar joined #puppet-openstack
00:32 openstackgerrit Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_trust for Keystone V3 API  https://review.openstack.org/200996
00:34 topshare joined #puppet-openstack
00:38 bklei joined #puppet-openstack
00:38 topshare joined #puppet-openstack
00:46 sergmelikyan joined #puppet-openstack
00:50 xingchao joined #puppet-openstack
00:54 rcallawa_ joined #puppet-openstack
00:55 aimon joined #puppet-openstack
00:55 sergmelikyan joined #puppet-openstack
01:00 serg_melikyan joined #puppet-openstack
01:10 topshare joined #puppet-openstack
01:13 xingchao joined #puppet-openstack
01:18 sergmelikyan joined #puppet-openstack
01:39 xiaohua joined #puppet-openstack
01:39 rcallawa joined #puppet-openstack
01:45 rcallawa_ joined #puppet-openstack
01:45 sergmelikyan joined #puppet-openstack
01:47 rcallawa joined #puppet-openstack
01:50 bklei joined #puppet-openstack
01:54 sergmelikyan joined #puppet-openstack
01:55 rcallawa joined #puppet-openstack
01:55 sergmelikyan joined #puppet-openstack
02:01 xingchao joined #puppet-openstack
02:02 sergmelikyan joined #puppet-openstack
02:17 ducttape_ joined #puppet-openstack
02:18 ericpeterson joined #puppet-openstack
02:25 linkedinyou joined #puppet-openstack
02:40 xiaohua joined #puppet-openstack
02:55 xiaohua joined #puppet-openstack
03:09 linkedinyou|2 joined #puppet-openstack
03:31 linkedinyou joined #puppet-openstack
03:33 ducttape_ joined #puppet-openstack
03:34 saneax joined #puppet-openstack
04:10 xingchao joined #puppet-openstack
04:19 gildub joined #puppet-openstack
04:27 zacdev joined #puppet-openstack
04:31 saneax joined #puppet-openstack
04:31 tlbr joined #puppet-openstack
04:35 stamak joined #puppet-openstack
04:57 xiaohua joined #puppet-openstack
05:13 xiaohua joined #puppet-openstack
05:30 stamak joined #puppet-openstack
05:44 xiaohua joined #puppet-openstack
06:12 jtomasek joined #puppet-openstack
06:14 topshare joined #puppet-openstack
06:15 xiaohua joined #puppet-openstack
06:57 LimorStotland joined #puppet-openstack
07:05 paramite joined #puppet-openstack
07:08 xingchao joined #puppet-openstack
07:09 sergmelikyan joined #puppet-openstack
07:13 openstackgerrit sac proposed openstack/puppet-manila: Add support to configure GlusterFS drivers with Manila shares.  https://review.openstack.org/200811
07:29 dgurtner joined #puppet-openstack
07:29 dgurtner joined #puppet-openstack
07:38 xiaohua joined #puppet-openstack
07:42 gfidente joined #puppet-openstack
07:47 topshare joined #puppet-openstack
08:03 jistr joined #puppet-openstack
08:10 dweaver joined #puppet-openstack
08:13 stamak joined #puppet-openstack
08:17 derekh joined #puppet-openstack
08:20 openstackgerrit Yanis Guenane proposed openstack/puppet-glance: Add 'glance' tag to package and services resources  https://review.openstack.org/204489
08:22 guimaluf joined #puppet-openstack
08:26 openstackgerrit Yanis Guenane proposed openstack/puppet-glance: Creation of glance::db::sync  https://review.openstack.org/196629
08:42 openstackgerrit Yanis Guenane proposed openstack/puppet-heat: Add 'heat' tag to package and service resources  https://review.openstack.org/204497
08:45 openstackgerrit Yanis Guenane proposed openstack/puppet-heat: Creation of heat::db::sync  https://review.openstack.org/196646
08:49 stamak joined #puppet-openstack
08:49 openstackgerrit Yanis Guenane proposed openstack/puppet-heat: Creation of heat::db::sync  https://review.openstack.org/196646
08:51 xiaohua joined #puppet-openstack
08:53 openstackgerrit Yanis Guenane proposed openstack/puppet-glance: Add 'glance' tag to package and service resources  https://review.openstack.org/204489
09:16 openstackgerrit Yanis Guenane proposed openstack/puppet-ironic: Add 'ironic' tag to package and service resources  https://review.openstack.org/204509
09:18 openstackgerrit Yanis Guenane proposed openstack/puppet-ironic: Creation of ironic::db::sync  https://review.openstack.org/196657
09:28 openstackgerrit Yanis Guenane proposed openstack/puppet-heat: Add 'heat' tag to package and service resources  https://review.openstack.org/204497
09:30 jasondotstar joined #puppet-openstack
09:33 rcallawa_ joined #puppet-openstack
09:34 openstackgerrit Yanis Guenane proposed openstack/puppet-sahara: Add 'sahara' tag to package and service resources  https://review.openstack.org/204517
09:36 mattymo_ joined #puppet-openstack
09:42 xingchao joined #puppet-openstack
09:46 rcallawa joined #puppet-openstack
09:48 openstackgerrit Yanis Guenane proposed openstack/puppet-sahara: Creation of sahara::db::sync  https://review.openstack.org/196683
09:49 openstackgerrit Bartłomiej Piotrowski proposed openstack/puppet-glance: Add swift_store_region parameter in glance::backend::swift  https://review.openstack.org/198669
10:01 openstackgerrit Yanis Guenane proposed openstack/puppet-cinder: Add 'cinder' tag to package and service resources  https://review.openstack.org/204532
10:05 openstackgerrit Yanis Guenane proposed openstack/puppet-cinder: Sync cinder::db::sync with new standard  https://review.openstack.org/196693
10:10 xiaohua joined #puppet-openstack
10:22 xiaohua joined #puppet-openstack
10:43 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/202689
10:44 openstackgerrit Martin Mágr proposed openstack/puppet-heat: Use resource for domain  https://review.openstack.org/204541
10:48 jasondotstar joined #puppet-openstack
11:04 gildub joined #puppet-openstack
11:12 dgurtner joined #puppet-openstack
11:21 xiaohua joined #puppet-openstack
11:31 LimorStotland joined #puppet-openstack
11:38 xiaohua joined #puppet-openstack
11:43 gildub joined #puppet-openstack
11:46 xiaohua joined #puppet-openstack
11:59 dprince joined #puppet-openstack
12:12 ducttape_ joined #puppet-openstack
12:12 rcallawa joined #puppet-openstack
12:28 openstackgerrit Denis Egorenko proposed openstack/puppet-glance: Add swift_store_region parameter in glance::backend::swift  https://review.openstack.org/198669
12:30 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/202689
12:42 prad joined #puppet-openstack
12:43 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/202689
12:50 LimorStotland_ joined #puppet-openstack
12:50 Mosibi_ hi, ceph question: is commit b76719ef98cfec1cc17ceec68773d7a7a0079dc7 (osd.pp) tested with the 'normal' situation?
12:51 Mosibi_ ceph-disk list | grep ' */dev/sdk .*ceph data, active'
12:52 Mosibi_ this regex does not return data, the previous code, without the extra space after the disk, does work
12: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/sdk2
12:54 openstackgerrit Yanis Guenane proposed openstack/puppet-ceilometer: Add 'ceilometer' tag to package and service resources  https://review.openstack.org/204592
12:57 rcallawa_ joined #puppet-openstack
13:00 stamak joined #puppet-openstack
13:02 openstackgerrit Yanis Guenane proposed openstack/puppet-ceilometer: Creation of ceilometer::db::sync  https://review.openstack.org/196607
13:06 morazi joined #puppet-openstack
13:10 openstackgerrit Yanis Guenane proposed openstack/puppet-keystone: Add 'keystone' tag to package and service resources  https://review.openstack.org/204598
13:18 xiaohua joined #puppet-openstack
13:21 xingchao joined #puppet-openstack
13:23 xiaohua joined #puppet-openstack
13:29 gildub joined #puppet-openstack
13:39 ducttape_ joined #puppet-openstack
13:46 bklei joined #puppet-openstack
13:52 dgurtner joined #puppet-openstack
13:52 dgurtner joined #puppet-openstack
13:55 ducttape_ joined #puppet-openstack
13:58 ducttape_ joined #puppet-openstack
14:02 dfisher joined #puppet-openstack
14:04 ducttape_ joined #puppet-openstack
14:09 ducttape_ joined #puppet-openstack
14:15 openstackgerrit Drew Fisher proposed openstack/puppet-glance: Remove POSIX users, groups, and file modes.  https://review.openstack.org/196390
14:17 richm joined #puppet-openstack
14:25 gchamoul sbadia: you there?
14:26 bklei joined #puppet-openstack
14:26 mtreinish richm: I saw your comment on https://review.openstack.org/#/c/198561/ so it's an ordering thing right?
14:27 mtreinish richm: would you be able to push up a new rev to fix things so they run in the right order?
14:28 bklei joined #puppet-openstack
14:28 gchamoul paramite: done
14:32 saneax joined #puppet-openstack
14:32 richm mtreinish: I need help from puppet gurus
14:32 sbadia gchamoul: yes!
14:32 sbadia (hi here)
14:32 mtreinish richm: heh, ok. That was my feeling too, which is why I asked you :)
14:33 mtreinish richm: let's just point crinkle at it :)
14:33 richm or EmilienM|off
14:33 gchamoul sbadia: what do you want about https://review.openstack.org/#/c/204025/1/metadata.json,cm ?
14:34 gchamoul sbadia: puppet-monasca is using the last release of jdowning/influxdb ...
14:34 sbadia gchamoul: agree with matt
14:35 sbadia gchamoul: just an interval (>= 1.x.x < 2.x.x) no?
14:35 gchamoul sbadia: ok
14:35 gchamoul sbadia: I could do that if you prefer
14:36 openstackgerrit Yanis Guenane proposed openstack/puppet-horizon: Add 'horizon' tag to package and service resources  https://review.openstack.org/204640
14:37 sbadia gchamoul: 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:38 gchamoul sbadia: no I will respect the convention! :-)
14:39 sbadia gchamoul: héhé, thx :)
14:40 openstackgerrit Gael Chamoulaud proposed openstack/puppet-monasca: Puppet-Monasca Project Namespace Change  https://review.openstack.org/204032
14:40 openstackgerrit Gael Chamoulaud proposed openstack/puppet-monasca: Fix metadata.json lint issues  https://review.openstack.org/204025
14:41 gchamoul sbadia: done ^^
14:41 sbadia thx!!
14:41 openstackgerrit Yanis Guenane proposed openstack/puppet-gnocchi: Add 'gnocchi' tag to package and service resources  https://review.openstack.org/204644
14:43 _nick hi gchamoul - can you take a look at my PR https://review.openstack.org/#/c/202065/
14:43 gchamoul _nick: sure thing!
14:44 _nick you 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 names
14:45 gchamoul _nick: oh yes I remember now!
14:46 mdorman joined #puppet-openstack
14:46 _nick gchamoul: 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:48 gchamoul _nick: sure!
14:51 gchamoul _nick: well I think we don't need to check the operatingsystemrelease cause we won't deliver kilo or liberty on RHEL6
14:52 openstackgerrit Yanis Guenane proposed openstack/puppet-manila: Add 'manila' tag to package and service resources  https://review.openstack.org/204650
14:52 iurygregory hello people o/
14:52 _nick gchamoul: 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 case
14:54 gchamoul _nick: ack .. will rebase the other change on top of your once you will provide the patchset 2
14:54 _nick gchamoul: righto, thanks!
14:55 crinkle richm: 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/gat​e-puppet-keystone-puppet-beaker-rspec-dsvm-trust​y/69828b4/console.html#_2015-07-20_21_18_23_039
14:56 crinkle richm: mtreinish you could try printing out the keystone.conf and seeing if the default domain is what we expect
14:56 iurygregory hey sbadia do you think this is ok? (https://review.openstack.org/#/c/204252/) i'm updating the README in puppet projects with little changes
14:57 mtreinish crinkle: that's a good idea, I'll cp the /etc/keystone.conf to the logs dir at the end
14:57 sbadia iurygregory: yep, perfect :)
14:58 mtreinish crinkle: oh wait it should be logged by oslo.log because we have debug on
14:59 iurygregory it's right to use openstack/module instand of puppetlabs/module for all puppet-openstack modules?
14:59 mtreinish crinkle: yeah it looks like it's not getting set correctly: http://paste.openstack.org/show/400323/
15:00 crinkle hmm richm ^
15:00 iurygregory sbadia,  and some projects are missing the topic Beginning with *Module" in Section Setup, but i don't know examples to put there =/
15:02 sbadia iurygregory: all 6.x modules are now on openstack namespace yes
15:02 iurygregory ok sbadia , can i upgrade all README ? :P
15:02 sbadia for beginning section, it's not really an issue no?
15:02 iurygregory update*
15:03 sbadia it's almost done by matt no?
15:03 iurygregory nops its just to look fine for begginers i think
15:03 sbadia ok, as you want
15:03 crinkle mtreinish: richm puppet claims to have changed it http://logs.openstack.org/61/198561/31/check/gat​e-puppet-keystone-puppet-beaker-rspec-dsvm-trust​y/69828b4/console.html#_2015-07-20_21_20_03_437
15:03 iurygregory thanks sbadia o/
15:04 sbadia np :)
15:04 crinkle mtreinish: richm maybe the service isn't getting restarted
15:04 mtreinish crinkle: hmm, is keystone started at that point?
15:04 mtreinish crinkle: heh, yeah that's what I'm thinking
15:04 sbadia oh ♥ we have the EmilienM's face for the openstack namespace on the puppet forge
15:05 mtreinish crinkle: in http://logs.openstack.org/61/198561/31/ch​eck/gate-puppet-keystone-puppet-beaker-rs​pec-dsvm-trusty/69828b4/logs/keystone.log I only see the config file parsed once, so we're probably not restarting it
15:05 iurygregory crinkle, 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:07 openstackgerrit Yanis Guenane proposed openstack/puppet-tuskar: Add 'tuskar' tag to package and service resources  https://review.openstack.org/204660
15:07 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/202689
15:07 crinkle mtreinish: 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/pu​ppet-keystone/tree/manifests/init.pp#n974
15:08 crinkle mtreinish: richm so i think you'd have to set default_domain => 'admin_domain' in the class { 'keystone': ... too
15:08 crinkle or instead
15:09 crinkle iurygregory: i will put it on my list for today
15:09 iurygregory thanks crinkle :D
15:10 openstackgerrit Merged openstack/puppet-cinder: Fix rspec 3.x syntax  https://review.openstack.org/203644
15:12 mtreinish crinkle: hmm I'll give that a shot, one sec
15:12 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/202689
15:16 openstackgerrit Matthew Treinish proposed openstack/puppet-keystone: acceptance: validate Keystone with Tempest  https://review.openstack.org/198561
15:16 richm crinkle: I tried that - same result - projects/users created before default domain was set
15:16 mtreinish crinkle, richm: ^^^
15:17 richm mtreinish: I think there is some confusion about the 'default_domain' parameter
15:18 richm mtreinish: 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 richm mtreinish: 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_ joined #puppet-openstack
15:18 richm mtreinish: with keystone::endpoint it is the latter
15:19 richm I regret that I caused this confusion, as this probably won't be the last time we see this
15:19 richm however, I do not think this is the cause of the original problem
15:19 xarses_ joined #puppet-openstack
15:20 mtreinish richm: well it seems related because the v2 thing is the original problem
15:20 richm mtreinish: I would expect this to fail - class keystone will attempt to ensure the 'admin_domain' which will conflict with the definition at line 67
15:24 richm The problem is that the code does not ensure that all domain scoped resources depend on the default domain being set first
15:24 openstackgerrit Gael Chamoulaud proposed openstack/puppet-murano: Puppet-Murano Project Namespace Change  https://review.openstack.org/204664
15:25 richm i.e. projects/users are allowed to be created before the default domain is set
15:27 richm trying with this
15:27 richm Exec['restart_keystone'] -> Keystone_user<||>
15:27 richm Exec['restart_keystone'] -> Keystone_tenant<||>
15:27 richm when the default domain is set
15:31 openstackgerrit Nick Jones proposed openstack/puppet-neutron: Accommodate for VPNaaS package name change in RDO  https://review.openstack.org/202065
15:32 crinkle richm: 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-ke​ystone/tree/lib/puppet/type/keystone_user.rb#n105
15:32 crinkle i was thinking the anchor would take care of it but it probably doesn't
15:32 openstackgerrit Yanis Guenane proposed openstack/puppet-swift: Add 'swift' tag to package and service resources  https://review.openstack.org/204668
15:32 crinkle I still think default_domain should be set in the keystone class, just remove the extra keystone_domain resource from the test
15:32 spredzy clayton, ping https://review.openstack.org/#/q/status:open​+branch:master+topic:tag_package_and_service,n,z
15:33 spredzy if 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 tagging
15:33 spredzy the db_sync reviews have been resubmitted to rely on those changes
15:33 openstackgerrit Gael Chamoulaud proposed openstack/puppet-murano: Fix rspec 3.x syntax  https://review.openstack.org/204671
15:36 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/202689
15:37 openstackgerrit Gael Chamoulaud proposed openstack/puppet-tuskar: Fix rspec 3.x syntax  https://review.openstack.org/204672
15:39 openstackgerrit Nick Jones proposed openstack/puppet-neutron: Accommodate for VPNaaS package name change in RDO  https://review.openstack.org/202065
15:45 gchamoul _nick: cook thanks! ^^
15:46 _nick gchamoul: no worries :)
15:56 Mosibi joined #puppet-openstack
15:56 rcallawa joined #puppet-openstack
16:07 openstackgerrit Gael Chamoulaud proposed openstack/puppet-neutron: Run neutron db sync also for each neutron module  https://review.openstack.org/200877
16: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 purpose
16:09 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: WIP: Introduce swift_service type  https://review.openstack.org/203220
16:11 richm crinkle: the tempest-test patch was not rebased on the patch that added anchors
16:11 richm crinkle: mtreinish: I'm trying it with a rebase now
16:14 ducttape_ joined #puppet-openstack
16:20 xarses joined #puppet-openstack
16:24 xingchao joined #puppet-openstack
16:24 dprince joined #puppet-openstack
16:29 ferest joined #puppet-openstack
16:32 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 dfisher to be fair, that guy wins just about every scrabble tourney
16:34 dfisher awesome.  i have kilo puppet configuring kilo solaris
16:34 dfisher kilo all the things!
16:34 dfisher man, when this stuff all works, it's amazing.
16:37 ducttape_ joined #puppet-openstack
16:38 sergmelikyan joined #puppet-openstack
16:39 ducttape_ joined #puppet-openstack
16:46 aimon joined #puppet-openstack
16:55 spredzy clayton, ack I understand better now why you had -package and -service
16:55 spredzy It seemed redundant a first, but it does make sense now
16:55 rcallawa_ joined #puppet-openstack
16:57 angdraug joined #puppet-openstack
17:11 dgurtner mgagne: 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:13 jasondotstar joined #puppet-openstack
17:14 xiaohua joined #puppet-openstack
17:17 crinkle dgurtner: +2
17:17 crinkle dgurtner: we should see if we can come to an agreement with bogdando on that
17:19 openstackgerrit Merged stackforge/puppet-ceph: Fix the two centos70 nodeset to install correct puppet  https://review.openstack.org/204020
17:28 dgurtner crinkle: thank you :)
17:32 dgurtner crinkle: sure, maybe bogdando can review current discussion and give us a +1/0/-1 based on that
17:33 dgurtner crinkle: but the concerns mentioned by mgagne on the first patchset and bogdando on the last one are basically mutually exclusive
17:34 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: osd: fix pattern when matching ${data}  https://review.openstack.org/201730
17:48 sergmelikyan joined #puppet-openstack
17:53 linkedinyou joined #puppet-openstack
18:01 sergmelikyan joined #puppet-openstack
18:06 openstackgerrit Yanis Guenane proposed openstack/puppet-glance: Add tag to package and service resources  https://review.openstack.org/204489
18:07 rcallawa joined #puppet-openstack
18:15 ducttape_ joined #puppet-openstack
18:23 openstackgerrit Yanis Guenane proposed openstack/puppet-heat: Add tag to package and service resources  https://review.openstack.org/204497
18:28 openstackgerrit Yanis Guenane proposed openstack/puppet-ironic: Add tag to package and service resources  https://review.openstack.org/204509
18:28 sergmelikyan joined #puppet-openstack
18:30 openstackgerrit Yanis Guenane proposed openstack/puppet-sahara: Add tag to package and service resources  https://review.openstack.org/204517
18:31 ericksonsantos joined #puppet-openstack
18:31 dgurtner joined #puppet-openstack
18:31 dgurtner joined #puppet-openstack
18:36 jasondotstar joined #puppet-openstack
18:37 openstackgerrit Yanis Guenane proposed openstack/puppet-cinder: Add tag to package and service resources  https://review.openstack.org/204532
18:42 sergmelikyan joined #puppet-openstack
18:52 richm crinkle: mtreinish: rebasing on top of the anchor code did not help
18:52 richm I got similar errors due to keystone_user and keystone_tenant resources being created before the default domain + keystone restart
18:52 richm I added this to class keystone in the default domain clause
18:53 richm +    Exec['restart_keystone'] -> Keystone_user<||>
18:53 richm +    Exec['restart_keystone'] -> Keystone_tenant<||>
18:53 crinkle richm: i think an autorequire needs to be added for the exec, the anchor doesn't relate to the exec
18:53 richm and now I get a completely different error
18:55 richm http://paste.fedoraproject.org/247106/14375912/
18:56 richm It's trying to start keystone, but keystone has already been started
18:56 richm seems like it should be doing a restart
18:58 richm crinkle: an autorequire in keystone_user.rb and keystone_tenant.rb, instead of the Exec['restart_keystone'] -> Keystone_user<||> in class keystone?
18:59 crinkle richm: yes, it would accomplish the same thing while avoiding collectors
19:00 crinkle richm: is there more context around that error?
19:00 richm looking
19:00 nibalizer could 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 ruby
19:01 openstackgerrit Yanis Guenane proposed openstack/puppet-ceilometer: Add tag to package and service resources  https://review.openstack.org/204592
19:02 richm crinkle: here is the full output - https://rmeggins.fedorapeople.or​g/puppet-keystone-tempest-1.txt
19:02 iurygregory crinkle, do you think i need to clarify about the apache? maybe add a note?
19:04 crinkle iurygregory: i would like to see more information about whether/how the puppetlabs-apache module will be used there
19:06 crinkle richm: that looks like it's happening before even the package is installed?
19:07 richm crinkle: does the anchor need a dependency?
19:07 iurygregory crinkle: 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-available
19:08 openstackgerrit Yanis Guenane proposed openstack/puppet-keystone: Add tag to package and service resources  https://review.openstack.org/204598
19:09 iurygregory i'll be in a meeting right now, tonight i'll send a patch ^^thanks for the review crinkle
19:09 crinkle iurygregory: i understand that, we use the puppetlabs-apache module to configure apache vhosts and mods and i'm wondering about those details
19:10 richm crinkle: iurygregory: in my poc code, I used the standard puppetlabs-apache vhosts, apache::mod, etc.
19:10 richm no changes required
19:10 crinkle richm: uh EmilienM|off wrote that, i'm pretty sure the require in the anchor is critical
19:10 richm crinkle: yeah, I meant perhaps an additional require is needed?
19:11 crinkle richm: on the exec? maybe
19:11 crinkle richm: i don't see why that service resource is getting executed first
19:12 rcallawa_ joined #puppet-openstack
19:14 openstackgerrit Yanis Guenane proposed openstack/puppet-horizon: Add tag to package resource  https://review.openstack.org/204640
19:17 richm crinkle: I'm using puppet version 3.8.1
19:17 richm fyi
19:20 openstackgerrit Yanis Guenane proposed openstack/puppet-gnocchi: Add tag to package and service resources  https://review.openstack.org/204644
19:22 crinkle richm: http://git.openstack.org/cgit/openstack/pu​ppet-keystone/tree/manifests/init.pp#n529 should make it impossible to execute that service resource as the first thing on a clean system
19:22 crinkle so i'm out of ideas
19:25 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: WIP: Introduce swift_service class  https://review.openstack.org/203220
19:28 richm crinkle: ok - I'm inclined to wait until EmilienM|off is back
19:29 openstackgerrit Yanis Guenane proposed openstack/puppet-manila: Add tag to package and service resources  https://review.openstack.org/204650
19:36 stamak joined #puppet-openstack
19:36 openstackgerrit Yanis Guenane proposed openstack/puppet-trove: Add tag to package and service resources  https://review.openstack.org/204656
19:37 sergmelikyan joined #puppet-openstack
19:41 sergmelikyan joined #puppet-openstack
19:42 openstackgerrit Yanis Guenane proposed openstack/puppet-tuskar: Add tag to package and service resources  https://review.openstack.org/204660
19:52 openstackgerrit Yanis Guenane proposed openstack/puppet-swift: Add tag to package and service resources  https://review.openstack.org/204668
20:07 gchamoul dgurtner: will!
20:12 jasondotstar joined #puppet-openstack
20:16 xiaohua joined #puppet-openstack
20:19 openstackgerrit Yanis Guenane proposed openstack/puppet-monasca: Add tag to package and service resources  https://review.openstack.org/204746
20:21 rcallawa joined #puppet-openstack
20:22 rcallawa_ joined #puppet-openstack
20:30 gildub joined #puppet-openstack
20:46 linkedinyou joined #puppet-openstack
20:46 ducttape_ joined #puppet-openstack
20:48 openstackgerrit Yanis Guenane proposed openstack/puppet-nova: Add tag to package and service resources  https://review.openstack.org/204756
20:52 spredzy clayton, around ?
20:52 clayton for a bit
20:52 spredzy I had a question about neutron and tagging
20:53 spredzy should only the neutron and neutron-server be tagged
20:53 spredzy or should also the agent and plugins packages be tagged ?
20:53 clayton I think neutron-common should be, and anything that requires neutron-common
20:54 spredzy clayton, ack
20:54 clayton on ubuntu the migrations are contained in python-neutron, but nothing requires that directly, all the packages depend on neutron-common which pulls in python-neutron
20:55 clayton I'm not sure about redhat
20:55 spredzy Can 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 now
20:55 clayton can do
20:56 spredzy thanks
20:56 sergmelikyan joined #puppet-openstack
20:56 bklei_ joined #puppet-openstack
21:00 clayton looks good, going through them
21:00 spredzy clayton, ack thanks.
21:00 clayton thanks for the patches :)
21:00 spredzy yw, it was blocking my db_sync reviews :p
21:05 aimon joined #puppet-openstack
21:06 sergmelikyan joined #puppet-openstack
21:08 dgurtner spredzy: on rhel openstack-neutron pulls in python-neutron (which contains the migrations, same as ubuntu)
21:09 openstackgerrit Yanis Guenane proposed openstack/puppet-neutron: Add tag to package and service resources  https://review.openstack.org/204764
21:09 mfisch gchamoul: you here?
21:10 mfisch vous et ici maintenant?
21:10 mfisch or is it vous est
21:10 spredzy dgurtner, thanks for the clarification.
21:10 spredzy mfisch, Etes vous ici ?
21:10 mfisch etes!
21:10 mfisch thanks
21:10 mfisch should stick with es tu
21:10 spredzy hehe
21:11 dgurtner spredzy: yw
21:11 spredzy yeah vous is too formal
21:11 mfisch sbadia: 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 mfisch spredzy: better to be formal than piss someone off ;)
21:12 spredzy mfisch, true. But its impossible to piss gchamoul off so its riskfree here :D
21:12 mfisch Its not like I could make you guys think Americans were any less cultured anyway
21:16 spredzy mfisch, ahaha we have no judgement over Americans :)
21:19 openstackgerrit Merged openstack/puppet-tuskar: Fix rspec 3.x syntax  https://review.openstack.org/204672
21:20 spredzy Ok I am out. Ttyl people
21:22 ericpeterson joined #puppet-openstack
21:30 linkedinyou joined #puppet-openstack
21:32 openstackgerrit Yanis Guenane proposed openstack/puppet-swift: Add tag to package and service resources  https://review.openstack.org/204668
21:47 aimon joined #puppet-openstack
21:53 sergmelikyan joined #puppet-openstack
22:01 jasondotstar joined #puppet-openstack
22:10 sergmelikyan joined #puppet-openstack
22:27 jasondotstar joined #puppet-openstack
22:51 sergmelikyan joined #puppet-openstack
22:56 xiaohua joined #puppet-openstack
22:59 openstackgerrit Sergii Golovatiuk proposed openstack/puppet-glance: Add swift_store_region parameter to glance::backend::swift  https://review.openstack.org/198669
23:16 xingchao joined #puppet-openstack
23:21 bklei joined #puppet-openstack
23:25 jasondotstar joined #puppet-openstack
23:26 openstackgerrit Merged openstack/puppet-murano: Puppet-Murano Project Namespace Change  https://review.openstack.org/204664
23:28 sergmelikyan joined #puppet-openstack
23:35 gildub joined #puppet-openstack
23:35 openstackgerrit Saksham Varma proposed openstack/puppet-neutron: N1Kv: Adding nexus1000v puppet class  https://review.openstack.org/204790
23:40 aimon joined #puppet-openstack
23:45 xarses joined #puppet-openstack
23:58 sergmelikyan joined #puppet-openstack

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