Cobbler drops errors on the floor during a replicate about cobbler HOT 7 OPEN

"add" in cobbler is used in two places, one in creating in-memory objects (save=False), another in saving them to disk. I don't think we want to log in this particular place because this would produce logging on invalid command line executions. Rather, I think it's more likely that when running replicate (did you mean replicate? It seems to make more sense here) we'd want to ensure it reported on the things it could not transfer.

from cobbler.

mpdehaan - sorry for the delay, yes I did mean during replicate

Yes, reporting is some way that things didn't transfer would be good.

from cobbler.

jimi, while that would improve matters, the one issue is that because the exception was dropped on the floor in add you don't know why it was dropped on the floor. Allowing the exception to fall back up to the caller would probably be the cleaner option, with the command line executions then being responsible for catching what is ultimately a mistake in the command line options... That's MHO anyway...

from cobbler.

True, but at least knowing that it failed is important, and lets you start debugging. I plan on this being more of a stop-gap anyway, as replicate needs some work. For example, doing a full data sync does not respect distro filters right now, so there's no way to do a selective replicate. That means if you do replicate systems or profiles, you HAVE to do a full data sync or the distros won't be brought over due to the fact that the initrd/kernel pairs are not available on the replicated system. That is probably the source of the original failures, to be honest.

from cobbler.

jimi, What was actually catching us out was the fact that the systems pointed to profiles/distros that just didn't exist any more, and somewhere in the past the logic hadn't caught this.

from cobbler.

but I'm fine with your point about it being a stop gap, if more work is planned.

from cobbler.

I'm wondering if I've come across this bug also. I had an issue where cobblerd wouldn't restart on a slave. I tracked it down to an extra '}' /var/lib/cobbler/config/repos.d/rpmforge-el6.json at the end of the file. During the replication process /var/www/cobbler/repo_mirror filled up also though I don't see how this could have caused my issue.

from cobbler.