Re: Elasticsearch Charm Promulgation

2017-08-31 Thread Tim Van Steenburgh
It's been promulgated, thanks!

On Wed, Aug 30, 2017 at 7:08 PM, Haw Loeung 
wrote:

> On Wed, Aug 30, 2017 at 02:38:42PM -0400, Tim Van Steenburgh wrote:
> > I checked with James Beedy and he's +1 on promulgating this one too.
> >
> > Looks like there's no bugs-url or homepage set on the charm though.
> Please
> > add those (see `charm set -h`) and then I'll be happy to promulgate it.
>
> I've updated the two:
>
> | $ charm show cs:~elasticsearch-charmers/elasticsearch bugs-url homepage
> | bugs-url: https://bugs.launchpad.net/elasticsearch-charm
> | homepage: https://launchpad.net/elasticsearch-charm
>
>
> Regards,
>
> Haw
>
> --
> Juju mailing list
> Juju@lists.ubuntu.com
> Modify settings or unsubscribe at: https://lists.ubuntu.com/
> mailman/listinfo/juju
>
>
-- 
Juju mailing list
Juju@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju


Re: Elasticsearch Charm Promulgation

2017-08-30 Thread Haw Loeung
On Wed, Aug 30, 2017 at 02:38:42PM -0400, Tim Van Steenburgh wrote:
> I checked with James Beedy and he's +1 on promulgating this one too.
>
> Looks like there's no bugs-url or homepage set on the charm though. Please
> add those (see `charm set -h`) and then I'll be happy to promulgate it.

I've updated the two:

| $ charm show cs:~elasticsearch-charmers/elasticsearch bugs-url homepage
| bugs-url: https://bugs.launchpad.net/elasticsearch-charm
| homepage: https://launchpad.net/elasticsearch-charm


Regards,

Haw


signature.asc
Description: PGP signature
-- 
Juju mailing list
Juju@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju


Re: Elasticsearch Charm Promulgation

2017-08-30 Thread Tim Van Steenburgh
On Wed, Aug 30, 2017 at 6:27 AM, Haw Loeung 
wrote:

> On Tue, Aug 22, 2017 at 08:54:14AM +0100, Tom Haddon wrote:
> > On Mon, Aug 21, 2017 at 02:53:10PM +0100, Tom Haddon wrote:
> > [...]
> > > I checked with some of the ~onlineservices-charmers team and they were
> happy
> > > for https://jujucharms.com/u/elasticsearch-charmers/elasticsearch/ to
> replace
> > > the current promulgated charm (which only supports precise and
> trusty). We
> > > should test that you can successfully charm upgrade from theirs to
> this charm
> > > (on trusty) - I'm happy to take that on, and will update here once
> we've done
> > > so.
> >
> > I've tested an upgrade on trusty from the existing upstream elasticsearch
> > charm to cs:~elasticsearch-charmers/trusty/elasticsearch and it seems
> to work
> > okay:
> >
> > https://pastebin.ubuntu.com/25368096/
> >
>
> I've confirmed that the existing charm also supports ElasticSearch 5.x
> (thanks to all the hard work from Ante).
>
> Any chance we could get cs:~elasticsearch-charmers/elasticsearch
> promulgated to cs:elasticsearch now?
>

I checked with James Beedy and he's +1 on promulgating this one too.

Looks like there's no bugs-url or homepage set on the charm though. Please
add those (see `charm set -h`) and then I'll be happy to promulgate it.


>
>
> Thanks,
>
> Haw
>
> --
> Juju mailing list
> Juju@lists.ubuntu.com
> Modify settings or unsubscribe at: https://lists.ubuntu.com/
> mailman/listinfo/juju
>
>
-- 
Juju mailing list
Juju@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju


Re: Elasticsearch Charm Promulgation

2017-08-30 Thread Haw Loeung
On Tue, Aug 22, 2017 at 08:54:14AM +0100, Tom Haddon wrote:
> On Mon, Aug 21, 2017 at 02:53:10PM +0100, Tom Haddon wrote:
> [...]
> > I checked with some of the ~onlineservices-charmers team and they were happy
> > for https://jujucharms.com/u/elasticsearch-charmers/elasticsearch/ to 
> > replace
> > the current promulgated charm (which only supports precise and trusty). We
> > should test that you can successfully charm upgrade from theirs to this 
> > charm
> > (on trusty) - I'm happy to take that on, and will update here once we've 
> > done
> > so.
>
> I've tested an upgrade on trusty from the existing upstream elasticsearch
> charm to cs:~elasticsearch-charmers/trusty/elasticsearch and it seems to work
> okay:
>
> https://pastebin.ubuntu.com/25368096/
>

I've confirmed that the existing charm also supports ElasticSearch 5.x
(thanks to all the hard work from Ante).

Any chance we could get cs:~elasticsearch-charmers/elasticsearch
promulgated to cs:elasticsearch now?


Thanks,

Haw


signature.asc
Description: PGP signature
-- 
Juju mailing list
Juju@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju


Re: Elasticsearch Charm Promulgation

2017-08-22 Thread Tom Haddon
On Mon, Aug 21, 2017 at 02:53:10PM +0100, Tom Haddon wrote:
[...]
> I checked with some of the ~onlineservices-charmers team and they were happy
> for https://jujucharms.com/u/elasticsearch-charmers/elasticsearch/ to replace
> the current promulgated charm (which only supports precise and trusty). We
> should test that you can successfully charm upgrade from theirs to this charm
> (on trusty) - I'm happy to take that on, and will update here once we've done
> so.

I've tested an upgrade on trusty from the existing upstream elasticsearch
charm to cs:~elasticsearch-charmers/trusty/elasticsearch and it seems to work
okay:

https://pastebin.ubuntu.com/25368096/

Thanks, Tom

-- 
Juju mailing list
Juju@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju


Re: Elasticsearch Charm Promulgation

2017-08-21 Thread Tom Haddon
On Fri, Aug 18, 2017 at 01:35:11PM -0500, Kevin Monroe wrote:
> Hey James!
> 
> First up, thanks so much for bringing the ES charm up to v5.x!  Can you
> tell me if the source for cs:~jamesbeedy/elasticsearch-7 is the same as
> what we've got here:
> 
> https://github.com/charms/charm-elasticsearch
> 
> Or is that source waiting on this PR to bring it in line with your es-7
> charm:
> 
> https://github.com/charms/charm-elasticsearch/pull/2
> 
> The PR LGTM, so I'll merge it on your ack.  Let me know if there are other
> outstanding bits that need to go into this repo.  Once that's done, I
> propose we push this to the ~elastic-ops namespace of which you're a member:
> 
> https://launchpad.net/~elastic-ops
> 
> And then promulgate the ~elastic-ops charm rev.  As Rick and others have
> mentioned, teams help distribute ongoing maintenance to people that have a
> vested interest in certain charms.

We've been working off https://code.launchpad.net/elasticsearch-charm which
we've pushed to the charmstore here
https://jujucharms.com/u/elasticsearch-charmers/elasticsearch/

This seems to be based off a similar version to the one James has on github,
and the ~elastic-ops team you mention above is already a member of the
elasticsearch-charm project.

We're using this charm on xenial successfully, so maybe it'd be worth trying
to work together to get
https://jujucharms.com/u/elasticsearch-charmers/elasticsearch/ as the
promulgated upstream charm.

> @Simon / Bret:  you are on the ~onlineservices-charmers team that owns the
> current promulgated ES charm.  Would you be willing to relinquish
> maintainership to ~elastic-ops (and would you like to be members of that
> team)?

I checked with some of the ~onlineservices-charmers team and they were happy
for https://jujucharms.com/u/elasticsearch-charmers/elasticsearch/ to replace
the current promulgated charm (which only supports precise and trusty). We
should test that you can successfully charm upgrade from theirs to this charm
(on trusty) - I'm happy to take that on, and will update here once we've done
so.

> Thanks!
> -Kevin Monroe
> 
> On Mon, Aug 14, 2017 at 12:12 PM, Rick Harding 
> wrote:
> 
> > I'd also encourage you to test an upgrade path from the current charm to
> > yours to help comfort any current users of the charm that if there's a
> > change it'll go smoothly for users.
> >
> > On Mon, Aug 14, 2017 at 1:00 PM Tim Van Steenburgh <
> > tim.van.steenbu...@canonical.com> wrote:
> >
> >> Hey James, my point was that if you want to replace the existing
> >> top-level ES charm with your own, we would need agreement from the
> >> maintainers of the existing charm.
> >>
> >> On Mon, Aug 14, 2017 at 12:45 PM, James Beedy 
> >> wrote:
> >>
> >>> @tim yeah ... possibly I'm not looking for promulgation then, and just
> >>> need to push it and grant everyone because its already promulgated?
> >>>
> >>> On Mon, Aug 14, 2017 at 9:29 AM, Tim Van Steenburgh <
> >>> tim.van.steenbu...@canonical.com> wrote:
> >>>
>  There's already a top-level elasticsearch charm.
>  https://jujucharms.com/elasticsearch/
> 
>  On Mon, Aug 14, 2017 at 12:13 PM, James Beedy 
>  wrote:
> 
> > Request for promulgation of Elasticsearch. The Elasticsearch charm can
> > be found at cs:~jamesbeedy/elasticsearch-7.
> >
> > Thanks,
> >
> > James
> >
> > --
> > Juju mailing list
> > Juju@lists.ubuntu.com
> > Modify settings or unsubscribe at: https://lists.ubuntu.com/
> > mailman/listinfo/juju
> >
> >
> 
> >>>
> >> --
> >> Juju mailing list
> >> Juju@lists.ubuntu.com
> >> Modify settings or unsubscribe at: https://lists.ubuntu.com/
> >> mailman/listinfo/juju
> >>
> >
> > --
> > Juju mailing list
> > Juju@lists.ubuntu.com
> > Modify settings or unsubscribe at: https://lists.ubuntu.com/
> > mailman/listinfo/juju
> >
> >

> -- 
> Juju mailing list
> Juju@lists.ubuntu.com
> Modify settings or unsubscribe at: 
> https://lists.ubuntu.com/mailman/listinfo/juju


-- 
Juju mailing list
Juju@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju


Re: Elasticsearch Charm Promulgation

2017-08-18 Thread James Beedy
Kevin,

"Or is that source waiting on this PR to bring it in line with your es-7
charm"
- This is the case

If we merge the xenial support PR, it will be even with the
cs:~jamesbeedy/elasticsearch-7.

~James

On Fri, Aug 18, 2017 at 11:35 AM, Kevin Monroe 
wrote:

> Hey James!
>
> First up, thanks so much for bringing the ES charm up to v5.x!  Can you
> tell me if the source for cs:~jamesbeedy/elasticsearch-7 is the same as
> what we've got here:
>
> https://github.com/charms/charm-elasticsearch
>
> Or is that source waiting on this PR to bring it in line with your es-7
> charm:
>
> https://github.com/charms/charm-elasticsearch/pull/2
>
> The PR LGTM, so I'll merge it on your ack.  Let me know if there are other
> outstanding bits that need to go into this repo.  Once that's done, I
> propose we push this to the ~elastic-ops namespace of which you're a member:
>
> https://launchpad.net/~elastic-ops
>
> And then promulgate the ~elastic-ops charm rev.  As Rick and others have
> mentioned, teams help distribute ongoing maintenance to people that have a
> vested interest in certain charms.
>
> @Simon / Bret:  you are on the ~onlineservices-charmers team that owns the
> current promulgated ES charm.  Would you be willing to relinquish
> maintainership to ~elastic-ops (and would you like to be members of that
> team)?
>
> Thanks!
> -Kevin Monroe
>
> On Mon, Aug 14, 2017 at 12:12 PM, Rick Harding  > wrote:
>
>> I'd also encourage you to test an upgrade path from the current charm to
>> yours to help comfort any current users of the charm that if there's a
>> change it'll go smoothly for users.
>>
>> On Mon, Aug 14, 2017 at 1:00 PM Tim Van Steenburgh <
>> tim.van.steenbu...@canonical.com> wrote:
>>
>>> Hey James, my point was that if you want to replace the existing
>>> top-level ES charm with your own, we would need agreement from the
>>> maintainers of the existing charm.
>>>
>>> On Mon, Aug 14, 2017 at 12:45 PM, James Beedy 
>>> wrote:
>>>
 @tim yeah ... possibly I'm not looking for promulgation then, and just
 need to push it and grant everyone because its already promulgated?

 On Mon, Aug 14, 2017 at 9:29 AM, Tim Van Steenburgh <
 tim.van.steenbu...@canonical.com> wrote:

> There's already a top-level elasticsearch charm.
> https://jujucharms.com/elasticsearch/
>
> On Mon, Aug 14, 2017 at 12:13 PM, James Beedy 
> wrote:
>
>> Request for promulgation of Elasticsearch. The Elasticsearch charm
>> can be found at cs:~jamesbeedy/elasticsearch-7.
>>
>> Thanks,
>>
>> James
>>
>> --
>> Juju mailing list
>> Juju@lists.ubuntu.com
>> Modify settings or unsubscribe at: https://lists.ubuntu.com/mailm
>> an/listinfo/juju
>>
>>
>

>>> --
>>> Juju mailing list
>>> Juju@lists.ubuntu.com
>>> Modify settings or unsubscribe at: https://lists.ubuntu.com/mailm
>>> an/listinfo/juju
>>>
>>
>> --
>> Juju mailing list
>> Juju@lists.ubuntu.com
>> Modify settings or unsubscribe at: https://lists.ubuntu.com/mailm
>> an/listinfo/juju
>>
>>
>
-- 
Juju mailing list
Juju@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju


Re: Elasticsearch Charm Promulgation

2017-08-18 Thread Kevin Monroe
Hey James!

First up, thanks so much for bringing the ES charm up to v5.x!  Can you
tell me if the source for cs:~jamesbeedy/elasticsearch-7 is the same as
what we've got here:

https://github.com/charms/charm-elasticsearch

Or is that source waiting on this PR to bring it in line with your es-7
charm:

https://github.com/charms/charm-elasticsearch/pull/2

The PR LGTM, so I'll merge it on your ack.  Let me know if there are other
outstanding bits that need to go into this repo.  Once that's done, I
propose we push this to the ~elastic-ops namespace of which you're a member:

https://launchpad.net/~elastic-ops

And then promulgate the ~elastic-ops charm rev.  As Rick and others have
mentioned, teams help distribute ongoing maintenance to people that have a
vested interest in certain charms.

@Simon / Bret:  you are on the ~onlineservices-charmers team that owns the
current promulgated ES charm.  Would you be willing to relinquish
maintainership to ~elastic-ops (and would you like to be members of that
team)?

Thanks!
-Kevin Monroe

On Mon, Aug 14, 2017 at 12:12 PM, Rick Harding 
wrote:

> I'd also encourage you to test an upgrade path from the current charm to
> yours to help comfort any current users of the charm that if there's a
> change it'll go smoothly for users.
>
> On Mon, Aug 14, 2017 at 1:00 PM Tim Van Steenburgh <
> tim.van.steenbu...@canonical.com> wrote:
>
>> Hey James, my point was that if you want to replace the existing
>> top-level ES charm with your own, we would need agreement from the
>> maintainers of the existing charm.
>>
>> On Mon, Aug 14, 2017 at 12:45 PM, James Beedy 
>> wrote:
>>
>>> @tim yeah ... possibly I'm not looking for promulgation then, and just
>>> need to push it and grant everyone because its already promulgated?
>>>
>>> On Mon, Aug 14, 2017 at 9:29 AM, Tim Van Steenburgh <
>>> tim.van.steenbu...@canonical.com> wrote:
>>>
 There's already a top-level elasticsearch charm.
 https://jujucharms.com/elasticsearch/

 On Mon, Aug 14, 2017 at 12:13 PM, James Beedy 
 wrote:

> Request for promulgation of Elasticsearch. The Elasticsearch charm can
> be found at cs:~jamesbeedy/elasticsearch-7.
>
> Thanks,
>
> James
>
> --
> Juju mailing list
> Juju@lists.ubuntu.com
> Modify settings or unsubscribe at: https://lists.ubuntu.com/
> mailman/listinfo/juju
>
>

>>>
>> --
>> Juju mailing list
>> Juju@lists.ubuntu.com
>> Modify settings or unsubscribe at: https://lists.ubuntu.com/
>> mailman/listinfo/juju
>>
>
> --
> Juju mailing list
> Juju@lists.ubuntu.com
> Modify settings or unsubscribe at: https://lists.ubuntu.com/
> mailman/listinfo/juju
>
>
-- 
Juju mailing list
Juju@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju


Re: Elasticsearch Charm Promulgation

2017-08-14 Thread Rick Harding
I'd also encourage you to test an upgrade path from the current charm to
yours to help comfort any current users of the charm that if there's a
change it'll go smoothly for users.

On Mon, Aug 14, 2017 at 1:00 PM Tim Van Steenburgh <
tim.van.steenbu...@canonical.com> wrote:

> Hey James, my point was that if you want to replace the existing top-level
> ES charm with your own, we would need agreement from the maintainers of the
> existing charm.
>
> On Mon, Aug 14, 2017 at 12:45 PM, James Beedy 
> wrote:
>
>> @tim yeah ... possibly I'm not looking for promulgation then, and just
>> need to push it and grant everyone because its already promulgated?
>>
>> On Mon, Aug 14, 2017 at 9:29 AM, Tim Van Steenburgh <
>> tim.van.steenbu...@canonical.com> wrote:
>>
>>> There's already a top-level elasticsearch charm.
>>> https://jujucharms.com/elasticsearch/
>>>
>>> On Mon, Aug 14, 2017 at 12:13 PM, James Beedy 
>>> wrote:
>>>
 Request for promulgation of Elasticsearch. The Elasticsearch charm can
 be found at cs:~jamesbeedy/elasticsearch-7.

 Thanks,

 James

 --
 Juju mailing list
 Juju@lists.ubuntu.com
 Modify settings or unsubscribe at:
 https://lists.ubuntu.com/mailman/listinfo/juju


>>>
>>
> --
> Juju mailing list
> Juju@lists.ubuntu.com
> Modify settings or unsubscribe at:
> https://lists.ubuntu.com/mailman/listinfo/juju
>
-- 
Juju mailing list
Juju@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju


Re: Elasticsearch Charm Promulgation

2017-08-14 Thread Tim Van Steenburgh
Hey James, my point was that if you want to replace the existing top-level
ES charm with your own, we would need agreement from the maintainers of the
existing charm.

On Mon, Aug 14, 2017 at 12:45 PM, James Beedy  wrote:

> @tim yeah ... possibly I'm not looking for promulgation then, and just
> need to push it and grant everyone because its already promulgated?
>
> On Mon, Aug 14, 2017 at 9:29 AM, Tim Van Steenburgh <
> tim.van.steenbu...@canonical.com> wrote:
>
>> There's already a top-level elasticsearch charm. https://jujucharms.com/
>> elasticsearch/
>>
>> On Mon, Aug 14, 2017 at 12:13 PM, James Beedy 
>> wrote:
>>
>>> Request for promulgation of Elasticsearch. The Elasticsearch charm can
>>> be found at cs:~jamesbeedy/elasticsearch-7.
>>>
>>> Thanks,
>>>
>>> James
>>>
>>> --
>>> Juju mailing list
>>> Juju@lists.ubuntu.com
>>> Modify settings or unsubscribe at: https://lists.ubuntu.com/mailm
>>> an/listinfo/juju
>>>
>>>
>>
>
-- 
Juju mailing list
Juju@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju


Re: Elasticsearch Charm Promulgation

2017-08-14 Thread James Beedy
@tim yeah ... possibly I'm not looking for promulgation then, and just need
to push it and grant everyone because its already promulgated?

On Mon, Aug 14, 2017 at 9:29 AM, Tim Van Steenburgh <
tim.van.steenbu...@canonical.com> wrote:

> There's already a top-level elasticsearch charm. https://jujucharms.com/
> elasticsearch/
>
> On Mon, Aug 14, 2017 at 12:13 PM, James Beedy 
> wrote:
>
>> Request for promulgation of Elasticsearch. The Elasticsearch charm can be
>> found at cs:~jamesbeedy/elasticsearch-7.
>>
>> Thanks,
>>
>> James
>>
>> --
>> Juju mailing list
>> Juju@lists.ubuntu.com
>> Modify settings or unsubscribe at: https://lists.ubuntu.com/mailm
>> an/listinfo/juju
>>
>>
>
-- 
Juju mailing list
Juju@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju


Re: Elasticsearch Charm Promulgation

2017-08-14 Thread Tim Van Steenburgh
There's already a top-level elasticsearch charm.
https://jujucharms.com/elasticsearch/

On Mon, Aug 14, 2017 at 12:13 PM, James Beedy  wrote:

> Request for promulgation of Elasticsearch. The Elasticsearch charm can be
> found at cs:~jamesbeedy/elasticsearch-7.
>
> Thanks,
>
> James
>
> --
> Juju mailing list
> Juju@lists.ubuntu.com
> Modify settings or unsubscribe at: https://lists.ubuntu.com/
> mailman/listinfo/juju
>
>
-- 
Juju mailing list
Juju@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju


Elasticsearch Charm Promulgation

2017-08-14 Thread James Beedy
Request for promulgation of Elasticsearch. The Elasticsearch charm can be
found at cs:~jamesbeedy/elasticsearch-7.

Thanks,

James
-- 
Juju mailing list
Juju@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju