Skip to content
This repository has been archived by the owner on Jun 4, 2024. It is now read-only.

Update terraform provider to use new APIs for roles, users and auth connectors #974

Closed
wants to merge 2 commits into from
Closed

Update terraform provider to use new APIs for roles, users and auth connectors #974

wants to merge 2 commits into from

Conversation

rosstimothy
Copy link
Contributor

@rosstimothy rosstimothy commented Nov 28, 2023 •
edited
Loading

  • Use Upsert, with an arity of 2 as the UpdateMethod
  • Use Create instead of Upsert as the CreateMethod
  • Remove GetWithNoContext

@rosstimothy rosstimothy changed the title Update terraform provider to use new APIs for roles, users and auth c… Update terraform provider to use new APIs for roles, users and auth connectors Nov 28, 2023
@rosstimothy rosstimothy force-pushed the tross/use_upsert branch 3 times, most recently from 5cb4c29 to f5e4257 Compare December 4, 2023 18:39
r0mant and others added 2 commits December 4, 2023 13:40
@r0mant @rosstimothy
Release 14.2.1 (#976)
f3b30e7
@strideynet @marcoandredinis @rosstimothy
Access Plugins: Support dynamic credential reloading (#956)
daf2503 
* Modify msteams/email plugins to support reloading credentials

* Start adjusting event-handler for identity file refresh

* Fix event handler config names for refresh

* Fix test for event handler config parsing

* Adjust go.mod to depend on main repo branch

* Ignore returned user from CreateUser

* SPAG

Co-authored-by: Marco André Dinis <[email protected]>

* Adjust to refer to v14 backport branch

* Fix go imports

* License header!!

---------

Co-authored-by: Marco André Dinis <[email protected]>
@rosstimothy rosstimothy closed this Dec 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@klizhentas klizhentas Awaiting requested review from klizhentas klizhentas will be requested when the pull request is marked ready for review klizhentas is a code owner

@russjones russjones Awaiting requested review from russjones russjones will be requested when the pull request is marked ready for review russjones is a code owner

@r0mant r0mant Awaiting requested review from r0mant r0mant will be requested when the pull request is marked ready for review r0mant is a code owner

@Joerger Joerger Awaiting requested review from Joerger Joerger will be requested when the pull request is marked ready for review Joerger is a code owner

@tcsc tcsc Awaiting requested review from tcsc tcsc will be requested when the pull request is marked ready for review tcsc is a code owner

@zmb3 zmb3 Awaiting requested review from zmb3 zmb3 will be requested when the pull request is marked ready for review zmb3 is a code owner

@camscale camscale Awaiting requested review from camscale camscale will be requested when the pull request is marked ready for review camscale is a code owner

@fheinecke fheinecke Awaiting requested review from fheinecke fheinecke will be requested when the pull request is marked ready for review fheinecke is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rosstimothy @r0mant @strideynet