Skip to content

feat: LTS Anaconda WebUI ISOs [pending https://github.com/ublue-os/titanoboa/pull/118] - #2942

Merged
castrojo merged 24 commits into
ublue-os:mainfrom
hanthor:webui
Aug 7, 2025
Merged

feat: LTS Anaconda WebUI ISOs [pending https://github.com/ublue-os/titanoboa/pull/118]#2942
castrojo merged 24 commits into
ublue-os:mainfrom
hanthor:webui

Conversation

@hanthor

@hanthor hanthor commented Aug 6, 2025

Copy link
Copy Markdown
Collaborator

This is dependent on ublue-os/titanoboa#118 landing. We can drop the builder_distro thing in Titanoboa but it will mean no secureboot in the LiveOS for LTS. there might be a way to hack it by curling the CentOS shim and putting it in the right place

@hanthor hanthor changed the title feat: LTS Anaconda WebUI ISOs feat: LTS Anaconda WebUI ISOs [pending https://github.com/ublue-os/titanoboa/pull/118] Aug 7, 2025
Comment thread .github/workflows/reusable-build-iso-anaconda.yml Outdated
flatpaks-list: ${{ github.workspace }}/iso_files/system-flatpaks.list
hook-post-rootfs: ${{ matrix.image_version == 'lts' && format('{0}/iso_files/configure_lts_iso_anaconda.sh', github.workspace) || format('{0}/iso_files/configure_iso_anaconda.sh', github.workspace) }}
kargs: ${{ steps.image_ref.outputs.kargs }}
builder-distro: ${{ matrix.image_version == 'lts' && 'centos' || 'fedora' }}

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This is the important bit

@@ -0,0 +1,188 @@
#!/usr/bin/env bash

@hanthor hanthor Aug 7, 2025

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Made an entirely different script for LTS, too many differences. May need the custom sbkey for GDX/HWE once we add akmods to those builds

@hanthor
hanthor marked this pull request as ready for review August 7, 2025 10:28
Copilot AI review requested due to automatic review settings August 7, 2025 10:28
@dosubot dosubot Bot added the size/L This PR changes 100-499 lines, ignoring generated files. label Aug 7, 2025
@hanthor
hanthor removed request for castrojo and p5 August 7, 2025 10:28
@hanthor
hanthor removed the request for review from Copilot August 7, 2025 10:28
@dosubot dosubot Bot added the area/iso Making installable images label Aug 7, 2025

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 adds support for LTS (Long Term Support) Anaconda WebUI ISOs, introducing a new configuration script specifically for LTS variants and modifying the build workflow to handle LTS images differently. The changes enable LTS builds to use CentOS-based components while maintaining WebUI functionality.

Key changes:

  • Introduces dedicated LTS Anaconda configuration script with CentOS-specific settings
  • Updates main Anaconda configuration to conditionally enable WebUI for LTS builds
  • Switches from filesystem-based storage cleanup to container-based storage optimization

Reviewed Changes

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

File Description
iso_files/configure_lts_iso_anaconda.sh New LTS-specific configuration script with CentOS bootloader settings and XFS filesystem
iso_files/configure_iso_anaconda.sh Updated to enable WebUI for LTS builds and switch from BTRFS to XFS
.github/workflows/reusable-build-iso.yml Replaced filesystem cleanup with container storage optimization
.github/workflows/reusable-build-iso-anaconda.yml Added conditional LTS configuration and CentOS builder distro support

Comment thread iso_files/configure_lts_iso_anaconda.sh Outdated
Comment thread .github/workflows/reusable-build-iso-anaconda.yml Outdated
hanthor and others added 6 commits August 7, 2025 16:26
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@castrojo

castrojo commented Aug 7, 2025

Copy link
Copy Markdown
Collaborator

Ok dependent PR landed, ok to merge?

@hanthor

hanthor commented Aug 7, 2025

Copy link
Copy Markdown
Collaborator Author

Ok dependent PR landed, ok to merge?

Need to make ARM64 centos builder images

Made them

@hanthor
hanthor enabled auto-merge August 7, 2025 15:55
@hanthor

hanthor commented Aug 7, 2025

Copy link
Copy Markdown
Collaborator Author

@m2Giles any input?

@hanthor
hanthor requested a review from castrojo August 7, 2025 20:16
@hanthor

hanthor commented Aug 7, 2025

Copy link
Copy Markdown
Collaborator Author

Ok dependent PR landed, ok to merge?

Ready!

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Aug 7, 2025
@castrojo
castrojo disabled auto-merge August 7, 2025 21:12
@castrojo
castrojo merged commit df8b8d2 into ublue-os:main Aug 7, 2025
23 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/iso Making installable images lgtm This PR has been approved by a maintainer size/L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants