Reg CLOUDSTACK-8616

2018-02-13 Thread Jayakarteek Vasana
Hi,

For CLOUDSTACK-8616(https://issues.apache.org/jira/browse/CLOUDSTACK-8616) , 
changes were  made in check_heartbeat.sh.tmpl

Logic was changed on how the diff  time is verified, with the changes the IF 
condition ( last time will always be smaller than this time) will fail every 
time and the ELSE log will fill the disk space.


So can someone check and confirm the logic for the change and whether the logs 
are needed in the else part

https://github.com/apache/cloudstack/pull/587/files#diff-002a85328505af2975427d354a11dbff

--Jayakarteek

DISCLAIMER
==
This e-mail may contain privileged and confidential information which is the 
property of Accelerite, a Persistent Systems business. It is intended only for 
the use of the individual or entity to which it is addressed. If you are not 
the intended recipient, you are not authorized to read, retain, copy, print, 
distribute or use this message. If you have received this communication in 
error, please notify the sender and delete all copies of this message. 
Accelerite, a Persistent Systems business does not accept any liability for 
virus infected mails.


[GitHub] cloudstack issue #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-04-13 Thread jayakarteek
Github user jayakarteek commented on the issue:

https://github.com/apache/cloudstack/pull/1918
  
tag:mergeready


---
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 #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-04-05 Thread jayakarteek
Github user jayakarteek commented on the issue:

https://github.com/apache/cloudstack/pull/1918
  
@nvazquez  @rafaelweingartner 
Done.


---
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 #1849: CLOUDSTACK-9690: Scale CentOS7 VM fails with error

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

https://github.com/apache/cloudstack/pull/1849
  
@sudhansu7 

Tested the fix, LGTM.
I am able to change from small computer offering to medium computer 
offering for centos-7.


---
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 #899: BUG-ID:CLOUDSTACK-8921 snapshot_store_ref table shoul...

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

https://github.com/apache/cloudstack/pull/899
  
@yvsubhash 

Tested the changes, tests LGTM.

below are the results
root@MgmtServerNitesh:/mnt/jayant/snapshots/2/120# ls -l
total 1142048
-rw-r-+ 1 root root 1168306688 Apr  3 18:13 
9c340447-5ff9-49f3-875e-1dbbcf63ccf9.vhd

mysql> select * from snapshot_store_ref;

++--+-+-+--+++-+---++--+---+--+-+-+---+
| id | store_id | snapshot_id | created | last_updated | job_id 
| store_role | size| physical_size | parent_snapshot_id | install_path  
   | state | update_count | ref_cnt 
| updated | volume_id |

++--+-+-+--+++-+---++--+---+--+-+-+---+
 |   0 | 2017-04-03 12:53:16 |   120 |
| 12 |6 |   6 | 2017-04-03 12:53:17 | NULL | NULL   
| Image  |  5368709120 |1168306688 |  0 | 
snapshots/2/120/9c340447-5ff9-49f3-875e-1dbbcf63ccf9.vhd | Ready |  
  2 |   0 





---
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 #921: CLOUDSTACK-8944 Template download possible from new n...

2017-03-31 Thread jayakarteek
Github user jayakarteek commented on the issue:

https://github.com/apache/cloudstack/pull/921
  
unable to reproduce in ACS environment


---
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 #883: CLOUDSTACK-8906: /var/log/cloud/ doesn't get logrotat...

2017-03-31 Thread jayakarteek
Github user jayakarteek commented on the issue:

https://github.com/apache/cloudstack/pull/883
  
Tested the PUll Request.
Logs are rotated as expected in /var/log/cloud folder


---
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 #1756: CLOUDSTACK-9585 UI doesn't give an option to select ...

2017-03-30 Thread jayakarteek
Github user jayakarteek commented on the issue:

https://github.com/apache/cloudstack/pull/1756
  
Tested the scenario, xen tools version is visible 


![pull-1756](https://cloud.githubusercontent.com/assets/23551298/24537378/4d032de8-15ff-11e7-9a00-febdd6d6af0a.jpg)




---
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 #883: CLOUDSTACK-8906: /var/log/cloud/ doesn't get logrotat...

2017-03-30 Thread jayakarteek
Github user jayakarteek commented on the issue:

https://github.com/apache/cloudstack/pull/883
  
Test results;
when I executed the setupxenserver.sh file , following is the output
iptables: Bad rule (does a matching rule exist in that chain?).
mv: invalid option -- n
Try `mv --help' for more information.
success

mv -n option is provided in line 52



---
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 #1844: CLOUDSTACK-9668 : disksizeallocated of PrimaryStorag...

2017-03-28 Thread jayakarteek
Github user jayakarteek commented on the issue:

https://github.com/apache/cloudstack/pull/1844
  
Tested the scenario.
results are as below
select * from storage_pool;
# id, name, uuid, pool_type, port, data_center_id, pod_id, cluster_id, 
used_bytes, capacity_bytes, host_address, user_info, path, created, removed, 
update_time, status, storage_provider_name, scope, hypervisor, managed, 
capacity_iops
'11', 'pr_2', '80a753bb-6fad-3938-b657-ae62f6119804', 'NetworkFilesystem', 
'2049', '1', NULL, NULL, '4140196372480', '590228480', '10.147.28.7', NULL, 
'/export/home/jayakarteek/PR', '2017-03-28 05:54:28', NULL, NULL, 'Up', 
'DefaultPrimary', 'ZONE', 'VMware', '0', NULL

select * from  op_host_capacity;   **After attaching disk** 
id, host_id, data_center_id, pod_id, cluster_id, used_capacity, 
reserved_capacity, total_capacity, capacity_type, capacity_state, update_time, 
created
'11', '11', '1', NULL, NULL, '18253611008', '0', '1180456960', '3', 
'Enabled', '2017-03-28 05:57:58', '2017-03-28 05:57:58'
select * from  op_host_capacity;   **After detaching and deleting the disk**
# id, host_id, data_center_id, pod_id, cluster_id, used_capacity, 
reserved_capacity, total_capacity, capacity_type, capacity_state, update_time, 
created
'11', '11', '1', NULL, NULL, '0', '0', '1180456960', '3', 'Enabled', 
'2017-03-28 06:22:58', '2017-03-28 05:57:58'




---
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 #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-03-22 Thread jayakarteek
Github user jayakarteek commented on the issue:

https://github.com/apache/cloudstack/pull/1918
  
@rafaelweingartner , @nvazquez 

Review the code changes.


---
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 #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-03-15 Thread jayakarteek
Github user jayakarteek commented on the issue:

https://github.com/apache/cloudstack/pull/1918
  
@rafaelweingartner  
Tried using both cpuentitlement cpu reservedCapacity but retrieved values 
from vmware are zero.

While going through docs , I saw **summary.runtime.maxCpuUsage** property 
which is max cpu that is allocated per vm, this looks accurate.

If I enable CPU cap in ACS UI , the cpu allocated returned by this property 
matches with the service offering.
If CPU cap is not enabled then cpu allocated value is determined by VMware.


So I am going with the above property using which I will calculate the CPU 
% and populate the POJO.






---
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 #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-03-14 Thread jayakarteek
Github user jayakarteek commented on the issue:

https://github.com/apache/cloudstack/pull/1918
  
@rafaelweingartner 
we can retrieve the CPU speed of the host but the utilisation will have 
mismatch.
create VM with  Medium Instance having 1000 Mhz , and VM is utilising 
200Mhz of CPU then utilisation is 20%.
But if we consider host CPU capacity with  2400Mhz  then utilisation in 
8.3%.

@nvazquez 

I checked CPUCounter API doc, even in this case values will have some 
missmatch.
as per the below document with the usage formula they are using.

https://www.vmware.com/support/developer/vc-sdk/visdk41pubs/ApiReference/cpu_counters.html




 



---
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 #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-03-13 Thread jayakarteek
Github user jayakarteek commented on the issue:

https://github.com/apache/cloudstack/pull/1918
  
@rafaelweingartner 

The CPU.Reservation value by **default** is 0, Also the value is set in 
VMware not from the ACS UI.
this gives ambiguity about the CPU allocated and Utilised.

Use case 1 :
CPU.Reservation is 0 by default, In this case we cannot calculate % as the 
value is always zero.

Use case 2:
CPU.Reservation is set by VMware admin say 500Mhz.
But while creating vm-instance  using ACS UI , admin selected service 
offering as Medium Instance which is 1Ghz CPU. so CPU utilisation will be wrong.
In case say cloud environment has 30 instances , then Vmware admin has to 
manually set the CPU.Reservation value for each of the VM Instance which  is 
cumbersome. 






---
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 #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-03-09 Thread jayakarteek
Github user jayakarteek commented on the issue:

https://github.com/apache/cloudstack/pull/1918
  
@nvazquez , @borisstoyanov , @rafaelweingartner 

Attached screen shots. where I set CPU Reservation as 500 Mhz using 
VSphere-Client , and same is obtained by the  API call.


![acs_vmstatistics](https://cloud.githubusercontent.com/assets/23551298/23786214/c4ea9b12-0591-11e7-8c41-1a4b17fdcf7e.png)

![acs_vmstatistics2](https://cloud.githubusercontent.com/assets/23551298/23786215/c4eb56c4-0591-11e7-867f-8ad8c4198254.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 issue #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-03-09 Thread jayakarteek
Github user jayakarteek commented on the issue:

https://github.com/apache/cloudstack/pull/1918
  
Hi @nvazquez , @borisstoyanov , @rafaelweingartner ,

Pardon me for very late reply.
I tried with new property "summary.config.cpuReservation" , but it deals 
with the CPU reservation for the VM that is configured by VMWare.

It is different from the CPU speed, we define in CPU service offerings, as 
explained by below link


http://www.vfrank.org/2013/09/19/understanding-vmware-reservations-limits-and-shares/







---
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 #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-02-13 Thread jayakarteek
Github user jayakarteek commented on the issue:

https://github.com/apache/cloudstack/pull/1918
  
HI  raferweingartner, nvazquez<https://github.com/nvazquez>,

Yes as specified in mail,
The issue is specific to VMware.

--JayaKarteek

From: Nicolas Vazquez [mailto:notificati...@github.com]
Sent: Tuesday, February 14, 2017 5:36 AM
To: apache/cloudstack <cloudst...@noreply.github.com>
Cc: Jayakarteek Vasana <jayakarteek_vas...@accelerite.com>; Mention 
<ment...@noreply.github.com>
Subject: Re: [apache/cloudstack] Management Server UI (VM statistics page) 
CPU Utilized value is incorrect. (#1918)


Hi @jayakarteek<https://github.com/jayakarteek> 
@rafaelweingartner<https://github.com/rafaelweingartner>,

I confirm issue, I tested using an instance with 1 CPU 500Mhz. I'll add 
screenshots for cases when CPU utilization is low and high:

First, with little utilization, vSphere shows 51Mhz CPU Utilization, while 
on Management Server statistics shows 51% CPU utilization

[cpu51]<https://cloud.githubusercontent.com/assets/5295080/22908776/1e437a10-f22f-11e6-87a5-6933c990f911.JPG>

[cpu512]<https://cloud.githubusercontent.com/assets/5295080/22908783/26db1ba6-f22f-11e6-9a83-8743d7abb0e8.JPG>

Then, ran a loop to increase CPU utilization increasing it to ~1609Mhz and 
Management Server statistics shows 1609% CPU Utilization,

[cpu1600]<https://cloud.githubusercontent.com/assets/5295080/22908949/fed18ad6-f22f-11e6-9635-23cd5da38cba.JPG>

[cpu16002]<https://cloud.githubusercontent.com/assets/5295080/22908951/01fb02fa-f230-11e6-8f72-739709c87984.JPG>

Hope this helps!

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on 
GitHub<https://github.com/apache/cloudstack/pull/1918#issuecomment-279564806>, 
or mute the 
thread<https://github.com/notifications/unsubscribe-auth/AWddQthernP9VQSqZ9_-mWxvHl93t9MIks5rcPADgaJpZM4LxMFs>.



DISCLAIMER
==
This e-mail may contain privileged and confidential information which is 
the property of Accelerite, a Persistent Systems business. It is intended only 
for the use of the individual or entity to which it is addressed. If you are 
not the intended recipient, you are not authorized to read, retain, copy, 
print, distribute or use this message. If you have received this communication 
in error, please notify the sender and delete all copies of this message. 
Accelerite, a Persistent Systems business does not accept any liability for 
virus infected mails.



---
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 #1918: Management Server UI (VM statistics page) CPU...

2017-01-30 Thread jayakarteek
GitHub user jayakarteek opened a pull request:

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

Management Server UI (VM statistics page) CPU Utilized value is incorrect.

VMware CPU Utilised  value is in Mhz , in MS value is displayed directly.
If CPU utilised is 500Mhz,  UI value is displayed as 500 %

Planned to restrict changes in resource layer VmwareResource.java, but cpu 
allocated to VM by VMware is not same as  allocated in the service offering.

If CPU limit flag is set to false in  service offerings  the Allocated CPU 
can go beyond what is assigned.
So picking actual user assigned Total CPU from database for calculating 
percentage.  
It is expected that percentage would go beyond 100 percent.

 so changes are made in UserVmJoinDaoImpl.java 


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

$ git pull https://github.com/jayakarteek/cloudstack cloudstack-9762

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

https://github.com/apache/cloudstack/pull/1918.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 #1918


commit 58d721603300b236bded82147363a06e80ecf67a
Author: Jayakarteek Vasana <jayakarteek_vas...@persistent.co.in>
Date:   2017-01-30T09:06:03Z

 Bug fix for CLOUDSTACK-9762   Management Server UI (VM statistics page) 
CPU Utilized value 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.
---


[GitHub] cloudstack issue #1756: CLOUDSTACK-9585 UI doesn't give an option to select ...

2016-11-21 Thread jayakarteek
Github user jayakarteek commented on the issue:

https://github.com/apache/cloudstack/pull/1756
  
@yvsubhash on testing PR  , faced issues while registering new templates.

LOG:

command=listConfigurations=json=xenserver.pvdriver.version&_=1479725144072
The given command:listConfigurations does not exist or it is not available 
for user with id:7

command=listConfigurations=json=xenserver.pvdriver.version&_=1479725144072


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