Charlie Drage
Charlie Drage
> @cdrage you're right, and it's already done in the existing code when editing a podman machine: > > ``` > if (effective) { > const state = podmanMachinesStatuses.get(machineInfo.name); >...
**Please feel free to dismiss my review** after questions answered as I am on PTO this upcoming week.
1.21.0 is out now. Are we good to merging this in? cc @jeffmaury 3 LGTM's + QE can test now after this.
https://github.com/user-attachments/assets/6ad452be-d03d-42c9-aab6-c0eeade34fa6 I'm experiencing this issue as well with a general lagginess when trying to delete / restart / etc.
@benoitf @deboer-tim @cbr7 I did extensive testing on Windows to get this to work with a bunch of different build combinations, and found this to be the one that covers...
> Hello @cdrage! I've just finished testing the PR on windows. These are my results: > > * Running the installer with /ALLUSERS (`./podman-desktop-1.20.0-next-setup-x64.exe /s /ALLUSERS`) installs PD on C:/Program...
> @cdrage merge ? Sure. Tests pass, e2e tested it, three other approvals. Hopefully in the future we also have the e2e test. Thanks for reviewing!
Silly question, but with this PR, unsure if this should be done within this PR or as a follow-up, but what about remembering sections in extensions that use SubmenuNavigation imported?...
> ### Bug description > Location for default managed config `/usr/share/podman-desktop/default-settings.json` does nothing in PD when it is installed from flatpak. > > ### Operating system > Linux > >...
Investigated this and ended up building it with the added change of `--filesystem=/usr/share/podman-desktop:ro` (after also updating electron since we updated to 1.24.2 with it)... which unfortunatley also failed. The reason...