Sidharth Jawale
Sidharth Jawale
Yes. There might be some breaking changes with this update, however, with latest Go, there's a high chance code's performance will improve.
Hi team, I'll be picking up where @shubham-cmyk left off to address the recent changes. I'll update the PR shortly with the necessary adjustments. Thanks for your patience, and I’ll...
@eyenx, I’ve taken care of the rebase and chart version bump. @shubham-cmyk is on vacation, so please review the changes and inform me if any additional updates are needed.
The requested changes have been pushed. Kindly review and merge this feature. Thanks for your time, @prudhvigodithi @eyenx @bbarani @DandyDeveloper @peterzhuamazon @gaiksaya @TheAlgo
Thanks @TheAlgo! The lint failures, and changelog version comparison is fixed!
Thanks @peterzhuamazon! The fix has been pushed and the minor versions have been bumped up, as per the change request.
Thanks @peterzhuamazon, Yes, I'll raise another PR with this change for branch `1.x` shortly.
Hi @peterzhuamazon, Here's the PR for backporting this change to `1.x` : https://github.com/opensearch-project/helm-charts/pull/578
Yes, the official context source code: https://cs.opensource.google/go/go/+/refs/tags/go1.24.0:src/context/context.go;drc=f966695ccea356e4e4e8cc0328276e2d00c9fc1e;l=162
Hi @prabhatsharma, I noticed that in the current Helm chart, the Dex configuration requires storing sensitive values (like client secrets) in a ConfigMap, which isn’t ideal from a security perspective....