brooklyn-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Thomas Bouron <thomas.bou...@cloudsoftcorp.com>
Subject Re: [PROPOSAL] Delete "classic-mode" (always use Karaf)
Date Mon, 09 Oct 2017 11:56:08 GMT
+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