-
Notifications
You must be signed in to change notification settings - Fork 432
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Track skipping due to upstream Spotify API issues. #669
Comments
I can see in we are using image @andrewn is it possible for you to rereate docker image of librespot from dev branch ? |
I opened andrewn/librespot-docker#2 which I think should allow a new image to be created from dev branch but I don't know how to validate. |
Because I am a masochist who doesn't know when to quite whilst he's ahead and I already possess a docker hub account I thought I'd try building the image myself and ran into a few issues: Modified # not sure if this should align with the dev branch build but it was unique so 🤷
# version = "0.5.0-dev" is dev branch version in cargo.toml
LIBRESPOT_VERSION="0.4.2-dev"
DOCKER_NAMESPACE="brettmillerit" exec: => WARN: FromAsCasing: 'as' and 'FROM' keywords' casing do not match (line 4) 0.1s
=> ERROR [internal] load metadata for docker.io/balenalib/rpi-alpine:edge 0.9s
=> CANCELED [internal] load metadata for docker.io/balenalib/rpi-alpine:3.14-run
ERROR: failed to solve: balenalib/rpi-alpine:edge: failed to resolve source metadata for docker.io/balenalib/rpi-alpine:edge: error getting credentials - err: exit status 255, out: `` I then tried rebuilding using > [librespot-builder 6/6] RUN cargo build --release --features alsa-backend,pulseaudio-backend --verbose:
0.997 error: failed to parse manifest at `/app/librespot-src/Cargo.toml`
0.998
0.998 Caused by:
0.999 feature `edition2021` is required
0.999
0.999 this Cargo does not support nightly features, but if you
0.999 switch to nightly channel you can add
0.999 `cargo-features = ["edition2021"]` to enable this feature
------
Dockerfile.pulseaudio.template:20
--------------------
19 |
20 | >>> RUN cargo build \
21 | >>> --release \
22 | >>> --features alsa-backend,pulseaudio-backend \
23 | >>> --verbose
24 |
--------------------
ERROR: failed to solve: process "/bin/sh -c cargo build --release --features alsa-backend,pulseaudio-backend --verbose" did not complete successfully: exit code: 101 I moved back to |
So I spent more time than I care to admit figuring this out. Was able to get the docker image built using @andrewn repo, pushed to my own docker hub repo for testing. Or at least I was able to get 2/4 images built. I was using docker in docker on my Windows machine to do this so not sure why it took so long but left it over night and it still hadn't finished the remaining two images. Switched out my BalenaSound project to use the image from my docker hub repo: Built a docker container for that project so I could install Balena CLI as I have no admin rights on this windows machine 🤦 Confirmed a push was using the correct image: [airplay] Step 1/10 : FROM mikebrady/shairport-sync:4.1.1 as shairport
[unbound] Step 1/4 : FROM klutchell/unbound:v1.19.1@sha256:125cd0c03ea2e5386aec2d9e162de00d6c59b20ee2adf4529818ee025b59e79d
[sound-supervisor] ---> 1b4441c7cf2c
[sound-supervisor] Step 2/15 : WORKDIR /usr/src
[spotify] Step 1/4 : FROM brettmillerit/librespot:0.4.2-dev-pulseaudio-rpi
[sound-supervisor] ---> Running in f021eb48c5d9
[sound-supervisor] Removing intermediate container f021eb48c5d9 |
Went back to check the original issue and seems Spotify have reverted their API version changes librespot-org/librespot#1322 (comment) However, when Spotify decide to make those changes Balena will need to utilise the new Guess I learned a tonne about the project 🤣 |
Also looks like they're pushing for a 0.5.0 release: |
Hi @brettmillerb, sorry for the delay. I can try and do a new build of the repo in the next couple of days. I guess 0.5.0 is backwards compatible with the old API too? How should I test it? |
Hi @andrewn
I am way past the realms of my existing knowledge as it is. I have noticed with the May be worth hanging fire for an official release of 0.5.0 before rebuilding the docker images since Spotify rolled back the changes their end 🤷♂️ |
Hi there, currently, i am again getting a lot of:
As suggested in docs, there is no actual solution, only to upgrade. There is already new librespot release v0.5.0 avalible. i forked balenasound and updated tag for docker image and also andrewn librespot, where i bumed tag versions to 0.5.0 Bui i am unable to build docker image with github actions, there is some weird cargo build error. |
It took a bit of fiddling around, but I have a version that seems to build v0.5.0 here: andrewn/librespot-docker#3 The GitHub actions have never worked, so I'm building them manually on my laptop. The only thing is that I don't use balena-sound anymore so once they're up, someone's going to need to run them on a Pi to make sure they work. I'll post here once they're on dockerhub. |
Cool, i have pi4 and pi3 at my disposal, so i can try to build it on device :) |
I can give it a go on a pi2 |
I left my laptop cross-compiling all the binaries and went out for the day. I meant it would be great to check the compiled binaries work with balena-sound once they’re uploaded to docker. |
@mivittek @wheredidthecostgo The new v0.5.0 images are now published to docker hub as Please test them out and let me know how they work for you |
Well, i managed to build it and pull form docker repo, thanks for that. But there is error while running build. Some manifest missing ?
i tried to debug it, but for main packge - without platform there is no manifest
When i edit it manually as you cen see it in commit, it works like a charm. |
@mivittek Thanks for reporting that! I forgot to enable the "create manifest" line in the script. 🤦 If you try again it should work. |
I've built and deployed! I'll test and see if it corrects the (os error 104) |
I redeployed image and since commit/f954c98 I am running 4+ hours and no error yet, so I got high hopes :) Anyway, this runs at my home eveyday, so i'll get back in few days. Thanks @andrewn 🥇 |
@andrewn It has worked perfect for me all day! Thank you!!! |
Its still working like a charm, should we create PR with update + bumped release version ? |
Hi there ! Log:
This issue is documented in librespot , without any real solution :( reference: |
Thanks!
|
Describe the problem
When connecting to Balena raspberryPi device the tracks skip continuously.
To Reproduce
Open Spotify and attempt to connect to Balena Airplay device.
Tracks just begin skipping, some tracks play but the majority do not.
Expected behavior
Track selected plays on Balena device
Screenshots
Additional Errors:
Device information
Additional context
Seems a Spotify API change is the cause and recommendation is to build librespot from dev branch which does not have the issue.
librespot-org/librespot#1322
The text was updated successfully, but these errors were encountered: