Skip to content
This repository has been archived by the owner on Nov 11, 2021. It is now read-only.

Deploying any container results in error "starting container with non-empty request body was deprecated since API v1.22 and removed in v1.24" #127

Closed
robcav862 opened this issue Aug 20, 2021 · 57 comments

Comments

@robcav862
Copy link

Problem/Motivation

Deploying any container results in error "starting container with non-empty request body was deprecated since API v1.22 and removed in v1.24"
image

(Why the issue was filed)

Expected behavior

Container deploys

(What you expected to happen)

Steps to reproduce

Deploy any container from the docker.io
nginxdemos/hello:latest
tutum/hello-world:latest

@LewisSpring
Copy link

Same issue here, just trying to deploy a container of an image I made with a dockerfile.

@frenck
Copy link
Member

frenck commented Aug 21, 2021

I will definitely take a look, however, please note, this add-on is mainly useful/meant for debugging purposes. Running additional containers on a Home Assistant OS or Supervised system is not supported by Home Assistant.

@pete-leese
Copy link

Also having the same issue here guys.

@DarkFox
Copy link

DarkFox commented Aug 22, 2021

I will definitely take a look, however, please note, this add-on is mainly useful/meant for debugging purposes. Running additional containers on a Home Assistant OS or Supervised system is not supported by Home Assistant.

This does also apply to containers installed via the add-on system. You can stop them, but can't start them again.

@frenck
Copy link
Member

frenck commented Aug 22, 2021

You can stop them, but can't start them again.

You should never do such a thing in a system with a Supervisor, so for that specific issue, it is actually good.

@Depechie
Copy link

All my own created portainer docker containers have the same issue... weird fact, for example pihole I got that fixed by using a portainer Stack instead of the UI

@ncresswell
Copy link

Neil here from Portainer. We will take a look at this from our side.. this only seems to impact HASS users tho, so something specific to that deployment.

@frenck
Copy link
Member

frenck commented Aug 23, 2021

All my own created portainer docker containers have the same issue.

That is not a supported scenario by Home Assistant and thus out of scope for this project and issue.

@timothymarq
Copy link

Same issue here. Hassio, just installed portainer add-on and same errors
Failure
starting container with non-empty request body was deprecated since API v1.22 and removed in v1.24

@frenck
Copy link
Member

frenck commented Aug 23, 2021

Please avoid posting "I have same issue" responses on GitHub, instead leave a 👍 on the initial issue description. Thanks 👍

@al3x1337
Copy link

al3x1337 commented Aug 23, 2021

Also having the same issue. Previously used with an older version of Hassio with no problem. Tried the EDGE version with the same error. Perhaps Hassio upgraded something in the new version that isn't compatible?

EDIT: I've locally installed v1.5.2 of this plugin and the error went away. Looks to be specific to v2.0 of the release.

@Fox350

This comment has been minimized.

@welchste
Copy link

Same issue as everyone above obviously. I don't use the Portainer addon to run containers on my HASSOS supervisor or anything. I use to to manage my docker containers on other systems. So when I tried to restart and re-pull an image to my surprise like many of you I can't start it.

@al3x1337

This comment has been minimized.

@Fox350

This comment has been minimized.

@ncresswell
Copy link

From what i can see, this v2 addon is mounting the docker socket connection as Read Only (RO).. as you can see by inspecting the mounts that Portainer uses when starting. This means Portainer is in "look, but dont touch" operating mode.. it cannot control Docker.
image

or if via SSH on HaaS.
image

@frenck
Copy link
Member

frenck commented Aug 27, 2021

@ncresswell

That has no effect on the capabilities, the socket still allows for writing. the ro/rw applies to the file descriptor, not the socket.

@oetken
Copy link

oetken commented Aug 31, 2021

Same here. Just installed and fails out of the box.

@luigiradano
Copy link

Had the same problem, however I tried pressing "Restart" instead of start and the container started. I don't know why it worked or if it will work for you, but try it.

@akrigator
Copy link

Faced with the issue on hassos with plex container. I've workaround it by creating the plex stack instead of a singleton container, here is the compose file for deployment:

