google.cloud icon indicating copy to clipboard operation
google.cloud copied to clipboard

Adding support for Google Secret Manager for issue 543

Open dcostakos opened this issue 2 years ago • 10 comments

SUMMARY

Add support for Google Secret Manager as both a module and lookup plugin. Fixes #543

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

gcp_secret_manager

ADDITIONAL INFORMATION

Basic support for GCP secret manager with some testing


dcostakos avatar Jun 21 '23 23:06 dcostakos

Thank you so much for reviewing this and trying the merge. You caught me at like the worst time on this, but I'm getting back to resolving the issues you brought up and the lint issues with the merge today/tomorrow.

Cheers, I'll make an updated commit soon.

dcostakos avatar Jul 13 '23 20:07 dcostakos

Thank you so much for reviewing this and trying the merge. You caught me at like the worst time on this, but I'm getting back to resolving the issues you brought up and the lint issues with the merge today/tomorrow.

Cheers, I'll make an updated commit soon.

sounds great! and no worries - apologies about my belated review here.

Just ping me when you're ready again - I'm trying to be a little more reactive to PR reviews.

toumorokoshi avatar Jul 14 '23 15:07 toumorokoshi

@toumorokoshi Just checking in on this. I'm sure you have a million things to do, but just wanted to check with you to see if this was still something that could potentially be merged. Thank you for your time on this.

dcostakos avatar Jul 31 '23 16:07 dcostakos

Hey just checking back on this. Again, I'm sure you're very busy. I also have no idea what your release plans are for this. Curious if this will be merged?

dcostakos avatar Nov 02 '23 15:11 dcostakos

Hi! Unfortunately I've left Google and am no longer the official repository maintainer.

That said: it looks like I still have permissions to merge. @SirGitsalot are you able to take a look? or do you want me to see if I can review and get this in?

toumorokoshi avatar Nov 07 '23 19:11 toumorokoshi

it's a real bummer that this got missed for another release. Shoudl we just close this issue if it won't be worked on?

dcostakos avatar Dec 20 '23 17:12 dcostakos

I'll ping @SirGitsalot again - as much as I'd like to help merge this in since I'm not an employee I think it's best if we leave it to the official maintainer.

toumorokoshi avatar Jan 03 '24 17:01 toumorokoshi

Thanks @toumorokoshi, for helping even though you've moved on. Can you assign @SirGitsalot to be the reviewer of this instead of you?

dcostakos avatar Jan 03 '24 18:01 dcostakos

Thanks @toumorokoshi, for helping even though you've moved on. Can you assign @SirGitsalot to be the reviewer of this instead of you?

done!

toumorokoshi avatar Jan 04 '24 15:01 toumorokoshi

It works like a charm, you made my day, thank you 🙇

kuisathaverat avatar Jan 29 '24 14:01 kuisathaverat

@SirGitsalot Hi guys! Why hasn't this been merged yet? It was created almost a year ago. I need this feature.

AlexAkulov avatar May 06 '24 07:05 AlexAkulov

I'm not sure if this will ever be merged. I'm the author, I know there's feedback on this, but I'm not open to making more changes to something that will never be merged. Not sure what Google's strategy on this collection is any longer. Feel free to try to manually merge my fork into a collection if you really need it.

dcostakos avatar May 07 '24 15:05 dcostakos

Hi, @SirGitsalot will that Secret manager going to be put into a release anytime soon by any chance? Our team would love to use it officially.

martiniDB avatar Jul 15 '24 16:07 martiniDB

Hi @SirGitsalot, repeating the request from @martiniDB. Any timeline for this module to be released?

azameeracn avatar Aug 01 '24 14:08 azameeracn

I haven't tried it out yet, but it looks like it got published in https://github.com/ansible-collections/google.cloud/releases/tag/v1.4.0 and docs are on Galaxy here! Thanks @SirGitsalot!

runofthemill avatar Sep 13 '24 23:09 runofthemill