[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 @abhinandanprateek In latest master the sequence of event described above only happens when the host has been determined as 'Down'. Refer to the below code. So the bug described won't

[GitHub] cloudstack issue #1637: Command route not available on CentOS 7

Github user lmrv commented on the issue: https://github.com/apache/cloudstack/pull/1637 Hi Resmo, Correct. Most net commands are now replace by the ip command. ip link set dev eth0 up ip route ip add etc. In our case, the ifconfig is not used by

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

Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1645 @jburwell No I didnt. As I said in the other PR, I only have a xenserver environment(my test run was only to see it didnt cause any regressions). If you can get a KVM environment running,

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

Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/1640 @jburwell @koushik-das @marcaurele When MS is unable to determine the state of the VM, Or it thinks VM requires a HA operation then it issues a stop command as part of fence

[GitHub] cloudstack issue #1643: CLOUDSTACK-9460: For long running transactions, if t...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1643 @abhinandanprateek I believe Github now allows you to modify a PR to point to another target branch. @rhtyd is my recollection correct? --- If your project is set up for it, you can reply to

[GitHub] cloudstack issue #1643: CLOUDSTACK-9460: For long running transactions, if t...

Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/1643 @jburwell should i open a new PR for 4.8 branch ? I think it should be easy and good for 4.8 too. --- If your project is set up for it, you can reply to this email and have your

Upcoming 4.8, 4.9, and master Freeze

All, The 4.8.2.0 and 4.9.1.0 releases have taken more time than expected to finalize. Per our original schedule [1], 4.9.2.0 and 4.10.0.0 were scheduled to freeze yesterday (18 September 2016). I propose the following schedule and end-of-life (EOL) changes: * 4.8.2.0 is still open, but

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 @jburwell Let me provide a bit of additional background for this PR. As noted in the initial comment, it was possible to spin up system VMs on managed storage prior to this PR.

[GitHub] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

Github user kiwiflyer commented on the issue: https://github.com/apache/cloudstack/pull/1644 @jburwell We found an issue on Friday related to a use case related to snapshot to template. We're working on it. More soon. --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request #1673: CLOUDSTACK-9071: Properly parse stats.output....

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1673#discussion_r79316781 --- Diff: server/src/com/cloud/server/StatsCollector.java --- @@ -285,9 +286,15 @@ private void init(Map configs) {

[GitHub] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1644 @kiwiflyer could you please post the detailed test results included the environment specification? @nathanejohnson have you had a chance to investigate the failures from the

[GitHub] cloudstack issue #1650: fix macchinina template specificied for vmware in Ma...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1650 @PaulAngus this issue seems to affect 4.8+. Since we have one more 4.8 (4.8.2.0) release, it would be nice to have this fix for those test runs. Therefore, could you re-point it to the 4.8

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1642 @mike-tutkowski @karuturi it seems like this PR should have all smoke tests + system VM integration test suites (SSVM, CPVM, and VR) + the new test integration test suite added for this PR run

[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 @karuturi was this PR tested on KVM? --- 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 #1664: CLOUDSTACK-8676 Deploy user instance from vm snapsho...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1664 @sateesh-chodapuneedi have you @nitt10prashant worked out combining the Marvin tests with this PR? Also, please investigate the Travis CI failures. Finally, have you had a chance to address

[GitHub] cloudstack issue #798: CLOUDSTACK-8830 - [Vmware] VM snapshot fails for 12 m...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/798 This bug would be a good fix to include for LTS. Therefore, it should be re-targeted to the 4.9 branch. @nvazquez are you planning to pick up this PR? If not, I can get someone to

[GitHub] cloudstack pull request #798: CLOUDSTACK-8830 - [Vmware] VM snapshot fails f...

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/798#discussion_r79315858 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareStorageManagerImpl.java --- @@ -1464,4 +1448,55 @@ private String

[GitHub] cloudstack pull request #798: CLOUDSTACK-8830 - [Vmware] VM snapshot fails f...

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/798#discussion_r79315873 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareStorageManagerImpl.java --- @@ -1464,4 +1448,55 @@ private String

[GitHub] cloudstack pull request #798: CLOUDSTACK-8830 - [Vmware] VM snapshot fails f...

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/798#discussion_r79315898 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareStorageManagerImpl.java --- @@ -1464,4 +1448,55 @@ private String

[GitHub] cloudstack pull request #798: CLOUDSTACK-8830 - [Vmware] VM snapshot fails f...

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/798#discussion_r79315948 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareStorageManagerImpl.java --- @@ -1464,4 +1448,55 @@ private String

[GitHub] cloudstack pull request #798: CLOUDSTACK-8830 - [Vmware] VM snapshot fails f...

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/798#discussion_r79315938 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareStorageManagerImpl.java --- @@ -1464,4 +1448,55 @@ private String

[GitHub] cloudstack pull request #798: CLOUDSTACK-8830 - [Vmware] VM snapshot fails f...

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/798#discussion_r79315920 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareStorageManagerImpl.java --- @@ -1464,4 +1448,55 @@ private String

[GitHub] cloudstack issue #1602: CLOUDSTACK-9422: Granular 'vmware.create.full.clone'...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1602 @serg38 @nvazquez I will merge this PR when I get into the office on Monday. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] cloudstack issue #1637: Command route not available on CentOS 7

Github user resmo commented on the issue: https://github.com/apache/cloudstack/pull/1637 speaking about things not available per default on CentOS 7 --> `ifconfig`. Found 1 or 2 of these as well. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1637: Command route not available on CentOS 7

Github user lmrv commented on the issue: https://github.com/apache/cloudstack/pull/1637 I will need your help John. Ok for JIRA ticket but I don't know how to squash my commits, how to retarget my pull request. I am not confortable at all with github. And for Jenkis, no

[GitHub] cloudstack issue #1637: Command route not available on CentOS 7

Github user lmrv commented on the issue: https://github.com/apache/cloudstack/pull/1637 Hi John, I just create the JIRA ISSUE : https://issues.apache.org/jira/browse/CLOUDSTACK-9501 --- If your project is set up for it, you can reply to this email and have your reply appear

Re: [DISCUSS] Replacing the VR

At this point in the discussion, I don't think we should rule anything out. I think it makes sense to explore all the options and then isolate some front runners in terms of software and architecture. On Sep 18, 2016 1:08 AM, "ilya" wrote: > Our options become much

[GitHub] cloudstack issue #1643: CLOUDSTACK-9460: For long running transactions, if t...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1643 @abhinandanprateek this fix seems like it would also be beneficial for 4.8.2.0. How difficult would it be to re-target it to the 4.8 branch? --- If your project is set up for it, you can

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

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1640 @abhinandanprateek @koushik-das @marcaurele have you been able to come to agreement about the correct functionality for this PR? --- If your project is set up for it, you can reply to this

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

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1659 @murali-reddy what is the status of addressing the review feedback. Also, have addressed the VPC-related issues as well? --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack issue #1591: Updating Alert codes

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1591 @rhtyd have you had a chance to analyze? I would like to include this change in 4.9.1.0 and 4.10.0.0 for which we are planning to cut RCs on 25 Sept 2016. --- If your project is set up for

[GitHub] cloudstack issue #1606: Allow CGN (RFC6598) to be used within a VPC

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1606 @leprechau I would like to get this fix into 4.8.2.0, 4.9.1.0, and 4.10.0.0. Have you had a chance to add the unit tests requested? Also, is there a JIRA ticket for this bug? --- If your

[GitHub] cloudstack issue #1623: CLOUDSTACK-9317: Enable/disable static NAT associate...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1623 @ProjectMoon I am extending the date for 4.8 to 25 Sept 2016. Therefore, if we can get the test failure fixed, we can get it into 4.8.2.0, 4.9.1.0, and 4.10.0.0. Also, could you please

[GitHub] cloudstack issue #1624: Fixes regarding VOLUME_DELETE events resulting from ...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1624 @ProjectMoon I want to get this PR into 4.8.2.0, 4.9.1.0, and 4.10.0.0 for which we will be cutting RCs on 25 Sept 2016. Tests should be the entire Marvin smoke test suite against VMware,

[GitHub] cloudstack issue #1637: Command route not available on CentOS 7

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1637 @lmrv I would like to include this PR in the upcoming 4.8.2.0, 4.9.1.0, and 4.10.0.0 releases. In order to merge it, the following items need to be addressed: * [ ] Squash commits

RE: [DISCUSS] Replacing the VR

On this note I also mentioned pfsense earlier. www.pfsense.org Regards, Marty Godsey -Original Message- From: ilya [mailto:ilya.mailing.li...@gmail.com] Sent: Sunday, September 18, 2016 1:09 AM To: dev@cloudstack.apache.org Subject: Re: [DISCUSS] Replacing the VR Our options become

[GitHub] cloudstack issue #1639: CLOUDSTACK-9453: WIP : Marvin optimizations and fixe...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1639 @abhinandanprateek ping regarding the Travis and Jenkins failures. I would like to include this PR in the upcoming 4.8.2.0, 4.9.1.0, and 4.10.0.0 releases for which we plan to cut an RC on 25

[GitHub] cloudstack issue #1560: CLOUDSTACK-9386: DS template copies don’t get dele...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1560 @serg38 @nvazquez I just noticed that this bug was merged to master, but it should also be included in LTS My thinking is that we rollback this commit from master, re-open this PR against the

[GitHub] cloudstack issue #1655: Fix ajaxviewer.js to solve console on Firefox

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1655 @lygutas I would like to get this PR into the upcoming 4.8 (4.8.2.0), 4.9 (4.9.1.0), and 4.10 (4.10.0.0) for which we will be cutting RCs in a week (25 Sept 2016). I see the following minor

[GitHub] cloudstack issue #1666: fix egress rule incorrect behavior

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1666 @murali-reddy have you had a chance to investigate the Jenkins failure(s) and the test failures identified by @karuturi? Does this PR address

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

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1511 @rhtyd given that we have one more release of 4.8 in the pipeline, would it make sense to re-target this PR to 4.8 and forward merge to 4.9 and master? --- If your project is set up for it,

[GitHub] cloudstack issue #1611: marvin: deploy clusters in separate threads

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1611 @rhtyd this enhancement/fix seems like it would be useful for on-going testing of LTS. Therefore, could you please re-target this PR to the 4.9 release branch? Also, could you please

[GitHub] cloudstack pull request #1611: marvin: deploy clusters in separate threads

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1611#discussion_r79303367 --- Diff: tools/marvin/marvin/deployDataCenter.py --- @@ -224,6 +225,15 @@ def createClusters(self, clusters, zoneId, podId, vmwareDc=None):

[GitHub] cloudstack issue #1546: [WIP][4.10+/long-term] Moving towards Java8 and Spri...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1546 @rhtyd Can we close this PR in favor of #1638? --- 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

[GitHub] cloudstack issue #1650: fix macchinina template specificied for vmware in Ma...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1650 @PaulAngus @abhinandanprateek could you investigate the VMware issue described by @serg38? --- If your project is set up for it, you can reply to this email and have your reply appear on

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

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1651 @serg38 @PaulAngus has been on vacation for the past week. Since this problem is present in 4.8 and 4.9, could you please retarget this PR to the 4.8 release branch? Also, as @serg38

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

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1652 @PaulAngus following up on PR feedback. @fmaximus did you apply this patch in another PR? If so, could you please provide a reference? If this patch has not been addressed elsewhere,

[GitHub] cloudstack issue #1662: Fix bug juniper srx

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1662 @digitalgust following up on @rhtyd's feedback. I see the following items to be addressed in order to get this PR merged: * [ ] Squash commits * [ ] Re-point the PR to the

[GitHub] cloudstack issue #1673: CLOUDSTACK-9071: Properly parse stats.output.uri in ...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1673 @wido this fix seems like it would be good for 4.8 and 4.9 as well. Would you mind re-pointing this PR to the 4.8 branch? Also, could you investigate the cause of the Jenkins failure?

[GitHub] cloudstack pull request #1673: CLOUDSTACK-9071: Properly parse stats.output....

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1673#discussion_r79302819 --- Diff: server/src/com/cloud/server/StatsCollector.java --- @@ -285,9 +286,15 @@ private void init(Map configs) {