Skip to content
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 support for azure resources integration TF resource #395

Closed
wants to merge 7 commits into from

Conversation

taimoorgit
Copy link
Contributor

@taimoorgit taimoorgit commented Jul 8, 2024

Issues

https://github.com/OpsLevel/team-platform/issues/414

OpsLevel/opslevel-go#430

Changelog

  • Add resource
  • Make a changie entry

@taimoorgit taimoorgit self-assigned this Jul 8, 2024
stringplanmodifier.RequiresReplace(),
},
},
"last_synced_at": schema.StringAttribute{
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this isn't a field on the AWS integration - are we sure it'll be in the public API?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rocktavious this will be in the public schema and this is for the Azure Resources integration only.

Copy link
Collaborator

@rocktavious rocktavious left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few comments/questions and a few fixes needed

@@ -0,0 +1,3 @@
kind: Feature
body: Add support for Azure Resource Integration TF resource
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
body: Add support for Azure Resource Integration TF resource
body: Add support for configuring the OpsLevel Azure Integration

LastSyncedAt types.String `tfsdk:"last_synced_at"`
Name types.String `tfsdk:"name"`
SubscriptionId types.String `tfsdk:"subscription_id"`
TenantId types.String `tfsdk:"tenant_id"`
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we also added ownershipTagKeys and tagsOverrideOwnership to the mutation and integration type

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good call out. This has been added in the upstream opslevel-go PR and I have not yet updated here

@davidbloss davidbloss mentioned this pull request Aug 1, 2024
2 tasks
@davidbloss
Copy link
Contributor

Completed by this PR

@davidbloss davidbloss closed this Aug 1, 2024
@taimoorgit taimoorgit deleted the ta/azure-integration branch August 7, 2024 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants