Tuesday, 2016-03-01

« Monday, 2016-02-29 Index Wednesday, 2016-03-02 »
*** ducttape_ has quit IRC00:01
*** furlongm has joined #puppet-openstack00:17
*** vinsh has joined #puppet-openstack00:20
*** vinsh_ has quit IRC00:20
*** _joes_ has quit IRC00:20
*** _joes_ has joined #puppet-openstack00:21
*** vinsh has quit IRC00:23
*** davideagle has joined #puppet-openstack00:25
*** ducttape_ has joined #puppet-openstack00:33
*** markvoelker has quit IRC00:38
*** davideagle has quit IRC00:39
*** saneax is now known as saneax_AFK00:40
*** ducttape_ has quit IRC00:45
*** xarses has quit IRC01:05
*** ducttape_ has joined #puppet-openstack01:11
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: IPv6: duak-stack support for Keystone  https://review.openstack.org/28634401:12
*** angdraug has quit IRC01:20
*** xarses has joined #puppet-openstack01:37
*** markvoelker has joined #puppet-openstack01:38
*** ducttape_ has quit IRC01:57
*** markvoelker has quit IRC02:08
*** markvoelker has joined #puppet-openstack02:08
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: IPv6: duak-stack support for Keystone  https://review.openstack.org/28634402:15
*** ducttape_ has joined #puppet-openstack02:20
*** baker has joined #puppet-openstack03:20
*** dhollinger has joined #puppet-openstack03:21
*** dhollinger has quit IRC03:21
*** richm has quit IRC03:37
openstackgerritMatt Fischer proposed openstack/puppet-keystone: WIP Keystone hooks support  https://review.openstack.org/28389803:40
openstackgerritMerged openstack/puppet-keystone: stop managing files that the package makes  https://review.openstack.org/28330803:41
*** saneax_AFK is now known as saneax04:06
*** baker has quit IRC04:09
*** ducttape_ has quit IRC04:22
*** fedexo has joined #puppet-openstack04:59
openstackgerritMatt Fischer proposed openstack/puppet-keystone: WIP Keystone hooks support  https://review.openstack.org/28389805:03
*** jaosorior has joined #puppet-openstack05:49
*** rcernin has joined #puppet-openstack06:09
*** trown|outtypewww has quit IRC06:34
*** trown has joined #puppet-openstack06:35
*** _matthias_ has joined #puppet-openstack07:32
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Always override X-Forwarded-Proto header for Heat  https://review.openstack.org/28642007:36
*** bnemec has quit IRC07:36
*** bnemec has joined #puppet-openstack07:39
*** hjensas has joined #puppet-openstack07:48
*** hjensas has joined #puppet-openstack07:48
*** jtomasek has joined #puppet-openstack07:50
*** dgurtner has joined #puppet-openstack07:58
*** xingchao has joined #puppet-openstack08:00
*** foxhunt_away is now known as foxhunt7208:05
openstackgerritNikolay Starodubtsev proposed openstack/puppet-murano: Switch murano to use keystone API v3 by default  https://review.openstack.org/27944408:10
*** paramite has joined #puppet-openstack08:18
*** jpena has joined #puppet-openstack08:21
*** xingchao has quit IRC08:23
*** xingchao has joined #puppet-openstack08:24
*** paramite is now known as paramite|afk08:32
*** paramite|afk is now known as paramite08:37
*** openstackgerrit has quit IRC08:48
*** openstackgerrit has joined #puppet-openstack08:48
*** xingchao_ has joined #puppet-openstack08:53
*** xingchao has quit IRC08:53
*** paramite is now known as paramite|afk08:55
*** paramite|afk is now known as paramite08:57
*** xingchao_ has quit IRC08:57
*** gfidente has joined #puppet-openstack08:58
*** xingchao has joined #puppet-openstack08:58
*** fedexo has quit IRC08:58
*** _degorenko|afk is now known as degorenko09:02
*** xingchao has quit IRC09:10
*** xingchao has joined #puppet-openstack09:12
*** xingchao has quit IRC09:12
*** xingchao has joined #puppet-openstack09:13
*** jistr has joined #puppet-openstack09:14
*** xingchao has quit IRC09:16
*** xingchao has joined #puppet-openstack09:18
*** igajsin1 has left #puppet-openstack09:20
*** xingchao has quit IRC09:23
*** ddmitriev has joined #puppet-openstack09:30
*** davideagle has joined #puppet-openstack09:34
*** derekh has joined #puppet-openstack09:39
*** xingchao has joined #puppet-openstack09:46
*** aarefiev has quit IRC09:48
*** aarefiev has joined #puppet-openstack09:49
openstackgerritXingchao Yu proposed openstack/puppet-oslo: Add oslo.middleware defines  https://review.openstack.org/27442209:56
*** davideagle has quit IRC10:10
*** xingchao has quit IRC10:13
*** dgurtner has quit IRC10:26
zigoEmilienM: That's off-topic for this channel, but Debconf 17 will be in Montreal ! :)10:35
zigohttps://wiki.debconf.org/wiki/DebConf17/Bids/Montreal10:35
zigoI'll get to see how your place is... :P10:36
-openstackstatus- NOTICE: Gerrit is going to be restarted due to poor performance10:38
*** ChanServ changes topic to "Gerrit is going to be restarted due to poor performance"10:38
*** ChanServ changes topic to "Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet"10:44
-openstackstatus- NOTICE: gerrit finished restartign10:44
openstackgerritNikolay Starodubtsev proposed openstack/puppet-murano: Switch murano to use keystone API v3 by default  https://review.openstack.org/27944410:50
*** davideagle has joined #puppet-openstack10:54
openstackgerritMerged openstack/puppet-heat: Implement crontab to purge deleted data  https://review.openstack.org/28629011:00
*** davideagle has quit IRC11:11
openstackgerritMerged openstack/puppet-neutron: Neutron subnet will now be able to update  https://review.openstack.org/28551811:37
*** davideagle has joined #puppet-openstack12:00
EmilienMgood morning12:01
EmilienMzigo: cool12:01
*** johnmilton has joined #puppet-openstack12:02
openstackgerritNikolay Starodubtsev proposed openstack/puppet-murano: Switch murano to use keystone API v3 by default  https://review.openstack.org/27944412:09
*** dgurtner has joined #puppet-openstack12:10
*** davideagle has quit IRC12:14
EmilienMdegorenko: I replied to https://review.openstack.org/#/c/285586/ a bit late but I did :)12:15
openstackgerritBabu Shanmugam proposed openstack/puppet-vswitch: Added custom type vs_config and OVS provider for it  https://review.openstack.org/28599612:16
*** leanderthal|afk is now known as leanderthal12:21
*** saneax is now known as saneax_AFK12:22
degorenkoEmilienM, no problem :)12:28
*** davideagle has joined #puppet-openstack12:29
*** jaosorior has quit IRC12:36
*** jaosorior has joined #puppet-openstack12:37
*** davideagle has quit IRC12:45
*** davideagle has joined #puppet-openstack12:48
*** ducttape_ has joined #puppet-openstack13:05
*** davideagle has quit IRC13:07
*** davideagle has joined #puppet-openstack13:10
*** paramite is now known as paramite|afk13:13
*** Aegil has quit IRC13:15
*** furlongm has quit IRC13:16
*** furlongm has joined #puppet-openstack13:21
*** davideagle has quit IRC13:23
*** jayg|g0n3 is now known as jayg13:23
*** xingchao has joined #puppet-openstack13:25
*** dprince has joined #puppet-openstack13:25
*** saneax_AFK is now known as saneax13:29
*** ducttape_ has quit IRC13:31
*** jpena is now known as jpena|lunch13:31
*** davideagle has joined #puppet-openstack13:32
openstackgerritNikolay Starodubtsev proposed openstack/puppet-murano: Switch murano to use keystone API v3 by default  https://review.openstack.org/27944413:34
*** richm has joined #puppet-openstack13:36
*** kashyap has joined #puppet-openstack13:37
*** furlongm_ has joined #puppet-openstack13:38
*** davideagle has quit IRC13:39
kashyapEmilienM: I'm looking at this change - https://review.openstack.org/#/c/286584/2/puppet/manifests/overcloud_compute.pp13:39
*** paramite|afk is now known as paramite13:39
EmilienMkashyap: hey! you here13:39
kashyapEmilienM: Aren't you doing two unrelated changes?13:39
kashyapYeah, just joined here to quickly ACK it, while it's fresh in memory13:39
EmilienMkashyap: no I moved the nova_config so I can reuse rbd_ephemeral_storage boolean13:39
EmilienMkashyap: in puppet, order matters13:40
kashyapEmilienM: Duh, I just needed to scroll down!13:40
kashyapI'll add a comment to your question there.13:40
EmilienMkashyap: do I need to configure live_migration_flag too ?13:40
EmilienMthanks!13:40
kashyapEmilienM: Well, for now, yes.  In future, it'll all be handled with a single tri-state flag.13:40
*** furlongm has quit IRC13:40
EmilienMok13:40
kashyapI'll add the details there for reference.13:40
EmilienMok, great, I'll iterate after that13:41
*** jtomasek_ has joined #puppet-openstack13:43
kashyapEmilienM: One more: You special-cased RBD, and used TUNNELLED for 'block_migration_flag', any reason?13:46
EmilienMkashyap: because of https://bugzilla.redhat.com/show_bug.cgi?id=1301986#c1213:46
openstackbugzilla.redhat.com bug 1301986 in rhel-osp-director "Block based migration fails with unable to execute QEMU command 'migrate': this feature or command is not currently supported" [High,New] - Assigned to hbrock13:46
*** cdelatte has joined #puppet-openstack13:46
*** fwdit has joined #puppet-openstack13:47
kashyapEmilienM: Oh, yes, that's the correct rationale.13:48
*** kashyap has left #puppet-openstack13:49
*** jtomasek_ has quit IRC13:54
*** rmart04 has joined #puppet-openstack13:56
EmilienMupstream meeting starts in one hour, feel free to add topics / bugs / patches: https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-2016030114:03
*** derekh is now known as ndipanov_14:04
*** ndipanov_ is now known as derekh14:04
*** mdorman has joined #puppet-openstack14:05
*** furlongm_ is now known as furlongm14:09
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: [inspector] use dnsmasq dhcp-sequential-ip flag in dnsmasq.conf  https://review.openstack.org/28661714:14
mfischEmilienM: that steverdore change broke me it seems14:15
mfischI'm going to have to put the long names back in hiera14:15
EmilienMmfisch: what version are you running?14:15
mfischstill an old copy of L14:15
mfischL before release really14:15
EmilienMand you running master?14:15
EmilienM(puppet-keystone)14:16
mfischI think this is stevedore issue14:16
mfischValueError: Empty module name14:16
mfischyeah, well im in process of trying the new module14:16
openstackgerritMerged openstack/puppet-openstack-integration: Bump RDO repo to February 29th trunk  https://review.openstack.org/28625814:16
EmilienMmfisch: the upstream doc and keystone folks says "use stevedore names"14:17
EmilienMmfisch: please file a bug, with logs, etc14:17
mfischonce I pin this down I will let you know, this is a test merge from about 3 weeks of stuff14:17
EmilienMmfisch: but you have hit this issue because you run on Liberty while our module is for master (mitaka)14:17
mfischare the docs you found for M?14:18
mfischif this is really for M then I'll work around it14:18
EmilienMmfisch: I don't know details14:18
EmilienMi looked at doc and how devstack works14:18
*** paramite is now known as paramite|afk14:21
mfischI'll dig into it today14:22
*** paramite|afk is now known as paramite14:22
mfischyep thats it14:23
mfischEmilienM: I fixed with hiera, good to know I can clean up in Mitaka14:23
EmilienMcool14:24
mfischkeystone::token_driver: 'keystone.token.persistence.backends.sql.Token'14:24
mfischkeystone::catalog_driver: 'keystone.catalog.backends.sql.Catalog'14:24
*** furlongm_ has joined #puppet-openstack14:25
*** furlongm has quit IRC14:26
openstackgerritNikolay Starodubtsev proposed openstack/puppet-murano: Switch murano to use keystone API v3 by default  https://review.openstack.org/27944414:27
*** vinsh has joined #puppet-openstack14:28
*** jpena|lunch is now known as jpena14:32
*** _ody has quit IRC14:40
*** dfisher has joined #puppet-openstack14:44
*** furlongm_ has quit IRC14:44
openstackgerritIvan Berezovskiy proposed openstack/puppet-ceilometer: Deprecate ceilometer alarm classes  https://review.openstack.org/28225814:46
*** furlongm_ has joined #puppet-openstack14:47
*** _ody has joined #puppet-openstack14:56
*** skolekonov_ has joined #puppet-openstack14:57
*** ducttape_ has joined #puppet-openstack14:58
openstackgerritMartin Millnert proposed openstack/puppet-horizon: Avoid causing redirect loop  https://review.openstack.org/28664715:08
*** skolekonov_ has quit IRC15:11
EmilienMmwhahaha: can you look https://review.openstack.org/285586 if you have time today?15:13
mwhahahasure15:14
*** xarses has quit IRC15:14
*** fwdit has quit IRC15:23
openstackgerritMartin Millnert proposed openstack/puppet-keystone: Make apache::vhost::headers configuration possible  https://review.openstack.org/28665615:25
*** haplo37 has joined #puppet-openstack15:27
*** vinsh has quit IRC15:32
mfischEmilienM: sorry I had a commute this morning15:32
*** vinsh has joined #puppet-openstack15:33
EmilienMmfisch: sorry for what?15:34
EmilienMah for the meeting, no worries, we had 0 item15:35
EmilienMmfisch: though you and chem have an action that is postponed for weeks15:35
EmilienMmaybe we can work on it soon15:35
*** jpena is now known as jpena|brb15:42
*** dhollinger has joined #puppet-openstack15:45
*** delattec has joined #puppet-openstack15:55
*** cdelatte has quit IRC15:55
*** xarses has joined #puppet-openstack16:02
*** xarses has quit IRC16:02
*** xarses has joined #puppet-openstack16:03
*** haplo37 has quit IRC16:04
*** jaosorior is now known as jaosorior_away16:06
*** rochaporto has joined #puppet-openstack16:06
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: IPv6: duak-stack support for Keystone  https://review.openstack.org/28634416:12
*** xingchao has quit IRC16:13
*** xingchao has joined #puppet-openstack16:14
*** paramite has quit IRC16:14
*** xingchao has quit IRC16:14
*** haplo37 has joined #puppet-openstack16:16
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: IPv6: duak-stack support for Keystone  https://review.openstack.org/28634416:17
*** leanderthal is now known as leanderthal|afk16:19
*** vinsh_ has joined #puppet-openstack16:29
*** vinsh has quit IRC16:29
*** jpena|brb is now known as jpena16:30
xarsesdmsimard: thanks for the laugh16:45
dmsimardxarses: oh ?16:46
xarseshttps://github.com/openstack/keystone/commit/7c6556ce18eb170fd8102168b33a50272b22ec5a16:46
dmsimardyeah16:46
dmsimardlol16:46
*** rochaporto has quit IRC16:46
xarsesthe tests fail 1 day, every 4 years16:46
xarsesoh, some one just mentioned that horizon had a leap year bug too16:47
dmsimardthis is almost y2k all over again16:49
EmilienMskolekonov: have you seen http://logs.openstack.org/86/285586/20/gate/gate-puppet-openstack-integration-scenario001-tempest-dsvm-trusty/bc7ce9f/logs/neutron/neutron-server.txt.gz#_2016-03-01_15_32_37_313 ?16:51
EmilienMsame question for http://logs.openstack.org/86/285586/20/gate/gate-puppet-openstack-integration-scenario001-tempest-dsvm-trusty/bc7ce9f/logs/neutron/neutron-server.txt.gz#_2016-03-01_15_32_37_31016:51
*** rcernin has quit IRC16:55
openstackgerritDenis Egorenko proposed openstack/puppet-heat: test do not merge  https://review.openstack.org/28671717:04
mfischEmilienM: the merge of removing /etc/keystone broke fernet tokens17:05
*** jistr has quit IRC17:05
mfischthe resource dep isnt working properly17:05
mfischI'm working on it17:05
dmsimardmfisch: thank you third party CI17:05
EmilienMlol17:05
EmilienMif I could, I would put mfisch in our CI jobs17:06
mfischyou mean my lab?17:06
mfischlol17:07
EmilienMyeah!17:10
mfischEmilienM: I dont get how key-manage fernet ran before keystone was installed17:10
mfischhttps://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L915-L92017:11
mfischthat look right to you?17:11
mfischhere's what my CI says: http://paste.openstack.org/show/488796/17:11
EmilienMmfisch: in your catalog, do you see the pkg installed after or before fernet?17:12
mfischoddly im not seeing it installed in this run17:14
mfischhmm17:14
EmilienMmfisch: maybe can you submit a patch in our CI to test it?17:14
mfischyeah17:14
EmilienMyou proposed to have a scenario deploying it17:14
EmilienMmfisch: please submit a patch, or at least a PoC, that we can finish but I have 0 experience in fernet17:15
mfischthis is our issue17:15
mfischbut I Think upstream should make the fernet keys folder17:15
mfischEmilienM: can you make a bug to add a CI test for fernet?17:16
mfischthe code there in upstream is not 100% thre17:16
mfischthere17:16
mfischthe bug is really "make fernet work out of the box + CI"17:18
EmilienMmfisch: upstream == keystone or puppet-keystone?17:19
mfischpuppet-keystone17:19
mfischwe have our own special sauce to get fernet 100% working that we use17:19
EmilienMfeel free to submit a patch then17:19
EmilienMand a CI patch too17:19
mfischI'll file a bug so that I dont forget17:19
EmilienMmfisch: thanks.17:22
EmilienMmfisch: and please send a patch to puppet-openstack-integration, pick up a scenario17:22
mfischyep will do17:25
EmilienMthanks17:25
openstackgerritMerged openstack/puppet-glance: Supports IPv6 in registry_host  https://review.openstack.org/28631917:26
*** degorenko is now known as _degorenko|afk17:31
openstackgerritDenis Egorenko proposed openstack/puppet-heat: test do not merge  https://review.openstack.org/28671717:32
*** trown is now known as trown|lunch17:37
openstackgerritMerged openstack/puppet-glance: Switch glance to rspec-puppet-facts  https://review.openstack.org/28218217:37
*** tiswanso has joined #puppet-openstack17:37
*** baker has joined #puppet-openstack17:43
*** derekh has quit IRC17:43
*** dgurtner has quit IRC17:43
EmilienM_ody: have you seen https://bugs.launchpad.net/puppet-horizon/+bug/1551759 ?17:47
openstackLaunchpad bug 1551759 in puppet-horizon "Redirect loop with root_url = '/'" [Undecided,In progress] - Assigned to Martin Millnert (r-martin-5)17:47
openstackgerritLeon Zachery proposed openstack/puppet-neutron: Cisco nexus config manifest - obsolete parameter (switch_replay_count).  https://review.openstack.org/28624317:55
*** hjensas has quit IRC17:58
*** hjensas has joined #puppet-openstack18:02
EmilienMmwhahaha: before I continue the work, can you please provide feedback on https://review.openstack.org/#/c/285600/ ?18:07
mwhahahasure18:07
EmilienMI plan to continue the work for having all APIs on SSL & IPv618:07
EmilienM(for scenario002)18:08
mwhahahaso should we use some sort of config class for these toggles?18:08
mwhahahaso that you don't have to specify all of them18:08
mwhahahaie create a openstack_integration::config class that you can use for ssl/ipv6 and just pull them in as defaults for the classes18:08
mwhahahaso you can just class { openstack_integration::config: ssl => true} and everything that then gets pulled in will have it enabled18:09
*** rahulait has joined #puppet-openstack18:09
EmilienMmwhahaha: or params?18:09
EmilienMwait, what would do config.pp?18:09
mwhahahalet me show you18:09
mwhahahasec18:09
EmilienMok cool18:09
*** rahulait has quit IRC18:10
*** jpena has quit IRC18:10
mwhahahaEmilienM: http://paste.openstack.org/show/488815/18:13
EmilienMoh nice18:14
EmilienMI like that18:14
EmilienMmwhahaha: can't I just use params.pp ?18:14
mwhahahasure18:14
mwhahahabut it's more a config18:14
EmilienMok I'll add config then18:14
mwhahahasince openstack_integration itself is more like an application than a module to install/config other stuff18:15
*** rmart04 has quit IRC18:15
EmilienMmakes sense18:15
mwhahahaglobals or config is better for what we're using it for18:15
mwhahahaparams classes usually don't have configurable params18:15
EmilienMright18:16
*** openstackgerrit has quit IRC18:18
*** openstackgerrit has joined #puppet-openstack18:18
EmilienMmwhahaha: I also plan to switch other ubuntu jobs on ubuntu-trusty instead of bare18:22
EmilienMwill be faster to run in our CI18:23
mwhahahaok18:23
*** aglarendil__ has joined #puppet-openstack18:30
*** jaosorior_away is now known as jaosorior18:31
*** trown|lunch is now known as trown18:35
*** aglarendil has quit IRC18:37
*** hjensas has quit IRC18:37
*** aglarendil__ is now known as aglarendil18:37
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: refactorize SSL configuration  https://review.openstack.org/28676718:38
*** imcsk8 has quit IRC18:43
*** imcsk8 has joined #puppet-openstack18:43
*** haplo37 has quit IRC18:46
*** itsuugo has quit IRC18:50
*** ducttape_ has quit IRC18:52
*** furlongm_ has quit IRC18:52
*** finchd has quit IRC18:52
*** artem_panchenko_ has quit IRC18:52
*** _ody has quit IRC18:52
*** furlongm_ has joined #puppet-openstack18:54
*** openstack has joined #puppet-openstack19:16
*** openstackstatus has joined #puppet-openstack19:18
*** ChanServ sets mode: +v openstackstatus19:18
*** openstackgerrit has joined #puppet-openstack19:19
*** hjensas has quit IRC19:25
*** mwhahaha has quit IRC19:25
*** nibalizer has quit IRC19:25
openstackgerritColleen Murphy proposed openstack/puppet-keystone: Add keystone::disable_admin_token_auth class  https://review.openstack.org/28092919:25
*** imcsk8_ is now known as imcsk819:26
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: deploy RabbitMQ with IPv6 binding  https://review.openstack.org/28678819:27
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: refacto: use params.pp for distro specific params  https://review.openstack.org/28678919:30
*** mwhahaha has joined #puppet-openstack19:31
*** nibalizer has joined #puppet-openstack19:31
*** wolfe.freenode.net sets mode: +o mwhahaha19:31
*** hjensas has joined #puppet-openstack19:32
*** hjensas has joined #puppet-openstack19:32
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: deploy RabbitMQ with IPv6 binding  https://review.openstack.org/28678819:32
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: refactorize SSL configuration  https://review.openstack.org/28676719:32
*** fc__ has quit IRC19:33
*** fc__ has joined #puppet-openstack19:33
*** delattec has quit IRC19:44
*** delattec has joined #puppet-openstack19:47
EmilienM8.0.0b1 is out19:51
EmilienMour first mitaka release (beta)19:51
dfisheryay!19:52
EmilienMzigo, jayg: note for packagers ^19:52
jaygEmilienM: thanks19:52
zigoEmilienM: Oh !!!!!!!!!! Much love for that!19:53
zigoFantastic.19:53
zigoEmilienM: It is JUST ON TIME for me.19:53
EmilienMwait, not all modules though19:53
zigoEmilienM: I've just finished packaging Fuel for Debian, and I needed to start testing.19:53
zigoThough I've seen lots of Ubuntu-ism there already so this will need to be patched first ...19:54
zigoEmilienM: 8.0.0b1, and not 8.0.0.0b1 ?!?19:57
EmilienMzigo: I respected the tags we currently have19:57
zigoEmilienM: You're not following the semantic version scheme?19:57
zigoQuite not.19:57
EmilienMhttps://github.com/openstack/puppet-nova/releases19:57
EmilienMwe == puppet group19:57
zigoOh...19:58
EmilienMso if you think that's bad tell it and we'll see if we need to change it19:58
zigoEmilienM: Well, that's bad because that's not following what other packages are doing.19:58
zigoNormally, you'd add another zero before the release.19:58
zigoAs in, 8.0.0.0b1, then 8.0.0.0rc1, then 8.0.019:59
EmilienMzigo: we are following this tag numbers way before openstack agreed on tag conventions19:59
zigoIf you do 8.0.0b1, it means you'd be release 8.0 later on.19:59
EmilienMbut I honestly don't care about that, we can change it.19:59
EmilienMzigo: http://lists.openstack.org/pipermail/openstack-dev/2016-March/087962.html19:59
EmilienMfor the list19:59
zigoIt's just a mater of consistency, and I need to know what to do on my packaging.19:59
zigoWill you later release 8.0.0 ?19:59
zigoOr 8.0 ?19:59
EmilienM8.0.0 if I follow what we did until now20:00
EmilienMbut again, we can change that if it's not good20:00
zigoLook at the tag for Nova for example: https://github.com/openstack/nova/releases20:00
EmilienMI don't care about tags20:00
zigoThey have one zero more ...20:00
zigoThat's a python stuff though.20:00
EmilienMyeah20:01
zigoI honestly don't care that much, because anyway I have to re-tag on top.20:01
EmilienMwe are puppet guys !20:01
zigoMeaning that if you tag 8.0.0b1, I will retag it as 8.0.0_b1, so that it becomes 8.0.0~b1 later.20:01
zigoSo, as long as I know what the future will be, what you did is ok for me! :P20:02
EmilienMcool20:02
EmilienMwe all want to make zigo happy20:03
zigo:)20:03
* zigo goes to sleep, 4am here...20:04
*** tiswanso has joined #puppet-openstack20:08
*** jaosorior has quit IRC20:12
*** francois1 has quit IRC20:42
*** ikkeT has left #puppet-openstack20:43
EmilienMcrinkle: do you plan to deploy infracloud with mysql/ssl too?20:49
crinkleEmilienM: mysql only listens locally so ssl isn't needed20:52
crinklein our setup20:52
EmilienMindeed, it makes sense20:52
EmilienMcrinkle: I think you don't need the enable_ssl in ::keystone class, it's for eventlet20:54
EmilienMbut I'm not sure20:54
crinkleEmilienM: i think you're right but i'm also not sure20:55
EmilienMit does not hurt to keep it imho20:55
*** jayg is now known as jayg|g0n321:19
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: CI run with RDO trunk - do not merge  https://review.openstack.org/28684521:22
*** johnmilton has quit IRC21:28
mfischEmilienM: ... mfisch CI found another issue21:37
EmilienMmfisch: shoot21:38
mfischneutron really needs python-keystoneclient I think21:38
mfischI get failure to auth tokens21:38
mfischstill digging but thats my initial take21:38
mfischyep, validated21:41
mfischdoes our CI run any neutron calls at all?21:41
*** derekh has joined #puppet-openstack21:41
EmilienMmfisch: lol we are running tempest for months21:42
EmilienMyou have been reviewing my patches21:42
EmilienMand you're asking me if we test neutron21:43
mfischwell21:43
EmilienMmfisch: https://github.com/openstack/puppet-openstack-integration#description21:43
mfischso somewhere in those modules you must have an explicit dep on keystone::client21:43
mfischright I know about those...21:43
EmilienMwe run single node21:43
mfischaha21:43
mwhahahait's cause we install keystone on the same host21:43
mfischthat explains21:43
EmilienMso obviously we have the package installed21:43
EmilienMbut I guess you don't21:43
EmilienMso you found a bug21:43
mfischyeah I will fix puppet-neutron21:44
EmilienMbut IMHO that bug should be fixed in packaging.21:44
mfischwell trye21:44
mfischtrue21:44
mwhahahait's jsut a revert of https://review.openstack.org/#/c/278213/21:44
EmilienMit's part of OpenStack requirements21:44
mfischno21:44
mwhahahano?21:44
mfischmwhahaha: because that class is a no-op now21:44
EmilienMeach OpenStack projects manages deps regarding OpenStack requirements21:44
EmilienMand packagers should follow that21:44
mwhahahaah21:44
EmilienMRDO follows it FWIW21:44
mfischI think an upstream ubuntu bug + work-around in puppet-neutron21:45
EmilienMI agree21:45
mfischok21:45
mfischwill do21:45
EmilienMmfisch: you made my day21:45
EmilienMthank you so much21:45
mfischpas ne probleme21:46
EmilienMmfisch: if you want to look what puppet-neutron actually test, please look http://status.openstack.org/openstack-health/#/g/project/openstack~2Fpuppet-neutron?groupKey=project&resolutionKey=hour21:46
mfischk21:46
EmilienMmfisch: we run tempest scenarios, neutron API calls, etc etc etc21:47
mfischI do know about the tests my question was more like could you tell me offhand without me looking21:47
mfischbeing lazy ;)21:47
EmilienMgrr21:47
mfischyou know how lazy us americans are !21:48
EmilienMmfisch: have you ever been in Europe? :-)21:49
mwhahahai want european holidays :(21:49
mfischlol21:49
EmilienM(on my side, I just want holidays) :-)21:50
mwhahahawhen i worked with guys from the UK and summer rolled around they were like going on holiday for a month later21:50
openstackgerritMatt Fischer proposed openstack/puppet-neutron: Workaround package bug and include keystone client  https://review.openstack.org/28685421:50
mfischmwhahaha: they're enjoying their month of sun21:50
mfischThere's the work-around21:50
_odyMartin Millnert21:53
_ody?? anyone know what is handle is?21:53
*** foxhunt_away is now known as foxhunt7221:54
*** foxhunt72 is now known as foxhunt_away21:59
*** trown is now known as trown|outtypewww21:59
openstackgerritEmilien Macchi proposed openstack/puppet-glance: swift backend: don't put / in the default auth url  https://review.openstack.org/28685922:05
EmilienMdmsimard: ^22:05
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: CI run with RDO trunk - do not merge  https://review.openstack.org/28684522:06
EmilienMtesting it with latest RDO trunk ^22:06
dmsimardEmilienM: and you'll rebase your CI on top ?22:06
dmsimardyeah ok22:06
EmilienMnext bug plese!22:06
dmsimardEmilienM: I'm not sure it's that22:06
dmsimardEmilienM: let's not get ahead of ourselves :P22:06
EmilienMdmsimard: yeah? you tested it :-P22:06
dmsimardbeen trying to find an explanation for the issue in the code22:07
*** dprince has quit IRC22:08
dmsimardEmilienM: the only thing I can see is this.. https://github.com/openstack/glance_store/commit/14eac131845bf2762668d665d3283b5abd87eeb2#diff-132c7ed276472dce48746e30e2ffe6daR77322:08
dmsimardnotice how there's @<AUTH_ADDRESS>/<CONTAINER>22:08
dmsimardso does it break because it does @127.0.0.1/v2.0//<CONTAINER> or something, dunno22:08
*** francois has joined #puppet-openstack22:11
*** tiswanso has quit IRC22:12
openstackgerritMatt Fischer proposed openstack/puppet-keystone: WIP Keystone hooks support  https://review.openstack.org/28389822:23
*** brandon has joined #puppet-openstack22:31
EmilienMcrinkle: why don't you put keystone ssl certif in /etc/keystone http://git.openstack.org/cgit/openstack-infra/puppet-infracloud/tree/manifests/controller.pp#n147 ?22:35
EmilienMI was wondering if you had a specific reason or if I could do that22:35
EmilienMnot certif, sorry. The key22:36
*** angdraug has quit IRC22:36
*** dgurtner has quit IRC22:39
crinkleEmilienM: no particular reason...it would probably make more sense to put it in /etc/keystone since it's already owned by the keystone user22:40
EmilienMok thanks22:40
crinklei think i was thinking it needed to be readable by the www-data user22:40
EmilienMcrinkle: I don't think so, the wsgi process is run under keystone:keystone22:41
*** jtomasek has quit IRC22:41
crinkleyeah so putting it in /etc/ssl was probably silly22:42
EmilienMhttps://github.com/openstack/puppet-keystone/blob/master/manifests/wsgi/apache.pp#L235-L23622:42
EmilienMno problem, just checking22:42
EmilienMI try to avoid to use /etc/ssl if possible, because we run SElinux and I'm aware about some rules preventing using it from some process22:43
crinkleah yeah that would make things hard22:44
EmilienMcrinkle: I also noticed you don't run metadata agent, is it on purpose?22:44
EmilienMneutron metadata agent*22:44
*** dhollinger has quit IRC22:46
crinkleEmilienM: yes, we use configdrive instead22:46
EmilienMok, indeed metadata is not needed22:46
EmilienMdmsimard: still broken, even with the puppet-glance patch22:48
EmilienMmwhahaha: I'm holding https://review.openstack.org/#/c/286859/ -- I thought it would help but in fact it's not.22:51
mwhahaha:(22:52
*** delattec has quit IRC22:57
*** delattec has joined #puppet-openstack23:00
*** haplo37 has quit IRC23:14
*** rahulait has joined #puppet-openstack23:18
*** baker has quit IRC23:25
dmsimardEmilienM: I don't know why it works for packstack and not for you then23:26
dmsimardhell, I even got both scenarios passing now https://review.openstack.org/#/c/286864/23:26
EmilienMdmsimard: I'll compare both configs23:26
EmilienMdmsimard: honestly: I would enable voting for packstack NOW before it breaks again23:27
dmsimardEmilienM: I have a review in project-config to add scenario00323:27
Anticimex_ody: I'm Martin Millnert23:30
*** ducttape_ has quit IRC23:38
*** mdorman has quit IRC23:40
_odyAnticimex: Left you a note on the puppet-horizon ticket.  Let me know if you want me to fix it.  The issue is my fault.23:42
AnticimexI just responded23:43
AnticimexI don't mind fixing it myself, would appreciate the learning of the tests I haven't written.23:43
Anticimexbut regardless, what's the proper way to avoid the described *possible* error? do our "one-liner" if/else ok?23:44
Anticimexno idea what the proper logic should be on testing this though?23:45
Anticimexi guess, attempting to provoke the error and checking that the puppet code didn't produce the apache::vhost::redirect -variables that would cause a loop23:45
EmilienMAnticimex: hopefully the work on having a SSL scenario in our CI will help to catch more bugs23:46
EmilienMiiuc the bug23:46
Anticimexit's not ssl specific though, afaict23:46
EmilienMoh ok23:46
Anticimexapache::vhost doesn't avoid you configuring '/' -> '/' redirects23:47
Anticimexone could call that a root problem perhaps :)23:47
Anticimexyou need to set $root_url to '/' to get the error23:48
Anticimexas i wrote in the bug, we did a functionally identical commit in our puppet-horizon fork last june and discovered this then23:48
Anticimexbut now i'm trying to close forks, and didn't need to attempt upstreaming that since _ody had just done so23:49
_odyAnticimex: Ah.  I took the approach that if someone wants to have a dashboard without anything after / they set root_url to ''...this is what I did when I forked puppet-horizon for out production cloud.  I do not remember my reasoning.23:49
_odyIt is possible that '' just required a modify fewer things in the module's codebase.23:50
*** imcsk8 has quit IRC23:50
*** imcsk8 has joined #puppet-openstack23:50
_odyWe can totall go the other direction and expect people to provide a / instead of ''.23:50
_odyUgh.  Can't remember at all why I chose that.23:50
_odyProbably something to do with the way I think about URLs, like how one doesn't actual ever indicate the trailing slash when the provide someone a URL.23:52
_odyWe tell our users to go to https://slice.ops.puppetlabs.net.23:52
Anticimexjust need to avoid having puppetlabs-apache setting up a redirect loop, no opinion on what the best fix is23:52
_odyOk.  What ever fixes is it fine by me then.23:53
Anticimexyour $root_url-configurability commit allows '/' though :-)23:53
Anticimexnot sure how apache::vhost treats '' vs '/'23:53
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: switch OpenStack APIs to SSL  https://review.openstack.org/28689923:54
_odyQuite differently.  If you set apache to forward to a wsgi script at /dashboard is is not the same as /dashboard/.23:54
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: switch OpenStack APIs to SSL  https://review.openstack.org/28689923:54
_ody'' vs '/' after a raw fqdn is handled a bit differently, fowarding to the script is both situations.23:55
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: switch OpenStack APIs to SSL  https://review.openstack.org/28689923:57
*** rahulait has quit IRC23:58
*** dfisher has quit IRC23:59
*** delattec has quit IRC23:59
« Monday, 2016-02-29 Index Wednesday, 2016-03-02 »

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!