You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Sumit Mohanty <sm...@hortonworks.com> on 2015/04/22 03:25:47 UTC
Review Request 33426: Support skipping install operations on hosts
that are already sys-prepped
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33426/
-----------------------------------------------------------
Review request for Ambari, Alejandro Fernandez, Ivan Mitic, and Sid Wagle.
Bugs: AMBARI-10386
None
Description
-------
Support skipping install operations on hosts that are already sys-prepped
Diffs
-----
Diff: https://reviews.apache.org/r/33426/diff/
Testing
-------
Ran all unit tests and passed.
File Attachments
----------------
AMBARI-10386
https://reviews.apache.org/media/uploaded/files/2015/04/22/fa01dd3f-8295-48be-9035-a2260141540b__AMBARI-10386.patch
Thanks,
Sumit Mohanty
Re: Review Request 33426: Support skipping install operations on
hosts that are already sys-prepped
Posted by Sumit Mohanty <sm...@hortonworks.com>.
> On April 22, 2015, 2 a.m., Alejandro Fernandez wrote:
> > Please include the patch as a diff instead of an attachment.
reviews.a.o is still complaining and I tried to create an account on the other review service but that never send me a confirmation email.
- Sumit
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33426/#review81113
-----------------------------------------------------------
On April 22, 2015, 1:25 a.m., Sumit Mohanty wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/33426/
> -----------------------------------------------------------
>
> (Updated April 22, 2015, 1:25 a.m.)
>
>
> Review request for Ambari, Alejandro Fernandez, Ivan Mitic, and Sid Wagle.
>
>
> Bugs: AMBARI-10386
> None
>
>
> Description
> -------
>
> Support skipping install operations on hosts that are already sys-prepped
>
>
> Diffs
> -----
>
>
> Diff: https://reviews.apache.org/r/33426/diff/
>
>
> Testing
> -------
>
> Ran all unit tests and passed.
>
>
> File Attachments
> ----------------
>
> AMBARI-10386
> https://reviews.apache.org/media/uploaded/files/2015/04/22/fa01dd3f-8295-48be-9035-a2260141540b__AMBARI-10386.patch
>
>
> Thanks,
>
> Sumit Mohanty
>
>
Re: Review Request 33426: Support skipping install operations on
hosts that are already sys-prepped
Posted by Alejandro Fernandez <af...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33426/#review81113
-----------------------------------------------------------
Please include the patch as a diff instead of an attachment.
- Alejandro Fernandez
On April 22, 2015, 1:25 a.m., Sumit Mohanty wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/33426/
> -----------------------------------------------------------
>
> (Updated April 22, 2015, 1:25 a.m.)
>
>
> Review request for Ambari, Alejandro Fernandez, Ivan Mitic, and Sid Wagle.
>
>
> Bugs: AMBARI-10386
> None
>
>
> Description
> -------
>
> Support skipping install operations on hosts that are already sys-prepped
>
>
> Diffs
> -----
>
>
> Diff: https://reviews.apache.org/r/33426/diff/
>
>
> Testing
> -------
>
> Ran all unit tests and passed.
>
>
> File Attachments
> ----------------
>
> AMBARI-10386
> https://reviews.apache.org/media/uploaded/files/2015/04/22/fa01dd3f-8295-48be-9035-a2260141540b__AMBARI-10386.patch
>
>
> Thanks,
>
> Sumit Mohanty
>
>
Re: Review Request 33426: Support skipping install operations on
hosts that are already sys-prepped
Posted by Alejandro Fernandez <af...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33426/#review81186
-----------------------------------------------------------
Ship it!
Ship It!
- Alejandro Fernandez
On April 22, 2015, 1:25 a.m., Sumit Mohanty wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/33426/
> -----------------------------------------------------------
>
> (Updated April 22, 2015, 1:25 a.m.)
>
>
> Review request for Ambari, Alejandro Fernandez, Ivan Mitic, and Sid Wagle.
>
>
> Bugs: AMBARI-10386
> None
>
>
> Description
> -------
>
> Support skipping install operations on hosts that are already sys-prepped
>
>
> Diffs
> -----
>
>
> Diff: https://reviews.apache.org/r/33426/diff/
>
>
> Testing
> -------
>
> Ran all unit tests and passed.
>
>
> File Attachments
> ----------------
>
> AMBARI-10386
> https://reviews.apache.org/media/uploaded/files/2015/04/22/fa01dd3f-8295-48be-9035-a2260141540b__AMBARI-10386.patch
>
>
> Thanks,
>
> Sumit Mohanty
>
>
Re: Review Request 33426: Support skipping install operations on
hosts that are already sys-prepped
Posted by Ivan Mitic <iv...@microsoft.com>.
> On April 22, 2015, 1:37 a.m., Ivan Mitic wrote:
> > Ship It!
I have tested out this patch e2e in the sysprep scenario, +1
- Ivan
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33426/#review81111
-----------------------------------------------------------
On April 22, 2015, 1:25 a.m., Sumit Mohanty wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/33426/
> -----------------------------------------------------------
>
> (Updated April 22, 2015, 1:25 a.m.)
>
>
> Review request for Ambari, Alejandro Fernandez, Ivan Mitic, and Sid Wagle.
>
>
> Bugs: AMBARI-10386
> None
>
>
> Description
> -------
>
> Support skipping install operations on hosts that are already sys-prepped
>
>
> Diffs
> -----
>
>
> Diff: https://reviews.apache.org/r/33426/diff/
>
>
> Testing
> -------
>
> Ran all unit tests and passed.
>
>
> File Attachments
> ----------------
>
> AMBARI-10386
> https://reviews.apache.org/media/uploaded/files/2015/04/22/fa01dd3f-8295-48be-9035-a2260141540b__AMBARI-10386.patch
>
>
> Thanks,
>
> Sumit Mohanty
>
>
Re: Review Request 33426: Support skipping install operations on
hosts that are already sys-prepped
Posted by Ivan Mitic <iv...@microsoft.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33426/#review81111
-----------------------------------------------------------
Ship it!
Ship It!
- Ivan Mitic
On April 22, 2015, 1:25 a.m., Sumit Mohanty wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/33426/
> -----------------------------------------------------------
>
> (Updated April 22, 2015, 1:25 a.m.)
>
>
> Review request for Ambari, Alejandro Fernandez, Ivan Mitic, and Sid Wagle.
>
>
> Bugs: AMBARI-10386
> None
>
>
> Description
> -------
>
> Support skipping install operations on hosts that are already sys-prepped
>
>
> Diffs
> -----
>
>
> Diff: https://reviews.apache.org/r/33426/diff/
>
>
> Testing
> -------
>
> Ran all unit tests and passed.
>
>
> File Attachments
> ----------------
>
> AMBARI-10386
> https://reviews.apache.org/media/uploaded/files/2015/04/22/fa01dd3f-8295-48be-9035-a2260141540b__AMBARI-10386.patch
>
>
> Thanks,
>
> Sumit Mohanty
>
>