Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1582
@borisstoyanov Can you run @blueorangutan tests on this please?
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1582
I see that its already using download.cloudstack.org. Its merge ready. Will
merge once I do some tests. Thanks @DaanHoogland
---
If your project is set up for it, you can reply to this email
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1994
Thanks @nvazquez Waiting for LGTMs
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1582
I want this to be updated to use download.cloudstack.org instead of
cloudstack.apt-get.eu.
I will send a PR with URL changes to this PR.
---
If your project is set up for it, you can
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/2009
@rhtyd Can you review the fix?
---
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 karuturi opened a pull request:
https://github.com/apache/cloudstack/pull/2009
CLOUDSTACK-9369 Fixed Ldap regression
Ldap auto creation of accounts is broken due to the security fix for
CLOUDSTACK-9369.
There was an explicit check to not allow login incase
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1994
@nvazquez any 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
enabled
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1582
@DaanHoogland any 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
enabled
Thank you Will for all the great work.
Congratulations Wido.
~ Rajani
http://cloudplatform.accelerite.com/
On March 16, 2017 at 10:30 PM, Will Stevens (sw...@apache.org)
wrote:
Hello Everyone,
It has been a pleasure working with you as the ACS VP over the
past year.
I would like to say Thank
GitHub user karuturi opened a pull request:
https://github.com/apache/cloudstack/pull/2007
CLOUDSTACK-9834: prepareTemplate API call doesn't work well with XenServer
& Local SR (Db_exn.Uniqueness_constraint_violation)
removed the host uuid from SR label so that any host which
you are in.
~ Rajani
http://cloudplatform.accelerite.com/
On March 14, 2017 at 7:15 PM, Pavan Kumar Aravapalli
(pavankuma...@persistent.com) wrote:
Hi,
I am unable to edit/add pages for the cwiki link
https://cwiki.apache.org/confluence/display/CLOUDSTACK/4.9+Design+Documents.
Can some one
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1582
systemvm template is now updated to 4.10
http://download.cloudstack.org/systemvm/4.10/ with pr #1982
---
If your project is set up for it, you can reply to this email and have your
reply
lets get to the blockers first.
~Rajani
http://cloudplatform.accelerite.com/
On Tue, Mar 14, 2017 at 11:39 AM, Haijiao <18602198...@163.com> wrote:
> Hi, Karuturi
>
>
> These PRs below seems ready to be merged too.
>
>
> #1883, CLOUDSTACK-9723: Enable unique mac addre
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1582
Hi @DaanHoogland Are you planning to change the url to
download.cloudstack.org?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
You(shanika) should have access to the edit the wiki now.
Thanks,
~ Rajani
http://cloudplatform.accelerite.com/
On March 12, 2017 at 8:43 PM, Shanika Ediriweera
(shanika...@cse.mrt.ac.lk) wrote:
Thank you.
Should I drop another mail saying I am interested in the above
mentioned
project
Best
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/2003
Thanks Will. Can you please add bug id(CLOUDSTACK-9811) to the PR and
commit message?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
1866 - failed BVT results(may not be related. But, needs to be
checked. Also, jenkins and travis failed)
1967 - no test results
rest merged.
Thanks,
~ Rajani
http://cloudplatform.accelerite.com/
On March 4, 2017 at 8:13 PM, Haijiao (18602198...@163.com) wrote:
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1967
@bvbharat can you start internal CI and post results?
---
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
We still have below three blockers open. I will create RC once
they are fixed.
~ Rajani
http://cloudplatform.accelerite.com/
On March 9, 2017 at 10:09 AM, Rajani Karuturi (raj...@apache.org)
wrote:
We currently have the below list of blockers:
BUGID - Summary - Assignee - Creator
CLOUDSTACK
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1994
@nvazquez I think any of the existing marvin tests didnt catch this bug. Is
it possible to add a marvin test for it?
---
If your project is set up for it, you can reply to this email and have
>
> >> On Mar 7, 2017, at 7:54 AM, Sergey Levitskiy <
> sergey.levits...@autodesk.com> wrote:
> >>
> >> Is VM has an VMsnaphsot? Is VM in Stopped state?
> >>
> >> On 3/6/17, 10:32 PM, "Tutkowski, Mike"
<mike.tutkow...@ne
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1991
I manually verified this. merging this 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
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1982
@rhtyd sorry for the confusion. ASF owns it(INFRA-13617). I asked him to
use these urls to avoid multiple updates.
---
If your project is set up for it, you can reply to this email and have
PRs are ready for the blockers. Waiting for reviews and test
results. Once they are ready, I will merge them(and a few more
bug fixes) and create RC2 (probably tomorrow, Wednesday)
Thanks,
~ Rajani
http://cloudplatform.accelerite.com/
On March 3, 2017 at 4:30 PM, Rajani Karuturi (raj
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1991
@borisstoyanov can you start the tests on this?
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1982
@kishankavala can you use the templates from
http://cloudstack.apt-get.eu/systemvm/4.10/ ?
---
If your project is set up for it, you can reply to this email and have your
reply appear
r/vmops | 25
+++--
1 files changed, 19 insertions(+), 6 deletions(-)
Jayapal, could you please have a look ?
2017-03-04 2:41 GMT+01:00 Rajani Karuturi <raj...@apache.org>:
Thanks for the update mike.
@wido, weizhou
Can you take a look at the issue please?
~Rajani
Sent fro
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1982
@wido Can you please update the systemvm templates at
http://cloudstack.apt-get.eu/systemvm ?
You can use the one from
https://builds.cloudstack.org/job/build-master-systemvm/148
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1792
code LGTM.
@nitin-maharana Please initiate internal CI run on @cloudmonger.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Problems caused when inputting double-byte numbers fo… #1967
在2017年03月03 19时00分, "Rajani Karuturi"<raj...@apache.org>写道:
I will create RC2 on Monday with the fixes mentioned in my
previous mail.
~ Rajani
http://cloudplatform.accelerite.com/
On March 3, 2017 a
51)
> >>at org.springframework.aop.framework.
> ReflectiveMethodInvocation.
> >> proceed(ReflectiveMethodInvocation.java:168)
> >>at org.springframework.aop.interceptor.
> ExposeInvocationInterceptor.
> >> invoke(ExposeInv
l.AsyncJobManagerImpl$5.
>> runInContext(AsyncJobManagerImpl.java:554)
>> at
org.apache.cloudstack.managed.context.ManagedContextRunnable$1.run(
>> ManagedContextRunnable.java:49)
>> at org.apache.cloudstack.managed.context.impl.
>> DefaultManagedContext$1.call(DefaultManaged
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1774
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
e results summaries. I’ve tried sending the execution logs
> but mail got rejected.
>
>
>
> > On Mar 2, 2017, at 9:14 AM, Rajani Karuturi <raj...@apache.org> wrote:
> >
> > On Wed, Mar 1, 2017 at 12:51 PM, Wido den Hollander <w...@widodh.nl>
> wrote
On Wed, Mar 1, 2017 at 12:51 PM, Wido den Hollander <w...@widodh.nl> wrote:
>
> > Op 1 maart 2017 om 2:12 schreef Rajani Karuturi <raj...@apache.org>:
> >
> >
> > Hi All,
> >
> > I've created a 4.10.0.0 release, with the following artif
Hi All,
I've created a 4.10.0.0 release, with the following artifacts up for a vote:
Git Branch and Commit
SH:https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.10.0.0-RC20170301T0634
Commit:7c1d003b5269b375d87f4f6cfff8a144f0608b67
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1832
> @karuturi Ok thanks for the clarifications, and it's the scenario I
thought about too. That being said, I'm currently thinking of a new approach
for the command sequencer because hav
Pierre-Luc did an amazing job in getting
https://builds.cloudstack.org/ working [1].
We can work on similar lines and probably get
packages.cloudstack.org working with mirrored entries form
apt-get.eu or openvm.eu or s3.downloads.accelerite or
packages.shapeblue etc.
[1]
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1768
Thanks everyone. merging this 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
Github user karuturi closed the pull request at:
https://github.com/apache/cloudstack/pull/1977
---
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
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1977
PR #1975 has the same fix. closing this in favour of #1975 as jenkins
already passed there.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1977
Thanks for the review anshul.
---
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
FYI,
master is broken due to a merge conflict.
I created PR https://github.com/apache/cloudstack/pull/1977 to
fix the same.
Will merge it ASAP.
~ Rajani
http://cloudplatform.accelerite.com/
GitHub user karuturi opened a pull request:
https://github.com/apache/cloudstack/pull/1977
fixed the merge conflict on merging #1941
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/Accelerite/cloudstack build-fix
Alternatively
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1976
This is not going to fix the build issue. The issue is different. I will
open a new PR.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user karuturi closed the pull request at:
https://github.com/apache/cloudstack/pull/1976
---
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
GitHub user karuturi opened a pull request:
https://github.com/apache/cloudstack/pull/1976
Revert "Merge pull request #1829 from syed/hvm-volume-attach-restart-fix"
This reverts commit d9bd01266ffb23f591283c058beff6e65d1cc4aa, reversing
ch
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1829
Both #1829 and #1941 are merged and is causing build failures. I am
creating new PR with the revert.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1768
Actually, BVT is not going to verify this as this is db upgrade related and
travis would have tested it.
I am merging this.
---
If your project is set up for it, you can reply
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1257
after code changes, suresh requested for another test run on vmware.
@rhtyd @borisroman can you run vmware tests?
---
If your project is set up for it, you can reply to this email and have
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1779
@priyankparihar can you run internal CI and post @cloudmonger test results?
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1813
@borisstoyanov can you post test results?
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1302
@blueorangutan @rhtyd @borisroman vmware tests please
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1241
@rhtyd @borisstoyanov can you run tests on 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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1873
@ustcweizhou can you review the updated 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
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1927
merging this.
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1768
@DaanHoogland Can you post test results?
---
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
che/cloudstack/pull/1945
> >
> > <https://github.com/apache/cloudstack/pull/1945>https://
> > github.com/apache/cloudstack/pull/1946
> >
> > <https://github.com/apache/cloudstack/pull/1946>https://
> &g
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/927
@serg38 Thanks for filtering the error :) My tests run has been with
encryption disabled. I think trillion runs with db encryption enabled. I will
check on how I can decrypt the value in tests
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/838
good fix. merging.
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/865
merging
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1961
@nvazquez Are you looking at the test failures?
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1896
@milamberspace Should the merge to master be an empty merge? Meaning is
this only for 4.9??
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1896
@rhtyd missed this one. Thanks for reminding. merging now.
@milamberspace Thanks for keeping the translations updated for every
release :)
---
If your project is set up for it, you can
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1841
@sateesh-chodapuneedi can you take a look at the test failures as well?
---
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
6 days to go for the first RC.
Please get the required lgtms and tests ready for the PRs you like to see
in 4.10.
I will try to merge all the PRs that meet the criteria.
Thanks,
~Rajani
On 19 Feb 2017 3:11 p.m., "Rajani Karuturi" <raj...@apache.org> wrote:
> noted.
>
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1926
LGTM. merging
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1900
@anshul1886 can you take a look at the travis and jenkins failures?
volume related tests failed on travis
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1924
merging
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1847
Hi @anshul1886, I just observed that the related PR #1735 was for 4.9.
Since this bug also exists in 4.9, can you change the base branch of this PR to
4.9 and rebase?
---
If your project
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1952
we have review from @ustcweizhou and @mike-tutkowski tested the fix.
merging this now.
Thanks you all.
---
If your project is set up for it, you can reply to this email and have your
00 Tutkowski, Mike
<mike.tutkow...@netapp.com>:
Hopefully we can now find someone with experience in CloudStack's
networking code to tell us if that fix is OK or if that was just
a
Band-Aid
and we should do something more thorough to correct the issue.
On Feb 16, 2017, at 11:48 PM, Rajani Kar
>
wrote:
Not sure, I will see if I can find some time tomorrow to look at
this.
Thanks...
*Will STEVENS*
Lead Developer
<https://goo.gl/NYZ8KK>
On Mon, Feb 13, 2017 at 11:58 PM, Rajani Karuturi
<raj...@apache.org>
wrote:
Thanks Will. I will take a look at this today and merge.
Is i
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1838
merging
---
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
(rohit.ya...@shapeblue.com) wrote:
Jenkins need to have jdk8 available, someone need to setup jenv
on it as well.
(The first job in Travis does apidocs/marvin/rat related checks
to validate changes and apidocs build).
Regards.
From: Rajani Karuturi <
pache/cloudstack/pull/1579/files
On 2/16/17, 2:42 AM, "Wido den Hollander" <w...@widodh.nl>
wrote:
> Op 16 februari 2017 om 6:21 schreef Rajani Karuturi
<raj...@apache.org>:
>
>
> unfortunately, running one set of tests itself is very time
> consuming(multiplie
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1847
oops.. will wait for travis
---
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
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1847
merging
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1727
merging
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1913
merging
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1927
yes, my question was about testing this PR in basic zone. 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
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1927
also, as @wido said, this happens only when ipv6 is set. So, its not a
blocker. I will wait and see if someone can run basic network BVT.
@wido, Will running BVT verify anything
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1927
@rhtyd, Can @blueorangutan run basic network tests?
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1834
merging
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1727
@nvazquez Can you resolve conflicts? It might be due to the merge of your
other PR #1747 :)
Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1747
merging
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1913
@borisstoyanov @rhtyd @DaanHoogland Can you start vmware tests on 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
Github user karuturi commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1913#discussion_r101452884
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/storage/resource/VmwareStorageProcessor.java
---
@@ -1577,11 +1577,15 @@ public Answer
This is the order in which they are merged.
* 7233ac3 N Merge pull request #977 from ustcweizhou/vm-snapshot
2 weeks ago [Rajani Karuturi]
* f10c8bf N Merge pull request #1700 from
wido/ipv6-basic-networking 3 weeks ago [Rajani Karuturi]
* 4721c53 N Merge pull request #1749 from
mike-tutkowski
FYI,
With strongswan[1] and java8[2] changes, a new systemvm template
is required on master.
Latest systemvm template is available at
https://builds.cloudstack.org/job/build-master-systemvm/lastSuccessfulBuild/
Thank you Pierre-Luc Dion for creating the build :)
[1]
unfortunately, running one set of tests itself is very time
consuming(multiplied by hypervisors, network modes etc. its
impossible to run all)
If I remember correctly, wido tested it with PR 1700 (IPV6 in
basic networking)
Can you try a commit before and a commit after that PR?
Thanks,
~
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1818
@ustcweizhou agreed. I am merging this PR. I added your comment to the
description of the bug(so that the bug is not closed and can be taken up later)
I am merging this for now. Thanks
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1818
@ustcweizhou mm.. I am confused now :) a new button definitely makes sense
but, a better error message doesn't worsen the current situation. So, until we
have new button, we can have
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1939
merging
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1741
It has required LGTMs and tests. I am merging this.
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1881
configuration cleanup. needs BVT
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1882
unused class is removed. needs BVT
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1818
@ustcweizhou in the above snippet, its adding the zoneid if its not
cross-zone. if no zoneid is provided, it will default to -1 which means cross
zone. So, this check is required
tack/pull/1929
2 LGTM
CI green
On 07/02/2017 13:07, Rajani Karuturi wrote:
Hi all,
If you know of any PRs which meet the below criteria, please
mail
them to
me.
1. No open questions.
2. Has two LGTMs
3. Has CI test results.
~Rajani
http://cloudplatform.accelerite.com
--
Daan
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1938
good point @swill. Thanks @borisstoyanov.
Since the fix is no different from the earlier version for 1.7, I am
assuming it should work for 1.8 as well on ubuntu.
merging this now
101 - 200 of 1399 matches
Mail list logo