Skip to content

Conversation

@Kaniska244
Copy link
Contributor

@Kaniska244 Kaniska244 commented Apr 21, 2025

Ref# #1192

Description: The change is mainly done to to extend installation support of anaconda feature to arm64 architecture.

Changelog: The following files files are changed.

  • src/anaconda/install.sh - Change is done to extend support for arm64 architecture, upgrade the anaconda installation package version, fixing permissions & upgrading python packages for which older, vulnerable packages were installed. Also further changes made to extend this feature to support RHEL based linux distributions.
  • src/anaconda/devcontainer-feature.json - Version bump.
  • test/anaconda/install_anaconda_bookworm.sh - Added test script for anaconda installation on debian bookworm.
  • test/anaconda/install_anaconda_bullseye.sh - Added test script for anaconda installation on debian bullseye.
  • test/anaconda/install_anaconda_jammy.sh - Added test script for anaconda installation on ubuntu jammy.
  • test/anaconda/install_anaconda_noble.sh - Added test script for anaconda installation on ubuntu noble.
  • test/anaconda/install_anaconda_noble_without_user.sh - Added test script for anaconda installation on ubuntu noble without user in the input.
  • test/anaconda/scenarios.json - Added for all test cases.

Checklist:

  • Checked that applied changes work as expected.

@Mathiyarasy Mathiyarasy self-requested a review July 18, 2025 07:09
@Kaniska244 Kaniska244 marked this pull request as ready for review July 18, 2025 07:53
@Kaniska244 Kaniska244 requested a review from a team as a code owner July 18, 2025 07:53
@Kaniska244 Kaniska244 requested a review from Copilot July 25, 2025 05:24

This comment was marked as outdated.

Kaniska244 and others added 6 commits July 25, 2025 11:06
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…copilot

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@Kaniska244 Kaniska244 requested a review from Copilot July 25, 2025 06:33

This comment was marked as outdated.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@Kaniska244 Kaniska244 requested a review from Copilot July 25, 2025 07:17
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR extends the Anaconda feature to support ARM64/aarch64 architecture and RHEL-based Linux distributions. The main purpose is to broaden the platform compatibility of the devcontainer feature beyond the previously supported x86_64 Debian/Ubuntu systems.

Key Changes

  • Added ARM64/aarch64 architecture support to the Anaconda installer
  • Extended Linux distribution support to include RHEL-based systems (AlmaLinux, RockyLinux, Fedora)
  • Updated Anaconda version from 2021.11 to 2024.10-1 and added security package updates

Reviewed Changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/anaconda/install.sh Core installation script enhanced with multi-architecture and multi-distro support, package manager detection, and security updates
src/anaconda/devcontainer-feature.json Version bump from 1.0.13 to 1.0.14
src/anaconda/README.md Updated documentation to reflect new OS support
test/anaconda/scenarios.json Added test scenarios for various Linux distributions
test/anaconda/*.sh Added test scripts for different distributions and architectures

Kaniska244 and others added 2 commits July 25, 2025 12:53
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@Kaniska244
Copy link
Contributor Author

Hello @AlvaroRausell ,

Thank you for the detailed review. I have done the corrections. Would you kindly have another look and let me know in case of any further corrections.

With Regards,
Kaniska

@Kaniska244 Kaniska244 requested a review from AlvaroRausell July 31, 2025 06:19
@AlvaroRausell AlvaroRausell merged commit 5fda443 into devcontainers:main Aug 12, 2025
12 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.

3 participants