[ClusterLabs Developers] resource-agents v4.1.0 rc1

[Dejan Muhamedagic]

Hi,

On Fri, Nov 17, 2017 at 10:25:55AM +0100, Oyvind Albrigtsen wrote:
> On 17/11/17 09:13 +0100, Kristoffer Grönlund wrote:
> > Dejan Muhamedagic <dejanmm at fastmail.fm> writes:
> > 
> > > Hi,
> > > 
> > > On Tue, Nov 14, 2017 at 03:16:04PM +0100, Oyvind Albrigtsen wrote:
> > > > ClusterLabs is happy to announce resource-agents v4.1.0 rc1.
> > > > Source code is available at:
> > > > https://github.com/ClusterLabs/resource-agents/releases/tag/v4.1.0rc1
> > > > 
> > > > The most significant enhancements in this release are:
> > > > - new resource agents:
> > > >  - aws-vpc-route53
> > > >  - LVM-activate
> > > 
> > > There was a long and, unfortunately, fruitless discussion
> > > regarding this new version of LVM RA and whether it can be merged
> > > with the existing LVM RA. In spite of no consensus, Kristoffer
> > > eventually merged it. There is now a plugin like system in LVM
> > > which should make adding new mechanisms such as lvmlockd easy,
> > > but the contributors didn't try yet to adjust the new LVM.
> > 
> > Yeah, my apologies for jumping the gun. I hope we can work this out
> > before release. :/
> > 
> > 
> > > Furthermore, in case we agree that having two LVM versions is the
> > > best way forward, such refactoring would also significantly
> > > reduce code duplication.
> > > 
> > > Thanks,
> > > 
> > > Dejan
> > > 
> > > P.S. Right now irrelevant, but LVM-activate doesn't seem like a
> > > good name.
> > 
> > If nothing else, it's in keeping with the rest of the mess of bad names
> > ;) Maybe we should decide on a naming standard? Abbreviations or no
> > abbreviations, CamelCase or lowercase, or lower_case? This could be
> > enforced through the CI checks, and the Makefile could make sure to
> > install aliases to the old names so as not to break existing
> > installations.
> I think we should keep them as is for current agents, but new agents
> should be lowercase with an optional -'s if that clarifies their
> usage.

My point was that it is arguably a pleonasm. The "activate" part
is superfluous. Sorry for not being clearer.

As for bad names, we need to look no further than the LVM RA:
in volgrpname the "name" part is just as much necessary.

> > Maybe we could even move IPaddr2 to ip or IP, and move the now-outdated
> > agents to a legacy provider...

Yeah, I hope we don't need to get into that ;-)

Cheers,

Dejan

> > 
> > Cheers,
> > Kristoffer
> > 
> > > 
> > > 
> > > 
> > > 
> > > >  - minio
> > > >  - NodeUtilization
> > > >  - oraasm
> > > >  - ovsmonitor
> > > >  - rkt
> > > >  - ZFS
> > > > 
> > > > - bugfixes and enhancements:
> > > >  - aws*: fixes and improvements
> > > >  - CTDB: fixes for newer versions
> > > >  - CTDB: fix for --logfile being replaced with --logging
> > > >  - DB2: fix HADR support for DB2 V98+
> > > >  - docker: add docker-native healthcheck
> > > >  - galera: fix for MariaDB 10.1.21+
> > > >  - mysql: set correct master score after maintenance mode
> > > >  - ocf-shellfuncs: improve locking (ocf_take_lock())
> > > >  - pgsql: add support for PostgreSQL 10
> > > >  - pgsql: allow dynamic membership
> > > >  - rabbitmq-cluster: fix to work on Pacemaker remote nodes
> > > > 
> > > > The full list of changes for resource-agents is available at:
> > > > https://github.com/ClusterLabs/resource-agents/blob/v4.1.0rc1/ChangeLog
> > > > 
> > > > Everyone is encouraged to download and test the new release candidate.
> > > > We do many regression tests and simulations, but we can't cover all
> > > > possible use cases, so your feedback is important and appreciated.
> > > > 
> > > > Many thanks to all the contributors to this release.
> > > > 
> > > > 
> > > > Best,
> > > > The resource-agents maintainers
> > > > 
> > > > _______________________________________________
> > > > Developers mailing list
> > > > Developers at clusterlabs.org
> > > > http://lists.clusterlabs.org/mailman/listinfo/developers
> > > 
> > > _______________________________________________
> > > Developers mailing list
> > > Developers at clusterlabs.org
> > > http://lists.clusterlabs.org/mailman/listinfo/developers
> > > 
> > 
> > -- 
> > // Kristoffer Grönlund
> > // kgronlund at suse.com
> > 
> > _______________________________________________
> > Developers mailing list
> > Developers at clusterlabs.org
> > http://lists.clusterlabs.org/mailman/listinfo/developers