*** hshan has joined #openstack-dragonflow | 02:06 | |
*** yamamoto has joined #openstack-dragonflow | 03:09 | |
*** yamamoto has quit IRC | 03:20 | |
*** yamamoto_ has joined #openstack-dragonflow | 04:08 | |
*** gongysh has joined #openstack-dragonflow | 05:00 | |
*** gongysh has quit IRC | 06:04 | |
openstackgerrit | Omer Anson proposed openstack/dragonflow: Move port creation/deletion testing to its own class https://review.openstack.org/275020 | 06:08 |
---|---|---|
openstackgerrit | Eran Gampel proposed openstack/dragonflow: Add support for DF built-in pub/sub https://review.openstack.org/263322 | 06:20 |
openstackgerrit | Omer Anson proposed openstack/dragonflow: Move subnet creation/deletion to external class https://review.openstack.org/275033 | 06:47 |
*** gampel has joined #openstack-dragonflow | 06:59 | |
gsagie | DuanKebo: here? | 07:02 |
openstackgerrit | Merged openstack/dragonflow: Fix ValueError in allocate_unique_key in Zookeeper https://review.openstack.org/273855 | 07:06 |
DuanKebo | yes | 07:11 |
DuanKebo | Hi Eran? | 07:12 |
gsagie | its Gal :) | 07:22 |
gsagie | you have 3 patches for the selective proactive | 07:22 |
gsagie | You know how to use --amend right? we need to "abandon" 2 of them | 07:22 |
gsagie | is this https://review.openstack.org/#/c/273877/ the most updated one? | 07:22 |
gsagie | When you want to send a fix to git review, you need to "git commit -a --amend" on your previous commit, that way git uses the older commit Change-Id and gerrit (review board) knows that its the same patch and send it as a fix, like "Patch number 2" on the same review | 07:24 |
gsagie | if you dont do that, it just sends another patch so you have 3 different patches right now for the selective proactive spec | 07:24 |
gsagie | nick-ma: ping | 07:25 |
gsagie | want to talk about the DB consistency ? :) we need to pick one solution over the other | 07:26 |
gsagie | raofei: please let me know if you have any questions regarding the DNAT | 07:34 |
DuanKebo | Hi Gal! | 07:53 |
gsagie | hi DuanKebo! :) | 07:54 |
DuanKebo | Sorry, I did know --amend. But Huang kun have told me. | 07:54 |
gsagie | dingboopt: kexiadong is by any chance near you? ;) i want to speak with him about Kuryr if he is aroudn | 07:54 |
gsagie | DuanKebo: no problem, is the patch i said is the latests? i will remove the others (or you can do it, just pick "Abandon" in the gerrit review site) | 07:55 |
dingboopt | no, I am not in hangzhou. But I have notified kexiaodong. He says he will login irc soon | 07:58 |
DuanKebo | https://review.openstack.org/#/c/273878/ is the latest. | 07:58 |
DuanKebo | I can abandon the other two. | 07:58 |
gsagie | dingboopt: thanks! | 07:59 |
gsagie | DuanKebo: thanks :) | 07:59 |
*** kexiaodong has joined #openstack-dragonflow | 07:59 | |
*** gampel1 has joined #openstack-dragonflow | 08:00 | |
*** gampel has quit IRC | 08:00 | |
kexiaodong | miao | 08:00 |
openstackgerrit | Merged openstack/dragonflow: Remove the test number of flow https://review.openstack.org/274743 | 08:03 |
openstackgerrit | Merged openstack/dragonflow: Add new OVS test for DHCP application pipeline https://review.openstack.org/273548 | 08:09 |
*** gampel1 has quit IRC | 08:20 | |
*** gampel has joined #openstack-dragonflow | 08:20 | |
gsagie | DuanKebo: might be good to change the commit topic for this -> https://review.openstack.org/#/c/273878/ | 08:21 |
gsagie | :) | 08:21 |
DuanKebo | OK, but how? | 08:28 |
gsagie | you need to fetch it, and then 'git commit -a --amend' and change the topic | 08:29 |
gsagie | for fetching it, you have a link in the upper screen of the review, where it says "Download" | 08:30 |
gsagie | pick the first one for Checkout | 08:30 |
DuanKebo | OK, let me try it, thank you! | 08:31 |
openstackgerrit | Merged openstack/dragonflow: Move port creation/deletion testing to its own class https://review.openstack.org/275020 | 08:39 |
openstackgerrit | Gal Sagie proposed openstack/dragonflow: Delete broadcast/multicast classifier flow on network delete https://review.openstack.org/275075 | 08:47 |
openstackgerrit | Shlomo proposed openstack/dragonflow: Add performance testing spec https://review.openstack.org/272559 | 08:57 |
*** gampel has quit IRC | 09:56 | |
*** gongysh has joined #openstack-dragonflow | 09:56 | |
*** openstackgerrit has quit IRC | 10:17 | |
*** openstackgerrit has joined #openstack-dragonflow | 10:18 | |
*** gongysh has quit IRC | 10:25 | |
*** yamamoto_ has quit IRC | 10:37 | |
*** gampel has joined #openstack-dragonflow | 11:48 | |
*** gampel1 has joined #openstack-dragonflow | 11:50 | |
*** gampel has quit IRC | 11:52 | |
*** gampel has joined #openstack-dragonflow | 12:06 | |
*** gampel1 has quit IRC | 12:07 | |
*** yamamoto has joined #openstack-dragonflow | 12:15 | |
*** yamamoto has quit IRC | 12:22 | |
*** yamamoto has joined #openstack-dragonflow | 12:23 | |
*** yamamoto has quit IRC | 12:27 | |
openstackgerrit | Omer Anson proposed openstack/dragonflow: Add ARP responder option. https://review.openstack.org/273139 | 12:37 |
*** yamamoto has joined #openstack-dragonflow | 12:46 | |
openstackgerrit | Gal Sagie proposed openstack/dragonflow: Add tenant_id as topics for core elements https://review.openstack.org/275172 | 12:54 |
gsagie | dingboopt: just as a suggestion, next time its better to split work for few patches, for example i see now in the secgroup patch that you do plugin side and also controller side | 12:55 |
gsagie | this makes it longer for the patch to get merged and makes it more difficult for you | 12:56 |
*** yamamoto has quit IRC | 12:56 | |
gsagie | with more merges | 12:56 |
openstackgerrit | Merged openstack/dragonflow: Delete broadcast/multicast classifier flow on network delete https://review.openstack.org/275075 | 12:56 |
dingboopt | gsagie: Yes, totally agree. So, how do I split this patch in two parts? | 12:59 |
gsagie | dingboopt: you can do it manually, but not sure its easier now, just for next time maybe :) | 12:59 |
dingboopt | ok, got it | 13:02 |
kun_huang | gampel: ping | 13:08 |
openstackgerrit | Gal Sagie proposed openstack/dragonflow: Add tenant_id as topics for core elements https://review.openstack.org/275172 | 13:10 |
openstackgerrit | duankebo proposed openstack/dragonflow: Selective Topology Distribution https://review.openstack.org/275199 | 13:23 |
*** gongysh has joined #openstack-dragonflow | 13:26 | |
*** yamamoto has joined #openstack-dragonflow | 13:27 | |
*** yamamoto has quit IRC | 13:27 | |
*** yamamoto has joined #openstack-dragonflow | 13:28 | |
openstackgerrit | Omer Anson proposed openstack/dragonflow: Add ARP responder option. https://review.openstack.org/273139 | 13:38 |
gampel | kun_huang: pong | 13:45 |
kun_huang | gampel: some news from #openstack-performance | 13:49 |
kun_huang | dina is pushing some IBM guys to publish OVN perfomance testing plan into performance-docs | 13:49 |
kun_huang | would you like to push something like this | 13:50 |
kun_huang | like rewriting documents https://review.openstack.org/#/c/272559/ | 13:51 |
gampel | yes that will be excellent ,we stated a some thing similar https://review.openstack.org/#/c/272559/ | 13:56 |
gampel | it will good if you could work with Shlomo on that | 13:57 |
gampel | kun_huang: can you please send us performance-docs example that was published for other projects | 13:59 |
kun_huang | gampel: an IBM guy is working on that, but haven't submitted it | 14:00 |
kun_huang | but you could have a look at this http://docs-draft.openstack.org/04/270204/4/check/gate-performance-docs-docs/9264b70//doc/build/html/ | 14:00 |
Shlomo_N | kun_huang: hi, thanks for the links, I will take a look at that | 14:04 |
gampel | Shlomo and kun can you guys work together to review it | 14:05 |
gampel | kun_huang: did you get the servers ? | 14:05 |
Shlomo_N | gample: sure | 14:07 |
kun_huang | gampel: those servers will be posted after chinese new year | 14:08 |
gampel | Ok keep us up to date | 14:10 |
*** DuanKebo_ has joined #openstack-dragonflow | 14:10 | |
gongysh | gampel, hi | 14:18 |
gongysh | what is the lowest ovs version to be used? | 14:18 |
openstackgerrit | duankebo proposed openstack/dragonflow: Selective Topology Distribution https://review.openstack.org/275199 | 14:23 |
openstackgerrit | Eran Gampel proposed openstack/dragonflow: Add support for DF built-in pub/sub https://review.openstack.org/263322 | 14:32 |
openstackgerrit | Eran Gampel proposed openstack/dragonflow: Add support for DF built-in pub/sub https://review.openstack.org/263322 | 14:35 |
*** yamamoto has quit IRC | 16:05 | |
*** saggi has quit IRC | 16:06 | |
*** yamamoto has joined #openstack-dragonflow | 16:07 | |
*** yamamoto has quit IRC | 16:08 | |
*** shihanzhang has quit IRC | 16:16 | |
*** shihanzhang has joined #openstack-dragonflow | 16:17 | |
*** gongysh has quit IRC | 16:18 | |
*** DuanKebo has quit IRC | 16:20 | |
*** DuanKebo has joined #openstack-dragonflow | 16:21 | |
*** shihanzhang has quit IRC | 16:27 | |
*** shihanzhang has joined #openstack-dragonflow | 16:28 | |
openstackgerrit | Eran Gampel proposed openstack/dragonflow: Add support for DF built-in pub/sub https://review.openstack.org/263322 | 16:28 |
openstackgerrit | Merged openstack/dragonflow: Prepare alembic scripts for subproject DB module https://review.openstack.org/274496 | 16:57 |
*** DuanKebo_ has quit IRC | 17:20 | |
*** gampel has quit IRC | 17:30 | |
*** DuanKebo has quit IRC | 17:31 | |
*** DuanKebo has joined #openstack-dragonflow | 17:31 | |
*** openstackgerrit has quit IRC | 17:32 | |
*** openstackgerrit has joined #openstack-dragonflow | 17:33 | |
*** shz has joined #openstack-dragonflow | 19:03 | |
*** shihanzhang has quit IRC | 19:12 | |
*** DuanKebo has quit IRC | 19:13 | |
*** DuanKebo has joined #openstack-dragonflow | 19:13 | |
*** hanzhang has joined #openstack-dragonflow | 19:44 | |
*** shz has quit IRC | 19:51 | |
*** DuanKebo has quit IRC | 19:52 | |
*** DuanKebo has joined #openstack-dragonflow | 19:52 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!