Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2014-01-23

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

All times shown according to UTC.

Time Nick Message
23:00 ryanycoleman joined #puppet-openstack
23:36 xarses joined #puppet-openstack
23:41 Guest90651 joined #puppet-openstack
23:43 pzimmer left #puppet-openstack
00:09 ryanycoleman joined #puppet-openstack
00:31 Guest90651 joined #puppet-openstack
00:42 ryanycoleman joined #puppet-openstack
00:50 hogepodge joined #puppet-openstack
01:16 prad_ joined #puppet-openstack
01:16 ianw joined #puppet-openstack
01:17 rharrison joined #puppet-openstack
01:19 xingchao joined #puppet-openstack
01:22 rmoe joined #puppet-openstack
01:56 ryanycoleman joined #puppet-openstack
02:13 rongze joined #puppet-openstack
02:30 starmer joined #puppet-openstack
03:30 xarses joined #puppet-openstack
04:19 ryanycoleman joined #puppet-openstack
04:48 rongze joined #puppet-openstack
05:22 rongze joined #puppet-openstack
05:54 rongze joined #puppet-openstack
06:01 saneax__ joined #puppet-openstack
06:02 14WABVY7A joined #puppet-openstack
06:35 rongze joined #puppet-openstack
06:39 tnoor joined #puppet-openstack
06:39 dgollub joined #puppet-openstack
06:39 rongze_ joined #puppet-openstack
06:42 tnoor1 joined #puppet-openstack
07:41 EmilienM good morning
07:48 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-keystone: Deprecate bind_host parameter  https://review.openstack.org/68281
07:53 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-glance: Update glance_image provider with API v2  https://review.openstack.org/67768
08:03 bauzas joined #puppet-openstack
08:23 britthou_ joined #puppet-openstack
08:57 mmagr joined #puppet-openstack
09:11 rongze_ joined #puppet-openstack
09:14 rongze joined #puppet-openstack
09:14 derekh joined #puppet-openstack
10:06 openstackgerrit Sebastien Badia proposed a change to stackforge/puppet-ceilometer: Fix package ceilometer-alarm type error on Debian  https://review.openstack.org/68588
10:22 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Add heat templates for 2_role scenario  https://review.openstack.org/67992
10:45 rongze joined #puppet-openstack
11:16 rongze joined #puppet-openstack
11:39 michchap_ joined #puppet-openstack
11:57 rongze joined #puppet-openstack
12:01 morazi joined #puppet-openstack
12:38 michchap joined #puppet-openstack
13:42 csschwe joined #puppet-openstack
13:46 csschwe joined #puppet-openstack
13:53 pzimmer joined #puppet-openstack
14:04 finnx joined #puppet-openstack
14:38 michchap joined #puppet-openstack
14:50 EmilienM mgagne: hey, since you would do bug triage. I think https://bugs.launchpad.net/​puppet-neutron/+bug/1264202 is fixed
15:10 openstackgerrit A change was merged to stackforge/puppet-neutron: Fixing neutron::server deprecated warnings  https://review.openstack.org/66968
15:20 dmsimard joined #puppet-openstack
15:23 dmsimard Hello #puppet-openstack *pokes with stick* you are pretty calm since the holidays
15:24 EmilienM dmsimard: i need help
15:24 EmilienM dmsimard: i fail to backport a patch we need in stable/havana
15:24 EmilienM https://review.openstack.org/#/c/66968
15:24 EmilienM dmsimard: i have a git conflict that I solve, but I can't amend then... Could you try please?
15:25 dmsimard you want to backport https://review.openstack.org/#/c/66968 in havana ? I can try
15:25 EmilienM dmsimard: thanks.
15:26 dmsimard EmilienM: Can you look at my ceilometer patches <3
15:26 EmilienM dmsimard: i can, right now
15:31 tnoor joined #puppet-openstack
15:31 dmsimard EmilienM: The problem with the backport is that stable/havana is pretty different than master - which the change is based against
15:31 dmsimard EmilienM: Would need to backport other stuff too IMO
15:32 prad_ joined #puppet-openstack
15:32 dmsimard EmilienM: I'll check what I cna do
15:33 tnoor1 joined #puppet-openstack
15:33 EmilienM dmsimard: i did the review
15:34 dmsimard Do we want to backport master to havana ? I am not familiar enough with neutron but I can do it
15:35 EmilienM dmsimard: yes I want to backport the patch I told you from master to havana.
15:35 EmilienM and i fail
15:39 michchap joined #puppet-openstack
15:42 csschwe joined #puppet-openstack
15:42 dmsimard havana depends on bodepd's puppet-vswitch and master depends on stackforge's
15:47 EmilienM dmsimard: uh? i already did some backports in puppet-neutron without issues
15:49 dmsimard EmilienM: Meeting be back in a bit
16:03 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Use a public ntp server  https://review.openstack.org/68450
16:19 prad joined #puppet-openstack
16:40 michchap joined #puppet-openstack
16:43 dmsimard EmilienM: Yeah so I tried to backport but I don't have enough context and knowledge on the changes to make sure it doesn't break anything :(
16:43 EmilienM dmsimard: are you able to commit?
16:43 EmilienM I'm not.
16:44 dmsimard Well I don't really have a commit to test on right now
16:45 dmsimard I can try and fix the concerns you have on my review and try to submit another patchset, see what happens
16:46 EmilienM dmsimard: please commit & push it
16:46 EmilienM dmsimard: I just want to see if jenkins is happy
16:46 dmsimard commit and push what ? The backports ?
16:46 EmilienM dmsimard: git review stable/havana
16:47 EmilienM dmsimard: yes the backports.
16:48 dmsimard EmilienM: But I have no idea if what I am comitting will break stuff :(
16:48 EmilienM dmsimard: again... i won't merge it.
16:48 dmsimard EmilienM: ok
16:49 EmilienM dmsimard: I just want to see if IT WORKS
16:49 EmilienM because I tried some times to backport and I have git errors I can't solve
16:49 EmilienM dmsimard: I can -2 the patch if you want
16:49 EmilienM I just want to see jenkins result and your patch cherry picked.
16:50 csschwe joined #puppet-openstack
16:55 dmsimard EmilienM: Ehh, I'll try a cherry pick - rebasing against master works but ends up crazy: # Your branch and 'origin/stable/havana' have diverged, and have 49 and 16 different commits each, respectively.
16:58 rmoe joined #puppet-openstack
16:59 angdraug joined #puppet-openstack
17:00 openstackgerrit David Moreau Simard proposed a change to stackforge/puppet-neutron: Fixing neutron::server deprecated warnings  https://review.openstack.org/68696
17:00 dmsimard EmilienM: ^
17:00 EmilienM dmsimard: thx, having a look
17:01 dmsimard EmilienM: I was trying to backport/rebase everything so the cherry pick wouldn't conflict, scary stuff
17:01 EmilienM dmsimard: good one :)
17:01 EmilienM heh
17:01 dmsimard EmilienM: It looks like it's good
17:02 dmsimard EmilienM: Feel free to let it through if you want, free commit for me ? :P
17:02 EmilienM dmsimard: thanks, +2
17:06 marun joined #puppet-openstack
17:10 starmer joined #puppet-openstack
17:12 dmsimard EmilienM: About shared_examples versus context, I tried context first but I wasn't able to get it to work ? It seems like spec tests are failing cause it's not picking up an OS Family ? Ex: http://paste.openstack.org/show/61770/
17:13 dmsimard EmilienM: My spec skills are not exactly awesome
17:13 dprince joined #puppet-openstack
17:41 michchap joined #puppet-openstack
17:41 michchap joined #puppet-openstack
17:43 xarses joined #puppet-openstack
17:48 csschwe joined #puppet-openstack
17:56 ryanycoleman joined #puppet-openstack
17:56 openstackgerrit A change was merged to stackforge/puppet-ceilometer: Fix package ceilometer-alarm type error on Debian  https://review.openstack.org/68588
17:57 dmsimard EmilienM: do you know what I should do ? ^
18:03 mgagne dmsimard: need more context to your example =) (actual tests)
18:18 sputnik13net joined #puppet-openstack
18:21 dmsimard mgagne: it's for https://review.openstack.org/#/c/68198/
18:24 mgagne dmsimard: shared_examples are just a container for tests. they still need to be included in some ways using include_examples (https://www.relishapp.com/rspec/rspec-co​re/docs/example-groups/shared-examples) or the special helper we created: it_configures
18:24 mgagne dmsimard: by including them, you inherit params and facts setup by the parent context.
18:25 dmsimard dmsimard: the shared_examples work, EmilienM was suggesting using context() instead
18:27 dmsimard mgagne: In puppet-nova, where the code in this commit is largely taken from, it's not a shared_example, it's a describe under a describe that sets fact osfamily to debian
18:41 michchap joined #puppet-openstack
18:42 michchap joined #puppet-openstack
18:44 mgagne dmsimard: whatever works. shared_examples clearly don't work the way you used it for the reasons mentioned above =)
18:53 mgagne joined #puppet-openstack
18:53 badiane_ka joined #puppet-openstack
18:55 mgagne1 joined #puppet-openstack
19:01 dmsimard mgagne: Oh crap I understand now
19:02 dmsimard mgagne: Thanks for that link
19:02 xarses joined #puppet-openstack
19:07 ryanycoleman joined #puppet-openstack
19:21 ryanycoleman joined #puppet-openstack
19:25 openstackgerrit Chris Hoge proposed a change to stackforge/puppet-heat: Release 3.0.0  https://review.openstack.org/68731
19:35 ryanycoleman joined #puppet-openstack
19:41 hogepodge joined #puppet-openstack
19:43 michchap joined #puppet-openstack
19:43 openstackgerrit Sumit Naiksatam proposed a change to stackforge/puppet-openstack: Pointing openstack modules to stable/havana.  https://review.openstack.org/68736
19:45 openstackgerrit Sumit Naiksatam proposed a change to stackforge/puppet-openstack: Pointing openstack modules to stable/havana.  https://review.openstack.org/68736
19:47 finnx joined #puppet-openstack
19:54 dpotyekhin joined #puppet-openstack
20:19 openstackgerrit David Moreau Simard proposed a change to stackforge/puppet-ceilometer: Remove log_dir from params and make logs configurable in init  https://review.openstack.org/68198
20:20 dmsimard mgagne: ^ was hard for me to wrap my head around shared_examples, contexts, merge to params and stuff but I did it !
20:20 mgagne dmsimard: cool
20:20 dmsimard mgagne: Shit, I submitted garbage, hang on
20:21 openstackgerrit David Moreau Simard proposed a change to stackforge/puppet-ceilometer: Remove log_dir from params and make logs configurable in init  https://review.openstack.org/68198
20:26 morazi joined #puppet-openstack
20:39 hogepodge_ joined #puppet-openstack
20:43 michchap joined #puppet-openstack
20:44 michchap joined #puppet-openstack
20:52 marun joined #puppet-openstack
20:54 sputnik13net joined #puppet-openstack
20:56 hogepodge joined #puppet-openstack
20:57 csschwe joined #puppet-openstack
20:58 rmoe_ joined #puppet-openstack
21:10 EmilienM dmsimard: sorry i was afk :(
21:10 EmilienM i see mgagne helped you :-)
21:10 xarses joined #puppet-openstack
21:14 ryanycoleman joined #puppet-openstack
21:20 dmsimard EmilienM: Yup, I think my last patch set should be good
21:21 EmilienM dmsimard: i'll review it tomorrow morning :-)
21:21 dmsimard EmilienM: thanks, hope we can merge the backports too (I'll fix these patchsets as well), I need them
21:21 ryanycoleman joined #puppet-openstack
21:25 EmilienM dmsimard: yep, me too I need it
21:25 EmilienM dmsimard: maybe you and mgagne could have a look at https://review.openstack.org/#/c/67726/
21:26 dmsimard dmsimard: im not core :(
21:27 dmsimard er
21:27 dmsimard EmilienM: ^
21:27 marun joined #puppet-openstack
21:28 EmilienM dmsimard: i was not core and i did reviews
21:28 EmilienM dmsimard: +1 is a good review also
21:28 dmsimard EmilienM: Oh, yeah, just trying to say I won't be able to put it through
21:28 EmilienM dmsimard: your feedback would be more than welcome
21:37 ryanycoleman joined #puppet-openstack
21:38 odyi joined #puppet-openstack
21:44 michchap joined #puppet-openstack
22:01 angdraug joined #puppet-openstack
22:08 ryanycoleman joined #puppet-openstack
22:12 ryanycoleman joined #puppet-openstack
22:22 rowleyaj joined #puppet-openstack
22:31 michchap joined #puppet-openstack
22:31 michchap joined #puppet-openstack
22:42 michchap joined #puppet-openstack
22:46 starmer joined #puppet-openstack

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