Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/2036
I tried to access the Jenkins details to see the checking problems, but I
am getting a 404.
Am I missing something?
---
If your project is set up for it, you can reply to this
GitHub user rafaelweingartner opened a pull request:
https://github.com/apache/cloudstack/pull/2036
[CLOUDSTACK-9858] Retirement of midonet plugin (build disabling)
Recently there have been two threads asking and discussing the
âmidonetâ integration with Apache CloudStack (ACS
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1935
@nvazquez now everything seems to be ok
LGTM.
Great job!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1935#discussion_r103529571
--- Diff: server/src/com/cloud/user/DomainManagerImpl.java ---
@@ -273,82 +284,145 @@ public boolean deleteDomain(long domainId, Boolean
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1935#discussion_r102538324
--- Diff: server/src/com/cloud/user/DomainManagerImpl.java ---
@@ -109,6 +112,20 @@
@Inject
MessageBus _messageBus
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1935#discussion_r103529869
--- Diff: server/test/com/cloud/user/DomainManagerImplTest.java ---
@@ -134,4 +164,69 @@ public void testFindDomainByIdOrPathValidId
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1278
@anshul1886, this pointing finger thing is not good.
I do not know why people did not do the work as it should have been done
before. I was probably not around when that was
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1859
Folks, what about a middle ground here?
I was checking the commits. For instance, all of the commits
"Added/implemented XXX ." could be all squashed by the same aut
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/837#discussion_r108469206
--- Diff: server/src/com/cloud/alert/AlertManagerImpl.java ---
@@ -767,7 +767,9 @@ public void sendAlert(AlertType alertType, long
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1918#discussion_r108463719
--- Diff: server/src/com/cloud/api/query/dao/UserVmJoinDaoImpl.java ---
@@ -196,6 +196,7 @@ public UserVmResponse newUserVmResponse
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1582#discussion_r106670299
--- Diff:
api/src/org/apache/cloudstack/api/response/AutoScaleVmProfileResponse.java ---
@@ -75,6 +75,7 @@
@Parameter(name
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1994#discussion_r106313430
--- Diff:
engine/schema/src/com/cloud/storage/dao/StoragePoolTagsDaoImpl.java ---
@@ -77,4 +92,71 @@ public void deleteTags(long poolId
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1994#discussion_r106306374
--- Diff:
engine/schema/src/com/cloud/storage/dao/StoragePoolTagsDaoImpl.java ---
@@ -77,4 +92,71 @@ public void deleteTags(long poolId
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1994#discussion_r106001103
--- Diff:
engine/schema/src/com/cloud/storage/dao/StoragePoolTagsDaoImpl.java ---
@@ -77,4 +92,71 @@ public void deleteTags(long poolId
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/837#discussion_r106188366
--- Diff: server/src/com/cloud/alert/AlertManagerImpl.java ---
@@ -767,7 +767,9 @@ public void sendAlert(AlertType alertType, long
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/837#discussion_r106186279
--- Diff:
plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java
---
@@ -512,7 +512,11 @@ public void
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/837#discussion_r106185631
--- Diff: engine/components-api/src/com/cloud/agent/AgentManager.java ---
@@ -141,7 +142,7 @@
public void
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/837#discussion_r106185108
--- Diff:
api/src/org/apache/cloudstack/api/command/admin/host/ReconnectHostCmd.java ---
@@ -100,17 +103,18 @@ public Long getInstanceId
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1918
Awesome! This seems a good approach to deal with the issue.
@jayakarteek and @nvazquez thanks for the effort.
@jayakarteek sorry for all of the stubbornness ;)
You
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1278
@anshul1886 what method that is called in multiple places are you talking
about?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1994#discussion_r105993059
--- Diff:
engine/schema/test/org/apache/cloudstack/storage/datastore/db/PrimaryDataStoreDaoImplTest.java
---
@@ -0,0 +1,151
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1994#discussion_r105992559
--- Diff:
engine/schema/src/com/cloud/storage/dao/StoragePoolTagsDaoImpl.java ---
@@ -77,4 +92,71 @@ public void deleteTags(long poolId
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1994#discussion_r105959436
--- Diff:
engine/schema/src/com/cloud/storage/dao/StoragePoolTagsDaoImpl.java ---
@@ -77,4 +90,68 @@ public void deleteTags(long poolId
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1994#discussion_r105959564
--- Diff:
engine/schema/src/com/cloud/storage/dao/StoragePoolTagsDaoImpl.java ---
@@ -77,4 +90,68 @@ public void deleteTags(long poolId
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1994#discussion_r105959827
--- Diff:
engine/schema/test/com/cloud/storage/dao/StoragePoolTagsDaoImplTest.java ---
@@ -0,0 +1,105 @@
+// Licensed to the Apache
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/837#discussion_r105952120
--- Diff: server/src/com/cloud/alert/AlertManagerImpl.java ---
@@ -767,7 +767,9 @@ public void sendAlert(AlertType alertType, long
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/837#discussion_r105942978
--- Diff: engine/components-api/src/com/cloud/agent/AgentManager.java ---
@@ -141,7 +142,7 @@
public void
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1918
@jayakarteek from what I understood so far, there will always be a mismatch.
For instance, the way it is implemented so far, if a VM uses a service
offering that provides 1000
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1918
This seems to be a great idea @nvazquez.
What do you think @jayakarteek ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1859
@nitin-maharana I was looking at the PR.
Do you need to split everything thing there in a different commit?
I think that we still do not have a clear understanding when and how
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/837#discussion_r105725769
--- Diff: server/src/com/cloud/alert/AlertManagerImpl.java ---
@@ -767,7 +767,9 @@ public void sendAlert(AlertType alertType, long
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/837#discussion_r105723834
--- Diff:
engine/orchestration/src/com/cloud/agent/manager/AgentManagerImpl.java ---
@@ -986,33 +986,28 @@ public Answer easySend(final Long
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/837#discussion_r105724153
--- Diff:
plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java
---
@@ -512,7 +512,11 @@ public void
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/837#discussion_r105723252
--- Diff: api/src/com/cloud/resource/ResourceService.java ---
@@ -50,7 +52,7 @@
Host cancelMaintenance(CancelMaintenanceCmd cmd
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/837#discussion_r105723444
--- Diff:
api/src/org/apache/cloudstack/api/command/admin/host/ReconnectHostCmd.java ---
@@ -100,17 +103,18 @@ public Long getInstanceId
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/837#discussion_r105723986
--- Diff: engine/components-api/src/com/cloud/agent/AgentManager.java ---
@@ -141,7 +142,7 @@
public void
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1918
@jayakarteek thanks for the explanation
I thought that ACS would configure those parameters automatically. The way
you described it feels like ACS is not limiting the use of CPU
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1878#discussion_r105678507
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java
---
@@ -1928,6 +1929,54 @@ protected
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1878#discussion_r105676576
--- Diff: engine/schema/src/com/cloud/vm/dao/NicDaoImpl.java ---
@@ -302,4 +309,17 @@ public int countNicsForStartingVms(long networkId
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1878#discussion_r105679910
--- Diff:
plugins/hypervisors/vmware/test/com/cloud/hypervisor/vmware/resource/VmwareResourceTest.java
---
@@ -216,6 +216,20 @@ public void
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1878#discussion_r105677642
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java
---
@@ -2072,6 +2121,17 @@ protected
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1994#discussion_r105488183
--- Diff:
engine/schema/src/com/cloud/storage/dao/StoragePoolTagsDaoImpl.java ---
@@ -77,4 +90,71 @@ public void deleteTags(long poolId
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1994#discussion_r105489528
--- Diff:
engine/schema/src/org/apache/cloudstack/storage/datastore/db/PrimaryDataStoreDaoImpl.java
---
@@ -409,15 +460,13 @@ public
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1994#discussion_r105488722
--- Diff:
engine/schema/src/com/cloud/storage/dao/StoragePoolTagsDaoImpl.java ---
@@ -77,4 +90,71 @@ public void deleteTags(long poolId
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1278
Great @anshul1886, we seem to start converging.
So, what you call setting context, is the execution of
âorg.apache.cloudstack.context.CallContext.getCallingAccount()â
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1918
@jayakarteek I did not understand your doubt.
I see by your SS that the reservation field contains the speed configured
in your service offering. From what I understood so far
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1994
@nvazquez I am assuming the change needed to fix the bug is line 135 at
PrimaryDataStoreHelper.java?
The other changes are regarding code conformity
---
If your project is set
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1582#discussion_r105259838
--- Diff:
api/src/org/apache/cloudstack/api/response/AutoScaleVmProfileResponse.java ---
@@ -75,6 +75,7 @@
@Parameter(name
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/669
@anshul1886 you asked to show you how it can be done.
I opened a PR https://github.com/anshul1886/cloudstack-1/pull/1 for the
branch you are using in this PR. Can you take a look
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/669#discussion_r104769820
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtCopyFileInVmCommandWrapper.java
---
@@ -0,0 +1,59
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/669#discussion_r104770076
--- Diff:
plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/wrapper/xenbase/CitrixCopyFileInVmCommandWrapper.java
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/669#discussion_r104768693
--- Diff:
plugins/hypervisors/hyperv/src/com/cloud/hypervisor/hyperv/resource/HypervDirectConnectResource.java
---
@@ -595,6 +602,25
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/669#discussion_r104769967
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java
---
@@ -1279,6 +1285,27 @@ private static
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1278
First, I would like to apologize for being so stubborn, but if I do not
understand something I like to ask questions. I hope you do not mind.
Before we proceed with the
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1278
@anshul1886 sorry, but I still do not understand you.
Your changes affect `NetworkHelperImpl.java` class. Do we agree on that?
You changed lines 334 and 512. You
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1987
@niteshsarda thank you very much :)
The less duplicated or useless code around the better.
The method 'updateDomain' is huge and you just changed a single line th
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1987
@niteshsarda I suggest you adding line 2281 from "managementService" to
"domainMgr". The only difference between those two methods is this line.
Assuming that yo
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1973
@dmabry as I posted on the other PR, I only suggested `git cherry-pick`
because I thought it would preserve the commit hash. It was a misunderstanding
from me. The git hash is
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1974
@dmabry what about using `git cherry-pick` to introduce the changes that
were already merged into master in 4.9? Then, we can maintain the same commit
hash. This would facilitate
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1896
Got it, thanks for the explanation. I saw that, and I was wondering if it
could have been a bug or something else.
LGTM then.
---
If your project is set up for it, you can
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1896
@milamberspace @rhtyd do you know why the entry "label.host.alerts" was
removed from some languages?
---
If your project is set up for it, you can reply to this email and
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1935
@serg38 I did, the code is great as always. However, I have a concern about
that `rollBackState` variable being static there. Because the
`DomainManagerImpl ` is a singleton, using
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1957
Actually, my concerns are not regarding the response. Reading the name of
the method one can for sure understand what it returns. My concern is that
reading the parameter name
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1907
Ok, great.
Thanks for the explanation.
---
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 rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1907
@swill LGTM for the changes.
The changes introduced now are different from the last ones, right? Now it
is basically the addition of a log and the skipping of a processing
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1935#discussion_r102536773
--- Diff: server/src/com/cloud/user/DomainManagerImpl.java ---
@@ -273,82 +289,133 @@ public boolean deleteDomain(long domainId, Boolean
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1935#discussion_r102535697
--- Diff: server/src/com/cloud/user/DomainManagerImpl.java ---
@@ -273,82 +289,133 @@ public boolean deleteDomain(long domainId, Boolean
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1935#discussion_r102534985
--- Diff: server/test/com/cloud/user/DomainManagerImplTest.java ---
@@ -134,4 +164,67 @@ public void testFindDomainByIdOrPathValidId
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1935#discussion_r102323551
--- Diff: server/src/com/cloud/user/DomainManagerImpl.java ---
@@ -273,82 +289,133 @@ public boolean deleteDomain(long domainId, Boolean
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1935#discussion_r102487341
--- Diff: server/src/com/cloud/user/DomainManagerImpl.java ---
@@ -273,82 +289,133 @@ public boolean deleteDomain(long domainId, Boolean
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1935#discussion_r102322603
--- Diff: server/src/com/cloud/user/DomainManagerImpl.java ---
@@ -273,82 +289,133 @@ public boolean deleteDomain(long domainId, Boolean
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1935#discussion_r102486318
--- Diff: server/test/com/cloud/user/DomainManagerImplTest.java ---
@@ -134,4 +164,67 @@ public void testFindDomainByIdOrPathValidId
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1935
@nvazquez was the static declaration there before?! I am so sorry I missed
it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1957
@Ashadeepa thanks for the information.
As a tip, try to avoid opening duplicated PRs, it gets hard for reviewers
to track changes and discussions.
As @ustcweizhou
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1948
Great, nice work.
LGTM
---
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 rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1935
@nvazquez great work.
However, there is a catch there that I think you might have overlooked.
This problem is caused by the method extraction I suggested.
If you take a
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1956#discussion_r102310842
--- Diff:
engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java ---
@@ -744,14 +744,17 @@ protected boolean checkWorkItems
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1956#discussion_r102309517
--- Diff:
engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java ---
@@ -744,14 +744,17 @@ protected boolean checkWorkItems
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1956#discussion_r102308166
--- Diff:
engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java ---
@@ -744,14 +744,17 @@ protected boolean checkWorkItems
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1956#discussion_r102300083
--- Diff:
engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java ---
@@ -744,14 +744,17 @@ protected boolean checkWorkItems
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1953#discussion_r102294407
--- Diff: server/src/com/cloud/storage/VolumeApiServiceImpl.java ---
@@ -2639,22 +2639,23 @@ private int getMaxDataVolumesSupported(UserVmVO
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1956#discussion_r102292787
--- Diff:
engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java ---
@@ -744,14 +744,17 @@ protected boolean checkWorkItems
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1953#discussion_r102291066
--- Diff: server/src/com/cloud/storage/VolumeApiServiceImpl.java ---
@@ -2639,22 +2639,23 @@ private int getMaxDataVolumesSupported(UserVmVO
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1956#discussion_r102289931
--- Diff:
engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java ---
@@ -744,14 +744,17 @@ protected boolean checkWorkItems
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1957
@Ashadeepa why do we have 2 PRs for the same problem?
It seems that one of them can be closed.
---
If your project is set up for it, you can reply to this email and have your
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1948
@ProjectMoon may I ask you a question?
The "net" object is already an array/map, right?
---
If your project is set up for it, you can reply to this email and have
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1878#discussion_r102226833
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java
---
@@ -2071,6 +2120,14 @@ protected
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1918
That is a good idea @nvazquez, thanks for the help here ;)
What do you think guys (@nvazquez, @borisstoyanov, @jayakarteek)? if we use
the "summary.config.cpuReservati
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack-www/pull/35
@swill, could you take a look in this PR?
---
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 rafaelweingartner opened a pull request:
https://github.com/apache/cloudstack-www/pull/35
Update "who" page to reflect the new additions to PMC and committers
Self descriptive PR title ;)
You can merge this pull request into a Git repository by running:
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1918
@nvazquez no problem. I think it is ok to use everything that is available
because the service offering is not restricting the CPU usage.
@nvazquez would it be possible for
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1918
@nvazquez, this helped.
I checked the âVmwareResource.javaâ at line 5113 is obtained the CPU
usage of VM. From @jayakarteek explanations I understood that the usage does
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1919
Thanks for the explanation @serbaut. That is exactly why I asked; so, it is
not a reboot/restart per se. It is a re-deploy; the old virtual machine is
destroyed and a new one is
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack-www/pull/25
Before this PR reaches its one year of age, I will merge this tomorrow if
no one objects.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack-www/pull/34
Merged; the good part is that I followed the new tutorial we developed, and
it worked like a charm ;)
---
If your project is set up for it, you can reply to this email and have
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1878
Hi @sureshanaparti
Would you mind extracting the code at lines 1967-1972 to a specific method?
Then, it allows you to write test cases for the newly created method and proper
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1907#discussion_r100904660
--- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py ---
@@ -659,7 +663,10 @@ def is_guest_gateway(self, bag, ip
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1881
@sureshanaparti Thanks for the cleanup. LGTM
@sateesh-chodapuneedi this is a good question. The changes here remove an
attribute of an Enum that is not referenced anywhere. In my
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1886
@jayapalu you changed the request type to `POST`. Is the server side (the
bit that will handle/deliver this request) expecting `POST` requests as well?
Also, how do you feel
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1907#discussion_r100897492
--- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py ---
@@ -659,7 +663,10 @@ def is_guest_gateway(self, bag, ip
1 - 100 of 626 matches
Mail list logo
