Re: [ClusterLabs] crmsh Question/Feature

2016-01-25 Thread Marc Smith
Thanks; I've submitted the pull request.

--Marc

On Mon, Jan 25, 2016 at 1:30 AM, Kristoffer Grönlund  wrote:
> Hi,
>
> Marc Smith  writes:
>
>> I think "push" fits. I haven't contributed to this project before --
>> I've updated the following from the master branch:
>> doc/crm.8.adoc
>> modules/cibconfig.py
>> modules/ui_configure.py
>>
>
> That sounds reasonable to me, yes.
>
>> I assume doc/website-v1/man-*.adoc gets created from crm.8.adoc or do
>> I update those as well (or just 2.0)?
>>
>
> No need, those are copies of the reference manual for the stable
> versions of the 1.x and 2.x branch. I update them when I release new
> versions.
>
> Cheers,
> Kristoffer
>
> --
> // Kristoffer Grönlund
> // kgronl...@suse.com

___
Users mailing list: Users@clusterlabs.org
http://clusterlabs.org/mailman/listinfo/users

Project Home: http://www.clusterlabs.org
Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
Bugs: http://bugs.clusterlabs.org


Re: [ClusterLabs] crmsh Question/Feature

2016-01-24 Thread Kristoffer Grönlund
Hi,

Marc Smith  writes:

> I think "push" fits. I haven't contributed to this project before --
> I've updated the following from the master branch:
> doc/crm.8.adoc
> modules/cibconfig.py
> modules/ui_configure.py
>

That sounds reasonable to me, yes.

> I assume doc/website-v1/man-*.adoc gets created from crm.8.adoc or do
> I update those as well (or just 2.0)?
>

No need, those are copies of the reference manual for the stable
versions of the 1.x and 2.x branch. I update them when I release new
versions.

Cheers,
Kristoffer

-- 
// Kristoffer Grönlund
// kgronl...@suse.com

___
Users mailing list: Users@clusterlabs.org
http://clusterlabs.org/mailman/listinfo/users

Project Home: http://www.clusterlabs.org
Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
Bugs: http://bugs.clusterlabs.org


Re: [ClusterLabs] crmsh Question/Feature

2016-01-22 Thread Marc Smith
I think "push" fits. I haven't contributed to this project before --
I've updated the following from the master branch:
doc/crm.8.adoc
modules/cibconfig.py
modules/ui_configure.py

I assume doc/website-v1/man-*.adoc gets created from crm.8.adoc or do
I update those as well (or just 2.0)?


Thanks,

Marc



On Thu, Jan 21, 2016 at 2:06 PM, Jan Pokorný  wrote:
> On 21/01/16 08:34 -0500, Marc Smith wrote:
>> On Thu, Jan 21, 2016 at 12:39 AM, Kristoffer Grönlund
>>  wrote:
>>> Hi,
>>>
>>> Marc Smith  writes:
>>>
 My question: Is there any other way to replicate this behavior using
 crmsh and importing a configuration from a file? If not, would the
 group be open to accepting a patch that adds a third load-action
 method that calls save() with no_remove=False?
>>>
>>> Absolutely! It sounds like something we should support.
>>>
>>> You can create a pull request here:
>>>
>>> https://github.com/ClusterLabs/crmsh
>>
>> Okay, I'll start on this. Any thoughts on the "method" keyword to use
>> that will describe this? I was thinking "merge" at first, but maybe a
>> different verb?
>
> push?
>
> Disclaimer: this is what pcs uses with replace semantics (as requested).
>
> --
> Jan (Poki)
>
> ___
> Users mailing list: Users@clusterlabs.org
> http://clusterlabs.org/mailman/listinfo/users
>
> Project Home: http://www.clusterlabs.org
> Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
> Bugs: http://bugs.clusterlabs.org
>

___
Users mailing list: Users@clusterlabs.org
http://clusterlabs.org/mailman/listinfo/users

Project Home: http://www.clusterlabs.org
Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
Bugs: http://bugs.clusterlabs.org


Re: [ClusterLabs] crmsh Question/Feature

2016-01-21 Thread Jan Pokorný
On 21/01/16 08:34 -0500, Marc Smith wrote:
> On Thu, Jan 21, 2016 at 12:39 AM, Kristoffer Grönlund
>  wrote:
>> Hi,
>> 
>> Marc Smith  writes:
>> 
>>> My question: Is there any other way to replicate this behavior using
>>> crmsh and importing a configuration from a file? If not, would the
>>> group be open to accepting a patch that adds a third load-action
>>> method that calls save() with no_remove=False?
>> 
>> Absolutely! It sounds like something we should support.
>> 
>> You can create a pull request here:
>> 
>> https://github.com/ClusterLabs/crmsh
> 
> Okay, I'll start on this. Any thoughts on the "method" keyword to use
> that will describe this? I was thinking "merge" at first, but maybe a
> different verb?

push?

Disclaimer: this is what pcs uses with replace semantics (as requested).

-- 
Jan (Poki)


pgpjvGA_58_qF.pgp
Description: PGP signature
___
Users mailing list: Users@clusterlabs.org
http://clusterlabs.org/mailman/listinfo/users

Project Home: http://www.clusterlabs.org
Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
Bugs: http://bugs.clusterlabs.org


Re: [ClusterLabs] crmsh Question/Feature

2016-01-21 Thread Marc Smith
On Thu, Jan 21, 2016 at 12:39 AM, Kristoffer Grönlund
 wrote:
> Hi,
>
> Marc Smith  writes:
>
>> My question: Is there any other way to replicate this behavior using
>> crmsh and importing a configuration from a file? If not, would the
>> group be open to accepting a patch that adds a third load-action
>> method that calls save() with no_remove=False?
>
> Absolutely! It sounds like something we should support.
>
> You can create a pull request here:
>
> https://github.com/ClusterLabs/crmsh

Okay, I'll start on this. Any thoughts on the "method" keyword to use
that will describe this? I was thinking "merge" at first, but maybe a
different verb?


--Marc

>
> Cheers,
> Kristoffer
>
> --
> // Kristoffer Grönlund
> // kgronl...@suse.com

___
Users mailing list: Users@clusterlabs.org
http://clusterlabs.org/mailman/listinfo/users

Project Home: http://www.clusterlabs.org
Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
Bugs: http://bugs.clusterlabs.org


Re: [ClusterLabs] crmsh Question/Feature

2016-01-20 Thread Kristoffer Grönlund
Hi,

Marc Smith  writes:

> My question: Is there any other way to replicate this behavior using
> crmsh and importing a configuration from a file? If not, would the
> group be open to accepting a patch that adds a third load-action
> method that calls save() with no_remove=False?

Absolutely! It sounds like something we should support.

You can create a pull request here:

https://github.com/ClusterLabs/crmsh

Cheers,
Kristoffer

-- 
// Kristoffer Grönlund
// kgronl...@suse.com

___
Users mailing list: Users@clusterlabs.org
http://clusterlabs.org/mailman/listinfo/users

Project Home: http://www.clusterlabs.org
Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
Bugs: http://bugs.clusterlabs.org