cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From chrismattmann <...@git.apache.org>
Subject [GitHub] cloudstack pull request: Pull Request - ignore -
Date Wed, 16 Mar 2016 18:48:34 GMT
GitHub user chrismattmann opened a pull request:

    https://github.com/apache/cloudstack/pull/1442

    Pull Request - ignore - 

    Why in the world is Apache Cloudstack creating an organization externally on Github? At
a minimum, you can't use the Apache Cloudstack logo? 

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/apache/cloudstack 4.8

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/1442.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1442
    
----
commit acfc19dc8285a07412ee0078fbc13d7319d5be8b
Author: Wei Zhou <w.zhou@tech.leaseweb.com>
Date:   2015-12-10T12:26:02Z

    CLOUDSTACK-9134: set device_id as the first device_id not in use instead of nic count
    
    when we restart vpc tiers, the old nics will be removed, and create a new nic.
    however, the device_id was set to the nic count, which may be already used.
    this commit get the first device_id not in use as the device_id of new nic.
    
    This issue also happen when we add multiple networks to a vm and remove them.

commit 709e0c555e5896f5f2abe16f9ba029010b4f42cc
Author: Wei Zhou <w.zhou@tech.leaseweb.com>
Date:   2015-12-10T13:09:45Z

    [UI] bug fix: Delete added ACL lists is not available for Domain Admin and normal users

commit 45f1e8d64fde1dd2330dd7a092f8f0fc3630b876
Author: Remi Bergsma <github@remi.nl>
Date:   2015-12-10T13:51:54Z

    Merge pull request #1208 from shapeblue/master-quota-greyicon
    
    ui/quota: Make the quota UI plugin icon grayMakes the quota ui plugin icon gray, based
on comment at:
    https://github.com/apache/cloudstack/pull/768#issuecomment-163364606
    
    * pr/1208:
      ui/quota: Make the quota UI plugin icon gray
    
    Signed-off-by: Remi Bergsma <github@remi.nl>

commit b14dfca82d5446cf3a7a473186514fc979089030
Author: Remi Bergsma <github@remi.nl>
Date:   2015-12-10T15:19:41Z

    Merge pull request #1187 from nlivens/4.6_test_improvement
    
    Adapted HypervisorUtilsTest to no longer give false positives
    
    * pr/1187:
      Adapted HypervisorUtilsTest to no longer give false positives
    
    This closes #1185
    
    Signed-off-by: Remi Bergsma <github@remi.nl>

commit 312b9aff9490214c51cf6adf90661262c3670cd3
Author: Remi Bergsma <github@remi.nl>
Date:   2015-12-10T15:24:36Z

    Merge release branch 4.6 to master
    
    * 4.6:
      Adapted HypervisorUtilsTest to no longer give false positives

commit 51c9f0e3b524f978eadb6a6ba630c3f7281b1a11
Author: Wilder Rodrigues <wrodrigues@schubergphilis.com>
Date:   2015-12-10T18:51:28Z

    CLOUDSTACK-9138 - Adds multiple providers back to VPC implementation
    
       - It is need and already allowed/used in the current implementation. For example, the
Default [redundant] VPC offerings use
         two LB providers. If we cannot create offerings with 2 LB providers, the whole internal
loadbalancer implementation won't work

commit 0d075906ad40b0bd681ece777045684379ba93a5
Author: Wilder Rodrigues <wrodrigues@schubergphilis.com>
Date:   2015-12-11T06:51:22Z

    CLOUDSTACK-4374 - Adds HA capabilities to redundant routers
    
       - Redundant Routers are not the same as Haigh Available.
         Having a router HA is actually saying that the given router will be controlled by
the High Availability monitor.
         Hence fix any problem we might face.

commit 1e17c3e38cf70d965d0fc84cbf67f78ef5203e71
Author: Abhinandan Prateek <abhinandan.prateek@shapeblue.com>
Date:   2015-12-08T04:29:06Z

    QUOTA: Ensuring that the dates displayed are as per user expectations
    
        When querying db we use start of next day to query quota usage for
        today, but while displaying it to user we still need to show it as
        todays date

commit bc8994ba1c24f68d24576986d29ff8322d00a8ed
Author: Abhinandan Prateek <abhinandan.prateek@shapeblue.com>
Date:   2015-12-08T11:51:00Z

    CLOUDSTACK-9122: latest credit entries should be incorporated when the balance is
    calculated

commit 67b4a7bd7827fea4923cba5b4e96cf5f399568ee
Author: Remi Bergsma <github@remi.nl>
Date:   2015-12-10T16:50:45Z

    Setup routes for RFC 1918 ip space
    
    Setup general route for RFC 1918 space, as otherwise it will be sent to
    the public gateway and not work. More specific routes that may be set
    have preference over this generic routes.
    
    When public network is RFC1918, we do not setup the routes to avoid
    problems with internal-only deployments.

commit 9b89dfd6438b8a669617c64c82198c92443a29d8
Author: Daan Hoogland <daan@onecht.net>
Date:   2015-12-11T10:37:42Z

    CLOUDSTACK-9139 make zwps default when defined on zone level

commit 590af145b0c703f9c957c6f81ef450856dc2c407
Author: Wilder Rodrigues <wrodrigues@schubergphilis.com>
Date:   2015-12-11T14:02:55Z

    CLOUDSTACK-9135 - Makes possible to add multiple providers per service
    
       - Same is done in the Java side. Check the createVpcOffering() method in the VpcManagerImpl
class

commit 43ce049d43ce7aa0e4d25b0630cbcd1c423156b9
Author: Wilder Rodrigues <wrodrigues@schubergphilis.com>
Date:   2015-12-11T14:03:50Z

    CLOUDSTACK-9135 - Makes test compliant with Redundant VPCs

commit 1ed5c209086aa38da1f75881056cf2d6d6c1afc9
Author: Remi Bergsma <github@remi.nl>
Date:   2015-12-11T16:14:42Z

    Merge pull request #1197 from ustcweizhou/vpc-router-by-keyword
    
    [4.7] CLOUDSTACK-9129: list vpc routers by keyword in Infrastructure -> Virtual Routers
    and two more changes:
    (1) add network name/vpc name in the listRouters response
    (2) add network name/vpc id, vpc name in the router details page
    
    * pr/1197:
      CLOUDSTACK-9129: list vpc routers by keyword in Infrastructure -> Virtual Routers
    
    Signed-off-by: Remi Bergsma <github@remi.nl>

commit c9985e96a5446395f402faccd422935d24b7c933
Author: Remi Bergsma <github@remi.nl>
Date:   2015-12-11T18:42:36Z

    Merge pull request #1214 from remibergsma/rfc1918_route
    
    CLOUDSTACK-9143 Setup routes for RFC 1918 ip spaceSetup general route for RFC 1918 space,
as otherwise it will be sent to the public gateway and likely to be dropped (internet providers
do not route ip space that is meant for internal use). More specific routes that may be set
have preference over this generic routes so this works even with private ranges used for public
ip space (as shown below).
    
    When using an internal DNS server some hosts may resolve to an RFC 1918 ip address. The
SSVM has a default gw to public so if it has no route for this ip address space, it will not
work. This PR makes generic RFC 1918 (so all internal ip adresses like 10.0.0.10 etc) to the
local management gateway. This makes them reachable. Without this fix, it is sent upstream
and it is dropped there.
    
    Should there be a more generic route (smaller prefix), this has preference over the generic
routes.
    
    Example in my dev environment:
    
    ```
    root@v-1-VM:~# route -n
    Kernel IP routing table
    Destination     Gateway         Genmask         Flags Metric Ref    Use Iface
    0.0.0.0         192.168.23.1    0.0.0.0         UG    0      0        0 eth2
    10.0.0.0        192.168.22.1    255.0.0.0       UG    0      0        0 eth1
    169.254.0.0     0.0.0.0         255.255.0.0     U     0      0        0 eth0
    172.16.0.0      192.168.22.1    255.240.0.0     UG    0      0        0 eth1
    192.168.0.0     192.168.22.1    255.255.0.0     UG    0      0        0 eth1
    192.168.22.0    0.0.0.0         255.255.255.0   U     0      0        0 eth1
    192.168.23.0    0.0.0.0         255.255.255.0   U     0      0        0 eth2
    ```
    
    Route `192.168.0.0/16` goes via `eth1` but `192.168.23.0/24` is more specific and has
preference and goes via `eth2`. It works:
    
    ```
    root@v-1-VM:~# ping 8.8.8.8
    PING 8.8.8.8 (8.8.8.8): 48 data bytes
    56 bytes from 8.8.8.8: icmp_seq=0 ttl=49 time=7.179 ms
    ^C--- 8.8.8.8 ping statistics ---
    1 packets transmitted, 1 packets received, 0% packet loss
    round-trip min/avg/max/stddev = 7.179/7.179/7.179/0.000 ms
    ```
    
    This solves a lot of the 'internal resolving' issues we face.
    
    When the public ip address is RFC1918 itself, we do not set the routes.
    
    * pr/1214:
      Setup routes for RFC 1918 ip space
    
    Signed-off-by: Remi Bergsma <github@remi.nl>

commit 6be2cc78c7585659a58d4715fc7082426c4192be
Author: Remi Bergsma <github@remi.nl>
Date:   2015-12-11T18:45:14Z

    Merge pull request #1215 from ekholabs/improve/mult-providers-CLOUDSTACK-9138
    
    CLOUDSTACK-9138 - Adds multiple providers back to VPC implementation   - It is need and
already allowed/used in the current implementation. For example, the Default [redundant] VPC
offerings use two LB providers. If we cannot create offerings with 2 LB providers, the whole
internal loadbalancer implementation won't work.
    
    * pr/1215:
      CLOUDSTACK-9138 - Adds multiple providers back to VPC implementation
    
    Signed-off-by: Remi Bergsma <github@remi.nl>

commit 879b6da4eda10af9a1a42018a4713ad85301a327
Author: Remi Bergsma <github@remi.nl>
Date:   2015-12-12T08:33:54Z

    Merge pull request #1192 from shapeblue/master-9122
    
    CLOUDSTACK-9122:  latest credit entries should be incorporated when thefuture credit entries
should be incorporated when the balance is calculated
    
    1. Some code is refactored so that unit tests (TODO in future) are more comprehensive
    2. The fix is to incorporate the credit entries on line 228.
    
    https://issues.apache.org/jira/browse/CLOUDSTACK-9122
    
    * pr/1192:
      CLOUDSTACK-9122: latest credit entries should be incorporated when the balance is calculated
    
    Signed-off-by: Remi Bergsma <github@remi.nl>

commit 66d7f413e8e67c83727ca159495df8f4a8dc1bd0
Author: Remi Bergsma <github@remi.nl>
Date:   2015-12-12T08:37:05Z

    Merge pull request #1188 from kansal/CLOUDSTACK-9086
    
    CLOUDSTACK-9086: ACS allows to create isolated networks with invalide gateway IP address
- Fixed and Test cases added
    
    Problem: There was no check for the network and broadcast IP addresses in the case where
we provide the gateway and netmask while creating the isolated network. As a result the provided
IP gets assigned to the eth0 interface of the VR.
    
    Note: This is in continuation of PR #1125 which I closed.
    
    * pr/1188:
      CLOUDSTACK-9086: ACS allows to create isolated networks with invalide gateway IP address
- Fixed and Test cases added
    
    Signed-off-by: Remi Bergsma <github@remi.nl>

commit 2071a9e395841a896f9cef3d80aa9b31822d8fa7
Author: Remi Bergsma <github@remi.nl>
Date:   2015-12-12T08:53:00Z

    Merge pull request #1222 from ekholabs/fix/rvr-ha-CLOUDSTACK-4374
    
    CLOUDSTACK-4374 - As a Developer I want to have HA enabled for routers that are part or
a redundant network or VPCThis PR fixes the issue we were facing with crashed redundant routers
not being picked up by the HA monitor due to an IF statement in the code.
    
    * Redundant Routers are not the same as Haigh Available. Having a router HA is actually
saying that the given router will be controlled by the High Availability monitor. Hence fix
any problem we might face.
    
    * pr/1222:
      CLOUDSTACK-4374 - Adds HA capabilities to redundant routers
    
    Signed-off-by: Remi Bergsma <github@remi.nl>

commit 52edbbb6cc2e2c58e360b737143c10adddc43880
Author: Remi Bergsma <github@remi.nl>
Date:   2015-12-12T10:00:41Z

    Show actual diff in commits after merge

commit 7fa62a576523ad7740563d7626b4caec6c040735
Author: Remi Bergsma <github@remi.nl>
Date:   2015-12-12T10:29:28Z

    Merge pull request #1221 from ekholabs/improve/test-internal-lb-CLOUDSTACK-9135
    
    CLOUDSTACK-9135 - As a Developer I want the test_internal_lb.py to test Redundant VPCsThis
PR refactors the existing tests and adds tests to cover internal LB with Redundant VPCs.
    
    In order to make it possible I had to change the base.py, allowing multiple providers
per service. This change requires the changes in the Java side which were dealt with PR #1215
.
    
    * pr/1221:
      CLOUDSTACK-9135 - Makes test compliant with Redundant VPCs
      CLOUDSTACK-9135 - Makes possible to add multiple providers per service
      CLOUDSTACK-9135 - Moves test into smoke directory
    
    Signed-off-by: Remi Bergsma <github@remi.nl>

commit 61ebb53e17f225fc17b9011332cd9ae344db4350
Author: Remi Bergsma <github@remi.nl>
Date:   2015-12-12T12:55:20Z

    Merge pull request #1227 from remibergsma/git-pr-show-commits
    
    Show actual diff in commits after merge with git-pr / git-fwd-mergeThis shows the diff
in commits after using `git-pr` and `git-fwd-merge` tools, like this:
    
    ```
    44e8c92 Merge pull request #1226 from borisroman/CLOUDSTACK-9148
    0554610 Removed .pydevproject from plugin kvm hypervisor.
    ```
    
    That helps in quickly seeing what change is applied and whether it makes sense. It doesn't
touch the actual code base. I'm using this patch for some time already.
    
    * pr/1227:
      Show actual diff in commits after merge
    
    Signed-off-by: Remi Bergsma <github@remi.nl>

commit 65bf2edfa196601d68e82f09012a393ee73499a2
Author: Remi Bergsma <github@remi.nl>
Date:   2015-12-12T12:58:48Z

    Merge pull request #1182 from ustcweizhou/CLOUDSTACK-9113-getVmStats
    
    [4.6] CLOUDSTACK-9113: skip vm with inconsistent state when getVmStats/getVmDiskStatson
KVM, if there is a vm has inconsistent state between hypervisor and db, the getVmStat will
terminate and return null, all vm stats will not be updated.
    we should skip the vm which has inconsistent state, and continue on others.
    
    * pr/1182:
      CLOUDSTACK-9113: skip vm with inconsistent state when getVmStats/getVmDiskStats
    
    Signed-off-by: Remi Bergsma <github@remi.nl>

commit bdaa60fd6c04f0e8d08bf889a11702eb6017ad53
Author: Remi Bergsma <github@remi.nl>
Date:   2015-12-12T13:03:43Z

    Merge release branch 4.6 to master
    
    * 4.6:
      Show actual diff in commits after merge
      CLOUDSTACK-9113: skip vm with inconsistent state when getVmStats/getVmDiskStats

commit 8c78f89c91cf139b81110f80fbfede89a6f3df8b
Author: Daan Hoogland <daan@onecht.net>
Date:   2015-12-12T13:38:27Z

    Merge pull request #1132 from shapeblue/4.6-vmware-diskcontrollers
    
    [4.6] CLOUDSTACK-4787 - vmware diskcontrollersSame as #1131 (see this for screenshots
etc)
    
    * pr/1132:
      CLOUDSTACK-4787: Allow users to select disk controller for VM/template
      CLOUDSTACK-4787 Allow selection of scsi controller type in vSphere
    
    Signed-off-by: Daan Hoogland <daan@onecht.net>

commit 1597a4c749c98383b838d88e8307eebc370012f4
Author: Remi Bergsma <github@remi.nl>
Date:   2015-12-12T13:40:16Z

    Merge pull request #1211 from ustcweizhou/delete-acl-items
    
    [UI] bug fix: Delete added ACL lists is not available for Domain Admin and normal usersOnly
the owner or domain admin/admin can access the page (vpc->Network ACL Lists), they are
also able to remove the network ACL lists, which is not allowed on UI.
    
    * pr/1211:
      [UI] bug fix: Delete added ACL lists is not available for Domain Admin and normal users
    
    Signed-off-by: Remi Bergsma <github@remi.nl>

commit 7f2bbcbfff909ad4bb155f6db1fa923794e85ef2
Author: Boris Schrijver <boris@pcextreme.nl>
Date:   2015-12-12T13:57:51Z

    Removed cloud-cli folder and contents, as it is not maintained or used anymore.

commit 27e23fb11c80e14518afc48c91d5a18977d898c0
Author: Boris Schrijver <boris@pcextreme.nl>
Date:   2015-12-12T14:13:48Z

    Remove docs/.tx/config
    The config file docs/.tx/config has been replaced with tools/transifex/.tx/config. It's
not maintained or used so it must be removed.

commit 2bebb7f8a38c3ef94bbd55c69d7ec1070450dc22
Author: Wilder Rodrigues <wrodrigues@schubergphilis.com>
Date:   2015-12-12T15:26:00Z

    CLOUDSTACK-9151 - Removes the replacement of the VRID in the CsRedundant file
    
       - Just like with RVRs, use the VRID 51 instead of making it dependent on the VPCID
       - Reason: arbitary unique number 0..255 used to differentiate multiple instances of
vrrpd running on the same NIC (and hence same socket). virtual_router_id 51

commit 6947afee3aa1d4ed2dd8a118cd51f8d0e4d2de67
Author: Milamber <milamber@apache.org>
Date:   2015-12-09T09:33:14Z

    Update L10N resource files with 4.7 strings from Transifex (20151211)

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message