← Back to issue list

chore!: remove remote-build service override

View original Github issue

Metadata

Project
snapcraft
Number
#6010
Type
pull request
State
merged
Author
mr-cal
Labels
PR: Run Manual Spread
Created
Updated
Closed

Current evaluation

Merged cleanup removing the remote-build service override and manual spread tests. This follow-up addresses overlooked work from PR #5994. Approved by two reviewers, modifying seven files.

Suggested action: —

No scores available.

Issue body

This addresses some overlooked work in #5994. I forgot to remove the RemoteBuild service and manual spread tests. (SNAPCRAFT-1197) --- - [x] I've followed the [contribution guidelines](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md). - [x] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/). - [ ] I've successfully run `make lint && make test`. - [ ] I've added or updated any relevant documentation. - [ ] I've updated the relevant release notes.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged cleanup removing the remote-build service override and manual spread tests. This follow-up addresses overlooked work from PR #5994. Approved by two reviewers, modifying seven files.
qwen3.6-35b-a3b-mtp-q6 — — Merged cleanup removing the RemoteBuild service override and manual spread tests. Completes follow-up work from PR #5994 and resolves SNAPCRAFT-1197.
qwen3.6-35b-a3b-mtp-q6 — — Merged cleanup PR removing the RemoteBuild service override and associated manual spread tests, finalizing follow-up work from #5994.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#5994 chore!: remove legacy remote builder snapcraft merged Merged changes removing the legacy remote builder, resolving issue #5676. Approved by three reviewers and passing CI checks. The update deletes 3,200 lines across 30 files to clean up deprecated build infrastructure.
82%
#4383 chore: rename remote-build test snapcraft merged Merged a code cleanup that renamed the test module to match the source file remote.py. Approved by two reviewers, passed CI checks, and successfully integrated into the main branch.
79%
#2682 chore: delete some dead code charmcraft open Removes 19 lines of dead code in one file, a follow-up to #2681 (remote-build project name fix). Approved by a maintainer, CI mostly green with a few failing/pending snap tests.
78%
#636 chore(ci): move base devel spread tests to manual rockcraft merged Merged CI changes repurposing the large spread test run into a manual cron job. This isolates flaky base devel tests while preserving visibility via scheduled execution. Approved by two reviewers and applied across six files.
77%
#4634 chore: spread cleanup snapcraft merged Merged by maintainer sergiusens after three reviewers approved. The PR cleaned up spread configuration across 85 files, removing 358 lines and adding 115. All CI checks passed with 100% coverage on modified code.
77%
#392 chore: remove codeowners starbase merged Merged after approval from two reviewers and passing all CI checks. The change removes the CODEOWNERS file, deleting 9 lines from a single file as discussed in standup.
76%
#753 chore(services/testing): reduce spread verbosity level craft-application merged Merged changes that disable verbose mode for the spread test command to reduce output noise. Approved by reviewers, passed all CI checks, and integrated into the codebase.
76%
#667 chore!(models): remove BuildInfo and BuildPlanner craft-application merged Merged a breaking change removing the BuildInfo and BuildPlanner models across nine files, deleting 430 lines and adding 36. Approved by two reviewers and passed CI checks.
75%
#6035 chore!: remove deprecated commands snapcraft merged Merged after maintainer approval and a clean rebase. The changes remove deprecated commands and options, resolving issue #5674.
75%
#1776 chore: remove unused code charmcraft merged Merged after removing 17,571 lines of unused code across 36 files. Temporarily set to draft for additional cleanup, then approved by two reviewers and merged.
75%