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

arm-builder: Always deploy arm-builders to eun #105

Merged

Conversation

henrirosten
Copy link
Collaborator

@henrirosten henrirosten commented Mar 21, 2024

Hard-code the arm builder location to 'northeurope' due to limited support of arm-based VMs in many other Azure regions. Specifically, ghaf-infra deployment on UAE would fail without this change, since Azure UAE locations do not support arm-based VMs. The workaround done in this change is to always deploy the arm-builder on 'northeurope', even if other ghaf-infra resources are deployed on locations other than eun.

For reference: this PR implements the workaround proposed in: #81 (review)

@henrirosten henrirosten force-pushed the always-deploy-arm-builders-to-eun branch from 964ab1e to c594226 Compare March 21, 2024 13:34
Hard-code the arm builder location to 'northeurope' due to limited
support of arm-based VMs in many other Azure regions. Specifically,
ghaf-infra deployment on UAE would fail without this change,
since Azure UAE locations do not support arm-based VMs. The workaround
done in this change is to deploy the arm-builder on 'northeurope', even
if other ghaf-infra resources are deployed on locations other than eun.

Signed-off-by: Henri Rosten <henri.rosten@unikie.com>
@henrirosten henrirosten force-pushed the always-deploy-arm-builders-to-eun branch from c594226 to 6e64e98 Compare March 21, 2024 13:36
@henrirosten henrirosten marked this pull request as ready for review March 21, 2024 13:59
@henrirosten henrirosten requested a review from a team March 21, 2024 13:59
@henrirosten henrirosten merged commit cde89e9 into tiiuae:main Mar 25, 2024
2 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.

1 participant