Nice, thanks for taking a look! I know in your try you weren't looking at at linux but I noticed an issue previously when I was looking at linux before... for some reason in my own linux machine that xrandr command works, but through pythons subprocess it seems to not properly remove the asterik in that sed command e.g. refresh rate in CI would get returned as 59.87* instead of 59.87 https://firefox-ci-tc.services.mozilla.com/tasks/bPVH9K24QG6rCSIVP9zbfA/runs/0/logs/public/logs/live.log#L975 not a big deal - just wanted to point that out since I noticed in your [diff](https://hg.mozilla.org/try/rev/49190c2d43b346b819b97f694d28be543e6a83e6) that it looks for a comparison for 60 in `if str(refresh_rate) != "60":` which wouldn't be satisfied for the current linux by the way, did you encounter any monitors that outputted an empty field instead of 24hz or 60hz? that was another case I got. Like this https://firefox-ci-tc.services.mozilla.com/tasks/JpGVERVIRDKmJh8Fd-18Kw/runs/0/logs/public/logs/live.log#L880
Bug 1917594 Comment 14 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
Nice, thanks for taking a look! I know in your try you weren't looking at linux but I noticed an issue previously when I was looking at linux before... for some reason in my own linux machine that xrandr command works, but through pythons subprocess it seems to not properly remove the asterik in that sed command e.g. refresh rate in CI would get returned as 59.87* instead of 59.87 https://firefox-ci-tc.services.mozilla.com/tasks/bPVH9K24QG6rCSIVP9zbfA/runs/0/logs/public/logs/live.log#L975 not a big deal - just wanted to point that out since I noticed in your [diff](https://hg.mozilla.org/try/rev/49190c2d43b346b819b97f694d28be543e6a83e6) that it looks for a comparison for 60 in `if str(refresh_rate) != "60":` which wouldn't be satisfied for the current linux by the way, did you encounter any monitors that outputted an empty field instead of 24hz or 60hz? that was another case I got. Like this https://firefox-ci-tc.services.mozilla.com/tasks/JpGVERVIRDKmJh8Fd-18Kw/runs/0/logs/public/logs/live.log#L880
Nice, thanks for taking a look! I know in your try you weren't looking at linux but I noticed an issue previously when I was looking at linux before... for some reason in my own linux machine that xrandr command works, but through pythons subprocess it seems to not properly remove the asterik in that sed command e.g. refresh rate in CI would get returned as 59.87* instead of 59.87 https://firefox-ci-tc.services.mozilla.com/tasks/bPVH9K24QG6rCSIVP9zbfA/runs/0/logs/public/logs/live.log#L975 not a big deal - just wanted to point that out since I noticed in your [diff](https://hg.mozilla.org/try/rev/49190c2d43b346b819b97f694d28be543e6a83e6) that it looks for a comparison for 60 in `if str(refresh_rate) != "60":` which wouldn't be satisfied for the current linux by the way, did you encounter any tasks that outputted an empty field instead of 24hz or 60hz? that was another case I got. Like this https://firefox-ci-tc.services.mozilla.com/tasks/JpGVERVIRDKmJh8Fd-18Kw/runs/0/logs/public/logs/live.log#L880