Skip to content

[oryx] - Optimize feature installation - #1756

Merged
Abdurrahmaan Iqbal (abdurriq) merged 4 commits into
devcontainers:mainfrom
v-Kaniska244:oryx-cleanup-fix
Oct 26, 2026
Merged

Abdurrahmaan Iqbal (abdurriq) merged 4 commits into
devcontainers:mainfrom
v-Kaniska244:oryx-cleanup-fix

Conversation

@v-Kaniska244

@v-Kaniska244 Kaniska (v-Kaniska244) commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Optimize Oryx feature installation by eliminating a redundant build step and reducing the retained installation footprint while preserving dynamic SDK installation support.

Changes

  • Remove the full-solution Debug build and publish the Oryx CLI and BuildServer directly as self-contained Release applications.
  • Remove published debug symbols and Oryx XML documentation artifacts.
  • Retain only the scripts, metadata, and optional PHP prerequisites needed for dynamic SDK installation instead of copying entire source directories.
  • Add disk-usage logging before and after source cleanup.
  • Feature version patch bump from 2.0.1 to 2.0.2.

Test coverage

  • Add checks that required dynamic-install assets are retained and executable where appropriate.
  • Verify that the source checkout, build tools, image definitions, and published debug artifacts are removed.
  • Strengthen SDK installation assertions and clear installed platforms before testing universal-image behavior so subsequent checks exercise fresh installations.
  • Preserve coverage for pre-existing .NET installations.

@v-Kaniska244
Kaniska (v-Kaniska244) marked this pull request as ready for review October 26, 2026 08:32
@v-Kaniska244
Kaniska (v-Kaniska244) requested a review from a team as a code owner October 26, 2026 08:32
Comment thread src/oryx/devcontainer-feature.json
@abdurriq
Abdurrahmaan Iqbal (abdurriq) merged commit a7714e2 into devcontainers:main Oct 26, 2026
9 checks passed
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