[GitHub] blueorangutan commented on issue #2925: kvm: use libvirtd as service name for all distributions

2018-10-27 Thread GitBox
blueorangutan commented on issue #2925: kvm: use libvirtd as service name for all distributions URL: https://github.com/apache/cloudstack/pull/2925#issuecomment-433671621 Trillian test result (tid-3126) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time t

[GitHub] blueorangutan commented on issue #2928: Migrating VM from ISO failures

2018-10-27 Thread GitBox
blueorangutan commented on issue #2928: Migrating VM from ISO failures URL: https://github.com/apache/cloudstack/pull/2928#issuecomment-433658472 Trillian test result (tid-3123) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 23582 seconds Mar

[GitHub] blueorangutan commented on issue #2927: server: fix unwanted txn commit warning messages

2018-10-27 Thread GitBox
blueorangutan commented on issue #2927: server: fix unwanted txn commit warning messages URL: https://github.com/apache/cloudstack/pull/2927#issuecomment-433656944 Trillian test result (tid-3125) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 2

[GitHub] marcaurele commented on issue #2898: docker: add compose file for database

2018-10-27 Thread GitBox
marcaurele commented on issue #2898: docker: add compose file for database URL: https://github.com/apache/cloudstack/pull/2898#issuecomment-433655723 @rafaelweingartner / @wido / @DaanHoogland / @GabrielBrascher are you using docker for your development? --

[GitHub] marcaurele commented on a change in pull request #2898: docker: add compose file for database

2018-10-27 Thread GitBox
marcaurele commented on a change in pull request #2898: docker: add compose file for database URL: https://github.com/apache/cloudstack/pull/2898#discussion_r228726515 ## File path: tools/docker/mariadb-docker/cloudstack-docker.cnf ## @@ -0,0 +1,7 @@ +[mysqld] Review co

[GitHub] rhtyd commented on issue #2898: docker: add compose file for database

2018-10-27 Thread GitBox
rhtyd commented on issue #2898: docker: add compose file for database URL: https://github.com/apache/cloudstack/pull/2898#issuecomment-433654230 Thanks, LGTM @marcaurele we just need another review and then we can merge this.

[GitHub] rhtyd commented on a change in pull request #2898: docker: add compose file for database

2018-10-27 Thread GitBox
rhtyd commented on a change in pull request #2898: docker: add compose file for database URL: https://github.com/apache/cloudstack/pull/2898#discussion_r228726201 ## File path: tools/docker/mariadb-docker/cloudstack-docker.cnf ## @@ -0,0 +1,7 @@ +[mysqld] Review comment

[GitHub] marcaurele commented on issue #2898: docker: add compose file for database

2018-10-27 Thread GitBox
marcaurele commented on issue #2898: docker: add compose file for database URL: https://github.com/apache/cloudstack/pull/2898#issuecomment-433654100 pushed another file which is included in the compose to customize the configuration. ---

[GitHub] rhtyd commented on issue #2898: docker: add compose file for database

2018-10-27 Thread GitBox
rhtyd commented on issue #2898: docker: add compose file for database URL: https://github.com/apache/cloudstack/pull/2898#issuecomment-433652829 @marcaurele see install/admin docs, those are recommended mysql settings. The max mysql conn. settings has to do with max job/thread pool size lim

[GitHub] marcaurele commented on issue #2898: docker: add compose file for database

2018-10-27 Thread GitBox
marcaurele commented on issue #2898: docker: add compose file for database URL: https://github.com/apache/cloudstack/pull/2898#issuecomment-433652641 Are those the custom configuration made for CS? This is an automated message

[GitHub] rhtyd commented on issue #2898: docker: add compose file for database

2018-10-27 Thread GitBox
rhtyd commented on issue #2898: docker: add compose file for database URL: https://github.com/apache/cloudstack/pull/2898#issuecomment-433652542 @marcaurele OK, what about the innodb settings? ``` innodb_rollback_on_timeout=1 innodb_lock_wait_timeout=600 max_connections=1000

[GitHub] marcaurele commented on issue #2898: docker: add compose file for database

2018-10-27 Thread GitBox
marcaurele commented on issue #2898: docker: add compose file for database URL: https://github.com/apache/cloudstack/pull/2898#issuecomment-433652278 @rhtyd it's maridb **10.0** and not *10.X* which is equivalent to mysql 5.6, so there's no need for any extra configuration. It's working out

[GitHub] rhtyd commented on issue #2880: [4.11.2]: rolling upgrade of VR results in duplicate VRs

2018-10-27 Thread GitBox
rhtyd commented on issue #2880: [4.11.2]: rolling upgrade of VR results in duplicate VRs URL: https://github.com/apache/cloudstack/issues/2880#issuecomment-433651944 I'm still unable to reproduce, likely the case happens due to concurrency issue when the VR is stopped/started simultaneous

[GitHub] rhtyd commented on issue #2898: docker: add compose file for database

2018-10-27 Thread GitBox
rhtyd commented on issue #2898: docker: add compose file for database URL: https://github.com/apache/cloudstack/pull/2898#issuecomment-433651710 @marcaurele does mariadb 10.x work out of the box, for example mysql 5.7 needs the following config: ``` [mysqld] server_id = 1 s

[GitHub] marcaurele commented on a change in pull request #2898: docker: add compose file for database

2018-10-27 Thread GitBox
marcaurele commented on a change in pull request #2898: docker: add compose file for database URL: https://github.com/apache/cloudstack/pull/2898#discussion_r228724838 ## File path: tools/docker/docker-compose.yml ## @@ -0,0 +1,17 @@ +version: '2' Review comment: Sur

[GitHub] rhtyd commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7

2018-10-27 Thread GitBox
rhtyd commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7 URL: https://github.com/apache/cloudstack/pull/2896#issuecomment-433649345 Test env failed. Fix please @svenvogel Mgmt server log: ``` 2018-10-27 15:19:05,64

[GitHub] rhtyd opened a new issue #2935: Network GC error on master/4.12

2018-10-27 Thread GitBox
rhtyd opened a new issue #2935: Network GC error on master/4.12 URL: https://github.com/apache/cloudstack/issues/2935 On latest master/4.12, network GC mysql errors are seen: ``` 2018-10-27 15:17:05,819 WARN [o.a.c.e.o.NetworkOrchestrator] (Network-Scavenger-1:ctx-db281a94) (logi

[cloudstack] branch master updated: saml: redirect saml2 failed login message to a configurable URL (#2185)

2018-10-27 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/master by this push: new bfc3263 saml: redirect saml2 failed login me

[GitHub] rhtyd closed pull request #2185: CLOUDSTACK-9976: Redirect saml2 failed login message to a configurable URL

2018-10-27 Thread GitBox
rhtyd closed pull request #2185: CLOUDSTACK-9976: Redirect saml2 failed login message to a configurable URL URL: https://github.com/apache/cloudstack/pull/2185 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of p

[GitHub] rhtyd commented on issue #2185: CLOUDSTACK-9976: Redirect saml2 failed login message to a configurable URL

2018-10-27 Thread GitBox
rhtyd commented on issue #2185: CLOUDSTACK-9976: Redirect saml2 failed login message to a configurable URL URL: https://github.com/apache/cloudstack/pull/2185#issuecomment-433648326 Yes @DaanHoogland LGTM. Will merge based on travis results and past test results (none of the failures relat

[GitHub] blueorangutan commented on issue #2925: kvm: use libvirtd as service name for all distributions

2018-10-27 Thread GitBox
blueorangutan commented on issue #2925: kvm: use libvirtd as service name for all distributions URL: https://github.com/apache/cloudstack/pull/2925#issuecomment-433648140 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests ---

[GitHub] rhtyd commented on a change in pull request #2898: docker: add compose file for database

2018-10-27 Thread GitBox
rhtyd commented on a change in pull request #2898: docker: add compose file for database URL: https://github.com/apache/cloudstack/pull/2898#discussion_r228723628 ## File path: tools/docker/docker-compose.yml ## @@ -0,0 +1,17 @@ +version: '2' Review comment: @marcaur

[GitHub] rhtyd commented on issue #2925: kvm: use libvirtd as service name for all distributions

2018-10-27 Thread GitBox
rhtyd commented on issue #2925: kvm: use libvirtd as service name for all distributions URL: https://github.com/apache/cloudstack/pull/2925#issuecomment-433648059 @blueorangutan test This is an automated message from the Apac

[GitHub] blueorangutan commented on issue #2927: server: fix unwanted txn commit warning messages

2018-10-27 Thread GitBox
blueorangutan commented on issue #2927: server: fix unwanted txn commit warning messages URL: https://github.com/apache/cloudstack/pull/2927#issuecomment-433627204 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --

[GitHub] rhtyd commented on issue #2927: server: fix unwanted txn commit warning messages

2018-10-27 Thread GitBox
rhtyd commented on issue #2927: server: fix unwanted txn commit warning messages URL: https://github.com/apache/cloudstack/pull/2927#issuecomment-433627175 @rafaelweingartner any more objections? We would like to get this merged before RC4 is cut. @blueorangutan test --

[GitHub] blueorangutan commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7

2018-10-27 Thread GitBox
blueorangutan commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7 URL: https://github.com/apache/cloudstack/pull/2896#issuecomment-433627055 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run

[GitHub] blueorangutan commented on issue #2928: Migrating VM from ISO failures

2018-10-27 Thread GitBox
blueorangutan commented on issue #2928: Migrating VM from ISO failures URL: https://github.com/apache/cloudstack/pull/2928#issuecomment-433627052 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests -

[GitHub] rhtyd commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7

2018-10-27 Thread GitBox
rhtyd commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7 URL: https://github.com/apache/cloudstack/pull/2896#issuecomment-433626992 @blueorangutan test This is a

[GitHub] rhtyd commented on issue #2928: Migrating VM from ISO failures

2018-10-27 Thread GitBox
rhtyd commented on issue #2928: Migrating VM from ISO failures URL: https://github.com/apache/cloudstack/pull/2928#issuecomment-433627038 @blueorangutan test This is an automated message from the Apache Git Service. To respon

[GitHub] rhtyd commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7

2018-10-27 Thread GitBox
rhtyd commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7 URL: https://github.com/apache/cloudstack/pull/2896#issuecomment-433626979 @blueorangutan package This i

[GitHub] blueorangutan commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7

2018-10-27 Thread GitBox
blueorangutan commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7 URL: https://github.com/apache/cloudstack/pull/2896#issuecomment-433614900 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2389 -

[GitHub] blueorangutan commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7

2018-10-27 Thread GitBox
blueorangutan commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7 URL: https://github.com/apache/cloudstack/pull/2896#issuecomment-433613230 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make

[GitHub] rhtyd commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7

2018-10-27 Thread GitBox
rhtyd commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7 URL: https://github.com/apache/cloudstack/pull/2896#issuecomment-433613206 Okay, let me kick tests. @blueorangutan package --

[GitHub] svenvogel commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7

2018-10-27 Thread GitBox
svenvogel commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7 URL: https://github.com/apache/cloudstack/pull/2896#issuecomment-433604893 @rhtyd changes done ;) we can merge :)) ---