Re: Review Request 15050: Add Palo Alto Networks Firewall Integration

2013-11-06 Thread Sheng Yang
> On Nov. 6, 2013, 6:08 p.m., Sheng Yang wrote: > > Ship It! Committed to MASTER. Thank you Will! - Sheng --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15050/#review28280 -

Re: Review Request 15050: Add Palo Alto Networks Firewall Integration

2013-11-06 Thread Sheng Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15050/#review28280 --- Ship it! Ship It! - Sheng Yang On Nov. 6, 2013, 3:42 a.m., Will

Re: Review Request 15050: Add Palo Alto Networks Firewall Integration

2013-11-05 Thread Will Stevens
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15050/ --- (Updated Nov. 6, 2013, 3:42 a.m.) Review request for cloudstack and Sheng Yang.

Re: Review Request 15050: Add Palo Alto Networks Firewall Integration

2013-11-04 Thread Sheng Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15050/#review28146 --- Hi Will, I think you need to rebase the patch to the latest MASTER,

Re: Review Request 15050: Add Palo Alto Networks Firewall Integration

2013-11-04 Thread Sheng Yang
> On Nov. 4, 2013, 10:44 p.m., Sheng Yang wrote: > > Hi Will, > > > > I think you need to rebase the patch to the latest MASTER, the current > > patch cannot apply to MASTER now. > > > > Also, you can add description to the patch(e.g. the description you added > > in the review board) By app

Re: Review Request 15050: Add Palo Alto Networks Firewall Integration

2013-11-01 Thread Will Stevens
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15050/ --- (Updated Nov. 1, 2013, 9:10 p.m.) Review request for cloudstack and Sheng Yang.

Re: Review Request 15050: Add Palo Alto Networks Firewall Integration

2013-10-31 Thread Will Stevens
> On Oct. 30, 2013, 5:34 p.m., Sheng Yang wrote: > > Ship It! > > Sheng Yang wrote: > No shipping for now. > > Sorry, press the wrong one. I meant to ship the other patch. > > Will, since it's a big change, could you send out a notice to the > community about this merge? >

Re: Review Request 15050: Add Palo Alto Networks Firewall Integration

2013-10-31 Thread Will Stevens
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15050/ --- (Updated Oct. 31, 2013, 8:48 p.m.) Review request for cloudstack and Sheng Yang

Re: Review Request 15050: Add Palo Alto Networks Firewall Integration

2013-10-31 Thread Will Stevens
> On Oct. 30, 2013, 5:34 p.m., Sheng Yang wrote: > > Ship It! > > Sheng Yang wrote: > No shipping for now. > > Sorry, press the wrong one. I meant to ship the other patch. > > Will, since it's a big change, could you send out a notice to the > community about this merge? >

Re: Review Request 15050: Add Palo Alto Networks Firewall Integration

2013-10-31 Thread Will Stevens
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15050/ --- (Updated Oct. 31, 2013, 8:45 p.m.) Review request for cloudstack and Sheng Yang

Re: Review Request 15050: Add Palo Alto Networks Firewall Integration

2013-10-30 Thread Sheng Yang
> On Oct. 30, 2013, 5:34 p.m., Sheng Yang wrote: > > Ship It! > > Sheng Yang wrote: > No shipping for now. > > Sorry, press the wrong one. I meant to ship the other patch. > > Will, since it's a big change, could you send out a notice to the > community about this merge? >

Re: Review Request 15050: Add Palo Alto Networks Firewall Integration

2013-10-30 Thread Will Stevens
> On Oct. 30, 2013, 5:34 p.m., Sheng Yang wrote: > > Ship It! > > Sheng Yang wrote: > No shipping for now. > > Sorry, press the wrong one. I meant to ship the other patch. > > Will, since it's a big change, could you send out a notice to the > community about this merge? >

Re: Review Request 15050: Add Palo Alto Networks Firewall Integration

2013-10-30 Thread Sheng Yang
> On Oct. 30, 2013, 5:34 p.m., Sheng Yang wrote: > > Ship It! No shipping for now. Sorry, press the wrong one. I meant to ship the other patch. Will, since it's a big change, could you send out a notice to the community about this merge? Thanks! - Sheng --

Re: Review Request 15050: Add Palo Alto Networks Firewall Integration

2013-10-30 Thread Sheng Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15050/#review27794 --- Ship it! Ship It! - Sheng Yang On Oct. 29, 2013, 8:52 p.m., Will

Review Request 15050: Add Palo Alto Networks Firewall Integration

2013-10-29 Thread Will Stevens
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15050/ --- Review request for cloudstack and Sheng Yang. Bugs: CLOUDSTACK-1275 https:/