Thanks for the reply. The initial "panic" might be over as the users found 2 ways to work around it.
1. using "--diff-only" 2. using "-r <reviewid>" So I'll just get the next update naturally, no rush and thanks again. - John On Wed, Jan 18, 2017 at 12:04 AM, Christian Hammond < [email protected]> wrote: > 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 a topic in the > Google Groups "reviewboard" group. > To unsubscribe from this topic, visit https://groups.google.com/d/ > topic/reviewboard/tcHk6dxuWaY/unsubscribe. > To unsubscribe from this group and all its topics, 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.
