Skip to content
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

fix: make dev desktops compatible with ubuntu 24 #562

Merged
merged 1 commit into from
Sep 20, 2024

Conversation

marcoieni
Copy link
Member

@marcoieni marcoieni commented Sep 4, 2024

We can only merge this after we migrated dev-desktops to ubuntu 24.

In the following an explanation of the changes.

libasound

libasound2 doesn't exist anymore in ubuntu 24

fix_llvm_55575.yml

Run the following on both ubuntu 22 and 24:

docker run -it ubuntu:22.04 bash   # run the same on 24.04
apt update
apt install lldb
lldb --version

As you can see the error mentioned in the task (llvm/llvm-project#55575) is not happening anymore in ubuntu 24

@marcoieni marcoieni force-pushed the ubuntu-24-dev-desktop branch 2 times, most recently from b0b0daf to eff248b Compare September 4, 2024 14:48
@marcoieni
Copy link
Member Author

ansible ran without issues 👍

dev-desktop-staging.infra.rust-lang.org : ok=189  changed=13   unreachable=0    failed=0    skipped=92   rescued=0    ignored=1

@@ -1,5 +1,7 @@
[storage]
driver = "overlay"
runroot = "/run/containers/storage"
graphroot = "/var/lib/containers/storage"
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these two fields are now required. Otherwise the service podman-auto-update.timer fails

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do these work/interfere with the rootless mode that we're using?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure but nobody complained until now 😂

- name: Uninstall postfix
apt:
name: postfix
state: absent
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I removed postfix because there was an alert related to it.
Instead of fixing it I decided to remove this package as we don't need it imo.

@marcoieni marcoieni marked this pull request as ready for review September 20, 2024 07:37
@marcoieni marcoieni force-pushed the ubuntu-24-dev-desktop branch from ce206e6 to 2fe5231 Compare September 20, 2024 07:39
@marcoieni marcoieni merged commit 30294e3 into master Sep 20, 2024
3 checks passed
@marcoieni marcoieni deleted the ubuntu-24-dev-desktop branch September 20, 2024 07:48
@jdno jdno mentioned this pull request Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants