summaryrefslogtreecommitdiff
path: root/heat/tests/openstack/neutron/test_neutron_port.py
Commit message (Collapse)AuthorAgeFilesLines
* Ignore NotFound error in prepare_for_replacerabi2018-07-311-0/+19
| | | | | | | | | | If the port is already deleted in neutron, we don't need to do anything. Change-Id: I9275610931a174919076330ec443f54f68cc1371 Story: #2003015 Task: 23035 (cherry picked from commit 3d542935fbe7def5e785c26a1b6a8b468cfeaa48)
* Don't replace failed neutron resources alwayshuangtianhua2017-01-091-0/+21
| | | | | | | | Override needs_replace_failed() for all neutron resource. Change-Id: I1789540fe2caae18fb446cd069b827036dd86f1c Blueprint: custom-update-replace-for-failed-resources
* Refactor port testshuangtianhua2016-12-131-351/+183
| | | | | | | | This changes: 1. use mock instead of mox 2. correct test_port_needs_update_network() Change-Id: I51c12a61bc42915da7f50cea4e6a55a07f19e77b
* Supports 'direct-physical' and 'baremetal' vnic type for porthuangtianhua2016-11-301-0/+10
| | | | | | | Supports vnic types for port sync with Neutron. Change-Id: Iba1b62cda697409e88bfe5ff69225361b9d142b0 Closes-Bug: #1644971
* Allow update inplace for allowed_address_pairsSergey Kraynev2016-09-151-1/+10
| | | | | | | | | | | | | | This patch allows to do update inplace for allowed_address_pairs properties. Scenario mentioned in bug works correct now. Also add couple fixes to related test: - Add explicit translation name to string, otherwise it returns objects, that raise error during resolving Property name, which should be a string. - Add check, that update of any of mentioned properties does not cause replace. Change-Id: I913fd36012179f2fdd602f2cca06a89e3fa896f3 Closes-Bug: #1623821
* Begin move of UpdateReplace back to its rightful locationZane Bitter2016-08-181-2/+3
| | | | | | | | | | | | | | | | | | | A deeply misguided effort to move all exceptions out of the heat.engine.resource module, where they belong, and into the heat.common.exception module, where they largely do not, broke the API for third-party resource plugins. Unfortunately this happened a couple of releases back already, so we can't simply put UpdateReplace back where it belongs as that would also break the de-facto third-party API. This change adds an alias in the correct location and a comment indicating that it will move back at some time in the future. It also switches all of the in-tree uses back to heat.engine.resource.UpdateReplace, in the hope that third-party developers will be more inclined to copy from that. This reverts commit 4e2cfb991ada853023cd0994db2bf9a39126c538. Change-Id: Iedd5d07d6c0c07e39e51a0fb810665b3e9c61f87 Closes-Bug: #1611104
* Add dns properties and attributesrabi2016-07-261-5/+32
| | | | | | | | | | | This adds dns related properties and attributes to neutron resources. DNS functionality offered by neutron can only be used by enabling the 'dns' extension driver. Change-Id: I1864a16e391d5eb2aa0ba6faace90ef5f26b3f59 Blueprint: neutron-dns-resolution
* Unit tests: compare stack names in PhysNameZane Bitter2016-03-101-71/+72
| | | | | | | | | | | | Previously we were ignoring the stack name when comparing a PhysName, since that part can be shortened. That allowed a lot of code to get changed that made the stack name out of sync with the one printed by the PhysName object. This brings everything back into sync and starts comparing for equality against the shortened stack name. This also introduces a semi-correct implementation of __hash__. Change-Id: I184cc7b0b746b1b7cddff231a90b7337ca42d35b
* Resolve name/id using translation rulesRabi Mishra2016-03-051-6/+6
| | | | | | | | | | | | | | | | This changes the first set of neutron resouces to use translation rules to resolve name_id properties. Changed reources: - OS::Neutron::Subnet - OS::Neutron::Port - OS::Neutron::Router - OS::Neutron::FloatingIp Change-Id: I13f30abb5e284beff4ece9e0c88ee7319ff1be80 Partial-Bug: #1514680 Closes-Bug: #1553362
* Revert "Resolve name/id using translation rules"Steven Hardy2016-02-101-6/+6
| | | | | | | | | | This has been reported as breaking the magnum gate, and we're not yet exactly sure why, so revert while we investigate. This reverts commit 0388da504765f9975aeadc509672c0a47bbcc244. Closes-Bug: #1544227 Change-Id: I9cd170eded8b54eb3b127d8d3f2c58a8b7d135a4
* Resolve name/id using translation rulesRabi Mishra2016-02-101-6/+6
| | | | | | | | | | | | | | | This changes the first set of neutron resouces to use translation rules to resolve name_id properties. Changed reources: - OS::Neutron::Subnet - OS::Neutron::Port - OS::Neutron:Router - OS::Neutron:FloatingIp Change-Id: I2119401e3554d0bafd34978e6c8e7610712148ac Partial-Bug: #1514680
* Refactor resource specific replace logicRabi Mishra2016-02-081-36/+55
| | | | | | | | | | | | Currently we override _needs_update for some resources, where replacement is required either based on tmpl_diff or prop_diff. This patch refactors and adds specific functions (needs_replace, needs_replace_with_prop_diff, needs_replace_with_tmpl_diff). These can now be overidden as required. Change-Id: I15542bef8c9e6ed6b80cd4656d2fd42e8406655b Blueprint: stack-update-restrict
* Don't pass [] for fixed_ips to neutronRabi Mishra2016-01-271-1/+2
| | | | | | | | | | | | | | | | | Earlier we used to remove `fixed_ips` attibute while calling neutron, if it's [] for both create and update. This was masking the behaviour of releasing fixed_ips from a port during update or creating a port without fixed_ips. The recent change[1] to fix this is backward incompatible with some tripleo templates. This patch fixes it by reverting to the original behaviour. [1] https://review.openstack.org/#/c/256328/ Change-Id: I72d9533ff7ed7656224849f6b134652db0532aef Closes-Bug: #1538473
* Fix port handle_update to use prop_diffRabi Mishra2016-01-211-197/+126
| | | | | | | Fix hanle_update of OS::Neutron:Port to use prop_diff. Change-Id: Iaaa41c1a8a81f272d4216555632f6a358640af66 Partial-Bug: #1521836
* Neutron port allow switching between network_id and networkSteven Hardy2015-12-161-16/+35
| | | | | | | | | | | The network_id property is deprecated, so we should tolerate switching to the not-deprecated network property without replacing the port. This also adjusts the logic in _needs_update slightly to enable easier backporting to stable branches. Change-Id: I289a88c1a9c3351df976424504bc14a5ec878573 Partial-Bug: #1514680
* Merge "Neutron port, tolerate switching network name/id"Jenkins2015-12-151-1/+74
|\
| * Neutron port, tolerate switching network name/idSteven Hardy2015-12-151-1/+74
| | | | | | | | | | | | | | | | | | It's valid to switch the network name for an ID referencing the same network, and vice-versa so allow that behavior when using the network property which can accept either name or ID. Change-Id: I0889ff8fc277dc9703d30b7e42fd799cfb57e975 Partial-Bug: #1514680
* | Provide the QoS capability for porthuangtianhua2015-12-151-1/+16
|/ | | | | | | Provide QoS capability for neutron port resource. Change-Id: Ie9fa1ce45c95f1e00af23b1b9d4069f67181d7ed Blueprint: support-neutron-qos
* Refactor neutron constraintsEthan Lynn2015-12-101-16/+32
| | | | | | | | | | | | | Add class NeutronContraint as base neutron constraint class, this class has 3 new properties: resource_name, cmd_resource, extension. The validation workflow will be: check extension available -> get resource id base on resource_name and cmd_resource This patch also and a new function find_resourceid_by_name_or_id to neutron client_plugin, which wrap find_resourceid_by_name_or_id from neutronclient, so that we can avoid to import neutronclient in many places. Change-Id: I9968444339e239ddf0b43ecf18c6aebe18fa5aea
* Translate properties before updateThomas Herve2015-11-251-4/+5
| | | | | | | | | | This adds the translation rule call to properties before update, so that we can remove resource code that refers to deprecated properties. The new ones are used everywhere with the expectation that the translation occurred. Change-Id: Ibdf91ae711f82de5c60896f2452e2872bb3dc9cb Closes-Bug: #1518676
* Change namespace for Neutron testsricolin2015-11-201-0/+873
Change namespace to more specified. Use path to specify test should be enough. Change-Id: I2d33df8d9b42f81d9bf1cd7d8671c045c5786fb6
View Lorry Controller Status