Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1944
@rhtyd Will be trying this out today - will post my comments.
---
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
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1944
@rhtyd On the contrary, I think you adding metrics as pluggable API is a
good idea. However, to seriously make the feature pluggable the UI component
should also load based on a plugin
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1944
@rhtyd I guess this is fine for now. Thanks!
---
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
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1852
@ustcweizhou The reason for not putting this in vmActionFilter is that the
snapshot action itself is allowed when a VM is shut down. We only want to
disable the option for memory snapshot
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1852
ping @koushik-das.
When VM is running and snapshot option is clicked, the memory option is
enabled.
 for
more
Github user rashmidixit closed the pull request at:
https://github.com/apache/cloudstack/pull/1889
---
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
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1889
@karuturi requesting you to merge these 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
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1889
@karuturi I have rebased with master and pushed. I still see failures in
Travis but they dont seem to be with my changes. Can you please check?
---
If your project is set up for it, you
Github user rashmidixit closed the pull request at:
https://github.com/apache/cloudstack/pull/1889
---
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
GitHub user rashmidixit reopened a pull request:
https://github.com/apache/cloudstack/pull/1889
CLOUDSTACK-9718: Revamp the dropdown showing lists of hosts available for
migration in a Zone
Refer to
[CLOUDSTACK-9718](https://issues.apache.org/jira/browse/CLOUDSTACK-9718) for
more
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1826
Closing this since #1944 has been merged.
---
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
Github user rashmidixit closed the pull request at:
https://github.com/apache/cloudstack/pull/1826
---
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
Github user rashmidixit commented on the pull request:
https://github.com/apache/cloudstack/pull/1546#issuecomment-220524210
While we are on the topic of updating the tech stack, have we considered
moving to using either LESS or Sass for CSS. CSS pre-processors bring more
modularity
GitHub user rashmidixit opened a pull request:
https://github.com/apache/cloudstack/pull/1819
CLOUDSTACK-9653 The system capacity was not getting calculated correctly
When listCapacity is used with the sortBy=usage option, the capacity was
not getting calculated correctly
GitHub user rashmidixit opened a pull request:
https://github.com/apache/cloudstack/pull/1826
CLOUDSTACK-9675: Mem Usage and Mem Allocation show NaN
More for information refer to
[CLOUDSTACK-9675](https://issues.apache.org/jira/browse/CLOUDSTACK-9675)
You can merge this pull
GitHub user rashmidixit opened a pull request:
https://github.com/apache/cloudstack/pull/1848
CLOUDSTACK-9693 Cluster View - Status symbol does not change based on
Cluster state
Refer to
[CLOUDSTACK-9693](https://issues.apache.org/jira/browse/CLOUDSTACK-9693) for
more details
GitHub user rashmidixit opened a pull request:
https://github.com/apache/cloudstack/pull/1852
VM snapshot is disabled if the VM Instance is off
Refer to
[CLOUDSTACK-9695](https://issues.apache.org/jira/browse/CLOUDSTACK-9695) for
more details.
You can merge this pull request into
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1852
As part of this fix, added a new property called "isDisabled" for a field
in a dialog. If this function returns true, then the field will be disabled.
---
If your project is set
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1848
This is done. Thanks @rhtyd.
---
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
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1826
@rhtyd - There have been four things fixed here. I have explained each
commit below. Your review/comments is most appreciated.
1. Goto Zone metrics or Hosts metrics. NUmerical
GitHub user rashmidixit opened a pull request:
https://github.com/apache/cloudstack/pull/1855
CLOUDSTACK-9697: Better error message if user tries to shrink VM ROOT
Volume size
Skip the API call altogether if the UI detects this case and throw a more
user friendly message
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1826
Added one more fix to this:
1. List of Virtual Machines will be incorrect if the number is greater than
the pagesize. Added -1 to the call of the API.
---
If your project is set up
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1852
@rhtyd This is a minor improvement and hence in master. I have updated the
corresponding bugs' fix version.
---
If your project is set up for it, you can reply to this email and have
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1826
@rhtyd Have you been able to take a look at the commits? Will appreciate
your comments.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
GitHub user rashmidixit opened a pull request:
https://github.com/apache/cloudstack/pull/1877
Change of network to rvr for baremetal caused vm deployment to get stuck
Refer to
[CLOUDSTACK-9716](https://issues.apache.org/jira/browse/CLOUDSTACK-9716) for
more details
You can merge
GitHub user rashmidixit reopened a pull request:
https://github.com/apache/cloudstack/pull/1819
CLOUDSTACK-9653 The system capacity was not getting calculated correctly
When listCapacity is used with the sortBy=usage option, the capacity was
not getting calculated correctly
Github user rashmidixit closed the pull request at:
https://github.com/apache/cloudstack/pull/1819
---
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
GitHub user rashmidixit opened a pull request:
https://github.com/apache/cloudstack/pull/1884
Add global setting for enable/disable Metrics feature
The Metrics view for each type of entity basically fires APIs and
calculates required values on the client end. For e.g. to display
GitHub user rashmidixit opened a pull request:
https://github.com/apache/cloudstack/pull/1889
CLOUDSTACK-9718: Revamp the dropdown showing lists of hosts available for
migration in a Zone
Refer to
[CLOUDSTACK-9718](https://issues.apache.org/jira/browse/CLOUDSTACK-9718) for
more
GitHub user rashmidixit opened a pull request:
https://github.com/apache/cloudstack/pull/1894
CLOUDSTACK-9700 Allow user to Register/Copy templates to multiple zones at
the same time
A user can currently copy or register a template with only one zone at a
time.
The idea of
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1884
Thank you @jburwell. I will incorporate the comments in a couple of days
and resubmit.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1826
@rhtyd Please take a look now. I have squashed the 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
Github user rashmidixit closed the pull request at:
https://github.com/apache/cloudstack/pull/1852
---
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
GitHub user rashmidixit reopened a pull request:
https://github.com/apache/cloudstack/pull/1852
CLOUDSTACK-9695: VM snapshot is disabled if the VM Instance is off
Refer to
[CLOUDSTACK-9695](https://issues.apache.org/jira/browse/CLOUDSTACK-9695) for
more details.
You can merge
Github user rashmidixit commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1889#discussion_r97313518
--- Diff: ui/scripts/ui-custom/migrate.js ---
@@ -0,0 +1,127 @@
+// Licensed to the Apache Software Foundation (ASF) under one
--- End diff
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1894
Attaching new screens for the changes made here.
![copytemplate-withsearchzone](https://cloud.githubusercontent.com/assets/13415006/22205048/27f14e6a-e19b-11e6-8ba3-e0d4d8e7993a.PNG
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1855
@sadhugit I have updated the bug description based on your comments.
---
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
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1884
@rhtyd Thanks for the update.
---
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
Github user rashmidixit commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1889#discussion_r101275471
--- Diff:
api/src/org/apache/cloudstack/api/command/admin/host/ListHostsCmd.java ---
@@ -206,7 +206,7 @@ public void execute
Github user rashmidixit commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1889#discussion_r101275818
--- Diff:
api/src/org/apache/cloudstack/api/command/admin/host/ListHostsCmd.java ---
@@ -206,7 +206,7 @@ public void execute
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1889
@karuturi ping.
---
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
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1819
Looked at the errors. None of them have anything to do with the code
changes.
ping @karuturi.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1826
@rhtyd I will take a look. However, this pull request is to do with
additional bug fixes. Can you please review.
---
If your project is set up for it, you can reply to this email and have
54 matches
Mail list logo