openstackgerrit | Merged openstack/puppet-tripleo: Add ML2/OVN support for neutron profile https://review.openstack.org/314875 | 01:23 |
---|---|---|
openstackgerrit | zhongshengping proposed openstack/puppet-nova: Deprecate old auth parameters of ironic https://review.openstack.org/361906 | 01:30 |
*** saneax is now known as saneax-_-|AFK | 01:32 | |
*** chem has quit IRC | 01:37 | |
openstackgerrit | Merged openstack/puppet-tacker: Fix authtoken https://review.openstack.org/361558 | 01:40 |
*** ducttape_ has joined #puppet-openstack | 02:18 | |
openstackgerrit | Merged openstack/puppet-cinder: Add auth_strategy for cinder authtoken https://review.openstack.org/361470 | 02:32 |
openstackgerrit | Marcus Furlong proposed openstack/puppet-heat: add parameter to configure the name of the node https://review.openstack.org/362578 | 02:51 |
*** ducttape_ has quit IRC | 03:02 | |
*** tdb has quit IRC | 03:10 | |
*** tdb has joined #puppet-openstack | 03:15 | |
*** jperry has joined #puppet-openstack | 03:25 | |
*** ducttape_ has joined #puppet-openstack | 04:02 | |
*** ducttape_ has quit IRC | 04:09 | |
*** ducttape_ has joined #puppet-openstack | 04:13 | |
*** ducttape_ has quit IRC | 04:15 | |
*** ntpttr- has quit IRC | 04:36 | |
*** ntpttr- has joined #puppet-openstack | 04:40 | |
*** ntpttr has quit IRC | 04:41 | |
*** ntpttr- is now known as ntpttr | 04:41 | |
*** ntpttr__ has joined #puppet-openstack | 04:41 | |
*** ntpttr has quit IRC | 04:44 | |
*** ntpttr has joined #puppet-openstack | 04:45 | |
*** ayoung has quit IRC | 04:58 | |
*** jaosorior has joined #puppet-openstack | 05:10 | |
*** ducttape_ has joined #puppet-openstack | 05:16 | |
*** ducttape_ has quit IRC | 05:21 | |
*** skramaja has joined #puppet-openstack | 05:26 | |
*** agrebennikov has joined #puppet-openstack | 05:32 | |
*** richm has quit IRC | 05:38 | |
*** agrebennikov has quit IRC | 05:53 | |
*** ebalduf has quit IRC | 06:00 | |
*** jperry has quit IRC | 06:15 | |
*** jperry has joined #puppet-openstack | 06:16 | |
*** rcernin has joined #puppet-openstack | 06:19 | |
*** paramite has joined #puppet-openstack | 06:21 | |
*** leanderthal|afk is now known as leanderthal | 06:30 | |
*** jperry has quit IRC | 06:39 | |
*** saneax-_-|AFK is now known as saneax | 06:47 | |
*** karimb has joined #puppet-openstack | 06:48 | |
*** tesseract- has joined #puppet-openstack | 07:07 | |
*** jpena|off is now known as jpena | 07:12 | |
*** dtantsur|afk is now known as dtantsur | 07:13 | |
*** foxhunt_away is now known as foxhunt72 | 07:14 | |
*** ducttape_ has joined #puppet-openstack | 07:18 | |
*** rmart04 has joined #puppet-openstack | 07:20 | |
openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic: Move parameters from ironic::drivers::deploy to ironic::conductor https://review.openstack.org/361066 | 07:22 |
*** ifarkas_afk is now known as ifarkas | 07:22 | |
*** ducttape_ has quit IRC | 07:23 | |
openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic: Improve management of enabled driver's manifests and dependencies https://review.openstack.org/359138 | 07:38 |
openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic: [WIP] Support configuration and dependencies for iLO and Drac drivers https://review.openstack.org/359313 | 07:40 |
*** shardy has joined #puppet-openstack | 08:03 | |
openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic: Support configuration and dependencies for iLO and Drac drivers https://review.openstack.org/359313 | 08:03 |
dtantsur | morning folks! when you're here, could you please check https://review.openstack.org/#/c/354125/? we need that for ironic in overcloud pretty urgently. thanks! | 08:06 |
*** derekh has joined #puppet-openstack | 08:07 | |
openstackgerrit | Jiri Stransky proposed openstack/puppet-tripleo: Defaults to ceph_mon_node_ips and ceph_mon_node_names https://review.openstack.org/361108 | 08:07 |
*** karimb has quit IRC | 08:11 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Updated from Puppet OpenStack modules constraints https://review.openstack.org/362691 | 08:16 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL https://review.openstack.org/362692 | 08:17 |
*** cgfbee has quit IRC | 08:18 | |
*** ducttape_ has joined #puppet-openstack | 08:19 | |
*** ducttape_ has quit IRC | 08:23 | |
openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic: Move enabled_drivers to ironic::conductor manifest https://review.openstack.org/362157 | 08:24 |
openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic: Improve management of enabled driver's manifests and dependencies https://review.openstack.org/359138 | 08:24 |
openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic: Support configuration and dependencies for iLO and Drac drivers https://review.openstack.org/359313 | 08:25 |
*** gfidente has joined #puppet-openstack | 08:25 | |
*** cgfbee has joined #puppet-openstack | 08:25 | |
*** karimb has joined #puppet-openstack | 08:25 | |
openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic: Move enabled_drivers to ironic::conductor manifest https://review.openstack.org/362157 | 08:27 |
openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic: Improve management of enabled driver's manifests and dependencies https://review.openstack.org/359138 | 08:28 |
openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic: Support configuration and dependencies for iLO and Drac drivers https://review.openstack.org/359313 | 08:28 |
openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic: Add ironic::drivers::agent manifest https://review.openstack.org/362026 | 08:29 |
dtantsur | rebase party ^^^ :( | 08:29 |
*** _degorenko|afk is now known as degorenko | 08:32 | |
*** chem has joined #puppet-openstack | 08:40 | |
*** markd_ has joined #puppet-openstack | 08:40 | |
*** karimb has quit IRC | 08:43 | |
*** karimb has joined #puppet-openstack | 08:52 | |
*** karimb has quit IRC | 09:03 | |
*** jaosorior is now known as jaosorior_lunch | 09:08 | |
*** karimb has joined #puppet-openstack | 09:09 | |
*** ducttape_ has joined #puppet-openstack | 09:19 | |
*** ducttape_ has quit IRC | 09:24 | |
*** chem has quit IRC | 09:34 | |
*** chem has joined #puppet-openstack | 09:35 | |
*** jaosorior_lunch is now known as jaosorior | 09:42 | |
*** davideagle__ has joined #puppet-openstack | 09:47 | |
*** r2bit__ is now known as r2bit | 10:03 | |
*** r2bit has quit IRC | 10:04 | |
*** r2bit has joined #puppet-openstack | 10:04 | |
*** cgfbee has quit IRC | 10:10 | |
social | chem: https://review.openstack.org/#/c/354125/ puppy eyes | 10:12 |
*** richm has joined #puppet-openstack | 10:12 | |
* dtantsur adds his pair of puppy eyes to the same request | 10:16 | |
*** cgfbee has joined #puppet-openstack | 10:18 | |
*** ducttape_ has joined #puppet-openstack | 10:20 | |
*** sanjay_ has joined #puppet-openstack | 10:22 | |
sanjay_ | Hello all, I am trying to setup keystone using puppet manifest. Can anyone guide me how to do that from the scratch? I have already installed keystone using r10k | 10:23 |
*** mjblack has quit IRC | 10:23 | |
*** mjblack has joined #puppet-openstack | 10:25 | |
*** ducttape_ has quit IRC | 10:25 | |
*** ntpttr has quit IRC | 10:30 | |
*** paramite has quit IRC | 10:31 | |
*** ntpttr has joined #puppet-openstack | 10:35 | |
degorenko | sanjay_, you can find example here https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/keystone.pp | 10:42 |
sanjay_ | @degorenko: Thanks for the reply. I am using standalone mode, so can i run this manifest file directly? | 10:43 |
degorenko | sanjay_, yes, you can | 10:44 |
*** karimb has quit IRC | 10:46 | |
sanjay_ | Hello degorenko: I ran this manifest, it ran successfully but no installation is done, keystone config files are not present, also ports 35357 and 5000 are not in listening mode, | 10:47 |
sanjay_ | can you suggest something, please? | 10:47 |
degorenko | sanjay_, this manifest is only class declaration, you need to call this class or make include | 10:48 |
sanjay_ | okay, any example how can I do that? I am pretty new to puppet, sorry if asking stupid queries. But really need an example to proceed further..can you please guide me | 10:49 |
degorenko | sanjay_, take a look on fixtures folder | 10:49 |
sanjay_ | ok, thanks for this degorenko, i will check this out | 10:51 |
sanjay_ | @degorenko: I tried these, but finding this error: | 10:53 |
sanjay_ | Error: Could not find class ::openstack_integration for controller on node controller | 10:53 |
sanjay_ | i have tried copying scenario-aio.pp inside manifest directory, and run again but same result | 10:55 |
degorenko | sanjay_, because you need fully prepare openstack-integration module | 10:58 |
sanjay_ | Hello degorenko, can you please elaborate a bit more? i am trying to understand how i can deploy single-single module and later on built a manifest that creates controller node services for 1 machine and compute node services on other node to achieve 2 node automated setup | 11:01 |
sanjay_ | So i need to setup keystone initially, if I am able to do this, i can simulate others , that will help my cause | 11:02 |
degorenko | sanjay_, as you said, you prepared already keystone puppet module - you need to do same for openstack-integration module, as you did it for keystone | 11:02 |
sanjay_ | i am trying this from last few days now | 11:02 |
iberezovskiy_ | sanjay_, hi, could you please try example from acceptance tests https://github.com/openstack/puppet-keystone/blob/master/spec/acceptance/keystone_wsgi_apache_spec.rb#L8-L69 ? | 11:05 |
iberezovskiy_ | please make sure you have newton repos enabled | 11:05 |
sanjay_ | hello iberezovskiy: I am trying this, how can i enable repos for newton? | 11:06 |
iberezovskiy_ | e.g. include openstack_integration::repos https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/repos.pp | 11:07 |
sanjay_ | @degorenko: I tried installing module named openstack-integration but it says bad request: am i doing something else than expected? | 11:15 |
sanjay_ | @iberezovskiy: okay, so as i understand I need to run the manifest "keystone_wsgi_apache_spec.rb" and before that i need to run repos.pp with 'newton' as release name? | 11:17 |
degorenko | sanjay_, just prepare openstack-integration module in the same way, as you did it for keystone | 11:17 |
degorenko | keystone_wsgi_apache_spec.rb is not a puppet manifest | 11:17 |
degorenko | you need run only selected lines | 11:18 |
*** ducttape_ has joined #puppet-openstack | 11:21 | |
*** ducttape_ has quit IRC | 11:26 | |
*** jperry has joined #puppet-openstack | 11:26 | |
openstackgerrit | Merged openstack/puppet-openstack-integration: Updated from Puppet OpenStack modules constraints https://review.openstack.org/362691 | 11:33 |
*** morazi has joined #puppet-openstack | 11:47 | |
openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Default haproxy listeners activation on hiera service _enabled https://review.openstack.org/362798 | 11:52 |
*** jayg|g0n3 is now known as jayg | 11:56 | |
*** ilbot3 has quit IRC | 11:57 | |
*** jpena is now known as jpena|lunch | 12:00 | |
*** ansmith has joined #puppet-openstack | 12:02 | |
*** trown|outtypewww is now known as trown | 12:02 | |
*** pradk has joined #puppet-openstack | 12:02 | |
*** ilbot3 has joined #puppet-openstack | 12:03 | |
*** jaosorior has quit IRC | 12:04 | |
*** jaosorior has joined #puppet-openstack | 12:05 | |
*** ducttape_ has joined #puppet-openstack | 12:06 | |
*** fultonj has joined #puppet-openstack | 12:06 | |
EmilienM | hello | 12:09 |
*** jaosorior has quit IRC | 12:11 | |
*** jaosorior has joined #puppet-openstack | 12:12 | |
dtantsur | morning EmilienM | 12:12 |
openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Default haproxy listeners activation on hiera service _enabled https://review.openstack.org/362798 | 12:13 |
dtantsur | EmilienM, is it possible you get to that huge PXE patch https://review.openstack.org/#/c/354125/ today please? | 12:13 |
EmilienM | dtantsur: it has -1 | 12:14 |
dtantsur | heh, it got -1 while I was asking :) | 12:14 |
dtantsur | precise timing! ok, sorry for the noise then | 12:14 |
chem | dtantsur: EmilienM I've just put it ... just a small typo in a warning | 12:14 |
*** iurygregory has quit IRC | 12:15 | |
chem | social: I've reviewed https://review.openstack.org/#/c/354125/ and it has a typo in a warning and some other question | 12:15 |
* social looks | 12:16 | |
dtantsur | and the other question is pretty serious, good find | 12:16 |
dtantsur | I've just become aware that we configure TFTP twice: in xinet.d and in inspector dnsmasq | 12:16 |
dtantsur | we should only leave dnsmasq | 12:16 |
dtantsur | oops | 12:17 |
social | chem: yeah /o\ I think I git grep tftpboot so many times | 12:17 |
dtantsur | xinetd of course | 12:17 |
chem | social: I feel your pain, we can still make it happen today though. | 12:17 |
openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Add Ceph RGW listener to HAProxy https://review.openstack.org/362816 | 12:18 |
social | yeah so drop enable-tftp ant tftp-root from the dnsmasq? | 12:18 |
* social goes to read dnsmasq man | 12:18 | |
dtantsur | social, yes, drop both. xinetd alone should be enough | 12:19 |
* dtantsur has no idea how we ended up with both and how the hell all this even works | 12:19 | |
social | dtantsur: I should test this manually to see if it won't break ;.; | 12:20 |
dtantsur | :( | 12:20 |
*** karimb has joined #puppet-openstack | 12:21 | |
social | dtantsur: manifests/pxe.pp: content => 'r ^([^/]) /tftpboot/\1', | 12:21 |
social | dtantsur: so content => ".... ${tftp_root}/\1? | 12:21 |
dtantsur | oh yeah, one more place | 12:21 |
dtantsur | correct | 12:21 |
social | chem: the warning and space in other manifests are for standalone patches not related to pxe refactor | 12:24 |
*** danpawlik has joined #puppet-openstack | 12:25 | |
*** dprince has joined #puppet-openstack | 12:26 | |
*** ducttape_ has quit IRC | 12:32 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO to latest trunk https://review.openstack.org/362836 | 12:38 |
EmilienM | ok | 12:40 |
EmilienM | wrong window | 12:40 |
*** cdelatte has joined #puppet-openstack | 12:49 | |
*** markvoelker has joined #puppet-openstack | 12:52 | |
*** watanabe_isao has joined #puppet-openstack | 12:54 | |
openstackgerrit | Lukas Bezdicka proposed openstack/puppet-ironic: Introduce ironic::pxe class https://review.openstack.org/354125 | 12:58 |
*** jpena|lunch is now known as jpena | 12:59 | |
*** iurygregory has joined #puppet-openstack | 13:00 | |
iurygregory | morning | 13:00 |
EmilienM | iberezovskiy_: can you look https://review.openstack.org/#/c/362487/ please? | 13:02 |
EmilienM | and https://review.openstack.org/#/c/362423/ | 13:02 |
*** agrebennikov has joined #puppet-openstack | 13:05 | |
*** agrebennikov has quit IRC | 13:06 | |
iurygregory | degorenko, the reason to have auth_strategy in watcher https://review.openstack.org/#/c/356792/4/manifests/api.pp | 13:07 |
iberezovskiy_ | sure, looking | 13:09 |
iberezovskiy_ | both +A | 13:11 |
EmilienM | dtantsur: your changes at the end of cycle scare me to be honnest | 13:12 |
EmilienM | im ok to +2 | 13:12 |
EmilienM | but it still scare me to do that so late | 13:12 |
degorenko | iurygregory, such approach a little bit confusing me | 13:12 |
*** ducttape_ has joined #puppet-openstack | 13:12 | |
dtantsur | yeah, I get it... | 13:12 |
iurygregory | degorenko, do you know if Alex is around? | 13:13 |
dtantsur | EmilienM, well, feel free to stop approving at any point :) only the PXE thing is pretty important | 13:13 |
degorenko | iurygregory, probably PTO | 13:13 |
EmilienM | dtantsur: as long as all CI is green its fine by me | 13:13 |
dtantsur | thanks! | 13:13 |
*** jperry has quit IRC | 13:14 | |
iurygregory | degorenko, for new modules i think we should not have the condition to include since anyone use yet (I think) | 13:14 |
degorenko | iurygregory, well watcher is new module :) | 13:15 |
iurygregory | degorenko, yeah i know =) | 13:15 |
iurygregory | like cookiecutter when creating a new module we can forget in the review to use auth_strategy... | 13:15 |
*** markvoelker has quit IRC | 13:17 | |
*** ducttape_ has quit IRC | 13:17 | |
*** karimb has left #puppet-openstack | 13:23 | |
iurygregory | EmilienM, authtoken is finished https://bugs.launchpad.net/puppet-aodh/+bug/1604463/ \o/ sorry about the problems | 13:23 |
openstack | Launchpad bug 1604463 in puppet-swift "Modules should use keystone::resource::authtoken to configure keystone_authtoken settings" [Medium,Fix committed] - Assigned to Iury Gregory Melo Ferreira (iurygregory) | 13:23 |
EmilienM | iurygregory: don't be sorry | 13:24 |
EmilienM | i'll switch all our ci to use it and also all tripleo | 13:24 |
EmilienM | to make sure everything works fine | 13:24 |
iurygregory | EmilienM, if you need help o/ | 13:24 |
iberezovskiy_ | we are on the way to switch the fuel as well | 13:25 |
EmilienM | i would highly suggest to send an email over the ml about this change | 13:25 |
EmilienM | to explain what people should do in their manifests | 13:25 |
EmilienM | iberezovskiy_: excellent | 13:25 |
EmilienM | iurygregory: why not mistral? | 13:25 |
iurygregory | mistral still use old options and not new ones (username, etc) https://github.com/openstack/mistral/blob/c2424d6ccedc6d8a4448d78612a82521472443e1/mistral/utils/openstack/keystone.py | 13:26 |
EmilienM | let me poke people | 13:26 |
iurygregory | EmilienM, ack | 13:26 |
iurygregory | is the same case for sahara and murano i think | 13:27 |
*** dgurtner has joined #puppet-openstack | 13:36 | |
*** dgurtner has quit IRC | 13:36 | |
*** dgurtner has joined #puppet-openstack | 13:36 | |
*** markvoelker has joined #puppet-openstack | 13:40 | |
*** jperry has joined #puppet-openstack | 13:43 | |
*** Vinsh has quit IRC | 13:43 | |
*** fultonj has quit IRC | 13:44 | |
*** markvoelker has quit IRC | 13:45 | |
*** fultonj has joined #puppet-openstack | 13:46 | |
*** dfisher has joined #puppet-openstack | 13:46 | |
*** ayoung has joined #puppet-openstack | 13:52 | |
*** ducttape_ has joined #puppet-openstack | 13:52 | |
*** jperry has quit IRC | 13:55 | |
*** jperry has joined #puppet-openstack | 14:02 | |
openstackgerrit | Giulio Fidente proposed openstack/puppet-ceph: Add rgw::keystone::auth class https://review.openstack.org/362893 | 14:04 |
*** chem has quit IRC | 14:13 | |
*** chem has joined #puppet-openstack | 14:13 | |
*** ebalduf has joined #puppet-openstack | 14:14 | |
chem | social: he, you have some kind of weird character there Unrecognized escape sequence '\1' at /home/jenkins/workspace/gate-puppet-ironic-puppet-syntax-3-centos-7/manifests/pxe.pp:105:48 | 14:17 |
social | chem: I'm looking into that, I think I just have to escape it for older puppet versions | 14:18 |
chem | social:ack | 14:18 |
*** Vinsh has joined #puppet-openstack | 14:25 | |
*** dtantsur is now known as dtantsur|mtg | 14:28 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO to latest trunk https://review.openstack.org/362836 | 14:31 |
openstackgerrit | Merged openstack/puppet-heat: Configure clients_keystone with admin URL https://review.openstack.org/362487 | 14:32 |
openstackgerrit | Merged openstack/puppet-neutron: Deprecate allow_sorting and allow_pagination options https://review.openstack.org/361885 | 14:32 |
jaosorior | Hey guys, where is the keystone URL set for the provider that creates the endpoints and such? | 14:38 |
*** jaosorior is now known as jaosorior_away | 14:45 | |
degorenko | jaosorior_away, it tries to get openrc file first, then OS ENV and then takes from keystone.conf | 14:45 |
jaosorior_away | degorenko: what's the default place for the openrc file? | 14:46 |
degorenko | jaosorior_away, https://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack/auth.rb | 14:48 |
xarses | /root/openrc | 14:48 |
degorenko | ah, place on system :) right, /root/openrc | 14:49 |
jaosorior_away | xarses, degorenko: thanks... unfortunately it seems we're not using openrc O_o so gotta figure out what are we using in tripleo (hadn't had to dig this before) | 14:49 |
*** xarses has quit IRC | 14:51 | |
*** xarses has joined #puppet-openstack | 14:52 | |
*** dgurtner has quit IRC | 14:53 | |
jaosorior_away | now I see | 14:56 |
jaosorior_away | but it seems that if I want to enable SSL for keystone | 14:56 |
jaosorior_away | and it's running under apache | 14:56 |
jaosorior_away | I have to do it BOTH for the keystone manifest and the keystone::wsgi::apache | 14:57 |
*** AnarchyAo has joined #puppet-openstack | 14:57 | |
jaosorior_away | is that the case? | 14:57 |
EmilienM | https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-20160830 | 14:59 |
EmilienM | if anyone has topics for today | 14:59 |
EmilienM | otherwise well skip it | 14:59 |
*** davideagle__ has quit IRC | 15:01 | |
EmilienM | meeting cancelled! | 15:02 |
degorenko | iurygregory, you wanted discuss something? | 15:02 |
xarses | jaosorior_away: we don't do anything but configure the api endpoints, with http|s and enable ssl in the LB | 15:06 |
*** jperry has quit IRC | 15:06 | |
*** jperry has joined #puppet-openstack | 15:07 | |
jaosorior_away | xarses: if you're talking about tripleo, I'm working on enabling TLS from the LB to the service itself | 15:07 |
xarses | (fuel) | 15:07 |
*** chem has quit IRC | 15:08 | |
jaosorior_away | ah | 15:08 |
jaosorior_away | alright | 15:08 |
*** chem has joined #puppet-openstack | 15:08 | |
*** rmart04 has quit IRC | 15:10 | |
*** saneax is now known as saneax-_-|AFK | 15:16 | |
*** dtantsur|mtg is now known as dtantsur | 15:25 | |
*** leanderthal is now known as leanderthal|afk | 15:26 | |
*** nibalizer has quit IRC | 15:29 | |
*** xarses has quit IRC | 15:34 | |
*** foxhunt72 is now known as foxhunt_away | 15:35 | |
*** dtantsur is now known as dtantsur|afk | 15:42 | |
*** derekh has quit IRC | 15:46 | |
*** trown is now known as trown|lunch | 15:52 | |
*** ntpttr__ has quit IRC | 15:54 | |
*** ducttape_ has quit IRC | 15:55 | |
*** ducttape_ has joined #puppet-openstack | 15:56 | |
*** dprince has quit IRC | 15:59 | |
*** xarses has joined #puppet-openstack | 16:00 | |
*** dprince has joined #puppet-openstack | 16:00 | |
*** ifarkas is now known as ifarkas_afk | 16:01 | |
*** fvollero has joined #puppet-openstack | 16:07 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Revert "Revert "run_tests: disable TestEncryptedCinderVolumes tests"" https://review.openstack.org/363000 | 16:10 |
EmilienM | jpena: in the meantime ^ | 16:10 |
jpena | EmilienM: checking the patch, there's no release notes :D | 16:11 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO to latest trunk https://review.openstack.org/362836 | 16:11 |
EmilienM | jpena: sign | 16:11 |
EmilienM | sigh | 16:11 |
jpena | looking at the updated cinder package, I see now a key_manager and barbican sections (it was keymgr before) | 16:12 |
openstackgerrit | Keith Schincke proposed openstack/puppet-openstack-integration: Introduce scenario004 https://review.openstack.org/339106 | 16:12 |
EmilienM | i'll look into it after lunch | 16:12 |
EmilienM | in the meantime we want to promote | 16:12 |
EmilienM | jpena: thx! | 16:12 |
EmilienM | #fixed_key = <None> | 16:13 |
EmilienM | WTF | 16:13 |
EmilienM | jpena: we need to hardcode the key? lol. | 16:13 |
* jpena hopes not | 16:14 | |
openstackgerrit | Christian Schwede proposed openstack/puppet-swift: Fix middleware order of proxy pipeline and add missing modules https://review.openstack.org/363003 | 16:18 |
*** watanabe_isao has quit IRC | 16:18 | |
EmilienM | dtantsur|afk: boom | 16:21 |
*** davideagle_ has joined #puppet-openstack | 16:33 | |
iurygregory | degorenko, well the problem in auth_strategy for ironic::api.pp i think it's bad add the include for ironic... | 16:40 |
*** foxhunt_away is now known as foxhunt72 | 16:45 | |
degorenko | iurygregory, yes, that's why i dropped -1 on review :) | 16:47 |
iurygregory | degorenko, yeah I think need more feedback the auth_strategy we use doesn't set any value in configuration file.. | 16:48 |
degorenko | yeah | 16:49 |
iurygregory | EmilienM, can you give feedback in https://review.openstack.org/#/c/361553/? ^^ | 16:49 |
openstackgerrit | Javier Peña proposed openstack/puppet-cinder: Update Cinder encryption parameters https://review.openstack.org/363031 | 16:50 |
jpena | EmilienM: I think ^^ should fix the Cinder issue. | 16:51 |
*** jaosorior_away has quit IRC | 16:51 | |
*** trown|lunch is now known as trown | 16:52 | |
iurygregory | degorenko, can you +A https://review.openstack.org/#/c/361555/? | 16:52 |
*** pradk has quit IRC | 16:53 | |
*** pradk- has joined #puppet-openstack | 16:53 | |
*** gfidente has quit IRC | 16:54 | |
degorenko | iurygregory, done | 16:56 |
iurygregory | degorenko, tks | 16:57 |
*** jpena is now known as jpena|off | 16:58 | |
*** jperry has quit IRC | 17:00 | |
*** iberezovskiy_ is now known as iberezovskiy_off | 17:02 | |
openstackgerrit | Merged openstack/puppet-ironic: Move enabled_drivers to ironic::conductor manifest https://review.openstack.org/362157 | 17:02 |
*** degorenko is now known as _degorenko|afk | 17:02 | |
*** pradk- is now known as pradk | 17:03 | |
openstackgerrit | Merged openstack/puppet-octavia: Fix authtoken https://review.openstack.org/361555 | 17:11 |
*** rcernin has quit IRC | 17:12 | |
EmilienM | iurygregory: ok I will | 17:14 |
EmilienM | jpena|off: thx!! | 17:14 |
iurygregory | EmilienM, tks =) | 17:14 |
*** tesseract- has quit IRC | 17:14 | |
*** dgurtner has joined #puppet-openstack | 17:15 | |
EmilienM | iurygregory: really I have no thought on this patch | 17:22 |
EmilienM | i wonder which case we don't need keystone | 17:22 |
iurygregory | EmilienM, do you think is valid include ironic in api.pp just to make usage of auth_strategyfrom init? the inspector have auth_strategy too | 17:24 |
EmilienM | all of this is really complex | 17:24 |
EmilienM | i dont know anyone deploying ironic without keystone | 17:24 |
*** tesseract- has joined #puppet-openstack | 17:24 | |
EmilienM | do we really need this thing? | 17:24 |
iurygregory | if we want the include authtoken in the same pattern of other modules, yes | 17:25 |
openstackgerrit | Bradley Klein proposed openstack/puppet-monasca: Include ovs metrics when pruning non-infra projects https://review.openstack.org/363053 | 17:25 |
EmilienM | because we have it in other modules does not mean it's good | 17:26 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-guide: Move Meeting #91 to September https://review.openstack.org/363056 | 17:26 |
iurygregory | EmilienM, so i'll let ironic the way it is =) can be? | 17:27 |
*** tesseract- has quit IRC | 17:28 | |
EmilienM | iurygregory: probably | 17:28 |
iurygregory | EmilienM, tks for your time :D | 17:28 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO to latest trunk https://review.openstack.org/363064 | 17:36 |
*** foxhunt72 is now known as foxhunt_away | 17:38 | |
openstackgerrit | Merged openstack/puppet-openstack-guide: Move Meeting #91 to September https://review.openstack.org/363056 | 17:46 |
*** markvoelker has joined #puppet-openstack | 17:54 | |
*** davideagle_ has quit IRC | 17:54 | |
openstackgerrit | Keith Schincke proposed openstack/puppet-openstack-integration: Introduce scenario004 https://review.openstack.org/339106 | 17:59 |
*** alee has joined #puppet-openstack | 18:01 | |
alee | EmilienM, whats up? | 18:01 |
EmilienM | alee: hey | 18:01 |
EmilienM | alee: can you check https://review.openstack.org/363031 please? | 18:01 |
EmilienM | cinder broke again barbican things | 18:01 |
alee | EmilienM, ok - looking | 18:02 |
EmilienM | thx | 18:02 |
colonwq | EmilienM, Once 339106 is in. I will start on a gnoochi test if you want. | 18:02 |
EmilienM | alee: our ci failed to promote with latest cinder because of this | 18:03 |
EmilienM | colonwq: cool | 18:03 |
EmilienM | colonwq: once your patch is merged i'll enable voting on it | 18:03 |
alee | EmilienM, excellent -- its doing what its supposed to do .. | 18:03 |
EmilienM | and move it to gate | 18:03 |
EmilienM | alee: are we also supposed to configure fixed_key param? | 18:03 |
alee | EmilienM, back in a few mins.. need to grab a quick bite .. | 18:04 |
*** davideagle has joined #puppet-openstack | 18:05 | |
*** tesseract- has joined #puppet-openstack | 18:05 | |
*** tesseract- has quit IRC | 18:05 | |
*** tesseract- has joined #puppet-openstack | 18:06 | |
*** tesseract- has quit IRC | 18:08 | |
*** tesseract- has joined #puppet-openstack | 18:10 | |
*** ducttape_ has quit IRC | 18:18 | |
*** markvoelker has quit IRC | 18:22 | |
iurygregory | random question, anyone received answer from TSP? =( | 18:24 |
*** tesseract- has quit IRC | 18:30 | |
*** tesseract- has joined #puppet-openstack | 18:30 | |
*** tesseract- has quit IRC | 18:30 | |
*** tesseract- has joined #puppet-openstack | 18:31 | |
colonwq | EmilienM, does p-o-i register a Cinder endpoint for Openstack M? I was running into issues doing an "openstack volume create" but it could have been me. | 18:36 |
*** tesseract- has quit IRC | 18:37 | |
iurygregory | master p-o-i is for Newton, but it will create the endpoint | 18:38 |
*** davideagle has quit IRC | 18:38 | |
*** davideagle has joined #puppet-openstack | 18:40 | |
*** ducttape_ has joined #puppet-openstack | 18:41 | |
alee | EmilienM, I think the issue is simply that the wrong key manager class is being written in the puppet tests | 18:47 |
*** danpawlik_ has joined #puppet-openstack | 18:48 | |
*** delattec has joined #puppet-openstack | 18:48 | |
colonwq | iurygregory, thanks. I will look for the issue between the keyboard and chair. | 18:48 |
alee | EmilienM, that is -- https://review.openstack.org/#/c/363031 changes the way that the key manager parameters are written in the config file to conform with the castellan way of doing things | 18:49 |
alee | but it does so in a non-backwards compatible way | 18:49 |
*** saneax-_-|AFK is now known as saneax | 18:51 | |
*** cdelatte has quit IRC | 18:51 | |
*** jtomasek has quit IRC | 18:51 | |
EmilienM | colonwq: yes | 18:51 |
*** basilAB has quit IRC | 18:52 | |
*** _degorenko|afk has quit IRC | 18:52 | |
*** LarsErikP has quit IRC | 18:52 | |
*** hecliunyx has quit IRC | 18:52 | |
colonwq | EmilienM, Thanks. I'll look at what I did wrong. | 18:52 |
EmilienM | alee: this thins is a mess to be honnest | 18:52 |
EmilienM | alee: volume encryption is a mess | 18:52 |
*** LarsErikP has joined #puppet-openstack | 18:53 | |
openstackgerrit | Ivan Chavero proposed openstack/puppet-swift: Fix doc typho. https://review.openstack.org/363122 | 18:53 |
EmilienM | alee: what is we change the default api_class? | 18:53 |
*** davideagle has quit IRC | 18:54 | |
alee | EmilienM, so here is what happened .. in cinder, the way we used to define the keystore is using [keymgr] api_class="....." | 18:54 |
EmilienM | I'm going to disable this feature in our CI | 18:54 |
EmilienM | it's too unstable atm | 18:54 |
alee | EmilienM, no -- its easily fixed | 18:54 |
alee | EmilienM, we expected a change that needed to happen at some point to go to castellan | 18:55 |
alee | EmilienM, but its supposed to be backwards compatible. I suspect that without https://review.openstack.org/#/c/363031, it would have continued to work .. | 18:56 |
alee | EmilienM, though we'd be using the deprecated config | 18:57 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Finish to configure authtoken with modern parameters https://review.openstack.org/363129 | 18:57 |
EmilienM | alee: it doesn't work | 18:57 |
EmilienM | without | 18:57 |
alee | EmilienM, so if we keep https://review.openstack.org/#/c/363031, we simply need to change the barbican class | 18:57 |
*** davideagle has joined #puppet-openstack | 18:58 | |
*** hecliunyx has joined #puppet-openstack | 18:58 | |
alee | EmilienM, which would be here .. | 18:58 |
EmilienM | alee: see https://review.openstack.org/#/c/362836/2 | 18:58 |
EmilienM | alee: see logs at PS2 | 18:58 |
EmilienM | alee: tests are failing | 18:58 |
EmilienM | alee: keymgr_api_class you mean? | 18:59 |
alee | yup | 18:59 |
EmilienM | if we set the correct keymgr_api_class in puppet cinder? | 18:59 |
EmilienM | does it work with old packaging? | 18:59 |
EmilienM | the prob is that we override it in p-o-i | 18:59 |
EmilienM | let's disable the test first | 18:59 |
alee | EmilienM, https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/cinder.pp#L66 | 18:59 |
EmilienM | we'll reenable it right after | 18:59 |
alee | EmilienM, sure | 19:00 |
*** jperry has joined #puppet-openstack | 19:00 | |
*** basilAB has joined #puppet-openstack | 19:01 | |
EmilienM | alee: and should I do this ? https://review.openstack.org/#/c/363064/1/manifests/cinder.pp | 19:01 |
alee | EmilienM, but yeah, it just needs to be set to "castellan.key_manager.barbican_key_manager.BarbicanKeyManager" | 19:01 |
alee | in p-o-i | 19:01 |
alee | exactly | 19:02 |
EmilienM | let's try again | 19:02 |
*** jtomasek has joined #puppet-openstack | 19:03 | |
alee | EmilienM, so you need that change + the puppet-cinder change | 19:03 |
EmilienM | yes | 19:04 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-cinder: Update Cinder encryption parameters https://review.openstack.org/363031 | 19:04 |
alee | fingers crossed .. | 19:04 |
*** _degorenko|afk has joined #puppet-openstack | 19:06 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO to latest trunk https://review.openstack.org/363141 | 19:08 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Enable Volume encryption testing again https://review.openstack.org/363142 | 19:08 |
EmilienM | alee: yeah let's try ^ | 19:08 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Enable Volume encryption testing again https://review.openstack.org/363142 | 19:09 |
*** davideagle has quit IRC | 19:13 | |
iurygregory | colonwq, good luck o/ if need anything help we try to help if possible =) | 19:14 |
EmilienM | colonwq: don't forget to "check experimental" everytime you commit on this patch. | 19:18 |
EmilienM | colonwq: your poi patch doesn't work | 19:21 |
EmilienM | http://logs.openstack.org/06/339106/34/experimental/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7-nv/fe965fa/console.html | 19:22 |
EmilienM | second puppet run is not idempotent | 19:22 |
EmilienM | colonwq: I really have no time to work on it but something is wrong in puppet ceph | 19:23 |
colonwq | EmilienM, I'll take a look at the logs and see what needs to be updated | 19:33 |
EmilienM | colonwq: I'm working on a fix... | 19:35 |
*** dprince has quit IRC | 19:35 | |
colonwq | EmilienM, ok is the lines here? http://logs.openstack.org/06/339106/34/experimental/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7-nv/fe965fa/console.html#_2016-08-30_19_16_34_856769 | 19:37 |
colonwq | Three SELinux lines and a reload command? | 19:37 |
EmilienM | yes | 19:37 |
*** foxhunt_away is now known as foxhunt72 | 19:39 | |
*** rcernin has joined #puppet-openstack | 19:39 | |
EmilienM | colonwq: it should do the job but not sure 100% https://review.openstack.org/363164 | 19:41 |
openstackgerrit | Merged openstack/puppet-openstack-integration: Revert "Revert "run_tests: disable TestEncryptedCinderVolumes tests"" https://review.openstack.org/363000 | 19:42 |
colonwq | EmilienM, does the selinux_ignore_defaults make it not relabel the files and semanage handle it? | 19:42 |
EmilienM | colonwq: yes. | 19:43 |
colonwq | EmilienM, OK. that is a good tool to know. | 19:43 |
openstackgerrit | Martin André proposed openstack/puppet-tripleo: Manage tripleo-ui configuration files with puppet https://review.openstack.org/363167 | 19:43 |
colonwq | The reload command is needed only for hammer on centos/rhel 7. Is there a time for when we do not want to support that? | 19:44 |
EmilienM | colonwq: ok please remove it then | 19:44 |
EmilienM | I'll let you take over the patch | 19:45 |
EmilienM | please remove the reload thing in the same patch | 19:45 |
EmilienM | rebase your rgw keystone on top of it | 19:45 |
EmilienM | and recheck poi. ok? | 19:45 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Add puppet-panko in OpenStack modules list https://review.openstack.org/363170 | 19:46 |
colonwq | Ok. Ill submit 363000 for review in a couple of mins. | 19:46 |
EmilienM | colonwq: not this one | 19:47 |
EmilienM | https://review.openstack.org/363164 | 19:47 |
colonwq | that explains why I did not see it in my review list. | 19:47 |
*** dprince has joined #puppet-openstack | 19:50 | |
openstackgerrit | Keith Schincke proposed openstack/puppet-ceph: Make RGW deployment idempotent https://review.openstack.org/363164 | 19:53 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: midonet: update neutron parameter https://review.openstack.org/363178 | 19:59 |
openstackgerrit | Keith Schincke proposed openstack/puppet-openstack-integration: Introduce scenario004 https://review.openstack.org/339106 | 20:00 |
*** shardy has quit IRC | 20:02 | |
*** shardy has joined #puppet-openstack | 20:05 | |
*** danpawlik_ has quit IRC | 20:06 | |
EmilienM | colonwq: like I said... you need to rebase https://review.openstack.org/#/c/347956/ on top of my patch | 20:06 |
EmilienM | or the CI won't pass in poi scenario004 | 20:07 |
colonwq | ok. let me do that. | 20:11 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3 https://review.openstack.org/347956 | 20:11 |
*** pradk has quit IRC | 20:11 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Introduce scenario004 https://review.openstack.org/339106 | 20:12 |
EmilienM | colonwq: done | 20:12 |
EmilienM | colonwq: sorry | 20:12 |
EmilienM | I didn't see your message | 20:12 |
colonwq | EmilienM, No problem. You are faster than I can learn. I have been rebasing to master instead of a patch. | 20:12 |
EmilienM | colonwq: use Gerrit UI | 20:12 |
EmilienM | you can rebase in one click. | 20:12 |
iurygregory | best button in the UI | 20:13 |
colonwq | EmilienM, that is so much easier than by hand. | 20:13 |
*** pradk has joined #puppet-openstack | 20:16 | |
*** jperry has quit IRC | 20:16 | |
*** jperry has joined #puppet-openstack | 20:17 | |
*** ducttape_ has quit IRC | 20:19 | |
*** ducttape_ has joined #puppet-openstack | 20:22 | |
*** dgurtner has quit IRC | 20:22 | |
*** jayg is now known as jayg|g0n3 | 20:22 | |
*** saneax is now known as saneax-_-|AFK | 20:27 | |
alee | EmilienM, failed - but better -- looks like the barbican url can be versionless now .. | 20:33 |
*** rcernin has quit IRC | 20:33 | |
EmilienM | that thing is a nightmare | 20:33 |
*** rcernin has joined #puppet-openstack | 20:33 | |
alee | EmilienM, https://review.openstack.org/#/c/363064/1/manifests/cinder.pp | 20:34 |
alee | line 66 - remove the /v1 | 20:34 |
alee | and it should be good to go. | 20:34 |
alee | EmilienM, I see where its trying to get to /v1/v1 .. | 20:34 |
alee | EmilienM, this is actually better | 20:35 |
*** trown is now known as trown|outtypewww | 20:35 | |
EmilienM | sure lol | 20:36 |
EmilienM | it's better :) | 20:36 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Enable Volume encryption testing again https://review.openstack.org/363142 | 20:36 |
*** ansmith has quit IRC | 20:41 | |
*** foxhunt72 is now known as foxhunt_away | 20:49 | |
*** shardy has quit IRC | 20:53 | |
*** chem has quit IRC | 20:57 | |
*** alee has quit IRC | 21:04 | |
*** dprince has quit IRC | 21:08 | |
*** ayoung has quit IRC | 21:19 | |
*** morazi has quit IRC | 21:26 | |
EmilienM | colonwq: http://logs.openstack.org/06/339106/36/experimental/gate-puppet-openstack-integration-3-scenario004-tempest-centos-7-nv/3d10a4a/console.html#_2016-08-30_20_42_58_357667 | 21:26 |
EmilienM | rgw doesn't start | 21:26 |
EmilienM | colonwq: http://logs.openstack.org/06/339106/36/experimental/gate-puppet-openstack-integration-3-scenario004-tempest-centos-7-nv/3d10a4a/logs/ceph/radosgw.txt.gz | 21:27 |
EmilienM | Keystone token parse error: missing mandatory field access | 21:27 |
iurygregory | its a little strange because the ceph configuration have all parameters.. | 21:48 |
iurygregory | rgw_keystone_admin_password = secret | 21:48 |
iurygregory | rgw_keystone_admin_user = rgwuser | 21:48 |
iurygregory | rgw_keystone_admin_project = services | 21:48 |
iurygregory | rgw_keystone_admin_domain = default | 21:48 |
iurygregory | rgw_keystone_url = https://127.0.0.1:35357 | 21:48 |
iurygregory | rgw_keystone_api_version = 3 | 21:48 |
iurygregory | not sure if rgw_keystone_admin_domain want id or name information, if is name the value should be Default | 21:49 |
*** davideagle has joined #puppet-openstack | 21:49 | |
xarses | iurygregory: is accepted roles present? | 21:55 |
iurygregory | xarses, yes | 21:55 |
iurygregory | rgw_keystone_accepted_roles = admin,Member | 21:56 |
iurygregory | http://logs.openstack.org/06/339106/36/experimental/gate-puppet-openstack-integration-3-scenario004-tempest-centos-7-nv/3d10a4a/logs/etc/ceph/ceph.conf.txt.gz | 21:56 |
iurygregory | it's correct have this in the same line: rgw_frontends = civetweb port=8080 | 21:57 |
iurygregory | O.o | 21:57 |
*** alee has joined #puppet-openstack | 21:58 | |
xarses | can't bind to 8080 | 21:58 |
xarses | on xenial | 21:59 |
xarses | and it complains about signing | 21:59 |
*** jperry has quit IRC | 22:04 | |
*** davideagle has quit IRC | 22:06 | |
xarses | keystone is supposed to be https? | 22:09 |
xarses | iurygregory: ya, it seems to read it correctly | 22:09 |
openstackgerrit | Pradeep Kilambi proposed openstack/puppet-panko: Implement Panko Api service https://review.openstack.org/362305 | 22:28 |
*** ducttape_ has quit IRC | 22:34 | |
*** ansmith has joined #puppet-openstack | 22:59 | |
*** ayoung has joined #puppet-openstack | 22:59 | |
*** pradk has quit IRC | 23:00 | |
*** ducttape_ has joined #puppet-openstack | 23:03 | |
*** johnmilton has quit IRC | 23:06 | |
*** ducttape_ has quit IRC | 23:09 | |
*** saneax-_-|AFK is now known as saneax | 23:12 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: cinder: disable barbican https://review.openstack.org/363309 | 23:17 |
*** Vinsh has quit IRC | 23:17 | |
*** davideagle has joined #puppet-openstack | 23:17 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-cinder: Update Cinder encryption parameters https://review.openstack.org/363031 | 23:18 |
*** xarses has quit IRC | 23:18 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Enable Volume encryption testing again https://review.openstack.org/363142 | 23:18 |
EmilienM | yeah it's a mess | 23:18 |
*** dfisher has quit IRC | 23:28 | |
*** rcernin has quit IRC | 23:35 | |
*** Vinsh has joined #puppet-openstack | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!