-
-
Notifications
You must be signed in to change notification settings - Fork 8
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
Running DDEV GitHub Action with PHP 8.3 #30
Comments
That's a result of having an old DDEV version most likely. They let their key expire: It can also be a result of internet trouble. |
Oh, I see - it has the old key. The new key has to be installed. |
But why are the deb.sury.org versions being installed on Ubuntu? No PHP installations are required to use ddev... |
I cannot find where this is done in the GHA.
I cannot find where this is done in the GHA. Have searched for ondrej no matches. |
It's not a nice nor the right fix, but small shell line, could "fix" it sudo add-apt-repository --remove ppa:ondrej/php If it's not needed, it can be assured it's not present, but I would rather figure out where it's coming from. |
I don't know how a PPA is being used anywhere for any reason. and don't find "ondrej" anywhere. I can't understand why a PPA would be installed. I do see that tests are being run with DDEV v1.22.3, which would have the expired key, but that would be a debian error and now show an Ubuntu PPA as being involved. |
It only happens in my TYPO3 Crawler workflow with PHP 8.3, not with 8.1 and 8.2. I don't see where it is coming from either. Perhaps @jonaseberle who started this action can point us into the direction? |
I think the error has nothing to do with this action, it comes from the jobs:
Acceptance:
runs-on: ubuntu-22.04 This could either be a temporary issue or something wrong with The only "problem" is that this action runs |
The fix could be the same as we have in the main
github-action-setup-ddev/lib/main.js Line 75 in 9dc2229
-cmd = `sudo apt-get update && sudo apt-get install -y ${ddevPackage} && mkcert -install`;
+cmd = `(sudo apt-get update || true) && sudo apt-get install -y ${ddevPackage} && mkcert -install`; |
I can provide a PR next week somewhen if you want. |
Here's where the problem is coming from, it's in the Ubuntu 20.04 runner: Why don't we just stop using Ubuntu 20.04? I don't think it offers any value going forward. It does look like they have failed to maintain that I guess, not sure. Mostly we should probably be testing this on Ubuntu 22.04 and upcoming 24.04 IMO. So just remove 20.04 from the matrix? |
I would say, skipping the 20.04 LTS sounds like a good idea and focus on to two latest LTS releases of Ubuntu. But on the other side, Ubuntu 20.04 has support till April 2025, so many will not update unless they are "forced" to. But your call. I can still create the PR somewhen next week if wanted. |
In general, Ubuntu is so very solid that it rarely makes any difference what version you use. My opinion would be to just test on ubuntu-latest and let it be. |
And |
|
Hmm, I am not sure about I can't reproduce in the tests from #32 |
This is an example of why we need See how many repos
In this case |
Ok. Let's get that in :) |
I can confirm that the issue is fixed downstream in the TYPO3 Crawler. |
When I ran the DDEV GitHub Action with PHP 8.3 I got the following error
https://github.com/tomasnorre/crawler/actions/runs/8900718640/job/24442912503#step:7:90
I know it was already mentioned in #27, but I think it's right to split them up as they are not related, just present at the same information in console.
Will see if I can find a way to reproduce this and perhaps come up with a fix.
The text was updated successfully, but these errors were encountered: