You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Sanjay Tripathi <sa...@citrix.com> on 2013/08/01 13:05:44 UTC

Review Request 13190: CLOUDSTACK-4010: Change required in the UI to pass startdate/enddate parameter rather than olderthan parameter in the archive/delete alerts/events API.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13190/
-----------------------------------------------------------

Review request for cloudstack, Brian Federle and Jessica Wang.


Bugs: CLOUDSTACK-4010


Repository: cloudstack-git


Description
-------

 CLOUDSTACK-4010: Change required in the UI to pass startdate/enddate
 parameter rather than olderthan parameter in the archive/delete alerts/events API.


Diffs
-----

  ui/scripts/events.js 873998e 

Diff: https://reviews.apache.org/r/13190/diff/


Testing
-------

Veirified the fix and functionality locally on cloudstack setup.


Thanks,

Sanjay Tripathi


Re: Review Request 13190: CLOUDSTACK-4010: Change required in the UI to pass startdate/enddate parameter rather than olderthan parameter in the archive/delete alerts/events API.

Posted by Brian Federle <br...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13190/#review24538
-----------------------------------------------------------

Ship it!


Ship It!

- Brian Federle


On Aug. 1, 2013, 11:05 a.m., Sanjay Tripathi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13190/
> -----------------------------------------------------------
> 
> (Updated Aug. 1, 2013, 11:05 a.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Jessica Wang.
> 
> 
> Bugs: CLOUDSTACK-4010
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
>  CLOUDSTACK-4010: Change required in the UI to pass startdate/enddate
>  parameter rather than olderthan parameter in the archive/delete alerts/events API.
> 
> 
> Diffs
> -----
> 
>   ui/scripts/events.js 873998e 
> 
> Diff: https://reviews.apache.org/r/13190/diff/
> 
> 
> Testing
> -------
> 
> Veirified the fix and functionality locally on cloudstack setup.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>


Re: Review Request 13190: CLOUDSTACK-4010: Change required in the UI to pass startdate/enddate parameter rather than olderthan parameter in the archive/delete alerts/events API.

Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13190/#review24540
-----------------------------------------------------------


Commit 80ae4830ae63ab73a06ab033bc3feb3dd701cbb2 in branch refs/heads/4.2 from Brian Federle
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=80ae483 ]

CLOUDSTACK-4010: Events/alerts: Pass start date / end date

Original submitter: Sanjay Tripathi <sa...@citrix.com>
Reviewed by: Brian Federle <br...@citrix.com>


- ASF Subversion and Git Services


On Aug. 1, 2013, 11:05 a.m., Sanjay Tripathi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13190/
> -----------------------------------------------------------
> 
> (Updated Aug. 1, 2013, 11:05 a.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Jessica Wang.
> 
> 
> Bugs: CLOUDSTACK-4010
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
>  CLOUDSTACK-4010: Change required in the UI to pass startdate/enddate
>  parameter rather than olderthan parameter in the archive/delete alerts/events API.
> 
> 
> Diffs
> -----
> 
>   ui/scripts/events.js 873998e 
> 
> Diff: https://reviews.apache.org/r/13190/diff/
> 
> 
> Testing
> -------
> 
> Veirified the fix and functionality locally on cloudstack setup.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>


Re: Review Request 13190: CLOUDSTACK-4010: Change required in the UI to pass startdate/enddate parameter rather than olderthan parameter in the archive/delete alerts/events API.

Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13190/#review24541
-----------------------------------------------------------


Commit 977b0eca973b721fe158768f3f3c5515b9d2aab7 in branch refs/heads/master from Brian Federle
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=977b0ec ]

CLOUDSTACK-4010: Events/alerts: Pass start date / end date

Original submitter: Sanjay Tripathi <sa...@citrix.com>
Reviewed by: Brian Federle <br...@citrix.com>


- ASF Subversion and Git Services


On Aug. 1, 2013, 11:05 a.m., Sanjay Tripathi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13190/
> -----------------------------------------------------------
> 
> (Updated Aug. 1, 2013, 11:05 a.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Jessica Wang.
> 
> 
> Bugs: CLOUDSTACK-4010
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
>  CLOUDSTACK-4010: Change required in the UI to pass startdate/enddate
>  parameter rather than olderthan parameter in the archive/delete alerts/events API.
> 
> 
> Diffs
> -----
> 
>   ui/scripts/events.js 873998e 
> 
> Diff: https://reviews.apache.org/r/13190/diff/
> 
> 
> Testing
> -------
> 
> Veirified the fix and functionality locally on cloudstack setup.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>


Re: Review Request 13190: CLOUDSTACK-4010: Change required in the UI to pass startdate/enddate parameter rather than olderthan parameter in the archive/delete alerts/events API.

Posted by "Jenkins Cloudstack.org" <hu...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13190/#review24434
-----------------------------------------------------------


Review 13190 failed the build test : FAILURE
The url of build cloudstack-master-with-patch #100 is : http://jenkins.cloudstack.org/job/cloudstack-master-with-patch/100/

- Jenkins Cloudstack.org


On Aug. 1, 2013, 11:05 a.m., Sanjay Tripathi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13190/
> -----------------------------------------------------------
> 
> (Updated Aug. 1, 2013, 11:05 a.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Jessica Wang.
> 
> 
> Bugs: CLOUDSTACK-4010
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
>  CLOUDSTACK-4010: Change required in the UI to pass startdate/enddate
>  parameter rather than olderthan parameter in the archive/delete alerts/events API.
> 
> 
> Diffs
> -----
> 
>   ui/scripts/events.js 873998e 
> 
> Diff: https://reviews.apache.org/r/13190/diff/
> 
> 
> Testing
> -------
> 
> Veirified the fix and functionality locally on cloudstack setup.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>