Skip to content

CI-5852 Add build for ubuntu 24#2702

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

CI-5852 Add build for ubuntu 24#2702
uaqq wants to merge 5 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)

  • remove ubuntu mirrors to use internal mirror via --add-host arg;
  • 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 Outdated
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

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.

fixed

Comment thread ci/Dockerfile.ubuntu Outdated
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?

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.

fixed

@uaqq uaqq force-pushed the CI-5852 branch 4 times, most recently from 67818f9 to adaa034 Compare July 1, 2026 11:06
@uaqq uaqq requested a review from Stolb27 July 1, 2026 11:59
Comment thread ci/Dockerfile.ubuntu Outdated
Comment thread ci/Dockerfile.ubuntu Outdated
Comment thread ci/Dockerfile.ubuntu Outdated
Co-authored-by: Maxim Gajdaj <maxim.gajdaj@gmail.com>
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.

3 participants