[GitHub] cloudstack pull request: CLOUDSTACK-8486: Refactoring LibVirt (KVM...

[bhaisaab]

Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/266#issuecomment-103812423
  
LGTM, thanks @wilderrodrigues loved the reports :)


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: Cloudstack-8301

[bhaisaab]

Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/263#issuecomment-103814374
  
Thanks @koushik-das for your reply and the fix, can you share any test 
results? 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 the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: Pep8 and some unit tests

[bhaisaab]

Github user bhaisaab commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/265#discussion_r30681931
  
--- Diff: systemvm/patches/debian/config/opt/cloud/bin/configure.py ---
@@ -555,35 +558,47 @@ def forward_vr(self, rule):
 self.portsToString(rule['internal_ports'], '-')
   )
 fw4 = "-j SNAT --to-source %s -A POSTROUTING -s %s -d %s/32 -o %s 
-p %s -m %s --dport %s" % \
-  ( self.getGatewayByIp(rule['internal_ip']),
+  (
+self.getGatewayByIp(rule['internal_ip']),
 self.getNetworkByIp(rule['internal_ip']),
 rule['internal_ip'],
 self.getDeviceByIp(rule['internal_ip']),
 rule['protocol'],
 rule['protocol'],
 self.portsToString(rule['internal_ports'], ':')
-  )  
+  )
 fw5 = "-A PREROUTING -d %s/32 -i %s -p %s -m %s --dport %s -j MARK 
--set-xmark %s/0x" % \
-  ( rule['public_ip'],
+  (
+rule['public_ip'],
 self.getDeviceByIp(rule['public_ip']),
 rule['protocol'],
 rule['protocol'],
 self.portsToString(rule['public_ports'], ':'),
 hex(int(self.getDeviceByIp(rule['public_ip'])[3:]))
-  )  
+  )
 fw6 = "-A PREROUTING -d %s/32 -i %s -p %s -m %s --dport %s -m 
state --state NEW -j CONNMARK --save-mark --nfmask 0x --ctmask 
0x" % \
-  ( rule['public_ip'],
+  (
+rule['public_ip'],
 self.getDeviceByIp(rule['public_ip']),
 rule['protocol'],
 rule['protocol'],
 self.portsToString(rule['public_ports'], ':'),
-  )  
+  )
+fw7 = "-A FORWARD -i %s -o %s -p %s -m %s --dport %s -m state 
--state NEW -j ACCEPT" % \
--- End diff --

@isoutham this PR says pep8/unit test fixes, why are we adding a forwarding 
rule here?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: CLOUDSTACK-8486: Refactoring LibVirt (KVM...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/266


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: Allow forward to fix port forwarding rule...

[bhaisaab]

Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/262#issuecomment-103815052
  
@isoutham I'm confused to merge which one since you said in the other PR to 
ignore a previous PR. Can you ask @wilderrodrigues to merge it? 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 project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: Allow forward to fix port forwarding rule...

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/262#issuecomment-103815511
  
@bhaisaab 

I'm on it... :)

Already got the PR and will get it merged in a bit.

Cheers,
Wilder


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: Build and create tomcat, fix several othe...

[rsafonseca]

Github user rsafonseca commented on the pull request:

https://github.com/apache/cloudstack/pull/238#issuecomment-103820696
  
Hello @wilderrodrigues :)
I guess the status of this PR is that it was waiting for me to fix the 
build issues... i was away since last wednesday and just got back yesterday, so 
i hadn't had time to look into it until last night.

The OVM3 failure in jenkins is indeed unrelated to my changes, it seems 
it's failing for a few PRs whenever it gets built on specific jenkins 
servers... probably due do different jenkins configs.

About the testing, i had only run the existing JUnit tests and did some 
manual testing on the mentioned platforms (service startup/config issues, 
tomcat performance/startup and java8 supportability for some things that were 
failing) and did not test the whole platform.. i still didn't have a look at 
Marvin, i hope i'll have time to start playing with it over the next week :)

@bhaisaab all fixed!




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: Pep8 and some unit tests

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/265#issuecomment-103829579
  
@bhaisaab 

