GZ
@wladyslawczyzewski can you fix the merge conflicts? I've approved the PR and will prioritize on merging this once you fixed the conflicts. Thank you!
Adding a `do-not-merge` and `needs-security-review` labels to wait on an update from security review on this approach.
Discussed with Kendra offline and she's fine to merge this. Removing `do-not-merge`.
Go 1.24.2 and 1.23.8 have these mitigations. This package does not have direct dependency with go modules, however, the packages we depend on `helm` and `kubectl` is currently using the...
> **Exemption Request** > > From the contribution guideline: > > > The only case where it is legitimate to break a public API is if the existing API is...
@yasamoka Sorry that the tool automatically closed the PR. I re-opened it and added an action item internally to review the close PR policy and I don't think it's valid...
> > Maybe I missed it, but I only saw the validation and definition of the property `throughput`, but I don't see where you assign it to `CfnVolume`? > >...
@badmintoncryer Apologize for the delay. I'm having some discussions around this PR with my coworkers. Will get back to you once we reach an conclusion. For now I'll mark this...
@badmintoncryer sorry that I haven't gotten back to you on this yet. Please wait a bit longer as we've still finalizing some proof of concepts.
@dviryamin do you need any help with the failed integ test?