-
Notifications
You must be signed in to change notification settings - Fork 421
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
docs(home): update layer version to 62 for package version 2.33.1 #3778
Conversation
Thanks a lot for your first contribution! Please check out our contributing guidelines and don't hesitate to ask whatever you need. |
|
thank you so much @ericbn - something indeed happened with our automation. For the first time, it only updated parts of it. |
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. |
Awesome work, congrats on your first merged pull request and thank you for helping improve everyone's experience! |
Docs rebuilt with the new Layer version. Thank you so much again @ericbn 🫶 ![]() ![]() |
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? |
Docs only. As layers were published, the JSON artefact upload partially at
random. The docs update rely on that JSON
…On Fri, 16 Feb 2024 at 10:09, Andrea Amorosi ***@***.***> wrote:
Thanks for the heads up @heitorlessa <https://github.com/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?
—
Reply to this email directly, view it on GitHub
<#3778 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAZPQBGFNC5CLXYCZS7PPR3YT4O45AVCNFSM6AAAAABDLC3SN6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBYGAYDMNBXGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Issue number: #3780
Summary
Changes
User experience
Checklist
If your change doesn't seem to apply, please leave them unchecked.
Is this a breaking change?
RFC issue number:
Checklist:
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.