On May 30, 2011, at 7:41 PM, Diogo Pinheiro wrote:

> message WebsiteReportDetail {
> required string websiteURL = 2;
> required int32 statusCode = 3;
> optional double responseTime = 7;
> optional double bandwidth = 8;
> }
> 
> 
> Why the responseTime is a double ? Shouldn't be an integer ? I think we will 
> not need anything more precise than miliseconds.
> And about the bandwidth ? Are you thinking sending it in Kbs/s or B/s ?


responseTime should be int64, since it will hold an epoch and bandwidth is ok 
to be int32 if we use B/s.

> 
> 2011/5/30 Zubair Nabi <[email protected]>
> You're correct about the ICMReport message. It's perfectly fine.
> 
> 
> And +1 for the hostname and IP fields.
> 
> 
> 2011/5/30 Adriano Monteiro Marques <[email protected]>
> Hi Zubair,
> 
> On May 30, 2011, at 11:04 AM, Zubair Nabi wrote:
> 
>> Guys,
>> 
>> 2 things:
>> 
>> 1. ICMReport message is not being used anywhere. Because we have two 
>> separate SendWebsiteReport and SendServiceReport messages.
>> Any thoughts on this this?
> 
> ICMReport is the header for WebsiteReport and ServiceReport messages and it 
> holds what these two reports have in common. Not sure I got the point, but it 
> seems to be used. Let me know if I'm wrong...
> 
>> 2. Secondly, I'm going to add two messages for Website and Service Suggestion
>> message WebsiteSuggestion {
>> required RequestHeader header = 1;
>> required string websiteURL = 2;
>> optional string emailAddress = 3;
>> }
>> 
>> message ServiceSuggestion {
>> required RequestHeader header = 1;
>> required string serviceName = 2;
>> optional string emailAddress = 3;
>> }
> 
> Service should be accompanied by a hostname and ip at least, I think, since 
> some services doesn't use the default port and we shouldn't try to figure 
> hostnames to test the service either.
> 
>> 
>> message TestSuggestionResponse {
>> required Response Header header = 1;
>> }
> 
> Other than my comments, everything seems to be good!
> 
>> 
>> Do you guys think these fields are enough?
>> 
>> 2011/5/29 Zubair Nabi <[email protected]>
>> Ah. Good catch!
>> 
>> So I'm going to add  
>> required string agentIP = 1;
>> required int32 agenPort = 2;
>> to message AgentData
>> 
>> and 
>> required string cipheredPublicKey = 6
>> to message RegisterAgentResponse.
>> 
>> 2011/5/29 Luís A. Bastião Silva <[email protected]>
>> Yes.
>> 
>> Also, in the RegisterAgentResponse it misses the public key ciphered with 
>> Aggregator private key.
>> 
>> Can you handle that Zubair?
>> 
>> 
>> 2011/5/28 Zubair Nabi <[email protected]>
>> Guys,
>> One more thing. Shouldn't the AgentData message also have an IP/Port no. 
>> field? I mean how else will peers connect?
>> 
>> 
>> 2011/5/28 Zubair Nabi <[email protected]>
>> Alan, I think that messageType field is redundant. One can use the Protobuf 
>> API to get the message type.
>> 
>> For example, for a variable 'trace' of data type 'Trace', you can get the 
>> message name through:
>> String messageType = trace.getDescriptorForType().getFullName(); // In Java. 
>> Python has a similar API
>> 
>> So I'm going to delete the messageType fields from the Request and Response 
>> headers.
>> 
>> Agree?
>> 
>> 2011/5/28 Zhongjie Wang <[email protected]>
>> Hi, Luis. 
>> I use the command you told me and it works. :)
>> But when I use the TortoiseGit, it doesn't work. 
>> Thanks!
>> 
>> 2011/5/29 Luís A. Bastião Silva <[email protected]>
>> 
>> Can you clone?
>> 
>> Which the command are you executing?
>> 
>> git submodule add http://git.umitproject.org/umit-common.git
>> 
>> 
>> 2011/5/28 Zhongjie Wang <[email protected]>
>> Hi Luis, higwidgets is ok. still not able to add the umit-common submodule...
>> 
>> 2011/5/28 Luís A. Bastião Silva <[email protected]>
>> 
>> Try again.
>> 
>> 
>> On Sat, May 28, 2011 at 3:48 AM, Zhongjie Wang <[email protected]> wrote:
>> This is the message I've got:
>> "
>> git.exe submodule add  -- "[email protected]:/higwidgets.git"  ""
>> 
>> Cloning into higwidgets...
>> ERROR:gitosis.serve.main:Repository read access denied
>> fatal: The remote end hung up unexpectedly
>> Clone of '[email protected]:/higwidgets.git' into submodule path 
>> 'higwidgets' failed
>> "
>> almost the same as cloning umit-common. 
>> 
>> 
>> On Sat, May 28, 2011 at 10:43 AM, Adriano Monteiro Marques 
>> <[email protected]> wrote:
>> Just fixed for higwidgets, but umit-common is readable/writable by everyone 
>> already. What is the error you're getting?
>> 
>> On May 27, 2011, at 10:38 PM, Zhongjie Wang wrote:
>> 
>>> Hi, I'm also having issues with downloading the higwidgets and umit-common. 
>>> Seems no reading permissions. :(
>>> 
>>> On Sat, May 28, 2011 at 3:48 AM, Zubair Nabi <[email protected]> 
>>> wrote:
>>> Okay guys, 
>>> I'm done with the .proto. 
>>> I've made a number of changes. Please have a look at the changelog to 
>>> follow them.
>>> I think these messages are now in a shape that we can start using them in 
>>> our code. 
>>> Just in case, please go through all the messages in detail (following the 
>>> flow of request/response) and if you think that stuff needs to be 
>>> added/deleted/modified then use this thread to discuss them. Also, bear in 
>>> mind that these messages should work for both the aggregator communication 
>>> as well as the p2p communication. 
>>> 
>>> 
>>> On Fri, May 27, 2011 at 11:30 PM, Adriano Monteiro Marques 
>>> <[email protected]> wrote:
>>> Hi Zubair,
>>> 
>>> Awesome!! Keep up with the good work. Let me know if you need anything else.
>>> 
>>> 
>>> Cheers!
>>> 
>>> On May 27, 2011, at 3:27 PM, Zubair Nabi wrote:
>>> 
>>>> Hi Adriano,
>>>> 
>>>> No. I was having issues adding the common repository to the mobile 
>>>> repository as submodule but Luis and Rodolfo helped me solve that problem. 
>>>> A couple of messages were missing from the .proto. I added them.
>>>> Now I'll add the optional messageType field for p2p messages.
>>>> 
>>>> On Fri, May 27, 2011 at 10:55 PM, Adriano Monteiro Marques 
>>>> <[email protected]> wrote:
>>>> Hi Zubair,
>>>> 
>>>> You have write perms there already... are you having any issues with that 
>>>> repo?
>>>> 
>>>> On May 27, 2011, at 1:04 PM, Zubair Nabi wrote:
>>>> 
>>>>> Sounds good. Let's keep it a string. The type would be the name.
>>>>> 
>>>>> Let's keep it optional. It will be used in p2p communication but not in 
>>>>> aggregator communication.
>>>>> So this is the extra field that I propose:
>>>>> 
>>>>> optional string messageType = x;
>>>>> 
>>>>> Does everyone agree?
>>>>> 
>>>>> Adriano - Do we have write access to the common ICM repository?
>>>>> 
>>>>> On Fri, May 27, 2011 at 8:59 PM, Zhongjie Wang <[email protected]> wrote:
>>>>> Hi Zubair,
>>>>>       That's a good way.  I agree. :)
>>>>> 
>>>>> 
>>>>> On Fri, May 27, 2011 at 11:49 PM, Zubair Nabi <[email protected]> 
>>>>> wrote:
>>>>> That's a very good point. In case of p2p agents we will only be sending 
>>>>> messages using a standard send function. So, there is no way to tell the 
>>>>> message type. We should add an int32 for message type considering that we 
>>>>> have a standard number of messages.
>>>>> If int32 messageType  == 1 then that could an authentication message and 
>>>>> so on. What do you say?
>>>>> 
>>>>> 
>>>>> On Fri, May 27, 2011 at 8:44 PM, Zhongjie Wang <[email protected]> wrote:
>>>>> Hi Adriano, 
>>>>> 
>>>>> Sorry, I didn't see the email of that doc. Then it's perfect. :)
>>>>> There's one more question, how to detect the message type when I receive 
>>>>> a message?
>>>>> Is there any field to indicate the type?
>>>>> 
>>>>> 
>>>>> On Fri, May 27, 2011 at 11:34 PM, Adriano Monteiro Marques 
>>>>> <[email protected]> wrote:
>>>>> Hi Alan,
>>>>> 
>>>>> On May 27, 2011, at 12:30 PM, Zhongjie Wang wrote:
>>>>> 
>>>>> > Hi Zubair, Diogo:
>>>>> >     I hope you guys could join and figure out the detailed message 
>>>>> > format for the communication interfaces. :)
>>>>> > In the spec, we have decided to use RESTful webservice and RPC call for 
>>>>> > the communications. Now the
>>>>> > form of RPC call is somewhat obscure, we need to make a clearly defined 
>>>>> > message format. Do you think we
>>>>> > should still use protobuf, or json/xml, or customized binary/text 
>>>>> > format? This is important.
>>>>> 
>>>>> We've already decided on using protobuf, right? Event the ones we defined 
>>>>> today and shared with you in that google doc.
>>>>> 
>>>>> >     At first, there should be an authentication process after 
>>>>> > connected. Then we use the negotiated
>>>>> > symmetric key to encrypt the following messages. The messages will be 
>>>>> > in pair, like Request/Response.
>>>>> > For a request, there should be a RPC function name and then the 
>>>>> > parameters, finally maybe end with a checksum.
>>>>> > And for a response, it will indicate which request it response to, and 
>>>>> > then the result.
>>>>> >     If we use protobuf, then we need to put the function name out of 
>>>>> > the message. So after the agent read the name,
>>>>> > it will generate a proper class for the message.
>>>>> >
>>>>> > How do you think which one we should choose?
>>>>> >
>>>>> > Regards
>>>>> >
>>>>> > --
>>>>> > Zhongjie Wang
>>>>> > Master Candidate
>>>>> > Computer System Architecture
>>>>> > Peking University, China
>>>>> > ------------------------------------------------------------------------------
>>>>> > vRanger cuts backup time in half-while increasing security.
>>>>> > With the market-leading solution for virtual backup and recovery,
>>>>> > you get blazing-fast, flexible, and affordable data protection.
>>>>> > Download your free trial now.
>>>>> > http://p.sf.net/sfu/quest-d2dcopy1_______________________________________________
>>>>> > Umit-devel mailing list
>>>>> > [email protected]
>>>>> > https://lists.sourceforge.net/lists/listinfo/umit-devel
>>>>> 
>>>>> ---
>>>>> Adriano Monteiro Marques
>>>>> 
>>>>> http://www.thoughtspad.com
>>>>> http://www.umitproject.org
>>>>> http://blog.umitproject.org
>>>>> http://www.pythonbenelux.org
>>>>> 
>>>>> "Don't stay in bed, unless you can make money in bed." - George Burns
>>>>> 
>>>>> 
>>>>> 
>>>>> 
>>>>> -- 
>>>>> Zhongjie Wang
>>>>> Master Candidate
>>>>> Computer System Architecture
>>>>> Peking University, China
>>>>> 
>>>>> ------------------------------------------------------------------------------
>>>>> vRanger cuts backup time in half-while increasing security.
>>>>> With the market-leading solution for virtual backup and recovery,
>>>>> you get blazing-fast, flexible, and affordable data protection.
>>>>> Download your free trial now.
>>>>> http://p.sf.net/sfu/quest-d2dcopy1
>>>>> _______________________________________________
>>>>> Umit-devel mailing list
>>>>> [email protected]
>>>>> https://lists.sourceforge.net/lists/listinfo/umit-devel
>>>>> 
>>>>> 
>>>>> 
>>>>> 
>>>>> -- 
>>>>> Best,
>>>>> __
>>>>> Zubair
>>>>> 
>>>>> 
>>>>> 
>>>>> -- 
>>>>> Zhongjie Wang
>>>>> Master Candidate
>>>>> Computer System Architecture
>>>>> Peking University, China
>>>>> 
>>>>> 
>>>>> 
>>>>> -- 
>>>>> Best,
>>>>> __
>>>>> Zubair
>>>> 
>>>> ---
>>>> Adriano Monteiro Marques
>>>> 
>>>> http://www.thoughtspad.com
>>>> http://www.umitproject.org
>>>> http://blog.umitproject.org
>>>> http://www.pythonbenelux.org
>>>> 
>>>> "Don't stay in bed, unless you can make money in bed." - George Burns
>>>> 
>>>> 
>>>> 
>>>> 
>>>> -- 
>>>> Best,
>>>> __
>>>> Zubair
>>> 
>>> ---
>>> Adriano Monteiro Marques
>>> 
>>> http://www.thoughtspad.com
>>> http://www.umitproject.org
>>> http://blog.umitproject.org
>>> http://www.pythonbenelux.org
>>> 
>>> "Don't stay in bed, unless you can make money in bed." - George Burns
>>> 
>>> 
>>> 
>>> 
>>> -- 
>>> Best,
>>> __
>>> Zubair
>>> 
>>> 
>>> 
>>> -- 
>>> Zhongjie Wang
>>> Master Candidate
>>> Computer System Architecture
>>> Peking University, China
>> 
>> ---
>> Adriano Monteiro Marques
>> 
>> http://www.thoughtspad.com
>> http://www.umitproject.org
>> http://blog.umitproject.org
>> http://www.pythonbenelux.org
>> 
>> "Don't stay in bed, unless you can make money in bed." - George Burns
>> 
>> 
>> 
>> 
>> -- 
>> Zhongjie Wang
>> Master Candidate
>> Computer System Architecture
>> Peking University, China
>> 
>> ------------------------------------------------------------------------------
>> vRanger cuts backup time in half-while increasing security.
>> With the market-leading solution for virtual backup and recovery,
>> you get blazing-fast, flexible, and affordable data protection.
>> Download your free trial now.
>> http://p.sf.net/sfu/quest-d2dcopy1
>> _______________________________________________
>> Umit-devel mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/umit-devel
>> 
>> 
>> 
>> 
>> -- 
>> Luís A. Bastião Silva
>> Umit Project Developer
>> Skype: koplabs
>> http://www.umitproject.org
>> http://www.bastiao.org
>> 
>> 
>> 
>> 
>> 
>> -- 
>> Zhongjie Wang
>> Master Candidate
>> Computer System Architecture
>> Peking University, China
>> 
>> 
>> 
>> -- 
>> Luís A. Bastião Silva
>> Umit Project Developer
>> Skype: koplabs
>> http://www.umitproject.org
>> http://www.bastiao.org
>> 
>> 
>> 
>> 
>> 
>> -- 
>> Zhongjie Wang
>> Master Candidate
>> Computer System Architecture
>> Peking University, China
>> 
>> ------------------------------------------------------------------------------
>> vRanger cuts backup time in half-while increasing security.
>> With the market-leading solution for virtual backup and recovery,
>> you get blazing-fast, flexible, and affordable data protection.
>> Download your free trial now.
>> http://p.sf.net/sfu/quest-d2dcopy1
>> _______________________________________________
>> Umit-devel mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/umit-devel
>> 
>> 
>> 
>> 
>> -- 
>> Best,
>> __
>> Zubair
>> 
>> 
>> 
>> -- 
>> Best,
>> __
>> Zubair
>> 
>> 
>> 
>> -- 
>> Luís A. Bastião Silva
>> Umit Project Developer
>> Skype: koplabs
>> http://www.umitproject.org
>> http://www.bastiao.org
>> 
>> 
>> 
>> 
>> 
>> -- 
>> Best,
>> __
>> Zubair
>> 
>> 
>> 
>> -- 
>> Best,
>> __
>> Zubair
>> ------------------------------------------------------------------------------
>> vRanger cuts backup time in half-while increasing security.
>> With the market-leading solution for virtual backup and recovery, 
>> you get blazing-fast, flexible, and affordable data protection.
>> Download your free trial now. 
>> http://p.sf.net/sfu/quest-d2dcopy1_______________________________________________
>> Umit-devel mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/umit-devel
> 
> 
> ---
> Adriano Monteiro Marques
> 
> http://www.thoughtspad.com
> http://www.umitproject.org
> http://blog.umitproject.org
> http://www.pythonbenelux.org
> 
> "Don't stay in bed, unless you can make money in bed." - George Burns
> 
> 
> 
> 
> -- 
> Best,
> __
> Zubair
> 
> ------------------------------------------------------------------------------
> vRanger cuts backup time in half-while increasing security.
> With the market-leading solution for virtual backup and recovery,
> you get blazing-fast, flexible, and affordable data protection.
> Download your free trial now.
> http://p.sf.net/sfu/quest-d2dcopy1
> _______________________________________________
> Umit-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/umit-devel
> 
> 
> 
> 
> -- 
> Cumprimentos
> Diogo Pinheiro
> 

---
Adriano Monteiro Marques

http://www.thoughtspad.com
http://www.umitproject.org
http://blog.umitproject.org
http://www.pythonbenelux.org

"Don't stay in bed, unless you can make money in bed." - George Burns

------------------------------------------------------------------------------
Simplify data backup and recovery for your virtual environment with vRanger. 
Installation's a snap, and flexible recovery options mean your data is safe,
secure and there when you need it. Data protection magic?
Nope - It's vRanger. Get your free trial download today. 
http://p.sf.net/sfu/quest-sfdev2dev
_______________________________________________
Umit-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/umit-devel

Reply via email to