[GitHub] cloudstack issue #2041: CLOUDSTACK-9873: VR related periodic jobs are schedu...

2017-04-12 Thread ustcweizhou
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 project

[GitHub] cloudstack issue #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-04-12 Thread jayakarteek
Github user jayakarteek commented on the issue: https://github.com/apache/cloudstack/pull/1918 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 wishes s

[GitHub] cloudstack issue #1836: [4.10/master] Smoketest Health

2017-04-12 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1836 Trillian test result (tid-992) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 6 Total time taken: 41826 seconds Marvin logs: https://github.com/blueoran

[GitHub] cloudstack issue #1756: CLOUDSTACK-9585 UI doesn't give an option to select ...

2017-04-12 Thread yvsubhash
Github user yvsubhash commented on the issue: https://github.com/apache/cloudstack/pull/1756 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 wishes so,

[GitHub] cloudstack issue #1836: [4.10/master] Smoketest Health

2017-04-12 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1836 Trillian test result (tid-993) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 34872 seconds Marvin logs: https://github.com/blueoranguta

[GitHub] cloudstack issue #2033: [master/4.10+] CLOUDSTACK-9462: Support for Ubuntu 1...

2017-04-12 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2033 Trillian test result (tid-991) Environment: kvm-ubuntu (x2), Advanced Networking with Mgmt server u Total time taken: 36466 seconds Marvin logs: https://github.com/blueorangutan

Re: represent the object in the logs

2017-04-12 Thread Rafael Weingärtner
I disagree with the idea of logging anything (increasing logging just by quantity), like that fallacy of big data era of more and more. If you want to find a needle in a haystack, you do not add more hay, you remove them. However, increasing the quality of data being logged, I am totally in favor!

[GitHub] cloudstack issue #2037: CLOUDSTACK-9871: Set SQL Mode in SQL Session for MyS...

2017-04-12 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2037 Trillian test result (tid-990) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 29008 seconds Marvin logs: https://github.com/blueoranguta

Re: [PROPOSAL] Apache Cloudstack should join the gitbox experiment

2017-04-12 Thread Pierre-Luc Dion
Yes, I saw people put a lot of efforts for it :-) On Apr 5, 2017 10:20, "Rohit Yadav" wrote: > +1 in addition to my support for joining the experiment, I would also like > us to have the option to assign labels on the PRs. Such labels can be > useful for CIs and RMs to determine and merge PRs th

Re: represent the object in the logs

2017-04-12 Thread Rene Moser
Hi Rafael On 04/12/2017 08:27 PM, Rafael Weingärtner wrote: > Well, I think the missing point to understand why we have these different > situations is the understanding of the developer’s mind. It is so diverse > and unique from people to people, and because we do not have a policy on > that, eac

[GitHub] cloudstack issue #2036: [CLOUDSTACK-9858] Retirement of midonet plugin (buil...

2017-04-12 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/2036 I tried to access the Jenkins details to see the checking problems, but I am getting a 404. Am I missing something? --- If your project is set up for it, you can reply to this em

Re: represent the object in the logs

2017-04-12 Thread Rafael Weingärtner
Well, I think the missing point to understand why we have these different situations is the understanding of the developer’s mind. It is so diverse and unique from people to people, and because we do not have a policy on that, each developer is doing the best they know and think. For instance, at

[GitHub] cloudstack issue #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA pr...

2017-04-12 Thread serverchief
Github user serverchief commented on the issue: https://github.com/apache/cloudstack/pull/1960 Hi @koushik-das I believe you missed a discussion on this a while back - when this was initially proposed and we were gathering community feedback. please read this thread

[GitHub] cloudstack issue #1836: [4.10/master] Smoketest Health

2017-04-12 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1836 @rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7 mgmt + vmware55u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests --- If your project is set up for it

[GitHub] cloudstack issue #1836: [4.10/master] Smoketest Health

2017-04-12 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1836 @blueorangutan test matrix --- 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 wi

[GitHub] cloudstack issue #1837: [4.9] Smoketest Health

2017-04-12 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1837 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-637 --- 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] cloudstack issue #1836: [4.10/master] Smoketest Health

2017-04-12 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1836 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-636 --- 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] cloudstack issue #2033: [master/4.10+] CLOUDSTACK-9462: Support for Ubuntu 1...

2017-04-12 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2033 @rhtyd a Trillian-Jenkins test job (ubuntu mgmt + kvm-ubuntu) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] cloudstack issue #2033: [master/4.10+] CLOUDSTACK-9462: Support for Ubuntu 1...

2017-04-12 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/2033 @blueorangutan test ubuntu kvm-ubuntu --- 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 ena

[GitHub] cloudstack issue #1837: [4.9] Smoketest Health

2017-04-12 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1837 @rhtyd 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 reply appe

[GitHub] cloudstack issue #1836: [4.10/master] Smoketest Health

2017-04-12 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1836 @rhtyd 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 reply appe

[GitHub] cloudstack issue #1836: [4.10/master] Smoketest Health

2017-04-12 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1836 @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 and wishes

[GitHub] cloudstack issue #1837: [4.9] Smoketest Health

2017-04-12 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1837 @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 and wishes

[GitHub] cloudstack issue #2037: CLOUDSTACK-9871: Set SQL Mode in SQL Session for MyS...

2017-04-12 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2037 @rhtyd 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 reply ap

[GitHub] cloudstack issue #2037: CLOUDSTACK-9871: Set SQL Mode in SQL Session for MyS...

2017-04-12 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/2037 @pdion891 I'll run regression tests @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

represent the object in the logs

2017-04-12 Thread Rene Moser
Hi In logs we see all kind variations how an object is printed to the users. E.g. for VMs we have seen, the DB ID, UUID, internal name, name etc. in different debug log messages. And this makes debugging harder than it should. I wondered, why don't we just use the "toString()" in VO classes. E.g

[GitHub] cloudstack issue #2037: CLOUDSTACK-9871: Set SQL Mode in SQL Session for MyS...

2017-04-12 Thread pdion891
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/2037 Do you know if this have an impact if we still use MySQL 5.6 or MySQL 5.5 on another host and have CloudStack installed on Ubuntu or CentOS 7 ? --- If your project is set up for it, you can re

[GitHub] cloudstack pull request #2041: CLOUDSTACK-9873: VR related periodic jobs are...

2017-04-12 Thread Slair1
GitHub user Slair1 opened a pull request: https://github.com/apache/cloudstack/pull/2041 CLOUDSTACK-9873: VR related periodic jobs are scheduled twice and run twice on management servers The "start" method of VirtualNetworkApplianceManagerImpl schedules several period checks, inclu

Re: growing #threads problem

2017-04-12 Thread Daan Hoogland
I found the bug that let me to encounter this behaviour. I’ll keep an eye on this after I fixed it (simple stupid NPE). I don’t think it is a big problem as the DirectAgentCronJobs are maximised to 500 by default. Still strange that it keeps growing in a rather silent environment though. On 11/

CloudStack related changes in Ansible 2.3

2017-04-12 Thread Rene Moser
Hi CloudStack users Ansible 2.3 is about to be released, I would like to summarize the CloudStack related features and changes in this release. New modules --- - cs_host - cs_nic - cs_region - cs_role - cs_vpc Examples and usage for these modules can be found in the docs, http://docs.a

[GitHub] cloudstack pull request #2040: CLOUDSTACK-9872: Gather all S2S vpn statuses ...

2017-04-12 Thread Slair1
GitHub user Slair1 opened a pull request: https://github.com/apache/cloudstack/pull/2040 CLOUDSTACK-9872: Gather all S2S vpn statuses before outputting The checkbatchs2svpn.sh VR script returns ("via echo") that status of each requested S2S VPN check one-at-a-time. If there is even

[GitHub] cloudstack issue #2037: CLOUDSTACK-9871: Set SQL Mode in SQL Session for MyS...

2017-04-12 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2037 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-635 --- 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] cloudstack issue #2036: [CLOUDSTACK-9858] Retirement of midonet plugin (buil...

