Re: [DISCUSS] 4.5.2 bugfix sprint

On 05-08-15 18:37, Rohit Yadav wrote: > >> On 05-Aug-2015, at 11:01 am, Rajani Karuturi > > wrote: >> >> Hi Rohit, >> Any update on the release timelines for 4.5.2? I think based on the >> current >> upgrade path, we have to do 4.5.2 before 4.6.0. Else upgrade path need

[GitHub] cloudstack pull request: CLOUDSTACK-8301: Enable configuring local...

GitHub user bhaisaab opened a pull request: https://github.com/apache/cloudstack/pull/661 CLOUDSTACK-8301: Enable configuring local storage use for system VMs … …at zone level Backported from #263 for 4.5 branch More information on https://issues.apache.org/jira/brow

[GitHub] cloudstack pull request: CLOUDSTACK-8301: Enable configuring local...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/661#issuecomment-128280936 Created PR, still testing on KVM/4.5 --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-8301: Enable configuring local...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/661#issuecomment-128284375 @koushik-das I've backported your fix from master to 4.5 using the PR #263, please review --- If your project is set up for it, you can reply to this email and hav

[GitHub] cloudstack pull request: CLOUDSTACK-8301: Enable configuring local...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/661#issuecomment-128288397 Tests fixed, hoping this will pass Travis now --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If you

[GitHub] cloudstack pull request: CLOUDSTACK-8687: Prepare template only on...

Github user devdeep commented on the pull request: https://github.com/apache/cloudstack/pull/635#issuecomment-128292255 @DaanHoogland When prepareTemplate is called, it either falls into prepareTemplateInOneStoragePool if one primary has to be seeded with the template; or prepareTempl

Re: Super trivial code change and PR

H Mike, If you find a good title for such a trivial change bucket pr it makes creating release notes easier as well. Try to group them by mental state or character (can you say that in English?) On Thu, Aug 6, 2015 at 8:31 AM, Mike Tutkowski wrote: > Sounds good > > I could always "cache" those

[GitHub] cloudstack pull request: CLOUDSTACK-8698: Retrieve a new device ID...

Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/660#issuecomment-128293184 ok. There are lot of ifs in that part of the code which makes it difficult to read. I will have to try it to understand fully for which I dont have time now. So, I

[GitHub] cloudstack pull request: CLOUDSTACK-8696: Create Region fails with...

Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/657#issuecomment-128293895 I read it as this PR is good for master. If thats the case, can I get some +1s please. --- If your project is set up for it, you can reply to this email and have y

[GitHub] cloudstack pull request: CLOUDSTACK-8301: Enable configuring local...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/661#issuecomment-128300894 Did functional testing with two KVM hosts, across two Advance Zones with VLAN isolation: ![screen shot 2015-08-06 at 2 34 24 pm](https://cloud.githubuserco

[GitHub] cloudstack pull request: CLOUDSTACK-8696: Create Region fails with...

Github user kishankavala commented on the pull request: https://github.com/apache/cloudstack/pull/657#issuecomment-128306138 LGTM if 4.5.2 is released before 4.6.0 --- 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 p

[GitHub] cloudstack pull request: CLOUDSTACK-8711: public_ip type resource ...

GitHub user manuiiit opened a pull request: https://github.com/apache/cloudstack/pull/662 CLOUDSTACK-8711: public_ip type resource count for an account is not decremented upon IP range deletion Probleam: -- When you add an IP range and associate it to

[GitHub] cloudstack pull request: CLOUDSTACK-8638: install update_host_pass...

GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/663 CLOUDSTACK-8638: install update_host_passwd.sh in cloudstack-common installation You can merge this pull request into a Git repository by running: $ git pull https://github.com/ustcwe

Re: [DISCUSS] 4.5.2 bugfix sprint

Rajani, Wido, I think what Rohit says makes sense. If we put it in 451to452 and release 4.5.2 in the coming week and 4.6.0 at the end of the month we are alright. Except for Wido that is, he will have to fix his DB's removing the autoincrement in the id of regions by hand. On Thu, Aug 6, 2015 at

[GitHub] cloudstack pull request: CLOUDSTACK-8696: Create Region fails with...

Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/657#issuecomment-128337719 I will merge this to master and keep the bug open so that it can be fixed in 4.5.2 --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: CLOUDSTACK-8696: Create Region fails with...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/657 --- 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: CLOUDSTACK-8301: Enable configuring local...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/661#issuecomment-128342511 Changes LGTM. Make sure travis is green. --- 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 pull request: CLOUDSTACK-8301: Enable configuring local...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/661#issuecomment-128347115 @koushik-das thanks, looks like Travis failed due to some timeout issue and in other case due to dependency/jar issue. Will force push again to re-kick the travis j

[GitHub] cloudstack pull request: CLOUDSTACK-8696: Create Region fails with...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/657#issuecomment-128373260 Looks like some confusion is going on here; the changes need to go into: setup/db/db/schema-451to452.sql that I had suggested earlier. If you do that, then people w

[GitHub] cloudstack pull request: CLOUDSTACK-8638: install update_host_pass...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/663#issuecomment-128386362 LGTM --- 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

Re: [DISCUSS][PROPOSAL] Support for standard metadata/userdata URL

BTW you can speed up cloud-init to look for CloudStack first/only by overriding the datasource_list. The way I do it is by adding a file in my templates to the /etc/cloud/cloud.cfg.d directory with only CloudStack in it: echo "datasource_list: [ CloudStack ]" > etc/cloud/cloud.cfg.d/90_datasource

Re: [DISCUSS][PROPOSAL] Support for standard metadata/userdata URL

Would it not work to assign the address 169.254.169.254 on the VR? Erik Den torsdag 6. august 2015 skrev Chiradeep Vittal < chiradeep.vit...@citrix.com> følgende: > Today the metadata / userdata server is retrieved by the guest from > http:///latest/meta-data and > http:// of DHCP server>/latest

[GitHub] cloudstack pull request: CLOUDSTACK-8698: Retrieve a new device ID...

Github user mike-tutkowski commented on the pull request: https://github.com/apache/cloudstack/pull/660#issuecomment-128443925 Sounds good, @karuturi I'll go ahead and check the code in. Thanks for the comments. --- If your project is set up for it, you can reply to this email and ha

[GitHub] cloudstack pull request: CLOUDSTACK-8698: Retrieve a new device ID...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/660 --- 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

Re: Super trivial code change and PR

How's about this: Super-Trivial: :) On Thu, Aug 6, 2015 at 2:34 AM, Daan Hoogland wrote: > H Mike, > > If you find a good title for such a trivial change bucket pr it makes > creating release notes easier as well. Try to group them by mental state or > character (can you say that in English?)

[GitHub] cloudstack pull request: CLONE - Restore VM (Re-install VM) with e...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/659#issuecomment-128469563 LGTM. I tested it on a VM running on KVM and it works fine now with enable.storage.migration=false. The VM restore itself also still works. Please see my

Re: [DISCUSS][PROPOSAL] Support for standard metadata/userdata URL

Excellent tip! Didn’t notice this in the docs before: http://docs.cloudstack.apache.org/projects/cloudstack-administration/en/4.5/virtual_machines/user-data.html From: Carlos Reategui mailto:car...@reategui.com>> Reply-To: "dev@cloudstack.apache.org" mailto:dev@c

Re: [DISCUSS][PROPOSAL] Support for standard metadata/userdata URL

The Apache web server listens only on the eth0 IP. I added the 169.* address to eth0 and changed the Apache config to listen on that IP as well. While the Apache Webserver now receives the request, it returns a 404. Fiddling with the .htaccess file could fix that, but now that’s a bigger change t

[GitHub] cloudstack pull request: CLOUDSTACK-8704: Schedule restart of rout...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/656#issuecomment-128491021 @koushik-das Thanks for the fix! I was testing this on KVM tonight but run into an unrelated issue. Will try to verify your fix tomorrow so we can merge it soon.

Vr not working in master

Hi, We wanted to do some tests against master branch today. But we never got a VR up and running. Zone was in advanced networking, when creating a new vpc or private network the VR get created, vm start in xenserver but get shutdown and cloudstack console popup an error. cpvm and ssvm seams to wo

Off topic about fork on git hub and pull requests

Hi guys, I know this might not be the right place to post it, but I bet that you have the answer at the tip of your tongue. I have created a fork from https://github.com/apache/cloudstack, on that fork I have made some changes and created a pull request. The pull request was already accepted and m

[GitHub] cloudstack pull request: CLOUDSTACK-8714 Restore VM (Re-install VM...

Github user manuiiit commented on the pull request: https://github.com/apache/cloudstack/pull/659#issuecomment-128587836 @remibergsma It was an internal bug reported hence created a new issue and updated .Thanks for the review. --- If your project is set up for it, you can reply to

Re: Off topic about fork on git hub and pull requests

If the end goal is to just have the master branch on your fork equal to the master branch on origin, how's about fetching master on origin to your local system, then switching to your local master branch and executing "git reset --hard origin/master" (assuming you don't have any work in progress...

[GitHub] cloudstack pull request: CLOUDSTACK-8601. VMFS storage added as lo...

Github user mike-tutkowski commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/547#discussion_r36490233 --- Diff: core/src/com/cloud/agent/api/ModifyStoragePoolAnswer.java --- @@ -26,6 +26,7 @@ public class ModifyStoragePoolAnswer extends Answer {

Re: Vr not working in master

Can you please share the MS logs ? I think the VR might failed on configuring rules due to this VR stop. Observe the MS from the VR start onwards to VR stop. Thanks, Jayapal On 07-Aug-2015, at 3:18 AM, Pierre-Luc Dion wrote: > Hi, > > We wanted to do some tests against master branch today. Bu

[GitHub] cloudstack pull request: CLOUDSTACK-8601. VMFS storage added as lo...

Github user mike-tutkowski commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/547#discussion_r36490405 --- Diff: engine/schema/src/org/apache/cloudstack/storage/datastore/db/PrimaryDataStoreDaoImpl.java --- @@ -53,6 +53,7 @@ protected final

[GitHub] cloudstack pull request: CLOUDSTACK-8601. VMFS storage added as lo...

Github user mike-tutkowski commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/547#discussion_r36490618 --- Diff: engine/storage/volume/src/org/apache/cloudstack/storage/datastore/provider/DefaultHostListener.java --- @@ -71,6 +74,17 @@ public boolea

[GitHub] cloudstack pull request: CLOUDSTACK-8601. VMFS storage added as lo...

Github user mike-tutkowski commented on the pull request: https://github.com/apache/cloudstack/pull/547#issuecomment-128594438 Aside from a couple minor comments, LGTM. :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

[GitHub] cloudstack pull request: [WIP] Cloudstack 8656: do away with more ...

Github user mike-tutkowski commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/654#discussion_r36490932 --- Diff: services/console-proxy/server/src/com/cloud/consoleproxy/vnc/packet/server/RawRect.java --- @@ -50,26 +52,27 @@ public void paint(Buffer

[GitHub] cloudstack pull request: [WIP] Cloudstack 8656: do away with more ...

Github user mike-tutkowski commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/654#discussion_r36490966 --- Diff: utils/src/com/cloud/utils/nio/NioConnection.java --- @@ -334,6 +333,7 @@ protected void processTodos() { try

[GitHub] cloudstack pull request: [WIP] Cloudstack 8656: do away with more ...

Github user mike-tutkowski commented on the pull request: https://github.com/apache/cloudstack/pull/654#issuecomment-12859 Aside from my "break" question and indentation request, LGTM. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: Fixed Coverity issue "Dereference null re...

Github user manuiiit commented on the pull request: https://github.com/apache/cloudstack/pull/630#issuecomment-128602113 @mike-tutkowski I did not find any other possibility where createPhysicalDisk would return null unless is called from NfsManagedStorageAdaptor class.Please correct

[GitHub] cloudstack pull request: Fixed Coverity issue "Dereference null re...

Github user mike-tutkowski commented on the pull request: https://github.com/apache/cloudstack/pull/630#issuecomment-128604281 That is very well true, but I "worry" that a comment that specific would not be updated should the time come when another adaptor does, in fact, return null h

Mouting Dvd/Cd Rom from VM and Vm Networking

Hi All, I have installed Cloudstack 4.4 on Ubuntu 14.02 LTS. Have also created two vm's like Centos6.5 and Windows 7, both 64 bit. The Ubuntu 14.02 LTS is showing me a Realtek driver, when the lspci command is issued. When I go into VM1 of Centos 6.5, install it, detach the iso, and then reb

[GitHub] cloudstack pull request: Fixed Coverity issue "Dereference null re...

Github user manuiiit commented on the pull request: https://github.com/apache/cloudstack/pull/630#issuecomment-128614235 @mike-tutkowski I understood what u said.Updated.Will keep it in mind from next time.Thanks :) . --- If your project is set up for it, you can reply to this email