Hey John, We did. We have a fix that just finished going through review and landed in our release-2.5.x branch last night. You can pull that out and apply it locally if you want. We hope to release later this week or early next (we're going to be quite swamped with something Thursday through Monday, so we might release next week).
Christian On Mon, Jan 16, 2017 at 07:28 John McGowan <[email protected]> wrote: > Hi Christian, did you or the guys spot anything in relation to this? > > Cheers, > > John > > On Tuesday, 10 January 2017 12:13:41 UTC, John McGowan wrote: > > Hi Christian, > > Have an issue which posting a review and attempting to repost to it again. > I recently upgraded our RB server to the latest 2.5.7 and rbtools on that > server. Note the postreview.py we use is probably quite old (0.4.1) as its > been modified a bit to suit our environment but hasn't been touched in > ages. This exists on NFS for all to use the same version. > > Seems the repost to an existing review is failing as the changenum is None > as opposed to an Integer. I'm not sure how the server version is affecting > this but its the only thing thats changed recently: > > >>> Attempting to create review request on bfs-p4.bfs.openwave.com:1667 > for 1176766 > >>> HTTP POSTing to http://review.bfs.openwave.com/api/review-requests/: > {'changenum': '1176766', 'repository': 'bfs-p4.bfs.openwave.com:1667'} > >>> Got API Error 204 (HTTP code 409): The commit ID specified has already > been used. > >>> Error data: {u'stat': u'fail', u'review_request': {u'status': > u'pending', u'last_updated': u'2017-01-10T11:45:12Z', u'links': {u'diffs': > {u'href': u' > http://review.bfs.openwave.com/api/review-requests/33514/diffs/', > u'method': u'GET'}, u'repository': {u'href': u' > http://review.bfs.openwave.com/api/repositories/3/', u'method': u'GET', > u'title': u'bfs-p4:1667'}, u'changes': {u'href': u' > http://review.bfs.openwave.com/api/review-requests/33514/changes/', > u'method': u'GET'}, u'self': {u'href': u' > http://review.bfs.openwave.com/api/review-requests/33514/', u'method': > u'GET'}, u'update': {u'href': u' > http://review.bfs.openwave.com/api/review-requests/33514/', u'method': > u'PUT'}, u'last_update': {u'href': u' > http://review.bfs.openwave.com/api/review-requests/33514/last-update/', > u'method': u'GET'}, u'reviews': {u'href': u' > http://review.bfs.openwave.com/api/review-requests/33514/reviews/', > u'method': u'GET'}, u'draft': {u'href': u' > http://review.bfs.openwave.com/api/review-requests/33514/draft/', > u'method': u'GET'}, u'file_attachments': {u'href': u' > http://review.bfs.openwave.com/api/review-requests/33514/file-attachments/', > u'method': u'GET'}, u'submitter': {u'href': u' > http://review.bfs.openwave.com/api/users/earunach/', u'method': u'GET', > u'title': u'earunach'}, u'delete': {u'href': u' > http://review.bfs.openwave.com/api/review-requests/33514/', u'method': > u'DELETE'}, u'screenshots': {u'href': u' > http://review.bfs.openwave.com/api/review-requests/33514/screenshots/', > u'method': u'GET'}, u'diff_context': {u'href': u' > http://review.bfs.openwave.com/api/review-requests/33514/diff-context/', > u'method': u'GET'}}, u'depends_on': [], u'description_text_type': u'plain', > u'issue_resolved_count': 0, u'ship_it_count': 0, > u'close_description_text_type': u'plain', u'id': 33514, u'description': > u'', u'target_people': [], u'changenum': None, u'bugs_closed': [], > u'testing_done_text_type': u'plain', u'testing_done': u'', > u'close_description': None, u'time_added': u'2017-01-10T11:45:12Z', > u'extra_data': {u'calculated_trophies': True}, u'public': False, > u'commit_id': None, u'blocks': [], u'branch': u'', u'text_type': None, > u'issue_open_count': 0, u'approved': False, u'url': u'/r/33514/', > u'absolute_url': u'http://review.bfs.openwave.com/r/33514/', > u'target_groups': [], u'summary': u'', u'issue_dropped_count': 0, > u'approval_failure': u'The review request has not been marked "Ship It!"'}, > u'err': {u'msg': u'The commit ID specified has already been used.', > u'code': 204}} > >>> Review request already exists. Updating it... > >>> HTTP PUTting to > http://review.bfs.openwave.com/api/review-requests/33514/: {'changenum': > None} > >>> Got API Error 105 (HTTP code 400): One or more fields had errors > >>> Error data: {u'fields': {u'changenum': [u'"None" is not an integer']}, > u'stat': u'fail', u'err': {u'msg': u'One or more fields had errors', > u'code': 105}} > Error creating review request: One or more fields had errors (HTTP 400, > API Error 105) > > Any ideas, I was going to update our postreview file with the latest and > migrate our changes into it but I don't see it in the latest 0.7.7 RBTools > release in case it was a mismatch too far now. > > Thanks in advance, > John > > > > > > > > > > -- > > > Supercharge your Review Board with Power Pack: > https://www.reviewboard.org/powerpack/ > > > Want us to host Review Board for you? Check out RBCommons: > https://rbcommons.com/ > > > Happy user? Let us know! https://www.reviewboard.org/users/ > > > --- > > > You received this message because you are subscribed to the Google Groups > "reviewboard" group. > > > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected]. > > > For more options, visit https://groups.google.com/d/optout. > > > -- Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/ Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/ Happy user? Let us know! https://www.reviewboard.org/users/ --- You received this message because you are subscribed to the Google Groups "reviewboard" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
