Skip to content

Maintenance: compatibleArchitectures field not available in all AWS Regions #1400

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

Closed
1 of 2 tasks
dreamorosi opened this issue Apr 7, 2023 · 9 comments · Fixed by #3544
Closed
1 of 2 tasks

Maintenance: compatibleArchitectures field not available in all AWS Regions #1400

dreamorosi opened this issue Apr 7, 2023 · 9 comments · Fixed by #3544
Assignees
Labels
automation This item relates to automation completed This item is complete and has been merged/shipped internal PRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.) layers Items related to the Lambda Layers pipeline

Comments

@dreamorosi
Copy link
Contributor

dreamorosi commented Apr 7, 2023

Summary

In #1372 we have introduced two new fields to our public layers: compatibleArchitectures and license. This new field however caused the release process to fail due to one of the fields not being accepted in some regions.

To proceed with the release I have applied an hot-fix, however we should dedicate time to understand what are the implication of this field not being supported.

Why is this needed?

Because the compatibleArchitectures is not supported in some AWS regions (eu-south-2, eu-central-2, ap-southeast-4) and we are unsure of what are the implications.

Which area does this relate to?

Automation

Solution

No response

Acknowledgment

Future readers

Please react with 👍 and your use case to help us understand customer demand.

@dreamorosi dreamorosi added automation This item relates to automation internal PRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.) confirmed The scope is clear, ready for implementation labels Apr 7, 2023
@dreamorosi
Copy link
Contributor Author

As the commits linked above show, I ended up temporarily commenting out the parameter in the layer stack.

Let's discuss options in the coming weeks.

@am29d
Copy link
Contributor

am29d commented Apr 18, 2023

Python folks had similar problems with my recent changes here. They used aspects to traverse the nodes and fix the compatibleArchitectures field based on the region.

I will apply same fix to our layer package.

@dreamorosi
Copy link
Contributor Author

Aside from the immediate fix, do we have a reliable way of keeping track of when the field will be available? Or do we have to keep our eyes peeled on the what's new channel to see when it gets announced?

@am29d
Copy link
Contributor

am29d commented Jul 28, 2023

We will revisit the issue once we start working on #1374. I will block it for now.

@am29d am29d added the on-hold This item is on-hold and will be revisited in the future label Jul 28, 2023
@dreamorosi dreamorosi moved this from Working on it to On hold in Powertools for AWS Lambda (TypeScript) Jul 28, 2023
@am29d am29d added the layers Items related to the Lambda Layers pipeline label Aug 17, 2023
@dreamorosi
Copy link
Contributor Author

The Python team has worked around this limitation by modifying the region matrix from this:

matrix:
  region:
    [
      "af-south-1",
      "eu-central-1"
    ]

to this:

matrix:
  include:
    - region: "af-south-1"
       has_arm64_support: "true"
    - region: "ap-east-1"
       has_arm64_support: "true"

With the new has_arm64_support key passed as a parameter like this:

npx cdk deploy --app cdk.out --context region=${{ matrix.region }} --parameters HasARM64Support=${{ matrix.has_arm64_support }} ...

Then, in the CDK application, the parameter is converted to a set of CfnConditions which are in turn used to modify the construct in charge of publishing the layer using Aspects. See here for example.

We should look into adopting the same solution so we can forget about the issue for good.

@dreamorosi dreamorosi removed the on-hold This item is on-hold and will be revisited in the future label Feb 21, 2024
@dreamorosi dreamorosi moved this from On hold to Backlog in Powertools for AWS Lambda (TypeScript) Feb 21, 2024
@leandrodamascena
Copy link
Contributor

You can close it now. All the regions supports arm64.

@dreamorosi
Copy link
Contributor Author

Thank you for the info!

I opened a PR (#3544) to add back the field, which will close the PR once merged.

@dreamorosi dreamorosi moved this from Working on it to Pending review in Powertools for AWS Lambda (TypeScript) Jan 29, 2025
@github-project-automation github-project-automation bot moved this from Pending review to Coming soon in Powertools for AWS Lambda (TypeScript) Jan 29, 2025
Copy link
Contributor

⚠️ COMMENT VISIBILITY WARNING ⚠️

This issue is now closed. Please be mindful that future comments are hard for our team to see.

If you need more assistance, please either tag a team member or open a new issue that references this one.

If you wish to keep having a conversation with other community members under this issue feel free to do so.

@github-actions github-actions bot added pending-release This item has been merged and will be released soon and removed confirmed The scope is clear, ready for implementation labels Jan 29, 2025
@dreamorosi dreamorosi moved this from Coming soon to Shipped in Powertools for AWS Lambda (TypeScript) Jan 29, 2025
Copy link
Contributor

This is now released under v2.14.0 version!

@github-actions github-actions bot added completed This item is complete and has been merged/shipped and removed pending-release This item has been merged and will be released soon labels Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation This item relates to automation completed This item is complete and has been merged/shipped internal PRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.) layers Items related to the Lambda Layers pipeline
Projects
Development

Successfully merging a pull request may close this issue.

3 participants