Repository navigation
fix: update CI Docker install and cleanup for new runner image - #236
Open
santhosh-apphelix-2u wants to merge 3 commits into
Open
santhosh-apphelix-2u wants to merge 3 commits into
santhosh-apphelix-2u wants to merge 3 commits into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
CI has failed on every PR and scheduled run on master since 2026-10-22.
ubuntu-latestmoved to a newer Ubuntu image (reported as 26.04), and these workflows were written for Ubuntu 20.04.Changes
Both workflow files (
provisioning-tests.yml,cli-tests.yml):apt-key add(removed from the runner image) with a keyring file in/etc/apt/keyringsandsigned-by=./etc/apt/sources.list.d/docker.listusing the runner's codename andstable, instead offocal test.add-apt-repositoryrejected the old line.provisioning-tests.ymlonly:powershellfrom the disk-cleanupapt removelist. It is no longer an apt package, andapt removeaborted on it.Testing
This PR's own CI run: 11 of 12 jobs pass (the Linux CLI job and 10 of the 11 provisioning jobs). The Mac CLI job also passes.
designer+lmsfails; see below.Known unrelated failure:
designer+lmsThe container crashes at startup with
PermissionError: [Errno 13] Permission denied: '/var/tmp/edx.log'. In theedxops/designer-devimage that file is owned by root, while the container runs as uid 1000. It reproduces on all available image tags. This PR only unmasked it; it is not caused by this change. Reviewers have agreed this failure can be ignored for this PR.Possible follow-ups
azure-cliin the cleanup list could be the next package to disappear from the image.