Harri Avellan

Thanks for the comments @bradrydzewski, I carried out the changes you suggested and rebased on master.

@bradrydzewski , any ideas to improve this PR, or is it off the scope for the project, and should I close it and try to tackle this issue with another...

We noticed the same behavior when updating from 0.3.3 to 0.5.2. Apparently AWSSecrets without versionId cause the secret update process silently to "fail" here https://github.com/mumoshu/aws-secret-operator/blob/main/controllers/awssecret_controller.go#L146:L182 VersionId is required given the...

More than having to limit the number of metrics collected, I wish we could see AWS Container Insights metrics become AWS provided metrics free of charge. The current pricing is...

As these custom metrics are created by AWS documentation _AND_ utilized by AWS Container Insights Dashboard, these metrics should not be under "custom metrics", but under AWS metrics, and they...

How about this being controlled by a feature flag? Current situation is that the security policy requires custom workarounds to achieve this, which leads to very hackish solutions.

Great that you're working on this issue allowing proper lifecycle management of GA+AWS LB integration. Have you thought should e.g. `status.loadBalancer.ingress` contain the GA IPs instead of the LB hostname?...