-
Notifications
You must be signed in to change notification settings - Fork 43
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
chore: [CHAOS-6992]: Added support of chaos infrastructure #1138
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Saranya-jena <[email protected]>
identifier = "identifier" | ||
org_id = "org_id" | ||
project_id = "project_id" | ||
environment_id= "env_id" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good to fix the indentation here
org_id = "org_id" | ||
project_id = "project_id" | ||
environment_id= "env_id" | ||
namespace= "namespace" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
indentation fix
--- | ||
# generated by https://github.com/hashicorp/terraform-plugin-docs | ||
page_title: "harness_chaos_infrastructure Data Source - terraform-provider-harness" | ||
subcategory: "" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good to have Next Gen
as subcategory
|
||
```shell | ||
# Import using the Harness application id and environment id. | ||
terraform import harness_environment.dev <application_id>/<environment_id> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
terraform import harness_environment.dev <application_id>/<environment_id> | |
terraform import harness_chaos_infrastructure.example <chaos_infra_id> |
name = "name" | ||
org_id = "org_id" | ||
project_id = "project_id" | ||
environment_id= "env_id" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
indentation fixes at all places wherever required
Description: "Resource for creating a Chaos Infrastructure.", | ||
|
||
ReadContext: resourceInfrastructureRead, | ||
UpdateContext: resourceChaosInfraUpdate, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we pls follow the same naming convention for all the methods
Title: [Component/Feature] Short Description of the Change
Summary:
Provide a brief overview of what the PR does and why it is needed.
Details:
Explain the changes in detail, including any relevant context or background information.
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.
Screenshots:
Include before and after screenshots to visually demonstrate the changes.
Checklist:
PR Check triggers
trigger build
trigger subcategoryfieldcheck
trigger gitleaks