version: "2.1"
services:
  plex:
    image: ghcr.io/linuxserver/plex
    container_name: plex
    network_mode: host
    environment:
      - DEBIAN_FRONTEND=noninteractive
      - GUID=0
      - HOME=/root
      - LANG=en_US.UTF-8
      - LANGUAGE=en_US.UTF-8
      - PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
      - PLEX_ARCH=arm64
      - PLEX_DOWNLOAD=https://downloads.plex.tv/plex-media-server-new
      - PLEX_MEDIA_SERVER_APPLICATION_SUPPORT_DIR=/config/Library/Application Support
      - PLEX_MEDIA_SERVER_HOME=/usr/lib/plexmediaserver
      - PLEX_MEDIA_SERVER_INFO_DEVICE=Docker Container (LinuxServer.io)
      - PLEX_MEDIA_SERVER_INFO_VENDOR=Docker
      - PLEX_MEDIA_SERVER_MAX_PLUGIN_PROCS=6
      - PLEX_MEDIA_SERVER_USER=abc
      - PUID=0
      - TERM=xterm
      - VERSION=latest
    volumes:
      - /mnt/data/docker/volumes/plexconfig:/config
      - ds_plexscanner:/config/Library/Application Support/Plex Media Server/Scanners
      - ds_photo:/mnt/photo
      - ds_video:/mnt/video
      - ds_surveillance:/mnt/surveillance
    restart: unless-stopped

Where the ds_* volumes are nfs shares. But a root cause is still unclear for me.

 Add-on version: 2.0.0
 You are running the latest version of this add-on.
 System: Home Assistant OS 6.2  (aarch64 / raspberrypi4-64)
 Home Assistant Core: 2021.8.8
 Home Assistant Supervisor: 2021.08.1

@EpicLPer
Copy link

EpicLPer commented Sep 3, 2021

I will definitely take a look, however, please note, this add-on is mainly useful/meant for debugging purposes. Running additional containers on a Home Assistant OS or Supervised system is not supported by Home Assistant.

I mainly use Portainer on HASS to manage other hosts, even there it seems to happen. It also seems to be unable to "Recreate containers", it seems to just load indefinitely on Deploy which may be related to this issue.

@ncresswell
Copy link

OK, i have continued to investigate...

I cloned the add-on repo to my machine, and built an image from it... i then started the image with the following command: docker run -d -p 8099:8099 -v /var/run/docker.sock:/var/run/docker.sock --entrypoint /opt/portainer/portainer b26bdd27337f -p :8099 which started fine, and i can run/start/stop containers fine.. so something in the HaaS init script is causing this issue.

Frenck, i have emailed you directly if you want to triage this together with Portainer.

@sunib
Copy link

sunib commented Sep 4, 2021

I will definitely take a look, however, please note, this add-on is mainly useful/meant for debugging purposes. Running additional containers on a Home Assistant OS or Supervised system is not supported by Home Assistant.

@frenck: Could you please add this statement in the readme of the plugin? I was not aware of this also ran into problems while update my OS to version 6.1 (I'm not able to run my 2 small containers anymore). The whole portainer GUI is so nice that it just screams to be used to run simple containers. :-)

@edgarfu721

This comment has been minimized.

@edgarfu721

This comment has been minimized.

@omayhemo
Copy link

omayhemo commented Sep 7, 2021

Same problem with mine. I updated today and now I can't start new containers.

@shupershuff
Copy link

Any updates on this? Super annoying that it doesn't work at all...

@drdownload
Copy link

Is this problem only on supervised install or also on stock hass?

@MaxWinterstein
Copy link

As mentioned by @Depechie a temporary workaround is to deploy another portainer via a stack.

All my own created portainer docker containers have the same issue... weird fact, for example pihole I got that fixed by using a portainer Stack instead of the UI

Just head towards Stacks, and use this one:

version: '2'

services:
  portainer:
    image: portainer/portainer-ce
    ports:
      - "9000:9000"
      - "8000:8000"
    volumes:
      - portainer_data2:/data
      - /var/run/docker.sock:/var/run/docker.sock
      - /var/lib/docker/volumes:/var/lib/docker/volumes

volumes:
  portainer_data2:

The new portainer, that lives beside the other one, is available at http://homeassistant.local:9000/

Now I have some new portainer2 running that works just as needed, and I can deploy something.

Caution: Take care!

@MaxWinterstein
Copy link

As this annoyed me way more than I thought, I cloned the version before the current one and integrated it to my Home Assistant Add-on Repository: https://github.com/MaxWinterstein/homeassistant-addons

Maybe this helps someone else too ✌️

@kristjanbjarni
Copy link

As this annoyed me way more than I thought, I cloned the version before the current one and integrated it to my Home Assistant Add-on Repository: https://github.com/MaxWinterstein/homeassistant-addons

Maybe this helps someone else too ✌️

This is great 👍 finally someone steps up to get at least a working version available. I actually think it's time that the community should have a secondary repository with all the official Home Assistant addons that are currently broken in their latest version. I would prefer to have the option of using a repository that always had the last well known working version, since there is no way to downgrade addons.

@bri-s
Copy link

bri-s commented Sep 27, 2021

As this annoyed me way more than I thought, I cloned the version before the current one and integrated it to my Home Assistant Add-on Repository: https://github.com/MaxWinterstein/homeassistant-addons

Maybe this helps someone else too ✌️

this is perfect. thanks for the workaround

@mynameisvaro
Copy link

As this annoyed me way more than I thought, I cloned the version before the current one and integrated it to my Home Assistant Add-on Repository: https://github.com/MaxWinterstein/homeassistant-addons

Maybe this helps someone else too ✌️

Yap, totally working. Thanks a lot mate 😉

@noname9312
Copy link

Workaround

  1. Install SSH Addon
  2. Install latest portainer via SSH
docker volume create portainer_data_new

docker run -d -p 8002:8000 -p 9002:9000 --name=portainer --restart=unless-stopped -v /var/run/docker.sock:/var/run/docker.sock -v portainer_data_new:/data portainer/portainer-ce
  1. Use following path for volumes
    /mnt/data/supervisor/share/<your folder>

@shupershuff
Copy link

So... is this going to get fixed or just left in a broken state?

@EpicLPer
Copy link

So... is this going to get fixed or just left in a broken state?

DISCLAIMER: THE FOLLOWING SOLUTION IS UNSUPPORTED AND EVEN DISCOURAGED BY THE HASS TEAM DUE TO EXTERNAL REPOS BEING USED, PLEASE CONTINUE WITH CAUTION

What I did was using the following Repository by @alexbelgium for Portainer: https://github.com/alexbelgium/hassio-addons
Simply add it via the Repository option in the Add-on Store, go to your currently broken official Portainer install and export the Settings there, install Alex's Portainer fork, go into Config and delete the Password to see the initial setup screen and import your Backup from there. Now you've got your old Settings and Login working again.

@chriscrowe
Copy link

chriscrowe commented Oct 26, 2021

Its kind of nuts that this is just being ignored... I haven't been able to update/restart my containers in months.

@frenck
Copy link
Member

frenck commented Oct 26, 2021

Its kind of nuts that this is just being ignored... I haven't been able to update/restart my containers in months.

Its not ignored. I've been on holiday and in the hospital for surgery (can't plan for that shit) and still working my way through over a month of stuff to catch up on. My current list of GitHub issues and PR to work on/look at is (at the time of writing) 492.

As for this add-on, running custom containers IS NOT SUPPORTED by Home Assistant. This tool is never meant to be used for that either (as well, not supported).

The goal of this tool is to overview and access the docker eco system that runs your Home Assistant system; for the purpose of debugging. That goal works and isn't flawed at this point.

As also notes here on 24 Aug: #127 (comment)

@chriscrowe
Copy link

Gotcha, sorry I just hadn't seen any indication here that there was a plan to fix this, almost seemed like it was being binned into "not to be fixed". In particular this comment seemed to indicate that we're all doing something that you don't want to support:

That is not a supported scenario by Home Assistant and thus out of scope for this project and issue.

Hope you have a speedy recovery from surgery 👍

@frenck
Copy link
Member

frenck commented Oct 26, 2021

indicate that we're all doing something that you don't want to support

It's not about wanting to support; It is simply not supported by the Home Assistant project.

@chriscrowe
Copy link

Understood. So just to be clear -- are we expecting a fix for this or should we proceed under the assumption that it's not supported and thus may not get a fix?

@noname9312
Copy link

It would be nice if there would an official support, since not everybody has multiple Pi's or servers.

@frenck
Copy link
Member

frenck commented Oct 26, 2021

are we expecting a fix for this or should we proceed under the assumption that it's not supported and thus may not get a fix?

At this point, the Supervisor will get more checks towards if systems are in a supported state. It is possible to the add-on will not be able to be fixed or keep fixed in the future.

Again; The goal of this add-on (why I originally made it) is debugging and development. Not running extra containers on your Home Assistant system.

It would be nice if there would an official support, since not everybody has multiple Pi's or servers.

That is not up to the add-on, but up to the Home Assistant project. Its the amount of issues in the past that made it officially unsupported from upstream.

@Depechie
Copy link

Hey @frenck the comment about not allowing extra containers on a home assistant system, can that be detailed? As in what version of HA? Core, Supervised, OS?

Because the main problem we as users face ( the root of the cause ) is the fact that the team still allows own linux installs with manual install of Home Assistant Supervised. Small reminder, that was once pushed back by the team but allowed back after a lot of user complaints on that rule.

So even though the add-on has this small remark that it is not allowed, the tool is just so easy to use for the other containers we do install on our own Linux installs.

To be honest, I just installed it outside HA now and that works too. So the 'need' is less now, but the message is a bit mixed now.

And I get the other side too, allowing to much custom on an install has impact on what the team allows to spend time for logged issues because the environment is too large to grasp the possible bugs.
But my view, I would think HA became big because you allowed a lot a free roaming on installs.

My 2 cents...

@frenck
Copy link
Member

frenck commented Oct 26, 2021

Because the main problem we as users face ( the root of the cause ) is the fact that the team still allows own linux installs with manual install of Home Assistant Supervised.

@Depechie Yes, but without additional software.

This has been put into Architectural decision, which has been discussed, announced and approved by a lot of core developers:

image

https://github.com/home-assistant/architecture/blob/master/adr/0014-home-assistant-supervised.md

This architectural decision is in place for over a year now.

However, discussion on this is quite out of place for this issue.

@omayhemo
Copy link

I respect and thank Frenck for all the work he does for this community, but it appears there's not a commitment to making this add-on work like it did prior to 2.0, so I've moved to the unofficial add-on at this point. Hopefully it won't stop working at some point.

@frenck
Copy link
Member

frenck commented Oct 26, 2021

@omayhemo Do you even read?

@omayhemo
Copy link

I do, and I noticed a lot of word play that I eventually interrupted as you couldn't make guarantees. Perhaps I read and I don't comprehend, but that maybe due to my own ignorance and maybe due to your usage of a word storm to avoid a yes or no answer. Maybe you see it as more complex than that.

@noname9312
Copy link

noname9312 commented Oct 26, 2021

Short answer, no it will not be supported.
Use a workaround if you want to use it. But then you are out of support in case you HA behaves strange

@frenck
Copy link
Member

frenck commented Oct 26, 2021

as you couldn't make guarantees

No, not until I've sit down and looked at it.

@omayhemo
Copy link

That's what I interrupted, so I guess I do read and even comprehend to some level. Hope you're well.

@frenck
Copy link
Member

frenck commented Oct 26, 2021

That's what I interrupted, so I guess I do read and even comprehend to some level. Hope you're well.

No, you don't.

@Depechie
Copy link

This architectural decision is in place for over a year now.

However, discussion on this is quite out of place for this issue.

Ok, thx for pointing that out.
This was not known to me...

I guess we as users don't follow all changes as long as something works of course.
But I understand the need/reason, although I don't like it ;)

@omayhemo
Copy link

What is it you think I have misinterpreted? Please do clear it up, because I doubt I'm only one having trouble with the commentary. You have not committed to a fix (as I originally state when you asked me if 'I even read'), or even if a fix is possible because you have yet to get a chance to look at it, due to your schedule, yes?

@frenck
Copy link
Member

frenck commented Oct 26, 2021

But I understand the need/reason, although I don't like it ;)

Yeah, I understand why. But honestly, if you want that level of control, that using just plain Docker with the Home Assistant Container installation type is really a better way to go.

@frenck
Copy link
Member

frenck commented Oct 26, 2021

You have not committed to a fix

Goddammit son, I've explained above why I haven't looked at it, yet. I'm sorry, I did not choose this either. I'm not making promises, without looking, as I cannot hold up to it at that point. That is how it works. Furthermore, this is an open source project I do in my spare time; there is no schedule.

If you don't like that concept, and want promises, go somewhere else.

@hassio-addons hassio-addons locked and limited conversation to collaborators Oct 26, 2021
@frenck
Copy link
Member

frenck commented Nov 11, 2021

This add-on is being discontinued.

@frenck frenck closed this as completed Nov 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests