darrent at akurit
Apr 28, 2011, 11:04 PM
Post #1 of 2
Re: [Openais] An OCF agent for LXC (Linux Containers) - Linux-HA-Dev Digest, Vol 89, Issue 30
Thank you for the update.
I'll thread my remaining replies into the message :-)
On Thu, 2011-04-28 at 07:46 -0600,
linux-ha-dev-request [at] lists wrote:
> Date: Thu, 28 Apr 2011 11:56:30 +0200
> From: Florian Haas <florian.haas [at] linbit>
> Subject: Re: [Linux-ha-dev] [Openais] An OCF agent for LXC (Linux
> To: High-Availability Linux Development List
> <linux-ha-dev [at] lists>
> Message-ID: <4DB939CE.5000009 [at] linbit>
> Content-Type: text/plain; charset="utf-8"
> On 2011-04-28 10:21, Darren Thompson wrote:
> > Florina/TEAM
> > Thanks for your input and the link to the guidelines
> > I have updated my original ocf file in line with the guidlines, it
> > gave me a few tips on how to do things "better" so was well worth
> > time spent.
> > Please find the updated ocf file for LXC contianers as a cluster
> > resource attached.
> > Since I'm not an actual developer (or even a career coder)
> Do you think I am?
Until Today, i have had no experience whatsoever with github, so
compared to me... yes...
> > I do not have
> > the facility to host my own github fork so would appreciate
> > adopting this and integrating it into their git repository.
> OK, I have added this to a separate "lxc" branch in my own github
> I'd appreciate if you could at least get yourself an account on github
> so you can comment on commit line notes.
> I have added my comments to this page:
Yep, done.. I responded to nearly all of the you points (most of the
time to say, yep... agree).
You posted my "first attempt" and not the latest version, is it possible
to add that one as it addresses some( most hopefully) of the issues you
There are still some valid points you have raised however, So I'm going
to try to incorporate them into a "third version".
Is there some "clever way" of re-integrating all of this? (did I mention
that I'm not normally a coder).
> Some of those may have already been addresses in your updated version,
> but to keep things simple I've kept my comments to one commit for the
> time being.
> PS: We can stop CC'ing the openais list, this is in no way
> Corosync/OpenAIS related.
Agreed, I will stop pestering that list now :-)