Skip to content

Conversation

estolfo
Copy link
Contributor

@estolfo estolfo commented Mar 9, 2022

af-south-1 doesn't support compatible-architectures.
We encountered this error before so removed it in #112
But then we erroneously added it back in #141
We then get this error when trying to publish the ARNs CompatibleArchitectures are not supported in af-south-1.

This PR removes the compatible-architectures option again.

@estolfo estolfo requested a review from v1v March 9, 2022 12:56
@github-actions github-actions bot added the aws-λ-extension AWS Lambda Extension label Mar 9, 2022
@ghost
Copy link

ghost commented Mar 9, 2022

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-03-09T12:57:06.140+0000

  • Duration: 6 min 11 sec

Test stats 🧪

Test Results
Failed 0
Passed 120
Skipped 6
Total 126

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@estolfo estolfo merged commit 9a4f111 into main Mar 9, 2022
@estolfo estolfo deleted the estolfo/compatible-arch branch March 9, 2022 13:17
@trentm
Copy link
Member

trentm commented Mar 9, 2022

Ah, sorry for causing that breakage.

So I understand: We could make it so we set --compatible-architectures ... in regions that do support it, right? Not suggesting we need to do that now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aws-λ-extension AWS Lambda Extension
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants