[GitHub] cloudstack issue #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2017-01-18 Thread vilisseranen 
Github user vilisseranen commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
This PR was not working as expected. See PR from @remibergsma for a proper 
fix.


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2017-01-16 Thread swill 
Github user swill commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
We have proven that the fix in this PR does not work.  I have removed it 
from #1741.

A new PR which we have proven does work is at: #1907


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2017-01-16 Thread swill 
Github user swill commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@vilisseranen can you close this PR?


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-12-09 Thread swill 
Github user swill commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
The merge conflict between this PR and #1741 is a bit tricky, so I have 
included this fix in #1741.  @vilisseranen and I are currently running both PRs 
in production so we had to resolve the merge conflict for our deployment.  
Since the resolution is not trivial, I thought it made sense to just include 
this fix in my PR.


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-27 Thread rhtyd 
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@murali-reddy can you help review this? 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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-23 Thread rhtyd 
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@vilisseranen @nathanejohnson can you consider suggestion from @remibergsma 
and apply suitable changes, 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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-20 Thread blueorangutan 
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-195


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-20 Thread blueorangutan 
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-20 Thread rhtyd 
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@blueorangutan package

@nathanejohnson @vilisseranen can we have a conclusion on this PR, 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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-16 Thread remibergsma 
Github user remibergsma commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@pdion891 Sorry for being late to the party. We've seen this issue too and 
resolved it in a different way. Removing the ip from the databag probably won't 
work too well, as it is used to deprovision the old ip address from the router. 
 This is what works well for us instead: 
https://github.com/MissionCriticalCloud/cosmic/pull/114/files



---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-09 Thread jburwell 
Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@murali-reddy should we merge #1659 and then re-base this PR before running 
more tests?


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-09 Thread blueorangutan 
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
Trillian test result (tid-315)
Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 36714 seconds
Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1706-t315-vmware-55u3.zip
Test completed. 44 look ok, 4 have error(s)


Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | `Failure` | 
41.83 | test_network.py
test_network_rules_acquired_public_ip_2_nat_rule | `Failure` | 31.59 | 
test_network.py
test_network_rules_acquired_public_ip_1_static_nat_rule | `Failure` | 94.90 
| test_network.py
test_01_vpc_site2site_vpn | `Error` | 486.90 | test_vpc_vpn.py
test_01_redundant_vpc_site2site_vpn | `Error` | 743.77 | test_vpc_vpn.py
test_06_stop_cpvm | `Error` | 415.91 | test_ssvm.py
test_01_snapshot_root_disk | `Error` | 15.22 | test_snapshots.py
ContextSuite context=TestSnapshotRootDisk>:teardown | `Error` | 65.58 | 
test_snapshots.py
test_01_vpc_remote_access_vpn | Success | 146.77 | test_vpc_vpn.py
test_02_VPC_default_routes | Success | 315.46 | test_vpc_router_nics.py
test_01_VPC_nics_after_destroy | Success | 747.72 | test_vpc_router_nics.py
test_05_rvpc_multi_tiers | Success | 683.07 | test_vpc_redundant.py
test_04_rvpc_network_garbage_collector_nics | Success | 1554.80 | 
test_vpc_redundant.py
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
Success | 680.07 | test_vpc_redundant.py
test_02_redundant_VPC_default_routes | Success | 719.78 | 
test_vpc_redundant.py
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1473.36 | 
test_vpc_redundant.py
test_09_delete_detached_volume | Success | 25.98 | test_volumes.py
test_06_download_detached_volume | Success | 60.67 | test_volumes.py
test_05_detach_volume | Success | 100.27 | test_volumes.py
test_04_delete_attached_volume | Success | 10.22 | test_volumes.py
test_03_download_attached_volume | Success | 20.39 | test_volumes.py
test_02_attach_volume | Success | 53.78 | test_volumes.py
test_01_create_volume | Success | 460.61 | test_volumes.py
test_03_delete_vm_snapshots | Success | 280.27 | test_vm_snapshots.py
test_02_revert_vm_snapshots | Success | 237.15 | test_vm_snapshots.py
test_01_test_vm_volume_snapshot | Success | 146.47 | test_vm_snapshots.py
test_01_create_vm_snapshots | Success | 161.87 | test_vm_snapshots.py
test_deploy_vm_multiple | Success | 253.77 | test_vm_life_cycle.py
test_deploy_vm | Success | 0.03 | test_vm_life_cycle.py
test_advZoneVirtualRouter | Success | 0.03 | test_vm_life_cycle.py
test_10_attachAndDetach_iso | Success | 26.94 | test_vm_life_cycle.py
test_09_expunge_vm | Success | 125.20 | test_vm_life_cycle.py
test_08_migrate_vm | Success | 71.09 | test_vm_life_cycle.py
test_07_restore_vm | Success | 0.10 | test_vm_life_cycle.py
test_06_destroy_vm | Success | 10.16 | test_vm_life_cycle.py
test_03_reboot_vm | Success | 5.15 | test_vm_life_cycle.py
test_02_start_vm | Success | 20.26 | test_vm_life_cycle.py
test_01_stop_vm | Success | 10.16 | test_vm_life_cycle.py
test_CreateTemplateWithDuplicateName | Success | 262.25 | test_templates.py
test_08_list_system_templates | Success | 0.03 | test_templates.py
test_07_list_public_templates | Success | 0.04 | test_templates.py
test_05_template_permissions | Success | 0.06 | test_templates.py
test_04_extract_template | Success | 15.39 | test_templates.py
test_03_delete_template | Success | 5.11 | test_templates.py
test_02_edit_template | Success | 90.17 | test_templates.py
test_01_create_template | Success | 126.13 | test_templates.py
test_10_destroy_cpvm | Success | 328.61 | test_ssvm.py
test_09_destroy_ssvm | Success | 299.93 | test_ssvm.py
test_08_reboot_cpvm | Success | 186.65 | test_ssvm.py
test_07_reboot_ssvm | Success | 158.82 | test_ssvm.py
test_05_stop_ssvm | Success | 183.74 | test_ssvm.py
test_04_cpvm_internals | Success | 1.41 | test_ssvm.py
test_03_ssvm_internals | Success | 4.06 | test_ssvm.py
test_02_list_cpvm_vm | Success | 0.13 | test_ssvm.py
test_01_list_sec_storage_vm | Success | 0.13 | test_ssvm.py
test_04_change_offering_small | Success | 96.90 | test_service_offerings.py
test_03_delete_service_offering | Success | 0.04 | test_service_offerings.py
test_02_edit_service_offering | Success | 0.08 | test_service_offerings.py
test_01_create_service_offering | Success | 0.09 | test_service_offerings.py
test_02_sys_template_ready | Success | 0.14 | test_secondary_storage.py
test_01_sys_vm_start | Success | 0.25 | test_secondary_storage.py
test_09_reboot_router | Success | 120.82 | test_routers.py
test_08_start_router | Success |

[GitHub] cloudstack issue #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-09 Thread murali-reddy 
Github user murali-reddy commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@jburwell #1659 has a fix which complements this patch.Basically when first 
IP is associated with an interface routing table is created and should be 
deleted when last IP associated is removed. #1659 handles that case.

@vilisseranen  could you please see if there is regression. Looks like IP 
may be getting removed from ips.json data bag, but not really getting 
dis-associated from the interface. Failure in below tests indicate so.

test_network_rules_acquired_public_ip_3_Load_Balancer_Rule  Failure 36.36   
test_network.py
test_network_rules_acquired_public_ip_2_nat_ruleFailure 31.31   
test_network.py
est_network_rules_acquired_public_ip_1_static_nat_rule  Failure 93.29   
test_network.py




---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-08 Thread blueorangutan 
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been 
kicked to run smoke tests


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-08 Thread rhtyd 
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@blueorangutan test


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-08 Thread blueorangutan 
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-152


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-08 Thread blueorangutan 
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-08 Thread rhtyd 
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@blueorangutan package


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-03 Thread blueorangutan 
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
Trillian test result (tid-269)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 24772 seconds
Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1706-t269-kvm-centos7.zip
Test completed. 45 look ok, 3 have error(s)


Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_02_redundant_VPC_default_routes | `Failure` | 845.38 | 
test_vpc_redundant.py
test_oobm_zchange_password | `Failure` | 20.38 | test_outofbandmanagement.py
test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | `Failure` | 
36.36 | test_network.py
test_network_rules_acquired_public_ip_2_nat_rule | `Failure` | 31.31 | 
test_network.py
test_network_rules_acquired_public_ip_1_static_nat_rule | `Failure` | 93.29 
| test_network.py
test_10_attachAndDetach_iso | `Error` | 32.29 | test_vm_life_cycle.py
test_01_vpc_site2site_vpn | Success | 144.27 | test_vpc_vpn.py
test_01_vpc_remote_access_vpn | Success | 61.00 | test_vpc_vpn.py
test_01_redundant_vpc_site2site_vpn | Success | 227.25 | test_vpc_vpn.py
test_02_VPC_default_routes | Success | 253.81 | test_vpc_router_nics.py
test_01_VPC_nics_after_destroy | Success | 515.20 | test_vpc_router_nics.py
test_05_rvpc_multi_tiers | Success | 494.85 | test_vpc_redundant.py
test_04_rvpc_network_garbage_collector_nics | Success | 1402.39 | 
test_vpc_redundant.py
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
Success | 525.90 | test_vpc_redundant.py
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1235.82 | 
test_vpc_redundant.py
test_09_delete_detached_volume | Success | 15.35 | test_volumes.py
test_08_resize_volume | Success | 15.28 | test_volumes.py
test_07_resize_fail | Success | 20.35 | test_volumes.py
test_06_download_detached_volume | Success | 15.22 | test_volumes.py
test_05_detach_volume | Success | 100.24 | test_volumes.py
test_04_delete_attached_volume | Success | 10.16 | test_volumes.py
test_03_download_attached_volume | Success | 15.24 | test_volumes.py
test_02_attach_volume | Success | 43.61 | test_volumes.py
test_01_create_volume | Success | 617.32 | test_volumes.py
test_deploy_vm_multiple | Success | 338.31 | test_vm_life_cycle.py
test_deploy_vm | Success | 0.02 | test_vm_life_cycle.py
test_advZoneVirtualRouter | Success | 0.02 | test_vm_life_cycle.py
test_09_expunge_vm | Success | 125.21 | test_vm_life_cycle.py
test_08_migrate_vm | Success | 35.66 | test_vm_life_cycle.py
test_07_restore_vm | Success | 0.08 | test_vm_life_cycle.py
test_06_destroy_vm | Success | 125.63 | test_vm_life_cycle.py
test_03_reboot_vm | Success | 125.67 | test_vm_life_cycle.py
test_02_start_vm | Success | 10.13 | test_vm_life_cycle.py
test_01_stop_vm | Success | 35.22 | test_vm_life_cycle.py
test_CreateTemplateWithDuplicateName | Success | 65.67 | test_templates.py
test_08_list_system_templates | Success | 0.02 | test_templates.py
test_07_list_public_templates | Success | 0.03 | test_templates.py
test_05_template_permissions | Success | 0.04 | test_templates.py
test_04_extract_template | Success | 5.44 | test_templates.py
test_03_delete_template | Success | 5.11 | test_templates.py
test_02_edit_template | Success | 90.10 | test_templates.py
test_01_create_template | Success | 45.35 | test_templates.py
test_10_destroy_cpvm | Success | 131.23 | test_ssvm.py
test_09_destroy_ssvm | Success | 163.00 | test_ssvm.py
test_08_reboot_cpvm | Success | 101.22 | test_ssvm.py
test_07_reboot_ssvm | Success | 102.97 | test_ssvm.py
test_06_stop_cpvm | Success | 131.34 | test_ssvm.py
test_05_stop_ssvm | Success | 133.09 | test_ssvm.py
test_04_cpvm_internals | Success | 0.93 | test_ssvm.py
test_03_ssvm_internals | Success | 4.03 | test_ssvm.py
test_02_list_cpvm_vm | Success | 0.09 | test_ssvm.py
test_01_list_sec_storage_vm | Success | 0.09 | test_ssvm.py
test_01_snapshot_root_disk | Success | 11.01 | test_snapshots.py
test_04_change_offering_small | Success | 204.31 | test_service_offerings.py
test_03_delete_service_offering | Success | 0.03 | test_service_offerings.py
test_02_edit_service_offering | Success | 0.04 | test_service_offerings.py
test_01_create_service_offering | Success | 0.07 | test_service_offerings.py
test_02_sys_template_ready | Success | 0.09 | test_secondary_storage.py
test_01_sys_vm_start | Success | 0.12 | test_secondary_storage.py
test_09_reboot_router | Success | 30.21 | test_routers.py
test_08_start_router | Success | 25.19 | test_routers.py
test_07_stop_router | Success | 10.12 | test_routers.py
test_06_router_advanced | Success | 0.04 | test_routers.py
test_05_router_basic | Success | 0.04 |

[GitHub] cloudstack issue #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-02 Thread blueorangutan 
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been 
kicked to run smoke tests


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-02 Thread rhtyd 
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@blueorangutan test


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-02 Thread blueorangutan 
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-139


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-02 Thread blueorangutan 
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-02 Thread rhtyd 
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@blueorangutan package


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-01 Thread jburwell 
Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@murali-reddy have any of your patches addressed this issue?


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-01 Thread ustcweizhou 
Github user ustcweizhou commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
this is our change for similar issue
```
diff --git a/systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py 
b/systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py
index 9f9513d..fc2b639 100755
--- a/systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py
+++ b/systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py
@@ -118,6 +118,16 @@ class CsAddress(CsDataBag):
 if CsDevice(dev, self.config).waitfordevice():
 ip.configure(address)

+def delete(self, addr):
+for dev in self.dbag:
+if dev == "id":
+continue
+for address in self.dbag[dev]:
+if address['public_ip'] == addr:
+self.dbag[dev].remove(address)
+self.save()
+break
+

 class CsInterface:

@@ -537,6 +547,8 @@ class CsIP:
 route.copy_routes_from_main(devs, [self.dev])

 elif method == "delete":
+logging.info("delete %s from /etc/cloudstack/ips.json" % 
self.address['public_ip'])
+self.config.address().delete(self.address['public_ip'])
 logging.warn("delete route not implemented")

 self.fw_router()
```


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-10-29 Thread remibergsma 
Github user remibergsma commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@pdion891 saw your msg only now. Will have a look when I find some time. 
Thx!


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-10-21 Thread blueorangutan 
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-71


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-10-20 Thread blueorangutan 
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-10-20 Thread rhtyd 
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
@blueorangutan package


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-10-20 Thread rhtyd 
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
/cc @murali-reddy 


---
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 #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-10-12 Thread pdion891 
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1706
  
Hi @remibergsma,  

We found this issue CLOUDSTACK-9500 where Public IP's is not removed from a 
VR when the Public IP is release. this cause routing issues because the arp 
table is refresh at any operation on the VR which reset de MAC for the release 
IP that can be used into another VR in a VPC deployement model. 

Is this something you faced? 

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.
---