Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2017-03-29

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

All times shown according to UTC.

Time Nick Message
00:01 markvoelker joined #puppet-openstack
00:01 openstackgerrit Vitaliy Nogin proposed openstack/puppet-freezer master: Add few default parameters  https://review.openstack.org/450915
00:01 ducttape_ joined #puppet-openstack
00:01 ntpttr_ joined #puppet-openstack
00:09 ntpttr_ joined #puppet-openstack
00:14 openstackgerrit Vitaliy Nogin proposed openstack/puppet-freezer master: Change FIXME in README file  https://review.openstack.org/451121
00:14 ntpttr_ joined #puppet-openstack
00:23 ntpttr_ joined #puppet-openstack
00:31 ntpttr_ joined #puppet-openstack
00:38 ntpttr_ joined #puppet-openstack
00:42 ducttape_ joined #puppet-openstack
00:47 ntpttr_ joined #puppet-openstack
00:48 openstackgerrit zhongshengping proposed openstack/puppet-manila master: Password should be secured  https://review.openstack.org/450678
00:50 openstackgerrit zhongshengping proposed openstack/puppet-manila master: Password should be secured  https://review.openstack.org/450678
00:52 ducttape_ joined #puppet-openstack
00:53 ntpttr_ joined #puppet-openstack
00:59 ntpttr_ joined #puppet-openstack
01:05 ntpttr_ joined #puppet-openstack
01:08 openstackgerrit zhongshengping proposed openstack/puppet-neutron master: Password should be secured  https://review.openstack.org/450691
01:12 vnogin joined #puppet-openstack
01:13 ntpttr_ joined #puppet-openstack
01:14 dtantsur joined #puppet-openstack
01:18 ntpttr_ joined #puppet-openstack
01:23 ntpttr_ joined #puppet-openstack
01:29 ntpttr_ joined #puppet-openstack
01:35 ntpttr_ joined #puppet-openstack
01:38 openstackgerrit Merged openstack/puppet-barbican master: Password should be secured  https://review.openstack.org/450619
01:40 ntpttr_ joined #puppet-openstack
01:46 openstackgerrit Merged openstack/puppet-glance master: Password should be secured  https://review.openstack.org/450664
01:46 openstackgerrit Merged openstack/puppet-ironic master: Password should be secured  https://review.openstack.org/450673
01:46 openstackgerrit Merged openstack/puppet-ironic master: Add ability to translate swift account project name to UUID  https://review.openstack.org/449838
01:46 openstackgerrit Merged openstack/puppet-nova stable/newton: Add nova::compute::libvirt::qemu class to configure limits  https://review.openstack.org/442969
01:46 ntpttr_ joined #puppet-openstack
01:48 ilbot3 joined #puppet-openstack
01:48 Topic for #puppet-openstack is now CI Status: RED (scenario004-centos7 broken) | Puppet OpenStack modules http://docs.openstack.org/developer/puppet-openstack-guide/
01:51 ntpttr_ joined #puppet-openstack
01:52 openstackgerrit Merged openstack/puppet-octavia master: oslo log: check puppet resource instead of actual config in spec  https://review.openstack.org/450864
01:52 acormier joined #puppet-openstack
01:57 ntpttr_ joined #puppet-openstack
01:58 dixiaoli joined #puppet-openstack
02:03 ntpttr_ joined #puppet-openstack
02:06 ducttape_ joined #puppet-openstack
02:13 ntpttr_ joined #puppet-openstack
02:19 ntpttr_ joined #puppet-openstack
02:25 ntpttr_ joined #puppet-openstack
02:31 ntpttr_ joined #puppet-openstack
02:39 xarses joined #puppet-openstack
02:39 ntpttr_ joined #puppet-openstack
02:42 agrebennikov joined #puppet-openstack
02:44 openstackgerrit Merged openstack/puppet-manila master: oslo log: check puppet resource instead of actual config in spec  https://review.openstack.org/450828
02:44 openstackgerrit Merged openstack/puppet-neutron master: Switch neutron init spec to check oslo log resource  https://review.openstack.org/450859
02:44 openstackgerrit Merged openstack/puppet-ironic master: oslo log: check puppet resource instead of actual config in spec  https://review.openstack.org/450827
02:45 ntpttr_ joined #puppet-openstack
02:57 openstackgerrit zhongshengping proposed openstack/puppet-neutron master: Password should be secured  https://review.openstack.org/450691
03:07 ducttape_ joined #puppet-openstack
03:17 xarses joined #puppet-openstack
03:18 zerick joined #puppet-openstack
03:24 openstackgerrit Merged openstack/puppet-gnocchi master: oslo log: check puppet resource instead of actual config in spec  https://review.openstack.org/450816
03:24 openstackgerrit Merged openstack/puppet-watcher master: Update test-requirements.txt  https://review.openstack.org/448963
03:24 openstackgerrit Merged openstack/puppet-tempest master: Update test-requirements.txt  https://review.openstack.org/448959
03:26 dixiaoli joined #puppet-openstack
03:45 udesale joined #puppet-openstack
04:29 ntpttr_ joined #puppet-openstack
04:38 ducttape_ joined #puppet-openstack
04:47 dgurtner joined #puppet-openstack
04:59 ntpttr_ joined #puppet-openstack
05:01 openstackgerrit zhongshengping proposed openstack/puppet-freezer master: Update test-requirements.txt  https://review.openstack.org/451180
05:06 obre_ joined #puppet-openstack
05:07 ntpttr_ joined #puppet-openstack
05:31 openstackgerrit Merged openstack/puppet-barbican master: Update test-requirements.txt  https://review.openstack.org/448921
05:31 dixiaoli joined #puppet-openstack
05:35 ntpttr_ joined #puppet-openstack
05:39 rcernin joined #puppet-openstack
05:41 ntpttr_ joined #puppet-openstack
05:58 dixiaoli joined #puppet-openstack
06:05 ntpttr_ joined #puppet-openstack
06:08 ducttape_ joined #puppet-openstack
06:14 jaosorior joined #puppet-openstack
06:14 ntpttr_ joined #puppet-openstack
06:16 yprokule joined #puppet-openstack
06:19 ntpttr_ joined #puppet-openstack
06:22 dgurtner joined #puppet-openstack
06:24 ntpttr_ joined #puppet-openstack
06:27 paramite joined #puppet-openstack
06:31 ntpttr_ joined #puppet-openstack
06:37 ntpttr_ joined #puppet-openstack
06:45 ntpttr_ joined #puppet-openstack
06:52 udesale__ joined #puppet-openstack
06:55 ntpttr_ joined #puppet-openstack
06:57 hjensas joined #puppet-openstack
07:00 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-nova master: Add option to specify libvirt's CA file for nova's live migration  https://review.openstack.org/451212
07:01 ntpttr_ joined #puppet-openstack
07:07 ntpttr_ joined #puppet-openstack
07:09 fxpester joined #puppet-openstack
07:14 tesseract joined #puppet-openstack
07:17 ntpttr_ joined #puppet-openstack
07:25 ntpttr_ joined #puppet-openstack
07:35 ntpttr_ joined #puppet-openstack
07:35 dgurtner joined #puppet-openstack
07:39 ducttape_ joined #puppet-openstack
07:41 ntpttr_ joined #puppet-openstack
07:42 jaganathan joined #puppet-openstack
07:54 bogdando joined #puppet-openstack
08:01 ntpttr_ joined #puppet-openstack
08:03 shardy joined #puppet-openstack
08:07 ntpttr_ joined #puppet-openstack
08:08 gfidente joined #puppet-openstack
08:15 ntpttr_ joined #puppet-openstack
08:18 dixiaoli joined #puppet-openstack
08:21 ntpttr_ joined #puppet-openstack
08:27 ntpttr_ joined #puppet-openstack
08:31 dgurtner joined #puppet-openstack
08:33 ntpttr_ joined #puppet-openstack
08:39 ntpttr_ joined #puppet-openstack
08:45 ntpttr_ joined #puppet-openstack
08:46 dixiaoli_ joined #puppet-openstack
08:50 vnogin joined #puppet-openstack
08:52 salmankhan joined #puppet-openstack
08:56 ntpttr_ joined #puppet-openstack
09:02 ntpttr_ joined #puppet-openstack
09:04 pma joined #puppet-openstack
09:06 vnogin joined #puppet-openstack
09:08 openstackgerrit Vasyl Saienko proposed openstack/puppet-ironic stable/ocata: Allow to set port_setup_delay ironic::conductor param  https://review.openstack.org/451262
09:08 ccamacho joined #puppet-openstack
09:09 ducttape_ joined #puppet-openstack
09:10 ntpttr_ joined #puppet-openstack
09:10 jaganathan joined #puppet-openstack
09:16 ntpttr_ joined #puppet-openstack
09:20 derekh joined #puppet-openstack
09:21 ntpttr_ joined #puppet-openstack
09:21 rmart04 joined #puppet-openstack
09:22 openstackgerrit Vitaliy Nogin proposed openstack/puppet-freezer master: Add few default parameters  https://review.openstack.org/450915
09:25 openstackgerrit Merged openstack/puppet-ironic master: Allow to set port_setup_delay ironic::conductor param  https://review.openstack.org/450813
09:28 ntpttr_ joined #puppet-openstack
09:32 mkarpiarz Could someone please explain to me why does my unit test fail on RH-likes?
09:32 mkarpiarz https://review.openstack.org/#/c/450159/
09:33 mkarpiarz I take it's something due to the way this dictionary is passed.
09:34 ntpttr_ joined #puppet-openstack
09:34 mkarpiarz At least this is what I see in the logs.
09:35 mkarpiarz > expected: "OPENSTACK_HEAT_STACK = {", "    'enable_user_pass': True,", "}" got: "OPENSTACK_HEAT_STACK = {", "}"
09:40 ntpttr_ joined #puppet-openstack
09:45 ntpttr_ joined #puppet-openstack
09:48 openstackgerrit Merged openstack/puppet-mistral master: Update test-requirements.txt  https://review.openstack.org/448942
09:51 openstackgerrit Vasyl Saienko proposed openstack/puppet-ironic stable/newton: Allow to set port_setup_delay ironic::conductor param  https://review.openstack.org/451298
09:52 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph master: Require ceph::profile::client instead of ::base in mds.pp and rgw.pp  https://review.openstack.org/446227
09:52 ntpttr_ joined #puppet-openstack
09:53 openstackgerrit Vasyl Saienko proposed openstack/puppet-ironic stable/ocata: Allow to set port_setup_delay ironic::conductor param  https://review.openstack.org/451299
09:57 openstackgerrit Merged openstack/puppet-vswitch master: Update test-requirements.txt  https://review.openstack.org/448962
09:58 ntpttr_ joined #puppet-openstack
10:04 ntpttr_ joined #puppet-openstack
10:09 ntpttr_ joined #puppet-openstack
10:14 richm joined #puppet-openstack
10:15 dgurtner joined #puppet-openstack
10:15 dgurtner joined #puppet-openstack
10:20 ntpttr_ joined #puppet-openstack
10:26 ntpttr_ joined #puppet-openstack
10:31 pester joined #puppet-openstack
10:31 fxpester|2 joined #puppet-openstack
10:32 ntpttr_ joined #puppet-openstack
10:40 ntpttr_ joined #puppet-openstack
10:40 ducttape_ joined #puppet-openstack
10:50 ntpttr_ joined #puppet-openstack
10:55 ntpttr_ joined #puppet-openstack
11:04 ntpttr_ joined #puppet-openstack
11:07 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph master: Require ceph::profile::client instead of ::base in mds.pp and rgw.pp  https://review.openstack.org/446227
11:10 ntpttr_ joined #puppet-openstack
11:16 ntpttr_ joined #puppet-openstack
11:22 ntpttr_ joined #puppet-openstack
11:27 openstackgerrit Mariusz Karpiarz proposed openstack/puppet-horizon master: Adds interface for enable_user_pass  https://review.openstack.org/450159
11:28 ntpttr_ joined #puppet-openstack
11:33 openstackgerrit Ricardo Noriega proposed openstack/puppet-neutron master: Add support for Bagpipe driver to BGPVPN Service  https://review.openstack.org/449081
11:34 ntpttr_ joined #puppet-openstack
11:36 morazi joined #puppet-openstack
11:40 ntpttr_ joined #puppet-openstack
11:46 ntpttr_ joined #puppet-openstack
11:50 dixiaoli joined #puppet-openstack
11:52 ntpttr_ joined #puppet-openstack
11:58 ntpttr_ joined #puppet-openstack
12:04 ntpttr_ joined #puppet-openstack
12:05 openstackgerrit Vitaliy Nogin proposed openstack/puppet-freezer master: Change FIXME for keystone  https://review.openstack.org/451364
12:10 ntpttr_ joined #puppet-openstack
12:16 ntpttr_ joined #puppet-openstack
12:21 ntpttr_ joined #puppet-openstack
12:25 udesale joined #puppet-openstack
12:26 dprince joined #puppet-openstack
12:28 ntpttr_ joined #puppet-openstack
12:30 acormier joined #puppet-openstack
12:34 ntpttr_ joined #puppet-openstack
12:36 fultonj joined #puppet-openstack
12:40 ntpttr_ joined #puppet-openstack
12:41 ducttape_ joined #puppet-openstack
12:43 vnogin joined #puppet-openstack
12:46 ntpttr_ joined #puppet-openstack
12:46 ansmith joined #puppet-openstack
12:48 fultonj joined #puppet-openstack
12:50 agrebennikov joined #puppet-openstack
12:54 ntpttr_ joined #puppet-openstack
13:00 ntpttr_ joined #puppet-openstack
13:01 openstackgerrit Merged openstack/puppet-heat master: Password should be secured  https://review.openstack.org/450667
13:03 openstackgerrit Merged openstack/puppet-ec2api master: Password should be secured  https://review.openstack.org/450658
13:03 openstackgerrit Merged openstack/puppet-designate master: Password should be secured  https://review.openstack.org/450648
13:06 ntpttr_ joined #puppet-openstack
13:14 ntpttr_ joined #puppet-openstack
13:19 openstackgerrit Merged openstack/puppet-manila master: Password should be secured  https://review.openstack.org/450678
13:20 ntpttr_ joined #puppet-openstack
13:26 openstackgerrit Merged openstack/puppet-mistral master: oslo log: check puppet resource instead of actual config in spec  https://review.openstack.org/450840
13:26 openstackgerrit Merged openstack/puppet-ironic stable/ocata: Allow to set port_setup_delay ironic::conductor param  https://review.openstack.org/451299
13:26 openstackgerrit Merged openstack/puppet-ironic stable/newton: Allow to set port_setup_delay ironic::conductor param  https://review.openstack.org/451298
13:30 ntpttr_ joined #puppet-openstack
13:34 jpich joined #puppet-openstack
13:40 ntpttr_ joined #puppet-openstack
13:43 jaosorior joined #puppet-openstack
13:46 ntpttr_ joined #puppet-openstack
13:50 openstackgerrit Ricardo Noriega proposed openstack/puppet-neutron master: Add support for Bagpipe driver to BGPVPN Service  https://review.openstack.org/449081
13:52 ntpttr_ joined #puppet-openstack
13:53 vinsh joined #puppet-openstack
13:58 salmankhan joined #puppet-openstack
14:02 ducttape_ joined #puppet-openstack
14:04 ntpttr_ joined #puppet-openstack
14:10 ntpttr_ joined #puppet-openstack
14:16 ntpttr_ joined #puppet-openstack
14:19 acormier joined #puppet-openstack
14:24 pradk joined #puppet-openstack
14:26 ntpttr_ joined #puppet-openstack
14:31 dgurtner joined #puppet-openstack
14:31 ntpttr_ joined #puppet-openstack
14:43 vinsh joined #puppet-openstack
14:49 dfisher joined #puppet-openstack
14:57 bklei joined #puppet-openstack
15:01 colonwq mwhahaha, I did it! I can reproduce the error from the other day. Working on a pastebin of the info
15:01 mwhahaha oh noes
15:01 * mwhahaha hides
15:02 colonwq :>
15:13 colonwq mwhahaha, Time to cry or laugh.  http://paste.openstack.org/show/604677/
15:16 mwhahaha colonwq: hmm case sensitivty eh
15:17 mwhahaha colonwq: sounds like we need to .lower() for checking
15:17 mwhahaha colonwq: so that must mean someone is adding Member somewhere instead of member
15:17 colonwq mwhahaha, yeah.... the case thing slowly seeped in while I was sleeping last night.
15:18 colonwq mwhahaha, where is the check performed? puppet-keystone?
15:18 mwhahaha in the role provider
15:18 mwhahaha sec
15:18 colonwq ok
15:19 xarses_ joined #puppet-openstack
15:20 mwhahaha colonwq: https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/provider/keystone_role/openstack.rb#L66-L68
15:20 mwhahaha colonwq: probably need a .lower on both of those
15:20 andreww joined #puppet-openstack
15:21 mwhahaha or something
15:22 EmilienM cschwede: would you be interested to patch puppet-swift to support Wsgi deployment with Apache for Proxy service?
15:22 colonwq mwhahaha, something like "resources[name].provider.tolower = provider.tolower"
15:23 cschwede EmilienM: yeah; however we first need to check if that still works (proxy w/ apache). i don't know anyone running that at larger scale...
15:24 EmilienM cschwede: what is best now? with or without?
15:24 cschwede EmilienM: without Apache
15:24 mwhahaha colonwq: no the name
15:24 xarses_ joined #puppet-openstack
15:25 colonwq mwhahaha, oh in the if line.
15:25 EmilienM cschwede: interesting, ok. Then nevermind
15:25 colonwq let me test it locally
15:25 mwhahaha colonwq: not completely sure if that's the spot, but that is kinda the problem. I think there might be more problems than just that
15:25 cschwede EmilienM: there are some discussions around this. if there is a launchpad entry, RFE or similar, please feel free to assign it to me
15:27 EmilienM cschwede: there is not. Right now, there is a community goal where we ask projects to support deployments in WSGI
15:27 openstackgerrit Pradeep Kilambi proposed openstack/puppet-panko master: Remove explicit log file for panko  https://review.openstack.org/451457
15:28 mwhahaha colonwq: so i guess the other question would be where is Member getting added
15:29 colonwq mwhahaha, have to compare openstack n or openstack o
15:32 ntpttr_ joined #puppet-openstack
15:32 bklei joined #puppet-openstack
15:34 openstackgerrit Pradeep Kilambi proposed openstack/puppet-panko master: Remove explicit log file for panko  https://review.openstack.org/451457
15:34 colonwq mwhahaha, this fixed my case problem: if provider = roles.find{ |role| role.name.downcase == name.downcase }
15:34 mwhahaha colonwq: do you know if it will try and update the name in case of mismatched stuff?
15:35 colonwq mwhahaha, yes. I created rob and then tried to add Rob. The original code did not match. the add failed.
15:36 mwhahaha colonwq: no i mean when you added the .downcase
15:36 mwhahaha colonwq: does it then attempt to set name to Rob?
15:36 mwhahaha like would we get battling updates
15:36 colonwq mwhahaha, nope. It does not.
15:37 mwhahaha ok then that sounds like  a possible fix
15:37 mwhahaha i assume it's because you can't change the role after the fact on keystone so the name mismatch is not a big deal
15:38 mwhahaha brb
15:38 ntpttr_ joined #puppet-openstack
15:44 ntpttr_ joined #puppet-openstack
15:45 openstackgerrit Pradeep Kilambi proposed openstack/puppet-panko master: Remove explicit log file for panko  https://review.openstack.org/451457
15:46 pradk EmilienM, ^^ addressed your comment
15:46 EmilienM cool
15:47 ducttape_ joined #puppet-openstack
15:56 ducttape_ joined #puppet-openstack
16:06 openstackgerrit Mykyta Karpin proposed openstack/puppet-panko master: oslo log: check puppet resource instead of actual config in spec  https://review.openstack.org/451473
16:07 openstackgerrit Mykyta Karpin proposed openstack/puppet-panko master: oslo log: check puppet resource instead of actual config in spec  https://review.openstack.org/451473
16:10 bklei joined #puppet-openstack
16:12 openstackgerrit Merged openstack/puppet-gnocchi master: Update test-requirements.txt  https://review.openstack.org/448934
16:14 ntpttr_ joined #puppet-openstack
16:22 ntpttr_ joined #puppet-openstack
16:32 openstackgerrit Merged openstack/puppet-heat master: Set endpoint_type to <SERVICE DEFAULT>  https://review.openstack.org/449146
16:32 openstackgerrit Merged openstack/puppet-sahara master: Update test-requirements.txt  https://review.openstack.org/448956
16:33 iberezovskiy joined #puppet-openstack
16:52 shardy joined #puppet-openstack
17:00 ntpttr_ joined #puppet-openstack
17:06 ntpttr_ joined #puppet-openstack
17:13 mwhahaha ovn is broken on beaker because a copr repo is no longer available (wtf)
17:13 EmilienM mwhahaha: please ping russelb
17:13 Topic for #puppet-openstack is now CI Status: GREEN(ish) | Puppet OpenStack modules http://docs.openstack.org/developer/puppet-openstack-guide/
17:13 mwhahaha it's the dpdk-snapshot or something
17:14 mwhahaha i'll add it to the l ist to look into  later
17:14 EmilienM mwhahaha: let me ping russel
17:14 EmilienM mwhahaha: do you have a log handy?
17:14 colonwq mwhahaha, which is the correct role "Member" or "member" ? I think my keyboard make a mistake some time in the last 6 months.
17:15 mwhahaha EmilienM: http://logs.openstack.org/52/448952/1/check/gate-puppet-ovn-puppet-beaker-rspec-centos-7/3923824/console.html#_2017-03-29_13_45_13_180931
17:15 EmilienM thx
17:15 rwsu joined #puppet-openstack
17:15 mwhahaha colonwq: i think it's supposed to be Member
17:16 EmilienM mwhahaha: reported
17:16 ntpttr_ joined #puppet-openstack
17:16 mwhahaha colonwq: but we need to fix the name lookup
17:17 vnogin joined #puppet-openstack
17:17 colonwq mwhahaha, my tht rgw patch in O used admin, member and _member_. my puppet patch in N used admin and Member.
17:17 colonwq mwhahaha, i'll make a keystone patch and submit for review once I figure out a proper test.
17:18 mwhahaha might want to ask the keytone guys about member vs Member
17:18 mwhahaha i thought there was some weird historical thing around that
17:20 colonwq i have no memory of it. So much history
17:21 pradk can i get some reviews on https://review.openstack.org/#/c/451457/ please
17:21 mwhahaha colonwq: https://ask.openstack.org/en/question/4759/member-vs-_member_/ and http://stackoverflow.com/questions/20118519/openstack-member-role-and-member-role
17:21 mwhahaha heh
17:24 pradk thx
17:24 colonwq mwhahaha, ok.... let me think about this over lunch. I need a new flat spot on my forehead.
17:24 ntpttr_ joined #puppet-openstack
17:24 mwhahaha colonwq: it sounds like we made a non backwards compatible change in n->o. but we can work around it by making the role check not be case sensitive in puppet-keystone
17:30 ntpttr_ joined #puppet-openstack
17:30 openstackgerrit Emilien Macchi proposed openstack/puppet-ovn master: beaker: stop pulling OVS from dpdk-snapshot  https://review.openstack.org/451493
17:38 ducttape_ joined #puppet-openstack
17:38 ntpttr_ joined #puppet-openstack
17:42 EmilienM mwhahaha: https://review.openstack.org/#/c/451493/ is ready when you are
17:43 EmilienM openvswitch-2.6.1-4.1.git20161206.el7.x86_64
17:46 ntpttr_ joined #puppet-openstack
17:52 ntpttr_ joined #puppet-openstack
17:52 vnogin joined #puppet-openstack
17:55 openstackgerrit Merged openstack/puppet-ovn master: beaker: stop pulling OVS from dpdk-snapshot  https://review.openstack.org/451493
17:58 social joined #puppet-openstack
17:58 openstackgerrit Emilien Macchi proposed openstack/puppet-ovn stable/ocata: beaker: stop pulling OVS from dpdk-snapshot  https://review.openstack.org/451501
17:58 openstackgerrit Emilien Macchi proposed openstack/puppet-ovn stable/newton: beaker: stop pulling OVS from dpdk-snapshot  https://review.openstack.org/451502
18:00 ntpttr_ joined #puppet-openstack
18:01 openstackgerrit Merged openstack/puppet-neutron master: Password should be secured  https://review.openstack.org/450691
18:06 ntpttr_ joined #puppet-openstack
18:09 ducttape_ joined #puppet-openstack
18:12 ntpttr_ joined #puppet-openstack
18:14 acormier_ joined #puppet-openstack
18:16 openstackgerrit Merged openstack/puppet-ovn stable/ocata: beaker: stop pulling OVS from dpdk-snapshot  https://review.openstack.org/451501
18:17 openstackgerrit Merged openstack/puppet-oslo master: Password should be secured  https://review.openstack.org/450553
18:18 openstackgerrit Emilien Macchi proposed openstack/puppet-ovn stable/newton: (newton) beaker: remove testing  https://review.openstack.org/451502
18:18 ntpttr_ joined #puppet-openstack
18:26 ntpttr_ joined #puppet-openstack
18:32 ntpttr_ joined #puppet-openstack
18:36 openstackgerrit Merged openstack/puppet-panko master: Remove explicit log file for panko  https://review.openstack.org/451457
18:37 ducttape_ joined #puppet-openstack
18:38 ntpttr_ joined #puppet-openstack
18:44 ntpttr_ joined #puppet-openstack
18:46 openstackstatus joined #puppet-openstack
18:46 colonwq mwhahaha, should we keep the tht as is in o and/or change it back in p?
18:48 mwhahaha colonwq: not sure. I wouldn't change it back if it's supposed to be Member going forward. Maybe we should add a flag to skip the creation in rgw
18:48 mwhahaha colonwq: we should at least make the name check downcase
18:50 ntpttr_ joined #puppet-openstack
18:50 EmilienM mwhahaha: would you accept https://review.openstack.org/451502 ?
18:53 mwhahaha i guess?
18:54 dfisher charge him $10
18:54 colonwq mwhahaha, I am just trying to decide on the best test to add.
18:54 mwhahaha colonwq: just mock it out in the unit test
18:59 openstackgerrit Merged openstack/puppet-ovn stable/newton: (newton) beaker: remove testing  https://review.openstack.org/451502
19:01 morazi joined #puppet-openstack
19:06 openstackgerrit Merged openstack/puppet-freezer master: Change FIXME in README file  https://review.openstack.org/451121
19:06 ntpttr_ joined #puppet-openstack
19:09 openstackgerrit Merged openstack/puppet-freezer master: Add few default parameters  https://review.openstack.org/450915
19:12 ntpttr_ joined #puppet-openstack
19:14 openstackgerrit Keith Schincke proposed openstack/puppet-keystone master: WIP Add case independent check to keystone_role provider  https://review.openstack.org/451528
19:17 salmankhan joined #puppet-openstack
19:18 ntpttr_ joined #puppet-openstack
19:20 mwhahaha colonwq: i meant in https://github.com/openstack/puppet-keystone/blob/master/spec/unit/provider/keystone_role/openstack_spec.rb
19:23 colonwq mwhahaha, I think I understand. This is my first provider update.
19:23 mwhahaha welcome to the pain
19:24 ntpttr_ joined #puppet-openstack
19:26 colonwq the pain.... so much pain. only rum will help.
19:27 EmilienM mwhahaha: [deployment][forum] proposing a session about future of configuration management - ops + devs wanted!
19:27 EmilienM mwhahaha: I suggest to add that in your agenda for Boston
19:27 EmilienM I don't think we have a date yet
19:27 EmilienM but i'm working on it
19:27 openstackgerrit Pradeep Kilambi proposed openstack/puppet-panko stable/ocata: Remove explicit log file for panko  https://review.openstack.org/451537
19:28 mwhahaha EmilienM: ok
19:30 ducttape_ joined #puppet-openstack
19:35 pradk backport to ocata https://review.openstack.org/#/c/451537/
19:44 ntpttr_ joined #puppet-openstack
19:50 ntpttr_ joined #puppet-openstack
19:53 morazi joined #puppet-openstack
19:53 dgurtner joined #puppet-openstack
19:54 colonwq mwhahaha, I am either close or way off with this: http://paste.openstack.org/show/604728/
19:55 vinsh joined #puppet-openstack
19:56 mwhahaha colonwq: so the way i'd do it is mock out the response with one of the capital letters then have a test for the exists on a new resource with the opposite case
19:56 ntpttr_ joined #puppet-openstack
19:57 colonwq mwhahaha, but that makes much more sense than what I was doing.
20:07 pradk mwhahaha, need your input on something
20:07 * mwhahaha hides
20:08 pradk haha
20:08 pradk we have a way to disable ceilometer api using composable roles - https://github.com/openstack/tripleo-heat-templates/blob/master/environments/services/disable-ceilometer-api.yaml
20:08 pradk this should not deploy the api , which works great
20:09 pradk but* along wiht it we also skip user and service create because of this https://github.com/openstack/puppet-tripleo/blob/master/manifests/profile/base/keystone.pp#L249-L251
20:09 pradk the problem is we still want to keep ceilometer user
20:09 pradk as other services use it
20:10 pradk so i thought i could do https://review.openstack.org/#/c/451533/
20:10 pradk but this will create the service too, which we dont want as that wont be a valid endpoint
20:10 pradk i could set the configure_endpoint param to false on ceilo::keystone::auth
20:11 mwhahaha yea that sounds like the best bet
20:11 mwhahaha or only configure endpoint if ceilometer_api_enabled
20:11 pradk but with composable roles that tricky, if this was default to false, i can set to true in puppet/services/ceilo-api and if we dont include it all works
20:11 pradk since its true its always configured
20:12 mwhahaha pradk: i thought there was a way to do THT trickery
20:12 mwhahaha sec
20:12 pradk i also explored this http://paste.openstack.org/show/604731/
20:12 pradk but that wont take in the hiera from tht correct? in else case
20:13 mwhahaha pradk: can't you do a service_config_settings in the ceilometer role to have it set on the controler/keystone role?
20:14 ntpttr_ joined #puppet-openstack
20:14 mwhahaha pradk: like this https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/services/aodh-base.yaml#L89-L95
20:14 pradk hmm https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/services/ceilometer-base.yaml#L128 ?
20:14 mwhahaha but set it to true
20:14 mwhahaha pradk: yea
20:15 mwhahaha so by default be false unless ceilometer api enabled
20:15 pradk well ceilometer api is always enabled in ocata.. we provide env file to turn it off
20:16 mwhahaha pradk: the end file could set a param
20:16 pradk hmm i see what you mean
20:17 pradk so the keystone block in service_config_settings: can be turned on/off?
20:18 mwhahaha yea
20:18 mwhahaha so if you set ceilometer::keystone::auth::configure_endpoint: {get_param: CeilometerApiEnabled} it would do it
20:19 mwhahaha where CeilometerApiEnabled is true by default and the disabling template would set it to false
20:21 pradk yea only thing is extra param.. i guess thats no biggie if we pre-config the env anyway
20:22 pradk we still need the puppet-tripleo change
20:22 pradk so on top of that we add this
20:22 pradk hmm
20:22 mwhahaha no we wouldn't
20:22 mwhahaha oh yea we would
20:22 pradk yea other wise it wont include tha class
20:22 mwhahaha so you still need the auth but the endpoint creation is seperate and controllable via THT
20:23 pradk yep, cool i'll try this
20:23 pradk thx mwhahaha
20:23 pradk mwhahaha++
20:23 mwhahaha np
20:24 ntpttr_ joined #puppet-openstack
20:29 openstackgerrit joined #puppet-openstack
20:29 openstackgerrit Vitaliy Nogin proposed openstack/puppet-freezer master: Add freezer anchors and dependency management  https://review.openstack.org/451552
20:30 ntpttr_ joined #puppet-openstack
20:36 ntpttr_ joined #puppet-openstack
20:44 ntpttr_ joined #puppet-openstack
20:48 openstackgerrit Vitaliy Nogin proposed openstack/puppet-freezer master: Tune db sync  https://review.openstack.org/451558
20:50 rwsu joined #puppet-openstack
20:50 ntpttr_ joined #puppet-openstack
20:56 ntpttr_ joined #puppet-openstack
20:57 acormier joined #puppet-openstack
21:02 ntpttr_ joined #puppet-openstack
21:04 xarses_ joined #puppet-openstack
21:05 andreww joined #puppet-openstack
21:06 andreww joined #puppet-openstack
21:06 openstackgerrit Vitaliy Nogin proposed openstack/puppet-freezer master: Tune db sync  https://review.openstack.org/451558
21:08 ducttape_ joined #puppet-openstack
21:08 ntpttr_ joined #puppet-openstack
21:10 openstackgerrit Vitaliy Nogin proposed openstack/puppet-freezer master: Tune db sync  https://review.openstack.org/451558
21:14 ntpttr_ joined #puppet-openstack
21:17 openstackgerrit Merged openstack/puppet-ceilometer master: deprecate dispatcher::gnocchi::archive_policy  https://review.openstack.org/448587
21:17 openstackgerrit Merged openstack/puppet-ceilometer master: Password should be secured  https://review.openstack.org/450623
21:20 rwsu joined #puppet-openstack
21:20 ntpttr_ joined #puppet-openstack
21:23 openstackgerrit Vitaliy Nogin proposed openstack/puppet-freezer master: Tune db sync  https://review.openstack.org/451558
21:24 ccamacho joined #puppet-openstack
21:26 ntpttr_ joined #puppet-openstack
21:32 ntpttr_ joined #puppet-openstack
21:33 ansmith joined #puppet-openstack
21:38 ntpttr_ joined #puppet-openstack
21:41 openstackgerrit Vitaliy Nogin proposed openstack/puppet-freezer master: Tune db sync  https://review.openstack.org/451558
21:44 ntpttr_ joined #puppet-openstack
21:44 ducttape_ joined #puppet-openstack
21:50 ntpttr_ joined #puppet-openstack
21:53 openstackgerrit Vitaliy Nogin proposed openstack/puppet-freezer master: Tune db sync  https://review.openstack.org/451558
21:54 pradk mwhahaha, i can add it to keystone settings, seems weird to add ceilo param to kestone .. i worry we might forget to set one or other due to dup :/
21:54 mwhahaha either that or make it global
21:56 ntpttr_ joined #puppet-openstack
22:33 colonwq joined #puppet-openstack
22:35 rwsu joined #puppet-openstack
22:55 xarses_ joined #puppet-openstack
22:56 xarses_ joined #puppet-openstack
22:59 andreww joined #puppet-openstack
22:59 andreww joined #puppet-openstack
23:08 openstackgerrit Merged openstack/puppet-panko stable/ocata: Remove explicit log file for panko  https://review.openstack.org/451537
23:12 acormier joined #puppet-openstack
23:12 ntpttr_ joined #puppet-openstack
23:14 colonwq joined #puppet-openstack
23:28 ntpttr_ joined #puppet-openstack
23:46 ntpttr_ joined #puppet-openstack

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