Revert "Have the ability for the PFQ to both rev Chrome and ..."

Revert "Have the ability for the PFQ to both rev Chrome and ..."

This reverts the commit a8ac0ec8c3fd481c48e3846a8dfcae8105bab14a.

TBR the x86 PFQ buildbot fails

BUG=chromium-os:11447
TEST=
TBR=sosa

Change-Id: I019de669db13b753365e1208acc9923c4bb5c993

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=3eb3543

[Vincent Palatin, 2011-01-28 20:36:02]

We might need to revert it , it seems to make the x86 PFQ fail

[kliegs, 2011-01-28 20:43:52]

Please add the Treecloser bug to the BUG= line

Otherwise LGTM

On Fri, Jan 28, 2011 at 3:36 PM, <vpalatin@chromium.org> wrote:

> Reviewers: sosa, kliegs, rharrison, dgarrett, kuan,
>
> Message:
> We might need to revert it , it seems to make the x86 PFQ fail
>
> Description:
> Revert "Have the ability for the PFQ to both rev Chrome and ..."
>
> This reverts the commit a8ac0ec8c3fd481c48e3846a8dfcae8105bab14a.
>
> TBR the x86 PFQ buildbot fails
>
> BUG=
> TEST=
> TBR=sosa
>
> Change-Id: I019de669db13b753365e1208acc9923c4bb5c993
>
> Please review this at http://codereview.chromium.org/6339018/
>
> SVN Base: ssh://git@gitrw.chromium.org:9222/crosutils.git@master
>
> Affected files:
>  M bin/cbuildbot.py
>  M bin/cbuildbot_config.py
>  M bin/cros_mark_chrome_as_stable.py
>  M cros_mark_as_stable.py
>  M cros_mark_as_stable_unittest.py
>
>
>

[sosa, 2011-01-28 21:05:25]

Yeah not convinced this did it but sure why not.  Seems like cleaning
up chromiumos-overlay fixed the issue.  I don't think this CL actually
mucked that repo.

On Fri, Jan 28, 2011 at 12:43 PM, Jonathan Kliegman <kliegs@chromium.org> wrote:
> Please add the Treecloser bug to the BUG= line
> Otherwise LGTM
>
> On Fri, Jan 28, 2011 at 3:36 PM, <vpalatin@chromium.org> wrote:
>>
>> Reviewers: sosa, kliegs, rharrison, dgarrett, kuan,
>>
>> Message:
>> We might need to revert it , it seems to make the x86 PFQ fail
>>
>> Description:
>> Revert "Have the ability for the PFQ to both rev Chrome and ..."
>>
>> This reverts the commit a8ac0ec8c3fd481c48e3846a8dfcae8105bab14a.
>>
>> TBR the x86 PFQ buildbot fails
>>
>> BUG=
>> TEST=
>> TBR=sosa
>>
>> Change-Id: I019de669db13b753365e1208acc9923c4bb5c993
>>
>> Please review this at http://codereview.chromium.org/6339018/
>>
>> SVN Base: ssh://git@gitrw.chromium.org:9222/crosutils.git@master
>>
>> Affected files:
>>  M bin/cbuildbot.py
>>  M bin/cbuildbot_config.py
>>  M bin/cros_mark_chrome_as_stable.py
>>  M cros_mark_as_stable.py
>>  M cros_mark_as_stable_unittest.py
>>
>>
>
>

[dgarrett, 2011-01-28 21:16:40]

I think that a test run of a change to cbuildbot on my workstation tried to
uprev the package versions. This confused and broken the build server. I was
running with --debug, which is supposed to stop the script from trying to
push the uprevs, but I'm thinking there is a bug with --debug. If you want,
give me the bug for the build break, and I'll use it to track work fixing
the --debug problem.

On Fri, Jan 28, 2011 at 1:05 PM, Chris Sosa <sosa@chromium.org> wrote:

> Yeah not convinced this did it but sure why not.  Seems like cleaning
> up chromiumos-overlay fixed the issue.  I don't think this CL actually
> mucked that repo.
>
> On Fri, Jan 28, 2011 at 12:43 PM, Jonathan Kliegman <kliegs@chromium.org>
> wrote:
> > Please add the Treecloser bug to the BUG= line
> > Otherwise LGTM
> >
> > On Fri, Jan 28, 2011 at 3:36 PM, <vpalatin@chromium.org> wrote:
> >>
> >> Reviewers: sosa, kliegs, rharrison, dgarrett, kuan,
> >>
> >> Message:
> >> We might need to revert it , it seems to make the x86 PFQ fail
> >>
> >> Description:
> >> Revert "Have the ability for the PFQ to both rev Chrome and ..."
> >>
> >> This reverts the commit a8ac0ec8c3fd481c48e3846a8dfcae8105bab14a.
> >>
> >> TBR the x86 PFQ buildbot fails
> >>
> >> BUG=
> >> TEST=
> >> TBR=sosa
> >>
> >> Change-Id: I019de669db13b753365e1208acc9923c4bb5c993
> >>
> >> Please review this at http://codereview.chromium.org/6339018/
> >>
> >> SVN Base: ssh://git@gitrw.chromium.org:9222/crosutils.git@master
> >>
> >> Affected files:
> >>  M bin/cbuildbot.py
> >>  M bin/cbuildbot_config.py
> >>  M bin/cros_mark_chrome_as_stable.py
> >>  M cros_mark_as_stable.py
> >>  M cros_mark_as_stable_unittest.py
> >>
> >>
> >
> >
>



-- 
Don