Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NAS-132446 / 24.10.1 / Initial add of option to use internal APT repo for builds (by kmoore134) #757

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

bugclerk
Copy link
Contributor

@bugclerk bugclerk commented Nov 14, 2024

Added an option to set APT_INTERNAL_BUILD in the environment, which will toggle the builder to use the CDN Apt, or the iX Internal Apt Repo.

Additionally, the environment variable APT_BASE_CUSTOM can be set to a URL if you wish to override both options to pull from another location at build time.

Original PR: #754
Jira URL: https://ixsystems.atlassian.net/browse/NAS-132446

…or builds (#754)

* Initial add of option to use internal APT repo for builds

* Pet flake8

* Wrap a long f-string

* Use f-string in right place

* Fix usage of single / double quotes

* Move to apt_get_base_url as a function

* Undo dyslexia

* Log which base url we are using in bootstrap for debugging

* Add missing location to use the base url in bootstrap

* Use base url when bootstrapping the ISO as well

* Add APT_BASE_CUSTOM as another override option if the builder
wants to pull from some location other than the two in our manifest

(cherry picked from commit 5f95982)
@yocalebo yocalebo marked this pull request as ready for review November 14, 2024 15:28
@yocalebo yocalebo merged commit 8d5104c into stable/electriceel Nov 14, 2024
3 of 4 checks passed
@yocalebo yocalebo deleted the NAS-132446-24.10.1 branch November 14, 2024 15:28
@bugclerk
Copy link
Contributor Author

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@truenas truenas locked as resolved and limited conversation to collaborators Nov 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants