[GitHub] cloudstack issue #1601: CLOUDSTACK-9348: Reduce Nio selector wait time
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1601 Thanks @swill --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[VOTE] Split Marvin to its own repository
All, Based on a recent discussion thread [1], I want to start a voting thread to gather consensus around splitting Marvin from the CloudStack repository. On successful voting, we would extract and maintain Marvin as a separate library in a separate repository (example repository [2]) and various build/test systems such as Travis [3] can install it directly for usage with pip+git etc. Background: During the build process, a commands.xml generated to build apidocs is also used to generate CloudStack Cmd and Request classes are auto-generated, which is the only dependency why we needed Marvin and CloudStack together. The auto-generated cloudstackAPI module can be also generated against a live running CloudStack mgmt server which has api discovery (listApis) enabled. The integration tests will still be tied to a branch and will remain withing the repository. A PR [3] was sent to show that we can still execute tests using this approach, and this would finally allow us to build, release and use Marvin as an independent library. Vote will be open for 72 hours. For sanity in tallying the vote, can PMC members please be sure to indicate "(binding)" with their vote? [ ] +1 approve [ ] +0 no opinion [ ] -1 disapprove (and reason why) [1] http://markmail.org/thread/kiezqhjpz44hvrau [2] https://github.com/rhtyd/marvin [3] https://github.com/apache/cloudstack/pull/1599 Regards, Rohit Yadav
RE: [VOTE] Split Marvin to its own repository
Would it make sense to have tests include cloudstack versions along with 'other' requirements in their meta-data, so that they don't *have* to be tied to a branch. Kind regards, Paul Angus paul.an...@shapeblue.com www.shapeblue.com 53 Chandos Place, Covent Garden, London WC2N 4HSUK @shapeblue -Original Message- From: Rohit Yadav [mailto:bhais...@apache.org] Sent: 18 July 2016 10:44 To: dev@cloudstack.apache.org Subject: [VOTE] Split Marvin to its own repository All, Based on a recent discussion thread [1], I want to start a voting thread to gather consensus around splitting Marvin from the CloudStack repository. On successful voting, we would extract and maintain Marvin as a separate library in a separate repository (example repository [2]) and various build/test systems such as Travis [3] can install it directly for usage with pip+git etc. Background: During the build process, a commands.xml generated to build apidocs is also used to generate CloudStack Cmd and Request classes are auto-generated, which is the only dependency why we needed Marvin and CloudStack together. The auto-generated cloudstackAPI module can be also generated against a live running CloudStack mgmt server which has api discovery (listApis) enabled. The integration tests will still be tied to a branch and will remain withing the repository. A PR [3] was sent to show that we can still execute tests using this approach, and this would finally allow us to build, release and use Marvin as an independent library. Vote will be open for 72 hours. For sanity in tallying the vote, can PMC members please be sure to indicate "(binding)" with their vote? [ ] +1 approve [ ] +0 no opinion [ ] -1 disapprove (and reason why) [1] http://markmail.org/thread/kiezqhjpz44hvrau [2] https://github.com/rhtyd/marvin [3] https://github.com/apache/cloudstack/pull/1599 Regards, Rohit Yadav
Re: [VOTE] Split Marvin to its own repository
Hi I used to use marvin for setup simulator environments for using it as integration test environments (4.5-latest) for the Ansible CloudStack modules. It's been a while and I can not really remember exactly what it was caused it to fail but since a few weeks I was not able to setup such an environment. I think it was related to some "cypto" dependencies which didn't fit anymore. Would splitting marvin out help to make the setup more reliable for older cloudstack versions (e.g. 4.5 >=) as well? Thanks for clarification. Regards René On 07/18/2016 11:44 AM, Rohit Yadav wrote: > All, > > Based on a recent discussion thread [1], I want to start a voting thread to > gather consensus around splitting Marvin from the CloudStack repository. > > On successful voting, we would extract and maintain Marvin as a separate > library in a separate repository (example repository [2]) and various > build/test systems such as Travis [3] can install it directly for usage > with pip+git etc. > > Background: During the build process, a commands.xml generated to build > apidocs is also used to generate CloudStack Cmd and Request classes are > auto-generated, which is the only dependency why we needed Marvin and > CloudStack together. The auto-generated cloudstackAPI module can be also > generated against a live running CloudStack mgmt server which has api > discovery (listApis) enabled. The integration tests will still be tied to a > branch and will remain withing the repository. A PR [3] was sent to show > that we can still execute tests using this approach, and this would finally > allow us to build, release and use Marvin as an independent library. > > Vote will be open for 72 hours. > > For sanity in tallying the vote, can PMC members please be sure to indicate > "(binding)" with their vote? > > [ ] +1 approve > [ ] +0 no opinion > [ ] -1 disapprove (and reason why) > > [1] http://markmail.org/thread/kiezqhjpz44hvrau > [2] https://github.com/rhtyd/marvin > [3] https://github.com/apache/cloudstack/pull/1599 > > Regards, > Rohit Yadav >
[GitHub] cloudstack issue #1455: [CLOUDSTACK-9328]: Fix vlan issues from test suite t...
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/1455 @sanju1010 can you close and reopen this PR to kick off travis again? Thanks... --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1455: [CLOUDSTACK-9328]: Fix vlan issues from test suite t...
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/1455 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 0 Errors: 0 Duration: 3h 55m 23s ``` **Associated Uploads** **`/tmp/MarvinLogs/DeployDataCenter__Jul_18_2016_07_15_33_3UESMT:`** * [dc_entries.obj](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1455/tmp/MarvinLogs/DeployDataCenter__Jul_18_2016_07_15_33_3UESMT/dc_entries.obj) * [failed_plus_exceptions.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1455/tmp/MarvinLogs/DeployDataCenter__Jul_18_2016_07_15_33_3UESMT/failed_plus_exceptions.txt) * [runinfo.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1455/tmp/MarvinLogs/DeployDataCenter__Jul_18_2016_07_15_33_3UESMT/runinfo.txt) **`/tmp/MarvinLogs/test_network_33DRFL:`** * [failed_plus_exceptions.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1455/tmp/MarvinLogs/test_network_33DRFL/failed_plus_exceptions.txt) * [results.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1455/tmp/MarvinLogs/test_network_33DRFL/results.txt) * [runinfo.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1455/tmp/MarvinLogs/test_network_33DRFL/runinfo.txt) **`/tmp/MarvinLogs/test_vpc_routers_5DK6QE:`** * [failed_plus_exceptions.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1455/tmp/MarvinLogs/test_vpc_routers_5DK6QE/failed_plus_exceptions.txt) * [results.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1455/tmp/MarvinLogs/test_vpc_routers_5DK6QE/results.txt) * [runinfo.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1455/tmp/MarvinLogs/test_vpc_routers_5DK6QE/runinfo.txt) Uploads will be available until `2016-09-17 02:00:00 +0200 CEST` *Comment created by [`upr comment`](https://github.com/cloudops/upr).* --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1594: CLOUDSTACK-9407: vm_network_map table doesnt get cle...
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/1594 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 1 Errors: 0 Duration: 3h 52m 54s ``` **Summary of the problem(s):** ``` FAIL: Create a redundant VPC with 1 Tier, 1 VM, 1 ACL, 1 PF and test Network GC Nics -- Traceback (most recent call last): File "/data/git/cs2/cloudstack/test/integration/smoke/test_vpc_redundant.py", line 605, in test_04_rvpc_network_garbage_collector_nics self.check_routers_state(status_to_check="BACKUP", expected_count=2) File "/data/git/cs2/cloudstack/test/integration/smoke/test_vpc_redundant.py", line 353, in check_routers_state self.fail("Expected '%s' routers at state '%s', but found '%s'!" % (expected_count, status_to_check, cnts[vals.index(status_to_check)])) AssertionError: Expected '2' routers at state 'BACKUP', but found '1'! -- Additional details in: /tmp/MarvinLogs/test_network_2PRTDA/results.txt ``` **Associated Uploads** **`/tmp/MarvinLogs/DeployDataCenter__Jul_18_2016_07_15_50_GQQ4IO:`** * [dc_entries.obj](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1594/tmp/MarvinLogs/DeployDataCenter__Jul_18_2016_07_15_50_GQQ4IO/dc_entries.obj) * [failed_plus_exceptions.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1594/tmp/MarvinLogs/DeployDataCenter__Jul_18_2016_07_15_50_GQQ4IO/failed_plus_exceptions.txt) * [runinfo.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1594/tmp/MarvinLogs/DeployDataCenter__Jul_18_2016_07_15_50_GQQ4IO/runinfo.txt) **`/tmp/MarvinLogs/test_network_2PRTDA:`** * [failed_plus_exceptions.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1594/tmp/MarvinLogs/test_network_2PRTDA/failed_plus_exceptions.txt) * [results.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1594/tmp/MarvinLogs/test_network_2PRTDA/results.txt) * [runinfo.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1594/tmp/MarvinLogs/test_network_2PRTDA/runinfo.txt) **`/tmp/MarvinLogs/test_vpc_routers_NTI7U6:`** * [failed_plus_exceptions.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1594/tmp/MarvinLogs/test_vpc_routers_NTI7U6/failed_plus_exceptions.txt) * [results.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1594/tmp/MarvinLogs/test_vpc_routers_NTI7U6/results.txt) * [runinfo.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1594/tmp/MarvinLogs/test_vpc_routers_NTI7U6/runinfo.txt) Uploads will be available until `2016-09-17 02:00:00 +0200 CEST` *Comment created by [`upr comment`](https://github.com/cloudops/upr).* --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1594: CLOUDSTACK-9407: vm_network_map table doesnt get cle...
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/1594 @nvazquez I have fun this on two different CI environments with the same error coming up in both. Can you confirm this is not related to this change? Thanks... --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1608: Cleanup RBD contexts after exceptions to prevent pot...
Github user kiwiflyer commented on the issue: https://github.com/apache/cloudstack/pull/1608 We have QA'd the patch and confirm that it prevents the agent from crashing when an unexpected exception occurs from upstream RBD libs. 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 and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1480: CLOUDSTACK-9342: Site to Site VPN PFS not bei...
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1480 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1595: ui: show resize volume button to all users
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1595 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1455: [CLOUDSTACK-9328]: Fix vlan issues from test ...
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1455 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1608: Cleanup RBD contexts after exceptions to prev...
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1608 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1601: CLOUDSTACK-9348: Reduce Nio selector wait tim...
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1601 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1610: packaging: Add db.X.driver=jdbc:mysql to db.p...
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1610 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1609: [CLOUDSTACK-9430] Added fix for adding/editin...
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1609 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
Re: [VOTE] Apache Cloudstack 4.9.0 RC1
I have merged the following PRs in preparation for RC2: https://github.com/apache/cloudstack/pull/1610 https://github.com/apache/cloudstack/pull/1609 https://github.com/apache/cloudstack/pull/1608 https://github.com/apache/cloudstack/pull/1601 https://github.com/apache/cloudstack/pull/1595 https://github.com/apache/cloudstack/pull/1480 https://github.com/apache/cloudstack/pull/1455 I am doing a 3rd round of tests on the following PRs since they both showed the same error on two different CI environments on two consecutive runs: https://github.com/apache/cloudstack/pull/1594 https://github.com/apache/cloudstack/pull/1583 Any feedback on these last two would be appreciated. I would like to merge them before I push the RC2 if possible. I would like to cut RC2 tonight. Cheers, *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com *|* tw @CloudOps_ On Wed, Jul 13, 2016 at 2:54 AM, Milamber wrote: > > Please add this PR on your list. That will fix the spanish localization > display. Thanks > > https://github.com/apache/cloudstack/pull/1583 > > On 12/07/2016 19:17, Will Stevens wrote: > >> Thank you gentlemen. I have been following the 1608 PR from afar and I >> will review your list Wido. I will get my CI chugging with some of those >> PRs today. >> >> Thx, >> >> *Will STEVENS* >> Lead Developer >> >> *CloudOps* *| *Cloud Solutions Experts >> 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 >> w cloudops.com *|* tw @CloudOps_ >> >> On Tue, Jul 12, 2016 at 1:07 PM, Wido den Hollander >> wrote: >> >> Hi Will, >>> >>> These are on my list: >>> >>> - https://github.com/apache/cloudstack/pull/1610 >>> - https://github.com/apache/cloudstack/pull/1480 >>> - https://github.com/apache/cloudstack/pull/1608 >>> >>> Those 3 should fix all the -1 votes and discussion on the list. >>> >>> Wido >>> >>> Op 12 juli 2016 om 18:14 schreef Will Stevens : Thank you, so this RC has officially failed. Can you guys please review those PRs and give me code review so I can be confident with those PRs. >>> I >>> will run testing on those two PRs to make sure things don't break with >>> them. >>> Is there anything else that needs to be added to the next RC, please >>> speak >>> now so we can streamline this next RC. Thanks, *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com *|* tw @CloudOps_ On Tue, Jul 12, 2016 at 10:53 AM, Sean Lair >>> wrote: >>> Hi all, I vote -1 and would like to see the jdbc:mysql and site-to-site > vpn fixed in 4.9. > > https://github.com/apache/cloudstack/pull/1610 > https://github.com/apache/cloudstack/pull/1480 > > Thanks! > Sean > > -Original Message- > From: Wido den Hollander [mailto:w...@widodh.nl] > Sent: Tuesday, July 12, 2016 1:48 AM > To: Sean Lair ; dev@cloudstack.apache.org > Subject: RE: [VOTE] Apache Cloudstack 4.9.0 RC1 > > > Op 11 juli 2016 om 22:40 schreef Sean Lair : >> >> >> Hi all, >> >> One small comment since strongSwan didn't make it into 4.9. There is >> > still a very simple bug in enabling PFS for site-to-site VPNs. The > code >>> checks the Dead Peer Detection (DPD) variable instead of the PFS > variable >>> when determining whether or not to enable PFS for the site-to-site VPN. > >> Here is the 1-line of code that is broken. You can see how it >> > refers to >>> dpd to set pfs. > >> file.addeq(" pfs=%s" % CsHelper.bool_to_yn(obj['dpd'])) >> >> This pull request fixes the issue, but was not merged since we were >> > going to strongSwan. It would be nice if this bug fix was put into > 4.9.0 >>> https://github.com/apache/cloudstack/pull/1480 >> >> Would it make you a -1 for you without this PR? If so, please vote -1 > :) >>> Wido > > Thanks! >> Sean >> >> -Original Message- >> From: Will Stevens [mailto:williamstev...@gmail.com] >> Sent: Wednesday, July 6, 2016 3:52 PM >> To: dev@cloudstack.apache.org >> Subject: [VOTE] Apache Cloudstack 4.9.0 RC1 >> >> Hi All, >> >> I've created a 4.9.0 release, with the following artifacts up for a >> > vote: >>> Git Branch and Commit SH: >> >> >>> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.9.0-RC20160706T1546 >>> Commit: 643f75aa9150156b1fb05f339a338614fc7ad3fb >> >> I will be updating the Release Notes with the changes in this release >> > tomorrow. If the RC changes, I can adapt the release notes after. > >> Source release (checksums and signatures are available at the same >> location): >> https://dist.a
[GitHub] cloudstack issue #1583: Update L10N resource files with 4.9 strings from Tra...
Github user milamberspace commented on the issue: https://github.com/apache/cloudstack/pull/1583 @swill I do not see why changes on the L10N resource files have an impact on the VPC behavior ... This looks like a false positive ... --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1583: Update L10N resource files with 4.9 strings from Tra...
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/1583 Thanks, I couldn't either, but wanted to get a second opinion. I am running CI against it for the 3rd time to see if the same thing pops again. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
Starting jetty on another port
Hi, Is there a way to start Jetty on a port other than :8080? I've tried with mvn -pl :cloud-client-ui jetty:run -Djetty.port=8081 and it did not work. I need this because the machine where I am doing my dev is alreay running a service on port 8080 and I can't change that. -Syed
Re: Starting jetty on another port
Probably a question that could be asked on the user mailing list. On 18/07/2016 3:41 PM, Syed Mushtaq wrote: Hi, Is there a way to start Jetty on a port other than :8080? I've tried with mvn -pl :cloud-client-ui jetty:run -Djetty.port=8081 and it did not work. I need this because the machine where I am doing my dev is alreay running a service on port 8080 and I can't change that. -Syed What error did you get? Are you sure that 8081 is free? Ron -- Ron Wheeler President Artifact Software Inc email: rwhee...@artifact-software.com skype: ronaldmwheeler phone: 866-970-2435, ext 102
Re: Starting jetty on another port
Try changing in client/pom.xmlhttps://github.com/apache/cloudstack/blob/master/client/pom.xml#L459 ~ Rajanihttp://cloudplatform.accelerite.com/ On July 19, 2016 at 1:11 AM, Syed Mushtaq (syed1.mush...@gmail.com) wrote:Hi, Is there a way to start Jetty on a port other than :8080? I've tried with mvn -pl :cloud-client-ui jetty:run -Djetty.port=8081 and it did not work. I need this because the machine where I am doing mydev is alreayrunning a service on port 8080 and I can't change that. -Syed