Skip to content

App Submission: Music Assistant#4528

Open
phelipebf wants to merge 22 commits intogetumbrel:masterfrom
phelipebf:music-assistant
Open

App Submission: Music Assistant#4528
phelipebf wants to merge 22 commits intogetumbrel:masterfrom
phelipebf:music-assistant

Conversation

@phelipebf
Copy link
Contributor

App Submission

App name

Music Assistant

Icon

https://www.music-assistant.io/assets/transparent-logo.png

Gallery images

Captura de tela 2026-01-21 131843

I have tested my app on:

  • umbrelOS on a Raspberry Pi
  • umbrelOS on an Umbrel Home
  • umbrelOS on Linux VM

Copy link
Collaborator

@al-lac al-lac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @phelipebf,

thanks a lot for contributing the music-assistant app to the Umbrel App Store. 🥇

Really good job on the submission! I added some minor things that need to be adapted.

One issue I still see is that the app is running the UI on Port 8095 which is the one that adguard-home uses. I could not find a way to change this port as of now.

@al-lac al-lac added the awaiting changes Waiting for contributor to make requested changes label Jan 23, 2026
phelipebf and others added 7 commits January 24, 2026 22:05
Co-authored-by: Alexander L. <lackner.alex@gmail.com>
Co-authored-by: Alexander L. <lackner.alex@gmail.com>
Co-authored-by: Alexander L. <lackner.alex@gmail.com>
Co-authored-by: Alexander L. <lackner.alex@gmail.com>
Co-authored-by: Alexander L. <lackner.alex@gmail.com>
Co-authored-by: Alexander L. <lackner.alex@gmail.com>
Co-authored-by: Alexander L. <lackner.alex@gmail.com>
@phelipebf phelipebf requested a review from al-lac January 25, 2026 01:08
@al-lac
Copy link
Collaborator

al-lac commented Jan 27, 2026

Hey @phelipebf, I think I found away around the port clash issue. We could add the following file in the /data directory to set the port to 8896 for example:

{
  "server_id": "ea22c850e3e74462bd24515d860c67bf",
  "providers": {
    "builtin": {
      "values": {},
      "type": "music",
      "domain": "builtin",
      "instance_id": "builtin",
      "enabled": true,
      "name": "Music Assistant",
      "last_error": null
    },
    "fanarttv": {
      "values": {},
      "type": "metadata",
      "domain": "fanarttv",
      "instance_id": "fanarttv",
      "enabled": true,
      "name": "fanart.tv",
      "last_error": null
    },
    "lrclib": {
      "values": {},
      "type": "metadata",
      "domain": "lrclib",
      "instance_id": "lrclib",
      "enabled": true,
      "name": "LRCLIB",
      "last_error": null
    },
    "musicbrainz": {
      "values": {},
      "type": "metadata",
      "domain": "musicbrainz",
      "instance_id": "musicbrainz",
      "enabled": true,
      "name": "MusicBrainz",
      "last_error": null
    },
    "sendspin": {
      "values": {},
      "type": "player",
      "domain": "sendspin",
      "instance_id": "sendspin",
      "enabled": true,
      "name": "Sendspin",
      "last_error": null
    },
    "theaudiodb": {
      "values": {},
      "type": "metadata",
      "domain": "theaudiodb",
      "instance_id": "theaudiodb",
      "enabled": true,
      "name": "The Audio DB",
      "last_error": null
    }
  },
  "core": {
    "metadata": {
      "values": {
        "language": "en_US"
      },
      "domain": "metadata",
      "last_error": null
    },
    "webserver": {
      "values": {
        "base_url": "http://umbrel.local:8896",
        "bind_port": 8896
      },
      "domain": "webserver",
      "last_error": null
    }
  }
}

Not sure how important the server-id part is for music assistant.

@phelipebf
Copy link
Contributor Author

Hey @phelipebf, I think I found away around the port clash issue. We could add the following file in the /data directory to set the port to 8896 for example:

{...}

Not sure how important the server-id part is for music assistant.

Sounds good. I will test and commit.

@phelipebf
Copy link
Contributor Author

@al-lac I've tested and everything seems good. Follow the updates.

Copy link
Collaborator

@al-lac al-lac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @phelipebf,

thanks for applying and testing the suggested changes 🔝

I have some minor ones to add, we will already start with creating the gallery assets 🖌️

@al-lac al-lac added the awaiting gallery assets Umbrel team to make gallery assets label Jan 28, 2026
@github-actions
Copy link

🎉   Linting finished with no errors or warnings   🎉

Thank you for your submission! This is an automated linter that checks for common issues in pull requests to the Umbrel App Store.

Please review the linting results below and make any necessary changes to your submission.

Linting Results

Severity File Description
ℹ️ music-assistant/docker-compose.yml Service "web" uses host network mode:
The host network mode can lead to security vulnerabilities. If possible please use the default bridge network mode and expose the necessary ports.

Legend

Symbol Description
Error: This must be resolved before this PR can be merged.
⚠️ Warning: This is highly encouraged to be resolved, but is not strictly mandatory.
ℹ️ Info: This is just for your information.

@phelipebf phelipebf requested a review from al-lac January 28, 2026 18:51
@phelipebf
Copy link
Contributor Author

@al-lac done!

@al-lac al-lac removed the awaiting changes Waiting for contributor to make requested changes label Jan 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting gallery assets Umbrel team to make gallery assets

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants