Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2013-11-18

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

All times shown according to UTC.

Time Nick Message
00:01 dachary ceph::mon needs a way to tell you when you ask it to create a monitor at a place that is inconsistent with the way ceph --show-config --name $id sees it
00:02 dachary but it would be perfectly ok for it to raise this error the second time it passes and not the first time
00:02 dachary it would be very bad for ceph::mon to remain silent when doing something that can so easily be verified to be inconsistent
00:03 dalgaaf_ why would ceph-config care about it ... there is only one mon_data in the config since the global value is overwritten by the mon_data defined for your mon
00:04 dachary [mon.a] mon-data: /foobar
00:04 dachary will override it
00:04 dalgaaf_ jepp
00:04 dachary there is room for error and it's easy to check and it will make the maintenance easier
00:05 dachary with a few safeguards based on --show-config, we stay out of trouble
00:05 dalgaaf_ but you can't verify the global setting since it's no longer available (since overwritten in the [mon] section) except you grep the conf file
00:06 dachary you can verify that the mon.a will try to get data from a place that's different from the ceph::mon argument
00:06 dalgaaf_ --show-config only returns the current settings of the mon ... how do you get the global defaults to be verified ? or did I get something wrong?
00:06 michchap joined #puppet-openstack
00:07 dachary dalgaaf_: I'm not sure we can very everything. But do we need to ?
00:07 dachary dalgaaf_: you made me realize something important
00:07 dachary I thought of --show-config as a source for the config values and that there was no way around it
00:07 dalgaaf_ You want to verify if the mon path of the running mon is the same as in the config?
00:07 dachary but it is a way to check that the values you set are interpreted as you expect by ceph
00:08 dachary dalgaaf_: yes
00:08 dalgaaf_ --show-config dumps what you get with ceph --admin-daemon /var/run/ceph/ceph-osd.$id.asok config show
00:08 dachary yes
00:08 dalgaaf_ that's the value of the running daemon
00:08 dalgaaf_ and not of the config itself ... and the question is: does it make sense to verify that?
00:09 dalgaaf_ if puppet deployed that mon it should be correct ... or not?
00:09 dalgaaf_ no need to  verify
00:09 dachary dalgaaf_:  ceph --id FOOBAR --show-config --conf /dev/null
00:10 dachary wil work even when there is no daemon running
00:10 dachary http://dachary.org/?p=2515
00:10 dachary dalgaaf_: if it relied on a running daemon, ... I agree it would be pointless ;-)
00:11 dalgaaf_ jepp ... since even this changes the values of mon_data: mon_data = /var/lib/ceph/mon/ceph-FOOBAR
00:12 dalgaaf_ no easy way to get on the real defaults as defined by src/common/config_opts.h
00:14 dachary what matters is that ceph{-mon,-osd} will show us the value actually used. We don't need to worry how it does it. And if it disagrees with that ceph::mon think it is ( for whatever reason ) we can just display fail("you set XXX but show-config disagrees and think it's YYY, go in debug mode")
00:14 dalgaaf_ in which case will this happen?
00:15 dachary the case above
00:15 dalgaaf_ ceph::mon { mon-data: /tmp } and ceph_config { globa/mon-data: /foobar }  ?
00:15 dalgaaf_ or which case?
00:16 dachary ceph::mon { mon-data: /tmp } and ceph_config { mon.a/mon-data: /foobar }
00:16 dachary probably don't need a facter for that
00:16 dachary exec {} will do
00:17 dachary dalgaaf_: I need to get some sleep, thanks a lot for taking the time. I'll get rid of the two pass tomorrow :-) And the facter.
00:17 dalgaaf_ only to understand that ... what would ceph::mon { mon-data: /tmp }  write to the config?
00:18 dalgaaf_ to global?
00:18 dalgaaf_ or to a particular [mon.x] section
00:18 dachary I don't think it should write any config.
00:18 dachary but for compatiblity reason with the centos scripts I think it needs to write the [mon.X] configs
00:18 dachary right ?
00:19 dalgaaf_ not sure ... if they use the init.v scritps it should simply work ... but I have to take a look at it
00:20 dalgaaf_ bye
00:20 dachary night
01:28 rongze joined #puppet-openstack
01:58 rongze joined #puppet-openstack
02:22 michchap joined #puppet-openstack
02:33 thumpba joined #puppet-openstack
02:33 rongze joined #puppet-openstack
02:33 thumpba joined #puppet-openstack
02:52 brdude joined #puppet-openstack
02:55 rongze joined #puppet-openstack
02:58 thumpba joined #puppet-openstack
03:10 thumpba joined #puppet-openstack
03:30 rongze joined #puppet-openstack
04:10 thumpba joined #puppet-openstack
05:18 rongze joined #puppet-openstack
05:53 rongze joined #puppet-openstack
06:04 EmilienM michchap:
06:04 EmilienM michchap: good morning :)
06:04 EmilienM i was reading your comment on https://review.openstack.org/#/c/54350/ and just let you know i followed model from other middlewares
06:15 rongze joined #puppet-openstack
06:27 marun joined #puppet-openstack
06:52 bogdando joined #puppet-openstack
07:19 rongze joined #puppet-openstack
07:19 michchap EmilienM: ok, It just seemed strange to me to have templates for single lines of config
07:41 dvanders joined #puppet-openstack
07:43 michchap joined #puppet-openstack
08:13 tnoor joined #puppet-openstack
08:19 openstackgerrit A change was merged to stackforge/puppet-horizon: Fix readme typos  https://review.openstack.org/56708
08:19 openstackgerrit A change was merged to stackforge/puppet-swift: Add Quota middlewares support  https://review.openstack.org/54350
08:34 mmagr joined #puppet-openstack
08:47 rongze_ joined #puppet-openstack
08:50 rongze joined #puppet-openstack
08:52 bogdando joined #puppet-openstack
08:59 rongze joined #puppet-openstack
09:23 bauzas joined #puppet-openstack
09:38 bauzas joined #puppet-openstack
09:41 dpyzhov left #puppet-openstack
09:45 derekh joined #puppet-openstack
10:03 bauzas joined #puppet-openstack
10:11 EmilienM fc__: if you could have a review also on https://review.openstack.org/#/c/54366/
10:22 piliman974 joined #puppet-openstack
10:42 marun joined #puppet-openstack
10:53 marun joined #puppet-openstack
10:56 piliman974 joined #puppet-openstack
11:13 michchap joined #puppet-openstack
11:23 michchap joined #puppet-openstack
11:48 SPruvost joined #puppet-openstack
11:49 SPruvost Hi, i'm an IT student, especially in the field of free software, and I try to learn some informations from projects like puppet-openstack :)
11:51 SPruvost Could one of you tell me how you divide the work between all of you on a project like that ?
11:58 Joaquim joined #puppet-openstack
12:07 LAnthony_ joined #puppet-openstack
12:09 LAnthony_ Hello M. dachary ! Are you awake ?
12:14 rongze joined #puppet-openstack
12:15 michchap joined #puppet-openstack
12:34 rongze_ joined #puppet-openstack
12:53 bauzas joined #puppet-openstack
13:17 bauzas joined #puppet-openstack
13:31 bauzas1 joined #puppet-openstack
13:35 bauzas joined #puppet-openstack
13:49 hbroyon joined #puppet-openstack
13:54 dprince joined #puppet-openstack
13:59 hbroyon Hello, someone can review and approve https://review.openstack.org/#/c/56770/  please ?
14:11 morazi joined #puppet-openstack
14:12 openstackgerrit A change was merged to stackforge/puppet-nova: Removes unnecessary api-paste.ini configuration  https://review.openstack.org/54306
14:53 blentz joined #puppet-openstack
14:55 hbroyon left #puppet-openstack
15:31 dmsimard joined #puppet-openstack
15:37 mgagne joined #puppet-openstack
15:42 openstackgerrit Rongze Zhu proposed a change to stackforge/puppet-ceph: Fix error link  https://review.openstack.org/56963
15:46 otherwiseguy joined #puppet-openstack
15:54 tnoor joined #puppet-openstack
16:02 tnoor joined #puppet-openstack
16:05 tnoor1 joined #puppet-openstack
16:08 openstackgerrit A change was merged to stackforge/puppet-ceph: Fix error link  https://review.openstack.org/56963
16:13 openstackgerrit Rongze Zhu proposed a change to stackforge/puppet-ceph: Add more dashes to match title line length  https://review.openstack.org/56976
16:33 pabelanger joined #puppet-openstack
16:33 pabelanger joined #puppet-openstack
16:47 xarses joined #puppet-openstack
17:05 hogepodge joined #puppet-openstack
17:06 rmoe joined #puppet-openstack
17:14 badiane_ka joined #puppet-openstack
17:22 brdude joined #puppet-openstack
17:33 dprince joined #puppet-openstack
17:45 thumpba joined #puppet-openstack
17:48 hogepodge joined #puppet-openstack
18:04 angdraug joined #puppet-openstack
18:11 EmilienM hogepodge: i ack your message on ml2 stuff
18:12 EmilienM doing funct testing asap
18:12 hogepodge ok. I saw your comment, so I didn't want to merge if you didn't feel fully ready for it.
18:12 hogepodge (well, I we would need another +2, but i still wanted to make sure you were ready)
18:15 EmilienM hogepodge: before merging, i want to be sure that works. I'm going to test it on real servers, but i need a little time
18:16 hogepodge Cool. You just let me know when you're ready. :-)
18:16 EmilienM yep, sure
18:16 EmilienM pretty sure packstack guys want it also :P
18:19 openstackgerrit A change was merged to stackforge/puppet-keystone: Fix remove_warnings  https://review.openstack.org/53661
18:38 dachary michchap: hi. I would like to set the IP of the GRE endpoint of the OVS switch deployed on a compute. Do you know from the top of your head in which yaml file that would be ?
18:43 dachary data_mappings/tenant_network_type/gre.yaml probably
18:43 RomainS joined #puppet-openstack
18:49 blackst0ff joined #puppet-openstack
18:54 pabelanger joined #puppet-openstack
18:57 hogepodge joined #puppet-openstack
19:02 rmoe_ joined #puppet-openstack
19:09 EmilienM dachary: yes
19:10 EmilienM (michchap is on australian time)
19:12 hogepodge joined #puppet-openstack
19:17 Tim joined #puppet-openstack
19:19 derekh joined #puppet-openstack
19:21 * Tim slaps blackst0ff around a bit with a large trout
19:31 hogepodge joined #puppet-openstack
19:42 rmoe joined #puppet-openstack
19:51 thumpba joined #puppet-openstack
19:53 Tim joined #puppet-openstack
19:53 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-cinder: Add Swift as backup backend support  https://review.openstack.org/56305
19:54 EmilienM fc__: hogepodge ^ updated with your comments
19:55 EmilienM dachary: about https://review.openstack.org/​#/c/56302/2/manifests/glance.pp
19:55 EmilienM dachary: the glance_host is kept for backward
19:55 EmilienM the new flag is the array i set 'glance_api_servers'
19:56 EmilienM dachary: if you look at cinder code in cinder/image/glance.py
19:57 EmilienM cinder/image/glance.py
19:57 EmilienM oops
19:57 EmilienM for api_server in CONF.glance_api_servers
19:58 dachary ok
19:58 EmilienM dachary: glance_api_servers = $glance_host:$glance_port
19:58 dachary URL ?
19:58 EmilienM so i suggest keeping my design to allow easy arrays
19:58 EmilienM sure
19:58 EmilienM a sec
19:59 EmilienM dachary: https://github.com/openstack/cinder/bl​ob/master/cinder/common/config.py#L97
20:00 dachary the other thing I suggested is when the puppet module provide defaults that are different from the cinder defaults. When this is the case it should be explained: why is the puppet default different from the cinder default ?
20:00 EmilienM https://github.com/openstack/cinder/b​lob/master/cinder/image/glance.py#L97
20:00 EmilienM dachary: i always set default values from conf.sample in OpenStack
20:00 EmilienM afik everybody does
20:01 dachary why not just undef ? Is there a value in copying them over to the puppet module ? ( localhost != https://github.com/openstack/cinder/bl​ob/master/cinder/common/config.py#L88 )
20:01 dachary this is error prone, don't you think ?
20:02 EmilienM dachary: i could set undef, but i set default values for spec testing
20:02 EmilienM maybe i'm wrong
20:02 EmilienM what do you think other folks ?
20:02 dachary why does it help for spec tests ?
20:03 dachary What I see ( all over the place ) are subtle deviations from the actual defaults that are ( at best ) confusing.
20:04 dachary the  localhost != https://github.com/openstack/cinder/bl​ob/master/cinder/common/config.py#L88  is one example
20:04 dachary EmilienM: that being said, I don't feel strongly about all this. Only you asked for my review and I give it to you ;-)
20:04 mgagne joined #puppet-openstack
20:04 mgagne joined #puppet-openstack
20:04 dachary or maybe you did not
20:05 dachary in which case feel free to ignore me
20:07 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-swift: Add Bulk middleware support  https://review.openstack.org/54366
20:08 EmilienM dachary: no, i take in consideration your review
20:09 EmilienM dachary: is it fine if I set undef for glance_api_servers ?
20:10 dachary what do you mean ?
20:12 EmilienM dachary: if i set undev for glance_api_servers and let user create it
20:12 EmilienM since default value works
20:12 EmilienM i always think at the end user, making an error in the manifest...
20:13 xarses dachary: http://projects.puppetlabs.com/issues/18138 // http://projects.puppetlabs.com/issues/21478 << Puppet bugs to support that it dosn't support instances
20:13 xarses service instances through the upstart provider even
20:16 dachary xarses: thanks ;-)
20:17 dachary EmilienM: you think a default of undef will create problems ? I'm not sure I understand your question.
20:17 EmilienM dachary: no i ask if i change my patch with undef for glance_api_servers flag, are you fine with this ?
20:18 dachary I would be perfectly fine with undef, yes.
20:18 EmilienM dachary: perfect, thx
20:19 xarses darchy, in params, should we if $::operatingsystem == 'Ubuntu' { Service { provider => 'debian' }}
20:35 hogepodge joined #puppet-openstack
20:38 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-cinder: Add Glance support  https://review.openstack.org/56302
20:38 EmilienM dachary: done ^
20:45 xarses EmilienM: is there a ref doc for that?
20:54 EmilienM xarses: for what ?
20:54 EmilienM for params stuff ?
20:57 hogepodge joined #puppet-openstack
20:58 xarses ya like a description of the config options, defaults, and what that accomplishes. something like https://wiki.openstack.org​/wiki/Cinder-multi-backend or http://docs.openstack.org/trunk/config-r​eference/content/vmware-vmdk-driver.html
21:00 marun joined #puppet-openstack
21:09 openstackgerrit A change was merged to stackforge/puppet-neutron: Rename device_driver in neutron::agents::lbaas  https://review.openstack.org/56701
21:26 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-cinder: Add Glance support  https://review.openstack.org/56302
21:26 EmilienM mgagne: with your remarks ^
21:27 mgagne EmilienM: thanks for that one. I'll be able move to cinder::glance instead of an internal manifest with cinder_config ;)
21:27 EmilienM mgagne: me too, ahah
21:27 mgagne dude, you missed one option :P
21:28 mgagne glance_api_servers ^^'
21:28 EmilienM damn, i should go sleeping instead of making that
21:29 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-cinder: Add Glance support  https://review.openstack.org/56302
21:29 EmilienM the last one of the day
21:29 xarses EmilienM: I'd still like to see refrennce docs, this looks like a volume driver, but I'd like to learn more
21:29 mgagne EmilienM: forgot to commit?
21:30 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-cinder: Add Glance support  https://review.openstack.org/56302
21:30 EmilienM forgot to go at bed
21:30 EmilienM cheers !
21:30 mgagne jesus
21:30 xarses ;)
21:31 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-cinder: Add Swift as backup backend support  https://review.openstack.org/56305
21:31 EmilienM mgagne: anticipation ^^^^
21:31 EmilienM \o/
21:32 EmilienM xarses: i'll have a look tomorrow, sorry
21:34 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-cinder: Add Glance support  https://review.openstack.org/56302
21:48 mjblack_ joined #puppet-openstack
21:54 mjblack joined #puppet-openstack
23:12 hogepodge joined #puppet-openstack
23:24 openstackgerrit Andrew Woodward proposed a change to stackforge/puppet-ceph: Add ceph::repo RedHat support  https://review.openstack.org/57063
23:28 openstackgerrit Andrew Woodward proposed a change to stackforge/puppet-ceph: Add ceph::repo RedHat support  https://review.openstack.org/57063
23:42 openstackgerrit Andrew Woodward proposed a change to stackforge/puppet-ceph: Change 4 space indent to 2  https://review.openstack.org/57065
23:49 xarses anyone around?
23:50 mjblack joined #puppet-openstack

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