Sledgehammer logging changes have been reverted for now.
On Thu, May 30, 2013 at 7:03 AM, Victor Lowther <victor.lowt...@gmail.com>wrote: > On Thu, May 30, 2013 at 3:28 AM, Vincent Untz <vu...@suse.com> wrote: > >> Hi, >> >> I just realized that one of my commits that got merged will impact >> non-Pebbles branches, and might be an issue requiring changes in these >> branches too. This is the following commit: >> >> https://github.com/crowbar/crowbar/commit/29b97580c42bdfe1e0b548cff773811eaeea16d9 >> >> In short, this is changing /install-logs to >> /var/log/crowbar/sledgehammer. >> > > That is going to break rather a lot of stuff. > > >> Other branches that are using the sledgehammer-common/start-up.sh file >> from crowbar master should probably be updated with commits like these >> ones: >> >> >> https://github.com/crowbar/barclamp-provisioner/commit/0290fc89b4ca8d446eac9c3140247f18166fd6f1 >> >> https://github.com/crowbar/barclamp-logging/commit/9410dd876fc7603e0ed9f34cefd78d7a4dcd4633 > > > No -- that is all the builds in all the releases, as Sledgehammer is > globally applicable to them all. Your changes modifying these paths will > have to be reverted and applied in a way that is Pebbles specific, > preferably by modifying control.sh in the provisioner. > > >> >> FWIW, I find it a bit confusing that sledgehammer-common/start-up.sh >> isn't tracked in a Pebbles branch... >> >> Cheers, >> >> Vincent >> >> -- >> Les gens heureux ne sont pas pressés. >> >> _______________________________________________ >> Crowbar mailing list >> Crowbar@dell.com >> https://lists.us.dell.com/mailman/listinfo/crowbar >> For more information: http://crowbar.github.com/ >> > >
_______________________________________________ Crowbar mailing list Crowbar@dell.com https://lists.us.dell.com/mailman/listinfo/crowbar For more information: http://crowbar.github.com/
