Skip to content

adding powerplatform_connection and powerplatform_connection_share #377

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

Merged
merged 24 commits into from
Aug 1, 2024

Conversation

mawasile
Copy link
Contributor

This pull request includes significant changes to the Power Platform provider to add support for managing connections and connection shares. The changes include new data sources for fetching connection shares and connections, new resources for creating and managing connections and connection shares, and updates to the provider and testing code to support these new features. The changes are primarily focused on the powerplatform package and its sub-packages.

New Data Sources:

New Resources:

  • docs/resources/connection.md: Added a new resource to create and manage connections. This includes details on how to use the resource and its schema.
  • docs/resources/connection_share.md: Added a new resource to create and manage connection shares. This includes details on how to use the resource and its schema.

Updates to Provider:

Updates to Testing Code:

Examples:

Miscellaneous:

  • go.mod: Updated the github.com/google/uuid package to be a direct dependency.
  • docs/resources/rest.md: Fixed some typos in the documentation.

mawasile added 13 commits May 15, 2024 06:42
commit 1b8c806
Author: mawasile <[email protected]>
Date:   Wed May 15 01:37:57 2024 +0200

    resource/datasource primary id should not changes on reapply (#292)

    * chore: making unique resource/datasource id stable & fixing unit tests

    * chore: fixing acceptance tests

    * chore: update display_name in powerplatform environment resources

    * adding new acceptance tests for dlp policies

    ---------

    Co-authored-by: Eduardo Sanchez <[email protected]>

commit 6142aaf
Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Date:   Tue May 14 17:21:38 2024 -0600

    Bump hashicorp/local in /examples/resources/powerplatform_solution (#298)

    Bumps [hashicorp/local](https://github.com/hashicorp/terraform-provider-local) from 2.4.0 to 2.5.1.
    - [Release notes](https://github.com/hashicorp/terraform-provider-local/releases)
    - [Changelog](https://github.com/hashicorp/terraform-provider-local/blob/main/CHANGELOG.md)
    - [Commits](hashicorp/terraform-provider-local@v2.4.0...v2.5.1)

    ---
    updated-dependencies:
    - dependency-name: hashicorp/local
      dependency-type: direct:production
      update-type: version-update:semver-minor
    ...

    Signed-off-by: dependabot[bot] <[email protected]>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

commit 54a1173
Author: Matt Dotson <[email protected]>
Date:   Mon May 13 23:46:54 2024 -0700

    Addition of example directories for dependabot scanning (#296)

    * manual addition of example directories

    * fix duplicate path

commit b6b0eec
Author: Eduardo Sanchez <[email protected]>
Date:   Mon May 13 15:30:55 2024 -0600

    Updating Go version and modify tfproviderlint actions to use specified go version (#284)

    * Updating Go version.

    * Disable tfproviderlint.

    * Testing with go vet.

    * Changing run order.

    * Placing all runs inside one job.

    * testing with:         /usr/bin/tfproviderlint ./...

    * changing path for tfproviderlint.

    * testing setup go.

    * Show directory path.

    * Applying comments and updating version.

    * Updating devcontainer version.

    * Disabling tfproviderlint.

    ---------

    Co-authored-by: Eduardo Sanchez <[email protected]>
    Co-authored-by: mawasile <[email protected]>

commit 236a1bf
Author: mawasile <[email protected]>
Date:   Mon May 13 20:02:26 2024 +0200

    Updating developer guide with additional Information (#290)

    * chore: Update documentation for developer workflow and setup instructions

    * chore: Update developer documentation and add unit tests for new feature

    * update docs

    * chore: Update developer documentation, add unit tests, and regenerate docs

    * Update DEVELOPER.md

    Co-authored-by: Matt Dotson <[email protected]>

    ---------

    Co-authored-by: Matt Dotson <[email protected]>

commit ff470f7
Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Date:   Mon May 13 11:25:35 2024 -0600

    Bump goreleaser/goreleaser-action from 5.0.0 to 5.1.0 (#293)

    Bumps [goreleaser/goreleaser-action](https://github.com/goreleaser/goreleaser-action) from 5.0.0 to 5.1.0.
    - [Release notes](https://github.com/goreleaser/goreleaser-action/releases)
    - [Commits](goreleaser/goreleaser-action@7ec5c2b...5742e2a)

    ---
    updated-dependencies:
    - dependency-name: goreleaser/goreleaser-action
      dependency-type: direct:production
      update-type: version-update:semver-minor
    ...

    Signed-off-by: dependabot[bot] <[email protected]>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

commit dce39d9
Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Date:   Mon May 13 11:25:04 2024 -0600

    Bump goreleaser/goreleaser-action in /.github/workflows (#294)

    Bumps [goreleaser/goreleaser-action](https://github.com/goreleaser/goreleaser-action) from 5.0.0 to 5.1.0.
    - [Release notes](https://github.com/goreleaser/goreleaser-action/releases)
    - [Commits](goreleaser/goreleaser-action@7ec5c2b...5742e2a)

    ---
    updated-dependencies:
    - dependency-name: goreleaser/goreleaser-action
      dependency-type: direct:production
      update-type: version-update:semver-minor
    ...

    Signed-off-by: dependabot[bot] <[email protected]>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

commit 0db25c3
Author: Raphael Pothin <[email protected]>
Date:   Fri May 10 11:12:33 2024 -0400

    Fix proposition for the inversion between "Business" and "Non-business" connectors groups in Power Platform DLP policies (#291)
The code changes add a new output for the Azure OpenAI connection in the `powerplatform_connection` module. This output provides the description, value, and sensitivity of the connection.
@mawasile mawasile linked an issue Jul 26, 2024 that may be closed by this pull request
10 tasks
@mawasile mawasile changed the title [DRAFT] adding powerplatform_connection and powerplatform_connection_share adding powerplatform_connection and powerplatform_connection_share Jul 30, 2024
@mawasile mawasile marked this pull request as ready for review July 30, 2024 16:22
@mawasile mawasile added this to the Preview 2 milestone Jul 30, 2024
@mawasile mawasile added resource terraform resource data source data source labels Jul 30, 2024
eduardodfmex
eduardodfmex previously approved these changes Jul 30, 2024
Copy link
Contributor

@ianjensenisme ianjensenisme left a comment

Choose a reason for hiding this comment

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

Partial review, I'll try to run it tonight and add more notes. But so far the only "for sure" change request is a typo file rename.

@ianjensenisme
Copy link
Contributor

@mawasile I tried to test this locally and in a codespace, and got a mix of different errors in each place, probably related to configuration and Azure security. make test failed with symlink errors, which is probably a container problem. However I think I was able to successfully run the connections datasource example locally, and it looks like it missed the one OAI connection I have in the environment I was testing against. Let's block time to walk through what I'm trying and figure out whether I'm misguided or there's a real issue.

@mawasile mawasile merged commit abc20e0 into main Aug 1, 2024
8 checks passed
@mawasile mawasile deleted the mawasile/283-resource-powerplatform_connection branch August 1, 2024 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data source data source resource terraform resource
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Resource: powerplatform_connection
4 participants