-
Notifications
You must be signed in to change notification settings - Fork 300
Description
Hello π
Dear @asterisk-java/contributors, @asterisk-java/owners and everyone who want to support us!
I would like to invite all of you to this conversation.
These are the next steps of making the asterisk-java project great again.
β Disclaimer
This is only a proof-of-concept and should be treated as a proposal and a base for the discussion.
I hope responses will be faster.
βΌοΈ
If you don't agree with any changes or proposals, let me know by using the reaction button, or more helpfully, by giving constructive feedback on what is wrong and how it should be handled.
I've diagnosed a couple of problematic areas.
1. Code maintenance and releases
π₯ Problems:
- Right now, releases are done by some weird Dart release scripts, which are neither Java nor GitHub friendly.
- We don't have any changelog. Nowadays, this is bad practice and very uninformative.
- In addition, a lot of changes are made without any issue or even a short description of the reason for changes.
π‘ Solutions:
- Do release using GitHub Releases connected with GitHub Actions and Maven Release Plugin (or another way, but using Java).
- GitHub Releases helps us with the changelog.
- All changes must have an appropriate issue or pull request with a short description. Master (maybe main?) should be blocked for direct pushes. Every change should be done via pull requests (for everyone!) and reviewed.
2. Project structure
π₯ Problems:
- Right now everything is mixed into one module. If you want to run only fastagi or manager - you can't.
- There aren't boundaries between modules.
π‘ Solutions:
I suggest the following project structure:
asterisk-java-parent (POM)
βββ asterisk-java-agi (POM)
β βββ asterisk-java-agi-asyncagi (JAR) dependencies: asterisk-java-agi-command, asterisk-java-agi-core, asterisk-java-ami-manager
β βββ asterisk-java-agi-command (JAR)
β βββ asterisk-java-agi-core (JAR)
β βββ asterisk-java-agi-fastagi (JAR) dependencies: asterisk-java-agi-core, asterisk-java-agi-command
βββ asterisk-java-ami (POM)
β βββ asterisk-java-ami-action (JAR)
β βββ asterisk-java-ami-event (JAR)
β βββ asterisk-java-ami-manager (JAR) dependencies: asterisk-java-ami-action, asterisk-java-ami-event
βββ asterisk-java-common (JAR)
βββ asterisk-java-live (JAR) dependencies: asterisk-java-ami-manager
βββ asterisk-java-pbx (JAR) dependencies: asterisk-java-agi-fastagi, asterisk-java-ami-action, asterisk-java-ami-event
Why are modules good?
- They give an overview of how asterisk-java and Asterisk itself are designed. They also provide boundaries of responsibility.
- Prevent mixing objects (exceptions, models, etc.) between modules.
- Gives possibility to extend in the future (we can easily add e.g. ARI module).
3. Supported versions
π₯ Problems:
Currently the project doesn't have clearly defined Asterisk and Java supported versions.
π‘ Solutions:
Java
- All LTS versions shlould be supported and tested (8, 11, 17).
- If any Java version becomes EOL, we should drop support in the next major release (for Java 8 it's 2025-03-31).
Asterisk
For Asterisk versions 16, 18 and 20 the major should be 4.x.x. When 16 reaches its EOL in 2023-10-09, we should change
the major to 5.x.x. We should also change the major version when a new Asterisk LTS version is released.
asterisk-java project release cycle:
Asterisk Java | Asterisk | Java | EOL | Reason |
---|---|---|---|---|
4.x.x | 16, 18, 20 | 8, 11, 17 | 2023-10-09 | Java 21 released Asterisk 16 EOL |
5.x.x | 18, 20 | 8, 11, 17, 21 | 2024-10-19 | Asterisk 22 released |
6.x.x | 18, 20, 22 | 8, 11, 17, 21 | 2025-03-31 | Java 8 EOL |
7.x.x | 18, 20, 22 | 11, 17, 21 | 2025-10-20 | Asterisk 18 EOL |
8.x.x | 20, 22 | 11, 17, 21 | 2026-10-19 | Java 11 EOL Java 27(?) released Asterisk 24 released |
β Disclaimer
With this release process I have a lot of confusion. I'm not 100% sure it's a good way to do it.
4. Labels
π₯ Problems:
- Without labels, we cannot make any automation.
- Also issues and pull requests are uncategorized.
π‘ Solutions:
Create labels and use it.
Connected with particular Asterisk version.
asterisk/16
asterisk/18
asterisk/20
...
Type of issue or pull request.
kind/bug
kind/breaking
kind/dependencies
kind/documentation
kind/enhancement
kind/feature
kind/question
kind/refactor
kind/wontfix
Additional information for issue or pull request.
meta/skip-changelog
Connected for particular module.
module/agi
module/ami
module/live
module/pbx
5. Cleanup branches
π₯ Problem:
There is a bunch of the stale branches.
π‘ Solution:
Remove all of them, and leave only master.
6. Integration tests
π₯ Problem:
Right now, there aren't any integration tests. We are not sure that our lib working with supported Asterisk versions.
π‘ Solution:
We should running integration test for each supported Asterisk version. To achieve this, we can use Testcontainers.
Here is a proof-of-concept of super easy and a little bit dumb example, but this prove it's possible to run integration tests automatically.
Tests should be run for all supported Asterisk versions.
7. Web page asterisk-java.org
π₯ Problem:
Web page is very outdated.
π‘ Solution:
- Create new web page with introduction, modules overview, documentation, some explanations.
- We should use some static site generator like MkDocs or AsciiDoc or some other.
8. Logo
π₯ Problem:
Last but not least, asterisk-java doesn't have own logo π
π‘ Solution:
I've asked my good friend @tomekbbk, who is also a graphic and UX designer in company where I've working, to create logo.
This is what he designed for us:
BTW. Connect Java coffee cup with Asterisk logo = FTW π€©
How should we start?
In my opinion
- Create labels for issues and pull requests and start to use them
- Cleanup release cycle - use maven plugin and generate change log (GitHub Actions and GitHub Releases)
- Block master for direct pushes
- Cleanup stale branches
- Try to address and fix at least couple of issues
- Modularize project
- Add integration tests
- Web page
After that changes, version 4.0.0 should be #released!
If something needs more clarification let me know. I'll try to response.
Metadata
Metadata
Assignees
Labels
Type
Projects
Status