christopher_dearb...@dell.com (christopher_dearb...@dell.com) wrote: > I submitted a pull request that should resolve most of these issues > (https://github.com/crowbar/barclamp-network/pull/158). That pull was merged > on Friday, but does not appear to be in this build. > > Could whoever is managing the Travis builds take a look?
I realise that this is probably going to make me sound very lazy / selfish / difficult to work with etc. ;-) So apologies in advance, but ... I'm really keen that I (or James, or anyone else) don't become the bottleneck on Travis builds. I really think it needs to be a team-wide responsibility, e.g. if anyone sees test failures and thinks they may have caused them, that they take the lead in resolving them. That's especially true right now based on the other mail I just sent to the list, since we're in a situation where pull requests are not automatically tested prior to merge, and consequently trunk is at much higher risk than it should be. To this end, I have already spent quite a bit of effort trying to ensure that the whole testing setup is properly documented in a way that everyone can understand, e.g. https://github.com/crowbar/crowbar/blob/master/doc/devguide/testing/travis.md https://github.com/crowbar/crowbar/blob/master/doc/devguide/testing.md That said, I do admit that it is currently missing a "how to troubleshoot Travis test failures" section, and that is a sore omission - thanks for helping me realise that! However I have no time to write this since I have to start preparing the rspec talk for tomorrow, and anyway, I think I've done far more than my fair share of documentation work recently ;-) So can I suggest that someone else take ownership of writing this, and use it as a good exercise in learning about the Travis builds in the process? Shouldn't be a big job, and of course I'm happy to answer any questions which may arise. Thanks! _______________________________________________ Crowbar mailing list Crowbar@dell.com https://lists.us.dell.com/mailman/listinfo/crowbar For more information: http://crowbar.github.com/
