> On Jan. 9, 2015, 11:07 a.m., Manikumar Reddy O wrote: > > kafka-patch-review.py, line 96 > > <https://reviews.apache.org/r/29756/diff/3/?file=814216#file814216line96> > > > > Did you tested the failure senario? I did not get error message. I > > think we will get exception only after invoking jira.issue().
I did test a failure case, yes. Gave an incorrect username and password and that failed during login and did not proceed further. Here's the relevant output (notice that I intentionally use JIRA user name "foo"): ` Configuring reviewboard url to https://reviews.apache.org Updating your remote branches to pull the latest changes Verifying JIRA connection configurations JIRA user :foo JIRA password : Failed to login to the JIRA instance <class 'jira.exceptions.JIRAError'> HTTP 401: " <html> <head> <title>Unauthorized (401)</title> .... ` - Jaikiran ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29756/#review67414 ----------------------------------------------------------- On Jan. 9, 2015, 10:13 a.m., Jaikiran Pai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29756/ > ----------------------------------------------------------- > > (Updated Jan. 9, 2015, 10:13 a.m.) > > > Review request for kafka. > > > Bugs: KAFKA-1854 > https://issues.apache.org/jira/browse/KAFKA-1854 > > > Repository: kafka > > > Description > ------- > > KAFKA-1854 Allow JIRA username and password to be prompted in the absence of > a jira.ini file, during patch submission > > > Diffs > ----- > > kafka-patch-review.py b7f132f9d210b8648859ab8f9c89f30ec128ab38 > > Diff: https://reviews.apache.org/r/29756/diff/ > > > Testing > ------- > > > Thanks, > > Jaikiran Pai > >