2017-04-12 Thread wido
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/2036 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 feat

[GitHub] cloudstack pull request #2039: rbd: Use libvirt to create new volumes and no...

2017-04-12 Thread wido
GitHub user wido opened a pull request: https://github.com/apache/cloudstack/pull/2039 rbd: Use libvirt to create new volumes and not rados-java Since libvirt 1.2.2 libvirt will properly create volumes using RBD format 2. We can use libvirt to creates the volumes which s

[GitHub] cloudstack issue #2037: CLOUDSTACK-9871: Set SQL Mode in SQL Session for MyS...

2017-04-12 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2037 @rhtyd 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 reply appe

[GitHub] cloudstack issue #2037: CLOUDSTACK-9871: Set SQL Mode in SQL Session for MyS...

2017-04-12 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/2037 Lgtm @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

[GitHub] cloudstack issue #2037: CLOUDSTACK-9871: Set SQL Mode in SQL Session for MyS...

2017-04-12 Thread wido
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/2037 Done @rhtyd, how does this look like? --- 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 enab

[GitHub] cloudstack pull request #2038: ipv6: Allow all ICMPv6 traffic if -1 is provi...

2017-04-12 Thread wido
GitHub user wido opened a pull request: https://github.com/apache/cloudstack/pull/2038 ipv6: Allow all ICMPv6 traffic if -1 is provided as a ICMP type ip6tables no longer takes '--icmpv6-type any' as a argument. To allow all ICMPv6 traffic with ip6tables it has to be invoked

[GitHub] cloudstack issue #1996: CLOUDSTACK-9099: SecretKey is returned from the APIs

2017-04-12 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/1996 @koushik-das I have updated for your review comments. --- 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 h

[GitHub] cloudstack issue #2033: [master/4.10+] CLOUDSTACK-9462: Support for Ubuntu 1...

2017-04-12 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2033 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-634 --- 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

RE: How are router checks scheduled?

2017-04-12 Thread Sean Lair
The change to "/opt/cloud/bin/checkbatchs2svpn.sh" fixes the issues where no all of the VPN checks are returned. I'll create and issue and PR Sean -Original Message- From: Sean Lair Sent: Tuesday, April 11, 2017 2:33 PM To: dev@cloudstack.apache.org Subject: RE: How are router checks

[GitHub] cloudstack issue #2037: CLOUDSTACK-9871: Set SQL Mode in SQL Session for MyS...

2017-04-12 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/2037 LGTM @wido can you change the base branch to 4.9, to get this in 4.9 branch as well? Thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

[GitHub] cloudstack issue #1879: CLOUDSTACK-9719: [VMware] VR loses DHCP settings and...

2017-04-12 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1879 @karuturi thanks I'll see how I can help. This PR though looks like a bug to me, though not a blocker. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack issue #1980: CLOUDSTACK-9805: Display VR list in network details

2017-04-12 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1980 @karuturi this is a simple UI change with screenshots attached, and has enough LGTM. Please merge this. --- If your project is set up for it, you can reply to this email and have your reply appea

[GitHub] cloudstack issue #2034: [4.9+][network blocker] CLOUDSTACK-9838: Allow ingre...

2017-04-12 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/2034 Verified the mangle table firewall rules are now ACCEPT instead of RETURN. Pinging for review -- @wido @jayapalu @karuturi @abhinandanprateek @DaanHoogland /cc @PaulAngus --- If your project

[GitHub] cloudstack issue #2033: [master/4.10+] CLOUDSTACK-9462: Support for Ubuntu 1...

2017-04-12 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2033 @rhtyd 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 reply appe

[GitHub] cloudstack issue #2033: [master/4.10+] CLOUDSTACK-9462: Support for Ubuntu 1...

2017-04-12 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/2033 @ustcweizhou thanks, yes there is a minor difference around the init-helper version that's all @karuturi jenkins/travis re-kicked. @blueorangutan package --- If your project is set up fo

[GitHub] cloudstack pull request #2033: [master/4.10+] CLOUDSTACK-9462: Support for U...

