Skip to content

docs(home): update layer version to 62 for package version 2.33.1 #3778

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

Merged
merged 1 commit into from
Feb 16, 2024

Conversation

ericbn
Copy link
Contributor

@ericbn ericbn commented Feb 15, 2024

Issue number: #3780

Summary

Changes

Update layer version to 62 for package version 2.33.1. Not sure if this is an automatic process that was not triggered on the latest release, but the docs still mention the layer version 61 which has the package version 2.33.0 installed.

User experience

Users will have the correct reference to layer version 62 for when they want to use the latest 2.33.1 version of aws-lambda-powertools.

Checklist

If your change doesn't seem to apply, please leave them unchecked.

Is this a breaking change?

RFC issue number:

Checklist:

  • Migration process documented
  • Implement warnings (if it can live side by side)

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

@ericbn ericbn requested a review from a team February 15, 2024 22:57
@boring-cyborg boring-cyborg bot added the documentation Improvements or additions to documentation label Feb 15, 2024
Copy link

boring-cyborg bot commented Feb 15, 2024

Thanks a lot for your first contribution! Please check out our contributing guidelines and don't hesitate to ask whatever you need.
In the meantime, check out the #python channel on our Powertools for AWS Lambda Discord: Invite link

@pull-request-size pull-request-size bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Feb 15, 2024
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@heitorlessa
Copy link
Contributor

thank you so much @ericbn - something indeed happened with our automation. For the first time, it only updated parts of it.

cc @rubenfonseca

@heitorlessa
Copy link
Contributor

issue created #3780 - discovered the root cause.. it was the new GitHub Action upload-artifact major version which had a race condition in the last release, which led to the collection of Layer ARN be partial hence the PR reflecting parts with 62 version only.

cc @dreamorosi @am29d @rubenfonseca @sthulb so you're aware of the risk for next release until we use dynamic artifact + merge.

@heitorlessa heitorlessa changed the title chore: update layer version to 62 for package version 2.33.1 docs(home): update layer version to 62 for package version 2.33.1 Feb 16, 2024
@heitorlessa heitorlessa merged commit cc69e54 into aws-powertools:develop Feb 16, 2024
Copy link

boring-cyborg bot commented Feb 16, 2024

Awesome work, congrats on your first merged pull request and thank you for helping improve everyone's experience!

@heitorlessa
Copy link
Contributor

heitorlessa commented Feb 16, 2024

Docs rebuilt with the new Layer version. Thank you so much again @ericbn 🫶

image image

@dreamorosi
Copy link
Contributor

Thanks for the heads up @heitorlessa! Just to make sure I understand the risk: was the issue only confined to the ARNs in the docs, or was the layer publishing also affected?

@heitorlessa
Copy link
Contributor

heitorlessa commented Feb 16, 2024 via email

@ericbn ericbn deleted the update-layer-version branch February 16, 2024 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants