Comment 9 for bug 1678486

Revision history for this message
Colin Watson (cjwatson) wrote :

This looks fine as far as it goes, but further testing reveals a couple of other things I'd like to fix before enabling this again on production:

 * There are some comments whose author is just a string of all-caps initials. Trying to import these OOPSes because sanitize_name results in the empty string, we then append "-redhat-bugs", and the result of that isn't a valid LP username.
 * There's a lot of noise because we can't push comments to bugzilla.redhat.com; this is because we don't have credentials set up for doing so. IMO if we don't have credentials configured for a Bugzilla instance then we shouldn't consider it eligible for comment pushing (or back-linking).