richm | mdorman: The puppet-keystone providers should pull the creds from /etc/keystone/keystone.conf | 00:00 |
---|---|---|
*** gfidente has quit IRC | 00:05 | |
*** gildub has joined #puppet-openstack | 00:17 | |
gildub | EmilienM, hi | 00:18 |
EmilienM | gildub: pong | 00:20 |
gildub | EmilienM, never mind, I got confused because you archived a Trello card (trusts) but that's in MGMT and not in On going effort. | 00:21 |
EmilienM | so I improved https://wiki.openstack.org/wiki/Puppet | 00:37 |
EmilienM | I did not deleted any text, only moved | 00:37 |
EmilienM | indexing won't be an issue, since the landing page did not change | 00:37 |
EmilienM | any feedback is welcome ^ | 00:37 |
*** zhangjn has joined #puppet-openstack | 01:00 | |
*** tiswanso has joined #puppet-openstack | 01:06 | |
*** tiswanso_ has joined #puppet-openstack | 01:12 | |
*** tiswanso has quit IRC | 01:15 | |
*** ducttape_ has joined #puppet-openstack | 02:02 | |
*** angdraug has quit IRC | 02:03 | |
*** xingchao has joined #puppet-openstack | 02:08 | |
*** sergmelikyan has joined #puppet-openstack | 02:25 | |
*** sergmelikyan has quit IRC | 02:30 | |
*** aimon has joined #puppet-openstack | 02:41 | |
*** aimon_ has quit IRC | 02:45 | |
*** zhangjn has quit IRC | 03:04 | |
*** zhangjn has joined #puppet-openstack | 03:08 | |
*** ducttape_ has quit IRC | 03:13 | |
*** aimon has quit IRC | 03:13 | |
*** aimon has joined #puppet-openstack | 03:14 | |
*** markvoelker has quit IRC | 03:15 | |
*** aimon has quit IRC | 03:16 | |
*** ducttape_ has joined #puppet-openstack | 03:16 | |
*** aimon has joined #puppet-openstack | 03:16 | |
*** ducttape_ has quit IRC | 03:22 | |
*** mihgen has quit IRC | 03:27 | |
*** mgagne has quit IRC | 03:28 | |
*** tiswanso_ has quit IRC | 03:28 | |
*** ducttape_ has joined #puppet-openstack | 03:31 | |
*** mgagne has joined #puppet-openstack | 03:31 | |
*** mihgen has joined #puppet-openstack | 03:33 | |
*** ducttape_ has quit IRC | 03:35 | |
*** mihgen has quit IRC | 03:40 | |
*** mgagne has quit IRC | 03:41 | |
*** xingchao has quit IRC | 03:43 | |
*** benh57 has quit IRC | 03:45 | |
*** v1k0d3n has quit IRC | 03:48 | |
*** v1k0d3n has joined #puppet-openstack | 03:56 | |
*** v1k0d3n has quit IRC | 04:02 | |
*** mgagne has joined #puppet-openstack | 04:03 | |
*** mihgen has joined #puppet-openstack | 04:03 | |
*** michchap_ has quit IRC | 04:09 | |
*** michchap has joined #puppet-openstack | 04:09 | |
*** markvoelker has joined #puppet-openstack | 04:15 | |
*** markvoelker has quit IRC | 04:21 | |
*** cdelatte has quit IRC | 04:45 | |
*** rcarrillocruz has left #puppet-openstack | 04:46 | |
*** cdelatte has joined #puppet-openstack | 04:46 | |
*** xingchao has joined #puppet-openstack | 05:08 | |
*** richm has quit IRC | 05:37 | |
*** markvoelker has joined #puppet-openstack | 06:18 | |
*** markvoelker has quit IRC | 06:22 | |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_trust for Keystone V3 API https://review.openstack.org/200996 | 06:26 |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_trust for Keystone V3 API https://review.openstack.org/200996 | 06:28 |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_trust for Keystone V3 API https://review.openstack.org/200996 | 06:28 |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_trust for Keystone V3 API https://review.openstack.org/200996 | 06:29 |
gildub | xingchao, Hi - Would you mind reviewing back port https://review.openstack.org/222965 , it's part of a trilogy of domain bugs issues | 06:35 |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Endpoints can be APIs version-less https://review.openstack.org/213957 | 06:49 |
*** sergmelikyan has joined #puppet-openstack | 06:52 | |
*** sergmelikyan has quit IRC | 06:56 | |
gildub | michchap, hi - When you get a chance, please review this second one (out of 3) - back-ports for the domain bugs issues: https://review.openstack.org/222965 | 06:58 |
michchap | gildub: ack | 06:58 |
*** smakar has joined #puppet-openstack | 07:11 | |
*** jtomasek has joined #puppet-openstack | 07:14 | |
*** social has quit IRC | 07:23 | |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_trust for Keystone V3 API https://review.openstack.org/200996 | 07:24 |
*** social has joined #puppet-openstack | 07:30 | |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_trust for Keystone V3 API https://review.openstack.org/200996 | 07:31 |
*** itsuugo has joined #puppet-openstack | 07:32 | |
*** itsuugo has quit IRC | 07:33 | |
*** itsuugo has joined #puppet-openstack | 07:34 | |
*** paramite has joined #puppet-openstack | 07:34 | |
openstackgerrit | Ricardo Carrillo Cruz proposed openstack/puppet-ironic: Allow passing the whole contents of bifrost_global_vars https://review.openstack.org/222914 | 07:35 |
*** social has quit IRC | 07:35 | |
*** jpena has joined #puppet-openstack | 07:37 | |
openstackgerrit | Merged openstack/puppet-keystone: able to use wrong value for default_domain_id https://review.openstack.org/222965 | 07:46 |
*** gildub has quit IRC | 07:49 | |
*** paramite is now known as paramite|afk | 07:50 | |
*** paramite|afk is now known as paramite | 07:57 | |
openstackgerrit | Ricardo Carrillo Cruz proposed openstack/puppet-ironic: Allow passing the whole contents of bifrost_global_vars https://review.openstack.org/222914 | 08:00 |
*** gfidente has joined #puppet-openstack | 08:00 | |
openstackgerrit | Merged openstack/puppet-openstack-integration: Add LICENSE file https://review.openstack.org/201296 | 08:03 |
*** karume has joined #puppet-openstack | 08:06 | |
openstackgerrit | Merged openstack/puppet-tripleo: Allow a user to specify the syslog address for HAProxy https://review.openstack.org/222595 | 08:08 |
openstackgerrit | Merged openstack/puppet-heat: Remove deprecated parameter stack_user_domain https://review.openstack.org/221857 | 08:08 |
*** itsuugo has quit IRC | 08:11 | |
*** zhangjn has quit IRC | 08:12 | |
*** derekh has joined #puppet-openstack | 08:13 | |
*** zhangjn has joined #puppet-openstack | 08:14 | |
*** obbs has joined #puppet-openstack | 08:16 | |
openstackgerrit | Merged openstack/puppet-heat: Create Heat Domain with Keystone_domain resource https://review.openstack.org/220105 | 08:16 |
*** rochaporto has joined #puppet-openstack | 08:16 | |
*** jistr has joined #puppet-openstack | 08:18 | |
*** itsuugo has joined #puppet-openstack | 08:19 | |
*** markvoelker has joined #puppet-openstack | 08:19 | |
*** social has joined #puppet-openstack | 08:20 | |
*** zhangjn has quit IRC | 08:22 | |
*** zhangjn has joined #puppet-openstack | 08:23 | |
*** markvoelker has quit IRC | 08:23 | |
*** zhangjn has quit IRC | 08:26 | |
*** apuimedo has joined #puppet-openstack | 08:33 | |
openstackgerrit | Merged openstack/puppet-cinder: Move default value to '<SERVICE DEFAULT>' https://review.openstack.org/221005 | 08:37 |
*** chandankumar has joined #puppet-openstack | 08:46 | |
openstackgerrit | Yanis Guenane proposed openstack/puppet-cinder: Introduce cinder::db class https://review.openstack.org/208354 | 09:06 |
*** itsuugo has quit IRC | 09:09 | |
openstackgerrit | Yanis Guenane proposed openstack/puppet-cinder: Introduce cinder::db class https://review.openstack.org/208354 | 09:09 |
openstackgerrit | Yanis Guenane proposed openstack/puppet-gnocchi: Introduce gnocchi::db class https://review.openstack.org/208364 | 09:14 |
*** markvoelker has joined #puppet-openstack | 09:19 | |
openstackgerrit | Yanis Guenane proposed openstack/puppet-ironic: Introduce ironic::db class https://review.openstack.org/208381 | 09:20 |
*** itsuugo has joined #puppet-openstack | 09:23 | |
openstackgerrit | Denis Egorenko proposed openstack/puppet-sahara: Add new parameters for Sahara https://review.openstack.org/220090 | 09:24 |
*** markvoelker has quit IRC | 09:24 | |
openstackgerrit | Denis Egorenko proposed openstack/puppet-sahara: Add new parameters for Sahara https://review.openstack.org/220090 | 09:26 |
openstackgerrit | Yanis Guenane proposed openstack/puppet-keystone: Introduce keystone::db class https://review.openstack.org/208394 | 09:27 |
openstackgerrit | Yanis Guenane proposed openstack/puppet-manila: Introduce manila::db class https://review.openstack.org/208402 | 09:28 |
openstackgerrit | Yanis Guenane proposed openstack/puppet-manila: Introduce manila::db class https://review.openstack.org/208402 | 09:30 |
openstackgerrit | Yanis Guenane proposed openstack/puppet-neutron: Introduce neutron::db class https://review.openstack.org/208415 | 09:31 |
spredzy | degorenko, ping https://review.openstack.org/#/c/220090/ | 09:32 |
spredzy | If you want to go ahead and update it with the pattern applied in the above reviews ^ I'd be thankful ;) | 09:33 |
degorenko | spredzy, pong, yep :) i'll upload a new patch | 09:37 |
*** paramite is now known as paramite|afk | 09:55 | |
*** pradk has quit IRC | 09:59 | |
*** sanjayu has joined #puppet-openstack | 10:02 | |
*** richm has joined #puppet-openstack | 10:06 | |
*** skolekonov has joined #puppet-openstack | 10:08 | |
*** pradk has joined #puppet-openstack | 10:14 | |
*** xingchao has quit IRC | 10:16 | |
*** paramite|afk is now known as paramite | 10:21 | |
*** _morgan has quit IRC | 10:31 | |
*** Hunner has quit IRC | 10:31 | |
openstackgerrit | Yanis Guenane proposed openstack/puppet-neutron: Introduce neutron::db class https://review.openstack.org/208415 | 10:31 |
openstackgerrit | Yanis Guenane proposed openstack/puppet-manila: Introduce manila::db class https://review.openstack.org/208402 | 10:40 |
openstackgerrit | Yanis Guenane proposed openstack/puppet-keystone: Introduce keystone::db class https://review.openstack.org/208394 | 10:43 |
*** smakar has quit IRC | 10:46 | |
*** IBerezovskiy has joined #puppet-openstack | 10:50 | |
openstackgerrit | Yanis Guenane proposed openstack/puppet-ironic: Introduce ironic::db class https://review.openstack.org/208381 | 10:53 |
*** ferest has joined #puppet-openstack | 10:53 | |
IBerezovskiy | EmilienM, hi! Can we discuss puppet-murano a little bit? | 10:53 |
IBerezovskiy | EmilienM, I've read you email about puppet-murano governance, and I have some question about patches from Alexey Deryugin | 10:54 |
IBerezovskiy | these patches implement most of all murano functionality | 10:54 |
openstackgerrit | Yanis Guenane proposed openstack/puppet-gnocchi: Introduce gnocchi::db class https://review.openstack.org/208364 | 10:55 |
IBerezovskiy | can we merge them somehow before removing murano-core group from puppet-murano? | 10:55 |
*** itsuugo has quit IRC | 10:56 | |
openstackgerrit | Yanis Guenane proposed openstack/puppet-keystone: Introduce keystone::db class https://review.openstack.org/208394 | 10:57 |
*** smakar has joined #puppet-openstack | 10:58 | |
*** itsuugo has joined #puppet-openstack | 11:02 | |
openstackgerrit | Yanis Guenane proposed openstack/puppet-manila: Introduce manila::db class https://review.openstack.org/208402 | 11:02 |
openstackgerrit | Yanis Guenane proposed openstack/puppet-cinder: Introduce cinder::db class https://review.openstack.org/208354 | 11:06 |
*** ferest has quit IRC | 11:07 | |
*** ducttape_ has joined #puppet-openstack | 11:13 | |
openstackgerrit | Yanis Guenane proposed openstack/puppet-neutron: Introduce neutron::db class https://review.openstack.org/208415 | 11:14 |
spredzy | degorenko, ping https://review.openstack.org/#/c/208415 | 11:15 |
*** ducttape_ has quit IRC | 11:17 | |
degorenko | spredzy, pong :) | 11:17 |
*** xingchao has joined #puppet-openstack | 11:18 | |
spredzy | degorenko, I think there is something were we are not understanding each other :( | 11:18 |
degorenko | spredzy, yep :) | 11:18 |
spredzy | The current pick pattern remains fully backward compatible and provide the exact same feature as before | 11:18 |
degorenko | yes | 11:19 |
spredzy | ie. a user using hiera won't use a non-hiera pattern. So the value can't be defined twice | 11:19 |
degorenko | exactly | 11:19 |
spredzy | having undef in both values, always the right one will be picked | 11:19 |
spredzy | So we go from the most specific one | 11:19 |
spredzy | To the most global one | 11:19 |
*** ducttape_ has joined #puppet-openstack | 11:19 | |
spredzy | neutron::db::X then neutron::server::X | 11:20 |
spredzy | then default | 11:20 |
*** _morgan has joined #puppet-openstack | 11:20 | |
*** markvoelker has joined #puppet-openstack | 11:20 | |
degorenko | well, we should have undef only for old params - in neutron::server. In new db class we can set our new params to config defaults. | 11:20 |
degorenko | so, it will be like: pick neutron::server::X then neutron::db::X | 11:20 |
*** Hunner has joined #puppet-openstack | 11:20 | |
*** Hunner has quit IRC | 11:20 | |
*** Hunner has joined #puppet-openstack | 11:20 | |
degorenko | because some users can still use old neutron::server::X | 11:21 |
degorenko | and this variable will be taken | 11:21 |
*** karume has quit IRC | 11:21 | |
spredzy | degorenko, that is the end purpose cf. https://review.openstack.org/#/c/208354/8/manifests/db.pp,cm | 11:21 |
degorenko | but if this variable will not passed (undef) we should take neutron::db::X (which by default - default from config) | 11:21 |
degorenko | spredzy, yes ^^^ that's right | 11:23 |
degorenko | i'm about 208354 | 11:23 |
degorenko | i think, the same pattern should be for all projects | 11:23 |
degorenko | and for now, we can just use default config values for params in db classes | 11:23 |
*** markvoelker has quit IRC | 11:25 | |
openstackgerrit | Yanis Guenane proposed openstack/puppet-neutron: Introduce neutron::db class https://review.openstack.org/208415 | 11:26 |
spredzy | degorenko, ack will do ^ | 11:26 |
*** itsuugo has quit IRC | 11:27 | |
degorenko | spredzy, thanks man o/ I'll do the same for sahara | 11:28 |
spredzy | ack will also update the other I have pending | 11:28 |
*** xingchao has quit IRC | 11:31 | |
*** smakar has quit IRC | 11:35 | |
skolekonov | spredzy, hello. Speaking about my question, I meant that currently it's impossible to call both classes (server and db) from upper class. Isn't it the valid case? | 11:35 |
*** itsuugo has joined #puppet-openstack | 11:36 | |
*** stamak has joined #puppet-openstack | 11:36 | |
*** morazi has joined #puppet-openstack | 11:42 | |
EmilienM | good morning | 11:42 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Initial module implementation https://review.openstack.org/211043 | 11:43 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Implementaion of api and engine classes https://review.openstack.org/216682 | 11:43 |
EmilienM | IBerezovskiy: sure, discussion is open | 11:43 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Implementaion of api and engine classes https://review.openstack.org/216682 | 11:44 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: MySQL and PostreSQL class implementation https://review.openstack.org/218221 | 11:44 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Keystone auth class implementation https://review.openstack.org/218234 | 11:44 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Client and dashboard class implementation https://review.openstack.org/218333 | 11:44 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Application resource implementation https://review.openstack.org/220238 | 11:44 |
IBerezovskiy | EmilienM, morning :) What if we will keep murano-core rights till Alexey's patches will be merged? As I mentioned in email, these patches implements most of Murano deployment stuff | 11:44 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: RabbitMQ class implementation https://review.openstack.org/218288 | 11:45 |
EmilienM | IBerezovskiy: so don't you think it's weird? | 11:45 |
EmilienM | I don't understand why we need to continue to make an exception | 11:45 |
IBerezovskiy | Emilien, a little bit, but Murano is like a new project | 11:45 |
*** cdelatte has quit IRC | 11:46 | |
IBerezovskiy | and it's puppets aren't ready yet | 11:46 |
EmilienM | IBerezovskiy: Murano == new project? I don't think so, I've seen its first demo in OpenStack in Action, Bucarest 2013 | 11:46 |
EmilienM | yeah puppet module is recent <-- true | 11:46 |
EmilienM | but I don't understand why you need governance on it | 11:47 |
EmilienM | it's a Puppet OpenStack module and each project has its governance | 11:47 |
IBerezovskiy | you (OpenStack puppet team) said some time before (about Murano and Sahara too), that you aren't familiar with them, and you need to get reviews from Murano/Sahara core | 11:48 |
*** itsuugo has quit IRC | 11:48 | |
IBerezovskiy | that's why I want to make the process of merging Murano easier for you | 11:49 |
fvollero | EmilienM: morning :) | 11:49 |
*** dprince has joined #puppet-openstack | 11:50 | |
EmilienM | IBerezovskiy: you don't use cookiebutter to bootstrap the module, while it's our process | 11:51 |
EmilienM | IBerezovskiy: -1/+1 would be enough, I don't understand why you need +2 | 11:52 |
*** ericpeterson has joined #puppet-openstack | 11:52 | |
IBerezovskiy | EmilienM, may be cookiecutter? | 11:53 |
*** jpena is now known as jpena|lunch | 11:53 | |
EmilienM | IBerezovskiy: spredzy and sbadia did that project for that purpose - there is documentation > https://wiki.openstack.org/wiki/Puppet/New_module | 11:54 |
EmilienM | IBerezovskiy: using it is not an option if you want your module compliant | 11:54 |
IBerezovskiy | EmilienM, rights are to merge faster. some +1 and one +2 from OpenStack puppet team, and the same for Murano. And patch can be merged, isn't ? | 11:55 |
*** ducttape_ has quit IRC | 11:55 | |
iurygregory | good morning | 11:55 |
EmilienM | IBerezovskiy: that's where I disagree - we | 11:55 |
EmilienM | we don't care about fast here | 11:55 |
*** ducttape_ has joined #puppet-openstack | 11:55 | |
EmilienM | IBerezovskiy: we care about code quality and keeping our modules consistents | 11:56 |
EmilienM | IBerezovskiy: we wanted collaboration - you need to accept reviews, and understand things do not go as fast as you want when you contribute upstream | 11:57 |
EmilienM | so I have two comments on the murano reviews: | 11:57 |
EmilienM | 1/ it does not use cookiebutter -> the module is not compliant (see zaqar, they just did it) | 11:57 |
*** ericpeterson has quit IRC | 11:57 | |
IBerezovskiy | EmilienM, sure, quality and consistent is most important things. but expertise from Murano cores (and murano deployers) are useful | 11:58 |
EmilienM | 2/ dashboard implementation looks like we could use puppet-horizon for that: https://review.openstack.org/#/c/218333/6/manifests/dashboard.pp,cm | 11:58 |
EmilienM | IBerezovskiy: so we should but all OpenStack teams core of our modules? | 11:58 |
EmilienM | I don't see why Murano should be an exception - Murano Core can -1/+1 - their feedback will be valuable & taken in consideration | 11:59 |
*** chandankumar has quit IRC | 11:59 | |
IBerezovskiy | EmilienM, I agree with your point, only OpenStack puppet team should have rights on puppet projects. All that I want is too merge most of important Murano changes with good quality and before deadline of Liberty | 12:00 |
EmilienM | IBerezovskiy: so use cookiebutter | 12:00 |
EmilienM | for a first patch | 12:00 |
IBerezovskiy | that's why I still think that murano-core is useful here | 12:00 |
*** ducttape_ has quit IRC | 12:00 | |
EmilienM | see https://github.com/openstack/puppet-zaqar/commit/33c0956a92e77ffa356e244d32ecb092b112ed90 | 12:01 |
IBerezovskiy | we will fix that | 12:01 |
EmilienM | IBerezovskiy: I don't think you're agreeing with my point - since you want murano team core able to +2 puppet patches | 12:02 |
*** sanjayu has quit IRC | 12:03 | |
*** yogesh-pc has joined #puppet-openstack | 12:04 | |
IBerezovskiy | EmilienM, as I said these rights can be useful only till we (Murano and OpenStack puppet teams) are working on the most difficult and important parts. | 12:05 |
EmilienM | IBerezovskiy: is murano dashboard integrated in Horizon? or separated dashboard at all? | 12:06 |
*** paramite is now known as paramite|afk | 12:06 | |
IBerezovskiy | EmilienM, murano-dashboard is plugin for horizon | 12:07 |
IBerezovskiy | it's not integrated | 12:07 |
EmilienM | ok so then it's ok to have it in puppet-murano | 12:08 |
EmilienM | I'll drop this -1 | 12:08 |
IBerezovskiy | EmilienM, thanks! | 12:08 |
*** paramite|afk is now known as paramite | 12:08 | |
EmilienM | I go breakfast now | 12:09 |
fvollero | EmilienM: Enjoy the breakfast :) | 12:09 |
fvollero | IBerezovskiy: Honestly I don't see why murano cores need to have the +2 status | 12:10 |
*** cdelatte has joined #puppet-openstack | 12:13 | |
*** pradk has quit IRC | 12:14 | |
*** delattec has joined #puppet-openstack | 12:14 | |
*** delattec has quit IRC | 12:15 | |
*** jayg|g0n3 is now known as jayg | 12:15 | |
*** karume has joined #puppet-openstack | 12:18 | |
*** cdelatte has quit IRC | 12:18 | |
*** markvoelker has joined #puppet-openstack | 12:19 | |
IBerezovskiy | fvollero, -2 also can be useful sometimes :) +2 shows more value than +1, isn't? | 12:21 |
*** itsuugo has joined #puppet-openstack | 12:22 | |
*** karume has quit IRC | 12:23 | |
iurygregory | I think if they are core from the openstack component the +/- 1 is more meaningful (the don't need give +/- 2) and the puppet core team can take into consideration ^^ | 12:25 |
*** cdelatte has joined #puppet-openstack | 12:25 | |
EmilienM | iurygregory: this is exactly what we're doing, thanks for confirming | 12:28 |
iurygregory | o/ | 12:28 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: RabbitMQ class implementation https://review.openstack.org/218288 | 12:28 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: RabbitMQ class implementation https://review.openstack.org/218288 | 12:30 |
*** karume has joined #puppet-openstack | 12:37 | |
fvollero | IBerezovskiy: i agree with iurygregory on this topic. I mean, i saw more than one component of Ceilometer core that does not have the +2 but have the +1 and does not get less traction | 12:39 |
*** pradk has joined #puppet-openstack | 12:40 | |
*** itsuugo has quit IRC | 12:42 | |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Initial module implementation https://review.openstack.org/211043 | 12:43 |
*** richm has quit IRC | 12:43 | |
*** itsuugo has joined #puppet-openstack | 12:43 | |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Initial module implementation https://review.openstack.org/211043 | 12:44 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Implementaion of api and engine classes https://review.openstack.org/216682 | 12:44 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: MySQL and PostreSQL class implementation https://review.openstack.org/218221 | 12:44 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Keystone auth class implementation https://review.openstack.org/218234 | 12:44 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Client and dashboard class implementation https://review.openstack.org/218333 | 12:44 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Application resource implementation https://review.openstack.org/220238 | 12:44 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: RabbitMQ class implementation https://review.openstack.org/218288 | 12:44 |
*** jpena|lunch is now known as jpena | 12:51 | |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Initial module implementation https://review.openstack.org/211043 | 12:53 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Implementaion of api and engine classes https://review.openstack.org/216682 | 12:53 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: MySQL and PostreSQL class implementation https://review.openstack.org/218221 | 12:53 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Keystone auth class implementation https://review.openstack.org/218234 | 12:53 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Client and dashboard class implementation https://review.openstack.org/218333 | 12:53 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Application resource implementation https://review.openstack.org/220238 | 12:54 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: RabbitMQ class implementation https://review.openstack.org/218288 | 12:54 |
*** richm has joined #puppet-openstack | 12:57 | |
*** chandankumar has joined #puppet-openstack | 13:00 | |
*** chandankumar has quit IRC | 13:03 | |
openstackgerrit | Yanis Guenane proposed openstack/puppet-gnocchi: Introduce gnocchi::db class https://review.openstack.org/208364 | 13:03 |
openstackgerrit | Yanis Guenane proposed openstack/puppet-ironic: Introduce ironic::db class https://review.openstack.org/208381 | 13:04 |
openstackgerrit | Yanis Guenane proposed openstack/puppet-keystone: Introduce keystone::db class https://review.openstack.org/208394 | 13:04 |
openstackgerrit | Yanis Guenane proposed openstack/puppet-manila: Introduce manila::db class https://review.openstack.org/208402 | 13:04 |
spredzy | skolekonov: re- https://review.openstack.org/#/c/208415/ | 13:05 |
spredzy | skolekonov, neutron::db is not meant to be called in a manifest, it is meant to be returned by an ENC | 13:06 |
spredzy | it is a process that tells an host which class to apply | 13:06 |
spredzy | degorenko, https://review.openstack.org/#/q/status:open+branch:master+topic:class_db,n,z all up-to-date | 13:07 |
skolekonov | spredzy, but does it mean that we will keep database parameters both in server and db classes? | 13:07 |
spredzy | skolekonov, while we need to support both non-hiera and hiera deployment yes | 13:07 |
skolekonov | So deprecation of parameters in server class is useless, I guess | 13:08 |
skolekonov | spredzy, understood, thank you. +1 from me than | 13:09 |
spredzy | skolekonov, where are the parameters deprecated ? | 13:10 |
skolekonov | spredzy, nowhere, I was just thinking about it :) | 13:10 |
spredzy | skolekonov, ack. No they are not deprecated, from neutron::server | 13:11 |
spredzy | :) | 13:11 |
*** itsuugo has quit IRC | 13:13 | |
*** jpena has quit IRC | 13:14 | |
*** jpena has joined #puppet-openstack | 13:15 | |
*** tiswanso has joined #puppet-openstack | 13:15 | |
*** tiswanso has quit IRC | 13:16 | |
*** tiswanso has joined #puppet-openstack | 13:17 | |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: RabbitMQ class implementation https://review.openstack.org/218288 | 13:17 |
*** itsuugo has joined #puppet-openstack | 13:23 | |
*** chandankumar has joined #puppet-openstack | 13:23 | |
*** xingchao has joined #puppet-openstack | 13:29 | |
*** yogesh-pc has quit IRC | 13:38 | |
EmilienM | degorenko: I moved your CI issue (sahara) on open discussion | 13:40 |
EmilienM | because I commented the review, and there is nothing much to say but your patch is not working with RDO packaging | 13:40 |
degorenko | EmilienM, i've tested my patch many times locally - it works, no any errors | 13:42 |
*** yogesh-pc has joined #puppet-openstack | 13:43 | |
EmilienM | degorenko: on RDO? | 13:43 |
EmilienM | degorenko: and it's idempotent? | 13:43 |
EmilienM | degorenko: look my comment I show you CI logs, they say "your patch is not idempotent, second puppet runs change POSIX permissions" <--- because RDO packaging is setting different permissions than your init.pp | 13:44 |
degorenko | EmilienM, currently Sahara uses cbs, isn't it? | 13:46 |
degorenko | EmilienM, i've ran right now manifests from my patch twice - no any error | 13:46 |
*** itsuugo has quit IRC | 13:47 | |
EmilienM | degorenko: why does it fail in our CI? Have you seen logs? | 13:47 |
EmilienM | look http://logs.openstack.org/42/222142/3/check/gate-puppet-sahara-puppet-beaker-rspec-dsvm-centos7/ba59219/console.html#_2015-09-15_12_25_17_283 | 13:48 |
*** tiswanso has quit IRC | 13:48 | |
EmilienM | /Stage[main]/Sahara/File[/etc/sahara/]/group: group changed 'root' to 'sahara' | 13:48 |
EmilienM | /Stage[main]/Sahara/File[/etc/sahara/sahara.conf]/group: group changed 'root' to 'sahara' | 13:48 |
EmilienM | RDO packaging must set it to 'root' | 13:48 |
degorenko | oh | 13:48 |
EmilienM | while Puppet wants 'sahara' | 13:48 |
EmilienM | degorenko: you got it? | 13:48 |
degorenko | yep | 13:48 |
degorenko | sorry | 13:48 |
degorenko | i forgot about second run for idempotent | 13:49 |
*** itsuugo has joined #puppet-openstack | 13:49 | |
EmilienM | degorenko: though it's weird: https://github.com/openstack-packages/sahara/blob/rpm-master/openstack-sahara.spec#L9-L10 | 13:50 |
EmilienM | degorenko: please ping someone on #rdo | 13:50 |
EmilienM | and explain your issue, number80 is a core maintainer for openstack packages | 13:50 |
EmilienM | or apevec | 13:50 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: puppet-murano: Initial commit https://review.openstack.org/223597 | 13:51 |
degorenko | EmilienM, yeah, it seems like problem with package, btw, i have patch for sahara to RDO: https://review.gerrithub.io/#/c/234556 | 13:53 |
degorenko | but it outdated now | 13:53 |
degorenko | because of dependency package issues | 13:53 |
degorenko | i'll update them | 13:54 |
EmilienM | degorenko: I can't help on packaging | 13:54 |
openstackgerrit | Yanis Guenane proposed openstack/puppet-gnocchi: Introduce gnocchi::db class https://review.openstack.org/208364 | 13:54 |
degorenko | EmilienM, i'm not expected any help from you :) just fyi | 13:55 |
*** tiswanso has joined #puppet-openstack | 13:55 | |
EmilienM | I love hearing that | 13:55 |
EmilienM | lol | 13:55 |
*** yogesh-pc has quit IRC | 13:55 | |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: puppet-murano: Initial commit https://review.openstack.org/223597 | 13:55 |
degorenko | EmilienM, also, can i use some hack for my revert, like manually change group in puppet? Because it will be reverted in next patch anyway (which has +1) | 13:56 |
degorenko | and also i'll remove my patch from agenda | 13:56 |
EmilienM | degorenko: yeah, otherwise CI won't pass | 13:56 |
EmilienM | we can make an exception I guess, but explain it in the commit message | 13:56 |
degorenko | EmilienM, oh! thanks! i'll update it | 13:57 |
openstackgerrit | Yanis Guenane proposed openstack/puppet-ironic: Introduce ironic::db class https://review.openstack.org/208381 | 13:57 |
*** ducttape_ has joined #puppet-openstack | 13:58 | |
*** tiswanso has quit IRC | 14:00 | |
*** sergmelikyan has joined #puppet-openstack | 14:02 | |
*** sergmelikyan has quit IRC | 14:04 | |
*** skolekonov is now known as _skolekonov | 14:04 | |
*** sergmelikyan has joined #puppet-openstack | 14:04 | |
*** itsuugo has quit IRC | 14:09 | |
degorenko | spredzy, hey, are you around? | 14:10 |
spredzy | degorenko, yup | 14:15 |
degorenko | i have question about https://review.openstack.org/#/c/208394/ | 14:15 |
degorenko | spredzy, where you take max_overflow = 20 ? :) | 14:15 |
degorenko | by default it is None | 14:16 |
degorenko | spredzy, ^ | 14:16 |
spredzy | degorenko, I expected this question to come, so the answer is https://github.com/openstack/puppet-neutron/blob/master/manifests/server.pp#L220 | 14:16 |
spredzy | that was to stay consistent with a param we already have in another module | 14:17 |
degorenko | spredzy, well, for neutron - yeah, it is correct. Okay, thanks :) i'm going to upload the same patch for sahara and just want to know anout max_overflow | 14:17 |
degorenko | s/anout/about | 14:17 |
spredzy | degorenko, if someone -1 for it I don't mind making it to undef. Anyway most of this code will switch to <SERVICE DEFAULT> soon | 14:20 |
spredzy | :D | 14:20 |
degorenko | spredzy, yeah :D agree | 14:20 |
spredzy | neutron or not, it is managed by oslo.db | 14:20 |
degorenko | yep | 14:21 |
*** itsuugo has joined #puppet-openstack | 14:21 | |
social | crinkle: I have question about this https://review.openstack.org/#/c/189301/ | 14:26 |
*** llua^_^AznSmile- is now known as llua | 14:27 | |
*** llua has joined #puppet-openstack | 14:27 | |
social | crinkle: any idea why I had to add require lines loading the files? It should work without those | 14:27 |
*** kbyrne has quit IRC | 14:31 | |
*** apuimedo has quit IRC | 14:36 | |
*** chem has joined #puppet-openstack | 14:37 | |
*** kbyrne has joined #puppet-openstack | 14:37 | |
*** kbyrne has quit IRC | 14:38 | |
*** mdorman has joined #puppet-openstack | 14:38 | |
*** chandankumar has quit IRC | 14:39 | |
*** kbyrne has joined #puppet-openstack | 14:40 | |
*** itsuugo has quit IRC | 14:41 | |
openstackgerrit | Yanis Guenane proposed openstack/puppet-keystone: Introduce keystone::db class https://review.openstack.org/208394 | 14:42 |
*** tiswanso has joined #puppet-openstack | 14:42 | |
*** yogesh-pc has joined #puppet-openstack | 14:49 | |
*** angdraug has joined #puppet-openstack | 14:50 | |
*** skolekonov has joined #puppet-openstack | 14:52 | |
EmilienM | our weekly meeting starts in a few minutes, feel free to participate. Our agenda URL is here: https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-20150915 | 14:54 |
*** xarses has joined #puppet-openstack | 14:56 | |
*** yogesh-pc has quit IRC | 14:57 | |
*** itsuugo has joined #puppet-openstack | 14:59 | |
*** yogesh-pc has joined #puppet-openstack | 15:00 | |
crinkle | social: i'm not really sure | 15:02 |
crinkle | social: is it just in puppet 4? is it a problem in rspec-puppet or just in general? | 15:03 |
social | crinkle: I think the issue is only on puppet4 and diffs between versions | 15:03 |
social | crinkle: 4.0~4.1 work without require but they probably just eat the error in the process | 15:03 |
*** yogesh-pc_ has joined #puppet-openstack | 15:04 | |
crinkle | social: i'm not sure what 4.0-4.1 means | 15:04 |
crinkle | social: 4.1 works? | 15:05 |
social | yes | 15:06 |
social | 4.2 does not | 15:06 |
*** yogesh-pc has quit IRC | 15:06 | |
openstackgerrit | Denis Egorenko proposed openstack/puppet-sahara: Introduce sahara::db class https://review.openstack.org/223638 | 15:07 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: puppet-murano: Initial commit https://review.openstack.org/223597 | 15:07 |
degorenko | spredzy, https://review.openstack.org/223638 - sahara db :) | 15:08 |
crinkle | social: weird | 15:09 |
crinkle | social: you might want to ask in #puppet-dev about it or create a ticket | 15:10 |
crinkle | inconsistent behavior in the load path is no good | 15:10 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Initial module implementation https://review.openstack.org/211043 | 15:13 |
openstackgerrit | Denis Egorenko proposed openstack/puppet-sahara: Add new parameter 'plugins' for Sahara https://review.openstack.org/220090 | 15:16 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Implementaion of api and engine classes https://review.openstack.org/216682 | 15:19 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: MySQL and PostreSQL class implementation https://review.openstack.org/218221 | 15:19 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Keystone auth class implementation https://review.openstack.org/218234 | 15:20 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Client and dashboard class implementation https://review.openstack.org/218333 | 15:20 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Application resource implementation https://review.openstack.org/220238 | 15:20 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: RabbitMQ class implementation https://review.openstack.org/218288 | 15:20 |
openstackgerrit | Denis Egorenko proposed openstack/puppet-sahara: Add new parameter 'plugins' for Sahara https://review.openstack.org/220090 | 15:21 |
openstackgerrit | Denis Egorenko proposed openstack/puppet-sahara: Add new parameter 'plugins' for Sahara https://review.openstack.org/220090 | 15:22 |
*** itsuugo has quit IRC | 15:24 | |
openstackgerrit | Denis Egorenko proposed openstack/puppet-sahara: Add new parameter 'plugins' for Sahara https://review.openstack.org/220090 | 15:31 |
*** yogesh-pc has joined #puppet-openstack | 15:32 | |
*** yogesh-pc_ has quit IRC | 15:33 | |
*** xingchao has quit IRC | 15:34 | |
*** stamak has quit IRC | 15:38 | |
*** itsuugo has joined #puppet-openstack | 15:38 | |
*** Thaxll has joined #puppet-openstack | 15:38 | |
Thaxll | Hi, can someone with ceph-puppet experience can have a quick look at that: http://pastebin.com/3P6e4dm7 I'm stuck and I don't understand what the problem is... I'm trying to setup a quick ceph poc on a single node. | 15:40 |
*** xingchao has joined #puppet-openstack | 15:44 | |
*** xingchao has quit IRC | 15:46 | |
aderyugin | Hi, folks! Review https://review.openstack.org/#/q/project:openstack/puppet-murano+branch:master+topic:murano-initial-implementation,n,z please | 15:48 |
*** skolekonov2 has joined #puppet-openstack | 15:52 | |
*** skolekonov has quit IRC | 15:56 | |
*** skolekonov2 is now known as skolekonov | 15:56 | |
*** richm has quit IRC | 15:59 | |
*** skolekonov has quit IRC | 16:00 | |
openstackgerrit | Timothy Swanson proposed openstack/puppet-neutron: Remove use of template based config from cisco ml2 nexus plugin. https://review.openstack.org/222451 | 16:01 |
*** paramite has quit IRC | 16:02 | |
openstackgerrit | Merged openstack/puppet-openstack-integration: Prepare Tempest and run smoke https://review.openstack.org/217352 | 16:08 |
openstackgerrit | Merged openstack/puppet-murano: puppet-murano: Initial commit https://review.openstack.org/223597 | 16:10 |
*** rochaporto has quit IRC | 16:11 | |
*** itsuugo has quit IRC | 16:12 | |
*** richm has joined #puppet-openstack | 16:13 | |
*** social has quit IRC | 16:13 | |
*** IBerezovskiy has quit IRC | 16:14 | |
*** chandankumar has joined #puppet-openstack | 16:17 | |
*** richm has quit IRC | 16:18 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-openstacklib: Create is_service_default function https://review.openstack.org/223672 | 16:19 |
*** richm has joined #puppet-openstack | 16:20 | |
*** aimon has quit IRC | 16:23 | |
mdorman | anybody using multiple domains in keystone with the puppet keystone providers? | 16:24 |
*** jistr has quit IRC | 16:24 | |
mdorman | i am running into the issue described here: https://openstack.nimeyo.com/52755/openstack-operators-automating-keystone-configuration-command the admin token, or whatever creds the providers are using, are not scoped to a domain so keystone doesn’t know what domain is being requested | 16:24 |
EmilienM | mdorman: chem is developping it | 16:24 |
*** itsuugo has joined #puppet-openstack | 16:25 | |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-sahara: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197023 | 16:32 |
*** arnaud_orange has quit IRC | 16:32 | |
*** jpena is now known as jpena|away | 16:38 | |
openstackgerrit | John Trowbridge proposed openstack/puppet-ironic: Add ironic-inspector support https://review.openstack.org/223690 | 16:40 |
*** xingchao has joined #puppet-openstack | 16:47 | |
*** cdelatte has quit IRC | 16:48 | |
openstackgerrit | John Trowbridge proposed openstack/puppet-ironic: Add ironic-inspector support https://review.openstack.org/223690 | 16:50 |
*** yogesh-pc_ has joined #puppet-openstack | 16:51 | |
*** itsuugo has quit IRC | 16:51 | |
*** xingchao has quit IRC | 16:52 | |
*** ericpeterson has joined #puppet-openstack | 16:53 | |
*** yogesh-pc has quit IRC | 16:54 | |
*** ducttape_ has quit IRC | 16:56 | |
*** yogesh-pc has joined #puppet-openstack | 16:57 | |
*** yogesh-pc_ has quit IRC | 16:59 | |
*** social has joined #puppet-openstack | 17:00 | |
*** yogesh-pc has quit IRC | 17:01 | |
*** chandankumar has quit IRC | 17:03 | |
*** aimon has joined #puppet-openstack | 17:04 | |
*** itsuugo has joined #puppet-openstack | 17:06 | |
*** cdelatte has joined #puppet-openstack | 17:07 | |
*** derekh has quit IRC | 17:11 | |
*** karume has quit IRC | 17:11 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-openstacklib: Create is_service_default function https://review.openstack.org/223672 | 17:23 |
*** oc714|3 has joined #puppet-openstack | 17:26 | |
*** oc714|3 has left #puppet-openstack | 17:26 | |
*** ericpeterson has quit IRC | 17:27 | |
*** ducttape_ has joined #puppet-openstack | 17:28 | |
*** oc714 has joined #puppet-openstack | 17:29 | |
oc714 | hi Guys, I am new in here on how to use puppet for deploying Openstack | 17:31 |
oc714 | I am used to using rdo wedsite for davstack/ | 17:31 |
oc714 | Does puppet have to thru the long way of installing each component? | 17:31 |
*** xarses has quit IRC | 17:34 | |
*** degorenko has quit IRC | 17:37 | |
*** itsuugo has quit IRC | 17:47 | |
*** kbyrne has quit IRC | 17:49 | |
*** dprince has quit IRC | 17:54 | |
*** dprince has joined #puppet-openstack | 17:56 | |
*** xarses has joined #puppet-openstack | 17:59 | |
*** karume has joined #puppet-openstack | 18:07 | |
EmilienM | oc714: hi, you might want to start by https://wiki.openstack.org/wiki/Puppet | 18:08 |
oc714 | I did but there is no step 1 and so on just like the rdo website or maybe I not seeing it of the bat | 18:09 |
*** karume has quit IRC | 18:11 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: DNM - CI test https://review.openstack.org/223735 | 18:11 |
oc714 | EmilienM: can i install puppet on vbox for now | 18:12 |
*** stamak has joined #puppet-openstack | 18:13 | |
EmilienM | oc714: yes but there is no official doc explaing how now, we are currently working on this | 18:14 |
*** dprince has quit IRC | 18:14 | |
oc714 | ah ok. in the meantime I can watch some youtube videos. if i have questions than i can ask the community | 18:14 |
*** dprince has joined #puppet-openstack | 18:15 | |
iurygregory | Hey EmilienM, the common attributes for service provider are: methods (auth section) and the respective plugin , so we need a common class to share this attributes? | 18:17 |
imcsk8 | EmilienM: i'm creating the acceptance test for sriov, i see that there's some stuff already here: https://github.com/openstack/puppet-neutron/blob/master/spec/classes/neutron_plugins_ml2_spec.rb#L211 is it ok just to add the new stuff? | 18:18 |
*** stamak has quit IRC | 18:22 | |
*** v1k0d3n has joined #puppet-openstack | 18:28 | |
EmilienM | imcsk8: this is not acceptance files but unit testing ;-) | 18:31 |
EmilienM | dprince: https://docs.puppetlabs.com/references/latest/metaparameter.html#tag | 18:32 |
imcsk8 | EmilienM: oh damn! right :P | 18:33 |
EmilienM | clayton: have you tried to run --tags attribute when playing with containers? | 18:35 |
EmilienM | iurygregory: any example of auth section? | 18:35 |
*** cdelatte has quit IRC | 18:35 | |
iurygregory | EmilienM, http://paste.openstack.org/show/463362/ | 18:39 |
EmilienM | iurygregory: if it's only this... no need for common class I think | 18:40 |
iurygregory | ok ^^ | 18:40 |
*** kindjal has joined #puppet-openstack | 18:49 | |
*** cdelatte has joined #puppet-openstack | 18:50 | |
kindjal | Does anyone use puppet openstack in multi-domain mode? I find that it is not supplying —domain default in calls to /usr/bin/openstack, and thus fails to manage resources. | 18:50 |
mdorman | kindjal: i’m having the exact same problem | 18:50 |
EmilienM | kindjal: chem is actually working on this | 18:50 |
EmilienM | chem: if you around :) | 18:51 |
kindjal | I just filed: https://bugs.launchpad.net/puppet-keystone/+bug/1496087 | 18:51 |
openstack | Launchpad bug 1496087 in puppet-keystone "puppet-keystone fails to manage service users in multi-domain mode" [Undecided,New] | 18:51 |
iurygregory | kindjal, i think someone is working with this.. chem is developping it | 18:51 |
kindjal | that’s great news… | 18:51 |
kindjal | is there some public visibility into the work? I’d love to patch/test whatever’s available. | 18:52 |
mdorman | kindjal: i’ve been trying to get it working manually with the openstack cli directly, but even that is being problematic | 18:52 |
EmilienM | https://trello.com/c/xDDgtctf/22-extend-keystone-config-to-support-multiple-domains-with-a-config-file-per-domain | 18:52 |
EmilienM | https://review.openstack.org/#/c/219289/ | 18:52 |
EmilienM | mdorman: is it what you need ^ ? | 18:53 |
mdorman | looking | 18:54 |
mdorman | no that’s not really my problem (although a good feature.) it’s more a problem of how the keystone_user and keystone_user_role providers are calling keystone via openstackclient | 18:55 |
kindjal | It looks like it *should* be passing —domain default to openstackclient, but it’s not. | 18:56 |
mdorman | it’s the bug that kindjal just created | 18:56 |
EmilienM | looking | 18:56 |
EmilienM | richm: ^ | 18:56 |
mdorman | imo it’s more a problem with openstackclient and/or how the keystone v3 api handles default domain | 18:57 |
*** jpena|away is now known as jpena | 18:57 | |
mdorman | but i believe it would be solved for our providers with adding —domain default to the openstackclient command lines. | 18:57 |
mdorman | been trying to get something togehter here to validate that theory, but it’s been a busy day :-/ | 18:57 |
kindjal | mdorman: I agree. | 18:58 |
mdorman | plus i am kind of confused about the whole keystone domains thing and how tokens get scoped to them | 18:59 |
mdorman | it looks like you can assign roles to users on domains (instead of projects) in v3, which is kind of blowing my mind a little bit. | 18:59 |
*** dprince has quit IRC | 19:00 | |
*** dprince has joined #puppet-openstack | 19:01 | |
*** tiswanso has quit IRC | 19:03 | |
richm | looking | 19:03 |
openstackgerrit | Alex Schultz proposed openstack/puppet-cinder: Add posibility to specify privileged user for Cinder https://review.openstack.org/219275 | 19:03 |
richm | kindjal: so you set domain_specific_drivers_enabled=True in keystone.conf? | 19:05 |
kindjal | yes | 19:05 |
mdorman | richm; yep, me too | 19:05 |
richm | kindjal: are you hitting https://bugs.launchpad.net/puppet-keystone/+bug/1485508 ? | 19:05 |
openstack | Launchpad bug 1485508 in puppet-keystone "when domain_specific_drivers_enabled=True keystone_user provider fails " [Undecided,In progress] - Assigned to Vasyl Saienko (vsaienko) | 19:05 |
kindjal | That looks to be it, yes. | 19:06 |
mdorman | richm: i bleliee that’s what i’m hitting too | 19:06 |
kindjal | I was really hoping someone would say, “Oh, you just have to set this other value over here…” | 19:08 |
mdorman | :) | 19:08 |
mdorman | it is never that simple!! | 19:08 |
mdorman | have to run right now, will check back in on this later today | 19:09 |
*** mdorman is now known as mdorman_away | 19:09 | |
richm | kindjal: mdorman_away: The patch looks good, but the author has some rspec debugging to do to make the tests pass, and the review hasn't been updated since Aug. 28 - https://review.openstack.org/#/c/213906/ | 19:11 |
*** aimon has quit IRC | 19:11 | |
*** aimon has joined #puppet-openstack | 19:12 | |
openstackgerrit | John Trowbridge proposed openstack/puppet-ironic: Add ironic-inspector support https://review.openstack.org/223690 | 19:12 |
kindjal | richm: Thanks, I’ll give it a shot | 19:12 |
EmilienM | vsaienko: ^ | 19:13 |
EmilienM | vsaienko: any chance to have a follow up? | 19:13 |
kindjal | richm: I think theh patchset is short one more thing... | 19:19 |
kindjal | Error: /Stage[main]/Nova::Keystone::Auth/Keystone::Resource::Service_identity[nova service, user nova]/Keystone_user[nova]: Could not evaluate: Execution of '/usr/bin/openstack token issue --format value' returned 1: ERROR: openstack Authentication cannot be scoped to multiple targets. Pick one of: project, domain or trust | 19:19 |
kindjal | That error follows: /usr/bin/openstack token issue --format value | 19:19 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: DNM - CI test https://review.openstack.org/223771 | 19:21 |
richm | kindjal: I believe that is coming from the keystone_user.password, to verify the user's password is correct | 19:21 |
richm | kindjal: it could mean that the user isn't a member of any projects, or that project lookup failed, and keystone_user.password was unable to get a domain scoped token to verify the password | 19:22 |
*** tiswanso has joined #puppet-openstack | 19:23 | |
richm | kindjal: not sure if running with debug could provide additional information | 19:23 |
kindjal | richm: If I have multi-domain support on, with domain specific features, and the feature here is LDAP identity backend… | 19:27 |
kindjal | then for keystone_user_role... | 19:27 |
kindjal | would I specify the user as $USER@$DOMAIN and not $USER@$PROJECT? | 19:28 |
kindjal | or, in other words, if I want user at domain to have role admin in project services… I’m not sure how to define that. | 19:29 |
richm | kindjal: keystone_user_role {'username::user_domain_name@project_name::project_domain_name', roles => ['admin'] } | 19:29 |
kindjal | thanks | 19:30 |
richm | kindjal: That also means you have to define your keystone_user and keystone_tenant like this: | 19:33 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-specs: Changes in enabling federation spec https://review.openstack.org/223777 | 19:33 |
richm | keystone_user { 'username::user_domain_name': ...} | 19:33 |
richm | keystone_tenant { 'project_name::project_domain_name': ...} | 19:33 |
iurygregory | richm, mfisch, EmilienM https://review.openstack.org/223777 when have time please take a look | 19:34 |
kindjal | Error: /Stage[main]/Main/Keystone_user_role[user::domain@project::domain]: Could not evaluate: undefined method `get_domain_id' for Puppet::Type::Keystone_user_role::ProviderOpenstack:Class | 19:34 |
kindjal | oh my fault | 19:37 |
kindjal | I missed some of the patch | 19:37 |
kindjal | richm: Thanks. I’ll keep playing with this. | 19:37 |
*** Thaxll has quit IRC | 19:49 | |
*** jpena has quit IRC | 19:49 | |
*** xingchao has joined #puppet-openstack | 19:52 | |
EmilienM | oc714: https://wiki.openstack.org/wiki/Puppet/Deploy#Single_node_deployment_scenario | 19:53 |
*** karume has joined #puppet-openstack | 19:56 | |
*** dprince has quit IRC | 19:56 | |
*** xingchao has quit IRC | 19:57 | |
*** stamak has joined #puppet-openstack | 19:58 | |
*** karume has quit IRC | 20:00 | |
*** kindjal has quit IRC | 20:11 | |
*** jayg is now known as jayg|g0n3 | 20:29 | |
*** morazi has quit IRC | 20:33 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: add Volume service https://review.openstack.org/223800 | 20:43 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-specs: Changes in enabling federation spec https://review.openstack.org/223777 | 20:43 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: add volume service https://review.openstack.org/223800 | 20:48 |
*** stamak has quit IRC | 20:51 | |
*** cdelatte has quit IRC | 20:51 | |
mwhahaha | EmilienM: should the puppet-unit tests properly pull in Depends-On references? https://review.openstack.org/#/c/219275/ | 20:52 |
EmilienM | mwhahaha: nope | 20:52 |
mwhahaha | boo :( | 20:52 |
EmilienM | it use .fixtures.yaml and not zuul cloner | 20:52 |
EmilienM | go ahead and patch rspec! :P | 20:53 |
EmilienM | I think we could trick this | 20:53 |
EmilienM | if we prepare the modules | 20:53 |
EmilienM | in a dir | 20:53 |
mwhahaha | i tested it locally like that | 20:54 |
EmilienM | mwhahaha: it would be a nice feature | 20:55 |
EmilienM | mwhahaha: crinkle already did the work for beaker jobs | 20:55 |
EmilienM | we could reuse the bits for unit | 20:55 |
mwhahaha | ok if i get some time maybe i'll take a look | 20:56 |
*** karume has joined #puppet-openstack | 20:56 | |
*** pradk has quit IRC | 20:59 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: add volume service https://review.openstack.org/223800 | 20:59 |
*** karume has quit IRC | 21:01 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: add volume service https://review.openstack.org/223800 | 21:05 |
*** tiswanso has quit IRC | 21:06 | |
mwhahaha | EmilienM: where are the beaker scripts? | 21:09 |
mwhahaha | or crinkle if you're around -^ | 21:12 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: add volume service https://review.openstack.org/223800 | 21:15 |
*** v1k0d3n has quit IRC | 21:31 | |
*** richm has quit IRC | 21:38 | |
*** aimon has quit IRC | 21:47 | |
openstackgerrit | Ivan Chavero proposed openstack/puppet-neutron: Split SR-IOV configuration file into driver and agent pieces https://review.openstack.org/221991 | 21:50 |
*** gfidente has quit IRC | 21:55 | |
*** richm has joined #puppet-openstack | 21:55 | |
EmilienM | mwhahaha: puppet-openstack-integration/install_modules.sh | 22:01 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: add volume service https://review.openstack.org/223800 | 22:04 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: add volume service https://review.openstack.org/223800 | 22:24 |
*** aimon has joined #puppet-openstack | 22:29 | |
*** ducttape_ has quit IRC | 22:32 | |
*** aimon_ has joined #puppet-openstack | 22:38 | |
*** aimon has quit IRC | 22:41 | |
*** karume has joined #puppet-openstack | 22:46 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-cinder: Make sure cinder is created before creating Cinder::Type https://review.openstack.org/223862 | 22:50 |
*** karume has quit IRC | 22:50 | |
*** gildub has joined #puppet-openstack | 22:51 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: add volume service https://review.openstack.org/223800 | 22:51 |
*** xingchao has joined #puppet-openstack | 22:54 | |
*** xingchao has quit IRC | 22:59 | |
*** markvoelker has quit IRC | 23:10 | |
*** xingchao has joined #puppet-openstack | 23:14 | |
*** openstackgerrit has quit IRC | 23:16 | |
*** openstackgerrit has joined #puppet-openstack | 23:17 | |
*** gildub has quit IRC | 23:24 | |
EmilienM | mtreinish: currently testing tempest/cinder - I have interesting results (see comment on) https://review.openstack.org/#/c/223800 | 23:26 |
*** sergmelikyan has quit IRC | 23:37 | |
*** mdorman_away has quit IRC | 23:48 | |
*** xingchao has quit IRC | 23:53 | |
*** markvoelker has joined #puppet-openstack | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!