Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2013-10-30

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

All times shown according to UTC.

Time Nick Message
00:05 michchap joined #puppet-openstack
00:08 tnoor1 joined #puppet-openstack
00:10 tnoor2 joined #puppet-openstack
00:17 xarses Initial commit for multi-backend https://github.com/xarses/puppet-cinder/comp​are/bp;cinder-volume-multi-backend?expand=1
00:30 michchap_ joined #puppet-openstack
00:39 openstackgerrit A change was merged to stackforge/puppet-ceph: development installation instructions  https://review.openstack.org/54034
00:47 michchap joined #puppet-openstack
00:50 bitblt joined #puppet-openstack
00:51 bitblt left #puppet-openstack
00:53 openstackgerrit Michael Chapman proposed a change to stackforge/puppet-openstack: Add keystone dependencies to neutron resources  https://review.openstack.org/53980
00:53 openstackgerrit A change was merged to stackforge/puppet-openstack: Prevent dollar signs and single quotes in variables from being removed by shell  https://review.openstack.org/51314
01:19 xarses joined #puppet-openstack
01:23 rmoe joined #puppet-openstack
01:41 comptona joined #puppet-openstack
01:47 comptona joined #puppet-openstack
01:48 openstackgerrit Dan Prince proposed a change to stackforge/puppet-neutron: Run neutron-db-manage upgrade.  https://review.openstack.org/50162
01:53 rongze joined #puppet-openstack
01:56 rongze joined #puppet-openstack
02:00 comptona joined #puppet-openstack
02:02 xingchao joined #puppet-openstack
02:03 rongze joined #puppet-openstack
02:08 rmoe joined #puppet-openstack
02:28 openstackgerrit Michael Chapman proposed a change to stackforge/puppet-openstack: Add keystone dependencies to neutron resources  https://review.openstack.org/53980
02:55 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-swift: Pin puppetlabs-concat fixture to 1.0.0  https://review.openstack.org/54464
02:58 lnxnut joined #puppet-openstack
03:13 openstackgerrit A change was merged to stackforge/puppet-swift: Pin puppetlabs-concat fixture to 1.0.0  https://review.openstack.org/54464
03:14 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-swift: Revert "Fix compatibility with latest puppetlabs-concat"  https://review.openstack.org/54465
03:43 tnoor1 joined #puppet-openstack
03:43 lnxnut joined #puppet-openstack
03:45 tnoor2 joined #puppet-openstack
04:19 openstackgerrit Michael Chapman proposed a change to stackforge/puppet-neutron: Ensure keystone endpoint set before service starts  https://review.openstack.org/54473
04:19 openstackgerrit Michael Chapman proposed a change to stackforge/puppet-neutron: Ensure keystone endpoint set before service starts  https://review.openstack.org/54473
04:21 e1mer joined #puppet-openstack
04:25 openstackgerrit Michael Chapman proposed a change to stackforge/puppet-neutron: Ensure keystone endpoint set before service starts  https://review.openstack.org/54473
04:43 rongze joined #puppet-openstack
04:43 openstackgerrit Michael Chapman proposed a change to stackforge/puppet-nova: Ensure keystone endpoint set before service starts  https://review.openstack.org/54474
04:47 openstackgerrit Michael Chapman proposed a change to stackforge/puppet-glance: Ensure keystone endpoint set before service starts  https://review.openstack.org/54476
04:54 openstackgerrit Michael Chapman proposed a change to stackforge/puppet-glance: Ensure keystone endpoint set before service starts  https://review.openstack.org/54476
05:17 CoilDomain joined #puppet-openstack
06:28 rongze joined #puppet-openstack
07:12 mattymo joined #puppet-openstack
08:21 mmagr joined #puppet-openstack
08:42 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-swift: Add Quota middlewares support  https://review.openstack.org/54350
08:53 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-swift: Add Quota middlewares support  https://review.openstack.org/54350
08:54 mjeanson joined #puppet-openstack
09:02 EmilienM xingchao: ping
09:05 derekh joined #puppet-openstack
09:18 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-swift: Add Bulk middleware support  https://review.openstack.org/54366
09:26 mjblack joined #puppet-openstack
09:28 xingchao EmilienM: I'm here
09:34 EmilienM xingchao: hey, i can see that current patchs on swift are blocked by the problem with concat, right ?
09:37 xingchao EmilienM: yes, see this change: https://review.openstack.org/#/c/54465/
09:38 EmilienM xingchao: ok, so i should wait the storm for my patchs on swift, maybe could you have a quick review ?
09:41 xingchao EmilienM: ok, I will review it and give any comment when I'm free, I'm working now  :)
09:42 EmilienM xingchao: sure, have a nice day
11:53 morazi joined #puppet-openstack
12:14 dprince joined #puppet-openstack
12:19 mmagr joined #puppet-openstack
12:23 rongze joined #puppet-openstack
13:03 e1mer joined #puppet-openstack
13:06 e1mer joined #puppet-openstack
13:10 e1mer joined #puppet-openstack
13:12 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-neutron: Add ML2 plugin support  https://review.openstack.org/48289
13:13 e1mer joined #puppet-openstack
13:15 rcrit joined #puppet-openstack
13:16 mjeanson joined #puppet-openstack
13:17 e1mer joined #puppet-openstack
13:19 otherwiseguy joined #puppet-openstack
13:24 e1mer joined #puppet-openstack
13:30 e1mer joined #puppet-openstack
13:36 e1mer joined #puppet-openstack
13:41 rongze joined #puppet-openstack
13:44 dmsimard joined #puppet-openstack
13:52 xingchao_ joined #puppet-openstack
13:54 dmsimard joined #puppet-openstack
13:59 mmagr joined #puppet-openstack
14:09 bogdando joined #puppet-openstack
14:40 dmsimard joined #puppet-openstack
14:41 bandas joined #puppet-openstack
14:43 sileht joined #puppet-openstack
14:45 xingchao joined #puppet-openstack
14:50 xingchao_ joined #puppet-openstack
14:51 otherwiseguy joined #puppet-openstack
15:00 mgagne dprince: ping
15:00 dprince mgagne: hi
15:01 mgagne dprince: hi
15:01 mgagne dprince: question regarding puppetlabs-concat
15:01 dprince mgagne: okay
15:01 mgagne dprince: we pinned fixtures for our rspec tests to 1.0.0. I'm about to revert the fix I made yesterday.
15:01 mgagne dprince: my understanding is that you updated puppetlabs-concat on smokestack to latest version.
15:02 dprince mgagne: I did to keep things running.
15:02 mgagne dprince: yes, I understand
15:02 dprince mgagne: Do you mind if I approve the revert then?
15:02 dprince mgagne: So I can coordinate things (and avoid further breakage)
15:02 mgagne dprince: I was waiting for you to review as I saw reverting the fix broken the build
15:03 dprince mgagne: Excellent. Thanks for the heads up.
15:03 mgagne dprince: you are testing against stable/grizzly?
15:03 dprince mgagne: no. Only master (what will be Icehouse at this point)
15:04 mgagne dprince: ok, I'll revert fix in stable/grizzly too once approved in master
15:05 mgagne dprince: let me know when you are ready
15:07 dprince mgagne: https://review.openstack.org/#/c/54465/ (I approved it)
15:08 dprince mgagne: changes in SmokeStack are reverted as well.
15:08 mgagne dprince: about smokestack, is it still reporting results in comments for puppet module projects?
15:08 dprince mgagne: So we should now be standardized on the 1.0.0 puppet-concat I think.
15:08 mgagne dprince: I had to dig myself in result page to find results
15:08 mgagne dprince: yes
15:08 dprince mgagne: yes. It reports back on success only ATM.
15:08 mgagne dprince: of smokestack that is
15:08 dprince mgagne: I can change that easily.
15:08 dprince dprince: of smokestack that is
15:09 * dprince copy paste error
15:09 mgagne dprince: I would appreciate it. Good news are good but with tests, bad news are more important I say =)
15:09 openstackgerrit A change was merged to stackforge/puppet-swift: Revert "Fix compatibility with latest puppetlabs-concat"  https://review.openstack.org/54465
15:10 dprince mgagne: Since SS isn't a gate I've been guarded about negative results. I think things are running well enough that we can probably enable them for puppet stuff though.
15:13 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-swift: Pin puppetlabs-concat fixture to 1.0.0  https://review.openstack.org/54561
15:13 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-swift: Revert "Fix compatibility with latest puppetlabs-concat"  https://review.openstack.org/54562
15:16 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-swift: Add Bulk middleware support  https://review.openstack.org/54366
15:17 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-swift: Add Quota middlewares support  https://review.openstack.org/54350
15:18 e1mer joined #puppet-openstack
15:21 mgagne dprince: mind approving that one too? https://review.openstack.org/#/c/54561/
15:22 dprince mgagne: np. done
15:23 mgagne dprince: thanks
15:24 openstackgerrit A change was merged to stackforge/puppet-swift: Pin puppetlabs-concat fixture to 1.0.0  https://review.openstack.org/54561
15:24 openstackgerrit A change was merged to stackforge/puppet-swift: Revert "Fix compatibility with latest puppetlabs-concat"  https://review.openstack.org/54562
15:28 bitblt joined #puppet-openstack
15:28 bitblt joined #puppet-openstack
15:29 bitblt left #puppet-openstack
15:48 mjeanson_ joined #puppet-openstack
15:56 dmsimard joined #puppet-openstack
16:00 lnxnut joined #puppet-openstack
16:01 dmsimard joined #puppet-openstack
16:23 rmoe joined #puppet-openstack
16:42 bogdando joined #puppet-openstack
16:53 xarses joined #puppet-openstack
17:05 dmsimard mgagne: If you +1 https://review.openstack.org/#/c/54176/ i'll merge it
17:14 openstackgerrit A change was merged to stackforge/puppet-ceph: ceph::repo integration tests  https://review.openstack.org/54176
17:17 mgagne bodepd_: ping
17:17 mgagne bodepd_: looks like the same problem we have with upstream puppet module versions: https://etherpad.openstack.org/p/iceho​use-summit-preemptively-integrate-the
17:29 xarses any thoughts on https://github.com/xarses/puppet-cinder/comp​are/bp;cinder-volume-multi-backend?expand=1 before i start on all the spec tests?
17:30 mgagne xarses: should I comment inline?
17:31 xarses however you feel =)
17:31 mgagne xarses: looks like I can't
17:31 xarses should i start a review?
17:31 mgagne xarses: could be an idea. you can benefit from automated tests and allow reviews
17:32 xarses didn't know if it would be appropriate at this point
17:32 mgagne xarses: you can mark it as "Work in progress" if work is still going on
17:32 mgagne xarses: nothing wrong in that
17:33 xarses just in the commit message?
17:33 mgagne xarses: you can mark change as "Work in progress" in the interface
17:33 xarses ok, so after i submit
17:34 mgagne xarses: yes, unfortunately, it's not a builtin feature of gerrit (yet) and git review does not allow you to do it when submitting
17:34 openstackgerrit Andrew Woodward proposed a change to stackforge/puppet-cinder: [bp] Add support for Cinder multi-backend  https://review.openstack.org/54591
17:34 mgagne xarses: see the button?
17:35 xarses yes
17:35 mgagne xarses: cool =)
17:36 prad joined #puppet-openstack
17:37 prad joined #puppet-openstack
17:40 xarses non-obvious though
17:42 starmer joined #puppet-openstack
17:46 rongze joined #puppet-openstack
17:48 mgagne xingchao_: ping (sleeping yet?)
17:49 xingchao_ mgagne: not yet :)
17:49 mgagne xingchao_: have experience with "boot from volume" or any similar setup where root volume is hosted on a centralized/distributed SAN?
17:50 mgagne xingchao_: I'm ready https://blueprints.launchpad.net/nova/+sp​ec/bring-rbd-support-libvirt-images-type and someone referred to UnitedStack regarding this aspect.
17:50 mattymo joined #puppet-openstack
17:52 xingchao_ mgagne: Hmm,  we have used this bp in our nova private codes
17:53 mgagne xingchao_: my question is: Are instances started by UnitedStack hosted/backed by a non-local storage backend? If so, which backend? Is it RBD?
17:54 xingchao_ mgagne: yes, it's hosted on rbd. Our nova instance, glance image and cinder volume backend is rbd.
17:56 mgagne xingchao_: I see. I'm looking around on how to have the benefit of "boot from volume" (non-local volume) without the hassle of "creating a volume from an image through Cinder" and than "booting an instance on this volume". It's a 2 steps process I wish to avoid or simplify.
17:56 xingchao_ mgagne: And nova compute don' need to copy image from glance, it boot directly from rbd. So the boot time is very short.
17:57 mgagne xingchao_: really? which version of OpenStack is used? do you mind glance images can have metadata about the "real" location of the image so nova doesn't have to download and copy it to a newly created volume?
17:58 mgagne mind -> mean
18:01 xingchao_ mgagne: This method can provide a natural way  for user to boot instance
18:01 xingchao_ mgagne: 2013.2.b2 and  some private patches
18:03 mgagne xingchao_: yes, that's what I'm looking for (natural way)
18:03 mgagne xingchao_: ok, I'll have to check Havana soon
18:03 mgagne xingchao_: is it leveraging this blueprint? https://blueprints.launchpad.net/gl​ance/+spec/multiple-image-locations
18:04 xingchao_ mgagne: you could ask more detailed questions to my colleague haomai, he is responsible for this bp :)
18:06 xingchao_ mgagne: I am not sure about this, I will ask him tomorrow
18:06 mgagne xingchao_: cool! thank you very much!
18:07 mgagne xingchao_: fixing this mess is one of the 56 tasks in the backlog ^^'
18:09 xingchao_ mgagne: so much todo, hard work :)
18:10 mgagne xingchao_: yep, I don't complain about the work itself, just that I don't have enough time to do it all. :P
18:10 xingchao_ mgagne: It's am 2:09, I will go to sleep now, see you :)
18:10 mgagne xingchao_: good night!
18:17 starmer joined #puppet-openstack
18:44 hogepodge joined #puppet-openstack
18:58 xarses mgagne: any comments for ?
18:58 xarses https://review.openstack.org/#/c/54591/
18:59 mgagne checking
19:14 mgagne xarses: do you have a blueprint for this?
19:14 xarses https://blueprints.launchpad.net/puppet-​cinder/+spec/cinder-volume-multi-backend
19:14 mgagne thanks
19:20 xarses thanks for the comments
19:42 bodepd_ I'm seeing this failure today: Error: Execution of '/sbin/ip link set br-tun up' returned 1: Cannot find device "br-tun"
19:42 xarses mgagne: how would i load the credentials from the env vars instead?
19:42 bodepd_ I have a recollection of that being added, but I can't find the patch
19:42 mgagne xarses: if using puppet, exec resource allows you to include env variables
19:43 mgagne bodepd_: keystone custom types
19:43 bodepd_ you mean neutron
19:43 bodepd_ that seems to cause a breakage
19:43 mgagne bodepd_: both? =)
19:44 bodepd_ it's not there
19:44 mgagne xarses: exec { 'whatever': environment => [] }
19:45 mgagne xarses, bodepd_: https://github.com/stackforge/puppet-keystone/b​lob/master/lib/puppet/provider/keystone.rb#L76
19:45 mgagne xarses: bodepd_: https://github.com/stackforge/puppet-neutron/b​lob/master/lib/puppet/provider/neutron.rb#L63
19:46 xarses hmm forgot about the admin token
19:46 bodepd_ mgagne: that is definitely not what I was asking about :)
19:47 mgagne bodepd_: oops, sorry, didn't see the 2nd message
19:47 mgagne or first
19:47 mgagne bodepd_: which module?
19:51 bodepd_ I'm going to file a ticket
19:58 pushok joined #puppet-openstack
20:04 bodepd_ mgagne:https://bugs.launchpad.net/​puppet-neutron/+bug/1246460
20:04 bodepd_ mgagne: probably not something you can help with b/c it's an ovs failure
20:04 bodepd_ I have a feeling there is a race condition
20:06 bodepd_ mgagne: is everything concat related resolved?
20:08 EmilienM bodepd_: in neutron, br-tun is created by ovs plugin agent
20:08 EmilienM automatically in gre mode
20:09 dachary EmilienM: is ovs-vsctl add-br atomic ?
20:09 EmilienM maybe you have an error, because the agent is not installed / started yet
20:09 EmilienM dachary: if you try to create a bridge that already exists, you have an error
20:10 dachary i.e. is it guarantee that the corresponding bridge exists after it returns or could it be created asynchronously ?
20:10 mgagne bodepd_: looks to be solved
20:11 EmilienM dachary: yes
20:11 dachary is there a doc / source that shows that ?
20:11 mgagne bodepd_: is it related to https://bugs.launchpad.net/​puppet-vswitch/+bug/1245973 ?
20:11 EmilienM dachary: let me 2 minutes please
20:12 dachary ok
20:12 dachary mgagne: that's me but it should not be realted to bodepd_ problem
20:15 openstackgerrit Andrew Woodward proposed a change to stackforge/puppet-cinder: [bp] Add support for Cinder multi-backend  https://review.openstack.org/54591
20:16 dachary my money is on the race condition ;-)
20:16 openstackgerrit Andrew Woodward proposed a change to stackforge/puppet-cinder: [bp] Add support for Cinder multi-backend  https://review.openstack.org/54591
20:17 EmilienM dachary: https://github.com/openstack/neutron​/blob/master/neutron/plugins/openvsw​itch/agent/ovs_neutron_agent.py#L694
20:19 EmilienM bodepd_: i've never done ip link up for a br-tun device
20:21 dachary Does this line show it's happening before the command returns ? I'm not familiar with neutron.
20:21 EmilienM bodepd_: puppet should rather checks if br-tun exists with "br-exists"
20:22 EmilienM dachary: this line is executed: when you start OVS agent with tunneling enabled & bridge not existing.
20:23 EmilienM bodepd_: i've never had this problem and we deploy compute with havana / ovs / gre here
20:24 EmilienM weird
20:24 dachary I'm not sure I understand how it relates to https://github.com/stackforge/puppet-vswitch/blob​/master/lib/puppet/provider/vs_bridge/ovs.rb#L15
20:25 dachary anyway, I'm out of my depths there ;-) good luck with the hunt !
20:29 tskden joined #puppet-openstack
20:29 beddari joined #puppet-openstack
20:32 hogepodge_ joined #puppet-openstack
20:38 openstackgerrit Andrew Woodward proposed a change to stackforge/puppet-cinder: [bp] Add support for Cinder multi-backend  https://review.openstack.org/54591
20:38 xarses mgagne: another review if you please
20:42 otherwiseguy joined #puppet-openstack
20:49 bodepd_ thanks everyone. I just saw it for the first time.
20:49 bodepd_ that is the ticket I was looking for mgagne, but it does not seem to be related
20:54 EmilienM bodepd_: i would need some help on my ml2 patch
20:54 EmilienM bodepd_: https://review.openstack.org/#/c/48289/
20:54 bodepd_ I can have a look
20:55 EmilienM bodepd_: it's not urgent, but i would like a review
20:55 bodepd_ what in particular?
20:55 EmilienM bodepd_: to have it merged soon, ml2 + l2 pop is so important
20:55 EmilienM bodepd_: the manifest
20:55 bodepd_ I see lint failures. and the errors look pretty pretty self explanatory
20:56 EmilienM bodepd_: i'm using define, to make loops
20:56 EmilienM not sure it's the right way
20:56 bodepd_ puppet doesn't recommend nesting them
20:56 bodepd_ just put it in it's own file
20:56 EmilienM oh, i see
20:56 bodepd_ the top scope ones are usually a mispelled variable
20:56 bodepd_ it's kind of like use strict :)
20:57 EmilienM bodepd_: i have to create one file per loop then ? in plugin dir ?
20:58 bodepd_ looking now.
20:58 bodepd_ you may just want to create a custom function
20:58 bodepd_ I would not use defined resource types for validation
20:58 EmilienM bodepd_: ok
20:58 marun joined #puppet-openstack
20:59 bodepd_ just put the defines in their own files
20:59 bodepd_ just a file per define
20:59 bodepd_ the code should mostly be the same
20:59 bodepd_ you just can't be sharing variables between scopes
20:59 EmilienM bodepd_: but you suggest me not using define
20:59 bodepd_ you have to pass them through interfaces
20:59 EmilienM ok
21:00 EmilienM bodepd_: what do you think about advanced testing like i do ?
21:01 EmilienM bodepd_: i.e. test if ip address is actually multicast valid, etc
21:02 bodepd_ EmilienM: I made inline ntes
21:02 bodepd_ notes
21:02 bodepd_ let me know if you have any questions
21:02 EmilienM bodepd_: thank you, i'll consider that and make a new patchset
21:02 bodepd_ I'm all about early validation
21:02 bodepd_ in terms of the logic, I'll trust you on what needs to be validated
21:03 bodepd_ I don't have time to dive into ml2
21:03 bodepd_ atm
21:03 EmilienM bodepd_: sure :)
21:03 bodepd_ but since it does not touch anything else (just new files)
21:03 bodepd_ as long as the code is sane, I think we can merge it
21:03 bodepd_ mgagne: may want to have a tougher review for taht
21:04 mgagne bodepd_: yes, I already suggested ways to improve his code. Don't know what has been done after that.
21:04 EmilienM bodepd_: don't merge it yet,  i want to add more tests
21:04 EmilienM mgagne: i've considered all your suggestions
21:04 EmilienM but i'll recheck
21:04 ezziepat joined #puppet-openstack
21:06 EmilienM bodepd_: the guys sitting next me at the office is one of the ml2 devs, so we can trust him :)
21:27 dmsimard mgagne: Welcome to Internap :p
21:27 mgagne haha
21:28 e1mer joined #puppet-openstack
21:33 openstackgerrit Andrew Woodward proposed a change to stackforge/puppet-cinder: [bp] Add support for Cinder multi-backend  https://review.openstack.org/54591
21:33 mgagne marun: ping
21:35 xarses mgagne: having some troubles with the rspec tests http://paste.openstack.org/show/50254/
21:39 mgagne xarses: checking
21:41 mgagne xarses: title is required to test defines
21:41 mgagne xarses: some tests are missing title
21:58 tnoor1 joined #puppet-openstack
21:59 mgagne joined #puppet-openstack
21:59 mgagne joined #puppet-openstack
21:59 mgagne xarses: got it?
21:59 tnoor2 joined #puppet-openstack
22:05 xarses mgagne: specifically working on getting cinder_backend_iscsi_spec to work
22:05 xarses gives error 1 still
22:05 larsks :wq
22:06 * xarses grins
22:06 larsks Well, that't wont work here...
22:06 mgagne xarses: should only be a matter of adding: let (:title) { 'yep-it-works' }
22:15 xarses mgagne: ok thanks, figured it out. the scoping of the let (:title) appears to be quite narrow
22:16 mgagne xarses: shouldn't be
22:16 xarses it appears to be required under each describe
22:16 mgagne xarses: title is required for the test to run
22:17 mgagne xarses: you can declare one in the top most context/describe
22:17 mgagne xarses: 1st level that is. if you only define it in the 2nd level in the 1st test, 2nd test won't inherit it as it will be in a different context
22:19 tnoor1 joined #puppet-openstack
22:22 xarses hmm that works now, i guess i was getting overly confused with the error messages
22:22 mgagne =)
22:25 tnoor1 joined #puppet-openstack
22:26 rongze joined #puppet-openstack
22:51 xarses ok, now i need to figure out how to get those keystone credentials correctly
22:57 openstackgerrit Andrew Woodward proposed a change to stackforge/puppet-cinder: [bp] Add support for Cinder multi-backend  https://review.openstack.org/54591
22:59 xarses mgagne: yay jenkins is more-or-less happy now =)
23:07 rmoe_ joined #puppet-openstack
23:26 rongze joined #puppet-openstack
23:30 rmoe joined #puppet-openstack

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