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

azurerm_data_factory_pipeline - migrate to hashicorp/go-azure-sdk #28768

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

stephybun
Copy link
Member

@stephybun stephybun commented Feb 13, 2025

Community Note

  • Please vote on this PR by adding a 👍 reaction to the original PR to help the community and maintainers prioritize for review
  • Please do not leave comments along the lines of "+1", "me too" or "any updates", they generate extra noise for PR followers and do not help prioritize for review

Description

  • Moved the expand/flatten function for variables and parameters into the pipeline resource since that is the only place they're used
  • Added a complete test with all resource properties specified and multiple activity types in the activities_json
  • Update the update test to update all updateable properties as well as update back to the basic config
  • Removed the custom serialize and deserialize functions for the activities_json which were replaced by the Unmarshaler for discriminated types in the go-azure-sdk
  • Removed the azuresdkhacks folder and contents since this has been replaced by go-azure-sdk

Note: I opted to not also split the Create and Update methods here, since this is already quite a big change to the resource

Testing

image
image

Change Log

Below please provide what should go into the changelog (if anything) conforming to the Changelog Format documented here.

  • dependencies: azurerm_data_factory_pipelines update to use hashicorp/go-azure-sdk [GH-00000]
  • azurerm_data_factory_pipelines - fix issue where a web activity with a header in activities_json could not be unmarshaled [GH-00000]

This is a (please select all that apply):

  • Bug Fix
  • New Feature (ie adding a service, resource, or data source)
  • Enhancement
  • Breaking Change

Related Issue(s)

Fixes #24998

Note

If this PR changes meaningfully during the course of review please update the title and description as required.

catriona-m
catriona-m previously approved these changes Feb 13, 2025
Copy link
Member

@catriona-m catriona-m left a comment

Choose a reason for hiding this comment

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

LGTM!

continue
}

output[k] = (*v.DefaultValue).(string)
Copy link
Member

Choose a reason for hiding this comment

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

could pointer.From be used here?

Copy link
Member Author

Choose a reason for hiding this comment

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

It can! Thanks for the spot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants