-
Notifications
You must be signed in to change notification settings - Fork 20
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
Latest Image Broken #130
Comments
I've been trying to upgrade the dependencies lately. This might have broken some stuff, but it also would reveal that we don't have enough testing. Perhaps some end-to-end testing is in order. I'll try to straighten it out in the next day or so. |
Got ya, no worries. Was able to get my feeders back up and working on the old image for now. But +1 on the testing for sure. Appreciate it! Sidenote: Is there anyway to grab a previous release of the package? https://github.com/feedernet/petnet-feeder-service/pkgs/container/petnet-feeder-service, it looks like only tagged releases here are |
Since it is breaking the docker container, take out the build path value until we can fix the templating engine to work with all of the javascript files. Possibly related to issue #130
Since it is breaking the docker container, take out the build path value until we can fix the templating engine to work with all of the javascript files. Possibly related to issue #130
I noticed that a new setup (without an existing /data directory) caused 422 errors when adding a pet. Switching the Content-Type header on the /api/v1/pet POST request from text/plain to application/json resolved this for me, but it seems like all requests are text/plain and cause the same 422 error: {"detail":[{"loc":["body"],"msg":"value` is not a valid dict","type":"type_error.dict"}]} |
Yep, was able to hack together some quick js to get my pets created, feeders assigned, and schedules set. What's strange is I'm able to delete pets and scheduled events through the ui, but anything to do with POST or PUT methods seem to be failing |
I saw that the I tried to manually register the feeder and it seems to register, but says:
and the feeder never connects. Clicking on "Edit Feeder" also presents me with a blank page and react error. |
@ky13d3nt0n any chance you stashed your js workarounds in a form you can share? (Or, have you been able to identify a working image, suitable for a Pi 4?) |
I created a release and tagged it as '1.0.0' which should be stable. It's before I started upgrading all the backend stuff and realized our existing tests aren't covering everything. |
Thanks @kruton ! This is an awesome project. Pet & feeder setup worked with the 1.0.0 image, and the manual feed is working. Logs around successful manual feed:
Logs around scheduled feed post:
There did seem to be a correlated flurry of asyncio exceptions around the time of the scheduled feed the first few times I was testing (same as mentioned in #61 (comment)) but it has not been consistent. Any troubleshooting advice? |
@zoeglynn I was able to get things somewhat working with the form solution, but not everything. I ended up having to dig out one of my old sd cards with an image from maybe 8-10 months ago that is stable. |
thanks for the note @ky13d3nt0n. Any chance the image holds any clues about the revision it was built from? Or even the timestamp from |
I think I've fixed all the problems from the latest Bootstrap 5 upgrade with #206. I am still working on some end-to-end tests that test the Node.js frontend together with the Python backend to make sure we don't run into the same problems. |
Had to reset my database and found that the latest image seems to be having troubles finding new feeders.
Was able to find an old image on Docker: https://hub.docker.com/r/tedder42/petnet-feeder-service and things are back up and running.
Checked error logs before and nothing was outputting as an issue, feeders were just never found.
The text was updated successfully, but these errors were encountered: