brooklyn-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mark McKenna <m4rkmcke...@apache.org>
Subject Re: [PROPOSAL] Delete "classic-mode" (always use Karaf)
Date Mon, 09 Oct 2017 13:04:09 GMT
+1

*Mark McKenna*

*Twitter ::* @m4rkmckenna <https://twitter.com/m4rkmckenna>

*Github :: *m4rkmckenna <https://github.com/m4rkmckenna>

*PGP :: A7A9 24DE 638C 681A 8DEA FAD4 2B5D C759 B1EB 76A7
<https://pgp.mit.edu/pks/lookup?op=get&search=0x2B5DC759B1EB76A7>*

On 9 October 2017 at 13:09, Graeme Miller <graeme@cloudsoft.io> wrote:

> +1
>
> On 9 October 2017 at 12:56, Thomas Bouron <thomas.bouron@cloudsoftcorp.com
> >
> wrote:
>
> > +1
> >
> > On Mon, 9 Oct 2017 at 12:39 Geoff Macartney <
> geoff.macartney@cloudsoft.io>
> > wrote:
> >
> > > +1
> > >
> > > On Mon, 9 Oct 2017 at 12:38 Duncan Godwin <dgodwin@redwintechnologies.
> > com>
> > > wrote:
> > >
> > > > +1
> > > >
> > > > On 9 October 2017 at 12:24, Richard Downer <richard@apache.org>
> wrote:
> > > >
> > > > > I offer my opinion through the medium of GIFs:
> > > > >
> > > > > https://media.giphy.com/media/vohOR29F78sGk/giphy.gif
> > > > >
> > > > > Richard.
> > > > >
> > > > >
> > > > > On 9 October 2017 at 12:13, Aled Sage <aled.sage@gmail.com>
wrote:
> > > > >
> > > > > > Hi all,
> > > > > >
> > > > > > I propose that we *delete* Brooklyn classic-mode from master
now,
> > in
> > > > > > preparation for the 1.0.0 release.
> > > > > >
> > > > > > ---
> > > > > >
> > > > > > In 0.12.0, we switched the main distro to be the karaf-mode.
We
> > also
> > > > > built
> > > > > > the classic-mode distro - the intent being for users to have
a
> > usable
> > > > > > classic mode, rather than being forced immediately to switch
to
> > karaf
> > > > > > without advanced warning.
> > > > > >
> > > > > > However, we unfortunately did not deprecate classic-mode as
> clearly
> > > as
> > > > we
> > > > > > should have (e.g. didn't explicitly say that it will be deleted
> in
> > an
> > > > > > upcoming release, and didn't deprecate the `Main` class).
> > > > > >
> > > > > > I think it's still ok to delete it for several reasons:
> > > > > >
> > > > > > 1. This is a "mode" of running Brooklyn, rather than underlying
> > > > > >    Brooklyn functionality.
> > > > > > 2. The `Main` class [1] etc should not be considered part of
our
> > > > > >    *public* API (even though we didn't explicitly tell people
> that
> > it
> > > > > >    was internal).
> > > > > > 3. As part of 0.12.0 release, we added to the docs instructions
> for
> > > > > >    upgrading to Karaf [2].
> > > > > > 4. Supporting classic to the same standard as Karaf will become
> > > > > >    increasingly painful as we do more and more with Bundles!
> > > > > > 5. It's a major release, so if we are going to delete it then
> 1.0.0
> > > is
> > > > > >    the perfect time!
> > > > > >
> > > > > > Aled
> > > > > >
> > > > > > [1] https://github.com/apache/brooklyn-server/blob/master/server
> > > > > > -cli/src/main/java/org/apache/brooklyn/cli/Main.java
> > > > > >
> > > > > > [2] http://brooklyn.apache.org/v/0.12.0/ops/upgrade.html#upgrade
> > > > > > -from-apache-brooklyn-0110-and-below
> > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > >
> > --
> >
> > Thomas Bouron • Senior Software Engineer @ Cloudsoft Corporation •
> > https://cloudsoft.io/
> > Github: https://github.com/tbouron
> > Twitter: https://twitter.com/eltibouron
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message