[GitHub] cloudstack issue #1849: CLOUDSTACK-9690: Scale CentOS7 VM fails with error

2017-04-04 Thread jayantpatil1234
Github user jayantpatil1234 commented on the issue:

https://github.com/apache/cloudstack/pull/1849
  
 @sudhansu7 
There is one file "Upgrade4910to4920.java", which does not have any 
changes, still that get added in PR. If possible could you remove that file? 

Otherwise code looks good. LGTM for code


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #892: CLOUDSTACK-8910: The reserved_capacity field increase...

2017-04-03 Thread jayantpatil1234
Github user jayantpatil1234 commented on the issue:

https://github.com/apache/cloudstack/pull/892
  
I have tested this and LGTM for test.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1734: CLOUDSTACK-9567 Difference in the api call outputs f...

2017-03-27 Thread jayantpatil1234
Github user jayantpatil1234 commented on the issue:

https://github.com/apache/cloudstack/pull/1734
  
I have tested this and **LGTM** for test. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1987: CLOUDSTACK-9814 : Unable to edit a Sub domain, which...

2017-03-05 Thread jayantpatil1234
Github user jayantpatil1234 commented on the issue:

https://github.com/apache/cloudstack/pull/1987
  
I have tested this and **LGTM** for test.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1940: CLOUDSTACK-9781:ACS records ID in events tables inst...

2017-03-01 Thread jayantpatil1234
Github user jayantpatil1234 commented on the issue:

https://github.com/apache/cloudstack/pull/1940
  
@blueorangutan test cases which are failing, not related to my code 
changes. Could you restart test suites once again. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1940: CLOUDSTACK-9781:ACS records ID in events tables inst...

2017-02-27 Thread jayantpatil1234
Github user jayantpatil1234 commented on the issue:

https://github.com/apache/cloudstack/pull/1940
  
@borisstoyanov I have done required fix and now all test cases are passing, 
please have a look. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1940: CLOUDSTACK-9781:ACS records ID in events tables inst...

2017-02-16 Thread jayantpatil1234
Github user jayantpatil1234 commented on the issue:

https://github.com/apache/cloudstack/pull/1940
  
Please see screenshots.

![after](https://cloud.githubusercontent.com/assets/12637540/23013214/bd8c7da2-f44f-11e6-9ec7-1f0105196970.PNG)

![before](https://cloud.githubusercontent.com/assets/12637540/23013215/bd8dd6b6-f44f-11e6-92bd-f097ad1213b8.PNG)


 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request #1940: CLOUDSTACK-9781:ACS records ID in events tabl...

2017-02-14 Thread jayantpatil1234
GitHub user jayantpatil1234 opened a pull request:

https://github.com/apache/cloudstack/pull/1940

CLOUDSTACK-9781:ACS records ID in events tables instead of UUID.

ISSUE
=
Wrong presentation of volume id in ASC events.
While creating a snapshot, only volume ID is mentioned in the events. For 
example, “Scheduled async job for creating snapshot for volume Id:270". On 
looking into the notification, user is not able to identify the volume. So 
modified event description with UUID.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/Accelerite/cloudstack CS-48313

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1940.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1940


commit 7f216e8960e466acbd610fd90d44583bb8b6c15e
Author: Jayant Patil <jayant.pa...@accelerite.com>
Date:   2017-02-14T08:51:30Z

CLOUDSTACK-9781:ACS records ID in events tables instead of UUID.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request #1926: CLOUDSTACK-9768: Time displayed for events in...

2017-02-01 Thread jayantpatil1234
GitHub user jayantpatil1234 opened a pull request:

https://github.com/apache/cloudstack/pull/1926

CLOUDSTACK-9768: Time displayed for events in UI is incorrect

Time displayed for events in UI is incorrect. Let's say, when we login 
using Japanese language the time displayed in the events is GMT instead of JST. 
However with English language the time is JST, as expected.
Example: 
Time is displayed in the event is 10:40, if you are logged in using English 
language. 
Whereas, time in the event shows 19:40 If you login with Japanese language.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/jayantpatil1234/cloudstack CS-50157

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1926.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1926


commit dcf7b7dbb591128258408bac4b6628bff0320fa6
Author: Jayant Patil <jayant.pa...@accelerite.com>
Date:   2017-02-02T04:28:22Z

CLOUDSTACK-9768: Time displayed for events in UI is incorrect




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---