| *** itsuugo has quit IRC | 00:04 | |
| *** ducttape_ has quit IRC | 00:05 | |
| *** ducttape_ has joined #puppet-openstack | 00:14 | |
| *** xarses_ has joined #puppet-openstack | 00:21 | |
| *** xarses has quit IRC | 00:24 | |
| *** ducttape_ has quit IRC | 00:31 | |
| *** dhollinger has joined #puppet-openstack | 00:37 | |
| EmilienM | iurygregory: I will help you | 00:57 |
|---|---|---|
| *** iurygregory_ has joined #puppet-openstack | 01:06 | |
| iurygregory_ | EmilienM, tks o/ | 01:06 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-swift: Add support for Swift Ring Builder 2.9.1+ https://review.openstack.org/354476 | 01:36 |
| EmilienM | dmburmistrov: ^ | 01:36 |
| EmilienM | damn ! | 01:36 |
| EmilienM | dmsimard: ^ | 01:37 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO to latest trunk https://review.openstack.org/354355 | 01:38 |
| EmilienM | I'm adding unit tests | 01:41 |
| *** a7ndrew has joined #puppet-openstack | 01:54 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-swift: Add support for Swift Ring Builder 2.9.1+ https://review.openstack.org/354476 | 01:57 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO to latest trunk https://review.openstack.org/354355 | 01:58 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstacklib: Add missing namevar method to openstack_config provider https://review.openstack.org/354096 | 02:00 |
| *** ducttape_ has joined #puppet-openstack | 02:02 | |
| *** ducttape_ has quit IRC | 02:06 | |
| EmilienM | dmsimard: ok REGEX worked, it passed unit tests | 02:07 |
| *** gfidente has quit IRC | 02:11 | |
| openstackgerrit | Merged openstack/puppet-neutron: Change MidoNet Plugin package name https://review.openstack.org/351235 | 02:23 |
| openstackgerrit | Merged openstack/puppet-monasca: Allow tuning of database connection thread pool settings https://review.openstack.org/354356 | 02:30 |
| *** itsuugo has joined #puppet-openstack | 02:31 | |
| *** ayoung has joined #puppet-openstack | 02:49 | |
| *** rhefner has quit IRC | 02:55 | |
| *** rhefner has joined #puppet-openstack | 02:55 | |
| *** mwhahaha has quit IRC | 02:55 | |
| *** davidmichaelkarr has quit IRC | 02:56 | |
| *** mwhahaha has joined #puppet-openstack | 02:59 | |
| *** davidmichaelkarr has joined #puppet-openstack | 03:02 | |
| *** ducttape_ has joined #puppet-openstack | 03:03 | |
| *** ducttape_ has quit IRC | 03:07 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-gnocchi: Keystone authoken resource change https://review.openstack.org/347417 | 03:15 |
| dmsimard | EmilienM: nope, failed on nova | 03:18 |
| dmsimard | EmilienM: http://logs.openstack.org/55/354355/3/check/gate-puppet-openstack-integration-3-scenario003-tempest-centos-7/c78e2a5/logs/nova/nova-compute.txt.gz#_2016-08-12_02_42_27_366 | 03:18 |
| openstackgerrit | Xiaohua Yuan proposed openstack/puppet-nova: add missing options for monkey_patch https://review.openstack.org/346494 | 03:18 |
| dmsimard | http://logs.openstack.org/55/354355/3/check/gate-puppet-openstack-integration-3-scenario003-tempest-centos-7/c78e2a5/console.html#_2016-08-12_02_51_47_688517 | 03:18 |
| dmsimard | Seeing a couple of those mysql errors in neutron-server http://logs.openstack.org/55/354355/3/check/gate-puppet-openstack-integration-3-scenario003-tempest-centos-7/c78e2a5/logs/neutron/server.txt.gz#_2016-08-12_02_49_49_497 | 03:20 |
| iurygregory_ | neutron is like a black box | 03:24 |
| iurygregory_ | it's a little hard to debug | 03:24 |
| dmsimard | iurygregory_: you're telling me | 03:27 |
| iurygregory_ | is just my thoughts XD | 03:30 |
| *** ayoung has quit IRC | 03:45 | |
| *** iurygregory_ has quit IRC | 03:49 | |
| openstackgerrit | Keith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3 https://review.openstack.org/347956 | 03:49 |
| dmsimard | We need to merge https://review.openstack.org/#/c/354476/ ASAP | 03:55 |
| dmsimard | going to sleep | 03:55 |
| dmsimard | night | 03:55 |
| openstackgerrit | Keith Schincke proposed openstack/puppet-ceph: Update ceph::rgw::keystone to support array argument https://review.openstack.org/353646 | 04:02 |
| *** ducttape_ has joined #puppet-openstack | 04:03 | |
| *** ducttape_ has quit IRC | 04:08 | |
| *** skramaja has joined #puppet-openstack | 05:00 | |
| *** jaosorior has joined #puppet-openstack | 05:04 | |
| *** moravec has quit IRC | 05:15 | |
| *** moravec has joined #puppet-openstack | 05:16 | |
| *** rcernin has quit IRC | 05:51 | |
| *** saneax_AFK is now known as saneax | 05:54 | |
| *** saneax is now known as Guest89739 | 05:54 | |
| *** ducttape_ has joined #puppet-openstack | 06:05 | |
| *** ducttape_ has quit IRC | 06:10 | |
| *** fultonj has quit IRC | 06:17 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL https://review.openstack.org/353845 | 06:18 |
| *** rcernin has joined #puppet-openstack | 06:21 | |
| openstackgerrit | Saravanan KR proposed openstack/puppet-neutron: Configure the max_vfs for SRIOV interfaces https://review.openstack.org/344918 | 06:23 |
| *** dhollinger has quit IRC | 06:25 | |
| *** furlongm has quit IRC | 06:28 | |
| openstackgerrit | Carlos Camacho proposed openstack/puppet-cinder: Change pick array order to set the Keystone password in Cinder API profile https://review.openstack.org/354543 | 06:29 |
| *** furlongm has joined #puppet-openstack | 06:32 | |
| *** tesseract- has joined #puppet-openstack | 06:59 | |
| openstackgerrit | Michael Liu proposed openstack/puppet-magnum: Add itesm for keystone_authtoken using class keystone::resource::authtoken https://review.openstack.org/351980 | 07:03 |
| *** ebalduf has quit IRC | 07:03 | |
| *** ducttape_ has joined #puppet-openstack | 07:06 | |
| *** ducttape_ has quit IRC | 07:11 | |
| *** foxhunt_away is now known as foxhunt72 | 07:21 | |
| *** jpena|off is now known as jpena | 07:25 | |
| *** markvoelker has quit IRC | 07:34 | |
| *** fxpester has joined #puppet-openstack | 07:39 | |
| *** amoralej|off is now known as amoralej | 07:44 | |
| *** Guest89739 has quit IRC | 07:45 | |
| *** ifarkas_afk is now known as ifarkas | 07:46 | |
| *** saneax has joined #puppet-openstack | 07:46 | |
| *** saneax is now known as Guest34085 | 07:47 | |
| *** flepied has joined #puppet-openstack | 08:04 | |
| *** ducttape_ has joined #puppet-openstack | 08:08 | |
| *** social has joined #puppet-openstack | 08:08 | |
| openstackgerrit | Martin André proposed openstack/puppet-ironic: Add CORS settings https://review.openstack.org/354571 | 08:10 |
| openstackgerrit | Erno Kuvaja proposed openstack/puppet-manila: [WIP] Add CephFSNative driver logic https://review.openstack.org/353928 | 08:11 |
| *** shardy has joined #puppet-openstack | 08:11 | |
| *** danpawlik has joined #puppet-openstack | 08:12 | |
| *** ducttape_ has quit IRC | 08:12 | |
| *** skramaja has quit IRC | 08:18 | |
| *** rmart04 has joined #puppet-openstack | 08:20 | |
| openstackgerrit | Martin André proposed openstack/puppet-heat: Add CORS settings https://review.openstack.org/354578 | 08:20 |
| openstackgerrit | Martin André proposed openstack/puppet-heat: Add log directory to .gitignore https://review.openstack.org/354581 | 08:25 |
| openstackgerrit | Martin André proposed openstack/puppet-ironic: Add log directory to .gitignore https://review.openstack.org/354583 | 08:27 |
| openstackgerrit | Martin André proposed openstack/puppet-keystone: Add log directory to .gitignore https://review.openstack.org/354584 | 08:30 |
| *** markvoelker has joined #puppet-openstack | 08:34 | |
| *** gfidente has joined #puppet-openstack | 08:37 | |
| *** gfidente has quit IRC | 08:37 | |
| *** gfidente has joined #puppet-openstack | 08:37 | |
| *** markvoelker has quit IRC | 08:39 | |
| *** dtantsur|afk is now known as dtantsur | 08:52 | |
| *** iberezovskiy|afk is now known as iberezovskiy | 08:58 | |
| amoralej | any progress with the issue with scenario003?, i'm investigating the issue from the logs in rdo-ci | 09:03 |
| openstackgerrit | Erno Kuvaja proposed openstack/puppet-manila: [WIP] Add CephFSNative driver logic https://review.openstack.org/353928 | 09:08 |
| *** ducttape_ has joined #puppet-openstack | 09:09 | |
| *** dgurtner has joined #puppet-openstack | 09:09 | |
| *** dgurtner has quit IRC | 09:09 | |
| *** dgurtner has joined #puppet-openstack | 09:09 | |
| *** ducttape_ has quit IRC | 09:13 | |
| *** chem has quit IRC | 09:15 | |
| *** chem has joined #puppet-openstack | 09:23 | |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Move ::roles and ::endpoint after ::keystone and ::wsgi https://review.openstack.org/354613 | 09:31 |
| openstackgerrit | Merged openstack/puppet-swift: Add support for Swift Ring Builder 2.9.1+ https://review.openstack.org/354476 | 09:32 |
| *** markvoelker has joined #puppet-openstack | 09:36 | |
| *** markvoelker has quit IRC | 09:40 | |
| openstackgerrit | Michael Liu proposed openstack/puppet-magnum: Add itesm for keystone_authtoken using class keystone::resource::authtoken https://review.openstack.org/351980 | 09:50 |
| openstackgerrit | Merged openstack/puppet-neutron: Cleanup for sriov-agent sriov-agent-required https://review.openstack.org/330544 | 09:53 |
| *** ducttape_ has joined #puppet-openstack | 10:09 | |
| openstackgerrit | Erno Kuvaja proposed openstack/puppet-manila: [WIP] Add CephFSNative driver logic https://review.openstack.org/353928 | 10:09 |
| *** ducttape_ has quit IRC | 10:14 | |
| openstackgerrit | Martin André proposed openstack/puppet-tripleo: UI profile for tripleo https://review.openstack.org/354630 | 10:14 |
| *** dtantsur is now known as dtantsur|brb | 10:20 | |
| *** leanderthal|afk is now known as leanderthal | 10:27 | |
| *** johnmilton has quit IRC | 10:36 | |
| *** sanjay_ has joined #puppet-openstack | 10:49 | |
| sanjay_ | I am trying to setup keystone using manifest file, I am getting below error "Error: Puppet::Parser::AST::Resource failed with error ArgumentError: Could not find declared class keystone::db::mysql at /etc/puppet/manifests/keystone.pp:14 on node controller". I have used below code inside my manifest file: | 10:50 |
| sanjay_ | class { 'keystone::db::mysql': user => 'keystone', password => 'root123', dbname => 'keystone', allowed_hosts => '%' , } | 10:50 |
| sanjay_ | Any Idea what am in missing? | 10:50 |
| sanjay_ | Now this error: Error: Could not autoload puppet/type/keystone_config: Could not autoload puppet/provider/keystone_config/ini_setting: undefined method `provider' for nil:NilClass | 10:54 |
| *** ducttape_ has joined #puppet-openstack | 11:10 | |
| *** ducttape_ has quit IRC | 11:14 | |
| openstackgerrit | Michael Liu proposed openstack/puppet-magnum: Add itesm for keystone_authtoken using class keystone::resource::authtoken https://review.openstack.org/351980 | 11:27 |
| *** jaosorior has quit IRC | 11:29 | |
| *** jaosorior has joined #puppet-openstack | 11:30 | |
| *** dtantsur|brb is now known as dtantsur | 11:34 | |
| EmilienM | hello | 11:36 |
| EmilienM | dmsimard: damn | 11:37 |
| *** markvoelker has joined #puppet-openstack | 11:37 | |
| EmilienM | degorenko, iberezovskiy: thx guys, you rock | 11:37 |
| EmilienM | dmsimard: swift fixed | 11:38 |
| degorenko | hey | 11:38 |
| degorenko | no problem :) | 11:38 |
| EmilienM | iberezovskiy: saw your comment, fixing it now | 11:38 |
| iberezovskiy | hey, thx) | 11:38 |
| *** jistr has quit IRC | 11:39 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-swift: unit: improve output description for swift ring builder https://review.openstack.org/354668 | 11:40 |
| *** markvoelker has quit IRC | 11:42 | |
| *** johnmilton has joined #puppet-openstack | 11:51 | |
| iurygregory | morning people o/ | 11:52 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO to latest trunk https://review.openstack.org/354355 | 11:53 |
| iurygregory | EmilienM, since when triple-O is voting? | 11:53 |
| EmilienM | iurygregory: 2 days, I was about to announce it | 11:53 |
| iurygregory | https://review.openstack.org/#/c/347417/ =( | 11:53 |
| EmilienM | because now TripleO has a job in the gate, using OpenStack Infra nodes | 11:54 |
| EmilienM | yeah do recheck | 11:54 |
| EmilienM | we had a repo error last night | 11:54 |
| EmilienM | it's fixed now | 11:54 |
| iurygregory | oh awesome :D | 11:54 |
| iurygregory | gnocchi is ready i think =) | 11:54 |
| iurygregory | we need to see if we will let barbican no voting in scenario002 | 11:55 |
| sanjay_ | Hello experts, I am facing this issue while running a manifest file: "Error: Could not autoload puppet/type/keystone_config: Could not autoload puppet/provider/keystone_config/ini_setting: undefined method `provider' for nil:NilClass" | 11:55 |
| sanjay_ | can you please give some pointers? | 11:55 |
| iurygregory | wow o.o | 11:56 |
| iurygregory | do you have your manifest? | 11:56 |
| EmilienM | iurygregory: awesome thanks dude | 11:56 |
| iurygregory | EmilienM, np ^^ | 11:56 |
| sanjay_ | yeah, I am new to puppet and trying to install keystone using puppet, I have added this to manifest file :class { 'keystone::db::mysql': user => 'keystone', password => 'root123', dbname => 'keystone', allowed_hosts => '%' , } # Install Keystone class { 'keystone': verbose => true, package_ensure =>latest, client_package_ensure => latest, #log_debug => true, catalog_type => 'sql | 11:57 |
| iurygregory | sanjay_, can you do a paste in paste.openstack? will make more readable =) | 11:58 |
| sanjay_ | @iurygregory: its here | 12:00 |
| sanjay_ | http://paste.openstack.org/show/555966/ | 12:00 |
| *** Guest34085 is now known as saneax_AFK | 12:00 | |
| *** morazi has joined #puppet-openstack | 12:04 | |
| iurygregory | sanjay_, wich version you are trying to install? | 12:06 |
| *** jayg|g0n3 is now known as jayg | 12:06 | |
| sanjay_ | i havent given any version info anywhere. From where I can check? if I remove the initial exec class from manifest, then also the error is same. | 12:07 |
| *** morazi has quit IRC | 12:08 | |
| iurygregory | did you run puppet module install openstack/keystone ? | 12:09 |
| *** morazi has joined #puppet-openstack | 12:09 | |
| iurygregory | or you can try https://docs.puppet.com/puppet/latest/reference/modules_installing.html#install-from-another-module-repository | 12:10 |
| sanjay_ | yes, i did. If i try reinstalling, it says, Module 'openstack-keystone' (v9.1.0) is already installed. | 12:10 |
| sanjay_ | So its 9.1.0 | 12:10 |
| iurygregory | woa is the master hehehe | 12:10 |
| *** ducttape_ has joined #puppet-openstack | 12:11 | |
| sanjay_ | :) so, any pointers buddy.. | 12:11 |
| iurygregory | looking XD | 12:11 |
| iurygregory | sanjay_, wich OS? | 12:14 |
| iurygregory | ubuntu 16? | 12:14 |
| sanjay_ | 14.04 LTS ubuntu | 12:15 |
| *** ducttape_ has quit IRC | 12:15 | |
| iurygregory | maybe you should try mitaka (8.2.0, 8.1.0) and not 9.1.0 | 12:15 |
| iurygregory | i remeber people having some issues that we could not help much with trusty and the master version i think humm | 12:16 |
| *** ducttape_ has joined #puppet-openstack | 12:16 | |
| sanjay_ | how can i install specific versions? | 12:16 |
| iurygregory | also a good example of how we deploy keystone https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/keystone.pp https://github.com/openstack/puppet-keystone/blob/master/spec/acceptance/keystone_wsgi_apache_spec.rb | 12:17 |
| openstackgerrit | xianming.mao proposed openstack/puppet-cinder: Update homepage with developer documentation page https://review.openstack.org/354693 | 12:17 |
| iurygregory | puppet module install openstack/keystone --version 8.2.0 | 12:18 |
| iurygregory | this will do the job | 12:18 |
| sanjay_ | @iurygregory: Thanks for your help. I will try this out. Actually I am very new to Puppet and trying to write a manifest that will automate the installation of openstack controller node. I was following this "https://www.cybera.ca/news-and-events/tech-radar/lets-build-cloud-using-puppet-install-openstack/" | 12:19 |
| sanjay_ | Do you have something similar that states how can I achieve this for mitaka release? | 12:19 |
| iurygregory | sanjay_, you're well come =) the openstack-integration maybe usefull to you =) you just need to switch to stable mitaka branch | 12:20 |
| iurygregory | https://github.com/openstack/puppet-keystone/tree/stable/mitaka | 12:20 |
| iurygregory | https://github.com/openstack/puppet-openstack-integration/tree/stable/mitaka | 12:20 |
| sanjay_ | yeah, iurygregory, i checked this out, but its more specific to all in one setup. But my requirement is a bit different. So do you suggest going by this all in one and modify it accordingly? | 12:22 |
| *** amoralej is now known as amoralej|lunch | 12:23 | |
| *** jistr has joined #puppet-openstack | 12:24 | |
| *** jpena is now known as jpena|lunch | 12:25 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: Tune number of apache processes/threads for Keystone in apache. https://review.openstack.org/297342 | 12:25 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: Tune number of apache processes/threads for Keystone in apache. https://review.openstack.org/297342 | 12:26 |
| *** fultonj has joined #puppet-openstack | 12:27 | |
| *** jistr has quit IRC | 12:31 | |
| *** jistr has joined #puppet-openstack | 12:32 | |
| *** markvoelker has joined #puppet-openstack | 12:35 | |
| *** fultonj has quit IRC | 12:35 | |
| iurygregory | sanjay_, yep you just need keystone information XD | 12:35 |
| *** fultonj has joined #puppet-openstack | 12:36 | |
| openstackgerrit | xianming.mao proposed openstack/puppet-cinder: Alter the homepage in setup.cfg into the developer web page https://review.openstack.org/354693 | 12:38 |
| *** jistr has quit IRC | 12:41 | |
| *** ducttape_ has quit IRC | 12:42 | |
| openstackgerrit | xianming.mao proposed openstack/puppet-cinder: Alter the homepage in setup.cfg into the developer documentation page https://review.openstack.org/354693 | 12:42 |
| *** vinsh has quit IRC | 12:44 | |
| iurygregory | mwhahaha, you back from PTO? | 12:54 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstacklib: WIP - Allow to give JSON to Puppet parameters https://review.openstack.org/353672 | 12:56 |
| *** amoralej|lunch is now known as amoralej | 12:59 | |
| openstackgerrit | Martin André proposed openstack/puppet-tripleo: UI profile for tripleo https://review.openstack.org/354630 | 13:02 |
| *** jistr has joined #puppet-openstack | 13:03 | |
| openstackgerrit | Lukas Bezdicka proposed openstack/puppet-ironic: WIP: Introduce ironic::ipxe class https://review.openstack.org/354125 | 13:07 |
| mwhahaha | iurygregory: yea | 13:11 |
| iurygregory | mwhahaha, awesome :D, did you saw my comments about auth_strategy? | 13:13 |
| *** jistr has quit IRC | 13:13 | |
| *** dprince has joined #puppet-openstack | 13:18 | |
| *** dprince has quit IRC | 13:19 | |
| *** dprince has joined #puppet-openstack | 13:20 | |
| openstackgerrit | Sven Anderson proposed openstack/puppet-ec2api: beaker: test more classes https://review.openstack.org/352552 | 13:31 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: CI test - never merge https://review.openstack.org/354724 | 13:31 |
| *** jpena|lunch is now known as jpena | 13:31 | |
| mwhahaha | iurygregory: not yet I'll look on a bit | 13:32 |
| iurygregory | mwhahaha, ack ;) | 13:32 |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Promote RDO to latest trunk https://review.openstack.org/354355 | 13:33 |
| openstackgerrit | Sven Anderson proposed openstack/puppet-ec2api: Remove S3 service https://review.openstack.org/354732 | 13:41 |
| *** ducttape_ has joined #puppet-openstack | 13:43 | |
| *** ducttape_ has quit IRC | 13:48 | |
| *** dhollinger has joined #puppet-openstack | 13:59 | |
| openstackgerrit | David Moreau Simard proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL https://review.openstack.org/353845 | 14:00 |
| iurygregory | mwhahaha, +1 for your idea ;) | 14:00 |
| mwhahaha | consistency :) | 14:00 |
| dmsimard | EmilienM: ^ I updated https://review.openstack.org/#/c/353845/ with the same hash as agreed earlier | 14:01 |
| iurygregory | yeah :D | 14:01 |
| EmilienM | dmsimard: approved | 14:01 |
| dmsimard | what, you approved without jenkins? :P | 14:01 |
| iurygregory | i thought that jenkins was changed to zuul but i still see jenkins haha | 14:02 |
| EmilienM | dmsimard: not for promotion jobs | 14:04 |
| *** danpawlik has quit IRC | 14:04 | |
| openstackgerrit | Merged openstack/puppet-swift: unit: improve output description for swift ring builder https://review.openstack.org/354668 | 14:06 |
| *** xarses_ has quit IRC | 14:07 | |
| *** bnemec is now known as beekneemech | 14:07 | |
| *** ebalduf has joined #puppet-openstack | 14:13 | |
| gfidente | mwhahaha, hey thanks on the selinux change | 14:19 |
| mwhahaha | sure | 14:19 |
| gfidente | I think selinux::dircontext was indeed nicer | 14:20 |
| gfidente | but it was going to run the same exec | 14:20 |
| gfidente | in the selinux module | 14:20 |
| gfidente | plus we'd needed another one in -ceph to see if $data was directory or not | 14:21 |
| gfidente | so overall, not sure how beneficial it'd have been | 14:21 |
| gfidente | if at all | 14:21 |
| *** vinsh has joined #puppet-openstack | 14:26 | |
| openstackgerrit | Keith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3 https://review.openstack.org/347956 | 14:26 |
| *** saneax_AFK is now known as saneax | 14:28 | |
| *** iberezovskiy is now known as iberezovskiy|off | 14:28 | |
| *** saneax is now known as Guest47565 | 14:28 | |
| *** ducttape_ has joined #puppet-openstack | 14:32 | |
| *** paramite has quit IRC | 14:33 | |
| *** xarses_ has joined #puppet-openstack | 14:36 | |
| *** jistr has joined #puppet-openstack | 14:39 | |
| *** ntpttr__ has joined #puppet-openstack | 14:41 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: puppet4: fix ordering for compute agent https://review.openstack.org/354756 | 14:41 |
| openstackgerrit | Lars Kellogg-Stedman proposed openstack/puppet-tripleo: [WIP] updates to fluentd support https://review.openstack.org/353507 | 14:45 |
| *** ntpttr__ has quit IRC | 14:50 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: puppet4: ensure rabbitmq is ready before starting nova https://review.openstack.org/354762 | 14:53 |
| *** vinsh_ has joined #puppet-openstack | 14:56 | |
| *** _ducttape_ has joined #puppet-openstack | 14:59 | |
| *** vinsh has quit IRC | 15:00 | |
| *** ducttape_ has quit IRC | 15:03 | |
| *** Guest47565 is now known as saneax_AFK | 15:05 | |
| *** leanderthal is now known as leanderthal|afk | 15:09 | |
| *** ayoung has joined #puppet-openstack | 15:10 | |
| *** rmart04 has quit IRC | 15:13 | |
| *** jaosorior has quit IRC | 15:15 | |
| *** jistr is now known as jistr|call | 15:23 | |
| EmilienM | iurygregory: gnocchi commented | 15:24 |
| colonwq | EmilienM: I am confused by your comment on 353646. Your code and mine look the same to me. What am I missing? | 15:25 |
| EmilienM | colonwq: mistake on my | 15:27 |
| EmilienM | colonwq: +2 | 15:27 |
| colonwq | whew! Thanks. I will +1 my drink count tonight. | 15:27 |
| EmilienM | beagles: I confirm, keystone beaker job on xenial is broken | 15:28 |
| EmilienM | I'm on it | 15:28 |
| beagles | EmilienM, merci bien | 15:29 |
| openstackgerrit | Martin André proposed openstack/puppet-mistral: Add CORS settings https://review.openstack.org/354791 | 15:29 |
| EmilienM | http://logs.openstack.org/42/297342/11/check/gate-puppet-keystone-puppet-beaker-rspec-ubuntu-xenial/776a6e3/console.html#_2016-08-12_12_47_40_349908 | 15:29 |
| *** beagles is now known as beagles_brb | 15:29 | |
| *** haplo37_ has joined #puppet-openstack | 15:30 | |
| openstackgerrit | Martin André proposed openstack/puppet-mistral: Add log directory to .gitignore https://review.openstack.org/354794 | 15:31 |
| *** ifarkas is now known as ifarkas_afk | 15:33 | |
| *** ayoung has quit IRC | 15:33 | |
| EmilienM | chem: if I git blame the code, you wrote this part | 15:34 |
| EmilienM | does this failure ring a bell? | 15:34 |
| chem | EmilienM: which part | 15:35 |
| EmilienM | and why is it suddenly break our ci | 15:35 |
| EmilienM | https://github.com/openstack/puppet-keystone/blob/master/manifests/ldap_backend.pp#L468 | 15:35 |
| EmilienM | chem: ^ | 15:35 |
| chem | EmilienM: looking at the log ... | 15:35 |
| EmilienM | sounds like the getparam doesn't work or | 15:36 |
| *** foxhunt72 is now known as foxhunt_away | 15:38 | |
| EmilienM | chem: it suddenly fail 1 day ago | 15:41 |
| *** jistr|call is now known as jistr | 15:42 | |
| chem | EmilienM: only on xenial, right ? | 15:42 |
| EmilienM | yeah | 15:42 |
| EmilienM | might be the puppet version | 15:42 |
| chem | EmilienM: I think so, I may have an idea | 15:42 |
| EmilienM | though getparam is taken from stdlib iirc | 15:43 |
| mwhahaha | i have never seen bool2num | 15:43 |
| mwhahaha | perhaps you need a str2bool in there :D | 15:44 |
| EmilienM | mwhahaha: https://github.com/puppetlabs/puppetlabs-stdlib/blob/master/lib/puppet/parser/functions/bool2num.rb | 15:47 |
| mwhahaha | ah interesting, also terrible | 15:47 |
| EmilienM | it's actually a function_str2bool | 15:47 |
| EmilienM | yeah | 15:47 |
| EmilienM | ahahah | 15:47 |
| *** rcernin has quit IRC | 15:47 | |
| openstackgerrit | Merged openstack/puppet-ceph: Ensure SELinux file context is applied on data directories https://review.openstack.org/353673 | 15:48 |
| *** _ducttape_ has quit IRC | 15:53 | |
| *** ducttape_ has joined #puppet-openstack | 15:54 | |
| *** beagles_brb is now known as beagles | 15:57 | |
| *** vinsh_ has quit IRC | 15:58 | |
| *** vinsh has joined #puppet-openstack | 15:58 | |
| *** haplo37_ has quit IRC | 16:00 | |
| chem | mwhahaha: EmilienM what is wrong with the bool2num call, that's actually the best way to ensure a safe typecast "true"/"false" test, no ? | 16:00 |
| mwhahaha | it's not necessarily wrong | 16:02 |
| EmilienM | chem: it's good, it used to work | 16:02 |
| EmilienM | just wondering why it fails now | 16:02 |
| mwhahaha | it's just that if you have to use something like that you're not in a good place about understanding all the possible inputs | 16:02 |
| chem | mwhahaha: here the input should be the => true symbol from the getparam call, so making sure the test is valide using bool2num seemed like a good idea | 16:04 |
| chem | mwhahaha: EmilienM here the getparam return an empty string instead of true as show in the error message | 16:05 |
| chem | got "" for identity/domain_specific_drivers_enabled | 16:05 |
| chem | and empty convert to false | 16:05 |
| mwhahaha | i just hate evaluating bools at numbers i can never remember which is true/false because it's language specific :) | 16:06 |
| chem | fair enough, but typecasting in puppet in kinda blurry ... anyway no idea why it should return a empty string the opensntack/integration module didn't change neither... what is wrong with xenial. | 16:10 |
| openstackgerrit | Sven Anderson proposed openstack/puppet-ec2api: Remove S3 service https://review.openstack.org/354732 | 16:10 |
| *** xarses_ is now known as xarses | 16:15 | |
| mwhahaha | wonder if it's an ordering thing | 16:15 |
| mwhahaha | like it's trying to query the config prior to the definition now due to some parse order | 16:16 |
| chem | EmilienM: mwhahaha here the xenial test is ok https://review.openstack.org/#/c/346494/ | 16:16 |
| mwhahaha | it was on a keystone change, maybe it needs a rebase or something | 16:17 |
| EmilienM | chem: you'll notice it's beaker jobs only in puppet-keystone | 16:18 |
| EmilienM | it's when testing ldap | 16:18 |
| *** bkero is now known as bkero-pto | 16:18 | |
| chem | EmilienM: yep, I cannot miss that, what is your point ? | 16:19 |
| chem | mwhahaha: for the ordering stuff, I don't think that could be it as getparam is reading the catalog and the catalog is compiled once before the run | 16:21 |
| EmilienM | chem: the puppet-nova patch is passing because it doesn't test ldap | 16:22 |
| EmilienM | chem: while puppet-keystone beaker tests run ldap config | 16:22 |
| EmilienM | that was my point | 16:22 |
| chem | EmilienM: that's a good point :) | 16:23 |
| chem | EmilienM: sorry, it's starting to be a long day here ... | 16:23 |
| EmilienM | no worries :-) | 16:23 |
| EmilienM | chem: you're almost in week end, keep it up ! | 16:24 |
| mwhahaha | chem: but if getparams is a parser function it is order dependent as it's executated when parsed right? | 16:26 |
| EmilienM | mwhahaha, chem: https://www.diffchecker.com/o927artr | 16:31 |
| EmilienM | puppet 1.5.3-1xenial -> 1.6.0-1xenial | 16:31 |
| EmilienM | now go figure diff between 1.5.3 and 1.6.0 :) | 16:31 |
| chem | EmilienM: is that from that https://github.com/puppetlabs/puppet-agent ? | 16:33 |
| chem | http://www.ubuntuupdates.org/package/puppet_repo/xenial/PC1/base/puppet-agent | 16:34 |
| chem | EmilienM: mwhahaha ^ this repository doesn't support changelog ... | 16:35 |
| *** tesseract- has quit IRC | 16:35 | |
| EmilienM | :( | 16:36 |
| mwhahaha | 4.5.2 vs 4.6.0 | 16:37 |
| mwhahaha | https://docs.puppet.com/puppet/latest/reference/about_agent.html | 16:37 |
| mwhahaha | anyway my brain is not yet working still so maybe after lunch i can figure something out :D | 16:38 |
| iurygregory | EmilienM, ack | 16:38 |
| EmilienM | https://github.com/puppetlabs/puppet/compare/4.5.2...4.6.0 | 16:39 |
| EmilienM | only 541 commits ! | 16:39 |
| *** dtantsur is now known as dtantsur|afk | 16:41 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: functions: clone poi mitaka when cloning ceph/hammer https://review.openstack.org/354835 | 16:43 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: functions: clone poi mitaka when cloning ceph/hammer https://review.openstack.org/354837 | 16:44 |
| openstackgerrit | Merged openstack/puppet-heat: Add CORS settings https://review.openstack.org/354578 | 16:45 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceph: Update .gitreview to point at stable/hammer https://review.openstack.org/350511 | 16:45 |
| chem | EmilienM: mwhahaha oki, testing the keystone manifest with puppet agent 1.6 (but on centos, don't have xenial at my disposal right now) keeping you updated as I go along | 16:45 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceph: Refresh keys if caps are changed https://review.openstack.org/350512 | 16:45 |
| EmilienM | chem: cool ! | 16:46 |
| *** jpena is now known as jpena|off | 16:52 | |
| openstackgerrit | Merged openstack/puppet-mistral: Add api_workers to puppet-mistral https://review.openstack.org/353912 | 16:58 |
| openstackgerrit | Merged openstack/puppet-neutron: Add ml2_conf_sriov.ini to neutron server daemon https://review.openstack.org/333882 | 16:58 |
| chem | EmilienM: mwhahaha oki I reproduced it with puppet 1.6 on centos | 17:06 |
| chem | puppet-agent 1.6 and puppet 4.6 | 17:08 |
| mwhahaha | https://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L1144-L1148 | 17:08 |
| mwhahaha | bet it's the processing of the ensure_resource vs getparam | 17:08 |
| chem | mwhahaha: yeap something like that :) | 17:09 |
| chem | mwhahaha:going into debug mode | 17:09 |
| *** ayoung has joined #puppet-openstack | 17:13 | |
| *** ntpttr__ has joined #puppet-openstack | 17:15 | |
| chem | mwhahaha: EmilienM hum ... the getparam and ensure_resource works ok: http://paste.openstack.org/show/556141/ | 17:18 |
| mwhahaha | what if you put the ensure_resource in an included class | 17:19 |
| *** amoralej is now known as amoralej|off | 17:27 | |
| openstackgerrit | Merged openstack/puppet-keystone: domain backend drivers set in domain config https://review.openstack.org/353657 | 17:28 |
| openstackgerrit | Keith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3 https://review.openstack.org/347956 | 17:39 |
| chem | mwhahaha: EmilienM so for sure keystone::init.pp is not evaluated. I added the ensure_resource in the ldap_backend and it worked, then I looked for the ensure_resource debug message with the original setup and it's not there. So the ldap_backend is doing its stuff and the ensure_resource is not run I try adding a dependency | 17:46 |
| EmilienM | :( | 17:47 |
| EmilienM | sounds like a bug in puppet | 17:47 |
| chem | EmilienM: or a new feature | 17:47 |
| EmilienM | _ody: any thoughts? | 17:47 |
| EmilienM | Hunner: ^ | 17:47 |
| chem | EmilienM: mwhahaha oki, I have something that seems to be working, not sure I understand why. It will help other to understand the why maybe | 17:52 |
| *** gfidente has quit IRC | 17:52 | |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: Fix ldap_backend and puppet 4.6 https://review.openstack.org/354872 | 17:57 |
| chem | EmilienM: mwhahaha so on my test this works ^ | 17:57 |
| *** dgurtner has quit IRC | 17:58 | |
| *** chem is now known as chem|off | 18:16 | |
| *** johnmilton has quit IRC | 18:28 | |
| mwhahaha | my assumption is it's a parse order thing | 18:30 |
| *** sanjay_ has quit IRC | 18:34 | |
| openstackgerrit | Keith Schincke proposed openstack/puppet-openstack-integration: Introduce scenario004 https://review.openstack.org/339106 | 18:41 |
| iurygregory | EmilienM, we should go with mwhahaha idea about auth_strategy? ^^ | 18:50 |
| iurygregory | https://review.openstack.org/#/c/345789/ | 18:50 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: functions: clone poi mitaka when cloning ceph/hammer https://review.openstack.org/354835 | 18:52 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: functions: clone poi mitaka when cloning ceph/hammer https://review.openstack.org/354837 | 18:53 |
| EmilienM | iurygregory: lookinng | 18:53 |
| EmilienM | iurygregory: I agree with Alex | 18:54 |
| iurygregory | EmilienM, me too i'll update =) | 18:54 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceph: Update .gitreview to point at stable/hammer https://review.openstack.org/350511 | 18:55 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceph: Refresh keys if caps are changed https://review.openstack.org/350512 | 18:55 |
| *** johnmilton has joined #puppet-openstack | 19:03 | |
| *** fultonj has quit IRC | 19:04 | |
| *** fultonj has joined #puppet-openstack | 19:04 | |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-aodh: Move other-requirements.txt to bindep.txt https://review.openstack.org/354973 | 19:05 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-barbican: Move other-requirements.txt to bindep.txt https://review.openstack.org/354974 | 19:05 |
| *** ntpttr__ has quit IRC | 19:05 | |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-ceilometer: Move other-requirements.txt to bindep.txt https://review.openstack.org/354975 | 19:05 |
| iurygregory | o.o | 19:05 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-ceph: Move other-requirements.txt to bindep.txt https://review.openstack.org/354976 | 19:05 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-cinder: Move other-requirements.txt to bindep.txt https://review.openstack.org/354977 | 19:06 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-designate: Move other-requirements.txt to bindep.txt https://review.openstack.org/354978 | 19:06 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-glance: Move other-requirements.txt to bindep.txt https://review.openstack.org/354979 | 19:06 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-gnocchi: Move other-requirements.txt to bindep.txt https://review.openstack.org/354980 | 19:06 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-heat: Move other-requirements.txt to bindep.txt https://review.openstack.org/354982 | 19:07 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-horizon: Move other-requirements.txt to bindep.txt https://review.openstack.org/354983 | 19:07 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-ironic: Move other-requirements.txt to bindep.txt https://review.openstack.org/354984 | 19:07 |
| *** johnmilton has quit IRC | 19:07 | |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-keystone: Move other-requirements.txt to bindep.txt https://review.openstack.org/354985 | 19:07 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-magnum: Move other-requirements.txt to bindep.txt https://review.openstack.org/354987 | 19:08 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-manila: Move other-requirements.txt to bindep.txt https://review.openstack.org/354988 | 19:08 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-mistral: Move other-requirements.txt to bindep.txt https://review.openstack.org/354989 | 19:08 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-modulesync-configs: Move other-requirements.txt to bindep.txt https://review.openstack.org/354990 | 19:08 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-murano: Move other-requirements.txt to bindep.txt https://review.openstack.org/354991 | 19:09 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-neutron: Move other-requirements.txt to bindep.txt https://review.openstack.org/354992 | 19:09 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-nova: Move other-requirements.txt to bindep.txt https://review.openstack.org/354993 | 19:09 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-openstack-cookiecutter: Move other-requirements.txt to bindep.txt https://review.openstack.org/354994 | 19:09 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-openstack_extras: Move other-requirements.txt to bindep.txt https://review.openstack.org/354995 | 19:10 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-openstack-integration: Move other-requirements.txt to bindep.txt https://review.openstack.org/354996 | 19:10 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-openstacklib: Move other-requirements.txt to bindep.txt https://review.openstack.org/354997 | 19:10 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-openstack_spec_helper: Move other-requirements.txt to bindep.txt https://review.openstack.org/354998 | 19:10 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-oslo: Move other-requirements.txt to bindep.txt https://review.openstack.org/354999 | 19:11 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-ovn: Move other-requirements.txt to bindep.txt https://review.openstack.org/355000 | 19:11 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-rally: Move other-requirements.txt to bindep.txt https://review.openstack.org/355002 | 19:11 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-sahara: Move other-requirements.txt to bindep.txt https://review.openstack.org/355004 | 19:12 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-swift: Move other-requirements.txt to bindep.txt https://review.openstack.org/355005 | 19:12 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-tempest: Move other-requirements.txt to bindep.txt https://review.openstack.org/355006 | 19:12 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-trove: Move other-requirements.txt to bindep.txt https://review.openstack.org/355007 | 19:12 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-vitrage: Move other-requirements.txt to bindep.txt https://review.openstack.org/355008 | 19:13 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-vswitch: Move other-requirements.txt to bindep.txt https://review.openstack.org/355009 | 19:13 |
| openstackgerrit | Andreas Jaeger proposed openstack/puppet-zaqar: Move other-requirements.txt to bindep.txt https://review.openstack.org/355010 | 19:13 |
| EmilienM | please review this puppet4 thing :) https://review.openstack.org/#/c/354756/ | 19:14 |
| EmilienM | that gives you an idea of the list of our repos | 19:25 |
| EmilienM | https://review.openstack.org/#/q/topic:bindep-mv+project:%22%255Eopenstack/puppet-.*%2524%22+status:open | 19:25 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-gnocchi: Keystone authoken resource change https://review.openstack.org/347417 | 19:26 |
| iurygregory | omg | 19:27 |
| beagles | lol | 19:27 |
| iurygregory | 35 | 19:28 |
| beagles | (but imagine that with a going-around-the-bend kind of lol) | 19:28 |
| EmilienM | I've +2'ed them all | 19:30 |
| EmilienM | I'll let another core approving it | 19:30 |
| iurygregory | EmilienM, you have a bot '-' | 19:30 |
| EmilienM | note: we won't backport it | 19:30 |
| iurygregory | EmilienM, did you have time to see the nova patch? .-. | 19:32 |
| beagles | ah fuel CI, why do you hate me so | 19:33 |
| EmilienM | iurygregory: I'm looking | 19:35 |
| iurygregory | EmilienM, thanks \o/ | 19:36 |
| mwhahaha | fuel ci hates everyone, and everyone hates it back | 19:42 |
| openstackgerrit | Merged openstack/puppet-ec2api: beaker: test more classes https://review.openstack.org/352552 | 19:47 |
| beagles | mwhahaha, :) | 19:52 |
| iurygregory | mwhahaha, fuel ci hates developers only XD | 19:53 |
| EmilienM | I forget to tell everyone tripleo jobs are voting now | 19:53 |
| EmilienM | but they should be pretty stable | 19:53 |
| mwhahaha | no, pretty sure it hates everyone equally :D | 19:53 |
| EmilienM | and if not we'll revert and set them non voting | 19:53 |
| EmilienM | iurygregory: so | 19:55 |
| EmilienM | let me show you :) | 19:55 |
| EmilienM | first you go in console.log http://logs.openstack.org/89/337389/23/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/b454803/console.html.gz#_2016-08-02_04_01_24_653647 | 19:55 |
| EmilienM | and you grep "Error:" | 19:55 |
| EmilienM | you can see the nova_flavor provider is not working, 500 error | 19:55 |
| openstackgerrit | Merged openstack/puppet-modulesync-configs: Move other-requirements.txt to bindep.txt https://review.openstack.org/354990 | 19:55 |
| EmilienM | so 500 error on a provider, it's usually the API service | 19:55 |
| openstackgerrit | Merged openstack/puppet-openstack-cookiecutter: Move other-requirements.txt to bindep.txt https://review.openstack.org/354994 | 19:55 |
| EmilienM | you go now in nova-api logs: http://logs.openstack.org/89/337389/23/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/b454803/logs/nova/nova-api.txt.gz#_2016-08-02_03_59_40_689 | 19:56 |
| EmilienM | oh ! we have an error ! | 19:56 |
| EmilienM | EndpointNotFound: admin endpoint for identity service not found | 19:56 |
| iurygregory | right | 19:56 |
| iurygregory | this i've found XD | 19:56 |
| EmilienM | ok | 19:56 |
| EmilienM | now, let see how config looks like | 19:56 |
| EmilienM | auth_uri=https://[::1]:5000 | 19:56 |
| EmilienM | auth_url=https://[::1]:35357 | 19:57 |
| EmilienM | looks pretty good, let's compare config before/after | 19:57 |
| iurygregory | nova.conf right? | 19:57 |
| EmilienM | https://www.diffchecker.com/lzwtqtho | 19:58 |
| EmilienM | ok config diff sucks | 19:59 |
| EmilienM | let's clean the files before compare | 19:59 |
| iurygregory | ok | 19:59 |
| EmilienM | I use sed -e 's/#.*$//' -e '/^$/d' nova.conf | 20:00 |
| EmilienM | save this command! | 20:00 |
| iurygregory | saved | 20:00 |
| EmilienM | let me 2 min | 20:00 |
| EmilienM | https://www.diffchecker.com/a0nv4kzz | 20:02 |
| EmilienM | ok better | 20:02 |
| EmilienM | so it seems like memcached_servers is not here anymore in the new config | 20:03 |
| iurygregory | admin_password ? O.o | 20:03 |
| EmilienM | iurygregory: left is new config | 20:03 |
| EmilienM | iurygregory: right is old config (the one we know it worked) | 20:03 |
| iurygregory | EmilienM, ok | 20:04 |
| iurygregory | EmilienM, both conf are from puppet jobs? | 20:04 |
| EmilienM | yes sir | 20:05 |
| iurygregory | EmilienM, i'll give a shot with old parameters to see how will work | 20:05 |
| EmilienM | no wait :) | 20:06 |
| EmilienM | let's continue the investigation | 20:06 |
| iurygregory | EmilienM, ok =D | 20:06 |
| EmilienM | memcached_servers might not be the root cause of the auth issue | 20:06 |
| EmilienM | but it's missing in the conf | 20:06 |
| EmilienM | so we found a problem in your patch | 20:06 |
| iurygregory | yeah | 20:06 |
| EmilienM | the pick doesn't work | 20:06 |
| EmilienM | other than that, the config looks good to me | 20:08 |
| iurygregory | maybe thats the reason we have the warnings with cinder and zaqar.. | 20:08 |
| EmilienM | sometimes I go in nova's gate and I look how they config in devstack | 20:08 |
| iurygregory | https://review.openstack.org/#/c/337389/23/lib/puppet/provider/nova.rb maybe related i think | 20:09 |
| EmilienM | http://logs.openstack.org/99/354499/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/a292937/logs/etc/nova/nova.conf.txt.gz | 20:09 |
| mwhahaha | is this the same thing from like 2 weeks ago | 20:09 |
| iurygregory | looking at ironic ivan changed OS_AUTH_URL to have q['auth_uri'] | 20:09 |
| EmilienM | so user_domain_name and project_domain_name are missing, but I don't think it's a big deal | 20:09 |
| EmilienM | i think it's in the provider | 20:10 |
| EmilienM | though the memcached_servers is also a problem we need to solve | 20:10 |
| iurygregory | yeah | 20:10 |
| EmilienM | mwhahaha: have you seen all the warnings in our authtoken things? | 20:10 |
| EmilienM | I think it's because we don't include ::cinder::api in ::cinder::keystone::authtoken | 20:10 |
| EmilienM | the warning is not critical, its just ugly | 20:11 |
| mwhahaha | load order thing again | 20:11 |
| EmilienM | ah yeah | 20:11 |
| mwhahaha | we had the same problem with the vnc stuff | 20:11 |
| EmilienM | right | 20:11 |
| EmilienM | and never fixed it afik | 20:11 |
| mwhahaha | cause i don't think there is a fix :D | 20:11 |
| iurygregory | we still have | 20:11 |
| iurygregory | LOL | 20:11 |
| iurygregory | :O | 20:11 |
| iurygregory | so we will have the warnings forever? O.o | 20:14 |
| mwhahaha | until we drop the pick | 20:14 |
| mwhahaha | so next cycle | 20:14 |
| EmilienM | yea | 20:15 |
| EmilienM | I'm looking https://review.openstack.org/#/c/337389/23/lib/puppet/provider/nova.rb | 20:15 |
| EmilienM | what is if @credentials.version == '3' ? | 20:17 |
| EmilienM | where does it come from? | 20:17 |
| mwhahaha | Puppet::Provider::Openstack::Auth i think | 20:17 |
| EmilienM | mhh | 20:18 |
| EmilienM | I'm looking how we did lib/puppet/provider/cinder.rb | 20:19 |
| EmilienM | let's see in keystone logs what the request looks like | 20:23 |
| iurygregory | we have this info? | 20:24 |
| iurygregory | richm try to help me but we got nothing | 20:25 |
| EmilienM | http://logs.openstack.org/89/337389/23/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/b454803/logs/keystone/keystone.txt.gz#_2016-08-02_03_59_09_035 | 20:25 |
| EmilienM | just wanted to see what keystone says | 20:25 |
| EmilienM | ok, next step is to reproduce the bug :-) | 20:25 |
| *** dprince has quit IRC | 20:25 | |
| *** ntpttr__ has joined #puppet-openstack | 20:26 | |
| EmilienM | mwhahaha: I wonder if in lib/puppet/provider/nova_flavor/openstack.rb we really need @credentials = Puppet::Provider::Openstack::CredentialsV2_0.new | 20:28 |
| openstackgerrit | Pradeep Kilambi proposed openstack/puppet-tripleo: Move ceilometer api to run under apache wsgi https://review.openstack.org/352476 | 20:29 |
| iurygregory | i don't think | 20:29 |
| mwhahaha | i don't think so, not sure tho | 20:30 |
| EmilienM | we actually might need Puppet::Provider::Openstack::CredentialsV3.new instead | 20:30 |
| EmilienM | iirc Puppet::Provider::Openstack::CredentialsV3.new is backward compat with v2 | 20:30 |
| EmilienM | let me check again | 20:30 |
| EmilienM | iurygregory: can I take over? | 20:32 |
| iurygregory | https://review.openstack.org/#/c/337389/23/lib/puppet/provider/nova.rb line 112 should be OS_PROJECT_NAME i think.. | 20:32 |
| iurygregory | EmilienM, of course! | 20:32 |
| *** shardy has quit IRC | 20:32 | |
| EmilienM | ah right ! | 20:32 |
| iurygregory | sorry to be not able to handle D: | 20:32 |
| EmilienM | you are right | 20:33 |
| EmilienM | yes, let's update that | 20:33 |
| EmilienM | iurygregory: please do it ! | 20:33 |
| iurygregory | i'll look in others projects to update to use auth_strategy and update providers too =) | 20:33 |
| EmilienM | just OS_PROJECT_NAME for now | 20:33 |
| iurygregory | ack | 20:33 |
| EmilienM | don't change the @credentials = Puppet::Provider::Openstack::CredentialsV3.new thing | 20:33 |
| EmilienM | v2 should still wor | 20:34 |
| EmilienM | work* | 20:34 |
| *** ntpttr__ has quit IRC | 20:34 | |
| iurygregory | EmilienM, ok =) | 20:34 |
| EmilienM | though it would be good to call v3 by default | 20:34 |
| *** jayg is now known as jayg|g0n3 | 20:34 | |
| EmilienM | ok so now the memcache thing | 20:34 |
| iurygregory | at some point in next release we will need | 20:34 |
| iurygregory | after have all components working with v3 | 20:34 |
| EmilienM | yup | 20:35 |
| EmilienM | baby steps :) | 20:35 |
| iurygregory | yeah | 20:35 |
| iurygregory | EmilienM, so you will update the puppet-nova and i'll go with the other patches right? | 20:36 |
| EmilienM | iurygregory: what do you want me to update? | 20:36 |
| colonwq | Random Question: In Anchor a proper thing to do or is it a puppet equivalent to 'goto'? | 20:37 |
| iurygregory | EmilienM, you asked if you can take over the patch no? | 20:37 |
| iurygregory | O.o | 20:37 |
| EmilienM | yeah but finally uou can do it :) | 20:37 |
| EmilienM | it's your patch ! :P | 20:37 |
| iurygregory | EmilienM, oh ok :D | 20:37 |
| EmilienM | just change OS_PROJECT_NAME now | 20:37 |
| EmilienM | we'll see how it works | 20:37 |
| EmilienM | now I'm looking the memcachce thing | 20:37 |
| iurygregory | ok | 20:37 |
| EmilienM | it looks like the value is not lookup | 20:38 |
| EmilienM | do we need to include ::nova? | 20:38 |
| EmilienM | or inherit | 20:39 |
| *** haplo37_ has joined #puppet-openstack | 20:39 | |
| EmilienM | iurygregory: can you try to include ::nova? I think nova::memcached_servers is not in the scope or something | 20:40 |
| iurygregory | EmilienM, ack | 20:40 |
| EmilienM | iurygregory: in the new class | 20:40 |
| iurygregory | ok | 20:40 |
| iurygregory | EmilienM, i need to update lib/puppet/provider/nova_flavor/openstack.rb right? | 20:44 |
| EmilienM | no :) | 20:46 |
| EmilienM | again, just change OS_PROJECT_NAME now | 20:46 |
| iurygregory | EmilienM, ok | 20:46 |
| EmilienM | and include ::nova in authtoken | 20:46 |
| iurygregory | and the include | 20:46 |
| iurygregory | running tests =D | 20:46 |
| openstackgerrit | Keith Schincke proposed openstack/puppet-openstack-integration: Introduce scenario004 https://review.openstack.org/339106 | 20:47 |
| -openstackstatus- NOTICE: The Gerrit service on review.openstack.org is restarting for a scheduled upgrade, but should return to service momentarily: http://lists.openstack.org/pipermail/openstack-dev/2016-August/101394.html | 20:51 | |
| openstackgerrit | Merged openstack/puppet-vswitch: Modify the OVS DPDK package name to openvswitch https://review.openstack.org/351522 | 20:51 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-nova: Move nova to authtoken https://review.openstack.org/337389 | 20:58 |
| *** ducttape_ has quit IRC | 20:58 | |
| iurygregory | enrr i forgot the gerrit maintence D: | 20:58 |
| -openstackstatus- NOTICE: The Mediawiki service at wiki.openstack.org will be offline from 21:00 UTC until approximately 23:00 UTC for a planned upgrade http://lists.openstack.org/pipermail/openstack-dev/2016-August/101395.html | 21:00 | |
| *** ChanServ changes topic to "The Mediawiki service at wiki.openstack.org will be offline from 21:00 UTC until approximately 23:00 UTC for a planned upgrade http://lists.openstack.org/pipermail/openstack-dev/2016-August/101395.html" | 21:00 | |
| iurygregory | EmilienM, i'll updates all other patches tonight or tomorrow (i'll be travelling) thanks for your help o/ | 21:01 |
| iurygregory | we still need to define if we will let barbican no voting in scenario 002 o/ | 21:01 |
| *** fultonj has quit IRC | 21:01 | |
| *** Mosibi has quit IRC | 21:05 | |
| EmilienM | iurygregory: take care | 21:09 |
| iurygregory | EmilienM, you too o/ tks | 21:09 |
| EmilienM | iurygregory: for barbican, like I said, you need to disable in poi first, first puppet and reenable it | 21:09 |
| EmilienM | err | 21:09 |
| EmilienM | then puppet and then eenable it | 21:09 |
| iurygregory | EmilienM, ack i'll do :D | 21:09 |
| EmilienM | thanks | 21:09 |
| EmilienM | and let me know if you need help | 21:09 |
| iurygregory | EmilienM, ack o/ | 21:10 |
| EmilienM | mfisch: can you look https://review.openstack.org/#/c/354756/ please ? | 21:16 |
| EmilienM | and https://review.openstack.org/#/c/354122/ | 21:16 |
| *** ntpttr__ has joined #puppet-openstack | 21:39 | |
| *** jistr has quit IRC | 21:47 | |
| *** haplo37_ has quit IRC | 21:50 | |
| *** jistr has joined #puppet-openstack | 21:51 | |
| EmilienM | have a great weekend everyone ! | 21:53 |
| * EmilienM off | 21:53 | |
| beagles | EmilienM, cheers! | 21:54 |
| *** jistr has quit IRC | 22:00 | |
| *** ducttape_ has joined #puppet-openstack | 22:07 | |
| *** jistr has joined #puppet-openstack | 22:09 | |
| *** dhollinger has quit IRC | 22:11 | |
| *** beekneemech is now known as bnemec-pto | 22:12 | |
| *** jistr has quit IRC | 22:19 | |
| *** jistr has joined #puppet-openstack | 22:25 | |
| *** ntpttr__ has quit IRC | 22:29 | |
| *** ntpttr__ has joined #puppet-openstack | 22:33 | |
| *** ducttape_ has quit IRC | 22:35 | |
| *** ducttape_ has joined #puppet-openstack | 22:36 | |
| *** ducttape_ has quit IRC | 22:40 | |
| openstackgerrit | Merged openstack/puppet-ironic: Add CORS settings https://review.openstack.org/354571 | 22:50 |
| *** vinsh has quit IRC | 22:53 | |
| *** markvoelker has quit IRC | 22:53 | |
| *** xarses has quit IRC | 23:01 | |
| -openstackstatus- NOTICE: ok https://wiki.openstack.org/ is now running Mediawiki 1.27.0; please let us know in #openstack-infra if anything seems wrong | 23:02 | |
| *** ChanServ changes topic to "ok https://wiki.openstack.org/ is now running Mediawiki 1.27.0; please let us know in #openstack-infra if anything seems wrong" | 23:02 | |
| mwhahaha | welp all those other-requirements.txt to bindep.txt seem to fail cause the job is looking for other-requirements.txt. will have to look into it next week | 23:07 |
| * mwhahaha gives up | 23:07 | |
| mwhahaha | for the p-o-i jobs | 23:07 |
| *** ChanServ changes topic to "CI status: green - Puppet OpenStack modules http://docs.openstack.org/developer/puppet-openstack-guide/" | 23:08 | |
| -openstackstatus- NOTICE: https://wiki.openstack.org/ is now running Mediawiki 1.27.0; please let us know in #openstack-infra if anything seems wrong | 23:08 | |
| beagles | mwhahaha, the fuel ci jobs *are* voting, right | 23:19 |
| * beagles checks before he "rechecks" | 23:19 | |
| mwhahaha | No | 23:20 |
| beagles | then I am glad I checked before rechecking | 23:20 |
| mwhahaha | Which patch are you having politicked with | 23:20 |
| mwhahaha | Err spell check | 23:20 |
| beagles | https://review.openstack.org/#/c/354096/ | 23:21 |
| mwhahaha | Problems not politicked. How is that even in my phone | 23:21 |
| beagles | :) | 23:21 |
| beagles | probably just a jargonesque permutation | 23:21 |
| openstackgerrit | Merged openstack/puppet-manila: Move other-requirements.txt to bindep.txt https://review.openstack.org/354988 | 23:21 |
| *** chem|off has quit IRC | 23:22 | |
| beagles | whoa.. google says it is the past tense of the derogatory connotation of "politic" | 23:22 |
| beagles | so... in this case, probably the proper spelling !?! | 23:22 |
| mwhahaha | Yea not related to your patch. If you want you can try 'fuel: recheck' | 23:23 |
| beagles | cool thanks! | 23:23 |
| *** xarses has joined #puppet-openstack | 23:26 | |
| *** saneax_AFK is now known as saneax | 23:26 | |
| openstackgerrit | Merged openstack/puppet-magnum: Move other-requirements.txt to bindep.txt https://review.openstack.org/354987 | 23:26 |
| openstackgerrit | Merged openstack/puppet-rally: Move other-requirements.txt to bindep.txt https://review.openstack.org/355002 | 23:27 |
| *** saneax is now known as Guest48269 | 23:27 | |
| *** chem|off has joined #puppet-openstack | 23:27 | |
| openstackgerrit | Merged openstack/puppet-murano: Move other-requirements.txt to bindep.txt https://review.openstack.org/354991 | 23:34 |
| openstackgerrit | Merged openstack/puppet-designate: Move other-requirements.txt to bindep.txt https://review.openstack.org/354978 | 23:39 |
| *** ntpttr__ has quit IRC | 23:48 | |
| *** ducttape_ has joined #puppet-openstack | 23:53 | |
| *** markvoelker has joined #puppet-openstack | 23:54 | |
| *** ntpttr__ has joined #puppet-openstack | 23:56 | |
| *** markvoelker has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!