-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
CI flakes with "no such service: web" #1415
Comments
This is plaguing the 22.5.0 release: |
Okay, went down the 🐰 🕳️ on #1423 #1424 ... afaict this is due to no change we introduced either in @aminvakil (or anyone else) Can you repro this locally? I tried but I'm on a new laptop and am running afoul of #914. 😩 |
@chadwhitacre Sorry for late reply, I will test this on a Debian 10 Buster in the next hours and report back. |
On Debian 10 Buster:
Using docker-compose 1.29.2, I couldn't reproduce it on 22.1.0, 22.2.0, 22.3.0, 22.4.0 . I ran And tested it using |
Dang, the plot thickens! Something really specific to GitHub Actions environment? 🤔 |
And why it started happening to my fork a couple days before it started to happen on main repo? |
This issue just felt oddly familiar datastack-net/dockerized#39 . The way to reproduce it locally might be to run the script without a TTY (e.g. I really don't know if this is related, but thought to let you know just in case. |
Thanks for sharing, @boukeversteegh! |
This issue has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you label it "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
We have removed docker-compose 1.x versions and I have not seen this on docker-compose 2.x versions ever, seems like it has been fixed by removing the problem :) As no one has reported this outside of CI, I will close this. |
We stopped testing docker-compose v1 in CI precisely because of this problem. Are we really ready to drop docker-compose v1 support overall? |
self-hosted/.github/workflows/test.yml Lines 35 to 39 in 518a9ec
|
I don't think we can do so, for one ansible docker-compose plugin only supports < 2 , and many other users are using v1.x . But about this issue, I would say opening it when we are testing this inside CI or users experience it, AFAICT this only happens in GHA and there is no bandwidth to fix this, is there? |
@aminvakil So I guess we should bring this back in CI and find a workaround for GHA? |
We will not drop support for docker-compose v1, but it's eol in a couple of months anyway https://www.docker.com/blog/announcing-compose-v2-general-availability/
Now that I read my sentence again, I get that it can be interpreted a different way. I meant let's just open this issue if and only if users report that they are experiencing it inside their environment outside of GHA. And we should not use docker-compose v1 anymore anyway and encourage (not mandatory though) users to use docker-compose v2. We can make it mandatory on October 2022 when there will be no security fixes for docker-compose v1 then. |
Docker compose v1 support is dropped completely now, seems safe to close this |
It's been a couple of days I'm seeing this regression on docker-compose 1.28.0 and 1.29.2 on my fork:
https://github.com/aminvakil/self-hosted/actions/runs/2068250465
https://github.com/aminvakil/self-hosted/actions/runs/2074457629
https://github.com/aminvakil/self-hosted/actions/runs/2080468698
https://github.com/aminvakil/self-hosted/actions/runs/2083753352
This is a clean installation of course :)
Here is the error:
This started to happen since last nightly run on https://github.com/getsentry/self-hosted CI too:
https://github.com/getsentry/self-hosted/runs/5843062307?check_suite_focus=true
https://github.com/getsentry/self-hosted/runs/5843062379?check_suite_focus=true
The text was updated successfully, but these errors were encountered: