-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add apikey resource #124
base: main
Are you sure you want to change the base?
add apikey resource #124
Conversation
4ca4685
to
797ce6f
Compare
TY for the contribution! |
542a791
to
d25d46f
Compare
done! Also updated the docs. Will update the test with SA once that PR is merged/landed |
d25d46f
to
683e629
Compare
@kwadhwa-openai did anything change with the search attributes resource or test in this PR? For some reason those tests are now failing. Everything else passes though.
|
It is failing on |
What was changed
Adding apikey CRUD support
Why?
API keys are currently managed using tcld. Adding support with terraform plugin will make it easy to maintain it in code
Checklist
Closes [Feature Request] Add API key resource support for CRUD #123
How was this tested:
make testacc
using real service-account-idAny docs updates needed?
generated the docs