Github user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/1639
@jburwell rebased it
@rhtyd is there a way to move a PR to another branch without creating a new
one ?
---
If your project is set up for it, you can reply to this email and
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1643
@jburwell yes that is possible, @abhinandanprateek please edit the PR and
change the base branch, then rebase your PR branch against 4.8 branch.
---
If your project is set up for it, you can repl
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 happen
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 cl
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, which
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 operation.
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 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
reply
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 4
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 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 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) {
s_logger.inf
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
@bvbhara
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 br
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1619
@milamberspace we are targeting the 4.10.0.0 RC for 25 Sept 2016. Will you
be able to investigate/resolve the Jenkins build failure by that time?
---
If your project is set up for it, you can
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1642
@mike-tutkowski is there a JIRA ticket for this enhancement? Also, are
there any documentation updates necessary?
---
If your project is set up for it, you can reply to this email and have yo
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 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
e
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 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 pic
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 del
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 del
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 del
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 del
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 del
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 del
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 y
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 ap
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 mor
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 o
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 better if we consider BSD base
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 repl
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 ema
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 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 i
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 pr
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 invest
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, KVM,
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
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 mu
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 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 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 it
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
[CLOUDSTACK-9480](https://issues.apache.
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, yo
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 investig
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 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 this
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 GitHu
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 suggests
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 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 4.
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 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) {
s_logger.inf
54 matches
Mail list logo