Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1957
tested. LGTM
btw, you can use "git push --force" to overwrite the code
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1873
code LGTM
one small suggestion:
Could you modify line 2253 (_vmIpFetchThreadExecutor.execute) to be
executed only when hypervisor is XenServer(already supported) or Vmware
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1873
@jayapalu Thanks. Have you made changes for kvm ? I cannot find
GetVmIpAddressCommand in plugins/hypervisors/kvm(actually only find it in
plugins/hypervisors/xenserver/)
---
If your
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1965
@bvbharatk can you format the 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
Github user ustcweizhou commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1950#discussion_r102716218
--- Diff: .gitignore ---
@@ -51,7 +51,6 @@ tools/cli/build/
*.jar
*.war
*.mar
-*.zip
--- End diff --
@wido ,yes
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1922
LGTM as it is same to commit edd839ab10226760749d4fc15703a41eddc1dfdd
not tested yet, as VPC with RVR is working fine, so I assume this fixes the
issue in VPC with single VR
Github user ustcweizhou commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1972#discussion_r103072546
--- Diff: engine/schema/src/com/cloud/network/dao/LoadBalancerDaoImpl.java
---
@@ -75,4 +84,40 @@ protected LoadBalancerDaoImpl
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1950
@rhtyd The permission issue seems to be caused by
./scripts/vm/systemvm/injectkeys.sh
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1950
updated.
changes
(1) add init-system-helpers version
(2) add ExecStartPre to avoid the issue caused by JAVA_HOME
---
If your project is set up for it, you can reply to this
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1972
code 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
enabled and wishes so
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1907
code 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
enabled and wishes so
Github user ustcweizhou commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1978#discussion_r103416124
--- Diff: server/test/resources/createNetworkOffering.xml ---
@@ -34,22 +34,23
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1241
I am wondering if this is a general issue when create other resources
(volume,vm,etc) using integration port 8096.
---
If your project is set up for it, you can reply to this email
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1771
@nitin-maharana you may add "ignoreProject: true" in
createURL('dedicateGuestVlanRange') if scope is set to Domain
---
If your project is set up for it, you can rep
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1771
@nitin-maharana I thought dedicating guest vlan range is already
implemented in CLOUDSTACK-8958.
Actually they are two stories.
In this case, we have to remove the Domain scope from
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1980
is there any issue for normal users ?
---
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 ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1773
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
enabled and wishes so, or if
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1980
tested. 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
enabled and wishes so
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1868
Do we need to add a global setting to determine whether deploy to other
hosts if the specified host is out of capacity ?
---
If your project is set up for it, you can reply to this email
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1868
@anshul1886
Considering about backwards compatibility. I would like to add the global
setting, then users will have two options
(1) deploy vm fails if specified host does not have
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1771
@nitin-maharana should projectId dependsOn domainId ? If the domain
changes, the project list should change according to the new domainid.
---
If your project is set up for it, you can
Github user ustcweizhou commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1726#discussion_r104370668
--- Diff: server/src/com/cloud/storage/StorageManagerImpl.java ---
@@ -2263,27 +2264,43 @@ public void cleanupDownloadUrls
Github user ustcweizhou commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1991#discussion_r104393162
--- Diff: scripts/vm/hypervisor/xenserver/vmops ---
@@ -1267,9 +1267,9 @@ def cache_ipset_keyword():
type = getIpsetType()
tmpname
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1991
code LGTM. not tested.
---
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 ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1965
@bvbharatk I think the new password should be saved to one of the running
routers, not all running routers.
I disagree with change line 797.
---
If your project is set up for it, you
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1873
@jayapalu LGTM
I have two questions
(1) Is it possible to disable the check ?
(2) Could you please fix CLOUDSTACK-9413 (with this PR or in new PR) ?
---
If your project
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1970
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
enabled and wishes so, or if
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1868
@anshul1886
Sorry I think it would be better to add a global setting.
This is campatible with 3.X, but not with 4.X which has more users.
---
If your project is set up for it, you
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1771
@nitin-maharana I think it is better to list projects by domain.
Just an example, if there are 3 projects with same name created in 3
domains, it will be difficult to choose.
---
If
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1771
@nitin-maharana cannot we dedicate guest vlan range to projects in other
domains ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1771
@nitin-maharana
if this is performed in project view, should projectid be passed ?
projectId: args.context.projects[0].id
---
If your project is set up for it, you can reply to
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/827
@yvsubhash Sorry I still disagree with it.
---
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 ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1771
@nitin-maharana yes, you are right. I forgot it.
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
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1988
awesome
@DaanHoogland could you upload /etc/cloudstack/management/k8s-master.yml
and /etc/cloudstack/management/k8s-node.yml as well ?
For the license on top of some files
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1868
@anshul1886 good, I will test it.
Could you please change the default value to true ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1995
@abhinandanprateek I woud say the change is good, but it does not really
fix the issue you described.
The issue seems to be caused by missing
/var/cache/cloud/cloud-scripts-signature
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1995
@abhinandanprateek can you spend time on what caused the issue ?
The VR will not be stopped with this PR, it will help troubleshooting the
issue, good.
I never saw this issue before
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1995
@rhtyd thanks for your explanation.
@abhinandanprateek 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
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1999
two questions
(1) is the network id set in response ? It seems not. Why do we set vpcid
but not networkid ?
(2) what if the vm is in multiple vpc tiers ?
I suggest to add
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1999
@resmo VM can be part of any isolated networks, shared networks, vpc tiers.
You can try adding another vpc tier to an existing vm in a vpc.
Anyway, adding Vpcid in NicResponse
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1999
@resmo I still think vpcid should not be added to UserVmResponse, because
vm can be added to multiple vpc tiers (same as networks, this is why I asked
about networkid at the beginning), I
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1999
@resmo please create another vpc and some vpc tiers, and have a retry
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1999
@resmo I agree with adding vpcid to NicResponse, then you do not need an
extra API call. You only need to search NicResponse in loop.
---
If your project is set up for it, you can reply to
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1965
@bvbharatk Yes, you might know only the vm has sshkey attached will have
the password in vm details. If the vm does not have ssh keypair, then the vm
password will not saved into
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1889
@rashmidixit We have ported it in our branch and found it inconvenient to
click the OK button in dialog with warning "No more hosts are available for
migration" frequently.
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/2003
not tested it. but it seems the 'else' part also need to be changed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub a
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1889
@rashmidixit thanks for your work.
---
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 ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/2003
@swill what I mean is , if you do not want to change the logic, do you need
to change 'else:' to 'elif index == -1:' ?
---
If your project is set up for it, you can rep
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/2003
@borisstoyanov are you using vpc ?
---
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 ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/2003
the ips.json is very very weird
---
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 ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/2003
it seems nicDevId is not set correctly in java 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
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/2003
@borisstoyanov what's the last commit in your code ?
could you check how many nics attached to the VR in database, select * from
nics where instance_id= ?
---
If your project is s
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/2003
@borisstoyanov moreover, can you please restart the network with cleanup,
to see if the new VRs can start ?
---
If your project is set up for it, you can reply to this email and have your
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1726
code LGTM
@yvsubhash it would be nice if you can reformat part of the code with java
code conventions.
---
If your project is set up for it, you can reply to this email and have your
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/2003
nics in database seems ok, so it looks some issue in creating
ip_associations.json
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/2003
@swill
first_i_p in json -> firstIP in java. This is formatted by google.gson
it is defined in api/src/com/cloud/agent/api/to/IpAddressTO.java, and used
in server/src/com/cl
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/2003
@borisstoyanov I think this is caused by KVM plugin.
can you please perform the following on KVM host ?
(1) change INFO to DEBUG in /etc/cloudstack/agent/log4j-cloud.xml, and
restart
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/2003
@borisstoyanov
I suspect this is caused by the name of your public interface (p55p1).
can you please try the following change
```
diff --git
a/plugins/hypervisors/kvm/src
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1889
@rashmidixit thanks for the update.
Compared with old patch, I notice you changed the return if args.page == 1.
However, it does not solve my issue. We have several hosts (more than 1
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1889
@rashmidixit finally I decide to say LGTM to 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
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1889
LGTM, please merge it.
---
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
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1873
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
enabled and wishes so, or if
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/2003
@borisstoyanov could you please test with my suggestion? I think it should
fix the duplicated nics in VR.
---
If your project is set up for it, you can reply to this email and have your
GitHub user ustcweizhou opened a pull request:
https://github.com/apache/cloudstack/pull/2011
CLOUDSTACK-9811: fix duplicated nics on VR caused by nic name
pp
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ustcweizhou
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/2011
@borisstoyanov first 4.9, then merge into master.
this patch can also be applied to master.
```
wget --no-check-certificate
https://github.com/apache/cloudstack/pull/2011.patch
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/2003
@karuturi to be honest, I think PR#2011 fixes the issue, not this one.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/2011
@DaanHoogland @karuturi @borisstoyanov
Here is a short explanation why the issue happened:
Normally getBroadcastUriFromBridge method in KVM plugin will get the URI
from a
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1726
LGTM now
---
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
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1980
@rhtyd it seems we can discard 'router' in vpc part.
---
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 projec
GitHub user ustcweizhou opened a pull request:
https://github.com/apache/cloudstack/pull/2025
Fix error in restart network in 4.10.0.0 RC
The PR fixes the error in restart network:
2017-04-04 10:27:39,217 DEBUG [c.c.n.r.NetworkHelperImpl]
(API-Job-Executor-2:ctx-08904854 job
Github user ustcweizhou commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1950#discussion_r109633875
--- Diff: debian/control ---
@@ -16,14 +16,15 @@ Description: A common package which contains files
which are shared by several C
Package
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1950
it would be nice if this can be merged into 4.10.0.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
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/2033
@karuturi LGTM. I did not see big difference from #1950
---
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
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/2041
@Slair1 this is already fixed by commit ba9dcba in 4.10/master
---
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 ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1716
this is not an isolated issue.
See https://issues.apache.org/jira/browse/CLOUDSTACK-8186
it would be better to fix it in framework/db/
---
If your project is set up for it, you can
Github user ustcweizhou commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1212#discussion_r47437500
--- Diff: server/test/com/cloud/user/AccountManagerImplTest.java ---
@@ -185,6 +186,8 @@
@Mock
GlobalLoadBalancerRuleDao
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/1212#issuecomment-164198697
@DaanHoogland I will add some tests in the file when I have time.
currently I have some other tasks to do (not porting).
---
If your project is set up for
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/1198#issuecomment-164199095
@DaanHoogland @wilderrodrigues
assume there are two routers now routerA (master) and routerB (backup).
if we destroy routerA (master) at first, then
Github user ustcweizhou commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1209#discussion_r47437641
--- Diff: engine/schema/src/com/cloud/vm/dao/NicDaoImpl.java ---
@@ -222,11 +222,20 @@ public String getIpAddress(long networkId, long
instanceId
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/977#issuecomment-164401512
@remibergsma
rebase with latest master 7c83e1b2402db9e359fb086b158d65c25309a70e
result of squashing the commits:
(before suqash, 16 commits
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/1264#issuecomment-166223301
@nitin-maharana
I am not sure if it is necessary to add the following change after success,
but I added it in my fix, basides your change
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/866#issuecomment-166890582
good, this might also be useful for restartnetwork, see #1198
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/1264#issuecomment-168148620
LGTM.
This is same to the fix in our internal branch.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/1309#issuecomment-169267155
LGTM, just code review. simple but really make sense.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/1325#issuecomment-170458897
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 ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/1297#issuecomment-170847583
this does not work if there are multiple shared network in advanced zone
with sg.
I will fix it.
---
If your project is set up for it, you can reply to
GitHub user ustcweizhou opened a pull request:
https://github.com/apache/cloudstack/pull/1342
CLOUDSTACK-6181: Add 'root disk size' field in instance wizard for KVM
templates
This commit includes three changes:
(1) Revert commit 13bf1ec5da9891eca98ef5dab61bf74b6db953
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/1309#issuecomment-172261610
I agree we need to fix some issues like this , which is difficult to be
reproduced but it is obviously an issue from code view.
---
If your project is set up
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/1297#issuecomment-172261703
LGTM, based on manual functional testing in advanced zone with security
groups.
---
If your project is set up for it, you can reply to this email and have
GitHub user ustcweizhou opened a pull request:
https://github.com/apache/cloudstack/pull/1357
CLOUDSTACK-8958: release dedicated ip range in domain removal
We are able to assign didacated vlan ip ranges after the merge of
CLOUDSTACK-8958.
These ip ranges need to be released
GitHub user ustcweizhou opened a pull request:
https://github.com/apache/cloudstack/pull/1363
Fix issue in scale VM to dynamic service offering
This reverts commit 9c4162ac7f451fc3e2155418dcfff224c8c08a4a and
16baa1289b7de383e98d0070717b3f1873fa2db3
Before change
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/1342#issuecomment-175894714
add (GB) to the label. Please test it.
---
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 ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/1363#issuecomment-176945786
@alexandrelimassantana I agree with you about the code changes.
However, this commit only reverts the commit 9c4162a and 16baa12 to fix the
issue
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/977#issuecomment-180465442
rebased
---
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 ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/926#issuecomment-180467220
rebased
---
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 ustcweizhou commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1357#discussion_r52076608
--- Diff: server/src/com/cloud/configuration/ConfigurationManagerImpl.java
---
@@ -4932,6 +4932,32 @@ public Domain getVlanDomain(long vlanId
GitHub user ustcweizhou opened a pull request:
https://github.com/apache/cloudstack/pull/1410
CLOUDSTACK-6928: fix issue disk I/O throttling not applied
Disk IO throttling (for KVM) is not applied in the merge of 4.2.
Tests passed:
(1) start vm
(2) attach volume
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/844#issuecomment-145445260
guys, anyone has 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
GitHub user ustcweizhou opened a pull request:
https://github.com/apache/cloudstack/pull/918
CLOUDSTACK-8941: fix NPE when migrate vm to other zone-wide pools the
second time
This is because the pod_id is set to NULL at the first time when I migrate
the instance to a zone-wide
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/844#issuecomment-146839857
@wilderrodrigues thanks for you testing. I believe this PR will not impact
the current network/vr/storage.
It will only change the owner of a vm, and add the
1 - 100 of 473 matches
Mail list logo
