Re: Build failed in Jenkins: build-systemvm-master #624

ok. thanks. I will attempt a fix. ~Rajani On Tue, Sep 15, 2015 at 11:34 AM, Rohit Yadav wrote: > The ISO url needs to be fixed to use 7.9.0 as the 7.8.0 archive url is > not available. > http://cdimage.debian.org/cdimage/archive/7.9.0/ > > On 15-Sep-2015, at 9:59 am, Rajani Karuturi > wrote:

[GitHub] cloudstack pull request: Cherry-pick PR #795 into 4.5 branch.

Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/817#issuecomment-140301614 @bhaisaab Done. --- 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 pull request: CLOUDSTACK-8625: Systemd profile for Clou...

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/813#issuecomment-140303120 @bhaisaab Could you take a quick peek? --- 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 d

[GitHub] cloudstack pull request: CLOUDSTACK-8840: Systemd service for the ...

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/820#issuecomment-140305692 @remibergsma Yes, stupid. Fixed that. Pushed a new commit --- 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 pull request: CLOUDSTACK-8645: Improve logging of RBD f...

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/821#issuecomment-140306807 @DaanHoogland Pushed a fix for that log line --- 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 pro

[GitHub] cloudstack pull request: 4.5.3 upgradepath for master branch

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/824#issuecomment-140307070 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 enabled

[GitHub] cloudstack pull request: CLOUDSTACK-8645: Improve logging of RBD f...

Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/821#issuecomment-140308111 @wido Can you squash both commits into one? --- 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-8645: Improve logging of RBD f...

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/821#issuecomment-140315801 @borisroman Done! --- 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 fea

[GitHub] cloudstack pull request: CLOUDSTACK-8645: Improve logging of RBD f...

Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/821#issuecomment-140316281 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

[GitHub] cloudstack pull request: CLOUDSTACK-8826: XenServer - Use device i...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/792#issuecomment-140317652 Can anyone review 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 project does not

[GitHub] cloudstack pull request: Replaced all occurences of Charset.forNam...

GitHub user borisroman opened a pull request: https://github.com/apache/cloudstack/pull/825 Replaced all occurences of Charset.forName(UTF-8) with StringUtils.getPreferredCharset(). You can merge this pull request into a Git repository by running: $ git pull https://github.co

[GitHub] cloudstack pull request: CLOUDSTACK-8645: Improve logging of RBD f...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/821 --- 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-8834: Fixed unable to download...

Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/804#issuecomment-140329820 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 en

[GitHub] cloudstack pull request: CLOUDSTACK-8834: Fixed unable to download...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/804 --- 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: Replaced all occurences of Charset.forNam...

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/825#issuecomment-140332659 Pending travis 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 f

[GitHub] cloudstack pull request: Fixed: Error given when creating VPN user...

GitHub user kansal opened a pull request: https://github.com/apache/cloudstack/pull/826 Fixed: Error given when creating VPN user in one network if VR for an… …other network is stopped. Problem: Adding a VPN user to an IP on an isolated network fails with an exception i

[BLOCKER] - Guest VMs are not getting IPs as the DHCP port is not opened in VR

Hi, JIRA - ticket https://issues.apache.org/jira/browse/CLOUDSTACK-8843 Can someone please pick up this ticket ? This is failing a bunch of the BVTs tests in Basiczone. Thanks, Raja

[GitHub] cloudstack pull request: Replaced all occurences of Charset.forNam...

Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/825#issuecomment-140339356 Travis failed due to a time-out. --- 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 d

[GitHub] cloudstack pull request: Fixed: Network Update from RVR offering t...

Github user kansal commented on the pull request: https://github.com/apache/cloudstack/pull/818#issuecomment-140339650 @remibergsma Yes its CLOUDSTACK-8844. It's already syncing the comments in the JIRA issue. --- If your project is set up for it, you can reply to this email and hav

[GitHub] cloudstack pull request: BUG-ID: CLOUDSTACK-8849: Usage job should...

GitHub user yvsubhash opened a pull request: https://github.com/apache/cloudstack/pull/827 BUG-ID: CLOUDSTACK-8849: Usage job should stop usage generation in ca… …se of any exception You can merge this pull request into a Git repository by running: $ git pull https://github

[GitHub] cloudstack pull request: CLOUDSTACK-8854: Apple Mac OS/X VM get cr...

