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-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-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

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 applying, I meant

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

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

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

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
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
--- 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.,

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 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! 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?

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