| 00:41.35 | *** join/#asterisk-dev pchero (~pchero@x1-6-d8-d7-75-5e-99-79.cpe.webspeed.dk) |
| 06:34.14 | *** join/#asterisk-dev tsearle (~tsearle@37.19.10.33) |
| 06:34.14 | *** mode/#asterisk-dev [+o tsearle] by ChanServ |
| 06:46.44 | *** join/#asterisk-dev CELYA_ (~Thunderbi@LStLambert-657-1-90-73.w80-13.abo.wanadoo.fr) |
| 07:10.30 | *** join/#asterisk-dev tzafrir (~tzafrir@local.xorcom.com) |
| 07:10.30 | *** mode/#asterisk-dev [+o tzafrir] by ChanServ |
| 07:38.23 | *** join/#asterisk-dev hehol (~hehol@gatekeeper.loca.net) |
| 09:46.43 | *** join/#asterisk-dev jkroon (~jkroon@196.223.97.98) |
| 10:40.03 | *** join/#asterisk-dev jkroon (~jkroon@uls-154-73-35-201.wall.uls.co.za) |
| 11:04.56 | *** join/#asterisk-dev Worldexe (~Worldexe@95-107-33-134.dsl.orel.ru) |
| 11:21.21 | *** join/#asterisk-dev CELYA_ (~Thunderbi@LStLambert-657-1-90-73.w80-13.abo.wanadoo.fr) |
| 13:20.57 | *** join/#asterisk-dev sduthil (~sduthil@modemcable094.94-70-69.static.videotron.ca) |
| 13:58.27 | gtjoseph | coreyfarrell: the python 3 patches fail 'make ari-stubs' for me on Fedora 27 |
| 13:58.45 | gtjoseph | i'm updating the jenkins check jobs to do a make ari-stubs. |
| 13:59.07 | coreyfarrell | aw hell. they work on fedora 26 :( |
| 14:00.10 | gtjoseph | fyi... Python 2.7.14 |
| 14:01.25 | coreyfarrell | weird that's what's on f26 too |
| 14:02.45 | *** join/#asterisk-dev kharwell (kharwell@nat/digium/x-xgkavobdljgfinng) |
| 14:02.45 | *** mode/#asterisk-dev [+o kharwell] by ChanServ |
| 14:03.40 | coreyfarrell | I'm downloading it again to see if maybe I forgot to retest my last changes. |
| 14:04.14 | coreyfarrell | yep that's it. the rules for relative imports really suck. |
| 14:04.48 | gtjoseph | git branch |
| 14:04.55 | gtjoseph | oops |
| 14:09.11 | *** join/#asterisk-dev scgm11_ (~scgm11@r186-52-123-131.dialup.adsl.anteldata.net.uy) |
| 14:15.08 | scgm11_ | is there any reason why hep.conf does not allow capture_address = homer:9061 and allow capture_address = 192.168.1.3:9061 |
| 14:16.08 | scgm11_ | <PROTECTED> |
| 14:16.24 | scgm11_ | I can ping homer just fine |
| 14:16.47 | file | Depends on if it resolves the IP address or not |
| 14:20.24 | scgm11_ | if I can ping it it resolves the ip :) |
| 14:20.39 | scgm11_ | from that instance |
| 14:21.39 | file | I mean the code |
| 14:21.54 | gtjoseph | The ports in your config and in the error message are different |
| 14:23.04 | scgm11_ | yes the firts was just an example sorry |
| 14:24.09 | *** join/#asterisk-dev rmudgett (rmudgett@nat/digium/x-qhgwyyhswlqbnhbi) |
| 14:24.09 | *** mode/#asterisk-dev [+o rmudgett] by ChanServ |
| 14:26.12 | scgm11_ | full real output |
| 14:26.17 | scgm11_ | capture_address = 172.18.0.4:9061 |
| 14:26.17 | scgm11_ | module reload res_hep.so |
| 14:26.18 | scgm11_ | Module 'res_hep.so' reloaded successfully. |
| 14:26.18 | scgm11_ | <PROTECTED> |
| 14:26.20 | scgm11_ | capture_address = homer:9061 |
| 14:26.20 | scgm11_ | [Apr 9 14:25:51] WARNING[333]: res_hep.c:383 hepv3_data_alloc: Failed to create address from homer:9061 |
| 14:26.22 | scgm11_ | ping homer |
| 14:26.23 | scgm11_ | PING homer (172.18.0.4): 56 data bytes |
| 14:26.24 | scgm11_ | 64 bytes from 172.18.0.4: icmp_seq=0 ttl=64 time=0.306 ms |
| 14:26.24 | scgm11_ | 64 bytes from 172.18.0.4: icmp_seq=1 ttl=64 time=0.196 ms |
| 14:28.17 | gtjoseph | res_hep does not seem to be DNS aware. |
| 14:29.03 | gtjoseph | it's attempting to parse "homer" into a sockaddr structure |
| 14:31.10 | *** join/#asterisk-dev bford (uid283514@gateway/web/irccloud.com/x-oqieoqlevoybuifm) |
| 14:31.10 | *** mode/#asterisk-dev [+o bford] by ChanServ |
| 14:34.45 | scgm11_ | that should be change right? |
| 14:36.20 | file | it could be |
| 14:58.59 | gtjoseph | coreyfarrell: did you test changing the templates and re-running make ari-stubs to insure that the stubs were regenerated correctly? |
| 14:59.38 | coreyfarrell | not sure what you mean by that? |
| 15:00.03 | gtjoseph | ari-stubs regenerates the c stubs |
| 15:01.34 | coreyfarrell | oh. a bunch of the changes were because python3 was generating them incorrectly once I got it to run, especially involving the websockets code. |
| 15:02.48 | *** join/#asterisk-dev tuxd00d (~tuxd00d@wsip-70-166-203-100.ph.ph.cox.net) |
| 15:03.41 | coreyfarrell | I'm not sure that covers exactly what you're asking? before some of my changes py3 was incorrectly including asterisk/http_websocket.h to all resource modules so I tested until that was fixed. |
| 15:05.05 | coreyfarrell | one thing that has changed is in the generated documentation. python3 does not use the same sort order when dumping JSON objects so the JSON in documentation has keys in a different order. |
| 15:07.04 | gtjoseph | ok i think we're covered |
| 17:03.24 | rmudgett | gtjoseph: file: https://gerrit.asterisk.org/#/c/7710/ updated. Now it is part of a patch series. Enjoy! :) |
| 17:08.16 | rmudgett | It is best to look at it as a completely new patch. |
| 17:10.25 | file | moo. |
| 17:23.16 | *** join/#asterisk-dev snuff-work (~snuff-wor@210.9.148.102) |
| 17:23.16 | *** mode/#asterisk-dev [+o snuff-work] by ChanServ |
| 17:32.02 | *** join/#asterisk-dev scgm11__ (~scgm11@r186-52-125-57.dialup.adsl.anteldata.net.uy) |
| 18:11.11 | gtjoseph | rmudgett: is there a reason you can't collapse the pjsip_scheduler reviews into just 1??? |
| 18:12.37 | rmudgett | I could collapse them to 3. I am adding two "features" and I didn't want to make the fixes patch too big. |
| 18:14.56 | rmudgett | *too big or confusing* |
| 18:15.21 | gtjoseph | Since they're all in the same file I think 1 patch will make things a lot easier to understand, test and merge. |
| 18:16.04 | gtjoseph | also for the task sync patch, what if anythign do external module writers need to know? |
| 18:18.42 | *** join/#asterisk-dev infobot (ibot@rikers.org) |
| 18:18.42 | *** topic/#asterisk-dev is Asterisk Development Discussion -=- http://www.asterisk.org/developers -=- Tier 2 and 3.14159265 support is in #asterisk -=- Check out our blog! blogs.asterisk.org -=- Follow on Twitter at @AsteriskDev |
| 18:19.24 | gtjoseph | you're not the one who has to review test and merge. :) |
| 18:21.29 | rmudgett | https://gerrit.asterisk.org/#/c/8742/ and https://gerrit.asterisk.org/#/c/8744/ are technically not part of the ASTERISK-26806 fix series. They depend upon changes in that fix series but are not necessary for the fix. |
| 18:22.53 | gtjoseph | well, the task sync review has the scheduler trace review as it's parent. Is that necessary? |
| 18:23.15 | gtjoseph | the trace review has corner cases as its parent |
| 18:23.22 | gtjoseph | is that necessary? |
| 18:24.42 | gtjoseph | can you change the topics on the reviews that aren't absolutely required for the OPTIONS patch? |
| 18:24.46 | rmudgett | the task sync review could be in a different order relative to the scheduler changes but the OPTIONS patch requires that change. |
| 18:25.27 | gtjoseph | yeah i know options requires the task sync and that's fine. how about the others though? |
| 18:26.35 | rmudgett | I just changed the topics for the new scheduler features. |
| 18:27.41 | gtjoseph | how about the parents? |
| 18:32.30 | rmudgett | what about the parents? It's simpler to keep the reviews straight this way. |
| 18:33.52 | gtjoseph | well, when you do a git review -d, you'll still get commits pulled in that aren't really needed |
| 18:37.17 | rmudgett | If you are going to run the OPTIONS patch then you'll probably appreciate the sorted sched output patch being there. |
| 18:38.37 | gtjoseph | well then, you're choice. Either change the parents or collapse the scheduler reviews into one. |
| 18:41.01 | *** join/#asterisk-dev CELYA_ (~Thunderbi@2a01cb0583d31d0069f09b47b68abfd9.ipv6.abo.wanadoo.fr) |
| 18:48.47 | *** join/#asterisk-dev scgm11_ (~scgm11@r186-52-125-57.dialup.adsl.anteldata.net.uy) |
| 18:51.56 | *** join/#asterisk-dev tzafrir (~tzafrir@62-90-199-247.barak.net.il) |
| 18:51.56 | *** mode/#asterisk-dev [+o tzafrir] by ChanServ |
| 19:05.07 | rmudgett | gtjoseph: I think you confused gerrit. |
| 19:06.14 | gtjoseph | i haven't touched gerrit in hours |
| 19:06.51 | rmudgett | Your insistence on parents did it |
| 19:52.35 | *** join/#asterisk-dev dadrc (~quassel@unaffiliated/dadrc) |
| 23:21.57 | *** part/#asterisk-dev kharwell (kharwell@nat/digium/x-xgkavobdljgfinng) |
| 23:39.47 | *** join/#asterisk-dev snuff-work (~snuff-wor@210.9.148.102) |
| 23:39.57 | *** mode/#asterisk-dev [+o snuff-work] by ChanServ |