-
Notifications
You must be signed in to change notification settings - Fork 1
Add meshstack_location resource and local acceptance tests as PoC
#81
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
base: main
Are you sure you want to change the base?
Conversation
aacb9bd to
e7d1012
Compare
b9cbcf4 to
f09fcc8
Compare
f09fcc8 to
23cd354
Compare
…via .env file later setting TF_ACC=1)
23cd354 to
336d1f1
Compare
| Ref: struct { | ||
| Kind string `tfsdk:"kind"` | ||
| Name string `tfsdk:"name"` | ||
| }{ |
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.
f: make this an explicit type instead of inlining it multiple times
| "kind": schema.StringAttribute{ | ||
| MarkdownDescription: "meshObject type, always `meshLocation`.", | ||
| Computed: true, | ||
| Default: stringdefault.StaticString("meshLocation"), | ||
| PlanModifiers: []planmodifier.String{stringplanmodifier.UseStateForUnknown()}, | ||
| }, |
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.
f: in order for this to plug cleanly into a platform resource location_ref.kind must be made optional there since it's currently always being set automatically
Needs (internal) PR https://github.com/meshcloud/meshfed-release/pull/9088
See single commits in this PR.