OK - points taken.
So taking your ideas and extending them a little, I'm thinking:
- retry upgrade steps on failure (with inter-attempt delay)
- indicate when there's upgrade problems by setting the machine agent
status
- if despite the retries the upgrade won't complete, report this
On Tue, 15 Jul 2014, Aaron Bentley wrote:
> I am surprised that juju-reports was not considered a known client. I
> certainly made many comments on the first draft of the original proposal.
It is listed under known clients in the spec, and we mentioned your request
down below. What we lack is y
On 16/07/14 01:57, William Reade wrote:
> On Tue, Jul 15, 2014 at 2:51 PM, Wayne Witzel
> mailto:wayne.wit...@canonical.com>> wrote:
>
> If we aren't stopping the line when our CI is in the red, then what
> is the point of even having CI at all? If we are not prepared to
> adjust the c
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 14-07-15 11:43 AM, Richard Harding wrote:
> On Tue, 15 Jul 2014, Aaron Bentley wrote:
>
>> On 14-07-15 10:17 AM, roger peppe wrote:
>>> The most significant change is that all endpoints refer just
>>> to a single charm or bundle, rather than being
On 14/07/14 09:43, Ian Booth wrote:
Hi all
So last week we had a Juju induction sprint for Tanzanite and Moonstone teams to
welcome Eric and Katherine to the Juju fold. Following is a summary of some key
outcomes from the sprint that are relevant to others working on Juju (we also
did other stu
On Tue, 15 Jul 2014, Aaron Bentley wrote:
> On 14-07-15 10:17 AM, roger peppe wrote:
> > The most significant change is that all endpoints refer just to a
> > single charm or bundle, rather than being "bulk" calls as they were
> > before.
>
> That sounds like the opposite of what juju-reports want
My concern was with the speed of response. I'm happy to have a QA Team
switch that must be fixed (with an associated email to juju - dev so
everyone knows why their patch won't land). I *would* like us to be
tracking stuff like how long we go into regression mode, etc.
I think ideally the process
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 14-07-15 10:17 AM, roger peppe wrote:
> The most significant change is that all endpoints refer just to a
> single charm or bundle, rather than being "bulk" calls as they were
> before.
That sounds like the opposite of what juju-reports wants. Doe
On Tue, Jul 15, 2014 at 1:29 AM, John Meinel wrote:
> It seems worthy to just run "go test github.com/juju/..." as our CI testing,
> isn't it? (i.e., run all unit tests across all packages that we write on all
> platforms), rather than *just* github.com/juju/juju.
Ah!. That looks easy. We could a
I think that's a fair assessment.
Perhaps the easiest fix is to install a switch QA could throw to change the
required merge message to something like !!ThisFixesCI!!
On Tue, Jul 15, 2014 at 9:57 AM, William Reade
wrote:
> On Tue, Jul 15, 2014 at 2:51 PM, Wayne Witzel
> wrote:
>
>> If we aren
Based on feedback from Rick Harding, amongst others,
we have made some changes to the proposed charm
store API. The new document is here:
https://docs.google.com/a/canonical.com/document/d/1TgRA7jW_mmXoKH3JiwBbtPvQu7WiM6XMrz1wSrhTMXw
The most significant change is that all endpoints
refer just to
On Tue, Jul 15, 2014 at 2:51 PM, Wayne Witzel
wrote:
> If we aren't stopping the line when our CI is in the red, then what is the
> point of even having CI at all? If we are not prepared to adjust the
> culture of our development. To truly halt forward progress in favor of
> chasing down regressi
If we aren't stopping the line when our CI is in the red, then what is the
point of even having CI at all? If we are not prepared to adjust the
culture of our development. To truly halt forward progress in favor of
chasing down regressions then I struggle to find the benefit that CI and
testing is
I don't think we need to stop the world to get these things fixed. It is
the responsibility of the team leads to make sure someone's actively
working on fixes for regressions. If they're not getting fixed, it's our
fault. We should have one of the team leads pick up the regression and
assign som
Yeah, I use it along side of sortimports. Looking at the source for
goimports, I think it would be a clean entry point to add sorting to it. I
imagine a pull request of that nature would be well received by the project.
On Tue, Jul 15, 2014 at 9:19 AM, roger peppe
wrote:
> Although goimports is
Although goimports is a great tool (I use it all the time) it doesn't implement
the same import grouping rules that we use, or change any existing groups.
I'm tempted to submit a change to it that does
that, but I don't know how well it would be received.
On 15 July 2014 13:20, Wayne Witzel wrot
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 14-07-14 11:48 PM, Ian Booth wrote:
>>
>> * FAIL: managedstorage_test trusty ppc64 from 2014-06-30 which
>> had a secondary bug that broke compilation.
>> https://bugs.launchpad.net/juju-core/+bug/1336089
>>
>
> This bug brings up another issue.
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 14-07-14 11:42 PM, David Cheney wrote:
> Who is 'we' ?
>
> Is some automated process going to manage this moratorium ? Do we
> switch off the bot, or perturb it's access ?
No, we don't want to switch off the bot, we want to limit the kinds of
bra
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 14-07-14 11:38 PM, Ian Booth wrote:
>> I propose a change in policy. When a there is a regression in CI,
>> no new branches can be merged except those that link to the
>> blocking bug. This will encourage engineers to fix the
>> regression. One way
Along the same lines, I've been using goimports (
https://github.com/bradfitz/goimports).
Micheal pointed to it. It wraps go fmt and supports the same flags, but
also adds a layer that removes and adds imports that are unused or needed
respectively. I've had good luck with it running on save with
FWIW, we could set some error status on the affected agent (so users can
see there's a problem) and make it return 0 (so that upstart doesn't keep
hammering it); but as jam points out that's not helpful when it's a
transient error. I'd suggest retrying a few times, with some delay between
attempts,
21 matches
Mail list logo
