Fix Freepbx CDR backend #2942
Merged
Fix Freepbx CDR backend #2942
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
holta
reviewed
Aug 13, 2021
holta
reviewed
Aug 15, 2021
holta
reviewed
Aug 15, 2021
|
I didn't have Ubuntu 20.04 handy so this PR was initially test installed on Debian 11, which yielded: Do we care that one of the 4 lines ( (As compared to #2942 (comment) — or is that not consequential?) |
|
Another test install of this PR is running on Ubuntu Server 20.04 (10.8.0.34) for comparison. It should complete right around 9PM EDT / NYC Time. |
Not really this was an out of the box test, what works and what doesn't? ip to poke at? |
The result is almost identical on Ubuntu Server 20.04 (10.8.0.34) here: iiab-diagnostics output: http://sprunge.us/kPhFOk?en (Feel free to log in.) |
|
Not sure what is up with that, and I forgot to check before I reran mk-odbc but the library builds...
|
|
@holta The above 4 pastebin commands are useful perhaps a quick script to gather and post the info could be useful here? |
|
The KISS commit was the quick fix for b4e93f5#commitcomment-54840916 |
Typo? |
|
Well sort of, needed the full web path and not just the commit hash, corrected now. |
|
@jvonau I suggest this PR be merged shortly, to give us a new baseline for @lemueldsouza & all to try out? (Or do you prefer more testing of any particular kind?) |
|
Lets let the chips fall where they may and I'll punt over to those who are actively testing for feedback. |
holta
added a commit
to holta/iiab
that referenced
this pull request
Aug 17, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes bug:
#2938 (comment)
Description of changes proposed in this pull request:
Somewhere in the muck I got the CDR backends to showup
Needs full clean testing