GitHub user sureshanaparti opened a pull request: https://github.com/apache/cloudstack/pull/828 CLOUDSTACK-8854: Apple Mac OS/X VM get created without USB controller in ESXi hypervisors CLOUDSTACK-8854: Apple Mac OS/X VM get created without USB controller in ESXi hypervisors

[GitHub] cloudstack pull request: CLOUDSTACK-8848: extra state to handle; n...

GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/829 CLOUDSTACK-8848: extra state to handle; no power report from host there no no real handling code now so this can be regarded as a DISCUSS item. It may well be an improvement over existing c

[GitHub] cloudstack pull request: CLOUDSTACK-8858: listVolumes API fails fo...

GitHub user sureshanaparti opened a pull request: https://github.com/apache/cloudstack/pull/830 CLOUDSTACK-8858: listVolumes API fails for a particular domain with NPE. CLOUDSTACK-8858: listVolumes API fails for a particular domain with NPE. Summary: listVolumes API fails wh

[GitHub] cloudstack pull request: 4.5.3 upgradepath for master branch

Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/824#discussion_r39497551 --- Diff: engine/schema/src/com/cloud/upgrade/dao/Upgrade453to460.java --- @@ -30,12 +30,12 @@ import com.cloud.utils.exception.CloudRuntimeExcep

[GitHub] cloudstack pull request: 4.5.3 upgradepath for master branch

Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/824#discussion_r39497595 --- Diff: engine/schema/src/com/cloud/upgrade/DatabaseUpgradeChecker.java --- @@ -120,149 +121,151 @@ public DatabaseUpgradeChecker() {

[GitHub] cloudstack pull request: 4.5.3 upgradepath for master branch

Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/824#discussion_r39498235 --- Diff: engine/schema/src/com/cloud/upgrade/dao/Upgrade453to460.java --- @@ -30,12 +30,12 @@ import com.cloud.utils.exception.CloudRuntimeException

[GitHub] cloudstack pull request: 4.5.3 upgradepath for master branch

Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/824#discussion_r39498281 --- Diff: engine/schema/src/com/cloud/upgrade/DatabaseUpgradeChecker.java --- @@ -120,149 +121,151 @@ public DatabaseUpgradeChecker() {

[GitHub] cloudstack pull request: 4.5.3 upgradepath for master branch

Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/824#discussion_r39498419 --- Diff: engine/schema/src/com/cloud/upgrade/DatabaseUpgradeChecker.java --- @@ -120,149 +121,151 @@ public DatabaseUpgradeChecker() {

[GitHub] cloudstack pull request: 4.5.3 upgradepath for master branch

Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/824#discussion_r39498765 --- Diff: engine/schema/src/com/cloud/upgrade/DatabaseUpgradeChecker.java --- @@ -120,149 +121,151 @@ public DatabaseUpgradeChecker() {

[GitHub] cloudstack pull request: CLOUDSTACK-8854: Apple Mac OS/X VM get cr...

Github user sateesh-chodapuneedi commented on the pull request: https://github.com/apache/cloudstack/pull/828#issuecomment-140357559 LGTM. Can you please share more details on testing you have performed? --- If your project is set up for it, you can reply to this email and have yo

[GitHub] cloudstack pull request: CLOUDSTACK-8826: XenServer - Use device i...

Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/792#discussion_r39499607 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java --- @@ -3407,6 +3390,18 @@ public void h

[GitHub] cloudstack pull request: CLOUDSTACK-8826: XenServer - Use device i...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/792#issuecomment-140359521 one remark, 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 t

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/714#issuecomment-140360164 @mike-tutkowski @DaanHoogland @karuturi @miguelaferreira @rafaelweingartner @pedro-martins First of all let me just get one thing clear: Rafael ment

[GitHub] cloudstack pull request: Replaced all occurences of Charset.forNam...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/825#issuecomment-140361064 LGTM, thanks @borisroman --- 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 n

[GitHub] cloudstack pull request: Replaced all occurences of Charset.forNam...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/825 --- 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: Changed variable s_logger to non-static a...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/714#issuecomment-140365763 @wilderrodrigues I commented: "travis timeouts are unrelated. analysis passes, LGTM". So it was compiled otherwise the analysis would not have passed. What I mi

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/778#issuecomment-140366106 @karuturi @remibergsma @rafaelweingartner @DaanHoogland @miguelaferreira Please, before proceeding with this PR see my comments on the previous (#71

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/778#issuecomment-140366714 @rafaelweingartner do you think "Re-pushing the PR 714" is a good commit header? I mean when someone goes thought the commit list looking for something,

Build failed in Jenkins: build-master-noredist #4653

See Changes: [Boris Schrijver] Replaced all occurences of Charset.forName(UTF-8) with StringUtils.getPreferredCharset(). -- [...truncated 6099 lines...] [debug] execute contextualize

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/714#issuecomment-140367124 Hi @DaanHoogland, what I don't understand is how travis compiled and 15 days ago @rafaelweingartner said that he forgot to push 1 file. Looking at t

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/714#issuecomment-140368629 and later we see:  We now found a po

Re: ACS 4.5 - volume snapshots NOT removed from CEPH (only from Secondaryt NFS and DB)

On 15-09-15 13:56, Dmytro Shevchenko wrote: > Hello Wido, I saw you updated this code again. Maybe you know, what > procedure for rebuilding this library in Apache maven repository? > Because here http://repo.maven.apache.org/maven2/com/ceph/rados/ still > present only old 0.1.4 version and it's

[GitHub] cloudstack pull request: CLOUDSTACK-8854: Apple Mac OS/X VM get cr...

Github user sureshanaparti commented on the pull request: https://github.com/apache/cloudstack/pull/828#issuecomment-140369704 @sateesh-chodapuneedi These changes were successfully tested on Apple hardware. When a VM is created with guest OS as Apple Mac OS, the USB controller (EHCI+U

[GitHub] cloudstack pull request: CLOUDSTACK-8854: Apple Mac OS/X VM get cr...

Github user sateesh-chodapuneedi commented on the pull request: https://github.com/apache/cloudstack/pull/828#issuecomment-140370176 @sureshanaparti that sounds good. Thanks for the patch. --- If your project is set up for it, you can reply to this email and have your reply appear

VPC router issue

Hi all, This morning I shared a message on our Slack channel about the current issue I’m facing with VPC routers. The problem was caught by the test_vpc_routers.py tests, which are unable to verify the following: * Create a VPC with 2 Tiers, 3 VMs, 2Pub IPs, and 1 ACL * Stop the VPC router * St

Re: [BLOCKER] - Guest VMs are not getting IPs as the DHCP port is not opened in VR

Assigned to myself. Thanks, Jayapal > On 15-Sep-2015, at 2:49 pm, Raja Pullela wrote: > > Hi, > > JIRA - ticket https://issues.apache.org/jira/browse/CLOUDSTACK-8843 > Can someone please pick up this ticket ? This is failing a bunch of the BVTs > tests in Basiczone. > > Thanks, > Raja

Re: [BLOCKER] - Guest VMs are not getting IPs as the DHCP port is not opened in VR

I have the feeling that we are fixing 1 issue and created 2. Please, make sure the routers tests are executed and green before creating the PR. I will do the same. Cheers, Wilder > On 15 Sep 2015, at 14:23, Jayapal Reddy Uradi > wrote: > > Assigned to myself. > > Thanks, > Jayapal >> On 15

Build failed in Jenkins: build-master-jdk18 #352

See Changes: [Boris Schrijver] Replaced all occurences of Charset.forName(UTF-8) with StringUtils.getPreferredCharset(). -- [...truncated 5515 lines...] [INFO] Apache CloudStack Plugin -

[GitHub] cloudstack pull request: Replaced all occurences of Charset.forNam...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/825#issuecomment-140378109 @borisroman @wido Was this PR compiled locally? Master is now broken. When I try to build I get: BrocadeVcsApi.java:[378,26] error: cannot

Re: [BLOCKER] - Guest VMs are not getting IPs as the DHCP port is not opened in VR

Wilder, That is true. All these issues are because of the VR refactoring. I don’t know what are the unit test cases run on VR refactor code. VR (shell) scripts are stabilized over years and now suddenly moved to new implementation. What we have figured out is that this new implementation is do

[GitHub] cloudstack pull request: Replaced all occurences of Charset.forNam...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/825#issuecomment-140378819 @wilderrodrigues If something breaks master, any committer can revert it in order to restore stability. I didn't verify yet, but feel free to revert and ping the

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/714#issuecomment-140379362 @wilderrodrigues I got your point, and I also hate when projects that I work start breaking as a consequence of bad/poor code. Here goes an explanation in

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/778#issuecomment-140380153 @wilderrodrigues thanks for your comments, I have just answered your comments on PR 714. I will work on a rebase today and push it again. @miguelaf

[GitHub] cloudstack pull request: Replaced all occurences of Charset.forNam...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/825#issuecomment-140380659 Reverted by commit 6e426fa46d8cda50ec76f33429320d4590701678 --- If your project is set up for it, you can reply to this email and have your reply appear on G

Reverted PR 825

Guys, Unfortunately, PR 825 broke Master. Due to that, Remi and I just reverted it. @Boris: please fix the error and reopen the PR. Cheers, Wilder

[GitHub] cloudstack pull request: Replaced all occurences of Charset.forNam...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/825#issuecomment-140388758 Building is passing again: [INFO] [INFO] BUILD SUCCESS [INFO] -

Re: [BLOCKER] - Guest VMs are not getting IPs as the DHCP port is not opened in VR

Hi Jayapal, I would not suggest to move back to the sceptres, but focus on fixing and for every fix test it very well so we get confidence. If we start calling scripts com the python code it will have a huge chance to become a real mess! Let’s stick to it and fix it and once it’s stable we can

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user mike-tutkowski commented on the pull request: https://github.com/apache/cloudstack/pull/714#issuecomment-140392192 "In this particular case, how is possible to get 2 LGTM without even compiling the project locally?" I was depending on the info Daan refers to and, a

[GitHub] cloudstack pull request: CLOUDSTACK-8834: Fixed unable to download...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/804#issuecomment-140395721 We got a :-1: from @DaanHoogland and still the PR was merged without any explanation from the author. Was it tested by anyone? ping @bhais

Re: VPC router issue

6 days ago test_pc_routers.py succeeded with PR 788: https://github.com/apache/cloudstack/pull/788 It’s in the test results I put there. I’m now testing Master to verify if it’s still occurring there. I tested the PR #808 (https://github.com/apache/cloudstack/pull/805), which had a bunch of t

[GitHub] cloudstack pull request: CLOUDSTACK-8834: Fixed unable to download...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/804#issuecomment-140400310 @wilderrodrigues Not sure if @DaanHoogland actually :-1: it, he said he wouldn't do that. It's confusing though ;-) --- If your project is set up for it, you ca

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user mike-tutkowski commented on the pull request: https://github.com/apache/cloudstack/pull/714#issuecomment-140402029 I could say this again, but it doesn't seem like it's a popular opinion with regards to CloudStack development: If we are really "late in the game" wi

[GitHub] cloudstack pull request: CLOUDSTACK-8850: revertSnapshot command d...

GitHub user yvsubhash opened a pull request: https://github.com/apache/cloudstack/pull/831 CLOUDSTACK-8850: revertSnapshot command does not work Updating the documentation before the api is actually supported You can merge this pull request into a Git repository by running: $ g

[GitHub] cloudstack pull request: CLOUDSTACK-8834: Fixed unable to download...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/804#issuecomment-140403283 @remibergsma indeed. Perhaps I have been through many untested PRs today and one more just got my brain in a fuzzy state. My argument remains as the

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/714#issuecomment-140403471 @mike-tutkowski **we** don't have an awesome suite of regression tests because **we** don't write them. And if so, we should not rely on the builds we have,

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/778#issuecomment-140403964 @rafaelweingartner I totally understand. Inspiration is not something that always comes when needed. Please do take your time. --- If your project is set up

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user mike-tutkowski commented on the pull request: https://github.com/apache/cloudstack/pull/714#issuecomment-140404617 Right, Miguel (as you and I were discussing for that other PR with regards to tests). My point then is we especially should not be putting in this muc

[GitHub] cloudstack pull request: Replaced all occurences of Charset.forNam...

GitHub user borisroman opened a pull request: https://github.com/apache/cloudstack/pull/832 Replaced all occurences of Charset.forName(UTF-8) with StringUtils.getPreferredCharset(). You can merge this pull request into a Git repository by running: $ git pull https://github.co

After box restart CS 4.5.2 fails to start

Hi All, Today I installed CS 4.5.2 on CentOS 6.7 and able to start the app successfully. All of a sudden the box reboot then while I started the cloudsatck it fails to start where I could see a different issue and ERROR on catalina.out ERROR == INFO [c.c.s.ConfigurationServerImpl] (main:nul

[GitHub] cloudstack pull request: CLOUDSTACK-8834: Fixed unable to download...

Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/804#issuecomment-140420588 @wilderrodrigues -> @DaanHoogland Didn't -1, he actually said the opposite. The screenshots where added to provide context (question from daan). I tested them to

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/714#issuecomment-140423615 I completely agree with you in this Mike. We should stop churning in features when we don't even have a proper build that prevents **syntax** errors! --- If

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user mike-tutkowski commented on the pull request: https://github.com/apache/cloudstack/pull/714#issuecomment-140424617 Maybe the compiler is overworked and just did a cursory glance and reported back "LGTM." ;) --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/778#issuecomment-140423091 @wilderrodrigues conflicts solved, I tried to do a maven install, to run all tests and compile everything, but that did not work, hence there is a class mi

[GitHub] cloudstack pull request: changed the systemvm base iso file from d...

GitHub user karuturi opened a pull request: https://github.com/apache/cloudstack/pull/833 changed the systemvm base iso file from debian-7.8.0 to debian-7.9.0 7.8.0 url is not available(404 Error) This fixes the jenkins http://jenkins.buildacloud.org/job/build-systemvm-mast

[GitHub] cloudstack pull request: changed the systemvm base iso file from d...

Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/833#issuecomment-140460941 I really dont know the implications of change it from debian-7.8.0 to debian-7.9.0. If there is a better fix available, please advise. --- If your project is set

Re: Build failed in Jenkins: build-systemvm-master #624

Rohit, Can you review https://github.com/apache/cloudstack/pull/833 ? Thanks, ~Rajani On Tue, Sep 15, 2015 at 12:45 PM, Rajani Karuturi wrote: > ok. thanks. I will attempt a fix. > > ~Rajani > > On Tue, Sep 15, 2015 at 11:34 AM, Rohit Yadav > wrote: > >> The ISO url needs to be fixed to use 7

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/778#issuecomment-140487166 @rafaelweingartner Thanks for the time you spent on this. The commit header is indeed better. It now conveys what is being done. If you happen to have to cha

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/778#issuecomment-140491359 Actually, the ticket is as simple as that. It was just an exercise to an intern that is working with me in my thesis. We are doing few very small

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/778#issuecomment-140493836 I do agree that prepending "s_" to static variables is a waste of characters. Modern IDEs will signal static variables for you. It's a bit like prepending (o

[GitHub] cloudstack pull request: CLOUDSTACK-8850: revertSnapshot command d...

Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/831#issuecomment-140508485 Please note revertSnapshot for KVM has been merged into master. --- If your project is set up for it, you can reply to this email and have your reply appear on G

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/778#issuecomment-140527418 I do agree with you that removing duplicated code, reducing cyclomatic complexity and writing test case is much better. However, those tasks require certai

[GitHub] cloudstack pull request: changed the systemvm base iso file from d...

Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/833#issuecomment-140554664 @karuturi Our mirror at PCextreme still hosts the Debian 7.8.0 image. http://debian.apt-get.eu/cd-images/7.8.0/i386/iso-cd/debian-7.8.0-i386-netinst.iso

Build failed in Jenkins: build-systemvm-master #625

See Changes: [anshul.gangwar] CLOUDSTACK-8834: Fixed unable to download Template , when in multi zones [wido] CLOUDSTACK-8645: Improve logging of RBD functionality in KVM [Boris Schrijver] Replaced all occurences of Charset

Re: After box restart CS 4.5.2 fails to start

This looks like a user privilege issue. Can you make sure that you are running cloudstack with enough privileges (sudo included). > On 15-Sep-2015, at 8:24 pm, Keerthiraja SJ wrote: > > Hi All, > > Today I installed CS 4.5.2 on CentOS 6.7 and able to start the app > successfully. > > All of a su

Re: Recurring snapshot taking full every time

Can you check in the SMlog on the concerned host if you see any error when the plugin is being invoked ? > On 14-Sep-2015, at 5:01 pm, raja sekhar wrote: > > Hi abhinandan, > > Thanks for your reply. > We are facing the issue only for root volumes in that host. The snapshot > recurring policy fo

Re: [BLOCKER] - Guest VMs are not getting IPs as the DHCP port is not opened in VR

Hi Wilder, CLOUDSTACK-8843 This ticket is actually NOT regression. It is not implemented/tested for the basic zone VR. What I have observed in the VR refactor is that sanity testing of components the code affects is not done. Now we are spe

[GitHub] cloudstack pull request: CLOUDSTACK-8834: Fixed unable to download...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/804#issuecomment-140624485 I think Daan did not say -1 or :-1: while unit tests are welcome, does not hurt to be merciful on one or few line changes if they indeed fix a bug --- If your proj

[GitHub] cloudstack pull request: changed the systemvm base iso file from d...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/833#issuecomment-140624609 Thanks @karuturi LGTM :+1: --- 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

[GitHub] cloudstack pull request: changed the systemvm base iso file from d...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/833#issuecomment-140624682 @karuturi no need to worry on the implications, except for openswan we don't have a version reliance of a packages; and for openswan we install a specific version

[GitHub] cloudstack pull request: changed the systemvm base iso file from d...

Github user harikrishna-patnala commented on the pull request: https://github.com/apache/cloudstack/pull/833#issuecomment-140624768 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 f

RE: VPC router issue

Hi Wilder, BTW, I just checked my setup and it was passing on Sep 9th and was regressed after that. Thanks for checking this out, Raja -Original Message- From: Wilder Rodrigues [mailto:wrodrig...@schubergphilis.com] Sent: Tuesday, September 15, 2015 7:12 PM To: dev@cloudstack.apache.or

[GitHub] cloudstack pull request: 4.5.3 upgradepath for master branch

Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/824#discussion_r39593369 --- Diff: engine/schema/src/com/cloud/upgrade/DatabaseUpgradeChecker.java --- @@ -120,149 +121,151 @@ public DatabaseUpgradeChecker() {

RE: [BLOCKER] - Guest VMs are not getting IPs as the DHCP port is not opened in VR

I agree with jayapal, so far none of the BVTs for the hypervisors are at 100% except for the simulator's. Also, changes/refactoring some of the very core functionality such as VR should be thought through and thoroughly tested before being pushed as the impact will be huge and stabilization w

Re: [BLOCKER] - Guest VMs are not getting IPs as the DHCP port is not opened in VR

I agree. Most of the blockers we see now are VR related. ~Rajani On 16-Sep-2015, at 10:15 am, Raja Pullela wrote: > I agree with jayapal, so far none of the BVTs for the hypervisors are at 100% > except for the simulator's. > Also, changes/refactoring some of the very core functionality s

[GitHub] cloudstack pull request: CLOUDSTACK-8820: Support for VMware vCent...

Github user sateesh-chodapuneedi commented on the pull request: https://github.com/apache/cloudstack/pull/806#issuecomment-140627846 @sureshanaparti code LGTM. I will test this patch today in my lab and will get back. --- If your project is set up for it, you can reply to this ema

[GitHub] cloudstack pull request: changed the systemvm base iso file from d...

Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/833#issuecomment-140628293 Thanks guys. Will merge in sometime. --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-8866 : restart.retry.interval ...

GitHub user maneesha-p opened a pull request: https://github.com/apache/cloudstack/pull/834 CLOUDSTACK-8866 : restart.retry.interval is being used instead of migrate.retry.interval during host maintenance The frequency at which Cloudstack tries to migrate the VMs is currently contr

[PROPOSAL] stable master and 4.6 release

Hi all, There is little progress on 4.6 blockers this week. To release 4.6, we all should come together and fix the blockers. In recent days, master has broken multiple times with compilation failures/regressions. If master breaks, time spent on figuring out what happened and fixing it is time *not

[GitHub] cloudstack pull request: CLOUDSTACK-8848: extra state to handle; n...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/829#issuecomment-140641376 @DaanHoogland this is for just master, or also applies for 4.5; if so please send a PR for 4.5 as well. Thanks. --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request: CLOUDSTACK-8820: Support for VMware vCent...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/806#issuecomment-140641828 Alright, let's wait for lab tests to confirm the patch and only then we should merge it. Thanks for helping test it @sateesh-chodapuneedi --- If your pro

  1   2   >