-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
[🐛 Bug]: Videos no longer recorded #2156
Comments
@MJB222398, thank you for creating this issue. We will troubleshoot it as soon as we can. Info for maintainersTriage this issue by using labels.
If information is missing, add a helpful comment and then
If the issue is a question, add the
If the issue is valid but there is no time to troubleshoot it, consider adding the
If the issue requires changes or fixes from an external project (e.g., ChromeDriver, GeckoDriver, MSEdgeDriver, W3C),
add the applicable
After troubleshooting the issue, please add the Thank you! |
@MJB222398, before launching containers, can you update steps as below mkdir -p ${VIDEOS_DIRECTORY:-./videos}
sudo chown 1200:1201 ${VIDEOS_DIRECTORY:-./videos}
docker compose up -d This is needed after #2122 |
@VietND96 thanks that does seem to have worked locally for me. Getting that to work on the CICD machine might be more of an issue... But why does #2122 say that it's not a breaking change? If that hasn't broken my process then what has? Also why is having to run the above command before spinning up containers a good thing? Maybe I am missing something but it worked before with less effort. |
It looks like an issue is with linux, if the folder is not present, it will be created by root, and before the container runs as root by default, so the issue could not be seen. Line 402 in c323bed
Can you check is this help in your CICD? |
In case you don't worry about rootless for this container, I think you can specify |
@VietND96 I cannot write
So I wrote If I instead use
but if I do an |
echo UID=$UID > .env
docker compose up -d So, which way is suitable for you? |
@VietND96 Thanks for your help, I didn't realise that not all environment variables are made available automatically (my VIDEOS_DIRECTORY one is for instance). All working now using UID locally and on my CICD machine. |
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
What happened?
I use a selenium grid with 3 browser nodes (chrome, edge, and firefox) and have a video container for each node to record them. I spin these all up using a docker compose. All was working fine with version 4.16.1-20231219 and ffmpeg-6.1-20231219. But now I upgraded to 4.18.0-20240220 and ffmpeg-6.1-20240220 and now videos are no longer recorded.
Command used to start Selenium Grid with Docker (or Kubernetes)
Relevant log output
Operating System
Ubuntu 20.04
Docker Selenium version (image tag)
4.18.0-20240220
Selenium Grid chart version (chart version)
No response
The text was updated successfully, but these errors were encountered: