On Mon, Jun 8, 2015 at 9:57 AM, Leif Madsen leif.mad...@avoxi.com wrote:
On Mon, Jun 8, 2015 at 9:35 AM, Russell Bryant russ...@russellbryant.net
wrote:
On Tue, Jun 2, 2015 at 8:05 PM, Matthew Jordan mjor...@digium.com
wrote:
Personally, I'd like some way to present any user of Asterisk
absolute numbers measure. A survey could still produce useful
relative numbers and help identify some trends over time.
--
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing
, you'll get fewer
servers reporting data but it's the much friendlier thing to with respect
to privacy.
+1.
--
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
On Mon, Apr 27, 2015 at 8:20 AM, Olle E. Johansson o...@edvina.net wrote:
On 24 Apr 2015, at 15:42, Russell Bryant russ...@russellbryant.net
wrote:
On Fri, Apr 24, 2015 at 8:31 AM, Joshua Colp jc...@digium.com wrote:
Olle E. Johansson wrote:
Playing around following Matt's wiki page
warrants further
changes they also need to be squashed back into a single commit with the
original changes. The single commit you post for review is what is reviewed
and merged into the branch.
Gerrit can also work on a patch series, and tracks dependencies between
those patches.
--
Russell
On Fri, Apr 24, 2015 at 12:17 PM, Matthew Jordan mjor...@digium.com wrote:
On Fri, Apr 24, 2015 at 8:42 AM, Russell Bryant
russ...@russellbryant.net wrote:
On Fri, Apr 24, 2015 at 8:31 AM, Joshua Colp jc...@digium.com wrote:
Olle E. Johansson wrote:
Playing around following Matt's
.
--
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
very different from a source management
perspective, the actual release deliverable shouldn't look any different.
menuselect is in the tarball just like it was before. Or am I missing
something?
--
Russell Bryant
reviews, when patches are updated, when
patches merge, or when people make comments (all optionally on a per
repository basis). So, another option would be to drop sending to the list
completely and let people configure their own notifications that suit their
needs and interests.
--
Russell
--
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
On Tue, Apr 14, 2015 at 10:55 AM, Matthew Jordan mjor...@digium.com wrote:
On Tue, Apr 14, 2015 at 9:43 AM, Dan Jenkins dan.jenkin...@gmail.com
wrote:
On Tue, Apr 14, 2015 at 3:18 PM, Russell Bryant
russ...@russellbryant.net
wrote:
On Tue, Apr 14, 2015 at 8:47 AM, Matthew Jordan
Russell Bryant has posted comments on this change.
Change subject: Add .gitignore and .gitreview files
..
Patch Set 2:
(1 comment)
https://gerrit.asterisk.org/#/c/42/2/.gitignore
File .gitignore:
Line 21: addons/mp3
I
Russell Bryant has posted comments on this change.
Change subject: Add .gitignore and .gitreview files
..
Patch Set 2:
(1 comment)
https://gerrit.asterisk.org/#/c/42/2//COMMIT_MSG
Commit Message:
Line 15: Tested-by: George
Russell Bryant has posted comments on this change.
Change subject: Add .gitignore and .gitreview files
..
Patch Set 3: Code-Review+1
--
To view, visit https://gerrit.asterisk.org/42
To unsubscribe, visit https
removing it is
another option.
Anyway, the goal is to kick off the move of Asterisk to Git
immediately after we get the next batch of releases out. I'll send out
an e-mail once we know exactly when that is.
\o/
--
Russell Bryant
Russell Bryant has posted comments on this change.
Change subject: Add .gitignore files
..
Patch Set 1: Code-Review+1
looks sane
--
To view, visit https://gerrit.asterisk.org/8
To unsubscribe, visit https
Russell Bryant has posted comments on this change.
Change subject: Add a .gitreview file for the testsuite
..
Patch Set 1: Code-Review+1
(1 comment)
https://gerrit.asterisk.org/#/c/7/1/.gitreview
File .gitreview:
Line 5
Russell Bryant has posted comments on this change.
Change subject: Add a .gitreview file
..
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.asterisk.org/5
To unsubscribe, visit https://gerrit.asterisk.org/settings
a .gitreview file added [1].
[1] https://gerrit.asterisk.org/#/c/3
--
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit
Russell Bryant has uploaded a new change for review.
https://gerrit.asterisk.org/3
Change subject: Add .gitreview file
..
Add .gitreview file
This file tells 'git review' how to contact the gerrit server.
Change-Id
Russell Bryant has posted comments on this change.
Change subject: .gitignore: Add a .gitignore file that mirrors the previous
svn:ignore
..
Patch Set 2:
You can create .gitignore files in subdirectories instead of listing
compile time option and the core show
locks CLI command?
https://wiki.asterisk.org/wiki/display/AST/CLI+commands+useful+for+debugging
--
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com
a different model. I don't see why anyone would use them
for day-to-day work like team branches have been used in the past.
--
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev
. Let users
make their own arrangements be it GitHub or in my case, my own GIT
infrastructure.
+1. I don't think it makes sense with git. github or whatever should work
just fine for that purpose.
--
Russell Bryant
be architected in
my view.
Well done, dev community. Keep kicking ass.
--
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options
)
and not in Asterisk itself.
2) Based on this backtrace, it shows chan_sip calling into this client
library, which doesn't exist in Asterisk code, so it could be a problem
specific to modifications made in your version.
--
Russell Bryant
--
Russell Bryant
was designed in such a way that
people could run it against their own deployment to do some validation, but
quite far from how it actually works. So, what use cases are you aiming to
allow exactly?
--
Russell Bryant
--
_
-- Bandwidth
scratch or if I can use this code and still submit it at some
point...
Trying to get hold of the original author, but no reply so far.
It should be fine. Users have never been able to use reviewboard without
an active contributor license agreement. (AFAIK)
--
Russell Bryant
On Mon, Oct 13, 2014 at 9:44 AM, Tzafrir Cohen tzafrir.co...@xorcom.com
wrote:
On Mon, Oct 13, 2014 at 09:18:10AM -0400, Russell Bryant wrote:
On Mon, Oct 13, 2014 at 1:58 AM, Tzafrir Cohen tzafrir.co...@xorcom.com
wrote:
Hi
I finally got the asterisk-testsuite accpted
On Thu, Sep 18, 2014 at 12:29 PM, Russell Bryant russ...@russellbryant.net
wrote:
On Thu, Sep 18, 2014 at 11:31 AM, Samuel Galarneau sgalarn...@digium.com
wrote:
On Tue, Sep 16, 2014 at 5:01 PM, Russell Bryant
russ...@russellbryant.net wrote:
On Tue, Sep 16, 2014 at 3:48 PM, Matthew
this is going to be MUCH better
no matter what specific infrastructure you go with.
--
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options
On Tue, Sep 16, 2014 at 10:52 PM, Matthew Jordan mjor...@digium.com wrote:
On Tue, Sep 16, 2014 at 5:01 PM, Russell Bryant russ...@russellbryant.net
wrote:
On Tue, Sep 16, 2014 at 3:48 PM, Matthew Jordan mjor...@digium.com
wrote:
So, to set what I hope are a few guidelines:
(1) I know
page (http://status.openstack.org/zuul/), it has been launching
about 500-600 jobs per hour today.
-- Process Recommendation
I discussed this a good bit above, but I'm happy to answer questions.
--
Russell Bryant
On Tue, Sep 16, 2014 at 6:01 PM, Russell Bryant russ...@russellbryant.net
wrote:
From a high level, all patches go to a code review system. *Every* patch
must be peer reviewed (usually by 2 people, but that's a policy decision).
*Every* patch must also pass tests. Once a patch passes both
On Tue, Sep 16, 2014 at 6:12 PM, Russell Bryant russ...@russellbryant.net
wrote:
On Tue, Sep 16, 2014 at 6:01 PM, Russell Bryant russ...@russellbryant.net
wrote:
From a high level, all patches go to a code review system. *Every* patch
must be peer reviewed (usually by 2 people, but that's
used. chan_iax2 has to do it that
way as all data for *every* call is arriving on the same UDP socket, so it
can't put that one socket fd on every channel.
--
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3952/#review13195
---
Ship it!
looks sane to me
- Russell Bryant
On Aug. 27
to another buffer for every frame you're
sending.
Good luck with your work!
--
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options
on the problem you're having? Without
more detail, nobody will be able to help you.
--
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options
Heh, but you just posted your own patch. Are you still interested in help
here?
--
Russell Bryant
On Wed, Jul 16, 2014 at 11:23 PM, Krandon krandon.br...@gmail.com wrote:
Up'ing bounty to $1,000
--
KB
On Wednesday, July 16, 2014 at 5:56 AM, Krandon wrote:
ARI /continue fails
))
same = n,StartMusicOnHold()
same = n,Wait(15)
same = n,StopMusicOnHold()
same = n,StopMixMonitor()
same = n,Wait(5)
same = n,Hangup()
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3424/#review11562
---
On April 8, 2014, 7:49 p.m., Russell Bryant wrote:
---
This is an automatically
= n,StopMixMonitor()
same = n,Wait(5)
same = n,Hangup()
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit
to
bump the module refcount for func_periodic_hook.so so that the module
cannot be unloaded while there is an active mixmonitor. You can decrement
the module refcount when the mixmonitor is destroyed.
Russell Bryant wrote:
I started looking at this, but I'm not sure the alternative is better
an argument, and that's a pain to do when not using this
helper, so start by doing this conversion.
Diffs
-
/trunk/res/res_monitor.c 412023
Diff: https://reviewboard.asterisk.org/r/3429/diff/
Testing
---
Thanks,
Russell Bryant
---
On April 8, 2014, 7:49 p.m., Russell Bryant wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3424
---
On April 8, 2014, 7:49 p.m., Russell Bryant wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3424
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3423/#review11511
---
Ship it!
lgtm
- Russell Bryant
On April 8, 2014, 12:03 a.m
module.
- Russell
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3424/#review11515
---
On April 8, 2014, 1:28 a.m., Russell Bryant
= n,Wait(5)
same = n,Hangup()
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com
---
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
ast_category_delete()
if (cat-name == category) {
from ast_variable_browse()
if (config-last_browse (config-last_browse-name == category)) {
etc.
Diffs
-
/trunk/main/config.c 394685
Diff: https://reviewboard.asterisk.org/r/2684/diff/
Testing
---
Thanks,
Russell
for now
- Russell
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2684/#review9325
---
On July 18, 2013, 9:44 a.m., Russell Bryant wrote
On April 8, 2014, 6:31 p.m., Tzafrir Cohen wrote:
I know I really shouldn't be writing this, as I added one of the defined()
in that line, but: shouldn't this test be done in autoconf?
Ideally, yes, but seems like a reasonable improvement that could be made later,
beyond the scope of
platforms
- Russell Bryant
On April 8, 2014, 3:31 p.m., Sebastian Wiedenroth wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3426
412023
Diff: https://reviewboard.asterisk.org/r/3429/diff/
Testing
---
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update
()
exten = 105,1,Answer()
same = n,MixMonitor(test.gsm,B(3))
same = n,StartMusicOnHold()
same = n,Wait(15)
same = n,StopMusicOnHold()
same = n,StopMixMonitor()
same = n,Wait(5)
same = n,Hangup()
Thanks,
Russell Bryant
)
same = n,Set(PERIODIC_HOOK(${BEEP_ID})=off)
same = n,Wait(20)
same = n,Set(PERIODIC_HOOK(${BEEP_ID})=on)
same = n,Wait(20)
same = n,Hangup()
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http
it off. The change to make it
the only way it works is really the right thing to do. It was equivalent
to an option called make_things_work_properly=yes.
--
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http
)
same = n,Wait(20)
same = n,Set(PERIODIC_HOOK(${BEEP_ID})=on)
same = n,Wait(20)
same = n,Hangup()
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3362/#review11476
---
On April 1, 2014, 4:43 p.m., Russell Bryant wrote
,Wait(20)
same = n,Set(PERIODIC_HOOK(${BEEP_ID})=on)
same = n,Wait(20)
same = n,Hangup()
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
welcome and more information to come once we
begin the actual migration.
+1! :-)
--
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update
On March 31, 2014, 5:58 p.m., Corey Farrell wrote:
/trunk/funcs/func_periodic_hook.c, lines 141-143
https://reviewboard.asterisk.org/r/3362/diff/6/?file=56951#file56951line141
Macro is deprecated, why not use Gosub?
Russell Bryant wrote:
I tried that. GoSub() doesn't
On March 31, 2014, 5:58 p.m., Corey Farrell wrote:
/trunk/funcs/func_periodic_hook.c, lines 141-143
https://reviewboard.asterisk.org/r/3362/diff/6/?file=56951#file56951line141
Macro is deprecated, why not use Gosub?
Russell Bryant wrote:
I tried that. GoSub() doesn't
On March 31, 2014, 5:58 p.m., Corey Farrell wrote:
/trunk/funcs/func_periodic_hook.c, lines 141-143
https://reviewboard.asterisk.org/r/3362/diff/6/?file=56951#file56951line141
Macro is deprecated, why not use Gosub?
Russell Bryant wrote:
I tried that. GoSub() doesn't
On March 31, 2014, 5:58 p.m., Corey Farrell wrote:
/trunk/funcs/func_periodic_hook.c, lines 141-143
https://reviewboard.asterisk.org/r/3362/diff/6/?file=56951#file56951line141
Macro is deprecated, why not use Gosub?
Russell Bryant wrote:
I tried that. GoSub() doesn't
= n,Set(PERIODIC_HOOK(${BEEP_ID})=off)
same = n,Wait(20)
same = n,Set(PERIODIC_HOOK(${BEEP_ID})=on)
same = n,Wait(20)
same = n,Hangup()
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http
/#review11441
---
On March 29, 2014, 8:27 p.m., Russell Bryant wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3362
,Hangup()
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk
)
same = n,Set(PERIODIC_HOOK(${BEEP_ID})=on)
same = n,Wait(20)
same = n,Hangup()
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
being created or re-enabled.
Unless this behaviour is documented I would expect the first run after
create or resume to be delayed state-interval seconds.
Russell Bryant wrote:
Fair. I had originally intended for it to run immediately, but I think
it makes more sense to delay
)
same = n,Set(PERIODIC_HOOK(${BEEP_ID})=off)
same = n,Wait(20)
same = n,Set(PERIODIC_HOOK(${BEEP_ID})=on)
same = n,Wait(20)
same = n,Hangup()
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3362/#review11455
---
On March 31, 2014, 4:11 p.m., Russell Bryant wrote
)})
same = n,Wait(20)
same = n,Set(PERIODIC_HOOK(${BEEP_ID})=off)
same = n,Wait(20)
same = n,Set(PERIODIC_HOOK(${BEEP_ID})=on)
same = n,Wait(20)
same = n,Hangup()
Thanks,
Russell Bryant
--
_
-- Bandwidth
On March 31, 2014, 5:58 p.m., Corey Farrell wrote:
/trunk/funcs/func_periodic_hook.c, lines 141-143
https://reviewboard.asterisk.org/r/3362/diff/6/?file=56951#file56951line141
Macro is deprecated, why not use Gosub?
Russell Bryant wrote:
I tried that. GoSub() doesn't
()
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
]
exten = 100,1,Answer()
same = n,Set(BEEP_ID=${PERIODIC_HOOK(hooks,beep,5)})
same = n,Wait(20)
same = n,Set(PERIODIC_HOOK(${BEEP_ID})=off)
same = n,Wait(20)
same = n,Hangup()
Thanks,
Russell Bryant
(PERIODIC_HOOK(${BEEP_ID})=on)
same = n,Wait(20)
same = n,Hangup()
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit
/
--
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
something like the existing validate-docs
Makefile target that only runs if dev mode is on. Add a
validate-linear-migrations target that runs in dev mode or something like
that.
--
Russell Bryant
--
_
-- Bandwidth and Colocation
On Tue, Mar 18, 2014 at 10:07 AM, Matthew Jordan mjor...@digium.com wrote:
On Tue, Mar 18, 2014 at 8:31 AM, Russell Bryant
russ...@russellbryant.net wrote:
On Mon, Mar 17, 2014 at 1:01 PM, Joshua Colp jc...@digium.com wrote:
Matthew Jordan wrote:
snip
Technically, it's
,Answer()
same = n,Set(BEEP(5)=on)
same = n,Wait(20)
same = n,Set(BEEP()=off)
same = n,Wait(20)
same = n,Hangup()
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com
every 30 seconds
exten = foo,1,Set(PERIODIC_HOOK(30,beep@dialplan_hooks)
Does that sound better than the current BEEP()?
- Russell Bryant
On March 15, 2014, 6:15 p.m., Russell Bryant wrote:
---
This is an automatically generated e
.
Diffs
-
/branches/11/res/res_musiconhold.c 409286
Diff: https://reviewboard.asterisk.org/r/3282/diff/
Testing
---
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com
no bridged instance, but this seems like a pretty
safe function input sanity check.
Diffs
-
/tags/1.8.18.1/res/res_rtp_asterisk.c 405155
Diff: https://reviewboard.asterisk.org/r/3300/diff/
Testing
---
Thanks,
Russell Bryant
://reviewboard.asterisk.org/r/3282/#review11024
---
On March 2, 2014, 8:45 p.m., Russell Bryant wrote:
---
This is an automatically generated e-mail. To reply, visit:
https
- Russell
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3282/#review11020
---
On March 2, 2014, 8:45 p.m., Russell Bryant wrote
/r/3282/diff/
Testing
---
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com
On Feb. 28, 2014, 12:51 p.m., Matt Jordan wrote:
I'm not sure I understand the need for this patch.
Setting a configuration option twice - when that option doesn't support
being set multiple times - would generally have undefined behaviour. Your
patch changes it so that Asterisk
/
Testing
---
Created some nested includes in extconfig.conf. Confirmed error before the
patch and loading fine afterwards.
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com
that. It at least doesn't blow
up immediately ...
I wanted to get the patch and analysis up to get some more eyes on it.
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com
On Jan. 22, 2014, 1:14 a.m., Paul Belanger wrote:
branches/12/contrib/ast-db-manage/config/versions/2fc7930b41b3_add_pjsip_endpoint_options_for_12_1.py,
line 1
https://reviewboard.asterisk.org/r/3148/diff/1/?file=53005#file53005line1
You should be creating new alembic scripts not
to get some more eyes on it.
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com
-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3135/#review10631
---
On Jan. 17, 2014, 10:18 p.m., Russell Bryant wrote:
---
This is an automatically generated e-mail
in production or anything like that. It at least doesn't blow
up immediately ...
I wanted to get the patch and analysis up to get some more eyes on it.
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http
.
In any case, Fedora has had a systemd unit for Asterisk for quite some time.
For reference, you can view it here:
http://pkgs.fedoraproject.org/cgit/asterisk.git/tree/asterisk.service
- Russell Bryant
On Dec. 24, 2013, 4:49 p.m., Tzafrir Cohen wrote
) executed *CLI sip reload
4) Repeat step 1, observe outboundproxy no longer set when the patch is applied
I also verified that step 4 fails without the patch
Thanks,
Russell Bryant
--
_
-- Bandwidth and Colocation Provided by http
with outboundproxy set, observed with 'sip show peer foo'
2) removed outboundproxy from the config
3) executed *CLI sip reload
4) Repeat step 1, observe outboundproxy no longer set when the patch is applied
I also verified that step 4 fails without the patch
Thanks,
Russell Bryant
think all of this stuff is really great to have. I would also encourage
anyone else that comes up with anything more extensive like this for testing to
help get it into the tree somewhere. As it starts to pile up, we will start to
have a nice set of tests available ...
--
Russell Bryant
Senior
1 - 100 of 274 matches
Mail list logo