Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2012-11-27

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

All times shown according to UTC.

Time Nick Message
00:04 dachary joined #puppet-openstack
00:56 e1mer joined #puppet-openstack
00:56 e1mer joined #puppet-openstack
02:12 pdurbin left #puppet-openstack
02:39 ben__duyujie joined #puppet-openstack
02:44 e1mer joined #puppet-openstack
03:51 ben__duyujie_ joined #puppet-openstack
07:14 dachary joined #puppet-openstack
08:55 dachary joined #puppet-openstack
09:09 derekh joined #puppet-openstack
10:01 dachary joined #puppet-openstack
10:02 e1mer joined #puppet-openstack
10:02 e1mer joined #puppet-openstack
15:20 dachary joined #puppet-openstack
17:08 comptona joined #puppet-openstack
22:01 e1mer joined #puppet-openstack
22:01 e1mer joined #puppet-openstack
22:57 bodepd JoeJulian: to answer your question from Fri.
22:57 bodepd I did not have this use case in mind.
22:57 bodepd I know that some people have looked into how to do it in the past
22:58 bodepd aimon_: looking at that pull request
22:58 aimon_ thx boped :)
23:02 bodepd wow, that is fairly complicated
23:02 bodepd its too bad there are not seperate configs for each of things that can be enabled
23:02 aimon_ yeah would make things easier.. still you end up with conflicts on nova:api etc..
23:03 bodepd ah, b/c it is setting it as well?
23:03 aimon_ yeah nova-api is run on compute and controller.. so it is in common class
23:03 aimon_ same with nova-netowrk.. though the net creation part is only done on controller...
23:03 aimon_ so there were some conditions that needed to be sorted
23:04 bodepd can you just send a note to the list explaining it in detail?
23:04 aimon_ I know the pieces well enough though that it wasn't very hard
23:04 bodepd I dont quite have the bandwidth atm
23:04 aimon_ yes doing final testing.. then will doc it well
23:04 aimon_ np on that
23:04 bodepd I am rediculously behind on pull requests.
23:04 aimon_ there is a small PR in there also
23:04 bodepd I'm trying to write some test automation tools to ease the process
23:04 aimon_ option passtrhough
23:05 bodepd that has been one of my focuses atm
23:05 aimon_ this change needs a few days of vetting anyways
23:05 bodepd cool.
23:05 bodepd I have a whole ton of low hanging fruit that I want to get merged.
23:05 aimon_ I won't submit PR on it until its really solid, well docd and rspec'd
23:05 bodepd there seem to be tons of cases:
23:05 bodepd is it multi_host
23:06 bodepd is cinder vs. nova-volume running
23:06 aimon_ eactly :)
23:06 bodepd the best we may be able to do is validation
23:06 bodepd and fail with reasonable messages.
23:06 aimon_ yep
23:06 aimon_ there is some of that in there already.. but need to add more
23:06 bodepd I dont remember if the native type for nova_config handles arrays
23:07 aimon_ if a is set then require that b and c are set.. etc
23:07 bodepd but trying to append on elements would be tricky...
23:07 bodepd b/c the user could be trying to set them
23:07 aimon_ in those cases you let user override
23:07 aimon_ or use reasonable defaults
23:07 aimon_ but don't try to merge defaults and user changes
23:07 bodepd yeah, shit.
23:08 aimon_ anyways.. seems its working well at this point.. just needs vetting etc
23:09 aimon_ I have tested the various cases .. needs testing for quantum and Cinder though.. Joejulian will be testing on puppet 3.. not sure if he is enabling cinder and/or quantum
23:09 bodepd I verified that things seem to work on puppet 3
23:10 aimon_ I have a few hardware configs I will test on.. will set them up on different configs to validate possible combinations
23:10 aimon_ cool :)
23:10 bodepd I still have a whole ton of pull requests to merge
23:10 bodepd it's unfortunately going to take some time
23:10 bodepd how automated is your CI?
23:11 bodepd can you push a button and run all of the tests?
23:11 bodepd against any version of these things?
23:11 aimon_ almost.. bit of hand editing build script in jenkins UI
23:12 aimon_ but other then that.. end to end.. git tp deb
23:12 aimon_ s/tp/to/
23:12 bodepd as a warning, I want to start rejecting anything without unit tests
23:12 aimon_ as soon as jenkins sees change it runs full st and packages it for us
23:12 aimon_ +1 to that
23:12 bodepd I just don't have hte bandwidth to accept patches without tests.
23:13 bodepd maybe I will go through and start mentioning that.
23:13 aimon_ yeah we have to let CI do the work :)
23:13 bodepd I merged some stuff without tests b/c the cases were handled by existing tests
23:13 aimon_ or we'll have to nick name you "The Butler"
23:13 bodepd but its such a pain to even have to go through and see if PRs may be tested.
23:14 aimon_ ever thought about eliciting community help on managing the code base?
23:14 bodepd I need help badly :)
23:14 aimon_ .. I know it would require an infrastructure of communication that may not exist yet
23:14 bodepd I would be happy to give people merge rights.
23:14 bodepd maybe I should draft a contributing guide
23:15 aimon_ I think a guiding book or rulebook needs to be written first
23:15 bodepd but yes. I would like to get people other than myself merging code.
23:15 aimon_ yep ;)
23:15 bodepd but, I am a little afraid that the testing infra and tools dont exist yet for this
23:15 aimon_ bit of work up front.. pay off in backend
23:15 bodepd people are doing a great job of communicating.
23:16 bodepd I've almost got a script that goes through and checks out branches that I mark with
23:16 bodepd test_it
23:17 bodepd s/branches/pull requests/
23:17 bodepd it then runs my basic tests script for redhat and ubuntu
23:17 bodepd I hope to get this running through jenkins in a couple of weeks
23:17 bodepd and then linking results back to github
23:17 bodepd (kind of like travisci)
23:17 aimon_ nice
23:18 bodepd puppetlabs-openstack_dev_env is about to start changing a lot
23:47 aimon_ hoped I have some code that auto calculates num_networks and network_size for nova::network based on fixed_range .. this eliminates need to specify those two args...
23:47 aimon_ *boped
23:47 aimon_ I'll submit.. it will auto calc if those two args are not set
23:48 aimon_ this way user can always override

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