Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/970#issuecomment-151182430
I will test if renaming the plugin works (and make sure there are no links
to its name).
---
If your project is set up for it, you can reply to this email and h
Github user wido commented on the pull request:
https://github.com/apache/cloudstack/pull/977#issuecomment-151160292
@ustcweizhou I want a stable master branch which works all the time.
Executing binaries is not reliable as we can't do proper exception handling.
Also, virsh do
+1 what Wido said.
--
Sent from the Delta quadrant using Borg technology!
Nux!
www.nux.ro
- Original Message -
> From: "Wido den Hollander"
> To: dev@cloudstack.apache.org
> Sent: Monday, 26 October, 2015 09:59:18
> Subject: Re: [DISCUSS] Move to GitHub
> On 25-10-15 10:52, Milamber wr
Github user xenserverarmy commented on the pull request:
https://github.com/apache/cloudstack/pull/970#issuecomment-151157372
@koushik-das One of the goals of a8212d9ef458dd7ac64b021e6fa33fcf64b3cce0
was to ensure that we have clarity in which hypervisor we support since Xen !=
XenSer
Hi
On 10/26/2015 10:59 AM, Wido den Hollander wrote:
>> But, I thinks that would be better to keep the control of the source
>> code repo. This is the core of your work.
>>
>
> Agree. I love Github and it works great, but make it THE primary source
> for the source code and go away from ASF? Well
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/977#issuecomment-151155683
@wido it depends on which is more important for us, more features in
cloudstack or removing virsh from cloudstack? we know "virsh" provides more
features than li
Github user wido commented on the pull request:
https://github.com/apache/cloudstack/pull/977#issuecomment-151153457
@ustcweizhou I understand the reasoning and that you want this feature in
4.7, but there is no guarantee it will be fixed afterwards. Once the feature is
in there nobod
Can someone help me how to set the community vlan in cloudstack I can see
only options as SecondaryVLAN id.
Thanks,
Keerthi
On Mon, Oct 26, 2015 at 3:54 PM, Keerthiraja SJ wrote:
> Hi All,
>
> Today I tried to create Advance Zone I added new network with using of
> vlan primary 41 secondary vla
GitHub user serbaut opened a pull request:
https://github.com/apache/cloudstack/pull/981
CLOUDSTACK-8993: DHCP fails with "no address available" when an IP isâ¦
⦠reused
Repopulate /etc/dhcphosts.txt to remove old entries with the same IP
address.
You can merge this pul
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151145630
Hi @borisroman
Although my first round of tests failed, after looking into the test code
it seems to be environment related:
```
Test a
GitHub user borisroman opened a pull request:
https://github.com/apache/cloudstack/pull/980
CLOUDSTACK-8992 Allow 32 disks to be attached to a KVM VM.
Tested manually.
Added 32 disks to a VM, booted the VM, formatted all disks and mounted them
all. Also created volume group
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/978#issuecomment-151136095
change default host from empty to 'Default'. done.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/978#issuecomment-151133235
@borisroman yes, sure
---
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 ha
Github user borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/978#issuecomment-151131758
@ustcweizhou By "user" I meant the root admin. So my question is: Could you
adjust the dialog you created to show something like "Default" instead of an
empty lab
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/977#issuecomment-151130476
@wido good to know it, thanks. I will implement some features in
libvirt-java.
I would like to keep current command line and merge it to cs 4.7, and
change i
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/894#issuecomment-151130113
LGTM, based on a set of tests that I run on this branch:
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
tags=advanced,required_hardwa
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/978#issuecomment-151127784
@borisroman sorry my comment made you confused. The users I meant is one
who use cloudstack for deployment.
This window is only visible for root admins. For
Github user nlivens commented on the pull request:
https://github.com/apache/cloudstack/pull/801#issuecomment-151120955
@remibergsma, I've rebased against current 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
Github user borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151116939
Jenkins failed due to the following tests:
```
org.apache.cloudstack.utils.hypervisor.HypervisorUtilsTest.checkVolumeFileForActivityTest
```
https
Cool,
BTW, The reason for failure for the following tests is downloading of
templates/ISOs is taking a long time to download
1)
integration.smoke.test_vm_life_cycle.TestVMLifeCycle.test_10_attachAndDetach_iso
2)
integration.smoke.test_internal_lb.TestInternalLb.test02_internallb_haproxy_sta
Github user borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/978#issuecomment-151116076
@ustcweizhou Oke! Though can you make that visible to the user? Now it
seems like you "have to" select a host, like you're forced to do so.
---
If your project i
Github user borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/920#issuecomment-151115924
@runseb Could you rebase and push again? If so, and Travis and Jenkins are
green, we can merge.
Ping @remibergsma @karuturi
---
If your project is set
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/978#issuecomment-151115822
@borisroman None means cloudstack will allocate the host to start the vm. I
think it is also needed for users. Not everyone wants to specify a host, right?
---
Github user borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/978#issuecomment-151115265
@remibergsma @karuturi I've run the following tests:
```
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/978#issuecomment-151115178
@ustcweizhou Thanks for implementing this feature. I've got 1 comment,
could you select the first host available instead of none? See image.
![screenshot
I like the idea. Few issues I can think of;
- Project will become dependent on Github (but luckily, there is a migration
path from Github to Gitlab, in case anything goes wrong in future)
- Github issues and wiki are quite limited, compared to JIRA and confluence
- Possible split brain issues, wh
I have narrowed it down to multithreading being the cause. I have a set of
workers running through an executor service, and it seems cglib is not
respecting the @DB annotation on the method being called by the thread.
I can force the lock to be acquired if the entire method is executed in a
transa
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/961#issuecomment-151105807
@K0zka the page is blank. I tried sometimes with the same result (blank
page with this commit, works without this commit).
---
If your project is set up for it
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151104355
thanks for the tests, @karuturi! I will test this PR.
Cheers,
Wilder
---
If your project is set up for it, you can reply to this email and have
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/970#issuecomment-151104185
Good point, @koushik-das.
@remibergsma: do you agree on that one as well?
---
If your project is set up for it, you can reply to this email and have
Github user K0zka commented on the pull request:
https://github.com/apache/cloudstack/pull/961#issuecomment-151102999
@ustcweizhou could you share some details about the problem? Are you not
able to login?
---
If your project is set up for it, you can reply to this email and have you
Github user insom commented on the pull request:
https://github.com/apache/cloudstack/pull/848#issuecomment-151097482
@remibergsma Hi, rebased and pushed!
---
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 do
Github user karuturi commented on the pull request:
https://github.com/apache/cloudstack/pull/860#issuecomment-151096712
3rd attempt. I hope it succeeds this time.
---
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 user borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/860#issuecomment-151096159
@karuturi Jenkins failed;
[ERROR] Error in Cobertura Report generation: Unable to execute Cobertura.
Could maybe force push again?
---
If your p
Github user wido commented on the pull request:
https://github.com/apache/cloudstack/pull/977#issuecomment-151094003
@ustcweizhou It's just a matter of sending patches to RedHat and they will
accept it. You can also request a new release of the bindings when you need to.
It se
Github user koushik-das commented on the pull request:
https://github.com/apache/cloudstack/pull/970#issuecomment-151092704
Ping @runseb.
Is there a handle/userid for Tim Mackey?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Hi All,
Today I tried to create Advance Zone I added new network with using of vlan
primary 41 secondary vlan 42 all the basic necessary been done on switch.
While I create a VM I could see error as
nw-label: cloudstack, isolated-pvlan: 42, op: add, primary-pvlan: 41, due
to There was a failure
Github user koushik-das commented on the pull request:
https://github.com/apache/cloudstack/pull/970#issuecomment-151091001
Since a8212d9 changed names from "xen" to "xenserver", shouldn't the change
be to rename the files s3xen to s3xenserver and swiftxen to swiftxenserver in
scripts
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/977#issuecomment-151090163
@wido I agree with you. The ideal solution is to implement the features in
libvirt-java.
At this moment, I do not know how long it will take and when it will
Github user borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/860#issuecomment-151089307
@remibergsma @karuturi I tested this PR with the changes in #979. Results:
```
nosetests --with-marvin
--marvin-config=/data/shared/marvin/mct-zone1-k
GitHub user borisroman opened a pull request:
https://github.com/apache/cloudstack/pull/979
CLOUDSTACK-8989 test_ps_limits.py tests can also be run on a "basic" setup.
The tests in the test_ps_limits.py file can all be run on a "basic" setup.
This PR adds that functionality.
You ca
Github user wido commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/977#discussion_r42975042
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
---
@@ -3388,4 +3419,83 @@ public String mapRbdDevice(
Github user wido commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/977#discussion_r42975005
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtRestoreVMSnapshotCommandWrapper.java
---
@@ -0,0 +1,121 @@
+//
Github user wido commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/977#discussion_r42975057
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
---
@@ -3388,4 +3419,83 @@ public String mapRbdDevice(
GitHub user ustcweizhou opened a pull request:
https://github.com/apache/cloudstack/pull/978
CLOUDSTACK-8990: start a stopped machine on a specific determinable host on
UI
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ustcweiz
On 25-10-15 10:52, Milamber wrote:
>
>
> On 25/10/2015 07:05, Sebastien Goasguen wrote:
>> Folks,
>>
>> We have been using GitHub for PR now for a while, it seems to work
>> despite the fact that we still have to get used to the workflow.
>> We lack some privileges on the mirror to be able to u
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/961#issuecomment-151087205
guys, CS UI is broken by this commit on my env.
I am using Chrome (Version 46.0.2490.80 m)
It works after reverting this commit.
---
If your project is s
Github user karuturi commented on the pull request:
https://github.com/apache/cloudstack/pull/962#issuecomment-151086437
@snuf @NuxRo KVMGuru and LXCGuru are defined at
https://github.com/apache/cloudstack/blob/master/server/resources/META-INF/cloudstack/server-compute/spring-server-co
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/970#issuecomment-151083526
Code look good to me, no rocket science involved.
I will test it with XenServer 6.5, but I might need help figuring out how
to get it covered.
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/970#issuecomment-151081518
@remibergsma @borisroman - I'm starting tests now.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub a
Github user pvr9711 commented on the pull request:
https://github.com/apache/cloudstack/pull/960#issuecomment-151071811
+1 move this test out of BVTs - this was brought up on almost 3-4 weeks
ago.
@runseb - I guess, my understanding is - we need 100% passrate on all BVTs
- we can
Github user pvr9711 commented on the pull request:
https://github.com/apache/cloudstack/pull/902#issuecomment-151070859
LGTM, I have tested this on a private setup and is working.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user sanju1010 commented on the pull request:
https://github.com/apache/cloudstack/pull/902#issuecomment-151068985
@pvr9711 can you please review this PR? I have made changes as per the
review comments and rebased it with master.
---
If your project is set up for it, you can r
Github user miguelaferreira commented on the pull request:
https://github.com/apache/cloudstack/pull/950#issuecomment-151057998
It's already included as a dependency in the `noredist` profile. This PR
includes it as a dependency in the `developer` profile, and forces the MySQL
driver
Github user borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/950#issuecomment-151055701
Can't we include it as dependency? Or are there licencing issues?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user ustcweizhou commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/977#discussion_r42966712
--- Diff:
api/src/org/apache/cloudstack/api/command/user/snapshot/CreateSnapshotFromVMSnapshotCmd.java
---
@@ -0,0 +1,191 @@
+// Licensed to the
Github user sateesh-chodapuneedi commented on the pull request:
https://github.com/apache/cloudstack/pull/977#issuecomment-151054893
@ustcweizhou
I think it's handy to include account access checks for the backup snapshot
from vm snapshot.
---
If your project is set up for it, y
Github user sateesh-chodapuneedi commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/977#discussion_r42966399
--- Diff:
engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java ---
@@ -1702,6 +1705,18 @@ protected boolean checkVmOnHost(fin
Github user sateesh-chodapuneedi commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/977#discussion_r42966249
--- Diff:
api/src/org/apache/cloudstack/api/command/user/snapshot/CreateSnapshotFromVMSnapshotCmd.java
---
@@ -0,0 +1,191 @@
+// License
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/977#issuecomment-151050842
@sateesh-chodapuneedi Thanks for your comment, I will fix it soon.
This is an implementation of vm snapshot for KVM.
You may find the FS
at:https://cwiki
Github user miguelaferreira commented on the pull request:
https://github.com/apache/cloudstack/pull/950#issuecomment-151050165
The erros, travis and Jenkins report are due to missing mysql connector
jar. Shall I make Travis build with the developer profile (i.e. `-Pdeveloper`)?
Or ju
Github user sateesh-chodapuneedi commented on the pull request:
https://github.com/apache/cloudstack/pull/977#issuecomment-151049279
@ustcweizhou Can you please share FS or design document for this work?
It'd help greatly in reviewing this patch.
Thanks.
---
If your project is
Github user sateesh-chodapuneedi commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/977#discussion_r42965130
--- Diff:
api/src/org/apache/cloudstack/api/command/user/snapshot/CreateSnapshotFromVMSnapshotCmd.java
---
@@ -0,0 +1,191 @@
+// License
Github user karuturi commented on the pull request:
https://github.com/apache/cloudstack/pull/860#issuecomment-151047752
@borisroman yes this is real :) . I think this might exist in 4.5 as well.
Haven't tested it there.
I rebased with the current master
---
If your project is se
Github user koushik-das commented on the pull request:
https://github.com/apache/cloudstack/pull/883#issuecomment-151047344
Changes LGTM. I haven't tested it.
---
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 projec
101 - 165 of 165 matches
Mail list logo