| *** cwolferh has joined #puppet-openstack | 00:03 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: repos: bump rdo to latest trunk https://review.openstack.org/328007 | 00:03 |
|---|---|---|
| openstackgerrit | Emilien Macchi proposed openstack/puppet-octavia: Implement api.pp https://review.openstack.org/326142 | 00:04 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial https://review.openstack.org/327678 | 00:06 |
| iurygregory_ | EmilienM, i think i have an account '-' | 00:10 |
| *** xarses has joined #puppet-openstack | 00:10 | |
| EmilienM | iurygregory_: yeah I follow you now | 00:11 |
| iurygregory_ | but i don't use because i can't run with my smartphone =X | 00:11 |
| EmilienM | so i'll check on you that you do running :P | 00:11 |
| EmilienM | buy a garmin! | 00:11 |
| iurygregory_ | a what? O.o | 00:11 |
| EmilienM | https://explore.garmin.com/en-US/forerunner/ | 00:11 |
| iurygregory_ | lol | 00:11 |
| *** ansmith has quit IRC | 00:12 | |
| iurygregory_ | this is expensive i think O.o | 00:12 |
| iurygregory_ | omg $220 | 00:13 |
| EmilienM | iurygregory_: you can have one for < 100 USD | 00:13 |
| iurygregory_ | EmilienM, the reason to not use my smartphone is just because my city does not have security =X (many thieves) | 00:16 |
| iurygregory_ | =P | 00:16 |
| iurygregory_ | but $100 is a good price i think. maybe i can find in Barcelona :D | 00:16 |
| EmilienM | iurygregory_: stay safe! | 00:17 |
| *** igorbelikov has quit IRC | 00:17 | |
| *** petems has quit IRC | 00:18 | |
| *** aglarendil has quit IRC | 00:18 | |
| *** patrickeast has quit IRC | 00:19 | |
| *** kgalanov has quit IRC | 00:19 | |
| *** igorbelikov has joined #puppet-openstack | 00:20 | |
| *** aglarendil has joined #puppet-openstack | 00:20 | |
| iurygregory_ | EmilienM, oh yeah! and this month is crazy because we have a traditional festival XD (30 days of party lol) | 00:20 |
| *** petems has joined #puppet-openstack | 00:21 | |
| *** patrickeast has joined #puppet-openstack | 00:22 | |
| *** kgalanov has joined #puppet-openstack | 00:22 | |
| *** jgrassler has quit IRC | 00:31 | |
| *** jgrassler has joined #puppet-openstack | 00:31 | |
| *** zigo has quit IRC | 00:37 | |
| *** sthillma has quit IRC | 00:43 | |
| iurygregory_ | Thanks good your patch for octavia has passed EmilienM \o/ | 00:48 |
| iurygregory_ | god* | 00:48 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: beaker: remove multi domain config resources output test https://review.openstack.org/327894 | 00:49 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: Add nodepool-xenial.yml file https://review.openstack.org/320554 | 00:49 |
| richm | EmilienM: can you confirm that you started seeing the failures after https://github.com/openstack/puppet-openstack-integration/commit/427f6dda0a45a0bc4f60a627921597ef1dc4c89f ? | 00:50 |
| richm | that is, after updating to the latest version of centos7 rdo? | 00:50 |
| EmilienM | richm: again, we update every day | 00:51 |
| EmilienM | richm: but yes | 00:51 |
| EmilienM | something in OpenStack broke us the last 2 days | 00:51 |
| EmilienM | I'm investigating what we need to disable to make it work again | 00:51 |
| richm | EmilienM: I think it has to do with this: https://github.com/openstack/keystone/commit/d6b016dd91c743a2f454a3b4f9d055510c2215ae | 00:52 |
| EmilienM | all fails I think | 00:52 |
| EmilienM | richm: but our integration tests work fine :/ | 00:53 |
| richm | EmilienM: in the last good run, I don't see the message Reset password for user admin. | 00:53 |
| richm | EmilienM: in the bad run, the first auth operation after that message fails | 00:53 |
| EmilienM | in keystone logs? | 00:53 |
| richm | yes | 00:53 |
| EmilienM | mhh | 00:53 |
| EmilienM | damn | 00:53 |
| richm | trying to find in the puppet-keystone code where keystone-manage bootstrap is run and, if so, what is the password argument | 00:54 |
| EmilienM | richm: https://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L1077-L1084 | 00:54 |
| richm | I guess you need to use bootstrap to create the default domain? But now it nukes the admin password. | 00:55 |
| EmilienM | richm: this is how we deploy keystone in our CI https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/keystone.pp | 00:55 |
| richm | you don't specify the bootstrap argument, and the default is `true` | 00:56 |
| openstackgerrit | Dmitry Ilyin proposed openstack/puppet-ec2api: Add all other classes https://review.openstack.org/314598 | 00:56 |
| openstackgerrit | Dmitry Ilyin proposed openstack/puppet-ec2api: Add db related classes https://review.openstack.org/314597 | 00:56 |
| openstackgerrit | Dmitry Ilyin proposed openstack/puppet-ec2api: Add 3 api classes and params https://review.openstack.org/314596 | 00:56 |
| richm | oh - you do pass the admin token as the password argument | 00:56 |
| EmilienM | yes | 00:56 |
| richm | assuming admin_token == password for the user 'admin' | 00:57 |
| richm | but if you're using token auth everywhere, it should not matter what the password is | 00:57 |
| EmilienM | richm: FYI https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/keystone.pp#L79 | 00:58 |
| richm | when you create the admin user, do you set the password to the admin_token? | 00:58 |
| EmilienM | we run: keystone-manage bootstrap --bootstrap-password ${admin_token} | 00:59 |
| EmilienM | and yes they are same passwords | 00:59 |
| EmilienM | well wait | 01:00 |
| EmilienM | admin_token = 'admin_token' | 01:00 |
| richm | it may be a red herring, but it is the only clue I have so far | 01:00 |
| EmilienM | and admin = 'a_big_secret' | 01:00 |
| EmilienM | see https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/keystone.pp | 01:00 |
| *** zigo has joined #puppet-openstack | 01:03 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Import copy_logs.sh script from openstack-infra/project-config https://review.openstack.org/328014 | 01:03 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Import copy_logs.sh script from openstack-infra/project-config https://review.openstack.org/328015 | 01:04 |
| EmilienM | mwhahaha: backports ^ auto approved, so we can have logs in our stable CI | 01:04 |
| richm | EmilienM: so after the bootstrap is run, the puppet-keystone openstack provider tries to do password auth with user 'admin' and password 'a_big_secret', but the password was changed to 'admin_token'? | 01:04 |
| richm | so auth fails? | 01:05 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: copy_logs: grab syslog with journalctl on Ubuntu https://review.openstack.org/328016 | 01:06 |
| iurygregory_ | lol we change the password is change? O.o | 01:06 |
| iurygregory_ | we change the password for the admin user?* | 01:06 |
| richm | iurygregory_: yes | 01:07 |
| iurygregory_ | enr '-' | 01:07 |
| *** ayoung has joined #puppet-openstack | 01:07 | |
| richm | I think this line in keystone/init.pp is the problem: | 01:07 |
| richm | command => "keystone-manage bootstrap --bootstrap-password ${admin_token}", | 01:07 |
| richm | should use ${admin_password} | 01:07 |
| richm | but we don't have the admin_password in this context :-( | 01:07 |
| iurygregory_ | if we pass admin_token this is token to be used .-. if we change admin password but we try to use only the admin token this should work i think | 01:10 |
| richm | the --bootstrap-password argument is used here: https://github.com/openstack/keystone/blob/master/keystone/cmd/cli.py#L87 | 01:11 |
| richm | then here: https://github.com/openstack/keystone/blob/master/keystone/cmd/cli.py#L143 | 01:11 |
| richm | then here: https://github.com/openstack/keystone/blob/master/keystone/cmd/cli.py#L223 | 01:11 |
| EmilienM | richm: so admin token & admin password should be the same in our CI? | 01:14 |
| EmilienM | one day I'll understand how keystone works I swear! | 01:14 |
| richm | EmilienM: that would be the most immediate fix, yes | 01:15 |
| richm | EmilienM: don't worry - no one else will understand it either :P | 01:15 |
| iurygregory_ | ahsuahsaushaush | 01:15 |
| EmilienM | ok | 01:15 |
| * richm blames ayoung . . . | 01:15 | |
| EmilienM | let's try it | 01:15 |
| EmilienM | we always blame ayoung | 01:15 |
| EmilienM | everybody blames ayoung | 01:15 |
| ayoung | Yep | 01:15 |
| * bkero blames ayoung | 01:15 | |
| iurygregory_ | i don't D: | 01:15 |
| ayoung | It's all my fault | 01:15 |
| EmilienM | richm: can you patch puppet-keystone please? I dont even know what to write in commit message | 01:16 |
| EmilienM | we should rewrite keystone in GO | 01:16 |
| iurygregory_ | LOL | 01:16 |
| ayoung | so..I missed the early part of this | 01:16 |
| EmilienM | and this problem would be solved | 01:16 |
| * richm runs | 01:16 | |
| EmilienM | richm: no no stay here | 01:16 |
| nibalizer | EmilienM: yep letsget started | 01:16 |
| * EmilienM locks the door | 01:16 | |
| richm | I'm running from go | 01:16 |
| nibalizer | we can put raft in it and everything | 01:16 |
| ayoung | admin_token is, I think, a thing of the past once we talk bootstrap. What you wrote above looks sane | 01:16 |
| richm | ayoung: the problem is this: command => "keystone-manage bootstrap --bootstrap-password ${admin_token}", | 01:17 |
| richm | that should be $admin_password | 01:17 |
| richm | the token and the password are different | 01:17 |
| EmilienM | richm: it's a critical backward incompatible change | 01:18 |
| EmilienM | IMHO | 01:18 |
| ayoung | richm, EmilienM we should remove references to admin_token now that we are using bootstrap. | 01:18 |
| ayoung | EmilienM, here is the short version | 01:18 |
| * EmilienM unlock the door and run | 01:18 | |
| iurygregory_ | lol | 01:18 |
| ayoung | admin_token was a hacky, not very secure way to allow a user to do initiaol configuration of the system. It was hard to disable. | 01:18 |
| EmilienM | yeah, right | 01:18 |
| *** cwolferh has quit IRC | 01:19 | |
| richm | EmilienM: I think I need to change openstack_integration? | 01:19 |
| ayoung | notmorgan (Fainberg) took exception to this and fixed it. THe admin_password should be different from admin_token in a long lived system | 01:19 |
| EmilienM | richm: yes sorry | 01:19 |
| ayoung | if we were still using admin token, it could easiler be something like my old favorite FreeIPA4All, because, by the time the system is up and running, it should be disabled | 01:19 |
| ayoung | where as admin_password should be `uuid-gen -r` and will be used by the admin user to authenticate after the system is in production | 01:20 |
| EmilienM | ok now in french please | 01:20 |
| EmilienM | well admin_password & admin_token are different in our CI | 01:20 |
| EmilienM | but beaker tests do not work anymore | 01:21 |
| EmilienM | maybe because we run bootstrap or something? | 01:21 |
| ayoung | admin_token était un pas de manière très sécurisée hacky pour permettre à un utilisateur de faire la configuration initiaol du système. Il était difficile de désactiver. notmorgan (Fainberg) a pris exception et réparée. THe mot_de_passe_admin doit être différent de admin_token dans un système de longue durée. si nous utilisions encore administrateur jeton, il pourrait easiler être quelque chose comme mon vi | 01:21 |
| ayoung | eux FreeIPA4All favori, parce que, au moment où le système est en marche, il doit être désactivé. où comme mot_de_passe_admin devrait être `uuid-gen -r` et sera utilisée par l'utilisateur admin pour authentifier après que le système est en production. | 01:21 |
| EmilienM | ah, thanks, it's better now | 01:21 |
| ayoung | Mon aéroglisseur est plein d'anguilles. | 01:22 |
| * iurygregory_ is not understanding anything | 01:22 | |
| *** EmilienM changes topic to "CI status: orange (centos7 beaker jobs failing on puppet-keystone) - Puppet OpenStack modules - http://docs.openstack.org/developer/puppet-openstack-guide/" | 01:22 | |
| EmilienM | ayoung: lol | 01:22 |
| ayoung | what are the failing tests? | 01:23 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: CI test - never merge https://review.openstack.org/328005 | 01:24 |
| EmilienM | ayoung: http://logs.openstack.org/44/327844/1/check/gate-puppet-keystone-puppet-beaker-rspec-centos-7/e7a05a5/console.html#_2016-06-09_18_36_55_349 | 01:24 |
| iurygregory_ | best way to get -1 in CI LOL | 01:25 |
| ayoung | nothing resembling a stack trace? | 01:25 |
| EmilienM | ayoung: maybe richm has more thoughts on the error, I havent spent time, I'm really multi tasking on multiple CI issues | 01:26 |
| iurygregory_ | http://logs.openstack.org/44/327844/1/check/gate-puppet-keystone-puppet-beaker-rspec-centos-7/e7a05a5/logs/apache/keystone_wsgi_admin_access.txt.gz | 01:28 |
| iurygregory_ | python-openstackclient keystoneauth1/2.8.1 python-requests/2.10.0 CPython/2.7.5 rich told something about this if i'm right | 01:28 |
| openstackgerrit | Richard Megginson proposed openstack/puppet-openstack-integration: the admin_password is now the same as the admin_token https://review.openstack.org/328026 | 01:28 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: fail if repos are not installed when asked https://review.openstack.org/328028 | 01:29 |
| richm | This was the first error I found - looked like many other errors were caused by this one - I didn't look any further | 01:30 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: CI test - never merge https://review.openstack.org/327844 | 01:30 |
| EmilienM | richm: let's see if beaker jobs work, I used depends-on ^ | 01:30 |
| EmilienM | richm: see my comment on your patch though | 01:31 |
| openstackgerrit | Richard Megginson proposed openstack/puppet-openstack-integration: the admin_password is now the same as the admin_token https://review.openstack.org/328026 | 01:33 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: CI test - never merge https://review.openstack.org/327844 | 01:33 |
| *** ducttape_ has joined #puppet-openstack | 01:34 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial https://review.openstack.org/327678 | 01:39 |
| openstackgerrit | Merged openstack/puppet-openstack-integration: refacto: move run_puppet & catch_selinux_alerts into functions https://review.openstack.org/327197 | 01:39 |
| *** ilbot3 has quit IRC | 01:45 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: beaker: remove multi domain config resources output test https://review.openstack.org/327894 | 01:45 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: Add nodepool-xenial.yml file https://review.openstack.org/320554 | 01:46 |
| *** ilbot3 has joined #puppet-openstack | 01:48 | |
| *** iurygregory_ has quit IRC | 01:50 | |
| richm | EmilienM: Can we see the ci testing for https://review.openstack.org/#/c/327844/ in progress? I'd like to see if it can at least get past the point of the first failure | 01:50 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Import copy_logs.sh script from openstack-infra/project-config https://review.openstack.org/328015 | 01:51 |
| EmilienM | richm: zuul.openstack.org | 01:52 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Import copy_logs.sh script from openstack-infra/project-config https://review.openstack.org/328014 | 01:52 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: fail if repos are not installed when asked https://review.openstack.org/328028 | 01:55 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: repos: bump rdo to latest trunk https://review.openstack.org/328007 | 01:56 |
| EmilienM | richm: they are still running, watch 327844 | 01:56 |
| EmilienM | 2 min left | 01:56 |
| richm | uh - telnet? | 01:57 |
| EmilienM | zuul ui | 01:57 |
| richm | ah, cool | 01:57 |
| EmilienM | richm: status.openstack.org/zuul/ | 01:58 |
| richm | looks like it got past the point of the failure | 02:00 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Import ::nova class from THT https://review.openstack.org/325982 | 02:04 |
| EmilienM | richm: success! | 02:04 |
| EmilienM | richm: so yeah it works, that's cool. I just want to highlight we found a critical thing in our CI that might be a non backward compatible change in keystone cc ayoung | 02:06 |
| *** EmilienM changes topic to "CI status: green - Puppet OpenStack modules - http://docs.openstack.org/developer/puppet-openstack-guide/" | 02:08 | |
| richm | it's not easy being green . . . | 02:10 |
| EmilienM | lol | 02:21 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: fail if repos are not installed when asked https://review.openstack.org/328028 | 02:24 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Implement Nova API profile https://review.openstack.org/325937 | 02:25 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Deploy nova::db classes in api profile https://review.openstack.org/326027 | 02:27 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Implement Nova API profile https://review.openstack.org/325937 | 02:28 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Deploy nova::db classes in api profile https://review.openstack.org/326027 | 02:30 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: beaker: remove multi domain config resources output test https://review.openstack.org/327894 | 02:33 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: Add nodepool-xenial.yml file https://review.openstack.org/320554 | 02:33 |
| *** ducttape_ has quit IRC | 02:39 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Import copy_logs.sh script from openstack-infra/project-config https://review.openstack.org/328014 | 02:45 |
| *** richm has quit IRC | 02:52 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Import copy_logs.sh script from openstack-infra/project-config https://review.openstack.org/328014 | 02:55 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial https://review.openstack.org/327678 | 02:58 |
| openstackgerrit | Merged openstack/puppet-openstack-integration: the admin_password is now the same as the admin_token https://review.openstack.org/328026 | 03:03 |
| *** Shaik has quit IRC | 03:04 | |
| *** Shaik has joined #puppet-openstack | 03:12 | |
| *** markvoelker has quit IRC | 03:14 | |
| *** saneax is now known as saneax_AFK | 03:27 | |
| *** jaosorior has joined #puppet-openstack | 03:48 | |
| openstackgerrit | Venkata Mahesh Jonnalagadda proposed openstack/puppet-mistral: Provide default service_name for keystone endpoint https://review.openstack.org/324356 | 04:06 |
| openstackgerrit | Venkata Mahesh Jonnalagadda proposed openstack/puppet-heat: Provide default service_name for keystone endpoint https://review.openstack.org/324764 | 04:21 |
| *** saneax_AFK is now known as saneax | 04:30 | |
| openstackgerrit | Venkata Mahesh Jonnalagadda proposed openstack/puppet-murano: Provide default service_name for keystone endpoint https://review.openstack.org/324749 | 04:32 |
| *** cwolferh has joined #puppet-openstack | 04:51 | |
| *** Shaik has quit IRC | 04:56 | |
| *** darkxploit has quit IRC | 04:56 | |
| openstackgerrit | Venkata Mahesh Jonnalagadda proposed openstack/puppet-heat: Provide default service_name for keystone endpoint https://review.openstack.org/324764 | 05:03 |
| openstackgerrit | Venkata Mahesh Jonnalagadda proposed openstack/puppet-mistral: Provide default service_name for keystone endpoint https://review.openstack.org/324356 | 05:07 |
| openstackgerrit | Venkata Mahesh Jonnalagadda proposed openstack/puppet-murano: Provide default service_name for keystone endpoint https://review.openstack.org/324749 | 05:10 |
| *** darkxploit has joined #puppet-openstack | 05:14 | |
| *** markvoelker has joined #puppet-openstack | 05:15 | |
| *** markvoelker has quit IRC | 05:20 | |
| openstackgerrit | Venkata Mahesh Jonnalagadda proposed openstack/puppet-murano: Provide default service_name for keystone endpoint https://review.openstack.org/324749 | 05:24 |
| *** darkxploit has quit IRC | 05:53 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for keystone https://review.openstack.org/327029 | 06:06 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for heat https://review.openstack.org/327069 | 06:06 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for glance API and registry https://review.openstack.org/327473 | 06:06 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for RabbitMQ https://review.openstack.org/327482 | 06:06 |
| *** yolanda has joined #puppet-openstack | 06:06 | |
| *** darkxploit has joined #puppet-openstack | 06:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL https://review.openstack.org/328090 | 06:12 |
| *** danpawlik has joined #puppet-openstack | 06:24 | |
| *** cwolferh has quit IRC | 06:32 | |
| *** openstackgerrit has quit IRC | 06:32 | |
| *** foxhunt_away is now known as foxhunt72 | 06:33 | |
| *** openstackgerrit has joined #puppet-openstack | 06:33 | |
| *** cwolferh has joined #puppet-openstack | 06:37 | |
| openstackgerrit | Venkata Mahesh Jonnalagadda proposed openstack/puppet-murano: Provide default service_name for keystone endpoint https://review.openstack.org/324749 | 06:43 |
| *** jmikolajczak has joined #puppet-openstack | 06:51 | |
| *** dgurtner has joined #puppet-openstack | 06:55 | |
| *** xarses has quit IRC | 06:59 | |
| *** xarses has joined #puppet-openstack | 06:59 | |
| openstackgerrit | Venkata Mahesh Jonnalagadda proposed openstack/puppet-designate: Provide default service_name for keystone endpoint https://review.openstack.org/326031 | 07:11 |
| openstackgerrit | Merged openstack/puppet-tripleo: Enable bulk delete in swift for gnocchi https://review.openstack.org/327214 | 07:13 |
| openstackgerrit | Daniel Pawlik proposed openstack/puppet-watcher: This commit adds the initial project files. https://review.openstack.org/326593 | 07:14 |
| *** permalac has joined #puppet-openstack | 07:15 | |
| *** markvoelker has joined #puppet-openstack | 07:16 | |
| *** rcernin has joined #puppet-openstack | 07:17 | |
| *** markvoelker has quit IRC | 07:21 | |
| *** amoralej|off is now known as amoralej | 07:23 | |
| *** xarses has quit IRC | 07:23 | |
| *** xarses_ has joined #puppet-openstack | 07:24 | |
| *** xarses_ has quit IRC | 07:24 | |
| *** xarses_ has joined #puppet-openstack | 07:24 | |
| *** dtantsur|afk is now known as dtantsur | 07:25 | |
| *** tesseract has joined #puppet-openstack | 07:27 | |
| *** aimon_ has joined #puppet-openstack | 07:29 | |
| *** darkxploit has quit IRC | 07:31 | |
| *** aimon has quit IRC | 07:31 | |
| *** openstackgerrit has quit IRC | 07:33 | |
| *** openstackgerrit has joined #puppet-openstack | 07:33 | |
| *** jpena|off is now known as jpena | 07:34 | |
| openstackgerrit | Venkata Mahesh Jonnalagadda proposed openstack/puppet-trove: Add missing keystone service and endpoint tests https://review.openstack.org/323960 | 07:38 |
| *** darkxploit has joined #puppet-openstack | 07:49 | |
| *** cwolferh has quit IRC | 07:54 | |
| openstackgerrit | Venkata Mahesh Jonnalagadda proposed openstack/puppet-designate: Add user & user_role parameters in auth manifest and related tests https://review.openstack.org/324296 | 07:56 |
| *** karimb has joined #puppet-openstack | 07:56 | |
| *** cwolferh has joined #puppet-openstack | 08:00 | |
| *** jtomasek_ has joined #puppet-openstack | 08:03 | |
| *** shardy has joined #puppet-openstack | 08:03 | |
| *** markvoelker has joined #puppet-openstack | 08:17 | |
| *** jtomasek_ has quit IRC | 08:20 | |
| *** markvoelker has quit IRC | 08:21 | |
| *** queretaro has joined #puppet-openstack | 08:24 | |
| openstackgerrit | Daniel Pawlik proposed openstack/puppet-watcher: This commit adds the initial project files. https://review.openstack.org/326593 | 08:26 |
| *** jaosorior has quit IRC | 08:30 | |
| *** jaosorior has joined #puppet-openstack | 08:30 | |
| *** hjensas has joined #puppet-openstack | 08:31 | |
| *** hjensas has quit IRC | 08:32 | |
| *** ifarkas has joined #puppet-openstack | 08:37 | |
| *** darkxploit has quit IRC | 08:42 | |
| *** hjensas has joined #puppet-openstack | 08:45 | |
| *** hjensas has quit IRC | 08:45 | |
| *** hjensas has joined #puppet-openstack | 08:45 | |
| *** jmikolajczak has quit IRC | 08:46 | |
| *** darkxploit has joined #puppet-openstack | 08:55 | |
| *** darkxploit has quit IRC | 09:02 | |
| *** dgurtner has quit IRC | 09:08 | |
| *** jmikolajczak has joined #puppet-openstack | 09:16 | |
| *** _degorenko|afk is now known as degorenko | 09:19 | |
| *** dgurtner has joined #puppet-openstack | 09:21 | |
| *** dgurtner has quit IRC | 09:21 | |
| *** dgurtner has joined #puppet-openstack | 09:21 | |
| *** dgurtner has quit IRC | 09:25 | |
| *** dgurtner has joined #puppet-openstack | 09:25 | |
| *** evidex has joined #puppet-openstack | 09:32 | |
| *** todin has joined #puppet-openstack | 09:36 | |
| *** dtantsur is now known as dtantsur|lunch | 09:42 | |
| *** foxhunt72 is now known as foxhunt_away | 09:43 | |
| *** iberezovskiy|off is now known as iberezovskiy | 09:44 | |
| *** queretaro has quit IRC | 10:05 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: copy_logs: grab syslog with journalctl on Ubuntu https://review.openstack.org/328016 | 10:15 |
| *** queretaro has joined #puppet-openstack | 10:17 | |
| *** markvoelker has joined #puppet-openstack | 10:18 | |
| *** markvoelker has quit IRC | 10:22 | |
| openstackgerrit | Oleksiy Molchanov proposed openstack/puppet-ceph: Fix activate check https://review.openstack.org/310507 | 10:24 |
| *** johnmilton has quit IRC | 10:25 | |
| *** cwolferh has quit IRC | 10:32 | |
| *** cwolferh has joined #puppet-openstack | 10:36 | |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker. https://review.openstack.org/309069 | 10:36 |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker. https://review.openstack.org/309069 | 10:37 |
| *** foxhunt_away is now known as foxhunt72 | 10:43 | |
| openstackgerrit | Venkata Mahesh Jonnalagadda proposed openstack/puppet-heat: Provide default service_name for keystone endpoint https://review.openstack.org/324764 | 10:46 |
| *** dtantsur|lunch is now known as dtantsur | 10:58 | |
| openstackgerrit | Venkata Mahesh Jonnalagadda proposed openstack/puppet-mistral: Provide default service_name for keystone endpoint https://review.openstack.org/324356 | 11:04 |
| *** dmk0202 has joined #puppet-openstack | 11:10 | |
| *** ansmith has joined #puppet-openstack | 11:10 | |
| openstackgerrit | Venkata Mahesh Jonnalagadda proposed openstack/puppet-murano: Provide default service_name for keystone endpoint https://review.openstack.org/324749 | 11:13 |
| *** johnmilton has joined #puppet-openstack | 11:16 | |
| *** foxhunt72 is now known as foxhunt_away | 11:21 | |
| openstackgerrit | Venkata Mahesh Jonnalagadda proposed openstack/puppet-designate: Provide default service_name for keystone endpoint https://review.openstack.org/326031 | 11:21 |
| *** foxhunt_away is now known as foxhunt72 | 11:21 | |
| *** jpena is now known as jpena|lunch | 11:22 | |
| *** dmk0202 has quit IRC | 11:23 | |
| *** dmk0202 has joined #puppet-openstack | 11:36 | |
| openstackgerrit | Venkata Mahesh Jonnalagadda proposed openstack/puppet-murano: Provide default service_name for keystone endpoint https://review.openstack.org/324749 | 11:41 |
| openstackgerrit | Dmitry Ilyin proposed openstack/puppet-ec2api: Add all other classes https://review.openstack.org/314598 | 11:43 |
| openstackgerrit | Dmitry Ilyin proposed openstack/puppet-ec2api: Add db related classes https://review.openstack.org/314597 | 11:43 |
| openstackgerrit | Dmitry Ilyin proposed openstack/puppet-ec2api: Add 3 api classes and params https://review.openstack.org/314596 | 11:43 |
| *** dprince has joined #puppet-openstack | 11:44 | |
| *** arnewiebalck has quit IRC | 11:50 | |
| *** Vinsh has quit IRC | 11:53 | |
| *** morazi has joined #puppet-openstack | 11:55 | |
| *** danpawlik has quit IRC | 11:56 | |
| *** danpawlik has joined #puppet-openstack | 11:56 | |
| *** dgurtner has quit IRC | 12:02 | |
| *** dgurtner has joined #puppet-openstack | 12:03 | |
| *** dgurtner has quit IRC | 12:03 | |
| *** dgurtner has joined #puppet-openstack | 12:03 | |
| *** karimb has quit IRC | 12:05 | |
| *** ducttape_ has joined #puppet-openstack | 12:05 | |
| *** karimb has joined #puppet-openstack | 12:08 | |
| *** jayg|g0n3 is now known as jayg | 12:12 | |
| *** foxhunt72 is now known as foxhunt_away | 12:14 | |
| *** foxhunt_away is now known as foxhunt72 | 12:15 | |
| *** ayoung has quit IRC | 12:15 | |
| *** markvoelker has joined #puppet-openstack | 12:18 | |
| *** ducttape_ has quit IRC | 12:19 | |
| *** jpena|lunch is now known as jpena | 12:21 | |
| *** dmk0202 has quit IRC | 12:25 | |
| EmilienM | good morning | 12:28 |
| openstackgerrit | Merged openstack/puppet-tripleo: Make neutron server/ml2 profiles match t-h-t https://review.openstack.org/326119 | 12:35 |
| *** dmsimard is now known as dmsimard|afk | 12:37 | |
| *** karimb has quit IRC | 12:37 | |
| *** foxhunt72 is now known as foxhunt_away | 12:38 | |
| *** dmk0202 has joined #puppet-openstack | 12:39 | |
| *** foxhunt_away is now known as foxhunt72 | 12:44 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Import copy_logs.sh script from openstack-infra/project-config https://review.openstack.org/328014 | 12:45 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: workaround to run beaker with puppet4 https://review.openstack.org/327725 | 12:50 |
| EmilienM | degorenko: hey | 12:52 |
| *** qba73 has joined #puppet-openstack | 12:52 | |
| EmilienM | degorenko: can I have a second eye on https://review.openstack.org/#/c/327678 ? I'm struggling to see why second puppet run removes the file instead of first run | 12:52 |
| EmilienM | maybe ordering | 12:53 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial https://review.openstack.org/327678 | 12:54 |
| iurygregory | morning =D | 12:55 |
| *** chem has quit IRC | 12:57 | |
| *** dmk0202 has quit IRC | 12:58 | |
| iberezovskiy | hi | 12:58 |
| *** dmk0202 has joined #puppet-openstack | 12:59 | |
| *** chem has joined #puppet-openstack | 13:00 | |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker. https://review.openstack.org/309069 | 13:06 |
| *** jaosorior has quit IRC | 13:07 | |
| *** cwolferh has quit IRC | 13:10 | |
| *** richm has joined #puppet-openstack | 13:10 | |
| *** derekh has joined #puppet-openstack | 13:11 | |
| *** cwolferh has joined #puppet-openstack | 13:12 | |
| degorenko | EmilienM, hey | 13:15 |
| EmilienM | degorenko: yoyoyo | 13:15 |
| degorenko | i was afk | 13:15 |
| EmilienM | no worries | 13:15 |
| degorenko | looking at 327678 | 13:15 |
| EmilienM | if you have 5 min to look this patch | 13:15 |
| EmilienM | ok | 13:15 |
| EmilienM | degorenko: doing reviews now, anything I should look in prio? | 13:15 |
| degorenko | EmilienM, no, i dont have prio reviews :) | 13:15 |
| *** ayoung has joined #puppet-openstack | 13:16 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL https://review.openstack.org/328090 | 13:18 |
| EmilienM | iberezovskiy: can you look https://review.openstack.org/#/c/328028/ ? | 13:19 |
| *** amoralej is now known as amoralej|lunch | 13:20 | |
| *** jistr is now known as jistr|mtg | 13:20 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-octavia: Implement api.pp https://review.openstack.org/326142 | 13:20 |
| iberezovskiy | EmilienM, so you are actually check that mirror was changed? | 13:27 |
| iberezovskiy | if mirror change wasn't applied it's not ok | 13:27 |
| EmilienM | iberezovskiy: I check puppet installed the repo | 13:27 |
| EmilienM | before we had set +2 | 13:27 |
| EmilienM | so puppet could have failed, we missed it | 13:27 |
| *** zhongshengping has joined #puppet-openstack | 13:29 | |
| iberezovskiy | yep, just wanted to clarify. merged | 13:29 |
| *** dmk0202 has quit IRC | 13:29 | |
| EmilienM | thx | 13:33 |
| EmilienM | clayton: no -1 but still some question on https://review.openstack.org/327823 - please look when you have time | 13:33 |
| clayton | read the comment above in deps.pp :) | 13:34 |
| clayton | support-packages is used for things that don't require restarting services | 13:34 |
| clayton | so the client would be a support package | 13:34 |
| EmilienM | clayton: right | 13:34 |
| EmilienM | good | 13:34 |
| clayton | for neutron, keepalived and dnsmasq is a support package | 13:34 |
| EmilienM | clayton: I'm done for this review, +2 | 13:35 |
| clayton | awesome :) | 13:35 |
| clayton | the same pattern is used for neutron and nova and keystone | 13:35 |
| clayton | it wasn't needed for any modules before that | 13:35 |
| EmilienM | cool, excellent work | 13:35 |
| clayton | thanks :) | 13:35 |
| *** qba73 has quit IRC | 13:35 | |
| EmilienM | iberezovskiy: damn, I broke something | 13:36 |
| *** arnewiebalck has joined #puppet-openstack | 13:36 | |
| iberezovskiy | where? | 13:37 |
| EmilienM | nothing critical | 13:37 |
| EmilienM | I'm fixing it | 13:37 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: copy_logs: use journalctl only if present https://review.openstack.org/328303 | 13:39 |
| EmilienM | iberezovskiy: ^ | 13:39 |
| iberezovskiy | don't we want to have journalctl always available? | 13:39 |
| EmilienM | iberezovskiy: not on trusty ;) | 13:39 |
| EmilienM | it's a sysvinit system | 13:39 |
| iberezovskiy | oh yeah :) | 13:39 |
| *** zhongshengping has quit IRC | 13:42 | |
| *** zhongshengping has joined #puppet-openstack | 13:42 | |
| *** zhongshengping has quit IRC | 13:43 | |
| *** zhongshengping has joined #puppet-openstack | 13:44 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/puppet-tripleo: Add ironic profiles https://review.openstack.org/319297 | 13:45 |
| *** zhongshengping has quit IRC | 13:46 | |
| *** zhongshengping has joined #puppet-openstack | 13:47 | |
| *** zhongshengping has quit IRC | 13:51 | |
| *** dfisher has joined #puppet-openstack | 13:51 | |
| *** foxhunt72 is now known as foxhunt_away | 13:54 | |
| *** ifarkas has quit IRC | 13:56 | |
| EmilienM | degorenko: can you look https://review.openstack.org/#/c/328303/ please? | 13:57 |
| EmilienM | iberezovskiy: also, if you have time, I'm trying to address the ubuntu changes in https://review.openstack.org/#/c/327770/ but CI still fails, any idea why second puppet run is not idempotent? | 13:57 |
| iberezovskiy | i"ll take a look in ~10mins | 13:58 |
| *** zhongshengping has joined #puppet-openstack | 13:59 | |
| *** saneax is now known as saneax_AFK | 14:02 | |
| *** zhongshengping has quit IRC | 14:03 | |
| EmilienM | iberezovskiy, degorenko: also need review on https://review.openstack.org/#/c/328014/ to get logs back on mitaka jobs | 14:05 |
| *** jistr|mtg is now known as jistr | 14:09 | |
| *** cwolferh has quit IRC | 14:09 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: run_tests: fail if repos are not installed when asked https://review.openstack.org/328028 | 14:09 |
| *** ducttape_ has joined #puppet-openstack | 14:11 | |
| iurygregory | only me in the mid-cycle etherpad lol | 14:11 |
| *** cwolferh has joined #puppet-openstack | 14:12 | |
| *** xarses_ has quit IRC | 14:14 | |
| EmilienM | iurygregory: you can do the midcycle yourself? :) | 14:17 |
| *** ducttape_ has quit IRC | 14:17 | |
| *** foxhunt_away is now known as foxhunt72 | 14:19 | |
| *** ducttape_ has joined #puppet-openstack | 14:22 | |
| *** amoralej|lunch is now known as amoralej | 14:23 | |
| openstackgerrit | Dmitry Ilyin proposed openstack/puppet-ec2api: Add all other classes https://review.openstack.org/314598 | 14:26 |
| openstackgerrit | Dmitry Ilyin proposed openstack/puppet-ec2api: Add db related classes https://review.openstack.org/314597 | 14:26 |
| openstackgerrit | Dmitry Ilyin proposed openstack/puppet-ec2api: Add 3 api classes and params https://review.openstack.org/314596 | 14:26 |
| EmilienM | iberezovskiy: is Dmitry here? ^ | 14:27 |
| iberezovskiy | nope, I'll ping him | 14:27 |
| EmilienM | mhh ok | 14:27 |
| *** mkarpin has quit IRC | 14:29 | |
| *** mkarpin has joined #puppet-openstack | 14:31 | |
| *** ifarkas has joined #puppet-openstack | 14:32 | |
| openstackgerrit | Merged openstack/puppet-tripleo: Import ::nova class from THT https://review.openstack.org/325982 | 14:32 |
| iurygregory | EmilienM, yes i can =P | 14:33 |
| *** xarses_ has joined #puppet-openstack | 14:33 | |
| iberezovskiy | EmilienM, about https://review.openstack.org/#/c/327770/: I see there problem related to rabbimq-server: ERROR: epmd error for host ubuntu-xenial-rax-iad-1576843: timeout (timed out) | 14:33 |
| openstackgerrit | Dan Radez proposed openstack/puppet-congress: Updating code to new puppet module stucture https://review.openstack.org/328322 | 14:33 |
| *** zhongshengping has joined #puppet-openstack | 14:34 | |
| EmilienM | iberezovskiy: I'll recheck it once https://review.openstack.org/328303 merge | 14:38 |
| iberezovskiy | http://logs.openstack.org/70/327770/2/check/gate-puppet-openstack-integration-3-scenario002-tempest-ubuntu-xenial/499b8da/logs/rabbitmq/startup_log | 14:38 |
| iurygregory | EmilienM, since congress already have code , i think it's good use cookiecutter to generate a new structure for the module and then add the previous files. what you think? | 14:39 |
| EmilienM | iurygregory: yes | 14:39 |
| iurygregory | ok =D | 14:39 |
| iberezovskiy | probably smth wrong with hostname, let's wait for your patch | 14:39 |
| *** zhongshengping has quit IRC | 14:40 | |
| *** zhongshengping has joined #puppet-openstack | 14:40 | |
| EmilienM | iberezovskiy: have you thought about mu issue on xenial/newton thing with keystone? | 14:42 |
| *** zhongshengping has quit IRC | 14:42 | |
| openstackgerrit | Carlos Camacho proposed openstack/puppet-tripleo: Add mongodb profiles https://review.openstack.org/304780 | 14:43 |
| iberezovskiy | EmilienM, about the idempotency? | 14:43 |
| EmilienM | yes | 14:43 |
| iberezovskiy | I've statrted tests locally | 14:43 |
| *** jmikolajczak has quit IRC | 14:47 | |
| *** zhongshengping has joined #puppet-openstack | 14:51 | |
| *** saneax_AFK is now known as saneax | 14:52 | |
| *** zhongshengping has quit IRC | 14:54 | |
| *** dhollinger has joined #puppet-openstack | 14:57 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Implement Nova API profile https://review.openstack.org/325937 | 14:58 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial https://review.openstack.org/327678 | 15:01 |
| EmilienM | iberezovskiy: ^ see change, degorenko suggested something cool | 15:01 |
| iberezovskiy | subscribe, cool | 15:04 |
| *** tesseract has quit IRC | 15:09 | |
| *** colonwq has quit IRC | 15:12 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Import copy_logs.sh script from openstack-infra/project-config https://review.openstack.org/328014 | 15:17 |
| *** rcernin has quit IRC | 15:19 | |
| *** zhongshengping has joined #puppet-openstack | 15:22 | |
| *** zhongshengping has quit IRC | 15:24 | |
| *** foxhunt72 is now known as foxhunt_away | 15:24 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: profile/base/nova: declare nova class properly https://review.openstack.org/328347 | 15:29 |
| degorenko | EmilienM, typo here? https://review.openstack.org/327725 | 15:29 |
| EmilienM | degorenko: you're nitpicking me? :P | 15:30 |
| EmilienM | it's Frriday! | 15:30 |
| degorenko | EmilienM, nope :D | 15:31 |
| degorenko | but can - can't | 15:31 |
| degorenko | :) | 15:31 |
| openstackgerrit | Merged openstack/puppet-openstack-integration: copy_logs: use journalctl only if present https://review.openstack.org/328303 | 15:37 |
| iurygregory | degorenko, just give a +1and not +2 =P | 15:37 |
| *** chem has quit IRC | 15:38 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-octavia: Implement api.pp https://review.openstack.org/326142 | 15:39 |
| *** shaikapsar has joined #puppet-openstack | 15:43 | |
| shaikapsar | I'm having an issue with https://github.com/openstack/puppet-magnum/blob/master/manifests/keystone/domain.pp | 15:45 |
| shaikapsar | I can;t use domain.pp with out openrc file | 15:46 |
| degorenko | shaikapsar, probably you faced with this bug: https://github.com/openstack/puppet-magnum/blob/master/manifests/keystone/domain.pp#L69 | 15:51 |
| iberezovskiy | EmilienM, still not idempotent | 15:53 |
| iberezovskiy | and I don't see the update for package resource | 15:53 |
| shaikapsar | No, domain.pp is trying to create keystone_user, keystone_domain and keystone_user_role | 15:54 |
| EmilienM | iberezovskiy: weird | 15:54 |
| shaikapsar | but those types need openrc file access the keystone service | 15:54 |
| shaikapsar | if i place openrc file then those resource getting created | 15:55 |
| EmilienM | degorenko, iberezovskiy: can you guys took over this one? I'm already trying to migrate beaker jobs on xenial | 15:56 |
| shaikapsar | if i run domain pp in Fuel controller node I see an other issue , /etc/magnum/manum.conf doesn;t exist | 15:56 |
| iberezovskiy | hm, but locally it work | 15:56 |
| shaikapsar | If i run magnum on controller node it will work | 15:56 |
| shaikapsar | Can we remove https://github.com/openstack/puppet-magnum/blob/master/manifests/keystone/domain.pp#L72-L84 from domain pp and magnum_config with domain details in an other manifest file? | 15:58 |
| iberezovskiy | let me retry | 15:59 |
| *** evidex has quit IRC | 15:59 | |
| degorenko | shaikapsar, fuel doesn't support magnum | 15:59 |
| degorenko | shaikapsar, you should ask vanditboy | 16:00 |
| degorenko | but he is not here now | 16:00 |
| *** shaikapsar has quit IRC | 16:03 | |
| *** shaikapsar has joined #puppet-openstack | 16:06 | |
| shaikapsar | I'm working on fuel plugin for magnum and i got the issue | 16:07 |
| *** ifarkas has quit IRC | 16:10 | |
| degorenko | then possibly you need manually put all needed configs in config file | 16:12 |
| degorenko | but as i know fuel plugin starts installing of service when keystone already finished | 16:12 |
| degorenko | just use openrc | 16:12 |
| shaikapsar | I already implemented openrc config, but i want to check with you that https://github.com/openstack/puppet-magnum/blob/master/manifests/keystone/domain.pp#L72-L84 line are valid in domain.pp | 16:16 |
| openstackgerrit | Carlos Camacho proposed openstack/puppet-tripleo: Add mongodb profiles https://review.openstack.org/304780 | 16:19 |
| *** queretaro has quit IRC | 16:20 | |
| *** ducttape_ has quit IRC | 16:29 | |
| *** ducttape_ has joined #puppet-openstack | 16:32 | |
| *** ayoung has quit IRC | 16:33 | |
| iberezovskiy | EmilienM, can I try to rework this subscribe a bit? | 16:34 |
| EmilienM | iberezovskiy: please take over it | 16:34 |
| EmilienM | I'm already busy on other tasks for today | 16:34 |
| openstackgerrit | Dmitry Ilyin proposed openstack/puppet-ec2api: Add all other classes https://review.openstack.org/314598 | 16:35 |
| openstackgerrit | Dmitry Ilyin proposed openstack/puppet-ec2api: Add db related classes https://review.openstack.org/314597 | 16:35 |
| openstackgerrit | Dmitry Ilyin proposed openstack/puppet-ec2api: Add 3 api classes and params https://review.openstack.org/314596 | 16:35 |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial https://review.openstack.org/327678 | 16:36 |
| iberezovskiy | @bbs 30min, I think test will show smth | 16:38 |
| *** iberezovskiy is now known as iberezovskiy|afk | 16:38 | |
| *** chem has joined #puppet-openstack | 16:42 | |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker. https://review.openstack.org/309069 | 16:48 |
| *** colonwq has joined #puppet-openstack | 16:52 | |
| *** dtantsur is now known as dtantsur|afk | 16:52 | |
| openstackgerrit | Merged openstack/puppet-murano: Use regexp by whole name for murano_application https://review.openstack.org/326021 | 16:56 |
| *** shaikapsar has quit IRC | 16:57 | |
| *** iberezovskiy|afk is now known as iberezovskiy | 17:02 | |
| *** aimon_ has quit IRC | 17:10 | |
| *** aimon has joined #puppet-openstack | 17:11 | |
| *** derekh has quit IRC | 17:14 | |
| *** ayoung has joined #puppet-openstack | 17:14 | |
| iberezovskiy | EmilienM, I see only one solution here, remove target file. using define won't be idempotent | 17:15 |
| EmilienM | iberezovskiy: how? | 17:15 |
| iberezovskiy | I wann file { ensure => absent } with the hardcoded path | 17:16 |
| iberezovskiy | I wanna try* | 17:16 |
| iberezovskiy | but it's horrible | 17:16 |
| EmilienM | iberezovskiy: I did it before | 17:17 |
| EmilienM | I didn't work fine | 17:17 |
| iberezovskiy | EmilienM, did you try with subscribe? | 17:17 |
| EmilienM | iberezovskiy: no | 17:18 |
| iberezovskiy | let's do it :) | 17:18 |
| iberezovskiy | I'll upload | 17:19 |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial https://review.openstack.org/327678 | 17:20 |
| *** cwolferh has quit IRC | 17:21 | |
| EmilienM | iberezovskiy: I have some patches | 17:22 |
| EmilienM | https://review.openstack.org/#/c/327894/ | 17:22 |
| EmilienM | and https://review.openstack.org/#/c/320554/ | 17:22 |
| *** dgurtner has quit IRC | 17:22 | |
| EmilienM | I'll need to add the nodesets on all modules | 17:23 |
| EmilienM | and then we can switch our beaker jobs to xenial, no more blockers | 17:23 |
| iberezovskiy | I can add nodesets files but only later (after 2-3 hours) | 17:24 |
| * iberezovskiy reviewing patches | 17:24 | |
| EmilienM | iberezovskiy: I'm on it already | 17:25 |
| iberezovskiy | EmilienM, nodesets for keystone could be merged, right? | 17:26 |
| iberezovskiy | I'm wondering because gate-puppet-openstack-integration-4-scenario001-tempest-ubuntu-trusty is red | 17:27 |
| openstackgerrit | Dan Radez proposed openstack/puppet-congress: Updating code to new puppet module stucture https://review.openstack.org/328322 | 17:29 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Add nodepool-xenial.yml file https://review.openstack.org/328401 | 17:30 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-barbican: Add nodepool-xenial.yml file https://review.openstack.org/328402 | 17:30 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: Add nodepool-xenial.yml file https://review.openstack.org/328403 | 17:30 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-cinder: Add nodepool-xenial.yml file https://review.openstack.org/328404 | 17:30 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-designate: Add nodepool-xenial.yml file https://review.openstack.org/328405 | 17:30 |
| openstackgerrit | Martin Mágr proposed openstack/puppet-tripleo: New composable service: Sensu https://review.openstack.org/323303 | 17:30 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ec2api: Add nodepool-xenial.yml file https://review.openstack.org/328406 | 17:30 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: Add nodepool-xenial.yml file https://review.openstack.org/328407 | 17:30 |
| EmilienM | iberezovskiy: it's a random error: http://logs.openstack.org/54/320554/17/experimental/gate-puppet-openstack-integration-4-scenario001-tempest-ubuntu-trusty/9eb119e/console.html#_2016-06-10_16_23_49_902 | 17:31 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-gnocchi: Add nodepool-xenial.yml file https://review.openstack.org/328408 | 17:31 |
| EmilienM | I think we can safely merge them | 17:31 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-heat: Add nodepool-xenial.yml file https://review.openstack.org/328409 | 17:31 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-horizon: Add nodepool-xenial.yml file https://review.openstack.org/328410 | 17:31 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ironic: Add nodepool-xenial.yml file https://review.openstack.org/328411 | 17:31 |
| * iurygregory runs spam time | 17:31 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-magnum: Add nodepool-xenial.yml file https://review.openstack.org/328412 | 17:31 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-manila: Add nodepool-xenial.yml file https://review.openstack.org/328413 | 17:32 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-mistral: Add nodepool-xenial.yml file https://review.openstack.org/328414 | 17:32 |
| *** degorenko is now known as _degorenko|afk | 17:32 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-murano: Add nodepool-xenial.yml file https://review.openstack.org/328415 | 17:32 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: Add nodepool-xenial.yml file https://review.openstack.org/328416 | 17:32 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: Add nodepool-xenial.yml file https://review.openstack.org/328417 | 17:32 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-octavia: Add nodepool-xenial.yml file https://review.openstack.org/328418 | 17:33 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_extras: Add nodepool-xenial.yml file https://review.openstack.org/328419 | 17:33 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstacklib: Add nodepool-xenial.yml file https://review.openstack.org/328420 | 17:33 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-oslo: Add nodepool-xenial.yml file https://review.openstack.org/328421 | 17:33 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ovn: Add nodepool-xenial.yml file https://review.openstack.org/328422 | 17:33 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-sahara: Add nodepool-xenial.yml file https://review.openstack.org/328423 | 17:33 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-swift: Add nodepool-xenial.yml file https://review.openstack.org/328424 | 17:33 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tempest: Add nodepool-xenial.yml file https://review.openstack.org/328425 | 17:33 |
| *** jpena is now known as jpena|off | 17:34 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-trove: Add nodepool-xenial.yml file https://review.openstack.org/328426 | 17:34 |
| *** dprince has quit IRC | 17:34 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-vswitch: Add nodepool-xenial.yml file https://review.openstack.org/328427 | 17:34 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-watcher: Add nodepool-xenial.yml file https://review.openstack.org/328428 | 17:34 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-zaqar: Add nodepool-xenial.yml file https://review.openstack.org/328429 | 17:34 |
| *** dprince has joined #puppet-openstack | 17:35 | |
| iberezovskiy | awesome :D | 17:36 |
| *** mentat has quit IRC | 17:36 | |
| *** cwolferh has joined #puppet-openstack | 17:37 | |
| *** blkperl has quit IRC | 17:37 | |
| iberezovskiy | probably tacker is left only | 17:37 |
| *** xarses_ is now known as xarses | 17:37 | |
| *** Hunner has quit IRC | 17:37 | |
| *** bmjen has quit IRC | 17:38 | |
| iberezovskiy | EmilienM, idempotency doesn't like us today, still the same | 17:38 |
| EmilienM | damn | 17:38 |
| EmilienM | iberezovskiy: ack for tacker | 17:39 |
| EmilienM | well, tacker needs spec/acceptance thing | 17:39 |
| iurygregory | same for congress i think | 17:41 |
| iberezovskiy | and what about puppet-ceph? | 17:41 |
| iurygregory | iberezovskiy, https://github.com/openstack/puppet-ceph/tree/master/spec/acceptance/nodesets | 17:41 |
| iurygregory | already have | 17:41 |
| iberezovskiy | great | 17:41 |
| iurygregory | module sync too =) | 17:42 |
| iberezovskiy | then monasca and rally | 17:42 |
| iurygregory | lol monasca =P | 17:42 |
| iurygregory | rally makes sense =) | 17:43 |
| iberezovskiy | yeah yeah, my bad | 17:43 |
| iberezovskiy | just walked through github | 17:43 |
| iurygregory | congress/tacker will need to use cookiecutter so we don't need =D | 17:43 |
| iurygregory | because they should use modulesync and cookiecutter :D | 17:44 |
| iberezovskiy | EmilienM, you know, I see interesting thing. This file removal doesn't apply at first puppet run, only for second | 17:47 |
| iurygregory | lol | 17:48 |
| EmilienM | iberezovskiy: yeah, that's what I figured too | 17:50 |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial https://review.openstack.org/327678 | 17:51 |
| *** shaikapsar has joined #puppet-openstack | 17:53 | |
| *** permalac has quit IRC | 17:54 | |
| *** shardy has quit IRC | 17:56 | |
| iberezovskiy | bbs ~1 hour | 18:00 |
| *** iberezovskiy is now known as iberezovskiy|afk | 18:00 | |
| *** shaikapsar has quit IRC | 18:01 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: workaround to run beaker with puppet4 https://review.openstack.org/327725 | 18:09 |
| openstackgerrit | Dan Prince proposed openstack/puppet-tripleo: Pacemaker: only create DB on pacemaker master https://review.openstack.org/328446 | 18:16 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: profile/base/nova: declare nova class properly https://review.openstack.org/328347 | 18:19 |
| *** paramite has quit IRC | 18:25 | |
| *** amoralej is now known as amoralej|off | 18:28 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: remove mariadb prior to run Puppet https://review.openstack.org/328449 | 18:34 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Import copy_logs.sh script from openstack-infra/project-config https://review.openstack.org/328015 | 18:36 |
| *** dgurtner has joined #puppet-openstack | 18:48 | |
| *** dgurtner has quit IRC | 18:48 | |
| *** dgurtner has joined #puppet-openstack | 18:48 | |
| *** yolanda has quit IRC | 18:53 | |
| dprince | EmilienM: did you want me to update this? https://review.openstack.org/#/c/319464/2 | 19:01 |
| EmilienM | dprince: yes please so we're consistent with other modules | 19:01 |
| EmilienM | oh wait | 19:02 |
| EmilienM | rebase it | 19:02 |
| EmilienM | it was adressed it | 19:02 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: Switch $service_name to 'glance' for keystone https://review.openstack.org/319464 | 19:02 |
| EmilienM | dprince: good now ^ | 19:02 |
| EmilienM | dprince: +A | 19:02 |
| dprince | EmilienM: cool with me :) | 19:03 |
| *** johnmilton has quit IRC | 19:03 | |
| *** yolanda has joined #puppet-openstack | 19:03 | |
| *** shaikapsar has joined #puppet-openstack | 19:16 | |
| openstackgerrit | Bradley Klein proposed openstack/puppet-monasca: Schema change for deterministic alarms defs https://review.openstack.org/328460 | 19:16 |
| shaikapsar | Please review https://review.openstack.org/#/c/321154/ | 19:17 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: bump RDO to latest OpenStack commit https://review.openstack.org/328463 | 19:25 |
| *** ntaylor1781 has quit IRC | 19:28 | |
| openstackgerrit | Merged openstack/puppet-watcher: Add nodepool-xenial.yml file https://review.openstack.org/328428 | 19:29 |
| openstackgerrit | Merged openstack/puppet-barbican: Add nodepool-xenial.yml file https://review.openstack.org/328402 | 19:30 |
| openstackgerrit | Merged openstack/puppet-designate: Add nodepool-xenial.yml file https://review.openstack.org/328405 | 19:30 |
| openstackgerrit | Merged openstack/puppet-ovn: Add nodepool-xenial.yml file https://review.openstack.org/328422 | 19:31 |
| openstackgerrit | Merged openstack/puppet-murano: Add nodepool-xenial.yml file https://review.openstack.org/328415 | 19:32 |
| openstackgerrit | Merged openstack/puppet-magnum: Add nodepool-xenial.yml file https://review.openstack.org/328412 | 19:33 |
| openstackgerrit | Merged openstack/puppet-octavia: Add nodepool-xenial.yml file https://review.openstack.org/328418 | 19:34 |
| openstackgerrit | Merged openstack/puppet-manila: Add nodepool-xenial.yml file https://review.openstack.org/328413 | 19:34 |
| *** iberezovskiy|afk is now known as iberezovskiy | 19:50 | |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial https://review.openstack.org/327678 | 19:50 |
| openstackgerrit | Merged openstack/puppet-ironic: Change default service_name to 'ironic' https://review.openstack.org/327117 | 19:59 |
| openstackgerrit | Merged openstack/puppet-ceilometer: Add nodepool-xenial.yml file https://review.openstack.org/328403 | 19:59 |
| openstackgerrit | Merged openstack/puppet-swift: Add nodepool-xenial.yml file https://review.openstack.org/328424 | 19:59 |
| openstackgerrit | Merged openstack/puppet-zaqar: Add nodepool-xenial.yml file https://review.openstack.org/328429 | 20:09 |
| openstackgerrit | Merged openstack/puppet-cinder: Add nodepool-xenial.yml file https://review.openstack.org/328404 | 20:17 |
| *** dgurtner has quit IRC | 20:23 | |
| *** johnmilton has joined #puppet-openstack | 20:29 | |
| *** dprince has quit IRC | 20:30 | |
| openstackgerrit | Merged openstack/puppet-gnocchi: Add nodepool-xenial.yml file https://review.openstack.org/328408 | 20:30 |
| openstackgerrit | Merged openstack/puppet-horizon: Add nodepool-xenial.yml file https://review.openstack.org/328410 | 20:30 |
| *** dprince has joined #puppet-openstack | 20:31 | |
| openstackgerrit | Merged openstack/puppet-oslo: Add nodepool-xenial.yml file https://review.openstack.org/328421 | 20:34 |
| openstackgerrit | Merged openstack/puppet-aodh: Add nodepool-xenial.yml file https://review.openstack.org/328401 | 20:35 |
| *** ducttape_ has quit IRC | 20:36 | |
| *** johnmilton has quit IRC | 20:37 | |
| openstackgerrit | Merged openstack/puppet-openstacklib: Add nodepool-xenial.yml file https://review.openstack.org/328420 | 20:38 |
| openstackgerrit | Merged openstack/puppet-keystone: beaker: remove multi domain config resources output test https://review.openstack.org/327894 | 20:38 |
| openstackgerrit | Shaik Apsar proposed openstack/puppet-trove: Option to specify the trove guestagent log file. https://review.openstack.org/321154 | 20:40 |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial https://review.openstack.org/327678 | 20:41 |
| iurygregory | bye people o/ have a great weekend | 20:42 |
| openstackgerrit | Merged openstack/puppet-murano: Provide default service_name for keystone endpoint https://review.openstack.org/324749 | 20:42 |
| *** iurygregory has quit IRC | 20:43 | |
| EmilienM | iurygregory: o/ | 20:43 |
| *** jayg is now known as jayg|g0n3 | 20:44 | |
| *** iberezovskiy is now known as iberezovskiy|off | 20:44 | |
| *** ayoung has quit IRC | 20:49 | |
| *** johnmilton has joined #puppet-openstack | 20:49 | |
| openstackgerrit | Merged openstack/puppet-sahara: Add nodepool-xenial.yml file https://review.openstack.org/328423 | 20:50 |
| openstackgerrit | Merged openstack/puppet-trove: Add nodepool-xenial.yml file https://review.openstack.org/328426 | 20:50 |
| *** dprince has quit IRC | 20:51 | |
| EmilienM | beagles: in octavia, do keystone options are required more than api service? ie: worker, health manager, etc | 20:51 |
| *** ducttape_ has joined #puppet-openstack | 20:52 | |
| openstackgerrit | Merged openstack/puppet-tempest: Add nodepool-xenial.yml file https://review.openstack.org/328425 | 20:53 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-octavia: Implement worker.pp https://review.openstack.org/328495 | 20:54 |
| openstackgerrit | Merged openstack/puppet-ironic: Add nodepool-xenial.yml file https://review.openstack.org/328411 | 20:56 |
| EmilienM | mwhahaha: omg I'm glad we have nova::flavor provider | 20:56 |
| EmilienM | mwhahaha: octavia requires to create a flavor with a predictable id | 20:56 |
| EmilienM | so we can reuse the provider in puppet octavia to manage the flavor | 20:56 |
| mwhahaha | well that's good | 20:57 |
| EmilienM | I want to buy a beer to denis now | 20:57 |
| EmilienM | was it denis? | 20:57 |
| mwhahaha | no | 20:57 |
| EmilienM | ah no | 20:57 |
| EmilienM | Trevor McCasland | 20:57 |
| EmilienM | oops sorry | 20:57 |
| EmilienM | I'll buy 2 beers! | 20:58 |
| EmilienM | mwhahaha: wdyt | 21:03 |
| EmilienM | should we manage the flavor by default or not? | 21:03 |
| mwhahaha | for octavia? | 21:03 |
| mwhahaha | yea probably | 21:03 |
| mwhahaha | with a $configure_flavor option to disable | 21:04 |
| EmilienM | yeah | 21:04 |
| openstackgerrit | Merged openstack/puppet-openstack_extras: Add nodepool-xenial.yml file https://review.openstack.org/328419 | 21:08 |
| openstackgerrit | Merged openstack/puppet-glance: Add nodepool-xenial.yml file https://review.openstack.org/328407 | 21:08 |
| *** shaikapsar has quit IRC | 21:09 | |
| mwhahaha | EmilienM: looks like mistral packaging is broken, http://logs.openstack.org/14/328414/1/check/gate-puppet-mistral-puppet-beaker-rspec-centos-7/ab5ac6d/console.html#_2016-06-10_20_44_19_280 | 21:15 |
| mwhahaha | not sure why mistral is requiring magnumclient | 21:16 |
| EmilienM | mwhahaha: packaging is fixed | 21:20 |
| EmilienM | recheck | 21:20 |
| EmilienM | we fixed it in rdo today | 21:21 |
| EmilienM | and I just promoted our CI | 21:21 |
| EmilienM | mwhahaha: still in gate: https://review.openstack.org/#/c/328463/ | 21:21 |
| mwhahaha | k | 21:21 |
| EmilienM | mwhahaha: in puppet, can I use an hash to deploy a provider? | 21:22 |
| EmilienM | like $hash = { 'id' => '1' } and nova_flavor { id => $hash[id] } ? | 21:23 |
| mwhahaha | wat | 21:23 |
| mwhahaha | yea | 21:23 |
| EmilienM | maybe in puppet parser or puppet4 only :/ | 21:23 |
| EmilienM | how? | 21:23 |
| mwhahaha | with create resource | 21:23 |
| mwhahaha | it takes a hash i think | 21:23 |
| mwhahaha | so you could just create_resource('nova_flavor', $hash) | 21:24 |
| EmilienM | ah right | 21:24 |
| EmilienM | nice!! | 21:24 |
| mwhahaha | brb | 21:24 |
| *** dgurtner has joined #puppet-openstack | 21:27 | |
| *** dfisher has quit IRC | 21:29 | |
| openstackgerrit | Merged openstack/puppet-heat: Provide default service_name for keystone endpoint https://review.openstack.org/324764 | 21:29 |
| *** dhollinger has quit IRC | 21:34 | |
| *** dgurtner has quit IRC | 21:39 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-octavia: Implement worker.pp https://review.openstack.org/328495 | 21:40 |
| EmilienM | mwhahaha: I need to write some missing tests, but you can already tell me what you think ^ | 21:40 |
| mwhahaha | sure | 21:40 |
| *** ducttape_ has quit IRC | 21:41 | |
| EmilienM | well amp_flavor_id to os_service_default sucks | 21:41 |
| EmilienM | because it will make catalog fails by default... | 21:41 |
| EmilienM | maybe I can pick a random value, like 65 | 21:41 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-octavia: Implement worker.pp https://review.openstack.org/328495 | 21:43 |
| EmilienM | mwhahaha: need you to review the hash thing, I'll finish the rest later | 21:44 |
| mwhahaha | yea that'll work, minor note on the name string | 21:45 |
| EmilienM | ok thx | 21:46 |
| EmilienM | I don't see the note | 21:46 |
| mwhahaha | just posted | 21:46 |
| EmilienM | ah thanks | 21:46 |
| EmilienM | mwhahaha: does my reply makes sense? | 21:49 |
| *** dhollinger has joined #puppet-openstack | 21:49 | |
| EmilienM | except it's Nova_flavor | 21:49 |
| EmilienM | not Nova::Flavor | 21:50 |
| mwhahaha | yea but you get the idea :D | 21:50 |
| EmilienM | yeah | 21:50 |
| EmilienM | cool! | 21:50 |
| mwhahaha | but you may want to tag to restrict it | 21:50 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-octavia: Implement worker.pp https://review.openstack.org/328495 | 21:50 |
| mwhahaha | but yea | 21:50 |
| *** Hunner has joined #puppet-openstack | 21:51 | |
| *** Hunner has joined #puppet-openstack | 21:51 | |
| openstackgerrit | Merged openstack/puppet-keystone: ubuntu: do not attempt to start apache during keystone setup https://review.openstack.org/327770 | 21:51 |
| openstackgerrit | Merged openstack/puppet-openstack-integration: bump RDO to latest OpenStack commit https://review.openstack.org/328463 | 21:51 |
| *** mentat has joined #puppet-openstack | 21:52 | |
| openstackgerrit | Merged openstack/puppet-keystone: Fix facts for policy test https://review.openstack.org/326642 | 22:02 |
| openstackgerrit | Merged openstack/puppet-vswitch: Add nodepool-xenial.yml file https://review.openstack.org/328427 | 22:05 |
| EmilienM | mwhahaha: wait, can I test a provider in classes rspec? | 22:06 |
| EmilienM | not sure is_expected.to contain_nova__flavor works | 22:06 |
| mwhahaha | yea | 22:06 |
| mwhahaha | it should | 22:06 |
| EmilienM | mhh ok | 22:06 |
| mwhahaha | contain_nova_flavor | 22:06 |
| mwhahaha | contain_nova__flavor would be for a resource nova::flavor | 22:06 |
| mwhahaha | but contain_nova_flavor is for the resource nova_flavor :D | 22:06 |
| EmilienM | ah right | 22:07 |
| EmilienM | damn | 22:07 |
| EmilienM | I'm tired | 22:07 |
| EmilienM | what day is it again? :) | 22:07 |
| mwhahaha | friday i hope :D | 22:07 |
| EmilienM | mwhahaha: thx for your last review, really awesome | 22:09 |
| mwhahaha | sure | 22:09 |
| *** dhollinger has quit IRC | 22:11 | |
| openstackgerrit | Merged openstack/puppet-keystone: Add nodepool-xenial.yml file https://review.openstack.org/320554 | 22:12 |
| openstackgerrit | Merged openstack/puppet-heat: Add nodepool-xenial.yml file https://review.openstack.org/328409 | 22:47 |
| openstackgerrit | Merged openstack/puppet-ceph: Fix keystone part for non-pki tokens https://review.openstack.org/322810 | 22:47 |
| EmilienM | mwhahaha: its weird | 22:48 |
| EmilienM | tests pass for all distro except ubuntu: | 22:48 |
| EmilienM | Evaluation Error: Error while evaluating a Function Call, Could not autoload puppet/type/nova_flavor: Could not autoload puppet/provider/nova_flavor/openstack: cannot load such file -- puppet/provider/nova at /home/emilien/git/openstack/puppet/puppet-octavia/spec/fixtures/modules/octavia/manifests/worker.pp:63:7 on node xps | 22:48 |
| EmilienM | (unit tests) | 22:48 |
| EmilienM | on ubuntu-14.04-x86_64 | 22:49 |
| mwhahaha | Do we need the require ../../.. hack thing | 22:49 |
| EmilienM | ah | 22:49 |
| EmilienM | but why only ubuntu? | 22:50 |
| EmilienM | it's the first loaded? | 22:50 |
| EmilienM | in the test, yes | 22:50 |
| mwhahaha | I hate that error cause I've never really gotten to the bottom of it :D | 22:51 |
| EmilienM | maybe we need require File.join(File.dirname(__FILE__), '..','..','..', 'puppet/provider/nova') | 22:51 |
| mwhahaha | Yea that | 22:51 |
| EmilienM | mwhahaha: I don't even know what to write in commit message. Do you see the picture dog? it's me now. | 22:54 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: providers: add hack to require nova providers outside puppet-nova https://review.openstack.org/328525 | 22:54 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-octavia: Implement worker.pp https://review.openstack.org/328495 | 22:54 |
| mwhahaha | Fixing provider load issues when nova flavor is used outside of nova or something | 22:55 |
| mwhahaha | :) | 22:55 |
| openstackgerrit | Merged openstack/puppet-neutron: Add nodepool-xenial.yml file https://review.openstack.org/328416 | 22:57 |
| EmilienM | mwhahaha: have a nice week-end I'm off a bit now, beer o clock | 22:58 |
| mwhahaha | later | 22:58 |
| *** aimon has quit IRC | 23:16 | |
| *** aimon has joined #puppet-openstack | 23:23 | |
| *** ducttape_ has joined #puppet-openstack | 23:24 | |
| *** ducttape_ has quit IRC | 23:25 | |
| *** ducttape_ has joined #puppet-openstack | 23:25 | |
| *** xarses has quit IRC | 23:48 | |
| *** rwsu has quit IRC | 23:54 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!