Github user niteshsarda commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1987#discussion_r104283067
--- Diff: server/src/org/apache/cloudstack/region/RegionManagerImpl.java ---
@@ -229,7 +232,7 @@ public boolean deleteUser(DeleteUserCmd cmd) {
Github user niteshsarda commented on the issue:
https://github.com/apache/cloudstack/pull/1987
@rafaelweingartner : As per your suggestion, I have changed the search
query and also removed the unused method updateDomain from
managementserviceimpl class.
Please check.
---
If
Github user remibergsma commented on the issue:
https://github.com/apache/cloudstack/pull/1980
@rhtyd Looks great! Shall we do the same for the VPC routers, so that the
overviews look the same?
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user cloudmonger commented on the issue:
https://github.com/apache/cloudstack/pull/883
### ACS CI BVT Run
**Sumarry:**
Build Number 422
Hypervisor xenserver
NetworkType Advanced
Passed=105
Failed=0
Skipped=7
_Link to logs Folder (search
I think these PRs have met the criteria for merging. Can we have them included
in RC2 ?
CLOUDSTACK-9569: propagate global configuration
router.aggregation.command.each.timeout to KVM agent #1856
Advanced isolated network egress destination cidr support added#1866
CLOUDSTACK-9698 [VMware] Make
If no one else objects before Monday and they have tests, I will merge.
~Rajani
Sent from phone.
On 4 Mar 2017 8:13 p.m., "Haijiao" <18602198...@163.com> wrote:
I think these PRs have met the criteria for merging. Can we have them
included in RC2 ?
CLOUDSTACK-9569: propagate global configur
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 there,
so I
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1776
@blueorangutan package
---
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 an
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1776
Thanks for this fix @priyankparihar , if it addresses only the input
validation should be pretty easy to test.
---
If your project is set up for it, you can reply to this email and have y
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1776
@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep
you posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
re
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1961
Yes @serg38, but currently testing it. So this batch doesn't have 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 yo
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1862
yes, @sateesh-chodapuneedi those are addressed in another 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 proje
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1776
Packaging result: âcentos6 âcentos7 âdebian. JID-561
---
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 d
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1776
@blueorangutan 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 w
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1776
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has
been kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1981
Thanks, the tests failures looks to be not related to these changes.
@prashanthvarma
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1813
Hi @serg38 ,
Here's the management log:
```
2017-03-04 20:43:40,972 INFO [c.c.h.v.r.VmwareResource]
(DirectAgent-249:ctx-585f426d 10.2.2.73, job-29/job-32, cmd: StartCom
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1955
I think that's the issue at our environment, it's trying to ssh via
hostname instead of IP. Can you change that to IP please @nathanejohnson ?
`Trying SSH Connection: Host:pr1955-t
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1776
Trillian test result (tid-930)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 35187 seconds
Marvin logs:
https://github.com/blueoranguta
Github user nitin-maharana commented on the issue:
https://github.com/apache/cloudstack/pull/1859
ping @rajesh-battala @sowmyakrishn
---
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 feat
Github user sateesh-chodapuneedi commented on the issue:
https://github.com/apache/cloudstack/pull/1862
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
21 matches
Mail list logo
