terraform-provider-elasticstack icon indicating copy to clipboard operation
terraform-provider-elasticstack copied to clipboard

Add role mapping resource

Open ottramst opened this issue 3 years ago • 5 comments
trafficstars

Trying to add an initial implementation of the role mapping resource. Pretty basic implementation where the user can specify role_templates and rules as a HEREDOC or jsonencode.

Go easy on me - first time implementing something in a Terraform provider and not so good at golang, yet :)

One issue which I can't figure out - during make docker-testacc, the test errors with role_mapping_test.go:18: Step 1/2 error: Check failed: Check 1/4 error: elasticstack_elasticsearch_security_role_mapping.test: Attribute 'name' expected "m16rozeuju", got "", although everything should be correctly implemented. If someone can help me figure this out, I'll happily fix this.

ottramst avatar Feb 14 '22 11:02 ottramst

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

elasticmachine avatar Feb 14 '22 11:02 elasticmachine

❌ Author of the following commits did not sign a Contributor Agreement: 120d2c3cf3449603a9042e20073dadf837ca3f46, ce7301274f65c8a13acce9c25e4103f16c8b7226, d2f76cfd9a1ccdbc328308ebccd65bd45478ad82, e13b999271e46d0e7961f195a7d2dc47445c8eb0, 3149e577dd72a2d13154d1e7d21409fa102e4c42, cea7ea3a6df30e0ab100daad718591df43edbe60, dd4bfde29ef00b69cf380756593ecef8999c6c64, , b539bd2441fd37dbc1036c6cff2437ba05f51c0e, b4a8094c6a33383485634a913ff9dccc4fb9f91b, , a89a043952ed713d6998afa2295980f0bbb76f63, 4fc0f2b4f391629519f60b478e591a3caa07ea94

Please, read and sign the above mentioned agreement if you want to contribute to this project

Any progress on this? @ottramst @olksdr

Andyandpandy avatar May 04 '22 16:05 Andyandpandy

Just gonna do another poke here, since it's over 2 months ago I last poked. I'm very keen on getting this as a feature. @ottramst @olksdr

Andyandpandy avatar Jul 08 '22 07:07 Andyandpandy

Any news on this? @olksdr Seems that validation review is missing. @ottramst ?

fludo avatar Aug 29 '22 12:08 fludo

Hey, guys!

Sorry for non-responsiveness on this issue. Unfortunately, because of my busy schedule, I can not continue with this pull request. Anyone is more than welcome to take over from me, but I just can't find the time for this right now.

Again, sorry..

ottramst avatar Sep 01 '22 21:09 ottramst