| *** zhangjn has joined #puppet-openstack | 00:04 | |
| *** zhangjn has quit IRC | 00:04 | |
| *** zhangjn has joined #puppet-openstack | 00:04 | |
| openstackgerrit | Mike Dorman proposed openstack/puppet-nova: Nova cells support for nova_aggregate provider https://review.openstack.org/267866 | 00:09 |
|---|---|---|
| *** zhangjn has quit IRC | 00:16 | |
| *** zhangjn has joined #puppet-openstack | 00:17 | |
| *** morazi has quit IRC | 00:17 | |
| *** xarses has joined #puppet-openstack | 00:17 | |
| *** petems has joined #puppet-openstack | 00:17 | |
| petems | nibalizer: I’m here now :P | 00:18 |
| nibalizer | petems: welcome | 00:21 |
| petems | RE: the infra-as-code-but-for-Openstack-puppet-resources thing, I remember someone talking about making a Fog type and provider | 00:21 |
| petems | then using it to implement openstack as a provider | 00:22 |
| petems | bingo, easy servers | 00:22 |
| petems | I dunno what became of that though... | 00:22 |
| *** saneax_AFK is now known as saneax | 00:22 | |
| *** saneax is now known as Guest42435 | 00:23 | |
| *** zhangjn has quit IRC | 00:25 | |
| *** karimb has quit IRC | 00:32 | |
| *** xingchao has joined #puppet-openstack | 00:40 | |
| *** xarses has quit IRC | 00:44 | |
| *** xingchao has quit IRC | 00:45 | |
| *** ccie6747 has joined #puppet-openstack | 00:45 | |
| *** zhangjn has joined #puppet-openstack | 00:53 | |
| *** zhangjn has quit IRC | 00:58 | |
| *** zhangjn has joined #puppet-openstack | 01:02 | |
| *** ccie6747 has quit IRC | 01:09 | |
| *** zhangjn has quit IRC | 01:14 | |
| *** jayg is now known as jayg|g0n3 | 01:19 | |
| *** ferest has quit IRC | 01:30 | |
| *** ferest has joined #puppet-openstack | 01:30 | |
| *** zhangjn has joined #puppet-openstack | 01:36 | |
| *** xarses has joined #puppet-openstack | 01:38 | |
| *** xingchao has joined #puppet-openstack | 01:41 | |
| *** zhangjn has quit IRC | 01:43 | |
| *** xingchao has quit IRC | 01:45 | |
| *** zhangjn has joined #puppet-openstack | 01:46 | |
| *** ferest has quit IRC | 01:46 | |
| *** jfluhmann has quit IRC | 01:47 | |
| *** cwolferh has quit IRC | 01:49 | |
| *** zhangjn has quit IRC | 01:50 | |
| *** zhangjn has joined #puppet-openstack | 01:53 | |
| openstackgerrit | Liao Penghui proposed openstack/puppet-ceilometer: Move workers options into specific sections. https://review.openstack.org/266523 | 02:01 |
| *** zhangjn has quit IRC | 02:02 | |
| *** cwolferh has joined #puppet-openstack | 02:06 | |
| *** zhangjn has joined #puppet-openstack | 02:08 | |
| *** richm has quit IRC | 02:26 | |
| *** woon has joined #puppet-openstack | 02:30 | |
| *** rahulait has quit IRC | 02:33 | |
| *** xingchao has joined #puppet-openstack | 02:34 | |
| *** zhangjn has quit IRC | 02:37 | |
| *** jfluhmann has joined #puppet-openstack | 02:38 | |
| *** ilbot3 has quit IRC | 02:45 | |
| *** ilbot3 has joined #puppet-openstack | 02:47 | |
| *** xarses_ has joined #puppet-openstack | 02:53 | |
| *** xarses has quit IRC | 02:54 | |
| *** andreww has joined #puppet-openstack | 02:56 | |
| *** xarses_ has quit IRC | 03:00 | |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-keystone: Update keystone::resource::authtoken https://review.openstack.org/266723 | 03:15 |
| *** aimon_ has quit IRC | 03:16 | |
| *** ferest has joined #puppet-openstack | 03:26 | |
| *** ferest has quit IRC | 03:32 | |
| *** baker has joined #puppet-openstack | 03:43 | |
| *** ferest has joined #puppet-openstack | 03:57 | |
| *** zhangjn has joined #puppet-openstack | 04:17 | |
| *** zhangjn has quit IRC | 04:25 | |
| *** xingchao has quit IRC | 04:28 | |
| *** zhangjn has joined #puppet-openstack | 04:35 | |
| *** ferest has quit IRC | 04:51 | |
| *** ferest has joined #puppet-openstack | 04:51 | |
| *** zhangjn has quit IRC | 04:53 | |
| *** ferest has quit IRC | 04:56 | |
| *** baker has quit IRC | 05:18 | |
| *** jaosorior has joined #puppet-openstack | 05:19 | |
| *** zhangjn has joined #puppet-openstack | 05:35 | |
| *** zhangjn has quit IRC | 05:36 | |
| *** zhangjn has joined #puppet-openstack | 05:37 | |
| openstackgerrit | Marcus Furlong proposed openstack/puppet-neutron: update README to reflect new puppet-nova parameter https://review.openstack.org/267943 | 05:38 |
| *** zhangjn_ has joined #puppet-openstack | 05:39 | |
| *** zhangjn_ has quit IRC | 05:40 | |
| *** zhangjn_ has joined #puppet-openstack | 05:41 | |
| *** zhangjn_ has quit IRC | 05:42 | |
| *** zhangjn has quit IRC | 05:43 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable X-Forwarded-Proto header for cinder https://review.openstack.org/267945 | 05:43 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable X-Forwarded-Proto header for keystone admin endpoint https://review.openstack.org/267946 | 05:43 |
| *** zhangjn has joined #puppet-openstack | 05:43 | |
| *** zhangjn has quit IRC | 05:47 | |
| *** zhangjn has joined #puppet-openstack | 05:47 | |
| *** ferest has joined #puppet-openstack | 05:51 | |
| *** ferest has quit IRC | 06:02 | |
| *** jfluhmann has quit IRC | 06:33 | |
| *** ferest has joined #puppet-openstack | 06:44 | |
| *** xingchao has joined #puppet-openstack | 06:45 | |
| *** jaosorior has quit IRC | 06:45 | |
| *** jaosorior has joined #puppet-openstack | 06:46 | |
| *** jaosorior has quit IRC | 06:55 | |
| *** gildub has quit IRC | 07:02 | |
| *** jaosorior has joined #puppet-openstack | 07:11 | |
| *** magicboiz has quit IRC | 07:27 | |
| *** jtomasek has joined #puppet-openstack | 07:31 | |
| *** samuelBartel has quit IRC | 07:32 | |
| *** ferest has quit IRC | 07:52 | |
| *** magicboiz has joined #puppet-openstack | 07:56 | |
| *** magicboiz has quit IRC | 08:00 | |
| *** spredzy- has joined #puppet-openstack | 08:13 | |
| *** magicboiz has joined #puppet-openstack | 08:15 | |
| *** zhangjn has quit IRC | 08:19 | |
| *** gfidente has joined #puppet-openstack | 08:19 | |
| *** gfidente has quit IRC | 08:19 | |
| *** gfidente has joined #puppet-openstack | 08:19 | |
| *** dgurtner has joined #puppet-openstack | 08:20 | |
| *** dgurtner has quit IRC | 08:20 | |
| *** dgurtner has joined #puppet-openstack | 08:20 | |
| *** samuelBartel has joined #puppet-openstack | 08:21 | |
| *** ferest has joined #puppet-openstack | 08:26 | |
| *** foxhunt_away is now known as foxhunt72 | 08:28 | |
| *** dgurtner has quit IRC | 08:29 | |
| *** dgurtner has joined #puppet-openstack | 08:29 | |
| *** dgurtner has quit IRC | 08:29 | |
| *** dgurtner has joined #puppet-openstack | 08:29 | |
| *** jfluhmann has joined #puppet-openstack | 08:34 | |
| *** spredzy has quit IRC | 08:38 | |
| *** spredzy- is now known as spredzy | 08:38 | |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-keystone: Refactor keystone::resource::authtoken https://review.openstack.org/266723 | 08:53 |
| *** ferest has quit IRC | 09:03 | |
| *** zhangjn has joined #puppet-openstack | 09:04 | |
| *** jistr has joined #puppet-openstack | 09:27 | |
| *** zhangjn has quit IRC | 09:34 | |
| *** gildub has joined #puppet-openstack | 09:36 | |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-nova: Change section name for memcached_servers param https://review.openstack.org/266136 | 09:44 |
| *** derekh has joined #puppet-openstack | 09:48 | |
| *** xingchao_ has joined #puppet-openstack | 09:55 | |
| *** paramite has joined #puppet-openstack | 09:56 | |
| *** xingchao has quit IRC | 09:59 | |
| *** paramite has quit IRC | 10:01 | |
| *** xingchao has joined #puppet-openstack | 10:01 | |
| *** xingchao_ has quit IRC | 10:04 | |
| *** _degorenko|afk is now known as degorenko | 10:05 | |
| *** karimb has joined #puppet-openstack | 10:11 | |
| *** paramite has joined #puppet-openstack | 10:14 | |
| *** yolanda has joined #puppet-openstack | 10:27 | |
| yolanda | hi, i'm trying to use puppet modules, and i have a problem in getting a self signed cert to work, can i get help? | 10:28 |
| yolanda | i'm getting an openstack ssl error, complaining about the self signed cert | 10:28 |
| *** gchamoul is now known as gchamoul|afk | 10:37 | |
| *** paramite has quit IRC | 10:38 | |
| degorenko | yolanda, hi, which module? | 10:38 |
| yolanda | hi degorenko, i'm having problems with keystone | 10:38 |
| yolanda | i'm using keystone::apache to setup the certs, and a ssl validation shows is ok, although self signed | 10:39 |
| yolanda | when trying to setup the service i get | 10:40 |
| yolanda | Error: Could not prefetch keystone_service provider 'openstack': Execution of '/usr/bin/openstack service list --quiet --format csv --long' returned 1: ERROR: openstack SSL exception connecting to https://controller00.hpuseast.ic.openstack.org:35357/v3/services | 10:40 |
| degorenko | yolanda, did you specified all ssl config path in puppet? | 10:41 |
| degorenko | https://github.com/openstack/puppet-keystone/blob/master/manifests/wsgi/apache.pp#L142-L148 | 10:41 |
| yolanda | degorenko, yes. And i verified config in apache, and the paths are pointing to the right files | 10:41 |
| yolanda | admin shows something like | 10:41 |
| yolanda | 10:41 | |
| yolanda | SSLEngine on | 10:41 |
| yolanda | SSLCertificateFile "/etc/ssl/certs/controller00.hpuseast.ic.openstack.org-keystone.pem" | 10:42 |
| yolanda | SSLCertificateKeyFile "/etc/ssl/private/controller00.hpuseast.ic.openstack.org-keystone.pem" | 10:42 |
| yolanda | SSLCertificateChainFile "/etc/ssl/certs/controller00.hpuseast.ic.openstack.org-ca.pem" | 10:42 |
| yolanda | SSLCACertificatePath "/etc/ssl/certs" | 10:42 |
| *** yottatsa has quit IRC | 10:42 | |
| yolanda | cert verification is fine as well | 10:42 |
| yolanda | openssl verify -purpose sslserver -CAfile /etc/ssl/certs/controller00.hpuseast.ic.openstack.org-ca.pem /etc/ssl/certs/controller00.hpuseast.ic.openstack.org-keystone.pem | 10:42 |
| yolanda | /etc/ssl/certs/controller00.hpuseast.ic.openstack.org-keystone.pem: OK | 10:42 |
| *** gildub has quit IRC | 10:42 | |
| degorenko | and keystone is also has proper configuration? | 10:43 |
| *** jgrassler has quit IRC | 10:44 | |
| yolanda | degorenko, i just set enable_ssl => true, and service_name => httpd on keystone | 10:44 |
| yolanda | degorenko, http://paste.openstack.org/show/483988/ | 10:45 |
| degorenko | yolanda, what's about these https://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L555-L558 and https://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L531-L536 parameters? Do you have proper keystone config ssl section? | 10:47 |
| degorenko | yolanda, https://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L720-L728 | 10:47 |
| openstackgerrit | Merged openstack/puppet-ceilometer: Move workers options into specific sections. https://review.openstack.org/266523 | 10:47 |
| *** jgrassler has joined #puppet-openstack | 10:47 | |
| degorenko | does this section contain your ssl certs? | 10:48 |
| yolanda | degorenko, i thought this wasn't used, because we use apache to manage the ssl config | 10:48 |
| yolanda | i can try to setup them | 10:49 |
| *** xingchao has quit IRC | 10:50 | |
| yolanda | degorenko, same error | 10:53 |
| degorenko | did you restarted apache/keystone? | 10:54 |
| yolanda | yes | 10:54 |
| degorenko | hm | 10:54 |
| *** jpena has joined #puppet-openstack | 10:55 | |
| degorenko | yolanda, i will try to test in my env | 10:56 |
| yolanda | thanks | 10:56 |
| degorenko | mgagne, ping | 10:58 |
| chem | xenolog-: he, do you have 2 min to discuss https://review.openstack.org/#/c/262809/ | 11:07 |
| chem | xenolog-: The Neutron parses the wrong json revert to shell/csv. business | 11:07 |
| yolanda | derogenko, i made it work by tweaking puppet-openstacklib button, adding a --insecure flag. But this cannot be populated in parameters? | 11:08 |
| yolanda | openstacklib module, i mean | 11:08 |
| openstackgerrit | Merged openstack/puppet-neutron: LBaaSv2 support https://review.openstack.org/234728 | 11:21 |
| openstackgerrit | Merged openstack/puppet-keystone: Testing some keystone types alone fails https://review.openstack.org/267699 | 11:34 |
| degorenko | yolanda, no :) | 11:40 |
| degorenko | my env is not ready yet, some another stuff | 11:40 |
| degorenko | i will ping you back, when i test it | 11:41 |
| yolanda | thanks derorenko | 11:41 |
| yolanda | degorenko | 11:41 |
| degorenko | yolanda, use tab :) | 11:41 |
| yolanda | yep :) | 11:41 |
| *** dprince has joined #puppet-openstack | 12:11 | |
| openstackgerrit | Merged openstack/puppet-horizon: Add List/None type value in the local_settings. https://review.openstack.org/266814 | 12:13 |
| *** cdelatte has joined #puppet-openstack | 12:35 | |
| *** karimb has quit IRC | 12:36 | |
| EmilienM | good morning | 12:46 |
| *** karimb has joined #puppet-openstack | 12:48 | |
| *** openstackgerrit has quit IRC | 12:50 | |
| *** openstackgerrit has joined #puppet-openstack | 12:51 | |
| *** karimb has quit IRC | 12:55 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-openstack-integration: [WIP] scenario002: enable application catalog service https://review.openstack.org/250436 | 12:57 |
| *** skolekonov has joined #puppet-openstack | 12:58 | |
| degorenko | morning :) | 13:04 |
| *** ducttape_ has joined #puppet-openstack | 13:06 | |
| EmilienM | degorenko: my patch to refactor does not work for beaker jobs | 13:11 |
| degorenko | EmilienM, i saw | 13:11 |
| degorenko | looks like sahara repo wasn't clonned | 13:12 |
| EmilienM | yeah! | 13:12 |
| EmilienM | I'll investigate a bit more | 13:12 |
| *** jayg|g0n3 is now known as jayg | 13:14 | |
| *** paramite has joined #puppet-openstack | 13:18 | |
| *** Guest42435 is now known as saneax_AFK | 13:23 | |
| *** ducttape_ has quit IRC | 13:26 | |
| *** ducttape_ has joined #puppet-openstack | 13:26 | |
| *** ducttape_ has quit IRC | 13:31 | |
| *** cdelatte has quit IRC | 13:33 | |
| *** cdelatte has joined #puppet-openstack | 13:34 | |
| *** trown has joined #puppet-openstack | 13:41 | |
| openstackgerrit | Liao Penghui proposed openstack/puppet-nova: Fix inconsistent type/provider for paste configs. https://review.openstack.org/266692 | 13:41 |
| *** morazi has joined #puppet-openstack | 13:44 | |
| *** trown is now known as trown|outttypeww | 13:44 | |
| *** gchamoul|afk is now known as gchamoul | 13:46 | |
| *** iurygregory has quit IRC | 13:47 | |
| *** jpena has quit IRC | 13:49 | |
| *** baker has joined #puppet-openstack | 13:54 | |
| *** richm has joined #puppet-openstack | 13:59 | |
| *** ccie6747 has joined #puppet-openstack | 14:00 | |
| *** baker has quit IRC | 14:07 | |
| *** xingchao has joined #puppet-openstack | 14:19 | |
| openstackgerrit | Liao Penghui proposed openstack/puppet-trove: Disable exists_notification_transformer for all OS https://review.openstack.org/268157 | 14:22 |
| openstackgerrit | Liao Penghui proposed openstack/puppet-trove: Disable exists_notification_transformer for all OS https://review.openstack.org/268157 | 14:24 |
| openstackgerrit | Liao Penghui proposed openstack/puppet-trove: Disable exists_notification_transformer for all OS https://review.openstack.org/268157 | 14:29 |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-keystone: Refactor keystone::resource::authtoken https://review.openstack.org/266723 | 14:35 |
| EmilienM | degorenko: http://logs.openstack.org/63/267763/1/check/gate-puppet-sahara-puppet-beaker-rspec-dsvm-trusty/cb9948b/console.html#_2016-01-14_23_02_52_096 | 14:36 |
| EmilienM | the module is installed :/ | 14:36 |
| degorenko | why then it was not found? | 14:38 |
| degorenko | :( | 14:38 |
| *** jaosorior has quit IRC | 14:45 | |
| *** jaosorior has joined #puppet-openstack | 14:46 | |
| *** trown|outttypeww is now known as trown | 14:52 | |
| *** ducttape_ has joined #puppet-openstack | 14:54 | |
| *** andreww has quit IRC | 15:06 | |
| *** ericksonsantos has quit IRC | 15:11 | |
| *** jaosorior has quit IRC | 15:11 | |
| *** jaosorior has joined #puppet-openstack | 15:12 | |
| mfisch | mgagne: clayton had a good idea last night he said that he thought my array had been stringified somewhere along the path | 15:23 |
| mfisch | mgagne: I will try to confirm that today | 15:23 |
| *** rahulait has joined #puppet-openstack | 15:31 | |
| mfisch | mgagne: thats it | 15:31 |
| *** derekh has quit IRC | 15:33 | |
| *** derekh has joined #puppet-openstack | 15:33 | |
| *** baker has joined #puppet-openstack | 15:35 | |
| mfisch | mgagne: found it | 15:38 |
| *** andreww has joined #puppet-openstack | 15:44 | |
| mfisch | mgagne: let me know when you get in, you'll like it | 15:44 |
| *** jfluhmann has quit IRC | 15:49 | |
| clayton | where was the stringify? | 15:51 |
| mfisch | hiera() | 15:53 |
| mfisch | " The result of the lookup must be a string;" | 15:54 |
| mfisch | it has to be that | 15:54 |
| mfisch | docs claim also " any other result will cause an error." | 15:54 |
| mfisch | except it doesn't error, it just makes it a string for you | 15:54 |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-keystone: Refactor keystone::resource::authtoken https://review.openstack.org/266723 | 15:55 |
| *** dfisher has joined #puppet-openstack | 15:57 | |
| clayton | were you doing an explicit lookup? | 15:58 |
| clayton | I've not seen that happen for automatic lookup | 15:58 |
| mfisch | there's 2 layers | 15:58 |
| mfisch | its like this | 15:58 |
| mfisch | allowed_hosts: | 15:58 |
| mfisch | - a | 15:58 |
| mfisch | - b | 15:58 |
| mfisch | then cinder::foo::db::blah: hiera('allowed_hosts') | 15:58 |
| mfisch | designate:: keystone::: etc etc | 15:59 |
| mfisch | I have a fix | 15:59 |
| *** mdorman has joined #puppet-openstack | 16:19 | |
| *** xingchao_ has joined #puppet-openstack | 16:22 | |
| *** xingchao has quit IRC | 16:26 | |
| *** xingchao has joined #puppet-openstack | 16:28 | |
| *** foxhunt72 is now known as foxhunt_away | 16:29 | |
| *** angdraug has joined #puppet-openstack | 16:29 | |
| *** xingcha__ has joined #puppet-openstack | 16:31 | |
| *** xingchao_ has quit IRC | 16:31 | |
| *** ericksonsantos has joined #puppet-openstack | 16:31 | |
| *** xingchao has quit IRC | 16:34 | |
| *** degorenko is now known as _degorenko|afk | 16:37 | |
| *** zhangjn has joined #puppet-openstack | 16:38 | |
| *** zhangjn has quit IRC | 16:51 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add resources to set ssl middleware for Nova https://review.openstack.org/268230 | 16:51 |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-nova: Change section name for memcached_servers param https://review.openstack.org/266136 | 17:00 |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-glance: Introduce keystone::resource::authtoken https://review.openstack.org/268233 | 17:01 |
| *** xingchao has joined #puppet-openstack | 17:04 | |
| *** samuelBartel has quit IRC | 17:05 | |
| openstackgerrit | Merged openstack/puppet-monasca: Add support for symlinked agent virtualenv https://review.openstack.org/265357 | 17:06 |
| *** dhollinger has joined #puppet-openstack | 17:07 | |
| *** xingcha__ has quit IRC | 17:08 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add resources to set ssl middleware for Nova https://review.openstack.org/268230 | 17:09 |
| chem | xenolog-: he, are you around ? | 17:13 |
| *** jpena has joined #puppet-openstack | 17:16 | |
| *** dgurtner has quit IRC | 17:17 | |
| *** xingchao_ has joined #puppet-openstack | 17:21 | |
| *** dmsimard is now known as dmsimard|afk | 17:24 | |
| *** xingchao has quit IRC | 17:25 | |
| *** derekh has quit IRC | 17:25 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add resources to set ssl middleware for Nova https://review.openstack.org/268230 | 17:25 |
| *** saneax_AFK is now known as saneax | 17:26 | |
| *** saneax is now known as Guest82082 | 17:27 | |
| *** IvanBerezovskiy has left #puppet-openstack | 17:39 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-heat: Implement crontab to purge deleted data https://review.openstack.org/268249 | 17:43 |
| *** rahulait has quit IRC | 17:45 | |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-glance: Unify configuration of [keystone_authtoken] parameters https://review.openstack.org/268233 | 17:49 |
| *** ericksonsantos has quit IRC | 17:53 | |
| *** trown is now known as trown|lunch | 17:57 | |
| *** ntpttr has joined #puppet-openstack | 17:57 | |
| openstackgerrit | James Slagle proposed openstack/puppet-tripleo: Upgrade all packages after puppet managed ones https://review.openstack.org/268257 | 17:58 |
| *** ericksonsantos has joined #puppet-openstack | 18:05 | |
| *** dgurtner has joined #puppet-openstack | 18:11 | |
| *** dgurtner has quit IRC | 18:11 | |
| *** dgurtner has joined #puppet-openstack | 18:11 | |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-keystone: Refactor keystone::resource::authtoken https://review.openstack.org/266723 | 18:12 |
| *** xingchao has joined #puppet-openstack | 18:13 | |
| *** xingchao_ has quit IRC | 18:16 | |
| *** xingchao has quit IRC | 18:19 | |
| EmilienM | _ody: hey, about https://review.openstack.org/#/c/267763/ - yeah it seems sahara is not installed properly. Though if you look at my patch, I'm just using the gem now, and we did not change anything in there | 18:21 |
| EmilienM | _ody: see https://github.com/openstack/puppet-openstack_spec_helper/blob/master/lib/puppet-openstack_spec_helper/beaker_spec_helper.rb | 18:22 |
| *** trown|lunch is now known as trown | 18:27 | |
| *** ducttape_ has quit IRC | 18:32 | |
| _ody | EmilienM: This line is confusing me too, http://logs.openstack.org/63/267763/1/check/gate-puppet-sahara-puppet-beaker-rspec-dsvm-centos7/d244b06/console.html#_2016-01-14_22_27_53_259. | 18:38 |
| _ody | I can't seem to find where or why sahara would be cloned to "lib" then needing to be moved. | 18:38 |
| EmilienM | _ody: oh nice catch | 18:39 |
| _ody | I assume it is https://github.com/openstack/puppet-openstack_spec_helper/blob/master/lib/puppet-openstack_spec_helper/beaker_spec_helper.rb#L49 | 18:39 |
| EmilienM | yes | 18:40 |
| *** ericksonsantos has quit IRC | 18:40 | |
| _ody | That means the "proj_root" variable is for some reason parsing bad. | 18:40 |
| *** ericksonsantos has joined #puppet-openstack | 18:40 | |
| _ody | ...or I guess a new beaker bug, maybe? | 18:40 |
| _ody | puppet_module_install is a beaker helper as I reccall | 18:41 |
| *** skippy has left #puppet-openstack | 18:42 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add resources to set ssl middleware for Nova https://review.openstack.org/268230 | 18:49 |
| *** ericksonsantos has quit IRC | 18:53 | |
| openstackgerrit | Ben Nemec proposed openstack/puppet-tripleo: Handle redirects for Horizon https://review.openstack.org/268288 | 18:54 |
| *** dgurtner has quit IRC | 19:05 | |
| *** ducttape_ has joined #puppet-openstack | 19:11 | |
| *** ntpttr has left #puppet-openstack | 19:13 | |
| openstackgerrit | Merged openstack/puppet-modulesync-configs: Add rspec-puppet-facts to the spec_helper https://review.openstack.org/264830 | 19:14 |
| *** xingchao has joined #puppet-openstack | 19:15 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add resources to set ssl middleware for Nova https://review.openstack.org/268230 | 19:18 |
| *** xingchao has quit IRC | 19:20 | |
| *** imcsk8 has quit IRC | 19:25 | |
| EmilienM | _ody: back from lunch, looking now | 19:26 |
| *** imcsk8 has joined #puppet-openstack | 19:27 | |
| EmilienM | _ody: I think I got it | 19:27 |
| EmilienM | _ody: proj_root is not the path we want | 19:28 |
| EmilienM | it takes the new path in lib/ | 19:28 |
| *** ericksonsantos has joined #puppet-openstack | 19:32 | |
| EmilienM | _ody: I'm looking for another variable that would give the path of the module | 19:35 |
| EmilienM | we're using File.expand_path(File.join(File.dirname(__FILE__), '..')) | 19:36 |
| *** jistr has quit IRC | 19:37 | |
| EmilienM | I'm preparing a pathc | 19:41 |
| *** rahulait has joined #puppet-openstack | 19:42 | |
| _ody | EmilienM: I was comparing vs other modules and it looked like we had been using that variable for a while. | 19:42 |
| EmilienM | _ody: we need to change https://github.com/openstack/puppet-openstack_spec_helper/blob/master/lib/puppet-openstack_spec_helper/beaker_spec_helper.rb#L9 | 19:42 |
| EmilienM | _ody: because it's taking the path of the gem | 19:43 |
| EmilienM | and we want to the module path | 19:43 |
| _ody | Ah! | 19:43 |
| EmilienM | _ody: it's funny because the metadata of the module can be found | 19:43 |
| _ody | Because it is a gem now and not in the module. | 19:43 |
| EmilienM | _ody: yeah | 19:43 |
| _ody | Make sense now. | 19:43 |
| EmilienM | _ody: but I don't want to hardcode the module path | 19:43 |
| EmilienM | _ody: otherwise it will break people who do not run the same paths as our CI does | 19:44 |
| EmilienM | _ody: can't we get the path from where we run the require? | 19:44 |
| * EmilienM is reading http://ruby-doc.org/core-2.1.1/File.html#method-c-expand_path | 19:44 | |
| _ody | You can always brag CWD. | 19:45 |
| EmilienM | _ody: the CWD seems like the in lib dir, in the gem, no? | 19:45 |
| _ody | CWD should come from the runtime environment | 19:46 |
| _ody | So which directory you are in when you run ruby | 19:46 |
| *** dmsimard|afk is now known as dmsimard | 19:47 | |
| EmilienM | _ody: should I use http://ruby-doc.org/core-2.3.0/Dir.html#method-c-pwd ? | 19:47 |
| EmilienM | _ody: like proj_root = File.expand_path(File.join(Dir.getwd , '..')) | 19:49 |
| EmilienM | I tested it, it works | 19:50 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_spec_helper: beaker_spec_helper: build proj_root from cwd https://review.openstack.org/268312 | 19:53 |
| EmilienM | crinkle, _ody: maybe you can review when you have some time, thanks ^ | 19:53 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_spec_helper: beaker_spec_helper: build proj_root from cwd https://review.openstack.org/268312 | 19:53 |
| openstackgerrit | Ethan Gafford proposed openstack/puppet-trove: Use identity_uri and auth_uri in api module https://review.openstack.org/233229 | 20:00 |
| *** aimon has joined #puppet-openstack | 20:05 | |
| *** aimon_ has joined #puppet-openstack | 20:06 | |
| *** foxhunt_away is now known as foxhunt72 | 20:06 | |
| *** aimon has quit IRC | 20:09 | |
| *** angdraug has quit IRC | 20:09 | |
| openstackgerrit | Ben Nemec proposed openstack/puppet-tripleo: Handle redirects for Horizon https://review.openstack.org/268288 | 20:17 |
| *** xingchao has joined #puppet-openstack | 20:17 | |
| nibalizer | t | 20:17 |
| *** jfluhmann has joined #puppet-openstack | 20:20 | |
| *** xingchao has quit IRC | 20:21 | |
| *** jfluhmann has quit IRC | 20:24 | |
| *** ericksonsantos has quit IRC | 20:33 | |
| *** ericksonsantos has joined #puppet-openstack | 20:36 | |
| *** trown is now known as trown|afk | 20:38 | |
| _ody | EmilienM: done. That is what I had in mind. | 20:46 |
| EmilienM | cool | 20:46 |
| * _ody is fighting issues with horizon under uwsgi and nginx | 20:47 | |
| EmilienM | nginx? | 20:47 |
| EmilienM | never tried | 20:47 |
| _ody | Yeah. That is what we use here. | 20:47 |
| EmilienM | it's good! | 20:47 |
| _ody | It isn't working out so far but it is the last piece of our openstack deployment that is without puppet | 20:47 |
| *** ccie6747 has quit IRC | 20:48 | |
| _ody | We have really old apache module we can't easily get rid of so using puppetlabs-apache is also difficult. | 20:48 |
| EmilienM | _ody: feel fee to submit a patch in puppet-horizon, we already support apache ;-) | 20:48 |
| _ody | When I figure it out...requires a uwsgi module I quickly threw together. | 20:48 |
| _ody | I'll have to clean it up up publish it when it works | 20:49 |
| *** foxhunt72 is now known as foxhunt_away | 20:49 | |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Add simple keystone acceptance test https://review.openstack.org/263461 | 20:53 |
| *** angdraug has joined #puppet-openstack | 21:01 | |
| *** jaosorior has quit IRC | 21:05 | |
| *** _matthias_ has quit IRC | 21:06 | |
| *** cdelatte has quit IRC | 21:10 | |
| *** _matthias_ has joined #puppet-openstack | 21:14 | |
| openstackgerrit | Merged openstack/puppet-openstack_spec_helper: beaker_spec_helper: build proj_root from cwd https://review.openstack.org/268312 | 21:17 |
| *** xingchao has joined #puppet-openstack | 21:18 | |
| EmilienM | let's recheck https://review.openstack.org/#/c/267763/ now | 21:20 |
| *** ccie6747 has joined #puppet-openstack | 21:21 | |
| *** xingchao has quit IRC | 21:22 | |
| *** dprince has quit IRC | 21:43 | |
| *** dfisher has quit IRC | 21:45 | |
| *** jtomasek has quit IRC | 21:49 | |
| openstackgerrit | Mike Dorman proposed openstack/puppet-nova: Nova cells support for nova_aggregate provider https://review.openstack.org/267866 | 21:57 |
| *** trown|afk is now known as trown | 21:58 | |
| *** cdelatte has joined #puppet-openstack | 21:59 | |
| *** cdelatte has quit IRC | 22:00 | |
| *** cdelatte has joined #puppet-openstack | 22:00 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-heat: Implement crontab to purge deleted data https://review.openstack.org/268249 | 22:01 |
| *** ccie6747 has quit IRC | 22:16 | |
| *** xingchao has joined #puppet-openstack | 22:18 | |
| *** jayg is now known as jayg|g0n3 | 22:20 | |
| *** xingchao has quit IRC | 22:23 | |
| *** jpena is now known as jpena|sleep | 22:26 | |
| openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Add simple keystone acceptance test https://review.openstack.org/263461 | 22:31 |
| *** ducttape_ has quit IRC | 22:31 | |
| EmilienM | I think I broke CI | 22:43 |
| EmilienM | http://logs.openstack.org/49/268249/2/check/gate-puppet-heat-puppet-beaker-rspec-dsvm-trusty/c1c7734/console.html#_2016-01-15_22_10_06_017 | 22:43 |
| EmilienM | on a Friday, nice | 22:43 |
| chem | well you'll have all the week-end to fix, np :) | 22:45 |
| dmsimard | EmilienM: ah so there's indeed a heat problem then ? | 22:45 |
| EmilienM | dmsimard: no it's something else | 22:45 |
| EmilienM | it's beaker jobs | 22:45 |
| EmilienM | not integration | 22:45 |
| dmsimard | oh, okay. | 22:45 |
| dmsimard | I'll keep looking :p | 22:45 |
| EmilienM | and beaker jobs are voting ! | 22:45 |
| chem | that's a strange module name gate-puppet-heat-puppet-beaker-rspec-dsvm-trusty | 22:46 |
| EmilienM | yes... | 22:46 |
| chem | an job env var put in the wrong place | 22:46 |
| EmilienM | chem: https://github.com/openstack/puppet-openstack_spec_helper/commit/8a5c3e36de3b33f54811e88903527b311ccc04aa | 22:46 |
| EmilienM | that's my fault | 22:46 |
| EmilienM | we might want to drop the '..' | 22:47 |
| chem | EmilienM: sorry, I don't know where to look ... ah oki | 22:47 |
| chem | EmilienM: Ah !!! you did it again :) | 22:47 |
| *** baker has quit IRC | 22:48 | |
| *** trown is now known as trown|outttypeww | 22:48 | |
| chem | EmilienM: just revert it. You cannot know where the getwd (process directory) will be. | 22:49 |
| EmilienM | in fact yes | 22:49 |
| chem | EmilienM: you can ? | 22:49 |
| EmilienM | when you run rspec, you're supposed to be in the root dir no? | 22:49 |
| EmilienM | I mean, in theory... | 22:49 |
| chem | not necessarily you can be further down the hierarchy | 22:50 |
| EmilienM | I know | 22:50 |
| EmilienM | but I don't know how to deal with that in another way then | 22:50 |
| EmilienM | actually, we could try to find the metadata.json | 22:51 |
| EmilienM | and get the path from there | 22:51 |
| chem | EmilienM: that's what we already doing no ? | 22:51 |
| EmilienM | no | 22:51 |
| EmilienM | modname = JSON.parse(open('metadata.json').read)['name'].split('-')[1] | 22:51 |
| chem | module_name = JSON.parse(File.read(File.join(module_root, 'metadata.json')))['name'] | 22:51 |
| chem | .sub(/^(?:[^-]+-)(.*)/, '\1') | 22:51 |
| chem | 22:51 | |
| chem | that's one of mine :) | 22:52 |
| EmilienM | chem: where do you see the sub? | 22:52 |
| chem | EmilienM: ? | 22:52 |
| EmilienM | chem: where do you see this code? | 22:52 |
| chem | EmilienM: sorry, it's local it seems ... | 22:53 |
| chem | but your split is better ... | 22:53 |
| *** gfidente has quit IRC | 22:53 | |
| chem | for module_root I use module_root = File.expand_path(File.join(File.dirname(__FILE__), '..')) | 22:53 |
| EmilienM | chem: what do you suggest for proj_root ? | 22:53 |
| chem | so that won't help | 22:53 |
| chem | oki, let me think about the context ... you're a lib not in the module path anymore | 22:54 |
| chem | ... | 22:54 |
| EmilienM | proj_root = File.expand_path(File.join(Dir.getwd)) <--- that would work but not if you run beaker in spec dir for example | 22:54 |
| EmilienM | chem: right | 22:54 |
| * chem thinking | 22:54 | |
| *** baker has joined #puppet-openstack | 22:54 | |
| *** dhollinger has quit IRC | 22:55 | |
| EmilienM | oh dear I broke all CI | 22:55 |
| chem | oki, yes just remove the '..' you'll be fine 99 of the time | 22:55 |
| EmilienM | yeah | 22:55 |
| EmilienM | add maybe a comment for doc | 22:56 |
| chem | You can add a check for an existing Rakefile or metadata.json in the directory and exit with a nice error message if it's missing | 22:57 |
| chem | but not all project have a metadata.json or a Rakefile ... (puppet-pacemaker someone ?) | 22:57 |
| EmilienM | chem: puppet-pacemaker does not belong to us | 22:57 |
| chem | in the checked project in openstack I think they all have them | 22:57 |
| EmilienM | they do have | 22:57 |
| chem | EmilienM: I know :) | 22:58 |
| chem | so you add the check and the nice error message | 22:58 |
| chem | so you can add the check and the nice error message | 22:58 |
| EmilienM | ok | 22:58 |
| chem | so know you cover 100% of the case \o/ | 22:58 |
| *** mdorman has quit IRC | 22:59 | |
| openstackgerrit | Mike Dorman proposed openstack/puppet-nova: Sort host_aggregate hosts list for comparison https://review.openstack.org/268364 | 22:59 |
| *** mdorman has joined #puppet-openstack | 23:01 | |
| EmilienM | chem: patch coming | 23:04 |
| chem | ack | 23:05 |
| *** baker has quit IRC | 23:05 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_spec_helper: beaker_spec_helper: fix module root path https://review.openstack.org/268365 | 23:06 |
| EmilienM | chem: ^ | 23:06 |
| EmilienM | crinkle, _ody: this is outstanding, please look | 23:07 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_spec_helper: beaker_spec_helper: fix module root path https://review.openstack.org/268365 | 23:16 |
| *** xingchao has joined #puppet-openstack | 23:19 | |
| *** xingchao has quit IRC | 23:24 | |
| *** EmilienM sets mode: +o crinkle | 23:27 | |
| *** EmilienM sets mode: +o mgagne | 23:27 | |
| *** EmilienM sets mode: +o clayton | 23:27 | |
| *** EmilienM sets mode: +o spredzy | 23:27 | |
| EmilienM | any core around, please review https://review.openstack.org/#/c/268365/ <-- this is outstanding | 23:32 |
| *** ducttape_ has joined #puppet-openstack | 23:35 | |
| openstackgerrit | Merged openstack/puppet-openstack_spec_helper: beaker_spec_helper: fix module root path https://review.openstack.org/268365 | 23:49 |
| openstackgerrit | Mike Dorman proposed openstack/puppet-nova: Allow metadata hash for nova_aggregate resources https://review.openstack.org/268376 | 23:53 |
| EmilienM | it does not work | 23:57 |
| EmilienM | mv /etc/puppet/modules/gate-puppet-sahara-puppet-beaker-rspec-dsvm-centos7 /etc/puppet/modules/sahara | 23:57 |
| EmilienM | not sure that's what we want | 23:58 |
| EmilienM | https://jenkins01.openstack.org/job/gate-puppet-sahara-puppet-beaker-rspec-dsvm-centos7/46/console | 23:58 |
| EmilienM | oh it works actually | 23:58 |
| EmilienM | chem: ^ | 23:58 |
| chem | EmilienM: great :) | 23:59 |
| EmilienM | chem: thanks for your help and also to reviewers | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!