Thanks Jayapal!
I won't propose this change as a pull request since this is a pretty custom job.
Myipset (with a different name) will include all our data centre (AS) subnets,
the end result being that with a simple "iptables-save -c" I can now know what
traffic was done against our data centre
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1460#issuecomment-205705013
@jburwell the failure is not in rat but in the unit tests for the secondary
storage controller project
---
If your project is set up for it, you can reply to
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1459#issuecomment-205715741
the other one is persistant i'm afraid:
[1459.results.vpc_redundant.txt](https://github.com/apache/cloudstack/files/204056/1459.results.vpc_redundant.txt)
Github user dsclose commented on the pull request:
https://github.com/apache/cloudstack/pull/1413#issuecomment-205716220
@wilderrodrigues @ustcweizhou Sorry to arrive late to the party but this
appears to break/ignore some RvR functionality. 4 things in particular stand
out:
Hi Wei,
Have you noticed the PR #1413 at https://github.com/apache/cloudstack/pull/1413
? The commit at
https://github.com/apache/cloudstack/commit/8bbea5eeb6598b213b78a324c184841dbba69280
changes the check_is_up method considerably such that your patch would
no-longer apply.
That PR #1413 fo
Hi Daan,
Yes, this should run only in Virtual Router. Instead of moving it to existing
“if” block, better to have a seperate check only for normal router, since we
don’t support baremetal deployment in VPC.
Regarding the later question, with the current implementation I don’t think we
can dist
> Op 4 april 2016 om 9:28 schreef Daan Hoogland :
>
>
> H, for those interested in how we get to improve, here is something I
> stumbled upon on reddit [1]
>
> I must admit I think most of us are already working as described but
> still,... it touches on some of the pitfalls we have been trippi
thakns Hari, I'll make a small pr for putting the baremetal-vr.py under a
condition.
On Tue, Apr 5, 2016 at 11:10 AM, Harikrishna Patnala <
harikrishna.patn...@accelerite.com> wrote:
> Hi Daan,
>
> Yes, this should run only in Virtual Router. Instead of moving it to
> existing “if” block, better
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1397#issuecomment-205755506
@bhaisaab I'm kind of confused, you write about three parts but I count
four. I would agree that the bouncy castle is a completely seperate one. I
think mvn ch
Github user GabrielBrascher commented on the pull request:
https://github.com/apache/cloudstack/pull/1459#issuecomment-205755861
Thanks @DaanHoogland, I will check on that.
---
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 DaanHoogland opened a pull request:
https://github.com/apache/cloudstack/pull/1463
CLOUDSTACK-9336 surround the execution of baremetal-vr.py with condition
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/DaanHoogland/
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1463#issuecomment-205763867
integration tests pending
---
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 sanju1010 opened a pull request:
https://github.com/apache/cloudstack/pull/1464
[CLOUDSTACK-9337]Enhance vcenter.py to created data center in vcenter
server automatically
These changes have been made to support vmware deployments in CI.
For CI to create cloudstack s
Github user abhinandanprateek commented on the pull request:
https://github.com/apache/cloudstack/pull/1454#issuecomment-205780575
Updated marvin test to use the builtin template. @swill
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
sorry for not giving prior notice. I am updating plugins on
jenkins.buildacloud.org. It will restart.
--
Daan
Github user kiwiflyer commented on the pull request:
https://github.com/apache/cloudstack/pull/1413#issuecomment-205804779
@dsclose - just FYI. As far as I'm aware, this PR has not been committed
yet. @ustcweizhou has requested a new PR/rebase.
I just pulled these into our bu
Github user kollyma commented on the pull request:
https://github.com/apache/cloudstack/pull/1449#issuecomment-205809122
@Swill: tried to squash the commits shortly from holidays. Did not worked
as expected, messed up my PR. I need to debug this when I am back next week...
---
If you
Thank you, I'm going to raise a separate bug report about the issues I've found
with RvR setups. If I work on a patch I'll fork from the master branch unless I
hear otherwise.
Dean
-Original Message-
From: kiwiflyer [mailto:g...@git.apache.org]
Sent: 05 April 2016 14:28
To: dev@cloudst
Github user kiwiflyer commented on the pull request:
https://github.com/apache/cloudstack/pull/1413#issuecomment-205820963
@dsclose This bug exists as far back as 4.6 I believe, as that was when RvR
for VPC was introduced. So you might want base your patch on 4.7 and then it
can be pu
Github user dsclose commented on the pull request:
https://github.com/apache/cloudstack/pull/1413#issuecomment-205831656
Thank you. I've raised CLOUDSTACK-9339 on Jira about the non-VPC RvR
issues. As my company needs those issues solved in the short term I'll begin
working on a fix f
Github user nvazquez commented on the pull request:
https://github.com/apache/cloudstack/pull/1457#issuecomment-205857772
@rafaelweingartner cool! I added 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 projec
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/1397#issuecomment-205867477
@DaanHoogland feel free to split this however you feel, I've already moved
the Travis fix to a separate PR
---
If your project is set up for it, you can reply to
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1457#issuecomment-205882042
For me everything (with the code) is ok now, LGTM for the PR.
Could you please run functional tests?
---
If your project is set up for it, you can re
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1457#issuecomment-205883002
@rafaelweingartner I can run tests via KVM, but I don't have the ability to
run tests against VMware hardware. Are tests against KVM good enough in this
case?
---
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1454#issuecomment-205885703
@abhinandanprateek: Thank you. I will run the whole set of tests against
it again tonight. I have to let the current tests finish and run this on its
own because I
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1457#issuecomment-205885858
Good question, those changes affect every single hypervisor.
I have reviewed the code and after that I can say that if those changes
work for KVM, I s
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1454#issuecomment-205892559
@abhinandanprateek slight delay. I am going to blow away my CI setup and
reinstall it now to get it running on SSD drives so I can better parallelize my
tests. This
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1457#issuecomment-205892175
Perfect. Thank you both for verifying this. I probably won't be able to
get to testing this till tomorrow evening because I am about to reinstall my CI
to get my te
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1463#issuecomment-205909585
two failures. they seem unrelated but double ckecking
[1463.results.network.txt](https://github.com/apache/cloudstack/files/205002/1463.results.network.txt)
Github user nvazquez commented on the pull request:
https://github.com/apache/cloudstack/pull/1457#issuecomment-205917619
thanks @rafaelweingartner and @swill !
---
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 proj
Github user syed commented on the pull request:
https://github.com/apache/cloudstack/pull/1332#issuecomment-205922970
@bvbharatk it looks to be a problem with the test. Volume creation should
work irrespective of my changes.
---
If your project is set up for it, you can reply to thi
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1332#issuecomment-205926253
@syed can you squash the commits. This is best applied as a single commit.
Thx.
---
If your project is set up for it, you can reply to this email and have your
rep
Github user GabrielBrascher commented on the pull request:
https://github.com/apache/cloudstack/pull/1459#issuecomment-205932840
@rafaelweingartner now this commit is not cherry-picked from the
commit (b9181c689e0e7b5f1e28c81d73710196dfabd0ba). When this PR be merged
(or closed),
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1441#issuecomment-205937640
I will get CI run against this PR as soon as I have my CI reinstalled.
Thanks @mike-tutkowski
---
If your project is set up for it, you can reply to this email and
Github user mike-tutkowski commented on the pull request:
https://github.com/apache/cloudstack/pull/1441#issuecomment-205949475
Thanks, Will!
On Apr 5, 2016, at 12:35 PM, Will Stevens
mailto:notificati...@github.com>> wrote:
I will get CI run against this PR as s
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1459#issuecomment-205968363
Well, if this PR gets merged, I do not know much how we can close the PR
#561. Maybe @swill or @DaanHoogland can help with that.
Now the code LG
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1463#issuecomment-205968247
I replayed the functionality tested in the second test and it is working.
Will have to look into the test code to see what is happening.
retesting the
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1463#issuecomment-205975447
@harikrishna-patnala can you test this as well, somehow?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user syed commented on the pull request:
https://github.com/apache/cloudstack/pull/1332#issuecomment-205981012
Sure @swill . I've sqashed them
---
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 no
Saw ACP Doctor in CCP release notes from Accelerite.
Curious what it is, is it integrated into cloudstack or collection of
shell scripts?
Thanks
ilya
It used to be CCP Doctor and it is not in ACS from my understanding. It is
a set of scripts that will do basic validation of a CloudStack setup. It
does things like verify the system VMs are running and the connectivity is
working between all of the systems. It also does some checking to make
su
So this topic has been very slow moving, so I will keep pushing this along.
@Chris, are you on the ASF Infra team? From your perspective are we
missing anything to move forward?
@Sam & David, is there anything else I need to do from the ASF side other
than working out the logistics with the Inf
Hey Will, I’m not on the infra@ team, but I am a board member
and have provided by +1 and support (and I think there is general
support there informally).
However, I would point you at recent threads related to expanding
the MATT experiment that have surfaced on board@ in recent days
and I would r
Thank you. Will do.
On Apr 5, 2016 6:38 PM, "Chris Mattmann" wrote:
> Hey Will, I’m not on the infra@ team, but I am a board member
> and have provided by +1 and support (and I think there is general
> support there informally).
>
> However, I would point you at recent threads related to expandin
Github user alexandrelimassantana commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1464#discussion_r58633962
--- Diff: tools/marvin/marvin/lib/vcenter.py ---
@@ -183,7 +192,157 @@ def get_clusters(self, dc, clus=None):
"""
I don't believe this is freely available, rather is a tool Citrix developed
for helping troubleshoot CCP customer deployments. I would imagine that
Accelerite owns this tool now and it is likely available if you are an ACP
customer, but not necessarily for ACS users.
Probably best for Accelerite t
46 matches
Mail list logo
