Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-10-20

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

All times shown according to UTC.

Time Nick Message
00:04 zhangjn joined #puppet-openstack
00:08 nlahouti joined #puppet-openstack
00:09 openstackgerrit Merged openstack/puppet-nova: Update nova::db class to match other module pattern  https://review.openstack.org/227196
00:44 nlahouti joined #puppet-openstack
00:47 guessi joined #puppet-openstack
00:50 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: First implementation of init class  https://review.openstack.org/237352
00:54 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: db: take params from aodh::db  https://review.openstack.org/237354
00:58 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Update aodh::logging  https://review.openstack.org/237356
00:59 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/216821
01:01 nlahouti joined #puppet-openstack
01:02 zhangjn joined #puppet-openstack
01:03 ducttape_ joined #puppet-openstack
01:06 nlahouti joined #puppet-openstack
01:09 khushbu joined #puppet-openstack
01:14 ducttape_ joined #puppet-openstack
01:31 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/216821
01:40 markvoelker joined #puppet-openstack
01:48 nlahouti joined #puppet-openstack
01:53 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/216821
02:12 ducttape_ joined #puppet-openstack
02:14 khushbu_ joined #puppet-openstack
02:56 nlahouti joined #puppet-openstack
02:57 nlahouti1 joined #puppet-openstack
03:02 nlahouti joined #puppet-openstack
03:16 nlahouti joined #puppet-openstack
03:43 khushbu_ joined #puppet-openstack
03:43 simon-AS5591 joined #puppet-openstack
03:55 guessi left #puppet-openstack
05:03 sergmelikyan joined #puppet-openstack
05:12 spredzy_ joined #puppet-openstack
05:52 sanjayu joined #puppet-openstack
06:10 sergmelikyan joined #puppet-openstack
06:17 sergmelikyan joined #puppet-openstack
06:23 gfidente joined #puppet-openstack
06:26 markvoelker_ joined #puppet-openstack
06:37 sergmelikyan joined #puppet-openstack
07:02 nlahouti joined #puppet-openstack
07:08 nlahouti joined #puppet-openstack
07:13 rochaporto joined #puppet-openstack
07:20 dgurtner joined #puppet-openstack
07:20 dgurtner joined #puppet-openstack
07:23 paramite joined #puppet-openstack
07:23 simon-AS559 joined #puppet-openstack
07:42 jpena joined #puppet-openstack
07:44 openstackgerrit mkarpin proposed openstack/puppet-heat: add more heat.conf settings to heat, engine and logging classes  https://review.openstack.org/233621
07:47 openstackgerrit joined #puppet-openstack
07:47 openstackgerrit mkarpin proposed openstack/puppet-heat: add more heat.conf settings to heat, engine and logging classes  https://review.openstack.org/233621
08:21 derekh joined #puppet-openstack
08:27 jistr joined #puppet-openstack
08:27 v1k0d3n joined #puppet-openstack
08:46 openstackgerrit Derek Higgins proposed openstack/puppet-heat: Revert "Add hooks for external install & svc management"  https://review.openstack.org/237466
08:47 derekh Hi all, looks like this commit ^^ broke the tripleo use case https://bugs.launchpad.net/tripleo/+bug/1507934
08:47 openstack Launchpad bug 1507934 in puppet-heat "Could not find resource 'Anchor[heat::db::begin]'" [Undecided,In progress] - Assigned to Derek Higgins (derekh)
08:47 sergmelikyan joined #puppet-openstack
08:56 sergmelikyan joined #puppet-openstack
09:04 openstackgerrit Michael Polenchuk proposed openstack/puppet-openstacklib: Fallback to default rcfile  https://review.openstack.org/227762
09:04 sergmelikyan joined #puppet-openstack
09:20 sergmelikyan joined #puppet-openstack
09:35 sergmelikyan joined #puppet-openstack
09:35 openstackgerrit Simon Murray proposed openstack/puppet-horizon: Allow Orchestration of Image Backend  https://review.openstack.org/236948
09:37 sergmelikyan joined #puppet-openstack
09:38 openstackgerrit Yanis Guenane proposed openstack/puppet-heat: Do not make hard dependency on Anchor is their not part of the catalog  https://review.openstack.org/237487
09:39 openstackgerrit Yanis Guenane proposed openstack/puppet-heat: Do not make hard dependency on Anchor is their not part of the catalog  https://review.openstack.org/237487
09:39 openstackgerrit Yanis Guenane proposed openstack/puppet-heat: Do not make hard dependency on Anchor is their not part of the catalog  https://review.openstack.org/237487
09:40 spredzy derekh, anyway you can test that patch ^ ?
09:40 derekh spredzy: thanks, the ci job should be running on it, so we'll know soon
09:41 openstackgerrit Yanis Guenane proposed openstack/puppet-heat: Do not make hard dependency on Anchor if their not part of the catalog  https://review.openstack.org/237487
09:41 spredzy derekh, true
09:44 openstackgerrit Maksym Yatsenko proposed openstack/puppet-nova: [WIP]  https://review.openstack.org/233011
09:54 khushbu_ joined #puppet-openstack
10:11 tlbr joined #puppet-openstack
10:16 derekh spredzy: the revert worked https://review.openstack.org/#/c/237466/1  but I'm happy to wait and see if the fix works
10:24 openstackgerrit mkarpin proposed openstack/puppet-glance: add more settings to glance api, registry, rabbitmq  https://review.openstack.org/237035
10:24 iberezovskiy joined #puppet-openstack
10:24 spredzy derekh, ack, I'll ping you when I get the result
10:24 derekh spredzy: ok
10:25 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-murano: Use 'python-murano-dashboard' for dashboard_package_name  https://review.openstack.org/237505
10:26 EmilienM good morning
10:27 chem EmilienM: yo
10:27 iberezovskiy morning
10:31 sergmelikyan joined #puppet-openstack
10:38 sergmelikyan joined #puppet-openstack
10:39 sergmelikyan joined #puppet-openstack
10:41 zerda joined #puppet-openstack
10:43 openstackgerrit Denis Egorenko proposed openstack/puppet-nova: Rewrite nova_network provider with using only nova client  https://review.openstack.org/229548
10:46 degorenko morning o/
10:47 derekh spredzy: Oct 20 11:10:32 instack.localdomain heat-engine[20044]: ProgrammingError: (_mysql_exceptions.ProgrammingError) (1146, "Table 'heat.service' doesn't exist") [SQL: u'SELECT service.created_at AS service_created_at
10:47 derekh spredzy: as if the db sync didn't run maube ?
10:47 derekh *maybe
10:48 spredzy derekh, yep it looks like it I was looking at the why
10:48 derekh spredzy: ok, in the mean time I propose going ahead with the revert https://review.openstack.org/#/c/237466/
10:49 derekh clayton: ^ was that your patch ?
10:50 simon-AS559 joined #puppet-openstack
10:50 todin joined #puppet-openstack
10:53 todin could anyone explain how the os_service_default in puppet-cinder is used? I want to change the schedule_driver.
11:04 weihan joined #puppet-openstack
11:04 spredzy todin, the os_service_default  fact return the string '<SERVICE DEFAULT>' that when specified to a cinder_config will be equivalent to ensure => absent
11:08 todin spredzy: I do not understand. so how would I change the cinder schedule_driver von Simple to Filter?
11:10 spredzy todin, you would override it , so either in hiera if you're using hiera via cinder::scheduler::scheduler_driver: YOURDRIVER
11:10 spredzy of wherever you call class  {'cinder::scheduler' : $scheduler_driver => YOURDRIVER }
11:11 todin thanks, I use hiera. Could I see the changed driver in the output of facter?
11:12 spredzy todin, nop its not a fact just a parameter that will end up written in cinder.conf
11:13 jistr hi, can we please get this revert merged https://review.openstack.org/#/c/237466/
11:14 EmilienM spredzy: last week, I was asking to mwhahaha the status of service default in all our modules, the thing you initiated. Is there some integration progress? mwhahaha is working on puppet-cinder afik
11:14 jistr i saw that fix is in the works but in the meantime our CI jobs fail
11:14 markvoelker joined #puppet-openstack
11:14 EmilienM jistr: clayton is the author
11:17 spredzy EmilienM, we talked about doing it during the summit, I added it as a subject for our agenda today, to know if people might get interested in doing it as a group (either at the summit, or remote, but at the same time) so it can be a community effort
11:17 sergmelikyan joined #puppet-openstack
11:17 EmilienM spredzy: not sure people would have time during the summit
11:19 EmilienM spredzy: have you looked https://review.openstack.org/236004 ?
11:21 spredzy EmilienM, Well it is still valid to bring it up during the agent see if some people would make time for it or not. If not we'll work on it as ny other task
11:22 spredzy EmilienM, for the later not yet
11:24 iurygregory good morning people
11:24 openstackgerrit Denis Egorenko proposed openstack/puppet-murano: Add basic acceptance test  https://review.openstack.org/233591
11:24 spredzy EmilienM, s/agent/agenda
11:25 EmilienM spredzy: my concern is I'm preparing 7.0.0
11:25 EmilienM spredzy: I'm dressing a list of features / bugs we need to do/fix in 7.0.0
11:26 EmilienM the default parameter thing was something we might have wanted in 7.0.0
11:26 EmilienM but I'm afraid, this thing did not get enough attention
11:26 EmilienM if you and mwhahaha could lead this topic and make sure some work is in progress, that would be great
11:27 spredzy But then same question as last week, should it be in 7.0.0 or not ?
11:27 EmilienM spredzy: it won't be, it's too late now
11:27 EmilienM but I don't want to forget this work
11:27 EmilienM spredzy: it would be in 7.1.0
11:28 spredzy EmilienM, ack. This is the answer I wanted. Not to be forgotten. Only to be in order to prioritize things
11:28 EmilienM spredzy: I see you're assigned to https://bugs.launchpad.net/tripleo/+bug/1507934
11:28 openstack Launchpad bug 1507934 in puppet-heat "Could not find resource 'Anchor[heat::db::begin]'" [Undecided,In progress] - Assigned to Yanis Guenane (yanis-guenane)
11:28 EmilienM spredzy: if you're working on the bug, make sure to talk with clayton - he's the author
11:29 spredzy Yep, I don't have the issue as the bug mention but dbsync is for some reason not triggered https://review.openstack.org/#/c/237487/
11:29 spredzy waiting for clayton to be online
11:29 EmilienM spredzy: gate-tripleo-ci-ironic-overcloud-f21puppet-nonha is still failing with your patch
11:30 EmilienM spredzy or anyone core who is around please +A https://review.openstack.org/#/c/237466/
11:30 spredzy EmilienM, as stated above the dbsync is not triggered any more
11:30 EmilienM spredzy: ah ok, so the revert is useless, we can fix it with your patch?
11:31 spredzy EmilienM, to me yes. Just need to understand why the dbsync is not run. But I don't think we should revert the patch, just fix what's broken
11:31 * spredzy afk
11:31 EmilienM spredzy: ok
11:32 sergmelikyan joined #puppet-openstack
11:32 derekh spredzy: but you don't know how long a fix will take, meanwhile tripleo-ci is blocked
11:32 EmilienM derekh, jistr: is it ok if we try to fix it with https://review.openstack.org/#/c/237487/ ?
11:32 EmilienM ok that's what I wanted to know
11:33 derekh EmilienM: if we knew that we could fix it quickly I wouldn't have a problem but as it stands we don't know the problem so I'd prefer to revert if we can
11:34 EmilienM derekh: ok, reverting
11:34 derekh EmilienM: thanks
11:35 EmilienM any core around please look https://review.openstack.org/#/c/237466/
11:36 morazi joined #puppet-openstack
11:39 markvoelker joined #puppet-openstack
11:40 clayton I'm guessing the CI jobs dont include the main heat class for some reason?
11:40 clayton that's where the anchors are defined
11:43 aderyugin Hi, folks! Can you please take a look https://review.openstack.org/#/c/218333/
11:44 dprince joined #puppet-openstack
11:45 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: First implementation of init class  https://review.openstack.org/237352
11:46 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: db: take params from aodh::db  https://review.openstack.org/237354
11:47 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Update aodh::logging  https://review.openstack.org/237356
11:56 openstackgerrit Lukas Bezdicka proposed openstack/puppet-neutron: Drop nova_admin_tenant_id_setter  https://review.openstack.org/237530
12:09 spredzy clayton, ping https://review.openstack.org/#/c/237487/
12:10 spredzy Would you happen to know why dbsync isn't triggered ?
12:10 clayton spredzy: why is the CI not including the heat class?
12:10 clayton it seemed like a fair assumption to me that it would
12:11 spredzy clayton, the way tripleo works is that on the first step you only have the heat::db class is included, and then on another step the heat class is on the catalog
12:11 spredzy but not on the first one
12:11 clayton I'm concerned that making the anchors option could lead to silent failures in other cases
12:11 clayton what about creating a heat::deps or heat::anchors class that everything includes?
12:12 ducttape_ joined #puppet-openstack
12:12 spredzy that sounds like it would work, since every class will include them
12:13 spredzy so the resource would be present no matter which class you include (ie. db, init, etc...)
12:13 spredzy derekh, Is there anyway CI can pin HEAD^1 for puppet-heat temporarily ?
12:14 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: First implementation of init class  https://review.openstack.org/237352
12:14 clayton as far as db sync not running, I don't think we've seen that issue
12:15 spredzy clayton, I have it here http://logs.openstack.org/87/237487/4/check-tripleo/gate-tripleo-ci-ironic-overcloud-f21puppet-nonha/95d82cb/console.html when I tried using the Anchor <| name == 'X' |> pattern to not be blocked byt the dependencies
12:16 clayton was the issue there before that change?
12:17 spredzy I don't know, I start to think that it might be on my change with the heat::db class
12:17 spredzy Will check a run even before that on
12:17 spredzy e
12:18 clayton nod, we've been using that patch in production for a while, so if db sync doesn't work then it's probably an edge case
12:22 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: First implementation of init class  https://review.openstack.org/237352
12:23 openstackgerrit Yanis Guenane proposed openstack/puppet-heat: DNM - Revert "Introduce heat::db class"  https://review.openstack.org/237540
12:23 spredzy clayton, ^ So I am testing your patch + the Anchor <| |> pattern + revert the heat::db patch
12:24 spredzy let's see what the CI says
12:27 clayton my concern with the Anchor <||> change is that it will cause silent dependency failures if someone fails to include ::heat in any other scenario
12:27 openstackgerrit Sergey Kolekonov proposed openstack/puppet-neutron: Deprecate nova_admin_ options  https://review.openstack.org/235356
12:29 openstackgerrit Sergey Kolekonov proposed openstack/puppet-neutron: Add parameters for Neutron QoS support  https://review.openstack.org/216654
12:35 markvoelker_ joined #puppet-openstack
12:37 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: First implementation of init class  https://review.openstack.org/237352
12:42 openstackgerrit Sebastien Badia proposed openstack/puppet-aodh: metadata: Fix openstacklib module namespace  https://review.openstack.org/237548
12:42 openstackgerrit Sebastien Badia proposed openstack/puppet-aodh: db: Use postgresql lib class for psycopg package  https://review.openstack.org/237549
12:47 openstackgerrit Dmitry Ilyin proposed openstack/puppet-openstacklib: Add retries to the openstack command  https://review.openstack.org/220224
12:47 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: db: take params from aodh::db  https://review.openstack.org/237354
12:48 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Update aodh::logging  https://review.openstack.org/237356
12:52 derekh spredzy: was AFK, taking a look to see if we can pin the puppet module but a revert is still my preference, otherwise other regression will creep in while we are pinned
12:54 simon-AS559 joined #puppet-openstack
12:57 simon-AS5591 joined #puppet-openstack
12:58 simon-AS559 joined #puppet-openstack
13:02 openstackgerrit Merged openstack/puppet-aodh: params: add aodh package & service names  https://review.openstack.org/237235
13:02 openstackgerrit Merged openstack/puppet-aodh: auth: refactor to use *_url  https://review.openstack.org/237265
13:02 openstackgerrit Merged openstack/puppet-aodh: acceptance: add basic manifest  https://review.openstack.org/237267
13:02 openstackgerrit Merged openstack/puppet-trove: Prepare 6.1.0 release  https://review.openstack.org/237165
13:03 openstackgerrit Merged openstack/puppet-horizon: Prepare 6.1.0 release  https://review.openstack.org/237150
13:04 openstackgerrit Merged openstack/puppet-nova: Move cinder_catalog_info to init  https://review.openstack.org/237006
13:07 openstackgerrit Merged openstack/puppet-cinder: Prepare 6.1.0 release  https://review.openstack.org/237141
13:08 ducttape_ joined #puppet-openstack
13:09 spredzy clayton, are you working on it ? Do you want me go submit a chance ? heat::anchors that would be included in all classes that requires it ?
13:10 clayton I just finished getting the kids off to school haven't had a chance to look yet
13:12 simon-AS5591 joined #puppet-openstack
13:14 openstackgerrit Sergey Kolekonov proposed openstack/puppet-neutron: Deprecate nova_admin_ options  https://review.openstack.org/235356
13:14 spredzy clayton, ok
13:16 openstackgerrit Merged openstack/puppet-designate: Prepare 6.1.0 release  https://review.openstack.org/237142
13:17 ducttape_ joined #puppet-openstack
13:17 openstackgerrit Merged openstack/puppet-glance: Prepare 6.1.0 release  https://review.openstack.org/237143
13:17 mjb joined #puppet-openstack
13:18 openstackgerrit Sebastien Badia proposed openstack/puppet-barbican: db: Use postgresql lib class for psycopg package  https://review.openstack.org/237576
13:20 zhangjn joined #puppet-openstack
13:21 openstackgerrit Merged openstack/puppet-vswitch: Prepare 2.1.0 release  https://review.openstack.org/237166
13:21 openstackgerrit Merged openstack/puppet-openstacklib: Prepare 6.1.0 release  https://review.openstack.org/237158
13:21 openstackgerrit Merged openstack/puppet-tempest: Prepare 6.1.0 release  https://review.openstack.org/237164
13:21 simon-AS559 joined #puppet-openstack
13:21 zhangjn joined #puppet-openstack
13:22 openstackgerrit Merged openstack/puppet-heat: Prepare 6.1.0 release.  https://review.openstack.org/237146
13:22 zhangjn joined #puppet-openstack
13:23 zhangjn joined #puppet-openstack
13:23 openstackgerrit Merged openstack/puppet-ironic: Prepare 6.1.0 release  https://review.openstack.org/237151
13:23 simon-AS5591 joined #puppet-openstack
13:24 openstackgerrit Merged openstack/puppet-manila: Prepare 6.1.0 release  https://review.openstack.org/237154
13:24 openstackgerrit Merged openstack/puppet-sahara: Prepare 6.1.0 release  https://review.openstack.org/237162
13:24 openstackgerrit Merged openstack/puppet-neutron: Prepare 6.1.0 release  https://review.openstack.org/237139
13:25 openstackgerrit Merged openstack/puppet-nova: Prepare 6.1.0 release  https://review.openstack.org/237155
13:27 clayton spredzy: I've got about an hour I can spend on it now, if you're not already looking at it
13:27 spredzy clayton, I am not, didn't want to duplicate work I though you might be doing
13:28 clayton ok, let me see what I can get done.  in theory it's a simple change.
13:28 clayton you prefer designate::deps, designate::anchors or designate::dependency ?
13:30 openstackgerrit Merged openstack/puppet-keystone: Prepare 6.1.0 release  https://review.openstack.org/237153
13:30 clayton looking at it now, I think this is a lot better pattern anyway.  It was kind of gross to have that at the bottom of init.pp
13:31 openstackgerrit Martin Mágr proposed openstack/puppet-glance: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/237581
13:36 openstackgerrit Dmitry Ilyin proposed openstack/puppet-openstacklib: Add retries to the openstack command  https://review.openstack.org/220224
13:38 openstackgerrit Sebastien Badia proposed openstack/puppet-ceilometer: db: Use postgresql lib class for psycopg package  https://review.openstack.org/237583
13:46 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement API service  https://review.openstack.org/237587
13:47 weihan joined #puppet-openstack
13:48 richm joined #puppet-openstack
13:48 spredzy clayton, sorry I missed your messages
13:49 spredzy ::anchors make more sense to me
13:49 spredzy if the question still stand :)
13:50 EmilienM spredzy: ::heat::anchors?
13:50 clayton I already named it deps, but I originally had it as anchors
13:50 clayton but I changed it to deps, because the class is really more about managing the dependencies, and anchors are just used for the hooks
13:51 clayton and well, ::heat::dependencies is really long
13:52 openstackgerrit Sebastien Badia proposed openstack/puppet-gnocchi: db: Use postgresql lib class for psycopg package  https://review.openstack.org/237591
13:52 EmilienM do we really need a class for that?
13:53 dfisher joined #puppet-openstack
13:53 clayton EmilienM: the idea is to move all the anchors and dependency stuff associated with it, then just include that in all the classes.  Otherwise you have to assume anyone that is using heat is including ::heat
13:53 clayton if you make those deps via collectors, then it will just silently fail if the anchors and dependencies aren't included
13:57 ducttape_ joined #puppet-openstack
13:57 openstackgerrit Clayton O'Neill proposed openstack/puppet-heat: Move deps & external hooks into a standalone class  https://review.openstack.org/237592
13:57 clayton spredzy, EmilienM: ^^^
13:58 zhangjn joined #puppet-openstack
13:59 zhangjn joined #puppet-openstack
14:00 zhangjn joined #puppet-openstack
14:05 simon-AS559 joined #puppet-openstack
14:07 simon-AS5591 joined #puppet-openstack
14:08 TC01 joined #puppet-openstack
14:08 ferest joined #puppet-openstack
14:11 simon-AS559 joined #puppet-openstack
14:12 vinsh joined #puppet-openstack
14:15 EmilienM jamespage: I found an issue in aodh package, the wsgi file is not in aodh/api/app.wsgi
14:17 EmilienM oh it's somewhere else... ok..
14:19 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement API service  https://review.openstack.org/237587
14:22 iurygregory mwhahaha are you around? any ideia about http://logs.openstack.org/21/216821/13/check/gate-puppet-keystone-puppet-beaker-rspec-dsvm-trusty/0a3a636/console.html#_2015-10-20_02_09_01_810 =/
14:24 dfisher lord, i don't even know what aodh is and there's a puppet module for it...
14:29 openstackgerrit Sebastien Badia proposed openstack/puppet-ironic: db: Use postgresql lib class for psycopg package  https://review.openstack.org/237605
14:29 openstackgerrit Sebastien Badia proposed openstack/puppet-heat: db: Use postgresql lib class for psycopg package  https://review.openstack.org/237606
14:31 mattymo iurygregory, looking at your keystone patch
14:32 mwhahaha iurygregory: so that's what i was refering to when I said you may need to use the spaceship operator to do some something weird. the issue is that it's already set elsewhere so you need to override it somehow or it needs to be specified elsewhere
14:33 iurygregory i don't know about spaceship operator =/
14:33 iurygregory "do something weird" wow
14:34 iurygregory i've put a topic in the meeting =)
14:34 iurygregory thanks for looking mattymo o/
14:34 mwhahaha iurygregory: https://docs.puppetlabs.com/puppet/latest/reference/lang_collectors.html
14:35 iurygregory Thanks mwhahaha
14:36 mattymo yeah spaceship operator is how I'd do it... but I don't like it.. we want to append a custom_fragment..
14:36 mwhahaha the cleanest way is probably add a flag to keystone::wsgi::apache to include that configuration for you and it just has to be documented that the user needs to specify that
14:36 mattymo it's not like custom fragment should be an array of fragments
14:37 mwhahaha well we specify multiple of them depending on what you're doing, so it's kinda on the operator to figure out what needs to be included
14:38 mattymo I don't think the collector can append strings
14:38 openstackgerrit Lukas Bezdicka proposed openstack/puppet-neutron: Cleanup list output  https://review.openstack.org/237610
14:38 mwhahaha yea i don't think it can either
14:39 morazi joined #puppet-openstack
14:42 iurygregory i'll take alook
14:42 openstackgerrit Drew Fisher proposed openstack/puppet-heat: Remove POSIX users, groups, and file modes.  https://review.openstack.org/237612
14:42 angdraug joined #puppet-openstack
14:47 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement API service  https://review.openstack.org/237587
14:55 mdorman joined #puppet-openstack
14:56 EmilienM meeting starts in a few minutes!
14:58 iberezovskiy joined #puppet-openstack
14:59 sergmelikyan joined #puppet-openstack
15:02 nlahouti joined #puppet-openstack
15:02 openstackgerrit Lukas Bezdicka proposed openstack/puppet-neutron: Cleanup list output  https://review.openstack.org/237610
15:04 xarses joined #puppet-openstack
15:09 derekh Anoth regression https://review.openstack.org/#/c/237006/
15:09 derekh I think
15:09 derekh Oct 20 15:04:04 overcloud-novacompute-0 os-collect-config[2891]: Error: Duplicate declaration: Nova_config[cinder/catalog_info] is already declared in file /etc/puppet/modules/nova/manifests/init.pp:651; cannot
15:18 openstackgerrit Derek Higgins proposed openstack/puppet-nova: Revert "Move cinder_catalog_info to init"  https://review.openstack.org/237634
15:19 derekh EmilienM: testing this revert https://review.openstack.org/#/c/237634/
15:20 EmilienM derekh: we are in meeting atm
15:20 EmilienM derekh: will look right after this
15:20 derekh EmilienM: ack, thanks and sorry
15:20 EmilienM no worries
15:27 simon-AS559 joined #puppet-openstack
15:27 ericksonsantos joined #puppet-openstack
15:28 simon-AS5591 joined #puppet-openstack
15:29 simon-AS559 joined #puppet-openstack
15:33 jamespage EmilienM, ack
15:34 degorenko EmilienM, about tests for murano and iberezovskiy patch - we don't have spe for params
15:34 simon-AS5591 joined #puppet-openstack
15:34 degorenko in other modules too :)
15:34 EmilienM jamespage: no it's ok
15:34 EmilienM degorenko: yes we have in other modules
15:35 degorenko EmilienM, https://github.com/openstack/puppet-keystone/tree/master/spec/classes
15:35 EmilienM degorenko: https://github.com/openstack/puppet-nova/blob/master/spec/classes/nova_api_spec.rb#L326-L329
15:35 EmilienM it tests params.pp ;-)
15:36 degorenko EmilienM, that's true
15:36 degorenko but we don't have yet murano-dashobard module
15:36 degorenko so we can't test it
15:36 16WAAKF6R joined #puppet-openstack
15:37 EmilienM degorenko: the module is lacking in testing
15:37 EmilienM please fix it
15:37 degorenko EmilienM, ack
15:37 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Client and dashboard class implementation  https://review.openstack.org/218333
15:38 degorenko EmilienM, already did it for iberezovskiy patch ^ and acceptance: https://review.openstack.org/#/c/233591/ fyi
15:38 skolekonov left #puppet-openstack
15:38 degorenko some puppet issues for ubuntu and we don't have packages for centos
15:39 openstackgerrit Merged openstack/puppet-murano: Use 'python-murano-dashboard' for dashboard_package_name  https://review.openstack.org/237505
15:39 degorenko spredzy, do you need help with glance db and logging? I can do it
15:40 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Client and dashboard class implementation  https://review.openstack.org/218333
15:40 spredzy degorenko, we can split if you want. Pick any of the two I'll pick the other
15:41 degorenko spredzy, i take db then :)
15:41 spredzy logging it is for me :)
15:41 degorenko ;)
15:41 spredzy thanks degorenko
15:41 degorenko spredzy, no problem :)
15:41 iurygregory EmilienM, to not have a block for federation we can use concat?
15:42 EmilienM iurygregory: I don't like using concat in our module, can't we use puppetlabs-apache at all?
15:43 openstackgerrit Maksym Yatsenko proposed openstack/puppet-nova: [WIP]  https://review.openstack.org/233011
15:43 degorenko EmilienM, probably no, without hacking on apache side
15:43 degorenko we should check it
15:43 iurygregory well the virtualhost file will recive two additions.. the first is inside <VirtualHost> just add 1 line to it, we will use custom_fragment
15:44 iurygregory to add the location tags, concat is the fast and easyier way i think
15:45 iurygregory you mentioned problem with puppet-swift, but i'm not aware of it
15:45 degorenko +1
15:45 degorenko what's the problem?
15:46 dfisher joined #puppet-openstack
15:46 degorenko iurygregory, i guess, we can try our way at first
15:46 degorenko just to be sure that it works
15:47 iurygregory yeah, i'll try after lunch XD
15:47 degorenko :D have a nice lunch
15:47 iurygregory i'm waiting for the food be delivered =/
15:48 iurygregory i'll be back in 2hrs ^^
15:49 morazi joined #puppet-openstack
15:53 rahulait joined #puppet-openstack
16:00 EmilienM derekh: back
16:02 EmilienM derekh: I'm not sure why we revert the cinder catalog patch
16:02 derekh EmilienM: hi, so I was just letting you know about another revert I submitted as it was your patch
16:02 EmilienM we should not revert it
16:02 EmilienM I have a patch in THT for long time for that
16:02 EmilienM which is not merged
16:02 EmilienM https://review.openstack.org/225983
16:02 EmilienM if people would review it, it would unblock your CI
16:03 derekh EmilienM: ok, I didn't know that
16:03 derekh jistr: FYI ^
16:03 EmilienM derekh: passing CI btw
16:05 derekh EmilienM: ack, looking
16:11 khushbu joined #puppet-openstack
16:13 gfidente joined #puppet-openstack
16:14 derekh EmilienM: thats merged now, sorry for the confusion, I hadn't seen it
16:14 EmilienM derekh: no problem, please let me know if I can help
16:15 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service  https://review.openstack.org/237662
16:15 EmilienM lead
16:18 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: First implementation of init class  https://review.openstack.org/237352
16:19 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Update aodh::logging  https://review.openstack.org/237356
16:21 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Update aodh::logging  https://review.openstack.org/237356
16:22 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement API service  https://review.openstack.org/237587
16:22 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service  https://review.openstack.org/237662
16:25 openstackgerrit Sebastien Badia proposed openstack/puppet-tuskar: db: Use postgresql lib class for psycopg package  https://review.openstack.org/237666
16:25 openstackgerrit Sebastien Badia proposed openstack/puppet-sahara: db: Use postgresql lib class for psycopg package  https://review.openstack.org/237667
16:25 openstackgerrit Sebastien Badia proposed openstack/puppet-nova: db: Use postgresql lib class for psycopg package  https://review.openstack.org/237668
16:25 openstackgerrit Sebastien Badia proposed openstack/puppet-openstack-cookiecutter: db: Use postgresql lib class for psycopg package  https://review.openstack.org/237669
16:25 openstackgerrit Sebastien Badia proposed openstack/puppet-neutron: db: Use postgresql lib class for psycopg package  https://review.openstack.org/237670
16:25 openstackgerrit Sebastien Badia proposed openstack/puppet-manila: db: Use postgresql lib class for psycopg package  https://review.openstack.org/237671
16:25 openstackgerrit Sebastien Badia proposed openstack/puppet-keystone: db: Use postgresql lib class for psycopg package  https://review.openstack.org/237672
16:26 khushbu_ joined #puppet-openstack
16:28 openstackgerrit Sebastien Badia proposed openstack/puppet-murano: db: Use postgresql lib class for psycopg package  https://review.openstack.org/237679
16:29 khushbu_ joined #puppet-openstack
16:29 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service  https://review.openstack.org/237662
16:33 khushbu joined #puppet-openstack
16:33 weihan joined #puppet-openstack
16:33 sanjayu joined #puppet-openstack
16:36 khushbu_ joined #puppet-openstack
16:37 _nick joined #puppet-openstack
16:50 mkarpin joined #puppet-openstack
16:57 khushbu joined #puppet-openstack
16:58 sergmelikyan joined #puppet-openstack
17:05 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement API service  https://review.openstack.org/237587
17:05 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service  https://review.openstack.org/237662
17:08 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service  https://review.openstack.org/237662
17:16 openstackgerrit Dmitry Ilyin proposed openstack/puppet-openstacklib: Add retries to the openstack command  https://review.openstack.org/220224
17:18 linkedinyou joined #puppet-openstack
17:23 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service  https://review.openstack.org/237662
17:27 bitblt joined #puppet-openstack
17:34 dtalton joined #puppet-openstack
17:36 dtalton joined #puppet-openstack
17:39 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service  https://review.openstack.org/237662
17:39 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Client class  https://review.openstack.org/237706
17:43 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Notifier service  https://review.openstack.org/237708
17:45 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Listener service  https://review.openstack.org/237709
18:00 jfluhmann joined #puppet-openstack
18:03 nlahouti joined #puppet-openstack
18:19 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service  https://review.openstack.org/237662
18:20 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Client class  https://review.openstack.org/237706
18:20 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Notifier service  https://review.openstack.org/237708
18:21 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Listener service  https://review.openstack.org/237709
18:28 dprince joined #puppet-openstack
18:34 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service  https://review.openstack.org/237662
18:34 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Client class  https://review.openstack.org/237706
18:35 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service  https://review.openstack.org/237662
18:36 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Client class  https://review.openstack.org/237706
18:37 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Client class  https://review.openstack.org/237706
18:37 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Notifier service  https://review.openstack.org/237708
18:37 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Listener service  https://review.openstack.org/237709
18:41 nlahouti joined #puppet-openstack
18:41 sergmelikyan joined #puppet-openstack
18:46 nlahouti joined #puppet-openstack
18:57 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service  https://review.openstack.org/237662
18:58 rahulait joined #puppet-openstack
19:04 paramite joined #puppet-openstack
19:11 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service  https://review.openstack.org/237662
19:14 mwhahaha EmilienM: are we proceeding with $::os_service_default in other modules now too?
19:14 EmilienM mwhahaha: it would be nice
19:14 mwhahaha k
19:16 nlahouti joined #puppet-openstack
19:21 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Client class  https://review.openstack.org/237706
19:22 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Notifier service  https://review.openstack.org/237708
19:23 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Listener service  https://review.openstack.org/237709
19:24 EmilienM dprince: re- https://launchpad.net/bugs/1507934
19:24 openstack Launchpad bug 1507934 in puppet-heat "Could not find resource 'Anchor[heat::db::begin]'" [High,In progress] - Assigned to Clayton O'Neill (clayton-oneill)
19:24 EmilienM dprince: the long term approach would be https://review.openstack.org/#/c/237592/
19:25 EmilienM afik, spredzy is still investigating what's wrong with db
19:25 EmilienM see https://review.openstack.org/#/c/237487/
19:25 EmilienM I have no idea what's going on now
19:25 EmilienM maybe clayton knows
19:25 clayton me either
19:26 clayton I know CI pinned to the previous release, so I assume the patch that broke it wasn't reverted
19:26 clayton in which case, we need reviews/feedback on the heat::deps class
19:26 dprince EmilienM: okay, catching up a bit on these reviews
19:27 nlahouti joined #puppet-openstack
19:28 EmilienM clayton: I've never seen this kind of class anywhere else
19:28 EmilienM or it was not called "deps"
19:28 clayton well, most puppet modules now days seem to follow a install/config/service sort of pattern
19:29 clayton but we generally manage a lot of installs, configs and services in the openstack modules, so I see this is kind of a variant of that pattern
19:34 nlahouti joined #puppet-openstack
19:37 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service  https://review.openstack.org/237662
19:38 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Client class  https://review.openstack.org/237706
19:39 EmilienM clayton: it makes sense, yes
19:40 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Notifier service  https://review.openstack.org/237708
19:40 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Listener service  https://review.openstack.org/237709
19:41 dgurtner joined #puppet-openstack
19:41 dgurtner joined #puppet-openstack
19:43 nlahouti joined #puppet-openstack
19:46 nlahouti joined #puppet-openstack
19:49 openstackgerrit Sebastien Badia proposed openstack/puppet-cinder: db: Use postgresql lib class for psycopg package  https://review.openstack.org/237760
20:01 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Make Murano config providers inherit Openstack config provider  https://review.openstack.org/237761
20:03 bnemec joined #puppet-openstack
20:03 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Add basic acceptance test  https://review.openstack.org/233591
20:07 nlahouti joined #puppet-openstack
20:28 openstackgerrit Gilles Dubreuil proposed openstack/puppet-keystone: Deletes tenant parameter from keystone_user  https://review.openstack.org/228744
20:28 simon-AS5591 joined #puppet-openstack
20:36 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service  https://review.openstack.org/237662
20:37 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Client class  https://review.openstack.org/237706
20:37 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Notifier service  https://review.openstack.org/237708
20:37 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Listener service  https://review.openstack.org/237709
20:41 sergmelikyan joined #puppet-openstack
20:45 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/216821
20:47 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service  https://review.openstack.org/237662
20:50 sergmelikyan joined #puppet-openstack
20:55 sergmelikyan joined #puppet-openstack
20:55 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Make Murano config providers inherit Openstack config provider  https://review.openstack.org/237761
20:57 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Add basic acceptance test  https://review.openstack.org/233591
21:00 sergmelikyan joined #puppet-openstack
21:05 serg_melikyan joined #puppet-openstack
21:12 iurygregory any problem in the CI? O.o "gate-puppet-keystone-puppet-syntax-3-dsvm-centos7: NOT_REGISTERED"
21:25 EmilienM looking
21:25 EmilienM iurygregory: which patch?
21:25 EmilienM iurygregory: probably zuul issue
21:29 simon-AS5591 joined #puppet-openstack
21:30 EmilienM try 'recheck'
21:49 linkedinyou joined #puppet-openstack
21:59 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service  https://review.openstack.org/237662
22:18 gildub joined #puppet-openstack
22:36 jongchoi joined #puppet-openstack
22:40 ducttape_ joined #puppet-openstack
22:55 sanjayu joined #puppet-openstack
23:20 xarses joined #puppet-openstack
23:42 xarses joined #puppet-openstack
23:45 bogdando joined #puppet-openstack

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