-
Notifications
You must be signed in to change notification settings - Fork 34
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 protobuf support for resource identity messages and rpcs #472
Conversation
012f578
to
f1f8392
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
bytes type = 2; | ||
bool required_for_import = 3; | ||
bool optional_for_import = 4; | ||
string description = 5; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI -> I asked core if we needed an additional StringKind description_kind = 6;
here, but that shouldn't stop this PR/others from merging and moving forward!
cc60b58
to
d2a6210
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you rebased, you'll need to rebuild the proto files 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoops, didn't mean to approve 😆
Yeah, I'm on it.. But my local environment is silly and I can't seem to update my version of |
…r ApplyResourceChange.Request
d2a6210
to
4c092e5
Compare
First part of https://hashicorp.atlassian.net/browse/TFECO-8804
This PR adds new PRC calls and messages for resource identity