2017-04-12 Thread rhtyd
Github user rhtyd closed the pull request at: https://github.com/apache/cloudstack/pull/2033 --- 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

[GitHub] cloudstack pull request #2033: [master/4.10+] CLOUDSTACK-9462: Support for U...

2017-04-12 Thread rhtyd
GitHub user rhtyd reopened a pull request: https://github.com/apache/cloudstack/pull/2033 [master/4.10+] CLOUDSTACK-9462: Support for Ubuntu 14.04/16.04 with tomcat6/tomcat7 This extends work from @ustcweizhou from https://github.com/apache/cloudstack/pull/1950 by fixing some build

[GitHub] cloudstack issue #2033: [master/4.10+] CLOUDSTACK-9462: Support for Ubuntu 1...

2017-04-12 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/2033 @karuturi done, 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 wishe

[GitHub] cloudstack issue #2035: CLOUDSTACK-9867:VM snapshot on primary storage usage...

2017-04-12 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2035 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-633 --- 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] cloudstack issue #2035: CLOUDSTACK-9867:VM snapshot on primary storage usage...

2017-04-12 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2035 @abhinandanprateek 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 you

[GitHub] cloudstack issue #2035: CLOUDSTACK-9867:VM snapshot on primary storage usage...

2017-04-12 Thread abhinandanprateek
Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/2035 @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 enable

Re: MySQL 5.7 and SQL Mode

2017-04-12 Thread Wido den Hollander
> Op 12 april 2017 om 10:39 schreef Erik Weber : > > > Instead of hacking mysql settings, wouldn't it be better to fix the query? > I agree completely. But CloudStack does a lot of SQL queries and I can't tell for sure which still work or which do not. For now I suggest to change MySQL back

Re: MySQL 5.7 and SQL Mode

2017-04-12 Thread Erik Weber
Instead of hacking mysql settings, wouldn't it be better to fix the query? -- Erik On Wed, Apr 12, 2017 at 9:56 AM, Wido den Hollander wrote: > >> Op 12 april 2017 om 7:23 schreef Koushik Das : >> >> >> Hi Wido, >> >> Check initDataSource() in TransactionLegacy.java. The connection properties

Re: MySQL 5.7 and SQL Mode

2017-04-12 Thread Wido den Hollander
> Op 12 april 2017 om 7:23 schreef Koushik Das : > > > Hi Wido, > > Check initDataSource() in TransactionLegacy.java. The connection properties > are read from db.properties. Thanks! After looking into this I found that you can just add this to db.cloud.url.params sessionVariables=sql_mode=

[GitHub] cloudstack pull request #2037: CLOUDSTACK-9871: Set SQL Mode in SQL Session ...

2017-04-12 Thread wido
GitHub user wido opened a pull request: https://github.com/apache/cloudstack/pull/2037 CLOUDSTACK-9871: Set SQL Mode in SQL Session for MySQL 5.7 compatibility MySQL 5.7 has a more strict SQL mode by default with which CloudStack is not compatible. By setting the SQL Mod

[GitHub] cloudstack pull request #1996: CLOUDSTACK-9099: SecretKey is returned from t...

2017-04-12 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1996#discussion_r111075302 --- Diff: api/src/org/apache/cloudstack/api/command/admin/user/GetUserKeysCmd.java --- @@ -0,0 +1,76 @@ +// Licensed to the Apache Software Foun

[GitHub] cloudstack pull request #1996: CLOUDSTACK-9099: SecretKey is returned from t...

2017-04-12 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1996#discussion_r111075076 --- Diff: api/src/org/apache/cloudstack/api/command/admin/user/GetUserKeysCmd.java --- @@ -0,0 +1,76 @@ +// Licensed to the Apache Software Foun

[GitHub] cloudstack pull request #1996: CLOUDSTACK-9099: SecretKey is returned from t...

2017-04-12 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1996#discussion_r111078382 --- Diff: server/src/com/cloud/api/ApiDBUtils.java --- @@ -559,6 +561,8 @@ @Inject private VpcManager vpcMgr; @Inject