Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2016-03-01

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:17 furlongm joined #puppet-openstack
00:20 vinsh joined #puppet-openstack
00:21 _joes_ joined #puppet-openstack
00:25 davideagle joined #puppet-openstack
00:33 ducttape_ joined #puppet-openstack
01:11 ducttape_ joined #puppet-openstack
01:12 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: IPv6: duak-stack support for Keystone  https://review.openstack.org/286344
01:37 xarses joined #puppet-openstack
01:38 markvoelker joined #puppet-openstack
02:08 markvoelker joined #puppet-openstack
02:15 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: IPv6: duak-stack support for Keystone  https://review.openstack.org/286344
02:20 ducttape_ joined #puppet-openstack
03:20 baker joined #puppet-openstack
03:21 dhollinger joined #puppet-openstack
03:40 openstackgerrit Matt Fischer proposed openstack/puppet-keystone: WIP Keystone hooks support  https://review.openstack.org/283898
03:41 openstackgerrit Merged openstack/puppet-keystone: stop managing files that the package makes  https://review.openstack.org/283308
04:59 fedexo joined #puppet-openstack
05:03 openstackgerrit Matt Fischer proposed openstack/puppet-keystone: WIP Keystone hooks support  https://review.openstack.org/283898
05:49 jaosorior joined #puppet-openstack
06:09 rcernin joined #puppet-openstack
06:35 trown joined #puppet-openstack
07:32 _matthias_ joined #puppet-openstack
07:36 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Always override X-Forwarded-Proto header for Heat  https://review.openstack.org/286420
07:39 bnemec joined #puppet-openstack
07:48 hjensas joined #puppet-openstack
07:48 hjensas joined #puppet-openstack
07:50 jtomasek joined #puppet-openstack
07:58 dgurtner joined #puppet-openstack
08:00 xingchao joined #puppet-openstack
08:10 openstackgerrit Nikolay Starodubtsev proposed openstack/puppet-murano: Switch murano to use keystone API v3 by default  https://review.openstack.org/279444
08:18 paramite joined #puppet-openstack
08:21 jpena joined #puppet-openstack
08:24 xingchao joined #puppet-openstack
08:48 openstackgerrit joined #puppet-openstack
08:53 xingchao_ joined #puppet-openstack
08:58 gfidente joined #puppet-openstack
08:58 xingchao joined #puppet-openstack
09:12 xingchao joined #puppet-openstack
09:13 xingchao joined #puppet-openstack
09:14 jistr joined #puppet-openstack
09:18 xingchao joined #puppet-openstack
09:20 igajsin1 left #puppet-openstack
09:30 ddmitriev joined #puppet-openstack
09:34 davideagle joined #puppet-openstack
09:39 derekh joined #puppet-openstack
09:46 xingchao joined #puppet-openstack
09:49 aarefiev joined #puppet-openstack
09:56 openstackgerrit Xingchao Yu proposed openstack/puppet-oslo: Add oslo.middleware defines  https://review.openstack.org/274422
10:35 zigo EmilienM: That's off-topic for this channel, but Debconf 17 will be in Montreal ! :)
10:35 zigo https://wiki.debconf.org/wi​ki/DebConf17/Bids/Montreal
10:36 zigo I'll get to see how your place is... :P
10:38 openstackstatus NOTICE: Gerrit is going to be restarted due to poor performance
10:38 Topic for #puppet-openstack is now Gerrit is going to be restarted due to poor performance
10:44 Topic for #puppet-openstack is now Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet
10:44 openstackstatus NOTICE: gerrit finished restartign
10:50 openstackgerrit Nikolay Starodubtsev proposed openstack/puppet-murano: Switch murano to use keystone API v3 by default  https://review.openstack.org/279444
10:54 davideagle joined #puppet-openstack
11:00 openstackgerrit Merged openstack/puppet-heat: Implement crontab to purge deleted data  https://review.openstack.org/286290
11:37 openstackgerrit Merged openstack/puppet-neutron: Neutron subnet will now be able to update  https://review.openstack.org/285518
12:00 davideagle joined #puppet-openstack
12:01 EmilienM good morning
12:01 EmilienM zigo: cool
12:02 johnmilton joined #puppet-openstack
12:09 openstackgerrit Nikolay Starodubtsev proposed openstack/puppet-murano: Switch murano to use keystone API v3 by default  https://review.openstack.org/279444
12:10 dgurtner joined #puppet-openstack
12:15 EmilienM degorenko: I replied to https://review.openstack.org/#/c/285586/ a bit late but I did :)
12:16 openstackgerrit Babu Shanmugam proposed openstack/puppet-vswitch: Added custom type vs_config and OVS provider for it  https://review.openstack.org/285996
12:28 degorenko EmilienM, no problem :)
12:29 davideagle joined #puppet-openstack
12:37 jaosorior joined #puppet-openstack
12:48 davideagle joined #puppet-openstack
13:05 ducttape_ joined #puppet-openstack
13:10 davideagle joined #puppet-openstack
13:21 furlongm joined #puppet-openstack
13:25 xingchao joined #puppet-openstack
13:25 dprince joined #puppet-openstack
13:32 davideagle joined #puppet-openstack
13:34 openstackgerrit Nikolay Starodubtsev proposed openstack/puppet-murano: Switch murano to use keystone API v3 by default  https://review.openstack.org/279444
13:36 richm joined #puppet-openstack
13:37 kashyap joined #puppet-openstack
13:38 furlongm_ joined #puppet-openstack
13:39 kashyap EmilienM: I'm looking at this change - https://review.openstack.org/#/c/28658​4/2/puppet/manifests/overcloud_compute.pp
13:39 EmilienM kashyap: hey! you here
13:39 kashyap EmilienM: Aren't you doing two unrelated changes?
13:39 kashyap Yeah, just joined here to quickly ACK it, while it's fresh in memory
13:39 EmilienM kashyap: no I moved the nova_config so I can reuse rbd_ephemeral_storage boolean
13:40 EmilienM kashyap: in puppet, order matters
13:40 kashyap EmilienM: Duh, I just needed to scroll down!
13:40 kashyap I'll add a comment to your question there.
13:40 EmilienM kashyap: do I need to configure live_migration_flag too ?
13:40 EmilienM thanks!
13:40 kashyap EmilienM: Well, for now, yes.  In future, it'll all be handled with a single tri-state flag.
13:40 EmilienM ok
13:40 kashyap I'll add the details there for reference.
13:41 EmilienM ok, great, I'll iterate after that
13:43 jtomasek_ joined #puppet-openstack
13:46 kashyap EmilienM: One more: You special-cased RBD, and used TUNNELLED for 'block_migration_flag', any reason?
13:46 EmilienM kashyap: because of https://bugzilla.redhat.com/​show_bug.cgi?id=1301986#c12
13:46 cdelatte joined #puppet-openstack
13:46 openstack bugzilla.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 hbrock
13:47 fwdit joined #puppet-openstack
13:48 kashyap EmilienM: Oh, yes, that's the correct rationale.
13:49 kashyap left #puppet-openstack
13:56 rmart04 joined #puppet-openstack
14:03 EmilienM upstream meeting starts in one hour, feel free to add topics / bugs / patches: https://etherpad.openstack.org/p/pupp​et-openstack-weekly-meeting-20160301
14:05 mdorman joined #puppet-openstack
14:14 openstackgerrit Dmitry Tantsur proposed openstack/puppet-ironic: [inspector] use dnsmasq dhcp-sequential-ip flag in dnsmasq.conf  https://review.openstack.org/286617
14:15 mfisch EmilienM: that steverdore change broke me it seems
14:15 mfisch I'm going to have to put the long names back in hiera
14:15 EmilienM mfisch: what version are you running?
14:15 mfisch still an old copy of L
14:15 mfisch L before release really
14:15 EmilienM and you running master?
14:15 EmilienM (puppet-keystone)
14:16 mfisch I think this is stevedore issue
14:16 mfisch ValueError: Empty module name
14:16 mfisch yeah, well im in process of trying the new module
14:16 openstackgerrit Merged openstack/puppet-openstack-integration: Bump RDO repo to February 29th trunk  https://review.openstack.org/286258
14:17 EmilienM mfisch: the upstream doc and keystone folks says "use stevedore names"
14:17 EmilienM mfisch: please file a bug, with logs, etc
14:17 mfisch once I pin this down I will let you know, this is a test merge from about 3 weeks of stuff
14:17 EmilienM mfisch: but you have hit this issue because you run on Liberty while our module is for master (mitaka)
14:18 mfisch are the docs you found for M?
14:18 mfisch if this is really for M then I'll work around it
14:18 EmilienM mfisch: I don't know details
14:18 EmilienM i looked at doc and how devstack works
14:22 mfisch I'll dig into it today
14:23 mfisch yep thats it
14:23 mfisch EmilienM: I fixed with hiera, good to know I can clean up in Mitaka
14:24 EmilienM cool
14:24 mfisch keystone::token_driver: 'keystone.token.persistence.backends.sql.Token'
14:24 mfisch keystone::catalog_driver: 'keystone.catalog.backends.sql.Catalog'
14:25 furlongm_ joined #puppet-openstack
14:27 openstackgerrit Nikolay Starodubtsev proposed openstack/puppet-murano: Switch murano to use keystone API v3 by default  https://review.openstack.org/279444
14:28 vinsh joined #puppet-openstack
14:44 dfisher joined #puppet-openstack
14:46 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-ceilometer: Deprecate ceilometer alarm classes  https://review.openstack.org/282258
14:47 furlongm_ joined #puppet-openstack
14:56 _ody joined #puppet-openstack
14:57 skolekonov_ joined #puppet-openstack
14:58 ducttape_ joined #puppet-openstack
15:08 openstackgerrit Martin Millnert proposed openstack/puppet-horizon: Avoid causing redirect loop  https://review.openstack.org/286647
15:13 EmilienM mwhahaha: can you look https://review.openstack.org/285586 if you have time today?
15:14 mwhahaha sure
15:25 openstackgerrit Martin Millnert proposed openstack/puppet-keystone: Make apache::vhost::headers configuration possible  https://review.openstack.org/286656
15:27 haplo37 joined #puppet-openstack
15:32 mfisch EmilienM: sorry I had a commute this morning
15:33 vinsh joined #puppet-openstack
15:34 EmilienM mfisch: sorry for what?
15:35 EmilienM ah for the meeting, no worries, we had 0 item
15:35 EmilienM mfisch: though you and chem have an action that is postponed for weeks
15:35 EmilienM maybe we can work on it soon
15:45 dhollinger joined #puppet-openstack
15:55 delattec joined #puppet-openstack
16:02 xarses joined #puppet-openstack
16:03 xarses joined #puppet-openstack
16:06 rochaporto joined #puppet-openstack
16:12 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-tripleo: IPv6: duak-stack support for Keystone  https://review.openstack.org/286344
16:14 xingchao joined #puppet-openstack
16:16 haplo37 joined #puppet-openstack
16:17 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: IPv6: duak-stack support for Keystone  https://review.openstack.org/286344
16:29 vinsh_ joined #puppet-openstack
16:45 xarses dmsimard: thanks for the laugh
16:46 dmsimard xarses: oh ?
16:46 xarses https://github.com/openstack/keystone/commi​t/7c6556ce18eb170fd8102168b33a50272b22ec5a
16:46 dmsimard yeah
16:46 dmsimard lol
16:46 xarses the tests fail 1 day, every 4 years
16:47 xarses oh, some one just mentioned that horizon had a leap year bug too
16:49 dmsimard this is almost y2k all over again
16:51 EmilienM skolekonov: have you seen http://logs.openstack.org/86/285586/20/gate/g​ate-puppet-openstack-integration-scenario001-​tempest-dsvm-trusty/bc7ce9f/logs/neutron/neut​ron-server.txt.gz#_2016-03-01_15_32_37_313 ?
16:51 EmilienM same question for http://logs.openstack.org/86/285586/20/gate/g​ate-puppet-openstack-integration-scenario001-​tempest-dsvm-trusty/bc7ce9f/logs/neutron/neut​ron-server.txt.gz#_2016-03-01_15_32_37_310
17:04 openstackgerrit Denis Egorenko proposed openstack/puppet-heat: test do not merge  https://review.openstack.org/286717
17:05 mfisch EmilienM: the merge of removing /etc/keystone broke fernet tokens
17:05 mfisch the resource dep isnt working properly
17:05 mfisch I'm working on it
17:05 dmsimard mfisch: thank you third party CI
17:05 EmilienM lol
17:06 EmilienM if I could, I would put mfisch in our CI jobs
17:06 mfisch you mean my lab?
17:07 mfisch lol
17:10 EmilienM yeah!
17:10 mfisch EmilienM: I dont get how key-manage fernet ran before keystone was installed
17:11 mfisch https://github.com/openstack/puppet-keysto​ne/blob/master/manifests/init.pp#L915-L920
17:11 mfisch that look right to you?
17:11 mfisch here's what my CI says: http://paste.openstack.org/show/488796/
17:12 EmilienM mfisch: in your catalog, do you see the pkg installed after or before fernet?
17:14 mfisch oddly im not seeing it installed in this run
17:14 mfisch hmm
17:14 EmilienM mfisch: maybe can you submit a patch in our CI to test it?
17:14 mfisch yeah
17:14 EmilienM you proposed to have a scenario deploying it
17:15 EmilienM mfisch: please submit a patch, or at least a PoC, that we can finish but I have 0 experience in fernet
17:15 mfisch this is our issue
17:15 mfisch but I Think upstream should make the fernet keys folder
17:16 mfisch EmilienM: can you make a bug to add a CI test for fernet?
17:16 mfisch the code there in upstream is not 100% thre
17:16 mfisch there
17:18 mfisch the bug is really "make fernet work out of the box + CI"
17:19 EmilienM mfisch: upstream == keystone or puppet-keystone?
17:19 mfisch puppet-keystone
17:19 mfisch we have our own special sauce to get fernet 100% working that we use
17:19 EmilienM feel free to submit a patch then
17:19 EmilienM and a CI patch too
17:19 mfisch I'll file a bug so that I dont forget
17:22 EmilienM mfisch: thanks.
17:22 EmilienM mfisch: and please send a patch to puppet-openstack-integration, pick up a scenario
17:25 mfisch yep will do
17:25 EmilienM thanks
17:26 openstackgerrit Merged openstack/puppet-glance: Supports IPv6 in registry_host  https://review.openstack.org/286319
17:32 openstackgerrit Denis Egorenko proposed openstack/puppet-heat: test do not merge  https://review.openstack.org/286717
17:37 openstackgerrit Merged openstack/puppet-glance: Switch glance to rspec-puppet-facts  https://review.openstack.org/282182
17:37 tiswanso joined #puppet-openstack
17:43 baker joined #puppet-openstack
17:47 EmilienM _ody: have you seen https://bugs.launchpad.net/​puppet-horizon/+bug/1551759 ?
17:47 openstack Launchpad bug 1551759 in puppet-horizon "Redirect loop with root_url = '/'" [Undecided,In progress] - Assigned to Martin Millnert (r-martin-5)
17:55 openstackgerrit Leon Zachery proposed openstack/puppet-neutron: Cisco nexus config manifest - obsolete parameter (switch_replay_count).  https://review.openstack.org/286243
18:02 hjensas joined #puppet-openstack
18:07 EmilienM mwhahaha: before I continue the work, can you please provide feedback on https://review.openstack.org/#/c/285600/ ?
18:07 mwhahaha sure
18:07 EmilienM I plan to continue the work for having all APIs on SSL & IPv6
18:08 EmilienM (for scenario002)
18:08 mwhahaha so should we use some sort of config class for these toggles?
18:08 mwhahaha so that you don't have to specify all of them
18:08 mwhahaha ie create a openstack_integration::config class that you can use for ssl/ipv6 and just pull them in as defaults for the classes
18:09 mwhahaha so you can just class { openstack_integration::config: ssl => true} and everything that then gets pulled in will have it enabled
18:09 rahulait joined #puppet-openstack
18:09 EmilienM mwhahaha: or params?
18:09 EmilienM wait, what would do config.pp?
18:09 mwhahaha let me show you
18:09 mwhahaha sec
18:09 EmilienM ok cool
18:13 mwhahaha EmilienM: http://paste.openstack.org/show/488815/
18:14 EmilienM oh nice
18:14 EmilienM I like that
18:14 EmilienM mwhahaha: can't I just use params.pp ?
18:14 mwhahaha sure
18:14 mwhahaha but it's more a config
18:14 EmilienM ok I'll add config then
18:15 mwhahaha since openstack_integration itself is more like an application than a module to install/config other stuff
18:15 EmilienM makes sense
18:15 mwhahaha globals or config is better for what we're using it for
18:15 mwhahaha params classes usually don't have configurable params
18:16 EmilienM right
18:18 openstackgerrit joined #puppet-openstack
18:22 EmilienM mwhahaha: I also plan to switch other ubuntu jobs on ubuntu-trusty instead of bare
18:23 EmilienM will be faster to run in our CI
18:23 mwhahaha ok
18:30 aglarendil__ joined #puppet-openstack
18:38 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: refactorize SSL configuration  https://review.openstack.org/286767
18:43 imcsk8 joined #puppet-openstack
19:05 ilbot3 joined #puppet-openstack
19:05 Topic for #puppet-openstack is now Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet
19:09 EmilienM mwhahaha: I updated https://review.openstack.org/286767 if you want to look
19:11 imcsk8_ joined #puppet-openstack
19:12 mwhahaha yea that looks better
19:13 EmilienM cool
19:16 openstack joined #puppet-openstack
19:18 openstackstatus joined #puppet-openstack
19:19 openstackgerrit joined #puppet-openstack
19:25 openstackgerrit Colleen Murphy proposed openstack/puppet-keystone: Add keystone::disable_admin_token_auth class  https://review.openstack.org/280929
19:27 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario002: deploy RabbitMQ with IPv6 binding  https://review.openstack.org/286788
19:30 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: refacto: use params.pp for distro specific params  https://review.openstack.org/286789
19:31 mwhahaha joined #puppet-openstack
19:31 nibalizer joined #puppet-openstack
19:32 hjensas joined #puppet-openstack
19:32 hjensas joined #puppet-openstack
19:32 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario002: deploy RabbitMQ with IPv6 binding  https://review.openstack.org/286788
19:32 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: refactorize SSL configuration  https://review.openstack.org/286767
19:33 fc__ joined #puppet-openstack
19:47 delattec joined #puppet-openstack
19:51 EmilienM 8.0.0b1 is out
19:51 EmilienM our first mitaka release (beta)
19:52 dfisher yay!
19:52 EmilienM zigo, jayg: note for packagers ^
19:52 jayg EmilienM: thanks
19:53 zigo EmilienM: Oh !!!!!!!!!! Much love for that!
19:53 zigo Fantastic.
19:53 zigo EmilienM: It is JUST ON TIME for me.
19:53 EmilienM wait, not all modules though
19:53 zigo EmilienM: I've just finished packaging Fuel for Debian, and I needed to start testing.
19:54 zigo Though I've seen lots of Ubuntu-ism there already so this will need to be patched first ...
19:57 zigo EmilienM: 8.0.0b1, and not 8.0.0.0b1 ?!?
19:57 EmilienM zigo: I respected the tags we currently have
19:57 zigo EmilienM: You're not following the semantic version scheme?
19:57 zigo Quite not.
19:57 EmilienM https://github.com/openstack/puppet-nova/releases
19:57 EmilienM we == puppet group
19:57 zigo Oh...
19:58 EmilienM so if you think that's bad tell it and we'll see if we need to change it
19:58 zigo EmilienM: Well, that's bad because that's not following what other packages are doing.
19:58 zigo Normally, you'd add another zero before the release.
19:59 zigo As in, 8.0.0.0b1, then 8.0.0.0rc1, then 8.0.0
19:59 EmilienM zigo: we are following this tag numbers way before openstack agreed on tag conventions
19:59 zigo If you do 8.0.0b1, it means you'd be release 8.0 later on.
19:59 EmilienM but I honestly don't care about that, we can change it.
19:59 EmilienM zigo: http://lists.openstack.org/pipermail/​openstack-dev/2016-March/087962.html
19:59 EmilienM for the list
19:59 zigo It's just a mater of consistency, and I need to know what to do on my packaging.
19:59 zigo Will you later release 8.0.0 ?
19:59 zigo Or 8.0 ?
20:00 EmilienM 8.0.0 if I follow what we did until now
20:00 EmilienM but again, we can change that if it's not good
20:00 zigo Look at the tag for Nova for example: https://github.com/openstack/nova/releases
20:00 EmilienM I don't care about tags
20:00 zigo They have one zero more ...
20:00 zigo That's a python stuff though.
20:01 EmilienM yeah
20:01 zigo I honestly don't care that much, because anyway I have to re-tag on top.
20:01 EmilienM we are puppet guys !
20:01 zigo Meaning 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:02 zigo So, as long as I know what the future will be, what you did is ok for me! :P
20:02 EmilienM cool
20:03 EmilienM we all want to make zigo happy
20:03 zigo :)
20:04 * zigo goes to sleep, 4am here...
20:08 tiswanso joined #puppet-openstack
20:43 ikkeT left #puppet-openstack
20:49 EmilienM crinkle: do you plan to deploy infracloud with mysql/ssl too?
20:52 crinkle EmilienM: mysql only listens locally so ssl isn't needed
20:52 crinkle in our setup
20:52 EmilienM indeed, it makes sense
20:54 EmilienM crinkle: I think you don't need the enable_ssl in ::keystone class, it's for eventlet
20:54 EmilienM but I'm not sure
20:55 crinkle EmilienM: i think you're right but i'm also not sure
20:55 EmilienM it does not hurt to keep it imho
21:22 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: CI run with RDO trunk - do not merge  https://review.openstack.org/286845
21:37 mfisch EmilienM: ... mfisch CI found another issue
21:38 EmilienM mfisch: shoot
21:38 mfisch neutron really needs python-keystoneclient I think
21:38 mfisch I get failure to auth tokens
21:38 mfisch still digging but thats my initial take
21:41 mfisch yep, validated
21:41 mfisch does our CI run any neutron calls at all?
21:41 derekh joined #puppet-openstack
21:42 EmilienM mfisch: lol we are running tempest for months
21:42 EmilienM you have been reviewing my patches
21:43 EmilienM and you're asking me if we test neutron
21:43 mfisch well
21:43 EmilienM mfisch: https://github.com/openstack/puppet​-openstack-integration#description
21:43 mfisch so somewhere in those modules you must have an explicit dep on keystone::client
21:43 mfisch right I know about those...
21:43 EmilienM we run single node
21:43 mfisch aha
21:43 mwhahaha it's cause we install keystone on the same host
21:43 mfisch that explains
21:43 EmilienM so obviously we have the package installed
21:43 EmilienM but I guess you don't
21:43 EmilienM so you found a bug
21:44 mfisch yeah I will fix puppet-neutron
21:44 EmilienM but IMHO that bug should be fixed in packaging.
21:44 mfisch well trye
21:44 mfisch true
21:44 mwhahaha it's jsut a revert of https://review.openstack.org/#/c/278213/
21:44 EmilienM it's part of OpenStack requirements
21:44 mfisch no
21:44 mwhahaha no?
21:44 mfisch mwhahaha: because that class is a no-op now
21:44 EmilienM each OpenStack projects manages deps regarding OpenStack requirements
21:44 EmilienM and packagers should follow that
21:44 mwhahaha ah
21:44 EmilienM RDO follows it FWIW
21:45 mfisch I think an upstream ubuntu bug + work-around in puppet-neutron
21:45 EmilienM I agree
21:45 mfisch ok
21:45 mfisch will do
21:45 EmilienM mfisch: you made my day
21:45 EmilienM thank you so much
21:46 mfisch pas ne probleme
21:46 EmilienM mfisch: if you want to look what puppet-neutron actually test, please look http://status.openstack.org/opens​tack-health/#/g/project/openstack~2Fpuppet-neutron?groupKey=​project&resolutionKey=hour
21:46 mfisch k
21:47 EmilienM mfisch: we run tempest scenarios, neutron API calls, etc etc etc
21:47 mfisch I do know about the tests my question was more like could you tell me offhand without me looking
21:47 mfisch being lazy ;)
21:47 EmilienM grr
21:48 mfisch you know how lazy us americans are !
21:49 EmilienM mfisch: have you ever been in Europe? :-)
21:49 mwhahaha i want european holidays :(
21:49 mfisch lol
21:50 EmilienM (on my side, I just want holidays) :-)
21:50 mwhahaha when i worked with guys from the UK and summer rolled around they were like going on holiday for a month later
21:50 openstackgerrit Matt Fischer proposed openstack/puppet-neutron: Workaround package bug and include keystone client  https://review.openstack.org/286854
21:50 mfisch mwhahaha: they're enjoying their month of sun
21:50 mfisch There's the work-around
21:53 _ody Martin Millnert
21:53 _ody ?? anyone know what is handle is?
22:05 openstackgerrit Emilien Macchi proposed openstack/puppet-glance: swift backend: don't put / in the default auth url  https://review.openstack.org/286859
22:05 EmilienM dmsimard: ^
22:06 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: CI run with RDO trunk - do not merge  https://review.openstack.org/286845
22:06 EmilienM testing it with latest RDO trunk ^
22:06 dmsimard EmilienM: and you'll rebase your CI on top ?
22:06 dmsimard yeah ok
22:06 EmilienM next bug plese!
22:06 dmsimard EmilienM: I'm not sure it's that
22:06 dmsimard EmilienM: let's not get ahead of ourselves :P
22:06 EmilienM dmsimard: yeah? you tested it :-P
22:07 dmsimard been trying to find an explanation for the issue in the code
22:08 dmsimard EmilienM: the only thing I can see is this.. https://github.com/openstack/glance_store/co​mmit/14eac131845bf2762668d665d3283b5abd87eeb​2#diff-132c7ed276472dce48746e30e2ffe6daR773
22:08 dmsimard notice how there's @<AUTH_ADDRESS>/<CONTAINER>
22:08 dmsimard so does it break because it does @127.0.0.1/v2.0//<CONTAINER> or something, dunno
22:11 francois joined #puppet-openstack
22:23 openstackgerrit Matt Fischer proposed openstack/puppet-keystone: WIP Keystone hooks support  https://review.openstack.org/283898
22:31 brandon joined #puppet-openstack
22:35 EmilienM crinkle: why don't you put keystone ssl certif in /etc/keystone http://git.openstack.org/cgit/openstack-infra/pup​pet-infracloud/tree/manifests/controller.pp#n147 ?
22:35 EmilienM I was wondering if you had a specific reason or if I could do that
22:35 EmilienM not certif, sorry. The key
22:40 crinkle EmilienM: no particular reason...it would probably make more sense to put it in /etc/keystone since it's already owned by the keystone user
22:40 EmilienM ok thanks
22:40 crinkle i think i was thinking it needed to be readable by the www-data user
22:41 EmilienM crinkle: I don't think so, the wsgi process is run under keystone:keystone
22:42 crinkle yeah so putting it in /etc/ssl was probably silly
22:42 EmilienM https://github.com/openstack/puppet-keystone/b​lob/master/manifests/wsgi/apache.pp#L235-L236
22:42 EmilienM no problem, just checking
22:43 EmilienM I 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 process
22:44 crinkle ah yeah that would make things hard
22:44 EmilienM crinkle: I also noticed you don't run metadata agent, is it on purpose?
22:44 EmilienM neutron metadata agent*
22:46 crinkle EmilienM: yes, we use configdrive instead
22:46 EmilienM ok, indeed metadata is not needed
22:48 EmilienM dmsimard: still broken, even with the puppet-glance patch
22:51 EmilienM mwhahaha: I'm holding https://review.openstack.org/#/c/286859/ -- I thought it would help but in fact it's not.
22:52 mwhahaha :(
23:00 delattec joined #puppet-openstack
23:18 rahulait joined #puppet-openstack
23:26 dmsimard EmilienM: I don't know why it works for packstack and not for you then
23:26 dmsimard hell, I even got both scenarios passing now https://review.openstack.org/#/c/286864/
23:26 EmilienM dmsimard: I'll compare both configs
23:27 EmilienM dmsimard: honestly: I would enable voting for packstack NOW before it breaks again
23:27 dmsimard EmilienM: I have a review in project-config to add scenario003
23:30 Anticimex _ody: I'm Martin Millnert
23:42 _ody Anticimex: 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:43 Anticimex I just responded
23:43 Anticimex I don't mind fixing it myself, would appreciate the learning of the tests I haven't written.
23:44 Anticimex but regardless, what's the proper way to avoid the described *possible* error? do our "one-liner" if/else ok?
23:45 Anticimex no idea what the proper logic should be on testing this though?
23:45 Anticimex i guess, attempting to provoke the error and checking that the puppet code didn't produce the apache::vhost::redirect -variables that would cause a loop
23:46 EmilienM Anticimex: hopefully the work on having a SSL scenario in our CI will help to catch more bugs
23:46 EmilienM iiuc the bug
23:46 Anticimex it's not ssl specific though, afaict
23:46 EmilienM oh ok
23:47 Anticimex apache::vhost doesn't avoid you configuring '/' -> '/' redirects
23:47 Anticimex one could call that a root problem perhaps :)
23:48 Anticimex you need to set $root_url to '/' to get the error
23:48 Anticimex as i wrote in the bug, we did a functionally identical commit in our puppet-horizon fork last june and discovered this then
23:49 Anticimex but now i'm trying to close forks, and didn't need to attempt upstreaming that since _ody had just done so
23:49 _ody Anticimex: 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:50 _ody It is possible that '' just required a modify fewer things in the module's codebase.
23:50 imcsk8 joined #puppet-openstack
23:50 _ody We can totall go the other direction and expect people to provide a / instead of ''.
23:50 _ody Ugh.  Can't remember at all why I chose that.
23:52 _ody Probably 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 _ody We tell our users to go to https://slice.ops.puppetlabs.net.
23:52 Anticimex just need to avoid having puppetlabs-apache setting up a redirect loop, no opinion on what the best fix is
23:53 _ody Ok.  What ever fixes is it fine by me then.
23:53 Anticimex your $root_url-configurability commit allows '/' though :-)
23:53 Anticimex not sure how apache::vhost treats '' vs '/'
23:54 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario002: switch OpenStack APIs to SSL  https://review.openstack.org/286899
23:54 _ody Quite differently.  If you set apache to forward to a wsgi script at /dashboard is is not the same as /dashboard/.
23:54 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario002: switch OpenStack APIs to SSL  https://review.openstack.org/286899
23:55 _ody '' vs '/' after a raw fqdn is handled a bit differently, fowarding to the script is both situations.
23:57 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario002: switch OpenStack APIs to SSL  https://review.openstack.org/286899

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary