*** Aish has left #openstack-neutron | 00:00 | |
*** dannywil_ has quit IRC | 00:02 | |
*** markvoelker has quit IRC | 00:02 | |
*** annegentle has joined #openstack-neutron | 00:06 | |
*** yamamoto has joined #openstack-neutron | 00:06 | |
*** xgerman_ has quit IRC | 00:11 | |
*** Longgeek has quit IRC | 00:15 | |
*** sbalukoff has quit IRC | 00:16 | |
*** ijw_ has joined #openstack-neutron | 00:16 | |
*** ijw__ has joined #openstack-neutron | 00:18 | |
openstackgerrit | Itsuro Oda proposed openstack/neutron: Enable services on agents with admin_state_up False https://review.openstack.org/147032 | 00:18 |
---|---|---|
*** ijw has quit IRC | 00:19 | |
*** dboik has joined #openstack-neutron | 00:21 | |
*** ijw_ has quit IRC | 00:21 | |
*** jamiem has quit IRC | 00:22 | |
*** jamiem has joined #openstack-neutron | 00:26 | |
*** jamiem has quit IRC | 00:27 | |
*** padkrish_ has quit IRC | 00:28 | |
*** padkrish has joined #openstack-neutron | 00:28 | |
*** Flcn has joined #openstack-neutron | 00:28 | |
*** reed has quit IRC | 00:29 | |
*** padkrish_ has joined #openstack-neutron | 00:29 | |
*** reed has joined #openstack-neutron | 00:30 | |
*** reed has quit IRC | 00:30 | |
*** reed has joined #openstack-neutron | 00:31 | |
*** padkrish has quit IRC | 00:32 | |
*** Flcn has quit IRC | 00:33 | |
*** igordcard_ has quit IRC | 00:34 | |
*** yamamoto has quit IRC | 00:34 | |
*** vishwanathj has quit IRC | 00:37 | |
*** tflynn has joined #openstack-neutron | 00:38 | |
*** fredlhsu has joined #openstack-neutron | 00:39 | |
*** baoli has joined #openstack-neutron | 00:41 | |
*** pavel_bondar has quit IRC | 00:41 | |
*** suresh12 has quit IRC | 00:41 | |
*** pavel_bondar has joined #openstack-neutron | 00:42 | |
*** tflynn has quit IRC | 00:43 | |
*** catohornet has joined #openstack-neutron | 00:43 | |
*** baoli has quit IRC | 00:48 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Simplify base test cases https://review.openstack.org/167041 | 00:48 |
kevinbenton | oda-g: sorry about the delay | 00:48 |
kevinbenton | oda-g: i'm fine with cedric's suggestion to | 00:48 |
*** tiswanso has joined #openstack-neutron | 00:49 | |
*** fredlhsu has quit IRC | 00:49 | |
*** jamiem has joined #openstack-neutron | 00:53 | |
*** rangansw has quit IRC | 00:56 | |
*** nlahouti has quit IRC | 00:57 | |
*** SumitNaiksatam has quit IRC | 00:57 | |
*** banix has joined #openstack-neutron | 01:00 | |
*** tiswanso has quit IRC | 01:03 | |
*** tiswanso has joined #openstack-neutron | 01:03 | |
*** Longgeek has joined #openstack-neutron | 01:03 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Simplify base test cases https://review.openstack.org/167041 | 01:06 |
oda-g | kevinbenton: thank you for your support. it's really help to me. | 01:09 |
kevinbenton | oda-g: no prob. your fixing useful things | 01:09 |
*** salv-orlando has quit IRC | 01:10 | |
*** annegentle has quit IRC | 01:14 | |
*** crc32 has quit IRC | 01:16 | |
*** annegentle has joined #openstack-neutron | 01:17 | |
*** yamamoto has joined #openstack-neutron | 01:17 | |
*** zhhuabj has quit IRC | 01:21 | |
*** dims_ has quit IRC | 01:23 | |
*** ChuckC has joined #openstack-neutron | 01:26 | |
*** jamiem has quit IRC | 01:29 | |
*** _cjones_ has quit IRC | 01:30 | |
*** _cjones_ has joined #openstack-neutron | 01:31 | |
*** popw has joined #openstack-neutron | 01:33 | |
*** _cjones_ has quit IRC | 01:35 | |
*** SumitNaiksatam has joined #openstack-neutron | 01:35 | |
openstackgerrit | Itsuro Oda proposed openstack/neutron: Enable services on agents with admin_state_up False https://review.openstack.org/147032 | 01:35 |
*** ChuckC has quit IRC | 01:36 | |
*** stanzgy has joined #openstack-neutron | 01:38 | |
*** tflynn has joined #openstack-neutron | 01:39 | |
*** shaohe_feng has joined #openstack-neutron | 01:39 | |
openstackgerrit | Kevin Benton proposed openstack/neutron: Set floating IP port status to "N/A" https://review.openstack.org/122265 | 01:39 |
*** s3wong has quit IRC | 01:42 | |
*** baoli has joined #openstack-neutron | 01:43 | |
*** tflynn has quit IRC | 01:43 | |
*** annegentle has quit IRC | 01:48 | |
*** melwitt has quit IRC | 01:48 | |
*** yamahata has quit IRC | 01:49 | |
*** amotoki has quit IRC | 01:50 | |
*** happyeveryday has joined #openstack-neutron | 01:50 | |
*** Longgeek has quit IRC | 01:52 | |
*** banix has quit IRC | 01:52 | |
*** Longgeek has joined #openstack-neutron | 01:53 | |
*** dank_ has quit IRC | 01:55 | |
*** happyeveryday has quit IRC | 01:58 | |
*** Networkn3rd has joined #openstack-neutron | 02:00 | |
*** haigang has joined #openstack-neutron | 02:00 | |
*** vishwanathj has joined #openstack-neutron | 02:05 | |
*** mlavalle has joined #openstack-neutron | 02:06 | |
*** salv-orlando has joined #openstack-neutron | 02:08 | |
*** erkules_ has joined #openstack-neutron | 02:09 | |
*** erkules has quit IRC | 02:12 | |
*** gildub has joined #openstack-neutron | 02:14 | |
openstackgerrit | goocher proposed openstack/neutron: Change check_child_processes_interval default https://review.openstack.org/168204 | 02:16 |
*** oomichi_ has quit IRC | 02:17 | |
*** banix has joined #openstack-neutron | 02:19 | |
*** anshul has joined #openstack-neutron | 02:20 | |
openstackgerrit | Dan Prince proposed openstack/neutron: Revert "Fix validation of physical network name for flat nets" https://review.openstack.org/168207 | 02:21 |
*** achanda has quit IRC | 02:27 | |
*** haigang has quit IRC | 02:29 | |
*** tsg has quit IRC | 02:29 | |
*** padkrish_ has quit IRC | 02:30 | |
*** haigang has joined #openstack-neutron | 02:30 | |
*** padkrish has joined #openstack-neutron | 02:30 | |
*** padkrish has quit IRC | 02:35 | |
*** jckasper has joined #openstack-neutron | 02:39 | |
*** catohornet has quit IRC | 02:39 | |
*** pal_ has joined #openstack-neutron | 02:43 | |
*** jckasper has quit IRC | 02:45 | |
*** salv-orlando has quit IRC | 02:45 | |
*** annegentle has joined #openstack-neutron | 02:48 | |
*** annegentle has quit IRC | 02:54 | |
*** devlaps has quit IRC | 02:54 | |
*** oda-g has quit IRC | 02:59 | |
*** Murali has joined #openstack-neutron | 03:00 | |
*** tiswanso has quit IRC | 03:04 | |
*** ijw__ has quit IRC | 03:06 | |
openstackgerrit | Merged openstack/neutron: Simplify base test cases https://review.openstack.org/167041 | 03:12 |
*** achanda has joined #openstack-neutron | 03:13 | |
*** shwetaap has joined #openstack-neutron | 03:16 | |
openstackgerrit | Henry Gessau proposed openstack/neutron: Modify a different agent in test_update_agent_description https://review.openstack.org/168213 | 03:16 |
*** emagana has joined #openstack-neutron | 03:18 | |
*** anish_ has joined #openstack-neutron | 03:18 | |
*** liusheng has joined #openstack-neutron | 03:21 | |
*** suresh12 has joined #openstack-neutron | 03:23 | |
openstackgerrit | Kevin Benton proposed openstack/neutron: Don't eagerly load ranges from IPAllocationPool https://review.openstack.org/168214 | 03:25 |
*** atuvenie has joined #openstack-neutron | 03:25 | |
*** tsg has joined #openstack-neutron | 03:26 | |
*** Marga_ has quit IRC | 03:26 | |
openstackgerrit | Zengfa Gao proposed openstack/neutron: Add API tests for subnetpool allocation https://review.openstack.org/167611 | 03:26 |
HenryG | kevinbenton: https://review.openstack.org/149546 | 03:27 |
*** tflynn has joined #openstack-neutron | 03:28 | |
*** ajmiller has joined #openstack-neutron | 03:28 | |
*** Murali has quit IRC | 03:29 | |
lizk_ | hi, guys, would you please review https://review.openstack.org/#/c/155317/ and https://review.openstack.org/#/c/166539/ thx :) | 03:30 |
*** amuller_ has quit IRC | 03:32 | |
*** Longgeek has quit IRC | 03:34 | |
*** suresh12 has quit IRC | 03:38 | |
*** tflynn has quit IRC | 03:40 | |
*** tflynn has joined #openstack-neutron | 03:40 | |
*** padkrish has joined #openstack-neutron | 03:41 | |
openstackgerrit | Andrew Boik proposed openstack/neutron: Allow only one IPv4 and one IPv6 fixed IP on router gateway port https://review.openstack.org/167784 | 03:42 |
*** pal__ has joined #openstack-neutron | 03:44 | |
*** pal_ has quit IRC | 03:45 | |
*** padkrish has quit IRC | 03:46 | |
*** salv-orlando has joined #openstack-neutron | 03:46 | |
*** banix has quit IRC | 03:46 | |
*** sbalukoff has joined #openstack-neutron | 03:46 | |
*** _cjones_ has joined #openstack-neutron | 03:48 | |
*** _cjones_ has quit IRC | 03:49 | |
*** baoli has quit IRC | 03:49 | |
*** annegentle has joined #openstack-neutron | 03:50 | |
*** emagana has quit IRC | 03:51 | |
*** emagana has joined #openstack-neutron | 03:52 | |
*** baoli has joined #openstack-neutron | 03:52 | |
*** salv-orlando has quit IRC | 03:52 | |
*** tflynn has quit IRC | 03:54 | |
*** pal__ has quit IRC | 03:55 | |
*** annegentle has quit IRC | 03:55 | |
*** emagana has quit IRC | 03:56 | |
*** lykinsbd has quit IRC | 03:58 | |
*** vishwana_ has joined #openstack-neutron | 03:58 | |
*** _cjones_ has joined #openstack-neutron | 03:58 | |
openstackgerrit | shihanzhang proposed openstack/neutron: Reduce dhcp agent calling 'reload_allocations' https://review.openstack.org/168216 | 03:58 |
*** fredlhsu has joined #openstack-neutron | 03:59 | |
*** deepthi has joined #openstack-neutron | 04:00 | |
*** haleyb has joined #openstack-neutron | 04:02 | |
*** _cjones_ has quit IRC | 04:02 | |
*** baoli has quit IRC | 04:02 | |
*** _cjones_ has joined #openstack-neutron | 04:02 | |
*** atuvenie has quit IRC | 04:06 | |
openstackgerrit | Juergen Brendel proposed openstack/neutron: ARP spoofing patch: Low level ebtables integration https://review.openstack.org/141130 | 04:06 |
*** Longgeek has joined #openstack-neutron | 04:08 | |
*** annegentle has joined #openstack-neutron | 04:08 | |
*** tflynn has joined #openstack-neutron | 04:09 | |
*** yuanying has joined #openstack-neutron | 04:09 | |
*** annegentle has quit IRC | 04:13 | |
*** fredlhsu has quit IRC | 04:13 | |
*** nlahouti has joined #openstack-neutron | 04:17 | |
*** mlavalle has quit IRC | 04:19 | |
*** _cjones_ has quit IRC | 04:33 | |
*** tflynn_ has joined #openstack-neutron | 04:35 | |
*** tflynn has quit IRC | 04:35 | |
*** markvoelker has joined #openstack-neutron | 04:39 | |
*** bquan has joined #openstack-neutron | 04:39 | |
*** lizk has quit IRC | 04:41 | |
*** bquan has quit IRC | 04:41 | |
*** lizk_ has quit IRC | 04:42 | |
*** markvoelker has quit IRC | 04:43 | |
*** Networkn3rd has quit IRC | 04:43 | |
*** tflynn_ has quit IRC | 04:44 | |
*** tflynn has joined #openstack-neutron | 04:45 | |
*** yfauser has joined #openstack-neutron | 04:46 | |
*** yatin has joined #openstack-neutron | 04:47 | |
*** YorikSar has quit IRC | 04:48 | |
*** tflynn has quit IRC | 04:51 | |
*** fredlhsu has joined #openstack-neutron | 04:53 | |
*** lizk has joined #openstack-neutron | 04:53 | |
*** lizk_ has joined #openstack-neutron | 04:54 | |
*** fredlhsu has quit IRC | 04:54 | |
*** trinaths has joined #openstack-neutron | 04:58 | |
*** padkrish has joined #openstack-neutron | 04:58 | |
*** padkrish has joined #openstack-neutron | 04:59 | |
*** nlahouti has quit IRC | 05:00 | |
*** achanda has quit IRC | 05:00 | |
*** reed has quit IRC | 05:03 | |
*** ajmiller has quit IRC | 05:05 | |
*** shwetaap1 has joined #openstack-neutron | 05:09 | |
*** shwetaap has quit IRC | 05:13 | |
*** lpetrut has joined #openstack-neutron | 05:13 | |
*** Marga_ has joined #openstack-neutron | 05:14 | |
*** pal_ has joined #openstack-neutron | 05:16 | |
*** nplanel_ has quit IRC | 05:19 | |
*** annegentle has joined #openstack-neutron | 05:19 | |
*** fredlhsu has joined #openstack-neutron | 05:19 | |
*** yog__ has joined #openstack-neutron | 05:19 | |
*** annegentle has quit IRC | 05:24 | |
*** fredlhsu has quit IRC | 05:29 | |
*** tsg has quit IRC | 05:30 | |
*** chandankumar has joined #openstack-neutron | 05:36 | |
*** achanda has joined #openstack-neutron | 05:37 | |
*** markvoelker has joined #openstack-neutron | 05:39 | |
openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 05:40 |
*** jamielennox is now known as jamielennox|away | 05:41 | |
*** markvoelker has quit IRC | 05:43 | |
*** Networkn3rd has joined #openstack-neutron | 05:44 | |
*** salv-orlando has joined #openstack-neutron | 05:44 | |
*** lpetrut has quit IRC | 05:45 | |
*** lpetrut has joined #openstack-neutron | 05:46 | |
*** dims has joined #openstack-neutron | 05:46 | |
*** vthapar has joined #openstack-neutron | 05:46 | |
*** salv-orlando has quit IRC | 05:47 | |
*** trinaths has quit IRC | 05:47 | |
*** Networkn3rd has quit IRC | 05:48 | |
*** anish_ has quit IRC | 05:51 | |
*** anshul has quit IRC | 05:51 | |
*** anshul has joined #openstack-neutron | 05:52 | |
*** neeti has joined #openstack-neutron | 05:54 | |
*** YorikSar has joined #openstack-neutron | 05:55 | |
*** yalie has quit IRC | 05:57 | |
*** yalie has joined #openstack-neutron | 05:58 | |
*** vishwanathj has quit IRC | 05:59 | |
*** Murali has joined #openstack-neutron | 06:00 | |
*** aranjan has joined #openstack-neutron | 06:01 | |
*** Marga_ has quit IRC | 06:02 | |
*** trinaths has joined #openstack-neutron | 06:07 | |
*** dsneddon has quit IRC | 06:08 | |
*** anish_ has joined #openstack-neutron | 06:08 | |
*** aranjan has quit IRC | 06:11 | |
stockpirate | hi! someone who is awake? | 06:12 |
*** armax has quit IRC | 06:13 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex https://review.openstack.org/167499 | 06:15 |
*** aranjan has joined #openstack-neutron | 06:16 | |
stockpirate | i have been created a net and subnet. the neutron agent-list is down on all agents | 06:17 |
stockpirate | so i will delete those net and subnet i have been created. but it hang | 06:18 |
stockpirate | an example: neutron net-delete 04797502-0629-417f-8b13-8eda888c34ac | 06:18 |
stockpirate | nothing happend | 06:18 |
*** numan has joined #openstack-neutron | 06:18 | |
trinaths | stockpirate: is the DB active | 06:20 |
*** annegentle has joined #openstack-neutron | 06:20 | |
*** SridharG has joined #openstack-neutron | 06:21 | |
*** yfauser has left #openstack-neutron | 06:21 | |
stockpirate | it is: root@vitkenxfs03:/etc/neutron# netstat -an | grep LISTEN | 06:21 |
stockpirate | tcp 0 0 0.0.0.0:35357 0.0.0.0:* LISTEN | 06:21 |
stockpirate | tcp 0 0 0.0.0.0:9696 0.0.0.0:* LISTEN | 06:21 |
*** dims has quit IRC | 06:21 | |
openstackgerrit | Merged openstack/neutron-fwaas: Remove reference to testlib_plugin https://review.openstack.org/168123 | 06:21 |
stockpirate | and; | 06:22 |
stockpirate | MariaDB [(none)]> show databases; | 06:22 |
stockpirate | +--------------------+ | 06:22 |
stockpirate | | Database | | 06:22 |
stockpirate | +--------------------+ | 06:22 |
stockpirate | | information_schema | | 06:22 |
stockpirate | | neutron | | 06:22 |
stockpirate | +--------------------+ | 06:22 |
stockpirate | 2 rows in set (0.00 sec) | 06:22 |
*** shaohe_feng has quit IRC | 06:23 | |
*** anish_ has quit IRC | 06:23 | |
stockpirate | when i do: | 06:25 |
stockpirate | root@vitkencas03:/etc/neutron# neutron net-list | 06:25 |
stockpirate | +--------------------------------------+--------+-----------------------------------------------------+ | 06:25 |
stockpirate | | id | name | subnets | | 06:25 |
stockpirate | +--------------------------------------+--------+-----------------------------------------------------+ | 06:25 |
stockpirate | | 04797502-0629-417f-8b13-8eda888c34ac | test01 | 0b6f60ee-d6f9-4182-b243-3797fb943597 192.168.0.0/24 | | 06:25 |
stockpirate | +--------------------------------------+--------+-----------------------------------------------------+ | 06:25 |
*** annegentle has quit IRC | 06:25 | |
*** afazekas is now known as __afazekas | 06:28 | |
*** aranjan has quit IRC | 06:30 | |
openstackgerrit | Itsuro Oda proposed openstack/neutron: Make floatingip reachable from the same network https://review.openstack.org/161947 | 06:31 |
*** aranjan has joined #openstack-neutron | 06:31 | |
*** Bhavya has joined #openstack-neutron | 06:32 | |
*** achanda has quit IRC | 06:32 | |
*** shaohe_feng has joined #openstack-neutron | 06:33 | |
*** salv-orlando has joined #openstack-neutron | 06:33 | |
*** shwetaap has joined #openstack-neutron | 06:33 | |
*** shwetaap1 has quit IRC | 06:36 | |
*** ishant has joined #openstack-neutron | 06:36 | |
openstackgerrit | P. Watsalya Mishra proposed openstack/python-neutronclient: "neutron help router-update" help info updated https://review.openstack.org/141006 | 06:36 |
*** salv-orlando has quit IRC | 06:37 | |
*** aepifanov has joined #openstack-neutron | 06:39 | |
trinaths | stockpirate: can u check the logs in neutron server | 06:39 |
trinaths | stockpirate: when u execute the same CLI neutron command. | 06:39 |
stockpirate | yes. two windows up. one running neutron net-delete | 06:39 |
stockpirate | the other for logs | 06:40 |
*** markvoelker has joined #openstack-neutron | 06:40 | |
trinaths | stockpirate: when u post a huge logs for debugging, you may use http://paste.openstack.org so that, its easy to communicate the debug logs | 06:40 |
stockpirate | for neutron/server.log | 06:40 |
stockpirate | okei thanks for tip | 06:40 |
trinaths | stockpirate: look into the neutron/server.log on what wrong with the command your are executing for deleting the subnet from neutron cli. | 06:42 |
stockpirate | http://paste.openstack.org/show/196988/ | 06:42 |
stockpirate | almost nothing written in server.log amq connected etc | 06:43 |
stockpirate | some stuff from the horizon | 06:43 |
*** haleyb has quit IRC | 06:43 | |
*** anish_ has joined #openstack-neutron | 06:43 | |
trinaths | stockpirate: did neutron server start correctly. | 06:44 |
*** markvoelker has quit IRC | 06:44 | |
*** haleyb has joined #openstack-neutron | 06:45 | |
stockpirate | http://paste.openstack.org/show/196989/ | 06:46 |
stockpirate | updated what happend when im stopping and starting neutron-server service again | 06:46 |
vthapar | stockpirate: one more thing you can do is give --debug [or is it -debug? check the help] option when giving neutron cli commands, gives better idea of what went wrong. | 06:47 |
trinaths | stockpirate: yes, enable debug in neutron.conf | 06:47 |
trinaths | vthapar: thanks for this hint :) | 06:48 |
*** yamamoto has quit IRC | 06:48 | |
vthapar | trinaths: you're welcome :) | 06:48 |
*** aranjan has quit IRC | 06:49 | |
vthapar | stockpirate: from neutron server logs looks like server did come up, confirmed by netstat output. running neutron client cli with debug option will tell what response came from neturon server, if any. | 06:49 |
stockpirate | http://paste.openstack.org/show/196991/ | 06:49 |
stockpirate | updated | 06:50 |
stockpirate | thanks | 06:50 |
stockpirate | when im running neutron net-delete 04797502-0629-417f-8b13-8eda888c34ac --debug | 06:50 |
*** Longgeek has quit IRC | 06:50 | |
stockpirate | after the end of text, it just "blinking" hang and nothing more happend. | 06:51 |
trinaths | stockpirate: can u rung this command, curl -i http://10.55.0.28:9696/v2.0/networks/04797502-0629-417f-8b13-8eda888c34ac.json -X DELETE -H "X-Auth-Token: a505731b90204e5d8c4d12a5a6a5866d" -H "Content-Type: application/json" -H "Accept: application/json" -H "User-Agent: python-neutronclient" | 06:51 |
vthapar | stockpirate: so you're delete request goes to neutron server, nothing after that. second response in first output confirms neutrons erver is still running and reachable so network issues also ruled out. | 06:52 |
vthapar | stockpirate: next step would be to check neutron server logs, with debug enabled. | 06:53 |
*** Longgeek has joined #openstack-neutron | 06:53 | |
stockpirate | <trinaths> doing that command directly now. It just hang/blinking/wainting for "something". | 06:53 |
stockpirate | vthapar: thanks, you can see whap happend when i do neutron net-del **** --debud here,; http://paste.openstack.org/show/196991/ | 06:54 |
stockpirate | the logs say nothing | 06:54 |
stockpirate | A stupid question. Do i need to stop a service before doing a net-del or sub-del? | 06:55 |
*** bradjones has quit IRC | 06:55 | |
*** pradipta has joined #openstack-neutron | 06:55 | |
*** sridhar_ram has joined #openstack-neutron | 06:56 | |
*** haleyb has quit IRC | 06:56 | |
vthapar | stockpirate: yeah, I took a look at logs and looks like client-server communication is fine. now you need to take a look at neutron server logs and openvswitch agent logs. | 06:56 |
*** bradjones has joined #openstack-neutron | 06:57 | |
*** bradjones has quit IRC | 06:57 | |
*** bradjones has joined #openstack-neutron | 06:57 | |
*** gildub has quit IRC | 06:57 | |
openstackgerrit | shihanzhang proposed openstack/neutron: Reduce dhcp agent calling 'reload_allocations' https://review.openstack.org/168216 | 06:57 |
trinaths | stockpirate: no need to restart for delete | 06:58 |
vthapar | and you don't need to stop any service. you will need to restart a service for any conf file changes to take effect. | 06:58 |
*** ajo has quit IRC | 06:58 | |
stockpirate | okei thank i see | 06:59 |
stockpirate | just adding some logs | 06:59 |
*** sridhar_ram has quit IRC | 07:00 | |
*** nfedotov has joined #openstack-neutron | 07:01 | |
*** apuimedo has joined #openstack-neutron | 07:01 | |
openstackgerrit | P. Watsalya Mishra proposed openstack/python-neutronclient: "neutron help net-list" help info updated https://review.openstack.org/140628 | 07:02 |
stockpirate | http://paste.openstack.org/show/196993/ | 07:03 |
*** shwetaap has quit IRC | 07:04 | |
*** bjornar_ has joined #openstack-neutron | 07:04 | |
trinaths | stockpirate: ovsdb error? | 07:05 |
stockpirate | http://paste.openstack.org/show/196995/ | 07:07 |
stockpirate | adding ovs in the bottom | 07:07 |
stockpirate | sorry missing db | 07:08 |
stockpirate | a sec | 07:08 |
*** scheuran has joined #openstack-neutron | 07:08 | |
stockpirate | ovsdb added in the bottom | 07:09 |
*** anish_ has quit IRC | 07:12 | |
*** afazekas has joined #openstack-neutron | 07:13 | |
*** padkrish has quit IRC | 07:13 | |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Transform BaseOVSLinuxTestCase methods in helpers https://review.openstack.org/165866 | 07:15 |
*** chlong has quit IRC | 07:16 | |
*** anish_ has joined #openstack-neutron | 07:17 | |
*** bjornar_ has quit IRC | 07:18 | |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional/fullstack tests https://review.openstack.org/143412 | 07:23 |
stockpirate | Thanks for helping me !!! | 07:23 |
stockpirate | im | 07:23 |
stockpirate | think it only was a reboot | 07:23 |
stockpirate | there was a hang on one service | 07:24 |
stockpirate | finally i could delete the net!!! | 07:24 |
stockpirate | im rebooted everything so i dont really know what service it was. so i need to figure it out before im going in production some time | 07:25 |
*** anilvenkata has joined #openstack-neutron | 07:25 | |
trinaths | stockpirate: congrats :) | 07:25 |
*** Miouge has joined #openstack-neutron | 07:26 | |
stockpirate | thank you sir for you time!! | 07:26 |
*** Networkn3rd has joined #openstack-neutron | 07:28 | |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Replace BaseIPVethTestCase by FakeMachine https://review.openstack.org/160290 | 07:29 |
*** anish__ has joined #openstack-neutron | 07:30 | |
*** shaohe_feng has quit IRC | 07:30 | |
*** anish_ has quit IRC | 07:30 | |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Clean neutron.tests.functional.agent.linux.base https://review.openstack.org/161913 | 07:30 |
*** shaohe_feng has joined #openstack-neutron | 07:31 | |
*** angela-s has quit IRC | 07:32 | |
*** Networkn3rd has quit IRC | 07:32 | |
*** anish__ has quit IRC | 07:36 | |
openstackgerrit | P. Watsalya Mishra proposed openstack/python-neutronclient: "neutron help net-update" help info updated https://review.openstack.org/137279 | 07:40 |
*** markvoelker has joined #openstack-neutron | 07:40 | |
*** bquan has joined #openstack-neutron | 07:42 | |
*** bobmel has joined #openstack-neutron | 07:42 | |
*** lpetrut has quit IRC | 07:45 | |
*** markvoelker has quit IRC | 07:45 | |
*** bquan has quit IRC | 07:46 | |
*** haleyb has joined #openstack-neutron | 07:53 | |
openstackgerrit | Darragh O'Reilly proposed openstack/neutron: lb-agent: handle security group updates in main loop https://review.openstack.org/138512 | 07:55 |
*** haleyb has quit IRC | 07:58 | |
*** aranjan has joined #openstack-neutron | 08:00 | |
*** aranjan has quit IRC | 08:04 | |
*** pradipta has quit IRC | 08:06 | |
*** aleksandr_null has quit IRC | 08:07 | |
*** SumitNaiksatam has quit IRC | 08:08 | |
*** zhenguo has joined #openstack-neutron | 08:09 | |
*** shaohe_feng has quit IRC | 08:10 | |
openstackgerrit | enikanorov proposed openstack/neutron: OOP cleanup: start protected method names with underscore https://review.openstack.org/168056 | 08:11 |
*** erkules_ is now known as erkules | 08:11 | |
*** erkules has joined #openstack-neutron | 08:11 | |
*** anish_ has joined #openstack-neutron | 08:11 | |
*** matrohon has joined #openstack-neutron | 08:12 | |
*** lpetrut has joined #openstack-neutron | 08:12 | |
*** jpena has joined #openstack-neutron | 08:14 | |
*** salv-orlando has joined #openstack-neutron | 08:15 | |
*** safchain has joined #openstack-neutron | 08:15 | |
*** lpetrut has quit IRC | 08:15 | |
*** ronis has joined #openstack-neutron | 08:16 | |
*** anish_ has quit IRC | 08:17 | |
*** anish_ has joined #openstack-neutron | 08:18 | |
*** anilvenkata has left #openstack-neutron | 08:20 | |
*** salv-orlando has quit IRC | 08:20 | |
*** annegentle has joined #openstack-neutron | 08:22 | |
*** aleksandr_null has joined #openstack-neutron | 08:23 | |
*** sbalukoff has quit IRC | 08:24 | |
*** sbalukoff has joined #openstack-neutron | 08:25 | |
*** ajo has joined #openstack-neutron | 08:27 | |
*** lpetrut has joined #openstack-neutron | 08:27 | |
*** annegentle has quit IRC | 08:27 | |
*** marun has joined #openstack-neutron | 08:31 | |
*** doude has joined #openstack-neutron | 08:31 | |
*** ihrachyshka has joined #openstack-neutron | 08:32 | |
*** marun has quit IRC | 08:35 | |
*** enikanorov has joined #openstack-neutron | 08:36 | |
*** jlanoux has joined #openstack-neutron | 08:37 | |
*** shaohe_feng has joined #openstack-neutron | 08:40 | |
*** jistr has joined #openstack-neutron | 08:40 | |
*** markvoelker has joined #openstack-neutron | 08:41 | |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Allow to define enable_snat default value https://review.openstack.org/130588 | 08:42 |
*** ihrachyshka has quit IRC | 08:45 | |
*** markvoelker has quit IRC | 08:46 | |
stockpirate | more questions from me: tenant router gateway will not occupy ip from floating ip list. | 08:50 |
stockpirate | how could i list ip list form router gateway... to see if not have any ip or refresh it to get an ip ? | 08:53 |
*** nyechiel has joined #openstack-neutron | 08:53 | |
*** Madkiss has quit IRC | 08:59 | |
*** rotbart has joined #openstack-neutron | 09:00 | |
*** anilvenkata has joined #openstack-neutron | 09:01 | |
*** ygbo has joined #openstack-neutron | 09:03 | |
*** Madkiss has joined #openstack-neutron | 09:03 | |
*** Madkiss has quit IRC | 09:03 | |
*** Madkiss has joined #openstack-neutron | 09:03 | |
stockpirate | i find it when im clicking on router in horizon | 09:04 |
stockpirate | and status down | 09:04 |
*** nfedotov has quit IRC | 09:05 | |
openstackgerrit | Romil Gupta proposed openstack/neutron: Move values for network_type to plugins.common.constants.py https://review.openstack.org/166879 | 09:09 |
*** Madkiss has quit IRC | 09:09 | |
*** atuvenie has joined #openstack-neutron | 09:10 | |
*** Madkiss has joined #openstack-neutron | 09:10 | |
*** Madkiss has quit IRC | 09:10 | |
*** Madkiss has joined #openstack-neutron | 09:10 | |
*** yatin has quit IRC | 09:11 | |
*** pradipta has joined #openstack-neutron | 09:13 | |
*** pradipta has quit IRC | 09:14 | |
*** pradipta has joined #openstack-neutron | 09:14 | |
*** yatin has joined #openstack-neutron | 09:15 | |
openstackgerrit | Merged openstack/neutron: Enable services on agents with admin_state_up False https://review.openstack.org/147032 | 09:16 |
*** Networkn3rd has joined #openstack-neutron | 09:16 | |
*** bjornar_ has joined #openstack-neutron | 09:20 | |
*** Networkn3rd has quit IRC | 09:21 | |
*** shaohe_feng has quit IRC | 09:23 | |
*** lukasa has joined #openstack-neutron | 09:23 | |
*** flaviof has joined #openstack-neutron | 09:26 | |
stockpirate | I followed this: http://docs.openstack.org/juno/install-guide/install/apt/content/neutron_initial-networks-verify.html10.55.0.71 network:router_gatewayDOWNUP | 09:32 |
stockpirate | sorry:::: I followed this: http://docs.openstack.org/juno/install-guide/install/apt/content/neutron_initial-networks-verify.html | 09:32 |
stockpirate | but the status is down in router gateway: 10.55.0.71 network:router_gatewayDOWNUP | 09:32 |
stockpirate | doing a:: ip netns exec qrouter-e91277b9-f0ca-4d6f-aa85-a2a97e9fd69f ping 10.55.0.71 | 09:33 |
stockpirate | and get answer | 09:33 |
stockpirate | but I dont get any answer when im doing only: ping 10.55.0.71 | 09:33 |
gongysh | mestery: can u help to have a quick review on https://review.openstack.org/#/c/160430/ | 09:35 |
*** markvoelker has joined #openstack-neutron | 09:42 | |
*** gongysh has quit IRC | 09:44 | |
*** markvoelker has quit IRC | 09:46 | |
*** virtuallynathan_ has joined #openstack-neutron | 09:47 | |
*** nyechiel has quit IRC | 09:50 | |
hanzhang | hi all, who can help review this patch:https://review.openstack.org/#/c/151501/ | 09:50 |
*** dims_ has joined #openstack-neutron | 09:51 | |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Enable Process Monitor by default. https://review.openstack.org/165117 | 09:52 |
*** neiljerram has quit IRC | 09:53 | |
*** Flcn has joined #openstack-neutron | 09:58 | |
*** txomon|fon_ has quit IRC | 09:58 | |
*** txomon|fon_ has joined #openstack-neutron | 09:58 | |
*** hichihar_ has quit IRC | 09:58 | |
*** nyechiel has joined #openstack-neutron | 09:59 | |
*** yalie has quit IRC | 10:02 | |
openstackgerrit | Abhishek G M proposed openstack/neutron: Test to verify shared attribute of network https://review.openstack.org/167566 | 10:05 |
openstackgerrit | Abhishek G M proposed openstack/neutron: Test to verify shared attribute of network https://review.openstack.org/167566 | 10:09 |
*** pc_m has joined #openstack-neutron | 10:11 | |
*** amotoki has joined #openstack-neutron | 10:13 | |
*** nyechiel has quit IRC | 10:15 | |
*** nyechiel has joined #openstack-neutron | 10:15 | |
*** pradipta has quit IRC | 10:17 | |
*** deepthi has quit IRC | 10:17 | |
*** marun has joined #openstack-neutron | 10:20 | |
*** txomon|fon_ has quit IRC | 10:23 | |
*** annegentle has joined #openstack-neutron | 10:23 | |
*** marun has quit IRC | 10:24 | |
*** zilberstein has quit IRC | 10:26 | |
*** annegentle has quit IRC | 10:29 | |
*** salv-orlando has joined #openstack-neutron | 10:30 | |
*** gsagie has joined #openstack-neutron | 10:31 | |
*** Longgeek has quit IRC | 10:31 | |
*** salv-orlando has quit IRC | 10:34 | |
*** matrohon has quit IRC | 10:34 | |
*** shaohe_feng has joined #openstack-neutron | 10:37 | |
*** zilberstein has joined #openstack-neutron | 10:39 | |
*** gildub has joined #openstack-neutron | 10:39 | |
*** gsagie has quit IRC | 10:42 | |
*** nyechiel has quit IRC | 10:42 | |
*** markvoelker has joined #openstack-neutron | 10:43 | |
openstackgerrit | Romil Gupta proposed openstack/neutron: Move values for network_type to plugins.common.constants.py https://review.openstack.org/166879 | 10:43 |
*** trinaths has quit IRC | 10:46 | |
*** atuvenie has quit IRC | 10:47 | |
*** markvoelker has quit IRC | 10:48 | |
*** haigang has quit IRC | 10:48 | |
*** stanzgy has quit IRC | 10:49 | |
*** txomon|fon has joined #openstack-neutron | 10:51 | |
*** pradip_vedams has joined #openstack-neutron | 10:52 | |
*** yuanying has quit IRC | 10:54 | |
*** amotoki has quit IRC | 10:56 | |
*** amotoki has joined #openstack-neutron | 10:57 | |
*** yuanying has joined #openstack-neutron | 10:57 | |
*** flaviof has quit IRC | 11:03 | |
*** txomon|fon has quit IRC | 11:05 | |
*** Networkn3rd has joined #openstack-neutron | 11:05 | |
*** txomon|fon has joined #openstack-neutron | 11:07 | |
*** Longgeek has joined #openstack-neutron | 11:08 | |
*** Networkn3rd has quit IRC | 11:10 | |
*** baoli has joined #openstack-neutron | 11:11 | |
*** ihrachyshka has joined #openstack-neutron | 11:11 | |
*** pal_ has quit IRC | 11:12 | |
*** yog__ has quit IRC | 11:12 | |
*** pal_ has joined #openstack-neutron | 11:13 | |
openstackgerrit | Paul Michali proposed openstack/neutron-fwaas: FWaaS: Refactor callback mechanism https://review.openstack.org/167275 | 11:15 |
*** flaviof has joined #openstack-neutron | 11:16 | |
*** ajo has quit IRC | 11:18 | |
*** gildub has quit IRC | 11:20 | |
*** jamiem has joined #openstack-neutron | 11:21 | |
*** baoli has quit IRC | 11:22 | |
*** Flcn is now known as flcn | 11:24 | |
*** annegentle has joined #openstack-neutron | 11:24 | |
*** salv-orlando has joined #openstack-neutron | 11:24 | |
*** shaohe_feng has quit IRC | 11:26 | |
*** annegentle has quit IRC | 11:29 | |
*** ociuhandu has joined #openstack-neutron | 11:31 | |
*** Murali has quit IRC | 11:31 | |
pc_m | amotoki: ping | 11:34 |
*** rpothier has joined #openstack-neutron | 11:34 | |
amotoki | pc_m: pong | 11:34 |
pc_m | amotoki: Thanks for reviewing the VPN API documentation | 11:34 |
pc_m | amotoki: Just to be sure... I see that all of the list APIs are incorrect (not plural) | 11:35 |
amotoki | pc_m: yeah.. all resources are in a singular case :-( | 11:35 |
pc_m | amotoki: That's what you're seeing right? Nothing else? | 11:35 |
amotoki | pc_m: yes. I followed the result of the Jenkins job at the bottom and then open the PDF of networking-api. | 11:36 |
amotoki | pc_m: other than that, i could not find a good way to check the contents. | 11:37 |
*** blues-man has joined #openstack-neutron | 11:37 | |
pc_m | amotoki: OK. I just saw you mentioned the show command for ipsec connection. wasn't sure if you thought something was wrong there - I thought it was OK> | 11:37 |
amotoki | pc_m: ah.. my comment was ambiguous. | 11:38 |
pc_m | amotoki: Yeah I build the doc and access the pages locally | 11:38 |
pc_m | amotoki: I got the gist. | 11:38 |
pc_m | Just wanted to make sure I didn't miss anything else. | 11:38 |
*** mwagner_lap has quit IRC | 11:38 | |
pc_m | If you get a chance can you look at https://review.openstack.org/#/c/164466/ and https://review.openstack.org/#/c/165226/? | 11:38 |
pc_m | I'd like to get them into the RC, if we can get reviewers in time. It is a refactoring cleanup of VPN/FW and Neutron for callbacks to the services from L3 agent. | 11:39 |
amotoki | pc_m: sure. actually i am off from Wed and I am online only at night :-) | 11:39 |
pc_m | amotoki: thanks! | 11:40 |
*** Longgeek has quit IRC | 11:42 | |
*** johnbelamaric has joined #openstack-neutron | 11:43 | |
*** markvoelker has joined #openstack-neutron | 11:43 | |
*** tiswanso has joined #openstack-neutron | 11:44 | |
*** tiswanso has quit IRC | 11:45 | |
*** neeti has quit IRC | 11:45 | |
*** tiswanso has joined #openstack-neutron | 11:46 | |
*** jprovazn has joined #openstack-neutron | 11:46 | |
*** ferest has joined #openstack-neutron | 11:47 | |
*** markvoelker has quit IRC | 11:48 | |
*** neeti has joined #openstack-neutron | 11:50 | |
*** claudiub has joined #openstack-neutron | 11:50 | |
pc_m | amotoki: FYI: I just pushed up a new version of the API doc commit. https://review.openstack.org/#/c/167609/ | 11:54 |
amotoki | pc_m: thanks. check it after reviewing fwaas insertion in horizon | 11:55 |
pc_m | amotoki: Awesome! | 11:55 |
openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Revise functional test hooks (WIP) https://review.openstack.org/168115 | 11:57 |
*** tiswanso has quit IRC | 11:59 | |
*** pal_ has quit IRC | 11:59 | |
*** dane_leblanc has joined #openstack-neutron | 12:00 | |
HenryG | pc_m: Have you and Armando converged on https://review.openstack.org/164466 ? | 12:01 |
HenryG | And do you know what dougwig tried to ask? | 12:02 |
pc_m | HenryG: Last I checked :) He wanted the callback manager mocked, and that is in the latest (after several botched commits). PS 9 is essentially PS 6 with a rebase. | 12:03 |
pc_m | HenryG: dougwig had a Question and marked it -1, but the question contents are blank. Will wait for him to get back online and will see if he had any concerns. | 12:04 |
HenryG | pc_m: Don't worry about the botched commits, I got that :) | 12:04 |
pc_m | HenryG: Overall, I think it, and the VPN and FW commits are ready to go. If cores think it is good to go and have time to review, it can go into RC, IMHO. | 12:05 |
*** atuvenie has joined #openstack-neutron | 12:06 | |
pc_m | HenryG: If you get chance please take a look. | 12:06 |
HenryG | pc_m: The code looks fine to me. Will wait for dougwig to clarify. | 12:07 |
pc_m | HenryG: thanks! | 12:07 |
*** raildo|away is now known as raildo | 12:08 | |
*** pal_ has joined #openstack-neutron | 12:10 | |
amotoki | pc_m: re VPNaaS documentation, I still see a singular case. | 12:10 |
pc_m | amotoki: Which one? | 12:10 |
amotoki | pc_m: for exmaple, p.204 of http://docs-draft.openstack.org/09/167609/6/check/gate-api-site-tox-doc-publish-checkbuild/0840f26//publish-docs/api-ref-guides/bk-api-ref-networking-v2.pdf | 12:10 |
amotoki | pc_m: for example, /v2.0/vpnaas/vpnservice/{vpnservice_id} | 12:10 |
amotoki | pc_m: I think "vpnservice" should be "vpnservices". | 12:11 |
*** zilberstein has quit IRC | 12:11 | |
pc_m | amotoki: No, I think it is correct. For the LIST API they are plural, for the rest they are singular. | 12:11 |
amotoki | pc_m: checking with devstack | 12:13 |
*** baoli has joined #openstack-neutron | 12:13 | |
*** baoli has quit IRC | 12:14 | |
amotoki | pc_m: for example: neutron vpn-service-delete <xxxx> send DELETE http://172.16.18.35:9696/v2.0/vpn/vpnservices/115e08af-34dd-42a4-8572-32bb458a635d.json | 12:14 |
amotoki | pc_m: it is tested with Juno env. | 12:15 |
*** baoli has joined #openstack-neutron | 12:15 | |
*** apuimedo has quit IRC | 12:15 | |
pc_m | What option did you use to see the URL? I did with --debug and I didn't see that for IKEPolicy | 12:16 |
amotoki | pc_m: the command line is neutron -v vpn-service-delete 115e08af-34dd-42a4-8572-32bb458a635d | 12:16 |
*** pal_ has quit IRC | 12:16 | |
*** dboik has quit IRC | 12:16 | |
openstackgerrit | Pavel Bondar proposed openstack/neutron: Refactor db_base_plugin_v2.py to use new IPAM https://review.openstack.org/153236 | 12:18 |
*** markvoelker has joined #openstack-neutron | 12:18 | |
amotoki | pc_m: I will leave comments on the review. | 12:18 |
*** ociuhandu has quit IRC | 12:18 | |
*** amuller_ has joined #openstack-neutron | 12:18 | |
pc_m | amotoki: Hmmm. I thought they were all singular, but it looks like service is plural. IKEpolicy is singular. Will check others. | 12:19 |
pc_m | amotoki: Would have liked this to be consistent... | 12:19 |
*** mwagner_lap has joined #openstack-neutron | 12:22 | |
*** annegentle has joined #openstack-neutron | 12:22 | |
amotoki | pc_m: all urls have plural form. for example /v2.0/networks/<uuid> are used for create/update/delete | 12:23 |
*** zilberstein has joined #openstack-neutron | 12:23 | |
*** annegentle has quit IRC | 12:23 | |
pc_m | amotoki: You're right. Sorry. I was looking at --debug output and I think I was looking at the wrong info. | 12:26 |
amotoki | pc_m: plural form is always used in URL, but the body contains a singular form for create/delete/update/show. It is common in REST API. | 12:26 |
*** rushil has joined #openstack-neutron | 12:26 | |
*** yatin has quit IRC | 12:27 | |
openstackgerrit | Merged openstack/neutron: Enable Process Monitor by default. https://review.openstack.org/165117 | 12:28 |
*** ociuhandu has joined #openstack-neutron | 12:29 | |
pc_m | amotoki: Will post another patch in a few mins... building and testing... | 12:29 |
amotoki | pc_m: no need to hurry :) | 12:30 |
pc_m | amotoki: Nothing else to do on a rainy Friday :) | 12:30 |
*** atuvenie has quit IRC | 12:32 | |
*** anish_ has quit IRC | 12:33 | |
*** yamamoto has joined #openstack-neutron | 12:33 | |
*** blues-man has quit IRC | 12:38 | |
*** yamamoto has quit IRC | 12:40 | |
*** yuanying has quit IRC | 12:42 | |
*** rook_ has joined #openstack-neutron | 12:45 | |
*** amotoki has quit IRC | 12:50 | |
*** atuvenie has joined #openstack-neutron | 12:52 | |
*** john-davidge has joined #openstack-neutron | 12:56 | |
john-davidge | #openstack-lbaas | 12:57 |
john-davidge | haha, oops | 12:57 |
*** dboik has joined #openstack-neutron | 12:58 | |
*** amotoki has joined #openstack-neutron | 12:58 | |
*** suresh12 has joined #openstack-neutron | 12:58 | |
*** xuhanp has joined #openstack-neutron | 13:03 | |
*** 32NABM7ZF has joined #openstack-neutron | 13:03 | |
*** pal_ has joined #openstack-neutron | 13:04 | |
*** jaypipes is now known as leakypipes | 13:05 | |
*** kbringard has joined #openstack-neutron | 13:13 | |
*** neeti has quit IRC | 13:16 | |
*** pal_ has quit IRC | 13:16 | |
*** nplanel_ has joined #openstack-neutron | 13:21 | |
*** shwetaap has joined #openstack-neutron | 13:22 | |
*** lykinsbd has joined #openstack-neutron | 13:23 | |
*** annegentle has joined #openstack-neutron | 13:24 | |
*** baoli has quit IRC | 13:25 | |
*** atuvenie has quit IRC | 13:27 | |
*** ihrachyshka has quit IRC | 13:27 | |
*** shwetaap1 has joined #openstack-neutron | 13:27 | |
*** ihrachyshka has joined #openstack-neutron | 13:27 | |
*** baoli has joined #openstack-neutron | 13:28 | |
*** ilyashakhat has quit IRC | 13:28 | |
*** annegentle has quit IRC | 13:29 | |
*** shwetaap has quit IRC | 13:30 | |
*** julien_ZTE has joined #openstack-neutron | 13:30 | |
*** chem has joined #openstack-neutron | 13:30 | |
*** ishant has quit IRC | 13:31 | |
*** chandankumar has quit IRC | 13:31 | |
*** jckasper has joined #openstack-neutron | 13:32 | |
*** matrohon has joined #openstack-neutron | 13:32 | |
*** carlp has quit IRC | 13:34 | |
*** jpena is now known as jpena|lunch | 13:39 | |
*** yamamoto has joined #openstack-neutron | 13:40 | |
*** suresh12 has quit IRC | 13:41 | |
*** tiswanso has joined #openstack-neutron | 13:41 | |
*** tiswanso has quit IRC | 13:42 | |
*** tiswanso has joined #openstack-neutron | 13:42 | |
*** annegentle has joined #openstack-neutron | 13:43 | |
openstackgerrit | Pavel Bondar proposed openstack/neutron: IPAM reference driver https://review.openstack.org/150485 | 13:44 |
*** shwetaap1 has quit IRC | 13:45 | |
*** yamamoto has quit IRC | 13:46 | |
*** apuimedo has joined #openstack-neutron | 13:47 | |
*** banix has joined #openstack-neutron | 13:47 | |
*** dims_ is now known as dimsum__ | 13:49 | |
openstackgerrit | Pavel Bondar proposed openstack/neutron: Implement IPAM Driver loader https://review.openstack.org/147479 | 13:50 |
*** f13o has joined #openstack-neutron | 13:50 | |
*** padkrish has joined #openstack-neutron | 13:55 | |
*** ihrachyshka has quit IRC | 13:56 | |
*** flaviof has quit IRC | 13:57 | |
*** ihrachyshka has joined #openstack-neutron | 13:57 | |
*** padkrish has quit IRC | 13:57 | |
*** r-daneel has joined #openstack-neutron | 13:58 | |
*** padkrish has joined #openstack-neutron | 13:58 | |
*** suresh12 has joined #openstack-neutron | 13:58 | |
*** amuller_ is now known as amuller_afk | 13:58 | |
*** salv-orl_ has joined #openstack-neutron | 14:00 | |
*** ferest has quit IRC | 14:00 | |
*** flaviof has joined #openstack-neutron | 14:01 | |
*** dimsum__ has quit IRC | 14:01 | |
*** dimsum__ has joined #openstack-neutron | 14:02 | |
*** tiswanso has quit IRC | 14:02 | |
openstackgerrit | Pavel Bondar proposed openstack/neutron: Refactor db_base_plugin_v2.py to use new IPAM https://review.openstack.org/153236 | 14:03 |
*** salv-orlando has quit IRC | 14:03 | |
*** signed8bit has joined #openstack-neutron | 14:04 | |
*** Marga_ has joined #openstack-neutron | 14:06 | |
*** tiswanso has joined #openstack-neutron | 14:06 | |
*** tiswanso has quit IRC | 14:06 | |
*** tiswanso has joined #openstack-neutron | 14:06 | |
*** annegentle has quit IRC | 14:07 | |
*** _cjones_ has joined #openstack-neutron | 14:08 | |
*** Marga_ has quit IRC | 14:08 | |
*** saikrishna has joined #openstack-neutron | 14:09 | |
*** Marga_ has joined #openstack-neutron | 14:09 | |
*** shwetaap has joined #openstack-neutron | 14:09 | |
*** baoli has quit IRC | 14:10 | |
*** _cjones_ has quit IRC | 14:11 | |
*** ianw has quit IRC | 14:12 | |
*** ianw has joined #openstack-neutron | 14:12 | |
*** suresh12 has quit IRC | 14:13 | |
*** popw has quit IRC | 14:17 | |
*** chandankumar has joined #openstack-neutron | 14:19 | |
*** zzzeek has joined #openstack-neutron | 14:19 | |
*** kodoku has joined #openstack-neutron | 14:20 | |
kodoku | Hi, How I can configure neutron server for connect to keystone with HTTPS and with insecure option ? | 14:21 |
*** ianw has quit IRC | 14:21 | |
*** ianw has joined #openstack-neutron | 14:21 | |
kodoku | because I have issue " HTTP connection exception: Unable to establish connection to https://10.121.141.35:35357/" | 14:21 |
*** saikrishna has quit IRC | 14:27 | |
openstackgerrit | Andrew Boik proposed openstack/neutron: Support multiple IPv6 prefixes on internal router ports https://review.openstack.org/156360 | 14:28 |
openstackgerrit | Kyle Mestery proposed openstack/neutron-specs: Cleanup Kilo specs https://review.openstack.org/168351 | 14:28 |
*** joshw42 has joined #openstack-neutron | 14:28 | |
*** prad has joined #openstack-neutron | 14:29 | |
*** kodoku has left #openstack-neutron | 14:29 | |
*** bobmel_ has joined #openstack-neutron | 14:30 | |
*** Networkn3rd has joined #openstack-neutron | 14:31 | |
*** chandankumar has quit IRC | 14:31 | |
*** annegentle has joined #openstack-neutron | 14:32 | |
*** stockpirate has quit IRC | 14:32 | |
*** bobmel has quit IRC | 14:33 | |
*** reed has joined #openstack-neutron | 14:33 | |
*** achanda has joined #openstack-neutron | 14:35 | |
*** 32NABM7ZF has quit IRC | 14:35 | |
*** xuhanp has quit IRC | 14:35 | |
openstackgerrit | Andrew Boik proposed openstack/neutron: Allow only one IPv4 and one IPv6 fixed IP on router gateway port https://review.openstack.org/167784 | 14:37 |
*** chandankumar has joined #openstack-neutron | 14:38 | |
*** Rajeev has joined #openstack-neutron | 14:39 | |
*** wznoinsk has quit IRC | 14:39 | |
*** chem has quit IRC | 14:40 | |
*** leenheer has joined #openstack-neutron | 14:41 | |
*** zz_jgrimm is now known as jgrimm | 14:41 | |
*** lizk has quit IRC | 14:42 | |
*** achanda has quit IRC | 14:42 | |
*** _cjones_ has joined #openstack-neutron | 14:43 | |
*** wznoinsk has joined #openstack-neutron | 14:44 | |
*** banix has quit IRC | 14:47 | |
*** dansmith is now known as superdan | 14:47 | |
pc_m | dougwig: ping | 14:48 |
*** banix has joined #openstack-neutron | 14:48 | |
*** rangansw has joined #openstack-neutron | 14:50 | |
openstackgerrit | Zengfa Gao proposed openstack/neutron: Add API tests for subnetpool allocation https://review.openstack.org/167611 | 14:52 |
*** svinota|work has joined #openstack-neutron | 14:52 | |
*** aranjan has joined #openstack-neutron | 14:53 | |
*** emagana has joined #openstack-neutron | 14:54 | |
*** gsagie has joined #openstack-neutron | 14:54 | |
*** jpena|lunch is now known as jpena | 14:55 | |
*** Bhavya has quit IRC | 14:56 | |
*** Marga_ has quit IRC | 14:57 | |
*** aranjan has quit IRC | 14:57 | |
*** _cjones_ has quit IRC | 14:58 | |
*** bquan has joined #openstack-neutron | 14:58 | |
*** _cjones_ has joined #openstack-neutron | 14:59 | |
*** carl_baldwin has joined #openstack-neutron | 15:01 | |
*** Rajeev has quit IRC | 15:02 | |
*** bquan_ has joined #openstack-neutron | 15:02 | |
*** doude has quit IRC | 15:04 | |
*** annegentle has quit IRC | 15:04 | |
*** dsneddon has joined #openstack-neutron | 15:05 | |
*** annegentle has joined #openstack-neutron | 15:05 | |
*** bquan has quit IRC | 15:05 | |
*** julim has quit IRC | 15:07 | |
*** julim has joined #openstack-neutron | 15:08 | |
*** chem has joined #openstack-neutron | 15:09 | |
*** devlaps has joined #openstack-neutron | 15:10 | |
*** marun has joined #openstack-neutron | 15:10 | |
*** atuvenie has joined #openstack-neutron | 15:12 | |
*** julim has quit IRC | 15:13 | |
*** dannywilson has joined #openstack-neutron | 15:14 | |
*** julim has joined #openstack-neutron | 15:14 | |
*** dannywilson has quit IRC | 15:14 | |
*** ZZelle has quit IRC | 15:14 | |
*** dannywilson has joined #openstack-neutron | 15:14 | |
*** reed has quit IRC | 15:14 | |
*** marun has quit IRC | 15:15 | |
*** sbelous has quit IRC | 15:15 | |
*** xgerman_ has joined #openstack-neutron | 15:16 | |
*** amuller_afk is now known as amuller | 15:17 | |
*** 17WABICRT has quit IRC | 15:20 | |
*** chandankumar has quit IRC | 15:20 | |
*** ggnel has quit IRC | 15:20 | |
amuller | mestery: About https://review.openstack.org/#/c/164208/ | 15:20 |
*** fifieldt has quit IRC | 15:21 | |
mestery | amuller: Looking | 15:21 |
amuller | It looks like there's some confusion there, maybe some of it could be resolved if you split it up to 2 patches, one that enhances the text / captures what's currently going on, and the other patch is just the rename | 15:21 |
amuller | mestery: you're getting -1's for describing reality... | 15:21 |
amuller | which is ridiculous | 15:21 |
mestery | :) | 15:21 |
mestery | amuller: To be honest, I'd really like to converge with current neutron folks and mege it WITH those -1s | 15:22 |
amuller | yeah I get that | 15:22 |
mestery | Because I find it hilarious that people not involved with neutron, and who SUPPORTED AND ARCHITECTED THE BIG TENT MODEL, are complaining about this change | 15:22 |
mestery | PFft | 15:22 |
amuller | mestery: People are so afraid to make changes because 'other projects are not doing it', someone has to lead... | 15:23 |
mestery | amuller: Yes! We've done that as a team (neutron) a lot recently, so I have no fear :) | 15:23 |
mestery | amuller: We're in a much better place than we were last year for example. | 15:23 |
amuller | It's like a bunch of people looking left and right before crossing the street, but nobody crossing because no one else is | 15:23 |
mestery | amuller: Good analogy | 15:23 |
*** suresh12 has joined #openstack-neutron | 15:24 | |
*** _cjones_ has quit IRC | 15:24 | |
*** ajo has joined #openstack-neutron | 15:25 | |
*** armax has joined #openstack-neutron | 15:28 | |
*** suresh12 has quit IRC | 15:28 | |
openstackgerrit | Zengfa Gao proposed openstack/neutron: Add API tests for subnet-create with subnetpool https://review.openstack.org/168371 | 15:29 |
*** suresh12 has joined #openstack-neutron | 15:29 | |
*** garyk has joined #openstack-neutron | 15:29 | |
*** baoli has joined #openstack-neutron | 15:30 | |
*** baoli has quit IRC | 15:31 | |
pc_m | amuller: Hi. Can you look at https://review.openstack.org/#/c/164466/ again? It has what we talked about before, and a change to mock, based on what armax wanted. | 15:31 |
pc_m | dougwig: ping | 15:32 |
*** rook_ has quit IRC | 15:33 | |
amuller | pc_m: sure | 15:33 |
pc_m | amuller: Thanks. It has a -1 from doug, who had a question, but didn't put in the question. I'm trying to ping him to see what's up. | 15:34 |
*** xgerman has joined #openstack-neutron | 15:34 | |
*** baoli has joined #openstack-neutron | 15:35 | |
amuller | pc_m: +1 | 15:36 |
pc_m | amuller: Thanks! | 15:36 |
*** xgerman_ has quit IRC | 15:38 | |
*** padkrish has quit IRC | 15:38 | |
*** Shrews has joined #openstack-neutron | 15:39 | |
openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Revise functional test hooks (WIP) https://review.openstack.org/168115 | 15:39 |
*** zhhuabj has joined #openstack-neutron | 15:40 | |
*** stockpirate has joined #openstack-neutron | 15:40 | |
stockpirate | Hi, could i come with a question, again? | 15:42 |
stockpirate | ask* | 15:42 |
Shrews | Hello all. I'm am having a difficult time constructing the proper JSON body to update a router through the python API. The documentation is not very helpful here. I have tried these JSON bodies: http://paste.openstack.org/show/197181/ | 15:46 |
Shrews | can anyone shed some light on where I've gone wrong? | 15:46 |
*** vthapar has quit IRC | 15:48 | |
Shrews | I would use the --debug option to the command line client to see what it constructs, but I can't figure out how to specify values with the router-update command. | 15:49 |
*** _cjones_ has joined #openstack-neutron | 15:50 | |
*** dboik_ has joined #openstack-neutron | 15:51 | |
*** Rajeev has joined #openstack-neutron | 15:52 | |
ihrachyshka | baoli, fyi sent dibbler package for fedora review. now waiting for comments and merge. I guess next week is when we see it. | 15:52 |
ihrachyshka | baoli, which fedora versions do we need for the package? | 15:53 |
*** dboik has quit IRC | 15:54 | |
*** lykinsbd_ has joined #openstack-neutron | 15:56 | |
*** lykinsbd has quit IRC | 15:56 | |
*** chachajee has joined #openstack-neutron | 15:56 | |
*** dboik_ has quit IRC | 15:57 | |
*** scheuran has quit IRC | 15:57 | |
*** dboik has joined #openstack-neutron | 15:57 | |
*** tsg has joined #openstack-neutron | 15:57 | |
*** lykinsbd_ has quit IRC | 15:58 | |
*** chachaji has quit IRC | 15:58 | |
*** lykinsbd has joined #openstack-neutron | 15:58 | |
*** YorikSar has quit IRC | 15:59 | |
john-davidge | ihrachyshka: That's great! Thanks. I imagine 20 and 21 will be sufficient | 15:59 |
ihrachyshka | john-davidge, ack | 16:00 |
ihrachyshka | john-davidge, that's basically all that is supported :) | 16:00 |
john-davidge | ihrachyshka: :) | 16:00 |
ihrachyshka | I don't think there will be issues with f20 though, so fine | 16:00 |
baoli | ihrachyshka, thanks for doing the packaging. | 16:00 |
flcn | mestery , emagana Me, with haleyb finish review of https://review.openstack.org/#/c/143672/ Can you please review it one more time? Thanks | 16:00 |
Shrews | Ah, solved my own problem. Nevermind :) | 16:01 |
dougwig | pc_m: ack | 16:01 |
openstackgerrit | Zengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API https://review.openstack.org/159618 | 16:01 |
*** _cjones_ has quit IRC | 16:01 | |
*** _cjones_ has joined #openstack-neutron | 16:02 | |
*** bjornar_ has quit IRC | 16:02 | |
mestery | flcn: ack | 16:02 |
pc_m | dougwig: You gave a -1 on https://review.openstack.org/#/c/164466/ and had a question, but the body of the question was blank... | 16:03 |
*** lykinsbd has quit IRC | 16:03 | |
dougwig | oh nuts, gertty ate my comment. one sec. | 16:03 |
*** carl_baldwin has quit IRC | 16:03 | |
*** nlahouti has joined #openstack-neutron | 16:04 | |
dougwig | re-commented. | 16:04 |
pc_m | dougwig: thanks. will look | 16:04 |
*** thedodd has joined #openstack-neutron | 16:04 | |
amuller | dougwig: your question is answered in the commit message I think | 16:05 |
amuller | dougwig: the L3 uses an event notification system for the metadata proxy, vpnaas and fwaas | 16:05 |
dougwig | amuller: ahh, so that first line is the "old" mechanism? | 16:05 |
pc_m | dougwig: YEah, essentially, we want to do a replacement, but because other repos use, we need to do an add then remove. | 16:05 |
amuller | dougwig: yep | 16:05 |
dougwig | ok, got it. | 16:05 |
*** annegentle has quit IRC | 16:06 | |
amuller | armax: pc_m: Armando has a point though, maybe we want to release K with all systems using the old system, then make a complete switch during L | 16:06 |
amuller | pc_m: Unless there's an urgency here that I'm missing? | 16:06 |
dougwig | armax: i'm in favor of merging that refactor if we think it's low risk enough, because of the point made that if it merges early in liberty, since it's some structural change, it'll make bug fixing backports harder. | 16:07 |
emagana | flcn: Done! | 16:07 |
dougwig | depends on risk level. | 16:07 |
*** anilvenkata has quit IRC | 16:07 | |
pc_m | amuller: no urgency. Vyatta's out of tree drivers are waiting for this so they can update their driver, and it is pretty much done. | 16:08 |
*** Marga_ has joined #openstack-neutron | 16:09 | |
*** atuvenie has quit IRC | 16:09 | |
*** csoukup has joined #openstack-neutron | 16:09 | |
*** suresh12 has quit IRC | 16:09 | |
*** chandankumar has joined #openstack-neutron | 16:09 | |
pc_m | dougwig: right, backports wouldn't have to deal with the change. | 16:09 |
*** numan has quit IRC | 16:10 | |
*** ChuckC has joined #openstack-neutron | 16:11 | |
*** SridharG has left #openstack-neutron | 16:11 | |
flcn | emagana Thanks! | 16:13 |
emagana | flcn: n.p. | 16:13 |
emagana | flcn: Thanks for contributing to the code! | 16:13 |
amuller | pc_m: I'd wait until L | 16:13 |
*** dave-mccowan has joined #openstack-neutron | 16:14 | |
flcn | emagana thanks for help! | 16:14 |
*** Miouge has quit IRC | 16:14 | |
*** dave-mcc_ has joined #openstack-neutron | 16:15 | |
openstackgerrit | Zengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API https://review.openstack.org/159618 | 16:16 |
*** prad has quit IRC | 16:17 | |
*** Swami has joined #openstack-neutron | 16:17 | |
*** padkrish has joined #openstack-neutron | 16:19 | |
*** dave-mccowan has quit IRC | 16:19 | |
openstackgerrit | Merged openstack/neutron: If configured, set the MTU for fpr/rfp intefaces https://review.openstack.org/162312 | 16:19 |
*** yeungp has joined #openstack-neutron | 16:19 | |
openstackgerrit | ShaoRan Xiao proposed openstack/neutron: Definition of the Virtual Network Performance interface Implements:blueprint virtual-network-performance-monitor https://review.openstack.org/164814 | 16:22 |
*** crc32 has joined #openstack-neutron | 16:22 | |
*** aepifanov has quit IRC | 16:22 | |
openstackgerrit | Zengfa Gao proposed openstack/python-neutronclient: Add Neutron subnet-create with subnetpool https://review.openstack.org/165910 | 16:24 |
*** leenheer has quit IRC | 16:25 | |
*** tidwellr has joined #openstack-neutron | 16:25 | |
stockpirate | hmm need some help to understand. when im launch instance from miage in horizon i get http 500 error. And when i do: | 16:27 |
stockpirate | root@vitkencas03:/etc/neutron# nova secgroup-list | 16:27 |
stockpirate | ERROR: The server has either erred or is incapable of performing the requested operation. (HTTP 500) (Request-ID: req-d182fd73-6a33-498b-8398-60a2e47be01f) | 16:27 |
stockpirate | i get exatly the same | 16:27 |
*** ijw has joined #openstack-neutron | 16:27 | |
stockpirate | nova-api.log ConnectionFailed: Connection to neutron failed: Maximum attempts reached | 16:27 |
*** ijw has quit IRC | 16:28 | |
*** pal_ has joined #openstack-neutron | 16:28 | |
openstackgerrit | Zengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API https://review.openstack.org/159618 | 16:28 |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Move final remnants of router processing to router classes https://review.openstack.org/163222 | 16:28 |
*** ijw has joined #openstack-neutron | 16:28 | |
*** carl_baldwin has joined #openstack-neutron | 16:28 | |
*** claudiub has quit IRC | 16:29 | |
*** aranjan has joined #openstack-neutron | 16:29 | |
carl_baldwin | amuller: https://review.openstack.org/#/c/163222/ was an interesting conflict resolution after yours merged. I had to get a little creative. | 16:32 |
*** rangansw has quit IRC | 16:32 | |
carl_baldwin | amuller: I attempted to isolate state_change_callback to HA (mostly). It is passed as an extra parameter to HaRouter when it is created. | 16:33 |
*** chem has quit IRC | 16:33 | |
*** tidwellr has quit IRC | 16:33 | |
*** tidwellr1 has joined #openstack-neutron | 16:33 | |
*** chem has joined #openstack-neutron | 16:34 | |
openstackgerrit | Pavel Bondar proposed openstack/neutron: Refactor db_base_plugin_v2.py to use new IPAM https://review.openstack.org/153236 | 16:34 |
amuller | carl_baldwin: yeah I imagined it'd be a rough merge conflict | 16:35 |
amuller | sorry about that =p | 16:35 |
*** SumitNaiksatam has joined #openstack-neutron | 16:35 | |
*** fredlhsu has joined #openstack-neutron | 16:36 | |
carl_baldwin | amuller, no worries. I just had to deal with that callback and the process_monitor. I think what I've done better isolates that stuff to HA. | 16:36 |
openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support Dual-Stack Gateway Ports on Neutron Routers https://review.openstack.org/149068 | 16:37 |
*** ronis has quit IRC | 16:38 | |
*** suresh12 has joined #openstack-neutron | 16:39 | |
*** tsg has quit IRC | 16:39 | |
*** alaski is now known as lascii | 16:39 | |
*** afazekas has quit IRC | 16:40 | |
carl_baldwin | HenryG: ping | 16:40 |
HenryG | carl_baldwin: pong | 16:40 |
amuller | carl_baldwin: I'll have to look at that patch closely | 16:40 |
amuller | carl_baldwin: ... after lunch :) | 16:40 |
carl_baldwin | HenryG: How are we doing on https://review.openstack.org/#/c/149068/ ? | 16:41 |
carl_baldwin | amuller: Thanks. | 16:41 |
HenryG | carl_baldwin: dane_leblanc and dboik have figured out all the tempest failures and addressed them | 16:42 |
HenryG | carl_baldwin: I'll add a comment with a summary | 16:43 |
*** gsagie has quit IRC | 16:44 | |
*** yamamoto has joined #openstack-neutron | 16:45 | |
carl_baldwin | HenryG, thanks. I'll watch for it. Looks like tests are running on it too. | 16:47 |
*** toutuu has joined #openstack-neutron | 16:48 | |
toutuu | good evening to everyone! o/ | 16:48 |
toutuu | I have a quick question: what is physically an openstack router? is it a little VM? is it a collection of iptables rules? | 16:49 |
*** yamamoto has quit IRC | 16:49 | |
*** alexpilotti has joined #openstack-neutron | 16:50 | |
*** ygbo has quit IRC | 16:53 | |
*** stockpirate has quit IRC | 16:54 | |
carl_baldwin | toutuu: In the reference L3 implementation, the router is created in a Linux network namespace to isolate routing rules, conntrack tables, iptables rules, etc. | 16:55 |
carl_baldwin | toutuu, this is closely related to container (e.g. LXC, docker) networking and is not a full VM. | 16:56 |
*** marun has joined #openstack-neutron | 16:58 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: tests: don't rely on configuration files outside tests directory https://review.openstack.org/165237 | 16:58 |
HenryG | carl_baldwin: comment added | 17:00 |
*** matrohon has quit IRC | 17:00 | |
*** signed8b_ has joined #openstack-neutron | 17:01 | |
*** signed8bit has quit IRC | 17:02 | |
*** seizadi has joined #openstack-neutron | 17:03 | |
*** Marga_ has quit IRC | 17:03 | |
openstackgerrit | Andrew Boik proposed openstack/neutron: Support multiple IPv6 prefixes on internal router ports https://review.openstack.org/156360 | 17:03 |
carl_baldwin | HenryG, ack | 17:03 |
openstackgerrit | Andrew Boik proposed openstack/neutron: Allow update of ext gateway IP's w/out port delete https://review.openstack.org/161073 | 17:04 |
openstackgerrit | Andrew Boik proposed openstack/neutron: Auto-update gateway port after subnet-create https://review.openstack.org/161085 | 17:04 |
*** annegentle has joined #openstack-neutron | 17:04 | |
*** toutuu has quit IRC | 17:05 | |
carl_baldwin | HenryG, just posted my review. | 17:06 |
*** jlanoux has quit IRC | 17:06 | |
sc68cal | mestery: sorry to ask you dumb questions, but my latest e-mail I notice that both mechanism drivers for openvswitch and linuxbridge are enabled in devstack by default, but the openvswitch agent is only selected to run. Does the linuxbridge agent need to be running if you want both ovs and linuxbridge? | 17:07 |
sc68cal | or is there something that would prevent the ovs agent and lb agent from both running? | 17:07 |
HenryG | carl_baldwin: much appreciated! | 17:07 |
*** yamahata has joined #openstack-neutron | 17:07 | |
HenryG | carl_baldwin: anything left around l3agent refactoring you need eyes on? | 17:08 |
amuller | sc68cal: you'd run only either ovs or the lb agent on a single machine | 17:08 |
*** _cjones_ has quit IRC | 17:08 | |
amuller | sc68cal: and having both lb and ovs mech drivers enable on your server enables you to run a mixed environment where each node would run either/or | 17:08 |
sc68cal | amuller: ah.... | 17:08 |
*** seizadi has quit IRC | 17:09 | |
*** seizadi has joined #openstack-neutron | 17:09 | |
amuller | mestery: Are you seeing the discussion about devstack switching from OVS to LB for its default settings? | 17:10 |
amuller | mestery: "Status of the nova-network to Neutron migration work" | 17:11 |
amuller | marun: ^ | 17:11 |
amuller | I made the point that we can't make that switch even if we wanted to without gating on LB | 17:11 |
amuller | but that was ignored :) | 17:11 |
*** lpetrut has left #openstack-neutron | 17:11 | |
marun | who is doing the ignoring? | 17:11 |
*** nplanel_ has quit IRC | 17:12 | |
*** s3wong has joined #openstack-neutron | 17:12 | |
*** Aish has joined #openstack-neutron | 17:12 | |
amuller | To be fair Sean Dague did not reply since I made that comment | 17:13 |
amuller | So really I'm just blowing smoke | 17:13 |
amuller | but anyway, wanted you to be aware of that dicussion | 17:13 |
*** jistr has quit IRC | 17:13 | |
*** seizadi has quit IRC | 17:13 | |
*** seizadi has joined #openstack-neutron | 17:13 | |
marun | It's pretty embarrassing that even openstack devs seem to see ovs as scary. | 17:14 |
openstackgerrit | Ed Bak proposed openstack/neutron: Break calls to get_devices_details_list into subsets https://review.openstack.org/163594 | 17:14 |
sc68cal | Let's first see how bad it explodes - https://review.openstack.org/#/c/168423/1 | 17:15 |
amuller | marun: also they were talking about simplifying, using provider networks, well, you can do that with ovs | 17:15 |
marun | url 1 | 17:15 |
marun | amuller: having 2 code paths to have to exercise is, frankly asinine | 17:15 |
marun | wow, that comma makes that sentence all kinds of awkward | 17:16 |
russellb | ovs seems fairly mature at this point | 17:16 |
russellb | not sure why it's so scary | 17:16 |
marun | russellb: the unknown | 17:16 |
russellb | i also wish the conversation would focus more on concrete gaps people see | 17:16 |
*** achanda has joined #openstack-neutron | 17:16 | |
russellb | so far it's just "too complicated" | 17:16 |
amuller | the gap that OVS has is that its not LB :) | 17:16 |
marun | to be fair, debugging ovs does suck | 17:16 |
*** achanda has quit IRC | 17:16 | |
russellb | which is legit i suppose, assuming it's really just a docs issue ... | 17:16 |
marun | russellb: it kind of is, yeah. | 17:17 |
marun | russellb: lb is simple, at least | 17:17 |
* russellb nods | 17:17 | |
*** achanda has joined #openstack-neutron | 17:17 | |
marun | ovs was designed with a controller that could provide a birds-eye view of the world | 17:17 |
sc68cal | Check the notes from the ops summit - https://etherpad.openstack.org/p/PHL-ops-OVS | 17:17 |
*** dboik_ has joined #openstack-neutron | 17:17 | |
sc68cal | people want distributed router for linxubridge.... :-\ | 17:17 |
marun | in the absence of that controller, the complexity is hard to grapple with compared with lb | 17:17 |
marun | sc68cal: also, pony ;) | 17:17 |
amuller | DVR for LB will never happen | 17:18 |
sc68cal | marun: pony with a horn | 17:18 |
russellb | i think a lot of the complains on that etherpad are very ovs+neuron, not ovs itself | 17:18 |
marun | heh | 17:18 |
*** rushil has quit IRC | 17:19 | |
sc68cal | Part of it is knowledge gap - Matt (Sam-I-Am) and others have been trying to fix that with the new networking guide | 17:19 |
russellb | sc68cal: yes | 17:19 |
sc68cal | so that people are less scared of OVS | 17:19 |
mestery | marun amuller: I'm burned out of arguing with people who don't want to broaden their horizons | 17:19 |
sdague | russellb: too complex is valid. If you have learn a ton of new stuff all at once, that's a problem. Much like breaking up patch series into digestable chunks, these things have to be in digestable deploy chunks. | 17:19 |
mestery | I mean, LB isn't tested at all with neutron right now | 17:20 |
mestery | To say we're going to use that is scary at best | 17:20 |
russellb | sdague: yep, just worried there's more that isn't surfaced | 17:20 |
amuller | mestery: sc68cal pushed a devstack patch to change the default to LB. Let's see what happens | 17:20 |
*** dboik has quit IRC | 17:20 | |
amuller | (re: testing) | 17:20 |
*** bjornar_ has joined #openstack-neutron | 17:21 | |
mestery | amuller: Again, is that tested anywhere? | 17:21 |
sc68cal | If it explodes, that's a good data point to say look it's not feasible | 17:21 |
mestery | amuller: Because if it's not, we've just created a whole new mess for ourselves | 17:21 |
mestery | marun: ^^^^ | 17:21 |
sdague | russellb: if you've mentally stack overflowed people in looking at it, you aren't going to get good feedback | 17:21 |
*** leenheer has joined #openstack-neutron | 17:21 | |
*** ijw has quit IRC | 17:22 | |
sc68cal | One thing to ask - if we aren't testing linuxbridge, why do we have it in tree? | 17:22 |
*** apuimedo has quit IRC | 17:22 | |
sc68cal | I mean we've thrown vendors out for having no CI | 17:22 |
marun | sc68cal: we've been asking the same question for a whiel | 17:22 |
russellb | ovs has matured *a lot* in the last few years i'd say | 17:22 |
russellb | doesn't make as much sense anymore | 17:23 |
mestery | marun: ++ | 17:23 |
mestery | russellb: Right | 17:23 |
sc68cal | So is this an education issue? Everyone thinks "linuxbridge is simple, how could it possibly not work?" | 17:23 |
sdague | russellb: that's kind of irrelevant | 17:23 |
russellb | ? | 17:23 |
mestery | sdague: How so? | 17:23 |
*** saikrishna has joined #openstack-neutron | 17:23 | |
marun | I don't think we can afford to target both ovs and lb as well-tested solutions upstream. | 17:24 |
sdague | you still have to understand the ovs model to do work completely unrelated to networking when things go wrong | 17:24 |
marun | We need a single good path | 17:24 |
marun | and ovs covers more use-cases | 17:24 |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Honor the default ipv6 pool configuration setting https://review.openstack.org/168098 | 17:24 |
openstackgerrit | Merged openstack/neutron: Don't eagerly load ranges from IPAllocationPool https://review.openstack.org/168214 | 17:24 |
marun | So regardless of folks concern, unless they're willing to do yeoman's work in maintaining lb, I'm not sure we should keep supporting it directly. | 17:24 |
marun | i.e. it can be maintained outside of the tree by folks that care about it | 17:25 |
sdague | I realize I'm annoying tons of people, but this is the best compromise out there that I've managed to come up with. If someone wants to champion different approaches feel free | 17:25 |
sdague | but everything else has been face plant here | 17:25 |
marun | sdague: we can't make everyone happy | 17:25 |
sdague | I think this is a path to actual n-net deprecation | 17:25 |
marun | sdague: we fail hard when we try | 17:26 |
dougwig | sdague: are you saying we can't make OVS simple enough in the single box case for anyone to ever accept it? | 17:26 |
sdague | dougwig: yep | 17:26 |
sdague | and that many large deployers were reverting from ovs -> linuxbridge because of per reasons | 17:27 |
sdague | which came up at phili ops meetup | 17:27 |
dougwig | sdague: that strikes me as odd, since when i started with devstack, i had no clue what underlying network packages were at play, and it didn't matter. what is the use case stopper with OVS? | 17:27 |
sdague | s/per/performance/ | 17:27 |
*** jpena has quit IRC | 17:27 | |
marun | can we please quantify 'many large deployers'? | 17:27 |
*** zilberstein has quit IRC | 17:27 | |
sdague | people that spoke up | 17:28 |
marun | see -> we can't please everyone | 17:28 |
marun | we need to pick our battles on objective criteriea | 17:28 |
marun | criteria | 17:28 |
*** yamahata has quit IRC | 17:28 | |
dougwig | ok, so performance of large delpyments is a reason. seems like a reason for those deployers to help support linuxbridge. but ... how does that affect the devstack issue? | 17:28 |
sdague | sure, so we have this plan and are executing it. Someone else wants to actually work on solving this problem, please do. | 17:28 |
openstackgerrit | Maru Newby proposed openstack/neutron: Simplify retargetable test framework https://review.openstack.org/167075 | 17:29 |
*** s3wong has quit IRC | 17:30 | |
mestery | So, to me, if we are going to get LB in there a bit more, we really need to see it tested a bit more, that's the only thing that struck me in that thread. | 17:30 |
*** pal_ has quit IRC | 17:30 | |
*** ijw has joined #openstack-neutron | 17:30 | |
sdague | mestery: sure, testing was assumed | 17:30 |
sdague | sorry, I didn't think I needed to call that out explicitly | 17:31 |
mestery | sdague: What are infras thoughts on testing LB and OVS in the gate? It may increase our jobs. /cc marun | 17:31 |
*** s3wong has joined #openstack-neutron | 17:31 | |
amuller | So we start finding issues specific to the LB agent, who's going to fix those issues? What justification is there to spending more of our time fixing the LB agent when the OVS agent is more than twice as popular? | 17:31 |
*** gsagie has joined #openstack-neutron | 17:32 | |
mestery | amuller: I would hope the operators who want LB would dedicate some resources to it's maintenance | 17:32 |
marun | mestery: double the number of jobs? If not, we may have a disparity in the amount of testing and a consequent lack of confidence in the one that gets less attention. | 17:32 |
*** suresh12 has quit IRC | 17:32 | |
dougwig | sdague: the etherpad lists deployment/performance issues. that's certainly valid. but back to devstack, the only relevant item is "When it breaks. Not if, when.". any chance that can be characterized more? i haven't had that experience. | 17:32 |
marun | see: postgres | 17:32 |
*** safchain has quit IRC | 17:33 | |
*** tflynn has joined #openstack-neutron | 17:33 | |
mestery | I had hoped as part of documenting the nova-net/neutron gap we could show how OVS might fit in here too, my only concern is around mainting the LB implementatino | 17:33 |
sdague | dougwig: devstack at random code points breaks all the time, and you have to go poke at the system to figure out what's going on | 17:33 |
mestery | However, given we're planning to spin out the in-tree reference implementation, maybe this is all moot anyways | 17:33 |
russellb | mestery: yeah, part of the gap plan was supposed to be writing these use-case mapping docs | 17:34 |
*** gongysh_ has joined #openstack-neutron | 17:34 | |
*** gongysh_ has quit IRC | 17:34 | |
dougwig | sdague: right, i'm not disputing that folks are seeing issues. i was hoping for some specifics around 'breaks all the time'. | 17:35 |
*** Marga_ has joined #openstack-neutron | 17:35 | |
*** Networkn3rd has quit IRC | 17:36 | |
*** suresh12 has joined #openstack-neutron | 17:37 | |
sdague | dougwig: well I'll appologize for not having every scenario in my head at this point. I've spent a bunch of time trying to synthesize information coming in and put it into something digestable and with a path forward. | 17:38 |
*** dave-mcc_ has quit IRC | 17:38 | |
*** seizadi has quit IRC | 17:39 | |
mestery | Maybe the plan is we see the current state of LB agent, and try to map the (hopefully) easy use cases of nova-net into it, and then try to migrate folks to OVS? | 17:39 |
mestery | Too many steps? | 17:39 |
mestery | Too complicated? | 17:39 |
mestery | Thoughts? | 17:39 |
*** ajmiller has joined #openstack-neutron | 17:39 | |
sc68cal | mestery: I think that's the plan | 17:39 |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: Simple subnetpool allocation quotas https://review.openstack.org/165264 | 17:39 |
*** shwetaap has quit IRC | 17:40 | |
sc68cal | use linuxbridge to help ease transition from nova-net | 17:40 |
sc68cal | (if it works) | 17:40 |
mestery | And make it work for the use cases nova-net had | 17:40 |
mestery | not all the neutron cases | 17:40 |
sdague | mestery: correct | 17:40 |
*** Networkn3rd has joined #openstack-neutron | 17:40 | |
sdague | you saw my mixins email right | 17:40 |
mestery | That seems like a reasonable thing to do perhaps | 17:40 |
*** YorikSar has joined #openstack-neutron | 17:40 | |
mestery | YEah, slowly catching up | 17:41 |
sdague | if you want the complicated bits, you need to move to ovs | 17:41 |
sdague | but the base starting point is lb, because it's simple | 17:41 |
openstackgerrit | Maru Newby proposed openstack/neutron: Remove neutron.tests.sub_base https://review.openstack.org/168430 | 17:41 |
dougwig | sdague: that's great, and i appreciate it. i was just looking for if there was some feedback we could use to improve those issues. i'll admit to never having used LB with neutron, simply because devstack always installed OVS, and it's what I learned that way, and have grown to like it's flexibility, even if i railed at it a bit at first. though i've | 17:41 |
dougwig | railed at most of openstack at one point or another. | 17:41 |
*** aepifanov has joined #openstack-neutron | 17:41 | |
sdague | then you can get used to neutron services, how and/when they fail, ensure they are working, get comfortable | 17:41 |
sdague | then you flip a config to introduce ovs if you want to | 17:42 |
sdague | now you know if it's ovs and your environment going odd, as that change is bisectable | 17:42 |
sdague | same thing with tenant self service networks later | 17:42 |
*** sridhar_ram has joined #openstack-neutron | 17:43 | |
*** banix has quit IRC | 17:43 | |
*** _cjones_ has joined #openstack-neutron | 17:44 | |
*** SumitNaiksatam has quit IRC | 17:44 | |
*** amotoki has quit IRC | 17:45 | |
*** catohornet has joined #openstack-neutron | 17:45 | |
*** sbfox has joined #openstack-neutron | 17:46 | |
openstackgerrit | Maru Newby proposed openstack/neutron: Simplify retargetable test framework https://review.openstack.org/167075 | 17:47 |
openstackgerrit | Michael Smith proposed openstack/neutron: L3 Agent support for routers with HA and DVR https://review.openstack.org/139686 | 17:48 |
*** mwagner_lap has quit IRC | 17:49 | |
openstackgerrit | Maru Newby proposed openstack/neutron: Move API tests to neutron.test.api https://review.openstack.org/167320 | 17:51 |
*** seizadi has joined #openstack-neutron | 17:51 | |
*** gsagie has quit IRC | 17:53 | |
*** john-davidge has quit IRC | 17:53 | |
*** jprovazn has quit IRC | 17:53 | |
*** pritesh has joined #openstack-neutron | 17:54 | |
*** Rajeev has quit IRC | 17:54 | |
*** ronis has joined #openstack-neutron | 17:55 | |
openstackgerrit | Michael Smith proposed openstack/neutron: delay fip namespace cleanup/destroy https://review.openstack.org/151758 | 17:56 |
*** yamahata has joined #openstack-neutron | 17:57 | |
openstackgerrit | Kyle Mestery proposed openstack/neutron: Add language around re-proposing specs for new releases https://review.openstack.org/168434 | 18:01 |
*** Networkn_ has joined #openstack-neutron | 18:01 | |
*** saikrishna has quit IRC | 18:01 | |
*** sridhar_ram1 has joined #openstack-neutron | 18:02 | |
*** sridhar_ram has quit IRC | 18:04 | |
openstackgerrit | Maru Newby proposed openstack/neutron: Remove neutron.tests.sub_base https://review.openstack.org/168430 | 18:05 |
flcn | mestery Can you please review http://review.openstack.org/#/c/143672/ one more time? I fix all small problems | 18:05 |
*** Networkn3rd has quit IRC | 18:05 | |
*** leenheer has quit IRC | 18:06 | |
*** leenheer has joined #openstack-neutron | 18:07 | |
mestery | flcn: Looking now. | 18:07 |
*** nlahouti has quit IRC | 18:10 | |
mestery | marun: Do we do any testing of the metering agent in the gate at all? I know it runs, but any idea what tests we do? | 18:10 |
mestery | This is in relation to this one: http://review.openstack.org/#/c/143672/ | 18:10 |
marun | mestery: unsure | 18:10 |
mestery | marun: Just trying to understand what our test coverage is at this point. | 18:10 |
mestery | marun: YEah, me too | 18:10 |
marun | mestery: It looks like unit test only | 18:11 |
mestery | marun: That's what I was afraid of. I guess we can add that to what else we should look at adding more testing to in Liberty. | 18:11 |
marun | mestery: who's crazy enough to use the metering agent, anyway? | 18:12 |
mestery | marun: o_O | 18:12 |
*** rotbart has quit IRC | 18:12 | |
marun | mestery: it's not like it has any guarantee against failure | 18:12 |
mestery | marun: I hear you | 18:13 |
marun | see: ceilometer being an unmitigated disaster | 18:13 |
mestery | yup | 18:13 |
marun | ah, tgif :) | 18:14 |
mestery | rofl | 18:14 |
* sc68cal looks forward to when neutron is not the whipping boy anymore | 18:15 | |
amuller | marun: I love how you use every opportunity to diss (To put it nicely...) Ceilometer | 18:15 |
marun | amuller: it's such an easy target | 18:15 |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: Simple subnetpool allocation quotas https://review.openstack.org/165264 | 18:16 |
amuller | marun: Maybe I should learn more about it so I can feel more comfortable dissing it | 18:16 |
amuller | Pretty noble reason to learn about something I think | 18:16 |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Adds DVR functional test for multi-external networks https://review.openstack.org/163573 | 18:17 |
russellb | i think uninformed dissing is quite reckless, indeed :) | 18:17 |
*** ijw has quit IRC | 18:17 | |
mestery | amuller: lol | 18:17 |
*** MarceloAmaral has joined #openstack-neutron | 18:18 | |
*** sridhar_ram1 has quit IRC | 18:19 | |
MarceloAmaral | Swami: hi | 18:19 |
MarceloAmaral | Swami: I am interested in the ODL summer intership | 18:20 |
*** tidwellr1 has quit IRC | 18:20 | |
*** shwetaap has joined #openstack-neutron | 18:20 | |
*** sridhar_ram has joined #openstack-neutron | 18:20 | |
amuller | armax: re: https://review.openstack.org/#/c/126182/ - I'm looking in to manipulating list_columns according to the response the client got from the server | 18:21 |
*** banix has joined #openstack-neutron | 18:22 | |
Flcn_ | mestery thanks! | 18:22 |
Swami | MarceloAmaral: hi | 18:22 |
*** afazekas has joined #openstack-neutron | 18:23 | |
*** seizadi has quit IRC | 18:23 | |
*** seizadi has joined #openstack-neutron | 18:23 | |
*** seizadi1 has joined #openstack-neutron | 18:24 | |
*** seizadi has quit IRC | 18:24 | |
openstackgerrit | Maru Newby proposed openstack/neutron: Move API tests to neutron.test.api https://review.openstack.org/167320 | 18:25 |
openstackgerrit | Maru Newby proposed openstack/neutron: Simplify retargetable test framework https://review.openstack.org/167075 | 18:25 |
*** svinota|work has quit IRC | 18:25 | |
*** saikrishna has joined #openstack-neutron | 18:26 | |
*** lukasa has quit IRC | 18:27 | |
*** dkehn has quit IRC | 18:31 | |
*** enikanorov has quit IRC | 18:32 | |
*** Sukhdev has joined #openstack-neutron | 18:32 | |
*** ZZelle has joined #openstack-neutron | 18:33 | |
*** dane_leblanc has quit IRC | 18:33 | |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Implement default subnet pool configuration settings https://review.openstack.org/168098 | 18:33 |
*** dkehn has joined #openstack-neutron | 18:34 | |
*** ijw has joined #openstack-neutron | 18:36 | |
*** afazekas has quit IRC | 18:37 | |
*** yatin has joined #openstack-neutron | 18:37 | |
*** ijw_ has joined #openstack-neutron | 18:39 | |
ZZelle | hi | 18:39 |
ZZelle | it seems neutron-vpnaas code or tests are broken | 18:39 |
ZZelle | who should i inform? | 18:39 |
*** seizadi has joined #openstack-neutron | 18:40 | |
pc_m | ZZelle: me. | 18:40 |
*** seizadi1 has quit IRC | 18:40 | |
pc_m | ZZelle: I see it already. I'm working on a patch | 18:40 |
ZZelle | pc_m, ok, great | 18:40 |
*** annegentle has quit IRC | 18:40 | |
pc_m | ZZelle: I don't know what change caused the failure, and I don't know why the mock.patch is not working, but my thought for a solution is to mock it much higher (in VPN code), instead of in Neutron. | 18:41 |
*** annegentle has joined #openstack-neutron | 18:41 | |
ZZelle | pc_m, it would be better as i would also reduce coupling with neutron | 18:41 |
pc_m | ZZelle: The code tries to mock neutron.agent.linux.utils.execute() | 18:41 |
pc_m | ZZelle: I don't know why that patch is no longer working. | 18:42 |
*** ijw has quit IRC | 18:42 | |
pc_m | ZZelle: and only for the StrongSwan driver. | 18:42 |
*** smccully has joined #openstack-neutron | 18:43 | |
*** lukasa has joined #openstack-neutron | 18:43 | |
openstackgerrit | Andrew Boik proposed openstack/neutron: Allow only one IPv4 and one IPv6 fixed IP on router gateway port https://review.openstack.org/167784 | 18:43 |
*** crc32 has quit IRC | 18:46 | |
openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Revise functional test hooks (WIP) https://review.openstack.org/168115 | 18:47 |
*** yatin has quit IRC | 18:47 | |
ZZelle | pc_m, you mock neutron.agent.common.utils.execute | 18:47 |
ZZelle | pc_m, you SHOULD mock neutron.agent.common.utils.execute | 18:47 |
*** ihrachyshka has quit IRC | 18:47 | |
*** saikrishna has quit IRC | 18:48 | |
pc_m | ZZelle: You mean it was doing neutron.agent.linux.utils.execute, right? | 18:48 |
pc_m | ZZelle: I was thinking that the conditional import in common.utils.py was not being handled as exepected, but decided to mock in VPN code instead. | 18:49 |
*** med_ has joined #openstack-neutron | 18:49 | |
pc_m | ZZelle: neutron.agent.linux.utils.execute -> neutron.agent.common.utils.execute | 18:49 |
ZZelle | pc_m, yes | 18:50 |
pc_m | ZZelle: Just surprised me that it still didn't work. In any case, I'll do it in VPN code. | 18:50 |
*** SumitNaiksatam has joined #openstack-neutron | 18:50 | |
ZZelle | pc_m, mock should "mock" the imported module which is neutron.agent.common.utils.execute | 18:51 |
pc_m | ZZelle: I'm testing now. Hope to push it out soon (trying to fix a few other things in the UT as well). | 18:51 |
ZZelle | pc_m, https://github.com/openstack/neutron/commit/f0d9410a8268e01369a43c5159621f3083855b5f#diff-6a3c59b7b71c12cc29eca4c85268864eL22 | 18:51 |
openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 18:53 |
pc_m | ZZelle: thanks! | 18:54 |
*** padkrish has quit IRC | 18:54 | |
ZZelle | pc_m, it works? | 18:54 |
pc_m | ZZelle: I mean thanks for the link, I didn't know what the change was that altered this. | 18:54 |
*** padkrish has joined #openstack-neutron | 18:54 | |
*** tsg has joined #openstack-neutron | 18:55 | |
*** tsg_ has joined #openstack-neutron | 18:57 | |
*** imcsk8 has quit IRC | 18:58 | |
*** imcsk8 has joined #openstack-neutron | 18:58 | |
*** tidwellr has joined #openstack-neutron | 18:58 | |
openstackgerrit | Assaf Muller proposed openstack/python-neutronclient: Add HA router state to l3-agent-list-hosting-router https://review.openstack.org/126182 | 18:58 |
*** padkrish has quit IRC | 18:59 | |
*** tsg has quit IRC | 19:00 | |
*** fredlhsu has quit IRC | 19:02 | |
*** sridhar_ram1 has joined #openstack-neutron | 19:03 | |
*** seizadi has quit IRC | 19:03 | |
*** seizadi1 has joined #openstack-neutron | 19:03 | |
*** sridhar_ram has quit IRC | 19:04 | |
*** dane_leblanc has joined #openstack-neutron | 19:05 | |
*** seizadi1 has quit IRC | 19:05 | |
*** seizadi has joined #openstack-neutron | 19:06 | |
*** alexpilotti has quit IRC | 19:07 | |
*** nlahouti has joined #openstack-neutron | 19:08 | |
*** emagana has quit IRC | 19:08 | |
*** chandankumar has quit IRC | 19:12 | |
*** melwitt has joined #openstack-neutron | 19:14 | |
*** Shrews has left #openstack-neutron | 19:15 | |
openstackgerrit | Ed Bak proposed openstack/neutron: Return from check_ports_exist_on_l3agent if no subnet found https://review.openstack.org/154289 | 19:17 |
openstackgerrit | Merged openstack/neutron: Send only one rule in queue on rule create/delete https://review.openstack.org/143672 | 19:17 |
*** virtuallynathan_ has quit IRC | 19:17 | |
*** rangansw has joined #openstack-neutron | 19:19 | |
*** afazekas has joined #openstack-neutron | 19:24 | |
*** mwagner_lap has joined #openstack-neutron | 19:25 | |
*** flcn has quit IRC | 19:28 | |
*** flcn has joined #openstack-neutron | 19:30 | |
*** afazekas has quit IRC | 19:32 | |
*** igordcard_ has joined #openstack-neutron | 19:35 | |
*** devlaps has quit IRC | 19:36 | |
*** angela-s has joined #openstack-neutron | 19:39 | |
*** emagana has joined #openstack-neutron | 19:41 | |
*** kiran-r has joined #openstack-neutron | 19:41 | |
*** leakypipes has quit IRC | 19:42 | |
*** banix has quit IRC | 19:42 | |
openstackgerrit | Doug Wiegley proposed openstack/neutron-specs: Split neutron-lib out of neutron https://review.openstack.org/154736 | 19:45 |
*** emagana has quit IRC | 19:46 | |
openstackgerrit | Doug Wiegley proposed openstack/neutron-specs: Split neutron-lib out of neutron https://review.openstack.org/154736 | 19:47 |
openstackgerrit | Doug Wiegley proposed openstack/neutron-specs: Split neutron-lib out of neutron https://review.openstack.org/154736 | 19:47 |
*** yamamoto has joined #openstack-neutron | 19:47 | |
*** kiran-r has quit IRC | 19:49 | |
*** yamamoto has quit IRC | 19:52 | |
*** shwetaap has quit IRC | 19:55 | |
*** shwetaap has joined #openstack-neutron | 20:00 | |
*** emagana has joined #openstack-neutron | 20:01 | |
*** svinota|work has joined #openstack-neutron | 20:05 | |
*** svinota|work has left #openstack-neutron | 20:05 | |
*** baoli has quit IRC | 20:05 | |
*** banix has joined #openstack-neutron | 20:07 | |
*** afazekas has joined #openstack-neutron | 20:07 | |
*** baoli has joined #openstack-neutron | 20:07 | |
*** achanda has quit IRC | 20:08 | |
*** aranjan has quit IRC | 20:09 | |
*** matrohon has joined #openstack-neutron | 20:09 | |
*** padkrish has joined #openstack-neutron | 20:10 | |
*** fredlhsu has joined #openstack-neutron | 20:11 | |
*** achanda has joined #openstack-neutron | 20:12 | |
*** aranjan has joined #openstack-neutron | 20:14 | |
*** ociuhandu has quit IRC | 20:16 | |
openstackgerrit | Zengfa Gao proposed openstack/neutron: Add API tests for subnet-create with subnetpool https://review.openstack.org/168371 | 20:17 |
pc_m | ZZelle: ping | 20:18 |
ZZelle | pc_m, pong | 20:18 |
openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS Fix unit test breakage https://review.openstack.org/168472 | 20:18 |
pc_m | ZZelle: I pushed up a simple change. Had a more extensive one, not using Neutron, but see other issues, so need more time to really fix that right. | 20:19 |
pc_m | ZZelle: ^^ is the simple fix. | 20:19 |
ZZelle | pc_m, no pb | 20:19 |
*** emagana has quit IRC | 20:19 | |
*** padkrish has quit IRC | 20:19 | |
pc_m | ZZelle: I need to head out for a bit, but will check back later tonight. | 20:19 |
*** Networkn_ has quit IRC | 20:20 | |
pc_m | dougwig: mestery: Can you review https://review.openstack.org/168472. Fixes unit test breakage. | 20:20 |
openstackgerrit | Zengfa Gao proposed openstack/neutron: Add API tests for subnet-create with subnetpool https://review.openstack.org/168371 | 20:20 |
ZZelle | pc_m, could you remove your associated -1 on https://review.openstack.org/168169 ? | 20:20 |
*** baoli has quit IRC | 20:21 | |
pc_m | ZZelle: sure | 20:21 |
*** pritesh has quit IRC | 20:26 | |
*** pritesh_ has joined #openstack-neutron | 20:27 | |
*** Sukhdev has quit IRC | 20:32 | |
*** s3wong has quit IRC | 20:35 | |
*** s3wong has joined #openstack-neutron | 20:36 | |
*** lukasa has quit IRC | 20:37 | |
*** MarceloAmaral has quit IRC | 20:37 | |
*** lukasa has joined #openstack-neutron | 20:39 | |
*** britthouser has quit IRC | 20:39 | |
*** seizadi has quit IRC | 20:40 | |
*** seizadi has joined #openstack-neutron | 20:40 | |
openstackgerrit | Henry Gessau proposed openstack/neutron: Modify a different agent in test_update_agent_description https://review.openstack.org/168213 | 20:42 |
*** c_soukup has joined #openstack-neutron | 20:42 | |
HenryG | marun: ^^ that should help with the api job stability | 20:43 |
*** achanda has quit IRC | 20:43 | |
marun | HenryG: has it been flaky? | 20:43 |
marun | HenryG: and does that imply we shouldn't let it vote just yet? | 20:44 |
HenryG | marun: I only noticed it twice. Seems fairly rare. Details in the bug. | 20:44 |
*** raildo has quit IRC | 20:44 | |
*** csoukup has quit IRC | 20:45 | |
*** tiswanso has quit IRC | 20:45 | |
openstackgerrit | Dane LeBlanc proposed openstack/neutron: No allocation needed for specific IPv6 SLAAC addr assignment https://review.openstack.org/164943 | 20:46 |
marun | HenryG: Thanks! | 20:46 |
*** afazekas has quit IRC | 20:47 | |
*** lukasa has quit IRC | 20:47 | |
ZZelle | dougwig, mestery, hi | 20:48 |
mestery | ZZelle: yo! | 20:48 |
ZZelle | dougwig, mestery, could you look at https://review.openstack.org/168164? This change avoids breaking neutron-fwaas after a rework on neutron test code | 20:49 |
*** rangansw has quit IRC | 20:49 | |
*** emagana has joined #openstack-neutron | 20:50 | |
*** achanda has joined #openstack-neutron | 20:50 | |
mestery | ZZelle: Nice work! I'll leave it to dougwig to +A | 20:51 |
ZZelle | mestery, there is the same change for vpnaas but i wait for pcm change merge | 20:51 |
mestery | ZZelle: Thanks for fixing this! /cc pc_m | 20:52 |
ZZelle | mestery, we should find a way in liberty to better handle such cases through loose coupling or cross-repo tests | 20:53 |
*** emagana has quit IRC | 20:54 | |
*** mfisch has quit IRC | 20:56 | |
*** Guest90957 has joined #openstack-neutron | 20:58 | |
-openstackstatus- NOTICE: Gerrit maintenance commences in 1 hour at 22:00 UTC http://lists.openstack.org/pipermail/openstack-dev/2015-March/059948.html | 21:01 | |
*** aranjan has quit IRC | 21:02 | |
*** seizadi has quit IRC | 21:05 | |
*** seizadi has joined #openstack-neutron | 21:05 | |
openstackgerrit | Divya ChanneGowda proposed openstack/neutron: Update contribute.rst with Nuage plugin decomposition https://review.openstack.org/165536 | 21:06 |
*** achanda has quit IRC | 21:07 | |
*** rkukura has quit IRC | 21:08 | |
*** firl has joined #openstack-neutron | 21:08 | |
firl | anyone available to help me figure out where my problem is ? I am trying to create an external network and when I do I cannot ping on that net | 21:08 |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Implement default subnet pool configuration settings https://review.openstack.org/168098 | 21:08 |
firl | guess this should be in just the parent room of #openstack | 21:10 |
*** aepifanov has quit IRC | 21:10 | |
*** annegentle has quit IRC | 21:10 | |
*** padkrish has joined #openstack-neutron | 21:11 | |
*** absubram has quit IRC | 21:11 | |
*** absubram has joined #openstack-neutron | 21:12 | |
*** chan1 has joined #openstack-neutron | 21:12 | |
chan1 | hi! is there somebody here which could help me to understand something? | 21:13 |
chan1 | when im running: nova boot --flavor m1.tiny --image Udt01 --nic net-id=f10fa830-5bd2-455e-9c79-134421d76cb2 --security-group default --key-name demo-key demo-instance1 | 21:13 |
chan1 | the nova-api.log tells me: NoNetworksFound: No networks defined. | 21:14 |
chan1 | neutron net-list listening two networks. | 21:14 |
chan1 | but off course nova network-list is empty | 21:14 |
chan1 | my question is: Do i have to configure legacy networking also? | 21:16 |
*** bquan has joined #openstack-neutron | 21:16 | |
*** pritesh_ has quit IRC | 21:17 | |
chan1 | or is openstack networkin | 21:17 |
*** pritesh has joined #openstack-neutron | 21:17 | |
chan1 | sorry.. **or is openstack networking (openstack) or legacy nova network like a "whether or" | 21:18 |
*** baoli has joined #openstack-neutron | 21:18 | |
*** bquan_ has quit IRC | 21:19 | |
chan1 | i read the doc: Proceed to network configuration for the example OpenStack Networking (neutron) or legacy networking (nova-network) architecture. | 21:20 |
chan1 | both is conflict?? | 21:20 |
*** seizadi1 has joined #openstack-neutron | 21:20 | |
*** Marga_ has quit IRC | 21:21 | |
*** seizadi has quit IRC | 21:21 | |
*** rkukura has joined #openstack-neutron | 21:24 | |
*** dkehn has quit IRC | 21:24 | |
chan1 | I can buy you a beer for an answer!!! | 21:24 |
*** yfauser has joined #openstack-neutron | 21:25 | |
*** yfauser has left #openstack-neutron | 21:26 | |
pritesh | chan1: you can't have nova-network and neutron running at same time as far as i know, but since you see the network under neutron net-list it means you have configured neutron, which means the nova-network is disabled. | 21:26 |
*** achanda has joined #openstack-neutron | 21:27 | |
ZZelle | marun, | 21:28 |
ZZelle | marun, hi | 21:28 |
pritesh | chan1: here is the link to get it wokring for devstack : https://wiki.openstack.org/wiki/NeutronDevstack | 21:28 |
marun | ZZelle: hi | 21:28 |
ZZelle | marun, about https://review.openstack.org/#/c/165866/7/neutron/tests/common/net_helpers.py | 21:28 |
marun | ZZelle: re: ports | 21:29 |
marun | ZZelle: removing an ovs bridge automatically removes ports | 21:29 |
openstackgerrit | Merged openstack/neutron: Use router state in get_ha_device_name and ha_network_added https://review.openstack.org/167462 | 21:29 |
chan1 | Pritesh! thank you! i will remove nova-network from the node. ifconfig find a network card virbr0 | 21:29 |
chan1 | which is from nova-network | 21:30 |
openstackgerrit | Andrew Boik proposed openstack/neutron: Support multiple IPv6 prefixes on internal router ports https://review.openstack.org/156360 | 21:30 |
marun | ZZelle: so I don't see why fixture has to do it explicitly - it's unnecessary overhead | 21:30 |
marun | ZZelle: in the same way that manually removing rows in a table doesn't make sense if teardown removes the db | 21:30 |
marun | ZZelle: does that make sense? | 21:30 |
*** mlavalle has joined #openstack-neutron | 21:30 | |
chan1 | Pritesh: nova-api-log says: NoNetworksFound: No networks defined. | 21:31 |
ZZelle | marun, but deleting a db ensures table delete, deleting a bridge does not ensure it deletes ports in general as you don't know the backend behavior | 21:31 |
marun | ZZelle: it does when we're talking about ovs | 21:31 |
marun | ZZelle: I'm not saying we never delete | 21:32 |
marun | ZZelle: I'm saying for ovs, we don't need to | 21:32 |
marun | ZZelle: For other bridge types, I'm not saying we don't have to delete | 21:32 |
ZZelle | so if someone does with PortFixture(): it won't behave as expected | 21:32 |
marun | ZZelle: why not? | 21:32 |
ZZelle | because the port won't be deleted on port cleanup | 21:33 |
marun | hmmm | 21:33 |
marun | I guess I wasn't seeing that use-case represented in the patch in question | 21:33 |
pritesh | chan1: I am not sure why it says that but if you configure it correctly as per the link, you ca get devstack up and running with neutron. | 21:33 |
marun | ZZelle: Ok, that makes more sense. Is there a way to ensure that the cleanup is done only when used with 'with'? | 21:34 |
pritesh | chan1: you may have to tweak the localrc a bit, here is one posted by a user earlier: https://gist.github.com/amotoki/6520741 | 21:35 |
ZZelle | marun, normally no, we must change __exit__ behavior | 21:35 |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Move final remnants of router processing to router classes https://review.openstack.org/163222 | 21:35 |
*** baoli has quit IRC | 21:36 | |
ZZelle | marun, but it disallows explicit Fixture.cleanUp call | 21:36 |
*** baoli has joined #openstack-neutron | 21:36 | |
*** angela-s has left #openstack-neutron | 21:37 | |
marun | ZZelle: so either/or? | 21:37 |
chan1 | pritesh: thank you sir | 21:37 |
pritesh | np | 21:37 |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Move final remnants of router processing to router classes https://review.openstack.org/163222 | 21:38 |
ZZelle | marun, it seems not possible if we want to support the 3 ways to use fixtures | 21:38 |
russellb | garyk: i started working on the ovn automation via devstack today, still more to go though ... https://github.com/russellb/networking-ovn/commit/171cae4b5930f73d12b98f01b99f54a6c5aec152 | 21:39 |
marun | ZZelle: Honestly, I'm not entirely sure of the value of using 'with' | 21:39 |
marun | ZZelle: do you have examples in subsequent patches? | 21:40 |
ZZelle | marun, ovs monitor func tests perhaps | 21:40 |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Move final remnants of router processing to router classes https://review.openstack.org/163222 | 21:40 |
marun | ZZelle: My question is whether there is a compelling reason to use 'with' | 21:41 |
marun | ZZelle: I'm having trouble seeing examples of where that would be preferable to simply self.useFixture | 21:41 |
marun | seeing -> imagining | 21:41 |
ZZelle | marun, it could typically useful for tests with the pattern without-with-without | 21:42 |
marun | ZZelle: I would vote for YAGNI at this point | 21:42 |
marun | ZZelle: we can add support for 'with' if/when we find a compelling reason for it | 21:43 |
marun | ZZelle: But I'm not inclined to support it just because | 21:43 |
ZZelle | marun, so we implement a buggy fixture? | 21:43 |
marun | ZZelle: buggy how? | 21:43 |
marun | ZZelle: add an error message to __enter__ | 21:44 |
*** leenheer has quit IRC | 21:44 | |
marun | add an error message -> raise an exception | 21:44 |
*** lukasa has joined #openstack-neutron | 21:44 | |
ZZelle | marun, an explicit cleanUp call won't work as expected | 21:44 |
marun | ZZelle: :/ | 21:44 |
marun | ZZelle: can you point me to docs on this issue? | 21:45 |
marun | ZZelle: tbh, it's probably not something worth blocking on | 21:45 |
marun | ZZelle: we can always optimize in the future | 21:45 |
marun | ZZelle: but I would like to understand the mechanics of fixtures that makes this an issue | 21:46 |
*** Marga_ has joined #openstack-neutron | 21:46 | |
ZZelle | marun, there is no issue in fixtures, but in our PortFixture if we remove the cleanup and assume BridgeFixture will do it | 21:47 |
*** seizadi1 has quit IRC | 21:47 | |
chan1 | pritesh: yes! Need to clean up the nova-network part. This work perfectly without --nic parameter: -> nova boot --flavor m1.tiny --image cirros-0.3.3-x86_64 --security-group default --key-name demo-key demo-instance1 | 21:47 |
*** seizadi has joined #openstack-neutron | 21:47 | |
chan1 | and terminated nova-network service | 21:47 |
*** matrohon has quit IRC | 21:47 | |
marun | ZZelle: ah, I get it | 21:47 |
marun | ZZelle: no, I don't | 21:48 |
pritesh | chan1: good, so you need to get neutron part up and running and you should be able to get the network runnign as well. | 21:48 |
marun | ZZelle: I see 2 casees | 21:48 |
marun | cases | 21:48 |
ZZelle | marun, arg | 21:48 |
marun | ZZelle: 1 - with 2 - useFixture | 21:48 |
*** lukasa has quit IRC | 21:48 | |
marun | for useFixture, there is no expectation of local cleanup, so deletion on bridge cleanup is fine | 21:49 |
ZZelle | marun, 3 - use setUp/cleanUp manually | 21:49 |
marun | for with, we want local cleanup | 21:49 |
ZZelle | marun, no | 21:49 |
marun | ZZelle: arguably we shouldn't see that | 21:49 |
*** yamamoto has joined #openstack-neutron | 21:49 | |
marun | setUp/cleanUp explicitly I mean | 21:49 |
ZZelle | marun, yes | 21:49 |
marun | we don't have to support every possible permutation of use | 21:49 |
*** pritesh has quit IRC | 21:49 | |
marun | useFixture is the way fixture should be managed | 21:50 |
marun | ZZelle: so, what are the mechanics of the local cleanup required for with? | 21:50 |
marun | ZZelle: would addCleanup on __enter__ work, for example? | 21:50 |
* marun goes to look at fixtures.Fixture.__enter__ | 21:51 | |
ZZelle | marun, http://bazaar.launchpad.net/~testing-cabal/python-fixtures/master/view/head:/fixtures/fixture.py | 21:51 |
marun | ZZelle: Ah, still in bzr :/ | 21:52 |
marun | http://bazaar.launchpad.net/~testing-cabal/python-fixtures/master/view/head:/fixtures/fixture.py#L131 | 21:52 |
ZZelle | marun, long long time ago, we used it :| | 21:52 |
marun | ZZelle: so did I, for years even. Used to work with a bzr dev | 21:52 |
marun | ZZelle: I liked it, frankly, but the small community was problematic | 21:53 |
marun | ZZelle: in any case, it looks simple enough to override __enter__ | 21:53 |
marun | ZZelle: https://etherpad.openstack.org/p/5ZlQkQYaw7 | 21:53 |
*** yamamoto has quit IRC | 21:54 | |
marun | ZZelle: does that make sense? | 21:54 |
*** dane_leblanc has quit IRC | 21:54 | |
marun | ZZelle: so cleanup would be explicit for contextmanager use, but implicit without it | 21:55 |
*** ociuhandu has joined #openstack-neutron | 21:55 | |
marun | ZZelle: the port would already have a reference to the bridge, and the port name could be retrieved from the port | 21:56 |
*** ChuckC has quit IRC | 21:56 | |
ZZelle | marun, i am trying if we can have tricky case as 1- 2- can be mixed | 21:56 |
ZZelle | i ma tryin to see .. | 21:56 |
*** dkehn has joined #openstack-neutron | 21:57 | |
marun | ZZelle: Explicit cleanup adds some overhead, but doesn't harm the implicit use case | 21:57 |
ZZelle | marun, ie: a fixture used a context can use useFixture in its setUp | 21:57 |
marun | ZZelle: This is just an optimization to avoid deleting ports explicitly | 21:57 |
marun | ZZelle: I don't think fixtures should be smart | 21:58 |
marun | ZZelle: They should know how to achieve an end-state if a certain workflow is followed | 21:58 |
marun | ZZelle: but we don't have to make them magical, because magical == complicated to implement and reason about | 21:58 |
openstackgerrit | Carl Baldwin proposed openstack/python-neutronclient: Add Neutron subnet-create with subnetpool https://review.openstack.org/165910 | 21:58 |
openstackgerrit | Carl Baldwin proposed openstack/python-neutronclient: Allow passing None for subnetpool https://review.openstack.org/168499 | 21:58 |
*** ajmiller_ has joined #openstack-neutron | 22:00 | |
ZZelle | marun, let me update change i integrate your remarks | 22:00 |
amuller | carl_baldwin: I'm reviewing the refactor patch, should take me 20 more mins~ | 22:01 |
*** sambetts has quit IRC | 22:01 | |
marun | ZZelle: sounds good | 22:01 |
*** kbringard has quit IRC | 22:02 | |
*** ajmiller has quit IRC | 22:02 | |
carl_baldwin | amuller, hopefully you're on PS5 and my other botched attempts didn't trip you up. | 22:03 |
carl_baldwin | amuller, thanks for reviewing. | 22:03 |
openstackgerrit | Andrew Boik proposed openstack/neutron: Consider all address scopes in init_l3 https://review.openstack.org/168502 | 22:03 |
*** catohornet has quit IRC | 22:03 | |
*** dboik_ has quit IRC | 22:03 | |
*** annegentle has joined #openstack-neutron | 22:04 | |
amuller | carl_baldwin: indeed I'm on PS5... just ran the functional tests on your patch, completely unrelated that the process_monitor is trying to respawn neutron-keepalived-state-change after the router is deleted and before the test is over | 22:04 |
*** smccully has quit IRC | 22:04 | |
*** sambetts has joined #openstack-neutron | 22:04 | |
*** Swami has quit IRC | 22:04 | |
amuller | (The process monitor is on by default now) | 22:04 |
-openstackstatus- NOTICE: Gerrit is offline for maintenance, ETA 22:30 UTC http://lists.openstack.org/pipermail/openstack-dev/2015-March/059948.html | 22:05 | |
*** ChanServ changes topic to "Gerrit is offline for maintenance, ETA 22:30 UTC http://lists.openstack.org/pipermail/openstack-dev/2015-March/059948.html" | 22:05 | |
amuller | carl_baldwin: I'll have to ask ajo to investigate if that could cause issues | 22:05 |
marun | wow, is gerrit down? | 22:05 |
carl_baldwin | marun, seems to be. I have a vague memory of a maintenance window being announced. | 22:06 |
marun | http://lists.openstack.org/pipermail/openstack-dev/2015-March/059948.html | 22:06 |
marun | oof | 22:06 |
marun | not sure why they wouldn't do this outside of working hours like any normal ops procedure | 22:06 |
amuller | ahh crap | 22:07 |
amuller | this kind of sucks =D | 22:07 |
marun | indeed | 22:07 |
marun | if only I was using gertty ;) | 22:07 |
amuller | ETA is 21 minutes from now | 22:08 |
*** dimsum__ has quit IRC | 22:09 | |
*** dkehn has quit IRC | 22:10 | |
*** banix has quit IRC | 22:11 | |
*** leenheer has joined #openstack-neutron | 22:11 | |
*** leenheer has quit IRC | 22:11 | |
amuller | reduced to reviewing via a piece of paper | 22:12 |
amuller | the humanity | 22:12 |
*** leenheer has joined #openstack-neutron | 22:13 | |
*** aranjan has joined #openstack-neutron | 22:14 | |
amuller | marun: When do you think we could start running the functional tests only with the native ovsdb implementation? | 22:14 |
amuller | ever? | 22:15 |
amuller | marun: Maybe we can make the tests run only with the native implementation but offer an OS ENV variable so the gate could run it both ways? | 22:16 |
marun | amuller: I'm not sure what would stop us, though iirc the rootwrap daemon is actually pretty good. | 22:16 |
marun | amuller: a major limitation is python-openvswitch not being on pypi | 22:17 |
amuller | marun: I'm more talking about running with only one implementation, not specifically the native one | 22:17 |
amuller | just don't like doubling the run cost | 22:17 |
marun | amuller: well, how much cost are we talking about? | 22:18 |
marun | amuller: there's nothing wrong with a local optimization | 22:18 |
marun | amuller: but we can't switch to running one or the other in the gate so long as we need to support both | 22:18 |
amuller | The L3 functional tests take exactly twice when running with both implementations | 22:18 |
amuller | so I edit the code to make it run only with one then remove that modification before pushing | 22:18 |
*** pc_m has quit IRC | 22:18 | |
marun | amuller: I guess I'm confused, how is that enabled? | 22:18 |
amuller | marun: https://github.com/openstack/neutron/blob/master/neutron/tests/functional/agent/linux/base.py#L60 | 22:19 |
*** flcn has quit IRC | 22:19 | |
*** c_soukup has quit IRC | 22:19 | |
marun | amuller: that is pretty stupid | 22:19 |
marun | amuller: I'd merge a patch that made it conditional at least | 22:20 |
*** smccully has joined #openstack-neutron | 22:20 | |
*** ChuckC has joined #openstack-neutron | 22:20 | |
marun | amuller: scenarios = get_scenarios(), which can limit to one or the other as you say via env var | 22:20 |
*** dimsum__ has joined #openstack-neutron | 22:20 | |
amuller | marun: I'll add that to my TO DO list | 22:21 |
amuller | people run tests less if they take too long | 22:21 |
marun | amuller: I don't see much value in running both | 22:21 |
marun | amuller: it's not like a random functional tests is going to do anything interesting | 22:21 |
marun | amuller: that the ovs tests don't already do | 22:22 |
amuller | marun: maybe the right thing to do instead is to move the scenarios to the OVS test case instead of the OVS base class | 22:22 |
amuller | so only the explicit OVS tests run with both implementations | 22:22 |
amuller | but other tests that inherit from the OVS base class only use one | 22:22 |
*** dkehn has joined #openstack-neutron | 22:23 | |
marun | amuller: I'd be in favor of that | 22:24 |
marun | getting extra implicit coverage of ovs usage doesn't seem that valuable to me | 22:25 |
marun | especially given the potential penalty | 22:25 |
*** ChuckC has quit IRC | 22:25 | |
*** ChuckC has joined #openstack-neutron | 22:26 | |
*** thedodd has quit IRC | 22:27 | |
*** dkehn has quit IRC | 22:27 | |
*** ajmiller has joined #openstack-neutron | 22:29 | |
*** ajmiller has quit IRC | 22:29 | |
*** ronis has quit IRC | 22:30 | |
*** ajmiller_ has quit IRC | 22:30 | |
*** achanda_ has joined #openstack-neutron | 22:31 | |
*** jamiem has quit IRC | 22:31 | |
*** ChanServ changes topic to "Discussion of OpenStack Networking || for support join #openstack" | 22:33 | |
amuller | marun: Gerrit's up again | 22:34 |
*** achanda has quit IRC | 22:34 | |
*** achanda_ has quit IRC | 22:36 | |
*** leenheer has quit IRC | 22:37 | |
*** baoli has quit IRC | 22:38 | |
*** chan1 has quit IRC | 22:39 | |
*** markvoelker has quit IRC | 22:40 | |
*** shwetaap has quit IRC | 22:41 | |
*** bquan has quit IRC | 22:42 | |
*** leenheer has joined #openstack-neutron | 22:42 | |
*** baoli has joined #openstack-neutron | 22:43 | |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Transform BaseOVSLinuxTestCase methods in helpers https://review.openstack.org/165866 | 22:43 |
ZZelle | marun, ^^ | 22:44 |
*** aranjan has quit IRC | 22:44 | |
marun | ZZelle: I still don't see the point of 'get' | 22:45 |
marun | ZZelle: I'd rather see the port fixtures accept the necessary arguments to init | 22:45 |
marun | ZZelle: that is a standard and well-documented protocol | 22:45 |
ZZelle | marun, we need different implementations for each bridge type as they have strongly different implementations | 22:47 |
marun | ZZelle: so? | 22:47 |
marun | ZZelle: each port fixture can define it's own init | 22:47 |
marun | ZZelle: we don't have to pretend we care about inheritance and make init somehow conform across implementations | 22:47 |
ZZelle | marun, who chooses the right PortFixture implementation? | 22:48 |
marun | ZZelle: whoever is writing the test | 22:48 |
marun | ZZelle: I'm not seeing any justification for trying to treat port fixtures polymorphically | 22:49 |
marun | ZZelle: what am I missing? | 22:49 |
ZZelle | marun, typically it allows to define a test scenario for checking bridgefixture/portfixture | 22:50 |
marun | ZZelle: example? | 22:50 |
marun | ZZelle: YAGNI - complicating a design for the sake of a 'might be nice' is terrible practice | 22:51 |
*** signed8b_ is now known as signed8bit_ZZZzz | 22:51 | |
ZZelle | marun, it's used by fake machines in follow-up change | 22:52 |
marun | ZZelle: so make the change in the follow-up change | 22:52 |
marun | ZZelle: adding code that doesn't make sense in the context of the current change is something to be avoided if at all possible | 22:53 |
marun | ZZelle: it effectively means the patches can't be considered in isolation | 22:53 |
marun | ZZelle: and what's the point of having multiple patches if that's the case? | 22:53 |
ZZelle | marun, because you require smaller changes!!! | 22:54 |
marun | ZZelle: right | 22:54 |
marun | ZZelle: smaller _and_ cohesive changes | 22:54 |
marun | ZZelle: breaking up something arbitrarily and including stuff that doesn't make sense is not going to make for mergeable code | 22:54 |
ZZelle | so in this change i add 6 times XXXPortFixture, in next change i replace them? | 22:54 |
marun | ZZelle: yes | 22:54 |
marun | ZZelle: we're trying to get away from mix-ins | 22:55 |
marun | ZZelle: fixtures make that possible. | 22:55 |
marun | ZZelle: why go backwards and tie things together with inheritance if we don't need to? | 22:56 |
HenryG | amuller: ping | 22:56 |
amuller | HenryG: yo | 22:56 |
ZZelle | marun, it's not inheritance, it's an interface | 22:56 |
*** anshul has quit IRC | 22:57 | |
ZZelle | marun, how do you ensure a common behavior? with implicit duck typing? | 22:57 |
HenryG | amuller: https://review.openstack.org/#/c/167462/2/neutron/tests/functional/agent/test_l3_agent.py | 22:57 |
HenryG | amuller: lines 512-515 | 22:57 |
*** baoli has quit IRC | 22:57 | |
*** apuimedo has joined #openstack-neutron | 22:57 | |
*** apuimedo has quit IRC | 22:58 | |
marun | ZZelle: introduce the change where it is needed | 22:58 |
ZZelle | marun, no | 22:58 |
marun | ZZelle: don't complicate a patch with 'I'll need it next patch' | 22:58 |
amuller | HenryG: Yes? | 22:59 |
ZZelle | marun, so i merge it with next change | 22:59 |
marun | ZZelle: yes, please | 22:59 |
HenryG | amuller: 1 sec | 22:59 |
*** baoli has joined #openstack-neutron | 23:00 | |
ZZelle | marun, you will get a change twice bigger | 23:00 |
*** dane_leblanc has joined #openstack-neutron | 23:00 | |
marun | ZZelle: really? For the sake of a get() method? | 23:00 |
HenryG | amuller: nevermind, I can't count. sorry | 23:01 |
*** salv-orl_ has quit IRC | 23:01 | |
marun | ZZelle: the point of breaking patches up is _not_ due to some strictly mechanical judgement | 23:01 |
marun | ZZelle: It's to make sure that the reviewer can follow along and provide effective oversight | 23:01 |
*** tidwellr has quit IRC | 23:01 | |
amuller | HenryG: np I'm glad to know you're looking at that stuff :) | 23:01 |
marun | ZZelle: you may have the entire patch series in your head, but I don't review past the first change and it has to make sense to me | 23:02 |
ZZelle | marun, each change prepares the next change, the aim is to be efficient globally not locally | 23:02 |
marun | ZZelle: well, if you can find a reviewer that works that way, ask them to review instead of me | 23:02 |
marun | ZZelle: I merge stuff I understand and that is cohesive enough to stand on its own | 23:02 |
marun | ZZelle: If I get far enough along the learning curve that the point is obvious, there can be exceptions | 23:03 |
*** salv-orlando has joined #openstack-neutron | 23:03 | |
marun | ZZelle: but the point of review isn't just about merging code - it's making sure that someone else can understand what you're trying to accomplish | 23:03 |
marun | ZZelle: and no reviewer has as much time as the author to climb the curve. | 23:04 |
marun | ZZelle: so doing your best to provide a cohesive set of patches that make sense in isolation is to your benefit | 23:04 |
ZZelle | marun, get() is an helper for hidding portfixture behavior for each bridge type, i don't see how it's unrelated | 23:05 |
marun | ZZelle: it adds abstraction to no end | 23:06 |
marun | ZZelle: Nothing in the change requires that abstraction | 23:06 |
amuller | carl_baldwin: done | 23:08 |
*** carl_baldwin has quit IRC | 23:08 | |
*** rpothier has quit IRC | 23:10 | |
*** chem has quit IRC | 23:11 | |
marun | ZZelle: I'm afraid I'm done for the day. Will pick up next week if amuller, armax and HenryG don't beat me to it. | 23:12 |
*** marun has quit IRC | 23:12 | |
*** seizadi1 has joined #openstack-neutron | 23:12 | |
*** seizadi has quit IRC | 23:13 | |
*** baoli has quit IRC | 23:13 | |
*** tflynn has quit IRC | 23:13 | |
ZZelle | marun, good week end | 23:16 |
*** annegentle has quit IRC | 23:19 | |
*** flcn has joined #openstack-neutron | 23:20 | |
*** padkrish has quit IRC | 23:21 | |
*** _cjones_ has quit IRC | 23:21 | |
*** padkrish has joined #openstack-neutron | 23:21 | |
*** nlahouti has quit IRC | 23:21 | |
*** zhhuabj has quit IRC | 23:22 | |
*** shaohe_feng has joined #openstack-neutron | 23:23 | |
*** ajmiller has joined #openstack-neutron | 23:23 | |
*** jckasper has quit IRC | 23:24 | |
*** flcn has quit IRC | 23:24 | |
*** padkrish has quit IRC | 23:26 | |
*** xgerman has quit IRC | 23:28 | |
*** aranjan has joined #openstack-neutron | 23:31 | |
*** Aish has quit IRC | 23:32 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: Stop running L3 functional tests with both OVSDB interfaces https://review.openstack.org/168513 | 23:32 |
openstackgerrit | Assaf Muller proposed openstack/neutron: Stop running L3 functional tests with both OVSDB interfaces https://review.openstack.org/168513 | 23:33 |
*** dane_leblanc has quit IRC | 23:34 | |
*** nlahouti has joined #openstack-neutron | 23:36 | |
*** padkrish has joined #openstack-neutron | 23:37 | |
openstackgerrit | Michael Smith proposed openstack/neutron: manual add/remove router for dvr_snat agent https://review.openstack.org/122024 | 23:39 |
*** markvoelker has joined #openstack-neutron | 23:41 | |
*** ajmiller has quit IRC | 23:41 | |
*** padkrish has quit IRC | 23:42 | |
openstackgerrit | Michael Smith proposed openstack/neutron: L3 Agent support of manual dvr snat add/remove https://review.openstack.org/123815 | 23:43 |
*** vivek-ebay has joined #openstack-neutron | 23:46 | |
*** markvoelker has quit IRC | 23:46 | |
*** julien_ZTE has quit IRC | 23:47 | |
*** shaohe_feng has quit IRC | 23:48 | |
*** padkrish has joined #openstack-neutron | 23:48 | |
*** sbfox has quit IRC | 23:48 | |
*** signed8bit_ZZZzz is now known as signed8b_ | 23:49 | |
*** julien_ZTE has joined #openstack-neutron | 23:50 | |
*** amuller has quit IRC | 23:50 | |
*** flcn has joined #openstack-neutron | 23:50 | |
*** ociuhandu has quit IRC | 23:52 | |
*** julien_ZTE has quit IRC | 23:52 | |
*** dannywilson has quit IRC | 23:53 | |
*** flcn has quit IRC | 23:55 | |
*** yeungp has quit IRC | 23:56 | |
*** seizadi1 has quit IRC | 23:58 | |
*** dimsum__ has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!