onelogin / terraform-provider-onelogin

GNU General Public License v3.0
27 stars 19 forks source link

Early preview for delegated admin support? #46

Closed marcusvoss closed 3 years ago

marcusvoss commented 3 years ago

Hi,

as you have announced the delegated admin feature to be in preview, I was wondering if you have a "preview" version of the terraform provider available as well?

Thanks Marcus

Links as references https://onelogin.service-now.com/support?id=kb_article&sys_id=128c53b2db1f2050ca1c400e0b961989&kb_category=20880e84db738300d5505eea4b9619fb https://developers.onelogin.com/api-docs/1/privileges/list-privileges

dcaponi commented 3 years ago

Hey @marcusvoss we don't have a preview version of this functionality at this time. I'm happy to update here once its implemented.

dcaponi commented 3 years ago

Sorry, clicked the wrong button 😅

dcaponi commented 3 years ago

@marcusvoss if you're feeling adventurous and want to have a play around, I have this working on a branch https://github.com/onelogin/terraform-provider-onelogin/pull/53

It should work if you look at the example and follow it.

~I need to add a few guard rails since there are some rules you need to follow that arent immediately apparent unless you have the docs handy and I want to bake some of those in to the error checking for a nicer experience. Once those are done, it should be fully released later this week.~

I have learned that validation functions are not supported by Terraform for list type fields. This is as polished as we can make it right now. I'll be releasing a new version now.

dcaponi commented 3 years ago

@marcusvoss I'm closing this since the new version shipped with support for this 2wk ago and I haven't heard back since. If there's an issue with it, please open a new issue with details about the support you need.