Re: [jira] [Commented] (IGNITE-4052) Add ability to set up users for MESOS

[Вадим Опольский]

Hello , Prachi!

I reviewed reference. Removing occurrence it was the right action, Thank
you!

Vadim Opolski



2017-05-23 2:19 GMT+03:00 Prachi Garg (JIRA) :

>
> [ https://issues.apache.org/jira/browse/IGNITE-4052?page=com.
> atlassian.jira.plugin.system.issuetabpanels:comment-tabpane
> l=16020380#comment-16020380 ]
>
> Prachi Garg commented on IGNITE-4052:
> -
>
> [~ntikhonov] I have merged the changes and also made some edits. Within
> the Json code block, the following was mentioned twice.
> "MESOS_USER" : "userA",
>  "MESOS_ROLE" :  "role1"
>
> I have removed one occurrence. Please review and confirm that it was the
> right action.
>
> > Add ability to set up users for MESOS
> > -
> >
> > Key: IGNITE-4052
> > URL: https://issues.apache.org/jira/browse/IGNITE-4052
> > Project: Ignite
> >  Issue Type: Improvement
> >  Components: general
> >Affects Versions: 1.7
> >Reporter: Nikolay Tikhonov
> >Assignee: Prachi Garg
> >Priority: Trivial
> >
> > In current implementation Ignite Mesos Framework connects to MESOS
> cluster via current user. Need to add ability to configure this parameters
> via system env properties. Also need to add properties for mesos role.
> > See org/apache/ignite/mesos/IgniteFramework.java:537
>
>
>
> --
> This message was sent by Atlassian JIRA
> (v6.3.15#6346)
>

Fwd: [jira] [Commented] (IGNITE-4052) Add ability to set up users for MESOS

[Вадим Опольский]

Hello Prachi!

Have you seen the edits  in readme.io for Mesos Deployment page:
https://dash.readme.io/project/apacheignite/v2.0/suggested

Vadim Opolski



-- Forwarded message --
From: Denis Magda (JIRA) <j...@apache.org>
Date: 2017-05-15 23:02 GMT+03:00
Subject: [jira] [Commented] (IGNITE-4052) Add ability to set up users for
MESOS
To: vaopols...@gmail.com



[ https://issues.apache.org/jira/browse/IGNITE-4052?page=
com.atlassian.jira.plugin.system.issuetabpanels:comment-
tabpanel=16011240#comment-16011240 ]

Denis Magda commented on IGNITE-4052:
-

[~pgarg], please review and apply the edits suggested by Vadim in readme.io
for Mesos Deployment page:
https://dash.readme.io/project/apacheignite/v2.0/suggested

> Add ability to set up users for MESOS
> -
>
> Key: IGNITE-4052
> URL: https://issues.apache.org/jira/browse/IGNITE-4052
> Project: Ignite
>  Issue Type: Improvement
>  Components: general
>Affects Versions: 1.7
>Reporter: Nikolay Tikhonov
>Assignee: Prachi Garg
>Priority: Trivial
>
> In current implementation Ignite Mesos Framework connects to MESOS
cluster via current user. Need to add ability to configure this parameters
via system env properties. Also need to add properties for mesos role.
> See org/apache/ignite/mesos/IgniteFramework.java:537



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Re: [jira] [Commented] (IGNITE-4052) Add ability to set up users for MESOS

[Вадим Опольский]

Hello Prachi!

I reassigned issue https://issues.apache.org/jira/browse/IGNITE-4052 to you
to approve changes of page -
https://apacheignite.readme.io/docs/mesos-deployment#section-ignite-mesos-framework

Vadim

2017-05-11 11:54 GMT+03:00 Вадим Опольский :

> Hello guys!
>
> I want to update https://apacheignite.readme.io/docs/mesos-deployment
> with new text according with https://cwiki.apache.org/
> confluence/display/IGNITE/Documentation
> But have a problem with authentication, registered login
> vaopols...@gmail.com.
> Help me, please.
>
> New text:
>
> 3.Copy the following application definition (in JSON format) and save to
> marathon.json file. Update any parameters which would like to change.
>
>
>
> *A role name must be a valid directory name, so it cannot:  • Be an empty
> string  • Be . or ..  • Start with -  • Contain any slash, backspace, or
> whitespace character*
>  If doesn't set restriction on cluster then the framework will try to
> occupy all resources in Mesos cluster. See Configuration section below.
>
> JSON
> {
>   "id": "ignition",
>   "instances": 1,
>   "cpus": 2,
>   "mem": 2048,
>   "ports": [0],
>   "uris": [
> "http://host/ignite-mesos--jar-with-dependencies.jar"
>   ],
>   "env": {
> "IGNITE_NODE_COUNT": "4",
> "MESOS_MASTER_URL": "zk://localhost:2181/mesos",
> "IGNITE_RUN_CPU_PER_NODE": "2",
> "IGNITE_MEMORY_PER_NODE": "2048",
> "IGNITE_VERSION": "1.0.5",
>
>
> *"MESOS_USER" : "userA","MESOS_ROLE" :  "role1"*  },
>   "cmd": "java -jar ignite-mesos--
> jar-with-dependencies.jar"
> }
>
> Vadim
>
>
> 2017-04-27 19:29 GMT+03:00 Nikolay Tikhonov (JIRA) :
>
>>
>> [ https://issues.apache.org/jira/browse/IGNITE-4052?page=com.
>> atlassian.jira.plugin.system.issuetabpanels:comment-tabpane
>> l=15986959#comment-15986959 ]
>>
>> Nikolay Tikhonov commented on IGNITE-4052:
>> --
>>
>> [~javaller],
>> Thank you for your contribution. I've fixed some minors issue and pushed
>> your changes into {{ignite-4052}} branch. Please look at the changes.
>>
>> >I dont work with Mesos and think that anyone who has experience should
>> make it. OK?
>> I think that it good time to try it. ;) It looks strange when developer
>> don't run own code. [2] Also would be great to update docs [1]. Use for it
>> {{suggest edits}} button.
>>
>> 1. https://apacheignite.readme.io/docs/mesos-deployment
>> 2. http://mesos.apache.org/gettingstarted/
>>
>> > Add ability to set up users for MESOS
>> > -
>> >
>> > Key: IGNITE-4052
>> > URL: https://issues.apache.org/jira/browse/IGNITE-4052
>> > Project: Ignite
>> >  Issue Type: Improvement
>> >  Components: general
>> >Affects Versions: 1.7
>> >Reporter: Nikolay Tikhonov
>> >Assignee: Vadim Opolski
>> >Priority: Trivial
>> >
>> > In current implementation Ignite Mesos Framework connects to MESOS
>> cluster via current user. Need to add ability to configure this parameters
>> via system env properties. Also need to add properties for mesos role.
>> > See org/apache/ignite/mesos/IgniteFramework.java:537
>>
>>
>>
>> --
>> This message was sent by Atlassian JIRA
>> (v6.3.15#6346)
>>
>
>

Re: [jira] [Commented] (IGNITE-4052) Add ability to set up users for MESOS

[Вадим Опольский]

Hello guys!

I want to update https://apacheignite.readme.io/docs/mesos-deployment with
new text according with
https://cwiki.apache.org/confluence/display/IGNITE/Documentation
But have a problem with authentication, registered login
vaopols...@gmail.com.
Help me, please.

New text:

3.Copy the following application definition (in JSON format) and save to
marathon.json file. Update any parameters which would like to change.



*A role name must be a valid directory name, so it cannot:  • Be an empty
string  • Be . or ..  • Start with -  • Contain any slash, backspace, or
whitespace character*
 If doesn't set restriction on cluster then the framework will try to
occupy all resources in Mesos cluster. See Configuration section below.

JSON
{
  "id": "ignition",
  "instances": 1,
  "cpus": 2,
  "mem": 2048,
  "ports": [0],
  "uris": [
"http://host/ignite-mesos--jar-with-dependencies.jar"
  ],
  "env": {
"IGNITE_NODE_COUNT": "4",
"MESOS_MASTER_URL": "zk://localhost:2181/mesos",
"IGNITE_RUN_CPU_PER_NODE": "2",
"IGNITE_MEMORY_PER_NODE": "2048",
"IGNITE_VERSION": "1.0.5",


*"MESOS_USER" : "userA","MESOS_ROLE" :  "role1"*  },
  "cmd": "java -jar ignite-mesos--jar-with-dependencies.jar"
}

Vadim


2017-04-27 19:29 GMT+03:00 Nikolay Tikhonov (JIRA) :

>
> [ https://issues.apache.org/jira/browse/IGNITE-4052?page=
> com.atlassian.jira.plugin.system.issuetabpanels:comment-
> tabpanel=15986959#comment-15986959 ]
>
> Nikolay Tikhonov commented on IGNITE-4052:
> --
>
> [~javaller],
> Thank you for your contribution. I've fixed some minors issue and pushed
> your changes into {{ignite-4052}} branch. Please look at the changes.
>
> >I dont work with Mesos and think that anyone who has experience should
> make it. OK?
> I think that it good time to try it. ;) It looks strange when developer
> don't run own code. [2] Also would be great to update docs [1]. Use for it
> {{suggest edits}} button.
>
> 1. https://apacheignite.readme.io/docs/mesos-deployment
> 2. http://mesos.apache.org/gettingstarted/
>
> > Add ability to set up users for MESOS
> > -
> >
> > Key: IGNITE-4052
> > URL: https://issues.apache.org/jira/browse/IGNITE-4052
> > Project: Ignite
> >  Issue Type: Improvement
> >  Components: general
> >Affects Versions: 1.7
> >Reporter: Nikolay Tikhonov
> >Assignee: Vadim Opolski
> >Priority: Trivial
> >
> > In current implementation Ignite Mesos Framework connects to MESOS
> cluster via current user. Need to add ability to configure this parameters
> via system env properties. Also need to add properties for mesos role.
> > See org/apache/ignite/mesos/IgniteFramework.java:537
>
>
>
> --
> This message was sent by Atlassian JIRA
> (v6.3.15#6346)
>

Re: [jira] [Commented] (IGNITE-4052) Add ability to set up users for MESOS

[Вадим Опольский]

Hi guys!

Nikolay, the next improvements are maden (
https://github.com/apache/ignite/pull/1783):

   - changed code according with code style (extra space, if else blocks,
   javadoc)
   - written two tests one which will testing new parametes and other which
   will testing role validation
   - used PowerMock framework to getRole getUser testing
   - ridden #setEnv method

I did not test this parameters on real mesos cluster.

Nikolay, I dont work with Mesos and think that anyone who has experience
should make it. OK?

Vadim Opolski

2017-04-19 17:01 GMT+03:00 Nikolay Tikhonov (JIRA) :

>
> [ https://issues.apache.org/jira/browse/IGNITE-4052?page=
> com.atlassian.jira.plugin.system.issuetabpanels:comment-
> tabpanel=15974725#comment-15974725 ]
>
> Nikolay Tikhonov commented on IGNITE-4052:
> --
>
> [~javaller]
> I've looked at changes and have minor comments:
> * code style (extra space, if esle blocks, javadoc and etc.
> https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute);
> * let's split #testIgniteFramework() test: one which will testing new
> parametes and other which will testing role validation;
> * let's to use PowerMock in test (distributed under apache 2.0 licence
> which acceptable for our product) and rid #setEnv method;
>
> And again, did you test this parameters on real mesos cluster?
>
> Thank you for your contribution.
>
> > Add ability to set up users for MESOS
> > -
> >
> > Key: IGNITE-4052
> > URL: https://issues.apache.org/jira/browse/IGNITE-4052
> > Project: Ignite
> >  Issue Type: Improvement
> >  Components: general
> >Affects Versions: 1.7
> >Reporter: Nikolay Tikhonov
> >Assignee: Vadim Opolski
> >Priority: Trivial
> >
> > In current implementation Ignite Mesos Framework connects to MESOS
> cluster via current user. Need to add ability to configure this parameters
> via system env properties. Also need to add properties for mesos role.
> > See org/apache/ignite/mesos/IgniteFramework.java:537
>
>
>
> --
> This message was sent by Atlassian JIRA
> (v6.3.15#6346)
>

Re: [jira] [Commented] (IGNITE-4052) Add ability to set up users for MESOS

[Nikolai Tikhonov]

Vadim,

I've left comment in the ticket.

On Wed, Apr 12, 2017 at 3:00 PM, Вадим Опольский 
wrote:

> Hello guys!
>
> Nikolay, I looked at cassandra mesos integration, but cant find how to
> test ability to configure mesos and user via system env properties. I will
> continue to search in another projects which use Mesos.
>
> Review please pull request - https://github.com/vadopolski/ignite/pull/2.
> It contains new test and method for setting system environments.
>
> Can you close issue https://issues.apache.org/jira/browse/IGNITE-4052 and
> open new, about testing Mesos Integration.
>
> Vadim Opolski
>
> 2017-04-03 12:13 GMT+03:00 Nikolay Tikhonov (JIRA) :
>
>>
>> [ https://issues.apache.org/jira/browse/IGNITE-4052?page=com.a
>> tlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&
>> focusedCommentId=15953167#comment-15953167 ]
>>
>> Nikolay Tikhonov commented on IGNITE-4052:
>> --
>>
>> [~javaller],
>> For example you can look at cassandra mesos integration
>> https://github.com/mesosphere/dcos-cassandra-service/blob/5d
>> 6d15e646f1df2fa10fa98d91993d3a6b4719c4/cassandra-commons/src
>> /main/java/com/mesosphere/dcos/cassandra/common/config/ServiceConfig.java.
>> List of application which use Mesos not secret and can be found on official
>> mesos site http://mesos.apache.org/documentation/latest/frameworks. ;)
>>
>> I'm really don't understand how the libs can help you. Could you clarify
>> it?
>>
>> > Add ability to set up users for MESOS
>> > -
>> >
>> > Key: IGNITE-4052
>> > URL: https://issues.apache.org/jira/browse/IGNITE-4052
>> > Project: Ignite
>> >  Issue Type: Improvement
>> >  Components: general
>> >Affects Versions: 1.7
>> >Reporter: Nikolay Tikhonov
>> >Assignee: Vadim Opolski
>> >Priority: Trivial
>> >
>> > In current implementation Ignite Mesos Framework connects to MESOS
>> cluster via current user. Need to add ability to configure this parameters
>> via system env properties. Also need to add properties for mesos role.
>> > See org/apache/ignite/mesos/IgniteFramework.java:537
>>
>>
>>
>> --
>> This message was sent by Atlassian JIRA
>> (v6.3.15#6346)
>>
>
>

Re: [jira] [Commented] (IGNITE-4052) Add ability to set up users for MESOS

[Вадим Опольский]

Hello guys!

Nikolay, I looked at cassandra mesos integration, but cant find how to test
ability to configure mesos and user via system env properties. I will
continue to search in another projects which use Mesos.

Review please pull request - https://github.com/vadopolski/ignite/pull/2.
It contains new test and method for setting system environments.

Can you close issue https://issues.apache.org/jira/browse/IGNITE-4052 and
open new, about testing Mesos Integration.

Vadim Opolski

2017-04-03 12:13 GMT+03:00 Nikolay Tikhonov (JIRA) :

>
> [ https://issues.apache.org/jira/browse/IGNITE-4052?page=com.
> atlassian.jira.plugin.system.issuetabpanels:comment-tabpane
> l=15953167#comment-15953167 ]
>
> Nikolay Tikhonov commented on IGNITE-4052:
> --
>
> [~javaller],
> For example you can look at cassandra mesos integration
> https://github.com/mesosphere/dcos-cassandra-service/blob/5d
> 6d15e646f1df2fa10fa98d91993d3a6b4719c4/cassandra-commons/src
> /main/java/com/mesosphere/dcos/cassandra/common/config/ServiceConfig.java.
> List of application which use Mesos not secret and can be found on official
> mesos site http://mesos.apache.org/documentation/latest/frameworks. ;)
>
> I'm really don't understand how the libs can help you. Could you clarify
> it?
>
> > Add ability to set up users for MESOS
> > -
> >
> > Key: IGNITE-4052
> > URL: https://issues.apache.org/jira/browse/IGNITE-4052
> > Project: Ignite
> >  Issue Type: Improvement
> >  Components: general
> >Affects Versions: 1.7
> >Reporter: Nikolay Tikhonov
> >Assignee: Vadim Opolski
> >Priority: Trivial
> >
> > In current implementation Ignite Mesos Framework connects to MESOS
> cluster via current user. Need to add ability to configure this parameters
> via system env properties. Also need to add properties for mesos role.
> > See org/apache/ignite/mesos/IgniteFramework.java:537
>
>
>
> --
> This message was sent by Atlassian JIRA
> (v6.3.15#6346)
>