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

feat: [PL-58641]: add oidc support for aws sm #1155

Merged
merged 11 commits into from
Feb 5, 2025
Merged

Conversation

GokulBansal0
Copy link
Contributor

@GokulBansal0 GokulBansal0 commented Jan 23, 2025

Title: Added OIDC Support for AWS SM

Summary:
Provide a brief overview of what the PR does and why it is needed.

Details:
Added the changes in resource schema, datasource, tests and docs

Related Issues:
Reference any related issues or tickets

Testing Instructions:
Describe how the changes were tested. Include any test cases or steps to verify the functionality.
Please include the below test scenario with your changes.

  • Create the resource and execute terraform apply. (Verify the resource is created)
  • Execute terraform apply again without any changes. (Verify no changes should be done)
  • Update the resource and execute terraform apply. (Verify the resource updated successfully)
  • Remove the content from resource file and execute terraform apply. (Verify the resource has been deleted)
  • Add again the resource and execute the terraform apply. (Verify the resource is created)
  • Verify the import the resource file is working fine.
  • In case of remote entity, Verify for both default and non default branch.

Provider Tests

go test -timeout 500s -run '^(TestAccResourceConnectorAwsSM_inherit|TestProjectResourceConnectorAwsSM_inherit|TestOrgResourceConnectorAwsSM_inherit|TestAccResourceConnectorAwsSM_manual|TestAccResourceConnectorAwsSM_manualWithUsePutSecretTrue|TestProjectResourceConnectorAwsSM_manual|TestOrgResourceConnectorAwsSM_manual|TestAccResourceConnectorAwsSM_assumerole|TestProjectResourceConnectorAwsSM_assumerole|TestOrgResourceConnectorAwsSM_assumerole|TestAccResourceConnectorAwsSM_oidc_platform|TestOrgResourceConnectorAwsSM_oidc_platform|TestProjectResourceConnectorAwsSM_oidc_platform|TestAccResourceConnectorAwsSM_oidc_delegate|TestOrgResourceConnectorAwsSM_oidc_delegate|TestProjectResourceConnectorAwsSM_oidc_delegate)$' github.com/harness/terraform-provider-harness/internal/service/platform/connector/secretManagers
ok      github.com/harness/terraform-provider-harness/internal/service/platform/connector/secretManagers        95.734s

DataSource Tests

 go test -timeout 500s -run '^(TestAccDataSourceConnectorAwsSM_inherit|TestProjectDataSourceConnectorAwsSM_inherit|TestOrgDataSourceConnectorAwsSM_inherit|TestAccDataSourceConnectorAwsSM_manual|TestAccDataSourceConnectorAwsSM_manualWithUsePutSecretTrue|TestProjectDataSourceConnectorAwsSM_manual|TestOrgDataSourceConnectorAwsSM_manual|TestAccDataSourceConnectorAwsSM_assumerole|TestProjectDataSourceConnectorAwsSM_assumerole|TestOrgDataSourceConnectorAwsSM_assumerole|TestAccDataSourceConnectorAwsSM_oidc_platform|TestOrgDataSourceConnectorAwsSM_oidc_platform|TestProjectDataSourceConnectorAwsSM_oidc_platform|TestAccDataSourceConnectorAwsSM_oidc_delegate|TestOrgDataSourceConnectorAwsSM_oidc_delegate|TestProjectDataSourceConnectorAwsSM_oidc_delegate)$' github.com/harness/terraform-provider-harness/internal/service/platform/connector/secretManagers
ok      github.com/harness/terraform-provider-harness/internal/service/platform/connector/secretManagers        58.025s

Screenshots:
Include before and after screenshots to visually demonstrate the changes.
Checklist:

  • Code changes are well-documented.
  • Tests have been added/updated.
  • Changes have been tested locally.
  • Documentation has been updated.
PR Check triggers
  • Build: trigger build
  • Sub Category Field Check: trigger subcategoryfieldcheck
  • gitleaks: trigger gitleaks

@GokulBansal0 GokulBansal0 marked this pull request as ready for review January 28, 2025 06:59
@GokulBansal0 GokulBansal0 merged commit 1023322 into main Feb 5, 2025
3 checks passed
@GokulBansal0 GokulBansal0 deleted the PL-58641_aws_sm branch February 5, 2025 12:29
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.

2 participants