[GitHub] cloudstack issue #1647: [lts] CLOUDSTACK-9462: Systemd support for Ubuntu 16...

Github user vincentbernat commented on the issue: https://github.com/apache/cloudstack/pull/1647 it is not possible to have systemd support and build on 12.04 at the same time. dh-systemd doesn't exist on 12.04 and its actions are quite important. If you remove it and just install

[GitHub] cloudstack issue #1511: 4.9/master bountycastle changes

Github user bvbharatk commented on the issue: https://github.com/apache/cloudstack/pull/1511 ### ACS CI BVT Run **Sumarry:** Build Number 50 Hypervisor xenserver NetworkType Advanced Passed=100 Failed=1 Skipped=3 _Link to logs Folder (search

[GitHub] cloudstack issue #1647: [lts] CLOUDSTACK-9462: Systemd support for Ubuntu 16...

Github user vincentbernat commented on the issue: https://github.com/apache/cloudstack/pull/1647 Otherwise, I think the change to be compatible with 12.04 is OK (no Upstart script should ever be provided in this case). --- If your project is set up for it, you can reply to this

[GitHub] cloudstack issue #1640: CLOUDSTACK-9458: Fix HA bug when VMs are stopped on ...

Github user koushik-das commented on the issue: https://github.com/apache/cloudstack/pull/1640 @marcaurele > What is the reason to try fencing off VMs when the MS is not able to determine its state? I cannot see a good reason so far but you seem to think there is at least

[GitHub] cloudstack issue #1651: Marvin Tests: fix expected return string for success...

Github user koushik-das commented on the issue: https://github.com/apache/cloudstack/pull/1651 @rhtyd Please go through #1563 as mentioned in my previous comment to get the context. As @serg38 also mentioned the response differs between debian and centos which needs to be addressed,

[GitHub] cloudstack issue #1647: [lts] CLOUDSTACK-9462: Systemd support for Ubuntu 16...

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1647 @vincentbernat I made some changes, built packages and was able to install and run agent and usage server on both Ubuntu 12.04, 14.04 using the same packages. @wido @vincentbernat can you

CS 4.9 NIO Selector wait time PR-1601

hi all We have the following environment: - OS: Debian 14.04 (hypervisors and management) - 4 KVM Hosts - Cloudstack Release 4.9 with local database Since we upgraded to Release 4.9 the KVM hosts no longer connect to the management Server. Upgrade procedure was followed as described:

[GitHub] cloudstack issue #1658: Added an additional JSON diff output to the ApiXmlDo...

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1658 Great idea, LGTM (based on the api-changes urls). @swill can we also sort the list of new APIs by name. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1647: [lts] CLOUDSTACK-9462: Systemd support for Ubuntu 16...

Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1647 @rhtyd Seems good, but it would still not force the install of Java 8 on Ubuntu 16.04 However, I don't think that's a blocker. LGTM for me --- If your project is set up for it,

[GitHub] cloudstack issue #1640: CLOUDSTACK-9458: Fix HA bug when VMs are stopped on ...

Github user marcaurele commented on the issue: https://github.com/apache/cloudstack/pull/1640 @koushik-das which is IMHO wrong. --- 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] cloudstack issue #1647: [lts] CLOUDSTACK-9462: Systemd support for Ubuntu 16...

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1647 @wido actually on a fresh system it would first attempt to find and install openjdk8, on existing ones if openjdk7 is installed then it won't force openjdk8 by default. Thanks. --- If your

[GitHub] cloudstack pull request #1659: CLOUDSTACK-9339 Virtual Routers don't handle ...

GitHub user murali-reddy opened a pull request: https://github.com/apache/cloudstack/pull/1659 CLOUDSTACK-9339 Virtual Routers don't handle Multiple Public Interfaces correctly for non-vpc networks As pointed out in CLOUDSTACK-9339, in case of multiple public IP's from different

[GitHub] cloudstack pull request #1647: [lts] CLOUDSTACK-9462: Systemd support for Ub...

Github user vincentbernat commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1647#discussion_r76039128 --- Diff: debian/rules --- @@ -5,18 +5,9 @@ PACKAGE = $(shell dh_listpackages|head -n 1|cut -d '-' -f 1) SYSCONFDIR = "/etc" DESTDIR =

[GitHub] cloudstack issue #1605: CLOUDSTACK-9428: Fix for CLOUDSTACK-9211 - Improve p...

Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1605 @nv, I am sorry, my bad; last time I read the code, I overlooked the method com.cloud.hypervisor.vmware.mo.VirtualMachineMO.getAllDeviceList(). Having said that, how do you feel about

[GitHub] cloudstack-docs-admin issue #40: index.po and ui.po translated in FR

Github user pdion891 commented on the issue: https://github.com/apache/cloudstack-docs-admin/pull/40 @lmrv This is a great start! I'm willing to merge this PR, but it's a work in progress so it will not be visible on the main site until the translation is mostly completed.

[GitHub] cloudstack-docs-admin issue #40: index.po and ui.po translated in FR

Github user pdion891 commented on the issue: https://github.com/apache/cloudstack-docs-admin/pull/40 @lmrv if you have a chance to update all files copyright header, to must have the apache2 v2 license header. Thanks --- If your project is set up for it, you can reply to

[GitHub] cloudstack issue #1645: On snapshot backup, this converts the rbd raw format...

Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1645 @jburwell as an update, the smoke test has completed successfully, and I'll let the component tests run overnight. Hopefully they will have finished by the morning. --- If your

Re: JuniperSRX firewall configure plugin source bug

I submitted PR: https://github.com/apache/cloudstack/pull/1662 > 在 2016年8月24日，20:36，Will Stevens 写道： > > Hey Gust, > Here are the details for contributing. > > https://github.com/apache/cloudstack/blob/master/CONTRIBUTING.md > > You need to fork cloudstack and the

[GitHub] cloudstack issue #1605: CLOUDSTACK-9428: Fix for CLOUDSTACK-9211 - Improve p...

Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1605 LGTM for testing. Vmware ESX 5.5 and 6.0 hypervisors, advanced networking, RHEL 6 management servers [root@ussarlabcsmgt41 ~]# cat /tmp//MarvinLogs/test_volumes_930LZ3/results.txt|grep

Removing old entries from storage in database

Hello, I have some entries in my database for storage arrays that are no longer in use. WHats the best way to go about removing these? The ones that have no GUID do not show in the UI and the one that does (id 5) wont let me delete it even though the XenServers cant mount it.

[GitHub] cloudstack pull request #1662: Fix bug juniper srx

GitHub user digitalgust opened a pull request: https://github.com/apache/cloudstack/pull/1662 Fix bug juniper srx if the public interface has marked untagged ,the vlan is “vlan://untagged” ,but old source is “untagged” , so it will raise a exception : Unable to parse

[GitHub] cloudstack issue #1647: [lts] CLOUDSTACK-9462: Systemd support for Ubuntu 16...

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1647 @wido based on the ML discussion, I've removed the specific change that removed 12.04 support. Can you send another PR for master for removing support for 12.04. --- If your project is set up

[GitHub] cloudstack pull request #1647: [lts] CLOUDSTACK-9462: Systemd support for Ub...

Github user rhtyd commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1647#discussion_r76004158 --- Diff: debian/rules --- @@ -5,8 +5,17 @@ PACKAGE = $(shell dh_listpackages|head -n 1|cut -d '-' -f 1) SYSCONFDIR = "/etc" DESTDIR =

[GitHub] cloudstack pull request #1656: CLOUDSTACK-9466: Fix fk constraint failure in...

Github user rhtyd closed the pull request at: https://github.com/apache/cloudstack/pull/1656 --- 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 #1656: CLOUDSTACK-9466: Fix fk constraint failure in...

GitHub user rhtyd reopened a pull request: https://github.com/apache/cloudstack/pull/1656 CLOUDSTACK-9466: Fix fk constraint failure in upgrade path In the 4.1.0-4.2.0 db upgrade path, it creates new tables to store secondary (nfs) storage in image_store table and volumes in

[GitHub] cloudstack issue #1591: Updating Alert codes

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1591 @dcarbone can you squash your commits and rebase against latest, thanks. Please also amend git commit message to include the JIRA ID. @jburwell I did not start my analysis, I'll do that once

[GitHub] cloudstack issue #1651: Marvin Tests: fix expected return string for success...

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1651 LGTM @koushik-das All tests are written to be execute on a Linux based environment, though if you look at the test it does ssh into the guest VM and then executes the ping. Afaik, we don't

[GitHub] cloudstack issue #1651: Marvin Tests: fix expected return string for success...

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1651 @PaulAngus please squash you commits, 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

[GitHub] cloudstack issue #1657: [lts/blocker] CLOUDSTACK-9467: Add symlink to key fi...

Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/1657 LGTM on code review, this will fix the broken symlink for usage server --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] cloudstack issue #1656: CLOUDSTACK-9466: Fix fk constraint failure in upgrad...

Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/1656 LGTM on code review, the fix is taking care of the removed column, thereby maintaining the constraints. --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request #1647: [lts] CLOUDSTACK-9462: Systemd support for Ub...

Github user wido commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1647#discussion_r76006449 --- Diff: debian/rules --- @@ -5,8 +5,17 @@ PACKAGE = $(shell dh_listpackages|head -n 1|cut -d '-' -f 1) SYSCONFDIR = "/etc" DESTDIR =

[GitHub] cloudstack issue #1647: [lts] CLOUDSTACK-9462: Systemd support for Ubuntu 16...

Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1647 @rhtyd Why another PR for against master? I thought we were merging forward? This stuff is really starting to confuse me. --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack issue #1647: [lts] CLOUDSTACK-9462: Systemd support for Ubuntu 16...

Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1647 @rhtyd When this one merges and is merged forward I will send a PR for master to remove 12.04 support. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack issue #1656: CLOUDSTACK-9466: Fix fk constraint failure in upgrad...

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1656 Thanks @serg38 @abhinandanprateek for the review. With enough lgtms and (manual) test result against a large pre-4.1 db, I'll proceed with merging. --- If your project is set up for it, you can

[GitHub] cloudstack issue #1657: [lts/blocker] CLOUDSTACK-9467: Add symlink to key fi...

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1657 Thanks @abhinandanprateek @serg38 for the review. With enough lgtms and travis/manual tests, I'll merge this (failing Travis tests are not due to this change, but due to a tearDown issue).

[GitHub] cloudstack pull request #1656: CLOUDSTACK-9466: Fix fk constraint failure in...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1656 --- 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] cloudstack pull request #1657: [lts/blocker] CLOUDSTACK-9467: Add symlink to...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1657 --- 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] cloudstack pull request #1647: [lts] CLOUDSTACK-9462: Systemd support for Ub...

Github user rhtyd commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1647#discussion_r76007356 --- Diff: debian/rules --- @@ -5,8 +5,17 @@ PACKAGE = $(shell dh_listpackages|head -n 1|cut -d '-' -f 1) SYSCONFDIR = "/etc" DESTDIR =

[GitHub] cloudstack issue #1647: [lts] CLOUDSTACK-9462: Systemd support for Ubuntu 16...

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1647 Thanks @wido that makes sense. --- 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 pull request #1634: [blocker] CLOUDSTACK-9452: add python-argpars...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1634 --- 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] cloudstack issue #1656: CLOUDSTACK-9466: Fix fk constraint failure in upgrad...

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1656 @jburwell 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,

[GitHub] cloudstack issue #1656: CLOUDSTACK-9466: Fix fk constraint failure in upgrad...

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1656 There are no upgrade tests to confirm the fix as all the changes are in an older 4.1.0->4.2.0 upgrade path. After this fix, here are manual test results to show that db upgrade finished cleanly

[GitHub] cloudstack issue #1648: test/integration: fix tearDown order in list_acl_ te...

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1648 @jburwell can you review/lgtm now? 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

[GitHub] cloudstack issue #1657: [lts/blocker] CLOUDSTACK-9467: Add symlink to key fi...

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1657 @jburwell no this is PR is related to packaging, while 1593 fixes something different. The Travis build failure has been fixed at #1648, the failures are unrelated to this PR. Since this is a

[GitHub] cloudstack issue #1634: [blocker] CLOUDSTACK-9452: add python-argparse depen...

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1634 Merging based on lgtms and test results (the failures are due to env issues, and failing tests not related to the pkging fix). --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack issue #1652: Marvin Tests: Fix VPC network offering selection .

Github user fmaximus commented on the issue: https://github.com/apache/cloudstack/pull/1652 I will make the change. I also plan to only create the vpc offering if there are Nuage VSP devices configured, which makes more sense for the users of CloudStack that do not use our plugin.

Re: JuniperSRX firewall configure plugin source bug

Hi, Jayapal why not submit a PR, in github desktop prompt: You may not have permission to access cloudstack. Check Preferences to make sure you’re still logged in. I ensure i loved in. > 在 2016年8月24日，12:52，Jayapal Uradi

[GitHub] cloudstack issue #1653: Prevent DNS reflection attacks

Github user NuxRo commented on the issue: https://github.com/apache/cloudstack/pull/1653 Understood, 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

Re: CS 4.9 NIO Selector wait time PR-1601

Thanks Will! yes the repo is pointing to 4.9 release for all KVMs and for the management server: /cloudstack:~# cat /etc/apt/sources.list.d/cloudstack.list // //deb http://cloudstack.apt-get.eu/ubuntu trusty 4.9/ All KVM agents and the mgmt server are upgraded to release 4.9 based on the

[GitHub] cloudstack issue #1658: Added an additional JSON diff output to the ApiXmlDo...

Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/1658 I don't think this implementation should change. This simply outputs the results to a JSON object (which is unordered by definition). The generation of the documentation is separate and that

Virtio vs IDE emulation, per OS - where is this info ?

Hi all, Im trying to build the simple table for our support guys, which OS type (as seen in ACS) is virtio emualted, and which is IDE/Intel emulation. I dont seem to find this in DB, except the list of OS types, families etc. Can someone give some info, where should I find if OS is supposed to

Re: Virtio vs IDE emulation, per OS - where is this info ?

Andrija, It's actually hard coded and based on the template namethis really needs to be reworked. https://github.com/apache/cloudstack/blob/3a81a4498f6b56eb7330bc992f0c89f6cba86cb3/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java#L2885 - Si

[GitHub] cloudstack pull request #1661: Export UUID for zone creation event completio...

GitHub user ProjectMoon opened a pull request: https://github.com/apache/cloudstack/pull/1661 Export UUID for zone creation event completion. Action events have support for exporting their "first class entities" over the event bus, if it's turned on. This works mostly for

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

Github user kiwiflyer commented on the issue: https://github.com/apache/cloudstack/pull/1635 @ProjectMoon Jeff, could you help review this, as I believe you were the first to point it out on the list. From talking to Nathan, it sounds like we're not sure KVM is actually

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

Github user ProjectMoon commented on the issue: https://github.com/apache/cloudstack/pull/1635 All hypervisors are affected by this, since the parameter is just not passed into the VM Manager. That means its effects don't work at all anywhere. :) I will try to review it to tomorrow.

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

Github user kiwiflyer commented on the issue: https://github.com/apache/cloudstack/pull/1635 I'll let him comment on this, but If I recall, it had more to do with implementation of force stopping with KVM rather than the lack of the parameter being passing into the VM Manager. ---

Re: CS 4.9 NIO Selector wait time PR-1601

Martin, Were you able to fix your issue after installing packages from the repo Will shared and restarting the services? I've not personally tested the apt-get.eu repo, but I had earlier built this repo which I'm personally using in my local KVM-trusty based cloud:

[GitHub] cloudstack issue #1660: CLOUDSTACK-9470: [BLOCKER] Fix for test_network_acl,...

Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1660 Also we wrote a tester for SshHelper.sshExecute method. It was returning output of SSHexecute command as it gets received by sshExecute. Here they are 100 sequential runs (success if result is

[GitHub] cloudstack issue #1645: On snapshot backup, this converts the rbd raw format...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1645 @nathanejohnson excellent. I will keep any eye out for your updates. --- 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] cloudstack issue #1660: CLOUDSTACK-9470: [BLOCKER] Fix for test_network_acl,...

Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1660 LGTM for the test. Exception in network_acl_test **before the fix** [root@ussarlabcsmgt41 cloudstack]# cat /tmp//MarvinLogs//NB313W/results.txt Test network ACL lists and items in VPC

Re: CS 4.9 NIO Selector wait time PR-1601

The PR and fix already exists in 4.9.0 release. Please make sure to upgrade all of your management server(s) and KVM agents and then also restart them after the upgrade. If you are seeing SSL handshake failures, it could be due to network or security issue and most likely due to mismatch

Re: JuniperSRX firewall configure plugin source bug

Hey Gust, Here are the details for contributing. https://github.com/apache/cloudstack/blob/master/CONTRIBUTING.md You need to fork cloudstack and the the change in your fork. Then you create a pull request from your fork to ACS. Let us know if you have problems following that guide. Cheers,

Re: CS 4.9 NIO Selector wait time PR-1601

That PR is already merged, so you don't have to do anything to get that code, you already have it. @rohit, can you review this? I think this is a similar to the issue Simon reported earlier. Will On Aug 24, 2016 6:56 AM, "fanfarlo" wrote: > hi all > > We have the

[GitHub] cloudstack issue #1660: CLOUDSTACK-9470: [BLOCKER] Bug in SshHelper affectin...

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1660 Thanks, this looks interesting. I'll try to get back to you 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

[GitHub] cloudstack issue #1645: On snapshot backup, this converts the rbd raw format...

Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1645 @jburwell I'm hoping that I can spend a good part of today on the requests you've made on my three open pull requests. --- If your project is set up for it, you can reply to

Re: CS 4.9 NIO Selector wait time PR-1601

@rohit, I am guessing they should be installing the cloudstack-agent using the following repo right? That is what is described in the upgrade (trusty instead of precise though). http://cloudstack.apt-get.eu/ubuntu/dists/trusty/4.9/ @fanfarlo, are your repo's setup to point to the new 4.9

[GitHub] cloudstack pull request #1660: CLOUDSTACK-9470: [BLOCKER] Fix for test_netwo...

GitHub user nvazquez opened a pull request: https://github.com/apache/cloudstack/pull/1660 CLOUDSTACK-9470: [BLOCKER] Fix for test_network_acl, SshHelper ## Problem In a Vmware environment, running `test_network_acl` we examined why sometimes it failed and sometimes it passed.

[GitHub] cloudstack issue #1605: CLOUDSTACK-9428: Fix for CLOUDSTACK-9211 - Improve p...

Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1605 Thanks @rafaelweingartner for your review! I've been working on it and pushed changes based on your comments! The only thing I need to mention is that actually I can't remove `throws