Integrate Review Board with other Review Tool
Hello team, Why we cannot integrate review Board with other review Tool. Is it possible to use plug in to integrate few futures from review Board. Is only repositories can supports review board or other tools can support it. -- 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 reviewboard+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Re: Issue Migrating Database from 1.5.5 to 2.0.5 (model updates fail)
Hey Christian,
One of the previous admins told me that we were using reviewboard pre 1.0 -
this may explain where the random sql came from.
Can I get a quote on how much a support contract would cost?
-Zach
On Sat, Apr 4, 2015 at 1:35 AM, Christian Hammond
wrote:
> Hey Zach,
>
> I basically have this fixed, and will be getting it through a review cycle
> and ready for a new django_evolution release, but I have some concerns
> about your database. It seems to be missing a couple major releases worth
> of evolutions.
>
> The SQL you found seems to match the missing evolution, so I'm guessing
> someone's been applying them by hand?
>
> It's absolutely crucial that this be fixed for an upgrade, or things can
> end up quickly in a bad state. Future releases will be increasingly harder
> to upgrade to.
>
> There's basically two options here:
>
> 1) You can try installing a brand new site, using a brand new database, in
> order to generate a correct set of evolution data. You'd then want to copy
> the contents of these two tables (the same ones you sent me) from the that
> new database to the one you want to upgrade, completely replacing the
> copies in that database.
>
> 2) We can do this for you and deal with any other issues that come up
> along the way, and fully repair your database. We'd have you up and running
> under 2.0 quick. However, we can only do this under a support contract.
>
> I'll let you know when we have the django_evolution release out. I can
> also send you an early build to test, if you let me know what version of
> Python you're using.
>
> Christian
>
>
> --
> Christian Hammond - christ...@beanbaginc.com
> Review Board - http://www.reviewboard.org
> Beanbag, Inc. - http://www.beanbaginc.com
>
> On Fri, Apr 3, 2015 at 8:47 PM, Christian Hammond <
> christ...@beanbaginc.com> wrote:
>
>> I believe I've isolated the problem to a bug in django-evolution. Haven't
>> figured out the exact cause just yet, but working on it!
>>
>> Christian
>>
>> --
>> Christian Hammond - christ...@beanbaginc.com
>> Review Board - http://www.reviewboard.org
>> Beanbag, Inc. - http://www.beanbaginc.com
>>
>> On Fri, Apr 3, 2015 at 7:35 PM, Christian Hammond <
>> christ...@beanbaginc.com> wrote:
>>
>>> Hi Zach,
>>>
>>> I'm sorry, I haven't had any time to make progress on this. Been busy
>>> with some of our premium support customers, student program, and a few
>>> other things. I have a couple hours I can spend right now, so I'll see if
>>> anything stands out.
>>>
>>> Where did those SQL statements come from?
>>>
>>> FWIW, Django Evolution has been around since the 1.0 alphas.
>>>
>>> Christian
>>>
>>> --
>>> Christian Hammond - christ...@beanbaginc.com
>>> Review Board - http://www.reviewboard.org
>>> Beanbag, Inc. - http://www.beanbaginc.com
>>>
>>> On Thu, Apr 2, 2015 at 8:13 AM, Zach Auclair wrote:
>>>
Hey Christian,
Have you been able to make any progress on debugging this? I just
wanted to point out that I believe before the current version that to
upgrade to the current version we were on long ago, our previous maintainer
may have run some migrations by hand (before this had django_evolutions?).
Some of the sql I found:
```
CREATE INDEX "django_session_expire_date" ON "django_session"
("expire_date");
ALTER TABLE "diffviewer_filediff" ALTER COLUMN "source_file" TYPE
varchar(1024) USING CAST("source_file" as varchar(1024));
ALTER TABLE "diffviewer_filediff" ALTER COLUMN "dest_file" TYPE
varchar(1024) USING CAST("dest_file" as varchar(1024));
ALTER TABLE "diffviewer_diffset" ADD COLUMN "basedir" varchar(256) ;
UPDATE "diffviewer_diffset" SET "basedir" = '' WHERE "basedir" IS NULL;
ALTER TABLE "diffviewer_diffset" ALTER COLUMN "basedir" SET NOT NULL;
ALTER TABLE "diffviewer_filediff" ADD COLUMN "status" varchar(1) ;
UPDATE "diffviewer_filediff" SET "status" = 'M' WHERE "status" IS NULL;
ALTER TABLE "diffviewer_filediff" ALTER COLUMN "status" SET NOT NULL;
ALTER TABLE "reviews_reviewrequest" ADD COLUMN "last_review_timestamp"
timestamp with time zone NULL ;
UPDATE reviews_reviewrequest
SET last_review_timestamp = (
SELECT reviews_review.timestamp
FROM reviews_review
WHERE reviews_review.review_request_id =
reviews_reviewrequest.id
AND reviews_review.public
ORDER BY reviews_review.timestamp DESC
LIMIT 1);
ALTER TABLE "reviews_reviewrequest" ADD COLUMN "shipit_count" integer ;
UPDATE "reviews_reviewrequest" SET "shipit_count" = 0 WHERE
"shipit_count" IS NULL;
UPDATE reviews_reviewrequest
SET shipit_count = (
SELECT COUNT(*)
FROM reviews_review
WHERE reviews_review.review_request_id =
Re: Integrate Review Board with other Review Tool
Review Board is a standalone code review tool. We do not support embedding or porting Review Board or any of its features into competing tools. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.com On Mon, Apr 6, 2015 at 3:12 AM, Ravi wrote: > Hello team, > > Why we cannot integrate review Board with other review Tool. Is it > possible to use plug in to integrate few futures from review Board. Is only > repositories can supports review board or other tools can support it. > > -- > 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 reviewboard+unsubscr...@googlegroups.com. > 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 reviewboard+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Re: P4Python build error
Hi Karti, We'll need to keep discussion on reviewboard@googlegroups.com. Thanks! You're on the right track with using P4PythonInstaller instead of compiling by hand. Are you behind a proxy server at your work? It sounds like it's unable to fetch files from their FTP server. I've just tested locally and I'm able to install it without any problems, so I believe this was either a temporary issue or a network configuration problem. Christian -- Christian Hammond - christ...@beanbaginc.com Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.com On Mon, Apr 6, 2015 at 3:06 AM, Karti wrote: > > hi please help me resolve this, i am not able to go further. > Tried using easy_install P4PythonInstaller but that too gives an error > like this: > > $easy_install P4PythonInstaller > Searching for P4PythonInstaller > Reading https://pypi.python.org/simple/P4PythonInstaller/ > Best match: P4PythonInstaller 0.3 > Downloading > http://github.com/beanbaginc/P4PythonInstaller/tarball/master#egg=P4PythonInstaller-0.3 > Processing master > Writing > /tmp/easy_install-2a6nCn/beanbaginc-P4PythonInstaller-9fa4fcf/setup.cfg > Running beanbaginc-P4PythonInstaller-9fa4fcf/setup.py -q bdist_egg > --dist-dir > /tmp/easy_install-2a6nCn/beanbaginc-P4PythonInstaller-9fa4fcf/egg-dist-tmp-BQebfJ > error: [Errno 104] Connection reset by peer > > > > On Monday, April 6, 2015 at 12:13:18 PM UTC+5:30, Karti wrote: >> >> Hi, >> >> I am trying to install P4Python on linux 3.16.0-30-generic >> #40~14.04.1-Ubuntu SMP x86_64 x86_64 x86_64 GNU/Linux machine. >> Python version 2.7.6. >> p4python-2014.2.962887 and p4api version p4api-2014.2.978861. >> >> The error i get is shown below: >> >> $python setup.py build --apidir /root/Downloads/p4api-2014.2.978861/ >> API Release 2014.2 >> running build >> running build_py >> creating build >> creating build/lib.linux-x86_64-2.7 >> copying P4.py -> build/lib.linux-x86_64-2.7 >> running build_ext >> building 'P4API' extension >> creating build/temp.linux-x86_64-2.7 >> x86_64-linux-gnu-gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv >> -O2 -Wall -Wstrict-prototypes -fPIC -DID_OS="LINUX31X86_64" >> -DID_REL="2014.2" -DID_PATCH="962887" -DID_API="2014.2/978861" >> -DID_Y="2014" -DID_M="11" -DID_D="17" -I/root/Downloads/p4api-2014.2.978861/ >> -I/root/Downloads/p4api-2014.2.978861/include/p4 >> -I/usr/include/python2.7 -c P4API.cpp -o build/temp.linux-x86_64-2.7/P4API.o >> -DOS_LINUX -DOS_LINUX31 -DOS_LINUXX86_64 -DOS_LINUX31X86_64 >> cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for >> C/ObjC but not for C++ [enabled by default] >> x86_64-linux-gnu-gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv >> -O2 -Wall -Wstrict-prototypes -fPIC -DID_OS="LINUX31X86_64" >> -DID_REL="2014.2" -DID_PATCH="962887" -DID_API="2014.2/978861" >> -DID_Y="2014" -DID_M="11" -DID_D="17" -I/root/Downloads/p4api-2014.2.978861/ >> -I/root/Downloads/p4api-2014.2.978861/include/p4 >> -I/usr/include/python2.7 -c PythonClientAPI.cpp -o >> build/temp.linux-x86_64-2.7/PythonClientAPI.o -DOS_LINUX -DOS_LINUX31 >> -DOS_LINUXX86_64 -DOS_LINUX31X86_64 >> cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for >> C/ObjC but not for C++ [enabled by default] >> x86_64-linux-gnu-gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv >> -O2 -Wall -Wstrict-prototypes -fPIC -DID_OS="LINUX31X86_64" >> -DID_REL="2014.2" -DID_PATCH="962887" -DID_API="2014.2/978861" >> -DID_Y="2014" -DID_M="11" -DID_D="17" -I/root/Downloads/p4api-2014.2.978861/ >> -I/root/Downloads/p4api-2014.2.978861/include/p4 >> -I/usr/include/python2.7 -c PythonClientUser.cpp -o >> build/temp.linux-x86_64-2.7/PythonClientUser.o -DOS_LINUX -DOS_LINUX31 >> -DOS_LINUXX86_64 -DOS_LINUX31X86_64 >> cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for >> C/ObjC but not for C++ [enabled by default] >> x86_64-linux-gnu-gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv >> -O2 -Wall -Wstrict-prototypes -fPIC -DID_OS="LINUX31X86_64" >> -DID_REL="2014.2" -DID_PATCH="962887" -DID_API="2014.2/978861" >> -DID_Y="2014" -DID_M="11" -DID_D="17" -I/root/Downloads/p4api-2014.2.978861/ >> -I/root/Downloads/p4api-2014.2.978861/include/p4 >> -I/usr/include/python2.7 -c SpecMgr.cpp -o >> build/temp.linux-x86_64-2.7/SpecMgr.o >> -DOS_LINUX -DOS_LINUX31 -DOS_LINUXX86_64 -DOS_LINUX31X86_64 >> cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for >> C/ObjC but not for C++ [enabled by default] >> x86_64-linux-gnu-gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv >> -O2 -Wall -Wstrict-prototypes -fPIC -DID_OS="LINUX31X86_64" >> -DID_REL="2014.2" -DID_PATCH="962887" -DID_API="2014.2/978861" >> -DID_Y="2014" -DID_M="11" -DID_D="17" -I/root/Downloads/p4api-2014.2.978861/ >> -I/root/Downloads/p4api-2014.2.978861/include/p4 >> -I/usr/include/python2.7 -c P4Result.cpp -o >> build/temp.linux-x86_64-2.7/P4Result.o >> -DOS_LINUX -DOS_LINUX31 -DOS_LINUXX86_64 -DOS_LINUX31X86_64 >> cc1plus:
Can I create a new user in reviewboard using the Python Web API?
I've been chasing my tail for a few hours on this and thought it was time
for some help. For testing, I want to spin up a new rb site, which I have a
script to do, and then create 2 new users for posting reviews. I can
access existing users this way:
client = RBClient("http://localhost";)
root = client.get_root()
user = root.get_user(username="sallan")
print user
But I can't figure out how to create a new user in a script. Can someone
point me in the right direction?
Thanks
---Steve
--
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 reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
Re: Can I create a new user in reviewboard using the Python Web API?
Hi Steve,
There's no support in the API for creating users. This must be done through
the web UI or through an authentication backend, like LDAP or Active
Directory.
Christian
--
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
Beanbag, Inc. - http://www.beanbaginc.com
On Mon, Apr 6, 2015 at 2:35 PM, Steve wrote:
> I've been chasing my tail for a few hours on this and thought it was time
> for some help. For testing, I want to spin up a new rb site, which I have
> a script to do, and then create 2 new users for posting reviews. I can
> access existing users this way:
>
> client = RBClient("http://localhost";)
> root = client.get_root()
> user = root.get_user(username="sallan")
> print user
>
> But I can't figure out how to create a new user in a script. Can someone
> point me in the right direction?
>
> Thanks
>
>
> ---Steve
>
> --
> 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 reviewboard+unsubscr...@googlegroups.com.
> 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 reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
