Skip to content

CI-5852 Add build for ubuntu 24#2702

Open
uaqq wants to merge 3 commits into
adb-6.xfrom
CI-5852
Open

CI-5852 Add build for ubuntu 24#2702
uaqq wants to merge 3 commits into
adb-6.xfrom
CI-5852

Conversation

@uaqq

@uaqq uaqq commented Jun 18, 2026

Copy link
Copy Markdown

Add build for ubuntu 24 (#2702)

  • added ARG with OS_CODENAME for ubuntu mirrors;
  • use adb_python3_bin symlink for ubuntu 24;
  • install psutil only for ubuntu 22.

Task: CI-5852

@uaqq uaqq force-pushed the CI-5852 branch 11 times, most recently from 2bb431a to b100b85 Compare June 23, 2026 18:30
@uaqq uaqq changed the title CI-5852 Test CI-5852 Add build for ubuntu 24 Jun 23, 2026
Comment thread ci/Dockerfile.ubuntu
ARG UBUNTU_UPDATES_MIRROR=http://archive.ubuntu.com/ubuntu
ARG UBUNTU_BACKPORTS_MIRROR=http://archive.ubuntu.com/ubuntu
ARG UBUNTU_SECURITY_MIRROR=http://security.ubuntu.com/ubuntu
ARG OS_CODENAME=jammy

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Maybe remove this logic at all and use common approach with Greengage to change mirrors?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

The Greengage approach uses --add-host to redirect archive.ubuntu.com DNS to azure.archive.ubuntu.com's IP. This works because Azure's Ubuntu mirror serves packages at the same standard paths that apt expects:

http://<azure_mirror_ip>/ubuntu/dists/jammy/Release -> 200 OK
Our internal mirror uses a non-standard path structure:

/ubuntu/jammy/jammy/dists/jammy/Release <- actual path on our internal mirror.
/ubuntu/dists/jammy/Release <- what apt requests after --add-host redirect -> 404
--add-host only replaces the IP address at the DNS level - it does not rewrite URL paths. So after the redirect, apt still requests the standard paths, which return 404 on our mirror.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@deart2k, @magf, please discuss this question as was mentioned on the meet

Comment thread ci/Dockerfile.ubuntu
apt-get install -y ./*.deb; \
rm ./*.deb; \
else \
mkdir -p "$(dirname $adb_python3_bin)" && ln -sf "$(which python3)" $adb_python3_bin; \

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Maybe pass system python as build-arg for simplicity?

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