Skip to content

Conversation

@ramchale
Copy link
Contributor

@ramchale ramchale commented Oct 2, 2025

No description provided.

@ramchale
Copy link
Contributor Author

ramchale commented Oct 2, 2025

@microsoft-github-policy-service agree

@ramchale
Copy link
Contributor Author

ramchale commented Oct 2, 2025

Hi
I mainly raised this to get some feedback from the CI pipeline about where the issues are with this upgrade, but it appears that the CI run isn't kicking off

@jahnvi480
Copy link
Contributor

@ramchale Thanks for raising the PR for this fix, I would like you to please resolve the conflicts in this PR since now we will also be supporting PHP8.4 there are few conflicting changes
Please fix the issue post which I can review and run the Pipeline on your branch.

Comment on lines +58 to +63
RUN apt-get install -y python3.12-venv
RUN python3 -m venv py-venv && \
. py-venv/bin/activate && \
python3 -m pip install --upgrade pip && \
python3 -m pip install --upgrade requests && \
python3 -m pip install cpp-coveralls
Copy link
Contributor Author

Choose a reason for hiding this comment

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

After the Ubuntu upgrade the original version of this fails with error: externally-managed-environment. Using a virtual env lets the docker build pass, but there's a few options so another approach might be preferable

@ramchale
Copy link
Contributor Author

Please fix the issue post which I can review and run the Pipeline on your branch.

Hi @jahnvi480 . I've re-based that now, so hopefully it's a bit closer to being usable

@jahnvi480
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ramchale ramchale changed the title Update docker image to PHP 8.4 and latest Ubuntu LTS Update docker image to Ubuntu 24.04 LTS Jan 16, 2026
@ramchale ramchale marked this pull request as ready for review January 16, 2026 13:54
@jahnvi480 jahnvi480 merged commit 609073c into microsoft:dev Jan 16, 2026
5 checks passed
@jahnvi480
Copy link
Contributor

Merging your PR.
Thank you for contributing to Mircosoft peoject!

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