*** masaki has joined #openstack-neutron | 00:02 | |
*** crushil has quit IRC | 00:04 | |
*** SumitNaiksatam has quit IRC | 00:05 | |
*** gouthamr has quit IRC | 00:07 | |
*** yamamoto has quit IRC | 00:11 | |
openstackgerrit | LIU Yulong proposed openstack/neutron-specs master: Layer 3 IPs rate limit https://review.openstack.org/374506 | 00:21 |
---|---|---|
*** mlavalle has quit IRC | 00:22 | |
*** thorst_afk has joined #openstack-neutron | 00:26 | |
*** thorst_afk has quit IRC | 00:30 | |
openstackgerrit | LIU Yulong proposed openstack/neutron-specs master: Layer 3 IPs rate limit https://review.openstack.org/374506 | 00:31 |
*** yamamoto has joined #openstack-neutron | 00:34 | |
*** markstur has quit IRC | 00:34 | |
*** psahoo has joined #openstack-neutron | 00:35 | |
*** fnaval has quit IRC | 00:35 | |
*** svenkat has joined #openstack-neutron | 00:36 | |
*** yamamoto has quit IRC | 00:36 | |
*** yamamoto has joined #openstack-neutron | 00:36 | |
*** gouthamr has joined #openstack-neutron | 00:37 | |
mriedem | kevinbenton: the bulk query floating ip thing didn't work i don't think | 00:41 |
*** yamamoto has quit IRC | 00:41 | |
mriedem | otherwise cfriesen said the network-changed scoped to the port series is helping - he has an email in the dev ML abou it | 00:41 |
mriedem | i need to clean up the nova side a bit though | 00:41 |
*** catintheroof has joined #openstack-neutron | 00:46 | |
*** limao has joined #openstack-neutron | 00:52 | |
*** mickeys has quit IRC | 00:52 | |
*** claudiub has quit IRC | 00:54 | |
*** svenkat has quit IRC | 00:57 | |
*** markstur has joined #openstack-neutron | 00:57 | |
*** gouthamr has quit IRC | 01:06 | |
*** catintheroof has quit IRC | 01:06 | |
*** ijw has joined #openstack-neutron | 01:15 | |
*** gianpietro has quit IRC | 01:18 | |
*** ijw_ has quit IRC | 01:19 | |
*** vhosakot has joined #openstack-neutron | 01:23 | |
*** s3wong has quit IRC | 01:28 | |
*** vhosakot has quit IRC | 01:32 | |
*** yamamoto has joined #openstack-neutron | 01:39 | |
*** lnicolas has quit IRC | 01:45 | |
*** yamamoto has quit IRC | 01:45 | |
*** yamamoto has joined #openstack-neutron | 01:46 | |
*** yamamoto has quit IRC | 01:46 | |
*** thorst_afk has joined #openstack-neutron | 01:49 | |
*** lnicolas has joined #openstack-neutron | 01:50 | |
*** crushil has joined #openstack-neutron | 01:52 | |
*** mickeys has joined #openstack-neutron | 01:53 | |
*** crushil has quit IRC | 01:53 | |
*** thorst_afk has quit IRC | 01:54 | |
*** yamamoto has joined #openstack-neutron | 01:57 | |
*** crushil has joined #openstack-neutron | 01:57 | |
*** mickeys has quit IRC | 01:57 | |
*** yamamoto has quit IRC | 01:59 | |
*** gianpietro has joined #openstack-neutron | 02:02 | |
*** bobmel has quit IRC | 02:04 | |
*** reedip has joined #openstack-neutron | 02:05 | |
*** gianpietro has quit IRC | 02:07 | |
*** yamamoto has joined #openstack-neutron | 02:09 | |
*** gianpietro has joined #openstack-neutron | 02:13 | |
*** yamamoto has quit IRC | 02:13 | |
*** yamamoto has joined #openstack-neutron | 02:18 | |
*** yamamoto has quit IRC | 02:18 | |
*** thorst_afk has joined #openstack-neutron | 02:28 | |
*** mickeys has joined #openstack-neutron | 02:30 | |
*** gianpietro has quit IRC | 02:32 | |
*** yamahata has quit IRC | 02:32 | |
*** thorst_afk has quit IRC | 02:33 | |
*** yamamoto has joined #openstack-neutron | 02:36 | |
*** mriedem has quit IRC | 02:37 | |
*** mriedem has joined #openstack-neutron | 02:37 | |
*** gianpietro has joined #openstack-neutron | 02:39 | |
*** iyamahat has quit IRC | 02:41 | |
*** crushil has quit IRC | 02:42 | |
*** yamamoto has quit IRC | 02:44 | |
*** limao has quit IRC | 02:46 | |
*** limao_ has joined #openstack-neutron | 02:46 | |
*** vks1 has joined #openstack-neutron | 02:46 | |
*** vhosakot has joined #openstack-neutron | 02:51 | |
*** crushil has joined #openstack-neutron | 02:51 | |
*** crushil has quit IRC | 02:55 | |
*** crushil has joined #openstack-neutron | 02:55 | |
*** masaki has quit IRC | 03:04 | |
*** gianpietro has quit IRC | 03:05 | |
*** SumitNaiksatam has joined #openstack-neutron | 03:08 | |
*** mriedem has quit IRC | 03:17 | |
*** david-lyle has quit IRC | 03:20 | |
*** gianpietro has joined #openstack-neutron | 03:28 | |
*** thorst_afk has joined #openstack-neutron | 03:29 | |
*** gianpietro has quit IRC | 03:29 | |
*** gianpietro has joined #openstack-neutron | 03:30 | |
*** vks1 has quit IRC | 03:30 | |
*** david-lyle has joined #openstack-neutron | 03:31 | |
*** david-lyle has quit IRC | 03:40 | |
*** gianpietro has quit IRC | 03:42 | |
*** mickeys has quit IRC | 03:47 | |
*** thorst_afk has quit IRC | 03:48 | |
*** gianpietro has joined #openstack-neutron | 03:52 | |
*** david-lyle has joined #openstack-neutron | 03:56 | |
*** vhosakot has quit IRC | 04:02 | |
*** gianpietro has quit IRC | 04:04 | |
*** limao_ has quit IRC | 04:04 | |
*** gianpietro has joined #openstack-neutron | 04:06 | |
*** mickeys has joined #openstack-neutron | 04:06 | |
*** armax has quit IRC | 04:06 | |
*** gouthamr has joined #openstack-neutron | 04:10 | |
*** gkadam_ has joined #openstack-neutron | 04:13 | |
*** tbachman has quit IRC | 04:20 | |
openstackgerrit | Merged openstack/neutron master: Fix functional test for ovsdbapp 0.4.0 https://review.openstack.org/466430 | 04:20 |
*** yamamoto has joined #openstack-neutron | 04:24 | |
*** yamamoto has quit IRC | 04:24 | |
*** yamamoto has joined #openstack-neutron | 04:25 | |
*** gianpietro has quit IRC | 04:25 | |
*** gcheresh has joined #openstack-neutron | 04:28 | |
*** yamamoto has quit IRC | 04:30 | |
*** gianpietro has joined #openstack-neutron | 04:31 | |
*** gouthamr has quit IRC | 04:34 | |
*** yamamoto has joined #openstack-neutron | 04:34 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/networking-sfc master: Updated from global requirements https://review.openstack.org/464426 | 04:38 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron master: Updated from global requirements https://review.openstack.org/465574 | 04:39 |
*** gianpietro has quit IRC | 04:41 | |
*** ihrachys has quit IRC | 04:42 | |
*** brault has joined #openstack-neutron | 04:43 | |
*** ramishra has joined #openstack-neutron | 04:43 | |
*** gcheresh has quit IRC | 04:47 | |
*** brault has quit IRC | 04:47 | |
*** gcheresh has joined #openstack-neutron | 04:58 | |
*** crushil has quit IRC | 04:58 | |
*** crushil has joined #openstack-neutron | 05:02 | |
*** aojea has joined #openstack-neutron | 05:08 | |
*** XueFeng has joined #openstack-neutron | 05:08 | |
*** aojea has quit IRC | 05:12 | |
*** yamamoto has quit IRC | 05:13 | |
*** limao has joined #openstack-neutron | 05:13 | |
*** mickeys has quit IRC | 05:14 | |
*** yamahata has joined #openstack-neutron | 05:15 | |
*** crushil has quit IRC | 05:22 | |
*** crushil has joined #openstack-neutron | 05:22 | |
*** gcheresh has quit IRC | 05:23 | |
*** markvoelker has quit IRC | 05:32 | |
*** thorst_afk has joined #openstack-neutron | 05:45 | |
*** thorst_afk has quit IRC | 05:51 | |
*** gcheresh has joined #openstack-neutron | 05:56 | |
*** mickeys has joined #openstack-neutron | 06:06 | |
*** Ming has joined #openstack-neutron | 06:08 | |
*** Brenda has quit IRC | 06:09 | |
*** markstur has quit IRC | 06:12 | |
*** yamamoto has joined #openstack-neutron | 06:14 | |
*** crushil has quit IRC | 06:16 | |
*** yamamoto has quit IRC | 06:22 | |
*** gcheresh has quit IRC | 06:23 | |
*** yamamoto has joined #openstack-neutron | 06:23 | |
*** iranzo has joined #openstack-neutron | 06:28 | |
*** iranzo has quit IRC | 06:28 | |
*** iranzo has joined #openstack-neutron | 06:28 | |
*** reedip has quit IRC | 06:44 | |
*** thorst_afk has joined #openstack-neutron | 06:46 | |
*** thorst_afk has quit IRC | 06:51 | |
*** aojea has joined #openstack-neutron | 06:56 | |
*** reedip has joined #openstack-neutron | 06:57 | |
*** reedip has quit IRC | 07:17 | |
*** rcernin has quit IRC | 07:17 | |
*** aojea has quit IRC | 07:20 | |
*** aojea has joined #openstack-neutron | 07:28 | |
*** reedip has joined #openstack-neutron | 07:29 | |
*** salv-orlando has joined #openstack-neutron | 07:30 | |
*** slaweq has joined #openstack-neutron | 07:32 | |
*** aojea has quit IRC | 07:32 | |
*** markvoelker has joined #openstack-neutron | 07:33 | |
*** salv-orlando has quit IRC | 07:39 | |
*** markstur has joined #openstack-neutron | 07:47 | |
*** thorst_afk has joined #openstack-neutron | 07:47 | |
*** markstur has quit IRC | 07:51 | |
*** thorst_afk has quit IRC | 07:51 | |
*** reedip has quit IRC | 07:57 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-neutron | 08:00 | |
*** markvoelker has quit IRC | 08:07 | |
*** mickeys has quit IRC | 08:08 | |
*** reedip has joined #openstack-neutron | 08:18 | |
*** limao has quit IRC | 08:19 | |
*** slaweq has quit IRC | 08:19 | |
*** slaweq has joined #openstack-neutron | 08:20 | |
*** iranzo has quit IRC | 08:21 | |
*** slaweq has quit IRC | 08:21 | |
*** aojea has joined #openstack-neutron | 08:29 | |
*** thorst_afk has joined #openstack-neutron | 08:33 | |
*** aojea has quit IRC | 08:34 | |
*** slaweq has joined #openstack-neutron | 08:48 | |
*** Ming is now known as Brenda | 08:48 | |
*** Brenda is now known as Brenda_Tian | 08:48 | |
*** thorst_afk has quit IRC | 08:53 | |
*** slaweq__ has joined #openstack-neutron | 09:00 | |
*** slaweq has quit IRC | 09:03 | |
*** markvoelker has joined #openstack-neutron | 09:04 | |
openstackgerrit | Merged openstack/neutron master: service: add callback AFTER_SPAWN https://review.openstack.org/462536 | 09:09 |
*** yamahata has quit IRC | 09:09 | |
*** mickeys has joined #openstack-neutron | 09:09 | |
*** mickeys has quit IRC | 09:13 | |
*** ataraday_ has joined #openstack-neutron | 09:13 | |
*** reedip has quit IRC | 09:21 | |
*** psahoo has quit IRC | 09:29 | |
*** markvoelker has quit IRC | 09:37 | |
*** aojea has joined #openstack-neutron | 09:40 | |
*** aojea has quit IRC | 09:40 | |
*** thorst_afk has joined #openstack-neutron | 09:50 | |
*** thorst_afk has quit IRC | 09:54 | |
*** psahoo has joined #openstack-neutron | 10:07 | |
*** yamamoto has quit IRC | 10:08 | |
*** yamamoto has joined #openstack-neutron | 10:09 | |
*** mickeys_ has joined #openstack-neutron | 10:10 | |
*** tbachman has joined #openstack-neutron | 10:14 | |
*** mickeys_ has quit IRC | 10:16 | |
*** psahoo has quit IRC | 10:18 | |
*** iranzo has joined #openstack-neutron | 10:23 | |
*** slaweq__ has quit IRC | 10:32 | |
*** slaweq has joined #openstack-neutron | 10:33 | |
*** markvoelker has joined #openstack-neutron | 10:34 | |
*** slaweq has quit IRC | 10:37 | |
*** claudiub has joined #openstack-neutron | 10:46 | |
*** sdatko has quit IRC | 10:51 | |
*** redondo-mk has joined #openstack-neutron | 11:02 | |
*** Brenda_Tian has quit IRC | 11:07 | |
*** Brenda_Tian has joined #openstack-neutron | 11:08 | |
*** markvoelker has quit IRC | 11:08 | |
*** mickeys_ has joined #openstack-neutron | 11:12 | |
*** mickeys_ has quit IRC | 11:16 | |
redondo-mk | Hi! Maybe someone can save me some time. policy.json with "get_security_groups": "rule:regular_user", "get_security_group": "rule:regular_user" => why this doesn't enable regular user to list all security groups in all projects ("openstack security group list --all-projects") ? | 11:17 |
*** gkadam_ has quit IRC | 11:19 | |
*** aojea has joined #openstack-neutron | 11:20 | |
*** gianpietro has joined #openstack-neutron | 11:24 | |
*** reedip_ has quit IRC | 11:25 | |
*** psahoo has joined #openstack-neutron | 11:31 | |
*** reedip_ has joined #openstack-neutron | 11:39 | |
*** tbachman has quit IRC | 11:41 | |
*** aojea has quit IRC | 11:46 | |
*** aojea has joined #openstack-neutron | 11:46 | |
*** aojea has quit IRC | 11:51 | |
*** thorst_afk has joined #openstack-neutron | 11:52 | |
*** gianpietro has quit IRC | 11:56 | |
*** thorst_afk has quit IRC | 11:56 | |
*** nicolasbock has joined #openstack-neutron | 11:57 | |
*** slaweq has joined #openstack-neutron | 12:12 | |
*** iranzo has quit IRC | 12:12 | |
*** slaweq has quit IRC | 12:17 | |
*** aojea has joined #openstack-neutron | 12:55 | |
*** psahoo has quit IRC | 12:58 | |
*** gcheresh has joined #openstack-neutron | 13:04 | |
*** markvoelker has joined #openstack-neutron | 13:05 | |
*** TTris has left #openstack-neutron | 13:10 | |
*** amuller[m] has joined #openstack-neutron | 13:12 | |
*** mickeys_ has joined #openstack-neutron | 13:14 | |
*** amuller[m] has left #openstack-neutron | 13:14 | |
*** aojea has quit IRC | 13:15 | |
*** mickeys_ has quit IRC | 13:18 | |
*** aojea has joined #openstack-neutron | 13:20 | |
*** sdatko has joined #openstack-neutron | 13:33 | |
*** markvoelker has quit IRC | 13:39 | |
*** reedip has joined #openstack-neutron | 13:48 | |
*** slaweq has joined #openstack-neutron | 13:48 | |
*** thorst_afk has joined #openstack-neutron | 13:53 | |
*** slaweq has quit IRC | 13:54 | |
*** gianpietro has joined #openstack-neutron | 13:57 | |
*** thorst_afk has quit IRC | 13:58 | |
*** gianpietro has quit IRC | 13:59 | |
*** gianpietro has joined #openstack-neutron | 13:59 | |
*** thorst_afk has joined #openstack-neutron | 14:06 | |
*** reedip has quit IRC | 14:07 | |
*** thorst_afk has quit IRC | 14:10 | |
*** donghao has joined #openstack-neutron | 14:10 | |
*** gcheresh has quit IRC | 14:11 | |
*** mickeys_ has joined #openstack-neutron | 14:14 | |
*** mickeys_ has quit IRC | 14:18 | |
*** reedip has joined #openstack-neutron | 14:19 | |
*** kerpan has quit IRC | 14:22 | |
*** crushil has joined #openstack-neutron | 14:25 | |
*** crushil has quit IRC | 14:32 | |
*** markvoelker has joined #openstack-neutron | 14:35 | |
*** aojea has quit IRC | 14:43 | |
*** baoli has joined #openstack-neutron | 14:45 | |
*** baoli has quit IRC | 14:52 | |
*** aojea has joined #openstack-neutron | 14:53 | |
*** brault has joined #openstack-neutron | 14:56 | |
*** brault has quit IRC | 15:01 | |
*** markvoelker has quit IRC | 15:09 | |
*** slaweq has joined #openstack-neutron | 15:09 | |
*** thorst_afk has joined #openstack-neutron | 15:11 | |
*** thorst_afk has quit IRC | 15:16 | |
*** kerpan has joined #openstack-neutron | 15:19 | |
*** vhosakot has joined #openstack-neutron | 15:20 | |
*** vhosakot has quit IRC | 15:21 | |
*** armax has joined #openstack-neutron | 15:28 | |
openstackgerrit | LIU Yulong proposed openstack/neutron-specs master: Layer 3 IPs rate limit https://review.openstack.org/374506 | 15:31 |
*** reedip has quit IRC | 15:34 | |
*** reedip has joined #openstack-neutron | 15:36 | |
*** slaweq has quit IRC | 15:50 | |
*** slaweq has joined #openstack-neutron | 15:51 | |
*** armax has quit IRC | 15:52 | |
*** slaweq has quit IRC | 15:55 | |
*** dklyle has joined #openstack-neutron | 16:00 | |
*** david-lyle has quit IRC | 16:00 | |
*** sdatko_ has joined #openstack-neutron | 16:00 | |
*** sdatko has quit IRC | 16:03 | |
*** baoli has joined #openstack-neutron | 16:10 | |
*** thorst_afk has joined #openstack-neutron | 16:12 | |
*** baoli has quit IRC | 16:15 | |
*** ihrachys has joined #openstack-neutron | 16:16 | |
*** mickeys_ has joined #openstack-neutron | 16:20 | |
*** dklyle has quit IRC | 16:23 | |
*** reedip__ has joined #openstack-neutron | 16:27 | |
*** reedip has quit IRC | 16:27 | |
*** slaweq has joined #openstack-neutron | 16:28 | |
*** thorst_afk has quit IRC | 16:31 | |
*** brault has joined #openstack-neutron | 16:32 | |
*** slaweq has quit IRC | 16:33 | |
*** david-lyle has joined #openstack-neutron | 16:39 | |
*** slaweq has joined #openstack-neutron | 16:46 | |
openstackgerrit | Isaku Yamahata proposed openstack/neutron master: use neutron-lib.callback.events.AFTER_SPAWN https://review.openstack.org/466486 | 16:55 |
*** yamahata has joined #openstack-neutron | 16:56 | |
*** reedip__ has quit IRC | 17:00 | |
*** Sukhdev has joined #openstack-neutron | 17:05 | |
*** ociuhandu has joined #openstack-neutron | 17:13 | |
*** salv-orlando has joined #openstack-neutron | 17:14 | |
*** mickeys_ has quit IRC | 17:24 | |
*** ociuhandu has quit IRC | 17:32 | |
openstackgerrit | Isaku Yamahata proposed openstack/networking-odl master: journal: run timer only if main process https://review.openstack.org/466494 | 17:41 |
*** fnaval has joined #openstack-neutron | 17:43 | |
*** donghao has quit IRC | 17:50 | |
*** tonygunk has quit IRC | 17:52 | |
*** salv-orlando has quit IRC | 18:00 | |
*** markvoelker has joined #openstack-neutron | 18:06 | |
*** slaweq has quit IRC | 18:09 | |
*** slaweq has joined #openstack-neutron | 18:10 | |
*** markvoelker has quit IRC | 18:11 | |
*** yamahata has quit IRC | 18:11 | |
*** thorst_afk has joined #openstack-neutron | 18:11 | |
*** thorst_afk has quit IRC | 18:11 | |
*** yamahata has joined #openstack-neutron | 18:13 | |
*** jhershbe has joined #openstack-neutron | 18:14 | |
*** haplo37 has quit IRC | 18:15 | |
*** ihrachys has quit IRC | 18:18 | |
*** slaweq has quit IRC | 18:18 | |
*** jhershbe has quit IRC | 18:20 | |
*** haplo37 has joined #openstack-neutron | 18:23 | |
*** mickeys_ has joined #openstack-neutron | 18:25 | |
*** mickeys_ has quit IRC | 18:29 | |
*** gcheresh has joined #openstack-neutron | 18:42 | |
*** mickeys_ has joined #openstack-neutron | 18:48 | |
*** donghao has joined #openstack-neutron | 18:50 | |
*** donghao has quit IRC | 18:56 | |
*** gcheresh has quit IRC | 19:04 | |
*** thorst_afk has joined #openstack-neutron | 19:12 | |
*** nicolasbock has quit IRC | 19:14 | |
*** fnaval has quit IRC | 19:17 | |
*** thorst_afk has quit IRC | 19:17 | |
*** fnaval has joined #openstack-neutron | 19:17 | |
*** slaweq has joined #openstack-neutron | 19:20 | |
*** fnaval has quit IRC | 19:21 | |
*** tbachman has joined #openstack-neutron | 19:21 | |
*** Sukhdev has quit IRC | 19:26 | |
*** tbachman_ has joined #openstack-neutron | 19:32 | |
*** tbachman has quit IRC | 19:32 | |
*** tbachman_ has quit IRC | 19:32 | |
*** tbachman has joined #openstack-neutron | 19:34 | |
*** tbachman has quit IRC | 19:47 | |
*** gcheresh has joined #openstack-neutron | 19:53 | |
*** mickeys_ has quit IRC | 19:56 | |
*** salv-orlando has joined #openstack-neutron | 19:57 | |
*** itzikb has joined #openstack-neutron | 20:03 | |
*** tonygunk has joined #openstack-neutron | 20:05 | |
*** markvoelker has joined #openstack-neutron | 20:07 | |
*** thorst_afk has joined #openstack-neutron | 20:13 | |
*** thorst_afk has quit IRC | 20:18 | |
*** mickeys_ has joined #openstack-neutron | 20:22 | |
*** fnaval has joined #openstack-neutron | 20:23 | |
*** markvoelker has quit IRC | 20:41 | |
*** gcheresh has quit IRC | 20:47 | |
*** donghao has joined #openstack-neutron | 20:54 | |
*** crushil has joined #openstack-neutron | 20:54 | |
*** donghao has quit IRC | 20:58 | |
*** fnaval_ has joined #openstack-neutron | 21:03 | |
*** fnaval has quit IRC | 21:07 | |
*** fnaval_ has quit IRC | 21:09 | |
*** fnaval has joined #openstack-neutron | 21:10 | |
*** thorst_afk has joined #openstack-neutron | 21:14 | |
*** iranzo has joined #openstack-neutron | 21:17 | |
*** mickeys_ has quit IRC | 21:23 | |
*** iranzo has quit IRC | 21:26 | |
*** thorst_afk has quit IRC | 21:33 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Add support for ingress bandwidth limit rules in ovs agent https://review.openstack.org/457816 | 21:36 |
*** abalutoiu_ has joined #openstack-neutron | 21:37 | |
*** markvoelker has joined #openstack-neutron | 21:38 | |
openstackgerrit | Merged openstack/networking-sfc master: Create Port_Mapping Support with OVS Driver and Agent https://review.openstack.org/458303 | 21:40 |
*** itzikb has quit IRC | 21:40 | |
*** abalutoiu has quit IRC | 21:40 | |
*** mickeys has joined #openstack-neutron | 21:46 | |
*** yamamoto has quit IRC | 22:00 | |
*** yamamoto has joined #openstack-neutron | 22:00 | |
*** gianpietro has quit IRC | 22:00 | |
*** markvoelker has quit IRC | 22:12 | |
*** mlavalle has joined #openstack-neutron | 22:13 | |
mlavalle | slaweq: ping | 22:14 |
slaweq | mlavalle: hellp | 22:14 |
slaweq | *hello | 22:14 |
mlavalle | regarding my comment to https://review.openstack.org/#/c/457816/7/neutron/agent/common/ovs_lib.py@727 | 22:14 |
slaweq | yes? | 22:15 |
mlavalle | my question is the following: assume you search ingress max-rate in QoS directly, L720-723 | 22:16 |
mlavalle | what I read in the doc is that for QoS, max-rate default is 100Mbps | 22:17 |
mlavalle | I am talking about QoS, not the queue | 22:17 |
slaweq | ok, I see | 22:18 |
slaweq | but when I was checking it on my dev host | 22:18 |
slaweq | if it wasn't set I didn't have it in "other_config" - it was empty | 22:18 |
slaweq | so IMHO it looks like this default value 100Mbps is not returned there | 22:18 |
slaweq | but I will check it once again tomorrow | 22:18 |
mlavalle | My comment wasn't saying your code is wrong | 22:19 |
slaweq | ok, no problem at all :) | 22:19 |
mlavalle | My comment was that based on what I read in the doc, I was questioning you | 22:19 |
mlavalle | the doc is the offical source, right | 22:20 |
mlavalle | ? | 22:20 |
slaweq | ok, I now see what was exactly Your question and I will try to check it once again | 22:20 |
slaweq | if I will not find any ansear I will also try to ask on ovs ML about it :) | 22:21 |
mlavalle | and a valid answer might be "based on my testing this is what I am finding" | 22:21 |
slaweq | will it be fine for You? :) | 22:21 |
mlavalle | slaweq: I was just trying to do a thorough review | 22:21 |
slaweq | that's great that You are asking about such things :) it's better to fix such things now than later report new bug about it :) | 22:22 |
slaweq | I have also different question to You, about comments in https://review.openstack.org/#/c/457816/7/neutron/agent/l2/extensions/qos.py | 22:22 |
slaweq | can You check my answer and tell me what You think about it? | 22:23 |
mlavalle | slaweq: hang on | 22:25 |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron-specs master: Add more detailed info about supported QoS rule types https://review.openstack.org/466500 | 22:26 |
*** mriedem has joined #openstack-neutron | 22:30 | |
*** thorst_afk has joined #openstack-neutron | 22:30 | |
mlavalle | slaweq: so look at https://review.openstack.org/#/c/457816/7/neutron/plugins/ml2/drivers/openvswitch/agent/extension_drivers/qos_driver.py@63 | 22:33 |
slaweq | I saw it | 22:34 |
mlavalle | why not refactor delete_bandwidth_limit into two methods | 22:34 |
mlavalle | delete_bandwidth_limit and delete_bandwidth_limit_ingress | 22:35 |
slaweq | IMHO it's not good idea because it will break some concept of naming methods | 22:35 |
slaweq | "ingress" or "egress" is only value of parameter in rule | 22:35 |
*** thorst_afk has quit IRC | 22:35 | |
slaweq | and names of methods are named with common pattern "action_rule_type" | 22:35 |
slaweq | and here for this one rule_type it would be "action_rule_type_<parameter>" | 22:36 |
mlavalle | but that conevntion is preventing you from expanding the api cleanly, isn't it? | 22:37 |
mlavalle | you now have to add an argument to the call and that breaks current api, doesn't it? | 22:38 |
*** crushil has quit IRC | 22:38 | |
slaweq | you're right but I'm not sure which solution is better :) | 22:38 |
mlavalle | it's always a trade off :-) | 22:39 |
slaweq | :) | 22:39 |
slaweq | maybe it would be good idea to ask also ihrahys what he things about it | 22:40 |
mlavalle | sure | 22:40 |
mlavalle | in any review that I make, please don't take my comments as a judgement. I see my role as reviewer as collaborating with you. I am only giving suggestions | 22:41 |
mlavalle | in this case, what I see is that to accomodate the ingress delete, because we have a naming convention, we break an api not only in one methos but in two. the second one is delete_dscp_marking | 22:43 |
mlavalle | so it might be time to re evaluate the naming convention | 22:44 |
mlavalle | but that is my opinion | 22:44 |
slaweq | ok, I will ask also other reviewers about the best solution and will apply it in patch :) | 22:44 |
slaweq | thanks a lot for Your help and ideas | 22:44 |
mlavalle | :-) | 22:44 |
mlavalle | we could discuss also during the QoS meeting | 22:44 |
slaweq | have a nice day (or night like me :)) | 22:45 |
mlavalle | it's evening, I am in Texas | 22:45 |
slaweq | yes, we can talk about it with ralonsoh and others on meeting | 22:45 |
mlavalle | enjoy the rest of your weekend | 22:45 |
slaweq | for me it's middle of night (I'm in Poland) | 22:45 |
mlavalle | and thanks for taking my opinions into consideration and taking the time to talk to me in the middle of the night | 22:46 |
mlavalle | :-) | 22:46 |
slaweq | :) | 22:46 |
mlavalle | BTW, regarding the other issue about OVS, it may be a good idea to check in the OVS ML | 22:46 |
slaweq | thx once again and good night | 22:47 |
*** abregman has joined #openstack-neutron | 22:47 | |
*** slaweq has quit IRC | 23:04 | |
*** slaweq has joined #openstack-neutron | 23:04 | |
*** markvoelker has joined #openstack-neutron | 23:09 | |
*** slaweq has quit IRC | 23:09 | |
*** markvoelker has quit IRC | 23:42 | |
*** thorst_afk has joined #openstack-neutron | 23:46 | |
*** thorst_afk has quit IRC | 23:46 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!