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

FFM-9085: Add a terraform resource to create a new target #660

Merged
merged 9 commits into from
Aug 30, 2023

Conversation

ribeirophillipe
Copy link
Contributor

Describe your changes

  • Implement CRUD actions for creating a new target in terraform
  • Add all the interfaces and implementations

Comment Triggers

PR Check triggers
  • Build: trigger build
  • Sub Category Field Check: trigger subcategoryfieldcheck

Copy link
Contributor

@rohitk-h rohitk-h left a comment

Choose a reason for hiding this comment

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

@ribeirophillipe
Copy link
Contributor Author

@rohitk-h unit tests were added!

Copy link
Collaborator

@rathodmeetsatish rathodmeetsatish left a comment

Choose a reason for hiding this comment

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

Remove makefile changes, tf_build.sh

@ribeirophillipe
Copy link
Contributor Author

@rathodmeetsatish done

davejohnston
davejohnston previously approved these changes Aug 29, 2023
Copy link

@davejohnston davejohnston left a comment

Choose a reason for hiding this comment

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

This looks good. Ship it

Copy link
Collaborator

@rathodmeetsatish rathodmeetsatish left a comment

Choose a reason for hiding this comment

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

generate docs and add changelog file

internal/service/platform/user/resource_user.go Outdated Show resolved Hide resolved
internal/service/cd/delegate/delegate_helper_test.go Outdated Show resolved Hide resolved
* Implement CRUD actions for creating a new target in terraform
* Add all the interfaces and implementations
* Add unit test
* Fix package path
* Add build script
* Fix code base
* Add release target
* Include documentation and examples
.changelog/670.txt Outdated Show resolved Hide resolved
examples/data-sources/harness_platform_slo/data-source.tf Outdated Show resolved Hide resolved
examples/resources/harness_platform_policyset/resource.tf Outdated Show resolved Hide resolved
examples/resources/harness_platform_slo/resource.tf Outdated Show resolved Hide resolved
internal/service/cd/delegate/delegate_helper_test.go Outdated Show resolved Hide resolved
davejohnston
davejohnston previously approved these changes Aug 30, 2023
Copy link

@davejohnston davejohnston left a comment

Choose a reason for hiding this comment

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

Looks good

@ribeirophillipe ribeirophillipe merged commit 28ba6a7 into main Aug 30, 2023
3 checks passed
@ribeirophillipe ribeirophillipe deleted the FFM-9085 branch August 30, 2023 14:13
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.

None yet

4 participants