| *** ducttape_ has quit IRC | 00:00 | |
| openstackgerrit | Alex Krzos proposed openstack/puppet-keystone: Tune number of apache processes/threads for Keystone in apache. https://review.openstack.org/297342 | 00:06 |
|---|---|---|
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 00:15 |
| *** rmallah has joined #puppet-openstack | 00:17 | |
| *** rmallah has quit IRC | 00:18 | |
| *** ducttape_ has joined #puppet-openstack | 00:30 | |
| *** ducttape_ has quit IRC | 00:33 | |
| *** ducttape_ has joined #puppet-openstack | 00:36 | |
| *** ducttape_ has quit IRC | 00:39 | |
| *** angdraug has quit IRC | 00:40 | |
| *** richm has quit IRC | 00:41 | |
| *** xarses has joined #puppet-openstack | 00:42 | |
| *** dfisher has quit IRC | 00:56 | |
| *** ducttape_ has joined #puppet-openstack | 01:03 | |
| *** ducttape_ has quit IRC | 01:35 | |
| *** ducttape_ has joined #puppet-openstack | 01:38 | |
| *** saneax is now known as saneax_AFK | 01:41 | |
| *** ducttape_ has quit IRC | 01:52 | |
| *** ducttape_ has joined #puppet-openstack | 02:06 | |
| *** ducttape_ has quit IRC | 02:14 | |
| *** brandon has quit IRC | 02:15 | |
| *** ducttape_ has joined #puppet-openstack | 02:19 | |
| *** ducttape_ has quit IRC | 02:21 | |
| *** saneax_AFK is now known as saneax | 02:23 | |
| *** ducttape_ has joined #puppet-openstack | 02:27 | |
| *** rmallah has joined #puppet-openstack | 02:34 | |
| *** ducttape_ has quit IRC | 02:35 | |
| rmallah | hi is there any howto on getting keystone working with puppet-openstack on ubuntu 14.04.4 LTS ? | 02:35 |
| rmallah | https://gist.github.com/rmallah/804090709379cb6b8de7 | 02:38 |
| rmallah | i am trying to get above working. | 02:38 |
| *** ducttape_ has joined #puppet-openstack | 02:40 | |
| rmallah | openstack is being called with unsupported args --default , openstack project show: error: unrecognized arguments: --domain Default | 02:42 |
| rmallah | https://gist.github.com/rmallah/8292d4541ada437f5ea6 | 02:43 |
| *** saneax is now known as saneax_AFK | 02:43 | |
| mwhahaha | rmallah: what version of the puppet scripts are you using and what version of openstack packages? | 02:44 |
| *** ducttape_ has quit IRC | 02:44 | |
| rmallah | firstly thanks for the response. | 02:44 |
| rmallah | ├── openstack-keystone (v7.0.0) | 02:45 |
| rmallah | ├── openstack-keystone (v7.0.0) | 02:45 |
| rmallah | root@controller01:~# openstack --version | 02:45 |
| rmallah | openstack 0.3.0 | 02:45 |
| rmallah | root@controller01:~# lsb_release -a | 02:45 |
| rmallah | No LSB modules are available. | 02:45 |
| rmallah | Distributor ID: Ubuntu | 02:45 |
| rmallah | Description: Ubuntu 14.04.4 LTS | 02:45 |
| rmallah | Release: 14.04 | 02:45 |
| rmallah | Codename: trusty | 02:45 |
| rmallah | how does one get to know if openstack version is Juno , Kilo , Liberty , Mitaka etc ? | 02:46 |
| rmallah | ok now i have added ubuntu cloud ring and upgraded openstack packages , lets see how it goes, | 02:59 |
| *** fedexo has joined #puppet-openstack | 03:03 | |
| rmallah | ************ OK , now it worked !! ************************* | 03:04 |
| rmallah | shall i attempt to patch the module to include ubuntu-cloud archive if version is 7.0.0 | 03:04 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 03:05 |
| _ody | rmallah: That is handled in https://github.com/openstack/puppet-openstack_extras/blob/master/manifests/repo/debian/ubuntu.pp. | 03:27 |
| rmallah | thanks | 03:28 |
| rmallah | i am new to openstack can anyone pls help with setting up admin-openrc.sh , i have already run puppet successfully. | 03:29 |
| rmallah | my admin role is being setup as : class { 'keystone::roles::admin': email => 'admin', password => 'super_secret', } | 03:30 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 03:30 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Export path when puppet 4. https://review.openstack.org/298509 | 03:30 |
| mwhahaha | ah sorry got side tracked | 03:33 |
| mwhahaha | rmallah: that seems like an old version of the openstack client | 03:33 |
| rmallah | yes | 03:34 |
| rmallah | now i included UCA manually then got | 03:34 |
| rmallah | root@controller01:~# openstack --version | 03:34 |
| rmallah | openstack 1.7.0 | 03:34 |
| *** fedexo has quit IRC | 03:34 | |
| mwhahaha | yea so that should probably address your problem | 03:35 |
| rmallah | yes i just now discovered https://github.com/openstack/puppet-openstack_extras from @_ody 's comment. | 03:35 |
| rmallah | the page also answeres one of my previously asked question. | 03:36 |
| openstackgerrit | Merged openstack/puppet-neutron: Set default branch to stable/liberty https://review.openstack.org/298260 | 03:38 |
| rmallah | can anyone pls help with OS_USERNAME , OS_PASSWORD , OS_AUTH_URL etc if openstack admin role is being setup as: class { 'keystone::roles::admin': email => 'admin', password => 'super_secret' } | 03:38 |
| *** ayoung has quit IRC | 03:46 | |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Add barbican client class and params https://review.openstack.org/296156 | 03:47 |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Moved logging to barbican-api module https://review.openstack.org/296157 | 03:47 |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Add helper files to configure paste file and barbican.conf https://review.openstack.org/296158 | 03:47 |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Add barbican-api manifest https://review.openstack.org/296159 | 03:47 |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Added KMIP plugin settings to barbican-api manifest https://review.openstack.org/296163 | 03:47 |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Added keystone config to barbican api manifest https://review.openstack.org/263461 | 03:47 |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Added settings for cert and secret store plugins https://review.openstack.org/296162 | 03:47 |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Added apache component Add test and mods for deployment as an apache module https://review.openstack.org/263462 | 03:47 |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Add keystone notification options to barbican-api manifest https://review.openstack.org/296161 | 03:47 |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Added Dogtag settings for barbican-api https://review.openstack.org/258851 | 03:47 |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Add quota and secret limit parameters to barbican-api manifest https://review.openstack.org/296160 | 03:47 |
| *** rmallah has quit IRC | 03:54 | |
| *** brandon has joined #puppet-openstack | 04:01 | |
| *** brandon has quit IRC | 04:03 | |
| openstackgerrit | Michael Chapman proposed openstack/puppet-tripleo: Add neutron profiles https://review.openstack.org/293436 | 04:07 |
| *** brandon has joined #puppet-openstack | 04:09 | |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 04:13 |
| *** aimon has quit IRC | 04:13 | |
| *** aimon has joined #puppet-openstack | 04:14 | |
| *** brandon has quit IRC | 04:15 | |
| *** rmallah has joined #puppet-openstack | 04:16 | |
| *** rmallah has quit IRC | 04:20 | |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 04:23 |
| *** rmallah has joined #puppet-openstack | 04:24 | |
| *** brandon has joined #puppet-openstack | 04:25 | |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 04:30 |
| *** brandon has quit IRC | 04:37 | |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 04:46 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-gnocchi: Switch gnocchi to rspec-puppet-facts https://review.openstack.org/298528 | 04:48 |
| *** rmallah has quit IRC | 04:56 | |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 04:56 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Disable secure_path https://review.openstack.org/298530 | 04:56 |
| *** saneax_AFK is now known as saneax | 05:01 | |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Disable secure_path https://review.openstack.org/298530 | 05:03 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 05:03 |
| *** rmallah has joined #puppet-openstack | 05:03 | |
| _ody | Nothin like spelunking... | 05:04 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Disable secure_path https://review.openstack.org/298530 | 05:08 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 05:08 |
| openstackgerrit | Michael Chapman proposed openstack/puppet-tripleo: Add neutron profiles https://review.openstack.org/293436 | 05:12 |
| *** aimon has quit IRC | 05:15 | |
| *** aimon has joined #puppet-openstack | 05:17 | |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Disable secure_path https://review.openstack.org/298530 | 05:28 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 05:28 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Disable secure_path https://review.openstack.org/298530 | 05:39 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 05:39 |
| *** jtsigma has quit IRC | 05:48 | |
| *** rmallah has quit IRC | 05:49 | |
| *** rmallah has joined #puppet-openstack | 05:50 | |
| *** jmikolajczak has joined #puppet-openstack | 05:56 | |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Dynamic secure_path https://review.openstack.org/298530 | 05:58 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 05:58 |
| *** jaosorior has joined #puppet-openstack | 06:08 | |
| *** rmallah has quit IRC | 06:10 | |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Update secure_path https://review.openstack.org/298530 | 06:11 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 06:11 |
| *** rmallah has joined #puppet-openstack | 06:12 | |
| *** rmallah has quit IRC | 06:14 | |
| *** tesseract has joined #puppet-openstack | 06:19 | |
| *** yolanda has joined #puppet-openstack | 06:19 | |
| *** tesseract is now known as Guest58782 | 06:19 | |
| *** xenolog has quit IRC | 06:20 | |
| *** rmallah has joined #puppet-openstack | 06:22 | |
| *** markvoelker has joined #puppet-openstack | 06:22 | |
| *** rmallah has quit IRC | 06:22 | |
| *** xenolog has joined #puppet-openstack | 06:23 | |
| *** markvoelker_ has joined #puppet-openstack | 06:23 | |
| *** ikkeT has left #puppet-openstack | 06:26 | |
| *** markvoelker has quit IRC | 06:27 | |
| *** markvoelker_ has quit IRC | 06:43 | |
| *** markvoelker has joined #puppet-openstack | 06:44 | |
| openstackgerrit | Saverio Proto proposed openstack/puppet-cinder: Add osapi_max_limit parameter https://review.openstack.org/298556 | 06:46 |
| *** jtomasek has joined #puppet-openstack | 06:50 | |
| *** foxhunt_away is now known as foxhunt72 | 07:07 | |
| *** samuelBartel has joined #puppet-openstack | 07:10 | |
| *** markvoelker has quit IRC | 07:12 | |
| *** jpena has joined #puppet-openstack | 07:23 | |
| openstackgerrit | Cody Herriges proposed openstack/puppet-glance: Autorequire keystone_endpoint https://review.openstack.org/298576 | 07:29 |
| *** rcernin has joined #puppet-openstack | 07:36 | |
| *** _degorenko|afk is now known as degorenko | 07:48 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-glance: Add posibility to configure Glance backend settings for Glare service https://review.openstack.org/298586 | 07:49 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 07:50 |
| *** gfidente has joined #puppet-openstack | 07:50 | |
| *** derekh has joined #puppet-openstack | 08:11 | |
| *** markvoelker has joined #puppet-openstack | 08:13 | |
| *** markvoelker has quit IRC | 08:17 | |
| *** jistr has joined #puppet-openstack | 08:23 | |
| *** mgagne has quit IRC | 09:08 | |
| *** khappone_ has joined #puppet-openstack | 09:09 | |
| *** mjb has quit IRC | 09:11 | |
| *** khappone has quit IRC | 09:11 | |
| *** mjb has joined #puppet-openstack | 09:14 | |
| *** mgagne has joined #puppet-openstack | 09:14 | |
| *** mgagne is now known as Guest68910 | 09:14 | |
| *** dtantsur|afk is now known as dtantsur | 09:29 | |
| openstackgerrit | Merged openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI https://review.openstack.org/276187 | 09:41 |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Export path when puppet 4. https://review.openstack.org/298509 | 09:47 |
| *** markvoelker has joined #puppet-openstack | 10:14 | |
| *** markvoelker has quit IRC | 10:20 | |
| openstackgerrit | Nick Jones proposed openstack/puppet-horizon: Make syslog logging handler optional https://review.openstack.org/298653 | 10:47 |
| *** markvoelker has joined #puppet-openstack | 11:16 | |
| openstackgerrit | Matthew J Black proposed openstack/puppet-keystone: domain backend drivers set in domain config https://review.openstack.org/298672 | 11:16 |
| openstackgerrit | Nick Jones proposed openstack/puppet-horizon: Make syslog logging handler optional https://review.openstack.org/298653 | 11:18 |
| EmilienM | hello | 11:21 |
| *** markvoelker has quit IRC | 11:21 | |
| *** johnmilton has joined #puppet-openstack | 11:23 | |
| openstackgerrit | Matthew J Black proposed openstack/puppet-horizon: Added ensure_resource for python-memcache package. https://review.openstack.org/298679 | 11:30 |
| mjblack | EmilienM: morning | 11:32 |
| *** dprince has joined #puppet-openstack | 11:32 | |
| colonwq | EmilienM, good morning | 11:36 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-murano: Update notification driver option https://review.openstack.org/298687 | 11:52 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-murano: Update notification driver option https://review.openstack.org/298687 | 11:56 |
| degorenko | morning :) | 12:00 |
| *** trown|PTO is now known as trown | 12:01 | |
| *** foxhunt72 is now known as foxhunt_away | 12:07 | |
| *** pegmanm has joined #puppet-openstack | 12:10 | |
| *** ducttape_ has joined #puppet-openstack | 12:12 | |
| *** jayg|g0n3 is now known as jayg | 12:15 | |
| *** markvoelker has joined #puppet-openstack | 12:17 | |
| *** markvoelker has quit IRC | 12:22 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-murano: Update notification driver option https://review.openstack.org/298687 | 12:24 |
| *** michchap has quit IRC | 12:26 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-aodh: Update notifications options https://review.openstack.org/298702 | 12:27 |
| *** darkxploit has joined #puppet-openstack | 12:27 | |
| *** ducttape_ has quit IRC | 12:28 | |
| *** alee has quit IRC | 12:28 | |
| mjblack | anyone using ceilometer with the same rabbitmq as all the other components? | 12:32 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-ceilometer: Update notifications options https://review.openstack.org/298705 | 12:32 |
| EmilienM | mjblack: our ci does | 12:37 |
| *** jaosorior has quit IRC | 12:38 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-cinder: Update notifications options https://review.openstack.org/298708 | 12:38 |
| *** jaosorior has joined #puppet-openstack | 12:39 | |
| *** jpena is now known as jpena|lunch | 12:39 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-murano: Update notification driver option https://review.openstack.org/298687 | 12:39 |
| *** degorenko is now known as _degorenko|afk | 12:41 | |
| *** darkxploit has quit IRC | 12:42 | |
| *** jaosorior has quit IRC | 12:42 | |
| *** jaosorior has joined #puppet-openstack | 12:43 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge https://review.openstack.org/296102 | 12:45 |
| openstackgerrit | Michael Polenchuk proposed openstack/puppet-glance: [WIP] Add image properties support to glance_image https://review.openstack.org/298718 | 12:46 |
| *** morazi has joined #puppet-openstack | 12:51 | |
| *** michchap has joined #puppet-openstack | 12:53 | |
| openstackgerrit | Matthew J Black proposed openstack/puppet-keystone: domain backend drivers set in domain config https://review.openstack.org/298672 | 12:55 |
| *** dprince has quit IRC | 13:01 | |
| openstackgerrit | Dimitri Savineau proposed openstack/puppet-tripleo: Enable HAProxy forwardfor option for Horizon. https://review.openstack.org/296497 | 13:03 |
| *** dprince has joined #puppet-openstack | 13:03 | |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: WIP: multi-domain testing with ldap backend. https://review.openstack.org/296370 | 13:03 |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-nova: Use oslo module for messaging (rabbit) configuraion https://review.openstack.org/292309 | 13:07 |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-ceilometer: Add 'ceilometer-service' to collector and agent/notification https://review.openstack.org/298733 | 13:08 |
| *** ayoung has joined #puppet-openstack | 13:09 | |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-ceilometer: Add 'ceilometer-service' tag to collector and agent/notification https://review.openstack.org/298733 | 13:11 |
| *** foxhunt_away is now known as foxhunt72 | 13:14 | |
| *** markvoelker has joined #puppet-openstack | 13:18 | |
| *** alee has joined #puppet-openstack | 13:19 | |
| openstackgerrit | Matthew J Black proposed openstack/puppet-horizon: Added ensure_resource for python-memcache package. https://review.openstack.org/298679 | 13:19 |
| *** markvoelker has quit IRC | 13:22 | |
| *** markvoelker has joined #puppet-openstack | 13:23 | |
| *** markvoelker_ has joined #puppet-openstack | 13:28 | |
| *** markvoelker has quit IRC | 13:32 | |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: WIP: multi-domain testing with ldap backend. https://review.openstack.org/296370 | 13:34 |
| alee | EmilienM, hey - so I'm trying to figure out how to do multiopt parameters | 13:36 |
| alee | EmilienM, specifically, I have some parameters that I need to set using barbican_config() | 13:37 |
| EmilienM | we have a provider for it. | 13:37 |
| EmilienM | let me check if it landed | 13:38 |
| EmilienM | alee: https://review.openstack.org/#/c/234727/ | 13:38 |
| openstackgerrit | Merged openstack/puppet-barbican: Add barbican client class and params https://review.openstack.org/296156 | 13:39 |
| EmilienM | alee: it's used here: https://review.openstack.org/#/c/234728/ | 13:39 |
| alee | EmilienM, ah cool - that helps - thanks | 13:40 |
| EmilienM | alee: https://review.openstack.org/#/c/234728/10/spec/acceptance/neutron_config_spec.rb look the multiple values for thisshouldexist3 | 13:40 |
| *** jpena|lunch is now known as jpena | 13:47 | |
| chem | mjblack: hi, is that ok for you if I take over https://launchpad.net/bugs/1554555, now that I have i basic ldap test env https://review.openstack.org/296370 | 13:49 |
| openstack | Launchpad bug 1554555 in puppet-keystone "openstack cli provider needs to pass domain in v3 calls" [Undecided,In progress] - Assigned to Matthew J Black (mjblack) | 13:49 |
| mjblack | chem: sure | 13:50 |
| chem | mjblack: It would mean for you to abandon https://review.openstack.org/#/c/289995 | 13:50 |
| chem | mjblack: ack thanks :) | 13:50 |
| openstackgerrit | Matthew J Black proposed openstack/puppet-horizon: Added ensure_resource for python-memcache package. https://review.openstack.org/298679 | 13:50 |
| mjblack | chem: change has been abandoned | 13:51 |
| chem | mjblack: ack, going to make a placeholder for it right now | 13:52 |
| openstackgerrit | Merged openstack/puppet-barbican: Moved logging to barbican-api module https://review.openstack.org/296157 | 13:53 |
| openstackgerrit | Merged openstack/puppet-barbican: Add helper files to configure paste file and barbican.conf https://review.openstack.org/296158 | 13:54 |
| *** ducttape_ has joined #puppet-openstack | 13:55 | |
| *** dfisher has joined #puppet-openstack | 14:00 | |
| *** dhollinger has joined #puppet-openstack | 14:02 | |
| *** richm has joined #puppet-openstack | 14:02 | |
| *** xarses has quit IRC | 14:07 | |
| iberezovskiy | EmilienM, hi. I'm working to switch some modules to puppet-oslo usage. now master is opened for N, so can I drop deprecated parameters (which has warning ? | 14:11 |
| EmilienM | iberezovskiy: yes | 14:11 |
| iberezovskiy | cool, thx :) | 14:11 |
| EmilienM | no need of deprecation, since the module never got a release. | 14:12 |
| EmilienM | iberezovskiy: puppet-oslo is not used anywhere, right? | 14:12 |
| iberezovskiy | I mean I'll drop deprecated parameters from other modules which I am switching to oslo usage | 14:12 |
| iberezovskiy | e.g. glance and https://github.com/openstack/puppet-glance/blob/master/manifests/notify/rabbitmq.pp#L110 | 14:13 |
| EmilienM | oh right | 14:13 |
| EmilienM | drop it | 14:13 |
| iberezovskiy | great, thx | 14:14 |
| EmilienM | iberezovskiy: please start with *one* module, before doing all | 14:14 |
| EmilienM | so we can first review and adjust if needed puppet-oslo | 14:14 |
| iberezovskiy | EmilienM, nova is ready https://review.openstack.org/#/c/292309/ | 14:14 |
| EmilienM | ok will look today | 14:14 |
| iberezovskiy | I didn't add release notes, but I will | 14:15 |
| *** markvoelker_ has quit IRC | 14:15 | |
| iberezovskiy | EmilienM, ping me please, when you add puppet-oslo to tripleo, so I'll publish new patchset | 14:18 |
| EmilienM | hopefully today | 14:18 |
| iberezovskiy | good | 14:18 |
| openstackgerrit | Gonéri Le Bouder proposed openstack/puppet-ironic: expose the ipxe_enabled parameter https://review.openstack.org/296774 | 14:19 |
| *** dhollinger has quit IRC | 14:23 | |
| *** jmikolajczak has quit IRC | 14:32 | |
| *** xarses has joined #puppet-openstack | 14:37 | |
| openstackgerrit | Lokesh Jain proposed openstack/puppet-neutron: Support for Nuage Neutron plugin in puppet-neutron https://review.openstack.org/298792 | 14:43 |
| *** dhollinger has joined #puppet-openstack | 14:44 | |
| _degorenko|afk | .тшсл вупщкутлщ | 14:54 |
| _degorenko|afk | oops | 14:54 |
| _degorenko|afk | sorry | 14:54 |
| *** _degorenko|afk is now known as degorenko | 14:54 | |
| *** haplo37 has joined #puppet-openstack | 14:54 | |
| dmsimard | degorenko: well, that's not very useful lol https://translate.google.ca/?safe=off&bav=on.2,or.r_cp.&bvm=bv.117868183,d.dmo&biw=2560&bih=1325&dpr=1&um=1&ie=UTF-8&hl=en&client=tw-ob#uk/en/%D1%82%D1%88%D1%81%D0%BB%20%D0%B2%D1%83%D0%BF%D1%89%D0%BA%D1%83%D1%82%D0%BB%D1%89 | 14:57 |
| iurygregory | lol | 14:57 |
| iurygregory | ukrainian to english O.o | 14:57 |
| degorenko | dmsimard, don't worry :) that's just letters on keyboard under english, here is no any logic :) | 14:58 |
| EmilienM | meeting in 2 min! | 14:59 |
| EmilienM | or in 15s | 14:59 |
| iurygregory | or now | 14:59 |
| iurygregory | XD | 14:59 |
| *** dhollinger has quit IRC | 15:02 | |
| *** openstackgerrit has quit IRC | 15:06 | |
| *** openstackgerrit has joined #puppet-openstack | 15:07 | |
| *** dgurtner has joined #puppet-openstack | 15:07 | |
| *** dgurtner has quit IRC | 15:07 | |
| *** dgurtner has joined #puppet-openstack | 15:07 | |
| *** xingchao has joined #puppet-openstack | 15:12 | |
| *** haplo37 has quit IRC | 15:23 | |
| *** dgurtner has quit IRC | 15:25 | |
| *** dgurtner has joined #puppet-openstack | 15:25 | |
| *** rcernin has quit IRC | 15:27 | |
| dtantsur | EmilienM, per new backport policy, should I abandon https://review.openstack.org/293366 ? | 15:33 |
| EmilienM | dtantsur: not sure | 15:34 |
| EmilienM | I think we can backport it, let me some time to look at it today | 15:34 |
| dtantsur | last time it failed CI due to outdated centos ironic-inspector package | 15:34 |
| dtantsur | dunno if it got fixed | 15:34 |
| dtantsur | still old: http://mirror.centos.org/centos/7/cloud/x86_64/openstack-liberty/openstack-ironic-inspector-2.2.2-1.el7.noarch.rpm :( | 15:35 |
| *** saneax is now known as saneax_AFK | 15:36 | |
| EmilienM | dtantsur: we'll land your patch if CI is passing | 15:36 |
| dtantsur | got it, thanks | 15:37 |
| _ody | EmilienM: Wasn't paying atention there are the end of the meeting. Yeah things are gong fine. You can see I simplified the change set yesterday by a lot. | 15:43 |
| *** mdorman has joined #puppet-openstack | 15:43 | |
| EmilienM | excellent | 15:44 |
| _ody | I was inject more opinion than functionality so I backed off. | 15:44 |
| *** derekh has quit IRC | 15:44 | |
| *** haplo37 has joined #puppet-openstack | 15:44 | |
| *** xarses has quit IRC | 15:45 | |
| *** angdraug has joined #puppet-openstack | 15:46 | |
| mwhahaha | openstack image list --long doens't include properties :( | 15:46 |
| *** dgurtner has quit IRC | 15:46 | |
| *** dprince has quit IRC | 15:47 | |
| *** dprince has joined #puppet-openstack | 15:47 | |
| EmilienM | mwhahaha: good for you, you'll contribute to osclient :P | 15:51 |
| mwhahaha | i already have and it was painful :D | 15:51 |
| mwhahaha | and since it wont land until N it wouldn't be helpful in the case | 15:51 |
| mwhahaha | time for terrible workaroudns! | 15:51 |
| *** dtantsur is now known as dtantsur|afk | 15:53 | |
| EmilienM | mwhahaha: it's fair | 15:53 |
| EmilienM | mwhahaha: you want it in mitaka? | 15:54 |
| mwhahaha | would be ideal but i'm trying to see how much work it is | 15:54 |
| mwhahaha | basically trying to kill this thing: https://github.com/openstack/fuel-library/blob/master/deployment/puppet/osnailyfacter/modular/astute/upload_cirros.rb | 15:55 |
| mwhahaha | but need to be able to support --property <key>=<value> which i could add easily to the provider | 15:56 |
| mwhahaha | but i'd like to do it correctly | 15:56 |
| mwhahaha | :) so we'll see | 15:56 |
| mwhahaha | or at least somethings that's not completely awful | 15:57 |
| *** xarses has joined #puppet-openstack | 15:58 | |
| *** ducttape_ has quit IRC | 15:59 | |
| openstackgerrit | Merged openstack/puppet-barbican: Add barbican-api manifest https://review.openstack.org/296159 | 16:00 |
| *** samuelBartel has quit IRC | 16:00 | |
| *** ducttape_ has joined #puppet-openstack | 16:01 | |
| *** Guest58782 has quit IRC | 16:03 | |
| *** jtsigma has joined #puppet-openstack | 16:04 | |
| *** aimon has quit IRC | 16:05 | |
| degorenko | crinkle, hey :) i've answered on your questions here: https://review.openstack.org/226862 Can you please take a look? | 16:05 |
| *** jtomasek has quit IRC | 16:07 | |
| jtsigma | Hi, this is more a puppet question than openstack but I'm trying to understand the keystone resource, as an example in the puppet-heat module I see that the only code that has keystone_user_role is in the form of Resource Reference. I don't see the initial resource established. How does that work? | 16:08 |
| jtsigma | https://review.openstack.org/226862 | 16:08 |
| jtsigma | whoops ignore that review | 16:08 |
| jtsigma | ubuntu@W110ER:~/Downloads/puppet-heat$ grep -Ri keystone_user_role|grep -v spec | 16:08 |
| jtsigma | manifests/keystone/auth_cfn.pp: Keystone_user_role["${auth_name}@${tenant}"] ~> | 16:09 |
| jtsigma | manifests/keystone/auth.pp: Keystone_user_role["${auth_name}@${tenant}"] ~> | 16:09 |
| jtsigma | I thought that in order to use a Resource reference, you have to specify the resource somewhere first. Using keystone_user_role { "title": } | 16:09 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint https://review.openstack.org/298397 | 16:09 |
| crinkle | degorenko: done | 16:09 |
| degorenko | crinkle, thank you :) | 16:10 |
| openstackgerrit | Merged openstack/puppet-glance: Add posibility to configure Glance backend settings for Glare service https://review.openstack.org/298586 | 16:12 |
| mwhahaha | jtsigma: i think we change out the title to match user@project | 16:12 |
| jtsigma | yes , sorry i just put something random in there | 16:12 |
| *** jistr has quit IRC | 16:14 | |
| mwhahaha | jtsigma: i think we define that resource in https://github.com/openstack/puppet-heat/blob/master/manifests/keystone/auth_cfn.pp#L208-L223 | 16:14 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint https://review.openstack.org/298397 | 16:15 |
| mwhahaha | so we're cheating because we know that keystone::resource::service_identity will create a keystone_user_role["user@project"] | 16:15 |
| mwhahaha | https://github.com/openstack/puppet-keystone/blob/master/manifests/resource/service_identity.pp#L165-L170 | 16:15 |
| jtsigma | mwhahaha, very interesting! Wow, i would've never figured that out. Thanks saved me a ton of time | 16:15 |
| alee | EmilienM, yeehah! finally .. | 16:16 |
| jtsigma | it's interesting to me tho that altho the auth_cfn manifest is in keystone folder that when you call 'keystone::resource::service_identity' it knows it's not a relative class. I didn't think it was calling keystone module because i thought you would then have to specify it absolute like ::keystone::resource::service_identity | 16:17 |
| *** todin has joined #puppet-openstack | 16:17 | |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Added KMIP plugin settings to barbican-api manifest https://review.openstack.org/296163 | 16:17 |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Added keystone config to barbican api manifest https://review.openstack.org/263461 | 16:17 |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Added settings for cert and secret store plugins https://review.openstack.org/296162 | 16:17 |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Added apache component Add test and mods for deployment as an apache module https://review.openstack.org/263462 | 16:17 |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Add keystone notification options to barbican-api manifest https://review.openstack.org/296161 | 16:17 |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Added Dogtag settings for barbican-api https://review.openstack.org/258851 | 16:17 |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Add quota and secret limit parameters to barbican-api manifest https://review.openstack.org/296160 | 16:17 |
| mwhahaha | that's a scope determination thing in puppet, it's not a hard requirement unless you have overlapping names | 16:18 |
| mwhahaha | it's a best practice to use the global leading :: but we've got some places where that hasn't been adopted | 16:18 |
| jtsigma | ah i see. thanks again | 16:18 |
| alee | EmilienM, degorenko -- those last ones should have all required changes at least for the bottom two patches. | 16:18 |
| _ody | keystone::resource::service_identity is a defined resources. It probably won't accept a leading ::. | 16:19 |
| alee | still working on the one that requires multi-opt | 16:19 |
| mwhahaha | i think resources can have a leading :: | 16:20 |
| * mwhahaha shrugs | 16:20 | |
| * _ody shurgs too | 16:20 | |
| _ody | Never done with with resources. | 16:20 |
| _ody | We only ever had the scoping problem with classes...but maybe we just didn't notice for some reason. | 16:21 |
| mwhahaha | yea i could see the class scope thing being more common | 16:21 |
| jtsigma | that newest keystone module is giving me probs because on trusty i have openstack client 1.0.3 and so the new keystone module uses "openstack domain list" which apparently isn't an option for my openstack client. | 16:22 |
| *** dhollinger has joined #puppet-openstack | 16:23 | |
| mwhahaha | yea you have to match the module to the client version and vice versa | 16:23 |
| jtsigma | ooh ok | 16:23 |
| _ody | jtsigma: Could aslo be that you are using the wrong keystone api version. | 16:23 |
| jtsigma | yes i'm using api v2 | 16:23 |
| _ody | Yeah. That won't work for domains. | 16:23 |
| jtsigma | but even if i set the API_VERSION=3 in my openrc, openstack client v1.0.3 doesn't appear to accept that | 16:24 |
| jtsigma | i tested with a newer version on a diff pc an that appears to be ok. but on this particular system 1.0.3 it doesn't like | 16:24 |
| *** derekh has joined #puppet-openstack | 16:29 | |
| *** ansmith has joined #puppet-openstack | 16:30 | |
| *** Guest68910 is now known as mgagne | 16:38 | |
| *** mgagne has quit IRC | 16:38 | |
| *** mgagne has joined #puppet-openstack | 16:38 | |
| *** derekh has quit IRC | 16:41 | |
| *** jtomasek has joined #puppet-openstack | 16:49 | |
| openstackgerrit | Merged openstack/puppet-nova: Use OpenstackClient for nova providers auth https://review.openstack.org/226862 | 16:54 |
| *** jpena is now known as jpena|off | 16:55 | |
| *** foxhunt72 is now known as foxhunt_away | 16:56 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-oslo: Add ability to configure oslo_messaging_notifications https://review.openstack.org/298874 | 16:59 |
| *** mgagne_ has joined #puppet-openstack | 17:06 | |
| *** holser_ has quit IRC | 17:07 | |
| *** mihgen has quit IRC | 17:07 | |
| *** aglarendil has quit IRC | 17:07 | |
| *** igorbelikov has quit IRC | 17:07 | |
| *** dmsimard has quit IRC | 17:07 | |
| *** jasondotstar has quit IRC | 17:07 | |
| *** mjb has quit IRC | 17:07 | |
| *** mgagne has quit IRC | 17:07 | |
| *** delatte has quit IRC | 17:07 | |
| *** bastelfreak has quit IRC | 17:07 | |
| *** saneax_AFK has quit IRC | 17:07 | |
| *** llua has quit IRC | 17:07 | |
| *** _matthias_ has quit IRC | 17:07 | |
| *** Anticimex has quit IRC | 17:07 | |
| *** mfisch has quit IRC | 17:07 | |
| *** mihgen has joined #puppet-openstack | 17:07 | |
| *** delattec has joined #puppet-openstack | 17:07 | |
| *** trown is now known as trown|lunch | 17:07 | |
| alee | EmilienM, degorenko https://review.openstack.org/#/c/296160 and the next one up .. | 17:07 |
| alee | should be ready to go. | 17:07 |
| *** llua` has joined #puppet-openstack | 17:07 | |
| *** dmsimard1 has joined #puppet-openstack | 17:07 | |
| *** Anticime1 has joined #puppet-openstack | 17:07 | |
| *** _matthias__ has joined #puppet-openstack | 17:07 | |
| *** mfisch has joined #puppet-openstack | 17:07 | |
| *** mjb has joined #puppet-openstack | 17:07 | |
| *** mgagne_ has quit IRC | 17:07 | |
| *** mgagne_ has joined #puppet-openstack | 17:07 | |
| *** mfisch is now known as Guest28049 | 17:08 | |
| *** jasondotstar has joined #puppet-openstack | 17:08 | |
| *** bastelfreak has joined #puppet-openstack | 17:08 | |
| degorenko | _ody, hey, why you abandoned your patch to glance? i don't see any alternative solutions, can you point me? | 17:11 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint https://review.openstack.org/298397 | 17:12 |
| _ody | degorenko: It is here, https://github.com/openstack/puppet-glance/blob/master/manifests/keystone/auth.pp#L184-L187. | 17:12 |
| degorenko | _ody, ok. And btw, autorequire equals to collectors? | 17:13 |
| _ody | They basically work the same. | 17:14 |
| _ody | If they find a resource then they create the relationship, if they don't then nothing happens. | 17:14 |
| degorenko | _ody, hm, i though that they have strong dependency | 17:14 |
| _ody | Nah. | 17:15 |
| _ody | degorenko: Can you take a loot at this, https://github.com/openstack/puppet-glance/blob/master/manifests/keystone/auth.pp#L184-L203. | 17:15 |
| *** mgagne_ is now known as mgagne | 17:15 | |
| _ody | I think that is the reason I went down the path of fixing glance, our endpoint name endes up being different then the relationship. | 17:15 |
| degorenko | _ody, correct, you're right https://docs.puppetlabs.com/puppet/latest/reference/lang_relationships.html#auto-relationships | 17:15 |
| degorenko | thank you for info :) | 17:16 |
| * degorenko looking | 17:16 | |
| *** ducttape_ has quit IRC | 17:16 | |
| *** holser_ has joined #puppet-openstack | 17:16 | |
| *** saneax_AFK has joined #puppet-openstack | 17:17 | |
| *** igorbelikov has joined #puppet-openstack | 17:17 | |
| *** saneax_AFK is now known as saneax | 17:17 | |
| degorenko | _ody, so, for your link | 17:17 |
| degorenko | it may be not happen, yes | 17:17 |
| degorenko | when we have undefined service_type parameter | 17:17 |
| degorenko | _ody, take a look here: https://github.com/openstack/puppet-keystone/blob/master/manifests/resource/service_identity.pp#L183 | 17:18 |
| _ody | yeah. While confusing at first it looks like service_name is being passed through | 17:19 |
| *** aglarendil has joined #puppet-openstack | 17:19 | |
| degorenko | _ody, as for glance we always have default https://github.com/openstack/puppet-glance/blob/master/manifests/keystone/auth.pp#L110 | 17:19 |
| degorenko | same for service_name | 17:20 |
| degorenko | so, it should work on defaults | 17:21 |
| *** dmsimard1 is now known as dmsimard | 17:21 | |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 17:22 |
| *** igorbelikov has quit IRC | 17:22 | |
| *** igorbelikov has joined #puppet-openstack | 17:23 | |
| *** ducttape_ has joined #puppet-openstack | 17:31 | |
| EmilienM | alee: looking | 17:32 |
| alee | EmilienM, thanks | 17:32 |
| *** guimaluf has joined #puppet-openstack | 17:33 | |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-glance: Use oslo module for messaging (rabbit) configuration https://review.openstack.org/298893 | 17:35 |
| EmilienM | alee: for quotas, is it worth having quota.pp like in puppet-neutron? | 17:37 |
| EmilienM | we usually try to not overload classes | 17:37 |
| alee | EmilienM, looking | 17:38 |
| EmilienM | degorenko, alee: I have the feeling we put everything in api.pp | 17:39 |
| EmilienM | it's not a blocker for me | 17:39 |
| EmilienM | it's just an highlight of an error of design | 17:39 |
| degorenko | EmilienM, alee i will look in a few minutes :) | 17:39 |
| alee | EmilienM, it makes for a smaller module, but I think also means you have to specify more resources to get a server | 17:40 |
| alee | EmilienM, I could see going either way with this. | 17:41 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-nova: Add release note for OpenStack client in nova providers https://review.openstack.org/298898 | 17:41 |
| degorenko | EmilienM, sorry, missed that for my openstack client in nova patch :D ^^ | 17:41 |
| alee | EmilienM, can I suggest we consider this as a refinement patch after these go in? | 17:42 |
| EmilienM | well, the other way is to make it clean by design | 17:44 |
| *** delatte has joined #puppet-openstack | 17:47 | |
| EmilienM | alee: I commented https://review.openstack.org/#/c/296161/ for alignments | 17:47 |
| alee | EmilienM, I can go either way on this - it makes it more composable, but also means you need to define more things when creating a server. | 17:48 |
| EmilienM | right, but that's not a problem | 17:48 |
| EmilienM | a lot of puppet modules outside our forge are using a single class for everything, we don't like it much | 17:49 |
| EmilienM | degorenko: thoughts? | 17:49 |
| EmilienM | in fact, I'll -1 all the patches | 17:51 |
| *** delattec has quit IRC | 17:51 | |
| *** foxhunt_away is now known as foxhunt72 | 17:51 | |
| EmilienM | every patch you do is in api.pp | 17:51 |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-ceilometer: Add 'ceilometer-service' tag to collector and agent/notification https://review.openstack.org/298903 | 17:51 |
| EmilienM | alee: look at other modules, please, they are all splitted | 17:51 |
| *** foxhunt72 is now known as foxhunt_away | 17:51 | |
| EmilienM | in neutron, we have quota.pp, and also notifications.pp | 17:51 |
| degorenko | EmilienM, you're suggesting to have separate classes for some options? | 17:52 |
| EmilienM | yeah | 17:52 |
| EmilienM | for quotas, notifications, plugins, etc | 17:52 |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-ceilometer: Set defaultbranch for git-review to stable/liberty https://review.openstack.org/298906 | 17:52 |
| degorenko | well, in fact, we can have separate classes for configuring this, yes | 17:52 |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-ceilometer: Add 'ceilometer-service' tag to collector and agent/notification https://review.openstack.org/298908 | 17:53 |
| degorenko | if we have some complex configurations, yes, i'm +1 for this | 17:53 |
| alee | EmilienM, thats because all the parameters that are added there are parameters that are used by the api server. What do you think should be in the api.pp ? | 17:53 |
| degorenko | alee, you can use include from api then | 17:53 |
| *** gfidente has quit IRC | 17:53 | |
| EmilienM | alee: this is not the right way to think | 17:53 |
| EmilienM | alee: a manifest is not a service | 17:53 |
| degorenko | well, it is same situation as for logging and db classes | 17:53 |
| *** delattec has joined #puppet-openstack | 17:53 | |
| EmilienM | it can also be a set of parameters | 17:53 |
| *** foxhunt_away is now known as foxhunt72 | 17:53 | |
| EmilienM | I don't want to end-up with a api.pp of 4000 lines | 17:53 |
| EmilienM | and this is happenning | 17:54 |
| degorenko | +100500 | 17:54 |
| degorenko | we can have some basic options for api class, but yeah, i'm also against to keep all possible options in one class | 17:55 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario002/centos7: switch RabbitMQ and OpenStack to IPv6 https://review.openstack.org/287521 | 17:55 |
| EmilienM | crinkle: commit message updated ^ | 17:55 |
| *** guimaluf has quit IRC | 17:55 | |
| openstackgerrit | Cody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint https://review.openstack.org/298397 | 17:55 |
| EmilienM | degorenko: specially the plugins | 17:56 |
| *** delatte has quit IRC | 17:56 | |
| alee | EmilienM, so what do you think should be in api.pp ? | 17:57 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 17:57 |
| EmilienM | alee: what is currently in there, is fine | 17:57 |
| EmilienM | but no more, imho | 17:57 |
| _ody | crinkle or EmilienM: do either of you have the openstack forge account password? | 17:58 |
| alee | EmilienM, degorenko I dont mind splitting it out -- I'm just trying to understand the rationale for a separate manifest as opposed to the api manifest | 17:58 |
| EmilienM | _ody: I have it | 17:59 |
| EmilienM | _ody: let me fix that | 17:59 |
| degorenko | alee, well, you should have for such classes only important options | 17:59 |
| degorenko | alee, if you need improve configuration with many options (with one logic) - create a new class | 18:00 |
| degorenko | alee, as i said above - it is exactly same situation as for logging and db classes | 18:00 |
| *** hjensas has joined #puppet-openstack | 18:03 | |
| alee | degorenko, EmilienM ok -- so for instance , you would say a plugins.pp with the parameters that determine which plugins are enabled, and then a dogtag.pp for the dogtag plugin, kmip.pp for the kmip plugin etc. ? | 18:03 |
| EmilienM | or you can have manifests/plugins/foo.pp | 18:04 |
| EmilienM | and manifests/plugins/bar.pp | 18:04 |
| degorenko | +1 for ^ | 18:04 |
| EmilienM | https://github.com/openstack/puppet-neutron/tree/master/manifests | 18:04 |
| EmilienM | alee: please look how we do in other modules | 18:05 |
| *** degorenko is now known as _degorenko|afk | 18:06 | |
| *** llua` is now known as llua | 18:06 | |
| *** llua has joined #puppet-openstack | 18:06 | |
| EmilienM | _ody: https://review.openstack.org/#/c/298530/7/run_tests.sh sounds like a huge hack, can't we live without that? | 18:07 |
| alee | EmilienM, ok | 18:08 |
| *** cdelatte has joined #puppet-openstack | 18:08 | |
| iberezovskiy | EmilienM, looks I need your advice again :) I didn't find rabbit_notification_exchange parameter in glance sample config, but we set it https://github.com/openstack/puppet-glance/blob/master/manifests/notify/rabbitmq.pp#L144. it looks like it should be 'controler_exchange'. Am I right? | 18:09 |
| EmilienM | iberezovskiy: should be in oslo messaging, isn't? | 18:10 |
| iberezovskiy | control_exchange is in default section | 18:10 |
| iberezovskiy | but I wanna set it from oslo::messaging_rabbit class | 18:11 |
| *** delattec has quit IRC | 18:11 | |
| EmilienM | wait | 18:12 |
| _ody | EmilienM: We could make symlinks into a directory that is in secure path, as the alternative. | 18:12 |
| EmilienM | why do we have oslo::messaging_rabbit and not oslo::messaging::rabbit? | 18:12 |
| *** trown|lunch is now known as trown | 18:12 | |
| iberezovskiy | https://github.com/openstack/puppet-oslo/commit/b86091eb6fa5bbca7931d6407cf3e2b1c0cf3f3d | 18:13 |
| _ody | I could also maybe re-arrange the order we define variables in run_tests.sh and use the full path to puppet, which might work. | 18:13 |
| *** cdelatte has quit IRC | 18:13 | |
| EmilienM | iberezovskiy: I just realized. | 18:13 |
| EmilienM | while we're building this module, it might be good to rename the class | 18:14 |
| iberezovskiy | np, I'll rename | 18:14 |
| EmilienM | it would be very clear to have oslo::messaing::<driver> | 18:14 |
| iberezovskiy | yep, like it's done for oslo::messaging::amqp | 18:14 |
| EmilienM | xingchao: ^ fyi | 18:14 |
| EmilienM | iberezovskiy: ok, please rename it and update the puppet-nova patch -using depends-on | 18:15 |
| EmilienM | we're very close! thanks for your efforts | 18:15 |
| iberezovskiy | sure | 18:15 |
| *** dilyin has joined #puppet-openstack | 18:16 | |
| EmilienM | mwhahaha: dilyin and I are working together on merging puppet-pacemaker efforts, see https://review.openstack.org/#/c/296440/ | 18:16 |
| EmilienM | mwhahaha: it would be great to run Fuel CI jobs on openstack/puppet-pacemaker, somehow maybe an external job | 18:17 |
| xingchao | EmilienM: get it | 18:17 |
| mwhahaha | yea we can probably get that up when we've got the two merged together | 18:17 |
| EmilienM | mwhahaha: so we can iterate on the merging without breaking both tripleo & fuel | 18:17 |
| EmilienM | xingchao: thx | 18:17 |
| dilyin | i can run unit tests already | 18:17 |
| EmilienM | dilyin: mhh I would prefer also have functional tests | 18:17 |
| dilyin | there is a job for them | 18:17 |
| EmilienM | and having fuel CI running would help us | 18:17 |
| *** cdelatte has joined #puppet-openstack | 18:18 | |
| *** jaosorior has quit IRC | 18:18 | |
| mwhahaha | hmm ok would probably need a special job for that | 18:18 |
| dilyin | yes, i'll adapt beaker tests too | 18:18 |
| EmilienM | mwhahaha: yes | 18:18 |
| iberezovskiy | igorbelikov, ^ | 18:18 |
| EmilienM | mwhahaha: an HA job that use openstack/puppet-pacemaker instead of fuel-infra/puppet-pacemaker | 18:18 |
| EmilienM | if I'm not wrong | 18:18 |
| mwhahaha | yea | 18:18 |
| EmilienM | cool. So we can iterate, and keep testing it. | 18:19 |
| dilyin | do we have beaker support in openstack CI? | 18:19 |
| EmilienM | dilyin: of course | 18:19 |
| dilyin | then we're cool it i'll be supporting acceptance tests | 18:19 |
| EmilienM | on both trusty & centos7. chem already did the work on centos7 for openstack/puppet-pacemaker -> https://review.openstack.org/#/c/281376/ | 18:19 |
| EmilienM | cool | 18:20 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Avoid secure_path https://review.openstack.org/298530 | 18:20 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 18:20 |
| EmilienM | dilyin: chem is currently the guy who actively works on the module, you might sync with him tomorrow | 18:20 |
| dilyin | ok, i will | 18:21 |
| openstackgerrit | Merged openstack/puppet-ceilometer: Add 'ceilometer-service' tag to collector and agent/notification https://review.openstack.org/298733 | 18:21 |
| EmilienM | _ody: I like it better :) | 18:21 |
| _ody | EmilienM: Lets give the full path thing a go. | 18:22 |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-oslo: Fix define name to oslo::messaging::rabbit https://review.openstack.org/298923 | 18:22 |
| chem | EmilienM: dilyin (back for a few moments) we have already sync a bit ... for the beaker jobs on ubuntu it's only expecting a failure (as pcs is not there) but we only have to remove the case and see if it works on both ubuntu and centos: https://github.com/openstack/puppet-pacemaker/blob/master/spec/acceptance/basic_one_node_spec.rb#L64-L67 | 18:25 |
| chem | EmilienM: dilyin but I'm using pcs command ... so :) | 18:25 |
| EmilienM | we'll have to hack on XML, probably | 18:25 |
| EmilienM | jayg: fyi ^ | 18:26 |
| *** iberezovskiy is now known as iberezovskiy_afk | 18:26 | |
| chem | EmilienM: dilyin it's only from https://github.com/openstack/puppet-pacemaker/blob/master/spec/acceptance/basic_one_node_spec.rb#L34 on that I use pcs to check the resource further. The first tests are comptatible | 18:26 |
| EmilienM | jayg: tl; dr: we're about to merge both puppet-pacemaker from fuel & tripleo - blocker is pcs does not exist on ubuntu - see https://review.openstack.org/#/c/296440/ | 18:27 |
| *** iurygregory has quit IRC | 18:29 | |
| EmilienM | dilyin, mwhahaha, chem: in term of actions: fuel can create a separated CI job for testing openstack/puppet-pacemaker with Fuel. What can we do in our side? | 18:37 |
| *** iurygregory has joined #puppet-openstack | 18:37 | |
| dilyin | yes, but it will be possible only when i finish merging, it will require some changes in fuel too | 18:37 |
| *** angdraug has quit IRC | 18:44 | |
| igorbelikov | EmilienM, mwhahaha, dilyin: we can enable fuel-ci for puppet-pacemaker by just adding it to the list of projects fuel-ci watches. It doesn’t matter if puppetfile points to fuel-infra instead of openstack, CI jobs will pick up the change | 18:46 |
| EmilienM | igorbelikov: how? | 18:46 |
| igorbelikov | EmilienM: it will fetch the change from gerrit before firing up librarian-puppet-simple, so librarian will skip it | 18:49 |
| EmilienM | mhh ok | 18:49 |
| EmilienM | I'll trust you on this one :-) | 18:49 |
| *** xingchao has quit IRC | 18:50 | |
| EmilienM | crinkle: https://github.com/openstack/puppet-neutron/commit/bb9ec5d7d30053cac232ec6626f431b4ed44c80d I think it also applis to OVS agent and all oslo messaging params | 18:55 |
| crinkle | EmilienM: huh you're probably right | 18:57 |
| EmilienM | I'm sending a path | 18:58 |
| EmilienM | patch | 18:58 |
| *** angdraug has joined #puppet-openstack | 18:58 | |
| EmilienM | mflobo: I created launchpad Newton serie | 18:59 |
| EmilienM | err mfish | 18:59 |
| EmilienM | he's not around | 18:59 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-release-tools: fix puppet-openstack-spec-helper name https://review.openstack.org/298939 | 19:00 |
| *** ducttape_ has quit IRC | 19:00 | |
| *** ducttape_ has joined #puppet-openstack | 19:01 | |
| *** Guest28049 is now known as mfisch | 19:01 | |
| *** mfisch is now known as Guest82454 | 19:01 | |
| EmilienM | Guest82454: ^ | 19:02 |
| EmilienM | crinkle: I'm wondering if subscribing to the whole Neutron_config is safe or not | 19:02 |
| EmilienM | because it's not only rabbit options, but also logging, etc | 19:02 |
| *** Guest82454 has quit IRC | 19:02 | |
| *** Guest82454 has joined #puppet-openstack | 19:02 | |
| *** Guest82454 is now known as mfisch | 19:03 | |
| mfisch | whats up | 19:03 |
| EmilienM | mfisch: I updated launchpads | 19:03 |
| EmilienM | mfisch: we have 9.0.0 serie open | 19:03 |
| EmilienM | look https://launchpad.net/puppet-keystone | 19:03 |
| crinkle | EmilienM: yeah idunno, i'd rather not have the agent restarting all the time but i also don't know what it needs to update on | 19:03 |
| EmilienM | crinkle: at least: amqp & logging options, but maybe more. | 19:04 |
| EmilienM | it's a lot of crappy code to create this list | 19:04 |
| EmilienM | crinkle: I could take your linuxbridge_agent_subscribe and create an array of all the params that we want, but it's crappy to me | 19:05 |
| EmilienM | we'll have to maintain that list | 19:05 |
| crinkle | yeah that sounds gross | 19:06 |
| crinkle | but i worry about ovs agent restarting all the time | 19:06 |
| crinkle | that will make an unhappy cloud | 19:06 |
| EmilienM | crinkle: since we're moving things to puppet-oslo, we can (later) subscribe on puppet-oslo changes | 19:06 |
| EmilienM | that is true, restarting agent sucks | 19:06 |
| crinkle | ¯\_(ツ)_/¯ | 19:07 |
| EmilienM | crinkle: I'll think about it again when we have puppet-oslo in place | 19:07 |
| crinkle | i don't have a good suggestion | 19:07 |
| crinkle | okay | 19:07 |
| clayton | I got stuck on this issue recently | 19:08 |
| clayton | I was working on adding hooks support for puppet-neutron, and really it should be subscribing to neutron_config | 19:09 |
| clayton | but it's clearly risky given the current tstate | 19:09 |
| EmilienM | crinkle: see https://review.openstack.org/#/c/292309/7/manifests/init.pp | 19:09 |
| EmilienM | maybe can we subscribe to the define? I'm not sure | 19:09 |
| clayton | I've got two patches in progress and hynekm has another to fix the issue | 19:09 |
| EmilienM | clayton: what are the links, please? | 19:10 |
| clayton | I'm expecting it'll land for newton and get backported at least to mitaka | 19:10 |
| clayton | this is the bug: https://bugs.launchpad.net/neutron/+bug/1514056 | 19:10 |
| openstack | Launchpad bug 1514056 in neutron "Restarting OVS agent drops VMs traffic when using VLAN provider bridges" [High,In progress] - Assigned to Hynek Mlnarik (hmlnarik-s) | 19:10 |
| clayton | with my two patches + hynek's patch, ovs agent restarts don't cause traffic drops | 19:10 |
| EmilienM | nice | 19:11 |
| clayton | this is the last core openstack service I know of that can't be gracefully restarted | 19:11 |
| EmilienM | clayton: what about l3 agent? | 19:12 |
| clayton | l3 agent has been safe for a while afik | 19:12 |
| clayton | although there are parallelization issues with restarting them on multiple boxes until mitaka | 19:12 |
| clayton | so if you're running a lot of l3 agents and restart all of them in a short period of time, you're going to have problems | 19:13 |
| clayton | but that's not really a problem puppet can solve, and we're dealing with mitaka anyway | 19:13 |
| clayton | well, we = people working on puppet module master branch only have to worry about mitaka+ :)( | 19:14 |
| clayton | I'm still trying to get us on liberty | 19:14 |
| EmilienM | clayton: you're running liberty now? | 19:14 |
| clayton | cinder, nova, neutron and glance are still on liberty | 19:14 |
| clayton | err, no | 19:14 |
| clayton | cinder, nova, neutron and glance are still on kilo | 19:14 |
| *** brandon has joined #puppet-openstack | 19:15 | |
| clayton | other services are on liberty | 19:15 |
| clayton | mfisch is rolling keystone up to liberty this week | 19:15 |
| EmilienM | nice | 19:15 |
| clayton | we probably won't seriously look at anything on mitaka until after the summit. probably heat or keystone first | 19:15 |
| mfisch | not keystone | 19:16 |
| clayton | given some of the bugs that got fixed late in the liberty stable cycle for neutron and nova, I'm not excited about rushing into mitaka | 19:16 |
| mfisch | not until perf is fixed | 19:16 |
| mfisch | heat after a real RC is out | 19:17 |
| mfisch | rc2 rolled this morning | 19:17 |
| clayton | we'll finish our liberty upgrades and finish moving neutron, glance and cinder into docker, and probably just cherry-pick fixes for a while as we need them | 19:17 |
| *** ericksonsantos has quit IRC | 19:23 | |
| *** iurygregory has quit IRC | 19:24 | |
| *** dalees` has quit IRC | 19:25 | |
| openstackgerrit | Lokesh Jain proposed openstack/puppet-neutron: Support for Nuage Neutron plugin in puppet-neutron https://review.openstack.org/298792 | 19:26 |
| EmilienM | mwhahaha: can you please look https://review.openstack.org/#/c/287521/ when you can? i'll follow up with other scenarios asap | 19:29 |
| mwhahaha | k | 19:29 |
| *** dalees` has joined #puppet-openstack | 19:38 | |
| *** jmikolajczak has joined #puppet-openstack | 19:39 | |
| igorbelikov | EmilienM, mwhahaha: fuel-ci now runs for puppet-pacemaker | 19:45 |
| mwhahaha | cool thanks | 19:46 |
| *** ducttape_ has quit IRC | 19:47 | |
| EmilienM | igorbelikov: thx a lot :-) | 19:48 |
| *** johnmilton has quit IRC | 19:53 | |
| *** ducttape_ has joined #puppet-openstack | 19:58 | |
| alee | EmilienM, I notice that in https://review.openstack.org/#/c/234728 , there are no unit tests in spec/unit/type/ and spec/unit/provider .. | 19:59 |
| alee | EmilienM, is there a template for one somewhere that I can copy in/modify ? | 19:59 |
| EmilienM | not afik | 20:00 |
| EmilienM | but acceptance + rspec is enough, for now | 20:00 |
| alee | EmilienM, ok | 20:00 |
| *** iurygregory has joined #puppet-openstack | 20:00 | |
| *** ducttape_ has quit IRC | 20:01 | |
| *** ducttape_ has joined #puppet-openstack | 20:02 | |
| *** jmikolajczak has quit IRC | 20:12 | |
| *** ericksonsantos has joined #puppet-openstack | 20:19 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: scenario002/centos7: switch RabbitMQ and OpenStack to IPv6 https://review.openstack.org/287521 | 20:19 |
| EmilienM | w00t | 20:20 |
| *** imcsk8 is now known as imcsk8|afk | 20:20 | |
| _ody | annoying sporadic CI failures today. | 20:20 |
| *** xingchao has joined #puppet-openstack | 20:21 | |
| EmilienM | _ody: which ones? | 20:21 |
| _ody | p-o-i. Package related the ones I've seen. | 20:21 |
| EmilienM | _ody: can you give me details please? logs? link? | 20:22 |
| _ody | Yeah. Getting. | 20:22 |
| EmilienM | I'm trying to define a list of issues | 20:22 |
| EmilienM | and kill them | 20:22 |
| EmilienM | one of them is: stop using EPEL | 20:22 |
| _ody | heh. | 20:22 |
| EmilienM | EPEL is quite often down | 20:22 |
| EmilienM | dmsimard: do you have some workaround to not use EPEL in our scenario001 ? | 20:23 |
| _ody | EmilienM: One example, http://logs.openstack.org/30/298530/8/check/gate-puppet-openstack-integration-3-scenario001-tempest-devstack-centos7/7876eaf/console.html#_2016-03-29_18_26_34_127 | 20:23 |
| *** foxhunt72 is now known as foxhunt_away | 20:23 | |
| dmsimard | EmilienM: For Ceph ? | 20:24 |
| EmilienM | dmsimard: yeah | 20:24 |
| dmsimard | EmilienM: It's not a pretty workaround. | 20:25 |
| EmilienM | lol | 20:25 |
| *** xingchao has quit IRC | 20:25 | |
| dmsimard | EmilienM: I explained the issue here: https://bugs.launchpad.net/puppet-ceph/+bug/1562998 | 20:25 |
| openstack | Launchpad bug 1562998 in puppet-ceph "EPEL requirement on CentOS is only because of leveldb" [Undecided,New] | 20:25 |
| dmsimard | I guess you could not use ceph::repo and instead set up the CentOS Storage SIG repo instead | 20:26 |
| dmsimard | The dependencies are self-contained | 20:26 |
| _ody | EmilienM: Other seems to be related to the job getting killed because it has ran for over 60 minutes. | 20:26 |
| EmilienM | dmsimard: _ody's issue is likely a RDO server downtime, isn't? | 20:26 |
| EmilienM | _ody: ouch... where does it timeout? It takes time to investigate and finds the root issue. | 20:26 |
| dmsimard | We haven't had downtime recently that I know about | 20:26 |
| EmilienM | "No more mirrors to try." | 20:26 |
| _ody | EmilienM: http://logs.openstack.org/57/296557/24/check/gate-puppet-openstack-integration-3-scenario001-tempest-ubuntu-trusty/c74c92b/console.html#_2016-03-29_19_43_19_232 | 20:26 |
| _ody | That is the other type of failure I am seeing | 20:27 |
| EmilienM | wait | 20:27 |
| EmilienM | python-pandas-0.17.0-2.el7.x86_64: [Errno 256] No more mirrors to try. | 20:27 |
| EmilienM | it's in EPEL? no..? | 20:27 |
| dmsimard | pandas is in the RDO repos | 20:27 |
| EmilienM | ok | 20:27 |
| dmsimard | delorean-deps for trunk | 20:27 |
| EmilienM | so yeah, you don't know it but RDO has also some downtimes recently | 20:27 |
| EmilienM | _ody: that's a git server issue | 20:27 |
| EmilienM | _ody: look the time it takes to checkout repos | 20:27 |
| EmilienM | _ody: when it happens, please jump on #openstack-infra and say it | 20:28 |
| EmilienM | it's likely an issue with infra servers | 20:28 |
| *** jtomasek_ has joined #puppet-openstack | 20:30 | |
| *** jtomasek_ has quit IRC | 20:32 | |
| *** ansmith has quit IRC | 20:48 | |
| *** jmikolajczak has joined #puppet-openstack | 20:50 | |
| EmilienM | _ody: w00t, scenario003 is passing puppet4 ? https://review.openstack.org/#/c/296557/ | 20:51 |
| openstackgerrit | Merged openstack/puppet-cinder: Establish cinder_type after keystone_endpoint https://review.openstack.org/298397 | 20:53 |
| *** imcsk8|afk is now known as imcsk8 | 20:53 | |
| EmilienM | dmsimard: ok so I guess we can stop using EPEL and use http://cbs.centos.org/repos/storage7-ceph-hammer-candidate/x86_64/os/Packages/ | 20:53 |
| EmilienM | the main motivation is to stop using EPEL | 20:54 |
| *** ayoung has quit IRC | 20:54 | |
| _ody | EmilienM: I think other might be passing. I've seen the whole 60 minutes thing on those jobs. | 21:02 |
| *** jayg is now known as jayg|g0n3 | 21:04 | |
| dmsimard | EmilienM: it kinda sucks but yeah | 21:05 |
| dmsimard | EmilienM: I believe there's a package you can install that sets up the repo | 21:05 |
| dmsimard | like yum install centos-release-ceph | 21:06 |
| dmsimard | or something | 21:06 |
| EmilienM | yeah? that would be cool | 21:06 |
| dmsimard | the ceph repo was added fairly recently yet so I don't know if it's there yet | 21:06 |
| dmsimard | checking | 21:06 |
| *** trown is now known as trown|outtypewww | 21:07 | |
| dmsimard | EmilienM: well isn't that convenient: http://paste.openstack.org/show/492362/ | 21:08 |
| EmilienM | dmsimard: :D | 21:09 |
| EmilienM | i'll test it after cross project meeting | 21:09 |
| *** dprince has quit IRC | 21:19 | |
| *** xingchao has joined #puppet-openstack | 21:22 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-glance: Add ability to set properties with glance_image https://review.openstack.org/298718 | 21:23 |
| openstackgerrit | Matt Fischer proposed openstack/puppet-openstacklib: Don't claim you are retrying when you are not https://review.openstack.org/298992 | 21:27 |
| *** xingchao has quit IRC | 21:27 | |
| mfisch | EmilienM: check out that awesome bug fix ^ | 21:29 |
| mfisch | retryin' was lyin' | 21:29 |
| mwhahaha | lol | 21:30 |
| mwhahaha | WHY YOU LIE TO ME! | 21:30 |
| EmilienM | I like mfisch commits | 21:30 |
| EmilienM | it usually reminds me how life is beautiful | 21:31 |
| EmilienM | and how much we have fun | 21:31 |
| mfisch | that threw me off for a bit | 21:31 |
| mfisch | debugging something else | 21:31 |
| mfisch | also FYI I'm seeing some cases on VMs where liberty keystone + OSC takes > 20 seconds to make an endpoint | 21:32 |
| mfisch | I'd like to bump that timeout | 21:32 |
| mfisch | under no other load this box takes 12s | 21:32 |
| mwhahaha | osc isn't the fastest thing in the world | 21:33 |
| mwhahaha | most of it is just loading all the client libs | 21:33 |
| mfisch | I think maybe somethineg like if it's a no retry we wait 60s | 21:33 |
| mfisch | dilyin: I think we should change this code so that if it's a no retry code we just wait 60s total | 21:42 |
| mfisch | right now we abort at 20s | 21:42 |
| mfisch | retry code would do retries up to 60s, no retry would just wait 60s | 21:42 |
| mwhahaha | is it not configurable? | 21:43 |
| mfisch | no | 21:43 |
| dilyin | can you point me to this code? | 21:43 |
| mwhahaha | i think we actually increased it | 21:43 |
| mwhahaha | it used to be less than what it was now | 21:43 |
| mfisch | i1s | 21:43 |
| *** morazi has quit IRC | 21:43 | |
| mfisch | https://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack.rb#L99-L105 | 21:43 |
| mfisch | if we just indent line 103 below line 101 will it work the way we want? | 21:43 |
| mfisch | basically dont abort the no-retry unless we waited the full 60 | 21:44 |
| mwhahaha | https://github.com/openstack/puppet-openstacklib/commit/701b6fe0acf56e9132e9b0a448f28670c714cedb | 21:44 |
| mwhahaha | it used to be 10 sec :D | 21:44 |
| mfisch | so its not really right | 21:44 |
| mfisch | I mean 20s is the hard cap on non-retry | 21:44 |
| mfisch | we should wait 60s for every command | 21:44 |
| mfisch | but only retry osme | 21:44 |
| mfisch | I'll post a swag on this | 21:45 |
| openstackgerrit | Matt Fischer proposed openstack/puppet-openstacklib: Wait longer on non-retry calls https://review.openstack.org/298992 | 21:46 |
| dilyin | look like it waits for 20 second for a command to finish | 21:46 |
| mfisch | somethingl ike that ^ | 21:46 |
| dilyin | then if it have failed waits for 3 sec | 21:46 |
| mfisch | it does not | 21:47 |
| dilyin | and tries again | 21:47 |
| mfisch | not for no-retry commands | 21:47 |
| mfisch | thats the issue | 21:47 |
| dilyin | but will fail after 60 sec if no success | 21:47 |
| mfisch | for example endpoint create takes > 20 seconds it just fails | 21:47 |
| dilyin | so yo want to set command timeout to 60 for non-retry comamnds& | 21:47 |
| mfisch | y | 21:47 |
| mfisch | see if that diff makes any sense | 21:47 |
| mfisch | I have cases where endpoint create > 20 seconds :* | 21:48 |
| mfisch | :( | 21:48 |
| dilyin | meybe just raise command timeout | 21:48 |
| *** ayoung has joined #puppet-openstack | 21:48 | |
| mfisch | will this code handle a case where something hasnt returned yet? | 21:49 |
| mfisch | thats what I dont know | 21:49 |
| mfisch | raising the timeout is also a solution | 21:49 |
| mfisch | so the command will have already failed, my idea would need more work | 21:49 |
| mfisch | I think we have to raise the timeout | 21:50 |
| dilyin | # timeout the openstack command | 21:51 |
| dilyin | # after this number of seconds | 21:51 |
| dilyin | # retry the command until the request_timeout | 21:51 |
| dilyin | def self.command_timeout(action=nil) | 21:51 |
| dilyin | return 60 if no_retry_actions.include action | 21:51 |
| dilyin | 20 | 21:51 |
| dilyin | end | 21:51 |
| dilyin | # run the openstack command | 21:51 |
| dilyin | # with command_timeout | 21:51 |
| dilyin | def self.openstack(*args) | 21:52 |
| dilyin | begin | 21:52 |
| dilyin | action = args[1] | 21:52 |
| dilyin | Timeout.timeout(command_timeout action) do | 21:52 |
| dilyin | openstack_command *args | 21:52 |
| dilyin | end | 21:52 |
| mwhahaha | oh noes | 21:52 |
| dilyin | rescue Timeout::Error | 21:52 |
| dilyin | raise Puppet::ExecutionFailure, "Command: 'openstack #{args.inspect}' has been running for more then #{command_timeout} seconds!" | 21:52 |
| dilyin | end | 21:52 |
| dilyin | end | 21:52 |
| dilyin | something like this? | 21:52 |
| openstackgerrit | Matt Fischer proposed openstack/puppet-openstacklib: Adjust timeout for non-retry calls https://review.openstack.org/298992 | 21:53 |
| mfisch | spasiba | 21:53 |
| mfisch | look right? ^ | 21:53 |
| dilyin | set include? there | 21:53 |
| mfisch | where? | 21:54 |
| dilyin | and you forgot action | 21:54 |
| dilyin | and command_timeout action | 21:54 |
| mfisch | got that | 21:54 |
| openstackgerrit | Matt Fischer proposed openstack/puppet-openstacklib: Adjust timeout for non-retry calls https://review.openstack.org/298992 | 21:54 |
| mfisch | oh I see what you mean | 21:55 |
| dilyin | but still maybe set comamnd timeot to 60 and request_timeout to 300 | 21:55 |
| mfisch | I think this is probably a good start | 21:55 |
| openstackgerrit | Matt Fischer proposed openstack/puppet-openstacklib: Adjust timeout for non-retry calls https://review.openstack.org/298992 | 21:55 |
| mfisch | added the include? | 21:55 |
| mfisch | I need to run but thanks for the assist can you +1/-1 it? | 21:55 |
| dilyin | action = args[1] | 21:56 |
| mfisch | (action=nil) thats not enough? | 21:56 |
| mfisch | I'm not super good at ruby | 21:56 |
| mfisch | begin | 21:56 |
| mfisch | action = args[1] | 21:56 |
| mfisch | Timeout.timeout(command_timeout action) do | 21:57 |
| openstackgerrit | Matt Fischer proposed openstack/puppet-openstacklib: Adjust timeout for non-retry calls https://review.openstack.org/298992 | 21:57 |
| mfisch | I will test this later | 21:57 |
| mfisch | ttyl | 21:57 |
| *** haplo37 has quit IRC | 22:07 | |
| *** markvoelker has joined #puppet-openstack | 22:13 | |
| *** mdorman has quit IRC | 22:16 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: repos: bump rdo to mitaka/current-passed-ci https://review.openstack.org/299005 | 22:17 |
| EmilienM | dmsimard: fyi ^ | 22:17 |
| *** zino has left #puppet-openstack | 22:20 | |
| *** markvoelker has quit IRC | 22:20 | |
| *** markvoelker has joined #puppet-openstack | 22:21 | |
| *** markvoelker_ has joined #puppet-openstack | 22:22 | |
| *** xingchao has joined #puppet-openstack | 22:24 | |
| *** o_sutur has joined #puppet-openstack | 22:24 | |
| *** markvoelker has quit IRC | 22:26 | |
| *** xingchao has quit IRC | 22:29 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-glance: Add ability to set properties with glance_image https://review.openstack.org/298718 | 22:31 |
| *** ducttape_ has quit IRC | 22:32 | |
| mwhahaha | glance_image needs reworking because v1 and v2 of the glance api have some important differences | 22:32 |
| EmilienM | just to be transparent, I realized I did a mistake during the release process, on puppet-aodh only. I created 8.0.0 tag on stable/liberty, because of an error in my script. Sorry for that. I'm creating 8.0.1 right now | 22:33 |
| EmilienM | mwhahaha: yeah, we were expecting that... | 22:33 |
| mwhahaha | it's not too bad i'll poke at it later | 22:33 |
| *** dhollinger has quit IRC | 22:38 | |
| *** jmikolajczak has quit IRC | 22:38 | |
| *** markvoelker_ has quit IRC | 22:39 | |
| *** xarses has quit IRC | 22:42 | |
| *** alee has quit IRC | 22:45 | |
| *** _nick has joined #puppet-openstack | 22:51 | |
| EmilienM | ok new fun error: http://logs.openstack.org/03/298903/1/gate/gate-puppet-ceilometer-puppet-beaker-rspec-ubuntu-trusty/b37eea0/console.html#_2016-03-29_22_05_42_191 | 22:57 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: CI test - never merge https://review.openstack.org/299020 | 23:00 |
| *** xarses has joined #puppet-openstack | 23:05 | |
| *** dfisher has quit IRC | 23:09 | |
| openstackgerrit | Matt Fischer proposed openstack/puppet-openstacklib: Adjust timeout for non-retry calls https://review.openstack.org/298992 | 23:13 |
| EmilienM | our CI is now suffering of a puppetlabs packaging issue | 23:16 |
| *** ducttape_ has joined #puppet-openstack | 23:17 | |
| EmilienM | hiera is no longer a dependency, it seems, so the package is not in nodepool images (trusty) and our CI tried to purge the package, but since it does not exist anymore, job fail. | 23:17 |
| EmilienM | discussion on #openstack-infra | 23:17 |
| EmilienM | wrong statement, it's a nodepool issue, not puppetlabs. | 23:19 |
| *** ducttape_ has quit IRC | 23:19 | |
| *** ansmith has joined #puppet-openstack | 23:24 | |
| *** jtomasek has quit IRC | 23:25 | |
| *** xingchao has joined #puppet-openstack | 23:25 | |
| _ody | ok. That would have been odd. | 23:29 |
| *** xingchao has quit IRC | 23:30 | |
| EmilienM | _ody: yeah nothing on puppetlabs side. | 23:31 |
| EmilienM | _ody: we think it was a typo in a script that build ubuntu images.. | 23:31 |
| *** alee has joined #puppet-openstack | 23:33 | |
| *** angdraug has quit IRC | 23:34 | |
| *** dilyin has quit IRC | 23:38 | |
| openstackgerrit | Matt Fischer proposed openstack/puppet-openstacklib: Adjust timeout for non-retry calls https://review.openstack.org/298992 | 23:38 |
| *** jtomasek has joined #puppet-openstack | 23:38 | |
| openstackgerrit | Matt Fischer proposed openstack/puppet-openstacklib: Adjust timeout for non-retry calls https://review.openstack.org/298992 | 23:46 |
| EmilienM | zuul is being restarted | 23:49 |
| EmilienM | expect some NOT_REGISTERED | 23:49 |
| EmilienM | topic is already there | 23:51 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!