For some reason this PR got the same commit of the previous PR (#262). 
@isoutham told me to ignore the previous one.

So I'm now testing this and it LGTM so far. I will just test redundant VPC 
and merge it. Below few details of the tests I have done manually:

* Create VPC
* Create Tier
* Create ACL + 2 rules
* Associate ACL with Tier
* Create VM
* Acquire new IP + Add PF rule

Then ping + ssh

[wrodrigues@mct-wrodrigues cloudstack]$ ping 192.168.23.8
PING 192.168.23.8 (192.168.23.8) 56(84) bytes of data.
64 bytes from 192.168.23.8: icmp_seq=1 ttl=64 time=3.24 ms
64 bytes from 192.168.23.8: icmp_seq=2 ttl=64 time=3.76 ms
64 bytes from 192.168.23.8: icmp_seq=3 ttl=64 time=3.96 ms
64 bytes from 192.168.23.8: icmp_seq=4 ttl=64 time=4.08 ms
^C
--- 192.168.23.8 ping statistics ---
4 packets transmitted, 4 received, 0% packet loss, time 3004ms
rtt min/avg/max/mdev = 3.241/3.762/4.080/0.330 ms
[wrodrigues@mct-wrodrigues cloudstack]$ ssh root@192.168.23.8
The authenticity of host '192.168.23.8 (192.168.23.8)' can't be established.
ECDSA key fingerprint is 3f:64:cc:0f:72:09:9a:e4:dd:c3:1b:a0:0c:17:f7:db.
Are you sure you want to continue connecting (yes/no)? yes
Warning: Permanently added '192.168.23.8' (ECDSA) to the list of known 
hosts.
root@192.168.23.8's password: 



---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: CLOUDSTACK-8487 : Add vMotion related tes...

[abhinavroy02]

GitHub user abhinavroy02 opened a pull request:

https://github.com/apache/cloudstack/pull/268

CLOUDSTACK-8487 : Add vMotion related tests

Adding a new test script testpath_vMotion_vmware.py in the 
test/integration/testpath folder. 
This script has vMotion related test cases for VMware.
Tests include :

1. Migrate VM with volume within/across the cluster both for vmfs and nfs 
datastores, windows and linux vms.
2. Migrate VM with volume within/across cluster for local storage.
3. Migrate across cwps and zwps.
4. Migrate across nfs and vmfs.
5. Negative scenarios
6. Migration tests when host is put in maintenance.
7. Migration tests when storage is put in maintenance.
-
Have also added 2 new functions in base.py

This scripts have been tested on my local setup.
Also, I have run BVTs with this code on my local setup and it doesn't break 
anything.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/abhinavroy02/cloudstack CLOUDSTACK-8487

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/268.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #268


commit 5de4b6898072688351efb9d1ebb3e0e9ea99b702
Author: Abhinav Roy 
Date:   2015-05-20T10:37:03Z

CLOUDSTACK-8487 : Add vMotion related 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 not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: Pep8 and some unit tests

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/265


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: Allow forward to fix port forwarding rule...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/262


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: CLOUDSTACK-8487 : Add vMotion related tes...

[gauravaradhye]

Github user gauravaradhye commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/268#discussion_r30691290
  
--- Diff: tools/marvin/marvin/lib/base.py ---
@@ -2520,6 +2520,35 @@ def reconnect(cls, apiclient, **kwargs):
 [setattr(cmd, k, v) for k, v in kwargs.items()]
 return(apiclient.reconnectHost(cmd))
 
+@classmethod
+def getState(cls, apiclient, hostid, state, resourcestate, 
timeout=600):
+"""List Host and check if its resource state is as expected
+@returnValue - List[Result, Reason]
+   1) Result - FAIL if there is any exception
+   in the operation or Host state does not change
+   to expected state in given time else PASS
+   2) Reason - Reason for failure"""
+
+returnValue = [FAIL, "VM state not trasited to %s,\
+operation timed out" % state]
+
+while timeout > 0:
+try:
+hosts = Host.list(apiclient, 
+  id=hostid, listall=True)
+validationresult = validateList(hosts)
+if validationresult[0] == FAIL:
+raise Exception("Host list validation failed: %s" % 
validationresult[2])
--- End diff --

Fix pep8 issues


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: CLOUDSTACK-8487 : Add vMotion related tes...

[gauravaradhye]

Github user gauravaradhye commented on the pull request:

https://github.com/apache/cloudstack/pull/268#issuecomment-103848262
  
Don't know why I can't comment on the testpath file. I don't see the file 
contents in the changes. Any idea why?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: CLOUDSTACK-8487 : Add vMotion related tes...

[abhinavroy02]

Github user abhinavroy02 commented on the pull request:

https://github.com/apache/cloudstack/pull/268#issuecomment-103851151
  
Hi Gaurav,

Even I am not getting that option. Is it because it is a new file or
something?

Thanks,
Abhinav
On May 20, 2015 16:54, "Gaurav Aradhye"  wrote:

> Don't know why I can't comment on the testpath file. I don't see the file
> contents in the changes. Any idea why?
>
> —
> Reply to this email directly or view it on GitHub
> .
>



---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[CLOUDSTACK DAYS SEATTLE] Call for Volunteers

[Likitha Shetty]

Hello CloudStack committers,

If you would like to volunteer to help review proposals submitted for 
CloudStack Day Seattle and are a CloudStack Committer, please reach out to me 
or Karen (cc'd). We can start reviewing the submissions post the CFP deadline 
which is June 12th .

Thanks,
Likitha

[GitHub] cloudstack pull request: Fixed blocker issues reported by sonarqub...

[karuturi]

GitHub user karuturi opened a pull request:

https://github.com/apache/cloudstack/pull/269

Fixed blocker issues reported by sonarqube in js files

All of them are trailing comma in array or object

more details @

https://analysis.apache.org/component_issues?id=org.apache.cloudstack%3Acloudstack#resolved=false|severities=BLOCKER|languages=js

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/karuturi/cloudstack sonarqube

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/269.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #269


commit 8571314406cb396dbe18eae1ad8da40884e6888b
Author: Rajani Karuturi 
Date:   2015-05-20T11:47:09Z

Fixed blocker issues reported by sonarqube in js files

All of them are trailing comma in array or object

more details @

https://analysis.apache.org/component_issues?id=org.apache.cloudstack%3Acloudstack#resolved=false|severities=BLOCKER|languages=js




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: analysis.apache.org

[Rajani Karuturi]

I fixed js issues in https://github.com/apache/cloudstack/pull/269

~Rajani

On Wed, May 20, 2015 at 3:06 AM, Erik Weber  wrote:

> On Tue, May 19, 2015 at 9:42 PM, Daan Hoogland 
> wrote:
>
> > H fellow devs,
> >
> > It has come to my attentions that after more then eight months the sonar
> > install at apache now does a daily analysis of cloudstack again.:) There
> > are 26 issues stamped 'Blocker' and a lot more fun in the lower
> categories.
> >
> > Shal I make issues per sonar 'Blocker'? I propose to make them critical
> and
> > not blocker.
> >
>
>
> I guess the javascript ones could be put in a common issue, as they are
> more or less similar.
>
> --
> Erik
>

[GitHub] cloudstack pull request: Fixed blocker issues reported by sonarqub...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/269#issuecomment-103860194
  
LGTM (less gomma's tis more:)


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: Fixed blocker issues reported by sonarqub...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/269


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: analysis.apache.org

[Daan Hoogland]

thanks guys and girl,

I saw you fixed a similar issue in the api Laszlo. I couldn't quite
translate it to the one I looked at. I will start creating issues for the
old ones. They are indeed mostly more then 3 years old.
Merged your pr, Rajani

Op wo 20 mei 2015 om 13:51 schreef Rajani Karuturi :

> I fixed js issues in https://github.com/apache/cloudstack/pull/269
>
> ~Rajani
>
> On Wed, May 20, 2015 at 3:06 AM, Erik Weber  wrote:
>
> > On Tue, May 19, 2015 at 9:42 PM, Daan Hoogland 
> > wrote:
> >
> > > H fellow devs,
> > >
> > > It has come to my attentions that after more then eight months the
> sonar
> > > install at apache now does a daily analysis of cloudstack again.:)
> There
> > > are 26 issues stamped 'Blocker' and a lot more fun in the lower
> > categories.
> > >
> > > Shal I make issues per sonar 'Blocker'? I propose to make them critical
> > and
> > > not blocker.
> > >
> >
> >
> > I guess the javascript ones could be put in a common issue, as they are
> > more or less similar.
> >
> > --
> > Erik
> >
>

[GitHub] cloudstack pull request: CLOUDSTACK-8486

[wilderrodrigues]

GitHub user wilderrodrigues opened a pull request:

https://github.com/apache/cloudstack/pull/270

CLOUDSTACK-8486

@bhaisaab and @DaanHooglan, there goes another one.

It has only unit tests changes, so not a big deal with integration tests. 
The Libvirt tests are running in 2.889 sec in the KVM environment, instead of  
257.879 sec

Removing real IPs from the tests because they cause a long running time for 
LibvirtComputingResourceTest
- In a local machine it takes 1.977s, but in a KVM test environment 
it's taking 257.879 sec

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/schubergphilis/cloudstack 
fix/libvirt_unittests_timeout

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/270.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #270


commit 982a0235a0143864e5b0eda1836fb913d91252f6
Author: wilderrodrigues 
Date:   2015-05-20T12:52:08Z

CLOUDSTACK-8486

  Removing real IPs from the tests because they cause a long running time 
for LibvirtComputingResourceTest
- In a local machine it takes 1.977s, but in a KVM test environment 
it's taking 257.879 sec




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: ListFirewallEgressRulesCmd: extend from B...

[resmo]

Github user resmo commented on the pull request:

https://github.com/apache/cloudstack/pull/249#issuecomment-103890479
  
@bhaisaab a good IDE helps indeed.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: CLOUDSTACK-8486

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/270


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack-cloudmonkey pull request: signatureversion and backward...

[ntavares]

GitHub user ntavares opened a pull request:

https://github.com/apache/cloudstack-cloudmonkey/pull/5

signatureversion and backward compatibility

Provide signatureversion as config.option for backward compatibility;
Follow specification: only provide signatureversion=&expires= in URL if 
signatureversion=3

http://docs.cloudstack.apache.org/en/latest/dev.html?highlight=signatureversion

PS: We want signatureversion to be an integer

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ntavares/cloudstack-cloudmonkey master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack-cloudmonkey/pull/5.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #5


commit 7a5052331a6ed40496995dcd43a82da2b86920d3
Author: Nuno Tavares 
Date:   2015-05-15T10:27:27Z

In the specific case of a "create networkoffering" with no services, the 
parameter 'supportedservices' gets stripped off because it's empty. However, 
the API requires it as mandatatory - and yes, the UI allows you to create a 
n.o. with no services.

commit 32229ea88d7f156ebc79ef1d3c67793669bd50c1
Author: Nuno Tavares 
Date:   2015-05-20T14:40:35Z

Provide signatureversion as config.option for backward compatibility;

Follow specification: only provide signatureversion=&expires= in URL if 
signatureversion=3

http://docs.cloudstack.apache.org/en/latest/dev.html?highlight=signatureversion

commit d0cce546f6170a4a7549bb6855e197a5ef668924
Author: Nuno Tavares 
Date:   2015-05-20T14:40:35Z

Provide signatureversion as config.option for backward compatibility;

Follow specification: only provide signatureversion=&expires= in URL if 
signatureversion=3

http://docs.cloudstack.apache.org/en/latest/dev.html?highlight=signatureversion

PS: We want signatureversion to be an integer

commit 1a934965349c814ba774b6002dce8f0ebad3c1fd
Author: Nuno Tavares 
Date:   2015-05-20T15:08:19Z

Merge branch 'master' of https://github.com/ntavares/cloudstack-cloudmonkey

Conflicts:
cloudmonkey/requester.py




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack-cloudmonkey pull request: signatureversion and backward...

[bhaisaab]

Github user bhaisaab commented on the pull request:


https://github.com/apache/cloudstack-cloudmonkey/pull/5#issuecomment-103931257
  
@ntavares looks good to me, though it fails to apply on master. Can you 
please rebase against master and send a patch that would apply cleanly? 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 project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: ListFirewallEgressRulesCmd: extend from B...

[bhaisaab]

Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/249#issuecomment-103934999
  
@resmo looks good to me, Travis is green too. Will merge tonight or 
tomorrow after some build tests. 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 project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: ListFirewallEgressRulesCmd: extend from B...

[bhaisaab]

Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/249#issuecomment-103938699
  
Builds fine, 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, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: ListFirewallEgressRulesCmd: extend from B...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/249


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack-cloudmonkey pull request: signatureversion and backward...

[ntavares]

Github user ntavares commented on the pull request:


https://github.com/apache/cloudstack-cloudmonkey/pull/5#issuecomment-103950328
  
@bhaisaab What a mess ahah... My branch was fresh, I believe I last apache' 
commit was from my pull request.. not sure what happened here and it got worse 
when I tried to rebase... maybe it was OK the first time, I'm not an 
experienced git user... can you try again, 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 if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack-cloudmonkey pull request: signatureversion and backward...

[ntavares]

Github user ntavares commented on the pull request:


https://github.com/apache/cloudstack-cloudmonkey/pull/5#issuecomment-103955301
  
Hopefully this is properly rebased 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 if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[SDN] Status of the OpenDayLight plugin

[Wilder Rodrigues]

Hi all,

I was wondering which is the current status of the ODL plugin. Anyone working 
on that?

If so, please let me know.

Cheers,
Wilder

Sent from my iPhone

Re: [SDN] Status of the OpenDayLight plugin

[Erik Weber]

On Wed, May 20, 2015 at 8:31 PM, Wilder Rodrigues <
wrodrig...@schubergphilis.com> wrote:

> Hi all,
>
> I was wondering which is the current status of the ODL plugin. Anyone
> working on that?
>
> If so, please let me know.
>
>
I asked the same in April, and judging from the response I don't think
anyone is working on it.
This is the merge message:
http://permalink.gmane.org/gmane.comp.apache.cloudstack.devel/35692


-- 
Erik

Re: [SDN] Status of the OpenDayLight plugin

[Sebastien Goasguen]

Wilder, you should ask Hugo.

> On May 20, 2015, at 8:36 PM, Erik Weber  wrote:
> 
> On Wed, May 20, 2015 at 8:31 PM, Wilder Rodrigues <
> wrodrig...@schubergphilis.com> wrote:
> 
>> Hi all,
>> 
>> I was wondering which is the current status of the ODL plugin. Anyone
>> working on that?
>> 
>> If so, please let me know.
>> 
>> 
> I asked the same in April, and judging from the response I don't think
> anyone is working on it.
> This is the merge message:
> http://permalink.gmane.org/gmane.comp.apache.cloudstack.devel/35692
> 
> 
> -- 
> Erik

[GitHub] cloudstack pull request: CLOUDSTACK-8231: Fixed UI empty drop-down...

[vadimkim]

GitHub user vadimkim opened a pull request:

https://github.com/apache/cloudstack/pull/271

CLOUDSTACK-8231: Fixed UI empty drop-down list for LB rules

It seems that getLbAlgorithms() function must be called if context is 
"undefined".  If context is defined then function returns empty list. That is 
why drop-down menu is also empty when I add new LB rule.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/vadimkim/cloudstack master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/271.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #271


commit 7ee9b1042f35a2998217d973b5ee447ca68497fb
Author: Vadim Kimlaychuk 
Date:   2015-05-20T19:08:29Z

CLOUDSTACK-8231: Fixed UI empty drop-down list for LB rules




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: CLOUDSTACK-8231: Fixed UI empty drop-down...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/271


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: CLOUDSTACK-8231: Fixed UI empty drop-down...

[bhaisaab]

Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/271#issuecomment-104026293
  
LGTM. Thanks @vadimkim 


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: [WWW] Website now managed via Git

[Jan-Arve Nygård]

2015-05-13 10:06 GMT+02:00 sebgoa :

>
> On May 13, 2015, at 9:37 AM, Jan-Arve Nygård 
> wrote:
>
> > I created a PR to add committers that fell out of the list during the
> move.
> > I'm not sure if you want non committers to build with middleman and then
> do
> > a PR so i just created a PR against the source for the next build until
> > instructions are ready.
> >
>
> Merged.
>
> Thanks a lot.
>
> Feel free to improve the site ….it needs love :)
>
>
There are some PRs waiting if you haven't noticed. Nothing big, but the 404
should be fixed.
It also seems there's no e-mail notifications to the mailinglist for this
repo?

I will try to look through the site a bit more thoroughly later :)

[ACS44]release 4.4.4?

[Daan Hoogland]

H,

Are there any fixes that people have that should go into 4.4? I would like
to create a release candidate over the weekend.